BUG: Enhancement of the Header

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
The current header of the application needs enhancements to improve its design, functionality, and user experience. This includes updating the layout, adding new features, and ensuring it is responsive across all devices.
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

Ok


Revert "Merge Development into Master"

Reverts fossasia/open-event-webapp#1001
It is making development still 723 commits behind master. Wrong procedure.

Sorry for this but the merge #1001 was wrong.

@aayusharora I am extremely sorry. Could you please explain the correct procedure?


[Storage] Move live test resources to correct subscription

Check what storage account the resources are under

Move resources to Azure SDK Test Resources subscription under storage-js resource group

^need to update the storage account name -> azuresdktestjs [After Harsha confirms if they want separate endpoint for Java or same]


Document a missing event gs.down.schedule.attempt

Summary
Document the gs.down.schedule.attempt event here
Can you do this yourself and submit a Pull Request?
Nope

This is done.


support babel!

For example:
mocha-phantomjs test/mocha-runner.html --compilers js:babel-core/register --recursive

mocha-phantomjs runs in the browser, not Node.js, that is not possible how you have it.
It's also not possible to do this generically, nor should it. The biggest difference between karma and mocha-phantomjs is that it gives you full control over your test runner HTML. Whatever works by going to your tests in a browser using Mocha will work with mocha-phantomjs (barring phantomjs issues, which there are quite a bit, less and less with phantomjs 2). So IIRC babel can work in the browser, so get that working, and it's working in mocha-phantomjs too.


Copy UnityFramework.framework: No such file or directory [IOS]

Hi
I try to integrate the unity project using this library and get Copy UnityFramework.framework: No such file or directory.
IOS 15, Xcode 13, emulator.

Copy UnityFramework.framework: No such file or directory.

Hi
I don't understand - why copy it?

I got that error while trying to build the app.
On real device all works

yes, this component only works on a real device. I didn't even try to make it work in emulators)

probably we should add this information to readme?)
I lost a few hours to try start it on emulator

Ok I'll add!
but I think that you can still run it - I just didn’t have such a task


Perf: improve performance of filtering large folders in the explorer

The explorer filter can be very expensive because for each file we need to resolve the list of actual sibling names. This results in quadratic complexity and we should revisit this to not always have to do this.
Code pointer: https://github.com/Microsoft/vscode/blob/a8f068a5400b6b7907fa9763615e13e6de7f195d/src/vs/workbench/parts/files/electron-browser/views/explorerViewer.ts#L691

We were focused on grid this milestone, next week I am on vacation, this is not for endgame -> June

@isidorn I looked a bit into this, some findings:

