run tests and benchmarks on m1 mac

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

Comparison is base (f398036) 62.99% compared to head (8fc245e) 62.99%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #500   +/-   ##
=======================================
  Coverage   62.99%   62.99%
=======================================
  Files          61       61
  Lines        4251     4251
  Branches      725      719    -6
=======================================
  Hits         2678     2678
- Misses       1424     1428    +4
+ Partials      149      145    -4

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


Add Python 3.11 support

By mistake I closed #1780 but here is the rebase with main and without _PIP_USE_IMPORTLIB_METADATA

great thanks !

Thank you for your quick response @trbs


Ran out of frames during seek.

hi  @dukebw
when i use lintel to extact frames from video ,it always arise  Ran out of frames during seek.
What does this message mean and whether an error has occurred! and after reading some video ,the program exited abnormally as follows:
Process finished with exit code 139 (interrupted by signal 11: SIGSEGV)

Met the same issue. And there is another kind of issue: "Ran out of frames. Looping". However, it does not currupt but gives a well performance on dataset UCF-101 (I test it with this)

This library is life saver dude, our cluster has a really bad file i/o right now, so I cached the binary videos in memory & used lintel to decompress on the fly, I commented on the print statement "Ran out of frames during seek." in C code that worked suppress the constant the error message. No effect on decompressed videos, verified by plotting. Brought down the training time from 1-2 weeks to 1 day


chore: fix merge vault

En cas de merge conflict affiche les conflict dans le vault file

:tada: This PR is included in version 1.47.2 :tada:
The release is available on:

v1.47.2
GitHub release

Your semantic-release bot :package::rocket:


DBZ-4878 Re-read incremental snapshot chunk on DDL event

A new schema can be applied to the chunk with an old schema if DDL event is within the chunk select window.

The chunk's JDBC schema in this case is the same as before. Schema change won't be detected. The binlog schema will get updated with the DDL event. The chunk is inserted after window close event. The events mapping uses the latest schema for the chunk.
The fix is simple: re-read chunk on DDL event. In this case chunk will have schema that is up to date with the binlog schema.

@kgalieva It seems that Db2 and Oracle connectors needs to be updated as well.

@kgalieva The code LGTM, unfortunately testing it would be quite difficult - but not impossible.
I think we can merge it when the project becomes buildable. Thanks a lot!

@kgalieva Applied, thanks a lot!


Hard for user to tell how a given untranslatable value was reached

Try running the test
def test_typeWeCantTranslateYet(self):
    with self.create_executor() as fora:
        x = type(lambda x:x)
        with fora.remotely:
            y = x

The exception says
PythonToForaConversionError: can't convert <type 'function'> (of type <type 'type'>) since we can't get its source code

But we cant tell what we're operating on.

as another example,
or example, if we haven't implemented a wrapper for scipy.special.beta, we'll get a stacktrace
Traceback (most recent call last):
File "Bla.py", line 621, in
...
File "/home/tom/volumes/src/packages/python/pyfora/Executor.py", line 179, in submit
futures = [self.define(fn)] + [self.define(arg) for arg in args]
File "/home/tom/volumes/src/packages/python/pyfora/Executor.py", line 143, in define
).walkPyObject(obj)
File "/home/tom/volumes/src/packages/python/pyfora/PyObjectWalker.py", line 219, in walkPyObject
self._registerFunction(objectId, pyObject)
File "/home/tom/volumes/src/packages/python/pyfora/PyObjectWalker.py", line 472, in _registerFunction
classOrFunction=_FunctionDefinition
File "/home/tom/volumes/src/packages/python/pyfora/PyObjectWalker.py", line 556, in _classOrFunctionDefinition
self.walkPyObject(resolution)
File "/home/tom/volumes/src/packages/python/pyfora/PyObjectWalker.py", line 219, in walkPyObject
self._registerFunction(objectId, pyObject)
File "/home/tom/volumes/src/packages/python/pyfora/PyObjectWalker.py", line 472, in _registerFunction
classOrFunction=_FunctionDefinition
File "/home/tom/volumes/src/packages/python/pyfora/PyObjectWalker.py", line 556, in _classOrFunctionDefinition
self.walkPyObject(resolution)
File "/home/tom/volumes/src/packages/python/pyfora/PyObjectWalker.py", line 219, in walkPyObject
self._registerFunction(objectId, pyObject)
File "/home/tom/volumes/src/packages/python/pyfora/PyObjectWalker.py", line 472, in _registerFunction
classOrFunction=_FunctionDefinition
File "/home/tom/volumes/src/packages/python/pyfora/PyObjectWalker.py", line 556, in _classOrFunctionDefinition
self.walkPyObject(resolution)
File "/home/tom/volumes/src/packages/python/pyfora/PyObjectWalker.py", line 225, in walkPyObject
self._registerClassInstance(objectId, pyObject)
File "/home/tom/volumes/src/packages/python/pyfora/PyObjectWalker.py", line 363, in _registerClassInstance
classId = self.walkPyObject(classObject)
File "/home/tom/volumes/src/packages/python/pyfora/PyObjectWalker.py", line 221, in walkPyObject
self._registerClass(objectId, pyObject)
File "/home/tom/volumes/src/packages/python/pyfora/PyObjectWalker.py", line 491, in _registerClass
classOrFunction=_ClassDefinition
File "/home/tom/volumes/src/packages/python/pyfora/PyObjectWalker.py", line 534, in _classOrFunctionDefinition
_, sourceLine = PyforaInspect.getsourcelines(pyObject)
File "/home/tom/volumes/src/packages/python/pyfora/PyforaInspect.py", line 258, in getsourcelines
lines, lnum = findsource(pyObject)
File "/home/tom/volumes/src/packages/python/pyfora/PyforaInspect.py", line 174, in findsource
raise IOError('could not find class definition')
IOError: could not find class definition

a similar thing will happen if you try to convert an object which holds something unconvertable, for example an Executor.
two approaches @braxtonmckee mentions:

in our recursive walkPyObject function, we could insert debug info on the stack, or
use exceptions and handle in stack unwinding using try/except

another option @braxtonmckee mentions is to introduce a "nonconvertible" symbol into the generated fora code, kind of like the pyforaUninitializedVariable thing. this moves the error to the runtime, which can then generate a stacktrace

This is in line with the way regular python programs behave: if you have a
variable that's undefined, you get an exception when you execute the code
that needs the variable. If that pathway is not touched by your program,
then you don't get an error. Pyfora should behave the same way: it's not an
error to reference untranslatable code if you don't actually execute it. In
particular, if you have a class which has methods with contain
untranslatable code, but you don't call those methods within code executing
in the Pyfora runtime, that should be OK.
On Thu, Jan 28, 2016 at 11:28 AM, Thomas Peters<EMAIL_ADDRESS>wrote:

another option @braxtonmckee https://github.com/braxtonmckee mentions
is to introduce a "nonconvertible" symbol into the generated fora code,
kind of like the pyforaUninitializedVariable thing. this moves the error to
the runtime, which can then generate a stacktrace
—
Reply to this email directly or view it on GitHub
https://github.com/ufora/ufora/issues/78#issuecomment-176263714.

--
Braxton McKee                             CEO / Ufora
www.twitter.com/braxtonmckee www.ufora.com

I've completed a first pass at this. There are a few related work-items to do, but I'll add issues for them as  I go.


Enable hover and autocomplete docs on macro generated items

Enable hover and autocomplete docs on macro generated items. This de-sugars doc comments into doc attributes in some cases, but not all. Comments and doc attributes are then merged together.
This PR is essentially a partial implementation of what's being suggested #3182, but it's not all the way there yet. ~I still need to add unit tests~, but I wanted to first get feedback on whether or not this was an acceptable path forward.
Fixes #4564
Fixes #3984
Fixes #3180
Related #3182

This is a much more elegant solution then then one I came up with!

Tests have been added. This should be ready for review now.

Why do we need to duplicate these two functions in two modules exactly?

I didn't originally see a common place to put those functions and I didn't think modifying the dependency tree was a good idea.
After closer inspection I noticed the comment below. I've removed those functions from ra_ide and exposed the same functionality via hir::Documentation::from_ast.
https://github.com/rust-analyzer/rust-analyzer/blob/992e1256d05d1ec046e284e597b2932e50ccff49/crates/ra_ide/Cargo.toml#L33-L35

Ah, you need to re-generate feature docs (cargo xtask codegen). We probably should move this to the release process...
bors d+

@matklad When I run cargo xtask codegen there are no changes. I'll rebase and try again but it's odd that CI is picking up something different. Are the /home/runner/work/rust-analyzer/rust-analyzer/docs/user/generated_features.adoc docs cached from a previous run?

I'll rebase and try again but it's odd that CI is picking up something different.

I think rebase might be the case here, I think auto merges might cause the dirft in line numbers.

Are the /home/runner/work/rust-analyzer/rust-analyzer/docs/user/generated_features.adoc docs cached from a previous run?

yup, we just commit generated code. It works great for syntax trees, but for docs I think we should just not do this, will try to fix later today.

bors r+


[CI] Add load-testing

Objective
We need to add optional load testing to DevNet environments.
Origin Document

https://discord.com/channels/824324475256438814/1138895490331705354/1256614409975894028
https://discord.com/channels/824324475256438814/1138895490331705354/1266483844618518538

Goals

Ensure the stress code remains functional after architectural changes (such as depinject module additions/modifications)
Investigate potential data points obtainable from running stress tests on DevNet as part of CI

Deliverables

[ ] CI modification to execute stress tests when a relevant GitHub label is attached to the PR

Non-goals / Non-deliverables

Regular stress tests for each PR; we aim to run them conditionally
Gain visibility into how the suite reacts to different code changes.

This may not be practical to implement with the current DevNet architecture, and more historical data is needed to determine the next steps for implementation. This can be addressed later once sufficient historical data is gathered.

General deliverables

[ ] Comments: Add/update TODOs and comments alongside the source code so it is easier to follow.
[ ] Testing: Add new tests (unit and/or E2E) to the test suite.
[ ] Makefile: Add new targets to the Makefile to make the new functionality easier to use.
[ ] Documentation: Update architectural or development READMEs; use mermaid diagrams where appropriate.

Creator: @okdas

I don't think we're going to add this to CI in the following iterations. I moved the ticket to look into adding load test to CI in November.


atom editor autocomplete

i am not getting auto complete suggestions in atom editor, i have installed all packages but still im not getting auto complete

This package is no longer included in Atom.  For most questions about Atom there are a lot of options, check out:

The Atom Flight Manual
The Atom FAQ on Discuss
Discuss, the official Atom and Electron message board
Join the Atom Slack team

On Discuss and in the Atom Slack team, there are a bunch of helpful community members that should be willing to point you in the right direction.


SlackRTMClient couldn't parse a message_changed message

Hi.
when you respond to a button callback with
{
  "response_type": "ephemeral",
  "replace_original": true,
  "text": "Recieved the Test Callback."
}

it sends the following message
{
  "type":"message",
  "message":{
    "text":"Recieved the Test Callback.",
    "username":"CartBot",
    "bot_id":"xxxxx",
    "type":"message",
    "subtype":"bot_message",
    "ts":"1482868318.000021"
    },
    "subtype":"message_changed",
    "hidden":true,
    "channel":"xxxxxx",
    "previous_message":{
      "text":"Test","username":"CartBot",
      "bot_id":"B3J1YCD0A",
      "attachments":[
        {
          "callback_id":"code-123456",
          "fallback":"backup message: code-123456",
          "text":"Do you want to accept?","id":1,"actions":[
            {"id":"1","name":"accept","text":"Accept","type":"button","value":"","style":"primary"}
            ]
        }
        ],
      "type":"message","subtype":"bot_message","ts":"1482868318.000021"
    },
  "event_ts":"1482868321.886281",
  "ts":"1482868321.000022"
}

