Unable to deploy cert-manager

Is this a BUG REPORT or FEATURE REQUEST?:
/kind bug
What happened:
I try to install cert-manager by helm with helm install --name cert-manager --namespace kube-system contrib/charts/cert-manager
But I have a message : file  contrib/charts/cert-manager not found
What you expected to happen:
Deploy cert-manager into kubernetes 1.8.5 cluster
How to reproduce it (as minimally and precisely as possible):
Install kubernetes 1.8.5 with kubeadm,
Install helm with sh script.
Install cert-manager from helm
Anything else we need to know?:
Environment:

Kubernetes version (use kubectl version): 1.8.5
Cloud provider or hardware configuration**: my own server (vps)
Install tools: helm
Others:

To be clear, since your steps don't include it, did you clone this git repo locally or download and extract a source tarball from the release tab, and then run the helm command from within that directory?
The user-guide page for helm doesn't make that clear, but I believe it's expected to be deployed in that manner for now (until a helm chart is published to the upstream charts repo)

Yeah our docs really could be clearer here - I intend to make a PR against
the main chart repo soon which should alleviate these issues! In the
meantime, cloning or downloading a release tarball is required!
On Sun, 10 Dec 2017 at 12:16, Euan Kemp<EMAIL_ADDRESS>wrote:

To be clear, since your steps don't include it, did you clone this git
repo locally or download and extract a source tarball from the release tab,
and then run the helm command from within that directory?
The user-guide page for helm doesn't make that clear, but I believe it's
expected to be deployed in that manner for now (until a helm chart is
published to the upstream charts repo)
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/jetstack/cert-manager/issues/226#issuecomment-350567295,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAMbP-w6lNzck0jfeJQC34dide2mXh5Nks5s_CACgaJpZM4Q8bX5
.

Thanks for your answers !  I clone the repo, and try to re install it, but now I think it is a rbac issue
Here the message :
Error: release cert-manager failed: namespaces "kube-system" is forbidden: User "system:serviceaccount:kube-system:default" cannot get namespaces in the namespace "kube-system"

It is an issue with helm, I fix it.
So I install cert-manager


Create ja.json

Hi @shirutan,
thank you for your PR! I incorporated your translations into version 1.4.0 of ToolUpgradeDeliveryService.


Improve the speed of UTxO queries

In order to find the UTxO or the balance of an address currently it is necessary to run a query that joins inputs and outputs. This could change if the output table itself had a new column which indicated whether an entry is consumed or not. This column can't simply be a flag, it needs to bear more info, like the exact blockNo which consumed the output. That way on rollbacks outputs that are "unconcumed" can return to their initial state.

Adding a blockComsumed column which would be a Maybe Word64 might make some sense. It goes against the original idea of making the database append only but it would drastically improve the speed of certain queries so much that its probably worth it. It would also be pretty trivial to write a migration to fill in this column without dropping and resyncing.

I do not think this is quite true:

It's normal to have sequential tx_in since we want to find all consuming inputs

IN the case of a rollback of one or more blocks, the blocks and all associated transactions will be deleted.  When new block are inserted, the ids of the blocks that have been deleted will be skipped and not reused.

This seems to considerably delay sync time. The migration from synced db state also takes many hours (up to 20h on a high end pc). So probably it won't be included in the default db-sync schema.
This could work though as an migration extention. Users will be able to run it using
cardano-db-tool -- run-migrations --tx_out_extention on an empty db or
cardano-db-tool -- tx_out_extention to migrate an existing db
Normal db-sync will be able to see the existence of the tx_out.consumed_by_tx_in_id column, validate that all values are updated and adjust the insertion queries for schema.


"dependentRequired" did not work

Hi there.
It seems the validation keyword for objects "dependentRequired" did not work,
I tried to find this keyword from source code but got nothing.
I just wanna make sure if this keyword has been implemented? (glad to contribute btw.)

Hi there! Yeah, dependentRequired is a draft2019-09+ validator, which isn't yet implemented. Certainly could use help if you'd like! There's extremely bare initial work on the draft2019-09 branch, if you had incremental small improvements we could probably look at what's there together?
Thanks!


zlib build failure

Host Environment

OS: windowx-x64
Compiler: MSVC 2019

To Reproduce
Steps to reproduce the behavior:
./vcpkg install zlib
Failure logs
-- Downloading https://repo.msys2.org/mingw/i686/mingw-w64-i686-pkg-config-0.29.
2-1-any.pkg.tar.xz...
-- Downloading https://repo.msys2.org/mingw/i686/mingw-w64-i686-pkg-config-0.29.
2-1-any.pkg.tar.xz... Failed. Status: 35;"SSL connect error"
CMake Error at scripts/cmake/vcpkg_download_distfile.cmake:181 (message):
  Failed to download file.
  If you use a proxy, please set the HTTPS_PROXY and HTTP_PROXY environment
  variables to "https://user:password@your-proxy-ip-address:port/".
  Otherwise, please submit an issue at https://github.com/Microsoft/vcpkg/is

sues
Call Stack (most recent call first):
scripts/cmake/vcpkg_find_acquire_program.cmake:438 (vcpkg_download_distfile)
scripts/cmake/vcpkg_fixup_pkgconfig.cmake:270 (vcpkg_find_acquire_program)
ports/zlib/portfile.cmake:48 (vcpkg_fixup_pkgconfig)
scripts/ports.cmake:79 (include)
Additional context

Hi @Keithcat1
Thanks for posting this issue.
Could you please help confirm if you can download
https://repo.msys2.org/mingw/i686/mingw-w64-i686-pkg-config-0.29.2-1-any.pkg.tar.xz
manually?

I was able to download it just now, but this a different computer on a different network

Yes, this is related with your network.

Same issue,

Finally had time to check this, sorry.
When I click the link you provided, my browser says this:
repo.msys2.org took too long to respond.
This error is also stopping me from building the SDL2 package, looks like it's a more general error.


Refactoring attribute module

Related issues
closes #3337
Short description and why it's useful
Attribute module refactored
Which environment this relates to

