Improving the UX for docs

Discussed in https://github.com/prjctimg/huetiful/discussions/62

Originally posted by prjctimg October 24, 2023
A lot of effort needs to be put into improving the overall UX of the documentation site.
Over the next month we'll be adding small but noticeable improvements to help make it easier to navigate the site. Contributions are greatly appreciated devs!!
Improvements
Homepage

[ ] #70

Reading View

[ ] #71
[ ] #72
[ ] #73
[ ] #74
[ ] #75

List View

[ ] #76

How about listing the things (UX improvements mentioned in https://github.com/prjctimg/huetiful/discussions/62) that need to be done here and make this issue a tracking issue?
We can then open separate issues for each task. This way it will be easier to keep track of what needs to be done and make easier the review process.

How about listing the things (UX improvements mentioned in https://github.com/prjctimg/huetiful/discussions/62) that need to be done here and make this issue a tracking issue?
We can then open separate issues for each task. This way it will be easier to keep track of what needs to be done and make easier the review process.

That's a brilliant idea! Feel free to handle the issue tracking as you see effecient!
I have no prior experience in working with GitHub Issues so I'm willing to learn the best practices!

@prjctimg, I have opened a couple of issues for the improvements. Feel free to modify them and add more details if needed.
@adarshswaminath, feel free to grab any issue(s) you feel comfortable with.

@prjctimg, I have opened a couple of issues for the improvements. Feel free to modify them and add more details if needed.
@adarshswaminath, feel free to grab any issue(s) you feel comfortable with.

@takanome-dev
This looks better and more manageable already.
Reviewing the new issues now 👍

Just updated the docs. The results are not so satisfying as I expected. The reading view looks okay on the mobile browser but styling seems to break on desktop browser 🤯. **slowly getting frustrated
You can preview the site here
There's two things I'm thinking could be causing the style inconsistencies

Originally the docs are generated in .md format and then I change the extension to .mdx after injecting the typedoc generated docs into the template files. Will using the plain .md format fix this issue?
Our stylesheet is removing the default Markdown styling therefore making all the elements appear similar.

I was on Microsoft Edge when I saw this issue on the desktop browser.
Please let me know if you notice the inconsistencies too on your side.
@adarshswaminath @takanome-dev @foss2cyber

Just updated the docs. The results are not so satisfying as I expected. The reading view looks okay on the mobile browser but styling seems to break on desktop browser 🤯. **slowly getting frustrated
You can preview the site here
There's two things I'm thinking could be causing the style inconsistencies

Originally the docs are generated in .md format and then I change the extension to .mdx after injecting the typedoc generated docs into the template files. Will using the plain .md format fix this issue?
Our stylesheet is removing the default Markdown styling therefore making all the elements appear similar.

I was on Microsoft Edge when I saw this issue on the desktop browser.
Please let me know if you notice the inconsistencies too on your side and have suggestions for the best way forward with least obstacles.
@adarshswaminath @takanome-dev @foss2cyber

I'm currently using Brave, and the desktop view is working great! However, I've noticed some areas for improvement in the mobile view. 😊
I want to apologize as I'm unable to work on any tasks right now due to my exams. Once my exams are over, I'll be able to make more contributions to the project @prjctimg

@prjctimg I've previewd the site on both Chrome & Firefox. Looks fine on desktop, but a few glitches in the mobile view.

@adarshswaminath @foss2cyber
Thanks for reviewing! 🚀
Can you please note down a few areas you think we should work on immediately ?
At the moment our goal is to make the docs easily readable so that we can improve the overall developer experience. In fact, we're hoping to release v1.7.8 on npm on Monday (at the moment we have prereleases).
We cannot add more features if we can't make developers use what we have already developed so far.

@prjctimg I've previewd the site on both Chrome & Firefox. Looks fine on desktop, but a few glitches in the mobile view.

Could you please share screenshots ? Maybe my Microsoft Edge is buggy

I'm currently using Brave, and the desktop view is working great! However, I've noticed some areas for improvement in the mobile view. 😊
I want to apologize as I'm unable to work on any tasks right now due to my exams. Once my exams are over, I'll be able to make more contributions to the project @prjctimg

No worries! We'll be sure to keep a heap of issues awaiting your resolve 😂 👍

@prjctimg I've previewd the site on both Chrome & Firefox. Looks fine on desktop, but a few glitches in the mobile view.

Could you please share screenshots ? Maybe my Microsoft Edge is buggy @foss2cyber

Sorry for replying so late, mate. Sharing screenshots below:

Desktop (Chrome):

Desktop (Firefox):

Mobile (Chrome):

Mobile (Firefox_screenshot 1-3):

@foss2cyber Thanks for testing it out! I think it was because of my internet connection being slow so it wasn't properly loading the stylesheet. Our network says 4G but behaves like slow 3G 😂
Making a PR for releasing v1.7.8 now. 👍


Panel with Bulk Actions doesn't appear on select finding

Panel with Bulk Actions doesn't appear on select finding
Panel with Bulk Actions doesn't appear on select finding. It appears only if choose "select_all".
Steps to reproduce
Steps to reproduce the behavior:

Go to Findings page
Click on the checkbox of any finding in the table. It is first column.
See panel "Bulk Actions" doesn't appear on the top of the table.

Expected behavior
Panel with Bulk Actions should appear if at least one finding was chosen in the table.
Deployment method (select with an X)

[ ] Kubernetes
[X] Docker
[ ] setup.bash / legacy-setup.bash

Actually the problem is in the initialization of event handle for checkboxes "onchange" in the table.
It was done before DataTable initialization. During DataTable initialization stage all "old" event handles for checkboxes were removed and only DataTable event handles.

Hm, Looks like this issue was solved in dev branch

yeah we did quite some javascript related fixes in dev since the last release on master.  please reopen when needed.


[Enhancement] Extend Gravity debug report

Extend Gravity debug report to include the following to start:

Capture last command from host
dmesg
coredumpctl - grep output from it

Duplicate of https://github.com/gravitational/gravity/issues/1273.


Issue with the error :

Hi @arvindrajan92 ,
Thank you for your efforts in replicating DTrOCR.
Recently, I tried the sample example notebook to fine-tune the model on IAM dataset from here
Data is getting loaded effectively, but getting Unused or unrecognized kwargs: padding. error. This issue is popping up for the same IAM dataset as suggested for finetuning.
I have tried to change padding=False and also padding='longest' and padding='do_not_pad' as arguments instead of padding='max_length', but do not seem to address the issue.
Please kindly let me know how to address this issue. I have attached a screenshot of the tracktrace for your reference. Thanks

@kasuba-badri-vishal, Thank you for bringing this to my attention. I will check in a moment.
Can you share with me the python and PyTorch versions? I'll see if I can reproduce.

Certainly, here are the details of my environment:
Python version: 3.10.12
PyTorch version: 2.3.1+cu121
GPU: NVIDIA RTX A6000
Driver Version: 535[535.183.01]
CUDA Version: 12.2
If you need any additional information, please let me know. Thanks

I managed to reproduce the issue and seems to be related to https://github.com/huggingface/transformers/pull/31950.

It should be gone now @kasuba-badri-vishal if you could pull the latest main branch. If you still see it, feel free to reopen this issue.


Overriding not working with 0.2

Hi! I'm trying to upgrade to 0.2 version and moving my simple config from:
customForms: theme => ({ focusShadow: theme("boxShadow.none") })

to (according to the docs)
customForms: theme => ({
  input: {
    "&:focus": {
      boxShadow: undefined
    }
  }
})

and still seeing boxShadow on inputs.
I did a quick console.log driven debugging and found out that my custom options are not even passed to addInput function. To do that and not to break custom variants it needs to be wrapped into default key, ie:
customForms: theme => ({
  default: {
    input: {
      "&:focus": {
        boxShadow: undefined,
      }
    }
  }
})

Is that right and docs are just missing that?

Yep you need the default key, sorry about that I’ll get the docs fixed today.
On Jun 23, 2019, 6:02 AM -0400, Ilya Zayats<EMAIL_ADDRESS>wrote:

Hi! I'm trying to upgrade to 0.2 version and moving my simple config from:
customForms: theme => ({ focusShadow: theme("boxShadow.none") })
to (according to the docs)
customForms: theme => ({
input: {
"&:focus": {
boxShadow: undefined
}
}
})
and still seeing boxShadow on inputs.
I did a quick console.log driven debugging and found out that my custom options are not even passed to addInput function. To do that and not to break custom variants it needs to be wrapped into default key, ie:
customForms: theme => ({
default: {
input: {
"&:focus": {
boxShadow: undefined,
}
}
}
})
Is that right and docs are just missing that?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Great, thank you!


cannot import name 'cublas'

I'm using chainer '2.0.0b1' (chainer 1.24 was working fine)
on CentOS Linux release 7.3.1611 (Core)
with CUDA release 8.0, V8.0.61
I'm getting the following error message when calling hainer.cuda.check_cuda_available()
RuntimeError                              Traceback (most recent call last)
<ipython-input-3-14ec8f3756fb> in <module>()
----> 1 chainer.cuda.check_cuda_available()

/work/alex/opt/lib/python3.6/site-packages/chainer/cuda.py in check_cuda_available()
     73                '(see https://github.com/pfnet/chainer#installation).')
     74         msg += str(_resolution_error)
---> 75         raise RuntimeError(msg)
     76     if (not cudnn_enabled and
     77             not _cudnn_disabled_by_user and

RuntimeError: CUDA environment is not correctly set up
(see https://github.com/pfnet/chainer#installation).cannot import name 'cublas'

I see several similar issues being posted, but a bit different, for example people are experiencing exceptions on importing libcuda, not libcublas.
On my CentOS libcublas.so is in
/usr/local/cuda-8.0/targets/x86_64-linux/lib, symlinked to /usr/local/cuda-8.0/lib64, included in  my $LD_LIBRARY_PATH and  $LIBRARY_PATH variables.

hm... I've tried adding /usr/local/cuda-8.0/lib64/stubs/ to LD_LIBRARY_PATH, now I'm getting
You are running using the stub version of libcublas

Did you install cupy? Maybe this is new problem because chainer 2.0.0 itself is not depend on CUDA. It just uses cupy, that is NumPy compatible matrix library using CUDA. So, i think it is cupy's problem.
Chainer 2.0.0 does not automatically install CuPy. So, you need to install cupy manually.
$ pip install cupy --pre

forcing cupy uninstall/ installing new one solved the issue.
thanks so much!
p.s.
BTW, installed (with --pre) version is cupy-1.0.0b1, not sure if it was supposed to be 2.x judging by your last comment


Add a rev named argument to the find and position of array and position of str

Partially fixes #4981.
Currently lacks support for reverse regex search; Could be implemented with regex_automata supposedly.
I can try to implement reverse regex search using regex_automata if we want to go that route.

I think going down the regex-automata route is a bit overkill. Notably, trim already does some reverse-regex stuff, though inefficiently and maybe with slightly different semantics than one might think (not sure).
We also still need to settle on naming. from: start | end flows quite well, though I'm still a bit concerned about .position(from: end) sounding like it counts from the back.

I see. I don't have a particular opinion; How should we go through with this?

How should we go through with this?

Unfortunately, I don't know either.

Closing until it's clear how we want to do this

I asked on Discord. One person so far was in favor of rev, which is a reasonable stance. It is slightly less nice than from: start | end but does not suffer from its problems. Code-wise the PR looked very good, so once we've decided, we can just reopen.


Work Completed ?

After install "npm i -g awox-smartlight", I got this error :
root@debian:/usr/bin# awoxlight
/usr/bin/awoxlight: ligne 1: /bin : is a directoy

And the "/usr/bin/awoxlight" is empty :)
Thank you for your help

Hi @FlorLouv,
Sorry, for the command line the work is not done but the library is fonctionnal. You can use it in your script. You have an example in the readme for how to import it in a script.
I will document every methods and add the command line tool later. For the moment you need to read the code to see what are the available methods.
If you need any further information don’t hesitate to ask me.

@dragouf @FlorLouv
project is die or?  Any progress about CLI and other bugfix?

Up!

If abandoned it could be transfered to @abandonware org see:
https://abandonware.github.io/


Deploy CRDs only as required by components

We no longer need the Submariner CRDs all the time; only when
connectivity is required. Service-discovery-only deployments shouldn’t
have the Submariner CRDs.
The operator however needs the CRDs for resources it manages; this
includes service discovery, so that CRD is added when setting up the
operator.
This patch distinguishes deployments by components, and only sets up
the relevant CRDs. It includes a stub Globalnet component which
shouldn’t be used as such but will be fleshed out later.
Fixes: #1098
Signed-off-by: Stephen Kitt<EMAIL_ADDRESS>

@skitt this one needs some love :)

The unit test job keeps failing - make fails with exit status 2 but I don't see a reason for it - all the tests pass.

The unit test job keeps failing - make fails with exit status 2 but I don't see a reason for it - all the tests pass.

I also don't see a PR here or in Shipyard that could have caused this.
It seems to have started just a few hours ago.
https://github.com/submariner-io/submariner-operator/actions/workflows/unit.yml

The unit test job keeps failing - make fails with exit status 2 but I don't see a reason for it - all the tests pass.

I also don't see a PR here or in Shipyard that could have caused this. I don't see similar failures in other repos.
It seems to have started just a few hours ago.
https://github.com/submariner-io/submariner-operator/actions/workflows/unit.yml

yeah it mysteriously started. One difference between the failing runs and the successful runs prior is that the former have this near the end just prior to failing:
[submariner-operator]$ uname
make: *** [/opt/shipyard/Makefile.inc:100: unit] Error 1

Not sure of the significance or where that is coming from.


swizzle NSStream?

Looks like NSStream is not swizzled by TrustKit and react-native's socket implementation is using that.
Is that possible to do?
This link might be helpful?
https://stackoverflow.com/questions/28413923/ssl-handshake-with-my-certificate-by-nsstream

Hello,
Swizzling tends to make the code a lot more complex, and also a lot harder to maintain. While we still support swizzling for NSURLConnection and NSURLSession, we advise developers to disable it and instead implement the proper code to call into TrustKit, in order to avoid the complexity of the swizzling implementation.
Because of this we will not add swizzling for more APIs (such as NSStream).


cloud-init settings application

What does RancherOS' cloud-init implementation do exactly? Does it scan the cloud-config / user-data settings on every boot and applies them idempotently (that's my impression, but I might be wrong)?
Documentation on the topic is scattered and unclear:

