[FEATURE] Kop needs to support the v0 version of SASL_HANDSHAKE

Is your feature request related to a problem? Please describe.
If SaslHandshakeRequest version is v0, a series of SASL client and server tokens corresponding to the mechanism are sent as opaque packets without wrapping the messages with Kafka protocol headers. However, KoP always parses the header before a request is processed.
Describe the solution you'd like
We need support sasl tokens are sent without Kafka SaslAuthenticate request/responses.
This feature belongs to the compatibility issue of the old version of kafka with kop.

@BewareMyPower I want to try to work on this feature, can you assign it to me first? Thanks.


Update banners in Wifi/BLE Mode

Updated UI banner. Will need to add the "Disconnect" functionality for BLE Mode.


No address associated with hostname

I get the error: No address associated with hostname
What is the problem?

try -H <IP_ADDRESS>


Relations don't work even with default values

Overview of the problem
When you add our module but keep all values default, you would expect the app to continue to work. But it doesn't.
With the following configuration :
Author
{
    "fluentMethods": true,
    "relationships": [
        {
            "relationshipName": "hisBook",
            "otherEntityName": "book",
            "relationshipType": "one-to-many",
            "otherEntityRelationshipName": "theAuthor"
        }
    ],
    "fields": [
        {
            "fieldName": "name",
            "fieldType": "String"
        },
        {
            "fieldName": "birthDate",
            "fieldType": "LocalDate"
        }
    ],
    "changelogDate": "20170418140037",
    "dto": "no",
    "service": "no",
    "entityTableName": "author",
    "pagination": "no"
}

Book
{
    "fluentMethods": true,
    "relationships": [
        {
            "relationshipName": "theAuthor",
            "otherEntityName": "author",
            "relationshipType": "many-to-one",
            "otherEntityField": "name"
        }
    ],
    "fields": [
        {
            "fieldName": "title",
            "fieldType": "String"
        },
        {
            "fieldName": "description",
            "fieldType": "String"
        },
        {
            "fieldName": "publicationDate",
            "fieldType": "LocalDate"
        },
        {
            "fieldName": "price",
            "fieldType": "BigDecimal"
        }
    ],
    "changelogDate": "20170418145015",
    "dto": "no",
    "service": "no",
    "entityTableName": "book",
    "pagination": "no"
}

We get the following error message :

Caused by: org.hibernate.tool.schema.spi.SchemaManagementException: Schema-validation: missing column [theAuthor_id] in table [book]

The column is indeed missing because in the following files :
./src/main/resources/config/liquibase/changelog/20170418145015_added_entity_constraints_Book.xml
./src/main/resources/config/liquibase/changelog/20170418145015_added_entity_Book.xml

We have the_author_id and not theAuthor_id. Both are generated from the relationshipName but the former is transformed by jhipster which uses snake_case and the latter just appends _id. Most probably, JHipster was expecting hibernate to apply SpringNamingStrategy which is not the case thanks to our module.
Suggest a fix
We probably will have to find pertinent value in either the ORM or the changelog and make on to copy the other.
Versions & configurations
JHipster 4.3.0

Ok, I confirmed that manually replacing the_author_id by theAuthor_id in the two concerned files correct the problem.
./src/main/resources/config/liquibase/changelog/20170418145015_added_entity_constraints_Book.xml
./src/main/resources/config/liquibase/changelog/20170418145015_added_entity_Book.xml

This option seems easy to apply, let's do it.


In search of example CircleCI config.yml

First off, thanks for open sourcing this!
I'm having some trouble using this plugin with CircleCI. To be clear, my issue is with CircleCI itself: I seem to be unable to persist my jest output, test-results.json, from one job to the next when the tests fail.
Tl;dr: the persist_to_workspace step doesn't run after my test step fails in the same job, so I have no way of accessing the results in the next job where I run Danger.
If anyone can shed some light on this issue, I'd appreciate it! I fully expect this issue to be closed immediately, but thought it was worth a shot in case anyone can share a working config.yml, or has suggestions. Thanks!

Would you be able to use the store_test_results or store_artifacts steps to access the test-results.json file (looking at these Circle CI docs). I'm not that familiar with the Circle CI 2.0, so I'm sorry I can't provide more specific help here.

Thanks for the suggestions @macklinu! Looks like store_test_results and store_artifacts only work with files stored in XML xUnit format from what I can see.
In the end I was able to get this working by running the tests inside a shell script that can optionally record the failure in an env var, then return 0 so the persist_to_workspace step can run.
In theory, after that you could have a new step that conditionally throws an error based on the env var, but that turned out not to be necessary because the danger job with danger-plugin-jest will fail the build once it can read the test results. Here's a gist with my jobs config if it's useful to anyone. Thanks again!


Using tf.estimator.Estimator with save_checkpoint_steps leads to Tensorboard warnings

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): No
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Linux 4.4.0-104-generic
TensorFlow installed from (source or binary): binary
TensorFlow version (use command below): v1.4.0-rc1-11-g130a514 1.4.0
Python version: 3.6.4
CUDA/cuDNN version: 8.0.61
GPU model and memory: GeForce GTX TITAN X

Describe the problem
When using tf.estimator.train_and_evaluate(...) with an tf.estimator.Estimator configurated with tf.contrib.learn.RunConfig(save_checkpoints_steps=10, ...), a CheckpointSaverHook will be created automatically. This CheckpointSaverHook will save the graph and graph_def to the summary writer every time it is triggered (see CheckpointSaverHook.before_run).
Basic code example:

estimator = tf.estimator.Estimator(
    model_fn, model_dir, params,
    config=tf.estimator.RunConfig(
        save_checkpoints_steps=100,
        save_summary_steps=100
    )
)
train_spec = tf.estimator.TrainSpec(train_fn)
eval_spec = tf.estimator.TrainSpec(eval_fn)
tf.estimator.train_and_evaluate(estimator, train_spec, eval_spec)

When starting Tensorboard on the written summary, it will output a hundreds of warnings because of multiple graph defs in the summary which I guess slows it down a lot on startup:

W0117 18:47:30.278879 Reloader tf_logging.py:86] Found more than one graph event per run, or there was a metagraph containing a graph_def, as well as one or more graph events.  Overwriting the graph with the newest event.
W0117 18:47:30.279753 Reloader tf_logging.py:86] Found more than one metagraph event per run. Overwriting the metagraph with the newest event.

I see there might be issues when using multiple graphs, but for a single graph this seems unpracticable.
Related stack overflow discussion: https://stackoverflow.com/questions/48316888

I don't think there's enough information here to identify a TF bug. Consider asking another question on Stack Overflow with the tensorboard tag.

I am having the same issue, is it possible some workaround or to understand what this is due to?
tf version == 1.11.0
I tried to use all three possible MonitoredSessions (that I know of..) and the result is always the same, that is why I think it is a problem of the CheckpointSaverHook
It seems like more processes are writing on the event for saving the graph maybe.
I am not sure how many processes are launched under the hood, but only the chief should write the tf.summaries I think, isn't this the default behaviour or what is the suspected issue here?
For all the 3 following cases the model is saved, and together with is the graph in a tb event I guess...
when I read the respective folder with tensorboard I obtain:
W1011 18:27:17.685001 Reloader tf_logging.py:120] Found more than one graph event per run, or there was a metagraph containing a graph_def, as well as one or more graph events.  Overwriting the graph with the newest event.
W1011 18:27:17.685001<PHONE_NUMBER>26944 tf_logging.py:120] Found more than one graph event per run, or there was a metagraph containing a graph_def, as well as one or more graph events.  Overwriting the graph with the newest event.
W1011 18:27:17.920676 Reloader tf_logging.py:120] Found more than one metagraph event per run. Overwriting the metagraph with the newest event.
W1011 18:27:17.920676<PHONE_NUMBER>26944 tf_logging.py:120] Found more than one metagraph event per run. Overwriting the metagraph with the newest event.

A simple code presenting the issue:
hooks = [
    tf.train.StopAtStepHook(last_step=300)
]

sessioncode = args.session
SAVE_STEPS = 50

if sessioncode == 'mts':
    print("I am using a MonitoredTrainingSession!!")
    basedir = "tempMonitoredTrainingSession"
    checkpoint_dir = basedir
    # summary_dir = basedir+"/tensorboard"

    sess = tf.train.MonitoredTrainingSession(checkpoint_dir=checkpoint_dir,
                                            save_checkpoint_steps = SAVE_STEPS,
                                            # summary_dir=summary_dir,
                                            hooks=hooks,
                                            config=sess_config)

elif sessioncode == 'ms':
    print("I am using a MonitoredSession!!")
    basedir = "tempMonitoredSession"
    checkpoint_dir = basedir
    # summary_dir = basedir+"/tensorboard"
    hooks.append(tf.train.CheckpointSaverHook(checkpoint_dir, save_steps = SAVE_STEPS))

    chiefsess_creator = tf.train.ChiefSessionCreator(config=sess_config, checkpoint_dir=checkpoint_dir)
    sess = tf.train.MonitoredSession(session_creator=chiefsess_creator, hooks=hooks)

elif sessioncode == 'sms':
    print("I am using a SingularMonitoredSession!!")
    basedir = "tempSingularMonitoredSession"
    checkpoint_dir = basedir
    # summary_dir = basedir+"/tensorboard"
    hooks.append(tf.train.CheckpointSaverHook(checkpoint_dir, save_steps = SAVE_STEPS))

    sess = tf.train.SingularMonitoredSession(checkpoint_dir=checkpoint_dir,
                                            hooks=hooks,
                                            config=sess_config)

else:
    raise ValueError("the session code passed is not contemplated!")

with sess:
    while not sess.should_stop():
        _, g_step = sess.run([training_op, global_step])
        print("global step: ", g_step)


[BUG] spv-store sync block fail

Description:
When using the application in China (via VPN), the network frequently experiences issues such as read tcp xxx: i/o timeout. This results in a poor user experience due to the API request failing without retrying.
Additionally, syncing fewer blocks would help optimize performance and reduce the likelihood of encountering these errors.
Steps to Reproduce:

Use the application from China with a VPN enabled.
Attempt to create new wallet.
Observe timeout errors like read tcp xxx: i/o timeout.

Expected Behavior:

The API should retry requests when encountering timeout errors to increase resilience against temporary network issues.
Support for syncing a smaller number of blocks to reduce strain on the network.

Suggestions:

Retry Mechanism: Implement a retry logic for API requests with an exponential backoff to handle intermittent connectivity issues gracefully.
Optimize Block Syncing: Provide an option to sync fewer blocks, particularly for regions with limited or unstable network connections.

Environment:

Location: China
VPN: Yes
Error: read tcp xxx: i/o timeout

Thanks for raising -- wrong repo. I believe this is related to the browser extension wallet. i'll move it.


Replace tmpdir usage with tmp_path in our tests

Outline & Motivation
tmpdir is considered legacy:

The tmpdir and tmpdir_factory fixtures are similar to tmp_path and tmp_path_factory, but use/return legacy py.path.local objects rather than standard pathlib.Path objects.

