Computational errors

Description
This PR changes the output type of computeProof to ComputationResult which can return an error value if the computation fails.
Checklist

[x] Commit sequence broadly makes sense and commits have useful messages
[ ] Code is formatted according to CONTRIBUTING.md
[ ] Self-reviewed the diff

There's a missing Generic instance: https://github.com/input-output-hk/formal-ledger-specifications/actions/runs/7194392950/job/19594830383?pr=328#step:5:9062
I've seen the nix errors on some other PRs as well, I assume something with hydra changed/broke.

The instance also needs a ToExpr constraint: https://github.com/input-output-hk/formal-ledger-specifications/actions/runs/7196262284/job/19600726021?pr=328#step:5:5353


docs: Add new class recordings to documentation

Added recording for class on 2023-10-15 titled "敘述文"

These recordings have been added to the 'recordings.md' file in the 'docs/resources' directory. The links to the recordings are hosted on Dropbox.

Going to merge directly so that we unblock the rest of the recordings.


check_paradigm should check for invalid keys in passed dict

Using the old nipy user logic, I passed amplitude=somethx instead of modulation=somethx in the make_design_matrix. I didn't crash but the values where ignored (e.g Error: unknown param, etc.). A default value of 1 was forced...

We could have warning when unexpected keys are given in events files.


[PT Run > Wox.Infrastructure.Image] Imrpove thumbnail loading and fix dispatcher crash

Summary of the Pull Request
This PR improves thumbnail loading and fixes the dispatcher crash in thumbnail loading code caused by the thumbnail preview handler of Adobe Acrobat Reader. With the changes the crash should be fixed and it should show the file type icon for PDF files instead.
IMPORTANT: The PR leads to a change in PT Run's thumbnail behavior: We now only show thumbnails for files other than images, if the thumbnails already exist in Explorers thumbnail cache.
How to reproduce the crash:

Install Acrobat Reader, enable thumbnail provider.
Exit PT.
Delete the %localappdata% folder of PT Run.
Start PT Run and search for an PDF file.
Note: Step 2 and 3 can be replaced with creating a new PDF. Important here is that no image exists in PT Run's image cache for the file.

PR Checklist

[x] Closes: #18166
[x] Communication: I've discussed this with core contributors already. If work hasn't been agreed, this work might be rejected
[x] Tests: Added/updated and all pass
[ ] Localization: All end user facing strings can be localized
[ ] Dev docs: Added/updated
[ ] New binaries: Added on the required places

[ ] JSON for signing for new binaries
[ ] WXS for installer for new binaries and localization folder
[ ] YML for CI pipeline for new test projects
[ ] YML for signed pipeline

[ ] Documentation updated: If checked, please file a pull request on our docs repo and link it here: #xxx

Detailed Description of the Pull Request / Additional comments

Validation Steps Performed

@crutkas, @jaimecbernardo
I think I have fixed it. But can you please test it on Win11 and can we create a test build to let our users test it too. (Btw, this fix might be relevant for Peek too.)

What exactly does "We now only show thumbnails for files other than images, if the thumbnails already exist in Explorers thumbnail cache." mean?
As I do not have Adobe Acrobat thumbnails activated, I want to see the thumbnail for every file in the results - even if the Explorer hasn't generated any.
It works with PowerToys PDF explorer extensions, so why change that behavior. You could at least check whether this feature is enabled.
Or at least add a config option to turn it on my own risk...

@daniel-richter
This is a good point.
What I mean is we check if the file is an image. If not we request only the already cached thumbnail instead the thumbnail from file/without restrictions.
The problem is that the code to read the thumbnail hangs instead of returning an hresult code when the problem occurs.
What we can do is either a global setting or implement the cache requirement only for pdf files.
Additional there is this strange bug that if no chached thumbnail exists, an transparent rectangle is returned instead of the file icons. For this I added a workaround code that checks if it is a transparent image and if yes we request the icon then.

@jaimecbernardo  thoughts on this being a hot fix?

By e-mail we discussed for the hotfix it might be better to just try blacklisting Adobe Reader as a thumbnail generator somehow.

By e-mail we discussed for the hotfix it might be better to just try blacklisting Adobe Reader as a thumbnail generator somehow.

I know how. I will update the PR within the next days.

@jaimecbernardo , @crutkas , @daniel-richter
I updated the code to only skip PDF files and only if Acrobat is set as thumbnail provider/handler. The configured provider gets evaluated in real time. The evaluation happens, if no thumbnail exists in PT Run's cache and it is a PDF file.
It is working like a charm:

@jaimecbernardo
Can you please test it again and especially if our provider (PT PDF Thumbnail) is respected. And can you please check the grammar of the new comments.

@jaimecbernardo
I did some tweaking and bug fixing:

New: We cache the last result for 5 seconds to reduce the amount of Registry queries.
Fixed: The PDF app detection was wrong if the assignment was changed from Explorer instead of Adobe Reader.
Improved: The Adobe detection note is written again, if an other provider was used temporary.
Improved: Log text.
Code clean up and improved comments.

/azp run


Fixed typo in README.md, missing /config from install path

