[Feature Request]: Petition to use guoyww/AnimateDiff fork instead of animatediff-cli

I don't think I like the "bat activation" implementation, there are more better than using animatediff-cli. Using the guoyww's fork will make it more usable especially on google colab (EX. Linaqruf's AnimateDiff notebook).
You can choose that original fork, but there are more recommended choices

https://github.com/talesofai/AnimateDiff (forked repo, can do initImage)
https://github.com/dajes/AnimateDiff/ (fork repo, longer video duration)

I know you don't know about google colab stuff, you probably have to try one to understand them.

The original animatediff relies on too many old libraries and is not suitable for modification...

Well then that was disappointing, no use then. I still think the idea of smoothly mid changing ai animation scenes had VERY BIG potential.

94a5812


Inconsistent config file structure

Expected Behavior
I would expect the following:
 ProxyPrefix  string `flag:"proxy-prefix" cfg:"proxy_prefix"`

Current Behavior
All other config options are x_y apart from proxy prefix, see https://github.com/pusher/oauth2_proxy/blob/402ce6f0cb3501486b959a0e629ba3ad810d390a/options.go#L24
Your Environment

Version used:

Master image

We should fix this as it is an easy fix, it would be good to find a way to do it in a backwards compatible way though, else we should bump to version 4.
Shall have to have a think on that.

I would propose a walkthrough of flag vs cfg as some are inconsistent such as:
https://github.com/pusher/oauth2_proxy/blob/402ce6f0cb3501486b959a0e629ba3ad810d390a/options.go#L36


reconfiguration

Hi! Is there any way to change configuration after starting emulation? I need to change gamepad configuration. I know that this can be done through the menu, but can it be done through javascript?

Let me look into it, I would have to find a way to make retroarch reload its keybinds from the config file.

Hi, I also need that feature! 👍 BTW, could you explain a bit how did you "reverse-engineering" the generated core_libretro.js file ? I realy want to use that generated script to add more features like volume adjustment and so on.
You did a great job, right now you've coded the hardest part! 🎉

What do you mean, reverse-engineering? Please clarify.

I mean, how did you found the key steps and variables in order to properly load libretro, game data and config ?

For example :
https://github.com/BinBashBanana/webretro/blob/4c437f7a55cb2f6898fcf10d567fbc657435da6a/assets/base.js#L920-L925
Is there documentation somewhere explaining how the web version of RetroArch works ? Did you read the C code or the generated emscripten code or maybe both ? I'm just curious about the technique you used to tackle some of the main steps :)

Some of the code for creating this was already present here, but it was VERY outdated, so I had to change some things (for example, Emscripten changed Module.FS_X to FS.X, and the retroarch configuration file path changed from /etc/retroarch.cfg to /home/web_user/retroarch/userdata/retroarch.cfg). Most of these I had to figure out through trial and error.
Emscripten FS documentation can be found here for basic usage, and here for advanced functions.
RetroArch Documentation
RetroArch Repository
The web version of retroarch essentially functions the same as any other unix retroarch.
You may also see calls to functions such as Module._cmd_save_state(), these are direct calls to C functions, as specified in EXPORTED_FUNCTIONS here
https://github.com/BinBashBanana/webretro/blob/4c437f7a55cb2f6898fcf10d567fbc657435da6a/source/overrides/RetroArch/Makefile.emscripten#L76  and here, to have the actual function declarations
https://github.com/BinBashBanana/webretro/blob/4c437f7a55cb2f6898fcf10d567fbc657435da6a/source/overrides/RetroArch/frontend/drivers/platform_emscripten.c#L53-L86
The game data is loaded automatically by specifying the rom path as the first startup argument (because the rom file is always created in the filesystem as rom.bin, there is no need for it to ever change)
https://github.com/BinBashBanana/webretro/blob/4c437f7a55cb2f6898fcf10d567fbc657435da6a/assets/base.js#L939
For loading the cores, each core is separately contained in its respective .js and .wasm files. There is no emscripten-generated code present until the .js file is added to the page (via the getScript function).

P.S. It would be impossible to read and understand the emscripten-generated code ;)

You are awesome ! Thank you very much for the explanation ;)

@dimabubenok I found a way to update the configuration during emulation.
This is a bit obscure and I only tested it with the snes9x core.
FS.writeFile('/home/web_user/retroarch/userdata/retroarch.cfg', updatedConfig);
wasmTable.get(108)();

If you try this, please let me know if it is also available with the other cores ;)

It works nice on snes9x, although I didn't test the other cores, I think it is almost always a really bad idea to directly call unidentified wasm functions from javascript. I just did some digging, and I should be able to get something set up that should work nicely, such as:
Module._cmd_reload_config();

I'll keep you posted.

I figured it out! Expect a release with the ability to change keybinds and download screenshots within the next month, at the latest. If you have any suggestions, feel free to leave them here!

It sounds promising. Waiting for an example

I'll send a screenshot when I have it looking nicer :)

Preview for the keybind manager

I'm getting window.Module._cmd_reload_config is not a function. Did you update the C code to export that new function ?

I did, but I haven't committed the update yet. On my end, the keybind manager is done, and I'm working on the screenshot one. I will send a screenshot of it when I have it looking better.

I'm facing a keybinding issue : I can't bind any of the numpad keys. Are you able to do this or do you know what type of value RetroArch needs ?

me a python starter: wtf is this

ye i also know javascript enough to make a calculator in terminal but thats it

@NicolasRoehm keypadX (e.g. keypad1)

Thank you very much, it's working ! 😍 Now I'm seeing that RetroArch is QWERTY only 😭

That's too bad :( however it is likely that it won't be a problem on the WASM emulator, because of how the keys are handled.

I might have to extend my deadline, I have been pretty busy with school.

Preview 2

Sorry it took me so long... But 6.4 is finally out! I finally got a chance to work on it during the break, and I also added a save/state manager. Enjoy!


Update sk.json

Coverage remained the same at 92.941% when pulling 560805afa16a4ea6bd3830b280889d0afef10660 on ja-som:patch-7 into f5b867788b937fc31793395858dd51c12df38670 on MarshallOfSound:master.


component gets initialised with null prop if ID is not defined in graphql schema?

The issue Im having is that my query's get executed, and I see a response coming back from the server. But my component does not receive the data via the props. At runtime it throws the following warning:
createFragmentSpecResolver: Expected prop streets to be supplied to Relay(SenderPage), but got undefined. Pass an explicit null if this is intentional.
The only difference I see from the example is that my schema does not specify IDs
so where the example has fragments like this:
fragment TodoApp_viewer on User {
id
...TodoListFooter_viewer
}
my fragments are similar but there is no viewer object and it misses the id on top. Is this mandatory for it to work? Or am I doing something else wrong.
query AppFetchStreetQuery($countrycode:String!, $postalcode:String!) {
...ListPage_streets
}

Once you're rendering, this library will have the same behavior as using a <QueryRenderer> in Relay normally. Most likely you are actually forgetting to pass down the prop or something.


[20.03] backport nvidia driver updates

Motivation for this change
Closes: #91054
Things done

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

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] 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.

@GrahamcOfBorg build linuxPackages.nvidia_x11_legacy340
Not sure if there are more attributes to build and I couldn't find a test

@GrahamcOfBorg build linuxPackages.nvidia_x11_legacy390 linuxPackages.nvidia_x11_legacy440
I hope these are the correct attributes...

Ah right, ofBorg won't build them because it's marked as unfree...


Pasrsing files (FormData)

Do you plan to support parsing Content-Disposition headers?

Yes it should probably be part of the library similar to ie helper for processing chunked encoding to be usable in at least most common cases without the need to write own each time.
Slicing through multipart/form-data should be definitely there, parsing form parts headers too (parseHeaders should handle this already so we just need to use it in a form-data parser)

Hi @tchaloupka !
httparsed is a very useful and high quality library! I have integrated it into my new project, Archttp, and added the body parsing part.
However, in the process of use, some code had to be modified to integrate.
In the future version I will make 'httparsed' an external dependency.
Thanks again!


Tile layer does not show in the samples sometimes

Mapsui Version
In the branch of this PR https://github.com/Mapsui/Mapsui/pull/2252 and probably today's master, did not test, which is before 5.0.0-preview.1.
Mapsui Platform
Maui Android
Device
RealMe C2
Describe the bug
Sometimes I don't see the openstreetmap map in the samples, after loading another sometimes I do see it again, sometimes. If I put a break point in the debugger. I get this exception:
An error occurred while fetching data. Exception: Cannot access a disposed object. Object name: 'AndroidMessageHandler'.
To Reproduce
Run Mapsui.Samples.Maui and go through a few samples.
Additional context
Looks like this bug:
https://github.com/dotnet/runtime/issues/93252

Shouldn't this be solved with the 8.0.100 version we got on Nov. 14th?

Yes it should. This PR upgrades the packages https://github.com/Mapsui/Mapsui/pull/2291. Did not test it yet.
btw, our global.json in the root still points to 7. This should be upgraded to 8. We probably also need to explicitly install 7 from the Scripts/InstallDependencies.cmd script. It currently only installs MAUI 6. And I think we need to remove MAUI 6 because it is not supported anymore.

I assume this is fixed with the release of .NET 8.


[1.12.2] Server crashing with ae2stuff <IP_ADDRESS> and AE2 RV5 alpha 8

My 1.12.2 server keeps crashing after updating AE2 vom alpha 7 to alpha 8:
Here is the crashlog:
https://pastebin.com/bWL5HJXK

As a quick note, we moved the factory methods for grid related objects into their own helper (AEApi.instance().grid()).
This is mainly intended to improve our injection annotation in the future. In particular to avoid reflecting from the API into the core or being able to inject just the required helper into another mod.
As we probably are not able to provide a real DI, it will most likely be limited to injecting them into static fields/methods or similar to JEI by instantiating the class and then passing the helper into it and leave it to the mod to handle anything afterwards.
The static access will of course still stay, it will just be an alternative for some cases.
Also due to the changes to StorageChannel, you will have to update the pattern encoder to use the API method AEApi.instance().storage().getStorageChannel( IItemStorageChannel.class ).createStack( itemStack ) not the internal one. (Avoiding these long chains is another reason..)

Thanks for the report, i'll try to make an update tomorrow.

Version <IP_ADDRESS> will be out in a sec with support for alpha 8.

Just for the future. If you want, I could create a PR for smaller API changes.
Also IAEItemStack.createItemStack() already creates a clean copy, so no need to copy it again.

I won't say no to any offers of help, but don't worry about it too much.
I'll remove the unnecessary copy.


Bug

