Microsoft.Intune application setting enforces "Users may register their devices with Azure AD" (poorly documented)

This page should also list that the Microsoft.Intune application, if it has "Enabled for users to sign-in?" property enabled, will force the "Users may register their devices with Azure AD" setting to "all" and grey it out. It would be very helpful for the page to list exactly how to reach the properties of the Microsoft.Intune application in Azure, because this isn't a straightforward process (Enterprise Applications, by default, do not list apps from Microsoft).

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

ID: a8119688-e88d-2916-3dcf-bf777a5fc80c
Version Independent ID: e767d695-1c9f-64e4-e6c4-fef07b0f3d2e
Content: Manage devices in Azure AD using the Azure portal - Microsoft Entra
Content Source: articles/active-directory/devices/device-management-azure-portal.md
Service: active-directory
Sub-service: devices
GitHub Login: @MicrosoftGuyJFlo
Microsoft Alias: joflore

@arp-mbender the article does state:

"...If you've configured either of these services, ALL is selected, and NONE is unavailable."

What do you mean about the Microsoft Intune application properties? Can you elaborate or provide an example of what you would like added?

My issue is with the bold fragment:

Users may register their devices with Azure AD: You need to configure this setting to allow users to register Windows 10 or newer personal, iOS, Android, and macOS devices with Azure AD. If you select None, devices aren't allowed to register with Azure AD. Enrollment with Microsoft Intune or mobile device management for Microsoft 365 requires registration. If you've configured either of these services, ALL is selected, and NONE is unavailable.

In our organization we're neither using Intune nor Microsoft Mobile Device Management, yet this setting was greyed out. What we had to do (after talking to Azure support) was find the Microsoft.Intune enterprise application (which is actually under the Microsoft application category, so it's not easily found) and then disable the Enabled for users to sign-in? setting under Properties. We have no idea how this setting got turned on, since we never used Intune and there appeared to be no settings in the Intune portal. My recommendation is to add this to the documentation since this setting is NOT easily found.

Got it. Let me see what I can do.

#label:"awaiting-product-team-response"

Reviewed with the Product Group and they advised it is a bug. Without Intune one should be able to set it to All/None. Contacting support in this case would be the correct course of action. #please-close


Feature alert config

@wistefan I will suggest we wait to land https://github.com/FIWARE-Ops/marinera/pull/56 in main first before we merge this, there are some fixes there and some conflicts as well...


Unable to add Custom Height and Width of Progress bar.

I am trying to make the progress bar smaller than the original one because by default its kinda big. I want to give custom height and width to progress bar but there is no such way.

Were you able to get the size you wanted by using the radius prop? If you're talking about the actual width of the circular line, that's set by the borderWidth prop.

No I am not talking about the radius prop. I am talking about the actual size of the progress bar which can be handled by giving custom height and width and that's been missing.

I think the radius props is used to adjust the size of the progress bar. I just tried it and it worked for me.


Additional elements in Leaves Hierarchy

Describe the bug
If there are elements in the leaves hierarchy that aren’t in any other hierarchy TM1py needs to create them explicitly.
To Reproduce
Source dimension to have additional elements in the Leaves hierarchy and not in any other hierarchies.
TM1py read the dimension and create a new copy of the dimension.
The resulting dimension will be missing the additional elements in the Leaves hierarchy.
Expected behavior
TM1py to create all the additional elements in the Leaves hierarchy when copying from another dimension
Version
TM1py 1,9,0
TM1 Server Version: = 11.8

Nice one. Thanks for the catch! We missed this edge case.
I created a fix. Please let me know if works for you:
pip uninstall tm1py
pip install https://github.com/cubewise-code/tm1py/archive/refs/heads/issue/create-elements-in-leaves-hierarchy.zip

Thanks that worked!


⚠ Moon has degraded performance

In f48218c, Moon (http://moon.sb) experienced degraded performance:

HTTP code: 200
Response time: 7164 ms

Resolved: Moon performance has improved in e632fd3.


A change corresponding to f4cea90

A traditional Chinese version of the updated README_ZH_CN.md.

Is this clearer/correct phrasing ? Can someone else verify? :smile:

The missing word '件' in traditional and simplified Chinese is the same.  This can be easily checked using Google Translate.  VundleVim/Vundle.vim#746 has been approved.

I did use google translate but didn't help me as it seemed to change the meaning in English.
Does adding this word makes the meaning clearer? If yes I will merge it

Does adding this word makes the meaning clearer?

The purpose of this pull request is the same as f4cea90 : to fix a typo.

I will take your word for this :smile:
Thanks


[suggestion] Adding book section

We should also include some books for learning competitive coding and for interview preparation such as Cracking the Coding Interview.

@Hina-softwareEngineer we added books section with direct download links.

@Hina-softwareEngineer we added books section with direct download links.


chore(apps/legacy-api): legacy-api health check

What this PR does / why we need it:
Adds _actuator/probes/... health check similar to Ocean API's
Which issue(s) does this PR fixes?:
Fixes #1060
Additional comments?:

Importing code directly from apps/ocean-api into apps/legacy-api causes some weirdness with Nest, hence duplicating the code is necessary at this stage --> trying to refactor

To deconflict with and merge after #1078

Would need this sooner than the rest to test out deployment on production.

Skipping the disconnected from ocean test for now (annotated with a TODO) because I'm not sure why jest.spyOn isn't calling the mock. Will address in separate PR.


do tree shaking as part of container build (feature request)

Some of my functions use pretty big modules (e.g. puppeteer, pdf gen, image manipulation). It would be useful to exclude them from the function containers that don't need them.

you can currently use "excludes" to remove files from the build.
functions:
  create:
    handler: functions/create.ts
    excludes:
      - afile
      - anotherflie

Does this help or are you wanting some other behaviour?

Great, that should do the trick. Thanks mate!


Repo with existing directory misleadingly fails with 500

Hi,

Gitea version (or commit ref):
Git version: 1.8.3
Operating system: Windows Server 2019 DS 1809
Database (use [x]):

[ ] PostgreSQL
[ ] MySQL
[x] MSSQL
[ ] SQLite

Can you reproduce the bug at https://try.gitea.io:

[ ] Yes (provide example URL)
[x] No (because i cant delete a repo without removing its directory there)
[ ] Not relevant

Description
we switched servers (and datbase backends from sqlite3 to mssql) but keeped the repository directories. I needed to recreate the former existing repository entries in gitea. After clicking "create repository" it resulted in a 500 error page. After refresh the empty repository was available.
In the logs I found this:
 2019/07/08 10:00:23 [...routers/repo/repo.go:146 handleCreateError()] [E] CreatePost: initRepository: initRepository: path already exists: d:\repos\secretuser\secretrepo.git
The error is misleading (and probably not wanted in that way) here as it works in the background. A notification message would be better to inform that the directory is already existing.

I have same error with empty MSSQL database and initial Gitea config. I tried add LOCAL_ROOT_URL   = http://<IP_ADDRESS>:3000 to config, change permissions, database but still same error.

@dimmuboy Did you also have an exist repository on the same path as your created?

@dimmuboy Did you also have an exist repository on the same path as your created?

No I created completely new repo dir

@dimmuboy Did you also have an exist repository on the same path as your created?

No I created completely new repo dir

So you should have a different issue as this one.

So you should have a different issue as this one.

Yes but IMHO it will be problem with database because main dir sctructure "repository/username/" was created. In original post @commel wrote about switch to MSSQL like me.

It has been fixed #24164 by the way


Remix error using concat in getErrorCodeMsg

https://github.com/Zondax/filecoin-solidity/blob/36bb7854dc2242bf19c5e88c61f02ea8eb9f871b/contracts/v0.8/utils/Actor.sol#L76-L78
A hackathon developer (DÉX_ZËRØ) noticed here in Discord that these lines cause errors in Remix:

"changed the code. Used Abi.encodePacked for low gas consumption. and now it compiled"

:link: zboto Link

Closed by #175


Display cancelled jobs as gray

In the pipelines list, in the following graph, you can't tell which jobs actually failed and which jobs were cancelled because another merge happened before the tests finished. We want to be able to notice failures of tests on master just by looking at the graph, without having to hover or click. It would be really useful if those builds showed up in a different color and I propose gray.

I would like to see this feature too. Also the criss or tick symbol aside the run in the pipeline view could also be gray...


WidgetSpan breaks SkShaper/SkParagraph

The font we are using in our application has issues when rendered by LibTxt/Minikin, but looks great using Skia SkParagraph. So I built a local version of the flutter engine using the --enable-skshaper and --always-use-skshaper flags. Now anytime I try to use a WidgetSpan I get the following error:
../../third_party/skia/modules/skparagraph/src/ParagraphImpl.cpp:838: fatal error: "assert(blockRange.start < fTextStyles.size() && blockRange.end <= fTextStyles.size())"
* thread #10, name = 'io.flutter.1.ui', stop reason = EXC_BREAKPOINT (code=1, subcode=0x10c36e300)
    frame #0: 0x000000010c36e300 Flutter`sk_abort_no_print() at SkMemory_malloc.cpp:49:5
   46  	#if defined(SK_DEBUG) && defined(SK_BUILD_FOR_WIN)
   47  	    __fastfail(FAST_FAIL_FATAL_APP_EXIT);
   48  	#elif defined(__clang__)
-> 49  	    __builtin_trap();
    	    ^
   50  	#else
   51  	    abort();
   52  	#endif

Steps to Reproduce

Build the flutter engine using --enable-skshaper and --always-use-skshaper
Stick a WidgetSpan inside some RichText
Watch the crash and burn

Logs

✓] Flutter (Channel stable, 2.5.3, on macOS 12.0.1 21A559 darwin-x64, locale en-US)
    • Flutter version 2.5.3 at /Users/michaelbrown/Documents/code/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 18116933e7 (5 weeks ago), 2021-10-15 10:46:35 -0700
    • Engine revision d3ea636dc5
    • Dart version 2.14.4

[!] Android toolchain - develop for Android devices (Android SDK version 29.0.3)
    • Android SDK at /Users/michaelbrown/Library/Android/sdk
    ✗ cmdline-tools component is missing
      Run `path/to/sdkmanager --install "cmdline-tools;latest"`
      See https://developer.android.com/studio/command-line for more details.
    ✗ Android license status unknown.
      Run `flutter doctor --android-licenses` to accept the SDK licenses.
      See https://flutter.dev/docs/get-started/install/macos#android-setup for more details.

[✓] Xcode - develop for iOS and macOS
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 13.1, Build version 13A1030d
    • CocoaPods version 1.10.1

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 3.6)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 1.8.0_212-release-1586-b4-5784211)