(from https://docs.pytest.org/en/7.1.x/how-to/tmp_path.html#the-tmpdir-and-tmpdir-factory-fixtures)
Pitch
Run pytest with pytest -p no:legacypath as the link above suggests and replace all occurrences.
This can be added in https://github.com/Lightning-AI/lightning/blob/94e6d52b7e2f2a9ffc21f7e11e087808666fe710/setup.cfg#L28-L32
 addopts =
     --strict-markers
     --doctest-modules
     --color=yes
     --disable-pytest-warnings
     --ignore=legacy/checkpoints
+    -p no:legacypath

Replace all tmpdir usages
Additional context
No response
cc @borda

Hello @carmocca ! I would like to Contribute My first Open Source Contribution. If there is something I could help you out with or fix some issues Please do ping me.
Also if this issue is open can you please assign it to me?

Go ahead!

Happy to contribute to this issue if no one has been assigned to it!

Hello, @yhl48 are you still working on this? I'm planning to work on this, would love to have your input on this. Thanks

@carmocca, in which file tmpdir with tmp_path should be replaced

@carmocca, in which file tmpdir with tmp_path should be replaced

in all files :)

All files of lightning-AI, can I work on it.

All files of lightning-AI, can I work on it.

Let's split is per topic:

[ ] integrations_app + legacy
[ ] parity_fabric + parity_pytorch
[ ] tests_app
[ ] tests_data + tests_store
[ ] tests_fabric
[ ] tests_pytorch
[ ] tests_store

May I do the above 4 files.

@Borda Got it, replacements have to be done on tests file, I'm starting working on tests files.

@Borda  will you please check my pull request and the changes I have done.

@Borda will you please check my pull request and the changes I have done.

I think it would be easier/faster if you create a PR per check item here: https://github.com/Lightning-AI/lightning/issues/16188#issuecomment-1691189400

@Borda , what we should do files, give me a idea

@Borda , what we should do files, give me a idea

Let's create another PR and perform replacement just for one sub folder as listed in comment/check-list above...

@Borda what replacement do we need to perform and on which sub-folder

@Borda what replacement do we need to perform and on which sub-folder

Could you please ping me on slack, would be easier to explain =)

@Borda , yaa sure tell me your slack Id

@Borda , yaa sure tell me your slack Id

You can find me as Jirka :)
I made a sample partial fix, so based on partitions in https://github.com/Lightning-AI/lightning/issues/16188#issuecomment-1691189400, I just chose a topic/folder and tried to apply changes...

chose one folder and create a draft PR (which can be almost empty) that we can link the issues
take it file by file and perform replacement
I would suggest each file change shall be one commit in the PR so it is easier to trace what changes are not compatible

@Borda ok, will do that.

@Borda is this still open for contribution, I would love to work on it 😊


broken URL fixes Nov 2024

clean up obsolete broken URLs
fix the raw .jsonld file link in the governance action example

Bonus: added the announcementBar switch in the docusaurus.config.js as just implemented on cardano.org

LGTM


Error 500 - possibly due to missing B-year

Looks like missing B-year is returning Error 500 - FUNCTION_INVOCATION_FAILED.
Some examples from the Fide Chess Olympiad 2024:
https://ratings.fide.com/profile/11305924
https://ratings.fide.com/profile/16700309
https://ratings.fide.com/profile/17700213
https://ratings.fide.com/profile/241000013 (also missing World Rank (All players) - but not sure if this is also an issue)
https://ratings.fide.com/profile/6100678

Hi @quinnphil , good night! The Olympiad was an awesome event!
It was an issue with missing data on some profiles that I just fixed. Can you confirm it's working now?
https://fide-api.vercel.app/player_info/?fide_id=241000013&history=false

Amazing - thank you @cassiofb-dev.
Confirmed as working for all examples.
The Olympiad was indeed an awesome event.  That Gukesh v. Wei Li game was incredible.


Another Case not Appearing in Search Results

Search for 'European Local Democracy Week' only returns the method entry and the case entry transferred from .net (image 1) but we're missing the most recent case entry (image 2)
IMAGE 1

IMAGE 2

see search indexing Master Issue: participedia/api#453


Add points of round and each player to the score sheet

Story #16 Add Points to Score Sheet
Sub-Tasks:
- [ ] Add points of each player to score sheet via REST /lobbies/{lobbyId}/score/{playerId}
Estimate: 0.5h

Different logic. Issue closed.


[Container Service]: Add Swagger for 2017-06-28

👋 hey y'all
The Azure Container Service recently announced support for the 2017-06-28 API - which solves a ton of issues we've raised previously, including:

#833 - Windows support
#1172 - Internal Clusters
https://github.com/terraform-providers/terraform-provider-azurerm/issues/79 - Tearing down Azure ACS resource does not tear down resource group
https://github.com/terraform-providers/terraform-provider-azurerm/issues/45 - Multiple Agent Pools

Would it be possible to add the Swagger file associated with this API version?
Thanks!

@colemickens can you please follow up on this (I believe this is your team, let me know if I got it wrong)?

@jhendrixMSFT @kirthik 👋🏻 given that @colemickens no longer works at Microsoft, would it be possible to re-assign this to someone else on the Container Service team? Thanks! :)

@rjtsdl - Could you please provide an ETA?

@Krithik and all,
The swagger change for 2017-07-01 version is in, but not used to build client sdk yet.
The reason is, ACS is migrating to a new code base. The timeline should be around 2 months from now. We should be able to open this api version 2017-07-01 globally right after that.
Btw, there is really no 2017-06-28, it is 2017-07-01. :) @kirthik, could you help to update the issue title?

@rjtsdl sorry I missed this; thanks for clarifying that - I've updated the title :)
In Terraform we use the API to provision Container Service clusters, as such we're unable to make use of the CLI suggestion - but thanks anyway!
As such - I'll set a reminder to check this in mid-November - but would it be possible to also post here if it's deployed before then? :)
Thanks!

@tombuilsstuff sure.
Once it is deployed, I would definitely want to resolve this. I will post here.

Hey @rjtsdl
Just checking back in here - I've taken a look in the Swagger repository but I can't see the updated Swagger Spec in Master? Is there an updated timeline for resolving this?
Thanks!

ping @rjtsdl - is there any update available here? :)

@anhowe see if there is something we can share about migration plan. Thx

I think this issue will be closed by #1956.


chore: release notes for 4.10.0

Includes some 4.9.0 entries for changes made after the release notes were written.

Thanks @tydeu for the suggestions. And for backported changes missing entries, I hope I've sorted it out now!


About the PRO-score

As the title, Do you know how to calculate the PRO-score for segmentation?
Sincerely hope that you can take time to reproduce it.

Maybe you can refer to this project:
https://github.com/YoungGod/DFR/blob/a942f344570db91bc7feefc6da31825cf15ba3f9/DFR-source/anoseg_dfr.py#L447

Maybe you can refer to this project:
https://github.com/YoungGod/DFR/blob/a942f344570db91bc7feefc6da31825cf15ba3f9/DFR-source/anoseg_dfr.py#L447

Maybe you can refer to this project:
https://github.com/YoungGod/DFR/blob/a942f344570db91bc7feefc6da31825cf15ba3f9/DFR-source/anoseg_dfr.py#L447

thx very much!

Maybe you can refer to this project:
https://github.com/YoungGod/DFR/blob/a942f344570db91bc7feefc6da31825cf15ba3f9/DFR-source/anoseg_dfr.py#L447

thx very much!


Exec no payload and complete

[ ] Bugfix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)

Proposed changes

This PR does two things...

It changes the default from "append msg.payload" - to be not appended. - Quite often we get users saying a simple command like ls doesn't work - which is because the default is to append a payload which again by default is often the timestamp from an inject node - so the command fails. Making the default to not be appended should fix this - and indeed it should be more obvious that if you want to append the payload you need to tick it...

