Item list width issue

Hi @AhmedLSayed9 at first thank you so much for making such helpful widget. Currently I am facing one issue with this widget.

As you can see image options list width not same as dropdown widget. I want to make options list same width as dropdown widget.
I have tried itemWidth etc but not worked.
Would you like to help me how to fix that?
Thank you

Hi @iamfarmankhan
In your case the dropdown TextFormField takes max width possible like other TextFormFields. If you want the dropdown menu to take same width too, just don't pass itemWidth value (or pass null) and It'll take max width possible too like the dropdown TextFormField.
Please let me know if this helped you.

Hi @AhmedLSayed9 , thanks for your reply. If I pass null in itemWidth it does not effect any. If I pass double.Infinity dropdown option items not showing. Getting below exception -
======== Exception caught by rendering library =====================================================
The following assertion was thrown during performLayout():
BoxConstraints forces an infinite width.
Updated layout information required for RenderAnimatedOpacity#b8abd NEEDS-LAYOUT NEEDS-PAINT to calculate semantics.
'package:flutter/src/rendering/object.dart':
Failed assertion: line 2747 pos 12: '!_needsLayout'
Either the assertion indicates an error in the framework itself, or we should provide substantially more information in this error message to help you determine and fix the underlying cause.
In either case, please report this assertion by filing a bug on GitHub:
https://github.com/flutter/flutter/issues/new?template=2_bug.md

You aren't supposed to pass double.infinity just null or ignoring itemWidth and the widget will automatically calculate width from max width of parent constraints. But I don't know what's different with your code, Can you drop your code please?

Hi @AhmedLSayed9 , I have just given widget part here.
return Scaffold(
body: SingleChildScrollView(
child: Form(
child: Padding(
padding: const EdgeInsets.all(16.0),
child: Column(
mainAxisAlignment: MainAxisAlignment.start,
crossAxisAlignment: CrossAxisAlignment.start,
children: [
const SizedBox(
height: 35,
),
InkWell(
child: const Icon(
Icons.arrow_back,
color: Colors.black,
size: 36,
),
onTap: (){
Navigator.pop(context,true);
},
),
Center(
child: Text(
'info_about_dog'.tr,
style: const TextStyle(
fontSize: 22,
color: Colors.black,
fontWeight: FontWeight.w700),
textAlign: TextAlign.center,
),
),
const SizedBox(
height: 30,
),
PrimaryTextField(
titleText: 'whats_it_called'.tr,
textEditingController: dogNameEditingTextController,
validator: _notEmptyValidator,
),
Text(
'date_of_birth'.tr,
style: kTextFieldTitle,
),
const SizedBox(
height: kMarginInTitleAndField,
),
Row(
children: [
SizedBox(
width: 60,
height: 40,
child: InkWell(
child: TextView(
title: date,
),
onTap: ()
{
showDtePicker(context);
},
),
),
const SizedBox(
width: 20,
),
SizedBox(
width: 60,
height: 40,
child: InkWell(
child: TextView(
title: month,
),
onTap: ()
{
showDtePicker(context);
},
),
),
const SizedBox(
width: 20,
),
SizedBox(
width: 60,
height: 40,
child: InkWell(
child: TextView(
title: year,
),
onTap: ()
{
showDtePicker(context);
},
),
),
],
),
const SizedBox(
height: kMarginFormField,
),
Text(
'male_or_female'.tr,
style: kTextFieldTitle,
),
const SizedBox(
height: kMarginInTitleAndField,
),
Row(
children: [
SizedBox(
height: 50,
child: OutlinedButton(
onPressed: () {
setState(() {
isGenderSelected = true;
isMaleSelected = true;
});
},
style: isGenderSelected
? (!isMaleSelected
? getBorderedButtonStyle()
: getFilledButtonStyle())
: getBorderedButtonStyle(),
child: Text(
'male'.tr,
style: TextStyle(
color: isGenderSelected
? (!isMaleSelected
? Colors.black
: Colors.white)
: Colors.black,
fontWeight: FontWeight.w400,
fontSize: 20),
),
),
),
const SizedBox(
width: 20,
),
SizedBox(
height: 50,
child: OutlinedButton(
onPressed: () {
setState(() {
isGenderSelected = true;
isMaleSelected = false;
});
},
style: isGenderSelected
? (isMaleSelected
? getBorderedButtonStyle()
: getFilledButtonStyle())
: getBorderedButtonStyle(),
child: Text(
'female'.tr,
style: TextStyle(
color: isGenderSelected
? (isMaleSelected
? Colors.black
: Colors.white)
: Colors.black,
fontWeight: FontWeight.w400,
fontSize: 20),
),
),
),
],
),
const SizedBox(
height: kMarginFormField,
),
Text(
'are_u_neutered'.tr,
style: kTextFieldTitle,
),
const SizedBox(
height: kMarginInTitleAndField,
),
Row(
children: [
SizedBox(
height: 50,
child: OutlinedButton(
onPressed: () {
setState(() {
isNeuteredSelected = true;
isYesSelected = true;
});
},
style: isNeuteredSelected
? (!isYesSelected
? getBorderedButtonStyle()
: getFilledButtonStyle())
: getBorderedButtonStyle(),
child: Text(
'yes'.tr,
style: TextStyle(
color: isNeuteredSelected
? (!isYesSelected
? Colors.black
: Colors.white)
: Colors.black,
fontWeight: FontWeight.w400,
fontSize: 20),
),
),
),
const SizedBox(
width: 20,
),
SizedBox(
height: 50,
child: OutlinedButton(
onPressed: () {
setState(() {
isNeuteredSelected = true;
isYesSelected = false;
});
},
style: isNeuteredSelected
? (isYesSelected
? getBorderedButtonStyle()
: getFilledButtonStyle())
: getBorderedButtonStyle(),
child: Text(
'no'.tr,
style: TextStyle(
color: isNeuteredSelected
? (isYesSelected
? Colors.black
: Colors.white)
: Colors.black,
fontWeight: FontWeight.w400,
fontSize: 20),
),
),
),
],
),
const SizedBox(
height: 16,
),
Text(
'whats_ur_size'.tr,
style: kTextFieldTitle,
),
const SizedBox(
height: kMarginInTitleAndField,
),
CustomDropdownButton2(
hint: sizeHintText,
dropdownItems: [
'Pequeño'.tr,
'medium'.tr,
'great'.tr,
],
value: sizeValue,
onChanged:  (newValue) {
setState(() {
sizeValue = newValue!;
print("Size Value $sizeValue");
});
},
icon:  Image.asset('images/down_arrow.png'),
buttonWidth: double.infinity,
),
const SizedBox(
height: kMarginFormField,
),
Text(
'how_does_it_behave_other_dogs'.tr,
style: kTextFieldTitle,
),
const SizedBox(
height: kMarginInTitleAndField,
),
CustomDropdownButton2(
hint: behaviourHintText,
dropdownItems: [
'kind'.tr,
'playful'.tr,
'aggresive'.tr,
],
value: behaviourValue,
itemWidth: double.infinity,
onChanged:  (newValue) {
setState(() {
behaviourValue = newValue!;
});
              },
              icon:  Image.asset('images/down_arrow.png'),
              buttonWidth: double.infinity,
            ),
            const SizedBox(
              height: 30,
            ),
            PrimaryGreenButton(buttonClickHandler: (){
              Navigator.of(context).push(MaterialPageRoute(
                  builder: (context) => const CapturePhotoScreen()));
            }, title: 'register_dog'.tr),
          ],
        ),
      ),
    ),
  ),
);

Hi @iamfarmankhan,
You're using the Custom widget which I've made with default values for my preferences. You can edit the marked line of code as here it'll give DropdownButton2 default itemWidth value of 140 if itemWidth is null. and we want to pass null here, so you should remove ?? 140 so that It'll send the null value instead of the default 140 value.
How to edit the Custom widget to your preferences:

Copy the custom widget with inside the package to your lib folder or wherever you want.
Change the class name to whatever you want to avoid duplication between your custom widget and package's custom widget.
Edit and use it as you want.

Also, There's DropdownButtonFormField2, you can try using it with Form as you can add validation, etc.. as shown in the examples.
Good luck!

Please, let me know if your problem has been fixed.

Hi @AhmedLSayed9 , I copied the code for CustomDropDown in my project and removed itemWidth and problem is solved. Thanks for your plugin @AhmedLSayed9 .


[SD Correction] Reviewing Audit Logs

Provide the location of the issue
Section <IP_ADDRESS>, Test 5
What is the correction request for the cPP? Please describe.
This test requires viewing audit logs on the TOE; however, there are no requirements that the TOE provide an interface to review local audit records.
Describe the solution you'd like
Remove this test.

Are there any concerns or objections to removing Section <IP_ADDRESS> FAU_STG_EXT.1/LocSpace Test 5? If there aren't any objections, this is a simple update.

Recommendation...
As recommended by Kenji in the issue description, remove FAU_STG_EXT.1/LocSpace Test 5.

Objection. If the TOE implements local audit, there must be an interface to access (review, manually upload, etc.,) the local audit trail. Otherwise what is the point of having it?

(strangely) local review of audit logs has never been required, so this seems like a feature request. At one point a TD confirmed that there just needed to be enough functionality to verify log rotation (which many times seeing log file status (e.g., filenames, sizes, last updated)).

I agree with @kenji-lightship. We should remove Test 5.

I disagree, we should not remove Test 5.

@KSinitski, instead of removing Test 5 would it be acceptable to modify the second sentence of Test 5?
Change from:  "During interruption, the evaluator shall perform a TSF-mediated action and verify the event is recorded in the audit trail."
Change to: During interruption, the evaluator shall perform a TSF-mediated action and, if the TOE provides a mechanism to view locally stored audit records, verify the event is recorded in the local audit trail.

The SFR does not specify local audit storage in sufficient detail and potentially makes it very misleading. In some cases (e.g., buffer) your proposal is reasonable, in other cases (e.g., log file) it is not. I recommend creating a new FAU_STG_EXT.1/LocSpace element that clarifies how the TOE implements this SF and tying Test 5 to that.
For example:
FAU_STG_EXT.1.x/LocSpace The TOE provided the following mechanisms to directly access locally stored audit records [selection: none, manual export, ability to view locally].
Test 5 [Conditional] In case manual export or ability to view locally is selected, during interruption the evaluator shall perform a TSF-mediated action and verify the event is recorded in the audit trail."

Reopening pending resolution to comments.

@KSinitski, the idea of allowing a selection of [selection: none, manual export] in a new FAU_STG_EXT.1.x/LocSpace sounds interesting, but I think that can be covered in TSS detail.  If we include a selection for "ability to view locally" then it seems like that should be tied to a selection-based SFR, such as FAU_SAR.1.   Perhaps we can do that in the next PP update after 3.0?


Add Google Java Format on push

This automatically formatted my local repo, so I think it must do something right.

(I apologize for this changing literally every source file in the repo.)

You might want to exclude the vendored code in /lib from the reformatting action.
Hopefully most of it can be replaced with jars to avoid that issue.

Done. Unfortunately the vendored code isn't in jars because there are no jars -- a lot of it is code that someone found on forums somewhere. :(

