Localparam not working properly

When the code was compiled using Surelog-yosys plugin, the bitstream was generated, but would not work properly. The State Machine would not cycle through, which caused each button on the counter to only work once until it was reset using the center button. This was caused by the use of localparam
This is probably caused by the parameters defaulting to one bit. Similar to this bug
The default Yosys parser has no problems with this.
Problem Code:
	localparam ZERO = 2'b00;
	localparam INC = 2'b01;
	localparam ONE = 2'b10;

Solution: Either changing each occurrences of ZERO, INC, and ONE with their respective values or using typedef enum to create a state type.
The bug appeared in
OneShot.zip written by Professor Mike Wirthlin

Thanks for reporting, we will investigate this.

After further investigation it appears the problem is not that it defaults to one bit, but that in a case statement the parameter is not handled correctly if it is multiple bits wide.
In OneShot.sv, if ONE: in the case statement is replaced by 2'b10: the code functions properly.

I have done further testing with this simplified design: test.sv, test.xdc
It looks like this bug is caused by the way that Yosys handles the UHDM model of the design. Parsing the design directly in yosys (using the versions that are installed in the conda environment) and printing the AST dump resulted in:
always @*
        begin
          led = 8'b 10101010;
          case (sw)
            0:
              led = 8'b 00000010;
            1:
              led = 8'b 00000111;
            -2:
              led = 8'b 00111100;
            -1:
              led = 8'b 11111111;
          endcase
        end

It processes the localparams as two's compliment so when even when sw = 2'b10 it does not match the value of -2 that is expected. I manually installed Yosys and the UHDM plugin on my machine using the yosys-systemverilog repo as a reference. When I parsed it again using this version, the AST dump was different.
always @*
        begin
          led = 8'b 10101010;
          case (sw)
            2'b 00:
              led = 8'b 00000010;
            2'b 01:
              led = 8'b 00000111;
            2'b 10:
              led = 8'b 00111100;
            2'b 11:
              led = 8'b 11111111;
          endcase
        end

I then replaced yosys from my f4pga install with this new version of Yosys. I replaced it in opt/f4pga/xc7/conda/envs/xc7/share and opt/f4pga/xc7/conda/envs/xc7/bin. When I ran the tool-chain with these changes the design worked correctly. What is the best way to update the version of Yosys that is installed with the conda environment?

As Zach227 has observed, this can be fixed by upgrading the tools.
With Yosys 0.27+22 (git sha1 0f5e7c244) and systemverilog_plugin (git sha1 e7070ca645), which are used here: https://github.com/antmicro/yosys-systemverilog
I'm getting the expected output for OneShot.sv.
I think the issue can be closed.


Access denied for ' '@'localhost' Doesnt seem to have the username i added in

So ive set all the correct information in the .env in the api file, but i still get an error such as

even though i have the username set it doesnt seem to show it

And the keys that are in their are just temporary doesnt matter if others see them

This is when i run setup:db

This is when i run setup:db

This is a bug, it seems that the sequelize script is not loading the configuration from the .env file.

Ah alright, any ETA on a fix? Or any quick fix I can do?
On Fri, Mar 27, 2020, 9:48 PM Francisco Pizarro<EMAIL_ADDRESS>wrote:

This is a bug, it seems that the sequelize script is not loading the
configuration from the .env file.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/dvaJi/ReaderFront/issues/662#issuecomment-605377190,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ANDL65XY3MIW2PXW4MQI6Z3RJVJQTANCNFSM4LUR6RDA
.

@Thatguy553 It is not a critical bug, so is not necessary to put an ETA. And I think the bug is in the .sequelizerc file, so you might take a look at it and check why the env variables are not being loaded.

Ah, I haven't been able to get it setup, when I go to do setup:db etc I get
access denied even though I've added the correct password and database
name, so I assumed because the username isn't loading it wasn't allowing me
to to it
On Fri, Mar 27, 2020, 11:00 PM Francisco Pizarro<EMAIL_ADDRESS>wrote:

@Thatguy553 https://github.com/Thatguy553 It is not a critical bug, so
is not necessary to put an ETA. And I think the bug is in the .sequelizerc
file, so you might take a look at it and check why the env variables are
not being loaded.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/dvaJi/ReaderFront/issues/662#issuecomment-605385350,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ANDL65VH6KOVGTXRXDWAKRLRJVR6HANCNFSM4LUR6RDA
.

also, this could be an issue for me because im not familiar with mariadb, i followed the steps and installed mariadb but once i opened the client and selected my database n such, the npm commands got syntax errors, and when i ran the commands in the directory through a normal command prompt the command ran but resulted in the pics above, at first it would tell me no database selected, have no idea how to select the database

@Thatguy553 I updated the .sequelizerc to load the .env file, could you try now?
And regarding mariadb, it's just exactly the same as MySQL.

Yea just a minute I'll load it up now
On Mon, Mar 30, 2020, 10:17 PM Francisco Pizarro<EMAIL_ADDRESS>wrote:

@Thatguy553 https://github.com/Thatguy553 I updated the .sequelizerc to
load the .env file, could you try now?
And regarding mariadb, it's just exactly the same as MySQL.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/dvaJi/ReaderFront/issues/662#issuecomment-606357405,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ANDL65QQI2LLMAENJZ6LLXTRKFHFPANCNFSM4LUR6RDA
.

alright so its different now, migrate:db seems to work except for whatever this is, doesnt seem to be a fatal error though?

but once i do db:seed it works fine up until here:

am i doing something wrong? i did a fresh install of all the files, but i did fill out the .env.example in the root and api folder and renamed them to .env, i did this assuming they needed to be filled out because without doing so it would usually tell me no database selected

I updated the code and tested creating a new database, now it should work.

Alright, I'll try it once I get home, thanks
On Tue, Mar 31, 2020, 11:30 PM Francisco Pizarro<EMAIL_ADDRESS>wrote:

I updated the code and tested creating a new database, now it should work.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/dvaJi/ReaderFront/issues/662#issuecomment-607006738,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ANDL65SRDYR7AJCN6FKUCF3RKKYONANCNFSM4LUR6RDA
.

so now it seems to have all worked. Thanks! one thing though, did i miss something on creating an admin account or is this another bug? When i run the dev:admin command it starts it up and brings me to a page with the option of logging in or to sign up, but in sign up no matter what it tells me there was an error signing up and i did try changing the password for the admin account in the database to test it that way but it just tells me incorrect credentials. Did i miss something?

it doesnt give any errors

I actually just checked the chrome console and seen this

The error net::ERR_CONNECTION_REFUSED means that you, either didn´t start the API or you got some errors.
Did you run npm run dev:api?

I did, but i got these errors then and now.