the tree is calling the isVisible method twice (filed https://github.com/Microsoft/vscode/issues/51601)
we could save a lot of time if the stat was keeping a Map of children names for explorer items that are folders, would that be possible? We could even compute that lazily when needed and I could wire this all the way through to the glob matching to make this fast
something for the future maybe: @joaomoreno any chance that the tree could not call isVisible for each item even if not visible in the viewport? That of course has the biggest perf impact

@bpasero No chance, because that affects the tree's content height.

@bpasero yes we could add a lazy map of children directories to the ExplorerItem.
Would you expect something like
item.isChildDirectory(name: string): boolean
Which when first called constructs the map and then all subsequent calls return in O(1)?

@isidorn the glob library actually needs to have a way to know if a specific filename is part of a folder, so currently what it is doing is string[].indexOf(filename)

Specific filename is part of a folder

then why not stat.parent.getChild(fileName) here
I guess I am missing something, feel free to drop my desk to discuss.

@isidorn that looks very promising indeed, I could try that. The only worry I have is that it calls toLowerCase each time for the file name it seems.

@bpasero great, please try that. Yes it calls toLowerCase each time on non-linux platforms. There is nothing better we can do than to call toLowerCase

The glob.ts code is so insanely complicated and tweaked that I am not able to make a change where siblingsFn?: () => string[] | TPromise<string[]> could change to something like hasSinlingsFn?: (name: string) => boolean | TPromise<boolean> without knowing what is going on, so I will have to defer this to @chrmarti who I believe added these optimizations to glob.ts to see if it is feasable to make this change. The idea is to just provide this function instead of having to compute the siblings. Clients can still decide to behave as we did before by doing an indexOf check in the array of sibling names, but others like the explorer can implement a potentially faster check by going directly to the model.

Fixed via https://github.com/Microsoft/vscode/pull/53994

Verify

files.exclude setting works as before for the explorer
also test with complex expressions like "**/*.js": { "when": "$(basename).ts" }


API doesn't return lists (categories, sections etc) in specified order.

Is this for help center, or classic forums? Any endpoint specifically? We should have a fixed order for the default calls to endpoints.

Help Center, https://*.zendesk.com/api/v2/help_center/articles
Given that the order is configurable via the UI, I believe the API should return in the specified order. Otherwise, one would need to recreate the ordering service, redundantly store the order of resources and then apply that order after the fact.

Hi, are you still seeing this problem? If so, can you send an email to api_at_zendesk.com and we'll look into your case more specifically. Thanks!


⚠ Processors has degraded performance

In d2f2800, Processors ($STATUS_URL) experienced degraded performance:

HTTP code: 200
Response time: 73 ms

Resolved: Processors performance has improved in 37eaadb.


@searchable can not be addded to the model

How did you install the Amplify CLI?
npm
If applicable, what version of Node.js are you using?
v21.6.1
Amplify CLI Version
12.10.1
What operating system are you using?
Mac
Did you make any manual changes to the cloud resources managed by Amplify? Please describe the changes made.
Do not remember exactly.
Note: The environment was cloned from another one.
No something special
Describe the bug
have a
type Recorder
@model
@auth(rules: [...]) {
{
id: ID!
firstName: String
...
}
Add @searchable to it like:
type Recorder
@model
@searchable
...
Expected behavior
@searchable should be added to the model
Reproduction steps

Add @searchable to the model
amplify push

Project Identifier
26019afdc69d49143fe99441bf73b9d3
Log output

        Deployment failed.
        appname             AWS::CloudFormation::Stack     UPDATE_FAILED                  Thu Jan 25 2024 19:10:26…

        SearchableStack                AWS::CloudFormation::Stack     CREATE_FAILED                  Thu Jan 25 2024 19:10:08…
        ConnectionStack                AWS::CloudFormation::Stack     UPDATE_COMPLETE                Thu Jan 25 2024 19:11:52

🛑 The following resources failed to deploy:
Resource Name: OpenSearchDomain (AWS::Elasticsearch::Domain)
Event Type: create
Reason: Invalid instance type: t2.small.elasticsearch (Service: AWSElasticsearch; Status Code: 409; Error Code: InvalidTypeException; Request ID: fde79bcf-4aeb-4e7c-b8a9-14eba3e9a3aa; Proxy: null)

🛑 Resource is not in the state stackUpdateComplete
Name: OpenSearchDomain (AWS::Elasticsearch::Domain), Event Type: create, Reason: Invalid instance type: t2.small.elasticsearch (Service: AWSElasticsearch; Status Code: 409; Error Code: InvalidTypeException; Request ID: fde79bcf-4aeb-4e7c-b8a9-14eba3e9a3aa; Proxy: null), IsCustomResource: false

Session Identifier: 7060d53c-d34b-4803-aa0b-2832b0dd3f7c

Additional information
Checked on the little bit older amplify and node.js version after that do updated it to the latest and the issue is still the same.
Before submitting, please confirm:

[X] I have done my best to include a minimal, self-contained set of instructions for consistently reproducing the issue.
[X] I have removed any sensitive information from my code snippets and submission.

Hi @biloshkurskyi-ss, I am unable to replicate the issue on my end with the same CLI version 12.10.1. When I add searchable to a model and deploy, I'm able to deploy a cluster with t2.small.search instance type.
Can you check your amplify/team-provider-info.json file and see if you have any overrides for instance type setup in your project? Doc link

Hello @phani-srikar,
Thank you for checking on your end.
Today I have tried to do a push and it still doesn't work.
 SearchableStack                AWS::CloudFormation::Stack     CREATE_FAILED
I've reviewed the app settings in the team-provider-info.json file and noticed that the OpenSearchInstanceType parameter is missing. I was under the impression that it would be automatically added after implementing @searchable.
 dev": {
    "awscloudformation": {
      "AuthRoleName": "amplify-app-name-xxxx-dev-xxxxxxx-authRole",
      "UnauthRoleArn": "arn:aws:iam::XXXXXXXXXX:role/amplify-app-name-xxxx-dev-xxxxxxx-unauthRole",
      "AuthRoleArn": "arn:aws:iam::XXXXXXXXXX:role/amplify-app-name-xxxx-dev-xxxxxxx-authRole",
      "Region": "xx-xxxx-x",
      "DeploymentBucketName": "amplify-app-name-xxxx-dev-xxxxxxx-deployment",
      "UnauthRoleName": "amplify-app-name-xxxx-dev-xxxxxxx-unauthRole",
      "StackName": "amplify-app-name-xxxx-dev-xxxxxxx",
      "StackId": "arn:aws:cloudformation:xx-xxxx-x:XXXXXXXXXX:stack/amplify-app-name-xxxx-dev-xxxxxxx/xxxxx-xxxxx-xxxxx",
      "AmplifyAppId": "xxxxxxxxxx"
    },
    "categories": {
      "auth": {
        "app-name-xxxx": {}
      },
      "api": {
        "app-name-xxxx": {}
      },
      "storage": {
        "s3app-name-xxxxstoragexxxxxxx": {}
      }
    }
  }

What additional information or settings would be helpful for you to investigate this issue further?
Also, could you please check if the system logs have been shared? If they haven't been exported, could you guide me on how to do so? I assumed they would be automatically transferred with the Session Identifier or a similar method. Is there a way for me to manually push these logs?
The latest logs:
The following resources failed to deploy:
Resource Name: OpenSearchDomain (AWS::Elasticsearch::Domain)
Event Type: create
Reason: Invalid instance type: t2.small.elasticsearch (Service: AWSElasticsearch; Status Code: 409; Error Code: InvalidTypeException; Request ID: 7d16c53d-2458-49cb-8541-eb9b4e0637bf; Proxy: null)

🛑 Resource is not in the state stackUpdateComplete
Name: OpenSearchDomain (AWS::Elasticsearch::Domain), Event Type: create, Reason: Invalid instance type: t2.small.elasticsearch (Service: AWSElasticsearch; Status Code: 409; Error Code: InvalidTypeException; Request ID: 7d16c53d-2458-49cb-8541-eb9b4e0637bf; Proxy: null), IsCustomResource: false

Session Identifier: a18a3328-116e-4f25-9f64-3af675f66339

Hey @biloshkurskyi-ss, Thank you for checking and providing the team-provider-info file. Can you please confirm the region where you are deploying this app?

The eu-north-1 region is used.

Hey @biloshkurskyi-ss, Appreciate your confirmation regarding the region. We've successfully replicated this issue in the 'eu-north-1' region, so we're flagging it as a bug for the team to investigate further.

@biloshkurskyi-ss, As a workaround, you can perform the following steps to switch the search instance type to t3.small.elasticsearch and inform us if it resolves your issue.

Run amplify override api.
Insert the following code snippet into \amplify\backend\api\<api name>\override.ts.

import { AmplifyApiGraphQlResourceStackTemplate, AmplifyProjectInfo } from '@aws-amplify/cli-extensibility-helper';

export function override(resources: AmplifyApiGraphQlResourceStackTemplate, amplifyProjectInfo: AmplifyProjectInfo) {
    resources.opensearch.OpenSearchDomain.elasticsearchClusterConfig = {
      ...resources.opensearch.OpenSearchDomain.elasticsearchClusterConfig,
      instanceType: "t3.small.elasticsearch",
    };

}

Hello @AnilMaktala  the workaround is working in dev and prod environments. Thanks for your help! 🤝


Add feature to capture button held pins from board

This PR adds a new functionality to the Pin Mapping configuration screen. It allows the user to press buttons on their controller to assign pins. It can be done for each button individually or all buttons one-by-one.
Currently, as per the implementation, it initializes all valid pins. Within a window of 5 seconds, if the user does nothing, it returns empty. Before anything is returned, all unused pins are also uninitialized.
Please review and merge, thanks!

So awesome, love this functionality!
I have some structural opinions that might help make this more reusable.
That or i'm missing something :)
It feels somewhat redundant that we have handlePinChange and getHeldPins.
We should be able to separate the recording to a hook or a button, i made a quick example here for the specific inputs(not for all) but i do believe we can use it.
This example also shows how to solve a bug that we have now. The request keeps going even if you change page, leaving it hanging.
Also happens when pressing Capture all button pins
(i will send video on discord as the file size is to big)
The only thing that is needed for this to work on the pinMapping page is.
<CaptureButton
  onChange={(pin) =>
    handlePinChange({ target: { value: pin } }, button)}
/>

With this we should also be able to use it on other pages.
Here is the captureButton
import axios from 'axios';
import React, { useEffect, useRef, useState } from 'react';
import { Button, Modal } from 'react-bootstrap';
import { useTranslation } from 'react-i18next';

const CaptureButton = ({ onChange }) => {
	const { t } = useTranslation('');
	const controller = useRef<AbortController>();
	const [modalVisible, setModalVisible] = useState(false);

	const newAbortSignal = () => {
		controller.current = new AbortController();
		return controller.current.signal;
	};

	const getHeldPins = async () => {
		setModalVisible(true);
		const { data } = await axios
			.get('http://localhost:8080/api/getHeldPins', {
				signal: newAbortSignal(),
			})
			.then(({ data }) => {
				onChange(data?.heldPins?.at(0));
				setModalVisible(false);
			});

		onChange(data?.heldPins?.at(0));
		setModalVisible(false);
	};

	useEffect(() => () => controller?.current?.abort(), []);

	const closeModal = () => {
		controller?.current?.abort();
		setModalVisible(false);
	};

	return (
		<>
			<Modal show={modalVisible} onHide={() => closeModal()}>
				<Modal.Header closeButton>
					<Modal.Title>Waiting for button press</Modal.Title>
				</Modal.Header>
				<Modal.Body>

				</Modal.Body>
				<Modal.Footer>
					<Button variant="danger" onClick={() => closeModal()}>
						Cancel
					</Button>
				</Modal.Footer>
			</Modal>
			<Button
				className="ms-3"
				variant="secondary"
				onClick={() => getHeldPins()}
			>
				{t('PinMapping:capture-button-label')}
			</Button>
		</>
	);
};

export default CaptureButton;

Also! My suggestion can be a future improvement, no need to do it in this PR if it's out of scope

@Pelsin thank you very much!
These are great suggestions, I'll try to get them integrated.

Testing with a FlatboxRev5RGB
The individual buttons works exactly as intended. Loaded incorrect firmware to controller and after navigating to the Web Configurator, performed the following steps.

Press button on screen
Prompted for input
Press button on controller
Pin is mapped on controller
Repeated steps 1-4 until all buttons were mapped
Once save button on page is pressed and controller restarted, the controller works as intended.

Issue
When using the Assign Gamepad Pins button in the Configuration>Pin Mapping page, initially it will register the negative edge of button presses as additional button presses. This means that the process is incremented by 2 steps when only 1 is met.
After a period of time leaving the Assign Gamepad Pins feature will proceed, but the mapping process stops registering inputs consistently. It will sometimes read inputs and others, it will time out until the next input comes up. Once this occurs, I am unable to navigate to any other pages in WebConfig, nor does refreshing the page resolve the issue.
Recommended Changes
The labels used in the modal dialog for mapping uses GP2040-CE labels regardless of what the Label Dropdown is set to. It would be helpful to have the labels match with the user selection

@InfraredAces thank you very much for testing and a very elaborate report!
Turns out we needed a debouncer for the pins.
Also the labels have been corrected.

As discussed in the Discord, on RP2040 boards that have access to Pin 29, this feature does not work when assigning buttons to Pin 29 and require manually entering the pin for assignment
Otherwise I was not able to find issue with the PR. All other pins are assignable by pressing the button, both individually and when entering then sequentially using the Assign Gamepad Pins button in WebConfig.

That is really interesting, and makes me think about something maybe we should add to BoardConfig.h
Would it make sense to add an array definition, something like
#define BOARD_AVAILABLE_PINS 0,1,2,3,4,5...24,25,26,27,28,29
then we can just wrap that in a static const uint32_t boardPins[] = {BOARD_AVAILABLE_PINS}; ?
For this PR, I'm going to merge it since its working as expected. Then I'll make a new issue to add board pin definitions to BoardConfig.h

That sounds good to me!
Thank you for noticing the issue in WebApi.js, this has been reverted.


[release-1.8] Fix concurrent sort of SNI hosts and CIDR

This is an automated cherry-pick of #30628

@istio-testing: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-telemetry-mc-k8s-tests_istio_release-1.8
9a24d363d865dd38187a106d5f301bdda2ed6cbb
link
/test integ-telemetry-mc-k8s-tests_istio_release-1.8

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


Closes #40 Adds scattering read operation to OHSA

test this please - first run failed due to virtual machine timing crappiness


Use AES256.

Use AES256.
closes #15

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


bots: Do not show not_subscribed warning for bots on private streams.

Fixes #9373.
I noticed that test for needs_subscribe_warning was msising, so added a test in the second commit. I divided this into two commits as the test was added for the whole needs_subscribe_warning and not just the is_bot change (let me know if this needs to be changed).

Nice, merged, thanks @shubham-padia!


abapGit format of abap2oauth2

Hi Ivan, just to let you know that I converted your ABAP objects in abapGit format here : https://github.com/sandraros/abap2oauth2. Please feel free to make it your own Github repository, so that I can fork it.
Sandra

Converted ready to be forked


Direct Integration of shape-LOD2 or CityGML-LOD2 data instead of converting to geoJSON in QGIS

Currently it's necessary to convert downloaded LOD2 data to geoJSON first to use it with the software as problems were encountered with shape-files. This issues needs to be resolved for a userfriendly software

https://github.com/citygml4j/citygml-tools
bundle or use docker image:

install https://hub.docker.com/r/citygml4j/citygml-tools (docker and WSL needed)
use

$ docker run --rm -u 1000 -v /path/to/your/data:/data citygml-tools to-cityjson *.gml

citygml4j/citygml-tools:latest
Note that /data is the default working directory inside the container. Relative paths to input files like in the above example are automatically resolved against /data by citygml-tools. If you mount your local directory at a different path inside the container, you must specify the full path to your input files, of course.


[SPARK-44618][INFRA][FOLLOWUPS] Make free_disk_space uninstall two unused packages completely

What changes were proposed in this pull request?
Make free_disk_space uninstall two unused packages completely
Why are the changes needed?
dpkg: warning: while removing php8.1-fpm, directory '/usr/share/php' not empty so not removed
...
dpkg: warning: while removing powershell, directory '/usr/local/share/man/man1' not empty so not removed
Purging configuration files for azure-cli (2.50.0-1~jammy) ...
dpkg: warning: while removing azure-cli, directory '/opt/az' not empty so not removed

we can not remove '/usr/local/share/man/man1' for powershell:

this path contains docs for other packages;
after removing it, powershell requires to remove '/usr/local/share`

so let powershell alone
Does this PR introduce any user-facing change?
no, infra-only
How was this patch tested?
updated CI and manually check

thanks, merged to master


Bump axe automated accessibility audit tools

Update the axe automated accessibility testing libraries to newer versions.
Bumps jest-axe from 3.5.0 to 5.0.1

Changelog
Commits

This also bumps axe-core from 3.5.5 to 4.2.1:

Release notes
Changelog
Commits

axe-core has a newer release, v4.3.2, but jest-axe pins us to v4.2.1. However there appear to be no changes between the two versions that affect us; I read the changelog and tested with both versions.

Note that this doesn't introduce any new test failures. We had a conversation recently about bumping axe in the alphagov/govuk-design-system repo, which does introduce test failures; we would expect to see some of those test failures recreated here, but we don't, which suggests the existing accessibility test coverage is not expansive enough.


The new site doesn't seem to show everyone in the same household.

Quite a few of these, not sure how to test?

This has happened before and it related to an error in the file for FC1
where all the occupants were all as one person households.
It was corrected and re uploaded.
It is one of those that I really need to know the Piece number to correct
it if it has not already been done.
Do you have that information?
On Sat, 16 Feb 2019 at 14:18, DeniseColbert<EMAIL_ADDRESS>wrote:

Quite a few of these, not sure how to test?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/FreeUKGen/FreeCENMigration/issues/610, or mute the
thread
https://github.com/notifications/unsubscribe-auth/ANC91s2ecEGs77dnIgZR-LjKJXDHnHNZks5vOBMogaJpZM4a_Afi
.

We need to have specific examples here or test cases.

If this is an 1841 issue then it is related to teh schedule number 0 problem. This problem is affecting both vld download and the display of vld search results to researchers.
Geoff

Does this problem still exist? The rewriting of the code that identified the break between households has been re-written and has removed several anomalies. This issue may also have been fixed by this re-write.
Geoff

The 1841 breakup problem has been resolved.
I have not come across the problem elsewhere.
I think that this issue can be closed.
Geoff

Closing as complete


Update docs to aviod using "simple"

I think it's great to avoid using words like “easy”, “simple” and “basic” in docs, especially when Introducing a new method of a thing, because if users have a hard time completing the task that is supposedly “easy,” they will question their abilities. And we want people to feel comfortable learning.

Hey @Ekwuno! Thanks for the PR.
I agree with you we don't want to make developers feel uncomfortable.  In this case, these sentences aren't referring to tasks a user is completing.
The first is calling the sandbox simple, which is true - it doesn't have robust configuration options available, which are available when configuring a script via Wrangler.
The second is referring to the Workers script we've defaulted to showing there, which is indeed the least complex script you can write with a worker.  Simple serves to tell developers that there are more complex Workers they could write in the sandbox, and that this is a starting point for them.
If you have other examples where we're implying that a task is actually easy, let me know!

Hey, @greg-mckeon I see what you mean, but I was trying to point out that using "simple" in docs can be a little off-putting, especially when the developer reading it doesn’t understand what just happened.
Don’t you think that if someone saw the "simplest" worker function and couldn’t understand it because maybe they didn't understand for example, what the respondWith runtime API was doing. It wouldn’t encourage them to even look at the complex ones?

Yep, that's a good point! I agree we should make our description of what's happening here a bit better, since it's a starting point for the platform (and especially since it re-uses the word essentially twice :) ).

Awesome! Let's get more people comfortable with the docs. Thanks for taking the time to review this.


Document undocumented admin test helpers

This PR adds all the remaining test helpers from packages/admin/.stubs.php to the documentation.
In the Stub file, I moved the two label assertions next to teach other like all the other paired assertions.
Will PR the other sections separately.

Added another PR for Table assertions #6263

Thanks!


How to install

There are no installation instructions anywhere
How do I properly install this library?

Hello,
Since i do not have published the library to any maven repository there is no "simple" way to add it to your project. This is an android Library. You have to clone the repository and then import it your project.
Clone the repository somewhere :
git clone https://github.com/L-Briand/ShadowLayout

To reference the external lib project, Insert this 2 lines on setting.gradle:
include ':ShadowLayout'
project(':ShadowLayout').projectDir = new File('/path-to-lib/ShadowLayout')

Then you can add it to the dependencies of your project:
dependencies {
    implementation project(':mylibrary')
}


Ability to change default search sort order

Description
It'd be nice to set what you want the default sort order for searches to be. I always prefer most recently updated rather than "best"
Screenshot

Example URL
https://github.com/search?q=asd

Unfortunate Refined GitHub is not about options: https://github.com/refined-github/refined-github/issues/2960#issuecomment-606765658


Remove: radium.launcher version 1.0.3

## Entire domain is 404

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

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

Microsoft Reviewers: Open in CodeFlow

URL: https://toine.ml/app/public/radium/download/Radium-Launcher-win-x64.exe
No such host is known. (toine.ml:443)
(Automated message - build 703)

Hi @Trenly,
Somehow this didn't trigger as a removal PR. This is the highest version of this PR in the repo - is it available from another location?


11970 web socket port

Sure.. but I do not think the Singleton() function is ever invoked other than in the WebSocket context.I'm here tomorrow, Monday
On Sunday, February 6, 2022, 03:43:09 PM PST, John R. Wolfe<EMAIL_ADDRESS>wrote:

@johnrwolfe requested changes on this pull request.
Let's discuss the singleton hack.
The rest of the requested changes are minor.
In src/tool-core/models/tool-core/architecture/classes/Attribute/Attribute.xtuml:

@@ -54,6 +54,7 @@ INSERT INTO O_TFR
'select any accessor related by self->AttributeAccessor[R4510.''value accessed through'']
where ( selected.accessor_type == AttributeAccessorType::GETTER );
gettername = accessor.getName();
+key = STRING::substr( s:self.name, begin:2, end:-1 );

While we both remember why this here right now, I think this line deserves a comment.
In src/tool-core/models/tool-core/architecture/classes/Attribute/Attribute.xtuml:

end if;
end if;
+key = STRING::substr( s:self.name, begin:2, end:-1 );

This one also deserves a comment.
In src/tool-core/models/tool-core/architecture/interfaces/Message/Message.xtuml:

@@ -56,6 +56,7 @@ while ( not_empty parm )
invocation_parameter_list = invocation_parameter_list + separator + parm.name;
separator = ", ";
pname = parm.name;

key = STRING::substr( s:pname, begin:2, end:-1 );

Comment, please.
In src/tool-core/models/tool-core/architecture/interfaces/Port Message/Port Message.xtuml:

@@ -149,6 +151,7 @@ while ( not_empty msg_parm )
end if;
message_parameter_list = message_parameter_list + sep + parm_dt.type_reference_name + ".deserialize(" + accessor + ")";
end if;

key = STRING::substr( s:pname, begin:2, end:-1 );

Comment, please
In src/tool-core/templates/component/t.componentdefinition.java:

@@ -6,8 +6,14 @@

 private Map<String, Class<?>> classDirectory;

public ${self.name}(IApplication app, IRunContext runContext, int populationId) {

private static ${self.name} singleton;

Hmmm. Not sure we want to make this hack unconditional. I was thinking that it would be used only when necessary (i.e., for models using WebSocket ports). Using it unconditionally means that this and all subsequent releases of the MT version of Ciera will not be capable of supporting deployments with more than one instance of each component.
Let's discuss this and decide how to proceed.
In src/tool-sql/templates/application/t.application.java:

public class ${self.name} implements IApplication {

private IComponent<?>[] components;

private IComponent<?>[] components;

Replace tab with spaces
—
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.
You are receiving this because you authored the thread.Message ID: @.***>


support multiple idempotent request payload

Hello,
I added support for multiple annotations: JdempotentRequestPayload

Made adjustments to unit tests and a review of Demo projects.
Updated the project structure: Jdempotent-spring-boot-couchbase-starter and Jdempotent-spring-boot-redis-starter
Added jacoco framework and owasp
Upgraded dependencies to mitigate vulnerabilities that had in the project.
Updated some points in the readme.

I versioned the project in 2.0.0 due to the high change that was made.
I await your return.

Issues:

#13
#23

Last commit removes the error that occurs in Java 17 with primitive objects.
Error: InaccessibleObjectException occurs when trying to call the following method: setAccessible.
Ex Error: java.lang.reflect.InaccessibleObjectException: Unable to make field private final byte[] java.lang.String.value accessible: module java.base does not "opens java.lang" to unnamed module


Add cards example.

I like this example and I think that at some point we should probably put something like the Automerge struct in the library proper.
It's probably worth removing some or all of the comments which (I think) are left over from the JS version of this example. They're confusing as the API is a bit different.

Last nit, can you add a Default implementation for the Automerge struct to satisfy clippy?


Empty cards

We want to display at least 3 fields for each card as described in #857.

Think the implementation could be improved but merging this for our test build.


Implements visual accessibility features for designer and companion.

Added features in the designer and companion for visual accessibility

HighContrast option in Form, sets some visual components to a high contrast color
BigDefaultText option in Form, increases the default font size of components
AlternateText option in Image, allows the user to write TalkBack friendly text for screenreaders.

Also added listeners so that these features can be changed programmatically

Can one of the admins verify this patch?

@AppInventorWorkerBee ok to test

Thank you for reviewing -- I have taken the suggestion and pushed that change.

Ok, I think I've made all the changes you requested. My companion is showing a green action bar with "MIT AI2 Companion" on it, so let me know if it still isn't showing up for you. I haven't tested in API 7, but thought it would be ok not to since I'm just changing some font size and colors. Everything should be pushed!

Looking at the res XML files in Android OSP, the default textHintColor is defined as #808080.
Regards,
Evan

On May 4, 2021, at 4:42 PM, Jeffrey Schiller @.***> wrote:
@jisqyv commented on this pull request.
In appinventor/components/src/com/google/appinventor/components/runtime/TextBoxBase.java https://github.com/mit-cml/appinventor-sources/pull/2472#discussion_r626094133:

 Enabled(true);

 fontTypeface = Component.TYPEFACE_DEFAULT;
 TextViewUtil.setFontTypeface(view, fontTypeface, bold, italic);
 FontSize(Component.FONT_DEFAULT_SIZE);
 Hint("");

if (isHighContrast || container.$form().HighContrast()) {

 view.setHintTextColor(COLOR_YELLOW);

}
else {

 view.setHintTextColor(0xFF9E9E9E);

It looks like before your change we didn't touch the hint text color. So, you can just not set it if not in high contrast mode (in the constructor). However, you will need to know what it is so that setHighContrast() will know what to do if it is called with false as the argument. To solve this, in the constructor you can call view.getHintTextColor().getDefaultColor() to learn what it is and save that in a variable (instance variable).
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub https://github.com/mit-cml/appinventor-sources/pull/2472#discussion_r626094133, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAKFANOMNITP7C3M7BSMZPTTMBL4FANCNFSM43B3P2SA.

@ewpatton I recommended the programmatic approach in case it varies based on SdK and Theme.

Then should I go ahead and define a default hint text color in components.java to be #808080? Or do I pull straight from a theme?

I think what Jeff is proposing is to save the old hint color when you update the property and then restore it when the property is reset.
Sent from my iPhone

On May 4, 2021, at 16:54, dunandmuri @.***> wrote:

Then should I go ahead and define a default hint text color in components.java to be #808080? Or do I pull straight from a theme?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.

@ewpatton I was actually proposing reading it out just once in the constructor.

Ok. I believe that should work as well. Theme switching in the companion occurs before the components are instantiated so the hint color will be in effect at that point. We will need to change the behavior in the future if we ever allow the user to customize the hint color.
Sent from my iPhone

On May 4, 2021, at 20:29, Jeffrey Schiller @.***> wrote:

@ewpatton I was actually proposing reading it out just once in the constructor.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.

Am I missing something? I'm doing:
private int hintColorDefault = view.getCurrentHintTextColor();
in TextBoxBase, but that of course won't work if I place it before view has been initialized.

You have to do it in the constructor.


[solved] Windows shrink after screen lock

Perhaps an incompatibility with the new update to Windows 10 2004 and Powertoys 0.18.1. After locking the screen, all open windows are arranged one above the other in the upper left corner. Powertoys still works, but all windows have to be moved back to their place. This effect occurs sporadically, but very often.

@transwarp2020
does it only happen for the windows that were zoned, or also for windows that were not zoned?

It happens for all the windows that were opened. I typically use zoned windows on different virtual desktops, but additionally unzoned RDP-Windows for example.

If it happens for all windows, it's a Windows bug. Try setting this option:

it may help restoring the windows to their position.

Thanks,
this seems to solve my problem.


pythonPackages.geojson: 2.5.0 -> 3.0.0

Description of changes
Version bump, https://github.com/jazzband/geojson/blob/3.0.0/CHANGELOG.rst#300-2022-01-26

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] 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
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.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
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

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

17 packages failed to build:

home-assistant-component-tests.geo_json_events
home-assistant-component-tests.openweathermap
python310Packages.aio-geojson-generic-client
python310Packages.pyowm
python311Packages.agate-sql
python311Packages.aio-geojson-client
python311Packages.aio-geojson-generic-client
python311Packages.aio-geojson-geonetnz-quakes
python311Packages.aio-geojson-geonetnz-volcano
python311Packages.aio-geojson-nsw-rfs-incidents
python311Packages.aio-geojson-usgs-earthquakes
python311Packages.crate
python311Packages.geojson
python311Packages.geojson-client
python311Packages.labelbox
python311Packages.osmpythontools
python311Packages.pyowm

16 packages built:

csvkit
home-assistant-component-tests.geonetnz_quakes
home-assistant-component-tests.geonetnz_volcano
home-assistant-component-tests.nsw_rural_fire_service_feed
home-assistant-component-tests.usgs_earthquakes_feed
python310Packages.agate-sql
python310Packages.aio-geojson-client
python310Packages.aio-geojson-geonetnz-quakes
python310Packages.aio-geojson-geonetnz-volcano
python310Packages.aio-geojson-nsw-rfs-incidents
python310Packages.aio-geojson-usgs-earthquakes
python310Packages.crate
python310Packages.geojson
python310Packages.geojson-client
python310Packages.labelbox
python310Packages.osmpythontools

Thanks for running nixpkgs-review, @mattmelling.
I have only tested python3, not python311 which is currently Python 3.11.1.
Due to a bug (?) in the project's setup.py, this version was not supported which I fixed through https://github.com/jazzband/geojson/pull/198.
I have just added a patch to this PR including this fix.
Now python311packages.geojson builds and I have just launched a nixpkgs-review job myself.

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

33 packages failed to build:

csvkit
home-assistant-component-tests.geo_json_events
home-assistant-component-tests.geonetnz_quakes
home-assistant-component-tests.geonetnz_volcano
home-assistant-component-tests.nsw_rural_fire_service_feed
home-assistant-component-tests.openweathermap
home-assistant-component-tests.usgs_earthquakes_feed
python310Packages.agate-sql
python310Packages.aio-geojson-client
python310Packages.aio-geojson-generic-client
python310Packages.aio-geojson-geonetnz-quakes
python310Packages.aio-geojson-geonetnz-volcano
python310Packages.aio-geojson-nsw-rfs-incidents
python310Packages.aio-geojson-usgs-earthquakes
python310Packages.crate
python310Packages.geojson
python310Packages.geojson-client
python310Packages.labelbox
python310Packages.osmpythontools
python310Packages.pyowm
python311Packages.agate-sql
python311Packages.aio-geojson-client
python311Packages.aio-geojson-generic-client
python311Packages.aio-geojson-geonetnz-quakes
python311Packages.aio-geojson-geonetnz-volcano
python311Packages.aio-geojson-nsw-rfs-incidents
python311Packages.aio-geojson-usgs-earthquakes
python311Packages.crate
python311Packages.geojson
python311Packages.geojson-client
python311Packages.labelbox
python311Packages.osmpythontools
python311Packages.pyowm

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

7 packages failed to build:

home-assistant-component-tests.geo_json_events
home-assistant-component-tests.openweathermap
python310Packages.aio-geojson-generic-client
python310Packages.pyowm
python311Packages.aio-geojson-generic-client
python311Packages.labelbox
python311Packages.pyowm

26 packages built:

csvkit
home-assistant-component-tests.geonetnz_quakes
home-assistant-component-tests.geonetnz_volcano
home-assistant-component-tests.nsw_rural_fire_service_feed
home-assistant-component-tests.usgs_earthquakes_feed
python310Packages.agate-sql
python310Packages.aio-geojson-client
python310Packages.aio-geojson-geonetnz-quakes
python310Packages.aio-geojson-geonetnz-volcano
python310Packages.aio-geojson-nsw-rfs-incidents
python310Packages.aio-geojson-usgs-earthquakes
python310Packages.crate
python310Packages.geojson
python310Packages.geojson-client
python310Packages.labelbox
python310Packages.osmpythontools
python311Packages.agate-sql
python311Packages.aio-geojson-client
python311Packages.aio-geojson-geonetnz-quakes
python311Packages.aio-geojson-geonetnz-volcano
python311Packages.aio-geojson-nsw-rfs-incidents
python311Packages.aio-geojson-usgs-earthquakes
python311Packages.crate
python311Packages.geojson
python311Packages.geojson-client
python311Packages.osmpythontools

Now this looks better.. somehow nixpkgs-review gets very confused if a GC run interferes.

python311Packages.labelbox already fails on the current master branch.
aio-geojson-generic-client expects geojson >= 2.4.0 and was thereby broken before.
pyowm even demands geojson >= 2.3.0.
home-assistant-component-tests.geo_json_events uses aio-geojson-generic-client and home-assistant-component-tests.openweathermap uses python311Packages.pyowm.

Thus, there don't seem to be any new breakages.

This PR became irrelevant after @mweinelt pushed e63354a73051709610567e986cb4976601237a3c.


Not able to perform write operation in pyspark for aws glue

I have tried to do a write operation using pyspark and got the following error:
'o132.save. com.microsoft.sqlserver.jdbc.SQLServerDriver does not allow create table as select.'
code snippet i had
df.write
.format("com.microsoft.sqlserver.jdbc.SQLServerDriver")
.mode("append")
.option("url", url)
.option("dbtable", table)
.option("user", username)
.option("password", password)
.option("tabLock","true")
.save()
If I change the format to jdbc its working. I did not have any issues while reading using the same format.

duplicate
.format("com.microsoft.sqlserver.jdbc.SQLServerDriver") should have been .format("com.microsoft.sqlserver.jdbc.spark")


Significant lag when opening or collapsing folders.

It is as it says. About .5 - 2 second wait before the folder opens or collapses. This seems to be dependent on how many notes are in the folder, but some of these folders are full of only files and no folders or files with icons.
I am a beginner programmer, but if I were to take a guess, it might have something to do with checking the files before the folder opens to see if they have icons assigned or not. No idea why that would lag on collapse tho.
(Also, the icons that look like file icons are provided by the theme, not this plugin. Still have the same issue with that theme disabled.)
(Also also, I have the file color plugin on. I have disabled it and tested without and it seems to be fine as long as the iconic plugin is not enabled.)

Thanks for the report! Which theme are you using, and does the lag keep happening when you switch to another theme?
Some community themes utilize a lot of expensive CSS operations, which can become noticeable when something rapidly updates the DOM tree (like Iconic does when you toggle a folder).

Hey @SandmansDreams, I'm just closing this issue for now because there's not enough info to understand why this slowdown happens for you. If you get time to narrow it down, come and reply here!

Hey @SandmansDreams, I'm just closing this issue for now because there's not enough info to understand why this slowdown happens for you. If you get time to narrow it down, come and reply here!

Yea sorry, it happens on both my systems (mac and PC) and is definitely due to this plugin (as disabling it removes the lag) but I have no idea why it happens. Additionally, the more folders with icons it has, the longer it takes to open / collapse at a proportional rate. I did try turning off all plugins and CSS snippets and there is still some lag, though significantly less. If it was at that speed with everything enabled, I likely would not have noticed. Sorry for the late response.
My theory is it could be compounded with the other folder plugins I have (auto-collapse-folder and file color) that makes it much worse. However, it still seems that this plugin is contributing the most significant portion of lag.


[ZEPPELIN-5305] Activate tests for Zeppelin plugins in our CI infrastructure

What is this PR for?
Activates the tests for all Zeppelin plugins in our CI infrastructure
What type of PR is it?

Bug Fix

What is the Jira issue?

https://issues.apache.org/jira/browse/ZEPPELIN-5305

How should this be tested?

CI

Questions:

Does the licenses files need update? No
Is there breaking changes for older versions? No
Does this needs documentation? No

I have one question. Some tests sometimes cause errors, why is this?

At the moment we have a CI problem. I have informed the other devs via our mailing list.
I see several reasons for the rest of the time.
* some parts of our application and test code are quite old and could be optimised

* Zeppelin has a large number of external dependencies, which are also needed in the integration tests. It starts with the programming language, e.g. Python, R, and ends with other large projects such as Flink, Spark, Hadoop, etc.

Thanks for your detailed explanation.

LGTM, thanks @Reamer

I will merge this into Master and Branch-0.9 next week.


Animate height between multiple views

Context
It very common to want to render different views in the same AnimateHeight container. To ensure the user has a great experience and not have to deal with content height snapping, and assuming they do not have any reduced motion requirements, a smooth transition between heights is by far the best and smoothest experience.
For example, cross fading smoothly between any render states that might abruptly change the height of a section are great candidates for this feature.
I don't feel AnimateHeight is overreaching here as the functionality is chiefly one concerning height.

Released in 4.2.0


Reorganizing of test hosts across library

Was getting started to look at adding more tests for decorators as part of https://github.com/Azure/cadl-azure/issues/827 and our current usage of test host in libraries is a bit of a mess. Trying to simplify this.
Redesign a bit the test host to be able to better reuse other libraries test hosts without having to manually redefine all the files to import.
Main changes:

/testing scope to packages so you can import this with the test utils
each package can provide a TestLibrary that provide the files needeed to use that library. Another library can then depend on it without having to manually reference every single paths.
Provide helper expectations for diagnostics

expectDiagnostics(diagnostics, [{ code: "suppress-error" }, { code: "no-id-property" }]);

@nguerrera merged all the others PRs, cleanup and added a helper to test diagnostics  if you want to take another look at this PR. It should be good now


[FLINK-6560] [build] Restore maven parallelism in flink-tests

Configure a default value for Maven variable 'flink.forkCountTestPackage'.

@StephanEwen I see that you +1'd the ticket but this PR has not been reviewed.


Port linkcheck settings from plone/documentation

I made some changes to the Sphinx configuration in Documentation that are worth consideration for Training.
Originally posted by @stevepiercy in https://github.com/plone/training/issues/574#issuecomment-1067702317

There are a lot of pages that do not like anchors, even though the link is valid. From time to time, we should comment out links that contain an anchor from linkcheck_exclude to verify whether excluded links are still valid.
One thing I found is that GitHub uses JavaScript to rewrite anchors in the HTML of the page by prepending the anchor name with user-content-. For example:
https://github.com/repoze/repoze.catalog/blob/master/docs/usage.rst#query-objects

gets rewritten to:
https://github.com/repoze/repoze.catalog/blob/master/docs/usage.rst#user-content-query-objects

Although we could rewrite all those anchors so that they pass linkcheck, this really should be handled in Sphinx. There is a related issue. https://github.com/sphinx-doc/sphinx/issues/9016


[20.09] mpv: backport security fix

Version 0.33.1 fixes a security vulnerability. See
https://github.com/mpv-player/mpv/releases/tag/v0.33.1 for more
information.
(cherry picked from commit ff4e2fa19d35e7d8b9311e6788411a04ed47d582)
Motivation for this change
https://github.com/NixOS/nixpkgs/pull/118639
@cript0nauta we can do 0.33.1 or patch, i don't have a specific preference
(I saw your comment as I was writing this)
Things done
cherry-pick

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

[x] NixOS
[ ] macOS
[ ] other Linux distributions

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

Looks like there are a few changes (dropped support for samba and sndio).

Please target release-20.09.

LGTM

Result of nixpkgs-review pr 118683 at a0711a2c run on aarch64-linux 1

1 package skipped due to time constraints:

plex-mpv-shim

13 packages built successfully:

celluloid
curseradio
jellyfin-mpv-shim
jftui
minitube
mpc-qt
mpv-unwrapped
mpvScripts.mpris
python37Packages.mpv
python38Packages.mpv
qimgv
sublime-music
tuijam


Adding readiness checkups in KCP

Is your feature request related to a problem? Please describe.
When deploying ArgoCD to KCP, one of the initial blocker we face is due to the readyz and health checkups by ArgoCD on the cluster, which as per my knowledge KCP still don't support.
So while deploying the argocd, the argo looks for some health and readiness checks from the cluster, while doing so it looks for a port service for checking if the cluster is ready or not, which while doing so on KCP turns out to be the blocker, As stts mentioned readiness in the sense of kube-apiserver /readyz; basically to serve /clusters/root:org:ws/readyz
Describe the solution you'd like
Enable readiness and health probes checkup in KCP in each workspace, for ex: /clusters/root:org:ws/readyz
so, in case I'm deploying argocd in org->A ws->test then, the readiness can be checked at /clusters/root:A:test/readyz
Describe alternatives you've considered
For now, to make ArgoCD deploy on KCP, we have disabled those checks hence this feature request will unblock us form that.
CC: @sttts, in case I missed some details, feel free to chime in, thanks!

I would like to work on this please.


python label - added compressed to appNewVersion curl

the page used to scrape appNewVersion seems to always return a compressed version. Adding --compressed to the download fixes this.
Fixes #1593

Thank you!


Feature Request: add .version

Could see how they do it in dask
https://github.com/dask/dask/blob/master/dask/_version.py

I should definitely add this to my todo list whenever I start a new project :-)
Thanks for bringing this up!


Enhancement of AI Grouping Accuracy and User Experience

Dear MichaelYuhe and Contributors,
I hope this message finds you well. I've been exploring the AI Group Tabs extension and I must commend the innovative approach to tab management. The use of AI to categorise and group tabs is a significant step forward in browser organisation.
However, I would like to suggest a couple of enhancements that could potentially elevate the user experience and the extension's functionality:

AI Grouping Accuracy: While the default categories are quite useful, there are instances where tabs could be more accurately grouped with additional context. Perhaps implementing a feature that allows the AI to learn from user corrections could improve accuracy over time. This could be in the form of a feedback loop where the user can indicate the correct group for a tab, and the AI adapts accordingly.

User Experience: The current process for customising categories is straightforward, yet it might benefit from a more intuitive interface. For example, a drag-and-drop functionality for organising categories and tabs within the popup could make the customisation process more user-friendly.

Performance Optimisation: As more tabs are opened and grouped, there might be a noticeable impact on performance. It would be beneficial to conduct a thorough performance analysis to ensure that the extension scales well with heavy usage.

Accessibility Features: Consider adding features that make the extension more accessible to a wider range of users, such as keyboard navigation support and high-contrast themes for those with visual impairments.

I believe these suggestions align with the roadmap's focus on improving the extension and could be considered for future releases. Additionally, I'm curious about the progress on the CI/CD implementation for build and release, as it's a crucial step for streamlined updates and maintenance.
Thank you for your dedication to this project. I'm looking forward to the upcoming features and improvements.
Best regards,
yihong1120

Thanks for your feedback yihong!
These features are all on our roadmap, I'll create a better document to articulate our future vision.


Ringing after logout

The app is ringing after logout

Is there any way to stop ringing after logout? like something to unregister from app

I think you should call VoipPushNotification.removeEventListener('notification') in your logout function

I think you should call VoipPushNotification.removeEventListener('notification') in your logout function

Yes, I tried, but it doesn't work for me. I believe the problem is with the call keep library because I use it.

@moshibilpk hey man, did you find a way to solve this?


feat: make plugin compatible with Serverless v3.x

The changes in this MR allow this plugin to be used with Serverless 3.x but do not impose any new backwards compatibility constraints:

The package:compileEvents event has been available since version 1.12.0.
The defineFunctionEvent method has been available since version 1.78.0

Ping @onebytegone. Not sure why, but it's not allowing me to assign a reviewer.


Filter stored procedures in navigation panel

The filter in the navigation panel to the left, currently works only for tables and views.
It would be nice if it also worked for stored procedures.

This filter actually works for any objects.
You can configure it using context menu:

This does not work for MS Sql Server stored procedures. No filtering is available.

There is no Filter option for procedures for MS SQL database

Will this be available soon?

@JasonGilley @AvronNet Yep. It is a bug in Generic driver. Thanks for the report. Will be fixed in 5.2


Custom controller actions and attributes structure

I have a Schema and a Controller which implements the standards. I want to add several actions to that controllers. Each post action accepts a different data structure. How can I handle that as the data structure is related to Schema fields definition. Or maybe I missed smth.
I am using cloudcreativity/laravel-json-api in previous projects . But as mentioned, for new projects, this package should be used  but I feel confused/lost with this new package

By "actions", do you mean custom actions? If you're using different data structures, for custom actions, there's not much I can do to support it - as the point of the package is to handle the JSON:API spec. Which means it can't handle custom things that deviate from the spec.
Unless I'm missing something about your question? You might need to provide an actual example - if you're doing something that is covered by the spec but you can't find a way to make it work?

I just need to to have a custom action within a controller using a custom request with rules. In previous version I was using Validators. But now, if I define rules for this specific action I need to define the fields within the schema, that could be different that the standard schema of the resource associated with this schema

Yeah I'm a bit lost, sorry. The schema is meant to define the resource representation in the JSON:API compliant server. I'm not sure why the schema would be changing? E.g. if you have a posts resource, then the schema should represent that resource... the resource shouldn't change structure in different scenarios as that would be unpredictable to the client.


Quick switch for reconnected devices

Hi xenolighting! I really like AudioSwitcher, it is a great helper!
Sadly I does not really support so nicely my setup:
I use Audioswitcher to switch between the laptop speakers, the sound system (connected via bluetooth) and my TV-Audio (connected via Display-Port).
So here comes the issue: When AudioSwitcher starts up, the later two audio-devices are usally not available (have to reconnect them first) which affects their favorite-status and makes it impossible to use the quick switch feature for them. Even though they were are marked as favorite before, they are now no  favorites anymore. But they seem to keep there id  (see AudioSwitcher.json.txt, hotkey section; for the hotkey-switching the reconnection works)
I see two solutions for this issue:

add a option to use quick-switch for all devices (favorite or not)
recognize the reconnected devices and keep there favorite status

I would be very happy, if you could fix that in a future version!

Noticed this issue in my Mediaportal plugin for it as well, had to add this workaround for now:
https://github.com/RickDB/MP1-AudioSwitcher/commit/a9d22e5928a3e1b1621ede5636d234e41fb3c2f0
Seems that you have to re-create the CoreAudioController in some cases as it will not return the proper device names but indeed retain their ID.
Hopefully this can get fixed in the library :)

I think there's two issues here. One that's known about.
I believe @StefanLuthardt is experiencing an issue where Windows thinks all newly connected devices get new Id's. It happens outside of AudioSwitcher, and at the operating system level. Sadly there is no "good" way around this problem, you could favourite "Device Names", but then devices can have the same name too.
@RickDB The bug you found is a lot more interesting! Can you raise an issue in the Api repo? https://github.com/xenolightning/AudioSwitcher
Please attach the version you are using. I know there was some issues in earlier versions with not updating the properties correctly when triggered by the system. This should be resolved in the v4.0 of the api.

@xenolightning
Issue added :)
https://github.com/xenolightning/AudioSwitcher/issues/27