I think files should be specified separately (assuming it's not working as-is on this branch, while it was before):
- uses: axel-op/googlejavaformat-action@v3
  with:
    args: "--replace"
    files:
      - "src/**/*.java"
      - "test/**/*.java"

Not certain that'll work looking at the code, it might have to be src/**/*.java\ntest/**/*.java, but I hope that passing it as an array works.

Well, I can try that too. Thanks, I'm basically clueless as far as what I'm doing here.

It looks to be single string, but a limited number of constructs are supported -- specifically, those in the @actions/glob library.
Try src/**/*.java to confirm we have anything right at all.
If we do, then [st]*/**/*.java is my terrible attempt at a supported glob that matches src, test and nothing else.

Is this then meant to make a commit that formats your code? I think that's a bad idea
I was looking at using spotless for gradle to apply java formatting and it worked really well. Maybe we can do that instead?
Also Google java formatting is different to our current styling. I'd be absolutely willing to move to that styling that though

Yeah it's committing from GitHub. I'd really like to not do it that way

If you prefer, I have a separate branch that uses spotless to format code. I'll close this PR out.
I don't feel strongly about it either way.

I appreciate it. It's just a real pain from expertise to get commits added to your PRs all the time. It clutters the commit log and causes extra merge commits when you forget to pull before your next change etc.
We could have GitHub actions check that the changed files would not change if styled though and add a warning to the PR if so.

Yeah, I agree. I didn't realize that's how it would work until I started tinkering.
I also agree that this could just be a check -- there's a --set-exit-if-changed flag, and also a skipCommit parameter that looks like it would do what we want. But I think applying formatting is orthogonal to checking that the formatting is correct.

You can add a commit hook that auto formats when committing to the local branch. I'd need to look this up because I don't remember the details but that was set up at a previous job I was at to enforce code style.

https://github.com/a1exsh/google-java-format-git-pre-commit-hook maybe

I do have a pre-commit hook in place for me personally, but I'd prefer to enforce something for everyone.


Add Contact Us in navbar and created contact form for same.

Related Issue

Add Contact Us in navbar and created contact form for same.

Closes: #326
Describe the changes you've made
I have used bootstrap and html to make the changes, add contact us to  navbar and also created file contact.html in which contact form will be there by clicking contact us on navbar.
Describe if there is any unusual behaviour of your code(Write NA if there isn't)
NA
Checklist:

[x] My code follows the style guidelines of this project.
[x] I have performed a self-review of my own code.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] I have made corresponding changes to the documentation.
[x] New and existing unit tests pass locally with my changes.

Screenshots
Put any screenshot(s) of the project here.

I did it. Okay I am all over doing it again.


add miktex 22.3

[ X ] Have you signed the Contributor License Agreement?
[ X ] Have you checked that there aren't other open pull requests for the same manifest update/change?
[ X ] Have you validated your manifest locally with winget validate --manifest <path>?
[ X ] Have you tested your manifest locally with winget install --manifest <path>?
[ X ] Does your manifest conform to the 1.0 schema?

Add new Upstream Version
Microsoft Reviewers: Open in CodeFlow

Thanks, I have forget it to commit.

@wingetbot waivers Add Policy-Test-1.2

It seems this package installer is no longer available on miktex.org. Please update.

I don't have windows here right now, can you please validate #69100 and install it locally?

The validate command failed. It seems manifest version value 1.2.0 is not supported.

winget validate --manifest .
Manifest validation failed.
Manifest Error: The multi file manifest has inconsistent field values. Field: ManifestVersion Value: 1.2.0 File: ChristianSchenk.MiKTeX.locale.en-US.yaml
Manifest Error: The multi file manifest has inconsistent field values. Field: ManifestVersion Value: 1.2.0 File: ChristianSchenk.MiKTeX.yaml

I changed the manifest value in all three files to 1.1.0 and then the validation works.

winget validate --manifest .
Manifest validation succeeded.

Thank you, I have pushed new changes for #69100 and waiting now for review and merge. I have found new PR with the same solution: #69103

Thank you! I can confirm that the installation completed successfully. I'm glad that this issue is resolved.


Added A New Algorithms folder and Code for Number Line Jumps

Question: Number Line Jumps
Problem Link :https://www.hackerrank.com/challenges/kangaroo/problem

Only do the tasks assigned to you.


v3.1 dev to main

[3.1] (2022-09-10)
This is major relase adding Resrobot functionallity BUT with some known upgrade issues due to some bad decisions earlier in development. During some cases the automatic upgrade of sensors might fail and to restore a broken sensor it must be fully removed and then recreated manually. Sorry.
BREAKING CHANGES

During some cases the automatic upgrade of sensors might fail, to restore a broken sensor it must be fully removed and then recreated manually. Sorry.
Service find_location renamed to sl_find_location
Service find_trip_id renamed to sl_find_trip_id
Service find_trip_pos renamed to sl_find_trip_pos

ADDED

API Communication for Resrobot 2.1
Departure sensor using Resrobot API
Arrival sensor using Resrobot API
Route sensor using Resrobot API
Lookup services for resrobot locations

Hello @DSorlov! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file custom_components/hasl3/__init__.py:

Line 60:80: E501 line too long (101 > 79 characters)
Line 64:80: E501 line too long (136 > 79 characters)
Line 74:80: E501 line too long (98 > 79 characters)
Line 78:80: E501 line too long (135 > 79 characters)
Line 87:80: E501 line too long (99 > 79 characters)
Line 93:80: E501 line too long (110 > 79 characters)
Line 97:80: E501 line too long (142 > 79 characters)
Line 106:80: E501 line too long (99 > 79 characters)
Line 112:80: E501 line too long (110 > 79 characters)
Line 116:80: E501 line too long (142 > 79 characters)
Line 117:24: W291 trailing whitespace
Line 126:80: E501 line too long (122 > 79 characters)
Line 132:80: E501 line too long (109 > 79 characters)
Line 136:80: E501 line too long (141 > 79 characters)
Line 148:80: E501 line too long (117 > 79 characters)
Line 154:80: E501 line too long (110 > 79 characters)
Line 158:80: E501 line too long (142 > 79 characters)
Line 177:80: E501 line too long (81 > 79 characters)
Line 181:80: E501 line too long (81 > 79 characters)
Line 185:80: E501 line too long (81 > 79 characters)
Line 189:80: E501 line too long (80 > 79 characters)
Line 212:80: E501 line too long (82 > 79 characters)
Line 213:80: E501 line too long (82 > 79 characters)
Line 214:80: E501 line too long (82 > 79 characters)
Line 215:80: E501 line too long (80 > 79 characters)
Line 239:80: E501 line too long (99 > 79 characters)
Line 241:80: E501 line too long (90 > 79 characters)
Line 247:80: E501 line too long (89 > 79 characters)
Line 250:80: E501 line too long (91 > 79 characters)
Line 253:80: E501 line too long (104 > 79 characters)
Line 256:80: E501 line too long (90 > 79 characters)
Line 259:80: E501 line too long (81 > 79 characters)
Line 308:73: W291 trailing whitespace

In the file custom_components/hasl3/config_flow.py:

Line 102:50: W291 trailing whitespace
Line 104:50: W291 trailing whitespace
Line 106:52: W291 trailing whitespace
Line 126:60: W291 trailing whitespace
Line 128:60: W291 trailing whitespace
Line 130:62: W291 trailing whitespace
Line 201:72: W291 trailing whitespace
Line 203:72: W291 trailing whitespace
Line 205:74: W291 trailing whitespace

In the file custom_components/hasl3/config_schema.py:

Line 156:80: E501 line too long (265 > 79 characters)
Line 160:80: E501 line too long (120 > 79 characters)
Line 161:80: E501 line too long (87 > 79 characters)
Line 162:80: E501 line too long (81 > 79 characters)
Line 164:80: E501 line too long (103 > 79 characters)
Line 168:1: E302 expected 2 blank lines, found 1
Line 171:80: E501 line too long (265 > 79 characters)
Line 175:80: E501 line too long (120 > 79 characters)
Line 176:80: E501 line too long (87 > 79 characters)
Line 177:80: E501 line too long (81 > 79 characters)
Line 180:6: W291 trailing whitespace
Line 182:1: E302 expected 2 blank lines, found 1
Line 185:80: E501 line too long (140 > 79 characters)
Line 189:80: E501 line too long (86 > 79 characters)
Line 190:80: E501 line too long (87 > 79 characters)
Line 192:10: W292 no newline at end of file

In the file custom_components/hasl3/const.py:

Line 81:80: E501 line too long (157 > 79 characters)
Line 83:80: E501 line too long (139 > 79 characters)
Line 84:80: E501 line too long (110 > 79 characters)

In the file custom_components/hasl3/haslworker/__init__.py:

Line 526:80: E501 line too long (88 > 79 characters)
Line 537:15: W291 trailing whitespace
Line 554:80: E501 line too long (88 > 79 characters)
Line 599:15: W291 trailing whitespace
Line 636:80: E501 line too long (80 > 79 characters)
Line 638:80: E501 line too long (132 > 79 characters)
Line 639:80: E501 line too long (117 > 79 characters)
Line 641:80: E501 line too long (128 > 79 characters)
Line 642:80: E501 line too long (113 > 79 characters)
Line 650:80: E501 line too long (116 > 79 characters)
Line 656:80: E501 line too long (103 > 79 characters)
Line 656:86: E231 missing whitespace after ','
Line 657:27: W291 trailing whitespace
Line 660:49: E127 continuation line over-indented for visual indent
Line 662:80: E501 line too long (81 > 79 characters)
Line 664:80: E501 line too long (82 > 79 characters)
Line 669:80: E501 line too long (85 > 79 characters)
Line 672:17: E303 too many blank lines (2)
Line 677:1: W293 blank line contains whitespace
Line 679:15: W291 trailing whitespace
Line 718:80: E501 line too long (80 > 79 characters)
Line 720:80: E501 line too long (132 > 79 characters)
Line 721:80: E501 line too long (117 > 79 characters)
Line 723:80: E501 line too long (128 > 79 characters)
Line 724:80: E501 line too long (113 > 79 characters)
Line 731:80: E501 line too long (114 > 79 characters)
Line 737:80: E501 line too long (103 > 79 characters)
Line 737:86: E231 missing whitespace after ','
Line 738:27: W291 trailing whitespace
Line 741:49: E127 continuation line over-indented for visual indent
Line 743:80: E501 line too long (81 > 79 characters)
Line 745:80: E501 line too long (82 > 79 characters)
Line 750:80: E501 line too long (85 > 79 characters)
Line 753:17: E303 too many blank lines (2)
Line 758:1: W293 blank line contains whitespace
Line 760:15: W291 trailing whitespace
Line 769:80: E501 line too long (81 > 79 characters)
Line 783:21: E265 block comment should start with '# '
Line 788:1: W293 blank line contains whitespace
Line 795:80: E501 line too long (107 > 79 characters)
Line 796:80: E501 line too long (107 > 79 characters)
Line 801:80: E501 line too long (95 > 79 characters)
Line 802:1: W293 blank line contains whitespace
Line 806:80: E501 line too long (85 > 79 characters)
Line 808:1: W293 blank line contains whitespace
Line 810:1: W293 blank line contains whitespace
Line 814:80: E501 line too long (91 > 79 characters)
Line 818:80: E501 line too long (117 > 79 characters)
Line 819:80: E501 line too long (125 > 79 characters)
Line 820:80: E501 line too long (117 > 79 characters)
Line 821:21: E265 block comment should start with '# '
Line 824:80: E501 line too long (82 > 79 characters)
Line 825:80: E501 line too long (113 > 79 characters)
Line 829:80: E501 line too long (89 > 79 characters)
Line 830:80: E501 line too long (87 > 79 characters)
Line 835:80: E501 line too long (81 > 79 characters)
Line 836:80: E501 line too long (82 > 79 characters)
Line 837:80: E501 line too long (92 > 79 characters)
Line 838:80: E501 line too long (90 > 79 characters)
Line 839:80: E501 line too long (82 > 79 characters)
Line 840:80: E501 line too long (82 > 79 characters)
Line 844:80: E501 line too long (81 > 79 characters)

In the file custom_components/hasl3/rrapi/const.py:

Line 7:80: E501 line too long (111 > 79 characters)

In the file custom_components/hasl3/rrapi/rrapi.py:

Line 23:1: E302 expected 2 blank lines, found 1
Line 40:80: E501 line too long (82 > 79 characters)
Line 48:80: E501 line too long (84 > 79 characters)
Line 53:80: E501 line too long (88 > 79 characters)
Line 58:80: E501 line too long (115 > 79 characters)
Line 58:89: E231 missing whitespace after ','
Line 73:80: E501 line too long (113 > 79 characters)
Line 73:95: E231 missing whitespace after ','
Line 77:11: E111 indentation is not a multiple of four
Line 77:80: E501 line too long (122 > 79 characters)
Line 78:11: E111 indentation is not a multiple of four
Line 79:11: E111 indentation is not a multiple of four
Line 80:11: E111 indentation is not a multiple of four
Line 81:11: E111 indentation is not a multiple of four
Line 82:11: E111 indentation is not a multiple of four
Line 83:11: E111 indentation is not a multiple of four
Line 84:11: E111 indentation is not a multiple of four
Line 85:11: E111 indentation is not a multiple of four
Line 85:31: W291 trailing whitespace
Line 86:1: W293 blank line contains whitespace
Line 92:1: E303 too many blank lines (4)
Line 99:80: E501 line too long (121 > 79 characters)
Line 110:80: E501 line too long (107 > 79 characters)
Line 110:89: E231 missing whitespace after ','
Line 112:1: E302 expected 2 blank lines, found 1
Line 120:80: E501 line too long (104 > 79 characters)
Line 120:87: E231 missing whitespace after ','
Line 121:1: W391 blank line at end of file

In the file custom_components/hasl3/sensor.py:

Line 175:80: E501 line too long (92 > 79 characters)
Line 176:80: E501 line too long (94 > 79 characters)
Line 181:80: E501 line too long (81 > 79 characters)
Line 187:80: E501 line too long (92 > 79 characters)
Line 188:80: E501 line too long (92 > 79 characters)
Line 193:80: E501 line too long (81 > 79 characters)
Line 195:5: E265 block comment should start with '# '
Line 199:80: E501 line too long (124 > 79 characters)
Line 200:80: E501 line too long (128 > 79 characters)
Line 204:5: E265 block comment should start with '# '
Line 205:80: E501 line too long (82 > 79 characters)
Line 351:1: E302 expected 2 blank lines, found 1
Line 373:80: E501 line too long (200 > 79 characters)
Line 377:21: E722 do not use bare 'except'
Line 378:80: E501 line too long (82 > 79 characters)
Line 389:80: E501 line too long (87 > 79 characters)
Line 425:5: E303 too many blank lines (2)
Line 440:80: E501 line too long (94 > 79 characters)
Line 452:80: E501 line too long (80 > 79 characters)
Line 457:80: E501 line too long (80 > 79 characters)
Line 460:80: E501 line too long (80 > 79 characters)
Line 461:80: E501 line too long (90 > 79 characters)
Line 462:80: E501 line too long (88 > 79 characters)
Line 463:80: E501 line too long (80 > 79 characters)
Line 464:80: E501 line too long (80 > 79 characters)
Line 466:80: E501 line too long (90 > 79 characters)
Line 469:9: E722 do not use bare 'except'
Line 471:80: E501 line too long (114 > 79 characters)
Line 693:1: E302 expected 2 blank lines, found 1
Line 709:80: E501 line too long (81 > 79 characters)
Line 717:80: E501 line too long (94 > 79 characters)
Line 722:24: E225 missing whitespace around operator
Line 724:39: E231 missing whitespace after ','
Line 734:80: E501 line too long (202 > 79 characters)
Line 738:21: E722 do not use bare 'except'
Line 739:80: E501 line too long (82 > 79 characters)
Line 751:80: E501 line too long (93 > 79 characters)
Line 872:80: E501 line too long (94 > 79 characters)
Line 887:9: E722 do not use bare 'except'
Line 889:80: E501 line too long (114 > 79 characters)
Line 891:19: W291 trailing whitespace
Line 893:1: E302 expected 2 blank lines, found 1
Line 917:80: E501 line too long (94 > 79 characters)
Line 922:24: E225 missing whitespace around operator
Line 924:39: E231 missing whitespace after ','
Line 934:80: E501 line too long (202 > 79 characters)
Line 938:21: E722 do not use bare 'except'
Line 939:80: E501 line too long (82 > 79 characters)
Line 950:80: E501 line too long (91 > 79 characters)
Line 994:9: E303 too many blank lines (2)
Line 1075:80: E501 line too long (94 > 79 characters)
Line 1089:9: E722 do not use bare 'except'
Line 1091:80: E501 line too long (114 > 79 characters)
Line 1093:19: W291 trailing whitespace

In the file custom_components/hasl3/slapi/slapi.py:

Line 164:80: E501 line too long (98 > 79 characters)
Line 164:82: E231 missing whitespace after ','
Line 177:68: E231 missing whitespace after ','
Line 177:80: E501 line too long (86 > 79 characters)


[BUG] defaults.yml in documentation

In documentation it's not stated that values from config.yml overwrite structure of defaults.yml.
So, if in defaults.yml some indicator is enabled, but in config.yml it's not defined (the expected behavior = disabled) value is taken from defaults.yml.
This, in turn leads to consequences - price_values for notification is calculated only if ohlcv for same period is explicitly enabled

Found another glitch between defaults.yml and config.yml:
defaults.yml
  obv:
    - enabled: true
      alert_enabled: true
      alert_frequency: once
      signal:
        - obv
      hot: 0
      cold: 0
      candle_period: 1d
  ohlcv:
    - enabled: true
      signal:
        - close
      candle_period: 1d
      period_count: 15

config.yml
obv - not defined
ohlcv:
  - enabled: true
    signal:
      - close
    candle_period: 5m
    period_count: 15

price_values for notification for obv is not available


Reduce image size to the required size for none-code contributors

Context
Right now we have VERY big images for some none-code contributors. This slows down the website a lot.
Project
File: documentation-website/public/assets/contributors/*.jpg

hey @Idrinth please check these ,made the required changes.


Define function types for prototype declaration

This is pull request implements what I have been describing in a recent WG meeting: providing and using type function definitions to declare symbols and pointers. This yields several benefits:

We stop code duplication. (It is less an issue nowadays since we generate the code, but still)
Now that prototypes can be deactivated, this allows users declaring prototypes at a finer granularity. For instance, we could use this to declare loader extension prototypes by using the type declaration from the header. For now, the definitions cannot be validated by a prototype from the headers since CL_NO_NON_ICD_DISPATCH_EXTENSION_PROTOTYPES is activated: see: https://github.com/KhronosGroup/OpenCL-ICD-Loader/blob/ee329edc23258a53f9b5799446f7417d853ee143/loader/icd_dispatch.c#L26-L32. Instead, we could now write:

static clGetICDLoaderInfoOCLICD_t clGetICDLoaderInfoOCLICD;
cl_int
clGetICDLoaderInfoOCLICD(
   cl_icdl_info param_name,
   size_t       param_value_size,
   void *       param_value,
   size_t *     param_value_size_ret)
{

I kept the original exports as some users could be parsing the headers for the original declarations (We do in the CLHPP testing for instance)
I did cut this PR in 3 main patches, should we decide to partially apply these changes:

Add prototypes to the extensions through the generator.
Add prototype to the core functions in a new generated file cl_function_types
Apply prototypes (manually) to cl_icd.h which benefits a lot from the change. Given how suffixes are not really used (and very inconsistently in cl_icd.h) we could also take it a step father and remove the pointer type definitions from this file and use the function types directly inside the dispatch structure. For now I decided to minimize the scope of the changes. I did remove duplicated prototype declarations in cl_icdl.h, and added a missing guard around cl_api_clCreateFromGLTexture which apparently only exist in OpenCL 1.2.

I tested using the ICD-Loader and CLHPP tests. No issues to report.

LGTM! I did some basic testing on Windows and everything seems to be in order.

Thank for taking a look. Should we discuss this next week at the teleconference before merging?


Added support for movable objects in mock::retrieve

https://sourceforge.net/p/turtle/tickets/67/

Changes Unknown when pulling f872785e26559148da0263fb4c1c8c8887f6f5e5 on support-retrieve-move-capture into ** on master**.


handleRedirectPromise throws an empty error for random users

Core Library
MSAL.js (@azure/msal-browser)
Core Library Version
3.6.0
Wrapper Library
Not Applicable
Wrapper Library Version
n/a
Public or Confidential Client?
Public
Description
For our Word add-in, we have a login component that handles the authentication with MSAL JS. It's a popup and it calls handleRedirectPromise.
For a set of our users, an error is thrown, but the error object is empty.
Our code is as follows:
 this.msalInstance.handleRedirectPromise().then(() => // do something with response).catch(err => {
// handle error
// this is randomly an empty object
});

We've upgraded to the latest MSAL package but issue persists.
While the add-in is using Angular, we don't use MSAL Angular, just MSAL JS.
Error Message
{}
Msal Logs
No response
MSAL Configuration
{
auth: {
      clientId: this.authConfiguration.clientId,
      authority: this.authConfiguration.authority,
      redirectUri: `${window.location.origin}/auth/login`,
      knownAuthorities: [B2C_AUTHORITY],
    },
    cache: {
      cacheLocation: 'localStorage', // needed to avoid "login required" error
      storeAuthStateInCookie: true, // recommended to avoid certain IE/Edge issues
    },
}

Relevant Code Snippets
See above

Reproduction Steps
See above
Expected Behavior
No empty errors occur.
Identity Provider
Azure B2C Custom Policy
Browsers Affected (Select all that apply)
Other
Regression
No response
Source
External (Customer)

Upon enabling trace logging, I can see this in the logs just before the issue happens:
<EMAIL_ADDRESS>: Info - Emitting event: msal:handleRedirectStart
<EMAIL_ADDRESS>: Verbose - handleRedirectPromise has been called for the first time, storing the promise
<EMAIL_ADDRESS>: Trace - BrowserCacheManager.getCachedNativeRequest called
<EMAIL_ADDRESS>: Trace - BrowserCacheManager.getTemporaryCache: No cache item found in local storage
<EMAIL_ADDRESS>: Trace - BrowserCacheManager.getCachedNativeRequest: No cached native request found
<EMAIL_ADDRESS>: Trace - handleRedirectPromise - acquiring token from web flow
<EMAIL_ADDRESS>: Trace -  BrowserCacheManager.getTemporaryCache: No cache item found in local  storage

It seems to mainly impact Mac users. For most users it has worked before, and it suddenly stopped working

I've found out why the error was an empty object. This is because an Error object by default doesn't serialize properly.
After serializing it better, I could see the unexpected error that causes this issue.
Please see below the value of 'message' of the error being thrown:
server_error: AADB2C90273: An invalid response was received : 'Error: server_error'\r\nCorrelation ID: e1c845a9-d0c2-4348-858e-81c004976bc8\r\nTimestamp: 2023-12-11 17:33:51Z\r\n","errorCode":"server_error","errorMessage":"AADB2C90273: An invalid response was received : 'Error: server_error'\r\nCorrelation ID: e1c845a9-d0c2-4348-858e-81c004976bc8\r\nTimestamp: 2023-12-11 17:33:51Z\r\n","subError":"","name":"ServerError","correlationId":"6f44051b-1803-4679-b2e6-e9eb8942859d"

There is zero information on this error code, the only thing I can find is other people with the same issue.

Update: Upon further investigation, discovered why the error is empty, because it's a standard Error object which doesn't serialize properly.
When serializing, I can see that this error is thrown:
window.crypto.randomUUID is not a function. (In 'window.crypto.randomUUID()', 'window.crypto.randomUUID' is undefined

This randomUUID is in the MSAL package.
Why is this happening?

Please confirm your browser supports randomUUID also note that crypto APIs are only supported in secure contexts.

@tnorling I've just added additional logging and it failed for one user on Safari 15.2; which I don't believe has randomUUID.
We're on Angular 16, which ships with browerslist for support including Safari 15. However, I now see that MSAL supports the last 2 browsers, which does not include Safari 15.2.
As a quick fix I've added a polyfill for randomUUID.
I'm not 100% sure this fixes it for everyone as we've had reports from people on Mac OS 12 as well which should have a newer browser version.

Sounds good. Closing this for now but if you see this issue in a supported browser let us know.


Value error during zipping of training batch

Hi,
I am trying to run train.py with my own dataset for a multi-class classification problem. I get a Value error during the generation of test batch. What can be the issue? It works fine on the provided +ve/-ve dataset

Ok. I found the issue.
In
def batch_iter(data, batch_size, num_epochs, shuffle=True):
sometimes randomly start_index and end_index were becoming the same.
so the overall
yield shuffled_data[start_index:end_index]
was returning an empty list


Conduct survey of Git contributions and build database for all campuses

@Vensberg I assigned this to you for now but feel free to reassign later once process is clearer.


[builder] Failed handshake: ssh: server has no host keys

Getting [error] Failed handshake: ssh: server has no host keys when trying to git push deis v1.12.2 an app.
Feb 01 05:06:00 ip-10-21-2-172.ec2.internal sh[23192]: [info] Re-trying template build. (Elapsed time: 12)
Feb 01 05:06:00 ip-10-21-2-172.ec2.internal sh[23192]: [debug] Recoverable error: exit status 1
Feb 01 05:06:00 ip-10-21-2-172.ec2.internal sh[23192]: [debug] Output: "2016-02-01T05:06:00Z ed4d09e2f375 confd[136]: ERROR template: builder:79:49: executing \"builder\" at <getv \"/deis/controll...>: error calling getv: key does not exist\n2016-02-01T05:06:00Z ed4d09e2f375 confd[136]: ERROR template: receiver:15:55: executing \"receiver\" at <getv \"/deis/controll...>: error calling getv: key does not exist\n"
Feb 01 05:06:03 ip-10-21-2-172.ec2.internal sh[23192]: [info] Re-trying template build. (Elapsed time: 15)
Feb 01 05:06:03 ip-10-21-2-172.ec2.internal sh[23192]: [info] Templates generated for <IP_ADDRESS>:4001 on run 5
Feb 01 05:06:03 ip-10-21-2-172.ec2.internal sh[23192]: [info] Watching confd.
Feb 01 05:06:03 ip-10-21-2-172.ec2.internal sh[23192]: [info] Downloading Docker images. This may take a long time. https://xkcd.com/303/
Feb 01 05:06:03 ip-10-21-2-172.ec2.internal sh[23192]: [info] Images downloaded
Feb 01 05:06:03 ip-10-21-2-172.ec2.internal sh[23192]: [info] Pushing deis/slugrunner:latest to <IP_ADDRESS>:5000. This may take some time.
Feb 01 05:06:11 ip-10-21-2-172.ec2.internal sh[23192]: [info] Finished pushing deis/slugrunner:latest to <IP_ADDRESS>:5000.
Feb 01 05:06:11 ip-10-21-2-172.ec2.internal sh[23192]: [debug] Name: EXTERNAL_PORT, Val: 2223
Feb 01 05:06:11 ip-10-21-2-172.ec2.internal sh[23192]: [info] Listening on <IP_ADDRESS>:2223
Feb 01 05:06:11 ip-10-21-2-172.ec2.internal sh[23192]: [info] Accepting new connections.
Feb 01 05:06:11 ip-10-21-2-172.ec2.internal sh[23192]: [info] Checking closer.
Feb 01 05:06:11 ip-10-21-2-172.ec2.internal sh[23192]: [info] Builder is running.
Feb 01 05:06:11 ip-10-21-2-172.ec2.internal sh[23192]: [info] Checking closer.
Feb 01 05:06:11 ip-10-21-2-172.ec2.internal sh[23192]: [info] Accepted connection.
Feb 01 05:06:11 ip-10-21-2-172.ec2.internal sh[23192]: [error] Failed handshake: ssh: server has no host keys (&{{0xc82017c9a0}})
Feb 01 05:06:11 ip-10-21-2-172.ec2.internal systemd[1]: Started deis-builder.
Feb 01 05:08:01 ip-10-21-2-172.ec2.internal sh[23192]: [info] Checking closer.
Feb 01 05:08:01 ip-10-21-2-172.ec2.internal sh[23192]: [info] Accepted connection.
Feb 01 05:08:01 ip-10-21-2-172.ec2.internal sh[23192]: [error] Failed handshake: ssh: server has no host keys (&{{0xc8201cc700}})
Feb 01 05:14:58 ip-10-21-2-172.ec2.internal sh[23192]: [info] Checking closer.
Feb 01 05:14:58 ip-10-21-2-172.ec2.internal sh[23192]: [info] Accepted connection.
Feb 01 05:14:58 ip-10-21-2-172.ec2.internal sh[23192]: [error] Failed handshake: ssh: server has no host keys (&{{0xc82028c150}})
Feb 01 05:17:33 ip-10-21-2-172.ec2.internal sh[23192]: [info] Checking closer.
Feb 01 05:17:33 ip-10-21-2-172.ec2.internal sh[23192]: [info] Accepted connection.
Feb 01 05:17:33 ip-10-21-2-172.ec2.internal sh[23192]: [error] Failed handshake: ssh: server has no host keys (&{{0xc8202d2bd0}})
Feb 01 05:18:14 ip-10-21-2-172.ec2.internal sh[23192]: [info] Checking closer.
Feb 01 05:18:14 ip-10-21-2-172.ec2.internal sh[23192]: [info] Accepted connection.
Feb 01 05:18:14 ip-10-21-2-172.ec2.internal sh[23192]: [error] Failed handshake: ssh: server has no host keys (&{{0xc8202e89a0}})

core@ip-10-21-1-171 ~ $ etcdctl ls --recursive /deis/builder/
/deis/builder/users
/deis/builder/users/olalonde
/deis/builder/users/olalonde/62:c7:52:a8:aa:41:1d:5c:d6:80:4c:3d:7b:45:da:30
core@ip-10-21-1-171 ~ $ etcdctl ls --recursive /deis/platform
/deis/platform/sshPrivateKey
/deis/platform/enablePlacementOptions
/deis/platform/domain

Problem fixed after deleting /sshHostKey from etcd (etcdctl rm /sshHostKey) and restarting the platform. Not sure why though.

also fixed in v2 now that we do not rely on etcd :)


Updates for rvm-rpm usage, selinux and other fixes

As promised, here are the updates for compatibility with RPM installs as well as adding selinux support for Redhat-ish systems.  My apologies for the number of commits, I was experimenting with the best way to do things along the way.
The first requirement for RPM support was to make the whole thing independent of the rvm install location.  In order to accomplish this, I created a new fact called rvm_binary which points to the rvm executable.  Both the manifests and providers make use of this fact.  Also, the variable $rvm::system::rvmpath contains the rvm installation path, normally /usr/local/rvm for a script install or /usr/lib/rvm for the RPM install.  To get the packaged install, you set the use_pkg class parameter to true.
The other major bit is selinux support.  This is conditional on the selinux fact, which will be true on selinux-enabled systems.  The main part of this is accomplished by adding an after_install hook to rvm, which runs every time a ruby version is installed.  It will adjust the selinux file contexts on the ruby directories.  Also, there's an extra exec when building passenger, since the Apache-related executables need some special contexts.  Note that I found a bug in rvm hooks along the way, see issues 744-745 on rvm for info.
This has all been tested on RHEL6.  I have no Ubuntu systems so I would greatly appreciate it if you make sure I didn't break Ubuntu, and I'd more than welcome any other feedback/discussion on the changes.  Thanks for providing this very helpful module!

Steve, this looks great at first glance.  I will test it out ASAP.
Would you consider rebasing the pull request and possibly squashing some of the commits to make it a little easier to see what exactly has changed?   Basically squashing some of the "try this..., fix that..." commits that fix stuff added in one of your earlier commits.  http://help.github.com/rebase/  I'll pull it regardless, but it would make it easier to go through each commit.

Oh and thanks a ton for this!!!!

Hmm, I never even knew rebase existed, let me take a look at this.  I'm kind of confused about how pushing the rebasing will work.  I will have to experiment.  I can see how things could be made a lot clearer though.

I'm still confused about how this works when I've already pushed commits (especially given the "It is considered bad practice to rebase commits which you have already pushed to a remote repo." warning).  Do I need to destroy my puppet-rvm fork and then re-fork so I can push out rebased changes?  I'm afraid I still have much to learn about git.  :)

Steve.  Don't worry about it, I can pull as is.
But.... If you want to worry about it ;), here's how I would proceed:
Add my repo as a remote (if you haven't already):
git remote add upstream git://github.com/blt04/puppet-rvm.git
git fetch upstream

Create a new branch to work on (since you've already pushed to master):
git branch rpm master
git checkout rpm

Rebase the branch against upstream master:
git rebase upstream/master

There may be conflicts in the above.  If so, resolve them, or "don't worry about it" and I'll pull as is.
After rebasing, rearrange and/or squash some of your commits to make it a little easier to read through the history:
git rebase -i HEAD~38

You may end up doing this over and over while you clean stuff up.
Finally, when it all looks good locally, push it to github and either update this pull request, or open a new one.

Brandon, thanks for the explanation, it makes perfect sense.    Go ahead and pull this one as is, and I'll take this as a lesson on how to be a better gitizen in the future.  Anyway, I highly recommend that you examine the final diffs rather than the individual commits on this one.  :P

I'm working on another patch to support rvm installing rubies and gems from local mirrors.  I'll submit this as another pull request when it's ready (a better one, I promise!).  One thing that is bugging me a little is that every time I add another parameter, I have to add it to the rvm class then pass it along to rvm::system.  I'm starting to wonder if it isn't better just to eliminate the builtin run stage support and let the user specify a run stage if needed, which would do away with the need for the intermediate class.  That's totally your call though, the code will work either way.

Steve.  Can we move the discussion on rvm parameters / stage support to a different github issue, just to keep this thread relating to your rpm pull request?
I finally started looking at your pull request.  It appears to break one-pass multi-stage installs.  Apparently facter only gets run once, regardless of how many stages there are.  So when puppet tries to run the main stage after installing RVM, it fails to find the RVM binary (because the fact isn't loaded).
So either we need to do away with stage support, or assign the binary path via a variable (that would default to /usr/loca/rvm/bin/rvm).  Neither of these are great solutions.  I really like the idea of being able to do a complete install in one-pass.  It helps with stuff like Vagrant and other server-less puppet set ups.  Thoughts?

I will test this more thoroughly, but looking at the commit, I see a problem with the reference to $rvmpath/bin/rvm.  The RPM package bases the rvm install in /usr/lib, but the binaries just go into /usr/bin.  So there is no /usr/lib/rvm/bin/rvm to execute.
One problem I had when developing this - my ruby and provider-writing skills are not great.  If there is a way to access puppet DSL variables from the provider, the best solution might be to set an $rvm_binary variable inside puppet then access it from the provider, which would get rid of the fact.  But I didn't know how to do that.

Hold that thought - it just occurred to me that maybe scope.lookupvar that works for templates will also work inside a provider?  I will try that out today if I get a chance.
Also, I meant that the RPM install is based in /usr/lib/rvm, not /usr/lib.  :)

I don't think you can access variables in a provider (unless they were passed to the provider via attributes (like name, ruby_version, etc).  I could be wrong.
The big problem is that the rvm_binary is used to determine if the provider is suitable, and I think that all happens before any data is passed to the provider.  Again, I could be wrong.

Well, there's nothing really wrong with the path-search method you have implemented in the providers.  I was just trying to create a single canonical source for the location of the rvm command.  But if that won't work, we can just make an rvm_binary variable in the DSL and use that, and leave the providers with your latest change.  Or, we could just add a path argument to any rvm execs that would catch rvm no matter where it is rather than making a DSL variable.

I tried the second approach (add a path argument to rvm execs) and passenger failed to install.  I think that may be the best compromise (at least the expected behavior is identical).  I'll see if I can figure out why passenger wouldn't install.


pull request for e2e test at 01/28/2017 01:26:17

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


cannot import a pretrained mobilenet multiple times

Please make sure that this is a bug. As per our GitHub Policy, we only address code/doc bugs, performance issues, feature requests and build/installation issues on GitHub. tag:bug_template
System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow):
OS Platform and Distribution (e.g., Linux Ubuntu 16.04):Linux Ubuntu 16.04
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device:
TensorFlow installed from (source or binary): binary
TensorFlow version (use command below):1.12
Python version:2.7
Bazel version (if compiling from source):
GCC/Compiler version (if compiling from source):
CUDA/cuDNN version:10.1.105
GPU model and memory: m60, 8gb

I am trying to use a pre-trained MobileNet for transfer learning however i am running into a problem when i am trying to run the same command the second time
import tensorflow as tf
try:
    del g1
except:
    pass
tf.reset_default_graph()
g1 = tf.Graph() #or g1 = tf.get_default_graph() both result in same behavior
with g1.as_default():
    base_model = tf.keras.applications.MobileNetV2(input_shape=(160,160,3),
                                                       include_top=False,
                                                       weights='imagenet')

I can run this cell in jupyter alright the first time but the second time i try to run this i get
TypeError: Cannot interpret feed_dict key as Tensor: Tensor Tensor("Placeholder_207:0", shape=(160,), dtype=float32) is not an element of this graph.
However, I can run this cell multiple times perfectly fine
base_model = tf.keras.applications.MobileNetV2(input_shape=(160,160,3),
                                                       include_top=False,
                                                       weights='imagenet')

I was able to reproduce the mentioned issue on Tensorflow 1.12.

Any updates on this?

This is not Build/Installation or Bug/Performance issue. Please post this kind of support questions at Stackoverflow. There is a big community to support and learn from your questions. GitHub is mainly for addressing bugs in installation and performance. Thanks!

How is this not a Bug?

@Nitinsiwach Sorry for the delay in my response. I ran your code in TF1.15.0 and I don't see the issue. That particular block I ran 7 times without any issue. Please check the gist here. Thanks

Automatically closing this out since I understand it to be resolved, but please let me know if I'm mistaken.Thanks!


css3 transform

I couldn't really put my problem into words so I decided to put it into a video https://www.youtube.com/watch?v=VxBLlPk14Ps&feature=youtu.be
This issue happens on my OnePlus One and also my Galaxy s4.

Going to close as not fixable by our middleware. Feel free to file a bug on crbug.com or on crosswalk's JIRA.


Research how to create a custom login popup

Multiple stories have popped up, that require a popup - #146 , #18 , #40 . A research how to achieve that is a requirement:

[x] Check how to use custom login form with angular-oauth2-oidc
[x] If needed - check keycloak docs how to use the custom login form(especially for external providers, navigating to creation page, forgotten password etc.)
[x] A working POC would be great

After some digging - angular-oauth2-oidc provides a way to open keycloak's login page into a popup - docs.
The steps should be:

Create a silent-refresh.html static page(an example one can be found here);
Add the page as a static resource into the angular project(the easiest way is to add the html file into the assets folder).
Add a URL to the silent-refresh.html page to the configuration of angular-oauth2-oidc - silentRefreshRedirectUri property(if added to assets should be something like http://localhost:4200/assets/silent-refresh.html)
Add the URL as a valid redirect url for the client in keycloak - select the realm -> Clients -> the client used by the app -> Valid redirect URIs)

Small example POC
login-popup-poc.zip


Option to disable the faction gear option

I would like to see an option to disable the faction gear option in my addon options

I don't understand what you mean?
Do you mean disable factions from receiving masks automatically? If so you can just leave the faction to none which is the default.

Yesterday I checked out the options and my mask was gone when I chose the none option. Today it didn't happen. Maybe it was something else


The props (single, selectedItems & canAddItems) don't mix!

Issue summary
SCENARIO DESCRIPTION:
(A) The prop "single" is set to true
(B) The prop "canAddItems" is set to true
(C) The prop "onSelectedItemsChange" is set to {(selectedItems) => {
this.setState({
selectedItems: selectedItems,
});
}}
(D) The prop "selectedItems" is set to {this.state.selectedItems}
(1) End User selects an existing option, e.g. Green
(2) End User manually enters a new option (instead of the previously selcted option), e.g. ABC
DESIRED RESULT:
-> ONLY the manually eneterd option (ABC) is selcted.
ACTUAL RESULT (WHICH IS INCORRECT):
-> Both options are selected (Green & ABC).
WORKAROUND:
Remove the prop "selectedItems".
By doing so, you lose the functionality of dynamically changing the dropdwon label (the prop "selectText") into the selected option.
To get this functionality, you can use the following code for the prop "selectText".
selectText={this.state.demo1_SelectedItems.length === 0 ? "Demo 1 Field" : this.state.demo1_SelectedItems[0]}
Library versions
expo: 32.0.0
react-native: "https://github.com/expo/react-native/archive/sdk-32.0.0.tar.gz"
react-native-multiple-select: 0.5.3
Reproducible Code
import React, { Component } from 'react';
import { View } from 'react-native';
import MultiSelect from 'react-native-multiple-select';

class Screen1 extends Component {
	state = {
		demo1_SelectedItems: [],
		demo1_ItemsList: [{
			id: '11',
			name: 'Orange',
		}, {
			id: '12',
			name: 'Apple',
		}, {
			id: '13',
			name: 'Banana',
		}, {
			id: '14',
			name: 'Tomato',
		}],
	};

	demo1_OnSelectedItemsChange = (selectedItems) => {
		console.log('demo1_OnSelectedItemsChange - SelectedItems: ', selectedItems);

		this.setState({
			demo1_SelectedItems: selectedItems,
		});
	};

	demo1_OnAddItem = (newItemsList) => {
		this.setState({
			demo1_ItemsList: newItemsList,
		});
	}

	render() {
		return (
			<View>
				<MultiSelect
					items={this.state.demo1_ItemsList}
					uniqueKey="id"
					displayKey="name"

					selectText="Demo 1 Field"
					searchInputPlaceholderText="Select or Type a value"
					filterMethod="full"

					single={true}
					selectedItems={this.state.demo1_SelectedItems}

					onSelectedItemsChange={this.demo1_OnSelectedItemsChange}

					canAddItems={true}
					onAddItem={this.demo1_OnAddItem}
				/>
			</View>
		);
	} // end of: render
} // end of: component

export default Screen1;

##WORKAROUND CODE:
				<MultiSelect
					items={this.state.demo1_ItemsList}
					uniqueKey="id"
					displayKey="name"

					// Workaround - Change#1: Replace the following line, with the one after it.
					// selectText="Demo 1 Field"
					selectText={this.state.demo1_SelectedItems.length === 0 ? "Employee's Role" : this.state.demo1_SelectedItems[0]}

					searchInputPlaceholderText="Select or Type a value"
					filterMethod="full"

					single={true}

					// Workaround - Change#2: comment the following line
					// selectedItems={this.state.demo1_SelectedItems}

					onSelectedItemsChange={this.demo1_OnSelectedItemsChange}

					canAddItems={true}
					onAddItem={this.demo1_OnAddItem}
				/>

You can hide the dropdown model by manually toggle the selector to add this command. (by using ref)
this.yieldRef._toggleSelector();
Then need to push the new items to existing array of items

        const obj = {
            id: value[0],
            name: value[0]
        }
        items.push(obj)

Then set the state to selected item as the current value
     this.setState({selectedYield: value});

Altogether simple pass this function onSelectedItemsChange
changeYieldText = value => {
        this.yieldRef._toggleSelector();
        const obj = {
            id: value[0],
            name: value[0]
        }
        items.push(obj)
        this.setState({selectedYield: value});
}


Update README regarding JS dependency

Cool stuff. As far as I see from the showcase page most stuff (sorting, filtering, pagination) is totally usable without JavaScript. As I am in search for a solution that also works without JavaScript it would be nice if that is reflected in the README. Or am I mistaken?
There might still be some minor issues (i.e. I found in-place-editing to surprisingly not working ;) ), but there a simple enabling of the edit-button only if JS is enabled shouldnt be too problematic.

I confirm that in-place-editing is not working but is a problem with Config initialization if you change hidden class :+1:
For example for Bootstrap 4 you need to use d-none and not hidden. I initialized d-none but when merged you have hidden class in your JS configuration.


Add Doctest infra and add doctests to EpiAwareUtils

As the title this PR makes a start on resolving #48 by adding doctests to @testitem and adding some to code base for EpiAwareUtils
After looking into it it doesn't look like there is a way to make sure example code is being run in docstrings unless we use doctests and definitely not if we want the output in Documenter (which I think we do to guide our users).

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

Project coverage is 93.18%. Comparing base (849b707) to head (21e093a).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #332   +/-   ##
=======================================
  Coverage   93.18%   93.18%
=======================================
  Files          50       50
  Lines         528      528
=======================================
  Hits          492      492
  Misses         36       36

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


Force local node to not use local node executor

it seems the node resource automagically discovers which node is the Rundeck server and defaults to using the "local node executor" instead of the Ansible executor.
how can I prevent this and force the "local node/server" to also use the Ansible executor?

answer here: https://docs.rundeck.com/docs/administration/configuration/plugins/configuring.html#node-execution
at the project level example:
service.NodeExecutor.default.local.provider=com.batix.rundeck.plugins.AnsibleNodeExecutor


Make sure release runs after all tests are green

Using https://github.com/badboy/travis-after-all-rs

Code in branch after-all, waiting for #70 to be merged


How to define Aperture Stop for off-axis simulation in a lens system?

Suppose i am designing a doublet. How to define Aperture Stop for off-axis simulation and design?

i found how to define the Aperture stop surface. it is given in the LensAssembly function.

fine


fix: remove variant classes from OK/Cancel buttons

First PR on this, so apologies if I miss something.
b-modal has ok-variant and cancel-variant to set the button variants.  These aren't working because we have explicit btn-primary/btn-secondary classes in there.  That means you get both classes inserted, and the primary wins.  We don't need these classes (I think) because variant will already ensure the correct one is inserted.

I don't think this should get merged.
The current implement has default colors as btn-primary and btn-secondary but you can actually override the colors by specifying any color you want; note that okVariant and cancelVariant are props with default only values.
I think it's best to allow the end user to theme as required rather than force a specific color.
Feedback
PR wise (as if it were to get merged):
I think it'd make sense to remove all variant related code since that is the purpose of the PR.
That is, Lines 35, 46, 79 & 108

Thanks for the comment.
If you try to do that, say to secondary, you end up with a button which has both the btn-primary and btn-secondary classes set on the same element. Primary is there because it's hardcoded, and secondary gets added via the variant prop.
In that case the btn-primary class wins out, and your attempt to override fails.
The default color values are already applied because of the default values of the variant prop. We don't need the classes to be hardcoded present too.
If that's not clear I can set up a repro so you can see it.
On 6 May 2022 00:53:29 BST, MrTanoshii @.***> wrote:

I don't think this should get merged.
The current implement has default colors as btn-primary and btn-secondary but you can actually override the colors by specifying any color you want; note that okVariant and cancelVariant are props with default only values.
I think it's best to allow the end user to theme as required rather than force a specific color.
Feedback
PR wise (as if it were to get merged):
I think it'd make sense to remove all variant related code since that is the purpose of the PR.
That is, Lines 35, 46, 79 & 108
--
Reply to this email directly or view it on GitHub:
https://github.com/cdmoro/bootstrap-vue-3/pull/380#issuecomment-1119147165
You are receiving this because you authored the thread.
Message ID: @.***>

Agreed to remove the hardcoded variants.

The current implement has default colors as btn-primary and btn-secondary

These could be props default values, not hardcoded generated html.

Ah my bad, it's best to remove those classes
I pass color props and it overrides on my side.
For e.g.
<b-modal
        id="exampleModal"
        v-model="showModal"
        size="xl"
        title="Modal title"
        okVariant="danger"
        cancelVariant="success"
        fade
        >...
</b-modal>

That being said, you are right in removing those classes there!
Edited my original reply to reflect that.

@MrTanoshii Phew!  It would be a bit embarrassing if my first PR on this project was dumb.
Now I just need to work out how to apply a local build of this package into my own project, so that I can make more significant fixes.  I haven't done that before, so I need to find out the obvious way of doing it that everyone knows...

you can use
Github Repo
  "dependencies": {
    "bootstrap-vue-3": "USER/REPO",
  },

Local repo
  "dependencies": {
    "bootstrap-vue-3": "file:PATH/TO/DIR",
  },

See https://docs.npmjs.com/cli/v8/configuring-npm/package-json#local-paths


Introduce DotNetBuildOuterRepo control

See https://microsoft-my.sharepoint.com/:w:/p/mmitche/EdchH8A4okpOoKhkki1sJEYB0LyAaaCyX5_6A887gL744Q?e=JERvht for full design:

When “true”, indicates that the infrastructure is executing within outer repo build or inner repo build phases
This is essentially ArcadeBuildFromSource or ArcadeBuildVertical.

Should be active when in the outer build or inner build (when invoked from the outer build)

Arcade changes complete.

Changes now complete in orchestrator.


Cannot launch gazebo bringup

I have been trying to use the Gazebo launcher, but the Franka robot cannot be visualized because Gazebo cannot load the .dae files from the franka_description package.  All other packages work fine.

Gazebo version: 11.10.2
I will await your response, thank you.

I managed to visualize the Franka arm changing the utils.xacro. I changed the filename of the visual and collision meshes to "file://$(find franka_description)/meshes..."
However, when I try to visualize the robot end effector, I have the same problem. Even with the patch I used with the Franka arm, the problem persists.
Here is a photo of the log.

As well, an important detail to add is that I can visualize and control the robot end effector in RVIZ with no problems.

Hi @JuanCarlosHR2003 ,
what exactly did you change from what string to what other string?
Maybe you can also give us the version you are using + franka_ros2 version so that we could check. At least we didn't see any problems at the moment :smile:
Cheers,
Andreas

I have same problem. I am using libfranka 0.14.1 and ros2 humble

I also had same problem. (ubuntu 22.04 + ros2 humble + libfranka 0.14.1)
As @JuanCarlosHR2003 said, I changed file name in franka_description/robots/common/utils.xacro
origin) mesh filename="package://${description_pkg}/meshes/robot_arms/${arm_id}/visual/${name}.dae"
replace) mesh filename="file://$(find franka_description)/meshes/robot_arms/${arm_id}/visual/${name}.dae"

After with these replacement, do colcon build.


Small error in the Over 13 VC description

Description : This card is a proof that you are over 13 yo.
Please verify that the same error is not on the over15 VC.

Issuer side


[Feature] set CustomPath with config

if configfile is set (via ENV or --param) gitea should look at a setting CUSTOM_PATH = xy and if it exist use this as custom path
(at the moment this is only posible to set via ENV)

well I't looks like I have to send a pull ;)

nop still didnt have time

@6543 I don't understand what you're proposing here.

you cant set CustomPath by config

this is because CustomPath is determined to find ConfigFile if nothing is set
but I like to be able to do it the other way around - it would be posible if 1. moving the "DeterminCustomPath" code into its own func. 2. use a temporary customPath if configFile not specified to determin config 3. read CustomPath from config - if none use temporary customPath as CustomPath

why I like to have this: I use the alpine package and had to edit the init script to be able to specify CustomPath but I would like to do it the "normal" linux way by just editing /etc/gitea/app.ini

:eyes:

Is it still needed? I do not understand the necessary either ....
Now the docker image uses a wrapper script, there would be no custom path problem for docker users.

Now we have "WORK_PATH" in ini, the custom-path is based on the work-path by default
So I think this problem should have been resolved.

well it was about an installation via alpine package, and it's can also be solved by re-organizing the storage structure

work-path is a more general path than custom-path.
Setting it in app.ini could resolve most legacy path-related problems IMO.
Or, what's the real case you would like to deal with?

https://gitlab.alpinelinux.org/alpine/aports/-/blob/master/community/gitea/APKBUILD#L72
/var/lib/gitea/

I see. Maybe the best approach for setting the "custom-path" is ln -s /my/custom /var/lib/gitea/custom .....
Although after the "path refactoring", adding a CUSTOM_PATH into "app.ini" is quite easy, but I guess we don't want to make the config system that complex.


[IOTS-455] Notification count doesn't reduce after a notification has been viewed in publisher

The count of notifications stays same after a listed notification was clicked and viewed. == Steps == 1. Logged in to publisher and notifications are listed at top. 2. Click notifications and all the notifications should be displayed as a drop down. 3. Click one notification in order to view it and that redirects back to application page (if the notification is for a rejected app) But the notification count remains same even after reading the listed notifications.(check attachment)Reference: https://wso2.org/jira/browse/IOTS-455

This option is disabled in publisher in the latest release.


Link "Specifying Test Settings for Visual Studio Tests" broken

Clicking on the link under See Also with the text "Specifying Test Settings for Visual Studio Tests" results in a 404 not found error.

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

ID: 21ac034a-a498-f3e3-28d0-175b6fead219
Version Independent ID: 747194a8-ffcf-60b3-b525-6f8fe00066a2
Content
Content Source

The broken link is still there.
Instead of removing the link, please can you point it at the right place? I'd like to follow it!

@mophdog This link should be fixed next week with https://github.com/MicrosoftDocs/visualstudio-docs-pr/pull/1553. Sorry, not sure what happened here before!


Update variable mappings for 2019 ACS

Using reference table in db-acs-2020 repo.

meta2019.csv
Draft updated 2019 metadata using reference table provided by population. Currently, all fields except for census_variable are pulled from a join with the previous metadata json file. We need to fill in base_variable, domain, rounding, source for the 84 new variables. We also need to know which variables are no longer relevant to include (for example, many of the new language variables replace old language variables that we no longer have data for).

Metadata with base variables, domains, and notes: meta2019.csv


Split reverse mode for Tapir

We've discussed this on Slack with @willtebbutt but I wanted to make sure where we stand on split reverse mode, i.e. separating the forward sweep from the reverse sweep.
The idea is being able to perform multiple reverse sweeps with different seeds after just one forward sweep. A typical example computing a Jacobian (where there is one seed per basis vector of the output space).
My question is the following: is that currently possible with Tapir's rrule? IIUC, the answer is no for functions that mutate their argument (https://github.com/gdalle/DifferentiationInterface.jl/issues/142), but what about simple allocating functions?
I took inspiration from https://github.com/withbayes/Tapir.jl/blob/f5e2b90cd17fd3127dd0fd8dfa617bc112275626/src/interface.jl#L9-L15
to try and write what I called value_and_pullback_split in DifferentiationInterface
function value_and_pullback_split(f, x)
    rule = build_rrule(f, x)
    tf = zero_tangent(f)
    tx = zero_tangent(x)
    out, pb!! = rule(CoDual(f, tf), CoDual(x, tx))
    y = copy(primal(out))
    function pullback(dy)
        dy_righttype = convert(tangent_type(typeof(y)), copy(dy))
        ty = increment!!(tangent(out), dy_righttype)
        new_df, new_dx = pb!!(ty, tf, tx)
        return new_dx
    end
    return y, pullback
end

But the behavior of the resulting closure changes at each call.
For some functions it gives different results:
julia> y, pullback = value_and_pullback_split(copy, [1.0])
([1.0], pullback)

julia> pullback([1.0])
1-element Vector{Float64}:
 1.0

julia> pullback([1.0])
1-element Vector{Float64}:
 3.0

julia> pullback([1.0])
1-element Vector{Float64}:
 6.0

For others it downright errors:
julia> y, pullback = value_and_pullback_split(x -> x .^ 2, [1.0])
([1.0], pullback)

julia> pullback([1.0])
1-element Vector{Float64}:
 2.0

julia> pullback([1.0])
ERROR: BoundsError: attempt to access 1-element Vector{Vector{Float64}} at index [0]
Stacktrace:
     ⋮ internal @ Unknown
 [4] Pullback
   @ ~/.julia/packages/Tapir/BqxEi/src/interpreter/s2s_reverse_mode_ad.jl:632 [inlined]
 [5] (::var"#pullback#10"{Vector{…}, Tapir.Pullback{…}, CoDual{…}, Vector{…}, NoTangent})(dy::Vector{Float64})
   @ Main ./REPL[16]:10
Use `err` to retrieve the full stack trace.
Some type information was truncated. Use `show(err)` to see complete types.

What should I copy to allow for independent pullback calls? Probably out, tf and tx?

Right, yes, so: it's not just the arguments which get modified on the reverse-pass -- any intermediate mutable data structures that get modified during the forwards-pass will also get modified during the reverse-pass. For multiple reverse passes to be safe given only a single forwards-pass, I think it would have to be the case that no mutation occurs on the forwards-pass.
I've not thought at all about how you might go about checking this, so I really do think it's the case that you'll have to do a single forwards pass per reverse-pass.
It would definitely be possible to do multiple reverse-passes at the same time if I modified the package to explicitly handle "chunked" reverse-mode, where you pass multiple cotangents back at the same time. This would involve a complete overhaul of the cotangent system and all of the rules though, so it's certainly not happening in the forseeable future.

These intermediate data structures are stored in the rule object?

Exactly. For example, if you have a function:
function f(x::Vector{Float64})
    y = map(sin, x)
    z = map(cos, y)
    return z
end

there will, somewhere in the rule derived to differentiate this function, be memory for y.

And hypothetically, if we want to compute many many pullbacks with the same forward sweep, would it make sense to deepcopy the rule?

Possibly, but most likely we would find out there are some intricacies that are annoying for one reason or another.


search in the select box not working ?

I tried using this component as my select box component.. all works well, I installed it according to the given guidance.. but it seems the searching box didn't work properly on my side ?
I try using the searching functionality but the record inside the component didn't sorted properly ? did I missed something ?
this is my html markup in jade language
ui-select(theme="select2" ng-model="cekbaru")
                        ui-select-match(placeholder="pickup countries" ) {{cekbaru.name}}
                        ui-select-choices(repeat="country in ui.countries")
                            small {{country.name}}

update apparently I miss the filtering system using filter... managed to implement it and now it's working! thanks!


setlanguage()怎么才能切换成功？

我使用setlanguage()语言切换后，页面语言一直变不了，怎么才能切换成功？

Could you please create a more detailed issue in English? I did not understand anything. Ideally, the new issue should include a reproduction repository.


@files trying to read non-existent files output an error instead of silently handling with empty node.

Missing files for @file on disk should just resolve to an empty node like so:

After looking for the problem by comparing with the original Leo: openFileHelper should not call g.es_exception for non existent file.

Fixed with #116


Update translation strings as of 20180831T233621

yarn run v1.9.4
$ eslint *.js settings VendorSearch
/home/jenkins/workspace/folio-org/ui-plugin-find-vendor/PR-2/project/VendorSearch/VendorSearchModal.js
3:21  error  Unable to resolve path to module '@folio/vendors/src/Main'  import/no-unresolved
✖ 1 problem (1 error, 0 warnings)
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.


Race wizard UI bug

Start the race wizard. Select an immunity. close the wizard, and select new race again.
All is reset to 25 points, but the immunity checkbox is still selected. This seems to be just an ui bug though, as you can still adjust the now invisible hab range with the buttons. The same bug is also true of the costs 75% extra on the research screen, that is also left selected after closing and restarting the wizard, if you have selected it.

Apparently I had to go to the editor and run it from there after pull to properly rebuild, not just run it from main menu. So the immunities bug was apparently fixed. But the 75% extra still needs to be fixed. And also, for the jack of all trades, it's text should be changed to "start at 4" ;-)


setUseSessionInUrl(false) is no more allowed in Magento 2.3.5

The use of setUseSessionInUrl(false) in Model/Template/Processor.php is no more allowed in Magento 2.3.5. The Result is the following exception:
User Deprecated Functionality: Session ID is not used as URL parameter anymore. in /var/www/magento-dev/vendor/magento/module-email/Model/Template/Filter.php on line 274 at /var/www/magento-dev/vendor/magento/framework/App/ErrorHandler.php:61)"} []
Since the function setUseSessionInUrl of Filter has no more function except to create the exception, it is sufficient to delete the line.

I can reproduce this and indeed deleting the function call to setUseSessionInUrl in Model/Template/Processor.php:77 solves this

Hi,
Still not working after removing that line too. Can you please help me?


python312Packages.plugwise: 0.37.2 -> 0.37.3

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python312Packages.plugwise is: Python module for Plugwise Smiles, Stretch and USB stick
meta.homepage for python312Packages.plugwise is: https://github.com/plugwise/python-plugwise
meta.changelog for python312Packages.plugwise is: https://github.com/plugwise/python-plugwise/releases/tag/v0.37.3
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.37.3 with grep in /nix/store/aw911r16pslilpycr5xgbg6viwl447k0-python3.12-plugwise-0.37.3
found 0.37.3 in filename of file in /nix/store/aw911r16pslilpycr5xgbg6viwl447k0-python3.12-plugwise-0.37.3

Rebuild report (if merged into master) (click to expand)

3 total rebuild path(s)

3 package rebuild(s)

First fifty rebuilds by attrpath
home-assistant-component-tests.plugwise
python311Packages.plugwise
python312Packages.plugwise

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/aw911r16pslilpycr5xgbg6viwl447k0-python3.12-plugwise-0.37.3 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python312Packages.plugwise https://github.com/r-ryantm/nixpkgs/archive/6d6ef740018f6abbdb651122cc00a3cfaa37b0ef.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/6d6ef740018f6abbdb651122cc00a3cfaa37b0ef#python312Packages.plugwise

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/aw911r16pslilpycr5xgbg6viwl447k0-python3.12-plugwise-0.37.3
ls -la /nix/store/aw911r16pslilpycr5xgbg6viwl447k0-python3.12-plugwise-0.37.3/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review --extra-nixpkgs-config '{ allowInsecurePredicate = x: true; }' run on x86_64-linux 1

5 packages built:

home-assistant-component-tests.plugwise
python311Packages.plugwise
python311Packages.plugwise.dist
python312Packages.plugwise
python312Packages.plugwise.dist

Maintainer pings
cc @fabaff for testing.

Result of nixpkgs-review pr 305368 run on x86_64-linux 1

5 packages built:

home-assistant-component-tests.plugwise
python311Packages.plugwise
python311Packages.plugwise.dist
python312Packages.plugwise
python312Packages.plugwise.dist


[Suggestion] Quicker access to frequently used or recently used icons

It would be nice to have faster access to frequently used icons. Could potentially be in the form of a row or two of icons in the dialog that pops up when users want to add an icon.

Lovely idea! The question is how many recent icons would be saved? What makes sense? 5 or 10?

Almost a cop-out answer, is it possible to add it as a plugin setting?
Otherwise, maybe 10, because I'm not sure how many people actually scroll that much to find icons as opposed to just looking them up via text.

Makes sense yeah :D it is possible!
I think I will just use 5 as a default value. I will try to add this feature asap :) thanks for the suggestion!

So the feature should be implemented. If you want, you can test it. I will try my best to release it soon!

This feature was released in version 1.0.6


Mockoon DOWN

🚨 Service Alert: Check the details below
Monitor Details

Monitor Name: Mockoon
Incident Status: TRIGGERED
Severity: critical
Monitor Status: DOWN
Monitor Health Checks: 1
Monitor Failure Threshold: 1

Actions

View Monitor

[start_datetime:1732974840] [end_datetime:1732955100]

The incident has been auto resolved
Total downtime: 1 minutes


Testsuite improvements

Removed obsolete tests.
Have another pass at making test parameter names more uniform.
Converted more tests to Hypothesis.
Systematically use utils.isclose over math.isclose.
Split and simplified a few tests.
Added a few additional tests for Vector.length:

the 3 axioms of normed vector space;
v.length ≃ √v².

Ok, now I'm super confused? The lint problem doesn't reproduce locally, and it doesn't exist.

bors r+

bors retry


ci: Fix license header check

TODO

[ ] Add missing license headers

I'm currently working on this, and doing it right broke things...


Reduce doc build time

Proposed changes
In order to reduce the documentation build time, the inheritance diagram was removed from the auto-generated API docs.
Types of changes
What types of changes does your code introduce?
Put an x in the boxes that apply

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Functionality update (non-breaking change which updates or changes existing functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] CI/CD update (an update to the CI/CD workflows, scripts, and/or configurations)
[x] Documentation update (an update to enhance the user experience when reading through the docs)

Checklist
Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code.

[x] I have followed the guidelines in the CONTRIBUTING document
[x] I have signed the CLA
[x] I have checked to ensure there aren't other open Pull Requests for the same update/change
[x] I have performed a self-review of my code
[x] My code follows the style guidelines of this project
[x] I have commented my code, particularly in hard-to-understand areas
[x] Basic linting passes locally with my changes
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] I have added necessary documentation (if appropriate)
[x] I have updated the Changelog with a brief description of my changes

Unfortunately this didn't decrease the build time as much as desired, but it does enhance the auto-generated drivers to point to the correct IEEE 488.2 classes.


Let NodePool controller wait if infraID is empty

What this PR does / why we need it:
The NodePool controller assumes infraID as contractual info (particularly uses it for naming generation). This https://github.com/openshift/hypershift/pull/1055 introduced a race so MachineDepoyment could be created without the infraID in the name.
This PR let the NodePool controller wait until th infraID is set in the HC API.
Which issue(s) this PR fixes (optional, use fixes #<issue_number>(, fixes #<issue_number>, ...) format, where issue_number might be a GitHub issue, or a Jira story:
Fixes https://github.com/openshift/hypershift/issues/1059
Checklist

[x] Subject and description added to both, commit and PR.
[x] Relevant issues have been referenced.
[ ] This change includes docs.
[x] This change includes unit tests.

@alvaroaleman PTAL


synth sound talking

is there any example to make the synth sound talking?

@taalua thanks for your question! I'm not quite sure what you are referring to regarding the synth sound talking? Do you mean speech synthesis?
torchsynth is currently focused on digital audio synthesis for machine learning. The architecture and dsp modules are inspired by musical synthesizers. -- we have examples of sounds from the default voice here: https://soundcloud.com/user-357924775/synth1k1 -- or an example of how to start creating your own synths using the available modules here: https://torchsynth.readthedocs.io/en/latest/modular-design/new-synths.html

@jorshi thank you for your answer. I am wondering if there is a way to mix speech and the synthesizers together to create a robotic voice or chipmunks for example. Do you have such examples?

@taalua Are you interested in contributing?
You might also consider looking pink trombone which we would like to integrate into torchsynth.


use obfuscator can not dynamic route imports

Describe the bug

use obfuscator build

not use obfuscator build

show err

Reproduction
https://stackblitz.com/edit/vitejs-vite-28ctb7?file=package.json
Steps to reproduce
npm install
npm run build

System Info
System:
    OS: Linux 5.10 Debian GNU/Linux 11 (bullseye) 11 (bullseye)
    CPU: (12) x64 Intel(R) Core(TM) i7-8700 CPU @ 3.20GHz
    Memory: 55.21 GB / 62.81 GB
    Container: Yes
    Shell: 5.1.4 - /bin/bash
  Binaries:
    Node: 18.16.0 - /usr/local/bin/node
    Yarn: 1.22.19 - /usr/local/bin/yarn
    npm: 9.5.1 - /usr/local/bin/npm
  npmPackages:
    @vitejs/plugin-legacy: ^2.0.0 => 2.3.1
    @vitejs/plugin-vue2: ^1.1.2 => 1.1.2
    vite: ^3.0.2 => 3.2.6

Used Package Manager
npm
Logs
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Make sure this is a Vite issue and not a framework-specific issue. For example, if it's a Vue SFC related bug, it should likely be reported to vuejs/core instead.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

Looks like it's because Vite's preload marker expects double quotes, but the obfuscator transforms it to single quotes. The fix would be to handle both single and double quotes:
https://github.com/vitejs/vite/blob/5731ac9caaef629e892e20394f0cc73c565d9a87/packages/vite/src/node/plugins/importAnalysisBuild.ts#L39

I went through it in a different way，This is my way:

vite.config.js

src/router/index.js


Add twitter and facebook sharing to each blog post

@ctubbsii  suggested this in an offline conversation.

If we add it to the overall page template, it would make it easier to share all pages, blog posts or otherwise.

https://simplesharingbuttons.com/
https://superdevresources.com/share-buttons-jekyll/

I added some basic support for just twitter with #40.  I looked into using the output of https://simplesharingbuttons.com/, but I was not sure how to integrate it into the site.


Update comments for gitignore

Coverage remained the same at 52.381% when pulling 1aa68f9ba28f11f3b974466ceeb0033084fe212b on develop into 8024f2746e3b21e4180a53077d8048c24ecf1f5c on master.

Coverage remained the same at 52.381% when pulling b6994e7bec02b51b5b94e3ace6def949e224847e on develop into 8024f2746e3b21e4180a53077d8048c24ecf1f5c on master.


Prevent smartparens from inserting a second single quote automatically in emacs-lisp-mode

Describe your request
I believe that, by default, smartparens shouldn't insert a second quote automatically after the user has entered one in emacs-lisp-mode, as single quotes never seems to be doubled in Emacs Lisp. Other users on Reddit seem to agree with this.
Briefly explain its use-case

Open an emacs-lisp-mode buffer.
The user should be able to type a single quote easily, without having to delete the new one inserted automatically by smartparens.

This actually seems to be a bug. Closing for now. I'll investigate this and open a bug report if necessary.


[FLINK-35344][cdc-base] Move same code from multiple subclasses to JdbcSourceChunkSplitter

Current, subclasses of JdbcSourceChunkSplitter almost share same code, but each have one copy. It's hard for later maintenance.
Thus, this Jira aim to move same code from multiple subclasses to JdbcSourceChunkSplitter, just like what have done in
AbstractScanFetchTask(https://github.com/apache/flink-cdc/issues/2690)

@PatrickRen , @ruanhang1993 , @leonardBang , CC

Seems MySQL CI is taking significantly more time (90min vs. 30min) and fails eventually. Could @loserwang1024 please take a look?
Failed CI job link: https://github.com/apache/flink-cdc/actions/runs/9073569533/job/24931065832

Could @loserwang1024 please take a look?
I'd like to do it. But this PR do nothing with mysql cdc.

Just rebase this pr to master.


Bug Fix #4710

default intents should not raise undefined parsing error:

showing error if the default intents and self.domain.intents to be added into domain

Status (please check what you already did):

[  ] made PR ready for code review
[ ] added some tests for the functionality
[ ] updated the documentation
[  ] updated the changelog
[  ] reformat files using black (please check Readme for instructions)

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Thanks for the PR! Sorry, I managed to change the entire function in the short time 🙈 can you try to fix the conflicts with your implementation? I will comment on it as well


TypeError: Cannot read property 'isValidUTF8' of undefined (Websocket)

Hi, I'm creating a very simple socket client to listen to Gdax server, but I always get this problem.
The code:
import request from 'request-promise';
import config from 'config';
import Gdax from 'gdax';

// get all assets pairs
const opts = {
  url: `${config.exchanges.gdax.api}/products`,
  headers: {
    'User-Agent': 'myapp'
  },
  json: true
};

request(opts)
  .then(data => data.map(d => d.id))
  .then((assets) => {
    const websocket = new Gdax.WebsocketClient(assets);
    websocket.on('message', (data) => {
      console.log(data);
    });
  })
  .catch((err) => {
    console.log(err.message);
  });

The error:
/Projects/exchanges/node_modules/gdax/node_modules/ws/lib/Receiver.js:532
            if (!Validation.isValidUTF8(messageBuffer)) {
                           ^

TypeError: Cannot read property 'isValidUTF8' of undefined
    at Projects/exchanges/node_modules/gdax/node_modules/ws/lib/Receiver.js:532:28
    at Receiver.applyExtensions (Projects/exchanges/node_modules/gdax/node_modules/ws/lib/Receiver.js:371:5)
    at Projects/exchanges/node_modules/gdax/node_modules/ws/lib/Receiver.js:508:14
    at Receiver.flush (Projects/exchanges/node_modules/gdax/node_modules/ws/lib/Receiver.js:347:3)
    at Receiver.finish (Projects/exchanges/node_modules/gdax/node_modules/ws/lib/Receiver.js:541:12)
    at Receiver.expectHandler (Projects/exchanges/node_modules/gdax/node_modules/ws/lib/Receiver.js:499:31)
    at Receiver.add (Projects/exchanges/node_modules/gdax/node_modules/ws/lib/Receiver.js:103:24)
    at TLSSocket.realHandler (Projects/exchanges/node_modules/gdax/node_modules/ws/lib/WebSocket.js:825:20)
    at emitOne (events.js:96:13)
    at TLSSocket.emit (events.js:191:7)
    at readableAddChunk (_stream_readable.js:178:18)
    at TLSSocket.Readable.push (_stream_readable.js:136:10)
    at TLSWrap.onread (net.js:561:20)
[nodemon] app crashed - waiting for file changes before starting...

After some searching, some people with the same issue said is something about the ws module version and is incompatibility with utf-8-validate. I've tried everything with every version and nothing.
Thank you!

What version of the ws module do you have installed in your node modules? Can you check the package.json of the /Projects/exchanges/node_modules/gdax/node_modules/ws/ directory?

I have in my node_modules the version ^3.0.0. On /Projects/exchanges/node_modules/gdax/node_modules/ws/ package.json is 1.1.1. Is the new version with the latest version?

There has been an upgrade I believe, try upgrading your gdax node package to latest and see if it is working 👍

I'll try to use github's version then, because the last available version on npm is 0.4.2 (which has the ws 1.1.1 version)

The ws newest version is working. Thanks @awitherow ! I'll close the issue

🎉

@fb55 would be good to have some release notes on this?


use mysql's default string length for mariadb

As I presented in #881 , mariadb doesn't have a default string size, unlike mysql, which leads to an sqlalchemy compile error. From what I understood (and tried), simply using the mysql default length and checking for the mariadb dialect.name is enough to prevent this error.

I realize now that it might be worth mentioning that the reason I wanted the default value was because setting the max_length metadata wasn't working. After some digging I ended up finding this pull request: #857 which fixes that part of the problem, so maybe these 2 could be merged as they fix different ends of the same problem.


Add form_redirect_uri to login page during reload

When the user session expires, this form_redirect_uri is used to retain the context. But, the value is lost when login page is reloaded due to any error (ie incorrect password, csrf). This fix adds the value to the model if present in the original request

Coverage increased (+0.1%) to 86.468% when pulling 8b45010b9a1ca78514b064098a65226a7b780a8b on GESoftware-CF:fix/restore_login_form_redirect_uri into b3b9bd123d38e737f72246e0c2d5f972f97a5e2b on cloudfoundry:develop.

Coverage increased (+0.1%) to 86.468% when pulling 8b45010b9a1ca78514b064098a65226a7b780a8b on GESoftware-CF:fix/restore_login_form_redirect_uri into b3b9bd123d38e737f72246e0c2d5f972f97a5e2b on cloudfoundry:develop.

@bsekar I'm wondering if this fix should be as far in as a controller.
after all, we process the redirect parameter in here
https://github.com/cloudfoundry/uaa/blob/8a20d975c89139ce50f6fe5bb1757d366f845662/server/src/main/java/org/cloudfoundry/identity/uaa/web/UaaSavedRequestCache.java#L82-L85
I'm wondering if we could call request.setAttribute here and save the redirect. I'll review it some more

Coverage decreased (-0.08%) to 86.393% when pulling 79d03176e35ef2737194dedf302cdf1779c0f89b on GESoftware-CF:fix/restore_login_form_redirect_uri into a1df68d731d3ca65388812f885fdf241afe3ff97 on cloudfoundry:develop.


Updates the KFP SDK overview

This update removes references to old ways of building components and pipeline steps, and replaces it with a short summary that links to the docs on building pipelines and components.

Preview: https://deploy-preview-2689--competent-brattain-de2d6d.netlify.app/docs/components/pipelines/sdk/sdk-overview/#building-pipelines-and-components

/lgtm
to me
/hold
to give others a chance to review

/lgtm

/lgtm

/unhold


Updates!!!

First major updates have been made.
CLIENT SIDE: v0.1.8

I have finally figured out how to get websockets to work as it should be. No more confusing blocks.
The server will give the clients data, instead of the clients constantly asking for data.
I've shortened everything down to a simple set of blocks: The socket data (reporter), the system status (reporter), connection status (boolean), connect to server (command block), disconnect from server (command block), and send data (command block).

SERVER SIDE: v1.6

I have cleaned up everything so it's easier to read.
I have simplified the protocol's overhead, so now it just sends data instead of useless headers and junk. Pure data, the least amount of overhead possible!
Should be a bit more effective and compact because I removed all things unnecessary.

I will be working on:

Figuring out how to send packets to individual clients (so I can get private communications working)
Better documentation and commenting
and a whole lotta other things

Version 0.1.9/01.7 (client/server) has fixed bugs and has private data streams restored. Closing this thread.


ApplicationModel should use IList instead of List

A number of items on ApplicationModel are exposed as List<T> - as general purpose cleanup, these should be IList<T>

Should probably do this as a sweep of public APIs

This was done fb21b73


Suggestion: don't reset window position on logout

This is pretty minor, but it would be nice if the app window didn't reset itself to the center of the main monitor whenever you hit the Logout button. I normally leave a scan running on a second monitor, and every time the connection drops such that I have to logout and back in (which I assume is a server issue that we can't do anything about), I have to re-position the window back where I want it.
Thanks to everyone who's worked on this.

+1
Or being able to save locations to a list would be a big improvement for running multiple instances.


Load hv_sock module

Add support for Hyper-V socket. It does nothing on Linux and macOS but load the module on Windows.

I moved the code to createdisk.sh. Although I don't see why we don't put most of the changes in snc.sh and keep createdisk.sh as minimal as possible.

I don't see why we don't put most of the changes in snc.sh and keep createdisk.sh as minimal as possible.

@guillaumerose the motivation is all the VM related changes are part of createdisk.sh and all the openshift cluster related changes are part of snc.sh to keep us bit sane :)


Matrix map_reduce: missing token for extractvalue in generated llvm code

When trying to compile the llvm code generated from the program below, clang will abort with the error
error: expected value token
    %acc_132593 = extractvalue {i32, i32, i32, [2 x [2 x float]]*} , 3
                                                                   ^

.plugin mem;
.plugin core;
.plugin matrix;

.let M1 = %matrix.Mat(2, (2, 3), %math.F32);
.let M2 = %matrix.Mat(2, (3, 2), %math.F32);
.let M3 = %matrix.Mat(2, (2, 2), %math.F32);

.con .extern map_reduce_test [mem : %mem.M,a: M1, b: M2, c: %mem.Ptr(M3, 0), return : .Cn %mem.M] = {
    .ret (`mem, prod) = internal_mapRed_matrix_prod(2,3,2, %math.f32) $(mem, a, b);
    .let `mem : %mem.M = %mem.store(mem, c, prod);
    return mem
};

Adding
.plugin clos;
produces valid llvm code, however, the generated llvm-code doesn't define the function map_reduce_test. Is there an easy fix for that or should I open another issue?

Adding .plugin clos; produces valid llvm code, however, the generated llvm-code doesn't define the function map_reduce_test. Is there an easy fix for that or should I open another issue?

The function is named map_reduce_test**_reshape**.
This is a relict from when reshape renamed every function.
I changed it to not rename externals. But somehow I forgot to merge the change into master.
Note: There are a few other things that appear weird (the three higher-order Pis without usage).

Alright, thanks a lot!