training issues on activitynet1.3 and TACoS

Excellent work!
I train the VLG-Net with your code and the provided resources, and I have two issues needing your help.

I train the VLG-Net on activitynet1.3 with your code, but only get Rank<EMAIL_ADDRESS>and Rank<EMAIL_ADDRESS>when inference. I don't know what's wrong and how to fix it.
When I train the VLG-Net on the TACoS dataset, the training loss of the first two batches is seemingly normal at epoch 1, and then it becomes to be 'nan'.

Thanks!!!

Dear @NeverMoreLCH thank you for reaching out.
Kindly provide me with a snapshot of your environment and the command you are using to train the models.
I will try and reproduce your issue on my end and give you a solution soon.

Dear @Soldelli , here is my conda environment.
I use 'python train_net.py --config-file configs/activitynet.yml OUTPUT_DIR outputs/activitynet' and 'python train_net.py --config-file configs/tacos.yml OUTPUT_DIR outputs/tacos' to train the models. And I just change the batch size to 16 when training on TACoS.
Thanks!
name: lchpy37_VLG
channels:

pytorch
cogsci
anaconda
http://mirrors.tuna.tsinghua.edu.cn/anaconda/pkgs/main
conda-forge
http://mirrors.tuna.tsinghua.edu.cn/anaconda/pkgs/free
http://mirrors.tuna.tsinghua.edu.cn/anaconda/cloud/conda-forge
https://mirrors.tuna.tsinghua.edu.cn/anaconda/cloud/conda-forge/
https://mirrors.tuna.tsinghua.edu.cn/anaconda/cloud//pytorch/
https://mirrors.tuna.tsinghua.edu.cn/anaconda/pkgs/main/
https://mirrors.tuna.tsinghua.edu.cn/anaconda/pkgs/free/
defaults
dependencies:
_libgcc_mutex=0.1=conda_forge
_openmp_mutex=4.5=1_gnu
blas=1.0=mkl
brotlipy=0.7.0=py37h5e8e339_1003
ca-certificates=2021.10.8=ha878542_0
certifi=2021.10.8=py37h89c1867_1
cffi=1.15.0=py37h036bc23_0
charset-normalizer=2.0.8=pyhd8ed1ab_0
colorama=0.4.4=pyh9f0ad1d_0
cryptography=36.0.0=py37hf1a17b8_0
cudatoolkit=10.2.89=h8f6ccaa_9
cymem=1.31.2=py37h6bb024c_0
cytoolz=<IP_ADDRESS>=py37h14c3975_1001
dill=0.2.9=py37_0
freetype=2.10.4=h0708190_1
h5py=2.10.0=py37hd6299e0_1
hdf5=1.10.6=hb1b8bf9_0
idna=3.1=pyhd3deb0d_0
intel-openmp=2021.4.0=h06a4308_3561
jbig=2.1=0
joblib=0.16.0=py_0
jpeg=9d=h36c2ea0_0
lcms2=2.12=hddcbb42_0
ld_impl_linux-64=2.36.1=hea4e1c9_2
lerc=3.0=h9c3ff4c_0
libblas=3.9.0=12_linux64_openblas
libcblas=3.9.0=12_linux64_openblas
libdeflate=1.8=h7f98852_0
libffi=3.4.2=h7f98852_5
libgcc-ng=11.2.0=h1d223b6_11
libgfortran-ng=7.3.0=hdf63c60_0
libgfortran5=11.2.0=h5c6108e_11
libgomp=11.2.0=h1d223b6_11
liblapack=3.9.0=12_linux64_openblas
libnsl=2.0.0=h7f98852_0
libopenblas=0.3.18=pthreads_h8fe5266_0
libpng=1.6.37=h21135ba_2
libstdcxx-ng=11.2.0=he4da1e4_11
libtiff=4.3.0=h6f004c6_2
libwebp-base=1.2.1=h7f98852_0
libzlib=1.2.11=h36c2ea0_1013
lz4-c=1.9.3=h9c3ff4c_1
mkl=2019.4=243
msgpack-numpy=<IP_ADDRESS>=pyh9f0ad1d_0
msgpack-python=0.5.6=py37h6bb024c_1003
murmurhash=0.28.0=py37he1b5a44_1001
ncurses=6.2=h58526e2_4
ninja=1.7.2=0
numpy=1.21.4=py37h31617e3_0
olefile=0.46=pyh9f0ad1d_1
openjpeg=2.4.0=hb52868f_1
openssl=1.1.1l=h7f98852_0
pillow=8.4.0=py37h0f21c89_0
pip=21.3.1=pyhd8ed1ab_0
plac=0.9.6=py_1
preshed=1.0.1=py37hf484d3e_1000
pycparser=2.21=pyhd8ed1ab_0
pyopenssl=21.0.0=pyhd8ed1ab_0
pysocks=1.7.1=py37h89c1867_4
pyspellchecker=0.5.2=py_1
python=3.7.12=hb7a2778_100_cpython
python_abi=3.7=2_cp37m
pytorch=1.5.1=py3.7_cuda10.2.89_cudnn7.6.5_0
readline=8.1=h46c0cb4_0
regex=2018.07.11=py37h470a237_0
requests=2.26.0=pyhd8ed1ab_1
scipy=1.1.0=py37h7c811a0_2
setuptools=59.2.0=py37h89c1867_0
spacy=2.0.12=py37h962f231_0
sqlite=3.36.0=h9cd32fc_2
termcolor=1.1.0=py_2
terminaltables=3.1.0=py_0
thinc=6.10.3=py37h637b7d7_1003
tk=8.6.11=h27826a3_1
toolz=0.11.2=pyhd8ed1ab_0
torchvision=0.6.1=py37_cu102
tqdm=4.62.3=pyhd8ed1ab_0
ujson=4.2.0=py37hcd2ae1e_1
urllib3=1.26.7=pyhd8ed1ab_0
wheel=0.37.0=pyhd8ed1ab_1
wrapt=1.13.3=py37h5e8e339_1
xz=5.2.5=h516909a_1
yaml=0.2.5=h7b6447c_0
zlib=1.2.11=0
zstd=1.5.0=ha95c52a_0
pip:

en-core-web-sm==2.0.0
future==0.18.2
importlib-metadata==4.8.2
mock==4.0.3
prettytable==2.4.0
protobuf==3.19.1
psutil==5.8.0
pytorch-model-summary==0.1.2
pyyaml==6.0
sentencepiece==0.1.96
six==1.16.0
stanfordcorenlp==<IP_ADDRESS>
tensorboardx==2.4.1
torchsummary==1.5.1
torchtext==0.6.0
typing-extensions==4.0.0
wcwidth==0.2.5
yacs==0.1.7
zipp==3.6.0

Dear @NeverMoreLCH I apologize for the delay.
I verified that I could reproduce evaluation and training on my side using my environment.
I was not able to easily install the environment you provided, but I will try my best to create one manually and match all the libraries.

I kindly ask you to provide me with the training logs (please just zip the output folder and send it to me).
Can you confirm you can run an inference with the pre-trained models and obtain the numbers mentioned in the repo?
Regarding TACoS, I believe something funky might be happening. Note that modifying the batch size call for a rescaling of the learning rate. Please do try to search across different hyperparameters settings.

I will do my best to help you. We can schedule a zoom call if you cannot solve the issues. If that is the case, please shoot me an email.

Dear @Soldelli , this is my training logs for activitynet, download link.
In addition, I can confirm that the test results with your pre-trained models are same as that in your repo.
Thanks for your reply!

Dear @NeverMoreLCH I notice from the logs that the loss is decreasing while the performance in evaluation is stable and not changing. Moreover, such performance are on par with random selection of the proposals. In my experience, after the first epoch you should reach at least 14% for R@1 IoU=0.7.

Please check that the updated model is correctly passed from the training script to the evaluation script at each inference call in the lib/engine/trainer.py file. Did you make any changes to those functions?
I believe there is no mistake in training nor in inference as the loss is decreasing correctly and you are able to reproduce the result of the pre-trained models.

I suggest cloning the repo from scratch and trying to train before making any change. According to my test, it should work off the shelf. Feel free to reach out anytime.
Best,
Mattia

Excellent work!
I train the VLG-Net with your code and the provided resources, and I have two issues needing your help.

I train the VLG-Net on activitynet1.3 with your code, but only get Rank<EMAIL_ADDRESS>and Rank<EMAIL_ADDRESS>when inference. I don't know what's wrong and how to fix it.
When I train the VLG-Net on the TACoS dataset, the training loss of the first two batches is seemingly normal at epoch 1, and then it becomes to be 'nan'.

Thanks!!!

hello! I also meet the problem 2. Could you tell me how you solve it finally?

Hi @tujun233 I am currently out of office so I cannot delve into the code right now.

Did you try reducing the learning rate as suggested?

Otherwise if that does not help, try debugging with IPDB setting the breakpoint in the loss calculation line and double checking why the insurgence of Nan happens. For this, I suggest setting the number of workers to zero in the config file. This created a data loader with a single thread and allows you to set breakpoints in any part of the code (even inside the training loop or model functions).

Hi @tujun233 I am currently out of office so I cannot delve into the code right now.

Did you try reducing the learning rate as suggested?
Otherwise if that does not help, try debugging with IPDB setting the breakpoint in the loss calculation line and double checking why the insurgence of Nan happens. For this, I suggest setting the number of workers to zero in the config file. This created a data loader with a single thread and allows you to set breakpoints in any part of the code (even inside the training loop or model functions).

Hi!
I try reducing the learning rates as suggested. But it doesn't work.
Through my debugging, I found that the GloVe tokens were fed into an LSTM with 5 layers in lstm_syntacGCN_encoder in /lib/modeling/language_modeling.py. Its outputs include many NaN. It happens when I train tacos datasets and change the batchsize to 8 in tacos config file. I don't know why, and I will try whether it will still happen when using original batchsize 32.

Dear @Soldelli , I clone the repo from scratch and trying to train before making any change. But I meet the same two problems as @NeverMoreLCH .

Dear @tujun233 I apologize for the inconvenience.
I will reach out with a solution as soon as I am back in the office.
Just to make sure:

Are you training with the default config file?
Are you training on a v100 (due to memory requirements)?
Are you using the same versions of the libraries as reported in the documentation?

I never faced the issue you are reporting so I need to be able to reproduce it before finding a solution. When I cloned from scratch the repo I was able to train the model correctly.
Best,
Mattia

Dear @Soldelli , I apologize for my delay,
Yes, I am sure that using default config file and the same versions of the libraries as reported. But I don't have v100,  I train it on single RTX 2080 Ti.
I also try my best to solve the problem I meet.

Hi! There is a good news that I have solved the problems. I install pytorch1.7.1 and torchvision0.8.2 on the official web. Then the code works. I guess that I use https://mirrors.tuna.tsinghua.edu.cn/anaconda/cloud/pytorch/ to install pytorch1.5.1 before for saving time. Maybe there is something wrong with the version of pytorch. And thank you for help!
Best,
tujun233

