Adding bpf tools to Kepler base image to debug runtime issues

What would you like to be added?
When bpf programs run into issues, it is hard to debug without tools such as bpftool perf bpftrace. We need a debug image to have these tools to help us debug what is going on the system
Why is this needed?
Debugging bpf issues without these tools is very challenging.

It is possible to enter in the container and install the tool?

It is possible to enter in the container and install the tool?

Same question as the base image is from docker.io/nvidia/cuda:12.1.1-base-ubi9 is it possible?

no, once in the container, the filesystem is read only, no pkg can be installed.

once in the container, the filesystem is read only, no pkg can be installed.

this is different to my thought, you are saying something like CoreOS ? otherwise general container can install software

seems I can do install on the pod (even though there is no repo)
or we need follow here to do a new image?
https://kubernetes.io/docs/tasks/debug/debug-application/debug-running-pod/#ephemeral-container
[root@kepler-exporter-kvf82 /]# yum install bpftool
Updating Subscription Management repositories.
Unable to read consumer identity

This system is not registered with an entitlement server. You can use subscription-manager to register.

cuda                                                                                11 kB/s | 3.5 kB     00:00
cuda                                                                                387 kB/s | 982 kB     00:02
Extra Packages for Enterprise Linux 9 - x86_64                                                                                12 kB/s |  30 kB     00:02
Extra Packages for Enterprise Linux 9 - x86_64                                                                                6.7 MB/s |  19 MB     00:02
Red Hat Universal Base Image 9 (RPMs) - BaseOS                                                                                11 kB/s | 3.8 kB     00:00
Red Hat Universal Base Image 9 (RPMs) - BaseOS                                                                                752 kB/s | 510 kB     00:00
Red Hat Universal Base Image 9 (RPMs) - AppStream                                                                                19 kB/s | 4.2 kB     00:00
Red Hat Universal Base Image 9 (RPMs) - AppStream                                                                                2.5 MB/s | 1.7 MB     00:00
Red Hat Universal Base Image 9 (RPMs) - CodeReady Builder                                                                                14 kB/s | 4.2 kB     00:00
Red Hat Universal Base Image 9 (RPMs) - CodeReady Builder                                                                                276 kB/s | 187 kB     00:00
No match for argument: bpftool
Error: Unable to find a match: bpftool

@rootfs
Dockerfile Update
I've made the following changes to the Dockerfile:

Added CentOS Stream 9 repository.
Installed bpftool.
Installed perf.
Installed bpftrace.

Build Error
I encountered an error while building the dockerfile.base image from the repository. The issue arises during the installation of bcc, and here is the error log:
11.91 Error:
11.91  Problem: conflicting requests
11.91   - nothing provides libLLVM-15.so()(64bit) needed by bcc-0.26.0-2.el9.x86_64 from @commandline
11.91   - nothing provides libLLVM-15.so(LLVM_15)(64bit) needed by bcc-0.26.0-2.el9.x86_64 from @commandline

Please take a look and verify on your end. If the error persists, I'll fix it. Additionally, I plan to address the issue #1047 related to this.
Thanks!

@Saurabhkr952 Yes, this is the issue mentioned in #1047. We are going to depreciate bcc images, so if you hit issues in bcc, just ignore it and focus on libbpf dockerfiles.

Dockerfile Modification Summary
Additions:

Enabled the CentOS Stream 9 repository.
Installed bpftool, perf, and bpftrace.

Removals:

Removed bcc and bcc-devel.

Before proceeding, please thoroughly review the changes and kindly confirm the following:

Are there any other modifications that need to be made?
Are there any additional packages that should be installed or removed?

If everything looks good, I'll proceed with creating the pull request.
Thanks

@Saurabhkr952 please update https://github.com/sustainable-computing-io/kepler/blob/main/build/Dockerfile.libbpf.base first . This is the libbpf image base, it doesn't have bcc rpms.


[BUG] Lower case HTTP method resulting in 400

Describe the bug

Lower case HTTP method (e.g. 'get') results in 400 Bad Request.

To Reproduce
import asyncio
from curl_cffi.requests import AsyncSession

async def main():
    session = AsyncSession()
    response = await session.request('get', 'https://httpbin.org/get')
    print(f'status_code: {response.status_code}')
    print(f'content: {response.content[:1000].decode()}')

if __name__ == '__main__':
    asyncio.run(main())

Resulting in:
status_code: 400
content:
<title>400 Bad Request</title>

400 Bad Request

Expected behavior

It could work or at least let me know, that the method has to be uppercase.

Versions

curl_cffi version: 0.7.1

Only upper case is supported, as specified in the RFC.

Only upper case is supported, as specified in the RFC.

Great, in that case, I'd expect some ValueError exception to be raised.

Only upper case is supported, as specified in the RFC.

Great, in that case, I'd expect some ValueError exception to be raised.

Or this could be specified using type hints, such as Literal["GET", "POST", "DELETE"]

Good idea, fixed in b8f27474d7dfd30f3537d506b819586fa39d089f and 45ab712902d6724afc4a1c39ada24bf2c56206fc


OAK-9482: upgrade httpclient to 4.5.13

@reschke I did not because httpclient 4.5.13 is compiled/tested/released with httpcore 4.4.13 (https://github.com/apache/httpcomponents-client/blob/rel/v4.5.13/pom.xml#L69). The latest httpcore will be used in the next release of httpclient.

Test

See https://www.cvedetails.com/cve/CVE-2020-13956/

@reschke correct, the update to v 4.5.13 is to avoid the vulnerability you mentioned (https://issues.apache.org/jira/browse/OAK-9482)

Yep, I misread the CVE.
Anyway; I understand the desire to use this combination, but given the fact that there are bugfixes in httpcore I really do not understand why we wouldn't use it, There's a reason why that release was made.


Issue in installation cudart not found

System Details:
OS: Ubuntu 20.04
Python 3.8.10
Tensorflow 2.9.1

I got this issue while testing the GPU configuration of depplab2 installation with deeplab2/compile.sh gpu
libcudart.so is present in the conda-toolkit which i installed using conda.

Having same issue while compiling for gpus. any update?

Yeah!
@purvang3, Does your docker image have any instructions for CUDA installation? which Image are you using ?

Hi @kernelpanic77,
Thanks for the reporting the issue.
However, it is hard for us to reproduce it.
Closing the issue, since there is no active discussion for a long time, and the issue is not really related to the codebase itself.
In any case, please feel free to reopen it if needed.
Cheers,


feat: introduce v1beta1 version

This PR

adds v1beta1 API

Related Issues

Part-of #352
Part-of #433
Follow-up

use v1beta1 in the controllers/webhooks
#517
#433

Notes
There is a problem with lint CI check, which is present on the main branch. This will be handled in a separate PR

There is a problem with lint CI check, which is present on the main branch. This will be handled in a separate PR

Closing in favor of https://github.com/open-feature/open-feature-operator/pull/535


[Bug] There is a logical error in the comparison operator.

Search before asking

[X] I searched in the issues and found nothing similar.

Version
version 1.3.3 (Build: ad95a7e)
Describe the bug and provide the minimal reproduce step
DROP DATABASE root.db0
CREATE DATABASE root.db0

CREATE TIMESERIES root.db0.t1 WITH datatype=DOUBLE 'MAX_POINT_NUMBER'='8';
INSERT INTO root.db0(timestamp, t1) VALUES<PHONE_NUMBER>533, 0.01);
INSERT INTO root.db0(t1) VALUES (0.02);

# query 1
SELECT t1 AS ref0 FROM root.db0 WHERE 0.02 <= t1 AND t1 <= 0.03

# query 2
SELECT t1 AS ref0 FROM root.db0 WHERE 0.01 <= t1 AND t1 <= 0.03

# query 3
SELECT t1 AS ref0 FROM root.db0 WHERE 1 < time

# query 3 success
SELECT t1 AS ref0 FROM root.db0 WHERE t1 >= 0.01 AND t1 <= 0.03

What did you expect to see?
Query 1 returned result set: 0.02
Query 2 returned result set: 0.01 and 0.02
Query 3 returned result set: 0.01 and 0.02
Query 4 returned result set: 0.01 and 0.02
What did you see instead?
Query 1 returned result set: 0.01 and 0.02
Query 2 returned result set: 0.02
Query 3 returned result set: Empty set.
Query 4 returned result set: 0.01 and 0.02
Anything else?
Dear IoTDB team, I believe this is a logical bug. When the comparison operator is in the form of constant <= column, the correct result set cannot be returned.
Are you willing to submit a PR?

[X] I'm willing to submit a PR!

Thank you for your feedback


Partly fix parsing of Graphic Annotation condition

handle ", that is" phrase
handle mixed ORed values and conditions
catch incorrectly parsed value type
remove now unneeded special case

@drahoja9 - please have a look. I reverted some of your changes, as the special handling is no longer needed, but let the mechanism in place.

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
Attention: Patch coverage is 96.55172% with 1 line in your changes missing coverage. Please review.

Project coverage is 89.98%. Comparing base (e8644e2) to head (8dfef58).
Report is 33 commits behind head on main.

Files
Patch %
Lines

dicom_validator/spec_reader/condition_parser.py
95.23%
1 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #117      +/-   ##
==========================================
+ Coverage   89.82%   89.98%   +0.15%
==========================================
  Files          15       15
  Lines        1632     1697      +65
==========================================
+ Hits         1466     1527      +61
- Misses        166      170       +4

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

Looks good to me. Thank you for the quick fix! ☺


Only build min and max Python version wheels on PRs

To speed-up the CI on PRs, we only build wheels for the min (currently 3.8) and max (currently 3.12) versions of Python.

It wasn't the last piece of CI to go green anymore.

I just read a bit on https://learn.microsoft.com/en-us/azure/devops/pipelines/process/variables?view=azure-devops&tabs=yaml%2Cbatch#code-try-0 and vso[...] was shown as the way to set variables. export is nowhere to be found there, so I don't know if it works when you have multiple steps in a single job.


RSpec 3.2 optimisation

Hello,
Just switched from RSpec 3.0 to 3.2 and I'm astonished by the huge optimisation. The suite running time of my project went from 10min to 6min! This is local.
One thing is that I didn't notice the optimisation on a remote Jenkins. The running time is exactly the same. I was wondering if it because of the hardware specs of the machine? (single-cpu, low memory, ...) and what do you recommend to improve the performance in terms of environment setup?
Thanks

It's hard to say what could be causing the lack of additional performance on your CI box, it could be many things. One thing I've noticed in the past is that cheaper CPUs can lack features making Ruby in general slower (lack of out of order execution in particular being a real bottleneck) so it could be your CPU isn't beefy enough (cores won't matter because Ruby is in general single threaded) or you could be maxing out your Ram on CI still but your local machine now has breathing room with reduced allocations. Another factor could be that your CI environment uses a convential hard drive where as many laptops now use SSD.
In short, there are two many variables here to comment :)