[✓] VS Code (version 1.62.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.28.0

[✓] Connected device (2 available)

Hi @mimbrown, thanks for filing the issue.
What platform did you compile for?
Can you also provide the full output of flutter doctor -v, the output is likely to be huge so kindly paste it into a .txt file and attach it.
Also, please provide a complete minimal reproducible code sample for step 2.
Thank you


Using jupyter notebook instead of markdown will be more interactive

Hello, @Avik-Jain
Your repository is really fantastic. Thank you for your work and passion 👍
I have used markdown and jupyter notebook for code presentation. And the jupyter notebook is more interactive than markdown file. It can show more intermediate result and in some aspect, the jupyter notebook contains all the feature of markdown.
I am passionate to the machine learning and your repository, too. Would you mind if I get involved for more interactive and detailed construction with jupyter notebook?

You can definitely create some detailed and descriptive notebooks, they will be helpful for the beginners.

thanks for your nice project .  I have waited for the others, hoping you to update your project quickly


problem with break in evaluate

hi, the following block in the kernel's evaluate function is faulty. Btw, just swapping the "break" for a "continue" won't be enough to fix it. Commenting out the block is a temporary fix
			int all_exclude = 1;
			for (int la_chunk = 0; la_chunk < LA_CHUNKS-1; ++la_chunk)
			{
				if (ta_state[la_chunk*STATE_BITS + STATE_BITS - 1] > 0)
				{
					all_exclude = 0;
					break;
				}
			}
			if ((ta_state[(LA_CHUNKS-1)*STATE_BITS + STATE_BITS - 1] & FILTER) > 0)
			{
				all_exclude = 0;
			}
			if (all_exclude)
			{
				break;
			}

Nice catch, @pedritolo! That bug caused the remaining clauses in the outer for-loop to not be evaluated. If a clause is "all exclude" it should not take part in the evaluation at all. However, the outer for-loop should continue to the next clause. Fixed by changing the last 'break' to 'continue'.


Add Scoping & BindingState documentation

@tgrapperon @mbrandonw @stephencelis I basically plagiarized what @tgrapperon said in https://github.com/pointfreeco/isowords/issues/170 . Hopefully it helps prevent others from finding the issue I encountered earlier :)

Hey @acosmicflamingo! I'm not 100% sure that this change is required, or at least in this form or in this spot.
BindingReducer (more precisely BindingActions) has some internal logic that detects if the BindingAction is expressed (that is, if the BindableState is mutated). So, if an action goes through without "its" BindingReducer catching and expressing it, it shows a message. Most often, everything is correctly wired, but you forgot to add the BindingReducer to your feature's body. But once you've checked that the BindingReducer is well installed into your feature, the next logical explanation is that your feature's Reducer didn't run at all. Most often, it's because you forgot to embed/scope this feature's reducer into its parent. But this mistake is absolutely general and can happen for any feature of your app. For this reason, it seems rather odd to mention this issue in this section dedicated to bindings.
So I guess, if there's something that can be improved, it's probably the warning message that can suggest checking if the feature hosting the BindingReducer is correctly embedded into its parent. But I would prefer Brandon or Stephen to weigh in on the subject, as I'm not really sure about the correct wording.

@tgrapperon I suppose it's not obvious that the example I use is a parent reducer (which is why I didn't include the BindingReducer there). I amended the commit to make this more clear but even then, I do agree with you that the best place for something like this would actually be in a warning! Developers will certainly be more likely to address the problem by just reading the warning, checking to see if it applies to them, and then fixing it if it does, than to open up the BindingState documentation. In either case, I can update this PR accordingly, and Brandon or Stephen can chime in with their thoughts :)

Hi @acosmicflamingo! Sorry for the delay. This came along before the holidays and we lost track of it. The isowords bug slipped in during refactoring, but we resolved it when we came back to it recently :smile:
I think we're going to hold off on Scope-specific documentation here, since it would apply to basically every helper in the library already. I am curious, though, if you had encountered this in a project of your own (and not a bug in our project), if the existing docs/warning would have been enough?

@stephencelis no need to apologize; I know you all are really busy and this isn't a high-priority issue, and thanks for responding!
To answer your question, I only encountered it in my project because I was using isowords source code as a template, so I think what you have is good enough. If I do encounter this kind of problem again in the future however, then I'll just create a new PR and reference this one.

ใครมาชะเมิดด้านข้อมูล


[flutter_tools] add status logs to determine where test is getting stuck

Description
This test has been flaking more recently, see https://cirrus-ci.com/task/5615992101404672?command=main#L253 . Add status logs to determine where it is getting stuck, since this doesn't reproduce locally.

@brightest-dazes please stop approving PRs that you are not reviewing or don't have context for. If you want to contribute to the Flutter framework I would start with https://github.com/flutter/flutter/blob/master/CONTRIBUTING.md

This test is getting stuck on CI, but we're not sure where. I added a print before each await so we can see where the test is running


investigate why concs are limited at ~10

maybe try without log

look at balfrin model output and mlflow

The issue was that the concentrations of ALNU and CORY were log, before normalization. Not taking the log fixes this issue and produces very reasonable results!


Compatibility with 0.2.0

Hello,
I have an issue using ember-cli-flash with ember-cli 0.2.0. It was installed with ember install:addon ember-cli-flash
The error is javascript Uncaught Error: Could not find module `ember-cli-flash/services/flash-messages-service` imported from `xk21/initializers/flash-messages-service`

sorry, it was caused by incompatible static asset


R.swift.Library v1.2.1 should be v1.3.0

As a result of the change in the library to use Swift 2.2 syntax.  It is not possible to stay update pods while staying on Swift 2.1 without specifying specific version of both R.swift and R.swift.Library.  This is because restricting R.swift to version 1.4.0 will still use R.swift.Library 1.2.1 since it uses the optimistic operator for version comparison so 1.2.1 satisfies the constraint '~> 1.2.0'.  Since this was a breaking change, the R.swift.Library should have become 1.3.0 and and potentially R.swift becomes 1.5.0.  The only way to fix this now would be either to republish what is 1.2.1 as 1.3.0 and remove the current 1.2.1 or publish what was 1.2.0 as 1.2.2 in addition to republishing 1.2.1 as 1.3.0

As a workaround:
You can specify the exact R.swift.Library version in the Podfile:
pod 'R.swift', '1.4.0'
pod 'R.swift.Library', '1.2.0'

So sorry about this! We'll work on fixing this by republishing as soon as possible.

Mea culpa, I rushed the new version out without taking versioning into account well enough. No I think about it this even should be a major release, since it is a breaking change.
This is what I'm going to do to fix this:

Release v1.4.2 that is compatible with Swift 2.1 (this will be exactly the same as 1.4.0)
Release v2.0.0 that is the Swift 2.2 version (this will be exactly the same as 1.4.1)

I'm currently in the process of releasing those versions.


WIP: Fixes #90 - Ignore interface typehinted functions

Looking good! Don't sweat the build failures, being dealt with in #92. :)

Er, #91. ;-)

I trust your tests are failing due to Symfony 4 stuff, not the new check 😆
Thanks a ton for picking this up and for the quick action, I'll tag a new release with this included later this week.

You're welcome. Thanks for your effort in this bundle!
I'll use dev-master for now. Thanks @rosstuck !


Hacktoberfest 2024 - Add take action to auto assign contributors, updated

This pull request introduces a new GitHub Actions workflow to automatically assign issues to contributors when they comment during the month of October. The workflow also logs when the action is skipped outside of October. The update also resolves bug that previously made action sometimes fail.
New GitHub Actions workflow:

.github/workflows/take.yml: Added a workflow named "Auto-assign issue to contributor" that triggers on issue comments. It includes two jobs:

assign: Assigns the issue to the contributor if the comment is made in October.
log_out_of_october: Logs a message indicating the action was skipped if the comment is made outside of October.

Updated with correct file & update to fix bug.


Why is the deployed token always called 'CustomToken'?

deploy_token_contract returns a dictionary that maps 'CustomToken' to the address of the token contract.  Shouldn't the dictionary map the specified name of the contract like 'TestToken' below?
    deployed_token = deploy_token_contract(deployer, tokens, TOKEN_DECIMALS, 'TestToken', 'TTT')

    token_address = deployed_token['CustomToken']

I guess the names given in the command line options should be used.

And does anybody rely on this particular naming?

There's no specific meaning to this name. It's just default value for the token_type parameter.


Support React 16

Could you please update the package.json file with newer version of react-transition-group ?
The version 1.2.1 supports React 16.
UPDATE
Noticed that React 16 is supported in master branch, so we just need a new release (tag).

Sure, should be able to get this out soon. AFAIK react-transition-group
didn’t make any code changes for 16 support, so the benefit is just
clearing up the peer dependency warnings.
On Wed, Oct 11, 2017 at 5:25 PM DrGor<EMAIL_ADDRESS>wrote:

Could you please update the package.json file with newer version of
react-transition-group ?
The version 1.2.1 supports React 16.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/google-fabric/velocity-react/issues/229, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AAP65Ue1oYBtkE2CgJ8pK-7NEWaQP85aks5srTJUgaJpZM4P2JlB
.

Any update on upgrading react-transition-group? The latest version is at 2.2.1

While I expect to upgrade to the most recent 1.x version to fix the peer
dependency, I would not expect to ever upgrade to 2.x.
I’ve looked into this in the past and as I remember it would require
changing a lot and it wasn’t actually a good fit for how this library
operates. To do Velocity tricks like stagger we need to have a top-level
approach to creating animations.
On Wed, Feb 21, 2018 at 7:16 PM Rory MacQueen<EMAIL_ADDRESS>wrote:

Any update on upgrading react-transition-group? The latest version is at
2.2.1
Agreed. v2 of react-transition-group is much more flexible and powerful
and it's cumbersome to have to support both versions in the yarn.lock
file.
Migrating to v2 should be fairly straightforward:
https://github.com/reactjs/react-transition-group/blob/master/Migration.md
I think it's just a matter of wrapping the child elements of
TransitionGroup in a Transition component and then passing the props down.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/google-fabric/velocity-react/issues/229#issuecomment-367523282,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAP65Q6Erjn94iwKC3ts1uWeG56z-prcks5tXLHEgaJpZM4P2JlB
.


[Chat] Android - Chat - User not returned to chat screen after viewing profile details and tapping X

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Expected Result:
User returned to Chat screen
Actual Result:
User returned to LHN
Action Performed:

Launch the app and login
Tap on a chat from LHN ( One on One Chat)
Tap on user name on the top to view profile details
4.Tap on the X to close the details

Workaround:
Unknown
Platform:
Where is this issue occurring?
Web
iOS
Android ✔
Desktop App
Mobile Web
Version Number:
Device and OS: Pixel 4 / Android 11
Build version : 1.0.7-0
Reproducible on production: No
Tester login credentials - Password	: Utest123
Tester login credentials - Email/Phone<EMAIL_ADDRESS>Notes/Photos/Videos: Any additional supporting documentation
Video
Expensify/Expensify Issue URL:

Triggered auto assignment to @KirkBarrett (AutoAssignerTriage), see https://stackoverflow.com/c/expensify/questions/4749 for more details.

Pretty sure this has been resolved. Feel free to reopen if that's incorrect!


Division operator in conditions interpreted as start-of-regex