Based on the payload you provided, I'm guessing it's failing to parse because we're expecting the user property to be on the message and previous_message objects. Inexplicably, it appears that slack is attaching a  username field instead. This is rather unusual, because no other payload uses username and the docs for the message_changed event show that ````user``` should be coming back instead.
It will take some time to set up a button integration to test this behavior, but once I do, I will try to reproduce this behavior and account for the inconsistency.

Is it still the case?

honestly I don't know.. the code was for a POC, and I'm not sure where it is anymore.. feel free to close if you can't reproduce

i  still get these errors


emojis as a rst transform

This patch allow a much simpler way to uses emojis. Instead a verbose role, it's use an standard substitution |emoji_slug| if that isn't defined in the document and match one of the available emojis

getnikola/nikola#1647 (required for this) has been merged with some changes: site.rst_transforms is used and is guaranteed to exist (no need to do the check).  This plugin will also need to be moved to v7 with MinVersion=7.3.2.


Restructure message event content types

There's some "required if" info in the description for the various message types, as well as some fields that are copy-pasted between the different message types a lot. These types should be updated to better capture the valid event contents.

Done in 16b9557.


Make test class ConfigurationTest available in console

Move the class defined in spec/run_time_settings_spec.rb into its own file so that it can be loaded into the console and used there. That would allow developers to interact with settings as well as try out new declarations.

Completed with Enable interaction with test model at console.


Isolated network can not define cidr

Problem file: network.js
In the 744 lines of code there is duplicate access to the isolated guest network gateway, the subnet settings are not getting correctly, causing the api to fail to get the subnet mask setting of the isolated guest network. Will be overridden by the zone's default subnet mask setting.

@ganlei333 how do you want this . Any 4.11 commit should be merged forward anyway. I don't think this should happen with a PR

@ganlei333 did you open a PR from 4.11 branch, your description does not match the diff


Resolves STREAMS-416

looks good to me.  it would be great to restore some of these features at a higher level of code quality and test coverage someday but for now i'd just as soon remove everything that isn't known to work as advertised. 👍


Switch to WithoutConversionCodecFactory to support k8s 1.16

With this change istio/istio can upgrade properly

@howardjohn: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

gencheck_operator
5df2f3a7f426cab55e388b1cff189eb26308c1ba
link
/test gencheck_operator

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


"ErrorPasswordChangeNotAllowed": "You are not allowed to change your password. Please contact your system administrator."

PassCore Server

OS: [Windows]

Provider: [Active Directory]

Settings file (without sensitive information):
{
"Logging": {
"IncludeScopes": false,
"LogLevel": {
"Default": "Debug",
"System": "Information",
"Microsoft": "Information"
}
},
"WebSettings": {
"EnableHttpsRedirect": true
},
"AppSettings": {
// The following options for AD Provider (remove if you don't use this Provider)
"UseAutomaticContext": true, // Set true to allow PassCore to reset password using the same credentials, or false if you will fill the credentials below
"RestrictedADGroups": [
"Administrators",
"Domain Admins",
"Enterprise Admins"
], // Set the AD groups to restrict the use of PassCore
"AllowedADGroups": ["Domain Users"], // Set the AD Groups to allow PassCore, if the array is empty all the groups no-restricted above are allowed
"IdTypeForUser": "UPN", // Possible values are "DN", "GUID", "Name", "SAM", "SID" and "UPN" (Default UPN)
"UpdateLastPassword": false, // Set true to allow PassCore to  update the last password timestamp
// The following options are for LDAP Provider (remove if you don't use this Provider)
"LdapSearchBase": "ou=people,dc=example,dc=com",
"LdapSecureSocketLayer": false, // Default for AD is true when using LDAPS 636
"LdapStartTls": false, // Default for AD is true when using LDAP 389
"LdapChangePasswordWithDelAdd": true,
"LdapSearchFilter": "(sAMAccountName={Username})", // Another value: "(&(objectClass=person)(cn={Username}))"
// General options (valid for both providers)
"LdapHostnames": [ "" ], // Set your hostname(s)
"LdapPort": 389, // Default for AD is 389, for LDAPS 636
"LdapUsername": "", // Set the username or distinguish name (DN) to bind the LDAP server
"LdapPassword": "", // Set the password for the username
"DefaultDomain": "" // Set your default AD domain here, or non "@" logins will not work! Use empty value to allow user to set the domain. This option is ONLY available with UPN.
},
"ClientSettings": {
"ValidationRegex": {
"EmailRegex": "^[a-zA-Z0-9.!#$%&’+/=?^_`{|}~-]+@[a-zA-Z0-9-]+(?:\.[a-zA-Z0-9-]+)$",
"UsernameRegex": "^[a-zA-Z0-9._-]{3,20}$"
},
"UsePasswordGeneration": false, //Set true to let PassCore create a new password for the current account. If true the user can not customize its new password.
"MinimumDistance": 0, //The minimum distance beetween the old and the new password, this is used to enforce the edit distance using the levenshtein distance algorithm.
"PasswordEntropy": 16, // the number of bytes of entropy to use for generated passwords
"ShowPasswordMeter": true,
"MinimumScore": 0, //The minimum acceptable score that the user's new password needs to get at being evaluated by ZXCVBN to be established as the new password.
"Recaptcha": {
"SiteKey": "", // ReCAPTCHA public key: replace this! or leave empty if you don't need ReCAPTCHA
"PrivateKey": "", // ReCAPTCHA private key: replace this! or leave empty if you don't need ReCAPTCHA
"LanguageCode": "en"
},
"UseEmail": "true",
"ApplicationTitle": "Change Account Password | Self-Service Account Management Tools",
"ChangePasswordTitle": "Change Account Password",
"ChangePasswordForm": {
"HelpText": "If you are having trouble with this tool, please contact IT Support",
"UsernameLabel": "Username",
"UsernameHelpblock": "Your organization's email address",
"UsernameDefaultDomainHelperBlock": "Your organization's username",
"CurrentPasswordLabel": "Current Password",
"CurrentPasswordHelpblock": "Enter your current password",
"NewPasswordLabel": "New Password",
"NewPasswordHelpblock": "Enter a strong password. You can use this tool to help you create one; use the XKCD (random sep, pad digit), or NTLM, options.",
"NewPasswordVerifyLabel": "Re-enter New Password",
"NewPasswordVerifyHelpblock": "Enter your new password again",
"ChangePasswordButtonLabel": "Change Password"
},
"ErrorsPasswordForm": {
"FieldRequired": "This field is required",
"UsernamePattern": "Please enter a valid username",
"UsernameEmailPattern": "Please enter a valid email address",
"PasswordMatch": "Passwords do not match"
},
"Alerts": {
"SuccessAlertTitle": "You have changed your password successfully.",
"SuccessAlertBody": "Please note it may take a few hours for your new password to reach all domain controllers.",
"ErrorPasswordChangeNotAllowed": "You are not allowed to change your password. Please contact your system administrator.",
"ErrorInvalidCredentials": "You need to provide the correct current password.",
"ErrorInvalidDomain": "You have supplied an invalid domain to logon to.",
"ErrorInvalidUser": "We could not find your user account.",
"ErrorCaptcha": "Could not verify you are not a robot.",
"ErrorFieldRequired": "Fulfill all the fields.",
"ErrorFieldMismatch": "The passwords do not match.",
"ErrorComplexPassword": "Failed due to password complex policies: New password length is shorter than AD minimum password length",
"ErrorConnectionLdap": "Unhandled error connecting to the LDAP server.",
"ErrorScorePassword": "The password you are trying to set is not secure enough.",
"ErrorDistancePassword": "The password you are trying to set is not diferent enough of your last password.",
"ErrorPwnedPassword": "The password you are trying to use is publicly known and can be used in dictionary attacks."
}
}
}

Log file (without sensitive information):

[Microsoft.AspNetCore.Mvc.IActionResult] Post(Unosquare.PassCore.Web.Models.ChangePasswordModel) on controller Unosquare.PassCore.Web.Controllers.PasswordController (Unosquare.PassCore.Web).
warn: Unosquare.PassCore.PasswordProvider.PasswordChangeProvider[0]
  Using AutomaticContext

Describe the bug

Testuser can change password on windows but not via passcore website
Error: "You are not allowed to change your password. Please contact your system administrator."

To Reproduce
Steps to reproduce the behavior:
Tried setting the "AllowedADGroups" to "domain users" and also tried it default (empty), but none work
Expected behavior
A succesful password change is to be expected, but not working
Desktop (please complete the following information):

OS: Windows 10, Windows Server 2012R2
Browser Chrome, Firefox

Hi,
I have the same issue with the same warn message.
No error message in the log file.
I tried to disable the automatic context, but I have an error 0x30005000 (I found that this message is due to the automatic context and the clue to slove the problem is to activate the automatic context).
I tested the connection between my server and one of my AD server on port 389 : all is fine.
When I changed my username, I had the message : user not found, so for me, the connection between the applicatopn end the server is OK.
I tried to change almost all the options to test, but I can't find something.
If someone has an idea....
Many thanks.

Hello,
nobody has an idea please?
@LingTjal : did you manage to find somenthing please?
Thanks.

Hello,
I had the same issue,
solved after setting "RestrictedADGroups": [],  //empty brackets

@hkml2000 : Thank you for your solutions.
It solved this problem.
But now, I have a new error message :

fail: Unosquare.PassCore.PasswordProvider.PasswordChangeProvider[888]
  ValidateGroups

System.NullReferenceException: Object reference not set to an instance of an object.
at Unosquare.PassCore.PasswordProvider.PasswordChangeProvider.b__7_0(Principal x) in C:\Unosquare\passcore\src\Unosquare.PassCore.PasswordProvider\PasswordChangeProvider.cs:line 175

We fixed this by following the instructions here and setting the minimum password age to 0. It was previously set to 1 in our environment.

I ran into the same error message this morning when testing passcore for my work environment and found what seems to be a bug.
In the "PasswordChangeProvider", there is a method invoked to validate the groups of the users and at some point it looks for the appsetting "AllowedADGroups". Here is what the code looks like :
return groups?.Any(x => _options.AllowedADGroups?.Contains(x.Name) == true) == true
    ? null
    : new ApiErrorItem(ApiErrorCode.ChangeNotPermitted, "The User principal is not listed as allowed");

The appsetting "AllowedADGroups" documentation says the following :

Set the AD Groups to allow PassCore, if the array is empty all the groups no-restricted above are allowed

However, with the way it is currently written, it appears that if the array is empty, it will return an ApiErrorItem instead. That is because _options.AllowedADGroups?.Contains(x.Name) would return null instead of true when _options.AllowedADGroups is empty.
What I think could fix this issue is by either making it so _options.AllowedADGroups is an empty list rather than null when the corresponding appsetting is empty, or by changing the previous condition to be _options.AllowedADGroups?.Contains(x.Name) != false.
As for me, I did apply the second fix (changing the condition) and I managed to successfully change the password of the AD accounts.

I ran into the same error message this morning when testing passcore for my work environment and found what seems to be a bug.
In the "PasswordChangeProvider", there is a method invoked to validate the groups of the users and at some point it looks for the appsetting "AllowedADGroups". Here is what the code looks like :
return groups?.Any(x => _options.AllowedADGroups?.Contains(x.Name) == true) == true
    ? null
    : new ApiErrorItem(ApiErrorCode.ChangeNotPermitted, "The User principal is not listed as allowed");

The appsetting "AllowedADGroups" documentation says the following :

Set the AD Groups to allow PassCore, if the array is empty all the groups no-restricted above are allowed