It adds msg.complete = true to the 3rd output of the node when in spawn mode. This is the output that returns the return code - but by adding msg.complete = true it can also be used to feed a join node to rejoin all the output from the spawned command. (However it still needs a change node to delete the payload otherwise that also gets joined... :-(

Checklist

[ ] I have read the contribution guidelines
[ ] For non-bugfix PRs, I have discussed this change on the forum/slack team.
[ ] I have run grunt to verify the unit tests pass
[ ] I have added suitable unit tests to cover the new/changed functionality

Coverage decreased (-0.04%) to 68.534% when pulling 7152d02d91aa2cd5c72eea74bf832c486b74f6a7 on exec-no-payload-and-complete into c40412d7c6036bab4ddf1656a3883717d12dcc9c on dev.

Coverage decreased (-0.04%) to 68.534% when pulling 7152d02d91aa2cd5c72eea74bf832c486b74f6a7 on exec-no-payload-and-complete into c40412d7c6036bab4ddf1656a3883717d12dcc9c on dev.

Looks good - happy for you to merge.

Looks good - happy for you to merge.


Issue with k8s.io/docs/setup/production-environment/turnkey/gce/

This is a Bug Report

Problem:
After clicking the button to "open a Google Cloud Shell with an auto-cloned copy of the Kubernetes source repo."
https://console.cloud.google.com/cloudshell/open?git_repo=https://github.com/kubernetes/kubernetes&page=editor&open_in_editor=README.md
I attempt to run make and receive the following error
josh_reich@cloudshell:~/cloudshell_open/kubernetes$ make Detected go version: go version go1.13 linux/amd64. Kubernetes requires go1.14.4 or greater. Please install go1.14.4 or later. !!! [0702 15:41:32] Call tree: !!! [0702 15:41:32]  1: hack/run-in-gopath.sh:31 kube::golang::setup_env(...) Detected go version: go version go1.13 linux/amd64. Kubernetes requires go1.14.4 or greater. Please install go1.14.4 or later. !!! [0702 15:41:32] Call tree: !!! [0702 15:41:32]  1: /home/josh_reich/cloudshell_open/kubernetes/hack/lib/golang.sh:780 kube::golang::setup_env(...) !!! [0702 15:41:32]  2: hack/make-rules/build.sh:27 kube::golang::build_binaries(...) !!! [0702 15:41:32] Call tree: !!!  [0702 15:41:32]  1: hack/make-rules/build.sh:27 kube::golang::build_binaries(...) make[1]: *** [Makefile.generated_files:195: _output/bin/prerelease-lifecycle-gen] Error 1 make: *** [Makefile:544: generated_files] Error 2
It is unclear how to upgrade the version of go used by cloudshell (some quick tries using apt failed). This probably isn't difficult, but having to spend more than a couple of minutes figuring out how to get this working completely contradicts the whole purpose of a "one click" to launch and experiment with k8s.
Proposed Solution:
Page to Update:
https://kubernetes.io/...

This list of issues tracks bug reports and feature requests for the Kubernetes documentation.
If you're looking for help, check Stack Overflow
You can also post your question on the Kubernetes Slack or the Discuss Kubernetes forum.
/triage support
/close


E2E: Add TestNodepoolTunedConfigGetsRolledout

What this PR does / why we need it:
This adds a new test that checks whether an addition to nodepool.spec.tunedConfig causes a roll out of a Tuned CR in hosted cluster.  This is a PR that accompanies https://github.com/openshift/hypershift/pull/1651 and will not work without it.
Jira story.
Checklist

[x] Subject and description added to both, commit and PR.
[x] Relevant issues have been referenced.
[ ] This change includes docs. (n/a)
[ ] This change includes unit tests. (n/a)

https://github.com/openshift/hypershift/pull/1651 needs to be merged first
/hold

/retest

/approve

Thank you for looking Alberto!
/hold cancel

@csrwng, anything I can do/adress to help this PR move forward?  Thank you!

/close by https://github.com/openshift/hypershift/pull/1729


Deploy failed Supabase in Azure Kubernetes Service

Bug report

[x] I confirm this is a bug with Supabase, not with my own application.
[x] I confirm I have searched the Docs, GitHub Discussions, and Discord.

Describe the bug
I try using this repo to deploy supabase in K8S, but when deploy with this repo, can't deploy to K8S, 2 pod supabase-auth, supabase-rest show error:
https://github.com/supabase-community/supabase-kubernetes
To Reproduce
Steps to reproduce the behavior:

Clone git https://github.com/supabase-community/supabase-kubernetes
cd supabase-kubernetes/charts/supabase/

Create JWT secret

kubectl -n default create secret generic demo-supabase-jwt \
  --from-literal=anonKey='eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.ewogICAgInJvbGUiOiAiYW5vbiIsCiAgICAiaXNzIjogInN1cGFiYXNlIiwKICAgICJpYXQiOiAxNjc1NDAwNDAwLAogICAgImV4cCI6IDE4MzMxNjY4MDAKfQ.ztuiBzjaVoFHmoljUXWmnuDN6QU2WgJICeqwyzyZO88' \
  --from-literal=serviceKey='eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.ewogICAgInJvbGUiOiAic2VydmljZV9yb2xlIiwKICAgICJpc3MiOiAic3VwYWJhc2UiLAogICAgImlhdCI6IDE2NzU0MDA0MDAsCiAgICAiZXhwIjogMTgzMzE2NjgwMAp9.qNsmXzz4tG7eqJPh1Y58DbtIlJBauwpqx39UF-MwM8k' \
  --from-literal=secret='abcdefghijklmnopqrstuvwxyz123456'

Create SMTP secret

kubectl -n default create secret generic demo-supabase-smtp \
<EMAIL_ADDRESS>\
  --from-literal=password='example123456'

Create DB secret

kubectl -n default create secret generic demo-supabase-db \
  --from-literal=username='postgres' \
  --from-literal=password='example123456'

Install the chart
helm -n default install demo -f values.example.yaml .

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots

System information

OS: [e.g. macOS, Windows]
Browser (if applies) [e.g. chrome, safari]
Version of supabase-js: [e.g. 6.0.2]
Version of Node.js: [e.g. 10.10.0]

Additional context
Add any other context about the problem here.

Anyone can support?

I had to change these tags to make it work:
rest:
  image:
    repository: postgrest/postgrest
    tag: latest

# Auth Service
auth:
  enabled: true # Disable the auth service
  image:
    repository: supabase/gotrue
    pullPolicy: IfNotPresent
    tag: "v2.145.0"

Supabase has stopped releasing images with latest tag for a while now. The newly merged #48 with the provided values.example.yaml has the latest image tag for all services respectively.

As a side note, the values.example.yaml is missing enabled: true for:

studio
auth
rest
realtime
meta
storage
imgproxy
analytics
vector
functions

Not sure if this is intentional but just mentioning it here vs opening new issue ticket or PR (which I certainly can if any of maintainers desires me to do so.)

@bartekus I'm afraid that the <service>.enable might not be as functional as you think.  It may be deprecated in future releases except for db and minio which can be disabled to allow for external providers.


Add directory watcher in inferred projects

This PR added directory watchers in inferred projects to detect the addition of tsconfig.json in the future. This fixes the workaround for issue #3244.

as we talked offline, closing this in favor of a PR addressing #4643 instead.


«&» are escaped as «&» resulting in bad calls

I was trying to asynchronously load SmartAdServer requests, but the urls are rewritten:
https://jsfiddle.net/x1m6mvv6/5/
Request URL:https://ww62.smartadserver.com/ac?nwid=0&siteid=2301&pgid=378531&fmtid=438&visit=m&tmstp=5021521458&orgfmtid=438&tag=sas_438&pgDomain=https://fiddle.jshell.net
=> should not have «&»
I know that SmartAdServer provides an asynchronous mode, but I have some passbacks that won't work after the main call.
Maybe related to: https://github.com/krux/postscribe/issues/98
Thanks.

Hi @dugwood,
I've looked into this more.  The escaping actually appears to be coming from SAS themselves.  I modified your fiddle as follows:

replaced the postscribe call with an inline call sas.call("std", { siteId: 2301, pageId: 378531, formatId:  438, target: ''});
clicked on https://ww62.smartadserver.com/config.js?nwid=62 which redirected to https://ww62.smartadserver.com/diff/js/smart.js.
copies the contents of that file
pasted the contents of that file at the top of the JavaScript pane
removed the external resources
added the following code at the top of the JavaScript pane: document.write = console.debug;
clicked Run and checked the Developer Tools console and observed:

<script id="sas_script1" type="text/javascript" src="https://ww62.smartadserver.com/ac?nwid=0&siteid=2301&pgid=378531&am…43&orgfmtid=438&tag=sas_438&pgDomain=https://fiddle.jshell.net"></script>

I think I won't have to use it anymore, but so far this document.write seems good to me: & should be escaped as & in url that are part of an attribute. But if there's a call (img.src or ajax call), those & should be dealt as &.
Thanks for your time and great script anyway.


koord-scheduler: support reservation score normalization

Ⅰ. Describe what this PR does
The plugin DeviceShare supports scoring reservations for multiple resources, and returns scores that exceed MaxReservationScore(100), so koord-scheduler should normalize the scores after the plugin scores the reservations.

Ⅱ. Does this pull request fix one issue?
fix #1613

Ⅲ. Describe how to verify it
Ⅳ. Special notes for reviews
V. Checklist

[x] I have written necessary docs and comments
[x] I have added necessary unit tests and integration tests
[x] All checks passed in make test

/lgtm


delete the hosted cluster after the managed cluster is deleted

Signed-off-by: Wei Liu<EMAIL_ADDRESS>

/assign @zhujian7

refer to https://github.com/stolostron/backlog/issues/20960

/lgtm


[zh] Remove cluster-resources/binding-v1.md

deleted: content/zh-cn/docs/reference/kubernetes-api/cluster-resources/binding-v1.md

This file has been relocated to /workload-resources/binding-v1.md included in PR #47807

/approve

/lgtm


New install path for the Xcode command line tools confuses new brew installations

Hi,
I got a new MacBook Pro and installed Mojave right after the first boot. I then tried to install homebrew but the window brew is opening to install the CLI tools kept failing with the following message:

Can’t install the software because it is not currently available from the Software Update server.

I decided to install Xcode and then install the CLI tools when opening Xcode. I did just that, tried to install homebrew again and hit the same error message.
While investigating I found out that the CLI tools were now in /Applications/Xcode.app/Contents/Developer instead of /Library/Developer/CommandLineTools!
I ran sudo ln -s /Applications/Xcode.app/Contents/Developer /Library/Developer/CommandLineTools and was able to proceed with the installation of homebrew. Maybe you should check for the CLI tools in /Applications/Xcode.app/Contents/Developer now too.

It does happen sometimes that Apple's software update server contains no CLI tools during transitions in their tools releases, or due to some configuration mishap on their end (usually I've seen it resolve itself within a day or so).
An installation of Xcode contains a full suite of the CLI tools. I would expect brew to detect that if you have a supported Xcode installation, that it should consider developer tools already installed and continue happily along.
Symlinking /Applications/Xcode.app/Contents/Developer to /Library/Developer/CommandLineTools is likely to confuse things further and I wouldn't recommend it.

I decided to install Xcode and then install the CLI tools when opening Xcode.

That doesn't install the CLI tools. To do so you need to run xcode-select --install. If that fails, you need to download them manually from developer.apple.com/downloads/

While investigating I found out that the CLI tools were now in /Applications/Xcode.app/Contents/Developer instead of /Library/Developer/CommandLineTools!

These are not equivalent.

I ran sudo ln -s /Applications/Xcode.app/Contents/Developer /Library/Developer/CommandLineTools and was able to proceed with the installation of homebrew.

This will break things.

Please always fill out the issue template in future.

Sorry for not using the issue template, because of a bad timing I thought it was a Firefox extension messing with the textarea content.
You are both right that the symlink was a bad idea, macOS wanted to download updates for all CLI tools since Sierra (!) and failed doing that, I had to remove the symlink.
The symlink is what allowed me to install homebrew thought. Note that https://developer.apple.com/downloads/ still don't provide the CLI tools for 10.14 and Xcode 10.1 final, so installing homebrew would have been impossible for my setup.


CAMEL-15105: make the CLI ConnectorFactory a plugin of the context

This is a proposed way to make the access to the ExtendedExchange plugins uniform. At first, this only implements this for the CliConnectorFactory. If accepted, I'll implement the same pattern for the others.

Also, marking as a draft to wait for the test results on my own CI.


Move default port configuration to JavalinConfig

By default, Javalin starts on port ::8080 and there's no option to configure this behavior within the configuration scope.
https://github.com/javalin/javalin/blob/4a6240bf57c161d8f6dc0bea5e2edf20aa05da42/javalin/src/main/java/io/javalin/Javalin.java#L121-L129
It'd be nice, if we could move this property to cfg:
val app = Javalin
  .create(cfg -> {
    cfg.defaultPort = 1234;
  })
  .start();

Effectively, we could even simplify this to:
// default ::8080
val app = Javalin.start()

// custom port/other cfg properties
val app = Javalin.start(cfg -> {
  cfg.defaultPort = 1234;
})

Edit
Same thing with hostname.

Is this to replace start(1234) ?

My intention is not getting rid of the start(port) method, although it might be treated as a replacement (just like we did with default routing). I'm mostly referring to the use-case when user don't want to specify port in the start method or uses the default one.
An example setup that could benefit of that is a scenario when an application is configured at some point via e.g. configuration file. The configuration file cannot be loaded inside the cfg scope, because we need to pass this port outside the instance configuration.
Same thing with e.g. plugin API - let's say we'd like to write a plugin that randomly starts an application on a port from a desired range or something like that - it's not possible atm.


ENS Avatar component

Description
A simple web3 avatar component
Why is this needed?
need a web3 avatar component
Proposed Solution or API
4 working files:

ENSAvatar.tsx
ENSAvatar.test.tsx
ENSAvatar.stories.tsx
index.ts

wagmi hook to use
build using the  Avatar element
component props :
ENSAvatar = { address: string, size?: string, shape?: string }
use jazzicon as a fallback
Additional Information
No response

Hey @e-roy I'll be happy to take care of it as my first contribution if you want


Enhance the project by adding simple time series plotting capabilities to visualize temporal data

Description
Add simple time series plotting capabilities to help users visualize temporal data.
Problem it Solves
Currently, users lack the ability to visualize temporal data effectively within the platform. This limitation makes it difficult for users to identify trends, patterns, and anomalies in their time series data.
Proposed Solution
Create basic time series plotting functions: Develop functions that can generate simple time series plots.
Add option to include time series plots in reports: Integrate these plots into the existing report generation framework.
Implement simple temporal feature importance: Provide insights on the importance of different features over time.
Alternatives Considered
Third-party Tools: Using external tools like Excel or Python libraries for time series analysis, but this requires users to export data and work outside the platform.
Static Reports: Providing static images of time series plots, which are less interactive and require constant updates.
Additional Context
By incorporating time series plotting capabilities, we can enhance the platform's utility for users dealing with temporal data. This feature will make it easier for users to visualize and analyze time-dependent trends within the platform, improving their overall experience and efficiency.

@sv410 go ahead! make sure have the changes in core explinanbleai python package codes, and not external codes or notebook


Update rancher-external-dns system chart to latest upstream

We need to update the 2.5 rancher-external-dns system chart to latest upstream release of External-DNS https://github.com/kubernetes-sigs/external-dns/releases/tag/v0.10.0

Note v0.11.0 is the current latest - https://github.com/kubernetes-sigs/external-dns/releases/tag/v0.11.0

Closing as v0.2.0 is the latest

Reopening as the system-chart is not using the latest upstream version yet - https://github.com/rancher/system-charts/blob/release-v2.6/charts/rancher-external-dns/v0.2.0/values.yaml#L4


Migrate examples in android TestApp to Kotlin

Many of the examples from the TestApp ported 1 to 1 to kotlin.

Thanks @birkskyum. I think Kotlin is the right language for Android. Let's wait with this pull request until we decided on #420.

Do we have some build instructions for this test app? Would like to try it out...

Yes, it's a matter of opening Android Studio is opened in the platform/android. Create a virtual device and run the MapboxGLAndroidSDKTestApp. Gradle will have to do a lot of work when you open the repo, so it can take a little while, but apart from that, it's quite few steps.

Great, I will give it a go and ping you if I am stuck...

What is the easiest way to review this pull request? The diff seems to be too large for git to recognize that the files were renamed and the code was changed...

The only way as I see it is to run the app before the change, see how each example works, and then apply the change and validate that the example works the same. Like a practical test.

Sure I can do that. I might also just rename the files back to .java and then do a diff...

This is like a first round of porting. Many small changes can be done for each file due to the new capabilities of Kotlin, but that'll be a second round.

I'm going through all the examples to add to the documentation. Just found out that missing parenthesis caused this funny regression.
https://github.com/user-attachments/assets/c63a2a4b-8f40-4acf-b8f1-eeb58c4b8e2d


hydration error on UButton involving disabled property

Environment

Operating System: Darwin
Node Version:     v20.11.1
Nuxt Version:     3.11.2
CLI Version:      3.11.1
Nitro Version:    2.9.6
Package Manager<EMAIL_ADDRESS>Builder:          -
User Config:      devtools, modules
Runtime Modules<EMAIL_ADDRESS>Build Modules:    -

Version
2.15.2
Reproduction
https://github.com/stlbucket/ui-nuxt-hydration-bug
Description
i have seen a number of closed issues related to this but involving UCheckbox, and they are all closed.
stackblitz doesn't show the error because it only happens when you refresh a page:
https://stackblitz.com/~/github.com/stlbucket/ui-nuxt-hydration-bug
instructions to repro locally are in the readme as well:
instructions
run this app, and hit refresh until you see this warning:
warn [Vue warn]: Hydration attribute mismatch on
<button class="focus:outline-none disab…nline-flex items-center" type="button" disabled="">
  - rendered on server: disabled="true"
  - expected on client: (not rendered)
  Note: this mismatch is check-only. The DOM will not be rectified in production due to performance overhead.
  You should fix the source of the mismatch.
  at <Link type="button" disabled=false class="focus:outline-none disabled:cursor-not-allowed disabled:opacity-75 flex-shrink-0 font-medium rounded-md text-sm gap-x-1.5 px-2.5 py-1.5 shadow-sm text-white dark:text-gray-900 bg-primary-500 hover:bg-primary-600 disabled:bg-primary-500 dark:bg-primary-400 dark:hover:bg-primary-500 dark:disabled:bg-primary-400 focus-visible:outline focus-visible:outline-2 focus-visible:outline-offset-2 focus-visible:outline-primary-500 dark:focus-visible:outline-primary-400 inline-flex items-center"  ... >
  at <Button disabled=false onClick=fn<onClicked> >
  at <App key=3 >
  at <NuxtRoot>

this happens when disabled can change
if you take away the disabled property, there is no error
// app.vue

<template>
  <div>
    <UButton
      :disabled="disableButton"
      @click="onClicked"
    >This is a button</UButton>
  </div>
</template>

<script setup lang="ts">
  const disableButton = computed(() => Math.random() > .5)

  const onClicked = () => {
    alert('clicked')
  }
</script>

Additional context
No response
Logs
No response

Given the reproduction you just sent, I can say this is normal behavior.
Basically, you are saying that Math.random() will have a different value on server and on client, and that the disabled state could change between the client and the server. So there is a 50% chance that you can get an hydration mismatch.
I don't know what is the current progress of your app, but if you are using Math.random() just for debug purpose, while waiting for the API to be ready, I can suggest the following changes:
const randomNum = useState('random-num', () => Math.random())
const disableButton = computed(() => randomNum.value > .5)

The trick here is to use useState. It holds in memory a value that could change between the server and the client, which prevents the hydration mismatch.
Now, if the value can really change in a real world app, then my best advice would be to make that part of your template ClientOnly so that you don't compute the same thing twice, as you'll get an hydration mismatch anyway.

I am not using Math.random anywhere, but have started seeing this behavior in many places.  Suddenly.  After running this app for months.
Yes, it is something in my app, no doubtjj.
So, I will, when I have time, rebuild the whole thing from ground up - it is very modular, so hopefully I can find it.

Closing this issue as the reproduction's bug is not a bug but expected behavior. Feel free to reopen when you can reproduce with a valid example !


💡 [Feature]: refactor the action naming

🎯 Aim of the feature
Align the actions naming to use project instead of current project
📷 Images (if possible) with expected result

Can I work on this one? ✋

sure.
Thank you for taking it on 👍
I am still working on a better contribution guide and more clarity in current issues and more good first issues, so if there is anything that you find blocking you from work just reach me out directly for help

@nicodecleyre just to let you know I updated the contributing guidance. Maybe it might be helpful in your development

Merged ✅
your change was released as a new pre-release v3.0.2 (pre-release are not installed automatically and you need to opt-in first time to get it)

at the end of this week I will be doing a new minor release of this extension and this change will be part of it.
Thank you for helping out in making the best SPFx tooling in the world 👏👏👏 You Rock 🤩


only collect metrics from the first prometheus

What problem does this PR solve?
What is changed and how it works?
Check List
Tests

Unit test
Integration test
Manual test (add detailed scripts or steps below)
No code

Code changes

Has exported function/method change
Has exported variable/fields change
Has interface methods change
Has persistent data change

Side effects

Possible performance regression
Increased code complexity
Breaking backward compatibility

Related changes

Need to cherry-pick to the release branch
Need to update the documentation

Codecov Report
All modified lines are covered by tests :white_check_mark:

Comparison is base (28c2e01) 30.19% compared to head (eb9f161) 30.19%.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #462   +/-   ##
=======================================
  Coverage   30.19%   30.19%
=======================================
  Files          52       52
  Lines        4415     4415
=======================================
  Hits         1333     1333
  Misses       2911     2911
  Partials      171      171

Flag
Coverage Δ

unittests
30.19% <ø> (ø)

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

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Creating bucket is not triggering cloudwatch event.

Hello henry,
I have been seen your code and when creating bucket with acl is not triggering the cloudwatch event. Creating bucket with with acl is triggering getbucketacl event not triggering createbucket event.
Nice to meet with you on github.
Thanks & Regards,

Are you creating the S3 bucket via API or via AWS web management console?
Which steps specifically are you taking to create the bucket?
Which events do you see appearing in your CloudTrail log?

Hello,
I'm using aws console to creating bucket. But creating bucket working fine now. i tried to get event for putbucketpolicy but it is not triggering event. that is the problem.
Thanks,

The CloudWatch Events that Henry currently processes are PutBucketAcl and CreateBucket, which cover API and Console based bucket creation and bucket permission editing.
As this is a bucket-centric approach, it doesn't currently support inspecting IAM policy updates to check to determine if a user (or more importantly, the anonymous user) has been granted the ability to access an S3 bucket by setting policies and applying them to a user/role.
I'll come back to this at some point when I've got more free time and investigate extending the tool in this way.

Sure.
Thanks a lot for support Henry.


Temporary live enpoint URLs - hotfix needed?

Authorize.Net are in the process of moving their servers. This will involve some downtime on the current live endpoint URLs leading up to October 2015. This one, for example:
https://github.com/thephpleague/omnipay-authorizenet/blob/master/src/AIMGateway.php#L24
'liveEndpoint'      => 'https://secure.authorize.net/gateway/transact.dll',

To avoid the downtime, the URL can be changed to this now (from start of July), as this endpoint has already gone through the server migration process:
https://secure2.authorize.net/gateway/transact.dll

Should we change this? Note that the endpoint URL can be set by hand when creating the gateway, so the merchant site can override it to use the secure2 version. However, that would need a merchant site change, while doing so here would be a packageist "hotfix" change.
Obviously needs to be tested on a live site first - I'm happy to do that.

The complete message from Authorize.Net (annoyingly, no link to any online article I can find):
Important Authorize.Net Networking Change
Dear Authorize.Net Developer:
At Authorize.Net, the reliability of our platform is our first priority. Your business and your merchants' businesses depend on our services, and we understand how important that is. To that end, we are implementing a new networking solution that will provide merchants with a superior level of processing reliability.
Please read this email thoroughly.
Akamai Network Technology
Over the coming months, Authorize.Net will be upgrading access to the Internet connections that serve our data centers. Instead of allowing direct connections, all Internet traffic will be routed through Akamai, a third-party cloud network service that routes and delivers Internet traffic.
Using Akamai network technology will help decrease latency and improve the reliability of our payment gateway. It will also help safeguard against interruptions caused by issues beyond Authorize.Net's direct control, such as Internet congestion, fiber cable cuts and other similar issues. Additionally, come October, merchants connected to Akamai should no longer be affected by planned downtimes.
Implementation Plan And Timeline
We are implementing Akamai's technology into our network in two phases.
Phase One (Now through June 2016) — We have created three new URLs for transaction processing that are hosted by Akamai. You can update your website or payment solution to point to the new transaction URLs today, which will provide the immediate benefits detailed above. Come October, merchants connected to Akamai should no longer be affected by planned downtimes.
Phase Two (June 2016) — We will automatically direct all of our existing transaction processing URLs to connect through Akamai in June of 2016. After this change, all transaction URLs will connect to Authorize.Net through Akamai.

IMPORTANT: To take advantage of the uptime benefits Akamai offers, we recommend you proactively update your website or payment solution during Phase One rather than waiting to be automatically updated during Phase Two.
Updating Your Solution
The new Akamai transaction URLs that are available now are:
https://api2.authorize.net/xml/v1/request.api
https://api2.authorize.net/soap/v1/Service.asmx
https://secure2.authorize.net/gateway/transact.dll

We will begin sending our initial emails about Akamai to merchants starting tomorrow.
Firewall And IP Address Considerations
If you do not use a firewall or do not connect directly via an IP address, then you can disregard this section.
Using Akamai technology, Authorize.Net's IP addresses will become dynamic rather than static. If your website or payment solution uses a firewall that has whitelisted Authorize.Net's IP address or connects to Authorize.Net directly via an IP address, you will need to make additional changes.
Firewall Considerations — If your solution uses a firewall to filter outbound connections, make sure that the firewall is set to permit outbound traffic to flow to the Akamai cloud by configuring your outbound firewall to "ANY."
If you do not update your firewall ahead of June 2016, you will be unable to process transactions after Phase Two is implemented.
IP Address Considerations — If your solution connects to Authorize.Net directly via an IP address, you will need to update it to connect by domain name. Continuing to connect directly via an IP address is strongly discouraged as merchants will not receive the benefits of routing through Akamai, and could suffer a loss of service if transactions are re-routed among our various data centers.

Akamai is available for testing now in our sandbox environment. We strongly encourage you to test your solution prior to making any production change to ensure that you will not experience any disruptions to transaction processing.
Questions?
If you have any questions about the Akamai upgrade, please refer to our Akamai FAQs or contact Developer Support.
Thank you for your attention to this important email. We appreciate your ongoing partnership and are excited to bring you the benefits and reliability that this enhanced service provides.
Sincerely,
Authorize.Net

Yes, we should get this changed stat. I can work on it tonight.

For those looking for a workaround, when you create your gateway, new URLs can be passed in:
$gateway = OmniPay::create('AuthorizeNet_AIM/SIM/DPM');
$gateway->setLiveEndpoint('https://secure2.authorize.net/gateway/transact.dll');

So these new url's can always be used, and it's just this line that needs changing? https://github.com/thephpleague/omnipay-authorizenet/blob/142a95f550a5320db09e66019ecf5c8b8c3885b9/src/AIMGateway.php#L24
If you can make a PR and test it, I see no problem in accepting it. But I don't have an Auth.net account, so I can't test it..


Use correct listLen for PrtclState

Previously, it was not valid CBOR.

:thinking: why didn't https://github.com/input-output-hk/cardano-ledger-specs/blob/master/shelley/chain-and-ledger/executable-spec/test/Test/Shelley/Spec/Ledger/Serialisation/Tripping/CBOR.hs#L123 catch this...

A test wouldn't catch it, as it can be en/decoded just fine. It's just that it's not valid format CBOR and external tools like cbor.me would choke on it.

Those tests wouldn't catch it, as it can be en/decoded just fine. It's just that it's not valid CBOR and external tools like cbor.me would choke on it.

oh yes, of course, we were explicitly encoding the length 5 token, thank you. our CDDL tests catch this kind of thing, but we do not have CDDL tests for PrtclState.


added support for ADM

https://user-images.githubusercontent.com/80992641/194728375-a9c2b45b-9a33-4124-9cdf-c38fe6201ff9.mp4
I changed the default download directory from: "/Aniyomi/downloads" to "/Downloads/Aniyomi/" because ADM can only download to the "downloads" and "documents" folders.
without this change ADM would give an error saying that you do not have access to the directory

Is this will be a new update? Because it's not working for me

This change will be included in the next update.

Can this also be done for 1DM?

Can this also be done for 1DM?

Umm, 1DM and 1DM+ support is already there.

Can this also be done for 1DM?

Umm, 1DM and 1DM+ support is already there.

It probably refers to the fact that in adm you can directly play the video from aniyomi as if it were downloaded from the same application

Can this also be done for 1DM?

nope, 1dm without allowing to specify the download path from the intent

Why wasn't this merged?


replace mvapich2 with spectrum-mpi in blueos configs

https://github.com/LLNL/axom/blob/3128c6cbd68cb184202488dd85feafc467faa6cf/scripts/uberenv/spack_configs/blueos_3_ppc64le_ib_p9/packages.yaml#L34
spectrum-mpi is now a package in Spack, so we can reference it instead of faking Spack into thinking it's mvapich2
s/mvapich2/spectrum-mpi/
s/@2.2/@release/

This applies to the configs in axom/scripts/uberenv/spack_configs/blueos_3_ppc64le_ib/packages.yaml as well


Set-up-Kanbanboard-with-react-space-Project

Click to see our Project
There are only two members in this project.
1- Muhammad Zunair Khan
2- Rosemutai.
All tasks have been equally assigned and dragon cards have been deleted.

Hi @zunairkhan811 & @rosemutai 🙋♂
Good job working on your Kanban board.👏
You still need to implement few on the project but you are almost there!!! :watch:
TO HIGHLIGHT:

Assigning the group assignments to both students.

STATUS: Required Changes :recycle:

STATUS: Required Changes :recycle:

I kindly suggest ensuring that each student should be assigned to the required categories of Setup, Fetch data, Lists render, Actions, Conditional components, Filters. This is to ensure that there is equal time and assignment distribution.

Optional suggestions
N/A
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread or slack channel if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.

As described in the Code reviews limits policy  you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.
HAPPY CODING AND GOOD LUCK :crossed_fingers: :four_leaf_clover:

STATUS: APPROVED :green_circle: :clap:
Hello Team🙋♂,
Your project is complete!  There is nothing else to say other than... Great job 👏

Congratulations! 🎉
TO HIGHLIGHT:

Assigning the group assignments to both students. ✔
The cards with required categories are evenly assigned ✔

Now you can merge this branch and move on :1st_place_medal:
Optional suggestions
Every comment with the [OPTIONAL] prefix won't stop the approval of this PR. However, I strongly recommend you to take them into account as they can make your code better. Some of them were simply missed by the previous reviewer and addressing them will really improve your application.
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.

As described in the Code reviews limits policy  you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.
HAPPY CODING AND GOOD LUCK WITH THE COMING ONES :crossed_fingers: :four_leaf_clover:


Update SOFTWARE-LIST.md for Xerox

Broken link fixed to point to latest Xerox Security Bulletin

🗣 Description

💭 Motivation and context

🧪 Testing

✅ Pre-approval checklist

[ ] This PR has an informative and human-readable title.
[ ] Changes are limited to a single goal - eschew scope creep!
[ ] All future TODOs are captured in issues, which are referenced
in code comments.
[ ] All relevant type-of-change labels have been added.
[ ] I have read the CONTRIBUTING document.
[ ] These code changes follow cisagov code standards.
[ ] All relevant repo and/or project documentation has been updated
to reflect the changes in this PR.
[ ] Tests have been added and/or modified to cover the changes in this PR.
[ ] All new and existing tests pass.

✅ Pre-merge checklist

[ ] Revert dependencies to default branches.
[ ] Finalize version.

✅ Post-merge checklist

[ ] Add a tag or create a release.

@stevea1 we have switched to a new data format for ingesting our data.
You would need to update this file: https://github.com/cisagov/log4j-affected-db/blob/develop/data/cisagov_X.yml
You can view the contribution guidelines here (https://github.com/cisagov/log4j-affected-db/blob/develop/CONTRIBUTING.md) for assistiance.

@justmurphy, got it, thanks. I will resubmit using the new format.


0.10.0 breaks derived AuthUser classes

QuartAuth taking a user_class argument that defaults to AuthUser means that the prior way of overriding the class of setting the user_class attribute of the class doesn't work.
I had code which did:
class ApiUser(AuthUser):
...
class ApiAuth(QuartAuth):
user_class = ApiUser
And this code broke in 0.10.0 because now self.user_class is always initialized to AuthUser.
I wonder if for backwards compatibility these should not default to None and then get initialized from the class's attribute.  Alternatively, I suppose you could deprecate the old way.

Yep, that was a mistake, fixed in 0.10.1. Thanks.


Jsdoc type intellisense not working when using custom file extension

VSCode Version: 1.5.3
OS Version: Windows 10

In a .js file, intellisense provides correct jsdoc types

In a custom extension file associated with javascript, types from jsdoc are not provided

Steps to Reproduce:

Create a .sj file
In settings.json, set "files.associations": { "*.sj": "javascript" }
Create a function with jsdoc parameters with types
Call function and check if intellisense provides types

@csvenke thank you for opening this issue. This is a great find!
@dbaeumer I have recreated this with latest TS and VS Code Insiders.
Here is the code to copy and paste.
/**
 * @param {string} a
 * @param {string} b
 * @param {string} c
 */
function asd(a, b, c) {}

Do you think this is on our end or on TS's end?

@waderyan a TS issue since the hover is computed by the tsserver. Is this with 1.6 insider. I thought they fixed this in 2.0.x since we reported this on 1.8.x already.

@dbaeumer this is with 1.6. I'll open an issue on their repo.

This issue was moved to Microsoft/TypeScript#11367


Tweak some spec IDs for HTML

These IDs also appear in the developer's edition of HTML (https://html.spec.whatwg.org/dev), and so including these allows caniuse integration with the dev edition (whatwg/html#2794).

Thanks!


feat: Add binaries to releases to avoid rebuilding on install

I want to get cargo binstall working so the install time can be cut down. As a first step, i have added one target: x86_64-linux-gnu. I have left he boiler plate code for the other targets from https://github.com/nicolas-van/rust-cross-compile-example commented out so they can be added if someone can get them working.

Shouldn't be too hard with the CI already set up, just need to add the other targets and the releasing of binaries

Shouldn't be too hard with the CI already set up, just need to add the other targets and the releasing of binaries

Wait this is a PR lol this GH client. Sorry my bad

Btw just wanna say u wanna follow conventional commits in this repo. So maybe like change ur commit name.

Hey, thanks for the PR but I'm going to have to close this.
Binaries are something I do want to add eventually, but I would like to have some discussion first around some of the specifics to make sure it's done right, such as the targets and formats and methods of distribution.
This PR also adds a full new release pipeline, duplicating a lot of the work of the existing one including creating duplicate releases. It also includes Windows/MacOS specifics, which aren't applicable.

FWIW: It seemed to have issues with gdk-sys when i tried for arch.

I'd check the PKGBUILD if you're building on Arch, there shouldn't be any special requirements.
Please feel free to open an issue.


About Page Custom Header Design

Miguel to provide PNG of SRF & inset seed
Ignore "About SRF" section, already done


[DNM] Improve free_to

Use Binary Search instead of Linear Search in free_to and add relate benchmarks.
Bench Result
Inflights::free_to/64@0 time:   [129.33 ns 134.89 ns 140.37 ns]
                        change: [+0.3479% +8.6803% +18.047%] (p = 0.05 < 0.05)
                        Change within noise threshold.
Found 2 outliers among 100 measurements (2.00%)
  2 (2.00%) high mild
Inflights::free_to/64@6 time:   [126.03 ns 133.64 ns 142.15 ns]
                        change: [-6.5282% +2.4475% +12.238%] (p = 0.61 > 0.05)
                        No change in performance detected.
Found 2 outliers among 100 measurements (2.00%)
  2 (2.00%) high mild
Inflights::free_to/64@16
                        time:   [127.97 ns 134.99 ns 142.34 ns]
                        change: [-11.196% -2.9282% +5.5251%] (p = 0.51 > 0.05)
                        No change in performance detected.
Found 1 outliers among 100 measurements (1.00%)
  1 (1.00%) high mild
Inflights::free_to/64@32
                        time:   [130.04 ns 137.27 ns 144.72 ns]
                        change: [-21.535% -14.753% -7.7207%] (p = 0.00 < 0.05)
                        Performance has improved.
Found 1 outliers among 100 measurements (1.00%)
  1 (1.00%) high mild
Inflights::free_to/64@48
                        time:   [128.06 ns 134.14 ns 140.62 ns]
                        change: [-36.361% -30.476% -24.428%] (p = 0.00 < 0.05)
                        Performance has improved.
Inflights::free_to/64@63
                        time:   [125.93 ns 131.81 ns 137.93 ns]
                        change: [-34.380% -28.776% -22.785%] (p = 0.00 < 0.05)
                        Performance has improved.
Found 2 outliers among 100 measurements (2.00%)
  1 (1.00%) high mild
  1 (1.00%) high severe
Inflights::free_to/256@0
                        time:   [92.512 ns 96.127 ns 99.768 ns]
                        change: [+8.1334% +22.507% +40.008%] (p = 0.00 < 0.05)
                        Performance has regressed.
Found 7 outliers among 100 measurements (7.00%)
  1 (1.00%) high mild
  6 (6.00%) high severe
Inflights::free_to/256@8
                        time:   [97.911 ns 101.71 ns 105.21 ns]
                        change: [+7.3463% +23.457% +45.876%] (p = 0.02 < 0.05)
                        Performance has regressed.
Found 4 outliers among 100 measurements (4.00%)
  1 (1.00%) high mild
  3 (3.00%) high severe
Inflights::free_to/256@64
                        time:   [110.95 ns 117.83 ns 124.59 ns]
                        change: [-36.738% -32.980% -28.922%] (p = 0.00 < 0.05)
                        Performance has improved.
Found 3 outliers among 100 measurements (3.00%)
  3 (3.00%) high mild
Inflights::free_to/256@128
                        time:   [109.01 ns 114.19 ns 119.61 ns]
                        change: [-60.409% -58.175% -55.703%] (p = 0.00 < 0.05)
                        Performance has improved.
Found 4 outliers among 100 measurements (4.00%)
  4 (4.00%) high mild
Inflights::free_to/256@192
                        time:   [103.50 ns 107.11 ns 110.52 ns]
                        change: [-68.973% -67.188% -64.837%] (p = 0.00 < 0.05)
                        Performance has improved.
Found 3 outliers among 100 measurements (3.00%)
  2 (2.00%) high mild
  1 (1.00%) high severe
Inflights::free_to/256@255
                        time:   [122.89 ns 127.26 ns 131.78 ns]
                        change: [-71.021% -69.327% -67.263%] (p = 0.00 < 0.05)
                        Performance has improved.
Found 1 outliers among 100 measurements (1.00%)
  1 (1.00%) high severe
Inflights::free_to/1024@0
                        time:   [111.85 ns 115.40 ns 118.80 ns]
                        change: [+28.825% +49.292% +73.372%] (p = 0.00 < 0.05)
                        Performance has regressed.
Found 6 outliers among 100 measurements (6.00%)
  2 (2.00%) high mild
  4 (4.00%) high severe
Inflights::free_to/1024@10
                        time:   [117.14 ns 122.60 ns 128.14 ns]
                        change: [+25.974% +39.473% +56.245%] (p = 0.00 < 0.05)
                        Performance has regressed.
Found 11 outliers among 100 measurements (11.00%)
  5 (5.00%) high mild
  6 (6.00%) high severe
Inflights::free_to/1024@256
                        time:   [115.73 ns 122.16 ns 129.36 ns]
                        change: [-74.795% -73.191% -71.418%] (p = 0.00 < 0.05)
                        Performance has improved.
Found 5 outliers among 100 measurements (5.00%)
  3 (3.00%) high mild
  2 (2.00%) high severe
Inflights::free_to/1024@512
                        time:   [124.00 ns 128.97 ns 134.10 ns]
                        change: [-82.226% -80.766% -78.773%] (p = 0.00 < 0.05)
                        Performance has improved.
Found 12 outliers among 100 measurements (12.00%)
  5 (5.00%) high mild
  7 (7.00%) high severe
Inflights::free_to/1024@768
                        time:   [124.97 ns 130.39 ns 136.01 ns]
                        change: [-87.938% -86.974% -85.828%] (p = 0.00 < 0.05)
                        Performance has improved.
Found 9 outliers among 100 measurements (9.00%)
  4 (4.00%) high mild
  5 (5.00%) high severe
Inflights::free_to/1024@1023
                        time:   [131.33 ns 157.64 ns 206.95 ns]
                        change: [-90.876% -89.816% -88.273%] (p = 0.00 < 0.05)
                        Performance has improved.
Found 4 outliers among 100 measurements (4.00%)
  3 (3.00%) high mild
  1 (1.00%) high severe
Inflights::free_to/4096@0
                        time:   [1.0208 us 1.2441 us 1.4336 us]
                        change: [-16.970% +25.455% +91.599%] (p = 0.28 > 0.05)
                        No change in performance detected.
Inflights::free_to/4096@12
                        time:   [1.0144 us 1.2620 us 1.4707 us]
                        change: [-17.191% +24.278% +87.836%] (p = 0.29 > 0.05)
                        No change in performance detected.
Inflights::free_to/4096@1024
                        time:   [1.0016 us 1.2439 us 1.4521 us]
                        change: [-68.291% -57.908% -44.113%] (p = 0.00 < 0.05)
                        Performance has improved.
Inflights::free_to/4096@2048
                        time:   [993.37 ns 1.2161 us 1.4039 us]
                        change: [-81.659% -75.628% -67.818%] (p = 0.00 < 0.05)
                        Performance has improved.
Inflights::free_to/4096@3072
                        time:   [1.0437 us 1.2743 us 1.4673 us]
                        change: [-86.708% -82.857% -78.546%] (p = 0.00 < 0.05)
                        Performance has improved.
Inflights::free_to/4096@4095
                        time:   [1.0184 us 1.2433 us 1.4322 us]
                        change: [-91.132% -87.923% -84.634%] (p = 0.00 < 0.05)
                        Performance has improved.
Inflights::free_to/16384@0
                        time:   [4.3876 us 4.7562 us 5.0588 us]
                        change: [-16.140% +3.0053% +27.902%] (p = 0.78 > 0.05)
                        No change in performance detected.
Inflights::free_to/16384@14
                        time:   [4.0367 us 4.4050 us 4.7188 us]
                        change: [-27.012% -10.294% +11.348%] (p = 0.33 > 0.05)
                        No change in performance detected.
Inflights::free_to/16384@4096
                        time:   [4.2099 us 4.5814 us 4.8818 us]
                        change: [-68.759% -62.836% -57.190%] (p = 0.00 < 0.05)
                        Performance has improved.
Inflights::free_to/16384@8192
                        time:   [4.3191 us 4.6934 us 5.0021 us]
                        change: [-79.622% -75.919% -72.027%] (p = 0.00 < 0.05)
                        Performance has improved.
Inflights::free_to/16384@12288
                        time:   [4.3328 us 4.7174 us 5.0344 us]
                        change: [-85.755% -83.421% -80.859%] (p = 0.00 < 0.05)
                        Performance has improved.
Inflights::free_to/16384@16383
                        time:   [4.4048 us 4.8108 us 5.1492 us]
                        change: [-88.752% -86.733% -84.698%] (p = 0.00 < 0.05)
                        Performance has improved.

Some thinkings
Because of the Binary Search algorithm, free_to the first log(n) items in Inflights can be slower than the origin Linear Search implementation.
And in the scenario of streaming messages, where the ProgressState of the follower is Replicate, the leader always frees the previous few items in Inflights each time receiving a MsgAppendResp so that Binary Search here may not be as the same obvious improvement as in bench results above.
But we can adjust the implementation to dynamically using Binary or Linear Search according to the receiving idx. Furthermore, maybe we can merge multiple continuously produced MsgAppendResp into one message benefitting from the fewer leader steps and network costs.
@Hoverbear @BusyJay @hicqu @siddontang

@BusyJay I wrote some thinkings about this in the last PR part Some thinkings. And we may find a better way to solve this :).

I tried to merge MsgAppendResponse and don't get benefits from it. You can see the bench result in :( https://github.com/tikv/tikv/pull/6182.
@BusyJay @hicqu

Maybe the cost of the loop is not significant at all. 😆

@BusyJay I think it also reduces the calling cost of step in the leader. But maybe it's also very subtle.

@Fullstop000 I notice there is binary_search_by on Vec<T>: https://doc.rust-lang.org/std/vec/struct.Vec.html#method.binary_search_by
Is it not usable here?

Is it not usable here?

It's only for Vec and hard to be applied for a struct. Is there some trait like BinarySearch @Hoverbear

@Fullstop000 Hmmm, I saw you were working on self.buffer which is  Vec here. Investigating it some more I'm not sure my advice is sound :)

Close this and I'll make another PR only containing the benchmarks.


Carry bug

Here is a commit that demonstrates the bug by adding a failing test.

The bug's locus is the two wrapping_adds on line 530 of cpu.rs

So that means if we remove the second wrapping_add, it should work right? Can you update your PR with the fix? I wonder if all the other unit tests will pass.

We need both wrapping_adds, but we also need to set the carry flag if b=0xFF and carry_before = true.
I'll try and code up a solution in the coming days

Thanks a lot for looking into this. 👍

Merged. 🥳
Thanks for your contribution. If you like to become a co-maintainer of this repo, please send me a mail. I'd love to have you on board.

If you like to become a co-maintainer of this repo, please send me a mail. I'd love to have you on board.
Message sent 👍


Enable IP version specification for realtime

https://github.com/supabase/realtime/pull/253 got merged into the upstream realtime container and works as expected if local development is happening purely in a docker-compose file but as the CLI does not support the REALTIME_IP_VERSION setting, having ipv6 disabled on a local system results in supabase start crashing the realtime component
A solution would be to add:
[realtime]
ip_version="ipv4"

Support for the config.toml file

This should be possible now with npx supabase@beta start
[realtime]
enabled = true
ip_version = "IPv4"

any news on this?

@Silur are you still running into the same error? CLI now defaults to IPv4 because docker support for IPv6 is not very good.

yes the issue still persists on latest cli versions when ipv6 is disabled


Add snode function to set node version

Add snode function to set node version from a variety of sources, in order of preference:

".node-version" file
".nvmrc" file
the ".engines.node" key in "package.json"

Resolves #65

I've been using this multiple times as a day over the past couple of months and I'm happy with this 👍


[android] no map rendered after upgrading 0.23 -> 0.24.2

Bug
After upgrading from 0.23 to 0.24.2 there is no map rendered on android. I've cleared the yarn cache, deleted node_modules and reinstalled, did ./gradlew clean, re-linked dependencies. Everything else has been left touched.
Has anyone encountered this issue? Do I have to change any native code for the upgrade?
Environment info
React native info output:
  React Native Environment Info:
    System:
      OS: Linux 4.15 Ubuntu 18.04.2 LTS (Bionic Beaver)
      CPU: (4) x64 Intel(R) Core(TM) i5-2500K CPU @ 3.30GHz
      Memory: 148.96 MB / 7.77 GB
      Shell: 4.4.19 - /bin/bash
    Binaries:
      Node: 8.15.1 - /usr/bin/node
      Yarn: 1.13.0 - /usr/bin/yarn
      npm: 6.4.1 - /usr/bin/npm
    SDKs:
      Android SDK:
        API Levels: 23, 25, 26, 27, 28
        Build Tools: 23.0.1, 25.0.2, 25.0.3, 26.0.1, 27.0.3, 28.0.3
        System Images: android-27 | Intel x86 Atom_64, android-28 | Google APIs Intel x86 Atom, android-28 | Google APIs Intel x86 Atom_64
    IDEs:
      Android Studio: 3.3 AI-182.5<IP_ADDRESS>99772
    npmPackages:
      react: ^16.8 => 16.8.6
      react-native: ^0.58.4 => 0.58.6
    npmGlobalPackages:
      react-native-cli: 2.0.1

Library version: 0.24.2
Steps To Reproduce

Install version 0.23 and create a working screen with map
Upgrade package.json to version 0.24.2 install deps and rebuild the project
...

Seems like it happens on simulator, when running Google APIs instead of Google Play system image :thinking: That was not the case before the upgrade. Can anyone confirm that?


sql/parser: add timestamp format used by JDBC setTimestamp

PR: https://github.com/cockroachdb/cockroach/pull/14712

We have closed this issue because it is more than 3 years
old. If this issue is still relevant, please add a comment and re-open
the issue. Thank you for your contribution to CockroachDB docs!


NPU usage is always keeps at 0% when running hello_world test

Hello_world.py file will run CPU and NPU inference, but I found NPU usage always keeps at 0% during running, could you please help to explain why NPU usage is always keeps at 0%.

This issue is duplicate of issue reported in Ryzen AI SW repo, hence closing


Projecting a point/box from Vehicle to Image

I have seen a lot of filed issues with projecting points/ 3d box labels from vehicle frame to image. None of them had a clear solution for the problems with precise localization from 3d to 2d.
Can you please publish some official code to make these projections easier?

see https://github.com/waymo-research/waymo-open-dataset/issues/146#issuecomment-620443276
I think that example is good enough.


Error calling execute class

I'm getting this calling ->execute()
"message": "Trying to access array offset on value of type null",
    "exception": "ErrorException",
    "file": "vendor/wrklst/docxmustache/src/WrkLst/DocxMustache/DocxMustache.php",
    "line": 224,

I'm using php 6.x and dev-master
The latest release 0.0.33 have lots of messed up paths.
Also would be nice to have the creating taking place at 'temp/' subfolder.
Now, the storagePathPrefix set the pathing for the whole instance.
The project looks promising, great work!

Just found out the problem happens when the function ImageReplacer calls $this->AddContentType('jpeg');

Hi @rmundel, not sure what the jpeg conversion has to do with this error. The error you are receiving seems to have something to do with https://github.com/wrklst/docxmustache/blob/95420aab35efdcb423573da3f8b14cffbcf3ad6c/src/WrkLst/DocxMustache/DocxMustache.php#L224 not having the offset that the function is calling for some reason. I wont be able to debug your circumstances, so you'll have to check for yourself what is actually happening in your environment. Feel free to reopen this debugging and error analysis, in case there is an issue with the class and not with your setup. Thanks for participating.


Update files-on-demand-mac.md

Enabling FOD at Registry

Based on https://onedrive.visualstudio.com/Sync/_workitems/edit/702606 and reported bug from Premier support case.


DiscordCE uses email/password.

Is it possible to instead use your discord token instead? You wouldn't need an auth code for that.

try to delete token.json in .minecraft folder, and restart the game. It should automatically enter the email for you, and ask your password. It does ask you your token, but it's not required.


How to use PDFME with customization and need to maintain it in separate repo and use it?

Describe the bug
I tried the separate repo approach, but It says "Cannot find module '@pdfme/common' or its corresponding type declarations." & it has in node_modules folders.

Need support & suggest to use this approach for customization.
To Reproduce
Hosted the PDFME in separate repo and use it.

Expected behavior
Able to run with customized package repo.
Your Environment
- pdfme package(@pdfme/generator or @pdfme/ui): Both
- pdfme version: 3.4.1
- Operating system: MacOS
- Node.js version or Browser name & version: 18

Your Error Log
Compiled with problems:
×
ERROR in ./src/App.tsx 7:0-41
Module not found: Error: Can't resolve '@pdfme/common' in '/Users/rammohan/Desktop/NewBeginnings2024/myapp/src'
ERROR
[eslint] EACCES: permission denied, mkdir '/Users/rammohan/Desktop/NewBeginnings2024/myapp/node_modules/.cache'
ERROR in src/App.tsx:3:26
TS2307: Cannot find module '@pdfme/common' or its corresponding type declarations.
    1 | import React, { useEffect, useRef, useState } from 'react';
    2 | import './App.css';
  > 3 | import { Designer } from '@pdfme/common';
      |                          ^^^^^^^^^^^^^^^
    4 | import type { Template } from '@pdfme/common';
    5 |
    6 |
ERROR in src/App.tsx:4:31
TS2307: Cannot find module '@pdfme/common' or its corresponding type declarations.
    2 | import './App.css';
    3 | import { Designer } from '@pdfme/common';
  > 4 | import type { Template } from '@pdfme/common';
      |                               ^^^^^^^^^^^^^^^
    5 |
    6 |
    7 | function App() {

Additional context
No response

I'm not entirely sure, but is this a bug?
Since it duplicates the following, I'll close it. Please await a response on the discussion page.
https://github.com/pdfme/pdfme/discussions/643


Don't Merge! PR real time test for webhook case

Docs Build status updates of commit 868957c:
:clock10: Pending: waiting for processors (19 builds ahead of you)
:warning: Docs Build is busy, currently there are 19 builds ahead of this one, for more information you can view the Build queue graph on the Docs Portal.

Docs Build status updates of commit 868957c:
:clock10: Full build: the average full build time is 7 min(s) 53 sec(s), this is based on the last 30 build(s) for this repository.

Docs Build status updates of commit 868957c:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-ref-conceptual/testing.md
:white_check_mark:Succeeded
View (azure-cli-latest) View (azure-cli-2017-03-09-profile) View (azure-cli-2018-03-01-hybrid)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


Deprecated Metric Usage

Issue:
The v2 release of kube-state-metrics removes several deprecated metrics, one with usage here.
Additional Context:
Referenced at: https://github.com/kubernetes/kube-state-metrics/blob/release-1.9/docs/README.md#metrics-deprecation
Resolution:
jsonnet/kube-prometheus/addons/weave-net/grafana-weave-net-cluster.json should be updated with the new form of kube_node_status_capacity_pods:
kube_node_status_capacity_pods ~ kube_node_status_capacity{resource="pods",unit="integer"}

Thanks. Do you want to have a go and creating the fix?


Bing Global Building Footprint

Hey @rguitar96
What do you think about https://blogs.bing.com/maps/2023-06/Bing-Maps-Global-Building-Footprints-released
Is it it worth to implement?
Cheers,
leander

Hey @l-kotzur!
It is definitely impressive work! For reference, the final data product is found here: https://planetarycomputer.microsoft.com/dataset/ms-buildings and evens comes with a nice, fast web visualization as well as a sample notebook.
I think this is very worth to implement as an alternative method in the rooftop segmentation, where currently the two options are using our own models (segmentation/prediction.py) or using OSM (segmentation/osm.py). If we build a module that generates masks in the same format as the other two, this would integrate seamlessly.
I will be quite busy during the next weeks (we need to talk!), but if I can get my hands on this I could give it a try :)

Hey @l-kotzur!
It is definitely impressive work! For reference, the final data product is found here: https://planetarycomputer.microsoft.com/dataset/ms-buildings and evens comes with a nice, fast web visualization as well as a sample notebook.
I think this is very worth to implement as an alternative method in the rooftop segmentation, where currently the two options are using our own models (segmentation/prediction.py) or using OSM (segmentation/osm.py). If we build a module that generates masks in the same format as the other two, this would integrate seamlessly.
I will be quite busy during the next weeks (we need to talk!), but if I can get my hands on this I could give it a try :)


