Update to libswoc 1.3.10

1.3.10 has some minor fixes, the biggest one being support for memory alignment in MemArena and better STL compatibility for TextView.
This also changes how libswoc and libyaml are built. The previous mechanism was based on tying libyaml to luajit for no apparent reason (I think Theo made those changes). The new mechanism splits it out so that more than one optional library can be built. This can't be done using SUBDIRS because library building and header export are separately controlled - using SUBDIRS mean both or neither. I did test builds of all four options involving header export and all worked as expected.

This also changes how libswoc and libyaml are built.

Why do you think we should make that change on this PR? Does that require the new version of libswoc?
And why 10-Dev?

I'm closing this because due to requirements from other PRs I had to put additional features in libswoc. I'll split this in to two, one to update the build, and the other to do the library update.


Add support for zipkin 128bit TraceID

I've noticed that the zipkin transport has no mapping for TraceIDHigh, I require this to join up my traces from another system. I had a go at adding the functionality onto the branch  https://github.com/Rajczyk/jaeger-client-go, I'm not sure if I have done it correctly, but I referenced zipkin-go-opentracing like the top of the file indicated.
Thanks

Feel free to post a pull request.

i've created on here #281

Resolved by:

Add support for 128bit trace ids to zipkin thrift spans. (#378)
Update zipkin propagation logic to support 128bit traceIDs (#373)


[BEAM-10248] Sets the BQ location when looking up a BQ job

Please add a meaningful description for your change here

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

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

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

R: @pabloem

Thanks.

Thanks both!


make static builds work again without docker

local go install (the netgo suffix causes permission denied)
don't check for "statically linked", they aren't.
also strip photos binary

This change is

This is ready.

LGTM


RSDK-7245: reconnect with backoff on initial connection

This re-attempts initial connection when calling createRobotClient according to backoff parameters. If noReconnect is specified it does not re-attempt.

https://github.com/viamrobotics/viam-typescript-sdk/pull/291
https://github.com/viamrobotics/viam-typescript-sdk/pull/292

Change log

Reduce cognitive complexity of createRobotClient
Use exponential-backoff to attempt reconnects if not explicitly disabled with noReconnect
Clean up reconnect exponential-backoff usage

Testing
Not sure what the testing practices here are - let me know what would be best!

@purplenicole730 yup! I tested this locally using the examples/connect-app and tried starting and stopping my robot. It reconnected as expected!


Bad inference result on sample after overfitting on same sample

Hi @zhulf0804 ,
I wanted to ensure the model can memorise a single training example. To do this I set the __len__() method in the Dataset to return 1. When training I printed the data_dict to ensure that the same sample was used for each iteration. Since the dataset length was set to 1, each epoch consisted of a single training step.
I visualised the train curves in tensorboard and as expected, all three losses eventually decreased to 0. Then I wanted to visualise the prediction of the model. For this I used the test.py script. However, when running on the same sample from training (000000.bin) the model produces zero predictions.
If I set the score_thr in pointpillar.py to 0, then I get a lot of predictions but they are obviously all very low confidence.
Any idea where I am going wrong?

Hi @jonasdieker, It's strange. Could you post the visualized predictions when setting score_thr to 0 ?
By the way, did you load the pretrained weight successfully?

Hi thank you for your very fast reply!
Sorry maybe I should have made it clear that I wanted to train from scratch on a single kitti sample to see if I can get decent predictions overfitting. Therefore, no pretrained weights were loaded, instead I loaded the model weights which I saved from my overfit-training run, produced as described above.
The reason: I tried to do the same for NuScenes to test if the model can memorise the new data when overfitting. In this case the model also predicts nothing, however I am not able to get a zero loss after playing with the parameters. So there is likely more parameter adjusting I need to do still ...
Here is the visualisation you asked for. (Note: I am using a different visualisation function because your one did not work for me over ssh)

Here are the confidences:
[0.0112691  0.01061759 0.01054672 0.01012148 0.01011159 0.00997026
 0.00983873 0.00945836 0.00936741 0.00894571 0.00888245 0.00886574
 0.00883586 0.00870235 0.00864896 0.00861476 0.00859446 0.00854981
 0.00853697 0.00851393 0.00847296 0.00834575 0.00832187 0.00829636
 0.00829282 0.00826259 0.00825665 0.00825058 0.00824824 0.00824112
 0.00823086 0.00821262 0.00817523 0.00817244 0.00815322 0.00815221
 0.00809674 0.00809228 0.00809175 0.00807787 0.00805884 0.00801394
 0.00799607 0.00798928 0.00394109 0.00385207 0.00380854 0.00376242
 0.00368402 0.00364244]

And the class counts:
[44, 4, 2]

Hope this is somewhat helpful for you!

One more comment worth making: In the kitti dataloader I am actually commented out the data_augment function.
I did this in order to consistently get the same data for training. I only use point_range_filter even for split="train".

Hello @jonasdieker, did you also visualize the G.T. result and the predicted result used the weights provided by this repo on 000000.bin. Are they reasonable ?

Yes, I did and they were fine. That is why I am confused by my experiments outcome!

Ok. One more thing, could you help to verify the single training example is 000000.bin again ?

So I tried it again and verified I was overfitting on the same sample as I was testing on. I tried it with 000000.bin and then also 000001.bin individually, and both times the loss was practically zero but returned no bounding boxes at all with the test.py script and the default setting from defined here:
https://github.com/zhulf0804/PointPillars/blob/b9948e73505c8d6bfa631ffdf76c7148e82c5942/model/pointpillars.py#L262-L266
Could you try to repeat this experiment? It should only take a few minutes.

@zhulf0804 Ok, I think this is kind of interesting:
The only difference between train and val in train.py is the fact model.eval() is called (which of course you should be calling). But if I comment out that line I get the following plots:

Doing the same in test.py I get:

which is perfect! So, overfitting works exactly as expected with this change. However, I do not understand how this impacts the performance, as changing from train mode to eval mode does the following:

I think I need to give this some more thought. Let me know if you have an explanation!

Hello, @jonasdieker.
Both validation cls loss and visualized prediction (using test.py) become well by just removing model.eval(), like the following line ?
https://github.com/zhulf0804/PointPillars/blob/b9948e73505c8d6bfa631ffdf76c7148e82c5942/train.py#L139

Hello @zhulf0804, yes that is exactly right!

Ok, and I'm also confused about the result. I'll test it when I have access to the machine.
Besides, looking forward to your explanation to this question.
Best.

Do you have any updates on this? @jonasdieker did you find out the issue? I am getting the same problem, overfitting on one (or few samples) loss goes to 0, but then 0 predictions using test.py. And even worse, when I run test.py multiple times with NO changes, i get different results (sometimes few bboxes, most of the time zero - [] [] [])

Hi @mdane-x, as far as I remember overfitting on one (or a few) sample(s) didn't work. I ended up commenting out model.eval(). I believe the issue was due to the normalisation. If you have a good explanation of what is going on, please add it here!

Hi @jonasdieker, thanks for the answer. I haven't managed to make it work, even after removing the eval() line. I am getting empty predictions with any trained model (on few samples)


osinfo-db: 20210202 -> 20210215

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/osinfo-db/versions.
meta.description for osinfo-db is: "Osinfo database of information about operating systems for virtualization provisioning tools"
meta.homepage for osinfo-db is: "https://gitlab.com/libosinfo/osinfo-db/"
meta.changelog for osinfo-db is: ""
Updates performed

Version update

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 20210215 with grep in /nix/store/dbkcrswllrvjcrmylhb2jysi87qcsrhf-osinfo-db-20210215

directory tree listing: https://gist.github.com/6b42a8656942f93b8aa47871415d0cd9

du listing: https://gist.github.com/615a03f0e2ef3d4b856cbf93f58769c9

Rebuild report (if merged into master) (click to expand)

60 total rebuild path(s)

20 package rebuild(s)

20 x86_64-linux rebuild(s)
20 i686-linux rebuild(s)
1 x86_64-darwin rebuild(s)
19 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
dropbox-cli
gnome-photos
gnome3.file-roller
gnome3.gnome-books
gnome3.gnome-boxes
gnome3.gnome-control-center
gnome3.gnome-documents
gnome3.gnome-photos
gnome3.gnome-terminal
gnome3.gnome-user-share
gnome3.gnome_control_center
gnome3.gnome_terminal
gnome3.nautilus
gnome3.nautilus-python
gnome3.tracker-miners
libosinfo
osinfo-db
pantheon.extra-elementary-contracts
tracker-miners
virt-manager

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/dbkcrswllrvjcrmylhb2jysi87qcsrhf-osinfo-db-20210215 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A osinfo-db https://github.com/r-ryantm/nixpkgs/archive/57531324f91a785483e128c40e2975f7a2e96145.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/dbkcrswllrvjcrmylhb2jysi87qcsrhf-osinfo-db-20210215
ls -la /nix/store/dbkcrswllrvjcrmylhb2jysi87qcsrhf-osinfo-db-20210215/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

2 packages marked as broken and skipped:

gnome3.gnome-books
gnome3.gnome-documents

14 packages built:

dropbox-cli
gnome-photos
gnome3.file-roller
gnome3.gnome-boxes
gnome3.gnome-control-center
gnome3.gnome-terminal
gnome3.gnome-user-share
gnome3.nautilus
gnome3.nautilus-python
gnome3.tracker-miners
libosinfo
osinfo-db
pantheon.extra-elementary-contracts
virtmanager (virt-manager)

Maintainer pings
cc @bjornfor for testing.

Result of nixpkgs-review pr 113739 at 57531324 run on aarch64-linux 1

2 packages marked as broken and skipped:

gnome3.gnome-books
gnome3.gnome-documents

13 packages built:

gnome-photos (gnome3.gnome-photos)
gnome3.file-roller
gnome3.gnome-boxes
gnome3.gnome-control-center (gnome3.gnome_control_center)
gnome3.gnome-terminal (gnome3.gnome_terminal)
gnome3.gnome-user-share
gnome3.nautilus
gnome3.nautilus-python
tracker-miners (gnome3.tracker-miners)
libosinfo
osinfo-db
pantheon.extra-elementary-contracts
virt-manager

1 suggestion:

warning: missing-phase-hooks
installPhase should probably contain runHook preInstall and runHook postInstall.
Near pkgs/data/misc/osinfo-db/default.nix:16:3:
   |
16 |   installPhase = ''
   |   ^


Update: Microsoft.dotnet version 3.1.407.15660

Removing old versions to reduce clutter
Microsoft Reviewers: Open in CodeFlow

/azp run


Only copy pandas categorical attribute if it exists

In case:

a model was trained on numpy arrays (not pandas dataframe)
was serialized with model_to_string
was deserialized by instantiating a Booster with the model_str parameter (see here)
was queried with .predict on a numpy array again

Then the current master branch crashes with:
  File "/usr/local/lib/python2.7/site-packages/lightgbm/basic.py", line 2216, in _to_predictor
    predictor.pandas_categorical = self.pandas_categorical
AttributeError: 'Booster' object has no attribute 'pandas_categorical'

This PR fixes it.

@rostyboost Thanks a lot for your help!
However, the problem seems to be more fundamental: model_to_string and dump_model methods ignore pandas_categorical field at all.
Let me dig deeper...
I remember, there was the same problem with Dataset class more than year ago (#218), which @wxchan solved.
@rostyboost It will be very helpful if you can post a reproducible example. Thanks.

@rostyboost Please try #1766, it should solve your problem.

#1766 works. thanks


Release into Melodic

All of the dependencies for this package are available in Melodic.  Would you mind doing a release of this package into Melodic so that we can unblock some other packages?  Thanks in advance!

Thank you!


Hyper does not start

[x] I am on the latest Hyper.app version
[x] I have searched the issues of this repo and believe that this is not a duplicate

Windows 10 Family 1803:
2.0.0:
Link of a Gist with the contents of your .hyper.js:
N/A (CMD+ALT+I on macOS, CTRL+SHIFT+I elsewhere):
The issue is reproducible in vanilla Hyper.app:

Issue

Hello when i try to launch Hyper app the window is not showing and the process is killing directly. I have the last build of Windows Anniversary Update, before the update Hyper was working prefectly.
see:
https://imgur.com/a/cn9Kkf9

Might be helpful to look at the Windows system logs. Don't know how to do that...
@chabou if you'd like some help triaging I'd be happy to slap labels like Windows on issues every so often

I try to find problem in Windows system log but nothing..

Could you try building it from source? It's pretty easy if you clone the repo follow the readme instructions. Let me know if you get stuck. ´yarn run dist´ should mostly cover it

I already try to start Hyper with different path

Did you figure it out? Any lessons learned for someone else who might encounter thid?

it was just a mistake d'électron. After hard reset of the electron package, Hyper restart

Sorry if I pushed the closed button too fast

Good to close resolved issues! Still don't understand what went wrong tho.


Camera controls

Looks good!

Few small comments, other than that it looks fine to me :+1:

I'll add the todo comments, will be removed soon.

Looks good!


Could not find module & "unexclaim" sample issue

Hi everyone
I try to use swift-html in my project.
Currently I have two Issues:

On my Mac M1, when I try to build App I get following error:
"Could not find module 'Html' for target 'x86_64-apple-ios-simulator'; found: arm64, arm64-apple-ios-simulator, at:"

I try to use sample function unexclaim() from readme.
But I get this Issue:

Maybe someone has hint for me?
Bests from Swiss Mountains
Philippe

Hi @mbrandonw  Thank you for fast fix of function.
I reverted everything and reinstall again Package. Now everything is working well.
Bests Philippe


Additionally run skynet-js PR CI on skynetfree portal

PULL REQUEST
Overview
Originally requested here:
https://github.com/SkynetLabs/skynet-js/pull/456#pullrequestreview-958770707
Didn't add siasky.net for all runs yet as it's too unstable. I had to add it for PRs from forks though, as they can't access the required repo secrets.
Checklist

[ ] All git commits are signed. (REQUIRED)
[ ] All new methods or updated methods have clear docstrings.
[ ] Testing added or updated for new methods.
[ ] Verified if any changes impact the WebPortal Health Checks.
[ ] Appropriate documentation updated.
[ ] Changelog file created.

@kwypchlo Awesome, thanks!

@kwypchlo Testing on more portals was requested by a few people, to find potential issues faster. e.g. if we do all our testing on a single portal, then a bug in that portal can make bad SDK code seem fine. See provided link. cc @MSevey @ro-tex
I was initially against this because it will make PRs harder to merge (the portals are not really stable honestly), and we should already be testing other portals in the daily CI. But I saw the "Run failed jobs" (recent addition I think?) feature in GitHub Actions and figured we could now easily restart any jobs that failed due to portal issues.

@m-cat in that case I would refactor the github action to have matrix with portals [ siasky.net skynetfree.net skynetpro.net] and add the if condition for fork in there - if: !github.event.pull_request.head.repo.fork || matrix.portal == "siasky.net"). I think that will be simpler than copy & paste the job 3 times

@kwypchlo Done, although unfortunately I couldn't place the conditional on the job level. It works on the step but now CI reports that siasky.net checks succeeded which could be misleading. Not a huge deal IMO since we plan to eventually enable siasky.net (if it's ever stable enough).


Proposal to Integrate Mint Coin Functionality into the Wallet

Hi @JoshOrndorff
cc: @AltiMario
This is Amit from Mlabs.
I want to raise a proposal regarding integrating the "mint coin" functionality into the current wallet system.
While the existing Money piece supports Mint, the functionality is not currently available in the wallet.
Given the educational purpose of the current wallet, I believe it would be beneficial to include support for minting coins.
This addition could enhance the wallet's utility and provide users with a more comprehensive experience.
Here is a suggested implementation:
1. Minting Coins for Default Account (e.g., SHAWN):
Command: ./target/release/tuxedo-template-wallet mint-coins 200
This command would mint coins for the default account (SHAWN in this case) with the specified amount (200).
2. Minting Coins Based on the Public Key of the Owner:
Command:  ./target/release/tuxedo-template-wallet mint-coins-based-on-public-key-of-owner --amount 900 --owner publicKeyOfUser
This command would allow minting coins for a specific account identified by its public key, with the specified amount (900).
If we prefer not to expose the mint coin functionality to users directly, an alternative use case could be considered:
Considering the wallet's capability to manage multiple key pairs (supported through generate-key and insert-key functionalities), we could automatically provide a default number of coins to every user when a new key is inserted into the wallet.
For instance, 100 coins could be allocated as a default for each user, either upon key insertion or upon user request. These coins would serve testing purposes, such as purchasing virtual items like a "kitty."
I am eager to hear your thoughts on this proposal and whether you believe it aligns with our goals for the wallet's functionality and user experience.
Thank you for considering this suggestion. I look forward to discussing it further.
BR,
Amit
MLabs

I think it's wise to add minting functionality. Good idea.
I like a CLI similar to what you proposed. Lets add a single new mint-coins subcommand. It has an optional amount argument (default is 100 or something) and an optional receiver address (default is to pull one randomly from the wallet).

Thanks @JoshOrndorff, let me make change and create the pull request for the same,


Is it possible to add a camera weighing about 60g at the end?

Hi, thank you for this amazing project. I'm new to the field of robotics and came across your project through the lerobot project. I've already managed to run the robotic arm and perform teleop operations. Now, I'd like to try visual control based on this arm. I'm planning to purchase an Intel D405 depth camera, which weighs 60g. How can I calculate whether the end of this robotic arm can support such a payload? If the load is supportable, I might try to 3D print a new end component to mount the camera.

Hi legatoo, did you make it? And if so, can you release your work. Thanks :)