So when it will be fixed, i have the same problem, second hdmi screen i snot connected the whole time, but when it reconnect it is no longer fav device, so quick switch won't work.
Regardless of the issue fix i think that the software should also have an option to just use quick switch for all devices, most people have just like 2, 3, so it would make sense for most to just use that.

@xenolightning
Any progress on this?

Hi, is there any progress on this? I have the issue with my USB headset. I have speakers plugged in all the time, but my USB headset often plugs in and out whenever I use it, but the favourite status always resets. In addition to this, I find I have to restart Audio Switcher every time I plug my USB headset in, as it appears as Unknown instead of the correct device.

The unknown bug will be fixed with in the next version.
I'm exploring the option of enabling quick switch for all playback devices when no favourites are selected. It seems like a fairly reasonable solution to the underlying ID problem.

@StefanLuthardt @xenolightning
Well, looks like this project is dead, almost year and this is still not resolved, i've been forced to install other software.

@andrewpros Hi, what other software are you using? This problem indeed still remains, so if you found the solution elsewhere I'd love to use that too. I don't like using dead software.

@cybersteel8 Well i changed to https://github.com/sirWest/AudioSwitch
Not ideal, but when i constantly enable/disable speakers output devices it finds them automatically and i quickly change it.
Funny thing is that windows 10 has now the ability to quick switch output device from audio tray icon, its almost fine, but you need expand it every time... so there is 1 vs 3 clicks.. i'm soooo damn lazy...