[Bug?]: page content gets duplicated when changing source code

Duplicates

[X] I have searched the existing issues

Latest version

[X] I have tested the latest version

Current behavior 😯
I created a solid project with pnpm create solid, selected the bare version with no ssr.
launched the dev  mode, when i change app.tsx, rendered content gets duplicated without even updating changes.
Expected behavior 🤔
update the rendered content to match the source code.
Steps to reproduce 🕹
Steps:

pnpm create solid
select bare
no ssr
launch dev mode and open a browser
change inside h1 tag on app.tsx

Context 🔦
No response
Your environment 🌎
System:
  OS: Linux <IP_ADDRESS>.realtime1-3-rt Arch
  CPU: (12) x86_64
Binaries:
  Node: 20.5.1
  pnpm: 8.12.2
npmPackages:
  @solidjs/start: ^0.5.6
  solid-js: ^1.8.14
  vinxi: ^0.2.1

I've locked the Vite plugin to minor version starting in 0.5.7. You can update to that for now until it is properly tested with SolidStart.

Ok, thanks for your time 🙏🏻

Sorry about the confusion.. I also reverted the Vite plugin. The change impacted too many projects.

We are still testing the new refresh issue. Since it was reverted and this was "fixed" we will address it when the new one is released if it shows up.


In-editor Tab Function Completion/Docs