However, with the way it is currently written, it appears that if the array is empty, it will return an ApiErrorItem instead. That is because _options.AllowedADGroups?.Contains(x.Name) would return null instead of true when _options.AllowedADGroups is empty.
What I think could fix this issue is by either making it so _options.AllowedADGroups is an empty list rather than null when the corresponding appsetting is empty, or by changing the previous condition to be _options.AllowedADGroups?.Contains(x.Name) != false.
As for me, I did apply the second fix (changing the condition) and I managed to successfully change the password of the AD accounts.

Looks like this should still be looked at...


maintainers: add deathraymind

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

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

Why are you PR'ing and closing the same commit several times?
https://github.com/NixOS/nixpkgs/pulls?q=is%3Apr+author%3Adeathraymind
Also, you don't need to add yourself in an extra PR, just add yourself to maintainers in an extra commit, when you make your first contribution :)

Hello,
Thank you for your request.
Could you please provide some context or motivation behind your request to be added to the maintainer list? It would be helpful for us to understand how this aligns with your intentions or contributions to the project.
In my opinion, it might be more effective to create a PR that demonstrates why being included in the maintainer list is relevant. For instance:
Are you planning to contribute to nixpkgs?
Do you intend to adopt and maintain specific packages?
If your goal is to adopt packages, there's no need for a separate PR for this request. Instead, you can include a commit in a PR where it's clear which packages you'd like to maintain.
Please note that this is not about discouraging new contributors, on the contrary! We value new maintainers, and some contextual information will help us streamline the process and support your contributions effectively.
Thank you.


fix: Provide accurate instance stats for active user count

Fixes https://github.com/appsmithorg/cloud-services/issues/1457
EE PR: https://github.com/appsmithorg/appsmith-ee/pull/3033
Summary by CodeRabbit

New Features

Introduced a new indicator to distinguish system-generated users from regular users.

Enhancements

Improved user count accuracy by excluding system-generated users from active user statistics.

Database Changes

Executed a migration to mark existing anonymous users as system-generated.

Bug Fixes

Adjusted user repository queries to correctly count non-system-generated users.

/ok-to-test

/ok-to-test


[WIP] Provide context path to comet session reload function

closes #1901
This attempts to fix the issue with context paths when comet sessions are lost. For some reason it doesn't appear to actually fix the issue... I'm at the point where I need a second pair of eyes on this. :/

@Shadowfiend Alright, I think this is ready for another look

@Shadowfiend hate to break it to you but in my testing the urlencoding breaks everything. It looks like the convention is for the context path to include the / which, when urlencoded, produces different results. Going to back that particular change out. We can leave in the quote escape though.

That makes sense---in JS, that's the difference between encodeURI and encodeURIComponent, I guess. It's possible browsers automatically handle that for window.location assignments, as well. Since context path probably won't be much more than a path, I think this is fine.


BPMSPL-127: Implement new DataIOEditor for defining task Input/Output Variables a…

…nd Assignments

Closed pending further work


Request for supporting previous Angular Version ( Angular 15, 16)

This issue is raised based on the message from ngx-popperjs home page

@anujithrb well done mate. Can we please separate, to better track the development? Let's go with 16 now.
Then we'll handle v15 with another issue.
Thank you

Done


Remove cascade deletion of warrants when an object is deleted

Describe your changes
This PR removes the cascade deletion of warrants related to an object when that object is deleted. Because some objects can be associated with a large number of warrants, this operation can be pretty expensive. Instead, it's left up to the user to delete all associated warrants before deleting the underlying object. Unreferenced warrants can also be cleaned up after the fact.
Issue number and link (if applicable)
N/A

Closing in favor of #175


upgrade GUI to resemble the logistic tab

10 columns
min/max requests
fixed number of rows

should be working in the latest branch.
also tried to fix the overlapping of the save/load/delete buttons with the request-table.


Bitcrazed/terminal timeline docs

Summary of the Pull Request
Documents the project's timeline.
References
N/A.

PR Checklist

[X] Closes #1986
[X] CLA signed. If not, go over here and sign the CLA
[ ] Tests added/passed
[X] Requires documentation to be updated
[X] I've discussed this with core contributors already. If not checked, I'm ready to accept this work might be rejected in favor of a different grand plan. Issue number where discussion took place: #xxx

Hey @factormystic, would you mind using the code review comment batching feature? Otherwise the whole team gets one e-mail per comment :smile:
Thanks for reviewing!

Hey @factormystic, would you mind using the code review comment batching feature? Otherwise the whole team gets one e-mail per comment 😄
Thanks for reviewing!

Ha yeah I didn't intend to do a real review, just a spot fix for a typo. Then after I hit comment, I noticed another one. Then another one... etc :)

Hey @factormystic, would you mind using the code review comment batching feature? Otherwise the whole team gets one e-mail per comment 😄
Thanks for reviewing!

Ha yeah I didn't intend to do a real review, just a spot fix for a typo. Then after I hit comment, I noticed another one. Then another one... etc :)

I totally know what you mean 😁

Your title still looks .. weird. Why's it got Bitcrazed/ in it?

Is it worth calling out that we will not support composing characters or grapheme clusters that are not representable with a single unicode codepoint? As part of the UTF-16/UCS-2/UTF-8 deliverable.


⚠ Samsung has degraded performance

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

HTTP code: 200
Response time: 2700 ms

Resolved: Samsung performance has improved in 447db0b.


Authorization Issues Connecting to MongoDB 3.0

