CCH22-3: Adjust the snippets/buildCache/cacheable-bundle sample for configuration cache

The sample project located under subprojects/docs/src/snippets/buildCache/cacheable-bundle does not meet the configuration cache requirements.
The build scripts in the groovy and kotlin parts of the sample project need to be adjusted to follow the requirements.
Context
This issue is a part of the Gradle Configuration Cache Hackathon 2022.
If you wish to participate and contribute the fix, please follow the suggested workflow.
Expected behavior

The sample project is fixed so that the related documentation tests pass with configuration cache enabled.
The tests related to this sample project are removed from the testsToBeFixedForConfigurationCache list in subprojects/docs/build.gradle. You can find the test IDs below.
The doc pages that include the snippets from this sample project might need to be updated to be consistent with the sample. Feel free to ask the team for help with this! You can find the pages listed below.

Test failure details
These tests running in :docs:docsTest check the sample project and are failing with configuration cache enabled:

snippet-build-cache-cacheable-bundle_groovy_cacheableBundle-caching.sample(Build Scan)
Task `:bundle` of type `NpmTask`: invocation of 'Task.project' at execution time is unsupported.

snippet-build-cache-cacheable-bundle_groovy_cacheableBundle.sample(Build Scan)
Task `:bundle` of type `NpmTask`: invocation of 'Task.project' at execution time is unsupported.

snippet-build-cache-cacheable-bundle_kotlin_cacheableBundle-caching.sample(Build Scan)
Task `:bundle` of type `Build_gradle$NpmTask`: invocation of 'Task.project' at execution time is unsupported.

snippet-build-cache-cacheable-bundle_kotlin_cacheableBundle.sample(Build Scan)
Task `:bundle` of type `Build_gradle$NpmTask`: invocation of 'Task.project' at execution time is unsupported.
Task `:printBundle` of type `org.gradle.api.DefaultTask`: cannot serialize Gradle script object references as these are not supported with the configuration cache.

How to reproduce
To reproduce the failure in the Gradle documentation tests, run the following in the gradle/gradlerepository:
./gradlew :docs:docsTest \
    -PrunBrokenConfigurationCacheDocsTests=true \
    -PenableConfigurationCacheForDocsTests=true \
    --tests "*snippet-build-cache-cacheable-bundle_groovy_cacheableBundle-caching.sample" \
    --tests "*snippet-build-cache-cacheable-bundle_groovy_cacheableBundle.sample" \
    --tests "*snippet-build-cache-cacheable-bundle_kotlin_cacheableBundle-caching.sample" \
    --tests "*snippet-build-cache-cacheable-bundle_kotlin_cacheableBundle.sample"

In the test build outputs, which you can find in the test report, the configuration cache problems are listed along with the URL pointing to the detailed HTML problems report that can help investigating the violation.
You can find more instructions on working with the sample projects in the FAQ.
Docs including the snippets from this sample project
The following doc pages include the snippets from this sample project:

subprojects/docs/src/docs/userguide/build-cache/build_cache.adoc;

Hello, I'd like to work on this

Have at it!


OSError: [Errno 28] Can't write data (file write failed: time = Wed Nov 30 22:27:02 2022

Greeting,
While running casanovo using denovo mode on my institutes HPC I faced and error. I suspect that it might be a storage space issue. Can I confirm with you if it is a storage space issue before contacting my HPC to request for more storage?
Please refer to the attatched log below the error in detail.
Regards,
Ben
Casanovo_Log_with_Error.txt

Greetings,
On the last line of the log the message was -
"filename = '/tmp/tmpy1v8xnxh/c90e3ad17a024dceb83502ce09231f83.hdf5', file descriptor = 11, errno = 28, error message = 'No space left on device', buf = 0x7fba9f4999f8, total write size = 105727680, bytes this sub-write = 105727680, bytes actually written =<PHONE_NUMBER>3709551615, offset = 0)"
Based on the error log, does this mean that Casanovo needs ><PHONE_NUMBER>3709551615 bytes<PHONE_NUMBER>3.709552765 GB)?

The<PHONE_NUMBER>3709551615 is likely incorrect because your computer couldn't handle being completely out of disk space. Casanovo actually tried to use 105727680 bytes = 105 MB for the 6 peak files you ran it on.
Casanovo will always try to write to the temporary directory (depending on where that is for your operating system). This behavior cannot be modified, so I recommend trying to run on a node with a larger temporary disk.


Fix independence of SciPyOptimizer objects

Summary
Fix Qiskit/qiskit-ibm-runtime#407 by avoiding the use of locals() in the Qiskit optimizers, which led to changes in previously created optimizer instances upon creating a new one.
Details and comments
Fix a bug where creating a new instance of a SciPy optimizer has overwritten the
settings of a previously created instance of the same type. For example, previously:
from qiskit.algorithms.optimmizers import COBYLA
optimizer = COBYLA(maxiter=1)
reconstructed = COBYLA(**optimizer.settings)

Created a new COBYLA instance with the wrong value maxiter=1000 (= the default value) and additionally reset
the iterations of the original optimizer instance to 1000 as well.

This PR does resolve this issue by making an internal copy of the options, I think, doesn't it? 🙂 And maybe it's better to avoid using locals(), this seems like it could backfire in many ways (as it did just now 😛 ).

locals() is not the reason for the failure. Its more that settings returns the options dictionary by ref and that ref is taken into a second optimizer
>>> optimizer = COBYLA(maxiter=1)
>>> optimizer.settings
{'max_evals_grouped': 1, 'options': {'maxiter': 1, 'disp': False, 'rhobeg': 1.0}, 'tol': None}
>>> newopt = COBYLA(**optimizer.settings)
>>> optimizer.settings
{'max_evals_grouped': 1, 'options': {'maxiter': 1000, 'disp': False, 'rhobeg': 1.0}, 'tol': None}

ie settings does this
    def settings(self) -> Dict[str, Any]:
        settings = {
            "max_evals_grouped": self._max_evals_grouped,
            "options": self._options,
            **self._kwargs,
        }

whether options is created from locals() or more manually it would seem they would still share things. (The above was done on original code, I did not try on the updated code). As far as I can see the only difference is that options manually built, rather than off locals. But options dictionary is still passed out of settings by reference and stored as given. Had settings done a copy, then they would not have been working off the same options dictionary and it would not have had the problem as far as I can see. Now how an optimizer handles the options when that is how maxiter is provided versus explicitly in the constructor I guess is an issue if its rebuilt off settings.

On the new branch this is the output:
>>> optimizer = COBYLA(maxiter=1)
>>> optimizer.settings
{'max_evals_grouped': 1, 'options': {'maxiter': 1, 'disp': False, 'rhobeg': 1.0}, 'tol': None}
>>> newopt = COBYLA(**optimizer.settings)
>>> optimizer.settings
{'max_evals_grouped': 1, 'options': {'maxiter': 1, 'disp': False, 'rhobeg': 1.0}, 'tol': None}
>>> id(optimizer.settings["options"])
140218066241968
>>> id(newopt.settings["options"])
140217797569424

Yeah I understand that the issue wasn't locals(), and in this branch it should copy the options in _options.update(options) 🙂 We can still keep using locals of course, but I thought it's clearer and easier to understand to readers (like me) if we avoid it 😄

Superseded by #8381 for now.


Add prow jobs for gardener-extension-shoot-rsyslog-relp repo

/kind enhancement
What this PR does / why we need it:
This PR adds prow jobs for unit tests and image builds of gardener-extension-shoot-rsyslog-relp repository.
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:

Thanks a lot for the review @oliver-goetz. I've addressed your comments.
PTAL

/lgtm
/approve


docs: Add Codecov badge to README

Tom from Codecov here. I thought I would go ahead and add the Codecov badge here in case it's useful! Also removed some extra whitespace.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Thanks @thomasrockhu!
The problem is, that right now we don't have codecov configured properly yet, so the reporting is incorrect.
Until then we would not want to merge this, as it's not yet accurately reflecting the current state.

@timsuchanek, got it. Can I do anything to help here?

@thomasrockhu I noticed strange things on Codecov
I created this issue for now #4349 Many Unable to find commit in Github in commits view
I didn't create an issue for that, I can see quite big spikes (~ +13%) in coverage chart and it doesn't make sense to me, it doesn't seem accurate.

Happy to discuss that in a private channel 😃

@Jolg42, the one thing I noticed on the spikes is that those typically are only uploading coverage reports under the client flag like here as compared to another commit. Is there something in CI that doesn't upload coverage for those paths?

@thomasrockhu Thanks, it seems that after switching to the GitHub Action (was Bash uploader directly before) and cleaning up the Actions the spikes don't show up anymore 💚
I'm closing this PR for now as we have some internal homework to do, we'll reopen and merge when ready!

@Jolg42 amazing! I'm glad that that helped. Feel free to let me know if you run into any other issues, or if we can be doing anything better.


Fix mtl-2.3 compat

Co-authored-by: @matobet
Cherry-picked from https://github.com/GetShopTV/swagger2/commit/b8048f37671a984005e5fb0f4e4f18b5d51e5d25
Tested on GHC-9.0.2 and GHC-9.2.4, without tests: cabal build -w ghc-9.2.4 --disable-tests --constraint='mtl>=2.3'  --constraint='transformers>0.6' --allow-newer=insert-ordered-containers:transformers
Tests currently don't compile due to the use of doctest, which breaks on the GHC dependency, which cabal-install tries to rebuild even though it shouldn't (ghc the library can't be built from hackage, if i understand correctly)

Thanks!
I'll make a new release when tests are fixed with 9.4.

I'll make a new release when tests are fixed with 9.4.

@maksbotan Would it be possible to make the new release? 9.4 tests have been fixed.

Done! https://hackage.haskell.org/package/openapi3-3.2.3


Generated JSON typescript type is not performant (source of infinite recursion)

Describe the bug
Currently generated JSON typescript type is as follows:
export type Json =
  | string
  | number
  | boolean
  | null
  | { [key: string]: Json | undefined }
  | Json[];

The second to last line in particular can cripple typescript if used with certain tooling that depends on that type. Resulting in the following error:
Type instantiation is excessively deep and possibly infinite.ts(2589)
column: string | number | boolean | Json[] | {
        [x: string]: string | number | boolean | Json[] | ... | null | undefined;
    } | null;

It feels like this generated type doesn't add much use to anyone anyways. I think the best solution would be to set any JSON column as unknown, which it technically is, and let developers decide on the exact type.
I manually set the JSON definition to the below, and my vscode (on an M2 pro macbook) became alive again 🥳:
export type Json = unknown;

Not sure there's an easy way to overwrite the JSON column type in the userland, is there? Typefest's MergeDeep only merges, but doesn't overwrite the types (which can make the problem even worse).
To Reproduce
TBD
Expected behavior
No chance for infinite recursions in generated types types.
Screenshots
TBD
System information
Rerun the failing command with --create-ticket flag.

Ticket ID: [e.g. ab1ac733e31e4f928a4d7c8402543712]
Version of OS: latest OSX
Version of CLI: 1.148.6
Version of typescript: 5.4.2

Additional context
Not relevant

I found a (temporary) fix to actually replace JSON types with typefest's MergeDeep:
MergeDeep<DatabaseGenerated, PUT_YOUR_MERGE_SCHEMA, { arrayMergeMode: "replace" }>

The default arrayMergeMode is "spread", which doesn't replace the JSON column by default, because it can be an array based on the default definition (wouldn't be a problem if it was unknown), so the resulting type ends up in:
{
    column_name: CustomType[] | (Json | CustomType)[];
}