Awesome work getting Livebook out.  My first contribution is a request.  :D
Tab Completion!! would be so awesome for us noobs getting familiar with Elixir and modules.

Duplicate of #47. I'm currently working on that =)

You Rock!  Guess my search for "tab" didn't get very far.  Thanks much, tracking with great anticipation!


Could you publish it in Scala 2.12 ?

jwt-scala_2.11 and jwt-scala_2.10 are already published, but we want to use your library in 2.12.
Could you publish this library in Scala 2.12 ?

forked and published for Scala 2.12 and 2.13

https://github.com/xuwei-k/jwt-scala/tree/v1.4.0
https://repo1.maven.org/maven2/com/github/xuwei-k/jwt-scala_2.12/
https://repo1.maven.org/maven2/com/github/xuwei-k/jwt-scala_2.13/


Upgrade dependency webdriver-manager

To include some new features like Appium / Android support
To fix Windows IEDriver url when downloading binaries

For version 10.1.0

webdriver-manager upgraded to 10.1.0.
It appears that the package.json is labeled as ^10.0.4 and should be fine. We could bump the version on protractor package.json to match.


Add select_label option

This PR adds the select_label option, which causes combine-pr to ignore any PR that does not have a label matching select_label.
This will be useful in a few ways:

PRs can be manually marked for combination by adding a the 'select label' value and matching any branch name.
Dependabot PRs can be combined into groups by target branch or update source. For example, tag dependabot PRs for golang with go-dep and PRs for Docker with docker-dep, then combine go changes together and docker changes together.

