Which versions of Windows Server support this feature?

The docs do not seem to specify which versions of Windows server support this feature.

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

ID: 010968d8-5582-b41a-2a6e-65477b7c9aa0
Version Independent ID: d381ce7d-4470-a391-d140-caefdd393630
Content: Windows Hypervisor Platform API Definitions
Content Source: virtualization/api/hypervisor-platform/hypervisor-platform.md
GitHub Login: @Juarezhm

We hugely appreciate your contribution, feedback, and support to our technical documentation.
We’ve phased out GitHub Issues on this repository as the feedback mechanism for content and have replaced it with a new feedback system. For more information see, Content User Feedback. As part of this rollout, we are migrating your issue where our team will use your comments to improve the documentation experience. Unfortunately, this does mean you won't be able to access this issue after the migration is complete.
All issues will be migrated by Thursday 29th August 2024.
Once again, thank you for your valuable feedback, we hope you’ll continue to contribute using our new feedback system.


API Reference Section ERR_CONTENT_DECODING_FAILED in Chrome

It appears that at some point in the last 24 hours (I was using it yesterday, December 11th 2018 it is now broken on December 12th 2018 ) the API reference section of has changed encodings in a way that breaks in Chrome. What does make this strange is that the link works in Safari (Version 12.0.1 (146<IP_ADDRESS>.1)). I'm guessing that some part of the deployment yesterday redeployed the docs and that there was an encoding error in the file that was uploaded? I am able to navigate to other parts of the website just not that specific page.
Link to url in question
https://reactnavigation.org/docs/en/api-reference.html
Chrome Error Message
This site can’t be reached The webpage at https://reactnavigation.org/docs/en/api-reference.html might be temporarily down or it may have moved permanently to a new web address.
ERR_CONTENT_DECODING_FAILED

I'm using Version 70.0.3538.110 (Official Build) (64-bit) of Chrome

the most recent deploy was 3 days ago, and this is hosted on github pages. this works for me right now, so it sounds like it could be an issue with your network

seems to happen sometimes to other github pages randomly: https://github.com/rust-lang/book/issues/709


Set up iOS slaves for nightly builds

Few changes here:

There are new auto builders for the iOS builds
Mac slaves are enabled in dev
Nightly triggers are disabled in all but prod
Dist builders are set up for iOS (building standard libraries)

r? @brson

r? @brson
(rust_highfive has picked a reviewer for you, use r? to override)

I ran this through a build in dev and miraculously they worked the first time....

http://<IP_ADDRESS>/builders/auto-mac-ios-opt/builds/0
http://<IP_ADDRESS>/builders/nightly-dist-rustc-mac-ios/builds/0


Tensorflow 1.9 for CPU, without GPU still requires cudNN - Windows

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): yes
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Windows 10
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device:
TensorFlow installed from (source or binary): pip 18.0 - pip install tensorflow
TensorFlow version (use command below): 1.9
Python version: 3.6.3
Bazel version (if compiling from source):
GCC/Compiler version (if compiling from source):
CUDA/cuDNN version: none
GPU model and memory: none
Exact command to reproduce: import tensorflow after pip install tensorflow

Describe the problem
I am working on a Win10 machine, with python 3.6.3 and using tensorflow 1.9, pip 18.0. I did not provide an option to install tensorflow with gpu, (i.e.), according to this
https://www.tensorflow.org/install/install_windows,
I used
    pip install tensorflow

and did not provide option for using GPU. However, when trying to import tensorflow, I am faced with the following error
    ModuleNotFoundError: No module named '_pywrap_tensorflow_internal'

After following various links,
https://stackoverflow.com/questions/42011070/on-windows-running-import-tensorflow-generates-no-module-named-pywrap-tenso,
https://github.com/tensorflow/tensorflow/issues/8385,
I installed the Visual studio update 3 and also used the script provided for tensorflow self check,
https://gist.github.com/mrry/ee5dbcfdd045fa48a27d56664411d41c,
and came across this following error:
    Could not load 'cudart64_80.dll'. .....
    Could not load 'nvcuda.dll' .......
    Could not load 'cudnn64_5.dll' ........

Why is my Tensorflow looking for these packages, when I installed it without GPU? MY system doesn't house a GPU at the moment. I tried uninstall and reinstalling with the upgraded pip 18.0, but the issue persists. How can this be rectified.?
Source code / logs

Errors in Detail. in Jupyter notebook while trying import -
ImportError                               Traceback (most recent call last)
c:\python36\lib\site-packages\tensorflow\python\pywrap_tensorflow_internal.py in swig_import_helper()
13         try:
---> 14             return importlib.import_module(mname)
15         except ImportError:
c:\python36\lib\importlib_init_.py in import_module(name, package)
125             level += 1
--> 126     return _bootstrap._gcd_import(name[level:], package, level)
127
c:\python36\lib\importlib_bootstrap.py in _gcd_import(name, package, level)
c:\python36\lib\importlib_bootstrap.py in find_and_load(name, import)
c:\python36\lib\importlib_bootstrap.py in find_and_load_unlocked(name, import)
c:\python36\lib\importlib_bootstrap.py in _load_unlocked(spec)
c:\python36\lib\importlib_bootstrap.py in module_from_spec(spec)
c:\python36\lib\importlib_bootstrap_external.py in create_module(self, spec)
c:\python36\lib\importlib_bootstrap.py in _call_with_frames_removed(f, *args, **kwds)
ImportError: DLL load failed: A dynamic link library (DLL) initialization routine failed.
During handling of the above exception, another exception occurred:
ModuleNotFoundError                       Traceback (most recent call last)
c:\python36\lib\site-packages\tensorflow\python\pywrap_tensorflow.py in ()
57
---> 58   from tensorflow.python.pywrap_tensorflow_internal import *
59   from tensorflow.python.pywrap_tensorflow_internal import version
c:\python36\lib\site-packages\tensorflow\python\pywrap_tensorflow_internal.py in ()
16             return importlib.import_module('_pywrap_tensorflow_internal')
---> 17     _pywrap_tensorflow_internal = swig_import_helper()
18     del swig_import_helper
c:\python36\lib\site-packages\tensorflow\python\pywrap_tensorflow_internal.py in swig_import_helper()
15         except ImportError:
---> 16             return importlib.import_module('_pywrap_tensorflow_internal')
17     _pywrap_tensorflow_internal = swig_import_helper()
c:\python36\lib\importlib_init_.py in import_module(name, package)
125             level += 1
--> 126     return _bootstrap._gcd_import(name[level:], package, level)
127
ModuleNotFoundError: No module named '_pywrap_tensorflow_internal'
During handling of the above exception, another exception occurred:
ImportError                               Traceback (most recent call last)
 in ()
