High deviation for a certain password

Regarding this password: Submit new password

on the web demo I get a score of 4 and a guesses count of<PHONE_NUMBER>00 (guesses_log10: 11.32675)
with this library I get a score of 2 and a guesses count of 48909168

Setup:

Same small additional user dictionary
PasswordMatcher removed from the matchers list
Version: 1.5.0

I saw in the Readme that there are differences between the two versions. But this one seems to be very different. Is there a way for me to achieve results that are as close as possible? I'm using both libraries for validation. I come into a bad situation telling the user in the frontend that the password strength is good. But after submitting the server says that it's not.

So without disabling the password matcher, I can see where we have some differences.
Here is the output with just the default settings from nbvcxz:
> java -jar .\nbvcxz-1.5.0.jar
Commands: estimate password (e); generate password (g); quit (q)
Please enter your command:
e
Please enter the password to estimate:
Submit new password
----------------------------------------------------------
Time to calculate: 75 ms
Password: Submit new password
Entropy: 22.098669538287247
Your password does not meet the minimum strength requirement.
Warning: This is a top-10 common password.
Suggestion: Add another word or two. Uncommon words are better.
Suggestion: Capitalization doesn't help very much.
Time to crack: ONLINE_THROTTLED: 25 days
Time to crack: ONLINE_UNTHROTTLED: 12 hours
Time to crack: OFFLINE_BCRYPT_14: 36 minutes
Time to crack: OFFLINE_BCRYPT_12: 9 minutes
Time to crack: OFFLINE_BCRYPT_10: 2 minutes
Time to crack: OFFLINE_BCRYPT_8: 33 seconds
Time to crack: OFFLINE_BCRYPT_5: 4 seconds
Time to crack: OFFLINE_SHA512: instant
Time to crack: OFFLINE_SHA1: instant
Time to crack: OFFLINE_MD5: instant
-----------------------------------
Match Type: DictionaryMatch
Entropy: 10.647458426454921
Token: Submit
Start Index: 0
End Index: 5
Length: 6
Dictionary: english
Dictionary Value: submit
Rank: 802
Length: 6
Leet Substitutions: false
Reversed: false
Distance: 0
-----------------------------------
Match Type: SeparatorMatch
Entropy: 3.3219280948873626
Token:
Start Index: 6
End Index: 6
Length: 1
-----------------------------------
Match Type: DictionaryMatch
Entropy: 3.8073549220576037
Token: new
Start Index: 7
End Index: 9
Length: 3
Dictionary: english
Dictionary Value: new
Rank: 14
Length: 3
Leet Substitutions: false
Reversed: false
Distance: 0
-----------------------------------
Match Type: SeparatorMatch
Entropy: 3.3219280948873626
Token:
Start Index: 10
End Index: 10
Length: 1
-----------------------------------
Match Type: DictionaryMatch
Entropy: 1.0
Token: password
Start Index: 11
End Index: 18
Length: 8
Dictionary: passwords
Dictionary Value: password
Rank: 2
Length: 8
Leet Substitutions: false
Reversed: false
Distance: 0
----------------------------------------------------------

So I see that the "submit" match has a much higher rank in our dictionary than it does in the zxcvbn dictionary. That's the main difference in the scoring for this example.
It seems like it would be useful to provide a "zxcvbn compatibility mode" which ensures we use the same dictionaries with everything ranked exactly the same, disable any extra features zxcvbn doesn't support, etc. Until someone provides a PR that works in that direction, or I get time to do it - the workaround:
You'll need to manually configure the dictionaries to be the same as the ones used by zxcvbn, not using the default dictionaries.  You can also disable Levenshtein distance to get closer results to zxcvbn.

Thank you!


Add crashlytics to project

I need a free crash reporter service to analyze app.

Add sentry fermium plan to your project.
https://sentry.io/signup/

Resolved in #43


Accelerate CPI by using batch prediction and Numpy array operations instead of for loop

I think that the loop over permutations in the CPI.predict function could be avoided by creating larger arrays, predicting over a batch and reshaping the output.
Old version:
for _ in range(B):
   X_permuted.shape = NxP
   y_pred_list.append(model.predict(X_permuted))
y_pred_perm = np.array(y_pred_list)
y_pred_perm.shape = B x N

New version:
X_permuted_all.shape = NBxP
y_pred_perm = model.predict(X_permuted_all)
y_pred_perm.reshape(B, N, )

Running a benchmark on my laptop for n_permutations = 100 leads to 30% faster.

A decent win indeed ! I guess that the arrays are small enough that having a big one does not harm performance.


A way to cancel dispatching events for RN.

Right now there's no way to cancel dispatching an event for React-Native.
Assign defaultPrevented to a returned value of the onChange callback for React-Native to prevent dispatching onChange/onFocus/onBlur for some cases.

Published in v8.1.0.


adding MANIFEST.in

adding MANIFEST.in

Pull Request Test Coverage Report for Build 643

0 of 0   changed or added relevant lines in 0 files are covered.
2 unchanged lines in 1 file lost coverage.
Overall coverage remained the same at 86.171%

Files with Coverage Reduction
New Missed Lines
%

spaghetti/network.py
2
90.0%

Totals

Change from base Build 642:
0.0%

Covered Lines:
997

Relevant Lines:
1157

💛  - Coveralls


Update link to recipes

Updated link to the "Testing recipes" page
It was: /docs/recipes.html
Now: /docs/testing-recipes.html
Full link: https://ru.reactjs.org/docs/testing-recipes.html

@xom9ikk thanks!

@xom9ikk thanks!


NumericUpDown tab index ignored on Windows

Describe the bug
TabIndex doesn't work for NumericUpDown.
To Reproduce
Create a window with a TextBox, a NumericUpDown, and a Button. Set the tab indices to 0, 1, and 2. At runtime, tab through the controls - on tabbing the focus will move from TextBox to Button and only after Button to NumericUpDown.
In general, all controls except NumericUpDown will receive focus first and only then will the NumericUpDown receive focus - i.e., NumericUpDown focus on tabbing is always last.
Expected behavior
Expect that specified tab index would be followed.
Avalonia version
11.0.11
OS
Windows
Additional context
No response

Try 11.1-rc2 builds. Tab/focusability was changed in the NumericUpDown.

In 11.1.0, this is still a bug.

Still a bug in 11.2.  It seems that setter of TabIndex property needs to set the same value on the NumericUpDown's inner TextBox.TabIndex, as that's what actually gets :focus-visible.  Please fix.


⚠ Turnos 24x7 has degraded performance