Some notes on edge cases:

Both select_label and ignore_label may be set. In this case, the PR is only combined if it has a label matching select_label and does not have any label matching ignore_label.
If select_label == ignore_label, and error will be raise because it will be impossible for any PRs to be matched and this is almost certainly not what the developer intended.

Would it be worth checking labels before CI or approvals, since label checks don't require additional API calls?

Would it be worth checking labels before CI or approvals, since label checks don't require additional API calls?

I think that would be a great idea! Could save some compute and a bit of time when executing this Action

You can either make the change in this PR or we can merge and you can implement on a fresh PR. Up to you! I reviewed the code and it all looks good to me.

Thanks! I'll switch the order of checks in a new PR to keep changes small. 🚀


add nestjs starter

Hey @halil07, I am closing this PR as we've restructured how we manage templates. Do you mind creating an issue using the updated instructions in the README and re-submitting the template? :)


KVM/networking: Add flannel default gateway parsing

It's simply parsing another json field and setting default gateway using code in:
https://github.com/containernetworking/cni/blob/master/plugins/main/bridge/bridge.go#L226-L245
Also checking if another default gateway is set (i.e. starting with --net=...,default) and returning error in this case.
cc @pskrzyns @jellonek @lukasredynk

All tests passed again, but functional test script failed after TestDockerVolumeSemanticsPodManifest in some cases...