I have this for iteration which I would like to use to detect if at least half of these strings $s1 are within 100 bytes of each other:
for (#s1/2) i in (1..#s1) : (@s1[i+1] - @s1[i] < 100)
However, this does not work because the / is interpreted as the beginning of a regex.
Yet, the following would work:
for (#s1+2) i in (1..#s1) : (@s1[i+1] - @s1[i] < 100) // 2 more than the number of $s1
for (#s1*2) i in (1..#s1) : (@s1[i+1] - @s1[i] < 100) // 2x more than the number of $s1

The division operation in YARA is a backslash () like the integer division in C.


Radio input inside swiper slide

Check that this is really a bug

[X] I confirm

Reproduction link
https://codepen.io/adasdad/pen/OJOaGZd
Bug description
Hi,
Swiper 8.0.6
If you add radio inputs inside the slide (and any form elements - buttons, checkboxes, text inputs, etc.) and for example click on the radio in the 4th slide, then the carousel "jumps" and makes the slide in which you checked the input active and moves it to the first position.
Before that, I used Swiper 6.6.2 of the swiper and there was no such problem.
Expected Behavior
No response
Actual Behavior
No response
Swiper version
8.0.6
Platform/Target and Browser Versions
Windows 10
Validations

[X] Follow our Code of Conduct
[X] Read the docs.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.
[X] Make sure this is a Swiper issue and not a framework-specific issue

Would you like to open a PR for this bug?

[ ] I'm willing to open a PR

I have the same problem with the Select tag

I could not solve it, I returned to v6.6.2

Yes, on 6.8.4 it is working. Thank you!

#5524

I have a solution that mobile touch on ui elements like links or select fields no longer makes the slide jump to first place. Touch events also trigger a focus event. I added a condition to check that the focus event was not triggered by touch.
#6139

also same issue in v10.2.0

Issue is closed because of outdated/irrelevant/not actual/needed more information/inactivity.
If this issue is still actual and reproducible for latest version of Swiper, please create new issue and fill the issue template correctly:

Clearly describe the issue including steps to reproduce when it is a bug.
Make sure you fill in the earliest version that you know has the issue.
Provide live link or JSFiddle/Codepen or website with issue


Textualizer: LionWeb.Core.UnsetFeatureException

Textualizer.AsString(node);  throws the following exception:
  LionWeb.Core.UnsetFeatureException
  Required Property name is not set
...
   at LionWeb.Core.Utilities.Textualizer.<AsStrings>g__ReferenceTargetAsString|4_2(INode target)
   at System.Linq.Enumerable.SelectIListIterator`2.Fill(IList`1 source, Span`1 results, Func`2 func)
   at System.Linq.Enumerable.SelectIListIterator`2.ToArray()
   at LionWeb.Core.Utilities.Textualizer.Join(IEnumerable`1 strings, String separator)
   at LionWeb.Core.Utilities.Textualizer.<>c__DisplayClass4_0.<AsStrings>g__ReferenceAsString|3(Reference reference)
   at System.Linq.Enumerable.SelectEnumerableIterator`2.MoveNext()
   at System.Collections.Generic.List`1.AddRange(IEnumerable`1 collection)
   at LionWeb.Core.Utilities.Textualizer.AsStrings(INode node)
   at LionWeb.Core.Utilities.Textualizer.AsString(INode node)
...

@erk-dik Can you implement a failing unit test for this somewhere?

@erk-dik @JojoEffect Should be fixed by PR #10 .

@erk-dik Can I make you collaborator of this repo? Then I can request reviews of PRs from you.

Sure  @dslmeinte

Sure @dslmeinte

Invite sent :)


Fix for compound nested list issues, passes regression tests.

Fix for compound nested list issues, passes regression tests.
License statement:
I am the original author of this fix and make it available under the same license as pegdown.
Fixes issues: #57, #123
This PR has the fixed list parsing that includes tight and loose sub-list handling that the previous fix did not address. It also does not mess up definitions because these relied on the old parser rule for list parsing. Definition list rule is now a copy of the old list parsing rule so that regression tests pass.
Change to pegdown.spec: AstText.ast changed to reflect that the new parsing of list items does not include the trailing blank lines in the item. This makes the new behavior leave out a blank line of 2 spaces at the end of the list causing a discrepancy of 3 characters for the BulletListNode. Expectation file modified to reflect new reality.
I'll create another PR in a day or two with new tests to validate compound nested lists.
sbt test result summary:
[info] Passed: Total 19, Failed 0, Errors 0, Passed 19
[success] Total time: 2 s, completed 14-Aug-2015 12:04:09 AM

Output results of parsing compound nested list. Minor differences with GitHub which makes list items above and below a blank line loose. Pegdown only makes the item following a blank line loose. It's a toss up but I think pegdown's results are more intuitive:
For:
2. XYZ 1

    XYZ 1's paragraph

    1. A 1
    2. A 2

    3. A 3

GitHub Produces:

XYZ 1
XYZ 1’s paragraph

A 1

A 2

A 3
A 3’s paragraph

while pegdown produces what you would expect by looking at the markdown source:

XYZ 1
XYZ 1’s paragraph

A 1
A 2

A 3
A 3’s paragraph

Full output: raw markdown, markdown - to let GitHub convert to HTML, HTML - converted by pegdown to render by GitHub for comparison, raw HTML for the curious.
-- raw markdown -------------------------------------------

1. B 1

    1. A 1

    2. A 2

1. B 1
    1. A 1

    2. A 2

1. B 1

    1. A 1
    2. A 2

2. XYZ 1

    XYZ 1's paragraph

    1. A 1
    2. A 2

    3. A 3

        A 3's paragraph

        - XYZ
            1. abc
            1. abc
            1. abc
        - XYZ
            - abc

                ```
                fenced code block of sub-sub-list abc
                ```

            - abc
            - abc
        - XYZ

        Lorem ipsum...

2. XYZ 2

    1. A 2
        - XYZ 2

 1. List 1 Item 1

    - sub list item 1
        sub list item 1 continuation line

            sub list item 1 code block 1

        sub list item 1 paragraph

            sub list item 1 code block 2

    - sub list item 2

    List 1 Item 1 paragraph

        List 1 Item 1 code block

    List 1 Item 1 paragraph or main code block

-- markdown -----------------------------------------------

B 1

A 1

A 2

B 1

A 1

A 2

B 1

A 1
A 2

XYZ 1
XYZ 1's paragraph

A 1

A 2

A 3
A 3's paragraph

XYZ

abc
abc
abc

XYZ

abc
fenced code block of sub-sub-list abc

abc

abc

XYZ

Lorem ipsum...

XYZ 2

A 2

XYZ 2

List 1 Item 1

sub list item 1
sub list item 1 continuation line
  sub list item 1 code block 1

sub list item 1 paragraph
  sub list item 1 code block 2

sub list item 2

List 1 Item 1 paragraph
List 1 Item 1 code block

List 1 Item 1 paragraph or main code block

-- HTML ---------------------------------------------------

B 1

A 1

A 2

B 1

A 1

A 2

B 1

A 1
A 2

XYZ 1
XYZ 1’s paragraph

A 1
A 2

A 3
A 3’s paragraph

XYZ

abc
abc
abc

XYZ

abc
fenced code block of sub-sub-list abc

abc
abc

XYZ

Lorem ipsum…

XYZ 2

A 2

XYZ 2

List 1 Item 1

sub list item 1sub list item 1 continuation line
sub list item 1 code block 1

sub list item 1 paragraph
sub list item 1 code block 2

sub list item 2

List 1 Item 1 paragraph
List 1 Item 1 code block

List 1 Item 1 paragraph or main code block

-----------------------------------------------------------
-- raw HTML -----------------------------------------------

B 1

A 1

A 2

B 1

A 1

A 2

B 1

A 1

A 2

XYZ 1

XYZ 1’s paragraph

A 1

A 2

A 3

A 3’s paragraph

XYZ

abc

abc

abc

XYZ

abc

fenced code block of sub-sub-list abc

abc

abc

XYZ

Lorem ipsum…

XYZ 2

A 2

XYZ 2

List 1 Item 1

sub list item 1
sub list item 1 continuation line

sub list item 1 code block 1

sub list item 1 paragraph

sub list item 1 code block 2

sub list item 2

List 1 Item 1 paragraph

List 1 Item 1 code block

List 1 Item 1 paragraph or main code block

Have a new PR with new tests and updated regression tests for fixes in the parser.


Adding support for data type validation for each property.

int

basic type validation for int
constrained validation, using operators such as >, >=, <, <=, ==, even, odd, divisible by k, one of 27, 93, 35 ..., range from x to y.

str

basic type validation for str
constrained validation, such as a list of constant status string like SUCCESS, FAILED.
pre-defined types, such as email, url, ipv4, ipv6, uuid
regex matching.

bool

basic type validation for bool

float

TBD

complex

TBD

Implemented type checking feature


PERM: operation not permitted, scandir.    error errno -4048

I have seen this errror in the issues in other spots from several years ago.  I am not doing anything tricky here.  Just a npm install.
Troubleshooting
Checkout how to troubleshoot failures and collect debug logs: https://docs.microsoft.com/en-us/vsts/build-release/actions/troubleshooting
Environment

Server - Azure Pipelines or TFS on-premises?
Azure Pipleines
https://microsoft.visualstudio.com/Xbox.Ambassadors/Xbox.Ambassadors Team/_build/results?buildId=13376049

Agent - Hosted or Private:

Hosted.   I tried Hosted VS 2017 and Hosted Mac
XboxAmbassadors agent
https://microsoft.visualstudio.com/Xbox.Ambassadors/_settings/agentqueues?_a=agents&queueId=2400
Issue Description
Running npm install is failing.  I was able to get it to suceed once but keeps failing with same error.
Task logs
[Enable debug logging and please provide the zip file containing all the logs for a speedy resolution]
Error logs
2018-12-13T16:57:33.3710096Z 38948 verbose unlock done using C:\Windows\ServiceProfiles\NetworkService\AppData\Roaming\npm-cache_locks\staging-7a43e413792d6dbe.lock for C:\buildagent_work\7\s\Microsoft.Ambassadors.Website\node_modules.staging
2018-12-13T16:57:33.3710096Z 38949 verbose stack Error: EPERM: operation not permitted, scandir 'C:\buildagent_work\7\s\Microsoft.Ambassadors.Website\node_modules\fsevents\node_modules'
2018-12-13T16:57:33.3710096Z 38950 verbose cwd C:\buildagent_work\7\s\Microsoft.Ambassadors.Website
2018-12-13T16:57:33.3710096Z 38951 verbose Windows_NT 6.3.9600
2018-12-13T16:57:33.3710096Z 38952 verbose argv "C:\buildagent\_work\_tool\node\8.9.1\x64\node.exe" "C:\buildagent\_work\_tool\node\8.9.1\x64\node_modules\npm\bin\npm-cli.js" "install"
2018-12-13T16:57:33.3710096Z 38953 verbose node v8.9.1
2018-12-13T16:57:33.3710096Z 38954 verbose npm  v5.5.1
2018-12-13T16:57:33.3710096Z 38955 error path C:\buildagent_work\7\s\Microsoft.Ambassadors.Website\node_modules\fsevents\node_modules
2018-12-13T16:57:33.3710096Z 38956 error code EPERM
2018-12-13T16:57:33.3710096Z 38957 error errno -4048
7.txt
2018-12-13T16:57:33.3710096Z 38958 error syscall scandir
2018-12-13T16:57:33.3710096Z 38959 error Error: EPERM: operation not permitted, scandir 'C:\buildagent_work\7\s\Microsoft.Ambassadors.Website\node_modules\fsevents\node_modules'
2018-12-13T16:57:33.3710096Z 38959 error  { Error: EPERM: operation not permitted, scandir 'C:\buildagent_work\7\s\Microsoft.Ambassadors.Website\node_modules\fsevents\node_modules'
2018-12-13T16:57:33.3710096Z 38959 error   stack: 'Error: EPERM: operation not permitted, scandir 'C:\buildagent\_work\7\s\Microsoft.Ambassadors.Website\node_modules\fsevents\node_modules'',
2018-12-13T16:57:33.3710096Z 38959 error   errno: -4048,
2018-12-13T16:57:33.3710096Z 38959 error   code: 'EPERM',
2018-12-13T16:57:33.3710096Z 38959 error   syscall: 'scandir',
2018-12-13T16:57:33.3710096Z 38959 error   path: 'C:\buildagent\_work\7\s\Microsoft.Ambassadors.Website\node_modules\fsevents\node_modules' }
2018-12-13T16:57:33.3710096Z 38960 error Please try running this command again as root/Administrator.
2018-12-13T16:57:33.3710096Z 38961 verbose exit [ -4048, true ]
2018-12-13T16:57:33.3710096Z

Actually my Agent was not set right.  It is running better on Hosted agent.


Get rid of unsafe, by storing two timestamps in each tsValue struct.

This fixes the data corruption problem in scotty.

Closing as we decided to take the simpler approach of 42 timestamps per page instead of the 84 timestamps for page that this PR offers.


Added logging for view compilation

Issue - #3755
@pranavkm @rynowak

Finally!

These just need timings and it looks perfect :smile:

:watch:

Updated.

:watch: for the log messages.

Updated.

:shipit:

@ajaybhargavb Can you paste a sample of console output?

@davidfowl


mind your cache

Hi, I'm trying to learn Leptos right now, but when I tried to run it's examples on a local machine I've seen just the leftovers from some eframe stuff I was doing before. When I refreshed the page that stuff was still there despite active Trunk with a Leptos app. Only force update helped (Ctrl+Shift+R).
I'm not saying that there is anything wrong with this template, but I encountered some confusion so I decided to share it. Also I know that there is code in this repo that is responsible for making cache work, maybe it's not 100% correct? is it possible to only ship it in release builds?
Browser: Firefox 113.0.1

I often get confused by this too. This is why there is a warning in the README about caching behavior and how to get around this.

assets/sw.js script will try to cache our app, and loads the cached version when it cannot connect to server allowing your app to work offline (like PWA). appending #dev to index.html will skip this caching, allowing us to load the latest builds during development.

Thanks for posting this. In my app it doesn't make sense to have a service worker so I just deleted that <script> in index.html.

Note, the cache was disabled in https://github.com/emilk/eframe_template/pull/102, so this issue can be closed. In fact, the service-worker currently does nothing but throw harmless errors.


fix: remove unnecessary dependencies to promises and @sentry/wizard

Sentry/capacitor had a runtime dependency on Sentry/wizard, which then depends on rxjs through inquirer.
This cause issues in monorepo configurations, for instance : https://github.com/ReactiveX/rxjs/issues/6628 and https://github.com/webpack/webpack/issues/14405.
Since Sentry/wizard is more or less a CLI tool, it is not required as a runtime dependency.

wizard may be required once https://github.com/getsentry/sentry-capacitor/issues/39 is implemented.

@olivier-blanc-openit could u please add a changelog entry?

Also, I believe the wizard should become a dev dependency then, no?

An update: Sentry Wizard will be supported by Capacitor, the promises package will be removed from the dependencies and Sentry Wizard will be moved to the devDependencies. All that after the integration with the Wizard is completed

An update: Sentry Wizard will be supported by Capacitor, the promises package will be removed from the dependencies and Sentry Wizard will be moved to the devDependencies. All that after the integration with the Wizard is completed

Should we reopen this issue? The stale bot closed it.

This PR will move to https://github.com/getsentry/sentry-capacitor/pull/400
Thank you @olivier-blanc-openit  for the contribution!


Add apis to unconditionally check attributes for an element

I think one additional change you might want to make here is to add an actual diffing step against the DOM.
For example, with this change, I believe if you have a Element that uses value as a property, this will cause a write to the value property, even if the value is the same. This can cause an <input>'s cursor position to be changed in some browsers (I forget which).
I think there are a couple of ways to handle this:

Pass the alwaysDiffAttributes through to queueChange, and then to the actually function that applies the attribute/property
Before running the diff, for all the cached attributes / properties, fetch the current value from the DOM node

The latter is a bit difficult, since we would need to know how the property is configured (which is currently decided when we go to apply the diff). The former has a slight caveat that anyone using this API with a custom hook for applying a value might want to add a secondary diff check.
For the former case, you should only check the value from the DOM if you are falling under this path, since we do not check the DOM as a performance shortcut.

Hmm so the case we are currently looking at is only looking at attributes like aria-label. Perhaps we can leave it as a TODO when this situation arrives?
The actual scenario we are trying to fix is basically manual dom manipulations not being represented in the cache.

Hmm so the case we are currently looking at is only looking at attributes like aria-label. Perhaps we can leave it as a TODO when this situation arrives?
The actual scenario we are trying to fix is basically manual dom manipulations not being represented in the cache.

Is the intent to have Incremental DOM re-set the attribute? That is, you call Incremental DOM initially with the value "one", some other DOM manipulation changes it to "two", and you call Incremental DOM with "one" again, which you now want to be set?
Or do you basically want to update the cached value Incremental DOM sees to be "two"?

Hmm so the case we are currently looking at is only looking at attributes like aria-label. Perhaps we can leave it as a TODO when this situation arrives?
The actual scenario we are trying to fix is basically manual dom manipulations not being represented in the cache.

Is the intent to have Incremental DOM re-set the attribute? That is, you call Incremental DOM initially with the value "one", some other DOM manipulation changes it to "two", and you call Incremental DOM with "one" again, which you now want to be set?
Or do you basically want to update the cached value Incremental DOM sees to be "two"?

I guess those actually achieve the same thing in the end...

Yeah what's happening is idom is initializing with "one", manual dom manipulations is setting it to "two" and we want to set it back to "one". However, that's not possible since the cached value is off. We've tried it both ways (updating the cache or pinning the cache). Not sure what's the better option.

Yeah what's happening is idom is initializing with "one", manual dom manipulations is setting it to "two" and we want to set it back to "one". However, that's not possible since the cached value is off. We've tried it both ways (updating the cache or pinning the cache). Not sure what's the better option.

If you don't have a lot of these nodes, it might be best to just re-do the diff rather than go through the effort of watching for the changes and updating the caches.  If there are only a few of these nodes, it probably doesn't make too much of a difference, so could go with this approach and loop back later.


Updating coords in image viewer

The coordinates of regions drawn on an image are only updated during click events, but some behaviors that modify the regions do not necessarily trigger the update (e.g. changing the definition of a region through option-click and dragging points). We should make sure any user-interaction that might manipulate regions also triggers an update.
Thanks to @d-v-b for reporting.

Fixed by https://github.com/lightning-viz/lightning-default-visualizations/commit/075bae9a89bd6cd4ed333226df95a3353aa0b8a2


Michael/app lookup proto

Supports getting the ID for a deployed app with a given name.

Oops accidentally branched from the tip of a larger feature branch


compile error with gcc 11.3

build with gcc 11.3.0 against tag v3.4.1 as well as master branch. got this error.
g++ (Gentoo Hardened 11.3.0 p4) 11.3.0
Copyright (C) 2021 Free Software Foundation, Inc.
This is free software; see the source for copying conditions.  There is NO
warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.

db/db_impl_compaction_flush.cc:696:21: error: loop variable ‘newf’ creates a copy from type ‘const std::pair<int, rocksdb::FileMetaData>’ [-Werror=range-loop-construct]
  696 |     for (const auto newf : c->edit()->GetNewFiles()) {
      |                     ^~~~

not very familiar with cpp though.

this is because the gcc version is too high so that rocksdb version doesn't compate. The simplest way to solve this is that using gcc at most 9.1。

good to know. I will try to build it with an older version of gcc then.
it will be nice if mention gcc version requirements in README (it only requires 4.8+ now).

update:
I see you are using -Wno-error=deprecated-copy -Wno-error=pessimizing-move in v3.4.1 and master. these options for gcc are added in gcc9. so better mention this in README (which says 4.8) i suppose.
after I setup another box with gcc 9.5, along with glog 0.5.0 and protobuf 3.19.3.
also, when I go for another build, I got
/tmp/pika/src/pika_repl_client_conn.cc: In member function ‘virtual int PikaReplClientConn::DealMessage()’:
/tmp/pika/src/pika_repl_client_conn.cc:50:98: error: no matching function for call to ‘google::protobuf::io::CodedInputStream::SetTotalBytesLimit(int, int)’
   50 |   decoder.SetTotalBytesLimit(g_pika_conf->max_conn_rbuf_size(), g_pika_conf->max_conn_rbuf_size());
      |                                                                                ^
In file included from /usr/include/google/protobuf/message_lite.h:47,
                 from /usr/include/google/protobuf/generated_enum_util.h:36,
                 from /usr/include/google/protobuf/generated_enum_reflection.h:44,
                 from /usr/include/google/protobuf/generated_message_reflection.h:46,
                 from /usr/include/google/protobuf/message.h:122,
                 from /tmp/pika/third/pink/pink/include/pb_conn.h:13,
                 from ./include/pika_repl_client_conn.h:9,
                 from /tmp/pika/src/pika_repl_client_conn.cc:6:

After some dig, it's caused by protobuf version. They added a single parameter version in 3.8 and removed deprecated version since 3.18.
It looks like the library pika master currently depends on are quite old, so maybe better to check those libraries version more strictly in Makefile or so.

update:
I see you are using -Wno-error=deprecated-copy -Wno-error=pessimizing-move in v3.4.1 and master. these options for gcc are added in gcc9. so better mention this in README (which says 4.8) i suppose.
after I setup another box with gcc 9.5, along with glog 0.5.0 and protobuf 3.19.3.
also, when I go for another build, I got
/tmp/pika/src/pika_repl_client_conn.cc: In member function ‘virtual int PikaReplClientConn::DealMessage()’:
/tmp/pika/src/pika_repl_client_conn.cc:50:98: error: no matching function for call to ‘google::protobuf::io::CodedInputStream::SetTotalBytesLimit(int, int)’
   50 |   decoder.SetTotalBytesLimit(g_pika_conf->max_conn_rbuf_size(), g_pika_conf->max_conn_rbuf_size());
      |                                                                                ^
In file included from /usr/include/google/protobuf/message_lite.h:47,
                 from /usr/include/google/protobuf/generated_enum_util.h:36,
                 from /usr/include/google/protobuf/generated_enum_reflection.h:44,
                 from /usr/include/google/protobuf/generated_message_reflection.h:46,
                 from /usr/include/google/protobuf/message.h:122,
                 from /tmp/pika/third/pink/pink/include/pb_conn.h:13,
                 from ./include/pika_repl_client_conn.h:9,
                 from /tmp/pika/src/pika_repl_client_conn.cc:6:

After some dig, it's caused by protobuf version. They added a single parameter version in 3.8 and removed deprecated version since 3.18.
after I downgrade protobuf to 3.17.3, it builds all well without problems.
It looks like the library pika master currently depends on are quite old, so maybe better to check those libraries version more strictly in Makefile or so.

we know that. rocksdb and pb are too old.
maybe fix in this year, current have no times to do that.

fixed now


MacOS app profile photo not updating

Updated profile photo on website, photo not updated in app, despite logging out + back in again.
App doesn't support updating the photo from the app.

Same here. Mac app does not pick up the profile pic from Github while the iPhone app does.


Refs #31615 -- Restored EXISTS clauses in extension operations.

Removed in original PR #12952 with comment from Mariusz: https://github.com/django/django/pull/12952#discussion_r430308740
I think leaving these clauses in here is good to prevent TOCTTOU bugs. Although not the recommended workflow, I've seen setups where migrations are run in parallel by all webservers during deployment, so if we can prevent errors there we should.

@adamchainz Thanks :+1: I added tests.

👍 Thanks, I missed them.


feat!(position): optimize max_len align & add fixed_col align

I've changed the max_len_align/padding to a new key position so that the user can choose one of 3 options instead of having 2 bools. It should also simplify adding more alignment modes in the future (such as "right" from rust-tools.nvim).
I've also changed the code to use the virt_text_win_col, so that we don't have to calculate the padding for each line.

@lvimuser Have you had a chance to look at this? I've been using this for over a month now. There is only minor 1 issue with this which is that if I enter text after my selected column, then the virtual text will hide the text that I've entered. Is there a setting I'm not seeing which will allow the virt_text to be hidden behind actual text instead? Fiddling with priority didn't help.

I haven't, no. You may look into the left_gravity option or perhaps avoid virt_text_win_col.

virt_text_pos might be a better alternative to virt_text_win_col.
Having said that, you can be a bit tricky and get the current line length and set the virt_text_win_col to
-- inside the loop
virt_text_win_col = math.max(opts.position.padding, line_len + 1)


MGDCTRS-257: Explicit set properties in KameletBinding

As today the KameletBinding that is being generated by the fleetshard-operator-camel does not include any connector property in the KameletBinding but instead, properties are added as a secret.
This PR aims at adding non sensitive properties to the KameletBinding which:

greatly simplify the reification process
makes it easy to troubleshoot issues as everything except sensitive properties are bound to the  KameletBinding
makes a clear distinction between sensitive an non sensitive data

apiVersion: camel.apache.org/v1alpha1
kind: KameletBinding
metadata:
  name: mctr-c8o92sl0qert6ga28tig
spec:
  sink:
    properties:
      bootstrapServers: kafka.acme.com:443
      id: c8o92sl0qert6ga28tig-sink
      password: '{{sa_client_secret}}'
      registryUrl: registry.acme.com/apis/registry/v2
      topic: lb-cos
      user: '{{sa_client_id}}'
      valueSerializer: org.bf2.cos.connector.camel.serdes.json.JsonSerializer
    ref:
      apiVersion: camel.apache.org/v1alpha1
      kind: Kamelet
      name: cos-kafka-sink
  source:
    properties:
      clientId: foo_bar
      clientSecret: '{{salesforce_client_secret}}'
      id: c8o92sl0qert6ga28tig-source
      objectName: Contact
      password: '{{salesforce_password}}'
      userName: bar
    ref:
      apiVersion: camel.apache.org/v1alpha1
      kind: Kamelet
      name: cos-salesforce-streaming-source

The counter argument to all of this is that if a kamelet property is missconfigured there might be sensitive data spilled around...


Content Suggestion: StoringBitcoin.info

Some weeks ago I started StoringBitcoin.info , an objective site without sponsoring, to help people determine how to store their bitcoin. Visit the site or look at https://github.com/storingbitcoin-info/website#overview to get an idea of "paths" one can take. It's Bitcoin only.

Link to Content: https://storingbitcoin.info/
Briefly Describe Content: StoringBitcoin helps you finding your strategy for storing Bitcoin
Suggested Section: I'm afraid it does not really fit an existing category, learning might be the best fit but not 100%

I like the concept, but the requirement for new users to enter sensitive information (how they are storing their bitcoin, if they deem this a large amount, if they are using a hardware wallet etc.) concerns me - these answers could be logged by a malicious server and tied to the users IP address.
For this reason I would feel uncomfortable merging this.

I like the concept, but the requirement for new users to enter sensitive information (how they are storing their bitcoin, if they deem this a large amount, if they are using a hardware wallet etc.) concerns me - these answers could be logged by a malicious server and tied to the users IP address.
For this reason I would feel uncomfortable merging this.

Thanks for the feedback and opening of issues on the github, will take a good look at them.
Just as with those online wallet generators, one could run the site locally. Everything someone does happens client side. There is no tracking, no statistics. The server can not know what someone clicked on.
Of course you will have to trust me to not put this in at a later phase, that's something I can't technically fix but it's something I can vouch to not do...
Does this background/info help?

Thanks for the feedback and opening of issues on the github, will take a good look at them.
Just as with those online wallet generators, one could run the site locally. Everything someone does happens client side. There is no tracking, no statistics. The server can not know what someone clicked on.
Of course you will have to trust me to not put this in at a later phase, that's something I can't technically fix but it's something I can vouch to not do...
Does this background/info help?

Thanks - I understand that. Would you consider adding instructions to run through it locally at the start? I suppose a wget would work

Thanks - I understand that. Would you consider adding instructions to run through it locally at the start? I suppose a wget would work

Will look at that and come back to you.

Will look at that and come back to you.

Have added instructions for running offline. It's a github zip-download of the MASTER of the repo. It works (English only though). Added instructions at the start of the site :).
Please reconsider adding the site :smile:

Seems like a good solution! ACK

Added
Will be visible in next site rebuild

Thanks man! Much appreciated.


Allow selecting version of fontawesome (4 vs 5 vs 6)

Icons have changed between versions. Our project is using version 4 of fontawesome, and we'd like to create a favicon that matches.

I will not be supporting multiple fontawesome versions. If you would like to use older versions feel free to use import_icon_data.py with an older version and run the site locally,


Backend

Enable common utilities among AutoML frameworks.

I have enabled black, please let me know what is your perspective on this.

Ok, I had to make some more changes, sorry for the trouble.
Things will look like: https://github.com/franchuterivera/Auto-PyTorch/tree/refacto_development_automl_common_backend/autoPyTorch


[Hacktoberfest]: Edit the FreeCodeCamp Podcast

🆕🐥☝ Beginners Only:
This issue is reserved for people who are new to Open Source. We know that the process of creating a pull request is the biggest barrier for new contributors. This issue is for you 💝
To do:

Edit the FreeCodeCamp Podcast Comments section to say The official podcast of the freeCodeCamp open source community

See below for link to step-by-step tutorial

Add your details in the contributors section at the bottom of the page using the same formatting as the last one

📋 Step by Step
Step-By-Step Tutorial
Remember to observe the Code of Conduct - by which we mean, in short - Be Kind, Be Positive, Be Helpful

I'll take this


refactor: extract state notification logic

Closes: #829
Extracts the notification state logic into it's own file.  Also refactored the test suite to be clearer on the scenarios being tested

I'm assuming this is pretty much just moved code, not any functional changes. Approving on that basis -- I haven't reviewed each individual line. If there are functionality changes too, please point them out so I can take a closer look.

that's correct, pretty much just moved across.


Notebooks

In prep of the working group training looking for the following examples notebooks. We have some of these already but capturing all of them for completeness. (You can edit this post to mark the box if complete)
[ ] @sssangha: Notebook to download GNSS stations over CA for a given period and time
[ ] @jlmaurer: Notebook that opens a small e.g. (10-15 row) dummy csv file and shows how to perform basic operations using pandas
- slice and dice through rows and columns
- mathematical operation between columns
- write multiple columns back out to a csv file
- Append multiple columns to an existing file
- perhaps how to make a basic plot
[ ] @jlmaurer: Break out the raiderDelay notebook and model set up into two separate notebooks.
- How to set up you model notebook
- How to run raider notebook and use its options.
[ ] @sssangha: Notebooks for stats class examples for its options. Would expand the examples to also include the residuals analysis in addition to the absolute delays.
- Capture the line for downloading the GNSS data
- Capture the line for running the raider code
- Capture the line how to make the difference between raider
- Capture the options in stats class
[ ] @dbekaert update landing page for RAIDER-docs

This is an example command to run raider:
raiderDelay.py --time 06:00 --model HRES --date 20090101  --station_file ../GNSS/gnssStationList_overbbox.csv --zref 30000 -v


[Feature] Export Table to CSV

It would be super nice if we could Export/Download the Table as csv (just like we can export the graphs).
My use case is I'm logging table-data to wandb across runs, and I would like to download the full concatanated table as a csv.

When a table is viewed within a run workspace there should be a "download" button in the upper right.  Are you able to see this?

No, all I see are these controls:

It would be really nice to have this feature, because it's not possible to replace it with anything else.
I can't log an artifact from multiple runs, nor can I join data from multiple runs from the GUI.

You probably missunderstood me, what I meant was the table from the "Tables" pane, where we log data manually:

Ahh I see.  Adding a CSV export to tables is definitely on the road map.  Until then you can export the data programmatically:
import wandb
import csv

api = wandb.Api()
art = api.artifact("USERNAME/PROJECT/run-RUN_ID-rewards:v0")
table = art.get("rewards")

with open("rewards.csv", "w", newline="") as f:
  writer = csv.writer(f)
  writer.writerow(table.columns)
  for row in table.data:
    writer.writerow(row)

What I meant is I want the data for all the runs (the resulting concatenated table).
The reason this is useful is because during a sweep I can have multiple workers log to the same table, at the same time, and after my sweep is done I can get a concatenated table.

@drozzy we have created an internal ticket for this feature. I'll notify you once this is possible.


updated email command

Coverage decreased (-0.09%) to 82.759% when pulling 3ea34ae8d878abf30907c467b3622e82705d18b3 on sidhmads:email_fix into 03f2ee12d7fb5e05c643c2ec7850103926451321 on CS2103AUG2017-T11-B1:master.


Folium and Google Earth Engine tiled images

@mccarthyryanc figured out how to ingest Google Earth Engine tiled images into a Folium map. See his example notebook and supporting code at https://github.com/mccarthyryanc/folium_gee
Just beware that in the notebook the GEE image will no longer show up, because GEE caches these images for only a couple of days or so. Plus, you need your own GEE credentials to run the notebook. Still, it'd be nice to add his notebook here.

That is pretty cool! @mccarthyryanc do you want to send that as an example here?
Or can you license (like BSD 3-Clause) that example so others can do that for you?

I submitted a pull request, #4, with an example notebook.

Thanks! That was an easy merge :smile:

Cool. Thanks to both.


Configure PR checks to run on Windows and MacOS

Description
To avoid issues in the future where code changes prevent the repo from compiling on Windows, I've added it as part of the configuration for the PR-check Github action; this action will be run for each OS specified under strategy. Since the majority of the developers use MacOSX, I've added that as well. These jobs run in parallel, so the time taken to run PR checks should not increase by any significant amount.
I've removed the chromium --version check as the command is not cross-platform compatible, the version of chrome on the path may not be the chrome the karma-runner is using, and trying to run the right command would add unneeded complexity to the file. Chrome is installed on all OS images by default, and the karma-runner prints out what version it's running with.
You can see an example of the PR action being run on all platforms here: https://github.com/JuliaABurch/amazon-sumerian-hosts/pull/3 (Note that the windows run will fail until  #109 is merged, but I have it passing locally as I pulled in that change to my repo )
Reviewer Testing Instructions
Github actions are a little tricky to test - I did it on my forked repo after manually enabling actions.
Submission Checklist
I confirm that I have...

[x] removed hard-coded Cognito IDs
[x] manually smoke-tested the BabylonJS integration tests
[x] manually smoke-tested the BabylonJS demos
[x] manually smoke-tested the Three.js integration tests
[x] manually smoke-tested the Three.js demo

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

How do we test the GitHub Action?

You can actually see it running in this PR - if you scroll down, you'll see the tests being run on the three platforms specified.
I can update the PR description if it's not very clear as to how I did it, but I used my forked repository to test after enabling github actions there.

How do we test the GitHub Action?

You can actually see it running in this PR - if you scroll down, you'll see the tests being run on the three platforms specified.
I can update the PR description if it's not very clear as to how I did it, but I used my forked repository to test after enabling github actions there.

I see. That looks pretty cool! Thank you!


OSSM-569 Use default resources for prometheus sidecar

There's currently no way for customers to set resources for the istio-proxy that comes with prometheus. This can cause scheduling problems in kubernetes.
Adding a /hold as we'll need to decide whether we want to ship this in 2.0.8. I verified that it works locally.
PS: This needs a manual cherry-pick for 2.1 as the template for prometheus has become an overlay

/test istio-operator_unittests

/test istio-operator_unittests

Removing hold as discussed on our call


NPE exception: Create a child actor when the current actor is under construction

The code will throw the NPE
class B : Actor {...}
class A : Actor {...}
  init() { actorOf(B.init, "B") }
}
system.actorOf(A.init, "A")

The reason is at the current actor A init phase, the actor's ref is not assigned. So it cannot call actorOf() which will use the ref field, to create actor B.

The current fix #d931cfb partially solve the problem. Now user can create child actor in preStart()
But the child actors fields can only be optional var, like
class A : Actor {
  var b:B!
  init() {...}
  preStart() { b = actorOf(B.init, "B") }
}

If we want to support let b:B!, the field assign must still be inside init(), then we still have to pass actorSystem/ref into init(), which causes we can not write arbitrary constructors of an actor.


[Tales of Xadia] Fix

Fixes to a few errors
Portraits/distinctions/assets associating with wrong character name
Missing specialty Cooking
Changes / Comments
Roll20 Requests
Comments are very helpful for reviewing the code changes. Please answer the relevant questions below in your comment.

[ ] Does the pull request title have the sheet name(s)? Include each sheet name.
[ ] Is this a bug fix?
[ ] Does this add functional enhancements (new features or extending existing features) ?
[ ] Does this add or change functional aesthetics (such as layout or color scheme) ?
[ ] If changing or removing attributes, what steps have you taken, if any, to preserve player data ?
[ ] If this is a new sheet, did you follow Building Character Sheets standards ?

If you do not know English. Please leave a comment in your native language.

Character Sheet Info Roll20 Internal Use only.


get call stack limit

https://stackoverflow.com/questions/7826992/browser-javascript-stack-size-limit
getMaxCallStackSize = () => {
    try {
        return 1+getMaxCallStackSize()
    } catch (e) { return 1 }
}
;[...Array(10)].forEach(() => getMaxCallStackSize ())
console.log(`${(getMaxCallStackSize ()/1000).toFixed(0)}K`)

hmmmm

That JIT comment is very interseting. I was not aware this.
10 is just an arbitrary number to catch the max, but I could probably get by with just 4 calls for the function I'm using. Some builds yield higher calls (50-400% more) by the 4th call. The absence of variables also increases the size (in V8 atleast)
On Android, Chrome and FF are occasionally unstable even at 10, so I've added rounding: (getMaxCallStackSize()/100).toFixed(0)*100

Found some interesting code about that on nowsecure.nl (protected by cloudfare) inside a service worker:
onmessage = function(e) {var gsb = function(){
    var sizeA = 0;
    var sizeB = 0;
    var counter = 0;
    try {
        var fn_1 = function () {
            counter += 1;
            fn_1();
        };
        fn_1();
    }
    catch (_a) {
        sizeA = counter;
        try {
            counter = 0;
            var fn_2 = function () {
                var local = 1;
                counter += local;
                fn_2();
            };
            fn_2();
        }
        catch (_b) {
            sizeB = counter;
        }
    }
    var bytes = (sizeB * 8) / (sizeA - sizeB);
    return [sizeA, sizeB, bytes];
}; postMessage({a3: gsb()})};

conclusions:

max stack size seems to be different with and without local variable
max stack size seems to
@abrahamjuliot Any ideas what bytes = (sizeB * 8) / (sizeA - sizeB); represents?

Yeah, bytes just represent the difference.


no-tabbar does not work on Safari for the messages page

This is a (multiple allowed):

[x] bug
[ ] enhancement
[ ] feature-discussion (RFC)
Framework7 Version: 1.4.2
Platform and Target: IOS , Safari

What you did
Here is my setup:
dynamic navbar
auto hide navbar/tabbar on scroll for all pages except one with messages.
in the messages page, I have the following:

Messaging page

    <form id='msgsFormId' class="toolbar-inner ">
      <input type="text" placeholder="write your msg.." class="ks-messages-input">
      send
    </form>

The above page works fine on Google Chrome. However, on safari the tabbar is hidden upon viewing the page for the first time. Once the page is scrolled down then up, the tabbar appears. I tried removing and adding keep-tabbar-on-scroll without any luck.
Any idea how to make the tabbar invincible on Safari ?
Expected Behavior
tabbar should be hidden on the messages page on both Chrome and Safari.
Actual Behavior
tabbar is hidden only on Google Chrome.

+1

I have a similar problem.  I have a tabbar that has a messages tab.  When you click on the tab, the message bar appears at the bottom.  This works fine on MD (tabbar is at top) but on IOS, the tabbar is on the bottom and overlays the tabbar so you can no longer move to another tab.  I thought it was a quirk with the messagebar replacing the tabbar on that tab but that isn't the case with MD.  I don't know what the solution is other than to move the messagebar up on top of the tabbar for IOS.

Issue is closed because of outdated/irrelevant/not actual/needed more information/inactivity.
If this issue is still actual and reproducible for latest version of Framework7, please create new issue and fill the issue template correctly:

Clearly describe the issue including steps to reproduce when it is a bug.
Make sure you fill in the earliest version that you know has the issue.
Provide live link or JSFiddle/Codepen or website with issue


window.onresize is not intercepted by zones

Repro snippet:
window.onresize = () => console.trace('stack from within onresize handler');

Zone check from within the onresize handler:
> window.Zone.current
Zone {_properties: Object, _parent: null, _name: "<root>", _zoneDelegate: ZoneDelegate}

We came across this via angular/angular#16143

I will check it.


Created a new Migration for Rating on Tickets

Fixes #198

same thing for user settings.

Thank you!
I'm really used to worn on more "controlled" environment and do that little migrations by hand, but you are right that they can be a big headache in a project like this
thanks!


Set max_paralel_dowloads higher by default?

Is there a reason that is 1 by default? It's much faster to use e.g. 10.

Higher quality on first show? But, overall Tyler hasn't been maintained that much or at all  in the past year. So, whatever feedback and improvements as PRs might come, we should merge 🙂

I have some free time now! :)

Go for it 😁


[Bug] In the example project, LoginXNFT from ModelH class don't work.

Describe the bug
The example project always fails to login when the user created/loaded a wallet already .
To Reproduce
1 - Run the example project and click in the wallet icon in the center of the screen;
2 - Load/Create a Wallet;
3 - Leave the wallet screen and go to the main screen where the wallet button is.
4 - Hit the wallet button again and expect to go direct to the wallet screen;
Expected behavior
A internal logind should be done and the user should be led to the WEallet screen.
But it always fails, the example project should be able to do login but it fails and always shows the login screen;
Screenshots
The screenshot below shows the part of the code inside LoginScreen.cs. This part of the code is responsible for testing the login (line 92), and if success, lead the user to the Wallet screen(line 103).

Unity Editor :

Version [2021.3.18f1]
OS: Windows 10

Additional context
The problem happens in the line 92 (see image above). The LoginXNFT from WalletH is not working and the entire flow is interrupted.
The LoginXNFT method from WalletH creates a XNFTWallet object and the Login method from this class is probably the source of the problem.

Hi @mixmob-julio , are you trying to create the wallet inside an XNFT (running in backpack)?
With a normal flow, the creare/import wallet works and show the wallet screen afterward, you can check in the live demo here: https://garbles-labs.github.io/Solana.Unity-SDK/
For an XNFT, running inside backpack, unfortunately AFAIK there is no way to create a wallet. The LoginXNFT will be called automatically and you will be logged with backpack account. Hope this helps.

No, I'm just playing with your example project.
Yes, after you load/create a wallet the app goes to the wallet screen, but this is not the problem.
The real problem is in the example project in the script LoginScreen.cs. This part of the code is responsible for testing the login (line 92), if success should lead the user to the Wallet screen(line 103) directly, without a manual login or the need to load/create a new wallet. This part is not working.
I think the problem is in the ModelH.instance.LoginXNFT call. The method LoginXNFT calls the login method from a fresh new XNFTWallet objected just created. This method returns nothing, triggers no error, and do nothinfg else at this time forcing the user to go to the LoginScreen everytime, and not leading the user to the WalletScreen in case of a successful login.

Hi @mixmob-julio ,
The user always perform a manual login when using an in-game wallet in the demo scene, the private key is stored encrypted, therefore the game need to ask the password at login time in order to decrypt and restore the public/private key.
It's possible to delay the login until the first transaction approval is required tough, and show the balance/wallet screen just using the public key. Is this what you are trying to achieve?
The LoginXNFT is only used for XNFT inside backpack, in which case the login is automatic and all the signing is delegated to backpack.

@GabrielePicco thansk for the clarification.
I could not find a doc explaning that, nor the other kinds of logins. Are you planing to extend the current documentation?
Answering your question, for now I'm just trying to learn how the example works.
About LoginXNFT , I'm still confused. Is it a requirement? I mean, do I need to LoginXNFT once at every execution? Have you a document for that?

@mixmob-julio yes the documentation definitely needs to be expanded, there is a short description here: https://solana.unity-sdk.gg/docs/configuration . We will expand it, also fell free to contribute, this is a community driven SDK.
Docs is here: https://github.com/garbles-labs/Solana.Unity-SDK/tree/docs/src/pages/docs , the website is automatically rendered form the markdown files there.
LoginXNFT is only used when used/working when inside an XNFT (in Backpack), you can read more here: https://solana.unity-sdk.gg/docs/xnft
In all the other cases LoginXNFT will just fail and users can login with standard login methods.


Using 'http' in develop and 'https' in production

Hey Guys
I am using ASP.Net Core to build a REST Api. We are enforcing https on our production server but not on develop. Is it possible to change the schema so that if env.IsProduction it uses https calls and if env.IsDevelopment it will use http calls?
Thanks for answering my question

If you are looking for a quick solution to this issue you can leverage our Swashbuckle extension package.
if (env.IsDevelopment()){
     options.UseSchemes(schemes => schemes.Add("http"));
} else {
     options.UseSchemes(schemes => schemes.Add("https"));
}

This is open source so feel free to copy past library code if you don't want to reference the library ^^.

@Franklin89 - by default Swashbuckle omits the "schemes" field in the Swagger output. This means that the swagger-ui tool will construct requests based on the scheme that was used to access the Swagger JSON. So, if you access the swagger-ui over HTTP in development, then that's what will be used to construct requests. And, if you access the swagger-ui over HTTPS in prod, then that will be used to construct requests. So, if I'm understanding your requirements correctly, this should just work correctly in both cases.
However, if you do want to specify an explicit scheme for each environment, you can use @sandorfr's extension OR you can leverage a simple hook to add the environment specific schemes prior to serializing the Swagger document into JSON:
app.UseSwagger(c =>
{
    var schemes = _hostingEnv.IsDevelopment() ? new[] { "http" } : new[] { "https" };
    c.PreSerializeFilters.Add((swagger, httpReq) => swagger.Schemes = schemes);
});


Master -> develop after 2.5.1 release

Pull Request Checklist:

[ ] Positive Test Case Written by Dev

[ ] Automated Testing

[ ] Sufficient User and Developer Documentation

[ ] Passing Jenkins Build

[ ] Peer Code review and approval

Positive Test Case

[Enter positive test case here]

METRIC
VALUE

https://sonarqube.cuahsi-workstation.com:9000/dashboard?id=hydroshare-5053


URL ParseError { invalid port number } if password contains "?"

let pool = mysql::Pool::new("mysql://user:password?@localhost:3306/test").unwrap();
throws the following error:
thread 'main' panicked at 'URL ParseError { invalid port number }',
...\mysql-7.1.2\src\conn\opts.rs:580

It doesn't matter at which position the "?" is, it will always throw the error.
My specs:
D:\Rust_Training>rustc --version
rustc 1.12.1 (d4f39402a 2016-10-19)

D:\Rust_Training>cargo --version
cargo 0.13.0-nightly (109cb7c 2016-08-19)

 Fresh mysql v7.1.2

I believe there was related issue already, but i couldn't find it..
Anyway. You should urlencode some characters according to RFC 3986.
I.e. something like this should work:
let pool = mysql::Pool::new("mysql://user:password%3F@localhost:3306/test").unwrap();

Closing it.
Feel free to reopen if needed.

Hm I got a # in my password and urlencoding doesnt help either. Other Ideas?

Issue may be related to the url crate which is used for URL parsing. I'll look into it tomorrow.

Hm I got a # in my password and urlencoding doesnt help either.

Looks like there is no issue, because this program:
extern crate mysql as my;

fn main() {
    let opts = my::Opts::from("mysql://root:pass%23word@localhost/");
    println!("password: {:?}", opts.get_pass());
    let pool = my::Pool::new(opts).unwrap();
    let row = pool.first_exec("SELECT 'foo'", ()).unwrap();
    println!("{:?}", row);
}

gives this output:
password: Some("pass#word")
Some(Row(Bytes("foo")))

So urlencoded # was parsed correctly and connection to the server was successful.

Other Ideas?

Note that your shell may interpret # symbol as a start of a comment. I.e.:
mysqladmin -uroot -p password pass#word

The password of the root user will be pass instead of pass#word.
To fix this you'll need to quote password:
mysqladmin -uroot -p password 'pass#word'

Please make sure that it's not you case and if it isn't please send me an output of the code above but with your connection string.

I think the reason is that I didn't know of the
let opts = my::Opts::from("mysql://root:pass%23word@localhost/");
that line isn't in the example I will try that.

my bash doesn't make the # to a comment but my # is at the end of the password so my string looks like mysql://root:password#@localhost:3307

Oh I think the usage of format!() is my problem it creates<EMAIL_ADDRESS>

Anyway please show me the output of this code:
extern crate mysql as my;

fn main() {
    println!("url: {}", <your exact connection string here>);
    let opts = my::Opts::from(<your exact connection string here>);
    println!("password: {:?}", opts.get_pass());
    let pool = my::Pool::new(opts).unwrap();
    let row = pool.first_exec("SELECT 'foo'", ()).unwrap();
    println!("{:?}", row);
}

mtrnord@riot:~/rust/IMAPServer-rs$ RUST_BACKTRACE=1 cargo run
    Finished dev [unoptimized + debuginfo] target(s) in 0.1 secs
     Running `target/debug/IMAPServer`

url<EMAIL_ADDRESS>thread 'main' panicked at 'URL ParseError { invalid port number }', /home/mtrnord/.cargo/registry/src/github.com-1ecc6299db9ec823/mysql-11.3.0/src/conn/opts.rs:582
note: Run with `RUST_BACKTRACE=1` for a backtrace

Looks like # isn't url-encoded. First line of output should be:
url<EMAIL_ADDRESS>

I will try to urlencode it first. one sec.

Finally got it my url encoding wasn't doing the expected. switching to the urlencoding crate did the job :) issue can be closed. It was broken urlencoding on my side. Still thanks for the help

No problem.

...Still waiting for the day when type safe languages can save us from poorly formatted strings.
Until then, here's a good urlencoding crate for this:
https://docs.rs/urlencoding/latest/urlencoding/


add license / feature exception handling to zpa isolation

Current zscaler-terraformer version
zscaler-terraformer v1.3.5 Terraform version: Terraform v1.10.3 on (windows_amd64)
Description
the ZPA import fails without excluding:

zpa_cloud_browser_isolation_banner
zpa_cloud_browser_isolation_certificate
zpa_cloud_browser_isolation_external_profile

with error:
time="2024-12-26T13:07:10-05:00" level=fatal msg="FAILED: GET, https://config.zpagov.net/cbiconfig/cbi/api/customers/XXXXXXXXXX/certificates, 404, 404 Not Found, {\"reason\":\"No handler found\",\"id\":\"resource.not.found\"}"
Use cases
It would be ideal if the ZPA import imported everything that was available in a tenant without having to exclude everything that is not licensed. This would greatly improve the import time if this iterative exclusion process is eliminated.
Potential zscaler-terraformer usage
run zscaler-terraformer import --resources="zpa" and have all supported features imported without error.
References
No response

We are not implementing this.
This is a Cloud browser isolation api limitation which does not provide us information on license status.
Please use the --exclude flag as per the readme.
Zscaler DevRel.


Implement dark mode

Its pretty:

~Streams in Lightmode are not as pretty atm:~ (Fixed!)

nice!


RC test Forces and Motion: Basics 2.3.0-phetiorc.3

@samreid @zepumph @kathy-phet @ariel-phet Forces and Motion: Basics 2.3.0-phetiorc.3 is ready for RC testing.
This RC has master SHA's so a full RC test is required (even though we are starting on rc.3).  This RC is to support adding Legends of Learning features to Forces and Motion: Basics. Since FAMB is a phetio sim, we need to deploy a phet-io version at the same time. Rather than deploy both PhET and PhET-iO RCs now, we are starting with the phetio version so that we won't have to create two new RC's if issues are found.  Here is the link to index.
Root Directory
Testing Matrix
If any new issues are found, please note them in phetsims/forces-and-motion-basics/issues and reference this issue.
Assigning @phet-steele and @ariel-phet to assign priority for testing.

Forgot to link in the issue, these issues should have been fixed from the previous RC, @phet-steele can you please verify?

[ ] https://github.com/phetsims/forces-and-motion-basics/issues/231
[ ] https://github.com/phetsims/forces-and-motion-basics/issues/230

@ariel-phet will you please iterate through the issues linked above and help identify which are essential to fix before publication, and which should be classified as "aesthetic/non-essential"?

@samreid I made comments in issues I thought were non-essential currently

Excellent, thanks!

@phet-steele just an update on this issue, changes for the above issues were small or aesthetic so we are going to try to patch fixes and do a spot check RC next (unless big things are found in the remainder of this test).

@jessegreenberg let's go ahead and get that spot check going.

Sounds good!

rc.4 will need a cherry-pick for https://github.com/phetsims/special-ops/issues/82.

A new RC was created in https://github.com/phetsims/QA/issues/54 this can be closed.


Add possibility to filter background noise from input audio

Any tests of vocal mics would be welcome.. a dynamic mic is a natural voice band pass, big diaphragm ,and singing mics it's heavy.   So background on non vocal don't seem to go to the offline voice  access that msft uses in win 11 previews, I'm testing with that but it's not open source as far as I know .   But it might use open sources.
I looked at and evaluated
Pico hotwords.  To me hotwords are key.    But they don't have a quick hot word to , "text to intent " Rhino transition , a delay was reported and they didn't want to fix it.  And it's a paid service.  I find the faster you talk the less data they have to process and or send, especially with Google assistant online.  And I hope to save time when this advanced stuff is fully integrated.
The clip on ,walk around type for speech could work.  Condenser mic with a simple low pass might be ok but will add a phase shift .  they tend to pic up high frequency noise and that's hard for the voice to text filters.   Using digital convolution filters in addition, seems asking for trouble.
But Using hotword low latency DSP on device is key or adding devices  even buttons ,taping or switching mic, so that so one can interrupt process or completion or offline work. For , No cancel, Don't sent it , etc.
But any have decent vocal mic should be ok.  Or a list of recommended ones.  It makes a night day difference vs my laptops mic array.. on mst voice access anyways.   Just starting to believe this can be generalized and replace mouse keyboard , hopefully by LLM , preprocess ,OCR, scrape, index , synonym, completion choice hotword using international NATO labels and hotword,as  in , alpha bravo, charlie, so  preprocess then offline ..
I hope to see completion in context , via synonym that are auto generated by llms, but when applied  can be a dimensional raising convexifixation  issue, or treated  as a dirichlet constraint .   It seems simple but with llms it's not.  I'll move this bit to another issue, I'm overextended.
when I learn more about what's projects are going or if they can be generalized because they should be IMO. Msft  Semantic kernel  is being steered many ways..now I'm steering it even another way but everyone is being heard ,concensus is way more likely than with visual  UI.   You discover features, see it,  say it.
Thx I'll try to find time to eval more .btw    I think I got them to agree do the NATo coms, but I can't bet it will be fully implemented  as I imagine .  Right now always  I rage quit the one they have and it's pretty amazing, billions invested, ,just not usable , but could be .
I'll compare w this when I get Julia working I can't build anything..


tengine: init init at v2.2.2

Motivation for this change
Add web server tengine.
Things done

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

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

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

@Izorkin i added a commit to fix the build with gcc7 so we do not need to use gcc6. Please test.

Thanks. Works.

@GrahamcOfBorg build tengine

Success on x86_64-linux (full log)
Attempted: tengine
Partial log (click to expand)
test -f 'objs/ngx_auto_config.h' && cp 'objs/ngx_auto_config.h' '/nix/store/5chz7isvbhb5jd2nl9ilj31b80m0vh9d-tengine-2.2.2/include'
make[1]: Leaving directory '/build/tengine-tengine-2.2.2'
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/5chz7isvbhb5jd2nl9ilj31b80m0vh9d-tengine-2.2.2
shrinking /nix/store/5chz7isvbhb5jd2nl9ilj31b80m0vh9d-tengine-2.2.2/bin/nginx
strip is /nix/store/1hi76hr87bd1y1q1qjk0lv8nmcjip1c8-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/5chz7isvbhb5jd2nl9ilj31b80m0vh9d-tengine-2.2.2/bin
patching script interpreter paths in /nix/store/5chz7isvbhb5jd2nl9ilj31b80m0vh9d-tengine-2.2.2
/nix/store/5chz7isvbhb5jd2nl9ilj31b80m0vh9d-tengine-2.2.2/bin/dso_tool: interpreter directive changed from "/bin/sh" to "/nix/store/dkh7l9a4sx7zqh8riqbj3z21sz25p8xy-bash-4.4-p23/bin/sh"
checking for references to /build in /nix/store/5chz7isvbhb5jd2nl9ilj31b80m0vh9d-tengine-2.2.2...

Failure on aarch64-linux (full log)
Attempted: tengine
Partial log (click to expand)
checking for TCP_INFO ... found
checking for accept4() ... found
checking for kqueue AIO support ... not found
checking for Linux AIO support ... not found

./configure: no supported file AIO was found
Currently file AIO is supported on FreeBSD 4.3+ and Linux 2.6.22+ only

builder for '/nix/store/5mv5k0hk08ppiz2izhci05yz8iz42f41-tengine-2.2.2.drv' failed with exit code 1
error: build of '/nix/store/5mv5k0hk08ppiz2izhci05yz8iz42f41-tengine-2.2.2.drv' failed

Success on x86_64-darwin (full log)
Attempted: tengine
Partial log (click to expand)
test -f 'src/http/modules/ngx_http_ssl_module.h' && cp 'src/http/modules/ngx_http_ssl_module.h' '/nix/store/gpz1sh6c7b7ahqc2c2ah56knbyj1d9r3-tengine-2.2.2/include'
test -f 'src/http/modules/ngx_http_reqstat.h' && cp 'src/http/modules/ngx_http_reqstat.h' '/nix/store/gpz1sh6c7b7ahqc2c2ah56knbyj1d9r3-tengine-2.2.2/include'
test -f 'objs/ngx_auto_headers.h'  && cp 'objs/ngx_auto_headers.h' '/nix/store/gpz1sh6c7b7ahqc2c2ah56knbyj1d9r3-tengine-2.2.2/include'
test -f 'objs/ngx_auto_config.h' && cp 'objs/ngx_auto_config.h' '/nix/store/gpz1sh6c7b7ahqc2c2ah56knbyj1d9r3-tengine-2.2.2/include'
make[1]: Leaving directory '/private/tmp/nix-build-tengine-2.2.2.drv-0/tengine-tengine-2.2.2'
post-installation fixup
strip is /nix/store/qfxcr8c4fg7lkybrny9n2sb77bmippdx-cctools-binutils-darwin/bin/strip
stripping (with command strip and flags -S) in /nix/store/gpz1sh6c7b7ahqc2c2ah56knbyj1d9r3-tengine-2.2.2/bin
patching script interpreter paths in /nix/store/gpz1sh6c7b7ahqc2c2ah56knbyj1d9r3-tengine-2.2.2
/nix/store/gpz1sh6c7b7ahqc2c2ah56knbyj1d9r3-tengine-2.2.2/bin/dso_tool: interpreter directive changed from "/bin/sh" to "/nix/store/qb76s7s3p6pnsi7082sjfc7rb64f3s6a-bash-4.4-p23/bin/sh"

aarch64 failure can be fixed later by someone familiar with the platform.


[5.5.x] do not treat context expiration as terminal when validating agent connection

Make agent connect timeout configurable via environment (GRAVITY_AGENT_CONNECT_TIMEOUT).
Ignore deadline exceeded errors when validating connection. As the original motivation for validating the connection to the peer (server) has been early detecting of terminal conditions, connection timeout should not be considered a terminal condition and the agent should be allowed to continue even if connection takes longer than the originally allotted time.

Updates https://github.com/gravitational/gravity.e/issues/4229.

I don't know, it seems odd to me that that this is a process for ding some early validation, and if it fails to do so that we should just ignore it and continue. I'm also really curious why it's timing out at all, shouldn't the other side be answering within the timeout?

As I wrote in the issue - I've been able to reproduce this with an increased network latency so it is quite a possible scenario. Although the validation connect times out (in my test it did so right after successfully creating a client but before it was able to send PeerJoin), the operation then succeeds with the change (albeit slower than usual).


Adding swoole for async event handling

Refactor connect method

Current error:
PHP Warning:  Swoole\Client::recv(): recv() failed, Error: Resource temporarily unavailable[11] in /home/ghostzero/tmi/src/Client.php on line 81


Fix table formatting in obsolete members article

Fixes #12520
Promotes assembly headings to H2 for automatic in-article TOC

Preview link

Thanks for the review Maira. Agreed the word breaks aren't the best, but it still looks better than before.


Appstudio update test-custom-default-branch-egai

Pipelines as Code configuration proposal
To start the PipelineRun, add a new comment with content /ok-to-test
For more detailed information about running a PipelineRun, please refer to Pipelines as Code documentation Running the PipelineRun
To customize the proposed PipelineRuns after merge, please refer to Build Pipeline customization

Pipelines as Code CI has failed.There was an error creating the PipelineRun: test-simple-build-mcmm-on-pull-request-
creating pipelinerun test-simple-build-mcmm-on-pull-request- in namespace build-e2e-swqr-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request

Pipelines as Code CI has failed.There was an error creating the PipelineRun: test-annotations-mijr-on-pull-request-
creating pipelinerun test-annotations-mijr-on-pull-request- in namespace build-e2e-swqr-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request

Pipelines as Code CI has failed.There was an error creating the PipelineRun: test-simple-build-ufgi-on-pull-request-
creating pipelinerun test-simple-build-ufgi-on-pull-request- in namespace build-e2e-swqr-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request

Pipelines as Code CI has failed.There was an error creating the PipelineRun: test-simple-build-uqzm-on-pull-request-
creating pipelinerun test-simple-build-uqzm-on-pull-request- in namespace build-e2e-swqr-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request

Pipelines as Code CI has failed.There was an error creating the PipelineRun: test-simple-build-nyrh-on-pull-request-
creating pipelinerun test-simple-build-nyrh-on-pull-request- in namespace build-e2e-swqr-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request


Port object should also include the type attribute form Topology Spec 2.0

Hi,
According to the Topology Data model spec 2.0.0, the Port object has the type attribute as described below:

The type attribute represents the technology and bandwidth of the physical port (or interface). type is an enum with only one value acceptable. The Topology data model specification supports only Ethernet. The type enum is 100FE, 1GE, 10GE, 25GE, 40GE, 50GE, 100GE, 400GE, and Other. When the value Other is chosen, no bandwidth guaranteed services will be supported in this port. The value Other was created to enable flexibility when the port is not on the enum. In case Other becomes recurrent, the SDX team must increase the specification subversion and add the correct bandwidth to the type enum. The specification version table must be updated with such info.

We do have some references to the port_v2.json testing data on PR #121 but I believe that it ended up being left behind:
https://github.com/atlanticwave-sdx/datamodel/blame/312996abf3ffec61f736fb8ac171e9eaebb764a3/tests/__init__.py#L37
That file does not exists on https://github.com/atlanticwave-sdx/datamodel/tree/312996abf3ffec61f736fb8ac171e9eaebb764a3/tests/data

Yes. And, in operation, matching port types, esp. on the NNI link is important. It's a known underimplemented issue.  Good timing to address it.


String handling (indentation)

aims to fix https://github.com/ufirstgroup/ymlr/issues/160

Requirements

[ ] Entry in CHANGELOG.md was created
[ ] Link to documentation on https://yaml.org/ is provided in the PR description
[ ] Functionality is covered by newly created tests

Closing this as it was done in #165

Pull Request Test Coverage Report for Build 2c5c2e203f9df4024db65da2be829a52d2238679-PR-166
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

8 of 8 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build<PHONE_NUMBER>7e45c25cd4ac0ec11bea0737282c69:
0.0%

Covered Lines:
115

Relevant Lines:
115

💛  - Coveralls

Pull Request Test Coverage Report for Build 2c5c2e203f9df4024db65da2be829a52d2238679-PR-166
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

8 of 8 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build<PHONE_NUMBER>7e45c25cd4ac0ec11bea0737282c69:
0.0%

Covered Lines:
115

Relevant Lines:
115

💛  - Coveralls


How to simply redirect no-www to www

Hi
I just want the domain to be redirected to a www if it is no-www  (https)
I tried this :

And

But it doesn't work.
I wanted to avoid going through Redirection Hosts

do you have a simpler solution?
Thank you

I'm pretty new to NPM, but if I remember correctly the upstream nginx-proxy script would add the custom nginx configuration inside the server { server_name; } block, so you wouldn't be able to do it the way you're trying to.

Yes, the advanced config does go inside the server block. So no, this would need code outside of the server block, which is unfortunately not possible with NPM. So the redirect host would be your best bet.