Add schema.yaml to Knative Serving package.

Feature Request
Carvel packaging recommends the use of a schema.yaml file to document configurable parameters for packages. TCE also has a make task, generate-openapischema-package, that will generate the valuesSchema stanza for the package.yaml file.
Describe alternatives you've considered
Continue using a values.yaml file and manually generating the schema for the package.yaml file.
Additional context

One caveat to using schemas is that some users might use the same file for having in a single location all the configuration for all the applied packages to the cluster. When schemas are introduced, no longer one can use this mechanism, as deployment of a package with a values file with entries not in the schema will fail.
More so, schemas might make updates of a package fail, as if a value is removed from the schema and the applied values are not updated, the update will fail, leaving the functionality in a non-working state.


Add audio from USB microphone

Great project! Any chance of adding audio from the USB microphone when creating the mp4 frames? If you have some pointers on how to do that I can do some coding and submit a PR.

Hi !
great work
for me is opus the best choise. it is in the ffmpeg and has better performance than libfdk
-c:a libopus -application lowdelay -b:a 32k   -movflags +faststart
is it possible to be come  an option for rtsp streaming ?


setting up saleor on subpath

Hi everyone,
i am trying to install saleor on my self hosted productive server, where i am already using nginx as proxy and have other projects. Saleor should still run on http, because nginx sets https and forwards on port 8000. Thereby i am forced to use saleor on subpath:
www.example.com/shop
I could not find any information about this, but tried to set force_script_name to shop
FORCE_SCRIPT_NAME = "/shop"
and also tried to change urls.py to something like:
prefixShop = "shop/";

non_translatable_urlpatterns = [
    url(rf'^{prefixShop}graphql/', csrf_exempt(GraphQLView.as_view(schema=schema)), name="api"),
    url(
        rf'^{prefixShop}sitemap\.xml$',
        sitemap,
        {"sitemaps": sitemaps},
        name="django.contrib.sitemaps.views.sitemap",
    ),
    url(rf'^{prefixShop}i18n/$', set_language, name="set_language"),
    url(f'{prefixShop}', include("social_django.urls", namespace="social")),
]

translatable_urlpatterns = [
    url(rf'^{prefixShop}', include(core_urls)),
    url(rf'^{prefixShop}checkout/', include((checkout_urls, "checkout"), namespace="checkout")),
    url(rf'^{prefixShop}jsi18n/$', JavaScriptCatalog.as_view(), name="javascript-catalog"),
    url(rf'^{prefixShop}order/', include((order_urls, "order"), namespace="order")),
    url(rf'^{prefixShop}page/', include((page_urls, "page"), namespace="page")),
    url(rf'^{prefixShop}products/', include((product_urls, "product"), namespace="product")),
    url(rf'^{prefixShop}account/', include((account_urls, "account"), namespace="account")),
    url(rf'^{prefixShop}feeds/', include((feed_urls, "data_feeds"), namespace="data_feeds")),
    url(rf'^{prefixShop}search/', include((search_urls, "search"), namespace="search")),
]

But the setup also fails. Is there any possibility to setup saleor on subpath?
Thanks in advance
Regards,
Maxim

@maxim-devlop Why don't you forward load to saleor on the nginx lvl? Nginx should handle all requests which come to {prefixShop} and forward it to Saleor

@korycins ok, if its easier way. I will try in the next few days and give feedback. Thank you.
P.S. do you have sample nginx configuration for this purpose?

@maxim-devlop, unfortunately, I don't have any nginx config


Duplicated submissions and edits

The current engine configuration treats every submission as unique as it includes a unique identifier in the filename.
We could use a different field but we would still need to have enough information to not have identity clashes.
All combinations I can think of have some form of caveat.
For instance:

Name-Affiliation will likely not be unique enough.
Email may be sufficient but will need manual update if members relocate.
...

We should also test if submitting different data (e.g. expertise) to an existing email record will be treated as an edit or something else will happen.

+1 for testing things!
I'm tempted to think that the unique identifier is actually a good solution. There may be the occasional duplicate, but so long as the entries do not interact, this shouldn't harm the list's functionality. There's also the possibility of occasionally running some simple code off-line to highlight possible duplicates, which can then be deleted manually.
In other words, I'd rather err on the side of having duplicates in the list than rejecting genuine entries because they look like duplicates to a simple algorithm.
Re edits: this could be difficult to provide as a functionality, as it would require authentication so people can't just submit different data for existing entries. That's another reason why I think IDed, non-interactive entries are best for now, to keep it simple.
Re existing email record: I would anyway opt to change the email field to just another text field in order to allow email obfuscation, in which case any new submission using the same email will just generate an independent duplicate, which we can handle by off-line monitoring as mentioned above.
Actually, this just gave me an idea, but I'm opening a new issue for that.

Unless dupes are involved in #13, I don't think they're a huge problem and as mentioned above we can always resolve them offline. Closing.


feat : added uppercase conversion to lowercase

I'm not sure whether this is a good fit for the repo, though.

I've decided that this doesn't warrant inclusion, since the "'convert to array, then transform, then join again' pattern" is already demonstrated elsewhere in the string folder (e.g. in the string reversal method) and IMO more cleanly (without requiring mixing it with ASCII knowledge).


support inference method for all models

tail = model.inference(h=10, r=20)
# e.g., TransE model returns the sum vector: head + rel

@wujunzhuo
Hello @wujunzhuo, I have an update related to the inference feature you requested in the development branch. Can you kindly try the example/inference.py and see if it fits what you want?


File import order is not preserved in singleBundle

When using singleBundle, the order of the imports does not match the file / pattern list. This affects some plugins like karma-mocha whose injected files must be in the correct order, otherwise you would get errors similar to
TypeError: undefined is not an object (evaluating 'window.mocha.setup')

I've also run into this issue.


Regression: Time range dropdown for monthly is broken

from @marccampbell

the dropdown to select “this month” doesn’t work, if you are looking for something to work on, that feels like a regression (likely mine)

https://replicated.slack.com/archives/C02Q6PGRZML/p1705347908158879

It appears that somethign is up with the nested react-bootstrap NavDropdown.Item having it's own href that is overriding the wrapping NextJS Link