from original cloudinit docs I get the impression it runs only on first boot,
from CoreOS cloudinit-cloud-config doc I see it runs on every boot.

RancherOS docs list the settings (and I guess the list is not complete), but don't specify at what moment they are applied.
BTW, is it possible that settings be applied immediately, e.g. on rancherctl config set, without reboot?

We are still working on having all the setting in rancherctl be dynamic and not require reboot, but for now a reboot is required (really bugs me too).
cloud-init is ran on every boot and will look at the data sources configured from cloud_init.datasources.


depreacted: nightmare

Motivation/Description of the PR

Deprecated Nightmare helper as it's not actively maintained anymore.

@kobenguyent something wrong with definitions
Could you regenerate them with
./runok.js def

And try again?

Sorry, now it conflicts with merged Protractor )))
Maybe you have one PR?


Be consistent in naming

Name of matchers must be TypeMust where Type is a original type.
Even though MustEq seems correct, names like PartialEqMust allows users to know what it's extending, and are really unlike to conflict with user's type.

Deprecated syntax


Exposing the OnEviction callback

Could the OnEviction callback in the lre cache be exposed by groupcache? Is there a reason this isn't exposed that may not be obvious?

Because it would complicate the API. Why does your program care?

Keeping the API simple sounds like a valid reason.
I suppose i'm really asking for a way to know if a key is in the cache or not. My end goal was, when making a decision on something, to be able to give more weight to a path that would use the cache. I had thought to do that by tracking the cached items externally but as I type that out now it sounds a bit over the top.
If a method were added to query for existence of a key in the cache without filling it would that also be considered an unnecessary complexity to the API?

Complex or not, it's racy. That makes it impossible to use correctly. Let's say you test for the presence of a key and find it, but then by the time you actually access the cache the key has expired. What have you gained?
Groupcache is supposed to be an abstraction that removes the need to think about this stuff. If it doesn't fit your use case, then you should take the groupcache code and adapt it so that it does. Adding more features to the API is probably not the right approach, as it is very hard to design a general purpose cache.

Yeah, the race condition is a problem, but in the case i'm thinking of, where i'm faced with a choice of N otherwise similar paths and I could choose one that's more likely to be in the cache based on the most up to date data then I don't lose much if that information turns out to be stale.
However, you are right. It doesn't fit the intended use of groupcache so it's not a good idea to add thant kind of functionality.
Thanks for taking the time to respond.


docs: Update dotnet configuration

Thanks for submitting a PR! Please check the boxes below:

[x] I have run pre-commit to check linting
[x] I have added information to docs/ if required so people know about the feature!
[x] I have filled in the "Changes" section below?
[ ] I have filled in the "How did you test this code" section below?
[x] I have used a Conventional Commit title for this Pull Request

Changes

Update .net configuration

Why have we deleted all the current content? Surely we need both pieces of info here? We've lost all the comments in the original file?


nodeMetadataStruct does not work in native VM

nodeMetadataStruct called inside onConfigure returns an empty struct for an envoy with non-empty node metadata.

onConfigure is only called if there is a non-empty configuration.
If you just want to get called when the VM starts up use the onStart hook.
See github.com/jplevyak/enovy-wasm on the wasm-node-metadata flag for details.
I added a dummy configuration and it prints out the metadata.  I also used onStart to print out the
metadata.
We need docs.  I am opening an issue for that.

https://github.com/envoyproxy/envoy-wasm/issues/16

Here is my envoy.yaml:
node:
  id: test
  metadata:
    labels: test
static_resources:
  listeners:
  - name: main
    address:
      socket_address:
        address: <IP_ADDRESS>
        port_value: 8080
    filter_chains:
    - filters:
      - name: envoy.http_connection_manager
        config:
          stat_prefix: ingress_http
          codec_type: auto
          route_config:
            name: local_route
            virtual_hosts:
            - name: local_service
              domains:
              - "*"
              routes:
              - match:
                  prefix: "/"
                route:
                  cluster: web_service
          http_filters:
          - name: envoy.wasm
            config:
              vm_config:
                vm: "envoy.wasm.vm.null"
                code:
                  inline_string: "envoy.wasm.metadata_exchange"
              configuration: "test"
          - name: envoy.router
            config: {}
...

Here is the code in the plugin:
void PluginContext::onConfigure(std::unique_ptr<WasmData> /* configuration */) {
  logInfo("configuring metadata exchange");

  // populate and encode node metadata
  auto metadata = nodeMetadataStruct();
  logInfo("metadata struct " + std::to_string(metadata.fields().size()));
}

Here is the output:
[2019-05-28 11:47:07.001][6634][info][wasm] [source/extensions/common/wasm/wasm.cc:1033] wasm log: configuring metadata exchange
[2019-05-28 11:47:07.001][6634][info][wasm] [source/extensions/common/wasm/wasm.cc:1033] wasm log: metadata struct 0

As you can see, node metadata does not propagate.

I branched off 1025deebb9d330501738b882da48a4152e3ab2e0 if it helps. Can you re-open so we can this resolved?

@jplevyak I compared your branch against mine, and the only notable difference is using WAVM in your case and native "sandbox" in mine. Can you try with native/null sandbox and see if it changes results?

OK, so I updated my example to use the null sandbox and pushed it.  It works as expected (by me).
PTAL, I'd love to figure out where the difference is.

I replaced nodeMetatadataStruct with getMetadataValuePairs, and that resolved my issue. The only difference seems to be a cache in the context. Seems like there is some issue with cache initialization.

obsolete


Support regex in MongoDB

Problem
I'm using MongoDB
I need accent-insensitive in queries so I need to transform string to RegExp. But $regex only support in findRaw.

Also I cannot transform to Regexp type in findRaw. It cause typescript error RegExp type:
prisma.thu.findRaw({
      filter: {
       name: {
        $regex: new RegExp('Jesús', 'i') // RegExp type
       }
      }
    })

Please open a separate issue for the bug if you want this issue to become a feature request.
Or open a separate issue for the feature request, if we should treat this as the bug issue.


Add ability for Galley to serve Mesh Config.

Update pkg/runtime to accept schema through config, instead of using metadata.Types directly. Processor now adds meshconfig when created. pkg/server adds meshconfig to schema before passing to processor and source.CollectionsOptions. Add meshconfig to integration test.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: jeffmendoza
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: ozevren
If they are not already assigned, you can assign the PR to them by writing /assign @ozevren in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

galley/OWNERS

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

@jeffmendoza: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
8fcf1ca05a1d21707d9885529b79918c8b29c2e1
link
/test istio-unit-tests-master

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

Looks good modulo the TODO comment. Let me know when you update and have the tests passing, and I can LGTM.

@jeffmendoza: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-simpleTests.sh
28d1c80f06ccc903cccf3612ac5ffbcb98d124ff
link
/test e2e-simpleTests-master

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test e2e-simpleTests-master

/lgtm

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

Needs approval from an approver in each of these files:

galley/OWNERS [ozevren]

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

@jeffmendoza: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-simpleTests.sh
28d1c80f06ccc903cccf3612ac5ffbcb98d124ff
link
/test e2e-simpleTests-master

prow/integ-mixer-k8s-presubmit-tests.sh
28d1c80f06ccc903cccf3612ac5ffbcb98d124ff
link
/test integ-mixer-k8s-presubmit-tests-master

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test integ-mixer-k8s-presubmit-tests-master

@jeffmendoza: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/integ-pilot-k8s-presubmit-tests.sh
28d1c80f06ccc903cccf3612ac5ffbcb98d124ff
link
/test integ-pilot-k8s-presubmit-tests-master

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@jeffmendoza: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/integ-pilot-k8s-presubmit-tests.sh
28d1c80f06ccc903cccf3612ac5ffbcb98d124ff
link
/test integ-pilot-k8s-presubmit-tests-master

