[CupertinoDatePicker] Selection background is broken for mode "time" and "dateAndTime"

Steps to Reproduce

Create an app with a [CupertinoDatePicker] and select the mode "time"
Inspect the result

Expected results: There should be one grey background around the current select (as it can be seen for the mode "date")

Actual results: The background of the selection if cut into pieces with weird rounding

Reproducible on 1.25.0-8.1.pre (current beta)
import 'package:flutter/material.dart';

void main() {
  runApp(MaterialApp(
    home: MyHomePage(),
  ));
}

class MyHomePage extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return Scaffold(
      body: Center(
        child: Container(
          height: 300,
          child: CupertinoDatePicker(
            mode: CupertinoDatePickerMode.time,
            onDateTimeChanged: (DateTime newDateTime) {},
          ),
        ),
      ),
    );
  }
}```

I was able to reproduce the issue on latest master, beta and dev channel.

stable channel
master, beta and dev channel

flutter doctor -v
[✓] Flutter (Channel master, 1.26.0-2.0.pre.168, on Mac OS X 10.15.7 19H2
    darwin-x64, locale en)
    • Flutter version 1.26.0-2.0.pre.168 at
      /Users/pedromassango/dev/SDKs/flutter_master
    • Framework revision 299e017e41 (21 hours ago), 2021-01-03 06:29:01 -0500
    • Engine revision 121b67fb85
    • Dart version 2.12.0 (build 2.12.0-179.0.dev)

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.2)
    • Android SDK at /Users/pedromassango/Library/Android/sdk
    • Platform android-30, build-tools 30.0.2
    • 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-6915495)
    • All Android licenses accepted.

[!] Xcode - develop for iOS and macOS (Xcode 12.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.1, Build version 12A7403
    ! CocoaPods 1.9.3 out of date (1.10.0 is recommended).
        CocoaPods is used to retrieve the iOS and macOS platform side's plugin
        code that responds to your plugin usage on the Dart side.
        Without CocoaPods, plugins will not work on iOS or macOS.
        For more info, see https://flutter.dev/platform-plugins
      To upgrade see
      https://guides.cocoapods.org/using/getting-started.html#installation for
      instructions.

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

[✓] Android Studio (version 4.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6915495)

[✓] IntelliJ IDEA Community Edition (version 2020.3)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    • Flutter plugin version 52.1.5
    • Dart plugin version 203.5981.152

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

[✓] Connected device (3 available)
    • Android SDK built for x86 (mobile) • emulator-5554 • android-x86    •
      Android 10 (API 29) (emulator)
    • macOS (desktop)                    • macos         • darwin-x64     • Mac
      OS X 10.15.7 19H2 darwin-x64
    • Chrome (web)                       • chrome        • web-javascript •
      Google Chrome 87.0.4280.88

! Doctor found issues in 1 category.

I was able to reproduce the issue on latest master, beta and dev channel.

stable channel
master, beta and dev channel

flutter doctor -v
[✓] Flutter (Channel master, 1.26.0-2.0.pre.168, on Mac OS X 10.15.7 19H2
    darwin-x64, locale en)
    • Flutter version 1.26.0-2.0.pre.168 at
      /Users/pedromassango/dev/SDKs/flutter_master
    • Framework revision 299e017e41 (21 hours ago), 2021-01-03 06:29:01 -0500
    • Engine revision 121b67fb85
    • Dart version 2.12.0 (build 2.12.0-179.0.dev)

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.2)
    • Android SDK at /Users/pedromassango/Library/Android/sdk
    • Platform android-30, build-tools 30.0.2
    • 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-6915495)
    • All Android licenses accepted.

[!] Xcode - develop for iOS and macOS (Xcode 12.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.1, Build version 12A7403
    ! CocoaPods 1.9.3 out of date (1.10.0 is recommended).
        CocoaPods is used to retrieve the iOS and macOS platform side's plugin
        code that responds to your plugin usage on the Dart side.
        Without CocoaPods, plugins will not work on iOS or macOS.
        For more info, see https://flutter.dev/platform-plugins
      To upgrade see
      https://guides.cocoapods.org/using/getting-started.html#installation for
      instructions.

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

[✓] Android Studio (version 4.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6915495)

[✓] IntelliJ IDEA Community Edition (version 2020.3)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    • Flutter plugin version 52.1.5
    • Dart plugin version 203.5981.152

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

[✓] Connected device (3 available)
    • Android SDK built for x86 (mobile) • emulator-5554 • android-x86    •
      Android 10 (API 29) (emulator)
    • macOS (desktop)                    • macos         • darwin-x64     • Mac
      OS X 10.15.7 19H2 darwin-x64
    • Chrome (web)                       • chrome        • web-javascript •
      Google Chrome 87.0.4280.88

! Doctor found issues in 1 category.

I will check it.

I will check it.


don't call accentDrop at all for empty input

I confirm that this contribution is made under the terms of the license found in the root directory of this repository's source tree and that I have the authority necessary to make this contribution on behalf of its copyright owner.

@geirst and/or @bratseth please review and merge


"About" screen on Mac shows "Version 1.0.0"

(i know, not a big deal, but thought you might want to know)

I'm aware, I just can't be bothered to edit info.plist in the .app file every time I make a new version.

Maybe just for the 1.x versions then?

Maybe, but the version is clearly visible in the window title and on the actual about screen in the program, so I'm not sure if I bother, to be honest...

Uless there are good reasons to do it (I'm not aware of any).


fix: don't panic on STUN requests for unknown interfaces

Resolves: #473.

While I accept we don't have to panic, I still believe the user should configure the ICE agent correctly. Silently discarding random UDP packets means it's easy to introduce bugs and not notice a misconfiguration.

We do use accepts_message in our code and I can still trigger this panic. That being said, we don't yet have https://github.com/algesten/str0m/pull/488in our fork so that might change things?

accepts_message only checks the message and doesn't have access to the destination SocketAddr so it cannot check whether it has a candidate for that:
https://github.com/algesten/str0m/blob/3c5edeea9bb12f62d3e5b185bae6be7165700228/src/ice/agent.rs#L767

We do use accepts_message in our code and I can still trigger this panic. That being said, we don't yet have https://github.com/algesten/str0m/pull/488in our fork so that might change things?

Interesting! That would be good to know. It makes a big difference to me if accepts_message says "yes" and handle_message says "panic"!
Spontaneously it seems these should be in lockstep.

We do use accepts_message in our code and I can still trigger this panic. That being said, we don't yet have algesten/str0m/pull/488in our fork so that might change things?

Interesting! That would be good to know. It makes a big difference to me if accepts_message says "yes" and handle_message says "panic"!
Spontaneously it seems these should be in lockstep.

See the comment above. i think we'd have to change accepts_message to accept a StunPacket to make it say "no" to packets from unknown interfaces.
Note that the way this panic gets triggered for us is:

We invalidate all other candidates as soon as we have a working path
We want to delay sending of relay candidates by ~1s to first check if we can hole-punch

Combinding (1) and (2) leads to a race condition where we might have already invalidated a candidate but the remote wants to still send use STUN requests.

My take on it is:

IceAgent::handle_packet must not panic, regardless what data you feed it. IMO, one of the aspects of a SANS-IO design is that you can (directly) feed it unstructured input from the network. Thus, it must not panic or otherwise we have a DoS attack vector. I think the correct solution for this fn is to discard the message.
IceAgent::accepts_message is useful when multiple agents are multiplexed on a single socket. IMO there isn't a lot of value in having this function (we've discussed this before but I'll mention it for completeness). Processing a message and checking whether we want to do so requires very similar code. Code that must not diverge. To me, it seems like we could just do whatever we do in accepts_message at the very top of handle_message and return true | false from it, indicating whether we've handled the message. It might not hold true for the entire Rtc instance but at least for the IceAgent, doing the actual processing should be very cheap.

Perhaps as a compromise, we could make accepts_message pub(crate) such that the Rtc instance can still call it but the public API of IceAgent doesn't expose it? Then it wouldn't be a silent discard but just returning false to indicate that we didn't handle it.

We can also (additionally) align the two APIs such that the entire packet can be packet can be passed to accepts_message and checked early.

If we return false now, should we also change stun_server_handle_message to return false in case we discard it based on an unknown candidate?

I tried that but didn't like where it was going. Places like if candidate.discarded { return are not clear whether it's true or false.
I decided that rather than muddle the semantics, the bool is exactly that of accepts_message.

I decided that rather than muddle the semantics, the bool is exactly that of accepts_message.

Okay, in that case, isn't that commit unrelated to this change? I am not sure how atomic you want to be on your PRs but it could be done separately in that case.
Otherwise, I am happy with this going in :)

Let's merge it! I'll reference the PR twice in the changelog.


Fock matrix with localized molecular orbitals

Hi all,
I'm interested in calculating the Fock matrix with (previously localized) localized molecular orbitals.
I'm gonna need the core Hamiltonian, but this is implemented for the AO basis only.
Do you have idea of how can I calculate the core Hamiltonian for previously localized LMO (or any molecular orbitals)?
On the other hand, for the two electron part of the Fock matrix, I think I can use the "ao2mo" module ... Is this the best way to do it?
Thanks in advance,
Daniel

Hi Daniel,
This is just a change of basis, which you can do with @.***@C`, where
either:

the columns of C are your localized MOs in terms of canonical MOs and
fock is a diagonal matrix of MO energies
the columns of C are your localized MOs in terms of AOs and fock is
the fock matrix in the AO basis

On Tue, Mar 15, 2022 at 6:23 PM DanielBajac @.***>
wrote:

Hi all,
I'm interested in calculating the Fock matrix with (previously localized)
localized molecular orbitals.
I'm gonna need the core Hamiltonian, but this is implemented for the AO
basis only.
Do you have idea of how can I calculate the core Hamiltonian for
previously localized LMO (or any molecular orbitals)?
On the other hand, for the two electron part of the Fock matrix, I think I
can use the "ao2mo" module ... Is this the best way to do it?
Thanks in advance,
Daniel
—
Reply to this email directly, view it on GitHub
https://github.com/pyscf/pyscf/issues/1210, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADGAJCZI7I6A4Q437XVUDTTVAEEVDANCNFSM5Q2DJPTA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Hi Timothy, thank you for your fast answer.
I'm glad to hear that it is just a change in basis.
However, I don't fully understand your answer,  what do you mean with @.***@C ?
On the other hand, the LMO (e.g Pipek-Mezey, Boys, etc..) obtained by defect in the PySCF are in term of AOs?

Hi Daniel,
The two ways Tim described are either:
1:
c_lo = Boys(mol).kernel(mf.mo_coeff).mo_coeff                          # LOs in terms of AOs
c_lo = np.linalg.multi_dot((mf.mo_coeff.T, mf.get_ovlp(), c_lo))       # LOs in terms of MOs
fock_lo = np.einsum('ix,i,iy->xy', c_lo, mf.mo_energy, c_lo)

