docker module: add support for docker lvm plugin

Adds support for docker lvm plugin

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

[x] NixOS
[ ] macOS
[ ] Linux

[ ] 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 nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

Closing as I do not need this anymore.


Return a tensor in training_step leads to a list of dictionaries in training_epoch_end

When I run the following code in my model
    def training_step(self, batch, batch_idx):
        loss = self._step(batch, batch_idx)
        print(type(loss))
        self.log("train_loss", loss, sync_dist=True)
        return loss

    def training_epoch_end(self, outputs):
        print(outputs)
        self.log("train_ppl", self.calculate_ppl(outputs), sync_dist=True)

I see that the training step returns a tensor. So I would expect that the aggregated values that are passed to training_epoch_end is a list of tensors. But, as this code shows, it is not. Instead it is a list of dictionaries that looks like this:
[{'loss': tensor(4.1520, device='cuda:0')}, {'loss': tensor(4.1750, device='cuda:0')}]

Annoyingly, the expected behavior does seem to occur in the validation and testing loops. So I am not sure whether I am simply doing something wrong, or whether the training_step forcefully collects dictionaries for a specific reason, or whether this is actually a bug.
PL version: 1.4.9
Torch: 1.9.1+cu111
cc @borda @carmocca @justusschock @ananthsub @ninginthecloud

hi,
Can you send the reproducible script?
I will try to check the issue.
With best regards,
Ranuga

@carmocca That is indeed my current approach. However, it seems that this conversion to a dict is only done for the training step and not for validation and test steps. As a user, I would expect the same behavior because that allows me to write a general *_epoch_end . Let's say I want to calculate perplexity at the end of every epoch for all scenarios, now I need to write something like this:
    def training_step(self, batch, batch_idx):
        loss = self._step(batch, batch_idx)
        self.log("train_loss", loss, sync_dist=True)
        return loss

    def training_epoch_end(self, outputs):
        self.log("train_ppl", self.calculate_ppl(outputs), sync_dist=True)

    def validation_step(self, batch, batch_idx, dataloader_idx=0):
        loss = self._step(batch, batch_idx)
        self.log("val_loss", loss, sync_dist=True)
        return loss

    def validation_epoch_end(self, outputs: List[torch.tensor]):
        self.log("val_ppl", self.calculate_ppl(outputs), sync_dist=True)

    @staticmethod
    def calculate_ppl(losses) -> float:
        # Getting inconsistent returns from training loop vs validation
        # https://github.com/PyTorchLightning/pytorch-lightning/issues/9968
        if isinstance(losses[0], torch.Tensor):
            losses = torch.tensor([t.item() for t in losses])
        else:
            losses = torch.tensor([list(d.values())[0].item() for d in losses])
        return torch.exp(losses.mean()).item()

This "reads" very counter-intuitive: my return value for both steps is the same, so why do I need to provide two possible scenarios to deal with the aggregated outputs?

Friendly ping for thoughts here @justusschock @awaelchli
I opened a PoC in #10878 with the required changes
This would be a breaking change for those returning Tensors in training_step or training_step_end who also implement training_epoch_end and expect a dictionary input.
To avoid this, we would need a deprecation process similar to the one proposed in https://github.com/PyTorchLightning/pytorch-lightning/issues/9737.
Is the UX improvement of respecting the user output type worth the deprecation process?

I align with @carmocca. I think the inconsistency between the training hooks and eval hooks should be fixed. At the same time, it is expected to be very common that users implement the step + epoch_end combo, and potentially many LightningModules could break if we don't do a deprecation. So I'm in favor of doing that, if it is possible.

Looking forward to any updates on this.

The plan is to add deprecations and an opt-in mechanism for this and #9737 before the 1.6 release


Provide a homeDir system variable for use in launch.json

*** This is just a revival of ticket 11337 since it was closed. I would still like this feature and having to mess with environmental variables isn't a great solution in my mind **
Version: 1.33.1 (user setup)
Commit: 51b0b28134d51361cf996d2f0a1c698247aeabd8
Date: 2019-04-11T08:27:14.102Z
Electron: 3.1.6
Chrome: 66.0.3359.181
Node.js: 10.2.0
V8: 6.6.346.32
OS: Windows_NT x64 10.0.17763
I have VS Code installed on two computers and use settings sync between the two. But both computers have different usernames. So the settings don't match up. Having a profile variable would make it seamless.
So instead of
"php.executablePath": "/Users//Documents/php-7.3.5-nts-Win32-VC15-x64/php.exe",
it would be
"php.executablePath": "${homeDir}/Documents/php-7.3.5-nts-Win32-VC15-x64/php.exe",
Below is the original ticket
https://github.com/microsoft/vscode/issues/11337

Dup of https://github.com/microsoft/vscode/issues/11337
Bet let's keep it open for now.

Is this for launch.json? That looks like a setting. But this seems exactly like a use case for using an environment variable. The only unfortunate part is that if you are supported multiple platforms, windows uses a different env var name vs mac/linux.

I'm unfamiliar with launch.json, but that is from my settings.json file. I have a PHP linter that requires a path to a PHP executable. But due to the path differences I have to manually adjust it for each. Additionally, it forces me to keep the Settings Sync extension from running automatically because it would break for one of the two computers each time it updates.

It sounds like the meta-issue is using Settings Sync to sync per-device settings between machines. You can configure settings sync to ignore certain settings to avoid having it overwrite the executable path when syncing them between machines. It sounds like the extension is the cause of your particular issue.
I think I agree with @roblourens here about the usage of environment variables in general. While os.homedir is accessible, using it inside a setting shared by multiple machines will depend upon each machine having the exact same layout of whatever files we're looking for inside the home directory. Using an environment variable would be a more resilient approach for this scenario, once https://github.com/microsoft/vscode/issues/2809 happens.


document-compliance-gateway mexico-cfdi api doc

@probalsap You need to sign the CLA before this can be merged.


chore: change API host to https://api.supabase.com

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
3704

Relevant Lines:
5932

💛  - Coveralls


Rmsw 33 - Code cleanup v1.0

Made a wiki page with the clean code standard.
Updated each file to comply with standard.
Renamed Src, Inc, Obj folders to comply with stm32 format standard.
Re organized Inc file directory to be the same as Src.

Note: If there is any updates to the clean code standard before the is pulled the code will need to be updated.

Thank you Tre'~


How to get the backup of dedicated server save file on S3?

Hello The Sudo,
I have enabled the backup s3 modules

but why i still dont get the backup file on s3?

it says: No object found
can you help me?
thank you, sir.

What version of the module are you using and what AWS region are you in.

@ridwanaliah also, set the backup_s3_bucket_arn = "" and backup_s3_bucket_name = "". Those inputs are only used when you have create_backup_s3_bucket = false because you have an existing S3 bucket you want to use.

What version of the module are you using and what AWS region are you in.

i used the sudo 1.1.0, my region ap-southeast-1

@ridwanaliah also, set the backup_s3_bucket_arn = "" and backup_s3_bucket_name = "". Those inputs are only used when you have create_backup_s3_bucket = false because you have an existing S3 bucket you want to use.

okay i'll try it now

Version 1.1.1 includes a bug fix to support AWS certain AWS regions with s3. If the palworld_backup_log.txt file in the /palworld-server directory says something like this:
An error occurred (IllegalLocationConstraintException) when calling the PutObject operation: The ap-southeast-1 location constraint is incompatible for the region specific endpoint this request was sent to.

Then you need to update the module version to 1.1.1 and do one of these 2 options:
Option 1
Delete your server and rerun terraform apply to get a new server with the new user_data script. This will build a new  /palworld-server/palworld_backup_script.sh on your server with the fix applied.
Option 2
SSH into your server and edit the existing backup script located in /palworld-server/palworld_backup_script.sh and look for these lines:
aws s3 cp "\$BACKUP_FILENAME" s3://"\$S3_BUCKET_NAME"/
aws s3 cp "\$GUS_BACKUP" s3://"\$S3_BUCKET_NAME"/

and add --region af-south-1 to the end of both of them. Example:
aws s3 cp "\$BACKUP_FILENAME" s3://"\$S3_BUCKET_NAME"/ --region af-south-1
aws s3 cp "\$GUS_BACKUP" s3://"\$S3_BUCKET_NAME"/ --region af-south-1

Can you give me some instruction how to update to 1.1.1 module?
anyway i still don't get any backup files

Option 1
Delete your server and rerun terraform apply to get a new server with the new user_data script. This will build a new /palworld-server/palworld_backup_script.sh on your server with the fix applied.
if i do this, all my dedicated server save games will lost or not?

hello, i've updated the module to 1.1.1 version
but can you help me how to do the option 2?

what command i use to ssh to the palworld_backup_script.sh
i need ur help
thank you so much.

what should i do next?

can i copy paste this manually to my local desktop?

If that file does not exist then your only option is to destroy the server and recreate it using version 1.1.1. Starting from here: https://youtu.be/H0gGtPqhxco?si=L1OIwfsIkaCVBQEj&t=1175 you an use SCP to move the SaveGame files off your EC2 instance onto your PC to save your data
I've tried this to scp it but it appears like this:

../../palworld_public_key is stating that you have the ssh key 2 directories/folders below the C:\Users\User\Desktop\Palworld-Terraform\ directory / folder. You have to provide the path to your SSH key relative to C:\Users\User\Desktop\Palworld-Terraform\.


Bump esbuild from 0.14.42 to 0.14.48

Bumps esbuild from 0.14.42 to 0.14.48.

Release notes
Changelog
Commits

updated-dependencies:

dependency-name: esbuild
dependency-type: direct:production
update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot]<EMAIL_ADDRESS>

Hitting other issues with this upgrade attempt. One of them is that the VM seems unhappy with the logical assignment ||= operator and the flag to disable that new behavior doesn't seem to be working: https://github.com/evanw/esbuild/commit/f808dab4718a34a4e1d4388167009b1bd72dd99f#r77502675


How to silent warning of luarocks.cmd.nix?

I always meet:
Warning: failed to load command module luarocks.cmd.nix: module 'luarocks.cmd.nix' not found:No LuaRocks module found for luarocks.cmd.nix
        no field package.preload['luarocks.cmd.nix']
        no file '/nix/store/b2vp610ys58jvzblsw4h3m7brjxlxn5d-luarocks-3.11.0/share/lua/5.1/luarocks/cmd/nix.lua'
        no file './luarocks/cmd/nix.lua'
        no file '/nix/store/iq8s1f6d5wjwy4k7dm24qa4jpa3iby54-luajit-2.1.1693350652/share/luajit-2.1/luarocks/cmd/nix.lua'
        no file '/usr/local/share/lua/5.1/luarocks/cmd/nix.lua'
        no file '/usr/local/share/lua/5.1/luarocks/cmd/nix/init.lua'
        no file '/nix/store/iq8s1f6d5wjwy4k7dm24qa4jpa3iby54-luajit-2.1.1693350652/share/lua/5.1/luarocks/cmd/nix.lua'
        no file '/nix/store/iq8s1f6d5wjwy4k7dm24qa4jpa3iby54-luajit-2.1.1693350652/share/lua/5.1/luarocks/cmd/nix/init.lua'
        no file '/nix/store/b2vp610ys58jvzblsw4h3m7brjxlxn5d-luarocks-3.11.0/share/lua/5.1/luarocks/cmd/nix/init.lua'
        no file '/nix/store/3jq4khyb2kq07a2j42dg9rqfbvy2nx6c-luarocks-luarocks-config.lua/share/lua/5.1/luarocks/cmd/nix.lua'
        no file '/nix/store/3jq4khyb2kq07a2j42dg9rqfbvy2nx6c-luarocks-luarocks-config.lua/share/lua/5.1/luarocks/cmd/nix/init.lua'

