Cache preprocessed speech features issue of filename.decode

After the merge of #285 .
I run the following command:
python -u run.py --config_file=example_configs/speech2text/lstm_small_1gpu.py --mode=train_eval --decoder_params/use_language_model=True --use_horovod=False --num_gpus=4 --enable_logs
The following error is shown:

File "/data1/nlp/NVIDIA-OpenSeq2Seq/open_seq2seq/data/speech2text/speech2text.py", line 339, in _parse_audio_transcript_element
params=self.params
File "/data1/nlp/NVIDIA-OpenSeq2Seq/open_seq2seq/data/speech2text/speech_utils.py", line 174, in get_speech_features_from_file
preprocessed_data_path = get_preprocessed_data_path(filename, params)
File "/data1/nlp/NVIDIA-OpenSeq2Seq/open_seq2seq/data/speech2text/speech_utils.py", line 89, in get_preprocessed_data_path
filename = filename.decode(
AttributeError: 'str' object has no attribute 'decode'

@mvankeirsbilck

This is because of how the filenames are written/read from the csv file.
If they're written/read as binary strings, you need to decode.
If they're written/read as normal strings, you don't and get this error.
It might be there's some conversion in the data loading.
https://stackoverflow.com/questions/17615414/how-to-convert-binary-string-to-normal-string-in-python3
You could try something like:
    try: filename = filename.decode('ascii')  # required b/c we read filename from csv in binary mode
    except AttributeError: pass

Thanks for your rapid reply. Could you please create a PR? I think if people follow the import_librivox.py and read it by default repo. The csv file is normal strings.

#314 . Thank your @mvankeirsbilck


EEException: Image.clipToBoundsAndScale: The geometry for image clipping must be bounded.

Hi, thanks for the great library. I ran into the following error:
collection_name = "MODIS/061/MOD13A2"
collection = ee.ImageCollection(collection_name) \  .filterDate('2019-11-01', '2019-12-31') .filterBounds(roi)
collection.wx.to_xarray()
EEException: Image.clipToBoundsAndScale: The geometry for image clipping must be bounded.
This error also show up when I remove the bounds filter

Just found the Modis examples. nvm


Explicit peer dependencies

Fixes #106

@adidahiya why tsutils is peerDependencies instead of just dependencies?

I guess it is similar to https://github.com/Microsoft/tslint-microsoft-contrib/pull/371.
We have an error Error: Cannot find module 'tsutils' after run tslint

Meh, this is kind of the reason I never listed explicit peer dependencies in the first place (it gets tedious to maintain and doesn't provide many benefits). The whole idea here is to prevent multiple versions of tsc from being used in TSLint and its rule implementations at runtime. If I make tsutils a regular dependency then I believe there's a possibility that NPM could install a nested typescript module inside tsutils/node_modules and now we have two versions in use at runtime (despite the fact that typescript itself is a peer dependency of tslint-react) -- I'll need to double-check this.

I agree, it is good reason. But typescript is a peer dependency of tsutils (https://github.com/ajafff/tsutils/blob/master/package.json#L43) and I guess that if you use that library with specific API for specific range of versions you should make it as dependency.

And now with<EMAIL_ADDRESS>we have an error about not-installed tsutils for tslint-react (yeah, it is bug of npm but I guess package.json of tslint-react should has tsulits as deps)

@ajafff do you have any opinions here?

As @timocov said, tsutils has typescript in peerDependencies. Hence there will only be one version of typescript installed.
The only reason to put tsutils in peerDependencies is to avoid different (major)  versions of it in tslint and rules packages. But I don't think that's necessary.

@ajafff given that logic, it should probably be safe to make tslint also a regular dependency then? Since it also declares typescript as a peer.

I think that tslint should be peer dependency because tslint-react is plugin for tslint. Yes, tslint-react depends on tslint but in development only and just for typings.

I agree with @timocov. A plugin should not bring it's own version of the library it plugs into.
Take vrsource-tslint-rules for example: It has a regular dependency on<EMAIL_ADDRESS>You can install this package even if you have<EMAIL_ADDRESS>installed. You won't notice any incompatibility until you experience a runtime crash.


fix: propagate Err up the call stack instead of unwrap()

Removes an unneeded unwrap() and allows errors to propagate all the way back up to the original caller.

Nice catch!


sfsu: Panic when parsing glob pattern "*"

Describe the bug
I try sfsu search and leave blank as scoop search will perform list all package available for install, but sfsu give
error: the following required arguments were not provided:
  <PATTERN>

Usage: sfsu.exe search <PATTERN>

For more information, try '--help'.

Ok, maybe sfsu doesnt want blank input, then i try * as scoop always use * as implicit all, then something interesting happend
name = "sfsu"
operating_system = "Windows 10.0.22621 (Windows 11 Enterprise) [64-bit]"
crate_version = "1.8.1"
explanation = '''
Panic occurred in file 'src\commands\search.rs' at line 51
'''
cause = """
Invalid Regex provided. See https://docs.rs/regex/latest/regex/ for more info: Syntax(
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
regex parse error:
    (?i)*
        ^
error: repetition operator missing expression
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
)"""
method = "Panic"
backtrace = """

   0: 0x7ff701c7eef4 - git_odb_object_data
   1: 0x7ff701d99ab7 - git_smart_subtransport_git
   2: 0x7ff701d99f73 - git_smart_subtransport_git
   3: 0x7ff701aaeada - <unresolved>
   4: 0x7ff701a9fb4f - <unresolved>
   5: 0x7ff701a554c6 - <unresolved>
   6: 0x7ff701a5595a - <unresolved>
   7: 0x7ff701c73c88 - git_odb_object_data
   8: 0x7ff701a55938 - <unresolved>
   9: 0x7ff701d8938c - git_smart_subtransport_git
  10: 0x7ffe976e257d - BaseThreadInitThunk
  11: 0x7ffe9910aa58 - RtlUserThreadStart"""

my guess is in your regex, you missing glob case, ie this
 let pattern = {
            Regex::new(&format!(
                "{}{raw_pattern}",
                if self.case_sensitive { "" } else { "(?i)" },
            ))
            .expect("Invalid Regex provided. See https://docs.rs/regex/latest/regex/ for more info")
        };

maybe you can provide fast path here instead of passing directly pattern for regex, but i dont know, just a suggest

This is not a bug.
sfsu uses Regex internally, and as such does not support glob patterns.
The regex equivalent is '.*'.
As can be seen in the error message, check 'https://docs.rs/regex/latest/regex/' for more info on regular expressions.

more clarify as maybe someone stomp upon this, sfsu use regex which mean
scoop search * = sfsu search . # or scoop search . if you has hooked

@NotAsea This is clarified in sfsu search --help
Arguments:
  <PATTERN>  The regex pattern to search for, using Rust Regex syntax

As well as in the search docs


Length field in wrong location in header

In the Sun Raster header documented at http://steve.hollasch.net/cgindex/formats/sunraster.html the Bitmap length field immediately follows the depth field, and then is followed by the data type field, cmap type field, and cmap length.
This change will ripple through all the routines in the first release, including the C programs to create images.  It can be part of the refactoring for the second release to use a driver in Elf/OS for accessing the TMS9118 video card.

Fixed.  Updated code in release 2 branch and merged fixes to update code in main as well.


Documentation: Update with explanations of Tracer methods

Fixes #123
https://github.com/open-telemetry/opentelemetry-ruby/pull/118#issuecomment-542497023

I signed it

I signed it

I signed it

I signed it


package 'self' from https://github.com/NixOS/nixpkgs/tree/master/pkgs/development/interpreters/self crashes on run.

Issue description
package self from https://github.com/NixOS/nixpkgs/tree/master/pkgs/development/interpreters/self
crashes on run.
Steps to reproduce

krs@funtoo ~ $ nix-env -i self
installing 'self-2017.1'
building '/nix/store/h0bfil5vy88srcgsprd82757ispb9vxp-user-environment.drv'...
created 24 symlinks in user environment
krs@funtoo ~ $ which Self
/home/krs/.nix-profile/bin/Self
krs@funtoo ~ $ cd .local/share/self/objects/
krs@funtoo objects $ Self

Internal error: signal 11 code 1 addr 0x8 pc 0x8190b60.

VM Version: 2017.1.13, Fri 09 Jul 21 21:06:22 Linux i386

Self process 13447 on funtoo has crashed.
Do you want to:
   1) Quit Self (optionally attempting to write a snapshot)
   2) Try to print the Self stack
   3) Try to return to the Self prompt
   4) Force a core dump
   5) Print the interrupted context registers
Your choice: 3

Cannot return to the prompt - Self is not properly initialized
Self VM fatal error (/build/source/vm/src/any/runtime/interruptedCtx.cpp, line 74): Reached supposedly impossible case

VM Version: 2017.1.13, Fri 09 Jul 21 21:06:22 Linux i386

Self process 13802 on funtoo has crashed.
Do you want to:
   1) Quit Self (optionally attempting to write a snapshot)
   2) Try to print the Self stack
   3) Try to return to the Self prompt
   4) Force a core dump
   5) Print the interrupted context registers
Your choice: 1
Enter snapshot name (hit return to omit snapshot) > blah
Attempting to write a snapshot to `blah
'...done.
Note that it cannot be guaranteed that the snapshot is good.
Even if it appears to start without problems, it is prudent
to verify the state of the system after startup using the
_Verify primitive.  If this reports problems, it is not
wise to continue using the snapshot.
Hit return to exit >

Technical details
Please run `nix-shell -p nix-info --run "nix-info -m"` and paste the result.
$ nix-shell -p nix-info --run "nix-info -m"
these paths will be fetched (0.00 MiB download, 0.00 MiB unpacked):
  /nix/store/fs4mvc0f27vhf4jv6hy54vmj7m7m8lqb-nix-info
copying path '/nix/store/fs4mvc0f27vhf4jv6hy54vmj7m7m8lqb-nix-info' from 'https://cache.nixos.org'...
 - system: `"x86_64-linux"`
 - host os: `Linux 5.10.46_p2-debian-sources, Funtoo, 1.4-release-std-2021-05-29`
 - multi-user?: `no`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.3.15`
 - channels(krs): `"nixpkgs-21.05.2050.465daf79b4a"`
 - nixpkgs: `/home/krs/.nix-defexpr/channels/nixpkgs`
 -

cc @doublec as maintainer

I've been experiencing this, as well as another person. I presume the maintainer @doublecc is inactive?

That's fine. When I have time, I'm planning to revisit the Self and other POOP (not an insult, it stands for Pure Object Oriented Programming) languages like Squeak and Pharo and fix them up.

package got removed in 23e8423184b0b22058b8a257538bde10e3cdade2


runtime: GODEBUG=memprofilerate=1 does not take effect.

What version of Go are you using (go version)?

$ go version
go version go1.19.1 darwin/arm64

Does this issue reproduce with the latest release?
yes
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GO111MODULE=""
GOARCH="arm64"
GOBIN=""
GOCACHE="/Users/r/Library/Caches/go-build"
GOENV="/Users/r/Library/Application Support/go/env"
GOEXE=""
GOEXPERIMENT=""
GOFLAGS=""
GOHOSTARCH="arm64"
GOHOSTOS="darwin"
GOINSECURE=""
GOMODCACHE="/Users/r/go/pkg/mod"
GONOPROXY=""
GONOSUMDB=""
GOOS="darwin"
GOPATH="/Users/robert/go"
GOPRIVATE=""
GOPROXY="https://goproxy.cn,direct"
GOROOT="/Users/r/workspace/testgo/go1.19.1/go"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/Users/r/workspace/testgo/go1.19.1/go/pkg/tool/darwin_arm64"
GOVCS=""
GOVERSION="go1.19.1"
GCCGO="gccgo"
AR="ar"
CC="clang"
CXX="clang++"
CGO_ENABLED="1"
GOMOD="/dev/null"
GOWORK=""
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -arch arm64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/j_/rq9ph2cd3h50w468sgv34lwh0000gn/T/go-build2412819877=/tmp/go-build -gno-record-gcc-switches -fno-common"

