Illustrations are not generated when I press generate

First of all, please understand that I am Japanese and therefore my English is strange in places.
I was using TheLastBen's stable diffusion and got the following sentence and could not generate illustrations.
What should I do?
Warning: Taming Transformers not found at path /content/gdrive/MyDrive/sd/stablediffusion/src/taming-transformers/taming
Warning: CodeFormer not found at path /content/gdrive/MyDrive/sd/stablediffusion/src/codeformer/inference_codeformer.py
Warning: BLIP not found at path /content/gdrive/MyDrive/sd/stablediffusion/src/blip/models/blip.py
Traceback (most recent call last):.
File "/content/gdrive/MyDrive/sd/stable-diffusion-webui/webui.py", line 13, in <module
initialize.imports()
File "/content/gdrive/MyDrive/sd/stable-diffusion-webui/modules/initialize.py", line 30, in imports
import sgm.modules.encoders.modules # noqa: F401
ModuleNotFoundError: No module named 'sgm'.

Hello, I am a paid user of google collab and paperspace. I have the same problem in google collab, I understand I need to install some XFORMERS, but I don't know where to find it and where to download it, I'm a regular user, I'm a grandfather.
paperspace has also stopped working and I am in a stupor.
Please help me.

Hello, I am a paid user of google collab and paperspace. I have the same problem in google collab, I understand I need to install some XFORMERS, but I don't know where to find it and where to download it, I'm a regular user. paperspace has also stopped working and I am in a stupor. Please help me.

Execute all the cells of the Colab, then stop the "Start Stable-Diffusion" cell, write this on a new cell and execute it:
!pip install --pre -U xformers
Then execute again the "Start Stable-Diffusion" cell.

Hello, I am a paid user of google collab and paperspace. I have the same problem in google collab, I understand I need to install some XFORMERS, but I don't know where to find it and where to download it, I'm a regular user. paperspace has also stopped working and I am in a stupor. Please help me.

Execute all the cells of the Colab, then stop the "Start Stable-Diffusion" cell, write this on a new cell and execute it: !pip install --pre -U xformers Then execute again the "Start Stable-Diffusion" cell.

not work
ERROR: pip's dependency resolver does not currently take into account all the packages that are installed. This behaviour is the source of the following dependency conflicts.
basicsr 1.4.2 requires lmdb, which is not installed.
gfpgan 1.3.8 requires lmdb, which is not installed.
clean-fid 0.1.31 requires requests==2.25.1, but you have requests 2.28.2 which is incompatible.
torchaudio 2.1.0+cu121 requires torch==2.1.0, but you have torch 2.1.2 which is incompatible.
torchdata 0.7.0 requires torch==2.1.0, but you have torch 2.1.2 which is incompatible.
torchtext 0.16.0 requires torch==2.1.0, but you have torch 2.1.2 which is incompatible.
torchvision 0.16.0+cu121 requires torch==2.1.0, but you have torch 2.1.2 which is incompatible.

I asked LastBen directly and received the following response, which resolved the issue successfully!
use the latest notebook https://colab.research.google.com/github/TheLastBen/fast-stable-diffusion/blob/main/fast_stable_diffusion_ AUTOMATIC1111.ipynb
and rename the folder named "sd" in your gdrive to "sd_old


get the incorrent result when there is a cell with Number Format Ref #703

Description
I add a cell with Number Format in WPS. then i read this file by excelize api, but there is a incorrrent result

Thanks for your issue. I have fixed it, please try to use the master branch code, and this patch will be released in the next version.


Cmake fails for cmake version 3.10

cmake fails when installing targets defined as subdirectories for cmake version 3.10.

Cmake 3.11 is now necessary


Nested PrimOps not supported (without SplitExp)

If you run FIRRTL on the following code without SplitExpressions you get an exception
circuit Test :
  module Test :
    input a : UInt<1>
    input b : UInt<1>
    input c : UInt<1>
    output z : UInt
    z <= and(and(a,b),c)

[error] (run-main-b) firrtl.EmitterException: Can't emit firrtl.DoPrim as PrimOp argument
Should this be supported or is it correct that SplitExpressions is required for correctness?

IMO, until we can guarantee that the emitted Verilog is correct, it should error.

I agree.  Right now it's definitely not correct (e.g., bits(mux()) was being emitted as mux(), silently dropping the bits()).
In general, it's difficult to emit correct Verilog with nested expressions, because of the strange width and sign rules.

Makes sense to me!


[manager] Support initialization/allocation of weights

Support initialization and allocation of weights for LayerV2
Signed-off-by: Parichay Kapoor<EMAIL_ADDRESS>

:memo: TAOS-CI Version: 1.5.20200925. Thank you for submitting PR #1305. Please a submit 1commit/1PR (one commit per one PR) policy to get comments quickly from reviewers. Your PR must pass all verificiation processes of cibot before starting a review process from reviewers. If you are new member to join this project, please read manuals in documentation folder and wiki page. In order to monitor a progress status of your PR in more detail, visit http://nnstreamer.mooo.com/.

:octocat: cibot: @kparichay, A builder checker could not be completed because one of the checkers is not completed. In order to find out a reason, please go to http://nnstreamer.mooo.com/nntrainer/ci/repo-workers/pr-checker/1305-202106171726000.07911205291748-9a454db7878049ecf0e3feca4a42684a1096060e/.

:octocat: cibot: @kparichay, A builder checker could not be completed because one of the checkers is not completed. In order to find out a reason, please go to http://nnstreamer.mooo.com/nntrainer/ci/repo-workers/pr-checker/1305-202106221353150.50658893585205-6310e23e13e8c4f66ac30751ed0f4959f70ec99c/.

:octocat: cibot: @kparichay, A builder checker could not be completed because one of the checkers is not completed. In order to find out a reason, please go to http://nnstreamer.mooo.com/nntrainer/ci/repo-workers/pr-checker/1305-202106221412390.88802599906921-440f54fe7cf47a57427fe0f8d8e426d29fa4fea3/.


Only references pages that are new or have changed since the last build

Issue
The builder currently doesn't access to the saved environment, so it can only see pages that are new or have changed since the last build, possibly leading to an incomplete sitemap.
Work around
To get around this you can manually clean the build directory or use the -E flag to not use the saved environment and rebuild completely as part of your deployment process.
Possible fix
The saved environment is pickled after the parsing stage, so accessing it may make it possible to always produce a full sitemap. Using doctree-resolved instead might be a possibility.
Temp fix
Test if sphinx is running a partial build and output a warning.

We are suffering from this bug.
Let me explain the context: We use Sphinx for a LaTeX FAQ that contains more than 1200 pages. We've worked hard to ensure that only modified pages are automatically (thanks to a GitLab CI/CD process) (re)generated each time a contributor modifies a source file (Markdown). In fact, recompiling all the pages each time was too time-consuming (around 15 minutes) and unnecessarily energy-consuming.
Now, when a contributor modifies a single source file, the sitemap.xml contains (almost?) only the path to the corresponding .html file.
We'd be delighted if you could provide us with a fix! :smiley:


inconsistency meaning

Hi,
I am trying to understand consistent problem of feature importance.
If I using the same dataset, and fitting two different parameters of model(e.g., xgboost),
then I got two different mean|SHAP| values where the ranking of feature were diffferent.
Does this mean inconsistency?

X_train,X_test,Y_train,Y_test = train_test_split(*shap.datasets.iris(), test_size=0.2, random_state=0)
model = xgboost.XGBClassifier(n_estimators=50, max_depth=12)
model.fit(X_train, Y_train)

model = xgboost.XGBClassifier(n_estimators=100, max_depth=3)
model.fit(X_train, Y_train)

A model (such as XGBoost) is just a way to approximate an unkown a functional relationship found in a dataset. Different training parameters will lead to different models and those models may depend a bit differently on different input features. This is what you are seeing. "Inconsistency" as used when defining the Shapley values is a case where the importance measure does not match what the model depends on. The differences you are seeing in this example are not this, but rather are due to real differences between the models (which I can say because TreeExplainer has consistency guarantees).

Thanks for the answer, it is helpful.
So in Tree SHAP arXiv paper Fig.1, there are two models were trained on the same one dataset, the output [Cough&Fever]*80 is the model_A's output but not the ground truth, but in model_B we let the Cough be a bit more important than Fever (I think maybe it's done by setting Cough feature a higher weight?).
I am not sure if there is any misunderstanding.

In that figure we are showing an example where the model is so simple that we can tell that Model B always depends more on Cough than Model A does. Model B depends more on Cough because we trained it on a dataset with different labels (ones that cause the model to depend more on cough).
While being inconsistent can (and often does) cause the importance ranking to be wrong, it is fundamentally a question of if we can find two models where the second model depends more on a specific feature than the first model (in a very obvious sense made precise by the axiom), but that feature gets less credit in the second model than the first. This is what we see in that figure.

Model B depends more on Cough because we trained it on a dataset with different labels

So actually Model A and B are trained on different target, if that, why could we compare feature importance between these two models?
Can I just judge from output of Model B : [Cough&Fever]*80+Cough*10 which is obviously that Cough important than Fever but result of Gain is not, so Gain is inconsistent?
Sorry about this issue doesn't helpful for SHAP package growth, but I'm really want to figure out.

Unless you have a random training process you will always get the same model when you train on the same labels, and comparing models is really only interesting when they are different. In this case I think the best way to think about it is as you said, Model B clearly has a higher dependence on Cough than Model A (just by intuitive inspection), yet the gain credit is lower for Cough in Model B. There is a way to make that intuitive inspection mathematically precise, but it is just the same idea and much more involved.

@slundberg ,

@slundberg ,
I have a similar kind of issue.
I am getting the inconsistent feature's contribution for same input dataset. Contributions are always different in every run. We are working on the compliance project and consistency of  contributions are very crucial for us.


Build error: Transforms ... all emitted asset ....

I have simple transformer, which combines two files into one js file. It causes a build error:
Build error:
Transforms {AlgoStoreMirror on adl_client|workerJs} all emitted asset adl_client|web/js/algo_store_mirror_worker.js.

The file is generated correctly and works fine, but the error message is annoying.
Here is the transformer.
library adl_client.lib.algo_store_mirror_transformer;

import 'dart:async';
import 'package:barback/barback.dart';

class AlgoStoreMirrorTransformer extends AggregateTransformer {

  final List<String> includes = [
    'lib/js_src/vendor/q/q.js',
    'lib/js_src/services/algo-store/algo-store-mirror-worker.js.tpl'
  ];

  AlgoStoreMirrorTransformer.asPlugin();

  classifyPrimary(AssetId id) {
    if (includes.contains(id.path)) {
      return 'workerJs';
    }
    return null;
  }

  Future apply(AggregateTransform transform) async {
    var buffer = new StringBuffer();
    var assets = await transform.primaryInputs.toList();
    assets.sort((x, y) => includes.indexOf(x.id.path).compareTo(includes.indexOf(y.id.path)));
    for (var asset in assets) {
      buffer.write(await asset.readAsString());
      buffer.write('\n');
    }
    var id = new AssetId(transform.package, 'web/js/algo_store_mirror_worker.js');
    transform.addOutput(new Asset.fromString(id, buffer.toString()));
  }
}