[x] Test version (https://test.storefrontcloud.io) - this is a new feature or improvement for Vue Storefront. I've created branch from develop branch and want to merge it back to develop
[x] RC version (https://next.storefrontcloud.io) - this is a stabilisation fix for Release Candidate of Vue Storefront. I've created branch from release branch and want to merge it back to release
[ ] Stable version (https://demo.storefrontcloud.io) - this is an important fix for current stable version. I've created branch from hotfix or master branch and want to merge it back to hotfix

Upgrade Notes and Changelog

[x] No upgrade steps required (100% backward compatibility and no breaking changes)
[ ] I've updated the Upgrade notes and Changelog on how to port existing VS sites with this new feature

Contribution and currently important rules acceptance

[x] I read and followed contribution rules
[x] I read the TypeScript Action Plan and adjusted my PR according to it
[x] I read about Vue Storefront Modules and I am aware that every new feature should be a module

Thanks! @alinadivante ive merged in to develop so you could test how the attributes are being loaded on the develop branch. The desired dstate is that after first entering product page then, the second time there is no request to attribute/list endpoint in the network tab and the message “Skipping attribute ...” is being displayed in console; that means the attributes are being loaded from the cache and no network request is needed

Thanks for the tips @pkarw ! I will start testing on Friday :)


Worker name not updated in GPUStack when changing worker name after registration

Version: 0.3.2
Step to reproduce:

macOS env. Start one server and register one worker.
Change worker name and restart.
Worker name not updated at GPUStack.

Worker name is immutable by design. You can delete the legacy worker entity in the UI.


Getting Decompression("incorrect magic number") error

Hi,
I am first time user of your lib. I am getting "Error: Decompression("incorrect magic number")" whenever I try below code,
let client = Client::default().with_url("http://clickhouse.mytest.net:8123");
let count = client
    .query("SELECT count() FROM mytable")
    .fetch_all::<u64>()
    .await?;

When I try the link http://clickhouse.mytest.net:8123/play in Chrome, it loads properly. So don't know what is the error. Can you help me?

It is working now. I should not have included play in the url. Thanks a lot


Updates to docs for include_in_copy

Follow up to https://github.com/gruntwork-io/terragrunt/pull/1935

Thanks for review! Merging now.


README.md should include link to ReadTheDocs

Under usage, we should add a link to ReadTheDocs for further information.

added


chore(deps): undo bump @backstage/plugin-techdocs-backend to 1.10.13

Description
corresponding pr: https://github.com/janus-idp/backstage-showcase/pull/1752/files
Which issue(s) does this PR fix

Fixes #? https://issues.redhat.com/browse/RHIDP-4116

PR acceptance criteria
Please make sure that the following steps are complete:

[ ] GitHub Actions are completed and successful
[ ] Unit Tests are updated and passing
[ ] E2E Tests are updated and passing
[ ] Documentation is updated if necessary (requirement for new features)
[ ] Add a screenshot if the change is UX/UI related

How to test changes / Special notes to the reviewer

/lgtm
/approve


Allowing modification of learning rate for mu

In Gaussian visible layer, mu should be able to have a learning rate different from 1/sharing.

Thanks.
Can you add (or expand existing) unit tests to check that the options are correctly recognized and the behaviour is as expected?

I've added a test for learning rates : #1460. It makes sure the learning rate for mu is implemented in the same way it is for beta. It runs fine on a dataset but I haven't put that in the test.


[RHOAIENG-6519] DW: Support workloads owned by either a RayCluster or a Job when querying CPU and Memory usage

Fixes the blocker https://issues.redhat.com/browse/RHOAIENG-6519
Description

We made the incorrect assumption that all Kueue Workload CRs will have an ownerReference to a Job CR which we can reference in a Prometheus query to get CPU and Memory usage for Jobs. In reality, when Kueue workloads are created from RayCluster, no Job CR gets created and the RayCluster itself is the owner. This meant our existing Prometheus query was not sufficient for getting CPU and memory usage for all workloads.
@Fiona-Waters and @KPostOffice, among others, worked out that we can use a different Prometheus query based on kube_pod_owner to include metrics from pods owned by both RayClusters and Jobs. This meant updating the UI logic so we index these usage values not just based on the job name, but based on the owner kind and owner name (in case there is a RayCluster and a Job with the same name).
This change adjusts logic that was based on job name to handle this extra dimension of first indexing by owner kind, and considers ownerReferences of a workload that are of either RayCluster or Job kind instead of just Jobs.

How Has This Been Tested?

Tested in local dev env while connected to a QE cluster with running RayClusters and Jobs. Will test an image on a cluster when this PR builds one.
Test Impact

Unit and cypress tests have been updated to cover the changes and make sure RayClusters and Jobs are both covered in the mock data.
Request review criteria:

Self checklist (all need to be checked):

[x] The developer has manually tested the changes and verified that the changes work
[x] Commits have been squashed into descriptive, self-contained units of work (e.g. 'WIP' and 'Implements feedback' style messages have been removed)
[x] Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
[x] The developer has added tests or explained why testing cannot be added (unit or cypress tests for related changes)

If you have UI changes:

[ ] Included any necessary screenshots or gifs if it was a UI change.
[ ] Included tags to the UX team if it was a UI/UX change (find relevant UX in the SMEs section).

After the PR is posted & before it merges:

[ ] The developer has tested their solution on a cluster by using the image produced by the PR to main

Self-approving with @andrewballantyne's permission. This has been reviewed by @gitdallas and I've verified the fix on the cluster where the issue was initially identified by @Srihari1192


CXBOX-450 | updated ui

done in https://github.com/CX-Box/cxbox-code-samples/pull/264, so i close this pr


Mobile version

HTML/CSS & JavaScript capstone project - Mobile Version

Set up a repo for the capstone project.
Choose a topic for my website.
Add index.html, style.css, and script.js files for the main page.
Add about.html and about.css files for the about page.
Add content for both pages in the mobile version.
Deploy the project to GitHub.

Build with

HTML and CSS

Okay, sorry for the inconvenience. Thanks & Regards. I'll complete all the
milestones and I'll request the review. 👍🙏
On Tue, Nov 16, 2021, 2:48 PM Abimbola Buari @.***>
wrote:

@.**** commented on this pull request.
In CSS/style.css
https://github.com/Mre55/Wara-Conference/pull/1#discussion_r750190617:

+.main-program-container {

display: flex;

flex-direction: column;

align-items: center;

+}
Kindly finish all the milestone and project requirements before
resubmitting 🙂
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Mre55/Wara-Conference/pull/1#discussion_r750190617,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AIH2BR4QFQI4O6ETFCZWKHLUMJAHVANCNFSM5IBPE5RA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.


How to test audio files with model trained on Gigaspeech

Hi authors,
Do you have any suggestions about how to test audio files with model trained on Gigaspeech (you provided)? Thanks so much.

There are multiple ways. You can choose either of them.

Go to https://huggingface.co/spaces/k2-fsa/automatic-speech-recognition
this approach is the simplest one. You only need a browser.

Use pretrained.py
Unfortunately, there is no pretrained.py in gigapseech folder. Normally, there will be a pretraniend.py
in each folder, e.g., librispeech, that you can use to decode waves with a pretrained model.

Use https://github.com/k2-fsa/sherpa/blob/master/sherpa/bin/pruned_transducer_statelessX/offline_asr.py
it also takes a pretrained model as input to decode wave files. You can find detailed usages inside the file.

There are also c++ binaries from https://github.com/k2-fsa/sherpa that you can use.
Please read the doc of sherpa for more details.


Experiment: Try using ArrowKt with Optional instead of nullable types

This is an experimental patch which looks at the possibilities of using ArrowKt for things like Option instead of returning null.
The reason I am investigating a solution like this is because it is far too easy to ignore a null in Kotlin which might be very important at runtime. Using a proper Option type provides more incentive to properly check what you received from each function.
This PR is primarily a proof-of-concept.

Would love to hear what you think about something like this @dhruvrajvanshi. I feel like this will have the consumer properly deal with edge cases, but it might seem foreign to users who are used to the kotlin/java world

I'm not sure if using optional is really safer in cases where it would matter. For chained calls, you'd where you'd use '?.foo?.bar', now you'd use .map { it.foo }.map { it.bar }. Really doesn't change anything apart from verbosity (and making it less idomatic).
I've rarely had to get anything from LLVM api anyway. I'm usually putting things into a module. For things like variable bindings, I use my own method fun getFunction(name: Name): FunctionValue.
It is impossible for this function to do return llvmModule.getFunction(lowerName(name)) because getFunction returns a nullable. If it returned an Optional, it would be exactly the same code, slightly more verbose.
So, my thinking is that it doesn't matter in terms of safety and if it doesn't, then why add another library?
For reference, here's my LLVMGen phase. This phase is pretty mechanical and since each syntax element is it's own function that doesn't take nullable args, the compiler forces me to deal with nullable values anyway (and using an optional won't change anything).
https://github.com/dhruvrajvanshi/hades-lang/blob/master/hadesc/src/main/kotlin/hadesc/codegen/LLVMGen.kt


Add a cleanup stage

Similar to the automatic clean up stages in https://github.com/AnalyticalGraphicsInc/gltf-pipeline, 3d-tiles-tools/tools could have a stage to perform any needed cleanup on the tileset. This may include adding a _3DTILESDIFFUSE semantic to the glTFs to support different color blend modes in Cesium: https://github.com/AnalyticalGraphicsInc/cesium/pull/4451

If this issue is still relevant, it may be necessary to add further details about what "cleanup" should refer to.


Support caching of schema instances in DefaultSchemaRegistryClient

This is being closed since #832 is supposed to fix and enhance the caching capabilities of the message converter


props.value is not obvious enought

When using props.value as an object, it should be more clear what the expected field are.
We've got an example, but the table down the readme is not clear enought

Thanks for raising this! I've updated the README to better document the props.


Adding the greatest game of all time

Not sure if this requires further description...

Looks good to me @thegoldenmule!


Startup is slow again...

Python is taking 6.5 sec, ouch!

Reduced to 3.5 sec... still too slow

Bridged implementation now takes ~2.5 and is largely independent of the view complexity! Rending 1000 Text nodes takes about 200ms. The original implementation took over 8 seconds!

Extracting the stdlib instead of using the zip gains about 500ms

Latest takes about 2 for python seconds or about 3-4 seconds total. React-native takes about 2-3 seconds on my phone, so I'd say this is reasonable.


[rapids] Update Spark RAPIDS to 22.12

Signed-off-by: liyuan<EMAIL_ADDRESS>update for spark-rapids v2212 release

verified. @cjac could you help build?

@sameerz @mengdong FYI

/gcbrun

version changes and removal of XGBOOST_GPU_SUB_VERSION outside of the context of spark != 3

yes, could you help merge the pr?


Implement style guide in Tailwind CSS config

Overview
As a developer, I should be able to use standard Tailwind CSS utilities to systematically customize font sizes and colors.
Proposal
Tailwind CSS supports theme customization. The tailwind.config.js file should be updated to create a theme for color and typography that uses a format like the following:

"Display Large" -> display-large
"Neutral" -> neutral

For each color, we should use a format like [color]-[token] where [color] is the identifier for content and [token] is a value uniquely identify a color.
(e.g. primary-1 would be the name of the lightest color in the primary set of colors).
So in code, we would be able to write something like the following:

  Text

Supplementary Information
Type Scale (Headings)

Type Scale (Content)

Color Usage

See the full spec for the style guide here.

As a note for @SmokinTurkey, the hex value for the second neutral color from the right is incorrect and should be updated in Figma to read as #353535.


SIEVE paper summary

Finished the summary of the SIEVE paper! Comments/edits are welcome :)

Thanks for reviewing! Will add these suggestions later today and then we can merge :)


How can reset lines...

Hi. Thank you for your Blazor-Diagrams..
Some many rects and connect lines..
but connected lines are complicated.
How can reset line this style..

Is possble?
Thank you for your project.

I'm sorry I don't understand what you want to achieve...
Do you want lines to go like this?

I'm sorry I don't understand what you want to achieve... Do you want lines to go like this?

Oh.. Hi!~ Thank you for your answer.
My meen is this.

If i create auto reset button.
button click auto reset clean position.
Thank you  for your answer.

You will need to use a Layout Algorithm.
Unfortunately, the library doesn't provide them for now.


cmake fail if enable DBUILD_OPTEL but disable DBUILD_OPTEL_OTLP_BENCHMARKS

Describe the bug
when I set BUILD_OPTEL=ON and BUILD_OPTEL_OTLP_BENCHMARKS=OFF, cmake configure fail
Regression Issue

[ ] Select this option if this issue appears to be a regression.

Expected Behavior
when I set BUILD_OPTEL=ON and BUILD_OPTEL_OTLP_BENCHMARKS=OFF, cmake configure should success.
Current Behavior
cmake -DCMAKE_PREFIX_PATH=/home/chuande/workspace/dingofs/third-party/installed -DBUILD_OPTEL=ON  -DBUILD_SHARED_LIBS=OFF  -DENABLE_TESTING=OFF  -DAUTORUN_UNIT_TESTS=OFF  -DBUILD_ONLY=s3 -DBUILD_OPTEL=ON -DBUILD_OPTEL_OTLP_BENCHMARKS=OFF ..
CMake Error at /home/chuande/workspace/dingofs/third-party/installed/lib/cmake/opentelemetry-cpp/opentelemetry-cpp-target.cmake:142 (set_target_properties):
  The link interface of target "opentelemetry-cpp::otlp_http_client"
  contains:

    nlohmann_json::nlohmann_json

  but the target was not found.  Possible reasons include:

    * There is a typo in the target name.
    * A find_package call is missing for an IMPORTED target.
    * An ALIAS target is missing.

Call Stack (most recent call first):
  /home/chuande/workspace/dingofs/third-party/installed/lib/cmake/opentelemetry-cpp/opentelemetry-cpp-config.cmake:131 (include)
  cmake/external_dependencies.cmake:114 (find_package)
  CMakeLists.txt:199 (include)

cmake -DCMAKE_PREFIX_PATH=/home/chuande/workspace/dingofs/third-party/installed -DBUILD_OPTEL=ON  -DBUILD_SHARED_LIBS=OFF  -DENABLE_TESTING=OFF  -DAUTORUN_UNIT_TESTS=OFF  -DBUILD_ONLY=s3 -DBUILD_OPTEL=ON -DBUILD_OPTEL_OTLP_BENCHMARKS=ON  ..
-- SKIPPING:Threads::Threads
-- SKIPPING:Threads::Threads
-- SKIPPING:opentelemetry-cpp::api
-- SKIPPING:opentelemetry-cpp::api
-- SKIPPING:opentelemetry-cpp::sdk
-- SKIPPING:opentelemetry-cpp::sdk
-- SKIPPING:opentelemetry-cpp::ext
-- SKIPPING:opentelemetry-cpp::ext
-- Configuring done (8.8s)
-- Generating done (0.3s)
-- Build files have been written to: /home/chuande/workspace/aws-sdk-cpp/build

-

Reproduction Steps
In build dir run
cmake -DCMAKE_PREFIX_PATH=/home/chuande/workspace/dingofs/third-party/installed -DBUILD_OPTEL=ON  -DBUILD_SHARED_LIBS=OFF  -DENABLE_TESTING=OFF  -DAUTORUN_UNIT_TESTS=OFF  -DBUILD_ONLY=s3 -DBUILD_OPTEL=ON -DBUILD_OPTEL_OTLP_BENCHMARKS=OFF ..

Possible Solution
In external_dependencies.cmake,if  BUILD_OPTEL enable, call find_package(nlohmann_json REQUIRED)
# Open telemetry client
if (BUILD_OPTEL)
    find_package(nlohmann_json REQUIRED)
    find_package(opentelemetry-cpp CONFIG REQUIRED)
endif ()

Additional Information/Context
No response
AWS CPP SDK version used
1.11.399
Compiler and Version used
gcc (GCC) 13.1.1 20230614 (Red Hat 13.1.1-4) Copyright (C) 2023 Free Software Foundation, Inc. This is free software; see the source for copying conditions.  There is NO warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE
Operating System and version
Rocky Linux release 8.9 (Green Obsidian)

When awssdk/opentelemetry-cpp/nlohmann_json  all as extrenal project like this https://github.com/chuandew/dingofs/blob/add_opentele/third-party/CMakeLists.txt
We have no chance to call find_pacakge.

we are only dependent on the SDK and API, they do not have a nlohmann_json. Only the otlp and http exporter have a json dependency. The SDK does not use those, only the sample code.
If you plan on using dependencies that require nlohmann_json you need to find those dependencies in your build.


tf.compat.v1.resource_loader.get_path_to_datafile