I think its generally pretty hard to calculate the end effector max load. But from experience I would say 60g is pretty ok.


move moment-js to a element for dedup ing

add a button version

This PR had quite a bit of extra cruft in it. I'll certainly consider moving moment.js into its own element.


conjoin and list-set before Racket 6.3

conjoin was in unstable/function and list-set in unstable/list until
Racket 6.3. Requiring those will be a harmless no-op for 6.3+, while
allowing <6.3 to work.

Well this didn't work as I'd expected.
I'll look at this more later when I have time.

You also need to add "unstable-list-lib" to the deps, right?

@AlexKnauth Yes, I believe that's why the 6.3 and HEAD jobs are failing. I'll push a commit to add that. There's also a doc build failure on the 6.2 job.

Looks like it's coming from https://github.com/lexi-lambda/racket-collections/blob/master/data/scribblings/collection/reference.scrbl#L229

That eval:alts/eval:check this is just borked. I tried fixing eval:check (and indeed, the change I made to it makes it "work" now), but the value proposition turned out to mostly be useless in practice. It would be really nice to fix eval:check to be more useful, but really, that should probably be removed.

Fixed in a15363bf6492f5df21051fd342a9dae79bfe29da.


change the self update url which pull the new version from wordpress plugins directory

for peoples which don't want to switch to the original version.

https://github.com/szepeviktor/fix-w3tc/issues/115

@nigrosimone should i add this into fix-w3tc? I was debating it because yea as @rabin-io mentions there are people who dont want to switch yet, meaning there are people that will want to switch when w3tc 0.9.5 does work out its several bugs.  So i was sitting on the fence whether to put in such a feature or not.
I could add a checkbox on the dashboard that allows the user to "Hide Update Notifications" but by default it is unchecked (meaning: when they install fix-w3tc it will show the new version notification message on the Plugins page, but by navigating to the fix-w3tc dashboard and checkmarking the "Hide Update Notification" it will no longer appear).
It's up to you, you're the lead on this project.  I am neutral on the matter since as you did with @rabin-io there is a solution made available right now in the threads.

@nigrosimone should i add this into fix-w3tc? [...] It's up to you, you're the lead on this project

thank you for considering me the "lead", but this is an open source project and it's not mine. You're free to do what you think is right for the project.
My 5 cents.

I could add a checkbox on the dashboard that allows the user to "Hide Update Notifications" [...]

I think this is the best choise for the users.
side note: @amiga-500 why you don't have asked frederik to enter in official W3TC repository? Your contribution to this project has been outstanding, you can be a great help in the official repository.

side note: @amiga-500 why you don't have asked frederik to enter in official W3TC repository? Your contribution to this project has been outstanding, you can be a great help in the official repository.

Perhaps we should talk about this on the slack channel that you created (https://fix-w3tc.slack.com/). @nigrosimone I sent you my email so you can send the invite. @amiga-500, it'd be great if you can join too by sending your email to @nigrosimone (nigro.simone@gmail.com).

@rabin-io just wanted to give you a heads up that i've added a "Hide Plugin Update Notification" checkbox under General Settings' Miscellaneous area (see pic below).  By default it is unchecked.  When you check it on (enable it), the Wordpress update notification message for w3tc will now go away.
I've merged it into to Master but haven't created a new release yet that incorporates it.  So you'll have to download Master for now.  Alternatively, you can just download the related Master files that provide it:
inc/define.php = download
inc/options/general.php  = download
lib/W3/ConfigKeys.php = download
lib/W3/Root.php = download
lib/W3/UI/Settings/General.php = download

side note: @amiga-500 why you don't have asked frederik to enter in official W3TC repository? Your contribution to this project has been outstanding, you can be a great help in the official repository.

Much appreciated.  Yea i am still debating this.  I probably will join you guys too.  It'll be nice to have familiar faces there :) .  I'm starting to wind down my support for fix-w3tc now that the official 0.9.5 is out but figured to resolve the remaining issues that were recently brought up.  Once done i'll decide.
Cheers
Kimberly

Thanks @amiga-500 , I saw the commit. much appreciated. thank you


Improve test playback

Reduces test recording size
Shaves off 25-30 seconds of playback runtime for tests
Update prompt for image generation to not accidentally trigger content filtering

API change check
API changes are not detected in this pull request.


Invalid python escape sequence in azure resource graph extension

Describe the bug
In python, spaces need no escaping, but I've heard that only newer versions of python complain if you do it anyway, i.e. use '\ ' in a string. _models_py3.py of the resource graph extension seems to contain such escape sequences, though: In line 58, 62, 1201, and 1205. Some az commands now print a warning because of this in my environment, although that hasn't happened in the past and I cannot reproduce it in all environments. I assume it depends on the python version used.
Related command
az graph
Errors
This is the output on my machine:

PS D:\git\scepman-psmodule\SCEPman> az graph query
time specifying the date and time for the inclusive start and exclusive end, i.e. [start, end).
C:\Users\ChristophHannebauer.azure\cliextensions\resource-graph\azext_resourcegraph\vendored_sdks\resourcegraph\models_models_py3.py:1196: SyntaxWarning: invalid escape sequence '\ '
"""Specifies the date and time interval for a changes request.
the following arguments are required: _subcommand
Examples from AI knowledge base:
https://aka.ms/cli_ref
Read more about the command in reference docs

Issue script & Debug output
PS D:\git\scepman-psmodule\SCEPman> az graph --debug
cli.knack.cli: Command arguments: ['graph', '--debug']
cli.knack.cli: __init__ debug log:
Enable color in terminal.
cli.knack.cli: Event: Cli.PreExecute []
cli.knack.cli: Event: CommandParser.OnGlobalArgumentsCreate [<function CLILogging.on_global_arguments at 0x00000150686CF420>, <function OutputProducer.on_global_arguments at 0x0000015068A6C180>, <function CLIQuery.on_global_arguments at 0x0000015068A91580>]
cli.knack.cli: Event: CommandInvoker.OnPreCommandTableCreate []
cli.azure.cli.core: Modules found from index for 'graph': ['azext_resourcegraph']
cli.azure.cli.core: Loading command modules:
cli.azure.cli.core: Name                  Load Time    Groups  Commands
cli.azure.cli.core: Total (0)                 0.000         0         0
cli.azure.cli.core: These extensions are not installed and will be skipped: ['azext_ai_examples', 'azext_next']
cli.azure.cli.core: Loading extensions:
cli.azure.cli.core: Name                  Load Time    Groups  Commands  Directory
cli.azure.cli.core: resource-graph            0.004         2         5  C:\Users\ChristophHannebauer\.azure\cliextensions\resource-graph
cli.azure.cli.core: Total (1)                 0.004         2         5
cli.azure.cli.core: Loaded 2 groups, 5 commands.
cli.azure.cli.core: Found a match in the command group table for 'graph'.
cli.knack.cli: Event: CommandInvoker.OnPreCommandTableTruncate [<function AzCliLogging.init_command_file_logging at 0x000001506AD51DA0>]
cli.azure.cli.core.azlogging: metadata file logging enabled - writing logs to 'C:\Users\ChristophHannebauer\.azure\commands\2024-12-13.14-11-57.graph.11056.log'.
az_command_data_logger: command args: graph --debug
cli.knack.cli: Event: CommandInvoker.OnPreArgumentLoad [<function register_global_subscription_argument.<locals>.add_subscription_parameter at 0x000001506ADB9800>]
cli.knack.cli: Event: CommandInvoker.OnPostArgumentLoad []
cli.knack.cli: Event: CommandInvoker.OnPostCommandTableCreate [<function register_ids_argument.<locals>.add_ids_arguments at 0x000001506ADB98A0>, <function register_cache_arguments.<locals>.add_cache_arguments at 0x000001506ADB99E0>, <function register_upcoming_breaking_change_info.<locals>.update_breaking_change_info at 0x000001506ADB9A80>]
cli.knack.cli: Event: CommandInvoker.OnCommandTableLoaded []
cli.knack.cli: Event: CommandInvoker.OnPreParseArgs []
C:\Users\ChristophHannebauer\.azure\cliextensions\resource-graph\azext_resourcegraph\vendored_sdks\resourcegraph\models\_models_py3.py:53: SyntaxWarning: invalid escape sequence '\ '
  """An interval in time specifying the date and time for the inclusive start and exclusive end, i.e. ``[start, end)``.
C:\Users\ChristophHannebauer\.azure\cliextensions\resource-graph\azext_resourcegraph\vendored_sdks\resourcegraph\models\_models_py3.py:1196: SyntaxWarning: invalid escape sequence '\ '
  """Specifies the date and time interval for a changes request.
urllib3.connectionpool: Starting new HTTPS connection (1): app.aladdin.microsoft.com:443
urllib3.connectionpool: https://app.aladdin.microsoft.com:443 "GET /api/v1.0/suggestions?query=%7B%22command%22%3A+%22graph%22%2C+%22parameters%22%3A+%22%22%7D&clientType=AzureCli&context=%7B%22versionNumber%22%3A+%222.67.0%22%2C+%22errorType%22%3A+%22MissingRequiredSubcommand%22%2C+%22correlationId%22%3A+%22cdc9d4af-640a-424f-8915-4105e2de1ba8%22%2C+%22subscriptionId%22%3A+%22627684de-d5d0-4a61-8fdb-fbcc37f8db7b%22%2C+%22eventId%22%3A+%22e5291b07-b59d-4196-a5cd-9ca9ef7cee63%22%7D HTTP/1.1" 200 None
cli.azure.cli.core.azclierror: Traceback (most recent call last):
  File "argparse.py", line 1931, in parse_known_args
  File "argparse.py", line 2196, in _parse_known_args
argparse.ArgumentError: the following arguments are required: _subcommand

cli.azure.cli.core.azclierror: the following arguments are required: _subcommand
az_command_data_logger: the following arguments are required: _subcommand

Examples from AI knowledge base:
https://aka.ms/cli_ref
Read more about the command in reference docs
cli.knack.cli: Event: Cli.PostExecute [<function AzCliLogging.deinit_cmd_metadata_logging at 0x000001506AD52020>]
az_command_data_logger: exit code: 2
cli.__main__: Command ran in 0.929 seconds (init: 0.284, invoke: 0.644)
telemetry.main: Begin splitting cli events and extra events, total events: 1
telemetry.client: Accumulated 0 events. Flush the clients.
telemetry.main: Finish splitting cli events and extra events, cli events: 1
telemetry.save: Save telemetry record of length 3824 in cache file under C:\Users\ChristophHannebauer\.azure\telemetry\20241213141157786
telemetry.main: Begin creating telemetry upload process.
telemetry.process: Creating upload process: "C:\Program Files\Microsoft SDKs\Azure\CLI2\python.exe C:\Program Files\Microsoft SDKs\Azure\CLI2\Lib\site-packages\azure\cli\telemetry\__init__.pyc C:\Users\ChristophHannebauer\.azure C:\Users\ChristophHannebauer\.azure\telemetry\20241213141157786"
telemetry.process: Return from creating process 43816
telemetry.main: Finish creating telemetry upload process.

Expected behavior
There should be no warning about the invalid escape sequence.
Environment Summary
azure-cli                         2.67.0
core                              2.67.0
telemetry                          1.1.0
Extensions:
resource-graph                     2.1.0
Dependencies:
msal                              1.31.0
azure-mgmt-resource               23.1.1
Python location 'C:\Program Files\Microsoft SDKs\Azure\CLI2\python.exe'
Extensions directory 'C:\Users\ChristophHannebauer.azure\cliextensions'
Python (Windows) 3.12.7 (tags/v3.12.7:0b05ead, Oct  1 2024, 03:06:41) [MSC v.1941 64 bit (AMD64)]
Legal docs and information: aka.ms/AzureCliLegal
Your CLI is up-to-date.
Additional context
The code is auto-generated by autorest, so probably, there is an upstream issue in autorest

Thank you for opening this issue, we will look into it.


Update aliwangwang to 7.06.00-8301

After making all changes to the cask:

[x] brew cask audit --download {{cask_file}} is error-free.
[x] brew cask style --fix {{cask_file}} left no offenses.
[x] The commit message includes the cask’s name and version.

Builds verified.


Minified files doesn't change images relative path

As the title, the minified files still use the relative paths of the original stylesheets, but the minified output will be in a subfolder, the reltive paths must be rewrited.

Hi @fede91it. Please read #23 or #38 to learn how you can use custom filters to rewrite the paths.


CVE services login/logout can be be improved

Since the current CVE services only offer a token based authentication, the https://github.com/xdrr/cve.js/ library uses

the PasswordCredential API available in Chrome to store and retrieve the there params needed for CVE services (users, shortName, token) https://whatwebcando.today/credentials.html
an passphrase based encryption to securely store the credentials in localStorage on browsers (Safari and FireFox) that do not implement  PasswordCredential.

Improvements needed

It currently uses the multiple browser prompts to collect these params. Instead use a single html dialog/form to collect these.
properly implement logout and timeout. Call navigator.credentials.preventSilentAccess()
The switch between production and test instances of CVE services need to be removed - but should be a configurable option i.e., build a separate Vulnogram standalone/serverless site for production vs test.
Make use of PasswordCredential iconURL
refactor the UI of the CVE Portal to provide better embedded login -> view or reserve IDs, or update records --> logout or timeout experience.

The login/logout workflow should be something like:

Fixed in cve-session branch.


[cinder-csi-plugin] Fix filesystem resize (#1434)

Now we use "findmnt" command to find the device path. Unfortunately,
this command may return multiple entries of the same mount, but in
fact we need just the first one.
To prevent this issue we add "--first-only" flag to the command.

What this PR does / why we need it:
cherry-pick: aa5e9fe41c38291cad2c888b81b9c5c524b1c209
Which issue this PR fixes(if applicable):
fixes #1436
Special notes for reviewers:

Release note:

NONE

/approve
good backport

/lgtm


Format tokens are incoherent with builtin .Net tokens

Hi,
Some tokens for the format does not match the .Net specifications.
https://docs.microsoft.com/en-us/dotnet/standard/base-types/custom-date-and-time-format-strings
For example, MM means month when using DateTime.ToString(), but minutes in your package
For mm, this is the inverse.

I am using the below library for the final display of the time, while it does not adhere to the .net spec I would rather not modify the library to maintain compatibility should it need to be updated at any time.
https://github.com/felixge/node-dateformat


Alert/Confirmation popup message DSLs does not wait for a timeout while asserting / perfoming some operations

Following DSLs immediately fails in case of popup message not present on UI
I <accept|dismiss> the <alert|confirmation> popup
the <alert|confirmation> popup message <should|should not> <be|contain|start with|end with|match regex|match xpath|match template|match template file> "<expression>"
the <alert|confirmation> popup message <should|should not> <be|contain|start with|end with|match regex|match xpath> <attribute>
I capture the <alert|confirmation> popup message

Due to which we need to add a hard wait to allow the poup to appear before performing some operation or asserting them. Otherwise it immediately fails.
Is there any mechanism in GWEN to remove this dependency on hard-wait?
Env Details
Gwen Version - 2.53
Chrome Driver - Latest (92)

Verified on latest release - gwen 2.53.3


VHACD.GenerateConvexMeshes() never calls DestroyVHACD()

The vhacd instance created in VHACD.GenerateConvexMeshes():
// VHACD.cs:138
var vhacd = CreateVHACD();

Should probably be destroyed before returning from that method to avoid a memory leak, e.g.:
// VHACD.cs:185
DestroyVHACD(vhacd); // proposed change
return convexMesh;

Sorry for this late response. Thank you for bringing it up. I have created an internal ticket AIRO-1486 and we will follow up on this issue.

Hi @garettbass, this has been added and resolved in #7, which is merged into the dev branch. I will be closing this issue, but feel free to reopen to continue discussion, or open a new issue if a new topic comes up. Thank you!


Seed mode: 2047-bit signing sub-key length when generating from 2048-bit master key

I just found out  Ledger generates 2047-bit RSA signing key and encryption key when seed mode is active.
sec   rsa2048/XXXXXXXXXXXXX  created: 2020-11-07  expires: never    --> master key
ssb>  rsa2047/XXXXXXXXXXXXX  created: 2020-11-07 expires: never     --> enc. key
                                card-no: 2C97 XXXXXXXX
ssb>  rsa2048/XXXXXXXXXXXXX  created: 2020-11-07 expires: never     --> auth. key
                                card-no: 2C97 XXXXXXXX
ssb>  rsa2047/XXXXXXXXXXXXX  created: 2020-11-07  expires: never    --> sign. key
                                card-no: 2C97 XXXXXXXX

When seed mode is off, the keys generated are of 2048-bit length.
This is probably a bug since it only happens with signature and encryption key.
Authentication key is generated as 2048-bit key just fine.
Not the first time, it appears:
https://www.chiark.greenend.org.uk/~sgtatham/putty/wishlist/puttygen-keys-one-bit-short.html
https://randomoracle.wordpress.com/2019/12/04/off-by-one-the-curious-case-of-2047-bit-rsa-keys/
The issue is a lot of tools happily mark RSA 2047 bit as insecure. :( So using Ledger's seed mode becomes impractical.
Paging @cslashm  :)

Hi,
Doing more tests these days, also after the app refactoring on v2, the generated key is 2048 bits.
pub  rsa2048/29B9FFAA721E6714 2024-02-29 testkey
sec>  rsa2048/29B9FFAA721E6714  created: 2024-02-29  expires: never
                                card-no: 2C97 955936E6
ssb>  rsa2048/4612132E5E57BE78  created: 2024-02-29  expires: never
                                card-no: 2C97 955936E6
ssb>  rsa2048/6B18E9BCB1E7189B  created: 2024-02-29  expires: never
                                card-no: 2C97 955936E6

Please double check...


Add top level keywords completion

This fixes the following issue: https://github.com/rust-analyzer/rust-analyzer/issues/4566.
Also added simple logic which filters the keywords which can be used with unsafe on the top level.

For the reference, here's how Kotlin handles keyword completion: https://github.com/JetBrains/kotlin/blob/f128e5222ae69635352a3cf31af772f35b04e333/idea/idea-completion/src/org/jetbrains/kotlin/idea/completion/KeywordCompletion.kt

For the reference, here's how Kotlin handles keyword completion: https://github.com/JetBrains/kotlin/blob/f128e5222ae69635352a3cf31af772f35b04e333/idea/idea-completion/src/org/jetbrains/kotlin/idea/completion/KeywordCompletion.kt

Do you think it makes sense to try to replicate the handling mechanism in rust-analyzer (to the possible extent of course)?

Do you think it makes sense to try to replicate the handling mechanism in rust-analyzer (to the possible extent of course)?

Not really. I do think we need a more principled approach for keyword completion. Basically, we need to ensure that all possible keywords are completed in the correct contexts, which I think can be achieved by one of two ways:

explicitly special casing every position
using the Kotlin's approach of "try insert the keyword and see if the parser is ok with it"

I am not sure which of the two approaches is the best, and I don't think there's an easy answer here. Seems like we just need to move slowly and try different things out, building a test suite along the way.

Here's an interesting diff for Kotlin which swtiched from hard-coded cases to a parser-based approach: https://github.com/JetBrains/kotlin/commit/549171d04365a293e53829803c5e8d96c1ef7f44

Here's an interesting diff for Kotlin which swtiched from hard-coded cases to a parser-based approach: JetBrains/kotlin@549171d

Do you think it makes sense to try to replicate the handling mechanism in rust-analyzer (to the possible extent of course)?

Not really. I do think we need a more principled approach for keyword completion. Basically, we need to ensure that all possible keywords are completed in the correct contexts, which I think can be achieved by one of two ways:
* explicitly special casing every position

* using the Kotlin's approach of "try insert the keyword and see if the parser is ok with it"

I am not sure which of the two approaches is the best, and I don't think there's an easy answer here. Seems like we just need to move slowly and try different things out, building a test suite along the way.

As far as I understand there is one difficulty with Kotlin's approach in our case. We don't have all errors in the syntax tree (seems Kotlin has them - https://github.com/JetBrains/kotlin/blob/ba6da7c40a6cc502508faf6e04fa105b96bc7777/compiler/testData/psi/annotation/ShortAnnotations.txt) and sometimes we have them in the situation when we still can complete the code to be correct. Here is one of the examples:
unsafe

->
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>"unsafe"
<EMAIL_ADDRESS>" \n"

Here we can definitely use unsafe and we should not filter it based on errors. I also don't know what other parsing logic can we use which can help us distinguish between completable code(prefix) and not-completable code.
So probably special casing approach would be more feasible here, but maybe I miss something.

I think Kotlin also deals with error elements problem somehow:

But yeah, I agree that we should start with special casing.

Another easy case is let

Another easy case is let

Actually what I want to do is to maybe try to use the kind of approach that Kotlin used before they changed to parsing errors. That is to match each keyword with certain filters that should be applied to check if we can use it. These filters will be defined as handlers. To check if that works I will today/tomorrow look at the grammar and see if this abstraction plays good with Rust's syntax.

More generally, I think there's a library solution here as well: describing positions in source code as patterns. At least, it's obvious that we can have
trait TreePattern {
  fn matches_node(&self, node: SyntaxNode) -> bool;
  fn matches_position(&self, root: SyntaxNode, position: Textsize) -> bool
}

We can also add all sorts of algebraic operations on top, like & and |.
However, I haven't actually used such API a lot, somehow just writing the cases manually was always easier.

More generally, I think there's a library solution here as well: describing positions in source code as patterns. At least, it's obvious that we can have
trait TreePattern {
  fn matches_node(&self, node: SyntaxNode) -> bool;
  fn matches_position(&self, root: SyntaxNode, position: Textsize) -> bool
}

We can also add all sorts of algebraic operations on top, like & and |.
However, I haven't actually used such API a lot, somehow just writing the cases manually was always easier.

Aha makes sense, just to clarify, so the patterns here would be essentially like filters and we can combine them etc. Then we would have to define patterns like InFunctionDefPattern, InImplPattern, InTraitPattern, InLoopPattern, AfterUnsafePattern etc. And then match them with each of the keywords. Is that the idea? If I understood correctly I will try to go forward with this approach.

Is that the idea?

Yup.

Is that the idea?

Yup.

Hi @matklad ! I started working with traits approach, but then I understood that we would anyway want to compute each of the filters only once with only one position and if we would combine the traits we would have to either do the computation multiple times or make them refer to some cache underneath. So we would probably not need to combine the traits, but we would need to combine the results.
I still think it makes sense to separate the patterns logic into some other file, so it would be less cluttered and maybe add additional namespace in the context, but I am not really sure. Because otherwise the fill method is getting really big and unreadable IMO.
Can you please check if this approach makes sense (also the way I am combining the results to determine if the keyword can be applied)? Please note that I don't filter them by prefix, because it seems that the client does it on its own.
Also in computing the respective patterns we would want to use fake token, because it gives the correct view on how the AST will look like (just clarifying, I didn't do this the first time)
If that is fine I will make the tests and cover the outstanding cases.

Yup, I think this setup makes sense!
I am worrying a bit that CompletionContext is becomming big, but we can solve it like this in the future:
// completion_context.rs
struct CompletionContext {
  kw_context: KwContext
}

// complete_keyword.rs
pub(crate) fn build_context(...) -> KwContext {}
pub(crate) fn complete(ctx: CompletionContext)

Ie, we can preserve the current control flow, but make it such that each completion can, in a single file, provide both a bit of context and the actual code completion logic.
I am not sure what's the best stup is though, this seems to be a bit of an overkill. My gut feeling that just cleaning up the current structure would work -- I won'dn't worry about 2koc fill_completion_context function, it seems like it could be implemented in a relatively clean manner.
But I also like how you move patterns to a separate file, maybe we should do that with some other patterns as well!

@matklad, just to confirm, so in this PR we are generating a list of completion labels with their types as a first letter and then matching whole list (a string) using insta comparison against snapshot? Thanks!

Yup!

Yup!

@matklad I implemented most of the keywords (I think like the main ones) and added test covering I think most basic situations. There is still room for a lot of improvements, for example, maybe variable names should be shorter and more concise, maybe patterns should be grouped for readability and all keywords sorted (or grouped) etc. Should we split the work into more PRs or issues?

I think we should land this, as it is getting pretty big :)
I've pushed some cleanups on top
bors r

bors r+


css: give logo a white background

The logo is placed on a color gradient. And the colors in the gradient are the same as the colors in the logo. This makes it very hard to recognize the parts of the logo.
Add a white background to the log to make the logo elements visible in the titlebar.
Also add a bit more space between the logo and the text.
Image of the result:

LGTM! I'll let @csimmonds approve #4 first then we can merge this on top.


ARROW-11561: [Rust][DataFusion] Add Send + Sync to MemTable::load

This PR adds Send + Sync to the MemTable::load method to allow implementation of a persist method like Spark's Dataframe in an async function.

Codecov Report

Merging #9448 (40e3b42) into master (a321cde) will decrease coverage by 0.05%.
The diff coverage is 66.52%.

@@            Coverage Diff             @@
##           master    #9448      +/-   ##
==========================================
- Coverage   82.14%   82.09%   -0.06%
==========================================
  Files         232      233       +1
  Lines       54150    54371     +221
==========================================
+ Hits        44484    44637     +153
- Misses       9666     9734      +68

Impacted Files
Coverage Δ

rust/arrow/src/datatypes.rs
78.33% <ø> (ø)

rust/benchmarks/src/bin/tpch.rs
12.15% <0.00%> (+0.06%)
:arrow_up:

rust/datafusion/src/datasource/memory.rs
80.00% <ø> (ø)

...tafusion/src/physical_plan/distinct_expressions.rs
90.29% <0.00%> (-0.89%)
:arrow_down:

...atafusion/src/physical_plan/expressions/average.rs
82.30% <0.00%> (-1.49%)
:arrow_down:

.../datafusion/src/physical_plan/expressions/count.rs
85.86% <0.00%> (-1.91%)
:arrow_down:

...atafusion/src/physical_plan/expressions/min_max.rs
91.54% <0.00%> (-1.76%)
:arrow_down:

...st/datafusion/src/physical_plan/expressions/sum.rs
77.27% <0.00%> (-1.02%)
:arrow_down:

rust/datafusion/src/physical_plan/mod.rs
86.00% <ø> (ø)

rust/datafusion/src/physical_plan/udaf.rs
75.00% <0.00%> (-3.95%)
:arrow_down:

... and 13 more

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 1c219e3...40e3b42. Read the comment docs.


⚠ Bofrost.de has degraded performance

In c202e89, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1897 ms

Resolved: Bofrost.de performance has improved in e5e793d after 6 minutes.


Apriori algo error

Error on: itemsets, rules = mf.frequency_item_set(columns = ['Ozone','Temp','Wind'], support=0.0)
Do I need to install pymining etc?
ZeroDivisionError                         Traceback (most recent call last)
 in ()
----> 1 itemsets, rules = mf.frequency_item_set(columns = ['Ozone','Temp','Wind'], support=0)
/home/deshana/Desktop/BadFish/badfish/badfish/badfish.pyc in frequency_item_set(self, columns, support, rules, confidence, engine)
371
372         # For now the same supports being used in FIM and Association Rules.
--> 373         rules = assocrules.mine_assoc_rules(item_sets, min_support=support, min_confidence=confidence)
374
375         rules_df = pd.DataFrame(rules, columns = ['X =>', 'Y', 'Support', 'Confidence'])
/usr/local/lib/python2.7/dist-packages/pymining/assocrules.pyc in mine_assoc_rules(isets, min_support, min_confidence)
12             _mine_assoc_rules(
13                 left, right, support, visited, isets,
---> 14                 min_support, min_confidence, rules)
15
16     return rules
/usr/local/lib/python2.7/dist-packages/pymining/assocrules.pyc in _mine_assoc_rules(left, right, rule_support, visited, isets, min_support, min_confidence, rules)
26
27     support_a = isets[left]
---> 28     confidence = float(rule_support) / float(support_a)
29     if confidence >= min_confidence:
30         rules.append((left, right, rule_support, confidence))
ZeroDivisionError: float division by zero

It appears you do have pymining.
This is the airquality dataset? I will try to reproduce the error at my end.

itemsets, rules = mf.frequency_item_set(columns = ['Ozone','Temp','Wind'], support=0)

Support can't be set as zero as it breaks the following line in pymining
confidence = float(rule_support) / float(support_a)

Good catch - should make sure the user's input is >0 and <=1

Resolved.
Printed a warning and made the method return.


Casting Issue with OpenApiAnyFactory

In master right now there is an issue with the OpenApiAnyFactory specifically the private static TryCast method.
When trying to cast a decimal to double the TryCast method is failing. I created a simple example of the issue below.
void Main()
{
    var foo = 4.3m;
    var bar = TryCast<double>(foo, out var doubles);
    // bar is null
}

private static bool TryCast<T>(object value, out T typedValue)
{
    try
    {
        typedValue = (T)value;
        return true;
    }
    catch (InvalidCastException ex)
    {
        typedValue = default(T);
        return false;
    }
}

I believe the root issue is the object type casting that happens in the method signature. While (double)4.3m is fine, what I think is being executed is  (double)(object)4.3m which throws an exception.

Thanks for digging into this. Seems like a fix maybe fairly straightforward. Interested in submitting a PR?


Storage Account blob container name parametrization

Right now, the storage blob container name is derived from the aks cluster's api server address/fqdn. So, in a way, it depends on the pattern for API server address of aks clusters.
Is there a possibility to parametrize it, defaulting it to what it does right now, and add a parameter in yaml file to take the blob storage container name?

Adding @sophsoph321 for visibility.

💡 Thank you for this. Linking this issue with - more detailed issue here: https://github.com/Azure/aks-periscope/issues/92 , Thanks heaps!

🙏 Thanks all for discussion, this is fixed as part of this recent release. https://github.com/Azure/aks-periscope/releases/tag/v0.5 , closing this issue.


[feature]: Add images to the game

What does this PR do?
This PR makes it so instead of just letters, it shows 16 cards from a collection of  images, some styling was modified as well to have a better presentation of such images. This new functionality can be extended by creating different themes for the game, adding replay value.
The images are smash bros related, because it was the first thing I thought of.
Related PRs and Issues
This solves issue #75
Have you updated the README.md file?

[ ] yes
[ ] no

Have you made any big breakthrough change?

[ ] yes
[ ] no

Is the code beautified?

[ ] yes
[ ] no

Is the code clear to understand with proper variable naming and comments?

[ ] yes
[ ] no

Thanks for the work!
Please fill the pull request template with appropriate checklist and screenshots so I can approve and merge it

@Aatmaj-Zephyr are those screenshots enough? Should I add anything else?

Thanks


Configure script automatically selects CUDA/cuDNN path instead of waiting for user input

Click to expand!
Issue Type
Bug
Have you reproduced the bug with TF nightly?
No
Source
source
Tensorflow Version
TF 2.10
Custom Code
No
OS Platform and Distribution
Fedora 37
Mobile device
No response
Python version
3.10
Bazel version
5.3.0
GCC/Compiler version
12.3.1
CUDA/cuDNN version
11.8,12.1/8.0
GPU model and memory
GTX 1660 Ti, 6 GB
Current Behaviour?
I am having multiple CUDA versions, and I am trying to build Tensorflow from source with CUDA support.
Now the problem lays when I try to configure the build system using ./configure. It will asks for relevant information for the build system. This includes:

Python path
Python packages path
Whether to support mROC
Whether to support CUDA
Whether to support TensorRT

Now, when I select CUDA support. the script seems to automatically selects my CUDA/cuDNN versions, and does not give me the possibility to select it manually, which is contradictory to what the documentation suggests at  https://www.tensorflow.org/install/source#gpu_support:  "If your system has multiple versions of CUDA or cuDNN installed, explicitly set the version instead of relying on the default"
Now, I was able to trace the issue exactly to the configure.py file.
In fact, I strongly suspects that there is a logical error on the section that parses the user input (Line 1244 on branch r2.11):
  environ_save = dict(environ_cp)
  for _ in range(_DEFAULT_PROMPT_ASK_ATTEMPTS):
    if validate_cuda_config(environ_cp):
      cuda_env_names = [
          'TF_CUDA_VERSION',
          'TF_CUBLAS_VERSION',
          'TF_CUDNN_VERSION',
          'TF_TENSORRT_VERSION',
          'TF_NCCL_VERSION',
          'TF_CUDA_PATHS',
          # Items below are for backwards compatibility when not using
          # TF_CUDA_PATHS.
          'CUDA_TOOLKIT_PATH',
          'CUDNN_INSTALL_PATH',
          'NCCL_INSTALL_PATH',
          'NCCL_HDR_PATH',
          'TENSORRT_INSTALL_PATH'
      ]
      # Note: set_action_env_var above already writes to bazelrc.
      for name in cuda_env_names:
        if name in environ_cp:
          write_action_env_to_bazelrc(name, environ_cp[name])
      break

    # Restore settings changed below if CUDA config could not be validated.
    environ_cp = dict(environ_save)

    set_tf_cuda_version(environ_cp)
    set_tf_cudnn_version(environ_cp)
    if is_windows():
      set_tf_tensorrt_version(environ_cp)
    if is_linux():
      set_tf_tensorrt_version(environ_cp)
      set_tf_nccl_version(environ_cp)

    set_tf_cuda_paths(environ_cp)

Now, from my understanding, the script will validate the given environment, and then if that fails will ask for user input.
With that, on the first iteration of the loop, the validation will not contain the required environment variables.
I was able to solve the issue by swapping the order as follow:
    environ_save = dict(environ_cp)
    for _ in range(_DEFAULT_PROMPT_ASK_ATTEMPTS):
      # Restore settings changed below if CUDA config could not be validated.
      environ_cp = dict(environ_save)

      set_tf_cuda_version(environ_cp)
      set_tf_cudnn_version(environ_cp)
      if is_windows():
        set_tf_tensorrt_version(environ_cp)
      if is_linux():
        set_tf_tensorrt_version(environ_cp)
        set_tf_nccl_version(environ_cp)

      set_tf_cuda_paths(environ_cp)
      if validate_cuda_config(environ_cp):
        cuda_env_names = [
            'TF_CUDA_VERSION',
            'TF_CUBLAS_VERSION',
            'TF_CUDNN_VERSION',
            'TF_TENSORRT_VERSION',
            'TF_NCCL_VERSION',
            'TF_CUDA_PATHS',
            # Items below are for backwards compatibility when not using
            # TF_CUDA_PATHS.
            'CUDA_TOOLKIT_PATH',
            'CUDNN_INSTALL_PATH',
            'NCCL_INSTALL_PATH',
            'NCCL_HDR_PATH',
            'TENSORRT_INSTALL_PATH'
        ]
        # Note: set_action_env_var above already writes to bazelrc.
        for name in cuda_env_names:
          if name in environ_cp:
            write_action_env_to_bazelrc(name, environ_cp[name])
        break

Standalone code to reproduce the issue
Assumption: Multiple CUDA versions on /usr/local

Command:
./configure

Input Example:
1. [Default Setting]
2. [Default Setting]
3. N
4. y
5. N

Relevant log output
No response

Hi @ramizouari ,
Tensorflow preconfigures paths of the CUDA and cuDNN toolkits which are installed as per Official instructions in documentation using Conda.If the script able to detect the path automatically then it won't ask the user to mention the paths.If the path not able to detectable by script then it will prompt the users to mention the path.Please refer the below example for same.
(tf) suryanarayanay@surya-ubuntu22-cuda-test:~/tensorflow$ ./configure
bash: /home/suryanarayanay/miniconda3/envs/tf/lib/libtinfo.so.6: no version information available (required by bash)
You have bazel 5.3.0 installed.
Please specify the location of python. [Default is /home/suryanarayanay/miniconda3/envs/tf/bin/python3]:

Found possible Python library paths:
  /home/suryanarayanay/miniconda3/envs/tf/lib/python3.9/site-packages
Please input the desired Python library path to use.  Default is [/home/suryanarayanay/miniconda3/envs/tf/lib/python3.9/site-packages]

Do you wish to build TensorFlow with ROCm support? [y/N]: n
No ROCm support will be enabled for TensorFlow.

Do you wish to build TensorFlow with CUDA support? [y/N]: y
CUDA support will be enabled for TensorFlow.

Do you wish to build TensorFlow with TensorRT support? [y/N]: n
No TensorRT support will be enabled for TensorFlow.

Could not find any cuda.h matching version '' in any subdirectory:
        ''
        'include'
        'include/cuda'
        'include/*-linux-gnu'
        'extras/CUPTI/include'
        'include/cuda/CUPTI'
        'local/cuda/extras/CUPTI/include'
of:
        '/lib'
        '/lib/x86_64-linux-gnu'
        '/usr'
        '/usr/lib/x86_64-linux-gnu/libfakeroot'

Asking for detailed CUDA configuration...

Please specify the CUDA SDK version you want to use. [Leave empty to default to CUDA 11]: 11.8

Please specify the cuDNN version you want to use. [Leave empty to default to cuDNN 2]: 8.6

Please specify the locally installed NCCL version you want to use. [Leave empty to use http://github.com/nvidia/nccl]:

Please specify the comma-separated list of base paths to look for CUDA libraries and headers. [Leave empty to use the default]:

So if the script is able to identify the path then tensorflow only facilitating the users right. However if you want to keep the cuda and cudnn libraries at a particular directory or want to use particular version of cuda/cudnn you can done this by removing cuda/cuDNN from standard download path and then the script will ask to enter the cuda path as seen in above example.
I would like to know how you installed the cuda/cuDNN and how the path has been set. Also please confirm whether the auto detection is causing any particular problem for your case. Please elaborate.
Thanks!

Hi @SuryanarayanaY ,
First of all, thank you for your help.
I installed both cuDNN and CUDA via Nvidia's RPM package. And so it is updated via the package manager.
The installation is on the standard path /usr/local/cuda.
Now to be more precise, for any update with version xx.y of CUDA. the package manager will:

install the update on /usr/local/cuda-xx.y folder
set /usr/local/cuda-x and /usr/local/cuda as a symbolic to  /usr/local/cuda-xx.y

With this, I effectively have many CUDA versions installed on the path /usr/local/cuda-xx.y, with the latest version acting as the default one.
The path is set on login. In fact, my ~/.bashrc file contain these two lines:
export PATH="/usr/local/cuda/bin:$PATH"
export LD_LIBRARY_PATH="/usr/local/cuda/lib64:$LD_LIBRARY_PATH"

However if you want to keep the cuda and cudnn libraries at a particular directory or want to use particular version of cuda/cudnn you can done this by removing cuda/cuDNN from standard download path and then the script will ask to enter the cuda path as seen in above example.

I am going to slightly disagree on this.
This should be the logical behaviour when there is exactly one installation (modulo some symbolic links).
But in my case, I have many different installations, and it will be better if the script asks for what version I expect.
Also, the documentation itself hints that the script should do such behaviour upon detecting many CUDA versions, which is not what is happening.

Hi @ramizouari ,
The script for ./configure can be found here.
If you are interested then go through the source code and analyse the behaviour and may let us know if you have any pointers for this behaviour.
Thanks!

@nitins17 - Please share your pointers on this issue.
CC - @learning-to-play


Photo taken by camera doesn't save in my Library

I downloaded the example , it worked well but when i import Pickle in my project, i take a photo (with default camera) and my library is still the same :( .
What did i forgot ??

Sorry i just try on a new project and it worked well.

I'm sorry but i don't understand why it doesn't saving in my library, i checked all my project i dont know where it is bad :(


Cursorplugin time parameter in formatTimeCallback is different from the wave surfer time

Hi there,
Not sure what's going on, but it seems that the time parameter is different from the one in audio process it's off by around 2 seconds
CursorPlugin.create({
    formatTimeCallback: function (time) {
        console.log(time); // returns 20.564427818666665
    },
}),

this.waveSurfer.on('audioprocess', function (time) {
     console.log(time); // returns 18.310791
});

Any idea what might cause this difference?
My values from the wave surfer instance if that is of any help:
pixelRatio: 10,
pxPrSec: 1,
cursorWidth: 1,
barWidth: 2,
barHeight: 2,
barGap: 2,

Thank you!

@Reached can you run ffprobe (part of ffmpeg) on the audio file and paste output here? It should be able to tell it's exact duration.

Hi @thijstriemstra , thanks for getting back to me!
The output from ffprobe is below:
Input #0, wav, from 'song.wav':
  Metadata:
    title           : Impact Moderato
    album           : YouTube Audio Library
    artist            : Kevin MacLeod
    genre           : Cinematic
  Duration: 00:00:33.53, bitrate: 512 kb/s
  Stream #0:0: Audio: pcm_s16le ([1][0][0][0] / 0x0001), 16000 Hz, 2 channels, s16, 512 kb/s


Use spaces instead of tabs

Funny stuff: my YAML 1.1 parser panics on these tabs, wondering if it's mandated by spec or just this lib specific behavior.

Regardless we use spaces for indentation in the repo (and ideally should have CI ensuring that at some point) so this looks good to merge to me. I suspect (but won't bother checking) that even though tabs are almost certainly allowed in YAML, mixing them as it seems has happened here is a recipe for really poor behavior. Thanks.

I don't get this. All these files are pure JSON, which disregards whitespace.  Given that YAML is a superset of JSON, I'd expect any YAML parser to handle tabs.

Given that YAML is a superset of JSON

That's what I read and believed to be true. Tried to find answers in YAML spec but I've given up. Probably just the parser I use (https://bitbucket.org/snakeyaml/snakeyaml) is faulty but cannot say this with 100% certainty.

You might consider opening a bug with them.  They can investigate, and if something is wrong, hopefully they fix it.

my YAML 1.1 parser panics on these tabs

These files are JSON, not YAML. You should use a JSON parser to read these files.
Contrary to what some claim, JSON is not a subset of YAML.
https://john-millikin.com/json-is-not-a-yaml-subset

YAML 1.2 is a superset of JSON, at least noone was able to prove otherwise :)

Just confirming that what @perlpunk says.
@karenetheridge if you read through the comments of the https://news.ycombinator.com/item?id=31406473 post you referenced,
you'll see that the YAML Development Team got involved and showed that about 5 cases brought up asserting that specific JSON inputs were not valid YAML 1.2, were shown to be incorrect according to the YAML 1.2 spec.
This is not to say that someone won't find one, but as of this moment no one has found any to my knowledge.
@harrel56 snakeyaml and many others are far from 100% correct as shown by @perlpunk's https://matrix.yaml.info/
Also @karenetheridge's advice to use a JSON parser to parse JSON is a good idea, unless you know that the YAML implementation you are using is in fact correct.


[webview_flutter] Adjust getTitle test

The getTitle test has been flaking on iOS, for reasons that are not
clear. This is a highly speculative attempted fix, in case the issue is
that the futere is completing too early (e.g., if 'about:blank' is
triggering a completion).
Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the relevant style guides and ran the auto-formatter. (Unlike the flutter/flutter repo, the flutter/plugins repo does use dart format.)
[x] I signed the CLA.
[x] The title of the PR starts with the name of the plugin surrounded by square brackets, e.g. [shared_preferences]
[x] I listed at least one issue that this PR fixes in the description above.
[x] I updated pubspec.yaml with an appropriate new version according to the pub versioning philosophy, or this PR is exempt from version changes.
[x] I updated CHANGELOG.md to add a description of the change, following repository CHANGELOG style.
[ ] I updated/added relevant documentation (doc comments with ///).
[x] I added new tests to check the change I am making, or this PR is test-exempt.
[x] All existing and new tests are passing.

I don't this this will actually work, but I thought it was worth trying first.
My other theory is that the premise of the test—that the title is available as soon as the page loads—is actually wrong. title is KVO-compliant, but we don't have a way to use that from the Dart side (maybe long term we should consider having an event stream for various page attributes), so if we still see flake I was thinking of adding a step where we run JS before checking the title, to ensure that the page has been fully parsed and handled.

Landing on red since the red on master is due to this test flaking.

(Reverted by https://github.com/flutter/plugins/pull/4539)


Enable configuration flags via component config

This change enables controller configuration via a versioned config file, referred to as "component config". Configuration options can now be passed to the controller by a ConfigMap.

https://book.kubebuilder.io/component-config-tutorial/tutorial.html

Signed-off-by: Michail Resvanis<EMAIL_ADDRESS>

@mresvanis I don't think we have the schema for the config file in this PR?

Codecov Report
Base: 71.02% // Head: 70.78% // Decreases project coverage by -0.23% :warning:

Coverage data is based on head (44fa0a2) compared to base (3741e9f).
Patch coverage: 0.00% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #143      +/-   ##
==========================================
- Coverage   71.02%   70.78%   -0.24%
==========================================
  Files          23       23
  Lines        2136     2143       +7
==========================================
  Hits         1517     1517
- Misses        548      555       +7
  Partials       71       71

Impacted Files
Coverage Δ

main.go
4.13% <0.00%> (-0.26%)
:arrow_down:

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

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

@mresvanis I don't think we have the schema for the config file in this PR?

Indeed there is no custom config schema defined in this PR and you are also correct that we need to add a custom schema, in order to support new controller configuration flags (as per the documentation).
But the purpose of this PR is to enable the default controller-runtime controller config schema, in order to keep our PRs smaller. In a following PR, where we add new controller config flags, we should definitely add a custom config schema.

@mresvanis is this PR part of HubSpoke? I mean, what other options do we need for our operator?

@yevgeny-shnaidman this is part of the Hub & Spoke effort. Having the options set in a ConfigMap makes it easier to configure KMM on Spokes.

@yevgeny-shnaidman this is part of the Hub & Spoke effort. Having the options set in a ConfigMap makes it easier to configure KMM on Spokes.

@qbarrand @mresvanis  we are talking about configuring which controllers to run on Hub vs Spoke, right? has it already been decided that we are going with runtime configuration as opposed to build time configuration?

@yevgeny-shnaidman this is part of the Hub & Spoke effort. Having the options set in a ConfigMap makes it easier to configure KMM on Spokes.

@qbarrand @mresvanis we are talking about configuring which controllers to run on Hub vs Spoke, right? has it already been decided that we are going with runtime configuration as opposed to build time configuration?

This is a good question and I suggest that we move this conversation to the KMM Hub & Spoke design document. This PR's goal is to replace random CLI configuration flags with a single CLI for the component config. As briefly explained here.
What do you think?

/ok-to-test

/lgtm
/assign @ybettan @yevgeny-shnaidman

/approve


Fixed by https://github.com/Korbeil/website/commit/3e37b61c1fd3fed1d8a7791ce0adc861982cdb26
Could you implement some tags to make it easier to switch between theses changes ?

Hi. sorry for that, i have made a tag for previous commit before the "major redesign". check it here: https://github.com/heiswayi/the-plain/releases/tag/v3.1.0

Thanks ! I used the tag to make it cleaner :) (cf. https://github.com/Korbeil/website/commit/b20cc476c7e1966189af44a46f9ea1562334f95b)
Again, thanks a lot for your theme ! ;)


chore: print information about used config file in help message

chore: print information about used config file in help message

@hadjian how's this for a cheap solution to orienting the user to where the config is found?


unauthorized: authentication required

From @eliassal on January 17, 2018 8:37
Hi, whenever I try
docker pull microsoft/aspnet, I get
Sending build context to Docker daemon  24.06kB
Step 1/7 : FROM microsoft/aspnet
latest: Pulling from microsoft/aspnet
3889bb8d808b: Downloading [=====>                                             ]    446MB/4.07GB
449343c9d7e2: Downloading [=================>                                 ]  445.9MB/1.304GB
ccd8f322a2de: Download complete
5358f1f8c04f: Download complete
27fb1950158d: Download complete
4447c3ffe068: Download complete
94eeeaf0d3bd: Download complete
fe1d276c8d64: Downloading
25871a4250c9: Waiting
b53cc298d0b7: Waiting
unauthorized: authentication required

any help? Opened a thread on docker site, nobody responded. My platform is Docker for windows as follows

Client:
Version:       17.12.0-ce
API version:   1.35
Go version:    go1.9.2
Git commit:    c97c6d6
Built: Wed Dec 27 20:05:22 2017
OS/Arch:       windows/amd64
Server:
Engine:
Version:      17.12.0-ce
API version:  1.35 (minimum version 1.24)
Go version:   go1.9.2
Git commit:   c97c6d6
Built:        Wed Dec 27 20:15:52 2017
OS/Arch:      windows/amd64

If you are having an issue with microsoft/aspnet, please open an issue on https://github.com/Microsoft/aspnet-docker.
This repository is for microsoft/aspnetcore and microsoft/aspnetcore-build.
Steps to reproduce the issue
(e.g. copy your Dockerfile or docker-compose.yml file here)
1.
2.
3.
Expected behavior
Actual behavior
Additional information (e.g. issue happens only occasionally)
Output of dotnet --info
(paste your output here)

Output of docker info
(paste your output here)

Copied from original issue: aspnet/aspnet-docker#352

Closing as this issue is stale and the cause of the issue appears to be an intermittent problem with docker.


new comments

Added description and example for "trailing comments"

[x] I have read freeCodeCamp's contribution guidelines.
[ ] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

Hi @chrisgithubok
It seems that similar changes have already been accepted earlier for this article you're editing, sorry about that. 😓
If you feel you have more to add, please feel free to open up a new PR.
Thanks again! 😊

If you have any questions, feel free to reach out through Gitter or by commenting below. 💬


[2906] Add a migration to fix subject areas naming

Context
The names of two subjects areas have incorrect casing.
Changes proposed in this pull request
Change the names of those subject areas so they appear correctly in Find:

Secondary: Modern Languages => Secondary: Modern languages
Further Education => Further education

Before:

After:

Guidance to review

run migration in teacher-training-api: bundle exec rails db:migrate RAILS_ENV=development
visit http://localhost:3002/results/filter/subject

Checklist

[x] Make sure all information from the Trello card is in here
[x] Attach to Trello card
[x] Rebased master
[x] Cleaned commit history
[x] Tested by running locally

 Subject filter page Subject Area heading


[Bug] Yarn warns about missing dependencies.

Describe the bug
When installing the package with Yarn, it displays warnings about missing dependencies.
Steps to reproduce the behavior

yarn init -y
yarn set version berry
yarn add -D @storybook/addon-styling
See errors in the output.

Expected behavior
No warnings in the output.
Screenshots and/or logs
➤ YN0002: │ @storybook/addon-styling@npm:1.0.8 [0fafd] doesn't provide less (p653d7), requested by less-loader
➤ YN0002: │ @storybook/addon-styling@npm:1.0.8 [0fafd] doesn't provide postcss (pa85f6), requested by postcss-loader
➤ YN0002: │ @storybook/addon-styling@npm:1.0.8 [0fafd] doesn't provide webpack (p71ff3), requested by css-loader
➤ YN0002: │ @storybook/addon-styling@npm:1.0.8 [0fafd] doesn't provide webpack (p742b7), requested by less-loader
➤ YN0002: │ @storybook/addon-styling@npm:1.0.8 [0fafd] doesn't provide webpack (p928e3), requested by postcss-loader
➤ YN0002: │ @storybook/addon-styling@npm:1.0.8 [0fafd] doesn't provide webpack (p0a83b), requested by sass-loader
➤ YN0002: │ @storybook/addon-styling@npm:1.0.8 [0fafd] doesn't provide webpack (p182a3), requested by style-loader
Environment

OS: Windows 11
Node.js version: 18.12..1
Yarn version: 3.5.1

The new version 1.3.3 that has just got released fixes this issue.


Remove .gitattributes file.

Makes git diff Glide lock files as text, which is helpful to see changes introduced by a glide update run.
@emilevauge as discussed on Slack. Please review. :-)

/cc @containous/traefik


Build Guide

Is it possible to give up to date instructions on how to try building these projects ourselves? The best I can find is a 3 year old blog post from yourself.
Whilst it is very detailed, it is beginning to get old and some bugs seem to happen on the build.
Thanks

Everything's possible, but I'm not going to do that, sorry :). There's also not that much to say: the guide I wrote has instructions how to set up masm, which is the biggest pain point. VS2019 will build some of them just fine (but older ones needs some adjustment/migration), perhaps you need to change default into defaultValue as it's clashing with the newer C++ version used in vs2019, and change BYTE to uint8_t as it's no longer used as a type in newer SDKs.
The code in this repository is basically outdated. All my newer cameras aren't in this repo, except cyberpunk but this cp camera isn't going to be updated. I kept it open as links are pointing to this repo, but no plans to change code in this repo.
For binaries, sub to my patreon: https://patreon.com/Otis_Inf.


There's an unpowered light on Bagel

Description

It's just above arrivals.
Screenshots

is it the newest version? there was a wire fix in https://github.com/space-wizards/space-station-14/pull/31566

This was like two rounds ago yea


Still not compatible with Traveler's Backpack

I downloaded the latest version of the just action autobuild, and in this version right clicking on the grave after dying with a backpack on doesn't cause the server to crash, but it also doesn't return the backpack to the player (the backpack disappears)I would appreciate it if you could resolve this issue!

Thank you for reporting this. I will need to do some extensive testing as I've done some changes in the last few commits to how items are stored (their drop rules are now also stored/cached at the same time), and some stuff might break. So at the moment it's possibly quite unstable, specifically with mod compats. Should have fixed this one issue in the latest commit though. Haven't noticed any other issue yet, but I still know some cases I need to test
Basically what I'm saying is it could work fine, but tread with caution, since things could also go wrong

Just experienced this issue myself, however upon reteving my grave it DID cause the server to crash with the following message in the console. Full log is attached as well.
crash-2024-01-21_13.53.47-server.txt
[13:53:47 ERROR] [Server thread]: [MinecraftServer] Encountered an unexpected exception
java.lang.NoSuchMethodError: 'void com.tiviacz.travelersbackpack.component.ITravelersBackpackComponent.syncToTracking(net.minecraft.class_3222)'
        at com.b1n_ry.yigd.compat.TravelersBackpackCompat$TBCompatComponent.storeToPlayer(TravelersBackpackCompat.java:81) ~[youre-in-grave-danger-2.0.0-beta.6.jar:?]
        at com.b1n_ry.yigd.components.InventoryComponent.applyToPlayer(InventoryComponent.java:443) ~[youre-in-grave-danger-2.0.0-beta.6.jar:?]
        at com.b1n_ry.yigd.components.GraveComponent.applyToPlayer(GraveComponent.java:480) ~[youre-in-grave-danger-2.0.0-beta.6.jar:?]
        at com.b1n_ry.yigd.components.GraveComponent.claim(GraveComponent.java:335) ~[youre-in-grave-danger-2.0.0-beta.6.jar:?]
        at com.b1n_ry.yigd.block.GraveBlock.use(GraveBlock.java:152) ~[youre-in-grave-danger-2.0.0-beta.6.jar:?]
        at net.minecraft.world.level.block.state.BlockBehaviour$BlockStateBase.use(BlockBehaviour.java:1133) ~[?:?]
        at net.minecraft.server.level.ServerPlayerGameMode.useItemOn(ServerPlayerGameMode.java:343) ~[?:?]
        at net.minecraft.server.network.ServerGamePacketListenerImpl.handleUseItemOn(ServerGamePacketListenerImpl.java:1140) ~[?:?]
        at net.minecraft.network.protocol.game.ServerboundUseItemOnPacket.handle(ServerboundUseItemOnPacket.java:34) ~[?:?]
        at net.minecraft.network.protocol.game.ServerboundUseItemOnPacket.handle(ServerboundUseItemOnPacket.java:8) ~[?:?]
        at net.minecraft.network.protocol.PacketUtils.lambda$ensureRunningOnSameThread$0(PacketUtils.java:22) ~[?:?]
        at net.minecraft.server.TickTask.run(TickTask.java:18) ~[?:?]
        at net.minecraft.util.thread.BlockableEventLoop.doRunTask(BlockableEventLoop.java:156) ~[?:?]
        at net.minecraft.util.thread.ReentrantBlockableEventLoop.doRunTask(ReentrantBlockableEventLoop.java:23) ~[?:?]
        at net.minecraft.server.MinecraftServer.doRunTask(MinecraftServer.java:782) ~[server-intermediary.jar:?]
        at net.minecraft.server.MinecraftServer.doRunTask(MinecraftServer.java:164) ~[server-intermediary.jar:?]
        at net.minecraft.util.thread.BlockableEventLoop.pollTask(BlockableEventLoop.java:130) ~[?:?]
        at net.minecraft.server.MinecraftServer.pollTaskInternal(MinecraftServer.java:764) ~[server-intermediary.jar:?]
        at net.minecraft.server.MinecraftServer.pollTask(MinecraftServer.java:758) ~[server-intermediary.jar:?]
        at net.minecraft.util.thread.BlockableEventLoop.managedBlock(BlockableEventLoop.java:139) ~[?:?]
        at net.minecraft.server.MinecraftServer.waitUntilNextTick(MinecraftServer.java:743) ~[server-intermediary.jar:?]        at net.minecraft.server.MinecraftServer.runServer(MinecraftServer.java:675) ~[server-intermediary.jar:?]
        at net.minecraft.server.MinecraftServer.lambda$spin$2(MinecraftServer.java:265) ~[server-intermediary.jar:?]
        at java.lang.Thread.run(Thread.java:1583) ~[?:?]

Hello, I'll try to release fix for this, I'll add back the deleted method, so it'll prevent the crash. It will be empty method, but will prevent the issue. I'll try to do this tomorrow

There's no need. I've uploaded a new version just now that fixes this. I've just been waiting for the last 2 hours for the version to get approved so I could respond and close the issue at the same time (as I like to only close issues once they've been released publicly)

do you release update for older versions too?

Oh, um... no I don't

Yeah, so I'll release update for older versions too

Okay, actually I've checked your code and in older versions you use equipBackpack method, which do not cause any errors. the issue was only present in 1.20.1 because you switched to manual syncing. I'm not going to release any updates and issue is resolved


gogs，eclipse git push出现401 Unauthorized（急）

gogs开发团队你们好，
我在用eclipse的git插件push的时候出现以下问题，
Can't connect to any repository: http://<IP_ADDRESS>:3000/tuan/ihome.git (http://<IP_ADDRESS>:3000/tuan/ihome.git: 401 Unauthorized)
使用Git Bash -》执行 git push origin master 是可以正常push
开始我怀疑是，我的eclipse git插件出现问题,紧接着我切换到github，gitosc，push都是正常！
Eclipse版本
Luna Service Release 1a (4.4.1),32位
所以我想请问gogs开发者，是否运到这个问题？
在搭建gogs的配置
[root@localhost gogs]# less custom/conf/app.ini
APP_NAME = Gogs
RUN_USER = root
RUN_MODE = prod
[database]
DB_TYPE  = mysql
HOST     = <IP_ADDRESS>:3306
NAME     = gogs
USER     = root
PASSWD   = root
SSL_MODE = disable
PATH     = data/gogs.db
[repository]
ROOT = /git/gogs-repositories
[server]
DOMAIN           = <IP_ADDRESS>
HTTP_PORT        = 3000
ROOT_URL         = http://<IP_ADDRESS>:3000/
DISABLE_SSH      = false
SSH_PORT         = 22
START_SSH_SERVER = false
OFFLINE_MODE     = false
[mailer]
ENABLED = false
[service]
REGISTER_EMAIL_CONFIRM = false
ENABLE_NOTIFY_MAIL     = false
DISABLE_REGISTRATION   = false
ENABLE_CAPTCHA         = true
REQUIRE_SIGNIN_VIEW    = false
[picture]
DISABLE_GRAVATAR        = false
ENABLE_FEDERATED_AVATAR = true
[session]
PROVIDER = file
[log]
MODE      = console, file
LEVEL     = Info
ROOT_PATH = /git/gogs/log
[security]
INSTALL_LOCK = true
SECRET_KEY   = 8MyRxC5oi20eDoo
(END)
ENABLED = false
[service]
REGISTER_EMAIL_CONFIRM = false
ENABLE_NOTIFY_MAIL     = false
DISABLE_REGISTRATION   = false
ENABLE_CAPTCHA         = true
REQUIRE_SIGNIN_VIEW    = false
[picture]
DISABLE_GRAVATAR        = false
ENABLE_FEDERATED_AVATAR = true
[session]
PROVIDER = file
[log]
MODE      = console, file
LEVEL     = Info
ROOT_PATH = /git/gogs/log
[security]
INSTALL_LOCK = true
SECRET_KEY   = 8MyRxC5oi20eDoo

出现同样的问题
日志如下：
Exception in thread "main" org.eclipse.jgit.api.errors.TransportException: http://run@<IP_ADDRESS>:10080/run/test.git: 401 Unauthorized
at org.eclipse.jgit.api.PushCommand.call(PushCommand.java:164)
at com.run.util.GitUtils.main(GitUtils.java:489)
Caused by: org.eclipse.jgit.errors.TransportException: http://run@<IP_ADDRESS>:10080/run/test.git: 401 Unauthorized
at org.eclipse.jgit.transport.TransportHttp$Service.openResponse(TransportHttp.java:875)
at org.eclipse.jgit.transport.TransportHttp$MultiRequestService.execute(TransportHttp.java:977)
at org.eclipse.jgit.transport.TransportHttp$Service$HttpExecuteStream.read(TransportHttp.java:903)
at org.eclipse.jgit.util.io.UnionInputStream.read(UnionInputStream.java:145)
at org.eclipse.jgit.util.IO.readFully(IO.java:247)
at org.eclipse.jgit.transport.PacketLineIn.readLength(PacketLineIn.java:186)
at org.eclipse.jgit.transport.SideBandInputStream.needDataPacket(SideBandInputStream.java:154)
at org.eclipse.jgit.transport.SideBandInputStream.read(SideBandInputStream.java:136)
at org.eclipse.jgit.util.IO.readFully(IO.java:247)
at org.eclipse.jgit.transport.PacketLineIn.readLength(PacketLineIn.java:186)
at org.eclipse.jgit.transport.PacketLineIn.readString(PacketLineIn.java:138)
at org.eclipse.jgit.transport.BasePackPushConnection.readStringLongTimeout(BasePackPushConnection.java:375)
at org.eclipse.jgit.transport.BasePackPushConnection.readStatusReport(BasePackPushConnection.java:327)
at org.eclipse.jgit.transport.BasePackPushConnection.doPush(BasePackPushConnection.java:202)
at org.eclipse.jgit.transport.TransportHttp$SmartHttpPushConnection.doPush(TransportHttp.java:807)
at org.eclipse.jgit.transport.BasePackPushConnection.push(BasePackPushConnection.java:154)
at org.eclipse.jgit.transport.PushProcess.execute(PushProcess.java:167)
at org.eclipse.jgit.transport.Transport.push(Transport.java:1250)
at org.eclipse.jgit.api.PushCommand.call(PushCommand.java:157)
... 1 more

0.11.4 和0.11.6 都有同样的bug。

Having the same issue with netbeans, but when I push trough terminal works OK.

Can you reproduce on demo site?

What do I need to do?

Can you reproduce this issue on demo site?

ok,Wait a moment！

日志信息:
[Macaron] 2017-05-18 19:29:34: Started POST /tuan001/ihome.git/git-receive-pack for <IP_ADDRESS>
2017/05/18 19:29:34 [TRACE] Session ID: 372aeef284564946
2017/05/18 19:29:34 [TRACE] CSRF Token: FL01DFJMl9InhjrxWR5cK745jGo6MTQ5NTEwNjk3NDcyMjg5NzY2Mw==
[Macaron] 2017-05-18 19:29:34: Completed POST /tuan001/ihome.git/git-receive-pack 401 Unauthorized in 15.947562ms
截图如下:
https://github.com/gyjx/test/blob/master/002.png
Eclipse问题如下:
https://github.com/gyjx/test/blob/master/无标题.png
Can't connect to any repository: http://<IP_ADDRESS>:3000/tuan001/ihome.git (http://<IP_ADDRESS>:3000/tuan001/ihome.git: 401 Unauthorized)

gogs部署在VM虚拟机 centos6.9_64

Thanks for your time and sorry about inconvenience.
However:

You are not answering my question: Can you reproduce this issue on demo site?
Please read issue template, the one you delete without a second thought: https://github.com/gogits/gogs/blob/master/.github/ISSUE_TEMPLATE.md

reproduced in our environment:
Eclipse reported:
Failed pushing to http://blablabla/bla/bla.git
http://blablabla/bla/bla.git: 401 Unauthorized
gogs web output read:
[Macaron] 2017-05-18 17:41:52: Completed POST /bla/blablabla.git/git-receive-pack 401 Unauthorized in 9.0005ms
gogs version: <IP_ADDRESS>5

If you can't reproduce on demo site and git bash works fine... I can't help you..

Can't connect to any repository: https://try.gogs.io/xiaoye/test.git (https://try.gogs.io/xiaoye/test.git: 401 Unauthorized)

我在你们demo site 通过eclipse提交代码，也是不行的！

According to https://github.com/gogits/gogs/blob/master/.github/ISSUE_TEMPLATE.md:

Please speak English, we have forum in Chinese.

sorry,my English is poor！
I have already posted a question description in your Chinese Forum. Please click on this hyperlink。
https://discuss.gogs.io/t/eclipse-git-https-try-gogs-io-https-try-gogs-io-xiaoye-test-git-401-unauthorized/1138
thank you!

@gyjx 请问这个问题解决了吗？能说下解决方案吗

@lrlovecode  sorry!

@lrlovecode  把仓库设置成私有的。Set your warehouse to private.


Provide a switch for ssllabs to unfilter "ECDHE parameter reuse"

Same way of the --grade options, can you please add a filter to allow results about ECDHE parameters?
Maybe a future proof version can be to use a regex on the results and allow everything whitelisted by that regex to go through.

This should be realized with #470


Gifu For Swift 2

For me Gifu isn't working under Swift 2. I assume it isn't updated yet but can anyone else confirm that this is currently expected?
Thanks!

Updated now in master.

Thanks for the updates. I use Carthage to build, and it updates Runes, but I can't seem to get it to update Gifu. Still seems to be Swift 1.2. In Headers directory, Gifu-Swift.h it has Swift 1.2. Using Sources there are operator compile errors. Should carthage update be resulting in Swift 2.0?

@lmacfadyen I haven't released 1.0.0 yet (still working on docs). Use "kaishin/gifu" "master" in your Cartfile for now. Runes is no longer a dependency so you can remove it from your project if you are not using it in your own code.

Thank you for a fast response! Really appreciate it and will update to master that way. Cheers!


[Suggestion] - Don't assign pod annotations to test pods

Suggestion
Currently the podAnnotations are being assigned to the Jira pods AND the test pods. Having these annotations assigned to the test pods poses a number of security concerns, for example, IAM binding in an AWS environment and certain annotations to inject secrets into the pod based on the service account. In the later example this will outright stop the test pods from running as they do not have the required service account assigned. Due to this I suggest not assigning the podAnnotations to the test pods or allowing a method to configure them separately or overwriting the annotations passed to the test pods.
Product
Jira, Confluence
Code of Conduct

[X] I agree to follow this project's Code of Conduct

Thanks @bianchi2
Any idea when that might?


SECURITY: Azure/login in some cases leaks Azure Application Variables to the GitHub build log

As per the title.
Running a command such as:
- name: 'Configure Deployment Target'
  run: az webapp config appsettings set --name <NAME> --resource-group <RG> --settings DEPLOYMENT_TARGET=<TARGET>

Results in a returned result of:
[
  ***
    "name": "MY_SUPER_SECRET_APP_SETTING",
    "value": "<some SUPER_SECRET ENV var from Azure Key Store>
  ***,
  ***
    "name": "APPINSIGHTS_INSTRUMENTATIONKEY",
    "value": "<some value>
  ***,

Since the AZ CLI is by default assumed be running / logging in a secure environment I understand some commands are going to output this information but they should not be carried over to the git hub log.
I would recommend some additional settings:
uses: azure/login@v1
  with:
    creds: ${{ secrets.AZURE_CREDENTIALS }}
    git_log: true    # default false

    OR

    azure_log: true
    blob: ${{ secrets.BLOB_CREDENTIALS }} / PATH

Hi @NoCopy! If you don't want to show the output of az cli commands, you can try the option -o None.

@MoChilia Yes, I am aware of that. However that is a dangerous thing to rely on. Out of a pool of say 100 devs, I bet 70% are not aware of that setting - especially since it not even in the docs of this repo as a warning. Of that 30% that are aware another 50% of those are probably going to forget, make a typo, or remember when it is an issue - or just see it, like I did. I would have never expected to be leaking ENV vars to the log if my build didn't fail initially. Had it worked, I would have probably never noticed it.
In that respect, I HIGHLY suspect MANY repos are leaking ENV vars that they are not aware of. This is an issue that should be taken seriously. Defaulting to no logs would be the safest, and at the very least this repo should update the readme with a very strict security warning at the top (not way down below) to warn users of this potential pitfall.

@NoCopy, this action won't pass any account information or ENV vars to the GitHub log. The command you mentioned is not within the scope of login action. Nevertheless, I appreciate your suggestion and I believe it deserves attention.  I will record it as an improvement for the azure/cli action and discuss the design within the team. Thanks!

@NoCopy, the scope of login action is within the step
-name: Run Azure Login
     uses: azure/login@v1
     with: xxx

Once you finished the login action, it is difficult to handle the users' behavior in the following steps, since other Azure actions may have their own method to control the output.
And I want to clarify that the login action does not pass any console output to the GitHub log. All of the outputs are prompts without any confidential Information, like
/usr/bin/az cloud set -n azurecloud
Done setting cloud: "azurecloud"
Note: Azure/login action also supports OIDC login mechanism. Refer https://github.com/azure/login#configure-a-service-principal-with-a-federated-credential-to-use-oidc-based-authentication for more details.
Login successful.


Enable travis CI

Add tox.ini
Enable travis tests

@kmike , regarding your comment about the branch constraint.
If I kept it, it wasn't triggering the build here, probably because it's not on master yet.
So, I'll leave it out for now, or apply that change before merging, so we can get the build here

👍 Looks great, +1 to merge.


Locating resources relative to the input file.

This commit allows the use of relative resource dictionaries without the use of the pack URI syntax, e.g:
<ek:UIRoot.Resources>
    <ResourceDictionary>
        <ResourceDictionary.MergedDictionaries>
            <ResourceDictionary Source="Resources.xaml"></ResourceDictionary>
        </ResourceDictionary.MergedDictionaries>
    </ResourceDictionary>
</ek:UIRoot.Resources>

Where Resources.xaml is defined in the same directory as the input file.

Cool, I did few tests and found some issues.

Using just parser context is not enough. I had to make change in ResourceDictionaryGenerator.
Using XamlReader.Parse with context is problematic. Method is using MemoryStream(UTF8Encoding.Default.GetBytes(xamlText)) and that's not good with some of my tests. Parsing fails on encoding. I tested

    using (Stream stream = new MemoryStream(Encoding.UTF8.GetBytes(inputFileContent)))
    {
        source = XamlReader.Load(stream, parserContext);
    }

Looks like that's working fine. I will test it more and if there is no other issue I will include this in next version.
Thanks

They are using UTF8Encoding.Default, which I is just Encoding.Default (which is the current system encoding, not necessarily UTF-8), that's poor programming style on their part. Nicely caught.
I'm glad this could be of use, I didn't like having to compile with the ResourceDictionary first before I could make use of it, I've started using a version with this change and it is making the use of shared resources much easier :)

If you want to use it now, change the Generate method in ResourceDictionaryGenerator too. There is IsAbsoluteUri test in it. This should work fine:
        public void Generate(ResourceDictionary dictionary, CodeTypeDeclaration classType, CodeMemberMethod initMethod, CodeExpression fieldReference)
        {
            foreach (var mergedDict in dictionary.MergedDictionaries)
            {
                string name = string.Empty;
                if (mergedDict.Source.IsAbsoluteUri)
                {
                    name = Path.GetFileNameWithoutExtension(mergedDict.Source.LocalPath);
                }
                else
                {
                    name = Path.GetFileNameWithoutExtension(mergedDict.Source.OriginalString);
                }

                if (string.IsNullOrEmpty(name))
                {
                    Console.WriteLine("Dictionary name not found.");
                    continue;
                }

                CodeMethodInvokeExpression addMergedDictionary = new CodeMethodInvokeExpression(
                        fieldReference, "MergedDictionaries.Add", new CodeFieldReferenceExpression(new CodeTypeReferenceExpression(name), "Instance"));
                initMethod.Statements.Add(addMergedDictionary);
            }
...