How to silent it?

without the initial command, it's difficult to say. Provide reproduction steps please.

Every command. For example: luarocks install --local dbus_proxy

What version, how do you get your luarocks ?
If I do on master, it works (I suppose it fails because it's missing cmake/make or some external library but I dont have your error)
$ nix develop
$ luarocks install --local dbus_proxy
Installing https://luarocks.org/dbus_proxy-0.10.3-2.src.rock
Missing dependencies for dbus_proxy 0.10.3-2:
   lgi >= 0.9.0, < 1 (not installed)

dbus_proxy 0.10.3-2 depends on lua >= 5.1 (5.1-1 provided by VM: success)
dbus_proxy 0.10.3-2 depends on lgi >= 0.9.0, < 1 (not installed)
Installing https://luarocks.org/lgi-0.9.2-1.src.rock

lgi 0.9.2-1 depends on lua >= 5.1 (5.1-1 provided by VM: success)

Error: Failed installing dependency: https://luarocks.org/lgi-0.9.2-1.src.rock - Failed initializing build back-end for build type 'make': module 'luarocks.build.make' not found:No LuaRocks module found for luarocks.build.make
	no field package.preload['luarocks.build.make']
	no file 'src/luarocks/build/make.lua'
	no file './luarocks/build/make.lua'

I get it by
      (
        # rocks.nvim needs luarocks 5.1
        luajit.withPackages (
          p: with p; [
            # pre-commit needs it
            luarocks-nix
            luaprompt
          ]
        )
      )

this was a nixpkgs issue, I've had the same when I tried to update the package sets. We've recently exposed in nixpkgs a "luarocks" generated from luarocks.org, and luarocks-nix was wrapped with that one, which hid the nix.lua.
https://github.com/NixOS/nixpkgs/pull/322224 should fix it


Updated replicationg logging to correct strategy

Fixed small typo for replication logging

Confirming signed CLA on file.
Merging; thanks!


Cost estimator

More of a nice-to-have.
Since building the index involves calls to GPT-3, it might be nice to have a verbose output option that counts the number of tokens involved, which will help devs get a feel for the costs involved with using this package.

great idea!

Brainstorming a few ideas here.
This includes building the index and executing the query.

Estimate cost before making the call to the LLM.
Compute the exact cost after making the call to the LLM. (Pretty sure we'll have this info)

@teoh sure! that makes sense to me


Compiler: initialize right-away constants in a separate function

Fixes the slowness in the compiler.
Alternative to https://github.com/crystal-lang/crystal/pull/10330
The idea is to still eagerly initialize the constants that can be initialized like that, but do that in a separate function. It seems like that LLVM has an easier time optimizing things.
The code for doing that was already there in the compiler: it's used for all other constants, but doing that in a wrapper that makes sure they are only initialized once.
Does it really work, and is it really fast?
Please help me try it out. Here's what you can do:

Run crystal build src/ecr/process.cr --release -s with the existing 0.36.0 compiler, or by compiling a compiler from master. Check how long the "bc+obj|" phase takes. For me this takes about 30 seconds.
Compile a compiler with the code in this PR. Then run bin/crystal build src/ecr/process.cr --release -s. It should be faster. For me this takes about 8 seconds.
Also try the previous point with #10330 . It should take about the same time as before (8 seconds for me).

Then with a compiler built from this PR make sure this benchmark is still fast.
You can also compare the times with a 0.35.1 compiler. For me that "bc+obj" phase takes about 6 seconds, not 8. I think LLVM 11 is a bit slower in this aspect.

Here's what I get:
Crystal 0.35.1
Codegen (bc+obj):                  00:00:09.678091387 (  92.28MB)

Crystal 0.36.0
Codegen (bc+obj):                  00:01:38.148248913 (  89.86MB)

Crystal this PR
Codegen (bc+obj):                  00:00:10.685466711 (  89.86MB)
ld: warning: directory not found for option '-L/usr/local/Cellar/crystal/0.34.0/embedded/lib'

For the last one, here's the version it showed
❯ ./bin/crystal -v
Using compiled compiler at .build/crystal
Crystal 1.0.0-dev [e1a0da726] (2021-01-29)

LLVM: 11.0.0
Default target: x86_64-apple-macosx

Here's what I get:
Crystal 0.35.1
Codegen (bc+obj):                  00:00:09.678091387 (  92.28MB)

Crystal 0.36.0
Codegen (bc+obj):                  00:01:38.148248913 (  89.86MB)

Crystal this PR
Codegen (bc+obj):                  00:00:10.685466711 (  89.86MB)
ld: warning: directory not found for option '-L/usr/local/Cellar/crystal/0.34.0/embedded/lib'

For the last one, here's the version it showed
❯ ./bin/crystal -v
Using compiled compiler at .build/crystal
Crystal 1.0.0-dev [e1a0da726] (2021-01-29)

LLVM: 11.0.0
Default target: x86_64-apple-macosx

@jwoertink Thanks! What's that ld warning? Is that a problem? First time I see it.

@jwoertink Thanks! What's that ld warning? Is that a problem? First time I see it.

Oh, and here's that benchmark with those 3 versions:
Crystal 0.35.1 (took 14s to build)
❯ ./kostya
17185365
00:00:01.240112000

Crystal 0.36.0 (took 1m36s to build)
❯ ./kostya
17191068
00:00:00.382423000

Crystal this PR (took 11s to build)
❯ ./kostya
17187241
00:00:00.397169000

Oh, and here's that benchmark with those 3 versions:
Crystal 0.35.1 (took 14s to build)
❯ ./kostya
17185365
00:00:01.240112000

Crystal 0.36.0 (took 1m36s to build)
❯ ./kostya
17191068
00:00:00.382423000

Crystal this PR (took 11s to build)
❯ ./kostya
17187241
00:00:00.397169000

@asterite I've never seen that ld warning before. It only seems to happen when I compile from this PR. I ran make clean crystal when I built the compiler. Maybe I missed a step?

@asterite I've never seen that ld warning before. It only seems to happen when I compile from this PR. I ran make clean crystal when I built the compiler. Maybe I missed a step?

It's also very strange that the error mentions a directory with Crystal 0.34.0 in it

It's also very strange that the error mentions a directory with Crystal 0.34.0 in it

@jwoertink thank you for trying these! Of course you can also use a compiler from this PR with whatever program was slow to compile for you, and see if it's fixed.

@jwoertink thank you for trying these! Of course you can also use a compiler from this PR with whatever program was slow to compile for you, and see if it's fixed.

It is
❯ ls -la /usr/local/Cellar/crystal/
total 0
drwxr-xr-x    3 jeremywoertink  staff    96 Jan 27 08:38 .
drwxrwxr-x  156 jeremywoertink  admin  4992 Jan 27 08:43 ..
drwxr-xr-x   14 jeremywoertink  staff   448 Jan 27 08:38 0.36.0

I definitely don't have 0.34 on here 😂

It is
❯ ls -la /usr/local/Cellar/crystal/
total 0
drwxr-xr-x    3 jeremywoertink  staff    96 Jan 27 08:38 .
drwxrwxr-x  156 jeremywoertink  admin  4992 Jan 27 08:43 ..
drwxr-xr-x   14 jeremywoertink  staff   448 Jan 27 08:38 0.36.0

I definitely don't have 0.34 on here 😂

Sweet! I ran the Lucky specs using this PR, and it took 1m30s. Interesting thing was that ld: warning showed up 5 times. I wonder if there's 5 different run macros and each has to be compiled separately?
lucky on  jaw/crystal0.36 [!?] via 🔮 v0.35.1
❯ /Users/jeremywoertink/Development/crystal/lang/bin/crystal spec
Using compiled compiler at /Users/jeremywoertink/Development/crystal/lang/.build/crystal
ld: warning: directory not found for option '-L/usr/local/Cellar/crystal/0.34.0/embedded/lib'
ld: warning: directory not found for option '-L/usr/local/Cellar/crystal/0.34.0/embedded/lib'
ld: warning: directory not found for option '-L/usr/local/Cellar/crystal/0.34.0/embedded/lib'
ld: warning: directory not found for option '-L/usr/local/Cellar/crystal/0.34.0/embedded/lib'
ld: warning: directory not found for option '-L/usr/local/Cellar/crystal/0.34.0/embedded/lib'

..........................................

Sweet! I ran the Lucky specs using this PR, and it took 1m30s. Interesting thing was that ld: warning showed up 5 times. I wonder if there's 5 different run macros and each has to be compiled separately?
lucky on  jaw/crystal0.36 [!?] via 🔮 v0.35.1
❯ /Users/jeremywoertink/Development/crystal/lang/bin/crystal spec
Using compiled compiler at /Users/jeremywoertink/Development/crystal/lang/.build/crystal
ld: warning: directory not found for option '-L/usr/local/Cellar/crystal/0.34.0/embedded/lib'
ld: warning: directory not found for option '-L/usr/local/Cellar/crystal/0.34.0/embedded/lib'
ld: warning: directory not found for option '-L/usr/local/Cellar/crystal/0.34.0/embedded/lib'
ld: warning: directory not found for option '-L/usr/local/Cellar/crystal/0.34.0/embedded/lib'
ld: warning: directory not found for option '-L/usr/local/Cellar/crystal/0.34.0/embedded/lib'

..........................................

If you pass -s I think it tells you which macro runs were compiled

If you pass -s I think it tells you which macro runs were compiled

Ah, yup.
- /Users/jeremywoertink/Development/crystal/lucky/lib/teeplate/src/lib/file_tree/macros/directory.cr: reused previous compilation (00:00:00.038240130)
 - /Users/jeremywoertink/Development/crystal/lucky/src/run_macros/generate_asset_helpers.cr: reused previous compilation (00:00:00.039536256)
 - /Users/jeremywoertink/Development/crystal/lucky/src/run_macros/infer_route.cr: reused previous compilation (00:00:00.040292692)
 - /Users/jeremywoertink/Development/crystal/lucky/lib/avram/src/run_macros/infer_table_name.cr: reused previous compilation (00:00:00.041060772)
 - /Users/jeremywoertink/Development/crystal/lang/src/ecr/process.cr: reused previous compilation (00:00:00.045328081)

Not a huge deal I guess. 1min is WAY better than the 14min I was seeing 😂

Ah, yup.
- /Users/jeremywoertink/Development/crystal/lucky/lib/teeplate/src/lib/file_tree/macros/directory.cr: reused previous compilation (00:00:00.038240130)
 - /Users/jeremywoertink/Development/crystal/lucky/src/run_macros/generate_asset_helpers.cr: reused previous compilation (00:00:00.039536256)
 - /Users/jeremywoertink/Development/crystal/lucky/src/run_macros/infer_route.cr: reused previous compilation (00:00:00.040292692)
 - /Users/jeremywoertink/Development/crystal/lucky/lib/avram/src/run_macros/infer_table_name.cr: reused previous compilation (00:00:00.041060772)
 - /Users/jeremywoertink/Development/crystal/lang/src/ecr/process.cr: reused previous compilation (00:00:00.045328081)

Not a huge deal I guess. 1min is WAY better than the 14min I was seeing 😂

Let's wait for someone else to see if they also get those warnings.

Let's wait for someone else to see if they also get those warnings.

I checked before/after this PR against llvm@10 and bc+obj when from 27s to 7s. The released 0.36.0 tar.gz llvm@6 is even 12s.
I'm happy with this. I'm not sure if @mattrberry can verify this before 0.36.1, that would be awesome.

I checked before/after this PR against llvm@10 and bc+obj when from 27s to 7s. The released 0.36.0 tar.gz llvm@6 is even 12s.
I'm happy with this. I'm not sure if @mattrberry can verify this before 0.36.1, that would be awesome.

@jwoertink what does bin/crystal env (and crystal env) shows on your environment?

@jwoertink what does bin/crystal env (and crystal env) shows on your environment?

@bcardiff
❯ bin/crystal env
Using compiled compiler at .build/crystal
CRYSTAL_CACHE_DIR=/Users/jeremywoertink/.cache/crystal
CRYSTAL_PATH=lib:/Users/jeremywoertink/Development/crystal/lang/src
CRYSTAL_VERSION=1.0.0-dev
CRYSTAL_LIBRARY_PATH=/usr/local/Cellar/crystal/0.34.0/embedded/lib
CRYSTAL_OPTS=''

❯ crystal env
CRYSTAL_CACHE_DIR=/Users/jeremywoertink/.cache/crystal
CRYSTAL_PATH=lib:/Users/jeremywoertink/Downloads/crystal-0.35.1-1/src
CRYSTAL_VERSION=0.35.1
CRYSTAL_LIBRARY_PATH=/Users/jeremywoertink/Downloads/crystal-0.35.1-1/embedded/lib
CRYSTAL_OPTS=''

Maybe I have something left over from a while ago?

@bcardiff
❯ bin/crystal env
Using compiled compiler at .build/crystal
CRYSTAL_CACHE_DIR=/Users/jeremywoertink/.cache/crystal
CRYSTAL_PATH=lib:/Users/jeremywoertink/Development/crystal/lang/src
CRYSTAL_VERSION=1.0.0-dev
CRYSTAL_LIBRARY_PATH=/usr/local/Cellar/crystal/0.34.0/embedded/lib
CRYSTAL_OPTS=''

❯ crystal env
CRYSTAL_CACHE_DIR=/Users/jeremywoertink/.cache/crystal
CRYSTAL_PATH=lib:/Users/jeremywoertink/Downloads/crystal-0.35.1-1/src
CRYSTAL_VERSION=0.35.1
CRYSTAL_LIBRARY_PATH=/Users/jeremywoertink/Downloads/crystal-0.35.1-1/embedded/lib
CRYSTAL_OPTS=''

Maybe I have something left over from a while ago?

Yes, it seems that you probably have a CRYSTAL_CONFIG_LIBRARY_PATH env set that is setting the CRYSTAL_LIBRARY_PATH in the built compiler

Yes, it seems that you probably have a CRYSTAL_CONFIG_LIBRARY_PATH env set that is setting the CRYSTAL_LIBRARY_PATH in the built compiler

I don't think mattrberry has ever compiled the compiler before, but I'm pretty sure it will work fine and fast

I don't think mattrberry has ever compiled the compiler before, but I'm pretty sure it will work fine and fast

I've built the compiler on a few occasions, including when I tested your changes to the constants prior to 0.36.0. I'm happy to try again in a few hours. CryBoy appears to fail compilation on 0.36.0 still due to some later changes that went into 0.36.0, so I still need to work those out. I'll update this thread this evening when I get a chance to look into it a little further

I've built the compiler on a few occasions, including when I tested your changes to the constants prior to 0.36.0. I'm happy to try again in a few hours. CryBoy appears to fail compilation on 0.36.0 still due to some later changes that went into 0.36.0, so I still need to work those out. I'll update this thread this evening when I get a chance to look into it a little further

Testing on CryBoy:
Compiler versions tested:

0.35.1
0.36.0
opt/const-and-class-var

For each of these versions, I tested 3 ROMs 3 times. Across all of those runs, both 0.36.0 and this branch seemed to have similar performance, both of which improved fps over 0.35.1 by 20-25% depending on the rom.
In terms of performance that I've noticed, this branch seems to be identical or close enough to identical to 0.36.0. If this improves upon the compilation speed of 0.36.0, I think this is a win-win :)

Testing on CryBoy:
Compiler versions tested:

0.35.1
0.36.0
opt/const-and-class-var

For each of these versions, I tested 3 ROMs 3 times. Across all of those runs, both 0.36.0 and this branch seemed to have similar performance, both of which improved fps over 0.35.1 by 20-25% depending on the rom.
In terms of performance that I've noticed, this branch seems to be identical or close enough to identical to 0.36.0. If this improves upon the compilation speed of 0.36.0, I think this is a win-win :)


Replace the term "attainment" with "course part?"

Teachers seem to dislike the term "attainment" and think "course part" is more intuitive. Is this so? Should it be changed?

Course part should be used


commander preflight ekf2 check estimator_status

ekf2Check was running regardless of an estimator_status update and not initializing the struct.

I took another pass to simplify a bit more and added a 20s delay before the EKF GPS check is enforced.

Rebased and tested.

@priseborough could you review the preflight ekf2Check change?https://github.com/PX4/Firmware/pull/7764/files#diff-a2bcb238e25d8a97ab2aee48f54dbc48R747
Many users get an initial "PREFLIGHT FAIL: EKF NOT USING GPS" message which is off putting.
I'm certainly open to a better way to do this rather than the hard coded magic number.

Given the GPS TTFF is highly variable and up to 30 seconds for a cold statrt, and the minimum time required  required for EKF GPS checks to pass after that is 10 seconds and typically 20, we should use the last time gnssCheck was false as the time reference instead of time from boot.
Longer term we can also improve the reporting if it does fail to indicate the type of GPS quality problem.
I will submit a patch for the timer change.

See https://github.com/priseborough/Firmware/tree/pr7764 for timer reference change.

Thanks, I've pulled your commit into this PR.


fix typos found by fossies codespell

Fixed 3 typos found by fossies Codespell Project in the 3.3/dev branch.
https://fossies.org/linux/test/coreruleset-3.3-dev.3b4a13b.200602.tar.gz/codespell.html
Those typos are also present in the 3.2 branch, however they are only in either comments or regression tests and I don't think we need to backport them. Otherwise let me know.

Thank you for the contribution. Most welcome. Merging now.


Scrollbar Dragging Haptic Feedback

iOS 13's draggable scrollbar was implemented in https://github.com/flutter/flutter/pull/35829, but it is missing the vibration at the start of the gesture.
We may need to check on a physical device running iOS 13 for the exact type of vibration.

Closing with https://github.com/flutter/flutter/pull/37724


[FLINK-22794][table-api]Upgrade JUnit Vintage for flink-sql-parser and flink-sql-hive-parser

…-hive-parser

What is the purpose of the change
(For example: This pull request makes task deployment go through the blob server, rather than through RPC. That way we avoid re-transferring them on each deployment (during recovery).)
Brief change log
(for example:)

The TaskInfo is stored in the blob store on job creation time as a persistent artifact
Deployments RPC transmits only the blob storage reference
TaskManagers retrieve the TaskInfo from the blob cache

Verifying this change
(Please pick either of the following options)
This change is a trivial rework / code cleanup without any test coverage.
(or)
This change is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(example:)

Added integration tests for end-to-end deployment with large payloads (100MB)
Extended integration test for recovery after master (JobManager) failure
Added test that validates that TaskInfo is transferred only once across recoveries
Manually verified the change by running a 4 node cluser with 2 JobManagers and 4 TaskManagers, a stateful streaming program, and killing one JobManager and two TaskManagers during the execution, verifying that recovery happens correctly.

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (yes / no)
The serializers: (yes / no / don't know)
The runtime per-record code paths (performance sensitive): (yes / no / don't know)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn/Mesos, ZooKeeper: (yes / no / don't know)
The S3 file system connector: (yes / no / don't know)

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

It seems that the flink-sql-parser and flink-sql-parser-hive modules are never tested in CI build. I just added them into test modules, cc @twalthr @zentol .

The CI build runs tests of sql-parser and sql-parser-hive in library now.
Will merge this PR.


Async external request support

We need to improve performance for the number in-progress requests. Presently we use blocking grpc and boost network implementations that mean it is conceivable that a trivial DoS could take place and not necessarily by a malicious party.

Addressed by #53


Link from README to Read The Docs

Using the badge and a textual link.

@cket Please don't just close issues. In a comment, link to the commit that resolves it. If there is no such commit, explain in a comment why you think the issue should be closed.

Closed by commit 19a72a61779e80f0ad88ff5ee4ce9120ccba9309 and merged into master by commit d342cd333c0ccbc7121c4fdd7e919f92b4aa75e4


Remove or improve double tap to 👍

Too many times scroll and stop is registered as double tap.
Sent with GitHawk

Really? I don’t think I’ve ever done this once. What could we do to improve it?
Sent with GitHawk

@rnystrom Then maybe it’s a iOS 12 bug🤔?
Sent with GitHawk


Support customized language

I'm using jfairy 0.5.6.
I want to add a chinese config jfairy_zh.yml and create Fairy by Fairy.create(Locale.CHINA).
In most situations it is ok to use default EnFairyModule as English is most popular language in the world.
However I met with an Exception
java.lang.IllegalArgumentException: No enum constant io.codearte.jfairy.producer.util.LanguageCode.ZH
at java.lang.Enum.valueOf(Enum.java:236)
at io.codearte.jfairy.producer.util.LanguageCode.valueOf(LanguageCode.java:7)
at io.codearte.jfairy.Bootstrap.getFairyModuleForLocale(Bootstrap.java:118)

I believe that return a default value (LanguageCode.EN) of LanguageCode.valueOf()  (or a wrapper method) for unknown locale may be more reasonable, as there is a coming switch-default structure to handle the unknown locale.
    LanguageCode code = null;
    try {
        code = LanguageCode.valueOf(locale.getLanguage().toUpperCase());
    } catch (IllegalArgumentException e) {
        LOG.warn("Uknown locale " + locale);
        code = LanguageCode.EN;
    }

==========
Another question is that ./gradlew build always fails with exceptions signing.keyId and signing.secretKeyRingFile has to be configured while building the project cloning from github. I'm not familiar with gradle :( (I use maven).
If you guys intend to support Chinese language , I would like to help. Thanks.

Hello @Lhfcws , thank you for your interest in our project. In order to add a new locale to jFairy, some more changes are needed, including adding a new code to LanguageCode.java and adding new data providers and a FairyModule class for your languge. If you are interested in doing that, you can use this @andidev Pull Request as an example: https://github.com/Codearte/jfairy/pull/60 . Alternatively, you can just provide the content yamls and submit to us so that we can do the rest, but this will definitely take some time.  Gradle does not build correctly cause it expects you to provide a gpg key id and keyring. In order to make it work, you will have to generate such a key and then put add the required values to a gradle.properties file placed in the user gradle home, by default set to user_home/.gradle in linux. You can find more information on the gradle environment here: https://docs.gradle.org/current/userguide/build_environment.html.

@Lhfcws In fact the problem with keys is a bug in our build script configuration (#79) which occurred in your case as the latest commit in master is also tagged with a release tag. For further reference the quicker workaround is to add -x sign to your Gradle call, e.g.
./gradlew build -x sign

Thanks for the replies.  ./gradlew build -x sign did work.
@OlgaMaciaszek I'll try to go through the project and make sure that my yaml config satisfies all features. It seems that it is not so simple as just providing a yaml config file. For example, the company url will become chinese if I just modify the given faked company names into Chinese.
If I have to guarantee that my yaml config works fine, posting a PR including tests by me is a better way to solve the Chinese support issue.


[client] Issue with resources amount

Describe the bug
When finishing the first quest, you get allocated 2.000.000 wheat and fish, to build a farm it costs you 900 but 900.000 get deducted from your fish balance. I think there's an issue with the multiplier somewhere or with the displayed amount needed to build buildings to the user.
Because of this, buildings that should be shown as disabled because the user doesn't have enough resources appear as buildable but the transaction fails in the smart contract
To Reproduce Steps to reproduce the behavior:

Finish first quest
Build a farm

Expected behavior Amount to build a building should be fixed in the UI

Got you, closing this issue and my PR then


Added test case for #187

Current output:
❯ npm test | faucet
✓ pass file when isNull()
✓ compile a single sass file
✓ compile a single sass file synchronously
✓ compile multiple sass files
✓ compile multiple sass files with includePaths
✓ emit error on sass errors
✓ emit error on sass errors when using sync true
✓ call custom error callback when opts.onError is given
✓ sourcemaps
# former bugs

/Users/callumacrae/Sites/gulp-sass/node_modules/tape/index.js:65
        throw err
              ^
Error: error reading values after body
npm ERR! Test failed.  See above for more details.
✓ former bugsok code 0
not ok 42 no plan found
⨯ fail  1

Closing as it is fixed in the 2.x branch

Did you write a test for it in that branch? If not, why not merge this test anyway?

@callumacrae because it would only result in a failing build for the current version


Migrate logging to prometheus/log.

/cc @antonlindstrom FYI

(this will log to stdout by default, and doesn't support logging to files anymore)

Looks good, thanks @juliusv!


Comments filter wording feedback

I find the comments filter dropdown unexpected. I would not expect labels to change:

Compare with timeline filter:

I am not sure about "Show ". I would have expected "[ ] Resolved Comments" and "[ ] Unresolved Comments". The fact that they show or not is given by the checkmark.
@hbons for UI


Is there a derivative free algorithm in the package for constrained quadratic optimization problem?

Do you have a specific algorithm in mind?


Remove redundant token_context directory

Codecov Report

Merging #1327 into master will increase coverage by 0.04%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1327      +/-   ##
==========================================
+ Coverage   60.02%   60.07%   +0.04%
==========================================
  Files         119      118       -1
  Lines       10740    10732       -8
==========================================
  Hits         6447     6447
+ Misses       3629     3621       -8
  Partials      664      664

Continue to review full report at Codecov.

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


Documentation says library is compatible with React 16.8.0, but peerDependencies is { "react": "^17.0.0 || ^18.0.0", "react-dom": "^17.0.0 || ^18.0.0" }

Describe the bug
From package.json of 0.3.1:
"peerDependencies": {
    "react": "^17.0.0 || ^18.0.0",
    "react-dom": "^17.0.0 || ^18.0.0"
  },

Steps to Reproduce the Bug or Issue

Try to install the library in a project that has react 16 (> 16.8.0):

npm ERR! Could not resolve dependency:
npm ERR! peer<EMAIL_ADDRESS>|| ^18.0.0" from<EMAIL_ADDRESS>npm ERR! node_modules/@tanstack/react-store
npm ERR!   @tanstack/react-store@"*" from the root project

Expected behavior
As per README, I'd expect the package to be installed without peer dependencies conflict.
Screenshots or Videos
No response
Platform

OS: Windows
Browser: N/A, this is a dependency issue
Version: N/A, this is a dependency issue

Additional context
No response

Thanks for the report!
The package.json has been updated to include 16.8.0 and 19 as well as-of 0.6.1
Sorry it took so long to fix.


teamcity-cpp/boost/teamcity_boost.cpp needs some changes to compile with boost 1.70.0

teamcity-cpp/boost/teamcity_boost.cpp needs something like:
#if BOOST_VERSION >= 107000
void log_build_info(std::ostream & out, bool) override {
log_build_info(out);
}
#endif
inside TeamcityBoostLogFormatter to compile with boost 1.70.0

@emmett-b
this repo looks abandoned. So, I use my own fork and it's already fixed for boost 1.70 %)

Hmmm, ok thanks!
Where are we supposed to get updated versions of this?
I guess I should ask JetBrains support...

Could you please send us a pull request with the change?

@dmitry-treskunov my repo really differ nowadays from its parent

@dmitry-treskunov Here it is: https://github.com/JetBrains/teamcity-cpp/pull/27


Wann90 changes

I want to improve a bit the parser to make the method parse_system callable.
This is because many advanced many-body methods (like DMFT) don't store anywhere the system geometry, except for from the Wannier90 output files themselves. This way I can call Wannier90Parser:parse_system in more advanced parser.

Done with the review here. Is the pipeline error expected?

Shouldn't be, as that metainfo is defined in nomad. @ndaelman-hu @ladinesa do you have any idea of why pylint is not passing here?

This is expected as we the target nomad is v1.0.8

Should we update it now to a higher, e.g. 1.1.5? Or something is preventing us of doing it so?
And also, this is just about modifying the yaml in .github/workflows? I am curious.

Yes, but the package should be released which I think is not.

Ok, then this is ready for merging. I deleted the extra unused lines, so I merge directly.


Fix camera cloning of ortho params

Related forum issue: https://forum.babylonjs.com/t/5-14-to-5-15-camera-clone-does-not-clone-camera-mode/32192/3

Just making sure - was this tested? Is it viable to serialize private methods?


#16 wallet balancing.

Coverage decreased (-32.41%) to 60.92% when pulling 4c173c746c689f43bccfa5e345e64454f9f48e44 on feature/#16 into 7fff30d539fa5c0104c38154b99480982a8b0b60 on develop.


Subscription deleted on Azure Service Bus after an exception

Is this a bug report?
Yes

Can you also reproduce the problem with the latest version?
Yes
Environment

Operating system: Windows
Visual Studio version: VS2017 (15.9.9)
Dotnet version: .net core 2.2

Steps to Reproduce

(Write your steps here:)

Consumer in .net core 2.2 which is listening messages on Azure Service Bus
"Random" error happens on Azure Service Bus SDK
MassTransit Consumer handle well this error and start to recycle the connection
When recycling, the subscription on the Topic is removed

Expected Behavior

The subscription should be created again after but is this case it's not.
In the Class ServiceBusNamespaceContext, the method CreateTopicSubscription remove the subscription but should also create it.
When I look the code MassTransit.Azure.ServiceBus.Core and in MassTransit.AzureServiceBusTransport it's almost the same but it's not. The duplication is obvious but I did not find any clue why the creation of the Subscription was removed from the core version.
If my project was in .net full it would work in fact.
Actual Behavior

The subscription is not created again.
Here are the logs I have that show the issue :

Reproducible Demo

(Paste the link to an example project and exact instructions to reproduce the issue.)

I did a PR with what I think will fix the issue :
https://github.com/MassTransit/MassTransit/pull/1391
On long term solution it would be better to have only one merged code to do this.

Thanks, merged. Long term the legacy transport will be removed from MassTransit and only the .NET Core version will be supported.

Thank you


Rework Add-Path to avoid expanding and request an environment refresh

Closes #1614
Closes #1657
A rework of #1657 after everything we have learned from https://github.com/axodotdev/cargo-dist/pull/1657#issuecomment-2555684533. Please read that comment / PR as it describes everything in detail.
I will once again require some help with snapshot updating!
Key points:

Powershell installers no longer forcibly expand "expandable variables" in your PATH, like %EXPAND%
Powershell installers no longer change your user-level Path registry property from REG_EXPAND_SZ to REG_SZ
Powershell installers no longer require a system restart, and now only require a shell restart, like Unix shell installers

I tested this myself by:

Completely resetting my Path to a state before dist touched it
Adding %TEMP% into my Path as a test "expandable" variable
Running a patched powershell installer with these changes, resulting in:

The Path stayed a REG_EXPAND_SZ data type
The Path retained my unexpanded %TEMP%
On a shell restart, I could load the cli tool I installed

Running the patched powershell installer again, nothing happened because it detected the existing install directory in the Path

I even went so far as to wipe out my Path property in the registry to see if it could create it "from scratch" if it wasn't there already, and it worked and was created with the right REG_EXPAND_SZ type.

FYI: We're probably not going to get a chance to review this until after the holidays.
Thank you again for the help, by the way! I know lots of folks are looking forward to not having to reboot after installing stuff, and looks like this PR improves other Path-related stuff too. :slightly_smiling_face:


[Feature] Add Sparse Retrieval Support

Hello there,
Milvus currently supports sparse vectors. I propose adding a new component, MilvusSparseEmbeddingRetriever, to the current integration for retrieving documents from a MilvusDocumentStore using sparse vectors.

@abmami Thanks for your suggestion, we will soon add this feature

@abmami we have added this feature: https://github.com/milvus-io/milvus-haystack/pull/26  FYI


⚠ Turnos 24x7 has degraded performance

In 88a3613, Turnos 24x7 (https://reservas-dev.alternativasinteligentes.com/) experienced degraded performance:

HTTP code: 200
Response time: 879 ms

Resolved: Turnos 24x7 performance has improved in 40b2c49.


Added jscomp runtime tests.

@brad4d expressed the desire to be able to execute jscomp runtime tests on an open source stack, I imagine @ChadKillingsworth will want to look at these changes before anything is done with them. Please let me know if any updates should be made, curious if anyone can offer advice as far as speeding up the build process (notes below).
Changes

Renamed travis_utils/ to utils/ and added build_runtime_tests.sh bash script to that directory, plus moved build_test.sh from root dir to utils/ with the other bash scripts for good measure, so all the shell/Travis stuff is in one spot. Can roll back if needed, just took the initiative on that since shell scripts were starting to pile up at project root.

The tests can be built with yarn runtime_tests:build, ran with yarn runtime_tests:run, or both with yarn runtime_tests. npm run ... will also work for all these.

package.json scripts were added for convenience, including traditional mappings to yarn clean, yarn build, and yarn test:

...
"dependencies": {
    "future-event": "^1.5.0",
    "glob": "^7.1.6",
    "google-closure-library": "^20200719.0.0",
    "jest": "^26.1.0",
    "jsdom": "^16.3.0"
},
...
"scripts": {
    "build": "mvn -DskipTests",
    "build:fast": "mvn -DskipTests -pl externs/pom.xml,pom-main.xml,pom-main-shaded.xml",
    "clean": "mvn clean",
    "runtime_tests:build": "./utils/build_runtime_tests.sh",
    "runtime_tests:run": "jest --testRegex .*/utils/js/.* --verbose=true",
    "runtime_tests": "npm run --silent runtime_tests:build && npm run --silent runtime_tests:run",
    "test": "mvn test"
}
...

Added deps:

future-event for awaiting the end of a JSDOM test session
glob for finding built tests in a directory
google-closure-library for compiling the runtime tests
jest for executing the runtime tests
jsdom for simulating the browser

Notes
I knew pretty much nothing about the compiler's internals or the testing suite going into this, and it was extremely challenging to figure out how to execute these runtime tests even naively in the browser, interop with JSDOM and jest was a whole different challenge, but as far as I can tell, this is a functional proof-of-concept. Any feedback (especially related to speeding up the build stage) would be much appreciated.
Building / Running
Any runtime test will include closure/third_party and goog.testing.testsuite as dependencies, in addition to potentially any other script in the runtime_tests/ directory (which are often imported and tested by other scripts). The only way I have found to build a single test is to provide the entire Closure Library and runtime_tests/ directories as inputs, which is extremely inefficient.
Compiling a single file
java -server -XX:+TieredCompilation \
    -jar $LOCAL_COMPILER \
    -O WHITESPACE_ONLY \
    --language_in ES_NEXT \
    --language_out NO_TRANSPILE \
    --process_common_js_modules \
    --module_resolution NODE \
    --dependency_mode PRUNE \
    --js node_modules/google-closure-library/ \
    --js $ABS_PATH/ \
    --entry_point $FILE

Building the ~150 runtime tests on my i7-4700 takes 5 minutes in WHITESPACE_ONLY mode and 15 minutes in SIMPLE mode, but Jest runs with an async timeout of 5s, and in WHITESPACE_ONLY mode, the output is so large that actually running the tests becomes an unpredictable process (timeout errors, process stalls, etc.). So 15 minutes is the best realistic build time right now with this implementation, which I'm sure can be dramatically improved.
I imagine it is possible to give the compiler all the tests, all the Closure Library scripts, and all the third party scripts and using the --chunk option, and export one big common chunk and then a chunk for each test. I will be trying to implement this myself over the next few days, if anyone has advice or help on that front it would be much appreciated.
My original approach was to serve an express server at the runtime_tests/ and node_modules/google-closure-library/ dirs and generate HTML pages which use <script type="module"> for modules, but that was messier and required running a small server to execute the tests, plus detecting whether or not a test is a module test, but it would skip the build stage completely.
Jest is used to spawn a JSDOM environment for each runtime test, and mark a test as failed if an error is thrown inside JSDOM.
UX
There are a few issues with some of the test files (a few attempt to import scripts that must have been deleted at some point, etc.), so you'll see the final results show 6 failures at present. The full results are logged to stdout for individual tests that trigger a failure.
Building tests
yarn run runtime_tests:build

Running tests
yarn run runtime_tests:run

Examining an individual failure

Also, meant to submit this as a draft - oops 😅

Does a draft mean that we should wait to review this?

@blickly No, please feel free to review whenever you wish.

Sorry it's taken so long for me to find time to review this.
Hey, no worries at all, thank you very much for the reviews, I am absolutely happy to break this up. Will update soon!

@brad4d Let me know if you have any issues with the node/yarn side of things: Make sure you have npm and yarn (npm -i g yarn), then do yarn install in the root dir.

I've imported this PR and sent it for internal review and testing.
Thanks @christiantjl


Add support on apple music

I tried to download music on apple music and it says unsupported URL, please add a support on apple music so we can download higher quality music using your app

This is the command failed outputs:
WARNING: Falling back on generic information extractor.
ERROR: Unsupported URL: https://music.apple.com/ph/album/1544839500?uo=4&app=music&at=1000lNNy&lId=22707162&cId=none&sr=1&src=Linkfire&itscg=30440&itsct=catchall_p1&ct=LFV_fd825e696d55e7ee0a322a2e145e14f4

This is the command failed outputs:
WARNING: Falling back on generic information extractor.
ERROR: Unsupported URL: https://music.apple.com/ph/album/1544839500?uo=4&app=music&at=1000lNNy&lId=22707162&cId=none&sr=1&src=Linkfire&itscg=30440&itsct=catchall_p1&ct=LFV_fd825e696d55e7ee0a322a2e145e14f4

You must be joking

You must be joking

You must be joking

No, it's real

You must be joking

No, it's real

youtube-dl doesn't support any sites which require DRM circumvention, if they would, this project couldn't exist publicly

youtube-dl doesn't support any sites which require DRM circumvention, if they would, this project couldn't exist publicly

youtube-dl can't support any services which would require DRM circumvention

Okay thanks 💞 I just thought there was no DRM circumvention on apple music that's why I posted this ...

youtube-dl can't support any services which would require DRM circumvention

Okay thanks 💞 I just thought there was no DRM circumvention on apple music that's why I posted this ...


Request: Support filtering by multiple types in \MangoPay\FilterTransactions

I would like to only get transactions that are either PAYINs or PAYOUTs. Now I have to do two requests to your API and then merge the results. This slows things down needlessly.
Allowing a comma separated list would solve it: $filter->Type = 'PAYIN,PAYOUT';

Hi @leihog, thanks for the feedback. We'll soon be releasing a dedicated reporting feature which will allow you to easily do this :-)


Expose "Change icon" to obsidian command

Is your feature request related to a problem? Please describe.
To change icon of current note user would need 2 mouse clicks
Describe the solution you'd like
Instead "Change icon" could be assign to hotkey (I assume Obsidian API allows that)
Describe alternatives you've considered
Use the damn mouse
Additional context

Related: https://github.com/FlorianWoelki/obsidian-iconize/discussions/547

So there is a hotkey already available for setting an icon (you can see it here). Additionally, there is also a command available which is called: "Set icon for file"

@FlorianWoelki Am I missing something?

You need to focus the file input

Aaah, I was in "reading" view, that's why.


add test configuration

looks really good! however, please check your indentation settings. should be always 2 spaces.


Propogate api version

fixes #802

:x: There is undocummented changes. Run chronus add to add a changeset or click here.
The following packages have changes but are not documented.

:x:@azure-tools/typespec-client-generator-core

Show changes

You can try these changes at https://cadlplayground.z22.web.core.windows.net/typespec-azure/prs/805/
Check the website changes at https://tspwebsitepr.z22.web.core.windows.net/typespec-azure/prs/805/


Context API

Is there a bug using the Context API? I have done an example in CodeSandBox: https://codesandbox.io/s/jz1j2row15
When moving the Draggable it starts mixing up things, even though they all have a key prop. Besides that, when using create-react-app and using the Context API, after moving around the Draggable it throws an error saying it can read the id of the element, which is being used by the key prop.

Hi @vititu ,
Thanks for submitting the issue with an example code. I think I know the problem here. I will commit a fix with the next version asap. I will let you know.

It is fixed on v0.4.8.
see https://codesandbox.io/s/1y25z3v7yl

Wow, that was fast! Thanks! 😁


Animations package hides widget, but widget is still accessible

I'm trying to use Fade for hiding or showing a FAB button with an animation, but after it is hidden it is still accessible.
It is possible to notice this in the Fade example. Add an action to the onPressed of the example FAB and tap HIDE FAB button. Then, tap the region where the FAB was. The action will still happen.

The FadeScaleTransition should probably make it clearer that you probably want to drop the widget from the tree when it fades to 0.
In general, I think we should at the least add an assertion around hit testing for widgets like this where they check if the opacity is 0, and if so assert with a suggestion that you're using the wrong kind of widget to do hit testing without drawing anything.  See https://github.com/flutter/flutter/issues/51000#issuecomment-587980377
/cc @goderbauer @shihaohong

You should be able to work around this today, btw, by just dropping the widget yourself in cases where the animation controller's value has hit zero.

I'm new to flutter. Before finding the animations package, I was hiding my fab using Visibility widget. Is it possible to set the visibility based on that? Like, checking the alpha for deciding the visibility or something in these lines. This would make the Visibility widget trigger at the right moment in the animation.

Yeah, something like this should work:
      floatingActionButton: AnimatedBuilder(
        animation: _controller,
        builder: (BuildContext context, Widget child) {
          return FadeScaleTransition(
            animation: _controller,
            child: child,
          );
        },
        child: Visibility(
          visible: _controller.value > 0.0,
          child: FloatingActionButton(
            child: Icon(Icons.add),
            onPressed: () {},
          ),
        ),
      ),

We should update the examples in the example app to actually do this correctly.

I can fix the example app! I'll probably create a separate issue to improve the sample docs with:

Short code samples (if possible, for cases like the one above)
Links to the Material motion system article that is now published

@dnfield thanks! That's what I was thinking about. I'll try that.

It seems Visibility alone won't be able to solve this. It looks like it is testing the controller's value for visibility before the animation even starts, which makes it flash into existence and disappear through the animation but still accessible or invisible when it should be shown.
Inverting the test to == 0 will show and hide when intended, but the hiding animation is hidden by visibility being triggered before we actually need it to. It will have the same behaviour of using _showFab here.
Looking at the Visibility documentation, I found IgnorePointer, which is what I'm going to use for now. It works pretty well:
      floatingActionButton: AnimatedBuilder(
        animation: _controller,
        builder: (BuildContext context, Widget child) {
          return FadeScaleTransition(
            animation: _controller,
            child: child,
          );
        },
        child: IgnorePointer(
          ignoring: !_showFab,
          child: FloatingActionButton(
            child: Icon(Icons.add),
            onPressed: () { },
          ),
        ),
      ),

What about if you still use the Visibility widget, but set visible to _showFab?
The problem with using just IgnorePointer is that it will still build the child widget, which may be non-trivial.  For a FAB it's probably not that big of a deal, but depending on what your actual widget is it could make a big difference, particularly if your actual widget animates/pumps frames on every tick.

Yes, not ideal, but it's going to be it for now.
If I use _showFab in Visibility, it does work, but the hiding animation isn't shown because Visibility is hiding the FAB before the animation starts.

Ahh ok, there should be some way to fix that but sounds good.

@bergamin Could you try using AnimationController.status == AnimationStatus.dismissed as well?

@shihaohong you mean in Visibility? It doesn't show the FAB.
I put a Toast in the show/hide button to check what values are being read at that time (which I believe is the same time Visibility is being triggered) and it shows status as AnimationStatus.forward and AnimationStatus.reverse

I tried the opposite, testing != dismissed and the animations are shown correctly, but the FAB is still accessible and on hitting it, the toast would show status as dismissed. I believe these status that require the animation to end before triggering Visibility won't work this way. Unless there is a way to delay Visibility or make it keep testing until we get to the desired animation stage


Your build of st as a patch.

I was trying to apply other patches from suckless.org but it failed. So I wonder that the changes you did are so extensive that they does not allow the other patches to be applied.
Would it be possible to create your build as a patch that it would allow to add other patches?
Regards,
Piotr

@ZaxonXP Maybe you could copy config.h to config.def.h and manually patch the required patches from suckless.org. It does work but it is time consuming.


Remove deprecated usage of generated_jit and additional cleanup

This PR replaces the deprecated generated_jit function with the overload mechanism. Additionally, it adds docstrings for most functions and removes the automatic thread count modification in the parallel query functions in favor of an explicit modification in the user's code.

This should also close #27.


update

add wepy support: https://github.com/wepyjs/wepy

I'm not sure if it should just be filetype=vue as the other one. Are wpy files different from vue files?

I changed it so it uses vue instead of vue.html.javascript.css, that should work. If it doesn't, feel free to open a new pull request.


ModInv

Fix this plz:
BigInteger.js

all 'BigInt(' -> replace to 'BigInteger(',
all 'BigInt.' -> replace to 'BigInteger('
I see you wanted to make this shorter, but this not working correctly.

replace BigInteger to Integer, not BigInteger. Because you have in code Integer.zero, Integer.one, etc...
BigInteger.prototype.modInv = function (n) {
var t = Integer.zero, newT = Integer.one, r = parseValue(n), newR = this.abs(), q, lastT, lastR;
while (!newR.equals(Integer.zero)) {

to do test this you can using this example:

import gmpy
gmpy.invert(1234567,<PHONE_NUMBER>)
mpz(989145189)

abs1 = new BigInteger('1234567');
modulus = new BigInteger('1000000007');
document.write(
'modInv:'+
''+abs1.toString()+
" modInv "+modulus+
' = '+abs1.modInv(modulus)
);

add new before BigInteger:

function toBase(n, base) {
base = new BigInteger(base);

Also BigInteger.toString(2) not working. There is not working base array function,
and I used custom function, with byte array, like this:

Sorry, I don't understand your question.
There is no BigInteger function in this library, only bigInt. new BigInteger('1234567') will throw an error because BigInteger is not defined.
I will not be renaming this function because that would break all of the code of people who use this library.

Also BigInteger.toString(2) not working.

The correct way to do this with this library is bigInt(2).toString().

Thanks for your fix. I see now binary string is working.
console.log(
	bigInt(-100500).toString(2) //-11000100010010100
);


Show users who has authored and approved changes

I'm not sure I like that has_one.

@AndrewVos apart from disliking has_one do you have any concrete concerns about it?

Yeah, the overloaded use of has_one is less explicit than I would like it to be. Consider adding approved_by_id => User


polygon2d - ability to see the active bone when painting weights

Godot version:
3.1
Issue description:
It is kind of hard to paint weights of joints which you can't see. Their position in space is quite important when deciding which verts to paint

I hope bone display is added :)

closing this- its a duplicate of https://github.com/godotengine/godot/issues/20950


Apply layout inside estimator

Summary
apply_layout is called inside the estimator.
This PR enables the following code:
n = 3
qc = QuantumCircuit(n)
qc.x(n - 1)
qc.h(range(n))
qc.cx(range(n - 1), n - 1)
qc.h(range(n - 1))

op = SparsePauliOp("IZI")

backend = FakeBelem()
trans_qc = transpile(qc, backend)

expval = Estimator().run(trans_qc, op).result().values[0]

Resolves https://github.com/Qiskit/qiskit-ibm-runtime/issues/338
Details and comments
I'll rebase this PR after #10947 is merged.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

41 of 42 (97.62%)  changed or added relevant lines in 5 files are covered.
9 unchanged lines in 2 files lost coverage.
Overall coverage increased (+0.002%) to 87.028%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

qiskit/quantum_info/operators/symplectic/sparse_pauli_op.py
16
17
94.12%

Files with Coverage Reduction
New Missed Lines
%

crates/qasm2/src/lex.rs
3
91.67%

crates/qasm2/src/parse.rs
6
97.6%

Totals

Change from base Build<PHONE_NUMBER>:
0.002%

Covered Lines:
74197

Relevant Lines:
85256

💛  - Coveralls

I close this PR. Instead, the documentation will clearly state how to use apply_layout.


[ANT-763] Assigned by default to a study imported all groups of the user

Problem Statement
As a user, when I import a study on Antares Web, the imported study must be assigned to all the groups to which I belong.

It's possible for a study to be affected to multiple groups.

This issue is related to https://github.com/AntaresSimulatorTeam/AntaREST/issues/1559 because to reproduce the VMs group behaviour (which is what's asked in the initial user request), we need to assign the user groups to the study and put its PublicMode to None


Does it work on 9.03

First thing,
Thanks for releasing this, I'm really excited to test it out!
Does it work on 9.03 tho?
The description is really unclear, I'm afraid it's only related to 9.00 & 11.00,
Thanks once again,
Looking forward for your explanation!

by the looks of things people are working on porting it to all firmwares 9.00 through 11.00
but none is done yet


Allowing the ci.yaml to be modified

Currenty, STEP-VERIFY-CHANGED-DIRECTORIES will fail if a PR includes changes to the ci.yaml file. The current implementation rejects any files in a PR that do not have a status of added therefore all modified files are rejected. This patch adds a single exception to that rule for the ci.yaml file.  This will allow users to set merge:false in ci.yaml and covers a few edge cases where a cert_projet_id may need to be modified.

Please include unit tests to get the code coverage up to 100%.

Closing in favor of #176.


Map symbology does not update when the last remaining inspection is deleted

The tree feature's details which depict the Condition of the tree also do not update.
This was one piece of feedback that came out of Sarat's testing of the app (result from test case 4.5).
Steps to repro:

Find a tree which only has one inspection record left.
Take note of the tree's condition and map symbol color, then delete the inspection.
Notice that neither the map symbol updates nor does the Condition attribute of the feature get cleared.

@mikewilburn A PR has opened that solves this.

@mikewilburn This change has been merged into master. Please confirm it has been fixed.

Verified on master.


SpeechInputSource, SpeechInputHandler, & SceneLauncher Update

https://github.com/Microsoft/HoloToolkit-Unity/issues/744 Updated obsolete KeywordManager usages in SceneLauncher.
Added PersistentKeywords properties to SpeechInputSource and SpeechInputHandler so that speech input can live across scenes.

Does the scene launcher really belong in the main folder of the toolkit?
I feel like it may belong in Examples or Tests

@ForrestTrepte I just did a major overhaul for you to do what I described above.  Let me know what you think.

@ForrestTrepte I believe I've covered all your recent change requests.  let me know if I've missed something.

Ship it!

I'd love to but I need one more review


hyprland/workspaces: fix crash on monitor off/on

Waybar would crash after turning off/on monitor (especially after https://github.com/hyprwm/Hyprland/pull/3713).
Might fix https://github.com/Alexays/Waybar/issues/2039 and https://github.com/Alexays/Waybar/issues/2015.

Nice thanks!

It is still crashing on my system everytime I turn off and on my monitor.
Arch / Hyprland
Waybar v0.9.24
[faugus@archlinux ~]$ waybar
[2023-12-07 17:06:16.239] [info] Using configuration file /home/faugus/.config/waybar/config
[2023-12-07 17:06:16.240] [info] Discovered appearance 'dark'
[2023-12-07 17:06:16.240] [info] Using CSS file /home/faugus/.config/waybar/style.css
[2023-12-07 17:06:16.242] [warning] Mapping is not an object
[2023-12-07 17:06:16.243] [info] Hyprland IPC starting
[2023-12-07 17:06:16.253] [info] Bar configured (width: 2560, height: 40) for output: DP-1
basic_string::_M_create
[2023-12-07 17:06:18.887] [info] Bar removed from output: DP-1
Segmentation fault (core dumped)
[faugus@archlinux ~]$


Babel 5.3.0

Looks like Babel 5.3.0 has some pretty significant speed improvements.  Any chance of bumping the version to 5.3.0?  See https://twitter.com/babeljs/status/598303570824421376.

This plugin actually relies on the broccoli-babel-transpiler plugin to do the actual transpilation. Luckily, they use a caret dependency version for babel-core (^5.0.0), so it'll use the highest 5.x.x version available. If you wipe your node_modules directory and reinstall, you'll see that it's using version 5.4.2. You can check in this file:
node_modules/ember-cli-babel/node_modules/broccoli-babel-transpiler/node_modules/babel-core/package.json

I imagine that we won't bump the major version of this plugin until a 6.x.x release of Babel.

Ya confirmed.

Confirmed after clearing npm cache.  Thanks.


Add jsonnet (https://jsonnet.org) installer

links:
https://golangrepo.com/repo/grafana-jsonnet-language-server-go-server-applications
https://github.com/grafana/jsonnet-language-server

Looks good!


2x challange 209, none 210

There is no link to week 210 on Challenges page.
https://github.com/manwar/theweeklychallenge/blob/3206cbd21588edcd1e710a071b283632f62253c3/data/challenges.yml#L4

Sorry for late response. This is no longer an issue, thanks for reporting.


Ignore Permissions option not working

Azure Data Studio Version:  1.13.1
SQL Server Schema Compare 1.0.0

When comparing two databases, I have selected the "Ignore Permissions" option, but differences are still flagged, and the only difference is the permissions.

Thanks for finding this bug! We will investigate why this option isn't being honored.
As a workaround, the permissions should be ignored if "Permissions" is unchecked in the Include Object Types

Although the "Ignore Permissions" checkbox has been removed (which aligns with Visual Studio):

..., the "Include Object Types" checkbox of "Permissions" is still resolutely ignored by schema compare - so the workaround proposed by @kisantia does not work.
On exporting the .scmp file, the XML element below was not found. Even copying this across from what Visual Studio produces, reloading the .scmp file in Data Studio and recomparing does not make any difference:
      <PropertyElementName>
        <Name>Microsoft.Data.Tools.Schema.Sql.SchemaModel.SqlPermissionStatement</Name>
        <Value>ExcludedType</Value>
      </PropertyElementName>

Version: 1.47.1 (system setup)
Commit: b6f7beb01f92adaa4b79b6b6f3ac704e95cafe6e
Date: 2024-01-10T02:15:02.628Z
VS Code: 1.82.0
Electron: 25.8.4
Chromium: 114.0.5735.289
Node.js: 18.15.0
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.19045


Fix default value when using sequelize fn

for example postgres function "now()" should be converted into
                "updated_at": {
                    "type": Sequelize.DATE,
                    "defaultValue": Sequelize.fn('Now'),
                    "allowNull": false
                }

which is not the case now

Thank you!


Completions: prefer slight misspelling over distant substring match

Issue Type: Bug
Type:
const supercaliFragilisticExpialidocious = 0;
const frajile = 1;

Type fragile().

Expected: Corrects to frajile.
Actual: Corrects to supercaliFragilisticExpialidocious.
VS Code version: Code - Insiders 1.25.0-insider (499122702e462b458117122a6885e6aeb3cec0a4, 2018-06-29T05:19:31.733Z)
OS version: Linux x64 4.13.0-43-generic

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-6700HQ CPU @ 2.60GHz (8 x 1536)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: disabled_softwarerasterization: disabled_softwarevideo_decode: unavailable_softwarevideo_encode: enabledvpx_decode: unavailable_softwarewebgl: enabledwebgl2: enabled

Load (avg)
1, 1, 1

Memory (System)
7.63GB (1.38GB free)

Process Argv
/usr/share/code-insiders/code-insiders tests/cases/fourslash/

Screen Reader
no

VM
0%

Extensions: none

Unsure... We only swap neighbouring characters but we don't skip or replace characters in the search word. Even that can confuse people and I don't know if more graceful matching is wanted. Tho, we now do have a setting in which this can be captured.

supercaliFragilisticExpialidocious doesn't contain fragile as a substring though, only fragil -- so it must be skipping to the E. There's no autocorrect if I use Oxpialidocious instead.

supercaliFragilisticExpialidocious doesn't contain fragile as a substring though, only fragil -- so it must be skipping to the E.

I don't call that correction but matching. The logic is that every character of the search string must appear (in order) in the word and that the first match must be strong (those are matches on upper-case letters or after separators like - and _)


[vue] Slider props minStepsBetweenThumbs missing

Description
Slider does not have props minStepsBetweenThumbs, apparently this happened after combining RangeSlider and Slider
@ark-ui/vue v0.8.0
https://github.com/chakra-ui/ark/blob/c8177031497f2f725b7062eec64b1e36a167292a/packages/vue/src/range-slider/range-slider.props.ts#L44-L46
Link to Reproduction (or Detailed Explanation)
https://github.com/chakra-ui/ark/blob/main/packages/frameworks/vue/src/slider/stories/range-slider.story.vue
Steps to Reproduce
https://github.com/chakra-ui/ark/blob/14d369dad381632d4fbfa2b48c14ec60c3dbc076/packages/frameworks/vue/src/slider/stories/range-slider.story.vue#L121-L133
Ark UI Version
0.10.0
Framework

[ ] React
[ ] Solid
[X] Vue

Browser
No response
Additional Information
No response

Thanks for the notice! Fixed now


new Knesset API should include data regarding votes

open the new API doc - https://docs.google.com/document/d/1FHqabNsYlSNyul1lArmnONLHnYGA4XQNMs3k-dPgUok/edit
look for data regarding votes

expected

should have votes data
possibly with relation to MKs and bills / laws

actual

no vote data

notes

voting data is still available in the old API - http://knesset.gov.il/Odata/Votes.svc/

@maryLoi @shevyk please notify Knesset

related issue: #135

@OriHoch I think @shevyk and me  discussed and explained this in slack before. The team responsible for the new hakika api has no connection or authority over votes. The new api for hakika is because the team lead is responsible for maagar hakika leumi. BTW also in the old api there is a workaround for the problem you encountered for wrong vote results. I'll try to find it again
Please close this. since currently this is as designed

I guess my memory is not so good in my old age.. sorry :older_man:
I think there is still some problem here as the new API has many details other then legislature, and no where in their doc it says it's only for legislature
the API doc even has mention of votes:

למשל הלינק של כל דף של חבר כנסת או קוד הזיהוי של הח"כ ב-ODATA של ההצבעות שונה מזה שנעשה בו שימוש בסנהדרין

@shevyk @maryLoi could you confirm this with Knesset people?
Also, if votes are indeed not part of new API, is there also other data not available?
Will Knesset keep supporting the old votes API?

also, if old votes API is the API knesset would like us to use - then it has some critical bugs (which are why we keep doing some part in HTML scraping..)

@OriHoch about the api doc It means the Id is different for mk.. in the new system and the old system. the new system exposes a translation table we can use if we need it (and we do)
About the critical bugs, there is a work around besides the html, I didn't implement since I hoped we'll get a better solution, but i've got it working correctly I'll post it here and we can open another issue to migrate the html vote there


added callback interface

looks good. the advantage of the callback is that whomever loads the DLL can choose how they want to synchronize the input queue between the two threads.
We can always make a narrower interface that wraps this one if needed as this one is the most flexible.


Add documentation about Security concerns

Like in the example today. It exposes 22. But on all interfaces. So when you work in a Public Wifi. Everyone can connecto your SSH.
Maybe all should be locally by default :)

good idea,
i'll add that to the docs, it's always a gap between explaining docker and the product :)

done :)
https://dockware.io/docs#security


Don't collapse new nested node

To repro:
Create a node with some text in it
Hit enter to create a new node, then hit tab to indent the node
The indented node will be created, but it's collapsed. It should not be collapsed.
https://recordit.co/6nIasMSuQ8

fixed with commit for #6

LGTM


[1.12.2] when i'm standing on charging pad minecraft is crashing.

Please use the search functionality before reporting an issue. Also take a look at the closed issues!
Issue description:
When I'm standing on chraging pad for Robit, minecraft is crashing. I'm playing on my prive server.
Version:
Forge:<IP_ADDRESS>68
Mekanism:<IP_ADDRESS>9
crash-2019-01-16_19.20.16-client.txt
latest.log

Some mod is not registering data manager properties properly, or you have a difference of mods between server/client. You'll need to remove mods one my one until it stops.


Invite me to the organization

git id - hi-tech-AI
open to contribute and learn and grow

Hello there, welcome to openAOD, I have sent an invite to join the org. Happy to have you here : )


Binding to <IP_ADDRESS> allows proxy request from remote machine

In Grabacr07.KanColleWrapper/KanColleProxy.cs
public void Startup(int proxy = 37564) calls FiddlerApplication.Startup(proxy, false, true); to listen to the proxy port. Setting the fourth parameter of Startup() true allows remote client to connect and make request.
If firewall on user machine is not properly configured, a malicious attacker can use this vulnerability to proxy any requests.

Was it a bug or a feature?
I didn't see any benefit on allowing remote connections, because KanColleViewer is supposed to intercept game requests for one user only.

Cite (from a chinese blog):
https://hjc.im/jing-ti-kancolleviewerda-kai-de-gong-wang-ipduan-kou-chan-sheng-de-an-quan-wen-ti/

In simple words, running KCV would effectively turns your computer into a open proxy if your computer is directly connecting to the internet or the broadband router is configured into DMZ mode (i.e. one-to-one IP mapping, effectively exposing your computer in the internet).

The issue seems fixed in upstream. Could anyone test and close the issue?

just a remark, this ticket is a duplicate of #138, can be closed.


Make debugging in release build a compiler error

This commit adds macros ddbg and dprintln, which are defined in debug builds, but not in release builds. So, if they appear in code they will result in a compile time error when doing a release build. These are only added to the oq3_semantics crate. They would have to be duplicated to use them in other crates. Or perhaps moved to the lexer crate, which all others depend on, directly or indirectly.
Using these requires also #[cfg(debug_assertions)] when importing them.
I have seen an implementation of something similar that includes a definition of the macros for release builds. This does not then require #[cfg(debug_assertions)] on import. We could do something similar, by making ddbg expand to something that fails to compile, say references an undefined variable.  But that would not be very robust.
There is a probably a standalone, registered, crate to do things like this. But I was unable to find such a crate. Nor did I find online any reference to the basic problem this is meant to solve.

Converted to draft mode. Half of what this PR (#43) does is done much better in

#47

But #47 does nothing to replace other half of #43 , that is dprintln! .

I'll close this. It could be re-opened later, if println! for debugging creeps into the main branch.


Allow override start script from now.json

It would be nice to have the ability to set a custom start script in the now.json file. This allows to have multiple "now" projects in a single repository.

Not sure how this would work, how would you have multiple projects in one directory 🤔
You could have multiple projects in one repository, in separate directories, all with their own now.json / package.json.

As of Now 2.0, there are no start or now-start scripts anymore. Please adapt your deployment, if you have the time! 😊


Use Templatus for F# code generation

https://github.com/kerams/Templatus

Is there any reason why?


Ros2 devel

Issue #, if available:
Description of changes:

Removed the commented code from CMakeLists.txt.
Changed turtlebot3_navigation to turtlebot3_navigation2
Ported the small_house.launch.py from ros1 to ros2 that view small_house_world in gazebo.
Deleted ros1 launch files and ported small_house_turtlebot_navigation.launch to ros2 that view small_house_world, spawn turtlebot3 waffle_pi along with navigation.
Steps to execute launch files:
$ source /opt/ros/dashing/setup.bash
$ export TURTLEBOT3_MODEL=waffle_pi
$ source /usr/share/gazebo/setup.sh
$ colcon build
$ source install.setup.sh
$ ros2 launch aws_robomaker_small_house_world small_house.launch.py gui:=true
OR
$ ros2 launch aws_robomaker_small_house_world small_house_turtlebot_navigation.launch.py gui:=true

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

When using small_house_turtlebot_navigation.launch.py, I got the following error:
[ERROR] [launch]: Caught exception in launch (see debug for traceback): "package 'turtlebot3_description_reduced_mesh' not found, searching: [...]"

I am aware of this issue stemming from the ROS1 version of this package. There are no instructions for obtaining this package.
However, I have confirmed that using small_house.launch.py works. Thanks.

The instructions for launching ros2 launch aws_robomaker_small_house_world small_house_turtlebot_navigation.launch.py gui:=true will need to be updated (as it depends on the turtlebot3_description_reduced_mesh package), but it looks like it is working.


LNURLp webhook stops working after a while (RaspiBlitz)

I have a few LNURLp links which are supposed to dispatch webhook requests. Initially this works without issue, but after a few hours of running LNbits, no request ever reaches my webhook server (running on the same local network).
On success, the response code shows up:

But after runnig for a while, LNbits doesn't even seem to try to dispatch the webhook request:

The server handling the webhooks is running entirely without issue during this.
Restarting LNbits always fixes this. I've tried sending 10 payments without restarting, and none of them triggered the webhook. After restarting, payments triggered it again.

What is the LNbits version/commit? Can you upgrade to the latest master? What do the logs show? We had some nursery problems (stuff related to the async listeners for incoming payments) in the past but they seem to have been mostly fixed by now.

Yes, that's what is needed.

Sadly, the issue persists. I got two donations during today's stream, but only the first one ever reached my webhook. Both confirmed as settled by my node, as well as LNbits itself. But the second one didn't get sent to the webhook. Interestingly enough, not even the first transaction has the wh_status: 200 included, even though my server did respond with that.

Just found something very interesting in the logs. This is from a payment that didn't make it to the webhook:
Jun 11 06:31:30 raspberrypi sh[11854]: Traceback (most recent call last):
Jun 11 06:31:30 raspberrypi sh[11854]:   File "/home/lnbits/lnbits/venv/lib/python3.7/site-packages/sqlalchemy/engine/base.py", line 1277, in _execute_context
Jun 11 06:31:30 raspberrypi sh[11854]:     cursor, statement, parameters, context
Jun 11 06:31:30 raspberrypi sh[11854]:   File "/home/lnbits/lnbits/venv/lib/python3.7/site-packages/sqlalchemy/engine/default.py", line 608, in do_execute
Jun 11 06:31:30 raspberrypi sh[11854]:     cursor.execute(statement, parameters)
Jun 11 06:31:30 raspberrypi sh[11854]: sqlite3.OperationalError: no such table: targets
Jun 11 06:31:30 raspberrypi sh[11854]: The above exception was the direct cause of the following exception:
Jun 11 06:31:30 raspberrypi sh[11854]: Traceback (most recent call last):
Jun 11 06:31:30 raspberrypi sh[11854]:   File "/home/lnbits/lnbits/venv/lib/python3.7/site-packages/hypercorn/trio/lifespan.py", line 29, in handle_lifespan
Jun 11 06:31:30 raspberrypi sh[11854]:     await invoke_asgi(self.app, scope, self.asgi_receive, self.asgi_send)
Jun 11 06:31:30 raspberrypi sh[11854]:   File "/home/lnbits/lnbits/venv/lib/python3.7/site-packages/hypercorn/utils.py", line 239, in invoke_asgi
Jun 11 06:31:30 raspberrypi sh[11854]:     await app(scope, receive, send)
Jun 11 06:31:30 raspberrypi sh[11854]:   File "/home/lnbits/lnbits/venv/lib/python3.7/site-packages/quart/app.py", line 2117, in __call__
Jun 11 06:31:30 raspberrypi sh[11854]:     await self.asgi_app(scope, receive, send)
Jun 11 06:31:30 raspberrypi sh[11854]:   File "/home/lnbits/lnbits/venv/lib/python3.7/site-packages/quart/app.py", line 2140, in asgi_app
Jun 11 06:31:30 raspberrypi sh[11854]:     await asgi_handler(receive, send)
Jun 11 06:31:30 raspberrypi sh[11854]:   File "/home/lnbits/lnbits/venv/lib/python3.7/site-packages/quart_trio/asgi.py", line 115, in __call__
Jun 11 06:31:30 raspberrypi sh[11854]:     break
Jun 11 06:31:30 raspberrypi sh[11854]:   File "/home/lnbits/lnbits/venv/lib/python3.7/site-packages/trio/_core/_run.py", line 741, in __aexit__
Jun 11 06:31:30 raspberrypi sh[11854]:     raise combined_error_from_nursery
Jun 11 06:31:30 raspberrypi sh[11854]:   File "/home/lnbits/lnbits/lnbits/extensions/splitpayments/tasks.py", line 16, in register_listeners
Jun 11 06:31:30 raspberrypi sh[11854]:     await wait_for_paid_invoices(invoice_paid_chan_recv)
Jun 11 06:31:30 raspberrypi sh[11854]:   File "/home/lnbits/lnbits/lnbits/extensions/splitpayments/tasks.py", line 21, in wait_for_paid_invoices
Jun 11 06:31:30 raspberrypi sh[11854]:     await on_invoice_paid(payment)
Jun 11 06:31:30 raspberrypi sh[11854]:   File "/home/lnbits/lnbits/lnbits/extensions/splitpayments/tasks.py", line 30, in on_invoice_paid
Jun 11 06:31:30 raspberrypi sh[11854]:     targets = await get_targets(payment.wallet_id)
Jun 11 06:31:30 raspberrypi sh[11854]:   File "/home/lnbits/lnbits/lnbits/extensions/splitpayments/crud.py", line 8, in get_targets
Jun 11 06:31:30 raspberrypi sh[11854]:     rows = await db.fetchall("SELECT * FROM targets WHERE source = ?", (source_wallet,))
Jun 11 06:31:30 raspberrypi sh[11854]:   File "/home/lnbits/lnbits/lnbits/db.py", line 48, in fetchall
Jun 11 06:31:30 raspberrypi sh[11854]:     result = await conn.execute(query, values)
Jun 11 06:31:30 raspberrypi sh[11854]:   File "/home/lnbits/lnbits/lnbits/db.py", line 26, in execute
Jun 11 06:31:30 raspberrypi sh[11854]:     return await self.conn.execute(query, values)
Jun 11 06:31:30 raspberrypi sh[11854]:   File "/home/lnbits/lnbits/venv/lib/python3.7/site-packages/sqlalchemy_aio/base.py", line 288, in execute
Jun 11 06:31:30 raspberrypi sh[11854]:     self._connection.execute, *args, **kwargs)
Jun 11 06:31:30 raspberrypi sh[11854]:   File "/home/lnbits/lnbits/venv/lib/python3.7/site-packages/sqlalchemy_aio/base.py", line 230, in _run_in_thread
Jun 11 06:31:30 raspberrypi sh[11854]:     return await _self._worker.run(_func, args, kwargs)
Jun 11 06:31:30 raspberrypi sh[11854]:   File "/home/lnbits/lnbits/venv/lib/python3.7/site-packages/sqlalchemy_aio/trio.py", line 70, in run
Jun 11 06:31:30 raspberrypi sh[11854]:     return resp.unwrap()
Jun 11 06:31:30 raspberrypi sh[11854]:   File "/home/lnbits/lnbits/venv/lib/python3.7/site-packages/outcome/_impl.py", line 138, in unwrap
Jun 11 06:31:30 raspberrypi sh[11854]:     raise captured_error
Jun 11 06:31:30 raspberrypi sh[11854]:   File "/home/lnbits/lnbits/venv/lib/python3.7/site-packages/sqlalchemy/engine/base.py", line 1003, in execute
Jun 11 06:31:30 raspberrypi sh[11854]:     return self._execute_text(object_, multiparams, params)
Jun 11 06:31:30 raspberrypi sh[11854]:   File "/home/lnbits/lnbits/venv/lib/python3.7/site-packages/sqlalchemy/engine/base.py", line 1178, in _execute_text
Jun 11 06:31:30 raspberrypi sh[11854]:     parameters,
Jun 11 06:31:30 raspberrypi sh[11854]:   File "/home/lnbits/lnbits/venv/lib/python3.7/site-packages/sqlalchemy/engine/base.py", line 1317, in _execute_context
Jun 11 06:31:30 raspberrypi sh[11854]:     e, statement, parameters, cursor, context
Jun 11 06:31:30 raspberrypi sh[11854]:   File "/home/lnbits/lnbits/venv/lib/python3.7/site-packages/sqlalchemy/engine/base.py", line 1511, in _handle_dbapi_exception
Jun 11 06:31:30 raspberrypi sh[11854]:     sqlalchemy_exception, with_traceback=exc_info[2], from_=e
Jun 11 06:31:30 raspberrypi sh[11854]:   File "/home/lnbits/lnbits/venv/lib/python3.7/site-packages/sqlalchemy/util/compat.py", line 182, in raise_
Jun 11 06:31:30 raspberrypi sh[11854]:     raise exception
Jun 11 06:31:30 raspberrypi sh[11854]:   File "/home/lnbits/lnbits/venv/lib/python3.7/site-packages/sqlalchemy/engine/base.py", line 1277, in _execute_context
Jun 11 06:31:30 raspberrypi sh[11854]:     cursor, statement, parameters, context
Jun 11 06:31:30 raspberrypi sh[11854]:   File "/home/lnbits/lnbits/venv/lib/python3.7/site-packages/sqlalchemy/engine/default.py", line 608, in do_execute
Jun 11 06:31:30 raspberrypi sh[11854]:     cursor.execute(statement, parameters)
Jun 11 06:31:30 raspberrypi sh[11854]: sqlalchemy.exc.OperationalError: (sqlite3.OperationalError) no such table: targets
Jun 11 06:31:30 raspberrypi sh[11854]: [SQL: SELECT * FROM targets WHERE source = ?]

Hmm, sorry, this is a different issue caused by the fact that I forgot to tell you to rerun the database migrations after the update.
The fact that this error has occurred means the asynchronous listener was working though.
Running the migrations should fix it. Should be something like https://github.com/rootzoll/raspiblitz/blob/fc1d44104bc22d85ebebd7b0d6ee2240722a9edf/home.admin/config.scripts/bonus.lnbits.sh#L302.

I actually ran migrations immediately afterwards, and haven't had any issues since then!

Great to hear! Let's close and please reopen if you have more issues. Or join https://t.me/lnbits to complain.


I updated the program and now keep saying that error MSB 3021 & MSCS 2046

[Enter feedback here]

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

ID: 30e1456c-8929-fed2-f265-ab7d437c86dd
Version Independent ID: 0eeb71c5-bbea-e7e0-bb1d-b08d9fe95022
Content: MSB3027: Could not copy 'source' to 'destination'. Exceeded retry count of 'number'. Failed. - MSBuild
Content Source: docs/msbuild/errors/msb3027.md
Product: visual-studio-windows
Technology: msbuild
GitHub Login: @ghogen
Microsoft Alias: ghogen

@Donnell1943 -- Donnell, thank you for your feedback.  This repo is solely for documentation issues, and this seems implementation-related.  Please consider these resources:

Visual Studio on Microsoft Q&A
Visual Studio MSDN Forum
Stack Overflow
Reddit

I am closing this issue now.  You are welcome to @ mention me for any followup.  We hope to hear from you again.


How to use Crater Deep and others?

Only 'rgHillyGround' has worked out for me by following the code in 'tgBoxGround'. But not so for CraterDeep and CraterShallow. If I want to make a custom obstacle ground, how do I go about doing that? Like in the pictures below:

I pulled them from google.

Hi chiku00,
When you say that the crater terrains haven't been working out for you, do you mean they do not compile? Or they do not appear as intended? Regardless, the green and blue objects seen on the ground in those simulations are tgBoxes and tgSpheres (see src/core). You can mimic the pictured terrains by following the examples in Brian's learning controller programs (he is now cc'd).
Best,
Steve

The boxes and craters are referred to as obstacles, not terrains. They are
added similarly to models, such in the example obstacles app:
tgBlockField* myObstacle = new tgBlockField();

// Add the model to the world

simulation.addModel(myObstacle);

On Mon, Sep 21, 2015 at 9:35 AM, Steve Lessard<EMAIL_ADDRESS>wrote:

Hi chiku00,
When you say that the crater terrains haven't been working out for you, do
you mean they do not compile? Or they do not appear as intended?
Regardless, the green and blue objects seen on the ground in those
simulations are tgBoxes and tgSpheres (see src/core). You can mimic the
pictured terrains by following the examples in Brian's learning controller
programs (he is now cc'd).
Best,
Steve
—
Reply to this email directly or view it on GitHub
https://github.com/NASA-Tensegrity-Robotics-Toolkit/NTRTsim/issues/175#issuecomment-142036593
.

Upon going through the master file of NTRTsim, I see many new things have been added that were not there in the one available in version 1.1 (available through the NASA website), particularly folder 'Models' under 'src'. I was having trouble installing the master version, so I had decided to stick with the one available at NASA.

Do you remember what those errors were? If you could provide them, along
with your operating system info, that'd be very helpful.
On Mon, Sep 21, 2015 at 9:56 PM, chiku00<EMAIL_ADDRESS>wrote:

Upon going through the master file of NTRTsim, I see many new things have
been added that were not there in the one available in version 1.1
(available through the NASA website), particularly folder 'Models' under
'src'. I was having trouble installing the master version, so I had decided
to stick with the one available at NASA.
—
Reply to this email directly or view it on GitHub
https://github.com/NASA-Tensegrity-Robotics-Toolkit/NTRTsim/issues/175#issuecomment-142181853
.


setting an array element with a sequence

File "<array_function internals>", line 200, in savez_compressed
File "D:\software\anaconda\Lib\site-packages\numpy\lib\npyio.py", line 686, in savez_compressed
_savez(file, args, kwds, True)
File "D:\software\anaconda\Lib\site-packages\numpy\lib\npyio.py", line 716, in _savez
val = np.asanyarray(val)
^^^^^^^^^^^^^^^^^^
ValueError: setting an array element with a sequence. The requested array has an inhomogeneous shape after 1 dimensions. The detected shape was (1880,) + inhomogeneous part.
Who also encounters this issue，Does anyone know how to solve it？Thx

Even i'm getting this error while trying to run the train.py file


Home Assistant Integration with the newest version not working!

Hello Dear Roleoroleo,
First of all Thank you very much for your fantastic firmware. I was able to use it as a Generic IP Cam in ONVIF.
I would like to use Home Assistant Yi Home Integration, but this is not working with this version.
Can you please maybe check my community thread about this, and see the informations I collected?
https://community.home-assistant.io/t/yi-home-1080p-integration/223742/2
Thank you very much,
Best regards,

Yi integration is made for old camera models.
Please check the wiki.

I see now okey, so that means from Home Assistant its not possible to use Microphone and Speaker modes like in Yi app.
Thank you.

It's not possible.

Thank you very much for the really fast response. Have a nice day. I close with comment. Best regards,


Add summary of code review anxiety paper

I think it would be beneficial to include a link to and summary of this preprint: https://doi.org/10.31234/osf.io/8k5a4
Might even be good to attempt to include some (optional) versions of the interventions they mention.  E.g. brainstorm thoughts and feelings around peer code review to increase self-awareness, practice cognitive restructuring (re-phrase negative thoughts in constructive and self-compassionate ways), and mention that exposure is important (if code review is scary, you should do it more often

While you're at it, maybe include a link to this: https://www.sortee.org/blog/2024/05/03/202404_code_club/


[iOS] the app is opened but no event get executed

I have used this plugin in the past for android and there it works well.
The problem I'm facing now is with iOS.
I have deployed the json file in my site here:
https://israelhiking.osm.org.il/.well-known/apple-app-site-association
I added the relevant config file paths:
https://github.com/IsraelHikingMap/Site/blob/217721012410e95bb9dc46df364cc4d96716baae/IsraelHiking.Web/config.xml#L15
I have registered the relevant events here:
https://github.com/IsraelHikingMap/Site/blob/217721012410e95bb9dc46df364cc4d96716baae/IsraelHiking.Web/sources/application/services/open-with.service.ts#L39
But I don't see any entry in the log file that indicate that the event is handled in the JS code.
Am I missing something?
The app is booting up when I click a link in the browser but no event is fired... not sure how to debug this... :-(

My bad, I had an if around the event registration that prevented the registration... Sorry...


Suspecting Memory Leak in portainer

We got a portainer instance to manage 15+ swarm clusters
On each cluster, we got an agent gloabally deployed
In a road to configure limits for my services, I just check how many RAM are consumed by portainer, and I see 3GB :scream:
When I look at prometheus graph, I see it grow constantly from 0 to 3 GB, in 9 days
The only specific configuration we have is to set snapshot to 1 min instead of 5
I just restarted portainer, and it starts at 8MB, I'll send here more info when I got them

Stats from 2 hours :

What are your Portainer, Docker and OS versions?

Portainer 1.19.1
Docker: 18.06.1-ce
OS: Container Linux by CoreOS 1855.4.0 (Rhyolite)

@WTFKr0 A memory leak related to snapshots was fixed in 1.19.2.
See: https://github.com/portainer/portainer/issues/2206

Ohh I missed that in my search !
It's certainly that, I'll upgrade soon
Thanx

It's seems to be OK in 1.19.2 thanx
We got another problem : https://github.com/portainer/portainer/pull/2137#issuecomment-426266794 :smile:


Formula select individual cells

Hello,
Can't use semicolon in formulas: SUM(A1;B3)
ORA-20000: [line 1, col 8] Unexpected token: ;

Hi,
It's explained in the README : Formula Support

Formulas must be entered in English locale, specifically:

Decimal separator = . (dot)
Argument separator, union operator, array row-item separator = , (comma)
Array row separator = ; (semicolon)

Ups sorry you're rigth.
Thanks Marc!


update kubernetes packages to 1.20

This updates k8s packages to 1.20.

add context as needed for https://github.com/dapr/components-contrib/blob/master/bindings/kubernetes/kubernetes.go ?


Adjectives antonyms switch

Transformation to switch adjective with their antonyms

Please pull "main" once.

This satisfies the review criteria and looks fine to me overall. You might want to add function docstrings in a subsequent PR  https://www.python.org/dev/peps/pep-0257/