RHTAPSRE-467: Webhook configuration for internal staging

The webhook configuration will allow to set the webhook target based on the repository url. Thus, external SCMs will emit webhooks to the smee-server, while internal SCMs will emit webhooks directly to the PAC endpoint on the cluster.

/lgtm

known issue not related to the change:
 [FAIL] [multi-platform-build-service-suite Multi Platform Controller E2E tests] ibm power pc dynamic allocation when the Component with multi-platform-build is created [It] The multi platform secret is populated [multi-platform, ibmp-dynamic]
  /tmp/tmp.pGksalKaBy/tests/build/multi-platform.go:467


[Proposal] Use ListenAsync (blocking) and fix CT usage

Currently the semantics of using the cancellation token vs dispose are not consistent: Sometimes the background task in StartMessagePumpAsync is stopped with the CT and sometimes with Dispose()
The CT in StartMessagePumpAsync is often not only used for cancelling the method itself but also to stop the started background task - this is a against best practices of using CTs

See for example: https://github.com/aspnet/Hosting/blob/master/src/Microsoft.Extensions.Hosting.Abstractions/BackgroundService.cs
To stop the processing, they use another (internal) CT

I propose to switch to ListenAsync which blocks (when awaited) and can be stopped with the provided CT

If you don't want to block, you just don't await and stop the pump with the cancellation token later - sometimes dispose will also stop the pump (not consistent everywhere yet).

Usage:
Inline:
using (var receiver = new AzureServiceBusReceiver(...))
{
    await receiver.ListenAsync(ct);
}

The listening can be stopped with the CT and then dispose cleans up the receiver. In theory you can call ListenAsync again (but usually this does not make sense).
With host builder:
public class MyBackgroundService : BackgroundService
{
    private IMessageReceiver _messageReceiver;

    public MyBackgroundService()
    {
        _messageReceiver = new AzureServiceBusReceiver(...);
    }

    protected override async Task ExecuteAsync(CancellationToken stoppingToken)
    {
        await _messageReceiver.ListenAsync(stoppingToken);
    }

    public override void Dispose()
    {
        _messageReceiver.Dispose();
        base.Dispose();
    }
}

public static async Task Main(string[] args)
{
    var host = new HostBuilder()
        .ConfigureServices(services => services.AddSingleton(new ))
        .Build();

    await host.RunAsync();
}

What do you think of this?

Hey sorry for the delay, this is all good, the only thing is changing interface method name - it will break the clients. So we should either push to v8 or keep the names compatible?

I’d rename it to clearly indicate that the semantics changed and the behavior is streamlined


We should derive the system chrome ui overlay style from a layer

On iOS, using AppBarBehavior.scroll can cause the status bar overlay to go from being painted over a dark background to over a light one (or vice versa).  Doing so without calling SystemnChrome.setSystemUIOverlayStyle() will mean that the status bar will be hard to see in one of the app bar states (either visible or scrolled off).
e.g. in the Flutter Gallery's contacts demo:
App Bar visible

App Bar scrolled away

/cc @HansMuller @Hixie maybe we should have have the option of showing a shadow in the app bar space... this is what the Google Photos app does in the single-photo page.

The contacts demo already adds a shadow. The shadow doesn't help in this case because it's being added to the flexibleSpaceBar which fades out when the app bar get smaller than its toolbar.
We can't just make the shadow (the DecoratedBox with a gradient child, see contacts_demo.dart) the AppBar's parent because the Scaffold expects it's appBar to be an AppBar so that it can change its elevation and etc.
We could make the AppBar render the shadow itself with a flag for turning that on and off. Not super fluttery but probably the simplest tack.

This only matters on iOS, right?

