New command structure

As @noplanman suggest in a comment commands structure can move to this configuration:
Copied form @noplanman commet:
In my mind (as far as I've understood the different commands so far), there should be 3 different kinds of commands:
Executed by the library / internal system
Executed by the Admins
Executed by the Users (& Admins obviously)
This way, we could separate these different commands more easily, as they have their own specific way of being executed. I see at the moment that many commands do pretty much the same thing, but each one uses lots of identical code, which really should be optimised.
Yes I saw that @ public vs. private. I think public should be the ones all Users can execute and private the ones only Admins can.
I haven't looked deeper into this, but maybe adding an extra layer could help out a lot:
-> Command
  |-> AdminCommand
    |-> SendToAllAdminCommand
  |-> UserCommand
    |-> WhoAmIUserCommand
  |-> SystemCommand
    |-> NewChatTitleSystemCommand

(my opinion)
In this way we can remove the public var ad show them according to the division

System commands are not shown in the /help.
User commands are visible to user in /help
Admin can see both user and admin commands in /help

Lets start the discussion!

What I've been thinking about in terms of Commands, is how a user can totally define his/her own ones, removing the existing ones the library brings along.
The issue is, that the Commands directory is always included and can't be ignored.
Let me illustrate with an example:
I want to create a very specific bot that only supports 3 commands, say /yesterday, /today and /tomorrow.
Ok, I create a new directory with those commands in them and add it as a commands directory, fine.
Also, I want to provide an InlineQuery for a special /now command, which returns a list of items selected from the DB.
How would I do this with the current library without touching any library code?

I think that we should implement a method that let you enable/disable certain commands during the setup. Then when you include a new dir of commands you can overwrite a present one if they have the same name

It would be really cool to be able to implement something like the hooks that WordPress uses.
Hooks allow a plugin or theme developer to "hook into" core functionality very very easily.
Given the example here, there would be a hook to set the command directories. This way a user could easily "hook into" the Telegram class and set / unset any custom command directories. The same method could be applied to enable / disable a list of commands.
About simply overriding existing commands, that's how it works at the moment already! :smiley:
(see getCommandClass)
All command related methods need some tweaking to simplify it all and make sure that they all have the same data to work with. At the moment getCommandsList and getCommandClass do some similar things, but differently. It would make sense to have dedicated methods for finding out which commands are available, etc.

After thinking about this in more depth and looking at how the commands are currently handled, here are my current view on how I would change it. I'm obviously looking at implementing the most flexible solution, making it easy to override commands and add / remove command directories with ease.
New file structure (for the core library):
Commands
|-> Admin
  |-> [All admin commands]
|-> User
  |-> [All user commands]
|-> System
  |-> [All system commands]

All commands extend a specific class, namely:

AdminCommand or
UserCommand or
SystemCommand

All these commands can be overridden by the user, by adding custom command directories and defining them as the same command names. If an Admin and User command have the same name, it depends if the user is an admin or not. If yes, the Admin command is used, if not, the normal User one is used.
The default command directories can also be removed, allowing to disable all commands (#93)
When fetching the command list, it also depends if we're an admin or not. This is used by the /help command and to find out if a called command exists.

Admins get: Admin & User commands
Users get: User commands

Does this make sense so far? I'm sure more things will come up as I go along and explore.
Any feedback and suggestions are very welcome, thanks :smiley:

I took a look at the code! Great work!
The travis ci fail can be due to the difference in the class name and the file name:
https://github.com/noplanman/php-telegram-bot/blob/new_command_structure_alpha/src/Commands/AbstractCommand.php#L21
/start command should be considered a users commad or a sys one?

I haven't looked at making it fit the current tests yet, as some tests will probably need to change a bit anyway. Also, I'll be writing new tests for the new structure :smiley:
/start should be a system command, as it's the system that automatically sends it when the chat starts, thanks for pointing that out!
If somebody wants to add their own message, they can just override the class :blush:

I fix some autoload problems, now here it works properly for InlineQueryCommand:
https://github.com/MBoretto/php-telegram-bot/commit/4dc6c13e4fbb27d2daf9b007910e864d396201c0
We should decide if the abstract command have to stay in the same directory with other commands or not.
Personally i think that they should not stay there, if user need a particular configuration can create his how abstract class and his command can inherit from it.
Some tips for autoloading:

Namespace must have to be the same of the directory structure
Class Name and Filename must be the same

...if user need a particular configuration can create his how abstract class and his command can inherit from it.

I don't fully understand what you mean by this. The whole reason to separate the different command types is to enforce certain access and functionality, giving the user a simple way to add custom commands.
If I understand you correctly, users should be able to make their own abstract AdminCommand class that extends Command?
The idea is great and I'm totally for "freedom to the user", but I really don't see how that should work.
The bot API provides us with the 3 different command "types" (System, Admin, User), so the library needs to offer a way to extend those with custom commands. We do that by letting users extend our abstract command classes. Now if users can create their own abstract classes, they would need to extend our abstract ones so that we can still find out what type of command it is. This would allow users to extend down as far as they wanted to.
At the moment, with the way it works right now, this is not possible as we only allow 1 layer, so custom user commands must extend our existing abstract classes directly.
How do you suggest we implement this (users making their own abstract classes)?

Regarding the namespaces and the abstract class files in the command directories, I'm not sure if it makes sense to prefix all abstract classes with Abstract. It adds a lot more text to the files and is unnecesarry I think.
What we could do though, is restructure the files a bit:
We could put all abstract classes directly into the Commands folder. The problem with having the abstract class in the command subfolder itself, is that a command can never have the same name, e.g. a user command named /user can't exist. Same for an /admin admin command. It would make sense to keep the subdirectories "clean", only containing commands. As for the namespace, we would have the abstract classes directly under \Commands, which I think is ok, since they represent a "type" of command, not the command itself.
So we would have these abstract classes:
Commands\Command, Commands\SystemCommand, Commands\AdminCommand and Commands\UserCommand.
On top of that, we should really consider having a Commander class or something, that manages all command related functionality. At the moment the commands are very tightly coupled with the Telegram class.
A seperate class would make it easier to manage the various command related functionality, like adding command directories, finding out which commands are available, etc.
What are you thoughts?
(P.S. Sorry for the long message, hope it all kind of makes sense)

In my bot i created an extension of the command class and all the command Inherit from my custom class. This was useful because you can share function ecc.
So every one can do this we don't have to implement anything. I said this to support the removal of abstract class in commands folders that's all!
I would remove Abstract from of the class too.
Sounds good to put a commander or command bus class to menage everything about commands!

So you extend the abstract Command class directly? Thing is, that won't work any more, because with the new structure it must be a child of SystemCommand, AdminCommand or UserCommand.
Unless I haven't fully realised how you're doing it.
:+1: Ok, will remove the "Abstract" from all of them and move them up a folder.
:+1: Ok regarding the commander class. Will look into that once the command structure is working well.

I think that with this new model i will have to create MyCommadBasic extension of Systemcommand (for example) and create my own commad extension of MyCommadBasic.
Don't worry about this now, we can have an working alpha version of the new command structure, then we can tune this 'exotic' feature!

You're right, let's continue with the alpha and then take another view.

A few bigger changes have been made to the inner workings of the commands, here's a list to explain the main ones so far. As always, feel free to comment, ask question, discuss, etc. :smiley:
(For reference, I've added links (:mag:) to the commits where the main parts of these changes were introduced)
Command return value :mag:
execute() method of commands must now return a ServerResponse object (instead of a boolean), so that the response can now be validated and tested in unit tests.
Remember last executed command response :mag:
The response of the last command that was executed when processing the updates from Telegram (via handle() or handleGetUpdates()) is now saved in the Telegram object. This allows for any extra "post-execute" data handling if that is required.
Custom command overriding :mag:
When adding custom commands, they override any existing commands with the same name. If multiple custom command folders get added, if they contain the same command, the last one "wins". Let me try to explain this overriding process a bit more using the help command as an example.

No custom commands folder: The default help command is used.
Add a custom commands folder: The help command from the custom commands folder is used.
Add another custom commands folder: Now, the help command from the second custom commands folder is used, overriding the one from the first.

On top of this, Admin commands take preference over User commands. So if we have a help command which is an AdminCommand, that one will be used when an admin executes the command. Non-admin users still get the non-admin help UserCommand. This allows for certain commands to react differently for Admins.

@MBoretto @akalongman
Before I continue, I need some feedback from you guys to know if this is the direction we all want to be going or if some adjustments need to be made.
Also, before refactoring the Telegram and putting all command related things together into a separate class, I think the new command structure should be merged into develop, just to split this whole thing up in stages. What are your thoughts on that?

I checked the code and everything works fine, i also try custom commands in separate folder! Merged

The discussion on refactoring the Telegram class and introducing a Commander class can be discussed in #106
Closing here...


Telangana contains a data point called 'Unknown', containing '-430' active cases.

Describe the bug
Telangana contains a data point called 'Unknown', containing '-430' active cases.
To Reproduce
Steps to reproduce the behavior:

Go to covid19india.org.
Click on 'Telangana' in the states list.
7th in the list of places is 'Unknown' with '-430' active cases.

Expected behavior
The number of active cases shouldn't be negative.
Screenshots

Desktop (please complete the following information):

OS: Ubuntu 20.04
Browser: Mozilla Firefox
Version: 76.0.1

Smartphone (please complete the following information):

Device: Xiaomi Note 5 Pro
OS: MIUI <IP_ADDRESS>
Browser: Google Chrome
Version: 81.0.4044.138

Refer to https://github.com/covid19india/covid19india-react/issues/1846#issuecomment-626884611

@shuklaayush I still don't get it. What numbers are fixed in the equation, from the data that we receive?
Active unknown = Confirmed Unknown - Recovered Unknown - deceased unknown.

So what happens is that sometimes states do not disclose the districts of the patients who are recovering. For confirmed cases, the data is usually released. In this case, the Unkown confirmed becomes less than the Unknown recovered, and therefore the Unknown active is negative.
Hope that clears things up.

@shuklaayush I think I understand better now. But, I still have a few doubts. Does this mean that Hyderabad which seems to have around 728 cases, actually has less than that? I'm curious as to whether its associated to numbers in other districts.
Depending on the answer to the earlier questions, would it make sense to do something like this:
Confirmed unknown = Actual Confirmed unknown + Recovered Unknown + deceased unknown

Active unknown = Confirmed Unknown - Recovered Unknown - deceased unknown = Actual Confirmed unknown


Add resources page

Continuation of https://github.com/pulumi/docs/pull/4172. I ended up creating a new PR to clean it up and hopefully make this whole thing easier to review.
@cnunciato you left a comment on the original PR about using CSS more than JS and another comment about making the filters "linkable". So I killed two birds with one stone with this new approach of using hashes in the URL to create the functionality we are looking for. Interested in seeing what you think about this but this approach adds no new JavaScript so probably an overall better approach for a static site.
To address why this is named resources it is because this will eventually contain more than just webinars, events, and videos. Eventually, we will also have ebooks, whitepapers, podcasts, and other pieces of content listed here.
Old PR description:
Fixes: #3961
This PR combines event and webinar pages into a new /resources page. I've moved the code that generates the list items to be stencil.js components and reorganized the content sections. Unfortunately, this PR will break the CMS which for the next few weeks will mean I'll have to add/edit/remove things from resources. This will unblock the reminder of the Cloud Engineering Summit as once this is merged in, I'll add the recordings with transcripts as well as a new resource type for "Podcasts" which will use to list the Summit content as a Podcast on Spotify, Apple Podcasts, etc.

cc @susanev for design 👀 as well.

One little thing is there's a gap between the icon and the label; ideally hovering over either would highlight both. You could do this by wrapping both in an a tag.


can't  rename css selector

example:
.icon-sort {
background-image: url(sprite.png);
background-position: -493px -273px;
width: 24px;
height: 20px;
}
'icon-' prefix  me   Cannot modify  ?

This is a duplicate issue from #51 and #58. You can adjust the cssSelector via cssOpts.cssSelector. Here is an example:
cssOpts: {
  cssSelector: function (sprite) {
    // Use sprite name as its own selector (e.g. `sort` -> `.sort`)
    return '.' + sprite.name;
  }
}

thanks


trailing 8

PR #4 introduced a bug that ouputs a trailing 8. Unfortunate because I haven't had a chance to add a test suite yet, which surely would have caught it.
# php build/db-ping.phar mysql --host localhost
DB-PING localhost:3306
time=2017-01-19 22:20:16, from localhost:3306, connected., delay=2000ms, repeat=1x, exec=0ms, since success=0s, since fail=0s, 8
time=2017-01-19 22:20:16, from localhost:3306, check passed., delay=2000ms, repeat=1x, exec=0ms, since success=0s, since fail=0s, 8
time=2017-01-19 22:20:18, from localhost:3306, check passed., delay=2000ms, repeat=1x, exec=0ms, since success=2.0174s, since fail=0s, 8
time=2017-01-19 22:20:20, from localhost:3306, check passed., delay=2000ms, repeat=1x, exec=0ms, since success=4.0191s, since fail=0s, 8
time=2017-01-19 22:20:22, from localhost:3306, check passed., delay=2000ms, repeat=1x, exec=0ms, since success=6.0208s, since fail=0s, 8
time=2017-01-19 22:20:24, from localhost:3306, check passed., delay=2000ms, repeat=1x, exec=0ms, since success=8.0229s, since fail=0s, 8
time=2017-01-19 22:20:26, from localhost:3306, check passed., delay=2000ms, repeat=1x, exec=0ms, since success=10.0248s, since fail=0s, 8
time=2017-01-19 22:20:28, from localhost:3306, check passed., delay=2000ms, repeat=1x, exec=0ms, since success=12.0265s, since fail=0s, 8
time=2017-01-19 22:20:30, from localhost:3306, check passed., delay=2000ms, repeat=1x, exec=0ms, since success=14.0282s, since fail=0s, 8
time=2017-01-19 22:20:32, from localhost:3306, check passed., delay=2000ms, repeat=1x, exec=0ms, since success=16.0299s, since fail=0s, 8
time=2017-01-19 22:20:34, from localhost:3306, check passed., delay=2000ms, repeat=1x, exec=0ms, since success=18.0317s, since fail=0s, 8
time=2017-01-19 22:20:36, from localhost:3306, check passed., delay=2000ms, repeat=1x, exec=0ms, since success=20.0334s, since fail=0s, 8
time=2017-01-19 22:20:38, from localhost:3306, check passed., delay=2000ms, repeat=1x, exec=0ms, since success=22.0355s, since fail=0s, 8
time=2017-01-19 22:20:40, from localhost:3306, check passed., delay=2000ms, repeat=1x, exec=0ms, since success=24.0373s, since fail=0s, 8
-- localhost:3306 database ping statistics ---
13 tries, 13 successes, 0 failures, 0% fail tries
24.8725s time, 24.8725s success, 0s fail, 0% fail time

Notice the trailing , 8 on each line.
I think it has to do with your format processing code and the a flag, but I'll have to debug it when I have a chance later.

I think this is coming from here. I'll submit a fix


feat: 1:1 migration from LauncherSettings >>> Settings

Contains
This does not (yet) make use of any feature we gain by having JavaFX properties or bindings, but on the upside this should be a feature-equivalent migration to the new settings POJO.
I know that the migration is not pretty, but I'd like to improve on the details in separate follow-up PRs to keep the review load bearable.
Most changes where done with IntelliJ's "Type Migration" and direct migration of getter/setter calls to access of the property value.
How to test
Ensure that all tests are still passing when running gradlew test.
Start the launcher and play around with the settings.

changed settings should have an effect, e.g., whether to show pre-releases (NOTE: changing the locale only has an effect after restart. This is a known issues).
saved settings should be persisted and stay the same between restarts
no unexpected errors or crashes
both settings files (Java properties and JSON) are written to the launcher user directory (defaults to ~/.terasologylauncher/)

Based on #666
Contributes to #522

 Launcher improvements

The locale setting indeed brakes with this PR. The reason is that we no longer init() the LauncherSettings, and therefore don't call the initLocale() method anymore. The crux here is that this initLocale() method actually mutates global state in Languages  :worried: (I think it's the only setting that does this).
https://github.com/MovingBlocks/TerasologyLauncher/blob/33786f61fa0e304ef3968e9631abc270e91c050a/src/main/java/org/terasology/launcher/settings/LauncherSettings.java#L105-L115
I'm currently trying to figure out at which place I'll add the Languages.update(...) call. Probably right after we've loaded the settings in the init task... :thinking: This is still more of a workaround than a proper solution, though.

@jdrueckert The language issue should be fixed by 286c4d7. I'd appreciate another review :upside_down_face:

Changes look fine, but there seems to be a test failure?

There's indeed a failing test in the CI run:
TestRunGameTask > testTerminatedProcess() FAILED
    java.lang.AssertionError at TestRunGameTask.java:185
OpenJDK 64-Bit Server VM warning: Sharing is only supported for boot loader classes because bootstrap classpath has been appended

62 tests completed, 1 failed
QuantumRenderer: shutdown

FAILURE: Build failed with an exception.

However, tests are running locally for me :confused:
gradlew test
Starting a Gradle Daemon, 1 incompatible Daemon could not be reused, use --status for details

> Configure project :
Inferred project: TerasologyLauncher, version: 4.6.0-dev.13.uncommitted+feat.switch.to.new.settings.object.286c4d7

> Task :compileJava
Note: /home/skaldarnar/Code/movingblocks/terasologylauncher/src/main/java/org/terasology/launcher/repositories/GithubRepositoryAdapter.java uses or overrides a deprecated API.
Note: Recompile with -Xlint:deprecation for details.
Note: /home/skaldarnar/Code/movingblocks/terasologylauncher/src/main/java/org/terasology/launcher/util/Languages.java uses unchecked or unsafe operations.
Note: Recompile with -Xlint:unchecked for details.

> Task :test
WARNING: An illegal reflective access operation has occurred
WARNING: Illegal reflective access by org.spf4j.log.SLF4JBridgeHandler$3 (file:/home/skaldarnar/.gradle/caches/modules-2/files-2.1/org.spf4j/spf4j-slf4j-test/8.8.5/5b9cd6e3ac26428529cff69edbcb7a714cd84804/spf4j-slf4j-test-8.8.5.jar) to field java.util.logging.LogRecord.needToInferCaller
WARNING: Please consider reporting this to the maintainers of org.spf4j.log.SLF4JBridgeHandler$3
WARNING: Use --illegal-access=warn to enable warnings of further illegal reflective access operations
WARNING: All illegal access operations will be denied in a future release

BUILD SUCCESSFUL in 19s
15 actionable tasks: 4 executed, 11 up-to-date

I'll re-run the job...


Add codestyle xmls and  task checkstyle

Changes

exec embulk migrate with embulk 0.8
update gradle to 2.7
reformat code with airlift codestyle

airlift is default codestyle in embulk

LGTM

I've merged this PR and release it as v0.5.0. Thanks!


Omit all tile edges for polylines

Fixes polyline builder never checking line segments for coincidence with tile edges (not sure when this was broken)
Modifies tile edge check to match edges of any tile, not just the current tile

@tallytalwar give this a quick look if you please.

Sure on it. Thanks


want to save the simulation result into a file

Now the observatory uses the on-memory storage.
The on-file storage is already implemented but has no GUI to create it and attach it to an interpreter is provided.
We will also have to pack the storage into a zip file so that it can be easily exchanged over the net.

implemented in https://github.com/ReMobidyc/ReMobidyc/commit/3173e12b7db8c46fcec7df77c4ca22972ebfcb52 .


db snapshot

Signed-off-by: zhangjinpeng1987<EMAIL_ADDRESS>Add a wrapper for db snapshot and some tiny functions,  they will be used later.

LGTM


Find has unexpected results

Why does find of a specific ID, add a select all of type? This even happens if you do ChildModel::find(id).
ParentModel::find(<someId>)->toRawSql();  yields
select * from "parent_model" where "parent_model"."type" = '<child-type>'

It's including every single child model irrespective of their actual ids!
This works correctly, though:
ParentModel::query()->where('id', <some-id>)

NVM, this is actually a bug with ->toRawSql() and the actual results of the find seem correct.


Add setup details to prerequisites documentation section

It took me a couple of hours to set up the web app due to a missing instruction on updating the dnx to 1.0.0-beta8 after installing WebToolsExtensionsVS14.
Without it, it was showing reference errors to the new changes from Context to HttpContext and SignOut() to await SignOutAsAsync().
The following commands need to be run in package manager console to update dnx:

dnx use 1.0.0-beta8
dnu restore

I'm not sure I see a lot of value in us re-documenting the process for setting up aspnet5.  Perhaps a link to the asp.net docs for those who don't know where to find them.
https://docs.asp.net/en/latest/getting-started/installing-on-windows.html

Adding the two package console command above should be enough. The instructions were very straight forward besides this.
The above link could also suffice for the upgrading to Beta8 of the documentation as it has all the details.

Closing as suggestion received too few feedback.


chore(main): release 1.0.0

:robot: I have created a release beep boop
1.0.0 (2024-07-14)
Features

add client generate (75fa2ca)
add cs-fixer (c722971)
add dependably.yml (8cf305c)
add fixer config (a172b2f)
add initial generation of code (b5945db)
add linting (40a1f33)
add template cleanup (30f4f05)
template composer json (fc0d555)
template jane file (106aef8)

Bug Fixes

add name to workflow (#3) (c4d302e)
Add php workflow (bf839dd)
checkout before linting (0ae9b7d)
force release-please when dependabot'ing (9055e9a)
ignore composer lock (83e4797)
limit concurrency (1627ef3)
make security-defined a warning (ad14256)
re-set rules (5f74b3f)
remove composer lock (cd0876f)
remove composer lock (83c6d77)
rename spec file (7c51792)
replace correct elements (a9946c9)
safe-replace (1b0caac)
Set php version to 8.3 (5ffe6e6)
use redocly cli (05f9428)
use templatable names (c9143c1)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

v1.0.0
:sunflower:


High CPU Utilization in Istio-Ingressgateway

Bug description
Exponential CPU Utilization in Istio Proxy after 1.3.*.
When upgrading from Istio 1.2.5 to 1.3.0, I noticed a dramatic increase in CPU Utilization in the istio-ingressgateway.  I admittedly didn't look directly into it after the upgrade, just assumed there were some underlying changes that required a bit more CPU.
Then when deploying a new workload to the cluster, it failed due to unavailable CPU.  I then ran a top on the pods and noticed that the HPA had scaled the pods up to the 5 maximum and each reaching close to +1000m CPU rather than the 10-100m that we saw normally.
Following some advice from a post on discuss.istio.io, I downgraded the istio-proxyV2 image to 1.2.8 and saw the CPU utilization drop back to the norm of a single pod and less than 100m CPU.
I don't want to have to keep an old version of this image going forward with upgrades.  Is there any explanation of the increase?
Expected behavior
CPU Utilization remains the same or only slightly increases
Steps to reproduce the bug
Upgrade from 1.2* to 1.3.*
Version (include the output of istioctl version --remote and kubectl version)
istioctl version --remote
client version: 1.3.3
citadel version: 1.3.3
egressgateway version: 1.2.5
galley version: 1.3.3
ingressgateway version: 1.3.3
pilot version: 1.3.3
policy version: 1.3.3
sidecar-injector version: 1.3.3
telemetry version: 1.3.3

kubectl version
Client Version: version.Info{Major:"1", Minor:"14", GitVersion:"v1.14.6", GitCommit:"96fac5cd13a5dc064f7d9f4f23030a6aeface6cc", GitTreeState:"clean", BuildDate:"2019-08-19T11:13:49Z", GoVersion:"go1.12.9", Compiler:"gc", Platform:"darwin/amd64"}

Server Version: version.Info{Major:"1", Minor:"14", GitVersion:"v1.14.6", GitCommit:"96fac5cd13a5dc064f7d9f4f23030a6aeface6cc", GitTreeState:"clean", BuildDate:"2019-08-19T11:05:16Z", GoVersion:"go1.12.9", Compiler:"gc", Platform:"linux/amd64"}

How was Istio installed?
Helm (template)
Environment where bug was observed (cloud vendor, OS, etc)
Azure AKS

Likely related to https://github.com/istio/istio/issues/18229

Has this been resolved in newer versions? I believe this is fixed now per above issue

It has!  Thanks.


Editor crashes very often when running 4.1 (through steam)

Godot version
4.1
System information
Godot v4.1.stable - Windows 10.0.22621 - Vulkan (Mobile) - dedicated NVIDIA GeForce RTX 2070 (NVIDIA; <IP_ADDRESS>40) - 12th Gen Intel(R) Core(TM) i5-12600K (16 Threads)
Issue description
My editor crashes very often, be it when I save or when I run my game.
It seems to sometimes corrupt some files though i'm not sure about that causality.
I would say it started around the 4.1 update.
It is now to a point where the editor is near unusable.
Steps to reproduce
Usually after a few modifications the project crashes when I press play or save.
I tried making a smaller project but it wouldn't crash, so you are stuck with my original project
Minimal reproduction project
reproducible.zip

After testing, it seems to work fine on the standalone, so it must be a problem with the steam version.
Maybe related is the fact that the steam overlay doesn't go away no matter what I do

Are you in the same situation as this comment ?


Cannot disable openapi validation during helm upgrade

Output of helm version:
$ helm version version.BuildInfo{Version:"v3.1.2", GitCommit:"d878d4d45863e42fd5cff6743294a11d28a9abce", GitTreeState:"clean", GoVersion:"go1.13.8"}
Output of kubectl version:
$ kubectl version Client Version: version.Info{Major:"1", Minor:"14", GitVersion:"v1.14.1", GitCommit:"b7394102d6ef778017f2ca4046abbaa23b88c290", GitTreeState:"clean", BuildDate:"2019-04-19T22:12:47Z", GoVersion:"go1.12.4", Compiler:"gc", Platform:"darwin/amd64"} Server Version: version.Info{Major:"1", Minor:"14+", GitVersion:"v1.14.6+999bb21", GitCommit:"999bb21", GitTreeState:"clean", BuildDate:"2020-02-17T18:41:39Z", GoVersion:"go1.12.12", Compiler:"gc", Platform:"linux/amd64"}
Cloud Provider/Platform:
I am using OpenShift 4.2
$ oc version Client Version: version.Info{Major:"4", Minor:"1+", GitVersion:"v4.1.0+b4261e0", GitCommit:"b4261e07ed", GitTreeState:"clean", BuildDate:"2019-07-06T03:16:01Z", GoVersion:"go1.12.6", Compiler:"gc", Platform:"darwin/amd64"} Server Version: version.Info{Major:"1", Minor:"14+", GitVersion:"v1.14.6+999bb21", GitCommit:"999bb21", GitTreeState:"clean", BuildDate:"2020-02-17T18:41:39Z", GoVersion:"go1.12.12", Compiler:"gc", Platform:"linux/amd64"}
The Problem:

I install the chart with the disable openapi validation.

$ helm install -f charts/values.yaml enterprise-pipeline ./charts/ --disable-openapi-validation
NAME: enterprise-pipeline
LAST DEPLOYED: Mon Apr 27 14:45:15 2020
NAMESPACE: mynamespace
STATUS: deployed
REVISION: 1
TEST SUITE: None

I try to upgrade my chart with the disable openapi validation

$ helm upgrade -f charts/values.yaml enterprise-pipeline ./charts/ --disable-openapi-validation
Error: unknown flag: --disable-openapi-validation

I remove the disable openapi validation. Now I am back to where I started. That is, trying to make each OpenShift objects compliant with the openapi validation, which does not seem possible right now.

$ helm upgrade -f charts/values.yaml enterprise-pipeline ./charts/
Error: UPGRADE FAILED: error validating "": error validating data: [ValidationError(BuildConfig.spec): missing required field "nodeSelector" in com.github.openshift.api.build.v1.BuildConfigSpec, ValidationError(BuildConfig): missing required field "status" in com.github.openshift.api.build.v1.BuildConfig]

Is this still an issue with helm 3.2.0?

@bacongobbler Looks like that did the trick :) 3.2.0 came out 5 days ago. When I was working on this last week, I checked to see if there was an update. Should have checked today :)
$ helm upgrade -f charts/values.yaml enterprise-pipeline ./charts/ --disable-openapi-validation
Release "enterprise-pipeline" has been upgraded. Happy Helming!
NAME: enterprise-pipeline
LAST DEPLOYED: Mon Apr 27 15:33:47 2020
NAMESPACE: mynamespace
STATUS: deployed
REVISION: 2
TEST SUITE: None


ci: add VERSIONSTRING Makefile overwrite parameter

The VERSIONSTRING parameter for the Makefile is required to be able
to skip the build step that runs git describe --always --dirty and
set the version suffix manually.
This change allows the packaging for Homebrew as during the build
process it cannot execute git commands.
This is a optional parameter and will fall back to the git command if
not set specifically.
make VERSIONSTRING=test DEST=buildir publishTo

Example usage of the parameter, would output a binary with the
version set to 1.0.0-test.

Cool, thanks @shawnfunke! Do you plan to make a Homebrew formula for Marksman?

This change allows the packaging for Homebrew as during the build process it cannot execute git commands.

Quick question about this. What would the version string be like when building with Homebrew? Is it going to be hardcoded in the formula?

Great! Thanks for the contribution @shawnfunke!


Matrix Driver Tests

Design and carry out tests to validate that the part will be suitable for the application you have planned. (Include photos/screenshots.)

Possible tests to carry out:

[ ] How many LEDs can be connected at once?
[ ] How many LEDs can be on at once?
[ ] Transition between colors (can you use a pwm signal?)

Currently, code only has 9 LEDs connected at once, can turn on 9 LEDs at once and can light up specific LEDs by passing arrays to the runLEDDebug function (see pic attached). Will attempt to expand the matrix as new parts deadline approaches, but currently there is a base to work with. Further discussion with group of it is necessary to see if color transition is necessary for the intended application of the debugger.

@wecodewithmichaelmccloskey please review

Looks good


Missing space between "samples" and "provide"

"The application development documentation and the Azure Sphere samplesprovide information about application design."

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

ID: e8b25cf9-8383-e69a-777f-b67925ed4855
Version Independent ID: 994e83e1-231a-9f71-ef5c-7fe9725d2a59
Content: Factory floor operations - Azure Sphere
Content Source: AzureSphere/articles/hardware/factory-floor-tasks.md
Product: azure-sphere
GitHub Login: @pennyo
Microsoft Alias: azurespheredocs

@msjsheridan Thank you very much for taking the time to report this minor detail. It is greatly appreciated. I will get this corrected now.

@pennyo I created a PR to address the following issue raised by @msjsheridan.


podvm: switch to attestation-agent in guest-component repo

The attestation-agent repo was merged into the guest-component repo. Updating the podvm makefile to pull attestation agent from the new repo. Also enabeling fetching explicit commits.
Fixes: #1126

any plans to update the Dockerfile.podvm_binaries.{centos, rhel} ?

Missed these, I've also added the override option there, so people can inject their fork/another version if wanted.

@stevenhorsman mentioned a blocking kata issue we should wait for in https://github.com/confidential-containers/cloud-api-adaptor/issues/1126#issuecomment-1614313768

@stevenhorsman mentioned a blocking kata issue we should wait for in #1126 (comment)

To add a bit of nuance - we aren't strictly blocked by kata-containers updating, but I think there is much more risk of something breaking if we go first as we'll have mismatched versions of the attestation-agent (and the builds all being in the same repo hasn't been integration tested). That kata-containers change itself is block on the kata-containers CC CI issues, that I'm still working through. The PR changes look good though. We'll have to be mindful about the version of attestation-agent that gets pulled into kata-containers and potentially bump this afterwards to try and stay in sync

Hey Paul, Ive done the kata bump now in https://github.com/kata-containers/kata-containers/pull/7228 I think there is a slightly newer commit there, but we should be unblocked on this one now

@stevenhorsman I've updated the commit hash to match the one from the kata PR.

@stevenhorsman I've updated the commit hash to match the one from the kata PR.

Thanks, I'll give it a quick test and see if I can create a podvm from it


Adds gojs for Go generated Wasm

This adds an experimental package gojs which implements the host side of Wasm compiled by GOARCH=wasm GOOS=js go build -o X.wasm X.go
This includes heavy disclaimers, in part inherited by Go's comments https://github.com/golang/go/blob/go1.19/src/syscall/js/js.go#L10-L11
Due to this many will still use TinyGo instead.
That said, this is frequently asked for and has interesting features including reflection and HTTP client support.
Ex.
// Grant the compiled wasm access to the default HTTP Transport.
ctx = gojs.WithRoundTripper(ctx, http.DefaultTransport)

// Execute the "run" function with the binary you compiled via
// `GOARCH=wasm GOOS=js go build -o X.wasm X.go`
err = gojs.Run(ctx, r, compiled, config)
if exitErr, ok := err.(*sys.ExitError); ok && exitErr.ExitCode() != 0 {
	log.Panicln(err)
} else if !ok {
	log.Panicln(err)
}

Fixes #432

note: I'm off for a week, so that's why this will be unusually stale for me

oh sorry mistakenly pushed a commit.. feel free to remove it

I'm going to finish this because it is too old

next step is to backfill tests for errors on all the functions that are easy enough to err (like files not found, http errors etc)

hmm the tests on CI take 40s which I assume is due to the amount of time it takes to compile. I might make a different approach where the tests use a single binary switched on arg. should reduce time and also flake

ok actually the execution speed of wasm is also a reason why it was slower. Regardless this is now compiling once for the package and re-using the same binary for parallel tests. Finishes in 1/4 the time on my laptop now.

Actually, the runtime performance is so bad I'm kindof wondering if we should just dump this.
Ex the below is literally returning a mock response
$ time go run stars.go
wazero has 9999999 stars. Does that include you?

real    0m4.523s
user    0m8.657s
sys     0m0.563s

Also a fake main takes a second to execute:
func main() {
	fmt.Println("hello")
}

@prep @paralin do you have a working example on https://github.com/prep/wasmexec that can compare execution time across its 3 runtimes? I want to double-check there's not something wrong with the impl here.
iotw I was expecting slow, but not second+ slow, and I'm not sure what use cases can be met by that slow as even docker is faster.

@paralin was thinking the same thing. I'll add a real benchmark which shows compile vs (instantiate/run). Instantiate and run are coupled as GOOS=js uses host state (host side values) coupled with wasm state (data segments etc).

ok @paralin's thoughts were right. Since @mathetake recently implemented compilation cache. Using GOARCH=wasm GOOS=js without it is incredibly slow due to the size and the complexity of the Wasm generated.
$ cd examples/gojs
$ go run .
2022/08/26 09:09:12 CompileModule took 4576ms with 36062KB cache
wazero has 9999999 stars. Does that include you?
2022/08/26 09:09:12 gojs.Run took 20ms

$ go run .
2022/08/26 09:09:15 CompileModule took 145ms with 36074KB cache
wazero has 9999999 stars.
I think people building apps than embed wazero that runs GOARCH=wasm GOOS=js in docker...
They should use a cache dir, possibly doing a one-time run as a part of building the image, as otherwise people using the container will suffer and extraordinary first hit penalty every time.

@mathetake PTAL I think we are good. I'm not going to make a full on bench for gojs right now as already way over time budget and there is too much to review as it is. However, I think things are in good enough shape to review for merge.

One thing I noticed, but won't dig into at the moment, is this has no ability to collaborate with ad-hoc JavaScript at the moment (ex define ad-hoc functions in JS like spicedb cc @josephschorr)
This was out of scope for reasons of doing something like that needing a JavaScript engine which isn't a zero dependency thing in go, and makes the PR bigger also.
Currently, the only js functions supported are those Go defines in their own source code, so basically no dynamic javascript parsing. This makes sense because as wazero is typically used for non-browser use, starting with only what go uses allows folks to at least use normal go, even if they can't inject dynamic functions into it.

The other thing left out is that Go allows via .s files the ability to import custom functions. This was left out as the calling convention Go uses is completely unlike the rest of WebAssembly and making something to implement that especially before people ask for it would be odd. https://wazero.io/languages/go/#user-defined-host-functions

Looks cool, will test!


Single filescore key (dev)

Codecov Report

Merging #303 (edff383) into dev (5f660c1) will decrease coverage by 0.02%.
The diff coverage is 50.00%.

@@            Coverage Diff             @@
##              dev     #303      +/-   ##
==========================================
- Coverage   67.23%   67.20%   -0.03%
==========================================
  Files         120      120
  Lines       10556    10561       +5
==========================================
+ Hits         7097     7098       +1
- Misses       3459     3463       +4

Impacted Files
Coverage Δ

assemblyline/common/constants.py
87.50% <ø> (ø)

assemblyline/common/identify.py
89.18% <25.00%> (-0.82%)
:arrow_down:

assemblyline/odm/messages/submission.py
96.15% <100.00%> (+0.15%)
:arrow_up:

assemblyline/odm/models/submission.py
84.44% <100.00%> (+0.17%)
:arrow_up:

assemblyline/common/heuristics.py
82.81% <0.00%> (-1.57%)
:arrow_down:

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 5f660c1...edff383. Read the comment docs.


subsys: fs: Extend storage_dev type beyond 'struct device'

Extend storage_dev type beyond 'struct device' by changing
the variable type to void pointer. This will be needed if we
have additional transactional layers b/w filesystem and the
actual backend device.
Signed-off-by: Ramakrishna Pallala<EMAIL_ADDRESS>

Codecov Report

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

@@           Coverage Diff           @@
##           master    #6970   +/-   ##
=======================================
  Coverage   55.63%   55.63%
=======================================
  Files         462      462
  Lines       44567    44567
  Branches     8296     8296
=======================================
  Hits        24795    24795
  Misses      16380    16380
  Partials     3392     3392

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 361ef34...939e46a. Read the comment docs.

@andrewboie at this stage, this is primarily needed for FatFS where we handle things differently based on the type of storage (Flash, RAM or SDHC). So making storage type opaque helps us abstract the low-level details.

@nvlsianpu @carlescufi
After doing a thorough analysis of NFFS library code and having tried few ideas to enable multiple instances of NFFS, I had come to the conclusion that it is not possible to have multiple instances of NFFS with current library code.
The reason is, NFFS is implemented as singleton file system which uses multiple flash areas to store information in the form of files and directories. Ideally, we can see these individuals or group of flash areas as logical partitions which can be formatted and mounted as NFFS file system. But the NFFS implementation does not provide any means to group these flash areas as a logical partition or drive and does not allow us to format or mount multiple NFFS instances.
The following global variables defines the NFFS file system and which can only form a single instamce of a file system:
struct nffs_area nffs_areas[NFFS_CONFIG_MAX_AREAS];
uint8_t nffs_num_areas;
uint8_t nffs_scratch_area_idx;
uint16_t nffs_block_max_data_sz;
struct nffs_area_desc *nffs_current_area_descs;
struct nffs_inode_entry *nffs_root_dir;
struct nffs_inode_entry *nffs_lost_found_dir;
Let me know if you think otherwise and share your inputs.

@ramakrishnapallala
Of Course you are right - for now NFFS code is able to support only one NFFS instance. I didn't check the scope of global variables (what you listed looks reasonable) - I can  look at this scope essentially if you need this.

@nvlsianpu thanks for the confirmation on NFFS multiple instances.

@nvlsianpu @carlescufi @lpereira @andrewboie Please review the PR and let me know your feedback.

I will review this In a few days

@nvlsianpu can you have a look at the PR?

@lpereira I have addressed your comments. Please have a look. Thanks.
@nvlsianpu Please review the PR as it is destined for v1.12 release.

CC @lemrey

@ramakrishnapallala I'm reviewing this now.

2'nd commit ext: fs: fat: Enable multiple volume support and 3'rd commit subsys: disk: Add support for multiple disk interfaces should be merged as changes are closely linked.

@nvlsianpu I have addressed your comment and updated the PR.
One additional change included is, I have added open file and directory Kconfigs at subsys/fs level. I have introduced the open files concept to fatfs as well which NFFS is already doing it.

@nvlsianpu Please have a look at the update the PR. Thanks.


fix: add maven repository

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

1.0.5
GitHub release
1.0.5

Your semantic-release bot :package::rocket:


Specific project not returned over API

While trying to reproduce #3732 I found that the project with slug open-gpias is not found/returned by the API:
$ http 'http://readthedocs.org/api/v1/project/open-gpias/?format=json'
HTTP/1.1 404 Not Found
Connection: keep-alive
Content-Encoding: gzip
Content-Language: en
Content-Type: text/html; charset=utf-8
Date: Mon, 05 Mar 2018 14:26:47 GMT
Server: nginx/1.10.3 (Ubuntu)
Transfer-Encoding: chunked
Vary: Accept-Encoding
Vary: Accept, Accept-Language, Cookie

While, another project like pip is returned:
$ http 'http://readthedocs.org/api/v1/project/pip/?format=json'
HTTP/1.1 200 OK
Cache-Control: no-cache
Connection: keep-alive
Content-Encoding: gzip
Content-Language: en
Content-Type: application/json
Date: Mon, 05 Mar 2018 14:27:00 GMT
Server: nginx/1.10.3 (Ubuntu)
Transfer-Encoding: chunked
Vary: Accept-Encoding
Vary: Accept, Accept-Language, Cookie
X-Deity: web02
X-Frame-Options: DENY

{
    "absolute_url": "/projects/pip/",

I double checked this in the admin and the slug is correct:

Another project with - on its name does work, like zorg-seeed.

Open GPIAS project is Protected as Privacy Level, so I suppose that could be the reason. Is this OK? Is this what we want?
Since the project is not exposed over the API I can't call import_project_from_live command to reproduce the error locally.

I think we already decide that this is correct or I maybe I don't remember... But what is for sure that privacy levels are going to be removed. Not sure if you want to keep this issue open.


docs: Mostly correct spaces and new lines usage in CONTRIBUTING.md, README.md and packages/smooth_app/fastlane/metadata/en-US/full_description.txt

Codecov Report

Merging #4273 (1316a45) into develop (c818bb1) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop    #4273   +/-   ##
========================================
  Coverage    10.87%   10.87%
========================================
  Files          285      285
  Lines        14170    14170
========================================
  Hits          1541     1541
  Misses       12629    12629

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Added find/replace box

Hello! Why did you close your pull request? I was ready to analyze and pull into release/1.1 😪


Add inline notifaction for too large file uploads

The maxFileUploadSize has been set to 0.25mb as I saw that in socketIo it defaults to 0, not sure if that value should have a default value other than 0? When the user tries to upload an image that is too large, it displays an exclamationcircle, this can either be hovered on, or clicked to display the error message. As soon as the user uploads an image that is the correct size, it disappears.

We need to really know if the max of image is correct
I agree. I just set it to 0.25mb(250000 bytes) temporarily for now as it was set no where else from what I could see.
I saw that for notification, you added two: The one from the store and the red icon before the help exclamation mark icon. i think that one would be enough.
Yeah I was also wondering about the notification one(it was there) but we can remove it, but I feel like having both is good. But objectively it may be better to just have the inline notification. The red icon only appears when the user uploads a image > than the maxUploadSize, and then disappears when the user uploads a valid image.

I think we need this with @ivanslabbert. I am not entierely sure with the max size of the images.

The max size is set, it's 10mb. Just try to add a file that is bigger than 10mb.


DO NOT SUBMIT

I am going to close this since it is not meant to land and the formatter had landed. 👍


Make manual LB invocation more straightfoward, and document it

Original author: Nicolas Bock
Original issue: https://charm.cs.illinois.edu/redmine/issues/277

Either explain in the documentation how to call StartLB() (see issue #276) or expose StartLB() such that it is unnecessary to reference the LBDatabase object.

Original date: 2013-10-09 16:49:50

Traced the path of a StartLB() call, looks like it needs to use myLDHandle.  I'll look into streamlining the process.
Meanwhile, #276 has been resolved, and now more detailed instructions are in the documentation.

Original date: 2013-10-10 22:22:50

Added new function CkStartLB() that starts load balancing without needing a reference to the LBDatabase object.


Can't lime test windows

Describe your problem here.

What should I do?
Are you modding a build from source or with Lua?
Source
What is your build target?
Windows x64
Did you edit anything in this build? If so, mention or summarize your changes.
No response

or smth wrong with source or smth wrong with libraries (includes visual studio libs)


feat: add array/base/cunone

Resolves #2321
Description

What is the purpose of this pull request?

This pull request:
adds array/base/cunone
Related Issues

Does this pull request have any related issues?

#2321
This pull request:

resolves #2321

Questions

Any questions for reviewers of this pull request?

No.
Other

Any other information relevant to this pull request? This may include screenshots, references, and/or implementation notes.

No.
Checklist

Please ensure the following tasks are completed before submitting this pull request.

[x] Read, understood, and followed the contributing guidelines.

@stdlib-js/reviewers

@adityacodes30 Would you mind ticking the checklist box? Thanks!

done


Development of the .characteristic_poly method for ODESystem

A novel approach for computing the characteristic polynomial is required, which can be integrated into the ODESystem as one of its features. This approach must have the capability to efficiently generate the characteristic polynomial for Ordinary Differential Equations (ODEs). It should be adaptable for systems with multiple degrees of freedom by leveraging matrix computations.

Method implemented successfully. I am closing the issue


Types that point to each other generate C code that won't compile

This issue is similar to #358 fixed by #360. The compiler accepts the following input
import Austral.Memory (
    Pointer
);

module body Test is
    pragma Unsafe_Module;

    record Bar: Free is
        fooptr: Pointer[Foo];
    end;

    record Foo: Free is
        barptr: Pointer[Bar];
    end;

    function main(): ExitCode is
        return ExitSuccess();
    end;
end module body.

but the generated code won't compile:

Standard error:
    /tmp/austral_fff442.c:3372:17: error: unknown type name 'mono_13'
         3372 | typedef struct {mono_13* barptr;} mono_12;
              |                 ^~~~~~~

The fix for this may be to generate forward declarations for all types before theirs definitions as @eudoxia0 suggested

Huh, I thought I was already doing this here: https://github.com/austral/austral/blob/master/lib/CodeGen.ml#L593-L609

Let me at least add a test for this.

This is fixed now: #372
Thanks for reporting!


Add explanation on CRDS variables for Macbooks

Any current description about CRDS is a bit limited and would be better if we made things more explicitly clear for Mac users (e.g. mention .zshrc etc.)

@jbrande In addition to the change you made about .zshrc, we should also explain what the CRDS_PATH should be. Several people have been confused as to where they are supposed to find this path, not realizing that this is a folder they should make somewhere useful to them to store the outputs. Since this same variable/folder is used by both HST and JWST, I recommend putting it somewhere in your home directory (e.g. /Users/tbell1/crds_cache for me on Mac, /home/tbell1/crds_cache for me on Ubuntu)

Got it. I'll make another update there.


Removed naming conflict with Qt's emit define

The alternative would be to configure Qt to not define emit. But obviously we are not able to do that in every project.

Hi @marcobusemann, I'm your friendly neighborhood Microsoft Open Technologies, Inc. Pull Request Bot (You can call me MSOTBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
We will now validate the agreement and then real humans will evaluate your PR.
TTYL, MSOTBOT;

Thanks!


[Bug]: Cannot delete document from index over OpenSearch

Bug Description
I cannot seem to call index.delete_ref_doc() on a VectorStoreIndex that I've build over a local OpenSearch index. Everything with the code below in "steps to reproduce" works fine up until the last line which throws me a NotFoundError(404, '{"_index":"error_demo","_id":"docid1","_version":1,"result":"not_found","_shards":{"total":2,"successful":1,"failed":0},"_seq_no":4,"_primary_term":1}') error.
Version
0.9.44
Steps to Reproduce
# Service and storage
text_splitter = SentenceSplitter(chunk_size=512, chunk_overlap=16)
service_context = ServiceContext.from_defaults(
    embed_model=BGESmallEnEmbeddings(embed_batch_size=2),
    text_splitter=text_splitter,
    llm=Llama2LLM(),
)
client = OpensearchVectorClient(
    "http://<IP_ADDRESS>:9200", "error_demo", 384,
)
vector_store = OpensearchVectorStore(client)
storage_context = StorageContext.from_defaults(vector_store=vector_store)

# Documents
documents = [Document(text="first doc text", id_='docid1'), Document(text="second doc text", id_='docid2')]

index = VectorStoreIndex.from_documents(
    documents, service_context=service_context, storage_context=storage_context
)

# Try to delete
index.delete_ref_doc('docid1', delete_from_docstore=True)

Relevant Logs/Tracbacks
No response

I have no idea how to do this, but I appreciate the quick response and if this becomes a high pri issue for me and I get bandwidth then I'll work on a PR!

@logan-markewich same seems to be true with index.update_ref_doc(); can you confirm?

@wdeaderick yeaaaa that function also calls delete() under the hood 😅

@logan-markewich the following implementation worked for me:
def delete_doc_id(self, doc_id: str) -> None:
        """Delete a document.

        Args:
            doc_id (str): document id
        """
        body = {
            "query": {
                "match": {
                'metadata.ref_doc_id': doc_id
                }
            }
        }
        self._os_client.delete_by_query(index=self._index, body=body)

There seem to be 3 identical fields in the OpenSearch document object (metadata.document_id, metadata.doc_id, and metadata.ref_doc_id), so I chose metadata.ref_doc_id somewhat arbitrarily to match against the argument doc_id (str): document id.
Not sure what else needs to happen in this function (error handling, state updates, etc.) but I believe this is the crux of the implementation.

Cool! I can probably add this to a PR then if it works for you

That'd be great; thanks!

@logan-markewich https://github.com/run-llama/llama_index/pull/11292


Add The No-Stop Tree by Raf Guy

Song Name
The No-Stop Tree
Artist Name
Raf Guy
Source
Youtube
Youtube Link
AQFHMlhN3jOaSHWR
Direct File Link
https://drive.google.com/file/d/1pQmri2se_c4oE2IMtMtuP7rHRElmw4hq/view?usp=drive_link
Song ID
557651
Extra
Created from Auto Nong.
Created for level: 106934320

reject
invalid YouTube link


Get dimensions of face

I'm selecting a face on a part that I know will be rectangular, but could be of unknown size.  I'd like to draw a circle on this face where the dimension of the circle is based on being "x" less than the minimum dimension of the rectangle.  What is the easiest way to achieve this?
import build123d as cad
box = cad.Box(100, 80, 10)
top = box.faces().sort_by()[-1]
# create a circle with diameter 10 less than the minimum rectangle dimension (with or height)
# in this example, it would simply be a diameter of 70, but how would I do this if I didn't know the face dimensions?

There are dedicated length and width methods for this or alternatively one can use the Face's bounding box as shown here:
import build123d as cad
from ocp_vscode import show_all

box = cad.Box(100, 80, 10)
top = box.faces().sort_by()[-1]
# create a circle with diameter 10 less than the minimum rectangle dimension (with or height)
# in this example, it would simply be a diameter of 70, but how would I do this if I didn't know the face dimensions?

print(top.geometry)  # RECTANGLE
circle_diameter = min(top.length, top.width) - 10
print(circle_diameter)  # 70
# or
circle_diameter = min(top.bounding_box().size.X, top.bounding_box().size.Y) - 10
print(circle_diameter)  # 70
hole = cad.Pos(top.center()) * cad.Circle(radius=circle_diameter / 2)

# Or in Builder mode
with cad.BuildPart() as bp:
    cad.Box(100, 80, 10)
    top = bp.faces().sort_by(cad.Axis.Z)[-1]
    circle_diameter = min(top.length, top.width) - 10
    with cad.BuildSketch(top):
        cad.Circle(radius=circle_diameter / 2)
    cad.extrude(amount=-1, mode=cad.Mode.SUBTRACT)

show_all()

Awesome, thanks - I thought I looked everywhere in the docs but I obviously missed this.  I appreciate the response.
As a follow-up, is there an easy way to know how a plane derived from a face will be oriented?  Consider the code below, which prints Plane(o=(0.00, -0.00, 35.00), x=(0.80, 0.60, -0.00), z=(-0.00, -0.00, -1.00)) for me.  I would have expected this plane to still have "x" oriented in (1,0,0), or maybe (-1,0,0), but it looks like x instead points towards one of the corners of the face.  Is that expected here?  If so, is there a rule to know how a face will be oriented ahead of time?
import build123d as cad
from ocp_vscode import show_object
sketch = cad.Sketch() + [
    cad.Rectangle(100, 80),
    cad.Plane(origin=(0, 0, 40)) * cad.Rectangle(80, 60),
]
solid = cad.loft(sketch, ruled=True)
bottom = solid.faces().sort_by()[0]
shell = cad.offset(solid, amount=-5, openings=bottom)

face = shell.faces().sort_by()[-2]
outer = cad.make_face(cad.offset(face.wires(), amount=-2))
inner = cad.make_face(cad.offset(face.wires(), amount=-10))
shell += cad.extrude(outer - inner, -5)

face = shell.faces().group_by()[-2].sort_by_distance((0,0,0)).first
print(cad.Plane(face))

show_object(shell)
show_object(face)

The short answer is no.  A Plane defines a coordinate system where the origin and directions of all three axes are defined.  A planar Face doesn't have enough information to fully define a coordinate system, the x and y axes are free to rotate about the z axis (as long as they are 90 degrees apart and follow the right-hand-rule). However, the internal OpenCascade algorithm for this is usually quite good at picking sensible directions - but not always.
However, it is possible to create a custom Plane as follows: Plane(origin=face.center(), x_dir=(1,0,0), z_dir=face.normal_at()) which allows the user to control the x axis.


chore: deprecate quota_type from supported instance types endpoint

Description
Related to https://issues.redhat.com/browse/MGDSTRM-10022.
Due to recent changes billing models specification (https://github.com/bf2fc6cc711aee1a0c2a/kas-fleet-manager/pull/1371, https://github.com/bf2fc6cc711aee1a0c2a/kas-fleet-manager/pull/1379) the quota_type attribute returned in the /api/kafkas_mgmt/v1/instance_types/{cloud_provider}/{cloud_region} is not used anymore and it will not reflect the real state of the supported quota type(s).
This PR takes care of:

Marking the quota_type attribute in the /api/kafkas_mgmt/v1/instance_types/{cloud_provider}/{cloud_region} response as deprecated
Renaming go types corresponding to that attribute with the Deprecated prefix
Renaming go types corresponding to that attribute at kafka supported instance types configuration level (config/kafka-instance-types-configuration.yaml) with the Deprecated prefix

This PR only marks the field as deprecated. It is still returned as part of the endpoint response and available in the kafka supported instance types configuration (to support returning it as part of the endpoint). Once the deprecation period finishes then we should implement the removal from both the endpoint and the configuration file.
Verification Steps

Checklist (Definition of Done)

[ ] All acceptance criteria specified in JIRA have been completed
[ ] Unit and integration tests added that prove the fix is effective or the feature works (tested against emulated and non-emulated OCM environment)
[ ] Documentation added for the feature
[ ] CI and all relevant tests are passing
[ ] Code Review completed
[ ] Verified independently by reviewer
[ ] All PR comments are resolved either by addressing them or creating follow up tasks
[ ] Required metrics/dashboards/alerts have been added (or PR created).
[ ] Required Standard Operating Procedure (SOP) is added.
[ ] JIRA has been created for changes required on the client side

cc @dimakis as we commented through other channels

I've had a look and Rama has had a look over this too. Neither of us sees this attribute being used in the CLI.
LGTM

Thank you @dimakis :+1: we will proceed with the deprecation of the attribute then


GitHub star button was added

Github star icon was added on a homepage

@DmitriyLukinov this looks odd, compared to the website design, and yes it was not mentioned in the issue ticket as well, but can we edit such that it suits the UI as well?


VS Code Settings only opens to a JSON file every time

Issue Type: Bug
Hello,
I can't open my settings.  It goes straight to a JSON file every time.  I don't know what to do with that JSON file when I want to add settings to my prettier extension.

You can always open settings editor from F1, Preferences: Open Settings (UI)

Good morning Simon.  I have enclosed a pic of what VS Code looks like when I open the settings editor using F1, Preferences: Open Settings or using the Command Palette.  I don't know how to get out of this and get to my settings.  Please help.
dp
On Sunday, June 20, 2021, 08:09:39 AM EDT, Simon Chan @.***> wrote:
Check your "workbench.settings.editor" setting is not set to "json".
You can always open settings editor from F1, Preferences: Open Settings (UI)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

Note: GitHub comments doesn't include pictures from emails. Please upload using web interface or mobile Apps.

"workbench.settings.editor" setting is set to "json".  Should I change it to something else?  Thanks.

You can delete that setting to get the default behavior of opening the gui


POC Feature/tts word by word highlight

It would be nice to have settings for both foreground text color and background color for the word/line that's highlighted during TTS.

It would be nice to have settings for both foreground text color and background color for the word/line that's highlighted during TTS.

that's a good idea, i can add that.


mingw-w64 build .dll version creation

This will create .dll version information for d3d11, d3d10, d3d10.1, d3d10core and dxgi dll's.
I used the version that was on the files in my Windows 10 VM installed box - 10.0.17763.1
The only thing changed vs "the original" is really Copyright note, cos I don't think its proper to use Microsoft there, so i set: 2019 It's FOSS!
Take a look at it, and do some tests.
OBS! This will NOT create version info for a winelib build, as the version info is really tied to the "fakedlls" and the dll.so will not show this if used. So for now this is just "disabled" with a not dxvk_winelib ? statement.
If "fakedlls" are to be created some day, this could be enabled when compiling those i guess.
Customize at will :)

what happen if use the wine build system?

@sl1pkn07
If you mean "what happens if i use the winelib build", nothing happens with versions because the "version info" is tied up to the "fakedlls" that winelib uses, and symlinking dll.so winelib files will NOT carry the dll version info over. So atleast with this PR the --winelib build option will build winelib dll.so's like it has done without any changes.
And as of current, dxvk does not build fakedll's with the "winelib" release. The only REAL point of building fakedll's would be to replace wine compiled d3d1* files with the dxvk dll.so's, and the corresponding fakedll's... in other words you would need a use that for a "custom" separate wine binary release.
Not really sure its a huge benefit to do so?

because i see problems with this
https://github.com/doitsujin/dxvk/blob/a46a294e7d8d96283ac67533769a696835be45e8/meson.build#L58-L60
when use build-wine64.txt or build-wine32.txt. bucause seems need installed the mingwin environment for build it even use wine build system
https://github.com/doitsujin/dxvk/blob/a46a294e7d8d96283ac67533769a696835be45e8/meson.build#L52-L61
not need set this before the else or add a "if" if not use winegcc/wineg++ compiler?

@sl1pkn07
What is the problem?

use the wine build environment is for avoid use mingw environment. use use mingw in the wine build environment defeat the wine build purpose

@sl1pkn07

use the wine build environment is for avoid use mingw environment. use mingw in the wine build defeat the wine build purpose

Yes. But where do you mean i actually USE mingw when building winelib?
example:
meson --cross-file build-wine64.txt --buildtype release --libdir=lib64 --strip build.w64
Checking if "winelib check" compiles: YES
This will not use mingw for anything.
Post logs if you have different experience ofc.
PS. The clue is in:
https://github.com/doitsujin/dxvk/blob/a46a294e7d8d96283ac67533769a696835be45e8/meson.build#L35-L46
IF i am building "dxvk_winelib", ELSE i am building mingw.... The --cross-file build-wine64.txt will follow the build if dxvk_winelib ELSE the following meson build:
meson --cross-file build-win64.txt --buildtype release --libdir=lib64 --strip build.w64
Checking if "winelib check" compiles: NO
And this will use mingw.
Agree? :)

okidoki. seems i not scroll more avobe into the changes/files :/
i got it all :)
i can make a little change?
diff --git a/meson.build b/meson.build
index 0ccec101..3bd31718 100644
--- a/meson.build
+++ b/meson.build
@@ -41,6 +41,8 @@ if dxvk_winelib
   lib_d3dcompiler_43 = declare_dependency(link_args: [ '-L'+dxvk_library_path, '-ld3dcompiler_43' ])
   lib_d3dcompiler_47 = declare_dependency(link_args: [ '-ld3dcompiler' ])
   dxvk_extradep += [ declare_dependency(link_args: [ '-ldl' ]) ]
+  wrc = [ find_program('wrc'), '-o', '@OUTPUT@', '@INPUT@' ]
+  winebuild = [ find_program('winebuild'), target_arch, '--dll', '--fake-module', '-E', '@INPUT@', '-o', '@OUTPUT@', '-F', '@OUTPUT@' ]
   exe_ext = 'exe'
   dll_ext = '.dll'
   def_spec_ext = '.spec'
@@ -49,7 +51,7 @@ else
   lib_d3d11   = dxvk_compiler.find_library('d3d11')
   lib_dxgi    = dxvk_compiler.find_library('dxgi')
   lib_d3dcompiler_43 = dxvk_compiler.find_library('d3dcompiler_43', dirs : dxvk_library_path)
-
+
   if dxvk_compiler.get_id() == 'msvc'
     lib_d3dcompiler_47 = dxvk_compiler.find_library('d3dcompiler')
     wrc = [ find_program('windres'), '-i', '@INPUT@', '-o', '@OUTPUT@' ]
diff --git a/src/d3d10/meson.build b/src/d3d10/meson.build
index ade29358..07a31771 100644
--- a/src/d3d10/meson.build
+++ b/src/d3d10/meson.build
@@ -2,21 +2,21 @@ if dxvk_winelib
   d3d10_res = custom_target('version_10.res',
     input   : 'version10.rc',
     output  : 'version10.res',
-    command : [ find_program('wrc'), '-o', '@OUTPUT@', '@INPUT@' ],
+    command : wrc,
     install : false,
   )

   d3d10_1_res = custom_target('version_1_10.res',
     input   : 'version10_1.rc',
     output  : 'version10_1.res',
-    command : [ find_program('wrc'), '-o', '@OUTPUT@', '@INPUT@' ],
+    command : wrc,
     install : false,
   )

   d3d10_core_res = custom_target('version10_core.res',
     input   : 'version10_core.rc',
     output  : 'version10_core.res',
-    command : [ find_program('wrc'), '-o', '@OUTPUT@', '@INPUT@' ],
+    command : wrc,
     install : false,
   )
 else
@@ -81,23 +81,23 @@ if dxvk_winelib
   d3d10_dll_target = custom_target('d3d10.dll',
     output  : 'd3d10.dll',
     input   : [ 'd3d10.spec', d3d10_res ],
-    command : [ find_program('winebuild'), target_arch, '--dll', '--fake-module', '-E', '@INPUT@', '-o', '@OUTPUT@', '-F', 'd3d10.dll' ],
+    command : winebuild,
     install : true,
-    install_dir : 'fakedlls')
+    install_dir : join_paths(get_option('libdir'), 'fakedlls'))

   d3d10_1_dll_target = custom_target('d3d10_1.dll',
     output  : 'd3d10_1.dll',
     input   : [ 'd3d10_1.spec', d3d10_1_res ],
-    command : [ find_program('winebuild'), target_arch, '--dll', '--fake-module', '-E', '@INPUT@', '-o', '@OUTPUT@', '-F', 'd3d10_1.dll' ],
+    command : winebuild,
     install : true,
-    install_dir : 'fakedlls')
+    install_dir : join_paths(get_option('libdir'), 'fakedlls'))

   d3d10_core_dll_target = custom_target('d3d10core.dll',
     output  : 'd3d10core.dll',
     input   : [ 'd3d10core.spec', d3d10_core_res ],
-    command : [ find_program('winebuild'), target_arch, '--dll', '--fake-module', '-E', '@INPUT@', '-o', '@OUTPUT@', '-F', 'd3d10core.dll' ],
+    command : winebuild,
     install : true,
-    install_dir : 'fakedlls')
+    install_dir : join_paths(get_option('libdir'), 'fakedlls'))
 endif

