[Usage]: Not able to run LLMEngine in two sequential tests using pytest

Your current environment
Collecting environment information...
WARNING 10-01 11:59:43 cuda.py:22] You are using a deprecated `pynvml` package. Please install `nvidia-ml-py` instead. See https://pypi.org/project/pynvml for more information.
PyTorch version: 2.4.0+cu121
Is debug build: False
CUDA used to build PyTorch: 12.1
ROCM used to build PyTorch: N/A

OS: Ubuntu 24.04.1 LTS (x86_64)
GCC version: (Ubuntu 13.2.0-23ubuntu4) 13.2.0
Clang version: Could not collect
CMake version: Could not collect
Libc version: glibc-2.39

Python version: 3.12.3 (main, Sep 11 2024, 14:17:37) [GCC 13.2.0] (64-bit runtime)
Python platform: Linux-6.8.0-41-generic-x86_64-with-glibc2.39
Is CUDA available: True
CUDA runtime version: 12.0.140
CUDA_MODULE_LOADING set to: LAZY
GPU models and configuration:
GPU 0: NVIDIA GeForce RTX 4090
GPU 1: NVIDIA GeForce RTX 4090

Nvidia driver version: 535.183.06
cuDNN version: Could not collect
HIP runtime version: N/A
MIOpen runtime version: N/A
Is XNNPACK available: True

CPU:
Architecture:                         x86_64
CPU op-mode(s):                       32-bit, 64-bit
Address sizes:                        43 bits physical, 48 bits virtual
Byte Order:                           Little Endian
CPU(s):                               16
On-line CPU(s) list:                  0-15
Vendor ID:                            AuthenticAMD
Model name:                           AMD EPYC 7232P 8-Core Processor
CPU family:                           23
Model:                                49
Thread(s) per core:                   2
Core(s) per socket:                   8
Socket(s):                            1
Stepping:                             0
Frequency boost:                      enabled
CPU(s) scaling MHz:                   102%
CPU max MHz:                          3100.0000
CPU min MHz:                          1500.0000
BogoMIPS:                             6200.45
Flags:                                fpu vme de pse tsc msr pae mce cx8 apic sep mtrr pge mca cmov pat pse36 clflush mmx fxsr sse sse2 ht syscall nx mmxext fxsr_opt pdpe1gb rdtscp lm constant_tsc rep_good nopl nonstop_tsc cpuid extd_apicid aperfmperf rapl pni pclmulqdq monitor ssse3 fma cx16 sse4_1 sse4_2 movbe popcnt aes xsave avx f16c rdrand lahf_lm cmp_legacy svm extapic cr8_legacy abm sse4a misalignsse 3dnowprefetch osvw ibs skinit wdt tce topoext perfctr_core perfctr_nb bpext perfctr_llc mwaitx cpb cat_l3 cdp_l3 hw_pstate ssbd mba ibrs ibpb stibp vmmcall fsgsbase bmi1 avx2 smep bmi2 cqm rdt_a rdseed adx smap clflushopt clwb sha_ni xsaveopt xsavec xgetbv1 cqm_llc cqm_occup_llc cqm_mbm_total cqm_mbm_local clzero irperf xsaveerptr rdpru wbnoinvd amd_ppin arat npt lbrv svm_lock nrip_save tsc_scale vmcb_clean flushbyasid decodeassists pausefilter pfthreshold avic v_vmsave_vmload vgif v_spec_ctrl umip rdpid overflow_recov succor smca sev sev_es
Virtualization:                       AMD-V
L1d cache:                            256 KiB (8 instances)
L1i cache:                            256 KiB (8 instances)
L2 cache:                             4 MiB (8 instances)
L3 cache:                             32 MiB (4 instances)
NUMA node(s):                         1
NUMA node0 CPU(s):                    0-15
Vulnerability Gather data sampling:   Not affected
Vulnerability Itlb multihit:          Not affected
Vulnerability L1tf:                   Not affected
Vulnerability Mds:                    Not affected
Vulnerability Meltdown:               Not affected
Vulnerability Mmio stale data:        Not affected
Vulnerability Reg file data sampling: Not affected
Vulnerability Retbleed:               Mitigation; untrained return thunk; SMT enabled with STIBP protection
Vulnerability Spec rstack overflow:   Mitigation; Safe RET
Vulnerability Spec store bypass:      Mitigation; Speculative Store Bypass disabled via prctl
Vulnerability Spectre v1:             Mitigation; usercopy/swapgs barriers and __user pointer sanitization
Vulnerability Spectre v2:             Mitigation; Retpolines; IBPB conditional; STIBP always-on; RSB filling; PBRSB-eIBRS Not affected; BHI Not affected
Vulnerability Srbds:                  Not affected
Vulnerability Tsx async abort:        Not affected

Versions of relevant libraries:
[pip3] mypy==1.11.2
[pip3] mypy-extensions==1.0.0
[pip3] numpy==1.26.4
[pip3] nvidia-cublas-cu12==<IP_ADDRESS>
[pip3] nvidia-cuda-cupti-cu12==12.1.105
[pip3] nvidia-cuda-nvrtc-cu12==12.1.105
[pip3] nvidia-cuda-runtime-cu12==12.1.105
[pip3] nvidia-cudnn-cu12==<IP_ADDRESS>
[pip3] nvidia-cufft-cu12==<IP_ADDRESS>
[pip3] nvidia-curand-cu12==<IP_ADDRESS>
[pip3] nvidia-cusolver-cu12==<IP_ADDRESS>
[pip3] nvidia-cusparse-cu12==<IP_ADDRESS>
[pip3] nvidia-ml-py==12.560.30
[pip3] nvidia-nccl-cu12==2.20.5
[pip3] nvidia-nvjitlink-cu12==12.6.68
[pip3] nvidia-nvtx-cu12==12.1.105
[pip3] pynvml==11.5.3
[pip3] pyzmq==26.2.0
[pip3] torch==2.4.0
[pip3] torchvision==0.19.0
[pip3] transformers==4.44.2
[pip3] triton==3.0.0
[conda] Could not collect
ROCM Version: Could not collect
Neuron SDK Version: N/A
vLLM Version: 0.5.5@09c7792610ada9f88bbf87d32b472dd44bf23cc2
vLLM Build Flags:
CUDA Archs: Not Set; ROCm: Disabled; Neuron: Disabled
GPU Topology:
GPU0    GPU1    CPU Affinity    NUMA Affinity   GPU NUMA ID
GPU0     X      SYS     0-15    0               N/A
GPU1    SYS      X      0-15    0               N/A

Legend:

  X    = Self
  SYS  = Connection traversing PCIe as well as the SMP interconnect between NUMA nodes (e.g., QPI/UPI)
  NODE = Connection traversing PCIe as well as the interconnect between PCIe Host Bridges within a NUMA node
  PHB  = Connection traversing PCIe as well as a PCIe Host Bridge (typically the CPU)
  PXB  = Connection traversing multiple PCIe bridges (without traversing the PCIe Host Bridge)
  PIX  = Connection traversing at most a single PCIe bridge
  NV#  = Connection traversing a bonded set of # NVLinks

How would you like to use vllm
I have built vLLM into my application for inference and it works great.
However, I'm running into issues when it comes to testing the application including vLLM. In the following is a minimal reproducing example to show the issue I'm running into.
# test_vllm.py
from vllm import LLMEngine, EngineArgs, SamplingParams

def test_with_tp_1():
    engine_args = EngineArgs(tensor_parallel_size=1)
    engine = LLMEngine.from_engine_args(engine_args)
    engine.add_request(
        inputs="hello",
        params=SamplingParams(),
        request_id="tp=1",
    )

def test_with_tp_2():
    engine_args = EngineArgs(tensor_parallel_size=2)
    engine = LLMEngine.from_engine_args(engine_args)
    engine.add_request(
        inputs="hello",
        params=SamplingParams(),
        request_id="tp=2",
    )

If I try to run this snippet using pytest like
pytest test_vllm.py

I run into the following error
AssertionError: tensor parallel group already initialized, but of unexpected size: get_tensor_model_parallel_world_size()=1 vs. tensor_model_parallel_size=2

And hence I'm wondering whether it is somehow possible to run multiple tests sequentially using different values for tensor_parallel_size across tests using the LLMEngine interface and pytest.
Note, I also experimented with the use of
from vllm.distributed import destroy_model_parallel, destroy_distributed_environment
destroy_model_parallel()
destroy_distributed_environment()

and
pytest-forked

But both also did not help getting around this.
Any hint would be appreciated - thanks!
Before submitting a new issue...

[X] Make sure you already searched for relevant issues, and asked the chatbot living at the bottom right corner of the documentation page, which can answer lots of frequently asked questions.

We have run into similar issues in the past and managed to work around them using this function. See if it can also solve your issue,