What did you do?

// main.go
package main

import (
	"fmt"
	"runtime"
)

func main() {
	fmt.Println(runtime.MemProfileRate, runtime.MemProfile)
}

go build main.go
GODEBUG=memprofilerate=1  ./main
What did you expect to see?
Expect output，ignore the second value which may be change every time:
1 0x10029a650
What did you see instead?
524288 0x10029a650
more
I also test this on linux-amd64 and it does not work either.
I also test this with different go version. go1.17 and go1.18 and go1.19 all failed. go1.16 works as expect.

I think it was introduced by https://go-review.googlesource.com/c/go/+/299671

I think it was introduced by https://go-review.googlesource.com/c/go/+/299671

Yes, you are right. After I revert this commit, it works!

Hi @cherrymui , does this need backport to 1.18 and 1.19 ?


Script to reset ckcp

This PR introduces a script to clean up the resources deployed/used by ckcp.
Signed-off-by: Satyam Bhardwaj<EMAIL_ADDRESS>

Apart from the comments inline, I see that there's no error handling anywhere.. what happens when something does not get deleted? The script is not handling those scenarios.

That's true. Now, there are few checks around.
Moreover, I've removed uninstallation of various operators because that could lead to unintentional loss of user data/resources.
If we feel like it's a good idea to have this option, we can use flags/arguments to trigger removal of various operators!

There seems to be a few unresolved conversation. Let us know when the PR needs another review.

Hi @Roming22 @bnallapeta, I've addressed all of the comments on this PR. Please let me know if you're okay with them and take a quick look at this PR again.
Few points -

For uninstalling operators and remove their traces from the cluster, this script will do force cleanup of CRDs by patching them.
This script wouldn't halt at errors(or engulf them) in case a resource meant to be deleted is already not available. This is for places where we're directly referencing files for e.g Syncer manifest or an Argo CD application referred.
It will halt at errors which would somehow stop the intended job of this script.

@bnallapeta Feel free to merge if you do not have any more feedback.


Added System test QA Quiz Question

Checklist:

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)

Adds to #102

The added question is regarding system testing.

Please also add hacktoberfest-accepted label before merging this pr, as I intend to count this pr under my Hacktoberfest 2022 contribution.


runBusyFuture() doesn't reliably set error property on ViewModel in error case. Stacked version 3.0.0

Hi,
The runBusyFuture()-method doesn't seem to reliably set the setError/modelError-properties when the Future throws an error.
Solely the _errorStates-Map will receive an Exception/Error-entry, but neither setError() nor modelError properties are updated.
My example:
 List<Subscription>? list =         await runBusyFuture<List<Subscription>?>(_userService.getSubscriptions(), busyObject: subscriptions);
I use stacked 3.0.0.

[edit]
This seems to be a wanted scenario when you have a busyObject. Because then you also have to check for error individually with
if(viewModel.error(busyObject)) ... // show error in UI
Wouldn't it be better if the hasError also gets set to true when we have any error ?

Hey @JohnnyRainbow81 I think it would be a good move to ensure hasError also returns true in that case. Do you think you can create a PR to implement this functionality for us?

Thank you. I'm excited to keep pushing updates. I have been very focused on my startup but now that we can push forward on sales with the product I have some time to create content and improve stacked again.
The only concern here is currentError . I don't think it's guaranteed that the value property on a map is ordered by LIFO. But I didn't do any research on it. If it is ordered in that way then I'm happy with the code you want to write.

Good point. I did some research and it isn't said that Map<K,V> doesn't guarantee to return the last element when you call map.entries.last, but LinkedHashMap<K,V> does!
That would involve to change you _errorStates - Map to LinkedMap.
Like this:
`  LinkedHashMap<int, dynamic> _errorStates = LinkedHashMap<int, dynamic>();
dynamic error(Object object) => _errorStates[object.hashCode];
/// Returns the error existence status of the ViewModel
bool get hasError => error(this) != null;
/// Returns the error existence status of any error that was caught by the Viewmodel
bool get hasAnyError => _errorStates.entries.isNotEmpty && _errorStates.entries.any((element) => element.value != null);
/// Returns recently added error to the ViewModel
dynamic get currentError => _errorStates.entries.last.value;
/// Returns the error status of the ViewModel
dynamic get modelError => error(this);`
I am in no way a senior dev, so I can only hope that this approach kinda suits OK. Just sayin :)
Wish you much luck with your startup! 🚀

That change should be fine. Given that the _errorStates map is not used externally the type of it doesn't really matter.
I'm happy using a LinkedHashMap

PR in progrss in #750 waiting on final review changes before merging.

The PR went stale. I'll leave it open and will attempt to merge it in on my next working session


Add badges and autorelease support

This should be the last step before actually releasing a 0.0.1 on PyPI.

Codecov Report

Merging #5 (9c523a7) into main (a73fc79) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##              main        #5   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           14        14
  Lines          359       364    +5
=========================================
+ Hits           359       364    +5

Impacted Files
Coverage Δ

ghcontribs/tests/conftest.py
100.00% <ø> (ø)

ghcontribs/query.py
100.00% <0.00%> (ø)

ghcontribs/contrib.py
100.00% <0.00%> (ø)

ghcontribs/monthly.py
100.00% <0.00%> (ø)

ghcontribs/__init__.py
100.00% <0.00%> (ø)

ghcontribs/json_utils.py
100.00% <0.00%> (ø)

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 a73fc79...9c523a7. Read the comment docs.


"Projects must be configuring" error during Gradle configuration with Kodein 7.13.1 in Kotlin/JS project

I tried to add JS support (again) to my multi-platform multi-module project incrementally.
Besides other things, I found out that adding org.kodein.di:kodein-di:7.13.1 as a JS project dependency produces the "Projects must be configuring" error, while org.kodein.di:kodein-di:7.13.0 does not.
Please see this issue for some more details: https://youtrack.jetbrains.com/issue/KT-52647

This issue is clearly not from us as we encounter it on no kodein dependent projects.
Closing the issue.


[Addition]:

Hotspot b58 Addresses
112DS7AA3HpxFcoYbYSDv1HRrqDgEBZBKsbFWDtn53tE6A9DtoXK
11ASSfQAnq1vsw3thmid4ZzF9ThYvHyLBAw3QgpT9xvveYsZpqe
1113N428CfQgycpb4vEtDD8S5da8MQLyaYdDKWrx1sE25qJXEC8
11yTS6dEj6rhj2KZ2k1QYwmwU39L3YrvapVVVQWGuyXwYLjd5dV
11b4gqCQbjYG2kcrgm6YxLwkrD3BmbPcijcm428KVRRVcvEdu5R
112XfEB4CmpzU9CUYQvzQimYrkrJvgLWeThAPBXtq8KXkB7GnT1q
11ewzUDkpGZrW6XMBXD17V5FtCsfmFXZkMMEoBTAsEa8aT7ccet
11cpxdhk8ZvPCafiPbSrtSGvzzABL8yxNxLvBLZthAy43Pzs2nU
11qUv6ovhqJPyC8vFQnzb1MUzVWpGGzCUm5asKahVpUTXczsYzp
11M9jgQhNnGLgpixXLZgKMkJzV4u4jxPez2RLr6R3SqN6XNpwKh
11uv63jpWRvbtHzxb8ZjWxwnkeStCRLyBVNJ89Jsrb57jtakamf
112C5sMomdXte1QQ9PJF8rsebFgtyN69gEJpRkPbDvgzi7PR3jDP
11teZKPr3eS5JQnuKFSXic6e966mzGF84dP6TUJRBGVxJu8ofj6
112RYmpkHrKeoMRMN97M6ceLk2Xr2JT3Wrgnt6S4E9eSGjEDBroE
11471uRm5D6i14LibJCbPiPYhaeyEAWQ1DVpxwDXLpbSQsmAcx5
11TMkY4156i2v5PpkTi6Q8oCaWh4yFtywzscddUzhD1QtwtPFUY
11HHq7GXkp4LYyjps5hSUS9ZqiMRRu7QK8CfNDkCq66xWcWJHcM
112tKSscNknV2zGYmQeozh8Vx67LsULhorC8PWPkkmf8ri4cAZ5o
112cd9uZSoY1vWGunrDBNLqWVQRNWHyspha7g9bXV1owz5GfY7Qu
1124RiThPps2R5muLQGNyKkt6G63UHWTwfQzKZJf46fKX1E6qF37
112LpVGed2CMWK2vqzjiUUoV2RakdHXFiPX859S6emT1LrJVo3BW
112ZHciEXV6sh87zYv9PmJCMqAbghS6FmF1VeqYiYtVMDirK4Wva
112PrtRSo1u9kL6kUvRb3CpXAe5tER3wMFqCSbsmFUTcCVX5kFhJ
112uZ6wEAT7AWRmhmtHpVSqSotpWpKMuGUb6riQkdSA1kb82Drks
112pLy9LFXDd1xFYdjBkhxAvzUmY7iK55Yy6kGtqKvbGtZCis7sz
112WiYpeZdrnQgVoeKxsQhq8J4CmnQThXkfb1rqNaADmzMsxE9Z5
112bYkCbH7fwNRNzYKjhNYFmkuerZfC5q7pVbVxmEoHn4PUNkzmG
Discord Handle
No response
Reason(s)
All mining machines are in one person's wallet. There are 50, distributed in different regions and cities. Only the mining machines in the wallet witness each other. It is obvious that they are a cheater and cheat helium.

guys really now?
Please continue to take care of these problems. It's annoying and damages the image of the network

We have taken into account this addition request in the next iteration of the denylist. Some or all of the Hotspots listed may be added to the denylist. This is an automated comment due to the volume of addition and removal requests. A future tool is in the works to provide more insight into the analysis approach made by the Helium team and community members that maintain this list.


[Mod]

Mod info

Mod name: Knes
Author(s):
License:
Homepage:
Download: curse forge or space dock

See https://github.com/KSP-CKAN/NetKAN/pull/8031, the author does not want this mod to be indexed on CKAN.


latent_diffusion.solver: assert msg must be f-strings

Solver step assert msg need to be f-strings

:facepalm:
I thought the linter checked this but it is actually in preview.


ReceiveWithTimeout should cancel outstanding timers

Is your feature request related to a problem? Please describe.
ReceiveWithTimeout doesn't cancel outstanding timers.
Describe the solution you'd like
Cancel outstanding timers.

This will be implicitly solved if/when https://github.com/temporalio/sdk-go/issues/986 is.


Get stuck on Windows with pnpm

System:
    OS: Windows 10 10.0.19043
    CPU: (8) x64 Intel(R) Core(TM) i7-10610U CPU @ 1.80GHz
    Memory: 666.18 MB / 15.73 GB
  Binaries:
    Node: 16.15.0
    PNPM: 7.9.0
  Packages:
    eslint: 8.22.0
    eslint-plugin-import: 2.26.0
    eslint-import-resolver-exports: 1.0.0-beta.2