I had implemented this code:
import tensorflow as tf
_flow_warp_ops = tf.load_op_library(
tf.compat.v1.resource_loader.get_path_to_datafile("./lib/flow_warp.so"))
But it is showing error:
Traceback (most recent call last):
File "", line 2, in
tf.compat.v1.resource_loader.get_path_to_datafile("./lib/flow_warp.so"))
File "/home/akshat_suwalka/anaconda3/lib/python3.7/site-packages/tensorflow_core/python/framework/load_library.py", line 57, in load_op_library
lib_handle = py_tf.TF_LoadLibrary(library_filename)
NotFoundError: libcudart.so.8.0: cannot open shared object file: No such file or directory
As you can see that file already i have in "lib/flow_warp.so"

@akshat-suwalka
Please provide details about what platform you are using (operating system, architecture). Also include your TensorFlow version.
We ask for this in the issue submission template, because it is really difficult to help without that information. Thanks!

I am using ubuntu 16.04 LTS
Tensorflow is the latest version as I had installed day before yesterday.

In anaconda i had installed all the latest packages on that day including cudatoolkit, cudnn.
libcudart.so.8.0 is showing bcz i dont have cuda 8.0 version.
If it is show then i search here: https://repo.anaconda.com/pkgs/main/linux-64/
where all the packages are available for linux but here i didn't find cuda 8.0 version.
Is there any possibility to run that code in latest version of cuda??
Any suggestion what should i do to resolve this problem?

@akshat-suwalka What version of CUDA are you using?


slight fixes on Japanese translation

modify some particles (助動詞)
append space between Japanese and English word

Thanks for the pull request! 🎉 Homebrew depends on contributions from community members like you and we're grateful for your support.


[Bug] Crash on Startup with Framework

Describe the bug
After Adding the Mod to my huge Modpack, it gives me an error with the library "Framework"
To Reproduce
Steps to reproduce the behavior:

Add the Mod to Modpack
Start the Server
Server crash on Startup

Expected behavior
Should load with offline Players
Screenshots
Versions (please complete the following information):

Minecraft: 1.21.1
Fabric Loader: 0.16.5
Mod Version: newest

Additional context
https://mclo.gs/egIpY3t

After having a quick glance over the log file I don't directly see anything that indicated that this issue is because of OfflinePlayers. Would it be possible for you to send the mods folder to me so I can test it on my machine?
I'll have a better look at the log file as soon as possible.

Hey, i'm at work, i will send the whole modpack to you, if i had time.
Will it send to you as a whole zip file ;)

Please don't use the Modpack to upload it somewhere, there are mods from exclusive curseforge and some mods exclusive modrinth, its a little bit weird....
and it is only an dev enviroment to fix a lots of errors.
if you would join you need only automodpack in the fabric client and then only try to connect to the server, the server would send the client the mods
offlineplayersreworked is only as disabled file, you only need to rename it
Zip file has complete modpack server pack.
https://we.tl/t-QIAVrnwN0O
these are the latest log without offline player
https://mclo.gs/ysyEA4T
latest.log
Receip errors and command errors you could ignore, it would be fixed in my next update

I might have found a possible problem. Am looking into a fix right now.
(Ps. I deleted your download link after downloading as you said it shouldn't be uploaded :))

I'm not able to run the server so I'm not sure if I fixed the issue or not. Before I make any new releases, could I ask you to test it out @suerion ?
Here's the new  dev-jar: https://www.mediafire.com/file/oe62c7v9wxhdmye/offlineplayersreworked-1.0.9-dev.jar/file

I'm unable to start up the server correctly, but I was able to see a different error when offlinePlayers was installed vs when it wasn't installed. I've updated the .jar building so that it shouldn't interfere with any other mods dependencies but I'm unable to test if it is working correctly. @suerion Could you test it out before I release a new version?
https://www.mediafire.com/file/oe62c7v9wxhdmye/offlineplayersreworked-1.0.9-dev.jar/file

Good Morning, sorry, yesterday was a long day.
I will trying to test it after work.
And thanks for deleting the link.
Could be, that the you need to change the bat file.
I had an direct link to my jdk21. With another jdk, lower than 21 it would not work, because 1.21.1 need the 21 version.
Thanks for your work.

@TheMisterFish
i think you had send me an forge dev mod file or something, could not test sorry
[16:45:49] [main/WARN]: Found 1 non-fabric mod:
        - offlineplayersreworked-1.0.9-dev.jar

Wrong file indeed, I'll have to send you the correct one but am currently busy with work. Wil try and send correct file asap.

Wrong file indeed, I'll have to send you the correct one but am currently busy with work. Wil try and send correct file asap.

yeah, no stress, had more time as yesterday :)

Wrong file indeed, I'll have to send you the correct one but am currently busy with work. Wil try and send correct file asap.

yeah, no stress, had more time as yesterday :)

I found the issue. I'm using a dependency called jsonDB as a small database for the offline players. The jsonDB library has a dependency on Reflections. com.mrcrayfish.framework.FrameworkFabric also has a dependency on Reflections. This is causing the error.
I was trying to create the jar that includes the jsonDB library with a own Reflections in the jar so they shouldn't clash, but I'm having issues building it correctly (as you saw)
I now feel like I have 3 options:

continue trying with shadowJar
remove jsonDB and create a custom json database class with no dependencies
call it a day... (won't do this one though 😅 )

I will keep you updated in this thread if that's not an issue for you, but I think it's gonna take a little more time than I wished it would.

@suerion could you try:
https://www.mediafire.com/file/qc4xo0sveu2ht84/offlineplayersreworked-1.0.9-bugfix.jar/file
I've tested that one on my own server first, should work fine.

[21:36:44] [main/INFO]: Hello from OfflinePlayersReworked!
will test it, server runs without issues yet :) will try on client

I think it work :)

Got only this issue on Server Console
[21:44:11] [Server thread/ERROR] (OfflinePlayersReworked) Cannot invoke "net.minecraft.class_2945.method_12778(net.minecraft.class_2940, Object)" because "this.field_6011" is null
 java.lang.NullPointerException: Cannot invoke "net.minecraft.entity.data.DataTracker.set(net.minecraft.entity.data.TrackedData, Object)" because "this.dataTracker" is null
        at net.rodofire.mushrooomsmod.entity.custom.PlotiEntity.initDataTracker(PlotiEntity.java:77) ~[mushrooomsmod-0.0.12-1.21.jar:?]
        at MC/net.minecraft.entity.Entity.<init>(MixinEntity.java:309) ~[server-intermediary.jar:?]
        at MC/net.minecraft.entity.LivingEntity.<init>(LivingEntity.java:270) ~[server-intermediary.jar:?]
        at MC/net.minecraft.entity.mob.MobEntity.<init>(MobEntity.java:148) ~[server-intermediary.jar:?]
        at MC/net.minecraft.entity.mob.PathAwareEntity.<init>(PathAwareEntity.java:17) ~[server-intermediary.jar:?]
        at MC/net.minecraft.entity.passive.PassiveEntity.<init>(PassiveEntity.java:29) ~[server-intermediary.jar:?]
        at MC/net.minecraft.entity.passive.AnimalEntity.<init>(AnimalEntity.java:42) ~[server-intermediary.jar:?]
        at net.rodofire.mushrooomsmod.entity.custom.PlotiEntity.<init>(PlotiEntity.java:34) ~[mushrooomsmod-0.0.12-1.21.jar:?]
        at MC/net.minecraft.entity.EntityType.create(EntityType.java:552) ~[server-intermediary.jar:?]
        at com.natamus.dailyquests_common_fabric.quests.types.TameAnimal.isAllowedIdentifier(TameAnimal.java:45) ~[dailyquests-1.21.1-1.8.jar:?]
        at com.natamus.dailyquests_common_fabric.quests.types.TameAnimal.getRandomQuestIdentifier(TameAnimal.java:53) ~[dailyquests-1.21.1-1.8.jar:?]
        at com.natamus.dailyquests_common_fabric.quests.functions.GenerateQuests.getNewQuests(GenerateQuests.java:122) ~[dailyquests-1.21.1-1.8.jar:?]
        at com.natamus.dailyquests_common_fabric.quests.functions.GenerateQuests.replaceAllPlayerQuests(GenerateQuests.java:80) ~[dailyquests-1.21.1-1.8.jar:?]
        at com.natamus.dailyquests_common_fabric.quests.functions.GenerateQuests.setInitialQuestScreen(GenerateQuests.java:23) ~[dailyquests-1.21.1-1.8.jar:?]
        at com.natamus.dailyquests_common_fabric.events.DailyQuestServerEvents.onEntityJoinLevel(DailyQuestServerEvents.java:46) ~[dailyquests-1.21.1-1.8.jar:?]
        at com.natamus.dailyquests.ModFabric.lambda$loadEvents$2(ModFabric.java:47) ~[dailyquests-1.21.1-1.8.jar:?]
        at net.fabricmc.fabric.api.event.lifecycle.v1.ServerEntityEvents.lambda$static$0(ServerEntityEvents.java:39) ~[fabric-lifecycle-events-v1-2.3.1+6c1df36019-a0320f6814921fbf.jar:?]
        at MC/net.minecraft.server.world.ServerWorld$ServerEntityHandler.handler$cpm000$fabric-lifecycle-events-v1$invokeEntityLoadEvent(ServerWorld.java:1730) ~[server-intermediary.jar:?]
        at MC/net.minecraft.server.world.ServerWorld$ServerEntityHandler.startTracking(ServerWorld.java:1611) ~[server-intermediary.jar:?]
        at MC/net.minecraft.server.world.ServerWorld$ServerEntityHandler.startTracking(ServerWorld.java:1570) ~[server-intermediary.jar:?]
        at MC/net.minecraft.server.world.ServerEntityManager.startTracking(ServerEntityManager.java:216) ~[server-intermediary.jar:?]
        at MC/net.minecraft.server.world.ServerEntityManager.addEntity(ServerEntityManager.java:186) ~[server-intermediary.jar:?]
        at MC/net.minecraft.server.world.ServerEntityManager.addEntity(ServerEntityManager.java:167) ~[server-intermediary.jar:?]
        at MC/net.minecraft.server.world.ServerWorld.addPlayer(MixinServerLevel.java:926) ~[server-intermediary.jar:?]
        at MC/net.minecraft.server.world.ServerWorld.onPlayerConnected(MixinServerLevel.java:912) ~[server-intermediary.jar:?]
        at MC/net.minecraft.server.PlayerManager.onPlayerConnect(MixinPlayerList.java:246) ~[server-intermediary.jar:?]
        at com.misterfish.patch.OfflinePlayer.createAndSpawnNewOfflinePlayer(OfflinePlayer.java:91) ~[offlineplayersreworked-1.0.9-bugfix-v2.jar:?]
        at com.misterfish.OfflinePlayersReworked.spawn(OfflinePlayersReworked.java:154) ~[offlineplayersreworked-1.0.9-bugfix-v2.jar:?]
        at com.misterfish.OfflinePlayersReworked.lambda$onInitializeServer$1(OfflinePlayersReworked.java:82) ~[offlineplayersreworked-1.0.9-bugfix-v2.jar:?]
        at com.mojang.brigadier.context.ContextChain.runExecutable(ContextChain.java:73) ~[brigadier-1.3.10.jar:?]
        at MC/net.minecraft.command.FixedCommandAction.execute(FixedCommandAction.java:29) ~[server-intermediary.jar:?]
        at MC/net.minecraft.command.FixedCommandAction.execute(FixedCommandAction.java:13) ~[server-intermediary.jar:?]
        at MC/net.minecraft.command.SourcedCommandAction.method_54405(SourcedCommandAction.java:8) ~[server-intermediary.jar:?]
        at MC/net.minecraft.command.CommandQueueEntry.execute(CommandQueueEntry.java:8) ~[server-intermediary.jar:?]
        at MC/net.minecraft.command.CommandExecutionContext.run(CommandExecutionContext.java:107) ~[server-intermediary.jar:?]
        at MC/net.minecraft.server.command.CommandManager.callWithContext(CommandManager.java:379) ~[server-intermediary.jar:?]
        at MC/net.minecraft.server.command.CommandManager.execute(CommandManager.java:309) ~[server-intermediary.jar:?]
        at MC/net.minecraft.server.network.ServerPlayNetworkHandler.executeCommand(ServerPlayNetworkHandler.java:1319) ~[server-intermediary.jar:?]
        at MC/net.minecraft.server.network.ServerPlayNetworkHandler.method_44356(ServerPlayNetworkHandler.java:1307) ~[server-intermediary.jar:?]
        at MC/net.minecraft.server.ServerTask.run(ServerTask.java:18) ~[server-intermediary.jar:?]
        at MC/net.minecraft.util.thread.ThreadExecutor.executeTask(ThreadExecutor.java:162) ~[server-intermediary.jar:?]
        at MC/net.minecraft.util.thread.ReentrantThreadExecutor.executeTask(ReentrantThreadExecutor.java:23) ~[server-intermediary.jar:?]
        at net.minecraft.server.MinecraftServer.executeTask(MinecraftServer.java:864) ~[server-intermediary.jar:?]
        at net.minecraft.server.MinecraftServer.executeTask(MinecraftServer.java:173) ~[server-intermediary.jar:?]
        at MC/net.minecraft.util.thread.ThreadExecutor.runTask(ThreadExecutor.java:136) ~[server-intermediary.jar:?]
        at net.minecraft.server.MinecraftServer.runOneTask(MinecraftServer.java:846) ~[server-intermediary.jar:?]
        at net.minecraft.server.MinecraftServer.runTask(MinecraftServer.java:840) ~[server-intermediary.jar:?]
        at MC/net.minecraft.util.thread.ThreadExecutor.runTasks(ThreadExecutor.java:145) ~[server-intermediary.jar:?]
        at net.minecraft.server.MinecraftServer.runTasks(MinecraftServer.java:810) ~[server-intermediary.jar:?]
        at net.minecraft.server.MinecraftServer.runTasksTillTickEnd(MinecraftServer.java:815) ~[server-intermediary.jar:?]
        at net.minecraft.server.MinecraftServer.runServer(MinecraftServer.java:702) ~[server-intermediary.jar:?]
        at net.minecraft.server.MinecraftServer.method_29739(MinecraftServer.java:281) ~[server-intermediary.jar:?]
        at java.base/java.lang.Thread.run(Thread.java:1583) [?:?]