In 34e8111, Turnos 24x7 (https://reservas-dev.alternativasinteligentes.com/) experienced degraded performance:

HTTP code: 200
Response time: 1380 ms

Resolved: Turnos 24x7 performance has improved in bef4d03.


Source EIA AEO from PUDL

Feature Request
For future work that utilizes the AEO- we should go to the PUDL table that contains this data.
Data:
https://catalystcoop-pudl.readthedocs.io/en/nightly/data_dictionaries/pudl_db.html#core-eiaaeo-yearly-projected-generation-in-electric-sector-by-technology
Relevant modules:

We'll need to access to this data in build_demand that's likely the place where accessing the sqlite db makes the most sense.

Suggested Solution
No response
Additional Info
No response

@ktehranchi Exploring the PUDL data from the AEO, I think maybe discussing this first would be good.
Right now I am planning to scale electricity only demand by total electricity projections, or end use energy demand for sector studies. PUDL is giving us electricity generation, and end use fuel (natural gas, coal, ect...), which are not sector specific, or only electricity sales for sector specific. This makes it difficult to coordinate a single dataset for both electricity/sector studies as PUDL does not give energy consumption by sector (which AEO does have).
Alternatively, I could use AEO heating/cooling degree day projections as a proxy for cooling and heating demand growth in the residential/commercial sector, but that will make breaking out electricity specific from heating/cooling difficult, and isnt really a great solution for the industrial sector.
Do you have any thoughts on this? Right now I would propose to just use the current AEO demand we pull to scale demand, as that way everything comes from a single source and we minimize conditional logic.

Yep- I agree it's definitely a good idea to source the AEO projects from one single point since PUDL doesn't have sector-level projects from AEO. Good point!

Scaling electricity growth now comes from PUDL. Sector studies still pull from EIA API directly.


Cannot bounce Puppetfile.lock!

When I try to run librarian-puppet install I get a Cannot bounce Puppetfile.lock! error.
The duplication was caused by having the dependency define in my metadata.json as well as the Puppetfile. Moving everything to use Puppetfile now...


Documentation: Stubbing a request with any query

I haven't found how to do this in the examples, so I'm filing this as a documentation issue.
How do I stub a request to a particular URL ("example.com/a/b"), not caring about the query string?
If I only stub the URL, it doesn't hit the stub.

How about regular expression?
https://github.com/bblimke/webmock#matching-uris-using-regular-expressions

That's how I worked around it. I thought there'd be a first class way to do this, okay then

@joallard alternatively .with(query: hash_including({}) but it's also a workaround.
WebMock treats query as part of the url, therefore if it's empty, it expects it to be empty.
Perhaps that should be changed in the future.

Thanks @bblimke !


feature(image): Add dark mode and medium zoom support to docs

All the images are located in the public/_images directory. After this PR, you should use the ACImage component to add an image to the doc instead of the ![](...) way. The mode property is used to control the display of the image in light or dark mode.
Eg.
<ACImage src="/_images/path_to_light_mode_image.jpg" mode="light" />
<ACImage src="/_images/path_to_dark_mode_image.jpg" mode="dark" />


Adding a License

Hi,
Thank you very much for open-sourcing this amazing repo! Would you consider adding a license? e.g. the MIT License.
https://docs.github.com/en/repositories/managing-your-repositorys-settings-and-features/customizing-your-repository/licensing-a-repository
Thanks!

Good suggestion. I have added the license (MIT).
Thank you.


NEW Method selection is now aware of "availability overrides" via Redux reducers

This pull request changes a couple of things:

Move Redux store provider to the boot script
Add a "withMethodAvailability" HOC which allows Redux reducers from third party modules to provide extra info on whether methods are available in the current context (e.g. if the browser is unsupported)
Updates enzyme and enzyme-react-wrapper-16 to the latest versions
Adds a new state for the Verify component where a method is selected but is not available (e.g. if WebAuthn is your default method and you log in using Safari)
Adds a new "ExclamationMark" component to render this:

Todo (separate issue): style the "more options" list when a method is not available
Resolves https://github.com/silverstripe/silverstripe-webauthn-authenticator/issues/13

Needs more docs, hold

The SilverStripe 4.1 branches have been deleted, I've removed it from our Travis matrix


Divide preClose() of AbstractWizardStep to "Previous Button clicked" and  "Next Button clicked"

I want to validate the fields of one step only if next button is clicked.
Right now it is not possible to react only on the click of "next"-button.

@mariodavid
Hi Mario,
kannst du dir mal anschauen, ob die Änderung sinnig ist? Es gibt auch schon einen Pull-Request dafür.
VG Nico

Hi,
Will answer in english, so that other people can also read the conversation :)
Thanks for the issue and PR. Perhaps you can elaborate a little bit on how thhe usage looks like? Perhaps i'm just missing the context a little. What peoblem does it solve?
Probably it also helps to know the use case when thinking about doing an implementation on the modern CUBA 7 UI APIs.
Thanks
Mario

Hi,
I just released version 0.8.0 which supports CUBA 7 based Screen APIs as well as the use-case you described with the possibility to identify which button was clicked.
Please try with the new version. I will close this for now.
Cheers
Mario


Release 0.21.0

Added

Add support to TaskFlow and improve dynamic task mapping support by @tatiana in #314
Render YML DAG config as DAG Docs by @pankajastro #305
Support building DAGs out of topologically unsorted YAML files by @tatiana in #307
Add support for nested task groups by @glazunov996 and @pankajastro in #292
Add support for templating on_failure_callback by @jroach-astronomer #252

Fixed

Fix compatibility with apache-airflow-providers-cncf-kubernetes>=10.0.0 by @tatiana in #311
Refactor telemetry to collect events during DAG run and not during DAG parsing by @pankajastro #300

Docs

Fix reference for HttpSensor in README.md by @pankajastro in #277
Add example DAG for task group by @pankajastro in #293
Add CODEOWNERS by @pankajkoti in #270
Update CODEOWNERS to track all files by @pankajkoti in #276
Modified Status badge in README by @jaejun #298

Others

Refactor dynamic task mapping implementation by @tatiana in #313
Remove pytest durations from tests by @tatiana in #309
Remove DAG retries check since many DAGs have different retry values by @tatiana in #310
Lint fixes after running  pre-commit run --all-files by @tatiana in #312
Remove redundant exception code by @pankajastro #294
Add GitHub issue template for bug reports and feature requests by @pankajkoti in #269

Closes: #223

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

Project coverage is 92.69%. Comparing base (80b885e) to head (7677ee1).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #315   +/-   ##
=======================================
  Coverage   92.69%   92.69%
=======================================
  Files          10       10
  Lines         726      726
=======================================
  Hits          673      673
  Misses         53       53

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


Update Startup2.cs

Being more explicit with the naming of these variables allows for readers of the code to easily digest what the variable actually is and what it's purpose may be.
Related
#17505
#17504

So I went down this rabbit hole and did a regex search for any lamda expression with single letter variables. I plan to update all of these with a succinct and more descriptive name.
My goal is to make the docs friendlier to new devs that may not have a grasp on lambda yet. A single letter may be descriptive enough with the surrounding context but I argue that this can lead to confusion and wasted brain cycles to determine a possible type for the variable in the lambda. This is especially so when you don't understand what a lambda is doing.
I'm also finding that sometimes the single variable letter doesn't even match the overall type name, which is further confusing.
All this to ask, would this change be considered if I go through and update all the single letters to meaningful names? I have begun but my simple regex has now uncovered ~1200 instances to be chagned.

I'm open to counter points/arguments.

Waiting for single PR


Add release notes section for remote write

#sign-off

@bwren Please have a look at this PR when you get a chance.

@bwren
Can you review the proposed changes?
Important: When the changes are ready for publication, adding a #sign-off comment is the best way to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team


Add cameraAutoOrbit, defaultAnimation, and animationAutoPlay to Viewer

The main change here is to add a few more Viewer properties/attributes:

Property
Attribute

cameraAutoOrbit
camera-auto-orbit-disabled

defaultAnimation
default-animation

animationAutoPlay
animation-auto-play

I also added a _createPropertyBinding helper function that just simplifies creating a "two way binding" between a property on the lower level state (Viewer, Scene, etc.) and the higher level HTML3DElement. Not all properties fit this pattern, but for the ones that do it is simpler (and I expect there will be more).

Please make sure to label your PR with "bug", "new feature" or "breaking change" label(s).
To prevent this PR from going to the changelog marked it with the "skip changelog" label.

Snapshot stored with reference name:
refs/pull/15759/merge
Test environment:
https://snapshots-cvgtc2eugrd3cgfd.z01.azurefd.net/refs/pull/15759/merge/index.html
To test a playground add it to the URL, for example:
https://snapshots-cvgtc2eugrd3cgfd.z01.azurefd.net/refs/pull/15759/merge/index.html#WGZLGJ#4600
Links to test babylon tools with this snapshot:
https://playground.babylonjs.com/?snapshot=refs/pull/15759/merge
https://sandbox.babylonjs.com/?snapshot=refs/pull/15759/merge
https://gui.babylonjs.com/?snapshot=refs/pull/15759/merge
https://nme.babylonjs.com/?snapshot=refs/pull/15759/merge
To test the snapshot in the playground with a playground ID add it after the snapshot query string:
https://playground.babylonjs.com/?snapshot=refs/pull/15759/merge#BCU1XR#0

Visualization tests for WebGPU (Experimental)
https://snapshots-cvgtc2eugrd3cgfd.z01.azurefd.net/refs/pull/15759/merge/testResults/webgpuplaywright/index.html

WebGL2 visualization test reporter:
https://snapshots-cvgtc2eugrd3cgfd.z01.azurefd.net/refs/pull/15759/merge/testResults/webgl2playwright/index.html


SwiftUI Support

With SwiftUI coming, it would be best if SideMenu supported it more readily without relying on UIKit. As of this writing, there are still some issues with presentation transitions as well as a lack of adoption in the community, but as that adoption changes it will become increasingly necessary to adapt SideMenu with it.
Voice your support on this feature request to help build interest in this feature. Additionally, if you are interested in helping create this next version, we can use the help!

I managed to integrate SideMenu with SwiftUI, but it was not easy, and it is not problem-free. Some notes, for anyone wanting to do the same:

The menu view controller can be easily implemented by means of UIHostingController.
A big problem is how to get the UINavigationController to set the pan gesture. This controller is not available from SwiftUI (there are some StackOverflow questions asking this). To get the navigation controller, I created an extension of class UINavigationController and overrode viewDidLoad. In that function, I stored a weak reference to self in an external list of UINavigationController, which later I use to set the pan gesture. To distinguish the "good one", I choose the one whose debug description contains the string "UINavigationController"; I am sure there are better ways, btw.
Another problem is that after dismissing the menu the first time, it never responded to tap gestures ever again. As a workaround, I re-generated the view controller every time the menu is hidden.

Thanks @noe. If you're comfortable sharing any of your implementation details here, please do.
I have also done some experimentation getting SideMenu to work in SwiftUI. These are some of the challenges I've noted:

Custom transitions (which SideMenu and UIKit use) do not seem to be supported from within SwiftUI; they do work when switching to UIKit and using UIKit to manage presentation.
In a pure SwiftUI implementation, a side menu would need to be defined at or near the root view hierarchy to function as it does currently. This is something I would prefer to avoid as it complicates implementation for novice developers. The native .sheet(...) call to present a view can be done from anywhere within the view hierarchy and appear from the bottom of the screen.
Passing back NavigationLink actions to the view that displayed the SideMenu in an intuitive way that does not require multiple NavigationLink to be defined on any screen that displays a SideMenu.

About this:

In a pure SwiftUI implementation, a side menu would need to be defined at or near the root view hierarchy to function as it does currently. This is something I would prefer to avoid as it complicates implementation for novice developers. The native .sheet(...) call to present a view can be done from anywhere within the view hierarchy and appear from the bottom of the screen.

The menu content can be placed in an ObservableObject that is then placed in the environment. Any view can then change it dynamically by retrieving the object from the environment and setting the new contents

I created a gist with the SwiftUI component that encapsulates SideMenu: https://gist.github.com/noe/d177bb9758f047ffc11b151d36bf5b53

Another problem that I found and forgot to mention:

The navigation bar does not play well with the side menu when using SideMenuPresentationStyle.menuSlideIn: the menu content is drawn just below the navigation bar, but the navigation bar is also occluded by what seems to be the menu itself. The separation line between the navigation bar and the rest of the screen is drawn over the menu, though. Here you can see a screenshot of the upper part of the screen where the menu is half-open. I added a red border to the SwiftUI menu to highlight what the drawable area is. You can see how the navigation bar ("My app") is occluded by the menu as it is open, but I can't draw on that part.

@noe this is wonderful, thank you for sharing. I'm still getting my bearings with SwiftUI, so there are going to be some things I haven't yet learned that will be helpful here -- perhaps we can collaborate.
My approach with SideMenu is for the most painless integration possible. With this in mind, I've tried to define what I think is the most sensible interface and then seeing if I could make that work through code without introducing ugly hacks. My current approach is this:

SideMenu should be displayable similar to how other screens are displayable (in UIKit it works with present/dismiss). Currently, .sheet(...) is the supported way to present a new screen (from anywhere in the view hierarchy) and so I am looking to follow this pattern.
SideMenu should allow for SwiftUI to be used within it.
SideMenu should deliver the existing set of functionality as much as possible, abstracting away behaviors wherever they can be so that the developer doesn't have to write code. Some notable challenges so far include passing back navigation actions to the main screen and gesture support.

Generally, I like your approach, however I do have some concerns:

It looks like you have to define SideMenu as your root view object, and you wrap both your main content and menu content underneath it. Per goal 1 above, I'll consider this implementation only if I can't find a clean way to implement its interface similar to .sheet().
View hierarchy inspections/traversals, e.g.:

    fileprivate func connect(controllerFactory: IControllerFactory) {
        self.rootController = (UIApplication.shared.windows.last?.rootViewController)!
        self.controllerFactory = controllerFactory
        replaceController()
    }

This may breakdown when multiple windows are used or if the developer switches to UIKit in other places.
extension UINavigationController {
    open override func viewDidLoad() {
        if self.debugDescription.contains("UINavigationController") {
            SideDrawerControl.registerNavigation(navigation: self)
        }
    }
}

This may not work when viewDidLoad is overridden or subclasses are used (unconfirmed).
I'm working on a beta now that I'll make available for your feedback soon; stay tuned.

I totally agree that my implementation is fragile and tailored to my use case. I'm looking forward to your progress @jonkykong !

For the .sheet way, I suggest you implement an extension to View together with a view modifier, much like they do in this popup library.

We're on the same page:
@available(iOS 13.0, *)
extension View {
    public func bottomSheet<Content: View>(isPresented: Binding<Bool>, onDismiss: (() -> Void)? = nil, @ViewBuilder content: @escaping () -> Content) -> some View {
        modifier(SideMenuModifier(isPresented: isPresented, content: content))
    }
}

The navigation bar does not play well with the side menu when using SideMenuPresentationStyle.menuSlideIn: the menu content is drawn just below the navigation bar, but the navigation bar is also occluded by what seems to be the menu itself. The separation line between the navigation bar and the rest of the screen is drawn over the menu, though. Here you can see a screenshot of the upper part of the screen where the menu is half-open. I added a red border to the SwiftUI menu to highlight what the drawable area is. You can see how the navigation bar ("My app") is occluded by the menu as it is open, but I can't draw on that part.
.

About the issue that I was having regarding the navigation bar not playing well with the side menu when using SideMenuPresentationStyle.menuSlideIn, what was happening is that the menu UIHostController was painting a navigation bar header. It went away by having a .navigationBarHidden(true) in the menu contents view.


Bump language version of crc go module. Bump go version

Minor changes. Bump go version according to openshift golang version.
https://github.com/openshift/release/blob/master/projects/origin-release/golang-1.13/Dockerfile#L9
I don't know why it's not the latest go 1.13 version which is 1.13.12. Perhaps someone here will know?

@guillaumerose The go version we have in the CI should be matching with what RHEL offical repo have (which is still 1.13.4) so that when we do the release (happen on RHEL machine) we shouldn't have any surprise there.

@guillaumerose we should take https://github.com/code-ready/crc/pull/1316/commits/4de6ee99d4c2f558a10f5bf80946cce2a44e40bd one in but not the go version one.

Thanks @praveenkumar ! I remove the commit about the upgrade.


Feature/es2015 features

@chrisrng ,
I was able to resolve the merge conflicts from #21 (linter) in this new branch, and updated whatever I needed to pass the new linting standing with the new build and verified the Demo page.
Anyways, closing #17 , and moving all our previous agreed comments to this PR.
Please take a quick good now and see if this PR is good to be merged, after having the to-to-date code & passing the linting?
P.S.: I have some question linting standard v.s. liberty of whitespace, which I will ask @thehindenburg , in the comment.

Hi @thehindenburg , good job on the linting config.
Just a question on the white space rule. I was block on a simple pattern such as:
if ( xyz = {} ) {
  ...
}

Is it possible to ease the level of whitespace linting to allow whitespace in block condition? It seems a gain in readability for code review. Plz have a consideration. Thanks.

Merging it thanks


Add CHANGELOG to rendered docs

I was looking for changes in the docs and they're not included. I (weakly) prefer everything being in one place since that's how our other projects track changes and I've developed a habit of expecting it there.
Changes made in this Pull Request:

Render CHANGELOG in docs
Slightly pretty-ify some of the CHANGELOG

PR Checklist

[ ] Tests?
[ ] Docs?
[ ] CHANGELOG updated?
[ ] Issue raised/referenced?

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

Project coverage is 66.34%. Comparing base (e0fc989) to head (db596b5).

Additional details and impacted files

https://openff-docs--116.org.readthedocs.build/projects/nagl/en/116/CHANGELOG.html#v0-3-8-2024-04-11

Pulling the trigger on this one just so I don't forget about it for a year :)


TestAdapterPath in runsettings does not take relative paths

Description
The TestAdapterPaths XML section does not understand relative paths unlike ResultsDirectory. This however works on using a testadapterpath commandline switch. With most adapters dropping the test adapter binaries in bin\debug, this inconsistency surfaces more.
Steps to reproduce

Create an MSTest V2 test project.
Form a runsettings with a TestAdapterPath in the RunConfiguration section.
Run vstest.console with this runsettings.

Expected behavior
The relative test adapter path gets honored and tests are discovered and run.
Actual behavior
The adapter path does not get honored and tests do not run.
Environment

Please share additional details about the test environment.
Operating system, Build version of vstest.console

AB#1865835

It looks like this is still a problem in VS 2017 15.7 Preview 3.0 and in vstest.console.exe 15.7.0-preview-20180320-02.
Is there a recommendation what to do, if devs have their sources on different locations on their machines?

Dear vstest team, any update of it?

This is a new feature and won't be implemented, we are focusing on adding new features to Testing.Platform instead. https://aka.ms/testingplatform


Fixed bug in mirc parser for timestamp, added mirc and skype test logs.

Awesome, thanks. I'll fix up the tests for them now.


Wrong version in composer file

In the release 1.2.0 is in the composer file the version 1.1.0 inserted. So I got the hint that there is a new version.. ;)

