Basic DSL to define operations that can fail

Overview
We introduce a thin DSL on top of dry-monads' result type to define operations that can fail.
Dry::Operation#steps accepts a block where individual operations can be called with #step. When they return a Success, the inner value is automatically unwrapped, ready to be consumed by subsequen steps. When a Failure is returned along the way, the remaining steps are skipped and the failure is returned.
Example:
require "dry/operation"

class MyOperation < Dry::Operation
  def call(input)
    steps do
      attrs = step validate(input)
      user = step persist(attrs)
      step notify(user)
      user
    end
  end

  def validate(input)
   # Dry::Monads::Result::Success or Dry::Monads::Result::Failure
  end

  def persist(attrs)
   # Dry::Monads::Result::Success or Dry::Monads::Result::Failure
  end

  def notify(user)
   # Dry::Monads::Result::Success or Dry::Monads::Result::Failure
  end
end

include Dry::Monads[:result]

case MyOperation.new.call(input)
in Success(user)
  puts "User #{user.name} created"
in Failure[:invalid_input, validation_errors]
  puts "Invalid input: #{validation_errors}"
in Failure(:database_error)
  puts "Database error"
in Failure(:email_error)
  puts "Email error"
end

The approach is similar to the so-called "do notation" in Haskell, but done in an idiomatic Ruby way. There's no magic happening between every line within the block (i.e., "programmable semicolons"). Besides not being something possible in Ruby, it'd be very confusing for people to require all the lines to return a Result type (e.g., we want to allow debugging). Instead, it's required to unwrap intermediate results through the step method. Notice that not having logic to magically unwrap results is also intentional to allow flexibility to transform results in between steps (e.g., validate(input).value_or({}))
Please, check also the first two commits that perform some extra setup for development.

@bkuhlmann I think there's something off with the test coverage setup :thinking:

Interestingly, there seems to be a bug with Git Lint. You can see this in the CI output:

If you run rake, locally, this becomes more clear as the commit messages have multiple errors:

The other issue, if you run reek locally (which would run after Git Lint passes anyway) is these errors:

There appears to be a bug with Git Lint that I've not seen before (this might be because I normally use Circle CI). I'll need to get this patched a.s.a.p. For now, you can easily correct by using Rake locally.


AQL Replace statement with autoCreate option

Hi,
Is it possible to automatically create a document during update or replace if this document is not found by given _key or given filter statement? I can not find this feature in the documentation.
If it's not possible so far, is it possible to implement this feature in the future, such as providing an option autoCreate, or something else
Best regards,
Kevin

Hi Kevin, can you read our minds? An upsert or findOrCreate is currently indeed missing. But Jan has just begun to implement it. It is expected to by available in 2.5.2.

That's great. Thanks for your answer.

+1
Today I just used this:
if (repository.collection.exists(key)) {
    // byId + replace
} else {
    // new + save
}

but I was thinking that there is a race condition in this case, when it might fail. I hope the implementation takes care of this as an atomic operation.

So this would be a replacert(), like upsert() for inserts?

@razvanphp : the following code will work fine when there is no concurrent access to the collection:
if (repository.collection.exists(key)) {
    // byId + replace
} else {
    // new + save
}

But as you pointed out it will be problematic when there are concurrent write accesses to the collection. The upsert method we're working on will be atomic. And it covers both update and replace. As you can also update with { } in case the document exists, it also covers the findOrCreate idiom.

As a suggestion, exists method should also be included in the Repository.
It is strange that byId throws an exception when document is not found instead of just returning null, but at least in this case you could check beforehand without accessing the (private) collection of the repo directly, as in my previous example.

I've added the exists method in devel and 2.5, see 142182d.

I think the original issue is covered by the new UPSERT operation in 2.6. I have written a short introduction about this feature a while ago:
http://jsteemann.github.io/blog/2015/03/27/preview-of-the-upsert-command/

Closing as done.


Lazy KeyFrames + Speed Controlled Keyframes + Pong Example

Lazy Keyframes:
Lets an animation pick up where another previous animation
left off.
Speed Controlled Keyframes:
Control an animation by defining the speed that you would like
it to animate at, rather than the time which it should have
its value at.
Pong Example:
A game of pong. Also an example of the previous two new
features.

[ ] Some bug fixes and features to add to the pong example before release.

Most objectives on this PR have been solved. A few missing ones:

[ ] Fix bug with pong paddles speed not being constant with "speed controlled keyframes" when at the extremes of their animation
[ ] Clean up cosmic time imports. Macros probably
[ ] Make the pong example playable (optional)


Integration with pytest-timeout plugin

Hi, when trying to add pytest-timeout to troubleshoot a hanging test, I get the following output:
Traceback (most recent call last):
  File "/opt/conda/lib/python3.6/site-packages/_pytest/main.py", line 203, in wrap_session
    session.exitstatus = doit(config, session) or 0
  File "/opt/conda/lib/python3.6/site-packages/_pytest/main.py", line 243, in _main
    config.hook.pytest_runtestloop(session=session)
  File "/opt/conda/lib/python3.6/site-packages/pluggy/hooks.py", line 289, in __call__
    return self._hookexec(self, self.get_hookimpls(), kwargs)
  File "/opt/conda/lib/python3.6/site-packages/pluggy/manager.py", line 87, in _hookexec
    return self._inner_hookexec(hook, methods, kwargs)
  File "/opt/conda/lib/python3.6/site-packages/pluggy/manager.py", line 81, in <lambda>
    firstresult=hook.spec.opts.get("firstresult") if hook.spec else False,
  File "/opt/conda/lib/python3.6/site-packages/pluggy/callers.py", line 203, in _multicall
    gen.send(outcome)
  File "/opt/conda/lib/python3.6/site-packages/pluggy/callers.py", line 80, in get_result
    raise ex[1].with_traceback(ex[2])
  File "/opt/conda/lib/python3.6/site-packages/pluggy/callers.py", line 187, in _multicall
    res = hook_impl.function(*args)
  File "/opt/conda/lib/python3.6/site-packages/_pytest/main.py", line 264, in pytest_runtestloop
    item.config.hook.pytest_runtest_protocol(item=item, nextitem=nextitem)
  File "/opt/conda/lib/python3.6/site-packages/pluggy/hooks.py", line 289, in __call__
    return self._hookexec(self, self.get_hookimpls(), kwargs)
  File "/opt/conda/lib/python3.6/site-packages/pluggy/manager.py", line 87, in _hookexec
    return self._inner_hookexec(hook, methods, kwargs)
  File "/opt/conda/lib/python3.6/site-packages/pluggy/manager.py", line 81, in <lambda>
    firstresult=hook.spec.opts.get("firstresult") if hook.spec else False,
  File "/opt/conda/lib/python3.6/site-packages/pluggy/callers.py", line 208, in _multicall
    return outcome.get_result()
  File "/opt/conda/lib/python3.6/site-packages/pluggy/callers.py", line 80, in get_result
    raise ex[1].with_traceback(ex[2])
  File "/opt/conda/lib/python3.6/site-packages/pluggy/callers.py", line 187, in _multicall
    res = hook_impl.function(*args)
  File "/opt/conda/lib/python3.6/site-packages/pytest_forked/__init__.py", line 35, in pytest_runtest_protocol
    reports = forked_run_report(item)
  File "/opt/conda/lib/python3.6/site-packages/pytest_forked/__init__.py", line 56, in forked_run_report
    result = ff.waitfinish()
  File "/opt/conda/lib/python3.6/site-packages/py/_process/forkedfunc.py", line 82, in waitfinish
    pid, systemstatus = waiter(self.pid, 0)
  File "/opt/conda/lib/python3.6/site-packages/pytest_timeout.py", line 140, in handler
    timeout_sigalrm(item, params.timeout)
  File "/opt/conda/lib/python3.6/site-packages/pytest_timeout.py", line 313, in timeout_sigalrm
    pytest.fail('Timeout >%ss' % timeout)
  File "/opt/conda/lib/python3.6/site-packages/_pytest/outcomes.py", line 113, in fail
    raise Failed(msg=msg, pytrace=pytrace)
Failed: Timeout >120.0s

Seems that the traceback includes only pytest-forked information.
Do you know if it's at all possible to get some traceback info from the wrapped test?
Thanks.

its not possible at the moment and i have no plans on implementing or investigating it personally as im no longer working on pytest-forked


Vault API reachable? problem

Hi,
When I was setting up a vault cluster at the last step problem occurred. When the playbook is trying to reach Vault API it is failing and errors appearing. How can I fix this?

I notice a tunnel in your output, do you use ssh or nc or something to set that up? Is it actually up?

VM that I am using is in the same server pool as nodes in Xenorchestra

Is vault running?
Is there a port 8200 listening?
Can you connect to that port from localhost?
Can you connect remotely?
Run molecule converge to compare a vault in docker.

Vault at the first node is running on second and third not. I can not connect to that port via localhost either remotely.

The issues here are for reporting bugs in the ansible-role, not for general support on Vault. Cannot reproduce it.


add vcmi and fix 0ad

vcmi.yml, have tested in deepin 15.5
app: vcmi

ingredients:
  dist: trusty
  sources:
    - deb https://mirrors.ustc.edu.cn/ubuntu/ trusty main restricted universe multiverse
  ppas:
    - vcmi/ppa