Dear @tujun233 that is great news indeed.
You might need to run hyperparameters search on the learning parameters now that you bumped down the batch size.
Please feel free to reach out for any other doubt or issue.
Cheers.


Update API to use Last 24 hours by default

In case start time and end time is not specified, use last 24 hour window instead of fixing start time to 00:00 UTC of that day.
One problem in the later approach is every time you query the API, response has variable points depending on the time of the day. If you happen to query during 00:00 to 00:05 AM UTC timeframe, API returns no data as StartTime is set to 00:00 AM UTC and end Time is very near to start time.

I already have a fix for this on my local deployment. will send a CR.

deployed and tested


Generating multiple files from the same markdown page

Hi,
we would like to generate multiple files from the same markdown page, but it seems that Jekyll does not support that. Do you have any suggestion on how to do that?
Here is what we tried and the reasons behind this need: https://github.com/gbif/hp-living-norway/issues/11
Thank you :)

I'm afraid I do not know up front how to do this either, but the generators you mention sounds right to me.
Reading issue gbif/hp-living-norway#11 I'm not sure that it is the correct approach either though. But I'm not entirely sure what the goal is - especially as the terms collections and projects are already in use in the GBIF APIs, but I suspect you mean something else.

Hi Morten :)
From our limited understanding, it is not possible to get all the datasets belonging to a project, as /dataset/suggest returns a maximum of 20 results. collections seem to be a group of institution and people, not datasets, so I would say that we are looking on how to define collections of datasets. We are relying on regular tags at the moment.

If we mean the same thing with project then you can get all dataset belonging to a project by filtering by the projectId and page over those results
https://api.gbif.org/v1/dataset/search?projectId=GRIIS
https://api.gbif.org/v1/dataset/search?projectId=GRIIS&size=100&offset=100
/dataset/suggest is intended for autocompletes - as in the user types ebi and the suggest returns an array of datasets that are typically shown in a dropdown.

And you are right - in GBIF terminology a collection is best understood as a museum collection. E.g. a herbarium from a specific institution. So it isn't a collection of datasets as such.

Thank you, Morten. We did not tought it was possible, as projectId is not mentioned as possible parameter for the /dataset/search endpoint :)

That is a mistake. I have added that. Thank you

@frafra I'm closing this issue as I believe you have already identified the correct approach - namely generators.
Lets continue the discussion about the Living Norway specific issue as gbif/hp-living-norway#11
Let me know if you believe this should be reopened.


predicted-result will be different for same video input and model

Hi Qijie,
I just found the computed probability will  be changed for same input and model. For example, for the video 12065806-102-009-013814.mp4, I predicted this video using your codes (have added softmax layer) and the predicted result will be different if run the command "python3 p3d_model.py 12065806-102-009-013814.mp4" multiple times
In my view, same video input, same model  and same weights, same codes, It must be same predicted result. Could you please help to analysis or fix it? Thanks in advance.

Predict a video using p3d is non-trivial, If you want to get the same result as the author's, you should first implement the same sampling steps and pre-processing steps. Detailly, 1) evenly sample continuous 16-frames from the given video. 2)pre-process these frames together, including 'resize' and 'normalize'. 3) average the results of the tested clips(each clip is 16-frames long).
Alternatively, you can sample like you want, you can also compute the final result with linear weights of all the results.  If you fix these, I believe the result will not vary anymore.


New feature to include BCC for multiple receivers

Hi Team,
American Express has signed the Corporate CLA (CCLA). Even I have signed the individual CLA. Still it is showing checks failed. Please help.
Thanks


Focus tests on Node LTS releases

The Travis file currently lists 0.10, 0.12, and 4.1:
https://github.com/indexzero/http-server/blob/master/.travis.yml#L3-L6
Based on the LTS schedule, I think it's prudent (sane) to:

[ ] drop 0.10
[ ] (possibly) drop 0.12
[ ] add 6.0
[ ] add 8.0 (once Travis is ready for that)

Think that should limit the pain of maintenance a bit. I'm primarily looking for feedback about dropping 0.12.
Thanks!

Node.js 8.0.0 is dropping in few days, might as well include it. Though it won't be LTS until 6 months later, but it everything goes according to the plan 8.0 => 8.x should be totally compatible.

Thanks @franciscop! Added it to the list.

In a nodejs.org survey April 2016, ~18% reported using 0.12 in production with 80% scheduled for upgrade to v4 within the year. Maybe 2017 survey results will be released soon and offer more recent data.
Seems safe to drop.

Version 8.0.0 is about to drop soon

I would drop 0.12 because it doesn't support es6

#370 has the goods. It's using Node.js 4-7 and removes some cruft from ages past.
Holler if something looks off. :grin:


rpc error: code = DeadlineExceeded desc = context deadline exceeded

Proto file(s)
List protocol buffer definitions
Command line arguments / config
List all command line arguments or config properties
Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:
Expected behavior
A clear and concise description of what you expected to happen.
Environment

OS: [e.g. macOS 10.14.3]
ghz: [e.g. 0.33.0]

Screenshots
If applicable, add screenshots to help explain your problem.

Additional context
Add any other context about the problem here.

Hello, can you please provide more information with a reproducible example. It seems like the server is unreachable by the ghz test client.

Hello,
could this be related to #169, #226 and #184?

This issue lacks reproducible details, but i think it may be resolved in 0.80.0

This issue lacks reproducible details, but i think it may be resolved in 0.80.0


How to set future date selection disabled on TextBox?

Bug Reports
You MUST have a JSBin recreation, or else your issue will be CLOSED without explanation.
Instructions: https://github.com/dalelotts/angular-bootstrap-datetimepicker/wiki/Reporting-Bugs

Overview of the Issue - if an error is being thrown a non-minified stack trace helps
Motivation for or Use Case - explain why this is a bug for you
angular-bootstrap-datetimepicker Version(s) - is it a regression?
Browsers and Operating System - is this a problem with all browsers or only specific ones?
Link to a live example - provide a live example (using JSBin, Plunker or JSFiddle.
Related Issues - has a similar issue been reported before?
Suggest a Fix - if you can't fix the bug yourself, perhaps you can point to what might be
causing the problem (line of code or commit)

Feature Requests
Search the issue tracker for an existing ticket before creating a new one.
Instructions: https://github.com/dalelotts/angular-bootstrap-datetimepicker/wiki/Requesting-Features
(Please erase the above text and begin typing. Thanks!)

Please use gitter or stackoverflow for usage questions - https://gitter.im/dalelotts/angular-bootstrap-datetimepicker

Ok thank you.


We need adaptive communication speed!

My device is compatible with HS and FS communication. It works in HS by default. When the HS handshake fails, it will automatically switch to the FS handshake.
I hope usb-sniffer can work in HS mode by default and be backward compatible with FS communication capture.
If working at a fixed communication speed, it will not meet my work needs in this case.

There is no way to automatically switch the speed. First of all, there is no reliable way of doing this. The speed negotiation is a process that can be interpreted one way by the tool and the other way by the devices.
And the more important thing - it is no the goal of this tool. This is a debugging tool that is supposed to be used in a controlled environment.
When debugging a device that is capable of operating at a multiple speeds, debug operation at each speed separately with a host that you know can do HS or FS. This way you know what speed is expected to be negotiated and you can set it up ahead of time.
If you need to understand why the speed detection failed, you can use either speed, since all of that happens with line-state changes, which are the same regardless of the configured speed.
If you can describe a legitimate scenario where this is necessary, I'll be glad to hear it. But I pretty much guarantee that I won't do auto speed switch.


Add fhir-validator profile

Draft PR  Still needs a fair bit of testing

Add Docker file for image with Alpine and JRE (this is a much smaller image than Ubuntu/JRE)
Add fhir-validator profile to validate fhir bundle against the HL7 FHIR validator
This will only work when Docker is installed.  A check will alert the user if Docker is not available.

Todo:

Add some tests
Try on system without Docker install to verify the correct behavior
Update docs

This is super cool! I have just skimmed through the code and I'm looking forward to trying it out.
One question: I believe the validator likes to have a local cache, something like ~/.fhir  where it can keep  profiles and value sets set it downloads in the course of validation.  Strictly a performance thing (i.e.  time to download dependencies -- not about correctness of results), but curious if you have run into this.

Tests are failing because the example-02 series of files have been modified by mistake in the v1.2.0 spec update (my bad); spec PR 217 fixes this, and will in turn fix the failing tests here.

Replacing this PR with another PR


Support UI Testing

Add support for easily faking network requests in your app during UI testing.

This would be a great feature to have.

So I'm to the point of wanting to use DVR to stub out network calls for XCTest UI scenarios. Is there more to do with this PR to make it mergeable. Can I help with any of that? Or have there been further developments since this PR opened that would make one able to use DVR with XCTest UI?
No pressure, I know OSS is a volunteer effort, just looking to help.


Error when cloning project

It seems that the current setting.gradle found in master, is missing a comma.
include ':ActivityRecognition:app', ':BasicLocationSample:java:app', ':BasicLocationSample:kotlin:app', ':Geofencing:app', ':LocationAddress:java:app', ':LocationAddress:kotlin:app' ':LocationUpdates:app', ':LocationUpdatesForegroundService:app', ':LocationUpdatesPendingIntent:app'
between ':LocationAddress:kotlin:app' and ':LocationUpdates:app'

thanks..,
this is the answer to error "Gradle startup error file setting: blah blah..."


[Task] Add a new monitoring type to monitor the FTP server

Description
At present, the FTP server has not been monitored in our project, but in some cases, we need to check the availability status of the file directory of the FTP server. Of course, there are some other indicators to be discussed.Let's take the first step！
Task List

[ ] Add an API to detect the availability of the file directory of the FTP server

I will try to complete this task！


Code looks correct - still fails when submitting

Challenge Change the Font Size of an Element has an issue.
User Agent is: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_12_4) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/58.0.3029.110 Safari/537.36.
Please describe how to reproduce this issue, and include links to screenshots if possible.


Config value for proxy.config.hostdb.host_file.interval is never read.

This config item is for setting how often the host_file is re-read.  It is documented and registered in RecordsConfig, but is never read into the hostdb_hostfile_check_interval variable.