diff --git a/src/d3d11/meson.build b/src/d3d11/meson.build
index ea8d352c..842edacc 100644
--- a/src/d3d11/meson.build
+++ b/src/d3d11/meson.build
@@ -2,7 +2,7 @@ if dxvk_winelib
   d3d11_res = custom_target('version_11.res',
   input   : 'version.rc',
   output  : 'version.res',
-  command : [ find_program('wrc'), '-o', '@OUTPUT@', '@INPUT@' ],
+  command : wrc,
   install : false)
 else
   d3d11_res = custom_target(
@@ -82,9 +82,9 @@ if dxvk_winelib
   d3d11_dll_target = custom_target('d3d11.dll',
     output  : 'd3d11.dll',
     input   : [ 'd3d11.spec', d3d11_res ],
-    command : [ find_program('winebuild'), target_arch, '--dll', '--fake-module', '-E', '@INPUT@', '-o', '@OUTPUT@', '-F', 'd3d11.dll' ],
+    command : winebuild,
     install : true,
-    install_dir : 'fakedlls')
+    install_dir : join_paths(get_option('libdir'), 'fakedlls'))
 endif

 d3d11_dep = declare_dependency(
diff --git a/src/dxgi/meson.build b/src/dxgi/meson.build
index 9e924cb0..ca09f1ca 100644
--- a/src/dxgi/meson.build
+++ b/src/dxgi/meson.build
@@ -2,7 +2,7 @@ if dxvk_winelib
   dxgi_res = custom_target('dxgi.res',
   input   : 'version.rc',
   output  : 'version.res',
-  command : [ find_program('wrc'), '-o', '@OUTPUT@', '@INPUT@' ],
+  command : wrc,
   install : false)
 else
   dxgi_res = custom_target(
@@ -43,9 +43,9 @@ if dxvk_winelib
   dxgi_dll_target = custom_target('dxgi.dll',
     output  : 'dxgi.dll',
     input   : [ 'dxgi.spec', dxgi_res ],
-    command : [ find_program('winebuild'), target_arch, '--dll', '--fake-module', '-E', '@INPUT@', '-o', '@OUTPUT@', '-F', 'dxgi.dll' ],
+    command : winebuild,
     install : true,
-    install_dir : 'fakedlls')
+    install_dir : join_paths(get_option('libdir'), 'fakedlls'))
 endif

 dxgi_dep = declare_dependency(

i've moved the fakedlls install directory to libdir\fakedlls and edit the winebuild and wrc commands into the dxvk_winelib like you doing when set wrc command if mingw is used. the package-release.sh is untouched because i'm not sure what i need touch
i cant add a commit because i don't want mess up you PR. if my patch is ok for you, you can add it for integrate in the PR as you code
greetings and sorry

@sl1pkn07
I did not really intend my last commit with winelib fakedll to be a PR tbh, as i dunno what it could be useful for, unless you want to use if for a "custom wine binary install" kind of thing.
The mingw commit https://github.com/doitsujin/dxvk/pull/993/commits/7632ffb1dac52347f14c2a4d1ced4776d6e28b7d actually makes sense as it fixes a game problem. "Make fakedll" is probably nothing to be used really, as it would be pretty cumbersome and pointless in a way.. i dunno.
I'm sure @doitsujin would take a couple of pointers from your suggestion tho, but do not expect to see it accepted.
PS. I firstly intended to do the fakedll's commit as a branch, but messed that up, and WHAM suddenly its "out there" hehe.

i have tested the fakedlls in my build, with little changes in my own winetricks verb (taked from @pchome gentoo overlay, then remove the dll overrides and points to fakedll instead of .so files). seems work without problem except a strange output in console when launch the triangle test. https://pastebin.com/HRPMrqLH

@sl1pkn07
Pointing the symlinks to the fakedll's will not make dxvk's d3d11.dll.so file work, so those error messages you see that seems strange, is from wined3d (wine opengl) d3d11.dll.so.
You cannot just point symlinks to the dxvk created fakedll's and think that d3d11.dll.so "outside of tree" dll.so's work.

Tsk, you are right.
I thinking the build of the fakedlls of wine pointed directly somehow to the real libs.
Ok

@doitsujin

In general, I'd like to leave out the winelib stuff entirely for now, building fake dlls will require additional changes and I'm not even sure how useful it is right now.

Aboslutely agree. As i said a bit up in the thread, i did a mistake when forgetting to branch this, and it made its way into the pr. That was not meant to be, as i (also said) dont really see the point of doing the whole fakedll thingy.
Since i am not so amazingly skilled in "git", I will try to do some reverts and whatnot, but worst case ill just resync my fork and create another PR. (Note to self: Do not fiddle with sources AFTER a PR)

You can modify the sources, then use commit --amend and force-push (push -f) to update the PR, no need to file a new one.

@doitsujin
Oki, created a updated hopefully :)
Removed the winelib stuff (as it was not intended), attempted a cleanup of the wrc commands + edited the version.rc files. Hopefully those should not cause any problems with copyright?

Thanks, I'll test this later and merge if everything works as expected.

Closing this as implemented with https://github.com/doitsujin/dxvk/commit/4f9dd8d3d06258ba3870f92e089176aa46f50c12

Right, I did the merge manually after doing some testing.

This breaks compilation on winelib systems, if the user doesn't have mingw installed. This seems to be enough to workaround the issue (and doesn't appear to break either 32-bit or 64-bit games for me):
https://github.com/fullmoonsoup/dxvk/commit/b77f77ef3904e4592fa3889e5466eec178d3a919