I'm having some problems connecting to a MongoDB database running version 3.0!
CommandError[code=13, errmsg=not authorized on myDb to execute command { insert: "user", documents: [ { .....},
code: BSONInteger(13)
}]
Is this some incompatibility issue? This is just bugging me! I have written lots of code for the persistence and now the very basic connection seems to be problematic is really scaring me.

Hi, please read the documentation (and have a look at authMode) and/or use the mailinglist to ask questions. Best regards.

I did try all possibilities and it seems like the problem is definitely in the ReactiveMongo library. I did try to log in using the command line with exactly the same credentials to the MongoLab instance and I was able to insert the entry into my collection! Here is the URL that I tried (with the auth mode):
mongodb://user:pass@XXXXX.mongolab.com:XXXXX/myDbName?authMode=scram-sha1

Hi, if you look at the MailingList, you can see that a demo app using MongoLab is successfully working at Heroku.
Please check your env and use the MailingList for further discussion. Best regards.


It does not finish installing. Windows 10

It does not finish installing. Windows 10
flutter doctor
does not finish 15 minutes later start ..
PS C:\> flutter doctor
Checking Dart SDK version...
Downloading Dart SDK from Flutter engine 1d0da7799583b089ede66b81068f40cc4597a429...
Unzipping Dart SDK...
Updating flutter tool...
No advance of this state.

@matsyendra Is this issue solved?

@anandwana001 at the moment, no.

I have seen this error mentioned when Flutter was installed in C:/Program Files or another directory where only the Administrator has write permissions.
Is this still an issue?


feat(Auth): Using helpers for passwordless workflows

Description

The PR aims to refactor MagicLink and OTP, to helpers so that code can be reused.
General Checklist

[x] Added new tests to cover change, if needed
[x] Build succeeds with all target using Swift Package Manager
[x] All unit tests pass
[x] All integration tests pass
[x] Security oriented best practices and standards are followed (e.g. using input sanitization, principle of least privilege, etc)
[ ] Documentation update for the change if required
[x] PR title conforms to conventional commit style
[ ] New or updated tests include Given When Then inline code documentation and are named accordingly testThing_condition_expectation()
[ ] If breaking change, documentation/changelog update with migration instructions

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

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

Comparison is base (87b603f) 67.99% compared to head (2dd8853) 87.03%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@                     Coverage Diff                     @@
##           feat/auth-passwordless    #3390       +/-   ##
===========================================================
+ Coverage                   67.99%   87.03%   +19.04%
===========================================================
  Files                        1091       12     -1079
  Lines                       36583      270    -36313
===========================================================
- Hits                        24873      235    -24638
+ Misses                      11710       35    -11675

Flag
Coverage Δ

API_plugin_unit_test
?

AWSPluginsCore
?

Amplify
?

Analytics_plugin_unit_test
?

Auth_plugin_unit_test
?

CoreMLPredictions_plugin_unit_test
?

DataStore_plugin_unit_test
?

Geo_plugin_unit_test
?

Logging_plugin_unit_test
?

Predictions_plugin_unit_test
?

PushNotifications_plugin_unit_test
87.03% <ø> (ø)

Storage_plugin_unit_test
?

unit_tests
87.03% <ø> (+19.04%)
:arrow_up:

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.


the size of generated pdf is too big after generating the pdf - how to fix it ?

Which version of PhantomJS are you using? Tip: run phantomjs --version.

± phantomjs --version
2.1.1

What steps will reproduce the problem?

load the html on localhost:1313
2.then run this command on terminal :
phantomjs rasterize.js 'http://localhost:1313/html/contract.html' suhag.pdf

Which operating system are you using?
mac

Did you use binary PhantomJS or did you compile it from source?
i dont know.

Please provide any additional information below.

heres the output  https://github.com/jpca999/phantom-contr-gen/blob/master/suhag.pdf
Required output pdf size :
https://github.com/jpca999/phantom-contr-gen/blob/master/original /AS IS Residential Contract for Sale  and Purchase.pdf
heres the code.
https://github.com/jpca999/phantom-contr-gen

Due to our very limited maintenance capacity (see #14541 for more details), we need to prioritize our development focus on other tasks. Therefore, this issue will be closed. In the future, if we see the need to attend to this issue again, then it will be reopened.
Thank you for your contribution!


Emoji

It doenot recognize all emojis

It's late but for future users:
go to this address : your_root_path/vendor/unicodeveloper/laravel-emoji/src/ and edit emoji.php file;
File have to be like this :
<?php

return [

    /*
    |--------------------------------------------------------------------------
    | Emoji Emoticons and their Unicode equivalents
    |--------------------------------------------------------------------------
    | Get the Emoji code from http://www.emoji-cheat-sheet.com/
    | and get its equivalent from http://unicode.org/emoji/charts/full-emoji-list.html
    |
    | Contribute, let's make our laravel apps more emojified
    |
    */

    "grinning"                  => "\u{1F600}",
    "grin"                      => "\u{1F601}",
    "joy"                       => "\u{1F602}",
    "smiley"                    => "\u{1F603}",
    "smile"                     => "\u{1F604}",
    "sweat_smile"               => "\u{1F605}",
    "laughing"                  => "\u{1F606}",
    "wink"                      => "\u{1F609}",
    "blush"                     => "\u{1F60A}",
    "yum"                       => "\u{1F60B}",
    "sunglasses"                => "\u{1F60E}",
    "heart_eyes"                => "\u{1F60D}",
    "kissing_heart"             => "\u{1F618}",
    "kissing"                   => "\u{1F617}",
    "kissing_smiling_eyes"      => "\u{1F619}",
    "kissing_closed_eyes"       => "\u{1F61A}",
    "relaxed"                   => "\u{263A}",

Then add your custom emojies to this file. You can choose your emojies from http://unicode.org/emoji/charts/full-emoji-list.html
In my case I did:
.
.
.
    "kissing_smiling_eyes"      => "\u{1F619}",
    "kissing_closed_eyes"       => "\u{1F61A}",
    "relaxed"                   => "\u{263A}",

    //My Own Emojies
    "star_eyes"                 => "\u{1F929}",
    "dollar_eyes"               => "\u{1F911}",
    "damaged"                   => "\u{1F915}"

And finally call them using Emoji::findByAlias("damaged");


Issue with Scene and status icons

Hi all, thanks foe the integration of this awsome icon pack.
However, I noticed that the icons with the prefix scene_ and status_ had some display bugs.
Is it me?

Hi, no - you are innocent :-)
The original SVG files use all kinds of shapes (rectangle, circles, ...) and transformations. HomeAssistant supports only 1 shape type: paths. Therefore I had to convert all shapes into paths - with some script magic. However, this still doesn't work perfect for all icons.
Could you create a list of icons which are broken? (at least the one's you already observed). Then I could try to enhance the conversion script for them.

@mampfes  You should probably try again with latest inkscape 1.1.1...

Duplicate of #3


Fix info of num of handlers in docs/design.md

Quick fix of infomation about num of handlers in docs/design.md

bors r+


Point to new repo location

It would be great if this could be merged and in addition this repository could be "Archived" to prevent new issues/PRs being reported here. You can archive the repo by going to https://github.com/appuri/robust-websocket/settings and clicking "Archive this repository"

Alternatively, @nathanboktae if you have the power to delete this repo - you could also delete it and contact<EMAIL_ADDRESS>to ensure that this repo redirects to your one.

I don't have that power anymore as I'm not apart of the Appuri org.

Perhaps @bilalaslamseattle or @jwerak could merge this?


Sents with custom limits(like wiremod) counting as sents when duping alongside normal sents

https://youtu.be/zo96YkKd1Jk
vanilla sent limit set at 2 using vanilla convar, not messing with sent limit in WUMA at all.

Let me get this straight, you have set some WUMA limits on sents, which should not count towards the vanilla convar sent limit. But when you duplicate the sents they count towards the vanilla convar sents limit after all?

I've found the issue that causes the problem your video shows. I'll push a fix out soon.

Fixed by 9e3f65d9e71b6e44df8896959c2b448e4625a2a3


Header templating doesn't work when simulation has a header matcher

Noticed a very likely related bug in templating:
If I have a "headers" matcher in my simulation, e.g.:
          "headers": {
            "Authorization": [
              {
                "matcher": "glob",
                "value": "Bearer test-*"
              }
            ]
          }

then in the response body a {{ Request.Header.Authorization }} returns an empty string. But if I remove the request "headers" matcher, I get the correct value printed to the response. @tommysitu
Originally posted by @ttiurani in https://github.com/SpectoLabs/hoverfly/issues/944#issuecomment-680741095

resolved


Add XR validation warning for not targeting Android 29 when using OpenXR

Summary
Adds an XR validation step to ensure an application is targeting Android 29, as per the OpenXR recommendations, if OpenXR is being used for a project.
This ensures that the tracking service can be correctly connected to despite Android 11's package visibility changes.
Contributor Tasks
These tasks are for the merge request creator to tick off when creating a merge request.

[ ] Pair review with a member of the QA team.
[ ] Add any release testing considerations to the MR for the next release.
[x] Consider any licensing/other legal implications for this MR e.g. notices required by any new libraries.
[x] If this MR closes a Jira issue, make sure the fix version on the JIRA issue is set to the correct one.

Reviewer Tasks

[x] Code reviewed.
[x] Non-code assets e.g. Unity assets/scenes reviewed.
[ ] Checked and agree with release testing considerations added to MR for the next release.
[x] Ensure that if an Android SDK version greater than 29 is listed, there is a warning in OpenXR validation.
[x] Ensure that if the "Fix" button is clicked, it automatically adjusts the target, and the minimum SDK level if it was higher than 29.

Closes JIRA Issue
Relates to OXR-190 (and is a warning to ensure the workaround continues to operate correctly).

@rblenkinsopp what happens if you change your project to use another plugin/runtime does the validation go away?

@rblenkinsopp what happens if you change your project to use another plugin/runtime does the validation go away?

So, if I had Android as a build target, set the target android version to 30, and then swap my build target to standalone.... I am still warned about the android version. I can press fix, and it goes away (and doesn't change my build target)
So essentially, it may warn you about your target android version even when not using android build settings, but it will still allow you to build etc while the warning is there, so is not an issue. and the wording is clear that is it android specific

I can't test this, but the changes look good.

@rblenkinsopp what happens if you change your project to use another plugin/runtime does the validation go away?

@Greg-UL So, if I had Android as a build target, set the target android version to 30, and then swap my build target to standalone.... I am still warned about the android version. I can press fix, and it goes away (and doesn't change my build target)
So essentially, it may warn you about your target android version even when not using android build settings, but it will still allow you to build etc while the warning is there, so is not an issue. and the wording is clear that is it android specific

Hmm, I should actually be able to make it so it only lists that as a warning if you have Android as a build target. We should probably do that.

@rblenkinsopp what happens if you change your project to use another plugin/runtime does the validation go away?

@Greg-UL So, if I had Android as a build target, set the target android version to 30, and then swap my build target to standalone.... I am still warned about the android version. I can press fix, and it goes away (and doesn't change my build target)
So essentially, it may warn you about your target android version even when not using android build settings, but it will still allow you to build etc while the warning is there, so is not an issue. and the wording is clear that is it android specific

Hmm, I should actually be able to make it so it only lists that as a warning if you have Android as a build target. We should probably do that.

Yes, that would be a good change


Updates for a new generation

Fix #82 by shuffling the responsibility of install_requires and test_requires to requirements
Leverage requirements in environment.yml
Update CONTRIBUTING doc
Switch from Travis to GH actions
Format code with black
Switch default branch to main
Update the copyright information

The plan for switching the default branch is to close this MR when I'm satisfied with it, then do the switch in github's ui. I don't know if there's a good way to properly deprecate the master branch but that's OK.
I'd also like to add support for updating/publishing the AUR package but I'd like to do that in a separate merge request since it'll be easiest with submodules and that stuff is a little fraught.

GH Actions doesn't test ppc but Travis broke when they switched from .org to .com and I think I need to talk to support to fix it at this point. I'm going to leave the yml file in-place for ppc but defer fixing it to a later date.

Released and the default branch has been switched. Closing!


npm install didn't install grunt,

I am a newbie, later ran later ran npm install -g grunt-cli to address the issue. Thanks.

@arisc097 thanks


Get UserID from Room and Group

In LINE dev docs, Room and Group structure contains user ID. However, this library doesn't extract the user ID from the JSON. https://github.com/noraesae/line/blob/master/src/Line/Messaging/Webhook/Types.hs#L208
Anyway, this is a solid library. Thanks for working on this!

User ID is attached only when the user has already agreed to Official Account user consent. Please refer to the official document for the detail.

https://developers.line.me/messaging-api/obtaining-user-consent

Thanks.

Yes, this SDK is out of date from some new APIs. Will work on in a while.

Awesome! 🙂


Add browserid_info tag

Hi,
It appears that Browser ID is broken without the {% browserid_info %} tag. We found that adding this meant that we could log in with browser id.
We are still looking at an issue where logging in causes a 500 error with DEBUG = False.
There may be few more fixes we will send through as we find them, as we have just launched a production instance of Crowdata. Thanks for all your hard work releasing this project.
Caleb

Thanks again, @tuttinator! Is this a production instance as in "real, live users using the system" instance? :)

We plan to have users using it soon - we are looking at electoral donations and expenses by candidates in the New Zealand General Election - and they've just only been released ten minutes ago.
Also we found that django-browserid needs something like this BROWSERID_AUDIENCES = [SITE_URL]

Awesome! Please hit me up on Twitter (@manuelaristaran) when you launch; I'd love to see what you've done with CrowData :)
Re: BrowserId woes — the main use case for CrowData was a project at La Nación (Argentina) and we integrated with the newspaper's own auth system, writing a custom Django auth backend. We never really tested BrowserID :(


App Menu: New Tab, New Window

Tracking for MVP milestone, both hotkeys and menu items.
I believe these are mostly working, but should not be considered complete without tests, as fragile as hotkeys are to begin with.

New Tab works, New Window does not, in either File menu, or hamburger menu

These work now in the new frontend. There are many parts of our code not being tested at the moment, so we should defer to https://github.com/mozilla/tofino/issues/657 instead of these individual issues that may get lost.


Standardize example descriptions/comments

You can see the new descriptions at https://gtp.scientificwebservices.com/geophires -> Examples tab

This looks fine. It seems the new rule is that comments start with "#". Can we update the input parameter line parsing code to check for lines that start with "#" and automatically ignore them, no matter what they contain? I ask for this because what I have noted in the past that if the comment lines contains commas, the parser will try to parse the line and will fail, producing an error. I went thru and removed all the commas in comments, but now if we use "#" to designate comments, we can have as many commas in the comments as we want if the parser code is updated to handle the new comment line delimiter. I suggest we try to maintain backward compatibility for comments that don't start with "#" as much as possible, with the caveat that those lines must not contain commas.

Good call - Added test cases with a variety of comment styles, and discovered and fixed some edge cases in doing so.


Improve TimeDynamicPointCloud test

This will hopefully fix the failing test that @mramato noticed in https://github.com/AnalyticalGraphicsInc/cesium/pull/6880. The test passes for me locally so I couldn't verify. @mramato could you test this on your computer?
Adjusted the test so it can't fail accidentally if two frames load in exactly the same amount of time.

Thanks for the pull request @lilleyse!

:heavy_check_mark: Signed CLA found.
:grey_question: CHANGES.md was not updated.

If this change updates the public API in any way, please add a bullet point to CHANGES.md.

Reviewers, don't forget to make sure that:

[ ] Cesium Viewer works.
[ ] Works in 2D/CV.
[ ] Works (or fails gracefully) in IE11.

I am a bot who helps you make Cesium awesome! Contributions to my configuration are welcome.
:earth_africa: :earth_americas: :earth_asia:

Thanks @lilleyse, I can't get this to fail anymore so I think we're good.


Collision with existing package on start server via npx

The doc says we can start server with npx mcs command, but mcs is existing npm package, which creates a collision

@Kater-auf-Dach thx for your response, we will fix readme in next release, right now you can use yarn mcs or npx mock-config-server command


react-dom-stream

Any chance to replace react-dom/server with react-dom-stream. It's looks more faster.

This looks awesome. Current SSR is pretty heavy on the server.
If we can reduce it that's great.
Move to here: https://github.com/kadirahq/react-mounter/issues/7


Cp/network details page

Closes #10

Hey what about the background gradients?

The network details page only had the blue/green gradients for the social links in the designs.


Update to Yarn berry

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

main

#62

#63   👈

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @NathanFlurry and the rest of your teammates on  Graphite


Fix app labels

Towards

https://github.com/giantswarm/giantswarm/issues/30245

What about the failing checks. Are you sure @njuettner ?

@marians right I fixed it by adding a small change to kustomization.yaml in config/helm. That slightly also changed the sorting of the labels, but I pushed them as well. Feel free to merge it 👍🏻

Thanks!


npm install ng-fullcalendar

Hello everyone,
When i install ng-fullcalendar i have this problem :
Cannot find module 'fullcalendar'
Any idea please.

ng-fullcalendar has a dependency fullcalendar, try
npm install fullcalendar


Buckling is inconsistent on passenger seats

When attempting to buckle, it doesn't always buckle you on the first go. Notably on the dropship, you may need to buckle twice to actually get in the seat. It sort of throws you off the seat the first time, as if you buckle into it with momentum.

it depends on if its a seat next to the walls, because of the collision it unbuckles you the first time i think

that'd do it probably, although never paid attention enough to see if it's an issue on the open seats


styling user avatar and fix some sass errors

@goldipl Branch has unresolved conflicts.
They happen to be in places where you removed commas.


Feature/code editor instructor view & test coverage

Checklist

[x] I run yarn run webpack:build:main: the project builds without errors.
[x] I run yarn lint: the project builds without code style warnings.
[x] I updated the documentation and models.
[x] I tested the changes and all related features on the test server https://artemistest.ase.in.tum.de.
[x] ~I added (end-to-end) test cases for the new functionality.~

Motivation and Context
Tutors/Instructors could use the code-editor to view/edit template/solution participations if we make the code-editor component more abstract.
Description
Features:

From the course administration, programming exercises can be opened in editor
In this view an additional bar is shown at the top that enables the instructor/ta to manage programming exercise code
The instructor/ta can switch between the template, solution, assignment and test repo
Changes to the repos can be persisted by saving and submitting
The assignment repo can be created from within the interface if not existing yet and can be reset (delete everything of the participation including build plan, etc. and create a new fork from the current template repo)
The instructor/ta can edit the instructions from within this interface
Right next to the exercise title an indicator is shown that displays the status of the template/solution repos (for solution repos all test cases should be successfull, for template repos all test cases should fail).

The two most important changes:

To make the code-editor reusable for a repo without a participation (test repo), I had to remove all traces of the participation from within the file-browser, code-editor and actions.
To do this I have introduced singleton services in the code-editor that store the participation (solution, template, assignment) or the exercise (test). The services subscribe to the CodeEditorDomainService to retrieve information on the currently selected domain (participation based vs test repo)
This also changes the RepositoryResource on the server, there is now an abstract class that is extended by each a participation based and a test repo based version. I made it specific to the test-repo, as I don't see any use cases atm to use repositories other than the test-repo without a participation.

Not in scope of this PR:

Atm changing the participation triggers a route change and this triggers the change detection guard. It can be annoying if the save button always has to be triggered when changing between the repos. The idea would be to bind the unsavedChanges to an attribute of the container and don't trigger route changes.

Testing

[x] component: code-editor-instructions
[x] component: code-editor-file-browser
[x] component: code-editor-ace
[x] component: code-editor-status
[x] component: code-editor-actions
[x] component: code-editor-build-output
[x] component: programming-exercise-instructions
[x] integration: code-editor-instructor
[x] integration: code-editor-student

Steps for Testing

Log in to ArTEMiS
Navigate to Course Administration
Open course with a programming exercise
Click button "Edit in Editor"

Screenshots

@krusche This PR is finally done :)
It is a lot of files, but roughly half of it are files in the spec folders (mocks and test files), index files & interface implementations, that cannot affect functionality.
I also changed the layout of the instructor view a bit after your feedback, I hope you like it better this way.

Sorry, I did not have time yet to review this. I suggest we add this to the next big release 3.2.0.
Would be great if you could fix the merge conflicts and we review this together next week, when I am back in Munich

Done :)

I added some UML diagrams to make the review easier.

I made a merge on Github, however the merge commit broke the build for some reason.
I removed the commit and did a force push with a local merge, therefore the force push.

There are also 2 issues that @SiggZ found, that I'll look into in this PR:

[ ] Sometimes the submit of the assignment repo didn't work for him
[x] After adding new tests to a test file, it appeared multiple times in the test case dropdown
This is not an issue with the instructor view, but due to the structure of the test case sample repo: When a test is placed in the MethodTest file, 3 test cases are created.

@krusche The bugs discovered by @SiggZ for this PR are fixed.

After merging the domain command improvements, I noticed an issue with the instructions component in the instructor view.
The fix for the issue would be to also load the feedbacks with the latest result, however this causes an issue in JPA where the feedbacks only for the templateParticipation can't be fetched. I'll look into this.


Extract commit ID with regex instead of shell and awk

Now in case the command failed the error appear immediately, also the output is more easy to read and understand
TestDallas: ocp-4.17-vcp

retest

Can we test this with OCP 4.7? Because this is the target version for this change.

I can run it on my ci machine, is it enough?

retest

TestDallas: ocp-4.17-vcp

We probably mean ocp-4.7-vcp?

TestDallas: ocp-4.17-vcp

We probably mean ocp-4.7-vcp?

I talked to @reihl and we will be testing with 4.8, I've made the change in the body of the PR

recheck

Can we test this with OCP 4.7? Because this is the target version for this change.

Good point, checking this one with @reihl

The last failure is due to a maintenance going on in RDO project, I'll retry once it's done.

TestDallas: ocp-4.17-vcp

We probably mean ocp-4.7-vcp?

I talked to @reihl and we will be testing with 4.8, I've made the change in the body of the PR

But the change only affects to OCP 4.7, doesn't it? According to this when condition:
  when: (release_version.split('.')[0]|int == 4) and (release_version.split('.')[1]|int <= 7)

TestDallas: ocp-4.17-vcp

We probably mean ocp-4.7-vcp?

I talked to @reihl and we will be testing with 4.8, I've made the change in the body of the PR

But the change only affects to OCP 4.7, doesn't it? According to this when condition:
  when: (release_version.split('.')[0]|int == 4) and (release_version.split('.')[1]|int <= 7)

Correct, I also change the when condition to be more readable with ansible.builtin.version filter. We face it because something that ignore we don't need rhcos.json for version newer then 4.7 and this is the reason I fix it.

But the change only affects to OCP 4.7, doesn't it? According to this when condition:

@ramperher correct, I pinged @reihl to see if he can share logs as he is using OCP 4.8, but the change in here shouldn't affect 4.8, so I'm not sure how exactly this is going to fix something in 4.8

We face it because something that ignore we don't need rhcos.json for version newer then 4.7 and this is the reason I fix it.

Could you please elaborate a bit more here?
For 4.8 and above a different set of tasks are performed to use oc  coreos print-stream-json, if you're using 4.8 this is the tasks that should get you what's needed without having to go to a commit as in 4.7-.
https://github.com/redhatci/ansible-collection-redhatci-ocp/blob/2f66e0f84b3037236cb408aafd57fb8d79b35f9e/roles/installer/tasks/23_rhcos_image_paths.yml#L39-L43

retest


Update README instructions for new versions of RN

Hi @NishanthShankar,
I had used this project previously with older versions of RN and it's great. Today I was adding it to a new project using RN 0.30 and found out that you recently merged support for it in a commit, but docs were not updated. Here are my 2 cents.
Thanks, cheers,
Miguel

Ok,
I've just realized that this project has been superseded by official StatusBarcomponent in React Native https://facebook.github.io/react-native/docs/statusbar.html

Thanks man. Appreciated


Docs: fix conntrack flow ct_state input

In the following envrionment:
ovs-vsctl (Open vSwitch) 2.11.0
DB Schema 7.16.1
we meet the following error during the tutorials
conntrack test:
"ovs-ofctl: field +est missing value"
"ovs-ofctl: field +trk missing value"
ovs-vsctl 2.9.0 has the same issue.
This patch gives the tutorials with right
conntrack input.
Signed-off-by: LIU Yulong<EMAIL_ADDRESS>

Thank you! I applied this to all relevant branches.


segfault in CloudContainer::Connect when taking javascript CPU profile of backend tests in chrome devtools [linux]

Describe the bug
Segfault in CloudContainer::Connect (see additional context for stack) when taking javascript CPU profile of transformer in chrome devtools. This occurs on Ubuntu Linux, I have not yet tried any other OSes.
This means to measure performance differences in changes I'm making I need to go back further on master to find a working addon which I will do soon.
To Reproduce
git clone https://github.com/iTwin/itwinjs-core.git
cd itwinjs-core
git checkout single-transformer-test-cloudcont-crash
rush install
rush build -t @itwin/core-transformer
(cd core/backend && node --inspect-brk node_modules/mocha/bin/mocha.js)
# at this point it will wait for you to connect with a debugger. I have only tested with chrome devtools
# you can open chrome devtools by opening the chrome browser and navigating to chrome://inspect
# it should detect the process waiting for a debugger to attach and you can just click the "inspect" link under "remote target"
# once it connects, in the "profiler" tab, press the blue "start" button (see image below)

Expected behavior
No segfault, able to use chrome devtools to produce javascript CPU profiles of iTwin.js code.
Screenshots
N/A
Desktop (please complete the applicable information):

OS: ubuntu linux 21.04
Browser: N/A
Version: N/A
iTwin.js Version: Local master, 25952e299b1d4132336a5d55f4ae5eb03526a8a5, native addon 3.4.6 aka b0e8520af50a86ef02c6dd660259dda9d84ab14d

Additional context
I could not reproduce with a locally built debug and non-debug addon. Any pointers for what might be different between the released addon and my local build of b0e8520af50a86ef02c6dd660259dda9d84ab14d with DEBUG and then NDEBUG?
Also if someone could confirm that b0e8520af50a86ef02c6dd660259dda9d84ab14d is the corresponding commit to<EMAIL_ADDRESS>I would appreciate that.
Stack from attaching via gdb to the process:
#0  0x00007ffd4a686638 in ?? () from /home/mike/work/imodeljs/common/temp/node_modules/.pnpm/@bentley+imodeljs-native<EMAIL_ADDRESS>#1  0x00007ffd4a686ffb in ?? () from /home/mike/work/imodeljs/common/temp/node_modules/.pnpm/@bentley+imodeljs-native<EMAIL_ADDRESS>#2  0x00007ffd4a663ec8 in ?? () from /home/mike/work/imodeljs/common/temp/node_modules/.pnpm/@bentley+imodeljs-native<EMAIL_ADDRESS>#3  0x00007ffd4a662923 in ?? () from /home/mike/work/imodeljs/common/temp/node_modules/.pnpm/@bentley+imodeljs-native<EMAIL_ADDRESS>#4  0x00007ffd4a6631bd in BentleyM0200::BeSQLite::CloudContainer::Connect(BentleyM0200::BeSQLite::CloudCache&) ()
   from /home/mike/work/imodeljs/common/temp/node_modules/.pnpm/@bentley+imodeljs-native<EMAIL_ADDRESS>#5  0x00007ffd4a396fb3 in ?? () from /home/mike/work/imodeljs/common/temp/node_modules/.pnpm/@bentley+imodeljs-native<EMAIL_ADDRESS>#6  0x00007ffd4a397135 in ?? () from /home/mike/work/imodeljs/common/temp/node_modules/.pnpm/@bentley+imodeljs-native<EMAIL_ADDRESS>#7  0x0000000000ab44dd in v8impl::(anonymous namespace)::FunctionCallbackWrapper::Invoke(v8::FunctionCallbackInfo<v8::Value> const&) ()
#8  0x0000000000d53cfe in v8::internal::MaybeHandle<v8::internal::Object> v8::internal::(anonymous namespace)::HandleApiCallHelper<false>(v8::internal::Isolate*, v8::internal::Handle<v8::internal::HeapObject>, v8::internal::Handle<v8::int
ernal::HeapObject>, v8::internal::Handle<v8::internal::FunctionTemplateInfo>, v8::internal::Handle<v8::internal::Object>, v8::internal::BuiltinArguments) ()
#9  0x0000000000d5511f in v8::internal::Builtin_HandleApiCall(int, unsigned long*, v8::internal::Isolate*) ()
#10 0x00000000015f0b79 in Builtins_CEntry_Return1_DontSaveFPRegs_ArgvOnStack_BuiltinExit ()
#11 0x000000000158354a in Builtins_InterpreterEntryTrampoline ()
#12 0x00003079bf9c1599 in ?? ()
#13 0x000033876bd31c19 in ?? ()
... goes to #27 but its just ??

gdb also gives me these hundreds of these warnings if someone knows what it means. I haven't looked into it yet.
warning: Corrupted shared library list: 0x50eb0c0 != 0x7ffff7ffda48
warning: Corrupted shared library list: 0x50eb0c0 != 0x7ffff7ffda48
warning: Corrupted shared library list: 0x50eb0c0 != 0x7ffff7ffda48

I managed to get the exact unstripped addon and got this callstack:
#0  0x00007ffd4a686638 in bcvContainerFree (pCont=0x0)
    at /home/platform-bld/vsts_agent_work/242/s/iModelCore/BeSQLite/SQLite/blockcachevfs.c:3797
#1  0x00007ffd4a686ffb in bcvfsAttach (pFs=<optimized out>, zStorage=<optimized out>, zAccount=<optimized out>,
    zContainer=<optimized out>, zAliasArg=<optimized out>, flags=<optimized out>, pzErr=<optimized out>)
    at /home/platform-bld/vsts_agent_work/242/s/iModelCore/BeSQLite/SQLite/blockcachevfs.c:4027
