Default keybindings work within tmux panes but not nvim panes

Hi!
I recently installed the plugin, however looks like there is no way I can make <C-hjlk> work.
I'm using the default configuration as mentioned in the readme, navigation within tmux panes works good but never across neovim splits.
Neovim config
  use({
      "aserowy/tmux.nvim",
      config = function() return require("tmux").setup({
          sync_clipboard = false,
          sync_unnamed = false,
      }) end
  })

tmux config
is_vim="ps -o state= -o comm= -t '#{pane_tty}' | grep -iqE '^[^TXZ ]+ +(\\S+\\/)?g?(view|n?vim?x?)(diff)?$'"

bind-key -n 'C-h' if-shell "$is_vim" 'send-keys C-h' 'select-pane -L'
bind-key -n 'C-j' if-shell "$is_vim" 'send-keys C-j' 'select-pane -D'
bind-key -n 'C-k' if-shell "$is_vim" 'send-keys C-k' 'select-pane -U'
bind-key -n 'C-l' if-shell "$is_vim" 'send-keys C-l' 'select-pane -R'

bind-key -T copy-mode-vi 'C-h' select-pane -L
bind-key -T copy-mode-vi 'C-j' select-pane -D
bind-key -T copy-mode-vi 'C-k' select-pane -U
bind-key -T copy-mode-vi 'C-l' select-pane -R

Curiously enough executing :lua require('tmux').move_right() (or any of the other variants) works well. I've tried to set up the keymaps manually with no success. I also tried binding the actions to a different set of bindings:
vim.keymap.set({'n', 't'}, '<leader>wj', function () tmux.move_bottom() end)
vim.keymap.set({'n', 't'}, '<leader>wk', function () tmux.move_top() end)
vim.keymap.set({'n', 't'}, '<leader>wl', function() tmux.move_right() end)
vim.keymap.set({'n', 't'}, '<leader>wh', function() tmux.move_left() end)

This works as expected, and is what I'm using now as a fallback while trying to solve this since ideally I prefer tmux bindings and nvim bindings to be the same
I tried to deactivate other plugins just in case they might be overwriting the specific default bindings but I had no luck with that either.
Any Idea on how to troubleshoot this one?
Thanks!

Heho,
can you please try to set
    -- enables default keybindings (C-hjkl) for normal mode
    enable_default_keybindings = true,

explicitly? Maybe we have a bug in the settings resolvement.
Kind regards
Alexander