@fullmoonsoup
Ill see if i can do some testing of this on a VM without mingw installed.
It did however seem as the latest wine-staging required mingw for something, so maybe it will be a thing that is somewhat requred for wine-staging eventually tho.
The wine-staging issue could be some weird ubuntu thing tho, since multilib compile with Ubuntu is somewhat of a exercise of sorts...


Any way to figure out which bundles are used by which all pages

From the bundle analyser I could figure out the size of each bundle. But it will be good to have other informations like. which all pages uses this bundle.

Coming in the next release (0.10.7), we use latest feature of Webpack webpackChunkName.
See the output of it:

@Atinux Thanks. Waiting for it !!


How to mount volume with browser container?

How to mount volume with browser container?
I want to download file and save them, or other way?

@lpnueg4 see volumes https://aerokube.com/selenoid/latest/#_other_optional_fields


Save and show final reconstructions in html page

In imaging inverse problems, we often end with a reconstruction that we would like to see for each benchmark run. On top of the curves plotted, it is important to see the final reconstruction when comparing methods/algorithms. it would be nice to store each reconstruction and, ideally, show it in the html file.

Hey, thanks for raising this issue.
We just landed in the dev version the save_last_result method to allow saving the last result of a solver.
So we would need to use this and then provide a plot that displays the image of each solver.


