micromamba feature does not install mamba, only configures shell

I added the micromamba feature to my python-debian based container, however, after the installation the /home/vscode/micromamba folder is practically empty, containing only the initialization script:
$ find /home/vscode/micromamba/
/home/vscode/micromamba/
/home/vscode/micromamba/etc
/home/vscode/micromamba/etc/profile.d
/home/vscode/micromamba/etc/profile.d/micromamba.sh
/home/vscode/micromamba/conda-meta

This is the build log:
2023-04-28 09:23:36.244Z: #19 0.702 ===========================================================================
2023-04-28 09:23:36.245Z: #19 0.702 Feature       : micromamba
#19 0.702 Description   : Installs micromamba, the fast cross-platform package manager.
#19 0.702 Id            : ghcr.io/mamba-org/devcontainer-features/micromamba
#19 0.702 Version       : 1.1.1
#19 0.702 Documentation : https://github.com/mamba-org/devcontainer-features/tree/main/src/micromamba
#19 0.702 Options       :
#19 0.702     VERSION="latest"
#19 0.702     ALLOWREINSTALL="false"
#19 0.702     CHANNELS=""
#19 0.702     PACKAGES=""
#19 0.702     ENVFILE=".devcontainer/env.yml"
#19 0.702     ENVNAME="testenv"
#19 0.702 ===========================================================================
2023-04-28 09:23:36.354Z: #19 0.781 Installing prerequisites for downloading micromamba...
#19 0.813 Downloading micromamba from https://micro.mamba.pm/api/micromamba/linux-64/latest...
2023-04-28 09:23:38.609Z: #19 2.931 Micromamba download complete.
#19 2.931 Micromamba executable installed.
2023-04-28 09:23:38.865Z: #19 3.324 Setting channel_priority to strict
2023-04-28 09:23:38.975Z: #19 3.434 Initializing Bash shell
2023-04-28 09:23:39.077Z: #19 3.471 Modifying RC file "/home/vscode/.bashrc"
#19 3.471 Generating config for root prefix "/home/vscode/micromamba"
#19 3.471 Setting mamba executable to: "/usr/local/bin/micromamba"
#19 3.471 Adding (or replacing) the following in your "/home/vscode/.bashrc" file
#19 3.471
#19 3.471 # >>> mamba initialize >>>
#19 3.471 # !! Contents within this block are managed by 'mamba init' !!
#19 3.471 export MAMBA_EXE="/usr/local/bin/micromamba";
#19 3.471 export MAMBA_ROOT_PREFIX="/home/vscode/micromamba";
#19 3.471 __mamba_setup="$("$MAMBA_EXE" shell hook --shell bash --prefix "$MAMBA_ROOT_PREFIX" 2> /dev/null)"
#19 3.471 if [ $? -eq 0 ]; then
#19 3.471     eval "$__mamba_setup"
#19 3.471 else
#19 3.471     if [ -f "/home/vscode/micromamba/etc/profile.d/micromamba.sh" ]; then
#19 3.471         . "/home/vscode/micromamba/etc/profile.d/micromamba.sh"
#19 3.471     else
#19 3.471         export  PATH="/home/vscode/micromamba/bin:$PATH"  # extra space after export prevents interference from conda init
#19 3.471     fi
#19 3.471 fi
#19 3.471 unset __mamba_setup
#19 3.471 # <<< mamba initialize <<<
#19 3.471
#19 3.473 Setting Bash shell to automatically activate
#19 3.483 Initializing zsh shell
#19 3.522 Modifying RC file "/home/vscode/.zshrc"
#19 3.522 Generating config for root prefix "/home/vscode/micromamba"
#19 3.522 Setting mamba executable to: "/usr/local/bin/micromamba"
#19 3.522 Adding (or replacing) the following in your "/home/vscode/.zshrc" file
#19 3.522
#19 3.522 # >>> mamba initialize >>>
#19 3.522 # !! Contents within this block are managed by 'mamba init' !!
#19 3.522 export MAMBA_EXE="/usr/local/bin/micromamba";
#19 3.522 export MAMBA_ROOT_PREFIX="/home/vscode/micromamba";
#19 3.522 __mamba_setup="$("$MAMBA_EXE" shell hook --shell zsh --prefix "$MAMBA_ROOT_PREFIX" 2> /dev/null)"
#19 3.522 if [ $? -eq 0 ]; then
#19 3.522     eval "$__mamba_setup"
#19 3.522 else
#19 3.522     if [ -f "/home/vscode/micromamba/etc/profile.d/micromamba.sh" ]; then
#19 3.522         . "/home/vscode/micromamba/etc/profile.d/micromamba.sh"
#19 3.522     else
#19 3.522         export  PATH="/home/vscode/micromamba/bin:$PATH"  # extra space after export prevents interference from conda init
#19 3.522     fi
#19 3.522 fi
#19 3.522 unset __mamba_setup
#19 3.522 # <<< mamba initialize <<<
#19 3.522
#19 3.535 Micromamba configured.
2023-04-28 09:23:39.227Z: #19 3.559
#19 3.559                                            __
2023-04-28 09:23:39.230Z: #19 3.559           __  ______ ___  ____ _____ ___  / /_  ____ _
#19 3.559          / / / / __ `__ \/ __ `/ __ `__ \/ __ \/ __ `/
#19 3.559         / /_/ / / / / / / /_/ / / / / / / /_/ / /_/ /
#19 3.559        / .___/_/ /_/ /_/\__,_/_/ /_/ /_/_.___/\__,_/
#19 3.559       /_/
#19 3.559
#19 3.559 Collect information..
#19 3.559 Cleaning index cache..
#19 3.559 Cleaning lock files..
#19 3.560 Cleaning tarballs..
#19 3.560 Cleaning packages..
#19 3.561 Done!
2023-04-28 09:23:39.378Z: #19 DONE 3.8s

Hey, thanks a lot for the feedback!
The directory /home/vscode/micromamba is for the Conda environment itself, whereas the micromamba executable is installed as a system binary in /usr/local/bin. (Having it installed system-wide allows it to not be dependent on any particular Conda env, and allowing it to create an initial Conda env from scratch.)
Given this information, does everything look in order?
Maybe we need to improve the docs to make this clear. (This repo is still quite young and to be improved.)

Just to be clear, this Feature installs micromamba, not mamba.
Could you check the micromamba command?

It's definitely confusing how the logs say mamba initialize when it's micromamba and not mamba. But that issue is upstream in micromamba and I've actually already complained about it already. (It's complicated, and actually the vision is for micromamba and mamba to someday become equivalent.)

By the way, if you want to install mamba directly, here is another Feature which you can install mambaforge with.
https://github.com/rocker-org/devcontainer-features/tree/main/src/miniforge

I wonder if #20 solves this issue. Micromamba should actually be using /opt/conda as the root prefix, not ~/micromamba. If you get the chance, could you please try again and see if everything works as expected now?


IndexError: dimension specified as 0 but tensor has no dimensions

🐛 Bug

I use SAGPool to creat a modle, but it has report a error
  File "main.py", line 107, in <module>
    test_acc,test_loss = test(model,test_loader)
  File "main.py", line 73, in test
    out = model(data)
  File "/root/miniconda3/envs/myconda/lib/python3.8/site-packages/torch/nn/modules/module.py", line 550, in __call__
    result = self.forward(*input, **kwargs)
  File "/mnt/DYTPool1/networks.py", line 51, in forward
    x, edge_index, _, batch, _ = self.pool3(x, edge_index, None, batch)
  File "/root/miniconda3/envs/myconda/lib/python3.8/site-packages/torch/nn/modules/module.py", line 550, in __call__
    result = self.forward(*input, **kwargs)
  File "/mnt/DYTPool1/layers.py", line 34, in forward
    perm = topk(score, self.ratio, batch)
  File "/root/miniconda3/envs/myconda/lib/python3.8/site-packages/torch_geometric/nn/pool/topk_pool.py", line 18, in topk
    num_nodes = scatter_add(batch.new_ones(x.size(0)), batch, dim=0)
IndexError: dimension specified as 0 but tensor has no dimensions

x is the data.x,I don't know why.
In the same training, sometimes it get an error, sometimes it don't
To Reproduce
Steps to reproduce the behavior:

Expected behavior

Environment

OS:
Python version:
PyTorch version:
CUDA/cuDNN version:
GCC version:
Any other relevant information:

Additional context

What's the output of x.size(0)?

What's the output of x.size(0)?

Is the output of x.size(0) the number of rows?

What's the output of x.size(0)?

It's equal to data.x.size(0).
I think it's the number of feature_matrix rows

Yes, it's the number of nodes. Just want to make sure that it's not zero.

Yes, it's the number of nodes. Just want to make sure that it's not zero.

Yes!you are right!
I find after the second pooling layer, there is only one node left!
So how do I solve this?

Can you show me minimal example to reproduce this? I will look into it.


Missing Page

I tried to use MS Teams 3.3  in connection with the latest plugin. The plugin installs wihout any error in Touch Portal but i am missing any page.
How can i get the page ?
brgds Dietmar

Hi.
I am not sure what is the question? The page to control Teams? You have to setup the page by yourself. The plugin only provide the controls and status values.

Hi,
so it was a missunderstanding from my side.
I thought there will be something like a example page. So i will try to make a simple page.
Thx


Serverless Pool: Msg 15872, Level 16, State 1, Line 13 NOT NULL is not allowed for external table columns.

SqlPackage or DacFx Version: <IP_ADDRESS>
.NET Framework (Windows-only) or .NET Core:
Environment (local platform and source/target platforms):

Steps to Reproduce:

Create an External Table in serverless pool

CREATE EXTERNAL TABLE [dbo].[test2]
						WITH (
						    LOCATION = '/stage/test',
						    DATA_SOURCE =[dls] ,
						    FILE_FORMAT = ParquetFormat
						)
						AS
						SELECT 1 as Id

Note that in SSMS it shows the column as not null

4 now try to recreate this table specifying NOT NULL
DROP EXTERNAL TABLE [dbo].[test]

CREATE EXTERNAL TABLE [dbo].[test]
(
	[Id] [int] NOT NULL
)
WITH (DATA_SOURCE = [dls],LOCATION = N'/stage/test',FILE_FORMAT = [ParquetFormat])
GO

this error occurs
Msg 15872, Level 16, State 1, Line 13 NOT NULL is not allowed for external table columns.
it is also worth noting that SMSS 19.3 will correctly script this out:
CREATE EXTERNAL TABLE [dbo].[test]
(
	[Id] [int] NULL
)
WITH (DATA_SOURCE = [dls],LOCATION = N'/stage/test',FILE_FORMAT = [ParquetFormat])
GO