simply open the “vllm/distributed/parallel_state.py” and look at the line 1059：
assert (
0 and get_tensor_model_parallel_world_size() == tensor_model_parallel_size
), ("tensor parallel group already initialized, but of unexpected size: "
this “assert” cause the program to be interrupt
i try to just delet the assert
amzing！！
find the program run well on my 4 GPU

i think ignor the “assert” dont cause much question

We have run into similar issues in the past and managed to work around them using this function. See if it can also solve your issue.

Thanks for the pointer, but that also did not help in my scenario.

We have run into similar issues in the past and managed to work around them using this function. See if it can also solve your issue.

Thanks for the pointer, but that also did not help in my scenario.

it looks strange. you may need to post the full code, and how you use this function. forking a process for the test should work in my opinion.


Save OBD data received from ELM327 usig traccar.

Hi,
I am using Traccar plugin to save the GPS data from my mobile phone onto OpenGts. Now, I have developed
an android app to talk to ELM327 - OBD2 dongle.  I am able to receive data from the dongle, now, I have to save OBD data like GPS data. How can this be done? Your help is appreciated.
Thanks,
Ranganath

If you are using OsmAnd protocol you can include as many additional parameters as you want.

Hi ranganathdg,
have you made some progress with your own OBD2 data integration?
Thanks,
MrFX


Include beta4 and latest projects correctly in solution

VS2015 also updated project GUIDs.

Hi @henkmollema, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by .NET Foundation and real humans are currently evaluating your PR.
TTYL, DNFBOT;

I'm ok with the update, has to happen. But I didn't realize we had xproj files in here at all... not sure that we want them. You can open a project.json in Visual Studio and having the xproj might confuse someone who is not a VS user running them from OSX or something.
What do you think, do you find value in having an xproj already there?

@glennc I get your point, but will non-VS users even bother about the solution/xproj files? They probably know they won't need them while people who do use VS might find it convenient when there is sln file with all the projects for testing/playing around.

Sounds reasonably. @BillHiebert or @PradeepKadubandi  Is there anything in here we should avoid?


eval hovers show content now

Deleted a single '#'.

Nice thank you very much! Not sure how I missed this :sweat_smile:


Making own dataset

I am trying to test this Non-parametric Network with some outdoor 3D scenes, including NPM3D, Kitti and data captured by ourselves. If you are also testing that, when would you expect to update the result and code in this repo?
I noticed a 'data_scan.py' file for processing the Scannet dataset exists, but no corresponding 'run_nn_scan.py'. If you have the code for running the Scannet with Non-parametric Network and can upload it, it may help me a lot! Or could you provide some instructions about preparing other datasets, especially on arranging the Point-Memory Bank?
Many thanks.

Thanks for your interest. Sorry for misleading. data_scan.py refers to the ScanObjectNN dataset for shape classification. We haven't try Point-NN on ScanNet dataset and would leave it as a future work.
For point-memory bank for scene-level dataset, I suppose it's better to refer to the bank of Point-NN for part segmentation, which we have open-sourced at here. The feature memory is better to be clustered by some rules to save GPU memory and obtain prototypical characters.

Thanks for your reply.


Datadog Fabric task doesn't respect aliases

Datadog's annotation @notify "eats" aliases.
E.g. I cant execute this task with alias "all" I can execute it only with it's name "install" having this configuration:
@notify
@task(alias="all", default=True)
def install():
...

Closing and reopening on  datadogpy repo.


[Feature] Support yarn per job mode for paimon web flink submission

Search before asking

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

Motivation
Currently, we support submission of yarn application mode and need to support yarn per job mode.
Solution
No response
Anything else?
No response
Are you willing to submit a PR?

[ ] I'm willing to submit a PR!

I'm interested in this issue and will try to support this feature. Please assign this ticket to me.

As I know, flink per job mode is not supported in flink1.17, So do you consider not doing this content first?


Filter-List Updated

#16 enhancement/dropdown-menu-added

Kindly update your forked repo as it has merge conflicts in the products.css file.


Secondary Species concentration

I simulated chloride transport alone with Cl- ,H+ and OH-. The results indicate that OH- is always 10^-20,  but it should be around 10^-9. Are all values below a certain threshold displayed as 1E-20? In bioRT-Flux-PIHM, only minerals are printed as 1E-20. That's why I wanted to ask this.

Are you running it using the transport only mode? When simulating transport, only aqueous species are considered, without reactions or speciation. So secondary species are not updated. Should we do speciation?

Yes, I was running on transport mode. What you say makes sense. If I were to use only Cl- or some other non reactive tracer, it doesn't matter if there is no speciation. But if H+ is involved, I feel OH- should be calculated according to speciation even if transport alone is simulated. Let's consult Lili on it.


Flyway command line cannot handle special character

What version of Flyway are you using?
4.2.0
Which client are you using? (Command-line, Java API, Maven plugin, Gradle plugin, SBT plugin, ANT tasks)
Command-line
What database are you using (type & version)?
Redshift
What operating system are you using?
unix (centos) and same on osx (el capitan)
What did you do?
(Please include the content causing the issue, any relevant configuration settings, and the command you ran)
./flyway info -user=test -password=AAYt!!$%gSL4j()($%%Gshs
or
./flyway info -user=test -password="AAYt!!$%gSL4j()($%%Gshs"
(same with all other commands..)
What did you expect to see?
The info of the migration
What did you see instead?
: Amazon Invalid operation: password authentication failed for user "test";
Note that when I put the exact same password in the flyway.conf (flyway.password) it works perfeclty

Duplicate of #1658
Please stop opening duplicate issues.

@axelfontaine I think I maybe running into a similar issue, my username is octo@octo-dev-dbif I if I use it as -user=octo@octo-dev-db I see the ending being truncated so only octo is being set. If I use it as '-user=octo@octo-dev-db' I am getting ERROR: Invalid operation: '-user=octo@octo-dev-db'


Revert "Try ActiveRecord 3.2 for the 3.0.gemfile on Travis"

This reverts commit d0b7cf291af843a7b3676077cc177149bba17793,
which broke Travis.

Example of broken Travis build (seems to only affect ruby 1.8.7 jobs):
Bundler could not find compatible versions for gem "activesupport":

  In Gemfile:

    railties (~> 3.0) ruby depends on

      activesupport (= 3.0.0) ruby

    activerecord (~> 3.2) ruby depends on

      activesupport (3.2.0)

Bundler could not find compatible versions for gem "rack":

  In Gemfile:

    railties (~> 3.0) ruby depends on

      actionpack (= 3.2.0) ruby depends on

        rack (~> 1.4.0) ruby

    rack-test (>= 0.6) ruby depends on

      rack (1.0.0)

From https://travis-ci.org/airblade/paper_trail/jobs/131233189

Closing, superseded by https://github.com/airblade/paper_trail/pull/820


fix inline mathjax handling

previously it was converting mathjax unsupported inline math delimiters $ to block math delimiters $$ which are not equivalent. here it instead converts inline math delim $ to the mathjax supported inline math delim \( \) which have the expected equivalent behavior

@chrisbresten, would you like to finish the small "cosmetic modifications" as suggested above, or can I take it over? :)

@chrisbresten, would you like to finish the small "cosmetic modifications" as suggested above, or can I take it over? :)

hi, i am sorry for the slow response. I am not that well versed in the style conventions and cleanliness standards for production in this context so i had kind of deferred to you implicitly. I should have stated this.

i checked my local copy and found changes to unit tests to reflect the new behavior properly. I defer to you @pbodnar for any changes that help maintain standards and style consistent with the codebase at large

coverage: 94.162% (+0.09%) from 94.073%
when pulling 5174389aeff5de6e98461827437ca808fd3b54b3 on chrisbresten:inline_mathjax
into 70e8e8dbc24b836e554f46e7f23c2cabc7dbd77a on miyuchina:master.

@chrisbresten, thanks once again. I did the final amendments and hopefully didn't break anything. ;)

BTW I have found this cool "cheatsheet" for the MathJax syntax: https://math.meta.stackexchange.com/questions/5020/mathjax-basic-tutorial-and-quick-reference/.

BTW I have found this cool "cheatsheet" for the MathJax syntax: https://math.meta.stackexchange.com/questions/5020/mathjax-basic-tutorial-and-quick-reference/.

thankyou. I write a lot of latex and there can be a lot of variation in what kind of syntax people choose to use out of habbit. there are a lot of options for some functionality due to improvements manifesting as new commands in order to leave the old ones for backward compatibility. also a lot of packages people use so ubiquitously that they forget the corresponding functionality is not builtin. mathjax seems to do a good job at keeping a pulse on this and providing a math environment that is compliant with this implicit standard


Back navigation from ErrorBoundary does not work properly if ErrorBoundary does not include

What version of Remix are you using?
1.3.5
Steps to Reproduce

Clone and run https://github.com/machour/francis-issue (thanks to @machour for helping to investigate this!)
Navigate from / to /boom via the link, confirm that the error boundary renders
Navigate back via browser controls, confirm that the root page correctly renders.
Navigate to /boom via the link, reload the browser tab, then navigate back. The error boundary will remain mounted and the app is unusable.

This issue has been confirmed intermittently on Chrome, and repeatably on Safari and Firefox on OSX 10.12.
Of note: if <Scripts /> is added to the ErrorBoundary, back button behavior after a refresh works as expected. This only occurs when the top-level error boundary does not load javascript.
Expected Behavior
Navigating back after reloading should return to the preceding page.
Actual Behavior
Navigating back after reloading, to any number of parent pages, leaves the error boundary mounted. A full reload is required to interact with the app.

Interesting fact: the issue only happens if there's no <Scripts/> in the ErrorBoundary 👀

Opened a bug-report PR at #2919.
Oddly enough, the bug fails with or without <Scripts /> in the boundary

Closing this in favor of #2919

Cross-referencing https://github.com/remix-run/remix/pull/2919#issuecomment-1629665909 from here - this was determined to be expected behavior


[dv/rstmgr] Update reset consistency checker instances

Signed-off-by: Guillermo Maturana<EMAIL_ADDRESS>

This was causing lots of unit test failures.


No module named 'ngtpy'

Thanks for releasing NGT.
I have tried running the provide sample python script.
It seems that the scripts works as expected (quick and quite accurate) after I create the conda environment for running NGT package.
However, after I deactivate and activate the same environment again, I get the error message:
ModuleNotFoundError: No module named 'ngtpy'
It seems to be the problem with how the path is specified.
But, I don't know how to fix this problem. Could you recommend how to fix this path problem, or may be could you give me away to force the path to both ngtpy and ngt modules?
I performed the following command lines, which leads to the problem:

Firstly, I installed NGT on my linux (base-environment)

$ unzip NGT-x.x.x.zip
$ cd NGT-x.x.x
$ mkdir build
$ cd build
$ cmake ..
$ make
$ make install
$ ldconfig /usr/local/lib

Then, I install the python package in base-environment using:

$ pip3 install pybind11
$ pip3 install numpy
$ cd NGT_ROOT/python
$ python3 setup.py sdist
$ pip3 install dist/ngt-x.x.x.tar.gz

Then, I create a conda environment, activate it to run the sample.py, and then, deactivate the conda environment.
After deactivation, I just simply activate the conda environment that I have just created. Then, it doesn't work.

It seems that anaconda has python3 but no pip3 in my environment. If your environment is the same as mine, you have may use /usr/bin/pip3 instead of anaconda's pip. It means that ngtpy was installed in the wrong place. Why don't you use "pip" that is placed in anaconda directory instead of "pip3".

Yes, it works! You saved my life !  Thanks so much !!!!!

@masajiro It would be great if the install/build instructions/readme.md included the latter half of Gabby's steps above. Thanks.

@ldmtwo What is the latter half?

$ pip3 install pybind11
$ pip3 install numpy
$ cd NGT_ROOT/python
$ python3 setup.py sdist
$ pip3 install dist/ngt-x.x.x.tar.gz

is written in README.md.
Or my comment?

It seems that anaconda has python3 but no pip3 in my environment. If your environment is the same as mine, you have may use /usr/bin/pip3 instead of anaconda's pip. It means that ngtpy was installed in the wrong place. Why don't you use "pip" that is placed in anaconda directory instead of "pip3".


Make style changes and improve clarity

What
Made some changes so that content is in line with the style guide and is more readable
How to review
Describe the steps required to test the changes.

Preview the content locally (see README) and check that it renders as expected.
Check that the content still makes sense from a technical perspective.

Who can review
Not @cmcnallygds

Reviewed changes to postgres documentation.

Reviewed changes to postgres documentation.


Hard-coded Wordle list has been changed by NYT

Hi there,
It appears that the NYT has removed all "offensive", "insensitive", and "obscure" words from their hard-coded Wordle list. It would be awesome if you could update the wordles list within wordle_dictionary.py to use the NYT hard-coded list (NOTE: this can still be found by inspecting the source code).
Cheers!

Fixed in commit 297764cc8bb88796a8031b57784ec8a130525b1c
New version 0.2.1 uploaded to PyPi.


Split recovery tests in individual tests