https://mclo.gs/KihM1gL
Here the log ;)

I'll have a look at it soon. Thanks for the log :)

I'll have a look at it soon. Thanks for the log :)
By the way, could I ask you to test version 1.1.0 when I refactor some things? I don't like the fact I need this many imports.

Hey, yeah thats why i'm using my dev enviroment here in my pc. To test, fix and debug some issues.
You could send it, i will test, if it will work. You could also say, what i should look for. ;)

Hey, i think this is not really an issue, it is only an compatibilty problem with other mod, but it does nothing. It is only an logging error.
But if we could fix this, i think it could be fine.
Will test after work again. Had an team meeting in 2 hours, hope it would be not that long

Hey, i think this is not really an issue, it is only an compatibilty problem with other mod, but it does nothing. It is only an logging error.
But if we could fix this, i think it could be fine.
Will test after work again. Had an team meeting in 2 hours, hope it would be not that long

The less error messages my mod causes, the happier I am 😇
Good luck with your meeting!

I think, it looks fine now :)
https://mclo.gs/DQHVQQ8
Only this comes from Automodpack ->
[19:42:42] [Server thread/ERROR]: [OFF]Suerion isn't in the players map.
https://github.com/Skidamek/AutoModpack/issues/292
Ignore Please the EOFException, this should be fixed in an update from an other mod :)
I think this are your Dev debug lines? https://mclo.gs/DQHVQQ8#L7140

I'll release 1.0.9 as soon as possible :)
Just to be sure, V3 gave no nullpointer exception, right?

Yeah, V3 give no errors

@suerion I tagged you in the other message as well but since you were not a part of that thread yet I'm not sure if the tagging worked correctly; if it did, sorry for the double-tagging.
1.0.9 has been released. Many thanks for your troubleshooting and log files, they were very handy.
I've created a 1.1.0-test.jar in https://github.com/TheMisterFish/Fabric_OfflinePlayersReworked/issues/34#issuecomment-2403144319


On OSX return "Mac OS" in os-dispatch rather than "Windows".

Fixes the ./install.sh script under OS X.
Previously ./install.sh was failing without much feedback because it attempted to invoke cmd.exe:
/s/cloxp-install git:master ❯❯❯ ./install.sh
installing cloxp pre-0.0.8
1. Checking dependencies...
(error installing cloxp:
 )

Ay, this is why one should test properly... thanks!


[Modules] VirtualNetworks - Enable 'Null' on DiagnosticLogs setting to support Azure US Government

Description
Azure US Government does not support any log collection with Diagnostic Logs (i.e., AllLogs is not supported). Allow selection of " '' " for the diagnosticLogCategoriesToEnable.
Type of Change
Please delete options that are not relevant.

[X] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[X] Update to documentation

Checklist

[X] I'm sure there are no other open Pull Requests for the same update/change
[X] My corresponding pipelines / checks run clean and green without any errors or warnings
[X] My code follows the style guidelines of this project
[X] I have commented my code, particularly in hard-to-understand areas
[X] I have made corresponding changes to the documentation (readme)
[X] I did format my code

Hey @shawntmeyer,
thanks for the contribution. Naturally we'd need to rollout the change to all modules instead of just one once we agree on the implementation. However, I'd first have a question:

If you cannot set 'allLogs' in the GovCloud, I assume you can only set specific logs, correct? If so, then why would you want to set an empty value? Is the scenario that you, for example, provide a Diagnostic Storage Account, but don't want to feed any logs into it? Or only metrics?

You cannot set any logs. It does do metrics, but no logs are available for selection in portal.
Shawn
Sent via the Samsung Galaxy S22 Ultra 5G, an AT&T 5G smartphone
Get Outlook for Androidhttps://aka.ms/AAb9ysg

