Improve UX of crashed Notifications

As ...
I want ...
so that ...
Hints / Details
The problem:
A notification was previously created and is now "approved" by a supervisor.
Now this notification should be sent to the EDC.
The backend does not represent that status if EDC is down or responds with an error.
The notification displays the success but the status will stay as it is -> From queued to queued.
From technical perspective this is correct, but the user needs a clear notification what happened.
Possible improvements:

Display appropriate error handling.
If EDC is down update the status to "sending..."
Retry a couple of times automatically
a. Update status to "failed to send" if notification is not sent
Provide "Retry" or standard "approve" and "cancel" options for "failed to send" notification
Provide detailed information on detail paged why the notification was not sent
If a webhook is already in place, inform the user that the notification was not sent

Acceptance Criteria
[x] Backend has been adapted to reflect if the notification status could be transfered to the next status.
[x] Frontend displays a success if the notification could be transfered
[x] Frontend displays a error and asks for retry of the user.
Out of Scope

...

https://jira.catena-x.net/browse/TRACEFOSS-924

Hey @ds-crehm  this might be a topic for you?

Implemented change that when there is an error in the creation of a notification, the error message from BE and a try again button is displayed:

By clicking on the try again button, the creation of the notification gets executed again.
This means, the FE is expecting an error with an appropiate message from the Backend. Currently the message is pulled from the Http response field 'error' which contains the string error message. BE needs to implement the error message in the response and with that the issue is done

Hi @ds-crehm feature has been installed on e2e environments. Testing can be done by sending an alert with an part which owns the company and using the bpn of the other company.
Example:

Opening e2e-a
Open parts
Select part
Start alert
Use bpn: BPNL00000003CNKC which is e2e-b
Create alert
Go to created alerts
Approve alert
Expected behaviour: Error message comes up with a retry button.

Repeat the steps in a successful scneario and you will see a success info.

Test successful (tested on E2E-A):

Alert stays in status "QUEUED"
Clicking on "Try again" brings back the approval window.
Clicking on "X" closes the popup
Popup disappears after a couple of seconds

Notes:

Error message was not very detailed in my test case.
The parts where I can hover/click are not accurate:

I can hover and click in the entire left area to highlight and activate the "Try again"-button. But the cursor only changes within the actual "Try again"-button.
For the "X" anywhere on the right area can be clicked to close the notification.

Hi @ds-crehm ,
issue is fixed and merged with https://github.com/catenax-ng/tx-traceability-foss/pull/1041. Ready to recheck, thanks!

Tested again. Now the hoverable/clickable areas are accurate.


About the icon SVG file will be updated every time official sync

Describe the bug
I use official synchronization to synchronize data between WIndows10, Android, and iOS devices
I found that every time I sync, I will re-sync the icon pack in the icon folder plugin
So I wonder if the modification time of the svg file of the icon pack is the latest every time, which triggers the official synchronization mechanism
Unfortunately the SVG files I see from my win10 device, indeed all modification times are up to date, but it hasn't been changed for a long time since I configured the icon folder
To Reproduce
Steps to reproduce the behavior:

Open any device of windows10, Android, iOS
Trigger official synchronization and start synchronizing data
It can be seen that every time it is synchronized, the Svg file of the icon pack will be re-synchronized (I only keep one icon pack in the icon folder, so I can recognize it)
This icon pack contains 1800+ SVG files, so each synchronization will take a considerable amount of time
The crux of the problem is that it has not been changed for a long time after I complete the folder configuration through the icon folder, and I only use 2 icons, so this problem really bothers me.
As can be seen from the figure, there are 1820 files, the last modification time was about 20 minutes ago, and all SVG files have this modification time. But the last time I configured the icon folder was half a month ago, and I only used 2 icons

Expected behavior

I hope that every time I synchronize, I don’t want to synchronize 1820 SVG files all over again

Unless the icon foder updates the configuration, it does not need to be synchronized

Hope to solve this problem soon
If you need any support from me, please feel free to contact me
@FlorianWoelki
looking forward to reply~

I have the same issue, might be because I sync between a Windows and OSX host and file compare has difficulties.
Workaround:
Add exclude from sync entry: .obsidian/plugins/obsidian-icon-folder/icons, and repair the Icon Packs afterwards.

I have the same issue, might be because I sync between a Windows and OSX host and file compare has difficulties. Workaround: Add exclude from sync entry: .obsidian/plugins/obsidian-icon-folder/icons, and repair the Icon Packs afterwards.

Thanks for the answer~
Does shielding need to be set on all terminals used?
For example, if I use window10+Android+iOS, then I only need to set up the PC, or do I need to set up the PC+two mobile phones?

I have the same issue, might be because I sync between a Windows and OSX host and file compare has difficulties. Workaround: Add exclude from sync entry: .obsidian/plugins/obsidian-icon-folder/icons, and repair the Icon Packs afterwards.

Thanks for the answer~
Does shielding need to be set on all terminals used?
For example, if I use window10+Android+iOS, then I only need to set up the PC, or do I need to set up the PC+two mobile phones?

It seems like you've to setup this exclution manually, once on each device you use Sync with and then repair the Icon Packs as well on each device to be sure the Icons are working correctly.

I have the same issue, might be because I sync between a Windows and OSX host and file compare has difficulties. Workaround: Add exclude from sync entry: .obsidian/plugins/obsidian-icon-folder/icons, and repair the Icon Packs afterwards.

Thanks for the answer~
Does shielding need to be set on all terminals used?
For example, if I use window10+Android+iOS, then I only need to set up the PC, or do I need to set up the PC+two mobile phones?

It seems like you've to setup this exclution manually, once on each device you use Sync with and then repair the Icon Packs as well on each device to be sure the Icons are working correctly.

OK , I try it. thanks~

Silly question maybe, but how do I exclude the icons folder from sync? I don't seem to be able to exclude anything from my .obsidian folders - but maybe I'm holding it wrong :-)

Silly question maybe, but how do I exclude the icons folder from sync? I don't seem to be able to exclude anything from my .obsidian folders. Putting the path in and clicking "Done" doesn't work for this folder - but maybe I'm holding it wrong :-)

Press enter after typing the path.

wow - thanks! I do feel silly now :-)

This is an official syncing issue and a possible solution might exist in #153.
The issue will be probably solved in v2.0.0 (#159)
Closing this issue for now.

Reply to this question. First of all, I am very grateful to the author for his hard work. I have been using this plugin and I have always liked it, but unfortunately this problem still exists, even if I update to v2.0.1
Let me introduce my solution below, and the effect is satisfactory：
Deleted the original icon pack, created a new custom icon pack folder, and added the required icon SVG files into it
After that, even if the plug-in is synchronized all the time, it only synchronizes these two files, which reduces the number and pressure of synchronization. It is an indirect solution.


Audio to Text Transcription

Description of the new feature / enhancement
To include audio to text transcription feature using AI through dictating or uploading an audio file.
Scenario when this would be used?
For Subtitling and Work related tasks where we can avoid the task of typing it manually.
Supporting information
No response

https://support.microsoft.com/en-us/windows/use-live-captions-to-better-understand-audio-b52da59c-14b8-4031-aeeb-f6a47e6055df

Awaiting your response

A few points:

Corporations do not upgrade to newer Windows versions as quickly as the real world. As a result, features and updates bundled/baked into the OS can be delayed by many months, if not years.
A meeting transcript feature is still needed and PowerToys may be the best option for Corporate users.
Many cell phone has better offline transcription capabilities than those available to my current Windows 10 Enterprise 22H2.
There are a variety of Windows applications which can record meetings and presentations to help capture and create subsequent meeting notes. PowerToys offers more flexibility to accommodate various input sources.
Most companies block access to external transcription services to prevent IP theft or security violations.

Given these observations, can PowerToys provide a local transcription capability that is more advanced than Windows/Office/365 by analyzing an audio/video/device stream and saving a transcript of the source to a local text file?
Bonus points for recognizing and labeling speakers while adding timestamps.

Can this please be added? The benchmark to beat is WhisperTyping.


Substraction precision is inaccurate

Here is an example code to show the substraction inaccuracy even with small values:
void main(){
  print(1 - 0.1); // 0.9
  print(1 - 0.2); // 0.8
  print(1 - 0.3); // 0.7
  print(1 - 0.4); // 0.6
  print(1 - 0.5); // 0.5
  print(1 - 0.6); // 0.4
  print(1 - 0.7); // 0.30000000000000004
  print(1 - 0.8); // 0.19999999999999996
  print(1 - 0.9); // 0.09999999999999998
}

flutter doctor -v
[✓] Flutter (Channel stable, 1.20.4, on Mac OS X 10.15.5 19F101, locale de-DE)
    • Flutter version 1.20.4 at /Users/nico/OneDrive/Development/_Libraries/flutter
    • Framework revision fba99f6cf9 (4 days ago), 2020-09-14 15:32:52 -0700
    • Engine revision d1bc06f032
    • Dart version 2.9.2

[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.2)
    • Android SDK at /Users/nico/Library/Android/sdk
    • Platform android-29, build-tools 29.0.2
    • ANDROID_HOME = /Users/nico/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6222593)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 12.0)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.0, Build version 12A7209
    • CocoaPods version 1.8.4

[!] Android Studio (version 4.0)
    • Android Studio at /Applications/Android Studio.app/Contents
    ✗ Flutter plugin not installed; this adds Flutter specific functionality.
    ✗ Dart plugin not installed; this adds Dart specific functionality.
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6222593)

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

[✓] Connected device (1 available)
    • iPhone 11 Pro (mobile) • F3CD1FAA-E7A4-414F-9791-C8775BCB42A7 • ios • com.apple.CoreSimulator.SimRuntime.iOS-14-0 (simulator)

Even web editors like dartpad.dev have this bug.

Hi @CLIDOapps , this isn't a bug with Dart, it is actually due to how real numbers are represented using "floating point" values. This practice is common with other programming languages that you might use today such as Java, JavaScript, C#.
There is a pretty good explanation on this stack overflow post here: https://stackoverflow.com/questions/588004/is-floating-point-math-broken

Hm, I'm a developer for 20 years, with many projects of excessively calculations in different languages. That is the first time I got this error. Even it is theoretically right for a floating point binary64 value, it is practically wrong. 1 - 0.7 is 0.3 and not other. Every calculator knows that.


getInitialProps query data changes when come back using safari

Bug report
Describe the bug
When come back from other site using safari browser (Mac, iPhone)
getInitialProps query data changes.
String, Array -> not changed
Boolean, Number -> into String
Object -> into Empty String
To Reproduce
Express.js server code
server.get('/test', (req, res) => {
  return app.render(req, res, '/test', {
    string: 'stringValue',
    boolean: true,
    object: {
      key1: 'key1Value',
      key2: true,
    },
    array: [
      'hello',
      1,
      false,
      { hello: 'world' },
    ],
  })
})

Page code
/* External dependencies */
import React from 'react'

/* Internal dependencies */

class Test extends React.Component {
  static getInitialProps({ query }) {
    console.log(query)
    return { ...query }
  }

  render() {
    return (

          Hello

        Go to google

    )
  }
}

export default Test

Steps to reproduce the behavior, please provide code snippets or a repository:

Click 'Go to google'
Slide touchpad right to go back
See terminal and safari console

Screenshots

System information