Using labels so we can run them individually for arm64, as they take too
long
It also uses a bit of trickery to determine if we are running the tests
with labels, which would indicate that we are running them individually
and skips the reset part to speed them up even further.
Currently only done for arm64, x86 tests are still run on the whole
suite
Signed-off-by: Itxaka<EMAIL_ADDRESS>

This has already passed before and it works as expected, only fixed one of the tests targets running 2 tests instead of one.


AWS Meilisearch Allow Saving to EBS Volume

I like the tool, but I don't like how data is stored on the EC2 instance in AWS and as far as I can tell can't be changed to live on an EBS volume that isn't the root instance.
Sorry not a bug, but there wasn't a place for this type really.

You can use the --db-path argument to specify a path of your choosing for data storage.

Hello @czabriskie. Thanks for the report. I don't have much context, but I'm curious to know what happens when you specify Meilisearch's db path to an EBS volume. My understanding from the docs here is that the EBS volume is directly connected to EC2, so I see no reason why it wouldn't work.

I'll close this issue out due to inactivity. Thanks for the report to begin. Please feel free to re-open if there is additional information or response we can continue with.


How to create labels for each embedded schema item in an array?

Love Simple Schema and Autoform - so useful - thanks!
I'm creating an operating hours record for customer.
For each of the days of the week I need to know if the store is open, when it opens and for how long.
I created a simple "day_schedule" schema  - then within a top level customer schema I created an array that requires 7 day_schedule schemas - one for each day of the week - fine!
I use a quickform - and all 7 days appear - but I have no way to label them "Sunday" "Monday" etc..
I can label the "day_schedule" object.. but I can't seem to provide a different label for every element in the array. Is there an easy way to do something like this?
 :)

I don't think it can be done with a quickForm. You can use a function for label, but I don't think the current array index is provided in that function.
The easiest might be to switch to an autoForm and do your own loop since you want exactly 7 anyway.

Okay
Can u point me to an example that would get me started?
C
Sent from my iPhone
On Feb 12, 2015, at 12:25 PM, Eric Dobbertin<EMAIL_ADDRESS>wrote:
I don't think it can be done with a quickForm. You can use a function for label, but I don't think the current array index is provided in that function.
The easiest might be to switch to an autoForm and do your own loop since you want exactly 7 anyway.
—
Reply to this email directly or view it on GitHub.

Okay... I found your online example with looping and I almost have it - but using autoform in a loop is giving me fits... I can see the three objects - but can't edit them. I'm sure it's something dumb on my part - but since I've returned only a piece of the doc - an internal array - and the schema is using and embedded schema I'm getting lost.
The three operating hours elements of each day as defined in the schema are shown - but I'm not setting up Autoform to edit them in place. ( I've tried a dozen different ways - not just what is here.)
I return the operating_hours array from the doc - for use in the loop and I return the whole doc for use by autoform itself.

I use a Table loop as shown in your online Demo example:

And the result is CLOSE... but no cigar! I see the objects - but am not rendering them as editable fields...

That's not quite what I meant. I think you can have a single autoForm with an each loop within that. In pseudo markup it would be something like:
#autoForm
  #each day

{{name}}

    {{> afQuickField name=fieldName label=false}}
  /each
/autoForm

Where your day helper returns
return [
  {name: 'Sunday', fieldName: 'operating_hours.0'},
  {name: 'Monday', fieldName: 'operating_hours.1'},
  // etc...
];


Patreon Logo svg

Can we think for adjust it to our needs?
https://www.vectorlogo.zone/logos/patreon/patreon-ar21.svg
I'm not on SVGs too much. ;d We can only get the "P" one, but should somehow invert the colors. Actually... the orange should be white and and the rest transparent?

Seems they have redesigned a new logo, we can get it here https://simpleicons.org/?q=patreon

Oooh! Cool, that's a lot better.


Support PHP 8.0

This PR includes changes necessary to improve compatibility with PHP 8, updates our CircleCI configuration, updates our unit test configuration to reflect and target the minimum PHP and WordPress versions we support, and fixes a number of issues with our documentation around parameter types (they remain functionally unchanged; our docblocks were just incorrect in their existing usage.)
To test these changes locally you will need a local MySQL installation. From the root directory run bin/install-wp-tests.sh which will output the proper command syntax. Provide your local database details. For the target WordPress version, specify 5.6 for this PR. After the command finishes, run composer checkup to run the test suite.

This PR is throwing an erroneous codecov failure for unknown reasons. It is detecting code changes in files that weren't changed. Once this is reviewed I'm going to force merge this and hopefully that will resolve the coverage diff issues.

This PR is throwing an erroneous codecov failure for unknown reasons. It is detecting code changes in files that weren't changed. Once this is reviewed I'm going to force merge this and hopefully that will resolve the coverage diff issues.

@lbalmaceda Thanks for your feedback! I addressed your questions and committed a few changes, whenever you get time to re-review. Thanks!

@lbalmaceda Thanks for your feedback! I addressed your questions and committed a few changes, whenever you get time to re-review. Thanks!


Datetime controls dont render properly in AppShell

Hi,
awesome package, thanks for all the work in this.
I have been playing around with the AppShell (using dmc docs as a reference). Every time I add the DatetimeInput to the AppShell header or navbar or the Main, the control appears distorted. Is this something you can help with please?
dmc.AppShellMain(children=[dmc.Stack(children=[
html.Div(
  [
      html.Div(
          [
              dmc.DatePicker(
                  id="date-picker-input",
                  label="Start Date",
                  description="You can also provide a description",
                  minDate=date(2020, 8, 5),
                  value=datetime.now().date(),  # or string in the format "YYYY-MM-DD"
                  w=250,
              ),
              dmc.Space(h=10),
              dmc.Text(id="selected-date-input"),
          ]
      )
  ]
)
])]),

ta!

Found the solution. for anyone else, looking at this, the answer lies on the page:
https://www.dash-mantine-components.com/getting-started


Enter key handling on collapses

We are currently not handling the enter key to toggle on collapse afforandances (at least non-exclusive ones), @scottohara was suprised by that.  It makes sense to me that it should, I kind of vaguely recall @jonathantneal having thoughts on this and maybe even introducing some code, so tagging him and creating an issue to track.

If there’s an enter key, I expect it will behave like a button, and if it’s a button, then I expect the enter key and the space key will both work for toggling.
Additional notes:

The enter key works on keydown.
The space key works on keyup.


modemmanager: fix cross-compile: add python3 to nativeBuildInputs

Description of changes
Redo of #286201, targeting staging this time.
Modem Manager was broken for cross-compilation from x86_64-linux to aarch64-linux.
This fixed it for me.
Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Can we set strictDeps to prevent regressions in the future?

Can we set strictDeps to prevent regressions in the future?

https://github.com/NixOS/nixpkgs/pull/286564


Use size_t for sequence sizes

Proposal
The sizes and capacities of sequences like list, str, bytes, etc. are mostly implemented in C with type int. It would be better if possible to use size_t for such object size quantities.
Motivation
Use of size_t for sequence sizes would make interoperation with established C libraries more natural and avoid artificially limiting sequence sizes with a signed type that may be smaller than the real maximum supported by memory.
Alternatives
Casting to size_t when integrating with C libraries that expect it does work to an extent but has the limitations described above.

@sydow can you comment on this? I think @ciaran2 proposal sounds reasonable but I really don't know this stuff very well. OK to switch to size_t or is there a good reason why we use int?

Isn't the purpose of size_t to provide a reasonable size measurement type adapted to each platform? That means size_t is highly platform dependent as far as I can see, and we've tried to avoid such C types elsewhere in favor of explicit bit widths in order to get a consistent behavior across all targets. So I don't know. I can see the merit of using size_t internally since that's the type used by many C library functions, but the type we choose will nevertheless have to relate to the Acton type that len() returns. And that type ought have a platform-independent bit width, I think.

But if the question is just size_t versus int, I'm all in favor of size_t. Though picking a fixed bit width type has additional merits, as it were.

@nordlander Right, so since int is of platform-dependent size as well, size_t is probably better than int... but you're saying that perhaps uint64_t is even better, right? Using uint64_t would solve the problem of arbitrary cutting the max limit in half, but it still means that whenever we do C lib integrations we prolly have to keep casting stuff.

Yep, that summarizes it well. And the casting that must remain isn't just a nuisance, it actually represents our responsibility to correctly map Acton sizes (as represented by an uint64_t, say) to whatever size_t means on each particular platform. And vice versa, of course.

I don't have more to add; at some point we should go through everything and replace all int types with explicitly sized types, but size_t is probably an improvement for now.
A minor thing is that it disturbs me a little to use a 64 bit field to hold e.g. the length of a list. A 32 bit field is enough for lists of four billion elements, but I will not fight for it, since it is probably a disturbance due to starting as a programmer in the late sixties...


Wrong link in document.

Pointed at performance article instead of cost

Hi @PeterSelchDahl, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. Real humans will now evaluate your PR.

TTYL, MSBOT;

@PeterSelchDahl : Thanks for your contribution to the Azure documentation! The author, @kumudd, has been notified to review your proposed change.

@PeterSelchDahl   Thanks for pointing this out. I agree with your suggestion and am signing off on it.
#sign-off

#sign-off

@KatieCumming


If a JS library adds CSS when triggered, would it be removed if not in the DOM at runtime?

Sorry if my question is covered I dint research much.
I am curious, often JavaScript apps will add CSS classes when events trigger them to be needed.  Is there any practical way for this app to detect that those should stay/remain even though they might not be present in the DOM when this runs?
I realize there might not be a real perfect solution here but perhaps there is above my knowledge level?

I've the same question.
My gulp-uncss configuration is
gulp.task('uncssme', function () {

return gulp.src('src/css/**/*.css')
    .pipe(uncss({
        html: ['src/**/*.html']
    }))
    .pipe(gulp.dest('dist/css'))
   });

And I am adding a 'newClass' through javascript
document.getElementById('good').className += ' newClass';
But by 'newClass' selector is getting removed from CSS file because it is not used directly in html files.
Please, add a way to add parse js files to rectify this problem.
I am already using,
/* uncss:ignore */
.newClass {
  background-color: blue;
}

But it is bad practice to mention this ignore comment every time I dynamically add a class.