@andrewpros The project isn't dead. This project is entirely funded by donations, which is totals about USD$10 per month, and in my SPARE time.
Expecting functionality to be added to an mostly unfunded, open source project is naive at best.
The great thing about open source, is that anyone can fix this issue. Maybe even you.
You don't have to use it, I'm not forcing you.

@xenolightning
Well i get it but the problem remains the same, i can't fix this, it doesn't work, you can't hope everyone is a great programmer who know everything and will fix what you can't.
So the issue is there, no update in almost year, dead for me.
As for now i have a code snipped in c# to monitor the device IDs and if they changing when reconnecting, i run this now and then and i just wait for some confirmation to happen.
Other thing is i know how to make it without bothered by the ids even if they change, but if i made a PR i have no guarantee that you will accept that code as it could not be the way you like it to see and how the code would work.

Monitoring the device id's would be awesome 👍 Might get a little more context around the devices, and why they get assigned new Id's.
I'm not too bothered about the "quality" of the code, the code in this project is a little wacky anyway. As long as it makes sense and isn't entirely unreadable then that' should be ok for me.
I might have time on my Saturday (NZT) to have a look and help you out.

@xenolightning
Well my idea is simple, if we can't relay on device IDs then there is a need for an all active output devices quick switch.
It at least gives us the possibility to iterate over available devices.
It should just get all current devices list and activate the next one, regardless what ID it was or is now, doesn't matter, as it work in a "current situation" mode, means, it doesn't relay on any saved ids.
If the reconnection change ID issue is true, then this is the only way.
And as of now i can confirm that my monitor script had detected a device ID change, this is real. It was due to installing new NV driver with HDMI audio output, this changed the ID.
But this happens randomly, even when you do nothing, so if there is no other ID we can use...