This post: https://github.com/react-bootstrap/react-bootstrap/issues/4131#issuecomment-566139677 suggests trying passHref


check dashboard display

I have the feeling that the dashboard and the map display are not aligned (there is no "grey" on the map and sometime nearly the whole dashboard values are grey (non-suitable)

Summary for further questions:
the symptoms
When building the documentation I realized that the map and the dashboard are displaying different results. Hear me out, the computation is the same, this is the display and thus interpretation that differs.
On this image you'll see the map built using the Lekoumou region in the current implementation:

Everything in orange is valued 1. There are no zeros on this image.
Now looking at the dashboard:

There are grey values "unsuitable for restoration" that cover 80% of the country. This value correspond exactly to the value that is filtered by the treecover_potential mask.

It means that the contraints filter are not displayed on the map but only on the dashboard
the issue
When building the suitability index, we first compute everything i.e.: (Sum(wheight*normalized_indicators)*constraints)/costs and then the results (scaled from 0 to 4) are rescaled from 1 to 5. Perfect but then the masked areas are rescaled as well and displayed as 1 instead of the expected 0.
solution
As a solution I split the computation in 2 steps:

(Sum(wheight*normalized_indicators))/costs rescaled from 1 to 5
add the masking contraints.

The final indicator is 0 for masked regions (darkgrey on the picture) and then 1 to 5 in evaluated pixels.
I think it will help users to "see" the influence of the masks they are setting and it also insure that the results in the dashboard are alined  with those in the map. I also changed the color map that is still color blind friendly but with more contrast to it.

As a side comment When a user sets too many filter the image will be fully grey. that's an important information that was impossible to get in the pervious implementation.


Bump OCS Operator to version 4.6.0

We've officially branched off release-4.5! Time to bump.

the go lint error fix is kind of surprising in the PR.
It should be a separate PR imho.

Agreed, but the ocs-operator-ci tests wouldn't pass locally without it. I'm not sure why I picked it up but no one else has... maybe I'm using a different version of golint somehow.

Other than that this looks good.
But needs rebase / fixing conflicts.

Done.

Rebased to fix merge conflict, @obnoxxx PTAL again

/test ocs-operator-e2e-aws


Odd line spacing behavior

Describe the bug
When the line spacing is higher than 1, some weirdness occurs.  As you can see in this screenshot, the 2nd and 3rd lines have a visibly smaller line spacing than the other lines (this is with the default line spacing).

This inconsistency is not longer present when I set the line spacing to 1.

Nor does it seem present in line spacings below 1 (this is .60)

To Reproduce
Steps to reproduce the behaviour:

Create a text box with enough text to extend for at least 3 lines
Ensure text themes are set up to produce the necessary line spacing

Expected behaviour
I would expect line spacing to be consistent throughout the paragraph.
Platform and software (please complete the following information):

OS: Windows
Pygame GUI version 0.6.11
Pygame-ce version 2.4.1

Hmm, I'm not seeing this with the default font after trying several different line spacings greater than 1.
Any chance you can share more specifics of where it breaks down?

Well, that's mildly concerning for my code haha
I'm seeing it in both normal text boxes and tool tip textboxes.  It also seems to only be present in paragraphs. So, for example, if I were to manually add a line break instead of having it wrap the text, then there would be no line spacing issues.
I'm also using the Notosans font and font size seems to make no difference besides making the effect more or less pronounced (the effect is still there, though.)
I'll try and do a deeper dive into our code to see if I can pinpoint a cause within our own codebase instead of pygame_gui.  This all worked perfectly fine before the 0.6.11 update (excepting the bottom cutoff that i had reported and you fixed), so I'd been convinced there was something changed with that bugfix that had caused this new issue.

Hi! I wanted to test to see if it was just the game or a possible issue with the newest update. I set line spacing to 1.5 for testing.
First image is on py gui 0.6.10, and second is after I updated to 0.6.11. These are screenshots of text boxes.
My project is rather simple and uses default font type and sizing. I only changed color and alignment aside from the spacing.
Further info:
OS: Linux
Pygame-ce: 2.4.1

Well, that's mildly concerning for my code haha
I'm seeing it in both normal text boxes and tool tip textboxes. It also seems to only be present in paragraphs. So, for example, if I were to manually add a line break instead of having it wrap the text, then there would be no line spacing issues.
I'm also using the Notosans font and font size seems to make no difference besides making the effect more or less pronounced (the effect is still there, though.)
I'll try and do a deeper dive into our code to see if I can pinpoint a cause within our own codebase instead of pygame_gui. This all worked perfectly fine before the 0.6.11 update (excepting the bottom cutoff that i had reported and you fixed), so I'd been convinced there was something changed with that bugfix that had caused this new issue.

It may be some kind of rounding issue with the specific line spacing/font height combo you are using that I am not hitting in my tests. If you could post any theme file details like:
"text_box":
    {
        "misc":
        {
            "border_width": "0",
            "shadow_width": "0",
            "line_spacing": "1.8"
        }
    },

That give you this problem that would be helpful!

Yeah for sure!
    "tool_tip.text_box": {
        "colours": {
            "normal_border": "#2F2918",
            "normal_text": "#000000",
            "dark_bg": "#AC9D72"
        },
        "font": {
            "name": "notosans",
            "size": "13"
        },
        "misc": {
            "shape": "rounded_rectangle",
            "shape_corner_radius": "8",
            "border_width": "2",
            "shadow_width": "0",
            "text_horiz_alignment": "center"
        }
    },

    "#text_box": {
        "colours": {
            "dark_bg": "#FFFF0000",
            "normal_text": "#000000",
            "selected_bg": "#776045",
            "selected_text": "#AC9D72",
            "link_text": "#2E8B57",
            "link_hover": "#556B2F",
            "link_selected": "#556B2F"
        },
        "font": {
            "name": "notosans",
            "size": "15"
        },
        "misc": {
            "border_width": "0",
            "shadow_width": "0"
        }
    },

And if it's helpful info, we do have a lot of prototyped text themes as well, though prototype or no prototype doesn't seem to make any discernible difference with this bug.

    "#text_box_22": {
        "prototype": "#text_box",
        "font": {
            "name": "notosans",
            "size": "11"
        }
    },
    "#text_box_22_horizcenter": {
        "prototype": "#text_box_22",
        "misc": {
            "text_horiz_alignment": "center"
        }
    },
    "#text_box_22_horizcenter_red": {
        "prototype": "#text_box_22_horizcenter",
        "colours": {
            "normal_text": "#ae1515"
        }
    },
    "#text_box_22_horizcenter_spacing_95": {
        "prototype": "#text_box_22_horizcenter",
        "misc": {
            "line_spacing": "0.95"
        }
    },

OK, these should be solved in the latest version.
Let me know if you have any more issues.


Peer puts off a rude principal

Describe The Problem
When I add a peer it takes the (Endpoint Allowed IPs (Required)) of the main network and not the network that I am registering the peer.
Expected Error / Traceback
Please provide the error traceback here

To Reproduce

OS Information:

OS: Debian 12
Python Version: 3.11.2

Hi! That would be the default value of what you set in Settings > Peer Settings

Yes I understand. But in my case I have several networks. <IP_ADDRESS>/24, <IP_ADDRESS>/24.
Each client uses a network.

Ahh gotcha.. at this moment it will apply to all peers. I will figure something out to overcome your issue :)
You can also edit them individually.. I know it might be tedious😭

