Issue with Code Complexity Discrepancy in Cobertura Report

Description:
Hello everyone,
I've encountered an issue related to code coverage in our project. Specifically, I've noticed a discrepancy between the cyclomatic complexity reported by our pipeline task and the complexity displayed in Visual Studio.
Problem Description:
When running one of our unit test suites, I noticed that a complex method in one of the tested classes is being shown as having a cyclomatic complexity of 1 in the coverage report generated by the pipeline. However, when I perform the same analysis in Visual Studio, the complexity of the method is displayed as 50.
Investigation Efforts:
I have been trying to search for a possible open issue online, but I have not been successful. I have attached the pipeline task responsible for running the unit test suite. I can also privately share the relevant class and Cobertura.xml if it would be of any use. Initially, I thought that the issue may be in the pipeline task itself. However, I attempted to collect the coverage locally through vstest.console.exe and encountered the same result. I've also tried to collect complexity for other test suites using the dotnet test command, and for the methods in those tested classes, the complexity was either the same or very similar to the one reported by Visual Studio.
vstest.console.exe .\NotificationHub\TransitionalTools.Tests\bin\Debug\TransitionalTools.Tests.dll /collect:"Code Coverage;Format=Cobertura" /settings:.\NotificationHub\TransitionalTools.Tests\coverage.runsettings

- task: VSTest@2
  displayName: 'Test transitional tools'
  inputs:
    testSelector: 'testAssemblies'
    testAssemblyVer2: 'TransitionalTools.Tests.dll'
    searchFolder: 'NotificationHub\TransitionalTools.Tests\bin\Release\'
    runInParallel: false
    otherConsoleOptions: '/collect:"Code Coverage;Format=Cobertura"'
    runsettingsFile: 'NotificationHub\TransitionalTools.Tests\coverage.runsettings'
    codeCoverageEnabled: true
    resultsFolder: '$(Agent.TempDirectory)\TestResults'
    platform: '$(BuildPlatform)'
    configuration: '$(BuildConfiguration)'
    publishRunAttachments: false
    failOnMinTestsNotRun: true
    minimumExpectedTests: '350'

Issue Details:
Looking at the Cobertura report, the branches (switch statements, if-statements, loops,...) in the covered method are not acknowledged.
Thank you all for your attention and assistance in advance.
Kind regards,
Peter

@peterm28 Can you please upload a minimal sample solution for this?

You are using Microsoft.Fakes in your scenario. In Microsoft.Fakes scenario, we cannot generate cobertura line conditions, and hence complexity is always zero. After running tests with using Microsoft.Fakes, cobertura line conditions are populated correctly as well as complexity.

@fhnaseer issue here is that we use Fakes V1 and Intellitrace? If yes can we upgrade to Fakes V2?


[PornHub] 240p format not being detected

Checklist

[x] I'm reporting a broken site support
[x] I've verified that I'm running youtube-dl version 2020.12.12
[x] I've checked that all provided URLs are alive and playable in a browser
[x] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[x] I've searched the bugtracker for similar issues including closed ones

Verbose log