prow/integ-mixer-k8s-presubmit-tests.sh
28d1c80f06ccc903cccf3612ac5ffbcb98d124ff
link
/test integ-mixer-k8s-presubmit-tests-master

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test integ-mixer-k8s-presubmit-tests-master

How are you going to surface the ConfigSources piece in the mesh config? It is supposed to list the set of all galleys from which Pilot will receive config from. You probably have to separate that from mesh config, or continue supporting the ability in Pilot to read mesh config from config files, in addition to Galley in certain cases.

@jeffmendoza: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/integ-mixer-k8s-presubmit-tests.sh
28d1c80f06ccc903cccf3612ac5ffbcb98d124ff
link
/test integ-mixer-k8s-presubmit-tests-master

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@rshriram no plans to change Pilot's behavior at the moment.

/test integ-mixer-k8s-presubmit-tests-master

I also doubt, serve MeshConfig for who?


Run gofmt for hack/tools.go

What this PR does / why we need it:
This PR runs gofmt for hack/tools.go.
Does this PR introduce a user-facing change?:
NONE

/assign @kron4eg

heh, I see why it wasn't caught previously by linter
golangci-lint run ./pkg/... ./test/... 🔥

@kron4eg What would happen if we used golangci-lint run ./... (or golangci-lint run which is equivalent to the former command) instead? Would it include vendor too maybe?

I suppose this is leftover since dark times when we had vendor :D

Oh, right, we don't have vendor anymore. :man_shrugging: I'll update the PR to include all packages for lint.

@kron4eg Updated the PR.

@kron4eg Reverted the change of linter settings.

/retest

/override pull-kubeone-e2e-aws-upgrade-1.13-1.14


Ben/fix extended example forms

Added an extra extended form example to show recursion and added concept schemes to both, to show meta ttl is also working.

Very nice


docs(*) mention 0.10.1 CORS regression

mentioned in 0.10.1 changelog
mentioned in the 0.10.x upgrade path

LGTM


Adding ministere education egypt - (Teachers Domain)

This is A Email Domain for Teachers,
by ministere education of egypt.

@fatheyabdelslam I am sorry, this domain represents an umbrella institution, not a specific single university or a college. Please apply for free educational license with an official document which proves that you are a student. You can do it using "OFFICIAL DOCUMENT" tab at https://www.jetbrains.com/shop/eform/students.
I have to close the request without merging.
Thank you for your understanding!


Customer Account API integration not working when logging out from hosted pages

What is the location of your example repository?
No response
Which package or tool is having this issue?
Hydrogen
What version of that package or tool are you using?
2024.4.1
What version of Remix are you using?
No response
Steps to Reproduce

Integrate the Customer Account API into Hydrogen (see guide)
As a customer, log in
Navigate to the Shopify hosted account pages (not in Hydrogen, so https://shopify.com/store-id/account) + click log out
Navigate back to Hydrogen, notice that you are still in a logged in state

Expected Behavior
The Hydrogen state is logged out when the customer logs out of the hosted pages.
Actual Behavior
The customer is still logged in to Hydrogen despite logging out of hosted pages.
Note: I am making some assumptions here, such as the fact that hosted pages syncs that state to Hydrogen. If this isn't the case, shouldn't it be?

I believe it is not synced with Hydrogen due to performance reasons. Otherwise Hydrogen would need to add an extra HTTP call in every single request to double check the session is still valid, instead of letting it expire naturally.
cc @blittle @juanpprieto do you remember more about this issue?

This is expected behaviour.
When user logout under the checkout domain, hydrogen site does not know about the logout til the user trigger a behaviour that does a Customer Account API fetch and Hydrogen will then register a logout in the UI.
Security wise this is not an issue since if an user navigate back, they are only shown the data they already fetched while login. And action that trigger a new fetch (for new data) will then trigger a logout.
@romainshopify (from Customer Account API team), I believed this is already an UX improvement that's on your radar and being prioritize?


How do I restart the autoplay if the items number was reached?

The idea is if carousel reaches the number of items to show with autoplay, it can come back to the initial position or index 0. This feature is with infinite = false. In this moment, carousel comes back to the previous item in regard to current item. For example, I have 3 items to show, so  if the carousel reaches up to the index 2, then, my idea is that carousel goes to the item 0 inmediately, in order to keep counting up to reach the index 2 again. is this feature possible?

In other words, It's to make the same thing when it's pressed the first indicator of the carousel. Nevertheless, the idea is that this performance comes up when Carousel reaches the last item, It re-starting to index 0 as the next item to show. Either way, I will try to do this with the available documentation

Hey, I'm closing this because there's no test case and it's a support request.


net: Rename net_is_xxx...() functions to net_xxx_is...()

Unify the function naming for various network checking functions.
For example:
net_is_ipv6_addr_loopback() -> net_ipv6_is_addr_loopback()
net_is_my_ipv6_maddr() -> net_ipv6_is_my_maddr()
etc.
Signed-off-by: Jukka Rissanen<EMAIL_ADDRESS>

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@f87c4c0). Click here to learn what that means.
The diff coverage is 52.63%.

@@            Coverage Diff            @@
##             master   #11042   +/-   ##
=========================================
  Coverage          ?   53.25%
=========================================
  Files             ?      217
  Lines             ?    26698
  Branches          ?     5925
=========================================
  Hits              ?    14219
  Misses            ?    10060
  Partials          ?     2419

Impacted Files
Coverage Δ

subsys/net/l2/ethernet/arp.c
58.9% <0%> (ø)

subsys/net/ip/icmpv6.c
31.66% <0%> (ø)

subsys/net/lib/dns/resolve.c
36.22% <0%> (ø)

subsys/net/lib/app/client.c
52.52% <0%> (ø)

subsys/net/ip/icmpv4.c
32.62% <0%> (ø)

subsys/net/ip/rpl.c
13.97% <0%> (ø)

subsys/net/lib/coap_sock/coap_sock.c
60.13% <100%> (ø)

subsys/net/ip/connection.c
78.13% <100%> (ø)

subsys/net/ip/route.c
50.56% <100%> (ø)

subsys/net/lib/coap/coap.c
60.66% <100%> (ø)

... and 9 more

Continue to review full report at Codecov.

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


Vue CLI下启动报sockjs  CORS policy

Question
#vue ui
[WDS] Hot Module Replacement enabled.
App running at:

Local:   http://localhost:8002/
Network: http://<IP_ADDRESS>:8002/

[HMR] Waiting for update signal from WDS...
index.js?245a:7 mock mounting
index.js?245a:18 mock mounted
login:1 Access to XMLHttpRequest at 'http://localhost:8000/sockjs-node/info?t=1555293167359' from origin 'http://localhost:8002' has been blocked by CORS policy: No 'Access-Control-Allow-Origin' header is present on the requested resource.
login:1 Access to XMLHttpRequest at 'http://localhost:8000/sockjs-node/info?t=1555293168619' from origin 'http://localhost:8002' has been blocked by CORS policy: No 'Access-Control-Allow-Origin' header is present on the requested resource.
login:1 Access to XMLHttpRequest at 'http://localhost:8000/sockjs-node/info?t=1555293169684' from origin 'http://localhost:8002' has been blocked by CORS policy: No 'Access-Control-Allow-Origin' header is present on the requested resource.
login:1 Access to XMLHttpRequest at 'http://localhost:8000/sockjs-node/info?t=1555293171755' from origin 'http://localhost:8002' has been blocked by CORS policy: No 'Access-Control-Allow-Origin' header is present on the requested resource.
login:1 Access to XMLHttpRequest at 'http://localhost:8000/sockjs-node/info?t=1555293175846' from origin 'http://localhost:8002' has been blocked by CORS policy: No 'Access-Control-Allow-Origin' header is present on the requested resource.
login:1 Access to XMLHttpRequest at 'http://localhost:8000/sockjs-node/info?t=1555293183866' from origin 'http://localhost:8002' has been blocked by CORS policy: No 'Access-Control-Allow-Origin' header is present on the requested resource.
login:1 Access to XMLHttpRequest at 'http://localhost:8000/sockjs-node/info?t=1555293200248' from origin 'http://localhost:8002' has been blocked by CORS policy: No 'Access-Control-Allow-Origin' header is present on the requested resource.

Request URL: ws://<IP_ADDRESS>:8002/sockjs-node/863/1f0el2ko/websocket
Request Method: GET
Status Code: 101 Switching Protocols

Describe the solution you'd like
Additional context

用命令启动 不要用 vue ui 启动。 vue ui 占用了 8000 端口。
或者 vue.config 把默认配置的 8000 端口设置为别的端口重启


how to create best quality image with mel for low freq short (150 ms) sound?

(the problem can be understood from the attached pictures: 1 - my linear, 2 - my mel, 3 - need)
i use
fftSize = 1024
stepSize = 1
sampleRate = 44100

Hi @AlexandrSpirin, can you provide additional information about how you generated the last image? Is the whole image only 150 ms wide?

Привет, @AlexandrSpirin , не могли бы вы предоставить дополнительную информацию о том, как вы сгенерировали последнее изображение? Ширина всего изображения составляет всего 150 мс?

Yes, last image for same sound (150 ms), i use EDM kick for example in WAV. This image can be obtained with "Edison" by Image Line

https://drive.google.com/file/d/14mqbJtz4HRRm-DuACueFrf28fGufKmfl/view?usp=sharing
Github does not allow attaching audio format files, file by link.
The picture shows a spectral pattern with a linear representation for the same sound. The picture is similar to what your library gives in linear mode
Perhaps this is not a spectrogram heatmap, but something else. It is important for me to get the correct display of low frequencies in mel mode.
It looks like your picture uses a voice that doesn't have enough low frequencies. But it also shows that not just pixels are used in displaying sound, but really a set of lines. It would be very cool if your library was capable of this.

Привет @AlexandrSpirin ,
Я изо всех сил пытаюсь понять, что это за изображение на самом деле ... Это не похоже на тепловую карту спектрограммы
Я использую термин «тепловая карта спектрограммы» для определения прямоугольного изображения, где каждый квадратный цвет пикселя представляет мощность частоты в определенное время. Изображение, которое вы пытаетесь создать, не похоже на тепловую карту спектрограммы.
Скриншот Эдисона на https://www.image-line.com/fl-studio-learning/fl-studio-online-manual/html/plugins/Edison.htm показывает тепловую карту спектрограммы ... но это не выглядит нравится, какой у вас имидж.
Конечно, цвета такие же, но изображение, которое вы пытаетесь создать, выглядит как серия линий, а не тепловая карта. Я не могу найти никаких примеров на странице Эдисона.
Как вы создали это изображение и возможно ли, что это не тепловая карта спектрограммы?
Также может быть полезно, если вы перетащите этот файл WAV в свой комментарий, чтобы мы оба могли быть уверены, что обсуждаем один и тот же сигнал.

For future reference TempKick.zip 😉

not just pixels are used in displaying sound, but really a set of lines. It would be very cool if your library was capable of this.