However ADS Schema compare scripts it out as NOT NULL
which it successfully builds in database project / sqlpackage however the resulting script fails when trying to deploy.
to be honest I'm not entirely sure where the bug is here. There seems to be a disagreement between SSMS and DACFX as to what is allowed. In any case this isn't desired or expected behavior.
I would assert we should be able to define a column as not null in the table definition in serverlesss pool or if not the behavior across all tools should be consistent.
(DacFx/SqlPackage/SSMS/Azure Data Studio)

Serverless SQL pools don't support null/not null in the external table column definition, so we need to fix in DacFx:

failing the SQL project build if the project targets SQL serverless and the columns specify null/not null
generating the table definition for external tables in SQL serverless should not contain the specifier

https://learn.microsoft.com/en-us/sql/t-sql/statements/create-external-table-transact-sql?view=azure-sqldw-latest&preserve-view=true&tabs=serverless#syntax-2
SSMS scripting it out with the null included is also not correct, that would throw an error that NULL is not allowed for external table columns.


Fix default PACKAGES_BRANCH value

Fix: https://github.com/jspricke/ros-deb-builder-action/issues/54

Thanks!


Can't load dataset items in total size > 512MB

Node has 512MB limit on string size, if the response from API is larger, the client fails to parse it.
I guess the issue might also manifest in key-value-stores? Other api endpoints probably never produce this large responses.
Reported on slack: https://apifier.slack.com/archives/C0L33UM7Z/p1652976377122799

I don't think this is a bug really. It's quite obvious that strings of unlimited sizes cannot be loaded or parsed. That's why the list functions of client have offsets and limits and the KVS functions have stream options.
I would vote for closing this.

Bug report originated here: https://apifier.slack.com/archives/C0L33UM7Z/p1652976377122799


9-deploy-to-google-cloud-run

Please fix your editor settings to keep ending new-lines at the end of files (see here).


Adding Data Sender to leshan client

This work is based on #1268.

I'll take a look at it today

Integrated in master (squashed commit 60a95c2229cc034093792d876010791af89d6da8)

Thx @adamsero for the review :+1: !


Adding missing straight option in repositories compare api

Whoops, it's definitely Friday! Fixed.

LGTM!


Training too slow

When trying to train a laypa model I'm not going anywhere because the training is very slow, here's part of the output:
[02/21 12:42:34 detectron2.engine.train_loop]: Starting training from iteration 0
[02/21 12:52:57 detectron2.utils.events]:  eta: 87 days, 12:43:12  iter: 19  total_loss: 3.551    time: 31.8952  last_time: 27.2403  data_time: 0.8271  last_data_time: 1.4133   lr: 3.9962e-06  max_mem: 8727M
[02/21 13:03:34 detectron2.utils.events]:  eta: 87 days, 12:33:07  iter: 39  total_loss: 1.935    time: 31.8659  last_time: 25.1255  data_time: 0.6448  last_data_time: 1.4127   lr: 7.9922e-06  max_mem: 8727M
^C[02/21 13:04:34 detectron2.engine.hooks]: Overall training speed: 39 iterations in 0:21:10 (32.5881 s / it)
[02/21 13:04:34 detectron2.engine.hooks]: Total training time: 0:21:11 (0:00:00 on hooks)

Is there any possibility to speed up the process or is it because my machine is too weak in characteristics?
Here's the environment info:
[02/21 12:09:07 laypa]: Environment info:
-------------------------------  -----------------------------------------------------------------------------
sys.platform                     linux
Python                           3.12.1 | packaged by conda-forge | (main, Dec 23 2023, 08:03:24) [GCC 12.3.0]
numpy                            1.26.3
detectron2                       0.6 @/opt/conda/envs/laypa/lib/python3.12/site-packages/detectron2
Compiler                         GCC 9.4
CUDA compiler                    not available
DETECTRON2_ENV_MODULE            <not set>
PyTorch                          2.2.0 @/opt/conda/envs/laypa/lib/python3.12/site-packages/torch
PyTorch debug build              False
torch._C._GLIBCXX_USE_CXX11_ABI  False
GPU available                    Yes
GPU 0                            NVIDIA GeForce GTX 1650 Ti (arch=7.5)
Driver version                   551.23
CUDA_HOME                        /opt/conda/envs/laypa
Pillow                           10.2.0
torchvision                      0.17.0 @/opt/conda/envs/laypa/lib/python3.12/site-packages/torchvision
torchvision arch flags           5.0, 6.0, 7.0, 7.5, 8.0, 8.6, 9.0
fvcore                           0.1.5.post20221221
iopath                           0.1.9
cv2                              4.9.0
-------------------------------  -----------------------------------------------------------------------------

Thanks in advance for any suggestions.

Hi there,
Could you please specify what config you are currently using? And are you trying to train a model from scratch or do you want to finetune an existing model?

Thanks for quick reply. I'm trying to finetune an existing model suggested for use by the loghi project, here's the link to the config
https://surfdrive.surf.nl/files/index.php/s/YA8HJuukIUKznSP?path=%2Flaypa%2Fgeneral%2Fbaseline#editor
The only things I changed was setting the DATALOADER.NUM_WORKERS to 1 and  SOLVER.IMS_PER_BATCH to 4
Here's the code I run:
docker run $DOCKERGPUPARAMS --rm -it -u $(id -u ${USER}):$(id -g ${USER}) -m 32000m --shm-size 10240m -v $LAYPADIR:$LAYPADIR -v $TRAINDIR:$TRAINDIR $DOCKERLAYPA \
        python main.py \
        -c $LAYPAMODEL \
        -t $TRAINDIR \
        -v $TRAINDIR \
        --opts SOLVER.IMS_PER_BATCH 4

I think the main issue is that you are retraining the model from scratch since you aren't loading the weights back in for the training. This is done using the MODEL.WEIGHTS config option. Then you can lower the learning rate (SOLVER.BASE_LR) and the number of iterations that you use (SOLVER.MAX_ITER). The config you are currently using is the one that was used for the full training which took a couple of day on a more powerful machine.
Also make sure the $DOCKERGPUPARAMS is set correctly so you are actually using the GPU. And I don't recommend having only one worker for the dataloader, this will stop parallel processing of the images.

I tried to load the weights, so now in my MODEL.WEIGHTS I have /home/user/laypa/general/baseline/model_best_mIoU.pth instead of detectron2://ImageNetPretrained/MSRA/R-50.pkl but now I get the following error -
    assert os.path.isfile(path), "Checkpoint {} not found!".format(path)
AssertionError: Checkpoint /home/user/laypa/general/baseline/model_best_mIoU.pth not found!

The file is there, I double-checked.

Because you are trying to run the training in a docker environment you need to also link the folder correctly. This may be your problem, but I don't know for sure. Is -v $LAYPADIR:$LAYPADIR this also where the weights are? Otherwise you may want to like -v <Location of weights>:<Location of weights>. And is your user just called user? Or is this not the actual path? If this doesn't work perhaps you could try to run the training outside of the docker environment?


Multiple audio codecs in HLS output can't be played in Shaka Player

It's not yet clear if this should be restricted in Streamer, tweaked in Packager, or improved in Player.
For now, restrict yourself to aac for HLS.

Packager's dash_only=1 should solve this: https://google.github.io/shaka-packager/html/tutorials/dash.html
Not sure if we should hard-code a dash_only=1 on WebM-format outputs (much simpler), or if we should make dash_only configurable in the bitrate/codec configs (more extensible).

@joeyparrish, to clarify, if the pipeline config includes the following:
# The codecs to encode with.
audio_codecs:
  - aac
  - ac3
  - opus
video_codecs:
  - h264
  - vp9

# Manifest format (dash, hls or both)
manifest_format:
  - dash
  - hls

Shaka Player will not be able to properly handle the HLS stream? So, we would want something like this to restrict HLS to only aac:
# The codecs to encode with.
audio_codecs:
  aac:
    dash_only: False
  ac3:
    dash_only: True
  opus:
    dash_only: True
video_codecs:
  - h264
  - vp9

# Manifest format (dash, hls or both)
manifest_format:
  - dash
  - hls

If we go with the configurable approach, do we want to allow users be able to restrict any codec to "dash_only", or just opus since it's the only one that maps to a webm formatted output?

Shaka Player will not be able to properly handle the HLS stream?

That was once true.  I believe Shaka Player now filters out such streams from HLS (though I can't remember the details).  But Shaka Streamer does not require Shaka Player, so we should not put WebM streams into HLS.
I think the simplest answer is just to put dash_only=1 into the Packager command for WebM output.  That would not require any configuration.
If we want to make it configurable, something like you suggested might work.  But I'm not 100% sure it's worth the effort.  What do you think?


Use param-based badge

Make sure to not encode it to keep URL readable
https://img.shields.io/static/v1?label=GH%20Pages&message=MichaelCurrin.github.io/coding-blog/&color=green

But convert space to %20.
See if a flag on the frontend makes sense.
Also the badge can be used on the Repo page to make a repo badge around social section.

Relates to #38


Feature Requests:  Schema Selection For Launch Device Side

Expected Behaviour
Modify the AEP Web SDK Extension to set a default XDM Schema in the extension configuration so that it does not have to be set for every data element that uses the Extension.
Actual Behaviour
NA
Reproduce Scenario (including but not limited to)
NA
Steps to Reproduce
NA
Platform and Version
NA
Sample Code that illustrates the problem
NA
Logs taken while reproducing problem
NA

Duplicate of #631 . Will be closing this one.


Client request buffer metrics

Adds two new metrics to the service client which measure the total load and the percentage of max load for the pending request buffer.

That code fold was very well placed, and looked like all that code change was in that one function.  How annoying is that.  Doing this on send def seems like the better place to be doing it :)

I'm concerned these metrics won't add much value.  The size of the pending buffer is usually not worth paying attention to unless it's full, and there's other more direct things we can do when that happens.  Both the queueSize and outputQueueFull methods in OutputController are public, meaning they are directly exposed to any user of ServiceClient.  So it should be possible to completely avoid accidentally overloading a client.  Right now these values aren't really used.  For example, the LoadBalancingClientcould use them to avoid accidentally overloading one of its managed connections.
Also, every time a client connection rejects a request due to the buffer being full, it fails the request with a ClientOverloadedException, again providing direct feedback.  I think if we do add a metric, we should be ticking the error rate every time this happens.  In fact, I think we should do that in every place we fail a request in the client.  That alone would probably provide a lot of insight as to when the buffer is full.
Lastly, I don't think a histogram is the right metric type to use, although I don't think there's a better choice right now.  Really something like a box-and-whisker metric that just gives you min, max, mean, and stddev is a better solution, since a histogram is a fairly expensive metric, but we don't have one yet.
I think this needs some more thought, though if anything it does highlight some shortcomings with metrics and back-pressure management in the ServiceClient.