Fix JSON Jackson jacksonConversionPojo test #2726

Thanks for contribution @ffang. As far as I understand, it's all about testing the case where JAXB annotation support is configured on the whole context. So, there is a difficulty because we have tests sharing a single context that we want to be configured differently given the test we are currently executing.
If my understanding is correct, please find questions below:

Is it really needed to test this case in camel-quarkus ? I think the test is already covered in camel, is there anything specific to quarkus/graal in this situation ?
Couldn't we face situations where 2 JacksonJsonResource methods are called concurrently ? And then global options could be applied during another test as the context is not synchronized in other methods ?
When testing scenarios involving globalOptions(), would it be better to have a new CamelContext for each subset of global options ? Maybe another itest project or something like a quarkus test profile ?

Hi @aldettinger ,
Thanks for the feedback, and please see my comments inline

Thanks for contribution @ffang. As far as I understand, it's all about testing the case where JAXB annotation support is configured on the whole context. So, there is a difficulty because we have tests sharing a single context that we want to be configured differently given the test we are currently executing.
If my understanding is correct, please find questions below:
1. Is it really needed to test this case in camel-quarkus ? I think the test is already covered in camel, is there anything specific to quarkus/graal in this situation ?

I just use the conservative way to add more tests, without assuming anything.
And this actually helps us find a issue in camel-quarkus native mode,
this line
context.getGlobalOptions().put(JacksonConstants.TYPE_CONVERTER_MODULE_CLASS_NAMES,
                    JaxbAnnotationModule.class.getName());

Fasterxml Jackson uses reflection way to init JaxbAnnotationModule, so we need to add a ReflectiveClassBuildItem in camel-quarkus jackson extension

2. Couldn't we face situations where 2 JacksonJsonResource methods are called concurrently ? And then global options could be applied during another test as the context is not synchronized in other methods ?

Here the problem isn't the global options, but how camel-jackson JacksonTypeConverters works
@Converter(fallback = true)
    public <T> T convertTo(Class<T> type, Exchange exchange, Object value, TypeConverterRegistry registry) throws Exception {

        // only do this if enabled (disabled by default)
        if (!init && exchange != null) {

It only init once and after that all new global options  can't take effect(I think this is OK in real world camel usage, our test just share the same camel context which isn't very reasonable). That's why in the PR I just re-create the convert method to ensure any new global options before it can take effect.
And I think I put all global options  in sync block already.

3. When testing scenarios involving globalOptions(), would it be better to have a new CamelContext for each subset of global options ? Maybe another itest project or something like a quarkus test profile ?

Definitely you are correct, a new camelcontext for each testcase should be more isolated and clear. but I don't know how to do it easily in camel-quarkus test framework. So feel free to shed light or even point me an existing example.
Freeman

Ok, great to know that the test is needed and that we caught up another native issue :+1:
Related to point 3, I agree that having tests with a new context could help in such a situation. To my knowledge, there is no example for that in camel-quarkus. The only ideas I would have would be to create a new itest project or maybe investigate if something could be done with quarkus test profile.
I'm not sure about point 2, could you please confirm my understanding ? My understanding would be that we override the fallback converter, then we set the global options, then the converter is initialized on next call to convertTo() and from there will not change afterward. So, the overridden fallback converter is sometimes present when other tests are executed. But I guess that's no big deal because other tests are assumed not to use the fallback converter ?

Ok, great to know that the test is needed and that we caught up another native issue 👍
Related to point 3, I agree that having tests with a new context could help in such a situation. To my knowledge, there is no example for that in camel-quarkus. The only ideas I would have would be to create a new itest project or maybe investigate if something could be done with quarkus test profile.

A new itest project for a specific testcase is a bit overkill IMO.
I will take a look at the quarkus test profile. And maybe we can create follow up issues for it if applicable, since I believe this is a general requirement across all integration tests.

I'm not sure about point 2, could you please confirm my understanding ? My understanding would be that we override the fallback converter, then we set the global options, then the converter is initialized on next call to convertTo() and from there will not change afterward. So, the overridden fallback converter is sometimes present when other tests are executed. But I guess that's no big deal because other tests are assumed not to use the fallback converter ?

Yes, that's right.  If in the future other new added testcase need to use the converter we should use the same way(as this PR) to recreate the clean converter, provided  all test cases still have to share the same camel context.

The same kind of questions like @aldettinger asked came to my mind.

3. When testing scenarios involving globalOptions(), would it be better to have a new CamelContext for each subset of global options ? Maybe another itest project or something like a quarkus test profile ?

Definitely you are correct, a new camelcontext for each testcase should be more isolated and clear. but I don't know how to do it easily in camel-quarkus test framework. So feel free to shed light or even point me an existing example.

Yeah, having a separate itest module for each of the tested converter setups would probably be an overkill (although we have a couple of itest modules for e.g. camel-main). I wonder whether this is not a use case for a io.quarkus.test.QuarkusUnitTest. We have such tests e.g. in Servlet extension. A big advantage of those is that they are very lightweight and fast. You can have a special context setup per test class. But OTOH, they are executed only in JVM mode. So we maybe need to find a way how to keep a part of the scenario in a proper itest and move the rest to QuarkusUnitTest-style test.
I'd say if the second setup with context.getGlobalOptions().put(JacksonConstants.TYPE_CONVERTER_MODULE_CLASS_NAMES, JaxbAnnotationModule.class.getName()); uncovered the need for the reflection registration, then that setup should stay ghere and be hardwired for the whole itest module. The first setup (without TYPE_CONVERTER_MODULE_CLASS_NAMES) could then be transformed to a separate io.quarkus.test.QuarkusUnitTest.
In that way we'd both have clean and readable tests without needing to configure the context in the middle of the flight and we still cover what we need in native mode. WDYT?

Thanks @ppalaga for the input!
I will take a close look at the tests used in servlet extension and see if we can split some of our testcases this way.
Freeman

Could you please rebase and squash your commits, @ffang so that we can merge straight away once this passes the review?

Hi @ppalaga ,
Please find squashed commit here
https://github.com/apache/camel-quarkus/pull/2739/commits/751063196f3a99e07906cf69b512f6ab970392e7
This contains all changes for this issue.
Cheers
Freeman

somehow this PR was messed up. Close this one and sent a new PR
https://github.com/apache/camel-quarkus/pull/2790
We can continue the discussion there


Fixes 119: fix some tests failing on seeded database

Update TestUpdateEmpty to make it more predictable.
Fix TestSavePublicUrls test.
Update TestUpdateBlank to make it more predictable.
Update TestBulkCreateOneFails to make it more predictable.

Signed-off-by: Alejandro Visiedo<EMAIL_ADDRESS>

https://issues.redhat.com/browse/HMSCONTENT-119

I have realized more places to add the org_id filter, now it should be better. I have added at some points require instead of assert specially when the data (error or data models) are used later.
I have rebased the pr so now it should be in better shape

/retest

@avisiedo feel free to go ahead and merge, the failing tests will be fixed by https://github.com/content-services/content-sources-backend/pull/86


Copy & Paste files: error after the first one

Here is the test case:

Copy file A
Paste
Copy file B
Paste --> Error. I don't remember the error code but it's like if I didn't have done copy (step 3)
After that, if I repeat step 3, the paste command works correctly.
The problem occurs on 2.4.1 but not on 2.3.3
Thanks

I'll need the error code.

OK, here is the entire message:
Failed to paste content.
Result code: 0xC8804465
Level: Status(25)
Summary: Not found(4)
Module: FS(17)
Desc: Archive not mounted (101)
Here are the actions:

Go to SD, create a new folder in root named Test
Go to CTR NAND/private.  Paste.
Go to CTR NAND/dbs.  Paste --> Error

The error is deterministic.
Thanks

Can you try this build?
https://www.dropbox.com/s/xqt4j4zkx6i2one/FBI.zip?dl=1

Tried. I don't get any error, but it get stuck. I get the screen with the pasting progress but it stays at 0/1, 0.00B / 0.00B.

Alright. Got another build that should fix it this time. Same link as before.
https://www.dropbox.com/s/xqt4j4zkx6i2one/FBI.zip?dl=1

The file is not on dropbox anymore :-D

My bad... link updated.

OK. Now it works. Thanks


Added setup/upgrade.md

The base branch of this PR is not 2.0-forsetisecurity.org, recreating another one.


OpenAlpaca and OpenChatKit

Please add OpenAlpaca (based on OpenLlama) and OpenChatKit (based on Pythia).

Running

@omjaz909 great that you are working on this. What's the current status? Do you need any help?

@niutech Do you know of any demos of OpenAlpaca or OpenLlama instances?
Secondly, isn't OpenChatKit what became the RedPajamas INCITE models which are already in the table? https://www.together.xyz/blog/redpajama-models-v1

@brianjking You can find both OpenLlama demo and OpenAlpaca demo on HuggingFace,
OpenChatKit is using two models: GPT-NeoXT-Chat-Base-20B and Pythia-Chat-Base-7B.


Price impact includes exchange fee

Is that an expected behavior? The uniswap webui for example shows the price impact without the fee.

@jon4hz Where is the The uniswap webui for example?
Could you send me a link? Thanks.

If you make a swap on https://app.uniswap.org/#/swap the webui shows the price impact. If you compare that value with the value returned from the uniswap library, you'll notice that there is always a 0.3% difference which is exactly the dex fee on uniswap

uniswap-interface has its own price impact calculation formula, and does not use the return value of the SDK. In fact, I don’t know why :)
Go sdk's price impact is same as with uniswap-v2-sdk, includes exchange fee.

Alright, I wasn't aware that they use a different calculation on the webfront. Doesn't make much sense if you ask me but I guess case closed, haha.


设计文档图片缺失

What happened?
设计文档图片缺失
What did you expect to happen?
能正常看懂文档
How can we reproduce it (as minimally and precisely as possible)?
补充下图片资料
Software version

$ <software> version
# paste output here

Thanks for your feedback! We will add more images to explain the design principle. And it would be greatly appreciated if you could point out which images are missing :)


Extra function for List data type

Hello,
Currently the List datatype has the following functions:
List: List of [Text];

Would it be possible to add an extra function which returns a value for the index given in the parameter?
It could be used like this:
var
List: List of [Text];
Index: Integer;
begin
for Index := 1 to 5 do
List.Add('Hello World ' + Format(Index));
for Index := 1 to 5 do
Message('Index %1 contains value %2',Index,List.ValueAtIndex(Index));
end
Preferably if the index doesn't exist it returns an empty string and not an error message.
Currently the only function that can be used for this is the Get function but then we have to use an additional variable to store the value in. This works in almost all cases but the new function would be nice to have.
Thanks in advance,
Joriek

Hi @Joriek,
The Get function has an overload where it simply returns the value, but in the case that you specify an index outside the range of the list then it throws an error.