From: Alexander Sehr @.>
Sent: Wednesday, July 26, 2023 2:45:52 AM
To: Azure/ResourceModules @.>
Cc: Shawn Meyer @.>; Assign @.>
Subject: Re: [Azure/ResourceModules] [Modules] VirtualNetworks - Enable 'Null' on DiagnosticLogs setting to support Azure US Government (PR #3448)
Assigned #3448https://github.com/Azure/ResourceModules/pull/3448 to @shawntmeyerhttps://github.com/shawntmeyer.
—
Reply to this email directly, view it on GitHubhttps://github.com/Azure/ResourceModules/pull/3448#event-9919852331, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ALWLDANKDYLME3V7QOXY4ITXSC4KBANCNFSM6AAAAAA2XNKZ7M.
You are receiving this because you were assigned.Message ID: @.***>

You cannot set any logs. It does do metrics, but no logs are available for selection in portal. Shawn Sent via the Samsung Galaxy S22 Ultra 5G, an AT&T 5G smartphone Get Outlook for Androidhttps://aka.ms/AAb9ysg
…
________________________________ From: Alexander Sehr @.> Sent: Wednesday, July 26, 2023 2:45:52 AM To: Azure/ResourceModules @.> Cc: Shawn Meyer @.>; Assign @.> Subject: Re: [Azure/ResourceModules] [Modules] VirtualNetworks - Enable 'Null' on DiagnosticLogs setting to support Azure US Government (PR #3448) Assigned #3448<#3448> to @shawntmeyerhttps://github.com/shawntmeyer. — Reply to this email directly, view it on GitHub<#3448 (comment)>, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ALWLDANKDYLME3V7QOXY4ITXSC4KBANCNFSM6AAAAAA2XNKZ7M. You are receiving this because you were assigned.Message ID: @.***>

Cotcha thanks. The implementation makes sense to me, but as I said we should update all modules at once.
I'd also suggest to add a comment behind the empty parameterset value to explain what it is for. For example // To be used if no logs can be set

You cannot set any logs. It does do metrics, but no logs are available for selection in portal. Shawn
…

Cotcha thanks. The implementation makes sense to me, but as I said we should update all modules at once. I'd also suggest to add a comment behind the empty parameterset value to explain what it is for. For example // To be used if no logs can be set

I will update the comment. However, please read my edited previous comment again. I have yet to find another module that doesn't support logs in AzureUSGovernment.

Fair point. I'd still argue though that we should solve this from a generic / consistency standpoint. As you said, the limitation for VNETs may be a limitiation for others as well, and I consequently don't think it would hurt to add the additional option of '' to make sure there is always a supported option.
Officially we currently don't 'support' the Gov-Cloud as there are several severe limitations, but this one has at least an easy compromise :D

Is this something I should do? Should I find all instance of 'allLogs' and then add a blank option to each array?

Is this something I should do? Should I find all instance of 'allLogs' and then add a blank option to each array?

I guess it is a bit more complicated than that as also the implementation of the logs variable was changed - not to mention the readme & JSON template re-generations.
If that seems a bit much, we can also convert this PR into an issue and pick it up from there. I am aware this scope wasn't really what you were aiming for when creating the PR ;)

Yeah let's do an issue.
Sent via the Samsung Galaxy S22 Ultra 5G, an AT&T 5G smartphone
Get Outlook for Androidhttps://aka.ms/AAb9ysg

From: Alexander Sehr @.>
Sent: Wednesday, July 26, 2023 9:37:12 AM
To: Azure/ResourceModules @.>
Cc: Shawn Meyer @.>; Mention @.>
Subject: Re: [Azure/ResourceModules] [Modules] VirtualNetworks - Enable 'Null' on DiagnosticLogs setting to support Azure US Government (PR #3448)
Is this something I should do? Should I find all instance of 'allLogs' and then add a blank option to each array?
I guess it is a bit more complicated than that as also the implementation of the logs variable was changed - not to mention the readme & JSON template re-generations.
If that seems a bit much, we can also convert this PR into an issue and pick it up from there. I am aware this scope wasn't really what you were aiming for when creating the PR ;)
—
Reply to this email directly, view it on GitHubhttps://github.com/Azure/ResourceModules/pull/3448#issuecomment-1651826995, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ALWLDAKZMEKQ3PCGTXVEHFTXSEMQRANCNFSM6AAAAAA2XNKZ7M.
You are receiving this because you were mentioned.Message ID: @.***>

Closing in favor of created linked issue #3449

Closing in favor of created linked issue #3449


Update ESLint config to 1.1.2

Coverage remained the same at 100.0% when pulling 8ffae52a7024e0a0813db01823a03d046fa41ab1 on jimjenkins5:task_eslint1_1 into 4a48c0222ba5d02801ad99f43a670c848deffd80 on silvermine:master.


iPad mini 2 offsets

I tried to find the offset by myself, but got different results:
iPad4,4:
SETOFFSET(shenanigans, 0xFFFFFFF00890FC40 + kernel_slide);
SETOFFSET(kernel_task, 0xFFFFFFF00887E200 + kernel_slide);

Check and verified that it can set my devices' nonce.

Since @hirakujira has tested his offsets, I'm merging #3.


Deploy IDE with Tact v1.4.1 support when it lands on NPM

just landed on npm<EMAIL_ADDRESS>


[next] Async load of translation files

Describe the feature
Hello,
With nuxt2 and nuxt-i18n, we can asynchronously load the translation files by doing this kind of thing: https://github.com/nuxt-modules/i18n/issues/1042#issuecomment-766424969
It is not possible with nuxt3 and nuxt-i18n@next.
https://github.com/nuxt-modules/i18n/issues/1448
https://github.com/nuxt-modules/i18n/blob/next/TODO.md
Is it planned for the future ?
I need to know because my huge current project is based on translations that can be updated anytime, and we don't want to restart the servers each time to reload the translations. I'm working on the migration from nuxt2 to nuxt3, so this feature is essential. If it is not planned, I have to find an alternative.
By the way, thanks for your great job !
Regards
Additional information

[X] Would you be willing to help implement this feature?
[ ] Could this feature be implemented as a module?

Final checks

[X] Read the contribution guide (The contribution guideline of nuxt-modules/i18n is compliant with Nuxt too).
[X] Check existing discussions and issues.

I also trying to migrate from nuxt 2 and that feature is not implemented :(

Hello contributors, could you please just first tell us if this feature is planned or not ?
Regards

close with #1938


Push Credentials Setup - invalid iOS bundle ID

When setting APNs Certificate attributes, Bundle ID with dash (-) character inside is not accepted. Error message is displayed instead: 'Please enter a valid iOS bundle ID (for example "io.getlime.app.MyApp")'
Screenshot:

The issue is caused by using the same regexp as for the Android package bundle, that does not allow '-' characters. We will relax the regexp a bit on iOS, to allow '-'....


MemoryPackable(GenerateType.VersionTolerant) MemoryPackOrder - duplicates allowed

Hi - is it possible to add check in the code analysis to find and give errors when there are multiple fields being serialized in a version-tolerant class that are given the same MemoryPackOrder number?
For example, this will serialize/deserialize with no warning:
    [MemoryPackable(GenerateType.VersionTolerant)]
    public partial class Tester
    {
        [MemoryPackOrder(0)]
        public int I1 { get; set; }

        [MemoryPackOrder(0)]
        public string S1 { get; set; }

        [MemoryPackOrder(0)]
        public bool B1 { get; set; }
    }

... but what comes out after deserialization is incorrect.

thanks, I've added MEMPACK038 All members order must be unique.


Constant if expression?

I'd like to use this to alter the template output when it's being compiled based on a .env setting that's fixed at deployment. We have two news sites running on the same codebase, and a few pages are drastically altered by which site is being served. We compile the templates during deployment, and I'd like to fix those alterations in the compiled output so that the if condition is not checked during runtime. Do you know if this is possible?
In other words, I'd like to tell the Blade compiler to evaluate the if condition during compilation instead of injecting <php if ($constant_condition): ?> into the compiled template. I suspect I'll have to extend the compiler itself, but I'm hoping you've already thought of that. :)

After digging through the BladeCompiler source, I can see this would not be doable without a large refactoring. The parser would need conditional-compilation to omit the parts of the template to cut out, and saving a simple runtime check of @if (config('app.site') === 'SITE1') wouldn't be worth the effort.

Hey @dharkness, ya, this is just some automation to register classes as directives and conditionals and using the container for any dependencies. Thanks for stopping by!

I got excited when I saw getConditionals(), but your code ended up making overriding some compiler methods for a different feature much easier. Thanks! :)


Tiny problem with the help prompt

When opening issues, there are two references to use https://github.com/jupyterhub/binder/new .  It looks like that should probably be "https://github.com/jupyterhub/binder/issues/new" (or just "https://github.com/jupyterhub/binder/issues", to encourage me to search for my problem?)

Good catch! I think it should be /binder/issues/new. Want to make a quick PR for the change? Am happy to review it quickly


remove labels that we dont use and generate  comments

/assign @Issif

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: cpanato
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [cpanato]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

LGTM label has been added.  Git tree hash: 97a8981fbf8457f4ce44fd0d4ccadac0ab6b5555

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: cpanato, Issif
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [Issif,cpanato]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Cosmos - Fix @EnableAuditing can't work with Java Modules

Fix https://github.com/Azure/azure-sdk-for-java/issues/18892. When @EnableAuditing is used with Java Modules, the below exception will be thrown. This PR is to open com.azure.spring.data.cosmos.core.mapping to spring.core.
Caused by: java.lang.reflect.InaccessibleObjectException: Unable to make com.azure.spring.data.cosmos.core.mapping.CosmosAuditingRegistrar() accessible: module com.azure.spring.data.cosmos does not "opens com.azure.spring.data.cosmos.core.mapping" to module spring.core

Already tested with the Java 11 sample here - https://github.com/Azure-Samples/azure-spring-data-cosmos-java-sql-api-getting-started

@kushagraThapar any idea why the emulator ci failed?

@kushagraThapar any idea why the emulator ci failed?

@saragluna - there are some issues with emulator right now, we are fixing them. So this PR is good to go in.

/check-enforcer override


Set auto_delete_log correctly when using --log-dir

Related to #23.

Codecov Report

Merging #24 (a4f7261) into main (88ea7b7) will increase coverage by 0.44%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main      #24      +/-   ##
==========================================
+ Coverage   83.58%   84.02%   +0.44%
==========================================
  Files           1        1
  Lines         335      338       +3
==========================================
+ Hits          280      284       +4
+ Misses         55       54       -1

Impacted Files
Coverage Δ

rsync_time_machine.py
84.02% <100.00%> (+0.44%)
:arrow_up:


multiplayer_client_manager::match_client crashes when latest_pending_read() is null

We have a Unity game, in certain circumstances it'll cancel the matchmaking/match when it isn't even in an online match/matchmaking session - we could further improve our logic/flow, but either way, we should not get a crash in this scenario. The following fixes this:
multiplayer_client_manager.cpp
std::shared_ptr<multiplayer_match_client>
multiplayer_client_manager::match_client()
{
    return latest_pending_read()->match_client();
}

change to:
std::shared_ptr<multiplayer_match_client>
multiplayer_client_manager::match_client()
{
    if (latest_pending_read() == nullptr)
    	return nullptr;

    return latest_pending_read()->match_client();
}

Thanks for the report.  Do you want to send a pull request?

Fixed this and other related issues here: https://github.com/Microsoft/xbox-live-api/pull/257


low priority:  Search for subtypes of classifiers does not work

e.g.

This is working now.  My error.  Need to clear everything.  Refreshing the page does not clear other prior selections lower down...  No problem here, in any case.  Thanks.


Investigate use of create-react-app for React tutorial

Recently, the utility create-react-app was made to facilitate starting up a new React project.
It's worth investigating what the utility does specifically. If it's very minimal, and easy to configure afterwards, we can consider using it for our React tutorial.

@wmonk just checked out your react-scripts-ts fork - it looks really solid, so I think we might just use it. 👍

That's awesome!
Will give me a push to add some more documentation.
Anything I can do to help make it more user friendly?

Right now the npm package doesn't have updates to the README, but other than that I think my only two other concerns are

The Webpack docs might have to be separated out. (probably appropriate anyway)
For more complex workflows, we might be leaving other users out of the rain. If that's the case, we can keep things mostly how they are, and refer them to the script if they want something way faster.

For the record, most of the office is out (and I'm on vacation) so we might have to circle back to this later on, but I wanted to ping you and get your thoughts on the matter. 😄

hey @DanielRosenwasser back to full time work now, so can help properly 😄
What can I do to help with this? Are you proposing to change the existing react/webpack guide to just use react-scripts-ts?

That's mainly it, though I haven't gotten around to it since I'm still sort of on vacation.
Part of this is that I have to really sit down and try out the workflow. Our users have asked for more comprehensive and prescriptive tutorials, giving details on writing tests, adding Redux and/or Mobx, etc. Unfortunately, as a guy who works on languages & compilers, front-end isn't as much my forte, so I'd need to run through the process before I try to write up a guide on those details. 😄
The way I see it is that first I'd like to convert the guide to use react-scripts-ts and have documentation on writing tests. Then we'd later revisit and add details on state management. I think we'd be willing to take contributions if you're interested, and I'd welcome any advice you have.

Sounds like a good plan! I think the great thing about create-react-app is that is gives you a massive head start, without any additional complexity. E.g. you will definitely need react, but not necessarily any state management, node server, router, etc - so just leaves those things out. For this basic use case it works brilliantly, but invariably people want more on top of it.
What kind of tests are you talking about, using something like enzyme. Currently there is a single test in there, which gives some base to build on top of - although if you're not familiar with testing react components it won't be especially helpful. I can certainly help with this side of things, but am not sure exactly how you want to present the "getting started with create react app" guide.


[Bug] Fix backend addteammembers api  is Fixed

Issue that this pull request solves
There is problem with addTeamMembers api. Because it frontend is unable to add team members and throwing errors. fixing it is most important to fix frontend problems. By merging this pr. Backend is add team member endpoint is ready for Frontend development.
Issue
Issue Link resolve #872
Closes: #872
Brief description of what is fixed or changed
I fixed the addteammembers endpoint file in backend. It has a function that function is wrong I corrected it.
How to test it

Way one

Check for the below thing in db. why because I added it.
{ description:"This is for checking the api", full_name: "testing", teams: ["Aman","anubhav"], __v: 0, _id: "664251826c3bb95cd441ebd9" }

way 2
try to add team members form this api.

Types of changes
Put an x in the boxes that apply

[x] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation update (Documentation content changed)
[ ] Other (please describe):

Checklist
Put an x in the boxes that apply

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] My changes does not break the current system and it passes all the current test cases.

@Kajol-Kumari can you please check this and merge it. then i will start working on #725

@Kajol-Kumari don't merge this pr.

@Kajol-Kumari don't merge this pr if you merge it again you will get error.

@Kajol-Kumari Now you can merge it.

@Kajol-Kumari can you please merge it. I will start my next issue.


Add monit plugin

Done, let me know if there's anything else you'd like me to do (the info was already in the README of the plugin).


Add session expiry time to 30min

In order to accomplish first question of Issue talked here (I can't refer because the issue was posted in the "genieacs" repo, not in "genieacs-gui")
I think it would be great to put it by default to 30min, or whatever you consider, @zaidka
I've tried the changes in my own local copy and it works. If you don't perform any action in 30 min, then it logs off.

Thanks for the PR. However, I don't see a compelling reason to not stick with whatever the defaults are in Rails. And this can be set by user anyway.


Maintenance

Adding more tests
Improving pipeline configuration
Fixing typos in the README

📊 Package size report   1%↑

File
Before (Size / Gzip)
After (Size / Gzip)

dist/index.js
4.3 kB / 1.4 kB
3%↑4.4 kB / 4%↑1.4 kB

dist/index.mjs
4.3 kB / 1.3 kB
3%↑4.4 kB / 4%↑1.4 kB

README.md
10.0 kB / 2.9 kB
0.9%↑10.1 kB / 2%↑2.9 kB

Total (Includes all files)
23.7 kB / 7.9 kB
1%↑24.1 kB / 2%↑8.1 kB

Tarball size
6.2 kB
2%↑6.3 kB

Unchanged files

File
Size (Size / Gzip)

dist/index.d.mts
770 B / 358 B

dist/index.d.ts
770 B / 358 B

LICENSE.md
1.1 kB / 667 B

package.json
2.6 kB / 963 B

🤖 This report was automatically generated by pkg-size-action

:rocket: PR was released in 0.0.6 :rocket:


RTDB's Order by doesn't seem to work

@scuffed-developer said that firebase rtdb's orderByValue is not working. Had some talks with him that i agree it is not that big of a deal (YET). So, we'll save it for later.
for reference : This issue.

FIXED in #23. Great job :tada: @scuffed-developer

Phew :sweat_smile:


Add CONTRIBUTORS.md

Create the contributors file for this project

In retrospect, I think it's best to include contributors list in the main README.md file to give contributors more exposure


Footer?

Is there a way to implement the following layout using your library?
----- item of the parent list.
--------- Items 1 of child List
...
--------- Items n of child List
----- footer of the parent list item.
...

@uburoiubu use a Multi*Adapter and have a different footer types


linux-stracciatella: update sources to RM1XX_5.4.70_v1.3.4 and add wireguard

Synced upstream to tag RM1XX_5.4.70_v1.3.4.
Usb network is now usb1.
It seems to work on RM2 <IP_ADDRESS>9 without keyboard folio; it needs testing with 3.x and on RM1
EDIT: now we build also wireguard as a module

I just tested wg using #743 and the Demo Server available at https://www.wireguard.com/quickstart/#demo-server. Everything seems in order.

It seems to work on RM2 <IP_ADDRESS>9 without keyboard folio; it needs testing with 3.x and on RM1

No testing is currently needed for 3.x as toltec doesn't yet support it. Everything will be tested for 3.x as part of adding support for it.

I've been trying to test this on my rM1, but I'm unable to due to the root partition not having enough space to extract the kernel. It looks like the kernelctl archive for this is 11MB while the stock one is 4MB.
Upon inspecting the kernel package it appears that every single one of the .ko files are larger than the stock ones. Some of the worst offenders are:

kernel/net/wireless/cfg80211.ko - stock: 362kB, stracciatella: 6MB
kernel/drivers/net/wireless/broadcom/brcm80211/brcmfmac/brcmfmac.ko - stock: 283kB, stracciatella: 4.7MB
kernel/drivers/input/touchscreen/cyttsp5.ko - stock: 81kB, stracciatella: 722.4kB
kernel/drivers/net/wireless/broadcom/brcm80211/brcmutil/brcmutil.ko - stock: 11kB, stracciatella: 204.6kB
kernel/drivers/input/touchscreen/cyttsp5_loader.ko - stock: 15kB, stracciatella: 196kB
kernel/drivers/input/touchscreen/cyttsp5_i2c.ko - stock: 7kB, stracciatella: 171.6kB
kernel/crypto/aes_generic.ko - stock: 29kB, stracciatella: 139.5kB

I've already gone through the exercise of moving systemd-journald logs off of the root partition and into the home partition, so I'm not really sure what else would be reasonable to remove.

It does not seem to be happy about finding the correct strip executable. It might make more sense to just strip the ko files as a manual step before compressing them.

It is still failing on my laptop:
$ make linux-stracciatella
./scripts/package_build.py  "linux-stracciatella"
The build directory 'build/package/linux-stracciatella' for recipe 'linux-stracciatella' already exists.
Would you like to [c]ancel, [r]emove that directory, or [k]eep it (not recommended)? [C/r/k] r
[    INFO] toltec.builder: linux-stracciatella [rm1]: Fetching source files
[    INFO] toltec.builder: linux-stracciatella [rm1]: Preparing source files
[    INFO] toltec.builder: linux-stracciatella [rm1]: Building artifacts
[    INFO] toltec.builder: linux-stracciatella [rm1] (linux-stracciatella): Packaging build artifacts
[    INFO] toltec.builder: linux-stracciatella [rm1] (linux-stracciatella): Only showing up to 50 lines of context. Use --verbose for the full output.
[   ERROR] toltec.builder: linux-stracciatella [rm1] (linux-stracciatella): make[1]: arm-remarkable-linux-gnueabihf-gcc: No such file or directory
[   ERROR] toltec.builder: linux-stracciatella [rm1] (linux-stracciatella): /bin/sh: line 1: arm-remarkable-linux-gnueabihf-strip: command not found
[   ERROR] toltec.builder: linux-stracciatella [rm1] (linux-stracciatella): make[2]: *** [scripts/Makefile.modinst:33: crypto/aes_generic.ko] Error 127
[   ERROR] toltec.builder: linux-stracciatella [rm1] (linux-stracciatella): make[1]: *** [Makefile:1354: _modinst_] Error 2
[   ERROR] toltec.builder: linux-stracciatella [rm1] (linux-stracciatella): package() failed

Is it possible that package() is not run in the docker image? I ran  find / -iname \*-gcc and got a listing of the gcc's I have on my system.

Ah, yes. That is the case as documented here: https://github.com/toltec-dev/toltec/blob/stable/docs/package.md#package-function
Since this is the case, I would recommend moving the make install to the build() method to a folder inside the srcdir, and then just copying that over as part of package(). It's a little odd that make install is where the strip is done.

I just realized that the config file they provided for rm1 has CONFIG_DEBUG_INFO=y while it is not set for rm2. I guess that this is where the problem lies. Let me try to get rid of it

@Eeems Can you try now?

Good catch! This does reduce it to near the same size as the stock kernel and allows it to be activated with kernelctl. I'm running it now with nothing obviously wrong.


[$250] Search - Chats - Selected chat is hidden when navigating via keyboard

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 9.0.31-14
Reproducible in staging?: Y
Reproducible in production?: N/A
Email or phone of affected tester (no customers<EMAIL_ADDRESS>Issue reported by: Applause Internal Team
Action Performed:
Precondition:

Account has many chats.

Go to staging.new.expensify.com
Go to Search > Chats.
Navigate to the bottom of the list via keyboard up button.

Expected Result:
The selected chat will be not hidden when navigating via keyboard.
Actual Result:
The selected chat is hidden when navigating via keyboard.
This issue happens with the chat at the bottom of the list.
Workaround:
Unknown
Platforms:

[ ] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

Screenshots/Videos
https://github.com/user-attachments/assets/4c359609-307e-43fe-83a6-23d64b6243da

View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~021833559992291211111
Upwork Job ID:<PHONE_NUMBER>291211111
Last Price Increase: 2024-09-10

Issue OwnerCurrent Issue Owner: @rayane-djouah

I don't see the Chats section in Search tab in production:

Seems this is a new feature and I don't think it should block the deploy

I am able to reproduce this problem!
https://github.com/user-attachments/assets/886ef038-9d41-428f-99c9-e8f39315951e
But observed one thing, it only happens when the chat size greater than initialy defined is selected

See this

Update Proposal

@rayane-djouah, can you please try to reproduce this? Thanks!

Proposal
Please re-state the problem that we are trying to solve in this issue.
The issue occurs when the user navigates to the bottom of the chat list using the keyboard, the selected chat becomes hidden instead of staying visible
What is the root cause of that problem?
The root cause of this problem is related to the focus and scrolling behavior when using keyboard navigation in the chat list.

The useArrowKeyFocusManager hook, which handles keyboard navigation, does not ensure the focused chat item remains visible when the user navigates to the bottom.

The scrollToIndex method in BaseSelectionList.tsx may not be properly scrolling to the last item in the list when it is selected.

What changes do you think we should make in order to solve the problem?

update the setFocusedIndex function in arrowUpCallback and arrowDownCallback to ensure that when the focused index changes, the list scrolls to bring the selected chat into view.
https://github.com/Expensify/App/blob/bd0132be946db3c2d9b6b5a835cdd184bb0b479c/src/hooks/useArrowKeyFocusManager.ts#L80-L83

Update the scrollToIndex function to ensure that when navigating to the last item in the list, the selected item is scrolled into view and remains visible.
https://github.com/Expensify/App/blob/bd0132be946db3c2d9b6b5a835cdd184bb0b479c/src/components/SelectionList/BaseSelectionList.tsx#L245-L261

onFocusedIndexChange passed to useArrowKeyFocusManager triggers the scrolling behavior in BaseSelectionList when the selected chat item changes.
https://github.com/Expensify/App/blob/bd0132be946db3c2d9b6b5a835cdd184bb0b479c/src/components/SelectionList/BaseSelectionList.tsx#L274-L283

the layout is properly calculated so that the selected chat remains in view when navigating to the bottom of the list.

const getItemLayout = (data, index) => ({
    length: ITEM_HEIGHT,
    offset: ITEM_HEIGHT * index,
    index,
});

What alternative solutions did you explore? (Optional)
An alternative approach could involve increasing the render window size of the list to ensure that more items are rendered and remain visible as the user navigates with the keyboard. However, this could lead to performance issues for long lists and is less optimal than ensuring the focus and scroll behavior work correctly.

@rayane-djouah, can you please provide an update? Did you review all the proposals? Thanks!

Okay! marked your words @rayane-djouah
Thanks for the review

@rayane-djouah Hi,
How might we dynamically determine the height of ReportActionListItemType items? - I think (though I'm not sure, this is just a hypothesis) we could try creating a function that splits the text into Markdown from the HTML value, then gathers all the attributes and adds them to the block size.
To make it clearer, here’s an example of what the code might look like:
function calculateHeight(inputString) {
  const lines = inputString.split('\n'); // or <br/>
  let height = 0;
  lines.forEach(line => {
    if (line.startsWith('<h1>')) {
      height += variables.h1; //
    }
    // ......
    else {
      height += variables.span;
    }
  });

  return height;
}

I will try this approach, and if everything works well, I will update the proposal by/on Monday.
Is the problem linked to both the getItemHeight function and the getItemLayout? - Yes
Here: https://github.com/Expensify/App/blob/41e696825344ae115c26ed03195a472383202607/src/components/SelectionList/BaseSelectionList.tsx#L172-L174
and then here:
https://github.com/Expensify/App/blob/41e696825344ae115c26ed03195a472383202607/src/components/SelectionList/BaseSelectionList.tsx#L357-L358
Regarding your optional solution: removing scrollToIndex causes a regression - Oh I'm sorry, yeah that makes sense

@rayane-djouah, we're keeping the original amount. Thanks!

@tushar-a-b
Hi, could you please test the following case:
Create two very large messages (approximately screen-sized) and check if switching between them works smoothly.
Then create three messages (each about one-third of the screen size) and observe how the switching behaves.
Also, try this with different screen sizes.
Thanks!

@rayane-djouah, can you please share an update? Thanks!

Bump @rayane-djouah for an update. I will also DM you for visibility. Thanks!

Cool, thanks @rayane-djouah for review, assigning @tushar-a-b 🚀

@rayane-djouah any updates? Thanks!

Payment summary:
@tushar-a-b requires payment - Offer sent in Upwork
@rayane-djouah requires payment automatic offer (Reviewer) $250 paid

All set!


Enlarging dataset option for custom datasets

I was wondering if there is an enlarging option for custom datasets for
users to try, such as the one used for MVTec dataset.
I'm currently training by setting the epoch to 2000
and also using the same training parameters used for MVTec dataset
to get the same performance, but it sure takes some time.
Thank you for taking your time!

It's not implemented, but it requires little change.
For MVTec, enlarging boils down to a few lines: here.
Adding a similar things to the ImageFolderDataset should be easy.
Something like
self.samples = self.samples * 10
self.anomaly_labels = self.anomaly_labels * 10

Welp, that's terribly simple!
I made a quick change to the ImageFolderDataset and made a PR if you want to implement it.
#29
Cheers!


Cannot share annotated data among users on the annotation panel.

When we add a user into a project, what the user did is not shown to the admin's side(and others).
There is no way to check what the added users do except for the statistics page.

Do you mean the admin can not access the annotation data made by the added (new) user?

yes, I found that all the user's work is aggregated and downloaded as a json file.
I think it would be great to sync what other users did on the admin's annotation panel.

@jinhojang6 I've implemented an option to enable all users on a project to share annotation progress in https://github.com/chakki-works/doccano/pull/318. Could you take a look and let me know if this suits your requirements?

Hmm, It didn't work on my end. I cannot find the list of projects

@c-w It works fantastically well. Thanks you for the guide

great works @c-w! I add collaborative_annotation function according to your commit, I'm confused about that admin can see user's annotation, but he can't modify the annotation, in annotation task, it's better for someone to check the annotation results in the panel, and modify it, could you please add the request?

@akailcy My team is currently working on improvements to roles in doccano, including introducing a label approver role similar to what you're mentioning. See https://github.com/chakki-works/doccano/issues/152#issuecomment-520959755 for more context. We're also working on a specific view to approve labels more effectively. Stay tuned for these changes.


gmm_conv fails to run

Hi, Matthias
I have another question, Is nn.conv.gmm_conv.py still under construction? It fails to run on my end, crashes at line 44:
self.mu = Parameter(in_channels, dim)

_TypeError: make_subclass(): argument 'data' (position 2) must be Tensor, not int

I think this should be:
self.mu = Parameter(torch.Tensor(in_channels, dim))

Also self.cov in line 53 doesn't exist. I'm assuming this should be self.sigma. The code however still doesn't run due to self.mu.view(1, F, D) and self.sigma.view(1, F, D).

RuntimeError: shape '[1, 150, 3]' is invalid for input of size 3

Thanks for the awesome lib! Kind regards,
Balder

Yes, I am currently working on it :D This should be fixed by tomorrow. Sorry!

hkHi, the GMMConv is fixed now.


Dolby is not working after patch

I have lenovo legion 5 with a Dolby atmos for gaming..
after patching, the viper doesn't works for me so I uninstall it and use my default dolby instead, but after restart my headphone jack where i patch is not working with dolby. I think that the patcher is the reason thats why dolby is not working on my headphone. What should I do?

I have the same problem


[KED-1876] Add support for custom queries with GBQTableDataSet

Description
Current implementation: GBQTableDataSet loads data from Google BigQuery. It uses pandas-gbq to read from BigQuery table. In extras/datasets/pandas/gbq_dataset.py
    def _load(self) -> pd.DataFrame:
        sql = "select * from {}.{}".format(self._dataset, self._table_name)  # nosec
        return pd.read_gbq(
            sql,
            project_id=self._project_id,
            credentials=self._credentials,
            **self._load_args
        )

This works well when dataset is small to medium in size. However, when we have "Big Data" there is a need to get specific columns or specific rows or specific partition from the dataset; hence using custom queries with filters is required.  The select * implementation in many ways, violates Best Practices  to fetch data from GoogleBigQuery.
Context
To comply with the GBQ Best Practices in order to make the queries cost efficient as well as time efficient; GBQTableDataSet can empower developers to pass custom queries as arguments; instead of select * ;  as in the current implementation.
Current implementation uses pd.read_gbq() to load data from Google Big Query. This function allows custom queries along with other arguments to be passed. The power of read_gbq() can be fully utilized by passing the allowed parameters of function as part of load_args in GBQTableDataSet.
As per documentation :
pandas.read_gbq(query: str, project_id: Union[str, NoneType] = None,
 index_col: Union[str, NoneType] = None, col_order: Union[List[str], NoneType] = None,
 reauth: bool = False, auth_local_webserver: bool = False,
 dialect: Union[str, NoneType] = None, location: Union[str, NoneType] = None,
 configuration: Union[Dict[str, Any], NoneType] = None, credentials=None,
 use_bqstorage_api: Union[bool, NoneType] = None, private_key=None,
 verbose=None, progress_bar_type: Union[str, NoneType] = None) → ’DataFrame’

To pass custom queries to GBQTableDataSet,  catalog.yml will look like:
my_dataset:
  type: pandas.GBQTableDataSet
  dataset: gbq_dataset_name
  table_name: gbq_table_name
  project: gbq_project_name
  load_args:
    query: "Select col1, col2, col3 from project.dataset.table_name where col4 < 100"

Hey @ajb7, thanks for raising the issue - this is actually perfect. We've had similar feedback recently and have recently opened a ticket on our backlog to handle exactly this (we agree with all your comments) - we're super thankful to have you taking this up in the PR you've opened!

Closed in https://github.com/quantumblacklabs/kedro/pull/443


how to create end to end automation, confluent cloud organization and login to that org with out opening browser interface.

I have created terraform org with terraform azapi provider but when use confluent cli command in terraform provisionar it is navigating browser inteferface.
confluent login --no-browser  --organization-id 9e71788d-2cf8-48ff-b4bb-7f567b2ac0d5
Navigate to the following link in your browser to authenticate:
Dow we have any option to login to confluent org wih terrafrom and create resources.

👋 @Maheshsuthari, thanks for opening this issue!
In order to use TF Provider for Confluent there's no expectation that a user would use AzAPI TF Provider, please see out Getting Started guide that will help you to create resources on Confluent Cloud in your org. Let me know if that helps!

But since this is a new organization created with terraform, how would i login to confluent without manual intervineaince  ?
We can create resources with api keys, concern is how we can get cloud api keys with manual effort.

In order to use TF Provider for Confluent a user has to provide a Cloud API Key owned by a user / service account with OrganizationAdmin role.
Could you describe your setup in a little bit more detail?

We are working on entire automation confluent cloud kafka.

Create confluent organization
Login to confluent and generate the cloud Api keys
Create cluster, environment, cluster-apikeys using confluent terraform provider.
Create private endpoints with kafka cluster private link services.

Oh I see,
nit: A more descriptive way for the 2nd step could be Login to confluent and generate a service account, assign OrganizationAdmin role to it and then create a Cloud API Key for that service account
Unfortunately I don't think the first 2 steps is something that is directly related to the source code of TF Provider for Confluent so you might want to contact Confluent Support to figure out a solution for it.


Dynamic Facebook Streamkey Supported?

There are some issues already about this very question but without an answer spanning back a few years.
#929
#1190
#1459
#1397 (the conversation turned into stunnel configuration. which already works for me if I statically place the FB streamkey into the exec command.)
The $name variable does not seem to contain all of the query args included in a Facebook live stream key. Example key<PHONE_NUMBER>578373?s_bl=4&s_ps=2&s_sw=1&s_vt=api-s&a=ANk-6kdQfURTY3ss. In the Nginx logs I see the following:
nginx-rtmp_1  | 2020/04/29 04:21:00 [info] 10#10: *38 connect: app='facebook' args='' flashver='LNX 9,0,124,2' swf_url='' tc_url='rtmp://localhost:1935/facebook' page_url='' acodecs=4071 vcodecs=252 object_encoding=0, client: <IP_ADDRESS>, server: <IP_ADDRESS>:1935

This is my rtmp application block. If I put the full key in the place of $name at the end of the exec call then it does work but there is no dynamic key support.
application facebook {
            live on;
            record off;
            allow publish all;

            # Transcode and push the stream to stunnel for FB Live RMTPS requirements
            exec ffmpeg -i rtmp://localhost:1935/facebook/$name
              -c:a libfdk_aac -b:a 128k -c:v libx264 -b:v 2500k -f flv -g 30 -r 30 -s 1280x720 -preset superfast -profile:v baseline -f flv rtmp://stunnel:19350/rtmp/$name;
        }

Thank you for clearing this up if possible.

You might have found a solution yet but for all others, maybe this woks (i dont stream to fb yet)
        application facebook {
            live on;
            record off;
            allow publish all;

            # Transcode and push the stream to stunnel for FB Live RMTPS requirements
            exec ffmpeg -i rtmp://localhost:1935/facebook/$name
              -c:a libfdk_aac -b:a 128k -c:v libx264 -b:v 2500k -f flv -g 30 -r 30 -s 1280x720 -preset superfast -profile:v baseline -f flv rtmp://localhost:1935/fbpush/$name;
        }

        application fbpush {
                live on;
                on_publish http://myapi.host.com/fbpush; # response with 303 redirect to rtmp://stunnel:19350/rtmp/dynamic_key
        }


KNOX-1933 - Add rewrite rules to fix Yarn RM application and logs URL

Testing Done:
Deployed on a cluster with Knox and made sure these links worked

@moresandeep


AttributeError: 'FakeFcntlModule' object has no attribute 'LOCK_SH'  (Also 'LOCK_UN' and 'LOCK_EX')

Describe the bug
After updating from pyfakefs 5.6.0 to 5.7.0  the following error occurs on flock with fcntl.LOCK_SH,  fcntl.LOCK_UN and LOCK_EX.
The error does not occur on version 5.6.0
self = <pyfakefs.fake_io.FakeFcntlModule object at 0x7f237a9ba760>, name = 'LOCK_SH'

    def __getattribute__(self, name):
        """Forwards any unfaked calls to the standard fcntl module."""
        fs: FakeFilesystem = object.__getattribute__(self, "filesystem")
        fnctl_module = object.__getattribute__(self, "_fcntl_module")
        if fs.patcher:
            if is_called_from_skipped_module(
                skip_names=fs.patcher.skip_names,
                case_sensitive=fs.is_case_sensitive,
            ):
                # remove the `self` argument for FakeOsModule methods
                return getattr(fnctl_module, name)

>       return object.__getattribute__(self, name)
E       AttributeError: 'FakeFcntlModule' object has no attribute 'LOCK_SH'

How To Reproduce
import fcntl
from os import path

def lock():
    with open('lock_file', 'a+') as lock_file:
        fcntl.flock(lock_file, fcntl.LOCK_SH)
        fcntl.flock(lock_file, fcntl.LOCK_UN)

def test_lock(fs):
    lock()

Your environment
Please run the following in the environment where the problem happened and
paste the output.
python -c "import platform; print(platform.platform())"
python -c "import sys; print('Python', sys.version)"
python -c "from pyfakefs import __version__; print('pyfakefs', __version__)"
python -c "import pytest; print('pytest', pytest.__version__)"

python -c "import platform; print(platform.platform())"
python -c "import sys; print('Python', sys.version)"
python -c "from pyfakefs import __version__; print('pyfakefs', __version__)"
python -c "import pytest; print('pytest', pytest.__version__)"
Linux-6.10.11-200.fc40.x86_64-x86_64-with-glibc2.39
Python 3.9.16 (main, Sep  6 2024, 09:21:47)
[GCC 14.2.1 20240801 (Red Hat 14.2.1-1)]
pyfakefs 5.7.0
pytest 8.3.3

Should be fixed in main now, please check!
I'll probably wait a couple of days, in case something else comes up, and make a patch release.

I confirmed that it now works as expect.   Thanks for the quick fix!


Allow writing custom values to the depth buffer from a fragment shader

I'd like to propose writing custom values to the depth buffer from a fragment shader.
In GLSL, you can write to gl_FragDepth to put custom values in the depth buffer, but this functionality seems to be missing in Godot's shader language.
This would enable stuff like sphere impostors which draw a sphere that correctly intersects with other objects, using only 2 triangles and by writing custom values to the depth buffer. See this OpenGL tutorial: Chapter 13. Lies and Impostors

It also allows deep parallax to have correct intersection with other objects in the scene, and probably a lot of other effects I'm not aware of.

Added in #22800


site support request

hi, can someone please add these sites to youtube-dl
http://www.tntdrama.com/shows/animal-kingdom/clips/inside-the-episode-karma
http://www.tbs.com/shows/american-dad/clips/makeover-plans
thanks.

are they not possible to download with youtube-dl?

Duplicate of #13658.


How to set fontfamily?

It pop "Unrecognized font family 'DengXian'" error, how can i set in style that all will use as fontFamily:system ?
Or is that anyway to ignore the font-family style?
I have tried set baseFontStyle={{ fontFamily:"System" }} but same error shows out.
"
testing
"

@nolife08021 To use custom fonts, you need to ship them with your app. Read this thread for more information.

@nolife08021 To use custom fonts, you need to ship them with your app. Read this thread for more information.

should this work with expo and version 2.1.5? i am not able to make my custom font work.


Host //rust-lang.org over HTTPS

It is possible to host a GitHub Pages hosted website that has a custom domain over HTTPS. I did this for my website using this tutorial and it didn't take too long.
I asked about this a few minutes ago on IRC and @steveklabnik basically just said it hasn't been done yet, which is fine. I didn't see an existing issue open for this so if allowing HTTPS is something that is planned, then I figure it might be good to open a GitHub Issue to track the progress.

It looks like https://github.com/rust-lang/rust/issues/13180 is also open (which I apparently also commented in a few months ago) so if that's a better place for discussion, this can be closed

I agree that https://github.com/rust-lang/rust/issues/13180 is where discussion on this should be happening, so closing in favor of that, but thanks for the report!


NIFI-4224: Reopened Variable Registry at a Process Group Level

NIFI-4224:

Ensuring the parent permissions are used when building the variable registry for a process group ancestry.

@mcgilman good catch! Changes look good. +1 merged to master.


wandbLogger log Image does not seem to work.

🐛 Bug
in normal pytorch I would log an Image in wandb like this:
wandb.log({"Reconstruction":[wandb.Image(recon, caption="recon")]})
in lightning documentation it says:
self.logger.experiment.some_wandb_function()
I have used this to successfully log other metrics, but if I do this with Images:
self.logger.experiment.log({"val_input_image":[self.logger.experiment.Image(val_batch[0][0].cpu(), caption="val_input_image")]})
I get this error:
 in validation_step(self, val_batch, batch_idx)
44           print("val_batch", val_batch[0][0].shape)
45           # wandb.log({"pred_image":[wandb.Image(mask, caption="pred")], "epoch":outputDict["epochNum"]})
---> 46           self.logger.experiment.log({"val_input_image":[self.logger.experiment.Image(val_batch[0][0].cpu(), caption="val_input_image")]})
47         return {'val_loss': val_loss, "val_acc" : acc, "log" : val_logs}
48
AttributeError: 'Run' object has no attribute 'Image'
Any ideas how to get around this??
cc @borda @rohitgr7 @awaelchli @morganmcg1 @AyushExel @borisdayma @scottire

Do this:
self.logger.experiment.log({"val_input_image":[wandb.Image(val_batch[0][0].cpu(), caption="val_input_image")]})

With a bit of luck, I have not introduced a typo.

Just wasted an hour on this using pytorch-lightning            1.5.8 :(
This worked:
self.logger.experiment.log({"image": [wandb.Image(image)]})

while this did not worked, but is in the documentation:
self.log({"image": [wandb.Image(image)]})

https://pytorch-lightning.readthedocs.io/en/latest/common/loggers.html#weights-and-biases

Yes, sorry the docs would need to be updated. Recently we also introduced self.logger.log_image: https://pytorch-lightning.readthedocs.io/en/latest/extensions/generated/pytorch_lightning.loggers.WandbLogger.html#pytorch_lightning.loggers.WandbLogger.log_image, this should make your life easier too.

Hey all, I'm at Weights & Biases, we'll get this fixed :)

Yes, sorry the docs would need to be updated. Recently we also introduced self.logger.log_image: https://pytorch-lightning.readthedocs.io/en/latest/extensions/generated/pytorch_lightning.loggers.WandbLogger.html#pytorch_lightning.loggers.WandbLogger.log_image, this should make your life easier too.

Hi! Why do I have 404 page not found error visiting your link?

Hey @lqrhy3
Because I posted this link more than 1 year ago and it wasn't a "permanent" link. I hope you understand.
This is the link to the latest docs: https://lightning.ai/docs/pytorch/stable/api/lightning.pytorch.loggers.wandb.html
There is a section for logging media, including images, with example code.


improve support cutting planes

add functions for adding cuts
add function to access tableau information
add test of a separator

I guess, I need to disable Codecov -- or even better: correct the coverage calculation...


No text or icon in Stream Deck Plus dial when use script service

dial is working but dont have text and icon
  - service: script.stream_deck_adjust_brightness
    service_data:
      brightness: '{{ dial_value() | int}}'
    icon: >
      {%- set state = 10 -%}
      {%- set min = dial_attr("min") -%}
      {%- set max = dial_attr("max") -%}
      {%- set pct = ((state - min) / (max - min)) * 100 -%}
      ring:{{  pct | round }}
    text: >
        {{ states('input_text.stream_deck_selected_light') }}
    allow_touchscreen_events: true
    delay: 0.1
    dial_event_type: TURN
    attributes:
      min: 0
      max: 250
      step: 25

I don't have a Stream Deck with dials so cannot help you effectively.
Feel free to hit the Sponsor button ;)

Looking to do the same! Would you share your HA script for this?
@rick0ff

Create input text helper for store entity id

Ha script : Use for button for toggle light or switch
alias: Toggle Light by Data
sequence:
  - target:
      entity_id: "{{ device }}"
    action: homeassistant.toggle
  - action: input_text.set_value
    metadata: {}
    data:
      value: "{{ device }}"
    target:
      entity_id: input_text.stream_deck_selected_light
mode: single

Ha script : Use for dials to adjust brightness
alias: Stream Deck Adjust Brightness
sequence:
  - action: light.turn_on
    metadata: {}
    data:
      brightness_pct: "{{brightness}}"
    target:
      entity_id: "{{ states('input_text.stream_deck_selected_light') }}"
description: ""

Streamdeck config (dial can use as normol but will not show anything on screen)
  buttons:
    - service: script.light_toggle_by_data
      service_data:
          device: switch.left_wall_switch
      text: |
          Main Lights
      icon_mdi: light-recessed
      icon_mdi_color: "{{ 'yellow' if is_state('switch.left_wall_switch', 'on') else 'gray' }}"
    - service: script.light_toggle_by_data
      service_data:
          device: switch.left_wall_switch
      text: |
          Main Lights
      icon_mdi: light-recessed
      icon_mdi_color: "{{ 'yellow' if is_state('switch.left_wall_switch', 'on') else 'gray' }}"

  dials:
    - service: script.stream_deck_adjust_brightness
      service_data:
        brightness: '{{ dial_value() | int}}'
      text: >
          {{ states('input_text.stream_deck_selected_light') }}
      allow_touchscreen_events: true
      delay: 0.1
      dial_event_type: TURN
      state_attribute: editable
      attributes:
        min: 0
        max: 250
        step: 50

Ah, you're trying to change brightness of a light. I am looking to adjust the brightness of the Stream Deck itself through a dial.
But i can see in the last config, that the "ring"-statement is missing, which is the one doing the ring-animation. Maybe my dial-config might help (it certainly won't win any beauty contests):
`# Screen Bar

entity_id: light.screenbar
service: light.turn_on
service_data:
brightness: >
{%- set min = dial_attr("min") -%}
{%- set max = dial_attr("max") -%}
{%- set bright = (dial_value() - min) / (max - min) * 255 -%}
{{ bright | int }}
icon: >
{%- set state = dial_value() -%}
{%- set min = dial_attr("min") -%}
{%- set max = dial_attr("max") -%}
{%- set pct = ((state - min) / (max - min)) * 100 -%}
ring:{{ pct | int }}
text: >
{%- set min = dial_attr("min") -%}
{%- set max = dial_attr("max") -%}
{%- set scale = (dial_value() - min) / (max - min) -%}
{%- set percenttmp = scale | round (1) -%}
{%- set percent = percenttmp * 100 -%}
{%- set ha_state = states('light.screenbar') -%}
{%- if ha_state == "off" and dial_value() == 0 -%}
Screen Bar
off
{%- else -%}
Screen Bar
{{percent | int}} %
{%- endif -%}
text_size: 18
state_attribute: brightness
allow_touchscreen_events: true
delay: 0
dial_event_type: TURN
attributes:
min: 0
max: 255
step: 26`

Sorry for that , my workaround in config other dial has only text its worked.
But yesterday i got something about missing entity_id or target i cant reproduce now its display but script didnt work.
Maybe i need too read some code more for this
Thank you all


How to use this nice util?

I suppose it's a plug-in for TLCV?
I like to replace it with my crappy one - http://rebel13.nl/pgn4web-3.05/live-test.html
I agree with Ferdy the board size is a little bit too big.

Hey!
This is not a plugin, but a server that I've deployed and connects to Graham's TLCV broadcasts. If you'd like to use it yourself, I could work to make it so the setup is more configurable (at this time a lot of values are specific to Graham's broadcasts).
If you'd like to switch to it quickly, I could also show you what exact values to change. Let me know if you have any other questions.

I agree with Ferdy the board size is a little bit too big.

I agree. I'm quite busy with outside stuff at this time, but plan to address styling preferences in the form of customization when I have a bit of time.

A setup would be great!
And I am not in a hurry :-)


Treatment of supervising organization vs station contacts in OSCAR/Surface vs WMDS vs WMDR

WMDS
Element 9-01 is supervising organization without specific cardinality
WMDR
ObservingFacility\responsibleParty is of type RepsonsibleParty which is modeled using CI_ResponsibleParty and has cardinality  [1..*]. CI_RespojnsibleParty allows specification of individual Name, etc
OSCAR/Surface
The element is interpreted as WMDS element 9-01, and only the organization is extracted. Cardinality [1..1] is imposed at any one time.
--> Need to agree on cardinality and range of elements (only organization, or full contact) that can be expressed.

Moved to https://github.com/wmo-im/wmdr/issues/18


Link to include folder seems wrong

https://github.com/cpputest/cpputest/blob/eb71de9015f04e078a17b707f7182f2bc2a66f00/build/MakefileWorker.mk#L279
Hello
$(CPPUTEST_HOME) = /usr/share/cpputest/ per default
I installed using apt-get install cpputest and I saw that my includes are in
CPPUTEST_MEMLEAK_DETECTOR_NEW_MACRO_FILE=/usr/include/CppUTest/MemoryLeakDetectorNewMacros.h
CPPUTEST_MEMLEAK_DETECTOR_MALLOC_MACRO_FILE=/usr/include/CppUTest/MemoryLeakDetectorMallocMacros.h
and not in the ones listed in the file& line i've referenced. Why is this? Do I need to move the includes? I also installed the script with the setup.sh file

My initial reaction is that this is indeed wrong and ought to be fixed.
That said, the MakefileWorker.mk isn't really made for using with the apt-get install. Still, can be fixed.

I'm not going to touch the Makefile.worker further for now. It hasn't been created with apt-get install in mind as it assumes the CppuTest git repository structure.


Need support for gfx1100 (7900xtx)

Without the optimization for gfx11, the advance of the        Navi31 new architecture cannot be taken into full play.
Personally I just want a higher speed for picture  generation in sdwebui.

I have no plan on working on AMD cards myself (and I have no experience with AMD cards), but the AMD folks seem to have implemented some version of FlashAttention here.

More instructions on FlashAttention on ROCm here:
https://github.com/ROCmSoftwarePlatform/flash-attention/tree/flash_attention_for_rocm2#amd-gpurocm-support

It looks like it hasn't been updated or maintained in a while.. Thanks anyway.