Will close this issue for now


Changing look of home page  and color theme

Category of documentation update
I would like to change home page and color theme
Describe the change you think might work
No response
Checklist

[X] I have checked the existing issues.
[X] I follow Contributing Guidelines of this project.
[ ] I am a IWOC'24 contributor.
[ ] I am a JWOC'24 contributor.
[X] I am a SWOC'24 contributor.

Code of Conduct

[X] I agree to follow this project's "Code of Conduct".

Please assign it to me @Rakesh9100


VK_ACCESS_MEMORY_READ_BIT warning for transition to VK_IMAGE_LAYOUT_PRESENT_SRC_KHR

I'm writing to a swapchain image from a compute shader and my code for the final image layout transition is:
VkImageMemoryBarrier barrier;
barrier.sType = VK_STRUCTURE_TYPE_IMAGE_MEMORY_BARRIER;
barrier.pNext = nullptr;
barrier.srcAccessMask = VK_ACCESS_SHADER_WRITE_BIT;
barrier.dstAccessMask = 0;
barrier.oldLayout = VK_IMAGE_LAYOUT_GENERAL;
barrier.newLayout = VK_IMAGE_LAYOUT_PRESENT_SRC_KHR;
barrier.srcQueueFamilyIndex = VK_QUEUE_FAMILY_IGNORED;
barrier.dstQueueFamilyIndex = VK_QUEUE_FAMILY_IGNORED;
barrier.image = swapchainImages[i];
barrier.subresourceRange.aspectMask = VK_IMAGE_ASPECT_COLOR_BIT;
barrier.subresourceRange.baseMipLevel = 0;
barrier.subresourceRange.levelCount = 1;
barrier.subresourceRange.baseArrayLayer = 0;
barrier.subresourceRange.layerCount = 1;

vkCmdPipelineBarrier(swapchainCommandBuffers[i], VK_PIPELINE_STAGE_COMPUTE_SHADER_BIT, VK_PIPELINE_STAGE_BOTTOM_OF_PIPE_BIT, 0, 0, nullptr, 0, nullptr, 1, &barrier);

And I'm getting the warning (SDK version 1.0.46):

VULKAN: !!! WARNING !!! DS: Dest AccessMask 0 [None] must have required access bit 32768 [VK_ACCESS_MEMORY_READ_BIT]  when layout is VK_IMAGE_LAYOUT_PRESENT_SRC_KHR, unless the app has previously added a barrier for this transition.

But according to section 29.6 of the spec (version 1.0.48):

When transitioning the image to VK_IMAGE_LAYOUT_PRESENT_SRC_KHR, there is no need to delay subsequent processing, or perform any visibility operations (as vkQueuePresentKHR performs automatic visibility operations). To achieve this, the dstAccessMask member of the VkImageMemoryBarrier should be set to 0, and the dstStageMask parameter should be set to VK_PIPELINE_STAGE_BOTTOM_OF_PIPE_BIT.

If I understand correctly, there's a disconnect between the spec and the validation.

I agree with you. This used to be less clear.

Cool! Thanks for the (eventual) fix


[MRG] only numpy arrays in extracellular

closes #358
this was really bugging me as I started reviewing #370 so I'm going to take a stab at it

Codecov Report

Merging #392 (da474cf) into master (d6b08ff) will decrease coverage by 0.11%.
The diff coverage is 75.00%.

:exclamation: Current head da474cf differs from pull request most recent head 55a899e. Consider uploading reports for the commit 55a899e to get more accurate results

@@            Coverage Diff             @@
##           master     #392      +/-   ##
==========================================
- Coverage   89.97%   89.86%   -0.12%
==========================================
  Files          16       16
  Lines        2934     2930       -4
==========================================
- Hits         2640     2633       -7
- Misses        294      297       +3

Impacted Files
Coverage Δ

hnn_core/extracellular.py
86.69% <75.00%> (-0.26%)
:arrow_down:

hnn_core/parallel_backends.py
82.28% <0.00%> (-0.58%)
:arrow_down:

hnn_core/viz.py
83.25% <0.00%> (-0.23%)
:arrow_down:

Continue to review full report at Codecov.

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

This is looking great! Just a few minor questions/suggestions but happy to merge when you're ready.

@ntolley I fixed or replied to your comment. There is one outstanding issue of _data being a list sometimes and an array sometimes. Unfortunately, I don't have the bandwidth to fix it immediately. If you think it's a dealbreaker for this PR, I could bring it back to WIP and hopefully fix it sometime later. It will need some work.

@jasmainak I'm happy to merge (after a rebase) as this PR since this PR is focused on extracellular. I think dealing with _data being a list will come with revamping how simulation data is handled from the NetworkBuilder side, which is a slightly different goal that may be lower on our priorities at the moment.

Just pushed the rebase, will merge after tests pass. Thanks @jasmainak!


Update installation-next-js.mdoc

went to far down with the ../../ stuff. Just one too many