I think this is what may be making the discussion complicated. At this time I think it's best to keep this library focused on making spectrograms (the rectangular/pixel kind, and what Wikipedia shows when you look-up the word).
I recognize there may be other ways to display spectral data, such as line plots, but that's probably outside the scope of this library. There are lots of great libraries to make line plots including https://scottplot.net
All that said, I'll take a closer look at the WAV file you sent to see if I can make it look better than what you originally posted 👍

Hi @AlexandrSpirin, I'm sorry it took me so long to get to this! The latest ScottPlot release took a lot more time than expected, then I got sucked into refactoring FftSharp's windowing system
Regarding this topic, I learned a lot more about the situation and will share what I found. First, here's some sample code which generates the following image:
static class Program
{
    static void Main()
    {
        // load data from the audio file
        (double[] values, int sampleRate) = ReadWavMono("data/kick.wav");

        // instantiate a spectrogram generator
        var sg = new Spectrogram.SpectrogramGenerator(sampleRate, fftSize: 4096, stepSize: 15, maxFreq: 5_000);

        // use a custom window to refine time and frequency resolution
        var window = new FftSharp.Windows.Hanning();
        double[] windowValues = window.Create(size: 1000);
        sg.SetWindow(windowValues);

        // use a custom colormap
        sg.Colormap = Spectrogram.Colormap.Magma;

        // pass audio data in for processing then save the spectrogram
        sg.Add(values);
        sg.SaveImage("kick.png", intensity: 5, dB: true);

        // create a Mel spectrogram too
        var mel = sg.GetBitmapMel(400, 100, dB: true);
        mel.Save("kick-mel.png");
    }

    static (double[] audio, int sampleRate) ReadWavMono(string filePath, double multiplier = 16_000)
    {
        using var afr = new NAudio.Wave.AudioFileReader(filePath);
        int sampleRate = afr.WaveFormat.SampleRate;
        int bytesPerSample = afr.WaveFormat.BitsPerSample / 8;
        int sampleCount = (int)(afr.Length / bytesPerSample);
        int channelCount = afr.WaveFormat.Channels;
        var audio = new List<double>(sampleCount);
        var buffer = new float[sampleRate * channelCount];
        int samplesRead = 0;
        while ((samplesRead = afr.Read(buffer, 0, buffer.Length)) > 0)
            audio.AddRange(buffer.Take(samplesRead).Select(x => x * multiplier));
        return (audio.ToArray(), sampleRate);
    }
}

You'll notice a couple things:

The images do not look as good as the one produced by iZotope
The mel-scaled image has black strips

I'll keep this brief but if you'd like me to discuss any of these topics in more detail, let me know. Here are my conclusions.

I can summarize this topic with "The Spectrogram package is designed to make spectrograms from large amounts of data." The task you described (analyzing extremely small amounts of data over extremely short periods of time) is outside the scope of what this package was designed for.

iZotope is doing a lot of "magic" in the frequency domain (vertical) and time domain (horizontal) to create their high quality image from extremely small amounts of input data. It's like non-linear anti-aliasing vertically, and carefully designed windowing and stepping horizontally. These functionalities are not built into this package, but if you wanted to you could implement it yourself. To do a deep dive into this, I recommend interacting with the FftSharp package directly (not Spectrogram) to create FFTs using a highly customized FFT size and window. You can implement the non-linear Mel-scaling yourself.

This module's Mel scaling system requires the number of bins (rows of pixels on the output spectrogram) to be significantly fewer than the input data (original, non-scaled spectrogram). The intent of this package is to create ~500px high Mel spectrograms from ~5,000 px high linear scaled spectrograms. When this ratio isn't respected the Mel scale will appear with black bars where there is missing data. What I think iZotope is doing is filling these gaps in with data extrapolated from the adjacent bins.

I know these explanations won't be particularly satisfying considering the type of images you're trying to achieve, but hopefully I was able to shed some light onto why the data is analyzed the way it is. If you want to achieve output similar to iZotope you probably can by writing some custom code interacting with the FFTs directly. Thanks for opening this issue! This topic (analyzing extremely small pieces of audio) isn't something I was really familiar with initially, and I'm happy we both got the opportunity to dive deeper into it.
Let me know if you have any more questions! Best of luck with your project 👍 🚀

Hi @swharden !
I went back to my project, studied the problem in more detail.
I use FFTSharp library for window selection, picked the one that looks better (Kaiser-Bessel). But I can not understand where and what exactly need to change the code to get extrapolated data from the neighboring cells to get a spectrogram without black bars.
From what I found in the free access - you can create "OverlapData" method to add data to fft list from neighboring array. But I can't do it correctly - as a result on spectrogram additional data is displayed at the top of spectrogram and it looks very strange.
I would be grateful for any instructions, and especially for code examples.
My current parameters for generating spectrogram:
fftSize = 1024
stepSize = 4
sampleRate = 44100
melBinsCount = 256
I think if we can figure this problem out, it could be added as an additional option in your module

Hi @AlexandrSpirin,
Thanks for the follow-up!

extrapolated data from the neighboring cells to get a spectrogram without black bars

Since I don't personally use Mel spectrograms, it's not very high priority for me to implement the interpolation functionality required to create the type of smooth non-linear spectrogram you're describing.
However, if you end up implementing this type of image interpolation yourself and finish creating a function that does it, I would be happy to review a pull request and merge your function into this package so others may use it as well!
Feel free to follow-up with additional comments here if you find additional information that would be useful to others interested in achieving a similar effect.
Thanks again for your comments, and good luck!
Scott


Mutating a statement including the ternary operator (?:)

Hi @rjust
I think the current version of Major does not mutate statements with the 'ternary operator'.
condition ? expression1 : expression2

I think we should mutate the condition part to, for example, true or false.
I attempted to solve this using the Major-MML feature, but I failed.
Could you please give me any suggestions?
Many thanks,
Donghwan

Hi @donghwan-shin,
Thanks for reporting this. The COR mutation operator should apply to the condition of a ternary operator. Do you have a minimal working example?
Thanks,
René

Hi @rjust,
How about Closure 7 as a small example for that? The following is ../Closure/patches/7.src.patch
diff --git a/src/com/google/javascript/jscomp/type/ChainableReverseAbstractInterpreter.java b/src/com/google/javascript/jscomp/type/ChainableReverseAbstractInterpreter.java
index fa7d47d..0afabc8 100644
--- a/src/com/google/javascript/jscomp/type/ChainableReverseAbstractInterpreter.java
+++ b/src/com/google/javascript/jscomp/type/ChainableReverseAbstractInterpreter.java
@@ -610,13 +610,9 @@ public abstract class ChainableReverseAbstractInterpreter
     public JSType caseObjectType(ObjectType type) {
       if (value.equals("function")) {
         JSType ctorType = getNativeType(U2U_CONSTRUCTOR_TYPE);
-        if (resultEqualsValue) {
+        return resultEqualsValue && ctorType.isSubtype(type) ? ctorType : null;
           // Objects are restricted to "Function", subtypes are left
-          return ctorType.getGreatestSubtype(type);
-        } else {
           // Only filter out subtypes of "function"
-          return type.isSubtype(ctorType) ? null : type;
-        }
       }
       return matchesExpectation("object") ? type : null;
     }

Many thanks,
Donghwan

Perfect, I'll take a look and let you know.
Thanks,
René

The latest version of Major, which is now the default in Defects4J, fixes this issue.


Option to pass-through SERVER_SIDE_RENDERED payload

Extend beyond the hubot spec. Enable powerful custom bots designed for Rocket.Chat
Enable bot to receive each message with an additional JSON formatted payload - containing the server-side pre-rendered message content in HTML+CSS.
Must maintain backwards compatibility with existing hubot scripts that DO NOT know about such extension.

@Sing-Li would this now be covered by https://github.com/RocketChat/Rocket.Chat/issues/6786
Or is it a different concern, to send HTML+CSS messages. Does Rocket.Chat even allow that?
If there's a RC server method to call that would send a HTML message, please point me at it because I think it would be quite a useful feature.


update opengl git url

see https://stackoverflow.com/questions/70663523/the-unauthenticated-git-protocol-on-port-9418-is-no-longer-supported
should hopefully fix the failing windows tests
Example failure from https://github.com/tlambert03/napari-micromanager/runs/5592092388?check_suite_focus=true

Looks like that worked.
@tlambert03 if you have a moment this should be an easy merge and make reviewing other PRs much easier on the eyes :)


feat: update kzg to the latest decoupled and bigendian version

updates kzg to latest decoupled and bigendian version
adds a cli flag to provide customized trusted setup txt file
updates trusted setup txt to latest devnet-6 version fron c-kzg main repo trusted setup
remove the bin=> txt conversion step to directly use the trusted setup txt format

:tada: This PR is included in v1.9.0 :tada:


pip install fails on OSX

mba-3:~ romanvg$ pip install feather-format
Collecting feather-format
  Downloading feather-format-0.1.2.tar.gz (99kB)
    100% |████████████████████████████████| 102kB 544kB/s
Requirement already satisfied (use --upgrade to upgrade): cython>=0.21 in ./.anaconda/lib/python2.7/site-packages (from feather-format)
Building wheels for collected packages: feather-format
  Running setup.py bdist_wheel for feather-format ... error
  Complete output from command /Users/romanvg/.anaconda/bin/python -u -c "import setuptools, tokenize;__file__='/private/var/folders/rb/k3q8j6j53_d5fcns670ntm380000gn/T/pip-build-7NVZlf/feather-format/setup.py';exec(compile(getattr(tokenize, 'open', open)(__file__).read().replace('\r\n', '\n'), __file__, 'exec'))" bdist_wheel -d /var/folders/rb/k3q8j6j53_d5fcns670ntm380000gn/T/tmpAlqZd2pip-wheel- --python-tag cp27:
  running bdist_wheel
  running build
  running build_py
  creating build
  creating build/lib.macosx-10.5-x86_64-2.7
  creating build/lib.macosx-10.5-x86_64-2.7/feather
  copying feather/__init__.py -> build/lib.macosx-10.5-x86_64-2.7/feather
  copying feather/api.py -> build/lib.macosx-10.5-x86_64-2.7/feather
  copying feather/compat.py -> build/lib.macosx-10.5-x86_64-2.7/feather
  copying feather/version.py -> build/lib.macosx-10.5-x86_64-2.7/feather
  creating build/lib.macosx-10.5-x86_64-2.7/feather/tests
  copying feather/tests/__init__.py -> build/lib.macosx-10.5-x86_64-2.7/feather/tests
  copying feather/tests/test_reader.py -> build/lib.macosx-10.5-x86_64-2.7/feather/tests
  copying feather/libfeather.pxd -> build/lib.macosx-10.5-x86_64-2.7/feather
  copying feather/ext.pyx -> build/lib.macosx-10.5-x86_64-2.7/feather
  running build_ext
  building 'feather.ext' extension
  creating build/temp.macosx-10.5-x86_64-2.7
  creating build/temp.macosx-10.5-x86_64-2.7/feather
  creating build/temp.macosx-10.5-x86_64-2.7/src
  creating build/temp.macosx-10.5-x86_64-2.7/src/feather
  gcc -fno-strict-aliasing -I/Users/romanvg/.anaconda/include -arch x86_64 -DNDEBUG -g -fwrapv -O3 -Wall -Wstrict-prototypes -Ifeather -I/Users/romanvg/.anaconda/lib/python2.7/site-packages/numpy/core/include -I/private/var/folders/rb/k3q8j6j53_d5fcns670ntm380000gn/T/pip-build-7NVZlf/feather-format/src -I/Users/romanvg/.anaconda/include/python2.7 -c feather/ext.cpp -o build/temp.macosx-10.5-x86_64-2.7/feather/ext.o -std=c++11 -O3
  In file included from feather/ext.cpp:283:
  In file included from /private/var/folders/rb/k3q8j6j53_d5fcns670ntm380000gn/T/pip-build-7NVZlf/feather-format/src/feather/api.h:18:
  /private/var/folders/rb/k3q8j6j53_d5fcns670ntm380000gn/T/pip-build-7NVZlf/feather-format/src/feather/buffer.h:19:10: fatal error: 'cstdint' file not found
  #include <cstdint>
           ^
  1 error generated.
  error: command 'gcc' failed with exit status 1

  ----------------------------------------
  Failed building wheel for feather-format
  Running setup.py clean for feather-format