Improve code completion for constraints that need offsets and insets

Closes Issue #57

Xcode 10 is supposed to have improvements with code completion. It might touch this subject as well, but I don't know this for sure. I'll run TC with Xcode 10 this week to see if the PR is still relevant.

I'm sorry but this is too much code for such a small advantage.


Ryzen 5 2600 Null Temperature

Description
I was hoping I'd be able to replace Open Hardware Monitor with this for monitoring through Home Assistant since I really only need the basics and this seems much sleeker, however it appears the API isn't able to grab my CPU temperature, it just reports as Null. Additionally the CPU voltage is blank, though I'm not especially concerned about that. I know some API's have had issues with Ryzen processors in the past. Is that what's going on here?
What Platform / OS are you running?
Windows
What version are you running?
2.0.0
Anything in the logs or a references that might be useful?
{
  "cache": {
    "l1d": 0,
    "l1i": 0,
    "l2": 3145728,
    "l3": 16777216
  },
  "cpu": {
    "manufacturer": "AMD",
    "brand": "Ryzen 5 2600 Six-Core Processor",
    "vendor": "AuthenticAMD",
    "family": "23",
    "model": "8",
    "stepping": "2",
    "revision": "2050",
    "voltage": "",
    "speed": 3.4,
    "speedMin": 3.4,
    "speedMax": 3.4,
    "governor": "",
    "cores": 12,
    "physicalCores": 6,
    "processors": 1,
    "socket": "AM4",
    "flags": "de pse msr sep mtrr mca cmov psn clfsh ds mmx fxsr sse sse2 ss htt tm ia64 pbe",
    "virtualization": false,
    "cache": {
      "l1d": 0,
      "l1i": 0,
      "l2": 3145728,
      "l3": 16777216
    }
  },
  "currentSpeed": {
    "min": 3.4,
    "max": 3.4,
    "avg": 3.4,
    "cores": [
      3.4,
      3.4,
      3.4,
      3.4,
      3.4,
      3.4,
      3.4,
      3.4,
      3.4,
      3.4,
      3.4,
      3.4
    ]
  },
  "temperature": {
    "main": null,
    "cores": [],
    "max": null,
    "socket": [],
    "chipset": null
  }
}