This import resolver made eslint got stuck on Windows (perhaps also related to pnpm).
With following debug error:

FYI, it works well on Linux and MacOS

Sorry for the late response, I somehow missed this :/ I added Windows support in beta.3, please drop a line if it doesn't fix your problem.


[5.2] Modified MySqlConnector to allow a config param to execute the use

Added a config param to call the exec Use Statement in the mysql connector

Is there any reason we don't just always call the use statement? Would it really cause problems to just always make the call?

It's probably better not to make such SQL statements when they are not required. There is the overhead of course, and it could only introduce problems, potentially.

By the way, what is the use case for this PR?

I think we should just always make the call. It takes 0.04ms to complete on my machine.

:-(

for a single call, 0.04ms is huge
why making unneeded calls? what good could it bring?
I'd be interested to know @mclubb's use case.

If you have the same table on multiple databases it defines the default database, which should always be the database you actually have configured.

On May 23, 2016, at 10:44 AM, vlakoff<EMAIL_ADDRESS>wrote:
:-(
for a single call, 0.04ms is huge
why making unneeded calls? what good could it bring?
I'd be interested to know @mclubb https://github.com/mclubb's use case.
—
You are receiving this because you modified the open/close state.
Reply to this email directly or view it on GitHub https://github.com/laravel/framework/pull/13589#issuecomment-221014337

The way that we have our replication setup, it requires us to issue a use statement for any changes made to the database, by putting the config in the if check, will allow us to choose to make this call all the time. Since this is a config parameter, it shouldn't affect anything for existing users or users that don't need that call. Is this enough for you guys to reopen the PR? This would be awesome to have in the code base so that we don't have to make changes with each upgrade.

See https://github.com/laravel/framework/commit/ef770edb08f3540aefffd916ae6ef5c8db58f0af.

@mclubb I think you should have had either:

added this statement in your application
or written a custom connector


Fix for Windows where subtitles path should not be encoded for VLC

Fix for Windows where subtitles path should not be encoded for VLC.
Previous to this fix, VLC would not load the subtitles due to "weird" characters.

Thanks. Out in 0.33.0


Gsp275 fixes

Module 10  Realtime predictions with Dataflow ( Java)
Fixed errors with predict API response JSON and rare column type bug.
Updated pom.xml to update dependency versions and replace deprecated Dataflow SDK 2.5 dependency with Apache Beam SDK 2.15.

@googlebot I signed it!


[docs] Put in null as a type of value in onChange in Rating component

[o] I have followed (at least) the PR section of the contributing guide.

The previous PR I created somehow got closed immediately after I reverted my previous change.  Please look at this PR instead.
@eps1lon you told me to look at material-ui/packages/material-ui/src/Rating/Rating.d.ts Line 82, but I think you meant I might take a look of a different line of Rating.d.ts.  I couldn't understand when I looked an line 82.
Put in null in onChange in Rating, and also put in handling NullLiteral in auto generation of the descriptions of a function and its parameters.
Closes #26497

onChangeActive shouldn't get the same treatment since value type is number in onChangeActive.
https://github.com/mui-org/material-ui/blob/next/packages/material-ui/src/Rating/Rating.d.ts#L89-L91
Not very sure the actual reason why he | is escaped.  I had a reason that | is displayed without a rogue \ in the current material-ui site for Autocomplete API, and because he | is displayed with a rogue \ in Autocomplete API in the current deployed doc, the Rating API fix about null can just follow the current code of Autocomplete API.  When the Rating API code follows the Autocomplete API code, I guess | is displayed without a rogue \ in the current material-ui site also for Rating API.
| is displayed without a rogue \ in the current material-ui site as T | T[] for onChange of Autocomplete API:
https://material-ui.com/api/autocomplete/
Rogue \'s are revealed as number \| null and T \| T[] in onChange of both Rating API and Autocomplete API in the deployed docs:
https://deploy-preview-26542--material-ui.netlify.app/api/rating/
https://deploy-preview-26542--material-ui.netlify.app/api/autocomplete/

onChangeActive shouldn't get the same treatment since value type is number in onChangeActive.

@sakura90 I think that the point that Sebastian wanted to make is the the current prop description and type definition seems out of sync with the implementation. Isn't it this callback called 1. when using the keyboard navigation and 2. with a null value is landing in the reset option?

@oliviertassinari onChangeActive and onChange are spelled similarly, but they are just 2 different event listeners.
I think about 3 input scenarios - mouse, touch (from Android/iPhone), and keyboard - about these 2 event listeners.
For mouse, right now, onChange is called 1. when using the mouse selection and 2. with a null value is landing in the reset option, but onChangeActive is called 1. when using the mouse navigation and 2. with a -1 value when the position of the mouse is outside Rating component.
For touch, right now, onChange is called 1. when using the touch selection and 2. with a null value is landing in the reset option
From the prop description, onChangeActive is more for hovering, so I guess it is better not to consider it in touch, but FYI, onChangeActive is called when Rating component is in focus/blur from my understanding.
For keyboard, I don't spend too much time on both onChangeActive and onChange as it is a low frequent scenario.
Back to your question, I think it is fair to let onChangeActive be what it is.  onChangeActive is mainly used for the mouse scenario.  It feels unusual to see a -1 value when the position of the mouse is outside Rating component while an event listener with a similar name onChange returns a null in a similar situation.  Having a new type such as OUT_OF_RANGE instead of -1 is going to be more descriptive, but because most users of onChangeActive are developers, I think developers can advance their work with -1 and onChangeActive can be left as what it is.

Alright, I'll fix it in a follow-up.

@eps1lon Subtle, but nice to fix when there is time to do.

@eps1lon If you don't have further comments about onChangeActive and other things, what about merging the PR?

@sakura90 Thanks for the exploration. I had a look too. It seems that onChangeActive shouldn't be fired on blur/focus. It's off topic but I'm leaving it here in case you want to work on a follow-up.
diff --git a/packages/material-ui/src/Rating/Rating.js b/packages/material-ui/src/Rating/Rating.js
index 55a52464a2..744fea2e97 100644
--- a/packages/material-ui/src/Rating/Rating.js
+++ b/packages/material-ui/src/Rating/Rating.js
@@ -357,10 +357,6 @@ const Rating = React.forwardRef(function Rating(inProps, ref) {
       hover: prev.hover,
       focus: newFocus,
     }));
-
-    if (onChangeActive && focus !== newFocus) {
-      onChangeActive(event, newFocus);
-    }
   };

   const handleBlur = (event) => {
@@ -378,10 +374,6 @@ const Rating = React.forwardRef(function Rating(inProps, ref) {
       hover: prev.hover,
       focus: newFocus,
     }));
-
-    if (onChangeActive && focus !== newFocus) {
-      onChangeActive(event, newFocus);
-    }
   };

   const [emptyValueFocused, setEmptyValueFocused] = React.useState(false);

The change seems to be nice.  But if I am going to make the change, I am going to manually test the change before PR.
I think if it is clearer about things about onChange and onChangeActive, it is going to be better.  onChange can listen to changes of mouse and touch while onChangeActive can listen to hover of only mouse.  If it is clearer that ontouchmove, which is the version of hover for touch, is not supported, developers do not have to try each functionality before using it and have an easier time using the functionality.
I am going to create a new issue, assign it to myself, and work on the issue.

@sakura90 Thanks!


Fix ConcurrentPipeWriterTests

Addresses https://github.com/aspnet/AspNetCore-Internal/issues/2839

This is easier to review with whitespace in diffs disabled.


Audit the set of patches we maintain against upstream

In preparation for the upcoming major version bump from upstream, we should audit our patches to make sure they are surgical, explainable and under test. Patch code is code that we wrote, so it must be code that we understand and test.
Patches:

0001-Allow-disabling-the-partner-name.patch
0002-Add-nil-checks-for-sql-database-instance-flattening.patch
0003-Only-set-bigquery-dataset-selflink-if-available.patch
0004-Add-flattening-of-self-managed-certificates.patch
0005-Remove-security-policy-default-versioned_expr.patch
0006-website-docs-d-tweaks.patch
0007-Significant-docs-changes.patch
0008-Exclude-scripts-dir-from-provider-go-module.patch
0009-Remove-duplicative-resource-token.patch: https://github.com/pulumi/pulumi-gcp/pull/1180

0001-Allow-disabling-the-partner-name.patch
Augments the partner name. The change is straight forward and well scoped. It is not tested.
0002-Add-nil-checks-for-sql-database-instance-flattening.patch
This is harmless, but should not be necessary. If this is necessary, we have a bridge bug. It is not tested.
0003-Only-set-bigquery-dataset-selflink-if-available.patch
CHANGELOG_OLD.md indicates this fixed a panic. There is no test.
0004-Add-flattening-of-self-managed-certificates.patch
Looks like this is a fix for https://github.com/hashicorp/terraform-provider-google/issues/12091. It is not tested.
0005-Remove-security-policy-default-versioned_expr.patch
This removes Default: "" from a field. Presumably this is because it is not compatible with the field validator: ValidateFunc: validation.StringInSlice([]string{"SRC_IPS_V1"}, false). We should confirm that this is still an issue and open a tracking issue in the bridge.
0006-website-docs-d-tweaks.patch
Docs changes. Mostly removing links to the tf registry.
0007-Significant-docs-changes.patch
Lots of docs changes, lots of removed links. We should consider programatically removing these links to cut down on our patch set.
0008-Exclude-scripts-dir-from-provider-go-module.patch
Exclude the scripts package by adding a go.mod file. Apparently there is a reference to a private module.
0009-Remove-duplicative-resource-token.patch
Removes some (presumably) duplicated resources from the sdkv2 provider. We should just back this change out, swapping MuxShimWithDisjointgPF with MuxShimWithPF if necessary.

Thank you! Two questions/asks:

Could you please open issues for each patch, the same we did for AWS?
Out of this list - are there patches that we recommend removing/changing for 7.0? If so, let's have estimates and move those issues to an iteration (M94 if small, M95 if large)

All my recommended changes are small (< 1 day), but they will take time. I have added them to #1183. Since the team is fully booked for this iteration, what should we bump to take these on?

Since there is now a tracking issue for each patch, I'm closing this issue.


Docs for NodePool are misleading

What happened?
The docs for the NodePool resource: https://www.pulumi.com/registry/packages/gcp/api-docs/container/nodepool/
are misleading since the name of a NodePool needs to be all lower-case.
Example
The example yields the following error:
Diagnostics:
  gcp:container:NodePool (primaryPreemptibleNodes):
    error: 1 error occurred:
        * creating urn:pulumi:dev::cluster::gcp:container/nodePool:NodePool::primaryPreemptibleNodes: 1 error occurred:
        * error creating NodePool: googleapi: Error 400: Invalid value for field "node_pool.name": "primaryPreemptibleNodes-b8a97f9". Must be a match of regex '(?:[a-z](?:[-a-z0-9]{0,38}[a-z0-9])?)' (only lowercase alphanumerics and '-' allowed, must start with a letter and end with an alphanumeric, and must be no longer than 40 characters).
    Details:
    [
      {
        "@type": "type.googleapis.com/google.rpc.RequestInfo",
        "requestId": "0x946efa1f93d34437"
      }
    ]
    , badRequest

  pulumi:pulumi:Stack (cluster-dev):
    error: update failed