Instead of
{
    column_name: CustomType;
}

type-fest doesn't really have proper docs, so it took me a bit of time to understand how to change the behavior.
The issue is still valid though, because unless you manually replace all instances of JSON columns, you can still unknowingly get pretty bad performance issues with the types. I had this for months, and only accidentally discovered the root cause.

Related

https://github.com/supabase/supabase-js/issues/808

Hello all. Thanks for opening (and for the suggestions)!
As the issue is open on the JS library, we would close this one to keep the discussion (and work) centralised.


sqlite: "create table" reports updated rows

System information:

Operating system (distribution) and version- Windows 10 Pro
DBeaver version 7.3.0
Additional extensions-none

Connection specification:

Database name and version - sqlite local file

Describe the problem you're observing:
First I had a table called fixed_callnums with about 17,000 rows in it.
Then I did: delete from fixed_callnums;
and confirmed the warning.
Then I had a change of heart and decided to delete the entire table and start a new one.
So I used right-click Delete on the table name in the Database Navigator and confirmed deleting it.
Then I ran the command in the screen shot to create a new table of the same name but with different column names/types from the first one.
How can dbeaver be reporting that Updated Rows number?  Where could that be coming from?  That isn't close to even the number of rows that had been in the old table, if somehow there was a trace of that still left.
I do have other tables in this same .db file that do have large numbers of rows on that scale, so I'm worried about what actually was "updated".

Unfortunately cannot reproduce using your steps. I deleted and created a new table with the same name. 0 updated rows are displayed in the ResultSet.

Maybe your number was received from the driver. What SQLite version do you have?

there is no update on issue for a month.
feel free to reopen it with an additional info

there is no update on issue for a month.
feel free to reopen it with an additional info


fico.i.lithium.com

Energized Protection - block ⚡

Let's make an annoyance free, better open internet, altogether!

Issue Submit Form

  Provide the following infos properly, which will help us to resolve your issue quickly.

Issue(s):
Type x in between [ ] and make sure there isn't any space between brackets. Example; for Your Selected Issue(s), type like this - [x]
You can select more than one category of issues if you need to!

[x] Whitelist
[ ] Blacklist
[ ] App Broken
[ ] Website Broken
[ ] Request
[ ] Other Issue

Pack(s):
Write the name of the pack(s) you are using.

Pack(s) Name: blu

Extension(s):
Also name the extension(s) if you are using any. If you don't then leave it blank.

Extension(s) Name:

Domain(s):
If you are submitting this issue for whitelist/blacklist issue, send us the domain(s) for whitelisting/blacklisting here. Kindly use the Code Tag to prevent tracking.

Domain(s):

fico.i.lithium.com

Your Config:
Just to ensure there is no issues or conflicts with other app/software/magisk module/extension/source list.
Make sure you are running Energized Protection Service only.

Client:

Version:

Details:
Write us a lil bit more about your issue or query. You can attach any screen shot or log of the issue or advert, this will help to highlight it.

Your Issue Detail:
Breaks page
https://ficoforums.myfico.com/
Please also consider whether other lithium related domains need to be blocked. Lithium is forum software so a lot of sites use their platform for online forms/discussion boards.

Thank you for making Energized Protection great with your kind help!

A project by Ador with ❤

I can see fico.i.lithium.com in Energized Unblock now, which is the centralised whitelist, so this seems like a solved case to me.


features: Inter project communication is EE

Individual Project Configuration is a licensed feature, but had a checkmark.
Also: SSO might happen in the future, there's just no set timeline.

Individual Project Configuration is not the same as Inter Project Communication ??

Good catch @sammachin! Fixed now

As it's approved, I'll self merge now


embed: 1.17 rejects types with underlying type []byte

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

$ go version
go version go1.17 linux/amd64

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="amd64"
GOBIN=""
GOCACHE="/home/tnozicka/.cache/go-build"
GOENV="/home/tnozicka/.config/go/env"
GOEXE=""
GOEXPERIMENT=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOINSECURE=""
GOMODCACHE="/home/tnozicka/go/pkg/mod"
GONOPROXY=""
GONOSUMDB=""
GOOS="linux"
GOPATH="/home/tnozicka/go"
GOPRIVATE=""
GOPROXY="direct"
GOROOT="/home/tnozicka/lib/go1.17"
GOSUMDB="off"
GOTMPDIR=""
GOTOOLDIR="/home/tnozicka/lib/go1.17/pkg/tool/linux_amd64"
GOVCS=""
GOVERSION="go1.17"
GCCGO="gccgo"
AR="ar"
CC="gcc"
CXX="g++"
CGO_ENABLED="1"
GOMOD="/home/tnozicka/go/src/github.com/scylladb/scylla-operator/go.mod"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build318278898=/tmp/go-build -gno-record-gcc-switches"

What did you do?
Updated go from 1.16.7 to 1.17 and the following code stopped compiling
var (
	//go:embed "basic.scyllacluster.yaml"
	BasicScyllaCluster ScyllaCluster
)

type ScyllaCluster []byte

func (sc ScyllaCluster) ReadOrFail() *scyllav1.ScyllaCluster {
	obj, _, err := scheme.Codecs.UniversalDeserializer().Decode(sc, nil, nil)
	o.Expect(err).NotTo(o.HaveOccurred())

	return obj.(*scyllav1.ScyllaCluster)
}

What did you expect to see?
No compile error as it was with 1.16.7 (and previous versions).
What did you see instead?
go:embed cannot apply to var of type ScyllaCluster

Type alias is
type ScyllaCluster = []byte

It is only documented to only work with string or []byte: https://pkg.go.dev/embed#hdr-Directives

The type of the variable must be a string type, or a slice of a byte type, or FS (or an alias of FS).

type aliases still work

Verified that it works for Go 1.16.

cc @rsc

I think the reason is that the following cl overrides the original logic.
https://github.com/golang/go/commit/7e0a81d2806b073c6455f73b10fbf2c811703f46#diff-93e5af6a86a5745929c27e317b158e986257f9cf4b6904f6e01e13f9bbebaabeL105

Strangely, 1.17 works for named string types, but not named byte slices.

@gopherbot please open a backport issue for 1.17.


Open Graduation Classes

Thanks for submitting your project to be featured in the Open Leadership Zone at MozFest 2017 :sparkles:
This is open to any project in Mozilla Open Leaders round 4 -- even if you're not coming to MozFest. This process ensures that you're following open practices and will help set you up for a sustained contributor community afterwards. If you have any questions, feel free to reach out to @acabunoc or ask questions in our chat room.
Open Project Checklist :clipboard:
Your project must complete the following to be a featured on Mozilla Pulse and the Open Leadership Zone at MozFest 2017. As you complete each exercise, check off the box and comment with a link to your completed resource. This template repository is here to help you if you get stuck!

[x] Complete Open Leadership 101
[x] Provide a GitHub repository for work and discussion on your project in a comment

new to GitHub?  Here's a step-by-step guide on using the #mozfest template

[x] Create file: README.md in your project repository. This file should help newcomers understand what your project is, why it's important, and kinds of help you're looking for.
[x] Create file: LICENSE to give your project an open license, allowing for sharing and remixing.
[x] Create file: CONTRIBUTING.md so others know how they can contribute. If you'd like, you can remix this template
[x] Create file: CODE_OF_CONDUCT.md. This can be the Mozilla Community Participation Guidelines or a code of conduct of your choice
[x] Turn on your Issue Tracker and create issues to describe each task that you need help with and how a contributor can get started on that task. If you're participating in Hacktoberfest, Create a label called hacktoberfest and apply it to your issues.

Ready to be featured :tada:

[x] Fill out this form to submit your project to Mozilla Pulse. Add the tag open leadership zone.
[x] Leave a comment on this issue with the text This is ready for the Open Leadership Zone. @acabunoc will review this issue. If accepted, your project will be approved on Mozilla Pulse and featured in the Open Leadership Zone.

If you get stuck at any point, feel free to look at the project templates or reach out to your mentor or @acabunoc. We're here to help you through this process.

Github repo:
https://github.com/felipez3r0/openclasses

Hi Abby, this is ready for the Open Leadership Zone.
Pulse project link: https://www.mozillapulse.org/entry/467

🎉 👍 looks great, you're public on pulse now!
https://www.mozillapulse.org/entry/467


[FLINK-12876][runtime] Add an adapter of region failover NG for legacy scheduler

What is the purpose of the change
Based on #8851
Brief change log

See #8851

Verifying this change
This change is already covered by existing tests, such as RegionFailoverITCase.
This change added tests and can be verified as follows:

Added unit tests for AdaptedRestartPipelinedRegionStrategyNG

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

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

Documentation

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

rebased to master

rebased to master

Rebased to master and squashed commits. Merging as soon as build is green.


beh(fcsfile): Add save() to save FCS to file

Closes #153

spec: Fixup skipped assertion in test_fcsfile.py
spec(utils): Add tests for parse_fcs_file
src(utils): Change parse_fcs_file() -> FcsFileIO.parse()
deps: Replace fcsparser with flowio
beh: Add IO exception for FcsFileIO
beh: Add FcsFileIO._write
spec: Add novocyte test fixture
beh(fcsfile): Add save() to save FCS to file

Codecov Report

Merging #154 (c7b033c) into master (eed7d17) will increase coverage by 0.19%.
The diff coverage is 95.73%.

@@            Coverage Diff             @@
##           master     #154      +/-   ##
==========================================
+ Coverage   93.63%   93.83%   +0.19%
==========================================
  Files          67       69       +2
  Lines        2906     3080     +174
==========================================
+ Hits         2721     2890     +169
- Misses        185      190       +5

Flag
Coverage Δ