I think this is actually correct? The route.ts file is nested 5 directories deep inside the src dir, so we need to go back 5x to be in the project root where the Keystatic config is.


panic: user: Current not implemented on linux/amd64

On Linux 14.04::
I have this error:
% ./antibody/bin/antibody
panic: user: Current not implemented on linux/amd64

goroutine 1 [running]:
github.com/caarlos0/gohome.home(0x0, 0x0)
	/Users/carlos/Code/Go/src/github.com/caarlos0/gohome/main.go:67 +0x9f
github.com/caarlos0/gohome.xdgCache(0x0, 0x0)
	/Users/carlos/Code/Go/src/github.com/caarlos0/gohome/main.go:55 +0x72
github.com/caarlos0/gohome.cacheFor(0x510610, 0x8, 0x509cd0, 0x5, 0x0, 0x0)
	/Users/carlos/Code/Go/src/github.com/caarlos0/gohome/main.go:29 +0x1f8
github.com/caarlos0/gohome.Cache(0x510610, 0x8, 0x0, 0x0)
	/Users/carlos/Code/Go/src/github.com/caarlos0/gohome/main.go:13 +0x57
github.com/caarlos0/antibody.Home(0x0, 0x0)
	/Users/carlos/Code/Go/src/github.com/caarlos0/antibody/api.go:70 +0x7a
main.main()
	/Users/carlos/Code/Go/src/github.com/caarlos0/antibody/cmd/antibody/main.go:15 +0xbe

goroutine 2 [runnable]:
runtime.forcegchelper()
	/usr/local/Cellar/go/1.4.2/libexec/src/runtime/proc.go:90
runtime.goexit()
	/usr/local/Cellar/go/1.4.2/libexec/src/runtime/asm_amd64.s:2232 +0x1

goroutine 3 [runnable]:
runtime.bgsweep()
	/usr/local/Cellar/go/1.4.2/libexec/src/runtime/mgc0.go:82
runtime.goexit()
	/usr/local/Cellar/go/1.4.2/libexec/src/runtime/asm_amd64.s:2232 +0x1

goroutine 4 [runnable]:
runtime.runfinq()
	/usr/local/Cellar/go/1.4.2/libexec/src/runtime/malloc.go:712
runtime.goexit()
	/usr/local/Cellar/go/1.4.2/libexec/src/runtime/asm_amd64.s:2232 +0x1

related to https://github.com/golang/go/issues/6376 ?

worked with v0.2.0.
did not v0.2.2.

@wkentaro yeah, the problem is in https://github.com/caarlos0/gohome/ .
I fixed it there in https://github.com/caarlos0/gohome/pull/3, but forgot to re-release here.
Releasing v0.2.3 right now...

@wkentaro done.
Let me know if there still any issue.
Thanks


i18n management

How are we going to manage translations? I'm ready to do some translations to Finnish but not sure which way to proceed. Strings are at least planned if not all already to be in separate files for each lang.
I see two options:

take translation management to dedicated platform.
copy original en i18n file and rename it to target language and accept as pull request.

The latter is taking the easy way for project, but probably not the best for community and for getting crowd of translators.

I tested OneSky service:

http://os7daht.oneskyapp.com/collaboration/

API Umbrella is using the same.

I could take this feature and keep on working with it given that this is the way to proceed.

Good suggestion, we just need to make sure the l10n strings are compatible with tap:i18n.

Also, the tap:i18n package uses i18next, which has a web translation interface.

Well either one looks good. Now we need decision.

@bajiat I believe this is PO decision at the end. Is there something, additional information, or else needed to make decision which way to go?

@kyyberi I will look into this and see if I need more information.

@kyyberi @brylie
If we take OneSky into use, we would need to modify the i18n json file  as described in #692. Is there anything else that is required?
I looked at Jarkko's experimentation and based on that the OneSky web interface looks easy to use and  it seems to allow easy crowdsouring (but only for up to 5 collaborators for free.) Also, looks likes it regularly updated (based on the Github repo).

Or then we could use transifex, which is also used a lot in open source community. https://www.transifex.com/apinfio/apinfio-api-management/dashboard/

Let's continue using OneSky. The UI seems nice to use and easy to understand. Signup is not too complicated either (esp when compared to Transifex).


Stereo of CHEBI:9110

Hi
I wonder if the stereo of CHEBI:9110 is the right one.
Its InChIKey is MNXFXXWQAAMVMW-PUYVOJRZSA-N but the InChIKey of its xref KEGG and CAS is MNXFXXWQAAMVMW-JEIATDTDSA-N

The structure has now been fixed. Thanks


BUG: unable to install gRPC 1.48 with Ruby 3.1.2

What version of gRPC and what language are you using?
Ruby 3.1.2
gRPC 1.48.0
What operating system (Linux, Windows,...) and version?
Windows 10
What did you do?
Tried to run bundle install on a project that had been upgraded from Ruby 3.0.3 to 3.1.2 on a new clean Ruby installation, however the installation fails when installing grpc, showing the errors that are described on the gem_make.out file.
What did you expect to see?
Correct installation of gem.
What did you see instead?
Error in it's installation
Make sure you include information that can help us debug (full error message, exception listing, stack trace, logs).
gem_make.txt

UPDATE: after trying to install it with version 1.49.1 (which could possibly fix the error), the error still happens 😔

I had the same issue when I switched to rbenv.
My config: ruby 3.1.2, grpc 1.49 or 1.48, mac M1 macOS Monterey
Quick fix until grpc comes with a fix on a new version: use grpc 1.47 (works for me):
gem 'grpc', '1.47.0'

I had the same issue when I switched to rbenv.
My config: ruby 3.1.2, grpc 1.49 or 1.48, mac M1 macOS Monterey
Quick fix until grpc comes with a fix on a new version: use grpc 1.47 (works for me):
gem 'grpc', '1.47.0'

Having the exact same issue. Getting this error message:
An error occurred while installing grpc (1.50.0), and Bundler cannot continue.
Moving down to grpc 1.47 for now works.

You can try bundle config build.grpc --with-ldflags="-Wl,-undefined,dynamic_lookup" and then bundle install