Failed to build feather-format
Installing collected packages: feather-format
  Running setup.py install for feather-format ... error
    Complete output from command /Users/romanvg/.anaconda/bin/python -u -c "import setuptools, tokenize;__file__='/private/var/folders/rb/k3q8j6j53_d5fcns670ntm380000gn/T/pip-build-7NVZlf/feather-format/setup.py';exec(compile(getattr(tokenize, 'open', open)(__file__).read().replace('\r\n', '\n'), __file__, 'exec'))" install --record /var/folders/rb/k3q8j6j53_d5fcns670ntm380000gn/T/pip-Obg7S3-record/install-record.txt --single-version-externally-managed --compile:
    Compiling feather/ext.pyx because it changed.
    [1/1] Cythonizing feather/ext.pyx
    running install
    running build
    running build_py
    creating build
    creating build/lib.macosx-10.5-x86_64-2.7
    creating build/lib.macosx-10.5-x86_64-2.7/feather
    copying feather/__init__.py -> build/lib.macosx-10.5-x86_64-2.7/feather
    copying feather/api.py -> build/lib.macosx-10.5-x86_64-2.7/feather
    copying feather/compat.py -> build/lib.macosx-10.5-x86_64-2.7/feather
    copying feather/version.py -> build/lib.macosx-10.5-x86_64-2.7/feather
    creating build/lib.macosx-10.5-x86_64-2.7/feather/tests
    copying feather/tests/__init__.py -> build/lib.macosx-10.5-x86_64-2.7/feather/tests
    copying feather/tests/test_reader.py -> build/lib.macosx-10.5-x86_64-2.7/feather/tests
    copying feather/libfeather.pxd -> build/lib.macosx-10.5-x86_64-2.7/feather
    copying feather/ext.pyx -> build/lib.macosx-10.5-x86_64-2.7/feather
    running build_ext
    building 'feather.ext' extension
    creating build/temp.macosx-10.5-x86_64-2.7
    creating build/temp.macosx-10.5-x86_64-2.7/feather
    creating build/temp.macosx-10.5-x86_64-2.7/src
    creating build/temp.macosx-10.5-x86_64-2.7/src/feather
    gcc -fno-strict-aliasing -I/Users/romanvg/.anaconda/include -arch x86_64 -DNDEBUG -g -fwrapv -O3 -Wall -Wstrict-prototypes -Ifeather -I/Users/romanvg/.anaconda/lib/python2.7/site-packages/numpy/core/include -I/private/var/folders/rb/k3q8j6j53_d5fcns670ntm380000gn/T/pip-build-7NVZlf/feather-format/src -I/Users/romanvg/.anaconda/include/python2.7 -c feather/ext.cpp -o build/temp.macosx-10.5-x86_64-2.7/feather/ext.o -std=c++11 -O3
    In file included from feather/ext.cpp:283:
    In file included from /private/var/folders/rb/k3q8j6j53_d5fcns670ntm380000gn/T/pip-build-7NVZlf/feather-format/src/feather/api.h:18:
    /private/var/folders/rb/k3q8j6j53_d5fcns670ntm380000gn/T/pip-build-7NVZlf/feather-format/src/feather/buffer.h:19:10: fatal error: 'cstdint' file not found
    #include <cstdint>
             ^
    1 error generated.
    error: command 'gcc' failed with exit status 1

    ----------------------------------------
Command "/Users/romanvg/.anaconda/bin/python -u -c "import setuptools, tokenize;__file__='/private/var/folders/rb/k3q8j6j53_d5fcns670ntm380000gn/T/pip-build-7NVZlf/feather-format/setup.py';exec(compile(getattr(tokenize, 'open', open)(__file__).read().replace('\r\n', '\n'), __file__, 'exec'))" install --record /var/folders/rb/k3q8j6j53_d5fcns670ntm380000gn/T/pip-Obg7S3-record/install-record.txt --single-version-externally-managed --compile" failed with error code 1 in /private/var/folders/rb/k3q8j6j53_d5fcns670ntm380000gn/T/pip-build-7NVZlf/feather-format/

For the record, there's someone publishing an earlier version of the package in conda, perhaps it could be worth posting it there too (or in a more "official" repo/community):
https://anaconda.org/jjhelmus/feather-format

Oh dear, just found this hack hidden in the PyPi docs:
export MACOSX_DEPLOYMENT_TARGET=10.10

That fixed it :+1:

Hi, just had this same issue happen to me doing $ pip install feather-format on OS X Sierra. Your hack fixed it, but just wondering if you could provide an explanation of why that worked, for reference. Thanks!


make users aware of the data licence

The emergency number data is licensed under OdBL. That has some restrictions on usage, the most important what happens if you merge it with other data. It's best to mention OdBL in the README.

Yes definitely. I will mention OdBL from here https://github.com/EmergencyNumberAPI/data/blob/master/LICENSE.md and have the gem as MIT. Thanks!

Fixed at https://github.com/shayonj/emergency_number/commit/7d7cddb26ff68ff0309110450692a8833ac92546


[HUDI-4902] Set default partitioner for SIMPLE BUCKET index

Change Logs
ISSUE: https://github.com/apache/hudi/issues/6723
When we use BUCKET index in spark, the default BUCKET_INDEX_ENGINE_TYPE is SIMPLE. While only SparkBucketIndexPartitioner supports the SIMPLE bucket index.
This pr sets default partitioner for SIMPLE BUCKET index
Impact
Describe any public API or user-facing feature change or any performance impact.
Risk level: none | low | medium | high
Choose one. If medium or high, explain what verification was done to mitigate the risks.
Contributor's checklist

[ ] Read through contributor's guide
[ ] Change Logs and Impact were stated clearly
[ ] Adequate tests were added if applicable
[ ] CI passed

@hudi-bot run azure


Forward port feature to see difference when rolling back a workload

When working on forward porting the feature to roll back a workload https://github.com/rancher/dashboard/pull/4178, I tried to also add the diff shown when you roll back a workload in Ember:

I found that in Ember, it looks like an Ember-specific function get is used to clean the data before the two revisions are compared. https://github.com/rancher/ui/blob/master/app/components/modal-rollback-service/component.js#L115
I didn't have enough time to create the diff in Vue, so opened this issue to track the diff portion separately.
@gaktive

get is just the way you access child properties of an object in ember.

On 2.6-head commit id 4375161

Diff in VUE

Diff in EMber

@catherineluse Is this expected?

@sowmyav27 Neill added the ability to see the diff view. Right now it is showing more differences than the Ember UI does. It would take some more work to filter out the less important differences, and that work is being tracked in https://github.com/rancher/dashboard/issues/4636.

Closing this as a separate GH issue is logged to track improvements.


Updated to Trusted Web Activities Quickstart

What's changed, or what was fixed?

Add generic featured image

CC: @petele

:tada: This has been pushed live to https://developers.google.com/web/


<EMAIL_ADDRESS>Use new quarto.exe binary

This PR modies the bin property to use the new quarto.exe binary now included in the bundle. This new exe file solves some issues in some system and is the recommended way to call quarto now.

[x] I have read the Contributing Guide.

/verify

Why not move it to 'main'?

I don't know about the rules between Extras and Main bucket. quarto is CLI only tool if it matters. I would be glad to see it in main


Pad out tooltip content on Services StatusBar

Before:

After:

Reviewer: @aldipower

Sorry I'm too late but I think this should be handled differently. I think we should use the @base-spacing-unit varaible here. Also, instead of adding padding to the top and bottom of each .tooltip-line-item element, we should apply margin to the bottom of those elements, like so:
.tooltip-line-item {
  margin-bottom: @base-spacing-unit / 6; // this might be the wrong denomenator

  &:last-child{
    margin-bottom: 0;
  }

}

Just for future reference. I'll tackle this in the CNVS upgrade.


Magnet and trash will not appear

Hi,
Magnet and Trash utilities will not appear, and I also cannot switch to creative+ gamemode or save inventory.
Cheat, utility and recipe modes show the same behaviour.
I am op(4)
Both global and world settings are identical.
Per world:

Per world with no overrides:

Global:

All yield same result:

MC 1.12.2
NEI: 1.12.2-<IP_ADDRESS>
JEI: 1.12.2-<IP_ADDRESS>
Forge: 1.12.2-<IP_ADDRESS>45
All configs for NEI and JEI are default. This result is the same whatever gamemode (survival, adventure, creative) chosen.

Works for me, your server (if any) is likely disabling those things. Check NEI's config on the server.


[Question] Cactbot hasnt updated in weeks?

I know the way the plugin worked has chanegd and I dont really understand it, but it feels like I havent got an udpate request for it in weeks? It used to udpate quite frequently? Also it feels like O12S stuff doesnt tell you much?

Last update was 17 days ago.
For old stuff you need to run the content a lot of times (with a full group in best case) and check for loops, skips etc which takes a long itme and sometimes the current creators need a break as well or just want to enjoy the game.
If you want trigger/timeline than create them (documentation can be found in the docs folder of this repository) as PRs are always welcome :) .
Also o12s has all important triggers in it (everything also translated into de, fr, ja, cn, ko). What exactly are you missing in o12s?

Aaah Thanks and sorry i meant E12S

For e12s P2 (afaik) some stuff is not called out due to different ways of playing it.
Usually cactbot tries to avoid "forcing" you into one way of playing.
The best place to ask is the default ACT Discord from Ravahn (also best place to ask for overlay plugin issue)

Oh yeha but soemthing like short/long fire/ice  wouldnt force you to do anything just inform.

Re: update frequency.  There's a lot of updates when content is new, but then it slows down over time.
If there's stuff missing that you would like, filing github issues or asking in discord is a good way to do it.
Sorry that there's no short/long ice/fire callout.  I'm a tank and I forgot about that one.

Aaah cheers would be amazing if its doable =)


Allowing multiple users to access the app

Does anyone knows how to work with the PHP API to allow multiple google users to access different folders/paths without interfering with the authentication?
Any response, ANY will be most appreciated.

