Cannot use global error handler for express routes

Description
I am using expressReceiver.app to add some global middleware and custom routes. When I am adding express error handler, it does not complain but when error is thrown it is not catched by the error middleware provided
What type of issue is this? (place an x in one of the [ ])

[ ] bug
[x] enhancement (feature request)
[x] question
[ ] documentation related
[ ] testing related
[x] discussion

Requirements (place an x in each of the [ ])

[x] I've read and understood the Contributing guidelines and have done my best effort to follow them.
[x] I've read and agree to the Code of Conduct.
[x] I've searched for any related issues and avoided creating a duplicate issue.

Reproducible in:
package version: 2.0.1
node version: 12.7.0
OS version(s): Windows 10
Steps to reproduce:

I am using expressReceiver in the index.ts file and adding my custom error middleware
index.ts code

// eslint-disable-next-line @typescript-eslint/triple-slash-reference
/// <reference path="../node_modules/ts-nameof/ts-nameof.d.ts" />
import './bluebird';
import 'reflect-metadata';

import { config } from './config';
import { appContainer } from './inversify.config';
import { PluginManager } from './plugins.config';
import { routes } from './routes';
import { errorMiddleware } from './server/middlewares/error.middleware';
import { logger } from './server/utils/logger';
import { app, expressApp } from './server/utils/slack-bolt-client';

expressApp.use('/', routes);
expressApp.use(errorMiddleware);

(async (): Promise<void> => {
  const pluginManger = appContainer.get<PluginManager>(nameof<PluginManager>());
  // Initialize all plugins
  await pluginManger.initalizePlugin();
  // require('./bluebird');
  // Start the server
  await app.start(config.serviceConfig.port);
  logger.info(
    `⚡ Slack Bot app is running on port ${config.serviceConfig.port}`
  );
})();

Custom error middleware

import { NextFunction, Request, Response } from 'express';

import { config } from '../../config';
import { Environment } from '../models/environment';
import { ErrorResponse } from '../models/error';
import { logger } from '../utils/logger';

export const errorMiddleware = (
  error: ErrorResponse,
  req: Request,
  res: Response,
  // eslint-disable-next-line @typescript-eslint/no-unused-vars
  next: NextFunction
): void => {
  Error.captureStackTrace(error, errorMiddleware);
  const status = error.status || 500;
  const message = error.message || 'Something went wrong';
  logger.error({ error });
  res.status(status).send({
    requestId: error.requestId,
    status,
    message,
    timestamp: error.timestamp,
    source: error.source,
    errorData: error.errorData,
    code: error.code,
    stack:
      config.serviceConfig.environment !== Environment.PRODUCTION
        ? error.stack
        : undefined,
  });
};

Expected result:
When i throw an error in the controller, my custom error middleware should catch and respond back rather it is stuck in the call
Actual result:
When an error is thrown, it is doing nothing and even the client still has connection. I have to manually cancel the request.
Attachments:

Fixed the issue by adding import 'express-async-errors'; to the top of main file


Typo in docs

> Is a container for any well-formed JSON object. A default JsonasdasdaObject contains an empty JSON object.
Looks like JsonObject is spelt "JsonasdasdaObject" on this page

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 6254d9e0-d1d8-c945-0acb-3fd178506ac4
Version Independent ID: a42e3814-18be-ffeb-72cc-9952a781c7c6
Content: JsonObject Data Type - Business Central
Content Source: dev-itpro/developer/methods-auto/jsonobject/jsonobject-data-type.md
Service: dynamics365-business-central
GitHub Login: @solsen
Microsoft Alias: solsen

Thanks for your input, this is now fixed.


top-up valve not working

mod: 1.1.1
Factorio: 2.0.19
The valve does not allow fluids to pass through at any threshold

Is that a new valve you placed after the update? Or an pre-existing valve?
If this was a pre-existing valve then it should be fixed by f18b5b27fea81afc3ac7b32ec33eff4af4b20816, which I just released. Otherwise please tell me exact reproduction steps.
Thanks for the report!

I tried it on the new version 1.1.2. Nothing has changed. I deleted, changed the threshold in it. Bottom line: the top-up valve doesn't work.


Configure Renovate

Welcome to Renovate! This is an onboarding PR to help you understand and configure settings before regular Pull Requests begin.
🚦 To activate Renovate, merge this Pull Request. To disable Renovate, simply close this Pull Request unmerged.

Detected Package Files

Dockerfile (dockerfile)

Configuration Summary
Based on the default config's presets, Renovate will:

Start dependency updates only once this onboarding PR is merged
Enable Renovate Dependency Dashboard creation.
Use semantic commit type fix for dependencies and chore for all others if semantic commits are in use.
Ignore node_modules, bower_components, vendor and various test/tests directories.
Group known monorepo packages together.
Use curated list of recommended non-monorepo package groupings.
Apply crowd-sourced package replacement rules.
Apply crowd-sourced workarounds for known problems with packages.
Pin Docker digests.
Pin github-action digests.
Pin dependency versions for devDependencies.
Preset with best practices from the Renovate maintainers. Recommended for advanced users, who want to follow our best practices.

🔡 Would you like to change the way Renovate is upgrading your dependencies? Simply edit the renovate.json in this branch with your custom config and the list of Pull Requests in the "What to Expect" section below will be updated the next time Renovate runs.

What to Expect
With your current configuration, Renovate will create 2 Pull Requests:

Pin ubuntu Docker tag to 152dc04

Schedule: ["at any time"]
Branch name: renovate/pin-dependencies
Merge into: master
Upgrade ubuntu to sha256:152dc042452c496007f07ca9127571cb9c29697f42acbfad72324b2bb2e43c98

Update ubuntu Docker tag to v22

Schedule: ["at any time"]
Branch name: renovate/ubuntu-22.x
Merge into: master
Upgrade ubuntu to sha256:2b7412e6465c3c7fc5bb21d3e6f1917c167358449fecac8176c6e496e5c1f05f

❓ Got questions? Check out Renovate's Docs, particularly the Getting Started section.
If you need any further assistance then you can also request help here.

This PR has been generated by Renovate Bot.

Renovate is disabled
Renovate is disabled because there is no Renovate configuration file. To enable Renovate, you can either (a) change this PR's title to get a new onboarding PR, and merge the new onboarding PR, or (b) create a Renovate config file, and commit that file to your base branch.


OnSessionCancelled gets called right after OnSessionStarted

Hi there,
I'm building a JuiceSSH plugin, and like it says in the title, I get calls to OnSessionCancelled right after OnSessionStarted. looking at the file here I think line 286 is missing a break;?
I guess the reason this passed uncaught before is because in the example plugin and performance monitors use an empty OnSessionCancelled callback.

Hi @mewais,
You're absolutely right!

fallthrough in switch statements is a cause of bugs!

**Fallthrough in 'switch' statement less... (⌘F1)**
Reports 'fall-through' in a switch statement. Fall-through is defined to occur when a
series of executable statements after a switch label is not guaranteed to transfer
control before the next switch label. In that case, control falls through to the
statements after that switch label, even though the switch expression does not equal the
value of the fallen-through label. While occasionally intended, this construction is
confusing, and is often the result of a typo.

I've forked this repo and made some minor changes including a bugfix for this particular issue!!
https://github.com/1951FDG/juicessh-pluginlibrary
My fork is available as a JitPack repository
https://jitpack.io/#1951FDG/juicessh-pluginlibrary


IN-721 match existing data