Describe the bug
Description of the bug here.Hey Taiko the Test 3 was way cooler than the Test 2, but I encountered an error which worth to be looked at. I tried transactions which displayed failed transaction but later went through, I hope this will be fixed in order not to have users perform transaction multiple times.
Steps to reproduce
Steps to reproduce here. I feel the bug or failed messages be checked out subsequently to avoid errors
Additional context
Additional context here.

OK that's great to know
On Mon, 12 Jun 2023, 5:00 pm Francisco Ramos, @.***>
wrote:

Hi @Possibledan https://github.com/Possibledan , this happens because
the Relayer managed to claim it right before your manual claim when
through. You said the funds got claimed, correct?, could you please next
time send also the transaction hash? or the link you see in that error
message where it says: Click here to see more on the explorer. That could
be very valuable to us
—
Reply to this email directly, view it on GitHub
https://github.com/taikoxyz/taiko-mono/issues/13917#issuecomment-1587624013,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A47MNN6FTIDW5F6ER5D566DXK44INANCNFSM6AAAAAAY57YR7A
.
You are receiving this because you were mentioned.Message ID:
@.***>


stack giving errors

*provider.local: version = "~> 1.2"
*provider.null: version = "~> 2.1"
*provider.oci: version = "~> 3.86"
*provider.tls: version = "~> 2.0"
Terraform has been successfully initialized!
You may now begin working with Terraform. Try running "terraform plan" to see
any changes that are required for your infrastructure. All Terraform commands
should now work.
If you ever set or change modules or backend configuration for Terraform,
rerun this command to reinitialize your working directory. If you forget, other
commands will detect it and remind you to do so if necessary.
Refreshing Terraform state in-memory prior to plan...
The refreshed state will be used to calculate this plan, but will not be
persisted to local or remote state storage.
data.null_data_source.vpus: Refreshing state...
module.worker.data.null_data_source.hostname_labels[3]: Refreshing state...
module.worker.data.null_data_source.hostname_labels[0]: Refreshing state...
module.worker.data.null_data_source.hostname_labels[1]: Refreshing state...
module.worker.data.null_data_source.hostname_labels[2]: Refreshing state...
data.oci_identity_availability_domains.ADs: Refreshing state...
data.oci_identity_regions.filtered_regions: Refreshing state...
data.oci_core_vcn.vcn_info: Refreshing state...
data.oci_core_subnet.cluster_subnet: Refreshing state...
data.oci_core_subnet.utility_subnet: Refreshing state...
data.oci_core_subnet.bastion_subnet: Refreshing state...
data.null_data_source.values: Refreshing state...

Warning: Interpolation-only expressions are deprecated
on iam.tf line 7, in resource "local_file" "key_file"
7: 	content = "${tls_private_key.key.private_key_pem}"
Terraform 0.11 and earlier required all non-constant expressions to be
provided via interpolation syntax, but this pattern is now deprecated. To
silence this warning, remove the "${ sequence from the start and the }"
sequence from the end of this expression, leaving just the inner expression.
Template interpolation syntax is still used to construct strings from
expressions when the template includes multiple interpolation sequences or a
mixture of literal strings and interpolations. This deprecation applies only
to templates that consist entirely of a single interpolation sequence.
(and 166 more similar warnings elsewhere)
Warning: Quoted type constraints are deprecated
on variables.tf line 312, in variable "CentOSImageOCID"
312:   type = "map"
Terraform 0.11 and earlier required type constraints to be given in quotes,
but that form is now deprecated and will be removed in a future version of
Terraform. To silence this warning, remove the quotes around "map" and write
map(string) instead to explicitly indicate that the map elements are strings.
(and 2 more similar warnings elsewhere)
Error: Invalid index
on modules/network/outputs.tf line 2, in output "vcn-id"
2: 	value = "${oci_core_vcn.cloudera_vcn.0.id}"
|----------------

| oci_core_vcn.cloudera_vcn is empty tuple

The given key does not identify an element in this collection value.
Error: Invalid index
on modules/network/outputs.tf line 6, in output "private-id"
6: 	value = "${oci_core_subnet.private.0.id}"
|----------------

| oci_core_subnet.private is empty tuple

The given key does not identify an element in this collection value.

this error only when we choose existing VCN
is the existing VCN must named as cloudera_vcn too otherwise will give the above error !!


How do things like yarn installs work within nodenv?

Context: I don't have a system version of NodeJS installed and I use node exclusively through nodenv.
I'd like to use yarn but when I run $ npm i -g yarn, after it completes I still don't have access to a $ yarn command in my terminal. I do have access to $ npx yarn, but I'm not sure if that's ideal or at what level yarn caching would work etc. in this scenario.
Questions:

how do I get yarn to be a command at the global level?
what happens if I uninstall a given version of npm? (do all yarn-installed modules go away from that version or is there some global thing outside of nodenv)?

Closing this issue, it looks like things resolved themselves once I rebooted my system.
For context for anyone else struggling with this, the binaries for global installs get installed and then aliased in the $(nodenv root)/versions/<version>/bin/ directory where they're then accessible via the $PATH.
e.g. for node v16.1.0 my binaries were in ~/.nodenv/versions/16.1.0/bin