This question is quite confusing to me. You can handle multiple users by maintaining multiple sessions (like any other app), and if you wanted to restrict different folders/paths in your application, you'd have to implement your own ACL or use a framework ACL for that.
Can you provide a little bit more on what you're trying to accomplish?

Put it this way:
I have this app that I want multiple users from the SAME browser to access it. I'm talking:  I want users from gmail to access my app. Currently, only the 1st user of Google/Gmail (the main user) is allowed to access my app. If I want another user, I have to trigger UserService::createLoginURL($_SERVER['REQUEST_URI']); if I want another user to access it. Most frustrating is that I can't have multiple users from the same browser accessing my app simultaneusly.
Is there a way I can overcome this? I used to play with the SACSID cookies so I could define one per user per path. But now, all of the sudden this week, that stopped from working... is like if the session manager is not reading any SACSID cookie that is not "root".

Basically, what I want is to mimick the google ?authuser=1 behavior. Any thoughts?

Ok, I was able to do this by:

Asking authorization to each user attempting to access the folders
Storing the user information into $_SESSION (into an array)
Comparing such array data to the user attepmting to access the path: if it was not authorized to that path, then prompts for authorization.

So far so good. I'm back in business.

Glad to hear it!


Windows Desktop App Crashes On Left Arrow Key

A user reported [case/695230] that the app crashes after they type in a description and use the left arrow key to move the cursor in the description field.
The issue is there with running and stopped entries, I've managed to reproduce it: https://screencast-o-matic.com/watch/cFhFXqbgdZ

Toggl Desktop crashes when I use the arrow keys to navigate in the description field. For instance, if I place the cursor at the end of "testing toggl char bug" and press the left arrow key, the program crashes. Right now I can reproduce it with any length of text, with both active and stopped time entries. I was on the beta release channel when I encountered this, and subsequently switched to stable, but that didn't seem to have any effect whatsoever.

User agent: windows_native_app/7.4.157-obm-87-obm-98-obm-99-obm-101
Version: [Latest Beta, released at (15-05-2018)] 7.4.157
UpdateChannel: stable

another case case/695246

I'm able to replicate this. It happens when pressing left while autocomplete dropdown is opened.


Specify Language Version

Is your feature request related to a problem? Please describe.
.Net Interactive is the perfect code playground, but being tied to a single language version/compiler is limiting.
Describe the solution you'd like
I would like to be able to explore new C# language features by specifying various language versions, like is possible with sharplab.io. Particularly cool would be "preview" compiler changes. Think this ask is related to, but not necessarily the same as, specifying the .Net version, which is here.
Describe alternatives you've considered
Alternative is to use sharplab.io or to specify the LangVersion property in a Visual Studio/VS Code C# project.

I just ran into the same problem. I tried running an F# notebook, and for some mysterious reason it installed fsharp.core, 4.7.0. It would be great if I could somehow see why, my system installation is dotnet 5.


Add inference time of models

Regarding Keras issue #13619
Hi, thank you for your great work.
What is problem?
In the Keras application document, It describes model's size, accuracy, etc.
When we use the model for our real world project, inference time is most important feature like 'accuracy'.
Of course, inference time generally tend to follow number of parameters. But these are not perfectly proportional.
So, checking the inference time of models will be useful when we search model for our project.
Method
As the inference time depends on the spec of machine, we can use fixed and general environment for the checking
(I used google-colab env. Please see test code)
Result

Previous document

Updated document

Thank you.

@googlebot I signed it!

Thanks for the PR! This is indeed very useful information to add. I would recommend:

Define it as "Time per inference step (in ms)"
Make sure you're optimizing inference speed by using a prefetched tf.data.Dataset
Compute it for both CPU-only and single-GPU, because many people do inference on CPU

In particular, you're currently using result = %timeit -n 10 -r 30 -o model.predict(x). I recommend doing instead:
input_shape = tuple(model.input.shape[1:])
batch_size = 32
n = 20
first_batch = np.random.random((batch_size,) + input_shape).astype('float32')
full_array = np.random.random((batch_size * n,) + input_shape).astype('float32')
ds = tf.data.Dataset.from_tensor_slices(full_array).batch(batch_size).prefetch(4)

### Warmup model
model.predict(first_batch)

result = %timeit -o model.predict(ds)
result /= n

(*untested code, may need touch ups)

@fchollet Thank you for your comment.
I changed my code and added CPU benchmark based on your example.
(GPU test code. CPU test code)
And changed document is like below.

...


Unable to debug functions locally with FunctionsEnableWorkerIndexing (Worker.Sdk >= 1.16.0)

Unable to debug functions locally in Microsoft.Azure.Functions.Worker.Sdk v 1.16.0, 1.16.1 and 1.16.2 (when FunctionsEnableWorkerIndexing started to default to True)
After starting debugging  the console window opens and hangs and displays:

A debugger was not attached within the expected time limit. The process will continue without a debugger.
Worker process started and initialized.

After these messages, the normal output (function metadata) is displayed.
What works:
Rolling back to 1.15.1
Running 1.16.x with <FunctionsEnableWorkerIndexing>False</FunctionsEnableWorkerIndexing>
Setup

macOS (arm, M2 Max)
func: 4.0.5455
vscode: 1.84.2
.Net 8.0.100

Related issues

#2070
#1818
#2062

Thank you. Was looking to log a bug for this.
I can confirm that both workarounds work for me as well.

@colbylwilliams would you be able to provide a repo so we can take a closer look at your setup? We're trying to repro this in a similar environment and it all seems to work as expected.
Can you please also share the output from func --version returned in the terminal?
Thanks!

@fabiocav is there any way to get more output during func start?

I was facing the same issue today on my macos + vs code, none of the functions would show up in the debugger right after the version upgrade in csproj file. It was working when published through. I upgraded all the packages to its latest version to no avail. After I figured this library when downgraded solves the issue I did the following even though I had installed this version earlier, just to make sure.
brew tap azure/functions
brew install azure-functions-core-tools@4
Now it is working.

Hello, please see the following repo for reproducing the M2 Pro Mac issue. Open the Solution and run the TestWebApp.Functions project with debugging.

I still can't debug locally unless FunctionsEnableWorkerIndexing is set to false.  @fabiocav can we reopen this issue?  @LloydNicholson provided a repro.

@fabiocav and team, 7 days inactivity threshold for marking issues stale for inactivity seems a little aggressive.  @LloydNicholson provided a repro.

Although I believe this to since have been resolved, re-opening for the moment so it can be closed explicitly if that is indeed correct. The bot seems to have not done the right thing in terms of presenting the options to keep closure from occurring.

Just confirmed this issue still exists in version 1.17.0

Just confirmed this issue still exists in version 1.17.0

Also exists in 1.17.2

Still persists with 1.22.0.
If I add "AzureWebJobsFeatureFlags": "EnableWorkerIndexing" as instructed in #2070, my local process hangs. Verbose log:
[2024-07-27T03:21:07.550Z] Initializing Warmup Extension.
[2024-07-27T03:21:07.577Z] Initializing Host. OperationId: '5d4d429f-b9f8-4800-ab19-ccdb36549059'.
[2024-07-27T03:21:07.582Z] Host initialization: ConsecutiveErrors=0, StartupCount=1, OperationId=5d4d429f-b9f8-4800-ab19-ccdb36549059
[2024-07-27T03:21:07.612Z] Loading functions metadata
[2024-07-27T03:21:07.614Z] Worker indexing is enabled
[2024-07-27T03:21:07.617Z] Fetching metadata for workerRuntime: dotnet-isolated
[2024-07-27T03:21:07.618Z] Reading functions metadata

@cathei Which version of the core tools are you currently utilizing? The version number should be displayed at the top of your terminal window. With the most recent stable release of the core tools, the issue you’re encountering is unexpected.

@colbylwilliams Let us know if you are still experiencing the issue with the latest core tools.

Hello @kshyju. I tested again and I think the issue has been fixed after I upgraded my VS (17.7.7 -> 17.10.5). Assuming core tools are tied with it? Current version is:
Azure Functions Core Tools
Core Tools Version:       4.0.5907 Commit hash: N/A +807e89766a92b14fd07b9f0bc2bea1d8777ab209 (64-bit)
Function Runtime Version: 4.834.3.22875


Midnight theme text box breaks with a extended sentence

It happens in any size I put discord in too.

Aware of this, will be adressed in future updates


fix: changed prop drilled fn to class fn, avoided array prop

Description
The list of fixes added in this PR :

If using a class component try creating class function rather than creating an arrow functions inside render method which creates a new reference every time the parent component is re-rendered.
Try avoiding to pass the formatted array via props until the formatted array does not creates a new reference every time the parent component is re-rendered.

Result:
The QueryEditor Component was being re-rendered 17 times which made EditorJSONtoForm Component re-render 30-34 times.
After the change the QueryEditor Component is still being re-rendered 17 times but the EditorJSONtoForm Component's re-rendering has been limited to 17 times, meaning once every time the parent re-renders.
Fixes #12787
Type of change

Please delete options that are not relevant.

Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?
Manually
Checklist:

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes

Test coverage results :test_tube:
:white_circle: Total coverage has not changed
// Code coverage diff between base branch:release and head branch: fix/perf-editor-jsontoform

Status
File
% Stmts
% Branch
% Funcs
% Lines

:red_circle:
app/client/src/pages/Editor/DataSourceEditor/index.tsx
30.21 (-0.32)
0 (0)
18.18 (0)
30.34 (0)

:red_circle:
app/client/src/pages/Editor/QueryEditor/EditorJSONtoForm.tsx
34.38 (-0.17)
25.77 (0)
0 (0)
35.81 (-0.04)

:green_circle:
app/client/src/pages/Editor/QueryEditor/index.tsx
28.03 (0.42)
5.8 (0)
5.56 (0.3)
29.51 (0.24)

:green_circle:
app/client/src/sagas/DatasourcesSagas.ts
13.98 (0.08)
2.03 (0.03)
8.33 (0)
16.42 (0.06)

:green_circle:
app/client/src/selectors/commentsSelectors.ts
85.25 (1.64)
64.71 (2.95)
73.33 (0)
90.59 (2.35)

:red_circle:
app/client/src/utils/autocomplete/TernServer.ts
52.71 (-0.23)
40.83 (-0.84)
36.21 (0)
56.74 (-0.25)

/ok-to-test sha=908f758

/ok-to-test sha=1baa417


Pass challenge objects to responders

Instead of challenge body objects.
Fixes #83

This change is

Current coverage is 99.94% (diff: 100%)

Merging #84 into master will not change coverage

@@             master        #84   diff @@
==========================================
  Files            25         25
  Lines          1897       1897
  Methods           0          0
  Messages          0          0
  Branches        171        171
==========================================
  Hits           1896       1896
  Misses            1          1
  Partials          0          0

Powered by Codecov. Last update d6d6a90...72fd37b

Reviewed 2 of 2 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


Complete website

Day 3 submission
In this pull request we have

Styled index.html page with sketchy bootstrap and CSS
Used JS to show and hide contents