@xenolightning
I can also confirm that https://github.com/sirWest/AudioSwitch has the same issue after all, there is no way to prevent device id change and there are no other device ids on windows that can be used, but hence it just iterates over available devices on click, it works without reconfiguration.

Not closed, but fixed in the next version

Thx, i see this will cycle thru available audio devices. There is nothing we can do more.

Would it be helpful to be able to manipulate the list of active, so we can "hide" certain devices. That way, when cycling through reconnected devices will be a clean list.
I have the same issue as described by @cybersteel8 and added to that some devices I don't need to manually switch to.
Example:
Active devices list: 1. headset jack on monitor, 2. some speakers option I never use, 3. some other installed device I've never heard of, 4. a USB headset I use once a week and 5. desktop jack.
My active devices are actually only 1, 4 and 5. 4 is often disconnected, but I'd still like to have it in the list. So by excluding 2 and 3, my active devices will always be as I expect them to be.
How about that? :-)

I don't know about an active list as such. Seems like a hard thing to manage when there a new devices being added.
You can always disable audio devices you don't use either in the standard sound menu, or via device manager. Audio Switcher respects these.

@bondt
As stated, windows do not provide unique audio device ids, this option is possible, but it will work only till windows don't change the ids again.
So basically it is the same as disabling 2, 3 in windows options.