ugh, the script was missing :(, I write it in the README file but not implemented in the package.json lol.
Now it should work.

Lmao, alright I'll try it again when I get on my pc
On Sat, Apr 4, 2020, 1:26 PM Francisco Pizarro<EMAIL_ADDRESS>wrote:

ugh, the script was missing :(, I write it in the README file but not
implemented in the package.json lol.
Now it should work.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/dvaJi/ReaderFront/issues/662#issuecomment-609061920,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ANDL65STDSZBUHE5NWI3JLLRK5UUJANCNFSM4LUR6RDA
.

so far so good, though it does seem the some of the dependencies installed are the wrong versions.

Once i created an account it told me there was an error but i received an email to verify my account, so i did that and it brought me here

If you need the rest of the page let me know and ill get all of it

I could not reproduce the error with babel-jest and jest.
and I fixed the error in the Activate component, it was related to an async function in useEffect

Alright I'll try again tomorrow thanks for the help!
On Tue, Apr 7, 2020, 11:30 PM Francisco Pizarro<EMAIL_ADDRESS>wrote:

I could not reproduce the error with babel-jest and jest.
and I fixed the error in the Activate component, it was related to an async
function in useEffect https://github.com/facebook/react/issues/14326
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/dvaJi/ReaderFront/issues/662#issuecomment-610732334,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ANDL65QNCI4AM5WWWQCSEVLRLPVVPANCNFSM4LUR6RDA
.

Doesnt post any errors in logs or anything but still tells me there was an issue creating your account no matter what i do.
the first account i created was with the email i used in the .env files to set up, but it didnt send me an email like before, so i tried creating an account with another email.
then it sent me an email to verify my account but once i logged in it told me unauthorized.
Then i went into the database, apparently the first email is verified but the password doesnt work.
So then i stole that accounts activation token and added it to the second account i created. It then let me in, so it seems to work as long as you go through those extra steps lol, whatever the issue is with creating an account makes it a pain in the arse

Its because the account needs to have the ADMIN role, right now it has to be modified directly in the database (check the user model https://github.com/dvaJi/ReaderFront/blob/master/api/src/modules/user/model.js#L17).
I will work like that until I or someone else works on #437, and I think I have to create a new issue to handle the initial setup.

Alright thanks for the info
On Wed, Apr 8, 2020, 6:13 PM Francisco Pizarro<EMAIL_ADDRESS>wrote:

Its because the account needs to have the ADMIN role, right now it has to
be modified directly in the database (check the user model
https://github.com/dvaJi/ReaderFront/blob/master/api/src/modules/user/model.js#L17
).
I will work like that until I or someone else works on #437
https://github.com/dvaJi/ReaderFront/issues/437, and I think I have to
create a new issue to handle the initial setup.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/dvaJi/ReaderFront/issues/662#issuecomment-611221560,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ANDL65WVQNVMUSXEEID5BXDRLTZJRANCNFSM4LUR6RDA
.


data_uri_to_file_1.toFile is not a function

I am going to use data-uri-to-filepackage in my angular 4.

toFile(dataURI).then(file => {
console.log(file.mimeType, file.data, file.extension);
});

But the error shows like this

Following as error message, toFile is not a function. Check out importing or file existing.

import { toFile } from 'data-uri-to-file';
I have already import the package.

It's basic of using modules, not related to this module, related to your dev environments? did you use it with ts?

Yes, I am using TypeScript.


Fix link in pod-security-admission

Resolves #29525
Points a broken link to the appropriate location in pod-security-admission to show appropriate configuration for exemptions.

/lgtm

/approve
Thanks @tylerauerbeck


ElevatedButton transparent is not transparent (But grey)

After upgrading to Flutter 2.8, an ElevatedButton shadow color cant be set to transparent. Prior to updating, this was working well.
Example: ElevatedButton(style: ElevatedButton.styleFrom(primary: Colors.transparent, shadowColor: Colors.transparent)
The transparent property is showing as grey now.

Works for me in Chrome and iPhone 12 emulator with the code below (see screenshots). My Scaffold background is lightgrey. Maybe you have some other parameters such as backgroundcolor on?
flutter --version:
Flutter 2.8.1 • channel stable • https://github.com/flutter/flutter.git
Framework • revision 77d935af4d (9 days ago) • 2021-12-16 08:37:33 -0800
Engine • revision 890a5fca2e
Tools • Dart 2.15.1
Code:
child: ElevatedButton(
              style: ElevatedButton.styleFrom(
                primary: Colors
                    .transparent, //Palette.traqYellow,
                shadowColor: Colors.transparent,
                padding: const EdgeInsets.symmetric(
                    horizontal: 20.0, vertical: 15.0),
              ),
              child: const Text(
                'Login',
                style: TextStyle(color: Colors.black),
              ),
),
```child: ElevatedButton(
              style: ElevatedButton.styleFrom(
                primary: Colors
                    .transparent, //Palette.traqYellow,
                shadowColor: Colors.transparent,
                padding: const EdgeInsets.symmetric(
                    horizontal: 20.0, vertical: 15.0),
              ),
              child: const Text(
                'Login',
                style: TextStyle(color: Colors.black),
              ),),

![Screenshot 2021-12-25 at 11 13 20](https://user-images.githubusercontent.com/56278665/147382840-fb286921-627a-456d-881a-5883e2e75165.png)

![Screenshot 2021-12-25 at 11 14 05](https://user-images.githubusercontent.com/56278665/147382851-e88600a5-57dd-425b-bc1b-8b1da487140d.png)

@Heidelbergg
Tried sample code on latest stable which works as expected:
void main() {
  runApp(MaterialApp(
    home: Scaffold(
      body: Center(
        child: ElevatedButton(
          style: ElevatedButton.styleFrom(primary: Colors.transparent, shadowColor: Colors.transparent),
          child: Text('test', style: TextStyle(color: Colors.red),),
          onPressed: () {},
        )
      )
    ),
  ));
}

 stable, master flutter doctor -v
[[✓] Flutter (Channel stable, 2.8.1, on Mac OS X 10.15.4 19E2269 darwin-x64,
    locale en-GB)
    • Flutter version 2.8.1 at /Users/dhs/documents/fluttersdk/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 77d935af4d (13 hours ago), 2021-12-16 08:37:33 -0800
    • Engine revision 890a5fca2e
    • Dart version 2.15.1

[✓] Android toolchain - develop for Android devices (Android SDK version 30)
    • Android SDK at /Users/dhs/Library/Android/sdk
    • Platform android-30, build-tools 30.0.3
    • ANDROID_HOME = /Users/dhs/Library/Android/sdk
    • Java binary at: /Users/dhs/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/202.7486908/Android
      Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6915495)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.3, Build version 12C33
    • CocoaPods version 1.10.1

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

[✓] Android Studio (version 4.1)
    • Android Studio at /Users/dhs/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/202.7486908/Android
      Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6915495)

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

[✓] Connected device (4 available)
    • Darshan's iphone (mobile)  • 21150b119064aecc249dfcfe05e259197461ce23 •
      ios            • iOS 14.4.1 18D61
    • iPhone 12 Pro Max (mobile) • A5473606-0213-4FD8-BA16-553433949729     •
      ios            • com.apple.CoreSimulator.SimRuntime.iOS-14-3 (simulator)
    • macOS (desktop)            • macos                                    •
      darwin-x64     • Mac OS X 10.15.4 19E2269 darwin-x64
    • Chrome (web)               • chrome                                   •
      web-javascript • Google Chrome 93.0.4577.82

• No issues found!

I had the same issue with AppBar on Android, after upgrading from flutter 2.5.1 to 2.8.1.
The appBar still had a shadow with the following theme:
final appBarTheme = AppBarTheme(
      foregroundColor: Theme.of(context).colorScheme.onBackground,
      backgroundColor: Colors.transparent,
      shadowColor: Colors.transparent, // Does not work
    );

The shadow is darker than without shadowColor: Colors.transparent, actually.
But I found a solution: the problem seems to be related to elevation, with elevation: 0, in the theme, the shadow disapeares, whether it's set to transparant or not.
final appBarTheme = AppBarTheme(
      foregroundColor: Theme.of(context).colorScheme.onBackground,
      backgroundColor: Colors.transparent,
      elevation: 0, // Removes the shadow
    );

This behavior is clearly different than in 2.5.1, where the shadow where always displayed, even with a elevation of 0.
Shadow coloration seems to work correctly tho.

$ flutter --version
Flutter 2.8.1 • channel stable • https://github.com/flutter/flutter.git
Framework • revision 77d935af4d (4 weeks ago) • 2021-12-16 08:37:33 -0800
Engine • revision 890a5fca2e
Tools • Dart 2.15.1

Hope this helped!

Thank you @delwinwal it worked the way you portrayed it.

I have noticed that on the Android Studio emulator, the transparent property is still not working even after the mentioned change above. But when running on my own device, it works perfectly.

@Heidelbergg
Regarding emulator behavior, please check if it happens with specific emulator or with other emulators too. Since it works on  device as you mentioned, it could be an emulator specific behavior.


Got incorrect position using renderObject when list is reversed and firstItemAlign is end

Only occurs when items are not enough to fill the screen
It's easy to reproduce
Scaffold(
      body: FlutterListView(
        reverse: true,//
        delegate: FlutterListViewDelegate(
        (BuildContext context, int index) {
      return Container(
        margin: EdgeInsets.symmetric(horizontal: 20, vertical: 5),
        decoration: BoxDecoration(
          borderRadius: BorderRadius.circular(8),
          color: Color(0xFF333333),
        ),
        height: 100,
        child: Builder(
          builder: (BuildContext context) {
            return GestureDetector(
              onTap: () {
                try{
                  var box = context.findRenderObject() as RenderBox;
                  final Offset offset = box.localToGlobal(Offset.zero);
                  var size = box.size;
                  Get.to(Material(
                    color: Colors.transparent,
                    child: Stack(
                      children: [
                        Positioned(
                          left: offset.dx,
                          top: offset.dy,
                          child: Container(
                            color: Colors.yellow,
                            width: size.width,
                            height: size.height,
                          ),
                        )
                      ],
                    ),
                  ));
                }catch(e){
                  print(e);
                }
              },
            );
          },
        ),
      );},
          childCount: 5,
          firstItemAlign: FirstItemAlign.end, //for sure the list is aligning top
        ),
      ),
    );

Thanks report the issues. I'm going to check it and fix it ASAP

Hi @Iridescentangle
I'm tested in my computer. It work fine. Could you please share your screenshot to me to identify what issue you got.

https://user-images.githubusercontent.com/33859295/205048220-5b958f8d-cc55-4a79-87f2-e2766471db39.mp4
@robert-luoqing

Only occurs when items are not enough to fill the screen It's easy to reproduce
Scaffold(
      body: FlutterListView(
        reverse: true,//
        delegate: FlutterListViewDelegate(
        (BuildContext context, int index) {
      return Container(
        margin: EdgeInsets.symmetric(horizontal: 20, vertical: 5),
        decoration: BoxDecoration(
          borderRadius: BorderRadius.circular(8),
          color: Color(0xFF333333),
        ),
        height: 100,
        child: Builder(
          builder: (BuildContext context) {
            return GestureDetector(
              onTap: () {
                try{
                  var box = context.findRenderObject() as RenderBox;
                  final Offset offset = box.localToGlobal(Offset.zero);
                  var size = box.size;
                  Get.to(Material(
                    color: Colors.transparent,
                    child: Stack(
                      children: [
                        Positioned(
                          left: offset.dx,
                          top: offset.dy,
                          child: Container(
                            color: Colors.yellow,
                            width: size.width,
                            height: size.height,
                          ),
                        )
                      ],
                    ),
                  ));
                }catch(e){
                  print(e);
                }
              },
            );
          },
        ),
      );},
          childCount: 5,
          firstItemAlign: FirstItemAlign.end, //for sure the list is aligning top
        ),
      ),
    );

@robert-luoqing Once you tap an item in the list, its position will be calculated by using its context，so the yellow container in the new page indicates the position of the item you tapped.You can find out it's just like the opposite of the original list before reversed

@Iridescentangle Thanks, I have fixed the issue. The version is 1.1.20.


docs: add documentation and examples for sdk macros

Documents log! macro
Documents testing_env! macro
Fixes some warnings relating to docs
Update MockedBlockchain reference to remove the need to import outside of macro. (there might be a better way to handle the SDK references, but I'm trying to keep changes contained)

TODO:

Documenting other parts of the module (extra credit from #360)

Please let me know thoughts, changes to wording, or any ideas on what is in need of documenting for this issue.
closes #360

Also, is there a reason for not supporting including only the VM context and config, where the match goes from 1 to 3 params that's worthwhile documenting? Or is this a change acceptable to make in this/another PR?

Thank you, @austinabell ! You are very appreciated. Today is my last day of PTO, I'll be looking into this more tomorrow, hopefully with more stable internet.


[CP] null check error in flutter attach

issue_link
https://github.com/flutter/flutter/issues/107709
Commit Hash
1371b8dc3cf3448226c8d2ab0f097741e8364734
Target
beta
pr_link
TODO
Impacted Users
Flutter developers using the command flutter attach for iOS targets
Impact Description
if a flutter developer uses the command flutter attach without the optional --app-id=$ID option, they will hit a null check _CastError, because of a mistake when migrating the file to null-safety.
Workaround
The workaround would be to manually provide the --app-id, would be the bundle identifier.
Risk
low
Test Coverage
yes
Validation Steps
To test, you would need a macOS host device with a working Xcode setup and a physical iOS device.

Ensure the iOS device is attached to the macOS host and is detected via flutter devices
Launch any flutter app on the device in debug mode with flutter run -d $phone
Detach the CLI tool from the app by pressing d
Re-attach to the device with flutter attach
Ensure the tool does not crash with a _CastError.

@christopherfujino is this reproducible on stable also?
CC: @zanderso for review.

CP lgtm, this is a fix for one of the null-safety migration mistakes that were anticipated.

@christopherfujino is this reproducible on stable also?
CC: @zanderso for review.

No, as the regression happened in between the two branch points. Also FYI I'm working on the PR now...

Thank you both for the info!


Get-PartnerCustomerDevice does not return assigned Policies on devices

I have imported new computers using New-PartnerCustomerDeviceBatch without specifying any Policy since the customer has configured that to be applied automaticly . I can see that this works when looking in PartnerCenter or in the debug output of Get-PartnerCustomerDevice.
When I issue the command Get-PartnerCustomerDevice to get all the imported computers in the batch, I dont get the Policies property set! I have used the command using -debug and there I can see that the Policy value is shown in the output but it doesn't get returned.
$device = Get-PartnerCustomerDevice -CustomerId $CustomerID -BatchId $Batch -Debug
Debug output:
Body:
{
"totalCount": 1,
"items": [
{
"id": "XXXX",
"serialNumber": "XXX",
"modelName": "XXX",
"oemManufacturerName": "XXX",
"policies": [
{
"key": "o_o_b_e",
"value": "AzureAD Join Default Profile"
}
],
"uploadedDate": "2020-01-30T09:10:33.2417951+01:00",
"attributes": {
"objectType": "Device"
}
}
],
"links": {},
"attributes": {
"objectType": "Collection"
}
}
Verifying returned object values:
$device | select *
AllowedOperations   :
DeviceId            : XXXX
HardwareHash        :
ModelName           : XXX
OemManufacturerName : XXX
Policies            :
ProductKey          :
SerialNumber        : XXX
UploadedDate        : 2020-01-30 08:10:33
As you can see, the value of Policies is empty...

Thanks for fixing the bug in the upcoming release!
We are going to have a huge rollout for a customer within a week or so and would really need this thing to work at that time so we can see whether the computer have recieved a policy or not.
Do you have a timeframe for the next release? Will it be in a day or two??? ;-) (Or would it be possible for you to send me a fixed version that we can use until you release it public?)

@Crille77 currently there is not a timeframe for the next release. However, if you are interested you can download the artifact from the build that happened when the code changes I made were merged. You can find the artifact here.

Hi!
I downloaded the artifact and unzip'ed the files in an empty directory (D:\temp\pc). Then I unloaded the old ParnterCenter Module. After that was done, I changed directory of my PS session to the PartnerCenter Artifact directory (D:\temp\pc) and there I imported the module by import-module .\PartnerCenter.psm1.
After that is done, I verified that the module was loaded. In the output, I can see that the version is 0.0 and that it is loaded from the correct directory.
get-module PartnerCenter | select ModuleType, Version, Name, ModuleBase, Path

ModuleType : Script
Version    : 0.0
Name       : PartnerCenter
ModuleBase : D:\temp\pc
Path       : D:\temp\pc\PartnerCenter.psm1

If I now tries to verify the Policy-value again, I end up in getting the same bad result...
$dev =Get-PartnerCustomerDevice -CustomerId $SelectedCustomer.CustomerID -BatchId "XXX" -Debug
DEBUG: [AccessTokenAuthenticator] The specified access token expires at 2020-02-05 08:55:27
DEBUG: [AccessTokenAuthenticator] Constructing the authentication result based on the specified access token
DEBUG: [AccessTokenAuthenticator] The UPN claim value is<EMAIL_ADDRESS>DEBUG: [AccessTokenAuthenticator] Constructing the resource account value based on specified access token
DEBUG: ============================ HTTP REQUEST ============================
HTTP Method:
GET

Absolute Uri:
https://api.partnercenter.microsoft.com/v1/customers/XXXXXXXX/deviceBatches/XXX/devices

Headers:
Accept                        : application/json
MS-PartnerCenter-Client       : Partner Center PowerShell
MS-CorrelationId              : X
X-Locale                      : en-US
MS-RequestId                  : X
MS-SdkVersion                 : <IP_ADDRESS>
DEBUG: ============================ HTTP RESPONSE ============================
Status Code:
OK

Headers:
MS-CorrelationId              : X
MS-RequestId                  : X
MS-CV                         : X
MS-ServerId                   : X
Request-Context               : appId=cid-v1:03
Date                          : Wed, 05 Feb 2020 08:08:27 GMT

Body:
{
  "totalCount": 1,
  "items": [
    {
      "id": "GUID",
      "serialNumber": "1234567890",
      "modelName": "HP ProDesk 600 G5 Desktop Mini",
      "oemManufacturerName": "HP",
      "policies": [
        {
          "key": "o_o_b_e",
          "value": "AzureAD Join Default Profile"
        }
      ],
      "uploadedDate": "2020-02-03T12:00:49.1484531+01:00",
      "attributes": {
        "objectType": "Device"
      }
    }
  ],
  "links": {},
  "attributes": {
    "objectType": "Collection"
  }
}

$dev.Policies doesn't give me any output and verifying if empty confirms it not being set and the debug output still outputs the correct info that should be in that variable...
!$dev.Policies
True

Do you have any other suggestions??? Am I doing something wrong??

OK... I have had a lot trouble when trying the artifact... I once succeded using it... Most of the times I end up in trouble getting "Sequence contains no matching element" when trying to import a batch.
I have tried both having 3.0.6 installed and loading the artifact using get-module .\PartnerCenter.psm1. I have also tried to replace all the files in my 3.0.6 module directory with the artifact files.
Is there something wrong in the artifact or am I doing something wrong???
Some Debug info:
DEBUG: ============================ HTTP REQUEST ============================
HTTP Method:
POST

Absolute Uri:
https://api.partnercenter.microsoft.com/v1/customers/xxx/deviceBatches

Headers:
Accept                        : application/json
MS-PartnerCenter-Client       : Partner Center PowerShell
MS-CorrelationId              : 382126df-bd9e-4b6b-8077-4831af400712
X-Locale                      : en-US
MS-RequestId                  : aee8cb30-2cc0-494a-adef-37c447afd336
MS-SdkVersion                 : <IP_ADDRESS>

Body:
{
  "BatchId": "xxx",
  "Devices": [
    {
      "ModelName": "HP ProDesk 600 G5 Desktop Mini",
      "OemManufacturerName": "HP",
      "Policies": [],
      "SerialNumber": "xxx",
      "UploadedDate": "0001-01-01T00:00:00Z",
      "Attributes": {
        "ObjectType": "Device"
      }
    }
  ],
  "Attributes": {
    "ObjectType": "DeviceBatchCreationRequest"
  }
}
DEBUG: ============================ HTTP RESPONSE ============================
Status Code:
InternalServerError

Headers:
MS-CorrelationId              : 382126df-bd9e-4b6b-8077-4831af400712
MS-RequestId                  : aee8cb30-2cc0-494a-adef-37c447afd336
MS-CV                         : 3fL0vvIO/kiZKfEl.0
MS-ServerId                   : 0000112D
Request-Context               : appId=cid-v1:03ce8ca8-8373-4021-8f25-d5dd45c7b12f
Date                          : Wed, 05 Feb 2020 16:17:28 GMT

Body:
{
  "code": 0,
  "description": "Sequence contains no matching element",
  "data": [],
  "source": "PartnerFD"
}

Update...
I have tested using New-PartnerCustomerDeviceBatch from 3.0.6 and then issued the exact same using the artifact. Version 3.0.6 works perfect but the artifact throws "Sequence contains no matching element" no matter what.
But one positive thing is that after I imported one computer using 3.0.6, I used Get-PartnerCustomerDevice from the artifact version to get the policies and it works! One fix and one more bug...

Hi @isaiahwilliams!
Do you have any update regarding bugs? (Fixed bug in Policy & New bug in artifact when trying to upload a new batch).
I would really need a fix for this asap... We will start deployment of 2000+ computers this week where this is quite important to have in place... Is it doable for you to fix the artifact so both uploading a new batch and getting computer policies works in a release or a new artifact quite soon?
TIA /Christian

@Crille77 version 3.0.7 was just released and it contains a fix for this issue. Please let us know if you have any other issues with the module.


Query: incorrect sql generated for query with groupby aggregate also projecting grouping key and ordering by said key afterwards

query:
ctx.Orders.GroupBy(
                        o => o.OrderID, o => new
                        {
                            o.OrderID,
                            o.OrderDate
                        })
                    .Select(
                        g => new
                        {
                            g.Key,
                            Avg = g.Average(e => e.OrderID),
                            Max = g.Max(o => o.OrderDate)
                        }).OrderBy(e => e.Key).ToList();

sql:
SELECT [o].[OrderID] AS [Key], AVG(CAST([o].[OrderID] AS float)) AS [Avg], MAX([o].[OrderDate]) AS [Max]
FROM [Orders] AS [o]
GROUP BY [o].[OrderID]
ORDER BY [o].[OrderID] AS [Key] DESC

Exception:
Incorrect syntax near the keyword 'AS'.

Duplicate of #12826


Migration to Typescript

This does not use the official prosemirror build chain, but instead keep our custom build chain based on Rollup.
While the commit is big I tried to minimize changes as much as possible. Specifically no logic should have been touched at all.
Typing is now generated by typescript and bundled by api-extractor, so that is always synced.

@marijnh I hope you don't mind me pinging you here. I know you are not maintaining this project. However I am in the progress of migrating this package from JS to TS, like you did recently all other prosemirror packages. And I meant to re-use your build tools to keep everything consistent. But I might need guidance on how you use them, and whether there are a good fit for this package too ?
First I am a bit confused by the absence of tsconfig.json in any prosemirror packages. How/where do you manage your TypeScript config ? eg: are you using strictNullChecks in your config ?
Then ./node_modules/.bin/pm-buildhelper src/index.ts will correctly building something in dist/. But it will always build, even if I have blatant typing error in my source. Is this on purpose ? or am I missing your tools ?
And finally what is your development workflow ? I could not find any "watch" options in your tools. Are you using tsc --watch directly ? but then what about tsconfig.json ?
PS: @ocavue, the conversion should be almost done. The only things missing are a solid build system and enabling strictNullChecks (no small feat :sweat_smile: ).

See the readme in https://github.com/prosemirror/prosemirror for my multi-package setup. But since this is maintained separately, you'll probably want to use a different setup for it.

@PowerKiKi Thank you so much for your contribution! I'm sorry that it takes me so long to find a time reviewing it. I posted some review comments above. Let me know what do you think about them.

I fixed everything, except the last point. Maybe you can have a look again...


Allow a "category" attribute

Maybe it is in the specs but i have not seen it. VCs are displayed in groups as " Identity cards", "loyalty cards", "professional.", etc. It would be great to specify in the output descriptors where the wallet should display the credential.

DIF discussion:

Thierry: suggestion to add to v1.
Daniel: shouldn't block v1 but can be included in some v1.x.
Discussion around moving to CM

Concerns around dependencies @ThierryThevenet

I propose to add "category" as an optional property of the entity style object considering that it is only a visual option (user could rearrange "cards" in his own custom categories like in a personal finance app)
value must be a string.
Not sure to find the correct way to make it interoperable but there is a list of domains ->  https://www.w3.org/TR/vc-use-cases/#user-needs  it could be a guide line for wallet providers/issuers : Education, Retail, Finance...
WC v1 would be great.

#29 would arbitrarily allow various fields, such as category to be accomodated.

well then i guess credman is ready to go to v1

@ThierryThevenet I believe we discussed you adding a PR to accommodate this in a prior DIF call? Adding you to the issue.


Error while training

with this training script:
project_root="......../simpsons_proj/pre-trained-keras-example"
model="resnet50"
path_to_data=$project_root/"the-simpsons-characters-dataset/simpsons_dataset"
path_to_weight_directory=$project_root/"weight_directory"
path_to_tensorboard_logdir=$project_root/"tensorboard_direcoty"
max_epochs=20000
python train.py
--pretrained_model=$model
--data-dir=$path_to_data
--weight-directory=$path_to_weight_directory
--tensorboard-directory=$path_to_tensorboard_logdir
--epochs=$max_epochs
the training does not proceed with the following error

2018-07-04 15:43:53.353028: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1041] Created TensorFlow device (/job:localhost/replica:0/task:0/device:GPU:1 with 67 MB memory) -> physical GPU (device: 1, name: Tesla P100-PCIE-16GB, pci bus id: 74f2:00:00.0, compute capability: 6.0)
train.py:177: UserWarning: Update your fit_generator call to the Keras 2 API: fit_generator(<generator..., use_multiprocessing=True, workers=4, validation_data=<generator..., steps_per_epoch=200, epochs=20000, callbacks=[<keras.ca..., validation_steps=10)
pickle_safe=True,
/usr/local/lib/python2.7/dist-packages/keras/engine/training.py:2087: UserWarning: Using a generator with use_multiprocessing=True and multiple workers may duplicate your data. Please consider using thekeras.utils.Sequence class.   UserWarning('Using a generator with use_multiprocessing=True' Traceback (most recent call last):   File "train.py", line 177, in <module>     pickle_safe=True,   File "/usr/local/lib/python2.7/dist-packages/keras/legacy/interfaces.py", line 91, in wrapper     return func(*args, **kwargs)   File "/usr/local/lib/python2.7/dist-packages/keras/engine/training.py", line 2224, in fit_generator     class_weight=class_weight)   File "/usr/local/lib/python2.7/dist-packages/keras/engine/training.py", line 1877, in train_on_batch     class_weight=class_weight)   File "/usr/local/lib/python2.7/dist-packages/keras/engine/training.py", line 1480, in _standardize_user_data     exception_prefix='target')   File "/usr/local/lib/python2.7/dist-packages/keras/engine/training.py", line 113, in _standardize_input_data     'with shape ' + str(data_shape)) ValueError: Error when checking target: expected dense_2 to have 4 dimensions, but got array with shape (64, 42)

I have the same problem too.

Thanks for @ykg3mp and @ghimiremukti we will try to take a look at this sometime soon.

I think I found the problem, at least for vgg19
The code was missing the flattening part in line 31:

output = Flatten()(model_base.output)

Now I am getting accuracy outputs and the neural net is training.

@ykg3mp and @ghimiremukti  thank you for pointing this out! I have just pushed up a fix. Please try pulling the latest repository version with git pull and re-running the training script. Let me know if it resolves the issue for you and I will close out this ticket.


multiple screen_name(s)

Hi, i'm able to lookup a single screen_name by doing the following:
var params = {
screen_name: statData.votes[0].twitter_username
}
client.get('users/lookup', params, function(err, data, res) {
console.log(params);
if(err){
callback(err, null);
return
}
console.log("data.name: "+ data[0].name);
console.log("data.geo_enabled: "+ data[0].geo_enabled);
console.log("data.location: "+ data[0].location);
});
But i'm trying to work out how to make a single get request for up to 100 screen_name(s). or does this params var only accept a single screen_name ?
Ideally screen_name in the params var would be an array of up to 100 users, probably created dynamically in a loop, for instance
screen_name: statData.votes[i].twitter_username
Are you able advise?
Thanks in advance!
Paulie

@PaulieScanlon this is in fact supported by the API. See: https://dev.twitter.com/rest/reference/get/users/lookup.  Either via the screen_name or user_id properties.
For screen_name:

A comma separated list of screen names, up to 100 are allowed in a single request. You are strongly encouraged to use a POST for larger (up to 100 screen names) requests.

For user_id

A comma separated list of user IDs, up to 100 are allowed in a single request. You are strongly encouraged to use a POST for larger requests.


Fixes issue with using $include for individual array elements

Individual array elements could be included from external files prior to this by formatting the original file like so:
array:
  -
    $include: el0.yml
  -
    $include: el1.yml

A prettier format like the following would effectively be a no-op b/c the regex for matching the $include tag only allowed for it to be prefixed by white space:
array:
  - $include: el0.yml
  - $include: el1.yml

This change updates the regex to also allow for the include tag to be prefixed by '-', which allows for the nicer format above to work.

@francescobianco : can we get this PR merged please ?

Sorry by the later approval. Thanks for your support. Please give stars and ask for stars to your friends


dev and contrib requirements inconsistent

I just rebuilt my virtualenv and noticed an error about inconsistent requirements:
ERROR: qiskit 0.6.1 has requirement ply>=3.10, but you'll have ply 3.4 which is incompatible.

It seems that our contrib requirements explicitly call for ply 3.4 while dev requirements call for qiskit 0.6.* and thus implicitly for ply >=3.10.
I checked whether the tests in qasm_import (the contrib package that uses ply) pass with 3.10 and they do. Time to bump ply to 3.10?

If i recall, we decided on 3.4 to be compatible with google3.
CC: @balopat

Steps to reproduce, including tests sensitive to ply version:
$ pip install -r dev_tools/conf/pip-list-dev-tools.txt
[...forces ply>=3.10...]
$ pytest cirq/contrib/qasm_import/
[...tests pass...]
$ pip install -r cirq/contrib/contrib-requirements.txt
[...reverts back to ply=3.4 and shows the error...]
ERROR: qiskit 0.6.1 has requirement ply>=3.10, but you'll have ply 3.4 which is incompatible.
[...]
$ pytest cirq/contrib/qasm_import/
[...now tests fail...]

We chatted with @dstrain115 about this back in May: https://github.com/quantumlib/Cirq/pull/1613#discussion_r282248630
What do you think about ply>=3.4?
On the long run someone could take on upgrading the google3 ply version...

Yeah, I think we should change to "ply>=3.4" unless there's a good reason to pin to that exact version.

SGTM. And just to be a bit more specific: in cirq/contrib/qasm_import only the qiskit tests fail as latest qiskit requires latest ply. All the other functionality is compatible with latest ply versions.


Query Next Records URL

Typically when more than 2000 records are returned, Salesforce will provide a nextRecordsUrl parameter that shall be used to retrieve the next batch. query and queryAll functions now recursively query  until Salesforce tells that it's done.
https://developer.salesforce.com/docs/atlas.en-us.api_rest.meta/api_rest/dome_query.htm
query and query_all have been refactored into a single line function calling query_with which use either query or queryAll parameter.
async-recursion has been added to the crates which hides the implementation details (dyn Box Future etc.)
However along DeserializeOwned, the Send trait to ensure thread safety must now be added when querying, so this is a breaking change, albeit mostly transparent.

@0cv Thank you for your contribution!!
The automatic recursively query call is good idea.
But I think this change has the disadvantage in memory usage because query or query_all handle all collection data.
I want us to be able to select memory effective way (recursive call manually) or easy way (automatically) on rustforce.
What are your thoughts on that?

@tzmfreedom I don't have such use case, using this library (or similar) for ETL tools, or data loading. Memory is not a concern but keeping logic simple and clear is a concern to me.
If you really would like to retrieve granularly records one batch at a time, a user could just use the current get method, and then call the nextRecordsUrl whenever it fits - in a similar way that this PR is doing, but then adding custom logic between each iteration.
This being said, at the moment querying "select id from account" will return maximum 2000 records and that's definitely a bug, that this PR is addressing
This PR has an issue though, batches are retrieved in the wrong order, I can push a commit fixing this

Thanks @0cv and @tzmfreedom for working on this.
I'll chime in with a data point that I'd like the direct access to the batches.  Coming from other uses of the Salesforce API I'm expecting to need to handle it anyway.  And I'd rather a library like this not eagerly make an unbounded number of serial requests before reporting any status.  In particular, I don't want a spurious network error in a later batch prevent me from making progress on earlier batches.
In simple cases I can see it being helpful, so I support the helper being available as an option.

FWIW this is the patch I've been using myself:
index 7a4056b..11a630f 100644
--- a/src/client.rs
+++ b/src/client.rs
@@ -242,6 +242,18 @@ impl Client {
         }
     }

+    /// Fetch additional records from a previous query
+    pub async fn query_more<T: DeserializeOwned>(&self, next_records_url: &str) -> Result<QueryResponse<T>, Error> {
+        let url = format!("{}{}", self.instance_url.as_ref().unwrap(), next_records_url);
+        let res = self.get(url, vec![]).await?;
+
+        if res.status().is_success() {
+            Ok(res.json().await?)
+        } else {
+            Err(Error::ErrorResponses(res.json().await?))
+        }
+    }
+
     /// Find records using SOSL
     pub async fn search(&self, query: &str) -> Result<SearchResponse, Error> {
         let query_url = format!("{}/search/", self.base_path());
diff --git a/src/response.rs b/src/response.rs
index 83d3767..a39764f 100644
--- a/src/response.rs
+++ b/src/response.rs
@@ -8,6 +8,7 @@ use std::collections::HashMap;
 pub struct QueryResponse<T> {
     pub total_size: i32,
     pub done: bool,
+    pub next_records_url: Option<String>,
     pub records: Vec<T>,
 }

Thanks @0cv and @tzmfreedom for working on this.
I'll chime in with a data point that I'd like the direct access to the batches. Coming from other uses of the Salesforce API I'm expecting to need to handle it anyway. And I'd rather a library like this not eagerly make an unbounded number of serial requests before reporting any status. In particular, I don't want a spurious network error in a later batch to prevent me from making progress on earlier batches.
In simple cases I can see it being helpful, so I support the helper being available as an option.

if you want to count the number of records of your query you can do something like select count(id) from account. It will return one batch. And if you want to get the first X records of your query, you can use a limit, e.g. select id from account limit 200. In any case, this library implements an implicit limit which is to me an unexpected behaviour.
Closing this issue as there is no progress on it


pytorch min return min and min_indicies

Issue #, if available:
Description of changes:
pytorch min return min and min_indicies so only use the min
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.
Please tag this pr with at least one of these labels to make our release process faster: BREAKING, new feature, bug fix, other change, dev setup

Then I guess the first sentence and example in the docs is wrong? https://pytorch.org/docs/stable/generated/torch.min.html#torch.min

Oh I see, it returns a tuple if dim is provided? That’s a bit confusing…

right... but this code block is never used since its univariate emission model...


docs(releases): improve 2.0.0 release notes

Fix some format issues
Fix some linting issues
Add command to delete metrics-server TLS secrets, otherwise cert-manager complains
Add instructions about the new secrets needed for alertmanager's configuration

After the PR has been merged, I'll update also on GitHub the release.

A couple of minor suggestions but overall LGTM. :shipit:

TIL that the proper spelling is Alertmanager, that's an odd choice from upstream 😄


DELETE hostedEntry workflow endpoint 500 error

Describe the bug
500 error showing up on prod for DELETE /workflows/hostedEntry/... endpoint.
Possibly a side-effect of deleting one version after another.
INFO [2020-11-05 03:52:24,916] io.dockstore.webservice.helpers.statelisteners.ElasticListener: Performing index update with UPDATE.
INFO [2020-11-05 03:52:24,916] io.dockstore.webservice.helpers.statelisteners.ElasticListener: Could not perform the elastic search index update.
ERROR [2020-11-05 03:52:24,939] org.hibernate.internal.ExceptionMapperStandardImpl: HHH000346: Error during managed flush [Batch update returned unexpected row count from update [0]; actual row count: 0; expected: 1]
INFO [2020-11-05 03:52:24,940] org.hibernate.engine.jdbc.batch.internal.AbstractBatchImpl: HHH000010: On release of batch it still contained JDBC statements
... "DELETE /workflows/hostedEntry/15770?version=2 HTTP/1.0" 500 ...

Expected behavior
No 500 error and the deletion succeeds.
Additional context

Webservice version: 1.9.2
host location: dockstore.org

┆Issue is synchronized with this Jira Story
┆Issue Number: DOCK-1635
┆Sprint: Sprint 48- Walrus
┆Issue Type: Story

@NatalieEO @GFJHogue Is the verification to make sure deleting a frozen hosted version does not throw a 500? It looks like it from PR but not from the description.

@coverbeck Yeah it shouldn't throw a 500, but the deletion should fail with a better error message.

@NatalieEO Sorry, I wasn't clear -- I was asking if this only happens for frozen versions, or if there were other cases where a 500 might be returned.

Confirmed on staging that attempting to delete a snapshotted version fails with a 400.
Webservice - 1.10.0-rc.1
UI - 2.6.3-77-gf123e815
Compose Setup - release/1.10
Deploy - 1.10.0-rc0-2-gdb73433


"could not read package timestamp" when packing an environment for osx-64

Thanks for developing this package. I'm having this issue that I haven't been able to debug.
Steps to reproduce it are:
curl -O https://raw.githubusercontent.com/gerlero/openfoam-app/refs/heads/main/pixi.toml
pixi run pixi-pack pack --environment openfoam --platform osx-64 #Not an issue if --platform osx-arm64

Which invariably causes this error:
⏳ Downloading 31 packages...
Error: could not download package: could not read package timestamp

This problem first started with pixi-pack 0.2.0 (it didn't fail with 0.1.8).

We made the packs reproducible in 0.2.0 by setting the correct timestamp. I'll take a look into your issue, it seems to me that one of your packages have a bad timestamp set in the metadata.

The mpi-1.0-openmpi.tar.bz2 package on conda-forge doesn't have a timestamp field in pixi.lock.

This is a more minimal reproducer
[project]
channels = ["conda-forge"]
name = "no-timestamp"
platforms = ["osx-64"]

[dependencies]
mpi = { version = "1.0", build = "openmpi" }

version: 5
environments:
  default:
    channels:
    - url: https://conda.anaconda.org/conda-forge/
    packages:
      osx-64:
      - conda: https://conda.anaconda.org/conda-forge/osx-64/mpi-1.0-openmpi.tar.bz2
packages:
- kind: conda
  name: mpi
  version: '1.0'
  build: openmpi
  subdir: osx-64
  url: https://conda.anaconda.org/conda-forge/osx-64/mpi-1.0-openmpi.tar.bz2
  sha256: 1326b28195e8808cebc18a593f84c5cbd606826a150dd7e0365f11b86238b5df
  md5: 8c3bc725bf4d10fc6e56031f7543771f
  arch: x86_64
  platform: osx
  license: BSD 3-clause
  size: 4394

pixi-pack pack --platform osx-64

With #61, it should now look something like this:
⏳ Downloading 1 packages...
[00:00:00] ----------------------------------------
2024-10-25T09:15:02.479199Z ERROR pixi_pack::pack: could not read timestamp of Some("mpi-1.0-openmpi.tar.bz2"), using default
📦 Created pack at /Users/pavel/projects/pixi-pack/environment.tar with size 11.00 KiB.

it will still create a working pack but will complain along the way because it couldn't find a proper timestamp.
in order for the error message to go away, i would advise to update mpi to 1.0.1 as the missing timestamp is not an issue there (or rebuild mpi 1.0 to include the timestamp). for this, you need to adjust https://github.com/conda-forge/openmpi-feedstock/blob/fc1f3fafd2b6f56d9864f38bc226a4857cf29457/recipe/meta.yaml#L72 s.t. openmpi gets a newer version than mpi 1.0

Thanks!


How to capture the image of scanned barcode in vision barcode reader vision api

In camera source activity the take picture method is given but how to use it and how to take picture?
Where to call the take picture method and how to store and display the captured picture in Resulting Activity??
Please help me soon

I have same issue~ When i tap it doesn't capture

The current Mobile Vision Barcode API does not appear to allow the developer to access the bitmap image that was used to successfully scan the barcode. If this is indeed possible, please could you advise how it can be achieved? Otherwise please consider this as a feature enhancement of the API.
This requirement is especially relevant to payments that involve the scan of a barcode on a bill or invoice. The ability to retain the image as part of the payment audit trail would provide evidence of the origin of the payment details, in the event of a dispute (i.e. the barcode could be re-scanned).
New EU Payments legislation (PSD2) demands segregation between the display and initiation of payments. This is challenging to achieve on a mobile device. The retention of the barcode image may help meet these segregation requirements.

You can easily retain images in your application without any modification to the API itself.
For example, you can wrap BarcodeDetector in another Detector<Barcode> and supplement it with your own logic (e.g. if the results array is non-empty, start a background task to save the Frame data):
public class CapturingBarcodeDetector extends Detector<Barcode> {
  private BarcodeDetector mDetector;
  public CapturingBarcodeDetector(BarcodeDetector detector) {
    mDetector = detector;
  }

  [...]
  @Override
  public SparseArray<Barcode> detect(Frame frame) {
    SparseArray<Barcode> barcodes = mDetector.detect(frame);
    if (barcodes.size() > 0) {
      // Do whatever you like with frame.
    }
  }
}

Then attach your custom Detector to CameraSource and you're done.


use apt to install specific versions from repo

I think when a package repo is available, packages should be installed with the system package manager, not downloaded with curl  or get_url.
apt can install a specific package version with "apt-get install package_name=package_version", so I replaced the get_url and install local deb part with an apt call.
When setting "jenkins_version" to "2.60.1-1", for example, this results in
"apt-get install jenkins=2.60.1-1"
Same should be possible for dnf / yum, but I can't check this out right now.

It looks like this PR is failing the Jenkins version tests; the difference here is that if you use a specific jenkins_version with the way the role is now, it will pull that specific version down. If you use a version as part of the package specification, then it will try pulling from configured repos. That's a different method than the current curl/deb-based install method, and I'd rather keep supporting it the way it is since it's a little more flexible.


nfs mount at boot

How do I manage to mount nfs shares like the if-up.d/mountnfs script does with ifup?

You can start them with the /etc/dhcpcd-exit.hook script. See dhcpcd-run-hooks (8) for details.

As /etc/network/if-up.d/mountnfs seems very SysV init centric I decided to roll my own in the /etc/dhcpcd.exit-hook.  For others' reference I ended up with:
#!/bin/sh
# vim: tabstop=4 shiftwidth=4 expandtab smarttab smartindent wrapmargin=0 textwidth=0
#
# dhcpcd exit hooks
#
# NB: Not directly, run, but sourced, by /usr/lib/dhcpcd/dhcpcd-run-hooks
#     and that uses /bin/sh, which is usually dash on Debian.

# set -u

log() {
    printf "%s: $@\n" $(date --iso-8601=seconds) >> /var/lib/dhcpcd/tmp/etc-exit-hook.log
}
###########################################################################
# Ensure any NFS mounts are attempted *after* addresses and routing
# are in place.
###########################################################################
# First check if mounts are already in place.  *Any* active NFS mount counts
mount | grep -E '^([0-9\.]{7,15}|[:0-9a-f]{2-47}):\/[^ ]+ on \/[^ ]+ type nfs ' > /dev/null 2>&1
if [ $? -eq 1 ];
then
# No NFS mounts present
    log "$0: No NFS mounts present"
    # Check this hook run is for expected interface and stage
    # log "$0: interface is: ${interface}"
    if [ "${interface}" = "lan0" ];
    then
        # We don't want to trigger at all stages, only if an address and route
        # has been obtained and added.
        # log "$0: reason is: ${reason}"
        case "${reason}" in
            BOUND|REBIND|REBOOT|BOUND6|REBIND6|REBOOT6|INFORM6)
                log "$0: invoking restart of remote-fs.target"
                systemctl restart remote-fs.target
                ;;
        esac
    fi
# else
#    log "$0: NFS mounts already present"
fi
###########################################################################

This works for me, but it is assuming that the various other NFS services are run properly by systemd.
It would, of course, be better if dhcpcd could somehow communicate to systemd that addresses/routes have been added so it can correctly invoke the necessary targets for NFS mounts if applicable.

It would, of course, be better if dhcpcd could somehow communicate to systemd that addresses/routes have been added so it can correctly invoke the necessary targets for NFS mounts if applicable.

systemd could listen to addresses and routes being added to the kernel and respond accordingly yes?
dhcpcd does this and systemd likely does to so it's just getting systemd to behave? I don't see the need for any extra API beyond what the kernel offers.

For the benefit of others, it turns out that the solution is to have x-systemd.requires=network-online.target as part of the options on any NFS mount.  This causes the auto-created system target for that mount to have the specified Requires= as part of its configuration, so that it's only attempted after this is met.
There is the caveat that systemd seems to consider you online even if only one of IPv4 and IPv6 have an active ... I think it's default route.  In my use case I had been using explicit IPv4 addresses for the NFS mounts, but have now changed to a hostname because dhcpcd completes IPv6 setup first.
In my case I'm using the ISC 'KEA' dhcp server software for both DHCP and DHCPv6.

Found via: https://geraldonit.com/2023/02/25/auto-mount-nfs-share-using-systemd/


Null check operator used on a null value

nitesharma@Niteshs-MacBook-Pro monkoodog_panel_mobile % flutter pub run change_app_package_name:main com.monkoodog.panel
Deprecated. Use dart run instead.
Resolving dependencies...
Downloading packages...
Got dependencies.
Building package executable...
Built change_app_package_name:main.
Unhandled exception:
Null check operator used on a null value
#0      AndroidRenameSteps.process (package:change_app_package_name/android_rename_steps.dart:29:41)

You can use the edited version below.
change_app_package_name:
  git: https://github.com/Aksoyhlc/change_app_package_name.git

Hi @Aksoyhlc
Problem persists

Hi @Aksoyhlc
Problem persists

Hello, can you share the android/app/build.gradle file?

There seems to be an issue with build.gradle file.or maybe you created a project with flutter 3.22.0
Please update to the latest version and test again, otherwise you can reopen this issue with android/app/build.gradle  file contents for me to debug.
Thanks @Aksoyhlc for the assist


"Run on page load" doesn't work

I created a simple alert() to run on the client side and it didn't respond when loading the page, then I created a simple return {notify: 'any text',}; to return from the server, and nothing...

Thanks, I didn't think about the result of the action when implementing this. This is a little bit more complicated but it should be possible to implement this. Currently, for instance table modifications should work, but you won't see the results returned (and as this is how the client-side js actions are implemented, that doesn't work either)

@glutamate It would be very good to be able to implement it immediately, since it occurs to me that it is a good way to be able to restrict access to certain users who do not meet a certain condition, such as a double security filter, for example for an eLearning application, in which that certain users who enroll with administrator approval can enter certain classrooms. And in the event that the access link is shared between users because they cannot get it by their own means, it does not allow them to enter either and redirects them to the beginning. And such a check could be executed with this automatic call to action.
I don't know if there is any other solution nowadays for this

implemented in #1850


document known issue for vsan [#164664689]

Signed-off-by: Huan Wang<EMAIL_ADDRESS>

@kbakas-piv looks good to me

Thanks, @farmermel!


Add Climakitae

Insert URLs to the project here:
https://github.com/cal-adapt/climakitae

[x] The projects is active, documented, open source licensed, shows usage from external parties and is directly targeting environmental sustainability. Find more details in the Contribution Guide.

All issues labeled as 'Good First Issue' of the project listed on OpenSustain.tech will be visible on ClimateTriage.com. This is a great way to welcome new community members to your project.

@nicolejkeeney @elehmer @vicford


Make it compile with idris v0.4.0

Hi,
in order to make it compile with version 0.4.0 I had to make some small changes.
I am not sure whether those are correct. Just ignore this PR if it doesn't make sense to you.
Regarding the removal of freeBuffer I found https://github.com/idris-lang/Idris2/pull/1538 and
regarding the IntX types this one https://github.com/idris-lang/Idris2/pull/1437

Thanks for fixing it! All those changes look correct. If you spot any other improvements feel free to create a PR (I'm not currently actively devoloping this but I'm happy to accept PRs).
P.S. Do you know how to make releases/tags?


Ability to protect branch from pull request

When a repo's specific branch is protected, I can still first fork it, and then commit to the new repo, and finally merge the changes to the original repo and the protected branch. The branch is not protected.

I would really like this feature. The protect branch functionality is only half worthewhile when you can get around it by creating a Pull Request and merging it even without permission on the branch it's being merged to.

I was just going to request this - it'd be really useful :)

Hi! I noticed this issue which to me is very important. Since this seems to have been reported a long time ago, is there any update or a workaround?


Button Press doesn't work anymore

Hello!
Ever since (I think) HA 2024.5.x the button press on the doorbell doesn't register anymore. Maybe more isn't working anymore either, but I'm only using the cameras (still works!) and the doorbell press event.
Would be amazing if this could be fixed!
HA: 2024.5.x
Doorbell: DHI-VTO3211D-P4-S2

I also enabled debugging as well as monitoring events - nothing coming up at all...

Did the same thing. Didn't see any events firing after button press.

Found these errors in the HA log file
2024-05-30 12:58:14.381 WARNING (MainThread) [homeassistant.helpers.frame] Detected that custom integration 'dahua' callsasync_add_job, which is deprecated and will be removed in Home Assistant 2025.4; Please review https://developers.home-assistant.io/blog/2024/03/13/deprecate_add_run_job for replacement options at custom_components/dahua/__init__.py, line 85: hass.async_add_job(, please create a bug report at https://github.com/rroller/dahua/issues 2024-05-30 12:58:14.427 WARNING (MainThread) [homeassistant.components.light] Entity None (<class 'custom_components.dahua.light.DahuaInfraredLight'>) is using deprecated supported features values which will be removed in HA Core 2025.1. Instead it should use <LightEntityFeature: 1> and color modes, please create a bug report at https://github.com/rroller/dahua/issues and reference https://developers.home-assistant.io/blog/2023/12/28/support-feature-magic-numbers-deprecation 2024-05-30 12:58:14.428 WARNING (MainThread) [homeassistant.components.light] None (<class 'custom_components.dahua.light.DahuaInfraredLight'>) does not set supported color modes, this will stop working in Home Assistant Core 2025.3, please create a bug report at https://github.com/rroller/dahua/issues

Anyone get button press working?  I'm using an EmpireTech DB2X-WP.  Button events not going through.  Happy to toss @rroller a coffee to fix. :-)

I'm so sorry everyone. I've been busy and not able to dig into this. I will make it a top priority soon. I'll try for this weekend

Not a problem, I know it's voluntary.  Figured a bribe may help.
Events and logs aren't helpful aside from the logs stating that the add-on isn't compliant with recently depreciated and removed Core changes.
I enabled SIP Server on my doorbell.  It's pushing events again.  Didn't need it before.

@derekcentrico Facing the same issue here but my doorbell has always had the SIP server enabled. Did you change anyhing else?

@lluiscab I disabled/cleared the SIP settings (basically a factory reset on device so that cleared) and re-enabled with the SIP server on the local doorbell if that helps.  Not sure at which point this made it function but SIP running on the local doorbell makes it work.  If I disable it it stops working.

Can confirm Disabling SIP stops button press from working on mine - VTO2211G-WP.
I may be a numpty but in the log when the button is pressed i get "Doorbell Button Pressed detected sound" and then 30 seconds later i get "Doorbell Button Pressed cleared (no sound detected)"
Is that normal behavior?
I only just got the button press working so am unsure.

Since the change to schedule_update_ha_state the integration is now using a hard fix value of 30 seconds to poll the VTO for any updates.  This can be changed by updating SCAN_INTERVAL_SECONDS = timedelta(seconds=30) under init.py. I updated this setting to 2 seconds and the doorbell press works faster. Home Assistance is now polling the doorbell every 2 seconds instead of waiting for the VTO to push a message.

So far I can't reproduce this. I have a VTO2202F, I press the button and I immediately get a button press event.
I do have SIP enabled...

Testing now with it disabled...

After updating to Home Assistant 2024.6.1 (not sure if that had anything to do with it) and restarting the VTO, I'm receiving the button press event again. Regardless of what the real cause for this issue might be, it feels like there's some error checking / reconnecting logic missing from this component in order to handle this kind of issues more smoothly, maybe we could get a "Connection to VTO lost" notification or similar.

Can confirm, the button press doesn't work when SIP is disabled.

https://github.com/rroller/dahua/releases/tag/0.9.55 released to fix a few of the deprecation warnings above. I'm not able to apply a fix for the button press as I've not been able to reproduce the issue yet

To make things more complicated:

I was on a very old (manual) version: version 0.50.0
SIP Server is enabled (always was, not sure what it does, though)
I rebooted the device and hopefully this will help
I updated to the latest version: 0.9.55. But since rrollerr states it hasn't been able to reproduce it yet, I'm not getting my hopes up :(

but of course, thanks to rrollerr for this integration! I'm very grateful for it!

Guys, I don't know about you, but after installing HA 2024.06.2 or 2024.06.03 (I installed them at the same time, so not sure which one), I suddenly get buttonpress-events again!
For me, the problem seems to be solved! Was an HA issue afterall, it seems.
Can anybody confirm?

Guys, I don't know about you, but after installing HA 2024.06.2 or 2024.06.03 (I installed them at the same time, so not sure which one), I suddenly get buttonpress-events again!
For me, the problem seems to be solved! Was an HA issue afterall, it seems.
Can anybody confirm?

Still borked on my install. See my post above yours.

Guys, I don't know about you, but after installing HA 2024.06.2 or 2024.06.03 (I installed them at the same time, so not sure which one), I suddenly get buttonpress-events again!
For me, the problem seems to be solved! Was an HA issue afterall, it seems.
Can anybody confirm?

Still borked on my install. See my post above yours.

That post was from 2 weeks ago. Did you upgrade your HA to the latest 2024.6.3? That's what apparently fixed it for me.

I updated and still not working. Gobe back to the dahua vto integration instead which isn't as good but at least it works with button press


Support year in date formatting

[ ] #8

I see, then the function makes sense, definitely.

Mika's question suggests the need to clarify in the docstring that ds_frequency confirms with the CMIP6 CV for frequency: https://github.com/WCRP-CMIP/CMIP6_CVs/blob/master/CMIP6_frequency.json

Mika's question suggests the need to clarify in the docstring that ds_frequency confirms with the CMIP6 CV for frequency: https://github.com/WCRP-CMIP/CMIP6_CVs/blob/master/CMIP6_frequency.json

Yes, but it's actually not that CV, but this one https://github.com/PCMDI/input4MIPs-cmor-tables/blob/master/input4MIPs_frequency.json
I will speak to Paul about this and then propagate back. I've made #10 as a reminder


Can't connect pathpoints to each other or characters anymore.

To make AI walk around the level OG uses pathpoint objects. These objects can be placed and moved in the level to lay out a path for characters. The connecting no longer works in the FOSS version of the engine.
Using 1500b7bd19ae685e84423648ddf4404bcdfda7ff
With Alt + LMB with a pathpoint selected you should be able to connect it to a character or another pathpoint. But this no longer works.

Should be fixed by 0256b3f30badd87a9450c329bd84d78fcf45394e


sim store integration

Preliminary PR with initial changes.
Summary of changes up to this point.

Removes all uses of Arrow files
Add several getter functions to different places
Add several fields to some of the store objects.
Initial approach to the simulation results object
addresses some deprecations in the use of HDF5
Makes some changes in the stages and simulations to have a cleaner execution process.
implements graceful exit infeasibilities are present.

The simulation store changes reduced the simulation times significantly. Now the most amount of time is spent in the stage updates

@daniel-thom This PR is passing the tests. There are still pending items but it should enable the next step of work to continue when merged

I will have to add more tests before merging into master, but the current ones cover the basic cases we need to continue development.


nixos/dave: init

Motivation for this change
WIP module for the dave WebDAV server.
Uses /run/dave as a working directory as dave requires the config file to be in its working directory. Maybe there is a more appropriate solution?
TODO:

[x] See if there is a better workaround for the config.yaml problem (ask upstream for ability to specify config location?)
[x] Harden systemd service
[ ] (Maybe) introduce a high-level option to automatically configure TLS
[x] Add release notes

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Alternatively you might consider this webdav implementation which looks pretty solid.

I could have sworn I looked for something like that when I started using dave, but then I saw it was only merged a week ago. Guess I'm not losing my mind :). I am already using dave via container, it works well for me, and is dead easy to set up, so I would like to continue with this module if it's not completely redundant.

No problem at all, was just mentioning in case you found it interesting 😄

Now we wait for upstream to respond. In the meantime I'll look into syslog.target.

Opened an issue re:syslog.target at #149721

Upstream is looking for new maintainers, I don't think it makes sense to continue to work on this PR until that is resolved.

Upstream is now officially unmaintained.


Update PR Template

[X] The commits are squashed in a cohesive manner and have meaningful messages.
[X] For commits that came from upstream, [UPSTREAM] has been prepended to the commit message
[ ] JIRA link(s):
[ ] The Jira story is acked
[ ] Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious)
[X] The developer has manually tested the changes and verified that the changes work.

/lgtm


Feature Request for organization in the activities selection on the Record Workout Form.

Currently, all roughly ~40 activities options are unordered. I believe it would be a QOL improvement to organize them, perhaps alphabetically, to help users find the appropriate activity.

@dovid11564
PR#158 should address this. Please take a look.


Question (self-hosted): is it possible to limit / configure the feed refresh rate to avoid IP blocking?

Hi,
I'm self hosting my instance and I have a lot of ArtStation feeds, so every day I get blocked. Luckly for me I have dynamic IP so it fixes automatically every so often.
But it would be great if I could set a refresh rate to avoid the issue.
Thanks!

Answering myself: I found the setting
 # minimum amount of time commafeed will wait before refreshing the same feed
 refreshIntervalMinutes: 5

I will try setting up a higher value.


question: will there be a custom resolution option added to this?

title says it all, would like to know

Widescreen support checkbox does add your monitors current resolution to the game. What is your specific use case for a custom resolution?

When I tick widescreen support, the game still letterboxes at 2560x1440 instead of 5120x1440x

When I tick widescreen support, the game still letterboxes at 2560x1440 instead of 5120x1440x

That sounds like you are using two 2560x1440 Monitors?

When I tick widescreen support, the game still letterboxes at 2560x1440 instead of 5120x1440x

That sounds like you are using two 2560x1440 Monitors?

It’s a 32:9 monitor. One panel.

Widescreen support checkbox does add your monitors current resolution to the game. What is your specific use case for a custom resolution?

can you tell me what the limit to that is? if i have a 1080p monitor but i my desktops resolution would it add that or am i getting this wrong


Error Messages running optimizer.ipynb -> Problem?

I ran your optimizer and even got an output, which seems quite reasonable. However I noticed, that I see this Error Message. I'm not sure, if I did something wrong or if this affects my optimizer.w parameters.
Question: Did I do something wrong?

ERROR: pip's dependency resolver does not currently take into account all the packages that are installed. This behaviour is the source of the following dependency conflicts.
lida 0.0.10 requires fastapi, which is not installed.
lida 0.0.10 requires kaleido, which is not installed.
lida 0.0.10 requires python-multipart, which is not installed.
lida 0.0.10 requires uvicorn, which is not installed.
plotnine 0.12.4 requires numpy>=1.23.0, but you have numpy 1.22.4 which is incompatible.
tensorflow 2.15.0 requires numpy<2.0.0,>=1.23.5, but you have numpy 1.22.4 which is incompatible.

Sorry, but this fork is not compatible with optimizer, as it changes the formula for calculating next interval (next stability). The approach of this app is to set a number of weights that fits your desired behavior for your schedule (using the Geogebra links in the README page to see the effect of each change), and not having to rely entirely in the algorithm or the optimizer to pursuit a given retention rate.


BinaryTreeSearch: too many pulls, too many pushes?

I'm checking right now why BinaryTreeSearch_true does not work, and somewhere in between source code and cfg strange things seem to happen. The code is:
                    if (curr.right != null) {       // line 81
                        curr = curr.right;          // line 82
                    } else {
                        curr.right = new Node(v, null, null);
                        break;
                    }

and the resulting cfg is:
Block6:
(ln 81) r7_value_03, r7_left_3254, r7_right_3265, r7_$dynamicType_5526, lastpush := pull(BinaryTreeSearch_true$BinaryTree$Node, r7)
(ln 81) assume (((((((((lastpush == 19) || (lastpush == 5)) || (lastpush == 23)) || (lastpush == 6)) || (lastpush == 7)) || (lastpush == 8)) || (lastpush == 9)) || (lastpush == 10)) || (lastpush == 11))
(ln 81) $r4 := r7_right_3265
    goto:
      if ($r4 == $null$): Block7
      if (!($r4 == $null$)): Block8
Block8:
(ln 82) r7_value_03, r7_left_3254, r7_right_3265, r7_$dynamicType_5526, lastpush := pull(BinaryTreeSearch_true$BinaryTree$Node, r7)
(ln 82) assume (((((((((lastpush == 19) || (lastpush == 5)) || (lastpush == 23)) || (lastpush == 6)) || (lastpush == 7)) || (lastpush == 8)) || (lastpush == 9)) || (lastpush == 10)) || (lastpush == 11))
(ln 82) r7 := r7_right_3265
(ln 82) lastpush := 7
(ln 82) push(BinaryTreeSearch_true$BinaryTree$Node, r7, [r7_value_03, r7_left_3254, r7_right_3265, r7_$dynamicType_5526, lastpush])
    goto:
      Block4

Two questions:

why are the pulls in Block6 and Block8 not merged? Block8 only has a single incoming edge; the fact that we have two separate pulls implies that we don't know that curr != null after line 82, which in the end leads to the spurious NullPointerException
why is there a push in Block8? curr is a local variable, so the assignment curr = curr.right should not lead to any pushes. In fact the push looks unsound, since this will copy the values of fields from curr to curr.right, which is not what the assignment actually does

Question 1:
They are not merged because of a restriction on moving push/pull across the CFG. Before moving e.g. the pull at the start of Block 8 up in the CFG, into Block 6, it is checked whether there is a path through Block 6 that leads to Block 8. There is, through 6, 4 and 10. It is then not moved, to prevent moving statements around in circles.
If we can identify loop headers, then I think we can relax this restriction to not moving past loop headers, but allowing moves inside the loop. That would allow to remove the extra pull in this example. @martinschaef do we have any code to detect loop headers somewhere? I know there's some graph library that we use, which can probably detect dominators etc?

Question 2:
What we do is add a pull-push pair around every field access (read or write), then apply the simplification rules to remove unneeded ones. There is currently no rule that says that in:
a1, a2 = pull(A, a);
..code..
push(a, [a1,a2]);

if ..code.. does not update the value of a1 or a2, the push can be removed. We can add that, but it will need to go over all statements in between all pull's and push's of objects of the same (or sub/super) class and see if they may alias. So it will be fairly inefficient.
As for the unsoundness, I think you're right. We are copying the values from curr into curr.right instead of vice versa. I need to think about this some more. There is a mapping from locals to fields where we keep track of copies etc, we should probably do something smarter there when these are of reference-type.

Concerning 1:
I see. It is not even necessary to compute loop heads though to prevent cycles. One can simple compute the distance of each block from the entry point (say, using Bellman-Ford), and then only allow pulls to move closer to the entry, and pushes to move further away

Concerning 2:
We can easily optimise by not adding a push in case of reads in the first place?
But I think the unsoundness problem has a different reason: we have to make sure that the reference a is not modified in code, regardless of whether we are handling reads or writes. Otherwise you will push back to a different object than you pulled from.

@rodykersten all graph utils are in soottocfg.cfg.util. In particular there is a class LoopFinder. That should have everything you need.

Thanks, but I already implemented it as @pruemmer suggested. Pushing shortly..

First two points are implemented in my last commit.

Thanks!
For BinaryTreeSearch, I now observe the following problem:
Method <BinaryTreeSearch_true: void main(JayArray_java_lang_String)>($in_0)
[...]
Block2:
(ln 123)	lastpush := 13
(ln 123)	push(JayArray_java_lang_String, r0, [undef_r0_0, undef_r0_1, JayArray_java_lang_String, undef_r0_2, undef_r0_3, undef_r0_4, undef_r0_5, undef_r0_6, lastpush])
(ln 123)	r0_$length_5810, r0_$elType_5821, r0_$dynamicType_5832, r0_atIndex_0_5843, r0_atIndex_1_5854, r0_atIndex_2_5865, r0_atIndex_3_5876, r0_atIndex_4_5887, lastpush := pull(JayArray_java_lang_String, r0)
(ln 123)	assume (lastpush == 13)
(ln 123)	$helper8 := r0_$length_5810
(ln 123)	$helper9 := 0
(ln 123)	$assert_13 := ($helper9 < $helper8)
(ln 123)	assert $assert_13

I can see two problems here: (1) why are the push and pull not merged? and (2) why do we push undef_ values? they seem to be the reason why we get an ArrayIndexOutOfBound exception one line later

The push is added by the MissingPushAdder. This class adds pushes for pulls that do not have an associated push yet. This is the case e.g. for objects that are returned by a library method or as in this case, are program input. If we don't add that push, the pull becomes an assume(false). You'll see that if you remove that one, you also get SAFE for the _false versions of BinaryTreeSearch.
Note that adding a push for any object returned by a library call is not sound in general, as it could be null or have fields that are uninitialized. However, if we do not assume that these objects and their fields are initialized, it will be practically impossible to verify programs with library calls. We may want to add an option for this in the future.

I see; but in this case, the additional push comes after some pull from the same object?

I don't see where that happens here? What should happen on a write to a field a.f is that first we add a pull before and a push after the field access. Then, if there is no push that can affect that pull, we create one with unbound variables. So then we'll have push(a, undef...); a = pull(), a.f = y; push(a, y, ...);. After that we apply the simplification rules, which may or may not remove some of these.

For the latest JayHorn, I get the following cfg; the push in Block2 should not be necessary, since there is already a push to the same object in Block1?
[...]

The duplicated push is no longer there. This is currently failing because of #126 though.

This now passes, closing.


Two Way SSL configuration for HTTP Binding

We are using dapr HTTP binding for all the external calls in our application. Our external application needs two-way SSL for communication in one of our use cases.
I wanted to check if it's supported right now and how to configure it.
cc : @rahulopengts @akhilac1 @mukundansundar @pravinpushkar

Moving to the components-contrib repo. Will need an option to specify what client cert to present


Customizable line limit

I use 70 instead of 50 for my line limit. Is there a way to customize this with this plugin?

This parser does not allows to customize line length limit. This value comes from the official git documentation (https://git-scm.com/docs/git-commit#_discussion) and from the old vim syntax highlighting.
If you're using nvim-treesitter, I'm quite you can adapt your highlighting with custom queries.


Spec: Appearance configuration objects for profiles

Summary of the Pull Request
Spec for #3062

PR Checklist

[ ] Closes #xxx
[x] CLA signed. If not, go over here and sign the CLA
[ ] Tests added/passed
[x] Is documentation
[ ] Schema updated.
[x] I work here

Detailed Description of the Pull Request / Additional comments
Read the spec

I have a computer virus
Jimmy R.
CONFIDENTIALITY: The information contained in this transmission may contain privileged and confidential information. It is intended only for the use of the person(s) named above. If you are not the intended recipient, you are hereby notified that any review, dissemination, distribution or duplication of this communication, and the information contained in it, is strictly prohibited. If you are not the intended recipient, please contact the sender and immediately destroy all copies of the original message.
On Thursday, December 3, 2020, 11:35 AM, Mike Griese<EMAIL_ADDRESS>wrote:
@zadjii-msft commented on this pull request.
In doc/specs/Configuration object for profiles.md:

+This is another location in the settings where parsing/loading the settings may fail. However, this is the case
+for any new setting we add so I would say that this is a reasonable cost for this feature.

+### Compatibility
+
+Should not affect compatibility.
+
+### Performance, Power, and Efficiency
+
+## Potential Issues
+
+Inactive tabs will be 'rendered' in the background with the UnfocusedRenderingParams object, we need to make
+sure that switching to an inactive tab (and so causing the renderer to update with the 'normal' parameters)
+does not cause the window to flash/show a jarring indicator that the rendering values changed.
+
+## Future considerations
Yea, I don't think this should conflict at all.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

Alright so I've thought on this for the weekend, and had a thought. I know that I was the one really pushing for the elevated config states, but this whole proposal would be like, 1000% simpler if we didn't concern ourselves with that. We couldn't find prior art for this, so maybe it's okay to just not do it?
We'll have the shield we're adding for #1939 to indicate a window is elevated. Maybe that's good enough for now. If we don't bother with the combinatorial logic of having multiple states, and only worry about unfocusedAppearance, then this whole spec is really straightforward.
Maybe I can loop back on having a separate elevated appearance post-#4000. Someone else mentioned doing this as an extension and I think I'm coming around on that idea - then we can make the logic whatever silly thing we want, and not have it officially supported, since it'll be coming from @zadjii not @zadjii-msft.
Thoughts?

Maybe I can loop back on having a separate elevated appearance post-#4000.

I like this idea of coming back to the 'elevated appearance' later - hopefully by then we have some idea of whether there are other states that users really want.

just gotta merge main!


Add a BigInt option to recs Type

Currently you can use Type.String and defineStringComponent to handle large numbers like uint256s client-side, but this means that you have to convert them to bigints any time you want to do arithmetic with them, and then convert them back to a string if you want to save the new value. Having the ability to get and set bigints directly to/from the component would be nicer.

done in https://github.com/latticexyz/mud/commit/29fefae43d96b107a66b9fd365b566cb8c466f8b


JupyterLab does not prompt for permission to control other applications using xlwings on M2 mac

Description

I am experiencing an issue where JupyterLab does not prompt for permission to control other applications when using xlwings on my M2 macbook pro. This issue prevents me from accessing and modifying other files through xlwings within JupyterLab. This issue does not occur, when using VS Code, which prompts the permission and allows to write "Hello world" in the first cell in excel.

Reproduce

To reproduce the issue, please follow these steps:

Launch JupyterLab on macOS.
Execute the following code snippet in a Jupyter notebook:

import xlwings as xw

# Start Excel
app = xw.App()

# Create a new workbook
wb = xw.Book()

# Add some data to the first cell of the first sheet
wb.sheets[0]['A1'].value = "Hello, world!"

# Save and close
wb.save('Test.xlsx')
wb.close()

# Quit Excel
app.quit()

Context

Operating System and version: Ventura 13.4.1 (M2 Pro)
JupyterLab-Desktop version:3.6.1
Installer: dmg

Troubleshoot Output

CommandError: Command failed:
		OSERROR: -1743
		MESSAGE: The user has declined permission.
		COMMAND: app(pid=4029).count(each=k.workbook)

Browser Output (required for interface issues after loading)

Paste the output from the DevTools JavaScript console here, if applicable.

Logs (required for issues with loading, server and startup)

Paste the logs from the `main.log` file here (optionally also from `renderer.log`).

Related issue: #667 (JupyterLab Desktop doesn't ask for permission to access the camera).


V0 10 1

Prepare next version.

Kept as draft as I encountered strange bugs related to TLS validity date: start date in the future. Probably related to a timezone issue or clock setting at boot time.


Added support for variables in versions

Added support for variables in versions:
<PackageReference Include="TestPackage" Version="$(PackageVersion)"/>

Thank you!


Disable action bar home button on main activity

Tidies up the UI slightly by removing the back arrow from the top level activity and prevents the display from flickering or "looping" when the home button is spammed.

Looks good.


Make python{2,3} version explicit

Motivation for this change
Changes for issue #18185
Things done

[ ] Tested using sandboxing
Built on platform(s)

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

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

@GrahamcOfBorg build searx

Success on x86_64-linux (full log)
Attempted: searx
Partial log (click to expand)
  Consider adding this directory to PATH or, if you prefer to suppress this warning, use --no-warn-script-location.
Successfully installed searx-0.14.0
/build/source
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/p3b982gscfmrs7hzcyw9ayv4ayw9zgxb-searx-0.14.0
strip is /nix/store/1hi76hr87bd1y1q1qjk0lv8nmcjip1c8-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/p3b982gscfmrs7hzcyw9ayv4ayw9zgxb-searx-0.14.0/lib  /nix/store/p3b982gscfmrs7hzcyw9ayv4ayw9zgxb-searx-0.14.0/bin
patching script interpreter paths in /nix/store/p3b982gscfmrs7hzcyw9ayv4ayw9zgxb-searx-0.14.0
checking for references to /build in /nix/store/p3b982gscfmrs7hzcyw9ayv4ayw9zgxb-searx-0.14.0...
wrapping `/nix/store/p3b982gscfmrs7hzcyw9ayv4ayw9zgxb-searx-0.14.0/bin/searx-run'...

Success on aarch64-linux (full log)
Attempted: searx
Partial log (click to expand)
Successfully installed searx-0.14.0
/build/source
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/xs454pbyzb2rqbjj518vvybwavk8l4g1-searx-0.14.0
strip is /nix/store/zrs21zqcchgyabjf4xfimncdq16njizc-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/xs454pbyzb2rqbjj518vvybwavk8l4g1-searx-0.14.0/lib  /nix/store/xs454pbyzb2rqbjj518vvybwavk8l4g1-searx-0.14.0/bin
patching script interpreter paths in /nix/store/xs454pbyzb2rqbjj518vvybwavk8l4g1-searx-0.14.0
checking for references to /build in /nix/store/xs454pbyzb2rqbjj518vvybwavk8l4g1-searx-0.14.0...
wrapping `/nix/store/xs454pbyzb2rqbjj518vvybwavk8l4g1-searx-0.14.0/bin/searx-run'...
/nix/store/xs454pbyzb2rqbjj518vvybwavk8l4g1-searx-0.14.0

Please drop 744d9d0 from this PR, I've included it in https://github.com/NixOS/nixpkgs/pull/44398.

@dotlambda Ok, I removed the bluez commit.

Thank you @dotlambda.


update readme to clarify for first timers

I got the picture reading the issues but as a first timer to semantic ui this was still unclear after reading through the readme.

thank you


STAT command truncates # good or # bad at 8 digits

Reported by ansley b manke on 5 Mar 2014 00:18 UTC
Keith Dixon ran into this. He has a variable on a 194 x 114 grid with 30 years of daily data (10958 time levels) for a total of 242,347,128 data points.
He used STAT to see how many good and bad data were present in a result (one could use @NGD here instead).  His result of STAT shows:
Total # of data points: 24234712<PHONE_NUMBER>581*1)
flagged as bad  data: 24234712
it should be...
Total # of data points: 242347128<PHONE_NUMBER>581*1)
flagged as bad  data: 242347125
                             ^

Here's a script we can run:
yes? use etopo5
yes? set mem/siz=2400

yes? define axis/t=1:50:1 tax
yes? let roseyt = if rose le -7000 then rose +0*t[! This is the number of points in the grid:
yes? list/prec=10 `rose,return=isize`*`rose,return=jsize`*50
 !-> list/prec=10 4320*2161*50
             VARIABLE : 4320*2161*50
          466776000.0

yes? stat roseyt

             IF ROSE LE -7000 THEN ROSE +0*T[GT=TAX](gt=tax]

yes?)
             LONGITUDE: 0.04W(-0.04) to 0.04W(360)
             LATITUDE: 90S to 90N
             Z:  N/A
             T: 0.5 to 50.5
             E:  N/A
             F:  N/A
             DATA SET: /home/users/tmap/ferret/linux/fer_dsets/data/etopo5.cdf

 Total # of data points: 46677600  (4320*2161*1*50*1*1)
 # flagged as bad  data: 46651585
 Minimum value: -10376
 Maximum value: -7000
 Mean    value: -7722.8 (unweighted average)
 Standard deviation: 630.84

! should be
 Total # of data points: 466776000 (4320*2161*1*50*1*1)
 # flagged as bad  data: 466515850

Migrated-From: http://dunkel.pmel.noaa.gov/trac/ferret/ticket/2148

Comment by ansley.b.manke on 5 Mar 2014 17:59 UTC
Fixed in var_stat.F. We compute NGOOD, NBAD, NTOT points.  Then NTOT and NBAD get formatted into output strings with calls to LEFINT.  Inside LEFINT, the number is written to a string using an integer format, and blanks removed to left-justify the result. Inside LEFINT, everything's done with a max of 16 digits, so that's all good.
In var_stat, LEFINT was declared as only CHARACTER*8, and that's where the truncation happened.

Sorry to bother you, would you mind not using the @ syntax in these messages, everytime you write @ngd it sends me an email.