sh: vite: not found

Describe the bug
I am following the official "Hello World" tutorial at https://vitejs.dev/guide/#scaffolding-your-first-vite-project
I expect sth like:
  VITE v4.4.9  ready in 332 ms

  ➜  Local:   http://localhost:5173/
  ➜  Network: use --host to expose

I got instead:
sh: vite: not found

Note: looks linked to #11968
Reproduction
Not relevant
Steps to reproduce
To reproduce via Docker:
docker run \
  -e NODE_ENV=production \
  --entrypoint sh \
  node:18-alpine \
  -c "npm create -y vite@latest my-app -- --template react && cd my-app && npm install && npm run dev"

Note: the issue disappears with NODE_ENV=development.
System Info
System:
    OS: Linux 5.15 Alpine Linux
    CPU: (4) arm64 unknown
    Memory: 5.21 GB / 7.67 GB
    Container: Yes
    Shell: 1.35.0 - /bin/ash
  Binaries:
    Node: 18.14.2 - /usr/local/bin/node
    Yarn: 1.22.19 - /usr/local/bin/yarn
    npm: 9.5.0 - /usr/local/bin/npm

Used Package Manager
npm
Logs
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Make sure this is a Vite issue and not a framework-specific issue. For example, if it's a Vue SFC related bug, it should likely be reported to vuejs/core instead.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

npm i skips devDependencies when NODE_ENV=production is set and because the create-vite template includes vite in devDependencies, Vite won't be installed.
https://docs.npmjs.com/cli/v9/commands/npm-install#:~:text=With the --production,to a project.


Make build error

run make build at first when i clone the sei repo

@VegeBun-csj i seem to be unable to repro this. Can you try make install ?

Fixe with apt-get install cmake gcc make


Creating setfable functions?

in c++ you would return a value by reference (&), when you want to be able to set it.
returning by reference seems not to be possible in dale ((fn (ref int) …))
will this be possible?
if not, it's no problem, I'd just write macros for my accessors

On Thu, Sep 01, 2016 at 08:05:50AM -0700, Fabio Krapohl wrote:

in c++ you would return a value by reference (&), when you want to be able to set it.
returning by reference seems not to be possible in dale ((fn (ref int) …))
will this be possible?
if not, it's no problem, I'd just write macros for my accessors

This is only supported insofar as a function can return a pointer,
which can then be used as the first argument to setf.  I don't know of
any compelling use cases for reference return, and it would likely
require a 'proper' (more C++-like) implementation of references.
Compared with the current model, where reference parameter types act
as pointers within procedures, that would complicate things in the
language core significantly.


feat(explorer): display only the top 10 validators in homepage

Description of change
Please write a summary of your changes and why you made them.
Links to any relevant issues
Closes #4573
Type of change
Choose a type of change, and delete any options that are not relevant.

Bug fix (a non-breaking change which fixes an issue)
Enhancement (a non-breaking change which adds functionality)
Breaking change (fix or feature that would cause existing functionality to not work as expected)
Documentation Fix

How the change has been tested
Describe the tests that you ran to verify your changes.
Make sure to provide instructions for the maintainer as well as any relevant configurations.
Change checklist
Tick the boxes that are relevant to your changes, and delete any items that are not.

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

Can we try to make Popular Packages and Top Validators spaced equally?

Maybe by moving the View All button to the top right it can be aligned easier with the Popular Packages since that one has the filters on the top right.


Bug: inline images are not being exported to Markdown at all (verified in playground)