script:
  - mv usr/lib/x86_64-linux-gnu/vcmi/* usr/lib/x86_64-linux-gnu/
  - mv usr/games/* usr/lib/x86_64-linux-gnu/AI usr/share/vcmi/* -t usr/bin
  - sed -i -e 's#/usr#././#g' usr/bin/vcmiclient usr/bin/vcmilauncher usr/bin/vcmiserver
  - find . -name menu-game.png -exec cp {} vcmi.png \;
  - cat > vcmi.desktop <<EOF
  - [Desktop Entry]
  - Type=Application
  - Name=vcmi
  - Icon=vcmi
  - Exec=vcmilauncher
  - Categories=Game;
  - Comment=Open-source engine for Heroes of Might and Magic III.
  - EOF

fix 0ad.yml, have tested in deepin 15.5
app: 0ad

ingredients:
  dist: trusty
  sources:
    - deb https://mirrors.ustc.edu.cn/ubuntu/ trusty main universe
  ppas:
    - wfg/0ad

script:
  - find usr/lib/games/0ad/ -type f -iname '*.so.*' -exec mv {} usr/lib/ \;
  - mv usr/games/* usr/bin/
  - mkdir -p usr/bin/share
  - mv usr/share/games usr/bin/share
  - sed -i -e 's#/usr#././#g' usr/bin/pyrogenesis
  - sed -i '1a HERE=$(cd "$(dirname "$0")";pwd) && cd $HERE' usr/bin/0ad
  - ln -s usr/lib/x86_64-linux-gnu/libmozjs-38.so.0.0.0 usr/lib/x86_64-linux-gnu/libjs.so.0

please check and test. and you can modify the mirror from mirrors.ustc.edu.cn to another one.

Hi @shouhuanxiaoji is there any chance you could send pull requests? This would make collaborating much easier. Thank you.

I have tried PR, and I found out that I didn't understand how to use %>_<%.

Can you elaborate? I am not sure what you were having trouble with but I'd like to help.

I have been working with ymls everyday, so if i take a pull request, it will conflict with your repo. but if every pr with alone branch, the network to github is so bad...


Localization for Labs Module

Now that i18n support has been added to HospitalRun via PR #248 and issue #141 we need to localize the app.  This issue is to track localization for the labs module.  Localization in HospitalRun uses Ember-I18n.  The translations/default text will be placed in app/locales/en/translations.js.
See https://github.com/jamesarosen/ember-i18n/wiki/Doc:-Translating-Text for information on how to localize the templates.

Getting started on the project with this issue.

Assuming that @James1x0 has this, so setting to In Progress per the new Contribution policy

Will tackle this at some point this week. :+1:

Thx.
Typed or dictated imperfectly on my iPhone<PHONE_NUMBER>)

On Mar 15, 2016, at 8:35 AM, James<EMAIL_ADDRESS>wrote:
Will tackle this at some point this week.
—
You are receiving this because you commented.
Reply to this email directly or view it on GitHub:
https://github.com/HospitalRun/hospitalrun-frontend/issues/279#issuecomment-196881756


[Config] Add send to panel.js

This will send data to Webview Panel.
ONE-vscode-DCO-1.0-Signed-off-by: Yeseo Kang<EMAIL_ADDRESS>

From Issue #58
From Draft #117

Your commeit message is like
* [Config] Add sendToPanel.js

ONE-vscode-DCO-1.0-Signed-off-by: Yeseo Kang<EMAIL_ADDRESS>
* format checked

ONE-vscode-DCO-1.0-Signed-off-by: Yeseo Kang<EMAIL_ADDRESS>
which does not have This will send data to Webview Panel.
Please update your commit message.

I just fixed it! Thank you.


one contributor section on HelpOps-Hub web

Describe the feature
@mdazfar2 your website need contribution section that attract contributor to contribute
please assign this to me
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

@pandeyji711 Great idea! You are assigned

@mdazfar2 please also assign this issue to me and i'm recommended by @pandeyji711

Sure you guys work together

thanks you soo much @mdazfar2

@mdazfar2 please assign this issue to me, I'm recommended by @pandeyji711 .

Let's do it 🔥🤝

It was very thoughtful of you @mdazfar2 .


Registry replacement to open Files UWP launches the app for everything else related to Explorer

Hi, after following these steps opening folders was fine but when I went to open "additional power options" on System > energy and suspension it opened Files UWP, so I went on the search bar to open the control panel and it also opened Files. I know this is just a "hack" but so letting you guys know if no one else already did.
Thanks

@dockyflash This is a known side affect of using this method. We don't recommend editing the registry if you need access to Control Panel.


WriteImageData not called

Describe the issue
Our use requires not to load images into memory at all, since there might Gigabytes of them. The most promising attempt at implementing this seems to be to use a custom ImageLoaderFunction and a custom ImageWriterFunction. The custom image loader simply does nothing, leaving the images empty. But writing the glTF model after loading does not work then, the images are completely omitted.
To Reproduce

macOS Sonoma
Xcode 15.3

bool LoadImageData(tinygltf::Image *, const int,
                   std::string *, std::string *, int, int,
                   const unsigned char *, int, void *) {
    return true;
}

bool WriteImageData(const std::string *, const std::string *,
                    const tinygltf::Image *, bool,
                    const tinygltf::FsCallbacks*, const tinygltf::URICallbacks *,
                    std::string *, void * ptr){
    REQUIRE(ptr != nullptr);
    auto counter = static_cast<int*>(ptr);
    *counter = *counter + 1;
    return true;

}

TEST_CASE("empty-images-not-written") {

    std::string err;
    std::string warn;
    tinygltf::Model model;
    tinygltf::TinyGLTF ctx;
    ctx.SetImageLoader(LoadImageData, nullptr);
    bool ok = ctx.LoadASCIIFromFile(&model, &err, &warn, "../models/Cube/Cube.gltf");
    REQUIRE(ok);
    REQUIRE(err.empty());
    REQUIRE(warn.empty());

    REQUIRE(model.images.size() == 2);
    REQUIRE(model.images[0].uri == "Cube_BaseColor.png");
    REQUIRE(model.images[1].uri == "Cube_MetallicRoughness.png");
    REQUIRE(model.images[0].image.empty());
    REQUIRE(model.images[1].image.empty());

    int counter = 0;
    ctx.SetImageWriter(WriteImageData, &counter);
    ok = ctx.WriteGltfSceneToFile(&model, "Cube.gltf");
    CHECK(ok);
    CHECK(counter == 2);

}

Expected behaviour
It should be possible to save the images to disk in the target directory. That does not happen because WriteImageData() is not invoked for images without actual image data, even though the image has a valid URI.
Moreover, images encoded as data URIs are lost, too, because their URI is discarded during parsing, but outside of the LoadImageData() function.
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.

WriteImageData is only called when both filename and image data are filled.
https://github.com/syoyo/tinygltf/blob/fea67861296293e33e6caee81682261a2700136a/tiny_gltf.h#L3316
Probably we may better allow calling WriteImageData callback always(when WriteImageData callback is set)?

I have a fix, will create PR soon.


Ability to access LocalStorage

Proposed changes
Hello. Added ability to work with local storage via Selenide.
Docs: https://developer.mozilla.org/en-US/docs/Web/API/Window/localStorage
Checklist

[x] Checkstyle and unit tests are passed locally with my changes by running gradlew check chrome_headless firefox_headless command
[x] I have added tests that prove my fix is effective or that my feature works
[x] I have added necessary documentation (if appropriate)

Great!


Github Action: Doesn't trigger on changed lockfile

The github action currently only runs if pom.xml is updated, should this be run if the lockfile is updated aswell? Theoretically these should be synced, but now someone could change the lockfile and as long as the pom is unchanged, the github action would not fail/trigger regeneration.

The github action currently only runs if pom.xml is updated, should this be run if the lockfile is updated aswell?

yes, good catch!


Add files via upload

@m05rx7 First stage of verification passed, official domain is thechalfontsgrammar.co.uk as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision.

@m05rx7 Pull request merged. Thank you.


Bug: Produces zero length edges

Hello
Thanks for the lib!

jcvBounds min x -6.418 y -5.500 max x 3.140 y 0.009
jcvPoints[0] x -5.544 y -3.492
jcvPoints[1] x -5.010 y -4.586
jcvPoints[2] x 3.030 y -3.045
jcvPoints[3] x -5.279 y -5.474
edge[0] pos[0] x -1.318 y -2.105 pos[1] x -1.428 y 0.009
edge[1] pos[0] x -0.667 y -5.500 pos[1] x -1.318 y -2.105
edge[2] pos[0] x -0.762 y -5.500 pos[1] x -0.762 y -5.500
edge[3] pos[0] x -6.418 y -4.618 pos[1] x -6.418 y -4.618
edge[4] pos[0] x -6.418 y -4.596 pos[1] x -1.318 y -2.105
edge[5] pos[0] x -6.418 y -4.643 pos[1] x -3.598 y -5.500
Gray lines are XY axis
Green box is bounds
Green lines are valid edges
Green crosses are input points
Two magenta crosses are problematic edge2 and edge3
I'm using double
#define JC_VORONOI_IMPLEMENTATION
#define JCV_REAL_TYPE double
#define JCV_ATAN2 atan2
#define JCV_SQRT sqrt
#define JCV_FABS fabs
#include <jc_voronoi/jc_voronoi.h>

I found this too. Edges with zero length are detected on line 1060:
if( jcv_point_eq(&e->pos[0], &e->pos[1]) ) ...
The spurious edge could be removed at that point, but edges are stored in a singly-linked list, so removing it is expensive. In my own project I just discard zero-length edges as I process them.
There are no back-links (for example from half edges to edges) to check for, so just discarding is sufficient.

As @david94133 mentions, the edges are inside a singly linked list.
I don't want to add extra memory or operations for a doubly linked list, due to the fact that iterating over the edges isn't the main part of this library, and also since skipping the invalid edges is easily done when required.
Therefore I added an iterator function const jcv_edge* jcv_diagram_get_next_edge( const jcv_edge* edge ); to do this automatically. I've updated the examples, and added a test for this too.
Will close this issue once it's merged into master.

Fixed in #26 (v0.5.0)


Remove L2 component since it's not used.

The L2 component is used to route external traffic into the PWD containers. Since that's a functionality that we're not using or
planning to use in PWG for the moment, I'm taking it away since it's not
required

Sounds sensible. Let's discuss tomorrow, including understanding whether https://github.com/play-with-go/play-with-go/tree/main/docker/l2 should therefore simply be removed?

including understanding whether https://github.com/play-with-go/play-with-go/tree/main/docker/l2 should therefore simply be removed?

Yes, that should be removed as well. I'll update this PR.


"Error loading script: lora_script.py" while starting SD

Using latest notebook
Error loading script: lora_script.py
Traceback (most recent call last):
  File "/content/gdrive/MyDrive/sd/stable-diffusion-webui/modules/scripts.py", line 256, in load_scripts
    script_module = script_loading.load_module(scriptfile.path)
  File "/content/gdrive/MyDrive/sd/stable-diffusion-webui/modules/script_loading.py", line 11, in load_module
    module_spec.loader.exec_module(module)
  File "<frozen importlib._bootstrap_external>", line 850, in exec_module
  File "<frozen importlib._bootstrap>", line 228, in _call_with_frames_removed
  File "/content/gdrive/MyDrive/sd/stable-diffusion-webui/extensions-builtin/Lora/scripts/lora_script.py", line 4, in <module>
    import lora
  File "/content/gdrive/MyDrive/sd/stable-diffusion-webui/extensions-builtin/Lora/lora.py", line 238, in <module>
    def lora_apply_weights(self: torch.nn.Conv2d | torch.nn.Linear | torch.nn.MultiheadAttention):
TypeError: unsupported operand type(s) for |: 'type' and 'type'

Startup then completes but LoRA is not listed under extra networks and cannot be used in generations. Entering a LoRA prompt returns
Skipping unknown extra network: lora

I have the same error

same here

Same here

No Lora for me either :(

same

same issue

I comminted out the lines after the function self call in lora.py. and it got it working again for me.
238
def lora_apply_weights(self): #: torch.nn.Conv2d | torch.nn.Linear | torch.nn.MultiheadAttention):
and
298
def lora_reset_cached_weight(self): # : torch.nn.Conv2d | torch.nn.Linear):

This error has been repored on AUTOMATIC1111
https://github.com/AUTOMATIC1111/stable-diffusion-webui/issues/8968

I comminted out the lines after the function self call in lora.py. and it got it working again for me.
238
def lora_apply_weights(self): #: torch.nn.Conv2d | torch.nn.Linear | torch.nn.MultiheadAttention):
and
298
def lora_reset_cached_weight(self): # : torch.nn.Conv2d | torch.nn.Linear):

I tried this and forced the file not to be updated.
And there is no error when SD starts.
But when I tried to generate image it shows error again.
activating extra network lora with arguments [<modules.extra_networks.ExtraNetworkParams object at 0x7fb6e40f6bb0>]: RuntimeError
Traceback (most recent call last):
File "/content/gdrive/MyDrive/sd/stable-diffusion-webui/modules/extra_networks.py", line 75, in activate
extra_network.activate(p, extra_network_args)
File "/content/gdrive/MyDrive/sd/stable-diffusion-webui/extensions-builtin/Lora/extra_networks_lora.py", line 23, in activate
lora.load_loras(names, multipliers)
File "/content/gdrive/MyDrive/sd/stable-diffusion-webui/extensions-builtin/Lora/lora.py", line 213, in load_loras
lora = load_lora(name, lora_on_disk.filename)
File "/content/gdrive/MyDrive/sd/stable-diffusion-webui/extensions-builtin/Lora/lora.py", line 175, in load_lora
module.weight.copy_(weight)
RuntimeError: output with shape [32, 32, 1, 1] doesn't match the broadcast shape [32, 32, 3, 3]

I comminted out the lines after the function self call in lora.py. and it got it working again for me. 238 def lora_apply_weights(self): #: torch.nn.Conv2d | torch.nn.Linear | torch.nn.MultiheadAttention): and 298 def lora_reset_cached_weight(self): # : torch.nn.Conv2d | torch.nn.Linear):

Thanks so much for finding the solution! It was really hard!!

Still not working. I am getting the same issuse as @f4cyw

Still not working. I am getting the same issuse as @f4cyw

I just checked the master file and it seems I'm missing some code in the master file itself. The master file Lora.py file is missing code on lines 238 and 298.

This seem to be a issue caused by the Stable diffusion v2 comptabilty change.  Not anything to do with colab so its just a version revertion and it fixed the problem.
!git checkout 64da5c4
Add this line to your notebook and it will turn back to a usable version

@netrunner-exe 's workaround has it temporarily fixed for me
In the colab interface go to /content/gdrive/MyDrive/sd/stable-diffusion-webui/extensions-builtin/Lora and open up lora.py
enter the following on a new line at the top of the file:
from __future__ import annotations
Then restart SD and you should be able to used LoRA again. Note that if you reinstall SD lora.py will reset and you'll need to add the line again.

@JarvisSan22 it gave me this error: ImportError: cannot import name 'ModuleAvailableCache' from 'lightning_utilities.core.imports' (/usr/local/lib/python3.9/dist-packages/lightning_utilities/core/imports.py)
WebUI doesn't launch at all now.

Me too! Besides, I don't know any programs or anything like that! There's no way to fix it even if you suggest a solution! I just hope this issue is anything big and general, and it will be fixed soon!

!git checkout 64da5c4

where exactly to add it, before git clone line?

This seem to be a issue caused by the Stable diffusion v2 comptabilty change. Not anything to do with colab so its just a version revertion and it fixed the problem. !git checkout 64da5c4 Add this line to your notebook and it will turn back to a usable version

Where do I add this? I tried using git command line but it is stuck on the same commit and I can't even change commits. git log shows only one commit and no history.

fixed no need to git checkout, just update the notebook

Still getting "Error loading script: lora_script.py".

do a screenshot to the error

do a screenshot to the error
Thanks always.
But I think I have same issue.
pls find below error in colab.

activating extra network lora with arguments [<modules.extra_networks.ExtraNetworkParams object at 0x7fca262b8190>]: RuntimeError
Traceback (most recent call last):
File "/content/gdrive/MyDrive/sd/stable-diffusion-webui/modules/extra_networks.py", line 75, in activate
extra_network.activate(p, extra_network_args)
File "/content/gdrive/MyDrive/sd/stable-diffusion-webui/extensions-builtin/Lora/extra_networks_lora.py", line 23, in activate
lora.load_loras(names, multipliers)
File "/content/gdrive/MyDrive/sd/stable-diffusion-webui/extensions-builtin/Lora/lora.py", line 214, in load_loras
lora = load_lora(name, lora_on_disk.filename)
File "/content/gdrive/MyDrive/sd/stable-diffusion-webui/extensions-builtin/Lora/lora.py", line 176, in load_lora
module.weight.copy_(weight)
RuntimeError: output with shape [32, 32, 1, 1] doesn't match the broadcast shape [32, 32, 3, 3]

fixed no need to git checkout, just update the notebook

thanks, resolved

this updates are just for broken things not improve nothing


Add Classic Context Menu option

I really hate the new one in Windows 11, do you think it's possible for you to add this...? I know it's possible through the registry editor during post-installation.
(I don't even know if you wanted people to post requests here...)

This can be achieved by adding this .reg file (found at How-To Geek) in the Scripts to run whenever a user logs on for the first time section:
Windows Registry Editor Version 5.00

[HKEY_CURRENT_USER\Software\Classes\CLSID\{86ca1aa0-34aa-4e8b-a509-50c905bae2a2}]
@=""

[HKEY_CURRENT_USER\Software\Classes\CLSID\{86ca1aa0-34aa-4e8b-a509-50c905bae2a2}\InprocServer32]
@=""

Would you be willing to add an easy check option for accessibility?

Well, since I will probably need that setting myself when switching to Windows 11, I have added it via https://github.com/cschneegans/unattend-generator/commit/3b34d7366a95d9b8e75c205cdf98bb627845eee7.
In the online form, see the new Use classic context (right-click) menu in Windows 11 checkbox. You might need to refresh (F5) the page.

Wow you're quick


Issue11-Add-dark-mode: Theme added

-> Added Theme icon in navigation bar

@Sathyan-B please change the colors wherever there is inconsistency. for eg. navbar shouldnt be white. the background grid needs to be white as well. you can do this using the tailwind class dark:


revs can spill jugs

Description
had a shitter just go into med spill jugs then defile for the rest of its life its terrible

this tbh

Wait. Weren't you the one who told me this was a planned feature?
https://github.com/space-wizards/space-station-14/issues/21554#issuecomment-1804892683

the solution is to not leave jugs of medicine everywhere imo

the solution is to not leave jugs of medicine everywhere imo

where else are they supposed to go? there's no fridge to put the jugs in

(why the fuck can revs spill things)

because revenants are really annoying

i would be fine with spilling if it made them corporeal so if they go on a shitter spree med has some recourse

what if chaplain could see revenants all the time and was alerted when one spawned? bonus points if bible banishes revenant (removing all of their points and abilities, if they get hit with no points they get round removed)

i would be fine with spilling if it made them corporeal so if they go on a shitter spree med has some recourse

what if chaplain could see revenants all the time and was alerted when one spawned? bonus points if bible banishes revenant (removing all of their points and abilities, if they get hit with no points they get round removed)

Please no. Revenant is already the weakest antagonist. He needs a buff, not a nerf. You literally can hit him one time with crossbow and he disintegrate.

Solution: fix the chem fridge already

But chem dispensers are already a thing? What is it if not a chem fridge once you empty it out of basic reagent jugs?

its possible to make ghetto smart fridge but i have only personally seen it done once (by me) and its still no reason for rev to be able to do this in complete safety


Button effects

Added active button effects
Fixes #846

We already have an effect for clicking on a text in the navbar

A follow-up effect after that makes the UI worse and moreover this does not suit well with the theme

Ok


Switch support

Fixes #54

Hi @mathieupotier, I just added some more messages to translation json files, can you please translate the new error messages in fr.json? For now there are original English messages.

#57 <== here


Big text fields in tables in details page

The table seems to be trying to include all of the text in a cell in the table representation.  This makes the tables extremely hard to interprete.  Could we set a maximum field length and perhaps use a ... or more type of thing...

moved to wishlist


In the tests, c2mop:direct-slot-definition-class is defined with &key rather than &rest

In tests/tests.lisp, there are several definitions of methods for c2mop:direct-slot-definition-class, all of them with &key in the lambda list. This is wrong, because the lambda list of  c2mop:direct-slot-definition-class is (class &rest initargs) https://clos-mop.hexstreamsoft.com/generic-functions-and-methods/.
With the &key, it means it doesn't allow any keys, so gives an error when the implemtation actually checks (because at least the name is in the initargs).

(Wow, LispWorks?! Nice! ;))
Unfortunately, you are correct. I had wrongly assumed that the c2mop:direct-slot-definition-class generic function specifies &key &allow-other-keys. Frankly, I think this should have been the case, or alternatively, the primary method specialized on standard-class could specify &key name initform initfunction type allocation initargs readers writers documentation. (Thoughts, @pcostanza?) My code would have been correct in either of those cases according to CLHS 7.6.4 Congruent Lambda-lists for all Methods of a Generic Function and CLHS 7.6.5 Keyword Arguments in Generic Functions and Methods.
I will perform a full review of my ready-to-use libraries with respect to this issue and issue new releases where appropriate. I expect to do this today, or otherwise very soon. I'll let you know when this is done.
(I should probably also start testing on LispWorks eventually, but I'm extremely busy and I have a deadline coming up...)

Alright, this is fixed in class-options 1.0.1. Here is the updated tarball. (Beware of caching issues. The new tarball has class-options_1.0.1 as top-level directory.)
I'm actually pretty livid that the CLOS MOP specification does not define the c2mop:direct-slot-definition-class and related generic functions as specifying &key &allow-other-keys, but whatever.

The CLOS MOP specification as we know it today is not a finished specification. There are a lot of these little omissions that, if there had been some proper standardization process around it, would have probably been caught. Alas, this is what we have to live with.

Thank you. I am still pretty solidly satisfied with the CLOS MOP specification. It is a GEM!
(Also, this could easily be fixed in a library.)


Default quiet=false when background with logfile

Closes #565

@xeron How's this? 😸

Well, it may help, but the whole situation raises some questions.

Why do you have different behavior in background/foreground mode?
Why do you have 2 variables controlling logging – VERBOSE and QUIET?

It's confusing
Seems like you don't respect VERBOSE in background mode? Or in both bg/fg modes?

I see this code in logger builder:
      def level
        if @verbose && !@quiet
          MonoLogger::DEBUG
        elsif !@quiet
          MonoLogger::INFO
        else
          MonoLogger::FATAL
        end
      end

but looks like in background mode you're using some different logic?

@xeron I agree that the current situation with verbose and quiet is confusing.  Legacy code!  (and some recent changes)
My understanding of the current intent, is:
default_quiet = false

if options[:background]
  default_quiet = true
end

if options[:logfile]
  default_quiet = false
end

log_level = if options.fetch(:quiet, default_quiet)
              FATAL
            elsif options.fetch(:verbose, false)
              DEBUG
            else
              INFO
            end

I'm very much ready to believe that the current implementation does not match this intent.  Supporting explicit log levels instead of the verbose / quiet thing sounds great to me, assuming that's what you're suggesting 😸

Yeah, I think keeping only logfile and loglevel is better. But this is up to you. With explanation of what QUIET / VERBOSE actually do it's a bit cleaner for me now (if it works as you described).

Soo, what happened to this PR and issue I reported? This is still affect me :(

I started using resque-scheduler and am noticing the same issue that @xeron is reporting here. Is there a workaround at least so we can get logs?


Debug tests: Warning: You should set SILENT_OBSERVATORY

I want to debug my tests.
Tried both in VS Code and AS.
Set some breakpoints, run debug from IDE:

Run from terminal (i tried to set a variable here, and it stills throws the same warning):

flutter doctor -v output:

Hi @subzero911
I see there's an open issue addressing the case you described. https://github.com/flutter/flutter/issues/63075
Please follow up on that issue, I'm closing the current one as a duplicate.
If you disagree, please write in the comments and I will reopen it.
Thank you


Add selected images by order of selection in Android

Hi, is there a way to add the selected images by order of selection? Currently its pretty random.
PR #1970 fixes the issue for iOS but not Android.
Has anyone been able to figure out how to fix this issue for Android?
Thanks
Originally posted by @kadinuguru in https://github.com/react-native-image-picker/react-native-image-picker/issues/1893#issuecomment-1682672562

Is this going to get implemented?

any news?

any updates?


Refresh nmclient after changing state

This is required to be able to have a chance to see changes after
applying a new state:
    previous_state = netinfo.show()
    netapplier.apply(state)
    import time; time.sleep(10)
    current_state = netinfo.show()

Without this, current_state and previous_state will be the same as
long as netapplier.aplly() does not run GLib.MainLoop.

Coverage increased (+0.1%) to 84.762% when pulling 2971f737c82438d4e829edf13a6b7282d0425e8c on tyll:refresh into 426f61d4f0afb305e8a037347cfdfd53b1258541 on nmstate:master.

What causes a refresh in case the mainloop is used?

The main loop makes the client process DBus messages with the new information.


Implement FindTaskCommand for TodoList

Fixes #175

Codecov Report

Merging #419 into master will decrease coverage by 0.47%.
The diff coverage is 12.50%.

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


md_video keeps repeating

Batocera build version
batocera-bcm2712-40-20240731
Your architecture
Raspberry Pi 5
Your Graphic Processor Unit(s) (GPU)
Standard RPi5 GPU
Issue description

md_video keeps repeating 3 times, then shows title screenshot

I want it to only play a video once, and then show the title screenshot after the first playback has ended.

Is there a md_video value for in a theme .xml, or a setting for batocera that will disable md_video looping?

Detailed reproduction steps
watch a video play
Details of any attempts to fix this yourself
tried looking around in a theme .xml
Details of any modifications you have made to Batocera.
using ARCADEPLANET theme
Logs and data
No response

Moved issue to batocera-emulationstation repo.


Code search using git grep doesn't respect branch name

Description
When using the code search on another branch, Gitea doesn’t respect the branch name and only searches the default branch. I guess the search URL loses the branch name from the context because there is no related information passed together with the request.
Gitea Version
1.22.0
Can you reproduce the bug on the Gitea demo site?
Yes
Log Gist
No response
Screenshots
No response
Git Version
git version 2.44.2
Operating System
Gentoo
How are you running Gitea?
Running the executable binaries from github release. And I think it doesn't matter to this issue.
Database
MySQL/MariaDB

Relevant line:
https://github.com/go-gitea/gitea/blob/f446e3b4ab2d3f787a17d1cfe09ee8f5f7b10089/routers/web/repo/search.go#L81

The root problem is that there is no "branch name" support on the web UI (and I guess there is no such support for other search engine/indexers).
So it could be an enhancement to introduce "current branch" support.

The root problem is that there is no "branch name" support on the web UI (and I guess there is no such support for other search engine/indexers).

IIRC the other code indexers will only index on the default branch, and GitHub also has the same behaviour, as if we allow searching on other/all branches our index will explode in size

I see. I’ve read the source code and found that the branch name is in the context. I thought there were some bugs because it doesn’t recognize the branch name. By the way, git grep should support searching by branch name, and the branch name is, in fact, part of the URL. Thus, I think technically we can pass the branch name from the URL to the code search box as another query parameter.

If we implemented search via git grep with branch name but other indexes will not support it. It's inconsistent between different indexes. Maybe it will result in confusing?


fix(pci.projects.failover-ips): display message on ipfo order

Is there any internal ticket reference related to this pull request?


Move documentation to Read the Docs

+1

Are we sure about this? Now everything is in the one place. Do we have enough docs to move our README to RTD?

I will answer myself: because on RTD it's easier to browse different versions of documentation (e.g. development vs latests stable release). Currently many people are confused about differences between PyPI version and Github.
So I'm +1.

Done.


Dll plugin docs

Hello. DllPlugin is cool. But there is no doc, how to use it. There are many things to mention. For example, where I should exclude server-side dependencies? Ininternals\webpack\webpack.dev.babel.js I can see such comment.
/**
 * If DLLs aren't explicitly defined, we assume all production dependencies listed in package.json
 * Reminder: You need to exclude any server side dependencies by listing them in dllConfig.exclude
 *
 * @see https://github.com/mxstbr/react-boilerplate/tree/master/docs/general/webpack.md
 */