Hi Robert,
I have seen and implemented the required changes as described in your
comments. And I successfully submitted the pull request again at 8:30 AM
this morning (utc+3). Later my dashboard said that I could successfully
submit my project. Since I only checked my dashboard, I didn't notice there
was no 2nd code reviewer who declared the project was approved and sent me
an email. This is why my partner also faces an issue with only seeing
'request review' in his dashboard. Even submitting the pull request again.
In short, we did ot receive a second code review.
Me and my partners pull request link for convenience

https://github.com/Yordinia/Awesome-Book/pull/4
Thank You.

On Thu, Dec 15, 2022 at 7:10 AM Robert Loterh @.***>
wrote:

@.**** requested changes on this pull request.
Hi Kamran & Yordi,
You did extremely great on this project! 👏 👏
There are however a few issues that you still need to work on to go to the
next project but you are almost there! 🚦 🚦 🚦
Highlights 👍

SPA feature properly implemented
App is running as intended

Required Changes ♻
Kindly check the inline comments under the review.
Optional suggestions
Every comment with the [OPTIONAL] prefix is not crucial enough to stop
the approval of this PR. However, I strongly recommend you take them into
account as they can make your code better.
Kindly check the inline comments marked [OPTIONAL] under the review.
You can also consider:

Live demo link in your ReadMe is broken, kindly endeavor to fix it.

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.
Please, do not open a new Pull Request for re-reviews. You should use the
same Pull Request submitted for the first review, either valid or invalid
unless it is requested otherwise.
As described in the Code reviews limits policy
https://microverse.zendesk.com/hc/en-us/articles/1500004088561 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 https://airtable.com/shrQAqnBwek5a0O0s.
In stylesheet.css
https://github.com/Yordinia/Awesome-Book/pull/4#discussion_r1049194898:

+footer {

margin: 1rem;

+}
Kindly improve upon the following aspects of your design:

The footer should always stick to the bottom of the page
The add button should align to the right within the form
OPTIONAL
We want to enhance the user experience of this application, and
therefore kindly ensure that an active menu link is highlighted(color
change or change in opacity). In other words, the current active page
should have the corresponding menu item highlighted, as seen in the
provided wireframe design
https://github.com/microverseinc/curriculum-javascript/blob/main/books/m4_full_website.md.
Kindly ensure to improve upon this aspect in your design, as we really want
our designs to mimic the provided wireframe template.

Your design: The original design:
[image: image]
https://user-images.githubusercontent.com/12745474/207770149-32cf3037-d68c-48e2-b0e7-79d979d11379.png [image:
image]
https://user-images.githubusercontent.com/12745474/143277255-658fcad7-519b-445d-b039-21a2187c6968.png
—
Reply to this email directly, view it on GitHub
https://github.com/Yordinia/Awesome-Book/pull/4#pullrequestreview-1218556937,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASBJHHCA7HGVEXOZRVZK3FLWNKK35ANCNFSM6AAAAAAS6K53UA
.
You are receiving this because you authored the thread.Message ID:
@.***>

Hi @Yordinia
I'll get back on the same soon!
Cheers!

@Yordinia
It's probably the next reviewer missed a step in the process reviewing your project.
I suggest you report this issue via the Zendesk so that problem regarding unable to change the update your project status on the dashboard is resolved the sooner.
Cheers! 😃


after updating dapr to version 1.0.1, pubsub example does not work any more

Expected Behavior
https://github.com/dapr/python-sdk/tree/master/examples/pubsub-simple works without reporting any errors
Actual Behavior
subscriber can not receive any messages:
> dapr run --app-id python-subscriber --app-protocol grpc --app-port 50051 --log-level debug  python3 subscriber.py
ℹ  Starting Dapr with id python-subscriber. HTTP Port: 40589. gRPC Port: 46343
INFO[0000] starting Dapr Runtime -- version 1.0.1 -- commit 45bc40d  app_id=python-subscriber instance=ryzen5 scope=dapr.runtime type=log ver=1.0.1
INFO[0000] log level set to: debug                       app_id=python-subscriber instance=ryzen5 scope=dapr.runtime type=log ver=1.0.1
INFO[0000] metrics server started on :44781/             app_id=python-subscriber instance=ryzen5 scope=dapr.metrics type=log ver=1.0.1
INFO[0000] standalone mode configured                    app_id=python-subscriber instance=ryzen5 scope=dapr.runtime type=log ver=1.0.1
INFO[0000] app id: python-subscriber                     app_id=python-subscriber instance=ryzen5 scope=dapr.runtime type=log ver=1.0.1
INFO[0000] mTLS is disabled. Skipping certificate request and tls validation  app_id=python-subscriber instance=ryzen5 scope=dapr.runtime type=log ver=1.0.1
INFO[0000] local service entry announced: python-subscriber -> <IP_ADDRESS>:43449  app_id=python-subscriber instance=ryzen5 scope=dapr.contrib type=log ver=1.0.1
INFO[0000] Initialized name resolution to standalone     app_id=python-subscriber instance=ryzen5 scope=dapr.runtime type=log ver=1.0.1
DEBU[0000] loading component. name: pubsub, type: pubsub.redis/  app_id=python-subscriber instance=ryzen5 scope=dapr.runtime type=log ver=1.0.1
INFO[0000] component loaded. name: pubsub, type: pubsub.redis/  app_id=python-subscriber instance=ryzen5 scope=dapr.runtime type=log ver=1.0.1
DEBU[0000] loading component. name: statestore, type: state.redis/  app_id=python-subscriber instance=ryzen5 scope=dapr.runtime type=log ver=1.0.1
INFO[0000] component loaded. name: statestore, type: state.redis/  app_id=python-subscriber instance=ryzen5 scope=dapr.runtime type=log ver=1.0.1
DEBU[0000] loading component. name: zipkin, type: exporters.zipkin/  app_id=python-subscriber instance=ryzen5 scope=dapr.runtime type=log ver=1.0.1
INFO[0000] waiting for all outstanding components to be processed  app_id=python-subscriber instance=ryzen5 scope=dapr.runtime type=log ver=1.0.1
FATA[0000] process component zipkin error: incorrect type exporters.zipkin  app_id=python-subscriber instance=ryzen5 scope=dapr.runtime type=log ver=1.0.1
⚠  Could not update sidecar metadata for cliPID: PUT http://<IP_ADDRESS>:40589/v1.0/metadata/cliPID giving up after 5 attempts
ℹ  Updating metadata for app command: python3 subscriber.py
⚠  Could not update sidecar metadata for appCommand: PUT http://<IP_ADDRESS>:40589/v1.0/metadata/appCommand giving up after 5 attempt

publisher report errors:
> dapr run --app-id python-publisher --app-protocol grpc --dapr-grpc-port=3500 --log-level debug python3 publisher.py
ℹ  Starting Dapr with id python-publisher. HTTP Port: 43039. gRPC Port: 3500
ℹ  Checking if Dapr sidecar is listening on HTTP port 43039
INFO[0000] starting Dapr Runtime -- version 1.0.1 -- commit 45bc40d  app_id=python-publisher instance=ryzen5 scope=dapr.runtime type=log ver=1.0.1
INFO[0000] log level set to: debug                       app_id=python-publisher instance=ryzen5 scope=dapr.runtime type=log ver=1.0.1
INFO[0000] metrics server started on :38619/             app_id=python-publisher instance=ryzen5 scope=dapr.metrics type=log ver=1.0.1
INFO[0000] standalone mode configured                    app_id=python-publisher instance=ryzen5 scope=dapr.runtime type=log ver=1.0.1
INFO[0000] app id: python-publisher                      app_id=python-publisher instance=ryzen5 scope=dapr.runtime type=log ver=1.0.1
INFO[0000] mTLS is disabled. Skipping certificate request and tls validation  app_id=python-publisher instance=ryzen5 scope=dapr.runtime type=log ver=1.0.1
INFO[0000] local service entry announced: python-publisher -> <IP_ADDRESS>:41515  app_id=python-publisher instance=ryzen5 scope=dapr.contrib type=log ver=1.0.1
INFO[0000] Initialized name resolution to standalone     app_id=python-publisher instance=ryzen5 scope=dapr.runtime type=log ver=1.0.1
DEBU[0000] loading component. name: pubsub, type: pubsub.redis/  app_id=python-publisher instance=ryzen5 scope=dapr.runtime type=log ver=1.0.1
INFO[0000] component loaded. name: pubsub, type: pubsub.redis/  app_id=python-publisher instance=ryzen5 scope=dapr.runtime type=log ver=1.0.1
DEBU[0000] loading component. name: statestore, type: state.redis/  app_id=python-publisher instance=ryzen5 scope=dapr.runtime type=log ver=1.0.1
INFO[0000] component loaded. name: statestore, type: state.redis/  app_id=python-publisher instance=ryzen5 scope=dapr.runtime type=log ver=1.0.1
DEBU[0000] loading component. name: zipkin, type: exporters.zipkin/  app_id=python-publisher instance=ryzen5 scope=dapr.runtime type=log ver=1.0.1
FATA[0000] process component zipkin error: incorrect type exporters.zipkin  app_id=python-publisher instance=ryzen5 scope=dapr.runtime type=log ver=1.0.1
⚠  Dapr sidecar is not listening on HTTP port: dial tcp <IP_ADDRESS>:43039: connect: connection refused
ℹ  Checking if Dapr sidecar is listening on GRPC port 3500
⚠  Dapr sidecar is not listening on GRPC port: dial tcp <IP_ADDRESS>:3500: connect: connection refused
⚠  Dapr sidecar might not be responding.
== APP == Traceback (most recent call last):

== APP ==   File "/data/tools/dapr/python-sdk/examples/pubsub-simple/publisher.py", line 21, in <module>

== APP ==     resp = d.publish_event(

== APP == TypeError: publish_event() got an unexpected keyword argument 'topic_name'

⚠  Could not update sidecar metadata for cliPID: PUT http://<IP_ADDRESS>:43039/v1.0/metadata/cliPID giving up after 5 attempts
ℹ  Updating metadata for app command: python3 publisher.py
⚠  Could not update sidecar metadata for appCommand: PUT http://<IP_ADDRESS>:43039/v1.0/metadata/appCommand giving up after 5 attempts

Steps to Reproduce the Problem
After start dapr 1.0.1 with dapr init. Then follow the readme in the example folder.

state store example also does not work any more for me:
⚠  Dapr sidecar is not listening on HTTP port: dial tcp <IP_ADDRESS>:44785: connect: connection refused
ℹ  Checking if Dapr sidecar is listening on GRPC port 45685
⚠  Dapr sidecar is not listening on GRPC port: dial tcp <IP_ADDRESS>:45685: connect: connection refused
⚠  Dapr sidecar might not be responding.
== APP == Traceback (most recent call last):

== APP ==   File "/data/tools/dapr/python-sdk/examples/state_store/state_store.py", line 27, in <module>

== APP ==     d.wait(5)

== APP ==   File "/data/tools/dapr/python-sdk/examples/pubsub-simple/.venv/lib/python3.9/site-packages/dapr/clients/grpc/client.py", line 810, in wait

== APP ==     raise e

== APP ==   File "/data/tools/dapr/python-sdk/examples/pubsub-simple/.venv/lib/python3.9/site-packages/dapr/clients/grpc/client.py", line 805, in wait

== APP ==     s.connect(host_port)

== APP == ConnectionRefusedError: [Errno 111] Connection refused

@callalilychen Not a fix: but for the pubsub example downgrade your runtime version to version 1 and upgrade the python libraries via pip if you have not. That worked for me.

@Ilozuluchris how to downgrading dapr runtime? I downgraded dapr cli, and try dapr init. The newest dapr runtime is then installed.

@callalilychen  Use dapr init --runtime-version runtime_version_number, where runtime_version_number is the version number you want to use, you might need to uninstall the current runtime first with dapr uninstall. You can find runtime versions here

Which version of the python-sdk are you currently using? Using dapr runtime v1.0.1 requires python-sdk >= 1.0.0:
pip show dapr
Name: dapr
Version: 1.0.0
Summary: The official release of Dapr Python SDK.
Home-page: https://dapr.io/
Author: Dapr Authors
Author-email<EMAIL_ADDRESS>License: MIT
Location: /usr/local/lib/python3.9/site-packages
Requires: protobuf, python-dateutil, grpcio, aiohttp
Required-by: dapr-ext-grpc, dapr-ext-fastapi

After I downgraded my dapr runtime to 1.0.0, I still facing the same problem.
And checked dapr python sdk and also dapr-ext-grpc version. It shows also v1.0.0:
pip show dapr
Name: dapr
Version: 1.0.0
Summary: The official release of Dapr Python SDK.
Home-page: https://dapr.io/
Author: Dapr Authors
Author-email<EMAIL_ADDRESS>License: MIT
Location: /data/tools/kuksa.val/kuksa_apps/dapr/pubsub/.venv/lib/python3.9/site-packages
Requires: protobuf, aiohttp, python-dateutil, grpcio
Required-by: dapr-ext-grpc

I tried the whole thing with my laptop, which installed the same OS like my pc. On my laptop, everything works fine. But on my pc, It worked with runtime 0.8.0, not any more with version >= 1.0.0. Maybe there is issues with ryzen CPU? On my PC, ryzen5 is installed. And I also see ryzen5 in the error message.

tested and it works, thanks very much :+1:


[docs] [cmake] switch to 'cmake --build'?

Description
Many CI scripts and docs throughout this project use a pattern like this for CMake:
https://github.com/dmlc/xgboost/blob/43704549a2746fcf134a89b3153502a02a94a295/doc/build.rst?plain=1#L130-L133
Would you support a series of PRs (or maybe just one, depending on how involved it gets) changing all such cases to this pattern?
cmake -B build -S . -DUSE_CUDA=ON
cmake --build build --target all -j4

Benefits of this work

reduce friction in building

(2 commands instead of 4)

makes it easier for users to change generators

(e.g. to Ninja instead of GNU Make)

Approach
Replace all uses of mkdir build && cd build && cmake .. or similar with the corresponding 2 cmake commands (cmake followed by cmake --build).
Notes
I did this in LightGBM a few months ago. We haven't received any complaints about it, and I've personally found it a bit easier to work with: https://github.com/microsoft/LightGBM/pull/6368
But won't be offended if you say "no thank you, we don't want this" 😅

Thank you for the suggestions, sounds good!


feat: support line number in fragment in MD051/link-fragments (e.g: '#L7')

Hello! :smile:
I'm the original author of the rule MD051, work started in this PR: https://github.com/DavidAnson/markdownlint/pull/495.
I'm also the author of https://github.com/theoludwig/markdownlint-rule-relative-links custom rule.
This custom rule, verify if relative links exists in the file system using Node.js node:fs, it also checks for links fragments referencing other files (similar to the rule MD051, by trying to have as much as possible the same behavior as the MD051 built-in rule in this regard).
GitHub supports links fragments for referring to a specific line, e.g:
[Link](#L7)

#L7 means "Go To Line 7`.
More context: https://github.com/theoludwig/markdownlint-rule-relative-links/issues/6
Currently MD051 reports an error for that kind of fragments, but it should not, as they are valid on GitHub.
This PR fixes the issue, similarly to how it is done in markdownlint-rule-relative-links.
In the custom rule, we can go even further and check if the file has enough lines to be a valid line number reference (e.g: #L7 in a empty file, is invalid, as line 7 doesn't exist), but that needs access to Node.js node:fs, which as far as I understand, you don't want to use for built-in rule for compatibility with the browser. So for now we're just considering it's valid without extra checks.
Feel free to suggest ideas, suggest better implementation, or maybe you have ideas of more test cases to add?
Thanks!

Test Repos seems failing by unrelated changes. As the errors are:
test-repos/v8-v8-dev/src/features/import-attributes.md: 59: MD034/no-bare-urls Bare URL used [Context: "https://chromestatus.com/featu..."]␊
  - test-repos/v8-v8-dev/src/features/import-attributes.md: 61: MD034/no-bare-urls Bare URL used [Context: "https://developer.apple.com/do..."]␊
  - test-repos/v8-v8-dev/src/features/import-attributes.md: 63: MD034/no-bare-urls Bare URL used [Context: "https://babeljs.io/blog/2023/0..."]␊

Which MD034 is not updated with this PR.
I've updated the snaptshots in this commit: d19f30a (#1117)

Cool, thanks, I'll start a review in a moment!
A couple of things:

Is the GitHub behavior documented anywhere? It may be nice to mention this scenario in the docs for this rule and link to that page.
Thanks for respecting the desire to work on the web and not use node:fs. :)

I fixed the test-repos issue. That specific test is sensitive because it pulls the latest from external repos and tests against them, failing if any violations have changed. This is deliberate on my part, so random failures in that test are a feature, not a bug.

Is the GitHub behavior documented anywhere? It may be nice to mention this scenario in the docs for this rule and link to that page.

Yes. It is documented here: https://docs.github.com/en/get-started/writing-on-github/working-with-advanced-formatting/creating-a-permanent-link-to-a-code-snippet#linking-to-markdown
I've added the explanation in md051.md. :smile:

The GitHub "specification" is pretty light on detail, but it was enough to remind me this syntax is more capable than just a single line number. For example, the following represents the full form I'm aware of which links to starting line/column and ending line/column: https://github.com/DavidAnson/markdownlint/blob/9c77f92616c063a5d42f165cae2edb9e3418ac53/CHANGELOG.md?plain=1#L30C11-L31C9
So the regular expression should be something more like ^L\d+(?:C\d+)?(?:-L\d+(?:C\d+)?)?$. I'll look at the rest of the changes now, but this probably means we should update the code, tests, and docs.

@DavidAnson Thank you for reviewing the PR. :smile:
Sorry for the delay.
I should have addressed most feedbacks from reviews, notably:

Updated the documentation, to specify that the rule mandates the use of lowercase for link fragments for headings.
Simplified the code, as we don't support anymore uppercase/lowercase, e.g: [Link](#L7abc), it should be [Link](#l7abc) with a valid heading.
Support specifying columns (e.g: [Valid](#L30C11-L31C9)).

This looks good, thank you! I think I want to iterate on the documentation a little after thinking about it some more - I'll do that tomorrow by pushing to this branch and then accept the PR.

For posterity since I tried the combinations out, these are all the possibilities and how GitHub handles them:
HIGHLIGHT
#L30C11-L31C9
#L30C11-L31
#L30-L31
#L30-L31C9
#L30
NO HIGHLIGHT
#L30C11-C9
#C11-L31C9
#L30C11
IGNORED
#C11
#C11-C9

... and this regular expression matches ONLY the top group above: ^(?:#L\d+(?:C\d+)?-L\d+(?:C\d+)?)|(?:#L\d+)$


Fail operation with 503 in hub region

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


nodejs error

got this error when running the action/workflow
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/checkout@v3
maybe need to use v4?

Yes, please use v4 and report back. I'll update the README accordingly based on your test result. :)

hey thanks for the quick reply,
after using v4,
I got an error with exit code 128.
no detail was given.

after using v4, I got an error with exit code 128.
no detail was given.

That is strange. How did you modify your workflow file? Note that the actions/checkout usage is officially documented here.

im very new to github actions,
so its very possible its my own mistake,
but here is my current .yml:
Simple workflow for deploying static content to GitHub Pages
name: Deploy Public Folder with GithubPageOverwriter
on:
Runs on pushes targeting the default branch
push:
branches: ["main"]
Allows you to run this workflow manually from the Actions tab
workflow_dispatch:
jobs:
publish:
runs-on: ubuntu-latest
steps:
- name: Check out
uses: actions/checkout@v4
  - name: Publish current workdir (which contains generated content) to GitHub Pages
    uses<EMAIL_ADDRESS>
    with:

      # Optional. Default value "." means the root directory of your project will be published.
      # You can use whatever directory your project uses, for example "wwwroot".
      # Such a directory does *not* have to already exist in your repo,
      # it could be an output directory created dynamically by your static website builder.
      source-directory: ./public

      # Optional. Default value "gh-pages".
      # It specifies the temporary branch which hosts the static website.
      # Each build will OVERWRITE this branch.
      target-branch: gh-pages

OK, so you are no longer seeing the initial "Node.js 16 ..." error. Then, your "actions/checkout@v4" change was good.
The "exit code 128" is an irrelevant error. You may try change your "./public" to "public" and retry.

thanks for the info,
after modifying the line to : source-directory: public
I am still getting the exit code 128 error.
but using v4 was required to get rid of nodejs error for sure.

You are hitting this error. Honestly I am not sure where it came from. Can you double check whether you enabled github pages for your project already? That is like a prerequisite. You may refer to the "2. In your Github Pages setting, ..." paragraph in GPO's readme.

@rayluo hey thanks for pointing that out,
turns out I need to
Check if "Read and write permissions" are enabled in Settings -> Actions -> General -> Workflow permissions:

Thanks for sharing. Good to know. Presumably, adding the "permissions" and "contents: ..." lines as below might be able to also get it working WITHOUT checking that "Read and write permissions" setting. You may give it a shot. :-)
jobs:
  publish:
    runs-on: ubuntu-latest
    permissions:
      contents: write
    steps:
      - name: Check out
        # You may use a newer version of https://github.com/actions/checkout
        uses: actions/checkout@v4
        ......


API for "search" in collection querying must have changed at some point

Trying to replicate the example in the README.md, I found that listing records of a collection with a search criteria is no longer working. Specifically, the following code returns all records, rather than just the ones containing "Bob":
for row in cv.collection.get_rows(search="Bob"):     print("We estimate the value of '{}' at {}".format(row.name, row.estimated_value))
Digging through requests in Chrome I found that the parameter name for a search query has changed from "query" to "searchQuery". I can fix this manually by changing line 325 in store.py: https://github.com/jamalex/notion-py/blob/56b7a904474619cf60c4768db435c921ca18f44f/notion/store.py#L325
to
"searchQuery": search,
I hope I have provided enough information for a fix. Thanks for the great work making a notion API available in Python!

Thanks! Changing to "searchQuery" fixed search issue

I've noticed filters API changed, too — I could work on a pull request on the next weekend.