Sounds fine by me!
@xenolightning, when is this next release with the cycling? :-)


Does weird things to my indentation

🐛 Describe the bug
At the end of an unordered list, I hit 'o'. Expected behaviour without obsidian.nvim installed is to enter insert mode, start a new line, and the cursor is at a position matching the indentation of the previous line (0/4/8 etc).
However, with obsidian.nvim enabled, it always adds an extra level of indentation (4/8/12 etc). In case it helps for debugging, it seems to highlight the characters before the cursor on the line, but that highlight disappears as soon as I start typing.
Versions
NVIM v0.9.1
Build type: Release
LuaJIT 2.1.0-beta3
Obsidian v1.11.0
local plugins = {
"yonchu/accelerated-smooth-scroll",
"iamcco/markdown-preview.nvim",
{
"overcache/NeoSolarized",
lazy = false,
priority = 1000,
config = function()
vim.cmd [[ colorscheme NeoSolarized ]]
end
},
"jose-elias-alvarez/null-ls.nvim",
"neovim/nvim-lspconfig",
"nvim-lua/plenary.nvim",
"tpope/vim-repeat",
"mbbill/undotree",
"tpope/vim-fugitive",
"machakann/vim-highlightedyank",
"lifepillar/vim-mucomplete",
"tpope/vim-surround",
"folke/which-key.nvim",
"tpope/vim-commentary",
"tpope/vim-unimpaired",
"nvim-lualine/lualine.nvim",
"kyazdani42/nvim-web-devicons",
{
"epwalsh/obsidian.nvim",
event = { "BufReadPre /home/will/Seafile/Seafile/WillWiki/.md" },
-- If you want to use the home shortcut '~' here you need to call 'vim.fn.expand':
-- event = { "BufReadPre " .. vim.fn.expand "~" .. "/my-vault/.md" },
dependencies = {
-- Required.
"nvim-lua/plenary.nvim",
    -- Optional, for completion.
    -- "hrsh7th/nvim-cmp",

    -- Optional, for search and quick-switch functionality.
    -- "nvim-telescope/telescope.nvim",

    -- Optional, an alternative to telescope for search and quick-switch functionality.
    -- "ibhagwan/fzf-lua"

    -- Optional, another alternative to telescope for search and quick-switch functionality.
    -- "junegunn/fzf",
    -- "junegunn/fzf.vim"

    -- Optional, alternative to nvim-treesitter for syntax highlighting.
    -- "godlygeek/tabular",
    "preservim/vim-markdown",

  },
  opts = {
    dir = "[redacted]",  -- no need to call 'vim.fn.expand' here
    -- Optional, if you keep notes in a specific subdirectory of your vault.
    -- notes_subdir = "notes",

    -- Optional, set the log level for Obsidian. This is an integer corresponding to one of the log
    -- levels defined by "vim.log.levels.*" or nil, which is equivalent to DEBUG (1).
    log_level = vim.log.levels.DEBUG,

    -- daily_notes = {
      -- Optional, if you keep daily notes in a separate directory.
    --   folder = "notes/dailies",
      -- Optional, if you want to change the date format for daily notes.
    --   date_format = "%Y-%m-%d"
    -- },

    -- Optional, completion.
    completion = {
      -- If using nvim-cmp, otherwise set to false
      nvim_cmp = false,
      -- Trigger completion at 2 chars
      min_chars = 2,
      -- Where to put new notes created from completion. Valid options are
      --  * "current_dir" - put new notes in same directory as the current buffer.
      --  * "notes_subdir" - put new notes in the default notes subdirectory.
      new_notes_location = "current_dir"
    },

    -- Optional, customize how names/IDs for new notes are created.
    note_id_func = function(title)
      -- Create note IDs in a Zettelkasten format with a timestamp and a suffix.
      -- In this case a note with the title 'My new note' will given an ID that looks
      -- like '1657296016-my-new-note', and therefore the file name '1657296016-my-new-note.md'
      local suffix = ""
      if title ~= nil then
        -- If title is given, transform it into valid file name.
        suffix = title:gsub(" ", "-"):gsub("[^A-Za-z0-9-]", ""):lower()
      else
        -- If title is nil, just add 4 random uppercase letters to the suffix.
        for _ = 1, 4 do
          suffix = suffix .. string.char(math.random(65, 90))
        end
      end
      return tostring(os.time()) .. "-" .. suffix
    end,

    -- Optional, set to true if you don't want Obsidian to manage frontmatter.
    disable_frontmatter = false,

    -- Optional, alternatively you can customize the frontmatter data.
    note_frontmatter_func = function(note)
      -- This is equivalent to the default frontmatter function.
      local out = { id = note.id, aliases = note.aliases, tags = note.tags }
      -- `note.metadata` contains any manually added fields in the frontmatter.
      -- So here we just make sure those fields are kept in the frontmatter.
      if note.metadata ~= nil and require("obsidian").util.table_length(note.metadata) > 0 then
        for k, v in pairs(note.metadata) do
          out[k] = v
        end
      end
      return out
    end,

    -- Optional, for templates (see below).
    templates = {
      subdir = "templates",
      date_format = "%Y-%m-%d-%a",
      time_format = "%H:%M",
    },

    -- Optional, by default when you use `:ObsidianFollowLink` on a link to an external
    -- URL it will be ignored but you can customize this behavior here.
    follow_url_func = function(url)
      -- Open the URL in the default web browser.
      vim.fn.jobstart({"open", url})  -- Mac OS
      -- vim.fn.jobstart({"xdg-open", url})  -- linux
    end,

    -- Optional, set to true if you use the Obsidian Advanced URI plugin.
    -- https://github.com/Vinzent03/obsidian-advanced-uri
    use_advanced_uri = false,

    -- Optional, set to true to force ':ObsidianOpen' to bring the app to the foreground.
    open_app_foreground = false,

    -- Optional, by default commands like `:ObsidianSearch` will attempt to use
    -- telescope.nvim, fzf-lua, and fzf.nvim (in that order), and use the
    -- first one they find. By setting this option to your preferred
    -- finder you can attempt it first. Note that if the specified finder
    -- is not installed, or if it the command does not support it, the
    -- remaining finders will be attempted in the original order.
    -- finder = "telescope.nvim",
  },
  config = function(_, opts)
    require("obsidian").setup(opts)

    -- Optional, override the 'gf' keymap to utilize Obsidian's search functionality.
    -- see also: 'follow_url_func' config option above.
    vim.keymap.set("n", "gf", function()
      if require("obsidian").util.cursor_on_markdown_link() then
        return "<cmd>ObsidianFollowLink<CR>"
      else
        return "gf"
      end
    end, { noremap = false, expr = true })
  end,
}

}

Fantastic - thanks, I have excluded it.
Is there a reason Vim's built-in markdown syntax highlighting is disabled on Obsidian files? Thanks!

Is there a reason Vim's built-in markdown syntax highlighting is disabled on Obsidian files? Thanks!

Hmm, this is not intended 🤔 I'm not super familiar with Vim syntax stuff, but maybe our additional syntax highlighting rules are conflicting with Vim's built-in. I'll do some digging.

When I disable vim-markdown I still get some highlighting, but headers are not highlighted...

Our syntax files have been removed so I'm closing this.


Below is an illustration of a raw output of this endpoint:

you should add an unhandled error
https://github.com/QwantResearch/idunn/blob/master/idunn/utils/logging.py#L34


Sum Global PR.

Context
This  pull request contains 3 commits addressing three issues.

Remove hardcoded debug/optimization '/Zi' & '/O2' compiler options and allow user DSL to override it.  Updated unit test to reflect new changes.  subprojects :  languageNative & platformNative  Issue #884 commit 9e74658
Use provider api for the WindowsResourceCompile Task to support building DSL consistent with other tasks in the  cpp-application plugin.  subprojects:  languageNative.  Issue #173 commit 83c70d2
Added to fix MD5 error when .gradle project cache is part of the root source code directory.  subprojects:  core.  Issue #5972 commit be96b6c

Contributor Checklist

[x] Review Contribution Guidelines
[x] Make sure that all commits are signed off to indicate that you agree to the terms of Developer Certificate of Origin.
[x] Check "Allow edit from maintainers" option in pull request so that additional changes can be pushed by Gradle team
[x] Provide integration tests (under <subproject>/src/integTest) to verify changes from a user perspective Able to reuse existing integration tests
[x] Provide unit tests (under <subproject>/src/test) to verify logic Modified existing test cases
[ ] Update User Guide, DSL Reference, and Javadoc for public-facing changes this needs work as all of gradle native documentation has fallen behind
[x] Ensure that tests pass locally: ./gradlew <changed-subproject>:check