unittests
93.83% <95.73%> (+0.19%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

setup.py
0.00% <ø> (ø)

tests/conftest.py
84.21% <ø> (ø)

tests/fixtures/compensated_events.py
96.29% <83.33%> (-3.71%)
:arrow_down:

cellengine/utils/fcs_file_io.py
90.56% <90.56%> (ø)

cellengine/resources/fcs_file.py
90.07% <93.10%> (+0.88%)
:arrow_up:

tests/unit/utils/test_read_write_fcs_file.py
98.50% <98.50%> (ø)

cellengine/utils/__init__.py
100.00% <100.00%> (ø)

cellengine/utils/api_client/APIClient.py
85.34% <100.00%> (+1.07%)
:arrow_up:

tests/fixtures/novocyte.py
100.00% <100.00%> (ø)

tests/unit/resources/test_compensation.py
100.00% <100.00%> (ø)

... and 3 more

Continue to review full report at Codecov.

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

Variants of these commits landed:

switched to FlowIO in f09c505a4e1fcd1665a68b17acf1e5cf72105f72
added FcsFile.create_from_dataframe() in c18f551adb578d2d527a25fea256afb191e941ab.


Rework Build System, Drop Node 10, Simplify Configs

I've been using this to work on the PRs I've made, dogfooding it in the project that I'm making that uses listr2.
Seems to work as before, but I have no idea if the CI will pass 😅
I also changed the event triggering PR CI builds to pull_request, as pull_request_target is meant for when you want to run things based on the event triggering it, not the things in the PR. This will also fix the errors in #615
I tried to remove ts-node too, by simplifying the jest setup to make the tests runnable with just jest, but ran into issues with the rewire shortcut paths not being updated without ts-node running it...

Replaces complex system with tsc, tsc-watch, tsconfig-replace-paths (which doesn't even work on windows!) with tsup.
This will change the output from being separate files to being one bundled file
Drops supported non-LTS versions of node (10)
Adds ESM build
Simplify (shorten, use presets in) jest, TS configs

Enabled allowSyntheticDefaultImports as it was required for tsup to build, and it's nicer to work with IMO 😄

No idea what's happening here, the CI seems to be running against master instead of my branch??

Ah, it's because of pull_request_target... Updated the body of the PR

Thank you again @BeeeQueue,
I am very grateful for this pull request since this was something that I should have done long ago. The only problem that I can see is the ESM-only dependencies. I guess it is not bundling the dependencies right? Since we still have a common-js version, updating those would be out of the question still. I was postponing ESM conversion till I had ditched rxjs and log-update dependencies to make it into breaking change bundle, which I have failed miserably in my last tries, but I guess we can go ahead with the error collection and make this a breaking change.
I am not sure to drop the common-js version or not since I rely on them in some legacy dependencies for work, so this would be a good compromise still.
Thank you again sincerely for your time on this. Again I will merge this in tomorrow hopefully if it is not a problem.

The only problem that I can see is the ESM-only dependencies. I guess it is not bundling the dependencies right?

Indeed, it does not bundle anything in dependencies or peerDependencies.
Even if it did, it still converts all bundled code to work with the module system it builds for so that'd still work 🙂

Hey @BeeeQueue,
Again sincerly thank you for your work on this. I merged it all to the temporary branch of develop but it has one big problem with ts-up which is ignoring extends field in tsconfig file. I might have to wait until the issue is closed.
https://github.com/egoist/tsup/issues/524

Huh i haven't experienced that myself... Maybe it's because of something in the tsconfig cuasing it to use a different compiler? In the issue he mentions decorators which I've never used in a project with tsup

It should not be much of a problem, we can just do until this  issue is fixed ofc, a direct tsconfig file with every option included.

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Added more search support

Hello,
I added support for Composite,News,Video Search see also:
https://datamarket.azure.com/dataset/bing/search#schema

Thanks a lot for the contribution :-)
So sorry for the delay. I've been on vacation, pretty disconnected.


Remove CodeCov badge

We haven't been uploading code coverage for a while now.

Let's also delete the codecov.yml in root


Add constant-encoded vectors to PyVelox

This PR depends on #3787

@kgpai

@save-buffer Can you rebase against latest main ?

Failures due to fuzzer failure and benchmark job (which shouldnt be affected by this change) .


fix: errors being raised without required args

Summary
Some of the errors being throw without the required msg argument to initialize that error type. Expecting handle_error method inside unit tests doesn't initialize the error instance. This PR modifies the raise error assertion inside unit tests and fixes where required.
Also removes an unused error type.
Test plan
All tests continue to pass.
Issues

Coverage increased (+0.07%) to 99.546% when pulling 134b401377106410e0a98ed5c98e50c1ea62c785 on oakbani/fix-errors into 1c19b8e8e1910878d3ece46850f611a8400b626b on master.


Control Center Remote Volume not working anymore

Describe Your Problem:
Since a while the volume control doesn’t work anymore when using the remote in control center. All buttons still work, besides volume control
Logs:
Show the Homebridge logs here, remove any sensitive information.

Plugin Config:
Show your Homebridge config.json here, remove any sensitive information.

Screenshots:

Environment:

Plugin Version:
Homebridge Version:
Node.js Version:
NPM Version:
Operating System:

Reboot your home hub. Apple Home gets confused sometimes.
Check your logs. Is the command being sent? Is the IP address right?

Yeah i have already tried that but it didnt change anything. I also checked the logs but no command is being send when i press the buttons. It just changes the volume of my phone and doesnt even show the Animation that it used to show where the button that was pressed showed up on the side of the screen. IP Adress is correct because as i said all Buttons like Up and Down work normaly, besides the volume.

That's an Apple HomeKit Remote Control issue. I've had it happen to me. The best solution was to reboot the iPhone and the Home Hub. Then wait for the accessories to be detected properly. Once the app properly detects the TV accessory as a TV it then enables the volume control buttons.

I just tried what you suggested, I unplugged my Apple TV and my HomePod mini waited a few minutes, then plugged them both back in and restarted my Phone and even HomeBridge but it’s still not working. The only thing I could think of that could have caused this problem is that I’m also running HomeAssistant with wich the TV is also connected. Could that be a problem or should they both just work fine and it’s something different?

This is purely an issue with the Apple Remote Control app getting confused.
The plugin has no control whatsoever about display of the Remote Control on the iPhone. That's handled by the iPhone.
I'll check mine when I get home later today.

Ok that’s weird. It also happens on my iPad so it’s not iPhone exclusive I guess

Hi. sorry but this is very much an iOS thing, the plugin has no impact on the Apple Remote Control display or on the keypresses sent by the Remote Control. Nothing I can do. So I'm closing this issue


Rename this repo into tufrepo.py

I'm not at vmware anymore so I don't really have authority on this one...
That said, a rename is so unlikely to happen that I'll just close this: naming is hard, you can't always get the name you want


Change the size of infoboxes

Currently the infoboxes can take up quite some space in the user's screen, especially in fixed mode.
Proposed changes:

Option to remove the spacing in between infoxboxes.
Remove padding from infoboxes and make them 30x30 or even 28x28 by default

Hopefully with the overlay moving, if it's dynamic, it'd be wonderful to also size it dynamically, if that makes sense

My hopes for when they're resizable is that you can resize them horizontally to limit the amount of infoboxes in a row etc.

Hey guys, I'd be happy to work on this but can you give me some specific info boxes you would like to see modified?


Update to support the latest Minecraft version

Now the official version of Minecraft has been updated to 1.19.4. Do you have any plans to update this Python library?

I'll have to update the documentation at one point, but most features should still work
For example, you can use 1.19.4+ blocks in mcschematic and save it as 1.19.3-, the schem will still work in 1.19.4+.
The only features that barely breaks are transform functions such as schem.getStructure().rotate( /* blah blah */) but you can still use them without worry if the blocks are 1.19.3-


Add missing file descriptor.

otherwise json output doesn't get sent back to std out and concourse errors

@drnic @longnguyen11288 sorry I missed this in my previous PR
Thanks,
Rob

bumped boshrelease

Great, thanks @longnguyen11288 - I've validated the final release of the boshrelease is compatible with concourse 0.69.0 and should be fine going forward


Specify algorithm for digit 11 and beyond

https://github.com/google/open-location-code/blob/main/docs/specification.md#code-precision mentions non-square areas starting at digit 11.
https://github.com/google/open-location-code/blob/main/docs/olc_definition.adoc also says

From 11 digits on, a different algorithm is used. The areas are slightly larger but the advantage is that the codes are shorter.

Alas, neither documents give any algorithm. Even what digit goes into what box is never explained.

The algorithm you are looking for is described in the Least significant five digits sections of the document.

I see. It looked like two separate algorithms that one could choose from.
P.S., in addition to the English description, it would help a lot to see code samples. Else it is rather hard to be sure one is reading it OK.


The translation API returns an empty body in Cloudflare Workers.

Hey @ashishwhydonate , thanks for the report. Could you give some details on your environment (which node version etc), and how your example is set up? I just set up a hello world project on a cloudflare worker, and needed to set node_compat = true in the wrangler.toml to be able to publish to the worker in the first place. Once I did this, I now get
A request to the Cloudflare API (/accounts/<accid>/workers/scripts/deepl-api-test) failed.

  Uncaught TypeError: globalThis.XMLHttpRequest is not a constructor
    at worker.js:4597:11 in checkTypeSupport
    at worker.js:4618:38 in
  node_modules/rollup-plugin-node-polyfills/polyfills/http-lib/capability.js
    at worker.js:8:56 in __init
    at worker.js:9091:5 in node_modules/rollup-plugin-node-polyfills/polyfills/http-lib/request.js
    at worker.js:8:56 in __init
    at worker.js:10074:5 in node-modules-polyfills:http
    at worker.js:8:56 in __init
    at worker.js:10178:21 in node-modules-polyfills-commonjs:http
    at worker.js:11:50 in __require
    at worker.js:10605:18 in ../node_modules/deepl-node/dist/index.js
   [code: 10021]

  If you think this is a bug, please open an issue at:
  https://github.com/cloudflare/workers-sdk/issues/new/choose

which matches some other report we had on this, so I'd be curious how you got it to execute an API request in the first place.

Also, please see my response in https://github.com/DeepLcom/deepl-node/issues/14

I responded in #14 with an update, closing this issue for now.


Question: How to best share Nuclear JS modules between projects?

I'm really enjoying the usage of modules as seen in the rest API example. I'm now using Nuclear JS in multiple projects and I have some functionality that is duplicated between projects.
For example, I have two projects that both have a Notifications module to manage the state of what notifications the UI should show the user. The module is exactly the same between projects. However, since each module needs access to the Reactor instance I haven't yet factored this functionality out into an NPM module.
Any thoughts on how best to handle something like this?
It would be nice to be able to do something like:
// flux.js
import Notifications from 'nuclear-js-notifications';
import { Reactor } from 'nuclear-js';
const flux = new Reactor({ debug: true });
flux.registerStores(Notifications.stores);
export default flux;

// actions.js
import Notifications from 'nuclear-js-notifications';
api.post('/thing', { title: 'some thing' })
.then(res => {
  Notifications.actions.notify({ type: 'success', text: 'Success!' });
})
.catch(res => {
  Notifications.actions.notify({ type: 'error', text: 'Oh no!' });
});

// component.js
import Notifications from 'nuclear-js-notifications';
export const App = React.createClass({
  getDataBindings() {
    notifications: Notifications.getters.notifications,
  },
  render() {
    return (

        {this.state.notifications.map(notification => (

            {notification.get('text')}

        ))}

    );
  },
});

Hi,
I think the best way to do it is to export a register method on the module, that method will get the a reactor as an argument and it will register its stores. You should also export the actions/getters, so that you can use them in your components or in other modules.
Notifications:
import notificationsStore from './stores/notifications';
import { * as actions } from './actions';
import { * as getters } from './getters';

export function register(reactor) {
  reactor.registerStores({ notificationsStore });
}

export actions;
export getters;

I did the same as what @raulmatei is suggesting but took it 1 step further.
All my modules export register, actions and getters (example).
All my actions take in a reactor as their first param (example)
I have an exposeModule(target, reactor, modules) method that exposes modules on target, registers them if needed and binds all actions given the reactor. Exposed modules are accessible under <name>Actions and <name>Getters. Real world example using this.
import notification from './modules/notification';

const reactor = new Reactor({ debug: true });
var target = {};

exposeModule(target, reactor, {
  notification
});

// You can now access
target.notificationActions.notify({ type: 'success', text: 'Success!' })
Reactor.observe(target.notificaionGetters.notifications, notifs => console.log(notifs));

Extra bonus from having modules not depend on a global reactor is that you are able to do server-side rendering too, if desired.

@balloob wow!
I have also tried a different and maybe similar approach here, not sure if it's a good idea to abstract how stores are created and use a convention which makes you name store handlers as event names (e.g. ADD_TO_CART -> addToCart(...)) but I'm playing with it and maybe its another example on how to use NuclearJS.

@raulmatei: I use Polymer as one would use React: a declarative API to express my UI. Added value is that Polymer contains UI components implemented by Google, making sure they follow guide lines, accessibility and responsiveness.

@raulmatei @balloob Thank you both for the suggestions and examples. I'll try out something like these approaches for my current projects

Going to go ahead and close this out. In addition to the examples above there now seems to be an example of this in the Iso Chat example.


trace: warning: qt5 now uses makeScopeWithSplicing which does not have "overrideScope'", use "overrideScope".

Describe the bug
For a while now, nix search nixpkgs <arg> gives me the trace warning:
trace: warning: qt5 now uses makeScopeWithSplicing which does not have "overrideScope'", use "overrideScope".

Not a big deal, but it'd be nice to fix that.
Notify maintainers
Maybe @ttuegel ? as the Qt5 maintainer?
Metadata
[user@system:~]$ nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 6.1.39, NixOS, 23.11 (Tapir), 23.11.20230723.97137d0`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.15.1`
 - nixpkgs: `/home/doron/repos/nixpkgs`

duplicate https://github.com/NixOS/nixpkgs/issues/242320


Should the initial parameter samples / values be recorded for each chain

Following from a discussion between myself and @mohamed82008 , he managed to set up a quick fix for passing an initial theta into a Turing model, the code looks as follows using the fix he has made a PR for here https://github.com/TuringLang/Turing.jl/pull/1281:
@model gdemo(x, y) = begin
    s ~ InverseGamma(2,3)
    m ~ truncated(Normal(0.0, sqrt(s)), 0.0, 2.0)
    x ~ Normal(m, sqrt(s))
    y ~ Normal(m, sqrt(s))
end;
model = gdemo(1.0, 1.0);
varinfo = Turing.VarInfo(model);
model(varinfo, Turing.SampleFromPrior(), Turing.PriorContext((m = 0.5, s = 0.5)));
init_theta = varinfo[Turing.SampleFromPrior()]
c = sample(model, HMC(0.01, 1), 100, init_theta = init_theta)

This results in the following:
julia > c = sample(model, HMC(0.01, 1), 1000, init_theta = init_theta)
┌ Info: Using passed-in initial variable values
│   init_theta =
│    2-element Array{Float64,1}:
│     1.0
└     1.0
Sampling 100%|██████████████████████████████████████████████████████████████████████████████████████████████| Time: 0:00:00
Object of type Chains, with data of type 1000×11×1 Array{Float64,3}

Iterations        = 1:1000
Thinning interval = 1
Chains            = 1
Samples per chain = 1000
internals         = acceptance_rate, hamiltonian_energy, hamiltonian_energy_error, is_accept, log_density, lp, n_steps, nom_step_size, step_size
parameters        = m, s

2-element Array{ChainDataFrame,1}

Summary Statistics
  parameters    mean     std  naive_se    mcse      ess   r_hat
  ──────────  ──────  ──────  ────────  ──────  ───────  ──────
           m  0.9333  0.0522    0.0017  0.0164   4.0161  1.7977
           s  1.1174  0.0723    0.0023  0.0206  12.6280  1.0097

Quantiles
  parameters    2.5%   25.0%   50.0%   75.0%   97.5%
  ──────────  ──────  ──────  ──────  ──────  ──────
           m  0.8084  0.9056  0.9439  0.9709  1.0116
           s  0.9544  1.0804  1.1237  1.1693  1.2343

julia > get(c, :m)
(m = [0.997230952122332; 0.99781519383941; … ; 0.8313393573236126; 0.8277092475063401],)

So the question now is: should the initial values appear in the chain? This would also verify that passing it in is working correctly though I am fairly confident of that.

Both options make sense to me.
As for MCMC, we do obtain a chain (X_0, ..., X_T) that includes the state sampled from \pi_0. So it's a reasonable option to include it in the chain.
On the other side, the indexing of the chain object actually starts from 1. So ignoring X_0 in the object aligns the indexing, i.e. c[t] refers to X_t.
I'm not sure if we can support c[0] easily but I think it's reasonable to at least provide an option to include or exlcude X_0 in c.


Sample UI update

Description of Change
Changes the sample app to look more like this:

Bugs Fixed

Related to issue #129

PR Checklist

[ ] Has tests (if omitted, state reason in description)
[ ] Has samples (if omitted, state reason in description)
[ ] Rebased on top of main at time of PR
[X] Changes adhere to coding standard
[ ] Updated documentation

The lowest level of pages (individual samples) haven't been touched yet. Those are next on the list. The gallery pages should all be in there and should work atm.

Awesome work!

Good work @sthewissen :)