So if the JS is added at startup as part of page render, the browser will detect the CSS classes, and UnCSS will keep them. If they are only added in response to events, then UnCSS has no way to detect that they could be added. Trying to do this automatically is essentially an impossible task.
As another potential avenue for a solution here, you could generate a "yourwebpage.html#RunForUnCSS" of sorts (or if it doesn't work, we should add support for such a feature), where your chose of debug flags triggers all your requisite javascript handlers to add all the relevant CSS classes you like in the rendered page, so that UnCSS will retain those classes in the generated CSS.
I'm going to mark this as closed. Please see #244 if you're curious why this is impossible to analyze JS code automatically in the general case.

@mikelambert We've more use cases for adding the class dynamically (using js) on events than adding classes through js in the beginning. Just think, how much useful is this latter part in real world web applications? Not much. Most applications have former part because that is why people want to use js to add classes dynamically on event happening. I was thinking, if we can have some js parser which takes into consideration some rules like wherever addClass, className, id, etc. are found then those classes/ids much be kept in css files. What say?

Yes, there are many reasons why you'd use js to add classes after the page loads. I in fact use many in my own projects as well. And yup, it'd be much more useful if you could handle that all automatically, too!
Unfortunately, you seem to miss that this is a very difficult task to do, and impossible to do correctly. You can add a js parser, and you can check for addClass and other similar names. But how deep do you want to go? For example, even this relatively straightforward code would fail to work:
function handleToggle(obj, visibleClass, invisibleClass) {
if (obj.something) {
obj.addClass(visibleClass);
obj.removeClass(invisibleClass);
} else {
obj.addClass(invisibleClass);
obj.removeClass(visibleClass);
}
}
handleVisibility(h1, 'main-visible', 'main-invisible');
handleVisibility(h2, 'secondary-visible', 'secondary-invisible');
And any debug CSS classes you use would get caught by such a parser and kept:
if (debug) {
obj.addClass('debugClass');
}
...when you actually don't want something like that in production.
Or as another example, even bootstrap JS runs a lot of code to add/remove classes, most of which you might never use/call, but that would trigger those bootstrap CSS classes to be kept around, as a result of the bootstrap JS.
So yeah...I provided a workaround in the above post. If you don't like that workaround, you're welcome to tackle it yourself. You'd have to integrate a javascript parser (and trying to figure out how to work that into the webdriver-based approach the system uses), and make it an optional component (due to the false-positives and false-negatives described above). Good luck!

You're correct that false-positive cases might get added. So, currently, I am fine with using ignore: [regex] to collectively ignore all the classes added with js(regex because, I've made prefix as ".jsa-newClass" for every class/selector added through js.) This helps for css I would create. But for compression of Bootstrap or other vendor files, I think it's better to use other gulp plugin like purifycss if it helps get it done. But anyways, thank you to give insights.


Avalanche market service

Overview
Add market service for acquiring pricing data for assets on avalance starting with avax and bridge assets
References and additional details
https://github.com/ava-labs/avalanche-bridge-resources/blob/main/avalanche_contract_address.json
Acceptance Criteria

Support spot pricing for avax and bridge tokens

Need By Date
No response
Screenshots/Mockups
No response
Estimated effort
No response

closing in favor of #811


Change curator to editor across the board

Coverage remained the same at 92.75% when pulling 50eb1c257c309460947868eec6ec510328d01efc on name-changes into 588059036b1c3c9635497c2caf14a54a58d4275f on master.

Coverage decreased (-0.06%) to 92.98% when pulling 319ccf6bf011cc6344ac12b461525f53d9e2ccea on name-changes into 14a7dc2a1b1f8da4b32d9b5051afa02403c8e9bc on master.

Jon: good to go, I think

Coverage remained the same at 93.04% when pulling ae0799f2404217d8f0ee2babe70a0584fb74e6f6 on name-changes into 14a7dc2a1b1f8da4b32d9b5051afa02403c8e9bc on master.


Add weak pointers GcWeak and GcWeakCell

This is a combination of work by EmperorBale and me. It adds weak-pointer version of Gc and GcCell, named GcWeak and GcWeakCell respectively. These can be created by calling downgrade() on the corresponding 'strong' struct (Gc/GcCell).
You can attempt to upgrade a weak pointer to a strong pointer by calling upgrade. This will succeed as long as the target object has not yet been freed (it will also fail during Phase::Sweep when the target object is going to be freed, but has not yet been freed).
To support this, GcBox now has two additional flags: alive and has_weak_ref. When an allocation with weak pointers is garbage-collected, we just run the destructor on the Collect value if there are still weak pointers. The GcBox itself is not deallocated, so that the weak pointers are still able to access the flags. In a later Phase::Sweep where all of the weak pointers are gone, we actually deallocate the memory for the GcBox.
I've also added comments to existing code, to make it clear how weak pointers interact with the existing invariants.

Sorry this took me a while to get to, I'm actually reviewing the PR now but it's a lot to load back in my head. From the first scan I did, the additional comments you wrote about how the gc works are just :ok_hand:

Okay, after reading over this and... hopefully... remembering how this garbage collector works and reviewing the change, I think the reasoning here is sound.
I have some concerns but they're mostly nits..

@EmperorBale and @Aaron1011 you both might understand the gc better than me at this point considering how long it's been since I've thought about this seriously, so if any of my feedback is just wrong, please point it out. Some of the documentation corrections I suggested were honestly to make sure I understood it myself, maybe i have it wrong? I don't remember half the things I do lol.

@kyren Thanks for the review - I've pushed a new commit addressing your comments.

Ah, also thank you for running under miri, that was gonna be the next thing I did! I'd like to have that as part of CI as well but I'm not asking you to add that in this PR.

LGTM!


Generator can't convert empty object literal to AST

Describe the bug

Empty object literal as a default value produces the following error at the Generate step: TypeError: Cannot convert value to AST: {}
To Reproduce
Repo with minimal example: https://github.com/slikts/gql-gen-default-value-issue/
git clone git@github.com:slikts/gql-gen-default-value-issue.git
cd gql-gen-default-value-issue
npm i
npx gql-gen

GraphQL schema:
scalar Bar

input Foo {
  bar: Bar = {}
}

Environment:

OS: macOS 10.13.6
@graphql-codegen/...: 1.0.4
NodeJS: 11.9.0

@slikts , thank you for reporting it, I'll take a look soon.

I managed to reproduce it. The method printSchema from graphql is expecting each scalar to be serializable, and by providing only scalar A (with the GraphQLScalarType object), makes it impossible to print.
We can use printSchemaWithDirectives from graphql-toolkit that should handle this case as well.
I'll check and update.

@slikts I tested it and it's not simple to avoid this error without building the schema correctly.
I think the easiest solution for you at the moment is to build your schema manually with your scalar resolvers, and let GraphQL Modules load that schema. Make sure to implement serialize method for the scalar resolvers:
        Bar: {
          serialize: (v) => {
            return JSON.stringify(v);
          }
        }

@ardatan @kamilkisiela what do you think?

Any news about this?
I'm getting the same error.


Compile time struct

?

？

？

What problem would this solve, and how would this work?

Performance enhancements by requiring less work during runtime and greater optimizations. Generally for speed-intensive operations the more that can be optimized at compile time, the better.

I'd say this is already implemented. Jou uses LLVM for optimizations, and LLVM can get rid of class instantiations in some cases.
For example, consider:
class Point:
    x: int
    y: int

def get_sum(x: int, y: int) -> int:
    p = Point{x=x, y=y}
    return p.x + p.y

If I try to run this with all optimizations turned on and verbose-verbose (-O3 -vv), I get output that ends with:
===== Optimized LLVM IR for file "a.jou" =====
; ModuleID = 'a.jou'
source_filename = "a.jou"
target datalayout = "e-m:e-p270:32:32-p271:32:32-p272:64:64-i64:64-f80:128-n8:16:32:64-S128"
target triple = "x86_64-pc-linux-gnu"

; Function Attrs: mustprogress nofree nosync nounwind readnone willreturn
define i32 @get_sum(i32 %0, i32 %1) local_unnamed_addr #0 {
block0:
  %signed_op = add i32 %1, %0
  ret i32 %signed_op
}

attributes #0 = { mustprogress nofree nosync nounwind readnone willreturn }

Emitting object file: ./jou_compiled/a/a.o
Running linker: '/usr/lib/llvm-14/bin/clang' "./jou_compiled/a/_assert_fail.o" "./jou_compiled/a/a.o" -o './jou_compiled/a/a' -lm
/usr/bin/ld: /lib/x86_64-linux-gnu/Scrt1.o: in function `_start':
(.text+0x17): undefined reference to `main'
clang: error: linker command failed with exit code 1 (use -v to see invocation)

The relevant part is:
define i32 @get_sum(i32 %0, i32 %1) local_unnamed_addr #0 {
block0:
  %signed_op = add i32 %1, %0
  ret i32 %signed_op
}

So LLVM turned the get_sum() function into basically def get_sum(x, y): return y + x. Here add i32 means the + operator for 32-bit numbers, and %1 and %0 are the arguments of the function. (I do not know why the order is reversed.)
The output ends with a linker error because I didn't define a main() function.

Beautiful. I wasn't sure if it would do that in Jou. To understand what I meant take a look at the difference from def and fn or class and struct in Jou which optimizes them differently.

I still don't know what you're talking about. Jou doesn't have struct or fn keywords, and that would be irrelevant anyway, because the optimizer doesn't know anything about Jou's syntax anyway.
But it seems like Jou does what you wanted it to do.


Enabling DKIM without Windows PowerShell

There are no instructions for how to enabling DKIM for Office 365 without Windows PowerShell. How can I enable DKIM using another OS?

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

ID: 249c059f-c3ce-a382-b99e-795a0c1f646a
Version Independent ID: 5656c114-ae16-6194-cd9f-353b33f4a1e1
Content: Use DKIM for email in your custom domain in Office 365, 2048-bit, 1024-bit, steps, How it works, SPF, DMARC - Office 365
Content Source: microsoft-365/security/office-365-security/use-dkim-to-validate-outbound-email.md
Service: o365-seccomp
GitHub Login: @MSFTTracyP
Microsoft Alias: tracyp

Hello @mikaelf Mikael, how are you? 👍
For now, you will have to rely on the admin center to do that, the article includes those steps too.
Hope this info helps you!

Hi! Thanks. Just to clarify, is it possible to enable DKIM for my custom domain from the web interface alone, without having to enter PowerShell commands? The article leads me to believe I have to do CLI stuff!

Hi! Thanks. Just to clarify, is it possible to enable DKIM for my custom domain from the web interface (the admin center) alone, without having to enter PowerShell commands? The article leads me to believe I have to do CLI stuff!

Yes, the article presents alternatives, you either use the admin center o powershell.

@mikaelf - Thank you for submitting feedback.
From our understanding, the issue you raised has been answered by @get-itips  so we will close this issue.
Thank you for your contribution to make the docs better! Much appreciated!

I managed to follow the enable DKIM now.
The hard part was actually to "Publish two CNAME records for your custom domain in DNS". Would be pretty sweet to be able to get the correct CNAME records right there in the documentation, without having to run the two PowerShell commands or figure it out yourself reading the format or the example.
@get-itips (@mypil)

I managed to follow the enable DKIM now.
The hard part was actually to "Publish two CNAME records for your custom domain in DNS". Would be pretty sweet to be able to get the correct CNAME records right there in the documentation, without having to run the two PowerShell commands or figure it out yourself reading the format or the example.
@get-itips (@mypil)

@mikaelf Thanks for your feedback, while it is true that the section you mention starts with a PowerShell cmdlet, later on, explains how the DNS records should be created, with real examples (for cohovineyard-com) and meaning of each component, do you still feel this is not clear enough? Thank you

