How a Image component showing by a absolute local path

I developed a customized image component to download the online image and saved to a local file. I can not find a way to show this local file by Image component. I am sure these file downloaded successfully, because I can open them. I have tried the following ways:
#1)
<Image                             source={{isStatic:true, uri:'file:///Users/jay/Library/Developer/CoreSimulator/Devices/00520C5F-1307-427E-AA55-9E1E020B2EFB/data/Containers/Data/Application/522D5404-A2EF-4117-A768-CF63A6A3EE8F/Library/Caches/6ea32f8825c207ec582ad684ac6a3494'}}                             style={imageStyle} />
#2)
<Image                             source={{isStatic:true, uri:'/Users/jay/Library/Developer/CoreSimulator/Devices/00520C5F-1307-427E-AA55-9E1E020B2EFB/data/Containers/Data/Application/522D5404-A2EF-4117-A768-CF63A6A3EE8F/Library/Caches/6ea32f8825c207ec582ad684ac6a3494'}}                             style={imageStyle} />

@facebook-github-bot stack-overflow

My example is from stackoverflow. I also read a github project named react
native image cache. None of them work.
I Think this should be put in official document if there is a way
On 16 Apr 2016 12:31 AM, "Nicolas Charpentier"<EMAIL_ADDRESS>wrote:

@facebook-github-bot https://github.com/facebook-github-bot
stack-overflow
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/facebook/react-native/issues/6998#issuecomment-210483536

I don't believe stackoverflow is the best place for this question because
this looks like a new feature being requested or a missing document for a
popular feature. Anyway if you want to be peaceful just neglect my point.
On 16 Apr 2016 12:38 AM, "Facebook Community Bot"<EMAIL_ADDRESS>wrote:

Hey @jaynsw https://github.com/jaynsw and thanks for posting this!
@charpeni https://github.com/charpeni tells me this issue looks like a
question that would be best asked on StackOverflow
http://stackoverflow.com/questions/tagged/react-native. StackOverflow
is amazing for Q&A: it has a reputation system, voting, the ability to mark
a question as answered. Because of the reputation system it is likely the
community will see and answer your question there. This also helps us use
the GitHub bug tracker for bugs only. Will close this as this is really a
question that should be asked on SO.
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/facebook/react-native/issues/6998#issuecomment-210485798

In fact, i'm totally agree with you and i'm wondering the same question as you.
However, as mentioned in the issue template this repo uses GitHub for bugs and StackOverflow for usage question. Feature requests can be post on [Product Pains](Product Pains).
Please feel free to ask it on StackOverflow and post the link in this issue. After that with a good answer we'll be able to update the documentation.

The following image url is from the cameraroll api, it works. I only tested this on IOS
example:
"assets-library://asset/asset.JPG?id=8E0344AD-41C4-45C4-9E0F-293BC421F078&ext=JPG"

@jaynsw did you manage to figure out how to do this ?

The url format should be same as other iOS and Android App. I suggest you
can setup a break point in cameraroll example of UIExplorer to look into it.
Jay
On 19 Dec. 2016 3:24 am, "Salman Hasrat Khan"<EMAIL_ADDRESS>wrote:

@jaynsw https://github.com/jaynsw did you manage to figure out how to
do this ?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/facebook/react-native/issues/6998#issuecomment-267830472,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFZBPoRE_i85AI_uEdnQaPINIVpt3XuXks5rJV42gaJpZM4IIXgP
.


Git action fix

fix release build

Codecov Report
Base: 33.07% // Head: 33.07% // No change to project coverage :thumbsup:

Coverage data is based on head (fdcbaf9) compared to base (84c5f7f).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #662   +/-   ##
=======================================
  Coverage   33.07%   33.07%
=======================================
  Files         134      134
  Lines       14355    14355
  Branches     4184     4184
=======================================
  Hits         4748     4748
  Misses       9501     9501
  Partials      106      106

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Please update the npm version for parity to the bower version.

Seems the npm version is older than what is on bower. Thanks in advance.

The latest tag is v1.0.10 and the version in package.json is 1.0.10. Bower uses the Git tag and so the NPM and Bower versions seem to be in sync. Could you please elaborate on the issue you're seeing?


#trivial Fix macOS example Podfile

Trivial, make pod works.

🚫 CI failed with log

Note: failed CI seems not related to this PR.


Convert use_seurat_comp/meta to S3

What is a comp? component? expression? I don't think this is a good suffix.
And why is it use_)? Couldn't it just be seurat_expr() and seurat_meta()?
Need to dispatch on v2 and v3.

does it need to be S3?
checking whether class is Seurat or seurat works too
so rename to seurat_ref (or seurat_avg) and seurat_meta?


LEA bug

I think your LEA implementation is wrong.
When I want to execute, for example, lea sp, ... the c variable op_from_addr is 0xf0008 which points inside mem to ES instead to SP. This is because you calculate
op_from_addr = REGS_BASE + i_reg * 2;

for your LEA implementation instead of just
op_from_addr = REGS_BASE + i_reg;

(without * 2).

Hi TheFox - I think the implementation is correct (but please let me know if you still think otherwise).
0xF0008 points inside mem to SP, not ES (which is at 0xF0010). mem is an array of unsigned char, but the registers are 16 bits. So, to convert the 16-bit register index i_reg extracted from the opcode to an index within mem, the calculation REGS_BASE + i_reg * 2 is correct.
-Adrian

Thank you for your very fast answer.
But why do I get the same address as ES, calculating REGS_BASE + REG_ES is also 0xF0008.

TheFox - what do you mean when you say "why do I get the same address as ES"?
The decoder for LEA calls DECODE_RM_REG, which calls GET_REG_ADDR, which always does the calculation REGS_BASE + 2 * reg_id. So it is never REGS_BASE + REG_ES - the index into mem is always REGS_BASE + 2 * REG_ES.

Ah I see. So my REGS_BASE + REG_ES assumption is wrong.

I think so. Also, from a heuristic point of view, if the LEA implementation was so wrong, many common programs would break, which we don't see.
-Adrian


Fix footers to use new legal param

What is the context of this PR?
Footers are currently broken with v27.0.0 of the DS being merged. This will fix them and update them to use the new legal param.
This PR is dependant on PR #410 being merged first
How to review
Describe the steps required to test the changes (include screenshots if appropriate).
Checklist

[ ] New static content marked up for translation
[ ] Newly defined schema content included in eq-translations repo

 Fix footers after DS v27.0.0 merge - update them to use new legal param


Update ug for addgrade

Closes #314
Closes #322
Closes #305

I think we should change the format for removeGrade from n/ to name/ as well


Allow implicit dynamic to be inferred from explicit dynamic

Consider the following code:
class Foo {
  List<dynamic> values;

  Foo(Iterable<dynamic> values)
      : values = new List.unmodifiable(values);
}

When I run dartanalyzer --strong --no-implicit-dynamic on this, I get:
[error] Missing type arguments for generic type 'List<dynamic>'. Either add an explicit type like 'dynamic', or enable implicit-dynamic in your Analyzer options.

This seems counterproductive. I'm explicitly declaring types at my API boundaries, and I expect them to be inferred elsewhere, just like they would be if I'd written <String> instead of <dynamic>.

merging into https://github.com/dart-lang/sdk/issues/26781


Add room

In this PR I:
Created form to add new room.

Status: Approved  :champagne:  ✔:confetti_ball:
Hi @gsmalik030 , The Add room page content is as expected. Please proceed and merge the PR.


RuntimeError: CUDA out of memory.

Hi, I am working in Kaggle with custom dataset. I got RuntimeError and I have no idea how to solve it. Can you help me?
RuntimeError: CUDA out of memory. Tried to allocate 2.44 GiB (GPU 0; 15.90 GiB total capacity; 9.95 GiB already allocated; 1.70 GiB free; 13.44 GiB reserved in total by PyTorch) If reserved memory is >> allocated memory try setting max_split_size_mb to avoid fragmentation.  See documentation for Memory Management and PYTORCH_CUDA_ALLOC_CONF

@uysalfurkan Can you try reducing the batch size? I think that should solve the issue.

@sovit-123 firstly thank you for your answer, yes it helps me but I need to compare yolov5 and faster r can in my task.
In yolov5, training batch_size=64 gave me the best results. So I need to complete this process with batch_size=64, is it possible ?

@uysalfurkan Try with smaller batch size, in my opinion. Faster RCNN models are generally larger YOLOv5 models. From my experience, Faster RCNN models give good results with batch size 4 as well.
Please try that. And if you like the library, I will surely love to get some feedback from you.

Thank you for the feedback.
@uysalfurkan I CSV file with mAP metric is already saved. Let me know what other things you want to track in the CSV file. I will add them as well as part of the library.

My purpose is to visualize the metrics (mAP [0.5 and 0.5_0.9], recall, precision, loss [object and box]) of both validation and training per epoch over the same plot. In order to do that I need a csv file which contains these metrics and epoch number.
Additionally, it would be perfect if we have a result_info.txt file that generated at the end of the training process and contains hyper parameters such as learning rate, optimizer name, batch size and model informations such as pertained model and backbone version.
I will be appropriate If you can add all this things as a part of the library. I am trying to add but I'm having trouble figuring out in which file the performance results are generated and what the variable names are.
Thank you!

@uysalfurkan
Hi,
Some of the things like an opt.yaml file containing all the hyperparameters and model names are already saved to the resulting directory.
I will try to add the other things. Some of the things like the validation losses are a bit difficult to add as the PyTorch Faster RCNN models don't output any loss values in eval() mode.
A lot of things apart from the validation loss are already saved. I will try to add the other things. But may take some time as I am the only person working on this project.

@uysalfurkan I have also updated WandB to plot everything according to epoch instead of iterations which is a bit easier to interpret.

@sovit-123 Thank you, I have another problem.
When I run the training command, wandb pop-ups appear on the screen. Under normal conditions, if the login does not occur within 30 seconds, the wandb integration should be ignored automatically and the training process should be continued. But that's not happening and it keeps waiting for wandb entry forever.
When I working in Kaggle, it is more suitable that complete the running process by 'save version button with save output choice'. Because it can work on the background even though my computer is sleeping. In other words, the fact that wandb does not time out causes the training process to not start.
(I would like to know if wandb login is possible in runs started with save version option)

@sovit-123 Hi, thank you for your interest. I am waiting for updates. I have added a screenshot of csv info example which I need to get.
By the way, can I modify the train.py file from the notebook? For example, I will change the learning rate value or
results_csv_example.csv
add a new plot function with confidence score to the annotation.py file.
This is easy when i work in local but how can i do it in kaggle environment. (For example, we replace the yaml file with %%writefile)
results_csv_example.csv