glib: Improve the API of Type

Make Type leaner, replace enum with assoc consts.
Make Type::is_a take other by copy
Introduce Type::is_valid and use it

This kind of handles https://github.com/gtk-rs/gtk-rs/issues/313 (for Type) at least, which is good.


Query regarding reservation

Here are some doubts regarding reservation:

No reservation ID is being sent in Start Transaction, even if the connector was reserved. As per OCPP1.6
"If a transaction for the reserved idTag is started, then Charge Point SHALL send the reservationId in
the StartTransaction.req PDU (see Start Transaction) to notify the Central System that the reservation is
terminated."
RemoteStartTransaction.req is being accepted even if the connector is reserved with other IdTag. As per OCPP1.6
"If the Charge Point accepts the reservation request, then it SHALL refuse charging for all incoming
idTags on the reserved connector, except when the incoming idTag or the parent idTag match the idTag
or parent idTag of the reservation."

How do I proceed further?
Do let me know if you plan to work on these issues or am I missing something?

May I please know how was this resolved? I am still seeing #2 with the latest version.


An improved version of the tutorial.

Hi!
I changed the writing of the example. I tried to make it more clear for users. Also, I tried to use a more Pythonic syntax. I added some timings and explanations about the html report.
Please tell me if I should rephrase anything.

Very nice, this is already much better. Thanks!
I added some comments and suggestions.

Thank you very much for the review! This will help me a lot.

Done!

Thanks again!


crash when connected to facebook group

so it is working fine , when i am using it in personal chat , but when i add the profile that is on the bot , the script crashes and gives me this error:
ERR! ERROR in listen -->  TypeError: Cannot call method 'toString' of undefined
ERR! ERROR in listen -->      at Object.formatTyp (/home/kawaii/Kassy/node_modules/facebook-chat-api/utils.js:294:24)
ERR! ERROR in listen -->      at parsePackets (/home/kawaii/Kassy/node_modules/facebook-chat-api/src/listen.js:97:42)
ERR! ERROR in listen -->      at Array.forEach (native)
ERR! ERROR in listen -->      at /home/kawaii/Kassy/node_modules/facebook-chat-api/src/listen.js:86:12
ERR! ERROR in listen -->      at tryCatcher (/home/kawaii/Kassy/node_modules/facebook-chat-api/node_modules/bluebird/js/main/util.js:26:23)
ERR! ERROR in listen -->      at Promise._settlePromiseFromHandler (/home/kawaii/Kassy/node_modules/facebook-chat-api/node_modules/bluebird/js/main/promise.js:507:31)
ERR! ERROR in listen -->      at Promise._settlePromiseAt (/home/kawaii/Kassy/node_modules/facebook-chat-api/node_modules/bluebird/js/main/promise.js:581:18)
ERR! ERROR in listen -->      at Promise._settlePromises (/home/kawaii/Kassy/node_modules/facebook-chat-api/node_modules/bluebird/js/main/promise.js:697:14)
ERR! ERROR in listen -->      at Async._drainQueue (/home/kawaii/Kassy/node_modules/facebook-chat-api/node_modules/bluebird/js/main/async.js:123:16)
ERR! ERROR in listen -->      at Async._drainQueues (/home/kawaii/Kassy/node_modules/facebook-chat-api/node_modules/bluebird/js/main/async.js:133:10)
ERR! ERROR in listen -->      at Async.drainQueues (/home/kawaii/Kassy/node_modules/facebook-chat-api/node_modules/bluebird/js/main/async.js:15:14)
ERR! ERROR in listen -->      at process._tickCallback (node.js:415:13)
ERR! ERROR in listen -->   [TypeError: Cannot call method 'toString' of undefined]
[TypeError: Cannot call method 'toString' of undefined]
Overall a great program/script here, but i'm mostly going to use if for a group chats , if you can fix it i would love it <3 . And i should add that some of the names of the people in the chat room are in Bulgarian, that uses diffrent alphabet than the english one. There are hole diffrrent.

@gogoneykov123 what version of the facebook api has npm installed? It works fine with group chats here except for issue https://github.com/mrkno/Kassy/issues/33

I managed to replicate using v1.0.3 of facebook-chat-api but it doesn't seem to exist in v1.0.2. For now just install v1.0.2 (because of https://github.com/mrkno/Kassy/commit/0fcda74c4be9e6a49fb4a540dd9869fcb7acbd68 you can just delete the folder facebook-chat-api from the node_modules directory, pull and then run npm install again and it will pull down the correct version).
For now as there is a workaround in place I will close this issue, I have opened an equivilent one on the api itself here: https://github.com/Schmavery/facebook-chat-api/issues/100.

Thanks I will try it now! :)


当前版本运行 npm run dev 后报错

(node:93509) Warning: To load an ES module, set "type": "module" in the package.json or use the .mjs extension.
(Use `node --trace-warnings ...` to show where the warning was created)

 ERROR  [worker] Worker terminated due to reaching memory limit: JS heap out of memory

  at new NodeError (node:internal/errors:371:5)
  at Worker.[kOnExit] (node:internal/worker:276:26)
  at Worker.<computed>.onexit (node:internal/worker:198:20)

(node:93509) Warning: To load an ES module, set "type": "module" in the package.json or use the .mjs extension.
(Use `node --trace-warnings ...` to show where the warning was created)

 ERROR  [worker] Worker terminated due to reaching memory limit: JS heap out of memory

  at new NodeError (node:internal/errors:371:5)
  at Worker.[kOnExit] (node:internal/worker:276:26)
  at Worker.<computed>.onexit (node:internal/worker:198:20)

Now fixed and upgraded to rc


findAndConnect clones WiFi configuration on Android 6

If it is an already configured by user connection on Android 6 findAndConnect method clones WiFi configuration and connects to the cloned one.
According to   Android 6.0 Changes

Your apps can now change the state of WifiConfiguration objects only if you created these objects. You are not permitted to modify or delete WifiConfiguration objects created by the user or by other apps.

@GRouslan See the changes I have added, it has forced wifi scanning and remove/forget wifi network when you don't need it. Maybe that can help you.
Thanks


UnsupportedFileSystemException No FileSystem for scheme ""alluxio""