I still believe it could be easier and clearer. The rest of the docs are super easy to understand.

@get-itips currently, the article says verbatim:

Sign in to Microsoft 365 with your work or school account.
Select the app launcher icon in the upper-left and choose Admin.
In the lower-left navigation, expand Admin and choose Exchange.
Go to Protection > dkim.
Select the domain for which you want to enable DKIM and then, for Sign messages for this domain with DKIM signatures, choose Enable. Repeat this step for each custom domain.

These steps lead to the dkim page in the old Exchange admin center, which shows the following banner:

DKIM has a new home and improved functionality. By December 1, 2020 – the DKIM experience will be removed from the Exchange admin center. Please use the updated experience in the Security and Compliance Center, DKIM page.

The table also provides no controls at all, so naturally one reads the banner and follows through to the Security and Compliance Center link, which leads to protection.office.com/dkimv2.
Infuriatingly, though, it only shows the status for non-DKIM-enabled domains, without any way to change it:

Status
No DKIM keys saved for this domain.

As it is, both the docs are outdated and the UI is nonfunctional. How am I supposed to enable DKIM?

@get-itips currently, the article says verbatim:
As it is, both the docs are outdated and the UI is nonfunctional. How am I supposed to enable DKIM?

I don't think there is another way anymore. It used to be there in the past then suddenly Powershell was needed. That worked, but then a new url was needed to run the scripts, which worked too, but now both does not work anymore on my Windows 10 and neither on my Linux with PowerShell for Linux installed. How can a self-employed person that wants to use a normal security feature set up Office 365 with DKIM. They cannot. Even if you have knowledge of the technology, you cannot get it to work. It is a mess.

@get-itips currently, the article says verbatim:
As it is, both the docs are outdated and the UI is nonfunctional. How am I supposed to enable DKIM?

I don't think there is another way anymore. It used to be there in the past then suddenly Powershell was needed. That worked, but then a new url was needed to run the scripts, which worked too, but now both does not work anymore on my Windows 10 and neither on my Linux with PowerShell for Linux installed. How can a self-employed person that wants to use a normal security feature set up Office 365 with DKIM. They cannot. Even if you have knowledge of the technology, you cannot get it to work. It is a mess.

@get-itips @MSFTTracyP @mypil I finally took the time to sort this out with MS365 support, which gave me the following answer (emphasis mine):

Hello Moritz,
Hope you are doing good!
I have checked all the possible ways to enable DKIM. However, it not possible without PowerShell. Please follow the below article to install PowerShell on macOS.
Installing PowerShell on macOS - PowerShell | Microsoft Docs
Thank you for choosing Microsoft.
Kind Regards,
Nidhi
O365 Ambassador

I see this as confirmation that the documentation is definitely outdated, wrong, and misleading. The only reason I care about this is because the whole thing really got on my nerves and wasted lots of time, so could you please get this sorted to spare others from this?

@get-itips @MSFTTracyP @mypil I finally took the time to sort this out with MS365 support, which gave me the following answer (emphasis mine):

Hello Moritz,
Hope you are doing good!
I have checked all the possible ways to enable DKIM. However, it not possible without PowerShell. Please follow the below article to install PowerShell on macOS.
Installing PowerShell on macOS - PowerShell | Microsoft Docs
Thank you for choosing Microsoft.
Kind Regards,
Nidhi
O365 Ambassador

I see this as confirmation that the documentation is definitely outdated, wrong, and misleading. The only reason I care about this is because the whole thing really got on my nerves and wasted lots of time, so could you please get this sorted to spare others from this?

I used this script on two domains and someone else did one with the same script. REPLACE the words in CAPITAL :)
Install-Module -Name ExchangeOnlineManagement
Import-Module ExchangeOnlineManagement
Get-Module ExchangeOnlineManagement
$credential = Get-Credential REPLACEWITHYOUROFFICE365MAILADRES
Connect-ExchangeOnline -Credential $credential -ShowProgress $true
New-DkimSigningConfig -DomainName DOMAINNAME -KeySize 2048  -Enabled $true

When you get an error, you could use this command:
Set-ExecutionPolicy RemoteSigned
and run again.
To check the DKIM status
Get-DkimSigningConfig -Identity DOMAINNAME
Replace the DKIM keys with a new one:
Rotate-DkimSigningConfig -Identity DOMAINNAME

I used this script on two domains and someone else did one with the same script. REPLACE the words in CAPITAL :)
Install-Module -Name ExchangeOnlineManagement
Import-Module ExchangeOnlineManagement
Get-Module ExchangeOnlineManagement
$credential = Get-Credential REPLACEWITHYOUROFFICE365MAILADRES
Connect-ExchangeOnline -Credential $credential -ShowProgress $true
New-DkimSigningConfig -DomainName DOMAINNAME -KeySize 2048  -Enabled $true

When you get an error, you could use this command:
Set-ExecutionPolicy RemoteSigned
and run again.
To check the DKIM status
Get-DkimSigningConfig -Identity DOMAINNAME
Replace the DKIM keys with a new one:
Rotate-DkimSigningConfig -Identity DOMAINNAME

@mypil Can you reopen this issue and find a solution?
Setting up DKIM should be a simple process verified by the custom domain setup like spf and other dns entries. https://admin.microsoft.com/Adminportal/Home?#/Domains

Definitely agree here. Having to use PowerShell to enable DKIM is completely crazy. You can't reasonably expect small business customers with limited tech knowledge (and no dedicated IT employees) to achieve this.
This is something that can be done in minutes with Google Workspace or e-mailing providers.

@mypil not responding at all is one way to handle customer feedback. This issue is still standing - the admin center doesn't provide any way to change DKIM keys, and the docs don't even reflect the UI change.

 I normally use Google Workspace and its a few clicks to get the DNS records I have to add. This needs to make it into the new security admin panel asap.

This shouldn't be closed.


Send money - IOU preview appears in one transaction report view

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 1.4.59-3
Reproducible in staging?: y
Reproducible in production?: n
Issue reported by:
Action Performed:

Go to staging.new.expensify.com
Go to 1:1 DM.
Click + > Send money.
Send money to the user.
Click on the IOU preview.

Expected Result:
There will not be IOU preview in the one transaction report view (combined report).
Actual Result:
There is an IOU preview in the one transaction report view (combined report), which opens the same transaction thread.
Workaround:
n/a
Platforms:

Which of our officially supported platforms is this issue occurring on?

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

Screenshots/Videos
Add any screenshot/video evidence
https://github.com/Expensify/App/assets/93399543/715de719-939d-4c20-80eb-8ba835b130f4
View all open jobs on GitHub

I can help with this one

cc @NikkiWines

PR is here https://github.com/Expensify/App/pull/39499

This will be $250 to @parasharrajat for review once all is wrapped up

This is ready for payment via ND, see summary: https://github.com/Expensify/App/issues/39490#issuecomment-2057506539

$250 approved for @parasharrajat


keyboard problem

We have written the needed data into
Issue Type: Bug
I can't write letter H in my vs code neither in search nor in editor or even not while renaming file. H letter is not working
VS Code version: Code 1.56.2 (054a9295330880ed74ceaedda236253b4f39a335, 2021-05-12T17:13:13.157Z)
OS version: Windows_NT x64 10.0.18362

System Info

Item
Value

CPUs
Intel(R) Pentium(R) CPU B960 @ 2.20GHz (2 x 2195)

GPU Status
2d_canvas: enabledgpu_compositing: enabledmultiple_raster_threads: disabled_offoop_rasterization: unavailable_offopengl: enabled_onrasterization: unavailable_offskia_renderer: enabled_onvideo_decode: enabledvulkan: disabled_offwebgl: enabledwebgl2: unavailable_off

Load (avg)
undefined

Memory (System)
3.59GB (1.36GB free)

Process Argv
--crash-reporter-id 2f4acb3f-406d-476b-8f57-7e650cd6c5a6

Screen Reader
no

VM
0%

Extensions (7)

Extension
Author (truncated)
Version

vscode-html-css
ecm
1.10.2

code-runner
for
0.11.4

python
ms-
2021.5.842923320

vscode-pylance
ms-
2021.5.3

jupyter
ms-
2021.6.832593372

LiveServer
rit
5.6.1

vim
vsc
1.20.3

A/B Experiments
vsliv368:30146709
vsreu685:30147344
python383cf:30185419
pythonvspyt700cf:30270857
pythonvspyt602:30300191
vspor879:30202332
vspor708:30202333
vspor363:30204092
pythonvspyt639:30300192
pythontb:30283811
vspre833:30267464
pythonptprofiler:30281270
vshan820:30294714
pythondataviewer:30285071
vscus158cf:30286554
vscgsv2ct:30301613
vscorehovct:30302760
bridgeflightcf:30302070
vscod805:30301674

your clipboard because it was too large to send. Please paste.

i had also tried by removing vim extension .

Please see this page for troubleshooting help https://github.com/microsoft/vscode/wiki/Keybinding-Issues

Let's first try to find out what command executes on your machine, with your settings and your keybindings when pressing H. Could you please try the following:

Run the command Developer: Toggle Keyboard Shortcuts Troubleshooting
This will activate logging of dispatched keyboard shortcuts and will open an output panel with the corresponding log file.
Press exactly the keybinding you desire.
check what VS Code detects and what command is invoked.

If you are having trouble understanding the output, please feel free to paste here and I can try to help.


Blog Form

To Do:

Styling "create blog" form.
Form "create blog".

DO NOT FORGET!
this is used for: (security) on every page in the top
require_once 'login_check.php';

let bamischijf;

https://stackoverflow.com/questions/1636877/how-can-i-store-and-retrieve-images-from-a-mysql-database-using-php/1638348#1638348


Update windows-10-subscription-activation.md

made changes to the note as Excluded Cloud Apps are configured in Conditional Access policies, not in a compliance policy
Fixes#https://github.com/MicrosoftDocs/windows-itpro-docs/issues/11335

@JohanFreelancer9 Please copy review

@frankroj Please merge this PR. Thanks

@frankroj Gentle reminder

Approving change


Assertion fail under outputNullDatum jet.