#2  sqlite3_bcvfs_attach (pFs=0x559aca8, zStorage=<optimized out>, zAccount=<optimized out>, zCont=<optimized out>,
    zAlias=<optimized out>, flags=2, pzErr=0x7fffffffc288)
    at /home/platform-bld/vsts_agent_work/242/s/iModelCore/BeSQLite/SQLite/blockcachevfs.c:4130
#3  0x00007ffd4a663ec8 in BentleyM0200::BeSQLite::CloudContainer::Connect(BentleyM0200::BeSQLite::CloudCache&)::$_3::operator()(char**) const (this=<optimized out>, msg=<optimized out>)
    at /home/platform-bld/vsts_agent_work/242/s/iModelCore/BeSQLite/CloudSqlite.cpp:249
#4  std::_Function_handler<int (char**), BentleyM0200::BeSQLite::CloudContainer::Connect(BentleyM0200::BeSQLite::CloudCache&)::$_3>::_M_invoke(std::_Any_data const&, char**&&) (__functor=..., __args=<optimized out>)
    at /usr/lib/gcc/x86_64-linux-gnu/6.3.0/../../../../include/c++/6.3.0/functional:1716
#5  0x00007ffd4a662923 in std::function<int (char**)>::operator()(char**) const (this=0x7fffffffc2e0,
    __args=0x7fffffffc288) at /usr/lib/gcc/x86_64-linux-gnu/6.3.0/../../../../include/c++/6.3.0/functional:2127