We have this internally in Yahoo (I think this didn't make its way to ASF because HostDB was merged just recently):
From 71f1c3d1e4d8c57149be0605bd56f0ebf301121e Mon Sep 17 00:00:00 2001
From: "Alan M. Carroll"<EMAIL_ADDRESS>Date: Sun, 23 Jan 2022 16:05:07 -0600
Subject: [PATCH] Fix HostDB sync time variable.

(cherry picked from commit 0830c0c368f263fab58f3e051dd3a85e99c3e88c)
---
 iocore/hostdb/HostDB.cc | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/iocore/hostdb/HostDB.cc b/iocore/hostdb/HostDB.cc
index e8124863fe..01f6ee20ef 100644
--- a/iocore/hostdb/HostDB.cc
+++ b/iocore/hostdb/HostDB.cc
@@ -208,12 +208,13 @@ extern MgmtConverter const &HostDBDownServerCacheTimeConv;
 MgmtConverter const &HostDBDownServerCacheTimeConv = HostDBDownServerCacheTimeVar.Conversions;

 // Not run time configurable, therefore no support beyond this class needed.
-ConfigDuration HostDBSyncFrequency{hostdb_sync_frequency};
+ConfigDuration HostDBSyncFrequencyVar{hostdb_sync_frequency};

 void
 HostDB_Config_Init()
 {
   HostDBDownServerCacheTimeVar.Enable("proxy.config.http.down_server.cache_time");
+  HostDBSyncFrequencyVar.Enable("proxy.config.cache.hostdb.sync_frequency");
 }

 // Static configuration information

So proxy.config.cache.hostdb.sync_frequency replaces proxy.config.hostdb.host_file.interval?

So proxy.config.cache.hostdb.sync_frequency replaces proxy.config.hostdb.host_file.interval?

Great observation. @cmcfarlen  fortunately used the right variable in his #9039 PR. That said, I'll pull in the hostdb.sync_frequency patch.

Closing since the fix in PR #9039 is merged.


questions about t-SNE visualization in FlexiViT

Hi, FlexiViT is a very inspirational idea.
However, I'm kind of stuck at the t-SNE visualization in Fig. 6 of the paper.
Does t-SNE employ the arccosine-transformed CKA as the precomputed metrics ?
If so, how do we calculate the CKA similarity? Is it between a FlexiViT at different patch sizes and a standard ViT at a fixed patch size？

@simonster implemented this, personally I don't remember the details for sure, but IIRC, it's only FlexiViT, and shows CKA between the layers of FlexiViT executed at different patch-sizes.


Includes updates to incorporate magnification

Requesting Diyu to review the changes made to SVMBIR to incorporate the magnification.
This is a review request and not expecting any merge to the main branch unless the full review is completed

I'm closing this because we don't make these kind of merges directly into master.  Any changes should go into "prerelease".


(New) Design for FPC Deployment Model/Chaincode type implementation

Depends on resolution of issue #132

Should be mostly handled by #320


Allow users to zoom in on images that has been opened

Maybe we have to stop restraining the image size to 100%? Should images be scaled to a specific size beforehand?
How does this work on mobile with pinch-zoom?

Probably some solution revolving around responsive image ([srcset]) sizes for full image would be good here.


Index file update with text and examples

Added text and examples for using the python client to create a new portfolio, secure risk exposure data and load to a dataframe

@bcwaldon I've included the text for OBSERVED_SCORE and OBSERVED_FREQUENCY in the indicator filter to forward proof this write up


Autocomplete: Improve sampling code and prepare for Honeycomb export

This PR improves the client site traces for autocomplete and makes it ready to be propagated to Honeycomb. An example trace can be found here: https://ui.honeycomb.io/sourcegraph/environments/cody/datasets/cody-client/result/z87pN9yLSan/trace/rv3ajXLDNV7?fields[]=s_name&fields[]=s_serviceName&span=2edb453047488acc

The changes can be summarized as:

We add a suggested event that we set when a completion callback makes it to the end and will result in being visible by a user
We add all exposed feature flags after the completion is ready (this means it will include eventual flags that are being evaluated as part of the completion request) as well as all the metadata that would also be added to analytics pings
Fix a bug that caused spans to be much shorter than they were. Whoopsie!

These changes were tested with the following OTel collector processor config:
processors:
  tail_sampling:
    decision_wait: 30s
    num_traces: 100000
    expected_new_traces_per_sec: 10
    policies:
      name: and
      type: and
      and:
        and_sub_policy:
          # We only want to sample traces for completions that are displayed to the user. We track
          # this with a suggested event on the root span
          - name: completion-was-suggested
            type: ottl_condition
            ottl_condition:
              error_mode: ignore
              spanevent:
                - "name == \"suggested\""
          # Only track events that originate from the completion provider
          - name: is-completion-event
            type: ottl_condition
            ottl_condition:
              error_mode: ignore
              span:
                - "name == \"autocomplete.provideInlineCompletionItems\""
          # Only track spans for users that are expose to the tracing experiment
          - name: is-tracing-enabled
            type: ottl_condition
            ottl_condition:
              error_mode: ignore
              span:
                - "attributes[\"cody-autocomplete-tracing\"] == true"
          # # Ignore completion callbacks that return the last candidate. We only care for traces that
          # # show a new completion
          - name: is-completion-event
            type: ottl_condition
            ottl_condition:
              error_mode: ignore
              spanevent:
                - "attributes[\"source\"] != \"LastCandidate\""

Test plan

Configure cody locally and run otel configurator (see https://docs.sourcegraph.com/dev/how-to/opentelemetry_local_dev)
Apply the filter that I pasted above into the honeycomb config and make sure that it's piped correctly

UPDATE: Use the latest version in the comment below

In OpenTelemetryService.node.ts remove the dev mode spanProcessor.
Look the events pipe into honeycomb (e.g.  https://ui.honeycomb.io/sourcegraph/environments/cody/datasets/cody-client/result/z87pN9yLSan/trace/rv3ajXLDNV7?fields[]=s_name&fields[]=s_serviceName&span=2edb453047488acc)

I made some more changes, now we have:

More granular logging on the cody gateway upstream request for fireworks (every yielded response chunk is now logged)
Make a sampling decision on the client

This also updates the tail sampler config used but I feel more confident that this provides actual value now, check:

With the following sampler config:
processors:
  tail_sampling:
    decision_wait: 30s
    num_traces: 100000
    expected_new_traces_per_sec: 10
    policies:
      name: and
      type: and
      and:
        and_sub_policy:
          # Only track events that originate from the completion provider
          - name: is-completion-event
            type: ottl_condition
            ottl_condition:
              error_mode: ignore
              span:
                - "name == \"autocomplete.provideInlineCompletionItems\""
          # Only track spans that were manually marked as to be sampled by the client
          - name: is-sampled
            type: ottl_condition
            ottl_condition:
              error_mode: ignore
              span:
                - "attributes[\"sampled\"] == true"

Did some more cleanup now. Here's a recent example: https://ui.honeycomb.io/sourcegraph/environments/cody/datasets/cody-client/result/FbSAQhtzuid/trace/96XAibG4QyM?fields[]=s_name&fields[]=s_serviceName&span=aded4798e8db6458
I’m very happy with this now. Let's try to enable this for client traces to Honeycomb and see how far we get by manually following the trace headers into the Cody Gateway traces

@valerybugakov This one is ready to review now for reals :D

The OTel collector changes are deployed, we can actually test this against dotcom now 😎
https://ui.honeycomb.io/sourcegraph/environments/cody/datasets/cody-client/result/wB[…]1prv/trace/xxaYoFdmQJX?fields[]=s_name&fields[]=s_serviceName


docs(*): address TODOs; update k3d img; set node-installer img; misc

address/delete TODOs
update k3d img via the v0.12.0 containerd-shim-spin release

cc @kate-goldenring to double-check shim release schedule and if we'll want/need to update this again soon.

set node-installer image during kwasm-operator install steps, from same release ^^
add/use SpinApp, SpinAppExecutor CRD entries to glossary

closes https://github.com/spinkube/documentation/issues/73

some spots where we weren't instructing to install the shim-executor
some spots to align cert-manager version across docs (to v1.14.3)

@vdice v0.12.0-rc1 was @Mossaka trying out the release pipeline. I would use the v0.12.0 release: https://github.com/spinkube/containerd-shim-spin/releases/tag/v0.12.0
We will probably cut a v0.13.0 release tomorrow with precompilation. Are these docs todos or also updating the defaults of the helm chart?

Thanks @kate-goldenring

We will probably cut a v0.13.0 release tomorrow with precompilation.

Okay, we can s/v0.12.0/v0.13.0 in the docs for the k3d and node-installer imgs when that is ready, assuming testing looks good.

I would like to merge this ASAP to prevent file conflicts, as other changes are going in.


Master

wordpress added

https://github.com/sindresorhus/awesome/blob/master/contributing.md

You also have a merge conflict.

Closing for lack of response.


added my name to README.md

What changes does this PR do?
Added my name to README.md
Screenshots (If any)
Questions

closing due to inactivity


Avoid using console in code that runs on browser

I have found a console.log() statement in services/github/submit-query.ts, I think it not advisible to use console in the code that runs on the browse

r

That code actually runs on the server. You shouldn't be seeing it on the browser console. Any chance you can send over a screenshot or something?


need-rebase plugin will not remove label when conflicts solved

Bug Report
need rebase can not remove the needs-rebase label automatically when the pull request has no conflicts

/remove-lifecycle rotten

upgrade the plugin codes, hope it can solve it:

https://github.com/PingCAP-QE/ee-ops/pull/1236


Broken link

There was a broken link in the Dependency injection section - higher-order component which links to this broken link https://github.com/krasimir/react-in-patterns/tree/master/patterns/higher-order-components.
Is it means the this link? https://krasimir.gitbooks.io/react-in-patterns/content/chapter-4/#higher-order-component

@SangKa yep. That was the intention.

Got it. I will fix.


Decouple Prompt Management with the Provider

Ideally, the prompt library management should work in a highly generic manner with no dependency on the LLM provider. Sorry if I am confused or missing anything. At the moment, I see the prompt creation is directly tied to the LLM Provider and model. This is a huge problem as the consumers won't be able to reuse the prompt template and run it against the preferred LLM provider.

Hi @ranjancse26 - Currently, once a prompt is saved on the platform, we do not allow changing the provider. The  workaround for this is to create a copy of the prompt using the duplicate button that is present on the prompt list screen. This will take you the prompt playground screen where everything is copied from the source prompt. Here, you can change the provider and then save it.
We are planning to reduce this friction in the next few days by allowing users to change provider in an already saved prompt. I hope that will solve your problem.

@VisargD Duplicate the prompt is not a solution. Sorry, I see a tight coupling and is creating a problem than help. When you say prompt library management, the whole prompt management itself should be designed in a highly decoupled manner. I hope you do understand the real issue with the current design.

As I mentioned in my comment, we are already working on reducing the friction by allowing to change provider in an already saved prompt. That should ideally address the issue.

The underlying system of prompt library management is not tightly coupled with a provider. Its just a UI/UX restriction that we had which does not allow users to switch between different providers. But as mentioned earlier, it should be solved in the next few days

@VisargD Any updates on this one? It's a major roadblock for our evaulation.

This is live now. You can change provider in saved prompts as well now.

@VisargD Sorry, I am not sure what updates you guys have provided as I still see the same as before. I got an error while switching the provided.

Thanks for flagging this, @ranjancse26. If you try hard refresh now, the Portkey app should get updated and you should be able to switch providers in the prompt playground. Please let me know if you're able to do that!

@vrushankportkey I did a logout, also tried with the hard refresh too. However, the error still persists. You can log in to your account. Choose one of the existing prompt template, say Anthropic. Try to switch the provider and see the error during the LLM provider switch.

Ok, checking this on priority. cc @sabbyasachiRPortkey will get back!

Will share a release build on priority for this issue.
Will update once build is available.

The fix on displaying the error screen while switching to a different AI Provider is fixed and pushed to production. Please wait  for application update option. Once done the issue should be fixed @ranjancse26 .

Closing this issue as the error is fixed now.


Inserting selected rows from grid to Sql

Could someone kindly suggest some Serenity reading or documentation regarding sql inserting of selected rows upon button click (yellow). I am using GridRowSelectionMixin to enable multiple row selection.
Thank you in advance.

The grid rows are pulled from SQL...what are you trying to do?

My grid is pulled from a sql view which is combining 2 tables.
I am trying to take the values from selected rows in this grid and insert those rows to to a third table.
That table is ready for the new data but I am a bit lost on how to execute the insertion.

I probably should put the code inside the "else" portion of this code in xxxGrid.cs:
`        protected override List GetButtons()
{
var buttons = base.GetButtons();
        buttons.Add(new ToolButton
        {
            CssClass = "tag-button",
            Title = "Færa í tollað",
            OnClick = delegate
            {
                var selectedIDs = rowSelection.GetSelectedKeys();
                if (selectedIDs.Count == 0)
                    Q.NotifyWarning("Veldu raðir með því að haka í reitina");
                else

                    Q.NotifySuccess("Þú hefur tollad " + selectedIDs.Count +
                        " línu(r) með ID: " + string.Join(", ", selectedIDs));
            }
        });

        buttons.RemoveAt(0);
        return buttons;`

I just need a little more know-how to implement this in C# and Serenity as I am more used to small vb programs!! :/

check the clone example in the northwind i think this is a start

Thanks for the reply.
Do you mean the product section of Northwind or something else by "the clone"?

It's part of the Basic Samples module. Serene.Script\BasicSamples\CloneableEntityDialog.cs

Ok, found it. I will give it a go.
Thank you for your help :) :+1:

No worries, glad that I could help!

Also have a look at bulk action sample.

Hi again, thank you all for your suggestion. I am somewhat closer to the end line, but I am missing something. I would be greatly thankful if someone could assist me on how to handle the SQL insertion trigger.
Please see below where my buttonClick code is, where I want to take selected rows and insert to SQL table. This is from my xxxGrid.cs on the Script side where I should put the trigger in the else portion.
`        protected override List<ToolButton> GetButtons()
        {
            var buttons = base.GetButtons();

            buttons.Add(new ToolButton
            {
                CssClass = "tag-button",
                Title = "Færa í tollað",
                OnClick = delegate
                {
                    var selectedIDs = rowSelection.GetSelectedKeys();
                    if (selectedIDs.Count == 0)
                        Q.NotifyWarning("Veldu raðir með því að haka í reitina");
                    else

                        Q.NotifySuccess("Þú hefur tollad " + selectedIDs.Count +
                            " línu(r) með ID: " + string.Join(", ", selectedIDs));
                }
            });
            return buttons;`

I have some questions:

Should I put the insertion trigger in the script project? I have noticed that script doesn't include any reference regarding sqlclient. The web project has those references but I assume that I should put the necessary code into the else portion of xxxGrid.cs which is on the script side. Where can I find information on how the Linq code should be?
I would be on familiar grounds if I can use the System.Data.SqlClient reference on the web side. Can I perhaps use a controller in the web project to handle the insertion, is it safe?

I have taken a look at the cloneable sample and also the bulkactionorder sample. I have copied and adapted the code from the BasicSampleXXXService.cs classes into my service class. It looks to me though as it is missing triggers for SQL insertion, am I right? See below:
`    [Imported, PreserveMemberCase]
    public partial class LagerService
    {
        [InlineConstant] public const string BaseUrl = "Inventory/Lager";

        [Imported, Serializable, PreserveMemberCase]
        public partial class LagerTollunRequest : ServiceRequest
        {
            public List<Int32> selectedIDs { get; set; }
        }

        [InlineCode("Q.serviceRequest('Inventory/Lager/Create', {request}, {onSuccess}, {options})")]
        public static jQueryXmlHttpRequest Create(SaveRequest<TolladRow> request, Action<SaveResponse> onSuccess, ServiceCallOptions options = null)
        {
            return null;
        }

        [InlineCode("Q.serviceRequest('Inventory/Lager/Update', {request}, {onSuccess}, {options})")]
        public static jQueryXmlHttpRequest Update(SaveRequest<TolladRow> request, Action<SaveResponse> onSuccess, ServiceCallOptions options = null)
        {
            return null;
        }

        [InlineCode("Q.serviceRequest('Inventory/Lager/Delete', {request}, {onSuccess}, {options})")]
        public static jQueryXmlHttpRequest Delete(DeleteRequest request, Action<DeleteResponse> onSuccess, ServiceCallOptions options = null)
        {
            return null;
        }

        [InlineCode("Q.serviceRequest('Inventory/Lager/Retrieve', {request}, {onSuccess}, {options})")]
        public static jQueryXmlHttpRequest Retrieve(RetrieveRequest request, Action<RetrieveResponse<TolladRow>> onSuccess, ServiceCallOptions options = null)
        {
            return null;
        }

        [InlineCode("Q.serviceRequest('Inventory/Lager/List', {request}, {onSuccess}, {options})")]
        public static jQueryXmlHttpRequest List(ListRequest request, Action<ListResponse<TolladRow>> onSuccess, ServiceCallOptions options = null)
        {
            return null;
        }

        [Imported, PreserveMemberCase]
        public static class Methods
        {
            [InlineConstant] public const string Create = "Inventory/Lager/Create";
            [InlineConstant] public const string Update = "Inventory/Lager/Update";
            [InlineConstant] public const string Delete = "Inventory/Lager/Delete";
            [InlineConstant] public const string Retrieve = "Inventory/Lager/Retrieve";
            [InlineConstant] public const string List = "Inventory/Lager/List";
            [InlineConstant] public const string Lagertollun = "Inventory/Lager/Lagertollun";
        }

        [InlineCode("Q.serviceRequest('Inventory/Lager/Lagertollun', {request}, {onSuccess}, {options})")]
        public static jQueryXmlHttpRequest LagerTollunBulkAction(LagerTollunRequest request, Action<ServiceResponse> onSuccess, ServiceCallOptions options = null)
        {
            return null;
        }
    }`

Maybe I am on the totally wrong track. But I hope this is a easy question for someone and I would really appreciate some direction.
Thank you in advance.


⚠ Jenkins Plugins Artifactory has degraded performance

In ff8e3d7, Jenkins Plugins Artifactory (https://get.jenkins.io/plugins/artifactory/) experienced degraded performance:

HTTP code: 200
Response time: 2405 ms

Resolved: Jenkins Plugins Artifactory performance has improved in cc78e41 after 16 minutes.


php: Update php to 8.3.0

Build php and php-debug

:beers: @BrewTestBot has triggered a merge.

I'm the only one where the etc directory was not created for 8.3. php -i outputs:
Configuration File (php.ini) Path => /opt/homebrew/etc/php/8.3
Scan this dir for additional .ini files => /opt/homebrew/etc/php/8.3/conf.d

But I only have directories for my previous php versions in my /opt/homebrew/etc/php directory. My upgrade process was:
brew uninstall --ignore-dependencies php
brew install<EMAIL_ADDRESS>
brew upgrade # because had issues with libsodium

brew install<EMAIL_ADDRESS>

It ends so up that there is not even opcache.


🔬 Metrics Prototype: Conditions everywhere edition

A potential way to integrate metrics into the Instrumentation:

If the OpenTelemetry::Metrics API is installed, go ahead and make things in base
Check for a configuration option, :send_metrics to be true before generating telemetry
Still very messy, do we like this general approach?

I think this is too clunky. I'm going to put forth #1129 for an in-depth review instead.


Add new Lint/RedundantRegexpQuantifiers cop

This cop checks for redundant quantifiers (a.k.a. repeat operators) in regular expression literals.
Such redundant quantifiers also trigger a warning in the Ruby parser / Onigmo:
ruby -e '/(?:a+)+/'
-e:1: warning: regular expression has redundant nested repeat operator '+': /(?:a+)+/

Before submitting the PR make sure the following are checked:

[x] The PR relates to only one subject with a clear title and description in grammatically correct, complete sentences.
[x] Wrote good commit messages.
[x] Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[x] Added tests.
[x] Ran bundle exec rake default. It executes all tests and runs RuboCop on its own code.
[x] Added an entry (file) to the changelog folder named {change_type}_{change_description}.md if the new code introduces user-observable changes. See changelog entry format for details.

@koic @bbatsov would you like to merge this? should we suppress the warnings caused by "bad code examples" mentioned in the comment above or just live with them?

would you like to merge this?

Yeah, sorry about the slow turnaround here!

should we suppress the warnings caused by "bad code examples" mentioned in the comment above or just live with them?

Ideally yes, if we figure out how to do it. :-) It's not a stopper to merge this for now, though - I'm preparing for a new release soon.

Such redundant quantifiers also trigger a warning in the Ruby parser / Onigmo:

You might also mention this in the cop description. More rationale is always better. (a follow-up PR is fine)


Unsupported metric 'h' for two numpy matrices of type np.uint8?

Hi There,
I was just checking out version 5.1.0 and I noticed that this code no longer works:
np.random.seed(42)

a_mat = np.ascontiguousarray(np.random.randint(0, 255, (100, 192), dtype=np.uint8))
b_mat = np.ascontiguousarray(np.random.randint(0, 255, (100, 192), dtype=np.uint8))

dist = np.array(simsimd.cdist(a_mat, b_mat, 'hamming'), dtype=np.uint8)

The error message that gets returned when I run the code is the following:
ValueError: Unsupported metric 'h' and datatype combination ('B'/'B' and 'B'/'B')

When using versions 5.0.1, 5.0.0, and 4.4.0 the code works exactly as I would expect.
Ideally given the changes in 5.1.0 I was hoping to be able to rewrite the code like so:
np.random.seed(42)

a_mat = np.ascontiguousarray(np.random.randint(0, 255, (100, 192), dtype=np.uint8))
b_mat = np.ascontiguousarray(np.random.randint(0, 255, (100, 192), dtype=np.uint8))

dist = np.array(simsimd.cdist(a_mat, b_mat, 'hamming', dtype="u8"))

To reduce the memory footprint when computing distances on very large matrices.
Am I missing something? My setup is Windows 11 + 13th Gen Intel(R) Core(TM) i7-13700HX.
Many Thanks,
Stuart Reid

Nice catch! There is a problem with how NumPy propagates binary data to CPython. Does it work if you set dtype=“b8”?
PS: Sorry for inconvenience 🤗

No worries, SimSIMD is epic. The following code:
np.random.seed(42)

a_mat = np.ascontiguousarray(np.random.randint(0, 255, (100, 192), dtype=np.uint8))
b_mat = np.ascontiguousarray(np.random.randint(0, 255, (100, 192), dtype=np.uint8))
dist = np.array(simsimd.cdist(a_mat, b_mat, metric="hamming", dtype="b8"))

Produces the same error using SimSIMD==5.1.0:
ValueError: Unsupported metric 'h' and datatype combination ('B'/'B' and 'B'/'B')

I've tried a bunch of variations but I'm not winning.

Can you please add this test at the end of test.py and I will submit a patch later today. Thanks!

Thanks @stuartatnosible! I've added tests covering the described use-case and added documentation to README.md. Please let me know if you notice any other issues 🤗


Cannot connect cause of proxy authentication

Describe the bug
I just installed new version of lens and it tries to use proxy conection for no reason.
I have never use proxy and i dont have set up them.
To Reproduce

I decide to install new version
after installed i cant connect cause of luck conection to proxy

Expected behavior
I dont wanna my lens use proxy conection if i dont have any proxy conections.
Screenshots

Environment (please complete the following information):

Lens Setup: 2024.9.300059-latest.exe
OS: Windows 10 22H2 19045.2965

Kubeconfig:
apiVersion: v1
clusters:
- cluster:
    certificate-authority-data: DATA+OMITTED
    server: ...
  name: ...
- cluster:
    certificate-authority-data: DATA+OMITTED
    server: ...
  name: ...
contexts:
- context:
    cluster: ...
    user: ...
  name: yc-general
- context:
    cluster: ...
    user: ...
  name: yc-kube-massive
current-context: yc-general
kind: Config
preferences: {}
users:
- name: ...
  user:
    exec:
      apiVersion: ...
      args:
      - k8s
      - create-token
      - --profile=...
      command: C:\Users\...\yandex-cloud\bin\yc.exe
      env: null
      interactiveMode: IfAvailable
      provideClusterInfo: false
- name: ...
  user:
    exec:
      apiVersion: ...
      args:
      - k8s
      - create-token
      - --profile=easycommerse
      command: ...
      env: null
      interactiveMode: IfAvailable
      provideClusterInfo: false

Aditional content:

k9s works fine at the same setup.
Close to https://github.com/lensapp/lens/issues/2189

Thank you @K4leri for your report. I addressed your issue to the corresponding team.

Hi @K4leri , the aforementioned proxy is internal to Lens and is there for legacy reasons. In the near future, we are planning to offer an option to opt-out of it, but it has been there for a long time (years I would say)... Can you connect to this cluster through plain kubectl and run for example kubectl get --raw /api?

@dechegaray yes i can. I found exectly what wrong. I installed exe to all users on my windows OS. Then i start Lens.exe whithout admin rights and in such case it gives this error to me.
When i start my Lens.exe with admin rights all is working

@dechegaray yes i can. But the same thing as i am describing in this message:
I found exectly whats wrong. I installed exe to all users on my windows OS. Then i start Lens.exe whithout admin rights and in such case it gives this error to me.
When i start my Lens.exe with admin rights => all is working. So for some reason it should work under admin rights. I dont know if its an error. If not, then the topic should be closed

Admin rights should not be required, but if users are under strict security rules, it must be checked that they can execute .exe files from C:\Users\<username>\AppData\. We had another case in which the user had to move Lens' folders to a different location because of its strict setup.
May I suggest the following:

Uninstall Lens.
Remove anything in C:\Users<username>\AppData\Local\Programs\Lens
Install again the latest release.

We had few cases (Windows only) in which remaining files in that location where causing some permission related issues


docs: add YARD annotations

Pull Request Test Coverage Report for Build 67

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

Totals

Change from base Build 65:
0.0%

Covered Lines:
201

Relevant Lines:
203

💛  - Coveralls


[typescript-urql-graphcache] add support for typesPrefix and typesSuffix options

Description
This PR gets the typescript-urql-graphcache plugin to respect the typesPrefix and typesSuffix options when generating type names.
Related #6544.
Type of change
Please delete options that are not relevant.

[x] Bug fix (non-breaking change which fixes an issue)

Screenshots/Sandbox (if appropriate/relevant):
None.
How Has This Been Tested?

[x] Added the relevant tests and made sure they pass
[x] Ran my modified version of the plugin against my in-development app and verified that the type names were correctly

Test Environment:

OS: macOS 11.5.1
@graphql-codegen/typescript-urql-graphcache: 2.1.1
NodeJS: v16.6.2

Checklist:

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

Further comments
None.

@shiwano Thank you 🥳


[Request] Possibility to add Expandable search

Is it possible to support 'Expandable Search'
Cfr. https://www.google.com/design/spec/patterns/search.html#search-in-app-search - Expandable Search section?
Wim

Hi @wervoets,
No, the library does not support expandable search at this point in time.
Regards,


[missing_small_caps_glyphs] is Just Wrong

Fonts without small caps pass this check. Indeed, every font in the GF library passes this check.
Here's why:
https://github.com/fonttools/fontbakery/blob/fb1c3dee36073a6e17e0f13fce67250becb16264/Lib/fontbakery/checks/missing_small_caps_glyphs.py#L30-L36
For each lookup which corresponds to an smcp or c2sc feature, we gather a list of all the substituted glyphs (stuff on the right hand side of a sub ... by statement, e.g. a.sc).
https://github.com/fonttools/fontbakery/blob/fb1c3dee36073a6e17e0f13fce67250becb16264/Lib/fontbakery/checks/missing_small_caps_glyphs.py#L37
And then from that set of substituted glyphs, we remove every glyph in the font.
The result is that, unsurprisingly, no glyphs are missing.

PR #4721


Beckn UAI Win room 21st Nov

Description
Participate in the Beckn UAI Win Room session scheduled for 21st November. The session will focus on aligning stakeholders, addressing project updates, and discussing key challenges and next steps related to Beckn UAI.
Goals

[x] Attend the Beckn UAI Win Room session on 21st November.
[x] Present updates and progress on Beckn UAI-related tasks.

Expected Outcome
A productive Beckn UAI Win Room session with aligned stakeholders, resolved challenges, and a clear roadmap for next steps.
Acceptance Criteria

[x] Session is attended and contributions are made to discussions and problem-solving.
[x] Key updates and progress are presented effectively.
[x] Challenges are addressed and action items are defined.

Mockups / Wireframes
NA
Product Name
Beckn UAI
Domain
Collaboration and Planning
Tech Skills Needed

Communication

Complexity
Medium
Category
Meeting
Sub Category
Planning and Alignment

Participated actively in the session, contributing to discussions and problem-solving.
Presented key updates and progress effectively, ensuring clarity and alignment with the objectives.
Addressed challenges identified during the session and defined clear action items to drive resolution.


Missing or empty columns on taxa bar plots

A strange output for taxa bar plots occurred when I was running some bar plots. The column was simply empty as you can see in this picture:

The relevant file will be sent via slack.

Hey @shiffer1 - take a look at the FeatureTable you provided to this visualizer --- sample AF10-16S actually has zero features in it!

Maybe in the future we can flash some kind of warning in this viz, but this seems to be a bigger problem (what does it even mean to still have this NULL sample in your FeatureTable?), so for now we feel like not displaying anything in the barplot at this position is a pretty reasonable approach. Thanks for filing!

Wow that is really weird! It is a good sample (as in not a blank) and has DNA concentrations comparable to others. I feel like something has definitely gone wrong in the sequencing here.
We definitely filter this out in selecting sampling depth so this wouldn't be included in further analysis after filtering the feature table. Perhaps @gregcaporaso would be interested in chiming in here.
When we are looking at these taxa plots we could be looking at unfiltered data. This is kind of a rare case here as this data is not filtered for sampling depth.
I think a warning might be interesting for having 0 or even very low sampling depth values in your plots.
In any case, I found another one for you:

I will send along the qzv via slack.

@shiffer1 - please check the table summary for the second screenshot you have provided --- do the samples with no bars also have no features associated with them? If they do, please provide us with the feature table, taxonomy, and metadata used to prepare the viz shown in the second screenshot.

I think the current behavior is reasonable since the sample contains no data (it has a sequence count of zero). Showing a blank bar in this case seems reasonable.
The bar plots don't perform rarefaction because the sequence counts are normalized to relative abundances. If you want these "empty" samples (or low abundance samples) removed you can use qiime feature-table filter-samples (it'll remove "empty" samples, along with whatever other filtering criteria you specify).
I'll let @shiffer1 follow up on @thermokarst's question about the other "empty" samples in the above screenshot to make sure it's operating as expected. @shiffer1 let me know if you're unsure what we're asking of you! :shipit:

I put together a brief MWE example of what I think is the cause of the problem showing up in @shiffer1's second example.
import biom
import pandas as pd
import qiime2
from q2_taxa._util import _extract_to_level

# Note, this taxonomy only has feat1 and feat2, no feat3
taxonomy_series = pd.Series(['a; b; c', 'a; b; d'],
                            index=['feat1', 'feat2'], name='Taxon')
taxonomy_series.index.name = 'Feature ID'
taxonomy = qiime2.Artifact.import_data("FeatureData[Taxonomy]", taxonomy_series)

table_biom = biom.Table(np.array([[1, 1, 0], [1, 0, 0], [0, 0, 10]]),
                        ['feat1', 'feat2', 'feat3'],
                        ['all_features_in_taxa', 'some_features_in_taxa', 'no_features_in_taxa'])
table = qiime2.Artifact.import_data("FeatureTable[Frequency]", table_biom)

collapsed_table = taxa.methods.collapse(table, taxonomy, 3)

print(collapsed_table.collapsed_table.view(pd.DataFrame))

#                      feat1  feat2. feat3????
                       a;b;c  a;b;d
all_features_in_taxa     1.0    1.0
some_features_in_taxa    1.0    0.0
no_features_in_taxa      0.0    0.0

I think that his issue is not related to barplot, but rather collapse (and its underlying util functions). When collapsing (which barplot calls behind the scenes), if a feature in the feature table is not present in the reference taxonomy, it is dropped in the collapsed table. This is basically a variation of his initial example, where there was an empty sample in the feature table.

I wonder if taxa collapse should error in the case where a feature table's ID is missing from the taxonomy -- this seems like a similar case to how we're starting to handle metadata (since taxonomy is a type of feature metadata). If you guys think this change makes sense I'll create another issue for collapse.
I think the behavior in barplot makes sense as-is (for @shiffer1's original case, and the one @thermokarst showed above).

Also, if you needed to restrict your table to the IDs that are found in the taxonomy, then there's always feature-table filter-*. So I don't see a downside to forcing the taxonomy to be a superset of the table.

Thanks all, I created #95 to change collapse behavior. Closing this issue in favor of that.

I liked the solution by the way! I think it is important for people that
perhaps don't know that they may be dealing with data that is not what is
expected and or combinations of data from the same run.
Thanks,
Arron
On Wed, Sep 27, 2017 at 10:30 AM, Jai Ram Rideout<EMAIL_ADDRESS>wrote:

Thanks all, I created #95 https://github.com/qiime2/q2-taxa/issues/95
to change collapse behavior. Closing this issue in favor of that.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/qiime2/q2-taxa/issues/94#issuecomment-332596303, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AJuoUgjGeuex1nhltLVKeDtbbnuAk1loks5smoYmgaJpZM4PejOm
.


Allow platform properties in worker configs to be templated

At the moment we need to do some fairly cloud-unfriendly gymnastics to dynamically set the value field in worker configs. For instance:
https://github.com/TraceMachina/nativelink/blob/00ff4a088365e616e6094c85d99d999a039338b8/deployment-examples/kubernetes/02_application.sh#L7-L15
This should instead be configurable via environment variables so that we can pass these values via yaml files directly.

Fixed by the new devcluster setup and Tekton.


Make player bounce off of Goomba when killing it

Please inform the person assigned to Goomba of any changes you make to any of the goomba files or player files. This is important because if not aware it may break their code

[ ] Player bounces off of Goomba when killing it


About the version format of tag

hi, I see that you have two tags, 0.1.0 and 0.2.0 respectively, but this seems to be inconsistent with the version number specification of go modules. I used go get<EMAIL_ADDRESS>pointing to v0.0.0-20220617064735-3fb829d9078f. The implication is whether the prefix v is missing, such as v0.2.0

@lynin you are right, I didnt know about the v factor https://github.com/golang/go/issues/34512
I have released a new version with the proper semantics https://github.com/alphadose/itogami/releases/tag/v0.3.0
pls check if this one is working for you or not
also thanks a lot for pointing out this major issue, I got to learn something interesting today


Don't Merge! PR real time test for source branch full build

Docs Build status updates of commit 4825ece:
:clock10: Pending: waiting for processors (34 builds ahead of you)
:warning: Docs Build is busy, currently there are 34 builds ahead of this one, for more information you can view the Build queue graph on the Docs Portal.

Docs Build status updates of commit 4825ece:
:clock10: Full build: the average full build time is 7 min(s) 37 sec(s), this is based on the last 30 build(s) for this repository.

Docs Build status updates of commit 4825ece:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docfx.json
:white_check_mark:Succeeded

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


Update AMPI manual section on global variable privatization

Original issue: https://charm.cs.illinois.edu/redmine/issues/1883

Section 3.4 has rusted a bit with the recent developments in -tlsglobals and-swapglobals. We should advise users that -swapglobals is not very portable, doesn't work in SMP builds of AMPI, and has runtime overhead proportional to the number of global variables. I think we should move that subsection to the end of the section too.
For TLS globals we need to update the documentation to say that shared objects are now support, but that it may only work with GCC. Also maybe provide a Fortran example for tlsglobals.

Original date: 2018-05-09 20:29:36

~~https://charm.cs.illinois.edu/gerrit/#/c/charm/+/4157/~~ https://github.com/UIUC-PPL/charm/commit/b6cc2bc1400051e478ea004df3f0469c2be76933


[FLINK-2991] Add Folding State and use in WindowOperator

This enables efficient incremental aggregation of fold window.
This also adds:

WindowedStream.apply(initVal, foldFunction, windowFunction)
AllWindowedStream.apply(initVal, foldFunction, windowFunction)

You are adding new .apply() variants to the API which are not documented with the PR.

I'll add doc and also fix some other stuff in the doc that wasn't updated.

I addressed the comments.

Doe it always require a "WindowFunction"? Can one also use only a FoldFunction?

You can also do just a "fold". The window function is for when you need the meta information about the window because the fold function does not get that.


[Feature] Publish dotnet tool

Hello!
It'd be really neat if we could install this globally as a dotnet tool! https://learn.microsoft.com/en-us/dotnet/core/tools/global-tools
Then it could be installed once and used in any project instead of either 1) modifying your PATH, 2) copy/pasting this everywhere you need it, or 3) keeping it only in my download directory.

