[cbsnews.com] change in site, no longer able to extract videos

Please follow the guide below

You will be asked some questions and requested to provide some information, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your issue (like that [x])
Use Preview tab to see how your issue will actually look like

Make sure you are using the latest version: run youtube-dl --version and ensure your version is 2017.05.29. If it's not read this FAQ entry and update. Issues with outdated version will be rejected.

[X ] I've verified and I assure that I'm running youtube-dl 2017.05.29

Before submitting an issue make sure you have:

[X ] At least skimmed through README and most notably FAQ and BUGS sections
[X ] Searched the bugtracker for similar issues including closed ones

What is the purpose of your issue?

[X ] Bug report (encountered problems with youtube-dl)
[ ] Site support request (request for adding support for a new site)
[ ] Feature request (request for a new functionality)
[ ] Question
[ ] Other

The following sections concretize particular purposed issues, you can erase any section (the contents between triple ---) not applicable to your issue

If the purpose of this issue is a bug report, site support request or you are not completely sure provide the full verbose output as follows:
Add -v flag to your command line you run youtube-dl with, copy the whole output and insert it here. It should look similar to one below (replace it with your log inserted between triple ```):
debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: [u'http://www.cbsnews.com/news/maria-ridulph-murder-will-the-nations-oldest-cold-case-to-go-to-trial-ever-get-solved/', u'-v']
[debug] Encodings: locale UTF-8, fs UTF-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2017.05.29
[debug] Python version 2.7.9 - Linux-2.6.32-042stab116.2-i686-with-debian-8.3
[debug] exe versions: ffmpeg 2.8.6-1, ffprobe 2.8.6-1, rtmpdump 2.4
[debug] Proxy map: {}
ERROR: Unable to extract playlist JSON info; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
  File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 760, in extract_info
    ie_result = ie.extract(url)
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 433, in extract
    ie_result = self._real_extract(url)
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/cbsnews.py", line 71, in _real_extract
    'playlist JSON info', group='json'), video_id)['state']
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 782, in _search_regex
    raise RegexNotFoundError('Unable to extract %s' % _name)
RegexNotFoundError: Unable to extract playlist JSON info; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

---

### If the purpose of this *issue* is a *site support request* please provide all kinds of example URLs support for which should be included (replace following example URLs by **yours**):
- Single video: https://www.youtube.com/watch?v=BaW_jenozKc
- Single video: https://youtu.be/BaW_jenozKc
- Playlist: https://www.youtube.com/playlist?list=PL4lCao7KL_QFVb7Iudeipvc2BCavECqzc

Note that **youtube-dl does not support sites dedicated to [copyright infringement](https://github.com/rg3/youtube-dl#can-you-add-support-for-this-anime-video-site-or-site-which-shows-current-movies-for-free)**. In order for site support request to be accepted all provided example URLs should not violate any copyrights.

---

### Description of your *issue*, suggested solution and other information

cbsnews.com worked last week.

Hi @ddurdle,
Apparently the source html has changed for the news pages.  I will submit a new PR shortly that should fix the problem.
Cheers,
Parmjit V.


cannot stop intro popups

Whenever I try to use the admin frontend the intro help pops up and either I skip or see all of them it pops up again when I click on any link of the sidebar. Am I missing anything?

Why is it that I still get this issue on v2.3.6? is it a regression? I have tried on 3 differnet envs and I get the same issue, but it suddenly disappeared on two of my envs (dev and staging) but not on prod.

@theil I test many times without success:

check this video for local testing:
http://www.screencast.com/t/skxnsPJj1
also I tested in camaleon server:
http://camaleon.tuzitio.com/plugins/demo_manage/
Could you please see if there is a problem saving for this in your logs?

Regards,

Owen thanks for the replay, actually it was our fault. The ajax call was being sent to the wrong server because of nginx. Anyway keep on the good work, and once again thanks.


When creating a product, getting console error in Number Validation when attribute created.

Terms

[X] Before you openning this issue, i have checked if the issue has already been reported.

Bagisto Version(s) affected
Master
Issue Description
When creating a product, getting console error in Number Validation when attribute created.
Created a attribute of Text type and there Input Validations is added for numbers.
After saving the attribute and assigning it to the attribute family.
When we create a product, check console and there we can see that the error is appearing.
Preconditions

Admin Must Be Logged In.

Steps to reproduce

Go to Admin -> Catalog -> Attributes -> Create attribute -> Select Type -> Add validation -> Number.
Go to Attribute Family -> Add attribuite -> Save
Go to Catalog -> Products -> Create Product -> Open Console -> Check errors.

Expected Result
There should be no console errors appearing.
Actual Result
Currently we can see the errors appearing.
Check the attribute screenshot here
Check the product edit page screenshot.

Screenshots
No response
Additional context
No response

Checked the Number Validation on the attribute and console error in the product edit page. Now the issue has been fixed and working fine.
Video
https://github.com/bagisto/bagisto/assets/92721837/bd6723c2-8ff7-47fd-8ac7-e7f8f7f3102b

Checked and found that the number validation issue has been fixed.
Video
https://github.com/bagisto/bagisto/assets/92721837/d46ec16c-b046-4c3d-8dfa-6e0a53503d56


Using Multiple Form in One Screen

I have a use case where I have multiple form builder in Scaffold. It is all fine until I add a third FormBuilder with more than one item. Then while trying to validate,  I get an error as The method 'saveAndValidate' was called on null. All the three FormBuilderState doesn't work after than. If I remove the second item from third form, it work again.
The following NoSuchMethodError was thrown while handling a gesture:
The method 'saveAndValidate' was called on null.
Receiver: null
Tried calling: saveAndValidate()

When the exception was thrown, this was the stack:
#0      Object.noSuchMethod (dart:core-patch/object_patch.dart:51:5)
#1      _InvoiceDetailsState.build.<anonymous closure>.<anonymous closure> (package:manage_cargo/screens/invoice_details.dart:476:42)
#2      _InkResponseState._handleTap (package:flutter/src/material/ink_well.dart:992:19)
#3      _InkResponseState.build.<anonymous closure> (package:flutter/src/material/ink_well.dart:1098:38)
#4      GestureRecognizer.invokeCallback (package:flutter/src/gestures/recognizer.dart:184:24)
...

Can you share your code?

Hi,
Please find the code.
On Sat, Sep 26, 2020 at 12:52 AM Danvick Miller<EMAIL_ADDRESS>wrote:

Can you share your code?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/danvick/flutter_form_builder/issues/473#issuecomment-699110420,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACQCKNQQXPJTATVLOPRR7KTSHTUXNANCNFSM4RWC5BPA
.

--
Best Wishes and Regards
Mohammed Sijas
Software Engineer | SAP Consultant | Android Developer | Web Developer
Phone: +91-8861104009
Web: http://sijas.in http://sijas.in/
Facebook https://www.facebook.com/sijas | Linkedin
https://in.linkedin.com/in/mohammedsijas

I solved by including all the form fields under one single form and not using multiple forms in one screen context.


[MAR-55] Implement paging for performance on timeline

From SyncLinear.com | MAR-55

Completed with a previous PR


Ref/bug/save as

Fix the scene id problem on the save as dialog.

Task linked: CU-861m92tkm fix: save as does save duplicate guid


The "method" provided on the document will create an invalid password on /rescue/etc/shadow, thus locking out the user from the server.

The instructions provided at the documentation page linked below will generate an incorrect user password hash on /rescue/etc/shadow
How to reset local Linux password on Azure VMs
Quote:
"This article introduces several methods to reset local Linux Virtual Machine (VM) passwords."
Actually, the article explains ONLY ONE METHOD with several steps, so this is not a good start.
The main problem are the steps 7, 8 and 9 because they require to copy the files used for user authentication, between the rescue VM and the root file system on the data disk,  to update /rescue/etc/shadow
This is a bad method to change/reset a user password on a Linux VM, because it works only if the both virtual machines meet these requirements:

the target system uses shadow passwords, and

/etc/shadow is not overridden by other mechanisms (via PAM, nss, etc.), and

the target system doesn't hash /etc/shadow, and

the target system has the same usernames as the source system, and

the UIDs on the target system are the same as the UIDs on the source system, and

the encryption methods used by the passwords need to be supported on the target system, and

/etc/passwd on the target system must be in sync with the injected /etc/shadow.

So, it's more likely you will end up with an invalid hash on /rescue/etc/shadow and the user will still be locked out from the server.
Below I'm describing better/safer alternatives to change a user password on a Linux server running on Azure.
Steps to reset the password from a user account on Linux ( without a rescue VM, requires WALinuxAgent )

Make sure that WA Linux Agent service is running on the affected virtual machine

Setup the environment variables and use the Azure CLI or Azure Cloud Shell ( Bash ) to perform the password reset

  AZ_RESOURCE_GROUP=""
  AZ_VM_NAME=""
  AZ_ADMIN_USER=""
  AZ_MSADMIN_PASS=""

  az vm user update -u $AZ_ADMIN_USER -p $AZ_MSADMIN_PASS -g $AZ_RESOURCE_GROUP -n $AZ_VM_NAME

Try to access the virtual machine using ssh

Note that you can also do a password reset from the Azure Portal going to "Reset Password" on the virtual machine dashboard and you also have the option to change the public keys for user authentication.
Please refer to the this document for further details: az vm user
Steps to remove the password from a user account on Linux ( requires a rescue VM, no WALinuxAgent Required )

Setup a rescue vm and attach a copy of the Cx OS disk to it as a data disk.

  AZ_RESOURCE_GROUP=""
  AZ_VM_NAME=""
  AZ_ADMIN_USER=""
  AZ_MSADMIN_PASS=""

  az vm repair create -g $AZ_RESOURCE_GROUP -n $AZ_VM_NAME --repair-username $AZ_ADMIN_USER --repair-password "$AZ_MSADMIN_PASS" --verbose

Check az vm repair for further details

Mount the root file system on the data disk on /rescue

  # You need to execute the commands below as the root user
  sudo -i

  # Identify the device name of the data disk attached to the VM using lsblk
  lsblk

  # Mount the OS disk attached as a data disk to the rescue VM
  mkdir /rescue
  mount /dev/sdc1 /rescue

  # [OPTIONAL] if the output from lsblk shows that the root file system is located on a logical volume, then you need to mount that logical volume instead
  mount /dev/rootvg/rootlv /rescue

  # Make sure password authentication is enabled on the OpenSSH server if you will attempt to login to the server using ssh and password authentication.
  egrep "^PasswordAuthentication" /rescue/etc/ssh/sshd_config

  # Enter the user name, it can be the name of the admin user or "root"
  USER_NAME="root"

  # Make a backup of the current /rescue/etc/shadow
  cp -av /rescue/etc/shadow /rescue/etc/shadow.$( date '+%Y.%m.%d_%H.%M.%S' )

  # Remove the root password from the file /rescue/etc/shadow
  sed -r -i.azbackup "s/${USER_NAME}\:[^\:]*/${USER_NAME}\:/g" /rescue/etc/shadow

  # Compare the differences between the password files
  diff /rescue/etc/shadow /rescue/etc/shadow.azbackup

Unmount the root file system and de-attach the OS disk copy from the rescue VM.

umount /rescue

Swap the OS disks on the customer virtual machine.

az vm repair restore -g $AZ_RESOURCE_GROUP -n $AZ_VM_NAME --verbose

Enter the server from the serial console ( preferable ) or using ssh with the user account where we removed the password, the new password is "empty", so all you have to do is hit enter when the system asks for user password.

5.1 If the serial console is not enabled on the virtual machine you will have to attach a storage account to it to enable boot diagnostics.

Use the "passwd" command to setup a new password for the user account intermediately.

Try to access the server using ssh and enter the new password you setup from the serial console.

Please take into consideration that these methods have been tested using Linux distributions officially endorsed by Azure on the link below:
Endorsed Linux distributions on Azure
If a customer is experiencing problems with a network appliance then it's necessary to reach out to the appliance vendor first to get instructions about how to perform a password reset safely, since changing a marketplace appliance's user account settings from a rescue VM might fail or break-up the setup of the appliance.
When the customer is using a customized virtual machine image based on one of the endorsed OS distribution you might proceed.
An ALWAYS make sure to create an snapshot of the OS disk before attempting to reset any user password on an OS disk.

Other advanced techniques to reset a user password on Linux ( don't attempt any of them unless you know what you're doing ):

Try to boot the OS on single user mode to access the rescue shell and reset the admin password to enter the server using the serial console from the Azure Portal.

Update /rescue/etc/shadow using crypt(3) libc function to reset a user password.

Use the command "chroot" to change root password on a root file system on a data disk.

Use systemd-containerd to create a container and boot the OS disk in rescue mode.

Setup QEMU on the rescue VM to start a nested virtual machine on Linux using  the data disk attached as OS disk.

Use Hyper-V to create a virtual machine that boots from a Live Rescue CD to boot and update the OS configuration on the data disk attached to the nested VM.

@carlosporter, I will forward your feedback to your engineers and investigate the problem.


[docker] Aerostack2 docker image

Feature Description
Scripts needed for docker image creation
Implementation Considerations
No response

@pariaspe, can we close this issue ?

@pariaspe, can we close this issue ?

See #80


neovim derivation failing on Darwin after libstdc++ dependency is added

Describe the bug
Since https://github.com/NixOS/nixpkgs/commit/abe84c21a9a45845f68eb02dcc5937ca34224dec was committed to nixpkgs-unstable, neovim is breaking upon installation.
Specifically it seems that libstdc++ is not available, and clang cannot run when building neovim from source.
CMake Error at /nix/store/03rqlijjbrf595jjdz5b74a9cc1jswpr-cmake-3.22.1/share/cmake-3.22/Modules/CMakeTestCCompiler.cmake:69 (message):
  The C compiler

    "/nix/store/fanf50m95p6848rbiyfvj008czh1iibp-clang-wrapper-11.1.0/bin/clang"

  is not able to compile a simple test program.

  It fails with the following output:

    Change Dir: /tmp/nix-build-neovim-unwrapped-0.6.1.drv-0/source/build/CMakeFiles/CMakeTmp

    Run Build Command(s):/nix/store/askn5hm06xf8b8lmyb5r5v362lv040pk-gnumake-4.3/bin/make -f Makefile cmTC_1b8fc/fast && /nix/store/askn5hm06xf8b8lmyb5r5v362lv040pk-gnumake-4.3/bin/make  -f CMakeFiles/cmTC_1b8fc.dir/build.make CMakeFiles/cmTC_1b8fc.dir/build
    make[1]: Entering directory '/private/tmp/nix-build-neovim-unwrapped-0.6.1.drv-0/source/build/CMakeFiles/CMakeTmp'
    Building C object CMakeFiles/cmTC_1b8fc.dir/testCCompiler.c.o
    /nix/store/fanf50m95p6848rbiyfvj008czh1iibp-clang-wrapper-11.1.0/bin/clang    -MD -MT CMakeFiles/cmTC_1b8fc.dir/testCCompiler.c.o -MF CMakeFiles/cmTC_1b8fc.dir/testCCompiler.c.o.d -o CMakeFiles/cmTC_1b8fc.dir/testCCompiler.c.o -c /tmp/nix-build-neovim-unwrapped-0.6.1.drv-0/source/build/CMakeFiles/CMakeTmp/testCCompiler.c
    Linking C executable cmTC_1b8fc
    /nix/store/03rqlijjbrf595jjdz5b74a9cc1jswpr-cmake-3.22.1/bin/cmake -E cmake_link_script CMakeFiles/cmTC_1b8fc.dir/link.txt --verbose=1
    /nix/store/fanf50m95p6848rbiyfvj008czh1iibp-clang-wrapper-11.1.0/bin/clang -Wl,-search_paths_first -Wl,-headerpad_max_install_names CMakeFiles/cmTC_1b8fc.dir/testCCompiler.c.o -o cmTC_1b8fc
    ld: library not found for -lstdc++
    clang-11: error: linker command failed with exit code 1 (use -v to see invocation)
    make[1]: *** [CMakeFiles/cmTC_1b8fc.dir/build.make:100: cmTC_1b8fc] Error 1
    make[1]: Leaving directory '/private/tmp/nix-build-neovim-unwrapped-0.6.1.drv-0/source/build/CMakeFiles/CMakeTmp'
    make: *** [Makefile:127: cmTC_1b8fc/fast] Error 2

  CMake will not be able to correctly generate this project.
Call Stack (most recent call first):
  CMakeLists.txt:7 (project)

Steps To Reproduce
Steps to reproduce the behavior:

Install neovim from unstable on Darwin

Expected behavior
neovim should install normally without error
Notify maintainers
@thornycrackers
Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
[user@system:~]$ nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-darwin"`
 - host os: `Darwin 20.6.0, macOS 10.16`
 - multi-user?: `no`
 - sandbox: `no`
 - version: `nix-env (Nix) 2.3.10`
 - channels(nasadorian): `"nixpkgs-22.05pre348596.8bd55a6a5ab, home-manager"`
 - nixpkgs: `/Users/nasadorian/.nix-defexpr/channels/nixpkgs`

https://github.com/NixOS/nixpkgs/pull/155688

there is an extensive discussion in https://github.com/NixOS/nixpkgs/pull/147658 . The current ldflags is problematic even on linux (you just have to use clang). I plan to remove it in https://github.com/NixOS/nixpkgs/pull/166162

I've removed the incriminating change


Deoptimise encoding into string and deny(unsafe_code)

This PR is a bit speculative. See issue #34 for more discussion.
It takes the first option mentioned in https://github.com/alicemaz/rust-base64/issues/34#issuecomment-313542184 -- as such it'll add an extra allocation and copy when encoding into a String.
This PR also adds unsafe_code to the crate-wide lint denies.

Thanks, I'll take a look. Is there a particular reason for this change (aside from generic "unsafe makes the world a worse place")?

There's not much more to it than that :)
The background was: I was running cargo-geiger on my downstream crate that depends on base64, and base64 is the only crate where unsafe is used but isn't fundamental to the purpose of the crate.

Fair enough. I'm guessing rustls? I want rustls to be unsafe-free. :)
I'm working on the line wrap removal this weekend. Hopefully I'll get that done quickly, at which point I'll benchmark this approach vs ChunkedEncoder and pick whichever is faster.

This was pretty painful, almost halving encode speed in most cases. #80 is a bit more complex but much less costly.


Mentioning of Express version is too short and unclear

It would be nice if the article would elaborate on SQL Server Express not supporting the publication in the tutorial and what options there are for Express and what are the disadvantages. The wording suggests that it is possible to create another type of publication for express subscribers.

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

ID: 16df2747-32ea-b464-e92c-535f6d2a788c
Version Independent ID: f50b285e-9a62-8823-c207-3b31ce6d4e7e
Content: Tutorial: Configure Merge Replication - SQL Server
Content Source: docs/relational-databases/replication/tutorial-replicating-data-with-mobile-clients.md
Product: sql
Technology: replication
GitHub Login: @MashaMSFT
Microsoft Alias: mathoma

Hi @Laniusexcubitor , thanks for contacting Microsoft! You can find information about what replication technology is supported by edition here: Editions and supported features for SQL Server 2019.
You'll notice that SQL Server Express supports being a subscriber to a merge replication. However, the objects that are created in this tutorial specifically do not support SQL Server express.
I hope that helps! I will close out this Git Issue for the time being but please feel free to commend should you need anything further.
Thanks again for reaching out and I hope you have a wonderful rest of your day!
Masha from the SQL Docs team


[Feature Request] Plugin Menu - add Filter by Tenant

In the plugin menu - ability to filter devices by tenant.
This would be great so I can give topology reports to customers based on their devices.

This is now available in the latest plugin release.


Fix Firefox URL matching

Fix Firefox URL matching.
Previously URLs used in the Facebook send dialog were triggering the URL handler, resulting in errors.
e.g. this URL should not trigger the event handlers anymore:
https://www.facebook.com/dialog/send?app_id=%20161927677344933&link=https://www.uproxy.org/invite/eyJ2IjoxLCJuZXR3b3JrTmFtZSI6IkZhY2Vib29rLUZpcmViYXNlLVYyIiwidXNlck5hbWUiOiJEYW5pZWwgQm9ya2FuIiwibmV0d29ya0RhdGEiOiJ7XCJ2XCI6MSxcInVzZXJJZFwiOlwiNjcwNjg0OFwiLFwicGVybWlzc2lvblRva2VuXCI6MTk0NTcxMTYwNzF9In0=&redirect_uri=https://www.uproxy.org/

:+1:


Possible to clear file list every time user selects files?

Using core mode, is it possible to clear the internal file list, every time the user selects files to upload?
Currently I'm binding to the change event of the underlying <input type=file> but this seems to fire too late.
The default behaviour is add the newly selected files to the internal list of files to upload, but to me it is more intuitive to only select the last uploaded files.

why do you care about the internal state?

so, since you ignored the template, which I feel would have answered the question above, I'm going to close this.


"Error CS8802: Only one compilation unit" running .NET tutorial

Problem encountered on https://dotnet.microsoft.com/en-us/learn/dotnet/hello-world-tutorial/run
Operating System: windows
Provide details about the problem you're experiencing. Include your operating system version, exact error message, code sample, and anything else that is relevant.
C:\Users\mt15k\MyApp>dotnet run
C:\Users\mt15k\MyApp\Program.cs(2,1): error CS8802: Only one compilation unit can have top-level statements. [C:\Users\mt15k\MyApp\MyApp.csproj]
The build failed. Fix the build errors and run again.
Running Microsoft Windows 11 Pro, version 10.0.22000 build 22000.

What are the contents of your MyApp directory? The dotnet new console -o MyApp command should have only created one .cs file called Program.cs with the following content:
// See https://aka.ms/new-console-template for more information
Console.WriteLine("Hello, World!");

Did you change anything in the project?

C:\Users\mt15k\MyApp>dir
Volume in drive C is OS
Volume Serial Number is C4BD-2029
Directory of C:\Users\mt15k\MyApp
01/31/2022  10:31 PM              .
01/31/2022  10:23 PM              ..
01/31/2022  10:28 PM              bin
01/31/2022  10:27 PM              myapp
01/31/2022  10:31 PM               249 MyApp.csproj
01/31/2022  10:28 PM              obj
01/31/2022  10:31 PM               105 Program.cs
           2 File(s)            354 bytes

           5 Dir(s)  25,490,673,664 bytes free

C:\Users\mt15k\MyApp>type program.cs
∩╗┐// See https://aka.ms/new-console-template for more information
Console.WriteLine("Hello, World!");
C:\Users\mt15k\MyApp>
You’ll see some strange characters before the “//See https:…”. They don’t show up when I use Notepad to view it.
I reran the tutorial and get the same problem.
From: Maira Wenzel @.>
Sent: Monday, January 31, 2022 11:12 PM
To: dotnet/core @.>
Cc: RizzKid1 @.>; Author @.>
Subject: Re: [dotnet/core] "Error CS8802: Only one compilation unit" running .NET tutorial (Issue #7156)
What are the contents of your MyApp directory? The dotnet new console -o MyApp command should have only created one .cs file called Program.cs with the following content:
// See https://aka.ms/new-console-template for more information
Console.WriteLine("Hello, World!");
Did you change anything in the project?
—
Reply to this email directly, view it on GitHub https://github.com/dotnet/core/issues/7156#issuecomment-1026461036 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AXRXQ7YRN5NNE5IUQ7ARBXTUY5MPJANCNFSM5NIBQMMA .
Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675  or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub .
You are receiving this because you authored the thread.  https://github.com/notifications/beacon/AXRXQ7357VSJGVD7CTEOCM3UY5MPJA5CNFSM5NIBQMMKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOHUXI23A.gif Message ID: @.*** @.***> >

You seem to have another myapp directory inside this project. Maybe delete everything and try again? Or just delete that internal myapp folder? If you ran the command inside the MyApp folder, it would create another Program.cs file causing this issue.

Deleted it all, started over, and it works. Thanks for the quick help.
From: Maira Wenzel @.>
Sent: Monday, January 31, 2022 11:34 PM
To: dotnet/core @.>
Cc: RizzKid1 @.>; Author @.>
Subject: Re: [dotnet/core] "Error CS8802: Only one compilation unit" running .NET tutorial (Issue #7156)
You seem to have another myapp directory inside this project. Maybe delete everything and try again? Or just delete that internal myapp folder? If you ran the command inside the MyApp folder, it would create another Program.cs file causing this issue.
—
Reply to this email directly, view it on GitHub https://github.com/dotnet/core/issues/7156#issuecomment-1026470615 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AXRXQ72HECZ3FCCMEF6UJM3UY5PEVANCNFSM5NIBQMMA .
Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675  or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub .
You are receiving this because you authored the thread.  https://github.com/notifications/beacon/AXRXQ75RAIV2L5H6VC74JZLUY5PEVA5CNFSM5NIBQMMKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOHUXLFVY.gif Message ID: @.*** @.***> >

Glad to hear. Closing the issue then.


Avoid use of Consistently in integration tests

What type of PR is this?
/kind cleanup

What this PR does / why we need it:
Avoid use of Consistently in integration tests
Which issue(s) this PR fixes:

Fixes #1234
Special notes for your reviewer:
Does this PR introduce a user-facing change?

NONE

/ok-to-test


Missing entries in Calendar

There is a display problem in the calendar when some month has no activity. E.g. for me, there were no (tracked) activities in January and February, and thus the value for march appeared in column 1.
The underlying problem is that the table is filled by looping over monthly_distances, which might not have a entry for every month.
One solution would be to edit monthly_distances such that it does have an entry for every month (of all years that are present).
Alternatively, the table can be filled by a loop over values 1...12.

The current bugged version:

With 0 km entries:

With empty entries (the spacing looks very odd to me, but could maybe be improved by minimum or fixed column widths):


Links in summaries cause cards to be misplaced

I noticed that after updating to v0.19.0 that links in summaries[1] cause e.g. the blog page to have those cards out of place.
[1]: E.g.
…
toc: true
---

When I have links in summaries on [Princes and Madmen](https://www.princesandmadmen.ca)
the list page cards 'jump' out of place.
{.p-first}
<!--more-->

I have worked around this using a frontmatter summary that doesn't have the link.

Yes, unfortunately, I did forget that links inside of links are impossible. It's obvious when you think about it, but I was in a hurry when I changed the whole card frame into a link.
Now, an additional link inside a card breaks the HTML structure.
There are two ways out of this:

Avoid links for text in cards.

Remove the link for the card frame and link the appropriate elements on the card to the posting (image, heading, and button).

For small cards showing the description value as small text, I will implement option 1.
For bigger ones with a button I will implement option 2.


[FEATURE REQUEST]  Add Responsive Footer

I propose adding a responsive footer to the repository. A well-designed footer enhances the overall user experience by providing important information and navigation links in a consistent manner across all devices.

i am intereted plz asign this to me !

someone is working on it

ok but on right part in Assignees no one assigned


NIFI-7805 ResultSetRecordSet can't generate a schema if first row con…

…tains NULL Decimal
Description of PR
When using ResultSetRecordSet, if the first row in a resultset contains a Decimal column with a NULL value, then NiFi fails to create a schema and throws an exception. This is because it is trying to read the Decimal Scale and Precision from the Decimal Value rather than from the Result Set Schema.
For all changes:

[x] Is there a JIRA ticket associated with this PR? Is it referenced
in the commit message?

[x] Does your PR title start with NIFI-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.

[x] Has your PR been rebased against the latest commit within the target branch (typically main)?

[x] Is your initial contribution a single, squashed commit? Additional commits in response to PR reviewer feedback should be made on this branch and pushed to allow change tracking. Do not squash or use --force when pushing to allow for clean monitoring of changes.

For code changes:

[ ] Have you ensured that the full suite of tests is executed via mvn -Pcontrib-check clean install at the root nifi folder?
[x] Have you written or updated unit tests to verify your changes?
[ ] Have you verified that the full build is successful on JDK 8?
[ ] Have you verified that the full build is successful on JDK 11?
[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[ ] If applicable, have you updated the LICENSE file, including the main LICENSE file under nifi-assembly?
[ ] If applicable, have you updated the NOTICE file, including the main NOTICE file found under nifi-assembly?
[ ] If adding new Properties, have you added .displayName in addition to .name (programmatic access) for each of the new properties?

For documentation related changes:

[ ] Have you ensured that format looks appropriate for the output in which it is rendered?

Note:
Please ensure that once the PR is submitted, you check GitHub Actions CI for build issues and submit an update to your PR as soon as possible.

@tpalfy @markap14 @simonbence
Small bug fix for https://github.com/apache/nifi/pull/4223. If you have a minute to look it over I'd appreciate it.

Good catch, LGTM


Add buffer Iterate support

As title.

Codecov Report

Merging #729 into master will increase coverage by 0.18%.
The diff coverage is 85.36%.

@@            Coverage Diff            @@
##           master    #729      +/-   ##
=========================================
+ Coverage   52.41%   52.6%   +0.18%
=========================================
  Files          35      35
  Lines        8226    8267      +41
=========================================
+ Hits         4312    4349      +37
- Misses       3431    3434       +3
- Partials      483     484       +1

Impacted Files
Coverage Δ

engine.go
56.56% <100%> (+0.17%)
:arrow_up:

statement.go
64.56% <100%> (+0.38%)
:arrow_up:

session_iterate.go
75.4% <83.33%> (+11.4%)
:arrow_up:

xorm.go
66.19% <0%> (-1.41%)
:arrow_down:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 57c2ebf...56ddb91. Read the comment docs.


Introduce new high-availability-config webhook in gardener-resource-manager

How to categorize this PR?

/area high-availability dev-productivity
/kind enhancement
What this PR does / why we need it:
This PR introduces a new webhook called high-availability-config in gardener-resource-manager with the purpose of making it easy for developers to adapt components according to the HA config of the respective cluster.
Concretely, please refer to this document for more information.
In an nutshell, the Deployments or StatefulSets in namespaces labeled with high-availability-config.resources.gardener.cloud/consider=true will be mutated:

.spec.replicas according to their high-availability-config.resources.gardener.cloud/type={controller,server} label
.spec.template.spec.topologySpreadConstraints according to their replicas, the number of AZs and the failure tolerance type
.spec.template.spec.affinity.nodeAffinity according to the number of AZs

This PR already actives the webhook for all namespaces running seed or shoot system components, or shoot control plane components. However, the components are not yet labeled with the type label (this will follow in a subsequent PR).
Only the gardenlet and the gardener-resource-manager itself have been adapted:

gardenlet "duplicates" part of the webhook's logic in its chart since it cannot benefit from the webhook since it's the root component in seed clusters.
gardener-resource-manager for seeds is serving the webhook, hence it cannot benefit from it and its deployment configuration must be manually adjusted (done in the resourcemanager component).
gardener-resource-manager for shoots is handled by the webhook.

This PR also enables the existing podtopologyspreadconstraints webhook for the components managed by Gardener in the shoot cluster's system namespaces.
Which issue(s) this PR fixes:
Part of https://github.com/gardener/gardener/issues/6529
Special notes for your reviewer:
/cc @timuthy
Release note:

The `failureToleranceType` field has been removed from `.spec.gardenlet.deployment` in the `seedmanagement.gardener.cloud/v1alpha1.ManagedSeed` API.

The `gardener-resource-manager` serves a new `high-availability-config` webhook for automatically mutating the HA-related configuration of `Deployment`s and `StatefulSet`s. Please refer to [this](https://github.com/gardener/gardener/blob/master/docs/development/high-availability.md) and [this](https://github.com/gardener/gardener/blob/master/docs/concepts/resource-manager.md#high-availability-config) document.

`Deployment`s or `StatefulSet`s deployed by extensions in seed or shoot clusters can now benefit from the new `high-availability-config` webhook for automatically mutating the HA-related configuration of these resources. Please refer to [this](https://github.com/gardener/gardener/blob/master/docs/extensions/conventions.md#high-availability-of-deployed-components) document.

/assign

/test pull-gardener-integration


Calling dockerized ort from TypeScript

Description

Refs: #34, #45
I this PR, I'm experimenting with calling dockerized ort from TypeScript. The basics are there, but needs more work.

Before you get started

[ ] ☝ Create an issue to discuss what you are going to do

Checklist
This section should be filled in by the creator of this pull request to make sure the pull request is ready to review.

[x] This pull request has a descriptive title
[x] Code is written according to the code quality guidelines
[x] Documentation if applicable is available
[x] Tests run successfully
[x] All checks below this pull request were successful

Instructions to review the pull request

git clone this repo
cd licenseguard
git checkout this branch
npm install
npm run build
npm run package
MYDIR=`pwd`

# cd to tempdir to facilitate writing the output and downloading the clones
cd $(mktemp -d --tmpdir ortolan.XXXXXX)
node $MYDIR/dist/index.js

Kinda works (see result below) but there's something odd with the async
.
├── in
│   ├── iomega
│   │   └── zenodo-upload
│   │       ├── CHANGELOG.md
│   │       ├── CITATION.cff
│   │       ├── CODE_OF_CONDUCT.md
│   │       ├── CONTRIBUTING.md
│   │       ├── docs
│   │       ├── LICENSE
│   │       ├── package.json
│   │       ├── README.md
│   │       ├── sonar-project.properties
│   │       ├── src
│   │       ├── test
│   │       ├── tsconfig.json
│   │       └── yarn.lock
│   └── xenon-middleware
│       └── xenon-cli
│           ├── appveyor.yml
│           ├── build.gradle
│           ├── CHANGELOG.md
│           ├── CITATION.cff
│           ├── conda
│           ├── Dockerfile
│           ├── gradle
│           ├── gradlew
│           ├── gradlew.bat
│           ├── LICENSE
│           ├── README.md
│           ├── src
│           ├── xenon-download.cwl
│           ├── xenon-ls.cwl
│           └── xenon-upload.cwl
└── out
    ├── iomega
    │   └── zenodo-upload
    │       └── analyzer-result.yml
    └── xenon-middleware
        └── xenon-cli
            └── analyzer-result.yml

$ cat out/iomega/zenodo-upload/analyzer-result.yml | head -n 10
---
repository:
  vcs:
    type: "Git"
    url: "https://github.com/iomega/zenodo-upload"
    revision: "4b850962abc1f467071d9e09348cb67deb4c8bb7"
    path: ""
  vcs_processed:
    type: "Git"
    url: "https://github.com/iomega/zenodo-upload.git"

Marked as Ready for review because this could use some input from others

This code is using ort, but that needs to be built first from its repo; maybe update the name to philipssoftware/ort

Thanks for reviewing. I'll merge it and then we can work on fixing some of the shortcomings we discovered


Fixes Update.

This fixes the Update issues in #1181
Which is some function needs to be changed in the Update-WinUtilProgramWinget.ps1

I know that there is an issue I ran into with my changes in my Pull Request in regards to installing Winget. Non-UWP apps refuse to install when scope is defined as machine (i.e. --scope=machine).


Application always reloads page even when hot module reload is specified

Describe the bug
Even when I define some parts of the application as hot-reloadable, the page always refreshes in full.
I must say that I don't really understand how HMR is setup in CRA due to the absence of documentation about it in the doc. Stylesheets do refresh without a full page refresh, but not javascript files.
Which terms did you search for in User Guide?
"HMR", "hot module reload", "hot reload"
Environment
Environment Info:

  current version of create-react-app: 3.4.1
  running from /home/romgrk/.cache/npm-packages/lib/node_modules/create-react-app

  System:
    OS: Linux 5.8 Arch Linux
    CPU: (4) x64 Intel(R) Core(TM) i7-4610M CPU @ 3.00GHz
  Binaries:
    Node: 12.18.0 - /tmp/fnm-shell-8746319/bin/node
    Yarn: 1.22.4 - ~/.cache/npm-packages/bin/yarn
    npm: 6.14.4 - /tmp/fnm-shell-8746319/bin/npm
  Browsers:
    Chrome: Not Found
    Firefox: 81.0.1
  npmPackages:
    react: ^16.13.1 => 16.13.1
    react-dom: ^16.13.1 => 16.13.1
    react-scripts: ^3.4.3 => 3.4.3
  npmGlobalPackages:
    create-react-app: 3.4.1

Steps to reproduce
Create a standard application and add in src/index.js:
if (module.hot) {
  module.hot.accept('./components/App', () => {
    const NextApp = require('./components/App')
    debugger
    render(NextApp)
    debugger
  })
}

I can assert that the module.hot.accept callback is called here due to the debugger statement, which is hit before the page refresh. The second debugger statement isn't hit.
Expected behavior
Only the App component should refresh, not the full page.
Actual behavior
Full page refresh.
Reproducible demo
Will provide if requested.

CRA itself doesn't supports HMR in the moment. You should configure your webpack/babel settings in your own.

Activity


Avoid GL_INVALID_ENUM generation by glDrawBuffers

When running Celeste64 from the command line a bunch of errors are generated of the form GL (ERROR:HIGH) GL_INVALID_ENUM in glDrawBuffers(invalid buffer GL_BACK).  I don't believe there's any meaningful impact on the rendering being done, but for the sake of a tidy command line this tiny patch addresses that error message.
References I used:
https://registry.khronos.org/OpenGL-Refpages/gl4/html/glDrawBuffers.xhtml
https://www.khronos.org/opengl/wiki/Default_Framebuffer
https://registry.khronos.org/OpenGL/api/GL/glcorearb.h

Thanks! I noticed some reports of this so definitely want to fix it and will merge this in ... but do you know the actual difference between GL_BACK_LEFT and GL_BACK? I figured that GL_BACK would be the correct case since we want to draw to the backbuffer.

My understanding is that GL_BACK, in the context of Framebuffer Objects, is an alias for both the GL_BACK_LEFT and GL_BACK_RIGHT buffers.  This is in contrast to the default framebuffer context, where GL_BACK refers to the (singular) back buffer.  The multiple buffers are for the possibility of stereoscopic rendering, but when you're not rendering in stereo then GL_BACK_LEFT (and GL_FRONT_LEFT) are the only active buffers.  I believe the way this is squared is that when GL_BACK_RIGHT is not active (i.e. when not rendering in stereo), GL_BACK works as an alias to GL_BACK_LEFT only;  in effect, GL_BACK_LEFT and GL_BACK are equivalent.  Why that wasn't extended to allow GL_BACK as an input to glDrawBuffers, I can only speculate.

That makes sense, thanks for the insights and digging into solving this! :)

Thanks!  Huge fan of your work.


We need better documentation for running locally

I have tried this now for about an hour, and I get the SAM service to run, and also nginx, but it wasn't straight forward and nginx only shows the frontend and then fails to forward the requests.  We should include an nginx.conf file that works on localhost, serves only localhost, and instructions to run nginx as a simple user process and not as a system service (i.e.
nginx -p `pwd` -c nginx.local.conf

).  This is my current state, I assume I am missing something, probably the uvicorn stuff?  All this is very cryptic:
http {
  upstream uvicorn {
    server unix:/tmp/uvicorn.sock;
  }

  map $http_upgrade $connection_upgrade {
    default upgrade;
    '' close;
  }

  server {
    listen 8080 default_server;
    server_name <IP_ADDRESS> localhost;
    client_max_body_size 20M;
    access_log nginx_access.log combined;
    error_log nginx_error.log;

    location / {
      proxy_set_header Host $http_host;
      proxy_set_header X-Forwarded-For $proxy_add_x_forwarded_for;
      proxy_set_header X-Forwarded-Proto $scheme;
      proxy_set_header Upgrade $http_upgrade;
      proxy_set_header Connection $connection_upgrade;
      proxy_redirect off;
      proxy_buffering off;
      proxy_pass http://uvicorn;
    }
  }
}

# events context
events {
  worker_connections 1024;
}

pid nginx.pid;
error_log nginx_error.log;

Agreed, I will walk through the steps on a windows laptop with WSL2 installed and document the process.

Added additional documentation for running the service based on a fresh install of the whole stack on a windows laptop.
See 0cade678814f0cecb13da435698feae52557acf4


Reduce warmup time with one second to avoid lockstep with tick

Reduce the chance of using an extra tick when there are some milliseconds
left of warmup time
I very often see logs like this:
 Refusing to resume until warm up period ends (in PT0.096547S)
this PR tries to avoid spending an extra tick (30 seconds) when there are just some ms left before warm up period ends

🥇

On a second thought we should probably increase, this was initially 30 sec, which due to the issue you describe made it effectively 1 min. That was not enough and it was increased to 1 min - effectively 1 min 30 sec. With this, we will essentially go back to pre- 16f7515017f67b9d939a19c33de7a96da391b20b result.
So this should probably be between 1 min - 1 min 29 sec, if on the higher end, that will actually increase it for CD (tick interval 15 sec). Or use a separate time for CD.

@baldersheim can you confirm?

If possible this should only be done in prod, in cd and test this could be 0.

Changes based on discussion above should be handled in https://github.com/vespa-engine/vespa/pull/12752


[BUG] During auto-scaling up/down of multiple Eventhub Receivers. Some events could not be pulled from certain partitions.

Describe the bug
During auto-scaling up/down of multiple EventHub Receivers. Some events could not be pulled from certain partitions.
After rebooting Receivers, missing events could be pulled back properly.

Numbers of Receivers:
Min : 1
Max: 10

Total Number of EventHub Partitions: 30

Exception or Stack Trace
Add the exception log and stack trace if available
To Reproduce
May be on multiple receivers case with scaling up/down. It might occur.
Code Snippet
Currently used Maven Repository.
compile group: 'com.microsoft.azure', name: 'azure-eventhubs', version: '3.1.1'
We are using EventProcessorHost . Below is the code which is used.
#############################
public EventProcessorHost createEventHubProcessorHost(ScheduledExecutorService scheduledExecutorService) {
EventProcessorHost event = EventProcessorHost.EventProcessorHostBuilder
.newBuilder(
EventProcessorHost.createHostName(azureProperties.getEventHubReceiver().getHostNamePrefix()),
azureProperties.getEventHubReceiver().getConsumerGroupName())
.useAzureStorageCheckpointLeaseManager(azureProperties.getStorage().getConnectionString(),
azureProperties.getEventHubReceiver().getCheckpoint().getContainerName(), null)
.useEventHubConnectionString(azureProperties.getEventHubReceiver().getConnectionString(),
azureProperties.getEventHubReceiver().getName())
.setExecutor(scheduledExecutorService)
.build();
log.info("Event hub EventProcessorHost created");
return event;
}
#############################
Expected behavior

Run one Receiver
Add more receivers
After sometime, stop newly added receivers and  let the first receiver continue running.
Now expectation is all 30 partitioned should be assigned back to first receiver.

Screenshots
If applicable, add screenshots to help explain your problem.
Setup (please complete the following information):

OS: [azul/zulu-openjdk:8]
IDE: [Running on Docker]
Library/Libraries: [compile group: 'com.microsoft.azure', name: 'azure-eventhubs', version: '3.1.1']
Java version: [openjdk: 8]
App Server/Environment: [Docker, AKS]
Frameworks: [Spring Boot 2.4.2]

If you suspect a dependency version mismatch (e.g. you see NoClassDefFoundError, NoSuchMethodError or similar), please provide

verbose dependency tree (mvn dependency:tree -Dverbose)

Additional context
Information Checklist
Kindly make sure that you have added all the following information above and checkoff the required fields otherwise we will treat the issuer as an incomplete report

[ ] Bug Description Added
[ ] Repro Steps Added
[ ] Setup information Added

/cc: @conniey , @anuchandy could you please take a look.

@JamesBirdsall Can you take a look at this please? Thanks!


Convert file picker to Fluent UI 9

Describe the Bug with repro steps
Existing file picker was ported from old designer and does not match Fluent (esp. v9) design patterns.
What type of Logic App Is this happening in?
Standard (Portal)
Are you using new designer or old designer
New Designer
Did you refer to the TSG before filing this issue? https://aka.ms/lauxtsg
Yes
Workflow JSON
No response
Screenshots or Videos

Browser
N/A
Additional context
No response

I can take this work on.


Update demos to use consistent parameter settings

I updated the parameter values for the demos so that:

They are all the same
They yield consistent image quality
They all use 2 levels of multi-resolution (except the prox demo)

Sure will work on that.
Diyu

From: Charles A Bouman @.>
Sent: Tuesday, June 7, 2022 12:06 PM
To: cabouman/svmbir @.>
Cc: Diyu Yang @.>; Review requested @.>
Subject: Re: [cabouman/svmbir] Update demos to use consistent parameter settings (PR #244)
---- External Email: Use caution with attachments, links, or sharing data ----
Diyu, can you just test my demos?
—
Reply to this email directly, view it on GitHubhttps://github.com/cabouman/svmbir/pull/244#issuecomment-1148871268, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AB2Q7OGJ7V5ORQD3X2QT3DLVN5XPNANCNFSM5YDPIJVQ.
You are receiving this because your review was requested.Message ID: @.***>

I tested all demo scripts. Everything works and the reconstruction results all look good.
There are two small issues though:

For demo_multires_comparison.py, the two recon images are saved to the same file name output/mr_3D_microscopy_recon.png. We might want to change the name of the fixed recon image to something like output/fixed_res_3D_microscopy_recon.png
For the fan beam demo the image results are not saved to disk. This will be a problem when someone runs the demo on a server, where the monitor display is not available (i.e. the plots do not pop up).

Diyu, OK, can you make both fixes?


41 - Backend Base Project configuration

Basic project configuration
main points:
-basic folder structure
-a basic user schema in order to be able to test, created using sqlmodel
-db folder will be used for connecting to DB in production and also can be used to generate a local db for testing
-versioning thru .env file (file needs to be provided or created with proper variables)
-test are working with testClient, there are 2 endpoints to test one with a mock user and the other will serve for general information (including version)
-routers folder added in order to organize the endpoints
-dockerfile
-readme completed with instructions to deploy in docker and as a local app for dev

Hi, the Lint workflow is failing because it expects dependencies to be managed as documented here: https://ocp-software-handbook.readthedocs.io/en/latest/python/requirements.html

Hi @Lombardoh, you seemed to be having trouble getting the tests to pass, so I added a commit.
In short: If you import a library in application code, it needs to be listed in requirements.in
If you import any other library in test code, it needs to be in requirements_dev.in

I see many commits relating to pre-commit issues. I highly recommend running:
pip install pre-commit
pre-commit install

That way, a pre-commit git hook will warn before committing.
You can also configure your editor to run pre-commit run whenever you save a file, for example.

FYI: The pre-commit.ci check failed, which is fine, as it often happens due to their infrastructure ("Temporary failure in name resolution" when trying to reach PyPI). This check is configured to be ignored, so the overall PR gets a check mark.


Running "fuzzing_AFL.sh" continuously will produce mixed code coverage

Verified on d33470e3b4f2c225a97a83d268b08f8ed5ea66b8
Ubuntu 20.04.4 LTS

Run the following two commands consecutively. When the encryption parameters are different, the code coverage of the test case tested before will be mixed with the current test. The code coverage report generated by the second command openssl will also contain the code coverage generated by the first command mbedtls.
libspdm/unit_test/fuzzing/fuzzing_AFL.sh mbedtls ON 60
libspdm/unit_test/fuzzing/fuzzing_AFL.sh openssl ON 60

Lcov should capture the specified folder instead of all build folders under "root_libspdm".
line 174 should change to "lcov --capture --directory $build_fuzzing --output-file coverage.info"
https://github.com/DMTF/libspdm/blob/3c231197440d4c55a6eee7875c856d8519fcbdcc/unit_test/fuzzing/fuzzing_AFL.sh#L170-L176

I will check it


Update Composer dependencies (2022-04-22-00-10)

Visual regression test passed!
View the visual regression test report


General improvement of JWT

JWT generation needs the following improvements:

create unit tests
rsakey pair generation (open for discussion: how & where will we generate and store it?)
current JWT Claims are hardcoded. How should this be done in the future?

The file is currently not available on develop branch, but can be viewed here:
https://github.com/Polkadex-Substrate/polkadexTEE-worker/blob/feature/fm-bh-openfinex-api-squashed/enclave/src/openfinex/jwt.rs

@simonsso Investigate what needs to be done here


[REVIEW] Dev Protocol Site

Project link
https://github.com/web3community/devprotocol.xyz

Great profile, reviewed on live stream https://www.youtube.com/watch?v=JQ1wZRjint8 ... not sure I can make any suggestions


Fix container query in comments

Provide the exact quote of the error or issue:
We should modify the _container query to use breakpoints provided by @yamada-ui/react.
contents\data-display\comments\with-html-content\index.tsx
contents\data-display\comments\comment\index.tsx
This issue will be automatically unassigned after two weeks have passed since it was assigned. Once unassigned, it may be picked up by someone else for work.

I will work on this.


feat(upload): create placeholder image url from file

Description

Checklist

[ ] Read the contributing guidelines.
[ ] Each commit follows the Conventional Commit spec format.
[ ] Update the readme (if applicable).
[ ] Update or add any necessary API documentation (if applicable)
[ ] All existing unit tests are still passing (if applicable).

[ ] Add some steps so we can test your bug fix or feature (if applicable).
[ ] Add new passing unit tests to cover the code introduced by your PR (if applicable).
[ ] Any breaking changes are specified on the commit on which they are introduced with BREAKING CHANGE in the body of the commit.
[ ] If this is a big feature with breaking changes, consider opening an issue to discuss first. This is completely up to you, but please keep in mind that your PR might not be accepted.

Current dependencies on/for this PR:

next

PR #140

PR #139

PR #141

PR #142

PR #143

PR #144   👈

PR #145

This comment was auto-generated by Graphite.

Graphite Merge Job
Current status: ⏳ Queued to merge
This pull request is currently queued to be merged as part of a stack.
This comment was auto-generated by Graphite.

Job Reference: yGgqIySrcaAmVh8uxOCg


[MRG] Unify configs

Responding to other email questions inline: Reading from the new config does work.
I'm also updating the scala code and documentation to point to the new syncer file.

This should be ready to merge.
I'd like to have some other config methods return structs (that aren't yet defined), but first want to clean up the python imports to help lessen confusion when refactoring.

Looks good. Could you add a comment regarding testing? OK to merge after
that.
On Tue, Mar 21, 2017 at 10:37 AM, Albert Carter<EMAIL_ADDRESS>wrote:

This should be ready to merge.
I'd like to have some other config methods return structs (that aren't yet
defined), but first want to clean up the python imports to help lessen
confusion when refactoring.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/mitdbg/modeldb/pull/226#issuecomment-288158777, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AFeWw49kAe8rAP6EOPjlSHTVIFvq69Dkks5roArjgaJpZM4Mh-YS
.

Tests are passing (or in the case of testPipelineEvent.py, sometimes failing) exactly as they did before.

My own notation here:
OK: /modeldb/client/python/modeldb/tests/basic/ModelDbSyncerBaseTest.py
OK: /modeldb/client/python/modeldb/tests/sklearn/testFitEvent.py
OK: /modeldb/client/python/modeldb/tests/sklearn/testGridSearchEvent.py
OK: /modeldb/client/python/modeldb/tests/sklearn/testMetricEvent.py
2Failures: /modeldb/client/python/modeldb/tests/sklearn/testPipelineEvent.py
======================================================================
FAIL: test_overall_pipeline_fit_event (__main__.TestPipelineEvent)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "testPipelineEvent.py", line 90, in test_overall_pipeline_fit_event
    utils.is_equal_transformer_spec(spec, expected_spec, self)
  File "/Users/arcarter/code/modeldb/client/python/modeldb/tests/utils.py", line 134, in is_equal_transformer_spec
    tester.assertEqual(len(spec1.hyperparameters), len(spec2.hyperparameters))
AssertionError: 14 != 10

======================================================================
FAIL: test_pipeline_first_fit_stage (__main__.TestPipelineEvent)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "testPipelineEvent.py", line 131, in test_pipeline_first_fit_stage
    utils.is_equal_transformer_spec(spec, expected_spec, self)
  File "/Users/arcarter/code/modeldb/client/python/modeldb/tests/utils.py", line 134, in is_equal_transformer_spec
    tester.assertEqual(len(spec1.hyperparameters), len(spec2.hyperparameters))
AssertionError: 7 != 3

OK: /modeldb/client/python/modeldb/tests/sklearn/testProjectCreation.py
OK: /modeldb/client/python/modeldb/tests/sklearn/testRandomSplitEvent.py
OK: /modeldb/client/python/modeldb/tests/sklearn/testTransformEvent.py

LGTM.
On Wed, Mar 22, 2017 at 10:27 AM, Albert Carter<EMAIL_ADDRESS>wrote:

My own notation here:
OK: /modeldb/client/python/modeldb/tests/basic/ModelDbSyncerBaseTest.py
OK: /modeldb/client/python/modeldb/tests/sklearn/testFitEvent.py
OK: /modeldb/client/python/modeldb/tests/sklearn/testGridSearchEvent.py
OK: /modeldb/client/python/modeldb/tests/sklearn/testMetricEvent.py
2Failures: /modeldb/client/python/modeldb/tests/sklearn/testPipelineEvent.py
FAIL: test_overall_pipeline_fit_event (main.TestPipelineEvent)
Traceback (most recent call last):
File "testPipelineEvent.py", line 90, in test_overall_pipeline_fit_event
utils.is_equal_transformer_spec(spec, expected_spec, self)
File "/Users/arcarter/code/modeldb/client/python/modeldb/tests/utils.py", line 134, in is_equal_transformer_spec
tester.assertEqual(len(spec1.hyperparameters), len(spec2.hyperparameters))
AssertionError: 14 != 10
======================================================================
FAIL: test_pipeline_first_fit_stage (main.TestPipelineEvent)
Traceback (most recent call last):
File "testPipelineEvent.py", line 131, in test_pipeline_first_fit_stage
utils.is_equal_transformer_spec(spec, expected_spec, self)
File "/Users/arcarter/code/modeldb/client/python/modeldb/tests/utils.py", line 134, in is_equal_transformer_spec
tester.assertEqual(len(spec1.hyperparameters), len(spec2.hyperparameters))
AssertionError: 7 != 3
OK: /modeldb/client/python/modeldb/tests/sklearn/testProjectCreation.py
OK: /modeldb/client/python/modeldb/tests/sklearn/testRandomSplitEvent.py
OK: /modeldb/client/python/modeldb/tests/sklearn/testTransformEvent.py
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/mitdbg/modeldb/pull/226#issuecomment-288475994, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AFeWw3OTeyh9o5WwXz9pgzLvozVWOkzmks5roVoDgaJpZM4Mh-YS
.


ui.pl doesn't support Chinese character

Go to advanced settings, set ui.pl to Chinese character. Refresh the page, internal error 500 will be returned.
Can only recovered by modifying the database setting table directly.

Useful
Info

Versions
Rancher v1.4.1 Cattle: v0.176.9 UI: v1.4.6

Access
localauth admin

Orchestration
Cattle

Route
admin-tab.settings

Does this still happen, and is it with a 3-byte character or 4-byte?  There's no reason settings shouldn't be able to handle 3-byte, but the default character encoding of tables doesn't support 4-byte characters (less common things, and many emoji).

@loganhz - can you give me an example of Chinese characters that should work.  The ones I put in still gave me a 500 page.

It looks good in rancher/server:v1.6.15-rc8. Can you please try 中国 ?

Version - v1.6.15-rc8
Verified fixed


Add vp2430 to the test environment

Let's wait for some successful test, at least basic-interfaces one.

@JanPrusinowski There is already older PR with this config: https://github.com/Dasharo/open-source-firmware-validation/pull/554

@philipandag  Consider looking at https://github.com/Dasharo/open-source-firmware-validation/pull/554 - maybe it has something more than present here already: https://github.com/Dasharo/open-source-firmware-validation/pull/554

The platform configs will be added in this PR https://github.com/Dasharo/open-source-firmware-validation/pull/554
This one is not needed anymore


paper1111: Blacklist ebcoo.net

paper1111 requests the blacklist of the website ebcoo.net. See the Metasmoke search here and the Stack Exchange search here.

Coverage remained the same at 63.155% when pulling 1bd3f683dea0dfeb88016ec2a3d51e918ad7ad32 on auto-blacklist-1512566323.1506896 into ae9166c54a09c7be69d2587b7e31e2b1fe17cb59 on master.

!!/approve


Github Action ignores baseline file

Describe the issue
Running checkov in github action (runs docker image bridgecrew/checkov:2.0.756) ignores the baseline, outputs all errors, and exits with non-zero status code.
Test 1
The path to the baseline file is defined in checkov config file. Test is KO. Checkov runs, but reports all errors, as-if the baseline did not exist, and Github action fails due to non-zero status.
Content of the config file is:
branch: master
baseline: charts/cardano/.checkov.baseline
directory:
- charts/cardano
download-external-modules: false
evaluate-variables: true
external-modules-download-path: .external_modules
framework:
- helm
output:
- github_failed_only
skip-check:
- CKV_K8S_21
- CKV_K8S_10
- CKV_K8S_11
- CKV_K8S_12
- CKV_K8S_13
skip-path:
- /cardano/templates/tests/*
- /redis/*

Github Action Logs (truncated):
running checkov on directory: charts/cardano
checkov -d charts/cardano   --quiet  --output sarif   --config-file charts/cardano/.checkov

       _               _
   ___| |__   ___  ___| | _______   __
  / __| '_ \ / _ \/ __| |/ / _ \ \ / /
 | (__| | | |  __/ (__|   < (_) \ V /
  \___|_| |_|\___|\___|_|\_\___/ \_/

By bridgecrew.io | version: 2.0.756
helm scan results:

Passed checks: 0, Failed checks: 24, Skipped checks: 0

Test 2
The path to the baseline is set in github action YAML settings. Test is KO. Checkov runs, but reports all errors, as-if the baseline did not exist, and Github action fails due to non-zero status.
Github Action Logs (truncated):
running checkov on directory: charts/cardano
checkov -d charts/cardano   --quiet  --output sarif   --config-file charts/cardano/.checkov    --baseline charts/cardano/.checkov.baseline

       _               _
   ___| |__   ___  ___| | _______   __
  / __| '_ \ / _ \/ __| |/ / _ \ \ / /
 | (__| | | |  __/ (__|   < (_) \ V /
  \___|_| |_|\___|\___|_|\_\___/ \_/

By bridgecrew.io | version: 2.0.756
helm scan results:

Passed checks: 0, Failed checks: 24, Skipped checks: 0

Test 3 (running the command line locally)
$ checkov -d charts/cardano --baseline charts/cardano/.checkov.baseline --config-file charts/cardano/.checkov
| check_id   | file   | resource   | check_name   | guideline   |
|------------|--------|------------|--------------|-------------|

---

$ checkov --version
2.0.735

Discussion is on the matching issue in checkov :), https://github.com/bridgecrewio/checkov/issues/2272

I'm closing this one since the matching issue was resolved.


ARROW-9489: [C++] Add fill_null kernel implementation for (array[string], scalar[string])

The Python test fails for the large_binary test, with output:
        arr = pa.array([b'a', b'bb', None], type=pa.large_binary())
        result = arr.fill_null('ccc')
        expected = pa.array([b'a', b'bb', b'ccc'])
>       assert result.equals(expected)
E       assert False
E        +  where False = <built-in method equals of pyarrow.lib.LargeBinaryArray object at 0x7f762fb38520>(<pyarrow.lib.BinaryArray object at 0x7f762fb38600>\n[\n  61,\n  6262,\n  636363\n])
E        +    where <built-in method equals of pyarrow.lib.LargeBinaryArray object at 0x7f762fb38520> = <pyarrow.lib.LargeBinaryArray object at 0x7f762fb38520>\n[\n  61,\n  6262,\n  636363\n].equals

Could this be a bug in equals for LargeBinaryArray?

Looking at the implementation of VisitArrayDataInline, which is quite similar to what is being done in FillNullFunctor<Type, enable_if_t<is_number_type<Type>::value>>, I don't see much room for improvement.

@pitrou i think this is ready to go/review.


refactor(story): fold render-template into render-component

https://coveord.atlassian.net/browse/KIT-3447
Review by commit :)

[ ] Regression: #code-root is missing

My only issue with this is that the HTML tab is not really representative of the whole html that is presented in the storybook. Before the refactor, in the product template story,  it was shown that the template was surrounded by a product-list .

My only issue with this is that the HTML tab is not really representative of the whole html that is presented in the story. Before the refactor, in the product template story, it was shown that the template was surrounded by a product-list .
Example of before
To give more context and a better example, in the tab manager test PR. Felix added a story for the tab-manager. But when looking at the html tab, you really can't understand what is going on and why certain facets are being shown or hidden.

The HTML tab is meant to reflect what the component looks like, not the whole page and 'everything going on'.
If I continue on the example you shared, the recommendation, reflects 'noise' with parameters of the recommendation-list, which is not the goal.
For more 'integrated' showcases/examples, I think MDX stories would be best.


Add info about deprecated flags to upgrade release doc(s)

Re: Upgrade to CockroachDB v19.1
Issue Description
customer question was whether they should make a switch from the deprecated start-up scripts using --host and --port to --listen-addr:port and/or --advertise-addr:port as part of the upgrade. The only area of the docs this is mentioned is in cockroach start.
Suggested Resolution
Change in behavior like moving from deprecated flags/commands/etc certainly should be a part of upgrade doc and/or known limitations for a particular release. I also recommend adding this section to every upgrade release doc for 19.2, 20.1, etc.

CC @jseldess this is another customer corner case

IDK if I'd classify deprecated flags/features as a "known limitation". We document the deprecations on relevant pages, and on our GA release notes page for each release. For example: https://www.cockroachlabs.com/docs/releases/v20.2.0.html#deprecations
If a deprecated feature results from (or is the result of) a known limitation, we should add that individual limitation to the page.
I'm going to remove the T-known-limitation flag from this issue.

Yes, deprecated flags should be called out in our GA release notes, and we've been trying to do that as of 20.2. This issues is about flags deprecated several releases ago, so I think it's ok to close. But @dbist, your point is well taken.


Update question Github issue template

Update question Github issue template to refer people to Slack channel, and not StackOverflow

Pull request information

Status: ready to merge
Kind of changes: new feature

Description

We have changed the way our users should ask question, to be through our Slack channel. However, this is the last piece that is still pointing to the StackOverflow :)

Pull Request Test Coverage Report for Build 7182

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 88.759%

Totals

Change from base Build 7085:
0.0%

Covered Lines:
9586

Relevant Lines:
10800

💛  - Coveralls


Company.service missing CEO

Describe the bug
CEO is missing from Company.service
Expected behavior
API docs include CEO in the payload
https://iexcloud.io/docs/api/#company
Versions
iexcloud_api_wrapper: 1.1.3
Edit:
KeyStats.service.d.ts is also missing the following:
ttmEPS
ttmDividendRate
dividendYield
nextDividendDate
exDividendDate
nextEarningsDate
peRatio
beta

Thanks for pointing out the issue. Now fixed.


Defer accessing config until after user-provided block is evaluated

This makes it possible for users to e.g. use dry-system plugins that add new settings, which otherwise raises a Dry::Configurable::AlreadyDefinedConfig error if any config has been accessed previously.
This would be fixed anyway once we address https://github.com/dry-rb/dry-configurable/issues/70, but that's a larger piece of work, and I'd like to make dry-system-rails more flexible in the meantime.
The reason I actually needed this is because I wanted to use the dry-system settings component, which requires use :env to be added within the Dry::System::Rails.container do ... end block. use :env adds a new setting, so we need to make sure this block is evaluated as early as possible to avoid that error above.
@solnic Does this look OK to you?

@solnic Does this look OK to you?

Yes but we need a test that covers this change too. Reproducing your use case in a spec would be enough.

@solnic OK, so short of a major overhaul of the whole testing infrastructure here (like we just discussed, to support testing specific aspects of this Railtie's behaviour in complete isolation), are you fine with the test I've just added now? It demonstrates the behaviour that this change supports.
And that test is red on master, green here.


TAB-8330 Vulnerable 3rd party component json-smart used

Fixed by upgrading json-path 2.7.0 to 2.8.0 and json-smart 2.4.7 to 2.4.10.
Fixed by adding json-smart in management-cli-oss maven build and adding it to 3rd party bom.

Why did they make json-smart a runtime dependency? Oh well.


HTML Pages all 404

Starting today, all of the HTML pages of my Kill The Newsletter Feeds are 404ing.
I typically open several newsletters in browser because they don't style well in my RSS reader.
https://kill-the-newsletter.com/alternate/<page-key>.html
This is happening on all of my feeds, not just one
Thanks!

Hi @SenorFusion,
Thanks for reaching out with the issue report.
The links should be https://kill-the-newsletter.com/alternates/<page-key>.html, not https://kill-the-newsletter.com/alternate/<page-key>.html (note the s in alternates). Can you please try those and see if they work for you?
I double-checked just now and the links Atom feed are correct (they’re the <link rel="alternate" ... /> under each <entry>). From where did you get the wrong links?

@leafac  Yes, I see what you mean. This may turn out to be a problem with my RSS reader.
-Fedora 30
-Liferea 1.13.5
What I've found:
-You are correct, the raw XML is providing the feed URL correctly.
-Somehow the headline (link to post) is getting converted to this format (I was incorrect in my OP):
https://kill-the-newsletter.com/alternate/<feed-key>/<page-key>.html
When I manually change that link to https://kill-the-newsletter.com/alternates/<page-key>.html it works fine.
It seems to be constructing an incorrect URL for the headline link based on what is in the feed.
Example shown in the pictures.

Upon further review, it seems like maybe the feedreader is expecting a URL in the <id> field and, not finding one, tries to construct one based on the feed URL combined with the item key:
<id>urn:kill-the-newsletter:<page-key></id>

No, the problem is that I deployed a new version of Kill the Newsletter! two days ago in which I changed the URLs.  It won’t be an issue for new entries coming up…
Mystery solved; I’m closing the issue for now. Let me know if it’s a problem with new entries.
And thanks for using Kill the Newsletter! 🙌


Difference between bokeh and mpl measles_example

In bokeh:

and matplotlib:

It seems to be down to logz=True not working right for matplotlib:

What version of matplotlib do you have? When I activate logz it seems to come out right:

Now enabled it in https://github.com/ioam/holoviews/pull/2629, but worth figuring out why it works for me.

What version of matplotlib do you have?

That would be '2.2.2'

If it helps, I do get these warnings:
/Users/jstevens/miniconda3/envs/analytics/lib/python3.6/site-packages/matplotlib/colors.py:1031: RuntimeWarning: invalid value encountered in less_equal
  mask |= resdat <= 0
/Users/jstevens/miniconda3/envs/analytics/lib/python3.6/site-packages/matplotlib/colors.py:504: RuntimeWarning: invalid value encountered in less
  xa[xa < 0] = -1

Urgh. Upgrading numpy fixed it. So I guess we need to pin numpy:
conda install numpy
Solving environment: done

## Package Plan ##

  environment location: /Users/jstevens/miniconda3/envs/analytics

  added / updated specs:
    - numpy

The following packages will be downloaded:

    package                    |            build
    ---------------------------|-----------------
    numba-0.36.2               |np114py36hc2f221f_0         2.3 MB

The following packages will be UPDATED:

    numba: 0.36.2-np113py36hceb9f0c_0 --> 0.36.2-np114py36hc2f221f_0
    numpy: 1.13.3-py36ha9ae307_4      --> 1.14.2-py36ha9ae307_1

Very mysterious! I think we are out of ideas so @jbednar maybe it would be instructive if you could try this example and report what you see?

No problem with logz with mpl for me:

though the colors do seem fairly different from the Bokeh version

Also, I notice that 'Alaska' has a white row at the start in the bokeh version (presumably missing data) but I don't see that in your screenshot. They do seem to be quite different.

It looks like one version (presumably matplotlib) is incorrectly inverted along the y-axis.

It looks like one version (presumably matplotlib) is incorrectly inverted along the y-axis.

Agreed. Perhaps this issue can aim to fix that bug and ignore the logz bug as I seem to be the only one affected by it so far (with no idea why?)


failed to send batch: ERROR: column "is_anonymous" of relation "users" does not exist (SQLSTATE 42703)

Describe the bug
An error occurs while updating locally to a remote DB.
To Reproduce
Steps to reproduce the behavior:

npx supabase db pull
npx supabase db dump --data-only -f supabase/seed.sql
npx supabase db reset

Expected behavior
There is no is_anonymous column in the users table of the local auth db.
Screenshots

System information
Rerun the failing command with --create-ticket flag.

Ticket ID:
Version of OS: MacOS 14.2.1
Version of CLI: 1.136.3
Version of Docker:
Versions of services:

Additional context

Version of Node.js v18.17.1

Based on output from supabase services, it appears that your remote gotrue version is ahead of local. You should be able to fix it by relinking your project, ie.
npx supabase db pull
npx supabase db dump --data-only -f supabase/seed.sql

npx supabase stop --no-backup
npx supabase link
npx supabase start

Based on output from supabase services, it appears that your remote gotrue version is ahead of local. You should be able to fix it by relinking your project, ie.
npx supabase stop --no-backup
npx supabase link
npx supabase start

this works -- thank you 🙌
follow up question:

is there a way to manually upgrade the supabase services version?
it turns out that after unlinking from the project, the local version defaults back to previous ones.

reasons:

want to ensure dev's locally hosted supabase is up-to-date without linking to staging/prod db
want to minimize chances of messing with staging/prod db

is there a way to manually upgrade the supabase services version?

There's a supabase/.temp directory created after linking that contains the service version files. Unlinking will delete them but you can recreate manually to have CLI pick up those updated versions.
Bear in mind that although it works in principle, we can't guarantee every combination of service version works due to the maintenance complexity involved.

There's a supabase/.temp directory created after linking that contains the service version files. Unlinking will delete them but you can recreate manually to have CLI pick up those updated versions.
Bear in mind that although it works in principle, we can't guarantee every combination of service version works due to the maintenance complexity involved.

awesome! yup, i made sure to use a combination of service versions that are proven to work (i.e. staging/prod)
also, supabase unlink removes the entire .temp folder (there is no way to keep the folder after unlinking), so i simply removed the project-ref file containing the project id.

thanks for your help @sweatybridge


Swiping to delete an item in table view, you will see a white view

solution: https://github.com/yoha/Ventori/commit/49757101844cc596f44181415fa21b68ce804950


Do you have any examples on how to create provider that supports resource owner password credentials?

I see that the library supports it:
https://github.com/idan/oauthlib/blob/master/oauthlib/oauth2/rfc6749/grant_types/resource_owner_password_credentials.py

Apologies for kicking up an old thread, but I've got myself into a bit of a kerfuffle.
I've selected the Resource Owner Password Credentials grant for my website API (users log into my own website via a log in form - once signed in, they go to the restricted area which makes use of my own API).
I've started implementing my own RequestValidator, using methods required for the Resource Owner Password Credentials grant; however, I'm not too sure what authenticate_client is used for in this instance?


Struct enums are not implemented as enums of records in the codegen.

Let's consider the following enum in Rust:
enum Point {Cartesian {x: f64, y: f64}, Polar {r: f64, phi: f64} }

Currently the codegen assumes that the corresponding Gluon definition looks like this:
type Point = | Cartesian Float Float | Polar Float Float

However, a reasonable person would expect the following Gluon type instead:
type Point = | Cartesian {x: Float, y: Float} | Polar {r: Float, phi: Float}

Is there any reason to not use this more intuitive form?
Context:
https://github.com/gluon-lang/gluon/blob/master/codegen/tests/derive_getable.rs#L107
https://github.com/gluon-lang/gluon/blob/master/codegen/tests/derive_pushable.rs#L235

Is there any reason to not use this more intuitive form?

I tried really hard to come up with the reason I did it this way, but I'm really not sure anymore. I think it was mainly because these two variants would the  map to the same Gluon record:
enum Enum {
    StructVariant { x: f64, y: f64 },
    TupleWithStruct(Point),
}

struct Point {
    x: f64,
    y: f64,
}

... but that's still not really a good justification. I guess people just rarely use struct variants, so nobody noticed.
I would agree that your solution is much more reasonable! :)

The record form does require an extra allocation to create the record, but I agree that is a better form 👍


Add deprecation warning on PetSet guide.

cc @erictune @foxish @kow3ns @janetkuo @kubernetes/sig-apps
Note that I flipped the petset.yaml back to v1alpha1 since the goal is for this document to still apply to pre-1.5 users. We may need to exclude it from the examples_test somehow since I don't think the validation code knows about PetSet anymore.

This change is

Also note that I've left the text of the deprecated document for users who don't want to upgrade past 1.4 as discussed in https://github.com/kubernetes/kubernetes.github.io/pull/1707#issuecomment-265288828.

Added a "Warning" note at the start to help make the notice more prominent. I'll add it to the other PR as well.


Fix  passing of the arguments in our Logger implementation

Fix logging regression after #500
Before this patch:
I0519 17:35:08.051810 02152 glog.go:50] --> Uploading ["/tmp/s2i052311221/upload/scripts/run" "/tmp"] to %!q(MISSING) ...

After this patch:
I0519 17:42:06.734580 03500 glog.go:50] --> Uploading "/tmp/s2i886301399/upload/scripts/run" to "/tmp" ...

PTAL @bparees @smarterclayton

Lgtm [merge]


SDK-762/Fix regression in JS SDK where for a onChange event card icon…

… does not go back to default icon  when value is empty
Signed-off-by: Shaik-Luqmaan<EMAIL_ADDRESS>

Diff is 0.00%


Create valuable-drop-prices

Simple plugin that is meant to replace the default drop notifications while adding high alch prices to the chat notification.
User can change whether to see high alch or grand exchange prices, or to just show them both. Other than this they can also decide if ha or ge is is deemed valuable, or if it should use both (highest valued item).
Colour of chat notification can also be modified.

Can this be made a lot simpler and durable by just piggybacking off the in-game chat message, instead of disabling it? You can edit the existing message node to append the GE/HA values, which would also remove the need for doing your own drop tracking.
The whole plugin could be ~15 lines of code, excluding boilerplate.
Disabling the native drop notifications will conflict with a lot of other popular plugins.

Can this be made a lot simpler and durable by just piggybacking off the in-game chat message, instead of disabling it? You can edit the existing message node to append the GE/HA values, which would also remove the need for doing your own drop tracking.
The whole plugin could be ~15 lines of code, excluding boilerplate.
Disabling the native drop notifications will conflict with a lot of other popular plugins.

That's a good idea, I don't know why I didn't think of that. I'll modify the code to piggybank off the default message instead of the current way I'm doing it :) Thank you!


x/pkgsite: inconsistency major version between pseudo version and base version

What is the URL of the page with the issue?
<EMAIL_ADDRESS>What did you do?
https://github.com/ricoberger/script_exporter has released an valid version "v2.18.0" (commit id: ebeaf9415eb1) (just an example no offence)
However script_exporter's mod has no major suffix with "/v2", but<EMAIL_ADDRESS>is an valid "pseudo version" according to "mod spec"
My concern is clearly v2 incompatible with v1, but import "v1.3.1-0.20240210123610-ebeaf9415eb1" that will breaks minimal version selection since both version (v1.3.0 and v1.3.1-0.20240210123610-ebeaf9415eb1) has same major version.
What did you see happen?
<EMAIL_ADDRESS>is valid version
What did you expect to see?
No valid version if pseudo version related commit is after different major semver tag.

Is this an issue for pkgsite or cmd/gp more generally?
I think this is working as intended, where a commit can have multiple identities, and while go generally relies on human convention to enforce semver/major version suffix matches for manual tags, all commits will have a an identity they can be imported as.
For a while, it was a well-known workaround to import prometheus this way (before it adopted the /v2 suffix).
cc @bcmills

I think this is a pkgsite(proxy) issue since bad version will be generate go.mod and zip (https://proxy.golang.org/github.com/ricoberger/script_exporter/@v/v1.3.1-0.20240210123610-ebeaf9415eb1.mod), which can be download by non-go tools like curl, etc.

My concern is clearly v2 incompatible with v1, but import "v1.3.1-0.20240210123610-ebeaf9415eb1" that will breaks minimal version selection since both version (v1.3.0 and v1.3.1-0.20240210123610-ebeaf9415eb1) has same major version.

Note that go mod tidy, go get -u, and similar operations prefer release versions over pseudo-versions — to add a dependency on a pseudo-version, you have to request its commit or pseudo-version explicitly. This pseudo-version behavior is fundamentally no different from accidentally introducing a bug or incompatibility during development.
And go get with the v2 release version shows an unambiguous error in this case:
$ go get<EMAIL_ADDRESS>go<EMAIL_ADDRESS>invalid version: module contains a go.mod file, so module path must match major version ("github.com/ricoberger/script_exporter/v2")

Also note that there is nothing preventing the owner of the module from adding a tag like v1.4.0 to that same commit to make it unambiguously part of the v1 line — the go command has no way of knowing whether the problem is the v2.18.0 tag or the non-v2 module path in the go.mod file.

Also note that<EMAIL_ADDRESS>displays a Go to latest button that leads to v1.3.0 — so it is also directing users to the released version rather than a pseudo-version.


Added None in some flags to fix HasFlags method

Specifically Linearstar.Windows.RawInput.Native.RawKeyboardFlags and Linearstar.Windows.RawInput.Native.RawMouseFlags

For now I reverted my changes and renamed the first flag to None to avoid confusion.
I also changed the name of LeftKey and RightKey to KeyE0 and KeyE1 seeing as they are supposed to be used as flags for if the scancode is preceded by E0 or E1 (https://www.win.tue.nl/~aeb/linux/kbd/scancodes-1.html)


Ruby code doesn't beautify after pressing Ctrl + Alt + B

Description
When I press Ctrl + Alt + B, I see a progress window, and then a warning:
The "Ruby - Rubocop Path (rubocop_path)" configuration option has been deprecated. Please switch to using the option in section "Executables" (near the top) in subsection "Rubocop" labelled "Path" in Atom-Beautify package settings.

After that my file content left as it was before. No beautification applied.
The results of beautification are not what I expect.
Input Before Beautification
This is what the code looked like before:
# frozen_string_literal: true

# This class is doing some heavy work
class HeavyWorker
  def work
    500.times { |i| i.times {} }
    more_work
  end

  def more_work
    500.times { |i| i.times {} }
end
end

Expected Output
The beautified code should have looked like this:
# frozen_string_literal: true

# This class is doing some heavy work
class HeavyWorker
  def work
    500.times { |i| i.times {} }
    more_work
  end

  def more_work
    500.times { |i| i.times {} }
  end
end

Actual Output
The beautified code actually looked like this:
# frozen_string_literal: true

# This class is doing some heavy work
class HeavyWorker
  def work
    500.times { |i| i.times {} }
    more_work
  end

  def more_work
    500.times { |i| i.times {} }
end
end

Steps to Reproduce

Add code to Atom editor
Run command Atom Beautify: Beautify Editor
This beautified code does not look right!

Debug
Here is a link to the debug.md Gist: https://gist.github.com/aruprakshit/7cfa781169850350ba1b29939cdc8d40
Checklist
I have:

[x] Tried uninstalling and reinstalling Atom Beautify to ensure it installed properly
[x] Reloaded (or restarted) Atom to ensure it is not a caching issue
[x] Searched through existing Atom Beautify Issues at https://github.com/Glavin001/atom-beautify/issues
so I know this is not a duplicate issue
[x] Filled out the Input, Expected, and Actual sections above or have edited/removed them in a way that fully describes the issue.
[x] Generated debugging information by executing Atom Beautify: Help Debug Editor command in Atom and added link for debug.md Gist to this issue

@aruprakshit Did you follow the instructions and verify that the rubocop executable is set up properly on your system? e.g.:
$ which rubocop
/Users/me/.rvm/gems/ruby-2.4.1@global/bin/rubocop

@ggriffin Yes I did. Does that mean every time, I switch to different Ruby version using Rvm, I need to go and change the settings manually in Atom? Ca't it detect from the current Ruby version selected by RVM?

@aruprakshit Sorry I missed this issue somehow.  That message is saying that the place to specify the path to the executable has changed.  Assuming you have it under Ruby -> Rubocop Path right now, remove it from there and put it under Executables -> Rubocop -> Binary/Script Path.

If anyone still having this issue, kindly see the below pic.
 Setting -> Executables -> Rubocop -> the_path_rubocop


changed makefile to give correct example

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

Hi @tuskiomi ,
As per discussion here, actual /cygpath/ style here probably won't work unless make is a MINGW32 make:
https://esp32.com/viewtopic.php?f=2&t=2173&p=10320
Maybe it's more accurate to  describe the the /c/xyz style as MINGW32-style?
Angus


Use semantic versioning for future releases

Is your feature request related to a problem? Please describe.
It would be great if Faker could use semantic versioning for future releases. Any breaking changes ("deprecations") should bump the major version number, so it's obvious to users that things may break. Then in my Gemfile I can specify (e.g.) ~> 1.9 to know that I will receive fixes for a specific version, but nothing will break.
Describe alternatives you've considered
The alternative is locking the version faker in my Gemfile to a specific version. Any upgrades will be as they are now, painful and involve being forced to look through the changelog to find what has changed, to be able to fix my code. If there are any security updates (ok, unlikely for a gem such as Faker, but you never know) I will miss out on them until I check to see what has changed.
Related issues that could be avoided by using semantic versioning:

https://github.com/stympy/faker/issues/1576
https://github.com/stympy/faker/issues/1642
https://github.com/stympy/faker/issues/1088

@lucaspiller thanks for raising this issue! We've been discussing this internally and will come up with semantics for v2. I'd encourage you to open a PR and propose the semantics in our contributing.md, so other members can interact and increment.


[CI/CD] Creating release subproject for jenkins to preform releases

OpenShift console extension only requires *.js, *.css files to run the extension on openshift. I want to trigger this build to cut a release so I can fetch it from github release page for this project instead of performing manual builds.
prerequisite is to have jenkins perform the following before cutting the release zip file
gem install bundler
sudo npm install
bower install
grunt build

I've removed that from the instructions. I'd like to see if this can get merged as I've got the building running off my local fork with this branch

does the release get pushed to github?

That is the next step in the process. For now this additional change only creates the zip there will be more work done on jenkins to do the full release

The release gets posted as an artifact in jenkins. But I'm planning on integrating github release next. For now I'll need to get this merged before I do that next step.


Fix minor typo

@radujipa : Thanks for your contribution! The author(s) have been notified to review your proposed change.

The typo fix looks good; thanks.


Adds instructions to docker testing file

Adds a comment block containing instructions for executing the
run_docker_test file in standalone. Applies PeP8/lint fixes and
removes orphan imports, functions.
See also: #54, #40

Wrong fork . :(


Update stdevp-transact-sql.md

Make OVER clause optional.

@daschall : Thanks for your contribution! The author, @edmacauley, has been notified to review your proposed change.

@daschall Thank you! Signing off.


No syntax highlighting in the submitted solution

I submitted a solution for hello-world, got no syntax highlighting on the code.

/cc @kytrinyx. GitHub has Idris highlighting support or worst case, Haskell highlighting would probably be better than nothing.

Does http://github.com/jneen/rouge have idris support under a different name? If so we can map it to what we have in https://github.com/exercism/exercism.io/blob/master/lib/exercism/syntax_highlighter.rb#L6-L15

Looks like it doesn't. Purportedly Pygments style sheets are compatible and iirc Pygments supports Idris. Could we adapt it for Exercism?

If pygments supports it, how hard would it be to add it to rouge?

I just looked into the pygmens repo, I wasn't able to find a lexer for idris, also it is NOT listed on the page of supported languages.
So there is no pygments support we could port.
So I forked rogue yesterday and played around a bit, it seems to be not that hard to implement new lexers. The DSL is pretty simple. A problem is though, that there is no formal description of Idris' syntax and the parser code of the compiler itself... Well it needs to get cleaned up a bit ;) So I can't extract the necessary information from there.
I'm slowly starting to adapt some snippets from the Haskell Lexer though. But since I do currently have only a very limited amount of spare time, it might take a while until I have something I could PR to rogue.

Not sure if this would be of any use: https://github.com/david-christiansen/idris-code-highlighter

If nothing else, you might be able to reverse engineer the output of idris --highlight (which doesn't need that repo).

Davids Highlighter transforms the output of idris --highlight into HTML or LaTeX, so it relies on Idris beeing installed, and does not actually parse/scan idris sources, but a preformatted output. that is meant to be read by machines.
Also reverse engineering idris --highlight will be near to impossible because of semantic highlighting. Also it works on the AST which was generated by the parser, which is a little bit messy and not really straightforward.

Quite right. There ought to be formal description somewhere..

There's something on readthedocs which is a mixture of semi formal
description and examples...
Some parts are defined by some bnf style text, while others are only by
example. Eg pragmas are bnf style, operators only by example and only a
very small subset, arithmetic and compare...
Since it is 1.0 now perhaps we will have luck asking on the issue tracker
or mailing list.
Eric Bailey<EMAIL_ADDRESS>schrieb am Sa., 10. Juni 2017, 10:19:

Quite right. There ought to be formal description somewhere..
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/exercism/xidris/issues/15#issuecomment-307550925, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AADmR7SRVbxVgpmhJP75dLowD4LehBYjks5sClGegaJpZM4MGFUu
.

I'd recommend IRC too.

It looks like maybe GitHub uses this for GFM.

Thanks. I can't read text mate, but at least I should be able to use it as
a base...
Eric Bailey<EMAIL_ADDRESS>schrieb am Sa., 10. Juni 2017, 10:30:

It looks like maybe GitHub uses this
https://github.com/idris-hackers/idris-sublime for GFM.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/exercism/xidris/issues/15#issuecomment-307551380, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AADmR1mFdQJu4jiM33aZGzR7FFlf4sYzks5sClQmgaJpZM4MGFUu
.


feat(cli): use insta_cmd for cli snapshot testing

closes #776

Codecov Report
Patch coverage: 100.00% and no project coverage change.

Comparison is base (1c6b87b) 92.54% compared to head (525d225) 92.54%.
Report is 2 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #791   +/-   ##
=======================================
  Coverage   92.54%   92.54%
=======================================
  Files         203      203
  Lines       41811    41827   +16
=======================================
+ Hits        38693    38709   +16
  Misses       3118     3118

Files Changed
Coverage Δ

crates/oxc_cli/src/command.rs
89.74% <100.00%> (+1.01%)
:arrow_up:

... and 1 file with indirect coverage changes

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


fix typo in validation regex for get-org-membership

Looks like this should be an "enum" with only two valid values?
https://developer.github.com/v3/orgs/members/#list-your-organization-memberships

Yep, thanks!

Pushed as 6.0.2.


Notifications

Get email notification notifications when your app errors or OpenAI is down.

Evaluate if this is worth
https://github.com/Helicone/helicone/issues/219


org chart team strange

updating org chart "Contributors" for version Strange

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 35.544%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
5174

Relevant Lines:
13605

💛  - Coveralls


Atom window disappears

Description
When launching the Atom editor, the window never appears. I can see it in my taskbar, and the preview looks fine. I have tried disconnecting extra monitors, and I have tried to open extra windows, but none of them appear, they are hidden in some magical way.
Steps to Reproduce
I have no idea.
Versions
Atom Version 1.7.3

What OS is this on?  In the future, please fill out the entirety of the issue template.

Sorry.
I'm on Windows 10.

If you shift-right click on Atom, you should be able to restore/maximize/move Atom.  Can you try doing that and see if you can get it to show up?

Also try pressing the Windows key and an arrow once you've pressed the icon in the task bar.
Windows is very bad about positioning Windows off-screen if you have changed resolution/external monitors :(

Refs/Duplicates that I stumbled upon a few days ago and have been meaning to comment on: https://github.com/atom/atom/issues/8737 https://github.com/atom/atom/issues/6939 https://github.com/atom/atom/issues/9172 https://github.com/atom/atom/issues/9223

Thanks @Ben3eeE. Closing this (and the others) as duplicate of #6939.


Cartesian product of constraint sets

The idea would be to introduce a child class of ConstraintSetBase<Scalar> which represents the Cartesian product of multiple constraint sets
  \mathcal{C} =
  \mathcal{C}_1 \times \cdots\times
  \mathcal{C}_n

and implements the relevant projection and subdifferential operators.
This would be meant mainly for internal use in the solver and help simplify its implementation (eliminate loops, manual extraction of relevant segments/blocks in vectors/matrices).

I ended up adding this to aligator itself in the next release. I'll backport.