2:
c_lo = Boys(mol).kernel(mf.mo_coeff).mo_coeff                          # LOs in terms of AOs
fock_lo = np.einsum('ax,ab,by->xy', c_lo, mf.get_fock(), c_lo)

(Note that these will only be the same for converged Hartree-Fock calculations)
As for your second question, the Fock matrix represents a one-electron operator and doesn't have a "two-electron part".
The best way to get the two-electron integrals of the ab-initio Hamiltonian depends on the kind of system, but generally you could use
c_lo = Boys(mol).kernel(mf.mo_coeff).mo_coeff                          # LOs in terms of AOs
nmo = len(mf.mo_energy)
eris = ao2mo.kernel(mol, c_lo, compact=False).reshape([nmo]*4)

To get the integrals in the LO basis.

Hi Max,
Thanks for the explanation! It was very helpul. Of course, I was confused about the two-electron part of the Fock matrix.
Thanks again to Tim for his answer.
Best wishes

If you're using PySCF's Boys functionality, then you should definitely use
option 2, as shown cleanly by Max. No need to go into the canonical MO
basis first. Just to add: my use of @ was because it is the matrix
multiplication operator as of Python 3.5, so Max's final line could be
replaced by:
fock_lo = ***<EMAIL_ADDRESS>On Wed, Mar 16, 2022 at 10:44 AM DanielBajac @.***>
wrote:

Hi Max,
Thanks for the explanation! It was very helpul. Of course, I was confused
about the two-electron part of the Fock matrix.
Thanks again to Tim for his answer.
Best wishes
—
Reply to this email directly, view it on GitHub
https://github.com/pyscf/pyscf/issues/1210#issuecomment-1069206280, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ADGAJC5KJBTKAER7AYDZDRTVAHXWLANCNFSM5Q2DJPTA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you commented.Message ID:
@.***>

I think when @tberkel is replying by email something is obfuscating matrix multiplies because they look like email addresses.
What I think was intended is that
fock_lo = np.einsum('ax,ab,by->xy', c_lo, mf.get_fock(), c_lo)
could be replaced by
fock_lo = c_lo.T @ mf.get_fock() @ c_lo

Wow you're absolutely right, thanks for catching that. Sorry for the confusion. I've edited my replies just so that they're clear in the future.


Add info about which signing keys will be used for published artifacts.

Add info about which signing keys will be used for published artifacts.
For security purposes, it would be great if you were able to publish the gpg public keys that are "valid" for use when verifying signing artifacts uploaded to maven central.
This allows for "out of band" verification of the expected signing key.
Some examples of other libs publishing their signing keys:
https://square.github.io/okhttp/security/security/#verifying-artifacts
https://github.com/eclipse/jetty.project/blob/jetty-10.0.x/KEYS.txt
https://downloads.apache.org/commons/KEYS
https://downloads.apache.org/logging/KEYS

annotations 2.7.1 seems to be signed with e77417ac194160a3fabd04969a259c7ee636c5ed
https://keyserver.ubuntu.com/pks/lookup?search=e77417ac194160a3fabd04969a259c7ee636c5ed&fingerprint=on&op=index

Seconded. I'm doing exactly this kind of due diligence for a project which uses Guava, which has just bumped its dependency to one which is signed by what appears to be @cushon's new key.
Confirmation of this in the form of a KEYS file in the repository (or some other mechanism) would be very welcome.

error_prone_annotations-2.18.0.jar looks to be signed with this key.
https://keyserver.ubuntu.com/pks/lookup?search=D364ABAA39A47320&fingerprint=on&op=index

The key used to sign releases has been published here: https://github.com/google/error-prone/blob/master/KEYS.txt


ldap: set the email to null if it couldn't be guessed

This way we avoid problems with two consecutive LDAP users trying to sign up
with no email. In order to do this, I've had to enable null values in the email
field, which needed some small modifications here and there.
Fixes #514
Signed-off-by: Miquel Sabaté Solà<EMAIL_ADDRESS>

LGTM, just take advantage of .blank? to make the code nicer

@Salzig good catch. Fixed.


DocumentCloud embeds break mobile layout

They're too wide :(

Looks like this worked. Thanks again, @knowtheory.


multimaster_fkie: 0.5.0-0 in 'indigo/distribution.yaml' [bloom]

Increasing version of package(s) in repository multimaster_fkie to 0.5.0-0:

upstream repository: http://github.com/fkie/multimaster_fkie.git
release repository: https://github.com/fkie-release/multimaster_fkie-release.git
distro file: indigo/distribution.yaml
bloom version: 0.5.21
previous version for package: 0.4.4-0

default_cfg_fkie
* default_cfg_fkie: added 'load_params_at_start' parameter.
  On start of default_cfg_fkie all parameters are loaded into ROS
  parameter server. If this parameter is set to False the parameter are
  loaded on first run of an included node.
* mulitmaster_fkie: fixed the missing leading SEP by groups
* Contributors: Alexander Tiderko

master_discovery_fkie
* master_discovery: fixed avg. network load calculation, added checks for some parameters
* multimaster_fkie: Set correct logging level to warning
* Contributors: Alexander Tiderko, Gary Servin

master_sync_fkie

No changes

multimaster_fkie
New Features:
* node_manager_fkie: the start with different ROS_MASTER_URI
* node_manager_fkie: added parameter to disable the highlighting of xml blocks
* node_manager_fkie: added ROS-Launch tags to context menu in XML editor
* node_manager_fkie: mark XML tag blocks
* node_manager_fkie: show the filename in the XML editor dialog title
* node_manager_fkie: close configuration items are now sorted
* node_manager_fkie: the confirmation dialog at exit can be deaktivated
  to stop all nodes and roscore or shutdown the host you can use the close
  button of each master
* node_manager_fkie: allow to shutdown localhost
* node_manager_fkie: shows 'advanced start' button also if the selected node laready runs
* default_cfg_fkie: added 'load_params_at_start' parameter.
  On start of default_cfg_fkie all parameters are loaded into ROS
  parameter server. If this parameter is set to False the parameter are
  loaded on first run of an included node.
Fixes:
* node_manager_fkie: fixed print XML content in echo_dialog
* node_manager_fkie: avoids the print of an error, while loads a wrongs file on start of the node_manager
* node_manager_fkie: fixed check of running remote roscore
* node_manager_fkie: fixed problem while echo topics on remote hosts
* node_manager_fkie: changed cursor position in XML editor after open node configuration
* node_manager_fkie: fixed replay of topics with array elements
* node_manager_fkie: fixed close process while start/stop nodes
* node_manager_fkie: fixed namespace of capability groups, fixed the missing leading SEP
* node_manager_fkie: fixed - avoid transmition of some included/changed but not needed files to remote host
* node_manager_fkie: fixed start node after a binary was selected from multiple binaries
* node_manager_fkie: removed "'now' FIX" while publish messages to topics
* node_manager_fkie: fixed log format on remote hosts
* master_discovery: fixed avg. network load calculation, added checks for some parameters
* multimaster_fkie: Set correct logging level to warning
* Contributors: Alexander Tiderko, Gary Servin

multimaster_msgs_fkie

No changes

node_manager_fkie
New Features:
* node_manager_fkie: the start with different ROS_MASTER_URI
* node_manager_fkie: added parameter to disable the highlighting of xml blocks
* node_manager_fkie: added ROS-Launch tags to context menu in XML editor
* node_manager_fkie: mark XML tag blocks
* node_manager_fkie: show the filename in the XML editor dialog title
* node_manager_fkie: close configuration items are now sorted
* node_manager_fkie: the confirmation dialog at exit can be deaktivated
  to stop all nodes and roscore or shutdown the host you can use the close
  button of each master
* node_manager_fkie: allow to shutdown localhost
* node_manager_fkie: shows 'advanced start' button also if the selected node laready runs
Fixes:
* node_manager_fkie: fixed print XML content in echo_dialog
* node_manager_fkie: avoids the print of an error, while loads a wrongs file on start of the node_manager
* node_manager_fkie: fixed check of running remote roscore
* node_manager_fkie: fixed problem while echo topics on remote hosts
* node_manager_fkie: changed cursor position in XML editor after open node configuration
* node_manager_fkie: fixed replay of topics with array elements
* node_manager_fkie: fixed close process while start/stop nodes
* node_manager_fkie: fixed namespace of capability groups, fixed the missing leading SEP
* node_manager_fkie: fixed - avoid transmition of some included/changed but not needed files to remote host
* node_manager_fkie: fixed start node after a binary was selected from multiple binaries
* node_manager_fkie: removed "'now' FIX" while publish messages to topics
* node_manager_fkie: fixed log format on remote hosts
* Contributors: Alexander Tiderko

Your python-rosdistro version is out of date - you need 0.4.4 or higher. Please upgrade and rerun bloom-release with --pull-request-only or -p or edit the commit to fix the REP version.

Thanks for the hint, I've created new pull request.


Add syntax highlight for language "django-html"

Is this request related to a problem? Please describe.
A clear and concise description of what the problem is (e.g. "I'm always frustrated when […]").
Describe the solution you'd like
A clear and concise description of what you want to happen.
Additional context
Add any other information or screenshots about the feature request here (e.g. a gif showing how another tool does it).

The extension asked me to post an issue, I'm not gonna bother filling it in, but i will post it. Lol

Well, thank you for opening the issue. That made us aware of the django-html and django-txt languages that are contributed by https://marketplace.visualstudio.com/items?itemName=batisteo.vscode-django
I added support for these, which will be part of the next release.
https://github.com/all-contributors please add Void-n-Null for ideas

@all-contributors please add Void-n-Null for ideas


sql count 统计值为0时，结果返回 Empty set

现象：
mysql> select count(1) from test2 a  inner join test b  on a.did = b.did group by a.did;
Empty set (1 min 27.16 sec)
期望：返回值为 0

There is a problem in this sql statement. When the result set of table a inner join table b is empty, it is not reasonable to group by a.did. You should remove the group by a.did clause then the return value is 0.


XL-search and Glycan search

update crosslink scoring method.
enable glycan search.
Fix bug reported by users.

This really needs to be broken up into two pull requests.

Cross-link stuff
Glycan search

I'm happy to merge any changes/fixes to cross-link.
The glycan search appears to affect a lot of core code. Also there is a huge 3 million line glycan database from another group attached? I think this search is going to require some discussion. Also, for the sake of Shortreed's sanity, I think we should hold off on merging large changes to the code until the new mzLib changes are merged in. I think it will affect how glycans are searched for too.

I don't think it is easy to break into two pull request. I change a lot of basic stuff. I'll delete this pull request. And fix the bugs reported first.

I don't know how to delete this pull request. Please help delete it if you want to.

If you want help splitting this I can talk to you about it. It will involve making another fork and copying some changes over to that one, then making a pull request here from that fork. I'll talk to you about it tmw

OK.

I had to do a lot of this breaking up of pull requests in July. It's a hassle, but helps clarify the pull request by making sure all the changes are related to a certain goal.
I'd recommend:

Make a separate clone fresh from the master branch on your computer
Make a branch (git checkout -b CrossLinkStuff) in the new clone
Transfer the minimal set of changes needed for the new feature
Push the new branch to smith-chem-wisc directly (git push origin CrossLinkStuff, where origin is smith-chem-wisc on the new clone)
Make a pull request from smith-chem-wisc/CrossLinkStuff to smith-chem-wisc/master. This also allows others (e.g. Rob or me) to push small commits to fix typos or the like.


CFFL

CFFL

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_7brx4nv6rf/clone/recipe/meta.yaml:14: pip_install_args: pip install should be run with --no-deps.
ERROR: /tmp/abs_7brx4nv6rf/clone/recipe/meta.yaml:20: missing_wheel: For pypi packages, wheel should be present in the host section
WARNING: /tmp/abs_7brx4nv6rf/clone/recipe/meta.yaml:48: missing_description: The recipe is missing a description
ERROR: /tmp/abs_7brx4nv6rf/clone/recipe/meta.yaml:48: missing_dev_url: The recipe is missing a dev_url
ERROR: /tmp/abs_7brx4nv6rf/clone/recipe/meta.yaml:48: missing_documentation: The recipe is missing a doc_url or doc_source_url
Errors were found

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_b3syl64h_p/clone/recipe/meta.yaml:14: pip_install_args: pip install should be run with --no-deps.
ERROR: /tmp/abs_b3syl64h_p/clone/recipe/meta.yaml:20: missing_wheel: For pypi packages, wheel should be present in the host section
ERROR: /tmp/abs_b3syl64h_p/clone/recipe/meta.yaml:48: missing_documentation: The recipe is missing a doc_url or doc_source_url
WARNING: /tmp/abs_b3syl64h_p/clone/recipe/meta.yaml:48: missing_description: The recipe is missing a description
ERROR: /tmp/abs_b3syl64h_p/clone/recipe/meta.yaml:48: missing_dev_url: The recipe is missing a dev_url
Errors were found


Modify record.py to accept a command line description of the task that is being performed

We would like to implement the following functionality:
python puterbot/record.py "<description of the task to be recorded>"

For example:
python puterbot/record.py "a physician auditing billing codes (excel and powerchart)"

This should be saved to the Recording table, and passed into large language model as context.

Implemented in https://github.com/MLDSAI/puterbot/pull/16


[BUG] Authorization header is not sent through swagger

It seems that swagger interface cannot send Authorization header.
I added test_auth endpoint with required header authorization like that:
async def check_auth(
        authorization: str = Header(...)
):
    pass

@router.post("/test_auth")
async def test_auth(
        auth = Depends(check_auth)
):
    pass

This works perfectly if I use REST-client with set Authorization header. But, if I use swagger UI it seems that Authorization header always missing. Server responses with status code 422 and:
  "detail": [
    {
      "loc": [
        "header",
        "authorization"
      ],
      "msg": "field required",
      "type": "value_error.missing"
    }
  ]
}

Another observation is if I change the header name to something else (not authorization) it works. But my goal is using Authorization...
FastAPI version: 0.49.0

@phy25 I've read. May you provide a small example using HTTPBearer, please? (there is no even mention of HTTPBearer in manual)

If you want to log an user maybe you should try OAuth2PasswordBearer

@alvarogf97 Just need to check JWT token. No user.

https://github.com/tiangolo/fastapi/blob/55afb70b3717969565499f5dcaef54b1f0acc7da/tests/test_security_http_bearer.py#L7-L12

Take a look of this example https://github.com/tiangolo/fastapi/issues/1034#issuecomment-591651300

Thanks for the help here @phy25 and @alvarogf97 ! :cake: :bow:
If that answers your question @symonsoft , you can then close the issue.


Add command-completion for PowerShell invocation command line

Summary of the new feature/enhancement
Command-completion (tab-completion) is a nice capability It would be helpful to have command-completion for parameters when invoking PowerShell from a PowerShell console.

Hi @Liturgist, I want to verify I understand your feature request, from within a PowerShell console or terminal running PowerShell, you want tab-completion to complete the pwsh.exe syntax below?
pwsh[.exe] [-Login] [[-File]  [args]]
[-Command { - |  [-args ]
|  [] } ]
[-ConfigurationName ] [-CustomPipeName ]
[-EncodedCommand ]
[-ExecutionPolicy ] [-InputFormat {Text | XML}]
[-Interactive] [-MTA] [-NoExit] [-NoLogo] [-NonInteractive] [-NoProfile]
[-OutputFormat {Text | XML}] [-SettingsFile ] [-SSHServerMode] [-STA]
[-Version] [-WindowStyle

@theJasonHelmick I'm pretty certain you're right and that's what they're asking for.  A build in argument completer for it would be great, I always found it strange that there wasn't one.

@theJasonHelmick, yes, I believe you understand correctly. My apologies for inattention. I still think this would be a helpful addition.

@Liturgist - Thank you for the response and the suggestion -- I think this could be a benefit to customers unfamiliar with pwsh.exe syntax.  I'll discuss further with the working group and report back.

I checked code in a few times, but always had to resolve some issue. I could never overcome a rebase, so gave up.

We could add completer for pwsh.exe

@Liturgist @iSazonov -- After discussing this with the working group, we agree that this would be a beneficial feature for users.  I'm marking this up-for-grabs if the community would like to consider building this solution.

Would it be OK to add a reference to System.CommandLine?

Would it be OK to add a reference to System.CommandLine?

Any reference we add essentially version locks that library for anyone else who wants to consume it. There's a pretty high bar for impact that needs to be hit for that to be considered

This is still the case. Just waiting for someone to take it on.
PS C:> $psversionTable.psversion.ToString()
7.3.9

This is still the case.
PS C:> $PSVersionTable.PSVersion.ToString()
7.4.2

@Liturgist I'll take this on. Let me know if someone is already working on it.

I made an argument completer for pwsh!
https://gist.github.com/jshigetomi/58cb79de086c804cda3173ef63d7dddf
Please leave some comments and ideas. I hope we can incorporate this completer during runtime once it's approved.

@jshigetomi, I will look forward to trying it out. Thanks.


Actions does not run when template is forked

I remember when I presented this, it was not the case. Should it not run? Do we need to update the three steps with this?

Test fork: https://github.com/layik/jsoncli

Trying again https://github.com/tgve/apptest2

We could commit the change in the step but fine as is

Yes only docker runs


Coded UI Tests

Hey folk. I have question about coded ui test. Is it possible with current frameworks like Selenium and WinAppDriver to test the UI of Avalonia via Pipeline? I look up my application with snoop and uiverify from win10 and cant find anything about my application.

Duplicate of https://github.com/AvaloniaUI/Avalonia/issues/2157


[EWEPNT] Phantasia - Crashes entering world after character creation

Module Information

Identifier: EWEPNT
Module Name: Phantasia
Version: (C)1995

Describe the bug
Crashes entering world after character creation
To Reproduce
Steps to reproduce the behavior:

Load Module
Enter Module
Create Character
See error

Expected behavior
Ability to create character and enter world
Screenshots
2020-11-15 17:24:53.7480 Warn MBBSEmu.HostProcess.ExportedModules.Majorbbs.prfmsg prfmsg() unable to locate message number 1973 in current MCV file EWEPNT.MCV
Unhandled exception. System.ArgumentOutOfRangeException: Specified argument was out of the range of valid values. (Parameter 'Unsupported OpCode: Fchs')
   at MBBSEmu.CPU.CpuCore.Tick()
   at MBBSEmu.HostProcess.ExecutionUnits.ExecutionUnit.Execute(IntPtr16 entryPoint, UInt16 channelNumber, Boolean simulateCallFar, Boolean bypassState, Queue`1 initialStackValues, UInt16 initialStackPointer)
   at MBBSEmu.Module.MbbsModule.Execute(IntPtr16 entryPoint, UInt16 channelNumber, Boolean simulateCallFar, Boolean bypassSetState, Queue`1 initialStackValues, UInt16 initialStackPointer)
   at MBBSEmu.HostProcess.MbbsHost.WorkerThread()
   at System.Threading.ThreadHelper.ThreadStart_Context(Object state)
   at System.Threading.ExecutionContext.RunInternal(ExecutionContext executionContext, ContextCallback callback, Object state)
--- End of stack trace from previous location where exception was thrown ---
   at System.Threading.ThreadHelper.ThreadStart()

Software Information:

OS: Windows Server 2019 Standard
Telnet Client: NetRunner
MBBSEmu Build: 111220-2

Wanted to let you know I'm the original author of Phantasia back in the MBBS days. Not sure I still have the source code anymore (and not sure it would even help). The error message looks like the MSG isn't in the MCV file. This might be a bug that the MSG is missing the message or the MCV is out of sync with the MSG.

Hey Bil! Welcome to the fun! 😄
The warning of the missing message above is a bit of a red herring, the line after it is the exception being thrown that the x86 core is missing the opcode implementation for FCHS.
We'll need to get FCHS added, which changes the sign value in ST(0) (negative->positive, positive->negative) in ht x87 FPU Stack

I found a new version 0.3Beta ... it throws this OPCode -- guessing it will need both though
2020-11-17 06:45:36.9601 Error MBBSEmu.HostProcess.ExportedModules.ExportedModuleBase.FormatPrintf Invalid Pointer: 2061:6E61
Unhandled exception. System.ArgumentOutOfRangeException: Specified argument was out of the range of valid values. (Parameter 'Unsupported OpCode: Fsubp')
   at MBBSEmu.CPU.CpuCore.Tick()
   at MBBSEmu.HostProcess.ExecutionUnits.ExecutionUnit.Execute(IntPtr16 entryPoint, UInt16 channelNumber, Boolean simulateCallFar, Boolean bypassState, Queue`1 initialStackValues, UInt16 initialStackPointer)
   at MBBSEmu.Module.MbbsModule.Execute(IntPtr16 entryPoint, UInt16 channelNumber, Boolean simulateCallFar, Boolean bypassSetState, Queue`1 initialStackValues, UInt16 initialStackPointer)
   at MBBSEmu.HostProcess.MbbsHost.WorkerThread()
   at System.Threading.ThreadHelper.ThreadStart_Context(Object state)
   at System.Threading.ExecutionContext.RunInternal(ExecutionContext executionContext, ContextCallback callback, Object state)
--- End of stack trace from previous location where exception was thrown ---
   at System.Threading.ThreadHelper.ThreadStart()

I'm working on a new branch right now with new opcodes for Crossroads of the Elements, I'll get these two added to that as well! 👍

@bsimser Hey Bil -- If you're able to locate the source code (for this or other modules) let me know! It would be a huge help to getting it emulated properly 😄
You can reach me directly at<EMAIL_ADDRESS>


Update DLR

I bumped up the System.CodeDom package for .NET 6 in the DLR but when make.ps1 package is run it's restoring the old version (because of the DLR reference in the IronPython.Console project) to the bin folder and causing the tests to fail on github CI (which does packaging before testing). @BCSharp any good ideas?

Strange that the package is being pulled in to the main bin directory. Would it help to replace the DLR package reference with regular references to the individual DLR DLLs in the stage directory?


[examples/pigweed-app] Added README file to the nRF Connect example.

Problem
There is lack of information how to build pigweed-app example on the nRF Connect platform.
Summary of Changes

Added README file containing information how to build and use example.

I think you should also mention source scripts/activate.sh step as that is necessary to build pigweed libraries.

I think you should also mention source scripts/activate.sh step as that is necessary to build pigweed libraries.

Right, done.

@saurabhst @BroderickCarlin @chrisdecenzo @jelderton @hawk248


Revert "chore(deps): update container image metallb/controller to v0.10.2"

Reverts Truxnell/home-cluster#191

/merge


Complete a transcript for the XC3 Version 2 video.

User stories
#1 As a DevOps Engineer, I have to write a transcript for the XC3 version 2 video.
Acceptance Criteria
What must be achieved to consider this story complete?

[x] Write transcript and ask team for review.

[ ] addressed all reviews and finalize the transcript.

https://docs.google.com/document/d/1JmzwrDJ46OPZ1RfBO5WPV0rwAkflQqHbusEPb1jI7co/edit?usp=sharing


redux-thunk "actions may not have undefined type property"

Since upgrading to 1.1.0 or above (previously on 1.0.4), using redux-thunk, dispatching a thunk fails with "Actions may not have an undefined "type" property. Have you misspelled a constant?" (this was introduced in 2f7fa4400e).
For example, my action creator returns a thunk:
export function fetchSomething(smthg) {
  return dispatch => {
    return API.someCall(smthg).then(things => {
      dispatch(plainActionCreator(things));
      return things;
    });
  };
}

As well, I've configured the mock store to use redux-thunk.
My tests simply dispatch this action using a mock store:
describe('Blah', () => {
    it('should work', () => {
       return store.dispatch(fetchSomething('code'));
   });
});

I am having this same problem. Please let me know if you find a solution.

Can you please add error logs? Would be easier identify a problem.

@finaiized Is there any specific reason why do you make return things; from action creator?

@dmitry-zaets - Sorry, should have simplified the code. The only error message I get is "Actions may not have an undefined "type" property. Have you misspelled a constant?". Basically, anytime I attempt to dispatch any thunk of form
export function thunkCreator() {
    return dispatch => //... do something
}

I run into this error.
All the other details (API calls, returns, etc) are just implementation details - they did work fine on 1.0.4. For what it's worth, calling those functions in my app works fine and as expected.

@finaiized Problem is really in details.
They were working before because the mock store was not checking for empty actions. Empty actions are forbidden by Redux itself and all that we've added on that commit - is check from Redux.
Would be much easier to reproduce your issue if you will provide more data.

@finaiized can you show how you're creating your mock store? It should look like configureStore([thunk]).

Thanks for the help guys. Unsurprisingly, it was a problem on my end - as I was coming up with a repro case for this and couldn't, I looked closer at my existing tests. The problem wasn't actually in my thunk action creators, but in a couple of tests where I mocked them. I returned the value of what I was expecting from my thunk action creators in the mock, failing to remember that I needed to return a function.
Eg.
export function thunkActionCreator() {
    return dispatch => Promise.resolve([]); // Do some real work here
}

// Wrong
// Returns a value
const stub = sinon.stub(actions, 'thunkActionCreator').returns(Promise.resolve([]));

// Right
// Returns a function
const stub = sinon.stub(actions, 'thunkActionCreator').returns(() => Promise.resolve([]));

Obviously, since my values have no type, it failed.
 Glad that it is useful for you 👍


WIP: DO NOT MERGE: Testing CDash PR link

The GitHub Check is configured, but we will have to revisit after the builds use a CTest script that populates the Update - Revision CDash entry (necessary for the check).


Ignore shoots in deletion when removing versions from a cloud profile

How to categorize this PR?
/area ops-producitify
/kind enhancement
What this PR does / why we need it:
Ensures that shoots currently being deleted are ignored when removing Kubernetes or image versions from a cloud profile.
Which issue(s) this PR fixes:
Fixes #4142
Special notes for your reviewer:
Release note:
When removing Kubernetes or image versions from a cloud profile, shoots currently being deleted are now ignored.

/invite @vpnachev @rfranzke

/cc @danielfoehrKn as he worked on deprecating K8s and machine image versions.


Fix Script to determine whether CCF TP is ready for user TXNs

Currently we have check-for-ccf-keys.sh as a proxy to determine whether CCF TP is ready for user txns. Based on observations from both @bvavala  and @cmickeyb, it seems insufficient given some of the recent PRs.
We need to re-visit check-for-ccf-keys.sh, rename it as is-ccf-tp-ready.sh and define it properly.

Rather than checking for the keys (which has any number of ways to break), is there a request we can ask of the TP? Similar to what we use the /info path for in each of the other services.

Is there a reason why we cant use the ping transaction? The ccf tp has a script ping_test.py that can at least be used to test for liveness.

@cmickeyb Is this Issue still relevant after 468?

I think this issue can be closed now, after the recent improvements to CCF TP start up ? @cmickeyb  Please comment.


Bot doesn't play any music, no errors in console

The music bot is silently failing to play any songs. After a !play command, it downloads the song, but it's never queued or played (no sound and !queue or !np report that there are no songs playing).
Connected!  Musicbot v1.9.5_2

Bot:   <ID>/Tenno DJ#9082
Owner: <ID>/Jroot#8988

Server List:
- Gaming chat

Bound to text channels:
- Gaming chat/jukebox
- Gaming chat/testing_grounds

Autojoining voice chanels:
- Gaming chat/Testing Grounds

Options:
 Command prefix: !
 Default volume: 15%
 Skip threshold: 4 votes or 50%
 Now Playing @mentions: Disabled
 Auto-Summon: Disabled
 Auto-Playlist: Disabled
 Auto-Pause: Enabled
 Delete Messages: Enabled
   Delete Invoking: Enabled
 Debug Mode: Disabled
 Downloaded songs will be deleted

Deleting old audio cache
Attempting to autojoin Testing Grounds in Gaming chat
Attempting connection...
Connection established.

[Command] <ID>/Jroot (!play she swallowed burning coals)
[Download] Started: https://www.youtube.com/watch?v=c3UGNFUIiV4
[Download] Complete: https://www.youtube.com/watch?v=c3UGNFUIiV4
[Command] <ID>/Jroot (!np)
<later>
[Command] <ID>/Jroot (!shutdown)
Unclosed client session
client_session: <aiohttp.client.ClientSession object at 0x7fa77f592320>

It has worked fine in the past, and this issue is pretty recent, which makes me think it's an issue with a python or system dependency.  I've checked the requirements.txt file and I have everything listed installed.
If you have any ideas about this, I'd really appreciate the help.

Have same issue since this afternoon.
The bot seems to have downloaded the file and then ignores it completely. As if he doesn't want to read it.
I tried updating and reinstalling the dependencies. Nothing seems to have worked.
I'm out of ideas.

I've tried to run through all the files / code.
I've just figured that the files the bot tries to read are "empty stream" for some reason.
My guess is that youtube-dl isn't working the same way it used to. Any other leads?

I have the same problem. When I drag n drop BOT through channels some people start listening to it.
Can you guys try this to see if its the same issue?


[feature] use a users time based token to derive secret key

I'd like to use this library kind of in reverse.
Could this library be used such that the secret token is somehow derivable given a users time based token?
I know this seems odd, but it would have some really interesting properties cryptographically, especially when regenerating a key from the app.

Given the use of HMAC in both RFC 4226 and 6238, you're really talking about the reversal of a hash function based on a given token and time to derive the secret token. This isn't a feature ask, it's asking to break the foundations of H/TOTP and what makes those algorithms widely usable and trusted for 2FA.

Yeah after reading closer into the scheme and what HMAC was I realized the
tokens are just an alignment check between a shared key.
I didn’t do enough reading, apologies, we can close this.
On Mon, May 11, 2020 at 5:18 PM Matthew Sexton<EMAIL_ADDRESS>wrote:

Given the use of HMAC in both RFC 4226 and 6238, you're really talking
about the reversal of a hash function based on a given token and time to
derive the secret token. This isn't a feature ask, it's asking to break the
foundations of H/TOTP and what makes those algorithms widely usable and
trusted for 2FA.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/yeojz/otplib/issues/304#issuecomment-626970189, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ACK2CXSYJKCICN6UUTWSNBTRRBTSTANCNFSM4M3SYYQA
.


Train on apache hadoop yarn takes more time as the worker >=2 in configuration

Train on apache hadoop yarn takes more time as the worker >=2 in configuration
It seems that the train speed is slower as the worker(node) in configuration changed from 1 to 2 or more.Can anyone tell me why this happens, is there anything wrong with my configuration?
The configuration：
booster = gbtree
objective = multi:softmax
eta = 0.5
max_depth = 5
num_class = 10
num_round = 50
save_period = 0
eval_train = 1
The Shell Script:
../../dmlc-core/tracker/dmlc-submit --cluster=yarn --num-workers=4 --worker-cores=2
../../xgboost parameter.conf nthread=16
data=hdfs://hadoop01:8020/xgb-demo/train
eval[test]=hdfs://hadoop01:8020/xgb-demo/test
model_dir=hdfs://hadoop01:8020/xgb-demo/model

Hello @wallyell I'm facing the same problem here when trying to train a dataset in apache spark.
Have you found a solution for this?
Here it stops after reaching the line #156 of XGBoost
val returnVal = tracker.waitFor()`
The tracker seems to take too long for returning a value, my config params are:
"silent" -> 1,
"objective" -> "reg:linear",
"booster" -> "gbtree",
"eta" -> 0.0225,
"max_depth" -> 26,
"subsample" -> 0.63,
"colsample_btree" -> 0.63,
"min_child_weight" -> 9,
"gamma" -> 0,
"eval_metric" -> "rmse",
"tree_method" -> "auto"
Have you tried to use other boosters than gbtree?


Rule application order: Update help.html

In my experience today, it appears that the lowest/bottom-most rule is the one that's applied. Perhaps they are all applied, in-order, and the last ones can overwrite preceding rules?

Hi @JustinMaxwell,
Closing this as we will follow the topic #281.
Bests


For the backup:
export AZURE_ACCOUNT_KEY=…
export AZURE_ACCOUNT_NAME=itbackup
export RESTIC_REPOSITORY="azure:pmdb-restic:/"
export RESTIC_PASSWORD="…"

restic backup \
    --verbose \
    /mnt/snap /etc /root

For the restore:
restic restore -vvv latest --target /srv/mysql

What backend/server/service did you use to store the repository?
Azure's S3 and Google's S3
Expected behavior
The latest snapshot is restored in a timely manner.
Actual behavior
This specific respository restored extremely slowly.
Instead of getting 60 MB/s to disk it varies between 1 MB/s and 10 MB/s.
I can observe that the number of restored files grows very very slowly. Between zero and three files per second. Although the files are just a few MB in size.
Steps to reproduce the behavior
Unfortunately the only way to reproduce the issue is using a backup of a MySQL data directory that I am not allowed to disclose.
Do you have any idea what may have caused this?
I have spent countless hours trying to nail down the issue. Among those:

try different virtual machines with different amounts of CPU cores (up to 8) and RAM (up to 32 GB)
try HDD and SSD
copy over the S3 repostory from Azure S3 to Google S3 and try the restore there
measure disk performance using iostat -d 10 /dev/sdc
measure network, CPU and disk performance using nmon
monitor the network usage using iftop
check the CPU usage using htop

I tried restoring another snapshot in the same environments that was taken from a PostgreSQL data directory. That snapshot restores quickly.
After many systematic tests I am sure that it happens with this specific Restic repository.
The S3 container at Azure is located in the same data center as other repositories that do not show performance issues.
I also tried reading the snapshot without writing to disk using restic check --read-data. But I found it hard to properly see the issue here. I mainly see that Restic is writing to /root/.cache/restic with 40-50 MB/s. That's why I suspected a slow disk at first but even on an SSD the restore speed goes down to 0-2 MB/s.
Next I compared the 'stats' for two repositories:
Good and fast PostgreSQL data directory snapshot:
  Total File Count:   111612
        Total Size:   123.104 GiB

Very slow MySQL data directory snapshot:
  Total File Count:   11608
        Total Size:   300.373 GiB

(This is not related to MySQL or PostgreSQL of course. It's just for me to dinstinguish the snapshots.)
Also noticeable is that the restore performance starts well but gradually degrades. This is not a scientific measurement but a "reproducible impression". This is the network speed for downstream traffic from S3:

600 Mbps at the start
500 Mbps at 10 minutes later
400 Mbps at 15 minutes
250 Mbps at 20 minutes
100 Mbps at 30 minutes
60 Mbps at 60 minutes

This is the output of iostat -d 10 /dev/sdc (disk performance in 10-second averages):
Device             tps    kB_read/s    kB_wrtn/s    kB_read    kB_wrtn
sdc               6.67         0.19      2370.07       7865  100743840
sdc               0.00         0.00         0.00          0          0
sdc               0.20         0.00        14.80          0        148
sdc             348.55         2.40      1854.15         24      18560
sdc             148.10         0.00       800.40          0       8004
sdc               0.40         0.00        28.80          0        288
sdc              61.10         0.00     26938.80          0     269388
sdc              96.50         0.00     45624.80          0     456248
sdc             123.70         0.00     58082.00          0     580820
sdc              46.90         0.00     21996.40          0     219964
sdc              40.00         0.00     19978.40          0     199784
sdc              76.20         0.00     37994.00          0     379940
sdc               4.00         0.00       342.80          0       3428
sdc             135.46         0.00     67148.85          0     672160
sdc              36.80         0.00     18074.40          0     180744
sdc             139.10        10.40     16331.60        104     163316
sdc             133.00         0.00     47563.60          0     475636
sdc              28.70         0.00     13872.40          0     138724
sdc              50.80         0.00     23439.20          0     234392
sdc              54.50         0.00     27154.00          0     271540
sdc              49.10         0.00     23638.40          0     236384
sdc              21.20         0.00     10422.00          0     104220
sdc              29.70         0.00     14709.20          0     147092
sdc              51.90         0.00     25535.20          0     255352
sdc              15.70         0.00      7685.60          0      76856
sdc              24.30         0.00     11848.40          0     118484
sdc              45.40         0.00     22531.20          0     225312
sdc              12.40         0.00      4949.60          0      49496
sdc              25.70         0.00     12697.20          0     126972
sdc              22.00         0.00     10816.80          0     108168
sdc              13.50         0.00      6642.80          0      66428
sdc              43.10         0.00     20210.00          0     202100
sdc              15.00         0.00      6974.40          0      69744
sdc               2.60         0.00       990.21          0       9912
sdc              52.10         0.00     24994.00          0     249940
sdc               8.90         0.00      4153.20          0      41532
sdc               1.90         0.00       496.80          0       4968
sdc              53.90         0.00     25932.40          0     259324
sdc              11.40         0.00      5422.00          0      54220
sdc              31.37         0.00     15681.52          0     156972
sdc              21.90         0.00     10575.60          0     105756
sdc              38.40         0.00     18154.00          0     181540
sdc              33.00         0.00     16350.00          0     163500
sdc              28.70         0.00     13724.40          0     137244
sdc              39.20         0.00     18215.20          0     182152
sdc              19.18         0.00      9082.12          0      90912
sdc               0.40         0.00        13.20          0        132
sdc              34.20         0.00     16855.60          0     168556
sdc               8.70         0.00      3072.80          0      30728
sdc               9.00         0.00      4035.20          0      40352
sdc              16.40         0.00      7517.20          0      75172
sdc              12.70         0.00      4382.80          0      43828
sdc              14.50         0.00      6881.20          0      68812
sdc              20.90         0.00      9936.40          0      99364
sdc               4.60         0.00      1943.20          0      19432
sdc              10.40         0.00      3683.60          0      36836
sdc              18.90         0.00      8140.00          0      81400
sdc               8.29         0.40      3619.98          4      36236
sdc              11.40         0.00      4186.40          0      41864
sdc               9.10         0.00      4058.40          0      40584
sdc              14.10         0.40      6185.20          4      61852
sdc               7.20         0.00      2853.60          0      28536
sdc              27.90         0.40     11121.60          4     111216
sdc              55.40         0.80     26985.20          8     269852
sdc               0.70         0.00        96.80          0        968
sdc              23.20         0.40     11268.80          4     112688
sdc              44.80         0.80     21332.00          8     213320
sdc              20.40         0.00      9988.40          0      99884
sdc              11.50         0.40      5428.00          4      54280
sdc              79.70         1.60     38252.00         16     382520
sdc              10.50         0.00      4990.40          0      49904
sdc               8.40         0.40      3934.80          4      39348
sdc              20.60         0.00      9941.60          0      99416
sdc              64.10         1.20     30054.40         12     300544
sdc              17.50         0.00      8505.60          0      85056
sdc               2.50         0.00       722.40          0       7224
sdc              55.00         1.20     25952.00         12     259520
sdc              13.00         0.00      5307.20          0      53072
sdc               0.80         1.20        35.20         12        352
sdc               0.40         0.00        10.00          0        100
sdc              42.80         0.80     20206.80          8     202068
sdc               7.60         0.40      3092.40          4      30924
sdc               1.30         0.00       318.00          0       3180
sdc              34.60         0.40     15734.80          4     157348
sdc               6.90         0.40      3198.00          4      31980
sdc               3.40         0.00      1413.60          0      14136
sdc              37.80         0.80     18456.40          8     184564
sdc               5.10         0.00       706.00          0       7060
sdc              13.30         0.00      6031.20          0      60312
sdc              27.30         0.40     12977.60          4     129776
sdc               2.30         0.00        22.40          0        224
sdc               5.70         0.00      2522.80          0      25228
sdc              17.58         0.00      8431.17          0      84396
sdc              13.20         0.00      6356.40          0      63564
sdc               7.80         0.00      2941.20          0      29412

I think it is apparent that the disk write performance is fluctuating wildly. I have stopped now after 20 minutes which just gave me 14 GB of restored data.
This machine currently has 8 CPU cores and two different SSDs – one for the restore and one for the root file system (where /root/.cache/restic is living). It looks like the root SSD is more busy than the actualy restore SSD. Neither CPU nor network nor disks seem to be used a lot.
As restic restore does not tell me anything about the progress I just used "df -i" to count the inodes on the target partition. After 10-20 minutes into the restore the number of files barely increases:
root@db-restore-test:~# while true ; do sleep 1 ; df -i /srv/mysql/ ; done
Filesystem        Inodes  IUsed     IFree IUse% Mounted on
/dev/sdc       163840000<PHONE_NUMBER>28038    1% /srv/mysql
/dev/sdc       163840000<PHONE_NUMBER>28038    1% /srv/mysql
/dev/sdc       163840000<PHONE_NUMBER>28038    1% /srv/mysql
/dev/sdc       163840000<PHONE_NUMBER>28038    1% /srv/mysql
/dev/sdc       163840000<PHONE_NUMBER>28037    1% /srv/mysql
/dev/sdc       163840000<PHONE_NUMBER>28037    1% /srv/mysql
/dev/sdc       163840000<PHONE_NUMBER>28036    1% /srv/mysql
/dev/sdc       163840000<PHONE_NUMBER>28036    1% /srv/mysql
/dev/sdc       163840000<PHONE_NUMBER>28034    1% /srv/mysql
/dev/sdc       163840000<PHONE_NUMBER>28034    1% /srv/mysql
/dev/sdc       163840000<PHONE_NUMBER>28034    1% /srv/mysql
/dev/sdc       163840000<PHONE_NUMBER>28034    1% /srv/mysql
/dev/sdc       163840000<PHONE_NUMBER>28033    1% /srv/mysql
/dev/sdc       163840000<PHONE_NUMBER>28033    1% /srv/mysql
/dev/sdc       163840000<PHONE_NUMBER>28033    1% /srv/mysql
/dev/sdc       163840000<PHONE_NUMBER>28031    1% /srv/mysql

Next I ran strace on the restic process to get an idea what it is doing. I grepped for "open" and found that over a long period of time the same files are appended to. Example:
root@db-restore-test:~# strace -fp 7669 2>&1 | grep open | grep tar
[pid  7690] openat(AT_FDCWD, "/srv/mysql/root/mysql-server_5.7.22-1debian9_amd64.deb-bundle.tar", O_WRONLY|O_APPEND|O_CLOEXEC) = 38
[pid  7678] openat(AT_FDCWD, "/srv/mysql/root/mysql-server_5.7.22-1debian9_amd64.deb-bundle.tar", O_WRONLY|O_APPEND|O_CLOEXEC <unfinished ...>
[pid  7681] openat(AT_FDCWD, "/srv/mysql/root/mysql-server_5.7.22-1debian9_amd64.deb-bundle.tar", O_WRONLY|O_APPEND|O_CLOEXEC <unfinished ...>
[pid  7681] openat(AT_FDCWD, "/srv/mysql/root/mysql-server_5.7.22-1debian9_amd64.deb-bundle.tar", O_WRONLY|O_APPEND|O_CLOEXEC) = 38
[pid  7683] openat(AT_FDCWD, "/srv/mysql/root/mysql-server_5.7.22-1debian9_amd64.deb-bundle.tar", O_WRONLY|O_APPEND|O_CLOEXEC <unfinished ...>
[…]

Maybe I'm reading the strace output wrong. But it seems that Restic is juggling a lot of files in parallel and only writing very little to them. So I used lsof to see how many files it has open:
root@db-restore-test:~# lsof | grep /srv/mysql | wc -l
768

I know, this is a lot of data to digest. I tried to make my steps as clear as possible. If you want me to try anything else just let me know. Thanks in advance.
Do you have an idea how to solve the issue?
No.
Did restic help you today? Did it make you happy in any way?
Totally. Two friends and I are using Restic on a daily basis to backup our cheap personal VMs to Wasabi S3 and it never let us down. We love Restic.

Can you try latest master? There is a known restore perf bug in 0.9.6 (originally reported as #2074 and few other places and fixed by #2195)

Brilliant. Why didn't I think of that? I felt so smart doing all those measurements. :)
I just took the binary from https://beta.restic.net/restic-v0.9.6-377-gc34c7316/ which apparently includes the #2195 merge.
Performance is now way up. Disk throughput went up to an average of 90 MB/s. So consider this another day that the merge has rescued. Thanks for your time.


Builder fails with "E: Version '1.20190718-1' for 'raspberrypi-kernel' was not found"

When attempting to build the image (and when the CircleCI build kicks off from this repo), the error "E: Version '1.20190718-1' for 'raspberrypi-kernel' was not found" causes the build to fail.
Is there updated documentation available somewhere to help with the new process using the Raspbian original kernel?
Thanks!

That's normally a sign that the Raspberry Pi Foundation has released another kernel and we have to update that version string. See https://github.com/raspberrypi/linux/releases for current release tag names.

Fixed with #314

Hello,
I am having the same issue with 1.20190819-1

@samouds looks like they just released  1.20190925-2
I tried that in my PR today morning, and had to roll back to the previous.
The version pinning is good, but not for this package, they only provide one version of the kernel 🤔

I just tried the 1.20190925-2 and it worked

1.20190925+1-1


Conditional Router does not stop correctly

Bug Description
Conditional Router does not stop correctly.  The input message display twice, followed by a error message: "Error parsing message".
It's not reproduced every time, try more times.

Reproduction

import the flow file
input hello in the playground
it's not reproduced every time, try more times.

Expected behavior
return hi, there I'm a robot directly and no error message
Who can help?
No response
Operating System
manjaro
Langflow Version
v1.0.19.post2
Python Version
None
Screenshot

Flow File
Simple Conditional Router.json

Hi, I have the same question.
It happened in both of version 1.0.17 and 1.1.0
With the workflow:
[python 3.12.3, langflow 1.0.17]
20241114_debug_conditional_router.json
20241114_debug_conditional_router_v2.json
Test Case
1.input=A, expected_output=A, real_output=A None None

2.input=B, expected_output=B, real_output=None B None

in this situation will cause 2 user messages to be generated

[python 3.12.3, langflow 1.1.0]
Still not work in this version:
20241114_debug_conditional_router_v2_使用langflow1.1.0版本.json

@joaoguilhermeS - I have given the details.
https://github.com/langflow-ai/langflow/issues/4590#issuecomment-2475895886

Please follow #https://github.com/langflow-ai/langflow/issues/4590


Subscription - Subscription settings and parts of details are missing

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 1.4.82-0
Reproducible in staging?: Y
Reproducible in production?: Y
Found when validating PR : https://github.com/Expensify/App/pull/43128
Logs: https://stackoverflow.com/c/expensify/questions/4856
Issue reported by: Applause-Internal team
Action Performed:

Navigate to https://staging.new.expensify.com/
Log in with a Gmail account
Create a workspace
Navigate to https://staging.new.expensify.com/settings/subscription
Swipe up

Expected Result:
The missing parts should be available.
Actual Result:
Subscription settings is missing completely. Pay-per-use and Subscription size is missing from Subscription details on iOS and Android apps.
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android: Native
[ ] Android: mWeb Chrome
[x] iOS: Native
[ ] iOS: mWeb Safari
[ ] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
https://github.com/Expensify/App/assets/115492554/a8e9132d-4928-4d53-8352-a356652e41a0

View all open jobs on GitHub

We think this issue might be related to the #collect project.

Heya @amyevans and @dannymcclain - sorry to assign you, but have we officially launched subscriptions on iOS and Android yet? If not, we could probably close this right?
If so, I guess we have a bug?

Yo @jliexpensify! We have not officially launched these yet—they are still in implementation.
Also, because of iOS and Android pricing/app rules, we do not show those sections or allow users to edit any information on the subscription page in native iOS and Android apps. So this is actually unreleased, expected behavior. Good to close! 🙂

Nice, thanks!


correct staticcheck issues

PR contains changes to fix the code issues flagged by staticcheck:
$ staticcheck ./...

client.go:273:2: should use 'return <expr>' instead of 'if <expr> { return <bool> }; return <bool>' (S1008)
client.go:501:2: should use a simple channel send/receive instead of select with a single case (S1000)
client.go:529:2: should use a simple channel send/receive instead of select with a single case (S1000)
client_test.go:502:2: this value of session is never used (SA4006)
example_test.go:119:20: session.New is deprecated: Use NewSession functions to create sessions instead. NewSession has the same functionality as New except an error can be returned when the func is called instead of waiting to receive an error until a request is made.  (SA1019)
example_test.go:139:24: *&x will be simplified to x. It will not copy x. (SA4001)
vssh.go:51:2: field ctx is unused (U1000)
vssh.go:60:2: field errors is unused (U1000)
vssh.go:62:2: field clients is unused (U1000)
vssh.go:369:19: using time.Tick leaks the underlying ticker, consider using it only in endless functions, tests and the main package, and use time.NewTicker here (SA1015)

I confirm that this contribution is made under the terms of the license found in the root directory of this repository's source tree and that I have the authority necessary to make this contribution on behalf of its copyright owner.

@dmitris Thanks Dmitry!


Add check for MSAL library when attempting ActiveDirectoryServicePrincipal authentication

Minor change. This check exists for other AD auth cases, but will just throw java.lang.ClassNotFoundException if ActiveDirectoryServicePrincipal is used and MSAL libraries are not in the classpath. In another project, I had spent some time trying to determine the issue, only to notice this missing check would have been helpful.
This just adds the check used in other AD auth cases to the ActiveDirectoryServicePrincipal case.

Caused by: java.lang.NoClassDefFoundError: com/microsoft/aad/msal4j/IClientCredential
at com.microsoft.sqlserver.jdbc.SQLServerConnection.getFedAuthToken(SQLServerConnection.java:5341)
at com.microsoft.sqlserver.jdbc.SQLServerConnection.onFedAuthInfo(SQLServerConnection.java:5293)
at com.microsoft.sqlserver.jdbc.SQLServerConnection.processFedAuthInfo(SQLServerConnection.java:5180)
at com.microsoft.sqlserver.jdbc.TDSTokenHandler.onFedAuthInfo(tdsparser.java:305)
at com.microsoft.sqlserver.jdbc.TDSParser.parse(tdsparser.java:128)

now returns:

Caused by: com.microsoft.sqlserver.jdbc.SQLServerException: Failed to load MSAL4J Java library for performing ActiveDirectoryServicePrincipal authentication.
at com.microsoft.sqlserver.jdbc.SQLServerConnection.getFedAuthToken(SQLServerConnection.java:5333)
at com.microsoft.sqlserver.jdbc.SQLServerConnection.onFedAuthInfo(SQLServerConnection.java:5290)
at com.microsoft.sqlserver.jdbc.SQLServerConnection.processFedAuthInfo(SQLServerConnection.java:5177)
at com.microsoft.sqlserver.jdbc.TDSTokenHandler.onFedAuthInfo(tdsparser.java:305)
at com.microsoft.sqlserver.jdbc.TDSParser.parse(tdsparser.java:128)

Thanks

Thank you for the contribution @caseyhall. We'll take a look and get back to you.


RefreshIndicator and absolute values

IMO, it doesn't make sense to accept ONLY absolute values.
I'm trying to create my own loading component like this:
<RefreshIndicator
      style={{display: 'flex', top: 'initial', left: 'initial'}}
      size={35}
      status="loading"
    />

This works great, but I get this warnings:
Warning: Failed propType: Required prop leftwas not specified inRefreshIndicator.
Warning: Failed propType: Required prop topwas not specified inRefreshIndicator.
Can we make them at least optional so the dev can choose between absolute values and the default behavior?

:+1: Had the same issue with the component a while back. I used circular progress instead. I would suggest you do the same since this component can be confused with the chromes own refresh indicator on android devices.

Sorry, what you used?

Oh, nevermind. Found it: http://material-ui.com/#/components/progress

By passing innerStyle={{ margin: 0 }} you can easily style the components with the animations you desire. you can take a look at my implementation.

I'm also having this issue. Accepting only absolute position is not very flexible. IMHO this component shouldn't care about its positioning at all as we can use it in a lot of contexts.

Agree that absolute position makes it very difficult to use.  I have comments and I'd like to display the indicator next to the load more button which moves depending on the window scroll.

I came up with a work-around.  I place the RefreshIndicator in a div and set the position to relative.  The width is set to the width of the RefreshIndicator.  Now the absolute position of the RefreshIndicator will equal the position of the div.  I like to make the div a child of a parent div whose display is set to flex.  Then I can leverage flexbox.

@rdagger It did the trick for me too, thanks !

    <RefreshIndicator
      size={100}
      left={-50}
      top={-50}
      status="loading"
    />

If the above solutions were not able to solve the problem feel free to comment below. I am closing this issue for now.

the solution above fixes it but is still ugly. I would prefer to opt-in position:absolute if I need it rather than current opt-out workaround.


New Field: Date Price Fixing

We need a field which transfer us the date, when the pricing for the offer has been done:
Name: "datePriceFixing" or "offerValidFrom"
description: "The date when the price has been fixed" OR "The date when the offer has been made"
Type: see Date
Can we add this field after the object "interest"?

Agreement MP / TKB on 29.09.22:
Renaming to "offerDate" and Replacement of the field to "Mortgage Offer".
Ok


feat: add samples for http cors

Add Functions samples and tests for HTTP CORS requests. This is based on the Python version here: https://github.com/GoogleCloudPlatform/python-docs-samples/blob/1523c225344bd7c38ab3b54e7bccf8ef9e7a3184/functions/http/main.py#L127-L152

Looks like the failed tests are unrelated to the change?

Yeah the failure is unrelated and will be addressed by #617.


Can this method be used for multi-label temporal localization?

Thanks for your great work!
The datasets that used here are all single-label datasets where there is just one action per time-step?
Can this model be used on multi-label datasets (i. e., MultiTHUMOS, Charades) by replacing loss functions or just replacing nms with multi-class nms.

Hi, since our method are not designed for multi-label datasets, you may need a little bit tweak to our current models.
Current cls head uses Focal loss, which already supports multi-label groundtruths. However, current regression head is class-agnostic, , you may need to design a class-aware regression head and modify the code.

Thank you very much, is there noting to do with NMS? if some modifications are needed during inference?

Hi, I think our current code already supports multi-class NMS. However, our code can not handle the case that multiple actions with same labels occurs at same time.

Closed due to inactivity.


Fixed typo in service-dataflows-incremental-refresh.md

Removed a typo by changing "it's is" to "it is".

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


syntax/json: Add comment support to the syntax highlighting

This feature already has a quite long story:

https://github.com/zyedidia/micro/issues/266
https://github.com/zyedidia/micro/issues/2701
https://github.com/zyedidia/micro/issues/2704
https://github.com/zyedidia/micro/pull/2736
https://github.com/zyedidia/micro/issues/2995

Since micro already uses json5 to parse his own internal json configurations and we changed the default behavior of the comment plugin to set // as the default comment type within json via https://github.com/zyedidia/micro/pull/3388 it's just a logical step to highlight it correctly in json's (reused https://github.com/zyedidia/micro/pull/2736). It was and it will be up to the user if he/she places comments with micro in json, because he/she should know if the file is parsed with a json5 parser or not.
Especially since https://github.com/zyedidia/micro/pull/3343 it's much more easy to edit micro's own configurations in micro itself by just reloading the changed files. As long as set, reset, bind and unbind (hopefully I didn't forget anything) isn't used micro doesn't rewrite the settings by his own and the comments are kept.
Additionally we can think about adding hjson (https://github.com/zyedidia/micro/pull/2203) and jsonc (https://github.com/zyedidia/micro/issues/2704) to the same json syntax definition, instead of creating new file types. The latter one is arguable, since via syntax includes it's also easy to extend/override definitions.

Adding hjson to json seems to go too far, since # (supported as comment in hjson) is not allowed by a json5 parser. So #2203 is still valid, but can be reduced afterwards.
Most probably it's the same for jsonc, which allows unquoted values.
For now we keep the PR as is and maybe consider the rest afterwards.


exclude storm's external libraries from storm-mesos release tarball

@JessicaLHartog : please review.

LGTM :shipit:


Feat: Submitting answer

It would be two commands:
:AocSubmitAnswer {day} {year} {answer}
and
:AocSubmitTodayAnswer {answer}
Note: user gets rate limited if too many requests are submitted.

I don't think I will go ahead implementing this feature.
After experimenting with this feature, I find that it is not quite the same "Advent of Code" experience as going to the website and submit the answer there.
Happy to consider it in the future if anyone else may find it useful. I, unfortunately, don't feel that way at the moment.


User request: copy PGN from finished puzzles

Copying the PGN directly from the move list is possible, but seems a little sketchy since it might be used to cheat the puzzles easier. However the request was for finished puzzles. When a puzzle ends, a link to the game it came from is displayed, which leads to the game where the PGN can be extracted from.

If there are other kinds of puzzles that don't conform to this pattern, ask to reopen this.


Zingo sync mempool

These changes are incorporated in #1579 .

Why is there not OutgoingTransparentCoin?

Why is there not OutgoingTransparentCoin?

OutgoingSaplingNote and OutgoingOrchardNote come from decrypting the outgoing notes in the transaction but there is no such transparent equivalent.

These changes are incorporated in #1579 .

this PR should be reviewed and merged separately to the removal of getset to keep this PR concise. also there is still ongoing discussion about getset removal


Restore the ZstdContinuousInputStream

It was left behind from the 1.0.0 release. We need it in order to be able to consume files that are not yet finished.

Done. patched into 1.0.0 release. It's now enabled with setConituous(true) on the ZstdInputStream object.


datetime from azure cli token is in the local timezone

While working on auto refresh I noticed that the token expiry from azure cli is in the local timezone but is interpreted as UTC in code. This causes the token not being refreshed in my timezone. (My time: 12:00 GMT+2, toke expiresOn time 13:20, token expiry in UTC 15:20).
This can be verified by running az account get-access-token --output json and comparing exp of the jwt with expiresOn from the cli.

In my testing, I saw a similar difference with expiresOn as indicated by @yvespp .  I went to check how the CLI implements this to make sure we're not relying on a local configuration regarding time zones.   That doesn't appear to be the case.
However, in my investigation, I came across the following from the SDK
        # Convert epoch int 'expires_on' to datetime string 'expiresOn' for backward compatibility
        # WARNING: expiresOn is deprecated and will be removed in future release.
        import datetime
        expiresOn = datetime.datetime.fromtimestamp(sdk_token.expires_on).strftime("%Y-%m-%d %H:%M:%S.%f")

        token_entry = {
            'accessToken': sdk_token.token,
            'expires_on': sdk_token.expires_on,  # epoch int, like<PHONE_NUMBER>
            'expiresOn': expiresOn  # datetime string, like "2020-11-12 13:50:47.114324"
        }

In the future, we may want to consider parsing the JWT token itself, which includes more details as to the expiry state (including a not-before and expires-on timestamp).


Provide an option to control the order of locators(elements) to be returned when more than one condition is met while using OR operator in the PageYAML

Selion always picks and returns only the last element locator when two locators in OR conditions are matched and available in the Page. Need an option to Pick any matching locators based on the request from the customer.
US: "//android.widget.TextView[1]| //android.widget.TextView[2] | //android.widget.TextView[3]"
In the above case when TextView[2] and TextView[3] are available in the page, Selion always find and return only android.widget.TextView[3].

I assume the API call you are making is getLocator on webelements that extend AbstractElement such as Link, Button, etc.
Two proposals:

public String getLocator()
Will return the full path in yaml, so in your example it will return
//android.widget.TextView[1]| //android.widget.TextView[2] | //android.widget.TextView[3]

Leave getLocator's behavior as is and create a new method, public String getLocators, which will return
//android.widget.TextView[1]| //android.widget.TextView[2] | //android.widget.TextView[3]

The only thing bad about number 1 is that we could break potential users who expect the behavior of this operation to return a true xpath.  I can't remember if the | symbol is selion specific delimiter or selenium will take an xpath with | symbols in it.  If its the former, then we will break users who are using getLocator and taking that string directly to use with APIs that take a true XPATH.


Investigate sscanf difference between compilers when parsing doubles that are too large.

1.8e309 is too large to be represented by a double. So, this is considered equal to inf. The parsing APIs rely on sscanf to do the heavy lifting for double parsing. However, for strings that are out-of-range, we have a difference in behavior of sscanf based on the compiler, where in some cases errno is set, while in others, it isn't.
Here's a sample program that illustrates the problem:
#include  <stdio.h>
#include  <stdint.h>
#include  <errno.h>

int main(void)
{
    double num = 0;
    int32_t n = 0;
    errno = 0;
    int ret = sscanf("1.8e309", "%20lf%n", &num, &n);
    printf("num=%lf, %d, %d, %d\n", num, n, ret, errno);

   // On MSVC:
   // num=inf, 7, 1, 0

  // On GCC, Clang:
  //  num=inf, 7, 1, 34
}

MSVC: https://rextester.com/SCYSI40711
Clang: https://rextester.com/UMMQ44933
Because of this, the following test fails on GCC/Clang (see https://github.com/Azure/azure-sdk-for-c/pull/885 for more info):
// fails, instead returns AZ_ERROR_PARSER_UNEXPECTED_CHAR
assert_int_equal(az_span_atod(AZ_SPAN_FROM_STR("1.8e309"), &value), AZ_OK);
assert_true(value == INFINITY);

cc @barcharcraz, @BillyONeal

The only functions documented as touching errno in the input/output section of the C standard are:

fgetpos
fsetpos
ftell
perror (except accepts an errno value as input)

Notably fscanf and scanf are not documented as affecting the value of errno, so you can't really assume anything about its value after such a call. fscanf/scanf might call strtod under the covers (but are not required to), which says C11 <IP_ADDRESS> "The strtod, strtof, and strtold functions"/10: (emphasis mine)

The functions return the converted value, if any. If no conversion could be performed,
zero is returned. If the correct value overflows and default rounding is in effect (7.12.1),
plus or minus HUGE_VAL, HUGE_VALF, or HUGE_VALL is returned (according to the
return type and sign of the value), and the value of the macro ERANGE is stored in
errno. If the result underflows (7.12.1), the functions return a value whose magnitude is
no greater than the smallest normalized positive number in the return type; whether
errno acquires the value ERANGE is implementation-defined.

Note also that POSIX prohibits scanf/fscanf from detecting this condition and setting ERANGE. See "ERRORS" in https://pubs.opengroup.org/onlinepubs/9699919799/functions/fscanf.html
Note also C11 7.22.1 "Numeric conversion functions"/1: (emphasis mine)

The functions atof, atoi, atol, and atoll need not affect the value of the integer
expression errno on an error. If the value of the result cannot be represented, the
behavior is undefined.

which suggests that if you want to provide reasonable overflow behavior here az_span_atod should be called az_span_strtod.

The following link seems to suggest checking for errno != 0 after calling sscanf, which does end up capturing some of the underflow, overflow cases while parsing.
https://linux.die.net/man/3/sscanf

Note also that POSIX prohibits scanf/fscanf from detecting this condition and setting ERANGE.

Where is that stated? I am having trouble finding it.

The only functions documented as touching errno in the input/output section of the C standard are

The "RETURN VALUE" section from the link you shared seems to state otherwise. Do you have somewhere I can read up on why I can't rely on errno when calling sscanf?

Upon successful completion, these functions shall return the number of successfully matched and assigned input items; this number can be zero in the event of an early matching failure. If the input ends before the first conversion (if any) has completed, and without a matching failure having occurred, EOF shall be returned. If an error occurs before the first conversion (if any) has completed, and without a matching failure having occurred, EOF shall be returned [CX] [Option Start]  and errno shall be set to indicate the error.

Regardless, if I can't rely on errno to be set when using sscanf for parsing doubles that return in inf for too large numbers, or 0 for too small numbers (such as 1e-400), do you have a recommendation on how I can detect those cases and deterministically fail, regardless of what compiler is being used?
Here's the current check for successful parsing (inspired by @JeffreyRichter's suggestion to use sscanf https://github.com/Azure/azure-sdk-for-c/issues/598#issuecomment-633277659):
https://github.com/Azure/azure-sdk-for-c/blob/c9396dc3ab682a3c55c5a31c9e1070a1d28a3bd1/sdk/src/azure/core/az_span.c#L156-L161
We can't use strtod because of the zero-termination requirement (which results in data copies). We had tried that approach and decided against it:
https://github.com/Azure/azure-sdk-for-c/pull/627

which suggests that if you want to provide reasonable overflow behavior here az_span_atod should be called az_span_strtod.

If we were providing an API that is trying to follow suite with the C runtime, I can see why we'd name it az_span_strtod, but overloading that could cause confusion with spans and strings (null termination vs not). Naming aside, the pressing matter is finding a way to provide reasonable overflow behavior in the first place :)

The following link seems to suggest checking for errno != 0 after calling sscanf, which does end up capturing some of the underflow, overflow cases while parsing.

That's not what the link says:

The value EOF is returned if the end of input is reached before either the first successful conversion or a matching failure occurs. EOF is also returned if a read error occurs, in which case the error indicator for the stream (see ferror(3)) is set, and errno is set indicate the error.

That is, I/O errors only, not ERANGE.

Where is that stated? I am having trouble finding it.

ERANGE is not one of the enumerated valid error values. All of the enumerated error values are I/O errors.

The "RETURN VALUE" section from the link you shared seems to state otherwise. Do you have somewhere I can read up on why I can't rely on errno when calling sscanf?

It isn't that you can't rely on errno when calling sscanf, it's that sscanf does not consider this condition an error. For special values like inf, even strtod does not consider parsing that an error.

Regardless, if I can't rely on errno to be set when using sscanf for parsing doubles that return in inf for too large numbers, or 0 for too small numbers (such as 1e-400), do you have a recommendation on how I can detect those cases and deterministically fail, regardless of what compiler is being used?

I believe strtod (and friends) are the only portable option.

For special values like inf, even strtod does not consider parsing that an error.

There is a difference between values that are semantically double inf, and actually/precisely inf. strtod does set errno for semantically infinity values.
Should we just reject any input that results in a non-finite double value when parsing? For example, fail when isfinite(result) == false?
// Same output on all 3 compilers: clang, msvc, gcc
#include <stdio.h>      /* printf, NULL */
#include <stdlib.h>     /* strtod */
#include <errno.h>

int main(void)
{
  errno = 0;
  char semantic_inf[] = "1e400";
  double d1 = strtod (semantic_inf, NULL);
  printf ("vaule: %lf, errno:%d\n", d1, errno); // vaule: inf, errno:34

  errno = 0;
  char actual_inf[] = "inf";
  double d2 = strtod (actual_inf, NULL);
  printf ("vaule: %lf, errno:%d\n", d2, errno); // vaule: inf, errno:0

  return 0;
}


Unable to Print Forwarding Addresses

The issue tracker is for reporting product deficiencies. "How do I?" questions should be posted to the discussion forum at https://groups.google.com/group/google-apps-manager. When in doubt, start at the discussion forum and return here only when instructed to do so.
Please confirm the following:

I have upgraded to the latest GAM release from https://github.com/GAM-team/GAM/releases and I still have this issue.
I am typing the command as described in the GAM Wiki at https://github.com/GAM-team/GAM/wiki

Full steps to reproduce the issue:

Execute gam all users print forwardingaddresses > (file path) in command prompt
Wait
Eventual error

Expected outcome (what are you trying to do?):
Trying to execute an export of addresses that have forwarding enabled  I'm running it on a Windows 10 environment using GAM 6.50, and just trying to export to a local repository on my computer using  gam all users print forwardingaddresses > (file path).
Actual outcome (what errors or bad behavior do you see instead?):
After a very long wait, we get the error ERROR: Authentication Token Error - invalid_grant: Invalid JWT: Token must be a short-lived token (60 Minutes) and in a reasonable timeframe.  Check your iat and exp values and use a clock with skew to account for click differences between systems. This outputs a blank csv file in the chosen location.  I have checked some forums on this, and they all suggest my clock is out of sync.  however, it is completely synced up with our domain upon checking.
To note:  I tried to do a show instead of a print for address forwarding to see if it would work in general, but due to the fact that we have over 58k addresses being pulled, it takes hours to process and doesn't fit all in one command prompt screen.  The farthest It's gotten through a show all is roughly 22k out of the 58k.  It fails out with Error 11001 Getaddrinfo failed and no additional elaboration.

The issue tracker is for reporting product deficiencies. "How do I?" questions should be posted to the discussion forum at https://groups.google.com/group/google-apps-manager. When in doubt, start at the discussion forum and return here only when instructed to do so.
Please confirm the following:

I have upgraded to the latest GAM release from https://github.com/GAM-team/GAM/releases and I still have this issue.
I am typing the command as described in the GAM Wiki at https://github.com/GAM-team/GAM/wiki

Full steps to reproduce the issue:

Execute gam all users print forwardingaddresses > (file path) in command prompt
Wait
Eventual error

Expected outcome (what are you trying to do?):
Trying to execute an export of addresses that have forwarding enabled  I'm running it on a Windows 10 environment using GAM 6.50, and just trying to export to a local repository on my computer using  gam all users print forwardingaddresses > (file path).
Actual outcome (what errors or bad behavior do you see instead?):
After a very long wait, we get the error ERROR: Authentication Token Error - invalid_grant: Invalid JWT: Token must be a short-lived token (60 Minutes) and in a reasonable timeframe.  Check your iat and exp values and use a clock with skew to account for click differences between systems. This outputs a blank csv file in the chosen location.  I have checked some forums on this, and they all suggest my clock is out of sync.  however, it is completely synced up with our domain upon checking.
To note:  I tried to do a show instead of a print for address forwarding to see if it would work in general, but due to the fact that we have over 58k addresses being pulled, it takes hours to process and doesn't fit all in one command prompt screen.  The farthest It's gotten through a show all is roughly 22k out of the 58k.  It fails out with Error 11001 Getaddrinfo failed and no additional elaboration.

closing this as it sounds like a specific configuration issue. If you are still seeing this issue though please start a discussion in git.io/gam-group or git.io/gam-chat.


Bug: Renaming a web shortcut crashes app

Description
I had a URL shortcut which I tried to rename. I deleted the existing name to nothing and then tried adding a name later and found out that renaming web shortcuts crashes the app.

A web shortcut

Delete existing name of shortcut

Now try renaming this shortcut

Steps To Reproduce

Create a shortcut using windows explorer (for any URL)
Open files and go to the directory containing the shortcut file
Try renaming the shortcut
Delete the existing name
Now you have a shortcut with no name '.url'
Now try renaming the shortcut again
Files app should crash

Steps to fix

Fix crash that occurs when renaming shortcuts with no name.
Or disable creation of shortcuts with no name in the first place

Files Version
Version: <IP_ADDRESS> OS Version: 10.0.19044.2075
Windows Version
Edition	Windows 10 Home Single Language Version	21H2 Installed on	31-03-2021 OS build	19044.2075 Experience	Windows Feature Experience Pack 120.2212.4180.0
Relevant Assets/Logs
debug_fulltrust.log
debug.log

Asking @yaichenbaum for approval.

I think we need to do some more investigation to see what the actual issue is.

This is what the log says.
	System.ArgumentException: String cannot be of zero length. (Parameter 'oldValue')
   at System.String.Replace(String oldValue, String newValue)
   at System.String.Replace(String oldValue, String newValue, StringComparison comparisonType)
   at Files.App.Helpers.UIFilesystemHelpers.RenameFileItemAsync(ListedItem item, String newName, IShellPage associatedInstance)
   at Files.App.Views.LayoutModes.DetailsLayoutBrowser.CommitRename(TextBox textBox)
   at System.Threading.Tasks.Task.<>c.<ThrowAsync>b__128_0(Object state)
   at Microsoft.UI.Dispatching.DispatcherQueueSynchronizationContext.<>c__DisplayClass2_0.<Post>b__0()
--- End of stack trace from previous location ---
   at WinRT.ExceptionHelpers.<ThrowExceptionForHR>g__Throw|20_0(Int32 hr)
   at ABI.Windows.ApplicationModel.Core.IUnhandledErrorMethods.Propagate(IObjectReference _obj)
   at Windows.ApplicationModel.Core.UnhandledError.Propagate()
   at Microsoft.AppCenter.Utils.ApplicationLifecycleHelperWinUI.<.ctor>b__0_3(Object sender, UnhandledErrorDetectedEventArgs eventArgs)
--- End of stack trace from previous location ---
   at Microsoft.AppCenter.Utils.ApplicationLifecycleHelperWinUI.<.ctor>b__0_3(Object sender, UnhandledErrorDetectedEventArgs eventArgs)
   at WinRT.EventSource__EventHandler`1.EventState.<GetEventInvoke>b__1_0(Object obj, T e)
   at ABI.System.EventHandler`1.Do_Abi_Invoke[TAbi](Void* thisPtr, IntPtr sender, TAbi args)```


lightning(dm): add config value to use local backend

What problem does this PR solve?

Issue Number: ref #3510
What is changed and how it works?
Check List
Tests

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

Questions
Will it cause performance regression or break compatibility?
Do you need to update user documentation, design documentation or monitoring documentation?
Release note
Please refer to [Release Notes Language Style Guide](https://pingcap.github.io/tidb-dev-guide/contribute-to-tidb/release-notes-style-guide.html) to write a quality release note.

If you don't think this PR needs a release note then fill it with `None`.

/run-dm-integration-test

/check-issue-triage-complete

/run-dm-integration-test

/run-dm-integration-test

/run-dm-integration-test

ptal

/cc @GMHDBJD @buchuitoudegou

ptal @GMHDBJD @buchuitoudegou

/merge

/run-dm-integration-test


[MNT] Updated diagnosis and assessment options

Closes #262

Checklist

[ ] PR has an interpretable title with a prefix ([ENH], [FIX], [REF], [TST], [CI], [MNT], [INF], [MODEL], [DOC]) (see https://neurobagel.org/contributing/pull_requests for more info)
[ ] PR links to GitHub issue with mention Closes #XXXX
[ ] Tests pass
[ ] Checks pass

For new features:

[ ] Tests have been added

For bug fixes:

[ ] There is at least one test that would fail under the original bug conditions.

As @rmanaem just noted in the tracking issue: we'll merge this PR with the terms that we currently have in our own nodes until https://github.com/neurobagel/federation-api/issues/12 will soon make the hard-coding of the terms unnecessary.


Debug GPU attach to vm failing

Users report they can't attach GPUs to their VMs

Fixed in https://github.com/kthcloud/go-deploy/pull/423


push has to validate the appname for characters not allowed by kubernetes.

Just tried to cd sample-app ; carrier push "x$(date +%Y%m%d_%H%M%S)" as a means of generating a dynamic app name so that I do not have to count myself when repeatedly pushing while working on the user feedback stuff.
Did not start ... Found the following error in the el-listener log:
PipelineRun.tekton.dev \"staging-pipeline-run-x20210208_144322-t8g67\" is invalid: metadata.name:
Invalid value: \"staging-pipeline-run-x20210208_144322-t8g67\": a DNS-1123 subdomain must consist
of lower case alphanumeric characters, '-' or '.', and must start and end with an alphanumeric character
(e.g. 'example.com', regex used for validation is
'[a-z0-9]([-a-z0-9]*[a-z0-9])?(\\.[a-z0-9]([-a-z0-9]*[a-z0-9])?)*')"

The issue is the name, i.e. x20210208_144322. The _ is not allowed. Kubernetes rules about naming are leaking through, and we are not checking about any of these.
Found that . is not allowed either, in some different place:
admission webhook \"validation.webhook.pipeline.tekton.dev\" denied the request: validation failed:
Invalid resource name: special character . must not be present: metadata.name"

I would say, we should start with:

uniqueness (in the org scope)
allowed characters
name size ( < 63 ? check k8s docs)


⚠ Bofrost.de has degraded performance

In 3362d42, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1687 ms

Resolved: Bofrost.de performance has improved in d13ad38 after 20 minutes.