Unless I'm mistaken, a .NET tool needs to be distributed as a nuget package. Since this is written in Rust, that won't be possible. You're right that this means you'll have to do either of the three options you've mentioned; I suppose my expectation is that you'll only use this binary very infrequently: once you run it, you will never need it again for that repo.
Based on that I'll close this as unable to take action, but don't let that stop you from adding to the conversation if there is an aspect I might have missed


Replacement suggestions?

Thanks @ashchan for creating this great tool so many years ago! 🎆
I really like how simple & unobtrusive it was.
Sadly, I'm now getting the auth errors, so I need to find a replacement.
Is anyone aware of anything comparable to Mail Notifr?
The closest I found is Mail+ from Rocky Sand Studios, but it seems to be a lot heavier than Mail Notifr with a bunch of extra features I don't want.

Thank you for the kind words. It's sad but the process to satisfy Google requirements is just too much for me as an individual developer.
As an alternative method, I just keep the Mail.app open 24X7. I do not worry about the ZERO or other unread numbers anymore. Don't have many messages anyway.
Thank you!

I miss the app so much... I've been using it for years, a part of my daily workflow has been taken away from me now that the app does not work anymore. Many thanks to @ashchan for this amazing app, would there be a way we could help you get through the Google requirements? Maybe if we join forces between a few fans, we could help you out? ;-)
Talking about alternatives: I've been testing Mail+ and Mia for Gmail. Mia is closest to Mail Notfr. However, all these apps require that you have to give full access to your gmail account (read/write permissions). I believe these permissions are way wider than the ones Mail Notfr needed (is that correct @ashchan?), so I'm not sure yet that I feel comfortable giving these permissions...