Hey @mattisdada , what's your OS? I don't have config on OSX, it's just in the WebIde100 dir.

Huh, I would have assumed it would be the same cross platform. But that was the directory for Windows. Same structure from .Webide60 - .Webide110

@mattisdada wanna change the PR to say  something like on windows: xxx and on unix-based platforms : xxx ?
It sounds simple, but I think it would help a bunch of people. :)

Yeah sure will
On 26 Jan 2016 7:11 PM, "Dayle Rees"<EMAIL_ADDRESS>wrote:

@mattisdada https://github.com/mattisdada wanna change the PR to say
something like on windows: xxx and on unix-based platforms : xxx ?
It sounds simple, but I think it would help a bunch of people. :)
—
Reply to this email directly or view it on GitHub
https://github.com/daylerees/material-peacock/pull/7#issuecomment-174884440
.

Just did this on Ubuntu, for reference the colours directory on linux platforms seems to be: ./WebIde100/config/colors

Must just be OSX that's different. :)


[FEATURE] Add support for ReadWriteOncePod access mode

Is your feature request related to a problem? Please describe (👍 if you like this request)
ReadWriteOncePod is another Kubernetes access mode that restricts access to just a single pod on a single node at any given time.
Describe the solution you'd like
Add support for it as another available option for Longhorn volumes.
Describe alternatives you've considered

Additional context
This [FEATURE] issue created to associate to community PR https://github.com/longhorn/longhorn-manager/pull/3236

@shikanime, please set yourself as "Assignee" and add a comment about the testing done so far on the feature.  Thanks!

Hello @james-munson, cannot assign to @shikanime, so assign to you first. You can handle the remaining tasks. Thank you.

@james-munson Let's merge it to v1.9.


Bug: Unable to update BackgroundIMage in Category by providing url

What are you trying to achieve?
I am unable to update the BackgroundIMage in a Category by providing the url of the image.
Steps to reproduce the problem

Open Saleor GraphQL Playground or POSTMAN. Use TokenCreate API for generating token.
Use the token created to call CategoryUpdate API.
Give the details of an existing Category and provide sample url for the backgroundImage to update the background image.
(Please refer the sample mutation and input provided in the logs.)

What did you expect to happen?
We should be able to update the background image via url.
Logs
Please find below CategoryUpdate mutation, input and response  for recreating scenario,
mutation CategoryUpdate($id: ID!, $input: CategoryInput!) {
categoryUpdate(id: $id, input: $input) {
category {
...CategoryDetails
__typename
}
errors {
...ProductError
__typename
}
__typename
}
}
fragment CategoryDetails on Category {
id
...Metadata
backgroundImage {
alt
url
__typename
}
name
slug
description
seoDescription
seoTitle
__typename
}
fragment Metadata on ObjectWithMetadata {
metadata {
...MetadataItem
__typename
}
privateMetadata {
...MetadataItem
__typename
}
__typename
}
fragment MetadataItem on MetadataItem {
key
value
__typename
}
fragment ProductError on ProductError {
code
field
message
__typename
}
Input:
{
"id": "Q2F0ZWdvcnk6Mg==",
"input": {
"backgroundImage": "https://m.media-amazon.com/images/I/515gKnUgwpL.SX679.jpg",
"description": "{}",
"name": "Testdata",
"seo": {
"description": "SeoDescription",
"title": "SeoTitle"
},
"slug": "test-data",
"metadata":[
{
"key":"testdata",
"value":"testdata"
}
],
"privateMetadata":[
{
"key":"testdata",
"value":"testdata"
}
]
}
}
Response:
{
"data": {
"categoryUpdate": {
"category": null,
"errors": [
{
"code": "REQUIRED",
"field": "backgroundImage",
"message": "File is required.",
"__typename": "ProductError"
}
],
"__typename": "CategoryUpdate"
}
},
"extensions": {
"cost": {
"requestedQueryCost": 1,
"maximumAvailable": 50000
}
}
}
Environment
Saleor version: 3.18.10
OS and version: Ubuntu 22.04.3 LTS

Hi Team, please let me know if any further details are required regarding this issue or to recreate the issue. Also, please let me know if there is any workaround to upload images via url.

Hi Team, please let me know if any further details are required regarding this issue or to recreate the issue. Also, please let me know if there is any workaround to upload images via url.

Hi @jyo-this,
You need to send the BackgroundImage with the use of the multipart request.


napari only opens first frame of SPE time series.

