Allow didcomm:// protocoll as invitation

Hello,
I've got a verification service which provides an invitation in the format didcomm://<IP_ADDRESS>?m=eyJAdHlwZSI6ImRpZD...7fX19.
The base64 decodes to:
{
  "@type": "did:sov:BzCbsNYhMrjHiqZDTUASHg;spec/present-proof/1.0/request-presentation",
  "@id": "2f237261-b670-46c3-98ed-398a36d0e05c",
  "request_presentations~attach": [
    {
      "@id": "libindy-request-presentation-0",
      "mime-type": "application/json",
      "data": {
        "base64": "ewogICJuYW1lIjogIlByb29mIHJlcXVlc3QiLAogICJub25fcmV2b2tlZCI6IHsKICAgICJmcm9tIjogMCwKICAgICJ0byI6IDE2NzM5NTI2MTIKICB9LAogICJub25jZSI6ICI1NjA0NzUxODkiLAogICJyZXF1ZXN0ZWRfYXR0cmlidXRlcyI6IHsKICAgICJJZEF0dHJpYnV0ZU5hbWUiOiB7CiAgICAgICJuYW1lcyI6IFsKICAgICAgICAiZmlyc3ROYW1lIiwKICAgICAgICAiZmFtaWx5TmFtZSIsCiAgICAgICAgImRhdGVPZkJpcnRoIiwKICAgICAgICAiZGF0ZU9mRXhwaXJ5IgogICAgICBdLAogICAgICAicmVzdHJpY3Rpb25zIjogWwogICAgICAgIHsKICAgICAgICAgICJjcmVkX2RlZl9pZCI6ICJGbmhZcGJScXVRZTZUYUVDZ2RZeFk1OjM6Q0w6MTI6ZGVmYXVsdCIKICAgICAgICB9CiAgICAgIF0KICAgIH0sCiAgICAiZ3Vlc3RBdHRyaWJ1dGVOYW1lIjogewogICAgICAibmFtZXMiOiBbCiAgICAgICAgImZpcnN0TmFtZSIsCiAgICAgICAgImxhc3ROYW1lIiwKICAgICAgICAidGl0bGUiLAogICAgICAgICJlbWFpbCIsCiAgICAgIF0sCiAgICAgICJyZXN0cmljdGlvbnMiOiBbCiAgICAgICAgewogICAgICAgICAgImNyZWRfZGVmX2lkIjogIkZuaFlwYlJxdVFlNlRhRUNnZFl4WTU6MzpDTDoxNzpndWVzdC1sb2NhbC1hYmMwMDEiCiAgICAgICAgfQogICAgICBdCiAgICB9CiAgfSwKICAicmVxdWVzdGVkX3ByZWRpY2F0ZXMiOiB7fSwKICAidmVyc2lvbiI6ICIxLjAiCn0="
      }
    }
  ],
  "~service": {
    "recipientKeys": [
      "B47YV1iNuYdJs181yrQBbqL5Du8YXnoFzvggkCiUsCvq"
    ],
    "routingKeys": [],
    "serviceEndpoint": "http://<IP_ADDRESS>:10004",
    "endpointName": "Verification Agent"
  },
  "~thread": {
    "thid": "2f237261-b670-46c3-98ed-398a36d0e05c",
    "sender_order": 0,
    "received_orders": {}
  }
}

Currently (v0.5.6), when I pass the url to Add connection, I get the following error:
E0117 10:54:05.700420      13 server.go:72] Returning GQL error connect-0: input: connect http conn serve: http conn serve: invitation translate: invalid invitation url format
Are there any plans to allow this?

Hi, and thank you for your feedback! Highly appreciated.
It will help us to test this: What agent is the verifier? ACAPy, something else?
I have to check the Aries protocol descriptions because, at first glance, the actual message is just the plain proof request. I suppose the thread.ID would be the connection_id, i.e., pairwise?
I'll get back to you when I have done some studies.

Hi lainio,
thanks for your quick response!
Yes the verifier is an ACA-Py 0.7.5.
The message is a connectionless proof request and the information within the ~service decorator is everything needed to reply to the verifier.
The thread id is the same thread id from the response of ACA-Py when you create the proof request via POST /present-proof/create-request.

Hi @pstuermlinger Unfortunately Findy Agency does not yet support connectionless proof requests 🙁  Let's see if we can fit this to our roadmap in near future 🙂

My intuition was at least half right. This is not a straightforward OOB case, but it's related to ~service decorator.
Here is a good summary of connectionless proof of request with ACAPy.
Because the service decorator is at the proposed state, and the connectionless request overlaps with OOB invitations, I would like to ask how important this feature is for you and what is your commitment to the findy-agency?
The actual service decorator doesn't seem such a big task at first glance, but as we all know devil lives in the details.
Anyway, I'll look into this more carefully and get back to you: what, when, how, you know 😃

That's a pitty. Any chance OOB could work? I saw there were some PRs at findy-vault. But I'm not sure how mature this feature is already and what exactly to pass into the "Add connection" textfield.

There has been done some initial work to core and vault to support out-of-band-invitations. Nevertheless, current implementation does not yet know how to parse and pass on the attached proof request, so I am afraid the parsing with the attached content will fail no matter if you use the URL or JSON format.

My intuition was at least half right. This is not a straightforward OOB case, but it's related to ~service decorator.
Here is a good summary of connectionless proof of request with ACAPy.
Because the service decorator is at the proposed state, and the connectionless request overlaps with OOB invitations, I would like to ask how important this feature is for you and what is your commitment to the findy-agency?
The actual service decorator doesn't seem such a big task at first glance, but as we all know devil lives in the details.
Anyway, I'll look into this more carefully and get back to you: what, when, how, you know 😃

AFAIK the ~service is mainly understood by mobile agents like Lissi, Trinsic and Esatus. Funnyn enough, ACA-Py itself does not have an endpoint to input such a message.
We're using Findy currently for internal development and showcase demo, since the network is highly restricted an we can't use any native mobile agent here. Findy is great to circumvent this problem.
Now we already started to implement a connectionfull present-proof as fallback mechanism.
Thanks @lainio and @lauravuo for your help!
Closing since all my questions have been answered.


Error when other plugins manage windows.

I saw the following error multiple times - when using ctrl-p and closing the ctrlp window via ctrl-c. Do you have any ideas?

It seems that wintabs is accessing a window that no longer exists, which results in this NPE. I just landed a possible fix (fixing the NPE) but not sure if it fixes the actual problem, can you try it?

I think this might have fixed it! Thank you.


RightToLeft Direction in BitCarousel causing display issues

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the bug
GitHub Issue for BitPlatform: RightToLeft Direction in BitCarousel causing display issues
Issue Description:
When using the RightToLeft Direction in BitCarousel, the carousel breaks, and the last image covers the section. Additionally, when the image changes, it goes under the image that was previously displayed. Removing the RightToLeft Direction resolves the issue.
Code Snippet:
<BitCarousel InfiniteScrolling="true" Direction="BitDirection.RightToLeft" Style="@(MustShoeImage? "width: 100%;height: 211px;" : "display:none")">
    @foreach (var image in Images)
    {
        <BitCarouselItem>
            <img class="image set-max-width carouselItem-image" src="@image" @onload="HandleImageOnLoaded" @onerror="HandleVehicleMarkerImageError" />
        </BitCarouselItem>
    }
</BitCarousel>

Environment:

BitPlatform Template Version: v-8.7.1
Browser/Device: Chrome-Android , Safari-IOS , Edge-Windows

Screenshots/Additional Information:
https://github.com/bitfoundation/bitplatform/assets/27281206/36731e12-1715-4147-8a67-4ea930d0d4f5
Note:
Please prioritize fixing this issue as it affects the live product. Keep us informed about the progress and resolution. Thank you!
Expected Behavior
The BitCarousel should function correctly when using the RightToLeft Direction, and images should display without covering the section or going under the previously shown image.
Steps To Reproduce

Use the provided code snippet with RightToLeft Direction in BitCarousel.
Observe the carousel breaking and the last image covering the section.
Notice that when the image changes, it goes under the previously displayed image.

Exceptions (if any)
No response
.NET Version
8.0.100
Anything else?
Additional Information:

Issue is impacting the live product.
The issue does not occur when the RightToLeft Direction is not used.

Hope this video helps :
https://github.com/bitfoundation/bitplatform/assets/27281206/d95db681-1128-4907-8113-df09959d543f

@hootanht I believe there is something in your custom CSS styles that conflicts with the CarouselItem style that creates this issue since the RTL sample in the Carousel demo page works fine.
Could you please share the source code so we can check the problem?

Sure, here is the custom CSS :
.carousel-wrapper {
    ::deep .bit-icon--ChevronRight {
        background: url("./images/icons/chevron-right.svg") center/cover
            no-repeat !important;
        color: transparent !important;
    }

    ::deep .bit-icon--ChevronLeft {
        background: url("./images/icons/chevron-left.svg") center/cover
            no-repeat !important;
        color: transparent !important;
    }

    ::deep .bit-csl-rbt {
        outline: 0;
        opacity: 1 !important;
        text-decoration: none;
        color: var(--bit-clr-act-hover-fg-pri);
    }

    ::deep .bit-csl-lbt {
        outline: 0;
        opacity: 1 !important;
        text-decoration: none;
        color: var(--bit-clr-act-hover-fg-pri);
    }

    ::deep .bit-csl-dot .current {
        background-color: $primary-sky-night !important;
        width: 10px !important;
        height: 10px !important;
    }

    ::deep .bit-csl-dot {
        display: flex !important;
        justify-content: center !important;
        align-items: center !important;
    }

    ::deep .bit-csl-dot div {
        border-radius: 50% !important;
        width: 7px !important;
        height: 7px !important;
        display: inline-block;
        margin-right: 5px !important;
        margin-left: 0px !important;
        background-color: $primary-soap !important;
    }
}

.carouselItem-image {
    height: 200px;
    display: block;
    margin: auto;
}

Dear Saleh, I uploaded another video that talks about this problem and hope this will help you.

@msynk

@hootanht Thanks for the additional data.
first, overriding internal/undocumented CSS classes is not recommended and should not be used IMO since we might overhaul these in the future and your code might break.
please file issues for your requirements so we can add new features to support our customer's needs.
second, it seems the issue is coming from this style:

could you please remove these CSS styles and let me know the result?
third, it would be much better to share the whole source code so we can also debug the problem.

Dear @msynk,
The issue at hand is not caused by CSS code. I have thoroughly tested various scenarios, and as of now, a solution has been identified. However, it comes at the cost of sacrificing our primary functionality, namely, displaying the carousel in a right-to-left orientation for our Persian customers.
In the scenario that I tested without encountering any issues (excluding the commenting out of SCSS codes), I removed `InfiniteScrolling` and Direction from the code. While this approach resolves the problem, it regrettably eliminates the right-to-left feature for our Persian clientele.

@hootanht It seems I didn't clearly express what I need. I need a full source code (a zip file or a github repo or ...) to start investigating the problem.
btw, I've tested different scenarios with our carousel component, and each one works fine. so I need a reproduction of the problem from your side to investigate further.

dear @msynk I created a repository and applied I style and functionality that we have including layout. but everything is fine. Can we have a meeting to solve this problem?
https://github.com/hootanht/CarouselProblem
If yes, I DM, you.

@msynk Can we check this problem tomorrow?


Better support for vi-mode indicator

