Question on calculation of annotations and each output parameter (is there any documentation?)

Hi,
I'm new to the toolbox and enjoy it quite a bit. Thanks for providing such a great tool. Yesterday I was writing some batch code and asked myself, how e.g. all the HRV functions work. Now writing this, I realized that I was looking up the wrong matlab function (HRVTool.m instead of HRV.m). However, saying that:
is there any documentation (e.g. a wiki) for quick access to all possible functions and their input parameters? I guess that would be nice for newcomers if its non-existent (but maybe, I just did not see it yet).
And another related question. When I calculate the annotations for a given ecg trace, it detects the heart beat quite nicely (I'm using the provided standard batch code). However, the timestamps are a little off (roughly by 10*dt with dt = 1/sample rate); see image. And even if I shift the time points, they do not match the peaks precisely. Is this a wanted feature or am I doing something incorrectly?
Thanks a lot for the help!
Best, Felix
Here is the code:
    % settings for automated beat detection
    load('qrs_settings.mat');
    s = 1;    % s=1 for human ECG settings
    Fs = 1024; % set your sampling frequency
    Beat_min = qrs_settings.Beat_min(s);
    Beat_max = qrs_settings.Beat_max(s);
    wl_tma = ceil(qrs_settings.wl_tma(s)*Fs);
    wl_we  = ceil(qrs_settings.wl_we(s,:).*Fs);
    d_fs = Fs;

    % Import waveform (ECG) - use an appropriate import function
    sig_waveform = myECG';   % --> just a vector with ecg trace
    Ann = [];

    % Heart beat detection
    seg = ceil(length(sig_waveform)/(300*Fs));
    if seg>2
        for i=0:seg
            sig_waveform_tmp = sig_waveform(max(300*Fs*i-10*Fs,1):min(300*Fs*(i+1),length(sig_waveform)));
            if sum(isnan(sig_waveform_tmp)) ~= length(sig_waveform_tmp)
                Ann_tmp = singleqrs(sig_waveform_tmp,Fs,'downsampling',d_fs,'Beat_min',Beat_min,'Beat_max',Beat_max,'wl_tma',wl_tma,'wl_we',wl_we);
                Ann = [Ann; Ann_tmp+max(300*Fs*i-10*Fs,1)];
            end
        end
        Ann = Ann(Ann>0 & Ann<=length(sig_waveform));
        Ann = unique(sort(Ann));
        Ann(diff(Ann)<.05*Fs)=[];
    else
        Ann = singleqrs(sig_waveform,Fs,'downsampling',d_fs,'Beat_min',Beat_min,'Beat_max',Beat_max,'wl_tma',wl_tma,'wl_we',wl_we);
    end
    Ann = Ann/Fs;

    %%
    dt = 1/1024;
    myt = 0:dt:(length(myECG)-1)*dt;
    figure
    plot(myt,myECG); hold on;
    plot([Ann Ann]',repmat([-2 2],length(Ann),1)','k');

Dear Felix,
the functions in HRV.m are commented with some explanation of input and output parameters, a short description along with references and example code. An external documentation does not exists.
Concerning the shift in the annotations: The beat detection algorithm used detects the beats at the strong slope of the R peak. This is detected by a filter method described in Robust Detection of Heart Beats using
Dynamic Thresholds and Moving Windows. The threshold-based annotations (which are integer locations/ frame numbers) are located between Q and R. As the procedure is done for all the beats, the intervals should not vary that much from RR intervals. Nevertheless, I would recommend to use this as a starting annotation and would include a refinement step in the analysis to find the R peak accurately. I would use a spline interpolation on the bandpass-filtered signal and recommend to use the extracted splines to define precise and sampling rate free locations of R peaks.


bug(general): Installing civitdl globally using make install fails (UNKNOWN-0.0.0)

What happened?
Trying to run make install (or pip install .) globally fails due to pip not installing the project correctly.
Steps to reproduce the behavior
cd civitdl
make install

Actual Behavior
Silently fails installation with UNKNOWN-0.0.0 installed.
Expected Behavior
Successfully installs civitdl-vX.Y.Z
System Detail

OS: Ubuntu 22.04.3
Python Version: 3.10.12

Additional Context

Current temporary solution for global install is to run make install2 instead while trying to figure out (note make install2 is not available in development yet, will be added soon in one of the feature branches):
install: uninstall
pip3 install .
install2: uninstall
python3 -m build
pip3 install -r ./requirements.txt
pip3 install --upgrade dist/*.whl


Localization HambergerMenu at run Time

Hi Jerry,
I have localized all the app and can change language at run time except shell.xaml. so hamburger menu is not changed at run time.
Is this a bug?  or do you have a solution?
Thanks

Are you localizing with x:UID?

Yes, I do with x:UID, and all pages is localized at run time except shell.  so shell needs to restart for resolving this problem.

http://stackoverflow.com/questions/32715690/c-sharp-change-app-language-programmatically-uwp-realtime
I will assume you are doing it wrong. If not, open a new issue.

I did all the work Jerry as your reference link, but I did not get result, I think, it's a good idea if you create a sample with minimum 2 pages as multilingual support of hamburger menu.
Thanks

Any news about this?
Is there an example to multilingual support of hamburger menu?


Remember hydrometer calibration temperature

My hydrometer is calibrated at 60F instead of 60f. It would be nice if Biermacht remembered the last calibration temperature.

I have a patch ready to go but it it against my tree with the notes fix applied. I can push it after that pull request is resolved.

Sounds like a good idea to me :)


Load-Balancing API Requests

Looking at the diagram, I don't see how Azure API Management would load-balance across multiple Azure Open AI instances. Typically, I would also expect to see an internal Azure Application Gateway between Azure API Management and Azure Open AI instances in addition to the depicted external Azure Application Gateway.

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

ID: 59e64035-d0cb-d2cf-0848-4153b5cfe876
Version Independent ID: 59e64035-d0cb-d2cf-0848-4153b5cfe876
Content: Implement logging and monitoring for Azure OpenAI large language models - Azure Architecture Center
Content Source: docs/ai-ml/openai/architecture/log-monitor-azure-openai.yml
Service: architecture-center
Sub-service: example-scenario
GitHub Login: @jakeatmsft
Microsoft Alias: jacwang

@simonkurtz-MSFT
Thanks for your feedback! We will investigate and update as appropriate.

@simonkurtz-MSFT  - We are leveraging the APIM policy to provide the load-balancing functionality for the backend Azure OpenAI resources.  Random load balancing is sufficient for many use cases, additionally, you can utilize the APIM native cache to persist the state of each resource/model if you want to be more sophisticated in your request routing.

Thank you for your feedback.  We have created internal ticket 177452 to track and investigate this issue.

#please-close


Proposal to have an SMI authentication policy

Users would like to optionally enable/disable mTLS between services. SMI should have an authentication policy that allows users to configure whether mTLS is enabled at different scoping levels - global scope, per service, per namespace.
This will give users flexibility to turn mTLS on/off as they desire.

I know there has been some talk in previous issues about this but don't know the details.
@grampelberg, @nicholasjackson  - any historical context would be greatly appreciated!

@shashankram would you mind explaining what problem you're solving? I'm super interested in the use case!

@shashankram would you mind explaining what problem you're solving? I'm super interested in the use case!

Sorry if the description wasn't clear enough.
I am looking for a way to allow users to control their authentication policies, primarily around mTLS authentication. Since SMI policies govern traffic between services in a mesh, exposing an SMI policy to configure authentication for communication seems like a natural extension. It would allow a means for users to control whether one or more workloads in the service mesh can receive plain text traffic, ie. non mTLS.


At what point are notifications of code review comments sent?

The documentation is not clear about the trigger for notifications to be sent out to the PR creator when reviewers post comments? Is it at the time a comment is added? Or, when an action is chosen for the PR (Approve, Reject, etc.)?

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

ID: 028e7b61-62e4-a10e-e420-3535f7843f0e
Version Independent ID: 1a6fdbe7-5ba4-a7f8-1b2e-271289a7b9fd
Content: Review and merge code with pull requests
Content Source: docs/git/pull-requests.md
Product: devops
GitHub Login: @steved0x
Microsoft Alias: sdanie

@jeyoung There are different notifications for when someone votes on a PR, and when someone comments on a PR. Depending on which subscriptions to notifications you have, you may not get notifications for all events. See View your subscriptions, op out as needed to see how to check your subscriptions and see what notifications you should be receiving. See Default and supported notifications and Manage your personal notifications for more information.
The notifications should go out as soon as the event, such as voting or commenting, occurs.
Does this info help? Thanks, Steve


Album Info

When listening to an album that includes 2 or more tracks.
When you hover over the album picture, "on "Album Name "" is displayed.
I suggest instead of "on" to write "Album".

It's a deliberate choice. The example was taken from Spotify Rich Presence. So this won't be changed

If/When discord allow use Listening activity type, this modification instantly switch to it (Because it already sends type 2 LISTENING. Discord client just ignores it). It will look like this


Reduce text in "Add Gradebook Item" screen when cat & weighting is enabled

The current "Add Gradebook Item" screen shows a big chunk of blue info text when cat & weighting is enabled:

Can we reduce this text to the following:

Items not assigned to a category will not count toward the course grade.

Confirmed in trunk:


Review and round out decimal64 testing

A key concern is accurate numeric outcomes for all arithmetic operations, including parsing and formatting. The last thing we want is some variant of 0.1 + 0.1 + 0.1 ≠ 0.3.
Some thought needs to go into the completeness of the tests such that all conceivable edge cases are covered. The following is a reasonably thorough but incomplete set of cases to consider.

Boundaries such the largest and smallest possible exponent
Every combination of positive and negative arguments
Every combination of kinds of arguments (SNaN, QNaN, ±∞, ±normal, ±subnormal, ±0)
Numbers close to ±0
The smallest normal number and its neighborhood
±10ⁱ and its neighborhood for every representable i
±2ⁱ and its neighborhood for every representable i
Operations jumping numbers into different phases (e.g., doubling ±the largest number = ±∞, halving ±the smallest number = ±0. Consider this for every operation
Parsing massive numeric representations (e.g., a number with 5000 digits)
Parsing out of range numbers (e.g., 1E1234 or 1E-1000)

3.1. Make sure to include the largest and smallest ±normals and ±subnormals.


Find a package:checks analogue of flutter_test's widget-finder matchers

In Flutter widget tests written with the venerable expect API, it's common to have expectations like
    expect(find.whatever(), findsOneWidget);

using the matchers findsOneWidget, findsNothing, and findsWidgets. See docs.
We're using package:checks instead, which is the projected, still-beta, successor to expect with a fancy type-safe API, so these "matchers" like findsOneWidget aren't available; and we don't currently have great analogues of them. One thing we currently often do in place of findsOneWidget is
    tester.widget(find.whatever());

which will indeed throw if there isn't exactly one such widget, but it's not great because it doesn't look like something intended as one of the checks that are the payload of the test — it looks like it's just part of the test's setup (and on top of that, isn't doing anything). A bit of discussion here: https://github.com/zulip/zulip-flutter/pull/207#discussion_r1256975139
Shouldn't be hard to write appropriate checks in the package:checks API. We should do that, convert our existing tests, and also make sure there's a discussion upstream about there being good support for using package:checks with package:flutter_test before the former is rolled out as stable.

Hello! I stumbled upon this looking to see if anyone else had written some widget-finder-with-checks tests.
I didn't see any here, so I went ahead and wrote my own: https://github.com/PHS-TSA/our_democracy/commit/d1eb20cc6144b15e52f5a4034f5a927badf009f5#diff-fd2549beda0a78cbca448d282c13fd259d03f2bbeb229f978e77a66afe2df8d1
I hope it helps!
If you want to just plain vendor it, feel free to, it's 99% code from flutter_test (BSD-3; should already be depended on). If you need my consent, I'm fine with licensing the remaining 1% under Apache-2. I'd argue it's not enough to be copyrightable, but I'm not a lawyer, so... 🤷♂

Very interesting, thanks!
I see the core of that implementation of findsOne is based on _FindsCountMatcher.matches from flutter_test.
I think the strategy I'd hope to be able to make work is to reuse the existing logic from there, rather than have to duplicate it. (There's a couple of thousands of lines in that matchers.dart file.) So for example something like:
  final matchState = {};
  if (!findsOne.matches(finder, matchState)) {
    return Rejection(
      // … call `findsOne.describe` and `findsOne.describeMismatch`, passing `matchState` to the latter …
    );
  }
  return null;

Even if that doesn't end up with something that would work well for a totally general implementation of the Matcher API, so long as it works well with the matchers that flutter_test actually supplies, that'd be a good outcome.

So you'd want an API like this?
check(find.whatever()).findsOneWidget();

or:
check(find.whatever()).legacyMatcher(findsOneWidget);

Or, I suppose, .findsOneWidget() as typesafe sugar for .legacyMatcher(findsOneWidget)?
Just figure I might as well make my impl work for y'all if I'm going to write it either way. I'm not particularly thrilled with the prospect of maintaining all of matcher 😀

I couldn't figure out async matchers, but here's an impl for sync matchers.
You can pass async matchers, and they pass, but I think it's mostly a noop.
Tested by making sure that all my tests still pass.
import 'package:checks/context.dart';
import 'package:flutter_test/flutter_test.dart';
import 'package:matcher/src/expect/async_matcher.dart';

extension LegacyMatcher<T> on Subject<T> {
  void legacyMatcher(Matcher matcher) {
    context.expect(() => [], (value) {
      final matchState = <Object?, Object?>{};
      final match = matcher.matches(value, matchState);
      if (!match) {
        final description = matcher.describe(StringDescription());
        final mismatchDescription = matcher.describeMismatch(
          value,
          StringDescription(),
          matchState,
          false,
        );

        return Rejection(
          which: [description.toString()],
          actual: [mismatchDescription.toString()],
        );
      }
      return null;
    });
  }

  Future<void> legacyMatcherAsync(AsyncMatcher matcher) async {
    await context.expectAsync(() => [], (actual) async {
      final Object? match = await matcher.matchAsync(actual);

      // Do something with the match.
    });
  }
}

Ok, wrote two basic packages, including async support. I'll probably publish it to pub at some time, any permissive license would work for y'all if it's external, right?

@gnprice, does https://pub.dev/packages/legacy_checks (& https://pub.dev/packages/flutter_checks) work for y'all?

Neat, thanks for building this!
So for the example in my original description above:
    expect(find.whatever(), findsOneWidget);

it looks like one would write:
    check(find.whatever()).findsOneWidget();

(And actually one would write findsOne instead of findsOneWidget, in both versions, since findsOne was introduced and deprecated findsOneWidget about a month after I originally filed this issue.)
That looks like a very clean usage pattern to me. I think the main remaining question I have is what the output looks like when things don't match: how does the output for some example compare between the old expect way and the new checks plus flutter_checks way?
That question comes to mind because the quality of output when a test fails is something that can make a big difference in the effort spent working out what a failed test is telling you (is it a bug in the change you just made? or is the test just brittle and needs updating? or something else?). It's therefore something both the expect and checks implementations spend significant effort on, and so I'm curious how well the matchers' output translates through the checks API.
And then I guess after that the only thing flutter_checks needs is to add similar wrappers for all the other matchers in that matchers.dart file in package:flutter_test. Which should be nice and straightforward given those handy wrappers in legacy_checks.

(I'll be offline next week and so might not reply; but I'll see any comments here when I'm catching up after that. Other Zulip folks might also reply before I do.)

Neat, thanks for building this!

You're welcome! 😁

So for the example in my original description above

Yup, exactly!

how does the output for some example compare between the old expect way and the new checks plus flutter_checks way?

Good question! Being perfectly honest, I forgot to add failures to the test suite so 🤷♂ (I did test it manually, they work, but I didn't pay attention to more than that)
I'll try to find some time either tomorrow or next week to investigate and probably publish a new version with better formatting.

And then I guess after that the only thing flutter_checks needs is to add similar wrappers for all the other matchers in that matchers.dart file in package:flutter_test. Which should be nice and straightforward given those handy wrappers in legacy_checks.

Yeah, I only wrote what I needed for some boilerplate tests for a project I'm starting, which wasn't much 🙃
It shouldn't be too hard to write some more, I just didn't want to write them all if y'all had something different in mind api-wise (If I'm going to publish it, might as well make it usable), given that I didn't need them myself.

I'll try to find some time either tomorrow or next week to investigate and probably publish a new version with better formatting.

Sounds great, thanks!

Yeah, I only wrote what I needed for some boilerplate tests for a project I'm starting, which wasn't much 🙃
It shouldn't be too hard to write some more, I just didn't want to write them all if y'all had something different in mind api-wise

Yep, good strategy.

I just got around to pushing and publishing<EMAIL_ADDRESS>It now supports all matchers from flutter_test.

Thanks again @lishaduck for building that flutter_checks package!
We've now started using it, with https://github.com/zulip/zulip-flutter/commit/90c7b26266c6e576534f1ecb3b10b527c4f9a11b / #949.
I'll close this issue, as a package:checks analogue of those matchers from flutter_test has been found. I'll file a separate follow-up issue for converting our existing tests from patterns like a bare tester.widget(find.whatever()); to patterns like check(find.whatever()).findsOne();.

Filed #952 for that follow-up.


403 Forbidden

I keep getting the error 403 Forbidden nginx/1.17.10
I cant find a way to fix this issue

web_1                | 2022/04/11 23:41:37 [error] 15#15: *14 open() "/usr/share                                                                                /nginx/html/vaer" failed (2: No such file or directory), client: <IP_ADDRESS>                                                                                , server: localhost, request: "GET /vaer HTTP/1.1", host: "play.lyrix.cf"
web_1                | <IP_ADDRESS> - - [11/Apr/2022:23:41:37 +0000] "GET /va                                                                                er HTTP/1.1" 404 556 "-" "Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/                                                                                537.36 (KHTML, like Gecko) Chrome/100.0.4896.75 Safari/537.36" "
web_1                | 2022/04/11 23:41:40 [error] 15#15: *14 directory index of                                                                                "/usr/share/nginx/html/" is forbidden, client: , server: localho                                                                                st, request: "GET / HTTP/1.1", host: "play.lyrix.cf"
web_1                | 3 - - [11/Apr/2022:23:41:40 +0000] "GET / H                                                                                TTP/1.1" 403 556 "-" "Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.                                                                                36 (KHTML, like Gecko) Chrome/100.0.4896.75 Safari/537.36" ""

You need to put something in the web directory


Extend error message when target disjoint connectivity

Closes #10731
In optimization level 0, the default layout strategy is trivial. In the context of a disjoined coupling map, trivial might lay the circuit in an incompatible setting. qiskit.transpiler.passes.layout.disjoint_utils.require_layout_isolated_to_component checks for that imcompatibility and currently raises Chosen layout is not valid for the target disjoint connectivity. However, the user might not actively choose any layout.
This PR adds more details to the error. For example: The qubit q[2] laid out in the physical qubit 2 is in a disconnected component with respect to other necessary qubits.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
2 unchanged lines in 1 file lost coverage.
Overall coverage increased (+0.04%) to 87.291%

Files with Coverage Reduction
New Missed Lines
%

qiskit/extensions/quantum_initializer/squ.py
2
80.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.04%

Covered Lines:
74395

Relevant Lines:
85226

💛  - Coveralls


Clean up Autotools

@dbry modernizing and simplifying the autotools part. There should be no functional changes.

Thanks so much for doing this! I am just out the door for the weekend, but I will try this all out first thing next week.

Travis and I are both having trouble with AM_ICONV. I updated Travis to focal (the latest) to get to Autotools 1.16, but still got failure. You can see them here. The Mac and Focal versions fail slightly differently, but they're both related to AM_ICONV.

can you try installing gettext too?

gettext (GNU gettext-runtime) <IP_ADDRESS>
Copyright (C) 1995-1997, 2000-2007 Free Software Foundation, Inc.
License GPLv3+: GNU GPL version 3 or later <http://gnu.org/licenses/gpl.html>
This is free software: you are free to change and redistribute it.
There is NO WARRANTY, to the extent permitted by law.
Written by Ulrich Drepper.

Where are you defining the dependencies you need to pull in for Homebrew in Travis?

As far as I know, everything is in this file and most of the Homebrew stuff is to build the man pages.

This worked for me to build locally:
touch config.rpath
./autogen.sh

Okay, I got the Travis Mac build fixed by upgrading from xcode11 to xcode12. And my touch fix for Ubuntu worked on Travis too, but that's probably not the right fix, correct?

For the touch fix, try the following:
cp /usr/share/gettext/config.rpath .

I guess that should go in autogen.sh since that's where everything needed to prep the environment should go, right? I don't imagine that the correct why to handle this is to put a recent copy of config.rpath in the repo, right? I made an attempt here and Travis is happy at last.


[String] Add Solution to Buddy Strings

Hi soapyigu, thanks for your reply. All resolved as requested, please review again. Thanks.

Could you please resolve the merge conflict?


Dialogs now has equal behaviour

Introduce boost.locale dependency in Msys, no need to update dependencies since it is automatically installed when installing boost
Fix crashes due to special characters
Correct start directory is now selected, slightly different behavior than gtk due to windows conventions
Default file extension is selected as .cpp
Less copies due to move semantics

Please merge!


Misleading entry in kubectl cheatsheet

This is a Bug Report

Problem:
https://kubernetes.io/docs/reference/kubectl/cheatsheet/#interacting-with-running-pods lists:
kubectl run nginx --image=nginx -n
mynamespace                                         # Run pod nginx in a specific namespace

There are at least two problems

This command doesn't interact with a running Pod, so it's in the wrong section
It's invalid code. If you paste that into a POSIX shell, first kubectl will tell you that you didn't specify a namespace, and then your shell will fail to find a command called mynamespace.

Proposed Solution:

Move that entry under Creating objects
(possibly with an explanatory comment)
I'd put it after kubectl explain pods.
Fix it so that it's valid POSIX shell
Consider revising the comment to explain that creating a Pod like that implies that the new Pod has 1 container named after the image (nginx in this case).

Page to Update:
https://k8s.io/cheatsheet

Additional Information:
/language en

/language en

/help
/triage accepted
/priority backlog

/assign

It's invalid code. If you paste that into a POSIX shell, first kubectl will tell you that you didn't specify a namespace, and then your shell will fail to find a command called mynamespace.

We need to first create a namespace mynamespace by running kubectl create namespace mynamespace, then it will work. So, there is nothing wrong with current command.

Do we still want to move this up to the Creating Objects section?

@sftim We need to first create a namespace mynamespace by running kubectl create namespace mynamespace, then it will work. So, there may be there is nothing wrong with the current command.

The linebreak is a problem too. If you paste that into a POSIX shell as-is: first kubectl will tell you that you didn't specify a namespace, and then your shell will fail to find a command called mynamespace.
We should fix both issues.

@sftim We need to first create a namespace mynamespace by running kubectl create namespace mynamespace, then it will work. So, there may be there is nothing wrong with the current command.

The linebreak is a problem too. If you paste that into a POSIX shell as-is: first kubectl will tell you that you didn't specify a namespace, and then your shell will fail to find a command called mynamespace.
We should fix both issues.
I removed the line break from the command. Hopefully, this should solve the issue.

@niteshseram is there a pull request you'd like reviewed?

@niteshseram is there a pull request you'd like reviewed?

I created this PR #29902 but it seems there is another similar issue #29278 with the PR, so I closed it. Let me know if it is different I can reopen it.

/remove-lifecycle stale


Managing my name

The latest commit have my name as 'Ankit gupta' (g lowercase).  Adding here would list those commits also in contrib list.
Thanks

Could you please add a test case?

@fxn done.


Evaluated value doesn't update when switching prepared statements on/off

User on discord is facing this issue intermittently for their queries. It is not necessarily triggered by the switching of prepared statements. Workaround for this currently is to trigger evaluation by editing the query.


Add this project to awesome rust

This is almost the first rust project in Kubernetes Ecosystem, It has lots referential value for users to use K8s in rust.
We could add this project to awesome-rust.

Submitted https://github.com/rust-unofficial/awesome-rust/pull/745.

@wonderflow https://github.com/rust-unofficial/awesome-rust/pull/745 has now been merged; This issue can now be closed.

Closing.
Thanks @ewbankkit


mdbook-katex: 0.3.15 -> 0.4.1

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/lzanini/mdbook-katex/releases.
meta.description for mdbook-katex is: A preprocessor for mdbook, rendering LaTeX equations to HTML at build time.
meta.homepage for mdbook-katex is: https://github.com/lzanini/mdbook-katex
Updates performed

Rust version update

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.4.0 with grep in /nix/store/hbxjj84mm65khqnp57r4dfmrjrfy7rhf-mdbook-katex-0.4.0
found 0.4.0 in filename of file in /nix/store/hbxjj84mm65khqnp57r4dfmrjrfy7rhf-mdbook-katex-0.4.0

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

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
mdbook-katex

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/hbxjj84mm65khqnp57r4dfmrjrfy7rhf-mdbook-katex-0.4.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

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

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/hbxjj84mm65khqnp57r4dfmrjrfy7rhf-mdbook-katex-0.4.0
ls -la /nix/store/hbxjj84mm65khqnp57r4dfmrjrfy7rhf-mdbook-katex-0.4.0/bin

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

1 package built:

mdbook-katex

Maintainer pings
cc @lovesegfault for testing.

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/lzanini/mdbook-katex/releases.
meta.description for mdbook-katex is: A preprocessor for mdbook, rendering LaTeX equations to HTML at build time.
meta.homepage for mdbook-katex is: https://github.com/lzanini/mdbook-katex
Updates performed

Rust version update

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.4.1 with grep in /nix/store/byni8mqhxlpjwxmkgbjzw5prcc89x445-mdbook-katex-0.4.1
found 0.4.1 in filename of file in /nix/store/byni8mqhxlpjwxmkgbjzw5prcc89x445-mdbook-katex-0.4.1

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

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
mdbook-katex

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/byni8mqhxlpjwxmkgbjzw5prcc89x445-mdbook-katex-0.4.1 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

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

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/byni8mqhxlpjwxmkgbjzw5prcc89x445-mdbook-katex-0.4.1
ls -la /nix/store/byni8mqhxlpjwxmkgbjzw5prcc89x445-mdbook-katex-0.4.1/bin

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

1 package built:

mdbook-katex

Maintainer pings
cc @lovesegfault for testing.


Add support for getting a tenant by ID

As documented in the Frontegg API.
I opted to not expose the fact that Frontegg returns an empty or single element list here. I'm open to bikeshedding the synthetic 404 error. I went with that because it felt more RESTful.

@benesch thanks for the review! I pushed updates to address your feedback. I also added a commit that allows for testing against in-use workspaces (as the cleanup methods will tear down everything). If this is undesirable, I can drop it.

If this is undesirable, I can drop it.

Not sure! I did this for the Orb API client because Orb workspaces are much harder to come by. Frontegg workspaces are easier to come by, but .. also maybe they're moving away from making workspaces cheap. So happy to take the commit for now! Can always revisit in the future if we add API endpoints that are harder to test in a live Frontegg workspace.

Closing this because it appears I need to submit PRs using a branch, not a satellite repo. I'll refile.


Where to give feedback?

You say "I would greatly appreciate it if you kindly give me any feedback on this document."
I'm currently reading your document and I will use some part for my Advanced R course.
Where do you want to get some feedback?

Thank you for the comment.
I was expecting the following feedbacks

corection of grammatical mistakes or those of difficult to read sectenses.
corection of mistakes in Rcpp.
Pointing out of the fix due to Rcpp version upgrade

I would appreciate if you could edit the .md file and send me a pull request.
But if you are not familier with git, I would welcome you to describe feed backs in this issue.

I'm familiar with Git but Fork+PR is sometimes too much for just one small change, I think.
Just one comment then, you say that return is mandatory, but it's not when the return type is void.

I fixed the point. Thanks!


Update power-analysis.md

add targeting gate and past experiment options

🚀 Deployed on https://645ebf018bb2215185851e11--cozy-fox-0defec.netlify.app


Update MechanicMain_tr_TR.lang

All translated and fixed few lines that was wrong b4 commiting

Fails to merge, please resolve


4D/BTi import error reading channel data

Describe the bug
Program fails to read a 4D data file, probably due to bad interpretation of 4D data file header
Steps and/or code to reproduce
import mne
fname = 'c,rfDC,fbp3-75n,o'
raw = mne.io.read_raw_bti(fname)
Expected results
read in the raw data
Actual results
crashes
/usr/local/lib/python3.5/dist-packages/mne-0.17.1-py3.5.egg/mne/io/bti/bti.py in _read_channel(fid)
660                 'index': read_int32(fid),
661                 'checksum': read_int32(fid),
--> 662                 'off_flag': read_str(fid, 16),
663                 'offset': read_float(fid)})
664
with UnicodeDecodeError: 'ascii' codec can't decode byte 0xa4 in position 4: ordinal not in range(128)
Additional information
Platform:      Linux-4.9.0-8-amd64-x86_64-with-debian-9.8
Python:        3.5.3 (default, Sep 27 2018, 17:25:39)  [GCC 6.3.0 20170516]
Executable:    /usr/bin/python3
CPU:           : 4 cores
Memory:        15.7 GB
mne:           0.17.1
numpy:         1.12.1 {lapack=lapack, blas=blas}
scipy:         0.18.1
matplotlib:    2.0.0 {backend=TkAgg}
sklearn:       0.18
nibabel:       2.1.0
mayavi:        Not found
cupy:          Not found
pandas:        0.19.2
dipy:          Not found
Proposed fix:
4D/BTi's $STAGE/map/src/include/dftk_channel_ref.h has
struct dftk_channel_ref_data
{ [...]
int32_t         checksum;
char            off_flag[4];
float           offset;
char            reserved[24];
};
and in mne/io/bti/bti.py 'off_flag' and fid.seek() should be adjusted:
def _read_channel(fid):
[...]
'checksum': read_int32(fid),
'off_flag': read_str(fid, 4),
'offset': read_float(fid)})
fid.seek(24, 1)

@ebeich can you make a PR with your proposed changes?
Any chance you can make and share a tiny recording that shows this problem so we can add it to our set of regression test files?


Generated client.dart-code has errors (Target of URI doesn't exist - database_connection.dart)

Describe the bug
Recently, I used transaction for the first time in my serverpod app. I used it in the implementation of endpoints. Now, the file protocol/client.dart, produced by the command 'serverpod generate' contains the line
import 'package:serverpod/src/database/database_connection.dart' as _i4;
This produces the error:
Target of URI doesn't exist: 'package:serverpod/src/database/database_connection.dart'.
Try creating the file referenced by the URI, or try using a URI for a file that does exist.
To Reproduce
Steps to reproduce the behavior:
In my case, the following endpoint implementation caused the error:
import 'package:eventer_server/src/generated/event_location.dart';
import 'package:serverpod/serverpod.dart';
class EventLocationEndpoint extends Endpoint {
Future create(Session session, EventLocation eventLocation,
Transaction? transaction) async {
return await EventLocation.db
.insertRow(session, eventLocation, transaction: transaction);
}
Future delete(Session session, EventLocation eventLocation,
Transaction? transaction) async {
return await EventLocation.db
.deleteRow(session, eventLocation, transaction: transaction);
}
}
Expected behavior
The generated code should be correct.
Serverpod versions

CLI Version: 1.2.7
Library version: 1.2.7

Platform information
I use viscose on Mac.

Hello @hoeverma!
The reason you are getting this error is that you have a transaction as an input parameter to an endpoint. This makes it generate on the client side as well.
Transactions are a server-only concept since they only make sense in the scope of an open database connection. Once your endpoint has returned the connection for that request to the database will be closed and the transaction would not be based in anything.
If you remove transaction from your endpoints parameters the error should go away.

Thank you very much, I got it!

@SandPod But if I wanted to run a transaction within the current endpoint and also call a method from another endpoint as part of the same transaction, I would have to pass the transaction object as a parameter to that other endpoint, right?

@SandPod But if I wanted to run a transaction within the current endpoint and also call a method from another endpoint as part of the same transaction, I would have to pass the transaction object as a parameter to that other endpoint, right?

We don't recommend calling endpoints within endpoints. In that case, I would suggest you extract the shared logic from the endpoint and just make it a regular function. Then the transaction could be a parameter to that shared function.

@SandPod But if I had to, is this a bad approach to do it ?
extension SessionX on Session {
  UsersEndpoint get usersEndpoint {
    final usersEndpoint = (server.endpoints
        .getConnectorByName(UsersEndpoint.endpointConnectorName)
        ?.endpoint as UsersEndpoint?);
    if (usersEndpoint == null) {
      throw Exception(
          'Endpoint connector "${UsersEndpoint.endpointConnectorName}" not found');
    }
    return usersEndpoint;
  }

  ConfigsEndpoint get configsEndpoint { ... }

  ...
}

And then in any other endpoint I can do this:
final users = await session.usersEndpoint.getAll(session);

@SalahAnwer-dev  I'm sorry, but I don't think I understand the motivation behind forcing the business logic to be defined inside of the endpoint. Could you elaborate a bit on what you see the as benefits and at the same time highligt the drawbacks of moving it out?
The main benefit that endpoint bring are:

They define the API layer of your server (serialization and endpoints definition).
Generated client code to make calling the server simple.
Authentication is handled by the framework.

From the example code you have presented, I don't understand how this is helpful.
From my perspective, it seems that you have two, or more, entry points into your server that rely on the same business logic.
This should be the perfect opportunity to encapsulate that business logic with a clear interface. It also adds the benefit that if your endpoints evolve in a different direction, the unit of your buisiness logic would still stay the same.


Feature: support language variant as django translation

A language variant should be considered to be supported, even if it's not defined in the PARLER_LANGUAGES (just as what django does).
For example,
In settings.py
PARLER_LANGUAGES = {
    None: (
        {'code': 'en',},
        {'code': 'fr',},
    ),
    'default': {
        'fallbacks': ['en'],
        'hide_untranslated': False,
    }
}

Then
assert model.safe_translation_getter('title', language_code='fr') == 'bonjour'
assert model.safe_translation_getter('title', language_code='fr-ca') == 'bonjour'
assert model.safe_translation_getter('title', language_code='en') == 'hello'

Or
with switch_language(model, 'fr-ca'):
    assert model.title == 'bonjour'

Hi,  https://github.com/edoburu/django-parler/issues/98 might relates to this one.

Good point, now I see what #98 could be about.
This needs some work across the whole app + unittests in place.
If someone can pick up that task and provide a PR, I'd be delighted!
Otherwise, you'll have to wait somewhat longer until I have time for it..
I'm in favor of using just PARLER_LANGUAGES to determine whether the language code should be rewritten before making the database query. When people want a fr-ca -> fr -> en fallback, it should be explicitly configured in the settings. Otherwise, when only fr and en are in PARLER_LANGUAGES, any variant will be rewritten.

I will work on this but I need some help. :)
I am agree with you to use PARLER_LANGUAGES to determine whether or not to re-write the language to do the query. It seems to be clear.
However, I still have some doutes:
Today the PARLER_LANGUAGES doesnt restrict the translation language. For example,
with switch_language(model, 'a-language-undefined'):
    model.title = 'some dirty strs'
    model.save()
assert model.safe_translation_getter('title', language_code='a-language-undefined) =='some dirty strs'

Therefore, we might have a problem: before we do the query, we dont know if the translation exists. For example,
PARLER_LANGUAGES = {
    None: (
        {'code': 'en',},
        {'code': 'fr',},
    ),
}

Then,
model.set_current_language('fr-ca')
model.title = 'bonjour'
// here we have to query first with fr-ca, not fr

Otherwise, we can make the translation query with all possible variants and get the closed one. For example, we query the translation with language_code like fr-xx or fr if the given code is fr or fr-fr. But it seems to complicate and be bad for the performance.
Another possible: we can restrict the translation with PARLER_LANGUAGES: we can not translate a model in chinese if cn is not defined in PARLER_LANGUAGES. To retrieve the translation, we also determine the language_code with PARLER_LANGUAGES.
How do you think ?


[sw/silicon_creator] Remove volatile qualifier from sec_mmio_ctx

This PR

(first commit) removes the volatile qualifier from the declaration of sec_mmio_ctx, and
(second commit) caches sec_mmio_ctx.last_index to reduce code size and launders i in upsert_register().

@luismarques it would be great to hear what you think about the second commit, specifically the caching part. I was first surprised to see that this change had an effect on code size but now I wonder if it's because sec_mmio_ctx is an extern. (I verified that we pass -Os to the compiler).
I compared the disassembly for ROM before and after this change:

.text shrinks by 20 bytes

upsert_register() shrinks by 14 bytes
sec_mmio_check_values() shrinks by 6 bytes

So, all the bytes are accounted for and there are no unexpected size changes.
See:

Partial disassemblies of the functions above before and after this PR.
Full disassembly before and after this PR

Note: Both gists have two files: see volatile_sec_mmio_ctx_t.dis for before, and non_volatile_sec_mmio_ctx_t.dis for after.
Signed-off-by: Alphan Ulusoy<EMAIL_ADDRESS>

Thanks! That change would not be problematic. Would you mind sending me a link that explains this, i.e. member order vs code size? I'm quite familiar with member order and the size of an object but I don't think I encountered this before.

That isn't a general C optimization. The issue is that the values and addrs members are large arrays, so the offset of the other members increases. Those latter offset values can then no longer fit the 12-bit immediate of e.g. addi, increasing code size. That further interacts with other optimizations. The exact behavior depends on the version of Clang. With Clang master that member reorder it even makes the "caching" of last_index unnecessary (but let's keep it).

Thanks! That change would not be problematic. Would you mind sending me a link that explains this, i.e. member order vs code size? I'm quite familiar with member order and the size of an object but I don't think I encountered this before.

That isn't a general C optimization. The issue is that the values and addrs members are large arrays, so the offset of the other members increases. Those latter offset values can then no longer fit the 12-bit immediate of e.g. addi, increasing code size. That further interacts with other optimizations. The exact behavior depends on the version of Clang. With Clang master that member reorder it even makes the "caching" of last_index unnecessary (but let's keep it).

Thanks for the explanation!


Build using JDK9 on travis

Just released gradle 4.1-milestone-1 will allow us to unblock progress on the JDK bridge.
Works locally, will proceed on making it work on travis.
Refs #294
Unblocks https://github.com/reactive-streams/reactive-streams-jvm/pull/296
Gradle release: https://github.com/gradle/gradle/releases/tag/v4.1.0-milestone-1

FYI, the upgrade was done using the recommended method ./gradlew wrapper --gradle-version=4.1-milestone-1

Stuck on attempting to get JDK9 on travis, gradle is ready ;<

Thanks to some dedective work by @TheSnoozer ( https://github.com/ktoso/maven-git-commit-id-plugin/issues/292#issuecomment-314550652 ) I was able to get JDK9 running, now it's a question of making the build such that we have one separate project that builds only on 9, and we can prepare the bridge artifact then.

See here https://travis-ci.org/reactive-streams/reactive-streams-jvm/builds/252653299?utm_source=github_status&utm_medium=notification

You may need a script entry export GRADLE_OPTS=-Xmx1024m to avoid the OOMKiller 137.

Closing, will be done as part of the other PR introducing the adapters I think


update changelog

What has changed from 2.x to 3?

I went through the issues in the changelog... but really couldn't decide which one was the breaking change?
All of our tests seem to pass, but just want to be on the safe side.

good question :D

see https://github.com/hapijs/qs/issues/80


The library doesn't work when targetting Android 31 and above

The error is:
PlatformException(error, co.cosmose.dealhunter.dev: Targeting S+ (version 31 and above) requires that one of FLAG_IMMUTABLE or FLAG_MUTABLE be specified when creating a PendingIntent.

+1

Currently using the same version 3.0.0 and it worked for me. If the issue still persists maybe try updating the google auth services in app level build.gradle file or follow the android documentation:
https://developers.google.com/identity/phone-number-hint/android#java


Introduce dmnd changes

Description:
Introduce changes from Desmond's forked version. https://github.com/dmnd/flow-coverage-report.git

Oh, whoops. Meant to do this on my forked version, to make it compatible with tests. Clicked the wrong button. My bad.


Regression in holidays in the Netherlands

I noticed the change Update Netherlands holidays: add holiday categories #1552 broke my unit tests as it no longer considers Liberation day as a holiday on the 5yr interval even though it is a holiday for most people in the Netherlands every 5 years.
On Liberation day (Bevrijdingsdag) the majority of employees have a day off every 5 years (2005, 2010, 2015, etc.). This was the previous behaviour which worked as expected.
Now the 5-year rule of Liberation day is combined with Good Friday in the OPTIONAL category. This equates the status of Liberation day with Good Friday, but this is not logical. Good Fridays is more similar to Liberation Day on the non 5yr-years as those are still a bank holiday but fewer people have a guaranteed holiday than in the 5-yr years. There is no option to add the non-5yr holidays it seems.
The behaviour I would expect is:

PUBLIC includes 5yr Liberation days
OPTIONAL includes Good Friday and the non-5yr liberation days

Hi @harmbuisman and thanks for bringing this up!
I understand your concern about the breaking change introduced in #1552 . Sorry about that.
I found a couple of .nl resources mentioning "Liberation day":

https://www.government.nl/topics/working-hours/question-and-answer/public-holidays-in-the-netherlands
https://business.gov.nl/regulation/holiday-entitlement/#art:liberation-day-5-may

@KJhellico, should this be revisited based on the report and the resources informaion I've just mentioned?
Thank you!

Yes, it seems that information about Liberation Day in Wiki is not entirely correct:

5 mei is sinds 1990 aangewezen als jaarlijkse nationale feestdag. Hiermee kwam een einde aan de situatie dat 5 mei alleen eens in de vijf jaar een nationale feestdag was.

@harmbuisman, thank you for this info, and sorry for inconvenience caused by these changes. I agree with this position:

PUBLIC includes 5yr Liberation days
OPTIONAL includes Good Friday and the non-5yr liberation days

but can you say whether there were any changes in 1990?

Hi, thanks for considering the changes. I was a kid in those days, but from what I understand is that.

Since 1990, Liberation Day has been a bank holiday every year during which people in some sectors are free (e.g., government), and every 5 years, most sectors have a holiday (as most labor agreements have this in).  (1990, 1991, 1992, 1993, 1994, 1995, 1996, ... etc.),
Before 1990, Liberation Day was a bank holiday every 5 years (so only 1950, .. 1985, 1990).


In mainactivity name is not showing properly

If name is more than specific length then we have to compress it and add '...' at the end

@maniyaom  Do it after length of 25 letters

it is overlapping after 17 characters


Choicefield implementation

Solves: https://github.com/ngOfficeUIFabric/ng-officeuifabric/issues/16

manually merged
took the liberty to change the icon implementation to reflect how other directives do it: (1) enum in it's own file, (2) error points to the actual enum in the master branch. Also removed a console.log() from the test for clean test runs.
Updated jsdoc: optional properties should use the = syntax, not a text label, for when we dynamically generate docs.


Update for new anthropic version please

https://github.com/anthropics/anthropic-sdk-python/releases/tag/v0.29.0 for the new claude model

https://github.com/anthropics/anthropic-sdk-python/releases/tag/v0.29.0 for the new claude model

Hi @BedirT the library should work with the latest version of anthropic SDK. Can you tell us a little bit more about the issue you are facing?

Closing this as the SDK already supports this

I see what the issue is. anthropic and openai are added as a dependency, which is wrong. Will fix this with a PR soon.


Messages number bug

If you type reddit random it's sending one message, but if you type for example reddit random facepalm it's sending a lot of messages.

Proposed fix  https://github.com/SerekKiri/RedditBot/pull/20


Move from webpack-dev-server to standard express also for backend routing

What is missing from the application? Please describe.
Currently the full webpack-dev-server package is used to serve webpack bundle. To reduce the overhead for serving the bundle, and as well provide the option to serve backend from within the application a migration to standard express is desirable.
Describe the solution you'd like

The current webpack-dev-server solution should be replaced with standard express.
The express server should have some defaults (compression, helmet) etc.
The express configuration should be configurable externally, for further customization.

Fixed with #30


add locale page

Docs Build status updates of commit afaf114:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

rd/Locale/index.yml
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


Difficulty loading stream with large number of messages in a single topic

Issue Description
We have encountered an issue with our self-hosted Zulip instance. In one of our streams, we have a topic that contains over 11,000 messages.
When the topic inside the stream is muted, we are unable to scroll up in the stream to load older messages. However, once the topic is unmuted, we are able to load and view old messages within the stream.
The large number of messages in the topic seems to be causing issues when loading older messages in the stream. No error messages or warnings are displayed in the browser console or logs when we attempt to load the messages.
Steps to Reproduce

Create a stream with a topic that contains a large number of messages (10,000+).
Mute the topic.
Try to scroll up in the stream to load older messages. The messages shouldn't be loading.
Unmute the topic.
Try to scroll up in the stream again. The older messages now should be loading correctly.

We're suspecting that this could be a bug with Zulip's handling of streams that contain a large number of messages within a single topic.
We're currently using Zulip server 3.2.13 - Zulip client v5.9.5.

Thanks for the report! It looks like you're running a version of Zulip (server 3.2) from a few years ago. We just released Zulip server 7.0 last week, so it's likely this issue has been fixed in the interim. There have also been many bug fixes and improvements made during that time as well!
The documentation for upgrading is here: https://zulip.readthedocs.io/en/stable/production/upgrade.html.
You will likely need to do some intermediary upgrades between 3.2 and 7.0. If you have any questions about the upgrade process at any point in time, you can always reach out in the #production help stream in the Zulip development community, and we’ll do our best to help you out.
I'm closing this issue, but if this is still an issue after you're running a more recent server version, then follow-up in the Zulip development community #issues stream.


How should we run NER with sequence_labeling.py

Hi Max,
Thank you very much for open sourcing the code of you brilliant paper.
I was wondering how should we use sequence_labeling.py to perform NER task.
I just simply replaced the data file paths in the run_sequencelabeling.sh, but it told me that
$ bash run_sequence_labeling.sh
2017-06-18 16:56:14,379 - Sequence Labeling - INFO - Creating Alphabets
2017-06-18 16:56:14,379 - Create Alphabets - INFO - Creating Alphabets: data/alphabets/
2017-06-18 16:56:14,379 - Create Alphabets - INFO - Processing data: ../data/split/train.conll.iob
Traceback (most recent call last):
  File "sequence_labeling.py", line 455, in <module>
    main()
  File "sequence_labeling.py", line 262, in main
    40000)
  File "NeuroNLP/neuronlp/io/data_utils.py", line 66, in create_alphabets
    pos = tokens[4]
IndexError: list index out of range

We would be very grateful if you can show an example of NER with conll-format dataset.
Thank you very much!
BTW, looking forward to your pytorch version NeuroNLP2!
Thank you very much!

Hi Bill,
The data format used in the code is in the following:
1 EU NNP I-NP S-ORG
2 rejects VBZ I-VP O
3 German JJ I-NP S-MISC
4 call NN I-NP O
5 to TO I-VP O
6 boycott VB I-VP O
7 British JJ I-NP S-MISC
8 lamb NN I-NP O
9 . . O O
So there is an indexing column at the beginning of each raw.
Thanks.
On Sun, Jun 18, 2017 at 4:58 PM, Bill Yuchen Lin<EMAIL_ADDRESS>wrote:

Hi Max,
Thank you very much for open sourcing the code of you brilliant paper.
I was wondering how should we use sequence_labeling.py to perform NER task.
I just simply replace the data file paths in the run_sequencelabeling.sh
but it tells me that
$ bash run_sequence_labeling.sh 2017-06-18 16:56:14,379 - Sequence
Labeling - INFO - Creating Alphabets 2017-06-18 16:56:14,379 - Create
Alphabets - INFO - Creating Alphabets: data/alphabets/ 2017-06-18
16:56:14,379 - Create Alphabets - INFO - Processing data:
../data/split/train.conll.iob Traceback (most recent call last): File
"sequence_labeling.py", line 455, in  main() File
"sequence_labeling.py", line 262, in main 40000) File
"NeuroNLP/neuronlp/io/data_utils.py", line 66, in create_alphabets pos =
tokens[4] IndexError: list index out of range
We would be very grateful if you can show an example of NER with
conll-format dataset.
Thank you very much!
BTW, looking forward to your pytorch version NeuroNLP2!
Thank you very much!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/XuezheMax/NeuroNLP/issues/1, or mute the thread
https://github.com/notifications/unsubscribe-auth/ADUtlpzgKGMELwUaUHwFHeeHFzQPFw9Uks5sFOavgaJpZM4N9cZi
.

--
Best regards,
Ma，Xuezhe
Language Technologies Institute,
School of Computer Science,
Carnegie Mellon University
Tel: +1<PHONE_NUMBER>

Thank you very much!
I have another question.
Since in the paper, we don't consider the pos tags or dependency roles of each token, so it is okay to assign each token with two fake placeholders as its pos tag and dependency role respectively, am I right?
I mean, will the following data work the same as yours?
1 EU A B S-ORG
2 rejects A B O
3 German A B S-MISC
4 call A B O
5 to A B O
6 boycott A B O
7 British A B S-MISC
8 lamb A B O
9 . A B O

(I'm doing a specific NER task without pos tag or dependency role.)
Thanks again!

Yes, if you only do NER, then the pos tags and dependency roles have no
impact.
On Thu, Jun 22, 2017 at 8:56 PM, Bill Yuchen Lin<EMAIL_ADDRESS>wrote:

Thank you very much!
I have another question.
Since in the paper, we don't consider the pos tags or dependency roles of
each token, so it is okay to assign each token with two fake placeholders
as its pos tag and dependency role respectively, am I right?
I mean, will the following data work the same as yours?
1 EU A B S-ORG
2 rejects A B O
3 German A B S-MISC
4 call A B O
5 to A B O
6 boycott A B O
7 British A B S-MISC
8 lamb A B O
9 . A B O
(I'm doing a specific NER task without pos tag or dependency role.)
Thanks again!
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/XuezheMax/NeuroNLP/issues/1#issuecomment-310371968,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADUtllKyu5Hi7pFLVhdA-V0dR1x1V4deks5sGmSAgaJpZM4N9cZi
.

--
Best regards,
Ma，Xuezhe
Language Technologies Institute,
School of Computer Science,
Carnegie Mellon University
Tel: +1<PHONE_NUMBER>

Thank you very much!


Canvas2d documentation, first version

I start the PR but will submit new stuffs along with in a short time.

When available, please add playground examples :)

Yep, that's tonight's plan!


Timeout Option for Read-Host

Summary of the new feature / enhancement
Can an optional timeout value be added to Read-Host?
Proposed technical implementation details (optional)
Syntax could look like this:
Read-Host [-timeout ]

Good idea, but your issue is a duplicate of:

#19664

“I see,” said the blind carpenter, as he picked up his hammer & saw.
I subscribed (I think) to the other topic. Can’t wait to hear what people think of the idea!
Thanks for the quick turnaround.
Dick Adams

The Cyber Hymnal™ <><
http://www.hymntime.com/tch

From: Michael Klement @.>
Sent: Saturday, May 20, 2023 10:49 AM
To: PowerShell/PowerShell @.>
Cc: Dick Adams @.>; Author @.>
Subject: Re: [PowerShell/PowerShell] Timeout Option for Read-Host (Issue #19684)
Good idea, but your issue is a duplicate of:

#19664https://github.com/PowerShell/PowerShell/issues/19664

—
Reply to this email directly, view it on GitHubhttps://github.com/PowerShell/PowerShell/issues/19684#issuecomment-1555939731, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ABPYO6SCYAS5HOZTMNT2HQLXHDRZFANCNFSM6AAAAAAYIUXHPM.
You are receiving this because you authored the thread.Message ID: @.***>


PAZUZU-REGISTRY-120 Clean DB creation…

and population with default features. #120

How will we handle resources? Should it be one-to-many relation to a table with blobs that store them?

The only place where the "File" where present in the DB creation.
As the tables weren't used, it is difficult to say if it was the correct way to design them.  Until we have an idea of the first ways we will use it, any tentative to design the table  will end up in a situation where we will to redo it.
The DB schema management tool is good enough to enable complex migration scenario (e.g. normalize the snippet out in the same table as binary blog).  But for now, the posgresql TEXT field allow to have simple table layout with illimited size text field and keeping very good performance.

There was a huge difference between the registry and the CLI and in order to make them compatible we had to change the registry api and the database structure to store only the docker snippet and the test snippet, cause all other data i.e files is not used by the CLI

👍

Looks good to me too.

👍

:+1

👍


Fix typo

There is a missing left quote in doc.

Thank you, @chanhyeong , for contribution; looking forward for more!


Support for schemas by URLs

Would you be interested in supporting passing in a URL as opposed to only a filename?
That way, we could do something like:
gpl2ts http://example.com/graphql

and get our generated types.
https://www.npmjs.com/package/fetch-graphql-schema supports taking a URL and returning JSON, so most of the work is already done.
I'd be willing to do a PR if this was something you were interested in. For now, I can do it in 2 steps (use the mentioned package to create the file, and then pass that file as input), but it'd be a lot nicer in our workflow to not have to create the intermediate file.

I'm not opposed to it, should just be a fetch w/ the introspection query.
Another idea could be to support stdin and then the result of the fetch-graphql-schema could be piped into gql2ts. That could be potentially useful

Yeah, that was an option I considered as well. It's one I tried before I checked to see if it was supported (it was not, haha).

yeah, i definitely think stdin and stdout could be immensely useful. i'm open to adding fetch support though as well

@CWSpear PR for accepting stdin and outputting to stdout: https://github.com/avantcredit/gql2ts/pull/44

v0.6.0 will now accept stdin/stdout


Discord Access

The link to access the Discord on the website/readme does not seem to be working for me. How can I get access - I have a few questions regrading this repo. Thanks.

Issue was on my side


GPT-4 version?

Excuse me, afaik GPT-4 is still under development and what Bing uses is still the current version of ChatGPT aka GPT-3.5, with some modifications though. Maybe we should clarify the usage of such terms.

Bing's AI uses GPT-4 as mentioned in the press conference where they demo'd it. I've also "confirmed" this from purported internal Microsoft sources.


Use standard directory highlight group

I'm not sure if this was intentional or not, but there is a specific highlight group for directories.
(This highlight group is used by other plugins e.g. netrw, nvim-tree)

I can't remember if this was intentional or not either. Looks good to me!


update README.md for Zotero 7

We need to change the security settings in the Advanced tab, along with the option to allow other applications to communicate with Zotero through the API, in order to use this plugin properly.

Hi @potatogim
Thanks for your contribution! I am assuming people use zotero-link in combination with zotero-bridge (otherwise it wouldn't work), which has relevant README guidance: https://github.com/vanakat/zotero-bridge?tab=readme-ov-file#prerequisites
Let me know if this is helpful

Actually, now that I read your PR more carefully, why do you need to change "security.external_protocol_requires_permission" ?

Hi @MunGell !
First of all, Thanks for your review!
Oh, It was my mistaken.
I changed the “security.external_protocol_requires_permission” setting because when I use Zotlit, it asks for permission every time I open an Obsidian link.
I'll close this PR as it is unnecessary :-)


[planning] failed to make a driving plan during a junction turn

Hi, found this planning bug in sim control mode.

video
recording
HD Map
To Reproduce
please see the video.
Expected behavior
Planning should be able to navigate the junction smoothly.
Desktop (please complete the following information):
OS: ubuntu 1804

@ntutangyun
Hi, I'm facing the same issue. Trying figure out what is happening behind it. Could you please provide some suggestions?
Best

I'm not quite sure. Maybe because the turning angle is too sharp...


Deactivate FastRender for SSR

I am wondering if its possible to deactivate FastRender for SSR. Currently when rendering our app on Server the DOM is loaded correctly and under the DOM output there is still the fast-render string. In our case we are loading around 1.000.000 Documents from our Collection so everytime when google is crawling us google needs to load around 6MB of Data (~5.8MB of data is fast-render data only).
So I am wondering why the SSR output still needs fast-render because the DOM is loaded correctly before and if its possible to deactivate.

This question was already here:
https://github.com/kadirahq/flow-router/issues/350
So it seems that fast-render is necessary currently. Hopefully there will be an update so it can be removed.

You might find this package a bit useful to render data and fetch data only on the client side
https://github.com/kadirahq/react-no-ssr
On 14 Jul 2016 6:37 am, at 6:37 am, "Timo Rütten"<EMAIL_ADDRESS>wrote:

Closed #667.

You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/kadirahq/flow-router/issues/667#event-721995820

Thanks @sahanDissanayake but this will only exclude some Components from the Server not the fast-render data, correct ?
If an component subscribes some data I would wrap this with NoSSR but then I dont have the data on Server I need. The problem I have is not that I have data on Client which I do not need on Server and want to exclude - my problem is that inside the returned DOM from SSR the injected data string is included.

I don't know inside out of NoSSR. But if you wrap a container that is subscribing to a publication inside a NoSSR block. then that subscription will not get data till the component is mounted on the client.
So which in return you will not see the data inside the injected data string on the DOM. I mean this is one way


Still doesn't work.

It says that unpacked files but the files are not anywhere.

That is not sufficient information to help you. What are you unpacking? How?

Crimsonland.
-unpack data-windows.pak

There's no errors, so it's definitely unpacking somewhere. The unpacked files will be in your current working directory.

No, they are not. Is that the "help" ?

Well...  If it were to fail creating the file, it would crash or show an error, but it's happily unpacking somewhere. If you are not able to find the unpacked files, then I suggest making triple sure your working directory is actually what you think it is

https://user-images.githubusercontent.com/43975882/222242466-9288eaf5-c06e-4447-8e85-cd8180d82db4.mp4

Yup, your working directory is C:\Users\PC, so I would expect you now have quite a bit of Crimsonland stuff in your profile folder 😅

So it unpacks to the working dir of the CMD and not the app exe. I see. I will have to clean my profile now.

I agree that would be a nice to have addition, but that's still the default behavior of well-behaving console apps. If there's no output path specified, or no way to specify then, then it'll operate on the working directory. The working directory isn't necessarily the .exe path


Create Model/Dataclass with/without alias name

Unexpected behavior
Output of python -c "import pydantic.utils; print(pydantic.utils.version_info())":
             pydantic version: 1.5
            pydantic compiled: True
               python version: 3.7.4 (tags/v3.7.4:e09359112e, Jul  8 2019, 20:34:20) [MSC v.1916 64 bit (AMD64)]
     optional deps. installed: ['typing-extensions']

I am using pydantic to create some API wrappers. At some point I am expecting the user to create instances of the defined schema and pass it into a function.
I am using the Model so I don't have to define a second dataclass with the exact same members.
I'd expect (at least for the pydantic dataclasses) the following to work out of the box
from typing import Dict, Any

from pydantic import BaseConfig, BaseModel, Extra, Field

BaseConfig.extra = Extra.forbid

class ItemChannelLinkDefinition(BaseModel):
    item_name: str = Field(alias='itemName')
    configuration: Dict[str, Any] = {}

ItemChannelLinkDefinition(item_name='asdf')

pydantic.error_wrappers.ValidationError: 2 validation errors for ItemChannelLinkDefinition
itemName
  field required (type=value_error.missing)
item_name
  extra fields not permitted (type=value_error.extra)

I tried both Dataclass and BaseModel and they show the same behavior.
The behavior however is (imho) rather un intuitive.
I know I can set
BaseConfig.allow_population_by_field_name = True

but PyCharm still shows an error

I think that it should be possible to create a Model/Dataclass with the defined variable names and that the alias mapping should be done in a factory function like
ItemChannelLinkDefinition.create(itemName='item', channelUID= 'chan')

which then would be consistent with
ItemChannelLinkDefinition.parse_obj({'itemName': 'item', 'channelUID': 'chan'})

Exporting models is consistent, too since the dict method accepts the by_alias argument.

This should have been a question, but I can't relabel it.
Behavior has been discussed in #565, but I'd really appreciate a hint how I can solve this.
Creating dataclasses with the same fields just for the users seems awfully backwards.

I understand your problem, but it seems you know of all the current workarounds.
The pycharm issue relates to the pycharm plugin please look or create issues there for stuff related to that
With pydantic, I'm not adding more methods like create. I could definitely consider adding a utility like model_to_dataclass, would that solve your problem. Otherwise what exactly are you looking for?

I want to use the model directly as an interface between my API and the user and I'm afraid the additional functions and members of the model will confuse him. That's why my intention is to go with the dataclass from pydantic.
However, I am not sure how to use the alias with dataclass
import dataclasses
from typing import Any, Dict

from pydantic import Field
from pydantic.dataclasses import dataclass

@dataclass
class ItemChannelLinkDefinition:
    item_name: str = Field(alias='itemName')
    channel_uid: str = Field(alias='channelUID')
    configuration: Dict[str, Any] = dataclasses.field(default_factory=dict)

o = ItemChannelLinkDefinition(item_name='asdf', channel_uid='asdfasdf')
o.__pydantic_model__.dict(by_alias=True)

because
o__pydantic_model__.dict(by_alias=True)

throws
TypeError: dict() takes exactly 1 positional argument (0 given)

@samuelcolvin
Could you give me a hint how I can serialize pydantic dataclasses with an alias?

I'm with the same problem here!
Unfortunately, my models have two "conflicting" attributes, fields and schema!
It'll be great if we can use alias option with dataclass, as well.
@spacemanspiff2007 Did you find some way to make alias work?
@samuelcolvin Could you provide an example for us?

Unfortunately not, I was trying to use the dataclass as a single input/output parameter of the api but since it was only a couple of keys I reverted to function parameters and build the dataclass now inside the function.

@thepabloaguilar the problem with conflicting names will change one day once #1001 is fully implemented, but that'll be a long time since we'll have to have a transition period when methods like fields remain with warnings.

Could you provide an example for us?

An example of what?

Using and serializing a dataclass with an alias.
See my previous question

@spacemanspiff2007 I'm afraid there's no way to use aliases with dataclasses as far as I know.
The problem is that we use standard python dataclasses:
https://github.com/samuelcolvin/pydantic/blob/0cee311be59ca09cd29a66c344cd06fb304e9ddd/pydantic/dataclasses.py#L89-L90
python's dataclasses effectively prepare a string of python code which creates the class, then evaluates it to create the dataclass object - that code is generated with argument names matching the field names - with no concept of aliases. Thus there's no way for pydantic dataclasses to have aliases without reimplementing the whole of dataclasses, which is outside the bounds of pydantic.

Can we use matadata for an alias? e. g.:
@dataclass
class MyDataclass:
    my_field: int = field(metadata=dict(alias='my_alias')

Thanks for using pydantic. 🙏
As part of a migration to using discussions and cleanup old issues, I'm closing all open issues with the "question" label. 💭 🆘 🚁
I hope you've now found an answer to your question. If you haven't, feel free to start a :point_right: question discussion.


Add wayland-cursor bindings

As discussed on IRC.
I am not sure if what I did here is right, I am not too familiar with zig yet.
I'll leave this as a draft until I have a chance to try it out.
I see potential issues in getting the CursorImage. Images aren't created by the user with a function, they are created with the Cursor. Getting an image basically looks like this in C:
struct wl_cursor *cursor = <do something to get the cursor>;
struct wl_cursor_image *cursor_image = cursor->images[0];

I am not sure if (and how) i have to tell zig that the type of cursor->images[] is the CursorImage type  defined here. Maybe Cursor should get a getImage(index: c_int) function to explicitly do that?
I am also not sure if this really belongs in wayland_client_core.zig, as these may also be of use to a Wayland server.

All done.


Retro Mode with a Controller: Seleect should toggle between Run and Walk

When playing on Retro mode, the Select button of the controller acts as the Shift key of the keyboard: it toggles Walk/Run if you're already moving, but pressing it while Idle does nothing.
Requirements:

[ ] The Select key must toggle between Walk and Run even when the character is Idle. Pressing it while moving will have the same effect as today

Fixed in #68


Updates validations.py python script

Update validations.py python script.
Fixed the behavior of valide_user function in validations.py
Closed: #1
Fixes #<issue_number_goes_here>

It's a good idea to open an issue first for discussion.

[ ] Tests pass
[ ] Appropriate changes to README are included in PR

@googlebot I signed it!


Don't auto-redirect if user has used the back btn

 NOTICE
I dedicate any and all copyright interest in this software to the
public domain. I make this dedication for the benefit of the public at
large and to the detriment of my heirs and successors. I intend this
dedication to be an overt act of relinquishment in perpetuity of all
present and future rights to this software under copyright law.

Fixes:

Needs some more polishing, but works.

[x] I made sure there are no unnecessary changes in the code*
[ ] Tested on Chromium- Browser OS
[x] Tested on Firefox

* indicates required

Closing because I deleted the branch


Rename files problem in IE all version

Hi everyone,
This is a great script i have implement this on my project. I have a small problem with rename files before uploading into folders. I have tried a lot to do so and now i am successful to achieve this but with little problem. This is working in chrome fine but except chrome it's giving me error something like this "SyntaxError: JSON.parse: unexpected character". Can anyone assist me how can i make it compatible with all browsers. I am sharing you my codes below. Please have a look at this code and let me know if you found some problem or suggest me better than it. Thanks
protected function handle_file_upload($uploaded_file, $name, $size, $type, $error) {
$file = new stdClass();
	$project = $this->trim_file_name($name, $type);
    $Ext = strrchr($project,".");
    $project = md5(uniqid(rand()));
    $project = substr($project,0,10);
    $project = $project."$Ext";

    $file->name = $project;
    $file->size = intval($size);
    $file->type = $type;
    $error = $this->has_error($uploaded_file, $file, $error);
    if (!$error && $file->name) {
        $file_path = $this->options['upload_dir'].$file->name;
        $append_file = !$this->options['discard_aborted_uploads'] &&
            is_file($file_path) && $file->size > filesize($file_path);
        clearstatcache();
        if ($uploaded_file && is_uploaded_file($uploaded_file)) {
            // multipart/formdata uploads (POST method uploads)
            if ($append_file) {
                file_put_contents(
                    $file_path,
                    fopen($uploaded_file, 'r'),
                    FILE_APPEND
                );
            } else {
                move_uploaded_file($uploaded_file, $file_path);
            }
        } else {
            // Non-multipart uploads (PUT method support)
            file_put_contents(
                $file_path,
                fopen('php://input', 'r'),
                $append_file ? FILE_APPEND : 0
            );
        }
        $file_size = filesize($file_path);
        if ($file_size === $file->size) {
        		if ($this->options['orient_image']) {
        		    $this->orient_image($file_path);
        		}
            $file->url = $this->options['upload_url'].rawurlencode($file->name);
            foreach($this->options['image_versions'] as $version => $options) {
                if ($this->create_scaled_image($file->name, $options)) {
                    $file->{$version.'_url'} = $options['upload_url']
                        .rawurlencode($file->name);
                }
            }
        } else if ($this->options['discard_aborted_uploads']) {
            unlink($file_path);
            $file->error = 'abort';
        }
        $file->size = $file_size;
        $this->set_file_delete_url($file);
    } else {
        $file->error = $error;
    }
    return $file;
}

I've posted a reply including code for the latest version here:
https://github.com/blueimp/jQuery-File-Upload/issues/622#issuecomment-4278885

Hi Its giving me error "SyntaxError: Invalid character" in IE all version neither in Firefox but its working fine in google chrome online and offline. But in my localhost its working fine in IE9, 8 not in IE7. In online its giving me error in IE all versions and in firefox. I am sharing u my codes plz tell me if something is going wrong.
class CustomUploadHandler extends UploadHandler
{
protected function trim_file_name($name, $type) {
$name = parent::trim_file_name($name, $type);
// Your file name changes: $name = 'something';
$Ext = strrchr($name,".");
$name = md5(uniqid(rand()));
$name = substr($name,0,10);
$name = $name."$Ext";
return $name;
}
}
$upload_handler = new CustomUploadHandler();


Degree Character is causing 500 Internal Server Error

I tried to create a recipe with the following line (Copy and pasted from a recipe) and kept getting a 500 Internal Server Error. When I removed the degree characters it was Okay.
(Put in the directions)
Preheat oven to 375˚F/190˚C .
 It's a problem with how the API/DB are saving the data that is causing issue.
I will take a deeper look into this.

As an addition I also noticed that it happens with the fraction characters as well:
Doesn't work:
"and set aside ¾ cup."
Does work:
"and set aside 3/4 cup."

After some investigation, look like it's a core MySQL issue (making me wish I had just used Postgres from the start). There seem to be some hacks that I can do to work around this, but right now it's not  high on my list :(
You can try some of the few things below. I will also be working on some better error messaging
See this for more info: https://code.djangoproject.com/ticket/13919

For anyone running into this issue:
Older MariaDB versions are per default only using 3-bytes unicodes. If you are experiencing the mentioned issue, you should migrate to 4-bytes unicodes, or use a more recent version of MariaDB (10.7.3 seems to be working fine for me) from the beginning.


Improve keyboard accessibility

#1866 again

Difficult to rebase branch into "development" since contents of master and development seem to be not well in sync. When trying to rebase branch I was getting lot of conflicts; from files, which I haven't touched at all (readme, min.js,....)

ok, doesn't seem that this is going to fly well, since trying to rebase changes against "development" branch raises up lot of merge conflicts, which are not related to changes in this PR at all.

and another one #1916


Update dependencies and translations to the most recent versions

/botio-linux preview

/botio test

/botio-linux unittest

/botio-linux preview

/botio test

What's wrong with the latest Puppeteer version, since it seems that you "reverted" the version slightly?

The slightly newer version didn't work on the bots because user contexts were implemented for the Webdriver BiDi implementation that we use for Firefox (see https://github.com/puppeteer/puppeteer/pull/11784), however on the Firefox side that was actually implemented quite recently in https://phabricator.services.mozilla.com/D198992. The Nightly version we have cached on the bots was from before that patch landed (because it's cached on version number and that stays at 124.0a1 until a new alpha version is created), so I decided to go back one version so we didn't have to await the version number to change.
However, I just figured now that we can also just purge the cached Nightly version for Linux and Windows from both bots. @calixteman Is that something you could do? If so, during the weekend I can redo this patch with a more recent Puppeteer version (and more recent translations if there are any new changes).

@timvandermeij done.


Automated cherry pick of #6252: fix: panic when disk has no valid storage_id

Cherry pick of #6252 on release/2.13.
#6252: fix: panic when disk has no valid storage_id

Codecov Report

Merging #6253 into release/2.13 will increase coverage by 0.05%.
The diff coverage is 22.59%.

@@               Coverage Diff               @@
##           release/2.13   #6253      +/-   ##
===============================================
+ Coverage          6.80%   6.85%   +0.05%
===============================================
  Files               535     536       +1
  Lines             90448   90602     +154
===============================================
+ Hits               6154    6215      +61
- Misses            83785   83879      +94
+ Partials            509     508       -1

Flag
Coverage Δ

#aFlag
6.85% <22.59%> (+0.05%)
:arrow_up:

Impacted Files
Coverage Δ

pkg/cloudcommon/db/db_dispatcher.go
0.00% <0.00%> (ø)

pkg/cloudcommon/policy/policy.go
1.58% <0.00%> (+0.04%)
:arrow_up:

pkg/compute/models/dbinstance_accounts.go
2.34% <0.00%> (ø)

pkg/compute/models/dbinstance_databases.go
4.24% <0.00%> (ø)

pkg/compute/models/disks.go
0.54% <0.00%> (-0.01%)
:arrow_down:

pkg/compute/models/dnsrecords.go
27.71% <0.00%> (-0.40%)
:arrow_down:

pkg/compute/models/elasticcache_instances.go
1.17% <0.00%> (ø)

pkg/compute/models/guests.go
0.56% <ø> (ø)

pkg/compute/models/loadbalancerbackendgroups.go
1.48% <0.00%> (ø)

pkg/compute/models/loadbalancerbackends.go
2.70% <0.00%> (ø)

... and 17 more

Continue to review full report at Codecov.

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


Dev/nofield/add ledger icons

This PR fixes #20128 by introducing the ledger icons into ADS. Another PR in STS adding functionality to display the images will follow.
Ledger table and View in OE:

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.003%) to 42.239%

Totals

Change from base Build<PHONE_NUMBER>:
0.003%

Covered Lines:
28172

Relevant Lines:
62332

💛  - Coveralls


Added new option to get numeric value of sensor only

Returns only the numeric value of the sensor.
Known types: temperature, humidity, "dew point"

due-point->dew-point, dew point,

Here is a suggestion for updating the PR:
diff --git a/utils/tempered.c b/utils/tempered.c
index 023ca5f..d9b3ab1 100644
--- a/utils/tempered.c
+++ b/utils/tempered.c
@@ -31,9 +31,9 @@ void show_help()
 "Known options:\n"
 "    -h\n"
 "    --help                 Show this help text\n"
-"    -t\n"
-"    --type                 Returns only the numeric value of the sensor.\n"
-"                           Known sensors: temperature, humidity, dew point\n"
+"    -t <type>\n"
+"    --type <type>          Returns only the numeric value of the sensor.\n"
+"                           Known types: temperature, humidity, \"dew point\"\n"
 "    -e\n"
 "    --enumerate            Enumerate the found devices without reading them.\n"
 "    -s <scale>\n"

Thanks for updating the PR.
This PR looks pretty useful, to get only the numeric value, which saves parsing or splitting the default, verbose output of tempered.

I guess it is intended, that the value is printed without a newline after it.


Feature request: CMD action

Please consider adding a cmd or shell command action, so we could use custom shell scripts to notify external services or even run auto-fix commands for certain problems.
The action is configured in the kibana.yml and the user UI will only choose one of the CMD actions and parameters. The UI would never change the comand executed nor accept subshells, to avoid security problems

I do not foresee SENTINL taking the responsibility to sanitize and execute potentially dangerous commands, but its quite trivial to build a custom API client to take dangerous jobs.
A very, very insecure example of both arguments, untested:
const exec = require('child_process').exec;
const fastify = require('fastify')()

fastify.get('/command', (request, reply) => {
  console.log(request.body)
  exec(cmd, function(error, stdout, stderr) {
    if (error) { reply.code(500); reply.send(stderr) }
	else { reply.code(200); reply.send(stdout); }
  });
})

// Run the server!
fastify.listen(3000, (err) => {
  if (err) {
    fastify.log.error(err)
    process.exit(1)
  }
  fastify.log.info(`server listening on ${fastify.server.address().port}`)
})

/**

	"webhook": {
          "method": "POST",
          "host": "local.service",
          "port": "3000",
          "path": "/command",
          "body": "echo 'received {{payload.hits.total}} hits'",
          "headers": {
            "Content-Type": "application/x-www-form-urlencoded"
          },
          "create_alert" : false
        }

**/

Perhaps it would be a cool idea to bundle a small working API receiver example with the Docs, helping anyone bootstrap their own execution back-end for Alerts?

my idea was to have a config file with the allowed commands to execute, the UI would only list then and allowed to select the command to run.
IF any arguments are passed, it should be a filename, that should store any value neededand the command should take care of reading/parse/whatever
So a example:
send-alert.sh
#!/bin/bash
curl http://alert-service:8888/send-alert -XPOST -d @$1

"cmd": {
     "name": "send alert",
     "script": "/path/send-alert.sh",
     "argument": "file"
 },

The UI should have then a simple pulldown list with the "send alert"
The API is not a bad workaround, but requires one to build API and manage another service, where scripts are already common on monitoring/alert
Thanks for the feedback


feat(android): check location service before permission

Platforms affected
Android
Motivation and Context

This is change for check Location Service enabled before request permission. For now when the location service is unavailable, the plugin request permission, open dialog and wait for timeout. Referencing Note: Browser permission VS OS permission in W3C, I interpreted location service availability is similar to OS permission and it should return PERMISSION_DENIED.
In W3C, it looks no distinction between fine and coarse location permissions. And WebView return coarse location ignoring enableHighAccuracy when denied permission for fine location.

Description

When request high accuracy location but only granted coarse location, onRequestPermissionResult() return PluginResult.Status.OK
check location service before permission.

Testing

I checked with virtual device location service on/off.
Checklist

[x] I've run the tests to see all new and existing tests pass
[ ] I added automated test coverage as appropriate for this change
[x] Commit is prefixed with (platform) if this change only applies to one platform (e.g. (android))
[ ] If this Pull Request resolves an issue, I linked to the issue in the text above (and used the correct keyword to close issues using keywords)
[x] I've updated the documentation if necessary

I do agree that if the OS level has GPS disabled, then returning a PERMISSION_DENIED is more reflective of the W3C standard.
Note that the android tests is failing does not appear to be related to this PR, I think the CI configs needs to be updated. I'll try to do that sometime soon and once merged, this PR will need to be rebased.


x/build/cmd/gopherbot: handle transferred issues

Gopherbot currently gets stuck in an error loop when it encounters an issue that has been transferred, such as golang/vscode-go#298, which is now #40101.
2020/08/07 19:14:35 close stale WaitingForInfo: GET https://api.github.com/repos/golang/vscode-go/issues/298/comments?per_page=1000&since=2020-07-07T17%3A07%3A54Z: 404 Not Found []
2020/08/07 19:14:35 gopherbot ran in 3.547726498s
2020/08/07 19:14:35 sleeping 30s after previous error.

It shouldn't get stuck when this happens.
/cc @golang/osp-team

This issue also affected a recently added "set vscode-go milestones" task. There is more information available in #45461.


chore(package): update browserify to version 14.4.0

Closes #7

Coverage remained the same at 99.296% when pulling bdbbb637d3431f094ccd8fe64bdb9823bfa0f72e on greenkeeper/browserify-14.4.0 into c558b612dcb25226aecb185ba8636a26a145f69e on master.

:tada: This PR is included in version 1.5.3 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Create SP w/ KV certificate integration

Investigate feasibility of creating a service principal where the certificate is requested from KeyVault.

https://github.com/devigned/level1/blob/master/scripts/create_service_principal.sh

Here are the commands to accomplish this today:
az keyvault certificate get-default-policy | az keyvault certificate create --vault-name myKeyvault -n myCert -p @-
az keyvault certificate download --vault-name myKeyvault -n myCert --file mycert.cer -e string
az ad sp create-for-rbac --cert @mycert.cer


Change selector which gives table border

Hi there.
I think that it is better to correct the table border.
If the table markup is as next code.(sorry for the long code)
The design is like capture, and it is difficult to understand the relationship between the contents.
So I fixed some of the selectors.
Please merge if you like.

      thead th
      thead th
      thead th

      tbody th
      tbody td
      tbody td
      tbody td

      tbody th
      tbody td
      tbody td
      tbody td

      tbody th
      tbody td
      tbody td
      tbody td

      tfoot td
      tfoot td
      tfoot td

I changed this a little to account for proper table markup - theads should only include ths, tfoots should only include tds, if I figured that out correctly.
Would it be better to just directly add the border to the thead and tbody instead of their children?

Thanks!
I'm not good at English, so I'm sorry if I can't communicate well.

I changed this a little to account for proper table markup - theads should only include ths, tfoots should only include tds, if I figured that out correctly.

I think this knowledge is little different.
It is possible to put th in thead. and it is also possible to put th in tfoot.
It is appropriate to markup the blank in thead not with th but with td.
https://www.w3.org/WAI/tutorials/tables/two-headers/
When we put the total on the last line, we may use th in tfoot.
https://developer.mozilla.org/en-US/docs/Web/HTML/Element/tfoot

Would it be better to just directly add the border to the thead and tbody instead of their children?

I tried to make it this time!
However, you may not be able to see well in certain old situations.
So if you decide on a supported browser, you may want to check in more detail.

I changed this a little to account for proper table markup - theads should only include ths, tfoots should only include tds, if I figured that out correctly.

To clarify,
,
, and
 must contain zero or more
 element(s), which can contain  and  elements. MDN has a good example table. You can also see the HTML Tables spec for more details.

Would it be better to just directly add the border to the thead and tbody instead of their children?

There's three kinds of selectors things we can use to style:

thead, tfoot to style the container that holds all header/footer rows (because there can be multiple header/footer rows!)
thead > tr, tfoot > tr to style one individual header/footer row
thead > tr > td, tfoot > tr > td to style one individual header/footer cell

(Disclaimer: I didn't double check my CSS selectors, but this should be close!)

@kylejrp Thanks for the more detailed comments.
@kognise I think that it is OK to merge if there is no problem with the current state?

Imo, given a layout with multiple rows in a thead or tfoot there should only be one border instead of one for each row. Thoughts?

I'm sorry. I did not understand it correctly.
I thought that every even tr would have a background color, but it was not.

thead > tr, tfoot > tr to style one individual header/footer row
thead > tr > td, tfoot > tr > td to style one individual header/footer cell

With the design that gives these two borders, I think it will be difficult for users to determine how far the thead is.

So, I tried to change the background color of the tr other than tbody to alternate, but what about?

Hey @kognise, this was merged into a hotfix branch. Should this be merged directly into master instead so this issue can be closed?

This merge request will need some work to change from Sass to CSS Variables due to the change in #71.
@kognise, any details on the merge into the hotfix branch? Maybe we could discuss git strategies in private, I've just been merging directly into master.


[TE] Self serve alert edit bug fix

Fixing graph bug in Self-Serve Alert Edit UI - the wrong granularity was being used.

wrong branch