Are you using Lazy.nvim by any chance?
I had the same problem as here (https://github.com/LazyVim/LazyVim/discussions/277), and had to override the default settings, by adding to keymaps.lua :
-- Keymaps are automatically loaded on the VeryLazy event
-- Default keymaps that are always set: https://github.com/LazyVim/LazyVim/blob/main/lua/lazyvim/config/keymaps.lua
-- Add any additional keymaps here
-- Move to window using the <ctrl> hjkl keys

local Util = require("lazyvim.util")
local function map(mode, lhs, rhs, opts)
  local keys = require("lazy.core.handler").handlers.keys
  ---@cast keys LazyKeysHandler
  -- do not create the keymap if a lazy keys handler exists
  if not keys.active[keys.parse({ lhs, mode = mode }).id] then
    opts = opts or {}
    opts.silent = opts.silent ~= false
    vim.keymap.set(mode, lhs, rhs, opts)
  end
end

map("n", "<C-h>", "<cmd>lua require'tmux'.move_left()<cr>", { desc = "Go to left window" })
map("n", "<C-j>", "<cmd>lua require'tmux'.move_bottom()<cr>", { desc = "Go to lower window" })
map("n", "<C-k>", "<cmd>lua require'tmux'.move_top()<cr>", { desc = "Go to upper window" })
map("n", "<C-l>", "<cmd>lua require'tmux'.move_right()<cr>", { desc = "Go to right window" })

Hopefully that solves your problem!

https://github.com/aserowy/tmux.nvim/issues/92#issuecomment-1452428973
Works for me, TY

Are you using Lazy.nvim by any chance?
I had the same problem as here (LazyVim/LazyVim#277), and had to override the default settings, by adding to keymaps.lua :
-- Keymaps are automatically loaded on the VeryLazy event
-- Default keymaps that are always set: https://github.com/LazyVim/LazyVim/blob/main/lua/lazyvim/config/keymaps.lua
-- Add any additional keymaps here
-- Move to window using the <ctrl> hjkl keys

local Util = require("lazyvim.util")
local function map(mode, lhs, rhs, opts)
  local keys = require("lazy.core.handler").handlers.keys
  ---@cast keys LazyKeysHandler
  -- do not create the keymap if a lazy keys handler exists
  if not keys.active[keys.parse({ lhs, mode = mode }).id] then
    opts = opts or {}
    opts.silent = opts.silent ~= false
    vim.keymap.set(mode, lhs, rhs, opts)
  end
end

map("n", "<C-h>", "<cmd>lua require'tmux'.move_left()<cr>", { desc = "Go to left window" })
map("n", "<C-j>", "<cmd>lua require'tmux'.move_bottom()<cr>", { desc = "Go to lower window" })
map("n", "<C-k>", "<cmd>lua require'tmux'.move_top()<cr>", { desc = "Go to upper window" })
map("n", "<C-l>", "<cmd>lua require'tmux'.move_right()<cr>", { desc = "Go to right window" })

Hopefully that solves your problem!

@LeoAdL I still cannot move through keybindings but am able to call require'tmux'.move_left(), do you need to run require('tmux').setup() somewhere in your code? much appreciated

can we do anything on this side to enable this by default?
@LeoAdL

Had the same problem and was driving me insane! Thank you @LeoAdL!!!

Are you using Lazy.nvim by any chance?
I had the same problem as here (LazyVim/LazyVim#277), and had to override the default settings, by adding to keymaps.lua :
-- Keymaps are automatically loaded on the VeryLazy event
-- Default keymaps that are always set: https://github.com/LazyVim/LazyVim/blob/main/lua/lazyvim/config/keymaps.lua
-- Add any additional keymaps here
-- Move to window using the <ctrl> hjkl keys

local Util = require("lazyvim.util")
local function map(mode, lhs, rhs, opts)
  local keys = require("lazy.core.handler").handlers.keys
  ---@cast keys LazyKeysHandler
  -- do not create the keymap if a lazy keys handler exists
  if not keys.active[keys.parse({ lhs, mode = mode }).id] then
    opts = opts or {}
    opts.silent = opts.silent ~= false
    vim.keymap.set(mode, lhs, rhs, opts)
  end
end

map("n", "<C-h>", "<cmd>lua require'tmux'.move_left()<cr>", { desc = "Go to left window" })
map("n", "<C-j>", "<cmd>lua require'tmux'.move_bottom()<cr>", { desc = "Go to lower window" })
map("n", "<C-k>", "<cmd>lua require'tmux'.move_top()<cr>", { desc = "Go to upper window" })
map("n", "<C-l>", "<cmd>lua require'tmux'.move_right()<cr>", { desc = "Go to right window" })

Hopefully that solves your problem!

I guess this is the solution. Im pinning and closing the issue.
tyvm for all the investigation!


Generate qartod aggregate flag based on config

https://github.com/ioos/ioos_qc/pull/23 added a gen_agg argument to QcConfig.run
qc = QcConfig({'qartod': {
    'gross_range_test': {
        'fail_span': [0, 12],
    },
    'spike_test': {
        'suspect_threshold': 3,
        'fail_threshold': 10,
    }
}})
inp = [-1, 0, 1, 2, 10, 3]
results = qc.run(
    inp=inp,
    gen_agg=True
)

While this works, it is not consistent with the general approach of defining all test configurations in the JSON/YAML object.
This update moves the aggregate to the config object itself:
qc = QcConfig({'qartod': {
    'gross_range_test': {
        'fail_span': [0, 12],
    },
    'spike_test': {
        'suspect_threshold': 3,
        'fail_threshold': 10,
    },
    'aggregate': {}
}})
inp = [-1, 0, 1, 2, 10, 3]
r = qc.run(
    inp=inp
)

See https://github.com/ioos/ioos_qc/issues/15

Controlling the computation of the aggregate from a config is the way to go :+1:.


[IOTS-343] Issue when accessing device statistics dashboard as a tenant

Not able to view the device statistics dashboard as a tenant by trying to access it through the device mgt UI. Where device statistics directs to https://localhost:9443/portal/t/wso2.com/dashboards/device-statistics-dashboard/ from the UI which works for carbon.super but not for the tenant. For tenant the url should be https://localhost:9443/portal/t/wso2.com/dashboards/device-statistics-dashboard/?shared=true And by default the device statistics dashboard should be shared with all tenants.Reference: https://wso2.org/jira/browse/IOTS-343

This is fixed hence closing


Update culler wait strategy

Signed-off-by: Luca Giorgi<EMAIL_ADDRESS>Changes to fix failures seen in Sanity runs for 4.10

Test run: https://opendatascience-jenkins-csb-rhods.apps.ocp-c1.prod.psi.redhat.com/job/rhods-ci-pr-test/802/

All pass with the exception of Verify Culler Kills Inactive Server which is intentionally failed since the drift was higher than 2 minutes


useMutation - throwOnError triggering on each render

Describe the bug
When i use a mutation in my react component, each render is triggering throwOnError.
Your minimal, reproducible example
https://stackblitz.com/edit/vitejs-vite-it3a6h?file=src%2FApp.tsx
Steps to reproduce

click on increase count - you won't see alert.
click on trigger mutation - you will see alert.
click on increase count again, you will see alert again.

Expected behavior
As a user, i want to listen throwOnError. But it triggers many times after each render.
How often does this bug happen?
Every time
Screenshots or Videos
No response
Platform
OS: Windows
Browser: Chrome
Tanstack Query adapter
react-query
TanStack Query version
v5.25.0
TypeScript version
No response
Additional context
No response

I've tried to write a custom error handler. It tries to handle the error.
If it can handle the error, it displays a modal with detailed information about the error and returns false on throwOnError, otherwise it returns true on throwOnError. However, this structure shows the error modal on every render.
How should I handle this case? It seems like throwOnError is not the right choice for my case.

throwOnError is not a function you can use to perform a side effect like rendering a modal. You should render a modal depending on the error field returned from useMutation or the onError callback.


Get SessionItems with GetSessionStatusAsync()

Describe the feature request
Hey guys!
I am currently in the process of moving from SignalR to MQTTNet because i had some issues with Mono. In SignalR i was using a custom ConnectionManager class to store additional information about the sessions like logged in users (since logging in happens after MQTT has initialized connection). My use case is:

When a user tries to login i have to search the open connections and see if that user is logged in somewhere else.
My initial though was that i can save the ConnectedUser inside the SessionItems dictionary and then iterate through SessionItems when i get the list of clients but it seems that GetSessionStatusAsync() does not return the SessionItems for each connection.
Is there any other way to get the SessionItems and then iterate through them to see if the user is logged in somewhere else or should i use a custom dictionary to store such data? Otherwise, what is the use of SessionItems? Unfortunately I could not find any useful info on that when googling.

Which project is your feature request related to?

Server

Describe the solution you'd like
Be able to get SessionItems for each connection when using GetSessionStatusAsync()
Describe alternatives you've considered
Using an external Dictionary<string, string> _connectedUsers that stores the ClientId as the key and the username as a value and then iterate through that list to figure out if the user is connected. The problem with this approach is that it does not scale. If we need to use a cluster to manage the server then i would also need to somehow share the _connectedUsers dictionary among the servers.
Thank you!

It seems that SessionItems is just named Items when you retrieve the IMqttSessionStatus. I'm not sure if that is a design decision but it is kind of confusing so maybe you should consider renaming it (i will make a pull request on this and you can either accept it or reject it) . In any case, my issue is fixed!


[DECO-2485] Handle Azure authentication when WorkspaceResourceID is provided

Changes
Handle Azure authentication when WorkspaceResourceID is provided
Get token for the correct subscription
Tests

Created Unit tests
Manually listed workspace cluster in the following scenarios:

User with wrong default tenant. No WorkspaceResourceID provided: Fail (expected). WARN log emitted.
User with wrong default tenant. WorkspaceResourceID provided: Succeed
User with no subscription. No WorkspaceResourceID provided: Succeed. WARN log  emitted.
User with no subscription. WorkspaceResourceID provided: Succeed (fallback mode, expected).

[X] make test passing
[X] make fmt applied
[x] relevant integration tests applied
https://github.com/databricks/eng-dev-ecosystem/actions/runs/6038981442

Codecov Report
Patch coverage is 85.18% of modified lines.

Files Changed
Coverage

databricks/sdk/core.py
85.18%

:loudspeaker: Thoughts on this report? Let us know!.


Changing parameter order for converted mlmodel.prediction

Issue: https://github.com/tf-coreml/tf-coreml/issues/213

Replace input_feed_dict dictionary type as OrderedDict to fix order of parameter of function mlmodel.prediction()

When one network is taking multiple input parameters with various input tensor size
(like image style transfer applied to different input size). order of parameters of function mlmodel.prediction() is changed by input_feed_dict which has non-static key order cause of
different tensor shape, tensor name, and other tensor properties. Thus to fix varying order of
parameter, input_feed_dict should be made with static key order in spite of different tensor properties

Can you write a simple test that fails if this change is not made?

Here is example could reproduce issue. I wrote two cases with different input size (360, 480) and (768, 1024). Please consider the result of model.input_description. The result of two cases are not same. Thus when I'm trying to  import .mlmodel to XCode, I have to modify parameter order of model.prediction().
I'm sorry being not able to attach .ckpt files cause of confidence. Actually those are not needed and you can import any dummy network because sample_func in the example does not use any of weight and can work independently.
example.zip

@minhohihi I looked into your example code and inspected the result of model.input_description. Yes, for different calls to the converter, the order of the input features in the description is permuted and not the same.
However, I do not see any reason to maintain a fixed order for the input features/parameters. If you have a special use case in my mind, kindly let us know.
Also, please realize that since we are talking about the inputs, you already have a direct access to them and thus, need not to be inferred from the mlmodel.


crash when creating or joining singleplayer world

Version
1.1.11-deobf
Crash Report
https://pastebin.com/HyVZNiST
Log
No response
Additional Context
Yes
Modifications
https://www.curseforge.com/minecraft/modpacks/create-au-gratin
this still uses old ponderjs, then i updated ponderjs (why does the curseforge jar has -deobf in the name?) and i can't create or join a world anymore
i haven't uploaded ponderjs to the multiplayer server yet so can't confirm if this is a server thing
Did the crash happen in singleplayer or on a server?
Singleplayer

oooooops. Our CI might uploaded the wrong file 😅

after installing the new version, it automatically installed Create Fabric in the curseforge launcher

Fixed it. Thanks for the report. Lytho added the wrong dependency.


import broken

Describe the bug
Clicking on import does nothing.
To Reproduce
Open website, click on import.
Expected behavior
A drop down with options to import.
Sim Links and Screenshots
If applicable, add links to your sim settings or screenshots to help explain your problem.
Desktop (please complete the following information):

OS: Windows 10
Browser : firefox and brave

Not sure if it's the same issue, but in my case the import feature also stopped working recently.
Turns out my pi-hole adlist was blocking the domains "cdn.jsdelivr.net" and "ajax.googleapis.com".
After whitelisting those domains, import started working again.

Not sure if it's the same issue, but in my case the import feature also stopped working recently.
Turns out my pi-hole adlist was blocking the domains "cdn.jsdelivr.net" and "ajax.googleapis.com". After whitelisting those domains, import started working again.

Same issue for me.


{containerapp} Rename duplicate Test class name

Related command

Description

Testing Guide

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change
[Component Name 2] az command b: Add some customer-facing feature

This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] The PR title and description has followed the guideline in Submitting Pull Requests.

[ ] I adhere to the Command Guidelines.

[ ] I adhere to the Error Handling Guidelines.

containerapp

Wait for PR: https://github.com/Azure/azure-cli/pull/27837

Hi @wangzelin007
Could you please help to merge this PR when you have chance? Thanks.

Hi @wangzelin007
Could you please help to merge this PR when you have chance? Thanks.

The real cause of the error is explained in this PR, but I think this PR is beneficial and harmless, so it is up to you to decide whether to merge it.


Got an error using the webapp to connect the card.

Got an error using the webapp to connect the card.
Please find error below:
Unhandled Rejection (Error): error function: executeAPDU, message: executeAPDU error: NotSupportedError: GATT operation failed for unknown reason.

It was due to MacOS issues and has been fixed since MacOS 12.3.0


Update docker-compose.yml

Added icon to container

Summary by CodeRabbit

New Features

Updated application metadata to include a visual icon for the BigBearCasaOS User Management service.

Thank you for the contribution. Do you own this icon?


samples/subsys/mcumgr/smp_svr: Define Image Version as Default

Define CONFIG_MCUBOOT_IMAGE_VERSION as default in the example, as this is crucial for versioning using mcumgr/smp_svr

Commit missing sign-off.

Commit missing sign-off.
Amended, sorry.

Can you squash that into one commit and force push to the branch?

I was not able to squash due to merging into the branch. Will start from scratch on a new branch.

@joaodullius Do not close this, just push your new branch here.

@joaodullius ~Do not close this, just push your new branch here.~
Sorry I have not noticed that you are working from main branch.

Yep... This is my first contribution attempt, and it got a little messy. I Closed it and opened again as #37208


Add xlswriter extension to php image

ISSUE TYPE

Feature request

SUMMARY
Would like to have https://xlswriter-docs.viest.me/en/an-zhuang/pecl this extension installed in php image it can be manually enabled.
Goal
Better extension support

@cytopia if it's okay for you I can add this feature too ?

@taliptako currently baking here: https://github.com/devilbox/docker-php-fpm/pull/172

https://github.com/cytopia/devilbox/pull/725 merged


Date ranges don't work in body of last headline

Describe the bug
Probably related to #303
Steps to reproduce

Following org file, cursor is at pipe:

* a
  <2022-05-27 Fri 14:00>--|

Try to enter the second date with <leader>oi.
Observe error: [orgmode] Failed to parse current headline. Make sure there are no errors in the document.

Expected behavior
date added
Emacs functionality
date added
Minimal init.lua
used default minimal_init.lua
Screenshots and recordings
No response
OS / Distro
ubuntu 20.04, alacritty
Neovim version/commit
master
Additional context
By the way, is it on purpose that when I do <leader>oi. to add a date (anywhere, not just in this specific context), the date gets put before the cursor and not after it? It doesn't make so much sense to me that it gets put before the cursor. I can open a separate issue if you want.

I can reproduce this. Furthermore, if I add any line below the date range, even in the same heading, its starts working:

I agree with you @Maltimore that the cursor is confusing. It behaves like pasting with p, when most of the time I would expect it to work like P. As you can see in the gif, I have to add the space between the first and the second date. This space then gets pushed to the end of the line, when actually I never wanted it to begin with

I asked about this parsing error here: https://github.com/milisims/tree-sitter-org/issues/39 but apparently from the tree-sitter side this is the expected behavior.
By the way @jgollenz I think (I'm not sure because it's difficult to see what exactly comes after the date in your gif) that what you experienced is not current anymore, i.e. adding another line below doesn't fix the behavior now.

Thanks for looking into this. I'm not really sure if it's about timestamps though, I think the culprit is the last line. I've noticed strange behavior on the last line of documents before.
Indeed, things seem to have changed. But now I don't get an error message when I try to add a second date, but it instead it changes the first date. Also: I noticed a crucial detail between what I'm doing in the gif and what you wrote in the issue. I have
* foo
<2023-02-11> |

but you tried
* foo
<2023-02-11>--|

Don't add the dashes, just one space after the timestamp. Still, the last line IS wonky. I'll look into it

There's an undocumented behavior of nvim-orgmode that is supposed to trigger in exactly your scenario (space after timestamp and cursor after that) where it automatically inserts a date range. I suggested in another issue that I think this (anyways undocumented) behavior is not necessary as it only adds two dashes.
I'm on mobile right now, will update this comment later with more specific info.

I also tried looking deeper into it and I saw that when things go wrong, the function File:get_node_at_cursor(cursor) in the file lua/orgmode/parser/file.lua finds a wrong node, namely it goes to the document node (is that the root node?).
Somewhat related to this, I also have a question regarding the code in this function, can't figure it out for the life of me: https://github.com/nvim-orgmode/orgmode/discussions/504

undocumented behavior

It's documented here, although hard to catch: When this command is used twice in succession, a time range is inserted. But in emacs orgmode, inserting a timestamp results in:
* foo
<2023-02-02> |

It adds a space after the timestamp. The cursor is on that timestamp and you easily add another one to create a range. We don't do that atm. After inserting a timestamp, the cursor is on the closing angled bracket. Do create a range you have to

enter insert mode after the timestamp (a)
enter a space ( )
exit to normal mode (<ESC>)
and finally execute the shortcut again (<leader>oi.)

Also, doing that will leave a trailing whitespace. But changing the current behavior to match the one in emacs will leave trailing whitespace after every inserted timestamp :/

This issue should be resolved for some time now.

There's an undocumented behavior of nvim-orgmode that is supposed to trigger in exactly your scenario (space after timestamp and cursor after that) where it automatically inserts a date range. I suggested in another issue that I think this (anyways undocumented) behavior is not necessary as it only adds two dashes. I'm on mobile right now, will update this comment later with more specific info.

For this, I did some changes how we append dates and explained it here https://github.com/nvim-orgmode/orgmode/pull/489#issuecomment-1974904753

Works perfectly, thank you!!


Add shortcut key for switching palette color

In Paint.net one can switch between primary and secondary color with the shortcut key X. When using Pinta, I'm really missing that shortcut, so I added it in!

Looks good, thanks!


Bug report on BayesSearchCV

Steps to reproduce:
from skopt import BayesSearchCV
from sklearn.datasets import load_digits
from sklearn.svm import SVC
from sklearn.model_selection import train_test_split

X, y = load_digits(10, True)
X_train, X_test, y_train, y_test = train_test_split(X, y, train_size=0.75, test_size=.25, random_state=0)

# log-uniform: understand as search over p = exp(x) by varying x
opt = BayesSearchCV(
    SVC(),
    {
        'C': (1e-6, 1e+6, 'log-uniform'),
        'gamma': (1e-6, 1e+1, 'log-uniform'),
        'degree': (1, 8),  # integer valued parameter
        'kernel': ['linear', 'poly', 'rbf'],  # categorical parameter
    },
    n_iter=32,
    cv=3
)

got this error:
Traceback (most recent call last):
  File "<stdin>", line 11, in <module>
  File "C:\ProgramData\Anaconda3\lib\site-packages\skopt\searchcv.py", line 288, in __init__
    return_train_score=return_train_score)
TypeError: __init__() got an unexpected keyword argument 'fit_params'

Tested on linux and windows. Got same error.
Reason of the bug: scikit-learn 0.21.0
How to solve: pip install scikit-learn==0.20.3

I get the same error.
this is due to the depreciation of self.fit_params to the constructor of BaseSearchCV in scikit-learn 0.21.0 ( https://github.com/scikit-learn/scikit-learn/commit/#13519 ).
A simple way to solve it is simply to remove fit_params = fit_params in super(BayesSearchCV, self).__init__(...) and add self.fit_params = fit_params. That's what I did for now.
Not sure if this is the best way to do (skopt might want to also remove fit_params to be consistent)  with sklearn), but it's a good way of using the latest version of scikit-learn.
@iaroslav-ai might be worth solving this before putting v0.6 on pip.

HEllo @YannDubs , I use colab google and I have the same problem.  But I can't edit  SearchCV file for applying your solution. It is available just for read-only. How Can I make the modification??

@LaymounaLhoulwa , you can import and override class methods by inheritance, like this

thanks a lot @escherba for your reply, It is helpful

any progress on fixing this to make scikit-optimize compatible with the latest scikit-learn?

@darenr I presume we are waiting for this: https://github.com/scikit-optimize/scikit-optimize/pull/772

But you should restrict version of sklearn to scikit-learn==0.20.3 waiting for sklearn fix

Is there an ETA for this issue? the PR seems ready to merge

@darenr I presume we are waiting for this: #772

please affecting us here too

I forked the project, fixed this issue and cherry picked some upstream pull
requests too
https://github.com/darenr/scikit-optimize
I'll continue to pull in any upstream changes that improve this project.
On Tue, Aug 13, 2019 at 6:53 PM Paul Steven Conyngham <
<EMAIL_ADDRESS>wrote:

@darenr https://github.com/darenr I presume we are waiting for this:
#772 https://github.com/scikit-optimize/scikit-optimize/pull/772
please affecting us here too
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/scikit-optimize/scikit-optimize/issues/762?email_source=notifications&email_token=ABJ52NIU6TSFXHCXB7BTXJ3QENQR3A5CNFSM4HMJUNJKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD4HO53I#issuecomment-521072365,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABJ52NOWOED73DLJKRFWPBLQENQR3ANCNFSM4HMJUNJA
.

Hi, I am experiencing a similar problem.

Am working in an Ubuntu environment
scikit-learn==0.20.3
-e git://github.com/YannDubs/scikit-optimize.git@d8a5aefe7d472b7c7b929ef9135d71a263af7717#egg=scikit_optimize

The last two bullet points are based on the discussion above. Thoughts on how to handle this problem?

I'm also experiencing this bug!

I get the same error.
this is due to the depreciation of self.fit_params to the constructor of BaseSearchCV in scikit-learn 0.21.0 ( https://github.com/scikit-learn/scikit-learn/commit/#13519 ).
A simple way to solve it is simply to remove fit_params = fit_params in super(BayesSearchCV, self).__init__(...) and add self.fit_params = fit_params. That's what I did for now, if you're just doing a few tests you can use my fork for now (only changed this problem): pip install -e git://github.com/YannDubs/scikit-optimize.git#egg=scikit-optimize (and then uninstall it once this is solved on the main branch, as mine will not be updated).
Not sure if this is the best way to do (skopt might want to also remove fit_params to be consistent) with sklearn), but it's a good way of using the latest version of scikit-learn.
@iaroslav-ai might be worth solving this before putting v0.6 on pip (+ it's only 2 lines to change).

I get the same error.
this is due to the depreciation of self.fit_params to the constructor of BaseSearchCV in scikit-learn 0.21.0 ( https://github.com/scikit-learn/scikit-learn/commit/#13519 ).
A simple way to solve it is simply to remove fit_params = fit_params in super(BayesSearchCV, self).__init__(...) and add self.fit_params = fit_params. That's what I did for now, if you're just doing a few tests you can use my fork for now (only changed this problem): pip install -e git://github.com/YannDubs/scikit-optimize.git#egg=scikit-optimize (and then uninstall it once this is solved on the main branch, as mine will not be updated).
Not sure if this is the best way to do (skopt might want to also remove fit_params to be consistent) with sklearn), but it's a good way of using the latest version of scikit-learn.
@iaroslav-ai might be worth solving this before putting v0.6 on pip (+ it's only 2 lines to change).

what files should be modified?

So whats the verdict regarding this issue. I see there are some active PR that seem to have fixed the bug. Are there any plans to release a new version or at least merge the fixes on the master branch?

Use https://github.com/darenr/scikit-optimize for a fork incorporating the simple fix to this bug.
conda uninstall scikit-optimize OR pip uninstall scikit-optimize
git clone https://github.com/darenr/scikit-optimize
cd scikit-optimize
pip install .

Use https://github.com/darenr/scikit-optimize for a fork incorporating the simple fix to this bug.
conda uninstall scikit-optimize OR pip uninstall scikit-optimize
git clone https://github.com/darenr/scikit-optimize
cd scikit-optimize
pip install .

I'm already using that fork. I am however curious about why it is taking long to merge the fix to the main repo. Has the author abandoned this project?

Use https://github.com/darenr/scikit-optimize for a fork incorporating the simple fix to this bug.
conda uninstall scikit-optimize OR pip uninstall scikit-optimize
git clone https://github.com/darenr/scikit-optimize
cd scikit-optimize
pip install .

I'm already using that fork. I am however curious about why it is taking long to merge the fix to the main repo. Has the author abandoned this project?

I'm also worried that this project might be abandoned.
@iaroslav-ai, thanks for creating such a useful tool. Is this actively maintained by anyone? If not, would it be possible to give @darenr merge permissions?

Hi! This project is indeed no longer maintained unfortunately. We are looking for people to take over.

For this issue:
pip install git+https://github.com/darenr/scikit-optimize
@glouppe I'm volunteer

@glouppe I'm also happy to volunteer.

Hi! This project is indeed no longer maintained unfortunately. We are looking for people to take over.

Hi! This project is indeed no longer maintained unfortunately. We are looking for people to take over.

I'd be happy to volunteer too.

Would be good to see if we can get Conda forge to point at the fixed version really.

Hello I've tried some of the solutions proposed here but I am still getting the error related to fit_params.
I am using scikit-learn 0.21.0


Allow to post process findings via an extension

Expected Behavior
Allow to write plugins which can mapilate detekt's Detektion result.
interface ResultFilter {
    fun filter(result: Detektion): Detektion
}

Current Behavior
The user can suppress findings via baseline file or suppress annotations.
Context
#2224

Implemented via ReportingExtension


Create MS-SQL database

Use #2 for DB schematics...

Done, see 2b9aff07057c90c2e0686ec0e9b66d7ac1a45091 .

about the DB structure:

Domain also needs an image (disk path) field
Maybe Domain should be better called Area
Message.UserID should be an int, (just as in Message.UserID)
there is no way to record what areas a user has chosen

take a close look at https://github.com/axd1967/mz/blob/comments_on_diagrams/Project/Doc/Analysis/class_analysis.png

Yes, made some mistakes during DB creation. Recreating now, fighting against some SQL SMS bugs though that prevent me from creating relations...

They don't need an image field because images are being loaded statically, not using the DB. The reason I implemented the Domain table was to provide a division for the forums.
Renamed the Domain table to Area
userID is an int? Not sure what you're referring to? https://gyazo.com/1424fdcbbc15331adefbc8abb8f65659
Was basing myself on an old diagram... Thanks!

yes, in Message, userid is an int, but it should also be an int in Rating (sorry wrote "Message")

Good catch, thank you!

Pushed 5ea741abe0f763efd4921d600d65dddd9a21f311
Relations: https://gyazo.com/a98da8961a9f46f2ad5a418f842ef5f3

that relations pic looks good, add it to the Doc folder.
careful for the social provider stuff, make sure you know what you are doing.

Roger roger! 3454b6c34bf6d0926bf650e3780130004d35b9bd


python311Packages.python-heatclient: 3.3.0 -> 3.4.0

Description of changes
Closes #280201
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 280229 run on x86_64-linux 1

4 packages built:

heatclient (python311Packages.python-heatclient)
heatclient.dist (python311Packages.python-heatclient.dist)
python310Packages.python-heatclient
python310Packages.python-heatclient.dist


Feature: a single faucet

We want a single faucet that can dispense funds for any rollup.
Closes https://github.com/astriaorg/eth-faucet/issues/1

closing in favor of https://github.com/astriaorg/rollup-deployer-site/pull/58


Adding new resource to DevOps Roadmap

Fixes Issue

Changes proposed

Added a new resource for learning linux by hands on lessons in DevOps Roadmap through the website https://linuxjourney.com/ , for people who prefer learning via Documentation rather than videos.
Screenshots

Note to reviewers

this is my first open source contribution, hoping to do more ahead


Update Documentation for Resource Server

Documentation for Resource Server
Link to ACL-APD design document for reference: https://docs.google.com/document/d/1j289zdt2n5i81vvwJJl-lJB-D4Ld8ihQajjN3qZuGM8/edit

logo to be added in docs and referenced in all documentation-related markdown files

reference to the logo to be added in docs: https://github.com/datakaveri/dx-acl-apd/blob/main/docs/cdpg.png

all the diagrams could be added in /docs folder

Change the references from IUDX to DX, CDPG

video tutorial could be added optionally

README.md 	- Introduction

updated overview diagram: https://drive.google.com/file/d/1x2e_XP6U9K1Nl7EGjv7dxJ08HGp1vhaC/view
Features
Explanation

Understanding DMP APD

Reference Solution_Architecture.md, postman collection, env file

How to guide

Setup and Installation

Reference Setup_and_Installation.md

Tutorial

Tutorials and Explanation (optional)

Reference:

API Docs: Reference Redocly URL of the openapi specs
FAQ: Reference FAQ.md

CONTRIBUTING.md

Git Merge workflow
Code style

tutorial to install plugins could be added optionally

Commit message section

LICENSE.md

No change required

CODE_OF_CONDUCT.md

Contributor code of Conduct
Our pledge
Our standards
Attribution
Reference: https://github.com/datakaveri/dx-acl-apd/blob/main/CODE_OF_CONDUCT.md

SECURITY.md 	* yet to be added in ACL-APD

overview
how to report security issues in an encrypted format
email address to mail to
acknowledgements if any
*inspirations: https://blogs.eclipse.org/post/marta-rybczynska/securitymd-should-i-have-it, https://github.com/microsoft/.github/blob/main/SECURITY.md

docs/Configurations.md

Modules (verticle and other configs)

Api Server Verticle

Key Name
Value Datatype
Value Example
Description

isWorkerVerticle
boolean
false
To check if worker verticle needs to be deployed for blocking operations

verticleInstances
integer
1
Number of instances required for verticles

httpPort
integer
8080
Port for running the instance DX ACL-APD Server

	- Other Configurations
		* include common configs and other configurations

Key Name
Value Datatype
Value Example
Description

version
Float
1.0
config version

zookeepers
Array
zookeeper
zookeeper configuration to deploy clustered vert.x instance

clusterId
String
iudx-acl-apd-cluster
cluster id to deploy clustered vert.x instance

commonConfig.dxApiBasePath
String
/dx/apd/acl/v1
API base path for DX ACL-APD. Reference : link

commonConfig.dxCatalogueBasePath
String
/iudx/cat/v1
API base path for DX Catalogue server. Reference : link

commonConfig.dxAuthBasePath
String
/auth/v1
API base path for DX AAA server. Reference : link

commonConfig.catServerHost
String
api.cat-test.iudx.io
Host name of DX Catalogue server for fetching the information of resources, resource groups

commonConfig.catServerPort
integer
443
Port number to access HTTPS APIs of Catalogue Server

commonConfig.authHost
String
authvertx.iudx.io
Host name of DX AAA Server

* it would help us to see if there any unused configurations that could be removed from config.json

Explanation.md

Tokens used in DX in a tabular format : Token | Purpose | Users

Token, Users and Roles

What tokens are used in the DMP APD Server, Which can use what token and the purpose of using the token in a tabular format : API | Users | Token

Users and Roles

What are the flow for each user, or what services can the consumer or provider could make use of
Diagram : https://drive.google.com/file/d/1W69dMniLSBl6iyetvBYUtkjS5Yf0xxV-/view

Terminologies and Definitions

Those words that are commonly used in the server that would help the consumers, providers to get started with the server by understanding the jargons

docs/FAQ.md

Frequently occuring issues or problems, questions, gaps can be listed here
This markdown file could be updated as and when a new question is asked or discussed

docs/Setup_and_Installation.md

Setup and Installation Guide: helps in the deployment of the server

Configurations: Reference the Configurations.md here
Dependencies:

List down external and internal dependencies used by the server

Prerequisites: to be added wrt each server

Keycloak registration for DX DMP APD as trustee and APD
RabbitMQ configurations

vhost, exchange, queue to exchange, user permissions in a tabular format
body for the API request

Reference: https://docs.google.com/document/d/17zJ-9LgW7izTo0h4sjKmcL-ZilSjZUneq-Yo_ydcI7g/edit#heading=h.q8ebfatvlign

PostgresQL

Setup of db and markdown table for : Table name | Purpose

Auditing

why and where is auditing done
markdown table for auditing : Table name | Purpose
Reference : Postgres and Immudb auditing table schema along with the indexes from auditing server

Database migration using flyway

Where are the flyway files
Why is db migration done
commands to do the db migration

Installation Steps:

Maven
JAR
Docker

Logging and Monitoring

log4j 2

why is it used, related files that the user can configure, any references

Micrometer

why is it used, related files references

Testing

Unit Testing

why is it done, commands to run unit tests, any prerequisites, related files, commands to run

Integration Testing

prerequisites for running integration tests, why is it done, related files, commands to run

Performance Testing

JMeter

why is Jmeter used, command to generate report, references

Security testing

Zed Attack Proxy

why is ZAP used, references

docs/Solution_Architecture.md

Diagram: https://www.google.com/url?q=https://drive.google.com/file/d/1MTkFf4t2VasWSIAwDQotm0Dy-tkPzZZN/view?usp%3Dsharing&sa=D&source=docs&ust=172595396576
Brief info about all the modules and services offered by the server to the user

Reference: IUDX Design Document for the respective server : https://docs.google.com/document/d/1j289zdt2n5i81vvwJJl-lJB-D4Ld8ihQajjN3qZuGM8/edit

docs/Tutorials.md: optional

reference any tutorial wrt server configuration, deployment, services, flows

docs/openapi.yaml

Remove references of IUDX and replace it with CDPG

contact:
name: For support contact CDPG Team at
email<EMAIL_ADDRESS>url: https://dataforpublicgood.org.in/
license:
name: Apache 2.0

/src/test/resources/README.md:

Making base path configurable in the postman environment
Key value pairs used in the postman environment in a markdown table: Key| Value Example | Description

/src/test/resources/resources/DX-Catalogue-Items/RESOURCES_USED.md:

Mention DX catalogue items used in the server in different flows for maintaining the lifecycle of the resources

Key
item ID
Purpose (unit testing, integration testing, other)
Details (the flow in which it is being used)

.github/ISSUE_TEMPLATE/bug-report.md

reference : https://github.com/datakaveri/dx-acl-apd/blob/main/.github/ISSUE_TEMPLATE/bug-report.md
template to create an issue on the repo

.github/ISSUE_TEMPLATE/new-feature.md

reference : https://github.com/datakaveri/dx-acl-apd/blob/main/.github/ISSUE_TEMPLATE/new-feature.md
template to add a feature on the repo

.github/PULL_REQUEST_TEMPLATE.md

reference: https://github.com/datakaveri/dx-acl-apd/blob/main/.github/PULL_REQUEST_TEMPLATE.md
template for every pull request

Wiki: for wiki, github actions are defined under .github/workflows/ as docker-image.yml, move_docs.yml, update_wiki.yml

reference : https://github.com/datakaveri/dx-acl-apd/tree/main/.github/workflows

Thi PR #556 , contains all the required changes


fix: add app support menu item

Add app support link to the menu. The link will open the default mail app, set the subject, and set the current URL in the body of the email.

This looks good- I have two suggestions, which I've added above:

Add a blank space after the colon
Change wording of "App support URL" to "Page" (i think it is more non-tech friendly)
Open the email client in a new tab instead of the current tab


[Azure] error installing docker 1.11.0 & machine 0.7.0 & Ubuntu 16.04

For docker 1.11.0 and docker-machine 0.7.0
Azure creation failed.
log
Installing Docker...
error installing docker:

WorkAround

ssh the machine

2 .edit /etc/apt/sources.list.d/docker.list
with
deb https://apt.dockerproject.org/repo ubuntu-xenial experimental

instal docker-engine
apt-get update
apt-get install docker-engine

logout ssh

docker-machine provision [machine-name]

Please try #3308 (comment), and open a new issue if that does not help you upgrade.

Hm, actually, I seem to have gotten this mixed up with another issue.  This seems to be a dupe of https://github.com/docker/machine/issues/3184 actually.


Fixing js example and cleaning up one sentence

Summary
Cleaning up JS example to have proper order. Cleaning up sentence to be clearer.
Test Plan
NA

Thank you!


PlayerTextDrawSetPos, PlayerTextDrawSetPreviewRot

Hi,
I created a playertextdraw.
When I want to change its position, PlayerTextDrawSetPos randomly adds one or two coordinates (for example, I specify X 510, and the textdraw moves to X 511).
PlayerTextDrawSetPreviewRot does not change textdraw rotation at all.
The condition to use TEXT_DRAW_FONT_MODEL_PREVIEW is met.
I suspected it was omp-server not working correctly with texdraws, but when I disabled the plugin I saw that everything was OK.
Plugin version 2.0.3.
Testing on omp-server (windows).

Are you sure you are using the DynamicPlayerTextDrawSetPos function?
Can I also see your code?

Closed for long periods of non-response.


Add basic Honeycomb instrumentation

When honeycomb.write_key is present, emit events for every incoming
HTTP request, except to the /healthz endpoint.
Authored-by: Nat Bennett<EMAIL_ADDRESS>

Hmm, it looks like the postgres tests are failing with errors like this:
An error occurred while loading ./spec/acceptance/async_bindings_spec.rb.
Failure/Error: CCInitializers.send(method, @config_hash)

It looks like the honeycomb-beeline gem is Apache 2 licensed, so there shouldn't be any licensing issues with adding this gem.

Force pushed some changes to the commit. Now we check only for the top-level honeycomb key, rather than a sub-key, because the call to get the subkey value with [] fails when the top level key returns nil.

Github not being aware that I've signed the CLA doesn't seem to stop me from merging things so 😬


Axum integration

FWIW, we implemented a custom axum integration, since we need manual control over headers during negotiation. It was quite straightforward. Deflate extension negotiation was a bit annoying at the lower level, but all in all not a problem.

Well, I don't know if I'm using the API wrong, but this took a bit of trial and error:
            let deflate_config = ratchet_rs::deflate::DeflateConfig::default(); // XXX: tweak config
            let deflate_ext_provider = ratchet_rs::deflate::DeflateExtProvider::with_config(deflate_config);
            let sec_websocket_extensions: Vec<ratchet_rs::Header> = if ENV_CONFIG.disable_ws_compression {
                // XXX: we disable all extensions
                vec![]
            } else {
                parts
                    .headers
                    .get_all(axum::http::header::SEC_WEBSOCKET_EXTENSIONS)
                    .into_iter()
                    .map(|v| ratchet_rs::Header {
                        name: axum::http::header::SEC_WEBSOCKET_EXTENSIONS.as_str(),
                        value: v.as_bytes(),
                    })
                    .collect()
            };
            let (extension, sec_websocket_extensions) = match deflate_ext_provider.negotiate_server(sec_websocket_extensions.as_slice()) {
                Ok(Some((extension, sec_websocket_extensions))) => (
                    ratchet_rs::NegotiatedExtension::from(Some(extension)),
                    Some(sec_websocket_extensions),
                ),
                Ok(None) => (ratchet_rs::NegotiatedExtension::from(None), None),
                Err(e) => {
                    return RpcError::new(RpcError::BAD_REQUEST, anyhow!("failed to negotiate deflate extension: {}", e)).into_response();
                }
            };

In general, for any websocket implementation, it would be nice if there was a more convenient wrapper for when none of the HTTP stuff is in ratchet's control. This means API should take in a read-only HeaderMap (or iterator of headers or something) and produce out statuscode with headers, but not write them out to anything. Passing "user defined headers" should not be needed, as ratchet should only output the websocket relevant headers and the caller is responsible for putting them in the response that the caller produces along with everything else it wants to do with the response.

The API has since been improved significantly for this use case, so the snippet is outdated.

The API indeed appears to be improved. Here's the rough equivalent of @nakedible-p 's code snippet (for convenience, I just pass in all of the headers, and I don't have the environment variable check):
    let deflate_config = ratchet_rs::deflate::DeflateConfig::default();
    let deflate_ext_provider = ratchet_rs::deflate::DeflateExtProvider::with_config(deflate_config);

    if let Ok(Some((extension, sec_websocket_extensions))) =
        deflate_ext_provider.negotiate_server(&request_headers)
    {
        ...
    }

I'm unsure what to do next though. It is wonderful to have established the dflate extension and its response headers, however I'm unsure what I can call on next. The next step is of course to upgrade the connection, but I don't have a stream representing the socket connection to be able to call accept_with. Any clues? Thanks.

Not full code, but this should get you far enough
            let ratchet_core::server::UpgradeResponseParts {
                response,
                subprotocol,
                extension,
                ..
            } = ratchet_core::server::response_from_headers(&parts.headers, provider, &protocols)
                .map_err(...)?;
            let Some(on_upgrade) = parts.extensions.remove::<hyper::upgrade::OnUpgrade>() else {
                return Err(...);
            };
            tokio::spawn(async move {
                match on_upgrade.await {
                    Ok(upgraded) => {
                        let upgraded = hyper_util::rt::tokio::TokioIo::new(upgraded);
                        let ws = ratchet_core::WebSocket::from_upgraded(
                            ratchet_core::WebSocketConfig::default(),
                            upgraded,
                            extension,
                            bytes::BytesMut::new(),
                            ratchet_core::Role::Server,
                        );
                    }
                    Err(e) => {
                           ...
                    }
                }
            });

            Ok(response)

Thanks for the fabulous guidance here. I've been able to get things integrated into Axum nicely. Here's the code I've landed on - I hope that it helps someone else:
async fn my_handler(
    request_headers: HeaderMap,
    Extension(on_upgrade): Extension<OnUpgrade>,
) -> Result<impl IntoResponse, StatusCode> {
    let deflate_config = ratchet_deflate::DeflateConfig::default();
    let deflate_ext_provider = ratchet_deflate::DeflateExtProvider::with_config(deflate_config);

    let ratchet_core::server::UpgradeResponseParts {
        response,
        extension,
        ..
    } = ratchet_core::server::response_from_headers(
        &request_headers,
        deflate_ext_provider,
        &ratchet_core::SubprotocolRegistry::default(),
    )
    .map_err(|e| {
        log::error!("Problem with headers: {e}");
        StatusCode::BAD_REQUEST
    })?;

    tokio::spawn(async move {
        match on_upgrade.await {
            Ok(upgraded) => {
                let upgraded = hyper_util::rt::tokio::TokioIo::new(upgraded);
                let mut ws = ratchet_core::WebSocket::from_upgraded(
                    ratchet_core::WebSocketConfig::default(),
                    upgraded,
                    extension,
                    bytes::BytesMut::new(),
                    ratchet_core::Role::Server,
                );
                let mut buf = BytesMut::new();

                while let Ok(msg) = ws.read(&mut buf).await {
                    match msg {
                        ratchet_core::Message::Text => {
                            if ws
                                .write(&mut buf, ratchet_core::PayloadType::Text)
                                .await
                                .is_err()
                            {
                                break;
                            }
                            buf.clear();
                        }
                        ratchet_core::Message::Binary => {
                            if ws
                                .write(&mut buf, ratchet_core::PayloadType::Binary)
                                .await
                                .is_err()
                            {
                                break;
                            }
                            buf.clear();
                        }
                        ratchet_core::Message::Ping(_bytes) => (),
                        ratchet_core::Message::Pong(_bytes) => (),
                        ratchet_core::Message::Close(_) => break,
                    }
                }
            }
            Err(e) => {
                log::error!("Problem upgrading: {e}");
            }
        }
    });

    Ok(Response::from_parts(response.into_parts().0, Body::empty()))
}

BTW that last line has to re-create the response as the one that ratchet provides is one with a body of (). If response_from_headers used Body::empty() then they'd be no need for a coercian.

Response in ratchet needs to use () because it isn't linked with any certain framework. To translate it to axum, just say:
            let mut response = response.map(|_| axum::body::Body::empty());

Response in ratchet needs to use () because it isn't linked with any certain framework. To translate it to axum, just say:
response.map(|_| axum::body::Body::empty());

That's tricky - I didn't realise that map just maps over body. Intuitively, I'd expect it to map over its parts and the body. Anyhow, what you state does indeed work. Thanks.
So, if I were to attempt to generalise this axum integration, would there be interest in a PR? @SirCipher ?
I'm thinking that an extractor similar to axum's existing WebSocketUpgrade would be the go, but along with some methods to add in the extensions and protocol adapters prior to calling on_upgrade within the Axum handler. Thoughts? Perhaps an additional crate to this project?


Locked file

https://github.com/gsuitedevs/apps-script-samples/blob/db3ad6816484c1ba9707d34e17d1916713b28007/advanced/bigquery.gs#L91

Can you clarify what the issue is?

The hosted CSV file referenced in the sample somehow lost public read access. I've restored it now.


🎮 Game Request
Game logic and basic description: The game will be a basic ping pong game where both players will control paddles using movement keys and try to prevent the ball from going into their area and score a point.
Point down the features
Game points

A Playing area with a ball
The ball will be bouncing between the players
The players have to defend their goals with their paddles.

Thank you for raising an issue 💗
You can self assign the issue by commenting /assign in comment 😀
Make sure you follow CODE OF CONDUCT and CONTRIBUTING GUIDELINES 🚀
Don’t Forget to ⭐ our GameZone🎮
Make sure you join our Discord🕹

Hey @AaryanManghnani ! Thank you so much for your raising the issue💗
It’s all yours, you can come anytime again and make some contributions! 🚀
Alone, we can do little, but together we can do so much! 😇


Clickable legend

The legend of mission areas should be clickable.

When we click on a particular selection on legend, it should show only points related to that selection


KeyError: 'unexpected key "epoch" in state_dict'

Hi, Thank you very much for your great job. When I tried to run the script to convert a model from Pytorch to Caffe, I faced this error :
(pytorch_env) hossein@hossein:/media/hossein/tmpstore/PytorchConverter-master/code$ python run.py

Traceback (most recent call last):
  File "run.py", line 92, in <module>
    pytorch_net.load_state_dict(torch.load(model_path, map_location=lambda storage, loc: storage))
  File "/media/hossein/tmpstore/PytorchConverter-master/pytorch_env/lib/python3.6/site-packages/torch/nn/modules/module.py", line 355, in load_state_dict
    .format(name))
KeyError: 'unexpected key "epoch" in state_dict'

How do I fix it?
Thank you very much in advance

It would help alot newbies to pytorch like me if you could provide an example on how to use this script! where to change and what not!
My error is now :
Traceback (most recent call last):
  File "run.py", line 92, in <module>
    pytorch_net.load_state_dict(torch.load(model_path, map_location=lambda storage, loc: storage))
  File "/home/hossein/anaconda3/lib/python3.6/site-packages/torch/nn/modules/module.py", line 721, in load_state_dict
    self.__class__.__name__, "\n\t".join(error_msgs)))
RuntimeError: Error(s) in loading state_dict for simplenetv1_imagenet_3p:
	Missing key(s) in state_dict: "features.0.weight", "features.0.bias", "features.1.weight", "features.1.bias", "features.1.running_mean", "features.1.running_var", "features.3.weight", "features.3.bias", "features.4.weight", "features.4.bias", "features.4.running_mean", "features.4.running_var", "features.6.weight", "features.6.bias", "features.7.weight", "features.7.bias", "features.7.running_mean", "features.7.running_var", "features.9.weight", "features.9.bias", "features.10.weight", "features.10.bias", "features.10.running_mean", "features.10.running_var", "features.13.weight", "features.13.bias", "features.14.weight", "features.14.bias", "features.14.running_mean", "features.14.running_var", "features.16.weight", "features.16.bias", "features.17.weight", "features.17.bias", "features.17.running_mean", "features.17.running_var", "features.19.weight", "features.19.bias", "features.20.weight", "features.20.bias", "features.20.running_mean", "features.20.running_var", "features.22.weight", "features.22.bias", "features.23.weight", "features.23.bias", "features.23.running_mean", "features.23.running_var", "features.25.weight", "features.25.bias", "features.26.weight", "features.26.bias", "features.26.running_mean", "features.26.running_var", "features.28.weight", "features.28.bias", "features.29.weight", "features.29.bias", "features.29.running_mean", "features.29.running_var", "features.32.weight", "features.32.bias", "features.33.weight", "features.33.bias", "features.33.running_mean", "features.33.running_var", "features.35.weight", "features.35.bias", "features.36.weight", "features.36.bias", "features.36.running_mean", "features.36.running_var", "features.39.weight", "features.39.bias", "features.40.weight", "features.40.bias", "features.40.running_mean", "features.40.running_var", "classifier.weight", "classifier.bias".
	Unexpected key(s) in state_dict: "epoch", "arch", "state_dict", "best_prec1", "best_prec5", "optimizer".

what should I do now?

If the model is trained by your own. Try another model saving method:
net_for_saving = net.module if use_nn_DataParallel else net
torch.save(net_for_saving.state_dict(), path)

it's not work,  EOFError occurred

@xialeizhou Hello, I have encountered the same problem. Can you solve it? Can I get your help?


Explicit validation failure in error report

I'm looking to address the issue I opened last week (https://github.com/samchon/typia/issues/953)
I'm not overly familar with the codebase, so any suggestions for improvements would be great!

https://github.com/samchon/typia/issues/953#issuecomment-1939088734


The guantlet

Hi Bonus reward on "The Guantlet" under Javascript does not award. I'd post the code but I redid it in python and I am exceedingly lazy. Simply do a line for line conversion from this:
loop:
self.moveRight()
enemies = self.findEnemies()
self.attack(enemies[0])
self.attack(enemies[0])

Recommend running the level again to try to re-trigger the reward.  Sometimes the levels don't fully trigger the first time.

The reason the bonus does not work is that the code is longer than x statements.  Counting statements in JavaScript is broken.  Since this is in another issue, I will close this one.

Other issues are - #1915 and #1808

Thanks popey, I queried for dupes but nothing for "guantlet" had come up.


after renaming resource as sharer the receiver cannot access it by WebDAV using the new name

Describe the bug
After the sharer changed the name of a shared resource, the receiver can see the new name, but cannot access it by using the new name.
d:href in the WebDAV response stays the old one, and that is how the resource can be accessed
Steps to reproduce

create a folder
share the folder
as sharer rename the folder
as receiver do a propfind on the share-jail curl -XPROPFIND -uuu1:uu1 https://host.d'ocker.internal:9200/dav/spaces/a0ca6a90-a365-4782-871e-d44447bbc668$a0ca6a90-a365-4782-871e-d44447bbc668' -k | xmllint --format  -

Expected behavior
d:href element and oc:name should correlate
Actual behavior
d:href shows the old name oc:name the current one
response (shortened)
 <d:response>
    <d:href>/dav/spaces/a0ca6a90-a365-4782-871e-d44447bbc668$a0ca6a90-a365-4782-871e-d44447bbc668/folder1/</d:href>
    <d:propstat>
      <d:prop>
        <oc:name>folder-renamed</oc:name>
        <d:displayname>folder-renamed</d:displayname>
      </d:prop>
      <d:status>HTTP/1.1 200 OK</d:status>
    </d:propstat>
  </d:response>

Setup
Please describe how you started the server and provide a list of relevant environment variables or configuration files.

PROXY_ENABLE_BASIC_AUTH=true OCIS_INSECURE=true PROXY_HTTP_ADDR=<IP_ADDRESS>:9200 OCIS_URL=https://host.docker.internal:9200 PROXY_TRANSPORT_TLS_KEY=../ocis.pem PROXY_TRANSPORT_TLS_CERT=../ocis.crt ./bin/ocis server

Similar findings with search also https://github.com/owncloud/ocis/issues/7262

do we use anywhere propfind on the share-jail? (I didn't find)
web uses propfind on the mounpoint of shared folder which gives <d:href>/dav/spaces/mounpoitUUID</d:href> - works correct
We raised the same issue here and after a long time we closed it as is:  https://github.com/owncloud/ocis/issues/6376

The moodle plugin is doing that using the PHP-SDK.
What happens there is

get all drives and finds the drive-id of the share-drive
gets the drive meta-data via /v1.0/drives/{drive-id}
makes a PROPFIND on that drive using its webDavUrl property
when selecting a sub-folder it will again make a PROPFIND doing webDavUrl of the drive + selected folder name (taken from the last PROPFIND response

@micbar wrote in https://github.com/owncloud/ocis/issues/6376

Once the share was accepted by the recipient, it will not be affected by a rename on the owner side. This prevents "naming  battles" because people tend to be very obsessed of naming and like to have it under their control.

and

We should not rename the file on the recipient side (actual behavior), so no notification is needed.

But in this issue here, the IS a rename happening on the recipient side.
oc:name and d:displayname show the new name, but that cannot be used for webDAV requests.
So as I understand #6376 d:href is currently correct but the rename of oc:name and d:displayname should not happen.
If that would be the case, the moodle plugin would work fine.

get all drives and finds the drive-id of the share-drive
gets the drive meta-data via /v1.0/drives/{drive-id}

Then I suggest you workaround (I hope it can help)

@individual-it I made the fix, that makes d:href and oc:name consistent.

CI is pass

@individual-it I made the fix, that makes d:href and oc:name consistent. But a lot of tests failed.

So now, the rename won't be populated to the receiver.
With fix:
<d:href>/dav/spaces/{virtual-share-drive-id}/afolder</d:href>
<d:propstat>
  <d:prop>
    <oc:name>afolder</oc:name>
    <d:displayname>afolder</d:displayname>
  </d:prop>
</d:propstat>

Before:
<d:href>/dav/spaces/{virtual-share-drive-id}/afolder</d:href>
<d:propstat>
  <d:prop>
    <oc:name>afolder-renamed</oc:name>
    <d:displayname>afolder-renamed</d:displayname>
  </d:prop>
</d:propstat>

Great!

[ ]  ToDo QA-team: cover the case with API tests

Needs clarification.
@butonic

Needs clarification.
@butonic
I followed on
https://github.com/owncloud/ocis/issues/6376#issuecomment-1611771332

@phil-davis Thanks for posting the expected behavior.
I can confirm that for ocis.

This feature is there to avoid naming collisions.  The WebDAV listing of the Share Jail should NEVER have any naming collisions.

For old/new webdav paths, the behavior is still the same.
https://localhost:9200/remote.php/webdav/Shares
https://localhost:9200/remote.php/dav/files/<username>/Shares

<d:href>/remote.php/dav/files/<username>/Shares/afolder</d:href>
<d:propstat>
  <d:prop>
    <oc:name>afolder-renamed</oc:name>
    <d:displayname>afolder-renamed</d:displayname>
  </d:prop>
</d:propstat>

CC @2403905 . Should we re-open the issue?

@micbar Should we make an effort to fix the endpoints output or it doesn't make sense because sharing ng is coming up?

@saw-jan merge the PR
Please validate.

@saw-jan merge the PR Please validate.

Fix cofirmed. Fixed for all webdav endpoints old/new/spaces
API test coverage added in https://github.com/owncloud/ocis/pull/8295
I am clsoing this issue here.
CC @2403905


Update KanaAppraiser.java

 Quality gate: Passed
 Everything looks good
Details

 Quality gate: Passed
 Everything looks good
Details


fix #1767

Currently, when a node allows SSL/port:3001 and does not provide a stable connection for WebSockets subscription the wallet fails to login.
Proposed solution:
fallback for HTTP/port:3000 and retry the connection, in case it fails wallet should fall-back to a different node seeking a valid Websocets connection.

rebased in https://github.com/symbol/desktop-wallet/pull/1770


Add Playstation Logo

The icon I want is:

Name: Playstation
URL: https://www.playstation.com/en-in/
Link to icon: https://upload.wikimedia.org/wikipedia/commons/thumb/0/00/PlayStation_logo.svg/300px-PlayStation_logo.svg.png

@edent I've read through the Contributing guide and I've committed the changes to my forked repo. Should I go ahead and send a PR? Is that okay

For reference, here’s the link to my forked repository:
https://github.com/kycilius/SuperTinyIcons/blob/72ff69e464935a7b93016b6dd71e13458574f111/images/svg/playstation.svg

Fixed in #860


[libdef] platform v1.x

Add libdef for platform
Add tests for it.

The travis error is rx for flow 0.29 - merging.


test error: error[E0405]: cannot find trait AuthoritiesApi in module consensus_authorities

I generate a substrate template node by $ substrate-new-node new-node.
And
$ cd new-node/runtime
$ cargo test

Then I happened below error. (I use ver1.0.0rc)
error[E0405]: cannot find trait `AuthoritiesApi` in module `consensus_authorities`
   --> src/lib.rs:296:30
    |
296 |     impl consensus_authorities::AuthoritiesApi<Block> for Runtime {
    |                                 ^^^^^^^^^^^^^^ not found in `consensus_authorities`
help: possible candidate is found in another module, you can import it into scope
    |
9   | use consensus_authorities::runtime_decl_for_AuthoritiesApi::AuthoritiesApi;
    |

error[E0425]: cannot find function `authorities_call_api_at` in module `consensus_authorities::runtime_decl_for_AuthoritiesApi`
   --> src/lib.rs:231:1
    |
231 | / impl_runtime_apis! {
232 | |     impl runtime_api::Core<Block> for Runtime {
233 | |         fn version() -> RuntimeVersion {
234 | |             VERSION
...   |
300 | |     }
301 | | }
    | |_^ not found in `consensus_authorities::runtime_decl_for_AuthoritiesApi`
help: possible candidate is found in another module, you can import it into scope
    |
9   | use client::runtime_api::runtime_decl_for_Core::authorities_call_api_at;
    |

error[E0425]: cannot find function `authorities_native_call_generator` in module `consensus_authorities::runtime_decl_for_AuthoritiesApi`
   --> src/lib.rs:231:1
    |
231 | / impl_runtime_apis! {
232 | |     impl runtime_api::Core<Block> for Runtime {
233 | |         fn version() -> RuntimeVersion {
234 | |             VERSION
...   |
300 | |     }
301 | | }
    | |_^ not found in `consensus_authorities::runtime_decl_for_AuthoritiesApi`
help: possible candidate is found in another module, you can import it into scope
    |
9   | use client::runtime_api::runtime_decl_for_Core::authorities_native_call_generator;
    |

error[E0405]: cannot find trait `AuthoritiesApi` in module `consensus_authorities`
   --> src/lib.rs:296:30
    |
296 |     impl consensus_authorities::AuthoritiesApi<Block> for Runtime {
    |                                 ^^^^^^^^^^^^^^ not found in `consensus_authorities`
help: possible candidate is found in another module, you can import it into scope
    |
9   | use consensus_authorities::runtime_decl_for_AuthoritiesApi::AuthoritiesApi;
    |

error[E0425]: cannot find function `authorities_call_api_at` in module `consensus_authorities::runtime_decl_for_AuthoritiesApi`
   --> src/lib.rs:231:1
    |
231 | / impl_runtime_apis! {
232 | |     impl runtime_api::Core<Block> for Runtime {
233 | |         fn version() -> RuntimeVersion {
234 | |             VERSION
...   |
300 | |     }
301 | | }
    | |_^ not found in `consensus_authorities::runtime_decl_for_AuthoritiesApi`
help: possible candidate is found in another module, you can import it into scope
    |
9   | use client::runtime_api::runtime_decl_for_Core::authorities_call_api_at;
    |

error[E0425]: cannot find function `authorities_native_call_generator` in module `consensus_authorities::runtime_decl_for_AuthoritiesApi`
   --> src/lib.rs:231:1
    |
231 | / impl_runtime_apis! {
232 | |     impl runtime_api::Core<Block> for Runtime {
233 | |         fn version() -> RuntimeVersion {
234 | |             VERSION
...   |
300 | |     }
301 | | }
    | |_^ not found in `consensus_authorities::runtime_decl_for_AuthoritiesApi`
help: possible candidate is found in another module, you can import it into scope
    |
9   | use client::runtime_api::runtime_decl_for_Core::authorities_native_call_generator;
    |

warning: use of deprecated item 'sr_api_hidden_includes_IMPL_RUNTIME_APIS::sr_api_client::runtime_api::runtime_decl_for_Core::Core::authorities': Please switch to `AuthoritiesApi`.
   --> src/lib.rs:231:1
    |
231 | / impl_runtime_apis! {
232 | |     impl runtime_api::Core<Block> for Runtime {
233 | |         fn version() -> RuntimeVersion {
234 | |             VERSION
...   |
300 | |     }
301 | | }
    | |_^
    |
    = note: #[warn(deprecated)] on by default

warning: use of deprecated item 'sr_api_hidden_includes_IMPL_RUNTIME_APIS::sr_api_client::runtime_api::runtime_decl_for_Core::Core::authorities': Please switch to `AuthoritiesApi`.
   --> src/lib.rs:231:1
    |
231 | / impl_runtime_apis! {
232 | |     impl runtime_api::Core<Block> for Runtime {
233 | |         fn version() -> RuntimeVersion {
234 | |             VERSION
...   |
300 | |     }
301 | | }
    | |_^
    |
    = note: #[warn(deprecated)] on by default

error: aborting due to 3 previous errors

Some errors occurred: E0405, E0425.
For more information about an error, try `rustc --explain E0405`.
error: aborting due to 3 previous errors

Some errors occurred: E0405, E0425.
For more information about an error, try `rustc --explain E0405`.
error: Could not compile `node-template-runtime`.
warning: build failed, waiting for other jobs to finish...
error: Could not compile `node-template-runtime`.

To learn more, run the command again with --verbose.

Will be fixed by: https://github.com/paritytech/substrate/pull/2306

Thanks. 👍


[bug] moment.duration accepts floating point milliseconds

The documentation states:
    To get the number of milliseconds in a duration, use moment.duration().milliseconds().
    It will return a number between 0 and 999.

However, moment.duration(1.23).milliseconds() returns 1.23
moment.duration(1.23).asMilliseconds() returns 1.23 (which seems correct)
To be consistent with the other duration functions (e.g. seconds(), asSeconds(), minutes(), asMinutes()), moment.duration(1.23).milliseconds() should return 1 (an integer between 0 and 999), but moment.duration(1.23).asMilliseconds() should return 1.23.

If I over-write the milliseconds() function to use Math.floor, I can make a passing test. But I'm not familiar enough with this project to know if something like this would be an appropriate solution.
const durationObject = moment.duration(1.23);
const milliseconds = durationObject.milliseconds.bind(durationObject);
durationObject.milliseconds = (): number => {
    return Math.floor(milliseconds());
};

expect(durationObject.milliseconds()).toBe(1);
expect(durationObject.asMilliseconds()).toBe(1.23);

@martin770 we should round all input units to make sure there are no partial milliseconds, seconds etc. I believe the current logic only allows for fractional milliseconds, but I might be wrong. Patches welcome!

I would like to work on it, if it is still open?

I closed my original pull request because it introduced a lot of changes, and there were no comments or reviews on it for quite a while.


Multiple webapps with one nuxtjs instance?

Hi,
Is there a way for a single nuxtjs webapp to serve multiple webapps on different domains for instance like so:
webapp1.com -> pages/webapp1/index.vue
webapp2.com -> pages/webapp2/index.vue
The problem is these sites will have most of the background functionality, layouts, stylesheets in common, only the purpose is slightly different and so some text and pictures are different.
So I would like to avoid maintaining them as two different projects.
I managed to set this up nginx to serve webapp1 and webapp2 by adding their individual paths. Everything loads well first but in the end of it it errors out with:

vue.runtime.esm.js:574 [Vue warn]: The client-side rendered virtual DOM tree is not matching server-rendered content. This is likely caused by incorrect HTML markup, for example nesting block-level elements inside , or missing . Bailing hydration and performing full client-side render.

Any suggestions?
Nicklas

This question is available on Nuxt.js community (#c2642)

@nwocykra I guess the error you get should be caused by an error in your code as it said and not regarding the serving.
I'm approaching something similar just with Varnish instead of Nginx and it seems to work pretty well.
Regarding your paths - why are you pointing directly to a index.vue file?

You can have an upstream server route both domain requests to the same app and treat the requests in a Nuxt middleware.


feat: Added --dry-run functionality which prints cURL command instead of executing HTTP operation.

Added --dry-run functionality which prints cURL command instead of executing HTTP operation.
Related issue: issue #8
Closes #8

I agree with your comment @gesarki


windows10  installation was successful. But the monitor list cannot be seen.

Detailed information in the picture

@itsmikethetech please help me.

Which sub/release version of Windows 10 are you using?

version is 10.18363.592.1

Which sub/release version of Windows 10 are you using?

10.18363.592.1

With a bit of research, to be able to enjoy this driver you have to use 10 build 1904* or later. This is the lowest that would work, I believe from reading official MS documents about the feature.:
Windows10 19041, also know as 20H1/Vb . Released May 2020. The version of the system software, which VDD is based on, isn't compatible with binding GPU and a whole set of other functions.
So if you aim to use this software, suggest you update OS to the most recent build.
Good Luck

thanks!

Same.
Win10Pro
OS: 22H2 ver /  19045.4780 build

anyone with issues, please may you try the beta enabling logs within the vdd_settings.xml then create an issue request and upload your logs so we can diagnose the issue

anyone with issues, please may you try the beta enabling logs within the vdd_settings.xml then create an issue request and upload your logs so we can diagnose the issue

Can you explain please the way to do this? Where should i look at to enable it?

beta enabling logs within the vdd_settings.xml

Ok... Seems i found the issue. In the youtube tutorial, author shows link from main repository page. But there are version for win11. I took this one: Virtual Display Driver <IP_ADDRESS> (Windows 10+) NonHDR version, and now i have 2 monitors.

Ah, installing wrong version then

Ah, installing wrong version then

And you can see that previous users also installed HDR version, that seems was taken from main page. But for different OS version. You can put kind of RED sign at main to be careful when downloading archive )))
Thank you, anyway, now its time to test it )))

If you look at the commit history that was me an hour ago adding the important notice, the links have always been here. We appreciate your consideration on release but as of now upholding two different drivers have been deprecated in favour for our beta release with single driver support. For the mistake of downloading the incorrect by clicking latest release, that is user error, if the user decides not to read the GitHub readme first then that's user error. The website itself may be easier for you to read downloads

Indded a user error. Also I am not a developer so sorry no idea on how release version control and other stuff in github works, so sorry on that part as well as part for not reading readme. I did not start at github I started at the video tutorial on youtube and jumped on to github with straight up to releases ( muscle memory for obtaining latest stuff ) so that way my bad in my use case to not read the instructions. Just posted images if someone looks back at their mistake same as mine to refer what went wrong. Hope it help.  Have a good one ahead. 😊

Yes, no problemo! I updated the readme and adjusted the install instructions to hopefully steer away from any confusion in future! Thanks for the feedback! And hopefully getting the beta out to release soon too!

@kuangke9527 did you resolve this issue too? Was it downloading incorrect version. If so I'll mark the issue as completed and close it

@kuangke9527 did you resolve this issue too? Was it downloading incorrect version. If so I'll mark the issue as completed and close it

sorry. I tried something else. I'm not sure if I downloaded the wrong version. You can turn it off. Thanks.

VDD.<IP_ADDRESS>.zip works for Windows 10 22H2.


advanced augmentation

🚀 Feature
Try more advanced augmentation:

https://github.com/albumentations-team/albumentations

Motivation

@edgarriba?

@edgarriba?

I'd play with Kornia instead :)))


module 'cdk8s' has no attribute 'Size' in cdk8s 0.33.0

Description of the bug:
Python cdk8s-plus-17 0.33.0 does not seem to be compatible with cdk8s 0.33.0
Reproduction Steps:
Setup using cdk8s 0.33.0:
mkdir cdk8s_test
cd cdk8s_test
cdk8s init python-app

pipenv install cdk8s-plus-17

I created this script
import cdk8s_plus_17 as kplus
import cdk8s

app = cdk8s.App()
chart = cdk8s.Chart(app, 'Chart')

kplus.ConfigMap(chart,
                id='myid',
                data={'testKey': 'testValue'}
                )

app.synth()

Then, I ran the script:
pipenv shell
python test_configmap.py

Error Log:
Traceback (most recent call last):
  File "test_configmap.py", line 1, in <module>
    import cdk8s_plus_17 as kplus
  File "/Users/moises.perez/.local/share/virtualenvs/cdk8s_configmap-2joyRsNi/lib/python3.7/site-packages/cdk8s_plus_17/__init__.py", line 670, in <module>
    name_mapping={"medium": "medium", "size_limit": "sizeLimit"},
  File "/Users/moises.perez/.local/share/virtualenvs/cdk8s_configmap-2joyRsNi/lib/python3.7/site-packages/cdk8s_plus_17/__init__.py", line 677, in EmptyDirVolumeOptions
    size_limit: typing.Optional[cdk8s.Size] = None,
AttributeError: module 'cdk8s' has no attribute 'Size'

Environment:

Framework Version: 0.33.0
OS: macOS Catalina 10.16.7

Other:
I was able to make it work by installing the older python cdk8s-plus (without the "-17") and changing the corresponding import reference.

This is :bug: Bug Report

@moises-perez-tfs

Python cdk8s-plus-17 0.33.0 does not seem to be compatible with cdk8s 0.33.0

You're right, we had some issue with this version of cdk8s-plus-17. If you upgrade both cdk8s, cdk8s-plus-17 and cdk8s-cli to the latest versions it should all work.

Thank you @iliapolo !
You are right, 1.0.0-beta.5 works just fine. I guess I thought 0.33.0 was the latest version since that is the one homebrew installs by default.


@connection does not respect required type fields

Describe the bug
If you create a required, @connected field, you can do a create mutation without the field.
To Reproduce
Steps to reproduce the behavior:

Create a project with this schema:

type Project
  @model
  @auth(rules: [
      {allow: owner}
    ])
{
  id: ID!
  title: String!
  codefile: ProjectCodeFile! @connection
}

type ProjectCodeFile
  @model
{
  id: ID!
  location: AWSURL!
}

Run this mutation in your AppSync Queries console:

mutation myMutation {
  createProject(input: {title:"P1"}) {
    id
    title
  }
}

A project is created

Expected behavior
The schema and mutation above should throw an error. Probably a validation exception.
Additional context
Not yet but will look for a solution.

You are correct. I believe the issue is here: https://github.com/aws-amplify/amplify-cli/blob/master/packages/graphql-connection-transformer/src/ModelConnectionTransformer.ts#L158. The definition for updateCreateInputWithConnectionField already expects a nonNull argument but is not being passed it https://github.com/aws-amplify/amplify-cli/blob/master/packages/graphql-connection-transformer/src/definitions.ts#L7. It looks like we will need to update these two lines https://github.com/aws-amplify/amplify-cli/blob/master/packages/graphql-connection-transformer/src/definitions.ts#L19 & https://github.com/aws-amplify/amplify-cli/blob/master/packages/graphql-connection-transformer/src/ModelConnectionTransformer.ts#L164

Hey Michael! Thank you for the analysis.


cmd/cover: TestFuncWithDuplicateLines failed on linux, windows, darwin ...

See https://build.golang.org/log/049bdcd59d296fca6358a91c75b1d411e3370e61

Dup of #30861


Feature Request: More Debug/Trace output

For debugging higher level protocols, it would be really handy to get the a Trace / Debug dump of the
request right before it is sent.
response right after it returns.
(At least request and headers, body would be nice)
I know it might be a controversial subject, but invaluable when trying to figure out what's going on.
If nothing else, at least in non-release build.
When I have these issues, I have to change all cargo.toml's and modify a local copy of reqwest...
I'll be happy to make a PR for it...

You might be interested in the connection_verbose() method of ClientBuilder, which will log all bytes written and read to a connection.

@seanmonstar
Thanks for getting back to me.
I will check it out.
Though, a generic debug/trace would be rather handy... the trace get's a little "noisy"...

A simple change to the response debug would do it... in most cases:
if cfg!(debug_assertions) {
debug!("response {:?}", res);
} else {
debug!("response '{}' for {}", res.status(), self.url);
}

This is indeed controversial. The concern is that someone can accidentally start logging sensitive data (like headers) just by setting RUST_LOG=debug or similar.
This is why the verbose option exists, as it prevents accidental logging.


[Bug] Unexpected usage when specifying language

Reproducible in vscode.dev or in VS Code Desktop?

[X] Not reproducible in vscode.dev or VS Code Desktop

Reproducible in the monaco editor playground?

[X] Not reproducible in the monaco editor playground

Monaco Editor Playground Code
In my Angular 13 project, using monaco-editor 0.33.0, I am loading the Monaco editor in my Angular project via this method, which adds the "loader.js" script tag to the document:
loadMonaco(): void {
    const loader = (): void => {
      const vsPath = this.monacoPath;
      (window as any).amdRequire = (window as any).require;
      (window as any).amdRequire.config({ paths: { vs: vsPath } });

      // Load monaco
      (window as any).amdRequire(['vs/editor/editor.main'], () => {
          this.ngZone.run(() => {
            this.isMonacoLoaded$.next(true);
            console.log('loader finished');
          });
      }, (error) => console.error('Error loading monaco-editor: ', error));
    };

    // Check if AMD loader already available
    const isAmdLoaderAvailable = !!(window as any).amdRequire;

    if (isAmdLoaderAvailable) {
      return loader();
    }

    const loaderScript: HTMLScriptElement = document.createElement('script');
    loaderScript.type = 'text/javascript';
    loaderScript.src = `${ this.monacoPath }/loader.js`;
    loaderScript.id = 'monacoLoader';
    loaderScript.addEventListener('load', loader);
    document.body.appendChild(loaderScript);
  }

I see the following JS resources loaded in my network activity:

loader.js
editor.main.js
editor.main.nls.js

In my component, I am observing isMonacoLoaded$:
ngOnInit(): void {
    this.editorService
        .isMonacoLoaded$
        .pipe(
          filter(isLoaded => isLoaded),
          take(1),
        )
        .subscribe((loaded) => {
          this.initMonaco();
        });
  }

From there, I am creating the monaco editor instance:
initMonaco(): void {
    const options: monaco.editor.IStandaloneEditorConstructionOptions = {
      ...this.defaultOptions,
      ...this.options,
      value: this.value,
    };

    this.editorService.setCustomTheme();

    this.editor = monaco.editor.create(this.editorElement.nativeElement, options);

    const model = this.editor.getModel();

    this.value = model.getValue();
  }

The problem is, that if I specify a language in the options, I get the following errors:
ERROR Error: Unexpected usage

Error: Unexpected usage
    at EditorSimpleWorker.loadForeignModule (editorSimpleWorker.js:460:1)
    at webWorker.js:38:1
    at _ZoneDelegate.push.23484._ZoneDelegate.invoke (zone.js:409:1)
    at Object.onInvoke (core.mjs:25568:1)
    at _ZoneDelegate.push.23484._ZoneDelegate.invoke (zone.js:408:1)
    at Zone.push.23484.Zone.run (zone.js:169:1)
    at zone.js:1326:1
    at _ZoneDelegate.push.23484._ZoneDelegate.invokeTask (zone.js:443:1)
    at Object.onInvokeTask (core.mjs:25555:1)
    at _ZoneDelegate.push.23484._ZoneDelegate.invokeTask (zone.js:442:1)
    at errors.js:12:1
    at timer (zone.js:3128:1)
    at _ZoneDelegate.push.23484._ZoneDelegate.invokeTask (zone.js:443:1)
    at Object.onInvokeTask (core.mjs:25555:1)
    at _ZoneDelegate.push.23484._ZoneDelegate.invokeTask (zone.js:442:1)
    at Zone.push.23484.Zone.runTask (zone.js:214:1)
    at push.23484.ZoneTask.invokeTask (zone.js:525:1)
    at ZoneTask.invoke (zone.js:514:1)
    at data.args.<computed> (zone.js:3108:1)

What am I doing wrong?
Actual Behavior
No response
Expected Behavior
No response
Additional Context
No response

I figured it out. You just have to access the right instance of the editor, which should be window.monaco.editor. Once you change which object you're using, everything works as expected.
The error is not helpful - at all, but I hope my post will help anyone who may come across this in the future as I had a hell of a time finding any sort of documentation regarding this. In fact, I didn't find any documentation at all. I just happened to try this from looking at other people's code samples.


File comment API documentation is hard to find

The comments here are hard to find:
https://github.com/kkinnear/zprint/blob/139e3abf724ed655a9fe8d718a5b3548abefebd5/src/zprint/core.cljc#L638-L674
Is this a supported API and can they be moved to the readme?

This is currently documented in the lein-zprint readme (towards the end).  Is that how you are using it, or are you interested in the programmatic interface to this capability, where you process a file directly?

On Dec 13, 2017, at 11:09 AM, Dieter Komendera<EMAIL_ADDRESS>wrote:
The comments here are hard to find:
https://github.com/kkinnear/zprint/blob/139e3abf724ed655a9fe8d718a5b3548abefebd5/src/zprint/core.cljc#L638-L674
Is this a supported API and can they be moved to the readme?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

We're using zprint-file-str directly in two ways:
Emacs:
Hijacking the cider-nrepl backend used by cider-format-* functions to use zprint-file-str, see https://github.com/clojure-emacs/cider-nrepl/issues/453
Non-Emacs:
We start a socket server as part of the development environment, which passes the connection input stream to zprint-file-str and streams the result back. In a custom shell script we pass stdin via ncat to the socket server and the result from the socket server to stdout.
Both have the advantage of not having to boot a jvm.

Thanks for the information.  I used to do exactly the same thing with a server for formatting, also to avoid starting up a JVM.  Then my wife introduced me to app-cds and I built the zprint-filter which starts in less than 1 second for me (using appcds of course).  If you haven't tried that, you might see what it does in your environment.  I find it nice just having it always there, and no requirement to start anything up to use it.
In any case, yes, I expect to support the ;!zprint capability long term.  I'll move the documentation for it (well, duplicate it from lein-zprint) into the zprint readme.  I still have it on my todo list to add an options map to zprint-file-str based on the earlier issue.
Thanks again for your interest!

Yes we tried app-cds and it worked as you say. But we want to introduce a workflow in our project where the files are reformatted before every save and provide a go-mode and reason-mode like experience. For this we need something measured in ms rather than in seconds :)
Thank you for your awesome work!

Great, just wanted to make sure you knew about app-cds.
I'm impressed with the progress you've made on the cider integration!  I'll reference the issue (since you kindly told people how to do it for now) in the readme, and when it gets all worked out I would very much welcome some words from you on how to configure cider to use zprint (which will make a lot more sense coming from you, as I'm not a cider user).
In looking at how you did the integration, would providing a zprint-file-str flavor with just one parameter (the file name) be helpful for people?  I could easily default the "zprint-specifier".  Just a thought.

In looking at how you did the integration, would providing a zprint-file-str flavor with just one parameter (the file name) be helpful for people? I could easily default the "zprint-specifier". Just a thought.

Not sure. I think it is helpful to have to specify it so error messages are easier to reason about.


Rancher Default Library reports 404 in GUI

What kind of request is this (question/bug/enhancement/feature request):
Bug
Steps to reproduce (least amount of steps as possible):
Install Rancher 2.1.6 on a 3 Node HA Cluster with external load balancer.
Result:
In the GUI, the Default Library Reports:
Unexpected HTTP status code 404 from [https://git.rancher.io/charts/index.yaml], expected 200
Environment information

Rancher version: v2.1.6
Installation option: HA

It just works again, despite us not changing anything.

for me i still have the same issue instead, while i can download the helm library


How to generating nested modules in JavaScript from Rust nested modules

Summary
How  to generating nested modules in JavaScript from Rust nested modules
Additional Details
Here is my use case
pub mod lunnar {
  #[derive(Debug)]
  #[wasm_bindgen]
  pub struct YearCalender {
      inner: Rlunnar::YearCalender,
  }
}

Here are wasm-pack generate:

export class YearCalender {
  //
}

I hope  JS calss YearCalender   under module lunnar

As far as I'm aware, you can't generate nested modules. You can, however, easily write a JS wrapper around generated code


Feature: Added swipe gesture for back/forward navigation

Resolved / Related Issues

[x] Were these changes approved in an issue or discussion with the project maintainers? In order to prevent extra work, feature requests and changes to the codebase must be approved before the pull request will be reviewed. This prevents extra work for the contributors and maintainers.
Closes #100

Validation
How did you test these changes?

[x] Did you build the app and test your changes?
[ ] Did you check for accessibility? You can use Accessibility Insights for this.
[ ] Did you remove any strings from the en-us resource file?

[ ] Did you search the solution to see if the string is still being used?

[ ] Did you implement any design changes to an existing feature?

[ ] Was this change approved?

[ ] Are there any other steps that were used to validate these changes?

Open app ...
Click settings button ...

If you have the time you're encourage to clone this PR and see if the gestures 'feel right' to you. Please leave a comment if there's anything that could be improved.

Please ignore the fact that there's 23 extra commits I accidentally pressed the wrong button

The UX for the indicator thingy needs to be improved. Currently, it's just a circle that comes out from the side of the page, with no indication of how far to go to trigger a navigation. I really like the way that Chrome does it but it'd be better to have some original element as well.

@heftymouse I noticed you added a setting to turn off this behavior, is there a reason you think we need a setting?

One could find themselves accidentally triggering it if they use a lot of
force when scrolling which could get annoying
On Sun, Apr 16, 2023, 8:49 PM Yair @.***> wrote:

@heftymouse https://github.com/heftymouse I noticed you added a setting
to turn off this behavior, is there a reason you think we need a setting?
—
Reply to this email directly, view it on GitHub
https://github.com/files-community/Files/pull/12043#issuecomment-1510411528,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOJEU7R7KDDZNXJYNPIEFFTXBQEYLANCNFSM6AAAAAAWZ36GB4
.
You are receiving this because you were mentioned.Message ID:
@.***>

Sorry, I don't have a device with touchpad

I tried. It is very cool and works well for me!
It would be better if it could refresh the page when swiping down.

One could find themselves accidentally triggering it if they use a lot of
force when scrolling which could get annoying

I have never had an unintended swipe, but I think there should be an option.

It would be better if it could refresh the page when swiping down.

We used to have this but it wasn't widely used and was unreliable, users navigate back and forth a lot more than they refresh.

We used to have this but it wasn't widely used and was unreliable

I didn't know it existed before. With this implementation, it appears to be reliable.

It wouldn't hurt to give it another try 👍

I agree that it's too sensitive, we can solve this by only triggering the action if the layout is already scrolled to the left or right.

I think this need to be disabled on the column layout.


create module-wide setting for returning as string by default

If you're writing in a context that requires your paths be strings - i.e., arcpy for python 2.x - then you have to wrap each one in str(). It would be a lot cleaner to have an environmental variable you set within the module, e.g.
import pyhere
pyhere.return_as_string = True

This would be False by default.
https://stackoverflow.com/questions/1977362/how-to-create-module-wide-variables-in-python

This is hilariously difficult to do! And I'm going to deem it as not actually needed, since it's only for arcpy... maybe it will make it into a future release, but not now.


Masthead:  Persistent search text in Masthead searchbar after search is executed

The problem
Hello, we are using the Masthead component with searchbar.
When the user types in a search keyword and executes search, the search keyword disappears from the searchbar after the user has been redirected to a new page.
The solution
We would like the search keyword to be always visible in the searchbar (after the search executes, after a redirect..)
Example can be found on ibm.com/search page,  e.g. https://www.ibm.com/search?lang=en&cc=us&q=api
The keyword is visible even after search executes and the user can continue writing
Additional information
Please, be aware that we are using the searchNoRedirect prop, because we do not want our users to be redirected to the default ibm.com/search page. We want them to be redirected to our own search page.
This feature is requested by the Knowledge Center team.
Expected:

Next Steps:  Reach out to the MH search team to see whether their code can be contributed back to Carbon for ibm.com Library.
Shixie agrees with the design, keeping the search term in the search box when results are presented.

JC can contribute the code during their next sprint which begins next week.  (Feb 22 - Mar 5)