@uysalfurkan The CSV file update has been made. It's slightly different as of now compared to what you are asking.
It has four losses, mAP @ 0.5 and mAP @ 0.50:0.95.
And yes, you can use the %%writefile method to overwrite your own modifications. It will work.

@sovit-123 I run a training but the generated csv file has just 8 columns which are epoch | map | map_05 | train loss | train cls loss | train box reg loss | train obj loss | train rpn loss
I need to get all of columns inside the csv that I attached above. I need to see train-val performances over the same plot for analyzing epochs and overfitting. Did you make another update for that?
Or which py file should I focus on to make this updates myself.
Thank you!

@uysalfurkan
I am not yet analyzing the validation values. It will take some time. It requires modification in the validation function inside the engine.py script. This is because Faster RCNN models do not output any validation loss values in model.eval() mode. They do so only in the model.train mode.

@sovit-123 Ok, I'm waiting, I would be very happy if you let me know when the modification is finished. Also, it would be great if train mAP values are in the file as well as validation.

@uysalfurkan mAP is a validation metric already. In object detection, we calculate mAP on the validation dataset only which is the case with this code base as well. I hope this helps.

@sovit-123 Hi, I'm a little confused after the conversation. How can I understand whether there is overfitting or not from the your result csv ?

@uysalfurkan
In object detection, you can check that a model is overfitting if the mAP starts decreasing instead of increasing. mAP is always calculated on the validation dataset.

Hi @sovit-123
I got the log below when I run the train.py with create_fasterrcnn_model as model.
UserWarning: This DataLoader will create 4 worker processes in total. Our suggested max number of worker in current system is 2, which is smaller than what this DataLoader is going to create. Please be aware that excessive worker creation might get DataLoader running slow or even freeze, lower the worker number to avoid potential slowness/freeze if necessary.
cpuset_checked))
Traceback (most recent call last):
File "train.py", line 505, in
main(args)
File "train.py", line 258, in main
build_model = create_model[args['model']]
KeyError: 'create_fasterrcnn_model'

Hello @uysalfurkan
That above warning says that are there 2 cores in the CPU but you are trying to use 4 cores (4 workers). I think this is a common warning on Kaggle. For now, you may ignore it or pass --workers 2 to the training command if you want to avoid this warning.
Regarding the KeyError. You need to pass a valid model name key to the --model flag in the train.py command. Looks like you have passed create_fasterrcnn_model as the key which is not valid.
By default the key is fasterrcnn_resnet50_fpn_v2.
You may also pass a model name key like this:
python train.py --model fasterrcnn_resnet50_fpn  <rest of the command>
You can find all the model name keys that you can pass here:
https://github.com/sovit-123/fasterrcnn-pytorch-training-pipeline#A-List-of-All-Model-Flags-to-Use-With-the-Training-Script

@sovit-123 Hi,  I want to get mAP_0.5:0.90 rather than mAP_0.5:0.95. How can I change the code ?

@uysalfurkan Hello, that would require changing the Pycocotools code. But at the moment, I cannot say for sure at which place we need to change the code.

Hi again,
After starting to run train.py command, I got the error below,
OSError: /opt/conda/lib/python3.7/site-packages/nvidia/cublas/lib/libcublas.so.11: undefined symbol: cublasLtHSHMatmulAlgoInit, version libcublasLt.so.11
How can I fix this ?

@uysalfurkan
Looks like CUDA issue. Which GPU do you have, is it an RTX or GTX GPU?

I am using Kaggle GPUs.

NVIDIA TESLA P100 GPU
TESLA T4 x2

@uysalfurkan
Ok. I understand the issue now. Looks like pip install -r requirements.txt is installing torch 1.13.1 which has issues with CUDA on Kaggle.
This is because of this line in the file torch>=1.12.0, !=1.13.0
For now, PyTorch 1.12.0 works best. I will update the requirements file with torch==1.12.0 by end of day. You may also manually install it in the Kaggle environment and it will work fine.

Hi @sovit-123 thanks for your fast replies.
I got the error below and could not figure it out. I set the epoch number as 100 but at epoch 55 the process becomes fails.
RuntimeError: DataLoader worker (pid 18111) is killed by signal: Killed.

@uysalfurkan
Are you running on Kaggle?

@sovit-123 Yes I am running on Kaggle


Update automation account region mapping

Overview/Summary
Replace this with a brief description of what this Pull Request fixes, changes, etc.
This PR fixes/adds/changes/removes

Fixes #449
Contributes towards #227

Breaking Changes

For customers deploying Management resources to East US or East US 2 regions, Terraform will want to destroy / create the automation account to move it to the correct location.

Customers not using Update management and who don't want to redeploy the automation account must add the following code to their module configuration:

  # within the module declaration
  configure_management_resources = {
    # other attributes removed for brevity
    advanced = {
      custom_settings_by_resource_type = {
        azurerm_automation_account = {
          management = {
            location = "eastus" # replace value as needed
          }
        }
      }
    }
  }

Testing Evidence
Please provide any testing evidence to show that your Pull Request works/fixes as described and planned (include screenshots, if appropriate).
Plan generated for deployment to northeurope:

Plan generated for deployment to eastus:

Plan generated for deployment to eastus2:

As part of this Pull Request I have

[x] Checked for duplicate Pull Requests
[x] Associated it with relevant issues, for tracking and closure.
[x] Ensured my code/branch is up-to-date with the latest changes in the main branch
[x] Performed testing and provided evidence.
[x] Updated relevant and associated documentation.
[x] Updated the "What's New?" wiki page (located in the Enterprise-Scale repo in the directory: /docs/wiki/whats-new.md)

/azp run unit

Let's make sure release notes surface the impact of the change.

100% - that's why I included them in the PR notes too 😄


Limit bootstrap peers

Currently Crust does 2 things we don't like:

bootstrap cache has no limit. That can cause the system to behave unpredictibly: consume too much memory in RAM and/or disk, act slow where it's not expected, like loading the cache, etc.
Service::bootstrap() will attempt all peers in parallel, even if there are thousands of cached peers. The problem with this is that many systems won't allow you to use more than ~1000 sockets in parallel.

So

introduce configurable bootstrap cache limit with the reasonable defaults (say 10000 which roughly results in 40 kB cache). When bootstrap cache exceeds the limit, remove the contacts with smallest trust level - see bellow for description.
Introduce a counter for bootstrap cache entries that logs how many times a peer was used to successfully bootstrap off. Then this counter will be used as a trust metric - the more we've bootstrap from the peer the more we trust it's availability.
Limit parallel connections during bootstrap to prevent from exhausting system sockets.

Running some QA checks on this, keeping notes here https://gist.github.com/S-Coyle/91aeba06ea13bdbd386118d1e44fe1af


Mafia III low fps when I choose "Continue" instead "New Game" after splash screen.

I have low fps when I choose "Continue" instead "New Game" after splash screen.
If I choose New Game game runs "ok" on my hardware, even when I press restart mission in menu. I don't notice this issue on WineD3D.
New Game:

Continue:

Software information
Mafia 3, V-sync and fps limit is off in game options. All details are set to minimum. Resolution: 1680x1050 in necessary windowed mode. Game don't want to launch in full screen mode. Sync to VBlank and Allow Flipping is off in Nvidia Settings.
System information

GPU: GeForce GTX 750 Ti (I know this GPU is weak, but the game should be playable)
Driver: 435.19.03
Wine version: runs on proton 4.11-3 (I replaced DXVK to newer version), on lutris the same issue.
DXVK version: 1.3.4, I tested older versions and the same issue.

Apitrace file(s)
https://drive.google.com/open?id=1o8PCAUqE0VvCVJPgXlPKDoOerKiZJDbI
Both files are here. The first is "New Game", the second is "Continue". I hope apitrace can help you.
Log files

d3d11.log:
launcher_d3d11.log

dxgi.log:
launcher_dxgi.log

This is almost certainly a case of your GPU not having enough VRAM to run the game, and the game probably recreates all its render targets after loading the level so they end up in sysmem. There's not much I can do here.

It's a pity. Strange thing is when I play after pressing New Game I can play long with good performance. One way to play with good fps is change permissions to read only on save game files. Then I run New Game, after loading I choose restart mission. This can be helpful but sometimes the game loads player in wrong place and mission dissappears. Maybe someday the problem will be solved. Thanks for the reply and best regards. I have been following the project for a long time and I think you are doing a great job.

New Game\Continue doesn't change anything to me, but i can reproduce that low fps with the frame limiter set to anything but unlimited. Maybe the game is enforcing that option somehow in your end?
http://www.framecompare.com/image-compare/screenshotcomparison/J9M11NNU
And yes, the game will hang if you try to launch it on fullscreen mode, but you should be able to change it to fullscreen while in game without problems. Just remember to change it back to windowed before you exit.

As doitsujin says, there is probably a problem with vram, but I found a workaround. I noticed that when I change the resolution of 1680x1050 to 1280x720, the game works just like after pressing New Game, so after thinking about it, I changed the resolution to 4K in videoconfig.cfg, then I changed the resolution in the game to my native one and everything works fine. I did it in Openbox to save Vram. I think the resolution affects vram and this is a fact.
The game works for me after choose "Continue" with a small amount of FPS as with the FPS limit enabled. The amount is probably even the same. The FPS limit in this game is not working properly. After setting to 60, it is probably set to around 20. After setting to 30, it is just set to around 10 FPS.You are right that the game works in full screen mode, but unfortunately it breaks down during the movies. Is there any way to solve this problem?

There is a tool called DxWnd that can force any game to work in boardless window mode, and last time I used it with Wine, it worked fine.

Can verify this issue.
Nvidia 960 (2GB vram) 510.60 driver. Latest dxvk (proton bleeding edge)

Continue

New game

Have not found that any dxvk conf settings affect the problem.
I also have another issue that i can't change the resolution ingame as it then freezes or black screens + freezes.

The original comment still holds true, it's most likely related to a lack of VRAM.

Even if it doesn't fill out?
I will try on my amd system which have 4GB

That's just the amount of memory the game uses. Other applications + fragmentation could fill up the rest.

Derp yes of course, makes sense.
Looking at nvidia-smi the vram is indeed smack at the top. And on my amd system with 4GB it does not happen.
If comparing the memory usage between the two then ideally the game should be allowed at least 300-400MB more than the nvidia system can deliver (around 1300). Maybe if i wasn't running a DE it could scrape by as plasma and wayland are around 400 combined.
Anyway yes cased closed i guess xD. On to the next issue :)