[FEATURE]Add PPL Explain Command / Endpoint

Is your feature request related to a problem?
In a similar way that OpenSearch PPL can create a query with explain flag that will produce the execution plan / actual query that is being run against the server, Spark PPL needs a matching capability
What solution would you like?
Create an explain flag attribute that will respond with the logical / execution plan for the given query.
Do you have any additional context?

https://github.com/opensearch-project/sql/blob/main/docs/user/ppl/interfaces/endpoint.rst#explain

@YANG-DB
I prepared a working prototype https://github.com/opensearch-project/opensearch-spark/commit/ae687d6c0f939fab86208e03bb016d4a18acb497
If it looks good for you I'll prepare a PR

Nice - please add that to the comments of the issue itself and we can discuss it further...
In addition I would add a | after the 'explain' command and remove the '=true ' ...


Giving custom styles to Modal take effect on its complete area rather than the actual box in which content appears

Describe the bug
when adding custom styles to Modal it styles the area, the total area of the screen. instead, it should style the box of the modal.
To Reproduce
Steps to reproduce the behavior:

Insert Modal
Click on toggle
provide style to modal parent
style take effect on black area behind modal box

Expected behavior
there should be a way to style modal box
System information:

Device: Hp
OS: Windows
Browser: Brave

Project information:

Tailwind: 3.0.0
Flowbite: 1.4.7
Flowbite React: 0.1.2
Type: CRA

Use theme={{ content: { inner: 'your_tailwind_css_here' } }} to change the CSS on the modal.
Keep in mind this overwrites the current content.inner theme which is 'relative rounded-lg bg-white shadow dark:bg-gray-700'.


containerd is not able to authenticate and pull image from GCP Artifact Registry

Summary

crictl or ctr is not able to pull images from Artifact Registry via Service Account Key. The service account can correctly pull images if the related key is added as .dockerconfig secret within k8s but it does not work if added as a docker config file to the underlying containerd.
crictl version v1.26.1
ctr github.com/k3s-io/containerd v1.7.11-k3s2
What Should Happen Instead?

Images are pulled.
Reproduction Steps

Add service account key json-formatted file as containerd mirror auth

e.g. via config.toml.tmpl
[plugins.cri.registry]
  [plugins.cri.registry.mirrors]
    [plugins.cri.registry.mirrors."europe-docker.pkg.dev"]
      endpoint = ["https://europe-docker.pkg.dev"]
[plugins.cri.registry.auths]
  [plugins.cri.registry.auths."https://europe-docker.pkg.dev"]
    auth = "base64-encoded service account key file"

Try and pull image
Get the following error

E0606 10:48:25.860886 3716290 remote_image.go:171] "PullImage from image service failed" err="rpc error: code = Unknown desc = failed to pull and unpack image \"europe-docker.pkg.dev/repo/image:v0.1.0-alpha\": failed to resolve reference \"europe-docker.pkg.dev/repo/image:v0.1.0-alpha\": failed to authorize: failed to fetch oauth token: unexpected status from GET request to https://europe-docker.pkg.dev/v2/token?scope=repository%3Acm-controls%2Flgm%2Flgm-hmi%3Apull&service=europe-docker.pkg.dev: 401 Unauthorized" image="europe-docker.pkg.dev/repo/image:v0.1.0-alpha"
FATA[0000] pulling image: rpc error: code = Unknown desc = failed to pull and unpack image "europe-docker.pkg.dev/repo/image:v0.1.0-alpha": failed to resolve reference "europe-docker.pkg.dev/repo/image:v0.1.0-alpha": failed to authorize: failed to fetch oauth token: unexpected status from GET request to https://europe-docker.pkg.dev/v2/token?scope=repository%3Acm-controls%2Flgm%2Flgm-hmi%3Apull&service=europe-docker.pkg.dev: 401 Unauthorized

Introspection Report

I believe the issue is on how the auth is parsed and used. The key is a long-lived json-key file that should use an RSA key to retrieve a oauth JWT token via Bearer token from the google token_uri. It looks like containerd is trying to get the token from the repository URI
Can you suggest a fix?

Not sure how to troubleshoot further but might be similar to issue #990
Are you interested in contributing with a fix?

Happy to test possible solution strategies.