#6  BentleyM0200::BeSQLite::CloudCache::CallSqliteFn(std::function<int (char**)>, char const*) (this=<optimized out>,
    fn=..., funcName=0x7ffd4f10dabc "attach")
    at /home/platform-bld/vsts_agent_work/242/s/iModelCore/BeSQLite/CloudSqlite.cpp:216
#7  0x00007ffd4a6631bd in BentleyM0200::BeSQLite::CloudContainer::Connect (this=<optimized out>, cache=...)
    at /home/platform-bld/vsts_agent_work/242/s/iModelCore/BeSQLite/CloudSqlite.cpp:249
#8  0x00007ffd4a396fb3 in IModelJsNative::JsCloudContainer::Connect (this=0x5566210, info=...)
    at /home/platform-bld/vsts_agent_work/242/s/iModelJsNodeAddon/JsCloudSqlite.cpp:197
#9  0x00007ffd4a397135 in Napi::details::TemplatedInstanceVoidCallback<IModelJsNative::JsCloudContainer, &IModelJsNative::JsCloudContainer::Connect>(napi_env__*, napi_callback_info__*)::{lambda()#1}::operator()() const (
    this=<optimized out>)
    at /home/platform-bld/vsts_agent_work/242/b/LinuxX64/static/BuildContexts/iModelJsNodeAddon/VendorAPI/Napi/node-src/napi-inl.h:181
#10 Napi::details::WrapCallback<Napi::details::TemplatedInstanceVoidCallback<IModelJsNative::JsCloudContainer, &IModelJsNative::JsCloudContainer::Connect>(napi_env__*, napi_callback_info__*)::{lambda()#1}>(Napi::details::TemplatedInstanceVoidCallback<IModelJsNative::JsCloudContainer, &IModelJsNative::JsCloudContainer::Connect>(napi_env__*, napi_callback_info__*)::{lambda()#1}) (callback=...) at /home/platform-bld/vsts_agent_work/242/b/LinuxX64/static/BuildContexts/iModelJsNodeAddon/VendorAPI/Napi/node-src/napi-inl.h:79
#11 Napi::details::TemplatedInstanceVoidCallback<IModelJsNative::JsCloudContainer, &IModelJsNative::JsCloudContainer::Connect> (env=<optimized out>, info=<optimized out>) at /home/platform-bld/vsts_agent_work/242/b/LinuxX64/static/BuildContexts/iModelJsNodeAddon/VendorAPI/Napi/node-src/napi-inl.h:178
#12 0x0000000000ab44dd in v8impl::(anonymous namespace)::FunctionCallbackWrapper::Invoke(v8::FunctionCallbackInfo<v8::Value> const&) ()
#13 0x0000000000d53cfe in v8::internal::MaybeHandle<v8::internal::Object> v8::internal::(anonymous namespace)::HandleApiCallHelper<false>(v8::internal::Isolate*, v8::internal::Handle<v8::internal::HeapObject>, v8::internal::Handle<v8::internal::HeapObject>, v8::internal::Handle<v8::internal::FunctionTemplateInfo>, v8::internal::Handle<v8::internal::Object>, v8::internal::BuiltinArguments) ()
#14 0x0000000000d5511f in v8::internal::Builtin_HandleApiCall(int, unsigned long*, v8::internal::Isolate*) ()
#15 0x00000000015f0b79 in Builtins_CEntry_Return1_DontSaveFPRegs_ArgvOnStack_BuiltinExit ()
#16 0x000000000158354a in Builtins_InterpreterEntryTrampoline ()
#17 0x0000387807dc1599 in ?? ()
#18 0x00000c0207955f19 in ?? ()
#19 0x0000000600000000 in ?? ()
#20 0x0000387807dc1669 in ?? ()
#21 0x000014511e18f6a1 in ?? ()
#22 0x000014511e190871 in ?? ()
#23 0x000004574e956d99 in ?? ()
#24 0x000014511e18f1d1 in ?? ()
#25 0x000016ee5595d0c9 in ?? ()
#26 0x000014511e190871 in ?? ()
#27 0x00003d3d7c6482a1 in ?? ()
#28 0x00000c0207955f19 in ?? ()
#29 0x000014511e18f6a1 in ?? ()
#30 0x000000ea00000000 in ?? ()
#31 0x000016ee5595d299 in ?? ()
#32 0x0000000000000003 in ?? ()
#33 0x000010fbb67a0ce1 in ?? ()
#34 0x00000c020797c089 in ?? ()
#35 0x00007fffffffc748 in ?? ()
#36 0x0000000001580b85 in Builtins_JSConstructStubGeneric ()
Backtrace stopped: frame did not save the PC

I believe this is an indication of heap corruption somehow. I see nothing wrong in the failing code, and i can't see how bcvContainerFree is ever called with 0x0. Is there any indication what's happening on the JavaScript side?

This was reproducible until instead of starting the CPU profiler I started it under the debugger. It now no longer reproduces, even after resetting the dependencies and repo state. Before that however I added some print debugging since the debugger wasn't working with the profiler, and I at least could see that it crashes after the addon is loaded and after the first test's describe is called in core/backend/src/test/categories/Category.test.ts, probably in the before call but I couldn't find out before it stopped reproducing.

maybe it'll reproduce if I reinstall chrome :) I have no idea how to proceed. I suppose we can close this as having a workaround.

I just reproduced again by switching to transformer tests from core-backend tests

I confirmed via printf debugging that the connect happens in javascript upon creating an empty snapshot, eventually cloudContainer.connect(this.workspace.cloudCache) is called in the ITwinWorkspaceContainer constructor upon adding a gcs workspace when loading the default databases for the "GeoCoordConfig".
So that looks normal. I will see if the workaround works so I can get back to what I was doing but this may need further debugging on the native side for a real fix.

Make sure you have these changes:
https://github.com/iTwin/itwinjs-core/pull/4108
https://github.com/iTwin/itwinjs-core/pull/4095
I'm available if you want help diagnosing.

confirmed switching to clang-7 from clang-13 didn't help.
I pulled native dependencies.
I also had to remove core/transformer/lib/cjs/test/standalone/.cache.
Some combination of the above means I can now reproduce with a locally built release-mode addon, debug probably works too and now I should be able to figure out the state of the addon and why pCont = 0
new release stack:
#0  __pthread_kill_implementation (no_tid=0, signo=6, threadid=140737348171712) at pthread_kill.c:44
#1  __pthread_kill_internal (signo=6, threadid=140737348171712) at pthread_kill.c:80
#2  __GI___pthread_kill (threadid=140737348171712, signo=signo@entry=6) at pthread_kill.c:91
#3  0x00007ffff7a96476 in __GI_raise (sig=sig@entry=6) at ../sysdeps/posix/raise.c:26
#4  0x00007ffff7a7c7b7 in __GI_abort () at abort.c:79
#5  0x00007ffff7a7c6db in __assert_fail_base (fmt=0x7ffff7c30770 "%s%s%s:%u: %s%sAssertion `%s' failed.\n%n",
    assertion=0x7ffd4e5dd4d9 "pCb->rc==SQLITE_OK && pCb->zErr==0",
    file=0x7ffd4e5a2298 "/home/mike/work/itwinjs-cospace/repos/imodel02/imodel02/iModelCore/BeSQLite/SQLite/blockcachevfs.c", line=533, function=<optimized out>) at assert.c:92
#6  0x00007ffff7a8de26 in __GI___assert_fail (assertion=0x7ffd4e5dd4d9 "pCb->rc==SQLITE_OK && pCb->zErr==0",
    file=0x7ffd4e5a2298 "/home/mike/work/itwinjs-cospace/repos/imodel02/imodel02/iModelCore/BeSQLite/SQLite/blockcachevfs.c", line=533, function=0x7ffd4e5dd4fc "void bcvfsFetchManifestCb(void *, int, char *, const u8 *, int)") at assert.c:101
#7  0x00007ffd48b38bdf in bcvfsFetchManifestCb ()
   from /home/mike/work/imodeljs/common/temp/node_modules/.pnpm/@bentley+imodeljs-native<EMAIL_ADDRESS>#8  0x00007ffd48b3da97 in bcvDispatchFree ()
   from /home/mike/work/imodeljs/common/temp/node_modules/.pnpm/@bentley+imodeljs-native<EMAIL_ADDRESS>#9  0x00007ffd489ba2d3 in bcvContainerFree ()
   from /home/mike/work/imodeljs/common/temp/node_modules/.pnpm/@bentley+imodeljs-native<EMAIL_ADDRESS>#10 0x00007ffd489bb211 in bcvfsAttach ()
   from /home/mike/work/imodeljs/common/temp/node_modules/.pnpm/@bentley+imodeljs-native<EMAIL_ADDRESS>#11 0x00007ffd489baed8 in sqlite3_bcvfs_attach ()
   from /home/mike/work/imodeljs/common/temp/node_modules/.pnpm/@bentley+imodeljs-native<EMAIL_ADDRESS>#12 0x00007ffd489716be in BentleyM0200::BeSQLite::CloudContainer::Connect(BentleyM0200::BeSQLite::CloudCache&)::$_3::operator()(char**) const (this=0x7fffffffbec0, msg=0x7fffffffbdf0)
    at /home/mike/work/itwinjs-cospace/repos/imodel02/imodel02/iModelCore/BeSQLite/CloudSqlite.cpp:249