@stysedo Google actually told me individual open source can have zero fee solutions but the toolkits they suggest don’t support Swift project at all.


Empty view is never matched

Describe the bug
If you have a view with only a name but no internal zones/data, it will be silently ignored and any access-control-view matching that view name appears to be dropped. An empty view can be useful to match a more specific IP block that otherwise matches a different view or can come about from automated tools. This silent ignoring leads to the wrong view being matched for an IP block and therefore unexpected exposure of data within that view.
To reproduce
Steps to reproduce the behavior:

Create some sample local-zone/local-data at the server top level (used to demonstrate bug)
Create an empty named view with view-first: no. Queries hitting this should not return the data set in step 1 above
Add an access-control-view that selects a relevant IP range and points to the empty view created in step 2
Query from specified IP range for the zone data setup in step 1, where I'd expect to match the empty view and therefore not to see the data from step 1 (due to view-first: no).
Now add a single local-zone entry to the otherwise empty view (with any content, it does not need to be something you will query for, only its presence matters)
Repeat the query in step 4 and see that the data from step 1 is now not returned, as the view is correctly matching

Expected behavior
An empty view should still match an ACL and not be silently ignored. The behaviour should match what is observed in step 6 above, where a dummy zone has been added to force the view to exist. From a quick browse of code, this could be due to:
https://github.com/NLnetLabs/unbound/blob/86fe9cbce533549e4310bd3fc7c1b89df70a33d4/services/view.c#L161
System:

Unbound version: 1.17.1
OS: Debian 12
unbound -V output:

Version 1.17.1

Configure line: --build=x86_64-linux-gnu --prefix=/usr --includedir=${prefix}/include --mandir=${prefix}/share/man --infodir=${prefix}/share/info --sysconfdir=/etc --localstatedir=/var --disable-option-checking --disable-silent-rules --libdir=${prefix}/lib/x86_64-linux-gnu --runstatedir=/run --disable-maintainer-mode --disable-dependency-tracking --with-pythonmodule --with-pyunbound --enable-subnet --enable-dnstap --enable-systemd --with-libnghttp2 --with-chroot-dir= --with-dnstap-socket-path=/run/dnstap.sock --disable-rpath --with-pidfile=/run/unbound.pid --with-libevent --enable-tfo-client --with-rootkey-file=/usr/share/dns/root.key --enable-tfo-server
Linked libs: libevent 2.1.12-stable (it uses epoll), OpenSSL 3.0.11 19 Sep 2023
Linked modules: dns64 python subnetcache respip validator iterator
TCP Fastopen feature available

This is explicit behavior in the code https://github.com/NLnetLabs/unbound/blob/86fe9cbce533549e4310bd3fc7c1b89df70a33d4/services/localzone.c#L1873 and the man page https://unbound.docs.nlnetlabs.nl/en/latest/manpages/unbound.conf.html#unbound-conf-view-local-zone.

Ok, sorry I missed that. I would suggest it’s somewhat surprising behaviour, but can’t argue with the docs 😂


Add application/zstd content-type and zstd content-encoding

Motivation:
ZSTD has a wide range of uses on the Internet, so should consider adding application/zstd HTTP media-type and zstd content-encoding, see  https://tools.ietf.org/html/rfc8478
Modification:
Add application/zstd HTTP media-type and zstd content-encoding
Result:
netty provides application/zstd HTTP media-type and zstd content-encoding as http headers

Although ZstdDecoder has not been added, considering users who use zstd extensively, should consider adding application/zstd HTTP media-type and zstd content-encoding to users

Thanks a lot!

@normanmaurer It is my pleasure


test go-coap with shared messageID

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 74.03%. Comparing base (df5db5c) to head (3568ab7).

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #509      +/-   ##
==========================================
+ Coverage   74.01%   74.03%   +0.01%
==========================================
  Files         124      124
  Lines        8741     8741
==========================================
+ Hits         6470     6471       +1
  Misses       1680     1680
+ Partials      591      590       -1

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


python3Packages.*: restore other missing packages

Motivation for this change
should fix ofborg, was able to do ofborg eval locally
accidental victims of: #96850
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

looks like this fixed it #96851


added cipher suite details

added easy getters for key exchange, authorizatoin, algorythm and mac method.

This has been hanging around long enough, closing.

@spoofzu do you want the changes re-created? I can submit a PR that addresses the comment...


request.files is empty after POSTing a file

Hey guys, I don't know if this is the right place to make this bug report, but I could sure use some help, I have been banging my head against a wall for a bit. See this repo where I'm reproducing my error: https://github.com/bepetersn/special-repo.
Using Flask, I'm seeing weird behavior around its request object. After uploading a file with the requests library, e.g. requests.post(uri, files=<my_files>), by the time the request propagated to my view function, request.files was empty. Oddly, I observed that the contents of the file itself was available under request.form[None].
After quite a lot of debugging, I saw that Werkzeug received roughly the following as the form/multipart-encoded data, and proceeded to try to parse it:

Content-Disposition: form-data; name*=utf-8''Spirit%20Airlines%20-%20cheap%20tickets%2C%0A  %20cheap%20flights%2C%20discount%20airfare%2C%20cheap%20hotels  %2C%20cheap%20car%20rentals%2C%20cheap%20travel.pdf; filename*=utf-  8''Spirit%20Airlines%20-%20cheap%20tickets%2C%0A%20cheap%20flights  %2C%20discount%20airfare%2C%20cheap%20hotels%2C%20cheap%20car%20rentals  %2C%20cheap%20travel.pdf\r\n

Do you notice the little "*" characters just after the "name" and "filename" header definitions? From my sense of things, Werkzeug looks for exactly "name" and "filename", doesn't know how to parse these in their place, fails to find an attachment name, and thus doesn't create a werkzeug.datastructures.FileStorage. For this reason, request.files is empty.
What does anyone think? Is requests responsible for adding these extra characters in?