Alluxio Version:
2.7.2
Describe the bug
"User class threw exception: org.apache.spark.sql.AnalysisException: org.apache.hadoop.hive.ql.metadata.HiveException: MetaException(message:Got exception: org.apache.hadoop.fs.UnsupportedFileSystemException No FileSystem for scheme ""alluxio"");; line -1 pos -1"
To Reproduce
https://spark.apache.org/docs/2.4.0/configuration.html#custom-hadoophive-configuration
val conf = new SparkConf().set("spark.hadoop.abc.def","xyz")
val sc = new SparkContext(conf)
set spark.hadoop.fs.alluxio.impl=alluxio.hadoop.FileSystem;  Why didn't this work？
`set spark.driver.extraClassPath=/data/sparkserver/spark_jars/sparkjars3/:/data/sparkserver/spark_jars/ranger-lib/:/data/sparkserver/spark_jars/atlas-lib/:/data/sparkserver/spark_jars/udf/hiido-uf-v011.jar:/data/sparkserver/spark_jars/udf/all.jar:/data/sparkserver/spark_jars/udf/hivemall-core-0.4.2-rc.2-with-dependencies.jar:/data/sparkserver/spark_jars/udf/hivemall-nlp-0.4.2-rc.2-with-dependencies.jar:/data/sparkserver/spark_jars/mainjar/hadoop-distcp-2.6.5.jar:/data/alluxio_client/alluxio-2.7.2-client.jar;
set spark.executor.extraClassPath=/data/sparkserver/spark_jars/sparkjars3/:/data/sparkserver/spark_jars/ranger-lib/:/data/sparkserver/spark_jars/atlas-lib/:/data/sparkserver/spark_jars/udf/hiido-uf-v011.jar:/data/sparkserver/spark_jars/udf/all.jar:/data/sparkserver/spark_jars/udf/hivemall-core-0.4.2-rc.2-with-dependencies.jar:/data/sparkserver/spark_jars/udf/hivemall-nlp-0.4.2-rc.2-with-dependencies.jar:/data/sparkserver/spark_jars/mainjar/hadoop-distcp-2.6.5.jar:/data/alluxio_client/alluxio-2.7.2-client.jar;
set spark.sql.hive.metastore.sharedPrefixes=com.mysql.jdbc,org.postgresql,com.microsoft.sqlserver,oracle.jdbc,alluxio;
set spark.hadoop.fs.alluxio.impl=alluxio.hadoop.FileSystem;
CREATE TABLE alluxio_u_user (
userid INT,
age INT,
gender CHAR(1),
occupation STRING,
zipcode STRING)
ROW FORMAT DELIMITED
FIELDS TERMINATED BY '|'
STORED AS TEXTFILE
LOCATION 'alluxio://fs-alluxio-master01.hiido.host.yydevops.com:19998,fs-alluxio-master02.hiido.host.yydevops.com:19998,fs-alluxio-master03.hiido.host.yydevops.com:19998/test/ml-100k';`

Logically, I could change this configuration alone, but it doesn't seem to be working.Because of the large scale of our cluster, if we change a single configuration, we need to change hadoop core-site. XML through Ambari and restart datanode, which is very costly and risky. I hope that only a single configuration of Spark can take effect. Is there any way to achieve this?

This configuration should be configured in metasotre service, the document is not clear and clear, I configured spark and Hadoop for a long time, but did not work, the result is to configure metaStore core-site. XML

Thanks @liangrui1988 for reporting! Do you mind helping update the doc based on your finding?

Let's fix the doc @HelloHorizon and close this issue.

@liangrui1988 I am a little bit confuse which step make this happen and which document you refer to? If you are using hive metastore, we have setup doc here https://docs.alluxio.io/os/user/stable/en/compute/Hive.html


Doesnt generate files or writes anything

I added the code to Cargo.toml:
[profile.release]
debug = 1

[features]
dhat-heap = []    # if you are doing heap profiling