Can you try grpc 1.52.0 or later?
I have verified they can installed successfully with ruby 3.2.2 and 3.1.4 (using ruby installer)

You can try bundle config build.grpc --with-ldflags="-Wl,-undefined,dynamic_lookup" and then bundle install

that worked for me on apple m1 (I was unable to install grpc 1.56.0 on ruby 3.1.0)

M1 build failure tracked in #33613, fix PR #33538 had merged, will release in next version.
gettimeofday error in the original post no longer happens in latest versions.

You can try bundle config build.grpc --with-ldflags="-Wl,-undefined,dynamic_lookup" and then bundle install

It worked on my Macbook M1. Thanks for your answer.

You can try bundle config build.grpc --with-ldflags="-Wl,-undefined,dynamic_lookup" and then bundle install

Worked on Macbook Pro M1 - macOS Monterey 12.4 using RVM for ruby 3.1.4 Thanks for the answer.

bundle clean --force
Worked with ruby 3.1.4


Google file stream/Google drive for desktop doesnt show in cloud section

Description

Steps To Reproduce
No response
Expected behavior
It should show the folder google drives linked to in the cloud drive section
Files Version
1.9.18
Windows Version
10.0.19043.1151
Relevant Assets
No response

Fixed in <IP_ADDRESS>


How to invite the bot to a server

I think that it works now, how would I invite the bot to the server?

You should try reading the readme of this repo. You have asked two questions that already have an answer there.
https://discordjs.guide/preparations/setting-up-a-bot-application.html


fix(table): remove min-height from buttons in expansion toggle

Close #1770
I think min-height was added to the button to increase the focus ring, but this was causing issues with vertical-align: baseline.
So removed the min-height which also fixes how the text in the cells align.

Deploy preview for pf-next ready!
Built with commit 3119371ab36cc686abdc679e05ef57d55d41b134
https://deploy-preview-1818--pf-next.netlify.com

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


diff_against does not include editable=False fields

Describe the (probably not a!) bug
In our application, we log history diffs. We recently realised some model fields were not being shown in them.
The reason simple-history does not log them, is they are editable=False fields.
"Well duh!" I hear you scream. editable=False fields are not editable, therefore simple-history should obviously ignore them!
But editable has unintuitive naming in django (at least from the DBs perspective). When you read the docs for editable:

If False, the field will not be displayed in the admin or any other ModelForm. They are also skipped during model validation. Default is True.

To me that definition means hidden, rather thaneditable (you could argue editable-through-forms-or-admin, but that's a bit of a mouthful).
And in our app, we were treating it based on the doc definition. It is hidden, and only editable by scripts/background-tasks. But we still want to see the changes in our diff log when a background task updates the model.
This sentence in simple-history docs makes me believe that ignoring editable=False fields in diff_against could be considered a bug.

By adding HistoricalRecords to a model or registering a model using register, you automatically start tracking any create, update, or delete that occurs on that model

To Reproduce
Steps to reproduce the behavior:

Create a django model with a field like blah = models.BooleanField(editable=False) and with HistoricalRecords()
Update the value of blah from django-shell/a script
Fetch the old and new historical records
call  delta = new_record.diff_against(prev_record) to generate a diff.
blah field is not included in that diff.

Expected behavior
Half of my brain expects the current behaviour based on the django attribute name. The other half expects blah to show up in the delta based on the doc-definition :))
I'm not sure if any code changes are sensible. Or even any doc changes for people to avoid this confusion, we may have a pretty niche and weird use-case.
Environment:

OS: Ubuntu 22 and OSX...something
Django Simple History Version: 3.0.0 (and I checked code of latest)
Django Version: 2.2.28
Database Version: 10

Additional context
https://github.com/jazzband/django-simple-history/blob/a9476559b24eb22dac02eab0d3a74aabca3b0d8d/simple_history/models.py#L784
Hopefully we are going to avoid using editable=False for future mutable db-fields in our django app.
However to deal with the ones we already have, we are having to copy-pasta-hack parts of simple-history, to force it to not ignore editable=False fields.

This seems to be caused by the model_to_dict function that's actually a function provided by Django itself, located in django.forms.models. This seems to exclude fields that have editable set to False.


CI to run tests

Harpoon 2 does not run tests in CI
Once Harpoon 2 gets more "prod ready" i would like all changes to go through PRs

i am taking it


support $ZDOTDIR instead of $HOME

ZSH manual states

There are five startup files that zsh will read commands from:
$ZDOTDIR/.zshenv
$ZDOTDIR/.zprofile
$ZDOTDIR/.zshrc
$ZDOTDIR/.zlogin
$ZDOTDIR/.zlogout
If ZDOTDIR is not set, then the value of HOME is used; this is the usual case.

So I think using ZDOTDIR will be better for compatibility.

The only place where zsh rcfile is used is ./scripts/setup-shell.sh zsh.
Please add the following code to '~/.zshrc'

~~~
# argc-completions
export ARGC_COMPLETIONS_ROOT="$HOME/.argc-completions"
export ARGC_COMPLETIONS_PATH="$ARGC_COMPLETIONS_ROOT/completions"
export PATH="$ARGC_COMPLETIONS_ROOT/bin:$PATH"
source <(ls -p -1 "$ARGC_COMPLETIONS_ROOT/completions" | sed -n 's/\.sh$//p' | xargs argc --argc-completions zsh)
~~~

Add the above code to '~/.zshrc'? (y/N):

If your rcfile is not ~/.zshrc, then you can manually modify it.  I don't want to introduce more complexity.

I did.
As the document says, users who do not have set ZDOTDIR will automatically fallback to HOME. So I thought using the ZDOTDIR will cover both type of users.


electron_node tsserver.js - running at 100%

Issue Type: Bug
Running a gatsby project with the base styled-components installed - tried the clearing of the typescript version cache.  tsserve processr is running at 100% anytime I edit any file with styled-components. I'm running the insiders edition with no extension active.
VS Code version: Code - Insiders 1.34.0-insider (f89390fcec44a24ce74dfc4efbc1438f35b7357a, 2019-04-18T13:31:41.962Z)
OS version: Darwin x64 18.5.0

System Info

Item
Value