🐛 Bug
When napari is made to open a SPE (https://imageio.readthedocs.io/en/stable/format_spe.html#spe) time series file, which e.g. imageio.volread reads as a proper 3d array (t, y, x), only the first frame is displayed.  Compare with e.g. opening a tiff time series, for which napari correctly displays a slider for selecting the frame number.
To Reproduce
Steps to reproduce the behavior:

Run napari /path/to/file.spe.  Unfortunately I don't have a small file I can share right now.
Observe the behavior described above.

Expected behavior
A slider for selecting the frame number appears.
Environment

Please copy and paste the information at napari info option in help menubar here:
napari, 0.2.5
Qt, 5.13.2
NumPy, 1.17.4
SciPy, 1.3.2
VisPy, 0.6.2
scikit-image, 0.16.2
Platform: Linux-5.3.8-300.fc31.x86_64-x86_64-with-glibc2.10
Python:   3.8.0 (default, Nov  6 2019, 21:49:08)  [GCC 7.3.0]
Backend:  PyQt5
GL version:  '3.0 Mesa 19.2.2'
MAX_TEXTURE_SIZE: 16384

Any other relevant information:

Additional context

Thanks for flagging @anntzer - our fileIO is still pretty early, though if imageio.volread does the right thing, we should be able to too. We'll look into it

Yeah the problem here is that we are using skimage.io.imread, which uses <backend>.imread, where backend is either tifffile for tiffs or imageio.imread for everything else. See this comment on skimage #4138 for more.
I honestly don't know what the right thing is here, because it's hard to tell ahead of time whether volread or imread is the correct call... Here's me reading a 3-channel tiff with different calls:
In [1]: import imageio as iio
In [2]: result = iio.volread('MAX_block6_tile.lsm_cropped.tif')
In [3]: result.shape
Out[3]: (3, 1024, 1024)

In [5]: from skimage import io
In [6]: result2 = io.imread('MAX_block6_tile.lsm_cropped.tif')  # uses tifffile
In [7]: result2.shape
Out[7]: (1024, 1024, 3)

In [8]: result3 = iio.imread('MAX_block6_tile.lsm_cropped.tif')
In [9]: result3.shape
Out[9]: (1024, 1024)

Wouldn't it work to always use imageio.volread (and check if the first dim is 1)?

I've found tifffile to be a much more reliable loader for tiffs, so your question might be better suited for scikit-image, where we already implement the tifffile > imageio cascade. I don't think I want to repeat that logic here... Could you maybe raise an issue on the scikit-image repo?
@sofroniewn might have different ideas about how much we want to take on in napari proper.

Ideally I’d much rather see something like this get solved upstream in skimage / imageio in a way where we don’t loose any of our existing functionality.
I worry if we start trying to develop special case logic inside napari it will be hard to test and maintain, and either that logic is generic enough that it should be upstream or so special case that it should be in a special fileIO plugin

To be honest, I sort of lost track of the layers between skimage, imageio, and tifffile/PIL.  I hope you (@jni) will consider the following (which is mostly addressed to you as a skimage devel rather than a napari devel) as constructive criticism but... why do skimage and imageio both vendor tifffile (different versions of it, as of master!) rather than depend on it? (they already have a dependency on numpy, which is something much more complex)  Why does imageio exist as something independent of PIL (which it mostly wraps), rather than having been contributed as PRs to pillow?  With all due respect, it's just too hard to keep track of all the different projects.
I guess that based on the above, I would agree with @sofroniewn's last comment -- adding yet another layer in napari would only make things worse.

why do skimage and imageio both vendor tifffile (different versions of it, as of master!) rather than depend on it?

Tifffile is essentially impossible to depend on, because development happens on Christoph Gohlke's computer, then it gets published to his homepage and to PyPI. The "issue tracker" is emailing Christoph. (Or, more often, pinging him on scikit-image.) For a while, Steven Sylvester tried to maintain a mirror on GitHub, but as you can imagine, mirrors are not a fun thing to maintain, so that effort was deprecated. (FYI I don't want to tag either of them because, honestly, Christoph's work has been an incredible boon to bioimaging in Python, and he is entitled to work as he chooses. If that's Don Knuth style, so be it. And Steven certainly has enough on his plate also, without rehashing a discussion he's probably seen dozens of times already.)
The current version of tifffile on PyPI does not support most compression schemes without also installing imagecodecs/imagecodecs-lite, neither of which provides wheels except for windows. For scikit-image, that's an unacceptable dependency. Hence, vendoring.

Why does imageio exist as something independent of PIL (which it mostly wraps), rather than having been contributed as PRs to pillow?

Tifffile, and I imagine many other similar reasons. imageio was supposed to solve the same problem that skimage.io does, which is that there's lots of libraries, none of them provide NumPy arrays under a unified API. Pillow in particular, is more focused on "consumer" images than scientific ones. It does not depend on NumPy nor give you a NumPy array as an output. The goal was for skimage.io to eventually be replaced by imageio, but both of them have a maintenance burden that is too great, and a concerted effort to unify the two has not been made because there are plenty of other fires to put out.

Thanks for the explanation, and sorry for distracting from the main issue at hand.  To be honest I am not convinced that the current state of things is optimal, but I cannot articulate this properly and do not wish to appear as belittling the very worthwhile work of the skimage community (in a broad sense), so let's close this side discussion.

Oh I absolutely don’t think it’s optimal! I was merely trying to explain how we might have got to where we are by doing gradient descent along various axes and given various constraints. ;) And that it probably requires a major energy kick to move to another basin!
As to articulating where that other basin might be, I don’t think PIL and imageio are redundant, but I do think that skimage.io and imageio are redundant, and that would be where I would focus my efforts in the short term if I wanted to optimise further.