Then to main.rs
#[tokio::main]
async fn main() -> Result<(), String> {
	#[cfg(feature = "dhat-heap")]
	let _profiler = dhat::Profiler::new_heap();

And installed the crate. I don't see neither any output or files from the tool. What did I do wrong?
P.S. I kill the app with Ctrl + C (error: process didn't exit successfully: target\debug\tool.exe (exit code: 0xc000013a, STATUS_CONTROL_C_EXIT)

You also need to update the global allocator. Add something like this to the top of your main.rs:
#[cfg(feature = "dhat-heap")]
#[global_allocator]
static ALLOC: dhat::Alloc = dhat::Alloc;

@GeorgeHahn is correct. Please read the full docs at https://docs.rs/dhat/latest/dhat/.

@GeorgeHahn @nnethercote sorry I missed it. I added it as well of course, in the time of writing my question.


make scala-xml a regular project

it doesn't need to be special anymore now that it's out of the bootstrap
fyi @ashawley

test run: https://scala-ci.typesafe.com/job/scala-2.13.x-integrate-community-build/1051/consoleFull (404 for now)

test run: https://scala-ci.typesafe.com/job/scala-2.13.x-integrate-community-build/1053/consoleFull

okay, we don't need to wait for the whole thing to run, I've seen enough to be sure this is mergeable

ahhhhhhhhhhhhhhhhhhhhhh

gah, in the commit message for b31ba8e I meant to say "assemble" build system, not "scala" build system
and also b31ba8e is actually wrong, because it no longer sets scalaVersion for every included project. I need to add something back in that does that. working on it.

@SethTisue I think you will still need to fetch via aether scala-library etc, using the assembly build system. Compare the dependency information section before:
[info] Project sbinary
[info]   depends on: cloc-plugin, scala
[info] Project scala-partest
[info]   depends on: cloc-plugin, scala
[info] Project sbt-testng
[info]   depends on: cloc-plugin, scala
[info] Project kind-projector
[info]   depends on: cloc-plugin, scala

and after:
[info] Project sbinary
[info]   depends on: cloc-plugin, scala-xml
[info] Project scalatest
[info]   depends on: cloc-plugin, scala-xml
[info] Project scala-records
[info]   depends on: cloc-plugin, scala-xml, scalatest

There is no longer a reference to a project that contains scala-library and scala-compiler. If dbuild cannot find a custom scala-library and scala-compiler as part of the build, it cannot adjust scalaVersion and scalaHome in order to refer to them, and it will just use the scala version defined by each project. So, you are not really testing the 2.13.x head anymore.

@cunei right, I'd figured that out, so the fix I'm trying is to set the Scala version in vars.base.extra.commands. you think it's better to just bring back assemble?

I've reverted b31ba8e27da9133cc72ddd81a485ce55ea86db63


Sidebar

Possible to have the client remember if you had the sidebar closed when last opened? Currently, when booting up the client, the sidebar is open/visible by default. Could it be done that if you have the sidebar open when you close the client, it will be open the next time it is booted up, and the same if you had it closed?
It's a little thing but closing the sidebar upon launching everytime is somewhat annoying as I never play with it visible, would be nice to remember the preferences of the user.

Duplicate of #2312


Add support for jnp.linalg.slogdet

Could slogdet() please be added? It seems to exist in JAX (https://jax.readthedocs.io/en/latest/_autosummary/jax.numpy.linalg.slogdet.html) and pytorch (https://pytorch.org/docs/stable/generated/torch.linalg.slogdet.html).

It appears to be built-in in all backend frameworks so that's an easy addition.

Done


added preloader

Description
It consists the code of adding a responsive preloader

This PR does the following:

Adds preloader

Related Issues
Fixes: #143
Changes

Added a new feature to frontend

Testing Instructions
Detailed instructions on how to test the changes. Include any setup needed and specific test cases.

Pull this branch.
Run npm install to install dependencies.
Run npm test to execute the test suite.
Verify that preloader is working o not

Checklist
Make sure to check off all the items before submitting. Mark with [x] if done.

[x] I have performed a self-review of my code
[x] I have commented my code, particularly in hard-to-understand areas
[x] My changes generate no new warnings
[x] I am working on this issue under GSSOC

@Jagpreet153 could change colour of preloader image to match with ui.

@Satyam1923 Check it now


Make Windows target explicit

It seems obvious, but I think Windows should be mentioned somewhere? :-)

Yeah, makes sense although I'm struggling with porting the script to Wine, which would make it even for linux. For now, however, there are missing important parts of Batch console/interpreter(?) - delayed expansion (that thing with !variable!), so... yeah, Win-only for now, thanks :P


feature: add Binary dtype

Have you tried latest version of polars?

yes

If the problem was resolved, please update polars. :)
What version of polars are you using?
0.7.4
What operating system are you using polars on?
Mac OSX 13.2.1 and Amazon Linux (AWS Lambda)
What node version are you using
node v16.19.1
Describe your bug.
The list of datatypes is incomplete. I see the python version has a binary type.
What are the steps to reproduce the behavior?
import * as pl from 'nodejs-polars';

pl.Series('bytes', Uint8Array.from([]), pl.DataType.Binary) // Does not compile

What is the actual behavior?
Uint8Array is cast to List(Uint8) which is not the same in Parquet and Arrow. We're trying to read the Parquet files in Snowflake but the binary columns end up as empty objects as Snowflake is confused by this custom List type.
What is the expected behavior?
A Binary type that adheres to Arrow and Parquet specs.

yes I agree this should be supported.
node polars is a bit behind the python implementation now, and it does not yet have theBinary dtype.

@flow-danny This code compiles in 0.8.1, can you please try it? Thx

This code works fine in 0.8.2, please re-open is this still an issue. Thx
import pl from 'nodejs-polars';
pl.Series('bytes', Uint8Array.from([]), pl.DataType.Binary);


Fetch Manifest TSV Full does not error if null is provided for a project UUID.

I am expecting that the project UUID is validated as a proper UUID that corresponds to an existing project.
Request
https://service.explore.data.humancellatlas.org/fetch/manifest/files?filters={"projectId"%3A{"is"%3A[null]}}&format=full
Response
{"Status": 301, "Location": "https://service.explore.data.humancellatlas.org/fetch/manifest/files?token=eyJleGVjdXRpb25faWQiOiAiMmQxMGZjYjMtOTgyYi00NmQ2LTkzODEtMGEyNGQ1YWEyYjYwIiwgInJlcXVlc3RfaW5kZXgiOiAxfQ==", "Retry-After": 1}
There is a bug in the browser project detail page that exposes this bug on the server. We will stop sending null as the project UUID ASAP but wanted to record that we currenly get a TSV back if we send  null as the project ID.

I mean to open this in azul. @theathorn is it easy to move?

Moved to https://app.zenhub.com/workspaces/orange-5d680d7e3eeb5f1bbdf5668f/issues/databiosphere/azul/1265


ResourceExhaustedError just to generate from pretrained model in NSynth

I use a Tesla V100 16GB and I want to generate new sounds using the pre-trained WaveNet model using This Method in NSynth.
However, even after I decreased the batch size to 1, the model requires way more memory to be available than a normal GPU, and throws a ResourceExhaustedError. (I confirmed that the GPU had all of its memory available before the operation.)
Are there any ways or workarounds for the sound generation to be available to GPUs with VRAM that are even smaller than 16GB (most consumer GPUs) and so?

Hi, I'm sorry but I'm not able to reproduce your error. I can run the code locally on a Titan X (12GB). Also, Also, the EZ NSynth Colab (https://colab.sandbox.google.com/notebooks/magenta/nsynth/nsynth.ipynb) runs the same methods on a Colab GPU, which I believe has <= 16GB.

Did a little more sleuthing. If your command is nsynth_generate --source_path=/path/to/file.wav you need to also specify --sample_length=number_of_audio_samples  where number of audio samples is >= 16000 * number of seconds. Otherwise it runs out of memory on the CPU. I'll submit a PR to avoid this.


Stm32F7: add external ram config

Configuration is improperly scoped, improperly name, and in the wrong Kconfig file.
Move unused configuration option to the /board/arm/stm32ft/your-board/Kconfig
Rename the option to CONFIG_YOURBOARD_EXTERNAL_RAM

I would like to insist to apply this patch as is.

STM32F7_EXTERNAL_RAM will select ARCH_HAVE_HEAP2, which is used in arch/arm/src/stm32f7/stm32_allocateheap.c. And there is no other way to select ARCH_HAVE_HEAP2. Now it is not obvious that stm32f7 has already support for second heap in arch/arm/src/stm32f7/stm32_allocateheap.c
Stm32 and stm32H7 aleady have the same options: STM32H7_EXTERNAL_RAM and STM32_EXTERNAL_RAM. For the sake of uniformity Stm32F7 should have the same option. Otherwise it is frustrating when migrating from stm32f4 to stm32f7 and not finding such a config.

Okay.  I think it is generally bad usage of Kconfig variables.  But there seems to be a precedence.

Thanks for the update.  I am supposed to wait for all of the PR-triggered checks to complete, then I will (squash) merge your changes.


Bug: Trying to set a user's role through the api doesn't seem to work. Defaults to 'student' even if 'admin' is sent.

fixed in PR #141


Duplicate pages, especially for translations

Hello,
Thank you very much for this package (and all the others) that help me build a headless CMS.
For the construction of my pages, I often use "flexible content".
When I have a page in English and I want to translate it into another language, my structure is not taken over. Same for some non-translatable fields (ex: images).
Is there a way with your package to "duplicate" the current content when creating a new translation? In such a way that my structure would remain the same and the images would not have to be added again.
Thanks for your help

Hey! Currently, it's not possible. However, I am working on a new major release of the page manager which would add that functionality.

Great, thank you @Tarpsvo. I am happy to see this new version.
Do you have an idea (without pressure) of when it will be available? Thank you


Client vs. Server - is version 2 logging an unnecessary QueryClientEvaluationWarning?

I have a Client vs. Server query that uses string.Join to combine some of the data as it is returned. Based on ver1 I though this was a 'safe' Client vs. Server  query because no QueryClientEvaluationWarning was logged. What I swapped to EF Core 2.0.0 then a QueryClientEvaluationWarning IS logged for the same query. Either:

You have improved the handing of this type of query in 2.0.0. and this query isn't now efficient.
EF Core 2.0.0. is logging an unnecessary QueryClientEvaluationWarning.

I build a version of the query without the string.Join and the SQL isn't any more efficient, so I assume that EF Core 2.0.0 is logging an unnecessary QueryClientEvaluationWarning.
The logging output (LogLevel.Information or above) from EF Core 2.0.0 is as follows
The LINQ expression 'join Author y.Author in value(Microsoft.EntityFrameworkCore.Query.Internal.EntityQueryable`1[DataLayer.EfClasses.Author]) on Property([y], "AuthorId") equals Property([y.Author], "AuthorId")' could not be translated and will be evaluated locally.
Executed DbCommand (0ms) [Parameters=[], CommandType='Text', CommandTimeout='30']
SELECT "x"."Title", "x"."BookId"
FROM "Books" AS "x"
Executed DbCommand (0ms) [Parameters=[@_outer_BookId='?'], CommandType='Text', CommandTimeout='30']
SELECT "b"."BookId", "b"."AuthorId", "b"."Order"
FROM "BookAuthor" AS "b"
WHERE @_outer_BookId = "b"."BookId"
Executed DbCommand (0ms) [Parameters=[], CommandType='Text', CommandTimeout='30']
SELECT "y.Author"."AuthorId", "y.Author"."Name"
FROM "Authors" AS "y.Author"
Executed DbCommand (0ms) [Parameters=[@_outer_BookId='?'], CommandType='Text', CommandTimeout='30']
SELECT "b"."BookId", "b"."AuthorId", "b"."Order"
FROM "BookAuthor" AS "b"
WHERE @_outer_BookId = "b"."BookId"
Executed DbCommand (0ms) [Parameters=[], CommandType='Text', CommandTimeout='30']
SELECT "y.Author"."AuthorId", "y.Author"."Name"
FROM "Authors" AS "y.Author"

Steps to reproduce
I have a Book entity class with a Many-to-Many relationship to an Author entity class, via a BookAuthor entity class linking table. I am simply trying to get a comma separated list of all the authors of a book.
I have a Book entity class with a collection of Review entity classes that contain NumVotes. I want to calculate the average number of votes. A Book can have zero to many Reviews.
My entity classes are:
public class Book
{
    public int BookId { get; set; }
    public string Title { get; set; }
    public string Description { get; set; }
    public DateTime PublishedOn { get; set; }
    public string Publisher { get; set; }
    public decimal Price { get; set; }
    public string ImageUrl { get; set; }

    //----------------------------------------------
    //relationships

    public PriceOffer Promotion { get; set; }
    public ICollection<Review> Reviews { get; set; }
    public ICollection<BookAuthor> AuthorsLink { get; set; }
}

public class Author
{
    public int AuthorId { get; set; }
    public string Name { get; set; }

    //------------------------------
    //Relationships

    public ICollection<BookAuthor>
        BooksLink { get; set; }
}

Linking Table is
public class BookAuthor
{
    public int BookId { get; set; }
    public int AuthorId { get; set; }
    public byte Order { get; set; }

    //-----------------------------
    //Relationships

    public Book Book { get; set; }
    public Author Author { get; set; }
}

The test code is shown below. EF Core 2.0.0 logs the QueryClientEvaluationWarning shown at the top of this comment. If run on EF Core 1.x or 2.0.0-preview1-final then a QueryClientEvaluationWarning is NOT logged.
Note: The method TwoBooksWithAuthors creates two books, the first with one Author, the second with two Authors.
private class Dto
{
    public string Title { get; set; }
    public string AuthorsString { get; set; }
}

[Fact]
public void AuthorsStringTwoBooks()
{
    //SETUP
    var options = SqliteInMemory.CreateOptions<EfCoreContext>();

    using (var context = new EfCoreContext(options))
    {
        context.Database.EnsureCreated();

        context.AddRange(SeedData.TwoBooksWithAuthors());
        context.SaveChanges();
        var logs = context.SetupLogging();

        //VERIFY
        var result = context.Books.Select(x => new Dto
        {
            Title = x.Title,
            AuthorsString = string.Join(", ", x.AuthorsLink.Select(y => y.Author.Name))
        }).ToList();

        //VERIFY
        result.First().AuthorsString.ShouldEqual("Author01");
        result.Last().AuthorsString.ShouldEqual("Author02a, Author02b");
        foreach (var log in logs)
        {
            _output.WriteLine(log);
        }
    }
}

If EF Core 2.0.0 is logging a QueryClientEvaluationWarning when it is not needed, then that will cause problems, especially if the developer has enable the throwing of an exception on a QueryClientEvaluationWarning
Further information
The log output from EF Core 2.0.0-preview1-final  (i.e. the one that works) is as follows
Executed DbCommand (0ms) [Parameters=[], CommandType='Text', CommandTimeout='30']
SELECT "x"."Title", "x"."BookId"
FROM "Books" AS "x"
Executed DbCommand (0ms) [Parameters=[@_outer_BookId='?'], CommandType='Text', CommandTimeout='30']
SELECT "y.Author"."Name"
FROM "BookAuthor" AS "y"
INNER JOIN "Authors" AS "y.Author" ON "y"."AuthorId" = "y.Author"."AuthorId"
WHERE @_outer_BookId = "y"."BookId"
Executed DbCommand (0ms) [Parameters=[@_outer_BookId='?'], CommandType='Text', CommandTimeout='30']
SELECT "y.Author"."Name"
FROM "BookAuthor" AS "y"
INNER JOIN "Authors" AS "y.Author" ON "y"."AuthorId" = "y.Author"."AuthorId"
WHERE @_outer_BookId = "y"."BookId"

Further technical details
EF Core version: 2.0.0
Database Provider: Microsoft.EntityFrameworkCore.Sqlite (also goes wrong on SqlServer
Operating system: Windows 10
IDE: VS2017 15.3

We have worsened the query here. It is N+1 query with N queries have translatable join but now we are not translating it anymore. Further we are not doing outer parameter on 2nd level navigation.
This could be related to our N+1 optimizations.

I have some more (useful?) information of this Client vs. Server Evaluation. I am now working on performance tuning and started to look at the SQL the expression shown in my first comment produces very significantly poorer code than version 1.1.0 and 2.0.0-preview1-final.
The SQL that EF Core version 2.0.0 produces is as follows:
Note the second SQL command which loads ALL authors rather that the Authors defined by the BookAuthor linking table, and is called for EVERY book that needs the AuthorsString.
SELECT [b0].[BookId], [b0].[AuthorId], [b0].[Order]
FROM [BookAuthor] AS [b0]
WHERE @_outer_BookId = [b0].[BookId]

SELECT [q.Author].[AuthorId], [q.Author].[Name]
FROM [Authors] AS [q.Author]

The correct SQL code, which version 1.1.0 and 2.0.0-preview1-final produce, is as follows
SELECT [q.Author].[Name]
FROM [BookAuthor] AS [q]
INNER JOIN [Authors] AS [q.Author]
ON [q].[AuthorId] = [q.Author].[AuthorId] WHERE @_outer_BookId = [q].[BookId]
ORDER BY [q].[Order]

Clearly the 2.0.0 the problem is not that it logs a QueryClientEvaluationWarning, but that it produces very inefficient code.

@JonPSmith I'm trying to reproduce the issue but I'm getting the expected sql:
SELECT [y.Author].[Name]
FROM [BookAuthors] AS [y]
INNER JOIN [Authors] AS [y.Author] ON [y].[AuthorId] = [y.Author].[AuthorId]
WHERE @_outer_BookId = [y].[BookId]

I'm not sure if we fixed the issue as a by-product of another fix or if I'm doing something wrong. Can you provide full code listing you were using to reproduce it?

I verified that the repro I used was producing a bad sql for 2.0.0, but it's producing a correct one for the current bits, closing as fixed


Is there any typo in the model https://huggingface.co/listen2you002/ChartLlama-13b/tree/main?

Hello. Thank you for releasing all these amazing work.
I am encountering a ClipVision issue. When I try to load the Lora weigths from ChartLlama-13b model, the "mm_vision_tower": "/mnt/share_1227775/yandali/multimodal/models/ft_local/clip-vit-large-patch14-336/" was n't accessiable. I changed it to the openai/clip-vit-large-patch14-336, but then it has another problem of generating texts.
I am wondering is there any typo or issues here? Maybe I am wrong. Canyou help to fix it?
Thank you so much

I faced the same issue. Did you resolve it?

How I fixed the issue:

First, download the listen2you002/ChartLlama-13 model from Hugging Face locally.
Next, download the liuhaotian/llava-v1.5-13b model from Hugging Face locally.
Then, download the openai/clip-vit-large-patch14-336 model from Hugging Face locally.
Navigate to the folder where ChartLlama-13 was downloaded, and in the config.json, replace "mm_vision_tower" with the full path of clip-vit-large-patch14-336.
For inference, pass the local paths of both liuhaotian/llava-v1.5-13b and listen2you002/ChartLlama-13.

It was a mismatch of paths. Hope this helps!


question about PMKError remplacing any NSError subclass

Hi,
I noticed that the error I pass in a rejecter(error) get "simplified" to a PMKError class...
Hence only pure NSError can be handled. If we want to have our hierarchy of subclass of NSError they get tossed away right there, in PMKPromise.m, line 534-5 (on master 2015-01-21 16h40 GMT):
if ([foo isKindOfClass:[NSError class]])
return [PMKError errorWithDomain:[foo domain] ?: PMKErrorDomain code:[foo code] userInfo:[foo userInfo]];
Why that choice ? Sur UserInfo is kept. But then only pure NSError can be used with PromiseKit.
It just happens that we have a whole hierarchy of NSError subclass, business oriented. What can I do if I get steamrolled like that ? :)
Thanks

PMKError is not for “simplification” it allows reference counting the error consumption so that unhanded errors can be reported.
However indeed, we shouldn't lose the type of the error the user provides. I can look into fixing that.

Will push a new PromiseKit to cocoa pods this week.


Create STREAM frames on demand

A size of sending STREAM frame is calculated with some numbers (pmtu, flow control credit) prior to creating a packet. It makes sending the frame with other frames difficult because the frame fulfill a packet. More specifically, this causes ACK only packets, which shouldn't be sent in response to received ACK only packets.
Probably STREAM frames have to be created on demand when QUICNetVConnection packetize frames.

It's a partial change but now frames are created on demand. 6e5b664a31c54d939867469520f5609130193d8a
Let's do IOBuffer related changes on #2657.


GUI not opening on Windows 10

OS:
Win10
GPMDP Version:
4.6.1
Issue Descriptions:
GUI not opening when starting.  The app shows up on the task bar but I can not see the GUI.
Steps to Reproduce:
Just open app

It seems Google Music Manager (official) has the same problem (for me and some others, maybe google just messed up with their API.

Links for a more recent build that fixes this are in #3660.

Same problem happens on MacOS Mojave

Same issue with Windows 10 here.

Same issue with Windows 10 here.

FYI I've also tried both 4.6.0 and 4.5.0 and got the same results

Same.
Clearing %\AppData\Roaming\Google Play Music Desktop Player\ and re-login doesn`t help.

See #3660


TypeError: init() missing 1 required positional argument: 'name'

I am facing this error in while running the init.py file.
File "C:\Users\Sean Gray\Desktop\Python\KivySurvey-master\KivySurvey-master\kivy_survey_init_.py", line 355, in init
self.db_interface = DBInterface(self)
TypeError: init() missing 1 required positional argument: 'name'
Can't detect the error on my own.

Any help is appreciated


how to share blob files

What would you like to know?

Hey there, can you provide more details please

@VB-007 Please see Tip 225 - How to share your Azure Blob Storage securely with Azure Data Share | Azure Tips and Tricks (microsoft.github.io). Let us know if you have any additional questions. Thank you.


Panic: polars, sort on a list

Repro:
1)
import polars as pl
from bastionlab.polars.policy import Policy, TrueRule, Log

repro_df = pl.DataFrame({ "hello": [1, 2, 3], "world": [1, 2, 3] })
repro_rdf = connection.client.polars.send_df(repro_df, policy=Policy(safe_zone=TrueRule(), unsafe_handling=Log(), savable=False))

(
    repro_rdf
    .groupby(pl.col("hello"))
    .agg(pl.col("world")) # replace with .agg(pl.col("world")).sum() for it to work
    .sort(pl.col("world"))
).collect().fetch()

results in
thread 'tokio-runtime-worker' panicked at 'this operation is not implemented/valid for this dtype: List(Int64)', /home/cchudant/.cargo/registry/src/github.com-1ecc6299db9ec823/polars-core-0.25.1/src/series/series_trait.rs:427:9

This is inside polars, not sure how to proceed here

I believe the operation you're trying to perform does no really make sense: the aggregation step does not involve an aggregation function which means you get the list of values in the group for the "world" column, then you try to sort the same column which fails as polars doesn't know how to sort lists.
>>> import polars as pl
>>> df = pl.DataFrame({ "hello": [1,2,3], "world": [1,2,3]})
>>> df.lazy().groupby(pl.col("hello")).agg(pl.col("world")).collect()
shape: (3, 2)
┌───────┬───────────┐
│ hello ┆ world     │
│ ---   ┆ ---       │
│ i64   ┆ list[i64] │
╞═══════╪═══════════╡
│ 2     ┆ [2]       │
├╌╌╌╌╌╌╌┼╌╌╌╌╌╌╌╌╌╌╌┤
│ 3     ┆ [3]       │
├╌╌╌╌╌╌╌┼╌╌╌╌╌╌╌╌╌╌╌┤
│ 1     ┆ [1]       │
└───────┴───────────┘
>>> df.lazy().groupby(pl.col("hello")).agg(pl.col("world")).sort(pl.col("world")).collect()
thread '<unnamed>' panicked at 'this operation is not implemented/valid for this dtype: List(Int64)', /home/runner/work/polars/polars/polars/polars-core/src/series/series_trait.rs:427:9
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home/dhalf/Documents/bastionlab/env/lib/python3.10/site-packages/polars/internals/lazyframe/frame.py", line 920, in collect
    return pli.wrap_df(ldf.collect())
pyo3_runtime.PanicException: this operation is not implemented/valid for this dtype: List(Int64)

That said, we can probably improve error reporting.


Broken Link

Why:
Closes #15367
What's being changed:
Link doesn't work with the subject line automatically, trying to add a new link that will send user straight to a form that they can share the integration information with GitHub. If there is a better link I can switch it but this is the best one I found thus far.
Check off the following:

[ ] I have reviewed my changes in staging (look for "Automatically generated comment" and click Modified to view your latest changes).
[ ] For content changes, I have completed the self-review checklist.

Writer impact (This section is for GitHub staff members only):

[ ] This pull request impacts the contribution experience

[ ] I have added the 'writer impact' label
[ ] I have added a description and/or a video demo of the changes below (e.g. a "before and after video")

@BenjaminMichaelis Thanks so much for opening a PR! I'll get this triaged for review :zap:

Hi @BenjaminMichaelis, thanks for creating this PR. Could you describe a bit more or share a screenshot of the error you're encountering with the current link? I'm having trouble reproducing it, and when I navigate to the link, I get the form with a prefilled subject of "New GitHub Integration." Thank you!

Hi @BenjaminMichaelis, thanks for creating this PR. Could you describe a bit more or share a screenshot of the error you're encountering with the current link? I'm having trouble reproducing it, and when I navigate to the link, I get the form with a prefilled subject of "New GitHub Integration." Thank you!

Of course!

When clicking on this link, it looks like it is optimized to automatically populate a message to github with the subject "New GitHub Integration" as the link is https://github.com/contact?form[subject]=New+GitHub+Integration but this just sends you currently to https://support.github.com/request so I was trying to update the link to automatically pull up a contact form and populate the subject line, and the closest I got to this is the change I pushed which at least got the user to a contact form directly, just without the subject prefilled. This is to make it as easy as possible for the user!

@BenjaminMichaelis that is great context, thank you. ⚡
What browser are you using? I've tested on Chrome, Firefox, and Safari, but I am getting the autopopulated form like in the screenshot below.

That is interesting.... I am using chrome. Here is a video of my result.
https://user-images.githubusercontent.com/22186029/153952087-b4749689-c3a0-4c96-a9d4-c43695a55e25.mp4

Interesting indeed. Thanks for sharing that video. For now, I'm looping the Docs engineering team in on this issue to see what might be going on. I'll post an update here once I've got some more info.


Install example instance of django-machina to the chigame sandbox directory

The objective of this task is to set up a default example instance of the django-machina forum library, verify that it runs correctly, and upload it to the chigame repo's sandbox directory (exact location TBD). This example instance will serve as the basis of a mock implementation ( #149 ) of the chigame forum that will be used for testing and evaluation purposes ( #99 ).

Issue #174 has been opened to create the chigame sandbox directory.

The sandbox directory has been added and issue #174 has been closed. I've also added the django-machina demo to the sandbox and submitted a pull request to merge it into dev.

#204

Closed after successfully merging and closing PR #204 . Thanks to @Benjoman12 for reviewing the PR.

Issue Score: Excellent
Comments:
For future reference, if someone reviewed the PR for an issue, they should not also be assigned to that issue!
Great work! Love you expressing your appreciation in the closing comment :)


Pull request to add log driver options

This pull request aims to fixes the Cannot create container for service nginx-xxx: unknown log opt 'max-size' for journald log driver error on Centos 7 running docker v1.13.1 and docker-compose v1.21.2.
It adds the service/xxx/logging/driver option in the docker-compose*.yml files, as well as the following environment variables in the .env.sample file

NGINX_WEB_LOG_DRIVER
NGINX_GEN_LOG_DRIVER
NGINX_LETSENCRYPT_LOG_DRIVER

The default value for those variables is "json-file"

Relevance: I just ran into this error on CentoOS which puzzled me!

I will test it over the week and push to the repo, ok?
Thanks!


autoscale db dao - [closed]

In GitLab by @luotian-github on Sep 11, 2018, 07:33
Merges autoscale -> dev
autoscale db dao

In GitLab by @wknet123 on Nov 13, 2019, 11:20
closed


CLOUDSTACK-9304: Add nuagevsp userdata testcase (Cloudstack-9095) & Refactor existing testcases

https://issues.apache.org/jira/browse/CLOUDSTACK-9304

Created folder / test / integration / plugins / nuagevsp
Moved test_nuage_vpc_network.py , test_nuage_vsp.py from / test / integration / component /  to---> / test / integration / plugins / nuagevsp/
Added new testcase test_nuage_password_reset.py (Cloudstack-9095)
Added Nuage class in / tools / marvin / marvin / lib / base.py
Added services in / tools / marvin / marvin / config / test_data.py

results:
test_01_UserDataPasswordReset (nuagevsp.test_nuage_password_reset.TestNuagePasswordReset) ... === TestName: test_01_UserDataPasswordReset | Status : SUCCESS ===
ok
Test Basic VPC Network Functionality with NuageVsp network Plugin ... === TestName: test_nuage_vpc_network | Status : SUCCESS ===
ok
Test NuageVsp network plugin with basic Isolated Network functionality ... === TestName: test_nuage_vsp | Status : SUCCESS ===
ok

XML: /root/report_xunit.xml
PassLogs.zip
Ran 3 tests in 2406.256s
OK
#CLOUDSTACK-9304

LGTM based on the code review.

@sneeladh @shivsg  Thanks for adding the tests, can you please run your code through pyflakes, pep8 and fix them ?
$ pyflakes *.py
nuageTestCase.py:52: 'attr' imported but unused
nuageTestCase.py:324: undefined name 'startVirtualMachine'
nuageTestCase.py:328: undefined name 'FAIL'
nuageTestCase.py:335: undefined name 'stopVirtualMachine'
nuageTestCase.py:341: undefined name 'FAIL'
test_nuage_password_reset.py:21: 'VirtualMachine' imported but unused
test_nuage_password_reset.py:21: 'PublicIPAddress' imported but unused
test_nuage_password_reset.py:26: 'createTemplate' imported but unused
test_nuage_password_reset.py:26: 'listVolumes' imported but unused
test_nuage_password_reset.py:64: undefined name 'cleanup_resources'
test_nuage_password_reset.py:72: undefined name 'list_volumes'
test_nuage_password_reset.py:84: undefined name 'Template'
test_nuage_password_reset.py:118: local variable 'vm_public_ip' is assigned to but never used
test_nuage_password_reset.py:217: undefined name 'startVirtualMachine'

Pep8 and Pyflakes compliant
[sneeladh@mvlinux26:nuagevsp]$
[sneeladh@mvlinux26:nuagevsp]$ pep8  --max-line-length=150 test_nuage_password_reset.py
[sneeladh@mvlinux26:nuagevsp]$ pep8  --max-line-length=150 test_nuage_vpc_network.py
[sneeladh@mvlinux26:nuagevsp]$ pep8  --max-line-length=150 test_nuage_vsp.py
[sneeladh@mvlinux26:nuagevsp]$ pep8  --max-line-length=150 nuageTestCase.py
[sneeladh@mvlinux26:nuagevsp]$
[sneeladh@mvlinux26:nuagevsp]$
[sneeladh@mvlinux26:nuagevsp]$ pyflakes nuageTestCase.py
[sneeladh@mvlinux26:nuagevsp]$ pyflakes test_nuage_password_reset.py
[sneeladh@mvlinux26:nuagevsp]$ pyflakes test_nuage_vpc_network.py
[sneeladh@mvlinux26:nuagevsp]$ pyflakes test_nuage_vsp.py
[sneeladh@mvlinux26:nuagevsp]$
[sneeladh@mvlinux26:nuagevsp]$
[sneeladh@mvlinux26:nuagevsp]$ pyflakes *.py
[sneeladh@mvlinux26:nuagevsp]$
[sneeladh@mvlinux26:nuagevsp]$

Jenkins failure is not related to these integration test changes, there are some junit tests failed
[ERROR] Failed to execute goal org.apache.maven.plugins:maven-surefire-plugin:2.18.1:test (default-test) on project cloud-utils: There are test failures.
[ERROR]
[ERROR] Please refer to /home/jenkins/jenkins-slave/workspace/cloudstack-pr-analysis/utils/target/surefire-reports for the individual test results.

otherwise, it LGTM.

LGTM based on the test results
test_01_UserDataPasswordReset (nuagevsp.test_nuage_password_reset.TestNuagePasswordReset) ... === TestName: test_01_UserDataPasswordReset | Status : SUCCESS ===
ok
Test Basic VPC Network Functionality with NuageVsp network Plugin ... === TestName: test_nuage_vpc_network | Status : SUCCESS ===
ok
Test NuageVsp network plugin with basic Isolated Network functionality ... === TestName: test_nuage_vsp | Status : SUCCESS ===
ok

Ran 3 tests in 2477.667s
OK
runinfo.txt
results.txt

I am not going to revert this merge at this time, but please ping me if you feel the PR is ready to be merged into master.  Thx...

@swill Yes, PR is ready to be merged. Do you see any problem with that?

If the concern is jenkins build, I mentioned that they are not related to these integration test code but they are junit tests.

It was already merged.  I will leave it for now, but I will take note of this PR to validate it once my CI is up and running to make sure it does not cause any issues.

sure.


GSSOC '24 : EDA-and-Perform-Modelling-on-Ionosphere-Dataset

Project Description:
The following project aims to predict class using various technical specifications (features) as input to the logistic regression algorithms.
Database Description:
Number of Instances: 351
Number of Attributes: 35 including the class attribute
Attribute Information:
Target column :-
Class Feature Columns range- V1- V35
Libraries Involved:
pandas
Numpy
Seaborn
Matplotlib
Sklearn
scikit-plot
pingouin
Steps Involved:
Importing the libraries
Loading the dataset
Data Preprocessing
train and test data split
Building the model
Compare model performance
selection model based on performance
Evaluation
Plot ROC and AUC curve

The issue has been assigned to you @sujanrupu.
Do follow the project guidelines for PR.

I would like to work on this project, could you please assign this to me too?


Update README.md

What does this PR do?
Add a note that install takes a long time.
Before submitting

[x] This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).

LGTM! (I assume you were able to run it?)


chore: bump ui-react and add ui-storage dep warning

Problem

Amplify UI released a new package for StorageManager (formerly FileUploader)
Solution

Bump up ui-react and add dependency warning for customers to install the new package in order to use StorageManager
Additional Notes

Links
Ticket

GitHub issue _____
Other links
Verification
Manual tests

Automated tests

[ ] Unit tests added/updated
[ ] E2E tests added/updated
[x] N/A - just bumping dependencies and adding a new warning message
[ ] deferred - (provide GitHub issue for tracking)

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

Codecov Report

Merging #977 (aff4cb8) into main (7299042) will not change coverage.
The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

@@           Coverage Diff           @@
##             main     #977   +/-   ##
=======================================
  Coverage   93.02%   93.02%
=======================================
  Files         120      120
  Lines        5293     5293
  Branches     1578     1578
=======================================
  Hits         4924     4924
  Misses        348      348
  Partials       21       21

Impacted Files
Coverage Δ

...ui-react/lib/react-required-dependency-provider.ts
100.00% <ø> (ø)

Continue to review full report in Codecov by Sentry.

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


Perform a batch operation on the built automata?

Hi! Thanks for the library!
I need to perform a batch operation for a given string s1 and a set of strings s2a, s2b, s2c from stdin. The way I understand the library, there's the "expensive" single-use build_dfa function and then distance can be measured. I failed to find a way to use it in mass operations, so I decided to apply a following patch to the library:
diff --git a/src/lib.rs b/src/lib.rs
index 9220461..26b0932 100644
--- a/src/lib.rs
+++ b/src/lib.rs
@@ -43,7 +43,7 @@ mod bench;
 mod tests;

 mod alphabet;
-mod dfa;
+pub mod dfa;
 mod index;
 mod levenshtein_nfa;
 mod parametric_dfa;

And then created src/main.rs:
use levenshtein_automata::{LevenshteinAutomatonBuilder, Distance};
use std::io::{self, BufRead};
use std::env;

fn measure_distance(dfa: levenshtein_automata::dfa::DFA, x: &str) -> Distance {
    let mut state = dfa.initial_state();
    for &b in x.as_bytes() {
        state = dfa.transition(state, b);
    }
    dfa.distance(state)
}

fn main() {

    let argv1 = env::args().next().unwrap();
    println!("{:?}", argv1);

    let lev_automaton_builder = LevenshteinAutomatonBuilder::new(2, true);
    let dfa = lev_automaton_builder.build_dfa(&argv1);
    let stdin = io::stdin();
    for result in stdin.lock().lines() {
        let line = result.unwrap();
        println!("{:?}, {:?}", line, measure_distance(dfa, &line));
    }
}

The problem I'm getting is the following compilation error:
   Compiling levenshtein_automata v0.2.0 (/home/d33tah/workspace/profound/tmp/experiments/levenshtein/levenshtein-automata)
error[E0382]: use of moved value: `dfa`
  --> src/main.rs:23:55
   |
19 |     let dfa = lev_automaton_builder.build_dfa(&argv1);
   |         --- move occurs because `dfa` has type `levenshtein_automata::dfa::DFA`, which does not implement the `Copy` trait
...
23 |         println!("{:?}, {:?}", line, measure_distance(dfa, &line));
   |                                                       ^^^ value moved here, in previous iteration of loop

error: aborting due to previous error

Does the interface need additional modifications or am I using it incorrectly? How could I modify either so that it would fit my needs?

Never mind. I was just missing a reference in my example.


Automation of "Setting up Application Gateway Ingress Controller on AKS"

Hi Team,
I am trying to do automation and have deployed application gateway as a ingress controller using ARM template. Now Application gateway , AKS and managed identity is creating properly but how to do this steps which are mention in "Setting up Application Gateway Ingress Controller on AKS" tutorial from VSTS pipeline?
can you please provide any sample pipeline where it has done those steps?

Thanks for asking. Investigating.

Added a CI/CD document for AzureDevOps
Closing.


docs: typo motiviate -> motivate

We're doing some rework on these tutorial chapters. Holding off on accepting this one in case it becomes irrelevant due to a rewrite of that section. If it stays relevant, we'll merge it. Thanks for pointing it out.

We fixed in the forthcoming massive update to ToH.  Thanks for catching it.


Exercise 4.1 (remote service) step 4 does not work

The instructions state to add the api key to env/.env2 however this does not work and leads to 401 errors.
The only way I cold get this to work is to create a top-level project .env file and add the api key like this:
cds.requires.BusinessPartnerA2X.credentials.headers.APIKey=<your api key>

I also notice that Build Code includes the API key automatically into  env/.env2 but its also not picked up by cds watch.
So just how should this be done.
Also - it would be good to show that simply forward the READ handler for BusinessPartner is no where near enough as it will not work with expands and the code can start to get quite ugly quickly. Joule needs to be able to handle this.

I managed to make it work by putting the apikey on the other env file, the first one.


Handle queue not existing correctly

@CarlosGamero I've restored the lint rule: https://github.com/kibertoad/message-queue-toolkit/pull/95/commits/06795a5553a5247f525e8a7257b8209e7400ea6e

Thank you so much Igor 🙌


Debian OS origin string

Affected Puppet, Ruby, OS and module versions/distributions

Puppet: all
Ruby: all
Distribution: all
Module version: v4.0.0

How to reproduce (e.g Puppet code you use)
Include the unattended upgrades class without additional parameters.
class profile::debian {
  include unattended_upgrades
}

What are you seeing
https://github.com/voxpupuli/puppet-unattended_upgrades/blob/v4.0.0/manifests/params.pp#L71
This line sets the origins for Debian 9 stretch to origin=Debian,archive=stable,label=Debian-Security. However, as of July 2019, Debian 9 stretch is no longer 'stable' but 'oldstable' and Debian 10 buster is now stable.
What behaviour did you expect instead
The origin string for Debian 9 stretch should be origin=Debian,archive=oldstable,label=Debian-Security
Output log
N/A
Any additional information you'd like to impart
I believe this is a simple fix of changing the names in the manifests/params.pp file. However, one may want to consider using the distribution codename in the string which may be more resistant to changes, i.e., when Debian upgrades what is considered 'stable'.
See these commits  for the last time this changed (2017): c20f1fcc99b207fd3cb9a8b08d7039b0128c2fef & 5fe39f89506c5737fcfa90efbbea43b10fbd88f5; also PR #102 .

I can confirm this behavior through all of my systems, mainly debian 9 stretch, as well.
I quickly checked that this will work on stretch as expected, the suggested switch to package filtering by codename sounds like a good idea:
"origin=Debian,codename=stretch,label=Debian-Security"
If we don't do this, it will break again in the future.
What I can't check if this will work on all other supported systems though. We should get someone with other/older debian and ubuntu versions to also check if the codename filter works there as well.
After we have all confirmations, we can adjust the already existing structure in params.pp to set codename where possible, and leave everything else in the old style.
Also if you want to find out about the naming of an apt source, information can be found in /var/lib/apt/lists.
Example:
# head /var/lib/apt/lists/security.debian.org_dists_stretch_updates_InRelease

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA256

Origin: Debian
Label: Debian-Security
Suite: oldstable
Version: 9
Codename: stretch
Date: Wed, 14 Aug 2019 06:21:29 UTC
Valid-Until: Sat, 24 Aug 2019 06:21:29 UTC

The attached PR should fully cover this issue and also move to codename based filtering where possible (jessie, stretch, buster), tests have been adapted, and some validation for codename based filtering provided in my comment above.
Not sure if there is any process for me to follow now for the PR to get reviewed/merged?


task(countDocuments): full test coverage for countDocuments

Add tests that were node-specific or missing from spec tests.
Fixes NODE-1559

Let me know if there are other cases to test that I missed!


[FLINK-24235][metrics] Only support reporters via factories

Following the deprecation in 1.16 this PR removes the reflection-based code path for instantiating reporters.

@flinkbot run azure