CPUs
Intel(R) Core(TM) i9-8950HK CPU @ 2.90GHz (12 x 2900)

GPU Status
2d_canvas: enabledchecker_imaging: disabled_offflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: enabledrasterization: enabledsurface_synchronization: enabled_onvideo_decode: enabledwebgl: enabledwebgl2: enabled

Load (avg)
3, 2, 2

Memory (System)
32.00GB (10.94GB free)

Process Argv
-psn_0_3789725

Screen Reader
no

VM
0%

Extensions (1)

Extension
Author (truncated)
Version

material-icon-theme
PKi
3.7.0

(1 theme extensions excluded)

Do you have @types/styled components installed under node_modules? If so, what version?

I do not have @types/styled in the project.

Sorry, should be @types/styled-components. I updated my comment for this.
Can you please try collecting the TS Server log to help us investigate:

Set "typescript.tsserver.log": "verbose",
Restart VS Code and reproduce the problem
In VS Code, run the TypeScript: Open TS Server log command
This should open a tsserver.log file in the editor

If you can share that log, I can take a look to see if anything stands out

⚠Warning: The TypeScript log may include information from your workspace, including file paths and source code. If you have any concerns about posting this publicly on Github, just let me know and we can arrange something else. On our side, we only use these logs to investigate issues like this

I do not have @types/styled-components in the project.
I've attached a TS Server log following your steps - let me know if there's anything else I can help to resolve.
tsserver.log

Thanks. I don't see anything in the logs that indicates a long running request or expensive ongoing operation. Was the high cpu usage happening when you collected the log? How long does the spike last?
One idea: can you try creating a jsconfig.json at the root of your project. This file explicitly defines a javascript project and may prevent a few js files under node_modules from being included

It was happening - if I do any typing in the code editor with a Gatsby project open it goes nuts, especially copy and pasting sections of code around - but the second I stop typing the CPU drops back down.
I created a jsconfig.json at the root of the project this morning and it did not help.

FWIW, I also encounter this issue with a similar setup

@LeeCampbellr Can you also please test this again using the latest VS Code insiders build (which includes TypeScript 3.5)
I believe you could have been hitting https://github.com/microsoft/TypeScript/issues/30819  which should now be fixed

Closing this as a likely duplicate of microsoft/TypeScript#30819 based on the repo

I'm consistently getting a 25%-40% CPU usage from electron_node tsserver.js, both on the last stable and last exploration build, with no extensions. Also, between 600 and 1.200 MB of RAM usage:

I don't have @types/styled-components installed under node_modules, however I do have<EMAIL_ADDRESS>I don't know if it can impact as well.


Version support for CPLEX 12.9

According to dep/builds.jl, the current version of this module does not support CPLEX 12.9

See https://github.com/JuliaOpt/CPLEX.jl/pull/226

https://github.com/JuliaOpt/CPLEX.jl/pull/226 is now passing tests. Just waiting for approval.


Retrieve Cover of a Track or Album

Is it possible to retrieve the Artwork image of a track or album using this library?
According to this Stackoverflow answer it can be done. But is there already a built-in way in this library?

Yes, this library fully supports all API endpoints.
There are the README, Javadoc and examples available in this repository that show you how to fetch an album object. Looking at the link you provided, you can see yourself that that is your goal.


Pass down the -Rmodule-loading flag to the frontend

@swift-ci Please test

@swift-ci Please test

mt.exe : general error c101008d: Failed to write the updated manifest to the resource of file "dispatch_overcommit.exe". Access is denied.

@swift-ci Please test Windows

CI is stressed today.


Increase Remote batch size

The remote batch size has not changed since the fast path optimisations.
The optimisations mean we are checking the queue considerably less
often, so the batch should be larger.  This has a dramatic improvement
on performance on a few of the mimalloc microbenchmarks.
It is set to 4096 as this should cover the worse case scenario of only
remote deallocation at 16 bytes for the 2^16 slab size.

@darach @licenser I think this will improve your scenario.  I was investigating a drop in perf on some producer/consumer for micro-benchmarks.  My running of your benchmark is quiet noisy at the moment, but the time spent inside the handle_message_queue_inner has dropped considerably with this change.

That sounds awesome :) we'll give it a try later! Thank you so much :)!

So on the mimalloc-bench configuration for xmalloc-test we get:
# --------------------------------------------------
# benchmark allocator elapsed rss user sys page-faults page-reclaims
xmalloc-testN mi    0.662 204872 56.41 0.64 0 375
xmalloc-testN sn    0.956 314744 50.27 2.85 0 464
xmalloc-testN sn-pr 0.488 314712 46.50 4.96 0 466

The time effectively halves going from 0.956, worse than mimalloc, to considerably better at 0.488.
I think there is room to do a parameter sweep on this value, but I don't have time to investigate at the moment. I think this value is reasonable.


2.9

add some missing methods on RedisPipeline.

@PhantomThief please open the PR against master. We usually backport stuff to the corresponding branches.


Support {rcases} and {drcases}

Support environments {rcases} and {drcases} from the mathtools package. Addresses #2144.
These environments are described on pages 18 & 19 of the mathtools instructions.

Codecov Report

Merging #2149 into master will decrease coverage by 6.3%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #2149      +/-   ##
==========================================
- Coverage   94.89%   88.59%   -6.31%
==========================================
  Files          81       81
  Lines        5285     5285
  Branches      926      926
==========================================
- Hits         5015     4682     -333
- Misses        247      530     +283
- Partials       23       73      +50

Flag
Coverage Δ

#screenshotter
?

#test
88.59% <ø> (+0.03%)
:arrow_up:

Impacted Files
Coverage Δ

src/environments/array.js
77.81% <ø> (-20.63%)
:arrow_down:

src/functions/underline.js
16.66% <0%> (-83.34%)
:arrow_down:

src/functions/verb.js
17.64% <0%> (-82.36%)
:arrow_down:

src/functions/overline.js
23.07% <0%> (-76.93%)
:arrow_down:

src/functions/lap.js
61.9% <0%> (-38.1%)
:arrow_down:

src/functions/delimsizing.js
64.89% <0%> (-31.92%)
:arrow_down:

src/svgGeometry.js
61.29% <0%> (-29.04%)
:arrow_down:

src/functions/phantom.js
72.22% <0%> (-27.78%)
:arrow_down:

src/tree.js
58.82% <0%> (-23.53%)
:arrow_down:

src/functions/symbolsSpacing.js
68.42% <0%> (-21.06%)
:arrow_down:

... and 22 more

Continue to review full report at Codecov.

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

Thank you for the PR!

Nice addition!


const initializer for LocalRb<T, [MaybeUninit<T>, N]>

I want to use Mutex<LocalRb<T, [MaybeUninit<T>, N]>> with static variable but there is no initializer method for LocalRb<T, [MaybeUninit<T>, N]>. Could you provide something like LocalRb<T, [MaybeUninit<T>, N]>::new_const()?
PS: I found another crate that supports this

Thanks for feature request!
I've added const fn constructor for RBs with Array storage (LocalRb, SharedRb)


Google Gemini

We need to import the Google Gemini Pro API. This can help many individuals create exciting projects for free.

See https://github.com/OpenDevin/OpenDevin?tab=readme-ov-file#picking-a-model for Gemini support
Also https://github.com/OpenDevin/OpenDevin/issues/417

Are there any tutorials available? I've attempted several times without success.

#417 is our call for tutorials--if you figure it out, comment there with your setup!


Add untracked support to the GUI

Right now the GUI only supports adding tracked series, not untracked series (i.e. scan style updates).

Fixed by https://github.com/mjnitz02/cbz-tagger/pull/36


feat: ini typography

Current dependencies on/for this PR:

main

PR #170   👈

This comment was auto-generated by Graphite.


Provide Python bindings

To allow wider and easier use of Adiar, we ought to provide Python bindings for the public API. This should be possible by using Cython.
Additional context
Enceuraged/Requested by Alessandro Cimatti to improve traction of Adiar .

Since we already transitively are dependent on Boost via. TPIE we may as well use the quite solid Boost Python library.


[BEAM-12068] Run Dataflow V2 performance benchmarks on Java 11

Please add a meaningful description for your change here

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

@pabloem would you be able to help review and run some seed jobs?

okay this is ready for review then?

Run Seed Job

okay this is ready for review then?

Yes @pabloem, I just want to test the new Jenkins jobs before, I think the seed job only runs in a mirror PR, from a commiter.

okay I'll do that...

Sorry, I pushed two new commits after you ran the seed job. We need to run it again.

I will run the seed job once more after the load jobs run fine in https://github.com/apache/beam/pull/15024

Ok, thanks Pablo!

Seed job has passed on the current PR state: https://ci-beam.apache.org/job/beam_SeedJob/8199/
Also, all the load jobs seem to have run properly.
I haven taken a good look at the code yet. I will do that tomorrow.

merged. thanks everyone!


Improve error message: Cannot find an overload for "" and the argument count: "".

This long-standing error message in PowerShell is accurate, but deceiving, because it doesn't indicate that the caller may have utilized the incorrect method signature. Even though I've been writing PowerShell code for 10+ years, this one still trips me up on a regular basis.
Can we improve the error message to include some more useful information that can point the developer the right direction? For example: "Cannot find a valid method overload for "" and the arguments: "String, Int, CustomDatatype".
I'm open to other ideas, just wanted to call this out.
Cheers,
Trevor Sullivan

It's possible, but I think a little tricky - a good first fix for somebody wanting a deeper understanding of how we use the DLR and dynamic call sites.

Where should that someone look in that case?

Start here:
https://github.com/PowerShell/PowerShell/blob/a2268ab3ecb6c1addffbedba0606aa6c95809dd4/src/System.Management.Automation/engine/runtime/Binding/Binders.cs#L6623
The (possibly) tricky part is generating a correct binding restriction here:
https://github.com/PowerShell/PowerShell/blob/a2268ab3ecb6c1addffbedba0606aa6c95809dd4/src/System.Management.Automation/engine/runtime/Binding/Binders.cs#L6680
I actually might be mistaken - we might already generate good binding restrictions for the error case - but it is something you have to understand before knowing your code is correct.


Rails 5 generating deprecation warning when using strong parameter

I'm using Rails 5 and this deprecation warning is been shown:
DEPRECATION WARNING: Method to_hash is deprecated and will be removed in Rails 5.1, as ActionController::Parameters no longer inherits from hash. Using this deprecated behavior exposes potential security problems. If you continue to use this method you may be creating a security vulnerability in your app that can be exploited. Instead, consider using one of these documented methods which are not deprecated: http://api.rubyonrails.org/v5.0.0/classes/ActionController/Parameters.html
This is my code:

track = API::Tracker::Track.new(track_params)

And this is my work around:

track = API::Tracker::Track.new(track_params.to_h)

Is there any permanent solution?

👍

@cesarjr are your track_params coming in as standard ActionController::Parameters before you run to_h?

@solnic want to push a new version or add me to the gem so I can?

@envygeeks just added you to rubygems.org as a new gem owner so you can push releases

Any fix for this?

Ah I forgot about this issue, I'll add it to my todo for this week.

Sooo?

+1

+1

+1 I'm using this code:
<%= params.except(:controller, :action).merge(rating: i).to_param %>
but am getting the same deprecation warning on to_param method. But, I've read the link and googled for an hour and have found no alternatives to achieve what I want (add a new get param to existing params and putting them into this format:
param=value&param2=value&param3=value

changing
def coerce(attributes)
      ::Hash.try_convert(attributes) or raise(
        NoMethodError, "Expected #{attributes.inspect} to respond to #to_hash"
      )
end