Running this fuzzer generated test case with assertions enabled and test.c:testTx1 as the transaction context:
test: primitive/elements/../../frame.h:93: skipBits: Assertion `n <= frame->offset' failed.
Program received signal SIGABRT, Aborted.
0x00007ffff7e17e35 in raise () from /lib64/libc.so.6
(gdb) bt
#0  0x00007ffff7e17e35 in raise () from /lib64/libc.so.6
#1  0x00007ffff7e02895 in abort () from /lib64/libc.so.6
#2  0x00007ffff7e02769 in __assert_fail_base.cold () from /lib64/libc.so.6
#3  0x00007ffff7e10566 in __assert_fail () from /lib64/libc.so.6
#4  0x0000000000422435 in skipBits (frame=, n=253) at primitive/elements/../../frame.h:93
#5  0x000000000042718d in skipBits (frame=, n=) at primitive/elements/jets.c:339
#6  outputNullDatum (dst=0x43cbd0, src=..., env=) at primitive/elements/jets.c:334
#7  0x00000000004141c9 in runTCO (env=0x7fffffffd560, len=, type_dag=0x43d070, dag=0x43c570, stack=, state=...) at eval.c:368
#8  evalTCOExpression (evalSuccess=evalSuccess@entry=0x7fffffffd644, output=output@entry=0x0, outputSize=outputSize@entry=0, input=input@entry=0x0,
inputSize=inputSize@entry=0, dag=0x43c570, type_dag=0x43d070, len=, env=0x7fffffffd560) at eval.c:686
#9  0x000000000040ce2b in evalTCOProgram (env=0x7fffffffd560, len=7, type_dag=, dag=, evalSuccess=0x7fffffffd644)
at primitive/../eval.h:47
#10 elements_simplicity_execSimplicity (success=success@entry=0x7fffffffd644, tx=, ix=ix@entry=0, cmr=cmr@entry=0x0, wmr=wmr@entry=0x0,
file=) at primitive/elements.c:491
The program is:
unsigned char t3out_139_crashes_id_000000_sig_06_src_002450_op_havoc_rep_2[] = {
0xce, 0x02, 0x00, 0x44, 0xd0, 0x42, 0x41, 0x09
};
unsigned int t3out_139_crashes_id_000000_sig_06_src_002450_op_havoc_rep_2_len = 8;

For the record, the program in question decodes as the (untyped) simplicity dag:
[Prim (someArrow version)
,Prim (someArrow version)
,Pair () () () 1 1
,Prim (someArrow outputNullDatum)
,Comp () () () 2 1
,Unit ()
,Comp () () () 2 1
]

Fixed by #23


Unable to install operator OpenShift 4.5

We are currently unable to install the operator in OpenShift 4.5 environment; here are some notes from what we have found and a link to updated clusterserviceversion to resolve.
https://bugzilla.redhat.com/show_bug.cgi?id=1882577 ( I believe you can see this )
https://github.com/kubernetes/kubernetes/issues/91395#issuecomment-633261523:
"properties defined as list map keys must either be required or have a default value. Otherwise merge behavior on list-map items that do not specify is undefined. that bug was fixed in 1.18.
You can specify default attributes using the v1 CRD API. To use list-map keys in a v1beta1 CRD, the properties must be made required (specifying default values is only available in v1 CRDs, see https://kubernetes.io/docs/tasks/access-kubernetes-api/custom-resources/custom-resource-definitions/#defaulting)"
https://kubernetes.io/docs/tasks/extend-kubernetes/custom-resources/custom-resource-definitions/#defaulting
Link to update CSV: https://gist.github.com/mrhillsman/568d0624bcc866b23865ef29e751e863

I believe this issue has been fixed in 1.3.0. I see the issue shows 1.2.0. Can you validate the newer version?
I'm working to get the version 1.4.1 pushed out to OpenShift soon.

Hey @bradfordcp
We currently only have the 1.2.0 version listed in the hub. Version 1.3.0 would need to be published.

any update on that? openshift 4.6 was just released which is also impacted.
please keep in mind that starting with openshift 4.6 there is per-version operator catalog, so you probably need to publish the fixed operator in both catalogs (one for 4.0-4.5, and the other for 4.6).


Highlight scoping implications for init scripts

Fixes #13362

Your PR is queued. See the queue page for details.

OK, I've already triggered a build for you.


Release plan for 0.9.3

I wanted to be able to install prqlc via winget in 0.9.0, but it failed because of #3080.
So I would like to release 0.9.3 as soon as possible if possible so that Windows users can easily install prqlc.
Is there anything else that should be included in 0.9.3?

Great! If you're up for releasing, we can do it now. Docs at https://prql-lang.org/book/project/contributing/development.html#releasing...

Is there anything else that should be included in 0.9.3?

I don't think we need anything in the Changelog.

If you prefer me to, lmk!

I understand that we can release right now, but I am also not comfortable with frequent releases. (We have already released three times in two days!)
Sorry for the way I wrote "as soon as possible", I was thinking of the next few days.

OK! Whenever you prefer (I'm more OK with frequent releases — though I would also have not made the mistakes to get three releases off!)

@max-sixty Just to confirm, am I understanding correctly that you are creating the tags from the GitHub web (release page) UI? (I only have experience with either pushing a local tag or having a tag pushed on GHA)
https://github.com/PRQL/prql/blob/71bd4709471ac9022c6f22d43c207743861f86a6/web/book/src/project/contributing/development.md?plain=1#L393-L401

am I understanding correctly that you are creating the tags from the GitHub web (release page) UI?

Yes!

#3189

#3189

Sorry if I missing something, let me know if this required something from me...

Sorry if I missing something, let me know if this required something from me...

Sorry for the confusion. I simply made a note that the fix for docs.rs might be a motivation for the release.

I simply made a note that the fix for docs.rs might be a motivation for the release.

Ah sorry, seems obvious in retrospect!
Yes shall I make a release?

Sorry, I was hoping to do the release work but may not have time to do it this week. (I started a new job in August and my life is kinda unstable)

Congrats on the new job! Hope your first few days go well, definitely prioritize that :)

If you are willing to do the release work today or tomorrow I would be very pleased.

OK!

Done!

Thanks!


 For consistency with CouchDB terminology.
And retain the old 'force_commit' option for backward compatibility.

Codecov Report

Merging #53 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master      #53      +/-   ##
==========================================
+ Coverage   96.34%   96.35%   +<.01%
==========================================
  Files          20       20
  Lines        1452     1453       +1
==========================================
+ Hits         1399     1400       +1
  Misses         41       41
  Partials       12       12

Impacted Files
Coverage Δ

couchdb.go
100% <100%> (ø)
:arrow_up:

bulk.go
100% <100%> (ø)
:arrow_up:

options.go
100% <100%> (ø)
:arrow_up:

db.go
97.82% <100%> (ø)
:arrow_up:

chttp/chttp.go
97.88% <100%> (ø)
:arrow_up:

replication.go
95.32% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


React to hosting

@davidfowl this looks a bit weird, but I think it works...
cc @divega

Merged 167e85a65d96cd72715e8491c6e45309e9a46b97


Skyhook incompatible with latest version of skylark

I updated to master just now and hit an issue on compile, specifically:
vendor\github.com\hippogryph\skyhook\convert\conv.go:16:2: undefined: resolve.AllowFreeze

It looks like freeze was just removed:
https://github.com/google/skylark/commit/ffb61b871f41afe93fdd2ab826a4f352cb770a83

this is fixed


firewall: alias nesting vs. geoip

Adding a GeoIP alias "CH" and selecting "Switzerland" will make the alias fail to fetch any data.
This changed from 17.7 to 18.1 according to https://forum.opnsense.org/index.php?topic=7078.msg32856#msg32856
Do we add better validation or can we reenable this behaviour?

ok, this should work, https://github.com/opnsense/core/commit/cdae884ec970ab4d214ea8b28010c421c03ba018 makes sure we don't skip resolving when the content equals our alias name.
Only downside is that when someone defines an alias "CH" with static content and has an alias containing both "CH" and "NL" the static alias is preferred.

Works, static preference sounds good. ❤


[No ticket] Hackathon - remove unused dependencies

Purpose
Get rid of dependencies we seem to not be using
Changes

Removed vine
Removed html5lib

Side Effects
I mean, if we are actually using them, it probably won't build. Otherwise is probably fine.
Ticket
N/A

Looks like tests are passing.

ujson==1.35 and
python-gnupg==0.4.1
Are on notice! can't find them anywhere as well.


Set Settings.product.transformation to true

..because v2v is now enabled by default.
This is a gaprindashvili version of https://github.com/ManageIQ/manageiq/pull/17711, see discussion there :).

Cc @gmcculloug


Build Error: could not create 'match/keywordprocessor.cpython-37m-x86_64-linux-gnu.so': No such file or directory

error: could not create 'match/keywordprocessor.cpython-37m-x86_64-linux-gnu.so': No such file or directory
Steps to reproduce:

conda create --name ucphrase python=3.7
wget https://www.dropbox.com/s/1bv7dnjawykjsji/data.zip?dl=0 -O data.zip
unzip -n data.zip
bash build.sh

Figured it out. Needed to use python 3.6.
Leaving the issue here so that others don't face the same issue.


For a 0.18.16 release

Codecov Report

Merging #604 (a202a68) into master (d0726dd) will increase coverage by 0.72%.
The diff coverage is 57.14%.

@@            Coverage Diff             @@
##           master     #604      +/-   ##
==========================================
+ Coverage   85.15%   85.87%   +0.72%
==========================================
  Files          37       37
  Lines        3192     3192
==========================================
+ Hits         2718     2741      +23
+ Misses        474      451      -23

Impacted Files
Coverage Δ

src/data/data.jl
93.68% <ø> (ø)

src/measures/loss_functions_interface.jl
75.43% <57.14%> (+2.52%)
:arrow_up:

src/measures/continuous.jl
97.70% <0.00%> (-2.30%)
:arrow_down:

src/measures/finite.jl
96.30% <0.00%> (-0.30%)
:arrow_down:

src/utilities.jl
78.21% <0.00%> (-0.22%)
:arrow_down:

src/composition/learning_networks/nodes.jl
69.93% <0.00%> (-0.21%)
:arrow_down:

src/univariate_finite/methods.jl
86.25% <0.00%> (-0.11%)
:arrow_down:

src/data/datasets_synthetic.jl
95.37% <0.00%> (-0.05%)
:arrow_down:

src/univariate_finite/types.jl
91.50% <0.00%> (+0.85%)
:arrow_up:

... and 2 more

Continue to review full report at Codecov.

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


Question regarding the LevelSetLoss implementation

Thanks for your awesome work.
I found in the implementation of LevelSetLoss that the term of length regularization is not calculated. Would you like to share the reason for it?
Besides that, what is the motivation that only calculates the loss for the positive samples in Eqn.(11) in the paper?
Best

@QianyiWu  Thanks.
In the ablation study, the length regularization of Level set loss is adopted, which is beneficial for the performance (especially when the performance is low).  In the main results, I found that the length regularization can bring the 0.2~0.4mAP drop.  So,  I commented the calculation for the better final results.
About  only calculting the the loss for the positive samples,  this follows the original SOLOv2 network.  The positive samples comes from the label(target) assignment considering the GT box location and mask prediction.  Without this scheme, the network convergence is very slow, and  it can't even work.


Fix/rpcs

Re-added acquired_resource field in dictionary returned by the places rpc

I have now changed the returned data to contain the acquired resources as list of resource keys. Sadly it is not possible at the moment to also return the user who holds the place lock.


Not effective nav bar in home page

i would like to make the nav bar more effective by designing it properly!!
please assign me with the required labels as per gssoc'24 extended program

I have seen many navbar issues, and i have mentioned it before for contributors to not create issues regarding navbar and there should be no more navbar issues, so i have closed this issue, if you have any queries, you can join discord server for further updates and communication.
we are not looking for navbar updates for now, as it was done and changes many times before, please understand.
and already it is in updation, so please understand.


Removed unused members settings module

This module is marked for deletion with a todo comment.  It also popped up as least covered place in the codebase, because it was never used.

@SimonBackx as you've added the original "todo" for removal of this module, can you please double check if I'm not missing anything with this change? The tests did pass locally, but I do not have enough context for members to know if something else to consider 🙃

Thanks Simon 🙌


TLIndexPathUpdates should optionally suppress scrolling when inserting rows

If the user is scrolled down 20 messages on a table, and a new message comes in, calling [TLIndexPathUpdates -performBatchUpdatesOnTableView: withRowAnimation:] will insert rows at the top, pushing the other cells down.  It can be confusing for a user to have the table scroll all of a sudden, so an option should be provided to suppress this behavior during row insertion.
This Stack Overflow answer describes one method for inserting rows at the top, and manually adjusting the table's contentOffset so the user never notices anything.

I noticed that this works for UITableView:
override func controller(controller: TLIndexPathController, didUpdateDataModel updates: TLIndexPathUpdates) {

    UIView.performWithoutAnimation {
        if self.isViewLoaded() && self.view.window != nil {
            self.tableView.beginUpdates()
            updates.performBatchUpdatesOnTableView(self.tableView, withRowAnimation: .None)
            self.tableView.contentOffset = /* your magic here */
            self.tableView.endUpdates()
        } else {
            self.tableView.reloadData()
        }
    }
}

I am fixing the contentOffset, by looking for a referenceItem that is present in the old and the new TLIndexPathDataModel and also is currently visible, determining the relative position from the top of the UITableView to the referenceItem and then basically scrolling to the new item after performBatchUpdates.
It only works with UIView.performWithoutAnimation which does not affect the row animations (row animation fade in etc. seems to work just fine).
Should I create a demo project?

@stefreak I've long since abandoned the project I was working on in 2014, so I would love to look at a sample project.

@getaaron KeepScrollPositionDemo.zip
Somehow the row animation doesn't work in the demo, but I'm pretty sure it did work in another project.
I am not sure if this approach could/should be ported to TLTableViewController / TLCollectionViewController though....

The row animation seems to work when the row is updated in my other project, so yeah it only works correctly without any animations. but it's still better than reloadData because the cells that did not change are preserved


RYA-362 Improved UX for case where create-pcj is called without args.

Description
Improved UX for case where create-pcj is called without args.
Also updated the order of operations so we fail for insufficient arguments before we prompt for the SPARQL query.
Tests
Added a test case to RyaAdminCommandsTest
Links
Jira RYA-362
Checklist

[ ] Code Review
[X] Squash Commits

People To Review
@DLotts
@meiercaleb
@amihalik
@pujav65

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/incubator-rya-master-with-optionals-pull-requests/474/


Dynamic custom icons

Is your feature request related to a problem? Please describe.

No
Describe the solution you'd like

Display custom icons which depend on the state. E.g. when I use the card to show RSSI signal strength I'd like to use the following icons:

Additional context

#401


Update README.md

fixes on overlays functions

Thank you for the catch, @atomotic - I should take more care to keep the documentation up to date.


Clarify use of "Community Version" of NGINX Controller for Private Cloud

TW-1578

Looks good to me.

Thanks, I'll merge it.


Simple execution error on Win32

The intent of program
The intent of the program is to crawl a top level directory and look at each file, using libmagic determine if that file is a data file.  If so then collect some metrics like create a histogram of the bytes of the file and run a variance against that result.
The issue
On Windows the program just hangs or crashes.  Using sysinternals nothing appears to be wrong as far as commands issued through Windows.  It should be said that this works flawlessly on Linux.
So I fire up cabal and check the function that does the work against a single file, works as intended.
However, if i do something like this in GHCI

ghci> let a =  scanFileSystem "C:\Users\Montez\Documents\Code"
ghci> :t a
a :: Pipes.ListT (Pipes.Safe.SafeT IO) Info
ghci> import Pipes
ghci> import Pipes.Safe
ghci> runSafeT $ runListT a >> mzero
:27:26:
No instance for (MonadPlus (SafeT IO))
arising from a use of mzero'    In the second argument of (>>)', namely mzero'    In the second argument of ($)', namely `runListT a >> mzero'
In the expression: runSafeT $ runListT a >> mzero
ghci> runSafeT $ runListT a