I think this issue should now be resolved with an imageio reader plugin that uses volread by default, or maybe creates a custom lazy volread. See our plugin docs for more info.

@anntzer Thanks for raising this issue. There's now several IO plugins for napari in the wild, and it should be easy to create one that suits your needs. (link again) Given that, I'm going to close this issue, but feel free to comment here if you have any different thoughts on how to fix the issue!


[google_sign_in] Switch Android to Credential Manager

Use case
Google Sign-In for Android is deprecated and will be removed from the Google Play Services Auth SDK. (com.google.android.gms:play-services-auth) in 2025
Proposal
there's already a community package but having an official one would be better, since it's such core concern

Yes now the only way to do it is via custom method channels or using 3rd party community plugin, it would be nice to have an official source

Based on this blog post Google Sign-In for Android will be removed in H2 2025 specifically.

I am hoping to publish a wrapper plugin in early (Q1) 2025, with the goal of having it be a replacement backend for the google_sign_in plugin (although we may publish a direct wrapper separately as well). This is likely to be paired with significant changes to the google_sign_in APIs, since every platform now has a different recommended underlying, so the current API may not be a good match (see https://github.com/flutter/flutter/issues/119300)
We will definitely have something ready in advance of H2 2025.

It seems quite urgent.


Add ENV var for OSImage default

Also cleans up some leftover github workflows for image downloader.

"error: You must be logged in to the server (Unauthorized)", "stderr_lines": ["error: You must be logged in to the server (Unauthorized)"], "stdout": "", "stdout_lines": []}

recheck


View MermaidJS Graph passing the code or URL

I would like to have a feature where I can pass the mermaidjs code through URL, so It can be displayed on the screen:
Example:
https://mermaidjs.github.io/mermaid-live-editor/#/view/?code=graph TD; A[Christmas] -->|Get money| B(Go shopping);

FYI.
I've made a PR and service to support embed image from code (#52).
You can now directly use base64 encoded mermaid code and get the image from mermaid.ink service:
For example, get base64 encoded mermaid code from mermaid-live-editor:
c2VxdWVuY2VEaWFncmFtCiAgICBwYXJ0aWNpcGFudCBBbGljZQogICAgcGFydGljaXBhbnQgQm9iCiAgICBBbGljZS0-Sm9objogSGVsbG8gSm9obiwgaG93IGFyZSB5b3U_CiAgICBsb29wIEhlYWx0aGNoZWNrCiAgICAgICAgSm9obi0-Sm9objogRmlnaHQgYWdhaW5zdCBoeXBvY2hvbmRyaWEKICAgIGVuZAogICAgTm90ZSByaWdodCBvZiBKb2huOiBSYXRpb25hbCB0aG91Z2h0cyA8YnIvPnByZXZhaWwuLi4KICAgIEpvaG4tLT5BbGljZTogR3JlYXQhCiAgICBKb2huLT5Cb2I6IEhvdyBhYm91dCB5b3U_CiAgICBCb2ItLT5Kb2huOiBKb2xseSBnb29kIQ

append the code after https://mermaid.ink/img/, for example:
https://mermaid.ink/img/c2VxdWVuY2VEaWFncmFtCiAgICBwYXJ0aWNpcGFudCBBbGljZQogICAgcGFydGljaXBhbnQgQm9iCiAgICBBbGljZS0-Sm9objogSGVsbG8gSm9obiwgaG93IGFyZSB5b3U_CiAgICBsb29wIEhlYWx0aGNoZWNrCiAgICAgICAgSm9obi0-Sm9objogRmlnaHQgYWdhaW5zdCBoeXBvY2hvbmRyaWEKICAgIGVuZAogICAgTm90ZSByaWdodCBvZiBKb2huOiBSYXRpb25hbCB0aG91Z2h0cyA8YnIvPnByZXZhaWwuLi4KICAgIEpvaG4tLT5BbGljZTogR3JlYXQhCiAgICBKb2huLT5Cb2I6IEhvdyBhYm91dCB5b3U_CiAgICBCb2ItLT5Kb2huOiBKb2xseSBnb29kIQ

you will get the jpg image:

@jihchi How can I generate those base64 encoded strings. I mean what is the algorithm? just put the graph as a text inside base64 algorithms?
Do I do this like
BASE64ENCODE(graph):
    return encoded_string

BASE64ENCODE("""
Graph TD
    A[Christmas] -->|Get money| B(Go shopping);
""")

Oh sorry, I've solved this. Using python and mermaid.ink service like this:
import base64

graph = """
graph LR;
    A-- 2 --->B;
    A-->C;
    B-->D;
    C-- 3 -->D;
"""

graphbytes = graph.encode("ascii")

base64_bytes = base64.b64encode(graphbytes)
base64_string = base64_bytes.decode("ascii")

print(f"Encoded string: {base64_string}")
import requests, io
from PIL import Image
import matplotlib.pyplot as plt
img = Image.open(io.BytesIO(requests.get('https://mermaid.ink/img/' + base64_string).content))
plt.imshow(img)

This reconstructs the graph like:

FYI for future peeps running into this issue, follow their serialize/de-serialize function
https://github.com/mermaid-js/mermaid-live-editor/blob/b5978e6faf7635e39452855fb4d062d1452ab71b/src/lib/util/serde.ts#L19

Basically, it is:
import { deflate } from 'pako';
import { fromUint8Array } from 'js-base64';

const formatJSON = (data: unknown): string => JSON.stringify(data, undefined, 2);
const serialize = (state: string): string => {
  const data = new TextEncoder().encode(state);
  const compressed = deflate(data, { level: 9 }); // zlib level 9
  return fromUint8Array(compressed, true); // url safe base64 encoding
}

const defaultState = {
  code: `flowchart TD
    A[Christmas] -->|Get money| B(Go shopping)
    B --> C{Let me think}
    C -->|One| D[Laptop]
    C -->|Two| E[iPhone]
    C -->|Three| F[fa:fa-car Car]
  `,
  mermaid: formatJSON({
    theme: 'default'
  }),
  autoSync: true,
  updateDiagram: true
};

const json = JSON.stringify(defaultState);
console.log(json)
const serialized = serialize(json);
console.log(`pako:${serialized}`);

Oh sorry, I've solved this. Using python and mermaid.ink service like this:
import base64

graph = """
graph LR;
    A-- 2 --->B;
    A-->C;
    B-->D;
    C-- 3 -->D;
"""

graphbytes = graph.encode("ascii")

base64_bytes = base64.b64encode(graphbytes)
base64_string = base64_bytes.decode("ascii")

print(f"Encoded string: {base64_string}")
import requests, io
from PIL import Image
import matplotlib.pyplot as plt
img = Image.open(io.BytesIO(requests.get('https://mermaid.ink/img/' + base64_string).content))
plt.imshow(img)

This reconstructs the graph like:

This works.


remove intersect_known_variants and rename filter_known_variants

remove intersect_known_variants and rename filter_known_variants to annotate_known_variants
related to https://github.com/wustl-oncology/analysis-wdls/issues/103

I'm testing this PR with a real dataset now.

My test succeeded.  In this case I expected no impact whatsoever on variant results.  I confirmed that the results are identical with and without this PR applied.


Customise global search button

How do I customize the Global Search edit box without adding a button?  I want to:

Make the Search button right justified like in react-bootstrap v1
Fire an update when the search edit box changes that updates the route/URL to enable a permalink

Below is what I have (default)
Many thanks in advance.
return (

      <ToolkitProvider
        keyField="id"
        data={rows}
        columns={envColumns}
        search={{ defaultSearch: '' }}
      >
        {
          props => (

              <SearchBar {...props.searchProps} />
              <BootstrapTable {...props.baseProps} filter={filterFactory()} condensed striped hover />

          )
        }
      </ToolkitProvider>

  )

@tester5 hello, The ability to custom on SearchBar component is limited, at least you can update class, style placeholder etc. but if you want to do more advance customization, you can check this example.
i think react-bootstrap-table2 is more customizable than version 1, we already allow developer to rearrange the position of component or even repaint whole the part of search, pagination etc.
Please pin me here or open another issue if you have any question. thanks

Hi @AllenFang many thanks for the message above, it was helpful.  Was able to customise the Search bar using this example: https://react-bootstrap-table.github.io/react-bootstrap-table2/storybook/index.html?selectedKind=Table Search&selectedStory=Fully Custom Search&full=0&addons=1&stories=1&panelRight=0&addonPanel=storybook%2Factions%2Factions-panel
However I also wish to be able to pass in a filter URL param like so:
/view/0?filter=check

where "check" would be updated in the input field value.  It actually does this and I am able to update the route by typing into the text box as well as get value updated on page refresh.
However, the filter is not activated on page refresh, and currently you have to hit enter (i.e. activate onKeyPress) to get the filter to apply on initial page refresh.  Any idea how this can be achieved?
This is a snippet of the custom search component I created.
const MySearch = (searchProps) => {
    let input
    const handleSearch = () => {
      searchProps.onSearch(input.value)
      props.history.push(`${baseUrl}?filter=${input.value}`)
    }

    return (

        {defSearch ? (

            <input
              id="searchInput"
              className="form-control"
              style={{ backgroundColor: 'white', width: '100%', marginBottom: '1px' }}
              ref={n => input = n}
              type="text"
              autoFocus={true}
              onChange={handleSearch}
              onKeyPress={handleSearch}
              title="Hit any key to filter..."
              value={defSearch}
            />
            Auto refresh
            <AutoRefreshButton />

        ) : (

              <input
                id="searchInput"
                className="form-control"
                style={{ backgroundColor: 'white', width: '100%', marginBottom: '1px' }}
                ref={n => input = n}
                type="text"
                autoFocus={true}
                onChange={handleSearch}
                placeholder="Start typing to filter..."
                title="Start typing to filter..."
              />
              Auto refresh
              <AutoRefreshButton />

          )}

    )
  }


Support for RDS

The Office version support apply on user installations only or also to RDS scenarios of Office installs? Like Shared Computer Activation?

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