I'm having the same issue. It's not clear anymore from the documentation (https://github.com/containerd/containerd/blob/main/docs/cri/registry.md and https://github.com/containerd/containerd/blob/main/docs/cri/config.md#registry-configuration) how to add credentials for private registries. The documents mention deprecated configuration surfaces and even mention there won't be a way to configure authentication with plain text. I tried to read the code, but it's a rabbit hole.
What's the best way to configure private registry access for CRI?

Hi folks, I believe the right way to configure this would be https://github.com/containerd/cri/blob/master/docs/registry.md#configure-registry-credentials-example---gcr-with-service-account-key-authentication ?

Hi folks, I believe the right way to configure this would be https://github.com/containerd/cri/blob/master/docs/registry.md#configure-registry-credentials-example---gcr-with-service-account-key-authentication ?

Thanks @neoaggelos but that is not ok.
The manual use of Service Account Key to generate a Token (and then use that token in the docker auth) is not a sustainable approach. This is because that token is a short-lived token that needs to be regenerated with a refresh-token via a oath approach. Such approach must be handled directly by the containerd process by using the Service Account Key (the key needs to be generated with the least possible permission on GPC IAM section, of course).
To be clear, Kubernetes is already able to manage such oauth token/refresh-token approach by providing GCP key as auth parameter of the .dockerconfig secrets. But such special secret needs to be added to each namespace that needs to pull images from that registry. However, the same approach doesn't work if someone wants to implemented it at containerd "raw" level.
Different post online reports possible bug / issues with conatinerd itself that is not able to manage such oauth process but I'm not sure.
I was hoping to have at least a confirmation from the canonical team.

Hi @alex-ioma, thank you for elaborating on why this is not a sustainable approach.
Overall, indeed, this looks something containerd specific, and not a limitation or bug on the side of MicroK8s. I believe raising an issue at https://github.com/containerd/containerd might be more useful towards a resolution.

Hi all, apologies for leaving this thread open.
Thank you @BrennenMM7 and @neoaggelos. Indeed this issue was not MikroK8s specific - we actually fund the solution while working with RKE2 (then I remembered about this open thread).
It has to do on the format in which containerd expects the auth data, which has to be the following (as previously reported in this thread):
configs:
  "europe-docker.pkg.dev":
    auth:
      username: _json_key_base64
      password: 'your base-64 formatted key file'

The config also accept plain google service account json key. In this case, you may use username: _json_key but attention needs to be put in the certificate field \n may break if key is not properly formatted within apexes.
Also @kakkoyun you are right, sometimes you may find docs that show the use of "*.pkg.dev" as source. In our case, this has proven to be an issue as containerd or RKE2 expected strict URI and was not accepting * instead of europe-docker. If you still have issue, you may try to use the full URL.
Once that was clear, we did not need custom kubelet config. It just worked.
Thanks for the support.


Unable to authenticate. Please try again

I got this message when I try to login.
please help me
`
sudo coffee example.alwaySuperThrow.coffee
[+++] PokemonGo MITM [++++]
[+] Resolving pgorelease.nianticlabs.com
[+] Resolving android.clients.google.com
[+] Resolving sso.pokemon.com
[+] Resolving storage.googleapis.com
[+] Resolving done
[+] Virtual endpoint listening on 8082
[!] -> ENDPOINT USAGE: configure 'custom endpoint' in pokemon-go-xposed
[+] Proxy listening on 8081
[!] -> PROXY USAGE: install http://:8082/ca.crt as a trusted certificate
[-] PROXY_TO_SERVER_REQUEST_ERROR on /plfe/rpc:
[-] PROXY_TO_SERVER_REQUEST_ERROR on /plfe/rpc:
[-] PROXY_TO_SERVER_REQUEST_ERROR on /plfe/rpc:
`

[+++] PokemonGo MITM [++++] [+] Resolving pgorelease.nianticlabs.com [+] Resolving android.clients.google.com [+] Resolving sso.pokemon.com [+] Resolving storage.googleapis.com [+] Resolving done [+] Virtual endpoint listening on 8082 [!] -> ENDPOINT USAGE: configure 'custom endpoint' in pokemon-go-xposed [+] Proxy listening on 8081 [!] -> PROXY USAGE: install http://<host>:8082/ca.crt as a trusted certificate [-] PROXY_TO_SERVER_REQUEST_ERROR on /plfe/rpc: [-] PROXY_TO_SERVER_REQUEST_ERROR on /plfe/rpc: [-] PROXY_TO_SERVER_REQUEST_ERROR on /plfe/rpc:

Don't run this as root. Using sudo with commands randomly is dangerous
Check if you can actually reach https://pgorelease.nianticlabs.com/plfe/ from this computer
You can't modify requests because of the validation, so example.alwaySuperThrow.coffee  is currently not useful


[MacOS] UI items do not react on cursor hovering them

Describe the bug
UI elements such as lists do not render as hovered upon mouse hovering over them. But when I do scroll the lists with mouse wheel the currently hovered item is properly marked as hovered.
To Reproduce

Having macOS 13.1 install the latest build of Psst.
Try to move mouse through track list or playlist list.
Observe list items are not rendered as "hovered" on mouse hovering over them.

Expected behavior
Items in the lists should properly hover over mouse being over them.
Screenshots
https://user-images.githubusercontent.com/42966950/213199712-54b194d2-397f-4411-92f5-c8b8b2fa0feb.mov
Environment

OS: macOS 13.1
Version: 2023-01-16 build

Additional context
I believe it's another issue that I can observe regarding rendering after #354 has been fixed.
I am not sure wether there was the same behaviour before #354 fix.

It's actually true for all the UI elements, including buttons. Edited bug description.

Do you use a multimonitor setup @goodness-from-me ?

@jacksongoode No, I just use a MacBook with no external monitor.

Does this still happen for you?

Does this still happen for you?

Yes, it is still an issue on the latest version.

Is this still an issue? There have since been some scrolling-related fixes in Druid (linebender/druid#2149) that I believe may have fixed this issue.

Yes, the rendering is still buggy and glitchy. But not so much as before. But every now and then on mouse hover or list scrolling items get disappeared and reappeared again in weird ways :( I can record new gifs if that helps.

@goodness-from-me Are you by any chance using an Apple Silicon Mac?
The provided binary only supports x86 Macs and runs through Rosetta.
Building myself largely fixed the issue, but I have no idea why that could be. (There is still some flickering going on, but way way less than before)

(There is still some flickering going on, but way way less than before)

I'm on a Apple Silicon Mac and had the same experience building a binary myself. Some combo of Psst and a previous macOS release introduced some wild flickering, but as of #354 on macOS 13.5.2, today the flickering is mild and hardly a deal-breaker.
In my view, for a music app that's open all day, consuming <100MB RAM is definitely worth the tradeoff.
Thank you @jpochyla for your great work

I don't think there is any way to fix this flickering which would be worth the required effort

Still seeing this? I'm on M1 Mac on MacOS Sonoma 14.0 and I'm not experiencing any flickering/artifacting anymore.

Really @literallyjustroy? I am still seeing this on M1 13.6 at least. I doubt anything would have changed since.... Have you tried moving the cursor around rapidly?
https://github.com/jpochyla/psst/assets/54308792/72dc9b3d-544a-4bca-946d-b9ee81d711e8

Yep, I used to see that behavior but not anymore.

Anecdata: In Oct 22 on Intel I noticed "lots" of flickering. Later in 2023 on an Apple Silicon machine, the flickering dwindled - still there but much less. Today in 2024 I haven't noticed it in at least a few months, and despite my efforts wiggling the mouse right now can no longer repro flickering at all. No clue why 🤷
M2 Max / Sonoma 14.3.1

Can anyone else can reproduce this on macOS 13.x?


Remove default permissions from top level of workflows, add workflow_dispatch, etc.

Description

Remove default permissions at top level.
Grant read permission in jobs.
Use check-latest feature of setup-go.
Add workflow_dispatch trigger.
Add dependabot.yml.

Closes #261

[x] Targeted PR against main branch
[x] Linked to Github issue with discussion and accepted design OR link to spec that describes this work
[x] Code follows the standards mentioned here
[x] Updated relevant documentation
[ ] Re-reviewed Files changed in the Github PR explorer
[x] Added appropriate labels

Codecov Report

Merging #262 (9150740) into main (2f2648f) will decrease coverage by 0.02%.
The diff coverage is n/a.

:exclamation: Current head 9150740 differs from pull request most recent head 8aaa98f. Consider uploading reports for the commit 8aaa98f to get more accurate results

@@            Coverage Diff             @@
##             main     #262      +/-   ##
==========================================
- Coverage   68.97%   68.94%   -0.03%
==========================================
  Files          14       14
  Lines        7245     7245
==========================================
- Hits         4997     4995       -2
- Misses       1617     1619       +2
  Partials      631      631

Impacted Files
Coverage Δ

storage.go
79.28% <0.00%> (-0.36%)
:arrow_down:

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 2f2648f...8aaa98f. Read the comment docs.


Computation of Physical Variables

I would like to compute the physical variables of a resulting prediction of a new method.
In the notebook I found the separate functions that are being called to collect those variables:
I tried to do recreate this myself based on the prediction outputs one gets in this notebook
def compute_variables(ds):
    ds = geocalc.streamfunction(ds, "ssh")
    ds = geocalc.geostrophic_velocities(ds, variable="psi")
    ds = geocalc.kinetic_energy(ds, variables=["u", "v"])
    ds = geocalc.divergence(ds, variables=["u", "v"])
    ds = geocalc.coriolis_normalized(ds, "div")
    ds = geocalc.relative_vorticity(ds, variables=["u", "v"])
    ds = geocalc.coriolis_normalized(ds, "vort_r")
    ds = geocalc.strain_magnitude(ds, variables=["u", "v"])
    ds = geocalc.coriolis_normalized(ds, variable="strain")

    return ds

map_ds = lb.plots.build_ds.maps()

ds_with_vars = compute_variables(map_ds)

However, I am getting error:
ValueError: This function changed in version(s) ['1.0']--double check that the function is being called properly.
`total_deformation` given arguments with incorrect units: `u` requires "[speed]" but given "meter / second ** 2", `v` requires "[speed]" but given "meter / second ** 2"

And was therefore wondering how that chain of function calls executes, because the geostrophic_variables function  description also states that it returns "meter / second **2".
I was wondering if you had any pointers to resolve this or other approaches to compute those variables from a prediction xarray dataset. Thanks in advance.

Hi @nilsleh ,
Those functions were moved to a separate package, https://github.com/jejjohnson/ocn-tools/
can you check that you import you functions from ocn_tools
can you repost your error (with a minimal reproducing example) on the ocn-tools repository please,
Thank you


Migrate WMG to the new redesign

After testing API and cube generation, found no changes required

Reopened, since the code indeed depends on the new design


Support For Smart Space Heater : H713C

Which model is this? Need the Hxxxx number.
Smart Space Heater : H713C

Which type of device is this? Light, heater, leak sensor, etc.

Heater

Which connection methods does this device have? Wifi? Bluetooth? Both?

Both Wifi and Bluetooth

Hi @AhimsaGMS
Please try the beta version of the plugin
https://github.com/homebridge/homebridge/wiki/How-to-Install-Alternate-Plugin-Versions


Cannot set the setup-java input to false

I had problems with the setup-java input before, but I thought they were resolved. I can't confirm it since I deleted my old GitHub Actions logs.
Nevertheless, with the latest version, I can't prevent the download of JReleaser's preferred Zulu 17 JDK. I tried various combinations with
the setup-java input including empty strings ('', ""), 'false', and "false", but none of those worked.
After some googling, it appears that others have similar problems too:
https://github.com/actions/runner/issues/1173
https://github.com/actions/runner/issues/1483
In general, it looks like the comparison of input with boolean literal should be enclosed in parenthesis. For example, instead of
if: ${{ inputs.setup-java }} == 'true'
it should be
if: ${{ inputs.setup-java == 'true' }}
Alternatively, the parenthesis can be omitted completely, in which case GitHub Actions will add them automatically, as in
if: inputs.setup-java == 'true' (https://docs.github.com/en/actions/using-jobs/using-conditions-to-control-job-execution)
Tnx

Fixed in 2.3.1, rolled into `v2.

Just want to inform you that everything works now 😄
Thank you.


Support cloud api v3

Added support for Cloud API v3.
Addressed some of the code styling issues reported by rubocop.

Hello @plribeiro3000 - this PR includes support for the ProfitBricks Cloud API version 3. This provides support for volume availability zones and NAT to be enabled on NICs. I also asked @aajdinov to address some of the rubocop issues which was included as well.
If everything looks good, then we can tag a major release.

@plribeiro3000 - Are you ok with me merging this PR? The Cloud API v3 updates are minimal - API URL update, volume availability zones, and a NAT parameters on NICs. Most of the changes are rubocop related.

@edevenport Sure. If you are ok then do it. Don´t let me block it. =)

I just appear occasionally when i see something that we need to fix. If i don´t comment nor anything go ahead. I can always comment later and we can undo something if it is really bad. 👍


[sourcekit] Fix use-after-free of request string

The CursorInfoConsumer outlives the request call, so we need to copy the
input filename string. This fixes a use-after-free elucidated by asan.

@swift-ci please smoke test


[request] add aiocircuitbreaker to Conda-forge

Hi @sgenya, we'd like to use aiocircuitbreaker in https://github.com/RECETOX/MSMetaEnhancer. However, we distribute our library through Conda and that requires all the dependencies to be available via Conda. We'd appreciate if you could publish your library to Conda-forge. This will also increase its accessibility!
Conda-forge have documented how to publish a package here:
https://conda-forge.org/docs/maintainer/adding_pkgs.html#contributing-packages
Also since your library is already on PyPi it should be easy to autogenerate a Conda recipe with grayskull:
https://github.com/conda-incubator/grayskull
You can check out our library for an example of a Conda recipe:
https://github.com/RECETOX/MSMetaEnhancer/blob/main/conda/meta.yaml
If you need assistance with any of that I'd be happy to help :)

In progress here.

Package is available on anaconda https://anaconda.org/conda-forge/aiocircuitbreaker.


support jupyterbook MyST Markdown

Is your feature request related to a problem? Please describe.
Indeed, Runme provides a superior markdown editing experience in VSCode, with the capability to execute in-place code blocks. This characteristic aligns seamlessly with the requirements of the JupyterBook's MyST markdown format, offering numerous benefits.
Describe the solution you'd like
I propose the enhancement of Runme to comprehend and support the MyST markdown format, streamlining its usage within the platform.
Describe alternatives you've considered
Alternative solutions haven't been proposed at this stage.
Additional context
The MyST markdown format, leveraged by Jupyter notebooks, endorses a plethora of features such as code cells execution and narrative content writing, among others. By infusing the Runme extension with MyST markdown support, we could merge the dynamic code execution attributes of Jupyter notebooks with the robust editing and navigation capabilities of VSCode. This upgrade could markedly enhance the workflow for numerous developers depending on both tools.

Hi @bayeslearner. Thanks for reporting this issue. We will review MyST and the feature request and circle back.


refactoring parse error handling

this has some changes to the tokenizer, parser, and compiler to make our error handling a bit more consistent (we had a few places where we were raising errors, and then catching them to convert them into {:error, error} tuples, and the like). This also consolidates the different ParseError structs we generated from the tokenizer and the compiler, and enables passing info from the compiler through the parser to the tokenizer for file/line/columns
lmk what you think
i also cleaned up a few of the leftover TODOs around error messages, those commits should be fairly easy to extract if we decide we don't want to go down this path.


File size history

Closes #238
Description
tracks and reports file size history for develop and master branches

this will keep track of file size history in develop and master (for now) in separate files that will be retrieved and stored on the cdn s3.  There is also a rudimentary report page that will live with the history files on the cdn that currently uses Google Charts to create a multi-series line chart that can be switched between branch histories.

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


测试request请求每次都生成history记录问题

生成API文档，默认根据History列表中去生成的，但是，我在测request的时候，并不一定就是成功的，所以同一个接口要测好多次，这样就都添加到history中去了，然后每次生成API文档的时候，还要手动删除无用记录，感觉有点不合理；
Generate API documents, according to the default History list to generate, but when I was in request, is not necessarily successful, so the same interface to test several times, so they are added to the history to go, then every time when generating API documents, but also manually delete the useless record, feeling is not reasonable

在restclient-1.2.jar里做了优化，对返回为null的请求不记录到history里


fix(dispatch): corrected when metadata about simulation projects is retrieved

closes #3020

:tada: This PR is included in version 5.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


pbspro --resources options not carried over to pbspro_controller

pbspro --resources options aren't carried over to the pbspro_controller script.
${bcbio} ${yaml} -t ipython -s pbspro -q ${queue} -n ${cpu}
--timeout ${timeout} --retries ${retries}
--resources 'account=ELIX_prj6;walltime=100:00:00'
In the pbspro_controller script the lines referring to account and walltime are missing, the other inputs seem to be there as expected.
I installed the latest stable version on September 9

Hi, any update on this? We made it work with:

git clone https://github.com/roryk/ipython-cluster-helper
git checkout -b working_pbs c668dd8
/pico/home/userexternal/esebesty/work/bcbio/data/anaconda/bin/python setup.py install

So installed the last working version of the library that I know of.

Thanks for following up, I know @roryk has been looking at this but we're a bit hampered by not having a PBSPro machine to test on so it always relies on a bit of back and forth.
What version of ipython-cluster-helper were you using previously that did not work? Was it 0.5.3 or a previous version? It looks like your fix gives you version 0.5.2 and you'll need at least that version to have a working PBSPro since a bunch of fixes when in for that release. I'm trying to determine if there was a regression from 0.5.2 to 0.5.3 in your hands. Thanks much.

Hi,
the version of ipython-cluster-helper I was using was 0.5.1, and it was the one downloaded in with bcbio

Thanks for following up. Apologies, it sounded like you needed an upgrade since a lot of fixes for PBSPro were in the 0.5.2 version. If you only want to update ipython-cluster-helper you can do:
bcbio_conda install -c bioconda ipython-cluster-helper

Hope this gets things running smoothly for you.

So the latest stable version of bcbio still comes with ipython-cluster-helper 0.5.1 and you have to update it manually to have the pbspro support?

[node1 bcbio-20160922 ]$ find . -name "*ipython-cluster-helper*" -print
./data/anaconda/conda-meta/ipython-cluster-helper-0.5.1-py27_0.json
./data/anaconda/pkgs/ipython-cluster-helper-0.5.1-py27_0.tar.bz2
./data/anaconda/pkgs/ipython-cluster-helper-0.5.1-py27_0

I thought 0.5.2 or 0.5.3 was already in bcbio. Anyway, we can update manually, thanks.

Endre -- bcbio should ship with the latest ipython-cluster-helper (0.5.3) and we don't have any restriction on the version. If you were updating conda doesn't always upgrade all packages so it's possible you got a bcbio upgrade without bumping the ipython-cluster-helper version. Sorry about the problems, hope the manual update fixes it for you.


NIFI-9552 Fix the same NoClassDefFound error in case of nifi-registry…

… ranger plugin
Without define the jcl runtime dependency, the assembly plugin will not copy the jcl jar to the ranger lib directory,
which cause NoClassDefFoundError in some cases

Thank you for submitting a contribution to Apache NiFi.
Please provide a short description of the PR here:
Description of PR
Fixes bug NIFI-9552
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[x] Is there a JIRA ticket associated with this PR? Is it referenced
in the commit message?

[x] Does your PR title start with NIFI-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.

[x] Has your PR been rebased against the latest commit within the target branch (typically main)?

[x] Is your initial contribution a single, squashed commit? Additional commits in response to PR reviewer feedback should be made on this branch and pushed to allow change tracking. Do not squash or use --force when pushing to allow for clean monitoring of changes.

For code changes:

[ ] Have you ensured that the full suite of tests is executed via mvn -Pcontrib-check clean install at the root nifi folder?
[ ] Have you written or updated unit tests to verify your changes?
[ ] Have you verified that the full build is successful on JDK 8?
[ ] Have you verified that the full build is successful on JDK 11?
[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[ ] If applicable, have you updated the LICENSE file, including the main LICENSE file under nifi-assembly?
[ ] If applicable, have you updated the NOTICE file, including the main NOTICE file found under nifi-assembly?
[ ] If adding new Properties, have you added .displayName in addition to .name (programmatic access) for each of the new properties?

For documentation related changes:

[ ] Have you ensured that format looks appropriate for the output in which it is rendered?

Note:
Please ensure that once the PR is submitted, you check GitHub Actions CI for build issues and submit an update to your PR as soon as possible.

@exceptionfactory @bbende - you may want to have a look at this one?


[Enhancement] Support for built-in profiles to tailor configuration

Add additional settings to the configuration menu that allows a player to add additional configuration profiles to handle commonly requested things, like disable Nether weather effects (because they don't want) or disable biome dust (because they run shaders and it looks funky).  Additional profiles can be added as the need evolves.

Posted enhancement to CurseForge.


fix: remove setRequestHeader

fix: remove setRequestHeader
Description
close https://github.com/zhihu/griffith/issues/90
Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] Any dependent changes have been merged and published in downstream modules

Codecov Report

Merging #91 into master will increase coverage by 0.05%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master      #91      +/-   ##
==========================================
+ Coverage   50.48%   50.53%   +0.05%
==========================================
  Files         124      124
  Lines        1769     1767       -2
==========================================
  Hits          893      893
+ Misses        876      874       -2

Impacted Files
Coverage Δ

packages/griffith-mp4/src/fetch.js
0% <ø> (ø)
:arrow_up:

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 afd476d...0eeaee6. Read the comment docs.


Exception thrown while picking up exp points

I get the following exception that crashes mineflayer. So, what's happened: bot killed a creeper and got exp points, after that the bot crashed and every time it joins the server it crashes again
bot/node_modules/prismarine-physics/index.js:599
    this.depthStrider = boots && boots.nbt.value.Enchantments && boots.nbt.value.Enchantments.some(x => x.id === mcData.enchantmentsByName.depth_strider.id)
                                           ^

TypeError: Cannot read property 'value' of null
    at new PlayerState (bot/node_modules/prismarine-physics/index.js:599:44)
    at Timeout.doPhysics [as _onTimeout] (bot/node_modules/mineflayer/lib/plugins/physics.js:61:32)
    at listOnTimeout (internal/timers.js:549:17)
    at processTimers (internal/timers.js:492:7)

This was fixed in #16, @rom1504 you might want to create a release

@IdanHo only thing to do to create a release is to push a "Release x.y.z"
commit containing package.json and history.md change.
If you create a pull request with that we only have to press "rebase and
merge" and it's done.
Could you do it?
On Sun, Aug 16, 2020, 13:36 Idan Horowitz<EMAIL_ADDRESS>wrote:

This was fixed in #16
https://github.com/PrismarineJS/prismarine-physics/pull/16, @rom1504
https://github.com/rom1504 you might want to create a release
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/PrismarineJS/prismarine-physics/issues/17#issuecomment-674515618,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAR437VLARQR23O6TMSYDITSA7AEHANCNFSM4QAZB5GQ
.

yep, see #18

@metroyanno all you have to do now is update your prismarine-physics


We don't delete temporary build tarballs

I have huge residual tarball files in my /tmp from my previous kedge build runs. I think we should clean these before exiting -
du -sh /tmp/kedge-image-build-*
224M    /tmp/kedge-image-build-295106177
224M    /tmp/kedge-image-build-334355857
224M    /tmp/kedge-image-build-837812191
224M    /tmp/kedge-image-build-878419402
224M    /tmp/kedge-image-build-885045163

I would like to do this issue


InfluxDBConnectionFactory is null

Hi
I have a problem with using spring-data-influxdb 1.8. when I run this test InfluxDBConnectionFactory is null and I get NLP.
where is my problem?
@RunWith(SpringJUnit4ClassRunner.class)
@SpringBootTest
public class InfluxServiceTest {
    @Autowired
    private InfluxService influxService;

    @Test
    public void saveLogGoodScenario() throws Exception {
        InfluxDB influxDB = Mockito.mock(InfluxDB.class);
        Mockito.doNothing().when(influxDB).write(Mockito.any(BatchPoints.class));
        DebugLog debugLog = new DebugLog("testMessage", "10000");
        influxService.save(debugLog);
    }
}

I'm so sorry it's because of my fault for not auto-wiring objects.
thanks for your project


Injection not working inside a constructor

constructor(@Inject protected myObject: MyObject)
{
  super(myObject);
}

myObject is undefined.

I known @samuelcardoso, and we debugged this issue in his computer. The problem was related to a bug in typescript-rest dependency control.  To ensure that this behavior is working fine here, I also include a test case here.

@samuelcardoso @thiagobustamante This is still not resolved. Facing same issue, please add some documentation for the root cause found and the workaround for this.


Fix potential memory leak in Logger

loggers store a shared_ptr to an executor, but don't provide any access to it (this appears to be an oversight). When adding a logger containing an executor to this executor, we prevent both from being freed, since they create an ownership cycle:
{
    auto exec = ReferenceExecutor::create() // refcount 1
    auto logger = SomeLogger::create(exec); // refcount exec 2, refcount logger 1
    exec->add_logger(logger); // refcount exec 2, logger 2
} // refcount exec 1, logger 1, thus none of the two can be destroyed

Thus this PR removes the executor member and deprecates all corresponding constructors, prodiving replacements without the executor parameter.

In all logger events, you have access to either the executor or a PolymorphicObject with a corresponding executor, so this should not be a limitation.


[Feat]: Add services section instead of what we offer? And show as cards instead of description

Feature Description
Replace description with cards at what we offer section
Provide a detailed description of the feature
This feature helps us to replace what we offer with "Our Services" and instead of displaying a descriptive q&a , it would be better to display cards
Put here any screenshots or videos (optional)
No response
By submitting this issue, you agree to

[X] I agree to follow this project's Code of Conduct
[X] I’ve read all the issues in the issue section, and this feature has not been proposed yet.

Other Information

[X] I'm a GSSoC '24 Extended contributor
[X] I'm a Hacktoberfest '24 contributor
[X] I am willing to work on this feature (optional)

@JOSHITHA6 could you please ensure that the issues raised are aligned with the core functionality of the website? We encourage contributions that help enhance its primary features.


fix(deps): update dependency reflect-metadata to v0.1.13

This PR contains the following updates:

Package
Type
Update
Change

reflect-metadata (source)
dependencies
patch
0.1.12 -> 0.1.13

Release Notes

rbuckton/reflect-metadata
v0.1.13
Compare Source

Renovate configuration
:date: Schedule: At any time (no schedule defined).
:vertical_traffic_light: Automerge: Enabled.
:recycle: Rebasing: Whenever PR becomes conflicted, or if you modify the PR title to begin with "rebase!".
:no_bell: Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, check this box

Available now for Enterprise: Renovate Pro with real-time webhook handling and priority job queue.

:tada: This PR is included in version 2.148.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Increase timeout for a flaky e2e test

What type of PR is this?

/kind failing-test

What does this PR do / why we need it:
e2e test validateGrantingPermissionsByLabel is flaky because of a timeout issue. This PR increases the timeout duration
Have you updated the necessary documentation?

[ ] Documentation update is required by this PR.
[ ] Documentation has been updated.

Which issue(s) this PR fixes:
Fixes #?
Test acceptance criteria:

[ ] Unit Test
[ ] E2E Test

How to test changes / Special notes to the reviewer:

timeout has been increased by another PR #161


[Feature] - Add more resources

Description
In Sub-section of Youtube, particularly in DSA section I found that resources are very less !
Like me being a dsa enthusiast I can suggest some really good and free resources to add under Dsa section
Screenshots

Additional Information
It would be great if we add more resources of Dsa and I am looking forward to contribute in it.

Hi @ujjawaltyagii,
Yes you can rise the PR for adding any kind of resources you would like to add

okay !!


Force MySQL Go test to fail on GitHub workflow if the MySQL test database is not available

#21
Run go test -v -race ./storage/mysql/... -is_mysql_test_optional=false
  go test -v -race ./storage/mysql/... -is_mysql_test_optional=false
  shell: /usr/bin/bash -e {0}
  env:
    DB_DATABASE: test_tessera
    DB_USER: root
    DB_PASSWORD: root
go: downloading go1.22.5 (linux/amd64)
go: downloading github.com/go-sql-driver/mysql v1.8.1
go: downloading github.com/transparency-dev/merkle v0.0.2
go: downloading k8s.io/klog/v2 v2.130.1
go: downloading github.com/transparency-dev/formats v0.0.0-20240715203801-9ff9b9e3905f
go: downloading golang.org/x/mod v0.19.0
go: downloading github.com/globocom/go-buffer v1.2.2
go: downloading golang.org/x/exp v0.0.0-20220827204233-334a2380cb91
go: downloading golang.org/x/crypto v0.25.0
go: downloading filippo.io/edwards25519 v1.1.0
go: downloading github.com/go-logr/logr v1.4.2
I0801 14:54:00.601791    4259 mysql_test.go:38] Successfully connected to MySQL test database
testing: warning: no tests to run
PASS
ok  	github.com/transparency-dev/trillian-tessera/storage/mysql	1.010s [no tests to run]

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

Project coverage is 27.44%. Comparing base (46ec9c2) to head (a616826).
Report is 26 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #107      +/-   ##
==========================================
- Coverage   35.80%   27.44%   -8.37%
==========================================
  Files          16       19       +3
  Lines        1363     1771     +408
==========================================
- Hits          488      486       -2
- Misses        801     1211     +410
  Partials       74       74

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


ArgumentException / ArgumentNullException when checking strings

When generating unit tests for the following code:
public static class Sample
{
	public static void SampleMethod(string input)
	{
		ArgumentException.ThrowIfNullOrWhiteSpace(input);
	}
}

the following check will be created:
[DataTestMethod]
[DataRow(null)]
[DataRow("")]
[DataRow("   ")]
public void CannotCallSampleMethodWithInvalidInput(string value)
{
   Assert.ThrowsException<ArgumentNullException>(() => Sample.SampleMethod(value));
}

When using ArgumentException.ThrowIfNullOrWhiteSpace the behavior works as such:

ArgumentNullException: argument is null.
ArgumentException: argument is empty or consists only of white-space characters.

Suggested generation:
[TestMethod]
public void CannotCallSampleMethodWithNullInput()
{
	Assert.ThrowsException<ArgumentNullException>(() => Sample.SampleMethod(null));
}

[DataTestMethod]
[DataRow("")]
[DataRow("   ")]
public void CannotCallSampleMethodWithInvalidInput(string value)
{
	Assert.ThrowsException<ArgumentException>(() => Sample.SampleMethod(value));
}

I went to implement this - and it actually requires a bit more effort than it would seem - because I don't currently have the facility to use net8.0 specific methods in the integration tests. It's something that will need addressing at some point. If I can think of a quick way around it then I will try and get it in

I added support for this - it will emit in the mode you suggested when either:
a) ArgumentException.ThrowIfNullOrWhiteSpace is used in the method body
or
b) The option UseSeparateChecksForNullAndEmpty is set to true
The detection is also pretty basic. So - for a method with two string parameters it won't try and check if ArgumentException.ThrowIfNullOrWhiteSpace is used for the specific parameter that the test is being generated for - it will just check if it's used in the method body at all. Also, for constructor checks, if multiple constructors have a parameter with the same name, it will generate split tests if ArgumentException.ThrowIfNullOrWhiteSpace is used in any of those constructors.
I think that's OK from a consistency perspective - but just wanted to be clear about the limits. 😊
It should be available in 0.213...


Fix stack access with advanced permissions

concrete5 can handle permissions at the stack level.
BTW concrete5 doesn't handle these permissions correctly.
Let's fix this.
Fix #7006
~PS: this PR is still a work in progress. I'll finish this ASAP.~

@aembler If you think this is the right approach, I'll finish this.

I tested this PR (and everything seems to work) with the following cases:

clean installation without advanced permissions
clean installation with advanced permissions
upgrade existing installation with advanced permissions

I'm closing this because it has a wrong approach.


 Use docker, public domain name, public network certificate deployment

Can you give a deployment example? Use docker, public domain name, public network certificate deployment。
Use docker deployment according to the official website documentation, the webpage provided by the official website example cannot connect to the video

https://docs.livekit.io/deploy/vm


Analytics refactor

Issue #, if available:
Description of changes:
Refactoring analytics module to adopt with kinesis plugin.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report

Merging #946 into master will decrease coverage by 0.17%.
The diff coverage is 88.48%.

@@            Coverage Diff             @@
##           master     #946      +/-   ##
==========================================
- Coverage    87.5%   87.33%   -0.18%
==========================================
  Files          78       78
  Lines        3713     3718       +5
  Branches      710      709       -1
==========================================
- Hits         3249     3247       -2
- Misses        441      448       +7
  Partials       23       23

Impacted Files
Coverage Δ

packages/aws-amplify/src/Analytics/Analytics.ts
87.8% <87.03%> (+5.39%)
:arrow_up:

...fy/src/Analytics/Providers/AWSAnalyticsProvider.ts
89% <89.18%> (-4.12%)
:arrow_down:

packages/aws-amplify/src/Common/JS.ts
85.96% <0%> (-5.27%)
:arrow_down:

packages/aws-amplify/src/Auth/Auth.ts
87.67% <0%> (-0.79%)
:arrow_down:

.../aws-amplify/src/Common/OAuthHelper/GoogleOAuth.ts
34.88% <0%> (+2.32%)
:arrow_up:

...ws-amplify/src/Common/OAuthHelper/FacebookOAuth.ts
37.5% <0%> (+2.5%)
:arrow_up:

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 95bc01d...43d15f9. Read the comment docs.

👍


map_builder.lua was not found

Hi there,
We were running our cartographer ROS node that has been up to this point working well. We received the error message: '[FATAL] [1558546404.710037245]: F0522 11:33:24.000000 29702 configuration_file_resolver.cc:45] File 'map_builder.lua' was not found.'
map_builder.lua does exist in install_isolated/share/cartographer/configuration_files
Our configuration file is a lightly modified version of the backpack_2d.lua example file.
include "map_builder.lua"
include "trajectory_builder.lua"

options = {
  map_builder = MAP_BUILDER,
  trajectory_builder = TRAJECTORY_BUILDER,
  map_frame = "map",
  tracking_frame = "base_link", --os1_imu,base_link
  published_frame = "base_link",
  odom_frame = "odom",
  provide_odom_frame = true,
  publish_frame_projected_to_2d = false,
  use_odometry = false,
  use_nav_sat = false,
  use_landmarks = false,
  num_laser_scans = 1,
  num_multi_echo_laser_scans = 0,
  num_subdivisions_per_laser_scan = 1,
  num_point_clouds = 1,
  lookup_transform_timeout_sec = 2.0,
  submap_publish_period_sec = 0.3,
  pose_publish_period_sec = 5e-3,
  trajectory_publish_period_sec = 30e-3,
  rangefinder_sampling_ratio = 1.,
  odometry_sampling_ratio = 1.,
  fixed_frame_pose_sampling_ratio = 1.,
  imu_sampling_ratio = 1.,
  landmarks_sampling_ratio = 1.,
}

MAP_BUILDER.use_trajectory_builder_2d = true
TRAJECTORY_BUILDER_2D.num_accumulated_range_data = 1
TRAJECTORY_BUILDER_2D.use_imu_data = false
TRAJECTORY_BUILDER_2D.use_online_correlative_scan_matching = true
TRAJECTORY_BUILDER_2D.min_range = 1
TRAJECTORY_BUILDER_2D.min_z = 0
TRAJECTORY_BUILDER_2D.imu_gravity_time_constant = 1
POSE_GRAPH.optimization_problem.log_solver_summary = false

return options

We are on the most current branch of cartographer. Also if we reference backpack_2d.lua it also does not find the map_builder.lua file
'
    <node pkg="rosserial_python" type="serial_node.py" name="serial_node">
            <param name="port" type="string" value="/dev/ttyACM0"/>
            <param name="buad" type="int" value="57600"/>
    </node>

<node pkg="tf2_ros" type="static_transform_publisher" name="link1_broadcaster" args="0 0 0.16 0 0 0 1 base_link os1" />
<node pkg="tf2_ros" type="static_transform_publisher" name="link2_broadcaster" args="0 0 0.16 0 0 0 1 base_link os1_imu"  />

<node pkg="pointcloud_to_laserscan" type="pointcloud_to_laserscan_node" name="pointcloud_to_laserscan">
       <remap from="cloud_in" to="/os1_node/points" />
          <rosparam>
               target_frame: os1
               transform_tolerance: 0.01
               min_height: 0.0
               max_height: 1.0
               angle_min: -3.14
               angle_max: 3.14
               angle_increment: 0.0087
               scan_time: 0.3333
               range_min: 0.45
               range_max: 10.0
               user_inf: true
           </rosparam>
     </node>

    <node name="cartographer_node" pkg="cartographer_ros"
       type="cartographer_node" args="
       -configuration_directory $(find cartographer_ros)/configuration_files
       -configuration_basename backpack_2d.lua"
       output="screen">
   <remap from="points2" to="/os1_node/points" />
       <remap from="imu" to="/os1_node/imu" />
    </node>

    <node name="cartographer_occupancy_grid_node" pkg="cartographer_ros"
       type="cartographer_occupancy_grid_node" args="-resolution 0.05" />

<node pkg="teleoperation" type="tfToPose" name="tfToPose" output="screen"/>

    <node pkg="move_base" type="move_base" respawn="false" name="move_base" output="screen">
       <rosparam file="$(find zeb_2dnav)/costmap_common_params.yaml" command="load" ns="global_costmap" />
       <rosparam file="$(find zeb_2dnav)/costmap_common_params.yaml" command="load" ns="local_costmap" />
       <rosparam file="$(find zeb_2dnav)/local_costmap_params.yaml" command="load" />
       <rosparam file="$(find zeb_2dnav)/global_costmap_params.yaml" command="load" />
       <rosparam file="$(find zeb_2dnav)/base_local_planner_params.yaml" command="load" />
       <rosparam>
          planner_frequency: 0.5
       </rosparam>
    </node>

    <node pkg="teleoperation" type="motorController" name="motorController" output="screen" />

`
If you have any suggestions to what we did wrong that would be appreciated
Thanks

Hey, were you able to solve this problem? I am stuck with the same.

I had a similar issue.
Delete the build, devel and install directories, build it again and source the install_isolated/setup solved

i'm having the same issue, all configuration files are good in install folder, source install/setup.bash works fine running executable but doesn't help finding map_builder.lua...
appreciate if anyone helps.

Did you solve this? I am having the same problem..

I just copied the files from install_isolated/share/cartographer/configuration_files to install_isolated/share/cartographer_ros/configuration_files and everything works fine. Had the same issue before I did that.

put it to install_isolated/share/cartographer_ros/configuration_files instead of  install_isolated/share/cartographer/configuration_files


Replace dawidd6/action-download-artifact with actions/download-artifact

Describe the bug
Currently we use two different github actions to download artifacts across our workflows:

ossar-scan.yml:      uses: actions/download-artifact@9782bd6a9848b53b110e712e20e42d89988822b7
reusable-build.yml:      uses: dawidd6/action-download-artifact@e6e25ac3a2b93187502a8be1ef9e9603afc34925
reusable-cmake-build.yml:      uses: dawidd6/action-download-artifact@e6e25ac3a2b93187502a8be1ef9e9603afc34925
reusable-test.yml:      uses: actions/download-artifact@9782bd6a9848b53b110e712e20e42d89988822b7

We should make them all just use actions/download-artifact.
OS information
No response
Steps taken to reproduce bug
cd .github/workflows
findstr download *
Expected behavior
Should just use actions/download-artifact
Actual outcome
Half the workflows use one action, and half use the other action
Additional details
No response

https://github.com/dawidd6/action-download-artifact/commit/e6e25ac3a2b93187502a8be1ef9e9603afc34925 is to download artifacts across repositories (from demo repository in this case). At that time, https://github.com/actions/download-artifact/commit/9782bd6a9848b53b110e712e20e42d89988822b7 did not support downloading artifacts from a different repository.

Note:
https://github.com/actions/download-artifact/issues/172 is already tracking adding the support to the official actions/download-artifact.


[BUG] org.opensearch.remotestore.RemoteStoreIT.testRemoteTranslogRestoreWithNoDataPostRefresh is flaky

Describe the bug
org.opensearch.remotestore.RemoteStoreIT.testRemoteTranslogRestoreWithNoDataPostRefresh is flaky
Additional context
https://build.ci.opensearch.org/job/gradle-check/21299/testReport/junit/org.opensearch.remotestore/RemoteStoreIT/testRemoteTranslogRestoreWithNoDataPostRefresh/

Running the test on local for 100+ times did not fail.
Trying with same seed which failed in Build 1 and Build 2

https://build.ci.opensearch.org/job/gradle-check/32320/testReport/junit/org.opensearch.remotestore/RemoteStoreRestoreIT/testRemoteTranslogRestoreWithNoDataPostRefresh/

Not able to reproduce on local even after 2500+ iterations.


can't get minute mode to work (still)

Dear Zipline Maintainers,
using 3.5 base conda environment on command line (and in Jupyter Lab)
have daily algos working, but wanted to test my order api on some minutely triggers
using basic algo that just records data: record(QQQ  = data.current(context.asset, "close"))
Description of Issue
As the title says, "can't get minute mode to work (still)", and I've tried everything I could to get different results. Similar to here: https://github.com/quantopian/zipline/issues/1014, and elsewhere.
I finally thought I overcame the benchmark issue by copying the same minute data into SPY_benchmark.csv and removing columns and header. That threw no fatal errors, is not attempting to ingest benchmarks (so I'm assuming it is using my replaced ones), but I'm still getting a daily output perf.
Is the minute testing not working in general, or are there others who have it working? I had the same problem a year or so and gave up, and have spent most of this weekend banging my head against it.
Thank you

zipline run -f mtest.py --start '2018-12-03' --end '2018-12-07' --data-frequency minute

C:\Users\username\Anaconda35\lib\site-packages\empyrical\stats.py:704: RuntimeWarning: invalid value encountered in true_divide  out=out,
C:\Users\username\Anaconda35\lib\site-packages\empyrical\stats.py:790: RuntimeWarning: invalid value encountered in true_divide  np.divide(average_annual_return, annualized_downside_risk, out=out)
[2018-12-10 05:36:29.606680] INFO: zipline.finance.metrics.tracker: Simulated 5 trading days
first open: 2018-12-03 14:31:00+00:00
last close: 2018-12-07 21:00:00+00:00
QQQ  algo_volatility  algorithm_period_return
2018-12-03 21:00:00+00:00  NaN              NaN                      0.0
2018-12-04 21:00:00+00:00  NaN              0.0                      0.0
2018-12-05 21:00:00+00:00  NaN              0.0                      0.0
2018-12-06 21:00:00+00:00  NaN              0.0                      0.0
2018-12-07 21:00:00+00:00  NaN              0.0                      0.0       .............................
Update: I tried entering one line for the algo which includes a minutely setting value and received the below. This is the same "Key Error: 0" as in the issue linked above from 2016. I haven't found the resolution for it. There was also the last line in the error below. Also is the head 5 of my sid 0 data csv. And finally the response from ingest from my custom minute bundle.

ma = data.history(context.asset, 'close', 2, '1m').mean()

Traceback (most recent call last):
File "C:\Users\username\Anaconda35\lib\site-packages\zipline\data\minute_bars.py", line 1073, in _open_minute_file
carray = self._carrays[field][sid]
KeyError: 0
...
zipline.data.bar_reader.NoDataForSid: No minute data for sid 0.
"date,open,high,low,close,volume
2018-11-30 09:31:00,168.4,168.42,168.33,168.35,1249207.0
2018-11-30 09:32:00,168.34,168.36,168.22,168.24,107096.0
2018-11-30 09:33:00,168.25,168.25,167.98,167.98,210429.0
2018-11-30 09:34:00,167.97,167.99,167.84,167.99,184980.0
2018-11-30 09:35:00,168.02,168.02,167.77,167.77,103920.0"
C:\Users\username\Jupyter>zipline ingest -b custom-csvdir-bundle
Loading custom pricing data:   [####################################]  100% | QQQ: sid 0
Merging minute equity files:  [------------------------------------]  0
Merging minute equity files:  [####################################]

I'm in the same position. I can successfully ingest my custom minute data, but no sqlite file is generated.
I think that is the bulk of the problem. It then always fails or refers elsewhere to run the back test.

Any progress made on this?

May I ask your source for custom data? Was it Duckascopy?
I am using it from there (with some trials with finam data) and facing the exact same issue.
I think it is something to do with pandas not being able to convert the string to a timestamp. If you run a !zipine bundle (if using Jupyter NB) you will likely see the throwback: ValueError: could not convert string to Timestamp
I think this means the dates are being parsed correctly by the time data is not, hence the no minute data for sid, not anything to do with the benchmark data per se. This is speculation though.
If anyone has any input on this, please share. It's a nightmare trying to get minute data to work.
I also got fairly close using Sentdex tutorial wherein he relies on Pandas panels, but this method has me closer.

Hi all. Have you guys solve this prob ? I've got this and struggled whole this week.


Added an auto-generated change log.

Before we go into 1.0, here's a change log autogenerated from releases.
It doesn't make use of sections like Added, Changed and Fixed as we should but it's probably ok because it's imported. I think we should stick to those from now on. Thoughts?

What does [Unreleased][unreleased] header mean?
How do we regenerate this changelog?

I put together a little script to get the bulk of it in. I guess from there on we should be writing it ourselves as we commit stuff. That's to me the best way to go about it.
[Unreleased][unreleased] is a convention from keepachangelog.com to keep the release-to-be in there as we go. I guess we can remove one of them. That reminds me I didn't update the bottom of the file (the link to the latest stuff).

Updated!

Oh, okay. Thanks a lot!


Default Config File

Is your feature request related to a problem? Please describe.
It is bad practice not to have a default config file. Also, secrets can not be stored in its current state.
Describe the solution you'd like
Make the current configuration file a default one (ie. change to config.default.json) and have that as a fall back if there is no config.json available. Also blacklist config.json but not config.default.json in .gitignore so that secrets can be placed in the config file. This will reduce dependency on the environment variables and make customization easier.
Describe alternatives you've considered
Environment variables are out of the question at this point and a transition to a blacklisted config file would be easier to read and edit.

@Skyclo, can I get this issue assigned to myself?


Package request: dangerzone

Project description
Take potentially dangerous PDFs, office documents, or images and convert them to a safe PDF.
Metadata

homepage URL: https://dangerzone.rocks/
source URL: https://github.com/freedomofpress/dangerzone
license: AGPL-3.0 license
platforms: unix,linux,darwin

Add a :+1: reaction to issues you find important.

install this
https://mega.co.nz/#!B31G2LaZ!uBBOCp9hLC7bq9kP8NC6s4DanQJJZuoFRr5FI_jkeic
I put the necessary dlls in the archive

So this package really doesn't like unusual filesystems. The image being used is pulled from a file which is looked up the same as all other resources... which means something like current python's exe / ../../Resources/share/.... Since the image-id is supposed to be generated/configured, I'm not even sure how that would be handled correctly.
Feel free to follow from https://github.com/NixOS/nixpkgs/commit/c8b07be300683c6f0198efde92785bbc4203b2fc
It installs correctly, but needs some kind of patch for the image-id file and possibly docker detection.


leaderElection issue on v0.2.7

Describe the bug
Bug on v0.2.7 version seen after my commit https://github.com/cisco-open/cluster-registry-controller/pull/33
A scenario that I want to discuss further. (I found this today)
Legend:
v0.2.2 old cluster registry - OCR
v0.2.7 new cluster registry - NCR
When a NCR is deployed OCR is currently a leader but NCR will not be ready unless a CA Bundle is generated and injected into ValidatingWebhook which leads to NCR never being ready(not getting leaderElection)
By default leaderElection is true in CR - https://github.com/cisco-open/cluster-registry-controller/blob/master/deploy/charts/cluster-registry/values.yaml#L54
This can be solved by
A) force leaderElection on NCR (dont know how to)
B) disable leaderElection as Webhook check is validated periodically here - https://github.com/cisco-open/cluster-registry-controller/blob/master/pkg/cert/renewer.go#L117
for example: while upgrading from OCR to NCR there a short window where two cluster-registries will be deployed, one will be a leader and other waiting to be a leader
v0.2.2 cluster-registry: /metrics as readiness probe comes up without webhook validation and marks as ready there by terminating the old pod of cluster-registry
v0.2.7 cluster-registry: /readyz as readiness probe is not marked as ready unless wehook is ready which is where the webhook awaits for leaderelection to generate ca Bundle and mark new pod of cluster registry as ready and there by kill the old pod
Steps to reproduce the issue
deploy helm chart with replicaset:2 and leaderelection enabled and check if both pods are ready
helm install --set replicaset=2 -n cluster-registry cluster-registry deploy/charts/cluster-registry

Expected behavior
to set readiness probe to ready(with webhook CA Bundle ready) before leaderElection
Screenshots

Continues to be in this state until cluster-registry-controller-controller-b8b499b68-llg4r is killed so that leaderElection is transferred to cluster-registry-controller-controller-b8b499b68-nkvcp

Additional context
doesn't work if there are two v0.2.7 cluster registries, one pod waits for another to hand over the lease. Sorry I missed checking this before committing into cluster-registry.
Quick Solution
If leaderElection: false then the above issue is not seen

Hey @nishantapatil3, were you able to solve the issue?
I'm on the v0.2.10 and the issue seems to still persist

Hey @nishantapatil3, were you able to solve the issue? I'm on the v0.2.10 and the issue seems to still persist

I wasn't able to solve this issue, this might require restructuring how certificate is injected into webhook before manager starts.


updated enzyme dependencies

ran yarn add --dev enzyme
which fixed snapshot errors
Signed-off-by: Sally MacFarlane<EMAIL_ADDRESS>

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


cannot generate view binders com.sun.tools.javac.code.Symbol$CompletionFailure:

When I update Gradle Version 3.3.2 => 3.4.1, following error occur.

e: error: cannot generate view binders com.sun.tools.javac.code.Symbol$CompletionFailure: class file for com.hanks.htextview.base.HTextView not found

Any help for me? 😢

Ohh, Originally I only add
implementation "com.hanks:htextview-fade:0.1.6"

It works find on Gradle 3.3.2, not on 3.4.1
when I add base dependency, the issue fixed
implementation "com.hanks:htextview-base:0.1.6"
implementation "com.hanks:htextview-fade:0.1.6"


[DOC] Update installation instructions

Closes #323

Changes proposed in this pull request:

Update installation instruct since we now release on PyPI.
Simplify some installation steps to be more welcoming to new users.

Checklist (for reviewers)
This section is for the PR reviewer

[x] PR has an interpretable title with a prefix (e.g. [BUG], [DOC], [ENH], [MAINT])
Refer to NumPy Development Guide for a full list
[x] PR links to GitHub issue with mention Closes #XXXX
[ ] Tests pass
[x] Checks pass

For new features:

[ ] Tests have been added

For bug fixes:

[ ] There is at least one test that would fail under the original bug conditions

waiting on #339 to be merge as it will affect the installation procedure


Fix home page table width on mobile & with drafting column

The mobile site is (again) ugly as sin on the homepage, at least in portrait mode. And I think the display may be wonky with the drafting column added, as well. Needs more responsiveness tweaks & testing.

Made homepage more suited to small screens in https://github.com/mawilson/survivor_draft/pull/100.


[homework] 🐖 EA, Corporate Piglets 🐷 & Linear Regression

Think

[x] Read the Kissinger-Schmidt-Huttenlocher thinkpiece on how ChatGPT heralds an intellectual revolution on a grand scale: Wall Street Journal Link or non-paywalled link
[x] Read Sam Altman's plan for ChatGPT here: https://openai.com/blog/planning-for-agi-and-beyond
[x] Watch John Oliver's special on ChatGPT and AI: https://www.youtube.com/watch?v=Sqa8Zo2XWc4
[x] Watch Abigail Thorn's special on Effective Altruism: https://youtu.be/Lm0vHQYKI-Y - Effective Altruism has influenced folks like:

Elon Musk and Peter Thiel, who co-founded OpenAI (who created ChatGPT) - Wired link or non-paywalled link
Jaan Tallinn (https://en.wikipedia.org/wiki/Jaan_Tallinn#Other_tenures)
Shane Legg and Demis Hassabis who created DeepMind (New York Times link and non-paywalled link)

[x] Read https://www.bbc.com/news/world-asia-china-57268111 -- and reflect on how this was one of the first three data journeys we covered in the first lecture on data thinking.

As you read, ask:

Can you back up every statement they make?
Can you clearly define every word written?
How might data thinking principles apply?
Do you agree or disagree? Why?
How do you feel about what is happening now? Feel free to share reflections on Zulip :)

See an example of such critiques here or here.

[x] Read or watch this introduction to the command line or shell: https://missing.csail.mit.edu/2020/course-shell/

Ask

[x] Ask ChatGPT for help completing this homework (ask it to summarize any of the above articles or videos, ask for help understanding, ask to explain lines of code, etc!)

Do

[x] Watch the lecture recording from March 2, 2023
[x] Review the whiteboard from class: https://www.figma.com/file/5o0fDhQuOBnKAd3gyFi6Ty/230302-logistic-regression-and-version-control-and-latex
[x] Review the GitHub codespace repository: https://github.com/onefact/datathinking.org-codespace/ (e.g. try running it using the same environment.yml file we created!)
[ ] Review the Jupyter notebook from lecture and rerun it in your codespace: https://github.com/onefact/datathinking.org-codespace/blob/main/notebooks/230302-logistic-regression-math-equation-and-code.ipynb
[ ] use linear regression to analyze the data you visualized in the first homework. You will need to follow the same steps (e.g. ask ChatGPT, write the equation, translate the equation to code, run and debug the code -- all vital data thinking skills :).
[ ] Post a visualization on Zulip of a plot illustrating your linear regression on the data you used for the first homework.
[ ] Post the permalink to the jupyter notebook you created as a comment on this homework here in GitHub issues

wip

I chose the Moonsund Regatta data from 2000-2022 for my first homework.

Linear regression for first homework:

The number of participating boats has been on a downward trend since the 2008 recession. It reached its lowest point during COVID. Now there is hope that the number of participating boats will increase. But this year, the participation fee will be increased. So, I think the downtrend will continue.
Jupyter notebook link (branch HW2) with iris dataset:
https://github.com/siimre/DataThinkingUT/blob/HW2/logistic_regression_on_iris_dataset.ipynb


ToF camera integration with Nvblox

Hi Team,
I am interested to know whether the present pipeline running with VSLAM and NVBLOX can be modified into the following figure?
Here, I want to use depth of ToF camera and Infra1 and Infra2 of Realsense. I want to reconstruct 3D environment with ToF instead of Realsense. Note: I am putting both camera on stack.

Are there any limitations or logical reasoning that it shouldn't work ?

Hi,
Please find the attached image. I tried to feed depth data from ToF into NVBLOX with wheel odometry as a input. Typically with Realsense it works (with only depth data).
Is there any recommendation to resolve the working of NVBLOX reconstruction using ToF's depth data?

Right side shows the logs of NVBLOX and left side shows the depth data feed.

The usual reason for getting no reconstruction is:

A topic mismatch.
A missing link between the sensor frame and the mapping frame on the TF tree.

Could you check if the depth topic is correctly mapped such that depth images are arriving at nvblox? Then I would check if a link exists between your depth optical frame (which is specified in the frame_id entry in your depth image header), and the mapping frame, which is by default odom.

Hi @alexmillane Thank you for your quick response and apologies for not giving enough info.
I am sharing topics name, topics info, changes done in files in this post.
First, my camera topics are:

/cam1/depth_image_nvblox
/cam1/camera_info_sync
and my camera frame id is > "camera_link_optical"

Topics mismatch
I have remapped two topics in https://github.com/NVIDIA-ISAAC-ROS/isaac_ros_nvblox/blob/v2.0.0/nvblox_examples/nvblox_examples_bringup/launch/nvblox/nvblox.launch.py

and changed frame id in config file given at https://github.com/NVIDIA-ISAAC-ROS/isaac_ros_nvblox/blob/v2.0.0/nvblox_examples/nvblox_examples_bringup/config/nvblox/specializations/nvblox_realsense.yaml

I have checked weather the data is arriving and proper? It seems perfect as shown in below image.

A missing link between the sensor frame and the mapping frame on the TF tree.
Once I launched the node, I checked TF tree. Here is the image showing my TF tree.

Let me know If I missed to provide any information.

Note that I have tested it in both v.2.0 and v0.31.0-dp.

Hi @alexmillane any inputs/suggestions ?

Hi, @naitiknakrani-eic can you post the following so we can try to reproduce this?
Rosbag containing:

real sense stereoimages
Tof images
Tf
Launch file
Nvblox config file

Thank you @alexmillane and @ashwinvkNV for your support. The time sync between depth and camera_info works for us and nvblox gave voxel map.


A configure script for ARM

I am starting on proper work to ensure that "make" in dist/gcc-compatible works on ARM platforms and compiles as much as we can.
In addition to build complexities, this already uncovered several errors, such as calls into ASM symbols coming from Vale not guarded behind if EverCrypt.TargetConfig.x64
I have not fixed these yet but I'd like to this landed so that I can setup github actions to, essentially, go run make make in dist/gcc-compatible and make sure our code continues to compile on ARM.

Thanks for the reviews!


Prepare for Belmont.RELEASE

Following artifacts/apps need to be released as part of Belmont RELEASE release train.

[x] app starters core - 1.2.0.RC1
[x] timestamp - 1.2.0.RC1
[x] jdbchdfs-local - 1.2.0.RC1
[x] composed-task-runner - 1.0.0.RC1
[x] spark-client - 1.2.0.RC1
[x] spark-cluster- 1.2.0.RC1
[x] spark-yarn- 1.2.0.RC1
[x] app-starters-release - Belmont.RC1

Belmont.RELEASE is complete.


Add datapane to conda-forge?

Thank you for developing this great package! I plan to add it as a dependency for my geemap Python package. Do you plan to make datapane available on conda-forge? Thanks.

Hi there, thanks for your comment - yes absolutely, we're planning to try and get it into conda-forge asap, hopefully this week!

Package accepted by conda-forge, waiting for it to become available - can track in https://github.com/conda-forge/datapane-feedstock

Awesome! Looking forward to it.

Now live on conda-forge!
https://anaconda.org/conda-forge/datapane - can install as conda install -c conda-forge datapane


Questions about the config file and the results

Dear author, thanks for the great work! I have a question with the config file and the results. Is the training epoch is set with 300, the batch size is set with 4, the datasets SD7K full dataset? If so, is there a demand for GPU? And if possible, could I know more details with the training process? Best regards.

Hi, thanks for your question. The training epoch is set to 300 and the batch size is normally set to 4 or 8. The params in the config file are not always correct.

Thanks for the reply! And which epoch is the usually best epoch? I try to reproduce this work these days, but just get 20 in psnr, and could not get more than 28 in psnr.

Could you please give me more details with the training process, like the training log or the tensorboard and so on?

We use four A6000, batch size with 4 and epoch with 300. Normally when the epoch comes to around 280, you get the best performance. Also you can try using our released pretrained models.

Could you please provide your configuration you set now? I am happy to help you.

Great thanks, I have sent my wechat through the email, hope that we could have a further discussion!

Hi, this is my configuration file, could you please have a look for me?

Hi, sorry for late reply. I think discussing here would be better cuz this could be a reference for those who have the same question.

That's fine, thanks

Seems like your config is good, did you use full dataset for training?

@xuhangc

Yeah, I use the full dataset for training.

I think your config is all good. GPU is only related to the training speed.

Then I can't think of where the problem is...I think we could help you with a training log, please give us a few time.

Let's discuss on Wechat and get back to here after we find the problem.

That's fine, thanks

Issue closed since the problem has been solved, we've provided log and details. Feel free to reopen it again.

The question has been finished, thanks!