Good catch! Thanks for letting me know, I will fix this tomorrow with a patch release.

Fixed in 1.2.1. Sorry for the delay.


Saving the frame buffer in 1.4x ospexampleviewer.cpp

Hello ,
Can you please tell how to save the frame buffer in ospexampleviewer.cpp . I tried this code but soesnt work. Please advise
	auto sgFB = renderer.child("frameBuffer").nodeAs<sg::FrameBuffer>();
		sgFB->child("size") = vec2i(width, height);

	if (!imageOutputFile.empty()) {
			auto *srcPB = (uint32_t*)sgFB->map();
			utility::writePPM(imageOutputFile + ".ppm", width, height, srcPB);
			sgFB->unmap(srcPB);
		}

What exactly is not working?


Nodeserver RTA vs Elk Keypad Burglar Alarm

When an alarm is activated and is disarmed from Elk Nodeserver, or an user code is entered from Elk keypad, alarm is silenced and Elk Nodeserver returns to "Ready to Arm". However, Elk keypad is reading "Burglar Alarm" with the zone causing the alarm. As previously discussed, Elk requires the entry of user code twice to return to RTA. Nodeserver, requires two "Disarmed" entries to accept an arm entry.
Previous issue was for an update to documentation. This issue is for Nodeserver displaying RTA before it is ready. See photo & screen shot.

Nodeserver RTA in Alarm.docx

The NS doesn't have control over this, Elk is sending back Ready To Arm so that's what is displayed.  Does the ISY module behave differently?  If so then I can look deeper, but not sure what I can do?

ok

Just not sure what I can do, I will have to put my system in testing mode this weekend if I have time and see what comes thru in the API, but something must be there if ISY module is showing it...

Jim, I wouldn't worry too much about this. I just thought it would be nice to have something to grab your attention. The Arm Up State, Alarm State, & Armed Status match between the Nodeserver and the Elk Module. The difference being, the admin console shows a red flashing light next to Burglar Alarm that is readily visible.
Short of being able to change font color or flash work Burglar Alarm in the Nodeserver, you probably can't do anything and my guess is Polyglot doesn't support any of these options.

I still need to test this to see what the ELK returns and elkm1_lib has available.

Jim, I have a developers version of RP2 and a few other testing apps, I might be able to provide some helpful guidance. I don't know how often you check GitHub si I'll message you from the forum with a link..

I get an email on all replies here.
Do you have a proposal to resolve this issue?

I just updated IoX and PG3x, let me see if the upgrade was successful and we'll take it from there Jim.

Jim, would you like me to find out if Kelly Carter at Elk's Engineering Department to see if has a more recent version of elkm1_lib?

Thanks, but the elkm1_lib is not developed by Elk, unless this guy works for them https://github.com/gwww/elkm1


Supporting arm64-architecture