ID: e0836a95-d3b7-3c43-d56b-5fb4ceeb4765
Version Independent ID: 8a0d366b-5ff7-22de-7aba-6adbe0ea94ea
Content: Sensitivity labels in Office apps - Microsoft 365 Compliance
Content Source: microsoft-365/compliance/sensitivity-labels-office-apps.md
Service: o365-seccomp
GitHub Login: @cabailey
Microsoft Alias: cabailey

@Kazzan You need the AIP Client on the RDS Server for it to work:
https://docs.microsoft.com/en-us/azure/information-protection/requirements

The Azure Information Protection clients are supported for Remote Desktop Services. If you delete user profiles when you use the Azure Information Protection clients with Remote Desktop Services, do not delete the %Appdata%\Microsoft\Protect folder.

Hi @VLG17, so the only MIP (AIP) Unified Client is supported on RDS and not the native "Sensitivity" button labeling in Office on RDS? Correct? Thanks for clarification.

Correct. You need the client, either AIP or the new Unified Labeling one. As far as i know, support for the native "Sensitivity" button labeling will come in the future.

@Kazzan  I checked up on this with the engineering team and was told that built-in sensitivity labels are supported with Remote Desktop Services.  So you don't need the AIP client to support this Office deployment.  Now added to the client comparison table: https://docs.microsoft.com/en-us/azure/information-protection/rms-client/use-client#compare-the-labeling-clients-for-windows-computers


Request for stu to clean up dataprep folder

can you either remove unnecessary files from this folder and/or better yet remove/relocate the folder so it's not in the root of the project?

also remove /lib and /src/main/main.iml if possible

I already did this, closing the issue -David on pêlers account


No Motion Data

I Tried gamecontroller.GameActivity Project, the buttons is ok, but i can't get motion and accelerometer data, which display none. Can I figure out how to debug it?

Accelerometer/motion data has requirements:

A compatible controller (currently PS4 and PS5 for motion data)
Android 12 or later
Device is running a kernel version that include the updated driver. Not all Android 12 devices have the relevant kernel upgrades. Recent Pixel phones, for example, include the required driver.

The short answer is if you are using a PS4 or PS5 controller on Android 12 or later, you should get motion data if your device has the appropriate kernel driver. If not, it won't be displayed.


[Bug]: Monkey365 is returning an empty plugin list when string not match

What happened?
The internal function that manages plugins is case sensitive, so is returning an empty array when the string passed to the -Analysis variable does not match.
How to reproduce it
The following command will fail, because a capital letter in "SharePointOnline" is wrong.
Invoke-Monkey365 -Instance Microsoft365 -Analysis SharePointOnline -Verbose -Debug -InformationAction Continue -ExportTo HTML

As a workaround, the following command can be used (Please, note the lack of capital letter in SharePointOnline):
Invoke-Monkey365 -Instance Microsoft365 -Analysis SharepointOnline -ExportTo HTML

Fixed in both main in develop branches


Include react-router-hash-link in react-router

Hi. I think it would be super useful to include https://github.com/rafgraph/react-router-hash-link functionality into this repository (copy and paste). So Link and NavLink components would be able to support # refs
<Link to="/some/path#with-hash-fragment">Link to Hash Fragment</Link >
I was super confused to know that React-Router can't handle this yet.
I'd like to describe my situation better. We have our internal library that extends functionality of React-Router by adding localization feature to the references. So, instead of Link we use LocalizedLink to have refs like /en/sdasdas.
Well, now,  I need to add some # refs, but because React-Router doesn't support it, our library doesn't support it as well.
Other people may have similar use cases and after all, as I understand it, React-Router was meant to cover all what regular  refs can do. If so, then it should be included to avoid other confusions.
I'd like to implement it if it's supported.

This is really functionality that should live elsewhere. It's not so much that React Router doesn't support hash links, but more that the content for the hash is generally not available in the browser when it would scroll to it, as React hasn't rendered it yet. That's not really React Router's domain, as we just care about mapping URLs to component trees and not the states of those components.
The core of React Router should definitely allow this kind of behavior, but shouldn't support every bit of functionality or use case as a built-in. We let others add it in so differing use cases can be handled in more specific ways. Trying to apply a one-size-fits-all approach in the core of the library isn't likely to make some people happy.

Hi! Yes, I see your point, one-size-fits-all is definitely a bad thing. At the same time one-size-undercover-some-basics also seems like an opposite extreme. The right answer somewhere in the middle as usual I think. Imagine you would have two types of  tags to cover "regular" and hash refs, would it be better?

... we just care about mapping URLs to component trees ...

well, hash is a part of the URL standard.

generally not available in the browser when it would scroll to it,

hm..if it's not available, then why the hash ref should be available at the moment?
What's an example of what you've described? A long list of elements with the hash refs between them? So, lets say, we have a hash ref in one of the first elements which points out at one of the last elements (that hasn't been uploaded yet)? Is this the case?
Ok,,then.. maybe it's possible to pass a callback to upload the element  when the ref is clicked.
Well, ok, I'll do my workaround anyway, maybe I'll figure out something and bring it back) I can't use HashLink straight away though, as I described, so I have to something else.