It's actually TestResumedFetch and TestExitCodeWithSeveralApps (click on "Full Log" at the top) but they're not related to this.
The issues are tracked on https://github.com/coreos/rkt/issues/2668 and https://github.com/coreos/rkt/issues/2779

@iaguis so can we merge it ?

I don't think I'm qualified to review this. Can you have a look, @steveeJ?

I'd like to block this on #2791.

@steveeJ we have to have it if we want create kube dns for kubernetes
I think we should merge it for the last time, than we focus on https://github.com/kubernetes/kubernetes/issues/26102 and start moving hypervisor specific code to cni like @jellonek wrote in https://github.com/coreos/rkt/issues/2791#issuecomment-225847471

@jjlakis do you have time to add tests so that we can get this in this release?

I've added test for default gateway and missing test for preserving netName (relates to https://github.com/coreos/rkt/issues/2627)

PTAL @steveeJ

@jjlakis In addition to commenting on this PR I fixed up the tests in a separate PR #2859. Feel free to cherry-pick the additional commit and include it here, we can also just merge the other PR.


Add ability to limit bids per author per round

In certain cases rather than blacklisting accounts I think it would be better to limit it to one bid per author per bidding round.

I'm ok with authors bidding on multiple posts per round, especially if it is different authors.  My concern is more focused on their general activity, posting 10+ times a day to maximize rewards on stolen or copypasta content.  That said, I'm not opposed to the feature at all, good to have options, never know where life may take us.