[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['http://www.pornhub.com/view_video.php?viewkey=648719015', '--list-formats', '--verbose']
[debug] Encodings: locale cp1252, fs mbcs, out cp1252, pref cp1252
[debug] youtube-dl version 2020.12.12
[debug] Python version 3.4.4 (CPython) - Windows-10-10.0.19042
[debug] exe versions: none
[debug] Proxy map: {}
[PornHub] 648719015: Downloading pc webpage
[PornHub] 648719015: Downloading m3u8 information
[PornHub] 648719015: Downloading m3u8 information
[PornHub] 648719015: Downloading m3u8 information
[PornHub] 648719015: Downloading m3u8 information
WARNING: unable to extract like count; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
WARNING: unable to extract dislike count; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
[info] Available formats for 648719015:
format code  extension  resolution note
hls-355      mp4        426x240     355k , avc1.640015, 29.97fps, mp4a.40.2
hls-377      mp4        426x240     377k , avc1.640015, 29.97fps, mp4a.40.2
hls-799-0    mp4        854x480     799k , avc1.64001f, 29.97fps, mp4a.40.2
hls-799-1    mp4        854x480     799k , avc1.64001f, 29.97fps, mp4a.40.2
hls-1500-0   mp4        1280x720   1500k , avc1.64001f, 29.97fps, mp4a.40.2
hls-1500-1   mp4        1280x720   1500k , avc1.64001f, 29.97fps, mp4a.40.2
480p         mp4        480p       2000k
720p         mp4        720p       4000k  (best)

Description

Something changed on PornHub the past 2 days and the 240p format isn't being detected for all videos I've tried. Also like/dislike count isn't being extracted consistently. Works for some videos and doesn't work for others (like the one used above, or this random one from the home page: https://www.pornhub.com/view_video.php?viewkey=ph5ea1a27312cf8).

This PR (#27356) addresses the like/dislike aspect of this issue

Thanks for the fixes. My bad for missing PR #27356, must've confused it with an earlier PR.


Add Auth0. Resolves #26

Fixing tests 😬

Also, it will be nice to cover all with the test this implementation.


Two SPFX Packages - Consume the same custom Azure AD secured APP / EndPoint

Category

[x] Question
[ ] Typo
[x] Bug
[ ] Additional article idea

Expected or Desired Behavior
We have two SPFX packages which consume a generic AAD secured Web API service. The AAD app has been setup and the first SPFX package has been deployed, approved via API management... all working fine.
We want to use the same service within the second SPFX package...
Observed Behavior / #### Steps to Reproduce
The package.json for both SPFX packages contains the same webApiPermissionRequests entry, example below:
"webApiPermissionRequests": [
{
"resource": "Windows Azure Active Directory",
"scope": "User.Read" },
{
"resource": "name-of-aad-app",
"scope": "user_impersonation"
}
],
So... after deploying via the AppCatalogue the API management requests are added, however we cannot approve them as it states that entries already exists, example error below:
[HTTP]:400 -  [CorrelationId]:xxxxxxxxx [Version]:<IP_ADDRESS>28 - An OAuth permission with the resource name-of-aad-app and scope user_impersonation already exists. Parameter name: permissionRequest
[HTTP]:400 -  [CorrelationId]:xxxxxxxxx [Version]:<IP_ADDRESS>28 - An OAuth permission with the resource Windows Azure Active Directory and scope User.Read already exists.
Parameter name: permissionRequest
I'm guessing we're doing something a little silly and there is a quick fix (fingers crossed!)?
#spfx #spfx-tooling #api-management

As long as you have approved the permission already for the same resource and permissions, things should work still fine even though the pending request is not approved, as these permissions are NOT by default provided for a solution... all solutions are sharing the same permission set (unless you are using preview of isolated web parts).
This means that the issue is mainly cosmetic on having additional permission request pending in the Admin UI, but it has no impact on the functionality as such. You could also just drop the permission request and then grant it directly using Approve-SPOTenantServicePrincipalPermissionGrant cmdlet (https://docs.microsoft.com/en-us/powershell/module/sharepoint-online/approve-spotenantserviceprincipalpermissiongrant?view=sharepoint-ps).
UI exception is clearly an issue which we need to fix, but it's not technically blocking you to use the API in either of your solutions.

Thanks Vesa, we'd wondered as the API management requests are package name and package version specific which implies that the approval is only specific to a particular app package... we're not using isloated web parts.
We'll tick this one of the list as a possible source of the problem we have.

They are not package specific, unless sppkg file has the isolated setting enabled. This is isDomainIsolated setting in the package-solution.json file. If that's set as 'true', then permissions are solution specific, if it's not set, then setting is false and permissions are shared.
Please do let us know if you can solve the difference or confusion here so that we can potentially close the issue or alternatively investigate more if needed.

I understand ref the isolated setting, I was more commenting on what's implied by how the API management screen works within SharePoint Admin.
Happy for this to be closed, but I think there's probably some changes required to the management pages as this is a little misleading and a fairly common use case.
We've fixed our issue (AAD permissions) and totally un-related to this, but really useful to clarify. thanks


Develop an SAP Fiori App Using SAP Business Application Studio

Tutorial URL: https://developers.sap.com/tutorials/appstudio-fioriapps-create.html
Please specify the step you are referring to
Hi, teams
I think this document is out of date. The Business Application Studio has been updated many times until now.
So could you please update the tutorial document base on the latest version? Thank you!

Hi,
An update is available and will be released in a few days.
Regards,
Raz

Thank you for your feedback. The issue seems to be resolved, so I am closing the issue.
If you still have questions, feel free to reopen the issue.


Cannot view User Profile of Primary Userstore users

When there're local claims only specific to the secondary user store and those claims have been configured only for the secondary user store. Primary user store user profiles cannot be seen.
Steps to Reproduce:

Configure a secondary user store (JDBC).
Add a custom claim for the secondary user store as below and make it supported by default.

List users and view the user profile of a primary user store user, below error will appear.

According to the current implementation, we can overcome this by disabling supported by default and mapping newly added claim to a local claim. End users expect even though claim mapping has not been done for Primary user store and supported by default enabled, still it should return user profile (for Primary user store) without error in profile UI.

IMO, there should be validation on claim creation/update flow to validate whether a mapped attribute is defined for PRIMARY.

This issue is being closed due to extended inactivity. Please feel free to reopen it if further attention is needed. Thank you for helping us keep the issue list relevant and focused!


Add links to other projects

I have included a few that I consider important (and that are maintained on an ongoing basis), but there are probably others that should be mentioned.
For example, those referenced from the following paper.
https://github.com/nuest/rockerverse-paper
https://journal.r-project.org/archive/2020/RJ-2020-007/index.html

Nice, this is great.
Speaking of, do we have any page that links to Daniel's paper or our rocker R-journal piece? though such things become stale more quickly, they still may be useful context to some users.

Speaking of, do we have any page that links to Daniel's paper or our rocker R-journal piece? though such things become stale more quickly, they still may be useful context to some users.

There is no link to them at this time.
I too thought it would be nice to have links to them. Perhaps a thesis section on the top page?


Add breaking change notes to docs for commit 531a8de72c439d8ddd064874bf364c00cedabb11

Commit 531a8de72c439d8ddd064874bf364c00cedabb11 introduced a breaking change where the $observe function no longer gets called even for attributes that aren't defined. There is a note about this on the github page, but there is nothing about it in either the changelog or the migration guide.
I have been working on migrating from 1.2 to 1.3 and it took me a while to track down an issue because of this new behavior. I think it's worth adding a breaking change note to the change log and adding this to the migration guide (https://docs.angularjs.org/guide/migration), so people can be made aware of this breaking change.

Anything happening with this?

I've added a BC note. Due to the nature of the docs, this will however only appear in the next 1.4 and 1.5 versions, not in the 1.3 version.


x86 debian binary

It would be great to have access to this tool in some Docker containers I have that must be in x86 architecture.

Thank you for the issue.
I'll think about what I can do.
If you have any specific ideas, please let me know.

Building a debian package for that architecture would be all we need!

Oh, the problem is that 386.deb doesn't exist?
Sorry, I'll add it.

Released v0.33.2 which includes https://github.com/noborus/ov/releases/download/v0.33.2/ov_0.33.2-1_386.deb.


Submitting AdsRequest Object

Hi,
I understand that the latest versions of videojs-ima don't require a call to ima.requestAds and that doing so will cause problems. However, I cannot find anything in the API docs or in your sample code that shows how to submit the AdsRequest object. There are 2 methods associated with AdsRequest -  setAdWillAutoPlay and setAdWillPlayMuted along with a number of option fields which includes the ad sizes, keywords, duration and several others. Since these were previously passed as the first parameter to requestAds which you are calling internally now, how do we include the AdsRequest object?
Thanks,
D

Can you clarify what you mean by "these were previously passed as the first parameter to requestAds"? The requestAds method, as far as I can tell, has never taken any parameters - it was just a call to tell the plugin to send the ads request it had created to the ad server.
I do see that there are properties of the AdsRequest object that are missing from IMA settings, but I don't think we ever had a way to provide these to the plugin. I can take this as an FR to allow you to pass a JSON object in the settings that contains your AdsRequest properties.

Thanks, that's perfect!


Add pit for join queries

Description

Added option to use search after with pit instead of scroll api for join queries

Issues Resolved
https://github.com/opensearch-project/sql/issues/2603
Check List

[ ] New functionality includes testing.

[ ] All tests pass, including unit test, integration test and doctest

[ ] New functionality has been documented.

[ ] New functionality has javadoc added
[ ] New functionality has user manual doc added

[ ] Commits are signed per the DCO using --signoff

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

Will you add unit test to cover the new logic?

Existing logic doesn't have unit tests so I didn't add to cover new logic also.

Will you add unit test to cover the new logic?

Existing logic doesn't have unit tests so I didn't add to cover new logic also.

I'm not sure the consensus in this repository, but I'd generally recommend adding unit test at least covering new logic.

1 major comment about where we create PIT. Please check if how we did that for Scroll.

We don't have PIT support for Scroll in sql. Looks like PIT was added from v2.4 (ref: https://opensearch.org/blog/point-in-time/) so when join queries were implemented, PIT didn't exist.


The wallet did not contain the tokens required for staking

when i try to stake my sig Sigmanauts Mining DAO token i get "The wallet did not contain the tokens required for staking" as the picture shows.
i have the token on my wallet and also its shown bellow amount that i have 1 available tokens to stake "1 SIG available"

@moon-miner This discrepancy can be confusing. Have you tried refreshing the wallet

@moon-miner This discrepancy can be confusing. Have you tried refreshing the wallet

Deleted the wallet from nautilus, restored it from seed phrase, connected it to paideia and still the same issue.

@moon-miner  It sounds like your query would be best dealt with by the support team.
I'll have to refer you to the official support live chat with the ticket ID SIG07638
Please see the link below to our dedicated support line:
Staking Sig Network DApps Support Ticket Requests.
Click on the live chat icon at the bottom corner of the page to initiate chat with a live support agent.

@moon-miner It sounds like your query would be best dealt with by the support team. I'll have to refer you to the official support live chat with the ticket ID SIG07638 Please see the link below to our dedicated support line: Staking Sig Network DApps Support Ticket Requests. Click on the live chat icon at the bottom corner of the page to initiate chat with a live support agent.

This is a scammer. Do not do this. Hop in sigs chat or dm me from sigs chat.

Also people have had luck consolidating and making sure bPaideia and the Sig token are the only things there.

Also people have had luck consolidating and making sure bPaideia and the Sig token are the only things there.

Have consolidated and still the same, its the same wallet i use for cyberverse wallet cant left sig dao token alone :S

I didn't have bpaideia in my wallet :B


Passing Data/Object between step

Hi!
Do you have any suggestion how to passing data between step?
Currently i am use liniear step

@lukmanfarizi Somewhere in your code, you build a list of steps to pass to the Stepper. Your step 2 model class could have a reference to your step 1 model class.


Create rtl8812au_v5.2.20_Makefile

CONFIG_PLATFORM_I386_PC = y -> n
CONFIG_PLATFORM_ARM_RPI = n -> y

First time I'm trying to use PullRequest !

Ok, I understand.
Thanks.


DataLake filesystem ACL issue

Package Name:  '@azure/storage-file-datalake';
Package Version<EMAIL_ADDRESS>Operating system:  Windows 10
[x] nodejs

version:  v12.16.1

[ ] browser

name/version:

[x] typescript

version<EMAIL_ADDRESS>

Is the bug related to documentation in

[ ] README.md
[ ] source code documentation
[ ] SDK API docs on https://docs.microsoft.com

Describe the bug
Trying to set an ACL entry on a hierarchical namespace file system.  Receiving the following error:
"XML specified is not syntactically valid."
"code": "InvalidXmlDocument"
To Reproduce
Steps to reproduce the behavior:
import { TokenCredential, AzureCliCredential as Cli } from '@azure/identity';
import * as azdatalake from '@azure/storage-file-datalake';

let credentials: TokenCredential;

credentials = new Cli();

const storageName = 'storageaccountname';

const adlsServiceClient = new azdatalake.DataLakeServiceClient(
	`https://${storageName}.dfs.core.windows.net`,
	credentials
);

async function main() {
	// Create a file system

	const fsName = 'mynewfs3';
	const adlsFSClient = adlsServiceClient.getFileSystemClient(fsName);
	const createResponse = await adlsFSClient.createIfNotExists();

	console.log(`Create file system ${fsName} successfully`, createResponse.requestId);

	//add the acl
	try {
		const aclResponse = await adlsFSClient.setAccessPolicy( undefined,
			[
				{
					'id': '8675309',
					'accessPolicy': {
						permissions: 'RWD',
						expiresOn: new Date('2099-09-28T08:49:37.0000000Z'),
						startsOn: new Date('2022-04-06T08:49:37.0000000Z')
					}
				}
			]
		);
	} catch (err) {
		console.error("something failed... bruh")
		console.error (err);
	}
}

main();

Expected behavior
A new ACL for the user id '8675309' should be created on the filesystem 'mynewfs3' with permissions of 'RWD'.
Screenshots
Additional context
Also unable to call the getAccessPolicy() on the same filesystem even when it exists.  The getAccessPolicy() returns a 404 too.

Hi @nu11modem ,
Storage service only accepts lower cased permissions string.  Could you try again with specify the permission with lower case letters?
permissions: 'rwd',
Thanks
Emma

Ok that helped.  Now i'm getting a 404 with this error message.
"message": "The specified resource does not exist."
Can you tell me what the format for the 'id' is supposed to be?  Is it just the 'id' parameter of the AAD user or Group?

Hi @nu11modem ,
The id is just a string. 404 means that the resource you reference doesn't exist, maybe it's deleted or not created.  We may need a repro for detailed investigation.
Thanks
Emma


Ionic Lab updates

[x] Make both iframes the same size
[x] Change iPhone 6 to iOS
[x] Make the header a link which opens that iframe in its own page
[ ] Landscape mode
[ ] Tablet mode (only one view at a time though)
[ ] Disable the entire webpage from bouncing when scrolling one of the iframes
[x] Give the lab a <title>
[x] More obscure url, /lab may already be used by someone's app. Maybe /ionic-lab
[ ] Option to have both apps navigate together, even though you only click one of the iframes to navigate
[ ] Pretty @bensperry designed way to change these settings. Clean/simple

Hello, can I suggest a new feature?
I use the --lab for development (more than just a final review) and it would be lovely if the livereload could maintain the route as well. It is a bit of hassle to having to navigate to somewhere just to see the new changes.

@Foxandxss I've been looking into that. I've not found a way yet to make just the iframe reload.
It appears the entire page reloads. However, each iframe points towards android/ios. Those should update individually, but do not. I think the entire window gets reloaded.
I'd love some help to figure it out though. Any ideas?

@jbavari yes I want this too ...
I love the "labs" option but yes, it should be able to maintain its route (for each iframe)
It's a pain to have to navigate to that view every time you make a change.
I want it so badly that I'd volunteer to work on it.
Especially since nobody else seems to be picking this up anytime soon.
Opinions?

It seems that issue has been resolved. I am going to close this issue but please feel free to reopen another issue. Thanks!


Ernest: randomizations heatmap should show node degree information

Generate a version of the heatmaps for parameter selection that simultaneously capture node degree, robustness and specificity.  Since degree is fixed for a node, it could be plotted at the margin of the heatmap.  Robustness and specificity depend on the parameters.  Perhaps we could have two versions of the plot:  (A) Color scale for robustness for each matrix element with average specificity of nodes shown in the margin and (B) color scale for specificity for each matrix element with average robustness of nodes shown in the margin.

@iamjli implemented this a long time ago I think


On Windows 10 Mobile DisplayName has no effect

Hi,
I can't seem to override the default DisplayName (so my app's name appears) on my live tile on Windows 10 mobile. I tried to set the DisplayName property on the TileVisual and on the TileBinding object and it works fine on desktop but not on mobile. I know that there's a limitation on the branding attribute on mobile so it only supports the name or none branding but this page doesn't say anything similar on the displayname.
https://msdn.microsoft.com/en-us/library/windows/apps/mt590880.aspx
I've also tried using the plain xml based approach with the same result so it's probably not a bug in the NotificationExtensions wrapper but here's my code, am I doing something wrong?
TileContent tile = new TileContent()
{
Visual = new TileVisual
{
DisplayName = report.CameraName,
Branding = TileBranding.Name,
TileMedium = new TileBinding
{
Content = new TileBindingContentAdaptive
{
BackgroundImage = new TileBackgroundImage { Source = new TileImageSource(report.SquareSnapshotPath) },
}
},
TileWide = new TileBinding
{
DisplayName = $"{report.CameraName} - {report.SnapshotTime.ToString("HH:mm")}",
Branding = TileBranding.NameAndLogo,
Content = new TileBindingContentAdaptive
{
BackgroundImage = new TileBackgroundImage { Source = new TileImageSource(report.WideSnapshotPath) },
}
},
TileLarge = new TileBinding
{
DisplayName = $"{report.CameraName} - {report.SnapshotTime.ToString("HH:mm")}",
Branding = TileBranding.NameAndLogo,
Content = new TileBindingContentAdaptive
{
BackgroundImage = new TileBackgroundImage { Source = new TileImageSource(report.SquareSnapshotPath) },
}
}
}
};

Hey Senkia, what build of Windows 10 Mobile are you running? Sounds like it's definitely a bug in Windows 10 Mobile since it works fine in Desktop, but I'd be happy to investigate the bug and see whether it's already been addressed.

I'm using version 1511, build 10586.29 on mobile and version 1511, build 10586.71 on desktop.

It's also strange that the built in weather app seems to use the displayname to show the current location and it works fine. I've also tried hiding the displayname and using a simple tile text instead but the tile's built-in padding makes it out of line from the other tile's names.

Hmm yeah, the Weather app uses displayName, so if it works there, it should be working for you too. I can't repro this issue on the latest builds, I'm trying the older 10586 builds and looking through old bugs to see if we fixed anything around this.

Ohh, I think I might know what's going wrong...

Is this for your primary tile?
Are you using the ShortName property in your app's manifest?

Mobile has a bug where, if ShortName is specified on the base tile, the displayName provided by the tile notification will not work (the ShortName is always displayed).
Workaround is to not use the ShortName property on your primary tile (open the Package.appxmanifest visual editor, select the Visual Assets tab, and on All Image Assets, make sure the "Short name" text box is blank)

Thanks! That did solve my problem.


Update the VNET peering doc

Updated the wording on the VNET peering until ExpressRoute works

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

@TylerMSFT, can you review this PR? I can't reach the author in GitHub.


[DO NOT MERGE] New ToC

IMO we should go with radically fewer, but more detailed, pages. I think Druid's large number of doc pages doesn't really help make things organized, it just makes it hard to find information. IMO, this is because for some questions people have, it's not clear which doc to click on, and their needs would be better served by longer doc pages that collect everything related to a topic.
For example see Spark, which has relatively few pages for how complex it is. The "Programming Guide" page is quite long and collects a lot of information together: http://spark.apache.org/docs/2.1.0/
For an extreme example see Kafka, which has just one, linear page for the whole project: https://kafka.apache.org/documentation/

What do people think of this? In my mind each of the top level bullets is its own page, and the secondary bullets are headings within that page -- not their own pages.

Getting started

Why Druid? (explains that we're "big olap" and explains how we compare to related systems)
Quickstart

Setup and installation

What do people think of this? In my mind each of the top level bullets is its own page, and the secondary bullets are headings within that page -- not their own pages.

Getting started

What is Druid? (explains that we're "big olap" and explains how we compare to related systems)
Use cases (what we're good for, plus "anti-use-cases" for things that Druid is not targeting)
Quickstart. Single machine.

Design

Motivation: describe why Druid is the way it is. I'd say the design is all about targeting maximum query and ingest performance for append-based workloads.
Introduce terminology: dataSource, segment, dimensions, metrics, rollup
Discuss segments and their internals and the fact that they are immutable once built
Discuss the segment timeline and how Druid uses that to partition a datasource. This should cover how locking works and how batch and realtime ingestion work together. Probably need a graphic here to make it clear.
Discuss implications of the segment timeline, such as how load/drop rules are able to use it to move data around quickly, and how the broker is able to use it to target specific segments based on your time filter.
Discuss how realtime indexing builds segments. Probably need a graphic here too.

Cluster setup

Hardware recommendations
Deep storage, metadata store, zookeeper
What configs are important to tune first
Scaling
High availability
Backup

Querying

SQL
Native Druid queries

Ingestion

Overview. Hit the basics. Rollup is a thing and can be on or off. You can ingest the same data multiple times with different rollups. Ingestion is largely pull-based, except for one exception: tranquility provides a way to push streams. Ingestion is always partitioned by time, and you can run different kinds of ingestion on different time buckets simultaneously, but not two at once on the same time bucket. And finally, link to "design" for people that want the gory details.
Index task
Hadoop index task
Kafka index task
Tranquility
Data formats that apply to all of those
Reindexing mechanisms that apply to all of the above
Strategies for handling updates

Configuration

All runtime properties for all node types, organized by what file they should appear in (common vs system specific). This will be long but will be very useful to have in one place.
All kinds of dynamic configuration, including coordinator dynamic config, load/drop rules, overlord worker config, & anything else like that.
JVM configuration. Discuss memory tuning and also call out things like user.timezone.

Administration and monitoring

All the endpoints on the coordinator, overlord, broker that do administrative things.
Docs about the administrative UIs that are available (coordinator/overlord console).
Docs about emitted metrics and alerts, and where they can be emitted to.

There's some stuff in the existing ToC I don't have in that outline. Not sure where to put them though. See: http://druid.io/docs/0.10.0/design/index.html
Like multitenancy, multi-value dimensions, information about extensions.


Allow specifying regular expression when attaching to process

When attaching to a process, I would like to automatically select a process that matches a regular expression.
Example Process Picker:
dotnet /usr/share/dotnet/dotnet /usr/share/dotnet/sdk/6.0.401/MSBuild.dll
bash
sleep 180
dotnet myapplication.dll -- --my-app-args

The regex would look something like this: /dotnet.*myapplication.dll/g
{
    "version": "0.2.0",
    "configurations": [
        {
            "name": ".NET Core Attach",
            "type": "coreclr",
            "request": "attach",
            "regex": "/dotnet.*myapplication.dll/g"
        }
    ]
}

If only 1 matching process was found then it automatically selects it and attaches. If multiple processes were found then it prompts the user to select 1

Please file this on the repo for that extension


Implement ipdk install [environment] option

This option adds the option to configure IPDK CLI to a predefined IPDK runtime environment as described in #72. The current predefined IPDK runtime environments are:

default: configure the default environment (currently Fedora33 container)
host: configure for a Vagrant/other VM or bare metal environment
fedora33: fedora core 33 container environment
ubuntu2004: ubuntu 20.04 container environment
ubuntu1804: ubuntu 18.04 container environment

The host environment option is a preparation for another development to integrate the Vagrant and Bare metal flow in the CLI.

@mestery I'm ready with the IPDK CLI Runtime Environment setup option 😄 Can you have a look ?? Thanks!!

@mestery Removed the CLI documentation part. Did you had time to review yesterday? If you did then its ready for merge I think...

@mestery Removed the CLI documentation part. Did you had time to review yesterday? If you did then its ready for merge I think...

I didn't have a chance to review it yesterday, I will do it later this morning once I get out of a few meetings. Thanks for working on this!

@stolsma I never got around to reviewing this yet, I will do this on Monday morning. Note that I have enabled CI now, so if you rebase and push, you should see a GitHub Action run and do basic CI.


Python 3.12 support

What is the goal that you want to achieve by the request?
Python 3.12 came out on October 2, 2023, by default Ubuntu 24.04 now comes with Python 3.12. It would be nice to implement support for it.
Additional context
To my understanding from #422 the only thing holding back is genicam package, but I might be wrong.

I have a large project, depending on Harvesters and it's been a pain that there's no support for 3.12. We will be switching to other methods when we can, to avoid Harvesters.

I'd like to second this. I tried to get in touch with the maintainer of the genicam package at the maintainer e-mail address (genicam@list.stemmer-imaging.com), which seems to be the same address as the one for harvesters. However, the address is no longer in use. I've asked the owners of the domain about this address, but they don't seem to have any knowledge of it. Is @kazunarikudo still around? They seem to be the sole person responsible for both genicam and harvesters

Hi @matthijsgeers-keysight Thanks for the hint that the e-mail adress is not working anymore.
I will check to update to an actual one.
About the Python 3.12 release: We are working on the release and I would expect to have the tested binaries ready on approx. 1 month.
Actually we are working on a new build infrastructure where future updates would be available much faster. I have to say sorry that it take so long for this 3.12 release. It would be much better in the future, I promise it.

I asked a genicam developer some weeks ago and he said a new genicam release is in the making and Python 3.12 support will be included but there was no timeline yet.

Ow, I just realized that we can manage who are public visible behind the "GenICam organization" on github. Right now, I configured me also as public member of the organization.
I can say, we are a bigger group working on the standard. I spend time to refactoring the whole python build infrastructure (of the genicam package). This allows as to create releases more efficient and faster...
Ans yes, we try to build python 3.13 as well. Since the release which I spoke is still on the old infrastructure which uses a few features depreciated in python 3.13 I'm not sure if we will finalize it. If the fixes small, than of course.

One question.!
Is the harvester and genicam compatible with python 3.11?

Ok, nice ! Because I'm having problems fetching the return buffer, which is always empty, and it could be due to my version of python.


TypeError: Cannot read property 'length' of undefined

      scope.inputChangeHandler = function(str) {
        if (str.length < minlength) {
          clearResults();
        }
        else if (str.length === 0 && minlength === 0) {
          scope.searching = false;
          showAll();
        }

        if (scope.inputChanged) {
          str = scope.inputChanged(str);
        }
        return str;
      };

First if in 1st line of this function is broken because of str is undefined.

Thanks for reporting this. When does it become undefined?

When I enter input field, type something, then erase text with backspace button. I see error message in browser console after deleting last character.
By the way, I'm using chrome for debug needs.

Would you please to answer something to my problem: https://github.com/ghiden/angucomplete-alt/issues/193#issuecomment-142222860

I don't see it with the example 10.
Can you create an example on plnkr?

i just ran into this when i added ng-required to the input element in my custom template instead of using the field-required attribute.

@bulbulas I'm not supporting adding ng-required to the template as it will only check the field is filled or not. It does not check if the field is filled with valid value from selection or not.

I have successfully add the n-required in my template as said here https://github.com/ghiden/angucomplete-alt/issues/277
This works fine for me.


[EPIC] Phase 1: ERC Registry

Problem
On Hedera both HTS tokens and ERC tokens are supported. However, HTS tokens have improved APIs and allow for easy retrieval of existence and balance across accounts on the network.
Like other EVM chains, it is not easy to know what all ERC tokens are specified within smart contracts on the network without first knowing the contracts and confirming they accurately implement the interface needed.
With this in mind it's difficult for the mirror node explorer or other apps to know and display ERCs
Solution
As a first pass to accelerate improved UX we can produce a registry of contracts that are beleived with high confidence to be  ERC's. The Mirror node explorer and other apps can take this list as a starting point to build out their UX for users.
Simple approach is to

create a self contained runnable npm project erc-repository-indexer under tools.
Index the networks contracts and verify they are ERC20 or ERC721 by matching the presence of interface function bytes. Utilize current POC on 1st pass if applicable and then sevm soon after. Ensure the tool support the specification of env, mirrorNodeUrlandstartingContractId` to allow for flexibility.
Create an ERC20s_.json and an ERC721s_.json file each with the appropriate but minimal details. Files should be placed under tools/erc-repository-indexer/registry. Entries should be sorted in ascending order of contractId
For additional confidence one may simulate an eth_call on the functions also. Consider pros and cons based on accuracy to this point
Make the tool runnable in CI with the output being a new PR with the file updates for approval and merge

A starting schema for an ERC20 may be
{
    "address": "0x....",  // Address of the ERC20 contract/token
    "contractId": "0.0.x",  // contract id of the ERC20 contract/token
    "name": "...",
    "symbol": "...",
    "decimals": x,
    "totalSupply": "..."
  }

A starting schema for an ERC721 may be
{
    "address": "0x....",  // Address of the ERC20 contract/token
    "contractId": "0.0.x",  // contract id of the ERC20 contract/token
    "name": "...",
    "symbol": "...",
  }

Alternatives
No response
### Tasks
- [ ] https://github.com/hashgraph/hedera-smart-contracts/issues/1029

@quiet-node and @natanasow to sync up on this and product a design doc first highlight approach.
Doc should be placed in tools/erc-repository-indexer/docs/registry_design.md

Suggestion from @acuarica to utilize the verification service to add additional confidence.
This could be done using the verification service APIs

Discussions about this Epic at https://github.com/hashgraph/hedera-smart-contracts/discussions/1035


Fix build issue with USE_CUDNN=0

Description
This was found during the flaky test investigation https://github.com/apache/incubator-mxnet/issues/10973 . I found that the windows code path is different from linux, so it was testing the MXNet conv and deconv op which was not being tested for linux. Building with USE_CUDNN=0 and USE_CUDA=1 was failing and I have added the fix here. I have also added the build and test as a stage to jenkins. Not sure if there was a reason to not check this in CI (configuration not supported by some ops etc.)
@marcoabreu
Checklist
Essentials
Please feel free to remove inapplicable items for your PR.

[ ] The PR title starts with [MXNET-$JIRA_ID], where $JIRA_ID refers to the relevant JIRA issue created (except PRs with tiny changes)
[ ] Changes are complete (i.e. I finished coding on this PR)
[ ] All changes have test coverage:
Unit tests are added for small changes to verify correctness (e.g. adding a new operator)
Nightly tests are added for complicated/long-running ones (e.g. changing distributed kvstore)
Build tests will be added for build configuration changes (e.g. adding a new build option with NCCL)
[ ] Code is well-documented:
For user-facing API changes, API doc string has been updated.
For new C++ functions in header files, their functionalities and arguments are documented.
For new examples, README.md is added to explain the what the example does, the source of the dataset, expected performance on test set and reference to the original paper if applicable
Check the API doc at http://mxnet-ci-doc.s3-accelerate.dualstack.amazonaws.com/PR-$PR_ID/$BUILD_ID/index.html
[ ] To the my best knowledge, examples are either not affected by this change, or have been fixed to be compatible with this change

Changes

[ ] Feature1, tests, (and when applicable, API doc)
[ ] Feature2, tests, (and when applicable, API doc)

Comments

If this change is a backward incompatible change, why must this change be made.
Interesting edge cases to note here

@szha Yep, planning to fix that.

the spatial transformer op seems to be a blocker with a bunch of issues when cudnn disabled: #11568. I am currently hiding the assertion error for this op and triggering ci again. Need to stop hiding the assertion error when the issues with the op are resolved: #11568.

We have to take care of conflicting changes when merging this PR and #11630.

I have resolved the conflicts and addressed the comments on #11630

This should fix #11339


[BUG] Lombok @Data not working - The annotation @Generated is disallowed for this location

Lombok is installed in STS, I've been using it on multiple other projects. I've imported a new project, tried Maven -> update project, tried cleaning projects.
Issue is only with the @Data annotation, it gives a red line underneath and shows the message "The annotation @Generated is disallowed for this location"
I've tried @Getter, @Setter, @EqualsAndHashCode and they all work fine.
Can't seem to find anyone with a similar issue and yesI have installed the plugin.
Using v1.18.10 Lombok and STS 4.5.0 release

Closed. No response from reporter.


You need a third programme to do it.
If you run the bot with PyCharm :

Go to the PyCharm window (with alt + tab in Windows).
Stop the programme with Ctrl + F2 (tested with Windows)

You can now long press on "Escape" key to stop the programme.


version_dir and scripts_dir do not exist in attributes/default.rb and README.md

If using the individual consul-ng::install and consul-ng::config recipes, without specifying the default['consul']['version_dir'] and default['consul']['scripts_dir'] values in an attributes file, you end up with an error like this:
       ArgumentError
       -------------
       You must supply a name when declaring a directory resource

       Cookbook Trace:
       ---------------
         /tmp/kitchen/cache/cookbooks/consul-ng/recipes/install.rb:34:in `block in from_file'
         /tmp/kitchen/cache/cookbooks/consul-ng/recipes/install.rb:33:in `each'
         /tmp/kitchen/cache/cookbooks/consul-ng/recipes/install.rb:33:in `from_file'

       Relevant File Content:
       ----------------------
       /tmp/kitchen/cache/cookbooks/consul-ng/recipes/install.rb:

        27:   node['consul']['version_dir'],
        28:   node['consul']['conf_dir'],
        29:   node['consul']['scripts_dir'],

        31:   node['consul']['config']['data_dir'],
        32:   node['consul']['log_dir']

        34>>   directory dir do
        35:      owner node['consul']['user']
        36:      group node['consul']['group']
        37:      mode node['consul']['mode']
        38:      recursive true

This looks to be due to variables that do not exist, being used as inputs to the .each.
Specifying them in the attributes/default.rb, in addition to the README.md would be ideal to avoid such an error on a fresh setup :)

@CpuID Keeping the attributes in default recipe works for config, install and other recipes to maintain attributes precedence.
How about adding a switch to disable install and config recipe? this way one can choose install or config recipe.
default['consul']['configure'] = true
default['consul']['install'] = true

Published new version. Please reopen if required.


[Request] Sprite animation customization.

It would be great if we were able to customize the types of animation per sprite. Apart from the current (no animation, facing and 2 frame facing walk cycle) I would love to have:

Idle animation (2+ frames)
Longer walk cycle

Both of those can be implemented by adding extra frames to the y axis on the sprite sheet (so idle 2 frame animation would be 16x32)
Making idle animations will really help create more enticing worlds.

I agree, especially on the idle animation point, that's been on my backlog for a while but I didn't want to wait before releasing. Will be looking into this.

Awesome, waiting for those idle animations!

If more than just an idle animations are implemented, another awesome feature would be to have an option of "play animation" from the drop down menu in code.

I'll pip in to say that just being able to change the timing of the animations would be great; Just that would make the walking look less hover-y, without needing to add support for custom framecounts yet.
That said, this is an awesome tool! I can't wait to see where it goes.

Sprite animation with animation speed controls are now part of the 1.1.0 release :-)


A question of train

Hello,
I am very interested in you work, and try to use this code in my segmentation task. but i have some question of it. I am try use code like train_LA_MultiHead_FGDTM_L1 in my segment work, and epoch_numbers is 10000, but when i use torch.save model to test data, they will get the predict all is 0. and the dist is also. and i try to print the predict_y that unprocess via argmax. and i foud that y[:,0,:,:,:] is all bigger than y[:,1,:,:,:]. it means the net can't distinguish the foreground and background. so, please give me some advice. thanks.

Hi @WeiliJiang ,
I apologize for the very late reply. Recently, I'm busy with the FLARE21 Challenge.
Regarding your question, I have not met this problem. I would recommend trying the plain V-Net to check whether such a problem still occurs.


Add asyncapi_gencpp, log_view, rpad, rpad_ros, and param_util source to rolling

References https://github.com/ros2-gbp/ros2-gbp-github-org/issues/88
Please Add This Package to be indexed in the rosdistro.
rolling
The source is here:

hatchbed_common
log_view
asyncapi_gencpp
opensw
opensw_ros

Checks

[x] All packages have a declared license in the package.xml
[x] This repository has a LICENSE file
[x] This package is expected to build on the submitted rosdistro

I have to agree with @cottsay that the names param_util and log_view are too generic.  For that matter, while rpad isn't that generic, I'd also argue that it is a bad name for it.  Maybe slamware_driver or something like that might be better.  At the end of the day, users are going to be doing things like installing:
ros-rolling-rpad, and that just doesn't seem to provide enough context about it.
For reference, here is our naming guidelines for packages: https://www.ros.org/reps/rep-0144.html .  Particularly, see https://www.ros.org/reps/rep-0144.html#global-rules

Yeah, I understand on param_util.  I'll come up with something else.
log_view has already been released in ros1 for a while, though, and also more recently in galactic: https://index.ros.org/p/log_view/github-hatchbed-log_view/
Good point about rpad, but I'm not sure about using a commercial trademark like slamware, slamtek, rplidar, etc in the name.  I'm fine with the name being a bit cryptic to avoid potential trademark liability since the package description and readme explain the use case.   I'm also open to a name that gives more context, but it's not clear to me what that would be.  Maybe opensw and opensw_driver are marginally better?

Here are some alternate name for param_util:

parameterize
rosparam_helper
rosparam_kit
rosparam_assist
param_kit
param_helper
param_assist
param_aid

Let me know if any of those work

Unfortunately most of those proposed names don't really help you if you want to search for this package. One easy way to think about this is what would I find if I searched for the name? If I search for param rosparam or any of the generic modifiers like kit how likely is it that I'll find documentation for this package? Obviously a full string match will likely find it but others not so much.
For the trademark you don't want to pretend to be them, but if you're writing something that works with a device you can talk about how you're compatible with that device. And if you use a good searchable term then people can find the package easily.

Unfortunately most of those proposed names don't really help you if you want to search for this package. One easy way to think about this is what would I find if I searched for the name? If I search for param rosparam or any of the generic modifiers like kit how likely is it that I'll find documentation for this package? Obviously a full string match will likely find it but others not so much.

Yeah, I see that.
Doing a search of rosparam on the ROS package index turns up:

rosparam_shortcuts - (1st page) a similar package to help with parameters, but different approach and set of features
rosparam_handler - (2nd page) a similar package, but different approach and set of features
rtt_rosparam - (2nd page) "Plugin for synchronizing ROS parameters with Orocos component properties."
core packages: rosparam, ros_comm, test_rosparam

Searching for param:

launch_param_builder - (1st page) "Python library for loading parameters in launch files"
choreo_param_helpers - (3rd page) "Implements routines for loading, saving, and checking for the existence of user parameters."
core packages:  rcl_yaml_param_parser, ros2param,

That seems pretty reasonable to me.  I think including rosparam in the package name gives more context than simply param and would certainly be more searchable outside of ROS.
A second word like kit or helper provides more context when paired with rosparam, but on it's own is very generic and would pull unrelated packages into the results even paired with rosparam.  I'm not sure in what context someone would search for just helper, kit, utils, etc.
So, is rosparam_<some acceptable modifier> a good approach?

rosparam_tools

another generic option, LOTS of _tools packages and results when searching for 'tools'
but it sounds good to me and seems the most on point

rosparam_toolset
yarpp

YetAnotherRosParamPackage :laughing:

rosparam_yarp

Open to suggestions

For the trademark you don't want to pretend to be them, but if you're writing something that works with a device you can talk about how you're compatible with that device. And if you use a good searchable term then people can find the package easily.

Right, I agree with that.  It shouldn't be an issue to describe that the software is compatible with the devices and that is in the README, package.xml, etc.  It's more of putting the trademark in the actual name of the package, which, not being a lawyer, but doing some web searching, looks like is generally not recommended.
Not sure how to add more context to the package name in this case, other than my previous suggestion of using opensw and opensw_driver which might be marginally better for context.
Open to suggestions, though.

@malban friendly ping

@malban friendly ping

Thanks for the reminder!

Ok, based on comments, I've renamed:

param_util -> hatchbed_common
rpad -> opensw
rpad_ros -> opensw_ros

Generally this looks good to me now.  Thanks for the iteration.
The log_view one is unfortunate, since it is really generic.  But it was accepted into other ROS 2 distributions, so we can take that one on precedent.


Store Username and Factor Option in Configuration

In an effort to make the workflow as quick and simple as possible, we should allow the user to store their username, and multi-factor of choice as part of the configuration.  This will allow really painless refreshing/changing of credentials.
For example:
$ gimme-aws-creds.py --profile lab
Using password from keyring for <user>
Authentication Success!  Calling Gimme-Creds Server...
Multi-factor Authentication required.
Okta Verify push sent...
writing role <role> to <credfile>

The only interaction here would be with the MFA factor.

Covered in #55

#55 adds this functionality


Update doc for interpreter support for markdown

Tested in https://github.com/wackyleo459/test-motoko-interpreter/pull/1

Yeah, not easy to translate. We could manually add back the attributes. There are not many of them.


Glitches while streaming video from a Surface

Hi,
I'm trying to stream Android display using RTSP with this library. I modified it to take input from a Surface instead of the Camera. I am successfully able to achieve this much so far. However, the rendered video has lots of glitches.
Anyone has any idea what could be going wrong or any tips on how could I debug?
I have a feeling that SPS and PPS are wrong. But I'm not too sure how to debug that.
@fyhertz
-Umair

Try uncommenting out these two lines. https://github.com/fyhertz/libstreaming/blob/master/src/net/majorkernelpanic/streaming/gl/TextureManager.java#L113
If that doesn't work, please post a screenshot.

Hi @sbaar
I instead moved to a different approach altogether. I am sending the raw h264 data directly to the web browser and inside browser, using BroadwayJS to decode the stream and render it. :)


Top Bar mobile_show_parent_link only appears at Small breakpoint

The JS generated markup:

Products & Services

It would be nice if the Parent Link would show at the $topbar-breakpoint instead of at Small only.

Can you submit a PR because this would help others?

The parent links visibility needs to match the top-bar breakpoint, not the grid breakpoint.

@gakimball Do we have a class that can signify a 'topbar breakpoint show/hide' behavior?

@jeaniec that's a good idea – I can already think of places outside the Top Bar where I would use this. Quite a few elements change appearance at the TB breakpoint...

I've recently added this as a breakpoint, and came here to add this issue of the parent link as a request .
I'd be willing to make a pull request if those in the know think there's a good chance they'd accept it.
What would be a good name for this breakpoint? Is there "official" foundation lingo for the full desktop menu vs the mobile menu?

@vita10gy If you want to submit a pull request and all looks good, we'd be happy to pull it in. There's no official lingo, but exporting a class something along the lines of "hide-for-topbar-mobile" or "hide-for-topbar-desktop" should be fine. After you submit the pull request, the team will look over it and see if there are any changes that we might want to commit on our end as well.

I also added this breakpoint in to the grid so I could do something the equiv of topbar-desktop-5 That way I could make the header above the menu play ball with the menu breakpoint and visibility classes we're talking about.
Would that interest people, or it that getting too specific?
Also, one "catch" to this change we're discussing is that, as far as I can tell at least, though truth be told I didn't spend a ton of time on trying alternatives: the breakpoint needs to be defined with the others in _globals and not _topbar.
Is that too "breaking" to justify for this?

Thanks for pointing this out. Ideally the visibility would match the top-bar breakpoints. We're not adding any features to Foundation 5 as we're concentrating on getting Foundation 6 out.


Exception on upload, dotnet 7

System.InvalidOperationException: No reading operation to complete.
  ?, in void ThrowHelper.ThrowInvalidOperationException_NoReadToComplete()
  ?, in void Pipe.AdvanceReader(BufferSegment consumedSegment, int consumedIndex, BufferSegment examinedSegment, int examinedIndex) x 2
  ?, in void Http2MessageBody.AdvanceTo(SequencePosition consumed, SequencePosition examined)
  ?, in void HttpRequestPipeReader.AdvanceTo(SequencePosition consumed)
  ?, in void ClientDisconnectGuardedPipeReader.AdvanceTo(SequencePosition consumed)+() => { }
  ?, in bool ClientDisconnectGuard.Execute(Action guardFromClientDisconnect, CancellationToken requestCancellationToken)
  ?, in void ClientDisconnectGuardedPipeReader.AdvanceTo(SequencePosition consumed)
  ?, in void MaxReadSizeGuardedPipeReader.AdvanceTo(SequencePosition consumed)
  ?, in async Task<long> TusDiskStore.AppendDataAsync(string fileId, PipeReader reader, CancellationToken cancellationToken)
  ?, in async Task<Tuple<long, CancellationToken>> WriteFileHandler.HandlePipelineWrite()
  ?, in async Task WriteFileHandler.Invoke()
  ?, in async Task<ResultType> TusProtocolHandlerIntentBased.Invoke(ContextAdapter context)
  ?, in async Task TusEndpointInvoker.Invoke(HttpContext context)
  ?, in async Task EndpointMiddleware.Invoke(HttpContext httpContext)+AwaitRequestTask(?)
  File "Middleware\IPAccessCheckMiddleware.cs", line 38, col 9, in async Task IPAccessCheckMiddleware.InvokeAsync(HttpContext context)
  ?, in async Task AuthorizationMiddleware.Invoke(HttpContext context)
  ?, in async Task AuthenticationMiddleware.Invoke(HttpContext context)
  ?, in async Task SwaggerUIMiddleware.Invoke(HttpContext httpContext)
  ?, in async Task SwaggerMiddleware.Invoke(HttpContext httpContext, ISwaggerProvider swaggerProvider)
  ?, in async Task DeveloperExceptionPageMiddlewareImpl.Invoke(HttpContext context)

Hi,
I'm not really sure what you want me to do with the stacktrace or the limited info you have provided? Disabling pipelines would get you some other error as the tech used to read the data from the client is completely different. What does your tusdotnet setup look like, what client are you using and what chunk size are you using in your client?

Client is Uppy. Without pipelines it gives nice error to browser. With pipelines it's not easy to understand what is a problem.
It is easy to hit this situation making very simple POC so it would be nice if nicer error can be logged somewhere.

I'm trying to help you here mate but you need to help me help you. I'm not going to spend time trying to recreate an issue that you obviously already know how to create but for some reason refuse to tell me.

It is easy to hit this situation making very simple POC so it would be nice if nicer error can be logged somewhere.

If it's such a simple POC, why are you stalling instead of just providing it?

It's the most basic example:
 var uppy = new Uppy();         uppy.use(Tus, { endpoint: '/files', chunkSize: 1048576 })         uppy.use(Dashboard, {             inline: true,             target: '#drag-drop-area',         )
    I don't need help any more as I have solved the problem. I'm just suggesting you add nice message to above exception like "Chunk size probably too large"

Right. Thanks for reporting this!

I can shed some additional light on this one.
Here's my server side setup:
`app.UseTus(httpContext => new DefaultTusConfiguration
{
// This method is called on each request so different configurations can be returned per user, domain, path etc.
// Return null to disable tusdotnet for the current request.
            // c:\tusfiles is where to store files
            Store = new TusDiskStore("/tusfiles/"),
            // On what url should we listen for uploads?
            UrlPath = "/tusupload",
            Events = new Events
            {
                OnFileCompleteAsync = async eventContext =>
                {
                    ITusFile file = await eventContext.GetFileAsync();
                    Dictionary<string, Metadata> metadata = await file.GetMetadataAsync(eventContext.CancellationToken);
                    //Tus accepts all uploads, but we need to triage them to the correct processing pipeline
                    //We can do this by looking at the metadata
                    if (metadata["processing_pipeline"].GetString(System.Text.Encoding.UTF8) == "nessus")
                    {
                        BackgroundJob.Enqueue<ProcessNessusFile>(x => x.Ingest(file, metadata, eventContext.CancellationToken));
                    }
                    else if (metadata["processing_pipeline"].GetString(System.Text.Encoding.UTF8) == "bloodhound")
                    {
                        //Process the Bloodhound upload
                    }
                    else if (metadata["processing_pipeline"].GetString(System.Text.Encoding.UTF8) == "ntds")
                    {
                        //Process the NTDS upload
                    }

                }
            }
        });`

On the Client (Using ts-js-client):
``

@bkuzmacissp Thanks but not really what I'm after. Based on the stacktrace I'm assuming you run some kind of IIS integration and would need to know how you setup your app to be able to reproduce the issue. Basically anything configured on the app builder related to the webhost or host and if you setup Inprocess mode in your web.config?
var builder = WebApplication.CreateBuilder(args);

// Are you running UseIIS?
// builder.WebHost.UseIIS()

// Or UseIISIntegration?
// builder.WebHost.UseIISIntegration()

Web.config example (see hostingModel property):
<aspNetCore processPath="%LAUNCHER_PATH%" arguments="%LAUNCHER_ARGS%" forwardWindowsAuthToken="false" stdoutLogEnabled="false" hostingModel="inprocess">

@bkuzmacissp Are you still experiencing this issue if using the code in https://github.com/tusdotnet/tusdotnet/commit/2b1834616fbf750df4e4dc3bd2d77778357fe59f ?

Gimme a couple of hours and I’ll repro it with that sampleSent from my iPhoneOn Feb 3, 2023, at 6:48 AM, Stefan Matsson @.***> wrote:
@bkuzmacissp Are you still experiencing this issue if using the code in 2b18346 ?
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: @.***>

@bkuzmacissp Any progress?

Closing due to inactivity


Unable to Build Wheels

I can't get the monkeypatch to install no matter what I try. It seems to be a cuda issue during the wheel building process. I'm on Win10, nvidia gpu, amd cpu. The following error might be from a docker image attempted build, but the error is almost identical to others so I feel like it's the same overall problem. The main issue I'm getting is
3 errors detected in the compilation of "src/alpaca_lora_4bit/quant_cuda/quant_cuda_kernel.cu".
130.1 error: command '/usr/local/cuda/bin/nvcc' failed with exit code 1
130.1 [end of output]
130.1
130.1 note: This error originates from a subprocess, and is likely not a problem with pip.
130.1 ERROR: Failed building wheel for alpaca-lora-4bit
130.1 Running setup.py clean for alpaca-lora-4bit
132.6 Failed to build alpaca-lora-4bit
132.6 ERROR: Could not build wheels for alpaca-lora-4bit, which is required to install pyproject.toml-based projects

At first I struggled to pip install monkeypatch. Read that it was a pip3 issue. Tried a different setup.py file, but that didn't work.
Went back to the https://github.com/johnsmith0031/alpaca_lora_4bit/tree/winglian-setup_pip version. Followed all the steps. Failed at "pip install .". I tried it in the /repositories/ folder and in the miniconda environment (installer files). I tried the docker image instructions but the docker file is looking for a version of cuda that doesn't exist on Docker Hub. I tried changing the docker file to the most similar version available on docker hub ( FROM nvidia/cuda:11.7.0-devel-ubuntu22.04 AS builder -> cuda:11.7.1-devel-ubuntu22.04 repeat for all 3 instances of 11.7.0 in the file). I have CUDA 12.x but I also tried installing 11.7, setting Path and CUDA_PATH to the 11.7 folder/bin folder.
I should also say, when I used a different repo that claimed to fix the monkeypatch install problem I got a different error " File "H:\...\modules\training.py", line 275, in do_train from monkeypatch.peft_tuners_lora_monkey_patch import ( ModuleNotFoundError: No module named 'monkeypatch.peft_tuners_lora_monkey_patch'; 'monkeypatch' is not a package". I don't remember if it built the wheels correctly or not, but I remember being happy that it seemed to get past that step. With this error I also tried adding the monkeypatch folder to the path variables and even to one of the webui files so it could find it but nothing worked so I gave up and tried the johnsmith version again.
I do have multiple python environments split on different drives (one is in the default windows location) but I believe they're all added to the path variables properly and nothing is broken with regular webui, it's just this step of the monkeypatch/lora install. I've also tried moving the 11.7 to the top of the path and cuda_path variables but that didn't change the error. I also tried setting system memory or whatever because that was a problem with webui being on a different drive. I don't know if that's related to this at all.
A Reddit comment mentioned precompiled wheels being available, but I can't find them anywhere.
I tried searching the kernal.cu file for "/usr/local/cuda/bin/nvcc" because that isn't the path of nvcc that I added to the path, but that string isn't in that file, and it's also not in setup.py so I have no idea where it could be or if that would even help. I have more/longer error messages but this post is already very long.
I appreciate any help anyone can give. I'm doing my best, but I've never formally learned python. If more information is needed I can try to provided as much as possible. Thank you.

Maybe you can try WSL2 in win10, which works fine.

Thank you for the help. Is the wsl2 just for compiling with the docker image? I got more problems with wsl2, but here’s the other errors I got when trying to build wheels without the docker:
Building wheel for alpaca-lora-4bit (setup.py) ... error error: subprocess-exited-with-error
× python setup.py bdist_wheel did not run successfully.
│ exit code: 1
╰─> [1438 lines of output]
running bdist_wheel
running build
running build_py
running build_ext
V:...\env\lib\site-packages\torch\utils\cpp_extension.py:359:
UserWarning: Error checking compiler version for cl: [WinError 2]
The system cannot find the file specified
warnings.warn(f'Error checking compiler version for {compiler}: {error}')
building 'alpaca_lora_4bit.quant_cuda' extension
creating V:...\repositories\alpaca_lora_4bit\build\temp.win-amd64-cpython-310
creating V:...\repositories\alpaca_lora_4bit\build\temp.win-amd64-cpython-310\Release
…
[2/2] V:...\env\bin\nvcc --generate-dependencies-with-compile --dependency-output
V:...\repositories\alpaca_lora_4bit\build\temp.win-amd64-cpython-310\Release\src/alpaca_lora_4bit/quant_cuda/quant_cuda_kernel.obj.d --use-local-env -Xcompiler /MD -Xcompiler /wd4819 -Xcompiler /wd4251 -Xcompiler /wd4244 -Xcompiler /wd4267 -Xcompiler /wd4275 -Xcompiler /wd4018 -Xcompiler /wd4190 -Xcompiler /EHsc
-Xcudafe
--diag_suppress=base_class_has_different_dll_interface -Xcudafe --diag_suppress=field_without_dll_interface
-Xcudafe
…
3 errors detected in the compilation of "V:/.../repositories/alpaca_lora_4bit/src/alpaca_lora_4bit/quant_cuda/quant_cuda_kernel.cu". quant_cuda_kernel.cu
ninja: build stopped: subcommand failed.
Traceback (most recent call last):
File "V:...\env\lib\site-packages\torch\utils\cpp_extension.py", line 1893, in _run_ninja_build subprocess.run(
File "V:...\env\lib\subprocess.py", line 526, in run
raise CalledProcessError(retcode, process.args,
subprocess.CalledProcessError: Command '['ninja', '-v']'
returned non-zero exit status 1.
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
File "", line 2, in
File "", line 34, in
File "V:...\repositories\alpaca_lora_4bit\setup.py", line 19, in
setup(
File "V:...\env\lib\site-packages\setuptools_init_.py", line 107, in setup
return distutils.core.setup(**attrs)
File "V:...\env\lib\site-packages\setuptools_distutils\core.py", line 185, in setup
…
File "V:...\env\lib\site-packages\torch\utils\cpp_extension.py", line 1909, in _run_ninja_build
raise RuntimeError(message) from e
RuntimeError: Error compiling objects for extension
[end of output]
note: This error originates from a subprocess, and is likely not a problem with pip.
ERROR: Failed building wheel for alpaca-lora-4bit Running
setup.py clean for alpaca-lora-4bit
Failed to build alpaca-lora-4bit
ERROR: Could not build wheels for alpaca-lora-4bit, which is required to install pyproject.toml-based projects
(V:...\env) V:...\repositories\alpaca_lora_4bit>
I’m confused because it just seems to work for everyone else and nothing I try seems to work. I can try to get wsl2 working but that will require more troubleshooting.

from alpaca_lora_4bit.monkeypatch.... import

Where do I get the correct finetune.py file? When I checked the file I have it seems to be the exact same as the one listed on the main page here on github.
After researching more about the errors I felt like it was an issue with pytorch and cuda, so I created a new environment to install pytorch and cuda, activated it using "conda activate", changed to the alpaca directory and tried setup again.
Then I got an error saying ninja was missing so I pip installed that.
Then I got an error about C++ so I reinstalled the desktop C++ checking all the boxes on visual studio, added the folder to PATH system variable, then reactivated the env and tried setup again. Now the errors are harder to understand so I don't even know what to post.
I believe it could be pytorch compiled with 11.8 and cuda 11.7 was installed (I don't know how that's possible, maybe I need to add the venv cuda path as well?) Doing these steps fixed a different install for the tts webui so I was hoping it would fix this too, but I'm still lost. Maybe I could activate the textgen environment to reinstall ninja and the rest? Are there specific cuda, python, pytorch, c++, etc. versions that are necessary to build the wheels? I feel like these would be installed during setup but maybe my path variables are mismatching what the setup is looking for I really don't know.
I haven't tried the docker again because if there are more errors I know even less about *nix systems so I will probably be even worse off. If that's my only option I'll do try it, but I feel like

The file is here:
https://github.com/johnsmith0031/alpaca_lora_4bit/blob/winglian-setup_pip/finetune.py
And I did not have those compilation issue on windows. Not sure what is causing your problem, but I believe using WSL2 would work.

I got miniconda3 and all the requirements set up in wsl2, but it's still not working. I notice attomicadd() is in there. My gpu can compute up to 8.6 which according to nvidia means I can't compute "atomic addition operating on float2 and float4 floating point vectors in global memory" no clue if this is related but here's a short description of the error I got in wsl2:
`home/.../lora/alpaca_lora_4bit/src/alpaca_lora_4bit/quant_cuda/quant_cuda_kernel.cu(974): error: no instance of overloaded function "atomicAdd" matches the argument list
argument types are: (__half *, c10::Half)
detected during instantiation of "void VecQuant4MatMulKernelFaster(const half2 *, const int *, scalar_t *, const scalar_t *, const int *, const int *, int, int, int, int, int) [with scalar_t=c10::Half]"
(895): here
  /home/.../lora/alpaca_lora_4bit/src/alpaca_lora_4bit/quant_cuda/quant_cuda_kernel.cu(1301): error: no instance of overloaded function "atomicAdd" matches the argument list
              argument types are: (__half *, c10::Half)
            detected during instantiation of "void VecQuant4MatMulV1KernelFaster(const half2 *, const int *, scalar_t *, const scalar_t *, const scalar_t *, int, int, int, int) [with scalar_t=c10::Half]"
  (1323): here

  /home/.../lora/alpaca_lora_4bit/src/alpaca_lora_4bit/quant_cuda/quant_cuda_kernel.cu(1566): error: no instance of overloaded function "atomicAdd" matches the argument list
              argument types are: (__half *, c10::Half)
            detected during instantiation of "void VecQuant4MatMulKernel_G(const half2 *, const int *, scalar_t *, const scalar_t *, const int *, const int *, int, int, int, int, int) [with scalar_t=c10::Half]"
  (1473): here

  3 errors detected in the compilation of "/home/.../lora/alpaca_lora_4bit`

it seems to be related to line 974 atomicAdd(&mul2[b * width + w], res);
This was a proposed fix
__device__ __forceinline__ void atomicAdd(__half* address, __half val) {     unsigned int *address_as_ui = reinterpret_cast<unsigned int *>(reinterpret_cast<char *>(address) - (reinterpret_cast<size_t>(address) & 2));     unsigned int old = *address_as_ui;     unsigned int assumed, new_val;     do {         assumed = old;         new_val = (assumed & 0xffff0000) | (__half_as_ushort(val) + (assumed & 0xffff));         old = atomicCAS(address_as_ui, assumed, new_val);     } while (assumed != old); }
I do have a second gpu in my system, but it's not gpu0. I can try taking that out and running it again as well. Any thoughts?

It's the compatibility issue with old video card and driver. You can remove VecQuant4MatMulKernel_G function in quant_cuda.cpp and quant_cuda_kernel.cu and then try again. This function is only used for inference.

After removing the old Pascal card wheels successfully build/installed on both wsl2 and win10! Thank you very much for your help. I haven't tested the changes you suggested with the old gpu in the system, would that be helpful to confirm your suggested changes?
I only had the old card in the system for extra vram but idek if it was helping with any of the applications.

I think if you don't need additional vram, it's actually enough for now.


ci(build-test-tidy): introduce success condition jobs

Description
I had initially made this change a part of the previous PR:

https://github.com/autowarefoundation/autoware.universe/pull/9709

But we removed it to allow a simpler structure.
But I think we cannot avoid it due to how we add things to Status checks that are required from rulesets menu.
Problem
Issue is, you add the required "jobs" to this list.

You cannot add workflows here, only jobs. And if jobs are within jobs as in reusable workflows, you need to include both names.
For example, for build-and-test-differential job, here is the list of allowed jobs currently available in this repo:

Add build-and-test-differential to list
❌ We end up with:

In a workflow. If we check more closely:

It actually needs build-and-test-differential / build-and-test-differential to complete.
Add build-and-test-differential / build-and-test-differential to list

It is solved ✅
This was to demonstrate how it works. But now I will share the problem part.
Add build-and-test-differential-cuda / build-and-test-differential
Success

It works if it is not skipped ✅
Skipping
❌ We end up with:

This happened because it was skipped. And sub-workflow did not execute at all.

Add build-and-test-differential-cuda to the list
Success
❌ We end up with:

Because the full job name is not in the list.
Skipping

It works if it is skipped ✅
Solution
This means, we cannot rely on skipping counted as success with reusable workflows.
So I'm bringing back these jobs that will run at all times and notify you about the success condition.
Related links
Tested PRs:

https://github.com/autowarefoundation/autoware.universe/pull/9766 (non-cuda change)
https://github.com/autowarefoundation/autoware.universe/pull/9768 (cuda change)

How was this PR tested?
Notes for reviewers
None.
Interface changes
None.

Effects on system behavior
None.

I improve the readability of the results.

Tests
CUDA Package Modification
🟢 Good Change
🔴 Bad Change

https://github.com/autowarefoundation/autoware.universe/actions/runs/12483758294 ✅

Normal Package Modification

@xmfcx
Thanks for the fix. I was curious so I took a little look at this.
It seems like another workaround is to just pass the if condition to the reusable workflow as an argument and skip the job inside the reusable workflow. This way, you will have the results from the sub-workflow even if it is skipped, and the required status check won't stop at "expected". (I've created a test PR on my fork here https://github.com/mitsudome-r/autoware.universe/pull/10 if you want to take a look)
This keeps the workflow code simpler, and developers would have similar user experience to check failed conditions as before. However, it is a bit of hacky way to solve it so I'm okay with your solution as well.
Maybe we can wait for developers reaction, and if they feel uncomfortable with the current fix, we can consider using this approach.

@mitsudome-r In my opinion, counting skipping as a success condition hides intention and confuses even more. It's better to make things explicit than implicit. For example, cuda workflows are marked as required but they can be skipped and it just passes, this, to me is confusing. But if this build-test-pr is marked as required, we explicitly report our success condition to developers.
Right now, if the workflow fails, they can click on the failed workflow and they will be pointed to where to look.
I've checked your solution and yes, it would solve this problem too. And with much less code.
Still, I feel it hides the success condition by allowing skipped items counted as suceeded implicitly.
If you really want to keep it the old way with the small fix, I understand too, we can apply your solution. 👍

By the way, the reason we have to deal with these complicated success conditions is that we started running CUDA jobs conditionally. Due to this, clang-tidy also branched into 2. It is the most efficient way to use limited compute resources. I think it's unavoidable to have this complicated structure as long as we aim to support cuda and non-CUDA builds in a large codebase that has large parts that explicitly doesn't require CUDA.

@xmfcx Thanks for your comment.
I agree that GitHub's specification of treating skipped workflow as success condition is confusing.
I just created the PR to the upstream just in case if we want to apply the modification: https://github.com/autowarefoundation/autoware.universe/pull/9771
Maybe we can come back and discuss this in the next Software WG call.


Move renovate config to the renovate repo

Move renovate config to the renovate repo
Renovate config for MintMaker will now be in a different repo. Changing
which version of the configration should be deployed can be done by
changing the git ref of the config.
Related PR: https://github.com/konflux-ci/mintmaker/pull/32
Refers to CWFHEALTH-3206

@qixiang @nvtnlucie @staticf0x @gnaponie please review

@querti could you update the commit message? the mintmaker-config repo is referenced.

@querti could you update the commit message? the mintmaker-config repo is referenced.

Fixed, thanks

/approve
/lgtm


fix: Wrong console command for adding Function-Call key with any methods to account

Resolves #327


Agile process template projects - 'Blocked' field error

The Agile process template within Azure DevOps does not use the 'Blocked' field and it does not exist in the table. This means any tables that use this field will fail on data-load/refresh.

Fixed in latest version 💯


Fix link pasting

Pasting link within text moved selection to its end:
https://user-images.githubusercontent.com/132642/187054081-5adf11b1-6221-4970-a43b-afed5de53d03.mov

Hi @fantactuka,  I think there is a typo in your test commit!
"Can insert inline element --> withing <-- text and put selection after it"

Hi @fantactuka, I think there is a typo in your test commit!
"Can insert inline element --> withing <-- text and put selection after it"

I'm gonna go ahead and merge this and we can make a separate PR to clean up the typo.


v0.2.1

Codecov Report

Merging #2 (7e75789) into main (d6a4063) will increase coverage by 2.06%.
The diff coverage is 88.37%.

@@            Coverage Diff             @@
##             main       #2      +/-   ##
==========================================
+ Coverage   19.45%   21.52%   +2.06%
==========================================
  Files          16       16
  Lines        1763     1863     +100
==========================================
+ Hits          343      401      +58
- Misses       1420     1462      +42

Impacted Files
Coverage Δ

examples/render_loop.rs
0.00% <ø> (ø)

src/interface.rs
0.00% <ø> (-37.17%)
:arrow_down:

src/layout/key_name.rs
0.83% <ø> (ø)

src/layout/keypress.rs
1.63% <ø> (ø)

src/layout/layout_fr_ch.rs
3.83% <ø> (+0.01%)
:arrow_up:

src/layout/position.rs
90.00% <ø> (+9.40%)
:arrow_up:

src/test.rs
100.00% <ø> (ø)

src/color/base.rs
54.04% <86.27%> (+18.58%)
:arrow_up:

src/keyboard.rs
5.88% <90.00%> (-10.79%)
:arrow_down:

src/color/buffer.rs
64.70% <100.00%> (+6.64%)
:arrow_up:

... and 7 more

Continue to review full report at Codecov.

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


Prevent deprecated callbacks from accidentally calling them

One can freely call after_build, after_create, before_create, causing weird issues to happen.
I occasionally type after_build instead of after(:build) by pure mistake, not for legacy reasons.
Make these methods protected/whatever? Hypothetical legacy users could just call them explicitly (maybe with send...).
Thanks!
Victor

@vemv any ideas for approaches here? Given how FG's internals handle attributes (by capturing everything), technically virtually everything can be written within the definitions themselves. I think going out of the way to call out potentially wrong attributes means maintaining a fairly large list.
Currently, I imagine the failure you're seeing is that it can't call after_create= on the instance; the alternative would be to introduce additional code to deliver a similar (but different) message stating roughly the same thing; because I'm a maintainer, I obviously prefer introducing less code over more code, but I'd love ideas if you have any to improve this.

Currently, I imagine the failure you're seeing is that it can't call after_create= on the instance;

Indeed, that's the case as of today!
Back when I opened this issue, the message/backtrace was much more long/confusing.
Maybe something has changed since Oct 4? Or maybe the unclear backtraces only happen under conditions that I can't reproduce anymore.
Closing!


Feature: add support differen models

Could you please add support models like "Hugging Face" or "Gpt4Free" gpt4free

ShellGPT roles and prompts are optimised only for OpenAI LLMs and they might not perform well for other LLMs.


Missed Implementation of account.setContactSignUpNotification#cff43f61

Hi, I saw that some methods are not available in tdlib like account.setContactSignUpNotification#cff43f61
Is there any special reason that some methods are eliminated?
I understand that some method has changed but has same functionality with different name.
Is there any equivalent to account.setContactSignUpNotification?
Thanks

It is accessible via option "disable_contact_registered_notifications".

Thanks @levlam!


Leaflet for R

I would like to implement leaflets in shiny with R. I try to go with the simples example I found online:
library(leaflet)
library(ggmap)

>somePlace <-ggmap::geocode("Vienna")
Information from URL : http://maps.googleapis.com/maps/api/geocode/json?    address=Vienna&sensor=false
>somePlace
lon      lat
1 16.37382 48.20817
>leaflet(somePlace) %>% addTiles() %>% addMarkers()

It shows a grey space with the blue marker. No trace of OSM or any other spatial visualisation.
Could you point out where is the problem? I do not understad the ERROR message Information from URL : http://maps.googleapis.com/maps/api/geocode/json?    address=Vienna&sensor=false
Thanks in advance!

Hi, glad to hear that you find Leaflet useful!
However, this issue tracker is used for reporting bugs and discussing new features. For questions on using Leaflet (with or without R), please use gis.stackexchange.com or stackoverflow.

Thnx for info!


[Update Snap] MinaPortal

Checklist
All items in the list below needs to be satisfied.

[x] Is the summary of the change documented in this ticket?
[x] Has a MetaMask Snaps team member reviewed whether the changes need to be vetted?
[x] If there are changes that need to be vetted, attach a description and the relevant fixes/remediations to this issue.
[ ] The corresponding pull request in this repo has been merged.

This change includes:

Updating the mina-signer npm package: https://github.com/sotatek-dev/mina-snap/pull/228
Change Square smart contract to AddOne: https://github.com/sotatek-dev/mina-snap/pull/229/commits/a3bda1e623c3009d6b04914518f58ff4279311d6
Change MinaScan text to "Explorer": https://github.com/sotatek-dev/mina-snap/pull/229/commits/bfd2e93e07a073bb517b3ed37ba08e4a1e79c2ef

The new version is 0.1.1 https://www.npmjs.com/package/mina-portal/v/0.1.1


Feature Gates

This PR proposes to start using Feature Gates in the operators to enable or disable functionality.

@tombentley I restructured it a bit. But apart from the one comment where I left some follow-up question, the rest should be incorporated.

Approved by 4 maintainers (+ my implicit vote).


Define standard template scenarios for expected test outcomes (for use in test case descriptions)

about the x.  here is the idea.
x is a set of actions that we want to test. Y is a vector of post conditions for F(x)=Y. we need a template that will take x
and execute it in different context (create tests). For each context i we need to verify that Fi(x)=Yi (transition for set of actions x on a template i makes post Yi)
The contexts are templates (but not limited to):
X is in transaction deployment code (create transaction)
X is in CREATE init code
X is in CREATE2 init code

X is in the contract code (being called by transaction)   and [OOG, REVERT, SELFDESTRUCT, NOTHING] happens
X is in the contract code (being called by CALL) and [OOG, REVERT, SELFDESTRUCT, NOTHING] happens
X is in the contract code (being called by CALLCODE) and [OOG, REVERT, SELFDESTRUCT, NOTHING] happens
X is in the contract code (being called by DELEGATECALL) and [OOG, REVERT, SELFDESTRUCT, NOTHING] happens
X is in the contract code (being called by STATICCALL) and [OOG, REVERT, SELFDESTRUCT, NOTHING] happens

Not just descriptions but actual test coverage.
For example x is a call to a newly defined opcode. Need to check its behavior by a template in all this cases.

Ori did some work in that direction:
https://github.com/ethereum/tests/blob/develop/src/Templates/DiffPlaces/templateGen.js


Doc 31 update documentation

Closes #31.

@ty-desj I tried to recreate the error I got for the code comments, but I wasn't able to.
Could you try to generate the documents and see if you get the errors? Maybe it was just a one off.

@DominicParent Sure thing, what is the command to generate them?

@ty-desj The documentation can be built using the sphinx-build -b html source build from the doc folder.

@ty-desj You'll have to add stuff in your conda environment (sphinx and myst-parser) but I've added those to the environment creation file.


Blank screen on  :  version (6.4.6-200.fc38.x86_64) workstation

i just recently updated my fedora  to version (6.4.6-200.fc38.x86_64) workstation
and when i am booting it up getting blank screen . i don`t  know the cause but its happening when i updated.
now i am using (6.3.8-200.fc38.x86_64) workstation.

This is the repository for https://fedora.lyrasis.org/
We don't have a 6.4.6 release.


Group Tests Together By Class

If I specify multiple --testclass=... options on the command line the test classes contain tests from different source classes.
I would rather have all my tests for a particular class residing in one test class rather than spread out over multiple test classes mixed with other tests.

Each test Daikon creates may invoke methods from arbitrary classes.  For example, a Daikon test might invoke 1 method from class A, then 8 methods from class B, then 1 method from class C.  It's not clear how you would want to segregate the tests or which class you would put the above test in.


named migrations

It would be great to have a migration with an option to pass the migration name

This is not how you suggest new features, the issue template have a reason so please don't ignore it. Something "being great" is not a reason (not talking about it being very subjective) - why would it be great? What problems would it solve? How would you expect the change to be done? Are we talking about file names, or just some metadata?
Migrations need to have names that allow ordering on the file system level, that is the main reason why the name is fixed (and in fact it is not hardcoded anywhere, you can customize it via naming strategy classToMigrationName method).

In case anyone else goes looking for this feature and finds this issue, I figured I would put the solution I wrote that is working quite nicely for my team. IMO I think this is enough, albeit kind of hacky, but doesn't require a change to Mikro-ORM.
We created a custom fileName function that is used to generate file names. During that we use the readline-sync package to ask the user for a short description of the migration. To keep the order preserved we have a 4 digit counter at the beginning of the file. The end result is a file name like 0012-short-description-from-user-input.ts.

Set migrations.fileName to this function in your options object.

/**
 * Asks the user for a short description and increments a counter.
 * Generates file names that follow the following naming structure.
 *
 * `0012-short-description-from-user-input`
 */
const descriptionFileName = () => {
  const DELIMITER = '-';

  // Get the last migration
  const lastFileName = fs.readdirSync(migrationsPath).pop();
  let counter = 0;

  if (lastFileName) {
    // Increment the counter based on the last migration
    counter = parseInt(lastFileName.split(DELIMITER)[0], 10) + 1;
  }

  // Ask user for a short description (keep asking until you get something or they quit with ctrl+c)
  let name = '';
  while (!name) {
    name = readlineSync
      .question("\nWhat does this migration do? (Keep it short, it's going in the file name)\n> ")
      .replace(/[^\w\s]/gi, '') // Remove any non-alphanumeric or whitespace characters
      .replace(/\s+/gi, DELIMITER) // Replace any number of whitespace characters with the delimiter
      .toLowerCase();
  }

  // New line to find question in logs easier
  console.log();

  return `${counter.toString().padStart(4, '0')}-${name}`;
};

now you can do that easily from version 5.7, for those who could be interested here's the PR that I made for this feature: https://github.com/mikro-orm/mikro-orm/pull/4250.
docs https://mikro-orm.io/docs/migrations#using-custom-migration-names.


Show actions on the bottom

In the "all collection" screen when you long press on an item it shows you the actions you can do with it on the top of the basically replacing
With this
app
Instead I would like it if we were able to choose if it should do what it does now or be placed right above this(or even replacing it) menu

I don't want to complexify design related settings, and not everything can be at the bottom.

Oops, I meant to do so. Thanks.


Support HTTP/2

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 100.00%. Comparing base (a6aabf6) to head (5c0f4c7).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff            @@
##              main       #11   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            8         8
  Lines          995       995
  Branches       104       104
=========================================
  Hits           995       995

Flag
Coverage Δ

unittests
100.00% <100.00%> (ø)

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

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Profile - Display Mentor/Mentee Profiles

Requirements for feature

[ ] Implement an API service to handle GET mentor/mentee (fill this to return dummy data until backend is up)
[ ] Setup a model for mentor/mentee that can be used for Profile View and Edit
[ ] Create Profile Display Scene and reuse logic using components etc whenever you see fit

Note - Don't link the "Book Session Via Calendly" button right now. This will be a separate task
Any context for feature

Profile View Display is the screen users can view their own profile or a different user's profile. The View will change depending on who is viewing who's profile.  For example, when a mentee is viewing a mentor's profile they will see a"Book Session Via Calendly" button that they can use to schedule via calendly. In addition, they will not see who the other mentee's are for that mentor. On the other hand, when the mentor is viewing that same profile (self), they should see a view my calendly button + see all their mentees.

If you are confused on some field, please ping on slack and we can address it with our PM and designers.

Resources / tips for development

You should setup a model that can be used in our ViewModel and Model for the Profile Screens. You should be able to reuse the same model for the Edit Profile screen

Closed by #98


Set up basics of music rating page (front-end & backend)

Create a very simple HTML page for the music rating screen a user is greeted with after logging in and pair it with functional PHP scripting / SQL database integration. This will later have a css stylesheet added to it in the future, and will incorperate needed logic.
This includes:
adding, deleting, and viewing rated songs

Closed as mentioned in PR #56


Publisher and subscriber div gets removed on orientation change.

In Opentok plugin version 1.0.3, when I rotate the display (to landscape), I hide the publisher and subscriber divs and call TB.updateViews() method. But soon after that the publisher and subscriber divs gets removed from the DOM. It works correctly in v1.0.2 of the plugin.

We noticed the same issue and the workaround was fairly straight-forward.   After the JS orientation event, have some code that checks if the publisher and subscriber divs are missing and if so, add them back to the DOM.


Microsoft Store version broken since <IP_ADDRESS>

Description
Once again it seems the Microsoft Store release was not tested?
Anyway, I just installed the update and now DBeaver CE won't open anymore. It probably does something but fails silently. No error message or anything. Where do I find the logs?
Now I can't do anything until it's fixed 😢 No going back to old version possible.
DBeaver Version
Community Edition <IP_ADDRESS>
Operating System
Windows 10 22H2 19045.3930
Database and driver
No response
Steps to reproduce

Install DBeaver CE from Microsoft Store
Try to start DBeaver CE

Additional context
No response

As a workaround for now - you can download DBeaver here https://dbeaver.io/download/

Thx. Unfortunately, this doesn't restore my session :/

Thank you for the report. It is a known issue, we are working on it. We also want to update DBeaver version in Store to the latest one.
Note: Microsoft Store app is broken because apparently it is not possible to test application before it was published. It passes all verifications and certifications but eventually doesn't work. So we have to do some code changes (blindly), publish it again and retest. It takes a lot of time :(

Can you please revert to the previous version for the time being?

Same for me. <IP_ADDRESS> still falls silently. Where can I find the logfile to shed some light on the issue? It seems it's failing before DBeaver core is even started as the last log I can find is from before the last working update.

DBeaver CE was updated in MS Store.
Please check

Third update (for me) today seems to work again.
Version <IP_ADDRESS>401100101 once again opens.

Verified

What was the issue after all?


[FEATURE] Dump and Load ES scripts

Is your feature request related to a problem? Please describe.
There is the need to dump and load also ES scripts.
The scripts can be used into ES indexes.
Describe the solution you'd like
I would to add the missing type into the dump and load functionalities.
Describe alternatives you've considered
No other alternatives.
Give some examples of implementations
elasticdump
--input=http://production.es.com:9200/my_index
--output=http://staging.es.com:9200/my_index
--type=script
Additional context
Add any other context or screenshots about the feature request here.

#908


Exercise 5.7_c possible error.

if (int ival = get_value())
    cout << "ival = " << ival << endl;
if (!ival)
    cout << "ival = 0\n";

What you wrote is probably also true, but the more glaring error to me is, that ival is defined inside the "if" condition. After the first if ends with 'cout'ing the string, the variable will lose it's scope and the second if cannot evaluate it's condition, or am I mistaken? (Same if you change it to an else/if.
So the correct code should be:
int ival = get_value();
if (ival)
   cout << "ival = " << ival << endl;
else
   cout << "ival = 0" << endl;

ig the correct code should be
int ival = get_value();
if(ival){
cout<<"ival= "<<ival<<endl;
}
else{
cout<<"ival= '<<ival<<endl;
}


hello! could you provide the contributions test data in your paper?  thank you very much!

❓ Questions and Help
Before asking:

search the issues.
search the docs.

What is your question?
Code

What have you tried?
What's your environment?

fairseq Version (e.g., 1.0 or master):
PyTorch Version (e.g., 1.0)
OS (e.g., Linux):
How you installed fairseq (pip, source):
Build command you used (if compiling from source):
Python version:
CUDA/cuDNN version:
GPU models and configuration:
Any other relevant information:


When editing an existing post, errors in the slug do not appear but you get "can not save the changes now"

Summary
So when you are making changes to an existing draft, mistakes in the "slug" prevent saving he changes, and the user get a vague message.
Steps to Reproduce

Make a draft, and set a slug to it.
Save the changes.
Change the "slug" to something invalid, like: "2018" (not allowed as it starts with a number).
Try saving the changes
You will get "Your changes could not be saved at this time" instead of the more specific: "Slug must start with a letter ... etc".

Additional info

Postleaf version: 1.0
Node version: 7.10.1
Affected browsers: Chrome
Operating system: Debian 8 (host) vs. Windows (Browser)

Note: This issue tracker is ONLY for bug reports and feature requests. If this is a personal support issue, please visit postleaf.org/support.

Confirmed. It looks like we're not handling the validation error from the post model.
I'll leave this open until I can get to it, or feel free to submit a PR instead :)


Slack Invite Link on the Website is no Longer Working

The link https://slack.labelstud.io/?source=site results in an error that the invite link is no longer valid.

Hi, thanks for report.
We updated the link.