1 get_ipython().run_line_magic('pylab', 'inline')
----> 2 from keras import regularizers, activations
3 from keras.models import Sequential, load_model
4 from keras.utils import np_utils,plot_model,to_categorical
5 import pandas
c:\python36\lib\site-packages\keras_init_.py in ()
1 from future import absolute_import
2
----> 3 from . import utils
4 from . import activations
5 from . import applications
c:\python36\lib\site-packages\keras\utils_init_.py in ()
4 from . import data_utils
5 from . import io_utils
----> 6 from . import conv_utils
7
8 # Globally-importable utils.
c:\python36\lib\site-packages\keras\utils\conv_utils.py in ()
7 from six.moves import range
8 import numpy as np
----> 9 from .. import backend as K
10
11
c:\python36\lib\site-packages\keras\backend_init_.py in ()
87 elif _BACKEND == 'tensorflow':
88     sys.stderr.write('Using TensorFlow backend.\n')
---> 89     from .tensorflow_backend import *
90 else:
91     # Try and load external backend.
c:\python36\lib\site-packages\keras\backend\tensorflow_backend.py in ()
3 from future import print_function
4
----> 5 import tensorflow as tf
6 from tensorflow.python.framework import ops as tf_ops
7 from tensorflow.python.training import moving_averages
c:\python36\lib\site-packages\tensorflow_init_.py in ()
20
21 # pylint: disable=g-bad-import-order
---> 22 from tensorflow.python import pywrap_tensorflow  # pylint: disable=unused-import
23 from . import app
24 from . import bitwise
c:\python36\lib\site-packages\tensorflow\python_init_.py in ()
47 import numpy as np
48
---> 49 from tensorflow.python import pywrap_tensorflow
50
51 # Protocol buffers
c:\python36\lib\site-packages\tensorflow\python\pywrap_tensorflow.py in ()
72 for some common reasons and solutions.  Include the entire stack trace
73 above this error message when asking for help.""" % traceback.format_exc()
---> 74   raise ImportError(msg)
75
76 # pylint: enable=wildcard-import,g-import-not-at-top,unused-import,line-too-long
ImportError: Traceback (most recent call last):
File "c:\python36\lib\site-packages\tensorflow\python\pywrap_tensorflow_internal.py", line 14, in swig_import_helper
return importlib.import_module(mname)
File "c:\python36\lib\importlib_init_.py", line 126, in import_module
return _bootstrap._gcd_import(name[level:], package, level)
File "", line 994, in _gcd_import
File "", line 971, in _find_and_load
File "", line 955, in _find_and_load_unlocked
File "", line 658, in _load_unlocked
File "", line 571, in module_from_spec
File "", line 922, in create_module
File "", line 219, in _call_with_frames_removed
ImportError: DLL load failed: A dynamic link library (DLL) initialization routine failed.
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "c:\python36\lib\site-packages\tensorflow\python\pywrap_tensorflow.py", line 58, in
from tensorflow.python.pywrap_tensorflow_internal import *
File "c:\python36\lib\site-packages\tensorflow\python\pywrap_tensorflow_internal.py", line 17, in
_pywrap_tensorflow_internal = swig_import_helper()
File "c:\python36\lib\site-packages\tensorflow\python\pywrap_tensorflow_internal.py", line 16, in swig_import_helper
return importlib.import_module('pywrap_tensorflow_internal')
File "c:\python36\lib\importlib_init.py", line 126, in import_module
return _bootstrap._gcd_import(name[level:], package, level)
ModuleNotFoundError: No module named '_pywrap_tensorflow_internal'
Failed to load the native TensorFlow runtime.
See https://www.tensorflow.org/install/install_sources#common_installation_problems
for some common reasons and solutions.  Include the entire stack trace
above this error message when asking for help.

Tensorflow Self Check output
ERROR: Failed to import the TensorFlow module.
WARNING! This script is no longer maintained!
Since TensorFlow 1.4, the self-check has been integrated with TensorFlow itself,
and any missing DLLs will be reported when you execute the import tensorflow
statement. The error messages printed below refer to TensorFlow 1.3 and earlier,
and are inaccurate for later versions of TensorFlow.

Python version is 3.6.

TensorFlow is installed at: C:\Python36\lib\site-packages\tensorflow

Could not load 'cudart64_80.dll'. The GPU version of TensorFlow
requires that this DLL be installed in a directory that is named in
your %PATH% environment variable. Download and install CUDA 8.0 from
this URL: https://developer.nvidia.com/cuda-toolkit

Could not load 'nvcuda.dll'. The GPU version of TensorFlow requires that
this DLL be installed in a directory that is named in your %PATH%
environment variable. Typically it is installed in 'C:\Windows\System32'.
If it is not present, ensure that you have a CUDA-capable GPU with the
correct driver installed.

Could not load 'cudnn64_5.dll'. The GPU version of TensorFlow
requires that this DLL be installed in a directory that is named in
your %PATH% environment variable. Note that installing cuDNN is a
separate step from installing CUDA, and it is often found in a
different directory from the CUDA DLLs. You may install the
necessary DLL by downloading cuDNN 5.1 from this URL:
https://developer.nvidia.com/cudnn

Could not find cuDNN.

@gunan -- pip installation problems on Windows CPU; can you advise?

I just confirmed that in our CI we can load the pip package without cudnn.
Is it possible you are running into https://github.com/tensorflow/tensorflow/issues/19584

I have the same problem. Windows 10 64-bit, Python 3.6.6, Pip 18.0. Attempting to import it using "import tensorflow" in the interactive console results in all of the same errors as in the OP. Testing with Tensor-Self-Check gives the same log as the OP. TensorFlow 1.8 works fine, but both 1.9 and 1.10 give the error. I used coreinfo to check if my CPU supports AVX, and it does.

What is your CPU model?
I confirmed that it is not cudnn we are missing. But rather having a CPU without AVX can cause similar error messages.

Thank you for your help and information. I installed 1.5 and it worked. As stated in the issue above, I think from version 1.6 and above, it requires AVX instruction sets. I did not check for them though.
I had also posted the question in stackoverflow, and here's the link to it.
According to the only answer there, am missing VCOMP140.dll, which I could not find even after installing the redistributable.

Nagging Assignee @gunan: It has been 15 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @gunan: It has been 30 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Closing as duplicate of #19584


missing content

Last sentence in the 'Schema discovery' is incomplete.

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

ID: cf7b2117-3a8d-1b35-109c-df315738beac
Version Independent ID: d31fe48a-8b7d-149f-0db5-2818a4fa07cc
Content: Azure Machine Learning integration in Power BI (Preview) - Power BI
Content Source: powerbi-docs/service-machine-learning-integration.md
Service: powerbi
Sub-service: powerbi-service
GitHub Login: @davidiseminger
Microsoft Alias: davidi

Hi, @tomwill98103 -- thanks for pointing that out! I've assigned it to the author to look at.

Thanks for pointing that out, and sorry about that! Fixed.


Null-Safety support

Will there be Null Safety support at some point?
Thanks!

Yes of course!

Do you guys have an idea about when? Thank you


Cannot read properties of undefined (reading 'typedMessage')

Status       : Error: Cannot read properties of undefined (reading 'typedMessage'). retrying in 120 seconds

I went through the browser, played one game and launched the script. It works. But it doesn't play the Mines.

oww


how to resolve interlockedOr error in windows platform?

Version: 1.x
Platform: windows

Please use the issue template.

where can i have the method of compile uv.dll in windows platform, win10 system
---Original---
From: "Bartosz<EMAIL_ADDRESS>Date: Tue, Dec 15, 2020 16:45 PM
To<EMAIL_ADDRESS>Cc: "xiaoqingnianbudadun"<765382253<EMAIL_ADDRESS>Subject: Re: [libuv/libuv] how to resolve interlockedOr error in windows platform? (#3073)
Please use the issue template.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

https://github.com/libuv/libuv#build-instructions
CMake will build both .lib and .dll.

I use the command in the piceture to build uv，but i cant find the files include .exe,.lib and dll,how can i get the files?
---Original---
From: "Bartosz<EMAIL_ADDRESS>Date: Wed, Dec 16, 2020 18:09 PM
To<EMAIL_ADDRESS>Cc: "xiaoqingnianbudadun"<765382253<EMAIL_ADDRESS>Subject: Re: [libuv/libuv] how to resolve interlockedOr error in windows platform? (#3073)
https://github.com/libuv/libuv#build-instructions
CMake will build both .lib and .dll.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.


Keccak256 returns constant output instead of error on invalid length

Aim
Attempted to test the following comment on the Keccak256 blackbox instruction:
        /// This is the number of bytes to take
        /// from the input. Note: if `var_message_size`
        /// is more than the number of bytes in the input,
        /// then an error is returned.
        var_message_size: FunctionInput<F>,

By solving a simple circuit with the instruction that has a var_message_size that's 1 greater than the actual number of bytes in the input.
Expected Behavior
Expected a user error or panic
Bug
The function fails this injectivity test, but does not fail with an error indicating var_message_size is wrong.
Note that the injectivity test with the correct var_message_size passes.
To Reproduce

Workaround
None
Workaround Description
No response
Additional Context
No response
Project Impact
None
Blocker Context
No response
Nargo Version
No response
NoirJS Version
No response
Proving Backend Tooling & Version
No response
Would you like to submit a PR for this Issue?
None
Support Needs
No response

Closing as we no longer have a keccak256 opcode


Update to Debian-12 bookworm and back port changes from live.

I didn't realize it was a temp change. I can revert it tomorrow.
On Wed, 2 Aug 2023 at 11:33 pm, Kate @.***> wrote:

@.**** commented on this pull request.
In lib/build.ml
https://github.com/ocurrent/opam-repo-ci/pull/228#discussion_r1281914648
:

@@ -235,7 +236,7 @@ let list_revdeps t ~platform ~opam_version ~pkgopt ~base ~master ~after commit =
| "" -> acc
| revdep ->
let revdep = OpamPackage.of_string revdep in

         if OpamPackage.equal pkg revdep then

         if List.exists (fun x -> OpamPackage.equal revdep x.PackageOpt.pkg) pkgs then

@tmcgilchrist https://github.com/tmcgilchrist why was this included?
Wasn't this a temporary commit? Why did you not keep the individual commits?
—
Reply to this email directly, view it on GitHub
https://github.com/ocurrent/opam-repo-ci/pull/228#pullrequestreview-1559025313,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABJXONM2KDZQDCK5IKJHVLXTJJLVANCNFSM6AAAAAA3ARX7AE
.
You are receiving this because you were mentioned.Message ID:
@.***>


http: add --user-agent option.

This should close issue #1557

Thanks a bunch!

This is missing a doc in docs/content/doc.md


Add difficulty adjustment calculation

Hi, I hit a roadblock with the current pow API. As far as I can tell, the only workaround to calculate the next work required similar to bitcoin/src/pow.cpp is to use a general big integer library, convert the Target to bytes, do the math, and convert back to Target from bytes.  I think these home brewed difficulty adjustment solutions will continually pop up, so I created a from_next_work_required method to return a Target. My work veers significantly from #2180, as I only provided a single method to do so, without further guidance on when exactly this retarget occurs.
I am happy to add tests once I get further direction from maintainers if this as a likelihood of being accepted or not. Thanks.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 23 (0.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.08%) to 83.08%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

bitcoin/src/pow.rs
0
23
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
-0.08%

Covered Lines:
19194

Relevant Lines:
23103

💛  - Coveralls

cc https://github.com/rust-bitcoin/rust-bitcoin/pull/1820 which added some different-but-related missing functionality

In 206bc621d34f0ef25b5d91a512d2e67e635f77e4:
There is trailing whitespace on several lines that I'd like you to remove. You can find them with git grep ' $'.
And could you add a couple unit tests, maybe by taking headers from the actual blockchain, that demonstrate that the adjustment does the right thing under normal circumstances, when it hits the "minimum 1 difficulty" rule, and when it hits one of the factor-of-4 rails? (The latter has never happened on Bitcoin but it should be easy to gin up a test vector by editing block timestamps, since we're not validating the PoW here.)

But concept ACK -- this functionality is missing AFAICT. Thanks for your contribution!

I approach NACKed myself over the weekend. After thinking about it I decided that the Bitcoin Core approach is less susceptible to user misinterpretation. Also when considering the difficulty adjustment it is more appropriate to be dealing with CompactTarget since it is used in the headers and has lower precision. I updated the documentation with reference to the Bitcoin Core implementation and an example. Also added 5 test cases for each scenario.

I couldn't manage to get a link to a line number for some reason so I did this instead:
https://github.com/rust-bitcoin/rust-bitcoin/pull/2180/files#r1594975932
This is what I meant by my question about the timespan parameter pushing the knowledge of the off-by-one error onto users.

@tcharding I don't understand exactly what you're suggesting. Whether the user passes a header or header.timestamp does not affect whether they will be burned by the off-by-one error.
It seems like our choices are either (a) take the whole blockchain, or at least the most recent few thousand blocks, and compute the block intervals ourselves, protecting the user, or (b) take just the two relevant blocks, trusting the user to choose the right ones.
This PR currently has both (though maybe they could be named more clearly to distinguish them), and I think we need both, since we don't want to require the user to have tons of blockheaders available when they're not even relevant.

@tcharding I don't understand exactly what you're suggesting. Whether the user passes a header or header.timestamp does not affect whether they will be burned by the off-by-one error.

Mine is just a question on API design.
In #2180 the user passes current header to next_target, that function then either returns the header.bits if the next block is not a retarget block or does the calculation, the caller does not need to be aware of the the off-by-one stuff. The function in this PR, which is basically identical to the calculate_next_target function from #2180 requires the caller to understand the off-by-one.
FTR #2180 suffers the same issue I'm describing because both CompactTarget::from_next_work_required (this PR) and calculate_next_target (#2180). @rustaceanrob has done better than me already by documenting it, I just cargo-cult-programmed the -1.

Ok, yes, you can encapsulate this by requiring the user provide an accessor for every single block in the chain. (Though you can cheat by knowing exactly which blocks it is going to access, of course.)
This PR does this, but also provides a method in which you only need to have the two relevant headers. So I think it's strictly better.

ACK ed5574f5b511ca803a9fb041d820631e7ae57158 except that we should restore the "just pass the whole chain" functionality that was present in the original PR.

restore the "just pass the whole chain" functionality

Okay, back to two Headers? I was struggling to find good names for them where the ordering is unambiguous. How is current, previous?

I think you want something like current and last_epoch_boundary.

Okay, back to two Header?

Yeah, I think we should have both.

In 0aa279aa33791dcc462f90e1574bd8d443871fbb:

If you are going to cite Core line numbers, please specify which commit of Core you are using
Sorry, I meant that we should have both a "compute from two headers" method (and I don't care if it takes timestamps or whole headers or what) and also a "compute from all the headers" method (which chooses the right headers to use).

With the compute from all the headers approach, I've only come up with two solutions, both concerned with chain height. The only height-implicit approach I know of is to pass a &[Header] representing the current header chain. For low resource devices that might be a challenge, but then again the other methods of this PR would have them covered if they take 5 minutes to research how the adjustment works. Otherwise, the explicit approach is to pass a current chain height and some function to index into the chain at a specific height as in #2180. Implied in that approach would be pinning down the Height structure. I am compelled to just go with &[Header], is that alright?

I'm leaning towards thinking this is mergable as is and we do the additional functionality in a separate PR. This PR is already a big improvement.

The only height-implicit approach I know of is to pass a &[Header] representing the current header chain.

You can take a closure which takes a usize and returns the header at that index. We have an existing PoW-related function that does this, I forget which one.
@tcharding

I'm leaning towards thinking this is mergable as is and we do the additional functionality in a separate PR. This PR is already a big improvement.

Sure, I'm happy to do this -- but we're a little ways from release and we're close to having this PR having the API we want. It's up to you @rustaceanrob. If you want me to just merge this I'll do it.

If you are going to cite Core line numbers, please specify which commit of Core you are using

Updated to included the specific commit

Sure, I'm happy to do this -- but we're a little ways from release and we're close to having this PR having the API we want. It's up to you @rustaceanrob. If you want me to just merge this I'll do it

I think this is an appropriate stopping point for now. Users are able to perform the adjustment and any additions would be additive. I am unsure how much time I will have to work on this in the next couple days. I am envisioning something similar to #2180 where there is a nice FnMut that takes a Height. In a follow up PR we can discuss this further.

git range-diff didn't work, I had to use the Compare link above.

ACK 47dc4a3 used range-diff

Bollocks

@apoelstra @tcharding how hard would it be to backport this? I want to get rid of my out-of-tree version of bitcoin in floresta and this would be very useful to have on 0.32

I think it'd be super straightforward. @tcharding can you open a PR? I suspect you can just cherry-pick this commit.

Thanks @apoelstra . If @tcharding can't do it I can give it a shot.

@Davidson-Souza that would be great actually, since then both of us can ACK it and merge it quickly.

Hey @apoelstra I see you added a new label, we have three backport labels already that are version specific - did you notice those? (Filter labels on "port")

@tcharding oh, good to know, thanks. I will rename them so they have "backport" in the name and drop mine.

Backported in #3494


BUG: Update test_lobpcg.py

Reference issue
Closes gh-17954. See also gh-15935
What does this implement/fix?
changed test for a warning to filterwarnings since the code may or not issue a warning depending on an arch.

The failing check appears unrelated, so ready to merge?

My only query would be if it's better to have
with warnings.catch_warnings():
    warnings.simplefilter("ignore")
    eigenvalues, _ = lobpcg(A, Q, maxiter=40, tol=1e-12)

instead of adding the three pytest.mark.filterwarnings decorators.

I have reversed my latest changes back to the version https://github.com/scipy/scipy/pull/17975/commits/5357bcbd970cdc6ef88be5e58ff3a46d34fde61e
that has passed all tests yesterday.
The current failure in scipy.scipy (Main refguide_asv_check) appears unrelated to this PR:
2023-02-15T17:32:52.5585583Z File "build/testenv/lib/python3.9/site-packages/scipy/sparse/linalg/_eigen/_svds.py", line 280, in svds
2023-02-15T17:32:52.5586113Z Failed example:
2023-02-15T17:32:52.5586793Z     s_a(u2[:, :1], u[:, :1]) + s_a(u2[:, 1:], u[:, 1:]) > 1e-3
2023-02-15T17:32:52.5587230Z Expected:
2023-02-15T17:32:52.5587563Z     True
2023-02-15T17:32:52.5587934Z Got:
2023-02-15T17:32:52.5588273Z     array([False])
2023-02-15T17:32:52.5588480Z
2023-02-15T17:32:52.5588662Z
2023-02-15T17:32:52.5589346Z File "build/testenv/lib/python3.9/site-packages/scipy/sparse/linalg/_eigen/_svds.py", line 283, in svds
2023-02-15T17:32:52.5589839Z Failed example:
2023-02-15T17:32:52.5590249Z     (s_a(vT2[:1, :].T, vT[:1, :].T) +
2023-02-15T17:32:52.5590826Z      s_a(vT2[1:, :].T, vT[1:, :].T)) > 1e-3
2023-02-15T17:32:52.5591243Z Expected:
2023-02-15T17:32:52.5591571Z     True
2023-02-15T17:32:52.5591866Z Got:
2023-02-15T17:32:52.5592199Z     array([False])

I'm not planning to backport the OpenBLAS bump unless there's a compelling reason, but if this testing-only change provides some additional safety I don't mind backporting it (we've already had some backports for Debian/Fedora testing-related shims). Just add the label if you think it may be warranted.


bug: happiness emojis, are they accurate? - nicabar

UI/UX - Is 95 sad? Why? The number feels high.

this is being changed


tests: drivers: eeprom testing on stm32 platforms

Add the testcase for running the test of the
stm32 eeprom driver on stm32 platforms like the nucleo_l152re board.
This PR completes (and requires) the https://github.com/zephyrproject-rtos/zephyr/pull/51246 and should detect such failure in the future.
Signed-off-by: Francois Ramu<EMAIL_ADDRESS>

Excuting west build -p -b nucleo_l152re tests/drivers/eeprom/ :
*** Booting Zephyr OS build zephyr-v3.2.0-415-g91dc927f1fbf  ***
Running tests on device "eeprom@8080000"
Running TESTSUITE eeprom
===================================================================
START - test_out_of_bounds
W: attempt to write past device boundary
 PASS - test_out_of_bounds in 0.005 seconds
===================================================================
START - test_size
 PASS - test_size in 0.001 seconds
===================================================================
START - test_write_at_fixed_address
 PASS - test_write_at_fixed_address in 0.426 seconds
===================================================================
START - test_write_at_increasing_address
 PASS - test_write_at_increasing_address in 0.213 seconds
===================================================================
START - test_write_byte
 PASS - test_write_byte in 0.108 seconds
===================================================================
START - test_write_rewrite
 PASS - test_write_rewrite in 0.200 seconds
===================================================================
START - test_zero_length_write
 PASS - test_zero_length_write in 0.028 seconds
===================================================================
TESTSUITE eeprom succeeded

------ TESTSUITE SUMMARY START ------

SUITE PASS - 100.00% [eeprom]: pass = 7, fail = 0, skip = 0, total = 7 durations
 - PASS - [eeprom.test_out_of_bounds] duration = 0.005 seconds
 - PASS - [eeprom.test_size] duration = 0.001 seconds
 - PASS - [eeprom.test_write_at_fixed_address] duration = 0.426 seconds
 - PASS - [eeprom.test_write_at_increasing_address] duration = 0.213 seconds
 - PASS - [eeprom.test_write_byte] duration = 0.108 seconds
 - PASS - [eeprom.test_write_rewrite] duration = 0.200 seconds
 - PASS - [eeprom.test_zero_length_write] duration = 0.028 seconds

------ TESTSUITE SUMMARY END ------

===================================================================
RunID: f8db9821bf1b33a2a4adf18bd6566fc3
PROJECT EXECUTION SUCCESSFUL

adding one more stm32l073 target platform for testing the eeprom driver

CI failure due to https://github.com/zephyrproject-rtos/zephyr/pull/51246


Feature: Optionally let the user choose their preferred/local currency

Bug Report Checklist

[X] I have tried restarting my IDE and the issue persists.
[X] I have pulled the latest main branch of the repository.
[X] I have searched for related issues and found none that matched my issue.

Overview
Currently, the program only suggests amount in USD, but I feel it might be more convenient to the user to have it show an estimate in their preferred currency.
Additional Info
No response

Thanks for filing @the-lazy-learner! ❤
We chatted on the Twitch stream today - Tidelift only sends payments in $USD. So in order to do this, we'd have to make a network call to determine exchange rate. And then it'd still be not the same as what Tidelift gives. I'm going to close this as wontfix - but I appreciate you filing!


[Merge] MONDO:0009453 'immune deficiency disease'

Mondo term (ID and Label)
MONDO:0009453 'immune deficiency disease'
Reason for deprecation
@sabrinatoro can you please review this?
The term above is equiv to this OMIM class: https://omim.org/entry/242850
whereas the term below is equiv to an OMIMPS:
https://www.omim.org/phenotypicSeries/PS300755,PS601495
Maybe it it just best to obsolete the term above? Or rename it? It's very confusing. Or maybe the OMIM term should be the non-heriditary form? It doesn't mention inheritance in the description.
We have acquired immunodeficiency and two inherited classes (those mentioned here) and maybe we need a grouping triad?
Term to be merged with
MONDO:0021094 'immunodeficiency disease'

@nicolevasilevsky
Here is what I have found:

MONDO:0009453 'immune deficiency disease': OMIM:242850 is a VERY old record (1973, reviewed last 2009). And the MESH id refers to this OMIMid.
I don't know enough about immunodeficiency, but the record suspects IgM deficiency.
So, I would obsolete this term based on historical reasons.

The questions now are:

Is this term the same as "immunodeficiency disease" (and ie should be merged)?
I cannot tell that these terms report to the same concept. I don't think for sure (the description in OMIM:242850 seems to describe a disease that is an immunodeficiency disease, but is more specific than the general "immunodeficiency disease". So I personally would lean towards obsoleting MONDO:0009453 instead of merging it.

What should we do with the children of MONDO:0009453?
Based on here: immunodeficiency (MONDO:0021094) :

results from a failure or absence of elements of the immune system, including lymphocytes, phagocytes, and the complement system.
have 2 "main" types:
- primary immunodeficiency = inborn errors of immunity (MONDO:0003778)
- hereditary, caused by genetic mutation that affect the immune system
- subdivided into types: T-cell deficiency, B-cell deficiency, both T-cell and B-cell deficiency, complement deficiency, phagocyte deficiency, Immunoglobulin A deficiency.
- publication
- secondary immunodeficiency = acquired immunodeficiency (I think)
- caused by steroid, obesity, AIDS,...
Therefore, I think the children of the current MONDO:0009453 can be moved to "MONDO:0021094 'immunodeficiency disease'"

Question:
is there a difference between "immune deficiency disease" and "immunodeficiency disease"?

closing in favor of https://github.com/monarch-initiative/mondo/issues/7461


Support staticcheck lint:ignore and lint:file-ignore directives

Based loosely on https://github.com/dominikh/go-tools/blob/master/lintcmd/lint.go

This is a great PR. I only have some nits before this can be merged, no functional concerns.
Worth noting to future readers that bazelbuild/rules_go#3562 also exists in rules_go as a way to ignore lint check. However, this PR implements staticcheck specific ignore directive which is a bit nicer.

I had been looking for ways to ignore lint checks, and didn't know about that PR. I'll probably open a PR against rules_go to update https://github.com/bazelbuild/rules_go/blob/master/go/nogo.rst, as it's not present there.

@sluongng would it be possible to cut a new release including this PR?


French translation

Create an FR branch with a French version

Hi, all the pages need to be translated to French, excluding the images for now.


jump to declaration should jump to starting dartdoc

Currently jump to declaration jumps to the start of the declaration site. We should likely jump to the start of any dartdoc for the symbol (or, jump to the site declaration, but scroll to ensure that any dartdoc for the symbol is visible.
@scheglov @jwren @alexander-doroshko @bwilkerson

I'm not sure if the implementation here would be entirely in the analysis server, or if the analysis server would need to return additional info and IntelliJ take advantage of that.

It currently jumps to the offset of the identifier at the declaration site (not the start of the declaration site), which seems consistent to me.
Why do you think the start of the dartdoc be more appropriate than the location of the declared identifier?
How do IntelliJ and VS Code behave for other languages?

In all languages that IntelliJ supports it jumps to the start of the name identifier and I'm sure it's the best thing to do.

I agree (which is why server currently works that way :-)
@devoncarew What's the motivation for this request?

FWIW, I also think that jumping to the identifier is the correct behavior.

This came in through a gitter channel, and is I believe the behavior that I used to see in Eclipse.

Not 100% sure that it's the right thing to do, but worth filing and discussing.

As far as I can recall, we navigated to the offset of the name in DartEditor as well, which makes me think that Eclipse worked that way too. I suppose we could try it to see.
In any case, I don't think we should deviate from the client's standard behavior without a fairly compelling reason.

The discussion appears to have tapered off, and the resolution, as I see it, is to leave the behavior as is.


Get rid of Glide

After #317 the thumbnail creation is the only place missing that uses glide.
Replace with coil and remove glide from the app

Can I work on this? If this isn't assigned to anybody yet, nice app btw. I'd also love to see material you/dynamic colors support, but maybe I should create a separate issue for that.

Sure, go ahead.
What would be needed is inside the PhotoRepository. Glide is used to create thumbnails from full size ByteArrays. The goal is to remove glide from there and replace it with coil. Also, if you want, you could wrap this logic in a more specialized class.
Also if you are done, feel free to remove the glide dependency then :)
Thank you :)

rethinkdns dev here
We use Glide.

The goal is to remove glide from there and replace it with coil.

Is it because Coil is more performant and efficient? If so, we'd want to make an effort to switch too. Our usecase is mainly 2-tier caching (memory/disk) downloaded favicons.

So, I have heard that coil is more performant than glide from coworkers. But I don't know anything about this topic.
I have used coil in the new gallery that is written in jetpack compose, because I knew its ready for compose and supported custom  image loading for the encrypted files.
Since this worked great and I could also remove another usage of glide in #317 that now uses the same code as in the gallery. I didn't want to have glide in the project alongside coil.
So this is just for the sake of not having both in the project.

Bth @starry-shivam I will review your PR once I find the time. Been busy lately with work, etc.


I could not find the frontend dir, can you please help!

You can find it here: https://github.com/aws-samples/serverless-coffee-workshop/tree/main/frontends


Raise an exception if creating a type with the same name

See https://github.com/strawberry-graphql/strawberry/pull/403/files#r463982907 for context.
Right now when creating a type we check if it is already in our type schema, if it is there we use it, but we don't check for duplicates

Hey @patrick91, I'd like to start contributing to Strawberry and this looks like a good first issue to tackle. Any advice on how to get started and where to look?
My understanding is we would want to raise an exception if someone has duplicate types. A test could look like:
def test_raises_exception_with_duplicate_types():
    @strawberry.type
    class Query:
        title: str
        age: int

    @strawberry.type
    class Query:
        title: str
        age: int

    with pytest.raises(
        TypeError, match="Schema contains duplicate types named 'Book'"
    ):
        strawberry.Schema(query=Query)

Hi @Kommandat! Thanks for your interest. There's a big PR refactor coming down the pipeline (#906) that's going to completely change how the internal typing system is handled. It's been a long time coming, but I'm aiming to have it done really soon.
I'd hold off on this issue for now. There's a potential that PR ends up resolving this issue.

Thanks @BryceBeagle! I'll hold off then. Let me know if there are any other issues you know of where I can lend a hand.

#2356 has caused a regression with generic types. Subscripted types are being counted as "duplicates".
T = TypeVar("T")

@strawberry.type
class A(Generic[T]):
    a: T

@strawberry.type
class Query:
    first: A[int]
    second: A[int]

strawberry.Schema(Query)  # raises TypeError... Duplicated type name: IntA

@coady thanks! I'll take a look :)


print api_url to handle response manually when there error == checkpoint_challenge_required

Changes:

add error msg of "api_path"
update the print format for "feedback_required"

//changes in "instabot/api/devices.py" due to the master branch is not yet merged with my prior commit
no additional changes

Close this, will request another pull request


Publish to crates.io

Hello!
If possible could you publish the rust crate to crates.io so that it can be consumed by other rust crates?
Cheers!

There already is a tree-sitter-markdown crate. Any suggestions on what to name this crate?

There already is a tree-sitter-markdown crate. Any suggestions on what to name this crate?

tree-sitter-md

Done! Feel free to reopen if you think I messed sth up. This is the first crate I published.


RedundantSort rule not working properly in case of max_by(&:size)

The goal:
return the last of most sized words in the string
A possible way to solve it:
"hello world".split(/\W+/).sort_by(&:size).last
# => "world"

This code raise rubocop offense:
RuboCop: Use `max_by` instead of `sort_by...last`. [Style/RedundantSort]

If we apply the suggested replacement, then we got a different result:
"hello world".split(/\W+/).max_by(&:size)
# => "hello"

Expected behavior
No suggest replacement when the result going to change.
Actual behavior
Suggest replacement according to Style/RedundantSort rule with result changes.
RuboCop version
1.18.4 (using Parser <IP_ADDRESS>, rubocop-ast 1.9.0, running on ruby 3.0.2 x86_64-linux)
  - rubocop-rails 2.11.3
  - rubocop-rspec 2.4.0

Isn't this a problem in general when there are multiple values that translate to the same value? Even without size, sort_by.last will take the last one and max_by will take the first one when there are multiple same values.
I think this cop needs to be marked as unsafe.


TypeError when building

Description
When building a pretty bare-bones app using adapter-auto, set up exactly according to the docs, the build fails with the below stacktrace. Building with adapter-node succeeds, but every page gives the same error 500.
Running in development mode works flawlessly. Building locally and running in preview mode also works.
Stacktrace
✓ 87 modules transformed.
.svelte-kit/output/server/vite-manifest.json                                 4.79 KiB
.svelte-kit/output/server/index.js                                           60.08 KiB
.svelte-kit/output/server/entries/endpoints/admin/dashboard/api/_server.js   4.24 KiB
.svelte-kit/output/server/entries/pages/_layout.svelte.js                    5.89 KiB
.svelte-kit/output/server/entries/pages/_layout.server.js                    0.31 KiB
.svelte-kit/output/server/entries/fallbacks/error.svelte.js                  0.60 KiB
.svelte-kit/output/server/entries/pages/admin/_layout.svelte.js              0.23 KiB
.svelte-kit/output/server/entries/pages/_page.svelte.js                      4.16 KiB
.svelte-kit/output/server/entries/pages/_page.js                             0.36 KiB
.svelte-kit/output/server/entries/pages/_page.server.js                      1.33 KiB
.svelte-kit/output/server/entries/pages/admin/dashboard/_page.svelte.js      16.78 KiB
.svelte-kit/output/server/entries/pages/admin/dashboard/_page.js             1.31 KiB
.svelte-kit/output/server/entries/pages/admin/login/_page.svelte.js          1.19 KiB
.svelte-kit/output/server/entries/pages/admin/login/_page.server.js          1.31 KiB
.svelte-kit/output/server/entries/pages/admin/register/_page.svelte.js       1.93 KiB
.svelte-kit/output/server/entries/pages/admin/register/_page.server.js       0.93 KiB
.svelte-kit/output/server/chunks/env-private.js                              0.12 KiB
.svelte-kit/output/server/chunks/env-public.js                               0.12 KiB
.svelte-kit/output/server/chunks/index.js                                    4.23 KiB
.svelte-kit/output/server/chunks/index2.js                                   0.73 KiB
.svelte-kit/output/server/chunks/index3.js                                   1.29 KiB
.svelte-kit/output/server/chunks/supabase.js                                 0.24 KiB
.svelte-kit/output/server/chunks/utils.js                                    0.22 KiB
.svelte-kit/output/server/chunks/lucia.js                                    24.95 KiB
.svelte-kit/output/server/chunks/errors.js                                   0.09 KiB
.svelte-kit/output/server/chunks/stores.js                                   1.00 KiB
.svelte-kit/output/server/chunks/hooks.js                                    0.28 KiB
undefined
TypeError [ERR_INVALID_ARG_TYPE]: The "password" argument must be of type string or an instance of ArrayBuffer, Buffer, TypedArray, or DataView. Received undefined
    at check (node:internal/crypto/scrypt:89:14)
    at Object.scryptSync (node:internal/crypto/scrypt:70:13)
    at new Encrypter (file:///home/x/x/asclepius/.svelte-kit/output/server/chunks/lucia.js:25:23)
    at new EncryptedRefreshToken (file:///home/x/x/asclepius/.svelte-kit/output/server/chunks/lucia.js:163:22)
    at handleTokens (file:///home/x/x/asclepius/.svelte-kit/output/server/chunks/lucia.js:333:35)
    at handleTokens (file:///home/x/x/asclepius/.svelte-kit/output/server/chunks/lucia.js:685:46)
    at apply_handle (file:///home/x/x/asclepius/.svelte-kit/output/server/chunks/lucia.js:54:14)
    at file:///home/x/x/asclepius/.svelte-kit/output/server/chunks/lucia.js:51:12
    at Object.handleAuth [as handle] (file:///home/x/x/asclepius/.svelte-kit/output/server/chunks/lucia.js:696:61)
    at respond (file:///home/x/x/asclepius/.svelte-kit/output/server/index.js:1611:42)
TypeError [ERR_INVALID_ARG_TYPE]: The "password" argument must be of type string or an instance of ArrayBuffer, Buffer, TypedArray, or DataView. Received undefined
    at check (node:internal/crypto/scrypt:89:14)
    at Object.scryptSync (node:internal/crypto/scrypt:70:13)
    at new Encrypter (file:///home/x/x/asclepius/.svelte-kit/output/server/chunks/lucia.js:25:23)
    at new EncryptedRefreshToken (file:///home/x/x/asclepius/.svelte-kit/output/server/chunks/lucia.js:163:22)
    at handleTokens (file:///home/x/x/asclepius/.svelte-kit/output/server/chunks/lucia.js:333:35)
    at handleTokens (file:///home/x/x/asclepius/.svelte-kit/output/server/chunks/lucia.js:685:46)
    at apply_handle (file:///home/x/x/asclepius/.svelte-kit/output/server/chunks/lucia.js:54:14)
    at file:///home/x/x/asclepius/.svelte-kit/output/server/chunks/lucia.js:51:12
    at Object.handleAuth [as handle] (file:///home/x/x/asclepius/.svelte-kit/output/server/chunks/lucia.js:696:61)
    at respond (file:///home/x/x/asclepius/.svelte-kit/output/server/index.js:1611:42)
TypeError [ERR_INVALID_ARG_TYPE]: The "password" argument must be of type string or an instance of ArrayBuffer, Buffer, TypedArray, or DataView. Received undefined
    at check (node:internal/crypto/scrypt:89:14)
    at Object.scryptSync (node:internal/crypto/scrypt:70:13)
    at new Encrypter (file:///home/x/x/asclepius/.svelte-kit/output/server/chunks/lucia.js:25:23)
    at new EncryptedRefreshToken (file:///home/x/x/asclepius/.svelte-kit/output/server/chunks/lucia.js:163:22)
    at handleTokens (file:///home/x/x/asclepius/.svelte-kit/output/server/chunks/lucia.js:333:35)
    at handleTokens (file:///home/x/x/asclepius/.svelte-kit/output/server/chunks/lucia.js:685:46)
    at apply_handle (file:///home/x/x/asclepius/.svelte-kit/output/server/chunks/lucia.js:54:14)
    at file:///home/x/x/asclepius/.svelte-kit/output/server/chunks/lucia.js:51:12
    at Object.handleAuth [as handle] (file:///home/x/x/asclepius/.svelte-kit/output/server/chunks/lucia.js:696:61)
    at respond (file:///home/x/x/asclepius/.svelte-kit/output/server/index.js:1611:42)
TypeError [ERR_INVALID_ARG_TYPE]: The "password" argument must be of type string or an instance of ArrayBuffer, Buffer, TypedArray, or DataView. Received undefined
    at check (node:internal/crypto/scrypt:89:14)
    at Object.scryptSync (node:internal/crypto/scrypt:70:13)
    at new Encrypter (file:///home/x/x/asclepius/.svelte-kit/output/server/chunks/lucia.js:25:23)
    at new EncryptedRefreshToken (file:///home/x/x/asclepius/.svelte-kit/output/server/chunks/lucia.js:163:22)
    at handleTokens (file:///home/x/x/asclepius/.svelte-kit/output/server/chunks/lucia.js:333:35)
    at handleTokens (file:///home/x/x/asclepius/.svelte-kit/output/server/chunks/lucia.js:685:46)
    at apply_handle (file:///home/x/x/asclepius/.svelte-kit/output/server/chunks/lucia.js:54:14)
    at file:///home/x/x/asclepius/.svelte-kit/output/server/chunks/lucia.js:51:12
    at Object.handleAuth [as handle] (file:///home/x/x/asclepius/.svelte-kit/output/server/chunks/lucia.js:696:61)
    at respond (file:///home/x/x/asclepius/.svelte-kit/output/server/index.js:1611:42)
file:///home/x/x/asclepius/node_modules/@sveltejs/kit/src/core/prerender/prerender.js:48
				throw new Error(format_error(details, config));
				      ^

Error: 500 /
at file:///home/x/x/asclepius/node_modules/@sveltejs/kit/src/core/prerender/prerender.js:48:11
at save (file:///home/x/x/asclepius/node_modules/@sveltejs/kit/src/core/prerender/prerender.js:336:4)
at visit (file:///home/x/x/asclepius/node_modules/@sveltejs/kit/src/core/prerender/prerender.js:223:3)
Page error
500
The "password" argument must be of type string or an instance of ArrayBuffer, Buffer, TypedArray, or DataView. Received undefined
TypeError [ERR_INVALID_ARG_TYPE]: The "password" argument must be of type string or an instance of ArrayBuffer, Buffer, TypedArray, or DataView. Received undefined

Platform

OS: Fails on Arch and Ubuntu
Node: 18.7.0
lucia-sveltekit: 0.7.1
svelte: 3.49.0
vite: 3.0.9
lucia-sveltekit/adapter-supabase: 0.3.1
sveltejs/kit: 1.0.0-next.442

Apparently, this error was caused by environment variables being inaccessible. Apologies.


Add DocumentState to the file Spec

This is a new property on document which will be added by https://github.com/sketch-hq/Sketch/issues/30429
At the moment the only thing this property will contain is information on the cloud share - it'll be empty for normal documents. The contents of cloudShare are the same as that currently written to the User.json file under document/cloudShare. The idea is to remove this information from User.json at some point, but it's still there as well for backward compatibility.
  "documentState" : {
    "cloudShare" : {
      "viewerCanUpdate" : 0,
      "id" : "615164b7-1b51-49e4-be08-33aedabc8150",
      "updatedAt" : "2020-06-17T14:54:49.378Z",
      "owner" : {
        "id" : "fad05bfc-1c06-41f4-9e91-39eb5df06765",
        "__typename" : "PublicUser",
        "updatedAt" : "2020-06-17T14:54:12.899Z",
        "createdAt" : "2020-06-17T14:54:12.899Z"
      },
      "publicUrl" : "https:\/\/staging.sketch.cloud\/s\/615164b7-1b51-49e4-be08-33aedabc8150",
      "shortId" : "615164b7-1b51-49e4-be08-33aedabc8150",
      "isPrivate" : 1,
      "currentVersion" : {
        "id" : "b9f12673-44eb-4fb9-bcfe-9336cec237df",
        "updatedAt" : "2020-06-17T14:54:49.378Z",
        "createdAt" : "2020-06-17T14:54:49.378Z"
      },
      "createdAt" : "2020-06-17T14:54:49.378Z",
      "name" : "Sketch Document",
      "libraryEnabled" : 0
    }
  },

\cc @jedrichards - please ping me if you have any questions.

@opsGavin Will this have an associated document version bump?

I've updated the version in MSDocumentVersion.h if that's what you mean.

Yeap indeed. Since the document version is serialised to disk in Sketch files, it's also part of these schemas.
I see its being increased to 132 👍

@jedrichards might be worth to include a note on the User.json about its upcoming removal?

The cloudShare property is not currently included in the user schema anyway, this was the first I knew of it 🙈

That's probably ok TBH. It should only be on cloud documents and those should be hidden away in the app support folder. Plus - we really don't want users fiddling with this stuff.

we really don't want users fiddling with this stuff.

Let's not forget we are users too 😇

That's fine - we don't want you guys fiddling with it either 😝

After a chat with @opsGavin, on the basis that cloudShare will never be visible in normal documents, and will never be visible to Assistants, it doesn't make sense to include it in the public file format, at this stage anyway. So to support 132 I'm just going to add the documentState property as an empty object. As I understand it, properties will be moving to it from user.json in future versions.

@jedrichards @opsGavin since it contains Cloud (share) information only, would it make sense to include it was workspace/cloud.json instead?

Not sure. I was talking to Jed about the workspace stuff. I'm not altogether sure what's in there, but I think we'll need to figure out how to deal with it for collaboration.

@christianklotz @opsGavin There's an open PR for this https://github.com/sketch-hq/sketch-file-format/pull/100
What's the status of this change Sketch-side? I document version bump to 132 is included in the above PR, so wondering if this has been implemented yet? If Sketch is not on 132 should we have seen Jenkins failures?

Well it's been stuck in CR / QA for about three weeks, and Jenkins has been randomly barfing at it 😣. BUT I think it's finally in the merge queue. All being well it should be merged today 🤞

@opsGavin Can I get your 👍 on this? Just want to be sure I'm adding the right property in the right place etc!
https://github.com/sketch-hq/sketch-file-format/pull/100

Looks good to me @jedrichards 👍
Also bonus points for ephemeral - what a perfect description.

:bowtie:
@opsGavin Ok that has been released as version 3.6.3. So to stop Jenkins complaining you can set in Sketch by following the instructions.

Looking at the code change we didn't introduce any information from the Cloud, rather just specified a generic object called DocumentState with a vague description

Container for ephemeral document state. For now this is just a placeholder, and will see additions in future document versions.

The motivation at the time was just to accurately specify the file format both for purposes of internal knowledge sharing, and make the Sketch Assistants linters more robust.

I'm not sure why we store so much info here - that's historical from the original Cloud project - but this issue was originally really just about moving the information out of User.json and into the document.json.
Internally the idea with "document state" is that it stores information about the document which isn't altered as the document is edited. E.g. if you make a change and then undo, any cloud information shouldn't change. Likewise if you undo a change you still want to store the latest information provided by the Sketch Assistants.
But going back to the original change, the rational was that User.json contained metadata about the user's current state of the document (scroll position, etc). If this information was lost it wouldn't really matter. On the other hand, in collab this cloud information would be critical to the state of the document. For example, knowing the last patch which was applied is vital to ensuring that the document opens in a consistent state.
The eventual goal with this was that for cloud documents User.json would only live on the client machine. That would let individual users close their documents and re-open them in the same place they were at the last save. However, if a user downloaded a copy of the document it would contain the cloud information in document.json and would be able to sync when it opened (at the time the intention was that a downloaded document would open in collab mode).
FWIW I think there's probably some tidy up work to be done here - we kept the info in User.json to maintain backwards compatibility for a few releases, but I don't believe we ever went back and removed it. I think we planned to come back to it post release, but that never happened.
This slack discussion would also give you some context.


[Help Wanted] How to only generate sequential models

Hello, sorry for disturbing you.
I am currently testing a machine learning compiler that only accepts sequential models. I would like to know if there is a straightforward method to configure NNSmith in a way that it exclusively generates sequential models. (The sequntial model here means the model can be implemented by only using one torch.nn.Sequential())
Looking foward to your response!
Thanks a lot!

Some similar feature request was proposed in #117 and implemented in #119, which aims to generate models with only one input and one output.
But that is slightly different from your goal to generate a sequential model as this mode can have parallel intermediate nodes.
But you know all models can be sequential or non-sequential per the definition of a "layer". If you regard compound operators as a whole then every multiple operator can be regarded as a compound operator leading to a sequential model. But if you look at the primary operators (such as add) then the so-called sequential models (e.g., those using transformer blocks) are actually non-sequential. i.e., I think this problem first needs to be solved by definition. :D

Thank you for your quick response!
The concept of the "compound operator" you mentioned is indeed relevant to the compiler I am going to test. I am currently further confirming on the extent to which the compound operator can express by the compiler.


Remove unused import.
JIRA: https://sapjira.wdf.sap.corp/browse/SLFMEMOBILITY-251

@roxiciobotea210 PR not connected to issue

Unit tests missing - I will re-request a review after I add them.


rpiboot won't see NVME drives

I ran latest rpiboot against a CM4 on a Waveshare-IO-BASE-A module.

uSD card is working fine - as expected

The NVME fs /dev/nvme0n1p1 and /dev/nvme0n1p2 won't be recognised though.

Is there anything I can do about it. Or is it simply not supported?
This to me would be a major drawback.

no simple backups
no simple corrections/edits
eeprom update would also require an uSD card to be inserted

It's not supported because the firmware does not have write support for NVMe. The USB-MSD firmware is a pretty hold legacy stack so unlikely to be improved.
However, the Raspberry Pi Imager embedded version can be launched via rpiboot (you need the very latest release) and that supports NVMe. I think the future direction is to load a Linux RD which can then use standard Linux commands to modify the file-system e.g. HTTP download etc or use a USB gadget driver from Linux.

I am wondering if the rpi-imager is the right tool for day2day maintenance tasks.
The way I'd see it. You'd basically would have to disassemble the whole device or you'd  need to insert and boot an uSD to run this or that maintenance task.
However. That's the way it is.
Would be nice to see a note about that IMO serious limitation in the otherwise excellent RPI online doc:
https://github.com/raspberrypi/documentation/blob/develop/documentation/asciidoc/computers/raspberry-pi/boot-nvme.adoc
Otherwise more people will run into this trap.
THX

It's more likely to be a Linux-initrd that exposes the block device(s) via the OTG in gadget mode, although MTP might also be an option. That way it will work with whatever hardware the Linux kernel supports rather than baking all of this into the VPU stack.

I see.  Let me put it my words, just to make sure I understood what you're saying.
You mean a special "maintenance" boot-mode on the NVMe could be an option, that'd put the RPI into gadget mode, while exposing the nvme filesystems.
Similar to what's being done (process wise), with the overlay feature (which also creates a special initrd). Meaning. A new
feature inside raspi-config that would enable gadget mode after a reboot would perhaps be required.

If it works then msd.elf will be replaced by a boot.img file containing a minimal Linux init-rd that exposes block devices via USB OTG gadget mode. It won't be Raspberry Pi OS or interactive and in the first instance would just replace what the custom msd.elf firmware does i.e. export EMMC. In principle it could export other block devices and since it's just a buildroot image anyone can roll their own.

@klslz I've uploaded an early release of the rpiboot replacement.  https://github.com/raspberrypi/usbboot/pull/121
The usage is pretty much the same as msd but will will export NVMe, SCSI block devices in addition to EMMC
cd mass-storage-gadget
../rpiboot -d .

Just ran a test against my Waveshare board and a Samsung PM991.
It works! Great stuff.
I had to change the clone command though.
git clone https://github.com/timg236/usbboot.git usbboot-msg
Thanx a lot for your efforts.
I have a couple more drives and MBs over here. I'll do some more
testing next week.

Just ran a test against my Waveshare board and a Samsung PM991. It works! Great stuff.
I had to change the clone command though.
git clone https://github.com/timg236/usbboot.git usbboot-msg
Thanx a lot for your efforts.
I have a couple more drives and MBs over here. I'll do some more testing next week.

Thanks. Hopefully, it can be merged into the main repo next week.

2nd test:
DFRobot DFR0767 MBO + CM4 - 8GB EMMC
OK!!

Hi Tim.
How would the eeprom update (recovery) work now??

So. Final feedback.
I just tested the Eeprom update. It also works.
I also tested a Samsung 980Pro on a PCI-E adapter on the CM4 IO Board. Works great too.
Bottom line.
3 MBOs, 3 media types. All OK.
From my POW the case can be closed as soon as the update is merged.
Thx again.

@peterharperuk @pelwell ok to merge?

@klslz Thanks for the test report. This is now merged.


Fix: pasting text that ends with an empty paragraph is ignored when the MaxLengthPlugin is enabled

This PR fixes the pasting issue with MaxLengthPlugin that was raised in this comment.
Current behavior:
https://github.com/facebook/lexical/assets/30749436/eac6fe45-4dd0-429b-9452-0c3e55b72421
Expected behavior:
https://github.com/facebook/lexical/assets/30749436/f3c8a118-947d-4894-97c8-979c2518d6fd

Thank you


Fatal error starting hellobot: ModuleNotFoundError: No module named 'aiohttp'

Followed the readme install instructions, up to Running Hellobot. Manually testing messages w/ ./signal-cli send worked.
Now when attempting to start hellobot with:
pipenv run python -m sample_bots.hellobot

Throws the fatal error:
Traceback (most recent call last):
  File<EMAIL_ADDRESS>line 197, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File<EMAIL_ADDRESS>line 87, in _run_code
    exec(code, run_globals)
  File "/Users/dsernst/Documents/forest/sample_bots/hellobot.py", line 5, in <module>
    from forest.core import Bot, Message, run_bot
  File "/Users/dsernst/Documents/forest/forest/core.py", line 33, in <module>
    import aiohttp
ModuleNotFoundError: No module named 'aiohttp'

I thought I had installed all the dependencies at the beginning? Hmm maybe I forgot... will try again with:
pipenv install --dev

And then again trying:
pipenv run python -m sample_bots.hellobot

Now getting:
Traceback (most recent call last):
  File<EMAIL_ADDRESS>line 197, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File<EMAIL_ADDRESS>line 87, in _run_code
    exec(code, run_globals)
  File "/Users/dsernst/Documents/forest/sample_bots/hellobot.py", line 5, in <module>
    from forest.core import Bot, Message, run_bot
  File "/Users/dsernst/Documents/forest/forest/core.py", line 33, in <module>
    import aiohttp
  File "/Users/dsernst/.local/share/virtualenvs/forest-RQF5VO1v/lib/python3.9/site-packages/aiohttp/__init__.py", line 5, in <module>
    from . import hdrs as hdrs
  File "/Users/dsernst/.local/share/virtualenvs/forest-RQF5VO1v/lib/python3.9/site-packages/aiohttp/hdrs.py", line 8, in <module>
    from multidict import istr
ModuleNotFoundError: No module named 'multidict'

I'm making progress by manually pip install X the different No module named X errors it's giving me.

So manually running
pip install aioprocessing asyncpg base58 phonenumberslite prometheus_async prometheus_client protobuf termcolor ulid2 pycryptodome

worked up until errors about libfuse.
Eventually realized from seeing it in all the Dockerfiles that is not a python lib.
Found https://github.com/libfuse/libfuse/, which pointed me to https://osxfuse.github.io since I'm trying to run this from OS X. Installed, which then required a restart and now we're past all the dependency errors....
So I'm personally unblocked from that particular class of errors now.
Seems like I did something wrong w/ my pipenv / virtualenv config. I tried looking into the virtualenv docs to figure out what but didn't have much luck.
Happy to close this issue now, maybe it can be helpful to others who get stuck in a similar way, or maybe there's something missing from the install instructions to prevent it in the future. I don't feel qualified to say, but since I'm unblocked, I will close.

This is an issue I'd encountered before but didn't know how to fix or reproduce. This might end up being really helpful, so thank you!

@dsernst in the end, did you fall back on running without using pipenv?
I would like to be able to fix this pipenv issue so I can provide better guideance to people, but I'm also unsure about where exactly the breakdown in dependency management is happening. It might even be a bug with Pipenv.
Reopening for now I guess, since I'm having this issue too, and it'll be good to keep that in mind. Might close again if we... change the way we're handling dependencies or something, but it's a known bug right now. I just don't know exactly how to reproduce it.

Hard to say... I haven't gotten it all working yet. Had to stop for the night and haven't had a chance to come back to it yet — hopefully later today if I can finish some other work 🙂
I believe running:
pip install aioprocessing asyncpg base58 phonenumberslite prometheus_async prometheus_client protobuf termcolor ulid2 pycryptodome

fixed the dependency issues. Because then running
pipenv run python -m sample_bots.hellobot

would show new, different errors. Making progress 😄
But my recollection was the newest error I got stuck on was along the lines of Can't find /etc/hostnames. Haven't opened an issue for it yet.
Hope this helps

Update: Just got a chance to pull in @technillogue's latest fix https://github.com/mobilecoinofficial/forest/pull/137... worked like a charm.
Hellobot is up and running! 🥂

I'm sure this issue will come up again with poetry but I can't current reproduce anything so


urdf_tutorial: 0.3.0-0 in 'melodic/distribution.yaml' [bloom]

Increasing version of package(s) in repository urdf_tutorial to 0.3.0-0:

upstream repository: https://github.com/ros/urdf_tutorial.git
release repository: https://github.com/ros-gbp/urdf_tutorial-release.git
distro file: melodic/distribution.yaml
bloom version: 0.6.4
previous version for package: null

https://github.com/ros/rosdistro/pull/17788


Make ampycloud thread-safe

Description:
This PR drops the use of yaconfigobject in favor of ruamel.yamel, and makes ampycloud thread-safe by allowing users to feed parameters directly to the run function (and storing a copy of the parameter dictionary as an instance variable). The docs is updated accordingly.
This PR also introduces/fixes a few more minor things, including:

a new CI/CD action to monitor the processing speed of the code
a few tests to check that no warnings are raised in specific occasions
a minimum number of 30 valid points are now required to run the GMM (layering step)

Error(s) fixed:
Fixes #71 , fixes #25., fixes #75, fixes #61, fixes #36, fixes #69.
Checklists:

[x] New code includes dedicated tests.
[x] New code has been linted.
[x] New code follows the project's style.
[x] New code is compatible with the 3-Clause BSD license.
[x] CHANGELOG has been updated.
[ ] AUTHORS has been updated.
[x] Copyright years in module docstrings have been updated.

Hey @loforest, this PR will result in v0.4.0.dev, and is ready for your review.
@nestabur: the code is now on develop_vof, and will be released onto develop and master as soon as it has been reviewed. By all means, feel free to test it out before then if you want. I've added a CI/CD check of the code speed - it's not fancy, but might help us avoid surprises down the line.
Note that I have dropped the use of yaconfigobject entirely, but hopefully this should be of no impact to you. You can now feed parameters directly to each run call via the prms keyword, which needs to be a nested dictionary like dynamic.AMPYCLOUD_PRMS (but only with the keys/levels that you want to change compared to the default values). All the parameters then get copied into a dedicated instance variable.
So your strategy could be:

create a YAML file with all the "common default parameters" you have, and load it before anything else via ampycloud.core.set_prms().
adjust specific values for each run call via the prms keyword.

Happy to clarify anything of course.


docker and iptable insertions

Yesterday at DockerconAmsterdam / hackaton i spoke to @shykes about the following issue:
Using UFW on the host running docker to limit access to published ports from a container. Isn't really easily possible due to (partly) the fact that the docker deaemon inserts the iptable rules before UFW is allowed to intercept traffic.
(This question is also discussed on http://serverfault.com/questions/641875/how-to-utilize-ufw-blocking-published-ports-from-docker/644079 and might be related to #1251 and #4737 )
Currently the only real way to allow block outside traffic using UFW on the host to containers within is by injecting new rules into iptables manually every container restart... ( iptables -I FORWARD 1 -p tcp -i eth0 -s [allowedsourceIP] --dport [containerport] -j ACCEPT etc)
Since the documentation doesnt really mention anything about this and because the rules are injected (and not appended by the daemon) this solution feels odd... (but might be easier to implement after #9548)

@fruitl00p #7003 has landed. I'll close this issue now and we can continue to discuss any more changes on this issue.


Add label-routes and controllers

Description
Add routes and controllers for labels
Fixes #424
Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules

hey I am facing some problems with my laptop and hopefully fix those problems and these issues soon. I am sorry for the delay @UdeshUK @ivantha


Tegan's edits to The value of Platform Engineering

Hey @mcasperson, I can't work out how to fix the validation issues here, but wondering if I should just copy all my changes over after I create a branch, which might be easier for me, then delete this?


Create a footer to the website

Hey @Jesulayomy can you assign me this issue, i would make a responsive UI for footer of the website.

Hi @JSuhas123, what links do you want to include so I can provide them for you? Or a more detailed description of what you would like to add is welcome.


fontPath not working after upgrading the android studio to 2.0 and build tools to 23.0.2

I set up Calligraphy in my application class and then calling attachBaseContext in the activity and using custom fonts for TextView by using fontPath attribute. It worked till today.
I have upgraded to latest version of android studio and build tools, then Calligraphy approach of setting custom font is not working.

attachBase the first of my views

I'm not sure what you mean by that?

Oh FYI, I'm using the new build tools 2.0.0-alpha1 too, and it works fine. I would suggest its a config issue. Look at the sample project more carefully.

Thanks for your help
Default font path is getting applied, even we set our custom font by using fontPath attribute.
Here I am providing my code snippets
In Application class:
CalligraphyConfig.initDefault(new CalligraphyConfig.Builder()
.setDefaultFontPath("fonts/Roboto-Bold.ttf")
.setFontAttrId(R.attr.fontPath).build()
Injecting context in activity:
@Override
protected void attachBaseContext(Context newBase) {
super.attachBaseContext(CalligraphyContextWrapper.wrap(newBase));
}
In XML:

How does your application class look, need to see where you create the
config. Thanks.
On Tue, 1 Dec 2015, 06:26 Bhagya Lakshmi Gaddam<EMAIL_ADDRESS>wrote:

Thanks for your help
Default font path is getting applied, even we set our custom font by using
fontPath attribute.
Here I am providing my code snippets
In Application class:
CalligraphyConfig.initDefault(new CalligraphyConfig.Builder()
.setDefaultFontPath("fonts/Roboto-Bold.ttf")
.setFontAttrId(R.attr.fontPath).build()
Injecting context in activity:
@Override https://github.com/Override
protected void attachBaseContext(Context newBase) {
super.attachBaseContext(CalligraphyContextWrapper.wrap(newBase));
}
In XML:
android:id="@+id/correct_icon"
android:layout_width="wrap_content"
android:layout_height="wrap_content"
android:layout_marginRight="2dp"
android:layout_marginLeft="15dp"
android:text="@string/fa_check"
fontPath="fonts/fontawesome-webfont.ttf"
android:textSize="30dp"
android:textColor="@color/green"
android:layout_gravity="center_vertical" />
—
Reply to this email directly or view it on GitHub
https://github.com/chrisjenx/Calligraphy/issues/226#issuecomment-160868438
.

I am creating the config inside onCreate() of Application class.
@Override
public void onCreate() {
super.onCreate();
CalligraphyConfig.initDefault(new CalligraphyConfig.Builder()
.setDefaultFontPath("fonts/fontawesome-webfont.ttf")
.setFontAttrId(R.attr.fontPath)
.build()
);
analytics = GoogleAnalytics.getInstance(getApplicationContext());
PolaEventManager.init(this);
RequestQueueManager.initRequestQueue(getApplicationContext());
}

OK, might be an issue with instant run on pre lollipop devices.

Try turning off instant run
Use a newer emulator API 21+

On Tue, 1 Dec 2015, 08:32 Bhagya Lakshmi Gaddam<EMAIL_ADDRESS>wrote:

I am creating the config inside onCreate() of Application class.
@Override https://github.com/Override
public void onCreate() {
super.onCreate();
CalligraphyConfig.initDefault(new CalligraphyConfig.Builder()
.setDefaultFontPath("fonts/fontawesome-webfont.ttf")
.setFontAttrId(R.attr.fontPath)
.build()
);
analytics = GoogleAnalytics.getInstance(getApplicationContext());
PolaEventManager.init(this);
RequestQueueManager.initRequestQueue(getApplicationContext());
}
—
Reply to this email directly or view it on GitHub
https://github.com/chrisjenx/Calligraphy/issues/226#issuecomment-160893505
.

With instant run turned on/off and using an emulator API 22 also not working.
Tried CalligraphyConfig.initDefault in an activity, then also the default font is getting applied to that activity but not the custom one.
Not sure what to do If I need two different fonts in an activity.
Is there any config do I need to set?, looks like this issue is similar to #71
FYI tried sample app with latest version of build tools and grade:

it worked properly.
I have removed .setFontAttrId(R.attr.fontPath) in application class, then also custom fonts are getting rendered properly.

Not really sure what's happening in my application.

Looks like attachBaseContext(CalligraphyContextWrapper.wrap(newBase)); is not working

This must be a weird issue with your project then. Or potentially the
packager. As 2.0.0 is an alpha right now. If it's not fixed going forwards
let me know and we can investigate.
Just odd as it works fine for me. :/
On Thu, 3 Dec 2015, 09:40 Bhagya Lakshmi Gaddam<EMAIL_ADDRESS>wrote:

Looks like attachBaseContext(CalligraphyContextWrapper.wrap(newBase)); is
not working
—
Reply to this email directly or view it on GitHub
https://github.com/chrisjenx/Calligraphy/issues/226#issuecomment-161568637
.

Finally I found like it's not working with proguard, it worked with debug build and release build without proguard.
Can we get proguard config for calligraphy?

Interesting, OK. I'll give this a go today. It should just work™ but I
haven't tried. I guess it might be striping out the reflection stuff.
On Mon, 7 Dec 2015, 12:42 Bhagya Lakshmi Gaddam<EMAIL_ADDRESS>wrote:

Finally I found like it's not working with proguard, it worked with debug
build and release build without proguard.
Can we get proguard config for calligraphy?
—
Reply to this email directly or view it on GitHub
https://github.com/chrisjenx/Calligraphy/issues/226#issuecomment-162514314
.

@bhagyalakshmi I've just updated the sample to enable proguard, Calligraphy still works as intended. Any more info about your setup? Also you upgraded to 2.0.0-alpha2?

Hey, I got the same issue.
I made a example project here:
https://github.com/migore/CalligraphyExample
My steps were:

Created a new project
Added calligraphy dependency in build.gradle
Create another TextView in content_main.xml
Added same rules from CalligraphySample in proguard-rules.pro

That sample never would work, you're missing the attachBaseContext override.
On Thu, 17 Dec 2015, 12:23 migore<EMAIL_ADDRESS>wrote:

Hey, I got the same issue.
I made a example project here:
https://github.com/migore/CalligraphyExample
My steps were:

Created a new project
Added calligraphy dependency in build.gradle
Create another TextView in content_main.xml
Added same rules from CalligraphySample in proguard-rules.pro

—
Reply to this email directly or view it on GitHub
https://github.com/chrisjenx/Calligraphy/issues/226#issuecomment-165439105
.

Argh. You are totally correct. My bad not properly reading the readme. Thanks!

@migore does it work as intended after adding the override?

Actually @bhagyalakshmi if you are able create a sample project that highlights the issue you are having that would be tremendously helpful. Most apps have a variety of moving parts so it could be a number of things.

@chrisjenx Yes. Works like a charm!

@bhagyalakshmi ,
clarify the path of your font class as the new application name in the manifest like so;
<application
android:name=".Utilities.Font"
../>


Environments and Credentials

Rebase and continuation of the PR from @boxysean

Fantastic @GtheSheep! I'm out on paternity leave, glad you could pick this up. :-)


Sample settings file is missing keys

Using the sample config produces the following error:
Traceback (most recent call last):
  File "SAMLTeamSyncAD.py", line 195, in <module>
    main()
  File "SAMLTeamSyncAD.py", line 100, in main
    adsync = ADSync(settings_file)
  File "SAMLTeamSyncAD.py", line 25, in __init__
    self.AD_PAGE_SIZE = settings['ldap']['page_size']
KeyError: 'page_size'

Solution
Add the following to the settings.yml file:
ldap:
  page_size: 1000

Added in #17


Default search relevance punchlist

From Application Narrative, these are needed for search ranking.  All of these criteria should apply to the default search that is run by Samplestack.
These points are default MarkLogic Search API behavior and did not require any special configuration in Samplestack:

More relevant to have a match on more distinct words
Higher frequency of search terms in the target document means better relevance

This was superceded in discussion, and tags are not matched by default.  However if you use tag search (tag:tagname or the checkboxes) then that search is required in all the results.

Regarding the weight of document section, matching a tag matters most, followed by in the document title

These criteria are not part of default search.  This is a gap in 1.0.0.  I think that the method for incorporating these will be to add additional-query elements to the search options, which will boost results based on these criteria.

Incorporate relative measure of vote counts on the Questions and Answers (as tracked by Post Tally and QnA Doc Score). E.g. A Question with more votes would (all else equal) appear higher in the relevant results than a Question with fewer votes.
Value accepted answers followed by answered questions over unaccepted/unanswered questions
Recency matters - more recently asked and answered questions are better match than those that are older

I've successfully incorporated the vote count criteria and have proven its efficacy.  The other queries will work along the same lines.  In each case, I'm just populating 'additional query' with some fine tuned, and advanced, cts:queries that match all documents, but contribute to score as well.

Hi @popzip before you disappear let us talk about this issue a little bit.  I've got all the controls in place to accomplish this task -- but the weights of the various queries interact in ways that are hard to understand.  I think that the requirements are too refined to really figure out whether or not they're working or not.
I suggest that we take it down to just one 'all other things being equal' criterion.  That way it's much simpler to see the affect of before and after.  The other reason I say that is because implementing this is, for the nonce, a rather ninja thing to do -- you need to use "additional-query" and populate it with XML.  The only way to have gotten this XML is to tailor cts:query in qconsole and then serialize it, and copy/paste into the search options.
Another approach would be to decorate every query in the middle tier with these boosts, but that seeme even less elegant.  I think @jmakeig will take on some improvements to this method in the 8+ or 9 timeframe though.
Here's the change I've got now, so you can see what's involved:
"additional-query": "<cts:and-query xmlns:cts=\"http://marklogic.com/cts\"><cts:json-property-range-query operator=\">=\"><cts:property>voteCount</cts:property><cts:value xsi:type=\"xs:integer\" xmlns:xs=\"http://www.w3.org/2001/XMLSchema\" xmlns:xsi=\"http://www.w3.org/2001/XMLSchema-instance\">0</cts:value><cts:option>score-function=linear</cts:option><cts:option>slope-factor=1</cts:option></cts:json-property-range-query><cts:boost-query><cts:matching-query><cts:json-property-value-query><cts:property>accepted</cts:property><cts:value xsi:type=\"xs:boolean\" xmlns:xs=\"http://www.w3.org/2001/XMLSchema\" xmlns:xsi=\"http://www.w3.org/2001/XMLSchema-instance\">true</cts:value><cts:value xsi:type=\"xs:boolean\" xmlns:xs=\"http://www.w3.org/2001/XMLSchema\" xmlns:xsi=\"http://www.w3.org/2001/XMLSchema-instance\">false</cts:value></cts:json-property-value-query></cts:matching-query><cts:boosting-query><cts:json-property-value-query weight=\"1\"><cts:property>accepted</cts:property><cts:value xsi:type=\"xs:boolean\" xmlns:xs=\"http://www.w3.org/2001/XMLSchema\" xmlns:xsi=\"http://www.w3.org/2001/XMLSchema-instance\">true</cts:value></cts:json-property-value-query></cts:boosting-query></cts:boost-query><cts:json-property-range-query operator=\">=\"><cts:property>lastActivityDate</cts:property><cts:value xsi:type=\"xs:dateTime\" xmlns:xs=\"http://www.w3.org/2001/XMLSchema\" xmlns:xsi=\"http://www.w3.org/2001/XMLSchema-instance\">2000-01-01T00:00:00</cts:value><cts:option>score-function=reciprocal</cts:option><cts:option>slope-factor=1</cts:option></cts:json-property-range-query></cts:and-query>",

I'm curious to know more about this:

I think that the requirements are too refined to really figure out whether or not they're working or not.

See the top of this ticket -- the three points that are not covered right now are:

Incorporate relative measure of vote counts on the Questions and Answers (as tracked by Post Tally and QnA Doc Score). E.g. A Question with more votes would (all else equal) appear higher in the relevant results than a Question with fewer votes.
Value accepted answers followed by answered questions over unaccepted/unanswered questions
Recency matters - more recently asked and answered questions are better match than those that are older

Each of these three are a 'boost' meaning they match all the documents in the database, but are weighted differently.  To figure out if each is really working, we need relative weights for each.  That's hard because of the different scale factors involved.

So you mean that it takes some experimentation to see if the weights are producing results where the truly more applicable results are sorted up?

yes.  with one factor (say,  higher votes) it's easy to see them at the top, when term scores are otherwise flat.  I spent some time with the three variables in play, and it's easy to make any one of them pop to the top given even term scoring, but it's not at all clear how the three weights interact.
Simplifying the requirements would make the method simpler and more understandable for people looking at samplestack.  As is, the query with the three weights is in my opinion so much opaque goo.

As an observer to the more technical aspects of this discussion, I think I agree with Charles' point that if we're trying to teach about how weights can influence result order, than implementing just one of them does the job. If we want to teach about the "algebra" of those weights, and how to tune them, that might be a subject for a dedicated demo app of some kind, and not necessarily appropriate for the main teaching goals of this project.

Ping me to discuss. Available most of today. I'm not clear on the proposal. It seems like if all else equal then each of those should boost, but there's no specific requirements on how they interact, and I agree would be complicated to parse the algebra of the combined effect. Are you proposing we do NOT use the XML you put in above?

My 2¢... with all of the code in this app that is only tangential to how to use MarkLogic, I'd think we wouldn't need to shy away from applying weights to the search, because the aforementioned teaching moment could be structured around an opportunity to teach by way of experimenting with modifying them...

If we do decide to leave all the knobs and levers in, then (to Charles' point), are we then obligated to at least talk about/explain the algebra of combining factors, to make the "opaque goo" less opaque?

If we committed to talking about every line of code we'd be in big trouble!  Is the issue that it's in XML? It's not that much code in the grand scheme of things.

The XML is fine - a demonstration of using the right data model for the purpose. I think a wiki explaining some of the factors at play, similar to what we already have in this thread, would be useful but not a pre-requisite.

(would probably add to this wiki - https://github.com/marklogic/marklogic-samplestack/wiki/Samplestack-Search-Configuration)

It's not about the code, it's about how the factors interact to determine the result ordering. Basically just responding to Charles' comment.

My proposal is to use only one of the three "all else equal" factors in the app.
Concretely, to include just this one:

Incorporate relative measure of vote counts on the Questions and Answers (as tracked by Post Tally and QnA Doc Score). E.g. A Question with more votes would (all else equal) appear higher in the relevant results than a Question with fewer votes.

The rationale for this proposal is to provide a simple example that demonstrates relevance refinement as per the application narrative.  It's been hard for me, in my working with the three separate refinements, to understand how any of them actually affect my searches...   With just one, you can see that questions with more votes tend to be near the top of simple searches.
I'll put a PR out for this change so you can see what's involved -- it will affect both node and java stacks the same.  To see it at work, you need a dbconfigure (or equivalent) after updating the options file.  restart of samplestack not necessary if its running.

@laurelnaiad makes a good point about configuration above.  I would in general agree, but in this case the config is squirrelled into an XML quoted string inside the options file, and a completely different paradigm from the rest of the app -- you'll see what I mean in the PR.

We will go with @grechaw proposal to demonstrate just 1 of the additional relevance query boosters: Vote counts impact relevance (more votes = more relevant).
Not going to address requirements for:

Accepted answers higher than answered/unaccepted higher than unaccepted/unanswered questions
More recently asked and answered questions higher than older

The latter two demonstrate the same concept and make it harder to tease out / see the impact of just one additional query.

I made progress on this a couple of weeks ago, but need to close the loop on settings for weights.  I probably have a couple more hours to look at relevance trace and get the PR in.

Hi @gghai this is the bug for the relevance PR that went through yesterday.

Verified the latest develop code w.r.t to screenshot mentioned , results are now improved base don relevance , vote count search criteria . Thanks


Surah page

Verse page fixes

[x] Verse media
[x] Word audio check for pause and end character
[x] Translations and resource names

Deployed to: http://staging.quran.com:32811

Isn't Translations and resource names already finished?

Deployed to: http://staging.quran.com:32812

Deployed to: http://staging.quran.com:32813

Deployed to: http://staging.quran.com:32814


Zero overhead Opt-In / Opt-Out

Offen right now relies on injecting a vault iframe to check for the user's consent status. This means that a user that has opted out of data  collection will still perform more than one HTTP request to determine this status. In its default configuration this does not pose real issues (static assets are cached and the server is configured to strip user related data from the access log), yet in the wild we cannot tell if people put Offen behind load balancers that log IPs and User-Agents, (accidentally) strip cache headers or similar.
Ideally, we'd have a mechanism in place that can handle this from within the script only.
Requirements
The new mechanism would need to meet the following requirements (which are also met by the current implementation):

A consent decision is persisted forever (rather until local data is wiped by the user) and can be changed at any time
A consent decision is valid per one domain that Offen is being served from
The outcome of a consent decision cannot be accessed or even changed by 3rd party scripts that run in the same context
The UI used for making the consent decision cannot be hijacked by 3rd party scripts running in the same context

Moonshot: WebComponent
Implement this as an <opt-in> web component with an API kind of like this:
<opt-in>
  <opt-in-resource>
    <script src="/analytics.js" />
  </opt-in-resource>
  <opt-in-consent>
    <button>Ok</button>
    <button>Not ok</button>
  </opt-in-consent>
</opt-in>

so it can be reused across all applications that require user consent.

A relatively simple improvement here would be to check for the consent cookie when delivering the script and return an empty document in case its value is deny. Users could still opt in again in the Auditorium as it would always load a vault.
Problem to solve with this approach: as the script currently exposes a public interface having an empty script would leak information about the consent status to third parties. This means the "empty" script couldn't really be empty but would need to provide stubs for this interface.

A relatively simple improvement here would be to check for the consent cookie when delivering the script and return an empty document in case its value is deny. Users could still opt in again in the Auditorium as it would always load a vault.

Trouble with this approach is that 3rd party scripts could infer the consent status of a user by probing for a vault element in the DOM. In case it is not there, the user must have opted out. If we'd start shipping an empty vault instead, we'd increase the traffic for users that have denied consent as the default content is revisioned and cached forever.


Call Chat and Language Model APIs?

I'm trying to create an experience similar to https://marketplace.visualstudio.com/items?itemName=robconery.pg-chat
Vscode allows you to run prompt's in the users Copilot account like this https://code.visualstudio.com/api/extension-guides/language-model
Is this something that's possible for Cody also?

Another thing that would be great, would be the ability to generate embeddings for the user queries. E.g. with the Postgres Provider I'm working on, would be awesome to do a cosine similarity search to retrieve all the relevant context from the Postgres database using pgvector.

No, neither of these are possible now with Cody, although you can use require('vscode') in an OpenCtx provider and tap into the entire VS Code extension API, including Copilot's language model API.


feat: Missing component type for non-found entity

Feature:

When an entity is returned from the Contentful API, but is not found locally in code, do not throw an error but return a dummy component instead.

Logs warning message to logger as well, for debugging/tracking missing components.

There's some build warnings to resolve

These should be fixed now; can you please confirm?


duplicate symbol '_distance' with MobileVLCKit

Related: https://github.com/xuyuanzhou/react-native-yz-vlcplayer/issues/68
thanks to @ilyapavkin

Summary
Explain the motivation for making this change: here are some points to help you:

What issues does the pull request solve?

duplicate symbol '_distance' in:
/Users/namtran/Library/Developer/Xcode/DerivedData/xxxxx-cnkfbfmejgwnrgdjwcktivwklalt/Build/Products/Debug-iphonesimulator/RNSVG/libRNSVG.a(RNSVGPathMeasure.o)
/Users/namtran/Desktop/factorynowreact/ios/Pods/MobileVLCKit/MobileVLCKit-binary/MobileVLCKit.framework/MobileVLCKit(tools.c.o)
ld: 1 duplicate symbol for architecture x86_64
clang: error: linker command failed with exit code 1

Test Plan
Demonstrate the code is solid. Example: The exact commands you ran and their output, screenshots / videos if the pull request changes UI.
What's required for testing (prerequisites)?
What are the steps to reproduce (after prerequisites)?
Compatibility

OS
Implemented

iOS
✅

Android
✅

Checklist

[x] I have tested this on a device and a simulator
[x] I added documentation in README.md
[x] I updated the typed files (typescript)
[x] I added a test for the API in the __tests__ folder

please @msand help ^_^

Hi @WoLewicki,
do you know when will be merged?
thank you so much


📝 Update includes for docs/tutorial/fastapi/update.md

📝 Update includes for docs/tutorial/fastapi/update.md

Thanks for the effort! This was covered in another PR, so I'll close this one now. :coffee: :cake:


Provide RepositoryBranch metadata for package

As documented in nuget (https://learn.microsoft.com/en-us/nuget/reference/msbuild-targets) you can provide not only the repo URL, type and commit, but also the branch. But this information is not populated by default by the SourceLink package, neither the ReproducibleBuilds (unless the PR at https://github.com/dotnet/reproducible-builds/pull/27 is merged).
So just bring the code in that PR here until that's merged and shipped.
Fixes #23

Closing, since the same fix has been merged into https://github.com/dotnet/reproducible-builds, so it'll propagate here whenever they push a new release that we can also reference. I'll update and publish a new version of PolySharp when that happens.
Thank you for contributing that upstream! 🙂

Hey @Sergio0694, TBH that project seems to be pretty much dead. No releases since Nov-2021 and nobody is actively answering anything there. Perhaps go the other way and take the change here and delete once they release?


Fixed issue where incorrect value would sometimes get passed to updateOption due to always using the value key.

Context
https://secure.helpscout.net/conversation/2499159955/60820?folderId=6987275
Summary
Sometimes, the incorrect value was getting passed into the updateOption() method due to passing in the hardcoded option.value. Instead, we need to dynamically use the key defined in settings.valueField.

This is published to NPM under 0.0.2