Fixed RI ELR messages failure

This PR fixed RI ELR message failure with missing OBR-4.1, 2, 3, 7.  The reason for failure is that the latest main livd table includes the test with "TestKit Name ID= Flowflex COVID-19 Antigen Home Test_ACON Laboratories, Inc" with all columns needed.  The cause is that two duplicated tests still remain in the livd-suplemental.csv table.  Therefore, the test is not unique and those fields are not populated.
Test Steps:

./gradlew clean package
./prime test

Changes

Include a comprehensive list of changes in this PR
(For web UI changes) Include screenshots/video of changes

Checklist
Testing

[x] Tested locally?
[x] Ran ./prime test or ./gradlew testSmoke against local Docker ReportStream container?
[ ] (For Changes to /frontend-react/...) Ran npm run lint:write?
[ ] Added tests?

Process

[ ] Are there licensing issues with any new dependencies introduced?
[ ] Includes a summary of what a code reviewer should test/verify?
[ ] Updated the release notes?
[ ] Database changes are submitted as a separate PR?
[ ] DevOps team has been notified if PR requires ops support?

Linked Issues

Fixes #issue
https://app.zenhub.com/workspaces/onboarding--operations-6166edbd409257001e09f1b2/issues/cdcgov/prime-reportstream/7484

To Be Done
Create GitHub issues to track the work remaining, if any

#issue

Specific Security-related subjects a reviewer should pay specific attention to

Does this PR introduce new endpoints?

new endpoint A
new endpoint B

Does this PR include changes in authentication and/or authorization of existing endpoints?
Does this change introduce new dependencies that need vetting?
Does this change require changes to our infrastructure?
Does logging contain sensitive data?
Does this PR include or remove any sensitive information itself?

If you answered 'yes' to any of the questions above, conduct a detailed Review that addresses at least:

What are the potential security threats and mitigations? Please list the STRIDE threats and how they are mitigated

Spoofing (faking authenticity)

Threat T, which could be achieved by A, is mitigated by M

Tampering (influence or sabotage the integrity of information, data, or system)
Repudiation (the ability to dispute the origin or originator of an action)
Information disclosure (data made available to entities who should not have it)
Denial of service (make a resource unavailable)
Elevation of Privilege (reduce restrictions that apply or gain privileges one should not have)

Have you ensured logging does not contain sensitive data?
Have you received any additional approvals needed for this change?

@oslynn Do you need to delete both entries in the supplemental table?

@oslynn are you actively working on this? Should I move it to the in progress column?


fix/date-is-not-datetime

Please review before merging.

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

GitHub release
v3.14.0

Your semantic-release bot :package::rocket:


client version 1.41 is too new

time="2022-07-20T02:33:08Z" level=error msg="failed to get the list of containers" err="Error response from daemon: client version 1.41 is too new. Maximum supported API version is 1.40"
time="2022-07-20T02:33:08Z" level=error msg="poll: docker version" err="failed to execute request: Error response from daemon: client version 1.41 is too new. Maximum supported API version is 1.40"
time="2022-07-20T02:33:08Z" level=error msg="poll: docker container list" err="failed to execute request: Error response from daemon: client version 1.41 is too new. Maximum supported API version is 1.40"
time="2022-07-20T02:33:08Z" level=error msg="poll: docker disk usage" err="failed to execute request: Error response from daemon: client version 1.41 is too new. Maximum supported API version is 1.40"
time="2022-07-20T02:33:08Z" level=error msg="poll: docker log size" err="failed to execute request: Error response from daemon: client version 1.41 is too new. Maximum supported API version is 1.40"
time="2022-07-20T02:33:08Z" level=error msg="failed to listen to docker events" err="Error response from daemon: client version 1.41 is too new. Maximum supported API version is 1.40"
time="2022-07-20T02:34:08Z" level=error msg="failed to listen to docker events" err="Error response from daemon: client version 1.41 is too new. Maximum supported API version is 1.40"

Hello
same issue here with 1.41
time="2022-08-07T17:25:36Z" level=error msg="failed to listen to docker events" err="Error response from daemon: client version 1.41 is too new. Maximum supported API version is 1.39"


Changing conda environments with spyder preferences does not work.

Hello,
my goal is to use conda environments in Spyder.
The conda env work fine in Jupyter Lab.
OS: Windows
Python 3.9.7
conda 4.12
I read the doc here:
https://docs.spyder-ide.org/current/faq.html
I did install Spyder with miniconda in base and installed spyder-kernels in the env.
I start Spyder in base and then try to change the env with tools\python interpreter\preferences.
I cannot change the env and get the error.
The code execution cannot proceed zlib.dll was not found.
Reinstalling the program may fix this problem.
Then afterwards I get the message "Invalid file path"
I reinstalled spyder and the file path is correct, I guess.
There are two similar issues here, describing maybe similar problems, but I am missing a solutions?
e.g. https://github.com/spyder-ide/spyder/issues/17380
Is this a version conflict with Python > 3.9
Do I need to install additional packages.
My assumption was that when I install spyder with conda I get all the requirements satisfied.
No fix in the doc.
How can I solve the errors?

Hi @RolfChung thank you for the feedback! What version of Python are you using in the created env? Could be Python 3.10.0?
Could you create a new env using the conda-forge channel? (I think Python 3.10.2 is not available in the default anaconda channel)
Let us know!

This issue is a duplicate of #17380 and we'll try to fix it in a future version (we still don't know why it happens).


Simplify the useAppDispatch definition

This PR is related to issue #1925 and shortens the definition of useAppDispatch in the documentation.

Yay, thanks! :)


Update username algo