Mock Drupal API for signup & reportback tests.

Changes

Test coverage restored & green! :traffic_light: Added tests for updated signup & reportback endpoints.
Mocks DrupalAPI class, so unit tests do not have external dependency on another app.

For review: @angaither @jonuy

mockery ++
:+1:


Milestone section is missing status in tasklist when adding project [4]

Describe the Bug:
Probability (how likely the bug is to happen, scored from 1-5): 2
(For example, probability of 5 is something like "it happens to all users every time they log in."
Probability of 1 "only happens to certain users when a really specific and unlikely path is followed.")
Effect (how bad the bug is when it does happen, scored from 1-5): 2
(For example, effect of 5 is "the entire app crashes and makes it unusable for all users" or "the bug causes the wrong data to be saved, with critical information (e.g. payment) being affected."
Effect of 1 is "It makes some styling look a little bit weird.")
Steps to reproduce the behaviour:

in dev
add a project
note that everything in the tasklist has a status except milestone reports

Screenshots:
https://www.loom.com/share/271d966cf1ea470a85a2f740c299c892
Additional information:

OS: [e.g. iOS]
Browser: [e.g. chrome, safari]
Version: [e.g. 22]
Additional context

still a bug Nov 30, 2022

A possible solution is to manually pass Not Started in the MilestoneTaskListItemsComponent when context.length === 0


Add locale labels for en_uk

BirdNET's English labels use US common names. This pull request adds UK common names to the list of locales.
The benefit will be clarification, for UK users, that results for species like Anas Crecca "Green-winged Teal" are actually a merged clsas "Eurasian/Green-winged Teal", and "Brant" - a rarity in the UK, is actually a record for "Brent Goose". Other common names, such as "Eurasian Thick-knee" will be given their local name "Stone-curlew". Additionally, UK spellings are used for "Grey", rather than "Gray".
A new label file for en_uk has been added, and also the translate.py file has the "en_uk" locale in the LOCALES list.
The eBird API download for the new locale has been tested and works, but the impact on the build for "BirdNET Analyzer GIU", whilst it should work,  has not been tested. I will defer to your exprerience on this, as I have not fathomed how it works.

LGTM, GUI should still work, because all files in the labels folder get packaged.

Thank you!

While using the GUI to batch analyze some files, I noticed that there are two 'UK' options showing up in the dropdown for the 'Locale' (see attached screenshot). Selecting either one translates the species names to Ukrainian and not English (UK). There is also no en_UK locale in the dropdown for the GUI.


fix

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (3389603) 92.82% compared to head (3389603) 92.82%.

:exclamation: Current head 3389603 differs from pull request most recent head 502f92f. Consider uploading reports for the commit 502f92f to get more accurate results

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@              Coverage Diff              @@
##           fix-ci-check      #44   +/-   ##
=============================================
  Coverage         92.82%   92.82%
=============================================
  Files               188      188
  Lines              5912     5912
=============================================
  Hits               5488     5488
  Misses              424      424

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Implements the spark and flink type task, now they can be cache the previous input

What is the purpose of the pull request
This PR is a subtask of #1675.
Implements the spark and flink type task, now they can be cache the previous input.
Brief change log

Edit resources.vue
Edit flink.vue
Edit spark.vue
Edit formModel.vue

Verify this pull request
This change added tests and can be verified as follows:

Manually verified the change by testing locally.

Codecov Report

Merging #1795 into dev will decrease coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##              dev    #1795      +/-   ##
==========================================
- Coverage   21.02%   21.01%   -0.01%
==========================================
  Files         292      292
  Lines       14069    14069
  Branches     2321     2321
==========================================
- Hits         2958     2957       -1
- Misses      10775    10776       +1
  Partials      336      336

Impacted Files
Coverage Δ

...scheduler/alert/template/AlertTemplateFactory.java
42.85% <0%> (-14.29%)
:arrow_down:

Continue to review full report at Codecov.

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

@break60

+1


Added List deployment rest call sample

@willsmythe, I don't have permission to merge the changes. Can you please merge the changes. Thanks.


Article may have incomplete information

There are references to 4.8 .Net framework in the install.cmd code example but the if statements at lines 25-30 do not contain goto for 4.8.
There seems to be a section for :NDP48 with the correct netfxregkey decimal value.
The links in the REM statements in lines 4-10 do not download the filename expected in lines 32, 37, 42, 47, 52, 57, etc.

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

ID: f082a39c-4081-f5c3-1e1d-effcfecb2fd7
Version Independent ID: e57c7222-bb36-5fa1-bdc5-f8f3cd87e3d8
Content: Install .NET on Azure Cloud Services (classic) roles
Content Source: articles/cloud-services/cloud-services-dotnet-install-dotnet.md
Service: cloud-services
GitHub Login: @tanmaygore
Microsoft Alias: tagore

@jtbatzer Thanks for the feedback! I have assigned the issue to content author to check and update the document as appropriate.We will update you accordingly.

@tanmaygore Can you please check and add your comments on this doc update request as applicable.