Limit rich text options available

Link to Ticket
Description of Changes Made
Remove image, embed and HR from the rich-text options available for all rich-text fields.
There is manual editing of older blog posts needed before this can be pushed to production - Lily has a list.
How to Test
Edit any page that has a paragraph block or other rich-text field, and look at the options available.
Screenshots

Expand to see more
Rich text block:

Rich text field:

MR Checklist

[x] Add a description of your pull request and instructions for the reviewer to verify your work.
[x] If your pull request is for a specific ticket, link to it in the description.
[x] Stay on point and keep it small so the merge request can be easily reviewed.
[x] Tests and linting passes.

Unit tests

[ ] Added
[ ] Not required

Documentation

[ ] Updated build docs
[ ] Updated editor guidelines (https://docs.google.com/document/d/1PAWccdQ4tfaZsrEWmpDhvP3GH5RRmBOARFVp4b-kje8/edit?usp=sharing)
[ ] Not required

Browser testing

[ ] I have tested in the following browsers and environments (edit the list as required)

Latest version of Chrome on mac
Latest version of Firefox on mac
Latest version of Safari on mac
Safari on last two versions of iOS
Chrome on last two versions of Android

[ ] Not required

Data protection

[x] Not relevant
[ ] This adds new sources of PII and documents it and modifies Birdbath processors accordingly

Accessibility

[ ] Automated WCAG 2.1 tests pass
[ ] Manual WCAG 2.1 tests completed
[ ] I have tested in a screen reader
[ ] I have tested in high-contrast mode
[ ] Any animations removed for prefers-reduced-motion
[x] Not required

Sustainability

[ ] Images are optimised and lazy-loading used where appropriate
[ ] SVGs have been optimised
[ ] Perfomance and transfer of data considered
[ ] If JavaScript is needed alternatives have been considered
[x] Not required

Pattern library

[ ] The pattern library component for this template displays correctly, and does not break parent templates
[ ] The styleguide is updated if relevant
[x] Changes are not relevant the pattern library

Closing in favour of https://github.com/torchbox/torchbox.com/pull/114


How to handle updates within @connect

When a Store is @connected to a Component, that observable instance gets filled with the initial props, and gets updated later when the props change. The current behaviour is that all values are overwritten when any value changes, removing any user modifications to those unchanged values.
React's suggestions for components is to be:

fully controlled: props would always overwrite user changes. we would have to prevent changes to the keys where incoming props were put. This behaviour is the same as our current, but without enforcing fully controlled (and thus resulting in user confusion).
fully uncontrolled: Incoming props changes are ignored unless key is changed to reset them.

Of these two, being fully uncontrolled will prevent issues like "why did my data reset?". For a full reset under this option, you just need to change the key prop in the parent render.
There already exists a mapProps option which can be used to control exactly how the props are merged into the observable, allowing the developer to update some or all of the data when props change and giving them full control over the behaviour.
@mickmcgrath13 @DesignByOnyx @imaustink

Based on article you likned to (https://reactjs.org/blog/2018/06/07/you-probably-dont-need-derived-state.html) I think being "uncontrolled by default" is a great change and very much follows the React way.
It's the point of least surprise, can be overcome by the user with a key prop, and we're still giving them a way to update based on new props in the mapProps function.  Sounds like wins all around.
I support this change

Uncontrolled is also how I went for raj-react. It makes the most sense for things that are themselves stateful and aren't easily derived from the new props.

Random Justin Notes:
=======
// {outer1: "valueA", outer2: "valueB"}

setState({outer1: "VALUE"});

render(){
    <Inner inner1={this.outer1} inner2={this.outer2}>
}

var vm  = new VM({outer1: "valueA", outer2: "valueB"})
vm.inner2 = "VALUE_X";

setState({outer1: "VALUE"});

//-> {inner1: "VALUE", inner2: "valueB"}

mapProps(newProps, ){
    newProps //-> {inner1: "VALUE", inner2: "valueB"}

    {type: "add", key: "foo", value: 1},
    {type: "set", key: "foo", value: 1},
    {type: "delete", key: "foo"}
}

onClick(){
    setState({})
}

render(){
    <Inner inner1={this.outer1} inner2={this.outer2} fn={}>
}

mapProps(newProps){
    newProps //-> {inner1: "SOMETHING_NEW", inner2: "valueB"}
}

innerVM.inner1 = "foo";
fn(){
    this.props.outer1 = "SOMETHING_NEW";
}

I think shallow diff is ok for a default


[ISSUE #340]polish quickstart

Fixes ISSUE#.
Motivation
Explain the content here.
Explain why you want to make the changes and what problem you're trying to solve.
Modifications
Describe the modifications you've done.
Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)
If a feature is not applicable for documentation, explain why?
If a feature is not documented yet in this PR, please create a followup issue for adding the documentation

Codecov Report

Merging #504 (e794bc3) into develop (a62bf65) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head e794bc3 differs from pull request most recent head 1e59aee. Consider uploading reports for the commit 1e59aee to get more accurate results

@@            Coverage Diff             @@
##             develop     #504   +/-   ##
==========================================
  Coverage      10.42%   10.42%
  Complexity       328      328
==========================================
  Files            240      240
  Lines          11743    11743
  Branches         999      999
==========================================
  Hits            1224     1224
  Misses         10419    10419
  Partials         100      100

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 a62bf65...1e59aee. Read the comment docs.


[GSSoC '23] Contributing a QR Scanner Project to Code-Canvas

Description
I am making this issue to contribute a qr scanner project build with vanilla JS to Code-Canvas open source project.
Objective
By contributing this project I want to help beginners build and learn more JavaScript Projects and have a good understanding in JavaScript, so this website will be a great project for everyone to have more knowledge about JS.
So Please assign this to me under GSSoC'23 @Dharmesh177

Sure assign to u
Complete it in 1 week and follow contributing.md file guide while making PR
Happy Coding !!


class nvinfer1::INetworkDefinition has no member named getErrorcorder

hello,thx for your job,but when I try to make this project, it has wrong, please tell me why? thank you very much!!

你这个是tensorRT版本不匹配的结果，请参考不同tensorRT的支持问题
默认配置的是8.x


Export of "Unknown" chats overwriting each other

During my run of the exporter, I noticed having multiple "Unknown" exports.
But, after finishing, only one "Unknown ().html" was in the /out folder.
Seemingly, the script doesnt check for the existence of files before writing to disk, meaning, if the tenant/user includes multiple unknown chats without unique IDs, all files before the last "Unknown ().html" export are lost.

It seems it's failing to get anything about the user you are chatting with. I've added extra logging around the problem area if you want to run verbose mode (https://github.com/evenevan/export-ms-teams-chats?tab=readme-ov-file#debugging).
I've also included a new flag, -avoidOverwrite which will append a number at the end of the file so it doesn't overwrite. To use this, you need to use the legacy running method (https://github.com/evenevan/export-ms-teams-chats?tab=readme-ov-file#2-run-the-script) and run via command line. The command is .\Get-MicrosoftTeamsChat.ps1 -Verbose -avoidOverwrite.
Hope this helps! If you can give me the verbose output (with your information and other PII redacted) that might help with a more proper solution.

Thanks for the Overwrite option, will run that soon.
The chats that came back as Unknown were with Skype or Teams Personal users, so no official Teams account info from Entra ID there.
I assume there is nothing more to be had from those accounts, since these are just regular MS accounts..


Nutrition/app

Blocked by #146.

@uglrl @worldofjoni @ElenaHaeussler @AlexKutschera Has anyone of you had some time to look into this yet? :-)

Any news on this?

@AlexKutschera ?


fix: Adding patches to fabricv4 for Terraform Provider Equinix migration to equinix-sdk-go

Removed additional properties from ServiceProfileAccessPointTypeVD to avoid unmarshal conflict with ServiceProfileAccessPointTypeCOLO because they're both in the anyOf property for ServiceProfileAccessPointType
Renamed CloudRouterPackageType to CloudRouterPostRequestPackage to avoid naming conflict for code generation. The first name would not create the correct model to be used for creating CloudRouters
Created PortResponse base model to be used for Port API Responses and then used in it the Port model so that Port could have required fields while PortResponse did not contain that restriction
Created SimplifiedAccountPortResponse model to account for Fabric Port API returning accounNumber and orgId as strings instead of int

@ctreatma , I made a small change after approval. Fabric Port team is still accepting Port Request Account values as int. So needed to make sure I only updated the response. Wanted to alert you to that before merging.


express.compress problem (express 3)

In case you're using express.compress (express v3), you'll see an error like:
events.js:72
    throw er; // Unhandled 'error' event
          ^
Error: write after end

It will happen if you use express-interceptor before compress.
Old issue reported: https://github.com/expressjs/session/issues/72

Easy workaround is to change the order of compress and express-interceptor middlewares to:
var express     = require('express');
var interceptor = require('express-interceptor');
var app = express();
var finalParagraphInterceptor = interceptor(function(req, res){ ... });
app.use(express.compress());
app.use(finalParagraphInterceptor);


Detect if Gemfile requires git?

This is a great project!
I tested boxing generate on my Rails app and it failed because my Gemfile contains a git dependency, like this:
git_source(:github) { |repo| "https://github.com/#{repo}.git" }
# ...
gem "rails", github: "rails/rails", branch: "7-0-stable"

It causes the following build error:
 => ERROR [gem 6/6] RUN gem install bundler:2.3.3     && bundle config --local deployment 'true'     && bundle config --local frozen 't  11.5s
------
 > [gem 6/6] RUN gem install bundler:2.3.3     && bundle config --local deployment 'true'     && bundle config --local frozen 'true'     && bundle config --local no-cache 'true'     && bundle config --local system 'true'     && bundle config --local without 'development test'     && bundle install -j "$(getconf _NPROCESSORS_ONLN)"     && find //src/app/vendor/bundle -type f -name '*.c' -delete     && find //src/app/vendor/bundle -type f -name '*.o' -delete     && find /usr/local/bundle -type f -name '*.c' -delete     && find /usr/local/bundle -type f -name '*.o' -delete     && rm -rf /usr/local/bundle/cache/*.gem:
#11 1.831 Successfully installed bundler-2.3.3
#11 1.831 1 gem installed
#11 4.857 Fetching gem metadata from https://rubygems.org/...........
#11 11.44 Fetching https://github.com/rails/rails.git
#11 11.46 You need to install git to be able to use gems from git repositories. For help
#11 11.46 installing git, please refer to GitHub's tutorial at
#11 11.46 https://help.github.com/articles/set-up-git

Could boxing detect if a Gemfile contains a git: option or git_source entry and add git to the list of build-time dependencies to install via apk?

Thanks for pointing out it which I forget to put into the roadmap. I think we can start working on it as a high priority.

@mattbrictson I have added the check for it and the new release v0.4.0 is supported 🎉

Thanks! It works great 🎉


Removed redux-simple-router, Updated git repo URLs

Both react-router-redux and redux-simple-router were listed in the dependencies. I removed redux-simple-router to clean it up since it has been renamed to the other module, and I also updated the "ericelliott/isomorphic-express-boilerplate" references to "cloverfield-tools/universal-react-boilerplate" just to keep it current despite the old URL redirecting to the new one.

Thanks!


Web public view

Related issue #13172

I posted a batch of comments on some of the preparatory commits that should help get them into a mergeable state.  I'm excited about the progress!

I merged the authentication exception handling changes as the series ending with 1f90a31fa3a2d05356948c9cffe85f7ab47eee25, after a bit of work to clean up the interfaces.  Thanks @amanagr!

topic: Rename get_topic_history_for_web_public_stream.

For this, what you had isn't what I'd had in mind.  I merged the series ending with 3b2a262b6f9135533ed91d84db5f8a5f8c933b5e to complete that section of work.
I pushed back to this PR, can you work on cleaning up the next few commits?  Thanks @amanagr!  I'm really excited about the progress here.

Following #16247, what are our next steps here?

This PR just needs tests to be good to review. I also need to figure out how to handle 2fa. Then we should be in a good state to have web-public-view.

backend: Allow anonymous access to webapp. is the major backend change which needs to be merged (needs tests). Other than that we only have minor backend changes.

I merged the commits through 37e77c77e2899101f426a08949fd3c40e7055923.  @amanagr would you be up for extracting the backend changes into a new PR with smaller commits that we can try to incorporate?

Sure!

We probably also should write some thoughtful assertions in the "logged-out visitor" code path that we're not leaking data we shouldn't be (or at least lay out the argument for why we aren't -- e.g. listing the specific 4 test classes that confirm this security behavior is as intended).

Okay, I will try to do this.

We need to establish consistent terminology to use to refer to "web public visitors" that doesn't get confused with is_guest users, who are a completely different concept. So I propose we do something like s/is_web_public_guest/is_web_public_visitor/ and do a grep to change all comments/strings/etc. that refer to this concept to avoid saying "web public guest". But we might want something like is_anonymous_web_user that's a bit more clear about the nature of this concept -- i.e. that it doesn't refer to specific user at all.

Done

Can you lay out your vision for what remaining steps we need to do in order to integrate this ~final backend logic commit? One option could be to add a settings.DEVELOPMENT guard to keep this feature disabled in production while we figure out the settings model for it.

settings.DEVELOPMENT guard sounds good to me. I added that. I was thinking of integrating the frontend changes I have here with the last commit. Then:

Add setting for creation of web public streams- web_public_streams_policy:

Web-public streams disabled (default)
Only organization administrators can configure web-public streams.
All stream administrators can configure web-public streams.
That controls whether one can create one (rather than checking at access time).

Then add this logic:

For a user visiting foo.zc.com for the first time (check cache to know this), we redirect to /login page and if user chooses to "Continue without login", we save it in the browser cache to not redirect to foo.zc.com/login
For logged out users who have the cache, we simply show web public view on visiting foo.zc.com
For a user visiting foo.zc.com/#narrow/a-web-public-stream - we simply show the stream (or topic) and save it in the cache that user prefers logged out view.

Then wait for #14638 or complete it myself.

The tests are now failing for documentation (You said you would help me to fix it.)

I was not sure what to add for asserting security, I added some lines anyway. I believe #14638 and web_public_streams_policy will go together. What do you want to do about them (assuming we want to do a release by the end of next week???)

I merged one prep commit.
Can you make a separate PR that's just home: Allow logged out user through home., aka the last backend change?  I'd like to have a separate CI space for working through getting that merged.

If you can rebase this now that the backend is merged, I'd love to play with it and incorporate any of the prep commits that are safe to do.  I'm also thinking we'll want to add a design document for this feature to the docs; let me see about prototyping that.

@timabbott rebased and added tests.

Merge commits only from #16961. The commits here are not updated. The comments are still relevant.

Merge commits only from #16961. The commits here are not updated. The comments are still relevant.


test: reduce test failures

This helps to reduce the amount of test failures on the main branch.
There is something wrong with the log capturing tests, https://github.com/probot/probot/pull/1855#issuecomment-1724540467

Waiting on https://github.com/probot/octokit-auth-probot/issues/266
Some Octokit dependencies were upgraded to versions that don't support NodeJS v10 which themselves included dependency upgrades to packages that don't support NodeJS v10

Is it worth to fix this? I mean isnt v13 around the corner?

It still would be nice to have tests passing in case we ever need to do a maintenance release and to make sure everything is in working order

There is only one test failure remaining,
FAIL test/probot-octokit.test.ts
  ● ProbotOctokit › with throttling enabled › retries requests when hitting the abuse limiter

    HttpError: You have triggered an abuse detection mechanism. This maybe caused by the lack of authentication. Reason: Neither "appId"/"privateKey" nor "token" have been set as auth options

      at fetch.then (node_modules/@octokit/request/dist-src/fetch-wrapper.js:68:27)

branch protection rules need to be updated to reflect https://github.com/probot/probot/pull/1917/commits/0082f46368acf028fe945e6bd1de56f69fbfcb05
Also, why are the tests only failing on mac os


Grouping API in APIM publisher portal

Describe your problem(s)
Grouping/tagging APIs in Publisher portal like we do/see in Dev portal. Our clients have APIs with almost similar names with minor changes like Service, ServiceAPI...Initially, there were few APIs but as the number of APIs is growing it is becoming difficult to search for the API we wanted for our development. We were looking for a feature where we can group the API based on functionality/portals/...We know we can categorize APIs but we can see them in categories only in the Dev portal. We were looking for the same feature in Publisher where we can group the APIs in the publisher and access that group during our development.

This can be done with API Category feature.
https://apim.docs.wso2.com/en/latest/reference/customize-product/customizations/customizing-the-developer-portal/customize-api-listing/api-category-based-grouping/


Not able to delete one tag of the image when there are multiple tags for the same repo with the same image-id.

acorn version v0.7.0-alpha1-16-g9007dba3+9007dba3
Steps to reproduce the problem:
Not able to delete one version of the image when image-id is the same for all the versions.
test-image                                                latest       a3aa847600bc
test-image                                                v1           a3aa847600bc
test-image                                                v2           a3aa847600bc

% acorn images rm test-image
✗ ERROR: deleting test-image: unable to delete test-image (must be forced) - image is referenced in multiple repositories

% acorn images rm test-image:latest
✗ ERROR: deleting test-image:latest: unable to delete test-image:latest (must be forced) - image is referenced in multiple repositories

% acorn images rm -f test-image:latest
test-image:latest

This results in all the versions of the image being deleted (latest,v1 and v2)

Figured that it's a regression introduced by https://github.com/acorn-io/acorn/pull/1327

Tested with PR
$ acorn images
REPOSITORY                       TAG       IMAGE-ID
bar/baz                          latest    fbe92ed033d0
bar/baz                          v1        fbe92ed033d0
bar/baz                          v2        fbe92ed033d0
foo                              latest    b2b380d31331
foo                              v2        b2b380d31331
harbor.thklein.dev/acorn/issue   <none>    95ead9eb9914
harbor.thklein.dev/acorn/issue   latest    84621d2e228e

# default to :latest
$ acorn image rm foo
foo

$ acorn images
REPOSITORY                       TAG       IMAGE-ID
bar/baz                          latest    fbe92ed033d0
bar/baz                          v1        fbe92ed033d0
bar/baz                          v2        fbe92ed033d0
harbor.thklein.dev/acorn/issue   <none>    95ead9eb9914
harbor.thklein.dev/acorn/issue   latest    84621d2e228e
foo                              v2        b2b380d31331

# remove a single tag from a multi-tag image
$ acorn image rm bar/baz:v1
bar/baz:v1

$ acorn images
REPOSITORY                       TAG       IMAGE-ID
harbor.thklein.dev/acorn/issue   <none>    95ead9eb9914
harbor.thklein.dev/acorn/issue   latest    84621d2e228e
foo                              v2        b2b380d31331
bar/baz                          latest    fbe92ed033d0
bar/baz                          v2        fbe92ed033d0

# default to :latest even if there is a reg/repo tag around
$ acorn image rm harbor.thklein.dev/acorn/issue
harbor.thklein.dev/acorn/issue

$ acorn images
REPOSITORY                       TAG       IMAGE-ID
harbor.thklein.dev/acorn/issue   <none>    95ead9eb9914
foo                              v2        b2b380d31331
bar/baz                          latest    fbe92ed033d0
bar/baz                          v2        fbe92ed033d0

# if there's no :latest, remove the reg/repo tag/image
$ acorn image rm harbor.thklein.dev/acorn/issue
harbor.thklein.dev/acorn/issue

$ acorn images
REPOSITORY   TAG       IMAGE-ID
foo          v2        b2b380d31331
bar/baz      latest    fbe92ed033d0
bar/baz      v2        fbe92ed033d0

While testing this, I figured that the output of acorn image rm is not concise, i.e.

We don't differentiate between untag and delete
We output the arg, not the actually used tag, i.e. we should return foo:latest if we defaulted to :latest and that matched

Here's an example output from docker:
docker image rm busybox
Alias tip: dirm busybox
Untagged: busybox:latest
Untagged: busybox@sha256:b5d6fe0712636ceb7430189de28819e195e8966372edfc2d9409d79402a0dc16
Deleted: sha256:7cfbbec8963d8f13e6c70416d6592e1cc10f47a348131290a55d43c3acab3fb9
Deleted: sha256:baacf561cfff825708763ce7ee4a18293716c533e6ece3bd39009a5fb3c804d2

-> While implementing this, I figured that there is no good way to do so without
a) Putting too much smarts on the CLI side (pkg/cli/images_rm.go) where it has to check if the image was deleted and if not compare the image.Tags slices before/after
b) Changing the Client interface
c) Using fmt.Print in pkg/client

Tested with acorn version v0.7.0-alpha1-49-gf5e99d78+f5e99d78
Able to delete one tag of an image when there are multiple tags with same image-id
I see "Untagged" message when deleting even tagged versions of the image which is very confusing. Will track this behavior in a different bug.


Add common components

close #4

Creating a draft pull request so I don't forget it exists

still need to change  @add-working-dir-input into @v6 once the updated actions are released; but I will merge this PR already so I can test the build action in the @add-working-dir-input branch.


Incorrect tooltip with overloaded function names is shown on mouse hover with Ctrl pressed over another function name in sql editor

Description
When hovering mouse over a function and then move mouse over another one, the tooltip still shows info about the first function.
It can be confusing for a user, especially when function names are similar.
Steps to reproduce
use this code:
select regexp_substr('1000', '1\d', 1, 1, 'i', 1);

select regexp_replace ('hello_world', 'hello', '');

press Ctrl and hold it
hover mouse pointer over the name of the first function name: regexp_substr
move mouse pointer over the list of overloaded functions, mouse mouse down through the items of the list and finally move the mouse pointer out of the list (to the empty space in sql editor)
don't release Ctrl key
move mouse pointer over the second function name: regexp_replace
move mouse ponter within the limits of the word "regexp_replace" in sql editor and see how the tooltip is blinking, repainting, and moving its x-y coordinates, following the mouse pointer location (it is usually not desired due to high strain on eyes)
notice the list of overloaded functions is shown for wrong function (regexp_replace expected, not regexp_substr)
now you can release the Ctrl key

https://github.com/user-attachments/assets/8e6414f5-eb05-4019-8b43-b491fa76e7c0
DBeaver Version
Community Edition <IP_ADDRESS>408012322
Operating System
Windows 10
Database and driver
PostgreSQL 16
PostgreSQL JDBC Driver 42.7.2

Thank you for bug report.


Record CPU and walltime from progress monitor in Verifier

Record CPU and walltime from progress monitor in Verifier. This way, we are able to know walltime and cpu time for both select and non-select queries

Assign back to me after you address my comments.

Address the reviews

Addressed @electrum  comments

Looks good


feat: ATL-6983 ZIO Stream Kafka PoC in background jobs

Description:
ATL-6983

coverage: 48.537% (+0.03%) from 48.504%
when pulling 16aeef005cef232fe47575414ba42835207c77bf on feat/ATL-6983-zio-stream-kafka-poc-in-connect-bg-jobs
into dc50537f0861f1214101fde6adfdc2e1888384e8 on main.


opencv program to read and display image

Please create a pull request of your opencv projects on this repository : https://github.com/qxresearch/All-OpenCV-Projects


Update Return Data Documentation

Return data is poorly defined and doesn't provide much information at all on the structure of the returns from the API - This will make it hard for Front-End development and debugging.
TODO - Add documentation that clearly describes return data type and format

More broadly, I'd like to have documentation on each endpoint: Headers required, body required, URL params, return type and format, etc.
This will help future users if people want to use the API


Web config sends out garbage string descriptors during device enumeration

The string descriptors for web config interface are not in the correct format and this might be the cause of web config not functioning on Linux due to repeated re-request of these descriptors before finally giving up and hang.

Hi @dogtopus ,
This is a known issue.  We are looking into some fixes that may solve this and allow for configuration on both Linux and Mac.
As a note - This project is no longer maintained. Please check out the Community Edition project and post your issue there if it continues. The Community Edition issues section can be found here: https://github.com/OpenStickFoundation/GP2040-CE/issues


Add translation for "Component State"

https://github.com/reactjs/es.reactjs.org/issues/4#issuecomment-460445652

Thank you very much for the corrections @carburo! I just committed the suggestions.

Ok, merging!


disable lint check on draft prs

no sarif annotations while drafted

likely redundant - for future reference, you can toggle annotations by pressing A in the files screen


Hawaii - Receiver Onboarding

What You Need to Know

[ ] Terms of Service agreement signed
[ ] Kickoff meeting completed
Megan Light is currently POC on helping SR and RS schedule a receiver kickoff with Hawaii. See comment below for email thread.
[ ] Receiver intake form completed: [paste info below this]

To Do

[ ] create individual tickets for this epic
[ ] RS team member who originates this ticket: add any dependencies, as needed
[ ] RS team member who originates this ticket: complete the following when creating this Issue.

[ ] Pipelines: Onboarding & Operations (New)
[ ] Labels: "Onboarding-ops" and "Receiver"
[ ] Epics: [select the epic for this receiver we're onboarding]

Megan L. emailed Hawaii on 12/16/2022:

Light Mueller, Megan (CDC/DDID/NCEZID/DPEI)
jonathan.johnson<EMAIL_ADDRESS>Judd, Michael (CDC/DDPHSS/OD);Xiong, Bernadette (CDC/DDPHSS/OD/HITSSU) (CTR);Wallace, Jayna (CDC/DDPHSS/OD/HITSSU)
Hi Jonathan and Denise,
I was forwarded the thread below, and am happy to hear you are reaching out to onboard for SimpleReport and ReportStream. I primarily work on the ELC HIS activities, but also work closely with USDS to help get ReportStream connections set up with public health departments.
Usually, the first step is to set up a kickoff call to walk you through what ReportStream is, how it works, what onboarding looks like, etc. As part of that, we also talk about SimpleReport so you can understand how it works for providers and facilities in your state. If you are already familiar with SimpleReport and ReportStream and want to move straight into onboarding, I am happy to facilitate that as well. Would you all prefer a kickoff call, or to move right into what is needed for onboarding?
Thanks!
Megan

Megan Light Mueller, MPH
CDC/NCEZID/DPEI
<EMAIL_ADDRESS>

Closing, work completed.


S3 state bucket default encryption using SSE-KMS, but objects still encrypted with SSE-S3?

Hello,
I tested terragrunt 0.38.12 (on terraform 0.12.31) in one of our AWS environments, applied the new state bucket settings as terragrunt prompted me to do so (WARN[0003]   - Bucket Server-Side Encryption, per the new feature to enforce those bucket settings, that appeared in v0.37.0), so that the bucket was in fact updated to use default encryption of SSE-KMS with the default aws/s3 key.
Now when I apply a new config (terragrunt apply), the state file is encrypted with SSE-S3, instead of SSE-KMS. Is this expected? I presume this would happen if the state saving S3 PUT request specifies SSE-S3 instead (overriding the bucket default of SSE-KMS).
I am currently not specifying a KMS key id in the remote state settings, so looking at https://terragrunt.gruntwork.io/docs/reference/config-blocks-and-attributes/#remote_state, I expected that to default to SSE-KMS (with key aws/s3), but it looks like we are getting SSE-S3 instead.
Any pointers are greatly appreciated.
Thanks,
Jukka

Furthermore, if we specify:
remote_state {
  backend = "s3"

  config = {
    encrypt              = true
    ...
    bucket_sse_algorithm = "AES256"
  }
}

and we apply with that config, we still get the WARN to update the bucket SSE settings on every plan.
Looking at https://github.com/gruntwork-io/terragrunt/blob/3a23346e08ee574a4a3a23112350ea625d577a02/remote/remote_state_s3.go#L1080 terragrunt seems to only accept aws:kms for the algorithm, to detect SSE being enabled?
Is the use of SSE-S3 discouraged here?
I myself find the terragrunt default bucket SSE settings vs what is actually being used by terraform somewhat confusing.

Hi, state file, in the end, is managed by terraform which encrypts objects with different encryption

All right, that explains the behavior.

@denis256 I believe this is a bug. I am using the latest versions of terragrunt, terraform, AWS provider.
I create a terraform state bucket outside of terragrunt with SSE-S3 encryption enabled, I store the state for this bucket in itself, I then plan in terragrunt and terragrunt tells me I still need to apply encryption configuration, and if I say y then it will apply KMS encryption configuration instead of SSE-S3/AES256. Am I using bucket_sse_algorithm incorrectly? Or is it not being respected in terragrunt for some reason?
root.hcl
remote_state {
  backend = "s3"
  generate = {
    path      = "remote_state.tf"
    if_exists = "overwrite_terragrunt"
  }
  config = {
    region               = "us-west-2"
    bucket               = "defensestorm-terraform-new"
    key                  = "${path_relative_to_include()}/terraform.tfstate"
    encrypt              = true
    bucket_sse_algorithm = "AES256"
  }
}

terragrunt.hcl

# defaults for this cloudposse module are very good already, including server side encryption of the bucket
inputs = {
  allow_encrypted_uploads_only = true
  allow_ssl_requests_only      = true

  bucket_name        = "terraform-new"
}

terraform {
  source = "git@github.com:cloudposse/terraform-aws-s3-bucket.git?ref=3.0.0"
}

include "root" {
  path           = find_in_parent_folders("root.hcl")
  expose         = true
  merge_strategy = "deep"
}

kevcube@computer s3 % terragrunt plan
WARN[0000] No double-slash (//) found in source URL /cloudposse/terraform-aws-s3-bucket.git. Relative paths in downloaded Terraform code may not work.
WARN[0005] The remote state S3 bucket terraform-new needs to be updated:  prefix=[/Users/kevcube/terragrunt/s3]
WARN[0005]   - Bucket Server-Side Encryption             prefix=[/Users/kevcube/terragrunt/s3]
WARN[0005]   - Bucket Root Access                        prefix=[/Users/kevcube/terragrunt/s3]
WARN[0005]   - Bucket Enforced TLS                       prefix=[/Users/kevcube/terragrunt/s3]
Remote state S3 bucket terraform-new is out of date. Would you like Terragrunt to update it? (y/n) n

Hi,
I will take a look on implementation and some more tests

@denis see #2337 please
I was wrong, the bucket doesn't actually change the encryption from AES256 to aws:kms after we've set the bucket_sse_algorithm, however it does still prompt us to update it on every run.
It just updates it to the same thing that it is already set to.

@bradonkanyid yeah that's what I have addressed in #2337, hoping for a review soon. Otherwise you can just say "yes" to bucket changes and nothing will happen, but this is confusing my team.
cc @denis256

Fix has been merged in version 0.43.2. It solved the issue. Thanks @kevcube

Hi all!
I've tried terragrunt version v0.44.5 with enabled properties bucket_sse_algorithm = "aws:kms" and bucket_sse_kms_key_id = my_kms_key and encryption on state file present as SSE-S3, not SSE-KMS.
Default encryption for the bucket is SSE-KMS and my_kms_key.
Even if I manually change the encryption of tf-state file to KMS and the proper key, after applying tf-state encryption is rollback to SSE-S3.
I'm trying to change encryption for tf-state files from SSE-S3 to SSE-KMS and customer-managed key.
Is the above relevant for this tread?

Hi all! I've tried terragrunt version v0.44.5 with enabled properties bucket_sse_algorithm = "aws:kms" and bucket_sse_kms_key_id = my_kms_key and encryption on state file present as SSE-S3, not SSE-KMS. Default encryption for the bucket is SSE-KMS and my_kms_key. Even if I manually change the encryption of tf-state file to KMS and the proper key, after applying tf-state encryption is rollback to SSE-S3. I'm trying to change encryption for tf-state files from SSE-S3 to SSE-KMS and customer-managed key. Is the above relevant for this tread?

See https://github.com/gruntwork-io/terragrunt/issues/2293#issuecomment-1263985974

I believe the main concerns and problems mentioned in this issue have been addressed, so I am closing this issue. Thanks everybody


chore(main): release 0.11.0

:robot: I have created a release beep boop
0.11.0 (2024-07-15)
Features

新增页面切换 transition 效果 (d7e1595)

Bug Fixes

解决 Windows 系统下项目路径解析错误的问题 (42c068a)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

v0.11.0
:sunflower:


Does anybody achieve the metric depth estimation on a custom dataset successfully?

Hi,
This post is just to discuss how to achieve metric depth estimation on a custom dataset, like I am using SCARED dataset. If anyone successfully fine-tune the model and achieve metric depth estimation, could you tell me which code did you modify?

I did it on the RealEstate10k unfortunately Depth does not exist on the dataset as such so evaluation is not possible but in general if you take an anecdotal look, it's pretty good.

@1ssb Have you seen results like this with the metric depth outdoor checkpoints?  With all of my experimentation it seems like the sky predictions are not good.  Although the relative depth predicts the sky and other "background" extremely well!

Hi @1ssb thanks for getting back to me!  I am using the outdoor checkpoints and just wondering if you (or anyone else) has seen similar results with the metric depth predictions?

Wow that's interesting about the metric vs relative thing.  I mean I could mask out the background using the relative depth network or a lightweight segmentation network but it seems like there should be a better way..

Oh this is interesting @Denny-kef I took a look at your image and it seems to me it is captured by either a fish eye lens or the image itself is a bit distorted as in to the eye itself it looks like the clouds are much closer than they actually are, this is a very cool thing as well. I am not sure I have an exact answer to this but it might as well be an OOD.

@Denny-kef Idk if this can help you in your task, but this is my personal interpretation:
authors report on the paper that they set the disparity value (inverse of depth) to 0 for all the pixels labeled as "sky" by a semantic segmentation model (see section 3.1 of the paper on arxiv). I haven't seen the implementation details of their code, but this can affect their training in different ways:

if the disparity value of 0 is considered as invalid and masked during training, it means that the MDE model is not trained to decode the original disparity/depth value for the sky
even if the disparity value of 0 is considered as "valid" and not masked to the model during training, the MDE model sees all the sky pixels as having the same disparity value (that is 0), so the learned features would not be expressive enough to eventually decode other values

That said, I think the features from their frozen encoder are really powerful for metric depth estimation, but I guess it would very hard to use them to produce correct values on an absolute scale for the sky.

The best solution that I found is this:

Retrieve the relative depth map as a secondary output from the metric depth estimation model.
Using that depth map (since it is much better at predicting the background) I was able to generate a binary mask to eliminate things like the sky from my metric depth results.

Importantly, these operations don't any significant time to the inference.

The best solution that I found for the "background" issue with metric depth estimation predictions is this:

Retrieve the relative depth map as a secondary output from the metric depth estimation model.
Using that depth map (since it is much better at predicting the background) I was able to generate a binary mask to eliminate things like the sky from my metric depth results.

Importantly, these operations don't add any significant time to the inference.

Bottomline: Don't try to predict skies or reflections.

I was not trying to predict skies but I was trying to remove them from the outputted depth map so they don't show up in the point cloud.  But yes do not try to predict the depth of the sky.

Hi @loevlie, if you are trying to detect the sky and remove it, you can try our relative depth models. The output value 0 from these models can be considered as the sky. Alternatively, you can use a pre-trained semantic segmentation model to detect the sky.

Hi @LiheYoung, yes that works very well!  Thank you!

@1ssb Have you seen results like this with the metric depth outdoor checkpoints? With all of my experimentation it seems like the sky predictions are not good. Although the relative depth predicts the sky and other "background" extremely well!

Could you kindly share with me the parameters you adjusted during fine-tuning? I've been encountering poor performance in my experiments with another dataset, and I've been struggling to resolve the issue. The details of the problem are as follows.https://github.com/LiheYoung/Depth-Anything/issues/172#issue-2292062398

@1ssb Have you seen results like this with the metric depth outdoor checkpoints? With all of my experimentation it seems like the sky predictions are not good. Although the relative depth predicts the sky and other "background" extremely well!

Hi Denny, would you be able to explain how you got the metric depth working and your output depth images? I've been trying to run the metric outdoor model on my custom dataset but have been running into a lot of issues. Any help would be greatly appreciated!

I did it on the RealEstate10k unfortunately Depth does not exist on the dataset as such so evaluation is not possible but in general if you take an anecdotal look, it's pretty good.

Hi @1ssb ,
Are you able to train metric depth estimation on a dataset without depth maps (labels) ?
Could you please share more details about your training trial?

Hi, I never train, but test time fine tune using a plug and play method on the RealEstate10k. Kindly remember that Realestate10k is not RGBD dataset, but you can use the triangulation method and use the control points to rescale the predictions directly. It's not very neat but it's the best you can do without retraining from scratch.
Best
Subhransu

From: Shilpa Ullas @.>
Sent: Thursday, June 27, 2024 6:04:17 PM
To: LiheYoung/Depth-Anything @.>
Cc: Subhransu Bhattacharjee @.>; Mention @.>
Subject: Re: [LiheYoung/Depth-Anything] Does anybody achieve the metric depth estimation on a custom dataset successfully? (Issue #68)
I did it on the RealEstate10k unfortunately Depth does not exist on the dataset as such so evaluation is not possible but in general if you take an anecdotal look, it's pretty good.
Hi @1ssbhttps://github.com/1ssb ,
Are you able to train metric depth estimation on a dataset without depth maps (labels) ?
Could you please share more details about your training trial?
—
Reply to this email directly, view it on GitHubhttps://github.com/LiheYoung/Depth-Anything/issues/68#issuecomment-2194051505, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AJWHFEH236HROYVD4WEG3DTZJPBQDAVCNFSM6AAAAABCVJF6ACVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCOJUGA2TCNJQGU.
You are receiving this because you were mentioned.Message ID: @.***>

Hi, I never train, but test time fine tune using a plug and play method on the RealEstate10k. Kindly remember that Realestate10k is not RGBD dataset, but you can use the triangulation method and use the control points to rescale the predictions directly. It's not very neat but it's the best you can do without retraining from scratch. Best Subhransu
…
________________________________ From: Shilpa Ullas @.> Sent: Thursday, June 27, 2024 6:04:17 PM To: LiheYoung/Depth-Anything @.> Cc: Subhransu Bhattacharjee @.>; Mention @.> Subject: Re: [LiheYoung/Depth-Anything] Does anybody achieve the metric depth estimation on a custom dataset successfully? (Issue #68) I did it on the RealEstate10k unfortunately Depth does not exist on the dataset as such so evaluation is not possible but in general if you take an anecdotal look, it's pretty good. Hi @1ssbhttps://github.com/1ssb , Are you able to train metric depth estimation on a dataset without depth maps (labels) ? Could you please share more details about your training trial? — Reply to this email directly, view it on GitHub<#68 (comment)>, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AJWHFEH236HROYVD4WEG3DTZJPBQDAVCNFSM6AAAAABCVJF6ACVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCOJUGA2TCNJQGU. You are receiving this because you were mentioned.Message ID: @.***>

Hi @1ssb ,
Could you please share more details about how you finetune at test time? Many thanks!


Transition spec

This copies the new mx_platform_api_beta.yml spec into the
mx_platform_api.yml spec. Once we have transitioned references away
from the "beta" spec, we will delete the "beta" spec and be fully
transitioned to the new spec.

:shipit:


Location Query Service and Tests

In this PR I implemented a service that wraps the location query API found at https://nominatim.openstreetmap.org/search/search.php?q={location}&format=jsonv2 .
Closes #8.

NICE!


[question] recommended settings for broker

I think the documentation could be improved by giving default suggestions for the following settings, for ease of giving things a spin when you're just learning MQTT (or just prefilling defaults).
"Reconnect period: Milliseconds between two reconnection tries"
I don't know what to put here.  10? 1000? How do I know what's a reasonable setting?
"Prefix: The prefix where all values are published"
How do I choose this? Is it totally arbitrary? Does it have to match anything somewhere else?
"QoS: Quality Of Service (check MQTT specs) of outgoing packets"
I get the basic idea of it, but I don't know what to prefer here.
"Retain: The retain flag of outgoing packets"
Clean: Sets the clean flag when connecting to the broker
Store: Enable/Disable persistent storage of packets (QoS > 0)"
How do I know what to choose for these?  Why should I prefer one or the other?

I think this is a dup of #147 that is more generic. CLosing this and continue there. I think the solution to this is to provide some defaults values and improve the docs

I think this is a dup of #147 that is more generic. CLosing this and continue there. I think the solution to this is to provide some defaults values and improve the docs


add(script): added stop script. Now the start script also log

add(script): added stop script. Now the start script also log in acestream.log

Useful addition, thanks!


Move / dedupe presentation exchange test vectors in web5-kt

we have quite a few test vectors for presentation exchange sitting in web5-kt's resources folder that we should move to web5-spec submodule.
this will help test the other web5 sdk's against the edge cases laid out in these test vectors, and serve as a start of discussion when a test vector tests fails in one web5-sdk but passes in another web5-sdk.

@jiyoontbd can we close this as a dupe of #171 ?

yea just edited the post in #171 ! closing


Appstudio update test-component-pac-eljk

Pipelines as Code configuration proposal
To start the PipelineRun, add a new comment with content /ok-to-test
For more detailed information about running a PipelineRun, please refer to Pipelines as Code documentation Running the PipelineRun
To customize the proposed PipelineRuns after merge, please refer to Build Pipeline customization

Pipelines as Code CI/test-component-pac-eljk-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
6 seconds
init

✅ Succeeded
11 seconds
clone-repository

✅ Succeeded
25 seconds
build-container

✅ Succeeded
46 seconds
inspect-image

✅ Succeeded
21 seconds
deprecated-base-image-check

✅ Succeeded
15 seconds
clair-scan

✅ Succeeded
10 seconds
sbom-json-check

✅ Succeeded
32 seconds
clamav-scan

✅ Succeeded
17 seconds
label-check

✅ Succeeded
11 seconds
show-sbom

✅ Succeeded
12 seconds
show-summary


Create React Admin version of AddIntersectionsTable

The Intersections table is currently accessible from the Artifacts table.

There is a button:

This is the current view:

https://github.com/Call-for-Code-for-Racial-Justice/Truth-Loop/blob/main/server/admin/src/components/AddIntersectionsTable.vue

Anyone have concerns about changing up how we manage the "intersections"? I was thinking of just putting this on the Artifacts' edit view.

@markstur something like this?

Thanks for providing nice screenshot details.  I was wondering what the old one looked like.
I knew the old one did lack something w.r.t. the workflow of how you add, then link. I think your suggestion is a step in the right direction.
In your new screenshots, I don't see them both "on the Artifacts edit view", but I'm assuming those two dialogs would both be part of one. So you could add intersections while doing the create/edit.
I think that would be an improvement. As long as you have the referential integrity DB part correct (i.e., it works), then I'd say there are no current users that would object (I wrote that first UI). So go ahead.
One little thing to keep in mind.  I was wondering... if I am entering an artifact, and the geography isn't there yet. Would it make sense to have a button to create a new geography.  I.e. add a new state called Puerto Rico.  That might be too much nested functionality but the original very relational point of view meant you'd have to leave and add this and that and then come back and link them.  It'd be real nice to do it all from here.  And to be fair, I'm saying that could be done later, but still think your suggestion looks good (I'll just miss the chain link icon).

I think we're on the same page as far as putting all the "intersections" on the view. My plan was to allow have the intersections added immediately rather than an explicit "Update" or "Apply" action. The thinking behind that is because it easy to add/remove intersections so if it is accidentally added/deleted it can be corrected w/o much effort.
With that in mind, we wouldn't be able to add/remove intersections during "Create" because there would be no artifact to intersect. That said, maybe the "Add" view doesn't go to the Table after successfully adding but instead leaves them in the edit view to continue adding intersections?
As far as adding a way to add geographies, officials, etc... on the fly, I am curious if we want to develop/maintain two different ways to accomplish the same goal.
Good conversation for sure. I'll focus on getting all of this actually added and working then maybe we can create some issues for the workflow improvements.

Tweaked the UI just a tad from the original. I'm ready to start handling the add/delete.


enc-dec triton backend support

Hi is there any update on when enc-dec models like T5 will get the TRT-LLM Triton backend support? Posting an issue for awareness and just wanted to know if its still being planned. Thanks in advance!
https://github.com/NVIDIA/TensorRT-LLM/discussions/424#discussioncomment-7732258

Hi @shannonphu , yes we're working on it. Right now it's at the stage of adding the C++ runtime. Tentative date for Triton enc-dec support is around mid to late January. Thanks for your patience

Hi @shannonphu , yes we're working on it. Right now it's at the stage of adding the C++ runtime. Tentative date for Triton enc-dec support is around mid to late January. Thanks for your patience

is it also included continuous batching?

is it also included continuous batching?
Our current plan is to reach there by steps: (1) C++ runtime (2) regular Triton support (3) continous batching. Eventually we want to enable continus batching, but for the mid to late January release it's more likely to only have (1) and (2), with (3) coming right after it

@symphonylyh Could share if theres an update on this?

Hi is there an update for this?

Hi @shannonphu , @sihanwang41 , @mlmonk , @shixianc ,
We have been actively working on this support but finding the amount of work is more than expected since we want to have a good implementation to support enc-dec and in general such 2-stage pipeline.
May I use this thread to collect your feedback so we can understand your need and prioritize better. I know @sihanwang41 specifically asked about continuous batching, i.e., inflight batching, but others didn't share the request info. Can you reply by describing if any of (1), (2), (3) would be helpful and can unblock you first:
(1) a Triton Python backend support to run enc-dec model
(2) a C++ runtime (no Triton) to run enc-dec model, without inflight batching
(3) a Triton C++ backend to run enc-dec model, without inflight batching
(4) a Triton C++ backend, with paged kv cache and inflight batching for enc-dec <-- final goal
Thanks

@symphonylyh  Thanks for the update! Starting with (3) would unblock our team.

@symphonylyh Thanks for the update! Starting with (3) would unblock our team.
May I assume this would also have the classic dynamic batching supported?

Got it, thanks for the input.
By dynamic batching, do you mean the Triton's dynamic batching that has nothing to do with the inflight/continuous batching concept. If so, yes.

@symphonylyh (1) and/or (3). I am not super clear on the difference between the Python vs C++ backend. I was using this to build the engine https://github.com/NVIDIA/TensorRT-LLM/blob/main/examples/enc_dec/README.md

We have been able to use Triton with enc_dec models, so I'm not sure what
the difference that and (1) is. We find that the TPS for that
implementation is quite slow are looking for ways to make it faster.
Agree that the end goal is (3).
On Fri, Feb 23, 2024, 5:31 PM Shannon Phu @.***> wrote:

@symphonylyh https://github.com/symphonylyh (1) and/or (3). I am not
super clear on the difference between the Python vs C++ backend. I was
using this to build the engine
https://github.com/NVIDIA/TensorRT-LLM/blob/main/examples/enc_dec/README.md
—
Reply to this email directly, view it on GitHub
https://github.com/NVIDIA/TensorRT-LLM/issues/800#issuecomment-1962087195,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACSI76N7HW6ZZB2ANBZNQBDYVEKFBAVCNFSM6AAAAABBMET6RGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNRSGA4DOMJZGU
.
You are receiving this because you were mentioned.Message ID:
@.***>

@mlmonk Oh interesting, I was under the impression that we just couldn't serve T5 models on Triton yet because the TRT-LLM backend wasn't ready for it yet.

@symphonylyh @shannonphu We have been able to use the Flan-T5 with Triton. I believe this is (1). You can reproduce it here. Note that this is much older version of both libraries when Flan-T5 was not officially supported.
Like @shixianc mentioned, (3) would unblock us and (4) would the ideal state. It would be great if you could share how far along you are with the (3) release.

hey @symphonylyh , do you have any updates on the progress?

@symphonylyh,  any progress?

Hello, @symphonylyh . Is there any progress on any of (1-4) ?

We would love (1)

Thanks for the update! This is excellent news, I'm sure it was a lot of effort to make it happen.

@HamzaG737 it's full-fledged now. For (1) Triton backend, you can follow the guide here: https://github.com/triton-inference-server/tensorrtllm_backend/blob/main/docs/encoder_decoder.md.
Also, closing this issue as support has been added


[incubator/jaeger] Deploy jaeger all-in-one image as default

Is this a BUG REPORT or FEATURE REQUEST? (choose one): FEATURE REQUEST

Version of Helm and Kubernetes: n/a
Which chart: incubator/jaeger
What happened:
Currently the default deployment of the chart creates a production deployment of Jaeger based on a Cassandra cluster.
What you expected to happen:
Jaeger also provides an "all-in-one" image which combines the capabilities of the individual binaries (agent, query+u and collector) into a single binary using an in-memory storage implementation. This image is only intended for demo/dev/test purposes.
We are considering decommissioning the current templates from the https://github.com/jaegertracing/jaeger-kubernetes repo, so users would only use this helm chart to install jaeger into kubernetes - but this means it would also need to offer a way to install the "all-in-one" image, and it seems like this would be the ideal default installation as it is fast.
The issue is that this would change the default behaviour - but it would make the choice of storage now explicit, rather than one (cassandra) being deployed as default.
cc @dvonthenen @mikelorant @pavelnikolov

I can fix this. As it is a breaking change, it would require major version upgrade.
Do you need it urgently?
Sent from my phone

On 22 May 2018, at 01:32, Gary Brown<EMAIL_ADDRESS>wrote:
Is this a request for help?:
Is this a BUG REPORT or FEATURE REQUEST? (choose one): FEATURE REQUEST
Version of Helm and Kubernetes: n/a
Which chart: incubator/jaeger
What happened:
Currently the default deployment of the chart creates a production deployment of Jaeger based on a Cassandra cluster.
What you expected to happen:
Jaeger also provides an "all-in-one" image which combines the capabilities of the individual binaries (agent, query+u and collector) into a single binary using an in-memory storage implementation. This image is only intended for demo/dev/test purposes.
We are considering decommissioning the current templates from the https://github.com/jaegertracing/jaeger-kubernetes repo, so users would only use this helm chart to install jaeger into kubernetes - but this means it would also need to offer a way to install the "all-in-one" image, and it seems like this would be the ideal default installation as it is fast.
The issue is that this would change the default behaviour - but it would make the choice of storage now explicit, rather than one (cassandra) being deployed as default.
cc @dvonthenen @mikelorant @pavelnikolov
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

@pavelnikolov No its not urgent - initially just checking if there were any objections to the idea.
Any idea of a timeframe? Just so we can plan for migration from the templates.

I don't mind adding the functionality, but question whether or not it should be the default behavior.
I guess it really comes down to what the primary goal of the chart is...

if the chart is primarily to enable demos, then this makes sense.
if it's to deploy a test/dev environment, that maybe the all-in-one is suitable. However, if you are going to have a dev/test environment, you might want to actually deploy a production-like environment.
if we want to support production type workloads, then I think it shouldn't be the default.

My take on it is to have the option to deploy the all-in-one but have it be an option and not be the default.

I’ll try to do it next week or the week after.
Sent from my phone

On 22 May 2018, at 06:54, Gary Brown<EMAIL_ADDRESS>wrote:
@pavelnikolov No its not urgent - initially just checking if there were any objections to the idea.
Any idea of a timeframe? Just so we can plan for migration from the templates.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

I have no objection and no strong opinion here. I happy for the all-in-one to be default, since any production deployment would include at least 1 values file to override values (we use 6 values files, some of which change based on environment).
Also the only time you use:
helm install incubator/jaeger

Is when you are developing/testing.
In all other cases (ideally) you will have some kind of CI/CD pipeline in place.
Sent from my phone

On 22 May 2018, at 07:03, David vonThenen<EMAIL_ADDRESS>wrote:
I don't mind adding the functionality, but question whether or not it should be the default behavior.
I guess it really comes down to what the primary goal of the chart is...
if the chart is primarily to enable demos, then this makes sense.
if it's to deploy a test/dev environment, that maybe the all-in-one is suitable. However, if you are going to have a dev/test environment, you might want to actually deploy a production-like environment.
if we want to support production type workloads, then I think it shouldn't be the default.
My take on it is to have the option to deploy the all-in-one but have it be an option and not be the default.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

@dvonthenen Understand your points - don't have a strong opinion one way or the other. Main reason for suggesting it as default is that in other (production) cases it is likely that user would want to specify their preferred storage and provide additional customisation options. But main requirement is to have the all-in-one available as an option.

If you guys agree on default vs not default, I’ll send a PR soon.
Sent from my phone

On 22 May 2018, at 07:15, Gary Brown<EMAIL_ADDRESS>wrote:
@dvonthenen Understand your points - don't have a strong opinion one way or the other. Main reason for suggesting it as default is that in other (production) cases it is likely that user would want to specify their preferred storage and provide additional customisation options. But main requirement is to have the all-in-one available as an option.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

@yurishkuro @jpkrohling @pavolloffay Any preference for whether chart should install all-in-one as the default?

I don't know how people are using the chats. If it is for the production or just to try things. So I don't have a strong opinion. More important is if the chart supports all-in-one.
all-ine-one can be a sensible default. Then ES or cassandra should require some configuration.

+1 to have the all-in-one as the default. For a production setup, I would expect the operator to read the docs and make a decision about the proper backing storage for their use case, so, they would need to dig into the configuration options anyway.

@unguiculus any preference?

@pavelnikolov When adding the all-in-one support, just want to make sure you will be using a daemonset for the agent? The current template in jaeger-kubernetes repo doesn't use a daemonset, but was something we were considering changing. So would be good to sort that out as part of including it in the helm chart.

Ok, I'll make sure that the agent is a daemonset and the rest of the components are in the all-in-one deployment.

I'm testing these charts out now so I figured I'd provide consumer input;
Having all 3 options available would be very nice;

Deploying this w/ Elasticsearch on my laptop via minikube has been a bit painful. I'm not sure what's up right now but I can't get any of the fabric8 discovery to connect the masters. I've used this in staging/prod envs so maybe this is a minikube limitation somewhere.
Deploying this w/ Cassandra is far less painful and a nice way to get this up and running very quickly.
In the end while I'm POC'ing this I'd love to not have to run any JVM containers or as few as possible as they take an obscene amount of time to terminate in any k8s environment (laptop, servers). I'm about to test setting the storage to memory but the values.yaml states the two options are cassandra and elasticsearch.

So, TLDR; I'd love all 3. I'd like memory for quick POC work and to throw to developers that are locally tracing via minikube, and the other two are great to quickly get things going on a real cluster.
Thanks for all your work.

I’m sorry for the delay on this. I’ll start working on the all-in-one very soon.
Sent from my phone

On 13 Jun 2018, at 08:33, Mike Johnson<EMAIL_ADDRESS>wrote:
I'm testing these charts out now so I figured I'd provide consumer input;
Having all 3 options available would be very nice;
Deploying this w/ Elasticsearch on my laptop via minikube has been a bit painful. I'm not sure what's up right now but I can't get any of the fabric8 discovery to connect the masters. I've used this in staging/prod envs so maybe this is a minikube limitation somewhere.
Deploying this w/ Cassandra is far less painful and a nice way to get this up and running very quickly.
In the end while I'm POC'ing this I'd love to not have to run any JVM containers or as few as possible as they take an obscene amount of time to terminate in any k8s environment (laptop, servers). I'm about to test setting the storage to memory but the values.yaml states the two options are cassandra and elasticsearch.
So, TLDR; I'd love all 3. I'd like memory for quick POC work and to throw to developers that are locally tracing via minikube, and the other two are great to quickly get things going on a real cluster.
Thanks for all your work.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

@mikejk8s You have no idea how valuable your feedback is: thank you for sharing it!

We use the jaeger chart in production but it would be nice to have an option to use the all-in-one image in dev environment (minikube).


Updated to use the download plugin

I'm new to Python, so I'm not sure why the download will fail on private trackers. It gives the "maybe a login page" error.
I'm using the same config file I used on my router (with transmission plugin) and it failed. I'm currently running FlexGet on Windows. The task has form plugin.

I think I found out why the download failed. It seems that the urlrewrite is not working.

If i remove the usage of download plugin, url rewrite plugin will kick in. Help from experience flexget coder needed.

Seems pretty good to me at first glance, is there something that is still not working as expected?

Everything is working fine.
Sent from my Samsung device
-------- Original message --------
From: Chase Sterling<EMAIL_ADDRESS>Date: 23/04/2015  5:40 am  (GMT+08:00)
To: Flexget/Flexget<EMAIL_ADDRESS>Cc: programatix<EMAIL_ADDRESS>Subject: Re: [Flexget] Updated to use the download plugin (#481)
Seems pretty good to me at first glance, is there something that is still not working as expected?

Reply to this email directly or view it on GitHub:
https://github.com/Flexget/Flexget/pull/481#issuecomment-95346168


[GWS][modify] settings icon

Coverage increased (+0.002%) to 82.915% when pulling 3b587a901026c3708b17e044299f66ba9d634b30 on tany:gws-settings-icon into 923f51ec18c0f2cf8904502dd44e87f17d59fd42 on shirasagi:dev/gws.


set brightness on dimmer

Hello,
I've a problem to set brigtness on my dimmer. On/Off works fine. But "colorset" doesn't.
$tuya->setState([   "name" => "PV Dimmer",   "command" => "colorSet",   "value" => [     "brightness" => 10   ] ]);
Originally posted by @hochland123 in https://github.com/Aymkdn/tuyacloud-php/issues/5#issuecomment-1178654542

Moving your issue to another thread.
Did you try the below:
$tuya->setState([
   "name" => "PV Dimmer",
   "command" => "brightnessSet",
   "value" => 50
]);

Hey, it works. Thank you a lot.


New Express 4 code does not work in production mode

because development flag encapsulates many middleware and express configuration in config/express.js
I can fix it when I get some time.

fixed in latest pull request

I have moved away from writing a lot of javascript and I want to see this library get the attention that it deserves. Would you be interested in becoming a collaborator on this project?

Yes, absolutely. Thank you!

My primary expertise is in Express, Sequelize and Node.js and I can help with the corresponding parts of the project more than the Angular part. I have some previous experience with Angular anyway.

Can we set up a google hangout? I'd like to collaborate with you and create a project roadmap.

Yes sure. I'd prefer text chat to voice conversation, however.

Okay I will chat you through google hangouts

Closing as this is fixed. :)

:shipit:


BUG: negative spacing is not supported, but check did not enforce that

Related to #2911 and #2917.
PR Checklist

[x] No API changes were made (or the changes have been approved)
[x] No major design changes were made (or the changes have been approved)
[ ] Added test (or behavior not changed)
[ ] Updated API documentation (or API not changed)
[ ] Added license to new files (if any)
[ ] Added Python wrapping to new files (if any) as described in ITK Software Guide Section 9.5
[ ] Added ITK examples for all new major features (if any)

@dzenanz thanks for doing this.
After reading Matt's and Brad's comments in #2917, and the need to at least read negative spacing images by third party applications, maybe commit d447f0452bb5ea92a555e630d05b57da535bd3a9 was thought to be a halfway compromise before the strict enforcement, as it is guarded by the FUTURE_LEGACY_REMOVE macro (but I still find the message inaccurate/contradicting the fact that setting the zero/negative spacing does actually take place).
For context, maybe some related discussion about zero/negative spacings are these (?):
https://itk.org/pipermail/community/2016-November/012275.html (newest reply from François)
https://discourse.itk.org/t/nifti-file-orientation-change-between-2019-08-26-and-itk-v5-1rc01/2594/12?
https://public.kitware.com/pipermail/insight-users/2016-October/054037.html
But reading negative (at least negative) spacing images (but immediately setting to an "equivalent" positive value) is still allowed, as Brad points in https://github.com/InsightSoftwareConsortium/ITK/issues/2917#issuecomment-984794713.
Thus, now I am not sure whether I unnecessarily opened a front with my PR and thoughts. Sorry, not the most lucid days.

Thus, now I am not sure whether I unnecessarily opened a front with my PR and thoughts. Sorry, not the most lucid days.

Jon, I am glad your efforts have gotten this issue straightened out. I recall encountering it messing around with SimpleITK and setting the spacing and getting a confusing error message.

@jhlegarreta I have "Allow edits and access to secrets by maintainers" enabled by default, and it is active for this PR too. Feel free to add an additional commits, or amend this one. Also, you can make a separate PR, possibly later. As the changes in this PR are minimal, I am happy to abandon it in favor of a more comprehensive one.

So the legacy remove build is doing its job :ok_hand:  and we have the first issue uncovered by it:
https://open.cdash.org/test/554773719
The exception seems to be thrown in here:
https://github.com/InsightSoftwareConsortium/ITK/blob/master/Modules/Registration/Metricsv4/test/itkImageToImageMetricv4Test.cxx#L597
So it looks likes some registration process does

@jhlegarreta I have "Allow edits and access to secrets by maintainers" enabled by default, and it is active for this PR too. Feel free to add an additional commits, or amend this one. Also, you can make a separate PR, possibly later. As the changes in this PR are minimal, I am happy to abandon it in favor of a more comprehensive one.

Done in 8df1ea4.

I guess we should change the metric exception testing to trigger it by something other than negative spacing.

Not sure about the positive spacing values that would still raise the metric exception, but there are a few other displacement field image properties that might raise an exception:

Still expect the legacy remove build failure.

Here it is #2923. Tested locally. Looks to be good enough to raise the exception.

This now passes tests. It is ready for reviewing and merging.


Autocorrect-bubble reversed

I tried something similar a while ago when implementing a messenger-functionality in our app. We came up with the same idea of reverting the tableview and then reverting the cells. So, the cell at the bottom contained a UITextField to enter the text to the other user. A problem we stumbled upon was the autocorrect-bubble:

In this example i just added a textfield to the cell to see what happens… if you type in something and autocorrect comes up with some better stuff, the bubble is reversed. We tried to walk through the view-hierarchy to rotate the bubble, but no solution was satisfying... do you have any idea how to take care of this?

Hi!
I guess that it might be solved this issue to turn off autocorrection with autocorrectionType.

This is "avoided", but not "solved" ;)

This is "avoided", but not "solved" ;)

It's exactly as you said.
I think UITableViewCell that contains UITextField is not a good UI structure in the first place.
Because UITableViewCell is reusable, it might be changed that's responder.
If you want that, put tapable UILabel to every cell.
When receiving UILabel tap event, calculating UILabel's position from tableView's superview.
Finally, put UITextField to that position.
A UITextFiled appears to be above that UITableViewCell.