Additional information
No response

I have the same issue. According to the [System Information](https://systemin formation.io) docs the WMIC module can have issues unless run as administrator. I have tried this, but it still doesn't report these sensors.
I don't think this app will ever be a replacement for OHM or other monitoring tools, but I think thermal sensors would be a nice addition if they work cross platform


📄🚀 – Expand documentation’s architecture section

Describe the feature you want and how it meets your needs or solves a problem
New TIDES users will want and need to understand how to put their transit agency's data into TIDES format, and part of that is understanding what TIDES table to use for each type of data.  In addition, they will have questions such as:

How should I handle multiple sources of AVL data for the same fleet?
How should I handle different versions of the same data (for example, raw vs. cleaned, or incomplete vs. complete)

The current documentation section is divided into architecture and table schemas.  The architecture section has a list of tables and the relationship between them, but for a particular use case or configuration.  A high-level description of the framework and a discussion of different possible use cases and configurations is missing.  Users can refer to the G-18 TCRP report but that cannot be updated and may not cover some details that are important for implementing TIDES.
Describe the solution you'd like
Draft a high-level description of the framework and a discussion of different possible use cases, discuss it as a group, and add it to the architecture section of the documentation.

I think it would be great to have a better introduction, especially one that includes motivating examples. The two examples you gave would fit well in a (much talked about, but unimplemented) best practices section.
I think a structure like this makes sense:

Introduction: what is TIDES, motivation, examples
Architecture: tables and the relationships among tables
Table Schemas: table definitions and descriptions
Best practices / FAQ: using TIDES as a format throughout the data processing pipeline, multiple sources, extensions

Other than a few motivating examples, see #32, I don't want to get into the trap of attempting to list all of the possible uses and configurations. This is a spec for data, not for processing it. The idea is to enable tools to work across agencies, so it is the responsibility of the tool maintainer to describe its capabilities and requirements.

Now is the time to create a Best Practices section of the Documentation

[ ] Build initial content in Google doc here
[ ] Create /docs/best-practices.md from Google doc
[ ] Modify scripts to added Best Practices section to documentation site


Improve performance alert tool

Improve performance alerting tool UI.
add additional information such as test link, dashboard link etc
Please add a meaningful description for your change here

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

[ ] Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
[ ] 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.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

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

UI Would look like https://github.com/apache/beam/issues/27191


Xtreme-Download-Manager: init at 7.2.11

Motivation for this change
https://github.com/NixOS/nixpkgs/issues/96557
Things done
Packaged xdman at release 7.2.11

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

after https://gist.github.com/GrahamcOfBorg/afffda3c1a919e8156d46c906d824b5f is resolved, I still think you will have issues with import-from-derivation (IFD)

Nixpkgs isn't allow to do import-from-derivation
cannot import '/nix/store/pa1555ank7g9axm8sp1frxnwg0xgalkk-source', since path '/nix/store/rddska8812a53ryzll92si5iks019q9a-source.drv' is not valid, at /var/lib/ofborg/checkout/repo/38dca4e3aa6bca43ea96d2fcc04e8229/mr-est/packet-spot-eval-3/pkgs/applications/networking/xtreme-download-manager/default.nix:10:13

Moved the import to top-level/all-packages.nix. Is that fine?

I think it's prolly best to not merge any uses of this mvn2nix tool into nixpkgs yet. The developer intends to showcase and open an RFC about it soon, see https://github.com/fzakaria/mvn2nix/pull/4#issuecomment-676697207.

My plan is to try and get this in a polished state before NixCon and submit a RFC (Request For Content) to talk about it.

If it ends up being merged, that would automatically solve your import-from-derivation problem.

Fine, I'll wait for mvn2nix to be in nixpkgs.


Rules priority

Good morning,
I have a question about the priority of the rules:
Is the priority of the rules in relation to their order? Rule 1 runs before rule 2?
I have the following rule settings:

I have verified that rule 3 is executed before rule 2. In my case, if it rains (rule 3) the blind is positioned at 25%, however, later than Astro Dusk, it should be closed completely due to rule 2.
Can anybody help me again?
Thank so much

Hallo,
there is a priority relation, but this is more difficult.
see https://github.com/rdmtc/node-red-contrib-sun-position/blob/master/blind_control.md#rules-execution-in-detail
in other words:

to the last rule of type "until" the rules will be evaluated from starting the first rule to the last rule of type until.
then the rules will be evaluated in reverse order

In your example the order of evaluating rules is following:

until civil Dawn rule 1 will be evaluated and only rule 1 gets active.
after civil Dawn the rules will be evaluated from last to first.
-rule 4

then rule 3
and then rule 2

Hallo
Sorry to ask you again, but I can't get the rules in order to be executed as I want.
Now I have order them as I show you in the screenshot

and rule 3 and rule 2 are evaluated before than rule 4.(astro dusk)
How should I order them so that Astro Dusk is a higher priority than Rain and Rain&presence
One more time...thank so much

Have you try it? In the documentation is a flow for a simulation.
I'm currently due to the situation very busy, so it will take some time until I can have a look into it.

You are right this no longer works as expected.
That was the case, I don't remember why I changed that. I will change that back with the next release.
This is still under test. If you feel brave you can already install it from command line from the node-red directory with (standard Linux):
npm install<EMAIL_ADDRESS>
If you are using a Homematic CCU the commands are (to use from ssh login):
source /usr/local/addons/redmatic/home/.profile
cd /usr/local/addons/redmatic/var
npm install --save --no-package-lock --global-style --save-prefix="~" --production<EMAIL_ADDRESS>

solved with Version 1.1.0


Make the repo code-pattern compliant

The readme should match the style of the repo above (like this: https://github.com/IBM/watson-vehicle-damage-analyzer)
There should be a blog about things you all have learned (like this: https://developer.ibm.com/code/2017/12/01/creating-offline-first-shopping-list-app-polymer-pouchdb/)
A video telling the reader how to deploy and run the app (like this: https://www.youtube.com/watch?v=SUj826ybCdU)

@Tomcli don't worry about 3. on the list above (the video), i just need a small clip from you showing the app being used on a few items.

[ ] create a blog (like this: https://developer.ibm.com/code/2017/12/01/creating-offline-first-shopping-list-app-polymer-pouchdb/)
[ ] draft the wiki (like this: https://github.com/IBM/watson-vehicle-damage-analyzer/wiki)

once those two are done your team is finished! let me know if you have any other questions!


Added filter: verified option to filter torrents

Added tests for Parser.isTorrentVerified()
Added defaults configurations to readme
Removed unecessary deps

@t3chnoboy Also, again, to publish this to npm:
git pull --all
npm i
npm test-all
npm version
npm publish

hah, that was fast 😂


Still having errors with qtreeview. If last item is moved into a parent, a NoneType object will appear, for some reason.

rowsInserted signal does not work as it is supposed to. I can't even begin to describe how annoying this was. I did a workaround.


improve hex and octal number handling of toInt32/toInt64 method in String type

in current implementation, the following code is assertion failure.
assert "0xFFFFFFFF".toInt32()! == -1
assert "0xFFFFFFFFFFFFFFFF".toInt64()! == -1L

related to #386


Split leaderboard into public and private

After a discussion with G. Planchat, he told me that the leaderboard could be frustating for contributors because they cannot earn as much badges as we can, and so do not have a chance be in the leaderboard.
We maybe could make two leaderboards: one for internal people and one for all people but counting only public badges.

To be closed. I think it is done.
ping @grena

No unfortunately, the leaderboard take all badges, regardless the tag :(
So this issue is still relevant!


how to  botkid work

How can I modify the text of the retaliation?

I do not understand the question, could you please explain more what you are trying to do?

https://github.com/howdyai/botkit/issues/1004 please dont create more than one issue


when i update the lvgl to the newest version, many errors!!

1、lv_indev_get_disp not define
error as:
win32drv.obj : error LNK2019: 无法解析的外部符号 lv_indev_get_disp，函数 lv_windows_get_indev_window_handle 中引用了该符号
2、not show ui, tips:
_lv_display_refr_timer: No draw buffer lv_refr.c:350

new verison has no lv_indev_get_disp,but in win32drv.chas the function:

Because LVGL's new main branch changed some ABI. I need to follow. Please wait.
Kenji Mouri

https://github.com/lvgl/lvgl/commit/e10f573150a95822df6422541bcc9ea1238f098a#diff-35a05ebbf5595318ed115bcabc7f70e6ae0b4f1f3ace3629e154a7434fe680f1
It looks the yesterday's LVGL mainline changed a lot of API namings.
Kenji Mouri

https://github.com/lvgl/lv_port_pc_visual_studio/commit/aa94c0504b8e1a1836d424723c46044cba5f5ee6
It should be fixed after this commit.
Kenji Mouri

still error:
when i update the newest version, open the .sln，then errors:
c1 : fatal error C1083: coould not open: “E:\lvgl\git\lv_port_pc_visual_studio-master\LvglPlatform\lvgl\src\draw\renesas\dave2d\lv_draw_dave2d_uitls.c”: No such file or directory

still:

still:

I don't think you have updated the newest version of this repository.
Because this old issue I have fixed.
Kenji Mouri

With the GitHub search feature, you will know I fixed this issue.
Kenji Mouri

Close this issue due to no further replies.
Kenji Mouri


Tuning up the docs page in time for the Picturefill 3.0.0 pre-pre-pre alpha

I want to do some serious reorganization on this page, but this is the overall vibe. Bunch of stuff missing, which I’ll just kind of wing as I put it together. Image is maybe FPO, but I kinda like the idea of shouting-out the images that’ve been with Picturefill since day one.

Alternate footer courtesy of @isaacdurazo:

How's this goin' @wilto? Need any help with this?

Finally have a chance to dig into it today.


Add RawAudio class and 'save to wav'

Following messages from #680
The 'save to wav' is my own simple implementation, using file specs, and hex viewer of a generated wav file.
Below the changes :

added RawAudio class, with .save(path) (support browser, webworker and nodejs)
modified some audio pipeline, to return RawAudio object
added properties isBrowserEnv and isWebworkerEnv to env

Example use :
const synthesizer = await pipeline('text-to-speech', 'Xenova/mms-tts-eng');
const output = await synthesizer('Hello, my dog is cute');
output.save("audio.wav");

Thank you @xenova for the review.
Here's the changes I have made :

split into two functions : toBlob() and save(path)
check type in constructor()
in save(), check running environment first before proceeding
reduce memory footprint, by using new Blob([wav_header, audio]) instead of allocating additional TypedArray new Uint8Array(buf_size + wav_header.length)

Thanks! 🤗 Would you mind benchmarking your code against https://www.npmjs.com/package/audiobuffer-to-wav, which I used in a demo a few months ago.
Other than that, I like the abstractions you introduced for the RawImage and RawAudio classes, and this will be perfect to merge into the v3 branch for my musicgen demo I'm working on 🔥

I have added support for 2 channels audio + interleave.
interleave(keepOriginalValues) will use a new buffer of length * 2 (keeping original), or a new buffer of length * 1 (overwriting original audio data)
Below a quick benchmark, comparing with encodeWAV(samples) used in the demo.
function benchmark(){
    let i, input, output

    console.time('encodeWAV')
    for(i=0; i<20000; i++){
        input = new Float32Array(i).fill(i)
        output = encodeWAV(input)
        output = new Blob([output])
    }
    console.timeEnd('encodeWAV')

    console.time('RawAudio')
    for(i=0; i<20000; i++){
        input = new Float32Array(i).fill(i)
        output = new RawAudio(input, 16000)
        output = output.toBlob()
    }
    console.timeEnd('RawAudio')
}

/*
encodeWAV: 3216.6669921875 ms
RawAudio: 2702.23291015625 ms
---
encodeWAV: 3296.2138671875 ms
RawAudio: 2768.235107421875 ms
*/

encodeWAV is slower, since it's hard copy all audio values, into a new Buffer.
    for (let i = 0; i < samples.length; ++i, offset += 4) {
        view.setFloat32(offset, samples[i], true)
    }

Thanks again! Just letting you know this PR is marked for the next release :)

I have merged branch v3 #545 into this PR


No way to set a dynamic yDomain?

Hello, I am trying to force the minimum value of one of my linearCharts to be 0, so I am doing something like this:
yDomain: [0, d3.max(mytable, function(x) { return x.N; } )]
This initially sets a correct yDomain, but the yDomain doesn't get updated as new data come in.
For example if initially mytable's maximum value is 10, the yDomain is correctly set to [0, 10] - but then if mytable changes and the maximum value is only 5, the yDomain will still be [0, 10], instead of changing to [0,5].
Am I missing something?
Thanks!

Initially domain stores just two numbers, so if the data changes, domain should be reassigned:
$scope.options.chart.yDomain = [0, d3.max(mytable, function(x) { return x.N; } )];
$scope.$apply(); //this line may not required

@krispo thank you very much for your help! Is this something that should be patched in angular-nvd3, or that I should add in my code?
The domain changes dynamically with the data if I don't set it manually myself, do you think there would be any way to have it auto-refresh without having me refresh it manually in the controller?
Thanks a lot for your reply! Let me know if I can help!