When using this prompt with vi keybindings, the mode-indicator is prepended to the first line of the prompt.
This will put the mode-indicator at the start of the second line, before the '❯' character.

@smh Thank you for adding this and sorry for the late response. Would you be willing to rebase this PR (#45) and #46 with master so we can release it?

@rafaelrinaldi no worries. Both rebased now. Love this prompt, but can't live without vi 😄

This looks great to me as is. Merging it in as well.
Thanks a lot @smh!


Unknown USB Device (Device Descriptor Request Failed)

Hi SensorLogic,
After I connected the sensor to my PC via USB, the red led blinked at 1 Hz. Then I followed the guide to go to the Device Manager. Under the Universal Serial Bus Controllers, I didn't see USB Serial Device (COM8). Instead, I saw Unknown USB Device (Device Descriptor Request Failed). I then tried to update the USB driver following your guide, but this issue is still there. What should I do next? What might be the problem? Any help is appreciated.
Many thanks!

It typically just works without even updating the driver.  Do you have any other NXP devices plugged in? Have you tried another computer or operating system, i.e. Linux?
Or, try deleting the driver and plugging the unit back  in.
-will


Explanation regarding setting the maximum number of training windows creation/sampling as a hyperparameter for the DeepAREstimator

Discussed in https://github.com/awslabs/gluon-ts/discussions/1889

Originally posted by vamp-ire-tap February 16, 2022
Hello, can anyone explain how the training windows are generated in the default call to the DeepAREstimator? I see that on the following line:
https://github.com/awslabs/gluon-ts/blob/8132e2462b649455ade4dbc66cc36114df1796bc/src/gluonts/model/deepar/_estimator.py#L271
There is a call to create the ExpectedNumInstanceSampler(num_instances=1.0, min_future=prediction_length) if there is no train_sampler is provided. By looking at the definition of the ExpectedNumInstanceSampler on the following:
https://github.com/awslabs/gluon-ts/blob/8132e2462b649455ade4dbc66cc36114df1796bc/src/gluonts/transform/sampler.py#L139
I see that num_instances=1.0 tells the sampling of training windows can occur by choosing any valid point in time series history and sampling some points before and after it to create a training window. However, the main problem I am having is to understand what hyperparameter dictates how many training windows should be sampled at max. If we look at the DeepAR paper at https://arxiv.org/pdf/1704.04110.pdf, then in table 3 it is stated that they take #500K training examples for the traffic dataset for example. How can I set this 500K hyperparameter by calling the DeepAREstimator or the ExpectedNumInstanceSampler?

Closed for double posting: anyone interested please refer to the discussion linked in the OP

it is not so much a discussion but an explanation that is requested from the developers/maintainers of the library, that is why it was posted here as an issue as well. I hope one of the contributors can help answer this question above

@vamp-ire-tap I understand, however, we rely on issues in order to track actionable items like missing features, bugs, proposals, and so on. I think discussions are better suited for questions and answers: They don’t get “closed” and have ranked answers.
Of course if a discussion turns out to identify an actual issue, it makes perfect sense to turn it into an issue so that developers can pick it up.

I understand, I hope I receive a response soon there then; plus i'd be grateful to you if you could answer maybe

Could the authors/contributors of the library answer the question posted above?


Option to let the mask return it's view value. And removed unwarranted responsibility

The mask cannot return it's view value, only the raw value.
In some cases, we would like the view value to be return.
This way we can simply just do:
<input type="text" name="CellPhone" ng-model="field.Value"
                             ui-mask="99999999" ui-mask-use-viewvalue="true" >

Also removed the responsibility of setting the required validation of ngModel. // No longer valid

+1

Closing since @inolasco gave a solution

How can I use the model-view-value?

Like this:
<input name="phone" type="text" ng-model="phoneNumber" ui-mask="(999) 999-9999" model-view-value="true">

@inolasco thanks for helps! but this doesn't works for me, I try

Make sure you're using the latest or recent version. I copied that almost verbatim from an app where it works fine. If it helps, here's how I have the dependency in my bower.json, although in my case I only require the mask module, hence the 'bower-mask':
"angular-ui-utils": "bower-mask"


Name-Value argument BaseValue not working

When specifying an initial base value using 'BaseValue no changes are made.
For example,
y = [8 15 33; 30 35 40; 50 55 62];
barh(y,'BaseValue',25)

fig2plotly(gcf, 'offline', false);

The previous code should produce an horizontal bar chart in which the 'zero value' or base value should be 25, however, it starts at 0 as usual.
Expected and generated (https://chart-studio.plotly.com/~xarico10/573/#/) plots:

Examples on using 'BaseValue can be found on:
https://www.mathworks.com/help/matlab/ref/barh.html#mw_caa2d25c-e3b8-47db-ad74-c7643062cbf8

Sorry Uzair, was looking for something in bar reference and found a solution to this...
Vertical bars without Basevalue argument...

Horizontal bars with BaseValue=25

Vertical bars with BaseValue=30


Older SSL/TLS versions

Is there a way to configure himalaya to connect even to servers that use outdated ssl versions?
himalaya
Error: cannot connect to imap server

Caused by:
    error:1408F10B:SSL routines:ssl3_get_record:wrong version number:ssl/record/ssl3_record.c:331:

Is there a way to configure himalaya to connect even to servers that
use outdated ssl versions?

No, there is no way to configure such thing. Do you know which SSL
version your server uses?

Strange because TLS 1.0 seems to be supported by native-tls: https://github.com/sfackler/rust-native-tls/blob/774a19cecb694dc6e414c2ada9bd5f768a0b9b75/src/imp/openssl.rs#L32. Could you share your Himalaya config?

Could you share your Himalaya config?

Quite basic, and I removed the (private) domain name:
display-name = "Test"
downloads-dir = "~/test/downloads"

[myserver]
default = true
email =<EMAIL_ADDRESS>
backend = "imap"
imap-host = "imap.myserver.org"
imap-port = 143
imap-login =<EMAIL_ADDRESS>imap-passwd-cmd = "pass<EMAIL_ADDRESS>
sender = "smtp"
smtp-host = "smtp.myserver.org"
smtp-port = 25
smtp-login =<EMAIL_ADDRESS>smtp-passwd-cmd = "pass<EMAIL_ADDRESS>

Is there anything I can do to further debug this?

Is there anything I can do to further debug this?

Did you try to use the port 993 instead of 143? Or to use STARTTLS with
the option imap-starttls = true?

I tried that and started himalaya. Now it hangs since several minutes. It can't be downloading since it didn't get my password via pass yet. And I assume there would also be a connection timeout.

I tried that and started himalaya. Now it hangs since several minutes.

So something is wrong with the config. I am a bit out of idea :/ is
there any way for me to have an access so I can try to debug also?

I will close this for now since I didn't find the time to further investigate.
I hope it's okay if I comment later again.

Yes sure! Just for information: the bug trackers moves away from GitHub, you need now to send an email at<EMAIL_ADDRESS>see https://github.com/soywod/himalaya/blob/master/CONTRIBUTING.md#contributing.


Fastapi

Migrating from Flask to FastAPI

Due to flask's limitations over performance as well as the lack of asynchronous execution. We have migrated to FastAPI an equally lightweight web framework.
FastAPI comes with automatic data vaildation providing with asynchronous features and a 800% performance boost when used with gunicorn compared to the previous uwsgi nginx flask setup.
FastAPI also has better testing interface that can be used with pytest for creating tests

Added index page
Once the server is launched , we can access the root path of server to find an index page with instructions

FastAPI also comes with with a Swagger UI that allows developers to explore the routes.

Status Codes
Previously the outcome of any server call was sent in the response content. Now http status codes are used.
Logging
Two loggers have been set up not including the gunicorn logger that logs to stream

The root logger logs every request inside server.log. Logs with ERROR level are logged to stream
The execution logger logs the argorithm execution to process.log

Auth changes
The authentication and authorization format is now up to OAuth2 standards

This branch will be commited once the required changes are made to toolkit

This branch will be commited once the required changes are made to toolkit


Nothing happens after install

Description
Hello there everyone,
I'm trying this plugin out, but nothing happens after plugin install, none of the configs added seem to render any effect.
If I change the configs to something that the plugin is not expecting, I get error messages, so the code is indeed being evaluated.
Expected Behavior
Plugin should work as show in the docs
Actual Behavior
Nothing happens. No hovers, No code actions, anything..
Details
Reproduce
I've installed lspsaga and configured it using the following snippet:
local saga = require 'lspsaga'
local action = require("lspsaga.action")
local action = require("lspsaga.codeaction")

saga.init_lsp_saga{
  border_style = "rounded",
  diagnostic_header = { " ", " ", " ", "ﴞ " },
  code_action_icon = "",
  code_action_num_shortcut = true,
  code_action_lightbulb = {
    enable = true,
    sign = true,
    sign_priority = 20,
    virtual_text = true,
  },
  finder_separator = "  ",
  max_preview_lines = 10,
  finder_action_keys = {
    open = "o",
    vsplit = "s",
    split = "i",
    tabe = "t",
    quit = "q",
    scroll_down = "<C-f>",
    scroll_up = "<C-b>", -- quit can be a table
  },
  code_action_keys = {
    quit = "q",
    exec = "<CR>",
  },
  rename_action_quit = "<C-c>",
  definition_preview_icon = " ",
  symbol_in_winbar = {
    in_custom = false,
    enable = false,
    separator = ' ',
    show_file = true,
    click_support = false,
  },
}

vim.keymap.set("n", "K", "<cmd>Lspsaga hover_doc<CR>", { silent = true })
vim.keymap.set("n", "gd", "<cmd>Lspsaga preview_definition<CR>", { silent = true })
vim.keymap.set("n", "<leader>ca", "<cmd>Lspsaga code_action<CR>", { silent = true,noremap = true })
vim.keymap.set("v", "<leader>ca", "<cmd><C-U>Lspsaga range_code_action<CR>", { silent = true,noremap = true })
vim.keymap.set("n", "gh", "<cmd>Lspsaga lsp_finder<CR>", { silent = true,noremap = true})
vim.cmd [[autocmd! CursorHold,CursorHoldI * lua require'lspsaga.diagnostic'.show_line_diagnostics()]]

vim.keymap.set("n", "<C-f>", function()
  action.smart_scroll_with_saga(1)
end, { silent = true })

vim.keymap.set("n", "<C-b>", function()
  action.smart_scroll_with_saga(-1)
end, { silent = true })

click_support = function(node, clicks, button, modifiers)
    -- To see all avaiable details: vim.pretty_print(node)
    local st = node.range.start
    local en = node.range['end']
    if button == "l" then
        if clicks == 2 then
            -- double left click to do nothing
        else -- jump to node's starting line+char
            vim.fn.cursor(st.line + 1, st.character + 1)
        end
    elseif button == "r" then
        if modifiers == "s" then
            print "lspsaga" -- shift right click to print "lspsaga"
        end -- jump to node's ending line+char
        vim.fn.cursor(en.line + 1, en.character + 1)
    elseif button == "m" then
        -- middle click to visual select node
        vim.fn.cursor(st.line + 1, st.character + 1)
        vim.cmd "normal v"
        vim.fn.cursor(en.line + 1, en.character + 1)
    end
end

Environment

nvim --version output:

NVIM v0.7.0
Build type: Release
LuaJIT 2.1.0-beta3
Compiled by<EMAIL_ADDRESS>
Features: +acl +iconv +tui
See ":help feature-compile"

   system vimrc file: "$VIM/sysinit.vim"
  fall-back for $VIM: "/opt/homebrew/Cellar/neovim/0.7.0/share/nvim"

Run :checkhealth for more info

Operating system: macOS Monterey 12.4
lspsaga commit: 9922e17c8e535328462fa246cc8fde32d9f72371

You need to install lsp servers using https://github.com/neovim/nvim-lspconfig first
@glepnir Maybe the click_support readme is a bit misleading?

I have a couple servers installed already:

Let me try and remove winbar

I thinkn there's some other plugin interaction preventing this from loading, hitting K over *http at line 10 renders it at the bottom section:

If anyone could please give me a hint on what might be taking place here or how to debug this, I'd appreciate it a lot!
Here's my dotfiles, precisely the lsp setup: https://github.com/lucazz/dotfiles/tree/master/nvim/lua/config/lsp

FWIW, I had a similar issue and it turned out that I still had coc.nvim enabled. After using only nvim-lspconfig, Saga started working as expected.

@lucazz  winbar is a feature of neovim 0.8 . it can not work in 0.7. and use verbose map K. check your k map , I think has a conflict

@glepnir I am running their config, there is no mapping for K, it's the default man.vim behavior
The problem is they thought nvim-lsp-installer auto setup servers configuration

I think the best way is use a min config to test. so maybe I can close this issue?

@lucazz In nvim/lua/config/lsp/lsp-installer your code is broken
Here is a working version for lua

If you are gonna source every plugin file in lua put them in nvim/after/plugin and they will load automatically after each plugin is loaded. For example, I put it in nvim/after/plugin/lspsaga.lua as shown in the screenshot.

Heya @nullchilly, thanks for the tip!
I did managed to fix my lsp-installer setup and everything works like a charm now.

I have exactly same problem, lsp server is installed and running but no results from lspsaga
Moreover lspsaga is not giving any erorrs
How to debug this situation?

first check the file have lsp server information LspInfo then check the lspsaga loaded if you use packer you can try
lua print(packer_plugins['lspsaga.nvim'].loaded) true mean is load . if you don't use packer check the rtp path
set rtp? it should be have lspsaga. press Lspsaga Tab if nothing happen that mean lspsaga not load

Lspsaga looks like it was loaded, I have the command available.
but LspInfo is probably wrong:
Language client log: /home/rk/.cache/nvim/lsp.log
Detected filetype:   go
0 client(s) attached to this buffer:
Configured servers list:

So this shows that nothing connected to lsp, but I can't tell if lsp is running correctly

0 client(s) attached to this buffer:

there should be 1 of gopls. so the gopls doesn't start for this buffer

OK, thanks for quick response. I didn't configured lspconfig correctly, so lspsaga couldn't use it.
I wish lspsaga have some kind of Lspsaga status command so it could tell me if lsp is reachable


[Update Request]: Remove Microsoft.VCRedist.2015+.x64 version 14.40.33617.1

What type of update are you requesting?
A new version of an existing package
Current Package Identifier
Microsoft.VCRedist.2015+.x64
Package Version
14.40.33617.1
Please describe the changes you would like to see
This is a pre-release version and should not be in the the GA identifier.
https://github.com/microsoft/winget-pkgs/pull/144318#issuecomment-2005611568
revert https://github.com/microsoft/winget-pkgs/pull/144318
related #147134

related #147134

fyi @grimson73 @stephengillie


Dependabot updates

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 97.24%. Comparing base (b8a3c69) to head (c73a78e).
Report is 2 commits behind head on main.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main      #11   +/-   ##
=======================================
  Coverage   97.24%   97.24%
=======================================
  Files           8        8
  Lines         218      218
  Branches       40       40
=======================================
  Hits          212      212
  Misses          1        1
  Partials        5        5

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


Can't build extension with the xk6-out-influxdb extension

Getting error every time when building xk6-mock with https://github.com/grafana/xk6-output-influxdb.  Wondering if this can be adjusted to work together as I am trying to upgrade to influxdb-v2 and have mocking as well.

Distributor ID: Ubuntu
Description:    Ubuntu 20.04.4 LTS
Release:        20.04
Codename:       focal

it doesn't build with k6 version >= v0.33.0 even with only one xk6-mock
Command: xk6 build v0.33.0 --with github.com/szkiba/xk6-mock
2022/10/04 19:12:48 [INFO] Temporary folder: /Users/antonparaskiv/GolandProjects/shipment-ext/buildenv_2022-10-04-1912.4138967645
2022/10/04 19:12:48 [INFO] Initializing Go module
2022/10/04 19:12:48 [INFO] exec (timeout=10s): /Users/antonparaskiv/sdk/go1.17.13/bin/go mod init k6
go: creating new go.mod: module k6
2022/10/04 19:12:48 [INFO] Pinning versions
2022/10/04 19:12:48 [INFO] exec (timeout=0s): /Users/antonparaskiv/sdk/go1.17.13/bin/go mod edit -require github.com/szkiba/xk6-mock@latest
2022/10/04 19:12:48 [INFO] exec (timeout=0s): /Users/antonparaskiv/sdk/go1.17.13/bin/go mod tidy -compat=1.17
2022/10/04 19:12:48 [INFO] exec (timeout=0s): /Users/antonparaskiv/sdk/go1.17.13/bin/go mod tidy -compat=1.17
2022/10/04 19:12:48 [INFO] Writing main module: /Users/antonparaskiv/GolandProjects/shipment-ext/buildenv_2022-10-04-1912.4138967645/main.go
2022/10/04 19:12:48 [INFO] exec (timeout=0s): /Users/antonparaskiv/sdk/go1.17.13/bin/go mod edit -require<EMAIL_ADDRESS>2022/10/04 19:12:48 [INFO] exec (timeout=0s): /Users/antonparaskiv/sdk/go1.17.13/bin/go mod tidy -compat=1.17
2022/10/04 19:12:48 [INFO] exec (timeout=0s): /Users/antonparaskiv/sdk/go1.17.13/bin/go mod tidy -compat=1.17
2022/10/04 19:12:49 [INFO] Build environment ready
2022/10/04 19:12:49 [INFO] Building k6
2022/10/04 19:12:49 [INFO] exec (timeout=0s): /Users/antonparaskiv/sdk/go1.17.13/bin/go mod tidy -compat=1.17
2022/10/04 19:12:49 [INFO] exec (timeout=0s): /Users/antonparaskiv/sdk/go1.17.13/bin/go build -o /Users/antonparaskiv/GolandProjects/shipment-ext/k6 -ldflags -w -s -trimpath
# github.com/szkiba/xk6-mock
../../../go/pkg/mod/github.com/szkiba/xk6-mock@v0.1.2/http.go:36:15: undefined: "go.k6.io/k6/js/modules/k6/http".ToURL
../../../go/pkg/mod/github.com/szkiba/xk6-mock@v0.1.2/http.go:41:10: cannot assign error to err in multiple assignment
../../../go/pkg/mod/github.com/szkiba/xk6-mock@v0.1.2/http.go:51:12: cannot assign httpext.URL to orig in multiple assignment
../../../go/pkg/mod/github.com/szkiba/xk6-mock@v0.1.2/http.go:51:12: cannot assign error to err in multiple assignment
2022/10/04 19:12:49 [INFO] Cleaning up temporary folder: /Users/antonparaskiv/GolandProjects/shipment-ext/buildenv_2022-10-04-1912.4138967645
2022/10/04 19:12:49 [FATAL] exit status 2

but it's ok for k6 version v0.32.0
xk6 build v0.32.0 --with github.com/szkiba/xk6-mock
other extensions may require newer version of k6

Thank you for feedback. It's fixed now, version v0.2.0
(the minimum k6 version is v0.43.0)

fixed in v0.2.0


Cannot load vendor templates

I'm stuck with loading vendor templates, that was automagically done in "old" Symfony apps.
For example, the menu from KnpMenu (KnpMenuBundle::menu.html.twig) is not loaded.
Steps to reproduce:

install this app
composer require  knplabs/knp-menu-bundle
create a menu as a service, as described in bundle docs (example provided below)
call menu from a template

Expected: the menu should show
Actual: exception Unable to find template "KnpMenuBundle::menu.html.twig" (looked into: /home/user/symfony-demo/templates, /home/user/symfony-demo/vendor/symfony/twig-bridge/Resources/views/Form).
Note: for now I added the path of vendor template in twig.yaml and of course if works, but I think this is just an ugly workaround and I expect that any vendor template should work out of the box

Menu class example, provided to easy reproduction:
<?php
// src/Menu/Builder.php
namespace App\Menu;

use Knp\Menu\FactoryInterface;

class Builder
{
    private $factory;

    public function __construct(FactoryInterface $factory)
    {
        $this->factory = $factory;
    }

    public function createMainMenu(array $options)
    {
        $menu = $this->factory->createItem('root');
        $menu->addChild('Home', array('route' => 'homepage'));

        return $menu;
    }
}

added in config/services.yaml
    app.menu_builder:
        class: App\Menu\Builder
        arguments<EMAIL_ADDRESS>        public: true
        tags:
            - { name: knp_menu.menu_builder, method: createMainMenu, alias: main }

added in templates/default/homepage.html.twig (inside body block)
{{ knp_menu_render('main') }}

Other example, easy to reproduce: try to use KnpPaginatorBundle with provided bootstrap3 template, getting Unable to find template "KnpPaginatorBundle:Pagination:twitter_bootstrap_v3_pagination.html.twig"

When the templating component is not installed and enabled, you cannot use the Bundle:Folder:file.html.twig notation for template name.
you would have to use the native Twig syntax (which is enabled in Symfony since 2.6 IIRC, but definitely a long time): @KnpPaginator/Pagination/twitter_bootstrap_v3_pagination.html.twig

So, I required symfony/templating, what do I need more?
Maybe this should be covered at least in docs, to make easier a transition from old applications

IMHO is the bundle who should change the notation to use Twig syntax by default.

I agree with @yceruto. This looks like it should be solved in the third-party bundle:

Sort term solution: they must add symfony/templating as a dependency if they use the bundle notation.
Long term solution: they should never use the bundle notation and transition to namespaced templates.

Right, but I'm missing something: even if I required templating, I'm stuck with the exception

It's working, using @rosier suggestion. Thank you everyone

guess that:
App\Menu:
resource: '../src/Menu'
arguments<EMAIL_ADDRESS>tags:
- { name: knp_menu.menu_builder, method: createFrontMenu, alias: main } # The alias is what is used to retrieve the menu


Signature Builder API

Work in progress!
The new SignatureBuilder API will enable users to create signatures flexibly and reliably.
I decided to introduce SignatureSubpacketGeneratorWrapper in order to ensure sane subpacket limitations (no duplicates of singleton subpackets for example).
Furthermore extracting certain setters into different interfaces enables us to limit use of setters in different scenarios (eg. it does not make sense to add recovation reason subpackets to certification signatures).

Coverage decreased (-0.5%) to 90.563% when pulling 063c63c63c7fe535f17c752c850302dc7e16b66c on signatureBuilder2 into 2b2639bde72ce106d0664e27949e3567d7d845a4 on master.

I'm a little bit frustrated with the idea of a signature builder, since we cannot use it anywhere we want.
For instance, while our implementation already can create subkey binding signatures, BCs KeyRingGenerator cannot make use of this, so I feel like we are driving two separate implementations :/
I guess we have to transform this away from a signature builder to a signature builder plus a sane subpacket builder API.
That way we can use the signature builder whenever we have to hand-craft signatures (e.g. when certifying other users keys), and can still use the sane subpackets builder even with BC classes (with some internal boilerplate code).


Update ibm_cloud_sdk_core and http gems

Pull in https://github.com/IBM/ruby-sdk-core/pull/33 to fix issues with ruby 2.7

cc @jaredm-ibm


Part Requests

LJDAM Bomb
PJDAM missile
Griffin missile
StormBreaker bomb
Gunslinger launcher
SDB Rack

some of the weapons you're asking for can be found in https://github.com/Spartwo/KP-Armory

Small update

SIDAM 25 Mistral
Sharpshooter turret

The SIDAM 25 Mistral seems to be highly requested.

A few final additions

Quad hellfire launcher
Turreted APKWS


Calendar replacement

I changed the calendar to https://fullcalendar.io/ and loaded the existing events. Please try it out. We have so many events that the calendar is really fully, we might want a filter. Please try the timezone thingy.

@jcklie This looks nice!
It seems that the make freeze failed. Can you try to figure out why?
Also I think @raosudha89 has updated main_calendar.json to add the plenary sessions information. Maybe you want to
cd sitedata_acl2020 && git pull

@jcklie  Sorry, I didn't know you were working on this. I created main_calendar.json slighly differently. Since there are too many tutorials and workshops to all be listed on main schedule, I just added them as overall sessions. Below is the screenshot.

@jcklie  and @hao-fang  I think it might better for me to revert the changes I made, let @jcklie  submit his changes to main_calendar.json and I can add the missing plenary sessions info.

@jcklie ok -- I fixed the make freeze error.
see this commit https://github.com/acl-org/acl-2020-virtual-conference/pull/190/commits/c7fb0d7897b6697f4a4f8816d9bce2af9b4d9b22 for the changes


Error in Raspian Jesse

When I "npm run start" on Raspbian Jesse (Debian for Raspberry Pi) I get an error. I checked how I configured it, and couldn't find an issue. It may just not work on Raspbian Jesse. Here's the command line readout:
pi@raspberrypi:~/reddit-placebot $ npm run start

><EMAIL_ADDRESS>start /home/pi/reddit-placebot
> node run.js

/home/pi/reddit-placebot/run.js:24
for (let user in users) { if (!queues[user]) scheduleUser(user) }
         ^^^^
SyntaxError: Unexpected identifier
    at Module._compile (module.js:439:25)
    at Object.Module._extensions..js (module.js:474:10)
    at Module.load (module.js:356:32)
    at Function.Module._load (module.js:312:12)
    at Function.Module.runMain (module.js:497:10)
    at startup (node.js:119:16)
    at node.js:906:3

npm ERR<EMAIL_ADDRESS>start: `node run.js`
npm ERR! Exit status 8
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>start script.
npm ERR! This is most likely a problem with the reddit-placebot package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     node run.js
npm ERR! You can get their info via:
npm ERR!     npm owner ls reddit-placebot
npm ERR! There is likely additional logging output above.
npm ERR! System Linux 4.4.13+
npm ERR! command "/usr/bin/nodejs" "/usr/bin/npm" "run" "start"
npm ERR! cwd /home/pi/reddit-placebot
npm ERR! node -v v0.10.29
npm ERR! npm -v 1.4.21
npm ERR! code ELIFECYCLE
npm WARN This failure might be due to the use of legacy binary "node"
npm WARN For further explanations, please read
/usr/share/doc/nodejs/README.Debian

npm ERR!
npm ERR! Additional logging details can be found in:
npm ERR!     /home/pi/reddit-placebot/npm-debug.log
npm ERR! not ok code 0

I have the same issue on regular old windows 10.

Maybe it's an old version of node that doesn't support the let keyword? Try running it with node src/run.js --harmony. The latest version does this by default.

Also be sure that you installed nodejs manually beforehand

@yottalogical you're running an old version of nodejs. Try to update it:
sudo curl -sL https://deb.nodesource.com/setup_7.x | sudo -E bash -
(Notice the "-" at the end!)
sudo apt-get install -y nodejs
And try to run it again :)

@DDerTyp Thanks for the help, but it looks like I do have the most recent version. I did what you said anyway, but nothing new was installed. The problem didn't go away either.

@Lenni I did make sure NodeJS was up to date, but thanks for suggesting that. I (as well as nearly everyone) has been known to make silly mistakes like that.

I mean, in the log you posted it says you're using v0.10.29, which is not the most recent version, it was released on 2014.
If you are certain you installed the 7.x version then you might have the PATH badly configured.

Reason for Errors: I didn't have the most recent version of NodeJS.
Reason I didn't have the most recent version: My hardware (original Raspberry Pi) was not supported. Whenever I tried to update it, it just installed the most recent supported version (v0.10.29).


Update cAdvisor godep

Minor nits. Overall LGTM.

LGTM

Thanks @vishh I'll send you a PR with those renames, but in Go-approved camelcase :D


app: fix build error from distinguish CONFIG of app

distinguish CONFIG of app to avoid build error if one of CONFIG of app disabled.
CONFIG of app :

CONFIG_PERIPHERAL_MANAGEMENT
CONFIG_PWRMGMT
CONFIG_DNX_SUPPORT
CONFIG_SOC_DEBUG_AWARENESS
CONFIG_SMCHOST
CONFIG_POSTCODE_MANAGEMENT
CONFIG_ESPI_PERIPHERAL_8042_KBC
CONFIG_ESPI_SAF
CONFIG_SAF_SPI_WINBOND

Signed-off-by: Richard Yim<EMAIL_ADDRESS>

There are 2 commits in this PR out of which one is already merged. So, please rebase and resubmit. We should see only one commit once you do the rebase.

I think you only added the commit message at the github PR but not onto the commit itself. You can do this on you local branch "git commit --amend" then push your changes in again.

ONFIG_PERIPHERAL_MANAGEMENT
CONFIG_PWRMGMT
CONFIG_DNX_SUPPORT
CONFIG_SOC_DEBUG_AWARENESS
CONFIG_SMCHOST
CONFIG_POSTCODE_MANAGEMENT
CONFIG_ESPI_PERIPHERAL_8042_KBC
CONFIG_ESPI_SAF
CONFIG_SAF_SPI_WINBOND


Call topi and external library through emit_te and add MLP example

This PR does the following:

Add test cases to call topi and external libraries (cblas) through emit_te. Task0 in https://github.com/tlc-pack/relax/issues/48.
Change block_builder.get() to return the full IRModule, so we can create multiple relax functions using multiple with scopes. Task4 in https://github.com/tlc-pack/relax/issues/44.
Add example code on creating, compiling, and running an MLP model (contrib.cblas.matmul and topi.nn.relu) in relax.

LGTM!

Thanks for demonstrating emit_te's usage btw


Feature request: property that indicates 'back' navigation

I'm working on a project where it would be really useful to know if the user navigated forward (push state) or backwards (pop state).
I had a look at the source code and it looks like the popstate event listener could call routeTo with an additional parameter that indicates the route is caused by the history popping a state.
Something like:
addEventListener('popstate', () => {
  routeTo(getCurrentUrl(), { isBack: true });
});

The parameter could then be stored in state and passed as a prop to the component and the onChangeRoute callback.
I did a little testing and got it working quite easily (see diffs here). I could make it a PR if this is something worth adding to preact-router.
Thanks!

I think this would be a nice addition. The 'back' concept gets a bit murky when the user uses forward history navigation, since that also fires the popstate event. The new App History API provides an explicit indicator for this, but it's only available in Chrome currently.

Do we wanna use the new App History API to implement the new feature?
Happy to work on it!


Revert "Inliner: inline generic thunks which return a partial_apply."

This reverts commit 66474ed5a2602e9059c31faa1e84957054920594.
The original commit triggers a crash in IRGen: rdar://problem/59456064
Reverting for now until the IRGen issue is fixed.
Reverts part of https://github.com/apple/swift/pull/29784

@swift-ci smoke test and merge


Essential updates

Chart data are no longer reversed following a table view (issues #131 and #132)
Chart labels are now responsive (issues #75 and #134)
The archive page is now works on Firefox (issue #124)
Integrates Markdown for rendering the Accessibility page
Incorporates correct hierarchy for the Accessibility page
Adds a table of contents to the Accessibility page
Changes the label colour on charts to a darker one for added contrast
Other minor fixes and improvements

 Bug: line charts reversed after toggle to table

 Reduce text size on axis labels for graphs on MOBILE to 12px to fix overcrowding.

 Feature: Integration of Markdown for accessibility page

 Bug: Chart data is distorted after switching between chart view and toggle view

 Bug: Archive page stuck on loading in Firefox


Adds DateTimeTypeConverter

Small nit with the usings order being changed, but other than that :+1: !


V3 of actions/upload-artifact & actions/download-artefact to be deprecated on 30/11/2024

User Story
As a Modernisation Platform Engineer
I need to update the versions of the GitHub actions/upload-artifact & actions/download-artefact to v4
So that the upcoming deprecation of v3 does not impact our workflows.
Brownouts of this action are scheduled as follows:
November 14, 12pm - 1pm EST
November 21, 9am - 5pm EST
Value / Purpose
It has been announced that v3 of actions/upload-artifact & actions/download-artefact are being deprecated on 30th November 2024. This will impact the following workflows:

scorecards.yml - https://github.com/ministryofjustice/modernisation-platform/blob/ab83f66befb5ce96078d7236baacd678c220b5f5/.github/workflows/scorecards.yml#L62

publish-gh-pages - https://github.com/ministryofjustice/modernisation-platform/blob/ab83f66befb5ce96078d7236baacd678c220b5f5/.github/workflows/publish-gh-pages.yml#L35

These need to be updated to v4 (the latest) and tested.
Context / Background
See above.
This was also posted by Anthony Bishop in the
Useful Contacts
No response
Additional Information

https://github.com/actions/upload-artifact?tab=readme-ov-file#breaking-changes
https://github.com/actions/upload-artifact/blob/main/docs/MIGRATION.md

Definition of Done

[ ] Update both actions and test before the first brownout.
[ ] Search for any other MP workflows that may be impacted & update those.
[ ] Review & close.

Having taken a look at this one I think we might not need to do it - or if we choose to, it would be centered around applying improvements to how/where we use upload-artifact.
You can see the hash value for v4.4.3 of the upload-artifact action is SHA:b4b15b8c7c6ac21ea08fcf65892d2ee8f75cf882.
If you search for this value in our GitHub organisation you can see that it's already being used.
That being said I don't personally) think we're using this for its intended purpose. We generally use this to create a blob that's consumed within the same workflow, whereas the intent of upload-artifact is to store data for use later outside of the workflow - eg, outputting build logs or binaries from a build. The cache action - for passing files between jobs in a workflow - is probably more appropriate for our uses.

see above, not required


Is Bat supposed to work under Windows?

I tried to do some work on Windows machine and ran into some issues when trying to import logfiles*.
If I follow the 'normal' pattern to import a log file, for example: conn_df = LogToDataFrame('conn.log') it crashes out with the following error message:
---------------------------------------------------------------------------
OSError                                   Traceback (most recent call last)
<ipython-input-13-c8896a68360f> in <module>()
----> 1 conn_df = LogToDataFrame('conn.log')

~\AppData\Local\Continuum\anaconda3\lib\site-packages\bat\log_to_dataframe.py in __init__(self, log_filename, ts_index)
     22
     23         # Create a bro reader on a given log file
---> 24         reader = bro_log_reader.BroLogReader(log_filename)
     25
     26         # Create a Pandas dataframe from reader

~\AppData\Local\Continuum\anaconda3\lib\site-packages\bat\bro_log_reader.py in __init__(self, filepath, delimiter, tail, strict)
     57                             'unknown': lambda x: x}
     58         self.dash_mapper = {'bool': False, 'count': 0, 'int': 0, 'port': 0, 'double': 0.0,
---> 59                             'time': datetime.datetime.fromtimestamp(0), 'interval': datetime.timedelta(seconds=0),
     60                             'string': '-', 'unknown:': '-'}
     61

OSError: [Errno 22] Invalid argument

I did verify that the file is in the current dir etc by a quick:
print(os.listdir(os.curdir))
['.ipynb_checkpoints', 'conn.log', 'Connection Time Diagrams.ipynb', 'data']

I've tried this with both Jupyter Notebook and on the Python command prompt with the same result.
Using the same approach/files/directory structure works just fine under Linux. I don't really have access to a Mac so haven't been able to test this on that platform.

Hi @swedishmike it's a bit hard to tell but it looks like the issue is on line 59 of the bro_log_reader.py. I'm guessing it might be either the 'time' or the 'interval' values with use the datetime/timestamp datetime/timedelta calls. Try removing the datetime stuff and just setting those to 0 (might give you an error down the road) but that would tell us that the issue is with one of those datetime calls.

Cheers - I'll give that whirl and poke around a bit and see if I can find something.

This is the error i think - https://github.com/home-assistant/appdaemon/issues/83
The datetime.datetime.fromtimestamp(0) fails in python 3.6. You should instead try datetime.datetime.fromtimestamp(86400), ie. the number of seconds in 1 day to get it to work.
Hope this helps

@rohanbapat Great catch!
Seems that the bug been open for quite some time as well. https://bugs.python.org/issue29097

Closing this with the merge of #49

Just tested and can confirm that the new version works on Windows as well now.
Excellent work!


rsyslog: 8.2102.0 -> 8.2108.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/rsyslog/versions.
meta.description for rsyslog is: "Enhanced syslog implementation"
meta.homepage for rsyslog is: "https://www.rsyslog.com/"
meta.changelog for rsyslog is: "https://raw.githubusercontent.com/rsyslog/rsyslog/v8.2108.0/ChangeLog"
Updates performed

Version update

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

/nix/store/kw01qwgx8yjn25q4a4r59jsi5bb2h1mb-rsyslog-8.2108.0/bin/logctl passed the binary check.

/nix/store/kw01qwgx8yjn25q4a4r59jsi5bb2h1mb-rsyslog-8.2108.0/bin/rscryutil passed the binary check.

/nix/store/kw01qwgx8yjn25q4a4r59jsi5bb2h1mb-rsyslog-8.2108.0/bin/rsyslogd passed the binary check.

3 of 3 passed binary check by having a zero exit code.

3 of 3 passed binary check by having the new version present in output.

found 8.2108.0 with grep in /nix/store/kw01qwgx8yjn25q4a4r59jsi5bb2h1mb-rsyslog-8.2108.0

found 8.2108.0 in filename of file in /nix/store/kw01qwgx8yjn25q4a4r59jsi5bb2h1mb-rsyslog-8.2108.0

directory tree listing: https://gist.github.com/ee27366404b90ad31cb25cbb88269dc5

du listing: https://gist.github.com/cf14c9cbe39dde25969d9f35a9966f2b

Rebuild report (if merged into master) (click to expand)

6 total rebuild path(s)

2 package rebuild(s)

2 x86_64-linux rebuild(s)
2 i686-linux rebuild(s)
0 x86_64-darwin rebuild(s)
2 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
rsyslog
rsyslog-light

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/kw01qwgx8yjn25q4a4r59jsi5bb2h1mb-rsyslog-8.2108.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A rsyslog https://github.com/r-ryantm/nixpkgs/archive/21f1fcdf8bedf239df336f53f51466d3fb23d251.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/kw01qwgx8yjn25q4a4r59jsi5bb2h1mb-rsyslog-8.2108.0
ls -la /nix/store/kw01qwgx8yjn25q4a4r59jsi5bb2h1mb-rsyslog-8.2108.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

2 packages built:

rsyslog
rsyslog-light

Maintainer pings

Result of nixpkgs-review pr 137763 at 21f1fcdf run on aarch64-linux 1

2 packages built successfully:

rsyslog
rsyslog-light


Publish on pypi

This package should be published to pypi so it can be easily installed with pip.

I agree that publishing the package on PyPI would be a good idea. Note, however, that pip can install directly from git repositories:
pip install git+https://github.com/stadt-karlsruhe/geoextract.git

I already use pip with the git url, but it is tedious to use as pip is quite fragile with everything non-standart


Create Message and create Run Assistant take too long time

Confirm this is an issue with the Python library and not an underlying OpenAI API

[ ] This is an issue with the Python library

Describe the bug
From the API I had constructed a number of calls to the Wizards I had created. However, as of today, when I run the following command:
client.beta.threads.messages.create(thread_id=thread.id, role="user", content=user_message)
and
client.beta.threads.runs.create(thread_id=thread.id,assistant_id=assistant_id)
Upon examination of the library during debugging, a significant loop was identified. However, due to my limited understanding of the library, I am unable to discern its nature.
The prolonged time required to create a message and a Run, coupled with the status of 'expired' assigned to the Run upon reaching the execution point of a calling function, has resulted in the unavailability of the implementation, which is already operational in a production environment.
Thanks a lot for the hard work maintaining this code!
To Reproduce
#Create any thread
thread = client.beta.threads.create()
client.beta.threads.messages.create(thread_id=thread.id, role="user", content=user_message) # Here the code takes more than 1 minute
client.beta.threads.runs.create(thread_id=thread.id,assistant_id=assistant_id,) # Here the code take more than 9 minutes

Code snippets
def get_response(client, thread):
    return client.beta.threads.messages.list(thread_id=thread.id, order="asc")

def process(client, assistant_id, question):
    time1 = time.time()
    print("init")
    thread, run = create_thread_and_run(client, assistant_id, question)
    print(f"Thread and run created in {time.time() - time1} seconds")
    run = wait_on_run(client, run, thread)
    if (
        run.status == "requires_action"
        and run.required_action.submit_tool_outputs.tool_calls[0].function.name == "GetTopHeadlinesNews"
    ):
        parameters = json_loads(run.required_action.submit_tool_outputs.tool_calls[0].function.arguments)
        response = GetTopHeadlinesNews(parameters)
        tool_outputs = [
            {
                "tool_call_id": run.required_action.submit_tool_outputs.tool_calls[0].id,
                "output": json_dumps(response),
            }
        ]
        timeStart = time.time()
        run = client.beta.threads.runs.submit_tool_outputs(
            thread_id=thread.id,
            run_id=run.id,
            tool_outputs=tool_outputs,
        )
        print(f"Time taken to submit tool outputs: {time.time()-timeStart}")
        run = wait_on_run(run, thread)
        return get_response(thread)

# Function calling
def GetTopHeadlinesNews(arguments):
    url = "https://newsapi.org/v2/top-headlines"
    parameters = arguments
    response = requests.get(url, params=parameters, headers={"X-Api-Key": api_key_news})
    data = response.json()
    return data

# Waiting in a loop
def wait_on_run(client, run, thread):
    timeStart = time.time()
    print("Waiting for run to complete")
    while run.status == "queued" or run.status == "in_progress":
        run = client.beta.threads.runs.retrieve(
            thread_id=thread.id,
            run_id=run.id,
        )
    print(f"Time taken to wait on run: {time.time()-timeStart}")
    print(f"Run status: {run.status}")
    return run

def create_thread_and_run(client, assistant_id, user_input):
    timeStart = time.time()
    thread = client.beta.threads.create()
    print(f"Time taken to create thread: {time.time()-timeStart}")
    run = submit_message(client, assistant_id, thread, user_input)
    return thread, run

def submit_message(client, assistant_id, thread, user_message):
    timeStart = time.time()
    print(f"Submitting message...")
    client.beta.threads.messages.create(thread_id=thread.id, role="user", content=user_message)
    print(f"Time taken to submit message: {time.time()-timeStart}")
    print(f"Creating run...")
    return client.beta.threads.runs.create(
        thread_id=thread.id,
        assistant_id=assistant_id,
    )

OS
Windows
Python version
Python v3.12.7
Library version
openai v1.52.0

Thanks for reporting!
This sounds like an issue with the underlying OpenAI API and not the SDK, so I'm going to go ahead and close this issue.
Would you mind reposting at community.openai.com?


Invariant Testing

Determine if this is beneficial for us:
https://twitter.com/PaulRBerg/status/1624014054254690304
https://github.com/foundry-rs/book/pull/760

https://twitter.com/gakonst/status/1624146214412521483?s=46&t=gH3q4dow54h4x7JQWCQSeg

We would have to determine if the protocol has any hard and fast rules like Uniswap's x*y=k. I don't think it's useful for the entire protocol but if we can find any "rules" within the protocol that we would expect to persist no matter the conditions then invariant testing would be useful to stress test those.

Perhaps @zgorizzo69 might have some ideas. I haven't personally done the deep dive on what exactly invariant testing is but after I do I would have a better idea myself.

/assign

@pavlovcik is this still open

I think the research is useful but it's @rndquu's decision if this bounty is invalidated.

I think the research is useful but it's @rndquu's decision if this bounty is invalidated.

I would close the following tasks:

https://github.com/ubiquity/ubiquity-dollar/issues/587
https://github.com/ubiquity/ubiquity-dollar/issues/563
https://github.com/ubiquity/ubiquity-dollar/issues/515
https://github.com/ubiquity/ubiquity-dollar/issues/291

All of them will be part of a security epic which we will break down into small tasks
@pavlovcik ok?

Just to clarify, you want to create a meta-issue with these as new subtasks? What's the benefit of remaking all of them vs just manually compiling them in a single new issue?

Just to clarify, you want to create a meta-issue with these as new subtasks?

yes

What's the benefit of remaking all of them vs just manually compiling them in a single new issue?

I don't really understand the question. We can't compile all of them in a single issue because each of those tasks is pretty time consuming.

I meant creating a new blank issue, and then linking all of the existing sub issues in a checklist.
The blank issue is now your "meta-issue"

I meant creating a new blank issue, and then linking all of the existing sub issues in a checklist.

I meant the same flow

I think the research is useful but it's @rndquu's decision if this bounty is invalidated.

I would close the following tasks (including the current issue):

Echidna Properties #587
Invariant Testing #563
Deep Fuzzing on Newly Modified Solidity Files #515
Make echidna working #291

All of them will be part of a security epic which we will break down into small tasks
@pavlovcik ok?

Can you handle this @rndquu ?

I think the research is useful but it's @rndquu's decision if this bounty is invalidated.

I would close the following tasks (including the current issue):

Echidna Properties #587
Invariant Testing #563
Deep Fuzzing on Newly Modified Solidity Files #515
Make echidna working #291

All of them will be part of a security epic which we will break down into small tasks
@pavlovcik ok?

Can you handle this @rndquu ?

I didn't forget about this task
We first need to remove mocks from the contracts because it doesn't make sense in fuzzing Dollar contracts which use mocks under the hood
I'll break down the "remove mocks" epic into subtasks soon

@rndquu this taks has "DevOps" label it's this expected to perform on CI or only local?

@rndquu this taks has "DevOps" label it's this expected to perform on CI or only local?

Invariant testing is a subpart of fuzzing tests which we already run on CI on push to the development branch. So I think for invariant tests we could use the same strategy (running on CI on push to the development branch).

Hey everyone, I've also started the UI testing issue #257
However, since it still needs to pass some checks and isn't ready to be picked up yet, should I unassign myself from it while I work on this one ?

You can be assigned to both.

Hi @0x4007, @rndquu, and team,
I wanted to follow up regarding my pull request that addresses this issue. I wanted to check if there are any updates or further changes needed from my end to move this forward. Any feedback or guidance would be greatly appreciated

Hi @0x4007, @rndquu, and team,
I wanted to follow up regarding my pull request that addresses this issue. I wanted to check if there are any updates or further changes needed from my end to move this forward. Any feedback or guidance would be greatly appreciated

Hey, sorry for a long response, I'll check it today

/start

I have been unassigned from the issue for some reason. Could you please provide any clarification ?

I have been unassigned from the issue for some reason. Could you please provide any clarification ?

As far as I understand the ubiquity-os bot unassigned you because there was some period of inactivity

I have been unassigned from the issue for some reason. Could you please provide any clarification ?

As far as I understand the ubiquity-os bot unassigned you because there was some period of inactivity

I think it doesn't read draft pulls right now. This will be fixed soon.

Hey, please let me know if any further changes are expected from my side to move this forward


[2.0.1] Filebeat upgrade to 7.12.1

Can be merged after v2.0 release

/azp run

I would just merge it. And I will test it more deeply together with OpenSearch PR.


Added log module

This PR adds the log module and replaces all logging instances where it could be applied

I will probably archive Karen Bot after this

sider failed, great
i will do it tommorow


Error: File not found: (...)/AppData/Local/Programs/Microsoft VS Code/Untitled-2

I opened a new tab and started using this extension, but it looks like it doesn't support running on a "non-existing" file?

Im really sorry to hear that. What happens if you first save the file?

It works ok

I will fix this in an upcoming release. 👍


You seem to not be depending on "@angular/core" and/or "rxjs". This is an error

Hi,
I was trying to install new version of this project in new folder structure in my PC and I have
npm --version : 4.4.4 && angular-cli global: "6.0.3" && Git vresion 2.17.0
and to save some volume in my machine, I copied package.json & package-lock.,json to the root folder where the project "Angular-ReactiveForms" downloaded and then run command "npm i" to install the node-modules so I can reference it in all other projects.
now after getting into APM-Updated folder and run command "ng serve" or "ng Build" I  get the error:
"You seem to not be depending on "@angular/core" and/or "rxjs". This is an error"
Appreciate your help on this error,

btw: I ran e2e on the root of APM-Updated project and it compiled successfully and the browser gets launched and Jasmine started and then browser terminated with error on the console
" workspace-project App
× should display welcome message
- Failed: No element found using locator: By(css selector, pm-root h1)"
......
Executed 1 of 1 spec (1 FAILED) in 2 secs.
[11:30:16] I/launcher - 0 instance(s) of WebDriver still running
[11:30:16] I/launcher - chrome #01 failed 1 test(s)
[11:30:16] I/launcher - overall: 1 failed spec(s)
[11:30:16] E/launcher - Process exited with error code 1
An unexpected error occurred: undefined

I'm on holiday in new Zealand and will look at your issue when I return.
But with just a quick look you may not have a new enough version of npm. Could you trust updating it?

Is the code below considered a safe way to upgrade npm in windows?
Set-ExecutionPolicy Unrestricted -Scope CurrentUser -Force
npm install -g npm-windows-upgrade
npm-windows-upgrade

Personally, I uninstall node using the Windows add/remove programs and then reinstall it. I've never tried the technique you specified above.
Also note that both the unit tests and end to end tests are generated when a new Angular project or component/service is created. These ONLY RUN SUCCESSFULLY when the project is new. As soon as we add code to the project, the tests will NO LONGER run successfully (unless we of course update the tests for the updated code).
This course did not include information on Angular testing and therefore did not include code to update the tests with the code we added. That is why the tests don't run successfully.


[Misc] Bump up transformers to v4.39.0 & Remove StarCoder2Config

This PR bump up the transformers version to v4.39.0. Thanks to the version up, we can delete StarCoder2Config.

@ywang96 Thanks for the quick review!

@WoosukKwon I have addressed conflicts that related to the Jais model.

@esmeetu Thanks for fixing the merge error! I actually added him as a co-author in the commit message 😅 . That's why I disabled and re-enabled auto-merge.

Oh it was in the commit message for the scheduled squash merge, so wasn’t visible.

There is a patch release out, see https://github.com/huggingface/transformers/releases/tag/v4.39.1 maybe bump to this already?

There is a patch release out, see https://github.com/huggingface/transformers/releases/tag/v4.39.1 maybe bump to this already?

@bufferoverflow Thank you for bringing this up! Yes we were aware of this patch (In fact 4.39.0 was breaking https://github.com/vllm-project/vllm/pull/3042 so we had to raise an issue on transformers because of it https://github.com/huggingface/transformers/issues/29789)
@WoosukKwon What's your opinion on explicitly requiring transformers >= 4.39.1?

We should do it. Could you submit a PR? Otherwise I will do it tmr.


[REQUEST] Add an error handler to the Mongoose sample injector

Is your feature request related to a problem? Please describe.
I had some problems writing a decent error handler and got errors from it (not the good ones).
Describe the solution you would like
I never figured out how to make it work in a good way so I'm hoping someone can look at it if my PR get's merged.
This can be considered low priority
Additional context
This issue is related to #9

An error handler was added with #32.


Running protractor on async mode skip my tests cases

Hi,
I'm running protractor to test a web app that supports multilple locale (ar/en), i need to get the language key from localStorage by browser, and then get the json file of this language (in node context).
The issue is that protractor didn't see any of my tests cases. here's the code  snippet.
/**
 * This variable must be included in all protractor e2e specs test
 */
var basePath = process.cwd();
var localStorage = require(basePath + '/test/helpers/localStorage');
var locale = require(basePath + '/test/helpers/locale');
/**
 * scenarios
 */
var testScenarios = function () {
    var cityLabel = element(by.linkText(locale.tr.call(locale, 'City')));
    it('test translation on label', function () {
        // check for labels
        cityLabel.getText().then(value, function () {
            expect(value).toEqual("City");
        });
    });
    /**
     *
     */

    it('test update company information', function () {
        update();
        expect(browser.getLocationAbsUrl()).toMatch("/login");
    });
};
/**
 *
 */
describe('Profile page', function () {
    // got to profile
    browser.get(ENV.Protractor.BASE_URL + '#/profile/');
    // get the current language key
    localStorage.getValue("NG_TRANSLATE_LANG_KEY").then(
        /**
         *  Init local dictionary object
         * @param value, current language key
         */
        function (value) {
            //
            locale.init.call(locale, value);
        }
    )
        // I need that all tests will executed after locale initialisation
    .then(testScenarios);
});

This sort of debugging/support question is much better suited for StackOverflow, Google Group discussion list, or Gitter (see our CONTRIBUTING.md).  But I think the issue is that jasmine is having difficulty parsing your spec file because you only reach the it command asynchronously


customization charts: whitegrid, legend

(what do you mean by that ? People can already click on the country they want to see to make it unique)

merlyn --> the title of the legend should be turned prettier

lisa sorry, I haven't read what you write, please could you write again
I fixed the date axis, it is adapted to the range of the values now (vaccination graph)
And I am currently trying to make a white background
and for the legends, you can already personnalize: try double clicking on a country, normally it should only display this country on the graph
yes i know haha :) Legend is the place where is list all the countries. The title of the legen still ugly, haha, is just style that we can do after
okok i can do that :P  at least start

I fixed the legend title: it's now bold and bigger


PSD-1168 Search filters case type

Case type filters, checkboxes for allegation, enquiry and project.
Description

Checkboxes for case types. When no checkboxes are selected it shows all cases, when all checkboxes are selected it shows all cases.
Checklist:

[x] Automated checks are passing locally.
[x] CHANGELOG updated if change is worth telling users about.

Thanks for the comments Tom.
Couple of things I missed:
The filters for assignee and case type were "should", they actually need to be "must". I've changed them all to "must" as they should be AND filters. Because they are all "must", when merged they need to add together instead of overwrite each other.

Thanks @samlionheart ! How does the AND filter requirement work within a given filter – i.e., if I select both "allegation" and "enquiry", that should be an OR query, right?


too much recursion when trying to fit bounds

Hi,
I'm using "Leaflet v1.3.4" and "leaflet.markercluste v1.4.1"
I'm trying to work out why I get this error:
too much recursion[Learn More] leaflet.js:5:1
The code is at its simplest:
    window.VARS.markerCluster = L.markerClusterGroup();

    data.markers.forEach(function(item, i){

            window.VARS.markerCluster.addLayer(
                    L.marker([item.latitude, item.longitude])
                    .bindPopup('A pretty CSS3 popup.<br> Easily customizable.')
                    .on("click", function(marker) {
                        console.dir(marker);
                    })
            );
    });

    // add the clusters to map
    window.VARS.Map_Modal.addLayer(window.VARS.markerCluster);

    // fit to map
   // HERE IS WHERE THE ERROR HAPPENS
    window.VARS.Map_Modal.fitBounds(window.VARS.markerCluster.getBounds());

The weird thing is that it DOES pull out to the correct bounds, but I see this error in the console. Am I doing something wrong?
Thanks
Andy

Yeah you are doing something wrong, just not sure what 😹
IIRC you can cause this if your map doesn't have a size set (0,0 size).
Otherwise make a complete minimal jsfiddle to reproduce.

Thanks @danzel. Mmm it does seem to work ok in a test case!
https://jsfiddle.net/youradds/3ugk7L9y/26/
If I run the code in FF I get the error about "too much recursion". In Chrome I get a more helpful error:
Uncaught RangeError: Maximum call stack size exceeded
    at T.extend (leaflet.js?bust=1539329226068:5)
    at new T (leaflet.js?bust=1539329226068:5)
    at z (leaflet.js?bust=1539329226068:5)
    at T.extend (leaflet.js?bust=1539329226068:5)
    at new T (leaflet.js?bust=1539329226068:5)
    at z (leaflet.js?bust=1539329226068:5)
    at T.extend (leaflet.js?bust=1539329226068:5)
    at new T (leaflet.js?bust=1539329226068:5)
    at z (leaflet.js?bust=1539329226068:5)
    at T.extend (leaflet.js?bust=1539329226068:5)

I'm not sure what that means though? (and its weird the EXACT same data and map works in jsfiddle but not on the site). Shoot me an email on<EMAIL_ADDRESS>if you wanna see it in the "real world" on the server. Its a dev server so is behind an IP block (so I can't just put a link up here)
Thanks
Andy

Sorry, don’t wanna debug your app. If you can reproduce on jsfiddle though
please let us know.
On Fri, 12 Oct 2018 at 8:28 PM, Andy<EMAIL_ADDRESS>wrote:

Thanks @danzel https://github.com/danzel. Mmm it does seem to work ok
in a test case!
https://jsfiddle.net/youradds/3ugk7L9y/26/
If I run the code in FF I get the error about "too much recursion". In
Chrome I get a more helpful error:
Uncaught RangeError: Maximum call stack size exceeded
at T.extend (leaflet.js?bust=1539329226068:5)
at new T (leaflet.js?bust=1539329226068:5)
at z (leaflet.js?bust=1539329226068:5)
at T.extend (leaflet.js?bust=1539329226068:5)
at new T (leaflet.js?bust=1539329226068:5)
at z (leaflet.js?bust=1539329226068:5)
at T.extend (leaflet.js?bust=1539329226068:5)
at new T (leaflet.js?bust=1539329226068:5)
at z (leaflet.js?bust=1539329226068:5)
at T.extend (leaflet.js?bust=1539329226068:5)
I'm not sure what that means though? (and its weird the EXACT same data
and map works in jsfiddle but not on the site). Shoot me an email on
<EMAIL_ADDRESS>if you wanna see it in the "real world" on the
server. Its a dev server so is behind an IP block (so I can't just put a
link up here)
Thanks
Andy
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Leaflet/Leaflet.markercluster/issues/924#issuecomment-429231938,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAX_fjrDayJK2bjajANGwWLLZWIlIIU7ks5ukES1gaJpZM4XXlgz
.

No problem - you don't ask you don't get =) haha. I'll keep playing to see if I can get it to happen in a basic test case . When you said:
IIRC you can cause this if your map doesn't have a size set (0,0 size).
Do you have an example of what you mean?