(This hangs and never does any work, similarly to how the application operates)
Library versions
pipes-4.1.4
pipes-safe-2.2.0
dirstream-1.0.0
Maybe relevant code gist?

I haven't had a chance to look at this yet, but I wanted to point you to my newly released turtle library, which provides a similar functionality (streaming over directory trees): https://hackage.haskell.org/package/turtle-1.0.1/docs/Turtle-Tutorial.html
I'm mentioning this for two reasons:

turtle has a simpler implementation and API and may work for you instead of dirstream.

I would like you to test your program against turtle if you could.  The reason why is that I want to see if it's an issue specific to pipes-safe or not

Awesome, will wire it up tomorrow.
On Feb 3, 2015 5:04 PM, "Gabriel Gonzalez"<EMAIL_ADDRESS>wrote:

I haven't had a chance to look at this yet, but I wanted to point you to
my newly released turtle library, which provides a similar functionality
(streaming over directory trees):
https://hackage.haskell.org/package/turtle-1.0.1/docs/Turtle-Tutorial.html
I'm mentioning this for two reasons:

turtle has a simpler implementation and API and may work for you
instead of dirstream.
I would like you to test your program against turtle if you could. The
reason why is that I want to see if it's an issue specific to
pipes-safe or not
—
Reply to this email directly or view it on GitHub
https://github.com/Gabriel439/Haskell-Pipes-Safe-Library/issues/17#issuecomment-72755462
.


change family selector to drop-down

Unlike languages, the number of families is finite.

... hm, this will break the tool because of #88. So perhaps it needs to wait a while.

resolved with 059b363d5dd72d8b330538afbe38415314ed01e5


Reformatted README.txt to README.md as per markdown syntax

Markdown syntax allows the README to render visually in Github which is more comfortable to read for readers, than a plain text README file. Markdown is plain text as well, but a format that Github understands and renders visually.
In this PR I have added README.md which is markdown formatted version of README.txt. This is how it looks: link
I have not deleted README.txt yet. That can be deleted once this PR is merged.

This file is intended for consumption by many users, including BITS itself (which references README.txt in multiple places), and by people looking at the disk image.  I don't have any objection to markdown formatting, but changing the filename will break too many things, including the ability for Windows users to easily open the file.
(And without changing the filename, I don't think the markdown formatting will have any effect.)

@joshtriplett - Didn't know that. So I'll close this PR then.


Ask user to validate mnemonic when creating a new identity

When the user creates a new identity using a fifteen-word mnemonic, they should be asked in a subsequent screen to type in some set of the words to validate they backed up their identity correctly.

Should have been opened in the Conseil repo.


Provide a schema for the experimental run

Every run should result in a bunch of standard results (as @yaseminbridges has already implemented), and a single results.yml file that contains metadata about the run. That metadata will be defined in a linkml schema moving forward, but for now, lets make sure the following is outputted with every run:
results.yml
tool: exomiser
tool_version: 13.01
config: default
run_timestamp:<PHONE_NUMBER>968
corpus: corpus1-default

The file con contain other things as well, but I would make these things required.

[ ] Provide provisional output for existing runners (@yaseminbridges)
[ ] Create schema (@souzadevinicius) - I will introduce you to the dark art

What would be going into the config here?

An id that represents the config, like exomiser-13.01-semsim1 or exomiser-13.01-semsim1 (basically reflecting the config dir structure as described on the slides). Could be also a path like exomiser-13.01/semsim1 - up to you. The point is you should be able to understand which config lead to this result.

Understood, thank you!

@matentzn can I just double check, this is something that should be implemented in PhEval and not in all the runner implementations separately?

The basic schema should be in pheval, but there should a special field in the schema like "tool_specific_configuration_options" that can be overwritten to the tool developers hearts content..


DCOS-56383: center aligns alert icon for inactive nodes

Testing
Change line 101 of plugins/nodes/src/js/components/NodesGridDials.js to if (!node.isActive()) {
Confirm the yield icon is center-aligned.
Trade-offs

Dependencies

Screenshots
Before:

After:

:tada: This PR is included in version 2.123.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Appimage file support

Please make the release files as .appimage, it's more convenient for launch

There was an attempt to do this, sadly it is a non trivial issue, but I will try to build it as appimage again(there are github actions to facilitate that).

Flatpak installs a lot of heavy (above 300 mb) packages that I already have in a system - how to exclude?

I do not have the capacity to set up the OBS to build the RPMs, same as with setting up AppImage build, feel free to take it on, the build from source itself is fairly trivial, you just need all the dependencies


lld v13.0.1 (again)

Rebased #33 to keep CI results for reference if we still need to fix something.
Closes #33
Closes #32
Closes #31
Closes #30
Closes #29
Closes https://github.com/conda-forge/staged-recipes/pull/18004

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

OK, this is still failing despite the llvm build having LLVM_ENABLE_LIBXAR=OFF as well. I went and raised an issue: https://github.com/llvm/llvm-project/issues/54001

Woohoo, a passing osx build 🥳
Now that the setting has taken effect in llvmdev, let's see if we can avoid it here. Together with the also-finally-passing https://github.com/conda-forge/libcxx-feedstock/pull/83, this should be bringing up the last relevant bits and pieces for LLVM 13 in conda-forge 🚀

Ping @conda-forge/lld
This is green and ready :)

Gentle ping @isuruf

OK cool, looks like this worked! :)
Let me know if you want me to clean up the commit message / history, otherwise this is ready.

Thanks


create 0523-continuous-subarray-sum.cpp

File(s) Modified: 0523-continuous-subarray-sum.cpp
Language(s) Used: cpp
Submission URL: https://leetcode.com/problems/continuous-subarray-sum/submissions/877908957/

Please fix the Submission URL.


Suggest 'Search/Filter' feature be moved to a more globalized function for reuse

I can explain my thinking further when we connect after the chandelier update.
This is not essential, but I think may be an improvement.

We can see about adding it to the core package.

No longer relevant.


adding D to camera info

Please ensure the above guidelines for contributing are met.
Fixes Issue: #
Changes proposed in this pull request:
-support of SR300 model
-adding D for camera_info

@nlyubova Thank you for submitting pull request.
Currently we are working on redesigning the nodelet. We are planning to add SR300 support once redesigning is done.
At this point we can accept changes for adding D in camera_info. Can you please create separate pull request just for those changes?

@rjingar ok, I've created a new pull request with a correct subject

+1 for quick support SR300.


BUG: script currently not working on Linux

When running the script on Ubuntu 20.04 (after linking to it using npm link), the following error occurs after gathering all the information:
node:internal/process/promises:279
            triggerUncaughtException(err, true /* fromPromise */);
            ^