I see the line response here: https://github.com/kennethreitz/requests/blob/35d083e1665beff39aabe47a79cd1f867b897b0c/requests/packages/urllib3/fields.py#L45. However, it's pretty clear that it's intentional. I'm curious if anyone can point me to what purpose it has (maybe something to do with RFC 2231?), -- and whether it's really correct, if I'm somehow giving requests bad data, or even if werkzeug isn't handling this stuff correctly..

This problem is almost certainly because you're passing a unicode string into requests 'files' parameter. Make sure your strings are all bytestrings first. =)

I will try that. :)

Well, I don't fully get the logic of it, but what I actually needed to do was get rid of the newline in my the filename I was passing to the files parameter of requests.post.

Ah, yes, that'll do it to.
The problem here is that the multipart/form-encoded spec doesn't ordinarily allow newlines in the form fields, so you need to do some awkward work to get around them. That involves the encoding we've used, which is sadly relatively poorly supported by many servers. =(

Thanks for your help, Corey.
On Mar 18, 2015 4:54 PM, "Cory Benfield"<EMAIL_ADDRESS>wrote:

Closed #2505 https://github.com/kennethreitz/requests/issues/2505.
—
Reply to this email directly or view it on GitHub
https://github.com/kennethreitz/requests/issues/2505#event-258690327.


Upload all past releases to /Dist folder

Fixes #287

:shipit:
My only question is do we have any logs or telemetry about which ones are being requested?  It may be that we really only needed a few of the old versions but there should be no problem with having them all, more or a curiosity about what is still being requested.


netlifyDeploy: wrap secretsMap

Without this, new secretsMap(s) cannot be introduced to the function call

I guess the same can happen with inputs.
bors r+


Deleting text from abstract section in ezEML does not delete text from the abstract element in final EML file

Example ezEML package download
Attempting to revise package edi.134.6. AFAIK, the ezEML package was saved locally on user's computer (not Fetched from EDI).
I went in and tested deleting the new text from the Abstract section in ezEML. There was no text in the ezEML Abstract section, but when I downloaded the EML file, there was text in the abstract element (the old text).
Message from user:

For the revision, I altered both the abstract and the methods.  When using ezEML, I deleted the old abstract/methods and copy/pasted in the new one from a word doc.  However, in the test data package, the new abstract was inserted before the old abstract, and there should be two paragraphs.  For methods, only the new methods text shows in the test data package, but it should be in 4 paragraphs.
I actually may have copied from a text file (not a word document) for either the abstract or the methods.
Please let me know if there’s a protocol I should follow when making changes that involve copy/paste.

This same problem occurs if the package (edi.134) is fetched from EDI


Make it work with react router v6

Hi, would be nice with a version for v6.
I made a fork but backwards compatibility is lost and at the moment custom link is gone.

Added support for custom link as well but with breaking changes, GenericHashLink.
https://github.com/XzaR90/react-router-hash-link

Does it need to be bumped all the way to React v17? RRDv6 still supports any version of React compatible with hooks, i.e. >=16.8.
https://github.com/remix-run/react-router/blob/main/packages/react-router-dom/package.json#L21

Oh, did not check. Well my fork is using typescript and I don,t know if you still want keep plain js. If you have any plan to migrate you can base it on my code if you find it okey and change the dependency as well.

Bumping this as an issue. Doing a bit of research on using anchor hash links with react-router-dom v6 and looks like they have no plan to implement this functionality into the library by default so
react-router-hash-link will be relevant and needs to be updated for v6!

It's on my list, but it might be a little while before I get to it.

The definition of NavLink.isActive has been changed in V6, please refer to
https://github.com/remix-run/react-router/issues/7991#issuecomment-916360778

In my fork it can be used like this,
<NavHashLink
  to="/some/path#with-hash-fragment"
  className={(props) => {
    return `${props.isActive ? 'isActive ' : ''}`;
  }}
  style={(props) => {
    return props.isActive ? { fontWeight: 'bold' } : {};
  }}
  // etc...
>Link to Hash Fragment</NavHashLink>

The scroll on this and the forked version does not work across all browsers on mobiles. Works fine on desktops. Any ideas?

Hey all.  Working with React-Router V6, I define my routes like
 <Route path='/account' element={<Account />} />, which I believe is causing a reload every time a hash link is clicked.  #83 mentions that component={Account} would be the way to go, but I do not believe that is possible.
Currently the <HashLink> only works reliably if I set a timeout because the page is reloading even though it doesn't need to, it should be scrolling to the same page without a refresh.
#83

I use react router v6, so works well i  use react router has link

I've built a component that solves this problem passively in the background without having to modify any react-router Links.
GitHub: ScrollToHashElement

thanks for sharing @ncoughlin ! worked like a charm.


Investigate why sphinx 1.7.0 does not like Parameters blocks

We have pinned sphinx to 1.6.4 because 1.7.0 throws a spurious warning:
Warning, treated as error:
/home/travis/build/NSLS-II/bluesky/doc/source/hardware.rst:57:Unexpected section title.
Parameters
----------
make: *** [html] Error 2

This should be reported upstream or fixed if it turns out we are doing something wrong.

sphinx is at 7.2.6 at the moment, the issue sounds out of date.
can we close it @danielballan ?


Support Swift SDKs

This PR adds support for Swift SDKs when using the SwiftPM build system, as defined by SE-0387. These are already supported by SourceKit LSP, so adding support here is primarily a matter of plumbing the swift.swiftSDK parameter along to the swiftPM.swiftSDK field of SourceKit LSP's startup options.
Additionally, with https://github.com/swiftlang/swift-package-manager/pull/6828 it is now possible to select Darwin triples as Swift SDKs on macOS — that is, you can build for iOS with swift build --swift-sdk arm64-apple-ios and so on. This support also applies to SourceKit LSP, and so we can update the "Select Target Platform" action to pick a Swift SDK instead.
Open questions

This PR mostly obsoletes the old swift.SDK parameter, but I haven't removed it yet because 1) Swift SDK support only works with SwiftPM right now and 2) I'm not sure whether removing swift.SDK would break the Windows-specific use cases for that parameter. Removing it would clean up a lot of code though. Would this be reasonable to do?
Support for Darwin SDKs as Swift SDKs is on main and has landed in time for the 6.1 release cut, but doesn't exist in 6.0. Is there a recommended pattern we can use for feature detection to only surface the "Select Target Platform" command if the compiler is new enough?

TODO:

[x] Tests

We need a https://github.com/swiftlang/swift-package-manager/pull/6828 like solution on Windows before we can get rid of swift.SDK…

@kabiroberai can you rebase please? We recently moved to GitHub actions to verify PRs

@award999 done!

Heya @award999, any updates on this? CI is green now ✅

Thanks for the contribution @kabiroberai!
In answer to your questions:

This PR mostly obsoletes the old swift.SDK parameter, but I haven't removed it yet because 1) Swift SDK support only works with SwiftPM right now and 2) I'm not sure whether removing swift.SDK would break the Windows-specific use cases for that parameter. Removing it would clean up a lot of code though. Would this be reasonable to do?
As long as SwiftPM continues to support the --sdk option, the old swift.SDK parameter should stick around.

Support for Darwin SDKs as Swift SDKs is on main and has landed in time for the 6.1 release cut, but doesn't exist in 6.0. Is there a recommended pattern we can use for feature detection to only surface the "Select Target Platform" command if the compiler is new enough?
The extension typically follows the pattern that the command is always enabled. If the command can't run, it presents an errorMessage. This allows presenting to the user the reason why the command couldn't be run (as opposed to it just not being there). For example "You don't have an SDK selected" vs. "This feature is only supported on Swift 6.0 and higher".

An example of this behaviour can be found at https://github.com/swiftlang/vscode-swift/blob/c3e87fde3119be3a0f712096b1a7e133c91da84f/src/commands/createNewProject.ts#L40

@kabiroberai looks like there is a merge conflict to resolve

@award999 fixed!

@matthewbastien when you're back can you drive this code review being the most familiar with toolchain/sdk selection?


function kc_overall has undefined output

Reported by sjoelund.se on 20 Feb 2013 18:24 UTC
function kc_overall has undefined output Real failureStatus. It is defined for no branch and should either be assigned a value (or I suppose be removed from the interface).
ModelicaTest.Fluid.Dissipation.Verifications.HeatTransfer.HelicalPipe.kc_overall:[Modelica 3.2.1/Fluid/Dissipation.mo:1617:7-1666:21:writable] Warning: failureStatus was used before it was defined (given a value).

Migrated-From: https://trac.modelica.org/Modelica/ticket/996

Comment by wischhusen on 21 Feb 2013 06:32 UTC
Replying to [ticket:996 sjoelund.se]:
Right. failureStatus is provided to indicate wether the function is still valid for the current inputs. In this case it is always valid (overall function). Thus I will put (see 08138dfcce064121bb520be7d921ed6177eb1133):
failureStatus := 0;

function kc_overall has undefined output Real failureStatus. It is defined for no branch and should either be assigned a value (or I suppose be removed from the interface).
ModelicaTest.Fluid.Dissipation.Verifications.HeatTransfer.HelicalPipe.kc_overall:[Modelica 3.2.1/Fluid/Dissipation.mo:1617:7-1666:21:writable] Warning: failureStatus was used before it was defined (given a value).

Comment by sjoelund.se on 21 Feb 2013 12:15 UTC
Makes sense. Easy fix.


Release AzConfig Track 2 Library Preview 3 for Python

API Review

[x] Proposal ready
[ ] Design approved

Features

[ ] Integrate authentication from azure-identity
[ ] React to swagger changes from the service team
[ ] React to arch board feedback

Docs

[ ] Samples
[ ] Readme update
[ ] Change Log

Preview 3 released. We will keep track remaining issues in preview 5


IE 11 Broken 5.0+, drop downs broken 4.13+

Hi,
Encountered a problem with px-data-table in IE 11 with any version 5.0.0+. The table will not render, the only error I encounter is a syntax error in aha-table.js from the console.
When downgrading to version 4.0.15 the table would render, but the dropdowns then did not show up, had to downgrade to 4.0.12. Looks like 4.0.13 introduced px-dropdown 3.1.0 from 2.1.0.
We initially upgraded to fix a defect when editing the input field was smaller than the cell size so it would unnecessarily scroll too early. But this upgrade breaks our current usage.
Testing was initially done on a Windows 7 IE11 machine then debugging was done on a VM for IE11 provided by MS.
Any guidance and feed back would be helpful.
5.0.1 with no table render, we use it twice both instances don't show up, console error

4.0.15 No dropdown on new row

4.0.12

Hi,
Thanks for your issue! Sadly, I hate to do it, but I just tested the version on www.predix-ui.com in IE11 and it 'works on my machine'. Confirmed that this is latest 5.x version also...

Just to elaborate; "latest" IE 11 on Windows 10.

I see could be a version mismatch of IE11 from Windows 10 to window 7.
I can’t even load predix-ui.com on the VM. Thanks for the feedback I know IE is a rabbit hole sometimes.

On Nov 15, 2017, at 9:52 AM, Martin Wragg<EMAIL_ADDRESS>wrote:
Just to elaborate; "latest" IE 11 on Windows 10.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

🤔 Weird. Just tried it on a Win 7 machine (a fine Dell vintage....) and all worked fine.

Been playing around with it all day, even took the script off the predix-ui.com site and replaced it in our vulcanized file. Got a undefined babelHelper error when doing this...
So my guess is the way we are vulcanizing is causing an issue. Could be a comment stripping issue, since we do inline CSS and Scripts when we vulcanize. I'll close this and see if I can find any solution and update this issue.

Hmmm... that is strange. I just can't think of anything that would cause that.
Check out the polymer.json below for how we build the site using polymer CLI: https://github.com/predix-ui/predix-ui.github.io/blob/source/polymer.json

(that repo is the source for predix-ui.com BTW if there's anything else that helps in there)


Feature request: Add option to force container re-create rather than restart

If you have a service configured similar to the following:
network_mode: "container:gluetun"
    healthcheck:
      test: "curl -sf https://google.com  || exit 1"
      interval: 1m
      timeout: 10s
      retries: 1
    labels:
      - deunhealth.restart.on.unhealthy=true

And the gluetun container is re-created due to a new version, the using container is marked as "unhealthy" but a container restart won't work because the container ID of the gluetun container is changed.
Consider adding an option "deunhealth.recreate.on.unhealthy=true" to resolve this.

It's a bit more complicated than that, but yes this is in progress :wink: This duplicates #11.
You might want to subscribe/read the bottom comments on #11 feel free to give your input too!


Wiki "Example Applications" page is abused

I don't know why, it looks more accidental than malicious, but the Devise wiki "Example Applications" page is destroyed and recreated every few days with garbage. See the page history here:
https://github.com/plataformatec/devise/wiki/Example-applications/_history
It was a useful page. Are there some measures we can take to preserve it from abuse?

Maybe we should move it to another page and hope this will stop?

Yep. Though curious why it keeps getting bashed.

No idea and apparently I have no way to lock a page. :( Could you please copy the old contents over to a new page? I am closing this, thanks for reporting.

I've restored the original page and updated it. I'll keep an eye on the page and if it gets bashed again I'll rename it and update the links to it on the repo readme and wiki home page. BTW, I've found my Devise example/tutorial/template is very popular (almost 300 watchers and 50 forks on GitHub).

Ah, you are the RailsApps guy! Thanks for the work on those tutorial/example apps, it really helps Devise adoption! Please let me know if we can help somehow.

I'm happy to help because I appreciate your work. Thanks for your kind offer.


Add continue on failure functionality

This should add the ability for stair to say that the build should continue on failure:
https://buildkite.com/docs/pipelines/wait-step#continuing-on-failure

Thanks @emptyflash!


github-actions-runner: allow deregistration of runners when a pod is deleted

No longer necessary. We shift to the operator.


Vault

This is documentation fulfill request to cover Vault basic functionality and workflow
Originally posted by @vladyslavvolkov in https://github.com/bn-digital/home/discussions/25#discussioncomment-1248119

Created Wiki page


Implicit order of choosing parameter names changed in 6.x, but still inconsistent with documentation

From the documentation:

If all names for a parameter contain dashes, the internal name is generated automatically by taking the longest argument and converting all dashes to underscores.

However, in the implementation, when the possible names are sorted, it’s only for the purpose of grouping them “long options first” — meaning “options with two dashes first”, not “longest name first”.
See current master: https://github.com/pallets/click/blob/2e856a5281a29eb4293059b37427643c3d52a19a/click/core.py#L1641
And compare with branch 5.x: https://github.com/pallets/click/blob/7ede2dbe5ca3bd291c7d18f7767f10752496c6ae/click/core.py#L1506
Between version 5.x and 6.x, the implicit (misdocumented) order had changed, which broke my script.
This bug manifests only when multiple long parameter (option) names are specified, i.e.:
@click.option('--shorter', '--longer-option')

…in which case different versions of Click would choose a different name between shorter and longer_option instead of always choosing longer_option as suggested in the documentation.

Duplicate of #1140


Load Packing for Uncached Loads

The UCE has an implicit assumption that all loads are 64-bits. This is not true for 32-bit, 16-bit and 8-bit operations. This is a problem because the cache expects the UCE to replicate the data based on the operation so that it just has to slice the data appropriately to get the output it needs. Therefore, load packing must be done for the uncached load data in the UCE.
Current implementation: https://github.com/black-parrot/black-parrot/blob/d7db362f00f5e2db678d9e43ee1ec76896ce1b2b/bp_me/src/v/cce/bp_uce.sv#L800

By the Bedrock bus definition, the mem_response data should be replicated by request size. So this is expected behavior (replicating the native word size until the fill width)


FIX: prettier formatting

i think the prettier version got updated, everyone need to run npm install again to update to latest version so it does format correctly again

Codecov Report

Merging #1395 (bded6fa) into main (a089188) will decrease coverage by 0.13%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #1395      +/-   ##
==========================================
- Coverage   48.65%   48.53%   -0.13%
==========================================
  Files         294      294
  Lines       16575    16575
==========================================
- Hits         8065     8045      -20
- Misses       8510     8530      +20

Impacted Files
Coverage Δ

lnbits/extensions/lnurlp/migrations.py
81.48% <0.00%> (-14.82%)
:arrow_down:

lnbits/db.py
81.20% <0.00%> (-10.07%)
:arrow_down:

lnbits/commands.py
73.61% <0.00%> (-1.39%)
:arrow_down:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

This is weird, the first commit was already included in https://github.com/lnbits/lnbits/pull/1367
Are you sure you pulled in the latest main and run npm install?

Ahhhh, thats what it was, worked for me


Attribute Error

Environment details
0.1.1
Issue variability
Persistent
Issue

Log
[2019-01-03 17:02:22] Initializing config module
[2019-01-03 17:02:22] Validating config
[2019-01-03 17:02:22] Starting azurlane-auto!
error: protocol fault (couldn't read status): Connection reset by peer
[2019-01-03 17:02:23] Navigating to map.
* daemon not running; starting now at tcp:5037
* daemon started successfully
Traceback (most recent call last):
  File "azurlane-auto.py", line 96, in <module>
    script.run_combat_cycle()
  File "azurlane-auto.py", line 41, in run_combat_cycle
    if self.modules['combat'].combat_logic_wrapper():
  File "C:\Users\Darryl\Downloads\Azur Lane\azurlane-auto-master\modules\combat.py", line 57, in combat_logic_wrapper
    Utils.wait_and_touch('map_{}'.format(self.sortie_map), 5, 0.85)
  File "C:\Users\Darryl\Downloads\Azur Lane\azurlane-auto-master\util\utils.py", line 196, in wait_and_touch
    if cls.find_and_touch(image, similarity):
  File "C:\Users\Darryl\Downloads\Azur Lane\azurlane-auto-master\util\utils.py", line 173, in find_and_touch
    region = cls.find(image, similarity)
  File "C:\Users\Darryl\Downloads\Azur Lane\azurlane-auto-master\util\utils.py", line 94, in find
    width, height = template.shape[::-1]
AttributeError: 'NoneType' object has no attribute 'shape'

Config
[Commissions]
Enabled: False

[Combat]
Enabled: True
Map: 1-1
KillsNeeded: 1
Hard: False
Formation:
AltClearFleet: False
BossFleet: False

[Missions]
Enabled: False

Sorry this is because there isn't an asset for 1-1 yet, been really busy IRL so hopefully in the next week I will update it to have assets for every single map as well as implement the page navigation. The only ones I that are on the repository are 6-1 and 6-2.
Also for now I would use this branch: https://github.com/perryhuynh/azurlane-auto/tree/feature/7-combat-fixes

oh alright thanks 6-1 and 6-2 are fine for now; take your time.

the map currently also has to be already on the screen when you click on the attack button until I implement the page navigation

ya i figured that already set it up and its running smooth so far, thanks for the hard work ^^


Remove siteSpecificLogout config param.

New flows should always be used with site-specific logout.

@marjuszkiewicz I've removed the test 👍
And yes, sessionServiceDomain should be required - I've got a fix for that in separate PR coming soon™


feat: update swagger events for only current instance

doc: add example in configuration -> multipleInstance
doc: add example in eventEmitter -> multipleInstance
It was impossible to create several instances because of the variable
"instance" to instantiate globally and not for the current swagger.
Instance verification condition, blocked the creation of another swagger
What kind of change does this PR introduce? (check at least one)

[ ] Bugfix
[X] Feature
[ ] Test
[X] Docs
[ ] Refactor
[ ] Build-related changes
[ ] Other, please describe:

Description:
The creation of a new instance was blocked.
Ability to create multiple swagger instances with their own swaggerEvents.
I added examples of multiple instance in the configuration and eventEmitter folder

@all-contributors please add @juliendu11 for code

@all-contributors please add @juliendu11 for code


Debug event display is wrong when DebugError occur

DebugEvent display is based on a passed-in fmt string and a list of RawVals.
However, the DebugError always inserts the err Status as the first RawVal (see event.rs ), and it is not obvious to the user. Therefore, the formatting is often out-of-position w.r.t the intended val index. Here is an example (note event 0):

This issue is duplicate of https://github.com/stellar/rs-soroban-env/issues/622 , which has a better example and explanation.
The Status argument is out-of-position. The above message (debug event 0) should instead be something like:
"Failed with status HostFunctionError(InputArgsWrongType): unexpected RawVal U32 for sign, need I32".


It will be great if you can provide a docker file to quickly bootstrap and play

This is in progress.

I have tested the docker images as described in Steve's instructions at https://github.com/ONSdigital/address-index-api under "How do I get a sneak peak of the API?" on my Windows 10 machine and it all worked perfectly. Ready for external uses to try.


Setting device clock forward then back suppresses clicks

On a project I'm working on, if I set the device clock (iPad in this case) forward an hour, let the time change propagate to the js system, then set the clock back to current time and let it propagate, the Code covering
"Prevent phantom clicks on fast double-tap (issue #36)" will preventDefault() all clicks, killing the app navigation (our app is a single page with multiple dynamic 'screens', no page reloads).
Tested and confirmed in @version 0.6.7 as of our current repo, but I notice the code is in fastclick latest pretty much unaltered (@ line 429).  Suggest Math.abs() wrapper around the comparison of event.timeStamp and this.lastClickTime in onTouchStart and onTouchEnd.

Our QA team recently reported a bug in one of our apps:

Buttons are not clickable after changing device's time.

Thought it was a prank at first...
Took a while, but finally tracked it to this code.
Then I saw it is already reported here. :)
CC @jcppman

Thanks @GrumpyBob for the debugging effort (3 years ago lol). Just patched this in our project with your Math.abs suggestion to get the bug fixed - works very well 👍


Logging for exception when getting cache subnet group

Calling this command with --debug should already log this exception from botocore, so this is a bit unnecessary.