Interestingly I think I just stumbled on something. I was trying to work out why the leaflet getBounds() was giving wrong values. It was showing the map location when it was init'd .. ie based on:
   window.VARS.Map_Modal = L.map('google_map_modal_inner', {
                  center: [window.VARS.lat,window.VARS.lng],
                  zoom: 10, //
                  maxZoom: 20,
                  zoomControl: false
                });

Setting it to this gets rid of that problem:
   window.VARS.Map_Modal = L.map('google_map_modal_inner', {
                  center: [window.VARS.lat,window.VARS.lng],
                  zoom: 2, // set to a higher zoom level, otherwise it messes up the getBounds later on
                  maxZoom: 20,
                  zoomControl: false
                });


PoC Vector

Codecov Report

:exclamation: No coverage uploaded for pull request base (dev@613dff2). Click here to learn what that means.
Patch has no changes to coverable lines.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@           Coverage Diff           @@
##             dev       #37   +/-   ##
=======================================
  Coverage       ?   100.00%
=======================================
  Files          ?        28
  Lines          ?       796
  Branches       ?       123
=======================================
  Hits           ?       796
  Misses         ?         0
  Partials       ?         0

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

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


lib/types: prioritise coercedType in coercedTo

This more intuitively matches types.either and allows paths to be coerced to submodules again.
Motivation for this change
#76861's changes allowed strings to typecheck as submodules when they appear to be a path (start with a slash). A somewhat common use of coercedTo is to turn single values into an attrset/submodule: with types; coercedTo str (value: { inherit value; }) valueSubmodule. coercedTo currently will prioritise the final type if possible, and now that submodules can be set from paths this causes havoc when the module system tries to import what was previously meant to be just a config value.
A similar issue occurs with types.either, and #76861 suggests changing the order of the types as a solution. That approach isn't possible here and coercedTo feels backward in comparison, so this change reverses its behaviour.
cc @Infinisil who made the original submodule change and suggested this patch
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