[Error: ENOENT: no such file or directory, open '/tmp/__create_geofs_plugin__3luZ5J/src/index.js'] {
  errno: -2,
  code: 'ENOENT',
  syscall: 'open',
  path: '/tmp/__create_geofs_plugin__3luZ5J/src/index.js'
}

This seems to be an error that occurs when copying nested files to the temporary directory. This isn't reproduced on Windows 10 (21H2), so this might be an issue with cross-platform handling of filesystem errors.

This seems to have been always broken (git bisect was bad all the way back to 7c3b01cdd453778638a3e9c5bf347ded72c2731b). The issue did end up being with creating the directories that files were nested in. fsExtra.outputFile seems to fix this on Ubuntu 20.04.


Add public creator for ReadResult

Add a public static create method to ReadResult so that clients can synthesize their own errors and/or use them in tests.

We can close this one, right?


FR: Running as non-root via env variable

It would be good option to have to allow us to specify an env variable to skip the permission updating that requires root so that the container could be run as non-root instead on eg Kubernetes.

Hey @samip5, I appreciate the suggestion. I hate the way that permissions are currently handled. For context, the current iteration came around due to a very high number of users complaining about perms on non-standard deployments or within restricted environments (e.g. Synology devices). Unfortunately, I don't have a restricted environment that I can openly test permissions changes in, which is why I've been hesitant to make any changes given the large number of people that rely on this repo.
I do like the idea of using an environment variable to toggle this behaviour. I'm currently ill, but I'll think the implementation details over soon

I do like the idea of using an environment variable to toggle this behaviour. I'm currently ill, but I'll think the implementation details over soon

I think some apps have an ENV var called RUNASUID0 or something like that.

preferable in Kubernetes this should be done with the securityContext, would appreciate it if there could be a way to run this container rootless.
If any help is needed for e.g. testing the container or any other related non root issue, you can ping me, would love to see this feature.

@98jan PRs are always welcome! I've been ill for the better part of the last week, and so I haven't had time to properly delve into this. I think the quick and janky way to handle this would be to add an environment variable called ROOTLESS or something like that, which is then checked whenever running superuser commands.
I'd rather resolve the core issue, though again, I do not have the restrictive hardware that others do to test this. This image was originally rootless, but was ultimately changed to its current state due to issue after issue about environments which I couldn't personally test within. Removing the root dependency is very easy from a code perspective, given that it's just a couple of bash scripts :D

Get well soon.
Thanks for your fast reply.
I will try to work on it and see if I can get it working for both options (rootless and non rootless where needed, there are sometimes needed)

created an PR #222
Open for any questions or ideas on how to make it better.
Tested the changes with my self-made helm chart for satisfactory: https://github.com/98jan/helm-charts/tree/main/charts/satisfactory

@samip5 thanks to @98jan's hard work, this is now achievable by setting ROOTLESS=true :)


[FEATURE REQ] Add new overload to AzureKeyCredential Policy for value prefix

We have services that requires to send:
Authorization: SharedAccessKey my-key-here

After discussion with the Azure Core alias, the proposition is to add a new overload (or option value, whatever works best for your language) to the already existing AzureKeyCredentialPolicy:
Example:

AzureKeyCredentialPolicy(“MyHeaderKey”) => MyHeaderKey: key-from-credential
AzureKeyCredentialPolicy(“HeaderName”, “Prefix”) => HeaderName: Prefix key-from-credential
AzureKeyCredentialPolicy(“Authorization”, “SharedAccessKey”) => Authorization: SharedAccessKey key-from-credential

I'm targeting the June release cycle (release early June) at worst, in order to plug this feature in our codegen, as we have services writing customization to workaround this right now.
Thanks!
See also:

[FEATURE REQ] Add new overload to AzureKeyCredential Policy for value prefix · Issue #35879 · Azure/azure-sdk-for-net (github.com)
[FEATURE REQ] Add new overload to AzureKeyCredential Policy for value prefix · Issue #34697 · Azure/azure-sdk-for-java (github.com)
[FEATURE REQ] Add new overload to AzureKeyCredential Policy for value prefix · Issue #30120 · Azure/azure-sdk-for-python (github.com)
[FEATURE REQ] Add new overload to AzureKeyCredential Policy for value prefix · Issue #25683 · Azure/azure-sdk-for-js (github.com)

We don't currently have a centralized AzureKeyCredentialPolicy at the moment. Instead each package declares their own version that handles setting the appropriate header, e.g.: https://github.com/Azure/azure-sdk-for-js/blob/2e1b431233fd70e322ac3b395c41527858b11d19/sdk/cognitivelanguage/ai-language-conversations/src/azureKeyCredentialPolicy.ts#LL25C1-L25C1
@mpodwysocki @willmtemple @joheredi is it worth creating a common one for DPG codegen purposes? Or do we have a different way to handle this?

I guess I don't see why the common policy is necessary if each package needs to describe the header and optional prefix it will use. I'd think we can DPG-generate a per-package policy about as easily as we can DPG-generate an instantiation of a common policy.

@xirzec if your story so far with codegen is to generate the policy everywhere, this issue is not about pushing you to change this pattern and make it to azure-core. It means you already have code that generates policy anyway, so I'm fine if you enable the behavior through that.

@xirzec if your story so far with codegen is to generate the policy everywhere, this issue is not about pushing you to change this pattern and make it to azure-core. It means you already have code that generates policy anyway, so I'm fine if you enable the behavior through that.

Thanks for clarifying. I'm going to assert for now that we just "handle it with codegen" and there's no new feature work required, but I'll let @joheredi confirm before we close this issue. 😄

Closing based on our discussion, will open soon an issue for codegen support. Thanks!

@xirzec My understanding is that we have a centralized api-key policy here which will be added when we build our RLC client here.

We don't currently have a centralized AzureKeyCredentialPolicy at the moment. Instead each package declares their own version that handles setting the appropriate header, e.g.: https://github.com/Azure/azure-sdk-for-js/blob/2e1b431233fd70e322ac3b395c41527858b11d19/sdk/cognitivelanguage/ai-language-conversations/src/azureKeyCredentialPolicy.ts#LL25C1-L25C1

Interesting! I didn't remember we had this policy for RLC. (It seems I spend too much time in core-client)


api: Add "users/{user_id}/status" endpoint to the API documentation.

This PR adds "users/{user_id}/status" endpoint to the API documentation which fetches the status of a specified user.
It completes the work started in #19498.

**Screenshots and screen captures:**

Self-review checklist

[x] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[x] Explains differences from previous plans (e.g., issue description).
[x] Highlights technical choices and bugs encountered.
[x] Calls out remaining decisions and concerns.
[x] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[x] Each commit is a coherent idea.
[x] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[x] Visual appearance of the changes.
[x] Responsiveness and internationalization.
[x] Strings and tooltips.
[x] End-to-end functionality of buttons, interactions and flows.
[x] Corner cases, error conditions, and easily imagined bugs.

@laurynmm I have addressed the feedback above.

One question I have is if we want to do anything about bot users as they would never have a status set. I haven't tested, but I assume that just returns an empty dict, which is probably fine...

@laurynmm I tested that and it seems like for bots, "No user found" error is raised.
I have addressed the recent feedbacks.

@zulipbot add "integration review"

Made a couple documentation tweaks, mainly to clarify the prior state for how to get these data. I also squashed the API documentation commit with the change adding the endpoint itself; it's generally an atomic change to add an API endpoint and document the new API endpoint. I also edited the PR description to fix https://github.com/zulip/zulip/pull/30059#discussion_r1610811208 and the commit message to fix #19079, which this new endpoint appears to resolve. Thanks @Vector73, @MSurfer20, and @laurynmm!
diff --git a/api_docs/changelog.md b/api_docs/changelog.md
index 9ee17a06ae..0955ccafe7 100644
--- a/api_docs/changelog.md
+++ b/api_docs/changelog.md
@@ -22,8 +22,8 @@ format used by the Zulip server that they are interacting with.

 **Feature level 262**:

-* [`GET /users/{user_id}/status`](/api/get-user-status): Added
-  a new endpoint to get a specified user's currently set
+* [`GET /users/{user_id}/status`](/api/get-user-status): Added a new
+  endpoint to fetch an individual user's currently set
   [status](/help/status-and-availability).

 **Feature level 261**
diff --git a/zerver/openapi/zulip.yaml b/zerver/openapi/zulip.yaml
index 2f636af455..619848b04b 100644
--- a/zerver/openapi/zulip.yaml
+++ b/zerver/openapi/zulip.yaml
@@ -8425,7 +8425,9 @@ paths:
         Get the [status](/help/status-and-availability) currently set by a
         user in the organization.

-        **Changes**: New in Zulip 9.0 (feature level 262).
+        **Changes**: New in Zulip 9.0 (feature level 262). Previously,
+        user statuses could only be fetched via the [`POST
+        /register`](/api/register-queue) endpoint.
       parameters:
         - $ref: "#/components/parameters/UserId"
       responses:


Run cargo-careful in CI

https://github.com/RalfJung/cargo-careful
This allows to catch UB like https://github.com/crossbeam-rs/crossbeam/pull/940.

bors r+

bors r+


Possible synergy between the KubeStellar and the Flux CD community

I recently had some casual conversations with the flux community. We then scheduled a formal meeting with the maintainer(s) to discuss the possibility to seek for the synergy between Flux and KubeStellar.

@waltforme what are the next steps for this effort?

can you reach out to kingdon to understand/document what the 'repo structure' challenge is?

I reached out to flux maintainers and tried to understand the problem. I didn't get a clear answer to the problem though. If my own understanding is correct, the problem is the growing complexity of the git repo structure when # of clusters increases. Unfortunately I don't think today KS has a solution to that. Let's close this issue for now. Will keep an eye on any possible engagement in the future.


Add exclude_tag functionality

Looks like the tags functionality wasn't also working as expected, so added now a test to make it bullet proof.

@adamrtalbot aren't we going for tag based filtering ? and can you be more specific and point out which part of the code you think is broken ?

@sateeshperi
Inclusion/exclusion by tag wan't working on this branch. Some of the issues were fixed in the other PR: https://github.com/adamrtalbot/detect-nf-test-changes/pull/15

ah both the PR's are named the same :shakefist:


Sky Q Black

Hi,
I tried to acquire the sky q remote control.
But the code is not acquired.
Where am I wrong? I acquired other remote controls without problems.

I'm not familiar with Sky Q Black, which protocol used for remote? is it rf? ir? or maybe something else?

HI,
ops.
Sorry, I forgot to close the issue.
I solved it by setting the Sky remote to IR instead of BT.


Callback for when audio recording starts

Currently, when we start recording audio there is no callback function which allows us to retrieve the time, or do some other operations.
Having this callback is an essential component to the audio recording operation.
Thanks, Elijah

not sure i understand. the recording should start right away once you call the api

If a user joins muted it doesn't start recording till the first audio frame. There's an event for that, you can start recording on the first occurrence

Thanks guys :)