Describe the problem/challenge you have
Currently, a nfs-server-image (itsthenetwork/nfs-server-alpine, provisioner/helper_kernel_nfs_server.go ) is used, which is only available for amd64 at latest tag.
Describe the solution you'd like
I think the best solution would be to use a Image with arm64-support. I solved this issue locally by building the used image for amd64 + arm64 (i'm using a mixed setup) and pushed it to the kubernetes-registry. Then I changed the deployment-yaml using this image instead.
Used build-command: docker buildx build --platform linux/amd64,linux/arm64 -t [REGISTRY_IP]:32000/itsthenetwork/nfs-server-alpine:latest --push .
The second solution would be imho to make it configurable which image is used.
Info: With this changes it is possible to mount a pv.
Environment:

OpenEBS version (use kubectl get po -n openebs --show-labels): 2.7.0
Kubernetes version (use kubectl version): v1.20.4-34+1ae8c29bbb48f7 (microk8s)
Cloud provider or hardware configuration: rock64, raspberry pi (4), intel nuc 5i3
OS (e.g: cat /etc/os-release): 3x Ubuntu 20.04
kernel (e.g: uname -a): Linux nuc 5.4.0-70-generic, Linux rock64 5.10.21-rockchip64, Linux pi4 5.4.0-1032-raspi
others: using jiva as backend

@logi85 Thanks for posting this issue. We are working on it to add the support for arm64

Both the solutions need to be implemented.
(a) Use a multi-arch image that supports both arm and amd64. This will be useful for clusters that have both arm and amd64 nodes.
(b) Make the image name configurable. This will be useful for air-gapped clusters that will not be able to download the default image from the public internet.

Closing this issue since multi-arch image changes are merged. Link for PR https://github.com/openebs/dynamic-nfs-provisioner/pull/26
Above PR has added support for the following architecture:

linux/amd64
linux/arm64
linux/arm/v7
linux/ppc64le


Improve accessibility

I want to merge this change because it improves accessibility (increased font size and contrast) and brings brand new light and dark theme ☀🌙
Screenshots

Pull Request Checklist

[ ] Privileged views and APIs are guarded by proper permission checks.
[ ] All visible strings are translated with proper context.
[ ] All data-formatting is locale-aware (dates, numbers, and so on).
[ ] Database queries are optimized and the number of queries is constant.
[ ] Database migration files are up to date.
[ ] The changes are tested.
[ ] The code is documented (docstrings, project documentation).
[x] GraphQL schema and type definitions are up to date.
[x] Changes are mentioned in the changelog.

I reviewed the new UI with @pwgryglak today and here are a few issues we found that we could improve in this PR.

[ ] Missing left/right margins around "Something went wrong" box:

[ ] Order details page - incorrect alignment:

[ ] the "more options" icon should have circular background (see the mockups on Figma)

[ ] timeline section:

"leave comment" input looks differently on the design, it should have different background than the entire page and a border. Submit button should be placed within the input box. Also I noticed that the input isn't cleared after adding a comment.
increase margin above the "Order timeline" title and add a horizontal rule below it
timeline line and dots should be lighter - see the design

[ ] Order details page - "Shipping carrier" management should be rendered in a box below addresses (I'm not sure if that's in the scope of this PR though, is not we can skip it)

We could also fix this small thing:

[ ] In the greeting on the homepage we always use user email. We should render user's first name and last name if both are available, otherwise use email.

Do we want to automatically detect, if a user on OS X uses dark mode?
https://drafts.csswg.org/mediaqueries-5/#prefers-color-scheme


Vidio write error?

Hi ,  I check the output_ds.avi video. There must be something error occured.
The frame in video not has recognize and tracker areas and frame not change from start to end of the video?
Any way, the output video is so big.. ~9GB. My source video is 1.2GB.

What is your video resolution (width, height) and fps ?
I have fixed the fps of output video in line 62 in yolov8_deepsort_trt.py
out = cv2.VideoWriter('output_ds.avi',cv2.VideoWriter_fourcc('M','J','P','G'), 30, (video_width,video_height))

Try change this fps values.

My video's fps is 30 and 1980*1080.

When you add --show flag, can you see result (detected box and object id) ?

show is allways right.

I'm not build opencv.
I tested reading and writing video directly opencv and found the same thing. It's possible that it's a problem with opencv.

Maybe it caused by large file, let try with smaller before build OpenCV

What tool do you use to play this output video?

I think you need to try open output video with VLC

use the 'mp4v' it's ok. Don't know what happend...


build infra before backend

Backend depends on infra and without this change backend failed to build for me.

@stalep which OS are you using?

Fedora


Use gogo/protobuf

Update .ci to use gogo/protobuf generation
Regenerate protos

The change looks fine to me, could you please update statsdex/m3admin to use this version and make sure all the kv get/set + placement get/set still works as is? Make sure to use a test/dummy environment during the test

Okay sure.

Coverage remained the same at 83.649% when pulling 11d30f48f18198fc24dbb55ac57801db4af3277c on juchan/gogoprotobuf into e1b7e7ec6df77a79720a43318f78d13a7495de30 on master.


Indexing multiple buckets does not happen

Current oauth-login branch seems to not index multiple buckets. Seeing this on a local docker installation using this test bucket register which grabs both the oin-hotosm and oin-hotosm-staging buckets. The indexing seemed to complete on the oin-hotosm-staging bucket but did not index any of the files on oin-hotosm.
When testing to try to debug this, first time I was using the environment variable OIN_BUCKET=oin-hotosm-staging. This only indexed the oin-hotosm-staging bucket. When switching the variable to be OIN_BUCKET=oin-hotosm, then it only indexed the oin-hotosm bucket.
I think the correct behavior here should be that the API should index all buckets that are listed in the bucket register. We do need to define environment variables for the OIN_BUCKET, the final bucket for the Uploader workers, but that shouldn't limit what is indexed.
@tombh Have you seen this behavior?

Fix pushed.


Fasta: Kindly add warning that FASTA/Sequence formats don't support side chains and modifications

Steps to Reproduce

Switch to Macro mode
Open from Clipboard >> Select Sequence and Peptides
Add CCCAAATTT >> Add to canvas
Connect any first and last peptides

switch to Sequence mode - we can see link for letters

Save >> Fasta >> new connection is lost

Actual behavior
The structure is saved without any notifications to the user and side chain connections are lost.
Expected behavior
The user should be warned that certain info will be lost when saving to FASTA or sequence format.
In case modified monomers are included into sequence:
"The chosen format doesn't support modified monomers, they will be replaced by unmodified versions."
In case there are side connections (including cyclic structures):
"The chosen format doesn't support side-chain connections, These connections will be lost."

https://github.com/epam/ketcher/issues/4280

Versions:
• OS: Windows 10 Pro (version 22H2) x64
• Browser: Сhrome
• Version 122.0.6261.131 (Official Build) (64-bit)
Version 2.20.0-rc.1
Build at 2024-03-15; 15:36:45
Indigo Toolkit
Version 1.19.0-rc.1.0-g8cd9f1e27-wasm32-wasm-clang-12.0.0

Test added


Remote connection rendering on Windows

Hello,
Really love the idea of this project! The TUI works great until the connection is made, at which point the terminal of the remote machine is not shown and only a few characters of a typed command can be displayed, if at all. I am running on Windows 10 and I tried the new windows terminal with powershell, normal old cmd prompt, and the powershell terminal.  Screenshots below.
Thanks for your work on this exciting project and I hope this helps.
Edit: I built the binary using go 1.18 and I connected using the wishlist command with no flags or positional arguments, selected the host, and hit enter.

Same prodlem here, built from source using go version go1.18.4 windows/amd64.

hmm, just to be sure, a few questions:

are you running the wishlist in local mode?

if so, are you running it within WSL?
if not, is the wishlist server running in Linux or WSL?

does this issue happens accessing linux machines only?

thanks

For me, I am just using the "demo" link from your readme ...

I built it and using it on Windows 10 (no WSl involved)

Hey, adding to this. Rendering is messed up only on Windows, it works perfectly fine on my WSL instance. Using the newest release (v0.14.0) via scoop on Windows and installed it on my Kali instance via the method described in the README.
The Windows output looks as follows:

This is the Linux output:

In both cases, I am connecting to the same server. To answer the previous questions:

In both cases, wishlist is running in local mode.
No wishlist server instance is running on the server.
I have not been able to test if this happens when trying to reach machines that run any other system than Linux.

thanks for all the info!
can you check if #241 fixes this?

I put the screenshot in the PR comments as well, but this is what it looks like when using that branches files to compile the exe file myself:

awesome, thank you for verifying 🙏🏻


use-cases: Add TH module for tests

Add a Spec.TH module with quoted expressions for creating PendingTx
values

Thank you for taking the time to review the PR. I requested a review from you because I thought this would be the quickest way to get it done and it's only a small change. I will add some comments to the numbers.


Adding a friend list to user profiles where each friend links to the friend's profile page

This PR adds a button to a user's own profile page that allows them to access a list of their friends. The list of friends contains a list of friend emails, each of which links to a friend's profile page.

Closing comment:
This PR handles the friend list button functionality on a user's profile page. With the Friend List button, users are able to see a list of their friends displayed using a FriendTable. Each friend's email links to that friend's profile page. All assignees worked together on this functionality.

Issue Score: Excellent
Comments:
Great work!


Typescript type definitions

I see that you already have an index.d.ts file in the project. Would be great if that was included in the npm package, or atleast made available under @types/react-async-component

Agreed, sorry, I am not a TS user so it escaped me. :)
Would you like to submit a PR?

It should be included with the latest package.

Thanks :)


BIO: Add BIO_f_prefix(), a text line prefixing filter

We have a fairly clumsy indentation function, BIO_indent, which does a one time indentation, and the use of which requires the participation of all involved printing code.
BIO_f_prefix() is a BIO filter that was implemented in apps/ for a while, and that didn't have the indentation feature.  It can be used to set the currently desired line indentation as well as line prefix.  The resulting output is the concatenation of the prefix, the indentation and the line to be output.
A couple of adaptations are added, as a proof of concept:

the printing functions that implement openssl asn1parse
EVP_PKEY_print_{priv,pub,param}

Ping!

CI failures are relevant.

intention vs indentation...  tough stuff :wink:

CIs are happy

This adds a new API without tests.  Sad panda.

I'll try to find time to write a test tomorrow

API test for BIO_f_prefix() added.

Ping!

Merged.
a0848daee50b30f441813ecc90042c6af14c62aa crypto/bio/build.info: split the source files in categories
319cee9e2fc6fcf6ad865564eccdac4c55e92c0a BIO: Add BIO_f_prefix(), a text line prefixing filter
e79ae962fbed99cf80feb32b326f701778ca7434 APPS & TEST: Adapt to use the new BIO_f_prefix()
0e79d20f1e9f9e32bfe45eb3760cb233fa8dfa5e ASN1: Adapt ASN.1 output routines to use BIO_f_prefix()
f1299839b7763d293925d57b28f533ab24c084ad EVP: Adapt EVP_PKEY_print_ routines to use BIO_f_prefix()
51a7c4b5f2a0b2d0f6bc0c87ec2ee44b9697dc78 TEST: Add test recipe and help program to test BIO_f_prefix()


🛰 Future development planning / overview

Hey all,
after shorter discussion with @bangom and others I thought about the future of this daemon. While thinking about all potential functions and processes a map felt useful. Here we go:

Feel free to chime in and discuss. Contributions welcome!

1.1
This function heavily depends on the miflora library but should be implemented async and in proper object oriented manner. Linked to #107
Success rate: A function already available but not very visible. Should be published to all conventions. Linked to #106
Availability: Currently missing and important for smart home system integration. A sensor should go "offline" after x failed polling attempts.
1.2
Passive listening for btle broadcasts by the sensors should be part of the miflora library development and integrated with 1.1
1.3
Not available through the gatttool integration, scanning is now available through bluepy. Should be added as a function and results published to MQTT
1.5
Wherever possible, the daemon should be configured via MQTT. Propose to reduce required file/env based configuration to

MQTT connection parameters
daemon unique ID (with a fallback to a suited host identifier)
For usability purposes I propose to publish a complete default configuration to MQTT - if no retained configuration for the daemon unique ID exists

Will support for other Xiaomi sensors at some point be on the road map?

Yes! I forgot to highlight that but the new structure in #107 should support that. I envision an architecture in which a device is pretty much self-contained, dealing with it's configuration, btle sensor reading, and MQTT communication independently of all other sensors. We can discuss this in #107 if you like.

1.1
This function heavily depends on the miflora library but should be implemented async and in proper object oriented manner. Linked to #107
Success rate: A function already available but not very visible. Should be published to all conventions. Linked to #106

Shouldn't be a big problem to publish it as another plant sensor/entity (in HA for example sensor.plant01_btle_stability)

Availability: Currently missing and important for smart home system integration. A sensor should go "offline" after x failed polling attempts.

Could be useful. Again perhaps another sensor (in HA for example sensor.plant01_btle_state = online/offline).