E:\gradle>gradlew.bat :core:check :platformNative:check :languageNative:check
.
BUILD SUCCESSFUL in 8m 56s
1385 actionable tasks: 1384 executed, 1 up-to-date

Gradle Core Team Checklist

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation
[ ] Recognize contributor in release notes

Wanted to give credit to the following participants for making this PR possible:

Hitesh Shah, our Amazon sponsor and contributor
Chip Dickson, Sum Global contributor
Charlie Walker, Sum Global contributor
Kent Fletcher, Sum Global contributor

The first obvious comment is to separate the various change into different PR. Let me look at the individual commit to provide more comment.

This PR 7409, has been broken up into the following 3 PR's.   Documenting here for reference.
1 of 3 https://github.com/gradle/gradle/pull/7432  Windows Resource Compile
2 of 3 https://github.com/gradle/gradle/pull/7437  Debug/Optimization Override
3 of 3 https://github.com/gradle/gradle/pull/7438  MD5 File System Snapshot Error

I'm closing this PR in favor of the split out PR. Let's continue the conversation on the other PR/issues.


Document CorrelationContext propagator under Built-in Implementations section

Originates from https://github.com/open-telemetry/opentelemetry-js/pull/838#issuecomment-630428319
Built-in Implementations section: https://github.com/open-telemetry/opentelemetry-js/tree/master/packages/opentelemetry-core#built-in-implementations

Closed via #1080, thank you @rubenvp8510


⚠ Samsung has degraded performance

In a02d5df, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2395 ms

Resolved: Samsung performance has improved in 1d9c04e.


feat: Implement/document SCSS/SASS usage

Description
The documentation only shows how to use css, but scss and sass are widely used. While Bulma is recommended in the documentation, it requires SCSS variables for theming, and it is not obvious how to get it working natively within Jaspr.
SASS is implemented using dart, so this should be relatively straightforward to integrate into the Jaspr build/serve process. https://sass-lang.com/dart-sass/
Additional Context
jaspr_pad has a sass folder and imports sass_builder, but simply importing  "sass_builder: ^2.2.1" into a base jaspr project leads to an error: "[ERROR] AssetNotFoundException: node_interop|pubspec.yaml". It is not clear what else is required.
The approach that I was able to get working was to compile scss into css separate from Jaspr. For example, in development:

create a lib/scss folder with main.scss
Add StyleRule.import('css/main.css') to styles.dart file
run "Jaspr serve" in one terminal and "sass --watch lib/scss:web/css" in another terminal.

A few implementation ideas for Jaspr:

Document above approach on the main website as a quick fix to get users onboarded fast.
Update Quickstart generation to create the above mentioned lib/scss/main.scss file and StyleRule.import('css/main.css') code. Jaspr serve/build run the necessary "sass --watch lib/scss:web/css" command under the covers.
Support sass and scss files in StyleRule.import (seamlessly compiling in the background)
Expose Jaspr build hooks and include the sass compilation logic by default in the quickstart. Build time hooks would be useful for more than just scss compilation, but would expose more complexity to users. For example sass dart build code, see the compile-sass.dart sample on the dart-sass page: https://sass-lang.com/dart-sass/

I think sass_builder already does everything needed.
For jaspr_pad and other projects using it, running jaspr serve or jaspr build already also runs sass compilation. Not this only works for files inside ' web/', not 'lib/' (you can import files from 'lib/' but these will not be compiled itself to ' .css' counterparts).

Running jaspr build having sass_builder installed runs the builder for it but it raises an error that I have no clue where is coming from:

AssetNotFoundException: node_interop|pubspec.yaml

And jaspr fails to build.
I can't see exactly if is correct to post it here or in node_interop, sass or sass_builder packages issues report.

You can try 'jaspr clean' before rebuilding.

sass_builder still has the same issue about AssetNotFoundException and there is no mode to solve it. Also, there is a bunch of warnings as errors generated when running jaspr serve that makes it useless, so it's better the mode as the OP is doing with sass.


Drawer: LTR/RTL Support - 2

Completed for AudioBar, Search, Settings and Text appearance(But this feature is bugged so we had to make do with what we can with Daisy UI)


Warning MSB3061

When I build the project I get the following warning:
c:\Windows\Microsoft.NET\Framework\v4.0.30319\Microsoft.Common.targets(3607,9): warning MSB3061: Unable to delete file "C:\Users\erapp\Documents\Visual Studio 2010\Projects\CRS1\CRS1\bin\PagedList.xml". Access to the path 'C:\Users\erapp\Documents\Visual Studio 2010\Projects\CRS1\CRS1\bin\PagedList.xml' is denied.
What is odd is that when I check that file, it won't allow me to open it, edit it, or change the owner.  The owner is listed as "..."
How should this file be configured?

Did you install via nuget?

Yes.
Sent from my iPhone
On Jan 29, 2012, at 9:32 AM, Troy<EMAIL_ADDRESS>wrote:

Did you install via nuget?

Reply to this email directly or view it on GitHub:
https://github.com/TroyGoode/PagedList/issues/21#issuecomment-3708649

Hmmm, I haven't heard of this happening before and cannot reproduce. Can you try uninstalling the package and reinstalling.

Sure.  I will try.  Is this warning "bad"?
On Mon, Jan 30, 2012 at 8:13 AM, Troy Goode <
<EMAIL_ADDRESS>
wrote:

Hmmm, I haven't heard of this happening before and cannot reproduce. Can
you try uninstalling the package and reinstalling.

Reply to this email directly or view it on GitHub:
https://github.com/TroyGoode/PagedList/issues/21#issuecomment-3719209

I don't think so. I think this is just a hiccup from the download via
nuget... let me know if uninstalling and reinstalling the package results
in the same issue.
On Mon, Jan 30, 2012 at 10:21 AM, elliotrapp <
<EMAIL_ADDRESS>
wrote:

Sure.  I will try.  Is this warning "bad"?
On Mon, Jan 30, 2012 at 8:13 AM, Troy Goode <
<EMAIL_ADDRESS>
wrote:

Hmmm, I haven't heard of this happening before and cannot reproduce. Can
you try uninstalling the package and reinstalling.

Reply to this email directly or view it on GitHub:
https://github.com/TroyGoode/PagedList/issues/21#issuecomment-3719209

Reply to this email directly or view it on GitHub:
https://github.com/TroyGoode/PagedList/issues/21#issuecomment-3721152

Closing until I hear that it is reproducible.


Holder of styling

lets switch the order, the available for mint should be always together with the holder of row
so its like this, without the mint button

test here: https://canary.kodadot.xyz/ahk/drops/reflection

👋


baseURL is not working

Environment

Operating System: Darwin
Node Version:     v20.12.2
Nuxt Version:     3.12.1
CLI Version:      3.12.0
Nitro Version:    2.9.6
Package Manager<EMAIL_ADDRESS>Builder:          -
User Config:      future, srcDir, devtools, components, modules, colorMode, ignore, appConfig, css, extensions, ssr, routeRules, vue, nitro, runtimeConfig, auth, router, vite, i18n, csurf, content
Runtime Modules<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>Build Modules:    -

Reproduction
setup auth module option inside nuxt.config.ts to
baseURL: 'http://project.local:3001',
Describe the bug
the <form> element from the default signin page keeps pointing to "localhost" instead of what was set in nuxtConfig.auth.baseURL

Additional context
No response
Logs
No response

Hi @zecar 👋
What other environment variables do you pass? Also, how do you overwrite nuxtConfig.auth.baseURL? If you are using an environment variable in your Nuxt Config, you will need to set it at built time.

@zoey-kaiser, sorry for not updating
I got this working by passing AUTH_ORIGIN and NEXTAUTH_URL
please update the docs of the project with this usecase


E2E: Override the external frr image