OS: macOS
Browser safari 10.0
Version of Next.js: 7.0.2
Express.js: 4.14.0

Additional context
Is this same issue?
https://github.com/zeit/next.js/pull/6652

This appears to be a different issue.

Also appear in next.js 8.0.4

This does appear to be related to #6652

waiting for 8.0.5 ?


[ Milestone 3 ]  Associations

Created a class Classroom with the following:

@label instance variable, should be initialized in the constructor.
Setter and getter for @label (remember about attr_accessor).

Created the has-many/belongs-to relationship between Classroom and Student.
Created a class Book with the following:

@title and @author instance variables, should be initialized in the constructor.
Setters and getters for instance variables (remember about attr_accessor).

Created a class Rental with the following:

@date instance variable, should be initialized in the constructor.
Setter and getter for @date (remember about attr_accessor).

Created the many-to-many (also has-many-through) relationship between Person and Book using the intermediate class Rental

Thank you for the review @tufoinnkuo10  😄


Cannot use r-tidyverse 2.0 (noarch) with R 4.4+

Comment:
Hi,
At https://github.com/bioconda/bioconda-recipes/pull/49285 I am trying to add aarch64/arm64 support for Bioconda strainr2 recipe but it fails with the following error:
07:09:39 BIOCONDA INFO (OUT) Could not solve for environment specs
07:09:39 BIOCONDA INFO (OUT) The following packages are incompatible
07:09:39 BIOCONDA INFO (OUT) ├─ r-base >=4.4,<4.5.0a0,>=4.4,<4.5.0a0  is installable and it requires
07:09:39 BIOCONDA INFO (OUT) │  └─ libdeflate >=1.20,<1.21.0a0 , which can be installed;
07:09:39 BIOCONDA INFO (OUT) └─ r-tidyverse is not installable because there are no viable options
07:09:39 BIOCONDA INFO (OUT)    ├─ r-tidyverse 1.2.1 would require
07:09:39 BIOCONDA INFO (OUT)    │  └─ r-base >=3.5.1,<<IP_ADDRESS>a0 , which does not exist (perhaps a missing channel);
07:09:39 BIOCONDA INFO (OUT)    ├─ r-tidyverse [1.2.1|1.3.0] would require
07:09:39 BIOCONDA INFO (OUT)    │  └─ r-base >=3.5,<3.6.0a0 , which does not exist (perhaps a missing channel);
07:09:39 BIOCONDA INFO (OUT)    ├─ r-tidyverse [1.2.1|1.3.0|1.3.1] would require
07:09:39 BIOCONDA INFO (OUT)    │  └─ r-base >=3.6,<3.7.0a0 , which conflicts with any installable versions previously reported;
07:09:39 BIOCONDA INFO (OUT)    ├─ r-tidyverse [1.3.0|1.3.1|1.3.2] would require
07:09:39 BIOCONDA INFO (OUT)    │  └─ r-base >=4.0,<4.1.0a0  but there are no viable options
07:09:39 BIOCONDA INFO (OUT)    │     ├─ r-base [4.0.0|4.0.1|4.0.2|4.0.3|4.0.5] conflicts with any installable versions previously reported;
07:09:39 BIOCONDA INFO (OUT)    │     ├─ r-base [4.0.5|4.1.0|4.1.1|4.1.2|4.1.3] would require
07:09:39 BIOCONDA INFO (OUT)    │     │  └─ libtiff >=4.3.0,<4.5.0a0  but there are no viable options
07:09:39 BIOCONDA INFO (OUT)    │     │     ├─ libtiff [4.3.0|4.4.0] would require
07:09:39 BIOCONDA INFO (OUT)    │     │     │  └─ libdeflate >=1.10,<1.11.0a0 , which conflicts with any installable versions previously reported;
07:09:39 BIOCONDA INFO (OUT)    │     │     ├─ libtiff 4.4.0 would require
07:09:39 BIOCONDA INFO (OUT)    │     │     │  └─ libdeflate >=1.12,<1.13.0a0 , which conflicts with any installable versions previously reported;
07:09:39 BIOCONDA INFO (OUT)    │     │     ├─ libtiff [4.4.0|4.5.0] would require
07:09:39 BIOCONDA INFO (OUT)    │     │     │  └─ libdeflate >=1.14,<1.15.0a0 , which conflicts with any installable versions previously reported;
07:09:39 BIOCONDA INFO (OUT)    │     │     ├─ libtiff 4.4.0 would require
07:09:39 BIOCONDA INFO (OUT)    │     │     │  └─ libdeflate >=1.13,<1.14.0a0 , which conflicts with any installable versions previously reported;
07:09:39 BIOCONDA INFO (OUT)    │     │     ├─ libtiff 4.3.0 would require
07:09:39 BIOCONDA INFO (OUT)    │     │     │  └─ libdeflate >=1.7,<1.8.0a0 , which conflicts with any installable versions previously reported;
07:09:39 BIOCONDA INFO (OUT)    │     │     └─ libtiff 4.3.0 would require
07:09:39 BIOCONDA INFO (OUT)    │     │        └─ libdeflate >=1.8,<1.9.0a0 , which conflicts with any installable versions previously reported;
07:09:39 BIOCONDA INFO (OUT)    │     └─ r-base [4.0.5|4.1.3|4.2.0|4.2.1|4.2.2] would require
07:09:39 BIOCONDA INFO (OUT)    │        └─ libtiff >=4.4.0,<4.5.0a0 , which cannot be installed (as previously explained);
07:09:39 BIOCONDA INFO (OUT)    ├─ r-tidyverse [1.3.1|1.3.2|2.0.0] would require
07:09:39 BIOCONDA INFO (OUT)    │  └─ r-base >=4.1,<4.2.0a0  but there are no viable options
07:09:39 BIOCONDA INFO (OUT)    │     ├─ r-base [4.0.5|4.1.0|4.1.1|4.1.2|4.1.3], which cannot be installed (as previously explained);
07:09:39 BIOCONDA INFO (OUT)    │     ├─ r-base [4.0.5|4.1.3|4.2.0|4.2.1|4.2.2], which cannot be installed (as previously explained);
07:09:39 BIOCONDA INFO (OUT)    │     ├─ r-base 4.1.3 conflicts with any installable versions previously reported;
07:09:39 BIOCONDA INFO (OUT)    │     ├─ r-base [4.1.3|4.2.2|4.2.3|4.3.0] would require
07:09:39 BIOCONDA INFO (OUT)    │     │  └─ libtiff >=4.5.0,<4.6.0a0  but there are no viable options
07:09:39 BIOCONDA INFO (OUT)    │     │     ├─ libtiff [4.4.0|4.5.0], which cannot be installed (as previously explained);
07:09:39 BIOCONDA INFO (OUT)    │     │     ├─ libtiff 4.5.0 would require
07:09:39 BIOCONDA INFO (OUT)    │     │     │  └─ libdeflate >=1.16,<1.17.0a0 , which conflicts with any installable versions previously reported;
07:09:39 BIOCONDA INFO (OUT)    │     │     ├─ libtiff 4.5.0 would require
07:09:39 BIOCONDA INFO (OUT)    │     │     │  └─ libdeflate >=1.17,<1.18.0a0 , which conflicts with any installable versions previously reported;
07:09:39 BIOCONDA INFO (OUT)    │     │     └─ libtiff [4.5.0|4.5.1] would require
07:09:39 BIOCONDA INFO (OUT)    │     │        └─ libdeflate >=1.18,<1.19.0a0 , which conflicts with any installable versions previously reported;
07:09:39 BIOCONDA INFO (OUT)    │     └─ r-base [4.1.3|4.2.3|4.3.0|4.3.1] would require
07:09:39 BIOCONDA INFO (OUT)    │        └─ libtiff >=4.5.1,<4.6.0a0 , which cannot be installed (as previously explained);
07:09:39 BIOCONDA INFO (OUT)    ├─ r-tidyverse [1.3.2|2.0.0] would require
07:09:39 BIOCONDA INFO (OUT)    │  └─ r-base >=4.2,<4.3.0a0  but there are no viable options
07:09:39 BIOCONDA INFO (OUT)    │     ├─ r-base [4.0.5|4.1.3|4.2.0|4.2.1|4.2.2], which cannot be installed (as previously explained);
07:09:39 BIOCONDA INFO (OUT)    │     ├─ r-base [4.1.3|4.2.2|4.2.3|4.3.0], which cannot be installed (as previously explained);
07:09:39 BIOCONDA INFO (OUT)    │     ├─ r-base [4.1.3|4.2.3|4.3.0|4.3.1], which cannot be installed (as previously explained);
07:09:39 BIOCONDA INFO (OUT)    │     └─ r-base 4.2.3 conflicts with any installable versions previously reported;
07:09:39 BIOCONDA INFO (OUT)    └─ r-tidyverse 2.0.0 would require
07:09:39 BIOCONDA INFO (OUT)       └─ r-base >=4.3,<4.4.0a0  but there are no viable options
07:09:39 BIOCONDA INFO (OUT)          ├─ r-base [4.1.3|4.2.2|4.2.3|4.3.0], which cannot be installed (as previously explained);
07:09:39 BIOCONDA INFO (OUT)          ├─ r-base [4.1.3|4.2.3|4.3.0|4.3.1], which cannot be installed (as previously explained);
07:09:39 BIOCONDA INFO (OUT)          └─ r-base [4.3.1|4.3.2|4.3.3] conflicts with any installable versions previously reported.

r-tidyverse 2.0.0 is the first noarch version of the recipe but for some reason it uses <4.4.0a0 that I cannot explain.
Any ideas what is the problem ?
Thanks!

I think the issue will be solved once https://github.com/bioconda/bioconda-recipes/pull/49091 is merged to Bioconda master!
Closing for now!
Sorry for the noise!


Delete duplicate assign statement

What was wrong?
computation_class = SpuriousDragonComputation has been assigned twice.https://github.com/ethereum/py-evm/blob/0e3663568651dcdcd8b7e4a1341219f9252bba96/eth/vm/forks/spurious_dragon/state.py#L39-L40
How was it fixed?
delete
Cute Animal Picture

Thanks!


Image has a long description attribute

Describe the new a11y feature or project enhancement

The longdesc attribute in images is deprecated and should not be used. It is not fully supported by HTML5, browsers, or screen readers. When present, it is commonly misused (it should point to an URL that is not an image).
Describe the solution you'd like

Show a warning if:

an image has the longdesc attribute. Recommend the removal of the attribute and adding the information on the same page (e.g. with an aria-describedby)

Show an error if:

The value of longdesc is not a URL (this may be tricky as it could be a relative path).
The value of longdesc contains spaces (then it would be text and not a URL)
The value of londgesc is a link to an image. This could be tested via CSS by checking if the end of the value is an image extension (e.g. ".jpg", ".jpeg", ".png", ".gif", etc.)
The value of longdesc is empty.

Testing the first two points may be tricky and lead to false positives (e.g. a URL that has not been encoded and contains white space). Because of that, maybe those two cases should be ignored.
Link(s)

Equalize Digital: Long Description Invalid
WebAIM: The WebAIM Million (longdesc)

This is a brilliant idea and a nice feature to add to a future version 👌

@jackdomleo7 , I would like to pick this issue. Please assign it to me.

@Deep1Shikha absolutely 🙂

Accidentally clicked the close button - for auditing