to
def coerce(attributes)
      (attributes.respond_to?(:permitted?) ? attributes.to_h : ::Hash.try_convert(attributes) or raise(
        NoMethodError, "Expected #{attributes.inspect} to respond to #to_hash"
      )
end

in AttributeSet
should fix the deprecation

+1

You can do this in your Virtus model:
def initialize(attributes)
  super(attributes.to_h)
end


Unable to add eventlistener for window events

I am trying to attach an event to window object
browser.addEventListener('onunload', 'window', (e) => { console.log('browser unloaded') console.log(e) })
and I see the below error, I am not sure if this is the right way

Cannot set property 'removeEventListener' of undefined

Is setting an eventListener for window object supported?

experiencing same issue

@anki06 this plugin is deprecated and does probably not work anymore.

@christian-bromann is there any alternate for the same?

And if it is deprecated it should be removed from webdriver io docs or should say somewhere that it is deprecated

It's not in the docs at all and:


Entity registry system

This will be needed for entity saving (#143) and possibly for other things. Not too sure how this should look, but it should be put in the world package.

Resolved by 008b3418d239c304caa0bcede3dd6d643197c69f.


thrift-3841: dart compact protocol incorrectly serializes doubles

The compact protocol expects doubles to be serialized/deserialized with a little endian byte order instead of with a network (big endian) byte order. This fixes that. Tested locally by running a go server and dart client with a service containing a method that sent and received a double and making sure what was received on both ends was correct.
@markerickson-wf @tylertreat-wf @stevenosborne-wf

+1

+1

@brianshannan-wf Can you close this PR?  I did not construct the commit message correctly for it to be closed automatically.


Update conditional-access-admin-roles.md

Added missing Teams Administrator role to list

@MicrosoftGuyJFlo
Can you review the proposed changes?
Important: When the changes are ready for publication, adding a #sign-off comment is the best way to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

That is not part of the core list of roles as defined in the Entra recommendation. More roles can absolutely be added to your policy as you see fit, if you are using them.

#please-close


Issue with k8s.io/docs/tutorials/kubernetes-basics/cluster-interactive/

This is a...

[ ] Feature Request
[ ] Bug Report

Problem:
Proposed Solution:
Page to Update:
http://kubernetes.io/...

hi @preethbshetty  - this looks like you've opened a bug on accident. I'm going to close this, but if there's an issue you'd like to bring up related to the documentation, please re-open and provide the details


The selected option on the original settings page is changed after splitting the settings tab

Storage Explorer Version: 1.27.0
Build Number: 20221124.10
Branch: rel/1.27.0
Platform/OS: Windows 10/Linux Ubuntu 18.04/MacOS Ventura 13.0 (Apple M1 Pro)
Architecture: ia32/x64
How Found: Ad-hoc testing
Regression From: Previous release (1.26.1)
Steps to Reproduce

Launch Storage Explorer -> Open 'Settings'.
Select one option like: Data Explorers on the settings page -> Right click the settings tab -> Click 'Split'.
Observe the original settings page.
Check whether the selected option is Data Explorers.

Expected Experience
The selected option is Data Explorers.
Actual Experience
The selected option is Application.
Addition Context

This issue doesn't reproduce when the option 'Application' is selected on the original settings page.
Go back the correct option after closing the split tab.
Here is the record:

@MRayermannMSFT @JasonYeMSFT @richardMSFT Do we know if this preserving selected category was ever intentional? If so, how was this done?

I see what's happening now. The selected category is a passive reflection of what section is in the visible area in the settings panel. After splitting, since the space of visible area has changed, the selected category will be updated to reflect what is currently being displayed. This behavior is by design.


Admin ghosts no longer require an empty hand to pull.

About the PR

see cl
Why / Balance

Technical details

Media

Requirements

[X] I have read and am following the Pull Request and Changelog Guidelines.
[X] I have added media to this PR or it does not require an ingame showcase.

Breaking changes

Changelog

:cl:

tweak: Admin ghosts no longer need a free hand to pull things.

Summary by CodeRabbit

New Features

Admin ghosts can now perform pulling actions without needing a free hand.
Introduced multiple user interface actions for admin ghosts, enhancing their interaction capabilities, including options for solar display, communications, radar, cargo, crew monitoring, and station records.

Could you also add stuff that every admin add to script?
Like:

SprintSpeed: 10-20
WalkSpeed: 4-5
ShowCriminalRecordsIconComponent
ShowJobIconsComponent
ShowMindshieldIconsComponent
ShowSyndicateIconsComponent


#6 still occurs

Just complied today from expermential, got this error:
which: no decrypt.py in (/usr/local/bin:/usr/bin:/bin:/usr/local/sbin:/var/lib/flatpak/exports/bin:/usr/lib/jvm/default/bin:/usr/bin/site_perl:/usr/bin/vendor_perl:/usr/bin/core_perl:/var/lib/snapd/snap/bin)
Decrypting: Metroid Prime - Federation Force (CTR-U-BCAE) (v1.2.0) (U).cia...
Unsupported CIA

Could you try which decrypt.py on terminal and post the output? Another thing, which OS are you using?

I'm using linux, specifically arch. Here's the output:
which: no decrypt.py in (/usr/local/bin:/usr/bin:/var/lib/snapd/snap/bin)

Oh, that’s the point, linux quit cia-unix since the command which send an error return code.
 I guess I'll just have to use my 3ds then.

91f845c44d474eaf34d2b86e6d0be1b0e5ff81db fixed that


👷 Change build system from Poetry to Hatch

Poetry has a probrem in dynamic versioning even though poetry-dynamic-versioning can cover it.
We have to change version in pyproject.toml in order to reflect the version to a package directly installed from GitHub.
Flit seems nice, but we feel Hatch is becoming popular.
If Hatch becomes de-facto standard, it is good to deal with it as early as possible .

Codecov Report

Merging #24 (629e1d9) into master (38f5030) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##            master       #24   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            5         5
  Lines          190       190
=========================================
  Hits           190       190

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Add missing variables from 1-resman to 2-security stage. Fix 2023

The 2-security stage is missing variables that were added to the 1-resman stage within the last 12 months. We've resolved this issue by including all the newly added variables in the output file 1-resman.auto.tfvars.json.
This pull request adds the missing variables to the 2-security stage to ensure that all variables are consistently defined across all stages, eliminating any Terraform warning that might have been issued. Please review and consider merging this pull request.

Thanks for this, but the warnings are harmless, and variables are only meant to be present if they serve a purpose in the specific stage. I'm going to close this, feel free to continue the discussion of you need more clarifications.


Wiki copy error of the example code from package diagram

The example code from packages is wrong, probably a copy error from deployment example.
https://github.com/jaime-olivares/yuml-diagram/wiki#package-diagram

Solved in the wiki