Make group rename command perform a bulkupdate querry

I think it would be useful if /lp group <group> rename <new name> just performs a bulk update querry, to also rename any saved instances like parents.
Because to fully rename a group, including its saves as parents, does require using the example bulkupdate command on the wiki, which already makes the rename command useless.

At least make it optional.

Added to the above issue ^


Mistake in documentation for parameter-overrides

In the documentation of parameters for this github action there is a mistake for specifying parameter-overrides from a file. For some reason the documentation suggest the following format: (e.g. file://file://${{ github.workspace }}/variables.json) where the file:// is duplicated, when in fact the following format works: (e.g. file://${{ github.workspace }}/variables.json).
It is very confusing, so it would be good to fix that.
Suggested fix:
description: 'The parameters to override in the stack inputs. You can pass a comma-delimited list or a file URL. Comma-delimited list has each entry formatted as <ParameterName>=<ParameterValue>. A JSON file can be a local file with a "file://" prefix or remote URL (e.g. file://${{ github.workspace }}/variables.json or http://example.com/variables.json). A local file needs to be specified with an absolute path to it. The file should look like: [ { "ParameterKey": "KeyPairName", "ParameterValue": "MyKey" }]'

Reference to the discussed line:
https://github.com/aws-actions/aws-cloudformation-github-deploy/blob/e52fd9690fef4df4a8d5086099f6fcb42207c75a/action.yml#L18

Resolved in the primary branch and waiting on release.


Bootstrapp should not modify input

Currently, Bootstrapp is defined as
func (btp *Bootstrapper) Bootstrapp(ct *Ciphertext) *Ciphertext

So it takes a ciphertext and outputs a bootstrapped ciphertext. This strongly suggests that the input is not modified, but for some reason, Bootstrapp modifies the input and returns a different output. This is neither intuitive nor documented. I suggest making Bootstrapp return a new value without modifying the input.

Hi @crockeea,
Thank you for your feedback. We will include this to the upcoming bootstrapping pull request (PR #139).


RuntimeError: CUDA out of memory during 3D reconstruction.

Hi, first of all thank you for the amazing work. I've tried the 3d reconstruction but encountered "CUDA out of memory" problem. I get this error while it's about to perform dense matching.
RuntimeError: CUDA out of memory. Tried to allocate 5.17 GiB (GPU 0; 9.76 GiB total capacity; 2.49 GiB already allocated; 4.22 GiB free; 3.50 GiB reserved in total by PyTorch) If reserved memory is >> allocated memory try setting max_split_size_mb to avoid fragmentation.
I though it could be batch size is too large, later i found out that batch_size is already set to 1 in match_dense.py. I've also tried playing around parameters like max_kps, dfactor, cell_size, resize_max, etc. but none worked. Perhaps you could shed some light on how to reduce the cuda memory needed? Much appreciated!

Could you tell me which line of code raise this error ?

i managed to reduce to memory needed by setting upsample_preds=False and changing height=480, width=640 in hloc/matchers/dkm.py. but the error remains. Full traceback is as following:
$ sh reconstruction.sh test

Thank you for your time.

https://github.com/xuelunshen/gim/blob/e06dba6c939ea5f4be8a11071df771d1956d3fcb/hloc/matchers/dkm.py#L98
Try to change 8192 to 4096, this will reduce the number of matches participating in reconstruction and may lead to worse reconstruction results


App in play store is locked down to Israeli Google accounts

Why not open to all countries?
Some of us are  using Non Israeli google accounts in the phone due to needing american apps.
Also, there might be foreign people in Israel with  foreign accounts as well, who currently live here, and should use the app as well.

Thanks! The reason for this is that many of us have their Play Store set in a different location than Israel. We changed the app to be available world-wide. It takes some time to propagate.

Thanks! The reason for this is that many of us have their Play Store set in a different location than Israel. We changed the app to be available world-wide. It takes some time to propagate.

@MiriShalev , any idea how much time it usually takes for this to update ? so far no change...

still no change


Move snapshot tests for the gen command

Technical Requirements

Move the tests from src/cli/generator/tests to tests/cli/gen.rs
Move fixtures also to the tests dir.
Use include_config! macro to load the configurations.
Ensure older tests/snapshots are deleted.
No semantic change should happen in the tests during the migration.

/bounty 30$

/attempt #2641


Issue fix 42

fixes #42 by using unicode_literals import from __future__
plus, we beautified some code.

Will close in favour of #47


acpica-tools: 20180629 -> 20180810

Semi-automatic update generated by https://github.com/ryantm/nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/acpica-tools/versions.
meta.description for acpica-tools is: '"ACPICA Tools"'.

Checks done (click to expand)

built on NixOS
/nix/store/hyaa240qdbaxncmpiwv3h553kmc8jb55-acpica-tools-20180810/bin/acpibin passed the binary check.
/nix/store/hyaa240qdbaxncmpiwv3h553kmc8jb55-acpica-tools-20180810/bin/acpidump passed the binary check.
/nix/store/hyaa240qdbaxncmpiwv3h553kmc8jb55-acpica-tools-20180810/bin/acpiexec passed the binary check.
/nix/store/hyaa240qdbaxncmpiwv3h553kmc8jb55-acpica-tools-20180810/bin/acpihelp passed the binary check.
/nix/store/hyaa240qdbaxncmpiwv3h553kmc8jb55-acpica-tools-20180810/bin/acpinames passed the binary check.
/nix/store/hyaa240qdbaxncmpiwv3h553kmc8jb55-acpica-tools-20180810/bin/acpixtract passed the binary check.
6 of 6 passed binary check by having a zero exit code.
0 of 6 passed binary check by having the new version present in output.
directory tree listing: https://gist.github.com/30906d00167b84955cacd3532b7b4e5d
du listing: https://gist.github.com/8d4d150302dc5ea3c9614e1a0d82d1bf

Test this build with Cachix (click to expand)

One time setup in nixpkgs Git checkout:
cachix use r-ryantm
git remote add r-ryantm https://github.com/r-ryantm/nixpkgs.git

Test this build:
git fetch r-ryantm && git checkout 606793b170f5b0ec93bb16b73fb98e2a1f6bd3dc && nix-shell --pure -I nixpkgs=. -p acpica-tools

cc @tadfisher for testing.

No attempt on x86_64-darwin (full log)
The following builds were skipped because they don't evaluate on x86_64-darwin: acpica-tools
Partial log (click to expand)

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowUnsupportedSystem = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowUnsupportedSystem = true; }
to ~/.config/nixpkgs/config.nix.

Success on x86_64-linux (full log)
Attempted: acpica-tools
Partial log (click to expand)
shrinking /nix/store/hyaa240qdbaxncmpiwv3h553kmc8jb55-acpica-tools-20180810/bin/acpiexec
shrinking /nix/store/hyaa240qdbaxncmpiwv3h553kmc8jb55-acpica-tools-20180810/bin/acpinames
shrinking /nix/store/hyaa240qdbaxncmpiwv3h553kmc8jb55-acpica-tools-20180810/bin/acpixtract
shrinking /nix/store/hyaa240qdbaxncmpiwv3h553kmc8jb55-acpica-tools-20180810/bin/acpihelp
shrinking /nix/store/hyaa240qdbaxncmpiwv3h553kmc8jb55-acpica-tools-20180810/bin/acpidump
strip is /nix/store/gpc2wld1s0c6qzx9326cwn1wcx29xzsj-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/hyaa240qdbaxncmpiwv3h553kmc8jb55-acpica-tools-20180810/bin
patching script interpreter paths in /nix/store/hyaa240qdbaxncmpiwv3h553kmc8jb55-acpica-tools-20180810
checking for references to /build in /nix/store/hyaa240qdbaxncmpiwv3h553kmc8jb55-acpica-tools-20180810...
/nix/store/hyaa240qdbaxncmpiwv3h553kmc8jb55-acpica-tools-20180810

Success on aarch64-linux (full log)
Attempted: acpica-tools
Partial log (click to expand)
shrinking /nix/store/n9m6xn8cjd0ydphdqbyv9zjzx5kpwr6m-acpica-tools-20180810/bin/acpinames
shrinking /nix/store/n9m6xn8cjd0ydphdqbyv9zjzx5kpwr6m-acpica-tools-20180810/bin/acpihelp
shrinking /nix/store/n9m6xn8cjd0ydphdqbyv9zjzx5kpwr6m-acpica-tools-20180810/bin/acpiexec
shrinking /nix/store/n9m6xn8cjd0ydphdqbyv9zjzx5kpwr6m-acpica-tools-20180810/bin/acpidump
shrinking /nix/store/n9m6xn8cjd0ydphdqbyv9zjzx5kpwr6m-acpica-tools-20180810/bin/acpibin
strip is /nix/store/ah0va6j4cnwj9nx4j6rwcfc8nh785jwm-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/n9m6xn8cjd0ydphdqbyv9zjzx5kpwr6m-acpica-tools-20180810/bin
patching script interpreter paths in /nix/store/n9m6xn8cjd0ydphdqbyv9zjzx5kpwr6m-acpica-tools-20180810
checking for references to /build in /nix/store/n9m6xn8cjd0ydphdqbyv9zjzx5kpwr6m-acpica-tools-20180810...
/nix/store/n9m6xn8cjd0ydphdqbyv9zjzx5kpwr6m-acpica-tools-20180810


Root call in Crab Imaging notebook

Got through all 3 Crab imaging workbooks fine until near the very end of Crab-DC2-ScAtt-Upsampling:

First I've seen this error, is Root expected for running these notebooks in DC2?

Hi @hirokiyoneda, I think this is resolved now. Can we close this issue?


Revisit mapping for Oculus touch grip trigger

The current grip trigger axis is set to 3.
According to @BastiaanOlij it should be set to 4 to match the value for the Oculus desktop plugin.

@BastiaanOlij Do note it's also set to 3 in the Oculus desktop plugin codebase.

Auch, the info page I wrote up says it should be 4:
https://github.com/GodotVR/godot_oculus/issues/4
I think most people react to the button (mapped to 2) instead of the axis so nobody really noticed until now.
I'm wondering if we shouldn't go the other way around, change the OpenVR plugin and make it 3 there. We can always keep axis 4 the same for now and mark it as a deprecated axis to be replaced once we have another purpose for it. that way people won't be caught out.

(well the MR boys might have a problem because their trackpads are mapped to axis 5/6 which we probably should keep the same)

Is there already an idea how to resolve this? (I would vote at the moment for keeping it at 3). Someone started working on adding some docs to the godot-docs here: https://github.com/godotengine/godot-docs/pull/3040 and I think adding the current button mapping in the docs would be really useful.

@BastiaanOlij @NeoSpark314 I also vote for keeping it at 3 since the Oculus version is also that way.
Along the way, we should deprecate 4 for the OpenVR plugin as you suggested, and make it 3 also.

(well the MR boys might have a problem because their trackpads are mapped to axis 5/6 which we probably should keep the same)

@BastiaanOlij What do you mean by that? Doesn't the MR headsets also use the OpenVR plugin (and mappings)?

I’m happy for it to stay at 3
The MR controllers have a track pad and joystick. As the original VIVE
controllers have a trackpad the openvr plugin maps the trackpad first, and
the joystick second. Oculus is the odd ball out.
This is one of the big reasons the action system was introduced so your
game no longer has to care, you define you have an action for a joystick
input and then map each controller accordingly, you no longer care about
the original inputs
On Sun, 28 Jun 2020 at 6:45 am, Fredia Huya-Kouadio <
<EMAIL_ADDRESS>wrote:

@BastiaanOlij https://github.com/BastiaanOlij @NeoSpark314
https://github.com/NeoSpark314 I also vote for keeping it at 3 since
the Oculus version is also that way.
Along the way, we should deprecate 4 for the OpenVR plugin as you
suggested, and make it 3 also.
(well the MR boys might have a problem because their trackpads are mapped
to axis 5/6 which we probably should keep the same)
@BastiaanOlij https://github.com/BastiaanOlij What do you mean by that?
Doesn't the MR headsets also use the OpenVR plugin (and mappings)?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/GodotVR/godot_oculus_mobile/issues/65#issuecomment-650623607,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAO262ISMKMUZDYZW5XPD4DRYZK47ANCNFSM4JJG56AQ
.

--
Kindest regards,
Bastiaan Olij
My personal profile on Facebook https://www.facebook.com/bastiaan.olij
My professional profile on LinkedIn
https://www.linkedin.com/in/bastiaanolij/
My profile on Twitter https://twitter.com/mux213
My tutorials on Youtube https://www.youtube.com/BastiaanOlij
My code repositories on Github https://github.com/BastiaanOlij

Thanks for the clarification!


MyBatis 3.5.2 can't get partial field values.

MyBatis version
3.5.2
Database vendor and version
MySQL 8.0.23
The problem
First SQL:
SELECT
  tt.id testId, ttp.title, tc.name className, tt.test_date testDate,
  (case when
            exists(SELECT tta1.id FROM t_lzxx_test_answer tta1 WHERE tta1.test_id = tt.id AND tta1.status != 2)
            then 0
        else 1 end) as `status`,
  (SELECT COUNT(*) FROM t_lzxx_test_answer tta2 WHERE tta2.test_id = tt.id AND tta2.status = 1) as `completeCount`,
  (SELECT COUNT(*) FROM t_lzxx_test_answer tta3 WHERE tta3.test_id = tt.id) as `totalCount`
FROM t_${schId}_test tt
  LEFT JOIN t_${schId}_test_paper ttp ON tt.id = ttp.test_id
  LEFT JOIN t_${schId}_class tc ON tt.class_id = tc.id
WHERE tt.is_del = 0 AND tt.create_by = #{teacherId}
ORDER BY tt.test_date DESC

parameters:
{
     "schId": "lzxx",
     "teacherId": "100005"
}

SQL result:

But mybaits can't get partial field values.

Then, I modified the SQL
SELECT testId, `title`, className, testDate, status, completeCount, totalCount  FROM (
        SELECT
            tt.id testId, ttp.title, tc.name className, tt.test_date testDate,
            (case when
                      exists(SELECT tta1.id FROM t_lzxx_test_answer tta1 WHERE tta1.test_id = tt.id AND tta1.status != 2)
                      then 0
                  else 1 end) as `status`,
            (SELECT COUNT(*) FROM t_lzxx_test_answer tta2 WHERE tta2.test_id = tt.id AND tta2.status = 1) as `completeCount`,
            (SELECT COUNT(*) FROM t_lzxx_test_answer tta3 WHERE tta3.test_id = tt.id) as `totalCount`
        FROM t_${schId}_test tt
            LEFT JOIN t_${schId}_test_paper ttp ON tt.id = ttp.test_id
            LEFT JOIN t_${schId}_class tc ON tt.class_id = tc.id
        WHERE tt.is_del = 0 AND tt.create_by = #{teacherId}
        ORDER BY tt.test_date DESC
        ) t

Now MyBatis can get the value

I would like to know why it is necessary to use a subquery temporary table for encapsulation.

Hello @Programer-Zhang ,
Only the result set matters to MyBatis, so I don't know why.
If you need further help, please create a portable demo project like these and share it on your GitHub repo.
There is little we can do if we cannot reproduce the problem. And, to reproduce the problem, we need everything including table schema, test data, Java class, config, etc..

@harawata
First, Thanks for your help!!!
I tried to build a project that can reproduce the bug, but it failed, everything is normal, the source project is too complicated, it is not convenient to use directly, so I think it is a certain SDK of the source project that affects the result, not the reason of MyBatis.


Lat, Long, Zoom as a URL parameter

Feature request: It'd be nice to be able to "deep link" to a specific view area by specific lat, long, and a zoom level as part of the URL.
My apologies if this is a bad method of submitting a feature request! The volunteer Waze map team for Utah uses your fantastic parcel viewer as a major way of verifying street names and addresses, especially new residential developments.

Welcome to Github :sparkles: and thank you for your feature request! We will take this into consideration for a future build.
Thanks again.


Analytics UI: query input field restrictions too loose / input errors do not give any feedback

e.g. select an an analytic, set the input field "maximum results" to -1. When you perform a query the spinner will display, but nothing else will happen: the user may potentially be expecting results.
Suggest twofold fix (using maximum results as an example, but may apply to other input field types):

When the backend returns an error message the user needs to be notified in some manner. e.g. error message is displayed on query page with red text. A generic message like "Invalid Query Input" would be sufficient. A more specific error would be better, e.g. "maximum results input field must be positive" - do not display an unprocessed stack trace returned from the backend: the error should be easily readable for non technical users.
Tightening input requirements using pattern validation.

e.g. for positive integer text input to display a green field background when valid, red when invalid:
input:invalid {
	background-color: red
}
input:valid {
	background-color: green
}
input:placeholder-shown {
    background-color: white
}
<input type="text" id="maximum_results" name="maximum_results" pattern="^[1-9][0-9]*$" title="Must be a positive integer greater than 0" placeholder=" ">

I'll claim this one.

Closed due to redundancy


doc fix imputation: 70 instead of 30

PR Checklist

[ ] This comment contains a description of changes (with reason)
[ ] Referenced issue is linked
[ ] If you've fixed a bug or added code that should be tested, add tests!
[x] Documentation in docs is updated

Description of changes
right? or was the idea to use 30 as default actually?

Technical details

Additional context

No, 30 was too conservative. Sorry, this one is on me. Thanks!


Function results can set FusionCacheEntryOptions.Duration

I have a use case where the returned data from a function passed to FusionCache contains the TTL (time to live), kind of like a DNS query answer.  I would like to use that result to set the duration from calls like GetOrSetAsync .
I see MemoryOptionsModifier was something that might have been the way to go but the property is marked obsolete and not wired in.
Of course I can call TryGetAsync and then SetAsync in the meantime.  @jodydonetti  curious about your thoughts on this?

I have a use case where the returned data from a function passed to FusionCache contains the TTL (time to live), kind of like a DNS query answer. I would like to use that result to set the duration from calls like GetOrSetAsync .

Yes, this is an absolutely reasonable requirement, so much so that other people alredy asked for it and is already in my backlog 😄 (see #17)

I see MemoryOptionsModifier was something that might have been the way to go but the property is marked obsolete and not wired in.

Yes, because it was an early experiment into that kind of feature, but the split between memory and distributed modifiers was not the greatest idea. Since I want to do something better, and the right design is not that straightforward, I postponed that decision and kept playing with different designs to strike the right balance.
I'm closing this since it is basically a duplicate of #17 but I hope to be able to have news on this feature quite soon!

This sounds great.  I figured my request was buried inside one of the other issues.  Thanks.

Hi @JoeShook !
I've finally found some time to put down in words a couple of proposals for this here #24 !
I would be really interesed in knowing what you think.


Reuse FlowProperty (name and synonyms) for other Flow properties (context, cas and unit)

There is a lot of duplication between FlowProperty used for Flow.name and Flow.synonyms and the classes:

Context
CAS
Unit

It makes sense to reduce duplication probably through inheritance.

I think this is nice to have - there are other things more important for now. I looked through the linked examples, the overlap is there but not huge.


Extracted common functions for CUDA dev

Extracts common parts out of the individual cuda kernel test files.
Makes it easier to write consistent tests (e.g., outputting time per repeat) and also some fixed, e.g., in some places, the old code was not properly free'ing malloced buffers; or claimed to test each block size, but in fact only tested one.
Some challenges:

When fusion the scale part into the attention kernel, we get different (unscaled) preatt results than the CPU kernel. Currently, this comparison is commented out. Similarly, checking the attention map doesn't seem to work.

Edit:
The error below was due to the test code not properly resetting the output memory. This is very important for testing backward, since there, we're generally adding instead of overwriting.

With the new tests, we're testing different block sizes that weren't tested before, and I now get a failing test for cross-entropy backward:

Mismatch of dlogits at 102926336: -0.041662 vs -0.083324

wow, meaty one :) agree that i think this makes sense at this point...

@ngc92 it sounds like this PR is still in flux (per your edit comment etc.), will wait to full review until you think it looks ready.

I think it should be ready now.
The remaining issues are not really a concern of the PR per-se, but more something  we need to keep in mind going forward:
We may not be able to arbitrarily mix-and-match forward and backward kernel implementations for the same op, because they might want to store different intermediate results.
The TODO comments in this PR are thus less about something actually missing here, and more that I wanted to bring this to your attention.

made a few small changes/fixes and merged ty 👍

nice


Fix loading animation missing for pin packaged

This PR should fix loading animation missing when pin is being created

Can you please update the changelog and versions so the package can be published after the update?


blog post: What is Seraphis, and Why Should You Care?

I wrote this post on u/dEBRUYNE's request. Let me know if I should add/clarify anything.
Before anyone whines about my mention of MobileCoin, you should keep in mind I wrote the first draft of the Seraphis paper while an employee at MobileCoin.

If a cryptocurrency uses RingCT, it is more work to upgrade to Seraphis than Triptych.
Is this due to the address format change or are there other reasons too?

Mainly due to the address change. You also need to audit two signature schemes instead of one. On the other hand, Triptych multisig implementation is wayyy more work, so maybe it balances out.

We already pushed 3 blog posts this week. IMO would be better to publish this not earlier than next week.

@UkoeHB Please change the date of the post to today's date.


Embedded Page Personalisation

Hi,
Would it be possible to personalised the default success or error page ?
<!DOCTYPE html>

    <meta charset="utf-8">
    <title>Authorization successful.</title>

Application has successfully obtained access credentials

This window can be closed now.

Also, displaying am  personalised "error page" could be great, instead of a 500 error with no message to users and outside our app.
Thank you for this package and your consideration.
Regards

    // TODO: We could introduce a user-defined redirect page.

Just saw the comment, it's a good idea 👍  😆

Would love to see that as well!


Fix the Image class's from_url method error

This should address the #1902 .
Also, I found there are some dependencies missing when running the test from a fresh virtual environment so add them in the pyproject.toml file. Please let me know if that is the best place to hold them? Thank you!

@okhat let me know if there is anything else you see potential improvements before it can be merged? Thanks and Happy Holidays!


Alerts – Local Storm Reports – US (v2)

The Local Storm Report API provides short text reports of observed severe weather such as wind or hail damage, snow depth that is parsed from NWS Local Storm Report (LSR) and Public Information Statement (PNS) bulletins. The reports are ranked and prioritized by TWC upon receipt from the NWS.

Need list of geoIds to make better queries so this is demoted in priority.

Couldn't get geoids


Editor window jumps on save / format

Does this issue occur when all extensions are disabled?: Yes/No

VS Code Version: 1.85.1
OS Version: Mac OS 14.3

Steps to Reproduce:

Have format on save on
Have a formatter extensions active, e.g. black or ruff
Save the file after making changes

Whenever a format on save event happens, the editor completely jumps, sometimes a page down, sometimes all the way down to the bottom of the file. I have to toggle the cursor (up / down arrows), for it to jump back where I was. This is very annoying. Worth noting that this does not only happen when there is extensive reformatting, it can be a very very small change. I expect the editor  window to stay centered on the cursor.

Please add the following

copy-paste friendly source sample
the extension that you are using

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


Keyboard navigation not recognised by NVDA in Flutter Desktop`

Details

If you create the default flutter application with flutter create counter, then run that application under Windows with NVDA running, controls aren't accessible via the keyboard.
While the keyboard navigation keys like tab do appear to move focus (you can verify by tabbing to the button), those focus changes aren't picked up by NVDA.
You can use object navigation to navigate to the created widgets, but it is by no means intuitive.

**Target Platform:Windows
**Target OS version/browser:Windows 10
**Devices:Desktop and laptop computers
Logs

Logs

Analyzing counter...
No issues found! (ran in 1.1s)

[ÔêÜ] Flutter (Channel master, 2.6.0-12.0.pre.936, on Microsoft Windows [Version 10.0.19043.1348], locale en-GB)
    ÔÇó Flutter version 2.6.0-12.0.pre.936 at C:\Users\chris\flutter
    ÔÇó Upstream repository https://github.com/flutter/flutter.git
    ÔÇó Framework revision 47a8f440c7 (4 hours ago), 2021-12-06 21:47:20 -0800
    ÔÇó Engine revision 66a281107b
    ÔÇó Dart version 2.16.0 (build 2.16.0-80.0.dev)
    ÔÇó DevTools version 2.9.1

[ÔêÜ] Android toolchain - develop for Android devices (Android SDK version 31.0.0-rc2)
    ÔÇó Android SDK at C:\Users\chris\AppData\Local\Android\sdk
    ÔÇó Platform android-31, build-tools 31.0.0-rc2
    ÔÇó Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java
    ÔÇó Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7249189)
    ÔÇó All Android licenses accepted.

[ÔêÜ] Chrome - develop for the web
    ÔÇó Chrome at C:\Program Files (x86)\Google\Chrome\Application\chrome.exe

[ÔêÜ] Visual Studio - develop for Windows (Visual Studio Community 2019 16.9.3)
    ÔÇó Visual Studio at C:\Program Files (x86)\Microsoft Visual Studio\2019\Community
    ÔÇó Visual Studio Community 2019 version 16.9.31129.286
    ÔÇó Windows 10 SDK version 10.0.19041.0

[ÔêÜ] Android Studio (version 2020.3)
    ÔÇó Android Studio at C:\Program Files\Android\Android Studio
    ÔÇó Flutter plugin can be installed from:
       https://plugins.jetbrains.com/plugin/9212-flutter
    ÔÇó Dart plugin can be installed from:
       https://plugins.jetbrains.com/plugin/6351-dart
    ÔÇó Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7249189)

[ÔêÜ] VS Code, 64-bit edition (version 1.62.3)
    ÔÇó VS Code at C:\Program Files\Microsoft VS Code
    ÔÇó Flutter extension version 3.27.0

[ÔêÜ] Connected device (3 available)
    ÔÇó Windows (desktop) ÔÇó windows ÔÇó windows-x64    ÔÇó Microsoft Windows [Version 10.0.19043.1348]
    ÔÇó Chrome (web)      ÔÇó chrome  ÔÇó web-javascript ÔÇó Google Chrome 96.0.4664.45
    ÔÇó Edge (web)        ÔÇó edge    ÔÇó web-javascript ÔÇó Microsoft Edge 95.0.1020.53

ÔÇó No issues found!

Hi @chrisnorman7, Thanks for filing the issue.

then run that application under Windows with NVDA running, controls aren't accessible via the keyboard.

Can you please share more about this what exactly is NVDA desktop and how to use it inorder to reproduce the issue.
Thanks

I'm not the original issue creater but I can do my best to explain this.
NVDA is a screen reader for the Windows platform. Visually impaired/blind individuals use it in order to operate their computer as well as use other applications such as those developed in flutter.
you can download it from www.nvaccess.org

TO add onto what @crentava said, screen readers communicate with accessibility APIs in order to tell the user what's happening in an application. Normally, when the keyboard focus changes, the requisite accessibility events are fired that make sure a screen reader is able to tell the user the focus is now on the new control, which is control XYZ of type ABC and with value DEF, if applicable. Flutter UI elements currently do not appear to be emitting these properly.

Hi,
To add to what @crentava and @zersiax have already stated (which is far more detailed than I'd have been able to provide, so thank you for that), NVDA is one of the screen readers which are advertised as working with Flutter Web, according to the accessibility page.

Thanks for the detailed explanation. Leaving this issue open for further insights from the team.

Whoever is working on this, great job! I've just upgraded Flutter on the master branch, and it's working really well.
I won't close yet in case I find more things, but I just wanted to massively thank whoever has improved things this much.

Thanks for the report! Closing for now. If you find any other issues, please open a new issue. (Desktop Triage)

@chrisnorman7 thanks for testing all the new a11y work landing in the Windows embedder!
As you run into any issues please file bugs and (if you remember), CC me (@cbracken). I'll be out for a couple weeks, but glad to take a look when I'm back. Thanks again!

/cc @crentava and @zersiax directly as well. Our master branch should now have relatively basic accessibility support on Windows. I'd encourage you to give it a go and file any bugs you run into and CC me on them so that I get notified. We're working on getting our screen-reader (and other a11y feature) support up to scratch across all our desktop platforms.


The method '[]' can't be unconditionally invoked because the receiver can be 'null' | Firebase Database | Flutter

`import 'package:firebase_database/firebase_database.dart';
class Drivers
{
String? name;
String? phone;
String? email;
String? id;
String? car_color;
String? car_model;
String? car_number;
Drivers({this.name, this.phone, this.email, this.car_color, this.car_model, this.car_number});
Drivers.fromSnapshot(DataSnapshot dataSnapshot)
{
id = dataSnapshot.key;
name = dataSnapshot.value["name"];
phone = dataSnapshot.value["phone"];
email = dataSnapshot.value["email"];
car_color = dataSnapshot.value["car_color"];
car_model = dataSnapshot.value["car_model"];
car_number = dataSnapshot.value["car_number"];
}
}`
What is the problem in my code??

For help with your own code, try one of the resources listed at https://flutter.dev/community#community-grid
Closing as this is not an issue in Flutter itself, which is what we track here.


Typo in 'Message center' para?

Under Message center, text says 'changes to hour Microsoft services'.
Should that be 'our' ?

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

ID: 9eadb9e2-bc3a-25e6-ec3d-e85e1f404405
Version Independent ID: 5b70c7ac-bce9-36ec-09ce-c9291c1744e0
Content: Administer Power Platform - Power Platform
Content Source: power-platform/admin/admin-documentation.md
Service: power-platform
Sub-service: admin
GitHub Login: @jimholtz
Microsoft Alias: jimholtz

Thanks for your comment, @standswell. This issue has been assigned to the doc author for review.

@standswell
Thanks for notice. We'll get this corrected.


Add Linux support

It's actually crazy that osu! Lazer supports Linux but these plugins don't. I don't know about mac, as I don't use it, but if it isn't supported you should change that.

If your "plugin" is the rurusetto-addon please report issues there. This repository is for the rurusetto website only.


TAs Appearing in the GB's Spreadsheet View/Export CSV

Just noticed this when testing import/export, and wanted to raise this really quick:
It looks like users with the role of "Teaching Assistant" are currently appearing as students to be graded within the spreadsheet interface.

Site Info screen: http://www.screencast.com/t/GLr6FJVil
New Gradebook Grades view: http://www.screencast.com/t/Mog5FYcI
GB Classic view: http://www.screencast.com/t/nM8CHedYN7e1

Any TA's in the new GB are also appearing in exported CSV files.

Ok this explains the issue of the grades not appearing as well. We fetch the grades for all students per assignment. So if one of those users throws the exception, the assignment is skipped. So when you had that student viewing it, it showed nothing as they didn't have permission. And when you were instructor it was also throwing the error because TAs were in the list.
As discussed, when you pass in the list of users to get grades it goes off and gets the users again and checks the ones passed in are in the allowed list. If not, exception which leads to skipping.
I think there is a mismatch in how we get the users that can have grades and the way the gradebook service is expecting them. We use the method that gets them all based on a permission, I think if we tweak this to match we should be good.

Ok this should be done and will appear on nightly soon, @kyleblythe can you please confirm? This should also fix the issue you saw with the scores not appearing. If so, high fives all round.


Adding Instructions for templates and variables.

Hi team,
I hope you're doing well. I'm currently using MS for a personal project and love how simple it is (even for me). I noticed the docs needed some changes so I hope you don't mind me adding the instructions to send emails using templates and variables to send emails through the SDK. Feel free to push back if changes are necessary and I hope I'm not stepping on anyone's toes on this.
As well feel free to reach out if you have any questions.
Best,
Javier

@devjavi Thank you for the contribution!


Initialize queryable index with init_graph?

I would like to be able to more quickly construct an NNDescent object from a precomputed distance matrix, and potentially RP Forest. This is sorta possible right now with the init_graph argument, but the NNDescent object constructed cannot be queried since there's no RP Forest (#103).

 Example of this failing
import pynndescent
from sklearn.datasets import make_blobs
from sklearn.model_selection import train_test_split

train, test = train_test_split(make_blobs(10_000)[0])

from_scratch = pynndescent.NNDescent(train, n_neighbors=15)
indices, _ = from_scratch._neighbor_graph
from_scratch.query(test)  # works

from_init = pynndescent.NNDescent(train, n_neighbors=15, init_graph= indices)
from_init.query(test)

---------------------------------------------------------------------------
TypeError                                 Traceback (most recent call last)
<ipython-input-15-d1ceea2a4663> in <module>
----> 1 from_init.query(test)

~/github/pynndescent/pynndescent/pynndescent_.py in query(self, query_data, k, epsilon)
   1585         """
   1586         if not hasattr(self, "_search_graph"):
-> 1587             self._init_search_graph()
   1588
   1589         if not self._is_sparse:

~/github/pynndescent/pynndescent/pynndescent_.py in _init_search_graph(self)
    953
    954         if not hasattr(self, "_search_forest"):
--> 955             tree_scores = [
    956                 score_linked_tree(tree, self._neighbor_graph[0])
    957                 for tree in self._rp_forest

TypeError: 'NoneType' object is not iterable

I think this would be fairly straightforward to make work. Here is a rough proof of concept:

 Hacky example of this working
from_scratch = pynndescent.NNDescent(train, n_neighbors=15)
indices, _ = from_scratch._neighbor_graph
rp_forest = from_scratch._rp_forest
query_indices_scratch, query_distances_scratch = from_scratch.query(test)

from_init = pynndescent.NNDescent(train, n_neighbors=15, init_graph=indices)
from_init._rp_forest = rp_forest
query_indices_init, query_distances_init = from_scratch.query(test)

# I think they won't always be exactly the same at the moment, but I think this could be addressed
np.testing.assert_allclose(query_indices_scratch, query_indices_init)
np.testing.assert_allclose(query_distances_scratch, query_distances_init)

Use case
The use case is in single cell analysis, where we are storing the neighbor graph for our dataset (as we use it for multiple purposes) and wanting to be able speed up querying the dataset using reconstructed graphs.

 Construction benchmark
Using fmnist data from ann benchmark
%time pynndescent.NNDescent(fmnist_train)
# CPU times: user 1min 3s, sys: 813 ms, total: 1min 4s
# Wall time: 5.92 s

# indices taken from ^
%time pynndescent.NNDescent(fmnist_train, init_graph=indices)
CPU times: user 10.5 s, sys: 49.2 ms, total: 10.5 s
Wall time: 1.4 s

Questions
I'm not sure how much validation should/ could be done to make sure initialization values are valid. To some extent, I think it's also alright to say "user beware" here. Some thoughts on checks that could be made:

Of course, the graph, forest, and dataset must have related sizes.
The graph should have the correct k.
If distances could be passed as well as indices, their values could be verified (also verifying the metric)

Could values like rp_forest, _search_function, be created on demand and cached via a property? That might make state easier to manage, and I believe all necessary parameters are already being stored in the NNDescent object.
This is more of an extension, but does the init_graph have to have the correct values of k? For instance, it would ostensibly speed up index creation even if you could only provide an initial graph for a smaller value of K.
(ping @Koncopd)

I certainly agree that the use case makes a lot of sense.  A question is how best to actually handle this. Queries will be slower without a forest / search tree as the initialization for the graph search will be slower. In general one doesn't just have trees lying around to pass in as you did in your example, so one would need to generate them. Potentially we could just make a single search tree if _rp_forest is None which may not be the best tree but will do the job. I'll have a look through the code and see if this is something that can be done easily (if so I'll just do it), or if it would require a little more refactoring (in which case it may take a while to get to).

I certainly agree that the use case makes a lot of sense.  A question is how best to actually handle this. Queries will be slower without a forest / search tree as the initialization for the graph search will be slower. In general one doesn't just have trees lying around to pass in as you did in your example, so one would need to generate them. Potentially we could just make a single search tree if _rp_forest is None which may not be the best tree but will do the job. I'll have a look through the code and see if this is something that can be done easily (if so I'll just do it), or if it would require a little more refactoring (in which case it may take a while to get to).

Short answer: basic support is easy, so I added it. In principle the current master branch should work for you now.
That leaves the harder questions about validation. I think "user beware" is reasonable to a degree here. Actually I'm surprised things worked as well as they did for you -- the whole init_graph was actually just a hack I dropped in to let me explore a few research questions, and I didn't bother to pull it out again. I wasn't really expecting other people to use it that much.
On the other hand you do present some pretty reasonable use-cases, so it is not unlikely that others will have similar uses. That means making things a little more user friendly is probably a good idea.
Some answers to specific questions:

Basic sizing of dataset and graph: yes, that's worth checking, and it is (Although not up front, so I could pull the check higher up as well -- the lower level function will still want it).
The graph should have the right k: Given that we go through heap construction this actually isn't necessary (with a little tweak): we can size the resulting graph correctly and use the information we've been given.
Distances could be passed as well as a check: This actually gets a little quirky. Pynndescent often cheats on distance functions -- any function that maintains the ordering on distances will do, and so we can often substitute a cheaper option. It is possible that this could be worked around, but it is probably more trouble than it is worth to cover all the cases. I'll give it some thought in case there is a good option here.
It might be possible to make _rp_forest and _search_forest properties, but I think ideally I just handle things correctly -- issues with "create on demand" along with consistency and caching can potentially create as many subtle problems as they fix.

Short answer: basic support is easy, so I added it. In principle the current master branch should work for you now.
That leaves the harder questions about validation. I think "user beware" is reasonable to a degree here. Actually I'm surprised things worked as well as they did for you -- the whole init_graph was actually just a hack I dropped in to let me explore a few research questions, and I didn't bother to pull it out again. I wasn't really expecting other people to use it that much.
On the other hand you do present some pretty reasonable use-cases, so it is not unlikely that others will have similar uses. That means making things a little more user friendly is probably a good idea.
Some answers to specific questions:

Basic sizing of dataset and graph: yes, that's worth checking, and it is (Although not up front, so I could pull the check higher up as well -- the lower level function will still want it).
The graph should have the right k: Given that we go through heap construction this actually isn't necessary (with a little tweak): we can size the resulting graph correctly and use the information we've been given.
Distances could be passed as well as a check: This actually gets a little quirky. Pynndescent often cheats on distance functions -- any function that maintains the ordering on distances will do, and so we can often substitute a cheaper option. It is possible that this could be worked around, but it is probably more trouble than it is worth to cover all the cases. I'll give it some thought in case there is a good option here.
It might be possible to make _rp_forest and _search_forest properties, but I think ideally I just handle things correctly -- issues with "create on demand" along with consistency and caching can potentially create as many subtle problems as they fix.

Thanks for the quick addition!

In general one doesn't just have trees lying around to pass in as you did in your example, so one would need to generate them.

For sure. My initial test of this was just to change this:
https://github.com/lmcinnes/pynndescent/blob/a0ca8400b681d73e48e4fc2ffe2e905ee89be8dd/pynndescent/pynndescent_.py#L715-L718
to
        if tree_init or n_trees != 0:
            self.tree_init = True
        else:
            self.tree_init = False

so a tree could be constructed even if the init_graph was used for
I guess I see there being multiple cases for passing an initial graph. I think I'd actually view this as making multiple ways to construct a NNDescent object.

From data Pass in data matrix, construct graph. The current initializer workflow.
From fully initialized graph Pass in a completely correct data matrix. This should n't require any optimization. This is the case that would have the most need for validation (at least w.r.t. parameters, distances may not be worth it). In this case, parameters are there for verification, otherwise can be derived from the input (e.g. k).
From partially initialized graph Pass in a partially initialized graph. This could have the wrong number of neighbors, or even have wrong distances. This is just used for initialization (can this be combined with rp_forest initialization reasonably?). Here, parameters override properties of the input.

2 can mostly be achieved from 3, though 2 would be nice for very fast instantiation. However, this probably depends on how quirky the distances get. Is this the case of "alternative" metrics which need to go through a reverse transform (e.g. what you mention here), or there more to it?

Thanks for the quick addition!

In general one doesn't just have trees lying around to pass in as you did in your example, so one would need to generate them.

For sure. My initial test of this was just to change this:
https://github.com/lmcinnes/pynndescent/blob/a0ca8400b681d73e48e4fc2ffe2e905ee89be8dd/pynndescent/pynndescent_.py#L715-L718
to
        if tree_init or n_trees != 0:
            self.tree_init = True
        else:
            self.tree_init = False

so a tree could be constructed even if the init_graph was used for
I guess I see there being multiple cases for passing an initial graph. I think I'd actually view this as making multiple ways to construct a NNDescent object.

From data Pass in data matrix, construct graph. The current initializer workflow.
From fully initialized graph Pass in a completely correct data matrix. This should n't require any optimization. This is the case that would have the most need for validation (at least w.r.t. parameters, distances may not be worth it). In this case, parameters are there for verification, otherwise can be derived from the input (e.g. k).
From partially initialized graph Pass in a partially initialized graph. This could have the wrong number of neighbors, or even have wrong distances. This is just used for initialization (can this be combined with rp_forest initialization reasonably?). Here, parameters override properties of the input.

2 can mostly be achieved from 3, though 2 would be nice for very fast instantiation. However, this probably depends on how quirky the distances get. Is this the case of "alternative" metrics which need to go through a reverse transform (e.g. what you mention here), or there more to it?

Yes, it is about the alternative metrics. The catch being that if and when any get added it is good to have as few places to make changes as possible (and ideally obvious ones). Potentially I can just add an extra parameter init_distances that defaults to None. If provided they will be used regardless of what the metric is defined to be, and no distance recomputation will take place. Of course any querying will still use the defined metric, but the distances themselves are only really needed for graph construction / refining so that wouldn't matter so much. It would also have a degree of "user beware". Although, for example, even euclidean distance uses squared euclidean instead, and the edge pruning will get messed up if you pass in a graph of euclidean distances. I worry that it will cause more harm than good as it will be easy to misuse.

Yes, it is about the alternative metrics. The catch being that if and when any get added it is good to have as few places to make changes as possible (and ideally obvious ones). Potentially I can just add an extra parameter init_distances that defaults to None. If provided they will be used regardless of what the metric is defined to be, and no distance recomputation will take place. Of course any querying will still use the defined metric, but the distances themselves are only really needed for graph construction / refining so that wouldn't matter so much. It would also have a degree of "user beware". Although, for example, even euclidean distance uses squared euclidean instead, and the edge pruning will get messed up if you pass in a graph of euclidean distances. I worry that it will cause more harm than good as it will be easy to misuse.

Although, for example, even euclidean distance uses squared euclidean instead, and the edge pruning will get messed up if you pass in a graph of euclidean distances.

Could this be solved by having a "reverse_correction" as well as "correction" for the alternative distances?

Also, I've been looking at #79 again, and had a question related to both of these.
How accurate is are the nearest neighbors found by nndecent? While this must vary with number of neighbors and iterations, I was wondering if there was some benchmarking that had been done for accuracy of graph construction (as opposed to querying, like the ANN benchmarks).
My concern here was: if you're providing poor guesses for initial indices, how bad can the results get?

Although, for example, even euclidean distance uses squared euclidean instead, and the edge pruning will get messed up if you pass in a graph of euclidean distances.

Could this be solved by having a "reverse_correction" as well as "correction" for the alternative distances?

Also, I've been looking at #79 again, and had a question related to both of these.
How accurate is are the nearest neighbors found by nndecent? While this must vary with number of neighbors and iterations, I was wondering if there was some benchmarking that had been done for accuracy of graph construction (as opposed to querying, like the ANN benchmarks).
My concern here was: if you're providing poor guesses for initial indices, how bad can the results get?

Could this be solved by having a "reverse_correction" as well as "correction" for the alternative distances?

Yes, I'm more concerned about the long term tail on maintaining such things for new distances, and not having something fall out of sync somewhere and creating hard to diagnose errors (I had similar concerns over adding the alternative distances initially, but there were some very significant performance wins for many standard use cases). Potentially this is a very niche use case, so it's a matter of balancing the amount of extra utility with long term overhead.

How accurate is are the nearest neighbors found by nndecent?

That's hard to answer because so much is dataset dependent. It can be very bad for certain metrics and data distributions, but largely it is pretty good. Anecdotally I generally expect 90%+ accuracy. I'm not aware of standard benchmarks for it however. Typically the way to check is to get the true nearest neighbors for a (small) subsample of points and get an estimate of the accuracy that way.

My concern here was: if you're providing poor guesses for initial indices, how bad can the results get?

Pure NNDescent assumes you get an initial graph with random edges and does well, so you can provide quite poor guesses and still do well, it will just take longer to converge. The greater risk is in providing an initial graph with suitable pathological properties (e.g. distinct connected components that don't relate to neighbor structure). You are unlikely to get that with a generic noisy guess at nearest neighbors, but it is certainly possible to generate such graphs (take, say, the leaves of a single rp-tree for example). Again, this is subtle in the ways it can go astray, so a degree of user expertise is required.

Could this be solved by having a "reverse_correction" as well as "correction" for the alternative distances?

Yes, I'm more concerned about the long term tail on maintaining such things for new distances, and not having something fall out of sync somewhere and creating hard to diagnose errors (I had similar concerns over adding the alternative distances initially, but there were some very significant performance wins for many standard use cases). Potentially this is a very niche use case, so it's a matter of balancing the amount of extra utility with long term overhead.

How accurate is are the nearest neighbors found by nndecent?

That's hard to answer because so much is dataset dependent. It can be very bad for certain metrics and data distributions, but largely it is pretty good. Anecdotally I generally expect 90%+ accuracy. I'm not aware of standard benchmarks for it however. Typically the way to check is to get the true nearest neighbors for a (small) subsample of points and get an estimate of the accuracy that way.

My concern here was: if you're providing poor guesses for initial indices, how bad can the results get?

Pure NNDescent assumes you get an initial graph with random edges and does well, so you can provide quite poor guesses and still do well, it will just take longer to converge. The greater risk is in providing an initial graph with suitable pathological properties (e.g. distinct connected components that don't relate to neighbor structure). You are unlikely to get that with a generic noisy guess at nearest neighbors, but it is certainly possible to generate such graphs (take, say, the leaves of a single rp-tree for example). Again, this is subtle in the ways it can go astray, so a degree of user expertise is required.

@lmcinnes this is tangentially related to the current discussion here about accuracy due to nearest neighbor descent, so can I just confirm that the current way pynndescent works is:
To build the index:

Build a forest.
Refine it with nearest neighbor descent.

To query the index:

Initialize guess with the forest from part 1 above.
Refine it with a nearest neighbor descent-like approach (breadth-first search using neighbor of neighbors).

Specifically, It seems like the _neighbor_graph that results from the nearest neighbor descent to build the index doesn't play any role in querying. That is: the refinement doesn't feed back into the forest in any way, the initialization of the query result uses the forest initialization (augmented with random points if necessary, but never the _neighbor_graph results which may not exist any more if compress=True), and the breadth-first search uses neighbors of neighbors taken from the _search_graph which is derived from the _search_forest. Is that correct?
The context for this question is that I continue to sporadically prod at creating a nearest neighbor descent package for R, and my implementation does not use any forest initialization. For very high-dimensional data, results are poor (#18 is still relevant here), in line with the accuracy determined by just using the _neighbor_graph from the index-building step. A "pure" NND approach gives about 45% accuracy via the _neighbor_graph without tree initialization for the dataset I have been using (this is in agreement with my R version which is reassuring). With the forest initialization pynndescent gets t a much more respectable 85% accuracy after querying.
I want to make sure I am not missing anything clever being done in the query routine that would work with a naive "use the NND results as initialization" approach. Unfortunately, it doesn't seem easy to replace the forest-initialized indptr and indices data with that from _neighbor_graph, so I haven't been able to check myself with pynndescent.

Rather than open an issue I guess I'll ask here: what format should init_graph take? It seems to be missing from the docstrings, so it's not obvious how to pass in a graph, or what will happen if I do. From looking at the code and reading this discussion I gather that I can initialize the kNN using my own graph rather than the rp tree approach.
Really I can figure out the answer to this myself, if it would be nice if there some hint in the documentation! 🙂

@jamestwebber : The docs are definitely lacking on that front. In reality I hacked something in for my own needs for some experiments, and never got around to making a user friendly version. The short answer is that, right now, it takes a 2D array of shape (n_samples, n_neighbors) such that the entry at (i, j) gives the index of the jth neighbour of the ith data sample. I would happily accept a PR for documentation on that, or perhaps even a user friendlier version (accepting, say, sparse adjacency matrices and networkX graphs as well maybe?), if you cared to try.
@jlmelville : Wow, sorry, this totally slipped through the cracks on me! I apologise for not getting back to you. I think the major thing is that the _search_graph is constructed from the neighbor_graph_; it is really just a pruned version of a symmetrized (i.e. undirected) version of the neighbor_graph_. On the other hand the tree to initialize the search is pretty useful because it ensures that we start from a "decent" place -- perhaps not ideal, but a lot less likely to run into local minima far from the query point. In my experience it mostly helps with speed of convergence though, and not accuracy.

The short answer is that, right now, it takes a 2D array of shape (n_samples, n_neighbors) such that the entry at (i, j) gives the index of the jth neighbour of the ith data sample.

Ah, so exactly what the output looks like! That's handy.

I would happily accept a PR for documentation on that, or perhaps even a user friendlier version (accepting, say, sparse adjacency matrices and networkX graphs as well maybe?), if you cared to try.

As you have no doubt noticed by now, I have a problem of saying I'll contribute PRs and then not getting to them, so I will demur this time. 😂


update Maven GAVC for correct Commons-IO

Updates correct groupId for commons-io for refine-model
Changes proposed in this pull request:

@thadguidry do you plan to work on this further?

Yeah let me work on updating this now.

@wetneb Ah! There is no need for this I found out.  It was a bad artifact in my M2 local repo actually from cross-pollination with another GitHub project.
We have the correct GAVC as shown officially in build distribution from Apache:  https://commons.apache.org/proper/commons-io/summary.html
So closing.


Implement more complete collection with support for Cisco SD-WAN 19.2 Always On Sandbox

This update includes changes to the existing calls specifically to work with the Cisco SD-WAN 19.2 Always On DevNet sandbox environment. This includes the addition of an extra call in authentication to obtain a token.
The collection is further expanded with all the known endpoints I could find documented for obtaining data from vManage.
Hope you and others find this useful.

I discovered there is a newer/different postman collection for the 19.2 Always on Environment. So these changes are probably better off against that repository: https://github.com/CiscoDevNet/Postman-for-AlwaysOn-Cisco-SD-WAN


Using token from getCallbackTokenAsync

I have used a token from Office.context.mailbox.getCallbackTokenAsync to call the API but it is not working. Can I use this token?
I have read this article
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 04ee7d3a-9e1f-dcef-0ac2-e8382d39a796
Version Independent ID: 15c4c20f-7d81-5d27-3de6-fa9868356c70
Content: Get Started with the Outlook REST APIs - Outlook Developer
Content Source: docs/rest/get-started.md
Product: outlook
Technology: ms-graph
GitHub Login: @jasonjoh
Microsoft Alias: jasonjoh

Add-in documentation has been moved to another repository https://github.com/OfficeDev/office-js-docs-pr. It's been too long since I've worked with the Office JS APIs to add any value here. You should probably ask your question on Q&A.


pnpm install failed in vite repo

Describe the bug
I clone the vite repo and run pnpm install but it failed

It seems that ts-node generate esm module which can't be running on nodejs directly(because lack type:module in package.json).
Reproduction
just clone the repo and run pnpm install
System Info
System:
    OS: macOS 11.1
    CPU: (8) x64 Apple M1
    Memory: 38.20 MB / 16.00 GB
    Shell: 5.8 - /bin/zsh
  Binaries:
    Node: 14.18.2 - ~/.nvm/versions/node/v14.18.2/bin/node
    npm: 6.14.15 - ~/.nvm/versions/node/v14.18.2/bin/npm
    Watchman: 2<IP_ADDRESS> - /usr/local/bin/watchman
  Browsers:
    Chrome: 96.0.4664.110
    Chrome Canary: 99.0.4810.0
    Edge: 96.0.1054.62
    Firefox: 93.0
    Safari: 14.0.2
  npmPackages:
    vite: workspace:* => 2.8.0-beta.1

Used Package Manager
pnpm
Logs
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Make sure this is a Vite issue and not a framework-specific issue. For example, if it's a Vue SFC related bug, it should likely be reported to https://github.com/vuejs/vue-next instead.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

Can't reproduce on GitHub Codespaces

Oh, I figure it out, there's an tsconfig.json in the ancestor directory of the cloned repo, and ts-node will recursively lookup tsconfig.json until root, may be we could setup an tsconfig.json in the project root to avoid this problem.

I think the tsconfig.json should be put in the scripts folder.


[improve][ci] Move some flaky ExtensibleLoadManager tests to flaky group until they are fixed

Motivation
There are at least 2 really flaky tests in Pulsar master:

ExtensibleLoadManagerCloseTest.testCloseAfterLoadingBundles https://github.com/apache/pulsar/issues/23412
BrokerRegistryMetadataStoreIntegrationTest.cleanup https://github.com/apache/pulsar/issues/23413

Modifications

Move the tests to the flaky group until they are fixed.

Documentation

[ ] doc
[ ] doc-required
[x] doc-not-needed
[ ] doc-complete

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

Project coverage is 74.56%. Comparing base (bbc6224) to head (9c91f5d).
Report is 642 commits behind head on master.

Additional details and impacted files

@@             Coverage Diff              @@
##             master   #23414      +/-   ##
============================================
+ Coverage     73.57%   74.56%   +0.99%
- Complexity    32624    34084    +1460
============================================
  Files          1877     1937      +60
  Lines        139502   145539    +6037
  Branches      15299    15901     +602
============================================
+ Hits         102638   108526    +5888
+ Misses        28908    28685     -223
- Partials       7956     8328     +372

Flag
Coverage Δ

inttests
27.68% <ø> (+3.10%)
:arrow_up:

systests
24.45% <ø> (+0.12%)
:arrow_up:

unittests
73.93% <ø> (+1.08%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.
see 619 files with indirect coverage changes


Proposal to customize date_picker icons

Use case
I want to have custom icons in date_picker
date_picker.dart itself has this
entryModeButton = IconButton(
          icon: const Icon(Icons.edit), //hardcoded icon
          color: onPrimarySurface,
          tooltip: localizations.inputDateModeButtonLabel,
          onPressed: _handleEntryModeToggle,
        );

calendar_date_picker has
RotationTransition(
                          turns: _controller,
                          child: Icon(
                            Icons.arrow_drop_down,
                            color: controlColor,
                          ),
                        ),

Proposal
expose these hardcoded icons, so we can change them to our needs.

Sounds like a reasonable request. Is it enough to expose the hardwired icons, or is there a use case for changing colors or the animation?

@HansMuller for me its enough to have option to replace icons, then other could come in and ask for animations, colors,.. i would try to expose as much as possible.


About dataset

Thank you for your wonderful work and making the dataset public. May I ask which sequences "HDR", "LL", "FWB", "FNB " represent which sequences respectively? And in the open source dataset, the sequence names "222", "333", "mul", "mul222 ", "motion" stand for what?

Same question, can you provide the attribute sequence list in FE240 and FE108?
Thanks for your great work. @Jee-King


Follow new ArviZ development

The new ArviZ is getting in shape and many functions in ArviZ-legacy are already usable see https://arviz-plots.readthedocs.io/en/latest/ we should evaluate what functionality we are going to use from that project and also informed its development given the requirements of arviz-dashboard

Moreover, there is still a lot of room to influence arviz-plots so it can be a better fit for dashboards. For example: https://github.com/arviz-devs/arviz-plots/issues/68


charges added

Thx for this nice integration!
I am trying to get an overview of all charges added to the EV. I now use the "myenergi_zappi_charge_added..". This show the last added session but this remains this number until a next charge session. So in grafana I see a lot of same numbers over time Is it possible that this sensor resets to 0 after it has been written to the database? Or do you have another way to keep track of the kwh added?

Yes this sounds like a good change, charge added today should reset for each day. I'll change from a cumilative counter to a normal value.

Thx for this nice integration!
I am trying to get an overview of all charges added to the EV. I now use the "myenergi_zappi_charge_added..". This show the last added session but this remains this number until a next charge session. So in grafana I see a lot of same numbers over time Is it possible that this sensor resets to 0 after it has been written to the database? Or do you have another way to keep track of the kwh added?

You can close this now, it’s resets each day


Updating readme to include updating patternfly dependency version

@jeff-phillips-18 added this per your comment.

LGTM


GBA: Mistake when describing modes 3, 4, 5

The good news is that the PPU actually implemented this functionality by including three extra modes, these are called bitmap modes [7]:
Mode 3: Allocates a single fully-coloured (8bpp) frame.
Mode 4: Provides two frames with half the colours (4bpp) each.
Mode 5: There’re two fully-coloured frames with half the size each (160x128 pixels).

Modes 3 and 5 are 16bpp (32768 colors), not 8bpp.  Mode 4 is 8bpp (256 colors), not 4bpp.
I thought the idea that a bitmap mode being restricted to only 16 colors sounded really strange.

Huh, I wonder what was I thinking back then.
Thanks!


refactor(app): update app settings advanced tab for OT-3 desktop

Overview
OT-3 desktop updates for the app settings page, including changing the default "Share Robot and App Analytics with Opentrons" to true. Creates a config value for toggling on agnostic OT-2/OT-3 protocol analysis, and a general "Enable Extended Hardware" feature flag to hide the agnostic OT-2/OT-3 protocol analysis section and any future OT-3 desktop updates that would otherwise be visible to current OT-2 users.
closes RCORE-233, closes RCORE-237
Changelog

Updates app settings advanced tab for OT-3 desktop
Changes analytics default to opt-in

Review requests
confirm match to designs, config/feature flag approach
Risk assessment
low

added feature flag works as expected and the changes look good to me. One thing that I want to know about this change. The design in Figma doesn't consider the Confirm Heater-Shaker Module Attachment section.
I'm curious about the order that the design team expected.

that section was added in 6.1: https://www.figma.com/file/695ogls23YsQYImRcGYDE1/6.1-Heater%2FShaker?node-id=5690%3A93333
looks like it should be between USB-to-Ethernet Adapter Information and Show Link to Get Labware Offset Data

added feature flag works as expected and the changes look good to me. One thing that I want to know about this change. The design in Figma doesn't consider the Confirm Heater-Shaker Module Attachment section.
I'm curious about the order that the design team expected.

that section was added in 6.1: https://www.figma.com/file/695ogls23YsQYImRcGYDE1/6.1-Heater%2FShaker?node-id=5690%3A93333
looks like it should be between USB-to-Ethernet Adapter Information and Show Link to Get Labware Offset Data

Thank you.
I just checked v6.1.0 app. Somehow the order issue passed Design Q&A.

Could you include this order issue fix in this PR?


Feature/dagger2

Added dagger2 to the Question Fragments in the app

Can you please check this out and approve it if its fine? I'll make changes to the #14 and #15 and push it. Thank you

@eunix56 try and work on the outstanding comment so that I can merge this PR
BTW well done, nice one, you're doing a great job!

Thank you. Will make update to the others and push them
On Sat, Oct 31, 2020 at 8:43 PM Joseph Olugbohunmi<EMAIL_ADDRESS>wrote:

Merged #13 https://github.com/mayorJAY/FlowOverStack/pull/13 into
development.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/mayorJAY/FlowOverStack/pull/13#event-3944443676, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AHRBQLE53HSV2YIEZXHOCA3SNRSELANCNFSM4TAYTUBQ
.


Dead code removal + simplifications

Found by running golangci-lint on the entire code base.

Except removing a few unused methods this looks like a duplicate of #2630

True. I'll restrict it to the unused methods.


fix: use relative paths in reports

We have to use relative paths from the basePath so that we don't capture filesystem paths. Also systems like SonarQube wants relative paths so that we can associate issues related to a given path.

I'm massively confused on how the tests pass locally but fail in CI.


provider: enable k3d based cluster provider

What type of PR is this?
/kind feature
What this PR does / why we need it:
This PR enables a new E2EClusterProvider for k3d based infra.
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
I have also extended the E2EClusterProviderWithImageLoader 's Image load related methods to take an additional arg ...string value to account for additional options that one can pass while performing image load operation alone.
This comes in handy with k3d where it provides a few different mode of loading images and cleanup workflows.
This change also doesn't break any existing API contract extended by the envfuncs.
Does this PR introduce a user-facing change?

enabled a new `E2eClusterProvider` for `k3d` that can be used as a new cluster provider for running e2e tests.

Additional documentation e.g., Usage docs, etc.:
NA

/cc @cpanato

go install of k3d is failing due to an issue similar to https://github.com/k3d-io/k3d/issues/1324

/hold
Holding this under the k3d install issue is resolved.

/hold clear

/unhold

/hold

Holding this until @cpanato  @vladimirvivien  @ShwethaKumbla also takes look

/retest

@vladimirvivien  PTAL. I has pushed the requested changes.

@vladimirvivien  PTAL when you can. Addressed the required changes in the message for wait for control plane.

/remove-lifecycle stale

@harshanarayana is this still on your radar for 2025?

I will get this done before we cut the christmas release..


dashboard: throttle incoming requests

To ensure service stability, let's rate limit incoming requests to our web endpoints.

Hey guys!
This change limits how we query reproducers from syzbot instance for https://github.com/cmu-pasta/linux-kernel-enriched-corpus
Could you suggest how we’d change our query approach to fetch the bug reproducers?
thanks!

Hi!
I'd suggest the following first

Cache queried bugs/reproducers on your side. If the bug is fixed, there will be no new reproducers. Even for open bugs, new reproducers don't appear too often.
Rate-limit queries to e.g. 1/sec on average. If (1) is implemented, there'll only be a few extra queries you need to make each time.

Re-querying everything each time looks really wasteful.

Thank you for the feedback! I didn’t know that new reproducers wouldn’t be added for already fixed bugs.
I’ve made the changes based on this and the CI system for the project is happy again. Thanks a bunch for quick help 💙


MarkdownString with 4 leading spaces is displayed as blank in hover.

Does this issue occur when all extensions are disabled?: No

VS Code Version: 1.64
OS Version: Windows 10

Steps to Reproduce:
I am using HoverProvider interface to show custom hovers on the editor of my vscode extension. For the hover content, I was using MarkdownString with 4 leading spaces for each line to enable syntax coloring. Till version 1.63, this was working fine. But on using vscode version 1.64, hover shows empty tooltip boxes as shown below -

Turns out that the issue gets resolved once the leading spaces are removed. But the syntax coloring does not show up if there are no leading spaces -

Here is the hover output (with 4 leading spaces added) till vscode version 1.63 -

Sample code to reproduce the issue shared here - https://github.com/Darshana-Suresh/hoverprovider-sample/blob/main/src/HoverProvider.ts

Might relate to #142445 which is reportedly fixed in the latest Insiders. Can you test with that?

Might relate to #142445 which is reportedly fixed in the latest Insiders. Can you test with that?

I've tested with the latest Insiders version (1.65.0). It doesn't solve the issue.

Is it the right way to use 4 leading spaces to enable syntax coloring in MarkdownString? Or should I be doing something else to achieve it?

Use a code fence, e.g.
```languageId
// code example
// More lines
```

@mjbvz @gjsjohnmurray this a breaking change for elixir-ls extension, see https://github.com/elixir-lsp/vscode-elixir-ls/issues/233

LSP specification states that:

If the format is markdown the content should follow the GitHub Flavored Markdown Specification

and GitHub Flavored Markdown Specification should support intended code blocks, see https://github.github.com/gfm/#indented-code-blocks


multi distro ceph cluster

Is it possible to add a client host but of a different distro ?
My Ceph cluster is installed using Ubuntu Xenial on all nodes, I 've tried to add another host as a client running Debian stretch but ansible-ceph breaks with
FAILED! => {"msg": "The task includes an option with an undefined variable. The error was: 'dict object' has no attribute 'codename'\n\nThe error appears to have been in '/opt/latest-ceph/ceph-ansible/roles/ceph-common/tasks/installs/debian_community_repository.yml': line 7, column 3, but may\nbe elsewhere in the file depending on the exact syntax problem.\n\nThe offending line appears to be:\n\n\n- name: configure debian ceph stable community repository\n  ^ here\n\nexception type: <class 'ansible.errors.AnsibleUndefinedVariable'>\nexception: 'dict object' has no attribute 'codename'"}
Harry.

Mea Culpa!
Thanks!

@grharry could have been nice to share the solution, perhaps someone could hit the same issue in the future.

Ansible was on an Ubuntu Xenial.
OSDs, mgrs etc on Ubuntu Xenial.
I just wanted to try a rados client on pre-existing  Debian  and it  broke with the message above.
All I did was a clean debian install  from scratch.


Update the Chinese language landing page to look similar to the English version

[ ] We need to: update the Chinese landing page to look like the English-language landing page and use the same assets and everything.
[ ] Because it would not have a learn page yet, we would need links to the Chinese language forum

For now this file needs updating: https://github.com/freeCodeCamp/error-pages/blob/master/landing/cn/index.html, which will eventually be replaced by the core learn lading page later.

@raisedadead I just talked with @miyaliu666  about this, and she is going to oversee the translation of the landing page as well as some of the key documents (code of conduct). For now we can use the use the same layout and copy and everything, and just link to the forum instead of linking to /learn. Then once we launch Chinese /learn we can update those links.

@raisedadead I'd like to take this up. Can I? And do I need to know Chinese?

@UsmanAhmadSaeed instead, why don't you look at the issues labeled with help wanted?

@Ieahleen I appreciate that. Except that, I've got busy and will remain so until the foreseeable future, so might not be able to contribute to any issues at all. I'd, however, try to contribute whenever I can.
Thanks and Best wishes!

Will not need this anymore.


Add support for paths with spaces when building

CMake does not work if there are spaces in paths. We can fairly easily fix this by replacing syntax of the form {VAR}/a/b/c with "{VAR}/a/b/c" in CMakeLists.txt and probably also in infra/utils.cmake.

Works as-is.


[Bug] conda install open3d requires GLIBC2.27, Ubuntu 16.04 does not support

Prerequisite

[X] I have searched Issues and Discussions but cannot get the expected help.
[X] I have read the FAQ documentation but cannot get the expected help.
[X] The bug has not been fixed in the latest version (dev-1.x) or latest version (dev-1.0).

Task
I'm using the official example scripts/configs for the officially supported tasks/models/datasets.
Branch
1.x branch https://github.com/open-mmlab/mmdetection3d/tree/dev-1.x
Environment
As the question, the environment configured according to the readme is reporting an error during inference verification. （The installation guide is located in the warehouse under docs/zh_cn/get_started.md）my Python is 3.8, cuda is 10.2, Pytorch is1.8.0.

According to the solution searched in issuse, I replaced open3d with version <IP_ADDRESS>, but this does not work because python 3.8 does not support replacing open3d with version <IP_ADDRESS>. How can I solve it?

Reproduces the problem - code sample
As the question, the environment configured according to the readme is reporting an error during inference verification. According to the solution searched in issuse, I replaced open3d with version <IP_ADDRESS>, but this does not work because python 3.8 does not support replacing open3d with version <IP_ADDRESS>. How can I solve it?
Reproduces the problem - command or script
As the question, the environment configured according to the readme is reporting an error during inference verification. According to the solution searched in issuse, I replaced open3d with version <IP_ADDRESS>, but this does not work because python 3.8 does not support replacing open3d with version <IP_ADDRESS>. How can I solve it?
Reproduces the problem - error message
As the question, the environment configured according to the readme is reporting an error during inference verification. According to the solution searched in issuse, I replaced open3d with version <IP_ADDRESS>, but this does not work because python 3.8 does not support replacing open3d with version <IP_ADDRESS>. How can I solve it?
Additional information
As the question, the environment configured according to the readme is reporting an error during inference verification. According to the solution searched in issuse, I replaced open3d with version <IP_ADDRESS>, but this does not work because python 3.8 does not support replacing open3d with version <IP_ADDRESS>. How can I solve it?

Why not use open3d of the higher version?

I followed the official installation instructions of mmdet3d and the default installation is python=3.8 and open3d=0.13.0, but this requires glibc=2.27, which is not supported by Ubuntu 16.04, who only has glibc=2.23 at the highest.

I searched the issues in mmdet3d and the solution mentioned is to open3d down to 0.9.0 (and the issue #331  says you have fixed this problem, but I still get the error). But  if I manually replace open3d with 0.9.0 it conflicts with python 3.8, which only supports open3d=0.10.0 as a minimum.

It's not that I don't want to use a higher version of open3d, but Ubuntu 16.04 doesn't support it, and using a lower version will conflict with the required python version according to your solution (and the python version can't be lowered, otherwise the installation will also error out). The above two problems, I can not solve, because they are completely contradictory.

@ranpin Two possible solutions:

A tricky solution. I guess the GLIBC2.27 is required by some dynamic link libs, like .so file. Thus, you could create a soft link for this high-version dynamic link lib to some similar low-version dynamic link libs.
You could try python3.7 and install the corresponding compatible libs.

Thanks! I tried to solve the problem from the above two perspectives.

For method 1, Since some underlying operations are involved and I don't have administrative privileges, the operation is too tedious and difficult for me.
For method 2,  I tried to change to python 3.7 version, in "pip install -v -e ." There are a lot of error warnings in this step, but we can ignore them for now. But when using the default version of open3d=0.13.0, he still reported an error that GLIBC_2.27 is needed. when I refer to the above mentioned open3d to 0.9.0 version, the error will be reported as follows.

To sum up, the main problem is still the python version, open3D version and GLIBC version compatibility issues. It is difficult for me to solve this problem, so I will try to contact the administrator to upgrade GLIBC_2.27 version. Thank you for your answer.

@ranpin Two possible solutions:

A tricky solution. I guess the GLIBC2.27 is required by some dynamic link libs, like .so file. Thus, you could create a soft link for this high-version dynamic link lib to some similar low-version dynamic link libs.
You could try python3.7 and install the corresponding compatible libs.


Don't look for jars if pom file is missing when legacy Maven behavior is disabled

Provide a way to opt-out of the legacy Maven behavior, which is to try to find a jar file if a module doesn't provide a pom file. This can lead to lots of unnecessary network requests when a module will not be found in a specific repository.

@melix I think we should also change the default behaviour when using the experimentalFeatures flag.

@melix I've "merged" your PR with a bunch of rework and refactoring. The big difference is that the link between the DSL metadataSources and the instances used in ExternalResourceResolver is much simpler and a lot more hard-coded. We translate the user config to a set of booleans to decide which metadata sources to use, and instantiate them directly if required. This is sufficient for what we need.
I brought over your LegacyArtifactLookupResolveIntegrationTest but I think we need a bit more test coverage for different combinations of metadataSources. Luckily the FlatDirectoryArtifactRepository is just an ivy repository with only the artifact() metadata source configured, but we probably want to test things like:

Opt-out of gradleMetadata() and see that the .module file isn't requested
Opt-in to artifact() with experimentalFeatures and see that we check for a jar file
Some other combinations of metadata sources, including just mavenPom() or just ivyDescriptor() or just gradleMetadata()

We might want to update AbstractModuleDependencyResolveTest so that in most cases we just test a single metadata source, and then have some separate tests that verify they work in combination.


Simplyfiying the implementation of custom content (Support for Custom HTML)

Feature Name
Support for Custom HTML
Description
I would like to propose a feature enhancement for the Swift DocC compiler to support the inclusion of custom HTML files, specifically header.html and footer.html, with the ability to apply custom elements to DocC documentations. This feature would greatly enhance the customization options for generated documentation.
The DocC compiler should be designed to automatically identify custom HTML files—such as header.html and footer.html—that are stored in a specified directory. By employing a wildcard search, the compiler can efficiently detect these files without requiring users to make manual adjustments to the build process. This approach simplifies management and integration for users who wish to incorporate multiple custom HTML files into their documentation seamlessly.
Furthermore, this enhanced functionality allows users not only to include these customized elements easily but also to style them using CSS. By leveraging CSS, users can ensure a cohesive and visually appealing presentation consistent with their branding across all documentation. This capability plays a vital role in creating a professional and unified look and feel, making the documentation more effective and engaging for its intended audience.
Motivation
Currently, the process of adding custom headers and footers to DocC documentation is not straightforward, which limits the ability to align documentation with project branding. By allowing users to include their own HTML files, we can create a more cohesive and branded documentation experience.
Importance
By streamlining this process, we could significantly enhance the overall usability of DocC documentation. Simplification would not only make it easier for developers to implement these features, but it would also open up greater flexibility and customization options. This would empower teams to align the documentation's visual aesthetics with their project branding more effectively.
A more user-friendly approach would allow developers to devote more time to content creation rather than getting lost in technical hurdles. Ultimately, investing in a streamlined process is vital for ensuring that documentation serves its purpose—providing clear, professional, and consistent guidance that reflects the essence of the project it represents. The potential benefits of such enhancements could lead to more engaging documentation experiences and stronger connections with users.
Alternatives Considered
No response

[Update 23/10/24 19:23]: While reviewing the pull requests, I found #1056, which introduces a custom-scripts.json file for managing custom scripts. This file can also be utilized to include HTML elements, allowing for greater flexibility in documentation customization. For instance, users could easily add their own header and footer HTML files through this configuration, streamlining the process of integrating custom elements into the generated documentation. For example, users can specify their custom HTML files, such as headers and footers, within the configuration like this:
{
   "htmlElements": {
       "header": "path/to/header.html",
       "footer": "path/to/footer.html"
   }
}

This approach can serve as an alternative to the previously mentioned wildcard detection method.
See the full pitch on forums.swift.org ...

As has already been pointed out in the Swift Forums thread that you linked to: DocC already supports including custom header and footer HTML files gated by the --experimental-enable-custom-templates feature flag.


Update Payment Terms & Conditions

Summary by CodeRabbit

New Features

Updated Terms of Service and Privacy Policy with new sections: "SUBSCRIPTION PAYMENT TERMS," "Payment Management," "Additional Fees," "DISCLAIMER," and "USER RESPONSIBILITIES."
Added clickable email link in the "Contact Us" section.
Included a new section showing the last revised date.

Style

Headings are now displayed in uppercase for better readability.

@sheriffjimoh -- did you check if this has the latest updated content revision shared? 👀

@sheriffjimoh -- did you check if this has the latest updated content revision shared? 👀

Haa 🤔, I updated it yeah. I check the commit too now and it's there 🤔
Or are you talking about this content? i thought it will be there too?


SP-Responsive-UI.css - image max-width breaks all SP image sprites

On ln 94 in SP-Responsive-UI.css:
img {max-width: 100%;}
This breaks all of the image sprites used for the search, ribbon, "get started" tiles, etc.

Hi, Thanks for your message. Can you share a screenshot of the issue? Thanks.

For sure, see attached. This sample is for Team Sites. Thanks!

Hi, this line doesn't exist anymore in the CSS file. This issue souldn't be closed ?

Well, actually the line is still there (https://github.com/SharePoint/PnP-Tools/blob/master/Solutions/SharePoint.UI.Responsive/SP-Responsive-UI.css#L94) but I cannot reproduce the issue ...

@PaoloPia, a question for my own culture : Why are the sources of this repo not updated in accordance with the Sites-Core repo ? Is there a reason ?
I have tried to reproduce this case too, but I didn't manage to.

hello,
i have the same problem with the responsive design.
if i activate this feature, i do not have some icons in the site.  ( i open the site with IE, Firefox and Safari)
Libary; if i upload a document: The Icons be missing

I'm guessing this is impacting ribbon icons also?  Responsive Package Active on team site on the left

Hi @DennisGoss, I have a question. This issue appear on a simple Team Site ? Without any other SP feature activated apart the RWD from PnP ? And this is a SP2016 here, isn't it ?

Hey @Laul0 - Yes brand new (vanilla) 2016 team site with Responsive Design (from PnP) Enabled. Farm is at Nov. 2016 Update - 16.0.4456.1000 though I did slipstream the updates into the Farm Install/Build which was completed via PowerShell DSC.

Hi @DennisGoss, I just tried with the latest version of Responsive-UI that comes from the Sites-Core Repo... I can't manage to reproduce this issue. Question : which source code did you use in your case ? PnP-Tools or Sites-Core ?
To prove it, look on my screenshot :

Could you give me more information... Maybe forecast one time a remote desktop to compare or whatever ? thx

I'm also having the same issue, here's whats happening:
@media only screen and (max-width: 768px)
SP-Responsive-UI.css:94
img {
    max-width: 100%;
}

The icons on the top bar are sprites and require their default size. The second you say max-width:100% the big sprite becomes a tiny image to fit the small 'window' div that displays it. Then because sprites work by offsets you no longer see the image

Hi @2toLeadMike, thx for this informations... I understand better now why I didn't manage to reproduce this issue.... May I suggest you to try this methods to "install" and activate the PnP Responsive ?:
Cause
You use the version of Responsive UI from the PnP-Tools repository but this is not the latest version of the Framework.
Bellow, one of this two steps allow you to get the latest versio.
@DennisGoss and @2toLeadMike keep me informed once you're try. thx by advance
By PowerShell Module

Go to PnP-PowerShell repo
Download the latest version in accordance with your environment
Install the PowerShell Module
Execute the Enable-PnPResponsiveUI cmdlet as is mentioned in this documentation
Test

Here an example of use case :
Connect-PnPOnline -Url https://yourdomain.sharepoint.com/ –Credentials (Get-Credential)
Enable-PnPResponsiveUI

By replacing CSS and JS files

Download the zip from the PnP-Sites-Core
Unzip archive on your computer
Get the sources files (.css and .js) into the path /PnP-Sites-Core/Core/OfficeDevPnP.Core/Resources
From the SharePoint Style Library, replace they or change the content by the content of this files
Test

FYI: I performed a PR for update the PnP-Tools repository. Once this one will be merged, you'll can enjoy this latest version directly from this repo

We closed this issue as it had not activity within last 180 days. This is a generic process we have decided to perform for issues, which have not been explicitly marked still to be "work in progress" based on tags. We are performing this cleaning to make sure that old issues that have already been solved (but not closed) or are no longer relevant are cleaned out and make the issues more manageable. If this issue still valid, we would ask you to open a new issue and follow the guidance in the issue template related on the recommended location. We do apologize any inconveniences this might cause. Please do remember that issues in the issue lists are also messages for others in the community, so you can also check if you can assist on any of them. “Sharing is caring!”


docs: adds seyph.com to showcase page

Update pages.json

chore: build showcases

Great @rickserraglia  It looks amazing

Thank you, it took a little effort but Qwik has proven to be the best choice in terms of performance.


Undefined variable in mcmc.m

When I call stan as...
RL_data = struct('n_s',ns,'n_t',nt,'Choice',choices,'Correct',outcomes,...
                 'Reward',rewards,'Subject',subj_nums,'Init',inits);

fitRL = stan('file','opzet_stan_kort_anne_2.stan','data',RL_data,'iter',1000,...
             'chains',4,'refresh',100,'warmup',500,'thin',10);

I get the following error:
??? Undefined function or variable 'rng'.

Error in ==> mcmc>mcmc.mcmc at 37
            self.rng_state = rng;

Error in ==> StanFit>StanFit.StanFit at 94
            self.sim_ = mcmc();

Error in ==> StanModel>StanModel.sampling at 744
         fit = StanFit('model',copy(self),'processes',p,...

Error in ==> stan at 137
   fit = model.sampling(p.Unmatched);

Error in ==> testRL_stan2 at 59
fitRL = stan('file','opzet_stan_kort_anne_2.stan','data',RL_data,'iter',1000,...

As far as I can tell right now it is not an issue with my call to STAN, but I may be wrong about this. Does anyone know what may be wrong? I wasn't planning on being a MatlabSTAN contributor, just a user, but I guess I might need to fix this if it isn't just operator-error.

Looks like this is actually a backwards compatibility issue, and I need to use a newer version of Matlab.

I'm not sure when rng() was introduced. I don't have a version without this function, but if you tell me how to set the random number seed in your version, we could build in some backwards compatibility.

I looked into it a bit and it seems like updating my version of matlab is the best idea. Thanks though. =)


Deploy.sh with HHVM Fatal Error

When you run deploy.yml with hhvm enabled, you get an error running post_finalize_commands.  This is due to it trying to restart php5-fpm without it existing.
TASK: [deploy | Run post_finalize_commands] ***********************************
failed: [ipaddress] => (item=sudo service php5-fpm reload) => {"changed": true, "cmd": "sudo service php5-fpm reload", "delta": "0:00:00.016006", "end": "2015-04-26 17:10:04.735646", "item": "sudo service php5-fpm reload", "rc": 1, "start": "2015-04-26 17:10:04.719640", "warnings": []}
stderr: php5-fpm: unrecognized service

FATAL: all hosts have already failed -- aborting

confirmed - face the same issue

FIX:
https://github.com/roots/bedrock-ansible/blob/master/roles/deploy/defaults/main.yml#L85-L86
handle if/else if hhvm is true
run
sudo service hhvm restart

For some reason, it just hangs there and is not ran.
I had to manually ssh into and run
sudo service hhvm restart

@rstormsf It's failing because of a sudoers issue. You can customize this variable: https://github.com/roots/bedrock-ansible/blob/5f97284dff3079563f5dcd811b9f27611a627135/roles/deploy/defaults/main.yml#L85-L86
project_post_finalize_commands:
  - sudo service hhvm restart

And this is needed too:
web_sudoers:
  - "/usr/sbin/service php5-fpm *"
  - "/usr/sbin/service hhvm *"

(or just remove php5-fpm if you aren't using it).
https://github.com/roots/bedrock-ansible/blob/5f97284dff3079563f5dcd811b9f27611a627135/group_vars/all#L17-L18

This is still an issue (bump)

@CFXd would you be interested doing a pull request? This should be easy to fix.

@louim not at this time—I only know enough to be dangerous :-/

Fixed by https://github.com/roots/trellis/pull/378


Duplicate export XRAY_ROI value in artix7 settings.sh file

The artix7 settings.sh file has the following;
https://github.com/SymbiFlow/prjxray/blob/5e1760e8ec474a3171dae09ebeb98a813b34deca/database/artix7/settings.sh#L3
and then a little further down;
https://github.com/SymbiFlow/prjxray/blob/5e1760e8ec474a3171dae09ebeb98a813b34deca/database/artix7/settings.sh#L6-L7
I assume it means the value actually being used is the second one and we can just remove the first one.

oops, this crept in during a merge issue a while back


OAuth Cipher issue on pixel 7

Can't apparently copy system log to clipboard.
Reproduce steps:

attempt to login
oauth method

Same as issue #533