Cleans up username algorithm
introduces suggestedUsername field on User, as a way to pass github usernames (or other ones in the future
looks at group slug table for conflicts.

fixed. merging now.


Metadata API deploy client timeout error after 2 hours, But deployment is running and successful on salesforce end.

Summary
The customer is deploying through SFDX where deployment is successfully running in salesforce, but SFDX is failing with the below error which is causing the time out after 2 hrs. and breaking the CI/CD.
ERROR running force:mdapi:deploy: Metadata API request failed: The client has timed out.
Steps To Reproduce:

CI/CD pipeline.
We need to deploy through SFDX.

Expected result
SFDX shouldn't fail with the timeout error after 2 hrs and break the CI/CD
Actual result
SFDX is failing with the timeout error which is causing the time out after 2 hrs and breaking the CI/CD pipleline.
System Information

Which shell/terminal are you using? (e.g. bash, zsh, powershell 5, powershell 7, cmd.exe, etc.) - gitlab

Hi @Saipraneeth-hub , can you fill the issue template with the requested info? See: https://github.com/forcedotcom/cli/blob/main/.github/ISSUE_TEMPLATE/bug_report.md
Also would help if you can share the full command with flags the customer is running.


upper bound and larger bound inconsistent with step sign

Hi, I'm trying to batch predict on some speech CSVs, at first some audio segments are a bit lengthy which caused the problem of https://github.com/gabrielmittag/NISQA/issues/11, so I saved the new model as with new max as 5500: nisqa_tts_5500 and nisqa_all_5500, the models seem to predict fine until the problem of "upper bound and larger bound inconsistent with step sign" show up.

Hi sorry for replying late, let me know if this is still an issue and I will reopen the item. You shouldn't need to save a new model, you only would need to update the max seg length in the config


refactor(rust/gui-client): begin isolating Tauri from our code

@thomaseizinger I moved a few modules and it compiles fine, can we merge this or should I try to move all of them?


Remote images cache problem with embedded javascript bundle

Working scenario:

Clone the repo
Open Examples folder cd react-native-photo-browser/Example
Checkout cache_test branch git checkout cache_test
Run npm install
Open Example project with Xcode ios/PhotoBrowserExample.xcodeproj/project.pbxproj
Open ios/PhotoBrowserExample/AppDelegate.m file and change jsCodeLocation value using the IP address of your machine.
Run the app on your device.
Select "Multiple Photos" option when the app starts.
Wait for the images to load.
Check Settings->General->Storage->Manage Storage->PhotoBrowserExample and note "Documents&Data" usage of the app. It should be around 38.4 MB.
Close the app and relaunch a few times. You'll see all photos initially available from cache.

Problem scenario:

Delete the app from your device.
Create javascript bundle file react-native bundle --minify --entry-file index.ios.js --platform ios --bundle-output ios/main.jsbundle
Open ios/PhotoBrowserExample/AppDelegate.m file. Remove the comment characters from line 44.

Now repeat all the steps between 7 and 11. You'll see that images will be reloaded after every time you relaunch the app. You'll see that the app's storage usage will go like 1 MB from 38.4 MB.
I tried removing dev mode, building the app with release option but none of them helped. I can see http cache headers in flickr http responses.
Notes:

I can't reproduce the problem when cached image storage is below 10 MB.
Tried on react-native versions 0.27.2, 0.26.3 and 0.25.1.
Tried changing image url protocols to http.

I tested the problem with react-native 0.32 and it's still there.

I dont't experience the problem with recent react native versions.


fix(dashboards): addition of the attribute 'zero' to DashboardWidgetYAxisLeft

Description
This PR addresses NR-108075, comprising of the following changes -

Addition of the attribute Zero to DashboardWidgetYAxisLeft to be sent to the API whenever set, for which the API is already programmed to fit the graph to scale if zero=false or print the graph in the given range of values if zero=true

Linked changes in dependencies
terraform-provider-newrelic : PR
newrelic-cli : PR

Closing this PR to avoid merge conflicts - new PR : #1028


[WIP] Build 2.12

Description

[ ] Still need macos version published
[ ] Need to add warning about TFA wind down

Addresses #2819

Hi @angerson we're seeing:
#23 39.00 ERROR: An error occurred during the fetch of repository 'local_config_cuda':
#23 39.00    Traceback (most recent call last):
#23 39.00 	File "/addons/build_deps/toolchains/gpu/cuda_configure.bzl", line 1089, column 38, in _cuda_autoconf_impl
#23 39.00 		_create_local_cuda_repository(repository_ctx)
#23 39.00 	File "/addons/build_deps/toolchains/gpu/cuda_configure.bzl", line 847, column 35, in _create_local_cuda_repository
#23 39.00 		cuda_config = _get_cuda_config(repository_ctx)
#23 39.00 	File "/addons/build_deps/toolchains/gpu/cuda_configure.bzl", line 630, column 30, in _get_cuda_config
#23 39.00 		config = find_cuda_config(repository_ctx, ["cuda", "cudnn"])
#23 39.00 	File "/addons/build_deps/toolchains/gpu/cuda_configure.bzl", line 1079, column 28, in find_cuda_config
#23 39.00 		auto_configure_fail("Failed to run find_cuda_config.py: %s" % exec_result.stderr)
#23 39.00 	File "/addons/build_deps/toolchains/gpu/cuda_configure.bzl", line 261, column 9, in auto_configure_fail
#23 39.00 		fail("\n%sCuda Configuration Error:%s %s\n" % (red, no_color, msg))
#23 39.00 Error in fail:
#23 39.00 Cuda Configuration Error: Failed to run find_cuda_config.py: Could not find any cuda.h matching version '11.2' in any subdirectory:
#23 39.00         ''
#23 39.00         'include'
#23 39.00         'include/cuda'
#23 39.00         'include/*-linux-gnu'
#23 39.00         'extras/CUPTI/include'
#23 39.00         'include/cuda/CUPTI'
#23 39.00         'local/cuda/extras/CUPTI/include'
#23 39.00 of:
#23 39.00         '/usr/local/cuda'

When building with bazel build --crosstool_top=@ubuntu20.04-gcc9_manylinux2014-cuda11.2-cudnn8.1-tensorrt7.2_config_cuda//crosstool:toolchain
on the new tensorflow/build:2.12-python images. Has there been any changes that you think would cause this?

Hi @kulinseth it looks like tensorflow-macos 2.12 didn't include a python 3.11 release. Was that a mistake or intended?
https://pypi.org/project/tensorflow-macos/2.12.0/#files

@seanpmorgan Yep, we moved to 11.8. Let me know if there's anything else that's not working.

Hi @kulinseth it looks like tensorflow-macos 2.12 didn't include a python 3.11 release. Was that a mistake or intended?
https://pypi.org/project/tensorflow-macos/2.12.0/#files

Hi @seanpmorgan yes , that was intended . We will have 3.11 support from next release onwards .

Hi @angerson Only the 2.12-python3.10 image is showing:
#24 2.[769](https://github.com/tensorflow/addons/actions/runs/4614368815/jobs/8157266720?pr=2821#step:8:770) ERROR: Exception:
#24 2.769 Traceback (most recent call last):
#24 2.769   File "/usr/local/lib/python3.10/dist-packages/pip/_internal/cli/base_command.py", line 160, in exc_logging_wrapper
#24 2.769     status = run_func(*args)
#24 2.769   File "/usr/local/lib/python3.10/dist-packages/pip/_internal/cli/req_command.py", line 247, in wrapper
#24 2.769     return func(self, options, args)
#24 2.769   File "/usr/local/lib/python3.10/dist-packages/pip/_internal/commands/install.py", line 415, in run
#24 2.769     requirement_set = resolver.resolve(
#24 2.769   File "/usr/local/lib/python3.10/dist-packages/pip/_internal/resolution/resolvelib/resolver.py", line 73, in resolve
#24 2.769     collected = self.factory.collect_root_requirements(root_reqs)
#24 2.769   File "/usr/local/lib/python3.10/dist-packages/pip/_internal/resolution/resolvelib/factory.py", line 491, in collect_root_requirements
#24 2.769     req = self._make_requirement_from_install_req(
#24 2.769   File "/usr/local/lib/python3.10/dist-packages/pip/_internal/resolution/resolvelib/factory.py", line 453, in _make_requirement_from_install_req
#24 2.769     cand = self._make_candidate_from_link(
#24 2.769   File "/usr/local/lib/python3.10/dist-packages/pip/_internal/resolution/resolvelib/factory.py", line 185, in _make_candidate_from_link
#24 2.769     self._editable_candidate_cache[link] = EditableCandidate(
#24 2.769   File "/usr/local/lib/python3.10/dist-packages/pip/_internal/resolution/resolvelib/candidates.py", line 322, in __init__
#24 2.769     super().__init__(
#24 2.769   File "/usr/local/lib/python3.10/dist-packages/pip/_internal/resolution/resolvelib/candidates.py", line 162, in __init__
#24 2.769     self.dist = self._prepare()
#24 2.769   File "/usr/local/lib/python3.10/dist-packages/pip/_internal/resolution/resolvelib/candidates.py", line 231, in _prepare
#24 2.769     dist = self._prepare_distribution()
#24 2.769   File "/usr/local/lib/python3.10/dist-packages/pip/_internal/resolution/resolvelib/candidates.py", line 332, in _prepare_distribution
#24 2.769     return self._factory.preparer.prepare_editable_requirement(self._ireq)
#24 2.769   File "/usr/local/lib/python3.10/dist-packages/pip/_internal/operations/prepare.py", line 633, in prepare_editable_requirement
#24 2.769     dist = _get_prepared_distribution(
#24 2.769   File "/usr/local/lib/python3.10/dist-packages/pip/_internal/operations/prepare.py", line 69, in _get_prepared_distribution
#24 2.769     abstract_dist.prepare_distribution_metadata(
#24 2.769   File "/usr/local/lib/python3.10/dist-packages/pip/_internal/distributions/sdist.py", line 46, in prepare_distribution_metadata
#24 2.769     self.req.isolated_editable_sanity_check()
#24 2.769   File "/usr/local/lib/python3.10/dist-packages/pip/_internal/req/req_install.py", line 505, in isolated_editable_sanity_check
#24 2.769     and not self.supports_pyproject_editable()
#24 2.769   File "/usr/local/lib/python3.10/dist-packages/pip/_internal/req/req_install.py", line 245, in supports_pyproject_editable
#24 2.769     return "build_editable" in self.pep517_backend._supported_features()
#24 2.769   File "/usr/local/lib/python3.10/dist-packages/pip/_vendor/pyproject_hooks/_impl.py", line 153, in _supported_features
#24 2.769     return self._call_hook('_supported_features', {})
#24 2.769   File "/usr/local/lib/python3.10/dist-packages/pip/_vendor/pyproject_hooks/_impl.py", line 321, in _call_hook
#24 2.769     raise BackendUnavailable(data.get('traceback', ''))
#24 2.769 pip._vendor.pyproject_hooks._impl.BackendUnavailable: Traceback (most recent call last):
#24 2.769   File "/usr/local/lib/python3.10/dist-packages/pip/_vendor/pyproject_hooks/_in_process/_in_process.py", line 77, in _build_backend
#24 2.769     obj = import_module(mod_path)
#24 2.769   File "/usr/lib/python3.10/importlib/__init__.py", line 126, in import_module
#24 2.769     return _bootstrap._gcd_import(name[level:], package, level)
#24 2.769   File "<frozen importlib._bootstrap>", line 1050, in _gcd_import
#24 2.769   File "<frozen importlib._bootstrap>", line 1027, in _find_and_load
#24 2.769   File "<frozen importlib._bootstrap>", line 992, in _find_and_load_unlocked
#24 2.769   File "<frozen importlib._bootstrap>", line 241, in _call_with_frames_removed
#24 2.769   File "<frozen importlib._bootstrap>", line 1050, in _gcd_import
#24 2.769   File "<frozen importlib._bootstrap>", line 1027, in _find_and_load
#24 2.769   File "<frozen importlib._bootstrap>", line 1004, in _find_and_load_unlocked
#24 2.769 ModuleNotFoundError: No module named 'setuptools'

Was something missed in the build? <3.10 and 3.11 work fine. We can hard code a fix but would rather it just be fixed in the image if possible

Was something missed in the build? <3.10 and 3.11 work fine. We can hard code a fix but would rather it just be fixed in the image if possible

Looks like its related to https://github.com/pypa/pip/issues/11770

Looks like its related to https://github.com/pypa/pip/issues/11770 . We will hardcode a pip version for now

Makes sense. Around Python 3.10 when distutils (?) was dropped, a lot of stuff broke. Good to hear there's a workaround.


Matrix dimension mismatch

Hi ! I dowloaded your implementation of SG filter and did not succeed to run it as is. The assertion comes at the line MatrixXf G = Sd * Rinv * RinvT; in the function sgdiff. With F=5 and k=3, it seems that the dimensions of Sd is 5x4 but the dimensions of Rinv and RinvT is 1x1. Hence, the multiplication Sd * Rinv * RinvT cause the program to abord. I did not edited anything, just downloaded the library, created a blank project in VS 2015, imported those plus the Eigen library. Any idea ?

Hi @Jeep1991 , sorry about the late response. For some reason, I did not get a notification from github about this issue. I was skimming through this repo this evening when I noticed the opened issue; hence my late response.
I did not test the code in Windows, sorry. It was developed in Ubuntu 14.04. I just compiled it again on ubuntu and it indeed does generate the Vandermonde matrix:
The size of Sd is [5 x 2] actually because we extract the rank deficient matrix from the vandermonde (or projection matrix) that we earlier calculated; For F = 5 and k = 3, R and Rinv are both square [2 x 2] matrices; So [5 x 2] x [2 x 2] x [2 x 2] gives matrix G which is [5 x 2] see this portion of the [code](https://github.com/lakehanne/Savitzky-Golay/blob/master/savgol.cpp#L68-#L71). Matrix Gis the matrix of differentiators and it is defined asG = S x (S' x S)^(-1)`  in eqn 8.3.90 in the book.
Vandermonde Matrix:
 1 -2  4 -8 16
 1 -1  1 -1  1
 1  0  0  0  0
 1  1  1  1  1
 1  2  4  8 16

, filtered values for the example numbers in the series
                               `[900 920 940 960 980]`

as
[955 960 940 940 945]
which turns out to be the same result you would get if you plug the savgol function in MATLAB with F as 5 and k as 3.
I made certain changes to the source code this evening that allows you to change the values of F, k and the time-series data at run-time. If you have the time, you could send a PR that reads an input stream of data from a text file and filters the read data.
Thanks for your comments and please let me know if you have further questions.


Clients with multiple RedirectUris do not have CORS flag set correctly

Apologies for bad format of this bug report - just under a bit of time pressure.
When there is more than one RedirectUri for a client, the CORS flag is not being set correctly.
This is because in Aguacongas.IdentityServer.Store.EntityExtensions, the public static Entity.Client ToEntity(this Client client) uses a FirstOrDefault match against the AllowedCors list. Changing this to a foreach loop with Where fixes this issue (I think).
foreach (var origin in client.AllowedCorsOrigins)
          {
              var cors = new Uri(origin);
              **foreach (var uri in uris.Where(u => cors.CorsMatch(u.Uri)))**
              {

Cheers,
Ben

@ben-jacobs Thanks to contribute. Are you sure you flags your Uris with CORS :

It works with theidserveradmin client so I don't think it's the issue

But not multiple uris with the same host - it'll only flag the first one.
Get Outlook for Androidhttps://aka.ms/ghei36

From: Olivier Lefebvre @.***>
Sent: Tuesday, 30 March 2021, 4:58 pm
To: Aguafrommars/TheIdServer
Cc: Ben Jacobs; Mention
Subject: Re: [Aguafrommars/TheIdServer] Clients with multiple RedirectUris do not have CORS flag set correctly (#395)
@ben-jacobshttps://github.com/ben-jacobs Thanks to contribute. Are you sure you flags your Uris with CORS :
[image]https://user-images.githubusercontent.com/3294829/112943242-883c2480-9131-11eb-91db-27df5e7a1e05.png
It works with theidserveradmin client so I don't think it's the issue
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/Aguafrommars/TheIdServer/issues/395#issuecomment-809948955, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AENPI6YHY2USY24ZERVGA7LTGFVPHANCNFSM42A5ZA5Q.

CORS only check scheme and host, not the path, that should not be an issue. So, just flag one host

The issue is not with how CORS works per se - just that multiple URIs with the same host don't all get the flag set and this creates errors when calling etc. This is an issue for example when using multiple oidc within the one app - to support persistent logins v. full logins. I was previously logging into the admin interface and setting the CORS flag on the additional URIs, but the foreach fixed the issue - I'll just keep it in my local branch if you're not convinced.

So, if you fixed it, can you create a PR ?


Accordion – Attempt #2

High-level change list:

The animation code is now much simpler and cleaner
Using <howto-accordion-heading> for headings, <section> for content panels
Using roving tab index
Custom event from headings to start expansion logic

@robdodson PTAL :)

Great job on this. A few things that got me thinking...
I wonder if it would be worth it to make the <section> it's own custom element. Then you wouldn't need to fiddle with its attributes from the parent accordion element. Right now you only set properties on the heading and it handles the rest. It seems like you'd want the same behavior for the section.
Also it got me wondering if this would be a good place to use Shadow DOM for a required element like the button. I realize Shadow DOM is its own can of worms but it feels like it's possibly a good fit here.

@robdodson @devnook PTAL :)

Live preview

@robdodson final PTAL? :D

Remove aria-hidden stuff and i think we're good

@robdodson Is this the moment?


Unable to toss multiple stacks in the same function

I have a function that looks like this:
function prepareInventory() {
        bot.tossStack(bot.inventory.items()[0], function(error) {
          console.log(error);
        });
        bot.tossStack(bot.inventory.items()[1], function(error) {
          console.log(error);
        });
        bot.tossStack(bot.inventory.items()[2], function(error) {
          console.log(error);
        });
}

Basically what it attempts to do is toss the first three stacks in the items array. What actually happens is it only tosses the third stack.
For trouble shooting I've added a function to log every change in the inventory, so we can see what happens. It looks like this:
bot.inventory.on('windowUpdate', function(slot, oldItem, newItem) {
  if(oldItem && newItem) {
    console.log(oldItem.name + " in slot " + slot + " replaced by " + newItem.name);
  }
  else if(oldItem) {
    console.log(oldItem.name + " in slot " + slot + " removed");
  }
});

I ran this with some stone brick, some dirt, and some wool. Here is the output in the console, undefined being the error callback. In game, the wool is what ended up being tossed.
stonebrick in slot 36 removed
dirt in slot 37 replaced by stonebrick
wool in slot 38 replaced by dirt
undefined
undefined
undefined

What it looks like is happening to me is it picks up the first item that it is instructed to toss, but instead of actually tossing it, it appears that it just clicks on the slot of the next item it was instructed to toss. Is there a known work around for this? If not, will there be a possible fix? I will poke around the source myself to see if I can find anything.
Not sure if I worded this to be as clear as possible, I'm writing this pretty late. I can explain anything if needed.

So in the simple_inventory.js, I see that they way tossStack works is that it clicks the slot the item is in, and then if the callback does not return an error, it clicks outside of the inventory. The problem with this appears to be that the callback doesn't run until after the function calling tossStack completes. I have virtually no knowledge of javascript, so I'm not sure how this kind of thing works. Would it be possible to just add the line clicking outside of the window right after the line clicking on the slot, but outside of the callback? I've tested it on a server and it works, but I'm not sure what will happen under non-ideal circumstances where it runs into errors.

I'm also finding the toss function to have similar issues. Trying to find a possible fix for that now too.

You are doing it wrong.
You need to execute the next action in the callback.
What you are trying to do right now is to toss 3 items at the same time, which obviously isn't possible and is going to break !

Ah, that makes a lot of sense now that you say it, thank you! Most of my experience is in Java and C# so I'm still trying to completely figure out this whole JavaScript and Node thing.
That said, I did get tossStack to work in the way I had originally tried, but I'll see about using it properly now :P

One more quick but completely unrelated question: Is it possible to allow the user to type into the command line while the bot is running, and to read whatever they have typed after they hit enter? To be able to provide commands from the command line. I'm positive there should be a way, but as a complete node noob, im wondering if you can just point me in the right direction

@cartycrabber You'll want to look at the "readline" nodejs module https://nodejs.org/api/readline.html . You can look at this for an example of how to use it (search for "rl.on('chat'")


Work around rate limit errors for unauthenticated requests coming from Travis CI

It looks like all builds on travis are sharing the same request limit across all builds, which it looks like it runs out of pretty quickly. That is a problem because we have to record requests that are unauthenticated for our fixtures.
Without any workaround, we will never get the record task to work: https://travis-ci.org/gr2m/octokit-fixtures/jobs/271878092#L461

I’ve workaround this using a revers proxy that can be configured using an environment variable: https://github.com/gr2m/octokit-fixtures/blob/654786c3425f286f596c79c895fc03ebbd70176d/bin/record.js#L44-L47
Maybe someone has a better idea, so I’ll leave this issue open

I removed unauthenticated requests altogether as they are not recommended anyway


Review: DCGAN -- Deep Convolutional Generative Adversarial Network (GAN).

Sik-Ho Tsang. Review: DCGAN — Deep Convolutional Generative Adversarial Network (GAN).

Overview
Unsupervised Representation Learning with Deep Convolutional Generative Adversarial Networks.
In this story, Deep Convolutional Generative Adversarial Network (DCGAN), by Indico Research and Facebook AI Research (FAIR), is reviewed. With DCGAN, a hierarchy of representations is learned from object parts to scenes in both the generator and discriminator. This is a paper in 2016 ICLR with about 6000 citations.

A Set of Constraints for Stable Training
All convolutional net replaces deterministic spatial pooling functions (such as max pooling) with strided convolutions (i.e. dilation convolution).

The generator learns its own spatial downsampling (relative to the dimension of input space) itself using convolution.
Similarly, the discriminator learns its own spatial upsampling (relative to the dimension of embedding space).

Eliminating Fully Connected Layers.

The first layer of the generator, which takes a uniform noise distribution $Z$ as input, could be called fully connected as it is just a matrix multiplication, but the result is reshaped into a 4-dimensional tensor and used as the start of the convolution stack.
For the discriminator, the last convolution layer is flattened and then fed into a single sigmoid output.
But there are no fully connected layers for all hidden layers.
Bye bye fully connected layers, haha~

Batch Normalization (BN).

BN stabilizes learning by normalizing the input to each unit to have zero mean and unit variance..
However, directly applying BN to all layers resulted in sample oscillation and model instability.
This was avoided by not applying BN to the generator output layer and the discriminator input layer.

Activation Functions.

The ReLU activation is used in the generator with the exception of the output layer which uses the Tanh function.
Within the discriminator, it is found that the leaky rectified activation (LeakyReLU) works well.

Network Architecture
Generator

The generator is as shown above. Only convolution, no fully connected layers according to the constraints.

Discriminator

The discriminator is just like a inverse of generator.

Experimental Results

Three datasets are trained: LSUN, ImageNet, and a newly assembled faces dataset.

LSUN

Generated bedrooms after one epoch.

Generated bedrooms after five epochs of training.

A model is trained on the LSUN bedrooms dataset containing a little over 3 million training examples.
The model is not producing high quality samples via simply overfitting/memorizing training examples.

Walking in the Latent Space

Interpolation between a series of 9 random points in Z show that the space learned has smooth transitions.

The vector $Z$ actually is a $n$ dimensional vector in a $n$ dimensional space.
If interpolation is performed between 2 $z$ vectors, a gradual change can be seen as shown above.
Walking in this latent space results in semantic changes to the image generations (such as objects being added and removed), we can reason that the model has learned relevant and interesting representations.

Visualizing the Discriminator Features

Left: Random Filter Baseline, Right: Trained Filters.

The above figure shows that the features learnt by the discriminator activate on typical parts of a bedroom, like beds and windows.

Forgetting to Draw Certain Objects

First Row: Models Without Dropping “Window” Filters, Second Row: Models With Dropping “Window” Filters.

By dropping “Window” filter, some windows are removed, others are transformed into objects with similar visual appearance such as doors and mirrors.
Although visual quality decreased, overall scene composition stayed similar, suggesting the generator has done a good job disentangling scene representation from object representation.

Vector Arithmetic on Face Samples

Smiling Woman — Neutral Woman + Neutral Man = Smiling Man.

Man With Glasses — Man Without Glasses + Woman Without Glasses = Woman With Glasses.

Simple arithmetic operations revealed rich linear structure in representation space.
e.g.: vector(”King”) — vector(”Man”) + vector(”Woman”) can result in a vector whose nearest neighbor was the vector for Queen.
Experiments working on only single samples per concept were unstable, but averaging the $Z% vector for three exemplars showed consistent and stable generations that semantically obeyed the arithmetic.
As shown in the figures above, for each column, the $Z$ vectors of samples are averaged. Arithmetic was then performed on the mean vectors creating a new vector $Y$. This $Y$ is fed into the generator as input.
A uniform noise sampled with scale ±0.25 was added to $Y$ to produce the other 8 samples.

A ”turn” vector was created from four averaged samples of faces looking left vs looking right.

Face pose can also be modeled linearly in $Z$ space.

Vector Arithmetic on Input Space.

In contrast, vector arithmetic on input space obtains poor results.

Reference
[2016 ICLR] [DCGAN] Unsupervised Representation Learning with Deep Convolutional Generative Adversarial Networks.

Generative Adversarial Network
[GAN] [CGAN] [LAPGAN] [DCGAN]


Use opaque ID of a user for signing keys

OCIS switched from user the user's opaque ID (UUID) everywhere,
so to keep compatible we have adjusted the signing keys endpoint
to also use the UUID when storing and generating the keys.
Fixes https://github.com/owncloud/ocis/issues/436
Doesn't work yet:
2020-08-07T11:04:23+02:00 ERR Account not found query="preferred_name eq '4c510ada-c86b-4815-8820-42cdf82c3d51'" service=proxy
2020-08-07T11:04:23+02:00 ERR could not create account error="{\"id\":\"com.owncloud.api.accounts\",\"code\":400,\"detail\":\"preferred_name '4c510ada-c86b-4815-8820-42cdf82c3d51' must be at least the local part of an email\",\"status\":\"Bad Request\"}" account={"accountEnabled":true,"creationType":"LocalAccount","onPremisesSamAccountName":"4c510ada-c86b-4815-8820-42cdf82c3d51","preferredName":"4c510ada-c86b-4815-8820-42cdf82c3d51"} service=proxy
2020-08-07T11:04:23+02:00 ERR Account not found query="preferred_name eq '4c510ada-c86b-4815-8820-42cdf82c3d51'" service=proxy
2020-08-07T11:04:23+02:00 ERR could not create account error="{\"id\":\"com.owncloud.api.accounts\",\"code\":400,\"detail\":\"preferred_name '4c510ada-c86b-4815-8820-42cdf82c3d51' must be at least the local part of an email\",\"status\":\"Bad Request\"}" account={"accountEnabled":true,"creationType":"LocalAccount","onPremisesSamAccountName":"4c510ada-c86b-4815-8820-42cdf82c3d51","preferredName":"4c510ada-c86b-4815-8820-42cdf82c3d51"} service=proxy
2020-08-07 11:04:23.994031 I | http: superfluous response.WriteHeader call from github.com/owncloud/ocis-proxy/pkg/middleware.AccountUUID.func1.1 (account_uuid.go:125)

the problem is here: https://github.com/owncloud/ocis-proxy/blob/9470b36c61d7cb1b36ec1678d08a97fa089ddc14/pkg/middleware/presigned_url.go#L28
ocis-proxy was passing the value of "OC-Credential" to try and resolve the user using the user name.
But now that value is the UUID and I didn't find anything in ocisoidc.StandardClaims that resembles the user id or UUID field. So maybe we don't need to resolve the user this way any more and read them from the account directly.

second part here: https://github.com/owncloud/ocis-proxy/pull/84

rebased and undrafted.
@butonic please review


Shared noise for different cases

Testing #158759
Error hints and No Inlay Hints on Line have the same sound

Good catch, but I think that is by design since VS Code stable behaves the same.
@hediet what is the exact use case when we play "no inlay hints on line"

Wouldn't that be confusing for someone who relies upon these sounds? I'd think they should be distinct

@jrieken added audio cues for inlay hints.

We play the sound when users execute the "Read Line With Inline Hints" and we figured that the error sound is a good fit. @meganrogge do you see shared sound use as an issue or is this just an observation? To me it's the same as our repeated use of the info, warning, and error icons

IMO, having no inlay hints on a line should not be categorized as an error - since it's not one - just info.

in fact, does a screen reader / audio cue user even care about having no inlay hints on a line (aren't those just a visible feature)?

aren't those just a visible feature

Audio cues is a feature that can help vision impaired users, that's why there is an explicit "read line with inline hint" commands. Not doing anything for the "no hint" cause is bad because you don't know if the feature is broken or if there are nothing to read

I agree, but I am suggesting a unique noise - not removing it

@jrieken the sound designer just created a "No Action Available" sound. Maybe this would be a better fit, and the sound that we can use for all the cases when there is no sound
Rename to .mp3
No-Action-Avail_v5.txt
fyi @AFre100

Copy this -- good points all around -- will look into this :)


A better way to handle redux action types?

I've been trying to work flow into a react-redux project but getting flow to interact smoothly with Redux actions is proving difficult. The issue is that each reducer needs to be aware of every single action type in order to work without error. This wouldn't be a huge problem for a simple app, but for a larger app, especially one that uses modules like react-router and react-redux-form, we're looking at a lot of different actions that need typing.
Here is an example of a working isolated reducer. This works great by itself, but when put into an app with other reducers, it will error out because right now this reducer only expects one of two action types (LOAD_PREFS_SUCCESS and UPDATE_PREF).
We can get around this error by adding a generic type to our actionType as a catch all. This however produces flow errors as seen here.
Is there a better way to handle action types in flow?

If you can't define a variant for all the actions, maybe try adding a fake extra case to the action variant. That would force you to define a default case in the reducer. Here's one ugly way to do it:
// @flow
const LOAD_PREFS_SUCCESS = 'LOAD_PREFS_SUCCESS';
const UPDATE_PREF = 'UPDATE_PREF';

export type actionType = {
  +type: typeof LOAD_PREFS_SUCCESS,
  prefs: Array<{_id: string, value: any}>
} | {
  +type: typeof UPDATE_PREF,
  id: string,
  value: any
} | {type: "Anything_Else"};

export default (state: {} = {}, action: actionType) => {
  switch (action.type) {
    case LOAD_PREFS_SUCCESS: {
      const newState = {};
      action.prefs.forEach(p => {
        newState[p._id] = p.value;
      });
      return newState;
    }
    case UPDATE_PREF: {
      return { ...state, [action.id]: action.value };
    }
    default:
      return state;
  }
};

Not pretty, but that seems to produce no errors. Thanks. Going to use the following for now.
export type actionType = {
  +type: typeof LOAD_PREFS_SUCCESS,
  prefs: Array<{_id: string, value: any}>
} | {
  +type: typeof UPDATE_PREF,
  id: string,
  value: any
} | { type: "FLOW_CATCH_ALL" };

I think I spoke too soon. It appeared to be working earlier but I guess I was looking at the wrong window. When I use the action type above and run my app, I get the following error when the application starts. Action type being run is @@redux/INIT
Uncaught TypeError: action must be one of: {
  type: string;
  prefs: Array<{
    _id: string;
    value: any;
  }>;
} | {
  type: string;
  id: string;
  value: any;
} | {
  type: "FLOW_CATCH_ALL";
}

Expected: {
  type: string;
  prefs: Array<{
    _id: string;
    value: any;
  }>;
} | {
  type: string;
  id: string;
  value: any;
} | {
  type: "FLOW_CATCH_ALL";
}

Actual: {
  type: string;
}

    at module.exports../app/reducers/prefs.js._flowRuntime2.default.annotate (http://localhost:1212/dist/renderer.dev.js:4432:54)
    at eval (webpack:///./node_modules/redux/es/combineReducers.js?:45:24)
    at Array.forEach (native)
    at assertReducerShape (webpack:///./node_modules/redux/es/combineReducers.js?:43:25)
    at combineReducers (webpack:///./node_modules/redux/es/combineReducers.js?:99:5)
    at Object../app/reducers/index.js (http://localhost:1212/dist/renderer.dev.js:4183:48)
    at __webpack_require__ (http://localhost:1212/dist/renderer.dev.js:671:30)
    at fn (http://localhost:1212/dist/renderer.dev.js:89:20)
    at Object../app/store/configureStore.dev.js (http://localhost:1212/dist/renderer.dev.js:4851:17)
    at __webpack_require__ (http://localhost:1212/dist/renderer.dev.js:671:30)

Oh, looks like you're using flow-runtime. That's going to add an interesting wrinkle to this. I'm not sure it's doable to get both static and runtime checking to work if we can't explicitly specify all the possible actions. Hmm...

Ah yes, sorry for not clarifying that. While I see value in both, I think static seems to be a cleaner solution (ie - don't have to include generic type: string). I'll probably just remove flow-runtime from the project for now.
I hope there is an easier usage of both static and runtime with redux in the future.

Might also be worth checking if the libraries generating these actions have libdefs that export a type for all the actions they throw. And if not, maybe open an issue or PR to make it happen. Would be nice to be able to say: all the actions from these three libs, plus my custom actions and get solid types.

Thanks to @csprance for the tip, but you can use this as a generic type that will still allow you type check your actions for the given reducer:
type BaseReduxAction = { type: $Subtype<string> };

So in my case the final action type would look like this:
export type actionType = {
  +type: typeof LOAD_PREFS_SUCCESS,
  prefs: Array<{_id: string, value: any}>
} | {
  +type: typeof UPDATE_PREF,
  id: string,
  value: any
} | { type: $Subtype<string> };

Hope this helps someone else running into this problem!

+type: typeof LOAD_PREFS_SUCCESS

This is actually not different from +type: string. This produces no errors in Flow:
const x = 'X';
const y: typeof x = 'Y';

See https://github.com/facebook/flow/issues/2639

Workaround is described here:
https://github.com/facebook/flow/issues/2377#issuecomment-262894389
In your case it should be
const LOAD_PREFS_SUCCESS: 'LOAD_PREFS_SUCCESS' = 'LOAD_PREFS_SUCCESS';
const UPDATE_PREF: 'UPDATE_PREF' = 'UPDATE_PREF';


Failed to clone module's repo assuming no case sensitivity

Problem
@cam72cam and I tried to clone the repo assuming no case sensitivity, i.e. git::https://github.com/terraform-aws-modules/terraform-AWS-iam?ref=v5.30.0. It failed.
Proposed solution
Use strings.ToLower when defining the module struct.
Question: is it expected behaviour?

I'm pretty sure that this is expected, given that we explicitly allow for mixed case in the regex.  @Yantrio is the one who added that code.

Yeah, I believe this is expected.

@cam72cam @cube2222 Thanks for your replies folks! The question is resolved.


Change term - scientificName usage comments

Term change

Submitter: Ian Engelbrecht @ianengelbrecht
Efficacy Justification (why is this change necessary?): Improve consistency in use of this term
Demand Justification (if the change is semantic in nature, name at least two organizations that independently need this term): Applies to any users of this term.
Stability Justification (what concerns are there that this might affect existing implementations?): Should improve consistency of usage of this term.
Implications for dwciri: namespace (does this change affect a dwciri term version)?: None

Current Term definition: https://dwc.tdwg.org/list/#dwc_scientificName
Proposed changes:

Usage comments (recommendations regarding content, etc., not normative): This term should not contain identification qualifications, which should instead be supplied in the IdentificationQualifier term. When applied to an Organism or Occurrence, this term should be used to represent the scientific name that was applied to the associated Organism in accordance with the Taxon to which it was or is currently identified. Names should be code-compliant where possible.

I fully support this change.

Related issue #391

I think it should say that the name "...should be compliant to the most recent nomenclatural code".
Citations of the codes would not be harmful and some details would be useful.
We don't expect users to read the nomenclatural codes, so some hints at what it a code compliant name would really help users.


Sandbox friendly build

Hi, @tazz4843 !
I've made some effort to simplify cmake machinery and make build sandbox-friendly (now it is not changing source directory), now this crate can be build with NIX Flake.
Any feedback appreciable!

I don't run Windows so I can't help much with the failing check unfortunately.

I don't run Windows so I can't help much with the failing check unfortunately.

Don't worry, I found symbol search path at whisper.cpp CI. Looks like it should work now (last time only rustfmt was broken).

Hello @tazz4843! Can I ask you to run CI pipeline?

GitHub doesn't appear to be letting me run the checks this time around, perhaps force push once more?

Can't get this to compile locally:
memory 19GiB/62GiB at 10-15 09:01:26 exit 101 took 6s156ms
on git sandbox_friendly is pkg v0.9.0-rc.2 via rs v1.75.0-nightly
niko@niko-desktop whisper-rs > cargo run --example basic_use
   Compiling whisper-rs-sys v0.7.3 (/tmp/whisper-rs/sys)
The following warnings were emitted during compilation:

warning: Unable to generate bindings: clang diagnosed error: wrapper.h:1:10: fatal error: 'whisper.h' file not found
warning: Using bundled bindings.rs, which may be out of date

error: failed to run custom build command for `whisper-rs-sys v0.7.3 (/tmp/whisper-rs/sys)`

Caused by:
  process didn't exit successfully: `/tmp/whisper-rs/target/debug/build/whisper-rs-sys-c3f9ea4e385de8cf/build-script-build` (exit status: 101)
  --- stdout
  cargo:rustc-link-lib=dylib=stdc++
  cargo:rerun-if-changed=wrapper.h
  cargo:rerun-if-env-changed=TARGET
  cargo:rerun-if-env-changed=BINDGEN_EXTRA_CLANG_ARGS_x86_64-unknown-linux-gnu
  cargo:rerun-if-env-changed=BINDGEN_EXTRA_CLANG_ARGS_x86_64_unknown_linux_gnu
  cargo:rerun-if-env-changed=BINDGEN_EXTRA_CLANG_ARGS
  cargo:warning=Unable to generate bindings: clang diagnosed error: wrapper.h:1:10: fatal error: 'whisper.h' file not found

  cargo:warning=Using bundled bindings.rs, which may be out of date
  CMAKE_TOOLCHAIN_FILE_x86_64-unknown-linux-gnu = None
  CMAKE_TOOLCHAIN_FILE_x86_64_unknown_linux_gnu = None
  HOST_CMAKE_TOOLCHAIN_FILE = None
  CMAKE_TOOLCHAIN_FILE = None
  CMAKE_GENERATOR_x86_64-unknown-linux-gnu = None
  CMAKE_GENERATOR_x86_64_unknown_linux_gnu = None
  HOST_CMAKE_GENERATOR = None
  CMAKE_GENERATOR = None
  CMAKE_PREFIX_PATH_x86_64-unknown-linux-gnu = None
  CMAKE_PREFIX_PATH_x86_64_unknown_linux_gnu = None
  HOST_CMAKE_PREFIX_PATH = None
  CMAKE_PREFIX_PATH = None
  CMAKE_x86_64-unknown-linux-gnu = None
  CMAKE_x86_64_unknown_linux_gnu = None
  HOST_CMAKE = None
  CMAKE = None
  running: cd "/tmp/whisper-rs/target/debug/build/whisper-rs-sys-f7e57ece2896f8a9/out/build" && CMAKE_PREFIX_PATH="" "cmake" "-Wdev" "--debug-output" "/tmp/whisper-rs/target/debug/build/whisper-rs-sys-f7e57ece2896f8a9/out/whisper.cpp/" "-DBUILD_SHARED_LIBS=OFF" "-DWHISPER_ALL_WARNINGS=OFF" "-DWHISPER_ALL_WARNINGS_3RD_PARTY=OFF" "-DWHISPER_BUILD_TESTS=OFF" "-DWHISPER_BUILD_EXAMPLES=OFF" "-DCMAKE_INSTALL_PREFIX=/tmp/whisper-rs/target/debug/build/whisper-rs-sys-f7e57ece2896f8a9/out" "-DCMAKE_C_FLAGS= -ffunction-sections -fdata-sections -fPIC -m64" "-DCMAKE_C_COMPILER=/usr/bin/cc" "-DCMAKE_CXX_FLAGS= -ffunction-sections -fdata-sections -fPIC -m64" "-DCMAKE_CXX_COMPILER=/usr/bin/c++" "-DCMAKE_ASM_FLAGS= -ffunction-sections -fdata-sections -fPIC -m64" "-DCMAKE_ASM_COMPILER=/usr/bin/cc" "-DCMAKE_BUILD_TYPE=Release" "-DCMAKE_VERBOSE_MAKEFILE:BOOL=ON"
  Running with debug output on.

  --- stderr
  wrapper.h:1:10: fatal error: 'whisper.h' file not found
  CMake Warning:
    Ignoring extra path from command line:

     "/tmp/whisper-rs/target/debug/build/whisper-rs-sys-f7e57ece2896f8a9/out/whisper.cpp/"

  CMake Error: The source directory "/tmp/whisper-rs/target/debug/build/whisper-rs-sys-f7e57ece2896f8a9/out/whisper.cpp" does not appear to contain CMakeLists.txt.
  Specify --help for usage, or press the help button on the CMake GUI.
  thread 'main' panicked at /home/niko/.cargo/registry/src/index.crates.io-6f17d22bba15001f/cmake-0.1.50/src/lib.rs:1098:5:

  command did not execute successfully, got: exit status: 1

  build script failed, must exit now
  note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

Can't get this to compile locally:
memory 19GiB/62GiB at 10-15 09:01:26 exit 101 took 6s156ms
on git sandbox_friendly is pkg v0.9.0-rc.2 via rs v1.75.0-nightly
niko@niko-desktop whisper-rs > cargo run --example basic_use
   Compiling whisper-rs-sys v0.7.3 (/tmp/whisper-rs/sys)
The following warnings were emitted during compilation:

warning: Unable to generate bindings: clang diagnosed error: wrapper.h:1:10: fatal error: 'whisper.h' file not found
warning: Using bundled bindings.rs, which may be out of date

error: failed to run custom build command for `whisper-rs-sys v0.7.3 (/tmp/whisper-rs/sys)`

Caused by:
  process didn't exit successfully: `/tmp/whisper-rs/target/debug/build/whisper-rs-sys-c3f9ea4e385de8cf/build-script-build` (exit status: 101)
  --- stdout
  cargo:rustc-link-lib=dylib=stdc++
  cargo:rerun-if-changed=wrapper.h
  cargo:rerun-if-env-changed=TARGET
  cargo:rerun-if-env-changed=BINDGEN_EXTRA_CLANG_ARGS_x86_64-unknown-linux-gnu
  cargo:rerun-if-env-changed=BINDGEN_EXTRA_CLANG_ARGS_x86_64_unknown_linux_gnu
  cargo:rerun-if-env-changed=BINDGEN_EXTRA_CLANG_ARGS
  cargo:warning=Unable to generate bindings: clang diagnosed error: wrapper.h:1:10: fatal error: 'whisper.h' file not found

  cargo:warning=Using bundled bindings.rs, which may be out of date
  CMAKE_TOOLCHAIN_FILE_x86_64-unknown-linux-gnu = None
  CMAKE_TOOLCHAIN_FILE_x86_64_unknown_linux_gnu = None
  HOST_CMAKE_TOOLCHAIN_FILE = None
  CMAKE_TOOLCHAIN_FILE = None
  CMAKE_GENERATOR_x86_64-unknown-linux-gnu = None
  CMAKE_GENERATOR_x86_64_unknown_linux_gnu = None
  HOST_CMAKE_GENERATOR = None
  CMAKE_GENERATOR = None
  CMAKE_PREFIX_PATH_x86_64-unknown-linux-gnu = None
  CMAKE_PREFIX_PATH_x86_64_unknown_linux_gnu = None
  HOST_CMAKE_PREFIX_PATH = None
  CMAKE_PREFIX_PATH = None
  CMAKE_x86_64-unknown-linux-gnu = None
  CMAKE_x86_64_unknown_linux_gnu = None
  HOST_CMAKE = None
  CMAKE = None
  running: cd "/tmp/whisper-rs/target/debug/build/whisper-rs-sys-f7e57ece2896f8a9/out/build" && CMAKE_PREFIX_PATH="" "cmake" "-Wdev" "--debug-output" "/tmp/whisper-rs/target/debug/build/whisper-rs-sys-f7e57ece2896f8a9/out/whisper.cpp/" "-DBUILD_SHARED_LIBS=OFF" "-DWHISPER_ALL_WARNINGS=OFF" "-DWHISPER_ALL_WARNINGS_3RD_PARTY=OFF" "-DWHISPER_BUILD_TESTS=OFF" "-DWHISPER_BUILD_EXAMPLES=OFF" "-DCMAKE_INSTALL_PREFIX=/tmp/whisper-rs/target/debug/build/whisper-rs-sys-f7e57ece2896f8a9/out" "-DCMAKE_C_FLAGS= -ffunction-sections -fdata-sections -fPIC -m64" "-DCMAKE_C_COMPILER=/usr/bin/cc" "-DCMAKE_CXX_FLAGS= -ffunction-sections -fdata-sections -fPIC -m64" "-DCMAKE_CXX_COMPILER=/usr/bin/c++" "-DCMAKE_ASM_FLAGS= -ffunction-sections -fdata-sections -fPIC -m64" "-DCMAKE_ASM_COMPILER=/usr/bin/cc" "-DCMAKE_BUILD_TYPE=Release" "-DCMAKE_VERBOSE_MAKEFILE:BOOL=ON"
  Running with debug output on.

  --- stderr
  wrapper.h:1:10: fatal error: 'whisper.h' file not found
  CMake Warning:
    Ignoring extra path from command line:

     "/tmp/whisper-rs/target/debug/build/whisper-rs-sys-f7e57ece2896f8a9/out/whisper.cpp/"

  CMake Error: The source directory "/tmp/whisper-rs/target/debug/build/whisper-rs-sys-f7e57ece2896f8a9/out/whisper.cpp" does not appear to contain CMakeLists.txt.
  Specify --help for usage, or press the help button on the CMake GUI.
  thread 'main' panicked at /home/niko/.cargo/registry/src/index.crates.io-6f17d22bba15001f/cmake-0.1.50/src/lib.rs:1098:5:

  command did not execute successfully, got: exit status: 1

  build script failed, must exit now
  note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

Have you downloaded submodules?
git submodule update --init --recursive

I tested this setup in few environments: OS X M1, X86 and ARM servers.

I always forget to clone the submodules, thanks for reminding me. Seems to work now.=

Thanks for the PR!


This deserves a blog post

GraphQL Serverless and Eventing is one of the most demanded topics with many people looking for example. I would love to promote the gql-source and even contribute after I will get some understanding how it works but it will be easier if there is simple blog post one can follow. I have tried to run example but still do not get how things work here.
@itsmurugappan

Thanks @wtrocki , I wrote a blog in medium, but i guess you are looking for more detailed one on how to get started right ?
I will write a detailed one on the pattern and how to use it.

Haven't seen it. I have 2 questions.
Is this for subscriptions only?
What is the flow - Let's say that I have subscribed and created new item.
In typical app my event source will be create that publish event and sink would be subscription to divert data. Diagram is the other way around which made me think that I do not understand KNative enough to get it.
What is connecting to sink?

The GqlSource subscribes to this endpoint and tracks the changes on menu and other info.

This sound to me like I have my own server and now GQL source will issue GraphQL subscriptions to that server and expose it to some sink. I think explanation that uses GraphQL queries will make your blog post every successful.
What query goes where etc.
What I was looking for is to have multiple serverless mutations that could become event sources and single statefull server that will utilize sink to send payload to subscribed clients. Are we in the same space?

sorry for a delayed response.

This sound to me like I have my own server and now GQL source will issue GraphQL subscriptions to that server and expose it to some sink. I think explanation that uses GraphQL queries will make your blog post every successful.

This is right , i will add the queries to the blog post. Thanks for the suggestion.

What I was looking for is to have multiple serverless services that support GraphQL mutations that could become event
sources and single statefull server that will utilize sink to send payload to subscribed clients. Are we in the same space?
This will enable serverless GraphQL subscribtions - subscribtions typically require stateless info about clients so are unfit to become functions on it's own

This is very interesting. Few q's

When you say multiple serverless services that support GraphQL mutations are those resolvers?
The client usually establishes a websocket connection with server to get the changes, how can this be delegated to a sink ?

Yes (subscription resolvers might need to be in statefull server instead of lambda)

Subscription resolvers are used to establish connection on non serverless server (Subscriptions handling) then publish subscribe mechanism is used on changes


Run faster than once per minute?

Since cron runs at most once per minute, a PDF job sent to pdf-bot may take one minute to process even if pdf-bot is not busy at the time.
Is there a way to have a shift:all run automatically when an API push is triggered? Or is there a trick to getting pdf-bot to process more quickly?

Its not super elegant but I have added 2 things in 48fe04da40239318133e666ff8d1d2a11e52347b

At the end of shift:all it will run again until there are no more unfinished jobs in the queue
Added postPushCommand to API configuration. You can use it like this:

api: {
    // The port your express.js instance listens to requests from. (default: 3000)
    port: 3000,
    // Spawn command when a job has been pushed to the API
    postPushCommand: ['/home/user/.npm-global/bin/pdf-bot', ['-c', './pdf-bot.config.js', 'shift:all']],
    // The token used to validate requests to your API. Not required, but 100% recommended.
    token: 'api-token'
},

it will spawn a process using postPushCommand as arguments after a job has been pushed to the queue


panic: runtime error: invalid memory address or nil pointer dereference

Hi,
I just caught a runtime error and can reproduce it.
version: 0.0.43 | compiled: 2022-12-08T12:58:30Z
vacuum generate-ruleset recommended
vacuum lint -r ruleset-recommended.yaml example.yml

example.yml as follows:
openapi: 3.0.3

info:
  version: 1.0.0
  title: Example

tags:
  - name: my-tag
    description: A description for My Tag

servers:
  - url: 'http://<IP_ADDRESS>:8080'

components:
  securitySchemes:
    bearerAuth:
      type: http
      scheme: bearer
      bearerFormat: JWT

paths:
  /v1/users:
    get:
      summary: Some summary
      operationId: users.get
      responses:
        '204':
          description: No content

    head:
      summary: Some summary
      operationId: users.head
      responses:
        '204':
          description: No content

    post:
      summary: Some summary
      operationId: users.post
      responses:
        '204':
          description: No content

    patch:
      summary: Some summary
      # operationId: users.patch
      responses:
        '204':
          description: No content

Let me know if I can provide more information that would be helpful.

panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x2 addr=0x90 pc=0x102f75340]

goroutine 72 [running]:
regexp.(*Regexp).doExecute(0x1037d3840?, {0x0?, 0x0?}, {0x0?, 0x1400079bba0?, 0x140004b1bd0?}, {0x140002c6078?, 0x0?}, 0x1037d33c0?, 0x140007931d0?, ...)
	/opt/hostedtoolcache/go/1.19.3/x64/src/regexp/exec.go:527 +0x50
regexp.(*Regexp).doMatch(...)
	/opt/hostedtoolcache/go/1.19.3/x64/src/regexp/exec.go:514
regexp.(*Regexp).MatchString(...)
	/opt/hostedtoolcache/go/1.19.3/x64/src/regexp/regexp.go:533
github.com/daveshanley/vacuum/functions/openapi.NoEvalInDescriptions.RunRule({}, {0x14000716030?, 0x8?, 0x103d413e0?}, {0x14000712180, 0x140007cd760, {0x1037a7a60, 0x1400079bb50}, {0x1037d33c0, 0x140007931d0}, ...})
	/home/runner/work/vacuum/vacuum/functions/openapi/no_eval_descriptions.go:41 +0x1d0
github.com/daveshanley/vacuum/motor.buildResults({0x140007cd760, 0x140001400a0, {0x10386b348, 0x140003f48e0}, 0x140007b85a0, 0x140006104d0, 0x1400058c180, 0x14000159000, 0x140001401e0, 0x0}, ...)
	/home/runner/work/vacuum/vacuum/motor/rule_applicator.go:387 +0x568
github.com/daveshanley/vacuum/motor.runRule({0x140007cd760, 0x140001400a0, {0x10386b348, 0x140003f48e0}, 0x140007b85a0, 0x140006104d0, 0x1400058c180, 0x14000159000, 0x140001401e0, 0x0})
	/home/runner/work/vacuum/vacuum/motor/rule_applicator.go:314 +0x304
created by github.com/daveshanley/vacuum/motor.ApplyRulesToRuleSet
	/home/runner/work/vacuum/vacuum/motor/rule_applicator.go:157 +0x628

Nice find! I'll look into this asap.

This should be fixed in v0.0.44

Thank you!


Host some single page .md guides as part of lborb?

Recently there were add some helpful documents such as Pointers. I wonder if some of these single paged books, where they are appropriately licensed, might be forked and merged into lborb?

I'd really like to avoid lborb including content as that would diverge from the original philosophy of being a handy place to find Rust books you'd not come across. Also, if we included content, we'd be responsible for maintaining it.
Sorry, but closing this issue on the basis of scope creep.


Error getting collection_stats with incorrect name

I'm trying to find collection stats for not existing collection and i recive

api = OpenseaAPI()
api.collection_stats(collection_slug="asd")

Traceback (most recent call last):
  File "/home/m4xx1m/jp3g3r/venv/lib/python3.10/site-packages/requests/models.py", line 910, in json
    return complexjson.loads(self.text, **kwargs)
  File "/usr/lib/python3.10/json/__init__.py", line 346, in loads
    return _default_decoder.decode(s)
  File "/usr/lib/python3.10/json/decoder.py", line 337, in decode
    obj, end = self.raw_decode(s, idx=_w(s, 0).end())
  File "/usr/lib/python3.10/json/decoder.py", line 355, in raw_decode
    raise JSONDecodeError("Expecting value", s, err.value) from None
json.decoder.JSONDecodeError: Expecting value: line 1 column 1 (char 0)

During handling of the above exception, another exception occurred:
Traceback (most recent call last):
  File "/usr/lib/python3.10/code.py", line 90, in runcode
    exec(code, self.locals)
  File "<input>", line 1, in <module>
  File "/home/m4xx1m/jp3g3r/venv/lib/python3.10/site-packages/opensea/opensea_api.py", line 236, in collection_stats
    return self._make_request(endpoint, export_file_name=export_file_name)
  File "/home/m4xx1m/jp3g3r/venv/lib/python3.10/site-packages/opensea/opensea_api.py", line 67, in _make_request
    return response.json()
  File "/home/m4xx1m/jp3g3r/venv/lib/python3.10/site-packages/requests/models.py", line 917, in json
    raise RequestsJSONDecodeError(e.msg, e.doc, e.pos)
requests.exceptions.JSONDecodeError: [Errno Expecting value] : 0

Hi! @m4xx1m ,
What you are saying is that there should be some meaningful error message saying that's a non-existing collection or Not found or something similar?

I'm closing this now, but if you have other issues in the future feel free to open a new one


upgrade instructions and changelog

This pr adds the changelog for UMH 0.13.3 (0.2.15)

@JeremyTheocharis The migration is quite long.
We could put it into a bash script and do a bash <(curl -s https://umh.docs.umh.app/path/to/script.sh)

It would also allow a bit cleaner handling of already existing stuff like users and databases

What about https://management.umh.app/upgrading/upgrading-to-0.95.3.sh ?

Im now creating the redirect


Route components don't have their constructors called when props are updated

Version
4.3.1
Test Case
https://codesandbox.io/s/00zomy7xj0
Steps to reproduce
Create a Route component that gets props from the component containing BrowserRouter, from the state from componentDidMount.
Expected Behavior
After componentDidMount runs, the props passed to the child component should also call the constructor.
Actual Behavior
The constructor is not called.

The constructor of a component is called only when it is first created, not after subsequent updates. Please see the docs about the React lifecycle for further info.
In your case, your example works if you just read from this.props.items in your render method instead of this.items, which is stale on the 2nd render. A fixed version is here.

Yep I know I can get this.props.items directly, but I have other logic in the constructor in my actual app, which would modify it. I'm setting state from the constructor, and so I can't put setState in render otherwise it results in an infinite loop.
I thought there would at least be a way to force the constructor to run again, perhaps?


v4 rewrite: All new design, core types, and stream-oriented interfaces

I spent the holiday rewriting this package from scratch with a completely new approach to handling archives and compression formats. This will become v4.
The new core APIs are completely stream-oriented and file-agnostic. The abstractions for files, directories, and file systems are virtualized thanks to the recently-added io/fs package in the Go standard library. I expect these design changes will close most open issues and PRs because it either fixes the problems or makes them irrelevant.
A significant number of issues relate directly to interactions with specific file systems (files on disk) and this new API does not deal with that directly anymore, except for a couple specific functions that read the disk in order to create the abstraction. Nothing in the core API writes to disk or deals with that. (Yay!) The stream and FS abstractions are highly flexible to build upon.
Another nice thing about the new design is that there's no more need for explicit composite types (like TarGz and TarBz2), because we have a new CompressedArchive type that composes an archive and a compression format. It's mainly used with Tar only, since Zip and Rar do their own thing, but another nice feature is the Identify() function that automatically gets you the right type:
// opening a file on disk for this example, but can be any ReadSeeker
unknownFile, err := os.Open("filename.tar.gz")
if err != nil {
	return err
}
defer unknownFile.Close()

// if you don't have a filename, leave it blank: identification uses both/either filenames and/or streams
format, err := archiver.Identify("filename.tar.gz", unknownFile)
if err != nil {
	return err
}

// we can now work with the file, for example, extract a file out of it
if ex, ok := format.(archiver.Extractor); ok {
	ex.Extract(context.Background(), unknownFile, "target.txt", func(_ context.Context, f File) error {
		// do something with the file ...
		return nil
	})
}

// or maybe it's just a compressed log file
if decom, ok := format.(archiver.Decompressor); ok {
	rc, err := decom.OpenReader(unknownFile)
	if err != nil {
		return err
	}
	defer rc.Close()
	// read from it ... all reads are decompressed now
}

I think the new APIs are pretty slick, and you should try them out and let me know what you think.
One of my favorite new features is the FileSystem() function. Give it a path on disk, and it will return a fs.ReadDirFS. Basically, it lets you read from real directories, regular files, archive files, and compressed archive files, ALL THE SAME WAY. This is pretty cool I think. You don't have to worry about whether the given file is just a regular file, a directory, or an archive file (which acts like a directory because it contains other files!) -- you can traverse it all the same way. And the archive format doesn't matter either, it's automatically identified for you! You literally won't even know things are being decompressed as you read from them:
fsys, err := archiver.FileSystem("/path/to/folder/or/file")
if err != nil {
	return err
}

// traverse everything except the ".git" folder...
err = fs.WalkDir(fsys, ".", func(path string, d fs.DirEntry, err error) error {
	if err != nil {
		return err
	}
	if path == ".git" {
		return fs.SkipDir
	}
	fmt.Println(path, d.IsDir())
	return nil
})
if err != nil {
	return err
}

// ...or just open one file
file, err := fsys.Open("example.txt")
if err != nil {
	return err
}
defer file.Close()

To make a new archive from files on disk, you might do this:
files, err := archiver.FilesFromDisk(map[string]string{
	"/path/on/disk/file1.txt": "file1.txt",
	"/path/on/disk/file2.txt": "subfolder/file2.txt",
	"/path/on/disk/folder":    "",
})
if err != nil {
	return err
}

out, err := os.Create("example.tar.gz")
if err != nil {
	return err
}
defer out.Close()

caf := archiver.CompressedArchive{
	Compression: archiver.Gz{},
	Archival:    archiver.Tar{},
}

err = caf.Archive(context.Background(), out, files)
if err != nil {
	return err
}

Notice how you have the flexibility of mapping each file (or folder) to a different path in the archive. You can also leave the mapped path blank to assume the base filename for convenience. Folders are added recursively.
Oh yeah, and I added basic context support.
The arc command has not been ported over yet, and as that has to deal directly with the file system, it will require some work before it is ready. This would be the only part of the repository that would write to disk, as the core library doesn't write to disk anymore.
Looks like I was able to delete about 70% of the code, however that count includes test files, the command, and the README, which have not been restored yet. Still, I could feel a significant code reduction in this new design when I wrote it.
Should make irrelevant / close / fix #118, #128, #131, #141, #146, #150, #227, #194, #204, #216, #239, #255, #262, #278, #282

I might rename this package to archives. It was suggested before but I didn't get around to it. Maybe now is a good time.

I might rename this package to archives. It was suggested before but I didn't get around to it. Maybe now is a good time.

I believe in the Go philosophy that if you're making a breaking change, that's a version bump, but if you're making a philosophical change, that's a new package.

This isn't a PR that one can simply review. This is a new project from the ground up.
Make a new repo, get some tests in, mark this as deprecated, and link to the new repo.
How do you want to handle tests in the new project?

Ha, sorry @coolaj86 , I tagged you as a reviewer not with the expectation that you'd actually give a full review -- I probably should have just @-mentioned you instead, since all I wanted to do was let you know that this is inbound. :)

After quite a few more trials, this seems to be working decently well at least, so I am going to merge it into master but I won't tag it yet (except pre-releases).
Some more tests would be nice, as well as an actual command, but for now I want to have some people start using this, and I need to move my focus onto some other projects here pretty soon anyway.
The new README shows off some of the new version's elegance and features, so I hope you'll enjoy!

@mholt this is a substantial update and a lot of really great additions.  I'm very interested in the stream options now as we have a new use for them for our air gap delivery tool that uses your library.  Thanks for the work on this, I see some notes about needing testing/validation, but also notice the tests seem to be less robust in this new cut.  I don't think we'll be able to jump to v4 without some more automated tests, but will play with this new version for sure.  I also saw you have other priorities soon, do you have an idea what the plan will be for automated tests with v4 going forward?
On another note, I spent the holiday break doing a "mega PR" myself for our project that caused some excitement 🤣:  https://github.com/defenseunicorns/zarf/pull/237.

@jeff-mccoy That's great to hear! I agree v4 could benefit from more tests.

I also saw you have other priorities soon, do you have an idea what the plan will be for automated tests with v4 going forward?

Yeah, so that's true... I'm really busy with Caddy/CertMagic right now, and what I have written for archiver v4 works for me and my needs. However, I do accept sponsorships, and at the right tiers I can absolutely prioritize writing some automated tests for you. :+1:
Otherwise, I can try to get around to them when I have a chance, or I'm happy to accept contributions (PRs) as well! Let me know what suits you best and we can do that.

Awesome.  I think we'll do some combination of those options.  We deeply value open source projects and know it can be a thankless endeavor sometimes.  We'll keep in touch for sure, thanks again for the quick reply and the great library.

@jeff-mccoy Sounds good -- would look forward to working with you on this if that can happen!

@jeff-mccoy Just checking in, how is this lib working for you? (It's been through several releases now that enhance its functionality and fix bugs.)


Normalize project url

Normalize project url in $HOME/.gitlab-ci-local/variables.yaml to allow git ssh remote urls

@firecow I am unable to get the tests to pass. The output formatting changes when I add more than 1 job to the .gitlab-ci.yml.

@firecow The code is working, the issue is with the output formatting. When there is only one job in .gitlab-ci.yml, the variable names are printed ingreenBright. When I add the second job (test-normalize-key), the variables are printed in red. So the output was not matching the expected array.


Unable to see the version of the app in profile icon due to smaller screen length.

Describe the bug
Unable to see the version of the app in profile icon due to smaller screen length.
To Reproduce

unlock the app
click on profile icon 4 th option

Current behavior
As screen size is smaller the version of the is not visible.
Expected behavior
Should have scroll bar which can help us to scroll and view the version details.
Screenshots

Smartphone (please complete the following information):
Device: iPhone7
OS: 15.4.1
Inji app version: 0.4.0 (16) in test flight
MOSIP Version: 1.2.1
MOSIP server: qa-121.mosip.net

verified as fixed in the build with commit id in the below screenshot.
https://drive.google.com/file/d/15uWjasH2tRj0qCau8Ljj8YFEoSXi6mfd/view?usp=drivesdk


CaffeineInc.Caffeine version <IP_ADDRESS>3

[ ] Have you signed the Contributor License Agreement?
[ ] Have you checked that there aren't other open pull requests for the same manifest update/change?
[ ] Have you validated your manifest locally with winget validate --manifest <path>?
[ ] Have you tested your manifest locally with winget install --manifest <path>?
[ ] Does your manifest conform to the 1.0 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

Re-installed from scratch and it works flawlessly for me.


FEATURE: Make SideBySideRenderer SSR compatible

Because of the use of window in https://github.com/AdamRisberg/react-image-magnifiers/blob/476bba1958e75fcf14f4f966928c1ee07e6be373/src/SideBySideRenderer.js#L28 the component is not SSR compatible throwing a reference error that window is not defined, obviously.
Don't know about the performance impact, but maybe it's possible to render the large preview container only onMouseEnter and calculate whether to use inPlace or not at that moment.
Maybe it's even enough to add the active variable to the check in line 28 because this should never be true during SSR.

It should be fixed now. If window doesn't exist, the expression will evaluate to false without trying to access innerWidth. I just published the fix to npm (version 1.0.5).

After some testing, I realized that this issue still existed. As of version 1.0.7, it is now fully fixed and all components support SSR.


Null should be nil

Purpose or Intent
Null should be nil, this is breaking Vm explorer page

@kbrock I blame it to too much time on the DB side :-D
Funky enough, seems like we are missing the specs for vm explorer and vm detail, cause those to throw 500 and specs are passing :-)

@Ladas thanks for noticing this.  Is this something I need to backport to Darga?

@chessbyte no, I think it got there like a few days ago

Ugh. Thanks @Ladas


Decoupling ChatServer from DataModel

Currently the ChatServer project indirectly depends on the DataModel.
The dependency can be shown as follows:
ChatServer -> Interfaces -> DataModel
The Interface project just uses the following stuff of the DataModel:
FriendViewItem -> can be moved to the Interfaces project
LetterHeader -> can be moved to the Interfaces project
Guild, GuildMemberInfo, GuildPosition -> ???

Should be fixed with https://github.com/MUnique/OpenMU/commit/fdb1445d34cd0cc9447ba38725d5cc22c41035bf.


wskdeploy does not display version using "version" command

The injection of a version would have to be done as part of the build process.
currently here is what you see:
$ ./wskdeploy version
wskdeploy build-version: unset--unset

Hi, the version of wskdeploy is currently depends on ldflags for injection in some places (Makefile, build.sh).
IMHO, it would be nice if we unified the building process, i.e. make. Therefore, the build can be always inject the build version via ldflags without adding from developers.
However, seems like there are still some problems in unit tests after quickly look around, in which the make build depends on test.


AttributeError: 'Manager' object has no attribute 'commit_select'

What I believe is simple code is returning an opaque error.  The call is
async def _get_acct_by_userid(self,user_id):
    ah = await db.execute(AccountHolder.select().where(AccountHolder.userid == user_id))
    if ah.exists():
       return ah.get()
    else:
        return None

but the output I get is AttributeError: 'Manager' object has no attribute 'commit_select'
Am I doing something wrong here?

@sscondie It's unclear, what is db? And how the model is defined? Seems like you're using Manager as model's Meta.database, but I'm not sure.
Also, the code you've provided won't work, because the result of await execute(...) will return iterable object of AccountHolder instances, and it doesn't contain .exist() method.
The code should be like that:
database = peewee_async.PooledPostgresqlDatabase('test')
db = peewee_async.Manager(database)

class AccountHolder(peewee.Model):
     class Meta:
          database = database

async def _get_acct_by_userid(self, user_id):
    try:
         obj = await db.get(user_id=user_id)
    except AccountHolder.DoesNotExist:
         pass

Or you may replace obj = await db.get(user_id=user_id) with the more complex select:
query = await db.get(AccountHolder.select().where(AccountHolder.user_id == user_id))


Added support for css classes on menu widgets & fixed bug in video widget

Added fixes talked about in issue #268

Thank you!  I am going to manually merge this into 4.6, then back to master in the next release.

This looks like it has been fully merged into 4.6 can I close this?

Yes, I think we are good.  Thanks!


Handle invalid nlog.config / XML better

When the xml-parser fails to parse the input stream, then NLog should scan the configuration-blob-stream for these strings (without trying to parse as xml):

throwConfigExceptions="true"
throwExceptions="true"

this needs probably a regex. Search should be case insensitive and sinqle quote and spaces should also work.
Unit tests are needed for this, TTD recommended

Can I have a look at this issue?

Yes, that would be great :+1:

@304NotModified is there any time(future releases) by which this fix has to go in as I was unable to spend much time on it yet.

Not really, is a nice enhancement but not critical

WIP here -> #3521

@304NotModified Guess this has been resolved with NLog 4.6.7 ?

Yes indeed! Thanks!
Fixed by https://github.com/NLog/NLog/pull/3562


Moving space pins after scaling results in constant offset

Some background
I am an undergraduate research assistant working on an MR bridge inspection project.  I am developing an application for the HoloLens 2 to perform bridge inspections.  The application would overlay a virtual model (a digital twin, if you will) accurately over the physical bridge.  This is the main feature I have been working on, as accurate alignment of such as large model has proved to be a challenge.  The application would also allow a normal bridge inspection to take place, with all inspection data (condition of each component, photos, measurements, etc.) being recorded and linked via the virtual model.  This model would then be able to be analyzed at a smaller scale( say, on an office table) and with the linked inspection data visible in some form or fashion.
Before starting this project in November 2021,  I had little to no previous experience in MR, Unity (including WLT and MRTK), C#, and developing in the Windows environment.   I am majoring in Computer and Electrical Engineering.  I take classes and work (<15 hrs/wk), so things are slow going.  I appreciate any feedback and help filling the gaps in my knowledge.
My Issue
I am trying to use space pins to align a bridge model at variable scale levels that can be changed at runtime.  I have a bridge model and space pins under the same Unity hierarchy so that when the bridge model is scaled, the pre-placed space pins will scale with it and stay put relative to the model features.  The issue I am having follows this general procedure:

I load up my Unity-created application in the HoloLens 2.  My bridge model and corresponding space pins are loaded in at around a couple meters in length.
I manipulate the space pins (before any manual scaling) and they accurately are placed.  Everything works as it should (as far as my understanding goes) and model placement shifts to reflect my pin placement as I move around.
I scale the model up (including the space pins).
With the now larger model, I grab a space pin and attempt to reposition it.  I am able to move it to a location and it accurately tracks my grab.  When I release the space pin to end manipulation, it appears to jump a certain distance away from where I released it (I think proportional to how much I scaled up).

It seems to me like WLT is registering my new adjustment of space pins differently after I scale them up.  To be clear, I have tested this inside Unity and the transform position of the space pins is not moving but rather the adjustment object containing the main camera. As far as I understand, this is the correct behavior and the fundamental way WLT keeps objects aligned.
I did some testing, and simply grabbing the space pin and releasing it (not moving it at all, or at least a negligible amount) still results in the space pin "jumping" some constant offset.  Also, it seems to offset the space pin a constant amount no matter how much I move it.  Simply grabbing and releasing the space pin results in the same offset as if I moved it 1 meters.
The tools I am using are:

Unity 2020.3.30f1
MRTK v2.7.3 (for all packages)
WLT v1.5.8 (Core and Samples)
HoloLens 2

Additional Notes

I am using the 'SpacePinOrientableManipulation.cs' from the example namespace.  I plan to change this in the future but have been using it for simplicity's sake.
I have tried moving the space pins into their own parent object and having a script update their scale to match the scale of the bridge model.  The same behavior resulted.
The space pins work at any scale as long as that scale is set in Unity before the project is built or tested.  A potential strategy to avoid this issue could be just having two bridge models at different scales and/or only activating the space pins when the model is at a 1:1 scale.  This would be possible but could create issues down the road.  I haven't been able to test such as setup yet, but am confident it could work.
I can submit photos and files as needed, but wasn't sure what would be most helpful. I am working out of a private GitHub repo right now, but could make it public or share the project files if that would be helpful.

Thank you!!

Thank you for a most excellently detailed report. This is an area in which I am very interested. In particular, scale in the transform chain has been under-tested.
Sadly, as happy as I am to receive your excellent issue report, the timing is a tad off, as I am about to take some time off. This will be top of my agenda when I am back to work.
Many thanks again for the excellent issue details!

Thank you for your fast reply!  I'm glad the detail wasn't too much.
I will be off all next week on spring break and won't be working anyways, so the timing is fortunate.  I'll look forward to discussing this issue with you when you come back to work!

I've made some progress here. I was able to repro the issue, and even fix it (or workaround it, I'm not sure yet).
After scaling the scene up or down, resetting the space pins makes subsequent space pin manipulations behave correctly.
You can find sample code for resetting the space pins in the AlignmentControl.cs script in the WLT Examples/Scripts.
It makes sense, after rescaling the space pins, their old poses are meaningless, and the system needs to be notified to discard them. But I want to make sure I'm fully understanding what's going on.

I haven't had much time to focus on this issue lately, but I was able to do some thinking and came to the same realization.  I hadn't had time to test it yet, so I'm glad to hear it worked for you.
When I get time, I will look into the AlignmentControl.cs script and see if I can apply the space pin reset everytime I resize or reposition my model.  It shouldn't take too much work.  I'll let you know how it goes.

From: Mark Finch @.>
Sent: Friday, April 22, 2022, 10:10 AM
To: microsoft/MixedReality-WorldLockingTools-Unity @.>
Cc: Daniel McDonald @.>; Author @.>
Subject: Re: [microsoft/MixedReality-WorldLockingTools-Unity] Moving space pins after scaling results in constant offset (Issue #275)
I've made some progress here. I was able to repro the issue, and even fix it (or workaround it, I'm not sure yet).
After scaling the scene up or down, resetting the space pins makes subsequent space pin manipulations behave correctly.
You can find sample code for resetting the space pins in the AlignmentControl.cs script in the WLT Examples/Scripts.
It makes sense, after rescaling the space pins, their old poses are meaningless, and the system needs to be notified to discard them. But I want to make sure I'm fully understanding what's going on.
—
Reply to this email directly, view it on GitHubhttps://github.com/microsoft/MixedReality-WorldLockingTools-Unity/issues/275#issuecomment-1106610309, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AULF2IESQARMMYDOEC6VWCTVGK6P7ANCNFSM5RU5FJOA.
You are receiving this because you authored the thread.Message ID: @.***>

Hi @demcdonald2000 , I spent some more time on this over the weekend. I'm afraid I was wrong in my initial evaluation.
I was failing to take scale into account in my transform chain caching. So when the scale changes after the modeling pose is captured in Start(), well, as you have seen, things go wrong.
The good news is that I have a fix. It needs a lot of testing but seems safe. I'm confident it will be in the next release.
The even better news is that you can workaround the issue by calling ResetModelingPose() on the SpacePins that get moved because of a rescaling up the scene graph hierarchy. I also suggest calling Reset() on them. So you would collect all of your affected SpacePins, and then iterate over them like so:
foreach( var pin in myPins)
{
    pin.Reset();
    pin.ResetModelingPose();
}

This same sequence is suitable, but unnecessary, for SpacePins that have been moved by the translation or rotation of a hierarchy.
I'll note that it is also necessary if the LOCAL position or rotation of a SpacePin is changed after Start().
I'll leave this issue open until the fix gets merged in, but I hope that's enough to unblock you.
Thanks again for your detailed report!


Missing description of {tenant} substitution

The starter pack policy files include a value substitution item in the METADATA and authorization_endpoint metadata elements using the syntax {tenant}:
<Item Key="METADATA">https://login.microsoftonline.com/{tenant}/.well-known/openid-configuration</Item>
<Item Key="authorization_endpoint">https://login.microsoftonline.com/{tenant}/oauth2/token</Item>
This syntax and how it can be used for this or other OIDC endpoints appears to be undocumented here.

Is {tenant} the only available value?
Are other values available?
Does this substitution apply to all metadata values in this TP, or just those two?

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

ID: 8cb586f8-c1ff-7676-08b3-915ee00383b0
Version Independent ID: f1770ad7-7ee4-090b-78f8-85924233b190
Content: Define an OpenID Connect technical profile in a custom policy - Azure AD B2C
Content Source: articles/active-directory-b2c/openid-connect-technical-profile.md
Service: active-directory
Sub-service: b2c
GitHub Login: @msmimart
Microsoft Alias: mimart

Thanks I'm also adding the issuer metadata.


New version: Google.AndroidStudio version 2<IP_ADDRESS>

Pull request has been created with komac v2.1.0 :rocket:
Microsoft Reviewers: Open in CodeFlow

Automatic Validation ended with:

2024-03-19T21:33:18.4174061Z ##[error] Manifest Error: DisplayVersion declared in the manifest has overlap with existing DisplayVersion range in the index. Existing DisplayVersion range in index:  [ [2023.2, 2023.2]]

(Automated response - build 870.)

Enter a pull request number: 145289

Manifest validation succeeded.
Found Android Studio [Google.AndroidStudio] Version 2<IP_ADDRESS>
This application is licensed to you by its owner.
Microsoft is not responsible for, nor does it grant any licenses to, third-party packages.
Downloading https://redirector.gvt1.com/edgedl/android/studio/install/2<IP_ADDRESS>/android-studio-2<IP_ADDRESS>-windows.exe
  ██████████████████████████████  1.07 GB / 1.07 GB
Successfully verified installer hash
Starting package install...
Successfully installed

Name          : Android Studio
Publisher     : Google LLC
Version       : 2023.2
ProductCode   : Android Studio
Registry Path : Computer\HKEY_LOCAL_MACHINE\SOFTWARE\Microsoft\Windows\CurrentVersion\Uninstall\Android Studio
Uninstall     : winget uninstall --id "Android Studio"

Press any key to continue . . .

This manifest has the same DisplayVersion as the previous version 2<IP_ADDRESS>: "2023.2".

?

@wingetbot run

Automatic Validation ended with:

2024-03-21T18:35:41.8273760Z ##[error] Manifest Error: DisplayVersion declared in the manifest has overlap with existing DisplayVersion range in the index. Existing DisplayVersion range in index:  [ [2023.2, 2023.2]]

(Automated response - build 872.)

cba


📎 All errors into warnings option

Description
An option (--warn-on-errors) to transform all errors into warnings, just like --error-on-warnings
https://discord.com/channels/1132231889290285117/1132231889911029825/1280516163071119411

I think there's some misunderstanding.
--error-on-warnings doesn't do any transformation, it simply instructs the CLI to exit with an error code if there's some warnings.
Can you please update the description and add some clarification?

I had the same misunderstanding. Sorry for spreading it! In that case we should ceratinly wait Biome 2.0 and investigate way to do that in the Biome configuration file.

Do what, sorry? I think there's something missing

Do what, sorry? I think there's something missing

So basically someone asked if there's a way to set all errors to warnings. (without manually setting them each in the biome config) So we thought the --error-on-warnings was the same, but the other way around (setting warnings to errors), turns out it's something different.
Should we then change the scope of this ticket to a new configurable option which could do both? (Setting all rules to warnings of errors)

Thank you for providing some context, that's definitely clearer!
There's already a discussion where some user proposed that. However, we should definitely understand the requirements. Users usually want to downgrade linting errors to warnings with one simple config.
Before jumping into that, we should provide a very reasonable rationale about the feature. This is important because we need to justify why a feature like this exist.
Second, what about formatting errors? And parsing errors? Should we treat them differently or not?
Once we answer all these questions, I believe we should have enough information for code requirements and documentation

Bit of a late response;
But I think a lot of info on the why part is answered here; https://github.com/biomejs/biome/discussions/3106

Ok, so the actual reasoning is to have warnings in the LSP (editor) but keep the error in CI?
I still struggle to understand the use case. The discussion has various comments that have different use cases and preferences, but I'm still worried about the confusion that might cause.
I understand the use case of lowering down the severity of all rules in the configuration, but I still don't understand why lowering down the severity only in the editor.
So, which one is it @Netail ?


flyctl: 0.0.260 -> 0.0.271

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/superfly/flyctl/releases.
meta.description for flyctl is: "Command line tools for fly.io services"
meta.homepage for flyctl is: "https://fly.io/"
meta.changelog for flyctl is: ""
Updates performed

Golang update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
/nix/store/chfq06k8jran7zlgdrj8xm0m0bb1jglg-flyctl-0.0.271/bin/flyctl passed the binary check.
1 of 1 passed binary check by having a zero exit code.
0 of 1 passed binary check by having the new version present in output.
found 0.0.271 in filename of file in /nix/store/chfq06k8jran7zlgdrj8xm0m0bb1jglg-flyctl-0.0.271

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

4 total rebuild path(s)

2 package rebuild(s)

2 x86_64-linux rebuild(s)
2 x86_64-darwin rebuild(s)

First fifty rebuilds by attrpath
docbookrx
flyctl

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/chfq06k8jran7zlgdrj8xm0m0bb1jglg-flyctl-0.0.271 \
  --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 flyctl https://github.com/r-ryantm/nixpkgs/archive/d66971e557bdf7219bb43b6e00bdef8841fbbcbc.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/chfq06k8jran7zlgdrj8xm0m0bb1jglg-flyctl-0.0.271
ls -la /nix/store/chfq06k8jran7zlgdrj8xm0m0bb1jglg-flyctl-0.0.271/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:

flyctl

Maintainer pings
cc @aaronjanse @kalbasit for testing.

Obsolete ; newer versions upstream


Add support for the Zicond ISA extension

The upcoming Zicond ISA extension introduces two new instructions:

czero.eqz
czero.nez

This patch adds these instructions to template.yaml.
The Zicond ISA extension is documented here:
https://github.com/riscv/riscv-zicond/blob/main/zicondops.adoc

Please add an example cgf file with the coverpoints also under the sample_cgfs directory. Further, you will need to add a changelog entry and a version update before the PR can be merged.

I have added the zicond.cgf that I have added in the ACT PR and the change log entry.


Expand configuration for Elastic to support serverless

Is your feature request related to a problem? Please describe.
We were looking to use Elastic's serverless option for our visibility store, but to connect we would need to set an Authorization header, which currently doesn't appear to be supported by the config for the ES client: https://github.com/temporalio/temporal/blob/66ab754e7e8d60421678457df1914fa7ddae9dc3/common/persistence/visibility/store/elasticsearch/client/config.go#L45-L60
Describe the solution you'd like
I'd like to be able to add a configuration like the following:
elasticsearch:
  # ...
  headers:
    Authorization: "ApiKey {{ .Env.TEMPORAL_ES_VISIBIILITY_STORE_PASSWORD }}"

Describe alternatives you've considered
Another option would be to have this kind of a format:
elasticsearch:
  # ...

  auth:
    basic:
      username: "..."
      password: "..."
    # or
    elastic:
      apiKey: "..."

But it feels far less flexible and a bit more complicated to implement.
Other than that, I can't really think of any alternative, besides having to set up a proxy that adds the header. Ideally, we avoid the operational overhead of another proxy and just do it straight on the client.

Feature Request: Elasticsearch client configuration for Authorization headers
Is your feature request related to a problem? Please describe.
Currently, the Temporal Elasticsearch client configuration does not support adding custom headers, specifically the Authorization header, which is required for connecting to Elastic's serverless Elasticsearch option. This limits the ability to use serverless Elasticsearch as a Visibility store.
Describe the solution you'd like
The ability to add custom headers to the Elasticsearch client configuration would resolve this issue. This could be achieved by adding a new configuration option like:
elasticsearch:
  # ...
  headers:
    Authorization: "ApiKey {{ .Env.TEMPORAL_ES_VISIBIILITY_STORE_PASSWORD }}"

Describe alternatives you've considered

Proxy solution: This involves setting up a separate proxy server that adds the Authorization header. However, this adds operational overhead and complexity to the deployment.
Predefined authentication types: Defining a pre-defined configuration for specific authentication types (e.g., basic auth, API key) offers less flexibility and might require more code changes.

Why the proposed solution is better
The proposed solution is more flexible and efficient. It allows users to specify arbitrary headers directly in the configuration, without the need for additional proxies or complex code changes. This makes it easier to integrate with services that require custom headers for authorization.
References

Elasticsearch Authorization
Temporal Elasticsearch client configuration

Thanks for the feature request! This is a good idea. It would definitely add flexibility to how users configure the ES client.
As a workaround, you can achieve similar functionality by configuring a proxy that adds the authorization header.
Another option would be to set up the Elastic Serverless option with basic authentication and provide the username and password in the elasticsearch section of the Temporal configuration. This would not be as flexible as adding headers, but it would be simpler to implement.
Let us know if you have any other questions or requests.

Thanks for the feature request. You can workaround this by setting up a proxy that adds the Authorization header.


FOR EVERYONE: Refactor Cord API Repository Files

Create tickets as needed. Add lines here to let others know.

https://github.com/CordField/cordtables/issues/287

https://github.com/CordField/cordtables/issues/288

https://github.com/CordField/cordtables/issues/289

https://github.com/CordField/cordtables/issues/294

#311
#312
#313
#314

Ethnologue

[ ] Create
[ ] Read
[ ] Delete
[ ] Update

Funding Accounts

[ Cord api Repo: Funding Accounts]

Field Zones

Cord api Repo: Field Zones

Field Regions

Cord api Repo: Field Regions

Locations

Cord api Repo: Locations