(https://github.com/flutter/flutter/issues/4430 will make adding the shade effect trivial.)

Yeah it only matters on iOS

The idea here is that we would have a StatusbarLayer (or whatnot) that sets the color, and after composition we would inspect the layer tree to see what layer ended up at the top center of the screen, and set the color of the status bar based on that color.

Then you would wrap whatever you wanted to have set the color in widgets that created these layers.

Another issue with the current mechanism is that the chrome color is set during AppBar's build and during page transitions, the outgoing AppBar's build can be called one last time after the incoming page's built so there's no good way for the incoming page to set the color in a contained way without being overridden.

Proposed solution:
Create an AnnotatedRegionLayer<T>, which is a LayerContainer with a valuefield of type T and a Size. Add a findRegion(Offset, Type) method to Layer which walks the layer tree backwards and finds the first AnnotatedRegionLayer<T> (where T is the given Type) that contains the given Offset, and returns the value of that layer. (The walk can be implemented by having each Layer implement findRegion appropriately, transforming the offset before passing it to its child, according to whatever logic is used to position the child; layers with bounds (e.g. the clipping layers and AnnotatedRegionLayer itself) would only propagate to child layers if the point is inside themselves. PictureLayers just return false always. The walk can be terminated as soon as anyone matches; children take precedence.)
Create an AnnotatedRegion<T> widget that pushes this layer to the layer tree. (Similar to Metadata.)
In the compositeFrame() method, after we build the scene, we should additionally call findRegion on the root layer for the top center of the screen, for a particular type (probably SystemUiOverlayStyle), and then use the setSystemUIOverlayStyle with the value found, if any.

Working on an initial implementation here https://github.com/flutter/flutter/pull/17672/files, seems to work OK even though it is only half implemented.

@jonahwilliams any update on this?

@tvolkert Took a break to work on android/buildroot issues but I'm back to working on it this week.

@jonahwilliams Please give an update. This is actually a blocker for mulligan.

PR is done waiting for LGTM and Green to land


Replace native routing with AJAX request

Now the list of routes is generated in a native way.
Replace it with AJAX request for faster page rendering.

Will be released in version 1.7.0


Error when open nerdtree

When I open nerdtree with this plugin it`s return errors, shown below
Error detected while processing function
177[2]..178[22]..181[7]..144[9]..142[36]..238[4]..NERDTreeGitStatusRefreshListener[2]..NERDTreeGitStatusRefresh:
line    6:
E484: Can't open file /tmp/vZEZ6gM/1
I install this plugins using pathogen and last fresh version from github.
Arch linux x64
I found problem.
I use fish instead of bash and when I add
if &shell =~# 'fish$'
    set shell=sh
endif

into my .vimrc, everything works fine

This was because that vim couldn't run system command in fish.
So what you did is the right way.

I think this can be added to FAQ

@Crandel +
Thanks for solve the problem

I have this exact problem, but i cant fix it with those lines on .vimrc


Make use of SFunArray compatible with old and new versions of SBV

sbv-8.15 removes the deprecated SFunArray type. This causes GHC to warn that the respective hiding clause in the import of Data.SBV is redundant.
The PR does the following:

removes the hiding clause from the import of Data.SBV that GHC warns about,
upgrades the cabal and and stack builds to use sbv-8.15, and
renames LegacySFunArray and SFunArray into PactSFunArray.

Ideally, we would also up-pin the nix-builds to sbv-8.15, but this would require the new libBF dependency which, in turn, requires hashable >= 1.3. The latter constraint can not be satisfied with our current nix package snapshot.

Re: nix not being able to do hashable >= 1.3: Can we just add a source override? Or fetchFromHackageDirect ?

Re: nix not being able to do hashable >= 1.3: Can we just add a source override? Or fetchFromHackageDirect ?

I tried to go down that road and gave up eventually, because there's too many packages with tight upper bounds on it. I concluded that it would be probably easier to just create a completely new nix-snapshot.

https://github.com/kadena-io/pact/pull/873/ bumps kpkgs to a version with a more recent reflex-platform which should have a more up to date nix-snapshot

The ci/gitlab/gitlab.com build failure is addressed in a different PR.


Restructure project

Restructure project from space.taran to dev.arkbuilders

@hieuwu @kirillt : This might be conflicting with what we've been doing in this branch: rename-package?
We are doing the same work !

@hieuwu @kirillt : This might be conflicting with what we've been doing in this branch: rename-package? We are doing the same work !

No worries, there might be miscommunication here . As @kirillt  told me to work on this. Although this conflicts with rename-package, let's use this as a reference because rename-package does not work. Feel free to close this PR when the job is done

@tuancoltech sorry, I indeed asked @hieuwu to help with this.. But we can compare the changes in both PRs since it's error-prone kind of work, so both PRs are useful.

@hieuwu We've finalized the changes in rename-package which serves the same purpose as of this PR.
Could you take a look and see if there's any differences between the two?
https://github.com/ARK-Builders/ARK-Navigator/pull/387

Good PR, we've merged the similar PR, seems to be almost same.


Orderly Client Shutdown

Bug Report
Version
0.7.1
Platform
Linux (probably not relevant)
Description
Not sure if this is a bug, a feature request, or just expected behaviour, but:
Currently, when a client is dropped, the server connection always ends with an error: unexpected end of file.
I assume that's because the client just drops the connection without properly closing it?
There doesn't seem to be any API to orderly terminate the connection.
This is mostly an annoyance because the errors pollute the server log.

@seanmonstar does hyper/h2 send a goaway when its dropped? I feel like I've seen that code before but I would assume that would handle the more graceful aspect of this.


Trainer won't rescan after reconnect (v1.9)

As it says in the title, the new version won't load the new values again after disconnecting and reconnecting the trainer to the Wii U.

Closed and re-opened? Are you connected via JGecko U as well?

Nothing else is connected. I think it's a simple UI bug.
It happens after you connect and load values for the first time. If you click the disconnect button, then click the connect button again, the trainer won't offer you the ability to refresh.
It probably happened when you added the initial load button to the top of the trainer instead of reusing the Refresh button like the older versions. My guess is the disconnect button is disabling the Refresh UI element, then on reconnect the trainer isn't re-enabling the Load Data button, as it's already hidden itself after the initial click.
Probably a quick fix by just having the disconnect button unhide the top Load Data button.

OK, I will investigate. Thanks!

This should be fixed now


Default assets for ember do not seem to work

Not sure if we have yet to add them, or if it's just something that's broken. Didn't have time to do research.
I'm talking /icons/user_default_thumb.png, etc.

These were working fine for me, so that's surprising. Curious what the issue is.

Closed in favor of #252.


feat: add automatic_channel_upgrade option

Add option to enable automatic_channel_upgrade, not active by default.
Define a configurable maintenance window auto upgrades
@timdeluxe
Better than a ticket :)

Tested on a system. See the plan, because I am waiting for the new module
Terraform will perform the following actions:

  # module.kubernetes.azurerm_kubernetes_cluster.k8s will be updated in-place
  ~ resource "azurerm_kubernetes_cluster" "k8s" {
      + automatic_channel_upgrade           = "patch"
        id                                  = "/subscriptions/..."
        name                                = "..."
        tags                                = {}
        # (26 unchanged attributes hidden)

      + maintenance_window_auto_upgrade {
          + day_of_week = "Monday"
          + duration    = 4
          + frequency   = "Weekly"
          + interval    = 1
          + start_time  = "04:00"
          + utc_offset  = "+00:00"
        }

        # (6 unchanged blocks hidden)
    }

Plan: 0 to add, 1 to change, 0 to destroy.

Thanks a lot! 🎁


Extend documentation

What this PR does / why we need it:
This elaborates on the current documentation by:

Places all cluster setup information in one document that demos and the user guide can reference.
Also updates base README with Discovery Handler updates such as  including Discovery Handlers in the architecture diagram.
Creates separate documentation for discovery handler and broker development
Updates architecture, agent, and roadmap documentation with new Discovery Handler model information
Makes OPC UA demo documentation consistent with new Discovery Handler model

@bfjelds @romoh any thoughts on the docs?


Use joint motors without TypeScript

Hi, I have been trying to build a robot simulator with Rapier.js. Thanks for the great tool.
I was trying to use joint motors. Here is how to use them in the documentation:
let joint = world.createImpulseJoint(params, body1, body2);
(joint as RAPIER.PrismaticImpulseJoint).configureMotorVelocity(1.0, 0.5);

(joint as RAPIER.PrismaticImpulseJoint) seems like a TypeScript grammar, but my project is in vanilla JavaScript, so the webpack complains about the grammar.
I wonder whether there is way to use joint motors without TypeScript?
I tried to use joint.rawSet.jointConfigureMotorPosition, but I couldn't find RawJointAxis, which is the second parameter of that function.

this is something related to dev problem, you can just remove all the TS definition it will work.
for example
let joint = world.createImpulseJoint(params, body1, body2);
joint.configureMotorVelocity(1.0, 0.5);

Thanks, @jcyuan. It's still not working because of #68. I tried to follow #73 to fix it, but joint.configureMotorVelocity is still undefined. Hope the bug can be fixed soon.

#77  fixed


Update configure-prover.md - edit the prover file in the correct space

If I am looking into the docker-compose, then in no place, the vim ~/zkevm/config.json is used. Hence, I am pretty sure, that we want to edit the file that is use in the zkevm-prover of the docker-compose

Thank you for your contribution. We have added it to our TODO. We will come back to you with questions, if necessary, after review.

Waiting for a response from one of the sme's ... before resolving this one.

@josojo I believe we merged the latest on this so I'm going to suggest I close this PR in the next day or two. Let me know if you think differently. Thank you :)


Improve Azure Functions documentation regarding ILogger

The logging documentation here:
https://learn.microsoft.com/en-us/azure/azure-functions/configure-monitoring?tabs=v2
Has the following note:

For .NET class library functions, these categories assume you're using ILogger and not ILogger<T>. For more information, see the Functions ILogger documentation.

Then I go to this other link and all it has to say about ILogger<T> is:

If you choose to instead use an ILogger<T>, the category name may instead be based on T.

This isn't really helpful, what's this supposed to mean?
After some trial and error I figured out what this does is to use the fully qualified class name of T as the logging category (e.g. Foo.Bar.Baz). not only that but using a namespace as a category covers all classes under that namespace.
Please add such details in the documentation.

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

ID: 86aff194-f810-3c21-e322-cdda05dfc734
Version Independent ID: 0864707c-97a7-1166-1bd4-ef9ea212c64e
Content: Configure monitoring for Azure Functions
Content Source: articles/azure-functions/configure-monitoring.md
Service: azure-functions
GitHub Login: @ggailey777
Microsoft Alias: glenga

@mattchenderson can you please provide feedback on this issue? Also, I feel that this is in-proc guidance and may be different for isolated process C# functions.


Re: 'HTML coding style' typo @ line 131

Problem with 'HTML coding style' (https://gds-way.cloudapps.digital/manuals/programming-languages/html.html)
Hello I just noticed a typo on this page, here below the details:
@ line 131
"Links should not open new tabs or windows by default, but if users choose to to so (through keyboard[...]"
correct word is  do
regards
Paola

Thanks for pointing this out, @PaolaDMadd!


Fix a regression caused by circular override checking

Unfortunately the requestification of inheritsSuperclassInitializers (https://github.com/apple/swift/pull/27936) broke a case that can trigger recursive override checking via associated type inference.
This PR adds a hack to bail out on encountering this case.

@swift-ci please smoke test

Spoke with @CodaFi and we're going to leave this one be for now and just commit the test case.
@swift-ci please smoke test and merge


Potential Error Detected: regarding Stable / Latest Labels

I'm pretty sure I published e.g. this module see screenshot as stable...
hovering over stable shows hint - "No stable version available yet"

Question related to this: Is it a best practice to also publish the egg.json?
In this case it would proof that my egg.json looks like:
{
  "name": "distance-calculator",
  "description": "Calculating the distance between two places based on latitude and longitude geocoordinates",
  "stable": true,
  "files": [
    "distance-calculator.ts",
    "mod.ts",
    "README.md",
    "test-it.ts"
  ]
}

Hey Michael! Thanks for reaching out. This is a known bug with our current API unfortunately. We have a new one almost ready for deployment, and when that is deployed, this issue will no longer occur!
Also, regarding your egg.json question: It would not affect anything, so feel free to keep it or remove it :)


[Dropdown] Placeholder gets overridden in a Searchable Dropdown

Describe the bug
After setting searchable true for a dropdown, the placeholder is just Search.
To Reproduce

Expected behavior
Search keyword should be appended before the Placeholder or it should keep the Placeholder as is for us to control the text displayed there.

:tada: This issue has been resolved in version 1.3.8 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[FEATURE] Prediction/training on all available channels

Hello,
First of all, I'd like to thank you for the great work.
I'm highly interested in multimodality and information contained across more than one channel. In that spirit, I wanted to use the --all_channels option to train a custom Cellpose model on all the channels I provide, without restricting myself to only 1-cyto (and 1 optional nucleus channel). I passed the --all_channels tag to the CLI and everything seemed to work fine.
When I wanted to predict using this new model, on the other hand, I got stuck by an unfulfilled condition at line 504 of the cellpose.models.py file, in the eval function of the CellposeModel. From what I read of the source code, I can do inference with such a model if I pass model_type=None when loading the new model and channels=None when using the eval function.
I am not allowed to pass channels=None due to this condition, I get "TypeError: object of type 'NoneType' has no len()":
 channels=channels[i] if (len(channels)==len(x) and
            (isinstance(channels[i], list) or isinstance(channels[i], np.ndarray)) and
             len(channels[i])==2) else channels,

I manually erased the if condition and just passed "channels" and it seems to work. Do you think it would be possible to rework this line in a way that we can actually use Cellpose on any channels without modifying the source code?
Thank you very much for your attention,

@remyeltorro can you share a small complete example that illustrates the problem you're having? I'll be better able to help with the actual code at hand

@mrariden  thank you very much, I'll try to elaborate:
The idea: pass two or more 'cyto-like' channels to Cellpose, without performing RGB merging or other preprocessing. The info on how to segment the cells is on more than one channel and I want this as the input to the underlying UNET. I want to keep everything else the same.
Say I produced a model with the following CLI:
python -m cellpose --use_gpu --train --dir /home/limozin/Desktop/segment_RICM/annotations/dataset/train/augmented/ --test_dir /home/limozin/Desktop/segment_RICM/annotations/dataset/train/test/ --learning_rate 0.001 --weight_decay 0.0001 --n_epochs 5000 --mask_filter _labelled --verbose --no_norm --batch_size 8 --all_channels
The model detects the number of channels in my images and trains smoothly, I think it does what I intended, after tracking down in the source code what the --all_channels tag does.
Then if I want to load the model for prediction, it'll look something like this:
model = CellposeModel(gpu=use_gpu, pretrained_model=model_complete_path, diam_mean=30.0,
#model_type=None,
)
And I'll predict using:
Y_pred, _, _ = model.eval([f], diameter = diameter,
flow_threshold = flow_threshold,
normalize=False,
channels=None,
)
where f is a two channel image XYC (say shape 1004x1002x2). It seems, from the source code, that the only way to activate the "all_channels" model is to pass channels=None, but doing this will trigger the TypeError: object of type 'NoneType' has no len()"  on the line I indicated above.
After remove the if condition, and forcing the UNET-eval function triggered by the eval function above to pass channels=None, it seems to do what I intended, which is to use both of my channels inside the UNET to predict the target mask (and not decomposed as cyto+nucleus).
I don't know if it's clearer, maybe I misunderstood how to do what I intended. I there another way to achieve this without modifying the source code?
Thanks again,

@remyeltorro Based on the CLI command you used, it looks like you're training based off the cyto model because you did not use the --pretrained None flag. This must mean that your images have 2 channels since the --all_channels flag didn't produce an error and the cyto model is 2 channel. I say this because the --all_channels flag appears to be broken, at least for my data with 3 channels. But let me know if I'm off on this.
Still, if you are content with using 2 channels to segment, then CP should be capable of that. The built-in models assume that you have a cellular stain and a nuclei stain, but there isn't any strict dependence on that. Since the image channels get passed into the net together, they could mark arbitrary components of the cell provided they are sufficient for segmentation and you have enough training data. In other words, you can pass 2 channels into CP that both represent the cytoplasm and train on your masks. As CP is written now, you'll have to call one of the channels a 'nuclei' channel, but that is based on the pretrained model nomenclature.
Going to 3+ channels isn't really (well) supported, but it is something that we are discussing implementing/improving.

Thank you for the answer, you are totally right, I did not pay attention to the pretrained_model parameter on the CLI command. I also revisited the problem with 3-channel images to be sure not to take a default number of channels of 1 or 2. The new CLI command is as follow:
python -m cellpose --use_gpu --train --dir /home/limozin/Desktop/MCF7_nuclei_cellpose/dataset/train/augmented/ --test_dir /home/limozin/Desktop/MCF7_nuclei_cellpose/dataset/train/test/ --learning_rate 0.001 --weight_decay 0.0001 --n_epochs 5000 --mask_filter _labelled --verbose --no_norm --batch_size 8 --all_channels --pretrained_model None

The two relevant warnings I got are:
2023-08-09 10:23:35,699 [WARNING] channels is set to None, input must therefore have nchan channels (default is 2)

and right below
2023-08-09 10:23:36,000 [INFO] >>>> training network with 3 channel input <<<<

The model seems to train normally.  No error. Here's the full log:
2023-08-09 10:28:30,732 [INFO] WRITING LOG OUTPUT TO /home/limozin/.cellpose/run.log
2023-08-09 10:28:30,732 [INFO]
cellpose version: 	2.2.2
platform:       	linux
python version: 	3.11.3
torch version:  	2.0.0+cu118
2023-08-09 10:28:31,573 [INFO] ** TORCH CUDA version installed and working. **
2023-08-09 10:28:31,573 [INFO] >>>> using GPU
2023-08-09 10:28:31,950 [INFO] 152 / 152 images in /home/limozin/Desktop/MCF7_nuclei_cellpose/dataset/train/augmented/ folder have labels
2023-08-09 10:28:32,019 [INFO] 25 / 25 images in /home/limozin/Desktop/MCF7_nuclei_cellpose/dataset/train/test/ folder have labels
2023-08-09 10:28:32,019 [INFO] >>>> during training rescaling images to fixed diameter of 30.0 pixels
2023-08-09 10:28:34,645 [INFO] flows precomputed
2023-08-09 10:28:34,789 [INFO] flows precomputed
2023-08-09 10:28:34,822 [WARNING] 3 train images with number of masks less than min_train_masks (5), removing from train set
2023-08-09 10:28:34,822 [WARNING] channels is set to None, input must therefore have nchan channels (default is 2)
2023-08-09 10:28:35,152 [INFO] >>>> median diameter set to = 30
2023-08-09 10:28:35,152 [INFO] >>>> mean of training label mask diameters (saved to model) 42.566
2023-08-09 10:28:35,152 [INFO] >>>> training network with 3 channel input <<<<
2023-08-09 10:28:35,152 [INFO] >>>> LR: 0.00100, batch_size: 8, weight_decay: 0.00010
2023-08-09 10:28:35,152 [INFO] >>>> ntrain = 149, ntest = 25
2023-08-09 10:28:35,152 [INFO] >>>> nimg_per_epoch = 149
2023-08-09 10:28:40,522 [INFO] Epoch 0, Time  5.4s, Loss 2.1554, Loss Test 1.8972, LR 0.0000
2023-08-09 10:28:43,297 [INFO] saving network parameters to /home/limozin/Desktop/MCF7_nuclei_cellpose/dataset/train/augmented/models/cellpose_residual_on_style_on_concatenation_off_augmented_2023_08_09_10_28_34.822385
2023-08-09 10:28:54,703 [INFO] Epoch 5, Time 19.6s, Loss 1.8863, Loss Test 1.3957, LR 0.0006
2023-08-09 10:29:08,875 [INFO] Epoch 10, Time 33.7s, Loss 1.3083, Loss Test 1.0401, LR 0.0010
2023-08-09 10:29:37,171 [INFO] Epoch 20, Time 62.0s, Loss 0.9777, Loss Test 0.8064, LR 0.0010

I'll let this model train and then try the prediction step, where it was breaking earlier.

I applied this three-channel-input model to segment other three channel images. I could load the model using:
model = CellposeModel(gpu=True, model_type=None, pretrained_model="/home/limozin/Desktop/MCF7_nuclei_cellpose/dataset/train/augmented/models/cellpose_residual_on_style_on_concatenation_off_augmented_2023_08_09_10_28_34.822385",
                      diam_mean=30.0, nchan=3)

It was important to pass nchan=3 in order not to trigger the following error:
size mismatch for downsample.down.res_down_0.proj.1.weight: copying a param with shape torch.Size([32, 3, 1, 1]) from checkpoint, the shape in current model is torch.Size([32, 2, 1, 1]).

Then I loaded and applied the model to some images that are 3-channel, and normalized:
files = glob('/home/limozin/Desktop/MCF7_nuclei_cellpose/dataset/train/test/*.tif')
for f in files:
    if not f.endswith('_flows.tif') and not f.endswith('_labelled.tif'):
        stack = imread(f)
        stack = np.moveaxis(stack, 0, -1)
        stack = stack[:,:,[0,1,2]]
        print(stack.shape) #(512,512,3)
        Y_pred, _, _ = model.eval([stack], diameter = model.diam_labels, flow_threshold=0.4, channels=None, normalize=False)
        Y_pred = Y_pred[0]

I experimented removing channels in the input data to see if the model can still take it, and it breaks for data that is 2-channel, and returns empty labels for data that is 1-channel. So it seems to do what was intended.   Of course, all of this was done after removing the L504 if-condition, from my first comment.
If put back the original line 504, I can still load the 3-channel model. But I can't predict as the eval-method will break at this line 504 due to
object of type 'NoneType' has no len(), i.e. we're trying to evaluate the length of channels=None. So it seems to all come down to this little if-condition.
I suppose the 3 channel+ mode is almost working if it wasn't for the if statement on channels. Thanks again for investigating this question,

There appears to be a bug in the channel detection logic when using models that aren't the pretrained model size like you described. It does seem to work for me if you pass a single image in rather than a list of image. i.e.
import torch
import numpy as np

from cellpose.models import CellposeModel

img_chan = 5
img = np.random.random((256, 256, img_chan))
print(img.shape)

model = CellposeModel(gpu=True, model_type=None, pretrained_model=None, diam_mean=30.0, nchan=img_chan)

masks, flowp, style = model.eval(img, diameter = 30, flow_threshold=0.4, normalize=False, channels=None)

I've checked this with many different channel sizes. Hopefully this is a work around for you.

The passing a single image instead of a list works indeed, without triggering the L504 error. Thank you for helping me narrow down the combination of parameters that enables the 3+ multichannel feature.

Hi @carsen-stringer any update to this? Because on the CLI it is currently broken
Using the --all_channels flag for training a custom model works, but specifying --all_channels when using the CLI to use the custom model results in the same error. I saw you set this up properly for the training with
https://github.com/MouseLand/cellpose/blob/40a0c7d945b0cccc05c963cc8c349d9252bc0334/cellpose/__main__.py#L260
But this is not set here while you do have access to nchan with CellposeModel
https://github.com/MouseLand/cellpose/blob/40a0c7d945b0cccc05c963cc8c349d9252bc0334/cellpose/__main__.py#L157
So would it not be possible to set it here?
https://github.com/MouseLand/cellpose/blob/40a0c7d945b0cccc05c963cc8c349d9252bc0334/cellpose/__main__.py#L163-L164
I can offer a PR if you wish

considering that all channels is already available, this is a #bug and not a feature

thanks @lacan  I think it's been fixed, as long as I use images with the same number of channels as the model, e.g. I have RGB images in this folder and I'm running a three channel model:
python -m cellpose --dir images/ --all_channels --pretrained_model neurips_cellpose_default --verbose


Added eslint

Kind of wild that we don't have eslint set up. It would have somewhat caught #6951
Do not merge yet
This is mostly for discussion. Right now we're at::

✖ 414 problems (414 errors, 0 warnings)
183 errors and 0 warnings potentially fixable with the --fix option.

So if anyone is bored, feel free to set up eslint properly and fix all the issues :smile:

We used to have eslint, but apparently I removed it in https://github.com/mitmproxy/mitmproxy/commit/0e9206ceb11b1c4a3ea298e6b5bda54b9baf08ee as part of the transition to esbuild. IIRC it wasn't that easy to set up with TypeScript back then. Happy to bring it back.
Strategically, I'd like to wait until @lups2000's branch is in to avoid any merge conflicts. Let's prioritize getting that into main to unblock efforts like this one. :)

When adopting eslint, we should also run it with --fix as part of our autofixer. :)


Add Clojure support

Do it please. Clojure is a poggers language.

Do it.....

Thanks!

Is there any chance, that we can use repl?

this has been brought up before, it's a bit outside the scope of this project and it would be a fairly significant thing to add, if it could even be added at all into the existing piston framework. it's probably safe to assume this won't ever be a thing in this project for a while, if ever

alright, thanks!


Creating OUs under current organization in Python

Hi there,
Thanks for this library!
I am having issues with code like the following:
from constructs import Construct
from aws_cdk import (
    Stack,
    RemovalPolicy,
)
import pepperize_cdk_organizations as orgs

class MyRootStack(Stack):
    def __init__(self, scope: Construct, construct_id: str, **kwargs) -> None:
        super().__init__(scope, construct_id, **kwargs)
        self.environments = ["foo", "bar", "baz"]

        for env in self.environments:
            ou = orgs.OrganizationalUnit(self, env,
                organizational_unit_name=env,
                import_on_duplicate=True,
                removal_policy=RemovalPolicy.DESTROY,
                parent=Stack.of(self).account,                         # HERE
            )

Namely, I would like to create OUs under the current account—i.e.: the account to which the AWS credentials used to run cdk belong to. I'm missing a way to specify "this account" for the parent parameter. cdk deploy returns the following error:
Key 'parent': Unable to deserialize value as @pepperize/cdk-organizations.IParent

If I specify a string constant with the ARN or ID of the organization where I want the OUs to be created, I get the following error:
Key 'parent': Unable to deserialize value as @pepperize/cdk-organizations.IParent

I feel like this use case is frequent enough, that saying parent=None should do the trick, but would settle for a method returning the IAccount for the current account—perhaps it is there and I haven't been able to find it?
Thanks in advance.
-lem

Hey @nerdlem ,
in your mangement account obtain the Organization.root to use as the parent for the first OUs
https://github.com/pepperize/cdk-organizations#organization

Thank you. This was what I needed.
Best regards
-lem


Can't login "There was an error provisioning"

"missing paper key in login context
Error code 218: GENERIC in method keybase.1.login.login"
I can't login neither from the Mac or iOS.

log id: 7334f34d0f3b53096abc531c
the ≡ menu doesn't appear on my iOS, may be because I can't login yet.

You need to reset either way.

I need help with this same issue. Unfortunately I am do not understand how you said to fix it. I had it as an app on my phone and I am trying to get it onto my computer but I get that same message. Screenshot attached.


Add JSON outputs for QuantStudio test examples 4:8

The corresponding JSON outputs should be added for QuantStudio test examples 4 through 8

Resolved with PR #98


chore: add codecov badge to readme

Description

Type of change

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

How Has This Been Tested?

Test Configuration:

Node Version:
OS:

Checklist:

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

Additional context

Welcome to Codecov :tada:
Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.
Thanks for integrating Codecov - We've got you covered :open_umbrella:


No MaterialLocalizations found

Not able to set the supported locale. Also, when I try to set the language of Date Dialogs I also face the same issue. Someone mentioned to use flutter_localizations for providing the localizations delegate for material but this lib is discontinued and removed from pub.dev
Below is the minimal code for producing the issue.

GetMaterialApp(
      title: "Test App",
      translations: TranslationConfig(),
      supportedLocales: [TranslationConfig.french],
      locale: TranslationConfig.french,
      theme: ThemeData(
        primarySwatch: Colors.blue,
        fontFamily: "Roboto"
      ),
      home: const HomeScreen(),
    );

Flutter Version:
Flutter 3.3.3
Getx Version:
get: 4.6.5

Need to import as because it is in the flutter sdk, not on pub.dev
 flutter_localizations:
    sdk: flutter


🚀 Feature: Settings Page and code quality update

🔖 Feature description

add settings page like previous
Code refactor remove unused API routes
Performance enhancement as of now everything is fetched from server even static things and their is not optimization
add Optimistic UI updates

🎤 Why is this feature needed ?

Because of performance issues and bugs

✌ How do you aim to achieve this?
Code refactor
🔄 Additional Information
No response
👀 Have you spent some time to check if this feature request has been raised before?

[X] I checked and didn't find similar issue

Are you willing to submit PR?
Yes I am willing to submit a PR!

go ahead


[DOCS] Separate "interpreter exec hooks (experimental)" from interpreter overview page

What is this PR for?
After #1470 merged, "(Experimental) Interpreter Execution Hooks" is added under https://zeppelin.apache.org/docs/0.7.0-SNAPSHOT/manual/interpreters.html page. But i think we need to keep this page as simple as possible since it's explaining the basic concept of Zeppelin interpreters. So I separated  "(Experimental) Interpreter Execution Hooks" section from interpreters.md and create new page interpreterexechooks.md.
And also fixed some wired markdown rendering issues.
What type of PR is it?
Documentation
What is the Jira issue?
N/A
How should this be tested?
Please see below screenshots :)
Screenshots (if appropriate)

Before (under manual/interpreter.md

After

Questions:

Does the licenses files need update? no
Is there breaking changes for older versions? no
Does this needs documentation? no

@agoodm Since you added this section, it'll be appreciated if you can review this one :)

Thanks for the update! I probably should have done this when I made the original PR.
Anyways, LGTM.

@agoodm No worries. Thanks!

@AhyoungRyu Tested and LGTM!

@agoodm @soralee Thanks for your review!
CI failed only in Selenium test like below and it's not related with this docs change.
Results :

Tests in error:
  ZeppelinIT.testAngularDisplay:157->AbstractZeppelinIT.waitForText:84 Â» StaleElementReference

Tests run: 18, Failures: 0, Errors: 1, Skipped: 0

Merge into master if there are no further discussions :)


Can't connect to ip of container

I installed this script in Proxmox, it said it was successful and gave me the ip to login at.
When I go to the ip on my local network nothing loads. I can access the container through
the console of the proxmox webui so I can at least see that it is running. Is there something more
that I need to do. (I am a linux beginner if that wasn't obvious)

Can you check that sshd is installed/running?
Otherwise you can set it up yourself: https://wiki.alpinelinux.org/wiki/Setting_up_a_ssh-server

Can you check that sshd is installed/running?
Otherwise you can set it up yourself: https://wiki.alpinelinux.org/wiki/Setting_up_a_ssh-server
EDIT: I think I misunderstood your question, did you mean you cannot access the Nginx Proxy Manager web page? The admin console is on port 81.

Yeah sorry I cannot access the admin page through port 81, "unable to connect".

Can you check that sshd is installed/running?
Otherwise you can set it up yourself: https://wiki.alpinelinux.org/wiki/Setting_up_a_ssh-server
EDIT: I think I misunderstood your question, did you mean you cannot access the Nginx Proxy Manager web page? The admin console is on port 81.

I installed SSH, can't login because it wants credentials. But when I type rc-status should I see nginx listed?
All I see is sshd, cron , networking.

NPM might not be running, did the install script issue any errors?
Try "ps aux" and see if openresty is running

NPM might not be running, did the install script issue any errors?
Try "ps aux" and see if openresty is running

Didn't notice any errors

Just ran it again (the script) here is what I got and I cannot connect to this one either.
`[info] Updating LXC template list...
[info] Downloading LXC template...
[info] Allocating storage for LXC container...
[info] Creating LXC container...
[info] Setting up LXC container...
wget: bad address 'raw.githubusercontent.com'
[info] Successfully created Nginx Proxy Manager LXC 109.
Nginx Proxy Manager is reachable by going to the following URL.
  http://<IP_ADDRESS>:81`

You have an error: wget: bad address 'raw.githubusercontent.com'
Try creating the Alpine container manually using Proxmox, then run just the installation script.

You have an error: wget: bad address 'raw.githubusercontent.com'
Try creating the Alpine container manually using Proxmox, then run just the installation script.

Ok did that, here is what I get after installing curl and attempting the script
-ash: bash: not found

Sorry for the ignorance, used the script at the bottom of the page and she's up and running. Thanks for the help.

No problem @lightsout

@lightsout This network-related issue has been logged a few times without proper feedback, #1, #2.
wget: bad address 'raw.githubusercontent.com'
This indicates that the container cannot resolve raw.githubusercontent.com via your DNS server on the initial setup.
Do you mind giving some insight as to how your networking is setup?

Sorry for writing here, but I have the same issue.
fresh LXC install (tried both ubuntu 22 and Debian 10 template)
and the WGUI is not accessable via http:// <IP_ADDRESS>:81
I did as you wrote a new start of the script and received this error:
[success] Cleaning old files
[error] Installing dependencies
/tmp/npm_install.sh: line 74: 'apt-get install -y --no-install-recommends $DEVDEPS gnupg openssl ca-certificates apache2-utils logrotate'
Sub-process /usr/bin/dpkg returned an error code (1)


CI: Move objchelloworld to use Engflow's remote execution

Description:
Move objchelloworld to use Engflow's remote execution
TODO: Add the results once the action runs
Risk Level: Low
Testing: Manually tested
Docs Changes: N/A
Release Notes: N/A
Signed-off-by: Luis Fernando Pino Duque<EMAIL_ADDRESS>

/cc @ulfjack @goaway


Could support DRAMAtical Murder (Steam Version)

Hi marcus,I used your tools to get the files from npk in  DRAMAtical Murder (Steam Version), but it didn't support it natively.
Can add a native support with it?
I already use SRLInjector to get the Code.

I sent you the file. Thank you!

Ok

added

Thank you!


Port from CUDA to OpenCL

Hello there,
I saw this very interesting and innovative software, but apparently it is limited to the proprietary nvidia cuda sdk.
Porting to OpenCL would enable everyone to run the software, which is very important if you want the software to gain traction and be use by a lot of researchers/users.
I know this will involve a lot of work. However, I am convinced using an open standard will benefit the project enormously in the long term.
You cannot simply ignore 20% of your potential users.

What about Linux then?

What about it? OpenCL is supported on all videocards under Linux as far as I can see.

Hi, we don't plan to switch to OpenCL or to add support for GNU/Linux in the near future.

Using a proprietary sdk will only hurt the project in the future, you are basically at the whims of Nvidia. Not to mention that if a lot of software would come to demand CUDA functionality, we would be giving Nvidia a monopoly. This is one of the many reasons why open APIs are better in the long run.

@cornytrace No one uses Open APIs in Deep Learning. You should ask it to Tensor Flow as well.


Cannot (really) control Roomba 980

Hi!
I'm working with the current script:
import asyncio
from roomba import Roomba

async def test():
        await roomba.connect()
        roomba.send_command('start')
        await asyncio.sleep(5)
        roomba.send_command('stop')
        roomba.send_command('dock')
        await roomba.disconnect()

roomba = Roomba('<IP_ADDRESS>')
loop = asyncio.get_event_loop()
loop.run_until_complete(test())

This indeed starts the Roomba 980 at <IP_ADDRESS>, but that's the last thing that works. The vacuum never stops or docks as it should. The output of this script is as follows:
Pending event queue size is: 1
Pending event queue size is: 1
/home/user/src/Roomba980-Python/roomba/roomba.py:607: RuntimeWarning: coroutine 'Roomba._disconnect' was never awaited
  self.loop.create_task(self._disconnect())
RuntimeWarning: Enable tracemalloc to get the object allocation traceback
Task exception was never retrieved
future: <Task finished name='Task-26' coro=<Roomba._disconnect() done, defined at /home/user/src/Roomba980-Python/roomba/roomba.py:609> exception=AttributeError("type object '_asyncio.Task' has no attribute 'all_tasks'")>
Traceback (most recent call last):
  File "/home/user/src/Roomba980-Python/roomba/roomba.py", line 612, in _disconnect
    tasks = [t for t in asyncio.Task.all_tasks() if t is not asyncio.Task.current_task()]
                        ^^^^^^^^^^^^^^^^^^^^^^
AttributeError: type object '_asyncio.Task' has no attribute 'all_tasks'
Traceback (most recent call last):
  File "/home/user/src/Roomba980-Python/roomba/./test.py", line 16, in <module>
    loop.run_until_complete(test())
  File "/usr/lib/python3.12/asyncio/base_events.py", line 687, in run_until_complete
    return future.result()
           ^^^^^^^^^^^^^^^
  File "/home/user/src/Roomba980-Python/roomba/./test.py", line 12, in test
    await roomba.disconnect()
TypeError: object NoneType can't be used in 'await' expression

You don’t seem to know how asyncio works.
roomba.connect() and roomba.disconnect() are not async methods.


Reimplement hmm_posterior_sample

Description
Fix bugs in hmm_posterior_sample and verify correctness using unit test created as test_hmm_posterior_sample.
Detail
Since hmm_posterior_sample returns a random output, there seemed to be no straightforward way to implement its unit test.
Initially, I considered computing the probabilities of each of the possible state sequences, running hmm_posterior_sample many times and comparing the sampled frequencies across the possible sequences with the theoretical probabilities.
This seemed needlessly complicated, and the frequencies tended not to uniformly converge to the theoretical values when performing a reasonable number of random samples.
In the end, I settled on the approach of computing the mode sequence of the generated random samples and comparing it to the posterior mode computed by Viterbi algorithm (using hmm_posterior_mode).
Issue
#8
Checklist:

[x] Performed a self-review of the code

Couldn't you use big_log_joint for say T=3 and K=2, which defines a 222-dim joint state space, and sample from this naively and compare marginals to hmm_posterior_sample?

@murphyk That's a good idea, I'll try that!


tests: Add policy testing support

Note: first four commits are from another PR (#128) that this PR depends on.
Policy can be installed and removed as a test step between other
tests, or applied to a single test with WithPolicy(). Go syntax for
WithPolicy() use requires scoping address-of operator, which is not
ideal.
Policy is expressed as yaml and must specify the namespace in which
the policy is to be applied. Same test cases can be repeated with different
policies and corresponding egress/ingress enforcement expectations.
Use Go 1.16 //go:embed to include policy YAMLs as strings to Go source
code. This makes test authoring mush easier.
cilium connectivity test now has --verbose (or -v) mode that prints additional diagnostic information such as test command output and policy revision information.
curl output is formatted to show the source and destination addresses and ports as well as the HTTP return code.
ping has been optimized to exit after first successful ping with the timeout (-w) option.
Design considerations:

Currently policies are expressed as stand-alone yaml, parsed in to CiliumNetworkPolicies. Policies need to specify in which namespace they apply and this must match the namespace in which the tests are executed. This, however, is a configurable parameter, and may need to be overridden after the YAMLs have been parsed.

More complicated policy tests may require use of multiple namespaces, but the current test setup with a single configurable test namespace seems ill-suited for that.

Infrastructure TODOs:

[ ] make sure multi-cluster testing is not broken by this

[ ] Apply testing policies in all appropriate clusters

[x] Finalize policy apply/delete infra within the test suite (i.e., between or with the tests)
[x] Add support for different expected test outcomes (currently drops are never expected)

[x] Allow the same test case be run against different policies with different expected outcomes

[ ] Add support for CiliumClusterwideNetworkPolicies
[ ] Add support for k8s NetworkPolicies

Policy test cases needed (Once the infra is in place adding these should not need any new Go code):

[x] L3 policy ingress
[x] L3 policy egress
[ ] L3/L4 policy ingress
[x] L3/L4 policy egress
[ ] toEntities/fromEntities
[ ] toEntities/fromEntities
[ ] CIDR
[x] HTTP (adds use of Envoy)
[ ] Kafka (adds use of proxylib with Envoy)
[x] toFQDNs
[ ] L3-dependent L7 with HTTP
[ ] L3-dependent L7 with toFQDNs with HTTP

Encryption test in the GH Kind run fails due to an apparent bpf complexity issue, Cilium agent logs:
2021-04-14T07:42:21.113832600Z level=error msg="Command execution failed" cmd="[tc filter replace dev lxc5782f00f5fb1 ingress prio 1 handle 1 bpf da obj 3081_next/bpf_lxc.o sec from-container]" error="exit status 1" subsys=datapath-loader
2021-04-14T07:42:21.114395307Z level=warning msg="Log buffer too small to dump verifier log 16777215 bytes (10 tries)!" subsys=datapath-loader
2021-04-14T07:42:21.114406107Z level=warning msg="JIT disabled, but 4/4 tail call maps in the program have JITed owner!" subsys=datapath-loader
2021-04-14T07:42:21.114410807Z level=warning msg="Error fetching program/map!"  containerID=394314095a datapathPolicyRevision=0 desiredPolicyRevision=1 endpointID=3081 file-path=3081_next_fail identity=27403 ipv4=<IP_ADDRESS> ipv6= k8sPodName=cilium-test/echo-same-node-66589c9569-vmr6p subsys=endpoint
2021-04-14T07:42:21.126640757Z level=warning msg="Regeneration of endpoint failed" bpfCompilation=1.295833427s bpfLoadProg=44.693748346s bpfWaitForELF=1.296197031s bpfWriteELF="168.802µs" containerID=394314095a datapathPolicyRevision=0 desiredPolicyRevision=1 endpointID=3081 error="Failed to load tc filter: exit status 1" identity=27403 ipv4=<IP_ADDRESS> ipv6= k8sPodName=cilium-test/echo-same-node-66589c9569-vmr6p mapSync="33.5µs" policyCalculation="37.2µs" prepareBuild=2.061726ms proxyConfiguration="14.6µs" proxyPolicyCalculation="24.001µs" proxyWaitForAck=0s reason="syncing state to host" subsys=endpoint total=45.993888926s waitingForCTClean=400ns waitingForLock="13.4µs"
2021-04-14T07:42:21.126825959Z level=error msg="endpoint regeneration failed" containerID=394314095a datapathPolicyRevision=0 desiredPolicyRevision=1 endpointID=3081 error="Failed to load tc filter: exit status 1" identity=27403 ipv4=<IP_ADDRESS> ipv6= k8sPodName=cilium-test/echo-same-node-66589c9569-vmr6p subsys=endpoint
2021-04-14T07:42:21.126904060Z level=warning msg="Failed while regenerating endpoint" endpointID=3081 subsys=endpoint

Seems similar to https://github.com/cilium/cilium/issues/14784

Encryption test passed with kube-proxy-replacement enabled.

Pushed a minor cleanup.

Added TODOs for multi-cluster policy testing.

@jrajahalme Let's start merging this in and then complete the rest.

Created an issue here for a small shortcoming I spotted during testing: https://github.com/cilium/cilium-cli/issues/174.
Ran into a few minor things today, will comment inline, but these can also be addressed later.

Thanks for the review @ti-mo! I did the fixes you proposed, with those this should be good to merge.


希望可以一个窗口打开多个项目

VSCode Version: Code 1.12.1 (f6868fce3eeb16663840eb82123369dec6077a9b, 2017-05-04T21:18:32.269Z)
OS Version: Darwin x64 16.5.0
Extensions:

Extension
Author
Version

php-debug
felixfbecker
1.10.0

php-intellisense
felixfbecker
1.2.1

auto-rename-tag
formulahendry
0.0.11

composer
ikappas
0.5.0

vscode-JS-CSS-HTML-formatter
lonefy
0.2.3

Go
lukehoban
0.6.59

Theme-MarkdownKit
ms-vscode
0.1.4

cpptools
ms-vscode
0.11.0

vscode-icons
robertohuertasm
7.6.0

Steps to Reproduce:

Translation of the issue's title: I hope to open multiple projects in one window.
Comment: I guess it is designed by purpose such that one can open at most one project in one window.. In Visual Studio it is the same case.

Duplicate of #396


fix configuration access for grid

This PR addresses #11300

Coverage increased (+0.01%) to 35.87% when pulling a49b3acd0e77bad4fec1d1b866270b5875ba3695 on bug/resultsSCrolling into 0835d3ed0f831f815dd0e50e3b63b93e6125519c on main.


new term request - 2'-deoxyuridine 3'-monophosphate(2-)

2'-deoxyuridine 3'-monophosphate(2-) conjugate base of CHEBI:46322 - 2'-deoxyuridine 3'-monophosphate, following the pattern for CHEBI:17622 - dUMP and CHEBI:246422 - dUMP(2−)

A ChEBI entry has now been created for 2'-deoxyuridine 3'-monophosphate(2−) = CHEBI:176905.
The entry will be visible on the ChEBI website by tomorrow.


Add support for error unwrapping

With this PR, any instances of errors.Error which have an associated cause will implement the method Unwrap() error.  Errors with no cause will not implement this method.  This means that errors.Is and errors.As will work on the errors produced by this package in the same way they would for errors produced by fmt.Errorf or errors.New.  For concrete examples, see the new test, TestUnwrapping.
The public API is completely unchanged, so this can serve as a drop-in replacement.

For justification on updating this API vs just using the standard library's error wrapping API (fmt.Errorf("something happened: %w", err)), see this comment

The diff is a little hard to parse, so here is a little introduction: I split the concrete error type into two types: baseError and wrappingError.  The latter embeds the former, so a wrappingError is also a baseError, it just has a cause attached.
Both New and Wrap can return an error of either type, depending on whether a cause is found in the input.  Of course, callers will only see the Error interface, so it all looks the same to them.
I made only minor changes to the tests to ensure that API behavior is unchanged.  I believe the only changes I made were:

switches from %v to %w in fmt.Errorf calls
changes to type-checking to accommodate the now-split concrete type

Thanks @oxtoacart!


Code of conduct

I have merely copied a Code of Conduct documentation which is widely popular in the open source space. Please feel free to have a look at it and suggest modifications, additions, fixes!
code_of_conduct.md

Closing for now.


New Duplicate Tab with CWD should be enabled by default.. as in the .profile

New Duplicate Tab with CWD should be enabled by default.., as in the .profile
Maybe map the usual Crtl + Alt + t as well :)
Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: 439baddf-3ec9-a46e-1b6e-3ec47cbd1d4c
Version Independent ID: 439baddf-3ec9-a46e-1b6e-3ec47cbd1d4c
Content: Opening a tab/pane in the same directory
Content Source: TerminalDocs/tutorials/new-tab-same-directory.md
Product: dev-environment
Technology: windows-terminal
GitHub Login: @zadjii-msft
Microsoft Alias: migrie

As much as I would love this, it's very much a Hard problem. It's non-trivial to update users' shell configs for them in a non-destructive manor.
We're still trying to work out easier ways for users to enable this, which is more-or-less tracked in https://github.com/microsoft/terminal/issues/13445.
Thanks!


StandardMaterialDefines methods slow in V8

Hi,
This is a Chrome/v8-specific issue, but each of the methods in StandardMaterialDefines currently run very slow in v8 due to the for..in loops, and the way optimization works in Chrome. (Basically, when v8 sees a for..in loop, in most cases it deoptimizes the whole function.)

Could these methods be changed to loop over Object.keys(this), or a predefined array of keys, instead?
(I apologize for not sending a PR, but I don't have typescript compilation set up - sorry!)

Will do..Chrome is definitely getting better :)


Add some documentations and example usages about two files in scripts folder

Hi, I really like this repo since it offers a simple yet powerful toolkit to reproduce the existing models and conduct some new experiments.
However, I'm curious about the two files in scripts folder (i.e. hn_mining.py and reduce_results.py) when I browse the source code because they are isolated and not being used by any of other files in this repo. So could you please add some documentation and example usages about these two files to make us better understand how to use them? Thank you in advance!

adding @xueguangma for triage


Overlap on Sticky

I have created a SwipeableUltimateViewAdapter, and have followed the usual steps to create sticky headers.
When I do this, the header overlaps on the first item below it. I have attached a screenshot, it is quite hard to see where the header view starts, but if you look at the shadow it is easier to see.

Do you mean you want to customize your header of the item?


fix: IDC windows机器开通前置策略 (closed #2301)

PR
CheckList（PR Assigners）
确认已完成以下操作：

[ ] 已将分支与当前合入分支同步
[ ] 已在关联 Issue 中补充「功能自测」部分内容
[ ] 已更新关联 Issue 的状态 backlog -> todo -> for test -> tested（需通过测试同学测试） -> done
[ ] 将 PR 添加上 pr/reviewable

CheckList（PR Reviewers）
确认已完成以下操作：

[ ] 检查关联 Issue 「功能自测」部分内容
[ ] 必要时打回 pr/reviewable，要求重新修改
[ ] 及时 Approve

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.v_gqpgguo seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

跳板机<IP_ADDRESS>，windows idc机器，shell方案(五步，原有四步，新增第二步)

跳板机<IP_ADDRESS>，windows idc机器，batch方案(四步，原有三步，新增第二步)

将第二步生成的指令在windows机器执行


Typo in documentation- Mob.java

Not really an "Issue" but still something that is bothering me, found here:
public final List<Player> getLocalPlayerList() {

Fixed, thanks.


Events page needs content update

The events page talks about okfestival in Berlin as upcoming. We need to update that as past and add details of upcoming ones as well.

Also the two calendar links on the left are ...a bit lame.  1st one is a link to calendar containing... no events.  I found another calendar on here which is less empty: http://okfnlabs.org/events/   .....maybe merge and have one calendar? The 2nd one is a link to wiki heading which no longer exists (should be http://wiki.okfn.org/Events  ?  Anyone updating anything there?

Moving events guide to handbook. Will open this page when the new OKFest is on


Semodel not showing up

this just recently started happening

where did you get the file? I can't seem to find it.


Remove the compiler and VM

Why?

Lambdas are not supported by the compiler
Only marginally faster (if at all) than the interpreter
More brittle and harder to work on

See also: https://github.com/jbboehr/php-mustache/issues/49
Closes #49

Closed by dfa8a3b


fix injection error in bundle

https://github.com/microsoft/ApplicationInsights-node.js/issues/938


boost_1_58 related build error..

Ubuntu 14.04 build error.. After a fresh checkout and boost upgrade this the start of a long list of errors.  Seems to be a known issue in boost
http://lists.boost.org/Archives/boost/2015/02/220149.php
In file included from /home/jcalfee/opt/boost_1_58_0/boost/context/all.hpp:13:0,
                 from /home/jcalfee/bitshares/graphene/libraries/fc/src/thread/context.hpp:3,
                 from /home/jcalfee/bitshares/graphene/libraries/fc/src/thread/thread_d.hpp:5,
                 from /home/jcalfee/bitshares/graphene/libraries/fc/src/thread/thread.cpp:5:
/home/jcalfee/opt/boost_1_58_0/boost/context/execution_context.hpp:202:52: error: ‘std::index_sequence’ has not been declared
                                               std::index_sequence< I ... >) {
...

http://lists.boost.org/boost-users/2015/04/84110.php
I think you can fix by adding this to the build flags:
-DBOOST_NO_CXX14_INTEGER_SEQUENCE

An equals was required to make it valid: -DBOOST_NO_CXX14_INTEGER_SEQUENCE=
I don't think it is in this version of boost (empty: egrep BOOST_NO_CXX14_INTEGER_SEQUENCE ~/opt/boost_1_58_0/ -r).  Also:
CMake Warning:
  Manually-specified variables were not used by the project:

    BOOST_NO_CXX14_INTEGER_SEQUENCE
``
same build error

https://github.com/boostorg/config/pull/58

Ran into this somewhere too, ended up changing indexed_sequence<...> to integer_sequence<size_t, ...> as suggested in one of the threads on the mailing list to work around it.  That might be simpler than upgrading gcc, which is probably the right solution.

The error is a little different: index_sequence< I ... > .. what would you translate that to?

whoops, I typed it wrong from memory, I meant that index_sequence<T> is supposed to be the same as integer_sequence<size_t, T>, a search and replace of index_sequence< for integer_sequence<size_t, should give you something that works, as long as your compiler knows what an integer sequence is...

Are you all actually editing the boost source code to get this to build?

Have not...  What do you recommend?

Fixed... The gcc route was easy.  I did need to do a fresh clone (clean up old cmake file).  I'll add a few notes to the wiki.  I'll PM theoretical about updating the wiki.
Install gcc-4.9
CC=gcc-4.9 CXX=g++-4.9 cmake -DBoost_NO_SYSTEM_PATHS="TRUE" -DBOOST_ROOT="~/opt/boost_1_58_0" -DCMAKE_BUILD_TYPE=Debug


Only for runtime.

So I've realized, that if I make a package (f.e. rocket:module) that depends on these functions, it won't work if that package is using MeteorFilesHelpers during build stage (before deploying to Atmosphere for example) because then there's no way to use these helpers in a given application to be able to define build steps based on features found in an application.
In rocket:module, I was developing with local packages at first inside a test application, and that worked fine because rocket:module's build plugin would exist inside an app, and these helpers would work. But moving rocket:module to it's own folder outside of any application then publishing doesn't work because there's no reference application when the build plugin runs.
It seems like I will now have to move all this logic into the runtime environment and integrate with the application build steps somehow, but I'm not sure how yet.
Any ideas?

Is there a way how you can determine the app path from the build plugin? If yes, I can extend this package to allow to specify a custom app path as option.

That's the problem, there's no way to do that in a build plugin unless the plugin is running in a locally installed package. If a build plugin could run for a package during app-build-time, that'd work out nicely.

Closing since there's no need to know things like the app path during publish builds. Plus, the new Plugin.registerCompiler API runs during app builds, so all these helpers work great.


Re-emitting ready/error event handlers may result in excess firing for any/all handlers

We have a special hook in the core ZeroClipboard.on and client ZeroClipboard.prototype.on methods that will re-emit certain important events ("ready" and most "error" types) in case the event handlers are not added in time to catch those events during their original emission:

src/js/core/private.js#L113-141
src/js/client/private.js#L77-106

However, I am now just for the first time seeing first-hand that this can result in several problems:

[ ] At the core ZeroClipboard.on level, the re-emitting is achieved by calling ZeroClipboard.emit, which does a global broadcast, ergo triggering other event handlers [which may have already received that event originally/previously]. Ideally, we should just be constructing an event object and invoking only the newly added event handlers with it.
[ ] At the client ZeroClipboard.prototype.on level, the re-emitting is achieved by calling this.emit, which does a client-wide broadcast, ergo triggering any other existing event handlers that the client may have for that event [which may have already received that event originally/previously].  While I don't believe this scenario is common, it would certainly be unpleasant to experience and try to make sense of it.
[ ] If a client instance were to add multiple handlers for the same re-emitting event (e.g. "error") after that event has already occurred, each handler will end up being invoked at least as many times as the number of unique error handlers.
[ ] As most of our events (including all of the re-emitting events) are dispatched asynchronously, a given event may have already been dispatched before the event handler was added but the event handlers will not have been executed yet. However, if that event is one of the re-emitting events, it will be received and handled by that new event handler at least twice.

All of these issues would go away if we removed the re-emitting code.  However, then most users will end up with critically missed events. So... we need to work through these issues soon.

The complexity of actually implementing this fix has me reconsidering the idea of just dropping the dropping the re-emitting of critical events after all.... :-1:

Either way, I'm deferring this.


installed but black screen

installed but after going to config file and adding the placement top_left i just get black screen when running magicmirror

Kindly share your log.

same issue for me...


how to train with this net and my data?

how to train with this net and my data? Can you give me the example code? Thanks

There are several available tutorials on transfer learning using Keras applications. Please refer to them directly. All concepts there will apply here.

Thanks. I have trained with b5. so can i train with b6,b7 now?

From scratch? Sure if you want to. However, they don't have pretrained weights so performance might be quite poor.

I trained b5 with the input size of 300x300. As far as I understand, your code will load the weight of b3. So why don't we use the weight of b5 for b7?

B5 and B7 don't have the same hyperparameters, so I can't load B5 weights for B7 model.

Oh, so, I got it. Thank you for your support


Add RwLock and RwLockTest

What is RwLock?
RwLock may either have many readers or a single writer at one time. At any moment in time a RwLock is in a certain mode either reading or writing.
How is RwLock implemented?

reading: $this->readerCount > 0
writing: $this->isWriting === true
released: $this->isWriting === false && $this->readerCount === 0
invalid (This should never happen): $this->isWriting && $this->readerCount > 0`

When the RwLock is in reading mode, calls to $lock->read() will run immediately while $lock->write() will wait for the next writing opportunity. When the RwLock is in writing mode, calls to $lock->read() and $lock->write() will wait for the writer to finish. When the RwLock is released (no current readers or writer), a call to $lock->read() or $lock->write() will happen immediately.

Also please rebase master, I just added GitHub CI to run tests.

Codecov Report

Merging #2 (316d41f) into master (352da43) will increase coverage by 29.56%.
The diff coverage is 76.92%.

@@              Coverage Diff              @@
##             master       #2       +/-   ##
=============================================
+ Coverage     19.14%   48.71%   +29.56%
+ Complexity       32       29        -3
=============================================
  Files             4        4
  Lines            94       78       -16
=============================================
+ Hits             18       38       +20
+ Misses           76       40       -36

Impacted Files
Coverage Δ

src/SOFe/RwLock/RwLock.php
76.92% <76.92%> (+76.92%)
:arrow_up:

Continue to review full report at Codecov.

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


Feature/200366 Redirecting BLiP Chat Url in iframe Widget

Redirecting BLiP Chat Url if the bot belongs to a tenant and the current URL doesn't starts with the tenant id

A new PR was open to add this functionallity


fix(sqlite): support all sqlite pragma statement

What was changed

I removed the extra underscore in the key
I removed the pragma key check

Why?

the extra underscore was breaking the pragma support, because sqlite couldn't recognise the pragma key
the pragma key check was too restricting, users should be able to configure any pragma statement from https://www.sqlite.org/pragma.html - removing the check all to use any pragma, without adding more work to maintain an hardcoded list

Checklist

Closes https://github.com/temporalio/cli/issues/681

How was this tested:

manually:

by running ./temporal server start-dev --db-filename temporal.sqlite --sqlite-pragma journal_mode=WAL --sqlite-pragma synchronous=NORMAL --sqlite-pragma busy_timeout=5000
and ensuring that 3 files have been created: temporal.sqlite, temporal.sqlite-shm and temporal.sqlite-wal

Any docs updates needed?

no, because it's just a bug fix. the docs are already there.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

@vbehar - We are curious, for our own tuning reasons, what are your use cases and what different pragmas might you want to set? We are also wanting to tune some defaults, do you think there are opportunities here for us to improve defaults?

@vbehar - We are curious, for our own tuning reasons, what are your use cases and what different pragmas might you want to set? Any reason we shouldn't continue to limit the pragmas we accept? We are also wanting to tune some defaults, do you think there are opportunities here for us to improve defaults?

yes, our use-case is that we're still running temporal in dev-server mode with sqlite until we properly deploy it - in a more "production-ready" setup. but as it's a shared setup (not on our own laptop), we want to make backups - but we had conflicts between our backup script and temporal.
we quickly found out that enabling the WAL journal is a good start, as well as setting the busy_timeout - cf https://litestream.io/tips/ for example which explains these different settings.
and that's when we discovered that the sqlite-pragma flag doesn't work, and is too restrictive.
so we want to run temporal with the following sqlite pragma:

journal_mode=WAL
synchronous=NORMAL
busy_timeout=5000

Thanks!


Added functionality to attach parameters to the profile response if an id is sent

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/incubator-trafficcontrol-PR/1350/
Test PASSed.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/incubator-trafficcontrol-PR/1351/
Test FAILed.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/incubator-trafficcontrol-PR/1352/
Test FAILed.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/incubator-trafficcontrol-PR/1354/
Test FAILed.


added new button

Added new amazing button
CSS only
animated wave background

The button seems great, Their are few issues though

Center the button
There are useless scrollbars showing up on your buttons, will be fixed by setting root margin and padding to 0
Fix These issues and tag me once done I'll be happy to merge your PR then


Mismatching metrics on canvas and sparklines

It may be justified by some transformation made by the sparklines, but it should be consistent on the canvas, otherwise it's very confusing for the end user.

Could be caused by different polling intervals?

MoC are sent via websockets every second, while the sparklines are updated with the details panel (every 30s).

MoC are sent via websockets every second, while the sparklines are updated with the details panel (every 30s).

Do they come from different data sources or just different polling intervals? Can we use the same data source/interval the metrics in the details panel in MoC?

Yeah, this needs reconciling the UI really.

Switch to details data source when available.


Add StakeValidatorHash to MustPayToOtherScript constraint

Should fix #424.

Pre-submit checklist:

Branch

[x] Tests are provided (if possible)
[x] Commit sequence broadly makes sense
[x] Key commits have useful messages
[ ] Relevant tickets are mentioned in commit messages
[ ] Formatting, materialized Nix files, PNG optimization, etc. are updated

PR

[x] Self-reviewed the diff
[x] Useful pull request description
[ ] Reviewer requested

I think it would be nice to test it properly to make sure that it works but I have no idea how to do it yet.

I think it would be nice to test it properly to make sure that it works but I have no idea how to do it yet.

We'll plan a testing strategy for the tx library when we rewrite it


Bot does not recognize command of form /command@usernamebot

Currently, creating a command handler (bot.command) recognizes commands like /start, but not /start@usernamebot. In supergroups, when there are multiple bots, if you click on a command provided by the bot, the Telegram client will automatically send a message with the command, followed by the bot username (like shown above).

Hey, thanks for raising this. Yeah, this is indeed a bug. Surely fix that shortly.
Thanks again :)

Thank you so much.
Also, a nitpick in Readme, in the Thank you section
Televerse is a powerful and easy-to-use library for building Telegram bots in TypeScript.
Should be Dart instead of TypeScript

Thank you so much. Also, a nitpick in Readme, in the Thank you section Televerse is a powerful and easy-to-use library for building Telegram bots in TypeScript. Should be Dart instead of TypeScript

Yeah yeah, I'm aware of that. Already fixed that on c9dd6fc3d7122fb4d5883efb976c3549d64efaa9. Of course, some GPT things 😬
Once again, thank you so much for reporting and this is all good on v1.9.9 :)


Fix crash when toggling flag property value

When using PROPERTY_HINT_FLAGS, the hint string is a comma separated list of field names. An empty name can be used to skip the corresponding field.
EditorPropertyFlags uses the field index to index the CheckButton controls, so the editor crashes if there are fields skipped. Changing it to the CheckButton's index fixes the crash.
To reproduce the crash, attach the following script to a Node2D node, and try to check "Option5" or "Option6".
@tool
extends Node2D

var flags: int

func _get_property_list():
	return [
		{
			"name": "flags",
			"type": TYPE_INT,
			"hint": PROPERTY_HINT_FLAGS,
			"hint_string": "Option1,,,,,Option5,Option6",
		}
	]

I think a comment could be added in the code, it's not really obvious why it's using flags.size() instead of i here.

Refactored a bit so it's easier to add comments.

Thanks!


Documentation for global color variables

Documentation for global color variables
Summary
As noticed in a PR to our developer website, we don't have the best documentation surrounding our global color variables. I wonder if we would be able to link to our color guide / style guide so contributors creating new components are able to reference them!

Here's some context on the comment: https://github.com/newrelic/developer-website/pull/675/files/700ca4b356d9abc1480d4fade3aff87cda86da59#diff-faacae4e5e261a5839e04cc5a76f9467


Why does the abolition of version 5.0 ?

Sorry, my English is not good.
I want to know why the 5.0 version was canceled.
I did not find the relevant instructions in the official document, so come here to ask.
My English level is not OK, I do not know if you can understand the meaning of the expression.

Please see https://github.com/netty/netty/issues/4466


help with multi-grid

i can't set up multi-grid from cluster-name
my config is:
labels:
  order:
    - instance
    - job
  color:
    static:
      - job
    unique:
      - cluster
      - instance
      - "@receiver"
      - "@cluster"
    custom:
      "@cluster":
        - value: eu-do
          color: "#70e1e1"
        - value: single
          color: "#ff6347"
      region:
        - value_re: ".*"
          color: "#736598"
      severity:
        - value: info
          color: "#87c4e0"
        - value: warning
          color: "#ffae42"
        - value: critical
          color: "#ff220c"
  valueOnly:
    - alertname
    - severity
    - job
    - region
    - "@cluster"
grid:
  sorting:
    order: label
    reverse: false
    label: cluster
    customValues:
      labels:
        cluster:
          ny-do: 1
          eu-do: 2
          ams-do: 3
        #severity:
          #critical: 1
          #warning: 2
          #info: 3
  auto:
    order:
      #- severity
      - cluster
listen:
  address: "<IP_ADDRESS>"
  port: 8080
  prefix: /
log:
  config: false
  level: info
receivers:
  keep: []
  strip: []
ui:
  refresh: 10s
  hideFiltersWhenIdle: true
  colorTitlebar: false
  theme: auto
  animations: true
  minimalGroupWidth: 420
  alertsPerGroup: 5
  collapseGroups: collapsedOnMobile
  multiGridSortReverse: false

but all messages set in one line and one grid

Did you configure it in the UI?
You don’t have “ui: multiGridLabel: cluster” in your config file.
Example: https://github.com/prymitive/karma/blob/main/demo/karma.yaml#L135

Did you configure it in the UI?
You don’t have “ui: multiGridLabel: cluster” in your config file.
Example: https://github.com/prymitive/karma/blob/main/demo/karma.yaml#L135

i added its in config, but not result.
BUT, i change multiGridSortReverse: false to multiGridSortReverse: true
worked!


yugabyte symlinks fail to resolve

Tried brew tap yugabyte/yugabyte and brew install yugabyte. Got an error with it trying to symlink psql and failing becuase it already exists. However it should not be trying to symlink psql at all. brew install yugabyte-cli works though. It is probably the same issue as #4
https://gist.github.com/WesleyW/d232694ac6bb4dc2d94638fb8e5349ef

Fixed by 1ce5a2814dab94b42dddb2d73b8d1e787d30f3b7


Error running ORB_SLAM3 with D435i camera: rs2::invalid_value_error'   what():  hwmon command 0x80( 5 0 0 0 ) failed (response -7= HW not ready)

Before opening a new issue, we wanted to provide you with some useful suggestions (Click "Preview" above for a better view):

Consider checking out SDK examples.
Have you looked in our documentations?
Is you question a frequently asked one?
Try searching our GitHub Issues (open and closed) for a similar issue.

All users are welcomed to report bugs, ask questions, suggest or request enhancements and generally feel free to open new issue, even if they haven't followed any of the suggestions above :)

Required Info

Camera Model
{ D435I}

Firmware Version
(<IP_ADDRESS>)

Operating System & Version
{ Linux (Ubuntu 18) /

Kernel Version (Linux Only)
(e.g. 4.14.13)

Platform
PC.

SDK Version
{ legacy / 2.53.1 }

Language
{C/C#/labview/nodejs/opencv/pcl/python/unity }

Segment
{Robot/Smartphone/VR/AR/others }

Issue Description
<I am trying to run ORB_SLAM3 with my Intel D435i camera using the command
./Examples/Monocular/mono_realsense_D435i Vocabulary/ORBvoc.txt ./Examples/Monocular/RealSense_D435i.yaml...
However, I am encountering the following error message:
terminate called after throwing an instance of 'rs2::invalid_value_error'
what():  hwmon command 0x80( 5 0 0 0 ) failed (response -7= HW not ready)
Aborted (core dumped)
Sure, based on the information you provided, here's an example of an issue message that you could post on the ORB_SLAM3 Github repository:
Title: Error running ORB_SLAM3 with D435i camera: invalid_value_error
Description:
I am trying to run ORB_SLAM3 with my Intel D435i camera using the command ./Examples/Monocular/mono_realsense_D435i Vocabulary/ORBvoc.txt ./Examples/Monocular/RealSense_D435i.yaml. However, I am encountering the following error message:
arduino
Copy code
terminate called after throwing an instance of 'rs2::invalid_value_error'
what():  hwmon command 0x80( 5 0 0 0 ) failed (response -7= HW not ready)
Aborted (core dumped)
This error message occurs after installing the RealSense Viewer 2.5.3. Before installing the viewer, ORB_SLAM3 was working fine with my D435i camera.>

Hi @saood-ahmad  Can you confirm please that your camera has firmware <IP_ADDRESS> installed, as this error might occur if you had SDK 2.53.1 installed but the camera had a firmware older than <IP_ADDRESS>.
It may be worth considering downgrading to SDK 2.51.1 and firmware <IP_ADDRESS>.

It works ! Alhamdulillah
thanks for your help

It's great to hear that it worked for you.  Thanks very much for the update!

Title: Auto Exposure not working with RealSense D435i camera in RGB-D mode
Description:
I am using ORB_SLAM3 with the RealSense D435i camera in RGB-D mode. When I run the command ./Examples/RGB-D/rgbd_realsense_D435i Vocabulary/ORBvoc.txt ./Examples/RGB-D/RealSense_D435i.yaml, the camera does not perform auto exposure.
I have tested the camera with other modes such as monocular and stereo, and auto exposure works fine. However, in RGB-D mode, auto exposure does not work.
I would appreciate it if someone could help me with this issue or provide any insights into why auto exposure is not working with the RealSense D435i camera in RGB-D mode.
Thank you.

Circumstances in which auto-exposure could be deactivated by the contents of a .yaml configuration file are if (a) auto-exposure is set to false, or (b) a manual exposure value is set, which would cause auto-exposure to automatically disable.

Hi @saood-ahmad  Do you require further assistance with this case, please?  Thanks!

I have made modifications to the RealSense_D435i.yaml configuration file in an attempt to enable auto-exposure. However, I have not observed any effect on the camera's exposure behavior.
After investigating further, I checked the supported options for the camera, and it appears that auto-exposure is indeed available. I came across an issue mentioning that auto-exposure could be deactivated by the contents of the YAML configuration file if auto-exposure is set to false or a manual exposure value is set.
I have uploaded my RealSense_D435i.yaml file https://drive.google.com/file/d/1prjbP7D-cmoKWUVwtZrf67JmoUxlicSD/view?usp=sharing  for your reference.
Screenshots:
To provide a clearer understanding of the issue, I have captured screenshots in different lighting conditions, including low light and sunlight. The screenshots showcase the camera's output and the observed exposure settings for each lighting condition. You can find the screenshots [.](https://drive.google.com/file/d/1aW5ipvz15v1MpXnxi93nbtblt3KyqUyv/view?usp=sharing, https://drive.google.com/file/d/1bbhWFzUwS7Ld7O21g2UYNCcS3rg6YbXz/view?usp=sharing, https://drive.google.com/file/d/1oJe0vAP05ZMcqEwagM2GvwFbrfkwpuWF/view?usp=sharing)

The ORB-SLAM3 yaml file does not have any configuration settings that would affect manual exposure or auto-exposure.
Thanks very much for the images of low-light and sunlight.  They resemble what an image may look like if the infrared sensors are saturated with light, which can occur if a scene is too dark or too sunny.  If auto-exposure is enabled then a way to test for sensor saturation is to cover your hand over the front of the camera for several seconds to give the auto-exposure time to make adjustments and then remove your hand and see whether the image auto-corrects.
https://github.com/IntelRealSense/librealsense/issues/2875 shows the effect that saturation can have on the infrared image in low-light and sunny conditions.

Hi @saood-ahmad  Do you have an update about this case that you can provide, please?  Thanks!

Hi @saood-ahmad  Do you require further assistance with this case, please?  Thanks!

Case closed due to no further comments received.


Spotify Stream Support?

Hello and thanks for that great Bot. On my Private Server i want to use Spotify as Service with the Bot. Is that possible? I own Premium, thats why i am asking.

That is not possible because:
a) It breaks Spotify's terms of service
b) Spotify streams are not open like youtube/soundcloud

@jagrosh perhapes it is possible to add spotify stream support. But maybe not for PUBLIC.
I talked to the developers from another github project that does support Spotify, withoud going into ToS from Spotify.
https://github.com/Frederikam/FredBoat/search?utf8=✓&q=spotify&type=
Maybe if you read trough the code on that search term, you can reverse engineer how they pulled it off. I really think it's handy to use!

If a Spotify link is given, you could just query Youtube/SoundCloud for the name of the Spotify song. Seems like a fair solution to me


Feature request: Allow multiple characters in the emoji param

I would like to have multiple versions of a similar model...
and ideally would like my little picture to be
🌀🐌
🌀⚡
not
🌀
⚡

For adding multiple models, you can accomplish this by adding a setting screen to a single model. Chain forge already allows you to add multiple versions of the same model, and just change the settings. For the emoji point, I suggest, adding the second emoji as the starting character of the name of the model. We will not support multiple emoji’s, because you can change the emoji via a character selector.


drivers: can: expand return values reported by the CAN timing functions

Expand the error return values reported by the CAN timing calculation functions to be able to distinguish between an an out of range bitrate/sample point, an unsupported bitrate, and a resulting sample point outside the guard limit.
Move the check for the maximum allowed CAN bitrate to the corresponding can_calc_timing()/can_calc_timing_data() function.
Signed-off-by: Henrik Brix Andersen<EMAIL_ADDRESS>

@warasilapm I have updated this PR based on your feedback and our discussions. Please take another look.


Item viewer search; Change condition to not error

Who is this for?
For the errors we're getting in the alerts channel
What is it doing for them?
This isn't a fix (so far) but it allows to get further in the bug fixing.
The error occurs on any query in https://wellcomecollection.org/works/shmr5v5a/items.
I think that maybe part of it is due to a change in the condition I did a while back, because of a TS error. Currently searchResults returns undefined, not null, I don't know if that's intended or not?
This is the searchService link in the transformedManifest : https://iiif.wellcomecollection.org/search/v1/b30534185
It says "No searchable text found for b30534185".
I'm not sure at what stage the error occurs. searchWithin displays if there is a searchService value, which there is, but the response is empty.

@gestchild would you have an idea of where this fails? Is that error sometimes expected/is this a question for Digirati?

The error occurs when someone tries to conduct a search. I don't know why the search service provided in the manifest would 404, that is a question for Digirati, but we could handle the error better, rather than leave the user hanging - something like this

The IIIF was rebuilt and fixed the issue for that work https://wellcome.slack.com/archives/CBT40CMKQ/p1707218899840299
Let's still merge in our fixes 👍


remove window.ENV from userDetails route page

remove window.ENV from userDetails route page  and use the env from ther server using the loader function
eg:
export const loader: LoaderFunction = async () => {
  const oAuthLinks = await getOAuthLinks();

  return json<LoaderData>({ ENV: oAuthLinks });
};

I would love to work on this issue ticket. @isVivek99 can you please assign me?


Fix typo in recipe "Parse from File with Headers"

The text mentioned an instance method, the code itself uses a class method.

Good catch!


List of received snaps is listed from old to new.

As the issue tells, you see your old snaps in the beginning of the list and new ones at the bottom.
@filfat just to let you know. I don't really know sometimes if I should report a bug or not? Or do you want us to report every single one we find?

Thanks, I'll quickly fix this by simply reversing the list.
Yes, report all bugs you find just double check that they haven't been reported already :)

Ok I'm on it 😊

Already done :P


Rebaseline DISP-S1 against minimal "golden" dataset

A new delivery of minimal expected inputs/outputs for the DISP-S1 SAS is now available here: https://artifactory-fn.jpl.nasa.gov/ui/repos/tree/General/general/gov/nasa/jpl/opera/adt/disp_s1/r2.1/interface/delivery_data_dolphin_small.zip.
With this new dataset, it would be good to rework the following:

[x] Update acceptance test procedure for DISP-S1 to reflect use of new dataset
[x] Update DISP-S1 runconfig under examples/ to reference only the minimal set of inputs

Closed by #301