I went to https://playground.lexical.dev/, inserted an inline image, it gets displayed okay. I then press the button for converting to Markdown. The markdown string contains no info about the image.
I understand that there often is no 1:1 correspondence between the JSON editor state and markdown, but images (inline or URL-based) can somehow be represented by markdown (e.g. by slightly abusing Markdown and using datauri in the ![](imagedatauri) syntax or by returning a dictionary of attachment files along with markdown string which are then addressed by image tags)
I've checked, regular "Insert Image" (non-inline) gets converted to markdown using the datauri syntax, but somehow inline image is not. I think it'd be better in this case to trim the precise inline formatting and still preserve the presence of image instead of the current behavior of completely stripping the image (probably a markdown transformer for InlineImageNode would need to be added to packages/lexical-playground/src/plugins/MarkdownTransformers/index.ts).
( Basically, I'm looking to create some basic rich-text editor like https://telegra.ph (quill-based), but using markdown + file attachments as backend (e.g. thing a GitHub repo containing images and markdown files) )

Maybe the better compromise would be to include these images in Markdown as regular images (even if at inverse conversion they would be come regular images and not inline images), or maybe as images with some minor custom HTML

After clicking on M↓ nothing gets exported to markdown at all:

Also, caption is not being exported to DOM for non-inline images:

Needs a FigurePlugin


get.keptn.sh not working for 0.8 on MacOs

When executing curl -sL https://get.keptn.sh | KEPTN_VERSION=0.8.0-rc1 bash on MacOS it doesn't work as the script tries to pull the CLI with the name "macOS" in it, which doesn't exist. The actual file contains the name "darwin" instead of "macOS".
Error log:
 ~ % curl -sL https://get.keptn.sh | KEPTN_VERSION=0.8.0-rc1 bash

We'll install specified Keptn version 0.8.0-rc1
Determined target architecture x86_64
Downloading keptn 0.8.0-rc1 for OS macOS with architecture amd64 from GitHub: https://github.com/keptn/keptn/releases/download/0.8.0-rc1/keptn-0.8.0-rc1-macOS-amd64.tar.gz
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
curl: (22) The requested URL returned error: 404
An error occured while trying to download keptn from GitHub. Please manually download a release from https://github.com/keptn/keptn/releases.

Potential fix:

Either update the get.keptn.sh file or the script that creates the CLI tarball for macOS.

Thanks, I'll fix the script!


Add support for project paths with spaces

I made some small changes to the AppleScript code to support space characters in the project path when it invokes the Python script via a shell command.

omg I was simply trying to change your script to use quoted form instead of manually appending the quotes and I absolutely bricked this. I'm sorry!! I'm very used to being able to just manually tweak pulls instead of asking folks to make changes but find the way that github requires that I do that totally baffling.
Anyway uh, I'm truly not sure how to un-brick the current pull! I'm sorry! If you'd like to re-submit the pull (with or without my change to use quoted form) I'll gladly just merge it, or if it's not worth the trouble for you I'll just apply the relevant changes to main directly.
So sorry for the trouble!

omg I was simply trying to change your script to use quoted form instead of manually appending the quotes and I absolutely bricked this. I'm sorry!! I'm very used to being able to just manually tweak pulls instead of asking folks to make changes but find the way that github requires that I do that totally baffling.
Anyway uh, I'm truly not sure how to un-brick the current pull! I'm sorry! If you'd like to re-submit the pull (with or without my change to use quoted form) I'll gladly just merge it, or if it's not worth the trouble for you I'll just apply the relevant changes to main directly.
So sorry for the trouble!

I opened a new pull request #9 that should fix the issue. I apologize for the delay! Git can be confusing sometimes. 😅


How is this compared to mcfgthread?

https://github.com/lhmouse/mcfgthread/

That will depend on the aspects that you are comparing.

Performance should be fairly similar for all mutually-supported versions of Windows, though it will vary depending on number of threads, amount of contention, and the version of Windows for which you are compiling. (Note: I have not benchmarked this)
Compatibility is a bit different; mingw-std-thread supports versions of Windows that mcfgthread does not. In particular, mingw-std-threads supports Windows XP and Vista, while mcfgthread does not.
Ease of use varies; mingw-std-thread can be used simply by including the headers, while (as I recall) mcfgthread requires modifying the compiler. mingw-std-thread also does not require calling a specialized exit function (see mcfgthread's notes section in the readme).

In short, if you need XP or Vista compatibility, choose mingw-std-thread. If not, use whatever you think will minimize bugs (eg. forget to call __MCF_exit()) or maintenance when writing library code (eg. conditionally including the mingw.*.h headers based on whether Windows is the target).


why there is no uninstall rule for server and cli?

So I wanted to move to the new sysrepo from devel branch.
For this, I am guessing I need Netopeer2, libyang and libnetconf2 from devel branch also.
How do I uninstall netopeer2 so that I can install netopeer2 from devel branch?

Hi,
there is now.
Regards,
Michal

Hi Michal,
I am overwhelmed by the real-time change requests, you are providing.
Thank you for the great work you guys are doing.
Is it possible to provide same for keystored also.
thanks
Govind.

Hi,
okay, I have added it for keystored as well even though it will be obsolete in a week or so (the new netopeer2 does not need it).
As for the changes, it all depends on whether I am working on something I do not want to interrupt and on how difficult the change is. This was trivial.
Regards,
Michal


Api Calls added for exchange rates

added api calls for both standard currency exchange conversion and pair conversion and logging them out

Great work.


V0.8 for dogfooding wechaty-puppet-official-account

use wechaty-puppet-official-account !

Let's try our branding new and awesome wechaty-puppet-official-account module for managing WeChat Official Account!


Fix polygon generation in BitMap

While working on #68380 some issues with the way BitMap does polygon generation showed up, I have tried to solve them here
There are a few things that have been changed:

An oversight in fill_bits that caused connected pixels to not be filled, causing duplicate polygons being generated
The procedure for traversing the bits and generating the polygon in _march_squares

For the fill_bits you could have a the following, starting at x and the red square would not be filled, causing in some cases overlapping polygons as the function would start over in the red square and create a polygon overlapping with the white area:

There were some issues with the traversing the squares to generate the polygon as well, causing entire areas to be ignored:

There were also cases where the polygon would get self intersections, like this:

Which break the triangulation
When the number of points is greater than the pixels in the BitMap it now returns an empty result rather than the points so far, I don't see how a result in that case is valid and shouldn't be returned
So my solution is this:

Fix the popping in the fill_bits, it seems to have been an oversight as the code says "The next loop over j must start normally." which I take to mean subtracting one, and from what I can see it doesn't cause any problems
Fix the behavior in diagonal cases:

Where traversal didn't take into account the direction we are coming from, as the direction to go is (as far as I can figure out) based on the direction as we traverse it counter-clockwise
Fix cases of self-overlap caused by my fix as well as other cases original to the code, this splits polygons at points where it reaches the same point again, which in some cases breaks triangulation (as it uses ear-clipping)

I had a few other approaches but this approach avoids filling in any bits that weren't filled, making assumptions or decisions on where should be included to solve the degenerate cases, at the cost of splitting polygons at these self overlap points
There might still be some issues in the reduce function that I haven't identified but it seems that it now generates valid polygons for general cases
For testing purposes here is a project where you can play around with a bitmap and see the different results:
BitPR.zip
Instructions:

Left mouse fills square
Right mouse clears square
Space gets polygons
Tab inverts bits
Escape clears

Fixes #31675

For cases when you might want polygons that are touching to be grouped I'm considering adding a "element" argument to the clip_opaque_to_polygons function to return what connected component a resulting polygon belongs to, but I'm not sure how much use that would get

Nested polygons are generated, which might be an issue in some cases, i.e. when a polygon has a hole that contains another polygon, the fix for that could be relatively straight forward but as it doesn't generate any directly broken behavior I haven't touched it here, and would probably require options or a discussion over how to exactly navigate that

To explain the stack and the processing of "crossing points" a bit more in detail:
In the case of the diagonals we can get a result where the polygon reaches the same point twice, which can cause cases that are not triangulatable, and that are always generating broken triangulations, in this case we track a stack of ranges between these points, and when we reach such a point a single time we roll up the range since the last case of such a point and the current point so that it becomes a new polygon

Looks great! Do you think you can provide some unit tests which check the edge cases you fixed?

@Geometror never written any before but I can take a stab at it!

You can orient yourself on the existing test cases in test_bit_map.h (starting from line 398, test case "[BitMap] Clip to polygon").

There added three tests, one for each diagonal, to ensure it catches everything, and splits correctly, and one for this shape to ensure it doesn't split that even though there is a crossing point in it:

Merged the diagonal test cases and added a test case to verify the fill fix (the y shape in the description)

Also I'm pretty sure adding the y position of the rectangle to the result is incorrect, so fixed that, correct me if that's an incorrect assumption

(Sorry didn't realize that removed the previous reviewers)

Thanks!


After caching config: crash > There is no existing directory at /laravel_folder/storage/logs and its not buildable: Permission denied

Laravel Version: 7.16.1
PHP Version: 7.3.19
Database Driver & Version: mysql

Description:
I just noticed a buggy behaviour when generating the config:cache via php artisan config:cache
after executing it, laravel gave this error message:
There is no existing directory at /laravel_folder/storage/logs and its not buildable: Permission denied
Funny thing is: The folder is there, has the right permissions and after deleting the config cache file it works again...

Steps To Reproduce:
In my case:

php artisan config:cache
navigate to laravel website...

I would not write a Issue here if there weren't other persons having the problem:
https://stackoverflow.com/questions/51041196/there-is-no-existing-directory-at-storage-logs-and-its-not-buildable-permissio
As far as I can see I can't point at the error: Is it the way the config cache is generated? why does it behave different than when the config isn't cached? Why does the error point to a folder, which apparently exists?

This seems like an obvious permission error. Please try a support channel:

Laracasts Forums
Laravel.io Forums
StackOverflow
Discord
Larachat
IRC

Thanks.

Seems but it isn't...
Why does it work then when the config is not cached (without changing permissions)?
If it was a permission error I would get the exception with or without caching...
I can reproduce the error as much as I want:
"php artisan config:cache" > Installation throws error
delete bootstrap/cache/config.php > Installation works
"php artisan config:cache" > Installation throws error

In case someon googles about that and arrives here like me, I figured out that the reason for that issue in my case was that the SSH connection to the shared hosting had a chroot and running php artisan config:cache from that chrooted environment put a wrong path in bootstrap/config.php...
The solution I’m about to implement is instead of doing php artisan config:cache from commandline, I will do a curl to a DeployController that calls artisan from web environment, thus do the config:cache correctly...

install php-dom
sudo dnf install php-dom


Error Running GiniClust - filter function

When I run Giniclust with my dataset It stop in the filter function.
My data are avaliable:
https://drive.google.com/drive/u/0/folders/1_07OCDjC9WL-MfUWGe5At_wP0G6wtjqG

I am Rui from GC Yuan's lab. It seems like there's conflict of "filter" function between "MAST" and "stats" packages in R. You can add package name before "filter" with "::" to solve this problem. If you could share your code, it would be better. Thanks.

Hi Rui,
Thank you for your reply.
When I run GiniClust using the sample dataset (Data_GBM.csv) using the command line:
Rscript GiniClust_Main.R -f Data_GBM.csv -t RNA-seq -o GBM_results
The code works fine  and I don't receive any error messages.
After that, I tried to use my dataset as a parameter with the following command line:
Rscript GiniClust_Main.R -f GSE36552_table.csv -t RNA-seq -o GSE36552_results
When I use my dataset as a parameter the program execution is interrupted when the script is running the filter task.
What do you think? Do you have any insights about it?
Thanks once again.

Hi Cassalhoo,
I tested the dataset you use and found GiniClust works on your dataset. GiniClust will report a cluster result file in your output directionally. If you could not find "GSE36552_results /GSE36552_table.csv_clusterID.csv" you can try GiniClust2.


: load current configuration error

1 when current config has no profiles, cli cannot run normly

 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.宏量 seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

1 when current config has no profiles, cli cannot run normly

Thank you for submitting the code, but I have not yet clarified what you said, please follow the issue template to submit a description, please also include the test code.
Thank you again for submitting the code. If have any other questions, please contact us.
Thank you for your attention to Alibaba Cloud CLI Project!

Thank you for submitting the code, but I have not yet clarified what you said, please follow the issue template to submit a description, please also include the test code.
Thank you again for submitting the code. If have any other questions, please contact us.
Thank you for your attention to Alibaba Cloud CLI Project!

Command Format: aliyun configure delete --profile $profilename; aliyun

Execution Command: aliyun configure delete --profile default; aliyun

Output: ERROR: load current configuration failed unknown profile default, run configure to check

when execute aliyun configure delete --profile default, the configure file will become like
{
    "current": "default",
    "profiles": [],
    "meta_path": ""
}

and when execute aliyun command, LoadProfile cannot get the 'default' profile,  then an error will be thrown

Thank you for submitting the code, but I have not yet clarified what you said, please follow the issue template to submit a description, please also include the test code.
Thank you again for submitting the code. If have any other questions, please contact us.
Thank you for your attention to Alibaba Cloud CLI Project!

Command Format: aliyun configure delete --profile $profilename; aliyun
Execution Command: aliyun configure delete --profile default; aliyun
Output: ERROR: load current configuration failed unknown profile default, run configure to check

when execute aliyun configure delete --profile default, the configure file will become like
{
    "current": "default",
    "profiles": [],
    "meta_path": ""
}

and when execute aliyun command, LoadProfile cannot get the 'default' profile, then an error will be thrown

Many uses of CLI tools are based on profile, so when the profile is empty, we will prompt that there is no default profile. We want users to know which account they are using to avoid loss, and forcing new profiles is invalid profile, we do not recommend use.

Thank you for submitting the code, but I have not yet clarified what you said, please follow the issue template to submit a description, please also include the test code.
Thank you again for submitting the code. If have any other questions, please contact us.
Thank you for your attention to Alibaba Cloud CLI Project!

Command Format: aliyun configure delete --profile $profilename; aliyun
Execution Command: aliyun configure delete --profile default; aliyun
Output: ERROR: load current configuration failed unknown profile default, run configure to check

when execute aliyun configure delete --profile default, the configure file will become like
{
    "current": "default",
    "profiles": [],
    "meta_path": ""
}

and when execute aliyun command, LoadProfile cannot get the 'default' profile, then an error will be thrown

Many uses of CLI tools are based on profile, so when the profile is empty, we will prompt that there is no default profile. We want users to know which account they are using to avoid loss, and forcing new profiles is invalid profile, we do not recommend use.

Aggree with you, but the problem is that if user occasionally delete the default one, then he/she cannot   run the command unless manually delete the json file, most time users don't know where it is.

Thank you for submitting the code, but I have not yet clarified what you said, please follow the issue template to submit a description, please also include the test code.
Thank you again for submitting the code. If have any other questions, please contact us.
Thank you for your attention to Alibaba Cloud CLI Project!

Command Format: aliyun configure delete --profile $profilename; aliyun
Execution Command: aliyun configure delete --profile default; aliyun
Output: ERROR: load current configuration failed unknown profile default, run configure to check

when execute aliyun configure delete --profile default, the configure file will become like
{
    "current": "default",
    "profiles": [],
    "meta_path": ""
}

and when execute aliyun command, LoadProfile cannot get the 'default' profile, then an error will be thrown

Many uses of CLI tools are based on profile, so when the profile is empty, we will prompt that there is no default profile. We want users to know which account they are using to avoid loss, and forcing new profiles is invalid profile, we do not recommend use.

Aggree with you, but the problem is that if user occasionally delete the default one, then he/she cannot run the command unless manually delete the json file, most time users don't know where it is.

There are two things to explain about the situation you mentioned. First, deleting the profile requires specifying the profile name. If the user deletes it by mistake, this is a user's problem. Second, when the user accidentally deletes the configuration, there is no need to delete the json file. You can use aliyun configure set [--profile <profileName>] ... to set the configuration file.

Thank you for submitting the code, but I have not yet clarified what you said, please follow the issue template to submit a description, please also include the test code.
Thank you again for submitting the code. If have any other questions, please contact us.
Thank you for your attention to Alibaba Cloud CLI Project!

Command Format: aliyun configure delete --profile $profilename; aliyun
Execution Command: aliyun configure delete --profile default; aliyun
Output: ERROR: load current configuration failed unknown profile default, run configure to check

when execute aliyun configure delete --profile default, the configure file will become like
{
    "current": "default",
    "profiles": [],
    "meta_path": ""
}

and when execute aliyun command, LoadProfile cannot get the 'default' profile, then an error will be thrown

Many uses of CLI tools are based on profile, so when the profile is empty, we will prompt that there is no default profile. We want users to know which account they are using to avoid loss, and forcing new profiles is invalid profile, we do not recommend use.

Aggree with you, but the problem is that if user occasionally delete the default one, then he/she cannot run the command unless manually delete the json file, most time users don't know where it is.

There are two things to explain about the situation you mentioned. First, deleting the profile requires specifying the profile name. If the user deletes it by mistake, this is a user's problem. Second, when the user accidentally deletes the configuration, there is no need to delete the json file. You can use aliyun configure set [--profile <profileName>] ... to set the configuration file.

Now the problem is I cannot use the command to recover

Thank you for submitting the code, but I have not yet clarified what you said, please follow the issue template to submit a description, please also include the test code.
Thank you again for submitting the code. If have any other questions, please contact us.
Thank you for your attention to Alibaba Cloud CLI Project!

Command Format: aliyun configure delete --profile $profilename; aliyun
Execution Command: aliyun configure delete --profile default; aliyun
Output: ERROR: load current configuration failed unknown profile default, run configure to check

when execute aliyun configure delete --profile default, the configure file will become like
{
    "current": "default",
    "profiles": [],
    "meta_path": ""
}

and when execute aliyun command, LoadProfile cannot get the 'default' profile, then an error will be thrown

Many uses of CLI tools are based on profile, so when the profile is empty, we will prompt that there is no default profile. We want users to know which account they are using to avoid loss, and forcing new profiles is invalid profile, we do not recommend use.

Aggree with you, but the problem is that if user occasionally delete the default one, then he/she cannot run the command unless manually delete the json file, most time users don't know where it is.

There are two things to explain about the situation you mentioned. First, deleting the profile requires specifying the profile name. If the user deletes it by mistake, this is a user's problem. Second, when the user accidentally deletes the configuration, there is no need to delete the json file. You can use aliyun configure set [--profile <profileName>] ... to set the configuration file.

Now the problem is I cannot use the command to recover

Hi, I noticed that your program name is aliyun-cli, do you use the go version or the python version? If it is the python version, we have stopped supporting it; if it is the go version, your set command is missing the authentication information such as AK. If you only specify the --profile flag, the effect is to set the default profile, and because your profile does not exist, it reports an error.


[BUG] Hyperlink are not correctly displayed

Describe the bug
Hyperlink are not correctly displayed
To Reproduce
Here is a reproducible example:
import pytermgui as ptg

with ptg.WindowManager() as manager:

    windowBody = ptg.Window(
            """
            Hello there

            This is just some test with an [~https://google.com]hyperlink[/~] just right here

            And some other test.
            """
    )

    manager.add(windowBody)

Expected behavior
Expecting to have a window with
            Hello there

            This is just some test with an hyperlink just right here

            And some other test.

displayed
Screenshots

Here is the result I got
System information

Possible cause
It seems that the hyperlink tag are not correctly rendered by ptg
Possible solution
The issue should be around here from what I understand

The one line problem come from the height being too small, my bad. But the PlainToken just before the link is not displayed. If some style is applied to it, it appear
Example code
v1
import pytermgui as ptg

with ptg.WindowManager() as manager:

    windowBody = ptg.Window(
            """
            Hello there

            This is just some test with an
            [~https://google.com]hyperlink[/~] just right here

            This is some other text with an other example [~https://example.com]here[/~]

            And some other test.
            """,
            height=50,
            width=80,
    )

v2
import pytermgui as ptg

with ptg.WindowManager() as manager:

    windowBody = ptg.Window(
            """
            Hello there

            [bold]This is just some test with an[/]
            [~https://google.com]hyperlink[/~] just right here

            This is some other text with an other example [~https://example.com]here[/~]

            And some other test.
            """,
            height=50,
            width=80,
    )

    manager.add(windowBody)

Results
v1

v2

The root of the problem seem to come from tokenize_ansi (called from Label.print() which call ptg.break_line)

Fixed by #108 (thanks!)


State column names singular. pool_labeled_function.

I turned the state column names to singular and added a function to get the pool and labeled record_ids in one query. I did not connect the function to the rest of the code yet.

Thanks for this PR. I think this will simplify some work on the middleware and top layers of ASReview (and maybe break some things for now). I will test this after the lint errors are resolved.

Thanks for this effort. Makes ASReview more future-proof!


Add convert-3d-models extension

Description

An extension that is used to convert 3D models from one file format to another. Utilizes FreeCAD as the 3D engine, which is required to be installed before use. Given the size of FreeCAD, I opted to not include the distribution directly, but rather provide detailed installation instructions mimicking that of FreeCAD documentation.
Screencast

https://github.com/raycast/extensions/assets/12619586/e535ba24-1723-43c7-a83f-dddaf96c2f85
Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[ ] I ran npm run build and tested this distribution build in Raycast
[x] I checked that files in the assets folder are used by the extension itself
[x] I checked that assets used by the README are placed outside of the metadata folder

Hi 👋
I made the CI happy, could you check my corrections to ensure I didn't break something 🙂

Hi 👋
I made the CI happy, could you check my corrections to ensure I didn't break something 🙂

The changes run perfectly fine! Thank you there! But... I'm so sorry, I broke it again. 😟

It's because "inputMethod": "Finder" | "Path Finder", - with above it's something else since it will be the name of frontmost application

It's because "inputMethod": "Finder" | "Path Finder", - with above it's something else since it will be the name of frontmost application

All good now! inputMethod was a preferences item, and once the explicit type definitions in preferences.ts got removed, I need to explicitly cast it to a string.

Could you look into the CI errors? It can be undefined if the application is removed after selecting it

Could you look into the CI errors? It can be undefined if the application is removed after selecting it.

Sorry about that! This should all be good now!


Maximum production(July Long challenge)

Description
Code for maximum production(July Long challenge) added in java
Fixes #1685
Replace issue_no with the issue number which is fixed in this PR
Type of change
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update
[ ] Documentation Update

Checklist:

[x] My code follows the style guidelines(Clean Code) 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 created a helpful and easy to understand README.md
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests/screenshots(if any) that prove my fix is effective or that my feature works

@Ananya-Misra current challenges are not accepted

@Amit366 Okay.


removed misplaced zip files from xhibit portal

Tracked upstream by #8393 in the modernisation platform repository.
Removes unneeded zip files prior to our .gitignore ignoring all zip files in the repository by default. See this Slack conversation for confirmation.

@dms1981 Thank you for deleting those archive files. Could you please add the path to .gitignore file as well? Thank you.


About the meaning of level

I'm sorry I wanna ask 3 very simple questions:

what does the "level" mean? Different levels have different visual background?
So if I set up the "num_level=500" ,does that mean I will train an agent on 500 different visual background game?
And how can I change to the generalization mode? just let the "num_level=0"?

Thx for your reply!~

For many of the environments they don't have just different visual backgrounds, but also different maps (so like boxes in difference places, see coinrun for a good example)
Yeah, you can try this out with the interactive script, by setting num_level=1 or num_level=2.
I believe that is correct.

See also https://github.com/openai/procgen/issues/56


Questions about the training time

Thanks for the interesting work!
May I know how long it takes to train this model, and what GPU configuration is on it?

I want know too... The paper did not presented more about GPU.


Citation for Pygeos

We are looking into a new numpy paper, and I thought of listing pygeos as one of the current custom dtypes, which would probably get along better with NumPy if we had new dtypes. Would the EuroScipy talk be good to cite?

Hi @seberg, good to hear that our work is picked up. I am not aware of a EuroScipy talk (maybe @jorisvandembossche ?)
I think currently the best reference would be my blogpost at https://caspervdw.github.io/Introducing-Pygeos/

Yes, I didn't do a talk at EuroScipy (only at FOSS4G Belgium, https://jorisvandenbossche.github.io/talks/2019_FOSS4GBE_pygeos/#1, but @caspervdw's blog is a better reference).
If you want an actual citation reference, we could make a Zenodo doi ?

I can basically flip a switch on Zenodo to make the citation (did it recently for trackpy). Would be nice to have! Shall I do it @jorisvandenbossche ?

That sounds good!

I have to admit, I think the mention was dropped from our numpy paper draft with the section that mentioned it being shortened, but I suppose it is good to have in any case :).

I flipped the switch at Zenodo, a DOI should be automatically created on the next release.

Done: see Readme for the new badge


BetterNether Stalagnate not deploying

Describe the Bug
On our server, we have been making farms with Create for various Better Nether flora. It seems like the Create deployer simply refuses to place Stalagnate (whereas we haven't had a problem with Nether Sakura or other trees).
I don't know why the placement code would do anything weird; growing is where Stalagnate has to look really far up or down. We are just trying to use a deployer to put it onto netherrack. I've tried various angles and it just doesn't want to place.
We're on Minecraft 1.19.2, Better Nether 7.1.2, Create version 0.5.0g-796. But, I just tried it it Minecraft 1.20.1, Better Nether 9.0.10, Create 0.5.1-d-build.1161 with the same behavior.
Paul at Better Nether speculated that a datapack is needed to list items that the deployer can use / blocks that it can place. I didn't find anything like that; this doesn't seem like a 'deployer recipe' sort of situation since it's just placing a sapling. Should we try making a deployer recipe?
Reproduction Steps

Load up Better Nether and Create in 1.19 or 1.20, with dependencies
Set up a powered deployer targeting an air block next to netherrack
Give the deployer stalagnate saplings
Result: deployer moves

Expected Result
Stalagnate should be placed, just like other Nether saplings would be in this situation
Screenshots and Videos
No response
Crash Report or Log
No response
Operating System
Windows 11
Mod Version
0.5.1d
Minecraft Version
1.20.1
Other Mods
Bether Nether 9.0.10
Additional Context
No response

screenshots of the setup?

Hi Tropheus,
I re-did some tests to get screenshots, and noticed a distinction I wasn't making previously.
The placement works as expected with a stationary deployer; the misbehavior comes with deployers hooked to a bearing or gantry.
For comparison, "hook mushrooms" from the same mod work fine in both cases.

(Setup on the left, with a deployer sitting still, works; setup on the right, with a mechanical bearing, doesn't.)


CAMS CAS Parsing support for Dividend payout transaction

Hi,
CAMS CAS has Dividend Payout transactions like below.

TRANSACTION_RE doesn't match since "units", "nav" and "balance" columns are missing in these entries.
Can the parser be updated to handle the Dividend Payout transaction? Not sure if Karvy CAS has similar format.
Also, the Dividend amount may need to be negated for XIRR calculations.

Many thanks! shall upload a fix for testing.

Updated TRANSACTION_RE in #12 to parse dividend transactions. This should work hopefully.

Works well..
Dividend Payout and STT Paid transaction are picked up now... Thank you.
Can another flag be added for Dividend Reinvestment for transactions like below (is_dividend_reinvestment).

I seem to have all sorts of transactions from earlier days!!! :)

Many thanks. These are really helpful. really difficult to come across use cases that cover all these scenarios. shall update the branch soon.
I have already been planning for adding a transaction type enum in the json with values PURCHASE, REDEMPTION, DIVIDEND_PAYOUT, DIVIDEND_REINVEST, TAX/STT, MISC but was not sure if these would cover all use cases.

Added commits to the branch fix/dividends to flag dividend reinvestments. Also added dividend_rate property in case of  dividend transactions.
Please let me know if this works fine for your files.

Code working fine... Everything being parsed as expected.
"I have already been planning for adding a transaction type enum in the json with values PURCHASE, REDEMPTION, DIVIDEND_PAYOUT, DIVIDEND_REINVEST, TAX/STT, MISC but was not sure if these would cover all use cases."
--> This will be very helpful in further analysis of parsed date.

Many thanks for testing the code. It is really appreciated.
I'll add transaction type detection in the next version. just needs to finalise the different types of transactions possible and the classification logic.

I have already been planning for adding a transaction type enum in the json with values PURCHASE, REDEMPTION, DIVIDEND_PAYOUT, DIVIDEND_REINVEST, TAX/STT, MISC but was not sure if these would cover all use cases.

Below are additional categories I can think of
SIP_PURCHASE, SWITCH_REDEMPTION, SWITCH_PURCHASE, MERGER_REDEMPTION, MERGER_PURCHASE


torch version >=1.12.0 to be compatible with torch.amp.autocast

Hi, torch.amp.autocast is used in the BLIP2-T5 model, while this API is added in PyTorch 1.12.0+.
An updated requirement for the torch version may be needed, not only to solve the current bugs, also for compatibility with future advanced functions.

https://github.com/salesforce/LAVIS/commit/baad2d7c8df599d8d9b081ba2e946626eaa2dc34 should now fix the issue.


Don't generate methods pertaining to classes and claims

Description of your changes

Checklist

I have:

[x] Run make reviewable to ensure this PR is ready for review.
[x] Ensured this PR contains a neat, self documenting set of commits.

Tested via https://github.com/crossplane/provider-gcp/pull/254


fix(Checkbox): Putting label at root level so everything is clickable

Checkbox isn't clickable on the actual checkbox. This isn't great.
To Test:
Go here: https://seekinternational.github.io/seek-asia-style-guide/checkbox
Click on the label, and see it select and unselect. Now click on the checkbox, does it work? It doesn't for me.
Now, load up this PR, and do the same thing. Hopefully clicking on the actual checkbox should work again.

@ashin I've another PR fix for that as #453 😅


Updated links to https

Updated a number of links from HTTP to HTTPS.

Hi @altonius, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by .NET Foundation and real humans are currently evaluating your PR.
TTYL, DNFBOT;


(maint) Properly stub configurer methods to prevent errors in tests

When running Puppet::Configurer::run_internal in spec tests,
attempting to send the report and write the class file error out
due to arbitrary files not existing on the filesystem. Stub
the methods responsible for these calls out so the error we are
trying to actually test can more cleanly be tested.

:+1:


Increase inotify limit

What this PR does / why we need it:
Based on several issues with reaching the inotify limit (~8k) this PR increases the limit for all nodes.
Upstream issues:

https://github.com/kubernetes/kubernetes/issues/7815
https://github.com/kubernetes/kubernetes/issues/10421
https://github.com/kubernetes/kubernetes/issues/74669

Special notes for your reviewer:
Increase the inotify limit to 1048576

/retest

/lgtm
/approve

/retest


Remove unnecessary (and the only) RenderObject.markParentNeedsLayout override

Nobody calls it except for markNeedsLayout, and the render object is a guaranteed relayout boundary.
Pre-launch Checklist

[ ] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[ ] I read the Tree Hygiene wiki page, which explains my responsibilities.
[ ] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[ ] I signed the CLA.
[ ] I listed at least one issue that this PR fixes in the description above.
[ ] I updated/added relevant documentation (doc comments with ///).
[ ] I added new tests to check the change I am making, or this PR is test-exempt.
[ ] I followed the breaking change policy and added Data Driven Fixes where supported.
[ ] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

Nobody calls it except for markNeedsLayout, and the render object is a guaranteed relayout boundary.

For my own understanding: We can remove this because the object is always a relayout boundary and RenderObject.markNeedsLayout only calls markParentNeedsLayout for render objects that are not relayout boundaries?
What about the instances where markParentNeedsLayout is called from markNeedsLayoutForSizedByParentChange? Or in the RenderBox.markNeedsLayout override?

Nobody calls it except for markNeedsLayout, and the render object is a guaranteed relayout boundary.

For my own understanding: We can remove this because the object is always a relayout boundary and RenderObject.markNeedsLayout only calls markParentNeedsLayout for render objects that are not relayout boundaries?
What about the instances where markParentNeedsLayout is called from markNeedsLayoutForSizedByParentChange? Or in the RenderBox.markNeedsLayout override?

Yeah for relayout boundaries markNeedsLayout doesn't call markParentNeedsLayout. Instead it adds the node to the PipelineOwner's dirty list. For this RenderObject sizedByParent is always true so markNeedsLayoutForSizedByParentChange will never be called. For RenderBox.markNeedsLayout I think we don't have to dirty OverlayPortal's render box (since its intrinsic dimensions / baseline offset don't depend on the layout of the overlay child), so the RenderBox.markNeedsLayout implementation should just work for _RenderDeferredLayoutBox.


Some improvements to the IO Framework

Added more descriptive JavaDocs to the framework.
Deprecated methods that created float arrays for every call
Added the enable/disable functionality to the gaze controller
Added code to disable unused IODevices

GearVRf-DCO-1.0-Signed-off-by: Rahul Rudradevan
<EMAIL_ADDRESS>

@parthmehta209 ?

Here is the updated 3d cursor sample:
https://github.com/gearvrf/GearVRf-Demos/pull/360


double-beep: Watch appsapk.com

double-beep requests the watch of the watch_keyword appsapk\.com. See the MS search here and the Stack Exchange search here.

appsapk\.com has been seen in 3 true positives, 0 false positives, and 0 NAAs.

Approved by Makyen in Charcoal HQ


Create values-schema mechanism to update values to conform to schema changes

Checklist

[ ] Architecture Design Records have been added as adr/*.md and appended to list in adr/_index.md.
[ ] The values-schema.yaml file and test/** fixtures have been updated to reflect code changes.
[ ] The OpenApi Schema from redkubes/otomi-api project is compatible with definitions from values-schema.yaml file.

Info
Closes #259.

I'm thinking of adding schema validation of values-changes.yaml, what do you think?

not needed as we will have to eat our own dogfood, because we have to mutate the files in .values folder with the latest entry in value-changes.yaml anyway. When that succeeds we know it is valid, and the subsequent tests will validate the resulting values anyway.

So we need to have a pre-push mechanism that checks if the current version in .values is older than the one in the schema, and then run your routine

I didn't ask for a review yet @Morriz


Cache layer for files (may be separate service backed by minio)

Current Decision - We dont need a cache layer for files. We will however remove dependency on min.io & provide some alternative adaptors


C++ wrapper lowering like inductor

The feature, motivation and pitch
Hello,
I'm wondering if FlagGems could have an implementation of a cpp wrapper similar to inductor's cpp_wrapper lowering of kernel launch code. It might help to reduce the runtime costs of launching operations from python.
Alternatives
Cudagraph / Cudagraph Trees?

Thanks for your suggestion. We are experimenting on a Triton AOT wrapper currently developed by AMD. We'll also take a look at Cpp Inductor. FlagGems employs a fairly complex wrapper & codegen procedure for most pointwise operators. This A cpp wrapper requires rewriting this part in cpp. It may take some time and we'll let you know the status.


AWS Lambda support (Docker)

Hi,
From early 2021, AWS lambda supports docker based deployment which supports upto 10GB in container image size.
https://aws.amazon.com/blogs/aws/new-for-aws-lambda-container-image-support/
Do you have a plan to support lambda based deployment using docker in the future releases?

Hello @vigneshgurusamy,
Why not 👍 Last time I checked though, there were some limitations regarding the payload size for HTTP requests, which may be limiting for some features.

See https://pkg.go.dev/github.com/awslabs/aws-lambda-go-api-proxy/echo.

Feel free to try @goyal-aman!

Hi @goyal-aman,
I found something that could be helpful: aws-lambda-web-adapter.
I've built an image using the Dockerfile below, and deployed it to AWS Lambda (including a function URL):
FROM gotenberg/gotenberg:8.3.0

# install AWS lambda web adapter
COPY --from=public.ecr.aws/awsguru/aws-lambda-adapter:0.8.1 /lambda-adapter /opt/extensions/lambda-adapter
ENV AWS_LWA_PORT=3001
ENV AWS_LWA_READINESS_CHECK_PATH=/health
ENV AWS_LWA_INVOKE_MODE=response_stream

EXPOSE 3001

ENTRYPOINT ["gotenberg", "--api-port=3001", "--log-level=debug"]

I can now invoke the Lambda via its URL, for example (in python):
import requests

lambda_url = 'https://<url_assigned_by_aws>.lambda-url.<aws-region>.on.aws'
files = {'files': open('path/to/file.doc', 'rb')}

response = requests.post(f'{lambda_url}/forms/libreoffice/convert', files= files)

The aws-lambda-web-adapter works as expected, and the request is passed to the gotenberg API.
However, the process fails due to the read-only filesystem of AWS Lambda:
{
    "level": "debug",
    "ts":<PHONE_NUMBER>.0916717,
    "logger": "libreoffice-api.libreoffice.usrliblibreofficeprogramsoffice.bin.stderr",
    "msg": "(process:21): dconf-CRITICAL **: 16:22:42.091: unable to create directory '/home/sbx_user1051/.cache/dconf': Read-only file system.  dconf will not work properly."
}

Any idea how to solve this issue?
AWS Lambda only allows you to write under the \tmp prefix.

Thanks @f-dig for your investigations!
Most I/O operations in Gotenberg are occurring within the /tmp folder. Did you mean that by \tmp prefix?
Regarding the issue, it looks like LibreOffice is trying to create files outside the user home directory (under /tmp):
https://github.com/gotenberg/gotenberg/blob/c11a2d0081c4e371a3f1bc16813d88b992881011/pkg/modules/libreoffice/api/libreoffice.go#L72
You'll have to find what are those file, and if a flag exist to specify a directory for them.

@gulien: it looks like LibreOffice uses the HOME env var to determine where to write files. I have now managed to solve the above issue by setting it to /tmp (in the definition of the Lambda function).
I also added the auto-start option to the Dockerfile:
FROM gotenberg/gotenberg:8.3.0

# install AWS lambda web adapter
COPY --from=public.ecr.aws/awsguru/aws-lambda-adapter:0.8.1 /lambda-adapter /opt/extensions/lambda-adapter
ENV AWS_LWA_PORT=3001
ENV AWS_LWA_READINESS_CHECK_PATH=/health
ENV AWS_LWA_INVOKE_MODE=response_stream
ENV AWS_LWA_ASYNC_INIT=true

EXPOSE 3001

ENTRYPOINT ["gotenberg", "--api-port=3001", "--log-level=debug", "--libreoffice-auto-start=true"]

Unfortunately, there is still an open issue. I try to convert a 25 KB .odt file, and the response content that I get back is empty:
{
'Date': 'Thu, 04 Apr 2024 08:18:49 GMT',
'Content-Type': 'application/pdf',
'Content-Length': '0',
'Connection': 'keep-alive',
'x-amzn-RequestId': '3c873107-8142-48bf-a72f-084f06723389',
'last-modified': 'Thu, 04 Apr 2024 08:18:49 GMT',
'content-disposition': 'attachment; filename="d905a9fe-037b-4302-9fa4-3c927d72be02.pdf"',
'gotenberg-trace': 'b5c111a6-39b4-476b-8806-67cab15a1dad',
'x-amzn-Remapped-content-length': '29319',
'X-Amzn-Trace-Id': 'root=1-660e6267-484293270e19c32b782a5332;parent=06fcacb40fa6e53b;sampled=0;lineage=8e2324df:0', 'accept-ranges': 'bytes',
'x-amzn-Remapped-date': 'Thu, 04 Apr 2024 08:18:49 GMT'
}

Maybe there is an intermediary service that removes the body for whatever reason? Does it works with the Chromium module?

The Chrome module fails to start:
{
    "level": "debug",
    "ts":<PHONE_NUMBER>.305541,
    "logger": "chromium.browser",
    "msg": "[0404/100013.302163:ERROR:bus.cc(407)] Failed to connect to the bus: Failed to connect to socket /run/dbus/system_bus_socket: No such file or directory\n"
}

Anyway, due to the AWS Lambda payload limit of 6 MB, it may not be the best service to use here.
I'll give AWS Fargate + API Gateway a try and keep you posted

I'm pretty sure it will work with AWS Fargate 👍 Also, a proven serverless solution for  container is Google Cloud Run. Not sure it's worth the hassle with AWS Lambda.


Wallet lock, unlock and unlock (staking only) tooltips and icons misleading

There are 3 states of the wallet. Locked, unlocked and unlocked for staking. Both locked and unlocked are showing and saying mixed signals with their tooltips and icons. Unlocked for staking is OK.
First is Locked wallet. Tooltip and icon in top right show/say locked but lock icon in collapsing menu on left shows an unlock icon.

Second is unlocked for staking. In both spots (upper right and menu icon) show yellow locked lock with correct tool tip.

Third is unlocked (PoS only unchecked). Tooltip and icon in upper right show black lock locked but say unlocked. Icon in collapsing menu on left shows a locked icon.

that's something we need crz to decide on, it also confuses me 👍  I'll ask him for input and to make the design decision on this.

Same here, it confused me too!

@crz- ^


Add non-strict (or corresponding) feature to_abap()

li_json->to_abap(
  exporting
    iv_correspoding = abap_true
  importing
    ev_container = ls_container
)

or/and
li_json->corresponding_fields_only( abap_true )->to_abap( importing ev_container = ls_container )

implemented


revoke early hints support on HTTP 1.1

For compatibility and security reasons, it is recommended to only send HTTP 103 Early Hints responses over HTTP/2 or later unless the client is known to handle informational responses correctly

see https://developer.mozilla.org/en-US/docs/Web/HTTP/Status/103

Not interested in that one, sorry, Tomcat will implement what the Servlet specification says.

The requirement to correctly handle 1xx responses has been in the HTTP specs for over 25 years. Any user agent (or proxy) that still can't handle this is broken and needs to be fixed. We will not remove useful features from Tomcat to work around buggy user agents.


Upgrade OBO to v0.3.1

rh-jira: https://issues.redhat.com/browse/OSPRH-10379

/retest
could not run steps: step src failed: error occurred handling build src-amd64: could not get build src-amd64: builds.build.openshift.io "src-amd64" not found

/retest
Failed to claim cluster from cluster pool.

Error in kuttl tests themselves:
case.go:378: failed in step 0-deploy
case.go:380: pods "ceilometer-0" not found
case.go:380: statefulsets.apps "ceilometer" not found
case.go:380: services "ceilometer-internal" not found

/retest


Update UpdateClusters to notify watchers with the added and removed clusters

Part of: https://github.com/weaveworks/weave-gitops-enterprise/issues/1282

What changed?

Updated clientFactory to include a slice of watchers.
Created a clientFactory method to create/remove watcher.
Updated UpdateClusters to notify watchers with the added and removed clusters.

Why was this change made?
This is part of the changes made to allow GetProfiles and GetProfileValues to work with any HelmRepository CR and on any cluster, in order to allow a user to select a Chart from a HelmRepository on a leaf cluster.
How did you validate the change?
Added unit tests.

subscribing to a factory just feels… incorrect?

Point, like you should be getting an update when something has been constructed?
ClustersManager is a bit of a classic.

Rename ClientsFactory to ClustersManager to reflect new changes.

Zounds! Nice one. WDYT @ozamosi

Is it a… ClustersPool? ClientPool?

I have heard but its not really a pool, thrown around a lot recently in reference to other things so would avoid that one. As this is also not really a pool.


Added package hasql-optparse-applicative.

optparse-applicative-<IP_ADDRESS> is out of bounds for:

[ ] hasql-optparse-applicative-0.2.1 (>=0.12 && < 0.14). Nikita Volkov<EMAIL_ADDRESS>@nikita-volkov. @sannsyn. Used by: library

I also didn't realize that @sannsyn was the owner of this library and not @nikita-volkov. So the placement of the package is incorrect.

Actually I'm just confusing myself perhaps. @nikita-volkov is listed as the maintainer on Hackage.
I've also tested building hasql-optparse-applicative against nightly with the latest optparse-applicative library. PR is open here.

Thank you, merged


Tasks: The security token included in the request is invalid

Context
My AWS role has: AdministratorAccess. Running zappa in a VPC.
I've setup some tasks like this:
@task
def cache_tracks(tracks):
    if tracks:
        print("Caching tracks for {0}".format(tracks[0]['profile_id']))
    else:
        return

    for track in tracks:
        if track['origin'] == sources.SOURCE:
            # SOURCE
            requests.get('a/get/request/')

Expected Behavior
The task should make some HTTP calls.
Actual Behavior
Same behavior with sns
  File "/var/task/django/core/handlers/exception.py", line 41, in inner
  response = get_response(request)
  File "/var/task/django/core/handlers/base.py", line 249, in _legacy_get_response
  response = self._get_response(request)
  File "/var/task/django/core/handlers/base.py", line 187, in _get_response
  response = self.process_exception_by_middleware(e, request)
  File "/var/task/django/core/handlers/base.py", line 185, in _get_response
  response = wrapped_callback(request, *callback_args, **callback_kwargs)
  File "/var/task/django/views/generic/base.py", line 68, in view
  return self.dispatch(request, *args, **kwargs)
  File "/var/task/profiles/models.py", line 59, in dispatch
  return super().dispatch(request, *args, **kwargs)
  File "/var/task/django/views/generic/base.py", line 88, in dispatch
  return handler(request, *args, **kwargs)
  File "/var/task/songs/views.py", line 73, in get
  cache_tracks(songs)
  File "/var/task/zappa/async.py", line 341, in _run_async
  aws_region=aws_region).send(task_path, args, kwargs)
  File "/var/task/zappa/async.py", line 143, in send
  self._send(message)
  File "/var/task/zappa/async.py", line 157, in _send
  Payload=payload
  File "/var/task/botocore/client.py", line 253, in _api_call
  return self._make_api_call(operation_name, kwargs)
  File "/var/task/botocore/client.py", line 557, in _make_api_call
  raise error_class(parsed_response, operation_name)
botocore.exceptions.ClientError: An error occurred (UnrecognizedClientException) when calling the Invoke operation: The security token included in the request is invalid

Possible Fix

Steps to Reproduce

Setup tasks
Call task

Your Environment

Zappa version used: 0.43.1
Operating System and Python version: Windows, 3.6
The output of pip freeze:
Link to your project (optional):
Your zappa_settings.py:

As I mentioned in your #1049 PR, I think the problem is that you're probably setting the AWS_ACCESS_KEY_ID and AWS_SECRET_ACCESS_KEY values in one of the environments, but Lambda sets these by default along with AWS_SESSION_TOKEN. I suspect that the AWS_SESSION_TOKEN is what is causing the problem here.
You can probably look at previous comment explaining what I think is happening in more detail.
I think the two options you have would be to:

Simple give your Lambda role the access you need and remove your AWS_* environment variables from zappa_settings (or where ever they are).
Where you absolutely need to have your custom AWS_* access keys, instead of using a call like boto3.client('service_name') grab the custom keys from the environment and create the boto client that way (if applicable). It would look something like:

access_key_id = os.getenv('MY_ACCESS_KEY_ID')
secret_access_key = os.getenv('MY_SECRET_ACCESS_KEY')
client = boto3.client('service_name', aws_access_key_id=access_key_id, aws_secret_access_key=secret_access_key)

@mjschultz Thanks buddy. I think having django-storages is a common thing to have setup so I had those keys defined for that. I had then tested the async calls which fails. Changing the env names "solves" it.
I feel this should be noted somewhere.

I'm glad you solved the problem! I've previously filed #1000 to make it more obvious when something like this happens, but haven't written a solution because I don't think people would see CloudWatch debug messages easily.
That said I think you're right and a note somewhere in the docs might help a few people with this class of problem.


Adds test for merging compaction metadata

LGTM, do you think it's worth adding this test to main as well?

Yeah that would be good may be able to add this to 2.1 since it has external compactions.  I noticed the need for this while working on #4109, but did not want to include it in that PR so pulled it out.  I will leave this PR open until I have a chance to look into moving it back.

LGTM, do you think it's worth adding this test to main as well?

Yeah that would be good may be able to add this to 2.1 since it has external compactions. I noticed the need for this while working on #4109, but did not want to include it in that PR so pulled it out. I will leave this PR open until I have a chance to look into moving it back.

I looked at moving it back to main and it wasn't too bad, the big thing is the constructor changed for ExternalCompactionMetadata so that would have to be resolved

@keith-turner I made the necessary changes for adding this test to 2.1 in #4119.
The changes are pretty minor overall.

@ddanielr can this be closed now?

@ddanielr can this be closed now?

Yes, I completed the merge of #4119 last night which put these changes in elasticity


Add methods to clear out peer certs

Once an SSL connection is established, applications may want to free up some memory by clearing out peer cert information.  This provides an API to let them free up peer cert information when needed.

Checklist

[x] documentation is added or updated

close/open to rescan for CLA.

Does this have any effect on renegotiation?
Does session resumption still work?

I don't think this should break either, but it would depend when you call it.  Certainly not during mid handshake, but if established, then clearing up peer certs shouldn't be a problem.  If code does something like:
SSL* s1 = createSSL();
SSL* s2 = createSSL();
connectSSLSync(s1, addr);
SSL_peer_certificate_clear(s1);
SSL_set_session(s2, SSL_get_session(s1));
connectSSLSync(s2, addr2);

Then I'm sure there could be oddities.  This call should be invoked after any new ticket callbacks.  I think it's valuable to provide this for applications where they establish a connection (either full handshake or from a duped session (using SSL_SESSION_dup for instance), do auth/mutual auth in the handshake / shortly after connect, and then no longer need cert information.

I think it is important to document and test any limitations or lack thereof with subsequent session behaviour.

Once inserted into the session cache, the SSL_SESSION may be attached to other SSL objects which may even live on other threads. They're thus usually considered immutable once established (don't have the numbers off-hand, but previous violations of this have gotten CVEs).
As this is an API call the caller is responsible for making, I suppose that could all just be suitably disclaimed in the documentation, but that seems a little error-prone. Perhaps instead a config that tells the library "after you've verified the certificate and before you've done anything particularly scary like insert it into the session cache, please drop the peer certificate; I promise I don't need it". That would retain the immutability invariant.

Some applications may want to put additional verification outside of the handshake logic once the connection has been established.  A config would require them to always use a verify callback, which may not be ideal under some instances.

So it seems this is an unsafe API at least for server-side sessions. Is the memory cost worth it?  The certs are freed and the total cost is a few KB per connection. I am not sure the savings are worth the reduced safety.

I think a few KB can be worth it for high fan in services or high fan out clients.  Servers can be setup in such a way where this is safe to use.  This discussion certainly implies a lot more documentation is needed around this.

Perhaps more than just "be careful" documentation is required.  It may be necessary to configure the SSL context in such a way that session sharing does not take place (say completely disable the internal session cache, allowing resumption only via session tickets, if that's safe, or not at all). If the call is not known to be safe, it should be a NOOP.

I'd really like to avoid overcomplicating this code if possible.  I don't think it should be held to a higher standard than something like SSL_certs_clear or even many of the publicly available *_free functions which are completely unsafe in a number of situations.

I disagree. Many of the _free() functions handle reference-counted objects, and are quite safe, and the ways in which they are unsafe are obvious and localized. This PR mutates a shared structure in subtle ways, and other code that makes use of the shared state may later change invalidating the assumptions that a developer made when originally mutating the state.
Therefore, if an application wants to save RAM, it should avoid having a session cache entirely, that saves even more RAM.  And even within a single unshared session, we need to check that later uses of the session (say renegotiation with TLS &le 1.2) don't depend on retaining the peer certificate chain.
The requirements are likely different for client vs. server sessions etc.  If making this robust is too much work, the PR should IMHO be deferred, rather than adopted half-done.

Ok I can look into making this robust as desired.  Do you think it is safe if the session ref count = 1 and prev/next on the session are NULL?

I'll look into ways to make this robust.  Deleted my previous comment since I think ref count == 1 is too limiting based on our tests.  Outside of checking if prev/next are NULL, I'll check how this affects renegotiation, but is adversely affecting renegotiation a deal breaker for this patch if we document it? (i.e. subsequent calls to SSL_renegotiate_abbreviated will behave like...).   I'll try to put together some tests.
FWIW, our use cases are already saving RAM by using tickets and disabling the session cache.  This ends up saving us more since we have some pretty high fan in/out, so the more we can save the more it helps us.

@vdukhovni what about slimming this down to just SSL_SESSION_clear_peer_certificate and having that method check if prev/next are NULL before proceeding?

prev/next aren't necessarily a sufficient check when extension session cache callbacks are used, so you would probably still need to suitably document the constraints.

There is another thread to throw in the mix, namely that TLS 1.3 may prompt us to add in a call to explicitly request "send a NewSessionTicket message now".  That is, of course, driven by the application code and so the application author could in theory enforce the requisite sequencing between clearing peer certs and sending the ticket, but it is additional complexity.

I've limited the scope to just SSL_SESSION now to prevent some accidental misuse and documented cautionary tales about using this.

Any additional thoughts on this or things the team would like for approval?


GoLang: Create CLI tool for finding all resources using a specific API

[ ] Should take one or more arguments for which API (or APIs) to check, for instance audit.k8s.io/v1beta1, and the return a list of resources that is using that API version.

This is useful for API migration projects, or when checking if anyone is using deprecated APIs.

Not needed


Can't install packages

sudo docker run -i -t sequenceiq/spark:latest /bin/bash
uname -a  :
Linux xxx 3.11.0-12-generic #19-Ubuntu SMP Wed Oct 9 16:20:46 UTC 2013 x86_64 x86_64 x86_64 GNU/Linux

This implies the image is built from an Ububtu image. But when in the Dockerfile I use this:
RUN apt-get update
It says apt-get : command not found
Where am I going wrong ? I need to install a lot of packages using this. What's the workaround ?

Hi,
The result of your second step is really strange.
If you check the docker file of docker spark it's base docker image is 'sequenceiq/hadoop-docker:2.6.0' and that image is based on centos-6.5 as you can see here:
https://github.com/sequenceiq/hadoop-docker/blob/2.6.0/Dockerfile#L5
Br,
Tamas

Hi,
I know that's unexpected, and I can also find yum package installer in the bin folder but then why is uname -a giving Ubuntu as an output.
Also, do we have any ubuntu based image for spark?
Thanks,
Ritesh

Hi,
I do not know what cause that problem, but that image should be Centos based. If you would like to use an Ubuntu based, please use the image with tag: 'sequenceiq/spark:1.3.0-ubuntu'
You can check the type/version of distro with 'cat /etc/issue' command on that image too.
Br,
Tamas

Hi,
The ubuntu one is not available I guess. I got this error :
INFO[1632] Error pulling image (1.3.0-ubuntu) from sequenceiq/spark, Server error: Status 504 while fetching image layer (f3dbf1b259ad6ef6b01019c6e0ff6d7a1e69a7e1bb0a942b3197ee8d2da39acc)
Something wrong with the servers? Can you try pulling one for yourself and see if you can fix it ? It'll be really great help.
P.S. : You should definitely look into the CentOS/ Ubuntu error. Can cause confusion amongst other users too ?

Hey ,
It's working now. I am able to download the image and its currently installing maven in it. Thanks for the help :)

Hi,
I tried to investigate what was the original problem of this thread and when the uname -a command is executed that returns with the host OS name as you can see in the following example where I get 'tinycore' as result because on OSX the boot2docker runs a tinycore linux to provision the docker containers.
 docker run -i -t sequenceiq/spark:latest /bin/bash
bash-4.1# uname -a
Linux 31953a7b04f4 3.18.11-tinycore64 #1 SMP Thu Apr 16 17:46:31 UTC 2015 x86_64 x86_64 x86_64 GNU/Linux

bash-4.1# cat /etc/issue
CentOS release 6.5 (Final)
Kernel \r on an \m

So if you would like to get the type of os in a docker container please use cat /etc/issue command if it is possible.
Br,
Tamas

Yes, I guess whenever we launch a container in interactive mode using -i -t flag, it performs all shell operations with help of host OS kernel and stores them as json objects and finally reads from them to display output. It uses just the kernel not the filesystem as then packages existing only on the image gets executed and not the packages on the host OS. At least that's how I think docker works.
Anyways, Thanks for clarifying my doubt :)


Allow specifying defaults

To make the configuration more compact and easier to read, we should allow specifying defaults for queues, exchanges and bindings.


Remove spaces from DevCommunity links

@nschonni Thank you for your contribution, I'm merging this and it should be live shortly.


Translate reference/glossary/volume-plugin.md in Korean

This is a Feature Request

What would you like to be added

Translate reference/glossary/volume-plugin.md in Korean
Why is this needed
No translation reference/glossary/volume-plugin.md in Korean
Comments

Page to update : https://kubernetes.io/ko/docs/reference/glossary/?core-object=true#term-volumeplugin

/assign
/language ko

/triage accepted

/close


Liquibase Database Change Log Identity is bad and should be changed.

 And to date, its means of identifying change log entries makes no sense, and is detrimental to the product.

Using ID, Author and Filename is borderline wrong on many levels (described below)
Having no primary key on the databasechangelog table is bad practice and breaks some databases (Galera cluster)

The changes should be identified by the ID alone, and it should be a primary key on the databasechangelog table.
This issue has most recently become higher priority for us, since we're trying to change some standards. As it stands, we need to "migrate our migrations" because of the rigidity of Liquibase's change log identity.
Rationale

Currently it's possible to have duplicate IDs, which makes no sense, especially in a collaborative environment. Two developers could choose the same ID (high chance if semantic version practices are in use), but be different authors and/or filenames. It's just confusing and bad.
Using author as an partial identifier is inflexible, as sometimes peoples' names change (marriage etc), or email addresses change if a company chooses to use that instead of just an arbitrary string (e.g. sometimes when contractors become full-time employees, etc). Or some companies may choose to use a team email as the author, which could change with a reorg.
Using the filename as an identifier is probably the worst of all, considering that one may want to change the company standard. While there is an attribute I can set to override this, it's entirely tedious and makes no sense. The filename should have no bearing on the identity of the migration, and quite frankly, that column isn't even necessary in the table at all. The content is what matters, and the ID should be the only identifier.
Even more ludicrous is that the filename isn't consistently relative to the changelog index. Instead, it is based on how the initially run file is passed to Liquibase. That is absolutely insane from a maintenance perspective. Dealing with 10 year old repositories and trying to change standards becomes very difficult.

We have hundreds of production databases using Liquibase, and hundreds of Liquibase changelog files. Liquibase's current approach does not scale, and it does not age well. We've gone as far as to write a preprocessor just to "correct" filename columns and keep them consistent, and in the past we've even added a primary key to the ID column just to protect against accidental duplication (or to make the table compatible with Galera clusters).
Actual Behavior
Liquibase incorrectly identifies migrations as unique simply based on how the root changelog file was passed to liquibase. It will then erroneously try to rerun all migrations.
Expected/Desired Behavior
~Because this would affect backward compatibility, I don't expect you to change this default behavior.~
[Edit: I think this could be backward compatible for teams that have had good practice around not duplicating IDs in the ID column. Not ensuring that makes working with the changelog table cumbersome.]
So, I propose the following:

Add a system property that will enable this new behavior for cases where Liquibase's current behavior is inappropriate.

When enabled:

Liquibase will add a primary key to the databasechangelog ID column,
Liquibase will only identify migrations by the ID column exclusively, and will ignore the author and filename columns, tracked only for auditing purposes only,
Liquibase will set the filename column as a path and filename relative to the changelog index itself (i.e. based on the relative locations of the files in the changelog index,
As a bonus, all changelog index file references should be relative by default, so that we don't have to set the relative attribute on every single element, every single time.
Add a flag of some sort to the databasechanglog table (or perhaps a long overdue liquibaseconfig table to store configs for a given database, to minimize parameters and options passed) to identify whether this new behavior is in use, so that nothing gets messed up if someone accidentally runs without the system property enabled.

Doing this will make Liquibase better, and I hope you seriously consider it.

hey @cbegin -- thanks for the thoughts. That is a pretty big change, and not without good reasons, for sure. But yea, pretty bold plan worth digging into.
You have a lot of info, which I love, and maybe we can chat directly so I can get a clearer picture of use cases, biggest problems/solutions, etc. If you re up for that, please check https://calendly.com/liquibase/product-feedback and schedule some time. It is a good way to get a jump-start on such a big idea.
(If not, no worries, we can work thru ideas here, too)
thanks!
Mario
(product manager)

Thanks  @mariochampion, I'd love to walk you through the chaos this caused for us. When dealing with databases, I feel that data safety has to be at the very top of the priority list. Booked you in for Monday

Any update on this? I ran liquibase separately via the cli with a different file name and was surprised to see that the same content along with ids would not be marked as applied (when run via spring) based on the fact that the path is different. It makes importing to liquibase a bit tricky.

I gotta say using the filename as an identity can lead to strange issues during production use.
Just to give you an example, I just ran into an issue because I was executing"mvn liquibase:update" on the command line and then later I did run liquibase:update by spring.
Because when running update , liquibase compares the filenames of the changesets,
and somehow recognizes that "classpath:changelogs/ChangeLog.xml" and "changelogs/ChangeLog.xml" mean the same thing (so it ignores classpath)
but does not recognize "src/main/resources/changelogs/ChangeLog.xml" and "changelogs/ChangeLog.xml" or "classpath:changelogs/ChangeLog.xml".
I had to debug liquibase to find that issue.
So in order to have reliable and comparable results when running liquibase from the command line
or while booting up the server, its better to always reference the relative path
This is the culprit code which handles the comparison in RanChangeSet.java:
    public boolean isSameAs(ChangeSet changeSet) {         return this.getChangeLog().replace('\\', '/').replaceFirst("^classpath:", "").equalsIgnoreCase(changeSet.getFilePath().replace('\\', '/').replaceFirst("^classpath:", ""))                 && this.getId().equalsIgnoreCase(changeSet.getId())                 && this.getAuthor().equalsIgnoreCase(changeSet.getAuthor());     }
As you can see it only eliminates the issue with "classpath".
I dont know if this warrants creating a new issue or if this is the right place here to address it.

I ran into this issue when switching from Ansible to Azure Pipelines.
It's not Ansible's fault, but our old Ansible script used to
  - name: Copy liquibase db-dati folder
    copy:
      src: "liquibase/db-dati"
      dest: "/opt/liquibase/tracking"
      owner: "{{ t_user }}"
      group: "{{ t_user }}"
      mode: 0660
    become: yes

  - name: Copy liquibase-metadati.properties
    template:
      src: liquibase-metadati.properties.j2
      dest: "/opt/liquibase/tracking/liquibase-metadati.properties"
      mode: 0660
      owner: "{{ t_user }}"
      group: "{{ t_user }}"
    become: yes

  - name: Copy liquibase-dati.properties
    template:
      src: liquibase-dati.properties.j2
      dest: "/opt/liquibase/tracking/liquibase-dati.properties"
      mode: 0660
      owner: "{{ t_user }}"
      group: "{{ t_user }}"
    become: yes

  - name: liquibase update db metadati
    shell: cd /opt/liquibase/tracking && /opt/liquibase/liquibase --defaultsFile=/opt/liquibase/tracking/liquibase-metadati.properties --changeLogFile=/opt/liquibase/tracking/db-metadati/changelog-master.xml update
    become: yes

  - name: liquibase update db dati
    shell: cd /opt/liquibase/tracking && /opt/liquibase/liquibase --defaultsFile=/opt/liquibase/tracking/liquibase-dati.properties --changeLogFile=/opt/liquibase/tracking/db-dati/changelog-master.xml update
    become: yes

Basically, it copied files under /opt/liquibase and ran Liquibase.
I created a pipeline from scratch with new tools, and on a blank machine. I had no reason to copy files to system folders, so I ran Liuibase from the pipeline workspace. And, by the way, Liquibase is now installed under /usr/local/liquibase
I found that Liquibase tries to run again old migration scripts. According to the command line I either found

/opt/liquibase/tracking/db-metadati/versions/master.xml
versions/master.xml

It means that Liquibase uses the (absolute) path of the changelog instead of a file-location-agnostic identifier.


rw_pass_cell: introduce loom testing

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#174  👈
#173
#171 : 1 other dependent PR (#172 )
#166
#165
#164
#161
master

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @gabriele-0201 and the rest of your teammates on  Graphite


Auth error for Credential Scan on Blob Storage notebook

Hello -
I'm receiving the below error in the first cell of section 2 - Authentication to Azure Storage, after authenticating in the web browser:
"The public API of azure-cli-core has been deprecated starting 2.21.0, and this method can no longer return a valid credential. If you need to still use this method, you need to install 'azure-cli-core<2.21.0'. You may corrupt data if you use current CLI and old azure-cli-core. See also: https://aka.ms/azsdk/python/identity/migration"
Currently running the cells sequentially as suggested. Thanks.

It was due to version update on new AML CI, problem fixed.

Fixed


CGO call in init() blocks forever

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

$ go version
go version go1.13.6 linux/amd64

Does this issue reproduce with the latest release?
yes
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GO111MODULE=""
GOARCH="amd64"
GOBIN=""
GOCACHE="/usr/path/.cache/go-build"
GOENV="/usr/path/.config/go/env"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GONOPROXY=""
GONOSUMDB=""
GOOS="linux"
GOPATH="/usr/path/gopkg:/user/path/go_repo/"
GOPRIVATE=""
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/usr/local/go"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/usr/local/go/pkg/tool/linux_amd64"
GCCGO="gccgo"
AR="ar"
CC="gcc"
CXX="g++"
CGO_ENABLED="1"
GOMOD=""
CGO_CFLAGS="-I/usr/path/rocksdb/include/"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-L/usr/path/rocksdb -lrocksdb -lstdc++ -lm -lz -lbz2 -lsnappy -llz4 -lzstd -ldl"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build809657240=/tmp/go-build -gno-record-gcc-switches"

What did you do?

small reproducible go file available here
https://play.golang.org/p/DtuWWXs43dn
(not runnable in play, needs rocksdb and for setup refer https://github.com/tecbot/gorocksdb)
steps to reproduce:

run the file once, it will populate data into rocksdb
run the file again,
when called from init(), open() takes forever to execute
when called from main(), open() takes milliseconds to execute

What did you expect to see?
open() should execute in ~milliseconds
What did you see instead?
in init(), execution of open() does not complete (waited for 20mins+)
SIGQUIT: quit
PC=0x7f202241826b m=0 sigcode=128
signal arrived during cgo execution

goroutine 1 [syscall, locked to thread]:
runtime.cgocall(0x5bcd10, 0xc0000c7b40, 0xc0000ae020)
	/usr/local/go/src/runtime/cgocall.go:128 +0x5b fp=0xc0000c7b10 sp=0xc0000c7ad8 pc=0x50913b
github.com/tecbot/gorocksdb._Cfunc_rocksdb_open_column_families(0x1b67ae0, 0x1b6eef0, 0xc000000002, 0xc0000943c0, 0xc0000943d0, 0xc0000943e0, 0xc0000ae020, 0x0)
	_cgo_gotypes.go:2358 +0x4e fp=0xc0000c7b40 sp=0xc0000c7b10 pc=0x5b3fce
github.com/tecbot/gorocksdb.OpenDbColumnFamilies.func3(0xc000082040, 0x1b6eef0, 0x2, 0xc0000c7d18, 0xc0000c7d00, 0xc0000c7d30, 0xc0000ae020, 0x5618f5)
	/usr/path/gopkg/src/github.com/tecbot/gorocksdb/db.go:120 +0x334 fp=0xc0000c7bb8 sp=0xc0000c7b40 pc=0x5b88c4
github.com/tecbot/gorocksdb.OpenDbColumnFamilies(0xc000082040, 0x8fa5c0, 0xc, 0xc0000c7e68, 0x2, 0x2, 0xc0000c7e20, 0x2, 0x2, 0x0, ...)
	/usr/path/gopkg/src/github.com/tecbot/gorocksdb/db.go:120 +0x350 fp=0xc0000c7d58 sp=0xc0000c7bb8 pc=0x5b62e0
main.openDb(0xc7a240, 0xc7a240, 0xc000094380)
	/usr/path/sample.go:88 +0x2fd fp=0xc0000c7eb8 sp=0xc0000c7d58 pc=0x5ba63d
main.open()
	/usr/path/sample.go:37 +0x26 fp=0xc0000c7f20 sp=0xc0000c7eb8 pc=0x5b9f96
main.init.0()
	/usr/path/sample.go:23 +0x20 fp=0xc0000c7f30 sp=0xc0000c7f20 pc=0x5b9f10
runtime.doInit(0xc465e0)
	/usr/local/go/src/runtime/proc.go:5222 +0x8a fp=0xc0000c7f60 sp=0xc0000c7f30 pc=0x53e2da
runtime.main()
	/usr/local/go/src/runtime/proc.go:190 +0x1da fp=0xc0000c7fe0 sp=0xc0000c7f60 pc=0x53229a
runtime.goexit()
	/usr/local/go/src/runtime/asm_amd64.s:1357 +0x1 fp=0xc0000c7fe8 sp=0xc0000c7fe0 pc=0x55a971

goroutine 17 [chan receive, locked to thread]:
runtime.gopark(0x901580, 0xc000084058, 0x170e, 0x3)
	/usr/local/go/src/runtime/proc.go:304 +0xe0
runtime.goparkunlock(...)
	/usr/local/go/src/runtime/proc.go:310
runtime.chanrecv(0xc000084000, 0x0, 0x1, 0x0)
	/usr/local/go/src/runtime/chan.go:524 +0x2e8
runtime.chanrecv1(0xc000084000, 0x0)
	/usr/local/go/src/runtime/chan.go:406 +0x2b
runtime.cgocallbackg1(0x0)
	/usr/local/go/src/runtime/cgocall.go:237 +0x207
runtime.cgocallbackg(0x0)
	/usr/local/go/src/runtime/cgocall.go:191 +0xc1
runtime.cgocallback_gofunc(0x0, 0x0, 0x0, 0x0)
	/usr/local/go/src/runtime/asm_amd64.s:793 +0x9b
runtime.goexit()
	/usr/local/go/src/runtime/asm_amd64.s:1357 +0x1

rax    0xca
rbx    0x1b6fc88
rcx    0x7f202241826d
rdx    0x2
rdi    0x1b6fc88
rsi    0x80
rbp    0x7ffddbf02e80
rsp    0x7ffddbf02dd8
r8     0x1b6fc88
r9     0x7f2023f4fa80
r10    0x0
r11    0x202
r12    0x0
r13    0x3b
r14    0x7ffddbf03100
r15    0x1b6f720
rip    0x7f202241826b
rflags 0x202
cs     0x33
fs     0x0
gs     0x0
exit status 2

I don't see the cgo call in the test case that you provided.  I assume it is somewhere in rocksdb?
It would be extremely helpful if you or somebody could provide a standalone test case that does not rely on a large external package that requires setup.  Thanks.
(Note that this is expected behavior if rocksdb uses any Go shared libraries, though that is not particularly likely.)

@tecbot Can you help with this to find the issue?

@ianlancetaylor Does go have any restrictions on things done in init() calls like limited memory or thread limits as such?

Package initialization—variable initialization and the invocation of init functions—happens in a single goroutine, sequentially, one package at a time. An init function may launch other goroutines, which can run concurrently with the initialization code. However, initialization always sequences the init functions: it will not invoke the next one until the previous one has returned.

There are no particular limits on init functions.
I took another look at the backtrace, and I see the problem.  When C code calls into Go code, we need to ensure that the Go code is fully initialized.  This particularly matters when C code starts a new thread that then calls into Go code.  Go code that is not invoked by an init function always assumes that initialization is complete--all init functions have been called, all package-scope variables have been set.  To implement that guarantee, we block the callback until Go initialization is complete.
That is what is happening here.  The call to rocksdb from an init function is causing a callback into Go code.  That callback is suspending until all init functions are complete, but the init function is waiting for rocksdb to return.  So there is a deadlock.
I don't think there is any to fix this in the Go tools or standard library, so closing the issue.  Please comment if you feel otherwise.


"nerd"

"nerd" and its derivatives like "nerdy" are missing from the lexicon

Suggested spellings (IPA via dictionary.cambridge.org [could only find 2, sorry] and Shavian via my own attempt at transliterating)
nerd - /nɜːd/ - 𐑯𐑻𐑛
nerds - 𐑯𐑻𐑛𐑟
nerdy - /ˈnɜː.di/ - 𐑯𐑻𐑛𐑦
nerdier - 𐑯𐑻𐑛𐑽
nerdiest - 𐑯𐑻𐑛𐑾𐑕𐑑


Possibly suspicious line?

This line in construction.build_central_mesh pads the input boundary by a fixed amount:
https://github.com/C-bowman/tokamesh/blob/feb0d0cf6063d9b4da7d41a90ab5e4e95a853a61/tokamesh/construction.py#L347
This looks like maybe it's supposed to pad by the height of two triangles, i.e. 2 * sqrt(3) * resolution * 0.5? I don't think this affects the validity of the function -- it looks like it would only shift the start of the resulting grid?
On a related note, the docstring for padding_factor in this function says:
    :param padding_factor: \
        A multiplicative factor which defines the minimum distance to the boundary
        such that ``min_distance = padding_factor*scale``. No vertices in the returned
        mesh will be closer to the boundary than ``min_distance``.

Should scale be resolution?

Hi Peter,
Yep that's for sure a bug - I guess it's managed to hide because I tend to make meshes where resolution is much less than 1, which leads to creating way more padding than is needed, but then it's trimmed off anyway. This could definitely cause issues for cases with resolution bigger than 1, especially for coarser meshes.
The reference to scale in the docs is also an error for sure - I switched the name of the kwarg from scale to resolution a while ago and I must have missed that reference.
I'll commit a fix for it after I've merged your unit test PR,
Cheers

Fix has been committed - thanks again!


Add SpeechBrain TTS code snippet

This depends upon internal PR https://github.com/huggingface/moon-landing/pull/3152
cc @TParcollet

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.

Closing this as it's outdated and we have a more generalizable structure now


Console printing incomplete

I've only observed this occurring when KTAB is compiled for release.  When the model is run in SMPQ, certain information statements don't get printed to the console.  I did some very brief investigation and noticed that the missed statements are all printed using cout.  The same issue does not occur with SMPC.  I've attached here four files from running the model on SOE-Pol-Comp.csv:

PCC_release.txt - release-compiled smpc, output saved with >>
PCG_release.txt - release-compiled smpq
PCC_debug.txt - debug-compiled smpc, output saved with >>
PCG_debug.txt - debug-compiled smpc

Since we've switched to logging with the easylogger++ library, this is a moot point, so I'm closing this issue.


Solve the delay problem when moving the viewer window

I found that when I try to move the Vispy viewer window in Glue there would be some delay phenomenon. Also it won't influence the whole data display but solving this problem could kind of polish the user experience.

I've created some minimal VisPy + Qt4 code (without glue) to show the issue here:
https://gist.github.com/astrofrog/aa1b4a490e61b82f9aa7
and have put an example here:
https://www.youtube.com/watch?v=wBg8SRpa580&feature=youtu.be
I'll contact the VisPy devs to see if anyone knows what this could be caused by.

I opened a VisPy issue here: https://github.com/vispy/vispy/issues/1158

As for https://github.com/glue-viz/glue-3d-viewer/issues/40, this is fundamentally a Qt/OpenGL/MDI issue. Will be fixed only by allowing constrained layouts as described in https://github.com/glue-viz/glue/issues/837


Memory injection ?

Hi,
Impressive work,
I am wondering if i can in some way inject into memory some views of the object with annotations and in inference time to be able to find similar object in providedvideo?

There was some discussion of this in issue #352 and #210. There's another technique for finding similar looking objects (not using the memory bank though), in issue #405.
Although this sort of similarity matching works with the SAM models, other models that are built specifically for this (DINOv for example) might perform better.


[vtk] Fix the dependency curl of proj

Fix https://github.com/microsoft/vcpkg/issues/32077

/usr/bin/ld: /home/leander/vcpkg/buildtrees/proj/src/9.2.1-3d4c79f852.clean/src/networkfilemanager.cpp:1606: undefined reference to `curl_easy_setopt'
/usr/bin/ld: /home/leander/vcpkg/buildtrees/proj/src/9.2.1-3d4c79f852.clean/src/networkfilemanager.cpp:1610: undefined reference to `curl_easy_setopt'
/usr/bin/ld: /home/leander/vcpkg/buildtrees/proj/src/9.2.1-3d4c79f852.clean/src/networkfilemanager.cpp:1615: undefined reference to `curl_easy_setopt'
/usr/bin/ld: /home/leander/vcpkg/buildtrees/proj/src/9.2.1-3d4c79f852.clean/src/networkfilemanager.cpp:1616: undefined reference to `curl_easy_setopt

[x] Changes comply with the maintainer guide
[ ] SHA512s are updated for each updated download
[ ] The "supports" clause reflects platforms that may be fixed by this new version
[ ] Any fixed CI baseline entries are removed from that file.
[ ] Any patches that are no longer applied are deleted from the port's directory.
[x] The version database is fixed by rerunning ./vcpkg x-add-version --all and committing the result.
[x] Only one version is added to each modified port's versions file.

PLEASE READ MY COMMENTS IN THE BUG REPORT. IT IS NOT ABOUT PRIVATE VS PUBLIC.

This is just to adapt the patch provided by vcpkg alone.

Fixed by https://github.com/microsoft/vcpkg/pull/32138


building on solaris <IP_ADDRESS> sunv4 sparc

I was able download yara.3.11 to a Ubuntu VM on Virtulbox, build and run my application which uses libyara.
When I download the same yara.3.11 to Solaris 11, configure it and build, it succeeded.
But when I ran my application, it got a Segmentation Fault.  Debugging the core, it was calling a function
in the libyara.so.3.10.0 shared library.  But it did not get loaded.  And yet, the call trace showed several functions
loaded from the same shared library.
I was able to resolve this issue by statically linking the needed library functions with my application.
Are there any special build/libtool/automake flags needed on Solaris to get it load needed shared library functions?
Alternatively, is there a way to build libyara statically?
thank you,
jim edele

I was able to get my application to link dynamically with the shared library of libyara by putting a symbolic link to it in /usr/local/lib/libyara.so.  elfdump of the application showed the RUNPATH of /usr/local/lib.


after recent cod warzone update no longer detecting the cod exe

1st thanks a lot its the only program that fake that fullscreen in a windowed mode the game forces my monitor to HDR mode unless in windowed mode)
recently after the 20 gb update on pc with the launch of recochet anticheat the SRWE program doenst detect Cod exe (the other programs detect cod)
i would really appreciate it if you can take a look
thanks again

what other programs detect cod if srwe can't? It checks with windows which processes are available. If there's anti-cheat in the game it likely hides the exe from windows so srwe can't see it (and that's not fixable). perhaps the game runs as admin tho, so running srwe as admin might fix it, other than that, it's something that's not fixable from my side, sorry

i found others they show the name of the exe with symbols they do the work but am not sur about the same performance anyway thank you for your time i appreciate it, hope it changes in the future
thanks again


Cannot find Temporary Table in metadata catalog causing SQL Editor failed to highlight temp table fields

Description
It happened to PostgreSQL but not sure about other variants.
You can reproduce by running the simple query below:
drop table if exists test_temp;

create temporary table test_temp as
select
	*
from
	generate_series(
		1,
		10,
		1
	) as a;

select
	a
from
	test_temp;

The query runs fine and it will output the result. However, if we check the editor, it highlights all the test_temp fields:

By checking the log, it appears to have some issue when reading meta catelog:
Table 'postgres.pg_temp_3.test_temp' not found in metadata catalog
DBeaver Version
Community Edition <IP_ADDRESS>403031740
Operating System
Windows 10
Database and driver

Database: PostgreSQL 16.2 (Debian 16.2-1.pgdg120+2)
Driver: PostgreSQL JDBC Driver 42.7.2

Steps to reproduce

Create any temp table and write a select statement
Check the syntax highlighting in SQL Editor

Additional context
No response

thanks for the report

Hi @uslss , do you have an ETA on when this will be fixed?

#1330 Looks like this is kind of releated.


Writing more than 1020 characters from the LabVIEW library results in a crash

I'm guessing that there's a 1024 character limit somewhere that's causing a crash. OUTP + 1020 = 1024.

Experienced that too...
In my case this happens when a LabVIEW build which was triggered via CI using LabVIEW-CLI fails and then tries to output the Error via CLI...

Will aim to get to this today - will increase the buffer to say 8k and then get the LabVIEW client to break up any messages longer than this

This should be fixed in v1.2.0 which I have just put up in the releases section. Please give it a go and let me know if there are problems.
I've rolled in both of your other changes as well, thanks for getting involved!

It's still crashing but now, the max number of characters that can be sent before crash is now 2997. Haven't looked at code but developers are reporting this.


HTTP/S Proxy upstream support

According to manual:
--proxy value  use upstream socks5 proxy
Does suo5 support Socks5 proxy only as upstream? Is HTTP/S or Squid Proxy supported?

Right now, the code only allows for SOCKS5 proxies, but I'm thinking that HTTP(S) proxies can also be supported. I'll double-check and if everything looks good, I'll lift the restriction.

Okay cool. I will also test if HTTP(S) proxies can be used

Only socks5 is supported, i did test it and verified it.

Only socks5 is supported, i did test it and verified it.

I removed this restriction  from the code https://github.com/zema1/suo5/commit/e941463383c72878b99419034672ddcf9f4e4571, so now you can use socks5 and http(s) as upstream proxies. Before the official release, you can test the binary compiled by the Github Action, or compile the code from the main branch manually. If you encounter any issues, feel free to comment here.
Use target-release in github action: https://github.com/zema1/suo5/actions/runs/4924332131

related to https://github.com/zema1/suo5/issues/25
Add in https://github.com/zema1/suo5/releases/tag/v0.7.0