Replaced the placeholder matching stuff with some real matching stuff, but I have had to comment out the actual updating (doesn't actually work) and the API tests (can never pass because no clients are updated).
Also added in some improvements to the skeleton data.
All comments and other nonsense will be rectified in part two - the updatening (IN-802)

superseded


[PREGEL] Tests for Pregel WCC

Tests to verify Pregel's WCC algorithm
JS producing vertices and edges for some graphs:

cycles of given length,
alternating cycles length: every second edge goes in the "wrong" direction,
full binary trees of given depth with edge going away from the root,
alternating full binary trees of given depth with all edges of every second level going in the "wrong" direction,
bidirected cliques of given size: all edges (v,w) and (w,v) for distinct vertices v and w,
linear cliques of given size: there is an edge (v,w) if v<w (vertices have indices)

Scope & Purpose
(Please describe the changes in this PR for reviewers, motivation, rationale - mandatory)

[ ] :hankey: Bugfix
[ ] :pizza: New feature
[ ] :fire: Performance improvement
[ ] :hammer: Refactoring/simplification

Checklist

[ ] Tests

[ ] Regression tests
[ ] C++ Unit tests
[ ] integration tests
[ ] resilience tests

[ ] :book: CHANGELOG entry made
[ ] :books: documentation written (release notes, API changes, ...)
[ ] Backports

[ ] Backport for 3.9: (Please link PR)
[ ] Backport for 3.8: (Please link PR)
[ ] Backport for 3.7: (Please link PR)

Related Information
(Please reference tickets / specification / other PRs etc)

[ ] Docs PR:
[ ] Enterprise PR:
[ ] GitHub issue / Jira ticket:
[ ] Design document:

bulk inserts from the verry beginning - I like!


There should be a simpler demo than the stars demo

There should be a demo of NetworkPolicy that has minimal moving parts.  This could be good for making the basic idea really clear, and for providing a simple use case for debugging and bug reports.

This is actually done now - see https://github.com/projectcalico/calico-containers/tree/master/docs/cni/kubernetes/simple-policy-demo


Set --metrics-max-age default value equal to relist interval

The--metrics-max-age flag has been introduced in #136 with a fixed default value of 20 minutes.
The current default value leads to the situation that when metrics drop out of Prometheus, they are still reported in the discovery information of the Custom Metrics API (/apis/custom.metrics.k8s.io/v1beta1) for 20 minutes.
This can be quite confusing if you don't know about the --metrics-max-age flag and expect the behaviour that is described in the docs. Namely that the Prometheus Adapter discovers all metrics that have been updated since the last discovery query (defined by the relist interval), which is also the reason the relist interval must be at least as large as the Prometheus scraping interval (see #229).
By setting the default value of --metrics-max-age equal to the relist interval, the expected behaviour as described in the docs is preserved, but you can still set --metrics-max-age to some other value if you know about it.
The other way to go would be to update the docs and expect all users to set the --metrics-max-age flag (and the Helm chart would need to be updated to use it too). But this would make the configuration of the Prometheus Adapter even more complicated than it already is. It is probably easier to just remember to set the relist interval at least as large as the Prometheus scraping interval, than to additionally have to figure out what the metrics-max-age is and what to set it to.

Despite the age of the PR, I think this is still reasonable, hence merging.


openjdk test jobs on test-packet-x64-windows-2012r2-1 failed for sha check

All openjdk test jobs on test-packet-x64-windows-2012r2-1 failed for jar sha check as this machine use window command instead of cygwin.  ( The jar sha check is part of the dependency job for functional tests. That is saying all other test categories, system, external, etc. , do not need this job. However we just haven't got the time to reconstruct the testframework to move the job to function specific. )
We have temporarily disabled the sha check on upstream openj9 project for windows without cygwin properly setting up. And will re-enable the check when  https://github.com/AdoptOpenJDK/openjdk-infrastructure/issues/378 is resolved.
https://ci.adoptopenjdk.net/view/Test_openjdk/job/openjdk8_j9_openjdktest_x86-64_windows/
@keithc-ca

Meanwhile I saw there is a new test windows available( may not be new, for over a month)
azure: win2012r2-x64-1: {ip: <IP_ADDRESS>, user: adoptopenjdk}
Which works as cygwin setting up properly. @gdams  any difference between azure and packet especially configuration with cygwin or jenkins?

I will try running tests on the azure machine to see if #407 has been addressed on this machine as looks like its cygwin setting up properly.

The PATH is different:
Packed:
PATH=/cygdrive/c/ProgramData/Oracle/Java/javapath:/cygdrive/c/windows/system32:/cygdrive/c/windows:/cygdrive/c/windows/System32/Wbem:/cygdrive/c/windows/System32/WindowsPowerShell/v1.0:/cygdrive/c/Program Files/Mellanox/MLNX_WinOF2/Management Tools:/cygdrive/c/Program Files/Mellanox/MLNX_WinOF2/Performance Tools:/cygdrive/c/Program Files/Mellanox/MLNX_WinOF2/Diagnostic Tools:/cygdrive/c/Program Files/Mellanox/MLNX_VPI/IB/Tools:/cygdrive/c/Program Files/Mellanox/MLNX_CIMProvider/lib/mft:/cygdrive/c/Strawberry/perl/bin:/cygdrive/c/Program Files/Java/jdk8u144-b01/bin:/cygdrive/c/apache-ant/apache-ant-1.10.1/bin:/cygdrive/c/Program Files (x86)/Windows Kits/8.1/Windows Performance Toolkit:/cygdrive/c/Program Files/Microsoft SQL Server/110/Tools/Binn:/cygdrive/c/Program Files (x86)/Microsoft SDKs/TypeScript/1.0:/cygdrive/c/Program Files/Microsoft SQL Server/120/Tools/Binn:/cygdrive/c/Program Files/Git/cmd:/usr/bin
azure:
PATH=/cygdrive/c/Program Files (x86)/Common Files/Oracle/Java/javapath:/usr/bin:/cygdrive/c/Windows/system32:/cygdrive/c/Windows:/cygdrive/c/Windows/System32/Wbem:/cygdrive/c/Windows/System32/WindowsPowerShell/v1.0:/cygdrive/c/Strawberry/bin:/cygdrive/c/Program Files/Git/cmd:/cygdrive/c/Program Files/Java/jdk8u172-b11/bin:/cygdrive/c/apache-ant/apache-ant-1.10.3/bin:/cygdrive/c/Program Files (x86)/Windows Kits/8.1/Windows Performance Toolkit:/cygdrive/c/Program Files/Microsoft SQL Server/110/Tools/Binn:/cygdrive/c/Program Files (x86)/Microsoft SDKs/TypeScript/1.0:/cygdrive/c/Program Files/Microsoft SQL Server/120/Tools/Binn


Added agesql python file

Changes

Added agesql.py file to combine all sql command used in python driver

Reason for changes

Improves re usability
If any command changes with future changes, we don't need to update in all the function.

I appreciate the effort, but having the SQL/Cypher commands written out is more readable for what is actually being sent. Separating this out into another file is unnecessary.


Automatically fetch specific page when an overridden method is called

This PR introduces a decorator for page methods called @overridable. When appended to a method, this decorator will insert a check to see if that method has been overridden in the subclass. If it's been overridden on the subclass, it'll call the sub-classes implementation with self.specific_deferred.
This behaviour is a bit magical, but I think it solves a common problem that we currently have with methods like get_admin_display_title where a developer might forget to call specific/specific_deferred before calling it (see my previous PR that solved a bunch of these, and also the couple of tests I had to fix in this PR).

An alternative to this could be to make it raise an exception if the decorated method was called on a non-specific Page object, so the developer is prompted to think about whether to use specific/specific_deferred.
This would be less magical, but I think one advantage this current implementation has is it will only fetch the specific object if the method has actually been overridden.

With this PR being quite old and very hard to rebase and without some kind of direction in an issue as to this feature being requested I will close this for now.
Thank you Karl for always bringing awesome ideas up to the forefront, maybe something like this will be picked up by others in the future.


@Environment(.dismiss) is not working correctly

isPresented state is not updated correctly when using @Environment(\.dismiss)

Could you provide some example of incorrect behavior? It would help me to fix it 😌

Here is minimal example of usage @Environment(\.dismiss):
struct ContentView: View {
    @State var isPresented = false

    var body: some View {
        Text("Open sheet")
            .onTapGesture {
                isPresented = true
            }
            .sheet(isPresented: $isPresented) {
                TestView()
            }
            .presentationDetents([ .large, .medium ])
    }
}

struct TestView: View {
    @Environment(\.dismiss) var dismiss

    var body: some View {
        Button("test", action: {
            dismiss()})
    }
}


Customizable tokenizer for RULER

Adding an optional environment variable TOKENIZER_MODEL which controls the tokenizer model to use for RULER data generation. With this option, the dataset length will be more precise when we evaluate models that do not use gpt-4 tokenizer.

Thanks for the review. The general workflow we use opencompass is via the CLI: opencompass --models [custom_model_config] --datasets ruler_4k_gen.py ... However, it seems that it is not possible to specify tokenizer for --datasets. Do you think this is a reasonable use case?

Thanks for the review. The general workflow we use opencompass is via the CLI: opencompass --models [custom_model_config] --datasets ruler_4k_gen.py ... However, it seems that it is not possible to specify tokenizer for --datasets. Do you think this is a reasonable use case?
That sounds like a reasonable usecase.


Project Logo

It would be nice if the project had a logo. When I was setting up the standalone helpdesk I quickly threw together this one https://github.com/django-helpdesk/django-helpdesk/blob/main/helpdesk/static/helpdesk/img/icon512.png I don't know if ya'll like it or not. In any case, I wanted to open up this ticket to track the concept of having a logo for standalone django-helpdesk.

We can use the logo for example for the icon for the github project and for the favicon in the standalone version.

IMO, it's a good idea to have a logo for the project, and your one does the job.
Yes, it could be improved to look more professional but for now, I think it's good! And if someone wants to help with the design, it could be easily swapped later.

To make a good social preview, Github provides a template :

@timthelion, do you think you can make a version of the logo using this? Maybe just writing django-helpdesk under the logo is enough. I think I can do it later, if ever.

Agree with above.

Where does the repo card get shown? I am not aware of that graphic.

It's the integrated picture that get displayed when you share the repo link on sites like Facebook, X, Discord, etc.

Who has admin access to this repo to change the icon?

I've seen that I'm able to do it. And I'm pretty sure @uhurusurfa can too.

Sorry, I tried changing it but it is the social preview image that I'm able to change.
The one you're referring to is in fact the avatar picture of the django-helpdesk group, and I don't have the rights to modify it.
Preview image looks like this on Discord for example:

Should we keep it or try creating one with better size (following recommendations of 1280×640px) ?

I can update the django-helpdesk group avatar image if there's agreement
from folks on what the new logo should be. Just tag me when the image is
ready to upload, in case you'd like to work on it some more.
On Mon, Dec 4, 2023 at 9:06 AM Benbb96 @.***> wrote:

Sorry, I tried changing it but it is the social preview image that I'm
able to change.
The one you're referring to is in fact the avatar picture of the django-helpdesk
group https://github.com/django-helpdesk, and I don't have the rights
to modify it.
Preview image looks like this on Discord for example:
image.png (view on web)
https://github.com/django-helpdesk/django-helpdesk/assets/15046586/1e1e3af5-e1f6-4ef4-aed0-cb48ec00ab6a
Should we keep it or try creating one with better size (following
recommendations of 1280×640px) ?
—
Reply to this email directly, view it on GitHub
https://github.com/django-helpdesk/django-helpdesk/issues/1129#issuecomment-1838710100,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABEKHU73MJRZXF3WE4U7EATYHXKFDAVCNFSM6AAAAAA6PTPIZOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMZYG4YTAMJQGA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Seems like this is all done then :)


HexBug not running after deletion of Hexbound

I tried running the generate_pattern_info.py script the other day but the program failed to compile because it could find https://hexbound.cypher.coffee/0.1.3+1.19.2/docs.json
logs: log.txt
It seems the call comes from here, but I'm not sure.

Temporarily fixed in 9214008d5384ec7b5b191b12754ba94940f03ca0 by removing Hexbound; I'll re-add it if/when I get around to setting it up with a hexdoc plugin.


lxd/storage/ceph: Fix to work on older releases

Signed-off-by: Stéphane Graber<EMAIL_ADDRESS>

Testsuite passed


[wip - do not review] Update rad init to generate bicep config

Description
This adds a default bicepconfig.json during rad init
Type of change

This pull request fixes a bug in Radius and has an approved issue (issue link required).
This pull request adds or changes features of Radius and has an approved issue (issue link required).
This pull request is a minor refactor, code cleanup, test improvement, or other maintenance task and doesn't change the functionality of Radius (issue link optional).

Fixes: #issue_number

Closing in favor of #7664


Add documentation for ManagementService

I was just scrolling through the code of Armeria and I have stumbled accross ManagementService. This service was something we absolutely needed but we actually never realized it existed internally in Armeria.
I think it would be nice to include a section for utilities such as ManagementService in documentation.

It's a good idea. There are still a lot of features to be added though.

I'll try to add documentation for this! 🙇

Thanks for your contribution.


docs: quick start page

What this PR does:
Create a quick start guide for Teletrace

mergify refresh

@mergifyio refresh


Updates the hcl linting

Lints all the terraform in the repo and fixes if needed. Perhaps I should change check_terraform to validate_terraform?
The second commit is after everything is applied.

It looks like there was some out of date information to use hcl_fmt in the contributing.md. Updated it to explain how to get your code linted.


Correct error

When i'm typing :
./speedtest.py --share --no-download
i'm obtain this :
Retrieving speedtest.net configuration...
Testing from Netrix (89.234.*.1)...
Retrieving speedtest.net server list...
Selecting best server based on ping...
Hosted by Orange (Paris) [15.19 km]: 2.373 ms
Skipping download test
Testing upload speed....................................................................................................
Upload: 246.74 Mbit/s
Share results: http://www.speedtest.net/result/0.png
And this picture it's not work

I'm not sure anything can be done here, other than documenting that this wont work.
I don't control the speedtest.net API, and they dictate what is valid.  Sending a 0 for upload or download will cause the API to fail and not render an image.


Fix sync and remove labels

Motivation

If different labels are selected multiple times, the previously selected labels cannot be deleted correctly.
When the committer wants to manually add the document labels, we need to synchronize these labels into the PR.

/cc @maxsxu @tisonkun @Anonymitaet

@maxsxu
could you please review this PR from a technical perspective? Thank you!

Can I summarize the issue on the current master is that the doc related labels can be more than they should be?
That's, no comments mangled regression as https://github.com/apache/pulsar-test-infra/issues/47, or the box is checked but still complain doc-label-missing.

I think we should try to decide which input is the source. Right now,

A change on labels manually by committers can trigger PR description modified.
A change in PR description can update labels.

I suppose we just use the PR description checkbox as the source, and this can also resolve @michaeljmarshall's #47 that the bot should not update PR description.

We have discussed this offline, using PR description as the only input, thanks all.

To publish to the Apache repository, we need PMC operations.

I noticed that this PR doesn't have a diff to remove PR edit logic. Will take a closer look now.

addAndCleanupHelpComment() method is safe, which doesn't add comments more than once.

@nodece since the manner changed, I think we should update the comment content also for simply inlining the information to ask the user to add a valid checkbox, instead of linking to the gdoc.
@nicoloboschi for your information. I noticed that you often omit the doc checkbox section and can be affected by what's achieved here.


change lock to interlocked atomics

I hereby agree to the terms of the CLA available at: https://yandex.ru/legal/cla/?lang=en

Pull request type

Please check the type of change your PR introduces:

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, renaming)
[x] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] Documentation content changes
[ ] Other (please describe):