It was my mistake. I have another AggregateTransformer which write two transform type into one file. This error goes away after I fixed the other transformer.


Make layout nice.

The layout is a little too fluid.

This is not a very good description, 2 months past me.

This is not a very good description, 2 months past me.

https://github.com/frcroth/elections/commit/6686ea99c0e9f32015b7f3d86aeb42e10bc48dcb

https://github.com/frcroth/elections/commit/6686ea99c0e9f32015b7f3d86aeb42e10bc48dcb


whereRaw for join clause

As The title says, this function is currently missing.

This can usually be solved by DB::raw. What is your use case?

I need to add an on clause to my join that use the postgres unaccent
extension so I need WhereRaw since where cant use unaccent function
2015-11-22 17:34 GMT+02:00 Anže Časar<EMAIL_ADDRESS>
This can usually be solved by DB::raw. What is your use case?
—
Reply to this email directly or view it on GitHub
https://github.com/laravel/framework/issues/11053#issuecomment-158767074
.

Not familiar with unaccent functions, but still something like that should work:
->on('col1', '=', DB::raw('my_function(col2)'));

Thank you I will give it a try
2015-11-22 20:58 GMT+02:00 Anže Časar<EMAIL_ADDRESS>
Not familiar with unaccent functions, but still something like that should
work:
->on('col1', '=', DB::raw('my_function(col2)'));
—
Reply to this email directly or view it on GitHub
https://github.com/laravel/framework/issues/11053#issuecomment-158789523
.

It can be achieved like this:
->on('col1', '=', DB::raw('my_function(col2)'));

But not like this:
->on('col1', '=', 'something')->where('col1', '=', DB::raw('my_function(col2)'));

Don't know why


mouse position for clicking on the external link

upon clicking of the google slides link on the git with seniors workshop card, the mouse has to be at a very specific position to get to the google slides

fixed with PR #40


Add airbnben to Contributors list

success!

@airbnben, I'm quite elated about your PR. I wanna evolve this project to addresses various problems faced by first-time contributors. I'd love to learn about your journey in open source community, the problems, pain points you had etc.
Could you explain how you felt when you went through the tutorial, made a PR and learned that I merged it?


mlogc  Invalid entry (failed to match regex)

Hi,
At first , the mlogc 2.9.2 can not find the log file of Modsecurity V3,
Then I  modified mlogc-batch-load.pl : replace /^\d{8}-\d+-.{24,}$/s with /^\d{8}-\d+-[\d\.]*$/s  at line 41. But it still can not send anything to waf-fle.
There are some debug info from mlogc:
[Thu Oct 12 18:33:52 2017] [4] [25040/0] Transaction initialization locking global mutex.
[Thu Oct 12 18:33:52 2017] [4] [25040/0] Transaction initialization started.
[Thu Oct 12 18:33:52 2017] [4] [25040/0] Transaction initialization completed.
[Thu Oct 12 18:33:52 2017] [4] [25040/0] Transaction initialization unlocking global mutex.
[Thu Oct 12 18:33:52 2017] [5] [25040/0] Internal state: [evnt "0"][curr "0"][next "0"][nbytes "65536"]
[Thu Oct 12 18:33:52 2017] [5] [25040/0] Read 143 bytes from pipe: `test.com - - - - \"-\" - - \"-\" \"-\" - \"-\" /20171012/20171012-1729/20171012-172928-150780056844.953879 0 1726 md5:5021d7fd7925f414d2e8c829c8209e51\n'
[Thu Oct 12 18:33:52 2017] [5] [25040/0] Received audit log entry (count 1 queue 0 workers 0): test.com - - - - \"-\" - - \"-\" \"-\" - \"-\" /20171012/20171012-1729/20171012-172928-150780056844.953879 0 1726 md5:5021d7fd7925f414d2e8c829c8209e51
[Thu Oct 12 18:33:52 2017] [4] [25040/0] Queue locking thread mutex.
[Thu Oct 12 18:33:52 2017] [4] [25040/0] Worker creation started.
[Thu Oct 12 18:33:52 2017] [4] [25040/0] Creating thread_pool.
[Thu Oct 12 18:33:52 2017] [4] [25040/0] Worker creation completed: e21658
[Thu Oct 12 18:33:52 2017] [4] [25040/0] Queue unlocking thread mutex.
[Thu Oct 12 18:33:52 2017] [4] [25040/0] Processed 1 entries from buffer.
[Thu Oct 12 18:33:52 2017] [5] [25040/0] Shifted buffer back 143 and offset 0 bytes for next read: `'
[Thu Oct 12 18:33:52 2017] [5] [25040/0] Internal state: [evnt "0"][curr "0"][next "0"][nbytes "65536"]
[Thu Oct 12 18:33:52 2017] [3] [25040/0] No more data to read, emptying buffer: End of file found
[Thu Oct 12 18:33:52 2017] [3] [25040/0] Waiting for queue to empty (1 active).
[Thu Oct 12 18:33:52 2017] [4] [25040/daf338] Signal thread: Starting.
[Thu Oct 12 18:33:52 2017] [4] [25040/daf2a8] Management thread: Starting.
[Thu Oct 12 18:33:52 2017] [5] [25040/daf2a8] Management thread: Processing
[Thu Oct 12 18:33:52 2017] [5] [25040/daf2a8] Management thread: Last checkpoint was 0 seconds ago.
[Thu Oct 12 18:33:52 2017] [4] [25040/e21658] Worker thread starting.
[Thu Oct 12 18:33:52 2017] [4] [25040/e21658] Worker fetch locking thread mutex.
[Thu Oct 12 18:33:52 2017] [4] [25040/e21658] Worker fetch started.
[Thu Oct 12 18:33:52 2017] [4] [25040/e21658] Getting one entry from the queue.
[Thu Oct 12 18:33:52 2017] [4] [25040/e21658] Worker fetch completed.
[Thu Oct 12 18:33:52 2017] [4] [25040/e21658] Worker fetch unlocking thread mutex.
[Thu Oct 12 18:33:52 2017] [4] [25040/e21658] Processing entry.
[Thu Oct 12 18:33:52 2017] [2] [25040/e21658] Invalid entry (failed to match regex): test.com - - - - \"-\" - - \"-\" \"-\" - \"-\" /20171012/20171012-1729/20171012-172928-150780056844.953879 0 1726 md5:5021d7fd7925f414d2e8c829c8209e51
[Thu Oct 12 18:33:52 2017] [4] [25040/e21658] Worker processing completed.
[Thu Oct 12 18:33:52 2017] [4] [25040/e21658] Worker fetch locking thread mutex.
[Thu Oct 12 18:33:52 2017] [4] [25040/e21658] Worker fetch started.
[Thu Oct 12 18:33:52 2017] [4] [25040/e21658] Removing previous entry from storage.
[Thu Oct 12 18:33:52 2017] [4] [25040/e21658] Getting one entry from the queue.
[Thu Oct 12 18:33:52 2017] [4] [25040/e21658] Worker fetch unlocking thread mutex.
[Thu Oct 12 18:33:52 2017] [4] [25040/e21658] No more work for this thread, exiting.
[Thu Oct 12 18:33:52 2017] [4] [25040/e21658] Worker shutdown locking thread mutex.
[Thu Oct 12 18:33:52 2017] [4] [25040/e21658] Worker shutdown unlocking thread mutex.
[Thu Oct 12 18:33:52 2017] [4] [25040/e21658] Worker thread completed.
[Thu Oct 12 18:33:52 2017] [4] [25040/0] Shutting down

I had fix this problem by myself.
Replace  %^--[0-9A-Fa-f]{8}-([A-Z])--$% with %^---[0-9A-Za-z]{8}---([A-Z])--$% at mlogc-batch-load.pl  line 90.
And the waf-fle 0.6.4 shall not apply to modsecurity V3 . With lots of regular expressions are wrong.

We would recommend the utilization of the JSON logging instead of the old native format. It can save you some trouble as the regular expressions may not be need any longer.


fix COCKPIT_ENV_ROOT path detection in /media/api controller

fixes #1200

Closing for now because it leads in the wrong direction.


Installation requires jq

Heads up - installation using Terraform on Ubuntu 18.04 requires 'jq' to be installed - otherwise it will fail mid-install and be difficult to roll back.

Closed in #172


[FIX] Fix syntax error in docker workflow and add relevant pre-commit hook

See also
Changes proposed in this pull request:

Fixes an error in the workflow that causes it to fail (see https://github.com/neurobagel/federation-api/actions/runs/6632688294)
Adds new pre-commit hook that will complain about invalid GitHub workflows in future

Checklist

[x] PR has an interpretable title with a prefix ([ENH], [FIX], [REF], [TST], [CI], [MNT], [INF], [MODEL], [DOC]) (see https://neurobagel.org/contributing/pull_requests for more info)
[ ] PR links to GitHub issue with mention Closes #XXXX
[x] Tests pass
[x] Checks pass

For new features:

[x] Tests have been added

For bug fixes:

[x] There is at least one test that would fail under the original bug conditions.

:rocket: PR was released in v0.1.0 :rocket:


Doesn't work with vite.js

This is the same problem stated here: https://github.com/vitejs/vite/issues/2139#issuecomment-1024852072
I'm not familiar enough with the build tools to provide a PR and make sure that there's no breaking changes, sorry.

Thanks @emi2k01 for the issue.
I will check this.

Same problem here.
Have you found a solution to this problem?

import ReactRouterPromptRaw from "react-router-prompt";

const ReactRouterPrompt = (ReactRouterPromptRaw as any).default
  ? (ReactRouterPromptRaw as any).default
  : ReactRouterPromptRaw;

then use <ReactRouterPrompt />

@SonOfWinter

Package now uses vite.js tooling. Should work with new version 0.5.0
Thanks for reporting issue here!


error:  only the following media types are accepted: application/json, application/yaml

What kind of request is this (question/bug/enhancement/feature request):
bug
Steps to reproduce (least amount of steps as possible):
install rancher 2.4.3 helm3 chart
access rancher and click on "Try Dashboard"
Result:
Blank page with following error:
error:  only the following media types are accepted: application/json, application/yaml
Other details that may be helpful:
Environment information

Rancher version (rancher/rancher/rancher/server image tag or shown bottom left in the UI):
|Versions|Rancher v2.4.3 UI: v2.4.22 |
Installation option (single install/HA):
HA

Cluster information

Cluster type (Hosted/Infrastructure Provider/Custom/Imported): Imported
Machine type (cloud/VM/metal) and specifications (CPU/memory): VM
Kubernetes version (use kubectl version):

Client Version: version.Info{Major:"1", Minor:"14", GitVersion:"v1.14.1", GitCommit:"b7394102d6ef778017f2ca4046abbaa23b88c290", GitTreeState:"clean", BuildDate:"2019-04-08T17:11:31Z", GoVersion:"go1.12.1", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"14", GitVersion:"v1.14.1", GitCommit:"b7394102d6ef778017f2ca4046abbaa23b88c290", GitTreeState:"clean", BuildDate:"2019-04-08T17:02:58Z", GoVersion:"go1.12.1", Compiler:"gc", Platform:"linux/amd64"}

Docker version (use docker version):

Client: Docker Engine - Community
 Version:           19.03.8
 API version:       1.40
 Go version:        go1.12.17
 Git commit:        afacb8b
 Built:             Wed Mar 11 01:27:04 2020
 OS/Arch:           linux/amd64
 Experimental:      false

Server: Docker Engine - Community
 Engine:
  Version:          19.03.8
  API version:      1.40 (minimum version 1.12)
  Go version:       go1.12.17
  Git commit:       afacb8b
  Built:            Wed Mar 11 01:25:42 2020
  OS/Arch:          linux/amd64
  Experimental:     false
 containerd:
  Version:          1.2.13
  GitCommit:        7ad184331fa3e55e52b890ea95e65ba581ae3429
 runc:
  Version:          1.0.0-rc10
  GitCommit:        dc9208a3303feef5b3839f4323d9beb36df0a9dd
 docker-init:
  Version:          0.18.0
  GitCommit:        fec3683

We are experiencing the same issue.

Same here

Use case 1:
Tested on Rancher v2.4.3  HA setup

Steps:

Create RKE cluster with K8s version v1.14.1-rancher1-2 [using RKE CLI]
Import the cluster in the HA setup.
Once the cluster became active, immediately access the Dashboard.
Result:
Observed the below error for 5s or so:

Accessed the dashboard again after 10 seconds. Dashboard was launched successfully.

Use case 2:
Observed the same behavior as above on a single node setup upgraded from v2.3.1 to v2.4.3.
[RKE cluster was imported in the upgraded setup]

Is it resolved?

Is there any news on this ? Still not working in any of my clusters.

Please check https://github.com/rancher/rancher/issues/26980 as well

Updated the k8s version to v1.17.5 under my Rancher HA instance but this don't working

I updated my k8s running ha rancher and its working now.


Use IPFS!

Use IPFS!
You don't need servers to host files after that. And it's the users job to keep their files for as long as they
need.

While there isn't a need with IPFS to have a public host it still makes everything much nicer when you don't need to have your computer on to serve files (it's not good for mobile phones to serve files stable for IPFS for battery and bandwidth issues).
Having hosting would still be very valuable if IPFS would be implemented. But of course, FileCoin would be nicer than centralized hosting.


Stack overflow/segfault in RocksDB stress test

Running into a stack overflow/segfault when running RocksDB in multiplayer::orb_can_render_peers_in_the_sphere_address_book, discovered in #655.
To summarize, introducing RocksDB causes a segfault in one of our stress tests, consistently reproducible locally and in CI, depending on things that we wouldn't expect to cause this, like the shape of some structs. In #655, we store a Storage instance in SphereDb. This change alone causes the segfault:
--- a/rust/noosphere-storage/src/db.rs
+++ b/rust/noosphere-storage/src/db.rs
@@ -40,6 +40,7 @@ where
     link_store: S::KeyValueStore,
     version_store: S::KeyValueStore,
     metadata_store: S::KeyValueStore,
+    storage: S,
 }

 impl<S> SphereDb<S>
@@ -52,6 +53,7 @@ where
             link_store: storage.get_key_value_store(LINK_STORE).await?,
             version_store: storage.get_key_value_store(VERSION_STORE).await?,
             metadata_store: storage.get_key_value_store(METADATA_STORE).await?,
+            storage: storage.to_owned(),
         })
     }