#13 0x00007ffd489715f2 in std::__invoke_impl<int, BentleyM0200::BeSQLite::CloudContainer::Connect(BentleyM0200::BeSQLite::CloudCache&)::$_3&, char**>(std::__invoke_other, BentleyM0200::BeSQLite::CloudContainer::Connect(BentleyM0200::BeSQLite::CloudCache&)::$_3&, char**&&) (__f=..., __args=@0x7fffffffbd50: 0x7fffffffbdf0) at /usr/lib/gcc/x86_64-linux-gnu/11/../../../../include/c++/11/bits/invoke.h:61
#14 0x00007ffd48971582 in std::__invoke_r<int, BentleyM0200::BeSQLite::CloudContainer::Connect(BentleyM0200::BeSQLite::CloudCache&)::$_3&, char**>(BentleyM0200::BeSQLite::CloudContainer::Connect(BentleyM0200::BeSQLite::CloudCache&)::$_3&, char**&&) (__fn=..., __args=@0x7fffffffbd50: 0x7fffffffbdf0) at /usr/lib/gcc/x86_64-linux-gnu/11/../../../../include/c++/11/bits/invoke.h:114
#15 0x00007ffd48971462 in std::_Function_handler<int (char**), BentleyM0200::BeSQLite::CloudContainer::Connect(BentleyM0200::BeSQLite::CloudCache&)::$_3>::_M_invoke(std::_Any_data const&, char**&&) (__functor=..., __args=@0x7fffffffbd50: 0x7fffffffbdf0) at /usr/lib/gcc/x86_64-linux-gnu/11/../../../../include/c++/11/bits/std_function.h:291
#16 0x00007ffd4896d894 in std::function<int (char**)>::operator()(char**) const (this=0x7fffffffbec0, __args=0x7fffffffbdf0) at /usr/lib/gcc/x86_64-linux-gnu/11/../../../../include/c++/11/bits/std_function.h:560
#17 0x00007ffd4896c855 in BentleyM0200::BeSQLite::CloudCache::CallSqliteFn(std::function<int (char**)>, char const*) (this=0x50676c0, fn=..., funcName=0x7ffd4ea895cc "attach") at /home/mike/work/itwinjs-cospace/repos/imodel02/imodel02/iModelCore/BeSQLite/CloudSqlite.cpp:216
#18 0x00007ffd4896dc22 in BentleyM0200::BeSQLite::CloudContainer::Connect (this=0x55cbd10, cache=...) at /home/mike/work/itwinjs-cospace/repos/imodel02/imodel02/iModelCore/BeSQLite/CloudSqlite.cpp:249
#19 0x00007ffd482ab830 in IModelJsNative::JsCloudContainer::Connect (this=0x55cbcf0, info=...) at /home/mike/work/itwinjs-cospace/repos/imodel02/imodel02/iModelJsNodeAddon/JsCloudSqlite.cpp:197
#20 0x00007ffd482ac01f in Napi::details::TemplatedInstanceVoidCallback<IModelJsNative::JsCloudContainer, &IModelJsNative::JsCloudContainer::Connect>(napi_env__*, napi_callback_info__*)::{lambda()#1}::operator()() const (this=0x7fffffffc158) at /home/mike/work/itwinjs-cospace/repos/imodel02/BuildOutput/LinuxX64/static/BuildContexts/iModelJsNodeAddon/VendorAPI/Napi/node-src/napi-inl.h:181
#21 0x00007ffd482abdf6 in Napi::details::WrapCallback<Napi::details::TemplatedInstanceVoidCallback<IModelJsNative::JsCloudContainer, &IModelJsNative::JsCloudContainer::Connect>(napi_env__*, napi_callback_info__*)::{lambda()#1}>(Napi::details::TemplatedInstanceVoidCallback<IModelJsNative::JsCloudContainer, &IModelJsNative::JsCloudContainer::Connect>(napi_env__*, napi_callback_info__*)::{lambda()#1}) (callback=...) at /home/mike/work/itwinjs-cospace/repos/imodel02/BuildOutput/LinuxX64/static/BuildContexts/iModelJsNodeAddon/VendorAPI/Napi/node-src/napi-inl.h:79
#22 0x00007ffd482aba5a in Napi::details::TemplatedInstanceVoidCallback<IModelJsNative::JsCloudContainer, &IModelJsNative::JsCloudContainer::Connect> (env=0x4bf73b0, info=0x7fffffffc1d0) at /home/mike/work/itwinjs-cospace/repos/imodel02/BuildOutput/LinuxX64/static/BuildContexts/iModelJsNodeAddon/VendorAPI/Napi/node-src/napi-inl.h:178
#23 0x0000000000ab44dd in v8impl::(anonymous namespace)::FunctionCallbackWrapper::Invoke(v8::FunctionCallbackInfo<v8::Value> const&) ()
#24 0x0000000000d53cfe in v8::internal::MaybeHandle<v8::internal::Object> v8::internal::(anonymous namespace)::HandleApiCallHelper<false>(v8::internal::Isolate*, v8::internal::Handle<v8::internal::HeapObject>, v8::internal::Handle<v8::internal::HeapObject>, v8::internal::Handle<v8::internal::FunctionTemplateInfo>, v8::internal::Handle<v8::internal::Object>, v8::internal::BuiltinArguments) ()
#25 0x0000000000d5511f in v8::internal::Builtin_HandleApiCall(int, unsigned long*, v8::internal::Isolate*) ()
#26 0x00000000015f0b79 in Builtins_CEntry_Return1_DontSaveFPRegs_ArgvOnStack_BuiltinExit ()
#27 0x000000000158354a in Builtins_InterpreterEntryTrampoline ()
#28 0x00000ae2c2f41599 in ?? ()
...
#47 0x0000000001580b85 in Builtins_JSConstructStubGeneric ()
Backtrace stopped: frame did not save the PC

looks like it now receives a SIGABRT from failing the assert in bcvfsFetchManifestCb
assert( pCb->rc==SQLITE_OK && pCb->zErr==0 );

not sure why it was a segfault in the release build.

@kabentley still reproducing on the latest master itwinjs-core and native branch with those two PRs now definitely included on the js end:
node: /home/mike/work/itwinjs-cospace/repos/imodel02/imodel02/iModelCore/BeSQLite/SQLite/blockcachevfs.c:533: void bcvfsFetchManifestCb(void *, int, char *, const u8 *, int): Assertion `pCb->rc==SQLITE_OK && pCb->zErr==0' failed.
Aborted

I'm going to go find an addon that works for my CPU profiling and move on, but I think there are some people I can ask to take a look in the meanwhile.

@nick4598 can you see if any of the above makes sense to Dan?

Looks like gdb showing the stack going into v8::sample::SignalHandler::HandleProfilerSignal from the chrome CPU profiler is the SIGPROF signal which I was able to confirm from the siginfo_t in the handler. That makes sense, I'm just not familiar with profiler implementation. This signal is sent at some frequency (maybe by the OS, didn't look into how) for the profiler to use to gather (js) stacks and do its job.
I'm not sure if this interacts with the async (c-ares) curl dns resolution implementation, but I could not reproduce locally until I consumed the version of the curl native dependency that uses c-ares instead of the default non-threadsafe signal-based curl implementation of dns resolution.

running into this while trying to set up profiling of an iTwin.js process. Might have to try using windows as a workaround or look into this.

I have a fix for bcvfs's abort which was a dangling stack reference, I gave details to @nick4598 to pass on to Dan. But I haven't been able to pin down why curl doesn't successfully request. Verbose curl logging doesn't say anything and c-ares doesn't seem to have its own logging. I do see timeouts but I'm not sure if that's because of me stepping through code too slowly. Both gdb and lldb crash on parts of this code.


Fix path bug

Related Issues:

#166
#164

Changed:

Fixed path bug

I reinstalled and tried. The path error is gone and it is as stable as it was before the feature was modified.
Thank you for your support.


JavaEsSpark.esRDD can not get data from ES

Issue description
Hi,
I have tried to use JavaEsSpark.esRDD read data from ES for multiple times.
At this morning, the JavaEsSpark.esRDD can still read the data successfully.
However, at 3:00 P.M ，I notice that the interface can not read es data to rdd anymore. When I print out rdd.count, it always shows "0", which means the size of the rdd is 0.
What's more, there are no error logs, so I  could not even to debug.
Is there anyone meet the same problems?
Thanks

unfortunately without any error logs or reproduction steps, this issue isn't really actionable. If you have any updates on this issue, I'd recommend posting them on our forums and hopefully something will surface that can help you get back on track.

Hi,
Here I attach the logo printed. Not sure if they are helpful.
2018-06-04 10:10:08 INFO  org.elasticsearch.hadoop.util.Version.logVersion(Version.java:108) Elasticsearch Hadoop v7.0.0-alpha1-SNAPSHOT [df2e678939]
2018-06-04 10:10:09 INFO  org.elasticsearch.hadoop.rest.RestService.findPartitions(RestService.java:247) Reading from [test/people]
2018-06-04 10:10:09 INFO  org.apache.spark.internal.Logging$class.logInfo(Logging.scala:54) Starting job: collect at EsRddDemoTmp.java:74
2018-06-04 10:10:09 INFO  org.apache.spark.internal.Logging$class.logInfo(Logging.scala:54) Got job 0 (collect at EsRddDemoTmp.java:74) with 1 output partitions
2018-06-04 10:10:09 INFO  org.apache.spark.internal.Logging$class.logInfo(Logging.scala:54) Final stage: ResultStage 0 (collect at EsRddDemoTmp.java:74)
2018-06-04 10:10:09 INFO  org.apache.spark.internal.Logging$class.logInfo(Logging.scala:54) Parents of final stage: List()
2018-06-04 10:10:09 INFO  org.apache.spark.internal.Logging$class.logInfo(Logging.scala:54) Missing parents: List()
2018-06-04 10:10:09 INFO  org.apache.spark.internal.Logging$class.logInfo(Logging.scala:54) Submitting ResultStage 0 (JavaEsRDD[0] at RDD at AbstractEsRDD.scala:39), which has no missing parents
2018-06-04 10:10:10 INFO  org.apache.spark.internal.Logging$class.logInfo(Logging.scala:54) Block broadcast_0 stored as values in memory (estimated size 1432.0 B, free 912.3 MB)
2018-06-04 10:10:10 INFO  org.apache.spark.internal.Logging$class.logInfo(Logging.scala:54) Block broadcast_0_piece0 stored as bytes in memory (estimated size 967.0 B, free 912.3 MB)
2018-06-04 10:10:10 INFO  org.apache.spark.internal.Logging$class.logInfo(Logging.scala:54) Added broadcast_0_piece0 in memory on <IP_ADDRESS>:64574 (size: 967.0 B, free: 912.3 MB)
2018-06-04 10:10:10 INFO  org.apache.spark.internal.Logging$class.logInfo(Logging.scala:54) Created broadcast 0 from broadcast at DAGScheduler.scala:996
2018-06-04 10:10:10 INFO  org.apache.spark.internal.Logging$class.logInfo(Logging.scala:54) Submitting 1 missing tasks from ResultStage 0 (JavaEsRDD[0] at RDD at AbstractEsRDD.scala:39)
2018-06-04 10:10:10 INFO  org.apache.spark.internal.Logging$class.logInfo(Logging.scala:54) Adding task set 0.0 with 1 tasks
2018-06-04 10:10:10 INFO  org.apache.spark.internal.Logging$class.logInfo(Logging.scala:54) Starting task 0.0 in stage 0.0 (TID 0, localhost, executor driver, partition 0, ANY, 8423 bytes)
2018-06-04 10:10:10 INFO  org.apache.spark.internal.Logging$class.logInfo(Logging.scala:54) Running task 0.0 in stage 0.0 (TID 0)
2018-06-04 10:10:10 INFO  org.apache.spark.internal.Logging$class.logInfo(Logging.scala:54) Finished task 0.0 in stage 0.0 (TID 0). 1008 bytes result sent to driver
2018-06-04 10:10:10 INFO  org.apache.spark.internal.Logging$class.logInfo(Logging.scala:54) Finished task 0.0 in stage 0.0 (TID 0) in 314 ms on localhost (executor driver) (1/1)
2018-06-04 10:10:10 INFO  org.apache.spark.internal.Logging$class.logInfo(Logging.scala:54) Removed TaskSet 0.0, whose tasks have all completed, from pool
2018-06-04 10:10:10 INFO  org.apache.spark.internal.Logging$class.logInfo(Logging.scala:54) ResultStage 0 (collect at EsRddDemoTmp.java:74) finished in 0.335 s
2018-06-04 10:10:10 INFO  org.apache.spark.internal.Logging$class.logInfo(Logging.scala:54) Job 0 finished: collect at EsRddDemoTmp.java:74, took 0.610615 s


[ISSUE-#12] added basic layout for the blog page

Made a basic layout for the blog page/section. Wasn't sure about the color red for the card background.

This commit is failing. Can you try npm run build


Crash during editing in VS 2017

[Window Title]
devenv.exe
[Main Instruction]
An unexpected error occurred
[Content]
Please press Ctrl+C to copy the contents of this dialog and report this error to our issue tracker.
[V] Show details  [Close]
[Expanded Information]
Build: 15.6.18072.2

System.ArgumentException: Start and End must be well ordered
   at Microsoft.PythonTools.SourceSpan.ValidateLocations(SourceLocation start, SourceLocation end)
   at Microsoft.PythonTools.Intellisense.LocationTracker.Translate(SourceSpan span, Int32 fromVersion, Int32 toVersion)
   at Microsoft.PythonTools.Intellisense.VsProjectAnalyzer.ConvertNavigations(Navigation[] navigations, LocationTracker translator, Int32 fromVersion, ITextSnapshot snapshot)
   at Microsoft.PythonTools.Intellisense.VsProjectAnalyzer.ConvertNavigations(Navigation[] navigations, LocationTracker translator, Int32 fromVersion, ITextSnapshot snapshot)
   at Microsoft.PythonTools.Intellisense.VsProjectAnalyzer.<GetNavigationsAsync>d__169.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Microsoft.VisualStudioTools.UIThread.<>c__DisplayClass18_0`1.<<InvokeTask>b__0>d.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Microsoft.PythonTools.Navigation.DropDownBarClient.<RefreshNavigationsFromAnalysisEntry>d__36.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Microsoft.PythonTools.Navigation.DropDownBarClient.<Microsoft-PythonTools-Editor-IPythonTextBufferInfoEventSink-PythonTextBufferEventAsync>d__35.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Microsoft.PythonTools.Infrastructure.VSTaskExtensions.<HandleAllExceptions>d__5.MoveNext()