But there is no doc with such name. Maybe you will provide some docs for DLLPlugin. Unfortunately I can not do it myself, because I do not know how it works.

Good idea! @datapimp can you pick this up please?

I'm currently working on SSR support for react-boilerplate and stuck with this. I still can't understand how it works (neither why it's needed). @mxstbr Is there any reference I can see?

How can I help?

On Nov 2, 2016, at 6:40 PM, Haz<EMAIL_ADDRESS>wrote:
I'm currently working on SSR support for react-boilerplate and stuck with this. I still can't understand how it works (neither why it's needed). @mxstbr Is there any reference I can see?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

@diegohaz here's an article from medium that @datapimp wrote explaining what the DLL plugin is for.
https://medium.com/@soederpop/webpack-plugins-been-we-been-keepin-on-the-dll-cdfdd6cb8cd7
I was also confused about exactly what it's purpose was til I read the article.

Closing due to inactivity, but this will def stay on the radar

You can find a section in your package.json where you can exclude packages:
"dllPlugin": {     "path": "node_modules/react-boilerplate-dlls",     "exclude": [       "chalk",       "compression",       "cross-env",       "express",       "ip",       "minimist",       "sanitize.css",       "nodemailer"     ],     "include": [       "core-js",       "lodash",       "eventsource-polyfill"     ]   },

@pengyanb what's core-js here?

@shekhardesigner core-js doc can be found here. But it is not relevant to the question here. The point is that you have the chance to exclude certain packages from the dll build process.

Because webpack crawls your dependency graph, beyond just what is in your
dependencies.  Babel uses core-js
On Fri, Apr 27, 2018 at 4:21 AM Shekhar K. Sharma<EMAIL_ADDRESS>wrote:

@pengyanb https://github.com/pengyanb I don't see core-js as part of
any dependencies within package.json
https://github.com/react-boilerplate/react-boilerplate/blob/master/package.json.
I wonder why add it in Dll in that case?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/react-boilerplate/react-boilerplate/issues/783#issuecomment-384915746,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AACn0OgRNxv6E-tIUq7f17kt9kahasblks5tsuMggaJpZM4JUENn
.


Using with Yeoman

I got everything working with little effort, just putting this here as it may be useful to other and could be considered part of the readme perhaps.
I am using the yeoman angular app/build config so the bower section of scripts gets dynamically built in the HTML file when 'grunt serve' is run or 'bower.json' is updated.
A quick update to the json file ensured that I had all the correct files in the correct order.
I did however need to make 1 change to "flexslider.bower.json"
"main": "./jquery.flexslider.js",
needed to be added in under main. Doing that and updating the bower.json file of the app ensures that everything loads every time and in the correct order that is required.
Thanks for the port I guess, it is working very well on my side.

I too am using the yeoman framework, I posted an issue  a few days ago. Have you been able to configure the slide show speed. It is not reacting to my changes.

@dataonline, never tried to change the settings, was happy with the defaults.

Thanks for the hint. so @RemeJuan do you think we should make that bower.json change standard?

@thenikso, I am not sure it would be required, however at the same time I do not think it would negatively impact others.
In my scenario when installing, if this line is present the --save switch would also write it into the bower section of my html, at processing time it also uses the same reference during the preparehtml function as then it runs through the bower config to make sure all the files referenced actually exist, but its existence is determined by the file references in the bower.json for each plugin.
For me without that reference, the file gets removed from the HTML at build time. It is possible that anyone using a similar setup, not specifically yoeman, would run into the same issue.
If not adding it as a default maybe to the FAQ or as a note in the README.

Well this issue should provide some documentation on it. I'm not confident on making the change myself as I have not the time to replicate the problem here. Feel free to open a PR tho!


Union does not support JSON and arrays

Search before asking

[X] I had searched in the issues and found no similar issues.

Version
2.0.1
What's Wrong?
CREATE TABLE t_json_array (
id varchar(1024) NULL,
array_int array<int(11)> NULL,
array_float array NULL,
array_double array NULL,
array_string array NULL,
array_date array NULL,
array_datetime array<datetimev2(0)> NULL,
array_decimal array<decimalv3(9, 0)> NULL,
json1 json NULL
) ENGINE=OLAP
DUPLICATE KEY(id)
COMMENT '数组JSON测试'
DISTRIBUTED BY HASH(id) BUCKETS 3
PROPERTIES (
"replication_allocation" = "tag.location.default: 1",
"is_being_synced" = "false",
"storage_format" = "V2",
"light_schema_change" = "true",
"disable_auto_compaction" = "false",
"enable_single_replica_compaction" = "false"
);
select  a.id,a.array_int,a.json1   from db_demos.t_json_array  a
union
select b.id,b.array_int,b.json1  from db_demos.t_json_array b
: errCode = 2, detailMessage = Doris hll, bitmap, array, map, struct, jsonb column must use with specific function, and don't support filter, group by or order by. please run 'help hll' or 'help bitmap' or 'help array' or 'help map' or 'help struct' or 'help jsonb' in your mysql client.
What You Expected?
Union support JSON and arrays
How to Reproduce?
No response
Anything Else?
No response
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

This error is within the expected range. It is recommended not to perform union on the array column if possible.


How Can I Know if the Plugin Is Running?

I apologise for my rookie-like curiosity. How can I know if the plugin is running?

Install LSP with Package Control
Install LSP-Grammarly with Package Control
Open a Markdown file
In the lower left corner, you should see some status about Grammarly installing and then running idle

Thank you.


Better http error codes for the login.py endpoint

https://developer.mozilla.org/en-US/docs/Web/HTTP/Status/403
https://developer.mozilla.org/en-US/docs/Web/HTTP/Status/401

LGTM

Thanks! And sorry for the delay reviewing this. 😅
There are some conflicts because this was all refactored, would you be up to updating it?


refactor(): Make interfaces smaller and use composition

Bigger interface - weaker abstarction.

@obalunenko thanks for your contribution, however we cannot merge that.
I really think exposing these public interfaces has been a mistake.
As interfaces are implicitely implemented in go, most of the time interfaces are declared at usage side.
I'd rather not add new interfaces we will have to support.


Make GCI/VCI affiliations accessible via API

Currently, GCI/VCI affiliations data is publicly accessible via a manually maintained spreadsheet. We need to make the affiliation data within our resource publicly available via API so that other ClinGen resources (such as the GeneTracker and GPM) can make use of this information.
https://broadinstitute.atlassian.net/browse/CGSP-451

Signed off by stakeholders, closing.


Git transcript processing tweaks

We ask GPT to respond in json and give a little more information about the commit.

The mocked transcript is stored in the same cache.

A new script is made to make the jsonls which selects transcripts with specified properties from the commit_information.json
git show --first-parent is used so we can get a git diff for pull requests (thanks gpt)
The repo name is inferred from the url instead of being constant.
We checkout main instead of head.

Looks good to me; have you tested this new system prompt? Does it work well?

Yeah it works pretty well. It correctly judges things to be docs or configuration. The complexity score is pretty tightly concentrated in 3-4 but I think it does provide enough signal. It does sometimes fail to reply in valid json. But only like 20% of the time.


ImgBot doesn't seem to work properly

I have been installing for 1 day and have not received any commits or pr.
I used the default configuration, which is just clicking the install button.

Hey sorry you're having trouble. I can't keep a look at this for you soon

I took a look at your compression runs and there are 2 issues at play.
First issue:

You are being heavily affected by #170 - we have a plan to fix it if you want to take a look at that issue description. I'm going to take this on ASAP and see if this can clear things up for your repo.

Second issue:

A few of your images are throwing  'ImageMagick.MagickCorruptImageErrorException - Unable to decompress the jpeg file.'
images\2017\11\-----2017-11-16-09.15.59.jpg
images\2017\11\-----2017-11-16-09.15.59-2.jpg
images\2017\11\-----2017-11-16-09.15.59-1.jpg

You might want to have a look at those files and see if there is anything obvious about them
I'll reply back here when #170 is resolved and updated and see if that does the trick
Stay tuned!

We released a fix today and that seemed to work - you should have your optimizations now if you want to confirm everything looks right on your end.
I noticed your repo took a really looooooong time to process and had quite a lot of trouble completing once those initial errors were out of the way.
We are going to have to make some performance improvements soon I think, but I'm going to close this issue in the meantime.
Thanks for opening this here and raising awareness. 👍

@dabutvin Thank you for your kind help. 🙏


Enhancement: Make a Proper Github Actions YAML

Self-Explanatory
I'll probably make a pull request doing this

was considering doing this, actions might come a bit later. but yeah we'll add them in.


All Camera Modes Implemented

DO NOT MERGE YET (still debugging specific areas)

all unique cameras for areas added
reorganizations of more than half of Camera.js (to look like og c file)
all general camera modes added
right trigger implementation soon
random bugs fixed

Alright, you can pr now, everything is done

finally done lol

You can also open draft PRs to prevent accidental merging


Update to v5 Legacy content collections section

📚 Subject area/topic
Update to v5 guide
📋 Page(s) affected (or suggested, for new content)
en/guides/upgrade-to/v5/
📋 Description of content that is out-of-date or incorrect
https://github.com/withastro/docs/blob/7d726121b0e784a3c62dcb322ab90426307cd2c7/src/content/docs/en/guides/upgrade-to/v5.mdx?plain=1#L150-L151
should be
    params: { slug: post.slug },
    params: { slug: post.id },

also
https://github.com/withastro/docs/blob/7d726121b0e784a3c62dcb322ab90426307cd2c7/src/content/docs/en/guides/upgrade-to/v5.mdx?plain=1#L132
gives no-useless-escape from eslint
I think this will work the same
loader: glob({ pattern: "**/[^_]*.md", base: "./src/data/blog" }),

🖥 Reproduction in StackBlitz (if reporting incorrect content or code samples)
https://github.com/OliverSpeir/astro-starter/blob/8a49d0fbdadd957e2b32b44a2abd6ced1d257135/src/pages/blog/[slug].astro#L8
https://github.com/OliverSpeir/astro-starter/blob/8a49d0fbdadd957e2b32b44a2abd6ced1d257135/src/content.config.ts#L5
https://stackblitz.com/github/oliverspeir/astro-starter

You're right I missed  src/pages/[id].astro and didn't realize we no longer suggested using [slug] at all

But if they had [id].astro they wouldn't have params: { slug: post.slug } they would have params: { id: post.slug },

Right, but the explanation in the text above says You may also need to update dynamic routing file names to match a changed getStaticPaths() parameter: So, we're assuming you HAD the [slug] param, and are showing updating to [id]. It's called out and intentional!
And, it's not like you can't use slug as your param, it's only that there is no slug property anymore. So in the case of the tutorial, it's not wrong: [...slug] is the param, so the params key is slug while the value is the post's id (which is the property each entry has).
src/pages/posts/[...slug].astro // [`slug`] parameter

params: { slug: post.id }, props: { post }, // params key = slug; value = id

I think it's right that we don't need the \, so the examples can be updated


[auto] Update benchmark results

Automated changes by create-pull-request GitHub action

Closing due to errors in table headers


Add support for breadcrumbs without scope

I am working on a component where I cannot define a scope (as the Zap logger is wrapped into further abstraction that I cannot unwrap to add a scope). To work around that I am adding an option to disable the "no breadcrumbs in global scope" rule if needed (I think the rule is a good idea, just really can't use it otherwise).
What do you think?

hi. why can't just use config.EnableBreadcrumbs ? it does the same thing

Hey, as far as I understood it does not. EnableBreadcrumbs requires me to use a scope that is not the global scope (by calling log.With(zapsentry.NewScope()), right? I can't do that because I never get the log object in hand due to the library I am using (I just get to provide a list of zap.Core's to attach to it).

ok, i got this. im thinking why we need this check "(c.sentryScope != nil".
m.b. we could replace it with c.sentryScope() too? or remove it at all. so by default we will write to the global scope. m.b. it's not good, but we have a limit=1000records. its not so many for a great memory leak, isn't it?

@02strich hi, what do you think?

@TheZeroSlave sorry for the delay. Yeah that would also work. I can adjust the PR to work that way if you like.

@TheZeroSlave sorry for the delay. Yeah that would also work. I can adjust the PR to work that way if you like.

np) yes, that would be great

Hey @TheZeroSlave, any plans on merging this? I also need this functionality for my app. I think you probably haven't merged the PR because the last change you requested hasn't been added. I can push that and then maybe you can merge this?

Hey @TheZeroSlave, any plans on merging this?
hi, i almost forgot about this PR because no changes were provided.
let's replace "c.sentryScope" with "c.scope()" and remove useless checking then.

@TheZeroSlave Why do you need sentryScope though? Why don't just use the hub current scope?

ok, closed due https://github.com/TheZeroSlave/zapsentry/pull/44


Investigate reduction of memory spikes during compaction

Especially when compacting larger time ranges, there are very notable spikes in memory usage.
When under querying load, the memory occupied by queries smoothens the spikes to a good degree, but not entirely.
It would be cool if it was possible to reduce the spikes a fair bit. For that we first need a proper understanding which allocations are actually causing it. Profiling so far did not give a clear answer yet.

After some investigation it turns out that compaction actually has a pretty low memory footprint. Most of it comes from starting new blocks. Several improvements have been made there. The remains largely need improvements in Prometheus itself.


testcase testcases for partition documents refactor

What problem does this PR solve?
Issue Number: close #xxx
Related PR: #xxx
Problem Summary:
NOTICE: in this pr we changed jdbc conf of regression test p0&cloud_p0. we expect it has no other effect.
Release note
None
Check List (For Author)

Test

[ ] Regression test
[ ] Unit Test
[ ] Manual test (add detailed scripts or steps below)
[x] No need to test or manual test. Explain why:

[ ] This is a refactor/code format and no logic has been changed.
[ ] Previous test can cover this change.
[x] No code files have been changed.
[ ] Other reason

Behavior changed:

[x] No.
[ ] Yes.

Does this need documentation?

[ ] No.
[x] Yes. https://github.com/apache/doris-website/pull/1364

Check List (For Reviewer who merge this PR)

[ ] Confirm the release note
[ ] Confirm test cases
[ ] Confirm document
[ ] Add branch pick label

run buildall

run buildall

run buildall


[Improvement]Add more tip when drop workload group failed

Proposed changes
Add more tip when drop workload group failed.

run buildall


node.js script execution

How can I use this node-cron to automate and execute my node.js script in another dir?

In the index.js in root


how can I increase small objects'mAP?

Hi
I selected 10 small objects in SUNRGBD and I made those trained in this model.
but, result showed only 0.1mAP( learning_rate = 0.001, epoch = 12, Objects were cup,bottle,bowl,book etc)
I guess this model was designed for big objects not small things, right?
then, how can I increase small objects' mAP? are there any parameters to do that?

Thank you sooooo much!

I'm compused a little
feature levels you meaned is n_scores=2? or n_reg_outs=6?
does what both mean?

Here we decrease n_outs, in_channels and n_scales.

I got it
Thank you!


Continuous plugin builds

https://wiki.jenkins.io/display/JENKINS/Hosting+Plugins#Continuous plugin builds

Done https://ci.jenkins.io/job/Plugins/job/aws-codecommit-trigger-plugin/


Tensorflow won't compile on windows c++

when I try to compile tensorflow master on windows10 with vs2015, I got below errors after compiling for half an hour.
"D:\Documents\GitHub\tensorflow\tensorflow\contrib\cmake\build\ALL_BUILD.vcxproj" (default target) (1) ->
"D:\Documents\GitHub\tensorflow\tensorflow\contrib\cmake\build\array_ops_gen_cc.vcxproj" (default target) (2) ->
"D:\Documents\GitHub\tensorflow\tensorflow\contrib\cmake\build\tf_array_ops.vcxproj" (default target) (3) ->
"D:\Documents\GitHub\tensorflow\tensorflow\contrib\cmake\build\tf_core_framework.vcxproj" (default target) (4) ->
(CustomBuild target) ->
C:\Program Files (x86)\MSBuild\Microsoft.Cpp\v4.0\V140\Microsoft.CppCommon.targets(171,5): error MSB6006: "cmd.exe" exited with code 3. [D:\Documents\GitHub\tensorflow\tensorflow\contrib\cmake\build\tf_core_framework.vcxproj]
"D:\Documents\GitHub\tensorflow\tensorflow\contrib\cmake\build\ALL_BUILD.vcxproj" (default target) (1) ->
"D:\Documents\GitHub\tensorflow\tensorflow\contrib\cmake\build\force_rebuild_target.vcxproj" (default target) (81) ->
C:\Program Files (x86)\MSBuild\Microsoft.Cpp\v4.0\V140\Microsoft.CppCommon.targets(171,5): error MSB6006: "cmd.exe" exited with code 3. [D:\Documents\GitHub\tensorflow\tensorflow\contrib\cmake\build\force_rebuild_target.vcxproj]
148 Warning(s)
2 Error(s)

Attached please find the full log
Thanks for your help!
[msbuild.log](https://github.com/tensorflow/tensorflow/files/2122908/msbuild.log

Got the same errors. As far as I see, the recent TF release is not compatible with the latest Bazel AND the source code for Protobuf is out of date.

Later I change to compile tensorflow r1.3 with cuda 8.0, cudnn 6 (previously I use cuda 9 with cudnn 7) according to https://www.zybuluo.com/kalluwa/note/1110511.
And it successfully compiled!!!

we just updated the protobuf version at head. So proto issues should be gone.
for bazel issues, @meteorcloudy.
To debug your problem better, would you mind filling the issue template?

It has been 14 days with no activity and the awaiting response label was assigned. Is this still an issue?


HDDS-4792. Remove redundant state check for handleDeleteChunk

What changes were proposed in this pull request?

Remove redundant state check for handleDeleteChunk.

What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-4792
How was this patch tested?
Locally test.

@elek @adoroszlai Could you help to review this PR? Thanks in advance.


Splash page non functional

Currently the main page (empty relative url) does nothing. You can't enter the exercise creation screen. You can't log in or create an account.

Automatically redirects to channel list


Cordova plugin - handleOpenUrl is never called

I'm having a lot of consistency problems with the cordova integration. It looks like BNCDevice.m pretty much does nothing from a standard install using
cordova plugin add https://github.com/BranchMetrics/Web-SDK.git
The handleOpenURL function is never called because the plugin is never told to listen for the handleOpen notification
The getOpenData seems redundant also. I can see some calls to it inside the build.js file supplied with the plugin, but the breakpoint in either getInstallData or getOpenData is never hit. Again this is in minified code so it's tricky to see what's going on but I'll start digging deeper to see why.
I guess what I'd like to know is are these things meant to be called. Or is the plugin actually redundant now and it should all work from the WebSDK? I'm getting a bit confused and I don't want to waste time going down the wrong path.
Any advice would be appreciated!

Hey @wedgybo - this is a pretty atrocious list of faults that we can dig into right away. I'm pretty sure you don't need to use handleOpenURL, as all you need to do is to execute this code once the ‘deviceready’ event fires and each time the ‘resume’ event fires
branch.init("YOUR BRANCH KEY HERE", function(err, data) {
app.initComplete(err, data);
});
You need to use the Cordova version instead of the Web SDK version because it does execute some device specific code, but the methods are the same as the Cordova library.
It seems like you're on the right path!

Ok. I'm calling the branch.init function on both of those events. Here's my test code.
function initBranch() {
    console.log('BRANCH INIT');
    branch.init(BRANCH_KEY, { isReferrable: true }, function (err, data) {
      console.log('BRANCH DATA', err, data);
    });
}

document.addEventListener('deviceready', initBranch, false);
document.addEventListener('resume', initBranch, false);

Now I see BRANCH INIT and  BRANCH DATA on my cold start no problem, from a warm start it only runs BRANCH INIT.
The problem appears to lie somewhere here. https://github.com/BranchMetrics/Web-SDK/blob/master/src/3_branch.js#L306
The SDK know's there's a session so it fires on with the web flow. Unfortunately it doesn't have the information from the click so the data object is never updated and always returns the first data object found.
Looking through the code, I expect it to get here https://github.com/BranchMetrics/Web-SDK/blob/master/src/3_branch.js#L319 and pull the information stored natively from processing the handleOpenUrl request.
I think the condition never being hit in my case because it looks like the close function isn't doing what it's meant to in this instance.
I'll have another play again to see what's going on if you think I'm on the right path?

hey @wedgybo great catch with this. I stepped through this and can confirm that line 306 is, in fact, the issue. I've added  afurther conditional to only init the Web SDK on the following line if it is the web environment (if WEB_BUILD) is true, so now, the init function on Cordova continues on, calls open, and loads the new data object. I'm going to open a PR for this - which you're welcome to pull down and give a run before we release the fix. Feel free to send any other problems you're having with this and I'll be happy to get quick fixes out!

@wedgybo Can you confirm that the latest version https://github.com/BranchMetrics/Cordova-Ionic-PhoneGap-Deferred-Deep-Linking-SDK did in fact resolve this?

I've not been back on to it recently due to other commitments. It wasn't working last I checked but I noticed there's been quite a few commits since then. I will try to find time to test it this week again.

@wedgybo
I'm going through all open issues. Did you have a chance to test whether the latest Cordova SDK works for you?

Going back to this today. So I'll let you know!

Hi @wedgybo - we've migrated the Cordova/Ionic SDK to a separate repository: https://github.com/BranchMetrics/Cordova-Ionic-PhoneGap-Deferred-Deep-Linking-SDK and: https://www.npmjs.com/package/branch-cordova-sdk This should be resolved.


fix: update default dash character

This character (U+2500 - Box Drawings Light Horizontal) is better suited for drawing lines since most fonts will render several of these as one continuous line with no overlap.
The previous character caused a weird overlap on some fonts.
Before:

After:

Thank you. I haven't tried U+2501 (current - Heavy Horizontal) with the other fonts.

Thank you for your work, it’s really helpful 😊


[Host] Refactor 'Retry' delegate in IConsumerErrorHandler as response type

While the discussed 'pipeline reimagination' was a little too ambitious for the time that I have available; I have refactored the implementation to extract the retry delegate from IConsumerErrorHandler.OnHandleError and to rather include it as a response value. This does introduce a breaking change, but allows the handler to perform all of the previous logic albeit with a small change for retry.
The OnHandlerError method signature therefore changes from:
Task<ConsumerErrorHandlerResult> OnHandleError(T message, Func<Task<object>> retry, IConsumerContext consumerContext, Exception exception);

to:
// where `attempts` is the execution count
Task<ConsumerErrorHandlerResult> OnHandleError(T message, IConsumerContext consumerContext, Exception exception, int attempts);

In order to retry the execution (with a new scope as requried), the implemented method must return ConsumerErrorHandlerResult.Retry. Further to this, abstract implmentations of IConsumerErrorHandler have been added with methods to support Failure(), Success(object? response = null) and Retry() for a cleaner experience (similar to return Ok(); with asp.net controllers).
Example:
public class RetryHandler<T> : ConsumerErrorHandler<T>
{
    private static readonly Random _random = new();

    public override async Task<ConsumerErrorHandlerResult> OnHandleError(T message, IConsumerContext consumerContext, Exception exception, int attempts)
    {
        if (attempts <= 3)
        {
            var delay = (attempts * 1000) + (_random.Next(1000) - 500);

            await Task.Delay(delay, consumerContext.CancellationToken);
            return Retry();
        }

        return Failure();
    }
}

This is an excellent and pragmatic approach to improving the error-handling pipeline while achieving the message scope recreation as discussed in issue #347. The implementation is mostly backward-compatible and could potentially outperform the Func<Task> next delegate approach.
I've suggested a few minor adjustments to the documentation for clarity and completeness.
Once those are addressed, I’ll be happy to approve the PR. Great work!

It's been released under v3 rc:
https://www.nuget.org/packages/SlimMessageBus.Host.AzureServiceBus/3.0.0-rc901


WIP 2.0 update: basic implementation

working search
moves SearchAPI functionalities to PublicAPI
drops examples from README (because Swagger works)

Signed-off-by: Mateusz Susik<EMAIL_ADDRESS>

Even if not 100% perfect, it would be great to update the distribution for 2.0 since 1.2 is no longer supported in the Public API (as of 2018-03-01)

@MSusik please do as you think. I moved out completely from the field, nowadays.


Allow distinguishing comments from code when searching.

It would be nice to search perl code independently from POD embedded in same file (and otherwise).
Real use case: look at something big like Mojolicious. It have a lot of documentation embedded in .pm files, and that documentation consists mostly of examples - which are perl code. So, when you run something like
    ack -- '->on\(request' Mojo*

you got 80% of results from POD and only 20% from module code.
Thing is, in most cases you look either for documentation OR for code, but not for both at one. While documentation doesn't contain code everything works fine, but if it contain a lot of real code examples... It makes no sense to ask grep to distinguish between perl code in code and in POD, but when we talk about ack it looks like native feature for it. :)

I would love this but I don't want to touch parsing of input files.


API error when creating token with rate_limit_window_size specified

Trying to specify a rate_limit_window_size when creating a project access token, the API returns the following strange error:
$ http POST https://api.rollbar.com/api/1/project/415372/access_tokens X-Rollbar-Access-Token:redacted name:='"bar"' scopes:='["read"]' status:='"enabled"' rate_limit_window_size=60
HTTP/1.1 400 Bad Request
Access-Control-Allow-Credentials: true
Access-Control-Allow-Origin: *
Alt-Svc: clear
Content-Length: 99
Content-Type: application/json; charset=utf-8
Date: Mon, 19 Oct 2020 07:07:36 GMT
Server: nginx/1.17.9
Via: 1.1 google
X-Rate-Limit-Limit: 5000
X-Rate-Limit-Remaining: 4999
X-Rate-Limit-Remaining-Seconds: 60
X-Rate-Limit-Reset:<PHONE_NUMBER>
X-Response-Time: 41ms

{
    "err": 1,
    "message": "u'60' is not one of [0, 60, 300, 1800, 3600, 86400, 604800, 2592000]"
}

However if I first create a token without specifying rate_limit_window_size, calling PATCH on that token does successfully update rate_limit_window_size.
$ http PATCH https://api.rollbar.com/api/1/project/415372/access_token/972e844d22934f94a044e6e775a5cf55 X-Rollbar-Access-Token:redacted rate_limit_window_size:=60
HTTP/1.1 200 OK
Access-Control-Allow-Credentials: true
Access-Control-Allow-Origin: *
Alt-Svc: clear
Content-Length: 14
Content-Type: application/json; charset=utf-8
Date: Mon, 19 Oct 2020 07:16:54 GMT
Server: nginx/1.17.9
Via: 1.1 google
X-Rate-Limit-Limit: 5000
X-Rate-Limit-Remaining: 4997
X-Rate-Limit-Remaining-Seconds: 19
X-Rate-Limit-Reset:<PHONE_NUMBER>
X-Response-Time: 76ms

{
    "err": 0
}

Ugh... it was my own typo.  I wrote rate_limit_window_size=60 instead of rate_limit_window_size:=60.  The former is the syntax for HTML form posting; the latter, which I intended, is the syntax for JSON body.


Traceback on installing BeautifulSoup==3.2.1

Hey there! Trying out pipenv on a project I work on that has a dependency on BeautifulSoup==3.2.1
When I try to install this dependency I get an error.
Using /usr/local/bin/python2 to create virtualenv…
⠋Running virtualenv with interpreter /usr/local/bin/python2
New python executable in /Users/cdean/.virtualenvs/soup_problem-7VIi_C6l/bin/python2.7
Also creating executable in /Users/cdean/.virtualenvs/soup_problem-7VIi_C6l/bin/python
Installing setuptools, pip, wheel...done.

Virtualenv location: /Users/cdean/.virtualenvs/soup_problem-7VIi_C6l
Creating a Pipfile for this project…
- - -
soup_problem 0
 🍔  - ❯❯❯ pipenv install BeautifulSoup==3.2.1
Installing BeautifulSoup==3.2.1…
Collecting BeautifulSoup==3.2.1
Installing collected packages: BeautifulSoup
Successfully installed BeautifulSoup-3.2.1

Adding BeautifulSoup==3.2.1 to Pipfile's [packages]…
Locking [dev-packages] dependencies…
Locking [packages] dependencies…
Traceback (most recent call last):
  File "/usr/local/bin/pipenv", line 11, in <module>
    sys.exit(cli())
  File "/usr/local/lib/python3.6/site-packages/pipenv/vendor/click/core.py", line 722, in __call__
    return self.main(*args, **kwargs)
  File "/usr/local/lib/python3.6/site-packages/pipenv/vendor/click/core.py", line 697, in main
    rv = self.invoke(ctx)
  File "/usr/local/lib/python3.6/site-packages/pipenv/vendor/click/core.py", line 1066, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/usr/local/lib/python3.6/site-packages/pipenv/vendor/click/core.py", line 895, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/usr/local/lib/python3.6/site-packages/pipenv/vendor/click/core.py", line 535, in invoke
    return callback(*args, **kwargs)
  File "/usr/local/lib/python3.6/site-packages/pipenv/cli.py", line 1621, in install
    do_lock(system=system)
  File "/usr/local/lib/python3.6/site-packages/pipenv/cli.py", line 1009, in do_lock
    project=project
  File "/usr/local/lib/python3.6/site-packages/pipenv/utils.py", line 447, in resolve_deps
    resolved_tree = resolver.resolve()
  File "/usr/local/lib/python3.6/site-packages/piptools/resolver.py", line 107, in resolve
    has_changed, best_matches = self._resolve_one_round()
  File "/usr/local/lib/python3.6/site-packages/piptools/resolver.py", line 195, in _resolve_one_round
    for dep in self._iter_dependencies(best_match):
  File "/usr/local/lib/python3.6/site-packages/piptools/resolver.py", line 274, in _iter_dependencies
    dependencies = self.repository.get_dependencies(ireq)
  File "/usr/local/lib/python3.6/site-packages/piptools/repositories/pypi.py", line 145, in get_dependencies
    self._dependencies_cache[ireq] = reqset._prepare_file(self.finder, ireq)
  File "/usr/local/lib/python3.6/site-packages/pipenv/patched/pip/req/req_set.py", line 634, in _prepare_file
    abstract_dist.prep_for_dist()
  File "/usr/local/lib/python3.6/site-packages/pipenv/patched/pip/req/req_set.py", line 129, in prep_for_dist
    self.req_to_install.run_egg_info()
  File "/usr/local/lib/python3.6/site-packages/pipenv/patched/pip/req/req_install.py", line 439, in run_egg_info
    command_desc='python setup.py egg_info')
  File "/usr/local/lib/python3.6/site-packages/pipenv/patched/pip/utils/__init__.py", line 707, in call_subprocess
    % (command_desc, proc.returncode, cwd))
pip.exceptions.InstallationError: Command "python setup.py egg_info" failed with error code 1 in /var/folders/jc/jdnjsh991dg3zbl75z4p5hh80000gp/T/tmpgx3mhdsobuild/beautifulsoup/
- - -
soup_problem 1
 🍔  - ❯❯❯ pipenv run python -c "import BeautifulSoup; print BeautifulSoup.__version__"
3.2.1
- - -
soup_problem 0
 🍔  - ❯❯❯

So the installation works but then something else crashes, causing pipenv to ultimately return an errorcode 1 (as seen in my prompt).
How do I find out what's going wrong behind the exception pip.exceptions.InstallationError: Command "python setup.py egg_info" failed with error code 1 ?
Thanks!

Additional proof it gets installed correctly, the imported module lives in the venv:
/Users/cdean/.virtualenvs/soup_problem-7VIi_C6l/lib/python2.7/site-packages/BeautifulSoup.pyc

There is a new Pipenv version, 7.5.1, would you mind testing with it?

Beautiful soup isn't packaged well, and won't run on Python 3. That's your problem.

@cpdean is it possible to use beautifulsoup4 in your project?

beautifulsoup4 can work

@kennethreitz I'd appreciate knowing more about the problem with the packaging for beautifulsoup so I can poke the maintainer to fix it.

@brainwane usually you can clone the problematic package and run python setup.py egg_info. the stacktrace should tell you the issue. lots of times its a circular dependency upon itself or assumes some package is installed


protocol: fixed deadline datarace #86

Fixed the race by using atomic.Value instead of unprotected time.Time
It was nessasarry to restructure the Conn struct because of this:
struct of size 80 could be 72 (fieldalignment)

Coverage increased (+0.02%) to 95.563% when pulling 3e554e44fe17cd8e092f7fb4337d446057fae502 on mschneider82:master into 2e44d7a76a851d66890ab341403253afae5caac2 on pires:main.

Thanks @mschneider82! Could you please turn on race mode for testing to prove this works (as it looks it does)?

TIL https://medium.com/@sebassegros/golang-dealing-with-maligned-structs-9b77bacf4b97

I have added a nil check for atomic.Value with default time.Time{} (no deadline timeout) because TestPassthrough failed. In case no deadline was set

done, ready for review & merge


STM32H7: Bus fault when reading corrupt flash sectors

Describe the bug
The stm32h7 flash driver crashes the system with a bus fault if the data it reads has a double ECC error (DBECCERR).
The stm32h7 flash has an integrated ECC capable of correcting single errors and detecting double errors. The bus fault is expected behavior from the hardware, but very unfortunate behavior for the flash API. It would be much better to return an error code from flash_read().
To Reproduce
You will need a flash word with a double ECC error.

Edit flash_stm32h7x.c and comment out the loop where it says "Check if 256 bits location is erased".
Build Zephyr for e.g. a Nucleo-144 board with the flash shell enabled
Use the flash shell to write the same flash word multiple times using different data. (Normally not allowed, but see step 1).
Use the flash shell to read that flash word. With high probability you should see a bus fault due to a double ECC error.

Expected behavior
The flash_read() API should return an error code when it finds a double ECC error, such as -EIO. It should probably also log the ECC error.
Impact
The bus fault stops mcuboot from working reliably when a firmware upgrade is interrupted in the middle, or when a flash word has developed an ECC error for some other reason.
Attempting to work around it on a different level than the flash driver is complicated, see: https://github.com/mcu-tools/mcuboot/issues/713#issuecomment-768546357
Logs and console output
uart:~$ flash erase 0x1e0000
Erase success.
uart:~$ flash write 0x1e0000 5555aaaa
Write OK.
Verified.
uart:~$ flash write 0x1e0000 aaaa5555
Write OK.
Verified.
uart:~$ flash read 0x1e0000 4
[00:00:25.082,000] <err> os: ***** BUS FAULT *****
[00:00:25.082,000] <err> os:   Precise data bus error
[00:00:25.082,000] <err> os:   BFAR Address: 0x81e0000
[00:00:25.082,000] <err> os: r0/a1:  0x24007af0  r1/a2:  0x081e0000  r2/a3:  0x00000004
[00:00:25.082,000] <err> os: r3/a4:  0x24007af0 r12/ip:  0x00000008 r14/lr:  0x08026501
[00:00:25.082,000] <err> os:  xpsr:  0x81000200
[00:00:25.082,000] <err> os: Faulting instruction address (r15/pc): 0x080357cc
[00:00:25.082,000] <err> os: >>> ZEPHYR FATAL ERROR 0: CPU exception on CPU 0
[00:00:25.082,000] <err> os: Current thread: 0x24004e00 (shell_uart)
[00:00:25.105,000] <err> os: Halting system

Environment (please complete the following information):

OS: Debian GNU/Linux
Toolchain: cross-compile
Commit: dccfe766

Suggested fix
It is possible to stop the bus fault from happening by using BFHFNMIGN and FAULTMASK during the read from the flash. Without the bus fault it is then possible to read the status from the flash controller and translate that to a status code. I have a PoC available here: https://github.com/endiantechnologies/zephyr/commit/d541116eeb66e4eb5cd92f457fab454e256e86c1
I'm a bit out of my depth when it comes to fiddling with such low-level ARM Cortex registers, so I think someone more knowledgeable should have a look at the code. Other places in the driver that read from the flash also need to be protected. But at least with the patch an error is returned instead of the system crashing:
uart:~$ flash read 0x1e0000 4
Read ERROR!
[00:00:05.709,000] <wrn> flash_stm32h7: Bank2 ECC error at 0x000e0000
[00:00:05.709,000] <err> flash_stm32h7: Status Bank2: 0x04000000

And I believe that mcuboot is better able to handle this kind of error code than it can handle a bus fault.

This looks like a good way to handle this.
I'll will pull your patch and test it out.

I made a test on nucleo_h723zg :  but I cannot reproduce the issue (when commenting the loop at Check if 256 bits location is erased)
uart:~$ flash write 0x1ff00 55 111 67 11 41 65
Reading back written bytes:
37 6f 43 0b | 29 41
uart:~$ flash read 0x1ff00 8
37 6f 43 0b | 29 41 ff ff

@weinholtendian I'm mivong this issue to enhancement as I don't see the reported behavior as a bug, though I agree that behavior could (/should) be improved.
Can you push your proposal as a PR and we'l perform a review so it can be integrated.

@FRASTM This was tested with an STM32H745 Nucleo-144 (NUCLEO-H745ZI-Q). I believe that the H723 also will exhibit the same behavior as otherwise described in this issue. Both reference manuals (RM0399 and RM0468) contain this type of text: When DBECCERR1/2 flag is raised, a bus error is generated. In case of successive double error detections, a bus error is generated each time a new data is sent back to the requester through the AXI interface. However, the method for deliberately creating a double ECC error is just something I improvised and verified on the H745 and it might be that it doesn't work on the H723 (or something is off with my instructions). It will still be possible to encounter double ECC errors in the field due to normal causes for flash failures, or when interrupting an mcuboot upgrade (as mentioned in mcu-tools/mcuboot#713).
@erwango Ack, will do this next week.

@FRASTM I might have figured out why you couldn't reproduce the bus fault from my instructions. The flash read command probably never went out to the flash device in your case because the data written with flash write was still in the cache. (For various reasons I happened to test this with the MQTT sample and it must have been doing enough things with the system that the cache line got flushed). Could you try again and reset the system after the flash write command?
Updated instructions to get a double ECC error, which can be used to verify that the suggested patch works as intended:

Get an STM32H745 Nucleo-144 (or equivalent).
Modify samples/hello_world/prj.conf:

CONFIG_SHELL=y
CONFIG_LOG=y
CONFIG_FLASH=y
CONFIG_FLASH_SHELL=y
CONFIG_FLASH_PAGE_LAYOUT=y
CONFIG_FLASH_MAP=y
CONFIG_FLASH_MAP_SHELL=y
CONFIG_FLASH_LOG_LEVEL_DBG=y
CONFIG_REBOOT=y

Find "Check if 256 bits location is erased" in drivers/flash/flash_stm32h7x.c and comment out the loop.
Build with west build -b nucleo_h745zi_q_m7 samples/hello_world/.
Flash with west flash.
Run the commands as in the transcript below.

*** Booting Zephyr OS build zephyr-v2.5.0-2336-g8c4ae05131be  ***
Hello World! nucleo_h745zi_q_m7

[00:00:00.000,000] <dbg> flash_stm32h7.stm32h7_flash_init: Flash initialized. BS: 32
[00:00:00.000,000] <dbg> flash_stm32h7.stm32h7_flash_init: Block 0: bs: 131072 count: 16
[00:00:00.000,000] <dbg> flash_stm32h7.flash_stm32h7_write_protection: Disable write protection
uart:~$ flash erase 0x1e0000
Erase success.
[00:00:04.937,000] <dbg> flash_stm32h7.flash_stm32h7_erase: Erase offset: 1966080, len: 131072
[00:00:04.937,000] <dbg> flash_stm32h7.flash_stm32h7_write_protection: Disable write protection
[00:00:05.730,000] <dbg> flash_stm32h7.flash_stm32h7_write_protection: Enable write protection
uart:~$ flash write 0x1e0000 5555aaaa
Write OK.
Verified.
[00:00:09.178,000] <dbg> flash_stm32h7.flash_stm32h7_write: Write offset: 1966080, len: 4
[00:00:09.178,000] <dbg> flash_stm32h7.flash_stm32h7_write_protection: Disable write protection
[00:00:09.178,000] <dbg> flash_stm32h7.flash_stm32h7_write_protection: Enable write protection
[00:00:09.179,000] <dbg> flash_stm32h7.flash_stm32h7_read: Read offset: 1966080, len: 4
uart:~$ flash write 0x1e0000 aaaa5555
Write OK.
Verified.
[00:00:13.061,000] <dbg> flash_stm32h7.flash_stm32h7_write: Write offset: 1966080, len: 4
[00:00:13.061,000] <dbg> flash_stm32h7.flash_stm32h7_write_protection: Disable write protection
[00:00:13.061,000] <dbg> flash_stm32h7.flash_stm32h7_write_protection: Enable write protection
[00:00:13.061,000] <dbg> flash_stm32h7.flash_stm32h7_read: Read offset: 1966080, len: 4
uart:~$ flash read 0x1e0000 4
001E0000: 55 55 aa aa                                      |UU..             |

[00:00:18.578,000] <dbg> flash_stm32h7.flash_stm32h7_read: Read offset: 1966080, len: 4
uart:~$ kernel reboot cold�*** Booting Zephyr OS build zephyr-v2.5.0-2336-g8c4ae05131be  ***
Hello World! nucleo_h745zi_q_m7

[00:00:00.000,000] <dbg> flash_stm32h7.stm32h7_flash_init: Flash initialized. BS: 32
[00:00:00.000,000] <dbg> flash_stm32h7.stm32h7_flash_init: Block 0: bs: 131072 count: 16
[00:00:00.000,000] <dbg> flash_stm32h7.flash_stm32h7_write_protection: Disable write protection
uart:~$ flash read 0x1e0000 4
[00:00:01.880,000] <dbg> flash_stm32h7.flash_stm32h7_read: Read offset: 1966080, len: 4
[00:00:01.880,000] <err> os: ***** BUS FAULT *****
[00:00:01.880,000] <err> os:   Precise data bus error
[00:00:01.880,000] <err> os:   BFAR Address: 0x81e0000
[00:00:01.880,000] <err> os: r0/a1:  0x24002550  r1/a2:  0x081e0000  r2/a3:  0x00000004
[00:00:01.880,000] <err> os: r3/a4:  0x24002550 r12/ip:  0x00000000 r14/lr:  0x08004329
[00:00:01.880,000] <err> os:  xpsr:  0x81000200
[00:00:01.880,000] <err> os: Faulting instruction address (r15/pc): 0x08009e08
[00:00:01.880,000] <err> os: >>> ZEPHYR FATAL ERROR 0: CPU exception on CPU 0
[00:00:01.880,000] <err> os: Current thread: 0x240003b8 (shell_uart)
[00:00:01.924,000] <err> os: Halting system

The flash now has a fault that you can use to verify that the patch works. After applying the patch and flashing again (west flash does not erase the fault we created) the double ECC error is reported as -EIO and there is no bus fault:
uart:~$ *** Booting Zephyr OS build zephyr-v2.5.0-2336-g8c4ae05131be  ***
Hello World! nucleo_h745zi_q_m7

[00:00:00.000,000] <dbg> flash_stm32h7.stm32h7_flash_init: Flash initialized. BS: 32
[00:00:00.000,000] <dbg> flash_stm32h7.stm32h7_flash_init: Block 0: bs: 131072 count: 16
[00:00:00.000,000] <dbg> flash_stm32h7.flash_stm32h7_write_protection: Disable write protection
uart:~$ flash read 0x1e0000 4
Read ERROR!
[00:00:03.582,000] <dbg> flash_stm32h7.flash_stm32h7_read: Read offset: 1966080, len: 4
[00:00:03.582,000] <wrn> flash_stm32h7: Bank2 ECC error at 0x000e0000
[00:00:03.582,000] <err> flash_stm32h7: Status Bank2: 0x04000000


Update restricted text

Partially closes https://github.com/MetaMask/metamask-docs/issues/1195
Updates language for restricted methods to be less narrow. Follow up to @jiexi restricted methods PR

Thanks @alexandratran. Applied your suggestions.