In #655, changing RocksDbStore's name property to be an Arc "fixes" the segfault:
--- a/rust/noosphere-storage/src/implementation/rocks_db.rs
+++ b/rust/noosphere-storage/src/implementation/rocks_db.rs
@@ -85,13 +85,13 @@ impl Storage for RocksDbStorage {

 #[derive(Clone)]
 pub struct RocksDbStore {
-    name: String,
+    name: Arc<String>,
     db: Arc<DbInner>,
 }

 impl RocksDbStore {
     pub fn new(db: Arc<DbInner>, name: String) -> Result<Self> {
-        Ok(RocksDbStore { db, name })
+        Ok(RocksDbStore { db, name: Arc::new(name) })
     }

While Arc is more appropriate here anyway, it shouldn't have an effect on this segfault. Using an even more appropriate Cow instead still fails. That is to say, there is some spooky issue regardless of #655 and using RocksDB.

Things we've tried:

Are we overflowing the stack?

No, running with unlimited stack size (ulimit -s unlimited) had no effect. Only using 2MB of 8MB stack space according to measuring address pointers in gdb.

Does the multi-threaded impl of RocksDB work?

No, using our single or multi threaded implementations have seemingly no effect.

While RocksDB is thread-safe, there are some issues with Sync and cf handles. Would wrapping the DB in a mutex fix it?

No, wrapping the DB in a mutex has no effect.

Using latest rocksdb from source, building from source

No effect.

Any hints from valgrind?

No invalid reads or writes, though some suspicious "definitely lost" blocks only in the fail scenario, which could be attributed to no cleanups occurring
We allocate 750MB of memory in this single test flow, could be relevant

Any RocksDB flags we can set that would avoid this, or give us more info?

Looking through logs, setting paranoid checks, no new useful information

What about nightly?

buzzer sounds

Any insight from running sanitizers?

RUSTFLAGS="-Z sanitizer=memory" cargo +nightly test --target x86_64-unknown-linux-gnu --features rocksdb,test-kubo orb_can_render
👍 Address sanitizer: passes, what??

Failed once due to broken CAR stream
Unexpected warning: WARN flush_to_writer{writer=SphereWriter { kind: Root { force_full_render: false }, paths: SpherePaths { root: "/tmp/.tmpRjestr" }, base: Once(Uninit), mount: Once(Uninit), private: Once(Uninit) }}: Content write failed: No such file or directory (os error 2)

Thread sanitizer: Fails earlier in the test flow, though test runner has issues with it https://github.com/rust-lang/rust/issues/39608
Memory sanitizer: does not work with test runner https://github.com/rust-lang/rust/issues/39610
Leak sanitizer: segfaults much earlier in the test flow: Thread 4 "tokio-runtime-w" received signal SIGSEGV, Segmentation fault. when setting a key and serializing a multihash (libp2p/ns)

Stack trace of the offending stack:
(cpp) rocksdb::TableCache::Get
..
BlockStore::get_block
..
OnceCell::get_or_try_init/closure
Sphere::to_memo
OnceCell::get_or_try_init/closure
Sphere::to_body
Sphere:get_content
Sphere::derive_mutation
Sphere::hydrate_with_cid
Sphere::hydrate
Sphere::hydrate_timeslice
Sphere::rebase
sync::fetch_remote_changes

Also reproducible on arm64 Macos


Undo after run plugin

Once the editor is updated with the plugin result, the undo stack is emptied. Change this so it adds the change to the undo stack and the user is able to undo the process if they want.

This is managed in the Monaco editor in the ITextModel by calling pushEditOperations.
Need to identify if the Angular wrapper used in Swish (@ng-util/monaco-editor) exposes this to us, and if needed find a way to expose it.


⚠ Faculté génie website (prod) has degraded performance

In b2782df, Faculté génie website (prod) (https://www.usherbrooke.ca/genie/) experienced degraded performance:

HTTP code: 200
Response time: 3682 ms

Resolved: Faculté génie website (prod) performance has improved in e5c026d.


When I selected default layout,how can I  set the layoutCardOffset

I want to set the distance of each card item,plz help me~~

⚠ You did not follow the contributing guidelines!
As stated in these:

You need to fill out the issue template. This step is mandatory! Not doing so will result in your issue getting closed. Don't take this personally if this happens, and feel free to open a new issue once you've gathered all the information required by the template.

Please read the doc and take a look at prop inactiveSlideScale.


Bump helm for newer version of dry-run BOP-308

Description
This image is used by helm-controller to run helm commands.
Bump helm to get newer version of dry-run command which allows us to run dry-run with --server option.

We should document this version somewhere for users to find. I ran some charts that needed to be used differently depending on if helm v1, v2, or v3 was being used

We should document this version somewhere for users to find. I ran some charts that needed to be used differently depending on if helm v1, v2, or v3 was being used

Created https://mirantis.jira.com/browse/BOP-341 to document dry-run command and the version.


dragonboat silently ignores statemachines with missed data

Dragonboat version
v3.3.8
Expected behavior
If you start OnDiskStateMachine which reports lower index than that in a previously applied snapshot, snapshot recovery should happen / OR / panic about inconsistency should happen
Actual behavior
it does nothin
Steps to reproduce the behavior
apply some operations to on disk state machine, add new node, stop that node, erase some unsynced data from state machine (i tried all data, leaving state machine index on 0), start again, snapshot recovery is silently skipped.
It looks like the problem is on that line https://github.com/lni/dragonboat/blob/release-3.3/internal/rsm/statemachine.go#L298
which is missing in trunk. After that we just skip snapshot recovery and just set new Index, leaving state machine without some operations applied.
Also I tried just commenting those lines, but it likes it is impossible to recover from Shrunk(what does that mean?) snapshot, but recovery is still required. In any case it looks like those situations should be observed somehow.
Could you please take a look?
Thanks in advance

@antmat thanks for raising the issue.
My understanding is that you have a snapshot applied into the SM, let's say the index of the snapshot is 100, then after a restart of the replica, your SM returns a last applied index value lower than 100. You expect the snapshot to be applied again as a part of the recovery procedure, but that didn't happen. Am I correct?

I think this issue is relate to #156. You may want to have a look.

Am I correct?
Yes, that's right. It is related to #156, thanks for the hint.

Please note that what you mentioned as "erase some unsynced data from state machine" is really corrupting the SM. I agree that should such corruption exists in the SM, it should just panic to report such corruption.

In my case I'm talking about my implementation of IOnDiskStateMachine. For simplicity you can imagine a state machine that just writes indexes into the file and periodically sync. File was synced on index, say, 33, then several entries were applied and power failure occured without sync. SM reports OnDiskIndex 33 and snapshot has for example 35. In this case my sm state is not corrupted, but just incomplete. Would this implementation be correct?
Anyway, it looks like severe issue, maybe it's possible to backport it to v3? And btw what is the status of stability of master and v4 version?
Thanks in advance.

@antmat After checking #156 and your reply, I have the feeling that your described problem is actually identical to one addressed in #156, your experienced issue has been fixed in the master branch long time ago.
v4 is pretty stable, it is the version I am using in two of my projects, it is also the version used in some other projects that I am familiar with, e.g. I know a project using v4 with dozens of devs and a dedicated testing team, they have been using v4 for quite long time and there is never stability related issues reported. please try v4 if you like.
the only reason why v4 has never been released is because I have the feeling that I might be able to further simplify the APIs a little bit.

Thanks, we will try to migrate to v4, but currently we have an installation on v3.
Do you have any tips or tools to migrate? We tried to upgrade as is and it looks like internal structures are not compatible
level=PANIC source=raft/dragonboat	msg=invalid UUID (got 8 bytes)

The error you saw is related to the NodeHostID change. Are you using the gossip feature? Back in release-3.3, NodeHostID values required by the gossip feature are just random 8 bytes integers. It is changed to UUID in v4.
I don't have a dedicated tool to convert and migrate the data at this stage as that integer NodeHostID is pretty short lived and it is the only breaking data change, the good news is almost all code required to do the conversion is available in the repo -
so the goal is to convert all those existing integer NodeHostID values to UUID values as defined in internal/id/nhid.go in the master branch.
you first need to convert the NODEHOST.ID file in your data folder. It is a so called "flag file" (see internal/fileutil/utils.go for the CreateFlagFile and the GetFlagFileContent functions) with the NodeHostID value stored (see internal/server/environment.go for how NODEHOST.ID is stored and loaded). You should be able to easily manipulate such NODEHOST.ID files using functions available in those two .go files.
then you need to convert all those NodeHostID values stored in your snapshots. They are actually in the metadata part of your snapshots, which are also stored as so called "flag files". you basically need to load the existing snapshot from disk, specify how you want to change the involved raft membership info and store the converted snapshot back to overwrite your existing ones. tools/import.go is actually doing something very very similar - please have a look at the godoc of the ImportSnapshot function.
Please feel free to post your questions here, will be more than happy to help. Good Luck.

Are you using the gossip feature?

No, we are not using it, so all membership info is stored as hostnames, however it seems, that nodeid is generated despite it's usage. It looks like in this case is it enough to just reassign new nodeid.
Thanks for your help, I'll try.

I have not checked code yet, but I guess you can just delete that NODEHOST.ID file to see whether that will help.


Support multiple response header values in raw web actions

This change allows multiple response header values to be set in raw
web actions by using a JSON array as the header value. For example:
function main() {
  return {
    headers: {
      "Set-Cookie": ["a=b", "c=d"]
    },
    code: 200
  }
}

I have signed the Apache CLA as "Benjamin M. Browning".

Great contribution, thank you :+1:

Thanks for the pointers! I pushed another commit addressing the issues raised so far. Normally I'd squash/rebase and do a force push, but wasn't sure how the project preferred to handle that during code reviews and before merging.

Thanks for the contribution and for not squashing the revision! Indeed it helps with reviews not to constantly squash.
Usually the committers can rebase, squash and update the commit message as the PR is merged.

PG1 1908 🔵

@bbrowning congratulations to your first contribution and welcome to the OpenWhisk gang 🎉


feature: Added iOS to integration tests

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
Basic integration testing for iOS
What is the current behavior? (You can also link to an open issue here)
Not implemented yet
What is the new behavior (if this is a feature change)?
Implemented
What might this PR break?
Nothing
Please check if the PR fulfills these requirements

[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

Other information:
No provisioning configured, so not testable on a real device as is. Works correctly in a simulator though.

Looks good to me, thanks 👍


fix: modify config.yml

closing this in favor of this: https://github.com/spacelift-solutions/demo/pull/37


[All Hosts] (debugging) add link to debugging article

Fixes https://github.com/OfficeDev/office-js-docs-pr/issues/3024

Docs Build status updates of commit 11ec81d:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/testing/debug-add-ins-using-devtools-edge-legacy.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Script to create debug nuget

Debug script to build local reactive domain nuget.
Note: The actual nugets are the resulting .nupkg files that will be in the tools folder. The version number is randomly generated. Once copying locally , the user must add the containing folder to the list of package sources in Visual Studio:
https://docs.microsoft.com/en-us/nuget/consume-packages/install-use-packages-visual-studio#package-sources

Moving this script to the package-and-deployment repo


🐛 Who’s in Europe

Cc: @JeanThirion @adamcogan

Hi Team
Describe the Bug

Feedback from @JeanThirion

Investigate and fix the error I had searching for “Who’s in Europe” (see above)
What happened?

Tasks

[x] Investigate
[ ] Fix

Screenshots

Figure: Got Error when asking Who’s in Europe
Definition of Done

[ ] All tasks are updated and closed
[ ] No high priority defects/bugs are on that user story
[ ] Code reviewed by one other team member (e.g. Checked by Bill)
[ ] All acceptance criteria have been met
[ ] Deployed to Dev
[ ] (If implemented) Pass automated UI testing
[ ] (If not implemented above) Do a manual test on all supported commands (at least one for each LUIS intent) against Dev
[ ] Complex code is documented (removing technical debt)
[ ] Deployed to Production
[ ] Add description about the changes for the PBI
[ ] Tick off the definition of done tasks in the PBI
[ ] (Optional) A Done Video, particularly for PBI with multiple stages

Thanks!

This error is no longer occuring. Closing the issue now.


Index JSON data available on Azure SQL?

The page says "Applies to: SQL Server 2016 (13.x) and later"  - does this mean that the functionality is not available on Azure SQL?
If so, is there a roadmap to make it avaiable?

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

ID: 647be77d-8f01-6d16-7c7f-9a539199abc1
Version Independent ID: 0ea0cc4f-b66e-e1ad-5a08-01a8b50aab41
Content: Index JSON data - SQL Server
Content Source: docs/relational-databases/json/index-json-data.md
Product: sql
Technology: ****
GitHub Login: @jovanpop-msft
Microsoft Alias: jovanpop

It is applicable to Azure SQL DB. This is an error in the docs.
@MikeRayMSFT why the file /MicrosoftDocs/sql-docs/live/docs/includes/applies-to-version/sqlserver2016-asdb.md contains just:
  [!INCLUDE [Applies to](../../includes/applies-md.md)] [!INCLUDE [SQL Server 2016](_ss2016.md)] and later

and doesn't contain the text that is mentioning Azure SQL Database? The error is in included text.

This has been fixed. Thanks.


Random generation of molecular crystals

@scottfredericks , I suggest you start with the code which we had for atomic crystals.
1, copy structure.py to a new file (structure_molecule.py?)
2, for a given molecule (like H2O), we still randomly pick a space group and wyckoff site, and then align the molecules with some random numbers.
3, In the structure class, we now have
numIons, space group, lattice, coordinates, wyckoffs
it would be good to add some parameters which represents the necessary information to reproduce the structure, like the followings,
molecular xyz
the alignment of molecules and random numbers.
I will come up with more suggestions later

@qzhu2017
Hopefully most of it should be simple. However, I'm not sure how the lattice's volume factor should change based on the molecule being used, or how we should check inter-molecular distances. Perhaps we could use some kind of minimum sphere or ellipsoid fitting to determine an approximate volume for the molecule? Using a spherical model would be easy to check, and would not require any change in the algorithm. However, calculating a sphere for the molecule may not be straightforward, since the center of mass of the molecule is not necessarily the geometric center of the binding sphere.
For an ellipsoid or other shape (perhaps something like an atomic orbital expansion?) I am not sure if an efficient algorithm exists for checking overlap (especially if we want to consider periodic boundary conditions). And again, calculating the binding surface could be difficult. But, it could prove slightly more accurate, and hopefully faster than checking each individual atomic distance.
Of course, it would be simplest to just check inter-atomic distances directly. However, I am not sure if the minimum distances need to be altered, since molecular bonds might be stronger than crystallographic ones. I think the computation cost goes up by a factor of n^2.
Please let me know what you think is best. In the mean time, I will work on getting the rest of the code finished.

For the estimation of volume, you can use the following scheme,
1, reorient the molecule based on the eigenvalue and eigenvector of the molecular inertia tensor;
2, calculate the volume by treating it as an orthogonal box with a certain thickness say 2 \AA, so the volume is (x_max-x_min+2)(y_max-y_min+2)(z_max-z_min+2)
For the minimum distance, we only check the intermolecular distance (i.e., distance between two atoms belonging to two molecules). We should set the minimum distance larger than the convenient bond length.
For the easy case, I suggest you use the example of H2O and CH4. Please upload your code asap. I have some matlab codes dealing with the volume estimation and distance check and I can easily modify your code to make it work.

@qzhu2017
I have implemented the volume and box-finding methods. The relevant code is in molecular_crystal.py. However, looking at check_compatibility, I realized that we will need to call orientation_in_wyckoff_position many times per crystal, which would slow down the code too much.
To avoid this problem, I can build a class for storing orientations with 0, 1, or 2 contraint vectors. orientation_in_wyckoff_position can return these objects, so that we only need to call it once per Wyckoff position/molecule (while checking the compatibility). Then, once a list of orientations is obtained for each Wyckoff position/molecule pair, we can easily choose a random orientation without recalling the function.
I think I can code this today. Once done, I will put the orientation class in matrix.py.

@qzhu2017
I think most of the functionality is programmed. However, I still need to edit merge_coordinate to check for valid orientations, and to make sure that the final coordinates are transformed correctly. Right now, there is an error in the process, so the molecules tend to overlap. I will continue work tomorrow.

@qzhu2017
I am now obtaining reasonable-looking results. In order to obtain results in a reasonable timeframe, I increased the default volume factor to 100. By default, molecular_crystal.py will attempt to generate water ice 1h: spacegroup P6_3/mmc (194), with 12 H2O molecules per unit cell. After a few attempts, some of the generated crystals have a hexagonal motif similar to that of actual ice. I am not sure if the structure is actually correct though, so please take a look.
I think that once we have a better distance checking method, we can reduce the volume factor without slowing down so much.
The main difficulties were:

Storing and accessing valid molecular orientations for each Wyckoff position/molecule
checking that merged wyckoff positions also have valid molecular orientations
generating the correct relative coordinates for the molecules once the centers are known
finding and using the correct generating (x,y,z) point for merged Wyckoff positions, so that the corresponding rotations could be generated

@scottfredericks The code runs very fast. It is a bit strange that we need to use such big volume factor. I am going to check the distance calculation function soon. Could you please check another issue in the mean time? If you check the symmetry of the generated structures, sometimes, it ends up with the structure with lower symmetry (P2_1/m), instead of the desired one (Cmc2_1). Maybe, there exist some bugs in orientation.
qiangzhu@u-10-240-0-36:~/Desktop/github/crystallography$ grep space_group_name out/*.cif
out/1.cif:_symmetry_space_group_name_H-M   P2_1/m
out/10.cif:_symmetry_space_group_name_H-M   Cmc2_1
out/2.cif:_symmetry_space_group_name_H-M   Cmc2_1
out/3.cif:_symmetry_space_group_name_H-M   Cmc2_1
out/4.cif:_symmetry_space_group_name_H-M   P2_1/m
out/5.cif:_symmetry_space_group_name_H-M   P2_1/m
out/6.cif:_symmetry_space_group_name_H-M   P2_1/m
out/7.cif:_symmetry_space_group_name_H-M   P2_1/m
out/8.cif:_symmetry_space_group_name_H-M   Cmc2_1
out/9.cif:_symmetry_space_group_name_H-M   P2_1/m

@scottfredericks  I have fixed some issue to avoid the using of very large volume factor.
can you also add an option of outputting the information about the used wyckoff sites. In molecular_crystal.py, add an option of verbosity, by default, it output only
Space group requested:  63 generated 63 vol:  169.345755
If verbosity is 1, output the wyckoff information about the multiplicity, letter, and the position of molecular center
like
4a (0.000, 0.193, 0.116)

@scottfredericks Very good design now. I just noticed another error. It failed to generate space group 64 for some reason.
qiangzhu@u-10-240-0-36:~/Desktop/github/crystallography$ python molecular_crystal.py -v 1 -a 10 -f 0.8 -s 63 -n 2
/Users/qiangzhu/miniconda3/lib/python3.6/site-packages/pymatgen/io/cif.py:44: UserWarning: Please install optional dependency pybtex if youwant to extract references from CIF files.
  warnings.warn("Please install optional dependency pybtex if you"
Space group requested:  63 generated 36 vol:<PHONE_NUMBER>
4c [0.         0.29704853 0.25      ]
Space group requested:  63 generated 36 vol:<PHONE_NUMBER>
4c [0.         0.69965202 0.25      ]
Space group requested:  63 generated 36 vol:<PHONE_NUMBER>
4c [0.        0.1669475 0.25     ]
Space group requested:  63 generated 36 vol:<PHONE_NUMBER>
4c [0.         0.28650741 0.25      ]
Space group requested:  63 generated 36 vol:<PHONE_NUMBER>
4c [0.         0.88176841 0.25      ]
Space group requested:  63 generated 36 vol:<PHONE_NUMBER>
4c [0.         0.30762532 0.25      ]
Space group requested:  63 generated 36 vol:<PHONE_NUMBER>
4c [0.         0.18704583 0.25      ]
Space group requested:  63 generated 36 vol:<PHONE_NUMBER>
4c [0.         0.25537717 0.25      ]
Space group requested:  63 generated 36 vol:<PHONE_NUMBER>
4c [0.         0.65369338 0.25      ]
Space group requested:  63 generated 36 vol:<PHONE_NUMBER>
4c [0.         0.79558808 0.25      ]

@qzhu2017
It was a careless mistake: I accidentally used mol to generate the final coordinates, instead of mo (which has the orientation already applied). Changing this fixes the issue for spacegroup 63.
Also, with the changes to distance checking, you'll need a volume factor larger than 1, since 1 corresponds to near-perfect orthogonal packing of the molecules. A factor of 4 worked fine for me, with the other parameters left unchanged.

@qzhu2017
I am now able to generate molecular 2d crystals for all layer groups, within a reasonable time (less than a minute for each). I have also used spglib to verify that the output crystals have the correct spacegroup. The script 2dmolecular.py will perform this test automatically for water. Next, I will try H2, NH3, CH4, and C60 (you can just run 'python 2dmolecular.py -e H2', and some other command line options which are used in molecular_crystal.py).

@qzhu2017
There seems to be an issue with generating lattices for 3d molecular crystals. With a volume factor of 1, lattice generation fails for many space groups. With a larger volume factor this is not the case, which leads me to believe it is due to an improper treatment of the minimum vector. I will look into this.

@scottfredericks
it is good to know why it fails. On the other hand, the volume factor for molecular crystals can be larger, I would say some number between 1-2.5 should be okay.

@qzhu2017
I've changed the calculation of minvector. Previously, it used twice the molecular radius. Now, it uses the minimum side length of the binding box. I am still running test_all with this change, but so far it has been able to generate lattices correctly.

@scottfredericks
Please close this issue after you are done with this check.

@qzhu2017
Adjusting the default volume factor to 2.5 for test_molecular, all spacegroups now generate. Some still take more than 60 seconds, though:
202, 209, 210, 216, 219, 221, 225, 226, 227, 228, 229, 230
These all have fairly high symmetry though, so maybe it's not really an issue.


Digital Ocean API V2 support

Must be via https://github.com/digitalocean/droplet_kit

Good idea !

Digital Ocean is sunsetting API v1 on November 1st 2015.
I have no competences in Ruby to port the code with the new API (https://developers.digitalocean.com/documentation/v2), but if someone has, feel free !

I'm started this work with refactoring into modular application structure. Tests for moved V1 works well, now i'm stuck on V2 and faraday gem version between old one and new one API gems. motdotla/digitalocean#47

For now problem solved. Fighting with tests :)

Nice !!

Released now.

Good job ! :) I will test it soon.


Problem: Hax service stuck while stopping rconfc

While shutting down cluster, Hax service stops after IOS & CONFD
stops. In this case when Hax tried to stop rconfc, it stuck in
confd election, since confds already shutdown it should not go
for election and mark as rconfc_failed instead.
Solution:
Currently rconfc only used to fetch filesystem stats, and it only
fetch stats if all Motr services are running. fs-stats-updater thread
start rconfc only when all Motr services are running. So whenever it
it finds that all services are not running it stops rconfc and again
start when all services up and running again.

What if a shutdown happens during fs-status fetch when rconfc is not stopped yet?..


proposal: crypto/x509: add API to create RFC-compliant CRLs

What version of Go are you using (go version)?

$ 1.13.4

This is a bit of a redux of #13931, but while the fix there changed the serialization on the wire of the default value, it didn't address the underlying problem, which is that the X.509 RFC pretty clearly states (https://tools.ietf.org/html/rfc5280#section-5):
   Conforming CAs are not required to issue CRLs if other revocation or
   certificate status mechanisms are provided.  When CRLs are issued,
   the CRLs MUST be version 2 CRLs, include the date by which the next
   CRL will be issued in the nextUpdate field (Section <IP_ADDRESS>), include
   the CRL number extension (Section 5.2.3), and include the authority
   key identifier extension (Section 5.2.1).

This means that (as specified a bit after that) although any conforming client should be able to parse and understand a CRL created with this function, technically every CRL created via this function is non-conforming, because even if the root is self-signed it's still a CA. There's an argument to be made that a third party could be signing and creating the CRLs, but the behavior there isn't a distinction that is made clear in the RFC and it seems reasonable to assume that the intention is for it to behave the same way.
This would probably require a new function to create v2 CRLs so that the parameters required for the extensions can be provided.
Obviously clients are free to then ignore this extra information, but that's up to the client; it seems pretty clear that anything acting as a CA is required to issue v2 CRLs.

CC @FiloSottile

Just edit the title as I suggested and include the API name from the CL. The CL refs this issue by number.
No action can be taken on the CL until a proposal is reviewed and accepted here.

I have a use case where I want to be able to specify SignatureAlgorithm when creating a CRL (currently it uses the default (0): https://github.com/golang/go/blob/master/src/crypto/x509/x509.go#L2222). This could be included in the CRLTemplate proposed by<EMAIL_ADDRESS>Instead of CRLTemplate, this struct could be named CertificateRevocationList; so that:

The function can be named CreateCertificateRevocationList to avoid collision with CreateCRL.
Eventually, there could be a ParseCertificateRevocationList to be consistent with ParseCertificate and ParseCertificateRequest.

I like the symmetry with Certificate and CertificateRequest and the option to add a parser later. CertificateRevocationList is a mouthful though. Just like we shortened CSR to CertificateRequest, how about shortening CRL to RevocationList?
Here's a suggested API. Note the following changes:

CreateRevocationList and RevocationList names
order of arguments to CreateRevocationList matching CreateCertificate
order of fields resembling Certificate and CertificateRequest
*big.Int for Number because it can be up to 20 bytes
ExtraExtensions instead of Extensions like Certificate and CertificateRequest (which I'm not a fan of, but we might as well keep it all consistent)

func CreateRevocationList(rand io.Reader, template *RevocationList,
	issuer *x509.Certificate, priv crypto.Signer) ([]byte, error)

type RevocationList struct {
	Version            int
	SignatureAlgorithm SignatureAlgorithm

	RevokedCertificates []pkix.RevokedCertificate

	Number          *big.Int
	ThisUpdate      time.Time
	NextUpdate      time.Time
	ExtraExtensions []pkix.Extension
}

Adding to proposal minutes for wider visibility. Seems OK.

@FiloSottile these changes look good to me.
One thing I'd suggest against is including Version, as far as I know there is no real call for creating anything other than X509v2 CRLs. I believe the field in Certificate only exists for parsing, as CreateCertificate only creates X509v3 certs. I think it's probably best to leave it out and then if a parsing method is added down the line to include it then in order to avoid confusion.

One thing I'd suggest against is including Version, as far as I know there is no real call for creating anything other than X509v2 CRLs. I believe the field in Certificate only exists for parsing, as CreateCertificate only creates X509v3 certs. I think it's probably best to leave it out and then if a parsing method is added down the line to include it then in order to avoid confusion.

Agreed, we can introduce it along with a parsing function if needed.

The proposed struct looks good to me too!

Based on the discussion above, this seems like a likely accept.

Assuming there are no other major comments I'll look to update my existing CL sometime next week to reflect the proposal by @FiloSottile.

No change in consensus, so accepted.


feature - improve linking to fragments

First, this is a really cool project. I've been having a look at the outputs created and they usually look great.
One slightly hidden current feature I really like is that we can hyperlink paragraphs, equations, figures and sections. This can help streamline discussions, referee reports, and I can even imagine referring to such a URL inside a paper.  I guess this was possibly with PDFs too, but much less appealing.
E.g., for this paper, https://ar5iv.labs.arxiv.org/html/1906.08684, I found I can add

I can link sections using the fragment id S{section number} for a section,S{section number}.SS{subsection number}  for a subsection etc, e.g. https://ar5iv.labs.arxiv.org/html/1906.08684#S1 and https://ar5iv.labs.arxiv.org/html/1906.08684#S2.SS2.SSS2

Paragraphs by adding p{paragraph number} to the complete section information, e.g. https://ar5iv.labs.arxiv.org/html/1906.08684#S2.SS2.SSS2.p2

Equations by adding the E{equation number} to the section, e.g., https://ar5iv.labs.arxiv.org/html/1906.08684#S2.E7

Abstract by id1.id1, e.g. https://ar5iv.labs.arxiv.org/html/1906.08684#id1.id1

Bibliography by bib, e.g. https://ar5iv.labs.arxiv.org/html/1906.08684#bib and particular items by `bib.bib{reference number}, e.g. https://ar5iv.labs.arxiv.org/html/1906.08684#bib.bib4

Footnotes by footnote{number} e.g.  https://ar5iv.labs.arxiv.org/html/1906.08684#footnote2

It appears that we can also link to parts of paragraphs, e.g. https://ar5iv.labs.arxiv.org/html/1906.08684#S2.SS2.SSS1.p1.3 gives the third part of that paragraph, and even inline math, e.g. https://ar5iv.labs.arxiv.org/html/1906.08684#S3.SS3.SSS2.p1.2.m1.2.2 but the fragment tags get quite complicated

Anyway, the feature request is that I wonder if there was a simply way to facilitate easier creation and sharing of these kinds of links. For example, if every equation number, section/subsection ... title, perhaps even the first word of each paragraph, etc, was a hyperlink to its tag, people could easily copy the links.

Very reasonable upgrade request.
In terms of user experience -- do you personally enjoy the github link-on-hover feature in github's markdown previews @andrewfowlie ? Say you go to the README here and hover over the ar5iv heading, you should see:

That little "link symbol emoji" 🔗 can then be clicked and the link to the fragment in question gets copied to your clipboard for reuse in any other application.
I'm personally fond of it, and I think it would be quite easy to provide another small piece of javascript that enables it.
A good place to note that before we set onto the path of building a "high quality viewer application" for arXiv, there are some preliminaries that need to be taken care of with setting up a proper javascript workspace -- that is likely the reasonable bit to do next, before I pile together too many separate independent mini-scripts.

Yes! That system works well and is a perfect example of the kind of problem and solution I had in mind.


Export the Unifi-defined name field as hostname, and do not export RSSI and Noise metrics for wired clients

The current hostname exported is the client-provided name, which is often much less descriptive (or nonexistence) compared to what I've set in the Unifi interface. This patch exports the Unifi hostname if provided, otherwise it uses the client provided hostname.
It also stops exporting metrics for wired clients with no AP MAC set.

This will fail to build until https://github.com/mdlayher/unifi/pull/1 is accepted.

You can ignore the Travis failure.  LGTM.
Now that we can differentiate wired devices, I have a couple more ideas.  Will open an issue if you'd like to help out some more, or I'll get to it eventually. :)

I broke the build - stationcollector_test has a legitimately failing test now that I switched to checking IsWired. I'll send another PR.

Oh cool.  I thought it was just coveralls acting up.  I appreciate it!


Add Cardboard to Compositors

This looks like a neat project! Thank you for the submission. I'd like to wait a bit before adding it to awesome-wayland, because Cardboard self brands itself as:

currently a Work In Progress, not suitable for day to day use

I'm trying to keep this list focused on Wayland things people can use well today! I'm definitely open to keeping this PR open for a while though and merging in once the Cardboard project gets further along.

Looks like Cardboard has progressed to "ready for day to day use" so I'm going to merge this in now!


Shin's Shop: neuro_cb: cmd: 3

Caused by dialogue sequence:

Okay. Give me the deck. ...
...

Done in 1981ebf911e2fe26bc83768532e239288d250842 commit.


amplify configure missing ca-central-1

Before opening, please confirm:

[X] I have installed the latest version of the Amplify CLI (see above), and confirmed that the issue still persists.
[X] I have searched for duplicate or closed issues.
[X] I have read the guide for submitting bug reports.
[X] I have done my best to include a minimal, self-contained set of instructions for consistently reproducing the issue.
[X] I have removed any sensitive information from my code snippets and submission.

How did you install the Amplify CLI?
npm
If applicable, what version of Node.js are you using?
16.14
Amplify CLI Version
1.8.5
What operating system are you using?
Windows
Did you make any manual changes to the cloud resources managed by Amplify? Please describe the changes made.
No manual changes made
Amplify Categories
Not applicable
Amplify Commands
configure
Describe the bug
Missing ca-central-1 option when selecting "Specify the AWS Region"
Expected behavior
ca-central-1 should be an option
Reproduction steps
aws configure
after logging into aws console
press enter to continue
GraphQL schema(s)

# Put schemas below this line

Log output

# Put your logs below this line

Additional information
No response

Hi, it looks like I see the options on running amplify configure

If you're running version 1.8.5 of the CLI, that could be why. Can you try upgrading to a newer version. Today, the latest version is 7.6.19, which is significantly different from 1.8.5.


Error reading namespaced RSS tags

I'm having trouble parsing an RSS feed for a podcast that includes namespaced tags (<itunes:summary>, <itunes:image>, etc.) because Twig thinks the colon is the start of a code block.
Example:

Description: {{ node.itunes:summary }}
.
=> Unexpected token "punctuation" of value ":" ("end of statement block" expected).

What's the best way around that? This is a wonderful plugin, by the way. Great work!

You'll have to use the array notation for that - have a look at the example code
So you'd use 'node['itunes:summary']'


Add some more strings

Adds some more strings to the blacklist

Adds some more strings to the blacklist

As you can tell, the current file has Discord related strings blacklisted only, hence the name "Token Logger Protection Provider".

Ah yes indeed it is only Token Logging related strings my bad, however the only two Minecraft specific strings in my commit are
.feather/accounts.json
essential/microsoft_accounts.json

so I believe the other three can stay.
Will add the Minecraft related strings to its own class (MinecraftProtectionProvider)

Will add the Minecraft

Sounds good!

Looks fine!

🤝


The problem of SDCFlows (and how to fix it)

What would you like to see added in this software?
Instead of directly starting from coding, I will explain what I currently think has happened.
I have come to the realization that I've unintendedly been over-complicating the implementation of SDCFlows' interface.
Let me first thank you for your patience on this front, @mgxd, @effigies, @eilidhmacnicol and others. It took too much time for me to realize.
Why first share rather than code away?
I'm under the impression that SDCFlows needs better documentation and better communication from its architect (which would be me, in this case). The notebook describing the problem was a good move, but a second notebook describing correction has been missing and has contributed to the failure to set clear goals (and very likely would have made me realize of the following). Since I don't have time to write that notebook right now, I will explain what I just realized.
Daunted by the size of the forest, I missed that I should've cared for the first tree only
Since October 2022 I've been banging my head against the same wall, trying to interpolate a B-Spline field on a grid that is not aligned with the target grid in a manageable manner (memory, time). This problem became apparent with Mathias' data because there's a rotation of the image w.r.t. the fieldmap. This seemed essential as well to implement the vision of being able to correct each volume of the EPI series with the exact fieldmap realization (when the head moves, the deformation field changes because the phase-encoding axis doesn't while inhomogeneity of the field does with the head).
 Instead:
Preconditions:

We have a "target EPI" - a BOLD or DWI dataset, without having gone through HMC or SDC.
We have also the list of HMC matrices
We have estimated the fieldmap's coefficients with SDCFlows
We have the "fieldmap-to-EPI" affine transform (or the reverse, and we invert) that aligns the target EPI and the fieldmap's "magnitude" (phasediff et al.) or "reference" (pepolar, syn).

The algorithm:

Project the voxel positions of the target EPI (that you want to correct) into the fieldmap space using the "fieldmap-to-EPI" transform (that is, if you use antsApplyTransform, then the fieldmap is the moving and EPI is the reference). This is nitransforms.linear.Affine.from_file("fieldmap-to-EPI.h5").map((x, y, z)), where x, y, z are the coordinates in mm of each voxel of the EPI grid. This can be done easily at once just by rotating the affine of the EPI through the fieldmap-to-EPI transform (meaning, we just get another NIfTI file of the EPI that a decent viewer with good implementation of the orientation will render aligned with the fieldmap, despite them being on different grids). The latter is also doable with a nitransforms one-liner.
We interpolate (only this once) the field there (and in this case our B-Spline coefficients are aligned with the grid, so it's fast), and we have the fieldmap in Hz interpolated on the EPI grid. If we just replace the affine with the original affine of the EPI, our fieldmap is now on the target EPI without any more interpolations!
~At this point, it would be convenient to convert the fieldmap from Hz into a voxel shift map (if I'm not wrong, this is just accounting for voxel zoom in the PE direction).~ (EDIT: see comment below)
Say you want to now resample your 60 volumes of BOLD with simultaneous correction of SD. For each volume you:
4.1. Project all the voxels of the reference onto the volume
4.2. Convert all those coordinates in mm into voxel coordinates
4.3. Instead of directly interpolating the BOLD value on the projected location, now you pull up your voxel shift map, and at that voxel coordinate, you get the displacement of that voxel to account for distortion. You update the coordinates of each voxel with this amount (minus/plus) and now yes, interpolate because you'll be off-grid almost always.
4.4. Now you have a data array on the same grid of the reference, and you can use its affine to build a new NIfTI where head motion AND susceptibility distortions are not present.

I hope the algorithm is clearly described. If not, it would be critical that you poked at me until we're all on the same page.
The good news is the above is trivial to implement with our current kit. And no one is doing this because of the fragmentation of code. I think this would be as trivial to implement for FSL as it is for us (in their case only with TOPUP, as opposed to us, where it would work for all estimation approaches). But I believe AFNI and others would not be as close.
Do you have any interest in helping implement the feature?
Yes
Additional information / screenshots
No response

Re: step 3
The voxel zoom is relevant to convert to voxels to mm (reasonably). To convert from Hz (vox/s) to voxels we need the readout time (s). And we actually have it already there, exactly in the way I was mentioning above:
https://github.com/nipreps/sdcflows/blob/891e81ba7b4c9f0faf07156858c006828e4c314e/sdcflows/transform.py#L197-L200

Okay, I will have to hold the reins a little because there's no way around reconstructing the field on a rotated grid. However, I think it would be cheap to start with an approximation and then consider if we want the very precise thing. See the above draft PR (and then new approx argument to the B0FieldTransform.fit().

Okay, just doing some math. I hope this is the same as what you've said, but let's be stupidly clear.
Let's define the following spaces:

Bold reference space: $\mathbf{B}$
Bold volume space: $\mathbf{V}$
Fieldmap space: $\mathbf{F}$

Each has an accompanying affine $A_S$ that translates an index $\vec i_S$ to a world coordinate $\vec c_S$. So:
$\vec c_B = A_B \vec i_B$
$\vec c_V = A_V \vec i_V$
$\vec c_F = A_F \vec i_F$
A transform $\mathcal{T}_{XY}$ is the affine transform from coordinates in space $\mathbf{X}$ to $\mathbf{Y}$. Assume we have invertible transforms:
$\mathcal T_{BF} : \mathbf{B} \rightarrow \mathbf{F}$
$\mathcal T_{VB} : \mathbf{V} \rightarrow \mathbf{B}$
$\vec c_F = \mathcal T_{BF} \vec c_B$
$\vec c_B = \mathcal T_{VB} \vec c_V$
and we will use the inverses when going in the opposite direction.
There is a fieldmap function $\Phi(\vec c_F)$ interpolated over a discrete field $\phi(\vec i_F)$ in Hz.
Finally, there is a constant readout field $\vec r(\vec c i_V)$ over $\mathbf V$ equal to the total readout time divided by the voxel size, constrained to the phase-encoding direction. This is most usefully recast as $\vec R(\vec c_v) = A_Vr(\vec i_V)$.
The problem is to correcting a bold volume in $\mathbf{V}$ simultaneously accounting for motion and distortion to resample in $\mathbf{B}$.
Given $\vec i_B$:
The field at that location is $\Phi(\mathcal{T}_{BF}A_B\vec i_B)$.
The uncorrected location in $\mathbf V$ is $\mathcal T_{VB}^{-1}A_B\vec i_B$. Recalling that \vec R() is a constant vector field:
$\vec c_V = \mathcal T_{VB}^{-1}A_B\vec i_B + \Phi(\mathcal{T}_{BF}A_B\vec i_B)\cdot \vec R(\cdot)$
Therefore, the voxel index in $\mathbf{V}$ from which we draw values to resample to $\vec i_B$ is:
$\vec i_V = A_V^{-1} \cdot (\mathcal T_{VB}^{-1}A_B\vec i_B + \Phi(\mathcal{T}_{BF}A_B\vec i_B)\cdot \vec R(\cdot))$

Okay, rereading what you wrote, I think I disagree with steps 3 and 4. I think step three is to convert to a coordinate shift map:
readout_vec = np.zeros((3,), 'int8')
readout_vec["ijk".index(pedir[0])] = 1
if pedir[1:] == "-":
    readout_vec *= -1

volspace_readout = trt * (vol.affine @ readout_vec)
refspace_readout = mc_affine @ volspace_readout

coordinate_shift = refspace_readout * fieldmap

Now for step four, I simply interpolate from a shifted world location:
target_coord = boldref.affine @ target_index
volume_coord = inv(mc_affine) @ (target_coord + coordinate_shift * field[target_index])

(This is all pseudocode; we need to handle the 1s tacked onto each index/coordinate.)

Yes, let's simplify further by dividing the problem into two halves.
Let's address first the most practical part of the problem: correcting the BOLD volume in $\mathbf{V}$, given a list of $\mathcal T_{BV}^t$ (where $t$ is the time index in $\mathbf{V}$) --estimated with, e.g., mcflirt--, the reference of head-motion estimation $\mathbf{B}$, and a voxel-shift map defined on $\mathbf{B}$, $\Theta_\mathbf{B}(\vec i_B)$.
$\Theta_\mathbf{B}(\vec i_B)$ is a mapping of the field in voxels, such that $\Theta_\mathbf{B}((20, 10, 6)) = 1.5$ is a displacement of 1.5 along the PE direction at that particular $(20, 10, 6)$ voxel. If the PE direction is j or j- (sign should be encoded in the mapping already), then that voxel's coordinates in the distorted $\mathbf{V}$ are $(20, 11.5, 6)$. The intent is to also clear out the affines to convert to physical space as often as possible.
Without distortion ($\Theta_\mathbf{B} = 0$), then the resampling problem is to find where every $\vec i_\mathbf{B}$ in the BOLD reference lands in the BOLD volume's grid ($\vec i_\mathbf{V}^t$). Then an interpolator will get those $\vec i_\mathbf{V}^t$ and calculate those off-grid values, and the interpolated values are then pulled back into a data grid on the BOLD reference space, so that the head-motion corrected $\vec i_\mathbf{B}^t$ are assigned these interpolated values:
$\vec i_\mathbf{V}^t = A_\mathbf{V}^{-1} \mathcal T_{BV}^t A_\mathbf{B} \vec i_\mathbf{V}$
Please note that the VOX/RAS conversions are necessary because $\mathcal T_{BV}^t$ are estimated in physical (RAS) coordinates.
Now, let's have a nonzero fieldmap $\Theta_\mathbf{B} = (0, \Delta \vec i, 0)$. My realization of today is that we just need to do:
$\vec i_\mathbf{V}^t = A_\mathbf{V}^{-1} \mathcal T_{BV}^t A_\mathbf{B} \vec i_\mathbf{B} + \Delta \vec i$,
which effectively is equivalent to applying $\mathcal T_{VB}^t$ to $\Theta_\mathbf{B}$ (i.e. moving the fieldmap with the head, please note how BV have been inverted w.r.t. above), and then calculating the voxel shift there. Suppose we work with the field in RAS shifts/displacements. In that case, everything gets even more complicated because you also need to project the transformed displacements and discard the components where there's no deformation.
Please also note how the "traditional" correction works, where you first use topup/applytopup to create a corrected version of the dataset, and then apply head motion correction:
$\vec i_\mathbf{V}^t = A_\mathbf{V}^{-1} \mathcal T_{BV}^t A_\mathbf{B} (\vec i_\mathbf{B} + \Delta \vec i)$
Then the second part of the problem is as follows. When we compute the fieldmap, we do not get it with reference to the target images (or references of target images). We get them in their native (fieldmap) space.
However, I think I'm going to leave it for later because you just posted something :)

Translating into nitransforms'esque:
$\vec i_\mathbf{V}^t = A_\mathbf{V}^{-1} \mathcal T_{BV}^t A_\mathbf{B} \vec i_\mathbf{V}$
has no direct API method, instead you can implement the physical to physical mapping:
$\vec c_\mathbf{V}^t = \mathcal T_{BV}^t \vec c_\mathbf{B}$
with:
vec_c_V_t = t_BV_xfm.map(vec_c_B)

However, the above ($\vec i_\mathbf{V}^t = A_\mathbf{V}^{-1} \mathcal T_{BV}^t A_\mathbf{B} \vec i_\mathbf{V}$) is internally calculated by apply (pseudocode):
volume_t_in_boldref = t_BV_xfm[t].apply(bold_volume[t])

In reality, apply will move all timepoints:
bold_in_boldref = t_BV_xfm.apply(bold_volume, reference=boldref)

Please note how pulling information through the transform in the image and mapping have I/O spaces "swapped".

(if we can agree on this first half https://github.com/nipreps/sdcflows/issues/345#issuecomment-1500411290, let's dig into the second half of the problem)

Sorry, had to run some errands, but have now wrapped my head around your comment, @oesteban. The fundamental difference between our two proposals is:
Yours: $\vec i_\mathbf{V}^t = A_\mathbf{V}^{-1} \mathcal T_{BV}^t A_\mathbf{B} (\vec i_\mathbf{B} + \Delta \vec i_B^t)$
Mine: $\vec i_\mathbf{V}^t = A_\mathbf{V}^{-1} \mathcal T_{BV}^t (A_\mathbf{B} \vec i_\mathbf{B} + \Delta \vec c_B^t)$ where $\Delta \vec c_B^t =  \Phi(\mathcal T_{BF}A_B\vec i_B)\cdot \mathcal T_{VB}^t\vec R_V$
I believe you're right that calculating $A_\mathbf{V}^{-1} \mathcal T_{BV}^t A_\mathbf{B}$ once and directly mapping indices will be more efficient, so for completeness:
$\Delta \vec i_B^t = A_B^{-1}\mathcal T_{VB}^t\vec R_V\cdot  \Phi(\mathcal T_{BF}A_B\vec i_B)$
I think we agree on the math at this point; the only thing I'm not sure we agree on is what the intermediate representation of the fieldmap should be in $\mathbf{B}$. I think it should be in Hz, since by the equation above it's the multiplication of a single orientation vector by a scalar field, while if I'm reading you correctly, you were proposing having a reference voxel displacement field that needs to be rotated according to each volume.
That feels like an empirical question of which works better, so happy to move on to the second half.

Thanks for initiating the conversation with the maths, in hindsight it's the only way that would've worked.
Unfortunately, it's a bit late for me. I'll try to write down the second half tomorrow.
(I'm on my phone now)

Okay, I decided to start writing this up into a single document with derivations that we could include in the docs, and worked myself around to your voxel displacement map:

First, let's review resampling. When resampling a particular volume $\mathbf V^t$ into the BOLD reference volume $\mathbf B$, you select a grid of indices $\vec i_B$, and then find values in $\mathbf V^t$ to fill them. So the goal is to find a mapping between indices. This is the standard derivation:
$$\begin{align}
\vec i_V^t &= A_V^{-1} \vec c_V^t \
&= A_V^{-1} \mathcal T_{BV}^t \vec c_B \
&= A_V^{-1} \mathcal T_{BV}^t A_B \vec i_B
\end{align}$$
We would like to find an additional displacement $\Delta \vec i_V^t$ that would account for distortion induced shift:
$$
\vec i_V^t = A_V^{-1} \mathcal T_{BV}^t A_B \vec i_B + \Delta \vec i_V^t
$$
Here we will make use of our readout field which indicates the direction that distortions will induce shifts, scaled by the magnitude of the inhomogeneity:
$$\Delta \vec i_V^t = \Phi_F(\mathcal{T}{BF}\mathcal T{VB}^t\vec c_V^t)\vec R_V$$
This is the fieldmap, resampled into the volume space $\mathbf V^t$, multiplied by our constant vector field.
Using our earlier calculations, we can decompose this into the fieldmap resampled into the reference space $\mathbf B$ and the readout vector:
$$\begin{align}
\Delta \vec i_V^t &= \Phi_F(\mathcal{T}{BF}\mathcal T{VB}^tA_V\vec i_V^t)\vec R_V \
&= \Phi_F(\mathcal{T}{BF}\mathcal T{VB}^tA_VA_V^{-1} \mathcal T_{BV}^t A_B \vec i_B)\vec R_V \
&= \Phi_F(\mathcal{T}_{BF} A_B \vec i_B)\vec R_V \
\end{align}$$
Hence, by resampling $\Phi_F$ once into $\mathbf B$,
$$\phi_B(\vec i_B) = \Phi_F(\mathcal{T}_{BF}A_B\vec i_B)$$
We are able to fully calculate the index-to-index mapping:
$$
\vec i_V^t = A_V^{-1} \mathcal T_{BV}^t A_B \vec i_B + \phi_B(\vec i_B)\vec R_V
$$
Somewhat surprisingly, the fieldmap does not need to be resampled into the individual $\mathbf V^t$ spaces in order to fully account for the effect of distortions, because it is a shift in indices based on the geometry of the target space $\mathbf B$.

I spent some time this morning to see if we could recast this final calculation as a traditional concatenation of transforms, but convinced myself that that would not work. We need to be able to create a voxel-shift-map in the target space, but operate on source indices.
However, I realized that this target space can be anything, not just the BOLD reference. If we want to do this correctly and consistently, we should resample the fieldmap to every target space that we do single-shot resampling to.
Here's the doc, btw: https://hackmd.io/@effigies/rethinking-resampling. It still assumes that our target space is the BOLD reference, but I'll aim to update it to be more general if we want to use this doc.


Room broadcast function realization

Hi!
First of all thank you your work on this library - maybe this is the only working socket.io client implementation up to date!
Any reason why this is not accepted yet?
Also I see that the tests doesn't pass on Travis CI - maybe we should updated to Golang 1.14.x?


Fix spelling of receiver

according to https://en.oxforddictionaries.com/definition/receiver

Thanks!


Q2 or later?

"The option to connect an Office 365 group to an existing site is not yet available and will be released during Q2 of calendar year 2018" - this article doesn't clarify whether that refers to the ability to do it via PowerShell or via the GUI - we're nearly into Q4 and the feature isn't in the GUI.

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

ID: 60de4a4c-bcb7-7bd4-5a11-2aa014d0a93b
Version Independent ID: 2a477349-3aa5-8718-17ea-661b9aa089d7
Content: Overview of the Connect to new Office 365 group feature
Content Source: docs/features/groupify/groupify-overview.md
Product: sharepoint
GitHub Login: @spdevdocs
Microsoft Alias: spdevdocs

It's documented in the "see also" section of this doc.


[staging] boehmgc: 8.2.2 -> 8.2.4

Description of changes
Update boehmgc and rebase/fix the Nix patch.
https://github.com/ivmai/bdwgc/blob/v8.2.4/ChangeLog
Compare changes on GitHub
Closes #234967, which couldn't be merged yet because the nix patch was not ported.
Ping @vcunat.
nixVersions in boehmgc.tests complained about removed nix versions, so I tested with:
$ echo '{ allowAliases = false; }' > config.nix
$ NIXPKGS_CONFIG=$PWD/config.nix nix build -f . boehmgc.tests --no-link

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[x] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-already-reviewed/2617/1311

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-already-reviewed/2617/1369

@ofborg build nix


Lock installer version

To have installer versioning we have to lock it in the workroom st2.yaml along with the st2 version.
Untested, don’t merge for now.

Tested it, everything works, the right st2installer version gets pulled. All we need to do now is keep st2installer tags organized and change st2.yaml once a new version is released. Good to merge?
cc @lakshmi-kannan @dzimine

High level this looks ok. Where's the code that looks at this hiera variable and clones the appropriate branch? I want to take a look first.

Here's the hiera reading: https://github.com/StackStorm/st2workroom/blob/59b3e9dd62dd34e823203d0af6b7aa987e91fce7/modules/profile/manifests/st2server.pp#L25
And here's the branch checkout: https://github.com/StackStorm/st2workroom/blob/59b3e9dd62dd34e823203d0af6b7aa987e91fce7/modules/profile/manifests/st2server.pp#L943

LGTM. +1. Thanks for the pointers.

Woot.


free tier database

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 88.398%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
5383

Relevant Lines:
5802

💛  - Coveralls


Group support for django-userena

It would be helpful to have group support included in userena by default.
Ideally, this should create and use Django's groups functionality of django.contrib.auth. An example attempt is posted on StackOverflow. Messaging could then happen within those user groups, not between (explicitly defined) single users as today.
Any plans on implementing this? Or reasons why this is (not going to happen or) better not done?

Please be more specific. What do you mean by "group support"? Userena is built on top of django.contrib.auth so it is compatible with groups by definition.
If by support you mean only the messaging provided by umessages then you should be still more specific. What do you mean by messaging within groups? Groups of contacts? Channels? Rooms? Broadcasts?
Still, I don't think that we will support group messaging in umessages because it was designed only to solve one simple problem and anything else. It is very unlikely we could extend it's scope without tremendous effort and complete redesign. Reason for that is simple: messaging, despite for many of us seems trivial, is in fact an extremely hard and complex topic.
If you need something more complex then you should search for a better solution designed to solve various messaging patterns in Django exclusively. If it is build with Django's built-in auth in mind then it is probably also compatible with userena.
My personal opinion is that if such complex messaging has to be one of core features that your users will use often then you should forget about any messaging built on top of Django's ORM. Technologies built exclusively for messaging like XMPP, or e-mail are usually the best choice in such situations.

I close this because there was no further discussion.


Bump triggers for staging bundle release v0.13.0

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/approve


make -f Makefile.usb_ep0 BOOTLOADER=1

allow building for cc-bootloader

I think that this is already covered by https://github.com/ps2/subg_rfspy/blob/dev/common.mk#L8-L15

I like yours better.


RFC: networking: Make fcntl for sockets work like Linux

The Zephyr implementation of fcntl() for sockets does not match the standard Linux implementation. The Zephyr version supports only one parameter (excluding the socket handle and the command index), whereas the Linux implementation uses variable arguments. The proposal is to re-implement fcntl to match the Linux implementation. This will make interacting with socket-based drivers somewhat easier and more flexible.
The Linux definition of fcntl uses variable arguments:
int fcntl(int fd, int cmd, ... /* arg */ );
However, Zephyr uses a single fixed flags parameter (defined in include/net/socket.h):
__syscall int zsock_fcntl(int sock, int cmd, int flags);
This limitation makes it impossible to use fcntl with any more than a single parameter, which is very restrictive. While it is possible to create a struct with additional parameters into a struct and pass a pointer to it in the flags parameter, this seems less than ideal as a general solution.
The current implementation appears to be a deliberate decision, based on this comment in subsys/net/lib/sockets/sockets.c:
/* As this is limited function, we don't follow POSIX signature, with
* "..." instead of last arg.
*/
int z_impl_zsock_fcntl(int sock, int cmd, int flags)

The proposed change is to re-implement fcntl so that it matches the Linux usage in all cases, i.e.:
int fcntl(int fd, int cmd, ... /* arg */ );
fcntl() for sockets is implemented in subsys/net/lib/sockets/sockets.c, defined in include/net/socket.h, and also defined in:
./lib/libc/minimal/include/fcntl.h
./lib/libc/arcmwdt/include/fcntl.h
./lib/libc/armstdc/include/fcntl.h
The proposed solution would be to re-implement functions z_impl_zsock_fcntl() and z_vrfy in sockets.c, re-implement zsock_fcntl_wrapper() and change the declaration of zsock_fcntl() in socket.h, and add a declaration for fcntl() to the fcntl.h header files above. The proposed solution would include patching existing usages of fcntl() as needed.
The impacts of this change should be minimal. There are ~76 instances of “fcntl” in Zephyr (excluding #include <fcntl.h>), and most of these should just work if the implementation of fcntl were to change to be a Linux style implementation.

There was some discussion about fcntl and varargs in #12002


Adding next / previous buttons

Hi @Unknwon,
it would be useful to have a next and previous button at the bottom of each page in the documentation. This way users don't need to scroll back to the top on longer pages and look for the next page in the sidebar.

Thanks your feedback!
This is indeed a helpful feature, I will think about it.

Thanks for your fast reaction :+1:

I have very limited time for developing new features, so you may need to wait for fairly long time.


chore(all): re-generate OpenAPI client(s)

This pull request was auto-generated
This PR adds the latest generated API clients.
Actions:

Review changes and validate against the examples.
Make changes where necessary.
Approve and merge.
Create a release.

Ops. That is why we should run generate on openapi PR.
CC @jsenko - we have breaking change in API that we need to fix


When clicking on token make it actually select that token as your current one and reload

When clicking on token make it actually select that token as your current one and reload

i think i know a possible work around related to setting related to basically:

Set a flag
Restart
Add newly selected ring
After that step while still in initializing, set the Module setting to the flag
Reset flag

Booom


Can't auto refetch data on mounted

I want to refetch the data every time my view is mounted.
I changed my default fetchpolicy
var apolloClient = new ApolloClient({ ...
    defaultOptions: {
    watchQuery: {
      fetchPolicy: 'network-only'
    },
    query: {
      fetchPolicy: 'network-only'
    }
  }
});

but this doesn't work, if the database have some changes and I mounted my view at least one time the data never will be updated until a web refresh.
Some hack that I did is to refetch on every mount
mounted: function() {
    this.$apollo.queries.clientType.refetch({});
...
}

but also the loading state of the query is not changing it only refetches...
I used some fix provided by other issue, but it doens't work:
#171

it maybe caused by ssrMode or ssrForceFetchDelay. in my case, setting ssrMode=false, ssrForceFetchDelay=0 for browser firgured out this problem.

I know there could be a bug here, but isn't this what subscriptions are for?

Maybe I can use a subscription, but I think it's a little bit overpowered and some cases a bit annoying.
In the case that an user want to interact with some data, change view and after some time want to return to the previous view would be awesome if there is updated data instead of fetching the data all the time

@BlueSeph28 Your code is invalid, it should be:
var apolloClient = new ApolloClient({ ...
  defaultOptions: {
    $query: {
      fetchPolicy: 'network-only'
    },
  }
});

https://vue-apollo.netlify.com/guide/apollo/special-options.html

Sorry I didn't see it was ApolloClient in your code. It's looking fine, not sure why it's not working.

So the issue is if one of your component is destroyed and mounted again, it won't do a request to the server?

Yes, it doesn't do a request... my app only get all the data from cache.
These are the steps to reproduce this.

The queries are fetched from the server.
Exit from the view a and go to view b.
Return to the view a
All data is showed to the user but is not updated from the server only fetched from cache.

I don't know if I'm misunderstanding the vue's lifecycle, apollo fetchPolicy or if this is a real bug.

Update
I noticed that only sometimes is refetching the queries, but frequently when the queries are refetched on view mount I have an error on the store
So, to fix that I need to run
this.$apollo.provider.defaultClient.resetStore();

on beforeDestroy to refetch all the data on view Mount

I'm seeing similar things in my app, as in it appears that the fetchPolicy does not make any difference and the data is always returned from cache...

Any updates on this? Still having the same issue in 2021


Feature/task 1

@ChrisKolep Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.


Question about lambdas and |> operator

The tutorial says that functions with only a lambda as an argument can be called without parenthesis.
I would have expected that these two would be the same:
[1, 2, 3].each{|e| print(e) }
{|e| print(e)} |> [1,2,3].each

This gives an error SyntaxError: Unexpected token a lambda. in the sandbox.
This works fine though, as expected:
({|e| print(e)}) |> [1,2,3].each

Huh, strange. Smells like it's a parser bug, hopefully. I'll take a poke at it later.


Support for translog pruning based on retention leases

Description

Adds support for translog pruning based on retention leases based on dynamic index setting - index.translog.retention_lease.pruning.enabled
Used primarily by replication plugin
Once replication is started on the index, this dynamic setting is enabled to prune translog operations based on the current index settings and retention leases

Issues Resolved
N/A
Check List

[ ] New functionality includes testing.

[x] All tests pass

[ ] New functionality has been documented.

[x] New functionality has javadoc added

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Is there an issue where this has been discussed? Why is this change needed? Some context will help.

@Bukhtawar - Adding tests in the next revision.
@itiyamas - Added issue here  https://github.com/opensearch-project/OpenSearch/issues/1100

start gradle check

start gradle check

start gradle check

we need to make sure these PRs are being labeled appropriately so we can track them in the change log


Network.ETH_SEPOLIA (eth-sepolia) doesn't work with createWebhook - ADDRESS_ACTIVITY

[REQUIRED] Environment

Browser version: --
Alchemy SDK version: 3.4.6

[REQUIRED] Describe the problem
Network.ETH_SEPOLIA is not compatible with alchemy.notify.createWebhook() (WebhookType.ADDRESS_ACTIVITY)
It fails with error:
Error: 503: {"message":"Internal server error","name":"ServerError"}

How to reproduce:
Se the code below
Relevant code or sample repro:
This code fails:
import { Alchemy, Network, WebhookType } from "alchemy-sdk";

const settings = {
  authToken: "your-notify-auth-token",
  network: Network.ETH_SEPOLIA,
};

const alchemy = new Alchemy(settings);

const addressActivityWebhook = await alchemy.notify.createWebhook(
  "https://webhook.site/your-webhook-url",
  WebhookType.ADDRESS_ACTIVITY,
  {
    addresses: ["0xd8dA6BF26964aF9D7eEd9e03E53415D37aA96010"],
    network: Network.ETH_SEPOLIA, // "eth-sepolia", but api requires "ETH_SEPOLIA"
  }
);

If you do the same with curl, but provide "ETH_SEPOLIA" value, this works just fine:
curl --request POST \
     --url https://dashboard.alchemy.com/api/create-webhook \
     --header 'X-Alchemy-Token: TOKEN_VALUE' \
     --header 'accept: application/json' \
     --header 'content-type: application/json' \
     --data '
{
  "network": "ETH_SEPOLIA",
  "webhook_type": "ADDRESS_ACTIVITY",
  "webhook_url": "https://example.com/rest/v1/alchemy/webhooks/address-activity",
  "addresses": [
    "0xC8B0E2ad9B19A2ffB5bFaCc88D0289E70122Ea4B"
  ]
}
'

And it fails with "eth-sepolia" value:
{
    "message": "Missing or invalid parameters.",
    "name": "ValidationError"
}

Sorry, my bad. I was providing incorrect webhook url.