Thanks. This was a frequent issue in 15.6 but has been fixed in later 15.x release.


Fix embedded artifactory test: Increased wait startup timeout

superseded by https://github.com/Playtika/testcontainers-spring-boot/pull/730


panicked at 'NoopWake cannot wake'

I get that error. I used futures_util::compat::Compat01As03 to be able to use some futures 0.1 with await!().
Any ideas?

I've only had bad luck with futures_util::compat so I would recommend not using it.
Note that actix_web_async_await::await works with both futures 0.1 and futures 0.3.
If you need to use compat layers manually grab tokio-async-await and look in the compat module.

I'm just re-exporting await from that module. So yes, it should be working. If you can share code/errors I might be able to help.

I had the error when I was using the futures-util-preview crate with futures_util::compat but now it works if I use into_awaitable(). I saw that from https://jsdw.me/posts/rust-asyncawait-preview/.
Thanks!


New app build fails

Environment

react-native -v: 4.10.0
npm ls react-native-macos<EMAIL_ADDRESS>node -v: v12.16.2
npm -v: 6.14.4
yarn --version:
xcodebuild -version: Xcode 11.3.1 Build version 11C505

Issue
Steps to Reproduce

Create a new project via npx.

$ npx react-native init foo --version 0.61.5

$  cd foo

$ cd macos && pod install && cd ..

$ npx react-native run-macos

Expected Behavior

Expected application to run .
Actual Behavior
T

Build fails  with below output. One thing of note is the path in the error points to an incorrect path with `...Pods/Pods/...
`
info Found Xcode workspace "spawn.xcworkspace"
info Building (using "xcodebuild -workspace spawn.xcworkspace -configuration Debug -scheme spawn-macOS")
.......
error Failed to build macOS project. We ran "xcodebuild" command but it exited with error code 65. To debug build logs further, consider building your app with Xcode.app, by opening spawn.xcworkspace. Run CLI with --verbose flag for more details.
note: Using new build system
note: Planning build
note: Using build description from disk
error: /Users/fitzmode/spawn/macos/Pods/Pods/Target Support Files/Pods-Shared-spawn-macOS/Pods-Shared-spawn-macOS.debug.xcconfig: unable to open file (in target "spawn-macOS" in project "spawn") (in target 'spawn-macOS' from project 'spawn')
error: /Users/fitzmode/spawn/macos/Pods/Pods/Target Support Files/Pods-Shared-spawn-macOS/Pods-Shared-spawn-macOS.debug.xcconfig: unable to open file (in target "spawn-macOS" in project "spawn") (in target 'spawn-macOS' from project 'spawn')
error: /Users/fitzmode/spawn/macos/Pods/Pods/Target Support Files/Pods-Shared-spawn-macOS/Pods-Shared-spawn-macOS.debug.xcconfig: unable to open file (in target "spawn-macOS" in project "spawn") (in target 'spawn-macOS' from project 'spawn')

** BUILD FAILED **

Running through XCode returns the same error.

Reproducible Demo

(Paste the link to an example project and exact instructions to reproduce the issue.)
Additional context
(Write any other additional context that may help in root causing the issue.)

Hi @fitzmode ,
I was not able to reproduce the issue.   In your repro steps there is no npx react-native-macos-init call, but I assume you just accidentally omitted that step in your report.   If I do:

$ npx react-native init foo --version 0.61.5
$ cd foo
$ npx react-native-macos-init
$ cd macos && pod install && cd ..
$ npx react-native run-macos
It succeeds and the 'foo' app launches.   Also, note step 4 is not actually necessary anymore as react-native-macos-init now does the pod install automatically.

Similar here (https://microsoft.github.io/react-native-windows/docs/rnm-getting-started)
node -v: v16.2.0
npm -v: 7.13.0
yarn -v: 1.22.4
xcodebuild -version: Xcode 12.4 Build version 12D4e
macOS: Big Sure v11.4, MacBook Pro (15-inch, 2018)
npx react-native init foo --template<EMAIL_ADDRESS>
npx react-native-macos-init

pod install

npx react-native run-macos


Develop public library in stages

steps to follow are given below:
https://github.com/vamseeachanta/assetutilities/blob/2024/docs/_public_library.puml

@saiachanta2728 , @samdansk2

@vamseeachanta
Hello sir  ,  Created a repository ( https://github.com/samdansk2/LibraryTemplate )
But a small doubt is , In branch1 which yaml file i should read ? can you please tell me .

https://realpython.com/python-modules-packages/
Naming conventions:
https://realpython.com/python-pep8/
https://realpython.com/lessons/python-naming-conventions/#transcript

change name to "librarytemplate"

put the below basic functionality in .yml file and read it

2 levels of data
1 list
read a 1 .csv file

See some detailed instructions (especially the username/password) to release to pypi here.
https://github.com/vamseeachanta/py_package/blob/improvements/README.md#publishing-to-pypi

Do I need to change the password sir (or) enter password as " pypi- my account password " in place of password in cmd

I meant your password. On Apr 2, 2024, at 10:53 AM, Samdan Shaik @.***> wrote:
Do I need to change the password sir (or) enter password as " pypi- my account password " in place of password in cmd
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: @.***>

@vamseeachanta
i have tried the below option :
"Edit > Paste" from the window menu, or enabling " Ctrl+Shift+C/V as Copy/Paste" in "Properties".
even though it is not working sir , i don't know why this is happening ?

@vamseeachanta
i am attaching the error below , please see it sir


Show only titles on Non XML ERC875 asset cards

When an ERC875 token does not have an XML file, only show the title and blank out the other fields and icons which would normally have n/a

@James-Sangalli @zhangzhongnan928 is it worth display the token's bytes32 value?

@hboon not for now, also there would be an array of bytes32

@James-Sangalli yeah, but we don't bundle them so there's only 1 per card.


feat(spdk): updating to SPDK 23.01

Bdev API can only accessed from SPDK thread
New fields in various SPDK structures
New AIO Bdev options
New malloc Bdev options
Async Qpair connect and subsystem events are made optional features

bors try

bors try

bors merge

bors try

bors try

bors merge


feat(select): update search icon color based on state

Description
Update search icon color based on state to match StyledControlContainer’s border-color: changes to negative400 on error and primary400 on focus.

Scope

[ ] Patch: Bug Fix
[x] Minor: New Feature
[ ] Major: Breaking Change

Looked through the Figma and it looks like the current spec is inconsistent. There are more style updates going to be introduced to all inputs in the v7. Maybe we should hold off on this change or sync with the designers on it.

on second thought, should we merge this, to make sure we a fix for this in v6?

hi @leyanlo  - we've touched based on this with design, and they prefer to keep it grey, as in the near future, we'll change base web to use less blue

@nadiia @gergelyke Is there a way to do this with overrides today? I’m digging through the code, and it doesn’t seem possible to me unless I’m missing something. In renderSearch(), SearchIcon does not receive sharedProps, which has the $isFocused prop I need in order to do styling. Only StyledSearchIcon receives these props, but this component is not overridable.
Should I make another pull request exposing this StyledSearchIcon as a SearchIconContainer as an override?

Should I make another pull request exposing this StyledSearchIcon as SearchIconContainer as an override?

I think that makes perfect sense - @nadiia what do you think?


General Fixes

Fixed typos, removed redundancy, added spot in the readme for the most important part (bot setup on the portal), added Unlicense

Love the Changes, Thanks for Helping with it, @Tacusin made some seperate Changes Hoping he can throw a pull request too!


Kfctl implements new semantics tracking

[ ] Add a new flag -f #20
[ ] -f takes both local and remote files #21
[ ] Implements build #22
[ ] Implements apply #23

@gabrielwen Can we close this? I think this is implemented in v0.7.0?
Duplicate of kubeflow/kubeflow#3518 ?

/close


Fixes Give Images Border Width Issue

Added a range to the border  8px <= width <= 12px to allow different levels of zoom.
Tested locally against Chrome (Latest), Firefox (Latest), and IE 11.  Was unable to get it to fail the test with a border width of 10px.  Fails properly with borders widths outside the range.
closes #3204
closes #3904
closes #3804

Woohoo! This should clear up a lot of student issues!


Hang when using CkCallback.setRefnum

See #3157.

Is this distinct from #3158?

@evan-charmworks Oh didn't realize I had already created an issue for this. Closing.


chore: renaming graphql_frontend rfc directory with pull request number

Checklist

[x] The PR conforms to DataHub's Contributing Guideline (particularly Commit Message Format)
[ ] Links to related issues (if applicable)
[ ] Tests for the changes have been added/updated (if applicable)
[x] Docs related to the changes have been added/updated (if applicable)

Hey one thing -- do we need to update references inside of datahub-docs-website/sidebars.js to point to the new docs? See: https://github.com/linkedin/datahub/blob/master/docs-website/sidebars.js#L115

@jjoyce0510 as of #2191, we don't need to update sidebars.js for RFCs 😄