Thanks for your dedication to our documentation. Unfortunately, at this time we have been unable to review your issue in a timely manner and we sincerely apologize for the delayed response. The requested updates have not been made since the creation of this issue, and the timeline for resolution may vary based on resourcing. We are closing this issue for now, but feel free to comment here as necessary. #please-close


Webhook signaling Not working

First of all, thank you for creating this great tool.
Secondly, I was trying to push notifications to my Signals Telegram Channel via Webhook, However, it does not reply to any request. Any advice?
Thanks

Hey @whaleagents ..
Which git branch are u using?
What is your config in "notifiers" section ?


Codec not found error

I am recording audio files through android using the MediaRecorder. When I try to process the audio file i get the following error:
"linc.com.amplituda.exceptions.processing.CodecNotFoundException: Failed to find codec!"

@gaynor Hi! This is cause Amplituda doesn't support AMR_NB codec.
Try to replace it on AAC, OPUS, VORBIS [. . .] and record the audio file with format according to one of the codecs.
I tested it with this code by replacing
setAudioEncoder(MediaRecorder.AudioEncoder.AMR_NB) to setAudioEncoder(MediaRecorder.AudioEncoder.AAC) and output audio file format to .aac

Awesome. That solved it. This library is a life saver :)

Do you get proper sample values? I am using the same configuration but I receive sample list not greater then 10. mostly they are 0s.
For example: [0, 0, 0, 0, 0, 2, 3, 1, 0, 0, 0, 0, 0, 0, 0, 0, 4, 6, 3, 2, 0, 1, 1, 3, 3, 1, 0, 0, 0, 3, 2, 1, 1, 1, 4, 0, 0, 1, 2, 2, 2, 3, 2, 1, 1, 1, 0, 0, 1, 2, 1, 0, 0, 0, 1, 2, 2, 2, 1, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 3, 6, 7, 1, 6, 5, 4, 2, 2, 1, 3, 1, 1, 1, 1, 0, 0]


feat(crypto): add bitcoin package with a small subset of utils and constants

Create a separate package for extension's bitcoin folder: https://github.com/leather-wallet/extension/tree/dev/src/shared/crypto/bitcoin
Use a small subset of constants and utils. Those should be moved to this repo in full in another PR

@pete-watters thank you a lot for suggestions!
This packages are going to morph into bigger or smaller versions of themselves as we go further into the weeds. I've tried to use the structure mentioned in this issue https://github.com/leather-wallet/extension/issues/4297#issuecomment-1809861163, but as we actually start using these packages we might want to refactor these.
I'd say the same goes for constants vs config stuff, i agree with you as we might want to use that package for more stuff other than constants, let's try to move more code in here and understand how everything should work together. Wdyt?
@kyranjamie thank you!
I was expecting to have there anything that is a constant basically, but probably that is too broad of a term. We might want to separate that package into smaller packages. The same goes for utils.

If a constant value is only used in one package, where does it belong?

Well i'd say it depends on what type of constant that is. If it's some kind of a HEADER_PADDING = "60px", then this constant is very specific to the component it is used in. If it's "IS_FEATURE_FLAG_ENABLED" and it's only used in one file, i'd say we should keep it in the shared package as it might be used in other files / components down the road

@pete-watters sounds good! We can do that way

In diagram I meant to indicate that @leather/bitcoin depends on @leather/core, a child of sorts.
If anything, I wonder if utils and constants should merge into a single "core" package, though not against trying the separated approach as a first try.


doc: release: NXP contributions

Adding NXP's contributions to v4.0

@jarmounia, your change suggestions have been integrated. Thanks.

@decsny @kartben , kindly re-visit this PR.

@decsny @kartben , kindly re-visit this PR.

i reviewed it 10 minutes before this post and my comments still arent addressed, the whole mention of MCXC enablement is missing still for example, and multiple other things still need fixed

@decsny @kartben , kindly re-visit this PR.

i reviewed it 10 minutes before this post and my comments still arent addressed, the whole mention of MCXC enablement is missing still for example, and multiple other things still need fixed

This has been addressed

@mmahadevan108 @danieldegrasse @kartben , please review.

I will be dismissing my review so that this can be approved by others if they feel this is good enough.

Mahesh is going to follow up with a PR to address the last comments


CORTX-31029: Port  number added in curl command to validate CSM API

Signed-off-by: Mukul Malhotra<EMAIL_ADDRESS>Describe your changes in brief
Port  number added in curl command to validate CSM API
Changes

[ ] Why is this change required? What problem does it solve?
[ ] If proposing a new change then please raise an issue first

How Has This Been Tested? (Optional)

[x] Please describe in detail how you tested your changes.
[ ] Include details of your testing environment, and the tests you ran to
[ ] How your change affects other areas of the code, etc.

Screenshots (if appropriate)
Checklist

[x] tested locally
[ ] added new dependencies
[x] updated the docs
[ ] added a test

One Minor Change is Required
curl --request POST '<https://:31169$HOSTNAME/api/v2/login?debug'
closing > bracket is missing after $HOSTNAME

Done