fix(tabs): fix issues on breakpoint

Fixes #2125.
CC @jnm2377
CC @emyarod
Changelog
Changed

Replaced usage of carbon--breakpoint-down() usage with "no-screensize-specified style is for mobile screen, style with media query for wider screen" approach. Mixed carbon--breakpoint/carbon--breakpoint-down() usage creates an intersection of both styles at the breakpoint (meaning, e.g. @media (max-width: 42rem) and @media (min-width: 42rem) are both applied).

Testing / Reviewing
Testing should make sure tab component is not broken, for mobile screen as well as for wider screen.
Note for reviewers:
Given the nature of above change, here's how to review the change:

Ensure styles in carbon--breakpoint-down() went to the corresponding no-screensize-specified style
Ensure such style is overriden by @media style, in either of the following ways:

With a style like inherit/auto/none
With the default value of the style property (e.g. visible for overflow)
A value that would have set unless it's specifically set (The wide-screen style in master (e.g. the-carbon-components.netlify.com) woud be the best reference)

Ensure properties in no-screensize-specified style replaced with mobile ones goes to wide-screen style

@shixiedesign My apologies, my note for reviewers may have been confusing (very technical and thus was for code reviews in dev team). As you may have imagined the change was only to tabs.


Speed up SupportInterface::ApplicationsExport

This did a lot of queries and loaded a lot of AR objects into memory, and it stopped running on prod because it caused database timeouts (the threshold is 10 seconds).
Using pluck and storing more objects in RAM rather than fetching from the DB sacrifices a bit of readability for performance. Even though we retain more bits of data this still seems to be cheaper in memory terms.
Not 100% sure this will fix the problem on prod, but it ought to help.
Performance stats, using 2020 forms with 2594 application choices:
Before
Benchmark
real time, seconds: 20.882151999976486
Memory profile
Total allocated:<PHONE_NUMBER> bytes (7731962 objects)
Total retained:  858798 bytes (9154 objects)
After
Benchmark
real time, seconds: 6.385142999934033
Memory profile
Total allocated: 141258273 bytes (1403405 objects)
Total retained:  168 bytes (1 objects)
Context
https://sentry.io/organizations/dfe-bat/issues/1689909459/?referrer=slack
Guidance to review
Does the code still do the same thing?
Link to Trello card
https://trello.com/c/A9JuYHfs/2436-speed-up-supportinterfaceapplicationsexport
Things to check

[ ] This code doesn't rely on migrations in the same Pull Request
[ ] If this code includes a migration adding or changing columns, it also backfills existing records for consistency
[ ] API release notes have been updated if necessary
[ ] New environment variables have been added to the Azure config

This looks like a sensible optimisation.
I think the same sort of problems will crop up with the candidate journey report too. As we scale up we'll perhaps have to think about doing these reports a bit differently, either by splitting them somehow or pre-calculating them overnight (or more regularly) in the background.

Yes. In theory the audit log knows everything, it just holds the information in an slightly awkward way. Maybe we could do something with that.
Updated to use .dig, much better!


although the error

<EMAIL_ADDRESS>global D:\a\opencv-python\opencv-python\opencv\modules\core\src\persistence.cpp (505) cv::FileStorage::Impl::open Can't open file: 'C:\Users\tuna_\OneDrive\Masa├╝st├╝\denemeler\FaceDetection-AttendanceSystem-main/haarcascade_frontalface_default.xml' in read mode
invalid literal for int() with base 10: ''
module 'cv2' has no attribute 'face'
module 'cv2' has no attribute 'face'

Hey, This doesn't seem to be issue with project itself but rather module or my guess is how you proceed to use it, there are no instruction, but that will be added later on
as of now can you mention the steps that lead you to this error?


🐞 [BUG] - The "Programming Languages" section is a complete mess

Issue Verification

[X] I have verified that this is a new bug report.
[X] I'm interested in working on this issue
[X] I'm part of GSSOC organization

Describe the bug
All the elements in the programming languages sections are overlapping each other. The entire section is a mess. It needs to be fixed immediately.
Expected behavior
I expected the list of languages written, then the testimonial sections follows it. Instead, all of them are overlapping.
Steps to reproduce

Go to the Resource Hub website
Click on Programming Languages card.
You can see the bug

Additional Context

Thank you for assigning this issue to me. I am working on it right away.


Disable pointer cast warning in spi_port.c