1.2
Passive listening for btle broadcasts by the sensors should be part of the miflora library development and integrated with 1.1
1.3
Not available through the gatttool integration, scanning is now available through bluepy. Should be added as a function and results published to MQTT

This is a nice idea which can make discovering and adding new sensor more user friendly.

1.5
Wherever possible, the daemon should be configured via MQTT. Propose to reduce required file/env based configuration to

MQTT connection parameters
daemon unique ID (with a fallback to a suited host identifier)
(every other configuration aspect is still supported for classic deployment)

For usability purposes I propose to publish a complete default configuration to MQTT - if no retained configuration for the daemon unique ID exists

I'm not sure about daemon-config&management residing in Smart Home System (controller) - there are more kinds of controller right now (HA, OpenHab, ...). Maintaining more versions could be hard...
My idea was rather master-slaves architecture.
Defining sensor-slave ownership with "@" part in sensor name (sensor location isn't used in HA anyway :-)
Purpose of a slave is "set it and forget it" - slave daemons rebuild automatically theirs sensor list from MQTT subscribed topic, publishes local sensor data and publishes nearby sensors so they can be added to the master daemon (notification on master: "new nearby miflora sensor found with MAC / RSSI" ... ).
Purpose of master daemon - single entity for management, discovery, notification monitoring.

Details see the attached image below where the draft of the proposed architecture is depicted.

Really looking forward to the passive integration as in https://github.com/custom-components/sensor.mitemp_bt

I was actually looking for removing "sensors config" in a slave, and doing mapping MAC->actual plant later, so i don't need to touch slave if sensor or plant gets moved. That would also allow to get all sensor reading through all slaves and then filter/downsample after a broker?

Well, let's break down this daunting task into smaller, more ignoreable chunks:
For a start, subdividing the sleep (1) and checking for a "magic string" mqtt message in between (2) to skip wait (read: sleep; 3), thus triggering a whole script rerun would be nice!

Well, let's break down this daunting task into smaller, more ignoreable chunks:
For a start, subdividing the sleep (1) and checking for a "magic string" mqtt message in between (2) to skip wait (read: sleep; 3), thus triggering a whole script rerun would be nice!

Maybe https://pypi.org/project/schedule/ can help for a hot-feature-patch?
Regarding future development you could probably set up the projects tab? I'm not sure, if you/we are looking for an update to this repo or a totally "new" MiBLE2MQTT daemon, on top of (as @vekexasia mentioned) Xiaomi Mijia BLE MiBeacon Monitor Git?
Thus, @CWempe would get his otter sensors.

1.2 would be very useful in increasing battery life. However, as the battery level is not advertised anymore in the latest firmware, it would make sense to implement an hybrid approach where one could get all the passive BLE information while retaining the possibility to poll specific values at defined timeframes.
So for example, ones could poll the battery once or twice a day, and gather all the other information passively. Or, if someone as a particular need for a specific value at a specific interval, it can still be done while the other info will come in passively.

Hi, Home Assistant moved to a passive BLE approach that also polls to get more info when needed: https://www.home-assistant.io/integrations/xiaomi_ble/, using this library: https://github.com/Bluetooth-Devices/xiaomi-ble. This has greatly improved battery life. Is there any plan for 1.2? If not I may try to do it myself, but I don't want to duplicate work ;)

Any news on that xiaomi-ble implementation, that would allow better battery life ?
I indeed have to change my miflora CR2032 way too often (and wonder how related it may be to miflora-mqtt readings - only with 2 readings a day on my side)


Allow SelectComponent to take an option group

Currently, it's hardcoded to use options_for_select. It'd be nice if we could also use grouped_options_for_select.
It would be even nicer if we were able to use any of the methods from form_options_helper.rb without straying from the original API

Implemented in #313


ID resolver refactoring: the resolvable vs irresolvable types

We seem to be keeping a list of semantic types that are "resolvable" by the ID resolver here.
If we are still using this list, perhaps it needs to be revised...there are now some semantic types that we can resolve that we did not before (like ChemicalEntity and MolecularMixture) and some that we now cannot resolve (like SequenceVariant).
Use this link to see what semantic types the SRI-based ID resolver resolves (note we're using dev right now)

Related to the migration to the new SRI-based ID resolver #249

Those semantic types are only used by the old resolver, the new one returns the semantic types as a simple string without any of the type checking.

@ericz1803 Can you comment on whether "resolvable" vs "irresolvable" is a distinction that we still make and use somehow?

For semantic types, no. The SRI resolver doesn't use the semantic type list and there aren't any "resolvable" or "irresolvable" semantic types

@ericz1803 I was under the impression that the concepts of "resolvable" and "irresolvable" semantic types and ID prefixes was used to:
A. decide whether to send something to the ID resolver service (now the SRI dev url) to begin with....
B. how to choose / format the equivalent IDs / names (now the node's name / biolink:xrefs, biolink:synonyms node-attributes)
Is that not what is actually happening?
And is the concept of resolvable / irresolvable semantic types involved in the logic of other code, like the api-response-transform, query-graph-handler modules?

I don't know how it is used outside the id resolver but I can explain what is happening inside the id resolver. "resolvable" ids are those that the id resolver, either the old one or SRI can find equivalent ids for, "irresolvable" is the opposite. We need to handle them differently because of this.
Before, there were two classes, one for resolvable ids and one for irresolvable ids (which is what you see if you console.log the response). With the SRI resolver, we are just building plain JS objects. For the user who actually sees the resolver output, the response shapes are the same, as in both resolvable and irresolvable IDs have the same properties/fields.
With the SRI resolver, we just pass the ID to them regardless of semantic type or prefix and it will figure out if it can resolve them so there sort of is a concept of "resolvable" and "irresolvable" semantic types and ID prefixes but when the response is sent back to the user, it is all in the same format so I don't think there should be a difference in how anything that uses the id resolver deals with "resolvable" and "irresolvable" semantic types.

Alright, I'll close this then, to keep as reference. It sounds like this stuff is basically deprecated and shouldn't affect anything downstream...


Revisit if targeting netstandard1.7 in test specification is necessary

Based on https://github.com/aspnet/EntityFramework/pull/7321#issuecomment-269560864, I checked in a change to bump up the tfm of Microsoft.EntityFrameworkCore.Relational.Design.Specification.Tests to netstandard1.7. Using this to track if an alternate change is required to determine if we want a non-breaking fix for this.

cc @divega


App-package-gen and unity-app-gen must be pinned to releases not the main branch

https://github.com/unity-sds/unity-ads-deployment/blob/bc07f0d509a08823933bd212439ee962c8cc83b3/ci_cd/install_group_runner_x86_64_shell.tftpl#L187-L191
Anytime a new gitrunner is deployed (or if the instance crashes and a new one is brought up) it will fetch the lates code regardless of if it's been tested or not. By pinning this to a release or a specific commit hash, we  know what will be deployed regardless of when the deployment happens.

Any updates on this?

Use

app-pack-generator v0.4.0
unity-app-generator v0.3.0

Preliminary support here:
https://github.com/unity-sds/unity-ads-deployment/commit/ac234e815d8ce2bbb5aa6bcc118114f5ecf31654

This task is complete.  When the U-ADS "auto clone/build pipeline" is deployed, unity-app-gen and app-pack-gen are installed for the selected tags.  Currently, the defaults are 0.4.0 for app-pack-gen and 0.3.0 for unity-app-gen.

@edwinsarkissian or @mcduffie  please close the ticket if you're done.


Flight simulator sample

Sample
Flight Simulator in Unreal Engine 5
Summary
This flight simulator was developed with functionality to take off and land from any airport. Initial starting position is at LAX. Users can use WASD for acceleration and Yaw, and 4568 on the number pad for pitch and roll. A gamepad controller is also supported, using the right trigger for acceleration, left for deceleration, left thumbstick for yaw, and the right thumbstick for pitch and roll. Additionally, landing gear was added to the plane and can be hidden/shown using the H key on the keyboard or by pressing the right thumbstick down. The Jet has full collision detection allowing the user to only control certain aspects of the plane when traveling at above a certain altitude (roll and disabling landing gear). Additionally the user must be traveling above a certain speed to take off from the ground.
Tests

Windows

ArcGIS Maps SDK Version
Maps SDK Preview for Unreal Engine

Could be useful to have a control guide UI, but it can make the screen to crowded. @mrnelson6 opinion?
For me it feels more natural if the keys for going up/down were swapped, but that may just be me.
In the restart UI, the button label is blocking the button, so you can't press the button when clicking on the text. In the widget designer, Drag&drop the label onto the button to attach it to button. Change the color of the button to white (not the background color). I would also increase the button height so that's easier to press
Pressing the restart button also removes the SampleSelector UI
Map disappears when the plane is tilted
Should the landing gear be closed at the beginning?
Some nice-to-haves if there is ever time: add units to speed and altitude, add sound, add multiple start points that the user can choose from,

I also think that the up and down seems reversed
Can you add units to height and speed?
I think the speed is not really speed because it keeps increasing.
I see some tile loading issues


Updated the scs-app-maven version to 1.2.3

There is a new release for the maven plugin available. Updated to that version rather. Closing this PR without merging.


Confirmation log out pop out screen pull request

@Mayuresh-22  i think everything is good to Merge now , time to move to the other issue

I am merging this PR to a Test branch of Minglr


Version 2.2.2

Deleting this PR and this branch - the update wasn't needed as we were able to point CODAP's stable branch to the previous version.


ci(workflows): reviewed reusable workflows

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


[WFLY-15150][WFLY-14888] Hibernate ORM to support JDK 17

for @scottmarlow

Thank you very much @Sanne!!! Very much appreciated!

This is for https://issues.redhat.com/browse/WFLY-15150

Here's a run with WF Preview and JDK 17:
https://ci.wildfly.org/viewLog.html?buildId=268937&tab=buildResultsDiv&buildTypeId=WF_WildFlyPreviewLinuxJdk17


TensorRT encountered issues when converting weights between types and that could affect accuracy

Hi @mingj2021.
When I export the model with demo.py, I get the following warning. Is this what you would expect?
[TRT] [W] TensorRT encountered issues when converting weights between types and that could affect accuracy.
[TRT] [W] If this is not the desired behavior, please modify the weights or retrain with regularization to adjust the magnitude of the weights.
[TRT] [W] Check verbose logs for the list of affected weights.
[TRT] [W] - 77 weights are affected by this issue: Detected subnormal FP16 values.
[TRT] [W] - 18 weights are affected by this issue: Detected values less than smallest positive FP16 subnormal value and converted them to the FP16 minimum subnormalized value.

You mention that you are not maintaining this repo any longer. I was wondering whether you successfully tested inference with the exported model?
Also, please let me know whether there is any additional useful information to get a full TRT SAM model up and running.
Many thanks

ok，Update later a version

Thanks for your reply @mingj2021.
Am I correct in thinking that you will update the repo with a fix for this?
Have you tested the tensorrt engine model for inference on an image?
Many thanks

Hi @mingj2021.
When I export the model with demo.py, I get the following warning. Is this what you would expect?
[TRT] [W] TensorRT encountered issues when converting weights between types and that could affect accuracy.
[TRT] [W] If this is not the desired behavior, please modify the weights or retrain with regularization to adjust the magnitude of the weights.
[TRT] [W] Check verbose logs for the list of affected weights.
[TRT] [W] - 77 weights are affected by this issue: Detected subnormal FP16 values.
[TRT] [W] - 18 weights are affected by this issue: Detected values less than smallest positive FP16 subnormal value and converted them to the FP16 minimum subnormalized value.

You mention that you are not maintaining this repo any longer. I was wondering whether you successfully tested inference with the exported model?
Also, please let me know whether there is any additional useful information to get a full TRT SAM model up and running.
Many thanks

hi, have you solved this problem? I have the same problem.


Revert "Remove free disk space cleanup"

Reverts test-network-function/cnf-certification-test#1821

Not needed, changes done in: #1857


[cloudflare_access_policy] error creating Access Policy for ID "" when omitting application_id

Confirmation

[X] This is a bug with an existing resource and is not a feature request or enhancement. Feature requests should be submitted with Cloudflare Support or your account team.
[X] I have searched the issue tracker and my issue isn't already found.
[X] I have replicated my issue using the latest version of the provider and it is still present.

Terraform and Cloudflare provider version
Terraform v1.8.3
on linux_amd64
+ provider registry.terraform.io/cloudflare/cloudflare v4.33.0
+ provider registry.terraform.io/hashicorp/aws v5.50.0
+ provider registry.terraform.io/hashicorp/cloudinit v2.3.4
+ provider registry.terraform.io/hashicorp/kubernetes v2.30.0
+ provider registry.terraform.io/hashicorp/random v3.6.1
+ provider registry.terraform.io/hashicorp/time v0.11.1
+ provider registry.terraform.io/hashicorp/tls v4.0.5

Affected resource(s)
cloudflare_access_policy
Terraform configuration files
After upgrading to the latest module version, where it mentions using cloudflare_access_application.policies instead of linking policies by application id, I attempted a migration of an existing resource.
resource "cloudflare_access_policy" "this" {
  # Removed both fields below, as described in the docs.
  #
  # application_id = cloudflare_access_application.this.id
  # precedence     = try(each.value.precedence, "1")

  # ...
  zone_id          = data.cloudflare_zone.this.id

  include {
    # ...
  }

  require {
    # ...
  }

  lifecycle {
    create_before_destroy = true
  }
}

# then, updated the application with the reference to the policies, eg:

resource "cloudflare_access_application" "this" {
  # ...

  policies = cloudflare_access_policy.this[*].id
}

Creating the cloudflare_access_policy fails, since it seems the API does not allow for a "" value on the application id field.
Link to debug output
https://gist.github.com/cilindrox/84f369bc8a87638c724b95d98a65c156
Panic output
No response
Expected output
Success
Actual output
│ Error: error creating Access Policy for ID "": error from makeRequest: POST method not allowed for the api_token authentication scheme (10000)
│
│   with module.cloudflare_access.cloudflare_access_policy.this["my policy name"],
│   on ../../../modules/cloudflare-access/main.tf line 64, in resource "cloudflare_access_policy" "this":
│   64: resource "cloudflare_access_policy" "this" {
│

Steps to reproduce
Create an cloudflare_access_policy with no application_id or precedence
The token I'm using has the folowing:
Account:

Access: Organizations, Identity Providers, and Groups Edit
Access: Apps and Policies  Edit

Zone:

Access: Apps and Policies  Edit

Additional factoids
Tried switching to a global access key, and got the same results, which makes me think this is an issue with the API or the client/provider setting the field to an unsupported value, rather than the permissions/scope.
References
Change introduced in https://github.com/cloudflare/terraform-provider-cloudflare/pull/3288
and https://github.com/cloudflare/cloudflare-go/pull/1956

Thanks for the context @GreenStage - so if I'm understanding correctly, then I would need to change the reference and remove zone_id, keep application_id and precedence fields, and reference the policy from cloudflare_access_application.this.policies, correct?
resource "cloudflare_access_application" "this" {
  # ...

+  policies = [cloudflare_access_policy.this.id]
}

resource "cloudflare_access_policy" "this" {
+  application_id = cloudflare_access_application.this.id
+  precedence     = "1"

  # ...
-  zone_id          = data.cloudflare_zone.this.id
}

@cilindrox I got the exact error today and was able to resolve it by doing @GreenStage suggestion, which is removing zone_id replace it with account_id.
resource "cloudflare_access_policy" "this" {
  # ...

-  zone_id    = data.cloudflare_zone.this.id
}

resource "cloudflare_access_policy" "this" {
  # ...

-  account_id = data.cloudflare_account.this.id
}

Also, ensure you have the necessary permission added to the API token


Template command to move to the end of a file

It would be useful to be able to use goto (with a negative value?) or some other template command to move from/to the end of the file instead of the start/anchor; e.g., to read the footer from a TGA file (which is the only place that contains offsets for certain parts of the file).

A current workaround is something like
proc moveToEnd {} {
	while {![end]} {
		uint8
	}
}

But this seems really inefficient.


Jobs Refactor

This refactor is required as it has been pointed out accessing services from a job is quite hard with the current structure. The changes include a breaking up of what defines a job and an actual job instance with parameters. I've also added a JobIdentifier which also includes a reference to the parameter type used by the job to ensure we don't pass bad parameters to a job.
// define email job parameters
struct EmailJobParameters: Codable {
    let recipient: String
    let subject: String
    let message: String
}

// add identifier for email job and associate with EmailJobParameters type
extension HBJobIdentifier<EmailJobParameters> {
    static var email: Self { .init(name: "email-job") }
}

let ses = SES()

// register email job, with job functionality
HBJobRegistry.registerJob(.email) { parameters, context in
    try await ses.sendEmail(to: parameters.recipient, subject: parameters.subject, body: parameters.message)
}

// Create job queue and push email job to queue. Because we have the association between
// the identifier and the required parameters we can ensure the correct parameter type
// is passed in
let jobQueue = HBRedisJobQueue(...)
jobQueue.push(
    id: .email,
    parameters: .init(recipient<EMAIL_ADDRESS>subject:"Test", message: "Testing, testing, 123")
)

For reference this is what the Jobs framework did previously with comment indicating the issue I'm trying to fix
struct EmailJob: HBJob {
    let recipient: String
    let subject: String
    let message: String

    func execute(logger: Logger) async throws {
        // How do I get access to ses to send the email here?
        // I can't include it in the init as the job is recreated by queue
        // at a later date, I can't use variable capture as I would in a closure
    }
}

EmailJob.register()

jobQueue.push(EmailJob(recipient<EMAIL_ADDRESS>subject:"Test", message: "Testing, testing, 123"))


Uses pipenv to manage deps

Use pipenv to manage deps rather than using global pip hopefully not cause any issues such as #6
Basic guide:
brew install pipenv libffi
export PKG_CONFIG_PATH="/usr/local/opt/libffi/lib/pkgconfig"
pipenv install
pipenv run python3 brave.py

The diff I just made was required for it to work for me


Added 2 commands, to 1. add and 2. remove symlinks to local packages

All the local packages in the monorepo should be able to symlink to their required packages, so that when testing them (either executing their local vendor/bin/phpunit, or through other means, such as executing them, running a web server on them, etc), modifying the local source code will show the changes immediately, and avoid using the package from Packagist.
For this, I added 2 commands:

symlink-to-local-packages: It adds all local packages under entry repositories in all composer.json files, with symlinks
remove-symlink-to-local-packages: Inverse operation, it removes all local package paths from the repositories entry in all composer.json files

Why
In my particular case, one of my dependencies is a WordPress plugin. To test it, I execute a command on the plugin's folder, which starts a Docker container with a new WordPress instance, and the plugin active. The plugin references many local packages in the monorepo. With symlinking, modifying the source of any required package will be visible immediately on the web server.
It also makes it easier to contribute to the local packages.
How to use it
monorepo-builder symlink-to-local-packages is meant for development. The idea is to already commit all composer.json symlinking to the local packages to the monorepo. Then, any developer cloning the project, browsing to the package, and doing composer install, will already have dependencies symlinked to their source code in the same monorepo.
monorepo-builder remove-symlink-to-local-packages is meant for production. Hence, it must be executed within split_monorepo.yml before deploying the code to the target repo, so that the target repo does not contain the symlinks, and retrieves all packages from Packagist.

Running the commands currently is slow because of #2692. Once that PR is merged, this one will be much faster

Git checkout does the same job. This way each command is inverted thanks to git.

I'm not aware how git checkout does the same thing. Can you please explain? I need to have every package in the monorepo symlink to the other packages, for which I created symlink-to-local-packages... if I can't have this merged, then how do I do it?

This command is for testing, not for mergin.
In case you want to relase packages, the package aliases is way to go - https://tomasvotruba.com/blog/2019/01/31/monorepo-composer-magic/
See how we handle it in Symplify

There's no incompatibility. I tested it, and it works also using the branch-alias set to 5.6-dev or whatever, there's no need to set it to dev-master to enable symlinking.

This command is for testing, not for merging.

Ah, my bad, I directly replicated it under /testing. If I move it to /merge, then is OK?

In case you want to release packages...

To remove the symlinks, yes, that's release. But to add the symlinks in first place to all composer.json, that's not release, it's merge

Question: can the monorepo-builder attach external commands, somehow? If so, if you don't want these new commands (which I need), they could exist under my own project, and injected in.
But I don't want to use a fork of Symplify. I just want to inject my own commands to Symplify.

Btw, I'm thinking of an alternative approach to this PR:

Instead of adding the repositories to all composer.json, add them under a parallel composer.local.json file
Merge the composer configuration (composer.json + composer.local.json) using Composer Merge plugin

I've done a test, and it works well: all local packages are symlinked to each other.
Then, there's no need for command remove-symlink-to-local-packages: I simply skip copying file composer.local.json when splitting the monorepo.
Would you consider this approach? I can work on a new PR


[Front] Tailwind version 3.2.4 -> 3.4.3

Description
Notably due to min-h/w-* that are now supported and that we used in multiple places cf thread
Deploy

Deploy front
Update of sparkle to be done in separate PR

Actually you're right, this should be well tested and likely not worth the effort. will close cf slack discussion mentioned above


How does sending emails exactly work?

Question
I am new to craftcms and i found sprig and i saw that you can send emails but my question is: How does this exactly work???. SomeEmailApi doesn't really help me because i tried to use craft its standard mailing but that does not allow me to do Mailer::send([<junk of information here>]);   So right now i am just trying to do this in another way but i just cannot really figure out how it works.
Additional context
this is my ContactForm.php
<?php
namespace sprig\components;

use putyourlightson\sprig\base\Component;
use craft\mail\Mailer;
use craft\mail\Message;

class ContactForm extends Component
{
    public $success;
    public $error;
    public $email;
    public $message;

    protected $_template = '_sections/footer';

    public function send()
    {

            $message = new Message();
            $message->setTo('a0e008df25-ea1d0f@inbox.mailtrap.io');
            $message->setSubject("Hello world.");
            $message->body($this->message);
            // $message->setHtmlBody(Craft::$app->getView()->renderTemplate('emails/_someMail'));
            $message->send();

    }
}

This is my form:

    <form class="mb-6" sprig s-action="send" method="post">
        {{ csrfInput() }}

            Email
            <input class="border border-black py-2 px-3 text-grey-darkest w-50" type="email" name="email" id="email" value="{{ email }}">

          Message
          <textarea class="border border-black w-50" name="message" value="{{ message }}"></textarea>

          <button class="border border-green bg-green block text-lg w-25 p-4 rounded" type="submit">Send Email</button>

      </form>

If your question is about how to send emails using Craft, then please refer to this answer:
https://craftcms.stackexchange.com/a/23807/180


Deploy using ORM

Community Note

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue or have submitted a pull request, please leave a comment

Description

Allow public ssh keys to be entered as text
Allow ssh keys on node pools to be optional
Upgrade base module
Providers don't need extra parameters

New or Affected Resource(s)

Potential Terraform Configuration

# Copy-paste any Terraform configurations for how the requested feature may be used.

References

#191

Added examples to 5.x user guide in https://github.com/oracle-terraform-modules/terraform-oci-oke/pull/703


Guidance for float and double max and min values

Hi there
Can we get some guidance on
type: number
format: float

and
type: number
format: double

Maximum and minimum values?
It sounds like Some floating point formats use an unrepresented hidden 1 bit in the mantissa (this allows for one extra bit of precision in the mantissa per https://softwareengineering.stackexchange.com/questions/294269/how-to-calculate-min-max-values-of-floating-point-numbers

Should float be a single-precision 32-bit IEEE 754 floating point?
Should double be a double-precision 64-bit IEEE 754 floating point?

This has since been addressed by multiple additions to the formats registry around floating point and decimal representations.


Add tag that allows for dynamic recolouring of glyphs/sprites

In some cases, a developer might want to recolour certain portions of a text element without regenerating the whole text element. Whilst this is possible to program in manually, Scribble could support this natively.
Given issue #14 and the impending changes with GMS2.3.0, now might be the time to spruce up the shader effects system.

This won't be added as a native feature but will be added as an example of how to use the shader functions.

Gonna shelve this one until 2.3.0 hits stable.


Renders without any styling

Hi,
Thank you for sharing this component!  I am tried including this in my material-ui project, but the field renders without any styling or functionality.  Am I missing something?
import HtmlField from 'material-ui-html-field';

// Set the state in constructor.

// In render()
      <HtmlField
        floatingLabelText="Html Editor"
        value={this.state.html}
        errorText={this.state.htmlError}
        {...editorConfig}
      />

Hi, I have fixed the bugs at version 0.1.2


Can't get data from Kafka to distributed table

Hello, i've faced a problem with reading data from Kafka.
So, i need to transfer data from Kafka queue (JSON inside), which i created and filled with my Java script, to ClickHouse (CH).
In our CH we have a cluster with 6 replicas, data must be acceptable from each of them. That means i need to use Distibuted table 'data'.
Also i need 'queue' table (Kafka engine) and materialized view 'consumer' both on each replica (please correct me if i'm wrong).
'Data' as distibuted table must point to a source table (which it'll get data), so i pointed it to 'queue' (not sure about it).
While i'm putting data into Kafka i pretty sure that tables accept data (simple select count(*) from data (distibuted table)), but i always get this:

"Progress: 1.55 thousand rows, 1.24 MB (297.46 rows/s., 237.18 KB/s.) Received exception from server (version 18.14.17):
Code: 159. DB::Exception: Received from host:port. DB::Exception: Failed to claim consumer: .
0 rows in set. Elapsed: 5.313 sec. Processed 1.55 thousand rows, 1.24 MB (291.94 rows/s., 232.78 KB/s.)"

When i stop filling Kafka i have a short time window at which i can complete my query. But after a few seconds i recieve - 0 counts on every table i have created.
I'm kind of stuck for name and tilted about this problem. And i'm pretty sure thah i don't understand someting.
Could you please help me out or provide some idea about how i can achieve my goal?

Problem was on my side: invalid columns in materialized view 'consumer'.
Btw, if anyone will need to do the same task here's data map:

Create 'local' tables on all hosts in cluster;
Create distributed tables on all hosts in cluster;
Create Kafka engine table 'queue' + materialized view 'consumer' on one host.


opsec way to collect

Is there any opsec way to collect the ca template name?
I just need this name, no need to do any information gathering
[*] Trying to get CA configuration for 'red-DC-CA-1' via CSRA

Seems like he's going to do something about BloodHound anyway hahaha

Hello. Currently there are no options like this for Certipy. In order to find the CA name and address, Certipy must extract it from LDAP. At the moment, I'm not planning on modifying Certipy to accommodate this requirement as I simply don't have the time.
Best regards


Update notusknot.com

This PR is:

[ ] Adding a new domain
[x] Updating existing domain size
[ ] Changing domain name
[ ] Removing existing domain from list
[ ] Website code changes (512kb.club site)
[ ] Other not listed

[x] I used the uncompressed size of the site
[x] I have included a link to the GTMetrix report
[x] The domain is in the correct alphabetical order
[x] This site is not a ultra lightweight site
[x] The following information is filled identical to the data file

I confirm that I have read the FAQ section, particularly the two red items around minimal pages and inappropriate content and I attest that this site is neither of these things.

[ ] Check to confirm

- domain: notusknot.com
  url: https://notusknot.com
  size: 2.89
  last_checked: 2022-02-06

GTMetrix Report: https://gtmetrix.com/reports/notusknot.com/WQdhnE2f/

Ultra minimal. Won't be merging, sorry.


jQuery dep has problematic version in package.json

This notation: "dependencies": { "jquery": ">=1.8" }, accepts also breaking versions (like 2.x, 3.x etc.), so it breaks semver security. It also creates problems when installing with yarn a package which has jquery.scrollTo as a dep:
$ yarn install --flat
yarn install v0.27.5
[1/4] Resolving packages...
info Unable to find a suitable version for "jquery", please choose one by typing one of the numbers below:
  1<EMAIL_ADDRESS>which resolved to "2.2.4"
  2<EMAIL_ADDRESS>which resolved to "3.2.1"

https://github.com/geniuscarrier/webpack-boilerplate/issues/7

What do you mean by breaking versions? They break scrollTo or the fact that the major version changed?

The second one, that it will install potentially breaking versions.

Would moving it to a peerDependency solve the issue for you?

Not really, the best solution is just to stick to minor version in package.json, like: "^2.2.2"

That would lock the plugin to only one major version at a time, which is incorrect and undesired

Well, I think it is correct, because if some breaking change in major version is incompatible with your code, then you break all of you users code. But it's ofc up to you


render component even when google maps is not loaded yet

I like your implementation, though I've encountered one problem over and over again.
Currently GoogleMapsLoader is deciding what to do when googleMaps is not loaded yet.
This might be just fine in many cases, but often it'd be much more practical to have the < TargetComponent /> decide what to do.
e.g.
const TargetComponent = ({ googleMaps }) => {
  if(! googleMaps) {
    return <Spinner /> // you could still can just return `null`
  } else {
    /* code */
  }
}

For that reason I removed the logic that returns null instead of rendering the TargetComponent. Now it would just render the TargetComponent no matter what, and let it decide how to handle the loading period.

Thank you for this. It is a very good idea

@MoeSattler I just published the v2.0.3

@xuopled thanks!

Can the documentation, particularly the readme example, be updated for this change?
We can't just assume it is ready to go in componentDidMount anymore. We need a way to know when the googleMaps prop stops being null and can be used using componentWillReceiveProps or something.
This change should not have been released as a semver patch; as it is a breaking change it should have been a v3.0.0 release.

These methods seem to work ok in my component class:
initAutocompleteService() {
  this.autocompleteService = new this.props.googleMaps.places
    .AutocompleteService()
}

componentDidMount() {
  if (this.props.googleMaps) this.initAutocompleteService()
}

componentDidUpdate(prevProps) {
  if (this.props.googleMaps && !prevProps.googleMaps)
    this.initAutocompleteService()
}

@jaydenseric

Can the documentation, particularly the readme example, be updated for this change?
why don't you do a PR for the readme? The change is pretty easy.

We need a way to know when the googleMaps prop stops being null
Why don't you just check for null? No need for lifecycle hooks.
If you want the previous behaviour, just do this:

const TargetComponent = ({ googleMaps }) => {
  if(! googleMaps) {
    return null
  } else {
    /* code */
  }
}

@xuopled
It should have indeed been v3.0.0 due to breaking change.

@jaydenseric you should do a PR for the README!

We can't just assume it is ready to go in componentDidMount anymore. We need a way to know when the googleMaps prop stops being null and can be used using componentWillReceiveProps or something.

@jaydenseric you could just use a HoC to take care of that for your

@MoeSattler Don't get me wrong, I think you PR is a good idea. It just needs to be documented how to use it efficiently; as the creator I assumed you would have a recommendation because it's not immediately obvious.
It is a bad idea to create a new instance of something within the render method, it should exist on the mounted component instance to be shared between renders. So to set that up when the component mounts, what do you do if googleMaps is null? You can't do anything but wait for it to be available when the props change. You have to use the lifecycle hooks.

Sorry @MoeSattler and @jaydenseric for releasing 2.0.3 instead of 3.0.0. So what should I do now? Revert the commit in 2.0.4 and publish it again in 3.0.0. Thank you for your help
,

Thats ok, I think the right thing to do here is use the npm deprecate command on the incorrect version, and explain in the message there was a breaking change that is now available in 3.0.0.

@jaydenseric
Implicitly returning null was very limiting and too opinionated, since it prohibited common use cases, like rendering simple spinners.
I think the underlying problem is more in terms of React itself instead of this lib in particular.
Composability is a core principles of React. Use that! If you have a use case, where you want your component not to be rendered before maps arrive, just compose yourself a new component.
const waitForMaps = Comp => props => props.googleMaps ? <Comp {...props} /> : null
const NewComponent = waitForMaps(MapsReliantComponent)

// or

const injectMaps = pipe(
  googleMapsLoader,
  waitForMaps,
)
const NewComponent = injectMaps(MapsReliantComponent)

@MoeSattler Those are not the concepts I was getting at.
Take for instance an address input with Places powered suggestions that are displayed on input:
import { Component } from 'react'
import googleMapsLoader from 'react-google-maps-loader'
import TextualInput from './textual-input'
import GooglePlacesSuggestions from './google-places-suggestions'
import { googleMapsAPIKey } from '../config'

class AddressInput extends Component {
  state = {
    suggestions: []
  }

  initAutocompleteService() {
    this.autocompleteService = new this.props.googleMaps.places
      .AutocompleteService()
  }

  componentDidMount() {
    if (this.props.googleMaps) this.initAutocompleteService()
  }

  componentDidUpdate(prevProps) {
    if (this.props.googleMaps && !prevProps.googleMaps)
      this.initAutocompleteService()
  }

  handleChange = ({ target: { value } }) => {
    if (!value) this.setSuggestions()
    else
      this.autocompleteService.getPlacePredictions(
        { input: value },
        this.setSuggestions
      )
    if (this.props.onChange) this.props.onChange(value)
  }

  setSuggestions = (suggestions = []) => {
    this.setState({ suggestions })
  }

  handleSuggestionPick = value => {
    this.setState({ suggestions: [] })
    if (this.props.onChange) this.props.onChange(value)
  }

  render() {
    const {
      googleMaps,
      // eslint-disable-next-line no-unused-vars
      onChange,
      ...props
    } = this.props
    return (
      <div>
        <TextualInput
          type="text"
          disabled={!googleMaps}
          {...props}
          onChange={this.handleChange}
        />
        {!!this.state.suggestions.length &&
          <GooglePlacesSuggestions
            suggestions={this.state.suggestions}
            onPick={this.handleSuggestionPick}
          />}
      </div>
    )
  }
}

export default googleMapsLoader(AddressInput, {
  key: googleMapsAPIKey,
  libraries: ['places']
})

You do not want to call new this.props.googleMaps.places.AutocompleteService() on every keystroke! Better to initialize the service once when the component is mounted.

Ok guys, I deprecated the 2.0.3 and release the 3.0.0.
I updated the README but if you think it should be an other way, PR are welcome ! Thank you

@jaydenseric
I understand. What I don't understand is, why you are not using simple composition or HoC to create a semantical equivalent of react-google-maps-loader pre 2.0.3.
Or any other behavior your use cases might require.

@xuopled thanks !

@jaydenseric
Maybe this makes it clearer what I mean. It's copy pasting my earlier example into your code.
'''
export default googleMapsLoader(props => props.googleMaps ? <AddressInput {...props} /> : null, {
key: googleMapsAPIKey,
libraries: ['places']
})
'''


Not working on 64bit

I installed 64bit notepad++ on windows 10 it gives following error.

same problem

Me too.

Then...

When opening Notepad++ I get the above message.

64 bit version of plugin for Notepad++ available?

https://notepad-plus-plus.org/community/topic/17375/emmet-installation

Duplicate of #19


Added OpenCHAMI middleware and updated go deps

Addresses #46 by adding the OpenCHAMI middleware.

Looks like the unit tests are failing:

Error: boot-script-service/routers.go:87:61: cannot use tokenAuth (variable of type *"github.com/OpenCHAMI/jwtauth/v5".JWTAuth) as *"github.com/go-chi/jwtauth/v5".JWTAuth value in argument to openchami_authenticator.AuthenticatorWithRequiredClaims

Looks like the unit tests are failing:

Error: boot-script-service/routers.go:87:61: cannot use tokenAuth (variable of type *"github.com/OpenCHAMI/jwtauth/v5".JWTAuth) as *"github.com/go-chi/jwtauth/v5".JWTAuth value in argument to openchami_authenticator.AuthenticatorWithRequiredClaims

Yep, I'm going to change the import to use the OpenCHAMI jwtauth.

 Did that happen from updating the packages? Maybe we'll just have to rollback the version for etcd if we can.

Yeah I think that was it.
Rolling back etcd fixes it.
diff --git a/go.mod b/go.mod
index efc6510..24a6993 100644
--- a/go.mod
+++ b/go.mod
@@ -37,7 +37,7 @@ require (
        github.com/Cray-HPE/hms-securestorage v1.13.0 // indirect
        github.com/aws/aws-sdk-go v1.55.5 // indirect
        github.com/cenkalti/backoff/v3 v3.2.2 // indirect
-       github.com/coreos/etcd v3.3.27+incompatible // indirect
+       github.com/coreos/etcd v3.3.13+incompatible // indirect
        github.com/coreos/go-semver v0.3.1 // indirect
        github.com/coreos/go-systemd v0.0.0-20191104093116-d3cd4ed1dbcf // indirect
        github.com/coreos/pkg v0.0.0-20240122114842-bbd7aa9bf6fb // indirect

(Make sure to go mod tidy after modifying go.mod)

BSS builds with the downgraded EtcD version.


[Question] Why enable Voice Over in vanilla packer templates?

I see in your vanilla packer templates, after you get past the initial setup screens and can see the desktop, you enable Voice Over
https://github.com/cirruslabs/macos-image-templates/blob/525f51a54e53036cd6bfdb0b28bfa35a49dd303c/templates/vanilla-sequoia.pkr.hcl#L66-L67
and then disable it again a few lines later
https://github.com/cirruslabs/macos-image-templates/blob/525f51a54e53036cd6bfdb0b28bfa35a49dd303c/templates/vanilla-sequoia.pkr.hcl#L76-L77
Just out of curiosity, what's the need for those steps?

Never mind, I get it now! It's so that you can navigate the UI of the System Preferences window via the keyboard. Clever!

Hey @fkorotkov I reopened this and deleted my previous comment because I actually still don't know why Voice Over is enabled. I thought initially it was to navigate System Preferences with the keyboard, but that's not actually necessary it seems. Any insight you can provide?

It's to emable Remote Sharing so VM is SSHable:
https://github.com/cirruslabs/macos-image-templates/blob/525f51a54e53036cd6bfdb0b28bfa35a49dd303c/templates/vanilla-sequoia.pkr.hcl#L66-L77
But the voice over it getting disabled in the end so you should not see it.


Run pre-merge CI actions in docker

This gives a significant speedup, nice! Only 3 minutes to run the tests now, vs 8 previously.


remove windows-sys unused feature

Builds on x86_64-pc-windows-msvc, so should be ok.

Nice catch! Yeah. Seems to build, so should probably be fine.


Document hash table sizes

Dan wrote this up, I just need to make sure it goes in the docs.

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 22 Jul 2011 at 4:43

Check out "Overview of CUDPP hash tables".

Original comment by<EMAIL_ADDRESS>on 23 Jul 2011 at 4:58

Changed state: Fixed


Add back breaking dependencies if in fact required

In this PR I removed some breaking dependencies. We need to figure out if they are in fact needed, and if they are, figure out how to get the package to build with them. In other words, the following command should work:
pip install -e .

So far, I see no reason why those dependencies are required. I am seeing this warning, which is annoying, and maybe why you had added those dependencies in the first place, @kavedder:
/Users/robertvoyer/Envs/arcs/lib/python2.7/site-packages/requests/packages/urllib3/util/ssl_.py:90: InsecurePlatformWarning: A true SSLContext object is not available. This prevents urllib3 from configuring SSL appropriately and may cause certain SSL connections to fail. For more information, see https://urllib3.readthedocs.org/en/latest/security.html#insecureplatformwarning.
  InsecurePlatformWarning


Not working scroll with Fullpage.js

Hi!
I have a website with https://github.com/alvarotrigo/fullPage.js
This script has own fullscreen scrolling by anchors. When I type URL with anchor in browser, it's ok, scroll is working. But, if I paste the same URL in href attribute of link Mmenu, scroll is not working, I click to link, panel closing without scrolling.
Please, help!

Have you seen the one-page example?
http://mmenu.frebsite.nl/examples.html

Thank you for your reply!
It was strange bug:
I used method of Fullpage.js "$.fn.fullpage.moveTo('section2')" with text anchor, It did not work.
Then I tryed method "$.fn.fullpage.moveTo(2)" with index anchor, and it's working!
P.S. without mmenu both methods is working.


Unable to retrieve values from stored session variable

Hi,
So, I am trying to implement session variables in Amazon Alexa which I am developing using Flask-Ask.
I am storing the value in a session variable savedColor , like -
session.attributes['savedColor'] = color //color='Blue'
Then, I am trying to fetch it as -
color = session.attributes['savedColor']
Error given by my flask-ask log as -
line 36, in whats_my_color
 color = session.attributes['savedColor']
KeyError: 'savedColor'
I have also tried alternatives like session['attributes']['savedColor'] = color
and fetching it as -> color = session['attributes']['savedColor']
And, also as - session.attributes.get('savedColor)
Each of them is failing at the same place, as mentioned above.

Your format seems alright; I have the below in my code:
session.attributes['day'] = surveyquestionday
sessionquestionday=session.attributes['day']

But I also have an import session at the top section:
from flask_ask import Ask, statement, question, session, request, version


Add scapy_unroot

This adds scapy_unroot to the list of tools. scapy_unroot allows to run scapy without root permissions, on systems where users might not have them and setting of netcaps is not advisable. It is achieved by a daemon that runs with root permissions, that dispatches sockets via a UNIX domain socket API to non-root clients, which use special SuperSockets to connect to it.

The tool looks cool. I will give it a try!