Errors during test

I added this module from the terraform registry as this:
module "aws_ebs_csi_driver" {
  source                = "andreswebs/eks-ebs-csi-driver/aws"
  version               = "1.1.0"
  cluster_name          = module.kubernetes_cluster.cluster_id
  cluster_oidc_provider = module.kubernetes_cluster.oidc_provider_arn
}

When running a terraform validation I got this:
╷
│ Error: Missing required argument
│
│   on .terraform/modules/test-application.aws_ebs_csi_driver/main.tf line 32, in module "resources":
│   32: module "resources" {
│
│ The argument "cluster_name" is required, but no definition was found.
╵
╷
│ Error: Unsupported argument
│
│   on .terraform/modules/test-application.aws_ebs_csi_driver/main.tf line 42, in module "resources":
│   42:   helm_atomic            = var.helm_atomic_creation
│
│ An argument named "helm_atomic" is not expected here.
╵
╷
│ Error: Unsupported argument
│
│   on .terraform/modules/test-application.aws_ebs_csi_driver/main.tf line 44, in module "resources":
│   44:   helm_wait              = var.helm_wait_for_completion
│
│ An argument named "helm_wait" is not expected here.
╵
╷
│ Error: Unsupported argument
│
│   on .terraform/modules/test-application.aws_ebs_csi_driver/main.tf line 46, in module "resources":
│   46:   helm_timeout           = var.helm_timeout_seconds
│
│ An argument named "helm_timeout" is not expected here.
╵

Seems like there is a problem in your test code.

I get the same on terraform plan

+1

+2


Discarding and saving a modified buffer does not work

Hi, first of all, I like this tabline plugin. It has finally replaced my old airline configuration. Thank you for that.
But I have noticed a problem with closing a modified buffer both through the close icon as well as calling the api  lua require('cokeline/mappings').by_step('close', '0')
I looked into the issue and it turned out that the result of the choice in utils.lua  buf_delete(bufnr, focus, wipeout) is not correctly evaluated or maybe my configuration broke it.
Instead of returning 1,2 or 3 it returns the String selected. I was able to locally fix it by modifying the code like this:
function(choice)
      if choice == 1 or choice == "Save and close" then
        vim.api.nvim_buf_call(bufnr, vim.cmd.write)
        buf_del_impl(bufnr, focus_next, wipeout, false)
      elseif choice == 2 or choice == "Discard changes and close" then
        buf_del_impl(bufnr, focus_next, wipeout, true)
      elseif choice == 3 or choice == "Cancel" then
        return
      end

But I don't know if that is a good solution.
I'm using both telescope.nvim as well as dressing.nvim on Neovim 9.2 if that helps.
Also an additional thing is that in the case of "Save and close", this only works if the buffer has already a filename. It would be nice if the filename could be specified if empty.

Thanks for reporting! I'll get a fix in for this soon.

Also, just a note that you should use . in requires instead of /, it's not very well documented but there is a difference, / behaves more similarly to dofile than require.

Should be fixed in 2f80eebe

Thank you, it works perfectly. Also thanks for the hint, I didn't know there was a difference between . and / in the require statement.

Yeah, not sure if it's Neovim-specific or lua specific, but changing all of the requires in cokeline to use . did make a difference when I did that refactor a few months ago.


dispose dialog in tests

@patflynn close #212

@loosebazooka

ping

PTAL. I got two of the tests to pass at the command line and cut out two others.

what's the status of this PR? can we close it?

Are we still seeing the memory leak? You may have turned off that test. I'll have to check.


Offical exmple does not work?

As you see , when i copy the exmple from offical packages, and run go run main.go, access browser just show 404. What is the mistake I got from , please help! Thanks in advance!
> go env
set GOARCH=amd64
set GOBIN=
set GOCACHE=C:\Users\Vector\AppData\Local\go-build
set GOEXE=.exe
set GOFLAGS=
set GOHOSTARCH=amd64
set GOHOSTOS=windows
set GOOS=windows
set GOPATH=D:\GoProject
set GOPROXY=
set GORACE=
set GOROOT=D:\Dev\go
set GOTMPDIR=
set GOTOOLDIR=D:\Dev\go\pkg\tool\windows_amd64
set GCCGO=gccgo
set CC=gcc
set CXX=g++
set CGO_ENABLED=1
set GOMOD=
set CGO_CFLAGS=-g -O2
set CGO_CPPFLAGS=
set CGO_CXXFLAGS=-g -O2
set CGO_FFLAGS=-g -O2
set CGO_LDFLAGS=-g -O2
set PKG_CONFIG=pkg-config
set GOGCCFLAGS=-m64 -mthreads -fmessage-length=0 -fdebug-prefix-map=C:\Users\Vector\AppData\Local\Temp\go-build291331690=/tmp/go-build -gno-record-
gcc-switches

fs.go line 74
files["/" + zipFile.Name] = f
fs.go line 125
func (fs *statikFS) Open(name string) (http.File, error) {
name = filepath.Clean(name)
if f, ok := fs.files[name]; ok {
name is "\hello.txt" on Windows. // filepath.Clean("/") → ""
fs.files["/hello.txt"] is ok but fs.files["\hello.txt"] is no.
How to fix:
fs.go line 74
files[filepath.Clean("/" + zipFile.Name)] = f

Hi, got 404 too on osx, been trying to pack swaggerui


Instructions to download ICP are incorrect in Install and set up page

Description
The instructions given here https://mi.docs.wso2.com/en/latest/install-and-setup/install/installing-integration-control-plane/#installing-the-integration-control-plane_1 to install ICP is incorrect.
Instructions are given like the following

It says to click on Download and then click Other Components/MI Dashboard and there is no such thing in the page where you land after clicking Download
The download page

Closing the issue as now it works. Might have been a caching issue


Add method to send heartbeat if necessary on consumer

As discussed in #355 implemented a method for exposing heartbeat from a consumer.
@michaelsauter FYI

I think the idea was to have a separate heartbeat method that actually triggers a heartbeat.

@headshota @dasch any plans on merging this? :)

@ale7714 sorry, lost track of this. I think we need to consider what clients would expect in terms of behavior – the "send if necessary" API is not very clear from the outside. I'd prefer an API that just forces the heartbeat to be sent.

@dasch Hmm, interesting question.
Right now, the gem seems to only have this "send if necessary" behaviour though, so that was the easiest to expose without changing anything.
Would you send the heartbeat then also if it doesn't make sense? E.g. if they happen in close succession?
The use case we have (see #355) is best served by "send if necessary" as a forced heartbeat would result in more heartbeats than necessary ...

@michaelsauter if you have an existing use case for "send if necessary" then I think that's a better indicator than my intuition – I'll merge this as-is.
Thanks! ❤


fix: hello 문자를 화면에 출력하도록 수정

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


Fix bugs from mock pe

Codecov Report
Base: 60.93% // Head: 60.93% // No change to project coverage :thumbsup:

Coverage data is based on head (cc88b49) compared to base (360d148).
Patch coverage: 33.33% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff            @@
##             master     #135   +/-   ##
=========================================
  Coverage     60.93%   60.93%
  Complexity      522      522
=========================================
  Files           105      105
  Lines          1989     1989
  Branches        218      218
=========================================
  Hits           1212     1212
  Misses          707      707
  Partials         70       70

Impacted Files
Coverage Δ

...logic/commands/DeletePersonFromMeetingCommand.java
84.61% <0.00%> (ø)

...ress/logic/commands/AddPersonToMeetingCommand.java
83.33% <50.00%> (ø)

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

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


Allow bpfctl get to apply to a kernel ID or bpfd UUID

Allow bpfd get to reference a kernel ID or bpfd UUID since today it only works with the program's Kernel ID

Moved to only using Kernel Id.


Incremented version to 1.3.4.

Signed-off-by: Zelin Hao<EMAIL_ADDRESS>Description
Increment version to 1.3.4
Part of https://github.com/opensearch-project/opensearch-build/issues/2205
Category
[Enhancement, New feature, Bug fix, Test fix, Refactoring, Maintenance, Documentation]
Why these changes are required?
What is the old behavior before changes and new behavior after changes?
Issues Resolved
[List any issues this PR will resolve (Is this a backport? If so, please add backport PR # and/or commits #)]
Testing
[Please provide details of testing done: unit testing, integration testing and manual testing]
Check List

[ ] New functionality includes testing
[ ] New functionality has been documented
[ ] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

This PR requires this PR to be backported to 1.3 branch in OpenSearch core.

@zelinh Re-running the integration tests. If it still fails, it would need a new security artifact to be built and pushed to the repository.


chapter 6 Imageviewer-native

When add "import Qt.labs.platform 1.0",  when running the application, ti failed and reported this problem: "qrc:/main.qml:14:14: Cannot assign object of type "MenuBar" to property of type "QQuickItem*" as the former is neither the same as the latter nor a sub-class of it."
I also changed some codes as follow:
    MessageDialog {         id: aboutDialog         title: qsTr("About")         Label {             anchors.fill: parent             text: qsTr("QML Image Viewer\nA part of the QmlBook\nhttp://qmlbook.org")             horizontalAlignment: Text.AlignHCenter         }         //standardButtons: StandardButton.Ok         StandardButton: StandardButton.Ok     }
Qt Creator 4.11.0
Based on Qt 5.14.0 (MSVC 2017, 32 bit)
Built on Dec 10 2019 12:27:13
From revision 017ed74400

@linzhongzi sounds a bit stupid, but you have to move your import Qt.labs.platform to the top, before ´import QtQuick.Controls`.

Still have that same error

If anyone still has the same error after the solution that @vgribov said
try it like this :
   MenuBar{

        Menu{
            title:  "&File"
            MenuItem{
                text: "&Open..."
                onTriggered: fileOpenDialog.open()
            }
        }

        Menu {
            title: qsTr("&Help")
            MenuItem {
                text: qsTr("&About...")
                onTriggered: aboutDialog.open()
            }

        }

    }

The issue seems to be actually the import for Qt.labs.platform shadowing the one for QtQuick.Controls 2.x.
The fix us to namespace the import:
import Qt.labs.platform as Platform
This fix is actually suggested for the problem regarding the Dialog being an abstract class solved in:
https://bugreports.qt.io/browse/QTBUG-67542

@linzhongzi sounds a bit stupid, but you have to move your import Qt.labs.platform to the top, before import QtQuick.Controls.

Hahaha, it works! Hmm.. perhaps it is the advanced technique mentioned at the beginning of the chapter.

try it like this:
Item {     MenuBar {         ... ...


Time derivative of jacobians

Hello, I have a question regarding Jacobian derivatives.
I am trying to compute the acceleration drift by multiplying the derivative of the Jacobian with the velocity. Specifically, I need to compute the derivative of the Jacobian such that it computes the acceleration drift when multiplied with the velocity, i.e., dJ*dq=a_drift.
Currently, I can obtain the acceleration drift by computing a=Jddq+dJdq using getFrameClassicalAcceleration with zero joint acceleration. However, I explicitly need the Jacobian derivative.
When I compute getJointJacobianTimeVariation, it provides me with a Jacobian derivative that computes the spacial acceleration. However, as far as I recall, this derivative does not include the w x w x r term (see https://en.wikipedia.org/wiki/Spatial_acceleration).
How can I obtain the actual dJ that computes the acceleration drift? I came across a similar issue in this Github discussion (https://github.com/stack-of-tasks/pinocchio/issues/901), but it does not provide a sufficient answer regarding dJ.
I would appreciate your help.
Best regards,
Max

Classic and spacial accelerations just differ at the linear quantity level, where:
$a_s = a_c - \omega \times v$.
And you know that, $a_s = J_l \ddot{q} + \dot{J}_l \dot{q}, \omega = J_a \dot{q},  v = J_l \dot{q} $
You can manually compose all these expressions to get your results.
Hope it helps you,
Best,
Justin


Theme upload erases data

I make changes on my local shopify files, I have custom sections and snippets I already added to the theme editor and filled them in.
Once I commit everything, I do theme upload to upload all files to shopify.
After that, if I refresh the theme editor, all the custom content I added to any page gets deleted.

So you edited files on the online editor, then you uploaded changes from your machine and they got wiped out?

If that is the case this is because shopify's theme api does not handle partial uploads nor does it diff changes. There is no way to merge your changes in any sane way using a REST API. However please consult this issue for catching moments when uploading with change the file https://github.com/Shopify/themekit/issues/330

I have the exact same problem. I'm editing the theme files on my local machine. When I use "theme upload" all my custom content like sections and blocks are erased when I go to my site. Since my site is only sections and blocks my site gets blank directly after a deploy. Is there anything I can do to make everything be as it was except from the changes I've made locally? It's so frustrating to create all content again for 5+ pages everytime I make a deploy. #

I have the same problem. I didn't change files on the online editor, I changed sections settings. They all wipe out every time I use theme upload. Is there any workaround for this?

Please note the answer already posted, please also see the work done on the open PR. Also if you are making changes to your theme on shopify then on your local machine, you should be downloading those changes because otherwise you will overwrite them. If you edit a word document in one folder and copy another word document over it do you expect that the word document now has both of the files contents?

I'm not making changes to the theme, I'm making changes to the sections settings in the editor. How do you download those? Do you understand what I mean? You really think I don't know how to do simple version control? 🤣

Those changes are made to your theme. They are reflected in your config/settings_data.json

And I didn't know that!
@tanema Thanks, now I know how to keep data from getting lost on Shopify and how to edit a Word document, what a day!

You may also want to watch out for config/settings_schema.json A lot of theme developers just ignore most of the files in the config folder.


How to Create a Dynamic Search Box Attached with the Same Object

I was trying to create a Seach panel at top of the Page to search the Records
Can you please help on this

you have two options to do it,

Simple option is to add a text box with search button to data table component and append search string as where clause filter to {!v.whereclause}  (test your SOQL in workbench before making this change) and call helper.queryRows in controller to re-render datatable

Complex one If you want to make search box as a separate component, you need to create an event and include it datatable component for it to listen and fire your helper.queryrows again. (same as #1). refer event handling in documentation for help.

Hi Ramanathan ,
Thanks for the reply
i will try it out
and really it was an amazing blog.
Regards,
Riaz
On Sun, Jun 11, 2017 at 8:27 PM, ramanathan<EMAIL_ADDRESS>wrote:

you have two options to do it,

Simple option is to add a text box with search button to data table
component and append search string as where clause filter to
{!v.whereclause} (test your SOQL in workbench before making this change)
and call helper.queryRows in controller to re-render datatable
2.
Complex one If you want to make search box as a separate component,
you need to create an event and include it datatable component for it to
listen and fire your helper.queryrows again. (same as #1
https://github.com/ramanathansj/salesforce-lightning-datatable/issues/1).
refer event handling in documentation for help.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/ramanathansj/salesforce-lightning-datatable/issues/1#issuecomment-307643809,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AbcdwnTkuY4XkjCfT1qhI6pD47ZiPCgXks5sDCNugaJpZM4Nweo-
.

Thanks for the feedback and glad to hear its useful.


Spiral Traversal of Binary Tree

Kindly check it out and do the needful.
Thanks!
Fixed #853

also use nullptr instead of NULL

All the required changes done.
Kindly review it.

@tk315 update the list in the tree folder.

@rudrakshi99 Already done

Please resolve merge conflicts.


Dg&u gupdate

Closes #3, closes #12, closes #7

Codecov Report

Merging #5 (47c578b) into master (c36220c) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master       #5   +/-   ##
=========================================
  Coverage     72.15%   72.15%
  Complexity      399      399
=========================================
  Files            70       70
  Lines          1232     1232
  Branches        125      125
=========================================
  Hits            889      889
  Misses          311      311
  Partials         32       32

Continue to review full report at Codecov.

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


⚠ Jenkins Plugins Artifactory has degraded performance

In f2b8bff, Jenkins Plugins Artifactory (https://get.jenkins.io/plugins/artifactory/) experienced degraded performance:

HTTP code: 200
Response time: 1038 ms

Resolved: Jenkins Plugins Artifactory performance has improved in a6ce170 after 12 minutes.


Can't start shadowsocks-rust as service via systemctl with v2ray-plugin activated

Can't start shadowsocks-rust as service via systemctl with v2ray-plugin activated. Says it can''t find file, certificate i guess.
Sep 28 04:41:25 vpn4 ssserver[52693]: 2024-09-28T04:41:25.447157224Z  INFO shadowsocks server 1.21.0 build 2024-09-22T14:50:01.940728947+00:00
Sep 28 04:41:25 vpn4 ssserver[52693]: 2024-09-28T04:41:25.44925582Z ERROR failed to start plugin "v2ray-plugin" for server <IP_ADDRESS>:8388, err: No such file or directory (os error 2)
Sep 28 04:41:25 ssserver[52693]: server aborted with No such file or directory (os error 2)
But it perfectly run under root user.
{
"server": "<IP_ADDRESS>",
"server_port": 8388,
"password": "Nu8ahch1weiPahn4Vo1e",
"method": "chacha20-ietf-poly1305",
"timeout": 300,
"mode": "tcp_and_udp",
"plugin": "v2ray-plugin",
"plugin_opts": "server;tls;cert=/opt/ss/fullchain.pem;key=/opt/ss/privkey.pem;host=my.domain.com;loglevel=debug;"
}

changed path of plugin to full path
"plugin": "/opt/ss/v2ray-plugin",
and it worked!


[WIP] test: reduce test time

Work in Progress

based on this number, it seems some tests are missing in this refactoring, could you revise your base script

The decrease in codecov test coverage seen before is because some tests results did not get picked up by codecov even though tests have actually run.
For instance, in this commit, the corresponding codecov build for this batch of tests does not seem to appear here, even though builds for a few other test batches are correctly shown.
It's not very clear to me why the above only occurred for some new test batches, it might be the case that the name provided in codecov action is too long?  But one possible fix is to remove this optional name field.


Import of vue-i18n fails on vue/tsconfig versions 0.4.0

When i try to install vue-i18n@9 it says:
Could not find a declaration file for module 'vue-i18n'. '/home/user/repo/node_modules/vue-i18n/dist/vue-i18n.esm-bundler.js' implicitly has an 'any' type.
There are types at '/home/user/repo/node_modules/vue-i18n/dist/vue-i18n.d.ts', but this result could not be resolved when respecting package.json "exports". The 'vue-i18n' library may need to update its package.json or typings.ts(7016)
Works fine with 0.2.0. Fails with every version greater than that.

I believe this has been fixed in vue-i18n 9.3.
If you still encounter this issue after updating vue-i18n, please open a new issue with a reproduction link. Thanks!.


gzstream seek and tell broken

I catched another minor bug, but this time I am not sure how to fix it, therefore the issue. Turns out ifgzstream, named_ifgzstream, ... do not correctly implement seeking and telling. Seek and tell is used when loading VTU meshes.
The issue should be reproducible by using GlVis to load any VTU mesh. If not, then it may be related to using clang with libc++ (instead of gcc with libstdc++).

Thanks for reporting this. I'm working on it.

It seems to be an issue in multiple instances here.
The XML reader complains on a perfectly valid XML VTU file that I can open in Paraview. E.g. an mfem build without zlib doesn't work either.
The second issue is with zlib activated, the seekg/tellg methods are not implemented for the ifstream wrappers.

The XML reader complains on a perfectly valid XML VTU file that I can open in Paraview.

Hmm, this is strange -- I think @pazner tested this when he added tinyxml2 in #1894. Maybe the specific file you have, uses some feature that is not supported by tinyxml2? I think ParaView itself uses libexpat for XML parsing so there definitely can be differences.

The second issue is with zlib activated, the seekg/tellg methods are not implemented for the ifstream wrappers.

One option could be to use re-open for this type of input instead of seekg here: https://github.com/mfem/mfem/blob/f71a028ffcc7e3db50ae06f5feb828584ccabda8/mesh/mesh.cpp#L3543-L3548
assuming the input is a named_ifgzstream like here: https://github.com/mfem/mfem/blob/f71a028ffcc7e3db50ae06f5feb828584ccabda8/mesh/mesh.cpp#L3568-L3569

I agree that this might be the only non-intrusive fix right now. Especially to support compressed inputs.

The XML reader complains on a perfectly valid XML VTU file that I can open in Paraview.

Hmm, this is strange -- I think @pazner tested this when he added tinyxml2 in #1894. Maybe the specific file you have, uses some feature that is not supported by tinyxml2? I think ParaView itself uses libexpat for XML parsing so there definitely can be differences.

I believe @jandrej was trying to load a VTU file with "appended binary data", which is not supported in MFEM's reader (neither is inline binary data, for that matter). A better error message would probably be helpful (Julian, if you send me the file I can find where in the code the error message should be placed).

Discussion continued in #2017.


fix(network): add network module to status and about

No new frontend pages yet, just adding the existance of the module in some pages.

It is ready for review, but I don't want to merge it until we have network in the setup

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


Multiple Workers with PyTorch Dataloader

First off, JAX is a great library, love using it.
My issue is that my PyTorch dataloader freezes whenever I use >0 workers. The dataset itself works with jnp.arrays. How can I fix this? Here's a minimal example of what I'm doing:
import os
import jax.numpy as jnp
import numpy as np
import torch
import torchvision

class SpecialMNIST(torch.utils.data.Dataset):
    def __init__(self, train, seed=0):
        super().__init__()
        self._data = torchvision.datasets.MNIST(os.getcwd(), train, download=True)
        self._data_len = len(self._data)

    def __getitem__(self, index):
        img, label = self._data[index]
        return jnp.asarray(np.asarray(img)), label

    def __len__(self):
        return self._data_len

def collate_fn(batch):
    if isinstance(batch[0], jnp.ndarray):
        return jnp.stack(batch)
    elif isinstance(batch[0], (tuple, list)):
        return type(batch[0])(collate_fn(samples) for samples in zip(*batch))
    else:
        return jnp.asarray(batch)

Then following code works:
dataset = SpecialMNIST(train=False)
dataloader = torch.utils.data.DataLoader(dataset,
                                         collate_fn=collate_fn,
                                         num_workers=0)
next(iter(dataloader))

But the following code just hangs:
dataset = SpecialMNIST(train=False)
dataloader = torch.utils.data.DataLoader(dataset,
                                         collate_fn=collate_fn,
                                         num_workers=1)
next(iter(dataloader))

The problem goes away when I change the jnp arrays to np arrays in the dataloader and collate function. However, in my actual use case, I have some complex data augmentation that I would like to JIT compile and run on the CPU. Is there any way to do this in JAX? Or do I have to stick to normal numpy functions when data loading? Thanks!

I think you can use np.asarray() to convert to normal numpy after the call to jax, this will avoid creating a copy of the data if I am not mistaken.

Thanks for the tip @cgarciae! Even with this change (i.e. returning an np.array and collating np.arrays), as long as I have any JAX calls in my dataset/dataloader, it hangs when using multiple workers.

I've seen this behavior on MacOS which you use threads within processes, not sure if its the same issue thought.

@n2cholas would adding
import torch.multiprocessing as multiprocessing
multiprocessing.set_start_method('spawn')

at the top of your script help? What you're seeing might be due to the XLA state being inconsistent after forking off the data loading processes. It will make their start-up quite a bit more expensive, but it might at least be more correct. Also, please remember to wrap your code in if __name__ == '__main__' if you do that (see the multiprocessing docs).

My follow up questions were not relevant to this issue, so I will close this issue and ask them in the new discussions tab. Thanks again @apaszke for your help!!


Some questions about ’resample‘

Hi，
Today I have some problems. When I use the MNE library for downsampling, I want to reduce the sampling rate from 5000Hz to 256Hz.
I used the following method.
Raw. resample (256, npad='auto')
Or should I use the following method?
Raw = raw. filter (None, 128)
Raw. resample (256, npad='auto')
I looked at the source code, but did not understand whether the function "resample" had its own filtering.

Yes, resampling includes the necessary anti-aliasing filter (see also here).

Looks resolved, closing


Don't set fixed width on the comment notifications dropdown

Longer words (as in German) can break the dropdown's layout.
Tested with Firefox 89 and Chromium 93 on macOS 11.4

The translation has been corrected as in 2.14.1, but the width is still set and the content overflows.

@krzysztofjeziorny thanks so much for taking the time to submit a PR, sorry it took so long to get a review.
I have merged this in via cf064f2b9990e90f5c5ca5d4405fe75a16604e0a
I just made one small refinement, changing the width to min-width instead of removing it, so it works nicely for super short translations also :)
I have added your name to the contributors list, please check and let me know if you would like it changed.
Honestly, thanks again for submitting a fix.

@lb- Glad I could help! Yeah, setting min-width is a better way to fix it.


dtype of NumPy array created from Julia array is object on Julia nightly

Affects: PythonCall
Describe the bug
I'm executing the following code
julia> using PythonCall

pkg> conda add numpy

julia> Py(randn(10)).to_numpy().dtype

I expect the dtype to be a float64, and indeed, on Julia v1.10.0 and earlier, the result is Python: dtype('float64'). On the Julia nightly, the result is Python: dtype('O'), which corresponds to an object dtype.
Your system
For v1.10.0:
julia> using CondaPkg, Pkg

julia> versioninfo()
Julia Version 1.10.0
Commit 3120989f39b (2023-12-25 18:01 UTC)
Build Info:
  Official https://julialang.org/ release
Platform Info:
  OS: Linux (x86_64-linux-gnu)
  CPU: 8 × 11th Gen Intel(R) Core(TM) i5-1135G7 @ 2.40GHz
  WORD_SIZE: 64
  LIBM: libopenlibm
  LLVM: libLLVM-15.0.7 (ORCJIT, tigerlake)
  Threads: 11 on 8 virtual cores
Environment:
  JULIA_NUM_THREADS = auto
  JULIA_EDITOR = code

julia> Pkg.status()
Status `/tmp/jl_lFXo6Z/Project.toml`
  [992eb4ea] CondaPkg v0.2.22
  [6099a3de] PythonCall v0.9.15

julia> CondaPkg.status()
CondaPkg Status /tmp/jl_lFXo6Z/CondaPkg.toml
Environment
  /tmp/jl_lFXo6Z/.CondaPkg/env
Packages
  numpy v1.26.3

For nightly:
julia> using CondaPkg, Pkg

julia> versioninfo()
Julia Version 1.11.0-DEV.1341
Commit 63188d58c09 (2024-01-19 23:30 UTC)
Build Info:
  Official https://julialang.org/ release
Platform Info:
  OS: Linux (x86_64-linux-gnu)
  CPU: 8 × 11th Gen Intel(R) Core(TM) i5-1135G7 @ 2.40GHz
  WORD_SIZE: 64
  LLVM: libLLVM-16.0.6 (ORCJIT, tigerlake)
Threads: 8 default, 0 interactive, 4 GC (on 8 virtual cores)
Environment:
  JULIA_NUM_THREADS = auto
  JULIA_EDITOR = code

julia> Pkg.status()
Status `/tmp/jl_oeTJsz/Project.toml`
  [992eb4ea] CondaPkg v0.2.22
  [6099a3de] PythonCall v0.9.15

julia> CondaPkg.status()
CondaPkg Status /tmp/jl_oeTJsz/CondaPkg.toml
Environment
  /tmp/jl_oeTJsz/.CondaPkg/env
Packages
  numpy v1.26.3

Not sure if my PR was related #441 but I see similar issue going Python -> Julia.

Oof that's a fun one! Personally I'm not going to worry about it until there is a release candidate though.


Main media gallery

Fixes/Closes/References #
Type

✨ Feature
🐛 Bugfix
📈 Test
🚑 Health

[ ] Demoed to @

Value
All image galleries are currently given in-article and in-content modifiers by default, which give them specific styles – these styles are overly specific as soon as we want to use the image gallery anywhere other than within the (hugely esoteric) article template.
This allows for the modifiers to be passed in at the time of creating the galleries so that new galleries will, by default, fill the width of their containers.
Including an image gallery as main media type was an exercise to see this working in practice – it should probably be removed from this PR as I don't think we'll want to use galleries in this way.
Screenshot

Checklist
All

[ ] PR labelled and assigned
[ ] Any introduced code complexity has been flagged

@davidpmccormick is this a "when you have time" CR?

Closing because the world's changed since it was opened.


Translation of French user manual into English

I tried to translate it as faithfully as possible. Used Google Translate, DeepL, and Cambridge French-English dictionary to guide me.
Please feel free to let me know if you have any questions or requests.

Don't mind too much about makefiles, they have some issues, I will push some fixes later.
@trebmuh are you interested in reading the document? I have put a current copy online.
http://jpcima.sdf1.org/software/documentation/ADLplug/manual/en/manual.html

@jpicma Thanks, I pushed a commit with your suggested changes.

@trebmuh are you interested in reading the document? I have put a current copy online. http://jpcima.sdf1.org/software/documentation/ADLplug/manual/en/manual.html

@jpcima @bsutherland , I did a read on it and here are some findings (hopefully right!):

general: not specific too the EN version

If you encounter a problem with the software, please create an issue on the project page:
https://github.com/jpcima/ADLplug
-> @jpcima : what about linking straight to the issues tracker? (https://github.com/jpcima/ADLplug/issues).

Binary downloads are provided for both Windows and Mac.
-> Binary downloads are provided for both Windows, Mac and Debian.

Specific to the EN version:

ADLplug is an FM synthesizer
-> ADLplug is a FM synthesizer

2 or 4 operator
-> 2 or 4 operators

It is usually written "plugin", but once it's written "plug-in" (in Description). It'd be good to be consistent (ie: keeping plugin everywhere).

Another bank an be selected from the included collections.
-> Another bank can be selected from the included collections.

Each operator in an algorithm acts as either a modulator or a carrier.
-> Each operator in an algorithm that acts as either a modulator or a carrier.

The OPL3 has three classes of algorithm:
-> The OPL3 has three classes of algorithms:

(not sure)

In 2op algorithms
-> In 2op algorithm

in gray
-> in grey

The release ® parameter
-> The release (R) parameter

which make it more suitable
-> which makes it more suitable

The ADplug software was developed
-> The ADLplug software was developed

Overall, great job @bsutherland ! Really great.
I hope that helps.

@trebmuh @jpcima Thank you for the reviews! I will take a closer look and update the files later today.

@trebmuh Thanks again for the proofreading, good catches. I applied most of your suggestions. In a couple of cases where I didn't, I'll comment inline with explanations.

@trebmuh has rightfully observed the documentation is not published under a license.
I will publish under a dual LAL 1.3 / CC-BY-SA 4.0.
Is it fine with you @bsutherland?
(probably, it's a little matter but would be indelicate of me to relicense the contribution without asking)

@jpcima Fine by me! Thanks for checking.
Also, please feel free to edit the commit history (eg squash, rebase) if you prefer.


Add LIMIT clause when executeTakeFirst()

Hello to everyone developing Kysely!
I have one suggestion.
I am using executeTakeFirst() when trying to retrieve just one record. The current function still retrieves the desired record, but I wanted to add a LIMIT clause for performance reasons.

I APPROVED by mistake. I'm sorry.


access forbidden

Expected content
Actual content
Link to page:

Hello @ankitvarmait,
We're sorry you're having this issue! Since this is not a docs-related issue, please contact our Support team directly and they can help you out. You can access support contact options by logging into https://support.sendgrid.com.
SendGrid Docs Team


Take a break

I've had some problems in my life recently, I may not be able to pay attention to this repo. I hope I can resolved my problem at soon and come back here in March. I promise I would not give up this repo. If you could sponsoring me without pressure I would be very grateful for that.
I'm really sorry about that. 🙏

https://twitter.com/Shini_92/status/1490288557726052352

@Shinigami92 Thank you. 🙏

https://twitter.com/patak_dev/status/1490374970870018056
Thanks for your amazing work @johnsoncodehk, take as much time as you need

@patak-dev Thank you. 🙏 I have no idea that the vue community was so loving. I will make improve vue DX to the extreme as my main goal!

Totally fine to take your time. Come back stronger. Also due to some issues github sponsorships are not working in my country rn and all of my previous sponsorships were cancelled automatically. Will try to fix that and resume it. Thanks for your invaluable contribution 😊💚

@soulsam480 Don't mind, I'm already very grateful, thank you. 🙏

Hey mate! Wish you all the best. Huge thanks for an awesome tool! Wanted to subscribe for a long time and this is a great reason =)

Take care @johnsoncodehk and all Vue users are really thankfull of the awesome tool you built. I'm still amazed by the quality of it💚💚

Thank you so much for all your hard work on this tool. It is a huge part of what makes using the new vue stuff so exciting :)
Take care of yourself!!

Another signal boost https://twitter.com/_jessicasachs/status/1490404586405965831?s=21
@johnsoncodehk Cypress would not have switched to Vue if not for your amazing work. Take the time you need. Sending love and support.

https://youtu.be/KsIN33ANybM?t=1571
Johnson is a beast 💪

Was already sponsoring...just triggered a one-time payment to help. Take care.
What I give back versus what I earn thanks to your work is not comparable... Thanks again.

sponsoring

https://twitter.com/haoqunjiang/status/1490523806515208196
Thank you for all the great help. Take care 🙏

Thank you for your hard work.
I wish you good health and happy every day.

Thank you !
Good luck !

Thank you for Volar, @johnsoncodehk! Truly a vital part of the Vue ecosystem. Take your time and see you soon <3

Thanks for all the sharing, thanks for all the encouragement, thanks for all the help, I'm really wordless gratitude. 🙏🙏🙏
I feeling everything is going to get better, will see you soon.

Do what you need to do!
If the repo has maintainers who you trust to review & merge PRs then you're already in a great place to take care of yourself.

I'm trying to create some extra charge features to increase income to help myself (https://twitter.com/johnsoncodehk/status/1496978502385422336). If you've sponsored me in the past, you can access these features as an early sponsor (I'm really appreciate you guys 🙏). If you're a new sponsor, now we have a new sponsor tier to access extra features.
But anyway, obviously the world has more important things to focus on now, take care yourself, protect those around you. 💪

Totally fine to take your time. Come back stronger. Also due to some issues github sponsorships are not working in my country rn and all of my previous sponsorships were cancelled automatically. Will try to fix that and resume it. Thanks for your invaluable contribution blushgreen_heart

Restored sponsorship today. :green_heart:


[BUG] - Number input increase / decrease action buttons not working

NextUI Version
2.1.13
Describe the bug
Number input, wrapped by a tooltip component, increase / decrease action buttons not working
Your Example Website or App
  const Test: React.FC = () => {
    return (
      <>
        <Input type='number' />  {/* It work */}
        <Tooltip>
          <Input type='number' />  {/* Increase / decrease action buttons not working  */}
        </Tooltip>
     </>
    )
  }

Expected behavior
click the increase / decrease action will change change the number in input
Screenshots or Videos
No response
Operating System Version
Arch Linux
Browser
Chrome

I also ran into this issue but I'm not using any Tooltips.
The draggable="false" workaround didn't work for me, but by chance found another workaround of adding an empty onKeyDown listener.
<Input onKeyDown={()=>{}} ....


As a registered and logged-in user, I want to be able to edit the properties of my photo to make changes if they don't match my original specifications

Acceptance Criteria

[ ] A user can set a new name for his photo
[ ] A user can trade his trophies for a boost

Development Tasks FE

[ ] Create EDIT button
[ ] Align delete button with every concerning picture
[ ] On press of edit button: Edit button renders formfield on editable property “name” of concerned and shows a button CHANGE LOCATION
[ ] On press of edit button: Buttons CANCEL (red) and ACCEPT CHANGES (neutral, as template button) appear next to the picture
[ ] On press of CHANGE LOCATION: map is shown as it is shown when you originally set a location and you can confirm location with a button (on click) you return to the edit overview
[ ] Include external API call in the code
[ ] Function for CANCEL in the edit overview: get back to the picture overview without effects on the concerned image and image description/name
[ ] Function for ACCEPT CHANGES: API PUT call to update name of the picture, get back to the original overview layout, rerender to show new name

Development Tasks BE

[ ] Create an endpoint to update properties of the image

[ ] Can category be set differently later on (pro: if new categroeis are added according to request, can assign picture to it; contra: you can try to be on highlights per category by simply change category

Changing the title does not make sense here in pictures because in home the title will be not shown anyways. Also the idea is that a user can delete a picture if he made a mistake with the tile. Therefore this user story will be deleted.


cdn.babylonjs.com and preview.babylonjs.com unaccessable

BabylonJS's can and preview subdomains cannot be reached right now? Even in my local javascript trying to access some warm etc. Therefore all systems down. Need to solve ASAP

It fixed


Cannot approve api expose from AAD

Category

[x] Bug

Describe the bug
I'm currently using admin center to approve an api expose from AAD, but met 400 error. Could you please help?

Steps to reproduce

DELETE THIS LINE BEFORE SUBMITTING - Provide steps to reproduce the behavior:

Expose a scope from aad

Here is how to expose api for AAD, FYI:

Click the "Add a scope" button under "Scopes defined by this API"

Click "Save and continue"

Add a scope name

Set "Who can consent?" to "Admins and users"

Add an admin consent display name

Add an admin consent description

Click "Add scope"

Save the Scope URL

upload the package with this scope
{
"resource": "api://6648fe4f-9ffd-4206-a3f9-7fbb94d7c0d5/Scope",
"scope": "Scope"
}

Go to admin page, try to approve this scope

Expected behavior
This scope can be approved
Environment details (development & target environment)

DELETE THIS LINE BEFORE SUBMITTING - Complete the following (if ignored you'll be prompted for it before we can address your issue... save the time and provide it in your initial submission).

Your Developer Environment: [e.g. Windows 10 | MacOS 10.15.x]
Target Environment: [e.g. SharePoint Server 2016 | SharePoint Server 2019 | SharePoint Online]
Framework: [e.g. .NET Framework v3.x | .NET Core v3.x | Node.js v6/v8/v10/v12]
Browser(s): [e.g. Chrome v79 | Safari v22 | FireFox v31 | IE v11 | Edge | Chromium Edge v79]
Tooling: [e.g. VS Code | SPFx v1.10.0 | Visual Studio 2019]
Additional details: The more context you can provide, the easier it is (and therefore quicker) to help.

Additional context

DELETE THIS LINE BEFORE SUBMITTING - Provide any other context about the problem.

Thanks for your contribution! Sharing is caring.
Submission guidelines

!!IMPORTANT!! - All submissions must complete the baseline sections included in this template. Ignoring or deleting this template may result in closing the issue with the label type:incomplete-submission.
Follow our guidance on How To Create Good Issues.
If you are reporting an issue around any of the documents or articles, please ensure that you have referenced the specific file or URL of the related document.
If you have technical questions about the framework, we monitor the tags #spfx, #spfx-webparts, and #spfx-tooling on SharePoint StackExchange.
Remember to include sufficient details and context.
If you have multiple questions, suggestions, or bugs, please submit them in separate issues.

DELETE THIS SECTION BEFORE SUBMITTING
Thanks for your contribution! Sharing is caring.

Find the root cause: the resource name must equal to aad name and cannot contain special chars.


C# interface to TypeScript class/interface not generating properties

The interface properties are not generated anymore on TypeScript interface.
            var schema = await JsonSchema4.FromTypeAsync<IPerson>();
            var schemaData = schema.ToJson();

            Console.WriteLine(schemaData);

        public interface IPerson
        {
            string Name { get; set; }
        }

Output:
{
  "$schema": "http://json-schema.org/draft-04/schema#",
  "title": "IPerson",
  "type": "object",
  "additionalProperties": false
}

I just noticed that there is a new option for generator (GenerateAbstractProperties). I apologize.

I have following use case:
public interface INode {
 string NodeType {get;}
}

public class Node1 : INode {
[DefaultValue('node1')]
string NodeType {get;} = 'node1';
}

public class Node2 : INode {
[DefaultValue('node2')]
string NodeType {get;} = 'node2';
}

public class NodeModel {
List<INode> Nodes {get; set; }
}

This way I can use the generated classes (Node1, Node2) from TS app to send the values back to the server without setting the NodeType value (since default value is generated in class). On server side I can use JsonInheritanceConverter or custom JsonConverter to convert the nodes back to specific type (by examination of nodeType property).
Unfortunately this scenario is not working because I'm getting 'System.InvalidOperationException: The JSON property 'nodeType' is defined multiple times on type 'xxx'. Actually I'm getting this when generating the ApiController proxy using NSwag but the error is thrown on JsonSchemaGenerator.LoadPropertyOrFieldAsync function.
It would be nice if something like following is generated in this case:
interface INode {
nodeType: string;
}

class Node1 implements INode { // not important to implement the interface
nodeType: string = 'node1';
}

class Node2 implements INode { // not important to implement the interface
nodeType: string = 'node2';
}

Do you have any idea how to handle this scenario?

My bad. I totally misunderstood the use of JsonInheritanceConverter and inheritance. Now the scenario described above is working as expected. I apologize again.

Maybe we should add a sample to the docs or make it more clear?

I think the doc about inheritance is fine - It's just me not paying attention more carefully. I was under impression that we can manually control the discriminator values using DefaultValue attribute but that is not the case. The builtin inheritance handling is implemented in the way to act transparently (i.e discriminator property is not supposed to be defined explicitly on the base type) and possible discriminator values are hard coded (i.e.: resolved by type name).


Use Sever Side Apply to Update Status and Manage Finalizers

Currently this is not done by SSA.

This would make more sense if the status would be written by multiple controllers, usually, that is not the case. Was checking, and neither the Kubernetes built-in operators seems to be using SSA to update the status.
Might we interesting however for the resource update, on the other hand with SSA we are sending just the intent, not the whole resource, so that might be confusing for the users that not the primary resource from the reconciler argument should be used, rather there should be the resource created from scratch and only the changes on the resource should be applied.


Use valid field name

This PR uses the valid product price field. Next migration (0106) renames price to price_amount.
Screenshots

Pull Request Checklist

[ ] Privileged views and APIs are guarded by proper permission checks.
[ ] All visible strings are translated with proper context.
[ ] All data-formatting is locale-aware (dates, numbers, and so on).
[ ] Database queries are optimized and the number of queries is constant.
[ ] Database migration files are up to date.
[ ] The changes are tested.
[ ] GraphQL schema and type definitions are up to date.
[ ] Changes are mentioned in the changelog.

Here is the report for 5f06888e351b683ff682f0cb591e28320b6e38f1 (mirumee/saleor @ fix/migration-price-field)
Base comparison is 65e00b0d1497d329320b33c8bcdbe1fa8febfb9d.
No differences were found. (click me)

# api.benchmark checkout
  test name                           	left count 	right count	duplicate count
  ------------------------------------	-----------	-----------	---------------
  add billing address to checkout     	         34	         34	             20
  add shipping to checkout            	          7	          7	              0
  checkout payment charge             	         14	         14	              0
  complete checkout                   	          6	          6	              0
  create checkout                     	         48	         48	             24

# api.benchmark homepage
  test name                           	left count 	right count	duplicate count
  ------------------------------------	-----------	-----------	---------------
  retrieve main menu                  	          5	          5	              0
  retrieve product list               	          4	          4	              0
  retrieve secondary menu             	          5	          5	              0
  retrieve shop                       	          2	          2	              0

# api.benchmark product
  test name                           	left count 	right count	duplicate count
  ------------------------------------	-----------	-----------	---------------
  product details                     	         13	         13	              3

# api.benchmark variant
  test name                           	left count 	right count	duplicate count
  ------------------------------------	-----------	-----------	---------------
  retrieve variant list               	         15	         15	              8


configure-private-chart-operator

adjust cluster-apps-operator to  add support for private clusters and use values to set chart-operator to run in private cluster mode

changelog fixed

@fiunchinho happy with changelog?


Update s3, sso, ssooidc to 2.28.1

About this PR
📦 Updates

software.amazon.awssdk:s3
software.amazon.awssdk:sso
software.amazon.awssdk:ssooidc

from 2.27.24 to 2.28.1
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "software.amazon.awssdk" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "software.amazon.awssdk" }
}]

labels: library-update, early-semver-minor, semver-spec-minor, commit-count:1

Superseded by #84.


Prisma schema in wrong location?

I'm using a monorepo, containing a server and a nextjs frontend. The prisma schema is ejected to a shared folder so it can be used in several apps and during builds. The shared client is imported via import {PrismaClient} from '@shared/prisma'
Folder structure:
/server
/frontend/pages/api/auth/[...nextauth].ts
/shared/schema/prisma.schema

When I run the frontend using "@next-auth/prisma-adapter": "^0.5.4", and "next-auth": "^3.29.0", I get this error when clicking "sign in":
Error:
Invalid `prisma.user.findUnique()` invocation:
Query engine exited with code 101

thread 'main' panicked at 'Could not open datamodel file ".../frontend/.next/server/pages/api/auth/schema.prisma"', query-engine/query-engine/src/opt.rs:196:53

Here I think the adapter is looking in the api file in .../pages/api/auth instead of in ./shared/schema. Is this a bug or is there a way to set this? The typescript seems to only accept undefined for options, and I cannot find any docs for the options.
If I manually copy in the file to the next build directory it works fine.

I don't think it's an issue with the adapter. we don't care about where you store your schema file. I'm happy to reopen if you have a reproduction

I've added a demo project here with the same problem - https://github.com/izaacdb/next-auth-test-case/tree/master

I'm not that experienced with Prisma, didn't know the schema has to be in a certain location for it to work. 👀. @ndom91 any ideas?

Hmm as far as I know the schema file has to be in. /prisma/schema.prisma in the root. Never used prisma in a mono repo and especially not with shared schema like this, sorry.
@williamluke4 any ideas?

I am also running into this. I have a monorepo with multiple apps who use Prisma, and I ma running into many issues. Seems like Prisma is trying to be too clever at times. At first I had the prisma stuff in a dedicated package, but after reading this thread, the best solution I found is to move the prisma folder into the root and then cp the schema.prisma file in the build script of all apps that use Prisma.
I would love to see improvements in this area, and a configurable prisma schema path is a good start. Ideally Prisma would look for the schema relative to the main file of the package it is in, so that Prisma can be put into a package and installed into any apps that want to use it. Currently the schema path resolution is dumb as a rock, it tried to look for the schema in a random folder at one point.

As mentioned above, this sounds like an upstream issue to me. We don't really care where your prisma client is generated, as long as it has the expected methods available. You could ask on their repo.


Unable to build powergate CLI

Followed the steps mentioned in https://docs.textile.io/powergate/ to clone and build Powergate CLI.
I am facing following errors and unable to proceed.
My environment is M1 macbook Pro with golang version go version go1.18 darwin/amd64
On running make install-pow following error is displayed.
go: downloading github.com/marten-seemann/qtls-go1-17 v0.1.0-rc.1
go: downloading github.com/matttproud/golang_protobuf_extensions v1.0.1
# github.com/lucas-clemente/quic-go/internal/qtls
../../../go/pkg/mod/github.com/lucas-clemente/quic-go@v0.21.2/internal/qtls/go118.go:5:13: cannot use "quic-go doesn't build on Go 1.18 yet." (untyped string constant) as int value in variable declaration
make: *** [install-pow] Error 2

Thought i found a workaround mentioned here https://github.com/ipfs/go-ipfs/issues/8819 .
Tried to proceed with it but I am running into the below error. Looks like go-ipfs version used is 0.8 which is very old and using some flags which have been removed in later dependency (quic-go).
CGO_ENABLED=0 go install -ldflags="-X github.com/textileio/powergate/v2/buildinfo.GitSummary=v2.6.2-6-gb82b011-dirty -X github.com/textileio/powergate/v2/buildinfo.Version=git -X github.com/textileio/powergate/v2/buildinfo.BuildDate=2022-05-12T00:08:11Z -X github.com/textileio/powergate/v2/buildinfo.GitCommit=b82b011 -X github.com/textileio/powergate/v2/buildinfo.GitBranch=master -X github.com/textileio/powergate/v2/buildinfo.GitState=dirty" ./cmd/pow
# github.com/ipfs/go-ipfs/core/node/libp2p
../../../go/pkg/mod/github.com/ipfs/go-ipfs@v0.8.0/core/node/libp2p/transport.go:21:7: undefined: quic.RetireBugBackwardsCompatibilityMode
make: *** [install-pow] Error 2

The newest libp2p has a limitation (I guess is temporal) for people compiling with Go v1.18.x, which is a release from some weeks ago.
You need to compile with v1.17.x or older. (i.e: I tried with v1.17.8 now and didn't have an issue compiling)
Be sure that your git repo is clean (no changes).

Thanks, will try on a 1.17 version then.
But, is it the lipp2p which is causing the issue? Because i don't see an open issue for the same under libp2p.
And the issue i have referenced above seems to have addressed this in go-ipfs and libp2p.
I see that go-ipfs-0.8 is mentioned as dependency in powergate which is an older version and hence must be having this limitation whereas the latest is 0.12.

Yes, it's libp2p since the error:

../../../go/pkg/mod/github.com/lucas-clemente/quic-go@v0.21.2/internal/qtls/go118.go:5:13: cannot use "quic-go doesn't build on Go 1.18 yet." (untyped string constant) as int value in variable declaration

Comes from libp2p.
Correct, this repo isn't using the latest version of libp2p.

I ran into this error and initially logged the issue in the wrong repo.
After installing Go 1.17.13, the error went away.

@jsign I forked this repo and added a note to the repo's home page about the Go versions that are supported in https://github.com/jsolly/powergate/commit/bda20f6bbd0e55d23dcd41ac2e6ab9b14d5a6584
I noticed I on the PR page that it didn't have the option to choose my fork for a PR. What's the process for a contributor to open a PR?
I checked the Contributor doc and it is empty.

Thanks! I opened a new PR to update the home page with information about installing Go.
https://github.com/textileio/powergate/pull/893


Trusted client is incompatible when connect to vision pro

I am trying to connect my vision pro to my windows 11 pc for the first time, and I got this warning: Trusted client is incompatible! Expected protocol ID:<PHONE_NUMBER>061257063, found:<PHONE_NUMBER>858005457
It turns up I can't connect my vision pro to my pc.
My ALVR is at Nightly.2024.03.20

You need to use a nightly server for then nightly client build

I literally had the same issue. If it's the official ALVR from Testflight, grab the nightly from ALVR v21.0.0-dev00+nightly.2024.03.24.
After running the server, I still had that message. It went away after I restarted SteamVR from the ALVR gui

Hello, @zarik5 @dblunk88 @Atagoxx
Now, I am trying to connect my quest 3 ( ALVR ) to PC (Linux), but its showing
Trusted client is incompatible! Expected protocol ID:<PHONE_NUMBER>4514957802, found:<PHONE_NUMBER>061257063
I am using "ALVR streamer v21.0.0-dev01+nightly.2024.08.21" in PC and "ALVR v20.10.0"
I tried previous versions of ALVR streams in PC, but no use. I thought of  using new updated ALVR in quest3, but its already updated.
Can someone help me here to solve this, Thanks !

@Kacchan16 you must always use compatible versions of server and client. in paricular, first number should match (v20 or v21) and -devXX number (or omitted).

Hi @zarik5
Thanks for your quick response, I tried with " ALVR v20.0.0-dev15+nightly.2023.06.01" just now. But when I use this old version, Its just not connecting to SteamVR and not detecting the headset.

Is there any way to install v21 in quest ??

I said you have to use matching versions. v20.0.0-dev15+nightly.2023.06.01 and v20.10.0 do not match. matches the first number (v20) but not the -devXX version. the client version has no -devXX

Thanks @zarik5
just for reference, can you tell me how to install the launcher in quest

The launcher is a pc app. You can find it in the usual downloads page


chore: initial version

This PR pushes the the initial version of the ARK messenger core to the repository.

Thanks for submitting this pull request! A maintainer will review this in the next few days and explicitly select labels so you know what's going on.
If no reviewer appears after a week, a reminder will be sent out.

Your pull request has been merged but was not assigned a bounty tier. A maintainer will assign a bounty tier to this pull request in the next few days.


JDBC connection to TDS

Hi
We are trying to configure a JDBC to access TDS
Which version of MS JDBC is needed?
What additional library are required?
Is there any sample JDBC connectionstring out there?

@thuanvan  We only support the interfaces as described in our TDS documentation.
@RichdiMSFT Any other comments?

@thuanvan I am waiting on internal feedback from our product team in case they have more to add.

@thuanvan We only support the interfaces as described in our TDS documentation. I have not been able to get any additional information from the product team.


Cook, OR

Looks like this is for Crook county, not Coos.

You're absolutely correct, thanks for the catch


Create Pull Request templates

To include some manual accessibility checks

After investigation, having multiple pull request templates is not advantageous, either in terms of usability or configuration. This issue  will be resolvable using one pull request template that covers all requirements as specific checks for a code change are minimal


Should we add a rust-toolchain.toml to pin the toolchain to the MSRV

This

You'll have to raise MSRV to 1.69.0 to use CStr::from_byte_with_nul.

makes me realize that our contributors are probably not using the same toolchain as our CI environment, which may confuse them a little bit when encountering CI failures that didn't happen on their local machine.
If we pin Nix's toolchain to the MSRC and keep it in sync with our CI,  our contributors can find the issue early and fix it, which would make our contributing experience better

I'm inclined against it.  We ought to work with every rust version, and the rust-toolchain.toml file might end up locking us down.  Plus, the Redox build won't even work on MSRV.

We ought to work with every rust version, and the rust-toolchain.toml file might end up locking us down. Plus, the Redox build won't even work on MSRV.

Get it, makes sense to me, close this issue


update custom PA kernel with support for fp8 kv cache dtype

custom PA kernel support added for fp8 kv cache dtype.
Change custom PA partition size to 512 to prefer throughput scenarios at cost of latency.
Unit Tested for f16 and fp8 kv cache, num_qheads=64 num_kvheads=8 only.
e2e tested on mlperf offline benchmark.
Unit tests to be updated in separate PR.

Have updated this on top of latest main and also enabled FP8 KV cache on BF16.

@mawong-amd I'll take a quick look.


New version: Tencent.QQMusic version 19.37

[X] Have you signed the Contributor License Agreement?
[X] Have you checked that there aren't other open pull requests for the same manifest update/change?

[ ] Have you tested your manifest locally with winget install --manifest <path>?
[X] Does your manifest conform to the 1.4 schema?

Microsoft Reviewers: codeflow:open?pullrequest=https://github.com/microsoft/winget-pkgs/pull/110861&drop=dogfoodAlpha

@wingetbot waivers Add Validation-Domain

@wingetbot run


[Fatal Error] netty-codec-socks-4.1.52.Final.pom:2:1: Content is not allowed in prolog.

Is there an existing issue for this?

[X] I have searched the existing issues
[X] I have read the guide to filing a bug

Steps to reproduce
Create a new flutter project
Try to run it using an emulator made by Android studio
Operating System: Windows
Development Environment: VS Code (Project Editing), Android Studio (Emulator)
Flutter version: 3.16.3
Gradle version: (not provided)
Installation: Flutter official repository
Network: Wi-FiStudio
Expected results
I want it to be properly running so that I can see it on the emulator
Actual results
it gave me that error
Launching lib\main.dart on sdk gphone64 x86 64 in debug mode...
[Fatal Error] netty-codec-socks-4.1.52.Final.pom:2:1: Content is not allowed in prolog.
FAILURE: Build failed with an exception.

What went wrong:
A problem occurred configuring root project 'android'.

Could not resolve all files for configuration ':classpath'.
Could not resolve io.netty:netty-codec-socks:4.1.52.Final.
Required by:
project : > com.android.tools.build:gradle:7.3.0 > io.grpc:grpc-netty:1.39.0 > io.netty:netty-handler-proxy:4.1.52.Final
Could not resolve io.netty:netty-codec-socks:4.1.52.Final.
Could not parse POM https://repo.maven.apache.org/maven2/io/netty/netty-codec-socks/4.1.52.Final/netty-codec-socks-4.1.52.Final.pom
Content is not allowed in prolog.

Try:

Run with --stacktrace option to get the stack trace.
Run with --info or --debug option to get more log output.
Run with --scan to get full insights.

Get more help at https://help.gradle.org

BUILD FAILED in 29s
Running Gradle task 'assembleDebug'...                             31.3s
Exception: Gradle task assembleDebug failed with exit code 1
firstly I just thought that there should be some error in my code but then I even tried it with a newly created flutter project.
Code sample
Code sample
[Paste your code here]

Screenshots or Video

Screenshots / Video demonstration
[Upload media here]

Logs
Logs
[Paste your logs here]

Flutter Doctor output
Doctor summary (to see all details, run flutter doctor -v):
[√] Flutter (Channel stable, 3.16.3, on Microsoft Windows [Version 10.0.22000.2538], locale en-US)
[√] Windows Version (Installed version of Windows is version 10 or higher)
[√] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
[√] Chrome - develop for the web
[!] Visual Studio - develop Windows apps (Visual Studio Build Tools 2019 16.11.32)
X The current Visual Studio installation is incomplete.
Please use Visual Studio Installer to complete the installation or reinstall Visual Studio.
[√] Android Studio (version 2023.1)
[√] VS Code (version 1.85.0)
[√] Connected device (4 available)
[√] Network resources
! Doctor found issues in 1 category.

@AhsanAli24117
What's the gradle version you are using in the project ?
If you try to run the same project from outside of IDE (ie, from command prompt), do you still get same error ?


Is there an interface to request access and get user location?

https://facebook.github.io/react-native/docs/geolocation.html

Please use the provided geolocation APIs from RN. I think we've decided that we will not be adding it to the component directly.
Thanks!

@lelandrichardson Can you explain why? It seem to be strange that we can see user position, but do not have an access to its coords. The main problem is that RNs navigator does not work properly, and RNs coords are different from coords of this dot. I just want to understand why it is impossible. I think that it is very common case - to open the map on region with user location. Thank you!


Logical Operators: Add math.logicalNot operation

This PR adds logicalNot operator for NDArrays which can be used as math.logicalNot(a).
@dsmilkov I need some inputs on this. Since this method will take a single input, should I make use of UnaryInputConfig and UnaryNode or make one for LogicalUnaryNode?

This change is

You can just use UnaryNode for this, even though it won't live in unary_ops.

@nsthorat Ready for review

Awesome work! Few comments inline.

Review status: 0 of 9 files reviewed at latest revision, 5 unresolved discussions.

src/math/logical_ops.ts, line 28 at r2 (raw file):

export class Ops {
  /**
   * Returns the truth value of NOT a element-wise.

remove "a"

src/math/logical_ops.ts, line 30 at r2 (raw file):

   * Returns the truth value of NOT a element-wise.
   *
   * @param a x The input NDArray.

s/a x/x

src/math/logical_ops.ts, line 33 at r2 (raw file):

   */
  @operation
  static logicalNot<T extends NDArray>(x: T): T {

question: since I came to the logical stuff a little late (daniel has been reviewing), what happens if the dtype passed is not boolean?

src/math/unary_ops_test.ts, line 1279 at r2 (raw file):

}

// LogicalNot:

can you move this to logical_ops_test?

src/math/unary_ops_test.ts, line 1284 at r2 (raw file):

  const tests: MathTests = it => {
    // Array1D:

remove these comment

Comments from Reviewable

@nsthorat

remove "a"

Done

s/a x/x

Done

question: since I came to the logical stuff a little late (daniel has been reviewing), what happens if the dtype passed is not boolean?

We've been throwing an error for this. I've added the assert statement for logicalNot as well.

can you move this to logical_ops_test?

Done

remove these comment

Done

Reviewed 7 of 10 files at r2, 3 of 3 files at r3.
Review status: all files reviewed at latest revision, 5 unresolved discussions, some commit checks failed.

Comments from Reviewable


[BUG]sozo typescript-v2 bindings errors

Describe the bug
Source files generated by sozo --typescript and sozo --typescript-v2 have errors.
To Reproduce

clone pistols
run: cd dojo && ./migrate dev
inspect dojo/bindings/typescript-v2/pistols.ts

Expected behavior
Generated files should not have any errors.
Screenshots
Attached generated source:
pistols.ts.txt
Those are the issues:

Import errors:

Module '"@dojoengine/torii-client"' has no exported member 'Client'. ts(2305)
Module '"@dojoengine/torii-client"' has no exported member 'ModelClause'. ts(2305)
Module '"@dojoengine/torii-client"' has no exported member 'valueToToriiValueAndOperator'. ts(2305)

import {
    Clause,
    Client,
    ModelClause,
    createClient,
    valueToToriiValueAndOperator,
} from "@dojoengine/torii-client";

Function arguments: Cannot find name 'props'.ts(2304)

This happens many times, always when a struct is passed. A couple of examples...
    async charge(payer: string, payment: Payment): Promise<void> {
        try {
            await this.execute("charge", [payer,
                props.payment.key,
                    props.payment.amount,
                    props.payment.client_percent,
                    props.payment.ranking_percent,
                    props.payment.owner_percent,
                    props.payment.pool_percent,
                    props.payment.treasury_percent])
        } catch (error) {
            console.error("Error executing charge:", error);
            throw error;
        }
    }

    async getTokenDescription(token_id: U256): Promise<void> {
        try {
            await this.execute("get_token_description", [props.token_id.low,
                    props.token_id.high])
        } catch (error) {
            console.error("Error executing getTokenDescription:", error);
            throw error;
        }
    }

Duplicated functions Duplicate function implementation.ts(2393)

Apparently, the generator is creating camelCase versions of functions that already have a camelCase declared I the interface, or functions that do not need camelCase (like name())
class DuelTokenCalls extends BaseCalls {
  // ...
    async ownerOf(token_id: U256): Promise<void> {
        try {
            await this.execute("owner_of", [props.token_id.low,
                    props.token_id.high])
        } catch (error) {
            console.error("Error executing ownerOf:", error);
            throw error;
        }
    }

    async name(): Promise<void> {
        try {
            await this.execute("_name", [])
        } catch (error) {
            console.error("Error executing name:", error);
            throw error;
        }
    }

the createClient() call is outdated: Expected 1 arguments, but got 2.ts. (2554)

        this.toriiPromise = createClient([], {
            rpcUrl: this.rpcUrl,
            toriiUrl: this.toriiUrl,
            worldAddress: this.worldAddress,
            relayUrl: this.relayUrl,
        });

closing this in favour of #2628 (--typescript-v2 is not being supported)


Introduce Contributor Covenant Code of Conduct

This pull request adds the Contributor Covenant Code of Conduct to the project, providing guidelines for collaboration. Grischa Erbe, the project's sponsor, is specified as the point of contact for any Code of Conduct issues.
By introducing this Code of Conduct, we aim to ensure a more inclusive and respectful environment for all contributors.
I believe no changeset is needed.

Hi @eerkmen,
as the linked website suggests, we, the Threlte maintainer team, didn't take it lightly to subscribe ourselves to these guidelines. While we think they are maybe a tad too formal and "disciplinary" for the current size of the Threlte community, we want to move forward and with growth these problems and headaches will probably rise. We acknowledge that this is kind of an OSS standard, so we will implement and enforce these guidelines.
Thank you very much for your contribution!


Output to RTMP?

Hi! I'm reading through the docs here, but I haven't come across a clear answer for my admittedly odd scenario. Can you verify for me if this is possible, and give me a nudge in the right direction?
I currently use Jellyfin to visualize my media library. I build playlists and then use ffplayout to stream those playlists to an rtmp server (in this case, Peertube, but the same basic idea would apply to Twitch, Youtube Live, Facebook Live, etc.)
Is it possible for me to use ErsatzTV for this, instead of building my playlists by hand? Perhaps removing ffplayout from the mix entirely? Or am I barking up entirely the wrong tree?

ErsatzTV does not support RTMP at this time. The following streams are the only ones available for this application; MPEG-TS, MPEG-TS (Legacy), HLS Direct, HLS Segmenter. RTMP is an old format that is rarely used in this day and age.

Thank you for the clarification. I'll see if there's a way for me to do what I need with HLS.


The coverage doesn't work stand alone

If we want the coverage we need start simplecov in our spec_helper like doing by example in dm-mapper ( https://github.com/datamapper/dm-mapper/blob/master/spec/spec_helper.rb#L6 )
It can be really great to put this information in devtools and insert it on spec_helper.rb by example.

I'm closing this issue as its really old and the scope of this tool changed meanwhile.


undefined symbol: _ZGVbN4v___expf_finite

I have problems using owl from the toplevel (also with utop). Both when compiling owl from source, and when installing via opam, I get the following error message:
# #require "owl";;
/home/pkl/.opam/4.04.0/lib/ocaml/str.cma: loaded
/home/pkl/.opam/4.04.0/lib/ocaml/unix.cma: loaded
/home/pkl/.opam/4.04.0/lib/ocaml/bigarray.cma: loaded
/home/pkl/.opam/4.04.0/lib/bytes: added to search path
/home/pkl/.opam/4.04.0/lib/integers: added to search path
/home/pkl/.opam/4.04.0/lib/integers/integers.cma: loaded
/home/pkl/.opam/4.04.0/lib/integers/integer_printers.cma: loaded
/home/pkl/.opam/4.04.0/lib/ctypes: added to search path
/home/pkl/.opam/4.04.0/lib/ctypes/ctypes.cma: loaded
/home/pkl/.opam/4.04.0/lib/ctypes/ctypes-top.cma: loaded
/home/pkl/.opam/4.04.0/lib/ctypes/cstubs.cma: loaded
/home/pkl/.opam/4.04.0/lib/plplot: added to search path
/home/pkl/.opam/4.04.0/lib/plplot/plplot.cma: loaded
/home/pkl/.opam/4.04.0/lib/dolog: added to search path
/home/pkl/.opam/4.04.0/lib/dolog/dolog.cma: loaded
/home/pkl/.opam/4.04.0/lib/eigen: added to search path
/home/pkl/.opam/4.04.0/lib/eigen/eigen.cma: loaded
/home/pkl/.opam/4.04.0/lib/gsl: added to search path
/home/pkl/.opam/4.04.0/lib/gsl/gsl.cma: loaded
/home/pkl/.opam/4.04.0/lib/owl: added to search path
/home/pkl/.opam/4.04.0/lib/owl/owl.cma: loaded
Cannot load required shared library dllowl_stubs.
Reason: /home/pkl/.opam/4.04.0/lib/stublibs/dllowl_stubs.so: /home/pkl/.opam/4.04.0/lib/stublibs/dllowl_stubs.so: undefined symbol: _ZGVbN4v___expf_finite.

I am running Debian stretch on my machine:
Linux phi 4.9.0-3-amd64 #1 SMP Debian 4.9.30-2+deb9u2 (2017-06-26) x86_64 GNU/Linux

@pklehre thanks for reporting this. I will have a look into this issue later this week.

I tried compiling openblas from source rather than using the Debian package, but I get the same error message.
Could it be related to glibc?
https://git.amper.me/open-source/glibc/commit/977a30801f61b7bbc27e8f185c1e7eb49675c60c?w=1#a8a3714871bdc09d9bfcd3e07b20f1dad7b041d7

I was also getting same error in OpenSuse and Debian distributions. However I am not sure if it is related to distribution.
I think problem seem to be that math lib is not being linked, so adding -lm to  CCLib+: fix this.
To test it clone fresh repo , edit _oasis file to add -lm and then run make oasis followed by make.
Hope this helps.

Yes, thanks. I can now use owl from the toplevel.

Worked for me as well but on the master version, not on the opam version.

With the switch from oasis to jbuilder, this bug seems to have reappeared.
It can be fixed by adding -m to c_library_flags in src/owl/jbuild

Many thanks @pklehre for reporting this, I have fixed the issue.


Add sq, lq, c_ldq

Hello, @herumi!
This small PR adds sq, lq, c_ldq.
Thank you in advance.

Hello, @herumi !
Are you planning to merge this PR. I think this functionality might prove useful.
Could you please review?
Thanks in advance.

I have so many tasks, but I am not feeling well. I will do it little by little, so please wait for me.

Thank you for the patch. But sq and lq are for RV128, which is working in progress, and riscv64-linux-gnu-as seems not to support those, so I won't append them until it supports them.
% riscv64-linux-gnu-as -march=rv128 a.s
Assembler messages:
Error: rv128: ISA string must begin with rv32 or rv64
a.s:1: Error: unrecognized opcode `lq x3,512(sp)'

% riscv64-linux-gnu-as -v
GNU assembler version 2.40 (riscv64-linux-gnu) using BFD version (GNU Binutils for Ubuntu) 2.40

@herumi what about adding them under some ifdef for future use?

If the correctness of the code can be verified with Binutils, it is acceptable to add it with ifdef. Do you know of any version to support them?

@herumi unfortunately, I couldn't find the proper toolchain to support them. So apparently, we should leave them be, until support is available. Thank you for your feedback!

Thank you for the PR. I've modified it a little and merged it.
But if you have a list of instructions you want, I'll support it step by step. It will reduce the costs that you make a PR and that I check it. How about it?

Okay, I'll close this PR now, and will support it later.
By the way, if you have a list of instructions you want, I'll support it step by step. It will reduce the costs that you make a PR and that I check it. How about it?

Hello, @herumi. Sorry for long answer. Thank you for your proposition, that sounds good.


Python polib fixes

Motivation for this change
Things done

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

[X] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[X] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[X] Tested execution of all binary files (usually in ./result/bin/)
[X] Fits CONTRIBUTING.md.

I've pushed cba8244 to master and 18.03. The other change can eventually also go in master.


Update mountinfo to support FreeBSD

This fixes https://github.com/containerd/nerdctl/issues/868

Now that the new moby/sys version with @Red54's code is released, we can bump it.
I created #1093 for this, which also re-enables CI, fixes docs, etc. I hope you don't mind @Red54 and thank you so much for the fix!

Now that the new moby/sys version with @Red54's code is released, we can bump it.
I created #1093 for this, which also re-enables CI, fixes docs, etc. I hope you don't mind @Red54 and thank you so much for the fix!

Thanks, let me close this in favor of #1093


Dead lock on non recursive mutex. may be can be used m_dynamicData.timePoint.getTime_Unix() directly.

https://github.com/alexa/avs-device-sdk/blob/f2dab7ee26e59caa41fa861e04df256fa8563ef1/capabilities/Alerts/acsdkAlerts/src/Alert.cpp#L824

Hello @alexismailov2 thanks for posting.
Thanks for bringing this bug to our attention, we will investigate this internally.

This was fixed in 1.25. The printDiagnostic() function now calls non-locking versions of those methods.


jadeite3.2pre-release7 ... project editor ... defaultSymbolDictName should be UserGlobals ...

... are you restricting the values for the packageConvention, packageFormat, and repositroyType fields ... I can supply you with methods that enumerate the the accepted values for those fields and you can build a drop down or use the list to validate ...

I changed the default to UserGlobals.   @dalehenrich Currently the gui is not doing any validation but it could.  Please provide the valid values.


[Auth] logged user persist on all requests even from other clients

I noticed that after a user logs in, that user becomes logged in on other browsers, even on Postman !!.
My config:
authenticator: session
session: cookie
So to make reproduction of the issue easier I packaged a minimal Adonis app having the issue
yardstick.zip
Steps to reproduce:

npm install
./ace migration:run
npm run dev
Browse to http://localhost:3333
Create a user
Login
Browse/request to http://localhost:3333 from other browsers, postman or curl and notice the currentUser username on the top of the page

curl -X GET -H "Cache-Control: no-cache" "http://localhost:3333"

Hey @helnokaly !
Thanks for your report! @thetutlage and myself will look into it soon!

It happened even from different domains as well. But change session driver to file may solve this. Not sure why.

Thanks for your contribution @kuntapong !
I have tried to debug it this morning but I can't figure out how this is possible (at the moment). The weird thing is I have a application build with Adonis running in production without this problem (https://www.qualitas-helvetica.ch/).
Going to debug deeper because this is a big issue.

@helnokaly Can u please confirm by installing adonis-framework from github? Later I will publish it to npm.

I think I may have filed a duplicate issue here: https://github.com/adonisjs/adonis-auth/issues/22

Also, that commit seems to break auth entirely. I used npm to grab the latest commit from GitHub and now it... look slike the CSRF token is stripped before the auth middleware can validate it?
Error: csrf secret missing
    at Shield._validateCsrf (/home/tyler/workspace/work/noc_dashboard/node_modules/adonis-middleware/src/Shield/index.js:162:21)
    at Shield.handle (/home/tyler/workspace/work/noc_dashboard/node_modules/adonis-middleware/src/Shield/index.js:248:12)
    at handle.next (<anonymous>)
    at onFulfilled (/home/tyler/workspace/work/noc_dashboard/node_modules/co/index.js:65:19)
    at process._tickCallback (internal/process/next_tick.js:103:7)

   * @public
   */
  _validateCsrf (request, csrfSecret) {
    /**
     * @description throw error when unable to find csrf secret
     */
    if (!csrfSecret) {
      const error = new Error('csrf secret missing')
      error.status = 403
      error.code = 'EBADCSRFTOKEN'

Another stacktrace. I'm playing around on my fork.

@thetutlage Also, it doesn't seem like the auth middleware supports different session drivers? I've tried the Redis and File driver to back sessions and I get Error: CSRF secret missing when I attempt a login.
Works fine with cookies as the driver but then... everyone can login when a single user is logged in.

I slept and woke up and poked at it some more and my yowling was mostly my fault.
I got session based authentication working using the Redis driver. The problem I was having before was that the secure option was set to true (my bad) in config/session.js and that messed with the CSRF token being received by the backend when serving the website over HTTP. Since I was just running on localhost it looked like everything was broken since everything ran fine with the session being established via the cookie driver - when I switched to the latest code/used a different driver everything broke.
However, I revisited testing out your change @thetutlage and can verify that it works when the secure option for sessions is set to false. I sign in, access an auth middleware controlled route, open an incognito window, and I'm denied access like expected. Thanks for the commit!

Cool, will release it soon with some other changes

Published as 3.0.4 in adonis-framework

@thetutlage sorry for the late reply, I tested it an it's fixed but is there a possibility that concurrent requests may share the same session before it is cleared?

@helnokaly Seems correct. Since drivers are singleton this will cause issue with Cookie driver.
Let me add a concept of singleton and non-singleton drivers since drivers like Redis should not open a new connection on each request and should be singleton, whereas drivers like Cookie should have a new instance tied to each request.
Thanks for bringing this up

@helnokaly What about something like this
https://github.com/adonisjs/adonis-framework/commit/c784fbb5a70d189957e27885ec49fc6d4e7f8159

@thetutlage looks good and reasonable to me, thanks.
I'm curious if there is a big performance hit with one instance per request approach.

Nope as soon as request finishes GC will clear it

Great :+1:


Hints Display overflow and spam

If you enter a very long hint it causes the hint display box to not expand correctly or wrap the text.
also noticed that hint_table is included more then once on the page, I'm up to about 120 challenges and its not uncommon for the entire page to start hard locking in chrome or firefox as a admin.

Also should note that adding hints is a little bit of a process, it seems like it was built for more using a hint for more then one challenge.

@JRWR, this is an easy fix, and I can implement it for the next release.  Also, you're absolutely correct... it was built with using a hint for more than one challenge in mind, but not as the primary use case.  We had several challenges that didn't have any hints, so we created one "hint" that said there weren't any hints and didn't deduct any points, then re-used it everywhere.  The implementation could be better for sure, I just honestly haven't put that much thought into it.  Any suggestions?

@JRWR Having trouble replicating.  Which browser are you seeing the overflow in?

Google Chrome is up to date
Version 64.0.3282.167 (Official Build) (64-bit)
Win10
On Fri, Feb 16, 2018 at 6:28 AM, Paul Jordan<EMAIL_ADDRESS>wrote:

@JRWR https://github.com/jrwr Having trouble replicating. Which browser
are you seeing the overflow in?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/mcpa-stlouis/hack-the-arch/issues/42#issuecomment-366211636,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAxE1Ccw0QT8JYJx3UGjhrwnBaU1Jejiks5tVWZrgaJpZM4SG9nL
.

Yeah, I got it.  Sorry.  That part is fixed.  I might need to spend some time thinking about how to not include the hint table for each challenge.  Currently, the buttons are hard coded to each challenge, so when you click "Add" on the hint, it goes to the challenge from which you opened the modal.  I'm looking at turning on fragment caching for your #43, which may fix it, but isn't ideal.

So simplify the input of hints where in the hints admin its just a open
form fields for Hint Text and a ajax submission, load the form only when
the admin switches to the hint tab for that challenge, keep it where you
don't need to go off page to add a hint. quick and easy :)
Also, I noticed it doesn't show how many points are needed for a hint to
the end user
On Fri, Feb 16, 2018 at 6:40 AM, Forrest Fuqua<EMAIL_ADDRESS>wrote:

Google Chrome is up to date
Version 64.0.3282.167 (Official Build) (64-bit)
Win10
On Fri, Feb 16, 2018 at 6:28 AM, Paul Jordan<EMAIL_ADDRESS>wrote:

@JRWR https://github.com/jrwr Having trouble replicating. Which
browser are you seeing the overflow in?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/mcpa-stlouis/hack-the-arch/issues/42#issuecomment-366211636,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAxE1Ccw0QT8JYJx3UGjhrwnBaU1Jejiks5tVWZrgaJpZM4SG9nL
.

Last commit actually was for #44.  Sorry for the confusion.


added OAuth callout

Wanted to call out OAuth as part of App Permissions under Manage App Permissions for customers looking for this capability
https://docs.microsoft.com/en-us/cloud-app-security/manage-app-permissions

@computeronix : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@computeronix : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Updated to OAuth apps


logger: align methods with Logger

Use the method signatures of Logger
for the NoopLogger.

There's no advanced logging going on so the mis-alignment probably doesn't matter, but I think this will improve the situation for a type checker.


Learning about object storage

Background
We learned that the NFS storage using the Amazon EFS service is very costly for just a few TB of storage, and that we want to make use of S3 Object Storage instead that is ~10-20 times cheaper.
Object storage learning goals
There may be plenty of things to learn and document about these, but for now I've just created a single issue listing some learning goals about working with object storage in general and object storage on AWS.

[ ] Understand how to access our s3 object storage bucket that we reference as the "scratch bucket"
You can use the aws s3 command, for example aws s3 cp <source> target> and you will copy something from one location to another, where a location can be a path on the local file system or it can be a location in object storage such as s3://jmte-scratch/consideratio which is what my SCRATCH_BUCKET environment variable evaluates to, while yours will be s3://jmte-scratch/<your-username>.
[ ] Understand sensible practices for bucket to bucket transfers
[ ] Understand possibilities of mounting buckets to the file system
[ ] Understand costs of S3 object storage
Some technical details: https://aws.amazon.com/s3/pricing/
What kind of s3 bucket storage are we currently allocating in our s3 scratch bucket?
[ ] Understand how we can use ephemeral storage in /tmp
I think we can't download more than ~80 GB per node for now, but that we can increase this by updating our machine configuration.

We've received amazing feedback from the Pangeo Cloud ops working group as can be seen in the notes from the meeting 14th June.
https://docs.google.com/document/d/1I-2VNNHoAjjeYvlCezQhFLmiu2OevqGDS5nUAP-6Hfw/edit#


accuracy changes when i scale the flame object

hi,
so i was experimenting, i scaled the flame object by different factors(scaling the vertices by a constant, in the .obj file), and the loss changes, so is there some scale factor which can give lesser loss, and is it different for man and woman?

also i observed, that if i print loss at every iteration, loss at ith iteration(<200) is sometimes lesser than the loss at 200th iteration, any opinion on this?
thanks


fix: remove extra line in dist/index.d.ts

Initial checks

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same update/change?
[x] Have you linked an issue to this pull request? (Create one if it does not exist)
[x] Have you used Conventional Commits format?

Bug
https://github.com/oas-tools/oas-commons/issues/10
Description
Removes extra line in dist/index.d.ts
Generate valid dist/index.d.ts
Implementation details

PR rebased and changed to add the line:
export * from '../typings'


[Collapse] Add orientation and horizontal support

Deprecate collapsedHeight property

(#10051)

[x] I have followed (at least) the PR section of the contributing guide.

One thing I am not sure about - how do we handle props deprecations? The yarn proptypes is not adding the deprecatedPropType util :thinking:

Oh and another thing - I am not able to run tests because of following error:
$ yarn test:unit --grep Collapse
...
✖ ERROR: No valid exports main found for '/home/darkowic/projects/github/material-ui/node_modules/dom-accessibility-api'

There are still 2 things I am not sure about the collapse:

Why this component needs 2 wrappers?
IMHO the wrapper should set width and height to 100% to fill the whole root space:

after setting the wrapper's height to 100%:

Oh and another thing - I am not able to run tests because of following error:

What OS are you using and what's the output of node -v?

Oh and another thing - I am not able to run tests because of following error:

What OS are you using and what's the output of node -v?

@eps1lon
$ uname -a
Linux pop-os 5.3.0-7648-generic #41~1586789791~19.10~9593806-Ubuntu SMP Mon Apr 13 17:50:40 UTC  x86_64 x86_64 x86_64 GNU/Linux
$ node -v
v13.5.0

@oliviertassinari I see you added v5 label - when do you think we can release it?. Why not release it in some 4.x version? I would love to use it (I use it now - I copied the whole component into my source). I tried to not introduce any breaking changes.

@darkowic We are freezing all new features from the v4 branch.

😒 Well, then I'm waiting for v5-alpha.1 🚀

@darkowic You can target the next branch with this.

Updated ;) One unit tests is failing:
  it('can fallback to findDOMNode in a StrictMode theme', () => {

with TypeError: Cannot read property 'style' of null when trying to get wrapperRef.current and some browser tests are failing with similar error - any tips?

Marked as breaking in anticipation of the removal of collapsedHeight...

btw yarn prettier command from contribution guide doesn't work

@darkowic What error are you getting?

@mbrookes
[darkowic@pop-os] ~/projects/github/material-ui (10051_horizontal_collapse)
$ yarn prettier
yarn run v1.22.4
$ node ./scripts/prettier.js
prettier.js [mode]

formats codebase

Positionals:
  mode  "write" | "check-changed" | "write-changed"
                                             [string] [default: "write-changed"]

Options:
  --help  Show help                                                    [boolean]

Error: Command failed: git rev-parse next
fatal: ambiguous argument 'next': unknown revision or path not in the working tree.
Use '--' to separate paths from revisions, like this:
'git <command> [<revision>...] -- [<file>...]'

    at ChildProcess.exithandler (child_process.js:303:12)
    at ChildProcess.emit (events.js:315:20)
    at maybeClose (internal/child_process.js:1051:16)
    at Socket.<anonymous> (internal/child_process.js:442:11)
    at Socket.emit (events.js:315:20)
    at Pipe.<anonymous> (net.js:670:12) {
  killed: false,
  code: 128,
  signal: null,
  cmd: 'git rev-parse next',
  stdout: 'next\n',
  stderr: "fatal: ambiguous argument 'next': unknown revision or path not in the working tree.\n" +
    "Use '--' to separate paths from revisions, like this:\n" +
    "'git <command> [<revision>...] -- [<file>...]'\n"
}
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

Fortunately, yarn prettier:all works ;)

Looks like you don't have a local next branch for listChangedFiles() to compare with. (Which pretter:all doesn't require.) Should probably have a defensive check...

Something's off with the horizontal collapse - it slides to the right rather than collapsing to the left when collapsedSize is set.

@darkowic Fetch to upstream remote again i.e. git fetch upstream if upstream is pointing to https://github.com/mui-org/material-ui.

Something's off with the horizontal collapse - it slides to the right rather than collapsing to the left when collapsedSize is set.

@mbrookes hmm how it should work? Do we want to add some prop that defines which direction it is animated?

It should behave the same as without collapsedSize, but stop collapsing at collapsedSize width (as it does with vertical).

@darkowic Could you update this PR please with the latest version of the next branch?

Yes, I will do it together with the fix/solution for collapse animation orientation

Rebased and improved demo. Actually it was looking weird because of space-between ;)
:ship: :tada:

@darkowic Great work, thanks!

Hi, I've been experimenting with this and I have found that the <Collapse /> component takes up the whole horizontal space with this structure:
<Grid container>
  <Collapse orientation="horizontal" in={showSidebar}>
    <Grid item xs={4}>
      <Sidebar />
    </Grid>
  </Collapse>

  <Grid item xs>
    <ResultList />
  </Grid>
</Grid>

In this case the Collapse component, when in = true, renders as 100% the width of the <Grid container>. I'm not actually sure that I'm using Grid correctly... But when I use any other Transition this is NOT the case. For example, replacing Collapse with
<Slide direction="right" unmountOnExit in={showSidebar}>
  ...
</Slide>

Then the layout behaves correctly (in that the slide component renders 'transparently', with the child Grid component sizing working as expected).
Looking at the HTML.
The Slide component gets classNames associated with the Grid (which I assume is the children?):

Whereas with the Collapse component, unlike with the Slide, no inline styles are giving, and the classes don't include any classes from the children:

...

Is this intended?
Rewriting my JSX:
<Grid container>
  <Collapse
    timeout={3000}
    className={clsx({
      'MuiGrid-root': true,
      'MuiGrid-item': true,
      'MuiGrid-grid-xs-4': showSidebar ? true : false,
    })}
    orientation="horizontal"
    in={showSidebar}
  >
    <Grid item xs={showSidebar ? 12 : 4}>
      <Sidebar />
    </Grid>
  </Collapse>

  <Grid item xs>
    <ResultList />
  </Grid>
</Grid>

This somewhat achieves what I'm looking for but in a very jumpy, unsatisfactory way (the timout={3000} allows me to see where the problems are).
I apologise if I'm misusing the Transition component in this case! If so, I would be super grateful for pointers in the correct direction.

@zachsa  Please open a new issue and fill out the template. PRs are not monitored for potential issues.

Thank you @eps1lon  - I've written up an issue here: https://github.com/mui-org/material-ui/issues/21971


Test

Link to reproduction
https://github.com/balazsorban44/nissuer
To reproduce
.
Which area(s) are affected?
No response

+1

+1

+1

Same issue!


RPM spec file should be in a separate repository

Hi,
I am mostly experienced in Debian packaging, but I think it's the same for RPM packaging.
One should never put the distro-specific packaging files in the upstream repository.
Why? Because the downstream specific packaging files might have different versions, and apply patches to the source. You don't want all these changes to be in the upstream sources.

Hmm, that makes sense for the cross-platform apps indeed. In this particular app we are building a SailfishOS-specific app for the Sailfish OS devices only. Still storing packaging in a separate repository could probably make sense if the current directory structure wasn't a part of the default Sailfish build process.
I am not sure all the script would work if packaging scripts were in another dir. And if you checkout them to the current place via e.g. git subtree that wouldn't make much difference from the current situation, would it?


Resolves SignalSwitchVisualizer is Obsolete

About the PR

Splits the SignalSwitchVisualizer into a component/system pair.
TODO:

[ ] Testing

Media

[ ] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Generic visualizer

It's unused. Into the bin it goes.


Logging: Add credentials on the obje

Some users want to debug credentials on logs, so this adds credentials
object to the liquid object
Fix THREESCALE-5723
Signed-off-by: Eloy Coto<EMAIL_ADDRESS>

I'm not sure this is a good idea. It could lead to security issues.
I think that at least, this should be optional and off by default.

Well, context.credentials is shared to any other liquid policy no? So user can send credentials in any liquid policy, like headers :\

I think that case is a bit different because the user needs to say explicitly in the configuration of the policy that something needs to be done with the credentials. In this case, the credentials will be logged just by enabling the policy.

No, in this case, users need to enable the credentials on logs, will not be logged as default in any case.

Ah I see, I misunderstood :+1: