Is the current applicable condition of t-test correct?

The Metric.calc_confidence_interval method considers t-test applicable only when a given evaluation metric score is a simple average of statistics.  I'm just wondering whether another condition needs to be checked or not. I would appreciate if there is a reference on the applicable condition of t-test.

@neubig @odashi I haven't found reliable resources in statistics about the condition. It would be great to point out such resources if any.

@neubig @tetsuok
I think we need to breakdown the condition of when it is applicable.

First, t-test can be applicable iff the target variables are sampled from some normal distribution. I think non-aggregated statistics satisfies this condition very rarely.
According to the central limit theorem, the sum (mean) of something tends toward a normal distribution regardless of the original distribution. So if the targeted variables are calculated by simple summation or mean, t-test may be applicable.

I believe this is correct. See 3.2.1 here: https://aclanthology.org/P18-1128.pdf

@neubig It looks the paper says the same thing with my comment.

According to the current implementation, the sufficient statistics of the t distribution is obtained from aggregating original values of the given sample.
https://github.com/neulab/ExplainaBoard/blob/cc2d315695cec30e21d369362d1c76ca2ce9d40c/explainaboard/metrics/metric.py#L429-L443
This means that the code does not satisfy the condition 2 of my comment, and this code looks available only when the original values are normally distributed.

Additional notes:
We may be able to assume that the variable my_mean is sampled from a normal distribution $\mathcal{N}_m$ according to the is_simple_average condition. However, we don't know the mean and the variable of $\mathcal{N}_m$ , which are different of ones calculated from the data.

Good question. Here is a reference that I think answers the question for large n: https://stats.stackexchange.com/a/44280
For small n (e.g. less than 30), the data being non-Gaussian could be an issue. Here is a nice simulation that examines how inaccurate the confidence intervals are for variously distributed data: https://stats.stackexchange.com/questions/242037/robustness-of-the-student-t-test-to-non-gaussian-data
To give some background: we used to calculate confidence intervals only using the bootstrap, but the bootstrap is slow and actually was actually the major speed bottleneck in ExplainaBoard analyses. So this is a faster (but possibly less precise) solution.
One option would be to use the t-test for larger values of n, and bootstrap for smaller values of n. But then how to choose this threshold would be another tricky question (also dependent on the underlying data distribution).

The CLT can not be applied to the original distribution.
The CLT can be applied to distribution of sums/means (with enough sample size) generated from the original data.

The problem here may be that the current code calculates the former distribution by assuming that the sample is normally distributed. If we want to apply the CLT assumption, we need to prepare datasets of sums somehow, not the original data.

Hi!

we need to prepare datasets of sums somehow, not the original data.

That's basically what the bootstrap test is doing right?
AFAIK, student's t-test is avoiding the necessity to do explicit bootstrapping by assuming that the sample mean follows Student's t-distribution (which converges to the normal at large n). This is true if the data is normally distributed, but not true for non-normal data with small n. Here is another reference that explains in a bit more detail: https://stats.stackexchange.com/questions/9573/t-test-for-non-normal-when-n50

Hi all, I learnt again about the theory around this.

That's basically what the bootstrap test is doing right?

Yes

t-test is avoiding the necessity to do explicit bootstrapping by assuming that the sample mean follows Student's t-distribution

Yes. and the topic we need to discuss is whether we are calculating the correct statistics of the samples.
First, I noticed that I have a misunderstanding around the following statistics:
https://github.com/neulab/ExplainaBoard/blob/cc2d315695cec30e21d369362d1c76ca2ce9d40c/explainaboard/metrics/metric.py#L434-L435
that calculate the correct variables applicable for one-sample t-tests. I originally thought that means/stds should be calculated from the statistics of means, but according to a text, it is sufficient to be calculated from the sample. Sorry for confusion!
As we discussed above, t-test is only applicable when we can assume that the sample mean is sampled from the normal distribution (not t distribution). The condition requires that the sample size is large enough according to the CLT. Therefore, the condition should be updated to at least:
is_simple_average and sample_size > threshold

I also noticed that there are other issues around the current code. It calculates a confidence interval of one sample, meaning that it is trying one-sample test and the test requires the equality of variations between two samples. I think this is not easily assumed in most cases, and we need to revise the underlying algorithm.
One possible way is applying Welch's test, which is enough robust against inequality of variances.
Another possibility is to apply the paird t-test as discussed in the paper Graham mentioned above. The problem here is that one-sample confidence interval is not applicable.

By either Welch's test or paird test, we need two samples at the same time, and one-sample confidence interval is not applicable.

Thanks! I'm not sure I totally understood this though:

It calculates a confidence interval of one sample, meaning that it is trying one-sample test and the test requires the equality of variations between two samples.

AFAIK, the t-test can also calculate the confidence interval of the mean of a single sample, in addition to calculating the difference between means of two samples. For instance, see the first two sections of this article: https://www.bmj.com/about-bmj/resources-readers/publications/statistics-square-one/7-t-tests

@neubig one-sample t-test is applicable only when we can assume that two samples has the same variance. It is in general not guaranteed in our pipeline.

@neubig Hi, I think my previous comment was almost the point that we need to make some consensus.
One-sample t-test is performed under an assumption that the population mean of the null hypothesis $\mu_0$ is strictly a constant. If the hypothesis follows this condition, e.g., "the accuracy of the proposed system is distributed around 70%", one-sample t-test is applicable I think.
If we need to compare two systems, we basically need to assume that two systems (baseline and proposed) are symmetry (both are distributed, and have the same property of distribution). In this case, confidence intervals calculated from one sample is not suitable to judge the hypothesis, and we need other tests that compares two systems explicitly: pairwise testing (including difference testing) or two-samples testing. In difference testing, we can also calculate the confidence interval against the distribution of differences, which is not the same as one-sample confidence interval.

OK, great, so I think we agree then? Just rephrasing to make sure that we're on the same page:
For calculating confidence intervals of a single sample, what we're doing now is probably OK, but we should add a check that the sample size is larger than a threshold (e.g. 30).
For comparing two systems, we need to implement something else (issue #325). Note that pairwise bootstrap tests are actually already implemented in explainaboard_web, so the code could probably be moved into ExplainaBoard: https://github.com/neulab/explainaboard_web/pull/259

@neubig We finally need to breakdown this issue to actual tasks I guess.
@tetsuok Could you check the whole thread?

@odashi sure.

@odashi I think required code change and the direction look good to me.

@tetsuok I think the task necessary to be applied is only modifying the condition:
is_simple_average and sample_size > threshold

If you have time, could you take some time to make a change?

@odashi sure.

Closing as the required change was merged as c7a9fad497d455a339272650ead766a83a22ba01.


Menu incorrectly hidden in gallery mode on iOS

Release with the issue:
latest dev build
Last working release (if known):
v5.2.x
Browser and Operating System:
iOS 16.6.1
Description of problem:
On iOS mobile, when looking at the recordings/snapshots view (with the tiles arrayed in rows/columns) when you scroll down, the Frigate menu bar that I have set to always be at the bottom of screen disappears. There is then no way to navigate out of this view without switching dashboards completely and switching back
Frigate card diagnostic information
will provide if needed

On iOS mobile, when looking at the recordings/snapshots view (with the tiles arrayed in rows/columns) when you scroll down, the Frigate menu bar that I have set to always be at the bottom of screen disappears. There is then no way to navigate out of this view without switching dashboards completely and switching back

Huh. Any chance you can provide a short video clip of that happening so I can be sure I understand. This only on iOS?

https://github.com/dermotduffy/frigate-hass-card/assets/37373320/5a940896-dd9a-444d-9773-980e728039b7
you can see in the attached video, in gallery mode the menu bar at the bottom goes away until the very bottom of the galley is reached. this was not happening in v5.2.x

Thank you!

this was not happening in v5.2.x

So this is a custom build from the dev branch? Mind sharing the config you're using?

sure, here is the output:
{
  "ha_version": "2023.9.2",
  "card_version": "5.2.0",
  "browser": "Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36",
  "date": "2023-09-16T17:11:44.893Z",
  "frigate_version": {
    "5db7149f926abc9b72a9114bd58ba62c": "5.0.0-beta.1/0.13.0-db6ee41"
  },
  "lang": "en",
  "timezone": "America/New_York",
  "git": {
    "build_version": "5.2.0-main+g8e3a7b6",
    "build_date": "Thu, 07 Sep 2023 06:30:48 GMT",
    "commit_date": "Sat, 22 Jul 2023 08:10:42 -0700"
  },
  "config": {
    "type": "custom:frigate-card",
    "cameras": [
      {
        "camera_entity": "camera.front",
        "live_provider": "go2rtc",
        "go2rtc": {
          "modes": [
            "webrtc"
          ],
          "stream": "front"
        },
        "id": "front",
        "frigate": {
          "camera_name": "front",
          "url": "<IP_ADDRESS>"
        },
        "triggers": {
          "occupancy": false
        }
      },
      {
        "camera_entity": "camera.back",
        "live_provider": "go2rtc",
        "go2rtc": {
          "modes": [
            "webrtc"
          ],
          "stream": "back"
        },
        "id": "back",
        "frigate": {
          "camera_name": "back",
          "url": "<IP_ADDRESS>"
        },
        "triggers": {
          "occupancy": false
        }
      },
      {
        "camera_entity": "camera.garage",
        "live_provider": "go2rtc",
        "go2rtc": {
          "modes": [
            "webrtc"
          ],
          "stream": "garage"
        },
        "id": "garage",
        "frigate": {
          "camera_name": "garage",
          "url": "<IP_ADDRESS>"
        },
        "triggers": {
          "occupancy": false
        }
      }
    ],
    "live": {
      "draggable": false,
      "lazy_load": true,
      "preload": true,
      "controls": {
        "timeline": {
          "mode": "above",
          "style": "ribbon",
          "media": "all",
          "show_recordings": true
        },
        "thumbnails": {
          "mode": "none"
        },
        "builtin": true
      },
      "transition_effect": "none"
    },
    "view": {
      "dark_mode": "off",
      "default": "live"
    },
    "dimensions": {
      "aspect_ratio_mode": "unconstrained",
      "aspect_ratio": "4:3"
    },
    "performance": {
      "profile": "high"
    },
    "menu": {
      "position": "bottom",
      "buttons": {
        "download": {
          "enabled": false
        },
        "camera_ui": {
          "enabled": false
        },
        "expand": {
          "enabled": false
        },
        "media_player": {
          "enabled": false
        },
        "substreams": {
          "enabled": false
        },
        "microphone": {
          "enabled": true
        },
        "timeline": {
          "enabled": false
        },
        "fullscreen": {
          "enabled": true
        },
        "cameras": {
          "enabled": false
        },
        "frigate": {
          "enabled": true
        }
      },
      "style": "outside"
    },
    "elements": [
      {
        "type": "custom:frigate-card-conditional",
        "conditions": {
          "camera": [
            "garage"
          ],
          "media_loaded": true,
          "view": [
            "live",
            "image"
          ]
        },
        "elements": [
          {
            "type": "custom:frigate-card-ptz",
            "orientation": "horizontal",
            "style": {
              "transform": "none",
              "left": "20px",
              "top": "100px"
            },
            "service": "amcrest.ptz_control",
            "data_left": {
              "entity_id": "camera.garage_amcrest",
              "movement": "left"
            },
            "data_right": {
              "entity_id": "camera.garage_amcrest",
              "movement": "right"
            },
            "data_up": {
              "entity_id": "camera.garage_amcrest",
              "movement": "up"
            },
            "data_down": {
              "entity_id": "camera.garage_amcrest",
              "movement": "down"
            }
          }
        ]
      }
    ],
    "timeline": {
      "style": "ribbon",
      "media": "snapshots",
      "show_recordings": false
    },
    "media_viewer": {
      "auto_play": "all",
      "snapshot_click_plays_clip": false,
      "controls": {
        "thumbnails": {
          "show_download_control": false,
          "show_favorite_control": false,
          "mode": "none",
          "show_details": false,
          "show_timeline_control": false
        },
        "timeline": {
          "show_recordings": true,
          "media": "all",
          "style": "ribbon",
          "mode": "above"
        },
        "next_previous": {
          "style": "chevrons"
        },
        "builtin": true
      },
      "draggable": false,
      "transition_effect": "none"
    },
    "media_gallery": {
      "controls": {
        "thumbnails": {
          "show_download_control": false,
          "show_favorite_control": false,
          "show_timeline_control": false
        },
        "filter": {
          "mode": "none"
        }
      }
    }
  }
}

update: figured out if I change aspect ratio from unconstrained to dynamic this issue goes away.
I'm going to close this for now since I'm not sure why I had even set unconstrained in the first place, and dynamic hasn't caused any other issues


added first version of createMergedQuery, parseQueryState

test after adding greenkeeper, jest and first version of createMergedQuery, parseQueryState

:tada: This PR is included in version 1.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Add users with pending requests

From https://github.com/orgs/redhat-cop/teams/container-cop/members

/lgtm


CWP: Contentreview integration

Make extension independent of model (also allow reviewing individual content blocks)
Ensure reports show all reviewable types
Ensure reminder email links work with all reviewable types

Contentreview hasn't yet started migrating to SS4.

Content review should be ready for integration testing on SS4 now, I'm going to triage as an enhancement

I have tested this at the page level and content review appears to be integrated. Reviewing Elemental Pages works as expected.
I suggest that this is raised as a separate feature request on the Content Review module itself. If we enable the review of individual content blocks, we will have to discuss how to approach nested content blocks and general data objects for instance.


Skip a failing Bagua test for manual optimization

What does this PR do?
Unblocks master.
 If yes, please list them.

Before submitting

[ ] Was this discussed/approved via a GitHub issue? (not for typos and docs)
[x] Did you read the contributor guideline, Pull Request section?
[x] Did you make sure your PR does only one thing, instead of bundling different changes together?
[ ] Did you make sure to update the documentation with your changes? (if necessary)
[ ] Did you write any new necessary tests? (not for typos and docs)
[ ] Did you verify new and existing tests pass locally with your changes?
[ ] Did you list all the breaking changes introduced by this pull request?
[ ] Did you update the CHANGELOG? (not for typos, docs, test updates, or minor internal changes/refactors)

PR review
Anyone in the community is welcome to review the PR.
Before you start reviewing, make sure you have read the review guidelines. In short, see the following bullet-list:

[x] Is this pull request ready for review? (if not, please submit in draft mode)
[x] Check that all items from Before submitting are resolved
[x] Make sure the title is self-explanatory and the description concisely explains the PR
[x] Add labels and milestones (and optionally projects) to the PR so it can be classified

Did you have fun?
Make sure you had fun coding 🙃
cc @borda @awaelchli @wangraying

@awaelchli
After an investigation of this problem, I found that this failure is caused by scikit-optimize library used by Bagua. In detail, scikit-optimize library currently uses an old builtin type of numpy (i.e., numpy.int), which is deprecated by numpy>=1.24.0.
To avoid this problem, users can choose to use numpy<1.24.0.
In the meantime, scikit-optimize is trying to fix this problem (PR: https://github.com/scikit-optimize/scikit-optimize/pull/1123). After that, users can use the updated scikit-optimize with numpy>=1.24.0 in Bagua.


Example code has extra 's' in cmdlet name

Causes error if you simply copy/paste.

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


Preform reservation

In this PR, we are working on the reservation pages

Create reducer actions

Add a reservation
delete a reservation

Add reducer to the  store
Add routes

Hello @Marcraphael12
Please fix the linters problems.
in line 24 package.json change this:
"tailwindcss": "^3.3.3"

for this
"tailwindcss": "^3.3.3",

You just need to add a (,) this will solve your problem.

Hello @Marcraphael12
As you can see here, I'm getting errors when I try to verify your changes.

That seems to be related to the slice you are currently using it in in MyReservation, NewCarReservation and NewReservation.
I think you are using the wrong slice (import { fetchVehicleDetails } from '../../redux/vehicle_details/vehicleDetailsSlice';) instead you should use VehicleSlice.
also in order to use VehicleDetailsSlice you should pass the id as arguments.
I'm not sure if a MyCarResertion page is necessary. What I understood was that you have to create a page to show a user's reservation and another to create the reservation.

Thanks for your thought @JavierAybar, i had to use Vehicle Slice instead of vehicleDetailsSlice. I was a bit confused about that. Thanks!

Hello @Marcraphael12
Please test your changes before request a new review

kindly don't import thing that you don't have in your application such as :
import '../../assets/style/NewReservation.css';

Nope... I'm getting this error and pain to fix it... let me push the version I use


[PROPOSAL] Redesign the SDKClient

What/Why
What are you proposing?
We should redesign the SDKClient from scratch, without referring to the existing client which is trying to do too many things and doing some of them poorly.
Our original vision (if we ever had one) for the SDKClient has morphed over time into an increasingly unmaintainable pile of spaghetti code.  It looks like we started wanting to use it just as a wrapper for the OpenSearch Client for Java (opensearch-java), but then:

We pivoted to use RestHighLevelClient for migration and added that
We tried to handle all the existing plugin calls but are using a different Client Interface than the plugins
We added support for transport actions to parallel what NodeClient does
We added support for remote/proxy transport actions on other extensions
We added support for RequestOptions (headers)

Core capabilities we need:

Handle transport actions (client.execute())
Provide a bridge for migration, internally using RestHighLevelClient calls and attempting to use the exact signature as existing clients including the same request/response classes

What users have asked for this feature?
#612, #729, #430, #616, TBD issue on serverless
What problems are you trying to solve?
Replace existing usages of NodeClient implementations in plugins, specifically

Transport Actions (client.execute()) internal to the extension
Index, Cluster, and Admin client actions that need to be handled by a REST client
Support for headers/authentication associated with the REST client

What is the developer experience going to be?
For migrating existing plugins, a minimal diff.
For developing new plugins, a set of clearly documented tools providing needed capabilities.
Why should it be built? Any reason not to?
Properly handle initialization
Properly handle resource usage/leakage
Properly handle multithreading
Permit DRY
What will it take to execute?
Re-thinking/re-scoping what the object does and does not do.
Any remaining open questions?

Should we be making the Java Clients part of the SDK API at all or just documenting how to use them? #612
Should we be initializing the (deprecated) RestClient as a singleton and how does that fit into the initialization sequence? #729
How do we avoid resource leaks if we do initialize it ourself?  We have a close method but that's pretty useless on our singleton.  Can we follow a try-with-resources model for all usages of the client?
Should we try to use the same client Request/Response classes that plugins do with the NodeClient, which differ slightly in some cases with the RestHighLevelClient?  (Would require a lot of translating one request class to another)

I am a big fan of deferring client responsibilities to opensearch-java and taking a runtime dependency on it.

I am a big fan of deferring client responsibilities to opensearch-java and taking a runtime dependency on it.

Can you be a little more specific what this means, @dblock ?  Currently we don't do much with the opensearch-java client other than initialize it from our settings.  We weren't planning on doing more; in fact, I'm proposing not depending on them or providing them at all and just giving a few lines of code a user can write themselves.


Gsoc2021 quad float

Full precision eval_sqrt as discussed in #80

Hi @sinandredemption it's always curiously failing on one single compiler in one individual test situation on GCC with std=gnu++14 or std=gnu++17. I am not sure if this is a tolerance issue or related to the extremely limited range of float's exponent.
Things you can try include:

Excluding those tests for float and covering up the problem for us to deal with later (OK by me for the moment).
Maybe adjust the exponent range in a line like this --- maybe try 0.1F instead of 0.2F.
This might ultimately be related to the similar root cause of #35. I can investigate.

One other problem I am encountering with cpp_quad_float<float> is that read from string and all kinds of other operations almost fail at the outset due to the extremely limited range of float's values of min_exponent10/max_exponent10, typically limited to 38 or so. The type has 30 decimal digits of precision and has almost overflowed/underflowed upon creation itself.
I would almost exclude (via static_assert) the existence of cpp_quad_float<float> for float having less than, say plus/minus 100 decimal range in its exponent.
Cc: @cosurgi

One other problem I am encountering with cpp_quad_float is that read from string and all kinds of other operations almost fail at the outset due to the extremely limited range of float's values of min_exponent10/max_exponent10

This is the problem with float_string_cvt.cpp that we have discussed before. The solution is to this is relatively straightforward (just replace the two eval_muls by powers of ten with a single one). Note that the max_exponent of cpp_quad_float<float> is not capped. So it seems to me correct to say that the problem also lies with the `float_string_cvt.cpp' code.

Also, in the above commit, I updated the test exponents to reflect the asymmetry of [max_exponent, min_exponent].

@ckormanyos Would you please look into why, of the all the tests, is test_exp.cpp failing (even though it was passing in the first few commits on this PR)?

@ckormanyos Would you please look into why, of the all the tests, is test_exp.cpp failing

I must admit, Fahad, there is an extremely unusual effect goinng on --- something I have not yet experienced.
@sinandredemption for some reason CI on your branch is attempting to run the _qf version of test_exp.cpp. However, I had only activated that in my branches. You did not pick that up from develop either. I studied this quite a bit, but could not solve the problem.
As a last resort, i have manually merged some of the needed stuff to get exponent tests working on quad into your branch in the hopes that you can move forward from there. I might have rattled the cage on some of your tolerances. if this goes green or fails, we must really look into those.
Let's see how CI cycles through...

Let's see how CI cycles through...

@sinandredemption so far so good. You must maybe replace a thing or two such as the following. I fiddled around with your new asymmetric handling of expenents in our homegrown add/sub/mul/div/sqrt tests. If (or when) this push cycles green, you need to re-try that.
I actually believe we ran into a Git glitch here, or we caused one via unanticipated automatic merge activities. But I was not able to reproduce the cause of these. So I did manual repairs.

The type has 30 decimal digits of precision and has almost overflowed/underflowed upon creation itself.

this is an interesting case. In that paper (which I sent last week) they were dicussing the possibility of creating a multi_float type with as many limbs as necessary to cover the entire exponent range. It looks like quad_float<float> is the first such scenario. For now I would simply relax the tests as much an necessary in order to not drop this type completely. Later it might be interesting to investigate the multi_float types having maximum number of limbs which cover whole exponent range.
And of course write in documentation that cpp_quad_float<float> is experimental and should not be used in production.
Or maybe disable it:

I would almost exclude (via static_assert) the existence of cpp_quad_float<float> for float having less than, say plus/minus 100 decimal range in its exponent.

This solution is also acceptable.

Force push via removing branch protection

See also: https://github.com/BoostGSoC21/multiprecision/issues/83#issuecomment-898895282


Changes to _script, LDA, yml

Added build_all_linux bash script to allow use on systems running bash shell (linux). It replicates the function of build_all.bat.
Added a line to _script/build_all.r to prevent errors by rmarkdown functions. Renames index.rmd to index.Rmd.
Added LDA.html to the _site.yml. Note: There appears to be two files named LDA.rmd. One in lesson 10, and one in lesson 9. Need to resolve?
Changed the name of a chunk in lesson 9 from import1 to import1abc. There were duplicate chunk names which prevented render_site() from finishing. As a note: all R chunk names across ALL rendered files have to have either no name, or a unique name (unique across all rendered files).

Thanks for adding Linux support.  I don't have a Linux box at home available to test this, but I've confirmed this doesn't break the windows version.


nixos/systemd-boot-builder: fix specialisations with special characters

Specilisation names can contain non-alphanumberic characters, for example if they have - as a seperator.
In this situation, the path for a specialisation will have its name at the end, and the old regex only checked for alphanumeric names, resulting in the rest of the code using a cut-off version of the full path, and erroring when it cannot find the file.
This also includes an additional check to make sure path parsing doesn't mess up.
For me specifically, this fixes an issue where a have a specialisation named nvidia-beta, that makes nixos-rebuild error out when installing a new boot entry, because it tries to remove the old boot entry named nvidia-beta, but the path is truncated to ...-specialisaion-nvidia.conf:
Traceback (most recent call last):
  File "/nix/store/38547vyyicbrldi0jiz2k0gr6h4hvm13-systemd-boot", line 546, in <module>
    main()
  File "/nix/store/38547vyyicbrldi0jiz2k0gr6h4hvm13-systemd-boot", line 529, in main
    install_bootloader(args)
  File "/nix/store/38547vyyicbrldi0jiz2k0gr6h4hvm13-systemd-boot", line 462, in install_bootloader
    remove_old_entries(gens, entries)
  File "/nix/store/38547vyyicbrldi0jiz2k0gr6h4hvm13-systemd-boot", line 354, in remove_old_entries
    os.unlink(disk_entry.path)
FileNotFoundError: [Errno 2] No such file or directory: '/boot/efi/loader/entries/nixos-generation-802-specialisation-nvidia.conf'

Description of changes

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

If we merge this PR, we should also revert https://github.com/NixOS/nixpkgs/pull/333952

I've made #334473 to address the boot counting compatibility, revert the faulty mitigation, and add a test. Sorry to take over like that!

Closing in favor of #334473!


Capy support

Make build example w/ https://github.com/capy-ui/capy
cc: @zenith391

This should be doable by copying the pixel data into a capy.ImageData struct
And then for rendering just use a window and capy.Image to show the image data


Handle Ecto lease count correctly when taking a lease

closes https://paradoxai.atlassian.net/browse/TD-1466

@ryanlchandler changes made. Please take another look. Thanks.


Postinstall

Based on https://github.com/snap40/serverless-aws-cdk/pull/4
Trying to make it possible to install this package from git, because I want to use the fixed version in that PR.
Unfortunately I'm stuck trying to build it with tsc:
cdk/toolkitStack.ts:5:32 - error TS2307: Cannot find module './environment' or its corresponding type declarations.

5 import { getEnvironment } from './environment'

I'm not really sure how I can install and test this module with that missing file

Hey @revmischa, sorry about this. I'm planning to cut release 0.6.0 in the next wee while which will contain the fix from #4, which should (hopefully) unblock you, but I've also pushed up a fix for this issue in #7 to remove the reference to the now-removed file

And subsequently more than happy to :+1: this PR too if you'd like :slightly_smiling_face:

Amazing thanks

Sorry, GitHub magically decided to close this based on #7 - reopening

I think this PR isn't really needed


Extra whitespace on cards in new gallery

The new links gallery is displaying cards with seemingly random heights. Some cards have a lot of extra whitespace.
Screenshot of the page rendered on Safari:

... but the same page rendered on Firefox looks better:

So maybe this is a Safari-specific bug?

Yes.  It looks like the Firefox display on Chrome, as well.  I'd say this is a Safari problem.  I'll see if I can diagnose it.

On my laptop's Safari browser, it looks like this:

What version of Safari are you using?  I am using Version 14.1.1 (156<IP_ADDRESS>.6, 15611).

Safari 14.0.3 (156<IP_ADDRESS>.7, 15610) on macOS Catalina 10.15.17

And my laptop is Catalina 10.15.7.

Have you done a force reload (Command+R) on the page?

Yes, no change.
I can try installing the latest Safari update.

You could, but I don't think that's the solution we want.  And the problem I have is that I cannot debug the issue.  😦

Just checked my other machine:
Safari 14.1.1 (156<IP_ADDRESS>.6, 15611) on MacOS Catalina 10.15.17
Here's what it looks like:

So it seems the problem is limited to an outdated version of Safari!
Maybe best to ignore it if there's no obvious fix.

But it doesn't seem that outdated.  😦
Does anyone else see this with their default browsers?

Hmm. I know basically nothing about CSS, but I see in the Safari release notes for version 14.1:

Added support for CSS individual transform properties: translate, rotate, and scale.

and I see what looks like a scale transform in our code here:
https://github.com/ProjectPythia/projectpythia.github.io/blob/0388706c249e1270e5b36b1c77e78f70eae19357/sphinx_pythia_theme/static/css/pythia.css#L2595
Could that have something to do with it?

The gallery page looks good on all of the browsers I have on my Windows 10 box: Chrome, Edge, Firefox, and Opera.
Also looks good on Firefox, using a laptop running Fedora 34.

@brian-rose:  I'm not entirely sure, but I don't think that those CSS classes are being used on the page, so I don't think that is it.  But I could be wrong.  I'm not a web developer, either.  ☹

Yes, I vote for closing. Perhaps we could put create a FAQ where this could be one of the potential issues a user might encounter?


How would you bind to a JWT claim with a dynamic name?

So I am trying to populate my Request object with a property from that Auth JWT token passed in the Authorization header. The challenge I have is that the name of the property is different per environment.
The token the property looks like so:-
"https://dev.my-domain.com/profileId": "a09d05bc-b07d-49f7-a166-b15789fe2a51" in my dev environment and it will have a different sub-domain in higher environments. Is there a way with FastEndpoints to populate my request model dynamically? Or do I need to roll my own custom model binder?
Cheers, and I love the framework by the way.

there's nothing built-in (even in asp.net) that can do it.
is there any possibility to break that in to two different claims at all?
i.e. domain & profileId?
that would be the easiest way to solve it.
if not, you'll have to write a custom model binder using an appropriate method explained here.

something like this should work:
interface IHasProfileId
{
    string? ProfileId { get; set; }
}

sealed class MyRequest : IHasProfileId
{
    public string? ProfileId { get; set; }
}

static class CustomBinding
{
    public static void Modifier(object req, Type _, BinderContext ctx, CancellationToken __)
    {
        if (req is IHasProfileId r)
        {
            r.ProfileId = ctx.HttpContext.User.FindFirst(c => c.Type.EndsWith("/profileId"))?.Value;
        }
    }
}

app.UseFastEndpoints(c => c.Binding.Modifier = CustomBinding.Modifier)

didn't test that out though. let me know if it's not working.

@dj-nitehawk - thanks so much for your swift response.

is there any possibility to break that in to two different claims at all?

Sadly not. I am using Auth0 as my IdP, and to add custom claims in the auth token seems to require namespacing.
The good news is your suggestion worked perfectly. Thanks so much.
👍👍👍👍👍👍👍👍👍👍


Track owner addresses in zos.network.json files

We should track the owner address of the App, Package, and Provider contracts deployed to a network in the zos.network.json file. This way, we can validate whether the from address actually has permissions to run a particular action, before attempting to actually execute it, and save the user from wasting time and gas into a failing tx.
This would complement the owner tracking of individual proxies in https://github.com/zeppelinos/zos/issues/3.

@mahim23 You've been approved! A WIP PR sooner than later helps, please let us know if you have any questions along the way 🙂 cc @spalladino

Hey! I have started looking through the codebase and tried a few things. Will clarify any doubts and send a WIP PR soon.

@spalladino I went through the code for the cli and the zos.network.json file is being handled by the ContollerFor network class. So, I should add the owner attribute to contract/app/package and change it in the networkFile attribute in the NetworkBaseController object, whenever it is being updated.
Is this approach right or am I missing something?

@mahim23 sounds good! The ZosNetworkFile is only handled by the NetworkBaseController (via its children NetworkApp and NetworkLib), so you should be able to injet these values on the fetchOrDeploy method.
The next step, which is actually checking that the owner matches the sender address before sending every tx in the future, is more tricky. The sender may not be available in the txParams, and we may need to query the node to actually get the default sender address. We'd also need to be careful to check that these addresses match in every public method of the NetworkControllers.
Let me know if this is clear!

Hey. Sorry I got caught up in something. Will send a PR by the end of the week.

Thanks @zachzundel! Do you have a rough estimate for a first WIP PR?

I think I should have one out by Thursday
On Tue, Nov 13, 2018 at 7:12 PM Scott Moore<EMAIL_ADDRESS>wrote:

Thanks @zachzundel https://github.com/zachzundel! Do you have a rough
estimate for a first WIP PR?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/zeppelinos/zos/issues/43#issuecomment-438511736, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AC8zTe_GRRkuvHpjpnJLMHtpKTBgOSG4ks5uu3wDgaJpZM4WL8pn
.

Hey @spalladino / @facuspagnuolo should I wait for #332 to get merged to work on this? What extra work is necessary after that point?

Hey @zachzundel! I think that #332 should be covering this issue, with no further work needed. @facuspagnuolo can you confirm? If that's the case, @ceresstation how should we handle this situation, in which we already have a PR for an issue by someone who didn't lock the issue in gitcoin?

Hmm, @spalladino we can split the bounty between both contributors or solely pay out the contributor whose PR is merged, what would you say a reasonable split would be?

I don't think that @zachzundel has sent a PR yet, but #332 has not been merged yet. @zachzundel, do you mind waiting to see if #332 gets merged before moving forward?

Works for me!
Zach
On Wed, Nov 21, 2018 at 05:57 Santiago Palladino<EMAIL_ADDRESS>wrote:

I don't think that @zachzundel https://github.com/zachzundel has sent a
PR yet, but #332 https://github.com/zeppelinos/zos/pull/332 has not
been merged yet. @zachzundel https://github.com/zachzundel, do you mind
waiting to see if #332 https://github.com/zeppelinos/zos/pull/332 gets
merged before moving forward?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/zeppelinos/zos/issues/43#issuecomment-440653462, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AC8zTYNLXDogwA9c9lioVin9Jx4IWe8Sks5uxU3RgaJpZM4WL8pn
.

Hey everyone. Hopefully I will finish #332 in a few days.
However, I don't think it will be enough to close this issue, as we only save owner yet, not use it to warn the user.
I hacked some code together to test whether this check works. Seems working for push command, though I'm not sure about where else we need that check. Happy to share my learnings with @zachzundel.


Java API document

Creating an online Java API document would be nice.
See  eg. http://owlapi.sourceforge.net/javadoc/

still work on it

Closing this issue since the Java API is not recommended anymore to users. See https://ontop-vkg.org/guide/getting-started.html#java-api-not-recommended.


bug: import with --collection still scans all collections

Describe the bug
When I import from Postgres using the --collection flag to only import a single table, Synth still scans all the tables. Since some of my other tables contains fields synth does not currently support, the import will fail.
To Reproduce
Steps to reproduce the behavior:

Command

RUST_LOG=synth synth import --from postgres://postgres:password@localhost:5432/postgres --collection table namespace

Logs
The logs will contain the following line for tables that were not in the --collection flag

Building ignored_table collection...

Expected behavior
I expect tables not in the --collection table to be ignored.
Environment (please complete the following information):

OS: MacOS
Version: 0.6.4

Additional context
See thread on Discord

I noticed this too. I assumed it was because it needs to access the catalog in order to find foreign keys etc. Would be useful to have fixed though.


entsoe plug-in throws an error while creating bdf = BeliefsDataFrame(...

Hi
I see an error when running the plugin (for day ahead prices and generation).
Run:  flexmeasures entsoe import-day-ahead-prices
Result:
[FLEXMEASURES] Task entsoe-import-day-ahead-prices encountered a problem: ""DataFrame should contain column named 'source'."". More details: {'src_module': '/usr/local/lib/python3.10/dist-packages/timely_beliefs/beliefs/classes.py', 'src_linenr': 910, 'src_func': 'init', 'src_code': 'raise KeyError("DataFrame should contain column named 'source'.")'}
This is happening with main repo: 41a89ad + Docker Image from the toy tutorial

Hi @tnlrgn2, let me have a look.

Can you tell me which timely-beliefs version you have?

I have followed the tutorial, but I wasn't able to reproduce it on my machine.

We are interested in reproducing this, though, because we saw this error happening in our FlexMeasures CI (Github Actions), but cannot reproduce on our dev laptops.
So all context information helps.
What OS are you on?
What is the docker image version? (then we can also derermine timely beliefs and other versions)

@tnlrgn2 I have answered this problem in #38.

Hi
Name: pandas
Version: 2.1.4
Name: entsoe-py
Version: 0.6.2
and latest flexmeasures docker container for tutorial + enstoe flexmeasures plugin v0.8 works as expected.
thanks.


Autobahn Test suite CI failures

AutobahnTestSuite is consistently flaky on OSX and occasionally on Win8, 10, 2012.
http://aspnetci/project.html?projectId=XPlat&testNameId=-7856782943084166959&tab=testDetails

cc @anurse

The Windows failures are because IIS appears to be flaky at forwarding WebSocket connections (see https://github.com/aspnet/AspNetCoreModule/issues/80). I'll disable IIS testing to get rid of that bit of flakiness.
The OSX failure seems very odd, so I'm getting a bit more information on it.
I'm really hesitant to skip these tests because they are the only way we ensure our WebSockets implementation conforms to the spec.

This issue was moved to aspnet/Home#2474


ERROR Facter::ClassDiscoverer - There is no module named Ol

On Oracle linux, running puppet bolt gives the error
ERROR Facter::ClassDiscoverer - There is no module named Ol
This is similar to https://tickets.puppetlabs.com/browse/BOLT-1545
Expected Behavior
There would be no error of this sort
Steps to Reproduce
Steps to reproduce the behavior:
$ bolt command run "hostname" --target localhost
This will produce
[date time] ERROR Facter::ClassDiscoverer - There is no module named Ol
Environment

Bolt Version - 2.8.0
Platform - Oracle Linux 6/7

Additional Context
It appears to be from os_detector.rb
In def create_hierarchy, addition of the following fixes it
    when :ol
      %w[EL]

This appears to be a facter issue in version 4.0.14. Perhaps this is not the place for this bug?

This seems to be due to Bolt bumping to use Facter 4 (facter-ng) which doesn't support oracle
linux. I'm not sure whether that functionality is planned or not (I can't find a ticket for it).
This code is probably being called when the analytics client is setup (we use Facter to determine the host OS). The workaround would therefore be to disable Bolt analytics.
We also should be catching errors that happen while setting up the analytics client, so that it can't prevent Bolt from functioning.

Ah none of those points are as dire as I thought. This doesn't apparently cause Bolt to fail to run and it seems to be fixed in 4.0.19. So in that case, we can just bump the version in Bolt.

I noticed 2.9.0 is out so perhaps 2.9.1 or 2.10.0?

Yeah, we should be able to get this in for next week's release.
In the meantime, you may be able to use /opt/puppetlabs/bolt/bin/gem install facter to upgrade facter in your install. I'm not 100% sure what will happen when you try to upgrade the package
later though...

@nicklewis  Can you point me to the file that has the facter version? I am unable to locate it.

It's a little roundabout.
puppet-bolt packages are built from the bolt-vanagon repo. That pulls in an artifact called bolt-runtime that contains most of its dependencies and comes from the puppet-runtime repo. The version of facter can be found in the file for the rubygem-facter component in that repo.

Oh! I'll leave it in your capable hands :)
Thank you very much for your help resolving this.


[ENH] Added ROCKAD anomaly detector to aeon

What does this implement/fix? Explain your changes.
Implemented the ROCKAD anomaly detector.
PR checklist
For all contributions

[x] I've added myself to the list of contributors. Alternatively, you can use the @all-contributors bot to do this for you.
[x] The PR title starts with either [ENH], [MNT], [DOC], [BUG], [REF], [DEP] or [GOV] indicating whether the PR topic is related to enhancement, maintenance, documentation, bugs, refactoring, deprecation or governance.

For new estimators and functions

[X] I've added the estimator to the online API documentation.
[ ] (OPTIONAL) I've added myself as a __maintainer__ at the top of relevant files and want to be contacted regarding its maintenance. Unmaintained files may be removed. This is for the full file, and you should not add yourself if you are just making minor changes or do not want to help maintain its contents.

For developers with write access

[ ] (OPTIONAL) I've updated aeon's CODEOWNERS to receive notifications about future changes to these files.

The CI tests are still failing. Can you check why this is the case?

@SebastianSchmidl should be better now

The issue might be caused by an overflow, see steps to reproduce :
from sklearn.preprocessing import PowerTransformer
transformer = PowerTransformer()
transformer.fit([[23.96], [23.98], [23.99]])

Gives
C:\Users\AntoineGUILLAUME\miniconda3\envs\watson\Lib\site-packages\sklearn\preprocessing\_data.py:3438: RuntimeWarning: overflow encountered in power
  out[pos] = (np.power(x[pos] + 1, lmbda) - 1) / lmbda
...
BracketError: The algorithm terminated without finding a valid bracket. Consider trying different initial points.

Changing the inputs slightly will cause the error to either appear or disappear, not familiar with the transformer, but the value lambda used in the power function seems to cause the overflow on some cases. Issue seems to be happening independently of the sklearn version.

The issue is caused by an overflow, see steps to reproduce :
from sklearn.preprocessing import PowerTransformer
transformer = PowerTransformer()
transformer.fit([[23.96], [23.98], [23.99]])

Gives
RuntimeWarning: overflow encountered in power: out[pos] = (np.power(x[pos] + 1, lmbda) - 1) / lmbda
...
BracketError: The algorithm terminated without finding a valid bracket. Consider trying different initial points.

Changing the inputs slightly will cause the error to either appear or disappear, not familiar with the transformer, but the value lambda used in the power function seems to cause the overflow. The issue seems to be happening independently of the sklearn version.

Thanks for the suggestions 👍, after testing a little I think the main reason seems to be the window size and stride applied to the input data which leads to the final bracket error.  I will add a fallback and a warning if this occurs.

Could we restrict the window_size and stride parameters in some way to prevent the error? E.g. limit window_size to >5, <=0.5*n_timepoints or similar?

Could we restrict the window_size and stride parameters in some way to prevent the error? E.g. limit window_size to >5, <=0.5*n_timepoints or similar?

As I implemented it now, power transform gets deactivated dynamically only if it leads to the error. I also added a ValueError if  n_neighbors are larger than the count of windows. The soft-deps test passes now. What do you think, is this sollution also fine?

If we decide to go with the current implementation, keeping an open issue with the context, the reproduction steps, and the related issues as backlog so we can keep an eye on it. Because as I understand it, the issue is not caused by our usage of the PowerTransformer.

If we decide to go with the current implementation, keeping an open issue with the context, the reproduction steps, and the related issues as backlog so we can keep an eye on it. Because as I understand it, the issue is not caused by the way we use the PowerTransformer right ? But if we can show that the error is bounded between some window size and stride values, we could enforce it.

As it turned out it was a problem with the way it was implemented. I now transpose the data before the transform is applied and now the error is gone, so I think its fine now.

Thank you for this contribution. Looks good already. I left some minor code improvements below.
Should we try to get rid of Pandas within the implementation? All things should be possible with NumPy. @baraline your take on this?

I removed all Pandas dependencies on the latest commit

Thanks I'll try to have a more throughout look at it this evening, but it looks fine at a glance


buildkite: Enable debug assertions in runtimes

Tested debug_assert in https://buildkite.com/oasisprotocol/oasis-core-ci/builds/14000.

I assume there is no noticable performance impact?

I didn't notice anything. SGX tests still run from 8-12min, one 18min, which is the same as before.


Refactor facet_grid and facet_wrap, customize linkxaxes, linkyaxes

 Implements #203
Example:
  Code
# ╔═╡ de609488-23c4-11ec-10b6-dbf425a6bbe8
begin
	using Pkg
	Pkg.develop("AlgebraOfGraphics")
	Pkg.add("CairoMakie")

	using AlgebraOfGraphics, CairoMakie
end

# ╔═╡ a8f90b4a-d4fb-48c7-8238-b039e4196dea
dta = let
	N = 100
	x = rand(1:10, N)
	(;
		x,
		y = x .+ 2 .* rand.(),
		c = rand(["a", "b", "c"], N),
		d = rand(["α", "β"], N)
	)
end

# ╔═╡ 3cbdd2e7-343e-4f62-9c35-582647cf1a5d
plt = data(dta) * visual(Scatter) * mapping(
		:x, :y, color = :c,
		layout = :c
)

# ╔═╡ ad438c1c-3478-41ad-a138-05ac1b5f3288
begin
	linkxaxes = false
	linkyaxes = false #false
	facet = (; linkxaxes, linkyaxes)

	draw(plt; facet)
end

https://user-images.githubusercontent.com/6280307/135813480-40f82f4b-0155-4ccb-8f43-630558a4eb68.mov
 I haven't changed the draw function yet.

I added facet as a keyword for draw and draw! and added examples to the gallery.
I updated the above example and screen capture accordingly.
I also split this PR into two. #286 merely refactors, and this PR adds new features. I hope this facilitates code review.
I am happy to revert the changes to draw and draw! for now if you want to keep thinking of the API. But I'd appreciate if the changes to the internal functions got merged soon.

I am not sure now if this fully addresse #203. I think @kleinschmidt's feature request is covered. AFAICT the current PR also matches what ggplot2 can do.
I think @piever commented on something different: row/col-wise linking which would be helpful in the following case:
df = (
	sepal_length = 1 .+ rand(100),
	sepal_width = 2 .+ rand(100),
	petal_length = 3 .+ rand(100),
	petal_width = 4 .+ rand(100)
)
xvars = Symbol.(["sepal_length", "sepal_width"])
yvars = Symbol.(["petal_length" "petal_width"])
layers = linear() + visual(Scatter)

data(df) * layers * mapping(xvars, yvars, col=dims(1), row=dims(2)) |> draw
end

which currently gives

but should rather give

I don't think ggplot2 can do this. I have code for this ready (https://github.com/greimel/AoGExtensions.jl/pull/1). I wonder if I should add this to this PR or if I should open another PR @piever?


Replace jshint/jscs with eslint

what is it ?
This replaces jshint and jscs with eslint.
why ?

Ramda also switched to eslint some time ago. So far nobody complained ;)
jscs will soon have its last release as they are merging with eslint
jshint is not very active and not even anywhere close to the abilities of eslint

how ?
The initial .eslintrc was created using polyjuice. The only change I did in the code was to add some missing indentation level. I suppose there are quite some eslint settings that could be added over time. We could even use the same config as Ramda if that is desirable? I also cannot guarantee that the tool I used did indeed pick up everything or does cover everything but the code is accepted as it is and the settings seem to make sense to me. All further adjustments could be made over time I suppose.
Looking forward to hear what folks think about this ;-)

Removed the added indentation in index.js and disabled the indent config in .eslintrc

I was unaware of the ESLint–JSCS merge. Thanks for informing me of this, and for submitting this pull request. :)

@davidchambers well the config is pretty tiny now ;-} I do think it would sense to add more rules over time. Also I think that pretty much all rules from the possible errors set should be just fine. Because I'm pretty sure nobody wants unnecessary semicolons, invalid regexes or the other stuff that is just plain "wrong".

I'd like to close this pull request and create a separate repository for our ESLint configuration, to allow it to be shared by the various Sanctuary projects.


Add option to set description of default stream groups in command

8bea18a made me realise I should start working on the registration flow tests for default stream group.

Great, merged, thanks @hackerkid!  And yeah, adding a registration flow test or two for this would be awesome :).


[mtproto-storage-fs] Unexpected end of JSON input

SyntaxError: ./storage.json: Unexpected end of JSON input
    at JSON.parse (<anonymous>)
    at /.../node_modules/jsonfile/index.js:36:18
    at /.../node_modules/graceful-fs/graceful-fs.js:78:16
    at FSReqWrap.readFileAfterClose [as oncomplete] (fs.js:437:3)

It happens, when several clients use the same storage.

This is fixed in last version, but this version not in npm still

I am glad that this version, at least, exists (wherever it is) 😄

@irudoy fixed version now published in npm, try<EMAIL_ADDRESS>


ModuleNotFoundError: No module named '_speech_py_impl' (again)

Hi,
I'm testing Azure Cognitive Services Speech-to-text with Python following the "Get started" HERE.
The error happens at first line when trying to import the library: import azure.cognitiveservices.speech as speechsdk
I already installed the library with pip install azure-cognitiveservices-speech, it is listed among all other libraries, the file "speech_py_impl.py" is there at site-packages folder (I even tried to put this folder on system path - no change).
Also installed last version of Microsoft Visual C++ Redistributable (here) as suggested on THIS similar issue.
I'm using Python 3.7 x64 on Windows 7, and SDK version is: azure-cognitiveservices-speech==1.19.0
For development I'm just using Notepad++ and other simple text editors, where I can run many other Python apps (including other Cloud APIs I'm testing to choose one for production).
I already generated the key, but the error happens even before the authentication step.
Something I'm missing to setup the environment?
Thanks and regards.
Full error message:
Traceback (most recent call last):
  File "D:\Python\Anaconda3\lib\site-packages\azure\cognitiveservices\speech\speech_py_impl.py", line 27, in swig_import_helper
    return importlib.import_module(mname)
  File "D:\Python\Anaconda3\lib\importlib\__init__.py", line 127, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
  File "<frozen importlib._bootstrap>", line 1006, in _gcd_import
  File "<frozen importlib._bootstrap>", line 983, in _find_and_load
  File "<frozen importlib._bootstrap>", line 967, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 670, in _load_unlocked
  File "<frozen importlib._bootstrap>", line 583, in module_from_spec
  File "<frozen importlib._bootstrap_external>", line 1043, in create_module
  File "<frozen importlib._bootstrap>", line 219, in _call_with_frames_removed
ImportError: DLL load failed: Não foi possível encontrar o procedimento especificado.

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "D:\Dropbox (AP)\DELPHI\Python\VoiceRecognition\MicrosoftAzure\AzureTeste1.py", line 3, in <module>
    import azure.cognitiveservices.speech as speechsdk
  File "D:\Python\Anaconda3\lib\site-packages\azure\cognitiveservices\speech\__init__.py", line 8, in <module>
    from .speech import *
  File "D:\Python\Anaconda3\lib\site-packages\azure\cognitiveservices\speech\speech.py", line 8, in <module>
    from . import speech_py_impl as impl
  File "D:\Python\Anaconda3\lib\site-packages\azure\cognitiveservices\speech\speech_py_impl.py", line 33, in <module>
    _speech_py_impl = swig_import_helper()
  File "D:\Python\Anaconda3\lib\site-packages\azure\cognitiveservices\speech\speech_py_impl.py", line 31, in swig_import_helper
    return importlib.import_module('_speech_py_impl')
  File "D:\Python\Anaconda3\lib\importlib\__init__.py", line 127, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
ModuleNotFoundError: No module named '_speech_py_impl'

@mrscarvalho Speech SDK supports Windows10 or later OS versions. The problem might be due to Windows 7 version you are using. Do you have possibility to upgrade the OS ?

Hi @jhakulin, thank you for the reply!
I just tested on Windows 10 and it worked (although a very different environment than my developing machine). :-)
But it limits a bit. Although officially ended support by Microsoft Win 7 is still used around, especially in companies that have a longer time life for software. I'm also testing the ComputerVision SDK and it works well on Win 7.
I also tested on Windows Server 2016, and I had the same error. It is annoying because I was planning to create a server application to process speech-to-text requests from our team preventing the need to install libraries for all users (all our servers have this OS). We have many Python apps running on them and never had compatibility issues (Python 3.7).
Anyway, I’ll keep exploring the SDK, that seems to be doing a good job (although our very difficult contents to convert because background music and fast paced speech). ;-)

@mrscarvalho On Windows Server 2016, have you also installed  Microsoft Visual C++ Redistributable package ?
How about do you have several Python versions installed on the system ? Sometimes several Python versions have conflicted and caused failures like this. Please check the older closed items in this samples repo by searching closed items with ModuleNotFoundError, you will find issues like this: https://github.com/Azure-Samples/cognitive-services-speech-sdk/issues/792

Hello @jhakulin, thank you for the reply!
After spending a couple of hours fighting with Azure Speech installation not working on a Win 10 computer and still having the “ModuleNotFoundError: No module named '_speech_py_impl'” error, I realized that the MS Visual C++ 2015-2022 Redistributable should match PYTHON’s bits, not Windows bits...
So, after installing the x86 version of the C++ redistributable it worked on that Win 10, and also in Windows Server 2016 with Python 3.7.2 (32 bits) we use.
Anyone having the same issue, check the Microsoft Visual C++ 2015-2022 Redistributable that matches to your PYTHON’s bits:
https://aka.ms/vs/17/release/vc_redist.x64.exe -> for PYTHON 64 bits!
https://aka.ms/vs/17/release/vc_redist.x86.exe -> for PYTHON 32 bits!
PS: It didn’t work on Windows 7 anyway, even after the update. :-(

@mrscarvalho Thanks for this information, this is also something we want to update in our documentation to be more clear.


consider adding fuzz testing with jazzer.js

https://github.com/hchiam/learning-jazzer

or maybe just yarn add @jazzer.js/jest-runner?


Require napalm-base>=0.23.0 for TRACEROUTE_VRF arg

<!-- Make sure you have read http://napalm.readthedocs.io/en/latest/contributing/index.html --!>

Coverage remained the same at 80.414% when pulling e78f5de0f069095f31d50b18b4b07cf9a70828b3 on nb-0.23.0 into dd1228cdd600fe7f8263b74fa10dfce3aba5bd45 on develop.

+1


Error in the powershell code

There is an error in provided code:
Add the SSH key to the VM
Add-AzVMSshPublicKey -VM $VirtualMachine  -KeyData $sshPublicKey
-Path "/home/azureuser/.ssh/authorized_keys"
If we are using other username than "azureuser" this will return error.
As there is a variable with username:
$UserName='demouser'
The code should be:
Add the SSH key to the VM
Add-AzVMSshPublicKey -VM $VirtualMachine  -KeyData $sshPublicKey
-Path "/home/$UserName/.ssh/authorized_keys"

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

ID: 675c83b0-1d6e-ca23-5434-0618c49d07b8
Version Independent ID: 8743626e-03a7-0fe8-c856-ecdf5bf66486
Content: Create Linux VM with PowerShell in Azure Stack Hub - Azure Stack Hub
Content Source: azure-stack/user/azure-stack-quick-create-vm-linux-powershell.md
Service: azure-stack
Sub-service: azure-stack-hub
GitHub Login: @BryanLa
Microsoft Alias: bryanla

@macgyver2004 - apologies for the delay, and thanks very much for your feedback. This issue has been resolved. The page will republish around 10am Pacific today.
#please-close


strip druid metric type when value is postagg

When some blank character ahead druid metric type specified as ‘postagg’, like :

Some error arised when explore it, like:

Personally would prefer getting that right on the way in as the data is saved instead of adding .strip() everywhere we use user inputed data in the codebase.

I agree with @mistercrunch, we should ensure that all inputs are stripped of leading and trailing whitespace when saving the form data.

Fine, Thx @mistercrunch  @john-bodley


types inside function error

Bug Report

🔎 Search Terms
types inside function error

🕗 Version & Regression Information

This is a bug
This is the behavior in every version I tried, and I reviewed the FAQ for entries about this

⏯ Playground Link
https://www.typescriptlang.org/play?ts=4.4.0-beta#code/MYewdgzgLgBAhjAvDA3gKBvAXDKAnAVwFMAaDGAIxwCYyBfAbjTSIA8AHEPWKAT3aIw8SVOQDaAayK8YASzAwpvEADNc-IqvgBdHCiFEIIADYA3IjgAUpuMat8BWuAEpJ07c6QA+GKZCyAEwYYRjQ6ZlBIWCohERQ6eAh4MF4mNEjoRWlqHCUtB001BGQAchLE5NT08EybY2IcmEsCp1cWtSVqbTiEuCS4FKYKN14ugDo8QxNza1sG5zTLS09EH3RMDNglXOl8jScRMoqBqo2a2DriHHb4MRulbuR448HyYYeJqbMiWfqiBbeI20nyM31+8yYmHe2WBk1BM0u-0hlBG4zh0x+iOoALozmWDCAA

Playground link with relevant code
💻 Code
const a = {
  a: true,
  b: 2,
};

export type r = {
  [key in keyof typeof a]: { resolve: (val: typeof a[key]) => void; };
}

const b: r = {} as any;

const key2: keyof typeof a = '' as any;
const value2: typeof a[typeof key2] = {} as any;
b[key2].resolve(value2);

(() => {
  const key: keyof typeof a = '' as any;
  const value: typeof a[typeof key] = {} as any;
  b[key].resolve(value); // errors
  b[key].resolve(value2);
  b[key2].resolve(value); // errors
  b[key2].resolve(value2);

 b[key].resolve(value as typeof value2); // works
})();

🙁 Actual behavior

As you can see in the image, for some reason when inside a function or another scope it is giving an error according to the types.

Argument type number | boolean is not assignable to parameter type typeof a["a"] & typeof a["b"]
🙂 Expected behavior
That there are no errors.

(typeof a)["a"] & (typeof a)["b"] is just number & boolean which is a type with no values - no value can be both a number and boolean. That behavior is expected.
I'm not sure why you only see it for a subset of locations - in the playground, all locations fail equally.

I see errors on all call sites in VS Code as well.


Sort searchable attributes

Like what was done in #943, it would be good if searchable attributes were also sorted in lexicographical order.

I will do it 👍

The order of the searchable attributes is very important, it indicates the order of importance of the attributes in a document, we must not reorder them!

@Kerollmops it doesn't matter anymore, since field pos and indexed_fields are two different things. The later is represented with a set anyways

Yeah, but we must display the searchable attributes in the order the engine treat them.

@MarinPostma, As a user how can I change the attribute position for the criterion attributes? 🤔

Can you?

Updating a document to change the attributes order will not change the engines behavior as it will keep the global attributes order (the searchable attibutes setting), but updating the order of the searchable attributes setting must and will change the behavior of the attributes criterion.

This needs to be tested and fixed, I will create an issue


ARROW-8147: [C++] add GCS library to ThirdpartyToolchain

This adds the GCS library to ThirdpartyToolchain, and enables the
library in some of the builds. A future PR will add the library to the
package manager configuration files and to any remaining builds where it
needs to go.

The "Java JNI" test passes locally with archery docker run debian-java-jni.  To me, it seems unlikely that my PR caused the failure, but it is possible I do not understand something here.
The appveyor failure seems to be a race condition in a test. The test has a SleepFor(0.01) which is generally a code smell.
In any case, if I am wrong do let me know and I will investigate / change things as needed.

/cc: @kou or @pitrou are you the right folks to look at this PR?  If so, please take a look when you have a chance.

Oh, arm64 build on Travis CI is failed:
https://app.travis-ci.com/github/apache/arrow/jobs/541265250#L2124
FAILED: CMakeFiles/crc32c.dir/src/crc32c.cc.o

/usr/bin/c++   -Iinclude -I/build/cpp/crc32c_ep-prefix/src/crc32c_ep/include -fdiagnostics-color=always -ggdb -O0 -g -fPIC -Wall -Wextra -Werror -fno-exceptions -fno-rtti   -fdiagnostics-color=always -ggdb -O0 -g -fPIC   -std=gnu++11 -MD -MT CMakeFiles/crc32c.dir/src/crc32c.cc.o -MF CMakeFiles/crc32c.dir/src/crc32c.cc.o.d -o CMakeFiles/crc32c.dir/src/crc32c.cc.o -c /build/cpp/crc32c_ep-prefix/src/crc32c_ep/src/crc32c.cc

In file included from /build/cpp/crc32c_ep-prefix/src/crc32c_ep/src/crc32c.cc:11:

/build/cpp/crc32c_ep-prefix/src/crc32c_ep/src/./crc32c_arm64_linux_check.h: In function 'bool crc32c::CanUseArm64Linux()':

/build/cpp/crc32c_ep-prefix/src/crc32c_ep/src/./crc32c_arm64_linux_check.h:36:37: error: the address of 'long unsigned int getauxval(long unsigned int)' will never be NULL [-Werror=address]

   36 |   unsigned long hwcap = (&getauxval != nullptr) ? getauxval(AT_HWCAP) : 0;

      |                          ~~~~~~~~~~~^~~~~~~~~~

cc1plus: all warnings being treated as errors

Forgot to update ARROW_THIRDPARTY_DEPENDENCIES in this pull request.

Oh, arm64 build on Travis CI is failed:
https://app.travis-ci.com/github/apache/arrow/jobs/541265250#L2124

That seems to be fixed upstream in crc32c, but the fix is not in any tag or release. Would you be Okay with me pinning to a SHA of this dependency?  Can I reproduce these builds myself to verify I have fixed such problems?
Also, the PR is closed, and it does not look like I can reopen it.  Should I simply create a new one?


MS prefix

Why do you use MS prefix everywhere?
Is this repo sponsored by Microsoft? ;-)

:P Movie Search ™


Add a today option for date wrangles and for create column-value

Add a parameter for all of the date wrangles that allows to use today as a value and returns today's date
Add also a today for create.column that returns today (datetime datatype)
wrangles:
  - create.column:
      output: Date Now
      value: today()

Also to do:
Allow create.column to also use the placeholder values that generate random data from the test connector.
e.g.
wrangles:
  - create.column:
      output: column name
      value: <sentence>

Added the following to create column:

_generate_cell_values function from connector-test to create column.
create.column can have today's date (normalized) and random dates from a given range
allow to give a date as value (normalized)

wrangles:
  - create.column:
      output: Random Date
      value: <2022-01-01 to 2022-12-31>
  - create.column:
      output: Today
      value: <date_today>
  - create.column:
      output: Year End
      value: <date 2022-12-31>

Data Sample:

col
Random Date
Today
Year End

Hello
2022-07-03 19:06:37
2022-10-03 00:00:00
2022-12-31 00:00:00

Fey
2022-03-29 07:33:57
2022-10-03 00:00:00
2022-12-31 00:00:00

Will look into adding time to the date. Getting a nanosecond error


Ask about ability to pay in all circumstances

If the tenant can pay rent and court costs, then tenant has right to pay to stay.
Terry said landlords can get around this by having another reason for eviction.

used existing will_offer_payment question (added line breaks to get markdown bullet points)
added will_offer_payment to rent_defenses_section eviction order
added a paragraph to what_you_can_do_at_the_hearing information collection
This doesn't address adding ability to pay to various defenses (as noted by attorneys)


cmd/server: write initial api routes for ICL files

Issue: https://github.com/moov-io/imagecashletter/issues/56

Codecov Report

Merging #57 into master will decrease coverage by 2.12%.
The diff coverage is 58.96%.

@@            Coverage Diff             @@
##           master      #57      +/-   ##
==========================================
- Coverage   93.38%   91.25%   -2.13%
==========================================
  Files          32       36       +4
  Lines        3855     4107     +252
==========================================
+ Hits         3600     3748     +148
- Misses        159      247      +88
- Partials       96      112      +16


canarymod not loading plug-in

Received the following error while loading the latest (version d79b26a0cb2b7fbe8e9ce628617d5436dd5f8465 from 2015-02-26)
[03:39:59] [CanaryMod] [ERROR]: Exception while enabling plugin: CanaryConsole
net.canarymod.exceptions.PluginLoadFailedException: Failed to load plugin
        at net.canarymod.plugin.lifecycle.JavaPluginLifecycle._load(JavaPluginLifecycle.java:36) ~[canary.jar:1.8.0-1.2.0]
        at net.canarymod.plugin.lifecycle.PluginLifecycleBase.load(PluginLifecycleBase.java:73) ~[canary.jar:1.8.0-1.2.0]
        at net.canarymod.plugin.DefaultPluginManager.enablePlugin(DefaultPluginManager.java:73) ~[canary.jar:1.8.0-1.2.0]
        at net.canarymod.plugin.DefaultPluginManager.enableLatePlugins(DefaultPluginManager.java:110) [canary.jar:1.8.0-1.2.0]
        at net.canarymod.Canary.enableLatePlugins(Canary.java:296) [canary.jar:1.8.0-1.2.0]
    at net.minecraft.server.dedicated.DedicatedServer.i(DedicatedServer.java:330) [canary.jar:1.8.0-1.2.0]
    at net.minecraft.server.MinecraftServer.run(MinecraftServer.java:450) [canary.jar:1.8.0-1.2.0]
    at java.lang.Thread.run(Thread.java:745) [?:1.7.0_75]
Caused by: java.lang.ClassNotFoundException: The     class com.macuyiko.canaryconsole.MainPlugin could not be found!
        at net.canarymod.CanaryClassLoader.findCI htink lass(CanaryClassLoader.java:56) ~[canary.jar:1.8.0-1.2.0]
        at java.lang.ClassLoader.loadClass(ClassLoader.java:425) ~[?:1.7.0_75]
        at java.lang.ClassLoader.loadClass(ClassLoader.java:358) ~[?:1.7.0_75]
        at net.canarymod.plugin.lifecycle.JavaPluginLifecycle._load(JavaPluginLifecycle.java:26) ~[canary.jar:1.8.0-1.2.0]
        ... 7 more

I want to be careful and say, I'm not demanding a fix or anything like that. I'm just letting you know that perhaps something isn't working the way you think it is. Are you testing on Ubuntu? I am running Ubuntu 14.04 on the Amazon server. I'll confess that I haven't been trying on Windows, since it is tangential to my goal. (Theoretically I could run a Windows server on Amazon, but I am hoping that can stay as Plan B. I find Linux much easier to work with, particularly if I'm limited to the command line.)
From my perspective, the version I used in the video was working great. Honestly I am perfectly happy running with the PlaceInCanaryMod version - I haven't combed through the code enough to recognize how the ServerPluginInterpreter is better, faster, stronger.

I think the reason might be because you are using Java 7. This is probably also the reason why you get the compilation errors for the editor.
The instructions I sent before were executed on Ubuntu 14.04. What do you get for java -version? Mine gives:
webmaster@tengwesta:~$ java -version
java version "1.8.0_25"
Java(TM) SE Runtime Environment (build 1.8.0_25-b17)
Java HotSpot(TM) 64-Bit Server VM (build 25.25-b02, mixed mode)

I'll look into recompiling the plugin with Java 7 if needed...

I've just commited a new version compiled with Java 7. Can you try again with that one?

Yep, you got it. Nice work, my friend. I just did
sudo apt-get install default-jre

on Ubuntu, which apparently is still Java 7. I can just install 8 if that's easier, but initializing the server works great! I actually made a little startup script because I was getting tired of typing it in every time, ha ha. https://github.com/gamestartdev/canarymod-deploy-here


[Feature Request] Headless chrome / Heroku integration

Is it possible to make this run with the headless version of chrome, and possibly host this on Heroku?

Yes you can set "headless": true and headless_login": true in the config. I'm not familiar with Heroku but I don't see any reason it wouldn't work.

 Someone might figure it out


/vouchers fix, confirm vouchers, etc

at the moment there are none

Domino's comment:
"This has to wait for Peter. I suggest hiding this panel for now."


Licenses are missing copyright information

Hi,
both the Apache 2 and the MIT license are missing the copyright information.
For the Apache 2 license, the copyright placeholder is not filled in: https://github.com/knurling-rs/defmt/blob/main/LICENSE-APACHE#L189
For the MIT license, the copyright part at the top (Copyright <YEAR> <COPYRIGHT HOLDER>, see here) is missing.

The copyright line in that part of the Apache-2.0 licence is a template and is not supposed to be modified in the licence file itself. If you changed it, then you have a) modified the licence, and b) are telling anyone who reads your modified copy of the licence that they must put your copyright in their files when they wish to release it under your modified Apache-2.0 licence.
The copyright line is missing in the MIT file and we should fix that. I note it is also missing from the copy at GitHub.com/rust-lang so maybe we copied their file.

You are correct, the Apache-2.0 license file should not be modified, my bad. Without any form of copyright notice, I'm unsure how the copyright is assigned when using this project under the Apache-2.0 license.
The Rust repository has a COPYRIGHT file.

We do have https://github.com/knurling-rs/defmt/blob/f5315e098129c372d32ed6fc0fe309d6302488ee/LICENSE-MIT#L1 is this not enough?

I added that five days ago :)

Perfect


Feature request-switch with threshold

Hi,
I love the plug-in thanks for developing it. One thing that I could see being helpful is a switch that flicks on when a threshold is exceeded. What I’d like to do is see when we are generating more than say 1kw and send a notification. Now I guess I could do this by setting an automation in Eve that looks at the Lux value and then flicks a dummy switch on but it would be neater if this was available within this plugin.

I'm going to leave that to the automation capabilities in Homekit (I prefer Home+ 4 instead of Eve for creating automations).


Adjust navigation menu for the survey response page

### Tasks
- [x] Change "response home" to "User Guide"
- [x] Remove the sankey diagram from response home/user guide page
- [x] Add generic step 1, step 2, step 3 text to the user guide page so Hazel can add more detail later
- [x] Delete the "Response to:" in the page header
- [ ] Move the survey ID to the bottom of the page

@hazelshapiro please confirm this looks good. I'm putting v1.4.0 in QA.


Pan/Zoom Refactor

Currently in the FE OSD spike, we have hacky PoC code to prove that OSD and custom  layer can zoom/pan in sync. However, we should clean up the code for production quality. Such as figuring out the best way to accomplish the code change and perhaps changing function signature from multiple arguments to object for DX
DoD:

Refactor code and maintain the same functionality

closing this in favor of #930


SIP over WebSocket appears to disconnect every 15 minutes

Hello, we are currently evaluating Baresip for use in a product, and we have run into an issue where the SIP Websocket connection disconnects every 15 minutes.
After experimenting with a variety of configurations and performing a tcpdump, we suspect there may be an issue in libre - specifically the TCP Idle Timeout timer in sip/transp.c:1074 never seems to be reset. Maybe I am missing something, but it seems to me that this is supposed to be reset by the conn_keepalive_handler, but that is never started when the SIP transport is set to WS or WSS in sip/keepalive.c (Also even if you enable it for WS/WSS, it's dependent on outbound being configured, and requires server re-configuration...)
I believe this can be patched by resetting the timer in the websock_recv_handler similar to how it is reset in the tcp_recv_handler, or alternatively just disabling the original timer, since the Websocket has its own keepalive.

Please correct me if I am missing something. Thank you.

Can you try https://github.com/baresip/re/pull/1075?

Should we fix this for v3.10?

Thank you for the prompt response.
I am testing with the branch now.