cy.get('@archive') inside interceptor yields returned value to current subject

Describe the bug
I see the cypress implementation has a interceptor which calls a few cypress commands when a request happens. It seems that when you do it, the command you called set the subject on global level. If the command happens with the right timing, it will replace the global subject right when actual test commands are running.
e.g.
cy.get(MY_SELECTOR_HERE).should('have.text', 'whatever')
when the interceptor calls cy.get('@archive'), it will replace the selector subject with the archive value. Meaning the assertion .should('have.text', 'whatever') will run against the archive value and will probably break.
To Reproduce
A little hard to reproduce as you depend on timing here... But if you have a few tests it should be enough
Expected behavior
I would expect it not to happen
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Tried to debug it with some tests I have locally... It seems that any kind of command you call on interceptor will replace subject whille tests are running and break things.
Maybe you need to rely on Cypress.env(key, value) or some other strategy to store archives? Tested it locally and it seems to work

Just wanted to say that if you're ok with the Cypress.env approach, I can open the PR if cypress is not priority for you guys ATM. I know its not even released oficcialy yet

Hi @tsirlucas, you're correct that we haven't released Cypress officially yet, though I appreciate your feedback.
We're actually going to take a different approach to archiving resources (using Chrome Devtools Protocol instead of cy.intercept()), so let's hold off on the PR. But thanks for pointing this out

Going to close this issue since, Chromatic went with an alternative approach.


Incorrect position of spans

The positioning for some spans seems a bit incorrect.
I can't upload the full json to github because the content is sensitive (I can share it in private) but I'm attaching some pictures that display the problem (I edited the name of the services).

The first span corresponds to an hystrix command (circuit breaker), and the child comes from an http client that calls that "bar" service.
The relations between span and parent span seem correct, as well as the start times and durations.

@worldtiki : Acknowledged. @bulicekj : Can you have a look at this?


Update for Selenium 4.10 compatibility

Some things broke with Selenium 4.10 which removed some deprecated options and changed the arguments to webdriver.Chrome(). This led to things like this error:
TypeError: __init__() got multiple values for argument 'options'

@kumaF can we merge this one? :pray:

@kumaF could you please merge it.

When do you think it will be available on PyPi guys?
Thanks for your work! ❤

Dear Sir @kumaF 🙏

What do we do to make the updated version available on PyPi?

Hi Guys, It's now available on PyPI.

Thank you sir!


Hyprland Crashes due to failiure at binding to socket @/tmp/.X11-unix/X0

Steps to reproduce the bug:

Installation of the dotfiles https://github.com/end-4/dots-hyprland/tree/main
Start Hyprland from TTY

Description of the bug:
From the login screen I encounter the image shwon below:

From TTY the core falls out:
Log files:

I wasn't able to find anything related to this issue, your help would be appreciated!

Output of journalctl -xeu postgresql.service:

https://github.com/hyprwm/aquamarine/issues/70 likely
post a debug stacktrace there


Improve scrolling in chat and studio

Use react-scroll-to-bottom to improve scrolling conversation to bottom.

Works, but package seems pretty stale?

Works, but package seems pretty stale?

Yes, it is indeed a bit stale, but works nevertheless, can rewrite it to our needs if it causes any issues


[BioRef] When running a query with more than one analyte, the histograms are not correct

They are equal and based on all the patients having at least one analyte, most probably there is an error in the sql query retrieving the patients

Fixed by #33 :+1:


A11y_Azure Tools Storage Explorer_Activities_Activities tab_ScreenReader: When the voiceover is turned on and the user uses Control + Option + Arrow keys on successful message notification, the user has to press Control + Option + Arrow keys three successive times to move to next item.

GitHubTags: #A11yMAS;#A11yTCS;#A11ySev2;#BM_AzureToolsStorageExplorer_May2022;#Azure Tools Storage Explorer;#MAC;#FTP;#DesktopApp;#WCAG1.3.1;#Voiceover;#AINotInScope;
Environment Details:
Application Name: Azure Tools Storage Explorer
Application Version: 1.24.0
Mac Version: macOS Monterey 12.3.1
Screen reader: Voiceover
Repro Steps:

Please do not close this bug. This bug should only be closed by TCS, C+AI Accessibility tester after verification of the fix.
Launch Azure Tools Storage Explorer.
Sign in to Azure.
Tab to 'Explorer' pane tree view.
Navigate to any of the file inside "Blob Containers"
Tab to 'Activities' tab.
Tab till any of the successfully completed message.
Turn on voiceover and use VO + Arrow keys (Control + Option referenced as VO)

Actual:
When the voiceover is turned on and the user uses Control + Option + Arrow keys on the 1st successful message notification, the user has to press Control + Option + Arrow keys three successive times to move to next item and all the three times while using the keys the same information is being read.
Expected:
When the voiceover is turned on and the user uses Control + Option + Arrow keys on successful message notification, the user should press Control + Option + Arrow keys once to move to next item.
User Impact:
Screen reader users will receive the same information thrice.
Attachment:

@MRayermannMSFT, I think your comment here is same as on the bug #5769. Here, the issue is that while moving from one text element to another where user has to press 3 times [Control + Option + Arrow] on text element to move to next instead of one keyboard stroke.

Hi @bbarik06
Please verify that this bug is fixed using the Storage Explorer build at: https://storageexplorerpublish.blob.core.windows.net/privatebuilds/20220624.3-accessibility-testing/StorageExplorer-darwin-x64.zip

Issue is not repro now. Hence, closing the bug. Please refer the attachment.
Version: 1.25.0-dev


Correct helm chart issue with external kafka

correcting template formatting issue for use with external kafka provider

This has been fixed


Add DisableAccount after logout.

Add function to disable account after logout.
Add enable account if user tries to login

something we can use
but i have to find the time for testing


Allow for selecting regions for alignment

Allow users to select regions to perform sobel filtering. This can allow users to determine localized alignment of graphene domains when growing on polycrystalline substrate.
Future idea: build an algorithm to auto segment the image based on alignment of the segmented regions.

Active?, Done?
if active, add labels, assign and move to backlog or in progress.
if done, set release and close


Support Multiple IAM adds for GCP data model

Background
Closes https://app.asana.com/0/1202002271043343/1202058632039542/f
Changes
Add Support for multiple IAM adds to GCP data model
Update Black to remediate this bug https://github.com/psf/black/issues/2964
Make fmt on some files that didnt get it pre-merge
Testing
Unit testing

looks like you have some failing tests

looks like you have some failing tests

Yeah, I need to get the Python 3.9 update merged. Updated Black to get around a bug, which introduced another etc etc.
3.9 coming very soon


Added Quick Select [C]

Fixes #10

By submitting this pull request I confirm I've read and complied with the below requirements.

[x] I have read the Contribution guidelines and I am confident that my PR reflects them.
[x] I have followed the coding guidelines for this project.
[x] My code follows the skeleton code structure.
[x] This pull request has a descriptive title. For example, Added {Algorithm name} [{Language}], not Update README.md or Added new code.

@yashLadha: Please mention an issue that is completely closed by this PR.

Also it is clear that you have made a false tick on the requirements mentioned for the PR. Please correct the errors.

@yashLadha : This PR is not consistent with contributing guidelines. Please refer to it and make the changes.

The closing issue link is mentioned in my commit message.

Ammended Necessary changes.

Looks okay. Let's wait for some more eyes.

@Monal5031 : Do things look good?


Do not release runtime lock when recording values : 1.8x faster value recording

Unfortunately we can only control concurrency mode for the entire library.
But we should not release the runtime lock in 'record_value', because that would:

introduce additional overhead in the form of a potential context switch to another thread (perhaps fundamentally altering the behaviour of the application that we're trying to observe)

it is not necessary because recording the value is supposed to take nanoseconds

A measurement on test1.ml shows a significant speedup:
Benchmark 1: ./old.exe
  Time (mean ± σ):      4.485 s ±  0.032 s    [User: 4.467 s, System: 0.002 s]
  Range (min … max):    4.411 s …  4.510 s    10 runs

Benchmark 2: ./new.exe
  Time (mean ± σ):      2.425 s ±  0.003 s    [User: 2.414 s, System: 0.002 s]
  Range (min … max):    2.422 s …  2.430 s    10 runs

Summary
  ./new.exe ran
    1.85 ± 0.01 times faster than ./old.exe

I tried a few other optimizations too (using int instead of int64_t, and manually inlining the foreign call in hdr_record_value, but those don't result in measurable differences.)

This change looks good to me. It feels fine not to release the runtime lock for what is expected to be a fast call.


fix gpgrest and gpgnative conflict when using gpgrest on android

Hi, why do you want to change this?
extension-gpgrest and extension-googleplaygames both uses the same class name (extension.gpg.GooglePlayGames), so that you can switch between them without modifying your code.
extension-multiads, takes advantage of that (as it never includes both extensions at the same time).
If you're having conflicts between extension-gpgrest and extension-googleplaygames it may be a indication that you're including both extensions at the same time (which you shouldn't do).
Am I missing something?


Can't open 2D export from Google Sketchup

I'm having trouble opening DXF exported (2D) from Google Sketchup 2016.
IxMilia.Dxf.DxfReadException was unhandled by user code
  HResult=-2146233088
  Message=Unexpected code pair group text
  Offset=3399
  Source=IxMilia.Dxf
  StackTrace:
       at IxMilia.Dxf.DxfCodePairGroup.FromBuffer(DxfCodePairBufferReader buffer, String groupName) in E:\IxMilia\Dxf\src\IxMilia.Dxf\DxfCodePairGroup.cs:line 86
       at IxMilia.Dxf.Objects.DxfObject.TrySetExtensionData(DxfCodePair pair, DxfCodePairBufferReader buffer) in E:\IxMilia\Dxf\src\IxMilia.Dxf\Objects\DxfObject.cs:line 135
       at IxMilia.Dxf.Objects.DxfXRecordObject.PopulateFromBuffer(DxfCodePairBufferReader buffer) in E:\IxMilia\Dxf\src\IxMilia.Dxf\Objects\DxfXRecordObject.cs:line 30
       at IxMilia.Dxf.Objects.DxfObject.FromBuffer(DxfCodePairBufferReader buffer) in E:\IxMilia\Dxf\src\IxMilia.Dxf\Objects\DxfObjectGenerated.cs:line 314
       at IxMilia.Dxf.Sections.DxfObjectsSection.ObjectsSectionFromBuffer(DxfCodePairBufferReader buffer) in E:\IxMilia\Dxf\src\IxMilia.Dxf\Sections\DxfObjectsSection.cs:line 43
       at IxMilia.Dxf.Sections.DxfSection.FromBuffer(DxfCodePairBufferReader buffer, DxfAcadVersion version) in E:\IxMilia\Dxf\src\IxMilia.Dxf\Sections\DxfSection.cs:line 70
       at IxMilia.Dxf.DxfFile.LoadFromReader(IDxfCodePairReader reader) in E:\IxMilia\Dxf\src\IxMilia.Dxf\DxfFile.cs:line 212
       at IxMilia.Dxf.DxfFile.Load(Stream stream) in E:\IxMilia\Dxf\src\IxMilia.Dxf\DxfFile.cs:line 144
       at FloorFlood.---.OpenDxFiles(IEnumerable`1 filePaths)

Lines 3399 to 3419 of DXF file
102
VTR_0.000_0.000_1.000_1.000_GRIDDISPLAY
 70
     3
102
VTR_0.000_0.000_1.000_1.000_GRIDMAJOR
 70
     5
102
VTR_0.000_0.000_1.000_1.000_DEFAULTLIGHTING
280
     1
102
VTR_0.000_0.000_1.000_1.000_DEFAULTLIGHTINGTYPE
 70
     1
102
VTR_0.000_0.000_1.000_1.000_BRIGHTNESS
141
0.0
102
VTR_0.000_0.000_1.000_1.000_CONTRAST
142
0.0
  0

Thank you for the bug report.  This looks like another case where the Autodesk DXF spec isn't complete; the spec says 102 codes are supposed to be of the format:
102
{some-string
// a bunch of other code pairs
102
}

But this looks like a specialized shortened version used when only one code pair would be in the group.
I have two questions for you:

How were you using this library?  NuGet package or clone/build locally?  What version/commit?
Can you email me a file that leads to this error<EMAIL_ADDRESS>

Figured about the 102{ stuff from source code, but I'm not familiar with DXF spec. If this VTR thing is uncommon, I can manage without you changing the code. I can just delete those 6 parameters from the file. I tried that and now it works.
I'm using online NuGet version 0.3.1.
Here are both Sketchup file and DXF export file.
FloorSkpAndDxf.zip

FYI, I've created PR #62 which will allow this file to open.
My original guess about the 102 handling was off and the problem was ultimately that the XRECORD object in your file didn't specify the duplicate record handling flag which means a subsequent read of the code 280 pair was mis-interpreted as that which caused the rest of the object to be read incorrectly.

@brettfo Thanks for the quick response and fix!


Optional lookup

Adding optional lookups
In use-cases when looked-up stack is not present, there should be an option to silently ignore such errors.
lookup:stack:::optional - will return empty string if there are no stack, instead of Exception for lookup:stack::

Build triggered. sha1 is merged.

Build started sha1 is merged.

All is well

Build triggered. sha1 is merged.

Build started sha1 is merged.

All is well

Build triggered. sha1 is merged.

Build started sha1 is merged.

All is well


Initial StarlingMonkey Build

This implements an initial port of js-compute-runtime to be built on top of StarlingMonkey. The new runtime build is entirely self-contained in runtime/fastly and the build scripts are located there, also runnable via npm run build:starlingmonkey:debug and npm run build:starlingmonkey.
In the CLI a new --starlingmonkey flag is added, but not documented, and the StarlingMonkey build is not included in the release.
In testing, we branch both the main index.js file into a StarlingMonkey-specific index-starlingmonkey.js in the integraiton tests and also the tests.json file into a tests-starlingmonkey.json. By extending the lists in these files, to eventually include all tests, we should eventually be able to hit parity with the main test suite and then remove these changes.
The main test.js now takes a --starlingmonkey flag along with the former --local flag, which then performs the branching into the above and also updates the build script to use the starlingmonkey build.
In CI, a new StarlingMonkey workflow will run the StarlingMonkey debug build and tests, using the above new flag.
Current test status is just the btoa test passing. But this is a good start from which in theory it is just a matter of extending support and coverage.

Moved to https://github.com/fastly/js-compute-runtime/pull/764.


test fixes, cs fixes

Coverage decreased (-1.4%) to 98.63% when pulling 5fc122e8a8cb141be72fec71a609804d9ba008b1 on refactor into 9bc438081e44e6793599d835ce9d9b445113f71d on master.

Coverage decreased (-1.4%) to 98.63% when pulling 5fc122e8a8cb141be72fec71a609804d9ba008b1 on refactor into 9bc438081e44e6793599d835ce9d9b445113f71d on master.

Coverage decreased (-1.4%) to 98.63% when pulling 5fc122e8a8cb141be72fec71a609804d9ba008b1 on refactor into 9bc438081e44e6793599d835ce9d9b445113f71d on master.


Workaround way to do offsetlazyload ?

I know you guys are working on offsetLazyload  on #390.
However, I would like to know whats workaround way of achieving that?
Basically, I would like to load prev and next image so that it can have transition

I was able to work around this with the following code:
class Carousel extends React.Component {
	constructor(props) {
		super(props);
		this.state = {
			images: props.images.map((image, index) => {
				// Add a loaded:true property on the first 3 images.
				const loaded = index < 3;
				return { ...image, loaded };
			})
		};
	}
	lazyLoad(targetImage) {
		// Set the loaded property to true for the target image.
		const state = this.state;
		const currentImages = state.images;
		const updatedImage = targetImage;
		updatedImage.loaded = true;

		this.setState({
			...state,
			images: [...currentImages, ...updatedImage]
		});
	}

	render() {
		const settings = {
			dots: false,
			infinite: false,
			speed: 500,
			slidesToShow: 1,
			slidesToScroll: 1,
			afterChange: currentImage => {
				// Load 2 slides ahead of the current slide, after the slide transition.
				// Setting state anywhere but inside beforeChange/afterChange will
				// cause a re-render and prevent the slide from transitioning properly.
				const targetImage = this.state.images[currentImage + 2];
				if (targetImage && !targetImage.loaded) {
					this.lazyLoad(targetImage);
				}
			},
		};

		const images = this.state.images.map(image => {
			const { id, uri, loaded } = image;
			// If the image should be loaded, insert its uri
			// otherwise insert blank.gif and add a class of lazy-image.
			return (
				<Image
					key={id}
					src={loaded ? uri : '/images/blank.gif'}
					className={loaded ? null : 'lazy-image'}
				/>
			);
		});

		return (
			<Slider ref="slider" {...settings}>
				{images}
			</Slider>
		);
	}
}

export default Carousel;

We have implemented the lazyLoad just like slick does, that is, you can pass ondemand or progressive value to lazyLoad prop to achieve two different kinds of lazyLoading.


Transforming "What Could Go Wrong" into a game that focuses on online communities by Vincent TO

In order to transform "What Could Go Wrong - LLM Edition" into a game that focuses on online communities, I thing that it needs to adjust the content and structure to reflect the dynamics, potential issues, and unique situations that can arise within these spaces. The core idea would remain the same: to encourage discussions and reflections on ethical issues, but with a focus on online communities.

It is also an array of changes I want to explore to


Fix wrong overhead memory when run in hybrid environments

What this PR does / why we need it:
Fix wrong overhead memory, when run in hybrid environments, such as x86 host together with arm host
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
virt-controller render launcher pod resource request, and call GetMemoryOverhead func to calc overhead memory for launcher pod,  this func use ClusterCPUArch for all vmis.  when run in hybrid environments, will get incorrect memory overhead value for those vmis whose architecture is different with virt-controller.
so change to replace ClusterCPUArch  with vmi.Spec.Architecture.
Special notes for your reviewer:
Checklist
This checklist is not enforcing, but it's a reminder of items that could be relevant to every PR.
Approvers are expected to review this list.

[ ] Design: A design document was considered and is present (link) or not required
[ ] PR: The PR description is expressive enough and will help future contributors
[ ] Code: Write code that humans can understand and Keep it simple
[ ] Refactor: You have left the code cleaner than you found it (Boy Scout Rule)
[ ] Upgrade: Impact of this change on upgrade flows was considered and addressed if required
[ ] Testing: New code requires new unit tests. New features and bug fixes require at least on e2e test
[ ] Documentation: A user-guide update was considered and is present (link) or not required. You want a user-guide update if it's a user facing feature / API change.
[ ] Community: Announcement to kubevirt-dev was considered

Release note:

none

Hi @wangjihai,
Thanks for contributing.
/retest

/test pull-kubevirt-build

/test pull-kubevirt-build

/retest

/retest

@wangjihai Sorry for the late reply. I have triggered the tests and will review them once they finish

/cc @fossedihelm

Thanks for the contribution! 🙂 The changes look good overall, let's see what the outcome of the tests. Is vmi.Spec.Architecture required in hybrid environment? I wonder because in case of migration there should be a different overhead that is not adjusted. If not, IMO, we should make some changes here too:
https://github.com/kubevirt/kubevirt/blob/7e3aa9fff0226fea881fa4efc7906bad3cfcf4be/pkg/virt-controller/watch/migration.go#L650

vmi.Spec.Architecture is necessary in hybrid environment. otherwise,  it maybe scheduled vm to different arch host,  thus vm can't run correctly.

Thanks @wangjihai! Can I ask you to squash the 2 commits in a single one (the first commit message for both of them is ok)? This will help the future us to understand the rationale :)
Thank you!

/retest

/retest

@wangjihai Thank you for the change. The PR should be automatically retested and merged.


Z order problem with GLWpfControl

Hi Hjoykim,
I am seeing weird z order rendering issue with GLWpfControl. Any idea what is wrong?

This was a project you uploaded previously, I turned off the wireframe and rotated the scene, the red cube should be in front. I have attached the project here for reference, Thanks in advance.
GLWPFDemo241107.zip

It runs properly on my computer.
It seems to be related to the driver program of the graphics card, but I don't know the exact cause.
My graphics card specifications are as follows.
NVIDIA Corporation
NVIDIA GeForce GTX 1060 3GB
Driver version : 560.94


High Vulnerability detected in AWS inspector for the lambdas

Hi Team,  We ran the cloud formation and enabled the AWS inspector. We got vulnerabilities that are very high. Attached is the screenshots. Also the description is given below. Could you please help us in fixing this. Also is it possible to include the fix for these high vulnerabilities in your next release.
body-parser-->body-parser is Node.js body parsing middleware. body-parser <1.20.3 is vulnerable to denial of service when url encoding is enabled. A malicious actor using a specially crafted payload could flood the server with a large number of requests, resulting in denial of service. This issue is patched in 1.20.3.
async-->In Async before 2.6.4 and 3.x before 3.2.2, a malicious user can obtain privileges via the mapValues() method, aka lib/internal/iterator.js createObjectIterator prototype pollution.

Thanks
Anju

Hi Thanks For reporting his. Which version of QnABot are you using?  Have you upgraded to the latest available version https://solutions-reference.s3.amazonaws.com/qnabot-on-aws/latest/qnabot-on-aws-main.template?

hi @fhoueto-amz , we are using version-v[6.1.1] - 2024-09-26.

hi @fhoueto-amz and @michaelin-96, we are using version-v[6.1.1] - 2024-09-26.
Could you please fix this asap.

@anjugds we are not seeing the same thing on v6.1.1. To confirm the version you are using, view all vulnerabilities, click on the vulnerability title, then find the  affected resource name and click on it, that should bring you to the lambda, then click on the configuration tab and then on tags. That will give you the cloudformation stack-id.  Go to cloudformation and find that stack. When you click on the stack name, you should see the description field under the stack info tab. This will give you the version  that has deployed that lambda in the format "(SO0189) QnABot with admin and client websites - Version v6.1.1". Can you share the version which has deployed the body-parser vulnerability and the async vulnerability under ESQIDLambda, ESCleaningLambda and ImportStep Lambda.

Hi,
Please find the screenshot from CloudFormation and CFN Lambda.

@fhoueto-amz  and @michaelin-96  Could you please help in this!

We are working on client environment and its moving towards escalations as they all are High vulnerabilities. @fhoueto-amz @michaelin-96

@anjugds , @preethy-1
I am confused. Your original post said that the body parser issue is solved in 1.20.3 but you were seeing an older version of body parser being used in QnABot v6.1.1.
Quoting your original submission:
body-parser-->body-parser is Node.js body parsing middleware. body-parser <1.20.3 is vulnerable to denial of service when url encoding is enabled. A malicious actor using a specially crafted payload could flood the server with a large number of requests, resulting in denial of service. This issue is patched in 1.20.3."
The screenshot attached though is showing that the version of body-parser used is already 1.20.3. It seems that the lambdas being scanned are not from 6.1.1. I would suggest you do a clean deployment of the latest version of QnABot in a separate region/account and run the inspector on it to avoid potential interference with previous versions of the lambdas.

@anjugds , @preethy-1
Any update from your side?

Closing this. Please reopen if needed


Aragog constant

This work introduces Aragog as an alternative module for interior modelling.
It uses constant physical properties and runs fine athough the coupled results seem to not behave properly, see here.
I suggest we create new issues to debug coupled behaviour and expand from this current state (add config parameters, use look up tables, plot results).

I am off next week so do not hesitate to change/approve/merge in my absence @planetmariana @timlichtenberg . I also add you @nichollsh as reviewer as you already managed to run this branch.

Thanks Laurent! All three of us @planetmariana @nichollsh and me should have a look through this and add a review. Once that is done, we can merge it in the next days.

This is what I get from the dummy config when replacing the dummy interior with aragog. I think this is generally looking okay - the planet at least cools down and the melt fraction decreases.


Swipe down to refresh too sensitive

Describe the bug
When scrolling up from a webpage, once you reach the top of the page, the swipe down to refresh begins to activate. Sometimes, this is way too sensitive, and a refresh gets triggered pretty much right away, without giving enough time to avoid it. It's like scrolling up the page also "revs up" the refresh motion, and once you reach the top of the page the arrow has already reached full circle and it causes a refresh.
How to Reproduce
Steps to reproduce the behavior:

Go on any web page
Scroll down a bit
Scroll back up to the top of the page
Repeat a few times until it happens

Expected behavior
Ideally, I think scrolling up from a page should never trigger a refresh unless you explicitly lift your finger from the screen and swipe down a second time after having reached the top of the page. This bug is annoying, but it's still possible to accidentally trigger a page refresh sometimes just from swiping down too much when scrolling up a page, and that can be a bit irritating.
Smartphone (please complete the following information):

DDG App Version: 5.75.1
Device: Nexus 5
OS: Android 7.1.2

https://user-images.githubusercontent.com/43770697/106349595-149aaa00-629d-11eb-81b5-99ade90d6999.mp4

Okay, so you don't actually need to be scrolling up much for this to happen. I've had it happen earlier where I was near the top of the page (youtube video), scrolled up just a little bit to reach the top of the page, and it just instantly refreshed the page.

Okay, so you don't actually need to be scrolling up much for this to happen. I've had it happen earlier where I was near the top of the page (youtube video), scrolled up just a little bit to reach the top of the page, and it just instantly refreshed the page.

Thanks for reporting the bug. I agree, I've noticed that non consistent behavior in our app and sometimes pull to refresh is triggered unexpectedly.
I've created an internal bug to track the issue: https://app.asana.com/0/414730916066338/1199888021167463/f
However, we accept community contributions to fix this bug. Thanks


Make default logs/stats capped collections smaller

please also provide some documentation re this functionality...

LGTM!


[3.x] Mono module depends on distutils, which is no longer part of the base Python distribution after 3.12

Tested versions

Reproducible in 3.x and 3.5 branches

System information
Fedora Linux 39 (KDE Plasma) - Wayland - Vulkan (Forward+) - dedicated AMD Radeon RX 7600M XT (RADV NAVI33) () - AMD Ryzen 7 7840HS w/ Radeon 780M Graphics (16 Threads)
Issue description
Python 3.12 removed distutils from the base distribution, and recent SCons versions also stopped relying on it, so pip install scons won't automatically pull it in.
Our Mono module depends on it currently, so it requires an extra configuration step with pip install setuptools which we either need to document, or, if we can, remove that dependency.
Building with flags: verbose=yes warnings=all werror=yes debug_symbols=no module_mono_enabled=yes mono_static=yes mono_glue=no
scons: Reading SConscript files ...
Auto-detected 4 CPU cores available for build parallelism. Using 4 cores by default. You can override it with the -j argument.
Scons cache enabled... (path: '/home/runner/work/godot/godot/.scons-cache/')
Checking for `thread_local` support... supported
Found Mono root directory: /usr/lib/pkgconfig/../../lib/..
ModuleNotFoundError: No module named 'distutils':
  File "/home/runner/work/godot/godot/SConstruct", line 757:
    SConscript("modules/SCsub")
  File "/opt/hostedtoolcache/Python/3.12.3/x64/lib/python3.12/site-packages/SCons/Script/SConscript.py", line 676:
    return method(*args, **kw)
  File "/opt/hostedtoolcache/Python/3.12.3/x64/lib/python3.12/site-packages/SCons/Script/SConscript.py", line 612:
    return _SConscript(self.fs, *files, **subst_kw)
  File "/opt/hostedtoolcache/Python/3.12.3/x64/lib/python3.12/site-packages/SCons/Script/SConscript.py", line 279:
    exec(compile(scriptdata, scriptname, 'exec'), call_stack[-1].globals)
  File "/home/runner/work/godot/godot/modules/SCsub", line 21:
    SConscript(name + "/SCsub")  # Built-in.
  File "/opt/hostedtoolcache/Python/3.12.3/x64/lib/python3.12/site-packages/SCons/Script/SConscript.py", line 676:
    return method(*args, **kw)
  File "/opt/hostedtoolcache/Python/3.12.3/x64/lib/python3.12/site-packages/SCons/Script/SConscript.py", line 612:
    return _SConscript(self.fs, *files, **subst_kw)
  File "/opt/hostedtoolcache/Python/3.12.3/x64/lib/python3.12/site-packages/SCons/Script/SConscript.py", line 279:
    exec(compile(scriptdata, scriptname, 'exec'), call_stack[-1].globals)
  File "/home/runner/work/godot/godot/modules/mono/SCsub", line 34:
    mono_configure.configure(env, env_mono)
  File "/home/runner/work/godot/godot/modules/mono/build_scripts/mono_configure.py", line 403:
    copy_mono_root_files(env, mono_root, mono_bcl)
  File "/home/runner/work/godot/godot/modules/mono/build_scripts/mono_configure.py", line 453:
    copy_mono_etc_dir(mono_root, editor_mono_config_dir, env["platform"])
  File "/home/runner/work/godot/godot/modules/mono/build_scripts/mono_configure.py", line 479:
    from distutils.dir_util import copy_tree

Maybe there's an alternative to distutils.dir_util.copy_tree in the Python stdlib now?
Steps to reproduce

Install Python 3.12 and SCons 4.7.0, without setuptools/distutils

Minimal reproduction project (MRP)
n/a

Fixed by #91159.


Make project modular

Right now the project is divided in two main part

mod_site
mod_api
But as we are adding more and more api the mod_api module is getting really big.
So to make the project more modular we should split the mod_api in 3 parts
mod_package
mod_auth
mod_search [as this will be one of the main functionality]

This is way code will more manageable

Why do you have the mod_ name space? Seems unnecessary.

Ok i will do that.
On 27-Dec-2016 1:33 PM, "Paul Walsh"<EMAIL_ADDRESS>wrote:

Why do you have the mod_ name space? Seems unnecessary.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/frictionlessdata/dpr-api/issues/175#issuecomment-269288884,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACE9Oe0gERPmCqxjQ80K5qcR86sEwpBSks5rMMZsgaJpZM4LWAW5
.

Fixed. Right now there are three moduled

package [holds all data package related apis]
auth [holds all authentication and authorization apis]
site [template rendering]

Future

search [will holds all search apis]


Block a specific IP

Hi, I can't make this work for some reason.
As I understand it, to block a specific IP I need 2 rules: one that blocks the IP and the other that allows the rest of the internet.
So I found my public IP (whatismyip.com) and I created those two rules, and because they are evaluated in order of priority (asc) I set the DENY to priority 1 and the ALLOW to priority 300.
My test failed: I can still communicate with my service.
I also tried the case of blocking everything (just add any rule with a random ip) and it blocks me correctly.
Could you please help?

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

ID: d90bbf62-44f9-30a6-d6ea-bbf776f6efc5
Version Independent ID: b3961eac-6c70-b345-f48c-e4c0ec67059c
Content: Azure App Service IP Restrictions
Content Source: articles/app-service/app-service-ip-restrictions.md
Service: app-service-web
GitHub Login: @ccompy
Microsoft Alias: ccompy

Just to be crystal clear, here is my resource manager configuration (I wrote X.X.X.X instead of my real IP):
...
"ipSecurityRestrictions": [
{
"ipAddress": "X.X.X.X/32",
"action": "Deny",
"tag": "Default",
"priority": 1,
"name": "Ban Me"
},
{
"ipAddress": "<IP_ADDRESS>/0",
"action": "Allow",
"tag": "Default",
"priority": 300,
"name": "Allow others"
}
],
...

@gmconte I was able to get your suggestion working by using the below values.
In my example, I tried taking the IP address off of my iPhone by going to into the WIFI settings section. This IP address was a private IP address though. I located the public IP address that my router was using by going to bing.com. I then rounded down the last two octaves to 0 and set a wide cidr just to be sure.
Since our solutions are pretty much the same, it looks like you grabbed the wrong IP address to try and block for your local machine.
ipSecurityRestrictions": [
{
"ipAddress": "X.200.0.0/12",
"action": "Deny",
"tag": "Default",
"priority": 100,
"name": "Block",
"description": "block"
},
{
"ipAddress": "<IP_ADDRESS>/0",
"action": "Allow",
"tag": "Default",
"priority": 300,
"name": "allow internet",
"description": "block"
}
We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.

Hi Bryan,
Thanks for looking into it. I still have some doubt considering that the
example you describe would block a huge number of ips.
My idea was to block a specific one, thus using the "/32" notation.
The IP I tried was external, identified using the whatismyip service and
the ipify javascript library.
Could you please try blocking a single "/32" ip address and confirm that it
works fine?
I want to make sure that this feature  can block a single IP instead of
working only with ranges.
On Fri, 24 Aug 2018, 00:14 Bryan Trach<EMAIL_ADDRESS>wrote:

@gmconte https://github.com/gmconte I was able to get your suggestion
working by using the below values.
In my example, I tried taking the IP address off of my iPhone by going to
into the WIFI settings section. This IP address was a private IP address
though. I located the public IP address that my router was using by going
to bing.com. I then rounded down the last two octaves to 0 and set a wide
cidr just to be sure.
Since our solutions are pretty much the same, it looks like you grabbed
the wrong IP address to try and block for your local machine.
ipSecurityRestrictions": [
{
"ipAddress": "X.200.0.0/12",
"action": "Deny",
"tag": "Default",
"priority": 100,
"name": "Block",
"description": "block"
},
{
"ipAddress": "<IP_ADDRESS>/0",
"action": "Allow",
"tag": "Default",
"priority": 300,
"name": "allow internet",
"description": "block"
}
We will now proceed to close this thread. If there are further questions
regarding this matter, please tag me in your reply. We will gladly continue
the discussion and we will reopen the issue.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/azure-docs/issues/13729#issuecomment-415602304,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADXVsoucTxrNdmH-k59FCekqppdGzfRBks5uTzc6gaJpZM4WFgZ_
.


Support GP3 IOPS and MB/S configuration parameters

Hello cpi team,
I wanted to ask if you plan to support the gp3 parameters for provisioning throughput and iops in the near future. We are in a bind because we have cases where the default throughput is not enough, but we would like to avoid having go io1 to avoid the disk credit trap.

Was bored. Wrote it myself: #126

For reference. I also wrote a PR for the bosh docs that can be merged once a release is built with iops and mbp/s abilities
https://github.com/cloudfoundry/docs-bosh/pull/762

#126 has been merged and has been shipped in: https://github.com/cloudfoundry/bosh-aws-cpi-release/releases/tag/v93


⚠ Bofrost.lu has degraded performance

In 14a7330, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 2240 ms

Resolved: Bofrost.lu performance has improved in 734703e after 13 minutes.


Messed up prayer times display

Runs fine for maybe a day but on a computer running 24/7 and a laptop that I close the lid and goes to sleep and wakes up regularly, the time displayed eventually ends up being messed up, trying to show the next Fajr time (even though the next prayer time is not Fajr) and not correctly - looks like it tries to show a range of time for hours, minutes, and seconds - no matter the time of day and the Azan does not play.
Desktop runs on Windows 11 and Laptop on Windows 10. Screenshots attached. Two are from when it messed up and one is when it is first run.

Just noticed an error in the part of the code that reloads the app at midnight, so I fixed that, but I don't know if it will be enough.
https://github.com/DBChoco/Muezzin/commit/20529aeec321a5add0ad784ce5c486732d861e69

Sorry to take so long getting back to you, basically leaving it on overnight and coming back the next day I noticed the Adhan didn't play at the prayer times in the app, and then opening the main window from the system tray it showed the incorrect times as above.
This happened if the PC it was running on was asleep (laptop) or stayed on the entire time (desktop). I'm not sure how long it took for it to stop functioning, usually I noticed it play the previous night when I first started the app and then the next day when I either turned the laptop on (from sleep) or was in the room the desktop was in, they just didn't play the Adhan.

Okay, I'll see if this is enough then. InshaAllah we'll figure it out.
Jazzaka Lahu khairan.

If it helps, I just noticed that if I go to settings and return back (no changes made), it looks like the times get corrected.

Tried the new version, 2.5.9, and it seems to have fixed the issue, Displaying the correct time and playing the Azan on schedule as it is supposed to, only tested on a windows 11 PC though. Closing this issue.


Add ability to convert a code folder

This file helps one to convert an entire directory of files to a single pdf.
All this takes place in one step.
It is based on code2tex.py.
The use case is when one has a folder full of code which has to be printed.
One can't use convert_all.py for this
purpose without some modifications.

Thanks for contributing!
You're right that convert_all.py doesn't do this, as it is specific to Moodle or Canvas downloads.  The intent is to have code2tex.py be flexible enough to handle the use case you're considering, too, though.  As noted in the readme, wildcards can be used to specify all files in a directory.  So I think something like python code2tex.py directory/* would do the same thing as this script without the need for an additional program.
I'd rather keep things simpler by having a single, flexible program.  If you think there is some functionality that isn't covered by code2tex.py that I'm missing, though, let me know, and I can probably add it to the existing script.

Thanks for contributing!
You're right that convert_all.py doesn't do this, as it is specific to Moodle or Canvas downloads.  The intent is to have code2tex.py be flexible enough to handle the use case you're considering, too, though.  As noted in the readme, wildcards can be used to specify all files in a directory.  So I think something like python code2tex.py directory/* would do the same thing as this script without the need for an additional program.
I'd rather keep things simpler by having a single, flexible program.  If you think there is some functionality that isn't covered by code2tex.py that I'm missing, though, let me know, and I can probably add it to the existing script.

Will   python code2tex.py directory/*  work on windows machines too?

Will   python code2tex.py directory/*  work on windows machines too?

I believe it should.  Wildcard expansion is done by the shell, so there is a chance that running it under some particular shell might not work, but I don't know of any shells that don't do wildcard expansion.
I'll go ahead and close this, but if you run into trouble getting it to work for you to convert a whole directory, feel free to open an issue.

I believe it should.  Wildcard expansion is done by the shell, so there is a chance that running it under some particular shell might not work, but I don't know of any shells that don't do wildcard expansion.
I'll go ahead and close this, but if you run into trouble getting it to work for you to convert a whole directory, feel free to open an issue.


Ceph FS fscrypt support

Add Ceph FS fscrypt support. Supports volumes, snapshots and clones.
Follow up to #3310 (adds fscrypt integration) and second in series towards #1563
Fixes: #1563
Usage Requirements

Kernel with Ceph FS fscrypt support

git://git.kernel.org/pub/scm/linux/kernel/git/jlayton/linux.git branch ceph-fscrypt
or https://github.com/ceph/ceph-client.git branch wip-fscrypt

https://github.com/google/fscrypt#runtime-dependencies
Ceph MDS support

v17: volumes supported
main branch: +snapshots
main branch+https://github.com/ceph/ceph/pull/48410: +clones

End-to-end Testing
Testing on minikube requires a custom ISO with the latest Ceph FS patches. To test snapshots and clones a main branch build of Ceph is needed as well.
Custom Minikube ISO
Use patches from https://github.com/irq0/minikube/tree/custom-ceph-fscrypt-kernel on top of minikube to build an ISO with the current Ceph FS development kernel.
Build summary:
make
make buildroot-image
make out/minikube-x86_64.iso

Then use minikube.sh with MINIKUBE_ISO_URL="file://path of minikube.iso"
Use Ceph Main Branch Images
Either use quay.io/ceph/daemon:latest-main-devel or build using ceph-containers project.
Build
In https://github.com/ceph/ceph-container.git clone:
eval $(minikube docker-env)  # will build inside minikube VM
make BASEOS_REGISTRY=quay.io/centos BASEOS_TAG=stream8 CEPH_DEVEL=true FLAVORS="main,centos,8" build

Optional: patch image with https://github.com/ceph/ceph/pull/48410

ceph-csi-encryption-kms-config configmaps attached as volumes (like their RBD counterparts). My read is that the update tests fail, because they are not present.

You can set https://github.com/ceph/ceph-csi/blob/devel/e2e/cephfs.go#L73 to true so that it will continue if the file is not present

/retest ci/centos/upgrade-tests-cephfs

/retest ci/centos/mini-e2e/k8s-1.22
/retest ci/centos/mini-e2e-helm/k8s-1.24
/retest ci/centos/mini-e2e-helm/k8s-1.23
/retest ci/centos/k8s-e2e-external-storage/1.22

/retest ci/centos/mini-e2e-helm/k8s-1.24

/retest ci/centos/mini-e2e-helm/k8s-1.23

/retest ci/centos/k8s-e2e-external-storage/1.22

/retest ci/centos/mini-e2e-helm/k8s-1.23

/retest ci/centos/mini-e2e-helm/k8s-1.23

Can you point to some document that can help us to test this feature on local systems (ceph image, kernel version to use), etc?

I uploaded a minikube ISO with the current development Ceph FS fscrypt kernel that might be helpful: https://github.com/irq0/minikube/releases
You can test most changes by setting the following when calling scripts/minikube.sh
ROOK_CEPH_CLUSTER_IMAGE=quay.io/ceph/daemon:latest-main-devel
MINIKUBE_ISO_URL="https://github.com/irq0/minikube/releases/download/2022-11-08-master%2Bfscrypt-kernel/minikube-amd64.iso"

My helper scripts are in https://github.com/irq0/dev-ceph-csi-fscrypt-config
Calling kube-deploy-cephcsi from there sets up ceph csi and a deployment using an already set up rook.

Push: Rebase to devel and changes for the comments above

Just a few nits, nothing that requires correcting in my opinion. I'm a little sad I can't easily test this out. As long as it does not break anything, we should be good for now.

Thanks! Me too. It would be awesome to get at least some CI coverage of the Ceph FS fscrypt path.

Ideally we start running weekly tests with a non-standard kernel config (minkube VM?) and Ceph release that contains support for fscrypt.

A minikube VM is probably the best option. Comment https://github.com/ceph/ceph-csi/pull/3460#issuecomment-1307347841 has some details on how.
I think a custom ISO is the only way to go for the foreseeable future, as minikube kernels tend to be behind and fscrypt kernel support isn't (fully) merged yet.

/test ci/centos/upgrade-tests-cephfs

/retest ci/centos/mini-e2e-helm/k8s-1.24

@mergifyio requeue


Extend ble data transmission

Extend the ble option to send more data so it has the same coverage as wifi.

Prototype works fine, will now include in dev branch and do long term testing to check battery life

Added to dev


[Feature] setting time interval for wandb.video

Is your feature request related to a problem? Please describe.
Hello:) Is there any handling for setting time interval for wandb.video?
gif moves so fast...
Describe the solution you'd like
Currently, manually setting time interval on off-line(using powerpoint)

Hey @hw-kim-ivan
I think the documentation for the wandb.Video might be of use to you.

@hw-kim-ivan could you please specify frames per second, for example fps=1
wandb.log({"video": wandb.Video(numpy_array_or_path_to_video, fps=4, format="gif")})


fix openlineage link

fix openlineage link

This is a nice catch, @isantillan1 ! Thank you so much for spotting it and improving our docs experience.


CredScan Enforcement for New Violations - updating templates

Required items, please complete
Change(s):

masked the keys failing in credscan results

Reason for Change(s):

See guidance below

Version Updated:

Required only for Detections/Analytic Rule templates
See guidance below

Testing Completed:

See guidance below

Checked that the validations are passing and have addressed any issues that are present:

See guidance below

Guidance <- remove section before submitting

Before submitting this PR please ensure that you have read the following sections and filled out the changes, reason for change and testing complete sections:
Thank you for your contribution to the Microsoft Sentinel Github repo.

Details of the code changes in your submitted PR.  Providing descriptions for pull requests ensures there is context to changes being made and greatly enhances the code review process.  Providing associated Issues that this resolves also easily connects the reason.

Change(s):

Updated syntax for XYZ.yaml

Reason for Change(s):

New schema used for XYZ.yaml
Resolves ISSUE #1234

Version updated:

Yes
Detections/Analytic Rule templates are required to have the version updated

The code should have been tested in a Microsoft Sentinel environment that does not have any custom parsers, functions or tables, so that you validate no incorrect syntax and execution functions properly.  If your submission requires a custom parser or function, it must be submitted with the PR.

Testing Completed:

Yes/No/Need Help

Note: If updating a detection, you must update the version field.

Before the submission has been made, please look at running the KQL and Yaml Validation Checks locally.
https://github.com/Azure/Azure-Sentinel#run-kql-validation-locally

Checked that the validations are passing and have addressed any issues that are present:

Yes/No/Need Help

Note: Let us know if you have tried fixing the validation error and need help.

References:

Guidance for Detection checks
General contribution guidance
PR validation troubleshooting

The changes have been merged in PR #7555 and PR #7557


Add testify support

add support to https://github.com/stretchr/testify
implements Errorf function

package foobar

import (
    "testing"
    . "github.com/franela/goblin"
    "github.com/stretchr/testify/assert"
)

func Test(t *testing.T) {
    g := Goblin(t)

    g.Describe("lala", func() {
        g.It("lslslslsls", func() {
            assert.Equal(g, 1, 10)
        })
    })
}

You're adding code that's not tested :S

@marcosnils can you tell where you test your Fail function?, I try to look  in the code but for my time schedule  I could not find the test case, the Errorf function is almost the same, with the adition  that allow format strings

@marcosnils can you tell where do you test your Fail function?, I try to look in the code but for my time schedule I could not find the test case, the Errorf function is almost the same, with the adition that allow format strings

https://github.com/franela/goblin/blob/b50404691661f34b0cea76cb2704fe5c0ccec90f/goblin_test.go#L160-L182

@criloz Fail and Failf have almost the same code. Can you please extract the common parts function so we don't have duplicated code?

@marcosnils sure, will work on that tomorrow. also update the readme with the testify support example

Amazing!. Thx a lot!. We're almost there :D

Is this feature complete?

What happened to this?

@alexdreptu seems like it never got completed and got abandoned.

@marcosnils as I can see, it is completed! No duplicated code) May be it is ok to merge it?)

I've checked it locally — works perfectly!)

I created #85 and fixed the 2016 issue with this pr

Closed by #85


Can the GATT CLIENT example used as HID Host? (IDFGH-2484)

Hi there,
I wonder if any of you used the GATT Client example below in order to implement HID Host please?
(https://github.com/espressif/esp-idf/blob/826b9f638/examples/bluetooth/bluedroid/ble/gatt_client/tutorial/Gatt_Client_Example_Walkthrough.md)
I wonder if this is easily achievable by modifying the GATT Client example as I am relatively new in this field and I have to implement a HID host in ESP32.
Many thanks,
FakeStar

Well, maybe its not you are looking for, because i did not use esp-idf, but recently i wrote HID keyboard host example with platformio and Kolban's library:
https://github.com/chegewara/esp32-hid-keyboard-client
It is easy to write host library for 1 single device, like keyboard or mouse, when you know HID report before you start and you dont have to parse it in real time. When you want to write universal HID host library it may be hard, especially if you dont know HID specification on expert level.

Aww.. Thanks a lot @chegewara for your prompt reply and the link to your HID keyboard host example. It sounds very promising as I only need to connect to one HID barcode scanner. I will give this a go and report accordingly- :)

Then it should be not so hard, but like i said, you have to know and understand HID report from that device.

Perfect, I will do.  I
f its OK I keep this thread open to ask my questions?

I believe that forum is better place. Maybe here:
https://esp32.com/viewtopic.php?f=18&t=13598&p=53566&hilit=hid#p53566
or new topic.

Perfect sure, so I close this now! :)


[Feature Request] Adding Crud Entities in the select field type

Hi...
thank you for this awesome package .... It has any feature that I want but one  feature  I couldn't  find .   Can we  add entry in select field ? for example we have two models , Book and Category. In Book view we can select a category by select or select2 field but cant add a category .   for this situation   we must go to category view and add it .
I attached a picture (from another admin package that has this feature ) that shows what  Im   after  .

So you mean that you want to be able to add a "Category" from within your "Book" form?
@tabacitu might have a better solution, but you'd most likely have to build this as a custom feature.
I don't think its easily generic enough for a core feature as "Category" might have validation and other information that it needs to work.
The easiest thing to do would be to use the custom html field and add a button yourself that says "new category" that takes you to the new category page

yes I want to be able add a "Category" from within my "Book"  from .
yes the solution that you've suggested is good but in my project I have many these situations for example  add tags from within "Post" form or add author from within "Post" form again ,  and so on .
I think its a good feature that this package need it.

Yes you might need it a lot, but I don't think it will be a priority for Backpack, for the reasons I mentioned, it is not always as simple.
Imagine if you had a "Category" which needed a URL slug, SEO title, Status/Active and had validation on "Category Description" and the database was $table->string('description'); and needed a value.
This is all far too complicated to to consider, baring in mind Backpack is designed to as flexible and simple as possible. This is not simple for users.
So with that being said, maybe one day something similar will be available, but for now unless @tabacitu has another idea, I think this would be too bespoke to considered "this package needs it".
Feel free to create a PR with the feature so it can be added, but I would suggest developing this yourself if you need it.
All the best

@tabacitu  what is your solution ?

We don't have a solution, this is your own custom desired bespoke functionality, I've suggested you can add a button to open a new window to add it, otherwise you can visit here and find out about bespoke costing

Well this topic got a little fire :-) Relax, guys, we all want the same thing: for Backpack to cover most of our recurring needs. But it's a process, so covering everything will take some time.
I guess I agree with both of you:

having an option to add an entry right there will be a huge UX improvement; the admin won't need to go into 2 forms to create that entry; and this does happen pretty ofter, so we do need to solve this problem;

However:

building a general solution will take some time, I do not expect we'll have this done very soon; 1-3 months, I expect;
if you need it right now for your project, you can create a custom field type, as @OwenMelbz  suggested;

I've reopened the issue, because it's a feature I do think we should build. Here are my thoughts how to implement it generally:

a new field type, select_from_ajax_or_add; it would show a select2, like the screenshot above, and an "add item" button by its side;
upon clicking on the "add item" button, a modal would show up, which would show the form for adding that entry (load the HTML code, not an iFrame);
when completing the modal form, the request would be submitted through AJAX, see if there are any validation errors and either:

A: show the validation errors;
B: load the category that was just inserted in the select2;

In order to achieve this, and not have to write the create/edit options twice in your CrudController, you'd basically need an API for category - the ability to add items without an actual interface. Which I think is a great idea, because it would also give a boost to those building mobile apps or using MVC javascript libraries on the front-end.
For completeness, the API would provide the following methods and routes, inside a CrudApi trait:

category/api/create @ storeFromAjax()

would run the create() method, but instead of redirecting to a new page, it would return a JSON response with either the errors or the item that was just inserted;

category/api/update @ updateFromAjax()

same

category/api/destroy @ destroyFromAjax()

same

This would need to happen while still keeping the controller Restful. And that's tricky.
I see no clean way we can achieve the API, while allowing the developer to write their before_create or after_create callbacks only once, except if we:

make CrudController::storeCrud() and CrudController::updateCrud() no longer depend on the interface; it would work just insert/update;
create beforeStore(), afterStore(), beforeUpdate(), afterUpdate() methods, where people can do their thing;
have separate methods for the interface and AJAX that construct the response, be it a JSON response or a redirect;

So something like this:
class TestCrudController extends CrudController
{
    public function setup() {}
    public function store(StoreRequest $request)
    {
        $this->beforeStore();
        $response = parent::storeCrud();
        $this->afterStore();
        // TODO: calculate redirect_location
        return $redirect_location;
    }
    public function update(UpdateRequest $request)
    {
        $this->beforeUpdate();
        $redirect_location = parent::updateCrud();
        $this->afterUpdate();
        // TODO: calculate redirect_location
        return $redirect_location;
    }
}

trait Api
{
    public function storeFromAjax()
    {
        $this->beforeStore();
        // TODO: do the validation (we can't use Laravel Form Request validation in the API)
        $response = parent::storeCrud();
        $this->afterStore();
        // TODO: construct the JSON response
        return $response;
    }
    public function updateFromAjax()
    {
        $this->beforeUpdate();
        // TODO: do the validation (we can't use Laravel Form Request validation in the API)
        $response = parent::updateCrud();
        $this->afterUpdate();
        // TODO: construct the JSON response
        return $response;
    }
    public function destoryFromAjax($id) {}
}

@OwenMelbz , what do you think? I'm 99% we can find a better way, this definitely needs more thought.

@tabacitu a modal was the only method I could see this working for.
It would be a shame to add ANOTHER field type which does the same as the others. I think ideally adding an extra param to them would be better as you could add a closure, which maybe houses the fields you want to display. Unless you expect to display them all from the setup method?
There's actually 2 other pull requests around that are already a start of what your suggesting,
I've added one which already has a 'beforeStore' etc which can be used, and the 'withUnicity' which enables extra routes, so you could do 'withQuickAdd("admin/category")' this would setup the extra end points.
I could technically start/draft this today, but would need the other PR first, and not sure how to keep them all synced without causing dupe PRs. (And lots of inline JS) would be nice to get the core ones into js files
But other than that, the solution seemed similar to what I would have done.

@tabacitu
Thanks very much for your help...I try that .... if I find a better idea.I will share it

@OwenMelbz , completely agree - those PRs will prove to be the baby steps we need to get this going. Let's keep this feature in mind and come back to in after they're all merged and public.

I think it would be much easier something like a select2 field where you write the new record that is checked on create/update to see if it exists in its object table and if not to be added (with some default attributes in case they are required - i mean i understand that you cannot make a list with all the categories of the books before adding the books but you can add just the category's name when you add the book and at the end you may just update that category however you want).

To be honest, the more I think about this, the more its re-creating the wheel.
Realistically I feel the most sensible option is to have a button next to the select menu that says "New Entity" you click it, which opens a new tab/window which is on the create screen of the entity, then when you close the tab/window a "refresh" button is next to the "new entity" which reloads the data in the dropdown.
Otherwise you'll end up potentially re-creating complicated "add" forms in a modal.

I've added this to the backlog now for future discussion when people have appropriate time :)

saw all your comments but what if I don't want to get the select box data from the database I just want to add a select box with some static values??

@krtrth docs reveal all https://laravel-backpack.readme.io/docs/crud-fields#section-select2_from_array


Rename azeventhubs constructor methods to reflect that a Result is returned

The current constructor methods for EventHubConnection, EventHubConsumerClient, and EventHubProducerClient are in the form of from_xxx but returns a Result. This seems not strictly following the convention that from_xxx methods should not return a result, and it should be changed to be in the form of try_from_xxx if a result is returned.

Summary of method name change proposal(s):

from_connection_string() -> try_from_connection_string()
from_connection_string() -> new_from_connection_string()

This comment will be updated as more ideas show up

I don't have a strong opinion here. I like the removal of namespace from the name though as to me the key differentiating factor here is the method to authenticate.

I don't have a strong opinion here. I like the removal of namespace from the name though as to me the key differentiating factor here is the method to authenticate.

I am planning to make this change in "0.14.1" and mark the original methods as deprecated

A new version "0.14.1" has been released on crates.io


Get the Trivy reports from the Image Cleaner?

Where can I get the Trivy reports from the Image Cleaner?
The documentation does not explain how we get those Trivy reports.

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

ID: abf71794-7a2a-bc5c-1791-057dcbc33128
Version Independent ID: 389578e5-68f5-c634-875f-b9d991a251a7
Content: Use Image Cleaner on Azure Kubernetes Service (AKS) - Azure Kubernetes Service
Content Source: articles/aks/image-cleaner.md
Service: azure-kubernetes-service
GitHub Login: @Nickomang
Microsoft Alias: nickoman

According to the documentation...

If you need to trigger another manual cleanup, you have to create a new ImageList or make changes to an existing one.

I have updated the existing ImageList using the following example.
cat <<EOF | kubectl apply -f -
apiVersion: eraser.sh/v1
kind: ImageList
metadata:
  name: imagelist
spec:
  images:
    - docker.io/library/alpine:3.7.3
EOF

But there is not a single Trivy string found in the logs after the ImageList is updated.
for pod in $(kubectl get pods --output json |  jq --raw-output '.items[] | select(.metadata.name|contains("eraser")) | .metadata.name') ; do
  echo $pod
  kubectl logs $pod | grep --ignore-case trivy
done

$ kubectl describe configmap --namespace kube-system eraser-manager-config | grep --after-context=10 scanner:
  scanner:
    enabled: true
    image:
      repo: mcr.microsoft.com/oss/eraser/eraser-trivy-scanner
      tag: v1.2.3
    request:
      mem: 500Mi
      cpu: 250m
    limit:
      mem: 2Gi
      # https://kubernetes.io/docs/concepts/configuration/manage-resources-containers/#how-pods-with-resource-limits-are-run

According to Trivy's documentation...

Scan reports are saved as VulnerabilityReport resources, which are owned by a Kubernetes controller.

$ kubectl get vulnerabilityreports.aquasecurity.github.io --all-namespaces
No resources found

@sheeeng
Thanks for your feedback! I've assigned this issue to the author who will investigate and update as appropriate.

Image cleaner doesn't output trivy report as eraser doesn't have it. If you want to know the scanner results, you can check the scanner pod logs:
kubectl logs -n kube-system  -c trivy-scanner
https://learn.microsoft.com/en-us/azure/aks/image-cleaner#how-to-review-images-were-cleaned-up-by-image-cleaner


Function name changes due to the recent test updates, or inadvertent function name edits by students result in test failures

Function name changes due to the recent updates, or inadvertent function name edits by students result in test failures

When Issue Happens

This is related to issue #7938, but is a more general-purpose request.  As with issue #7938, there were other challenges and tests that had changes to the function names, and even though the code was functionally correct, the fact that the prior solution that was being used still had the old names resulted in test failures.  Discussion in the contributors forum made it seem that there was merit to adding an additional check for the existence of the proper function name prior to running the test. Or perhaps has a first-level test before running the functional tests. This would cover the case where a student went back to a prior challenge to update the solution, without realizing that there was a change to required function names, and would also give a helping hand to someone who did not understand the challenge's instructions and edited the function name, resulting in similar failures.

Steps To Reproduce

For the case where a prior solution was revisited and the function names changed, all that you have to do is revisit your solution and try to run it.

@khaduch @SaintPeter @erictleung @sludge256 do any of you have any ideas how we might be able to redesign challenges to where changing function names don't break old solutions?
One idea I have is that we could redesign the challenge framework to check whether there's only one function in the code, go ahead and run it, assuming that there's a typo in the function name, or that the function name was inadvertently changed since the code last ran.

@QuincyLarson I like the idea. We can try executing the function name we expect, and if it doesn't find anything, we can search for a function definition and execute it instead. This is probably best from a user experience perspective. On the other hand, it could be confusing that the typo function would work, because it isn't the function name we see in the challenge tests (if shown). So this could raise questions as well.
Another, but not really scalable, way to work on this issue is to encode the correct function name in each function-required challenge and check for the correct function name and let the user know their function is incorrectly typed. I don't imagine this solution scaling very well, but it is very clear to the user why their code failed.
If we could somehow let the user know which function was being used if we just use whatever function was in the editor, that might be a viable option.

@erictleung I think both of those solutions are potentially viable, but I agree that it would be better to not have to have to add the function name as a field on every single challenge. I am going to close this issue for now and if you think of a graceful way to let the user know that they've somehow mistyped the function name (or in most cases, deleted boilerplate), we can reopen this and you can make a PR against it.


How to change the prompt under authorization

For the first login, the prompt under the required authorization is used for changeability?

For the first login, the prompt under the required authorization is used for changeability?

I'm not quite sure what you're asking for here. Do you want to change the method from HTTP Basic auth to something else? Or what exactly?

My English is terrible,

This is a browser behavior issue and out of scope for this project


SDK requires opentelemetry-common 0.19.7

In https://github.com/open-telemetry/opentelemetry-ruby/pull/1412, the SDK picked up a dependency on the untraced? method in opentelemetry-common, which was added in 0.19.7 (same PR) (https://github.com/open-telemetry/opentelemetry-ruby/blob/main/common/CHANGELOG.md#v0197--2023-05-30).

@robertlaurin does it read the commits or does it read the PR titles?

@robertlaurin does it read the commits or does it read the PR titles?

I thought it was the commits


Improved design for Search 'filters'

The current search 'filter' feel undesigned.

[x] New visual design
[ ] Front end implementation
[ ] Add this new pattern to Wagtail's UI pattern library

Here is an improved quick win version of this filtering component for better usability.

Hi
The design above is now agreed by the Wagtail UX/UI working group and ready for FE development for anyone who is keen to take this on.
Here's the Sketch file.
wagtail-search-filters.sketch.zip
Ben

Hi @benenright
Whilst further implementing this during the Minsk sprint some questions came to mind which is probably essential for the bigger picture. Especially for our mobile strategy. Right now the designs reflect on the desktop state which has 2 types of filtering on the search page.
Example screenshot from the Bakery Demo:

Will the page-type be removed or should it be an iconless adaption of the base filtering? (pages, images, documents, users)
How should we handle a long list of these filters in smaller viewports? Should we render multiple rows or should we have a sort of inline navigation (resize to 1280px)?
How should these filters behave on mobile?

Right now there isn't much room te play with on mobile and that's even without possible increased text labelling in other languages.

As of Wagtail 4 this now has an improved design:

I don't know how much of this was an intentional redesign of this particular component, as opposed to emerging organically from the site-wide style updates - but either way, I think this ticket has now outlived its usefulness, and any decision we make about updating this further needs to be re-evaluated in the context of the Wagtail 4 redesign.


Indexing fixes

This PR adds a couple things that should make indexing better:

use "clean" urls (eg; https://lethal.wiki/beginners-guide instead of https://lethal.wiki/beginners-guide.html)
use the automatic sitemap generation from vitepress
fix a typo that was showing up on google

right now both clean and non-clean ones show up on google. this PR should fix this hopefully

I'm 90% sure it'll cause no other issues, but I may revert it if GitHub Pages handles it weirdly in any way


Linking issue after CocoaPods 1.0.0 update

So here's what I've done:

Updated CocoaPods to stable 1.0.0 release;
Removed workspace and Pods folder;
Executed pod install;
Cleaned product and build folder.

After that project won't compile — RMUniversalAlert.m throws error:

'UIAlertView+Blocks/UIAlertView+Blocks.h' file not found

If i replace imports of UIAlertView, UIActionSheet and UIAlertController like this — everything compiles just fine:
@import UIAlertView_Blocks.UIAlertView_Blocks;
@import UIActionSheet_Blocks.UIActionSheet_Blocks;
@import UIAlertController_Blocks.UIAlertController_Blocks;

But that's dirty.
Is there a clean way to fix this? Is it CocoaPods related or what?

Thanks. Been using your libs for a long time.

+1

+1, please fix this please....

I updated import sections in Pods/RMUniversalAlert/RMUniversalAlert.m and build process succeeded.
#import "UIAlertView+Blocks.h"
#import "UIActionSheet+Blocks.h"
#import "UIAlertController+Blocks.h"

+1

+1

+1

fixed in 0.8


Error in ipld-dag-cbor deps

Hello everyone, we got this error today

Is this possible to pin all versions to make the library stable?

The package ipld-dag-cbor has been deprecated and they have not pinned other dependencies either. We are looking at this problem at the moment.

This should have been resolved by v0.18.3

There is still a case where this lib is being used:
https://github.com/Zondax/filecoin-signing-tools/pull/429/files#diff-793a0a178ed154cad5d52b81e0d4a3f0ec352aeba72c091a65d6cd2b5bc68fb5L32


Update usage.md code example

Adding missing N import (destructured) used for multiply function

thanks Kuba! 🚀


<EMAIL_ADDRESS>

[Enter feedback here]

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

ID: 4459a0c8-0451-bbdf-3d2f-f62f46dc2240
Version Independent ID: 7ebbbee6-1bd2-de5a-aabf-978d12099f3f
Content: Windows 10 roaming settings reference
Content Source: articles/active-directory/active-directory-windows-enterprise-state-roaming-windows-settings-reference.md
Service: active-directory
GitHub Login: @MarkusVi
Microsoft Alias: markvi

@Aunworapat Can you please let us know questions you have ?

@Aunworapat We will now proceed to close this thread. If there are further questions regarding this matter, please reopen it and we will gladly continue the discussion.


Bump console version

$subject to 2.5.2

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/6743454837

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/6743454837
Status: failure

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/6745065490

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/6745153140

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/6745153140
Status: failure

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/6745249348

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/6745249348
Status: success

Bumped the version from https://github.com/wso2/product-is/pull/17568


Error when running execution from Maven

When running the project through NetBeans12 (uses Maven 3.6.3) for execution I'd get the error.

Error: --module-path requires module path specification

Fixed the issue by making the changes in pom.xml changed:
<modulepath/>
to
<argument>${project.build.directory}/distribution</argument>
in the default jar executable class section.
Didn't seem to affect the run.sh script execution. Not sure what else it might affect.

thanks @emmanuel-lopez for the feedback and fix! applied and tested OK.


Travis config for creating and publishing multi arch images

Signed-off-by: Aishwarya Kamat<EMAIL_ADDRESS>

@aishwaryabk We can also push openshift-install-powervs:ocp$VERSION (without the tag version) which will be always the latest image version.
Courtesy @bpradipt

@aishwaryabk We can also push openshift-install-powervs:ocp$VERSION (without the tag version) which will be always the latest image version.
Courtesy @bpradipt

Thanks! Updated the changes.


Add visualizations to dashboard

Merging this now to have consistent version for KG build - will open new PR for further dashboard viz


React wrapper for lightweight charts

Hi,
I would like to inform you I made a react wrapper for your lib.
It's still in development, and all the methods are not available yet. However, I think it can be useful to many people to implement your chart lib quickly in their react app.
Here is the GitHub link: https://github.com/Kaktana/kaktana-react-lightweight-charts
It's not published on npm yet as I would like to ask you the permission to publish it :)
Also, if you have requests for the wrapper, please let me know so I can achieve this.
Thanks

Also, I added some options as autoWidth and autoHeight allow the chart to resize automatically with 100% of the width/height of his container if enabled. Also, I added a default color palette for multi-line charts so each line is clear without setting a different color manually.
Maybe these options could be added to the original library?

Thanks for creating the wrapper! I'm using your wrapper in a student project and could not get the chart to render in the desired area on page without it. Are only some options availble when using it? I can't seem to get line color to come through.

Thank you for the support :)
Don't hesitate to open issues on the wrapper repo if related to it!
I added some useful options to the wrapper as a legend handler, an auto-color assignment to multi-series charts, etc.
What do you mean by cannot seem to get line color to come through?
I will add a complete doc to the wrapper as well as examples in a few days.

Ok! I'll add to your repo.

@AurelReb would you be able to help how to do the autosize using the orginal library in react? I would really appreciate your help
Thanks

@IbrahimSam96 what do you mean on the original react library?

I would like to create a responsive lightweight-charts without the wrapper. Here is a snippet of my code

const  LivePortfolioGraph = (props) => {

const ayre = useRef(null);

const { width, height } = useResizeObserver({ ref: ayre.current });

useEffect( ()=> {

const chart = createChart(ayre.current, {
        width: width,
        height: height,
 .........
});

var areaSeries = chart.addAreaSeries({
topColor: 'rgba(38, 198, 218, 0.56)',
bottomColor: 'rgba(38, 198, 218, 0.04)',
lineColor: 'rgba(38, 198, 218, 1)',
lineWidth: 2,
crossHairMarkerVisible: false,
});

areaSeries.setData([
                    { time: '2018-10-19', value: 26.19 },
                    { time: '2018-10-22', value: 25.87 },
                    { time: '2018-10-23', value: 25.83 },
                    { time: '2018-10-24', value: 25.78 },
                    { time: '2019-04-22', value: 26.00 },
                    { time: '2019-04-23', value: 26.06 },
                    { time: '2019-04-24', value: 26.00 },
                    { time: '2019-04-25', value: 25.81 },
                    { time: '2019-04-26', value: 25.88 },
                    { time: '2019-04-29', value: 25.91 },
                    { time: '2019-04-30', value: 25.90 },
                    { time: '2019-05-01', value: 26.02 },
                    { time: '2019-05-02', value: 25.97 },
                    { time: '2019-05-03', value: 26.02 },
                    { time: '2019-05-06', value: 26.03 },
                    { time: '2019-05-07', value: 26.04 },
                    { time: '2019-05-08', value: 26.05 },
]);

var legend = document.createElement('div');
legend.classList.add('legend');
ayre.current.appendChild(legend);

var firstRow = document.createElement('div');
firstRow.innerHTML = 'GAV';
firstRow.style.color = 'white';
legend.appendChild(firstRow);

}, [ayre] )

return (

)

}

export default LivePortfolioGraph;


iox-#1196 Update LLVM installation action

Signed-off-by: Dietrich Krönke<EMAIL_ADDRESS>Pre-Review Checklist for the PR Author

[ ] Code follows the coding style of CONTRIBUTING.md
[ ] Tests follow the best practice for testing
[ ] Changelog updated in the unreleased section including API breaking changes
[ ] Branch follows the naming format (iox-123-this-is-a-branch)
[ ] Commits messages are according to this guideline

[ ] Commit messages have the issue ID (iox-#123 commit text)
[ ] Commit messages are signed (git commit -s)
[ ] Commit author matches Eclipse Contributor Agreement (and ECA is signed)

[ ] Update the PR title

Follow the same conventions as for commit messages
Link to the relevant issue

[ ] Relevant issues are linked
[ ] Add sensible notes for the reviewer
[ ] All checks have passed (except task-list-completed)
[ ] All touched (C/C++) source code files are added to ./clang-tidy-diff-scans.txt
[ ] Assign PR to reviewer

Notes for Reviewer

Checklist for the PR Reviewer

[ ] Commits are properly organized and messages are according to the guideline
[ ] Code according to our coding style and naming conventions
[ ] Unit tests have been written for new behavior

[ ] Each unit test case has a unique UUID

[ ] Public API changes are documented via doxygen
[ ] Copyright owner are updated in the changed files
[ ] All touched (C/C++) source code files have been added to ./clang-tidy-diff-scans.txt
[ ] PR title describes the changes

Post-review Checklist for the PR Author

[ ] All open points are addressed and tracked via issues

References

Closes TBD

This change is

Codecov Report

Merging #29 (48df699) into master (7713e14) will decrease coverage by 0.02%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master      #29      +/-   ##
==========================================
- Coverage   78.89%   78.87%   -0.03%
==========================================
  Files         379      379
  Lines       14487    14487
  Branches     2028     2028
==========================================
- Hits        11430    11426       -4
- Misses       2431     2433       +2
- Partials      626      628       +2

Flag
Coverage Δ

unittests
78.53% <ø> (-0.03%)
:arrow_down:

unittests_timing
14.90% <ø> (+0.01%)
:arrow_up:

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

Impacted Files
Coverage Δ

iceoryx_hoofs/source/concurrent/loffli.cpp
80.00% <0.00%> (-11.43%)
:arrow_down:


Stuck Transactions

Describe the bug
A lot of stuck transactions going back several hours. Different chains and different routers. What's weird is that many are from evmos to other chains and are still pending. If it was a subgraph issue, they shouldn't show up in the explorer in the first place
To Reproduce
Steps to reproduce the behavior:

try to bridge
??
you got stuck successfully

Additional context

Example transactions:
https://connextscan.io/tx/0x3504eb04d68511c99001ec113b4a82e181c563c655f37845fbe30c647ac771d8
https://connextscan.io/tx/0x28110fa34a63c9862e93c1f32dc9b1320afbe35aaa328da0edf5842608e6d662
https://connextscan.io/tx/0xfb85baa6c2b8aac585e204ad33a1a2871ef7d488d7f00dc9a57554a3feab340e
https://connextscan.io/tx/0xf12bbdb93318ab343efa4ef09fe2161b749cf2cc31b6679ab17d6baec8ea5bfb

Update: Polygon subgraph might be dead.

Polygon prepare TX: https://polygonscan.com/tx/0x4f1123826bb29afa3fb7e31b8ca765200c5eea78dc48a362ae3ccdf3194c42f5
Connext TX: https://connextscan.io/tx/0xf0ec94d34cf2cd2bb48c218b4a73360a21be289209b19310b84725940f692b52

Update 2: Optimism subgraph might be dead too.

Optimism fulfill TX: https://optimistic.etherscan.io/tx/0x4645967878aa8b4b12cb1bfd317c8751eb5ea551e1e794ab29cf728ed08efdd2
Connext TX: https://connextscan.io/tx/0x938ba00df96fc8dec8208ddec069acd7a3ed3e038f6f66589a7e5c6ae0007d52

Update 3: BSC subgraph also??

https://connextscan.io/tx/0x7a776242c0825ed4a4c8249ebfaf21084f89c029fcca0c4f14a59df36ea8bf30

Actually, looks like a front end issue on the explorer and the bridge
Error: No subgraphs available on this chain; unable to handle request.
    at c.request (https://connextscan.io/_next/static/chunks/pages/_app-8a52d249168d837e4ed8.js:215:87674)
    at async https://connextscan.io/_next/static/chunks/pages/_app-8a52d249168d837e4ed8.js:215:6174
    at async Promise.all (index 6)
    at async t.Subgraph.getActiveTransactions (https://connextscan.io/_next/static/chunks/pages/_app-8a52d249168d837e4ed8.js:215:6015)
    at async https://connextscan.io/_next/static/chunks/pages/_app-8a52d249168d837e4ed8.js:215:3274

fixed


Consider not emitting any scanner-specific LicenseRefs

[...]
Or we should actually go exactly the other way around and not emit any scanner-specific LicenseRefs here, but only "ort" LicenseRefs. Even if that means that we might need to map some "ort" and "scancode" LicenseRefs to the same LicenseRef so they are basically treated as aliases in the rules. But doing something scanner-specific here generally feels wrong to me.
Originally posted by @sschuberth in https://github.com/oss-review-toolkit/ort/pull/2598/review_comment/create

Actually https://github.com/oss-review-toolkit/ort/issues/1305 could be an idea how to solve this.

Closed as part of backlog grooming. Feel free to comment if you would like to contribute to this.


Name of the functions tested

I used your project and really liked the tool. I would like to continue this project. We could be a big competitor to pytest-report-html, haha. But here's what I did. See what you think:

Capturing function names
Capturing the function's test status
Developing an MD footer

See the example of how it turned out:
Test Report - platform-bs-users

:memo: Test results
Total Tests: 16
Passed: 16
Failed: 0
Passed Tests ✅

test/resource/test_users.py::TestUsers::test_get_list_all_users
test/resource/test_users.py::TestUsers::test_post_add_user
test/resource/test_users.py::TestUsers::test_edit_user
test/resource/test_users.py::TestUsers::test_delete_user
test/service/test_pgadmin.py::TestPgAdmin::test_close_connection
test/service/test_pgadmin.py::TestPgAdmin::test_commit
test/service/test_response.py::test_default_response_message
test/service/test_response.py::test_response_success
test/service/test_response.py::test_response_client_error
test/service/test_response.py::test_response_traceback_printed
test/service/test_response.py::test_logging_exception
test/service/test_response.py::test_response_exception
test/utils/test_magicrypt.py::test_compress_response
test/utils/test_magicrypt.py::test_decompress_payload
test/utils/test_pagination.py::test_pagination
test/utils/test_pagination.py::test_metadata

Failed Tests ❌

:bar_chart: Test Summary

Passed Tests: 16 ✅
Failed Tests: 0 ❌

Report styled with ❤ for platform-bs-users on 13-Dec-2024 at 22:20:10

It's not an error or a bug, it would just be improvements. I've never participated in a project like that, so I reported what I did internally to improve the test report.


Could not install package 'Square.Connect 2.14.0' with Framework 4.6

Is there not a .NETFramework version of this library?   I'm not running .NETStandard, but instead running Framework 4.6 and get the error during Nuget Install.   I had my fingers crossed as I have never written anything with .NETStandard and know little about it, but it was a no go...
Could not install package 'Square.Connect 2.14.0'. You are trying to install this package into a project that targets '.NETFramework,Version=v4.6', but the package does not contain any assembly references or content files that are compatible with that framework. For more information, contact the package author.

I just found that 4.71 has build in support for .NET Standard
https://github.com/dotnet/standard/issues/514

Also found that Square.Connect <IP_ADDRESS>7 supports Framework 4.5

Yeah, for the latest version we use .NET Standard 2.0 which requires .NET Framework 4.6.1 or higher. See chart here from Microsoft: https://docs.microsoft.com/en-us/dotnet/standard/net-standard


Adding reference

I added our work as a reference for the use of fadtk.

I added our work as a reference for the use of fadtk.

Great!


First pass of adding SAL annotations

Add static analysis annotations to the platform functions.
Signed-off-by: Alan Jowett<EMAIL_ADDRESS>

@Alan-Jowett please rebase and resolve conflicts


Slider value isRequired, preventing defaultValue

Hiya. In Slider, value is required. This is in conflict with the demoes in https://tleunen.github.io/react-mdl/#/sliders , which use defaultValue instead of value: https://tleunen.github.io/react-mdl/#/sliders .
Having both a value and defaultValue gives:
Warning: Input elements must be either controlled or uncontrolled (specify either the value prop, or the defaultValue prop, but not both). Decide between using a controlled or uncontrolled input element and remove one of these props. More info: https://fb.me/react-controlled-components

The documentation always mentioned them as required, but it wasn't the case in the code. So in the latest version, I set them as required.
The slider is therefore not compatible with the "uncontrolled" way. I should change the demos to reflect that.

What kind of work would have to go in to getting uncontrolled supported?

Not much probably...
Just change this line to execute it only when a value is provided.
https://github.com/tleunen/react-mdl/blob/master/src/Slider.js#L16
A PR is welcome to change that.


⚠ ClamAV has degraded performance

In 16b669e, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 208 ms

Resolved: ClamAV performance has improved in 8853f24.


feat(core): initialize state with location reducer

resolves #1011, please squash with merge my local branch was quite out of sync  🙏

Current coverage is 52.45% (diff: 25.00%)

Merging #1012 into master will decrease coverage by 3.09%

@@             master      #1012   diff @@
==========================================
  Files            11         11
  Lines            54         61     +7
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits             30         32     +2
- Misses           24         29     +5
  Partials          0          0

Powered by Codecov. Last update 5758b2c...316fba4


Refs #28643 -- Implemented the NullIf SQL function.

@atombrella Are you going to be able to continue carrying this forward? It doesn't feel like there is much left to do....

@atombrella Do you have time to finish this patch?

I've rebased the patch, and updated version numbers, but the example still needs to be revised. I can finish it, as it's only documentation that needs revision.

buildbot, test on oracle.

buildbot, test on oracle.


Separate scripts for data preparation, training and evaluation.

Allow reading already written prepared data, transformers and model.

Separated into prepare_training_data.py, train_model.py, evaluate.py, plots.py.


javacard-devkit: init at 2.2.2

Oracle has already released 3.0.5, but versions after 2.2.2 appear to be Windows-only.
This derivation will override javac in order to give it the paths required for JavaCard programming.
Things done

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

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

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

Note to self (got to go right now and was reminded to test all binary files by the checklist): @Ekleog, figure out why the cref wrapper is unhappy and make libjpcsclite.so non-executable.
Sorry for opening this PR before having checked everything!

No attempt on aarch64-linux (full log)
The following builds were skipped because they don't evaluate on aarch64-linux: javacard-devkit
Partial log (click to expand)

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

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

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

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

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

No attempt on aarch64-linux (full log)
The following builds were skipped because they don't evaluate on aarch64-linux: javacard-devkit
Partial log (click to expand)

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

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

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

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

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

Thanks for the review and hints! I've done what I could, things appear to more or less work (at least for the ones for which I know how they are supposed to be used).
Though, still remain a few concerns voiced in my answers to your remarks, unfortunately hidden by github as no longer relevant due to underlying code change.

No attempt on aarch64-linux (full log)
The following builds were skipped because they don't evaluate on aarch64-linux: javacard-devkit
Partial log (click to expand)

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

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

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

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

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

Thank you for your help, @aszlig!
So I think the last remaining question is how to handle the differences of javac between javacard-devkit and oraclejdk* / jdk*.
Basically, the differences between the two are that javacard-devkit needs for successful compilation:

Additions to the CLASSPATH that point into the store (in order to find the required dependencies)
Additional arguments, for instance -source 1.5 -target 1.5 to specify the Java version (it can go at least from 1.2 to 1.5, converter doesn't support anything more recent but I haven't tried anything older)

What I'm thinking is, reasonably we can't leave it up to the user to do 1, because that points into the store and would be a PITA. So this must be done by the javacard-devkit derivation.
For 2… it's more complex: maybe someone wants to compile an older Java Card applet that was designed for use with 1.2? Yet at the same time it's most likely to be unnecessary boilerplate and a cause for head-scratching from someone who wouldn't know these arguments are required. So the question looks like it's a bit “How far do we want to go from upstream for our users' sake?”. I'd lean towards leaving these arguments outside of javacard-devkit, mostly for the ancient-addon use case.
Now, if the idea is to keep only 1 included in javacard-devkit, remains the question of how to do this.  I see again two options:

Have a wrapped javac in the javacard-devkit derivation
Have a javacardc in the javacard-devkit derivation, that wraps oraclejdk's javac.

I can see advantages and disadvantages for both options:

Pros: Closer to upstream recommended building system. Consistent to most documentation that could be found online.
Cons: Might cause conflicts if installed globally at the same time as oraclejdk or jdk. (but is this actually an issue? anyway adding something to the CLASSPATH should hopefully not break anything, so it'd be a matter of making sure that javacard-devkit's javac overrides oraclejdk's or jdk's)
Pros: No conflict. Appears to be the name of the Java Card 3 tool (not sure whether this is actually a bonus or a malus, given JC3 has no linux version…)
Cons: Farther from the (few) docs that can be found online. Requires specific documentation and especially requires that the user actually reads said documentation.

A coin throw told me option 2 was better, but that's only a coin throw's opinion.
What do you think about this? :)

Given there appeared to be no contradictory opinion, I've now implemented the coin throw's result, and think this is ready to be merged :)

hey all! i'd love to see this merged.

No attempt on aarch64-linux (full log)
The following builds were skipped because they don't evaluate on aarch64-linux: javacard-devkit
Partial log (click to expand)

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

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

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

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

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


"if-exists" added only to "package" bindings but not to "class"bindings in episode file

<addIfExistsToEpisodeSchemaBindings> settings adds if-exists attribute not to all <bindings> elements.
For example, after generating episide file:
<bindings xmlns:tns="{my-tns}" if-exists="true" scd="x-schema::tns">
    <schemaBindings map="false">
      <package name="{my-package-name}"/>
    </schemaBindings>
    <bindings scd="~tns:{my-element}">
      <class ref="{my-element-class}"/>
    </bindings>
    <bindings scd="~tns:{my-element-2}">
      <class ref="{my-element-class}"/>
    </bindings>
</bindings>

But when I use this episode file in generating other classes, I get "didnt match any schema component" error if not all {my-element*} elements were used in XSD's that are being processing.
I didn't found any other solutions than to add if-exists="true" to bindings with <class> element inside, so it would be like:
<bindings xmlns:tns="{my-tns}" if-exists="true" scd="x-schema::tns">
    <schemaBindings map="false">
      <package name="{my-package-name}"/>
    </schemaBindings>
    <bindings if-exists="true" scd="~tns:{my-element}">
      <class ref="{my-element-class}"/>
    </bindings>
    <bindings if-exists="true" scd="~tns:{my-element-2}">
      <class ref="{my-element-class}"/>
    </bindings>
</bindings>

If I wrong about correctness of this solution, please advise me a better way or some reasons that that kind of way is wrong.
If I'm not, maybe addIfExistsToEpisodeSchemaBindings.xslt should be fixed?
I can send a pull request if it will need.

I think you're right. Please send the PR, would you? Thanks!