@Deep1Shikha are you still working on this one? If not, maybe I can pick it up :)

Doesn't look like @Deep1Shikha has made any progress yet (https://github.com/Deep1Shikha/Checka11y.css). Maybe we could give them a week to respond and if not, I can assign to @alvaromontoro?

@jackdomleo7 assign it to me. I will work on it this week(end).


librdf_raptor2: add patch for CVE-2017-18926

Motivation for this change
Fixes two heap overflows in the raptor2 rdf parsing library.
https://www.openwall.com/lists/oss-security/2017/06/07/1
Things done

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

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

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

I found the patch via https://security-tracker.debian.org/tracker/CVE-2017-18926.

Backports:
20.09: a62679c2476d0cb2cd7ec5f2b20caea12f53437c
20.03: 51105e33db6efa6dbe3dd7b25ecfa02aefe391f5
I also asked the author to link the patch on the CVE page.

And just today on [oss-security] Buffer Overflow in raptor widely unfixed in Linux distros
https://www.openwall.com/lists/oss-security/2020/11/13/1

Yeah I actually got to this issue because I read a german article from Hanno Böck where he was ranting about the problem ^^
https://www.golem.de/news/linux-distributionen-warum-ein-sicherheitsfix-drei-jahre-nicht-ankam-2011-152105.html

I'm a bit saddend by the following statement

Maybe noteworthy is that this didn't get a CVE in 2017. It seems many
distros rely on CVEs to get a process of backporting fixes rolling.
Given the fluctuating reliability of CVE assignments not sure this is
wise. I have now requested a CVE (CVE-2017-18926).

What else is there to go on really? But sure, maybe this is a discussion to be had.

Yeah that's somewhat depressing analysis. Don't think there is a better identification mechanism as CVEs atm in this heterogenous ecosystem…


various updates to redhat internal install task

Signed-off-by: Vasu Kulkarni<EMAIL_ADDRESS>

@susebot run teuthology

oh... tests are failed

jenkins test this please


new: usr: Added filter by province when searching a voter by name..

Will merge when the APIs in https://github.com/carlos-jenkins/tse2sql get updated


CLI Package - Update chokidar dependency

Describe the bug
chokidar depends on the glob-parent package which had a security threat.
chokidar > 3.5.2 has the version of glob-parent without the security risk.
The CLI package's package.json will need to be updated accordingly.
https://github.com/advisories/GHSA-ww39-953v-wcq6
To Reproduce
Steps to reproduce the behavior:
Not applicable - this is just a dependency change
graphql-code-generator/packages/graphql-codegen-cli/package.json:
-    "chokidar": "^3.5.1",
+    "chokidar": "^3.5.2",

Thanks, @callumfrance ! Please note that we use a range, so the eventual version depends on your project.
To make sure it's not causing issues for others, I bumped the range to enforce latest, available in<EMAIL_ADDRESS>


[Event-Change Request] codeunit 12 "Gen. Jnl.-Post Line" - OnCalcPmtDiscOnAfterCalcPmtDisc

Dear ALAppExtensions team,
through this issue we would like to request a change of event publisher OnCalcPmtDiscOnAfterCalcPmtDisc in codeunit 12 Gen. Jnl.-Post Line from:
[IntegrationEvent(false, false)]
local procedure OnCalcPmtDiscOnAfterCalcPmtDisc(DtldCVLedgEntryBuf: Record "Detailed CV Ledg. Entry Buffer"; OldCVLedgEntryBuf2: Record "CV Ledger Entry Buffer"; var PmtDisc: Decimal; var PmtDiscLCY: Decimal)
begin
end;

to:
[IntegrationEvent(false, false)]
local procedure OnCalcPmtDiscOnAfterCalcPmtDisc(var DtldCVLedgEntryBuf: Record "Detailed CV Ledg. Entry Buffer"; OldCVLedgEntryBuf2: Record "CV Ledger Entry Buffer"; var PmtDisc: Decimal; var PmtDiscLCY: Decimal)
begin
end;

adding the var tag to the DtldCVLedgEntryBuf parameter.

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 47942.


embedsOne/Many::associate() does not create timestamps

When using associate() to attach a new embedded document, timestamps are not generated.  If I use save() the timestamps are added as expected.

I think it's because associate doesn't send the document to the DB until you do save on the parent

Should associate generate timestamps? I don't think so.

@jenssegers It should if you've enabled timestamps on that model.  Why don't you think it should?  The developer is setting this option for a reason.

I could be wrong, but I don't think Eloquent does any timestamp updates when doing associate.

It's not that it happens in the associate method itself, but doesn't it do it when it saves the individual model?  A BelongsTo relation is slightly different than an EmbedsOne, of course.  There's really no such thing as an unassociated embedded document, so the only done to attach multiple embedded documents and save them all at once.  I would still like some way to put a timestamp on them.  Of course I could specify this manually, but it seems to me like it should just follow the existing standard for timestamps.


Use generic parser for kubeadm version check

The kubeadm version on the nodes is of the form v1.29.0-eks-a5ec690 and performing a semantic version comparison against v1.29.0 returns a -1 so the kube-vip manifest doesn't get patched. So we're using generic parser which successfully extracts the <major>.<minor>.<patch> from the kubeadm version.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/approve


Table Confusion

/genconfig and /nocompress are included in both axis
But the info about compatibility is contradictory .
I don't know which is right but it looks messed up

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

ID: 7c7e6db2-1517-e9b9-cd9a-6a6aaf84ccc3
Version Independent ID: daab80f9-640e-36c9-e60a-084901b11c05
Content: ScanState Syntax (Windows 10) - Windows Deployment
Content Source: windows/deployment/usmt/usmt-scanstate-syntax.md
Product: w10
Technology: windows
GitHub Login: @greg-lindsay
Microsoft Alias: greglin

@newark123 , We can use   scanstate    /genconfig   with  /nocompress
or     /nocompress  with /genconfig

Mark  X should be sremoved .  @e0i ,  Please look into this issue #8080

https://github.com/MicrosoftDocs/windows-itpro-docs/pull/8162


Player: See a map of of nearby QR codes

As a player, I want to be able to view a map which shows the geo-locations of all nearby QR codes scanned by other players so that I know where to scan new QR-codes and gain more points

Work in progress. Will have pins on map, but still need to implement the clicking of the map pins

Algorithm to get nearby qr codes is not done yet. Can look at Marco's code from search to see how to get nearby qr codes


Update error message with more details.

This resolves #1379

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @huonw (or someone else) soon.
If any changes to this PR are deemed necessary, please add them as extra commits. This ensures that the reviewer can see what has changed since they last reviewed the code. The way Github handles out-of-date commits, this should also make it reasonably obvious what issues have or haven't been addressed. Large or tricky changes may require several passes of review and changes.
Please see CONTRIBUTING.md for more information.

Thanks for the PR! I think that [lib] is somewhat separate from lib.rs, so perhaps the message could be something along the lines of:

no targets specified in the manifest
either src/lib.rs, src/main.rs, a [lib] section, or [[bin]] section must be present

Done updating the text. The quote in github looks like a double new line. I did a single new line. Let me know if you would like it doubled.
Thanks!

@bors: r+ 372e9d60b924a6075098765ee27d40023874c357
Thanks!

:hourglass: Testing commit 372e9d6 with merge ac61996...

:sunny: Test successful - cargo-linux-32, cargo-linux-64, cargo-mac-32, cargo-mac-64, cargo-win-32, cargo-win-64


context values cleared after handler

Hi all, I'm using Negroni together with gorilla mux router (updated to the latest version).
What I'm seeing is that the values in the request's context (using the new builtin context package) are cleared after the handler execution. I've also checked gorilla's documentation and I've updated to the latest version (they're now using the native context if Go 1.7 is used to build the program).
In particular, inside the voidMiddleware function, the last call to it doesn't have the context values se (it appears to me that the latest version of gorilla doesn't do anything and don't clear that values)
How can I solve?
Thanks,
Alessio
Example code:
package main

import (
	"context"
	"fmt"
	"log"
	"net/http"

	"github.com/urfave/negroni"
)

func main() {
	mux := http.NewServeMux()
	mux.HandleFunc("/", func(w http.ResponseWriter, r *http.Request) {
		r = r.WithContext(context.WithValue(r.Context(), "userId", 100))
		log.Printf("___userID in handler: %v", r.Context().Value("userId"))
		fmt.Fprintf(w, "Welcome to the home page!")
	})

	n := negroni.Classic()
	n.Use(negroni.HandlerFunc(MyMiddleware))
	n.UseHandler(mux)

	http.ListenAndServe(":3000", n)
}

func MyMiddleware(rw http.ResponseWriter, r *http.Request, next http.HandlerFunc) {
	next(rw, r)
	log.Printf("___userID after handler: %v", r.Context().Value("userId"))
}

Same issue, I need resp in the end to log. But can not do this.

I am seeing the same problem as well.  Is there a none workaround for this?

You can't inject the context into a endpoint handler, you can only use values from there because you can't pass the wrapped http.Request. You need to create a middleware that runs before
Try something like this:
func UserInjector(rw http.ResponseWriter, r *http.Request, next http.HandlerFunc) {
	rPrime = r.WithContext(context.WithValue(r.Context(), "userId", 100))
	next(rw, rPrime)
}

n.Use(negroni.HandlerFunc(UserInjector)) // put this before MyMiddleware

As @alecha notes, reassigning r in:
mux.HandleFunc("/", func(w http.ResponseWriter, r *http.Request) {
		r = r.WithContext(context.WithValue(r.Context(), "userId", 100))
		log.Printf("___userID in handler: %v", r.Context().Value("userId"))
		fmt.Fprintf(w, "Welcome to the home page!")
	})