[Issue #29] The Linux header for SPI programming expects 64-bit pointers, but on Raspberry Pi they’re only 32 bits wide. Since we can’t edit the header, we have to disable the warning temporarily.

Looks good. Thanks!


Update typescript version

Mostly for https://github.com/TypeStrong/atom-typescript/issues/562#issuecomment-147620723
Make sure the jsx issue that was reported doesn't exist : https://github.com/TypeStrong/atom-typescript/issues/643
/cc @s-panferov ... if that issue still exists we might need to track that down first before we update typescript.

Restricted by : https://github.com/Microsoft/TypeScript/issues/5278


Add a way to install subcommand for existing package.

Please answer the following questions before submitting your issue. Thanks!
What version of DC/OS + DC/OS CLI are you using (dcos --version)?
If DC/OS CLI version < 0.4.5, DC/OS version can be found at http://dcos_url/dcos-metada/dcos-version.json
0.4.11
What operating system and version are you using?
Ubuntu 16.04 on Azure Container Service
What did you do?
If possible please provide a recipe for reproducing.
Installed Cassandra via DC/OS Web UI. Was then unable to find a way to add the 'cassandra' subcommand to my CLI for further management.
What did you expect to see?
A dcos CLI command to just install the subcommand portion of the dcos package install workflow.
What did you see instead?
$ dcos cassandra connection
'cassandra' is not a dcos command.

--cli option wasn't clear from the help. Had to refer to Stack overflow:
https://stackoverflow.com/questions/36827158/dcos-cassandra-subcommand-error

We have plans to update the DCOS CLI to install subcommands automatically based on what has been install on the cluster. That should help with the user experience.


Remove AnchorPeers from sampleconfig/configtx.yaml

Type of change

Documentation update

Description
This section was only used by the deprecated configtxgen --outputAnchorsPeerUpdate command. As we move to the channel participation flow without a system channel, we should stop documenting/recommending this value be set in configtx.yaml.
Related issues
FAB-18381

Oops, there's one test that needs to be updated. I'll push an update shortly.

Oops, there's one test that needs to be updated. I'll push an update shortly.


Fix repo indexer and submodule bug (#3107)

Backport of #3107.

LGTM

LGTM

LGTM

make L-G-T-M work

@ethantkoenig please send an empty push for CI running correctly.

make L-G-T-M


Crash with Thermal Expansion and Vacuum Chest

I am not 100% sure this is related to your mod or TE, however the crash report does say a lot about EnderIO so I figured I would start here. If this is indeed related to TE please let me know and I will post this over on their site.
Here is the crash report:
http://paste.ubuntu.com/10565466/
What I did to cause it, I had a Vacuum Chest above a Tesseract connected to it with a power conduit and Item Conduit (I was using it as a quarry set up since Buildcraft updated and has decided to eat all my power and move 10,000 miles an hour it spits items everywhere!) anyways.. I used a wrench to pick up the Tesseract which was immediately picked up by the Vacuum Chest and the server crashed, I logged back in and the Tesseract was fine and inside the chest..

It looks like the item conduit tried to insert something into the Tesseract when the crash occurred. As the Tesseract wasn't there anymore, I can understand that it didn't quite liked it. ;)
I'll leave finding out why that happened to someone who knows the item conduit code better.

The crash here is in tesserract code. Are you on the latest releases of CoFH mods?


Textual descriptions CelebA

Hi, I was wondering how you created the textual descriptions for the CelebA dataset? I know that the dataset consists of images with 40 binary attributes, but to the best of my knowledge there are not official textual descriptions.
Thanks, David

I made textual descriptions using the 40 binary attributes with my text-auto-generating code. If 'Wavy Hair' is 1 and 'woman' is 1, the code generate text: 'The woman has wavy hair' and 'A woman's hair is wavy' etc. And article (e.g. 'A', 'The') is generated randomly.
Thx.


Netlify build command cleanup

This PR contains suggested improvements to the site build scripts and config.

Simplifies the netlify.toml build commands by using the NPM scripts. E.g., the production build command goes from:cd themes/docsy && git submodule update -f --init && npm install && cd ../.. && hugo --gc --minify

to more the more concise yet equivalent:npm run build:production

Factors out common env var in netlify.toml
Adds NPM scripts that simplify building and serving the site

/cc @nate-double-u

Anything left to be done here? /cc @thesuperzapper

@chalin Also, all our PR/historical branches are getting indexed by Google, we should fix that at the same time as this PR.
The goals would be:

The main www.kubeflow.org site should be indexed
All PR deploy-preview-XXXX--competent-brattain-de2d6d.netlify.app should NOT be indexed
All other v1-7-branch.kubeflow.org sites should be NOT be indexed:

(these are just CNAME records pointing to the branch domains like v1-7-branch--competent-brattain-de2d6d.netlify.app)

I believe your changes here achieve 2, because you are setting -e dev in the hugo command, and because this is not "production", docsy adds <meta> no index tags.
We need to be careful about 1. Are you 100% confident that not setting -e production or HUGO_ENV=production is safe?
To achieve 3, we could set the HUGO_ENV from [context.branch-deploy.environment] to dev, but it will probably propagate faster if we use a robots.txt disallow on those domains (otherwise, the <meta> tags will take until Google next indexes each page).

I've created a separate issue to track the full extent of the indexing issue, since more work will need to be done outside of the scope of this PR:

#3645

I believe your changes here achieve 2, because you are setting -e dev in the hugo command, and because this is not "production", docsy adds <meta> no index tags.

Correct.

We need to be careful about 1. Are you 100% confident that not setting -e production or HUGO_ENV=production is safe?

For reassurance, see https://discourse.gohugo.io/t/what-does-setting-hugo-env-to-production-do/24669/2. Since the site uses hugo.IsProduction, we're good.

To achieve 3, we could set the HUGO_ENV from [context.branch-deploy.environment] to dev, but it will probably propagate faster if we use a robots.txt disallow on those domains (otherwise, the <meta> tags will take until Google next indexes each page).

I'll answer this in #3645.

I believe what we are doing is now fine (e.g. we already use hugo extended as this is the default for Netlify), please reopen a new PR if there are still issues.
/close


chore: bdd-sbhg-1592923396 to 0.0.1

chore: Promote bdd-sbhg-1592923396 to version 0.0.1

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Strategy specific builders: to censor or not to censor

Opening this issue to start discussion around the topic of strategy censoring
Front-running is illegal in traditional financial markets so it is no stretch of the imagination to think that < large validator set run by corporate entity > [ Coinbase, etc ] may be required to enforce a similar restriction on its validator's MEV augmented profits.
Therefore, some type of strategy specific censorship is inevitable. And even if regulation isn't the driving force, it could be possible to see more altruistic focused validator sets coming from community governed groups such as Lido.
Some open questions around this:

How to form open standards around types of arbitrage?
Can this choice be made in protocol (builder, relay, validator?) or is it best left to be socially coordinated?
Can we accurately classify MEV types?
If a heuristic based classification model is used, what are the risks of misclassification?
Will ignoring certain types of MEV lead searchers to construct builders for their censored strategy, and would this be a bad thing?

Thanks @dmarzzz for bringing the topic here. A few more things that came to mind:

If this is regulated, who is the responsible party? The validator will receive a full block, with no option to front-run it. The builders are just sorting bundles, no front-running there. These bundles could even be encrypted with some futuristic stuff. The searchers? They can be anonymous. The protocol designers? Like they should protect their users, but is it actually possible? The relay? But then an anonymous relay can appear that skips regulation.
What is altruism in here? There's the argument that when there is MEV, the only best thing to do for network stability is to maximally extract it. What kind of incentives will build up when builders censor transactions?
Is censoring more legal than non-censoring?

Many many thoughts. But mostly questions. At flashbots we are thinking hard on this. It would be nice to hear more thoughts and experiences economic, legal an technical.
We are also designing the data collection and publication, so more people can contribute to the analysis.
https://github.com/flashbots/flashbots-data-transparency

Here is a post by Uri from @bloXroute-Labs talking about a good block-builder, and an invitation to discuss this further:
https://medium.com/@uri_61495/flipping-the-table-on-the-mev-game-dc31df8baaf7

Here is a tool by @pmcgoohan to visualize mev values and transaction ordering. Some transactions are labeled as toxic.
https://www.zeromev.org/
Are they toxic or is this how defi protocols are designed to work?
If they are toxic, is it possible to identify them in real time?
If so, should these toxic transactions have been censored?
If so, where?

I will add some more input to what for some may be "the bad"/ toxic MEV.
Figment recently posted this https://www.figment.io/resources/figments-mev-policy-supporting-neutral-secure-and-open-solutions
In the post they state that "The Bad" are sandwich attacks and generalized frontrunners. In that sense they also use a similar definition as zeromev.org does for "toxic" MEV.

There is a lot to be said on the topic of identifying "toxic" MEV. For starters, it's not easy (or possible?) to identify cases of front-running or sandwiching.
For example, in order to assess a builder's claim of not sandwiching, you could start off checking for sandwiches by checking swaps for a transaction immediately preceding and following it from the same address.
Searchers/builders could catch on to this and start doing sandwiches from different accounts, rebalancing funds occasionally.
Now one could check if a swap is preceded and followed by transactions touching the same pool (one a buy and one a sell), but one would already be mislabeling some normal occurrences as sandwiches.
Add on to this that the sandwiching transactions don't need to be immediately next to the sandwiched transaction (you could add unrelated transactions in between) and it becomes very hard to tell if a block contains a sandwich or just a bunch of transactions touching the same pool.
One could keep playing this cat-and-mouse game but ultimately it doesn't seem like a fruitful endeavour.
Perhaps there are some other metrics one could use like looking at average slippage per builder, but its not clear that such metrics wouldn't be gameable either.


Imdb and or tmdb numbers added to the .plexmatch

Sonarr creates .plexmatch and adds imdb ID and tmdb ID for seasons. So far only your code does .plexmatch for movies. Very thankful for that but any chance of getting the imdb and or tmdb ID added in like sonarr does? I feel plex doesn't recognize all movies and with those codes it can find them easier

This application uses the Plex API to get the media information. The Plex API does not return IMDB, TMDB, TVDB, or any other 3rd party IDs. Sonarr can add that information, because it already has that information.
Plex API: https://www.plexopedia.com/plex-media-server/api/


export database

Hello, I would like to know how to export the database where users and devices are. I want to install a new server without losing all users. The old instalation is in Windows, the new is in windows too.

If you are using default H2 database, you can just copy the content of data folder from one machine to another.

yes, try that, but the newly added devices do not work, it was possible to add them, but to put the id of those devices nothing happened, while aggregate equipment before if they worked. I thought I had done something wrong or that one ought to copy one of the files.Thanks

Did you stop Traccar instance before copying data?


DB Select statement do not return data in migration pretend mode

Laravel Version: 9.0
PHP Version: 8.0.21
Database Driver & Version:
Mysql

Description:
In database Laravel migration pretend mode Select statement do not return data which leads to errors in those migrations that read database. For example it could be a migration that extends values list in database field of enum type. To avoid hard-coded values lists such migration could first read enum type values list from database, then extend it and write the result back to database. Which works correctly in normal database migration mode but gives error when migration is launched in pretend mode.
Steps To Reproduce:
Create a database migration that launch Select query and then uses its result in some way - for example to compose then next database query. This gives when migration is launched in pretend mode because in that mode Select query returns empty data.

I see a PR was sent in so let's see how it goes.


Update bevy_egui to 0.22

Update the pinned version of bevy_egui to 0.22 no other changes needed for example to work

Closing in favor of #276. Thank you for contributing! 🙂


Parallel processing blocked

I've run into an issue trying to parallelize code using Peddy. I'm parsing a PED file and VCF, then splitting all variants into groups to process. This splitting makes it a strong candidate for parallelization, but I can't pickle the Ped() object, so multiprocessing is blocked.
_pickle.PicklingError: Can't pickle <class 'peddy.peddy.UNKNOWN'>: it's not the same object as peddy.peddy.UNKNOWN
Probably relating to the handling of unknown members in the Pedigree:
https://github.com/brentp/peddy/blob/master/peddy/peddy.py#L102-L104
This is completely non-urgent, and I'll see if I can work out a fix which can be ported upstream

It might be because of this: https://github.com/brentp/peddy/blob/master/peddy/peddy.py#L133
Instead of working on peddy, you might try somalier it's faster and scales to more samples.


[release-1.4] Add the version to the fetcher

Proposed Changes

Each branch of release should generate code based on the current version instead of the latest version.

Release Note

/test eventing-upgrade-tests
/lgtm
/approve


⚠ Fortnite has degraded performance

In f94f744, Fortnite (https://fortnite.com) experienced degraded performance:

HTTP code: 200
Response time: 1591 ms

Resolved: Fortnite performance has improved in 393556a.


feat(source): support create connector from specific state

What's changed and what's your intention?
PLEASE DO NOT LEAVE THIS EMPTY !!!
Please explain IN DETAIL what the changes are in this PR and why they are needed:
this pr introduces state store in connector soure. When creating a new connector streaming source, executor will wait until the first barrier and try to recover state from the barrier.epoch.prev. If it fails, the executor will read data from position specified in stream_source_splits, which is initialized in meta.

Summarize your change (mandatory)
How does this PR work? Need a brief introduction for the changed logic (optional)
Describe clearly one logical change and avoid lazy messages (optional)
Describe any limitations of the current code (optional)

Checklist

[x] I have written necessary docs and comments
[x] I have added necessary unit tests and integration tests

Refer to a related PR or issue link (optional)
part of https://github.com/singularity-data/risingwave/pull/1960

This PR no state saving operation is called? which means the take_snapshot method.

This PR no state saving operation is called? which means the take_snapshot method.

the change seems to dropped by rebase, will fix it soon


Adds goreleaser support to deterministically build and release binaries

This PR adds a goreleaser configuration and release script, along with two make
commands to start the release.

main

#3   👈

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @bg451 and the rest of your teammates on  Graphite


App crash on planetary network connect/disconnect on slower connections

When the connection is slower, the disconnect button appears before the connection is actually made.
When this disconnect button is pressed before it appears, the app crashes.

The disconnect button should wait to appear until the connection is actually made.

Disabled for now


⚠ Spryr has degraded performance

In ef08b0b, Spryr (https://spryr.com) experienced degraded performance:

HTTP code: 200
Response time: 5182 ms

Resolved: Spryr performance has improved in 801e37d after 7 minutes.


class="form-inline" only prints one page in Firefox

Sending page to printer/pdf in Firefox only outputs one page when using firefox. It prints as expected in Chrome.
Anything within the form-inline tag doesn't print after the first page:
<form class="form-inline">
blah blah blah for more than one page and it won't print
</form>

Here is my plunker:  https://plnkr.co/edit/k4J1lX
Now, if I add a d-block then it does print, but then it reverts the page to form-horizontal where the labels and inputs would be stacked instead of side by side (I don't demonstrate the stacking here, just the multi-page printing)
<form class="form-inline d-block">
blah blah blah for more than one page
</form>

Here is the sample plunker:  https://plnkr.co/edit/A3jfaJ8CY6wfGYU0ozQj

This is a browser bug. Unfortunately there is no general way to solve. There are some workarounds, those are incompletely but may help for some cases.
Also please see:
https://bugzilla.mozilla.org/show_bug.cgi?id=939897
Sorry but closing this since we can't fix.


Refactor to use babyapi

TODO:

[ ] Update integration test to use Client
[ ] Update tests to use latest testing features
[ ] Update API and routes to use latest babyapi methods
[ ] Use KVStorage extension?
[ ] Use HATEOAS extension


プロセカ can use the same icon with HATSUNE MIKU: COLORFUL STAGE!

About this discussion
These both are the same game, just different server. So I figured that they can probably just share the hatsune-miku-colorful-stage icon.
プロセカ com.sega.pjsekai/com.google.firebase.MessagingUnityPlayerActivity
HATSUNE MIKU: COLORFUL STAGE!
com.sega.ColorfulStage.en/com.google.firebase.MessagingUnityPlayerActivity

You can also do it yourself.
https://github.com/LawnchairLauncher/lawnicons/blob/develop/CONTRIBUTING.md#adding-an-icon-to-lawnicons


Set up and configure VEP annotation

Description of feature
The first major feature of the workflow provides variant annotation using ensembl-vep workflow.
For this, we should integrate the nf-core module of VEP.
The flags and arguments for the vep command should be costumizable.
However, for ZPM purposes we should discuss a number of standards arguments to include by default.
I suggest the following flags:

-- everything: Summarizes most important flags: --sift b, --polyphen b, --ccds, --hgvs, --symbol, --numbers, --domains, --regulatory, --canonical, --protein, --biotype, --af, --af_1kg, --af_esp, --af_gnomade, --af_gnomadg, --max_af, --pubmed, --uniprot, --mane, --tsl, --appris, --variant_class, --gene_phenotype, --mirna
--check_existing: Adds dbSNP IDs and Clin_SIG fields, e.g. from ClinVar
--no_escape: do not use URL escape signs , e.g., a p.A567= mutation would otherwise be dumped as p.A567%3D
--flag_pick: adds a flag picking only one transcript based on MANE, Canonical status and functional annotation (see here: https://www.ensembl.org/info/docs/tools/vep/script/vep_other.html#pick)
--format vcf & --vcf: Specified VCFs files as standard input and output.
--offline: Enables offline usage, which should be standard procedure and additonally needs the cache specified.
--cache: VEP Cache as input databases.

Further ideas for easy configuration:

Using either RefSeq or Ensembl transcripts by the configuration file. Needs to add --refseq flag or the --merged flag to VEP command and specific VEP cache for refseq/merged.

As mentioned during the discussion, it could be potentially helpful to pre-select clinically relevant transcripts based on a predefined reference set as f.e. https://www.lrg-sequence.org/ (Locus Reference Genomic Database).
The database seems to be deprecated since 21th march 2021 and it recommends the following:
"Ensembl and RefSeq transcripts that are specified by the MANE collaboration are preferred for all genes where available to help standardise clinical reporting. "
We could thus include a soft-filter routine for clinically relevant transcript based on MANE in a later iteration.

see PR #8


Changed: Updated dependencies

I've added:

[ ] USAGE.md docs if applicable
[x] CHANGELOG.md

Would you be up for using Dependabot to automatically create dependency-update PRs like this one? Only downside is it wouldn't create changelog entries for you, but it will link to the changelogs of each of the packages you're updating. (I built it and it's free.)

@greysteil nice looking service, great job! I'll give it a shot. Thank you.


return all not any

CrateDB version: 3.0.0
Problem description:
when search condition use   match(field_keywords,'ios android') using best_fields with (operator='and')
some of the result only contains ios or android , not both.
Or there is some solution to return both ios and android?

hi @junphine, can you provide the exact query and maybe also some sample data?

SELECT field_resume_keywords FROM "hr"."content_resume"   where match(field_resume_keywords,'ios android') using best_fields with (operator='and')  and field_resume_status!=1 order by _score desc   limit 100;
result:
field_resume_keywords
android ios 研发工程师
IOS &Android 工程师
android 和ios开发工程师
ios  leader
android技术部  |  android工程师
Android  leader
android  leader
iOS team leader / iOS架构师
iOS leader
iOS Leader
IOS Leader
IOS Leader
iOS leader
IOS 开发
Ios Leader
ios leader
iOS leader
iOS 开发
IOS Leader
IOS Leader
iOS Leader
iOS 开发

Unconventional， the data is insert use elasticsearch rest  api，not crate sql.

The usage of the ES API is deprecated and was always kinda unsupported (but was mostly safe for read operations).
As we cannot reproduce this issue using your supplied query and data set when using CrateDB's SQL for all operations (table creation, write, read), I'm closing this now.
Feel free to reopen if this still appears when using SQL for everything.

I use crate sql to create table,insert data.
the result is same.

Please tell us the steps to reproduce it.

simple produce data:
create table resume(
title string INDEX USING fulltext
);
insert INTO resume(title) values('android'),('ios'),('android both ios'),('ios before android');
then query:
SELECT * FROM "doc"."resume"  where match(title,'ios android')  using best_fields with (operator='and')
or use minimum_should_match
SELECT * FROM "doc"."resume"  where match(title,'ios android')  using best_fields with (minimum_should_match=2)
the result is：
ios before android
android both ios
ios
android
I expect result ：
ios before android
android both ios

use elasticsearch _search dsl ,the result is right.
the mapping is:
"properties" : {
"title" : {
"type" : "text",
"analyzer" : "standard"
}
}

@junphine Thanks for the detailed information, we can reproduce this issue now and are working on a fix.

@junphine We've fixed this by https://github.com/crate/crate/pull/7586, it will be released with the next 3.0 hotfix release (and the next nightly snapshot release). Thanks for reporting!


A few little fix ups

Nothing ground breaking
Remove binaries -- Add Makefile
Remove hard coded paths.
Split things into 2 files

Thanks for the input :) If you have time, could you please update the ReadMe to include compiling the C files?
Cheers

After looking more at the code, there will be issues with the set up script due to hard coded URLS, file permissions and the non-compiled code. I need to update the code to Git Clone this repository and build the compiled C code. For now I will revert until it is fixed. Sorry for the confusion


secretTemplate is not update the secret labels/annotations in reconcile scenario

Describe the bug:

After edit a certificate's secretTemplate field with a new label/annotation.
The issuing controller cannot apply the changes on the secret.
Expected behaviour:

certificate's created secret should be patched in reconcile scenario.
Steps to reproduce the bug:

create a certificate with reference to an existing Issuer, with the secretTemplate field with the labels
apiVersion: cert-manager.io/v1
kind: Certificate
metadata:
  name: cert
  namespace: <namespace>
spec:
  dnsNames:
  - <dnsName>
  issuerRef:
    kind: Issuer
    name: <issuer>
  secretName: cert
  secretTemplate:
    labels:
      aaa: bbb

add a new label to the secret template, and label won't added to the secret's labels
Anything else we need to know?:
Environment details::

Kubernetes version:
Cloud-provider/provisioner:
cert-manager version: v1.6.1
Install method: e.g. helm/static manifests

/kind bug

Hi, thanks for opening the issue. There have been recent changes to that part of the code- could you please update the issue description with what version of cert-manager you are using.

I used v1.6.1, I see that in v1.7.1 it works as expected.


Add an option to turn off pod webhook

The admission webhook to monitor the pod creation may cause performance burden on k8s api server. Thus before resolving the performance problem, please add an option to turn it off.
The problem webhook
https://github.com/open-telemetry/opentelemetry-helm-charts/blob/opentelemetry-operator-0.53.0/charts/opentelemetry-operator/templates/admission-webhooks/operator-webhook.yaml#L128
When significant amount of pods created at the same time, the performance burden will bring down the opentelemetry operator and the following k8s api server which killed the entire cluster. Even though we turnt off the instrumentation, it will cause unnecessary cost on k8s api server. Unlike the instrumentation and opentelemetry collector resource webhook, the performance cost of the pod webhook scaled with the cluster size.
The only optionsthat provide the similar function are namespace/object selector, but they are ideal as just remove this webhook when not used.

Could you go into some more detail about the problem you ran into? How many Pods you were creating, how much resources the operator was using, and what happened to the API Server as a result?

A newbie suestion, what is the purpose of the webhook ?
And what would be the potential problem to disable it ?
Thanks in advance.

The purpose of the webhook is to make changes to Pods before they're created. For the operator this involves injecting autoinstrumentation or sidecar Otel Collector containers. As Pods are immutable, this can only happen when they're created. The API Server calls the webhook on every Pod creation, and the operator gets to decide if it wants to make any changes to the Pod manifest.
Webhooks can also be used for other things, like doing validation, but for Pods specifically, the operator only does what I enumerated above.
It's not invalid to want to disable them, but this is fairly advanced customization of the operator, and can get you in trouble if you don't fully understand what you're doing. As a proper solution to @pureklkl's problem, I'd prefer to optimize the webhook so it can easily cope with large bursts of requests.

I encountered a similar issue with the operator running in my cluster, which caused delays in creating new resources. Upon investigation, I traced the problem back to the operator. Deleting the operator pod resolved the issue, restoring normal operation. As a measure, I have excluded the admission webhook(Refer) for all namespaces except the one housing the operator.
I'm still experiencing delays in resource creation within namespaces that host the operator. This namespace includes services like nginx, target allocator, otel collector, and others.
Could you please clarify the following questions for me?

Aside from autoinstrumentation or sidecar injection, what other roles and responsibilities does the operator fulfill?
Is it possible to skip the admission webhook for the namespace that contains the operator?


⚠ ClamAV has degraded performance

In 10e9443, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 203 ms

Resolved: ClamAV performance has improved in 5eb5ddf after 543 days, 18 hours, 26 minutes.


Update workflow execution with version histories check

What changed?
Update workflow execution with version histories check

Why?
Missed one place to update last write version.

How did you test it?

Potential risks

Coverage increased (+0.1%) to 59.427% when pulling b5ce25734fb5084e297c0e3dee379922d5823ccb on revert_replication_state into daba348a296d5a3a5a45dd47c2bcf29e9adf2b92 on master.


Script fails to run but action succeeds

I have some scripts that may fail to run, but the Action always succeeds whether it fails or not, I can't get the message that the Action's run failed

@furacas Are you providing the full path to the script you are wanting to run? Alternatively you could also cd to the directory within the SCRIPT_AFTER and then call the script.

@furacas Are you providing the full path to the script you are wanting to run? Alternatively you could also cd to the directory within the SCRIPT_AFTER and then call the script.

sure, What I want to express is that the script fails to run, but the Github Action runs successfully. I expect that when the script fails to run, Github Action prompts failure

Currently main action task is rsync deployment.
SSH scripts before/after are only logging results or error.
I can add additional param / env var to control this behaviour.
Default will still be just logging.

That would be really handy – we’re running migrations and reloading services in our post-deploy script, so any failures there are serious and we need to know about them. The individual action logs aren’t likely to be reviewed if they’re being logged as successful.
Thanks for the action, it’s a real timesaver.

Tacking on to this, we're seeing succesful taks even when rsync fails, logs below:
❌ [Rsync] error:
Error: rsync exited with code 23
    at ChildProcess.<anonymous> (/home/runner/work/_actions/easingthemes/ssh-deploy/main/dist/index.js:2:2603)
    at ChildProcess.emit (node:events:527:28)
    at Process.ChildProcess._handle.onexit (node:internal/child_process:291:12) {
  code: 23
}
❌ [Rsync] stderr:
rsync: [generator] recv_generator: mkdir "<dir-name>" failed: Permission denied (13)
*** Skipping any contents from this failed directory ***
rsync error: some files/attrs were not transferred (see previous errors) (code 23) at main.c(1333) [sender=3.2.3]

+1

+1

+1

Seems This project is supported this feature with this MR.
Just use easingthemes/ssh-deploy@main and pass  SCRIPT_AFTER_REQUIRED = true  variable to it.
Maybe we should mention it in the README.md

Readme updated: https://github.com/easingthemes/ssh-deploy/blob/main/README.md#12-script_after_required-optional-default-false
SCRIPT_AFTER_REQUIRED: true


Missing changelog

Hi Nathan,
I'm unable to find a changelog. As I'm willing to use your library in a production app, I need to know what changes happend since the last version to verify if I have to upgrade and what could go wrong if I do. It would be very appreciated if you could add a changelog/history. Thanks :+1:

git log <sha-for-release-1> <sha-for-release2>

They are also available on every release https://github.com/nathancahill/Split.js/releases

Besides the releases page, Split.js follows semver, with minor versions used for bug fixes and non-breaking changes. Major versions (only 1.0.0 has been released) have a list of Breaking Changes in the release notes.


Fix can not focus on the main composer after going back from thread

Details

Fix can not focus on the main composer after going back from thread
Fixed Issues

$ https://github.com/Expensify/App/issues/38700
PROPOSAL: https://github.com/Expensify/App/issues/38700#issuecomment-2010901551
Tests

Open any report
Send some messages
Hover over the message and click on Reply in thread
Go back to the previous chat by back button browser or back button in header
Hover over the message and click on Edit message
For small screen: click outside or x button to open main composer
Click on main composer and verify that the main composer

[x] Verify that no errors appear in the JS console

Offline tests

Same as above
QA Steps

Open any report
Send some messages
Hover over the message and click on Reply in thread
Go back to the previous chat by back button browser or back button in header
Hover over the message and click on Edit message
For small screen only: click outside or x button to open main composer
Click on main composer and verify that the main composer

[x] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is either coming verbatim from figma or has been approved by marketing (in order to get marketing approval, ask the Bug Zero team member to add the Waiting for copy label to the issue)
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(theme.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If the PR modifies the UI (e.g. new buttons, new UI components, changing the padding/spacing/sizing, moving components, etc) or modifies the form input styles:

[x] I verified that all the inputs inside a form are aligned with each other.
[x] I added Design label and/or tagged @Expensify/design so the design team can review the changes.

[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.

Screenshots/Videos

Android: Native
https://github.com/Expensify/App/assets/161821005/c0fa0cf5-7264-4935-8fbb-8207d6aeddf5

Android: mWeb Chrome
https://github.com/Expensify/App/assets/161821005/77fa45e5-5749-4019-95c7-873fb3d39021

iOS: Native
https://github.com/Expensify/App/assets/161821005/dde0103a-c44f-4767-bf3c-350eaf210f1a

iOS: mWeb Safari
https://github.com/Expensify/App/assets/161821005/2ba2b63e-59ae-4c67-a3fc-dbe6c0e3b9e4

MacOS: Chrome / Safari
https://github.com/Expensify/App/assets/161821005/c0790714-1bb1-460f-971c-253b00931231

MacOS: Desktop
https://github.com/Expensify/App/assets/161821005/9e54d8ec-73bb-4fd0-b2a7-4bb1b0356833

@dukenv0307 I updated.

@NikkiWines I updated.

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to staging by https://github.com/NikkiWines in version: 1.4.58-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/Beamanator in version: 1.4.58-8 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


Reduced error messages returned from oneOf/anyOf

I'll start off with great tool! I've been converting all of our schema validation from another, much slower, package. The "problem" I have is that we use the error messages returned to display the error to our application's front end. The "issue" I have is that oneOf/anyOf combined with if/then returns all of the failed errors making it impossible to determine why it failed.
I'm currently using 6.5.2
The follow example is hand typed because I'm on a disconnected network, so please excuse any typos:
{
  "type": "array",
  "additionalProperties": false,
  "items": [
    {
      "type": "object",
      "additionalProperties": false,
      "properties": {
        "type": {
          "type": "string",
          "enum": ["circle","triangle","polygon"]
        },
        "points": {
          "type": "array",
          "items": points,
          "oneOf": [
            {
              "if": {"properties": {"type": {"enum": ["circle"]}}},
              "then": { "minItems": 1, "maxItems": 1 }
            },
            {
              "if": {"properties": {"type": {"enum": ["triangle"]}}},
              "then": {"minItems": 3,"maxItems": 3}
            },
            {
              "if": {"properties": {"type": {"enum": ["polygon"]}}},
              "then": { "minItems": 3 }
            }
          ]
        }
      }
    }
  ]
}

points are defined as a latitude and longitude.
if I pass in json like this:
[
  {
    "type": "triangle",
    "points": [{"lat":"1N", "long":"2E"}, {"lat":"2N","long":"3E"}],
  },
]

I get a lot of errors back:
[
{ "path": "0.points", "type": "maxItems", "message: 'should NOT have more than 1 items" }
{ "path": "0.points", "type": "if", "message": "should match then schema" }
{ "path": "0.points", "type": "minItems", "message": "should NOT have less than 3 items" }
{ "path": "0.points", "type": "if", "message": "should match then schema" }
{ "path": "0.points", "type": "minItems", "message": "should NOT have less than 3 items" }
{ "path": "0.points", "type": "if", "message": "should match then schema" }
]

What I'd really like to get back is just:
{ "path": "0.points", "type": "minItems", "message": "should NOT have less than 3 items" }
{ "path": "0.points", "type": "if", "message": "should match then schema" }

Is it possible to add a flag for this? Or is this something I missed that already exits?

Your schema is not correct, in "if" you are not checking property type of the object on the top level (data[0].type), you are checking a sub-property of property points (data[0].points.type).
That's why all three ifs are valid and all three thens are checked.


Ignore console commands that have no actions

Q
A

Is bugfix?
yes

New feature?
no

Breaks BC?
no

Tests pass?
yes

Fixed issues
#16855

Changes summary:

Add filter to HelpController::getCommands in order to ensure that command exist and it has actions. It allows to remove these checks from other places.
UnknownCommandException::getSuggestedAlternatives ensure that command has default action before adding it to the suggested list.
Replace Nested Conditional with Guard Clauses

Tests added

Merged with adjustments https://github.com/yiisoft/yii2/commit/6e2b1782e52f8122302b84c331b30deec8972041. Thanks for contributing.


Update garrison's name in .mailmap

Summary
With this change, my name in Qiskit.bib will (eventually) match the name I use for academic publications.
Details and comments

The next time a tracked qiskit project releases (which will likely be tomorrow) qiskit-bot will regenerate the bib file

Hmm, now both variations of my name appear in the current Qiskit.bib.  I might try to track this down next week.


As a user, I want to monitor the pipeline cost

🧑🔬 User Persona(s)

Node Manager
💪 Motivation
...so that I can ensure I am within my budget / cost estimate
📖 Additional Details

[ ] Cost Monitoring User Guide

⚖ Acceptance Criteria
Given
When I perform
Then I expect

⚙ Engineering Details

deferring to future build


powershell encoding fixing

#3750

Looks good to me, but I don't know if it works yet.
I will ask (in the issue page) the issue ticket author to test it.

@Dansimp The PR fixes sample PowerShell code in a Windows Configuration article and resolves issue #3750. Please review and merge as the time permits. Thanks.

@Dansimp Followin up for updates.


pyaml --> pyyaml in setup.py

in the requirements pyaml made installation fail, I suppose it must read pyyaml. At least with this modification everything install fine with conda build.
Build still ends with an error, but running /bin/bash -x -e ~/anaconda/conda-bld/work/conda_build.sh did make it work (somehow the pyaml is still coded somewhere)
Addresses issue #468

Thanks, this looks good.


Add bindings for Start/Stop operation

This checklist is used to make sure that common guidelines for a pull request are followed.

For new extensions:

[ ] My extension description/summary conforms to the Extension Summary Guidelines.

About Extension Publish
There is a pipeline to automatically build, upload and publish extension wheels.
Once your PR is merged into master branch, a new PR will be created to update src/index.json automatically.
The precondition is to put your code inside this repo and upgrade the version in the PR but do not modify src/index.json.

If this PR is for a new extension or change to an existing extension, use the following to try out the changes in this PR:
docker run -it microsoft/azure-cli:latest
export EXT=<NAME>
pip install --upgrade --target ~/.azure/cliextensions/$EXT "git+https://github.com/NickKeller/azure-cli-extensions.git@nikelle/addstartstopcli#subdirectory=src/$EXT&egg=$EXT"

aks

hi @NickKeller is this PR ready to merge?


Suggestion: Unflattening could work for readonly properties

Unflattening is a really great feature. Thanks for it. 👍
It would be even greater, if it could work with readonly properties. We have these classes:
public class ContractingParty
{
    public string Name { get; set; }
    public string Company { get; set; }
}

public class Invoice
{
    public long Id { get; set; }
    public string DocumentNumber { get; set; }
    public ContractingParty Supplier { get; set; }
}

public class InvoiceDto
{
    public long Id { get; set; }
    public string DocumentNumber { get; set; }
    public string SupplierName { get; set; }
    public string SupplierCompany { get; set; }
}

If we Adapt from InvoiceDto to Invoice, everything works. Mapster creates a new instance of ContractingParty for Supplier property and fill its values.
But we would like to have Supplier property readonly to make sure, it is never null. And Mapster could just fill its properties without creating a new instance:
public class Invoice
{
    public long Id { get; set; }
    public string DocumentNumber { get; } = new ContractingParty();
    public ContractingParty Supplier { get; } = new ContractingParty();
}

Its probably not so easy, because I think that Mapster is just skipping readonly properties and so some effort should be made to consider them when unflattening is on. So I am writing this here really just as a suggestion. (Mayby I will try to look at it myself later.)

I can see use case for this. But for now how about just make setter internal or protected?

Hmmm. I did not realize I can do this and Mapster will still work. Good tip for simple workaround. Thanks.

Mapster 5.0 now support readonly prop https://github.com/MapsterMapper/Mapster/wiki/Mapping-readonly-prop


feat(#50): implement BytecodeIR#name() method

Implement BytecodeIR#name() method.
Closes: #50

PR-Codex overview
Detailed summary

Added a new test method retrievesName() in the BytecodeIrTest class.
Implemented the name() method in the BytecodeIr class.
Added a private inner class ClassName in the BytecodeIr class to extract the name from the bytecode.
Updated the name() method to parse the bytecode and return the name of the object from the bytecode.

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

@rultor merge

@rultor merge

@volodya-lombrozo Done! FYI, the full log is here (took me 2min)


Bootstrap 4 only

Hi.
I am using React and therefore I do not need the JavaScript part of Boostrap (especially I do not want to load JQuery and other dependencies)
Can you tell me a way to extract just the css parts, in a way the grid still works ?

Fix it


Copy automatically examples resource folder to binary output location

Description
After compiling examples, I noticed the resource folders where not copied automatically and needed to be manually copied.
I knew it was possible to copy folders with Cmake, so I did it.
Tasks
The commit is Cmake related, should work on all platforms

[x] Tested on Linux
[ ] Tested on Windows
[ ] Tested on macOS
[ ] Tested on iOS
[ ] Tested on Android

How to test this PR?
Set SFML_BUILD_EXAMPLES to TRUE in CMakeLists.txt and do the usual build commands.

Thanks for the contribution! 🙂
Doesn't this now copy the resources for every build of the examples?
Copying resources around and other things, should from my understanding only really been done during the "installation" (i.e. when building the INSTALL target), which I thought works correctly already. If that's not the case let me know.

This copy the resources once when you run cmake .. , but not for every build.
Indeed resources are copied correctly when installing; I didn't notice, because I was building the examples targets into my build folder (in this case resources are not copied), but not installing them, oopsie !

I think that's the expected behavior then. Just building things doesn't prepare stuff for usage, but once you install things, the resources, headers, binaries, etc. should be placed in the correct locations

This is normally what you use
add_custom_command(example POST_BUILD COMMAND ${CMAKE_COMMAND} -E copy_if_different ...)
for...


Draft: Abimo api

Replacement of the outdated Abimo precalculations and updating the Abimo-App with the current model.

This PR is deprecated.  #9  will include legend stylings that started here


Allow styling of the sidebar disclosure triangle icons

Enable styling/changing/hiding of the disclosure triangles next to folders in the explorer.

New icons!


Add code for school examples to examples repo #1295

Added code for centered_eight and non_centered_eight data to examples by loading the data using arviz.load_arviz_data.  Both files are in python notebooks (.ipynb) with comments added wherever necessary.

Only the PyMC3 one is enough I think. The most pressing goal is to make the code used to generate the examples in load_arviz_data available. In the issue I linked to the PyMC3 model used to generate the corresponding inference data objects, and the data is also available in the same notebooks. This will serve a double goal of helping users understand how the inference data object was generated and will ease updating the datasets whenever we update the inference data scheme.
It would also be great if you could make sure to use latest ArviZ and xarray releases. You'll see then that arviz.InferenceData objects have an html representation like the ones for xarray.Dataset and you won't need to have a cell for each group.

@sudip-padhye Why close this? Looks like you were on track

@canyon289 Sorry for that. I initiated a new pull request just now. I did this as I thought the updated files were not uploaded successfully and would need a new Pull Request to be initiated.


Trigger prod cli release - 0.18.3

Trigger prod cli release - 0.18.3
/hold
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/test eks-anywhere-presubmit

/unhold

/lgtm

/approve


[move-ide] Added virtualized source file reader

Description
This PR adds support for virtualized source file reader, allowing the compiler to build a package from files kept by the IDE in memory.
It also fixes an existing bug (https://github.com/move-language/move/issues/1082) where source files used in the symbolicator (obtained from resolution graph) and source files used by the compiler could be modified between the two uses resulting in different file hashes which can ultimately lead to crash when translating diagnostics (generated by the compiler and using "compiler file hashes") using symbolicator source files (and "symbolicator file hashes")
Test Plan
All existing tests must pass.

Superseded  by https://github.com/MystenLabs/sui/pull/16298


help wanted:question about windows

sir,I am unable to open the downloaded. exe file properly.Is it because my system is Win11?
Only win10 1903 and newer ones can work?
After I opened PatreonDownloader.App.exe, the pop-up window immediately disappeared.

https://github.com/AlexCSDev/PatreonDownloader/issues/172#issuecomment-1465239526


Pass the .js files by jsx loader

Specs need fixing

I have just removed JSX altogether. I think this will make things easier all around.


cdb_org_member_78ab1431540d62afc6035c120ead8c57 does not exist

Hello,
I have some problems deploying CartoDB with Docker.
When I try to create a new user using:
RAILS_ENV=development bundle exec rake cartodb:db:setup_user SUBDOMAIN="${SUBDOMAIN}"
PASSWORD="${PASSWORD}"
EMAIL="${EMAIL}"
I get the following error:
"PG::Error: ERROR:  role 'cdb_org_member_0c105a4c7f4a44eececb5c0d9dc8d660' does not exist"
"CONTEXT:  SQL statement "GRANT "cdb_org_member_0c105a4c7f4a44eececb5c0d9dc8d660" TO "development_cartodb_user_98eeac7c-8460-4ce5-9b83-95ca0806ecb5""
PL/pgSQL function cdb_organization_create_member(text) line 3 at EXECUTE statement"
All cartodb-postgresql.git installcheck tests had passed with no errors.
I run postgres, redis, sql-api, maps-api, and cartodb on different containers, with Ubuntu 12.04 images.
My kernel is 3.19.0-32-generic.
Thanks,
Eran

What docker image are you using?

ubuntu:12.04

https://github.com/sabala12/docker-cartodb

@sabala12 did you get it to work?

Yes

Hi,
I have the same issue, how did you solve it?

Hello,
If I remember correctly, I had to run "bundle exec rake db:migrate" before "
script/create_dev_user".
2016-07-27 8:40 GMT+03:00 GISlator<EMAIL_ADDRESS>
Hi,
I have the same issue, how did you solve it?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/CartoDB/cartodb/issues/6823#issuecomment-235492123,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ANehKBu5JwEwOs2m_S9t79iA_M50ocSQks5qZu9AgaJpZM4HsfTv
.

Thanks.
I still get the same error.

There has been no activity on this issue for more several months. We are closing it. If you think this still needs to be addressed please open a new issue.

I'm getting the same error on development database and I can't fix it with bundle exec rake db:migrate. If I recreate the database entirely, it doesn't appears, but this error continues appearing after some days of work. Why could it be happening?
D, [2019-02-26T12:59:17.916089 #21208] DEBUG -- :   Sequel::Postgres::Database (1.0ms)  SELECT cartodb.CDB_Organization_Create_Member('development_cartodb_user_f66d6fba-f391-49ee-b6aa-e7ce53f355e8');
E, [2019-02-26T12:59:17.916155 #21208] ERROR -- : PG::Error: ERROR:  role "cdb_org_member_f1f69e3b139d5800b89a1299feccef3e" does not exist
CONTEXT:  SQL statement "GRANT "cdb_org_member_f1f69e3b139d5800b89a1299feccef3e" TO "development_cartodb_user_f66d6fba-f391-49ee-b6aa-e7ce53f355e8""
PL/pgSQL function cdb_organization_create_member(text) line 3 at EXECUTE: SELECT cartodb.CDB_Organization_Create_Member('development_cartodb_user_f66d6fba-f391-49ee-b6aa-e7ce53f355e8');
D, [2019-02-26T12:59:17.916394 #21208] DEBUG -- :   Sequel::Postgres::Database (0.1ms)  ROLLBACK
I, [2019-02-26T12:59:17.916433 #21208]  INFO -- : (0.000161s) ROLLBACK
D, [2019-02-26T12:59:17.916782 #21208] DEBUG -- :   Sequel::Postgres::Database (0.2ms)  ROLLBACK
rake aborted!
PG::Error: ERROR:  role "cdb_org_member_f1f69e3b139d5800b89a1299feccef3e" does not exist
CONTEXT:  SQL statement "GRANT "cdb_org_member_f1f69e3b139d5800b89a1299feccef3e" TO "development_cartodb_user_f66d6fba-f391-49ee-b6aa-e7ce53f355e8""
PL/pgSQL function cdb_organization_create_member(text) line 3 at EXECUTE
/var/lib/gems/2.2.0/gems/sequel-3.44.0/lib/sequel/adapters/postgres.rb:145:in `exec'
/var/lib/gems/2.2.0/gems/sequel-3.44.0/lib/sequel/adapters/postgres.rb:145:in `block in execute_query'
/var/lib/gems/2.2.0/gems/sequel-rails-0.9.15/lib/sequel_rails/sequel/database/active_support_notification.rb:17:in `block in log_connection_yield'
/var/lib/gems/2.2.0/gems/activesupport-4.2.10/lib/active_support/notifications.rb:164:in `block in instrument'
/var/lib/gems/2.2.0/gems/activesupport-4.2.10/lib/active_support/notifications/instrumenter.rb:20:in `instrument'
/var/lib/gems/2.2.0/gems/activesupport-4.2.10/lib/active_support/notifications.rb:164:in `instrument'
/var/lib/gems/2.2.0/gems/sequel-rails-0.9.15/lib/sequel_rails/sequel/database/active_support_notification.rb:11:in `log_connection_yield'
/var/lib/gems/2.2.0/gems/sequel-rails-0.9.15/lib/sequel_rails/sequel/database/active_support_notification.rb:28:in `log_yield'
/var/lib/gems/2.2.0/gems/sequel-3.44.0/lib/sequel/adapters/postgres.rb:145:in `execute_query'
/var/lib/gems/2.2.0/gems/sequel-3.44.0/lib/sequel/adapters/postgres.rb:132:in `block in execute'


Pages don't link yet

Our test community pages do not link back to the main page and the main site does not always link back to the pages yet.

Solved with 14bff6f81607122bea779d465f505dc5c5f89058


[DO NOT MERGE]

debugging only

"Debug job" is only available from travis-ci.com.  Forking temporarily to debug.


Can we create 100+ databases in Azure MI?

Is this possible to create??

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

ID: 4dff594a-77ba-dca2-e3ae-f3a7bf0c3e1d
Version Independent ID: b496ce1e-c8e6-8fe4-d41d-6f57606f5034
Content: Resource limits - Azure SQL Managed Instance
Content Source: articles/azure-sql/managed-instance/resource-limits.md
Service: sql-managed-instance
Sub-service: operations
GitHub Login: @bonova
Microsoft Alias: bonova

@Anmolgan81 Thank you for the feedback. We are actively investigating and will get back to you soon

@Anmolgan81 Please follow this  document for more info about the MI resource limits. With regards to number of DB, the limit is 100.

@NavtejSaini-MSFT Thanks for providing the information
@Anmolgan81 Please let us know if you need more details.


Allow body to be empty

This change allows the body to be completely empty, that means without
gzip headers. This a requirement with status codes that do not allow
for a body, for example 204 No Content.
It also changes to a lazily initialise method for the gzip writer.
Fixes #5.

I've updated the code to run on < Go 1.7.

Ping.

Sorry for the delay, we had a change of repo ownership in the past week and this fell through the gaps.
This looks good to me. Thanks for contributing!

Thanks for adopting this one, @jprobinson! (And sorry, @Thomasdezeeuw, for leaving you hanging.)

Just to add an update...we found this introduced a bug for any handler calling WriteHeader before writing the actual body to the response.
We ended up altering the library in #19 to still allow empty responses, but only for 204 and 304.

Thanks.


Domain socket support in UI testing.

Remove Welder-web stand-alone UI test.
Add domain socket support for Cockpit integrated welder-web.
Add domain socket support for end to end test.
Fix a page element.
Increase wait time to 60 seconds.

I have another PR which packages lorax-composer and web in one container, then run end to end test against it.
My plan is to run end to end test on two scenarios:

welder-web + cockpit + bdcs-api.
welder-web + cockpit + lorax-composer.
This PR is for the first scenario. Sorry for making you confuse.

Thanks - my initial comment was made when I was only considering the lorax-composer case, and I'm unable to delete it so I'm just going to let it stay.  It doesn't apply to the case of testing bdcs.

The cockpit-test passed. Unit test issue will be fixed in a separate PR.


fix(provider/azure): Stop making containers public

This is an attempt to fix:

https://github.com/spinnaker/spinnaker/issues/5892
https://github.com/spinnaker/spinnaker/issues/6218

The following PRs attempted to address this issue, and appear to be
orphaned:

https://github.com/spinnaker/front50/pull/900 - Failing checks.
https://github.com/spinnaker/front50/pull/1002 - Fixing more than one
thing.  Should be two (2) PRs.

@jasonmcintosh, Ready to go.

@spinnaker/reviewers
Ahoy!
Could someone please take a look?  Let me kno

@pedrohdz I'm watching this now. Could you please rebase one more time and we can get this merged?

@jkschneider, Done...  Thanks!


refactor: useEstimateTransactionGas hook

What it solves
Resolves #3499
How this PR fixes it
With the TxModalWrapper refactor, the hook is only being used in one place. We only estimate gas now if a transaction is being executed.

Split up safeTxGasEstimation and gasEstimation
Move isCreation and isOffchainSignature out of the hook
Remove gas estimation for case isOffChainSignature && !isCreation
Only do gasEstimation if transaction is executing
Call estimateGasForTransactionExecution instead of estimateTransactionGasLimit
Remove usage of estimateGasForTransactionApproval
Remove unused manualSafeNonce, providerName from custom hook dependencies
Remove manualSafeNonce from TxModalWrapper
Pass canTxExecute instead of declaring it twice with different params
Rename canTxExecute to isExecution
Rename gasEstimation to safeTxGasEstimation

Added unit tests
Unit tests for useEstimateTransactionGas:

Should return manualGasPrice if it exists instead of estimation
Should return manualMaxPrioFee if it exists instead of estimation
Should return manualGasLimit if it exists instead of estimation
Should return failure state if checkTransactionExecution is false
Should return failure state and default estimations if getFeesPerGas throws
Should return failure state and default estimations if estimateGasForTransactionExecution throws
Should return failure state and default estimations if checkTransactionExecution throws
Should return failure state and default estimations if calculateGasPrice throws
Should return success state if checkTransactionExecution is true
Should return gasMaxPrioFee = 0 if pre EIP-1559

Unit tests for estimateSafeTxGas:

Should return 0 if feature flag SAFE_TX_GAS_OPTIONAL is set for safe version

How to test it
All transaction types and safe owner setups need to be tested

Send Funds in a 1/n safe and execute immediately
Send Funds in a 1/n safe and queue
Send Funds in a 1/n safe and execute from queue
Send Funds in a 2+/n safe and queue
Send Funds in a 2+/n safe and confirm and execute
Send Funds in a 2+/n safe and confirm
Send Funds in a 2+/n safe and execute from queue
Send Funds in a 2+/n safe and execute as a non-owner
...

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

44 of 45 (97.78%)  changed or added relevant lines in 5 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage increased (+0.7%) to 34.362%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/routes/safe/components/Transactions/helpers/TxModalWrapper/index.tsx
17
18
94.44%

Files with Coverage Reduction
New Missed Lines
%

src/routes/safe/components/Transactions/helpers/TxModalWrapper/index.tsx
1
65.89%

Totals

Change from base Build<PHONE_NUMBER>:
0.7%

Covered Lines:
3277

Relevant Lines:
8566

💛  - Coveralls

Tried the cases that were listed in the description plus tx rejections and the use of trezor, ledger and WC (MM phone app)
I also tried tx in Optimism since it doesn't use EIP1559
I had a conversation with Usame regarding some "estimation" calls done for tx that do no execute, but it seems that those are only used to get the recommended nonce and not the safetxGas
Looks good to me


Fix bug when evaluating resource status during service startup check

We have seen servers sometimes fail to pass the service status checker until the timeout is reached, even after all segments are online/in the expected state. Logs show:
Sleep for 10000ms as service status has not turned GOOD: MultipleCallbackServiceStatusCallback:IdealStateAndCurrentStateMatchServiceStatusCallback:Helix state does not exist, waitingFor=CurrentStateMatch, resource=table_REALTIME, numResourcesLeft=2, numTotalResources=802, minStartCount=802,;IdealStateAndExternalViewMatchServiceStatusCallback:Init;;

This is due to this check, which considers the table resource to have STARTING status if the external view/current state is null and ideal state is not. However, this isn't a valid assumption since the current state can be null if the last segment on the server is removed and the ideal state still exists. We primary see this behavior with completed segment redistribution turned on, on small tables.
The change here is meant to allow the resource status to return GOOD if the instance is no longer assigned any segment (when the server first started and collected all resources to monitor it was assigned).

Codecov Report
Attention: Patch coverage is 0% with 14 lines in your changes missing coverage. Please review.

Project coverage is 0.00%. Comparing base (59551e4) to head (714c736).
Report is 715 commits behind head on master.

Files
Patch %
Lines

...a/org/apache/pinot/common/utils/ServiceStatus.java
0.00%
14 Missing :warning:

:exclamation:  There is a different number of reports uploaded between BASE (59551e4) and HEAD (714c736). Click for more details.
HEAD has 53 uploads less than BASE

Flag
BASE (59551e4)
HEAD (714c736)

integration
7
1

integration2
3
1

temurin
12
1

java-21
7
1

skip-bytebuffers-true
3
0

skip-bytebuffers-false
7
1

unittests
5
0

unittests1
2
0

java-11
5
0

unittests2
3
0

integration1
2
0

custom-integration1
2
0

Additional details and impacted files
@@              Coverage Diff              @@
##             master   #13541       +/-   ##
=============================================
- Coverage     61.75%    0.00%   -61.76%
=============================================
  Files          2436     2482       +46
  Lines        133233   137220     +3987
  Branches      20636    21301      +665
=============================================
- Hits          82274        0    -82274
- Misses        44911   137220    +92309
+ Partials       6048        0     -6048

Flag
Coverage Δ

custom-integration1
?

integration
0.00% <0.00%> (-0.01%)
:arrow_down:

integration1
?

integration2
0.00% <0.00%> (ø)

java-11
?

java-21
0.00% <0.00%> (-61.63%)
:arrow_down:

skip-bytebuffers-false
0.00% <0.00%> (-61.75%)
:arrow_down:

skip-bytebuffers-true
?

temurin
0.00% <0.00%> (-61.76%)
:arrow_down:

unittests
?

unittests1
?

unittests2
?

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

I don't fully follow the change. When there is no partition assigned to the given instance, the loop should all be skipped, and it should return GOOD right?

Not sure I understand, are you referring to this loop? If so, the if condition linked in the PR description shows that it will return STARTING status if the current state is null
Or if you mean the outer loop, the issue is that _resourceIterator collected all assigned resources when the server first started up. The resources being iterated through are not necessarily still assigned to the server, and should be re-evaluated

My bad, now I see the problem.
Should we consider first looping over the ideal state and create a map from serving partition to state, then only loop over this map in the second loop? Similar to the current approach but without the early termination in the first loop.

My bad, now I see the problem. Should we consider first looping over the ideal state and create a map from serving partition to state, then only loop over this map in the second loop? Similar to the current approach but without the early termination in the first loop.

Yeah, I think that is clearer. Updated to use idealState.getRecord().getMapFields() as well


Plugin user confusion

The documentation states:
"# The plugin is run as kojid user so that user needs permissions for signing packages:
su kojiadmin; koji grant-permission admin kojid"
We do not run any builders on our hub so there is no kojid user on the hub.  Is the plugin meant to be run by the hub or by each builder?  If it is meant to be run by the hub, then what user does the plugin run as when there is no kojid user or can the user be specified?

Hm. I havent tested it with a tiered setup. But the signer plugin should be run on the Hub I think. If there is no kojid user on the hbu without a builder what other user is running the plugins? Its probably the same user as the hub is running as. Unfortunately I cant test this now as I dont have a running Koji setup anymore.

You ask a very good question that I have been wondering myself.  I'm having an issue finding that out.  We've never run plugins before in our Koji setup.  I am fairly sure Koji is a WSGI application, so if I find the correct config I could try setting the user it runs as.  As is, I can only guess that Koji itself is running as root or apache.  Alternatively, I could try finding and running a benign plugin that may give me some clue.

the plugin is triggered on "postTag". so let it build a target so plugin gets triggered and meanwhile have a ps afxu running so you see the user the plugin is run as

I'm leaving my job at the end of the week and I was just trying to use your plugin to make it easier on the other employees by taking out a step (setting up sigul client).  I went ahead and closed the issue because i have to focus on getting them set up for doing Koji builds, signing and syncing to Spacewalk.  If I have time after that, I'll definitely give your suggestion a try.  I definitely appreciate the pointers and, if I do get a chance to look at this further, I'll come back and let you know what worked so we can spread the knowledge!


fixes in v4 for front end frameworks

Summary

Related Issue: # (issue number)
Change Type

[ ] Bug Fix
[ ] Refactor
[ ] New Feature
[ ] Breaking Change
[ ] Documentation Update
[ ] Performance Improvement
[ ] Other

Checklist

[ ] My code follows this project's style guidelines
[ ] I've reviewed my own code
[ ] I've added comments for any hard-to-understand areas
[ ] I've updated the documentation if necessary
[ ] My changes generate no new warnings
[ ] I've added tests that prove my fix is effective or my feature works
[ ] All unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published

Additional Information

Branch Naming

I can easily revert these if we manage esm build problems.
 - "build": "yarn unbuild && yarn build:cjs && yarn build:esm && yarn build:typ",
 + "build": "yarn unbuild && yarn build:tsc",

but this needs to go
- import { extractChainIdFromBundlerUrl } from "utils/Utils";
+ import { extractChainIdFromBundlerUrl } from "./utils/Utils";