This commit adds the --external_frr_image flag
into metallb inv e2etest command, thus
overrides the external containers` frr image.
This is required as we're facing e2e test failures with  FRR BFD + echo test case, and to prevent
it we want to align the FRR version between
the speakers and the external container.

Can you add this as an extra commit of https://github.com/openshift/metallb/pull/119 so we know if it will take effect?

Can you add this as an extra commit of #119 so we know if it will take effect?

done

Closing this one, as those commits are now part of #119


Simplify initialization of opensl backend

Overriding this function hasn't been necessary for quite some time now.


Issue #10 Add play and download button when hovering on movie image

I have created play and download button which will appear on hover.
Also made the code much more easier to read by adding comments, indents and spaces cause it was becoming difficult to work on this kind of large project 😅 Hope It is fine.
Please let me know if you need any changes to be done
ISSUE #10
My Contribution will be from LGM-SOC

@aum-singhal resolve conflicts.

Resolve conflicts.
Website has been updated.
Make sure that this feature works on all pages and all images.


cmake: fix cross-compilation with gRPC_BUILD_GRPC_CPP_PLUGIN=OFF

commit 99752b173cfa reintroduced the cross-compilation issue which was previously fixed by da10b795e177
see #26292

It looks good. Would you run tools/buildgen/generate_projects.sh?

The result is PASSED. Do you need any more specific output?
$ tools/buildgen/generate_projects.sh
...
PASSED: ./CMakeLists.txt [time=0.4sec, retries=0:0]%

Thanks. Let me trigger the test again!

grpc_build_artifacts_multiplatform: running

grpc_build_artifacts_multiplatform: running

Everything green except of php distribtests, but that seems unrelated?

Rerunning tests, but otherwise LGTM.


Can't perform a React state update on an unmounted component

Seeing this error when I hide a div that contains many ChipSet components.
index.js:1452 Warning: Can't perform a React state update on an unmounted component. This is a no-op, but it indicates a memory leak in your application. To fix, cancel all subscriptions and asynchronous tasks in the componentWillUnmount method.
    in RippledComponent
    in div
    in ChipSet

It seems to happen right before execution of this line:
https://github.com/material-components/material-components-web-react/blob/a468f024df300c535d220f9bbc493e5c67af77d5/packages/chips/Chip.js#L74
I see there is no "mounted" check prior to this setState call, maybe this could be the source of error.

@cdock1029 ah yes, we should have an isMounted check before both the remove and addClass methods (similar to https://github.com/material-components/material-components-web-react/blob/master/packages/ripple/index.js#L34). The foundation is calling these methods, but its most likely after they have been destroyed.
If you'd like, please feel free to open a PR following the pattern in the above link. Otherwise we will get to it asap.

Fix it with this technique.
Look where I added _isMounted and follow the same in you code
class Page extends Component {
  _isMounted = false;

  state = {
    isLoading: true
  }

  componentDidMount() {
    this._isMounted = true;

    callAPI_or_DB(...).then(result => {
      if (this._isMounted) {
        this.setState({isLoading: false})
      }
    });
  }

  componentWillUnmount() {
    this._isMounted = false;
  }

  render() {
    return (

Whatever

    );
  }
}

export default Page;

Thanks @mohokh67! For a more permanent solution, the Chip component should have built in internally.

Try https://github.com/iShawnWang/withUnmounted package as a elegant solution

尝试使用https://github.com/iShawnWang/withUnmounted包作为优雅的解决方案

npm ERR! 404 Not Found: @iShawnWang/withUnmounted@latest

Sorry for the case-sensitive : npm install --save @ishawnwang/withunmounted works fine ~

fixed with #870

I get similar warning in tests
     <JssProvider generateClassName={generateClassName}>
        <SnackbarProvider>
          <StaticRouter location="/" context={{}}>
            <Provider store={mStore}>
              <Route path="/" component={Dashboard} exact />
            </Provider>
          </StaticRouter>
        </SnackbarProvider>
      </JssProvider>,

Warning: Can't perform a React state update on an unmounted component. This is a no-op, but it indicates a memory leak in your application. To fix, cancel all subscriptions and asynchronous tasks in the componentWillUnmount method.
in SnackbarProvider (at Dashboard.test.jsx:41)
in JssProvider (created by WrapperComponent)
in WrapperComponent

@mohokh67 What about solution for a functional component?

@5ervant you can use useRef for check if the component still mounted
example

look at it:
https://www.robinwieruch.de/react-warning-cant-call-setstate-on-an-unmounted-component

For functional components you can use a custom hook:
import { useEffect, useRef } from "react";

const useIsMounted = () => {
  const isMounted = useRef(false);
  useEffect(() => {
    isMounted.current = true;
    return () => (isMounted.current = false);
  }, []);
  return isMounted;
};

and then use it in your component:
const MyComp = () => {
  const isMounted = useIsMounted();

  useEffect(() => {
    if (isMounted.current) {
      // update your state here
    }
  }, [isMounted]);

  /// ...
};

PS: This might not be worth while.

I have met this problem in a situation where the keys were massed up. So it might be coming from above the component itself.
In my case the mistake was caused by how I have been generating the keys:

{collection.map(o =>
 {o.title}
)}

That causes a memory leak as React can't notice some elements must be removed or created as the key changes on any other change in the collection element.
My conclusion is that key should be unique during the life cycle so the state mechanism can rely on it.

how to resolve can't perform react state while using firestore

Try https://github.com/iShawnWang/withUnmounted package as an elegant solution

thanks @iShawnWang , work fine for me

i had to rerender the whole app  in a specific case, so had changed key in useEffect of the container component.
And this issue was coming.

what can I do I get the same problem that is can't perform a react state on an unmounted component...
`
const Header = () => {
const [collection, setCollection] = useState([]);
const FetchCollection = async () => {
const response = await fetch('http://localhost:5000/collection');
setCollection(await response.json([]));
}
console.log(collection)
useEffect(() => {
FetchCollection();
}, []);
`

what can I do I get the same problem that is can't perform a react state on an unmounted component...
`
const Header = () => { const [collection, setCollection] = useState([]);
const FetchCollection = async () => { const response = await fetch('http://localhost:5000/collection'); setCollection(await response.json([])); } console.log(collection)
useEffect(() => { FetchCollection(); }, []); `

In your case if the component is unmounted before the fetch return the setCollection will still be called. Also you don't return anything out of Header and I am not sure if that is intended.
One could use a ref to understand if the component was unmounted:
const Header = () => {
 const ref = useRef(null);
const [collection, setCollection] = useState([]);
const fetchCollection = async () => {
const response = await fetch('http://localhost:5000/collection');
ref.current && setCollection(await response.json([]));
}

useEffect(() => {
ref.current = true
fetchCollection()
return () => {
   ref.current = null
}
})
return
{collection.map(item =>

)}
</hrader>

}


Add .runSync - synchonous .run; make .sandboxed synchronous, run finalizers synchronously, make YieldMaxOpCount overridable

Experimental change, requesting comments.
I think that as a step towards SyncIO we should have a synchronous .sandbox operation. This PR makes .sandboxed synchronous by default and adds two more prim ops. I also took the liberty to run finalisers synchronously instead of forking them, since they were only forked for their ExitResult anyway.

I thought the RTS was supposed to do all the running? I am not sure you'd want "runny" methods in IO at all really...

It's not an analogue of unsafeRun, just a synchronous analogue for the current .run method (defined as fork >>= observe)

@kaishh I'd like to be even more ambitious and simply make .run "synchronous" (up to the limitations of the underlying IO). What do you think about that?

@jdegoes no objections, I've made .runSync separate just in case someone was relying hard on .run spawning a new fiber.


Remove type check for secret, fixes #9

This is to support Buffer like jsonwebtoken itself does.

Awesome! Once you remove the throws an error if secret is not a string test everything should pass again and we can make a new release.

How do I do that in the same PR on Github?

You can just remove the test and push back to your patch-1 branch and this pull request will be updated automatically. We still need this PR open since #11 just removes the test.

Ill start over tomorrow

All good! EI just pushed a fix in #12 and made a new patch release. Thanks a lot for doing the digging and the PRs, I hope this fixes your issue.

I can confirm the issue has been fixed. Thanks for moving so quickly.

Not sure how I missed this one but we shouldn't have removed the check entirely. If jwtSettings.secret is undefined we should still toss an error.

Ah nvm. Ignore me 🤦♂ Fixed properly in #10


Requiring files for coverage not working properly

I'm submitting a bug report
The documentation says we can use a structure like this to include/exclude files from coverage:
const srcContext = require.context('../../src/framework', true, /\.vue$/)
srcContext.keys().forEach(srcContext)

However, the RegEx doesn't do anything and webpack simply includes every single file it runs into, not taking the RegEx into account at all.
Webpack version:
2.2.1
Webpack Karma version:
2.0.2
Karma version:
1.4.1
Please tell us about your environment:
OSX 10.12.5 (16F73)
Browser:
PhantomJS prebuilt 2.1.14
Current behavior:
srcContext in the entry file doesn't actually do anything.
The following statement doesn't only take .vue file, it just includes everything webpack runs into:
const srcContext = require.context('../../src/framework', true, /\.vue$/)
In fact, even my index.js file in the test directory is included.
Expected/desired behavior:
Only .vue files to be included for coverage or anything I match with the RegEx.

If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal demo of the problem along with a gist/jsbin of your webpack configuration.
I'm using vue-cli webpack to setup a project. Everything is pre-configured. Just starting a new project, adding files to the project and adjusting index.js shows this bug.

What is the expected behavior?
See above

What is the motivation / use case for changing the behavior?
This feature is documented but apparently not working (in this case), which throws off our coverage reports by a lot.

Try to configure babel-plugin-istanbul instead
.babelrc
...
  "env": {
    "test": {
      "presets": ["env", "stage-2"],
      "plugins": [
        ["istanbul", {
          "include": ["**/*.vue"],
          "exclude": ["**/*.spec.js"]
        }]
      ]
    }
  }
...

Closing as this is 3 major versions behind and I'm unable to triage. If you're still having issues on the current stable release 5.0.0, feel free to open another issue and I'll be happy to assist!


Multiple jQuery Issue Conflict with Jcarousel

I am using jcarousel on Wordpress with a theme. I can see in the source that there's a couple versions of js loading and removing one will allow my test page (http://rochesterrealbeer.com/test-2/) for carousel to work. I was able to get it to work on another site (http://american-sweeps.com/) and comparing helped me lead to that it is an issue with too many js versions loading. I tried to use noConflict(); but frankly I have no idea what I am doing and I am willing to pay for someone to help me. If you think you can, email me at<EMAIL_ADDRESS>

Yep, you're loading jQuery 2 times. The second one is loaded after jCarousel and it overwrites the existing jQuery and all registered plugins.

Thanks jsor, I understand what the issue is, but I do not know how to solve it. I have seen you close other comments relating to this and I am reaching out this way because I do not know how to solve it. The plugin is awesome, but this issue seems to happen for other people as well. I even saw in your documentation that this is a common issue. However, I am looking for a solution or explanation because jQuery is not my expertise. If the solution is to use noConflict(); then what specific file should I change in this plugin or my Wordpress? I tried to use noConflict, but I don't know the proper way to use it. I want to continue using the plugin, so I am looking for help to continue using it. I respect that this plugin takes time and effort to create and willing to compensate someone's time to help me. I understand that the problem is not with your plugin, but how I chose to use it. If you know how to fix the issue, could you email me at<EMAIL_ADDRESS>to let me know what your rate would be to fix this? If it is simply a copy/paste of some code, I would be happy to pay for that as well or even if it is to pay for an hour of your time on Skype to tell me how to do it. If you can't help, could you refer me to a jQuery expert that can help me? Thanks~

This has nothing to do with jCarousel. The issue is, that you load jQuery 2 times. The second loading of jQuery resets all  registered plugins (not only jCarousel).
So, the solution is simple: Load jQuery only once.


Understanding C++ storage class

We ask that you please be patient as our team works through approving and publishing all articles/tutorials in a timely manner. Allow 1-3 days for a topic to be reviewed and/or approved - allow 3-7 days for an articles to be reviewed and/or published.
Be sure to visit our Resources Page for tools, resources, and example articles to go over.
NOTE: (Please ensure that you have only one open issue + linked pull request at a time. This will ensure that we complete the article in a timely manner from inception to publishing.)
NOTE: We tend to stray away or tend not to publish reviews/comparisons of commercial product offerings.
We're looking for the first 2-3 paragraphs of the article that appropriately summarize what your article will be about.
Proposed title of article
Understanding C++ storage class
Introduction paragraph (2-3 paragraphs):
Each variable has a data type when defined in C++ to help the user understand what type of variable it is. Since we were using the default Storage classes, we didn't identify any storage classes yet. When defining a variable, the compiler assigns a storage class by default.  In C++, storage classes are used to describe the characteristics of a variable and functions. It specifies the scope, lifetime, and visibility of variables and functions. These characteristics enable us to track the presence of a variable over the course of a program's execution. This article will look at various storage classes and code examples that show how they work.
Key takeaways:

What is the storage class in C++
Types of storage classes
Automatic storage class
Register storage class
Static storage class
External storage class
Mutable storage class

References:
N/A
Templates to use as guides

How To Guide Template
Software Review Template
Tutorial Template

closing for now - since there is another one approved one.
can reopen when the 1st one is published