will only result in the value being available in that handler (it won't modify the passed in r outside of the function).
@elvizlai @ronorato does this also help your use cases as well?

@jszwedko @alecha this does not solve my problem. I need stdout http resp log in middle ware. This can not be done.

@elvizlai can you please provide some example code that demonstrates your issue? I'm having difficulty understanding.

This does not solve my issue either.  I ended up abandoning the approach and went down a different path.  I could never get the value set by the middleware to show up in the gorilla based route.

@ronorato interesting, I'm happy to look at some sample code if its helpful; I feel like I might be missing something.

@jszwedko I would be happy to try and recreate the issue I was having.  I will look at trying to get something coded up tomorrow for you to look at.

@ronorato  an example blew
package main

import (
	"context"
	"log"
	"net/http"
	"time"

	"github.com/julienschmidt/httprouter"
	"github.com/pborman/uuid"
	"github.com/urfave/negroni"
)

type ContextResponseWriter struct {
	http.ResponseWriter
	Context context.Context
}

func main() {
	n := negroni.New()
	n.Use(LogMiddleware())
	n.UseHandler(SetRouterV1())

	http.ListenAndServe(":3000", n)
}

type Router struct {
	prefix string
	*httprouter.Router
}

func NewWithPrefix(prefix string) *Router {
	return &Router{prefix: prefix, Router: httprouter.New()}
}

func (r *Router) Register(uri string, handlers ...negroni.HandlerFunc) {
	n := negroni.New()
	for _, h := range handlers {
		n.UseFunc(h)
	}
	r.Handler("POST", r.prefix+uri, n)
}

func Wrap(handler http.HandlerFunc) negroni.HandlerFunc {
	return func(rw http.ResponseWriter, r *http.Request, next http.HandlerFunc) {
		handler(rw, r)
		next(rw, r)
	}
}

func SetRouterV1() *Router {
	v1 := NewWithPrefix("/v1")

	v1.Register("/mock/test", Wrap(MockTest))

	return v1
}

// We can not change the handler func now, the params can only be http.ResponseWriter and *http.Request
func MockTest(rw http.ResponseWriter, r *http.Request) {
	// TODO
	// I need set some resp body to ctx, so that I can print out in the end of LogMiddleware
}
func LogMiddleware() negroni.HandlerFunc {
	return func(rw http.ResponseWriter, r *http.Request, next http.HandlerFunc) {
		start := time.Now()
		tid := uuid.New()
		rw.Header().Set("X-Request-ID", tid)

		version := r.Header.Get("X-App-Version")

		ctx := r.Context()
		ctx = context.WithValue(ctx, "tid", tid)

		next(rw, r.WithContext(ctx))

		// TODO
		// I need get 'body resp' here
		if status := rw.(negroni.ResponseWriter).Status(); status == http.StatusOK {
			log.Printf("tid=%s, from=%s, method=%s, remote=%s, agent=%s, version=%s, req=%s, status=%d, took=%dms", tid, r.RequestURI, r.Method, r.RemoteAddr, r.UserAgent(), version, "body resp", status, time.Since(start).Nanoseconds()/1000000)
		} else {
			log.Printf("tid=%s, from=%s, method=%s, remote=%s, agent=%s, version=%s, req=%s, status=%d, took=%dms", tid, r.RequestURI, r.Method, r.RemoteAddr, r.UserAgent(), version, "body resp", status, time.Since(start).Nanoseconds()/1000000)
		}
	}
}

@elvizlai thank you for the example code!
It looks like you are going to run into the same issue I described in https://github.com/urfave/negroni/issues/160#issuecomment-271185113. You cannot set net/http/context values in the handler itself because the only way to modify a request's context is via WithContext which returns a new copy of the request with the new context. That Request struct will only exist within the scope of the handler. This is a result of the way that net/http implements the context pattern.
You can see https://github.com/urfave/negroni/issues/160#issuecomment-270258488 for an example of another route you can take by setting context values in their own middlewares (which are able to propagate the modified context).
Please let me know if this is still unclear.

@jszwedko thanks for you idea.
I know what #160(comment) does. But I need change all my handler(rw, r) to handler(rw, r, next), this may be a huge changes for my project.
May I'd better try to hack negroni.ResponseWriter

@elvizlai :+1: if you come up with anything, I'd be interested in seeing it.
You could also consider gorilla/context (http://www.gorillatoolkit.org/pkg/context) which allows you to associate metadata with a request through an external data structure (rather than on the request itself).

Closing this out since this isn't an issue with negroni, but with the way that context wants to be used.


SourceContext  not outputing correctly in Serilog logger

When logging using Seirlog when outputting the $sourcecontext property in the output template its not replacingith correctly with the context
serilog is configured as per
        Log.Logger = New LoggerConfiguration() _
            .Enrich.With(New ThreadIdEnricher()) _
            .Enrich.FromLogContext _
            .Enrich.WithMachineName _
            .Enrich.WithProcessId _
            .MinimumLevel.Is(Events.LogEventLevel.Debug) _
            .WriteTo.ColoredConsole(outputTemplate:=logLayout) _
            .WriteTo.File(logFileName, outputTemplate:=logLayout) _
            .CreateLogger()

outputTemplate is
{Timestamp:HH:mm} [{Level}] ({ThreadId}) ({SourceContext})  {Message}{NewLine}{Exception}

the context should be there eg.
11:01 [Information] (8) (AIMIE.MaintenanceService.Program)  OnStart at 03/05/2015 11:01:37

Smells like a duplicate of #25 (in essence) ?
Am open to suggestions here.

Ping @nblumhardt , any suggestions on what I can do here (if there is anything I can do) ? Cheers

Howdy all! Not a duplicate of #25 - Serilog doesn't grab this dynamically from the call stack at all. To get this property in there, you need to get a type-specific sub-logger along these lines:
var typeSpecific = someOtherLogger.ForContext("SourceContext", typeName)

Where I believe typeName in the default Serilog implementation is just Type.Name (IIRC).
Not sure if LibLog has any notion of this already, but it's pretty similar to log4net's LogManager.GetLogger(Type) mechanism I think.
HTH!

Thanks @nblumhardt !
Will see if I can hook this in.

Is this fixed by #99?

Yes it appears so.


how can i draw a curve in polar coordinates? r = 2 + 3.cos(theta)  I'm sorry if it was already asked

If this is a support request:
Please attempt to solve the problem on your own before opening an issue.
Between old issues, StackOverflow, and Google, you should be able to find
solutions to most of the common problems.
Include at least:

Steps to reproduce the issue (e.g. the command you ran)
The unexpected behavior that occurred (e.g. error messages or screenshots)
The environment (e.g. operating system and version of manim)

If this is a feature request:
Include the motivation for making this change.

You can force the sucker as a parametric curve.
All you need to do is create the array for the Cartesian coordinates.
import numpy as np
def PolarCurve(theta):
    r = 2 + 3 * np.cos(theta)
    vector = np.array([r * np.cos(theta), r * np.sin(theta), 0])
    return vector

Then add you do is call on it as a parametric function:
curl = ParametricFunction(PolarCurve, t_max=TAU, fill_opacity=0, stroke_width=3)

thank you very much. This community is the best on the internet (:

thank you very much. This community is the best on the internet (:

Depends on what you ask. There are a few things I'm still trying to figure out in Manim so as a community member I'm kind of obligated to help out when I can.

Edit:
If you are so inclined to necessarily have polar curves in your work you can try this:
Go to manimlib/mobject/functions.py and add this nice code in your work:
class PolarCurve(ParametricFunction):
    CONFIG = {
        "theta_min" : 0,
        "theta_max" : TAU,
        "color" : TEAL
    }

    def __init__(self, function, **kwargs):
        digest_config(self, kwargs)
        self.polar_function = \
            lambda t: np.array([function(t) * np.cos(t), function(t) * np.sin(t), 0])
        ParametricFunction.__init__(
            self,
            self.polar_function,
            t_min=self.theta_min,
            t_max=self.theta_max,
            **kwargs
        )
        self.function = function

    def get_function(self):
        return self.function

    def get_point_from_function(self, x):
        return self.polar_function(x)

Then, whenever you need it add you have to do is add the required function and voila:
func = PolarCurve(lambda theta: 1)
self.play(ShowCreation(func))

Okay technically you expect that to draw out a circle since there's a constant radius of 1 as an argument.
Easy


Data delay issues

We are running into some data delay issues, and were looking for the place where the delay is taking place.
Setup:
We have a single hub with 2 edge devices connected.
The first edge had 5 endpoints connected to an OPC server. The second has 10 endpoints connected to 2 OPC servers (5 each).
We are experiencing a data delay on the second edge that we can’t explain. All data coming over 3 of the 10 endpoints was delayed up to 8 hours.
Both edge devices are running version 2.8.3  and running with sufficient resources.
History:
Before we have seen that if some devices connected to the OPC server had connection issues, that this produced so much data that it caused the delay of all data of the devices connected to that (and only that) endpoint. Fixing the connection issue (or disconnecting the device) fixed the data overload.
Issue:
The first edge had a device with connection issues that caused a delay in data as we had seen before.
The second edge also displayed a delay of about the same time (both around 8 hours) but no excess data was being produced by the devices. This delay was seen on all data from devices connected to 3 of the endpoints.
We are looking for the point of delay but are so far unsuccessful. Does anyone have some suggestions on where the delay can be found?

can you please share some details on:

where you see the delay? is it at edgeHub's level or at IoTHub?
is it possible that the IoT hub runs out of message quota during the day?
are the messages pile up in the edgeHub because of upstream limitations?
how is the configuration of the publishers: e.g standalone or orchestrated, is batching enabled

I think the issue is throughput throttling somewhere on the upstream.

Hi,
First an update: The delay issue resolved itself over all endpoints when we disconnected the device producing excessive data. At this moment we are looking for options to check historic data or points to check when the issue reappears.
We notice the delay when using the data. For use we save the data in a database and record both the time of the data and the time of inserting the data into the database. Normally the time difference here is a few seconds. The time of alle the devices are the same.
Whether this delay is at the edgeHub, IoTHub, or data processing, that is what we are trying to find.
No we did not run into the message quota.
There is no indication in the logging of the edgeHub that there was a massage pileup. Also the diagnostic information of the publishers showed no message in the outgress buffer.
The publishers are orchestrated. Batching is enabled by default and the settings are unchanged.
If you know of other methods to check a message pileup on the edgeHub or any other point please let us know. As mentioned before, we are looking for where the data is delayed.

you can get more information from edgeHub module if you enable the Debug log level by adding the RuntimeLogLevel = Debug environment variable to the edgeHub's deployment manifest. You ca do this in portal on the edge device's Set Modules configuration like in the following print-screen:

@BGbakker on your second edge device with the delayed data: was all data delayed or only data from certain endpoints? I understand from your description above that it is only certain endpoints data.
You say "disconnected the device producing excessive data." What does excessive data mean? Does device mean OPC UA Server here? If yes: how many endpoints of the 10 endpoints are from this server? Is one of the 3 endpoints with delayed data from this server?

Hi,
No not all data was delayed, only data from certain endpoints.
"disconnected the device producing excessive data." does not mean disconnecting the OPC UA server. The OPC Server connects to multiple PLC's. We disconnect the PLC's from the OPC server and remove the corresponding  tags from the endpoint configuration.
If you have any more questions let me know.

Hope the above suggestion helped. Closing the issue due to inactivity.
Please feel free to re-open if the issue exists.


cmd/compile: Not printing value before (not object bound directly) produces wildly different behaviour (and assembly)

What version of Go are you using (go version)?

$ go version
go version go1.17 windows/amd64

Does this issue reproduce with the latest release?
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
set GO111MODULE=on
set GOARCH=amd64
set GOBIN=
set GOCACHE=C:\Users\malloc\AppData\Local\go-build
set GOENV=C:\Users\malloc\AppData\Roaming\go\env
set GOEXE=.exe
set GOEXPERIMENT=
set GOFLAGS=
set GOHOSTARCH=amd64
set GOHOSTOS=windows
set GOINSECURE=
set GOMODCACHE=C:\Users\malloc\go\pkg\mod
set GONOPROXY=
set GONOSUMDB=
set GOOS=windows
set GOPATH=C:\Users\malloc\go
set GOPRIVATE=
set GOPROXY=https://proxy.golang.org,direct
set GOROOT=C:\Program Files\Go
set GOSUMDB=sum.golang.org
set GOTMPDIR=
set GOTOOLDIR=C:\Program Files\Go\pkg\tool\windows_amd64
set GOVCS=
set GOVERSION=go1.17
set GCCGO=gccgo
set AR=ar
set CC=gcc
set CXX=g++
set CGO_ENABLED=1
set GOMOD=C:\Users\malloc\Desktop\gohack\go.mod
set CGO_CFLAGS=-g -O2
set CGO_CPPFLAGS=
set CGO_CXXFLAGS=-g -O2
set CGO_FFLAGS=-g -O2
set CGO_LDFLAGS=-g -O2
set PKG_CONFIG=pkg-config
set GOGCCFLAGS=-m64 -mthreads -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=C:\Users\malloc\AppData\Local\Temp\go-build2067272950=/tmp/go-build -gno-record-gcc-switches
GOROOT/bin/go version: go version go1.17 windows/amd64
GOROOT/bin/go tool compile -V: compile version go1.17

What did you do?

I'm trying to use my combinations result with a method wrap, the combinations result being the ptrdiff from a base pointer to determine a position from where to read at.
What did you expect to see?
Result of what happens when I print the value (I'd pass a call fail and I'd get a health integer variable which I'd just then be reading from memory)
What did you see instead?
My method constantly failing by the argument passed being bogus. It consistently tries to read at '0x12ffc88b3422b764', but, when I turn the result into a variable, then I try to turn it into an underlying content read, I fault at an address which is the correct result with the operation.
Failing code label for context:
...
			if found {
				//	Unless we do this, the process will literally fail
				//fmt.Printf("Found signature at base+%x\n", i+j+pad)
				var value uintptr
				value = uintptr(i+j+pad)

				//	This will fault with the correct result
				//result := process.readMemoryDll(dll, 4, *(*uintptr)(unsafe.Pointer(value)))

				result := process.readMemoryDll(dll, 4, value)

				//	result should be readable (and valid)
				if result == nil {
					continue
				}

				return result
			}
...

Full method:
//kPageSize is a constant which equals to 0x1000
func (process *Process) patternScan(dll *Dll, sig []byte, pad int) unsafe.Pointer {
	arrayLength := len(sig)
	scanLength := int(dll.size) - arrayLength

	//	skip first and last page
	for i := kPageSize; i < scanLength-kPageSize; i += kPageSize {
		//	read whole page
		current := process.readMemoryDll(dll, kPageSize, uintptr(i))

		//	page is unreadable, don't care
		if current == nil {
			continue
		}

		//	buffer
		buffer := *(*[kPageSize]byte)(current)

		//	run over every byte
		for j := 0; j < kPageSize-arrayLength; j++ {
			found := true
			for k := 0; k < arrayLength; k++ {
				opcode := buffer[j+k]
				if opcode != sig[k] && sig[k] != 0xCC {
					found = false
					break
				}
			}

			if found {
				//	Unless we do this, the process will literally fail
				//fmt.Printf("Found signature at base+%x\n", i+j+pad)
				var value uintptr
				value = uintptr(i+j+pad)

				//	This will fault with the correct result
				//result := process.readMemoryDll(dll, 4, *(*uintptr)(unsafe.Pointer(value)))

				result := process.readMemoryDll(dll, 4, value)

				//	result should be readable (and valid)
				if result == nil {
					continue
				}

				return result
			}
		}
	}

	return nil
}

Failing label's assembly when not printing:
.text:000000000048FD9D 48 89 44 24 20                    mov     [rsp+1040h+var_1020], rax
.text:000000000048FDA2 48 8B 54 24 28                    mov     rdx, [rsp+1040h+var_1018]
.text:000000000048FDA7 48 8D 34 10                       lea     rsi, [rax+rdx]
.text:000000000048FDAB 48 8B BC 24 70 10+                mov     rdi, [rsp+1040h+arg_28]
.text:000000000048FDAB 00 00
.text:000000000048FDB3 48 01 FE                          add     rsi, rdi
.text:000000000048FDB6 4C 8B 84 24 50 10+                mov     r8, [rsp+1040h+arg_8]
.text:000000000048FDB6 00 00
.text:000000000048FDBE 4D 8B 48 10                       mov     r9, [r8+10h]
.text:000000000048FDC2 49 8D 0C 31                       lea     rcx, [r9+rsi]
.text:000000000048FDC6 BB 04 00 00 00                    mov     ebx, 4
.text:000000000048FDCB 48 8B 84 24 30 10+                mov     rax, [rsp+1040h+var_10]
.text:000000000048FDCB 00 00
.text:000000000048FDD3 E8 28 FB FF FF                    call    github_com_cristeigabriel_gogo_internal_gogo___Process__readMemory
.text:000000000048FDD8 48 85 C0                          test    rax, rax
.text:000000000048FDDB 75 48                             jnz     short loc_48FE25
.text:000000000048FDDD 48 8B 44 24 20                    mov     rax, [rsp+1040h+var_1020]
.text:000000000048FDE2 48 8B 94 24 60 10+                mov     rdx, [rsp+1040h+arg_18]
.text:000000000048FDE2 00 00
.text:000000000048FDEA 4C 8B 84 24 58 10+                mov     r8, [rsp+1040h+arg_10]
.text:000000000048FDEA 00 00
.text:000000000048FDF2 EB 8F                             jmp     short loc_48FD83

Then failing, not succeeding, label's assembly, when printing:
.text:000000000048FDA5                   loc_48FDA5:                             ; CODE XREF: github_com_cristeigabriel_gogo_internal_gogo___Process__patternScan+1EB↓j
.text:000000000048FDA5 48 89 44 24 40                    mov     [rsp+1078h+var_1038], rax
.text:000000000048FDAA 48 8B 4C 24 48                    mov     rcx, [rsp+1078h+var_1030]
.text:000000000048FDAF 48 8D 14 08                       lea     rdx, [rax+rcx]
.text:000000000048FDB3 48 8B 9C 24 A8 10+                mov     rbx, [rsp+1078h+arg_28]
.text:000000000048FDB3 00 00
.text:000000000048FDBB 48 01 DA                          add     rdx, rbx
.text:000000000048FDBE 48 89 54 24 50                    mov     [rsp+1078h+var_1028], rdx
.text:000000000048FDC3 48 89 D0                          mov     rax, rdx
.text:000000000048FDC6 E8 D5 9B F7 FF                    call    runtime_convT64
.text:000000000048FDCB 44 0F 11 BC 24 60+                movups  [rsp+1078h+var_18], xmm15
.text:000000000048FDCB 10 00 00
.text:000000000048FDD4 48 8D 0D 45 89 00+                lea     rcx, unk_498720
.text:000000000048FDD4 00
.text:000000000048FDDB 48 89 8C 24 60 10+                mov     qword ptr [rsp+1078h+var_18], rcx
.text:000000000048FDDB 00 00
.text:000000000048FDE3 48 89 84 24 68 10+                mov     qword ptr [rsp+1078h+var_18+8], rax
.text:000000000048FDE3 00 00
.text:000000000048FDEB 48 8B 1D E6 8B 0B+                mov     rbx, cs:os_Stdout
.text:000000000048FDEB 00
.text:000000000048FDF2 48 8D 05 07 BD 03+                lea     rax, go_itab__os_File_io_Writer
.text:000000000048FDF2 00
.text:000000000048FDF9 BF 1B 00 00 00                    mov     edi, 1Bh
.text:000000000048FDFE 48 8D B4 24 60 10+                lea     rsi, [rsp+1078h+var_18]
.text:000000000048FDFE 00 00
.text:000000000048FE06 41 B8 01 00 00 00                 mov     r8d, 1
.text:000000000048FE0C 4D 89 C1                          mov     r9, r8
.text:000000000048FE0F 48 8D 0D 21 EF 01+                lea     rcx, unk_4AED37
.text:000000000048FE0F 00
.text:000000000048FE16 E8 E5 88 FF FF                    call    fmt_Fprintf
.text:000000000048FE1B 90                                nop
.text:000000000048FE1C 48 8B 8C 24 88 10+                mov     rcx, [rsp+1078h+arg_8]
.text:000000000048FE1C 00 00
.text:000000000048FE24 48 8B 51 10                       mov     rdx, [rcx+10h]
.text:000000000048FE28 48 8B 5C 24 50                    mov     rbx, [rsp+1078h+var_1028]
.text:000000000048FE2D 48 01 DA                          add     rdx, rbx
.text:000000000048FE30 48 8B 84 24 58 10+                mov     rax, [rsp+1078h+var_20]
.text:000000000048FE30 00 00
.text:000000000048FE38 BB 04 00 00 00                    mov     ebx, 4
.text:000000000048FE3D 48 89 D1                          mov     rcx, rdx
.text:000000000048FE40 E8 BB FA FF FF                    call    github_com_cristeigabriel_gogo_internal_gogo___Process__readMemory
.text:000000000048FE45 48 85 C0                          test    rax, rax
.text:000000000048FE48 75 4E                             jnz     short loc_48FE98
.text:000000000048FE4A 48 8B 44 24 40                    mov     rax, [rsp+1078h+var_1038]
.text:000000000048FE4F 48 8B 94 24 98 10+                mov     rdx, [rsp+1078h+arg_18]
.text:000000000048FE4F 00 00
.text:000000000048FE57 4C 8B 84 24 90 10+                mov     r8, [rsp+1078h+arg_10]
.text:000000000048FE57 00 00
.text:000000000048FE5F 90                                nop
.text:000000000048FE60 E9 1E FF FF FF                    jmp     loc_48FD83

As @randall77 pointed out in https://github.com/golang/go/issues/48035#issuecomment-907684633 , it is hard to understand what this code is intended to do. Could you be clearer about what the intended behavior is, and make sure the use of unsafe.Pointer is valid per https://pkg.go.dev/unsafe#Pointer . Thanks.

As @randall77 pointed out in #48035 (comment) , it is hard to understand what this code is intended to do. Could you be clearer about what the intended behavior is, and make sure the use of unsafe.Pointer is valid per https://pkg.go.dev/unsafe#Pointer . Thanks.

Hello! I'm sorry. I'll try to explain.
In general, before this, to use this:
I'm taking a 32bit process, making a snap of it's modules list, walking the modules list to capture a specific module, and then containing it alongside it's name, it's base address, and it's size.
In context:
I'm looking for a consistent listing of opcodes, for that, I'm iterating through the module from 0 (I'm using a helper function to add the DLL base address to my specified address when reading the memory), I'm skipping the first page as it is not relevant, and I'm going until I reach module size - argument byte array size - 1 page. My primary iteration goes page over page (i.e. I'm adding 4096/0x1000 every primary iteration run), and inside it, I'm reading 1000 bytes from the current place in memory, and then I'm iterating the 1000 bytes stored, and looking for the byte array passed as an argument. If we succeed in finding the byte array, I'm storing the padding from the primary iteration. If it's found, I do this:

			if found {
				//	Unless we do this, the process will literally fail
				//fmt.Printf("Found signature at base+%x\n", i+j+pad)
				var value uintptr
				value = uintptr(i+j+pad)

				//	This will fault with the correct result
				//result := process.readMemoryDll(dll, 4, *(*uintptr)(unsafe.Pointer(value)))

				result := process.readMemoryDll(dll, 4, value)

				//	result should be readable (and valid)
				if result == nil {
					continue
				}

				return result
			}

I'm attempting to read 4 bytes at the current iteration + aforementioned padding + argument padding (I.e. if I specify a data skip with the wildcard opcode 0xCC, if I want to capture a pointer address at runtime, I want to add 2 to get there and read there).
This process works perfectly! But there's a catch. It works perfectly... only if I print the value.
The unsafe usage there is just as a way to debug the contents of the value without directly printing it or any of that. I hope it didn't get picked up as legitimate usage. It was just another way to indicate or display the issue.
I didn't get much time to actually research more myself, but I've posted the assembly for preparing the function call above. I could provide a project with the whole ordeal, but it's dependent on a specific process currently.
I could write, if asked for, a project that will depend on another local small application and replicate the process which is currently bound to the process I'm now interacting with.
I'm sorry for not responding earlier. There's a language barrier which sometimes prevents me from fully expressing what I'm trying to say.

What does process.readMemoryDll do? In particular, what does it return?
What does "process will literally fail" mean? SEGV? Return wrong result? Panic? Something else?
What does "fault with the correct result" mean?
What does "result should be readable (and valid)" mean?

readMemoryDll is a wrap for a ReadProcessMemory WINAPI call, Dll part implies that the lpBaseAdress will be a formation of the passed DLL's bass address to uintptr + in this scenario (i+j+pad). It returns a pointer to what I'm writing the memory copied content to so I can interpret it as I wish.
"Process will literally fail" means that the RPM call will fail by trying to read at a bogus address, on that I detail more in the OP, obviously unless I do the aforementioned print. It'll panic because it is made to, because the WINAPI ReadProcessMemory result is 0. That is, again, because lpBaseAddress is bogus.
"Fault with correct result" is me trying to SEGV the program, and succeeding, for the debug log. The debug log specifies the segv address as the result I'm seeking of the aforementioned operation.
"Result should be readable (and valid)" is a specification that the returned pointer should not be nil, and should be from a readable memory page.

Not clear to me then.
_ = *(*uintptr)(unsafe.Pointer(value)) should fault at the same address as the uintptr passed as the third argument to process.readMemoryDll(dll, 4, value). If you're saying that readMemoryDll is trying to read from somewhere else, then it sounds like a bug in readMemoryDll. We'd need to see the code of that function (and anything it calls). Did you try putting a print inside readMemoryDll to print its argument?
I encourage you to do some debugging yourself. It's not a good use of anyone's time to debug remotely back and forth like this. Run under a debugger, set breakpoints, see where that bad address is coming from. If you can give us a whole program that we can run, then we might be able to help you.

No new information


Fix axis ordering for fiona.transform

Fix axis ordering of fiona.transform (see https://github.com/Toblerity/Fiona/issues/919)

Coverage increased (+0.4%) to 85.21% when pulling d0fbb9091be3b465bda34ea659ad6eed03ac3702 on rbuffat:fix_transform_axis_ordering into b4ab36986bc2d51749de35da1f7d88bcc8678739 on Toblerity:maint-1.8.

Coverage increased (+0.4%) to 85.21% when pulling d0fbb9091be3b465bda34ea659ad6eed03ac3702 on rbuffat:fix_transform_axis_ordering into b4ab36986bc2d51749de35da1f7d88bcc8678739 on Toblerity:maint-1.8.

Thanks @rbuffat !


2 all notes panel

Right panel displays title and body of a note.
Structure improved.
Store event emitter initialized.

👍


Fixed missing "'" in "Python3 1"

Missing "'" in "Python3 1" causes "Syntax error: Unterminated quoted string" error during execution.
Fixed by adding "'" at the end of the command.

wew

Thank you


fix add cjs file extension to jest config

Signed-off-by: Jessica Lynn Kim<EMAIL_ADDRESS>Adding cjs.js to jest paths to mitigate failing jest tests in circleci sg1-coverage-module.

A new version of the package @shipt/sg1-module-scripts (3.4.3) was published at 2022-09-26T18:29:08.633Z from
<IP_ADDRESS>. The shasum of this package was 02b3be23bc43e1bf68db028e01100c56db1a6a23.


opera: add libdrm and mesa dependencies

Motivation for this change
Fixes #120873 (Opera failing to build)
I'm not quite sure why the Opera build started failing, but adding the missing shared libraries fixes it.
Things done

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

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

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

Result of nixpkgs-review pr 120891 at f3709c80 run on x86_64-linux 1

1 package built successfully:

opera

2 suggestions:

warning: missing-phase-hooks
installPhase should probably contain runHook preInstall and runHook postInstall.
Near pkgs/applications/networking/browsers/opera/default.nix:114:3:
    |
114 |   installPhase = ''
    |   ^

warning: maintainers-missing
Package does not have a maintainer. Consider adding yourself?
Near pkgs/applications/networking/browsers/opera/default.nix:120:3:
    |
120 |   meta = with lib; {
    |   ^


let indexes using noUncheckedIndexedAccess in TS 5.5.2

🔎 Search Terms
"array", "noUncheckedIndexedAccess", "5.5"
🕗 Version & Regression Information

This changed between versions 5.4.5 and 5.5.2

⏯ Playground Link
https://www.typescriptlang.org/play/?noUncheckedIndexedAccess=true&noUnusedLocals=true&noUnusedParameters=true&ts=5.5.2#code/MYewdgzgLgBAhgLhmArgWwEYFMBOBtAXRgF4Y8BGAGhgCZqBmAgbgCgAbLWASyVU1xIxyrFlwBmACigBPAA5YQY+Hi5Fi6mACI+2HJoCULGMeWqA1GaA
💻 Code
const a: number[] = [1, 2, 3];
let i: number = 1;

if(typeof a[i] === "number")
    a[i]++

🙁 Actual behavior
Typescript reports the following error:
Object is possibly 'undefined'.(2532)
const a: number[]
🙂 Expected behavior
It should not report an error for such cases
Additional information about the issue
This obviously requires noUncheckedIndexedAccess to be enabled in the tsconfig.
This only happens when the variable used to access the array is non const (let in this case).
I think that TS 5.5 actually fixed the original behavior because in 5.4 if we remove the check:
const a: number[] = [1, 2, 3];
let i: number = 1;
a[i]++

TS does not raise the same error (which would be expected as a[i] could be undefined).
The current workaround:
const j = i;
if (typeof digits[j] === "number") {
  digits[j]++;
}

I think that TS 5.5 actually fixed the original behavior

See: https://devblogs.microsoft.com/typescript/announcing-typescript-5-5/#control-flow-narrowing-for-constant-indexed-accesses
And also the PR: #57847
This intentionally is only working for constant-like variables.

This does seem to meet the criteria in #57847 though:

With this PR we perform control flow analysis for element access expressions obj[key] where key is a const variable, or a let variable or parameter that is never targeted in an assignment.  If the initializer counts as an "assignment" in this context then it effectively doesn't work for let variables at all and that clause shouldn't be there.

Heh, this is a fun one... in a sense this is surprising. TS thinks i is global here. If we just add export {} to the end of this file then it works like expected: TS playground
Even though it doesn't exactly look like it... this feels like a duplicate of https://github.com/microsoft/TypeScript/issues/58345 . Obviously, this repro is wildly different from the other one but the implementation source of the problem is very likely the same

This is working as intended. We specifically exclude global variables from assignment analysis as the control flow analyzer can't see all references (there might be references in other files). Adding an export {} limits the scope to the current file at which point CFA can determinate the location of the last assignment and treat the variable as const for the remained of the file.

Thanks, for the clarifications, I was simply surprised because the behavior in a very "hello world" code is now a bit confusing for non-experienced people:
function goThroughArray() {
  const a: number[] = [1, 2, 3];
  for(let i = 0; i < a.length; i++) {
    if (typeof a[i] === "number") {
        a[i]++; // a[i] can be undefined
      }
  }
}

Which is not exactly the reproducer I have send in the original issue, as I was trying to narrow this down.
However given that the compiler can't know if i was modified in the loop or not, I guess it's the best we can do for now.

Ah, shoot. I forgot that those are still global variables (even if not available on globalThis) and can leak across scripts 😢

However given that the compiler can't know if I was modified in the loop or not, I guess it's the best we can do for now.

This would be a duplicate of https://github.com/microsoft/TypeScript/issues/58803 . CFA graph walk could likely check if an assignment happens between those 2 points but that could cost some perf.


[Backport 2.x] Add support for API versioning and fail open in weighted shard routing

Description
Backport PR for weighted routing changes

https://github.com/opensearch-project/OpenSearch/pull/5680
https://github.com/opensearch-project/OpenSearch/pull/5255
https://github.com/opensearch-project/OpenSearch/pull/5072
https://github.com/opensearch-project/OpenSearch/pull/5778

Issues Resolved
[List any issues this PR will resolve]
Check List

[ ] New functionality includes testing.

[ ] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[ ] Commits are signed per the DCO using --signoff
[ ] Commit changes are listed out in CHANGELOG.md file (See: Changelog)

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Gradle Check (Jenkins) Run Completed with:

RESULT: FAILURE ❌
URL: https://build.ci.opensearch.org/job/gradle-check/9169/
CommitID: 51322b1
Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green.
Is the failure a flaky test unrelated to your change?

Build failed due to flaky test failure - https://github.com/opensearch-project/OpenSearch/issues/5765

Test Failure : https://github.com/opensearch-project/OpenSearch/issues/5765


Updated showcase app to use leanback v27.1.1

Replaced:

v26 Fragment with v27.1.1 SupportFragment
android.app.Activity -> android.support.v4.app.FragmentActivity
android.arch.lifecycle.LifecycleActivity -> android.support.v4.app.FragmentActivity

Introduced a support inteface FragmentActivityKey for dagger to be used with FragmentActivity classes

Does this bug affect MainFragment? https://issuetracker.google.com/issues/78900694

I think yes.
In MainFragment onActivityCreated method is never called on api level 27 (I don't know why). To fix this, I moved the initialization code (3 setup* method calls) to onCreated method. If this is not done, then the initialization of test data fails.

This sample has been migrated to a new location and updates are happening there (check README for more information).
As recommended by GitHub, we are closing all issues and pull requests now that this older repo will be archived.
If you still see this issue in the updated repo, please reopen the issue/PR there. Thank you!


owner(ticdc): not use global resolvedTs as a part of barrier ts  if redo is not enabled

What problem does this PR solve?

Issue Number: close #7407
What is changed and how it works?
global resolvedTs not be used as a part of barrier ts  if redo is not enabled
Check List
Tests

Unit test
Integration test
Manual test (add detailed scripts or steps below)

Questions
Will it cause performance regression or break compatibility?
Do you need to update user documentation, design documentation or monitoring documentation?
Release note
`None`.

/run-all-tests

/run-integration-tests

/check-issue-triage-complete

/run-integration-tests

Codecov Report

Merging #7408 (915f214) into master (84ef430) will increase coverage by 0.0344%.
The diff coverage is 67.6815%.

Additional details and impacted files

Flag
Coverage Δ

cdc
67.6089% <77.4691%> (+0.1144%)
:arrow_up:

dm
51.9892% <100.0000%> (+0.0180%)
:arrow_up:

engine
60.6469% <18.7500%> (-0.2068%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.
@@               Coverage Diff                @@
##             master      #7408        +/-   ##
================================================
+ Coverage   59.7990%   59.8335%   +0.0344%
================================================
  Files           801        803         +2
  Lines         91968      92149       +181
================================================
+ Hits          54996      55136       +140
- Misses        32180      32217        +37
- Partials       4792       4796         +4

What about partition tables? What happens when a DDL affects two partitions?

/run-all-tests

/run-all-tests

/run-verify-test

/merge


witness: 0.1.11 -> 0.1.12

Description of changes

Things done

Built on platform(s)

[X] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

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

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

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

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

[X] Fits CONTRIBUTING.md.

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

1 package built:

witness

There is hash mismatch on darwin, is proxyVendor needed?

Likely. Updated & waiting for darwin builds


Upgrade TargetSdk, Kotlin, KSP, Gradle and Compose

Bump TargetSdk to 33
Bump Kotlin to 1.8.10
Bump  KSP to 1.8.10-1.0.9
Bump  Gradle to 7.6
Bump AGP to 7.4.1
Bump Compose Compiler to 1.4.2
Add Compose BOM 2023.01.00
Replace deprecated code

Let's remove files coming in from .idea? And also, would it be a good time to introduce either buildSrc or composite builds to this project?

Thanks for pointing it out @shubhamsinghmutualmobile.
I have removed the cached files and added them in .gitignore.
Sure we can work on buildSrc/composite build in a separate PR. 👍


Allow up to 7 decimal places for fractional seconds in DateTime values

Json.NET defaults to 7 decimal places when serializing DateTime values. It would be nice to accommodate that.

Thanks for the PR! So the other format is still supported, correct?

@RSuter correct, the previous test is still there and passes. Any ISO 8601 formatted datetime will pass with up to 7 decimal places.

Now in latest nuget package version


Error:    0: Websocket error    1: HTTP error: 200 OK

Error:
0: Websocket error
1: HTTP error: 200 OK
Location:
/Users/mick/.cargo/registry/src/mirrors.tuna.tsinghua.edu.cn-df7c3c540f42cdbd/aspeak-4.2.0/src/main.rs:70
Backtrace omitted. Run with RUST_BACKTRACE=1 environment variable to display it.
Run with RUST_BACKTRACE=full to include source snippets.

The demo at https://azure.microsoft.com/en-us/products/cognitive-services/text-to-speech/#overview is removed. So I guess the trial endpoint is no longer available.

The trial endpoint has been removed starting from aspeak v5.0.


Allow devices and hardware to be "hot-swappable"

I would love it if certain devices (for instance, the InGaAs array) could be loaded and unloaded into PyCMDS without requiring a restart.
I assume a massive structural change would be needed in order to make this happen, maybe it is something to think about for PyCMDS's successor...

This is something @untzag and I have discussed as one of the features to build into the successor from the beginning.


'VOC Density': characteristic was supplied illegal value

on higher VOC levels i get a warning like the following:
[05/01/2022, 19:58:04] [homebridge-airthings] This plugin generated a warning from the characteristic 'VOC Density': characteristic was supplied illegal value: number 1705.4481447162475 exceeded maximum of 1000. See https://git.io/JtMGR for more info.
[05/01/2022, 19:58:03] [Wohnzimmer WavePlus] {"battery":100,"co2":642,"humidity":46,"pressure":952.9,"radonShortTermAvg":96,"rssi":-48,"temp":18.4,"time":1641409069,"voc":556,"relayDeviceType":"hub"}

by using this online converter on the given data with a molecular weight of 78, i would expect a value of about 1.8, not 1705 like in the error message.

could it be, that the units "ppm" (as used by airthings) and "ppb" (as used by many formulas) have been mixed?

The Airthings device tech specs and app show they are measuring VOC as ppb, not ppm. Have you seen any references to ppm in the Airthings ecosystem?
It seems that the HomeKit Accessory Protocol Spec has a max value of 1000 for VOC Density, which is what's causing that error message. Seems like that's a low max value if Airthings is measuring ranges >2000 ppb or a reasonable max if they're in fact measuring ppm..

The Airthings device tech specs and app show they are measuring VOC as ppb, not ppm. Have you seen any references to ppm in the Airthings ecosystem?

i have to apologize: aparently i am the one who mixed up ppm and ppb.
You are right, VOC is measured in the App/Dashboard and referenced in all of Airthings documents as ppb.
My bad, sorry.
Now i am wondering about the homekit max value of 1000, but that ist none of our business i guess...  i think this issue can be closed.


Fix "method_lines" issue in src/ApiRouter.php #16

Addressing the Issue #16 to reduce the ApiRouter code.
https://codeclimate.com/github/illusi0nary/api-backend/src/ApiRouter.php

Thanks for your contribution @illusi0nary !

Hey @santiagolizardo  can you add the label hacktoberfest-accepted to that one here? :-)


Generate recipes from Github

Generate recipes for Github packages
This PR attempts to add to Grayskull the ability to generate recipes for those Python packages that exist as Github repositories but do not exist on PyPI.
Motivation

Presently Grayskull generates recipes only for those Python packages that are available on PyPI. This pre-requisite leaves out a number of Python packages from being in Grayskull's 'recipe-generatable' list.
Grayskull's ability to generate recipes for Github packages can reduce internal packaging efforts in companies.

Rationale

Grayskull extracts package metadata from two sources to generate the recipe - sdist metadata and pypi metadata. For a package not published on pypi, pypi metadata is obviously not present. The existent Grayskull machinery to extract sdist metadata was tweaked so that it could also be used got extract metadata for Github packages.
Version of a github package as specified by the user and the original version name might differ (for example: 1.0.0 / v1.0.0 / ver1.0.0/ ), string matching is used and the 'tag' with the closest name is considered for recipe generation.

Backwards Compatibility
Backwards Compatibility is maintained.
Alternatives Links to prototype or a sample implementation of the proposed change.
During the implementation of this new feature, subclassing and creating a separate machinery altogether for Github packages was considered. It is still under consideration and might be implemented in future.
Sample Implementation

We still use the pypi entry point
We let the name be either a pypi package name or a GitHub repository URL
We check for "name or url" in a number of places to alter the source from initial metadata and tarballs
Once tarball is downloaded, the rest of the PyPI logic takes over

Solves #161

The test it says is failing, is actually passing on my local machine.
How do I rerun these tests?

Codecov Report

Merging #244 (a9a22df) into main (9c3242e) will decrease coverage by 0.41%.
The diff coverage is 83.33%.

@@            Coverage Diff             @@
##             main     #244      +/-   ##
==========================================
- Coverage   91.30%   90.88%   -0.42%
==========================================
  Files          17       17
  Lines        1576     1657      +81
==========================================
+ Hits         1439     1506      +67
- Misses        137      151      +14

Flag
Coverage Δ

unittests
90.88% <83.33%> (-0.42%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

grayskull/cli/parser.py
84.61% <77.77%> (-15.39%)
:arrow_down:

grayskull/base/base_recipe.py
88.09% <80.00%> (-0.32%)
:arrow_down:

grayskull/pypi/pypi.py
94.65% <81.25%> (-1.50%)
:arrow_down:

grayskull/__main__.py
98.21% <100.00%> (+0.06%)
:arrow_up:

grayskull/utils.py
100.00% <100.00%> (ø)

grayskull/license/discovery.py
98.28% <0.00%> (+1.14%)
:arrow_up:

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 34d93e0...a9a22df. Read the comment docs.

Thank you very much @ForgottenProgramme for you contribution!
I am very impressed by the detailed message you wrote in the PR, pretty nice! Keep it up! :tada: :tada:
Lets try to get

https://github.com/conda-incubator/grayskull/pull/245

merged and after that, I will generate a new version with your modifications :)


Corrected include of GreeterTest.java

@stain Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.


Named tuple field names of linear algebra functions

Currently linalg.qr and linalg.svd specify the return tuple field names using lower case letters: q, r, u, s, vh.
Are there any previous discussions on the naming convention? It's common in the linear algebra domain to denote the matrices with upper case single letters. Therefore in PyTorch's linear algebra module (that already uses the namedtuple returns) it was chosen to use qr() -> (Q, R) and svd() -> (U, S, Vh) - upper case letters.
In Julia's LinearAlgebra module also upper case letters are used for accessing the result of QR and SVD decompositions.

No discussion to which to refer. Using lowercase letters was initially chosen for case consistency throughout the API standard and reduce the need for users to know what case to use when. However, we can obviously revise this and elect to allow tuple fields to be uppercase. We'd just need a list fields which would need to be modified.

I think for consistency with other APIs it's probably better to stay with lowercases. Also save us one key stroke 🙂

I think the upper case names are very reasonable, probably even preferred if we'd do a greenfield design. Now that PyTorch is the only library that implements this already, it certainly makes sense - we didn't check this carefully enough perhaps, and we do not want to force PyTorch to make a backwards-incompatible change here (and namedtuples are limited, adding aliases in there seems tricky).
So I'm +1 for changing this to upper-case.

In today's call, we decided to adopt @IvanYashchuk proposal to align with PyTorch in namedtuple field names for linear algebra APIs. I'll submit a PR shortly to update the specification.


Improving footer

The footer has some spelling and structuring errors. Also, the current year should be added and the page name should be linked to the main index.

let me work on that .. its easy..

I would like to work on it. Please assig it to me

Assig it to me

@Klaus6666 you have created PR to forked version. Please resolve it and create a PR to the main repository.

@FabianCristancho please go ahead!


SMAPP freezes when the system starts if the Internet connection is not active.

Environment

Smapp version: 1.2.1
Node version: node-not-started (settings screen, info section)
OS: Windows 11

Describe the bug
If the Internet connection with the provider is established later than the application autostarts, the application freezes at the authorization stage. After the Internet connection appears, the application continues to remain inactive and nothing happens as monitored by the task manager.
Steps to reproduce

Activate application autorun in settings.
Turn off the PC.
Disable the WAN port on the router.
Turn on the PC and wait for SMAPP to start.
Connect the provider's cable to the WAN port of the router.

On laptops, this problem is also present if the Wi-Fi connection is established with a delay.
Expected behavior
The warning about the lack of Internet connection when starting the application is not functional.
Actual behavior
What’s happening now?
To successfully launch the application you have to manually close and restart SMAPP.

Confirming that this issue exists: When starting the application with no internet connection and then connecting the PC to the internet afterward, we encounter an infinite loading screen.

The additional case we should take care of is when the node-config is not updated in our server's bucket, resulting in an empty object. This situation can also arise when our server, Firewall, VPN, or similar components cannot ping our server using the node-config.file or networks.json file.
We are getting messages from our community and here about the empty node-config file after updating to the latest version (versions 1.2.2, 1.2.3, and 1.2.4; we're not sure about other versions).
When the node-config file is empty, I tried to reproduce it, but during the update, I had to close smapp and finish the process that I opened with the --discovery flag. I believe there might be an issue when we cannot download the node-config file during an update, or when it's empty, causing problems with an empty node-config.
However, when I tried to mock an empty node-config.json file, I didn't have any success. Until we have the genesisID hash from the node-config.json file, our application doesn't load properly.
Since I can't reproduce it properly, it could be any error from our clients. Up to this point, I think we should have a modal that appears when we cannot download networks.json or node-config.json, preventing the app from working properly.
Maybe implement a timeout after 5 minutes of retries, which would describe the error. We're not sure if we will always get 404/500 errors from our bucket on the node-config.json request or it will be 200/304 with empty object.
Additionally, we may need to validate the node-config.json file to determine if it's a valid JSON or not. Currently, it results in an infinite loading screen.

Just for the protocol, this additional case is not related to the topic: Users in the reports have not just empty config files, but files with partially allocated space (~4kb of null bytes). Most likely they encountered some other problem with I/O.


Status of 'Official' Datadog SDK Integration

Hi there amazing developers!
Our company is building out a mobile application with Flutter, and we were pleasantly surprised to find that Dutchie has a 'quasi-official' Flutter implementation for Datadog, that Datadog officially approves of and is planning to turn their package into an official Datadog SDK with Flutter, as explained in this YouTube video.
That was in late October 2021, and I was wondering if there was an update in the 'transition' to official status?

Hey @mikhael28 sorry but I personally have no news. I last spoke with a DD developer a few weeks ago and I'm excited about the features they're building. Tagging @xgouchet here in case any official news can be disclosed.

Hi @mikhael28, we're actively working on an official Datadog SDK for Flutter, and although I can't give you a specific ETA, we plan on starting alpha testing soon, and beta testing should follow once we have enough confidence over what we built.

Thanks for chiming in @xgouchet , I appreciate it


fix raz orientation issue in leg script

come from the raz fonction

done, using joints and raz_jnt, see fonctions raz_ik_ctrl_translate_rotate and raz_fk_ctrl_rotate


Fix for #2046 Autocomplete/Suggestions don't scroll

Potential fix for the issue described in #2046

Fixes #1874 as well


Logs message filter now takes regexes

fixes #1574
Message flag now takes regexes.

@JosephGJ still planning for 0.14?

@ndegory I'll be able to get this in for 0.14, I'm just updating the smoke tests.


Chart type "outlabeledPie" is not supported

The example of outlabeledPie can't get the expected output image.
The expected output image.
Output image with python sdk.

This is a side effect of the outlabeledPie plugin.  It is sensitive to canvas size and resolution.  If there is not enough room for the labels, it won't be able to render them.
If you set devicePixelRatio to 2 in your Python example, for example, the labels will show.


Try to selcted only possible or supported extension with input box.

Right now video and other irrelevant files are being selected, do prevention of those file selection.

Hi I;m not sure what you are talking about - the demo here? https://codesandbox.io/s/72py4jlll6
It works ok for me in this regard, please add more detail

Yeah, it seems to work perfectly as expected. But when you select any other format below are the steps

select any image and crop it
now select any other video or unsupported files both files will be there.

So, think just broke the functionality when select unsupported media after supported media.
Otherwise its work perfectly, It's awwsome.....

For suggestion purpose:
You can restrict all unsupported media in HTML input type file issue will be resolve automatically.

Ah ok well this is perhaps an issue with the demo, which is just intended as a simple demo to show the library not as perfect behaviour in an app. Although after selecting something that isn't an image I saw the previous image that was selected and nothing changed except the file name of the selected item and I was able to select something else after so didn't seem like an issue to me. Anyway I've added an accept attribute to only select images now.

Great !!!!.... Thank you...


Initial conditions

Closes #159

Codecov Report

Merging #164 (d9413ae) into main (1fe08b8) will increase coverage by 0.26%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #164      +/-   ##
==========================================
+ Coverage   95.77%   96.04%   +0.26%
==========================================
  Files          33       34       +1
  Lines        1917     2046     +129
==========================================
+ Hits         1836     1965     +129
  Misses         81       81

Impacted Files
Coverage Δ

pybryt/annotations/__init__.py
100.00% <100.00%> (ø)

pybryt/annotations/initial_condition.py
100.00% <100.00%> (ø)

pybryt/annotations/value.py
94.95% <100.00%> (+0.39%)
:arrow_up:

pybryt/execution/__init__.py
100.00% <100.00%> (ø)

pybryt/execution/memory_footprint.py
100.00% <100.00%> (ø)

pybryt/execution/tracing.py
100.00% <100.00%> (ø)

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 1fe08b8...d9413ae. Read the comment docs.


Add more complicated filepath testing

These tests fail for me with workqueue. cc @dthain
This is motivated by a problem with PR #1182 which has exposed our lack of interesting file path testing in CI.

closing because stdout/stderr handling is not cleanly defined - see issue #363


Key the download directory by the instance id

Motivation
If a worker is assigned more than one instance of a particular function, the download paths of the function package currently clash with each other. This pr makes the download path keyed by instance id as well to eliminate that clash.
Modifications
Describe the modifications you've done.
Result
After your change, what will change.

@sijie @jerrypeng


Support MSVS in the deprecated ignore

Description of changes:
Support MSVS in the deprecated ignore, using #ifdef _WIN32 statement.
Issue #, if available:
MSVS compiler cannot recognize the GCC ignore syntax.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute
this contribution, under the terms of your choice.

Hi @lcheng9, similar to #134 can you combine this with PR #133 so all changes for MSVC can be reviewed together? Thanks.

Merged into https://github.com/awslabs/palace/pull/133


Fixed Responsiveness Of Contributors Page

Issue: #608
before:

now:

Please check it @agamjotsingh18

@agamjotsingh18 please review this


Not being able to drift on the version 1.20.1

I actually know this mod from the version 1.19.2 and it works perfectly fine on 1.19.2 but when I tried it on forge and fabric 1.20.1 the drift feature seems to not work.

drift is working for me on 1.20.1 with a custom modpack, could you provide more information please? error logs, video, etc

Drifting works fine for me too, on 1.20.1 Fabric.

i think he meant not being able to drift in superflat worlds

i think he meant not being able to drift in superflat worlds

Not sure why a superflat world break drifting, all worlds are worlds.
Or, do you mean drifting breaks at (or below) a certain height? That would make sense to why the problem may occur in default superflat worlds.


Array Programming

Can we have array programming constructs like ranges, slices, broadcasting, elementwise operators and array level operators, etc.

We have plans for some of the things you mention:

The . operator ("map" operator) will run an arbitrary function on all elements of an array: myArrayOfSpaceships.liftoff(5) will call the liftoff(spaceship, height) function on all elements (this is called the map operator).
The |. operator ("flatMap" operator) will do the same, and then flatten the result.

Ranges and slices can be implemented in Vale code, so they might make it into the standard library.
I'm not familiar with broadcasting, what is that? And what do you mean by array level operators?

https://www.tutorialspoint.com/numpy/numpy_broadcasting.htm
https://xtensor.readthedocs.io/en/latest/expression.html#broadcasting

Also see:

http://www.sac-home.org/

Thanks for suggesting this! That's an interesting feature that Julia added.
I'm not convinced that this is worth adding syntactical support for, in Vale. It seems like a very niche use, not something worth complicating the language for, especially when (it looks like) it can be done with a custom function.

Thanks for suggesting this! That's an interesting feature that Julia added.
I'm not convinced that this is worth adding syntactical support for, in Vale. It seems like a very niche use, not something worth complicating the language for, especially when (it looks like) it can be done with a custom function.


docs: Fix Link to PDF Specification

What?

Fix a link in the contributing guide.
Why?

To keep others from searching
How?

N/A
Testing?

N/A
New Dependencies?

N/A
Screenshots

N/A
Suggested Reading?

No
Anything Else?

Checklist

[ ] I read CONTRIBUTING.md.
[ ] I read MAINTAINERSHIP.md#pull-requests.
[ ] I added/updated unit tests for my changes.
[ ] I added/updated integration tests for my changes.
[ ] I ran the integration tests.
[ ] I tested my changes in Node, Deno, and the browser.
[ ] I viewed documents produced with my changes in Adobe Acrobat, Foxit Reader, Firefox, and Chrome.
[ ] I added/updated doc comments for any new/modified public APIs.
[ ] My changes work for both new and existing PDF files.
[ ] I ran the linter on my changes.

dupe


Need to test it with AmiBroker x64

While it works fine with AmiBroker x86 on both Windows x84 and Windows x64, still need to check if this plug-in works with AmiBroker x64

It's been 5 years but I can confirm that it works with AmiBroker x64. You just need to remember to target the project for x64 CPU architecture.

Any success with AFL plugin?

@duzers not yet but this is offtopic to this issue.


Add a Gitter chat badge to README.md

binduwavell/generator-alfresco now has a Chat Room on Gitter
@binduwavell has just created a chat room. You can visit it here: https://gitter.im/binduwavell/generator-alfresco.
This pull-request adds this badge to your README.md:

If my aim is a little off, please let me know.
Happy chatting.
PS: Click here if you would prefer not to receive automatic pull-requests from Gitter in future.

I will integrate the badge into the README manually


OCPBUGS-30319: oauthserviceaccountclient: expect that there might be no tokens associated with SA

/lgtm

/jira refresh


Small Changes on the vDataFrame interaction

I think we can do the following changes:

keep the index white
The footer is a little bit confusing, it says: "Showing 1 to 25 of 100 entries" whereas the vDataFrame has more entries. Maybe, we can write something more precise?

I think it refers to max_rows which is set to 100 by default and represents the number of rows that will be displayed. Interactive tables are built on top of vdataframes so they will inherit any display option from them.

I got it, please change the message as most of the users will not get it.


Tracks no longer show in html files

Hello,
Thanks for developing igv-reports.
Tracks do not load on HTML files that were previously working perfectly fine. In fact, none of the HTMLs that I had previously generated can load data.
Is this an issue with the server or has there been a recent change (last couple of days) that affects the behavior? Is someone else experiencing this?
Thank you very much,
Laura Hinojosa

The data for tracks in an igv-reports html file are internal to the file.  Is it possible to share one of the reports?  If you send an email to<EMAIL_ADDRESS>I can send you a secure dropbox link.
Of particular interest is what version of igv.js the reports are referencing.   This will be near the top of the html file.

Hi Jim,
Thank you for your quick reply.
The version is 2.9.1.
E-mail sent!

OK, got it.  Yes that is a problem caused by a change in our hosted genome file that is not compatible with 2.9.1.   I will fix it on our end, your reports should start working again shortly.

Perfect! Thank you very much!

OK they should be working again.

BTW that's an innovative use of igv-reports,  I take it you created a template file for this.  We don't really have much insight into how people are using this tool,  or if there is even much of a community, so this is nice to see.

Thank you very much for solving this so quickly! Everything is back to normal!


[UI] Settings: Upload K8s Config - Swap position of two buttons

Current Behavior
When adding a cluster, users are presented with this modal (screenshot) on the http://localhost:9081/settings Settings page.

Desired Behavior

"Cancel" should be on the right side while "Upload" is on the left.2.
Rename "Upload" to "Import"2.
"Cancel" should be using the light gray background color, while "Import" uses the charcoal-ish color.

Contributor Guides and Resources

🎨 Wireframes and designs for Meshery UI in Figma
🖥 Contributing to Meshery UI
🙋🏾🙋🏼 Questions: Layer5 Discussion Forum and Layer5 Community Slack

@leecalcote, I would love to work on this

Sounds great. Thanks, @Rohith-Raju

@Rohith-Raju while you are working on this, also consider correcting the spelling as shown in the pic.

Good eye, @Santosh1176

@Rohith-Raju The config file upload functionality seems to be broken. Could you check at your end.


It is not working on umbraco 11

It is not working on umbraco 11

We know https://github.com/Shazwazza/Articulate/issues/381


FEATURE: Add (optional) integration with the queuedjobs module

Automatically creates a ContentReviewNotificationJob to run the next day if the queuedjobs module exists

Does nothing if the queuedjobs module does not exist, to ensure it is optional

Update composer to suggest queuedjobs module, and updated docs to reflect both ways of configuring the module

Thanks Matt,
I have manually merged this into the cwp-support branch instead.