What is the current behavior?
Using lock to increment _txCounter
Issue Number: N/A
What is the new behavior?
Now implementation uses lock-free CAS based value computation, which would work faster, without allocating new object and creating a monitor.
Other information
None

Hello, sorry for late response.
Thanks for the fix :)
Can you rebase your PR and fix linter issues?


User Guide: missing ratings-data.csv

Four different pages in our user guide reference ratings-data.csv, but we never tell how to get it.

Hey. Can I work on this?

@karthikdash - thanks for offering to help, but this is probably not a good issue to start on. The dataset is not from UCI ML.
We are working on this issue and will hopefully have a solution soon.

Oh, alright. Thanks!

Closing as duplicate of https://github.com/apple/turicreate/issues/97


fix: Ensure FSX IAM Policy Permits Updating File Systems

Description

Updates FSX IAM Policy to permit File System Updates.
Motivation and Context

Aligns the policy provided with the recommended upstream.
Breaking Changes

None.
How Has This Been Tested?

[ ] I have updated at least one of the examples/* to demonstrate and validate my change(s)
[ ] I have tested and validated these changes using one or more of the provided examples/* projects

[x] I have executed pre-commit run -a on my pull request

This PR is included in version 1.4.2 :tada:


exceptions: more handling of nil status.

This was occurring on ARM CI.

Review period skipped due to critical label.

Review period skipped due to critical label.


10-final_vBrusentsov

end tasks with star
https://vbrusentsov.github.io/fe-course-basic-2023/vBrusentsov/about.html

Hey, @vBrusentsov, you should remove changes of the files, not delete files. I've wrote you simple way in tg

make a new branch from the fresh main
add you files to docs folder
create pr

OK, I'm closing this pr and waiting for the correct one


[Feature Request]: Make it possible to parse params in the query event?

Discussed in https://github.com/prisma/prisma/discussions/14158

Originally posted by tmtron July  5, 2022
In the query event we get the query which can include ? placeholders and the params which is an array-like string: e.g. [\"abc2\",2022-07-05 14:18:03.680719400 UTC,2]
We'd like to write a convenience function to replace the ? with the actual param values - just for readability of the log-messages during development (It can be quite difficult to understand a long query with multiple parameters)
I tried to write a simple parsing logic for the params, but I think, currently there is no reliable way to parse that string (i.e. to get the individual items), because quotes and commas within a string are not escaped

e.g. when the first parameter is a "," in a string, and the 2nd parameter is p2 the params array is [\"a \",\" in a string\",\"p2\"]
from the number of question marks in the query, we know that there must be 2 parameters , but it's not possible to know if these are

a "," in a string and p2
or a " and " in a string","p2"

Notes:

we use SQLite
Maybe, there there is already a function to do that?
Or is there a way to tell prisma to JSON encode the params?

Also https://github.com/prisma/prisma/issues/12441


Sources with different numbers of channels for sourcefolder dataset

🐛 Bug
First of all, I have to acknowledge the authors of open-unmix for this obviously awesome work ;)
My issue is about the sourcefolder dataset, which cannot handle sources with different numbers of channels. Let's assume that we have two folders of sources, the first one contains mono signals, the second one stereo signals. For training, we also set nb_channels to 1. In __getitem__ of SourceFolderDataset, an error is raised when trying to stack the sources, before summing them to create the mixture (line 358 of data.py).

To Reproduce
Steps to reproduce the behavior:

Create two folders of sources, one with stereo signals and the other one with mono signals.
Launch training with nb-channels to 1, below is the command I used:

python train.py --root ./data-sourcefolder --dataset sourcefolder --interferer-dirs noise --target-dir speech --nb-train-samples 20000 --nb-valid-samples 2000 --seq-dur 2.0 --source-augmentations gain --hidden-size 256 --nb-channels 1 --nfft 1024 --nhop 256 --nb-workers 4

We get the following error:

Traceback (most recent call last):
  File "train.py", line 294, in <module>
    main()
  File "train.py", line 177, in main
    scaler_mean, scaler_std = get_statistics(args, train_dataset)
  File "train.py", line 68, in get_statistics
    x, y = dataset_scaler[ind]
  File "/data/recherche/python/speech_enhancement/open-unmix-pytorch/data.py", line 385, in __getitem__
    stems = torch.stack(audio_sources)
RuntimeError: invalid argument 0: Sizes of tensors must match except in dimension 0. Got 2 and 1 in dimension 1 at /tmp/pip-req-build-58y_cjjl/aten/src/TH/generic/THTensor.cpp:689

Expected behavior
We could expect that because we set nb_channels to 1, stereo source signals would be downmixed so that they could be mixed with the monophonic sources.
Environment

PyTorch Version: 1.2.0
OS: Ubuntu
torchaudio loader: no
Python version: 3.7.3
CUDA/cuDNN version: CUDA 10.1 - cuDNN 7.6.0

Hi Simon, thanks for the issue. This is actually something that we could address as part of #33. I will leave this open until it's fixed.
Regarding a fix, do you agree that the downmix should be done in the magnitude spectrum domain?

Hi Fabian,
Yes it seems ok to me. Naturally, I would have done the downmix in the waveform domain, but actually I guess you chose the magnitude spectrum domain to avoid phase problems? It is then a good idea, and maybe better than selecting only one channel.

@sleglaive coming back to this, I am not sure how this can be fixed at the dataloader stage, when nb_channels=1 and a stereo file is loaded as we want the downmix to be done in the magnitude spectrogram domain. If we downmix in the waveform domain already the downmix in the spectrogram layer would never be called.
Instead, for heterogenous datasets, I would propose an upmix_to_stero=True switch for the dataloaders that would always upmix to stereo by copying the channels and later downmix in the spectrogram domain
@aliutkus additional thoughts?

@sleglaive this is now addressed by #56 and merged into the dev branch. Maybe you can comment on this before we release the new version

@faroit sorry for the late reply, the solution you propose sounds great to me, thanks a lot!

Great. I will close this when the dev branch is merged


Production URL not displaying content

Describe the bug
As per title, "/" and "/users" endpoints not returning same data as localhost
Steps to Reproduce

Go to https://fitness-app-beryl.vercel.app/ OR https://fitness-app-beryl.vercel.app/users
See text rendered briefly in browser before rendering blank page OR error message, respectively.

Solution

likely need to update environment variables, from memory it's trying to pull from localhost:5001,
Setup env variables in vercel and/or a production .env file
check fetch request, should be something along the lines of  const api = localEnvVar || prodEnvVar; await fetch(api)

Related/Duplicate Issue


nvidia_x11_legacy304/nvidia_x11_legacy340: remove

Apparently broken on all kernel versions: https://github.com/NixOS/nixpkgs/pull/49470#issuecomment-442785553

I just saw @volth @florianjacob in the git history maintaining these packages.
If there are still kernels where these works can we get an upper bound on the kernel version?

I still have 2 videocards in use, one of them (NVS440) requires nvidiaLegacy304 and another (8800GT) nvidiaLegacy340.
The first is difficult to replace, it is a 4 heads PCI card, so I will have to support nvidiaLegacy304 in my fork.

340 was recently updated in #50841 to prevent build failure with newer kernels
(and backported in #51014)

304 was discontinued end of 2017 and won't get updates for newer kernel / X.org versions by nvidia anymore. In theory, it should still work with 4.14, though. @volth can you confirm that? Not sure whether that failed as well in Mic's build failure list.
We could then whitelist it for 4.14 and drop it when NixOS drops 4.14, it will definitely never gain support for newer kernels, and will probably break with a future X.org version as well.
340 will still be updated by nvidia until the end of 2019 to support new kernels and X.org versions. With the recent update & backport, it should work across all kernels again, I have it running here based on 4.19. I'll try to provide the nvidia updates until then.
Source: https://nvidia.custhelp.com/app/answers/detail/a_id/3142

it should still work with 4.14, though. @volth can you confirm that?

I am mostly on 4.4 and 4.9 anyway

@c0bw3b I tested the build with the compilation fix included.

I made a new pull request that marks nvidia 304 as broken on newer kernels.

Fix for the patch failing to be applied to 340.107 with kernel 4.14 kindly provided by @binarin in #51457, by removing it completely.


Update for CUDA 12.4.0

xref: https://github.com/conda-forge/cuda-feedstock/issues/17

@conda-forge-admin , please re-render

Not seeing a Travis build following this PR, triggered one here https://app.travis-ci.com/github/conda-forge/cuda-visual-tools-feedstock/builds/269326604

Thanks Alex! 🙏
Yeah it is likely the same issue we have seen before
Travis CI says they are escalating our bug internally


Operator class in aqua

Hi, I was supposed to use Operator class of aqua as required by a course I am taking on edX. However, I have realised that Operator class doesn't exist anymore. I was searching up near class to construct a Hamiltonian as a sum of products of Pauli matrices, formerly done using Operator class

The nearest class I could find for such operation was WeightedPauliOperator() but its methods and arguments are not mentioned in aqua documentation hence I am unable to use it.
Kindly help.

What's called Operator in the text is indeed the WeightedPauliOperator. However with Aqua 0.7 a new framework for Operators has been introduced which mimics the mathematical workflow, see the documentation or this notebook for an introduction.
Your example would simply be
from qiskit.aqua.operators import X, Z, I
op = (3 * Z) ^ I ^ (2*X)  # ^ denotes a tensor product

you can get the "old" operator notation from this by calling to_legacy_op
old_op = op.to_legacy_op()
print(old_op.paulis)  # prints [[(6+0j), Pauli(z=[False, False, True], x=[True, False, False])]]

Btw, even if no documentation exists in qiskit.org, the code of the WeightedPauliOperator still is documented if you need to look something up.

Thank you very much!


[BUG] Increased diskSize is not reflected in the pods filesystem

What is the bug?
When we increase diskSize in the OpensearchCluster spec, the PVCs are resized accordingly. However, filesystem in the pods is not resized. Or, to be precise, it might be resized for some of the pods.
We use Openstack with Cinder volume (Storage class: csi-cinder-sc-delete).
How can one reproduce the bug?
In an already deployed cluster with diskSize for a nodePool equal to 5Gi, modify diskSize and increase it to 6Gi. Then apply the manifest.
Then increase it to 7Gi and apply the manifest again. So the nodePools config looks like
    nodePools:
    - affinity:
        podAntiAffinity:
          requiredDuringSchedulingIgnoredDuringExecution:
          - labelSelector:
              matchExpressions:
              - key: opster.io/opensearch-nodepool
                operator: In
                values:
                - data-nodes
            topologyKey: kubernetes.io/hostname
      component: data-nodes
      diskSize: 7Gi
      replicas: 3
      resources:
        limits:
          cpu: 500m
          memory: 2Gi
        requests:
          cpu: 500m
          memory: 2Gi
      roles:
      - data

Check the PVCs, they are 7Gi in size:
data-caas-opensearch-data-nodes-0         Bound    pvc-058d4eda-2800-42ae-9859-f3518db541a9   7Gi        RWO            csi-cinder-sc-delete   5d21h
data-caas-opensearch-data-nodes-1         Bound    pvc-af24413f-2eaf-4743-b234-2fc27c682cbf   7Gi        RWO            csi-cinder-sc-delete   5d20h
data-caas-opensearch-data-nodes-2         Bound    pvc-f270c56a-8ea6-4228-9177-ef4e554dceff   7Gi        RWO            csi-cinder-sc-delete   5d20h

Check the pods filesystem:
Pod-0:
/dev/vdh        5.9G  5.5M  5.9G   1% /usr/share/opensearch/data

Pod-1:
/dev/vdg        5.9G  5.1M  5.9G   1% /usr/share/opensearch/data

Pod-2:
/dev/vdi        4.9G  1.8M  4.9G   1% /usr/share/opensearch/data

So we can see that somehow 2 out of 3 pods saw the first resizing to 6Gi, but not the second resizing to 7Gi.
The 3rd pod is still as if no resizing took place.
I tried to delete the pods, but it doesn't help.
What is the expected behavior?
The PVCs should be 7Gi.
When checking the filesystem from inside the pod (df -h), the filesystem should be 7 gigabytes.
What is your host/environment?
Operator version 2.5.1 (also tested with 2.4.0).

[Triage]
Thanks @dmantas the steps implemented as part of the design https://github.com/opensearch-project/opensearch-k8s-operator/issues/112#issuecomment-1107854259 should have the size reflected inside the POD as well, can you see if there are any errors in the operator pod?
Thanks
Adding @dbason @swoehrl-mw @jochenkressin @pchmielnik @salyh @bbarani

Hi @prudhvigodithi , I followed these steps and I was successful, thank you. Although after deleting the sts, this was immediately being recreated by the Operator. I had to temporarily delete the Operator to prevent this. Is there any way I can temporarily prevent the Operator from messing with the deployment without deleting it completely? E.g. in a similar case for the Prometheus Operator, it has a pause flag to achieve this.
Other than this method, I understand it's not possible to resize volumes automatically, i.e. by modifying the OpensearchCluster manifest, correct? Because I did a test today and I noticed these errors in the Operator logs:
{"level":"dpanic","ts":"2024-04-02T11:51:56.608Z","msg":"odd number of arguments passed as key-value pairs for logging","controller":"opensearchcluster","controllerGroup":"opensearch.opster.io","controllerKind":"OpenSearchCluster","OpenSearchCluster":{"name":"caas-opensearch","namespace":"opensearch-operator-cluster"},"namespace":"opensearch-operator-cluster","name":"caas-opensearch","reconcileID":"7cea47f7-e5e4-430c-9d30-bc066f133eeb","ignored key":"9Gi","stacktrace":"github.com/Opster/opensearch-k8s-operator/opensearch-operator/pkg/reconcilers.(*ClusterReconciler).maybeUpdateVolumes\n\t/workspace/pkg/reconcilers/cluster.go:490\ngithub.com/Opster/opensearch-k8s-operator/opensearch-operator/pkg/reconcilers.(*ClusterReconciler).reconcileNodeStatefulSet\n\t/workspace/pkg/reconcilers/cluster.go:301\ngithub.com/Opster/opensearch-k8s-operator/opensearch-operator/pkg/reconcilers.(*ClusterReconciler).Reconcile\n\t/workspace/pkg/reconcilers/cluster.go:116\ngithub.com/Opster/opensearch-k8s-operator/opensearch-operator/controllers.(*OpenSearchClusterReconciler).reconcilePhaseRunning\n\t/workspace/controllers/opensearchController.go:319\ngithub.com/Opster/opensearch-k8s-operator/opensearch-operator/controllers.(*OpenSearchClusterReconciler).Reconcile\n\t/workspace/controllers/opensearchController.go:141\nsigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).Reconcile\n\t/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.15.0/pkg/internal/controller/controller.go:118\nsigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).reconcileHandler\n\t/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.15.0/pkg/internal/controller/controller.go:314\nsigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).processNextWorkItem\n\t/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.15.0/pkg/internal/controller/controller.go:265\nsigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).Start.func2.2\n\t/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.15.0/pkg/internal/controller/controller.go:226"}
{"level":"info","ts":"2024-04-02T11:51:56.608Z","msg":"Disk sizes differ for nodePool %s, Current: %s, Desired: %s","controller":"opensearchcluster","controllerGroup":"opensearch.opster.io","controllerKind":"OpenSearchCluster","OpenSearchCluster":{"name":"caas-opensearch","namespace":"opensearch-operator-cluster"},"namespace":"opensearch-operator-cluster","name":"caas-opensearch","reconcileID":"7cea47f7-e5e4-430c-9d30-bc066f133eeb","data-nodes":"8Gi"}
{"level":"info","ts":"2024-04-02T11:51:56.608Z","msg":"Deleting statefulset while orphaning pods caas-opensearch-data-nodes","controller":"opensearchcluster","controllerGroup":"opensearch.opster.io","controllerKind":"OpenSearchCluster","OpenSearchCluster":{"name":"caas-opensearch","namespace":"opensearch-operator-cluster"},"namespace":"opensearch-operator-cluster","name":"caas-opensearch","reconcileID":"7cea47f7-e5e4-430c-9d30-bc066f133eeb"}
{"level":"info","ts":"2024-04-02T11:51:56.928Z","msg":"object  is being deleted, backing off","controller":"opensearchcluster","controllerGroup":"opensearch.opster.io","controllerKind":"OpenSearchCluster","OpenSearchCluster":{"name":"caas-opensearch","namespace":"opensearch-operator-cluster"},"namespace":"opensearch-operator-cluster","name":"caas-opensearch","reconcileID":"7cea47f7-e5e4-430c-9d30-bc066f133eeb","name":"caas-opensearch-data-nodes","namespace":"opensearch-operator-cluster","apiVersion":"apps/v1","kind":"StatefulSet"}
{"level":"error","ts":"2024-04-02T11:51:57.050Z","msg":"Reconciler error","controller":"opensearchcluster","controllerGroup":"opensearch.opster.io","controllerKind":"OpenSearchCluster","OpenSearchCluster":{"name":"caas-opensearch","namespace":"opensearch-operator-cluster"},"namespace":"opensearch-operator-cluster","name":"caas-opensearch","reconcileID":"7cea47f7-e5e4-430c-9d30-bc066f133eeb","error":"StatefulSet.apps \"caas-opensearch-data-nodes\" not found","stacktrace":"sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).reconcileHandler\n\t/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.15.0/pkg/internal/controller/controller.go:324\nsigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).processNextWorkItem\n\t/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.15.0/pkg/internal/controller/controller.go:265\nsigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).Start.func2.2\n\t/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.15.0/pkg/internal/controller/controller.go:226"}
{"level":"info","ts":"2024-04-02T11:51:57.051Z","msg":"Reconciling OpenSearchCluster","controller":"opensearchcluster","controllerGroup":"opensearch.opster.io","controllerKind":"OpenSearchCluster","OpenSearchCluster":{"name":"caas-opensearch","namespace":"opensearch-operator-cluster"},"namespace":"opensearch-operator-cluster","name":"caas-opensearch","reconcileID":"80ef6069-6650-4532-8636-f3756e9a9f02","cluster":{"name":"caas-opensearch","namespace":"opensearch-operator-cluster"}}
{"level":"info","ts":"2024-04-02T11:51:57.089Z","msg":"Generating certificates","controller":"opensearchcluster","controllerGroup":"opensearch.opster.io","controllerKind":"OpenSearchCluster","OpenSearchCluster":{"name":"caas-opensearch","namespace":"opensearch-operator-cluster"},"namespace":"opensearch-operator-cluster","name":"caas-opensearch","reconcileID":"80ef6069-6650-4532-8636-f3756e9a9f02","interface":"transport"}
{"level":"info","ts":"2024-04-02T11:51:57.089Z","msg":"Generating certificates","controller":"opensearchcluster","controllerGroup":"opensearch.opster.io","controllerKind":"OpenSearchCluster","OpenSearchCluster":{"name":"caas-opensearch","namespace":"opensearch-operator-cluster"},"namespace":"opensearch-operator-cluster","name":"caas-opensearch","reconcileID":"80ef6069-6650-4532-8636-f3756e9a9f02","interface":"http"}
{"level":"info","ts":"2024-04-02T11:51:57.210Z","msg":"resource created","controller":"opensearchcluster","controllerGroup":"opensearch.opster.io","controllerKind":"OpenSearchCluster","OpenSearchCluster":{"name":"caas-opensearch","namespace":"opensearch-operator-cluster"},"namespace":"opensearch-operator-cluster","name":"caas-opensearch","reconcileID":"80ef6069-6650-4532-8636-f3756e9a9f02","name":"caas-opensearch-data-nodes","namespace":"opensearch-operator-cluster","apiVersion":"apps/v1","kind":"StatefulSet"}
{"level":"info","ts":"2024-04-02T11:52:00.592Z","msg":"Starting rolling restart of the StatefulSet caas-opensearch-data-nodes","controller":"opensearchcluster","controllerGroup":"opensearch.opster.io","controllerKind":"OpenSearchCluster","OpenSearchCluster":{"name":"caas-opensearch","namespace":"opensearch-operator-cluster"},"namespace":"opensearch-operator-cluster","name":"caas-opensearch","reconcileID":"80ef6069-6650-4532-8636-f3756e9a9f02","reconciler":"restart"}
{"level":"info","ts":"2024-04-02T11:52:00.596Z","msg":"Preparing to restart pod caas-opensearch-data-nodes-0","controller":"opensearchcluster","controllerGroup":"opensearch.opster.io","controllerKind":"OpenSearchCluster","OpenSearchCluster":{"name":"caas-opensearch","namespace":"opensearch-operator-cluster"},"namespace":"opensearch-operator-cluster","name":"caas-opensearch","reconcileID":"80ef6069-6650-4532-8636-f3756e9a9f02","reconciler":"restart"}

So it seems the Operator "knows" how to do the resize, but it doesn't work.
It even seems only the first pod in the StatefulSet is restarted, but even for that one the filesystem inside the pod isn't increased - but all PVCs are increased.
Is there something not working correctly in this implementation?
Again thanks for your help, at least I seem to have a consistent way to do this.


Clean up import/export portlet page

Describe the bug
The UX of the import / export portlets utility should be cleaned up.

Stop Upload button should only be enabled / displayed when portlets are being uploaded.
After a portlet is uploaded, the Remove from queue button is still present, though clicking on it does nothing.

Screenshots

Thanks @loulou2u !
That fix made the Stop Upload button behave, though the Remove from queue button is still there after 'upload' is pressed

Thanks @loulou2u !
That fix made the Stop Upload button behave, though the Remove from queue button is still there after 'upload' is pressed


Video Intelligence - invalid argument

NOTE: relocating from https://github.com/googleapis/gax-php/issues/69
My first call to annotateVideo throws an exception, but I double-checked the arguments and everything looks ok.  Stanley, can you dig deeper?  My code is here:
https://github.com/SurferJeffAtGoogle/php-docs-samples/tree/video/video
/usr/local/bin/php /usr/local/google/home/rennie/Downloads/phpunit-5.7.19.phar --configuration /usr/local/google/home/rennie/gitrepos/php-docs-samples/video/phpunit.xml.dist --teamcity
Testing started at 9:36 AM ...
PHPUnit 5.7.19 by Sebastian Bergmann and contributors.

Request contains an invalid argument.
 /usr/local/google/home/rennie/gitrepos/php-docs-samples/video/vendor/google/gax/src/ApiException.php:51
 /usr/local/google/home/rennie/gitrepos/php-docs-samples/video/vendor/google/gax/src/ApiCallable.php:151
 /usr/local/google/home/rennie/gitrepos/php-docs-samples/video/vendor/google/gax/src/ApiCallable.php:59
 /usr/local/google/home/rennie/gitrepos/php-docs-samples/video/vendor/google/gax/src/ApiCallable.php:90
 /usr/local/google/home/rennie/gitrepos/php-docs-samples/video/vendor/google/gax/src/ApiCallable.php:126
 /usr/local/google/home/rennie/gitrepos/php-docs-samples/video/vendor/google/gax/src/ApiCallable.php:192
 /usr/local/google/home/rennie/gitrepos/php-docs-samples/video/vendor/google/cloud-videointelligence/V1beta1/VideoIntelligenceServiceClient.php:411
 /usr/local/google/home/rennie/gitrepos/php-docs-samples/video/src/detect_face.php:27
 /usr/local/google/home/rennie/gitrepos/php-docs-samples/video/video.php:48
 /usr/local/google/home/rennie/gitrepos/php-docs-samples/video/vendor/symfony/console/Command/Command.php:260
 /usr/local/google/home/rennie/gitrepos/php-docs-samples/video/vendor/symfony/console/Tester/CommandTester.php:84
 /usr/local/google/home/rennie/gitrepos/php-docs-samples/video/test/CommandTest.php:54

Time: 723 ms, Memory: 27.75MB

ERRORS!
Tests: 1, Assertions: 0, Errors: 1.

The value of $uri is gs://cloudmleap/video/next/fox-snatched.mp4

This seems to be caused because FACE_DETECTION has not been made publicly available. Using the code below:
$video = new VideoIntelligenceServiceClient();
$features = [Feature::FACE_DETECTION];
//$features = [Feature::SHOT_CHANGE_DETECTION];

try {
    $operationResponse = $video->annotateVideo($uri, $features);
    $operationResponse->pollUntilComplete();
    if ($operationResponse->operationSucceeded()) {
        $result = $operationResponse->getResult();
        // doSomethingWith($result)
    } else {
        $error = $operationResponse->getError();
        // handleError($error)
    }
} catch (\Google\GAX\Exception $ex) {
    echo var_dump($ex->getMetadata());
}

I get this information in the metadata:
array(1) {
  'google.rpc.debuginfo-bin' =>
  array(1) {
    [0] =>
    string(255) "�[ORIGINAL ERROR] generic::invalid_argument:
        The request has not specified any features or has specified a feature
        that is visible only to some cloud projects. If the latter is the case, please
        contact<EMAIL_ADDRESS>to get access."
  }
}

Using Feature::SHOT_CHANGE_DETECTION works for me.
This is a nasty error given how hard it is to work out what is going on. See also https://github.com/GoogleCloudPlatform/google-cloud-php/issues/509 which relates to exception information.

I believe this can be marked as closed after the improvements in the generated exception messages in https://github.com/googleapis/gax-php/pull/80 and https://github.com/googleapis/gax-php/pull/70


Adjust Individual Marks in Groups

I suggest that having the ability to adjust individual marks within a group would make a good addition to MarkUs. This would give the professors or TAs the option to ask students to give feedback about their group members in case mark adjustments are needed.
As far as I know, there isn't something like this yet.

Reasons for marking invalid:

7 years later and there isn't a demand for this feature
This adds a lot of complexity for a feature that can be handled using single-membership groups/spreadsheets


Our Chef section could be better its very messy.

Hello @Anupkjha2601 ,
I am a contributor for GSSOC'23 and I would like to address the issue regarding the placement of the "Our chef section"
What issue are you trying to address?

This section is a bit messy as well as not responsive and the Cards can be improved a bit.
What improvements you are suggesting?
The social media icons arrangement can be improved as well as the chef cards can be improved. Some resizing is needed to be done. This section is not responsive we can make it responsive.
Kindly assign me this task under the label GSSOC'23


Will Placement support replicas scheduling?

Describe the enhancement
A clear and concise description of what the enhancement request is.
Moved from https://github.com/open-cluster-management-io/placement/issues/96
ivan-cai commented on Jan 6, 2023
for example, global replicas is 10, and 2 in cluster1, 3 in cluster2, 5 in cluster3 after scheduling.
xuc623671 commented on Jul 29
This is a good idea, it is needed in actual use

Likely related to the discussion at https://github.com/open-cluster-management-io/ocm/issues/155.


Use with bash

Hi there,
these tools are limited to zsh right?
I would love to use them with a good old bash!

I have added the init script for bash, but some functions are not implemented.
All functions should work on bash too, but we need to import zsh functions to bash.


'same' test fails on bigint

I'm writing a numeric utility library and when using 'same' comparison between bigints, it fails; even in situations where the equivalent test framed as 'ok' succeeds:
    const bigNum = numb(2n ** 53n);
    ln.ok(bigNum.isValid);
    ln.ok(bigNum.value === 2n ** 53n); // <--- passes
   ln.same(bigNum.value, 2n ** 53n); /// <--- fails

This issue is fixed in tap v14 by the release of tcompare 3.0.5.


python311Packages.tensordict: 0.3.0 -> 0.3.1

Description of changes
Changelog: https://github.com/pytorch/tensordict/releases/tag/v0.3.1

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

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

Result of nixpkgs-review pr 291751 run on aarch64-linux 1

2 packages marked as broken and skipped:

python312Packages.tensordict
python312Packages.tensordict.dist

4 packages failed to build:

python311Packages.tensordict
python311Packages.tensordict.dist
python311Packages.torchrl
python311Packages.torchrl.dist

Result of nixpkgs-review pr 291751 run on aarch64-linux 1

2 packages marked as broken and skipped:

python312Packages.tensordict
python312Packages.tensordict.dist

2 packages failed to build:

python311Packages.torchrl
python311Packages.torchrl.dist

2 packages built:

python311Packages.tensordict
python311Packages.tensordict.dist

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

2 packages marked as broken and skipped:

python312Packages.tensordict
python312Packages.tensordict.dist

2 packages failed to build:

python311Packages.torchrl
python311Packages.torchrl.dist

2 packages built:

python311Packages.tensordict
python311Packages.tensordict.dist

This update does not bring any regression.
torchrl is also broken on master as its dependency pygame is broken.


[SPARK-35357][GRAPHX] Allow to turn off the normalization applied by static PageRank utilities

What changes were proposed in this pull request?

Overload methods PageRank.runWithOptions and  PageRank.runWithOptionsWithPreviousPageRank (not to break any user-facing signature) with a normalized parameter that describes "whether or not to normalize the rank sum".
Why are the changes needed?

https://issues.apache.org/jira/browse/SPARK-35357
When dealing with a non negligible proportion of sinks in a graph, algorithm based on incremental update of ranks can get a precision gain for free if they are allowed to manipulate non normalized ranks.
Does this PR introduce any user-facing change?

No
How was this patch tested?

By adding a unit test that verifies that (even when dealing with a graph containing a sink) we end up with the same result for both these scenarios:
a)

Run 6 iterations of pagerank in a row using PageRank.runWithOptions with normalization enabled

b)

Run 2 iterations using PageRank.runWithOptions with normalization disabled
Resume from the preRankGraph1 and run 2 more iterations using PageRank.runWithOptionsWithPreviousPageRank with normalization disabled
Finally resume from the preRankGraph2 and run 2 more iterations using PageRank.runWithOptionsWithPreviousPageRank with normalization enabled

Can one of the admins verify this patch?

Merged to master


Need pre-trained kinetics weights for Pytorch

Hello everyone,
Has anyone successfully converted kinetics pre-trained weights into Pytorch TSN?

I have converted the inception V3 weights to Keras, if you want them tell me.

I have .hdf5 files for TF/Keras if you want.

@pedro-abreu  can you share them with me. Email address<EMAIL_ADDRESS>

@pedro-abreu Thanks a lot, but Im using PyTorch atm. For anyone that may be interested, you can download the caffe pretrained weights and convert them to pytorch weights using parse_caffe.py in
tsn-pytorch/tf_model_zoo/bninception.

@ekazakos Thanks man, I might switch to Pytorch eventually :). @fmthoker https://drive.google.com/drive/folders/13TaIjogd2ReNsKXNJ55o3_qsDWB5KlkM?usp=sharing here you go.

Thanks to folks! Do you mind sharing it to other users?

@yjxiong Sure, will do soon!

Everyone, you can find the pretrained Kinetics weights for TSN that I converted to work with PyTorcg here: https://drive.google.com/drive/folders/1R4n8vSQJ0LtB1mPhhjMHEQrKj3UDQS1U?usp=sharing
Load the weights normally with:
state_dict = torch.load(pretrained)
If you are using Pytorch 0.4 do:
for k, v in state_dict.items():
state_dict[k] = torch.squeeze(v, dim=0)

@ekazakos
Thanks! Do you mind we share the files in the project website?

@yjxiong It is totally fine.

Everyone, you can find the pretrained Kinetics weights for TSN that I converted to work with PyTorch here: https://drive.google.com/drive/folders/1R4n8vSQJ0LtB1mPhhjMHEQrKj3UDQS1U?usp=sharing
Load the weights normally with:
state_dict = torch.load(pretrained)
If you are using Pytorch 0.4 do:
for k, v in state_dict.items():
    state_dict[k] = torch.squeeze(v, dim=0)

Hello, thanks for sharing the weights. Do you have RGB Difference pretrained weights as well?

@AhmedGamal1496 Im really sorry for the very late answer, I do not know how I missed this one. I do not have RGB Difference. And yes I've used only BN-Inception.


(pdfcreator) Version 2.5.0 is timing out on the verifier

Something may perhaps have changed with the installer since the installation times out.
https://gist.github.com/choco-bot/458d0118020b16dc16215cd0e213aa6c
Opening this as I don't have the will to look into it today.

I believe I found out why it fails on the verifier, seems like the print spool needs to be running for the install to be successful (as well as for running pdf creator).
I'll see if I can't fix this, perhaps reuse the code used in cutepdf package (contributed by @Thilas)


Convert 14 instances of tabs to spaces

Fixing this https://github.com/chris-pearce/scally/pull/74#issuecomment-94673302

Nice one @stowball :+1:, I'll look into fixing Sublime or switch to Brackets :) You don't mind if I don't mention this update on Twitter :)

Oh, but I do mind :P


Apache Beam Java Lib and ANTLR

I am using Apache Beam with Google Dataflow in Java and I am trying to migrate the java lib from v2.51.0 to v2.59.0
We are using ANTLR v4.11.1 as a dependency of our Beam Pipeline.
With the v2.59.0 the pipeline fails to start with the error
ANTLR Tool version 4.7 used for code generation does not match the current runtime version 4.11.1
I can see both Apache Beam v2.51.0 and v2.59.0 are compiled with ANTLR v4.7 as a dependency HERE and HERE
My questions are the following:
Why do I suddenly get an exception at runtime with v2.59.0 when the v2.51.0 was working correctly?
I am not very confident to downgrade our version of ANTLR just because Apache Beam uses the v4.7. What if you upgrade it to v4.13? I would need to adapt again... this problem can be met by a lot of users of Apache Beam, is it possible to get rid of this dependency on Apache Beam lib?

We are running into a similar issue where we have a 4.13 dependency elsewhere in our codebase, and now 4.7 through Beam. Can Beam upgrade to 4.13?


Update QueryBuilder.cs

Check null before accessing nested object

Hi @markikordahi thanks! I don't suppose you could write a quick unit test just to help me better understand the case this protects against for posterity could you?

i had the same issue "https://github.com/tghamm/dynamic-linq-query-builder/issues/61"
as an example, i have a student class and a Course class. student can have 0 1 or many classes.
when searching for allstudents with class name = "ABC" this was building below query which was throwing object not set to an instance of an object since Any was trying to access courses which is null
item.Courses.AsQueryable().Any(Param_0 => ((True == True) AndAlso (Param_0.bookId.ToString().ToLower() == \"1\".ToLower())))
so now the query will look like this
((item.Courses != null) AndAlso item.Courses.AsQueryable().Any(Param_0 => ((True == True) AndAlso (Param_0.bookId.ToString().ToLower() == \"1\".ToLower()))))

Thanks for clarifying, I'll take a look.


Fossasia logo not visible in mobile screens

The FOSSASIA logo is not visible in mobile screens in the menu bar as opposed to in larger screens.
In desktop:

In mobile screens:

It is visible only when we click on the menu hamburger icon:

The logo should be visible in the left side of the menu bar in all screen sizes as that is how all sites are designed.

@mariobehling @abishekvashok @fossasia/pslab-admins I think this change is necessary. If you approve shall I start working on these issues.


(chocolatey-npm.extension) Add extension for installing NPM packages

Description
This is a new extension which allows to install and uninstall NPM packages.
Motivation and Context
There exist several Chocolatey packages which are installing NPM packages. Installing NPM packages, even global, will always install into user profile, which leads to issues with a system wide package manager like Chocolatey.
This extension allows to work around some of them. It checks if NodeJs has been installed and if installing using background mode will install into user profile of user which initiated the process instead of installing into the profile of the user under which the Chocolatey service is running.
How Has this Been Tested?
Tested locally, with a modified version of the markdownlint-cli package on a machine where Chocolatey was configured to run with background mode and on a machine where Chocolatey was configured to run without background mode.
Screenshot (if appropriate, usually isn't needed):
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Migrated package (a package has been migrated from another repository)

Checklist:

[x] My code follows the code style of this repository.
[x] My change requires a change to documentation (this usually means the notes in the description of a package).
[x] I have updated the documentation accordingly (this usually means the notes in the description of a package).
[x] All files are up to date with the latest Contributing Guidelines
[ ] The added/modified package passed install/uninstall in the chocolatey test environment.
[ ] The changes only affect a single package (not including meta package).

:white_check_mark: Package verification completed without issues. PR is now pending human review

Since I needed this package for work I added it to another repo (https://github.com/bbtsoftware/chocolatey-packages/pull/102). If there's interest to move it to core teams repo we still can move it later.

Out of curiosity, why are you doing this ? Whats wrong with cinst nodejs and then using npm directly ?


Post content should not be HTML encoded on edit form

What steps will reproduce the problem?
Create a Post with an HTML code in the content field, edit created Post
What is the expected result?
raw HTML code (unencoded)
What do you get instead?
HTML-encoded content
Additional info

Q
A

Version
1.0.?

PHP version

Operating system

Fixed. Thanks.


Create A Readme ( SpaceInvaders - Python )

Location of the Document
You have to create a Readme for SpaceInvaders
What needs to be added
Create and Add details about the game.

assign this to me


failed to parse package.json on windows

hi,
I am trying to run this hack command on windows 10 but it throws errors. anyone stumbled upon this?
`PS E:\WindowsApp> npm run setup-windows --logging

<EMAIL_ADDRESS>setup-windows
node_modules.bin\rn-nodeify.cmd --hack
failed to parse node_modules\resolve\test\resolver\malformed_package_json\package.json
failed to parse node_modules\eslint-plugin-react\node_modules\resolve\test\resolver\malformed_package_json\package.json
failed to parse: E:\WindowsApp\node_modules\resolve\test\resolver\malformed_package_json\package.json
failed to parse: E:\WindowsApp\node_modules\eslint-plugin-react\node_modules\resolve\test\resolver\malformed_package_json\package.json`

attaching my package.json for reference
package.json.zip

I am having the same issue
anyone found a solution?
./node_modules/.bin/rn-nodeify  --install  --hack  --force         not overwriting "assert" not overwriting "browserify-zlib" not overwriting "buffer" not overwriting "console-browserify" not overwriting "constants-browserify" not overwriting "react-native-crypto" not overwriting "dns.js" not overwriting "domain-browser" not overwriting "events" not overwriting "@tradle/react-native-http" not overwriting "https-browserify" not overwriting "react-native-os" not overwriting "path-browserify" not overwriting "process" not overwriting "punycode" not overwriting "querystring-es3" not overwriting "react-native-level-fs" not overwriting "react-native-udp" not overwriting "stream-browserify" not overwriting "string_decoder" not overwriting "timers-browserify" not overwriting "tty-browserify" not overwriting "url" not overwriting "util" not overwriting "react-native-tcp" not overwriting "vm-browserify" not overwriting "readable-stream" not overwriting "react-native-randombytes" not reinstalling inherits not overwriting shim.js. For the latest version, see rn-nodeify/shim.js failed to parse node_modules\resolve\test\resolver\malformed_package_json\package.json failed to parse: C:\React Native Projects\MaximumSecurity\node_modules\resolve\test\resolver\malformed_package_json\package.json

Pinging for any updates. Were you able resolve this?

Restoring the project to the time before installing notify and then reinstalling all the plugins together did the trick for me.
But it may be luck or something else that I have changed in the project since the restoration backup...


docs: update docs with behind reverse proxy

Type: document
The following has been addressed in the PR:

There is a related issue: close #207

Resolves:
Update docs about configure verdaccio behind reverse proxy

FYI. I will move this content and the wiki to other repository on orther to autogenerate a website with github pages. New website 😉😆🛫 will be shipped soon.


Error running Python script

Hello, I've been trying to test out the Coin Magi Python library so I could later incorporate it into my other projects but when I try to run the "simple program"  It spits out these errors:
Traceback (most recent call last):
File "C:\Users\toshiba2014\Downloads\Python Script\main.py", line 4, in
print(magi.statistics())
File "C:\Users\toshiba2014\Downloads\Python Script\magilib.py", line 160, in statistics
info = self.magi.getmininginfo()
File "C:\Users\toshiba2014\AppData\Local\Programs\Python\Python310\lib\site-packages\bitcoinrpc\authproxy.py", line 132, in call
self.__conn.request('POST', self.__url.path, postdata,
File "C:\Users\toshiba2014\AppData\Local\Programs\Python\Python310\lib\http\client.py", line 1282, in request
self._send_request(method, url, body, headers, encode_chunked)
File "C:\Users\toshiba2014\AppData\Local\Programs\Python\Python310\lib\http\client.py", line 1328, in _send_request
self.endheaders(body, encode_chunked=encode_chunked)
File "C:\Users\toshiba2014\AppData\Local\Programs\Python\Python310\lib\http\client.py", line 1277, in endheaders
self._send_output(message_body, encode_chunked=encode_chunked)
File "C:\Users\toshiba2014\AppData\Local\Programs\Python\Python310\lib\http\client.py", line 1037, in _send_output
self.send(msg)
File "C:\Users\toshiba2014\AppData\Local\Programs\Python\Python310\lib\http\client.py", line 975, in send
self.connect()
File "C:\Users\toshiba2014\AppData\Local\Programs\Python\Python310\lib\http\client.py", line 941, in connect
self.sock = self._create_connection(
File "C:\Users\toshiba2014\AppData\Local\Programs\Python\Python310\lib\socket.py", line 845, in create_connection
raise err
File "C:\Users\toshiba2014\AppData\Local\Programs\Python\Python310\lib\socket.py", line 833, in create_connection
sock.connect(sa)
ConnectionRefusedError: [WinError 10061] No connection could be made because the target machine actively refused it
I have installed python-bitcoinrpc and requests. I have also put in the library in the same folder as the main script. This has made me scratch my head a lot recently so I hope I could get some answers here.

I have realized this is a problem on the server's side so...

There's no server side, you need to have your own magi rpc running.


Fix some mistakes in README

Hi, thanks for creating this excellent library! You have the best Web Audio effects of all the projects I found.
I fixed a few errors I discovered in the documentation. The filters don't have a mix parameter. Also the delayNode defaults to a maxDelayTime of 1 second. You will see warnings in the console if you try to set the time parameter greater than this.
I have subclassed most of the effects for use in my project, except the Ping Pong Delay, which seems to produce wet signal only through the left channel.
Also, Flanger and Quadrafuzz both amplify the signal, so I made customizations to reduce the input gain to compensate. It's a hack but it works, and it means I can apply the effects while keeping the volume level the same. For example:
export default class Quadrafuzz extends Pizzicato.Effects.Quadrafuzz {
  constructor(args) {
    super(args);
    this.inputNode.gain.value = 0.3;
  }
  // ...

Thanks a lot @evoyy 👍
Also the remarks about the effects look very valid, don't hesitate to open issues and/or PRs so we can integrate your changes into the library or track the defects (e.g., signal amplification for Flanger and Quadrafuzz, the one-sided wet signal, etc).


specify which iFrame link to include in rc-test-template.md

rc-test-template.md currently contains this markdown:
- **[Simulation](all_phet_link)**
- **[iFrame](link)**

For Simulation, it explicitly asks for the "all_phet_link".   But it does not specify for iFrame, and there are 2 choices (en_iframe_phet.html, all_iframe_phet.html).  I'm guessing that you want all_iframe_phet.html. But please specify which iFrame link to include.

It should also be all.  I'll adjust the template.

Done in above commit.  Closing


Added XGuardian v0.1

@vitorgalvao It's from the README.md, happy to change it.

@vitorgalvao I'm actually having trouble squashing the commit for the updated homepage, doesn't look like adding a new file plays well with this :-1:

@vitorgalvao It's from the README.md, happy to change it.

The big requirement is that the download URL can be found on the app’s homepage. If there’s one for github there, we can accept it.

@vitorgalvao I'm actually having trouble squashing the commit for the updated homepage, doesn't look like adding a new file plays well with this :-1:

Why adding a new file? What’s the issue? If you can do it without squashing, just do it that way and I’ll see if I can do the squashing on this end.

@vitorgalvao phew, got it! :)   I was forgetting to force the push after squashing...

It still looks the same, though.

@vitorgalvao Ok, got it, is this ok now? thanks

@vitorgalvao k

@vitorgalvao thanks for your patience & pointers, should be good now :)

Thank you for all the work. Looks great.


number type doesn't work correctly with tcomb-form-native

For example, if my form structure is defined as:
var TcombType = transform({
  "type": "object",
  "properties": {
    "Age": { "type": "number" }
  }
});

When creating a small React Native app using tcomb-form-native after inputting a number into the Age field and calling this.refs.form.getValue() it's null and it shows a red box around the Age field.
This works fine if I just use tcomb-form-native directly with a structure like:
var TcombType = t.struct({
  age: t.Number
});

Not sure what's the problem, however 2 notes

The equivalent JSON schema is

{
  "type": "object",
  "properties": {
    "Age": { "type": "number" }
  },
  "required": ["Age"] // <= !
}

make sure you are not importing two different copies of tcomb (for example running npm list tcomb)

I figured this issue out. It was because I had two different versions of tcomb imported as you suspected.
My package.json had:
    "tcomb-form-native": "0.6.7",
    "tcomb-json-schema": "^0.2.5",

Running npm list tcomb showed:
├─┬<EMAIL_ADDRESS>│ └─┬<EMAIL_ADDRESS>│   └──<EMAIL_ADDRESS>└─┬<EMAIL_ADDRESS>  └──<EMAIL_ADDRESS>
I updated the package.json to:
    "tcomb-form-native": "0.6.7",
    "tcomb-json-schema": "^0.3.0"

Then when running npm list tcomb it showed:
└─┬<EMAIL_ADDRESS>  └──<EMAIL_ADDRESS>
Also for the record, the equivalent non JSON schema structure of my original JSON structure is:
t.struct({
  age: t.maybe(t.Number)
});


feat: Add SSH config as an option

Which issue(s) this PR fixes (If applicable): #

Checklist

[x ] I have followed the Contribution Guidelines and Code of Conduct.
[x ] I have commented my code following the Styleguide.
[x ] Relevant documentation is changed or added.
[x ] This is NOT A BREAKING CHANGE.

What kind of change does this PR introduce?

[ ] Bugfix.
[x ] Feature.
[ ] Code style update (formatting).
[ ] Refactoring (no functional changes).
[ ] Build or CI related changes.
[ ] Documentation content changes.
[ ] Project automation.
[ ] Other...

[ ] Please describe:

Description of Change
This PR adds the SSH config to the list of config files supported. The SSH config is frequently changed to add new hosts, so a shortcut function can be helpful.
The addition follows the model of all other configs through the addition of an _emc_ssh function. It also updates the USAGE string and fish completions.
What is the current behavior?
emc.fish does not support ssh config
What is the new behavior?
emc.fish will now support ssh config using the argument ssh
Does this PR introduce a breaking change?
No
Screenshots
Other Information

Hello @bradreaves, thanks for this amazing contribution, merging now. 🚀


dartx not compatible with Flutter master

I'm getting this error now on flutter master:
Because every version of flutter_test from sdk depends on characters 1.0.0 and dartx 0.4.1 depends on characters >=0.4.0 <1.0.0, flutter_test from sdk is incompatible with dartx 0.4.1.
And because no versions of dartx match >0.4.1 <0.5.0, flutter_test from sdk is incompatible with dartx ^0.4.1.
So, because documentation depends on both dartx ^0.4.1 and flutter_test any from sdk, version solving failed.
pub finished with exit code 1

Looks like characters just declared their api stable and published 1.0.0
I'm guessing you can just update the constraint to allow characters: ^1.0.0

Fixed in 0.4.2


Reach out to Aaron Lowe for Public Easement data

Get shapefiles for Public Easements

7/21/17, spoke with Aaron. He said the one-pager was circulated internally at DFAW (Dept of Forestry and Wildlife), of which he is acting director.
A challenge: He said they have deep concern about the project, their office was created to document over 2000 pathways on public land on O‘ahu alone, but they only share info publicly for about 123 of them, because they are fragile ecosystems or have vulnerable cultural artifacts. He said easements will be especially hard because many have unclear status, or contested status. I reassured him that we are not looking to drive masses of tourists to less known trails, and that we want to be extremely sensitive to the host culture and ecological concerns, that including DFAW as a project stakeholder is important to us. Also, he is transitioning out as the acting Director, ad will go back to being the trail manager for O‘ahu, so leadership for the project at DFAW is likely to transition. I let him know our timeframe for public launch as early 2019, so we would be happy to connect and work with whomever is in the top seat.  (Lots of internal bridge building to do here).
He also said that the shapefiles for all of the public trails statewide can be gotten from the City and County GIS Office of Planning, and we are welcome to them for the project. http://planning.hawaii.gov/gis/download-gis-data/
His contact info is<EMAIL_ADDRESS>Division of Forestry and Wildlife
Na Ala Hele Trail and Access Program
Department of Land and Natural Resources
1151 Punchbowl Street
Room 325, Kalanimoku Building
Honolulu, HI 96813
Phone:<PHONE_NUMBER>


YT and Twitter poster broken.

Unfortunately Youtube and Twitter poster don't work.
For Youtube I tried to add my account and upload a test video and it didn't publish it (not even after 10 min or after a bot restart)
For Twitter when I try to add a profile to follow, it tells me UNABLE TO FIND USER

I'm not continuing this project.

You can manually add the user and it will work.

Solved "UNABLE TO FIND USER" by switching to elevated from dev portal but the bot instead of publishing the tweet wrote the setup command WTF xD
Really weird.

Solved "UNABLE TO FIND USER" by switching to elevated from dev portal but the bot instead of publishing the tweet wrote the setup command WTF xD
Really weird.

Can I write to you about discord?

Solved "UNABLE TO FIND USER" by switching to elevated from dev portal but the bot instead of publishing the tweet wrote the setup command WTF xD
Really weird.

Can I write to your discord?

𝗟𝗼𝗴𝗮𝗻#5072


Add missing vsite exclusions

Description
Fixes #40 by adding missing bonds between v-sites and parent atom neighbours to the topology before calling createExclusionsFromBonds on the custom force. Exclusions between sites on the same atom are also added.
Todos
Notable points that this PR has either accomplished or will accomplish.

[X] add tests
[ ] pin toolkit version in new release

Questions

[ ] Question1

Status

[ ] Ready to go

Codecov Report

Merging #41 (d02f944) into main (b4e4ba0) will increase coverage by 1.11%.
The diff coverage is 92.00%.

@@            Coverage Diff             @@
##             main      #41      +/-   ##
==========================================
+ Coverage   78.62%   79.73%   +1.11%
==========================================
  Files           5        5
  Lines         276      301      +25
==========================================
+ Hits          217      240      +23
- Misses         59       61       +2

Flag
Coverage Δ

unittests
79.73% <92.00%> (+1.11%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

smirnoff_plugins/handlers/nonbonded.py
93.95% <92.00%> (-0.32%)
:arrow_down:

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

@j-wags thanks for taking a look over it, I agree I wasn't sure what was happening but I have managed to simplify it ever so slightly by removing the double counting of some topology bonds (that confused me for a while). The code is now trying to add a bond between each v-site and the neighbour of the parent to ensure they get the same exclusions as they have the same bonds. A review would be great so we can stay up to date with the 10-x line of the toolkit.

I can take over reviewing this (I'm working on a parallel implementation right now, actually). An edge case that I come across often is making virtual sites on nearby atoms (but not the same parent) are excluded from each other, i.e.
A0 - A1 - A2 - A3 - A4
     |         |
     V1        V3

Virtual sites V1 and V3 (with parent atoms A1 and A3) are easy to miss if only adding exceptions based on the atoms a parent interacts with.

Interesting I hadn't thought about this case but I would expect it to be handled by createExclusionsFromBonds as long as the bonds V1-A2 and V3-A2 are in the list as it should see them as forming an angle. Ill try and add a test for this case to make sure though.

Yeah, I think Josh is right above - The case that Matt showed should be handled just fine by the current code.
(not blocking) The thing that mostly confuses me about the initial state of this code was how it:

collected all the bonds
made exceptions from bonds (in a world without vsites, I'd think it'd just be done here)
Looked at the existing OpenMM NonbondedForce
Compared the CustomNonbondedForce's exceptions to the NonbondedForce's exceptions (I guess since the NonbondedForce knows about vsites)
Added any exceptions that are in the NonbondedForce but not the CustomNonbondedForce to the CustomNonbondedForce

My confusion was that steps 1-2 seem to be unnecessary/redundant if we're already doing steps 3-5. So I'm thinking that I must be missing something fundamental about how this works.

I ran into something I think related to #40 while working on #46, and merging this in got the simulations running.


self.tabBarController?.tabBar.isHidden = true

how can we preform self.tabBarController?.tabBar.isHidden = true or .hidesBottomBarWhenPushed = true

I have not looked into this but could possibly see if I can come up with a solution. If you have any ideas I am open to looking at a PR. If not I can get around to it soon.


Memcache constructor type does not fully express its valid uses

Got another bump that breaks on typescript compilation, this time on the memcache sub-package (https://github.com/backstage/backstage/pull/10705).
Specific error is:
147       store: new KeyvMemcache(this.connection),
                 ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

  node_modules/keyv-memcache/src/index.d.ts:7:30
    7     constructor(uri: string, opts: KeyvMemcache.Options);
                                   ~~~~~~~~~~~~~~~~~~~~~~~~~~
    An argument for 'opts' was not provided.

@iamEAP - we will get a fix for this out today.

@iamEAP - just published an update to NPM (version 1.3.2). Can you validate this is fixed?


Added Color Name and Hue tool Update index.md

[ ] I have read freeCodeCamp's contribution guidelines.
[ ] My pull request has a descriptive title (not a vague title like Update index.md)
[ ] My pull request targets the master branch of freeCodeCamp.
[ ] None of my changes are plagiarized from another source without proper attribution.
[ ] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

Congrats @Tripsev on your first pull request (PR)! 🎉
Thank you for your contribution to the page! 👍
We're happy to accept these changes, and look forward to future contributions. 📝


preferredStatusBarStyle not call when push view if swipe left menu not completed

I set status bar for specific view controller, main view is light but child is black, when I swipe left menu a little and swipe close immediately, I push to child view and method preferredStatusBarStyle not called, my status bar not change.
Please help me, thanks.

It work when I merge this pull
https://github.com/Friend-LGA/LGSideMenuController/pull/112

thanks for pointing out the fix


connected db with the code and used it whereever it is

need to inside get and post operatio
ns

@Codium-Agent /review

@CodiumAI-Agent /review

@CodiumAI-Agent /review

PR Reviewer Guide 🔍

⏱ Estimated effort to review: 3 🔵🔵🔵⚪⚪
🧪 No relevant tests
🔒 Security concerns
Sensitive information exposure: The MongoDB URI contains credentials and is exposed in the .env file, which is included in the version control. This can lead to unauthorized access to the database. Consider removing sensitive information from the codebase and using environment variables set outside of the version-controlled environment.
⚡ Key issues to review
Hardcoded URIThe MongoDB URI is hardcoded in the .env file with credentials, which is a security risk. Consider using environment variables to manage sensitive data securely.
Error HandlingThe error handling in the connectDB function logs the error but does not handle it further. Consider throwing the error or handling it in a way that the server can respond appropriately to the client.
Console LoggingDirect use of console.log for debugging and error messages in production code can lead to exposure of sensitive information. Consider using a logging framework that supports different levels of logging and is configurable based on the environment.


Add logging

I'm debugging an issue where a redb database is very slow to open, and being able to turn on logging for redb would be nice, to get some insight into what it's doing.

Is there any logging functionality in the Rust standard library? It looks like I need to use the log crate, which I guess is ok, but I'm kinda proud of having only libc as a dependency :grin:

There isn't, although the log crate is owned by the Rust project, so it's close to being official. But I actually don't need this anymore personally, so I wouldn't consider it high priority.


cannot import name 'EVENT_TYPE_OPENED' from 'watchdog.events'

@al1020119 is this because of an old version of watchdog? https://stackoverflow.com/questions/76228417/cannot-import-name-event-type-opened-from-watchdog-events


ERROR: connect ECONNREFUSED <IP_ADDRESS>:4444

Hi,
I am getting ERROR: connect ECONNREFUSED <IP_ADDRESS>:4444 while using the wdio-docker-service. Below is my Dockerfile:
`FROM ubuntu:latest
LABEL maintainer="Ankit Ranjan"
ENV LC_ALL=C
ENV DEBIAN_FRONTEND=noninteractive
ENV DEBCONF_NONINTERACTIVE_SEEN=true
RUN groupadd --gid 1000 node
&& useradd --uid 1000 --gid node --shell /bin/bash --create-home node
RUN apt-get -qqy update
RUN apt-get -qqy install
apt-transport-https
ca-certificates
curl
software-properties-common
RUN curl -sS -o - https://dl-ssl.google.com/linux/linux_signing_key.pub | apt-key add -
RUN echo "deb http://dl.google.com/linux/chrome/deb/ stable main" >> /etc/apt/sources.list.d/google-chrome.list
RUN echo "deb http://archive.ubuntu.com/ubuntu xenial main universe\n" > /etc/apt/sources.list
&& echo "deb http://archive.ubuntu.com/ubuntu xenial-updates main universe\n" >> /etc/apt/sources.list
&& echo "deb http://security.ubuntu.com/ubuntu xenial-security main universe\n" >> /etc/apt/sources.list
RUN apt-get -qqy update
RUN curl -sL https://deb.nodesource.com/setup_8.x | bash -
RUN apt-get -qqy --no-install-recommends install
nodejs \
firefox \
google-chrome-stable \
openjdk-8-jre-headless
x11vnc
xvfb
xfonts-100dpi
xfonts-75dpi
xfonts-scalable
xfonts-cyrillic
RUN export DISPLAY=:99.0
RUN Xvfb :99 -shmem -screen 0 1366x768x16 &
WORKDIR /home/node
ADD . .
RUN chown node:node -R .
RUN chmod 777 -R .
RUN google-chrome --version
RUN firefox --version
RUN node --version
RUN npm --version
USER node
RUN npm install
ENTRYPOINT ["tail", "-f", "/dev/null"]`
Below is my docker-compose file:
`version: "2"
services:
testapp:
build:
context: .
dockerfile: Dockerfile
networks:
- testing-lan
seleniumappchrome:
image: selenium/standalone-chrome-debug
volumes:
- /dev/shm:/dev/shm
ports:
- "4444:4444"
- "5900:5900"
networks:
- testing-lan
networks:
testing-lan:
driver: bridge`
And below is my wdio-conf.js file:
`exports.config = {
services: ['docker'],
specs: [
    'bdd-tests/features/featureFiles/*.feature'
],

maxInstances: 2,
capabilities: [{
    maxInstances: 1,
    //
    browserName: 'chrome'
},

],
dockerOptions: [{
image: 'selenium/standalone-chrome-debug',
healthCheck: {
url: 'http://<IP_ADDRESS>:4444/wd/hub',
inspectInterval: 1000,
startDelay: 15000
},
options: {
p: ['4444: 4444', '5900:5900'],
shmSize: '2g'
}
},
],
framework: 'cucumber',
reporters: ['cucumber-json', 'spec'],

reporterOptions: {
    outputDir: 'bdd-tests/output/jsonReport',
},
// If you are using Cucumber you need to specify the location of your step definitions.
cucumberOpts: {
    require: ['./bdd-tests/features/step-definitions/*.js'],// <string[]> (file/dir) require files before executing features
    backtrace: false,   // <boolean> show full backtrace for errors
    compiler: [],       // <string[]> ("extension:module") require files with the given EXTENSION after requiring MODULE (repeatable)
    dryRun: false,      // <boolean> invoke formatters without executing steps
    failFast: false,    // <boolean> abort the run on first failure
    format: 'json', // <string[]> (type[:path]) specify the output format, optionally supply PATH to redirect formatter output (repeatable)
    colors: true,       // <boolean> disable colors in formatter output
    snippets: true,     // <boolean> hide step definition snippets for pending steps
    source: true,       // <boolean> hide source uris
    profile: [],        // <string[]> (name) specify the profile to use
    strict: false,      // <boolean> fail if there are any undefined or pending steps
    tags: [],           // <string[]> (expression) only execute the features or scenarios with tags matching the expression
    timeout: 20000,     // <number> timeout for step definitions
    ignoreUndefinedDefinitions: false, // <boolean> Enable this config to treat undefined definitions as warnings.
}

}
And below is the error I am getting:[16:36:57] Using gulpfile ~/gulpfile.js
[16:36:57] Starting 'cucumber'...
[16:36:57] Starting 'prepareReportFolders'...
[16:36:57] Finished 'prepareReportFolders' after 5.99 ms
[16:36:57] Starting 'selenium'...
[16:36:57] Finished 'selenium' after 3.24 ms
[16:36:57] Starting 'wdio'...
[16:36:57]  COMMAND	POST 	 "/wd/hub/session"
[16:36:57]  DATA		{"desiredCapabilities":{"javascriptEnabled":true,"locationContextEnabled":true,"handlesAlerts":true,"rotatable":true,"maxInstances":1,"browserName":"chrome","loggingPrefs":{"browser":"ALL","driver":"ALL"},"requestOrigins":{"url":"http://webdriver.io","version":"4.14.4","name":"webdriverio"}}}
ERROR: connect ECONNREFUSED <IP_ADDRESS>:4444
chrome
at new RuntimeError (/home/node/node_modules/webdriverio/build/lib/utils/ErrorHandler.js:143:12)
at Request._callback (/home/node/node_modules/webdriverio/build/lib/utils/RequestHandler.js:342:43)
at self.callback (/home/node/node_modules/request/request.js:185:22)
at emitOne (events.js:116:13)
at Request.emit (events.js:211:7)
at Request.onRequestError (/home/node/node_modules/request/request.js:877:8)
at emitOne (events.js:121:20)
at ClientRequest.emit (events.js:211:7)
at Socket.socketErrorListener (_http_client.js:401:9)
at emitOne (events.js:116:13)
at Socket.emit (events.js:211:7)
at emitErrorNT (internal/streams/destroy.js:73:8)
at _combinedTickCallback (internal/process/next_tick.js:139:11)
at process._tickDomainCallback (internal/process/next_tick.js:219:9)
Wrote json report to [bdd-tests/output/jsonReport].
[16:36:58] 'wdio' errored after 906 ms
[16:36:58] Error in plugin 'gulp-webdriver'
Message:
wdio exited with code 1
[16:36:58] Finished 'cucumber' after 922 ms`
Please help me to find if I am missing anything.
Thanks,
Ankit Ranjan

@ariesranjan If you're trying to run tests agains a containerized app, you still need a selenium to run a test with. Look at one of the examples in the wiki https://github.com/stsvilik/wdio-docker-service/wiki/Testing-dockerized-application

@ariesranjan It is hard to discern what is that you're trying to do. Before dumping so much code, please explain your objective and I'll try to help you out as much as I can. Closing this for now.


📖  Use explicit releases instead of main

Summary
This PR is a bit of cleanup after #2515 . This PR does two things.

Use an explicit version in the reference to the OCM Quick Start.
Note that the upcoming release should replace the reference to KubeStellar main with that release.

Preview at https://mikespreitzer.github.io/kcp-edge-mc/doc-release-for-ocm/direct/release/
Related issue(s)
Fixes #

/approve