Output of pulumi about
.
Additional context
No response
Contributing
Vote on this issue by adding a 👍 reaction.
To contribute a fix for this issue, leave a comment (and link to your pull request, if you've opened one already).

This looks like it has been fixed, possibly upstream?


I hope that when the item is selected in the Pane, the "Setting" don't display, and comes with more consistent application of the system!

I hope that when the item is selected in the Pane, the "Setting" don't display, and comes with more consistent application of the system!

What?

Oh, I'm sorry, I come from non-English speaking countries, this is the result of Bing translation. In short, I want SplitView template is more like (animation and detail) Win10 native application (such as Groove App SpiltView)(●'◡'●)
---Thank you for your answer

If you are talking about "Groove music" then the current menu is pretty close to the app, and Groove also has "Settings". The speed of open/close or the place of humbergar icon is different though. When you see "news" app, it is actually more close.


Breaking change to HttpClient - EmptyContent instead of null

Please include breaking change to the response content of HttpClient described here:
https://nicolaiarocci.com/httpresponsemessage.content-is-non-nullable-in-net5/

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

ID: 5f68af3a-3848-48ef-c24b-cf33b880ca1b
Version Independent ID: 580c061d-20e5-248c-a24f-7a48a684c7e3
Content: Breaking changes in .NET 5 - .NET
Content Source: docs/core/compatibility/5.0.md
Product: dotnet-fundamentals
GitHub Login: @gewarren
Microsoft Alias: gewarren

@gyphysics This is actually not a breaking change, at least in the way the .NET team defines breaking changes. Please see https://github.com/dotnet/runtime/pull/35910#issuecomment-912814054.


Upgrade byte-buddy to 1.7.6(Unreleased yet)

This is an important bug, which has been confirmed by @raphw : https://github.com/raphw/byte-buddy/issues/347
We will upgrade our dependency to 1.7.6, after it released.
cc @ascrutae @IluckySi

Ref #501

Done


scummvm: 2.8.1 -> 2.9.0

Fix https://github.com/NixOS/nixpkgs/issues/367943

Things done

Built on platform(s)

[x] x86_64-linux
[x] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
25.05 Release Notes (or backporting 24.11 and 25.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 367970

x86_64-linux

:white_check_mark: 7 packages built:

beneath-a-steel-sky
broken-sword-25
drascula-the-vampire-strikes-back
dreamweb
flight-of-the-amazon-queen
lure-of-the-temptress
scummvm


文件权限的问题

文件的读写权限如何控制？

define your own FsSecurityChecker

Thanks. Like this.
fsService.setSecurityChecker(new FsSecurityCheckForAll());
define my FsSecurityChecker and use it.


Database Error on vendor logged in (me page)

accessing (after having logged in successfully):
http://localhost/vendor/me
gives this error:
A Database Error Occurred
Error Number: 1064

You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'ASC' at line 1

SELECT YEAR(FROM_UNIXTIME(date)) as year, MONTH(FROM_UNIXTIME(date)) as month, COUNT(id) as num FROM vendors_orders WHERE vendor_id = 1 GROUP BY YEAR(FROM_UNIXTIME(date)), MONTH(FROM_UNIXTIME(date)) ASC

Filename: modules/vendor/models/Vendorprofile_model.php

Line Number: 48

The problem occurs on PHP Version 7.4.12 and mysql  Ver 8.0.22 for Linux on x86_64 (Source distribution)
With the exact same data in the database it is working fine with:
PHP Version 7.3.24 and mysql  Ver 15.1 Distrib 5.5.65-MariaDB, for Linux (x86_64) using readline 5.1

@bpiltz You have no "ORDER BY" before the ASC

@kirilkirkov well that is original code from you. Otherwise I would not post it here ;-) No modification from my end on that. But I could debug it probably if you wish.

removed "ASC" now as there is no field within the clause meant to be used for ordering.

@bpiltz
Pull the code. Now the query must works fine.


HTML data in Paleo Tutorial?

I'm trying to follow the tutorial notebook  and don't have the contents to 'project/data/html/' folder. I got the data from the tar file, and placed it in 'project/data/pdf/' Am I doing something wrong?

The tar contains both a pdf and an html folder. Both should be placed in project/data/.

Thanks, I was using the bigger dataset.
Now, my PostgreSQL is exiting in cell 16, line 3 with error code 2- which means no data. I've checked the /tmp/ folder, and all the necessary data is present. Any pointers? I'm not opening a new issue because I'm not sure if this is my mistake.

Thanks, I was using the bigger dataset.

I'm not sure what dataset you are referring to.

Now, my PostgreSQL is exiting in cell 16, line 3 with error code 2- which means no data.

Is there data in your formation_measurement database?


to find first N pure numbers

Find the first N pure numbers
Given an integer N, the task is to print first Nth pure numbers. A number is said to be pure if
It has even number of digits.
All the digits are either 4 or 5.
And the number is a palindrome.
First few pure numbers are 44, 55, 4444, 4554, 5445, 5555, …

taking the issue


Improved en_GB and ro_RO

en_GB

improved postcode generator although not guaranteed to produce valid codes
landline formatter for phone numbers

ro_RO

adding real text

I'll implement the suggestions and split the changes into 2 separate PRs


Advanced Filters Independent of Columns

First of all, thanks for an awesome set of components, and for making the Blazor components free to use. And thank you in advance for considering this possibility.
Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
We have a current application that uses the Yii PHP framework. It has a pretty good Data Grid component, especially considering that we are using version 1.1 from over 10 years ago. But we would like to move to Blazor and Radzen in the future. For the Yii Data Grid, we use filtering that is separate from the columns. Below is a screen shot from our current application.

Describe the solution you'd like
A clear and concise description of what you want to happen.
Ideally we would like to have similar functionality as we have in the Yii application when we move to Radzen. Then we can have additional filters for data that is not shown in the columns, but is part of the dataset that is being partially shown in the data grid. In other words, it would be great to have the advanced filters that are available on the Radzen columns be available independently from the data grid display and yet be used to filter values in the Radzen data grid output.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
We currently have the Yii solution as shown above. and as attached
Additional context
Add any other context or screenshots about the feature request here.

Thanks for the suggestion! This looks like completely new component to me, maybe RadzenFilter that will accept Data and will provide View property that other data components like RadzenDataGrid will be able to consume.

@wakudrle - Maybe not as easy as a built-in control, but this functionality should also be possible by implementing the 'LoadData' method on the grid, along with a custom filter control of your own creation.  The custom filter control could either expose a 'LoadDataArgs' parameter that you could bind, or maybe simpler expose a 'FilterString' bound property that could be added to your query as you're building the current data page.
We often use a pattern such as:
<RadzenDataGrid @ref="dataGrid"
        TItem="SomeDataClass"
        IsLoading=@(__isLoading || LoadingState || !__hasLoaded)
        LoadData=@LoadGridData
        EmptyText="No data available">
...
</RadzenDataGrid>

    // ...
   bool __isLoading = false;
   bool __hasLoaded = false;
   // ... LoadingState comes from a base control that handles pre-assigning grid sort/filters based on user state
   // ... which sometimes causes the LoadGridData to be called 'prematurely'
    protected async Task LoadGridData(LoadDataArgs args)
    {
        if (__isLoading || LoadingState) { return; } // don't do anything here (re-entrant or premature)

        __isLoading = true;
        await Task.Yield(); // allow the loading indicator to render

        var query = dataService.GetBaseQueryable(/* [some args] */);

        // *** This is where you could pull the filter from the external control vs. what the grid provides
        if (!string.IsNullOrWhiteSpace(args.Filter))
        {
            query = query.Where(args.Filter);
        }

        var needsCount = __lastFilter == null || !string.Equals(__lastFilter, args.Filter);
        __lastFilter = args.Filter; // or your custom filter string from above

        var count = needsCount ? query.Count() : dataGrid.Count;

        if (!string.IsNullOrWhiteSpace(args.OrderBy))
        {
            query = query.OrderBy(args.OrderBy);
        }

        if (args.Skip.HasValue) { query = query.Skip(args.Skip.Value); }
        if (args.Top.HasValue) { query = query.Take(args.Top.Value); }

        var listData = await query.ToListAsync();
        dataGrid.Data = listData;
        if (needsCount || listData.Count == 0) // ensure empty text shows up
        {
            dataGrid.Count = count;
        }

        __isLoading = false;
        __hasLoaded = true;
    }

New DataFilter component can be used in such scenarios: https://github.com/radzenhq/radzen-blazor/commit/a53022bac01a52e68fd040233e4abd8ba1d6639d


Widget-based AuthenticationView

Hi guys, here is the AuthenticationView widgets attempt.
OAuth

SSL

User credentials

Client certificate view

@anmacdonald for the UsernamePassword (token auth) dialog and the SSL Unsigned Cert dialog, I think they would benefit from showing the requesting URL in text (don't currently see this in the provided screenshots). It is in the common design, and I believe the Qt Quick implementation has this, so there is good precedent to follow.
For OAuth - do we need to do anything special on the toolkit side to make some of the recent enhancements Guillaume worked on, such as dark mode or localized language work (can't recall if this is done in the API or Toolkit)?

I believe the Qt Quick implementation has this

@anmacdonald The corresponding QML are:

https://github.com/Esri/arcgis-runtime-toolkit-qt/blob/v.next/uitools/import/Esri/ArcGISRuntime/Toolkit/UserCredentialsView.qml#L74
https://github.com/Esri/arcgis-runtime-toolkit-qt/blob/v.next/uitools/import/Esri/ArcGISRuntime/Toolkit/SslHandshakeView.qml#L56

For OAuth - do we need to do anything special on the toolkit side to make some of the recent enhancements Guillaume worked on, such as dark mode or localized language work (can't recall if this is done in the API or Toolkit)?

For dark mode, I don't remember. For localized language, this information is in currentChallengeUrl, then this is fine. Is the C++ test app will be updated with this new widgets? If yes, it's possible to use it to verify if the localisation is correct.


React Native Support

Hi, I can see that this project uses theme-ui for it's UI. I think it can easily be ported to the react-native using theme-ui-native. It can be on a separate repo but that would mean maintaining the same codebase twice.
And because there are no native modules for this to work, it can be used with expo too. There are not that many chat products that work on the expo.

great idea @ysfuu! have you tested it out with theme-ui-native to see if it works?
happy to look into this 😄

I didn't try it but I can't see why it wouldn't work. The differences between the web version and the native version don't seem like a problem for this package.
https://github.com/samjbmason/theme-ui-native#differences-between-theme-ui-for-web-and-theme-ui-for-react-native

hey @ysfuu! we recently added a React Native component if you wanna try it out :) https://github.com/papercups-io/chat-widget-native
(it still needs some tweaks, but you should still be able to play around with it!)


Using a decorator

Thank you for blueprinter ❤ . I am using blueprinter with decorators
The decorator
class UserDecorator < Draper::Decorator
  delegate_all

  def full_name
    "#{user.first_name} #{user.last_name}"
  end

  def address
    "#{user.city} #{user.zipcode} #{user.country}"
  end
end

Right now calling .decorate individually for each field definition works
class UserBlueprint < Blueprinter::Base
  identifier :uuid

  field :full_name do |user|
    user.decorate.full_name
  end

  field :address do |user|
    user.decorate.address
  end
end

Is there a way to do this by calling .decorate only once?

I believe you can decorate your object before passing it to blueprinter. Like:
UserBlueprint.render_as_json(user.decorate)


fid does not match frame

您好，我在用NeRF-DS dataset training的時候發現，每個frame的uid與fid不匹配，如下圖，可以請問一下原因嗎

在as_novel_view的class理論上uid = 54 ，fid應該要等於54 / 845 = 0.064才對，不應該是上面的0.3290

Hi, thanks for your interest.
I found that in NeRF-DS dataset, the uid is not consistent with the actual frame id. So I used the image name as the fid to obtain the right sequence, which is shown in the code here.

可是NeRF-DS dataset 的 fid不是定義在這裡嗎https://github.com/ingra14m/Deformable-3D-Gaussians/blob/main/scene/dataset_readers.py#L459

sry for the wrong link. What I want to say is that the uid in NeRF-DS dataset is not consistent with the actual frame id. That's why in metadata.json, there is a dedicated field defined as time_id.


Error while loading rule 'import/group-exports': rule.create is not a function

I think, due to the export default within group-exports.js, it's causing the following error when I try to lint my JavaScript:
Error while loading rule 'import/group-exports': rule.create is not a function

because it compiles it to exports.default = {}; instead of module.exports = {.

Whoops. Yeah that needs fixing.

@ljharb could this get released? it looks like the latest release was published the day before this fix.


Code not working for pixel level random erasing

Hi there,
I'm r eally happy with this code and the function, in order to deal with occlusion in my dataset. Howvwer, I can get the random_eraser function to work to produce the black boxes, but not with the randomised pixel from the image itself.
Here is the code I used:
TRAIN_DIR = 'F:/all_species_cropped_balanced/train'

BATCH_SIZE = 32 #was 32 before
IMG_HEIGHT = 224
IMG_WIDTH = 224

#copied from this repo

def get_random_eraser(p=0.5, s_l=0.02, s_h=0.4, r_1=0.3, r_2=1/0.3, v_l=0, v_h=255, pixel_level=True): # was False before, as #true still doesn't help me do the pixel level implementation
    def eraser(input_img):
        if input_img.ndim == 3:
            img_h, img_w, img_c = input_img.shape
        elif input_img.ndim == 2:
            img_h, img_w = input_img.shape

        p_1 = np.random.rand()

        if p_1 > p:
            return input_img

        while True:
            s = np.random.uniform(s_l, s_h) * img_h * img_w
            r = np.random.uniform(r_1, r_2)
            w = int(np.sqrt(s / r))
            h = int(np.sqrt(s * r))
            left = np.random.randint(0, img_w)
            top = np.random.randint(0, img_h)

            if left + w <= img_w and top + h <= img_h:
                break

        if pixel_level:
            if input_img.ndim == 3:
                c = np.random.uniform(v_l, v_h, (h, w, img_c))
            if input_img.ndim == 2:
                c = np.random.uniform(v_l, v_h, (h, w))
        else:
            c = np.random.uniform(v_l, v_h)

        input_img[top:top + h, left:left + w] = c

        return input_img

    return eraser

datagen = ImageDataGenerator(
    rescale=1/255,
    preprocessing_function=get_random_eraser(v_l=0, v_h=1, pixel_level=True))

train_gen = datagen.flow_from_directory(
    TRAIN_DIR,
    target_size=(IMG_HEIGHT, IMG_WIDTH),
    batch_size=BATCH_SIZE
)

When I plot 5 images from the data set, they still have these black boxes, even though I have set pixel_level=True (see image attached)
Does anyone have any ideas? I would be very grateful!

Hi @Sicily-F ,
You're rescaling twice your data. Firstly, with rescale=1./255 and second time in with v_h=1 when calling random erasing. Use default v_h=255.
The following code will work
`
datagen = ImageDataGenerator(
rescale=1/255,
preprocessing_function=get_random_eraser(v_l=0, v_h=255, pixel_level=True))
`
Best regards,

Hello! Ah ok, that makes so much sense, I didn't pick up on that. THANK YOU! Success


All-in-one docker image: SAMPLING_STRATEGIES_FILE env var can't be used (without args overwrite)

Describe the bug
I'm using the all-in-one docker image in a Kubernetes deployment and want to set a sampling-strategies file.
The obvious thing (to me) seems to be setting the SAMPLING_STRATEGIES_FILE  env var in the K8s deployment descriptor.
But that doesn't work - still the default /etc/jaeger/sampling_strategies.json file gets used. And that is not even obvious as there isn't a log output giving info about which file is used (see #2197).
Only a look at the all-in-one dockerfile
https://github.com/jaegertracing/jaeger/blob/c991a00d5a1a34a5ba92e1d5af9d438a43f6e70c/cmd/all-in-one/Dockerfile#L67
reveals that the sampling.strategies-file param is set there - obviously having precedence.
So, I would either need to set args: [""] (in the K8s deployment description spec.template.spec.containers section) in order to be able to use the SAMPLING_STRATEGIES_FILE  env var.
Or, I would have to use args: ["--sampling.strategies-file=/path/my_sampling_strategies.json"]. But that means I have to check the all-in-one dockerfile to see what other parameters I'm possibly overwriting with this.
Suggested fix
To me it seems that things would be easier, more intuitive and less error-prone if the all-in-one image docker file would set a SAMPLING_STRATEGIES_FILE  env var with the default sampling-strategies file instead of using the parameter in the CMD entry.
Then a SAMPLING_STRATEGIES_FILE env var in the K8s deployment descriptor can override the default. Alternatively, using args: ["--sampling.strategies-file=/path/my_sampling_strategies.json"] in the K8s deployment descriptor instead should still work, the command line parameter having precedence then.
To Reproduce
Steps to reproduce the behavior:

Create a K8s deployment with the all-in-one image, defining a none-existing strategies file via env var, which should cause an error on startup

 env:
  - name: SAMPLING_STRATEGIES_FILE
    value: "none-existing-sampling-strategies.json"

See that there is no error on pod startup, meaning that setting SAMPLING_STRATEGIES_FILE has had no effect.
add args: [""] on the same level as env and update deployment to see pod startup failure - meaning this time the env var got actually used.

Expected behavior
SAMPLING_STRATEGIES_FILE env var can be used in the K8s deployment description without further required changes.
Version (please complete the following information):

OS: Linux
Jaeger version: 1.22
Deployment: Kubernetes

+1, would you like to submit a fix?

So, I would either need to set args: [""] (in the K8s deployment description spec.template.spec.containers section) in order to be able to use the SAMPLING_STRATEGIES_FILE env var.
Or, I would have to use args: ["--sampling.strategies-file=/path/my_sampling_strategies.json"]. But that means I have to check the all-in-one dockerfile to see what other parameters I'm possibly overwriting with this.

Or you could mount your strategies file in the place that is spefied in the CMD?

To me it seems that things would be easier, more intuitive and less error-prone if the all-in-one image docker file would set a SAMPLING_STRATEGIES_FILE env var with the default sampling-strategies file instead of using the parameter in the CMD entry.

Sounds good to me.

Then a SAMPLING_STRATEGIES_FILE env var in the K8s deployment descriptor can override the default. Alternatively, using args: ["--sampling.strategies-file=/path/my_sampling_strategies.json"] in the K8s deployment descriptor instead should still work, the command line parameter having precedence then.

I think we don't actually need this file to begin with... Perhaps it should just be removed from the Docker image?

I think we don't actually need this setting/file in the container to begin with... Perhaps it should just be removed from the Docker image?

+1

$ m cmd/all-in-one/sampling_strategies.json
{
  "default_strategy": {
    "type": "probabilistic",
    "param": 1
  }
}

The file tells all SDKs connected to all-in-one to use 100% sampling. There's no other way afaik to configure this on the backend. I assume without this file the default would be something like 0.001 (we could introduce a param to control that value without a file)

Is this also the default only for all-in-one?

Or simply keep the file in the image but make its usage optional, i.e. do not use the --sampling.strategies-file parameter by default but let the user decide to set it (or not)

It's the default for the SDKs using remote sampler.
all-in-one itself is internally traced with 100% (can be overridden via env)
func initTracer(metricsFactory metrics.Factory, logger *zap.Logger) io.Closer {
        traceCfg := &jaegerClientConfig.Configuration{
                ServiceName: "jaeger-query",
                Sampler: &jaegerClientConfig.SamplerConfig{
                        Type:  "const",
                        Param: 1.0,
                },
                RPCMetrics: true,
        }

Sorry, I meant to ask whether remote samplers getting the strategy from an all-in-one would get the same strategy as from a jaeger-collector. Are they both serving the same strategies? If so, shouldn't we be changing the default to 1 instead?

yes, it's the same remote sampling mechanism.

shouldn't we be changing the default to 1 instead?

I think that would be a dangerous breaking change.

So this means the strategies file is needed for now, and we can stick to defining the env var in the Dockerfile, instead of using the CMD parameter, correct?
ENV SAMPLING_STRATEGIES_FILE=/etc/jaeger/sampling_strategies.json

I would gladly provide a PR but since getting company approval for contributing here would take a while and since it's such a small change, I would appreciate it if someone else could do it.


Shard deadlock on SHUTTING_DOWN state

Posting as an issue on @MinnDevelopment request.
There's a (rare) issue on shard shutdown which causes some shards to get stuck on SHUTTING_DOWN.
The cause is a deadlock caused by JDA trying to put AND get from a TSynchronizedLongObjectMap at the exact same time (MainWS-ReadThread and MainWS-WriteThread) as seen on the dump, with one waiting to lock the other.
Highlight: https://im-ded-but.mantaro-is.life/90o8563-588.jpg as seen on here
{Thread dump file}
Important output:
Found one Java-level deadlock:
=============================
"JDA [33 / 51] MainWS-ReadThread":
  waiting to lock monitor 0x00007fc4640248a8 (object 0x00000001f77083d0, a gnu.trove.impl.sync.TSynchronizedLongObjectMap),
  which is held by "JDA [33 / 51] MainWS-Sending Thread"
"JDA [33 / 51] MainWS-Sending Thread":
  waiting to lock monitor 0x00007fc5b409c778 (object 0x00000001f778b698, a gnu.trove.impl.sync.TSynchronizedLongObjectMap),
  which is held by "JDA [33 / 51] MainWS-ReadThread"

Important stacktrace data: https://hastebin.com/akazulilih.sql
Status = https://cdn.discordapp.com/attachments/175599601942921216/344188294567034889/unknown.png (Was like this for over 2 minutes until the watcher rebooted it forcefully)
Running JDA version related to: https://github.com/DV8FromTheWorld/JDA/pull/410 on hash 46802e1 on Minn's request.
Note: Will post more data as I get it.

Fixed as of 6c7e405


Can't get charging state to show

Hello,
I have this YAML to embed the battery state card into my dashboard:
type: custom:battery-state-card
title: Ladestand
entities:
  - entity: sensor.handy_florian_battery_level
    charging-state:
      entity_id: binary_sensor.handy_florian_is_charging
      icon: mdi:flash
      state: 'on'

Display of battery state works fine, but charging state is ignored, i.e., no visual indication. The entity binary_sensor.handy_florian_is_charging is correct (copy & pasted it directly into the state overview) and is verified on when charging.
Anything I am missing here?
Thanks!

Can you please paste the screenshot of the *_is_charging entity listing (wrist state and attributes) from developer tools

Installed the card the other day and can confirm that there is no charging indication on my battery icon, either.
Battery state is accurate and the charging state is being reported in the entity, it just doesn't alter the representation of the battery state icon.

@floli there is a mistake on your config in charging_state should be underscore not dash

@floli there is a mistake on your config in charging_state should be underscore not dash

Ar, now it works:
entities:
  - entity: sensor.handy_florian_battery_level
    charging_state:
      entity_id: binary_sensor.handy_florian_is_charging
      state: 'on'

@OddSquirrel Do you have similar fault? For me, that issue can be closed, problem was between keyboard and chair.


Cookbook no longer runs xcode 14

macOS Version(s) Used to Build
macOS 12 Monterey
Xcode Version(s)
Xcode 14
Description
Error is:
/Cookbook/Cookbook/Cookbook.xcodeproj Missing package product 'CookbookCommon'
It expects that cookbookcommon is a package that should be added rather than integrated with the rest of Audiokit
Crash Logs, Screenshots or Other Attachments (if applicable)
No response

Try File > Packages > Reset Package Caches.


Add countdown clock for next releases

It would be great to have something like https://www.chromestatus.com/features/schedule
https://deno.land/manual@main/contributing/release_schedule is rather buried

@ry of course it's a draft, but could you share your thoughts ?

I changed my mind on this - it's too much work for us to maintain "next release" dates
https://github.com/denoland/dotland/pull/2127#discussion_r862831285


Include terms from GitLab

To merge the NESTOR database into the main HBP branch on GitLab, we need to:

[x]  add the remaining terms that were used there to this repository
[ ] make a new export
[ ] update the references in the BEL files

New export: https://github.com/pharmacome/terminology/commit/77c99fe195ccf112eb19f3d9f4add983d9ad58da
Commit SHA-512: 77c99fe195ccf112eb19f3d9f4add983d9ad58da


Create user supplied root directory on s3 bucket if it does not exist

Describe your changes and why you are making these changes
Create user supplied root directory (path) on s3 storage bucket if one does not exist. This object path is used for storing Aqueduct workflow artifacts.
Related issue number (if any)
ENG-1269
Loom demo (if any)
Checklist before requesting a review

[ x] I have created a descriptive PR title. The PR title should complete the sentence "This PR...".
[ x] I have performed a self-review of my code.
[ ] I have included a small demo of the changes. For the UI, this would be a screenshot or a Loom video.
[ ] If this is a new feature, I have added unit tests and integration tests.
[x ] I have run the integration tests locally and they are passing.
[x] I have run the linter script locally (See python3 scripts/run_linters.py -h for usage).
[x ] All features on the UI continue to work correctly.
[ ] Added one of the following CI labels:

run_integration_test: Runs integration tests
skip_integration_test: Skips integration tests (Should be used when changes are ONLY documentation/UI)

Ah yes good question, so you'll want to take a look at setup_storage_layer() in setup_integration.py. The semantics I'd want is for the user to be able to define a section in test-config.yml like:
storage:
    test_s3:
          root_dir: path/to/dir  # this is currently the unsupported part.

This will involve fetching information from both the test-credentials.yml and test-config.yml in that setup_storage_layer() method.

Ah yes good question, so you'll want to take a look at setup_storage_layer() in setup_integration.py. The semantics I'd want is for the user to be able to define a section in test-config.yml like:
storage:
    test_s3:
          root_dir: path/to/dir  # this is currently the unsupported part.

This will involve fetching information from both the test-credentials.yml and test-config.yml in that setup_storage_layer() method.

Can I create a sub issue for this and fix that separately?

Ah yes good question, so you'll want to take a look at setup_storage_layer() in setup_integration.py. The semantics I'd want is for the user to be able to define a section in test-config.yml like:
storage:
    test_s3:
          root_dir: path/to/dir  # this is currently the unsupported part.

This will involve fetching information from both the test-credentials.yml and test-config.yml in that setup_storage_layer() method.

done

Thanks! Sorry about the confusion.

NP Thanks. Will merge once integration test passes. I have opened a new issue for tracking the integration test, we can chat how we go about doing that.


KARAF-3727 - Cave generates the repository.xml with "bad" path on Windows

R: @jbonofre

Merged. Can you close this PR please ?


TypeError: must start with buffer, array, number or string

Guys, I've spoken with you in gitter about this bug I'm dealing with.
My test suites run with some faults to the end as expected but the Allure crashes with a message like:

This happens when I use this lib.
webdriverajax
My code that causes crash is similar to this one, but the assertion fails!
browser
  .url('http://foo.bar')
  .setupInterceptor() // capture ajax calls
  .expectRequest('GET', '/api/foo', 200) // expect GET request to /api/foo with 200 statusCode
  .click('#button') // button that initiates ajax request
  .pause(1000) // maybe wait a bit until request is finished
  .assertRequests(); // validate the requests

Hope you discover what's wrong :)

@sullenor You have got something like this, maybe you have something to add.

@just-boris Yeah, I had the same issue with webdriver and wdio-allure-reporter. Recently, I started to get this exception more often.
I'd try to make a small example to reproduce this issue, may be it will be easier to fix it.

Looks like I get this error in case allure can't find the file with screenshot. Despite that screenshot was successfully generated. Presumably, it can't resolve path to file.

I tried to log filename here: https://github.com/webdriverio/webdriverio/blob/master/lib/runner.js#L91 and looks like it generates different paths: absolute in one case and only filename in another.
ERROR_firefox_2016-04-05T14-44-08.997Z.png
Trace
    at EventEmitter.<anonymous> (/home/sullenor/yharnam/node_modules/webdriverio/build/lib/runner.js:121:41)
    at emitOne (events.js:77:13)
    at EventEmitter.emit (events.js:169:7)
    at Object.<anonymous> (/home/sullenor/yharnam/node_modules/webdriverio/build/lib/webdriverio.js:650:55)
    at Object.exec (/home/sullenor/yharnam/node_modules/webdriverio/build/lib/helpers/safeExecute.js:28:24)
    at Object.resolve (/home/sullenor/yharnam/node_modules/webdriverio/build/lib/webdriverio.js:171:29)
    at /home/sullenor/yharnam/node_modules/webdriverio/build/lib/webdriverio.js:384:32
    at Promise.apply (/home/sullenor/yharnam/node_modules/webdriverio/node_modules/q/q.js:1151:26)
    at Promise.promise.promiseDispatch (/home/sullenor/yharnam/node_modules/webdriverio/node_modules/q/q.js:774:41)
    at /home/sullenor/yharnam/node_modules/webdriverio/node_modules/q/q.js:1377:14

/home/sullenor/yharnam/errorShots/ERROR_firefox_2016-04-05T14-44-08.997Z.png
Trace
    at EventEmitter.<anonymous> (/home/sullenor/yharnam/node_modules/webdriverio/build/lib/runner.js:121:41)
    at emitOne (events.js:77:13)
    at EventEmitter.emit (events.js:169:7)
    at Object.<anonymous> (/home/sullenor/yharnam/node_modules/webdriverio/build/lib/webdriverio.js:650:55)
    at Object.exec (/home/sullenor/yharnam/node_modules/webdriverio/build/lib/helpers/safeExecute.js:28:24)
    at Object.resolve (/home/sullenor/yharnam/node_modules/webdriverio/build/lib/webdriverio.js:171:29)
    at /home/sullenor/yharnam/node_modules/webdriverio/build/lib/webdriverio.js:384:32
    at Promise.apply (/home/sullenor/yharnam/node_modules/webdriverio/node_modules/q/q.js:1151:26)
    at Promise.promise.promiseDispatch (/home/sullenor/yharnam/node_modules/webdriverio/node_modules/q/q.js:774:41)
    at /home/sullenor/yharnam/node_modules/webdriverio/node_modules/q/q.js:1377:14

Presumably, the problem lies somewhere here: https://github.com/webdriverio/webdriverio/blob/master/lib/webdriverio.js#L218
Reporter should fetch the content of file before attaching it: https://github.com/webdriverio/wdio-allure-reporter/blob/master/lib/reporter.js#L162

I also face this issue a lot. When an element does not exist the payload becomes empty:
Failed parsing stats for Allure reporter: must start with number, buffer, array or string. Dumped this.baseReporter.stats to C:\Users\adm
TypeError: must start with number, buffer, array or string
    at fromObject (buffer.js:140:11)
    at new Buffer (buffer.js:62:10)
    at C:\Users\admin\AppData\Roaming\npm\node_modules\wdio-allure-reporter\build\reporter.js:286:131
    at Array.map (native)
    at AllureReporter.addOutput (C:\Users\admin\AppData\Roaming\npm\node_modules\wdio-allure-reporter\build\reporter.js:257:20)
    at AllureReporter.parseStats (C:\Users\admin\AppData\Roaming\npm\node_modules\wdio-allure-reporter\build\reporter.js:153:50)
    at AllureReporter.<anonymous> (C:\Users\admin\AppData\Roaming\npm\node_modules\wdio-allure-reporter\build\reporter.js:68:12)
    at emitOne (events.js:90:13)
    at AllureReporter.emit (events.js:182:7)
    at BaseReporter.handleEvent (C:\Users\admin\AppData\Roaming\npm\node_modules\webdriverio\build\lib\utils\BaseReporter.js:302:35)

if event.payload.data is empty, a quick dirty fix is to check and assign an empty string:
new Buffer(event.payload.data || '', 'base64')

@eben0 @mDomingues can you please retry with the recent published version?

So far it seems fixed in version 1.4.23.
Good job!
On Jun 24, 2016 7:45 AM, "Christian Bromann"<EMAIL_ADDRESS>wrote:

@eben0 https://github.com/eben0 @mDomingues
https://github.com/mDomingues can you please retry with the recent
published version?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/webdriverio/wdio-allure-reporter/issues/7#issuecomment-228270355,
or mute the thread
https://github.com/notifications/unsubscribe/AKhNxsDOwYlW9NcHabWcpKeHlFi05Ag1ks5qO3z-gaJpZM4H_U3D
.

Cool! Closing then


Add support for Dockerfiles

Add basic support for Dockerfiles. It shows syntax highlighting for reserved words and comments when working with Dockerfiles and nano.

I don't know how I missed this!  I must have been on a hiatus from the internet then.  Anyway, looks good! Merging!

Why was this merged? There was already a Dockerfile nanorc. Now, this repo has two files that have the same content, but one of them is named Dockerfile.nanorc (the "old" one) and the other dockerfile.nanorc (the "new" one). On case-sensitive OSes, this confuses Git and other tools.
Please, revert this PR.

Thank you for bringing this up @kytta. Done.


TOMEE-2930 - MoviesArquillianHtmlUnitTest -> Invalid byte 2 of 2-byte UTF-8 sequence. error.

What does this PR do?
As discussed on the mailing list [3], we found test errors in MoviesArquillianHtmlUnitTest.
While trying to reproduce it on my Linux machine, I found, that the dependencies used for setting up the unit tests do not match with the current project version.
CI does not cover this as it can retrieve the old snapshot artifacts (either from local disk cache or from an apache snapshot repo).
With the version adjustments, the tests can be executed and do not throw the exception mentioned above
Question, @cesarhernandezgt : Can we run this PR on the CI environment to check, if the issue persists?
References

[1] https://issues.apache.org/jira/browse/TOMEE-2930
[2] https://ci-builds.apache.org/job/Tomee/job/tomee-7.0.x/org.superbiz$moviefun-functional-test/3/testReport/junit/org.superbiz.moviefun/MoviesArquillianHtmlUnitTest/org_superbiz_moviefun_MoviesArquillianHtmlUnitTest/
[3] https://openejb.markmail.org/search/?q=type:development#query:type%3Adevelopment+page:1+mid:yagkenymmvojwqvh+state:results

@rzo1 what about adding in surefire plugin the system property "project.version" and just do a System.getProperty("project.version", "default value SNAPSHOT") in the code? will stay inlined this way

@rzo1 what about adding in surefire plugin the system property "project.version" and just do a System.getProperty("project.version", "default value SNAPSHOT") in the code? will stay inlined this way

Good idea - will resolve such issues in the future. Will update.


Text background always black

Hello
I'm getting some strange behaviour on some of my computers and not on others. I don't know what the difference is, though. On the broken systems, the cells that have characters in them always have black background, no matter what color is set in the configuration.
The way to reproduce:

Start with the default configuration file
Set the colors.primary.background to (for example) '0xff0000'
Start alacritty and type something ‒ for example ls -l

What I get (on some computers) is red background on the blank cells and black background with the occupied ones. See the attached screenshot.
What I think I should get is red background even in the occupied cells.
Any idea what may be the cause or why it happens on some machines and not on others? Something I can try? Is it possible there's some cache that is broken?
Which operating system does the issue occur on?
Linux basilisk 4.11.0-basilisk #1 SMP Tue May 9 20:58:27 CEST 2017 x86_64 Intel(R) Celeron(R) CPU N2940 @ 1.83GHz GenuineIntel GNU/Linux

If on linux, are you using X11 or Wayland?
X11.

Could you please run Alacritty with -vvv, repro this, and share the log? This seems more likely to be an issue with your shell than Alacritty.

Here's the log.
Anyway, I'm not sure about the shell. I have the exact same setup on all the computers, even the ones that work. And I get the problems even when I run eg. vim inside alacritty without shell (alacritty -e vim).
log.txt

Here's a log from when it works, for comparison. To me it looks pretty much the same ‒ the background colour is never set, only the foreground ones and resets.
ok.txt

Hmm, I found the problem. I had an older copy of the alacritty binary from before and it was sooner in the path. My guess is the old binary didn't run well with new shaders, or something like that.
I must therefore apologize for the invalid report.


Giftcard in Cart is missing

There should be mutations and Cart model for appliedGiftCards

I am getting this error when trying to add appliedGiftCards in Cart.
OperationException(linkException: null, graphqlErrors: [GraphQLError(message: Field 'appliedGiftCards' doesn't exist on type 'Cart', locations: [ErrorLocation(line: 62, column: 7)], path: [mutation cartCreate, cartCreate, cart, appliedGiftCards], extensions: {code: undefinedField, typeName: Cart, fieldName: appliedGiftCards})])
That means appliedGiftCards is not available in cart


[Direct3D12][UWP]

The .NET native tool chain for UWP apps cannot handle D3D12, it seems.

Create a new project, of type "Blank App (Universal Windows)", C#
Add the NuGet package "SharpDX.Direct3D12" (3.0.0-beta01) to the project
Rebuild project as Debug. This will succeed, and the app can be run
Switch to Release mode, then rebuild again
The compilation will fail with the following output:

1>  Starting .NET Native compilation
1>  Processing application code
1>  Computing application closure and generating interop code
1>C:\Program Files (x86)\MSBuild\Microsoft\.NetNative\x64\ilc\IlcInternals.targets(886,5): warning :           Unresolved P/Invoke method 'D3D12GetDebugInterface_!d3d12.dll' in assembly 'SharpDX.Direct3D12, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b4dcf0f35e5521f1' because it is not available in UWP applications. Please either use an another API , or use [DllImport(ExactSpelling=true) to indicate that you understand the implications of using non-UWP application APIs.
1>  Generating serialization code
1>  Compiling interop code
1>  Cleaning up unreferenced code
1>  Generating native code
1>C:\Program Files (x86)\MSBuild\Microsoft\.NetNative\x64\ilc\IlcInternals.targets(1231,5): error : Error: NUTC300D:Internal Compiler Error: Type '$81_SharpDX.Direct3D12.RootSignatureDescription' is not valid for use outside of the managed environment, but is passed to native code while compiling method 'instance $81_SharpDX.Direct3D12.RootSignatureDescription $81_SharpDX.Direct3D12.RootSignatureDeserializer.GetRootSignatureDescription()'.
1>C:\Program Files (x86)\MSBuild\Microsoft\.NetNative\x64\ilc\IlcInternals.targets(1231,5): error : ILT0005: 'C:\Program Files (x86)\MSBuild\Microsoft\.NetNative\x64\ilc\Tools\nutc_driver.exe @"c:\users\zero\documents\visual studio 2015\Projects\D12Test\D12Test\obj\x64\Release\ilc\intermediate\nutcargs.rsp"' returned exit code 1

It does not make a difference whether x86 or x64. I have no problems with D3D11. I do have the latest UWP tools 1.1.1 update installed.

xoof What can be the issue from your point of view? Maybe I can the submit PR

Well the marshaler for this method is a bit hacky and seems not supported well by .NET native tool. Haven't dig into it...

Oh thank you for your detailed explanation.

Hello! I work on the .NET Native runtime and compiler team. When you do get time to investigate this a bit please don't hesitate to shoot us a mail at<EMAIL_ADDRESS>We'd be happy to help leverage our expertise so that sharpdx is in great shape for both our customers!

Any news on this issue - I just finished porting UWP + Xaml + DirectX 12 samples to C# and cant get my app running on xbox one due to this issue...

Hi,
Apart for merging PR, I don't have any more spare time left for digging into SharpDX issues, so I would really appreciate someone that is using SharpDX and requires this feature to dig into it, with the Microsoft .NET Native team.
Thanks

@jxl98c Can you send a mail to us .NET folks? It's a bit silly but it is the fastest way to get these things prioritized on our side. Squeaky wheels and all that. :-)

Curiosity got the better of me. Here's the error I get when compiling with the latest tools:
NUTC300D:Internal Compiler Error: Type '$81_SharpDX.Direct3D12.RootSignatureDescription' is not valid for use outside of the managed environment, but is passed to native code while compiling method 'instance $81_SharpDX.Direct3D12.RootSignatureDescription $81_SharpDX.Direct3D12.RootSignatureDeserializer.GetRootSignatureDescription()'.	App7	C:\Program Files (x86)\MSBuild\Microsoft.NetNative\x86\ilc\IlcInternals.targets	1297
I'll see what more information I can dig up but sending a mail really does grease the wheels for us.

Thanks @MattWhilden for bringing this up! I'm also from .NET native team and I've just took a quick look. It appears that this method is doing a calli:
IL_0018: calli $81_SharpDX.Direct3D12.RootSignatureDescription(System.Void*)

While RootSignatureDescription is an auto layout class:
.class public auto ansi sealed beforefieldinit $81_SharpDX.Direct3D12.RootSignatureDescription
extends System.Object
This means that CLR would fallback to use COM interop and marshal this one as a IUnknown, and the problem is that interop marshaling through calli instruction is not yet implemented in .NET native. However, it's not clear to me if this is intentional - I would expect SharpDX to do its own COM marshaling (as far as I can tell - but I could also be terribly wrong).
@xoofx You mentioned that this method is "a bit hacky". Can you share more details on what the method is trying to do?

Thanks @yizhang82 and @MattWhilden ! Good catch, the method RootSignatureDeserializer.GetRootSignatureDescription is indeed automatically generated but in fact has to be rewritten manually, as the direct marshalling of this RootSignatureDescription is not supported.
There is a code to serialize it in RootSignatureDescription.Serialize. The solution to solve this issue is to develop the equivalent deserialize in RootSignatureDeserializer
So as I said above, I'm no longer able to give spare time to SharpDX, so I would appreciate someone that needs this fix to help on this, PR are welcome. Thanks!

@xoofx I spent some time yesterday on a potential PR (https://github.com/sharpdx/SharpDX/pull/773)  and I haven't got a chance to test it yet (appreciate if you can share some ideas on how to test this. Presumably I just need to find a way to test this API without having to create a game :) ). Can you take a quick look and see if it is on the right direction?

@zerppa The PR just got merged into master last night. Feel free to pick up the latest version of SharpDX and give it a try. I've confirmed that picking up this version fixes the problem and .NET native compiler is happy now.


A new file has been added to ./student-bios

Checklists
General Checks

[x] the branch is up to date with main/master
[x] the code works when pulled and run locally
[ ] All CI checks pass
[ ] all conflicts are resolved (if any)
[x] PR has a descriptive title
[x] PR has appropriate labels and milestones for easy identification
[x] PR it is assigned to the owner
[x] reviewers are assigned
[x] the PR contributes only one focused change
[ ] It is in the appropriate column in the project board (if necessary)
[x] has short and clear description
[x] is linked to an issue (if it is related)
[ ] feedback is addressed (if any and if it is appropriate feedback.)

Markdown

[x] the markdown source is formatted
[x] spelling and grammar is correct in all text
[x] The markdown looks correct when you preview the file
[x] all links and images work
[x] any embedded code snippets are formatted

@minjupgeorge looks like you haven't run npm run format yet

@minjupgeorge looks like you haven't run npm run format yet

please move nature.jpg to the student-bios/assets folder and you are done! good job!

Done.!


[BUG]: Unable to Reduce Quantities for Multiple Items in Shopping Cart

Bug Description
When users select multiple quantities of a single item in the shopping cart and later attempt to reduce the quantity for one or more of those items, there is no option available to do so. The system does not provide a way to decrease the quantity of individual items once they have been added to the cart in multiples
Expected Behavior:
Users should be able to easily decrease the quantity of individual items in the shopping cart when multiple quantities have been selected.
Actual Behavior:
There is no option available to reduce the quantity of individual items in the shopping cart once multiple quantities have been selected.

Please assign me the issue , I can solve it

Hey, can i work on this issue...

@bj-jiwrajka Nice observations !
what i want you to do

shift range bar from products to cart window
allowing users to change product quantity from there
while also ensuring everything works just like before


But you can't change that now unfortunately I don't think.


2.7 alias fix

Fix https://github.com/pydantic/pydantic/issues/9216
Fix https://github.com/pydantic/pydantic/issues/9220
Initially introduced in https://github.com/pydantic/pydantic/pull/9144 because we forgot to account for AliasPath and AliasChoices validation aliases!
Selected Reviewer: @adriangb

Please review

One potential issue with this approach — the following code errors for normal validation:
from pydantic import BaseModel, AliasPath, Field

class MyModel(BaseModel):
    x: str = Field(alias=AliasPath('y', 1))

print(MyModel(y=['abc', 'def']))

print(MyModel(y='abc'))
"""
y.1
  Field required [type=missing, input_value={'y': 'abc'}, input_type=dict]
"""

However, with your current implementation, if you use model_construct I believe you'd end up with:
from pydantic import BaseModel, AliasPath, Field

class MyModel(BaseModel):
    x: str = Field(default='abc', alias=AliasPath('y', 1))

print(MyModel.model_construct(y=['abc', 'def']))
#> x='def'  (as expected)

print(MyModel.model_construct(y='abc'))
#> x='b'  (not expected)

Just added an isinstance(v, str) check to prevent that behavior, which now results in:
from pydantic import BaseModel, AliasPath, Field

class MyModel(BaseModel):
    x: str = Field(default='xyz', validation_alias=AliasPath('y', 1))

print(MyModel.model_construct(y=['abc', 'def']))
#> x='def'  (as expected)

print(MyModel.model_construct(y='abc'))
#> x='xyz'  (as expected)


Improving Ticks and Legends

This PR will contain work for improving ticks and legends.

This Fixes #51
Black border added to legends and border width is set to 2.


After creating an ehpa with a behavior, synchronize the behavior to its corresponding hpa

What type of PR is this?
fix bug
What this PR does / why we need it:
When the behavior of ehpa is not nil, the behavior of ehpa should be synchronized instead of the behavior of hpa.
Which issue(s) this PR fixes:

Fixes #
https://github.com/gocrane/crane/issues/310
Special notes for your reviewer:

/LGTM


Password Protected Sharing Credential Prompt

What's the Problem?
Context:
This is my first time using github, so please excuse any mistakes I make in this request.
I use a windows 10 NAS server as a long-term storage and as a media server. I turned on "Password protected sharing" 'cus I share it with my friends.
Problem:
When I try to access the device with the Files app, it doesn't show anything- not even a prompt asking for my credentials. I had to use windows explorer to get the login screen. Once I logged in on windows explorer though, Files is able to see all the folders.
I also tried using "Connect with Remote Desktop Connection" to login but it doesn't give access to the folders.
Just a minor issue that prevents me from turning on the "Default file manager" option.
Solution/Idea
Add support for password protected network locations.
Alternatives
I haven't explored any alternatives.
Priorities
| Ask for credentials when clicking a network device that has password protected sharing | Must |
| - | Should |
| Have an option to require entering credentials again after exiting the app or a set time to expire | Could |
| Saving credentials even after restart | Won't |
Files Version
Version <IP_ADDRESS>
Windows Version
Windows 10 Pro 21H2  19044.1526
Comments
No response

Merging with #7205


Zabit (SIGKILL)

Hello,
I'm migrating from one account to another that's on google. I understand that google has its limits, but why am I ending up with a "Killed (SIGKILL)" error.
imap-backup migrate<EMAIL_ADDRESS><EMAIL_ADDRESS>--destination-prefix=xxxx
WARN -- : [xxxx/Drafts] uid: 362975 (145/159) - append error: Message too large. https://support.google.com/mail/answer/6584#limit
WARN -- : [xxxx/Drafts] uid: 372731 (155/159) - append error: Message too large. https://support.google.com/mail/answer/6584#limit
Zabit (SIGKILL)
Thanks for the help.

Hi @michaldejmek
I imagine that the SIGKILL is caused by an out of memory condition.
Is this backup running on a small device, e.g. a Raspberry Pi?

Hi @joeyates
it's running in the virus server. 4 CPU / 4GB RAM

How big are the messages that are filing to migrate?

Ordinary emails occasionally some attachment .. When creating a backup it did not do.  Now I've run it with the  --verbose command

@joeyates
The application only allocates memory but does not release it incrementally :(

@michaldejmek
The application relinquishes memory occasionally, based on the default rules of the Ruby version you are using.
Are you able to ascertain how big these problematic messages are?
You have ≈1GB swap and ≈4GB of RAM. I think the best fix in this case is to increase the amount of swap available.

@joeyates  Created a swap of 80GB the clipboard has around 65GB.
ruby 3.1.2p20 (2022-04-12 revision 4491bb740a) [x86_64-linux-gnu]
W, [2023-09-30T01:40:06.508228 #74602]  WARN -- : [xx-xx-xx-xx/INBOX.Archives/2011] uid: 12180 (10661/10661) - append error: closed stream
/usr/lib/ruby/3.1.0/openssl/buffering.rb:354:in syswrite': closed stream (IOError)         from /usr/lib/ruby/3.1.0/openssl/buffering.rb:354:in do_write'
from /usr/lib/ruby/3.1.0/openssl/buffering.rb:453:in print'         from /var/lib/gems/3.1.0/gems/net-imap-0.3.7/lib/net/imap.rb:2239:in put_string'
from /var/lib/gems/3.1.0/gems/net-imap-0.3.7/lib/net/imap.rb:2211:in block in send_command'         from /usr/lib/ruby/3.1.0/monitor.rb:202:in synchronize'
from /usr/lib/ruby/3.1.0/monitor.rb:202:in mon_synchronize'         from /var/lib/gems/3.1.0/gems/net-imap-0.3.7/lib/net/imap.rb:2206:in send_command'
from /var/lib/gems/3.1.0/gems/net-imap-0.3.7/lib/net/imap.rb:1118:in block in examine'         from /usr/lib/ruby/3.1.0/monitor.rb:202:in synchronize'
from /usr/lib/ruby/3.1.0/monitor.rb:202:in mon_synchronize'         from /var/lib/gems/3.1.0/gems/net-imap-0.3.7/lib/net/imap.rb:1116:in examine'
from /var/lib/gems/3.1.0/gems/imap-backup-12.1.0/lib/imap/backup/client/default.rb:62:in examine'         from /var/lib/gems/3.1.0/gems/imap-backup-12.1.0/lib/imap/backup/client/automatic_login_wrapper.rb:23:in method_missing'
from /var/lib/gems/3.1.0/gems/imap-backup-12.1.0/lib/imap/backup/account/folder.rb:152:in examine'         from /var/lib/gems/3.1.0/gems/imap-backup-12.1.0/lib/imap/backup/account/folder.rb:35:in block in exist?'
from /var/lib/gems/3.1.0/gems/imap-backup-12.1.0/lib/retry_on_error.rb:4:in retry_on_error'         from /var/lib/gems/3.1.0/gems/imap-backup-12.1.0/lib/imap/backup/account/folder.rb:34:in exist?'
from /var/lib/gems/3.1.0/gems/imap-backup-12.1.0/lib/imap/backup/account/folder.rb:43:in create'         from /var/lib/gems/3.1.0/gems/imap-backup-12.1.0/lib/imap/backup/migrator.rb:17:in run'
from /var/lib/gems/3.1.0/gems/imap-backup-12.1.0/lib/imap/backup/cli/transfer.rb:53:in block in run'         from /var/lib/gems/3.1.0/gems/imap-backup-12.1.0/lib/imap/backup/cli/folder_enumerator.rb:34:in block in each'
from internal:dir:220:in glob'         from /var/lib/gems/3.1.0/gems/imap-backup-12.1.0/lib/imap/backup/cli/folder_enumerator.rb:30:in glob'
from /var/lib/gems/3.1.0/gems/imap-backup-12.1.0/lib/imap/backup/cli/folder_enumerator.rb:30:in each'         from /var/lib/gems/3.1.0/gems/imap-backup-12.1.0/lib/imap/backup/cli/transfer.rb:50:in run'
from /var/lib/gems/3.1.0/gems/imap-backup-12.1.0/lib/imap/backup/cli.rb:152:in migrate'         from /var/lib/gems/3.1.0/gems/thor-1.2.2/lib/thor/command.rb:27:in run'
from /var/lib/gems/3.1.0/gems/thor-1.2.2/lib/thor/invocation.rb:127:in invoke_command'         from /var/lib/gems/3.1.0/gems/thor-1.2.2/lib/thor.rb:392:in dispatch'
from /var/lib/gems/3.1.0/gems/thor-1.2.2/lib/thor/base.rb:485:in start'         from /var/lib/gems/3.1.0/gems/imap-backup-12.1.0/lib/imap/backup/cli.rb:65:in start'
from /var/lib/gems/3.1.0/gems/imap-backup-12.1.0/bin/imap-backup:12:in block in <top (required)>'         from /var/lib/gems/3.1.0/gems/imap-backup-12.1.0/lib/imap/backup/logger.rb:41:in sanitize_stderr'
from /var/lib/gems/3.1.0/gems/imap-backup-12.1.0/bin/imap-backup:11:in <top (required)>'         from /usr/local/bin/imap-backup:25:in load'
from /usr/local/bin/imap-backup:25:in `'

@michaldejmek
So, there's no more SIGKILL, just the impossibility of appending messages over a certain size (which depends on the server configuration). I think we can close this issue.


Flaky Test - TxPoolOptionsTest

txpoolForcePriceBumpToZeroWhenZeroBaseFeeMarket is failing consistently in CI, but not when run locally

Example failure:
TxPoolOptionsTest > txpoolForcePriceBumpToZeroWhenZeroBaseFeeMarket() FAILED     org.mockito.exceptions.verification.WantedButNotInvoked at TxPoolOptionsTest.java:104

Re-opening since the test is failing consistently in https://github.com/hyperledger/besu/pull/7510, see CI https://github.com/hyperledger/besu/actions/runs/10830553170/job/30055039117


Update egui to 0.25 for egui_window_glfw_passthrough

Hi, I was having trouble getting egui-glow to work with egui_window_glfw_passthrough so I decided to update to the latest egui version. I haven't tested the other crates in the workspace. Event::Key now has a physical_key field that should be set via the key scancode, but I'd rather not write a function that maps the scancode to the key right now.

I will fix the breaking changes and make a new release in the next hour

0.7.0 released with egui 0.25. :+1:

Wow wonderful, that was fast!


Leave out web info when there is no web

WHY are these changes introduced?
It used to be we could always assume there's a web. That's not true anymore.

WHAT is this pull request doing?

Don't show a web if there is none.
Before:

After:

How to test your changes?

Run app info on an extension-only app. It should correctly omit web and spacing should look normal.
It should also still work normally on an app with a home.
Measuring impact
How do we know this change was effective? Please choose one:

[x] n/a - this doesn't need measurement, e.g. a linting rule or a bug-fix
[ ] Existing analytics will cater for this addition
[ ] PR includes analytics changes to measure impact

Checklist

[x] I've considered possible cross-platform impacts (Mac, Linux, Windows)
[x] I've considered possible documentation changes
[x] I've made sure that any changes to dev or deploy have been reflected in the internal flowchart.

Still relevant! Just not highest priority at the moment

Still relevant! Just not highest priority at the moment


Help understanding output of TransportState::get_remote_static

Hello,
I'm trying to use TransportState::get_remote_static to get the remote end static public key to verify the identity of the remote peer in both ends. I call this right after the handshake is done, on the object returned by HandshakeState::into_transport_mode.
Here's a small program that does the handshake and sends some data from a tcp client stream to a tcp server stream, based on the example in the repo, but it also calls get_remote_static and prints everything from every step to stdout for debugging.
All static keys are hardcoded at the top of the program.
https://gist.github.com/hMihaiDavid/dd2c58378b64b6ae9062cd1a743fe618
I've put some comments with the log of 2 runs, one with .remote_public_key(...) in the builder chain and one without it. It uses an XX noise pattern. The logs are the same.
The output of get_remote_static does not coincide with the peer's static public key.
static STATIC_PRIVKEY_CLIENT: [u8; 32] = [ 52, 18, 25, 179, ...
static STATIC_PUBKEY_CLIENT: [u8; 32] = [  160, 244, 254, 226, ...
static STATIC_PRIVKEY_SERVER: [u8; 32]  = [ 136, 238, 164, 3, ...
static STATIC_PUBKEY_SERVER: [u8; 32]= [ 171, 232, 231, 4, ...

Whereas the output of get_remote_static is:
server: remote_pubkey_from_transport_state: Some([117, 128, 194, 127,  ...
client: remote_pubkey_from_transport_state: Some([52, 18, 25, 179, ...

Same output without .remote_public_key(...) in the builder chain.
Am I using this wrong? What should I expect from that api?
Also, as you can see, the Client (initiator), gets its static privater key as the output of TransportState::get_remote_static.
Thanks in advance, have a nice day.

Ok so right after writing this I just realised that my key gen code (at the bottom of the sample) used the pattern Noise_XXpsk3_25519_ChaChaPoly_BLAKE2s whereas the client and server use Noise_XX_25519_ChaChaPoly_BLAKE2s. When regenerating the static keys with the proper pattern the outputs of get_remote_static are as expected -- the pubkey of the other peer.
I've been stuck for days lol.
So I guess that problem solved, but why that weird behavior tho? Why did the client get its private key when calling get_remote_static?