Another reason why this change makes sense: Often coercedTo is used to have backwards compatibility with a more restrictive previous type (the left type) which gets converted to a submodule (the right type). The old behavior was that in case a value is of both types, the right one gets chosen. But this would make such a coercedTo change backwards incompatible. Not so with this PR which makes it choose the left one.


IntelliJ file ignored

Hey, thanks for opening the pull request. I think it doesn't hurt to merge this, so I'm +1.


Absolute staking target is incorrect

Here at the bottom right
https://enormous.cloud/dao/api3/tracker/
It reads

Staking Target 50,000,000 tokens
which doesn't match what's on the DAO dashboard.

The stakeTarget at the pool is represented in percentages (where 10^18 is 100%), which refers to a percentage of the total supply of API3. This was done to avoid having to keep adjusting the stake target with inflation/deflation.
Here's a snippet that describes what should happen
HUNDRED_PERCENT = ethers.BigNumber.from(`1${"0".repeat(18)}`);
stakeTargetInTokens = api3Supply.mul(stakeTarget).div(HUNDRED_PERCENT).div(ethers.utils.parseEther("1")).toNumber();

great details, thanks, I will correct this


Fixed docstrings in unit test

Description of your changes
Fixed the error messages in some unit tests (they were mentioning the wrong function due to copy/paste creation process.
Simplified the unit tests for the cluster client (eliminated an unnecessary loop)

Fixes #
I have:

[x] Read and followed Crossplane's contribution process.
[x] Run make reviewable test to ensure this PR is ready for review.

How has this code been tested
make test reviewable (suffices as this only affects unit-test code)

Do we still need to merge #71 before this one ?

We do not. #72 contains everything from #71.

From: Paolo Dettori @.>
Sent: Thursday, March 3, 2022 10:51 AM
To: crossplane-contrib/provider-ibm-cloud @.>
Cc: Harry Stavropoulos @.>; Author @.>
Subject: [EXTERNAL] Re: [crossplane-contrib/provider-ibm-cloud] Fixed docstrings in unit test (PR #72)
Do we still need to merge #71 before this one ? — Reply to this email directly, view it on GitHub, or unsubscribe. Triage notifications on the go with GitHub Mobile for iOS or Android. You are receiving this because you authored the thread. ZjQcmQRYFpfptBannerStart
This Message Is From an External Sender
This message came from outside your organization.
ZjQcmQRYFpfptBannerEnd
Do we still need to merge #71https://github.com/crossplane-contrib/provider-ibm-cloud/pull/71 before this one ?
—
Reply to this email directly, view it on GitHubhttps://github.com/crossplane-contrib/provider-ibm-cloud/pull/72#issuecomment-1058183335, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AGBPIL3JY7EE2VLOPOE272LU6DNXPANCNFSM5PSRYKYQ.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID: @.***>


"terminal.integrated.tabs.description": "${shellCommand}" doesn't include shell-command in description

Testing #234760
repro:

set settings:

  "terminal.integrated.tabs.title": "${cwd}",
  "terminal.integrated.tabs.description": "${shellCommand}"

create new terminal
:bug: see how the terminal tab name is there but not the description:

Version: 1.96.0-insider
Commit: 5d6531aabba3abfbc263c1e17e965db97df9210f
Date: 2024-12-03T05:13:15.471Z
Electron: 32.2.6
ElectronBuildId: 10629634
Chromium: 128.0.6613.186
Node.js: 20.18.1
V8: 12.8.374.38-electron.0
OS: Darwin arm64 24.1.0

Not sure. These're the settings related to terminal I have:
  "terminal.integrated.suggest.enabled": true,
  "terminal.integrated.suggest.quickSuggestions": true,
  "terminal.integrated.defaultProfile.osx": "zsh",
  "terminal.integrated.environmentChangesRelaunch": false,
  "terminal.integrated.gpuAcceleration": "on",
  "terminal.integrated.macOptionIsMeta": true,
  "terminal.integrated.showExitAlert": false,
  "terminal.integrated.tabs.title": "${cwd}",
  "terminal.integrated.tabs.description": "${shellCommand}"

@ulugbekna if you hover the tab do you see shell integration activated?

You could also try manually installing to see if that makes a difference:
https://code.visualstudio.com/docs/terminal/shell-integration#_manual-installation

@Tyriar

if you hover the tab do you see shell integration activated?

Yes:

@ulugbekna shell integration activated is actually a little ambiguous, it'll show up when any shell integration sequences comes in not all of them. This can actually happen with shells sending the non-vscode style shell integration as well.
I suspect this problem is related to your shell integration not fully activating. To investigate further you could enable trace logging and check the Pty Host channel, this should tell us info on the process getting launched as well as whether the SI sequences are going through.


Add an endpoint for notifying us of clientside exceptions.

This goes towards https://github.com/giantswarm/giantswarm/issues/6715.
We want to be notified when an error occurs in Happa (and perhaps gsctl too) and not having to wait until a client tells us about it.

re-ping @marians, fishing for approval from someone. expanding reviewrs to @giantswarm/sig-front-end  and @giantswarm/team-batman


Initialize lunr using a promise rather than a function?

Hi,
Is there a way to pass in a promise to initialize lunr instead of a function?
I want to do a non-blocking stream read to load data for lunr to index, a promise would allow me to keep things async. Any suggestions?
Thanks

Hi

Is there a way to pass in a promise to initialize lunr instead of a function?
BTW this is not an issue IMMO.

just wrap into a Promise the lunr() index creation:
function createLunrIndex(documents,fieldNames) {

    return new Promise( (resolve,reject) => {

      const idx = lunr(function() {
         //
        // load data
        // ...
      })

       resolve(idx)
    });
}

I want to do a non-blocking stream read to load data for lunr to index,

this is maybe another story. Do you want to add dynamically documents after the index creation?
That's seems not possible, see: https://stackoverflow.com/a/44086343/1786393

async function createIndex(): Promise<lunr.Index>{

    const builder = new lunr.Builder();
    builder.pipeline.add(
        lunr.trimmer,
        lunr.stopWordFilter,
        lunr.stemmer
    )
    builder.searchPipeline.add(
        lunr.stemmer
    )

    builder.ref('id');
    builder.field('keywords');

    await addObjectsToIndex();

    return builder.build();
}

async function addObjectsToIndex():Promise<void>{
   // do async stuff here
}


samples: fix s3-to-s3 transfer demo to reflect latest changes (#339)

…r parts (assets, dependencies)

Hey @MikhailGordienk, regarding your S3-to-S3 sample. This functionality has been implemented directly by the InlineDataFlowController and the different DataOperator implementations, so this sample can be removed IMO since it does not really add anymore value.
At the very least it needs to be updated to utilize the  InlineDataFlowController and the latest IDS implementations (Contracts, Negotiations, etc.) and ditch ids-rest in favor of ids-multipart.
Background: we are going to remove support for ids-rest soon and your sample is one of the last areas in the codebase where that is used.
Please let me know no later than Tuesday, January 18th 2022 how you want to procede.

Hello @paullatzelsperger !
Thanks for the feedback. You can remove the sample then.

sample will be removed in PR #523


Comments in Methodes

Write a comment at the beginning of each method with the purpose of the method.

Use this standard:
https://peps.python.org/pep-0257/


MinGHC 7.10.1 failed to bulid ``data-default-instances-old-locale``` version 0.0.1

I tried to build data-default-instances-old-locale version 0.0.1 on Windows. I got the following error message:
~ $ cabal install data-default-instances-old-locale
Resolving dependencies...
Configuring data-default-instances-old-locale-0.0.1...
Building data-default-instances-old-locale-0.0.1...
Failed to install data-default-instances-old-locale-0.0.1
Build log ( C:\Users\gl67\AppData\Roaming\cabal\logs\data-default-instances-old-locale-0.0.1.log ):
Building data-default-instances-old-locale-0.0.1...
Preprocessing library data-default-instances-old-locale-0.0.1...
[1 of 1] Compiling Data.Default.Instances.OldLocale ( Data\Default\Instances\OldLocale.hs, dist\build\Data\Default\Instances\OldLocale.o )
C:\Users\gl67\repos\MinGHC\7.10.1\ghc-7.10.1\mingw\bin\ar.exe: dist\build\libHSdata-default-instances-old-locale-0.0.1-6jcjjaR25tK4x3nJhHHjFM.a-11336\libHSdata-default-instances-old-locale-0.0.1-6jcjjaR25tK4x3nJhHHjFM.a: No such file or directory
cabal.exe: Error: some packages failed to install:
data-default-instances-old-locale-0.0.1 failed during the building phase. The
exception was:
ExitFailure 1

I can install data-default-instances-old-locale with ghc 7.10.1 on Linux (Debian) and no such error happened, it seems that MinGHC has some restrictions in locating the compiled objects.

This is a cabal bug, see: https://github.com/haskell/cabal/issues/2502

Since this Is Cabal's bug, this issue should be closed.

I agree completely. Perhaps we need to put out s message warning windows users to avoid GHC 7.10 until this bug is fixed.

We have to do something - will distributing an old copy of cabal.exe fix it?

Unlikely, as usual the new GHC only works with the newest version of cabal, which always brings in a new set of bugs to be dealt with.

Urk. I guess the newest cabal.exe works (since -j1 works) but the Cabal lib with GHC is broken. Do we need a GHC patch release?

I'll look into this a bit more over the next hour, I'm pretty familiar with Cabal breakages at this point.

In my testing:

-j1 has no effect
Installing newest Cabal and cabal-install has no effect
Unpacking and building from that directory with or without -j1 seems to work
Using runghc Setup.lhs works, which is unsurprising considering the previous bullet

It seems like we can tell users to try one of two workarounds:

If they run into the bug, run cabal unpack pkg-ver && cabal install .\pkg-ver
Just drop down to GHC 7.8.4 until cabal fixes this

I think a blog post with both of those options makes sense, what do you think Neil?

Agreed. I also think Cabal needs better tests.

OK, blog post published: https://www.fpcomplete.com/blog/2015/05/psa-ghc-710-cabal-windows

Aha, I abandoned 7.10 a while back because I couldn't get anything to build. Now I know why!

I wonder if this is related: https://github.com/kazu-yamamoto/ghc-mod/wiki/InconsistentCabalVersions

This is related to haskell/cabal#2502 , the fix being part of the cabal version released with the repackaged MinGHC 7.10.1 as discussed in #69 . I think this issue can be closed ?

Agreed, good call.


Null check operator exception in showToastWidget

I'm receiving Unhandled Exception: Null check operator used on a null value here:
https://github.com/bdlukaa/fl_toast/blob/master/lib/src/toast.dart#L60
I'm on Flutter v2.0.4, sdk ">=2.12.0-0 <3.0.0" and fl_toast 3.0.1

You need to make sure there's a Overlay widget above the context. It's provided by the MaterialApp.

Can you show your code?

You need to make sure there's a Overlay widget above the context. It's provided by the MaterialApp.

Ah I didn't know that.


Update Sandbox docs

sandbox docs at https://github.com/near/sandbox
runner overview docs at https://github.com/near/runner
runner-js docs at https://github.com/near/runner-js
docs.near.org pending PR with questions at https://github.com/near/docs/pull/805


Changelog for release

LGTM

Coverage remained the same at 99.805% when pulling 0a48d2a7dcff1d72a65d070b73a72f456d8fb85c on cr-changelog into b8f259d565cc4e0c215b41ad8d5636fb2d353a92 on master.


Add integration testing for UInt32 Object Type (#143)

this is integration tests for UInt32 visitor type.

create Post application/json UInt32 http trigger
create tests for http trigger
create model for http trigger

Would you please merge this PR with #259? I'll close this PR now.


refactor(core): Convert remaining Groovy to Java

Aside from https://github.com/spinnaker/front50/pull/836, this PR represents the remaining Groovy left in the main source set for front50-core. This does not have a dependency on #836 or vice-versa.
While the PR is large, this is just a direct migration from Groovy to Java. The large delta is due to porting heavy use of Groovy-only code to Java. One new class exists, UntypedUtils, which was written to help with a bunch of the logic around Application and friends that depended heavily on metaclassing. This PR doesn't fix the anti-pattern, it only makes it possible in vanilla Java via reflection.
I don't have a good solution as yet of how we could wrangle Application into something a little more sane. I think namespacing like configs into their own buckets would be an improvement, but I have no idea what a migration to that would look like without some dramatic ripple effects.

@ezimanyi

A bunch of the new classes have getters/setters (replacing the Groovy auto-generated ones) that could probably be Lombok to remove some boilerplate.

I don't trust Lombok when it comes to serialization. I realize lombok will make code more terse, but I never trust that a new Lombok annotation won't ruin Jackson serialization. I feel like Lombok'ing existing classes can come as a separate change. wdyt?

@robzienert : I'm fine with no Lombok; I've also had issues where Lombok was more trouble than it was worth (and also think that at times it makes it too easy to make everything public).  I think we've talked about even moving from Lombok to something else but decided there were enough other things (like removing Groovy) we should do first.  So yeah, totally fine with this as is, and even if we don't later lombok-it.


modules/terraform:  Rename existing InitAndPlan* functions to InitAndPlanWithExitCode*, add InitAndPlan* and Plan* functions with consistent signatures

Addresses issue #319

[Breaking Change] InitAndPlan, InitAndPlanE now return stdout/stderr from terraform plan command , not its return value
[Breaking Change] Old InitAndPlan, InitAndPlanE functions which returned OS exitcode from terraform plan command are renamed to InitAndPlanWithExitCode, InitAndPlanWithExitCode
Added Plan, PlanE functions
3 above changes are required for *Plan* functions to have consistent API with the rest of Terraform-related functions
Removed dependency on aws in TestPlanWithExitCodeWithChanges (test execution is sped up)

go test -v -run "^TestInitAndPlan"

passed for terraform versions 0.11.14, 0.12.0

force-pushed a commit with added comments

after I added comments, build job failed...

Hi @yorinasub17,
Looks fine or should I fix something?

Nope it was from a known intermittent test failure (packer on GCP has been fairly unstable) and I was rebuilding until it passed. Going to merge and release this now. Sorry for the delay: was caught up handling other things.

https://github.com/gruntwork-io/terratest/releases/tag/v0.17.0

Thank you @yorinasub17 !


mediainfo-gui: 18.08 -> 18.08.1

Semi-automatic update generated by https://github.com/ryantm/nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/mediainfo-gui/versions.
meta.description for mediainfo-gui is: '"Supplies technical and tag information about a video or audio file (GUI version)"'.

Checks done (click to expand)

built on NixOS
Warning: no invocation of /nix/store/hqgnal08lv431rvnxy2m12v8cwz20xlm-mediainfo-gui-18.08.1/bin/mediainfo-gui had a zero exit code or showed the expected version
0 of 1 passed binary check by having a zero exit code.
0 of 1 passed binary check by having the new version present in output.
directory tree listing: https://gist.github.com/5b7707d23ed049499ade0469fbd8b458
du listing: https://gist.github.com/95c184dd118ca1b2e3bcd82d02b8052f

Instructions to test this update (click to expand)

One-time optional setup to skip building using Cachix:

Install cachix from https://cachix.org/
Use r-ryantm's cache:cachix use r-ryantm

Test this update by entering a nix shell, seeing what is inside the
result, and if applicable, running some binaries:
nix-shell --pure -I nixpkgs=https://github.com/r-ryantm/nixpkgs/archive/1f415a13ea8a1cb41de06ea179792c4c70bbe652.tar.gz -p mediainfo-gui
ls -la /nix/store/hqgnal08lv431rvnxy2m12v8cwz20xlm-mediainfo-gui-18.08.1

cc @devhell for testing.

No attempt on x86_64-darwin (full log)
The following builds were skipped because they don't evaluate on x86_64-darwin: mediainfo-gui
Partial log (click to expand)

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowUnsupportedSystem = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowUnsupportedSystem = true; }
to ~/.config/nixpkgs/config.nix.

Success on x86_64-linux (full log)
Attempted: mediainfo-gui
Partial log (click to expand)
make[2]: Leaving directory '/build/MediaInfo/Project/GNU/GUI'
make[1]: Leaving directory '/build/MediaInfo/Project/GNU/GUI'
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/hqgnal08lv431rvnxy2m12v8cwz20xlm-mediainfo-gui-18.08.1
shrinking /nix/store/hqgnal08lv431rvnxy2m12v8cwz20xlm-mediainfo-gui-18.08.1/bin/mediainfo-gui
strip is /nix/store/h0lbngpv6ln56hjj59i6l77vxq25flbz-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/hqgnal08lv431rvnxy2m12v8cwz20xlm-mediainfo-gui-18.08.1/bin
patching script interpreter paths in /nix/store/hqgnal08lv431rvnxy2m12v8cwz20xlm-mediainfo-gui-18.08.1
checking for references to /build in /nix/store/hqgnal08lv431rvnxy2m12v8cwz20xlm-mediainfo-gui-18.08.1...
/nix/store/hqgnal08lv431rvnxy2m12v8cwz20xlm-mediainfo-gui-18.08.1

Success on aarch64-linux (full log)
Attempted: mediainfo-gui
Partial log (click to expand)
make[2]: Leaving directory '/build/MediaInfo/Project/GNU/GUI'
make[1]: Leaving directory '/build/MediaInfo/Project/GNU/GUI'
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/a4qfj8l1x8sy1492imjlv9r41m29w2nx-mediainfo-gui-18.08.1
shrinking /nix/store/a4qfj8l1x8sy1492imjlv9r41m29w2nx-mediainfo-gui-18.08.1/bin/mediainfo-gui
strip is /nix/store/y4ymnvgxygpq05h03kyzbj572zmh6zla-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/a4qfj8l1x8sy1492imjlv9r41m29w2nx-mediainfo-gui-18.08.1/bin
patching script interpreter paths in /nix/store/a4qfj8l1x8sy1492imjlv9r41m29w2nx-mediainfo-gui-18.08.1
checking for references to /build in /nix/store/a4qfj8l1x8sy1492imjlv9r41m29w2nx-mediainfo-gui-18.08.1...
/nix/store/a4qfj8l1x8sy1492imjlv9r41m29w2nx-mediainfo-gui-18.08.1


Let the definition argument accept files outside of the builtin definition path

Yes, it would be great! Imho its bad practice to rewriting builtin files just for adding some options. We must be able to pass custom definition file as argument.

Implemented in 3d552309ebc560e60e4b6c99acf69b6978e4ed1e


Update README.md

Adding wmWebStack

Just to inform you, this repo is dead, but I'm working on a remake here https://github.com/0PandaDEV/awesome-windows


Respect default namespace from kubectl config while installing

Right now it is hardcoded as "default". We have an optional default namespace coming from context information. We should use that (if present) as namespace when installing new chart.
In case there is forced namespace set, use it instead.

I see that this logic already exists
https://github.com/komodorio/helm-dashboard/blob/main/pkg/dashboard/static/actions.js#L132
const ns = $("#upgradeModal .rel-ns").val();
setHashParam("namespace", ns ? ns : "default")

Also, while testing locally I observed that the forced namespace feature is already working

Could you please add more information?

The issue is actually about different thing.
When you have a namespace configured for a context in your kubectl config file, this setting should be respected at least in the "Install new chart" dialog.