especially if it is different authors
This is looking at the author of the post, not the sender of the bid. So the idea is to limit one author getting votes on a bunch of their posts in the same round. It should also help limit people being able to buy votes on 10+ posts / day somewhat.

The problem is there are 50+ bots and 40,000 users publishing low effort content.
I like the option though, I would even go so far to add an option max_vote_day and max_vote_round.

I would like to add an option for max by day also, but that's going to be a more involved change since the software doesn't currently keep track of anything past the previous round.

This is done in this commit: https://github.com/MattyIce/postpromoter/commit/65ef0ecdb58b41c924411334d411af76ef048999


Update org.apache.commons:commons-compress version to 1.27.1

The currently used dependency is from September 2023 and has a vulnerability categorized as high. Lets please update this dependency to newer versions without that vulnerability.

Thanks for the contribution but I will close for the same reason as described here. For tracking purposes there is #8338.


Support for JDK8 rpm install

Do you have any plans to support JDK 8 installation via rpm? As of now, it looks like it supports only tar but no rpm.
recipes/default.rb
if node['java']['install_flavor'] != 'windows'
if node['java']['jdk_version'].to_i == 8 and node['java']['install_flavor'] != 'oracle'
Chef::Application.fatal!("JDK 8 is currently only provided with the Oracle JDK")
end
end

since there are 3 different oracle flavors you may try a string comparison which will allow all of oracle, oracle_i386, and oracle_rpm. I'm not so sure about overloading the flavor to also have sub-flavors, but this seems a simple enough possible solution. :
if node['java']['jdk_version'].to_i == 8 and !node['java']['install_flavor'].to_s.start_with?("oracle")
  Chef::Application.fatal!("JDK 8 is currently only provided with the Oracle JDK")
end

Installing via RPM is useful when there is a repo to connect to. Since oracle does not provide one, and if you have one you can just use chefs built in resource to install it, this cookbook wont be supporting downloading and installing the rpm for you.

hi eric, thanks for maintaining this cookbook.
the community java cookbook is a direct dependency of many other community cookbooks. having a common cookbook that we can all make use of is a great benefit to the entire community. We have been using this and other community cookbooks with great success.
as a best practice, we maintain a local yum repo for our packages. this includes the oracle jdk8 rpm.
we found some simple enhancements that can be made to this cookbook to enable its continuing support of this local yum repo, and the read me description for the oracle_rpm recipe fits that use perfectly.
oracle, in addition to providing the tgz, has side by side the rpm, which is where we pull it from.
what i don't understand, however, is why you would place a public yum repo hosted by oracle as a requirement for rpm support in this cookbook.
If i am understanding you correctly, your suggested workaround is to not use this cookbook at all and instead use the vanilla package resource if we want to install oracle jdk from rpm.
Is that correct?
Since there are so many cookbooks dependent upon the community java cookbook, this would mean that we, and any others who use oracle jdk8 via rpm, would need to create a fork of this cookbook, or a new one entirely.
i'm not understanding the reason and issues behind discontinuing or leaving broken this basic function.
Can you elaborate? maybe we can come to a more beneficial resolution for the many consumers of this cookbook who also use the oracle rpm from a local yum repo on their systems.


Update paths for Textile lookups

Belatedly.
From observer/lookup_{path} to lookups/lookup_{path} for actions handled by the 'LookupsController`
Deletes the redirect for the old observer/ paths, not necessary any more since calls to these actions come from MO itself. (The redirect works though, that is why i didn't notice the old paths still in the Textile class and test.)

Coverage remained the same at 92.628% when pulling ee2a1abbb19500e429c149a05d76bd1f687928d8 on nimmo-textile-paths into 39d0a40cb4ec5eca881ec4d63a7bc7c06eb9863b on master.


Remove top_n from distance sorting as no longer needed

top_n is used to define the number of sites to select the sites satistfyng the distance threshold from. In most cases a restriction on this is unnecessary and should be removed as it clouds SA analyses.

Was this addressed already Rose?

Closed in #514

Was this addressed already Rose?

Yes sorry, I thought I had linked the issue in the PR but I hadn't.


Added placeholder for course status

Added two strings to the PlaceholderAPI string config.
With the "status" placeholder, any course can be seen to be active or inactive.
Note; husk repository needed altering in order to fetch the necessary dependencies.

Hey, thank you for submitting this.
Parkour v6.X development (master) has seized while we continue to develop 7.0 as the codebase is considerably different.
If you want to port this over to the 7.0 codebase I'd be happy to accept it.


add JwL's variability stats to PMP

from this https://github.com/PCMDI/modes_of_variability/blob/master/devel/variability_mode/lib/calc_stat.py
add to https://github.com/PCMDI/pcmdi_metrics/tree/master/src/python/pcmdi
Consult with @doutriaux1 on how best to do this... one file as in JwL's code or an individual file for each statistic?
@lee1043

@doutriaux1 We don't have a PR for the @lee1043 variability modes, so for 1.2.0 we can leave this until 1.3.0

Currently working on it: https://github.com/PCMDI/pcmdi_metrics/pull/623

Variability code uses eofs and scipy which are currently not included as PMP dependencies. Should we consider including them or let users do their homework by installing those libraries by themselves? @gleckler1 what do you think?

@lee1043 Good question.  I think for right now its better to not include them.

Just chiming in, I'm pretty sure that with the dependencies of cdms you're already going to have scipy, eofs is unlikely
Take a peek at https://github.com/CDAT/cdms/blob/master/recipe/meta.yaml.in#L32-L47

@durack1 you are correct for the scipy, thanks for pointing it out.

PR merged.


Improve error message when the query is empty

https://issues.redhat.com/browse/LOG-3454
This PR adds better error messages when the query is empty, also it prevents to send an empty query to the backend

/cherry-pick release-5.6

/lgtm


fix link to tags for multilang

Thanks for the PR. I don't have a multilang site to test but it doesn't appear to affect single lang sites. Have you tested the patch?

I have tested it with a multilang site. It does fix the problem with tags under multilang sites. Merging.

Yes, I have tested it too, thanks!


Fix: updated unique derivation path validation

Changes:

Validate derivation path to be unique only with keys in same network

closes 1431

@egor0798 import is not fixed. I'm getting error that keys are duplicated.


[Add]: Awesome Public Datasets

Site URL
www.github.com/awesomedata/awesome-public-datasets
Description
Awesome Public Datasets is a curated list of publicly available datasets across various domains and disciplines. It is hosted on GitHub and maintained by the community. The repository contains a collection of links to high-quality datasets that are freely accessible for research, analysis, and other purposes.
Category
Datasets 📊
Checklist

[X] I have checked the existing issues.
[X] I have read the Contributing Guidelines.
[X] I am willing to work on this issue (optional).

what tags are you going to put?


[Question]: Help regarding unit test cases

Describe your problem
First of all, great thanks to the great infiniflow team to build the great project.
Have some unit tests with projects is a good practice to make the user have more confident with the project.
I'm trying to make some api unit tests against ragflow/api/apps/api_app.py, there's an issue can't bypass

E   NameError: name 'manager' is not defined

Attached pytestcode: [test_api_app.py]
test_api_app.txt
@KevinHuSh
Thanks for you code register_page in api/apps/init.py, it have developer to use manager without import needed,
but maybe this cause the issue, do you have any ideal about this?
@luckydrq
see your previous(https://github.com/infiniflow/ragflow/issues/644) concern relate this, hope you have some progress on this
``

ragflow/api/apps/api_app.py
This code is gona be deprecated soon.


Make dryRun messages distinguishable in the logs

We have more normal logs now, so finding what is dryRun can be hard. Make it more distinguishable, like prefix DRY_RUN or GITHUB_CALL or similar.

Possible solution might be injecting WIldflyBotConfig into the PullRequestLogger and prepending this message based dryRun() method


Bump config.guess and config.sub

Coverage remained the same at 80.103% when pulling c9debfece5bb37913cd806b7f0c517bc8eb31796 on kulti:master into 8b4485135ff13d829f14a4ddbd2543aeca8aa2e1 on tbeu:master.

Coverage remained the same at 80.103% when pulling c9debfece5bb37913cd806b7f0c517bc8eb31796 on kulti:master into 8b4485135ff13d829f14a4ddbd2543aeca8aa2e1 on tbeu:master.

Coverage remained the same at 80.103% when pulling c9debfece5bb37913cd806b7f0c517bc8eb31796 on kulti:master into 8b4485135ff13d829f14a4ddbd2543aeca8aa2e1 on tbeu:master.

Coverage remained the same at 80.103% when pulling c9debfece5bb37913cd806b7f0c517bc8eb31796 on kulti:master into 8b4485135ff13d829f14a4ddbd2543aeca8aa2e1 on tbeu:master.

Coverage remained the same at 80.103% when pulling c9debfece5bb37913cd806b7f0c517bc8eb31796 on kulti:master into 8b4485135ff13d829f14a4ddbd2543aeca8aa2e1 on tbeu:master.

Coverage remained the same at 80.103% when pulling c9debfece5bb37913cd806b7f0c517bc8eb31796 on kulti:master into 8b4485135ff13d829f14a4ddbd2543aeca8aa2e1 on tbeu:master.

Coverage remained the same at 80.103% when pulling c9debfece5bb37913cd806b7f0c517bc8eb31796 on kulti:master into 8b4485135ff13d829f14a4ddbd2543aeca8aa2e1 on tbeu:master.