@vindard Glad you got it sorted.
TL;DR Mostly likely, you just needed to run nodenv rehash after installing yarn. That wouldn't require a system restart (nor does it require a shell restart).
In more detail:
Node's default location for global modules is within its own installed directory. So node 16.1.0 would install its global modules to $(nodenv root)/versions/16.1.0/bin. (Notice to homebrew users: homebrew customizes this location such that global modules are preserved across node upgrades; hence why the behavior is different between a homebrew-installed node and one installed using a version manager.)
nodenv then creates shims for all executables found in all of your nodes' bin directories. These shims are added to your path by the nodenv init line that is added to your shell init script. (run nodenv shims to see them all)
These shims are what get executed when you run a "global" command like npm or yarn or even node itself. The shim is what is responsible for invoking nodenv and determining which of your node versions to "delegate" the command to.
What most likely happened to cause you trouble: you didn't have a yarn shim created after installing yarn, so the yarn executable wasn't in your PATH.
Something has to trigger nodenv to tell it when a new global module is installed, so that nodenv can create a shim for the executable(s). When you install a new node, there are callback hooks that run to make shims for all the executables in bin/; but when you later install global modules, nothing has told nodenv to create a shim for the yarn executable. In short: after installing a node module globally: run nodenv rehash. That will create shims for all the executables it finds in bin (including any newly installed global modules.
There is a plugin that used to make this unnecessary: https://github.com/nodenv/nodenv-package-rehash. It worked by running nodenv-rehash automatically using npm's global "postinstall" hooks. Unfortunately, npm has deprecated those hooks with npm7, so I don't believe that plugin works any more. It's also worth noting that now the yarn shim exists, you won't need to run nodenv rehash after installing yarn in any other nodes; but it's still good practice to run rehash after every global module install/uninstall.
Also, FWIW, there's another nodenv plugin https://github.com/nodenv/nodenv-default-packages which allows you to define a list of packages to install (globally) in every node. yarn is a prime candidate for that list. This way, whenever you install a new version of node, the default-packages plugin will ensure yarn is installed (globally) in that new node.

Thanks @jasonkarns, this all really great info!
I did install the rehash plugin as well, but I'll have to keep in mind that it may not always automatically work. Good to know that that's what rehash does 👍🏽


Portals created in the nether and are far away from any active portals in the overworld do not work at all (even when linked using the command)

Information

Server version: 3352-Spigot-dbf4938-f49e9d1 (MC: 1.18)

Full output of /mv version -p: Not possible, prints "Error posting to service" error in the console.

Server log: latest.log

Details
I was able to reproduce my issue on a freshly setup and up-to-date server with the latest version of Multiverse plugins with no other plugins and with no kinds of other server or client mods.
Description
Nether Portals made in the nether far away from any linked overworld portal will not work at all.
Steps to reproduce
Make a nether portal in the nether, far away from any active nether portal in the overworld. It won't be able to teleport you.
Expected behavior
For the portals to work like normal nether portals do using the plugin.
Screenshots

I couldn't reproduce the behaviour you mentioned. Try running /mvv -p again, and if it gives you an error send your latest.log file.


The terminal cannot input Chinese input method

Steps to Reproduce:

Version 1.53

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

Duplicate of #115814


[Feature]: use process env on ci/cd

What area(s) will this request affect?
App, Extension, Theme
What type of change do you want to see?
New feature
Overview
how we can use different vars in ci cd deploy?
Motivation
how we can use different vars in ci cd deploy?

Hi @michaelr231, perhaps you can be a bit more detailed about what you'd like?
You should be able to use any ENV variables you want, though the CLI only looks for certain ones which are relevant to its function.

Hi @michaelr231, perhaps you can be a bit more detailed about what you'd like?
You should be able to use any ENV variables you want, though the CLI only looks for certain ones which are relevant to its function.

for example
i use shopify cli to deploy checkout extensions
and i have 3 different envs: dev, stage and prod
each env has their own url
i want to ues github action deployment file for ci cd but i need to pass
the relevant url
i cannot find a way to pass variables like process env or even use secrets to pass them into shopify app deploy command
hope it was clear
Thanks


Support for parts of the iCalendar format

Would it make sense to support parts of the iCalendar specification in this library?
Specifically, I currently need multiplatform support for iCal's recurrence rules: https://icalendar.org/iCalendar-RFC-5545/3-8-5-3-recurrence-rule.html
Or, is this too much out of scope for this library and that would make more sense in a separate MP Kotlin iCal library?

If they are standard outside iCalendar, why not. If they are iCalendar specific, maybe we can add a contrib folder or something with iCalendar.kt with extension methods or classes so people can copy to their own projects? Or maybe add an artifact like klock-ext.
If someone creates code for that and makes a PR I don't see any problems including it here one way or another


Make results table more compact

Is your feature request related to a problem? Please describe
Make the results table more compact so that it fits better on smaller screens and optimized to show highest priority items.
Suggestions

Remove Node column
Update ordering of columns by priority
Remove unnecessary columns (e.g. checksum)
Rename “# of files” to “files”
Remove Total Size and abbreviate "Node"

New observation.  Search in the Obs4MIPs space and look at the "Additional" column.  I think Tom took a "general" approach by using the CoG code as a reference. I had forgotten about this case when we went over the columns, but hopefully not a major impact, but something to be aware of.

I just tried now and looks like the 'Additional; column is empty with obs4mips.
I can try to add some logic so that if the content is empty, then the column won't show up? Or is there something I should do so that content shows up there properly?


pg: WHERE ... id = ANY($1::TYPE[])  generates query with wrong returning named parameter

Version
1.18.0
What happened?
When generating an update query using a WHERE ... IN id = ANY($1::TYPE[]) statement, I expected to get a function that have a named parameter based on the column in the query which is id, instead of getting id the result from generated function is Column1, see playground link
Current workarround,
I guess I can wrap the generated function manually, but will be better if this fixed, considered if this one is a legit bug, but if not please tell me where I went wrong
Thank you 🙇
Relevant log output
No response
Database schema
CREATE TABLE authors (
  id   BIGSERIAL PRIMARY KEY,
  name text      NOT NULL,
  bio  text
);

SQL queries
-- name: UpdateAuthor :exec
UPDATE authors
SET name = $2
WHERE id = ANY($1::BIGSERIAL[]);

Configuration
No response
Playground URL
https://play.sqlc.dev/p/693bb4a6f7a05286b91f04bee63aae12cadbfaf5eab82e459b8886d0b2d8736f
What operating system are you using?
Linux, macOS
What database engines are you using?
PostgreSQL
What type of code are you generating?
Go

Please use named parameter.
https://play.sqlc.dev/p/9826b0fe13db496592fc778302961391e31a9562a7b08a08e9508a746a8a54cd

Thank you @orisano , I'' close the issue


Source code indicates different license than README.md

It's very minor, but: the comment at the beginning of ripser.cpp indicates that that file is licensed under the GPL, while README.md says LGPL.

Thanks for pointing this out, Gard.  It's fixed now both in dev and master.  Unfortunately GitHub still detects the GPL but not the LGPL addition.


Replace native.package_name() with package_name in BUILD file.

The notation native. will not be available in BUILD files in the future.

cc @ittaiz
The Travis test seems to fail randomly.

@laurentlb sorry for the latency. This looks good to me.
The reproducibility test is flakey on our jmh benchmark rules for reasons we haven’t run down yet. It only started in the last couple of months. Anyway.


handle blank lines in the input

it looks like the goreleaser config might need udpating to handle more recent versions, because that isn't working on my machine.  But I can build and run this with go build -o . -v ./...

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

4 of 5 (80.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.2%) to 89.13%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

cmd/routesum/main.go
4
5
80.0%

Totals

Change from base Build 622044836:
-0.2%

Covered Lines:
328

Relevant Lines:
368

💛  - Coveralls

Hi! I'm on board with the intent. Implementation looks good. Thanks!


Add a URL Reference Feature

One feature I'd like to see is an option to use the reference structure for URLs. Here's an example where I think that would be really nice:
The following table contains all the existing languages in the repository that start with the letter A:

| Language                                                                         | Article(s)                                                             | Issue(s)                                                                                | Test(s)                                                                        | # of Snippets |
| -------------------------------------------------------------------------------- | ---------------------------------------------------------------------- | -------------------------------------------------------------------------------------------------- | ------------------------------------------------------------------------------ | ------------- |
| [Abap](/TheRenegadeCoder/sample-programs/tree/main/archive/a/abap)               |                                                                        | [Here](/TheRenegadeCoder/sample-programs/issues?utf8=%E2%9C%93&q=is%3Aissue+is%3Aopen+abap)        |                                                                                | 1             |
| [Ada](/TheRenegadeCoder/sample-programs/tree/main/archive/a/ada)                 |                                                                        | [Here](/TheRenegadeCoder/sample-programs/issues?utf8=%E2%9C%93&q=is%3Aissue+is%3Aopen+ada)         | [Here](/TheRenegadeCoder/sample-programs/tree/main/archive/a/ada/testinfo.yml) | 1             |
| [Agda](/TheRenegadeCoder/sample-programs/tree/main/archive/a/agda)               |                                                                        | [Here](/TheRenegadeCoder/sample-programs/issues?utf8=%E2%9C%93&q=is%3Aissue+is%3Aopen+agda)        |                                                                                | 1             |
| [Algol68](/TheRenegadeCoder/sample-programs/tree/main/archive/a/algol68)         | [Here](https://sample-programs.therenegadecoder.com/languages/algol68) | [Here](/TheRenegadeCoder/sample-programs/issues?utf8=%E2%9C%93&q=is%3Aissue+is%3Aopen+algol68)     |                                                                                | 1             |
| [Apex](/TheRenegadeCoder/sample-programs/tree/main/archive/a/apex)               |                                                                        | [Here](/TheRenegadeCoder/sample-programs/issues?utf8=%E2%9C%93&q=is%3Aissue+is%3Aopen+apex)        |                                                                                | 2             |
| [Applescript](/TheRenegadeCoder/sample-programs/tree/main/archive/a/applescript) |                                                                        | [Here](/TheRenegadeCoder/sample-programs/issues?utf8=%E2%9C%93&q=is%3Aissue+is%3Aopen+applescript) |                                                                                | 1             |
| [Arkscript](/TheRenegadeCoder/sample-programs/tree/main/archive/a/arkscript)     |                                                                        | [Here](/TheRenegadeCoder/sample-programs/issues?utf8=%E2%9C%93&q=is%3Aissue+is%3Aopen+arkscript)   |                                                                                | 1             |
| [Astro](/TheRenegadeCoder/sample-programs/tree/main/archive/a/astro)             |                                                                        | [Here](/TheRenegadeCoder/sample-programs/issues?utf8=%E2%9C%93&q=is%3Aissue+is%3Aopen+astro)       |                                                                                | 1             |
| [Autohotkey](/TheRenegadeCoder/sample-programs/tree/main/archive/a/autohotkey)   |                                                                        | [Here](/TheRenegadeCoder/sample-programs/issues?utf8=%E2%9C%93&q=is%3Aissue+is%3Aopen+autohotkey)  |                                                                                | 1             |
| **Totals**                                                                       |                                                                        |                                                                                |                                                                                | 10            |

< [Previous (Z)](/TheRenegadeCoder/sample-programs/wiki/Z) | [Next (B)](/TheRenegadeCoder/sample-programs/wiki/B) >

This table is absurd. We could simplify it dramatically by converting these long links to numbers.

I already have an idea for how this would work. I think we would perform this in a single pass. Any time we encounter a link, we note down what it is in a dictionary while assigning it a number based on the current size of the dict. Then, when we finish passing over the doc, we dump that dictionary at the end of the document. There is going to need to be a new syntax for this, so we might have to include a new reference class. Alternatively, we could have InlineText render itself as a reference. Maybe we can add a function like as_ref(value) which renders a reference for that particular InlineText element. Lots to think about.


Palette Swapping

I think that this is a great shader, but it would be nice if there was a way to quickly swap between pallets to create a strobing or pulsing effect.

Can you show a GIF of how it would look like?

Sure.

Something like this:

Looks doable with a texture array.

How would I go about coding that for a shader?

How would one go about adapting the shader to use a texture array then?

I would Also settle for this:

Would a texture atlas work as well?

I finally had time to give this a proper check.
My original shader is about palette cycling, i.e. you have one palette and just cycle the indices.
What you request is palette swapping, which is something different. Cycling is more performant, because it doesn't involve color comparison.
That said, I made a simple (?) swapping shader based on your request:
Swapping.zip
You just need to provide a palette and FPS and it will use the first row of palette as reference and automatically animate the sprite. It's more expensive than my original shader, but I tried to optimize it so it's not bad.

I tried it out but it doesn't seem to be working.

I just tried it out and it works!

I just tried it out and the only nitpick I really have is that I would prefer to be able to map the palettes to a completely monochrome (black and white) image, but overall it's very good, thank you.

This is more what I had in mind:

Here's updated shader:
PaletteSwap.shader.txt
I added a skip_first_row property. If you enable it, the first row will only be used as reference and excluded from animation. This way you can just put grayscale on top. Also if you use the skip with only 2 rows, you get a regular non-animated palette swap.

Thanks, You certainly have talent.

Okay, just one last issue; When I apply use this to multiple sprites individually I can't use a different palette for each sprite.

It doesn't seem to work with tile-mapped backgrounds.

If I could just ask for one last feature; Would it be possible to make the shader turn a specific color completely transparent? Sort of like a chroma key?

Change
COLOR = vec4(texture(palette, uv).rgb, color.a);
to
COLOR = texture(palette, uv);
and it will use the alpha from the palette.

It's perfect.

Any plans for releasing this as a new asset? I know I am not the only one who has been looking for something like this.

I submitted the asset, it will appear under this link when approved: https://godotengine.org/asset-library/asset/4715
And here's the repo: https://github.com/KoBeWi/Godot-Palette-Swap-Shader
The shader is the same, except I added an option to use the alpha values or not. I might later also add a simple script that will generate palettes from PNGs.
In any case, this can be now closed.


Pratham/pbis/1251070 correct failing build

I think to make the build pass in CI we need to migrate away from Appveyor to Azure Devops. The publish to Nuget also no longer works because they changed it. I am no longer working on C# codebases so I will likely not have time to do this.
@ganeshlondhemindbodyonline If you or your team would like to maintain this repository I can add you as maintainers.


etcdserver: internal request union

prepare for v3 api request
The wire types of Request and InternalRaftRequest are different. So we do not need the magic number.
/cc @yichengq @barakmich @philips

I would rather see us move to a new API using an upgrade mechanism driven through consensus instead of hacking around this with guessing. Essentially I think that we need to make the Metadata proto look like
message Metadata {
	required uint64 NodeID    = 1 [(gogoproto.nullable) = false];
	required uint64 ClusterID = 2 [(gogoproto.nullable) = false];
	optional uint64 CurrentVersion = 3 [(gogoproto.nullable) = false];
	optional uint64 MaxVersion = 4 [(gogoproto.nullable) = false];
}

Then at some point when all members have MaxVersion that is greater than CurrentVersion we move up to the largest agreed upon MaxVersion of the internal protocol. That way we can gate things like say the ACL stuff being available to the client API until all members support that protocol.
We talked about this before in Documentation/internal-protocol-versioning.md but that doesn't apply anymore and the file should probably be removed.

@philips I think we are talking about different things here.
The first thing here is to  make etcdserver be able to understand potentially unlimited proto message.
The approach I took here is to use a union.  What to be included in the outer proto is another thing needed to be discussed.

The version coordination is a separate thing. And the metadata proto is not used in the wire message at all. It is used by WAL to identify the local member/cluster right now.

@philips @barakmich @yichengq Can you please take a look? We want to add v3 request proto to the union.

@yichengq I am going to merge this if there is no objection.

lgtm


Cannot install or Run 3.9.x or higher versions

I can`t run the python version even with modified version provided by this repo.
Can you help?
With embedd, the dll error ocurrs.
With installer, setup failed screen appears

Check the comments at https://github.com/adang1345/PythonWin7/issues/5 and https://github.com/adang1345/PythonWin7/issues/3 and see whether they help.


Test Eorror with Metrics

.\match_common_test.go:56:18: cannot use &testMetrics{} (value of type *testMetrics) as Metrics value in argument to NewLocalMatchRegistry: *testMetrics does not implement Metrics (missing method GaugeStorageIndexEntries)
.\matchmaker_test.go:1658:181: cannot use metrics (variable of type *testMetrics) as Metrics value in argument to NewRuntime: *testMetrics does not implement Metrics (missing method GaugeStorageIndexEntries)

I added fixes for this in my PR: https://github.com/heroiclabs/nakama/pull/1128

Fixed in 2f1882387b375742360ada136ac69c584e55265d ahead of the 3.19.0 release.


Fix ArsenalImageMounter dependency

This fixes issue #852.

It is a missing dependency but I didn''t get the time before to investigate. There is 2 options to add this dependency avoiding this message :

DriverFiles.zip file contains a folder DriverFiles\Win10\x64 with 6 drivers 5 have to be installed.
Launch the application and click yes with autohotkey.

@mandiant/commando-vm opinions?

dotnet-6.vm is failing to install in our test suite because of Arsenal Image Mounter requiring a different version from dotnet-6.0-desktopruntime than the one in dotnet-6.vm. @naacbin do you know when you will have time to rebase (sorry I shouldn't have merged the update of this tool) and add the missing driver in this PR?

dotnet-6.vm is failing to install in our test suite because of Arsenal Image Mounter requiring a different version from dotnet-6.0-desktopruntime than the one in dotnet-6.vm. @naacbin do you know when you will have time to rebase (sorry I shouldn't have merged the update of this tool) and add the missing driver in this PR?

I will rebase today, but I don't know when I will have time to add the missing drivers. So, we should merge this PR as it's only one click for user to install drivers if needed. In the same time, we open an issue or add it to the existing one for the drivers dependency.

@naacbin I think you have pushed the wrong changes.
It is ok for me to add the missing driver later. I have tracked it in https://github.com/mandiant/VM-Packages/issues/910

@naacbin I think you have pushed the wrong changes.
It is ok for me to add the missing driver later. I have tracked it in #910

Ahah, indeed it's fixed now

Thanks @naacbin! :rocket:


Add zgenom clean

This removes all repos which aren't used by the current configuration.
Use this to quickly get rid of all unused plugins
Closes https://github.com/tarjoilija/zgen/issues/103

This implementation is faulty since the branch name should also be checked.


"deis registry:set" creates a new release (AWS ECR)

I have setup a CI/CD pipeline that builds a docker image, pushes it to my private AWS ECR registry and deploys to deis using deis pull.
However, since the "the docker login command to authenticate to an Amazon ECR registry with an authorization token that is provided by Amazon ECR and is valid for 12 hours", I have to run deis registry:set password=${registry_pass} before each pull.
Since deis registry:set creates a new release each time, it slows down the build significantly and causes 2 deployments back to back. I was wondering if there is any workaround for that. I know I could set the AWS ECR registry but another alternative would be nice since I feel like doing that will be a PITA.

I'm pretty sure this is what deis/registry-token-refresher is supposed to solve, which was introduced shortly after deis registry:set was introduced. Does it refresh the tokens for you, or is there an issue we need to fix?

It's in the README:

The Registry Token Refresher service creates the imagePullSecret and updates it at regular interval of time for each app(namespace) created by Deis Workflow. The secrets are used by dockerbuilder and controller for authenticating with the private registry.

It should be doing this automatically. If it isn't on v2.10.0 then can you file an issue there?

I'm going to close this ticket due to inactivity, but please re-open if this still needs to be addressed. Thanks!


How to create an iterator that retains order of dataset?

Is it possible to have an iterator for a translation dataset that does not do any shuffling, i.e. retains the original order of the dataset on sentence level? I understand this will be less efficient.
In particular, I have a translation dataset consisting of 5M sentence pairs:
dataset = task.dataset("train")
With this dataset I initialise an iterator:
itr = task.get_batch_iterator(dataset=dataset, max_tokens=args.max_tokens, max_sentences=args.batch_size, shuffle=False,         max_positions=utils.resolve_max_positions(task.max_positions(), *[m.max_positions() for m in models],),         ignore_invalid_inputs=args.skip_invalid_size_inputs_valid_test,         required_batch_size_multiple=args.required_batch_size_multiple, seed=args.seed,         num_shards=args.distributed_world_size, shard_id=args.distributed_rank, num_workers=args.num_workers,         data_buffer_size=args.data_buffer_size,).next_epoch_itr(False)
In this case, itr does not retain the original order. Is there any way I can make sure that the order is not changed? If this is possible, I would be extremely grateful for a code-snippet that shows how to create an order-retaining iterator from dataset.
I have searched the docs, comments, and have used Google, but I have not been able to find an answer. I am not very familiar with fairseq, so I could be missing something obvious. Thanks for your patience and help!

I assume you are using TranslationTask.
So the "shuffling" is happening inside LanguagePairDataset
In other words, if you disable that and all .index_select(0, sort_order) below it,
you will get your original order.
--
If you do not wish to change the original code, so do I recommend not changing,
You will need to make a copy of both that_task.py and that_dataset.py, alter the copy and use --user-dir to make fairseq recognize your new implemented task.
If you are using other tasks, then you conduct a searching in that task.py.
Find out which dataset it is using and go find it.
 You can conduct a searching in fairseq_repository/fairseq/examples or the whole repository.
Though it won't tell you how to not shuffle data, it may tell you how to use user-dir, what is task? where does data be handled etc.
You are always welcomed to ask here anyway.

Thanks @gmryu, this solves my issue!


Sometimes metrics get returned for deleted HelmReleases

Describe the bug
We deploy some HelmReleases only for a short time (few minutes), and after removal of the HelmRelease, the corresponding resources get deleted as expected, but sometimes metrics (with status="Unknown") get still returned for the deleted HelmRelease. This triggers alerts in our monitoring system.
As a workaround the flux2 pods can be restarted, and afterwards the metrics for the deleted HelmReleases are no longer returned.
Steps to reproduce
Deploy a HelmRelease, wait for it to become ready. Then delete the HelmRelease, and after deletion check the returned metrics. The problem seems to show up sporadically, we don't know what exactly what triggers the problem.
Expected behavior
No metrics should be returned for deleted/non-existent HelmReleases.
OS / Distro
Azure AKS K8s Cluster with flux2
Flux version
flux version 0.31.1
Flux check
► checking prerequisites
✔ Kubernetes 1.22.6 >=1.20.6-0
► checking controllers
✔ all checks passed
We use a non-standard manifest, here are the used images:
ghcr.io/fluxcd/kustomize-controller:v0.25.0
ghcr.io/fluxcd/source-controller:v0.25.6
ghcr.io/fluxcd/helm-controller:v0.21.0
Git provider
Azure
Code of Conduct
I agree to follow this project's Code of Conduct

With me it's not so "sporadically"... Apply a HelmRelease and then deleting it triggers it 100% of the time.

This issue has been fixed in Flux 2.1


Polish the animation for TabViewPagerExperimental to match native curves

Any ideas on how to get started on this?


de-selecting of breakpoints via ui don't work

Another issue (not related to variables), is that I cannot de-select a breakpoint. I can delete the breakpoint from debug shell, but not untick it in the UI (the bp is still active even though not clinked in the UI).
Originally posted by @infn-ke in https://github.com/WebFreak001/code-debug/issues/373#issuecomment-1253551896

@infn-ke Please provide log output from a session where you stop on a breakpoint (possibly implied via stopAtEntry), create one via ui, then de-activate it, then delete it, ideally with a  info breakpoint on the debug command line before/after each step.
I guess the UI shows the deactivated breakpoint as deactivated and removes the deleted ones - but info break still shows them?

To this seems to happen when I enable or disable a breakpoint and target is running. Enable/disable in stopped mode seems to work. So maybe this works as expected or should added/removed breakpoints be reconciled between UI and gdb when target is stopped?

I see - that's #250 then.


Updated description of merge option

plasma-cli --help currently shows the wrong info for merge (it is showing the info for exit). This PR updates it to the same description used on README.md

oooooh great catch @jet86 thank you !


Multiple tags for 3.3.1

Hi guys, this might not be exactly an issue, but it is somehow confusing. Why there are 3 different tags for version 3.3.1 (two of them actually with a major set to 4)?
https://github.com/Acquisio/bootstrap-stylus/tree/3.1.1
https://github.com/Acquisio/bootstrap-stylus/tree/4.0.0
https://github.com/Acquisio/bootstrap-stylus/tree/4.0.1

We merged a breaking change in bootstrap stylus before releasing 4.0.0. Version was upgraded to 4.0.0 to support semver as not to break the implementation of all existing codebases that were using bs-stylus.
It also happenned numerous times in the past where we would release a matching version of Bootstrap, then it would contain bugs, this meant deleting the release and re-releasing it, which is bad practices as in theory it could mean that two different computers that run a project using BS Stylus, could have two different version of the library is one has npm-installed after a breaking-tag delete and re-release.
Hopefully that makes enough sense.
You can see the discussion in https://github.com/Acquisio/bootstrap-stylus/issues/72 if you are curious.

Thanks for the clarification! What if you add a VERY short changelog file and link it from the README? in that way these things stay documented for everyone to easily see them.

BTW, why not creating <IP_ADDRESS> and <IP_ADDRESS> releases like in previous cases as well? I'm just wondering what are you gonna do when BS 4 comes out?

Right makes sense.
as for x.x.x.x versions I stoped doing it because of:
https://github.com/Acquisio/bootstrap-stylus/issues/53
For the second part of you question, you can see this repo as : Boostrap 3.3.1 for Stylus, then someday it will be Bootstrap 4.0.0 for Stylus, the internal version is not important.
The release version is hard to match with the official version as explained in the example scenario I gave.
Lets say you are working and using 3.3.1 on your computer at the office.
Then we do a breaking change on 3.3.1 while trying to fix a bug.
We re-release 3.3.1 because we want to be on the version version as Bootstrap official.
When you get home a few days later and you did not realize that 3.3.1 was re-released, and you checkout your project on your home computer and then do npm install and for some reason nothing works because the release is currently bugged because it managed to pass our code-reviewing eyes, you wont be really happy.
Same goes for why I ended up bumping the version from 3.3.1 to 4 because I realized that if I re-released version 3.3.1 with the included breaking changes (a folder was moved/renamed) every user out there using the library would have had some work, however minor it is to do to make their project work again.
The semver system, if followed guarantees that no breaking changes will be introduced in a minor or patch version. Hence why I bumped the major version.
Hope this answers everything!

Gotcha, I didn't understand why semver would be broken adding a fourth digit. I get now it is just because of the idea behind major, minor and patch version numbers.
Thanks for the clarification!


wrangler was not installed successfully

wrangler was not installed successfully
/usr/lib/node_modules/@cloudflare/wrangler/node_modules/binary-install/src/binary.js:60       throw You have not installed ${this.name ? this.name : "this package"};       ^ You have not installed wrangler (Use node --trace-uncaught ...to show where the exception was thrown)

Happened the same to me on debian stretch. I supposed it was because my NPM install was done with APT.
I've tried another VM with Ubuntu 20.04, with Node installed from NVM and then:
`# npm -g install @cloudflare/wrangler
/root/.nvm/versions/node/v10.20.1/bin/wrangler -><EMAIL_ADDRESS>
<EMAIL_ADDRESS>postinstall /root/.nvm/versions/node/v10.20.1/lib/node_modules/@cloudflare/wrangler
node ./install-wrangler.js

internal/modules/cjs/loader.js:638
throw err;
^
Error: Cannot find module<EMAIL_ADDRESS>at Function.Module._resolveFilename (internal/modules/cjs/loader.js:636:15)
at Function.Module._load (internal/modules/cjs/loader.js:562:25)
at Function.Module.runMain (internal/modules/cjs/loader.js:831:12)
at startup (internal/bootstrap/node.js:283:19)
at bootstrapNodeJSCore (internal/bootstrap/node.js:623:3)
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>postinstall: node ./install-wrangler.js
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>postinstall script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
npm ERR! A complete log of this run can be found in:
npm ERR!     /root/.npm/_logs/2020-08-05T03_11_04_972Z-debug.log
`
Didn't work either.
Other packages like express install just fine:
`~# npm install -g express

<EMAIL_ADDRESS>`

So after a decent amount of time spent, I just grabbed the binary from here: https://github.com/cloudflare/wrangler/releases/download/v1.10.3/wrangler-v1.10.3-x86_64-unknown-linux-musl.tar.gz
And then the python example didn't worked on Debian 9 (probably due to python 3.5). I tried again in the Ubuntu 20.04 VM (where I've installed the binary directly too).
I tried to import json to serialize/deserialize from python and it shows the error:
        Can't import module 'codecs'
Having problem installing the tool and then it's not possible to do more than a hello world at python makes me think it's worth to rely on workers at the moment if you aren't trying to use javascript.

you might need to update node

I'm on node 15 and have the same issue.

npm install @cloudflare/wrangler
run npm run publish using example package json below, get the above error

example package.json:
{
  "private": true,
  "dependencies": {
    "@cloudflare/wrangler": "^1.11.0"
  },
  "scripts": {
    "worker:publish": "wrangler publish"
  }
}

I was having the same issue. I was unable to get the npm install to work, but I was able to get cargo install wrangler to work by installing perl.
I'm on Fedora Linux so sudo dnf install perl -y worked for me :)

@rodrigopv it looks like you're trying to run npm install -g @cloudflare/wrangler as root, since /root is appearing in your error message.
The wrangler npm installer does not support running as root, as npm runs the postinstall script as the nobody user to prevent malicious postinstall scripts from causing damage to your system. Please re-open this issue if you're unable to get the npm installer to work as a non-root user.

Hi,
I managed to solve the permission issue with ubuntu by doing the followings at below:

Open terminal and paste the following codes.
mkdir "${HOME}/.npm-packages"
npm config set prefix "${HOME}/.npm-packages"

Use nano editor
nano ~/.bashrc

Paste the following codes at the bottom => .bashrc
NPM_PACKAGES="${HOME}/.npm-packages"
export PATH="$PATH:$NPM_PACKAGES/bin"
export MANPATH="${MANPATH-$(manpath)}:$NPM_PACKAGES/share/man"

Finally
source ~/.bashrc
npm install -g @cloudflare/wrangler

Hope this helps. Happy coding....

Solved this by simply using yarn instead of npm.


Minor typo in Preface

I found a minor typo in the preface:

The highlighted word hould be "self-contained"?

Ahhhhh never mind. It's already raised.


Add an expermental alternative cpp backend

A implementation of cpp (need c++17). Wish to get a review! The generated cpp.h pass the clang-tidy check.

#10  In fact

I wish to add namespace in different module. So I wish if you can help me to finish it!
Also I wish if you can help me check the layout of struct/union in generate file.
Some of inspired by wasi-libc's tool but it have abi change in wasi-crypto so I make a simple cpp backend.

For current cpp backend I will:

Support return result as a Result(rust) like style as return value
Generate different namespace for each module
Adopt LLVM style's code
Require C++17 at least.
Support generate multiple files for each module (may not put in thie pull request).
Check the layout with static_assert


[Bug] prettier . --write does not run on .gjs/.gts files

🐞 Describe the Bug
prettier . --write does not run on .gjs/.gts files
🔬 Minimal Reproduction
I run prettier . --write
😕 Actual Behavior
It reformats all files, except for .gjs/.gts files. CLI output list all kind of files, like .js/.cjs/.mjs, but no .gjs/.gts
🤔 Expected Behavior
Should reformat  .gjs/.gts files.
🌍 Environment

prettier-plugin-ember-template-tag version: 1.0.2
ember-template-imports version: -

➕ Additional Context
When I run prettier path/to/component.gts --write then it does work! Also running prettier using VSCode does work correctly. So it seem only prettier . seems to ignore any .gjs/.gts files

I've reproduced this issue, and found that it does work when specifying a file, like npx prettier --write app/components/my-component.gjs, but doesn't work with directories, like npx prettier --write app/components.

My prettier command is now:
prettier -c "**/*.{js,gjs,cjs,mjs,json,hbs,ts,cts,mts,gts}"

:sweat_smile:
My changes: https://github.com/NullVoxPopuli/limber/pull/1312/commits/80433f77df98d4cc46c93f26110e1fb35c4ab50e#diff-fcf5729f4474308f357cea6ec1123607ce116df8a7a60516595fac5f5327e01bR33

Sorry for the annoying bug. :-(
Based on Prettier's documentation, prettier --write . should be working, but it seems like we actually need to do prettier --write --plugin prettier-plugin-ember-template-tag. I am finding other plugins using this hack too but no one seems to mention it's a hack?
I will update the documentation accordingly and open an issue on prettier.

https://github.com/prettier/prettier/issues/15351

Looks like this bug was introduced in prettier v3 and there is a fix in the works. More info at https://github.com/prettier/prettier/issues/15079
In the meantime at the plugin flag as I mentioned above.

Like @NullVoxPopuli commented on discord<EMAIL_ADDRESS>has been released this WE 🎉

README updated. I will re-open this if it's still borked.


DQN output is normalized?

Important Note: We do not do technical support, nor consulting and don't answer personal questions per email.
If you have any questions, feel free to create an issue with the tag [question].
If you wish to suggest an enhancement or feature request, add the tag [feature request].
If you are submitting a bug report, please fill in the following details.
If your issue is related to a custom gym environment, please check it first using:
from stable_baselines.common.env_checker import check_env

env = CustomEnv(arg1, ...)
# It will check your custom environment and output additional warnings if needed
check_env(env)

Describe the bug
The output of DQN should be the estimated Q values, while it seems that there is one softmax layer at the end of DQN network. I am wondering how is this DQN trained? Did I make some mistake understanding?
A clear and concise description of what the bug is.
Code example
Please use the markdown code blocks
for both code and stack traces.
from stable_baselines.common.atari_wrappers import make_atari
from stable_baselines.deepq.policies import MlpPolicy, CnnPolicy
from stable_baselines import DQN,ACKTR,A2C,PPO2
from stable_baselines.common.cmd_util import make_atari_env
from stable_baselines.common.vec_env import VecFrameStack
import cv2
import matplotlib.pyplot as plt
import numpy as np
from IPython import display
import os
game_list = ['Pong','Breakout','SpaceInvaders','Seaquest','BeamRider','Qbert','Enduro']
method_list = ['dqn','ppo2','a2c','acktr']
method = method_list[0]
for game in game_list:
    env = make_atari_env('{}NoFrameskip-v4'.format(game), num_env=1, seed=0)
    env = VecFrameStack(env, n_stack=4)
    model = DQN.load("trained_agents/{}/{}NoFrameskip-v4.pkl".format(method,game))
    env.reset()
    model.set_env(env);
    obs = env.reset()
    for i in range(1000):
        actions = model.action_probability(obs) #Here I want to get the Q values
        argmax_action = np.argmax(actions)
        action, _states = model.predict(obs)
        print(actions)
        print('the sum: {}'.format(np.sum(actions)))
        obs, rewards, dones, infos = env.step(action)
        episode_infos = infos[0].get('episode')

Traceback (most recent call last): File ...

System Info
[[0.16696884 0.16787794 0.16488545 0.16646059 0.16625851 0.16754872]]
the sum: 1.0
[[0.16700052 0.16733757 0.16515337 0.1665579  0.16647142 0.1674792 ]]
the sum: 1.0
[[0.16888289 0.16632704 0.16429803 0.16766034 0.16476472 0.168067  ]]
the sum: 1.0
[[0.16873147 0.1668533  0.1633186  0.16757944 0.16445793 0.16905922]]
the sum: 0.9999999403953552
[[0.16896197 0.16633949 0.16370982 0.16852172 0.16337387 0.16909312]]
the sum: 1.0
[[0.16926633 0.1685718  0.16038649 0.16665836 0.16892989 0.16618706]]
the sum: 0.9999998807907104
[[0.17271757 0.16515626 0.15838557 0.1654946  0.16886355 0.16938245]]
the sum: 1.0
[[0.17090748 0.16443412 0.15841582 0.16640182 0.17027445 0.16956638]]
the sum: 1.0
Describe the characteristic of your environment:

Describe how the library was installed (pip, docker, source, ...)
GPU models and configuration
Python version
Tensorflow version
Versions of any other relevant libraries

Additional context
Add any other context about the problem here.

Hello,
you are not looking at the q values but the action probability (cf doc).
PS: as mentioned in the issue template, please format your code using code block

Hi Araffin,
Thanks for your reply.
May I check how to output the Q values given a state s_t after I fully trained a DQN model?
From my understanding, the action probability is normalized from the output Q values. However, I did not find how to call this.
Thanks a lot

You need to check the code for that: https://github.com/hill-a/stable-baselines/blob/master/stable_baselines/deepq/policies.py#L149
Example call: https://github.com/hill-a/stable-baselines/blob/master/stable_baselines/deepq/dqn.py#L310

_, qvalues, _ = model.step_model.step(state, deterministic=True)

Use this line of code can obtain the q_values from the DQN model.
Hope this can help other researchers that look for this solution


NLB vs ALB?

All the docs claim the architecture uses an ALB, but the CFN templates clearly define an NLB. Is this an error, or an intentional change which didn't make it to the docs?

Thanks for the feedback, @troy-ameigh would you please help to update our doc?

To simplify the process to experience, using TLS require user to import/create certificate from ACM before attaching a certificate to the listener, which further require valid domain to verify the DCV. Open option for user and user can configure after deployment.

Guys, I need to configure the https, I am new in cloudformation. Fiz or deploy certain, the application has uploaded is ok. But as I said previously here the provision is an NLB and not ALB. What is the ideal shape for this configuration? Another duvid is, the superset of the stack see with the https port enabled by the parent? Because we only report the deploy parameters as vpcs, try that I have to adjust it in some way.

I need to configure the https, I am new in cloudformation. Make or deploy certain, the application has uploaded is ok. But as I said previously here the provision is an NLB and not ALB. What is the ideal shape for this configuration? Another duvid is, the superset of the stack see with the https port enabled by the parent? Because we only report the deploy parameters as vpcs, try that I have to adjust it in some way.

I need to configure the https, I am new in cloudformation. Make or deploy certain, the application has uploaded is ok. But as I said previously here the provision is an NLB and not ALB. What is the ideal shape for this configuration? Another duvid is, the superset of the stack see with the https port enabled by the parent? Because we only report the deploy parameters as vpcs, try that I have to adjust it in some way. thanks so much!


fix: completely remove .headerlink from documentation markup

Prevents this from showing up after recent markdown parsing changes on Discord's end:

Previously, all ¶ symbols would simply be removed, leaving an empty link name. Since empty link names are no longer permitted by Discord's markdown parser, these links are no longer hidden.
This PR removes the header links from the page hierarchy entirely, instead of relying on string operations for just that symbol alone.
ref: https://canary.discord.com/channels/755868545279328417/1099689844789227551

Hi, I'm sorry, but I need you to sign all of your commits


TS Server fatal error:  Debug Failure. False expression.

Type: Bug
❗❗❗ Please fill in the sections below to help us diagnose the issue ❗❗❗
TypeScript Version: 4.8.4
Steps to reproduce crash

Open workspace (medium size project)
Code normally
It crashes several times after small period of time

Global TS Server Plugins

@monodon/typescript-nx-imports-plugin

TS Server Log
❗ Server logging disabled. To help us fix crashes like this, please enable logging by setting:
"typescript.tsserver.log": "verbose"

After enabling this setting, future crash reports will include the server log.
TS Server Error Stack
Server: semantic
Error: Debug Failure. False expression.
    at _loop_3 (c:\Program Files\Microsoft VS Code\resources\app\extensions\node_modules\typescript\lib\tsserver.js:174658:34)
    at ProjectService.assignOrphanScriptInfoToInferredProject (c:\Program Files\Microsoft VS Code\resources\app\extensions\node_modules\typescript\lib\tsserver.js:174666:25)
    at ProjectService.assignProjectToOpenedScriptInfo (c:\Program Files\Microsoft VS Code\resources\app\extensions\node_modules\typescript\lib\tsserver.js:176335:26)
    at c:\Program Files\Microsoft VS Code\resources\app\extensions\node_modules\typescript\lib\tsserver.js:176609:97
    at Object.flatMap (c:\Program Files\Microsoft VS Code\resources\app\extensions\node_modules\typescript\lib\tsserver.js:525:25)
    at ProjectService.applyChangesInOpenFiles (c:\Program Files\Microsoft VS Code\resources\app\extensions\node_modules\typescript\lib\tsserver.js:176609:41)
    at Session.handlers.ts.Map.ts.getEntries._a.<computed> (c:\Program Files\Microsoft VS Code\resources\app\extensions\node_modules\typescript\lib\tsserver.js:177982:46)
    at c:\Program Files\Microsoft VS Code\resources\app\extensions\node_modules\typescript\lib\tsserver.js:180174:96
    at Session.executeWithRequestId (c:\Program Files\Microsoft VS Code\resources\app\extensions\node_modules\typescript\lib\tsserver.js:180165:28)
    at Session.executeCommand (c:\Program Files\Microsoft VS Code\resources\app\extensions\node_modules\typescript\lib\tsserver.js:180174:41)
    at Session.onMessage (c:\Program Files\Microsoft VS Code\resources\app\extensions\node_modules\typescript\lib\tsserver.js:180202:35)
    at process.<anonymous> (c:\Program Files\Microsoft VS Code\resources\app\extensions\node_modules\typescript\lib\tsserver.js:184368:31)
    at process.emit (node:events:526:28)
    at emit (node:internal/child_process:938:14)
    at process.processTicksAndRejections (node:internal/process/task_queues:84:21)

VS Code version: Code 1.73.0 (8fa188b2b301d36553cbc9ce1b0a146ccb93351f, 2022-11-01T15:34:06.111Z)
OS version: Windows_NT x64 10.0.19042
Modes:
Sandboxed: No
Extensions (33)

Extension
Author (truncated)
Version

better-comments
aar
3.0.2

nestjs-files
Abh
0.5.1

string-converter
ada
0.1.2

switcher
adr
1.0.4

Bookmarks
ale
13.3.1

cucumberautocomplete
ale
2.15.2

ng-template
Ang
15.2.0

angular-schematics
cyr
5.2.9

vscode-eslint
dba
2.4.0

gitlens
eam
13.3.1

EditorConfig
Edi
0.16.4

prettier-vscode
esb
9.10.4

vscode-jest-runner
fir
0.4.60

auto-rename-tag
for
0.1.10

shell-format
fox
7.2.5

codetogether-onpremises
gen
4.0.1-b00895

todo-tree
Gru
0.0.224

rest-client
hum
0.25.1

jbockle-format-files
jbo
3.4.0

Angular2
joh
13.0.0

intellij-idea-keybindings
k--
1.5.4

vscode-smart-column-indenter
lmc
0.0.13

fluent-icons
mig
0.0.18

angular-console
nrw
17.19.1

docthis
oou
0.8.2

material-icon-theme
PKi
4.24.0

vscode-sort-json
ric
1.20.0

markdown-preview-enhanced
shd
0.6.7

svg-preview
Sim
2.8.3

code-spell-checker
str
2.18.0

vscode-input-sequence
tom
0.2.0

sort-lines
Tyr
1.9.1

secret-reader
wir
0.3.0

(2 theme extensions excluded)

Thanks for creating this issue! It looks like you may be using an old version of VS Code, the latest stable release is 1.76.0. Please try upgrading to the latest version and checking whether this issue remains.
Happy Coding!

Does this reproduce in the latest VS Code insiders build with all extensions disabled?

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


RuntimeError: No such operator tutel_ops::cumsum

Hello, thanks for providing such a great work. However, I cannot use tutel successfully. I have followed the library installation steps:
* Install Pytorch for NVIDIA CUDA >= 11.3:
        $ python3 -m pip install --user torch==1.10.0+cu113 torchvision==0.11.1+cu113 -f https://download.pytorch.org/whl/torch_stable.html

* Install Tutel Online:

        $ python3 -m pip uninstall tutel -y
        $ python3 -m pip install --user --upgrade git+https://github.com/microsoft/tutel@main
        $ python3 ./tutel/setup.py install --user

But when I try the followed test:
* Quick Test on Single-GPU:

        $ python3 -m tutel.examples.helloworld --batch_size=16               # Test Tutel-optimized MoE + manual distribution

The followed error is reported：
Traceback (most recent call last):
  File "/root/miniconda3/envs/widenet/lib/python3.7/runpy.py", line 193, in _run_module_as_main
    "__main__", mod_spec)
  File "/root/miniconda3/envs/widenet/lib/python3.7/runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "/root/tutel-main/tutel/examples/helloworld.py", line 120, in <module>
    output = model(x)
  File "/root/.local/lib/python3.7/site-packages/torch/nn/modules/module.py", line 1102, in _call_impl
    return forward_call(*input, **kwargs)
  File "/root/tutel-main/tutel/examples/helloworld.py", line 85, in forward
    result = self._moe_layer(input)
  File "/root/.local/lib/python3.7/site-packages/torch/nn/modules/module.py", line 1102, in _call_impl
    return forward_call(*input, **kwargs)
  File "/root/tutel-main/tutel/impls/moe_layer.py", line 267, in forward
    logits_dtype, (crit, l_aux) = routing()
  File "/root/tutel-main/tutel/impls/moe_layer.py", line 261, in routing
    inequivalent_tokens = inequivalent_tokens,
  File "/root/tutel-main/tutel/impls/fast_dispatch.py", line 158, in extract_critical
    locations1 = compute_location(masks_se[0])
  File "/root/tutel-main/tutel/jit_kernels/gating.py", line 22, in fast_cumsum_sub_one
    return torch.ops.tutel_ops.cumsum(data)
  File "/root/.local/lib/python3.7/site-packages/torch/_ops.py", line 63, in __getattr__
    op = torch._C._jit_get_operation(qualified_op_name)
RuntimeError: No such operator tutel_ops::cumsum

what about expert FAST_CUMSUM=0 first?

Thank you. I have tried this, and the new error is reported:
Traceback (most recent call last):
  File "helloworld.py", line 120, in <module>
    output = model(x)
  File "/root/.local/lib/python3.7/site-packages/torch/nn/modules/module.py", line 1102, in _call_impl
    return forward_call(*input, **kwargs)
  File "helloworld.py", line 85, in forward
    result = self._moe_layer(input)
  File "/root/.local/lib/python3.7/site-packages/torch/nn/modules/module.py", line 1102, in _call_impl
    return forward_call(*input, **kwargs)
  File "/root/.local/lib/python3.7/site-packages/tutel-0.1-py3.7-linux-x86_64.egg/tutel/impls/moe_layer.py", line 271, in forward
    y = fast_encode(x.to(logits_dtype), crit, self.is_postscore).to(x.dtype)
  File "/root/.local/lib/python3.7/site-packages/tutel-0.1-py3.7-linux-x86_64.egg/tutel/impls/fast_dispatch.py", line 203, in fast_encode
    dispatcher.update(*critial_data[1:], is_postscore=is_postscore)
  File "/root/.local/lib/python3.7/site-packages/tutel-0.1-py3.7-linux-x86_64.egg/tutel/impls/fast_dispatch.py", line 108, in update
    self.func_fwd = jit_kernel.create_forward(self.dtype, indices_[0].is_cuda)
  File "/root/.local/lib/python3.7/site-packages/tutel-0.1-py3.7-linux-x86_64.egg/tutel/jit_kernels/sparse.py", line 35, in create_forward
    ''')
  File "/root/.local/lib/python3.7/site-packages/tutel-0.1-py3.7-linux-x86_64.egg/tutel/impls/jit_compiler.py", line 37, in generate_kernel
    return JitCompiler.create_raw(template)
  File "/root/.local/lib/python3.7/site-packages/tutel-0.1-py3.7-linux-x86_64.egg/tutel/impls/jit_compiler.py", line 26, in create_raw
    raise Exception('CUDA support is disabled during Tutel installation. Please configure CUDA correctly and reinstall Tutel to enable CUDA support, or report Tutel installation logs for help.')
Exception: CUDA support is disabled during Tutel installation. Please configure CUDA correctly and reinstall Tutel to enable CUDA support, or report Tutel installation logs for help.

Gotcha, this problem is not from tutel::cumsum. Instead, you may perform an improper installation of Tutel that only enables CPU support rather than CUDA.
The root cause could be an improper CUDA SDK configuration in your system that makes Tutel fail to build up some components, like missing cuda.h, libnccl.so, etc.
Suggestion:
Can you firstly purge old tutel (pip3 uninstall -y tutel), and reinstall the tutel via source (python3 ./tutel/setup.py install --user)? This method will show the full log of installing messages, including error messages. You may notice what is wrong with the environment. If you can not find the way to fix the environment, you can paste the full installation log here, so that we can help you diagnose the problem.

Thank you for your suggestion，I'll have a try.

export FAST_CUMSUM=0

Have you tried: export FAST_CUMSUM=0

I get this error after setting FAST_CUMSUM=0
[1,319]<stderr>:│ /root/.local/lib/python3.9/site-packages/tutel/impls/fast_dispatch.py:108 in │
[1,319]<stderr>:│ update                                                                       │
[1,319]<stderr>:│                                                                              │
[1,319]<stderr>:│   105 │   │   if self.is_cuda != indices_[0].is_cuda:                        │
[1,319]<stderr>:│   106 │   │   │   self.is_cuda = indices_[0].is_cuda                         │
[1,319]<stderr>:│   107 │   │   │   if self.is_cuda not in TutelMoeFastDispatcher.kernel_pool: │
[1,319]<stderr>:│ ❱ 108 │   │   │   │   self.func_fwd = jit_kernel.create_forward(self.dtype,  │
[1,319]<stderr>:│   109 │   │   │   │   self.func_bwd_data = jit_kernel.create_backward_data(s │
[1,319]<stderr>:│   110 │   │   │   │   self.func_bwd_gate = jit_kernel.create_backward_gate(s │
[1,319]<stderr>:│   111 │   │   │   │   TutelMoeFastDispatcher.kernel_pool[self.is_cuda] = sel │
[1,319]<stderr>:│                                                                              │
[1,319]<stderr>:│ /root/.local/lib/python3.9/site-packages/tutel/jit_kernels/sparse.py:21 in   │
[1,319]<stderr>:│ create_forward                                                               │
[1,319]<stderr>:│                                                                              │
[1,319]<stderr>:│    18   if not is_cuda:              [1,319]<stderr>:                                        │
[1,319]<stderr>:│    19 │   return JitCompiler.generate_cpu_kernel(kernel_type=0)              │
[1,319]<stderr>:│    20                                                                        │
[1,319]<stderr>:│ ❱  21   return JitCompiler.generate_kernel({'dtype': get_kernel_dtype(param_ │
[1,319]<stderr>:│    22 │   #define __dtype @dtype@                                            │
[1,319]<stderr>:│    23 │                                                                      │
[1,319]<stderr>:│    24 │   extern "C" __global__ __launch_bounds__(1024) void execute(__dtype │
[1,319]<stderr>:│                                                                              │
[1,319]<stderr>:│ /root/.local/lib/python3.9/site-packages/tutel/impls/jit_compiler.py:40 in   │
[1,319]<stderr>:│ generate_kernel                                                              │
[1,319]<stderr>:│                                                                              │
[1,319]<stderr>:│   37 │   def generate_kernel(keyword_dict, template):                        │
[1,319]<stderr>:│   38 │     for key in keyword_dict:                                          │
[1,319]<stderr>:│   39 │   │   template = template.replace('@%s@' % key, str(keyword_dict[key] │
[1,319]<stderr>:│ ❱ 40 │     return JitCompiler.create_raw(template)                           │
[1,319]<stderr>:│   41 │                                                                       │
[1,319]<stderr>:│   42 │   @staticmethod                                                       │
[1,319]<stderr>:│   43 │   def generate_cpu_kernel(kernel_type):                               │
[1,319]<stderr>:│                                                                              │
[1,319]<stderr>:│ /root/.local/lib/python3.9/site-packages/tutel/impls/jit_compiler.py:29 in   │
[1,319]<stderr>:│ create_raw                                                                   │
[1,319]<stderr>:│                                                                              │
[1,319]<stderr>:│   26 │   def create_raw(source):                                             │
[1,319]<stderr>:│   27 │   │   torch.cuda.init()                                               │
[1,319]<stderr>:│   28 │   │   if not hasattr(tutel_custom_kernel, 'inject_source'):           │
[1,319]<stderr>:│ ❱ 29 │   │   │   raise Exception('CUDA support is disabled during Tutel inst │
[1,319]<stderr>:│   30 │   │   __ctx__ = tutel_custom_kernel.inject_source(source)             │
[1,319]<stderr>:│   31 │   │                                                                   │
[1,319]<stderr>:│   32 │   │   def func(*inputs, extra=[], blocks=[]):                         │
[1,319]<stderr>:╰──────────────────────────────────────────────────────────────────────────────╯
[1,319]<stderr>:Exception: CUDA support is disabled during Tutel installation. Please configure
[1,319]<stderr>:CUDA correctly and reinstall Tutel to enable CUDA support, or report Tutel
[1,319]<stderr>:installation logs for help.
[1,176]<stderr>:╭───────────────────── Traceback (most recent call last) ──────────────────────╮

Gotcha, this problem is not from tutel::cumsum. Instead, you may perform an improper installation of Tutel that only enables CPU support rather than CUDA.
The root cause could be an improper CUDA SDK configuration in your system that makes Tutel fail to build up some components, like missing cuda.h, libnccl.so, etc.
Suggestion: Can you firstly purge old tutel (pip3 uninstall -y tutel), and reinstall the tutel via source (python3 ./tutel/setup.py install --user)? This method will show the full log of installing messages, including error messages. You may notice what is wrong with the environment. If you can not find the way to fix the environment, you can paste the full installation log here, so that we can help you diagnose the problem.

This is the solution. I was missing the cuda toolkit. apt-get install -y cuda-toolkit

I don't suggest you install cuda toolkit over default Ubuntu repository, as they are too old.
You should follow the instruction here: https://developer.nvidia.com/cuda-downloads?target_os=Linux&target_arch=x86_64&Distribution=Ubuntu
After CUDA SDK is successfully, please purge previous tutel and do a fresh installation based on those new CUDA SDK libraries.

Thanks for the suggestion! I'm using an older driver && once I added nv repo, I was able to select the version that I needed. add-apt-repository "deb https://developer.download.nvidia.com/compute/cuda/repos/ubuntu2004/x86_64/ /".


Bugfix/default aln method

Requirements

Filling out the template is required. Any pull request that does not include enough information to be reviewed in a timely manner may be closed at the maintainers' discretion;
Review the contributing guidelines for this repository; remember in particular:

do not modify code without testing for regression
provide simple unit tests to test the changes
if you change the schema you must patch the test databases as well, see Updating the schema
the PR must not fail unit testing

Description
Ensure there is a default align_type for protein_feature and it is ensembl
Use case
The align_type column in the protein_feature table is an enum. Existing scripts and pipeline that do not use this column should not have to worry about assigning it, hence it is useful to have a default value that makes sense. This is set to 'ensembl' by default. Other
Benefits
When new entries are added, the align_type will not be set to an empty string.
Possible Drawbacks
If applicable, describe any possible undesirable consequence of the changes.
This will be updating the protein_feature table in all existing core databases, so this might take a while to run
Testing
Have you added/modified unit tests to test the changes?
No additional tests required.
If so, do the tests pass/fail?
All tests pass, in particular schemaPatches, that ensures the patch file is in sync with table.sql
Have you run the entire test suite and no regression was detected?
Yes.

Coverage remained the same at 82.445% when pulling b10582d94e90d1010562505ea9005c0e36b5732f on bugfix/default_aln_method into 71dab88b9fc7b25da994668360ab29102e4a15b2 on master.

Isn't this a violation of the two separate changes in the same pull request guideline? Fixing a previous patch and adding a new change.


Add Support for Tree Shaking

I tried to use the issue template, but felt confined. Please triage and tag appropriately.
https://github.com/webpack/webpack/issues/2867

Closing as this is no longer a concern of mine. Please feel free to reopen open if you choose, or just let users find out the hard way.


[INFINITY-1365] Prepare refine resources

This should be sufficient changes such that a one line change in Capabilities will enable resource refinement.
I tested this on testing/master and testing/pull/1524 manually.

test this please

retest this please

retest this please


Min/Max for numeric/files

Min and Max validators should compare numeric values when value is numeric, or file size when value is file. Laravel documentation:

The field under validation must have a size matching the given value. For string data, value corresponds to the number of characters. For numeric data, value corresponds to a given integer value. For an array, size corresponds to the count of the array. For files, size corresponds to the file size in kilobytes.

Package version
5.0.8
Node.js and npm version
node 10.15.3
npm 6.9.0
Sample Code (to reproduce the issue)
class Offset {
    get validateAll() {
        return true;
    }

    get sanitizationRules() {
        return {
            offset: 'to_int',
        };
    }

    get rules() {
        return {
            offset: 'integer|min:1',
        };
    }
}

module.exports = Offset;

This example on data { offset: 0 } returns OK, but zero is below min value. Maybe this validators should check previous validators and then decide which type to use.

Sorry for the late reply. What you are really looking for is the above rule


Exponential logging question

If you deploy OpenSearch inside a VPC and then you enable VPC Flow Logs for the VPC, does this generate circular logs?
For example:

Log entry is PUT to OpenSearch.
Flow Log is generated for PUT request to OpenSearch.
VPC Flow Log goes to S3.
VPC Flow Log is PUT to OpenSearch.
Repeat step 2.

It feels like one log could generate recursive logging in a loop?

It does not circulate and continue to increase. Because VPC Flow Logs aggregates 1 minute or 10 minutes of logs and saves one file in S3. This is because es-loader processes one aggregated file at once. It's true that communication log to OpenSearch is included in the VPC flow logs. If you want to avoid it, log by subnet insted of VPC, or exclude the log of specific communication. see https://github.com/aws-samples/siem-on-amazon-opensearch-service/blob/main/docs/configure_siem.md#adding-an-exclusion-to-log-loading


Apply metadata to Opus output files

[x] Using latest version as provided on the master branch
[x] Searched for similar issues including closed ones

What is the purpose of your issue?

[ ] Bug
[x] Feature Request
[ ] Question
[ ] Other

Description

I'd like to be able to have metadata on Opus files. Currently, Opus files are saved in WebM files, which can't have all the metadata, so they would probably have to be converted to Ogg files instead. I'm 99% sure this is possible, and I think it should probably just be a small change to the FFMpeg commands, but I'm not sure.
Log

N/A

Sure, a PR would be appreciated!

Great, just wanted to be sure. I'll almost certainly have to change it to save as ogg instead of webm, is that fine?

Yeah... I guess most people use .mp3 for output. Make sure to update the README and the help description too.
cc: @ritiek

@PorygonZRocks Just a heads up. We use mutagen to apply metadata to the audio files. I don't remember whether we use ffmpeg to apply metadata.

Great, just wanted to be sure. I'll almost certainly have to change it to save as ogg instead of webm, is that fine?

I think saving it as ogg with extra metadata should be an extra cli flag.

Instead of removing .webm support, I'm just going to add Opus audio as a separate file extension. So the new question is: should the file extension be .ogg (Windows and Android support Opus but only with that extension), or should I leave .ogg for Vorbis to be added and use .opus?

I thing using .ogg is fine, we can change that later or introduce another flag for Vorbis if needed.

Ok, I'll use .ogg. One thing: if the input is .webm, rather than (probably needlessly) converting, I'm just going to extract the audio stream and assume it's Opus (or Vorbis, since it's .ogg it works for both), because that's what YouTube uses. So if YouTube changes the audio type, this'll need to be updated.

Another thing: Opus doesn't support 44.1k as a sampling rate, so I'm using 48k, which is the closest supported rate.

Also, I'm probably going to add Vorbis as .oga, since both Opus and Vorbis use the Ogg container they should use the same metadata code, so it's just a tiny bit of extra conversion code.

Pull request at #292

No support for anything but .mp3 as of now. Maybe open an issue and add it under new features?
I know it's late, just clearing out old issues.


Bacitracin/299 prop card a11y

For https://github.com/CodeForPhilly/vacant-lots-proj/issues/299
Some updates to make the property cards in the list keyboard navigable.

I went with making the address a button instead of a link because the url doesn't currently change when you click to see the single property detail.
Made card image decorative (left the description on single property view, TBD on whether we want to keep that or not)
The whole card is clickable

@marvieqa finally made this PR deployment happy, it's ready for a11y testing


[ECP-9297]Implement Boleto with custom JS renderer

Hello, no issue / bug.
There is a plan to include/support Boleto payments (Brazil)?

Thanks @khushboo-singhvi i could give you an help if needed


apparent data loss from recent update

Today my Oasis account was offered an update for Dillinger (I didn't look carefully at the time, but I presume it was to version 3.2.0-sandstorm1). I accepted the update. As a result, when I opened my existing grains, they no longer had any content. (Fortunately for me, I didn't have anything irreplaceable in Dillinger.)
I compared backups of a grain using the previous version with a grain using the new version. It appears that the previous version stored content in a text file called file.md, and the new version stores content in Redis.
I expected I could find the details of this change by looking in this Github repository, cem/dillinger-sandstorm. But I haven't found anything about version 3.2.0-sandstorm1 or Redis here.
I didn't lose anything important in this incident, but I am trying to learn how Sandstorm works. I had assumed that any update would include data migration, and that the code for all published changes could be found in the specified Github repository.

This is a major screwup on my part. I realized the same thing after updating Dillinger on my Oasis.
Quick fix (retrieve your lost data):
1- Open your grain
2- Download Backup (top bar)
3- In the zipfile, data/dillinger/file.md should contain the lost document.

I had assumed that any update would include data migration

Yes, updates should handle them. I will release an update that automatically upgrades the database.

and that the code for all published changes could be found in the specified Github repository

This is not necessarily true for non-BSD apps. Sandstorm even allows proprietary apps. In the case of my apps though, you should be able to find the code on github, unless I do things in a hurry and forget pushing :-). The repo is updated now.

The quick fix you provided works for me.
Thank you for the explanations.


Accept also max, min to be a number (the size of the string/buffer in bytes)

To create a summary report for some grunt-contrib modules it would be good to allow max & min be only the size of the string or buffer. Otherwise we would need to keep all the strings / buffers in memory which requires a lot of memory.
See https://github.com/gruntjs/grunt-contrib-cssmin/pull/200
Beside a test for it I also added a test for max, min being buffers.

@sindresorhus ping, if this something you want.

thanks @kevva, I changed it and squashed the commit

@sindresorhus: I changed it as you suggested

@sindresorhus thanks for adding my commit. Can you please also increase the package version as otherwise i can't use it in other libraries (it still downloads the old version registered under 1.0.1).

Yes, was going to, then my battery died. Published as 1.1.0.


Move server side OG meta tags to page

See this thread on Twitter for context:

API documentation can be found here:
https://nextjs.org/docs/basic-features/data-fetching#getserversideprops-server-side-rendering
We should validate this change after it's made using the following URLs:

Twitter validator
Facebook validator
Generic validator

Had a quick stab at this on my lunch break, see if it helps https://github.com/bvaughn/planner/pull/26


Load js of Search component after user interaction

What version of starlight are you using?
0.7.3
What is your idea?
<Search client:interaction="pointerenter,focusin,pointerdown" /> in packages/starlight/components/Header.astro
Inspiration from site https://material-web.dev
<lit-island on:interaction="pointerenter,focusin,pointerdown" import="/js/hydration-entrypoints/menu.js">

Why is this feature necessary?
We don't need js of Search component on a page in first time.
Do you have examples of this feature in other projects?
No response
Participation

[ ] I am willing to submit a pull request for this issue.

We already load the search component JS lazily after everything else is loaded, so I'm not sure delaying further would have a significant performance benefit while potentially worsening user experience by creating a noticeable delay at the moment they do want to search.
Have you noticed specific problems this would solve?

Thanks again for raising this @JosefJezek — closing for now as we haven’t had any more reports of this causing issues.


generic hasChanged method for any path with a value node

for example
if(gui.hasChanged("count")){
  regenerateList();
}

also do folders - if anything changes inside that folder then the folder has changed

fix undo/redo first #36

as in has change just ended

nah, the user can detect that


added var in AddRepos() for Chartfile handling is never increased

The variable added is never actually increased when adding a repo.
https://github.com/grafana/tanka/blob/2034d2db26694eb6e404a567e23c2328dc75d489/pkg/helm/charts.go#L189-L206

Closed by https://github.com/grafana/tanka/pull/641


dvb: no input DVB signal after 5,000 milliseconds

i am connecting in ubntu
previously the data was coming but now no input dvb signal after 5000 ms comes
please help

I am sure that you understand that providing help from such a tiny description is rather challenging.

i have TBS5520se tu tuner usb box
i have to locked the input signal coming to it
while using ts reader the signal is locked and it is working in window
but in Ubuntu while i am using
tsp -I dvb --device-name /dev/dvb/adapter0:1 --tune 11670:v:2:29500 -P until --seconds 20 -O file ts1.mpg
earlier the device was tuned and data was saved .
but now it shows error no input DVB signal after 5000 milliseconds
please help

When one configuration does not work and the other one works or used to, you must analyze the differences step by step.
If it used to work and no longer does on Ubuntu with TSDuck, what did change in the meantime?
Does it work on any other transponder?
If it works on Windows with TSReader and does not on Ubuntu with TSDuck, what is the relationship between the two machines? Two machines, same machine (dual-boot), one VM (which one is the guest and which one is the host)? Same physical tuner? If it is not the same tuner, did you check antenna connectivity?

i am using another configuration it is also not working ...

The relation is that ts reader is also used for locking the signals
i have dual boot machine  Ubntu with window
and tuner is same TBS2250SE Tv Tuner Box
antenna connectivity is same for both the process

but signal is neither locked so that data may occur
please help

i am using command
tsp -I dvb --device-name /dev/dvb/adapter0:1 --tune 11510:v:0:32720 -P until --seconds 10 -O file tsnew1.mpg

Error: dvb: no input DVB signal lock after 5000 milliseconds

it comes

hello sir
As u said i did i runs
Thanks a lot
now please tell me 2 things

i have to save anything that is on the terminal screen such as
i am extracting pcrverify it shows me on terminal screen but not save on the file command is
tsp -I dvb --tune 11510:v:0:32720 -P until --seconds 100 -P pcrverify --jitter-max 0 | -O file >pcrextract.txt

it is showing binary data in file and on the screen it is different thing
i want to save whatever is didplaying on screen

how to add local system time in nay .txt file .

sier where is ts duck repository in ubntu i am not finding anywhere
please help
or tell me how to run files from tsduck-master filke in ubntu

TSDuck is currently not in any official distro, neither Ubuntu, nor Fedora. It is probably too specific and nobody asked. So, you have to grab the binary packages from the release area.
To rebuild TSDuck from a freshly pulled repository, follow the builds instructions. But you probably already did it.
After building, you have several options:

Build the .deb package and install it:

make deb

Directly install in the system location:

make install

Install somewhere else, for instance:

make install SYSPREFIX=/usr/local

Run directly from the build directory. This is currently not well documented and I should probably improve this. The appropriate environment is setup using a script which is automatically generated in the bin directory. Depending on your target (release vs. debug and 32 vs. 64 bits), execute one of:

source src/tstools/release-x86_64/setenv.sh
source src/tstools/debug-x86_64/setenv.sh
source src/tstools/release-i386/setenv.sh
source src/tstools/debug-i386/setenv.sh

Example:
$ source src/tstools/release-x86_64/setenv.sh
$ which tsp
~/tsduck/src/tstools/release-x86_64/tsp
$ tsp --version
tsp: TSDuck - The MPEG Transport Stream Toolkit - version 3.12-730

ok sir i will do what u suggesgts thanks a lot ...............