When the client updates the payout address, it should also be updated server side.

We have no server api call to update the payout address!

We want to think of a payout address with 10k SJCX in it as an identity. For simplicity and for that reason, users will not be able to change their payout address. They will have to delete their identity and start over, but they can still use the same shards.
This would add a ton of complexity to the code, and open up the possibility of some malicious behavior because the payout addresses would be mutable.


I have a hap_ctg.fa file with 4.5G, used juicer I got 52.31G merged_nodups_txt,then I used 3D-DNA run-asm-pipeline.sh, I only got 55.72M final.hic file

I have a hap_ctg.fa file with 4.5G, used juicer I got 52.31G merged_nodups_txt,then I used 3D-DNA run-asm-pipeline.sh: 3d-dna/run-asm-pipeline.sh -r 0  hap1234.fa merged_nodups.txt, then, I only got 55.72M final.hic file, in the final.fasta file, has some words: The length of fasta does not match that suggested by the cprops file. Exiting!
would you tell me what's wrong, thank you, wish you best.

This is not sufficient description to offer any advice I am afraid. I would suggested providing a more thorough report. I would also suggest checking that your fasta does not have any hidden characters (return carriage e.g.). 3d-dna does not do well with those. -Olga

On Apr 12, 2023, at 12:46 AM, wyl1219 @.***> wrote:
I have a hap_ctg.fa file with 4.5G, used juicer I got 52.31G merged_nodups_txt,then I used 3D-DNA run-asm-pipeline.sh: 3d-dna/run-asm-pipeline.sh -r 0 hap1234.fa merged_nodups.txt, then, I only got 55.72M final.hic file, in the final.fasta file, has some words: The length of fasta does not match that suggested by the cprops file. Exiting!
would you tell me what's wrong, thank you, wish you best.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.Message ID: @.***>

ok ,I will try, thank you.

Hello, when I used run-assembly-visualizer.sh -q 0 kl.assembly kl_merged_nodups.txt, I got kl.hic, As shown below

Could you tell me what might be wrong? thanks.


prow.sh: allow shell commands in CSI_PROW_SANITY_POD

This is required for distributed provisioning where the Pod name is
dynamically generated as part of the DaemonSet.

/release-note-none

/lgtm


update node-scrcpy-client as a part of @u4/adbkit

Hi,
I forked adbkit more than one year ago, applying many significant changes.
Recently I integrated your code into it and tried to update it.
see https://www.npmjs.com/package/@u4/adbkit

The code had been refactored and upgrade to use scrcpy 1.20

The code had been refactored and upgrade to use scrcpy 1.24


Python scripts with comments inside triple quotes are not parsed correctly

Possibly not a very frequent pattern... but that's a pattern found in nbrmd tests, like at test_read_simple_python.py

Fixed in v0.4.3


New error with 'az ml model deploy' in 1.0.81

Extension name (the extension in question)
"extensionType": "whl",
"name": "azure-cli-ml",
"version": "1.0.81"

Description of issue (in as much detail as possible)
When running the following model deployment command, with azure-cli-ml 1.0.81 this is now throwing an error. This did not throw an error in azure-cli-ml 1.0.79.
az ml model deploy -n quote-clf --inference-config-file src/inferconfig.json --auth-enabled true --deploy-config-file src/deployconfig_stab.json --compute-target aks-aml-stab -m quote_clf:3 -m quote_tfidf:3 --workspace-name aml-stab --resource-group aml-stab --overwrite -v
Error:
Creating image
Running.............................................................................
Succeeded
ERROR: {'Azure-cli-ml Version': '1.0.81', 'Error': AttributeError("'bool' object has no attribute 'name'",)}
Image creation operation finished for image quote-clf:17, operation "Succeeded"
##[error]Script failed with error: Error: The process '/bin/bash' failed with exit code 1

src/inferconfig.json:
{
    "condaFile": "environment.yaml",
    "enableGpu": false,
    "entryScript": "entry_script.py",
    "extraDockerfileSteps": "extra_dockerfile_steps",
    "runtime": "python",
    "sourceDirectory": "src"
}

src/deployconfig_stab.json:
{
    "autoScaler": {
        "autoscaleEnabled": true,
        "minReplicas": 2,
        "maxReplicas": 4,
        "refreshPeriodInSeconds": 1,
        "targetUtilization": 70
    },
    "authEnabled": true,
    "computeType": "aks",
    "containerResourceRequirements": {
        "cpu": 1,
        "memoryInGB": 1
    },
    "enableAppInsights": true,
    "maxConcurrentRequestsPerContainer": 3,
    "maxQueueWaitMs": 5000,
    "scoringTimeoutMs": 120000
}

src/environment.yaml:
name: quote
channels:
  - defaults
  - conda-forge
dependencies:
  - pip=19.2.2
  - scikit-learn=0.20.3
  - spacy=2.1.7
  - pip:
      - azureml-defaults>=1.0.60
      - https://github.com/explosion/spacy-models/releases/download/en_core_web_sm-2.1.0/en_core_web_sm-2.1.0.tar.gz

@adiun: I ran into the same error. As a intermediate fix use the yml format for inference config - it worked for me. Attaching template.

@rsethur Thanks!
@gogowings Thanks for the updates and the tip about using extraDockerFileSteps. I received this guidance separately as well re. deployment performance so will definitely look into switching to that.


Rust - Recursively ignore target directory

Reasons for making this change:
Repositories with Rust code often have multiple modules in them. Those modules have a target folder themselves and can be arbitrarily nested.
For example:
.
├── jsonrpc
│   ├── src
│   ├── target
│   ├── v1
│   │   ├── src
│   │   └── target
│   └── v2
│       ├── src
│       └── target
├── src
│   └── bin
└── target

This rules makes sure that all of these target directories are ignored.
Links to documentation supporting these rule changes:
This is just the way cargo works.

This was partly a misunderstanding on my part. If the workspace is correctly configured, cargo only uses 1 output directory.


Portainer swarm deployment script in Raspberry Pi - Not loading UI - Network issue?

 <-- Click to watch a video
Question:
Why is the Portainer UI not loading when I attempt to install in swarm mode using the script?
Seems like the yml file has something misconfigured in the network.
Background:
I have installed Portainer in standalone mode without issues in a Raspberry Pi 4B (buster/debian) and Ubuntu 18.04.
Whenever I try to set it up in swarm mode, using the yaml file provided in the instructions, it fails to load the User Interface and keeps on building a new container because I can't setup the admin password.

My machines have IPV6 disabled (since I read something about it causing issues in the forums) but it doesn't help.
I also have the firewall disabled.
Troubleshooting:
Here's the output from the logs of each failed container so far.
Agent - (Running)

Portainer 1 - (Exited 1)

Portainer 2 - (Exited 1)

Portainer 3 - (Exited 1)

As you can see the container is working but I can't access it from another machine.  If I try to access it through the terminal in that machine, I have a response.

This leads me to believe that there is a network configuration problem in the networks.  But it seems to have everything that it needs.

Hi @PREngineer ,
I see on your portainer log an administrator account not created cause you can't access your portainer and I often see portianer stuck when something plays up in cache. You can try clearing cache or trying an incognito window or even trying a different browser.

Hi @PREngineer ,
I see on your portainer log an administrator account not created cause you can't access your portainer and I often see portianer stuck when something plays up in cache. You can try clearing cache or trying an incognito window or even trying a different browser.

I thought about that too.  I made sure to do a full refresh, incognito, and tried another browser.  The problem is not my computer.  It seems to be the connection to the Portainer container.  Like I said at the top.  I have no problem when it runs standalone.  :(

@PREngineer try to telnet your portainer-ip with port 9000 from outside your docker instance. If it succeed there is something wrong on your browser i think.

I'm 200% sure that my browser is not the problem.  It is somewhere in the stack.

The interface is plainly not accessible.

the simplest way to check your docker swarm instances can be accessed from your network is create nginx stack and access it with your browser. This is example my nginx-test.yml
root@rpi4:/home/ubuntu# cat nginx-test.yml
---
version: "3.4"

services:
  web:
    image: nginx:alpine
    networks:
      - app
    ports:
      - "8080:80"

networks:
  app:
    driver: overlay

run the stack
root@rpi4:/home/ubuntu# docker stack deploy --compose-file nginx-test.yml nginx-test
then access it from browser http://your-docker-ip:8080 , if it not succeed there is something wrong with your docker overlay network.

I have added a video where the problem can be seen clearly to the top of the thread.

already seen that video, I think something wrong with your overlay network. You can try deploy nginx stack with overlay network and see if u can access that.

Make sure you have the required ports open on your firewall… often this is caused by the firewall blocking access to the overlay network ports..
firewall-cmd --add-port=2377/tcp --permanent
firewall-cmd --add-port=7946/tcp --permanent
firewall-cmd --add-port=7946/udp --permanent
firewall-cmd --add-port=4789/udp --permanent
firewall-cmd --reload
the important one is 4789 UDP as that’s the overlay network (VXLAN) port.. but the others are needed for correct SWARM operations.

Thank you very much for your responses.
Adding this to help any other person who may possibly be looking for answers in the future.
Solution #1: Use  'sudo apt-get install -y docker docker-compose'
I decided to install the docker and docker-compose packages using apt-get instead of trying to retrieve the latest version from Docker's website even though the version is older.  That immediately solved the problem for me.  Apparently their package has some networking difference from the one that you pull out of the official website in the latest version.

To do:
I will try again with the latest version but opening the ports per ncresswell's reply and will report after.

I've been plagued by this in my own environment, nice to know its Docker's fault :+1:

@PREngineer Do you know if this issue is being tracked anywhere? It still appears to be a problem as of testing it today.

I know this is an old thread , but would like to add my experience here in case someone runs into the issues.
In my case, my problem was that when I installed Docker on my computer, I was connected to the internet via my company VPN? As a result of this during Docker install, but the docker network interfaces  were created and attached to the corporate network with a fix IP address. 17.x.x.x
later I installed Portainer (while off the VPN), and ran into the issue being discussed here.
I was not able to connect because Portainer through docker was running on a network different to the local machine (192.168.1.x).
To fix this issue, I had to disable and re-enable all the network interface on the laptop and restart docker service.
I issued the followign command,
nmcli networking off
nmcli networking on
sudo systemctl restart docker.service

I have been trying to get docker swarm working on Raspberry Pi 5s now for awhile.
I've tried

switching to Ubuntu
use systemd-networkd,
install Debian repo version of docker (20.10)

The same commands works out of the box on debian x86_64 VMs no problem.


[FEATURE] Reduce unit test duration

Description
Current unit tests in notebooks are lengthy, need to investigate ways to reduce complexity or change parameters to speed up testing of notebooks.
Here are the current test durations (for any test longer than 1s)  for different configurations in order to prioritize efforts.
not vw and not spark and not gpu |
-- | --
Duration | Test
155.85s | tests/unit/test_notebooks_python.py::test_rlrmc_quickstart_runs
41.76s | tests/unit/test_notebooks_python.py::test_lightgbm
38.63s | tests/unit/test_notebooks_python.py::test_surprise_deep_dive_runs
23.61s | tests/unit/test_notebooks_python.py::test_sar_deep_dive_runs
16.61s | tests/unit/test_notebooks_python.py::test_sar_single_node_runs
16.60s | tests/unit/test_notebooks_python.py::test_baseline_deep_dive_runs
3.51s | tests/unit/test_notebook_utils.py::test_is_jupyter
3.49s | tests/unit/test_notebooks_python.py::test_template_runs
2.00s | tests/unit/test_timer.py::test_timer
not vw and not spark and gpu |
-- | --
Duration | Test |
155.18s | tests/unit/test_notebooks_gpu.py::test_wide_deep
125.07s | tests/unit/test_notebooks_gpu.py::test_xdeepfm
107.35s | tests/unit/test_notebooks_gpu.py::test_ncf_deep_dive
39.24s | tests/unit/test_notebooks_gpu.py::test_ncf
35.85s | tests/unit/test_notebooks_gpu.py::test_fastai
29.56s | tests/unit/test_deeprec_model.py::test_dkn_component_definition
12.80s | tests/unit/test_ncf_singlenode.py::test_fit[NeuMF]
10.34s | tests/unit/test_ncf_singlenode.py::test_predict[NeuMF]
9.77s | tests/unit/test_tf_utils.py::test_evaluation_log_hook
8.57s | tests/unit/test_ncf_singlenode.py::test_fit[MLP]
7.48s | tests/unit/test_deeprec_model.py::test_xdeepfm_component_definition
7.19s | tests/unit/test_wide_deep_utils.py::test_build_model
6.96s | tests/unit/test_ncf_singlenode.py::test_fit[GMF]
6.88s | tests/unit/test_ncf_singlenode.py::test_predict[GMF]
6.78s | tests/unit/test_ncf_singlenode.py::test_predict[MLP]
1.93s | tests/unit/test_ncf_singlenode.py::test_neumf_save_load[4-8]
1.77s | tests/unit/test_ncf_singlenode.py::test_neumf_save_load[5-5]
1.33s | tests/unit/test_deeprec_utils.py::test_DKN_iterator
1.33s | tests/unit/test_ncf_singlenode.py::test_regular_save_load[NeuMF-5-5]
1.11s | tests/unit/test_ncf_singlenode.py::test_regular_save_load[MLP-5-5]
1.07s | tests/unit/test_ncf_singlenode.py::test_regular_save_load[GMF-5-5]
not vw and spark and not gpu |
-- | --
Duration | Test
145.73s | tests/unit/test_notebooks_pyspark.py::test_spark_tuning
134.66s | tests/unit/test_notebooks_pyspark.py::test_als_deep_dive_runs
88.78s | tests/unit/test_notebooks_pyspark.py::test_evaluation_runs
60.56s | tests/unit/test_notebooks_pyspark.py::test_mmlspark_lightgbm_criteo_runs
51.75s | tests/unit/test_notebooks_pyspark.py::test_als_pyspark_runs
34.85s | tests/unit/test_notebooks_pyspark.py::test_data_split_runs
3.12s | tests/unit/test_spark_splitter.py::test_chrono_splitter
2.71s | tests/unit/test_spark_evaluation.py::test_spark_python_match
2.53s | tests/unit/test_spark_evaluation.py::test_init_spark
2.13s | tests/unit/test_spark_evaluation.py::test_spark_precision
1.98s | tests/unit/test_spark_evaluation.py::test_init_spark_rating_eval
1.71s | tests/unit/test_spark_splitter.py::test_timestamp_splitter
1.68s | tests/unit/test_spark_splitter.py::test_stratified_splitter
1.51s | tests/unit/test_spark_evaluation.py::test_spark_exp_var
1.49s | tests/unit/test_spark_evaluation.py::test_spark_recall
1.32s | tests/unit/test_spark_evaluation.py::test_spark_rmse
1.28s | tests/unit/test_spark_evaluation.py::test_init_spark_rating_eval
1.21s | tests/unit/test_spark_evaluation.py::test_spark_ndcg
1.20s | tests/unit/test_spark_evaluation.py::test_spark_map
1.11s | tests/unit/test_spark_evaluation.py::test_spark_mae
Expected behavior with the suggested feature
Faster execution of unit tests
Other Comments

Looks like some Spark related notebooks take some time - I can help on that.

Looks like some Spark related notebooks take some time - I can help on that.

That would be great, I think we can add paramters to the tuning notebook and als to just do one pass through the hyperparameters.


⚠ Avatar API Endpoint has degraded performance

In 5d19b4d, Avatar API Endpoint (https://avatar.roblox.com/v1/avatar-rules) experienced degraded performance:

HTTP code: 200
Response time: 1199 ms

Resolved: Avatar API Endpoint performance has improved in c0ce905.


feat: switch to native date inputs

It looks like there is a similar MR #1096 but it's not finished so I hope that's okay.

rebased on next branch

Native datetime picker does not look like shit anymore?

looks ok for me in chrome and firefox, ...


Order Constructor Not Updated in Nuget?

Heya ,
I installed the latest Nuget package and was trying to create order's but the order constructor is different from the one here , it has arguments like Id etc , was trying the exchange.cs example ! but the constructor arguments weren't matching up ! Btw Nice package its really helpful !

Hi,
I can also confirm this issue:
It seems that updated nuget packages v. <IP_ADDRESS> and v. <IP_ADDRESS> contain updated source files but the binaries (WavesCS.dll) are old. For example if you try to use nuget v. <IP_ADDRESS> - there is still no method TransactionExtensons.GenerateId!

Ya after i wrote that found that many things were not updated , hopefully will be updated soon!

The binaries were updated in version <IP_ADDRESS>.


chore: Add stale bot config

:books: Documentation preview :books:: https://meltano-sdk--1846.org.readthedocs.build/en/1846/

This repo has fewer issues so I hope it won't be too noisy


[CIVP-11405] BLD: Fix functools32 installation

Hopefully fixes the following error on Python 3, but I haven't been able to replicate not installing from PyPI.
Using cached functools32-3.2.3-2.zip
Complete output from command python setup.py egg_info:
This backport is for Python 2.7 only.

Issue is described in the following
https://github.com/inveniosoftware/invenio-oaiserver/issues/112
https://github.com/Julian/jsonschema/issues/228
https://hynek.me/articles/conditional-python-dependencies/
https://wheel.readthedocs.io/en/latest/#defining-conditional-dependencies
And the differing usage of install_requires vs requirements.txt here
https://packaging.python.org/requirements/

@jseabold we need to make a patch release for this bugfix.  Do you mind merging against release-1.5 instead of master?

Do you mind also adding a line to the changelog?

@jseabold, actually hold on for a second before changing the merge branch.  We're settling on the naming convention for release branches.

Okay, go ahead and switch the merge into branch to release-1.5.  (We originally were using release-v1.5, but are going to move ahead without the leading v for versions).

Has this been merged? I'm running into this problem trying to reconstruct an environment with conda


Display more context when missing dependencies

This updates error messages to provide more context around which Plugin was attempting to require a missing dependency.
For a normal missing dependency:

Could not resolve token: "DepA", which is required by plugins registered with tokens: "MyParentToken". Did you forget to register a value for "DepA"?

For a missing dependency used by an enhancer plugin:

Could not resolve token: "DepA", which is required by plugins registered with tokens: "EnhancerOf<MyEnhancedToken>". Did you forget to register a value for "DepA"?

!merge


make ship fails with certain user ids

File /home/user/mpw-9/caravel_user_project3/mag/hexdigits/alpha_C.mag couldn't be read
Cell alpha_C is unavailable.  It could not be expanded.
File /home/user/mpw-9/caravel_user_project3/mag/hexdigits/alpha_D.mag couldn't be read
Cell alpha_D is unavailable.  It could not be expanded.
File /home/user/mpw-9/caravel_user_project3/mag/hexdigits/alpha_E.mag couldn't be read
Cell alpha_E is unavailable.  It could not be expanded.
File /home/user/mpw-9/caravel_user_project3/mag/hexdigits/alpha_F.mag couldn't be read
Cell alpha_F is unavailable.  It could not be expanded.

Looks like it's the last 2 digits of a 8 unique digit string.
Using 01234567, I get an error with 6 and 7.

Isn't it incorrect to be looking in the user project directory for the hexdigits subdirectory from caravel?  That should be pointing to the base caravel repository.

I believe it's read with -dereference and the first 6 unique digits work.

Magic 8.3 revision 373 - Compiled on Wed Mar  1 20:23:37 PST 2023.
Starting magic under Tcl interpreter
Using the terminal as the console.
Using NULL graphics device.
Processing system .magicrc file
Sourcing design .magicrc for technology sky130A ...
2 Magic internal units = 1 Lambda
Input style sky130(): scaleFactor=2, multiplier=2
The following types are not handled by extraction and will be treated as non-electrical types:
    ubm
Loading sky130A Device Generator Menu ...
Loading "/home/user/mpw-9/caravel_user_project3/mag/mag2gds_caravel.tcl" from command line.
Processing timestamp mismatches: user_id_textblockWarning:  Parent cell lists instance of "chip_io" at bad file path /home/user/mpw-9/caravel_user_project3/mag/chip_io.mag.
The cell exists in the search paths at ../mag/chip_io.mag.
The discovered version will be used.
, chip_ioWarning:  Parent cell lists instance of "open_source" at bad file path /home/user/mpw-9/caravel_user_project3/mag/open_source.mag.
The cell exists in the search paths at ../mag/open_source.mag.
The discovered version will be used.
, open_sourceWarning:  Parent cell lists instance of "copyright_block" at bad file path /home/user/mpw-9/caravel_user_project3/mag/copyright_block.mag.
The cell exists in the search paths at ../mag/copyright_block.mag.
The discovered version will be used.
, copyright_blockWarning:  Parent cell lists instance of "caravel_core" at bad file path /home/user/mpw-9/caravel_user_project3/mag/caravel_core.mag.
The cell exists in the search paths at ../mag/caravel_core.mag.
The discovered version will be used.
Duplicate cell in caravel_core:  Instance of cell simple_por is from path ../mag but cell was previously read from /home/user/mpw-9/caravel_user_project3/caravel/maglef.
Cell name conflict:  Renaming original cell to simple_por#0.
Warning:  Renaming read-only cell "simple_por"
Read-only status will be revoked and GDS file pointer removed.
, caravel_core, spare_logic_block, sky130_fd_sc_hd__conb_1, sky130_fd_sc_hd__tapvpwrvgnd_1, sky130_fd_sc_hd__decap_3, sky130_fd_sc_hd__fill_2, sky130_fd_sc_hd__decap_8, sky130_fd_sc_hd__decap_4, sky130_fd_sc_hd__fill_1, sky130_fd_sc_hd__decap_6Warning:  Parent cell lists instance of "RAM128" at bad file path ../mag/RAM128.mag.
The cell exists in the search paths at /home/user/mpw-9/caravel_user_project3/mgmt_core_wrapper/mag/RAM128.mag.
The discovered version will be used.
, caravel_clockingWarning:  Parent cell lists instance of "caravel_motto" at bad file path /home/user/mpw-9/caravel_user_project3/mag/caravel_motto.mag.
The cell exists in the search paths at ../mag/caravel_motto.mag.
The discovered version will be used.
, caravel_mottoWarning:  Parent cell lists instance of "caravel_logo" at bad file path /home/user/mpw-9/caravel_user_project3/mag/caravel_logo.mag.
The cell exists in the search paths at ../mag/caravel_logo.mag.
The discovered version will be used.
, caravel_logo.
Warning:  Dereferenced cell "sky130_fd_pr__res_xhigh_po_0p69_S5N9F3" not found in search paths;  using original location ../mag/primitives/sky130_fd_pr__res_xhigh_po_0p69_S5N9F3.
Warning:  Dereferenced cell "sky130_fd_pr__pfet_g5v0d10v5_ZEUEFZ" not found in search paths;  using original location ../mag/primitives/sky130_fd_pr__pfet_g5v0d10v5_ZEUEFZ.
Warning:  Dereferenced cell "sky130_fd_pr__pfet_g5v0d10v5_YUHPXE" not found in search paths;  using original location ../mag/primitives/sky130_fd_pr__pfet_g5v0d10v5_YUHPXE.
Warning:  Dereferenced cell "sky130_fd_pr__pfet_g5v0d10v5_YUHPBG" not found in search paths;  using original location ../mag/primitives/sky130_fd_pr__pfet_g5v0d10v5_YUHPBG.
Warning:  Dereferenced cell "sky130_fd_pr__pfet_g5v0d10v5_YEUEBV" not found in search paths;  using original location ../mag/primitives/sky130_fd_pr__pfet_g5v0d10v5_YEUEBV.
Warning:  Dereferenced cell "sky130_fd_pr__pfet_g5v0d10v5_3YBPVB" not found in search paths;  using original location ../mag/primitives/sky130_fd_pr__pfet_g5v0d10v5_3YBPVB.
Warning:  Dereferenced cell "sky130_fd_pr__nfet_g5v0d10v5_ZK8HQC" not found in search paths;  using original location ../mag/primitives/sky130_fd_pr__nfet_g5v0d10v5_ZK8HQC.
Warning:  Dereferenced cell "sky130_fd_pr__nfet_g5v0d10v5_TGFUGS" not found in search paths;  using original location ../mag/primitives/sky130_fd_pr__nfet_g5v0d10v5_TGFUGS.
Warning:  Dereferenced cell "sky130_fd_pr__nfet_g5v0d10v5_PKVMTM" not found in search paths;  using original location ../mag/primitives/sky130_fd_pr__nfet_g5v0d10v5_PKVMTM.
Warning:  Dereferenced cell "sky130_fd_pr__cap_mim_m3_2_W5U4AW" not found in search paths;  using original location ../mag/primitives/sky130_fd_pr__cap_mim_m3_2_W5U4AW.
Warning:  Dereferenced cell "sky130_fd_pr__cap_mim_m3_1_WRT4AW" not found in search paths;  using original location ../mag/primitives/sky130_fd_pr__cap_mim_m3_1_WRT4AW.
File /home/user/mpw-9/caravel_user_project3/mag/hexdigits/alpha_1.mag couldn't be read
Cell alpha_1 is unavailable.  It could not be expanded.
File /home/user/mpw-9/caravel_user_project3/mag/hexdigits/alpha_0.mag couldn't be read
Cell alpha_0 is unavailable.  It could not be expanded.

Cannot open /home/user/mpw-9/caravel_user_project3/gds/caravel.gds to write GDS-II stream output
Make ship completed.

Looks like the timestamp errors are displayed one cell at a time with a new line only at the end. This can lead to confusing error messages like

, caravel_mottoWarning:  Parent cell lists instance of "caravel_logo" at bad file path /home/user/mpw-9/caravel_user_project3/mag/caravel_logo.mag.

Do timestamp errors prevent the creation of the gds file? If not, what is the cause of

Cannot open /home/user/mpw-9/caravel_user_project3/gds/caravel.gds to write GDS-II stream output

The simple_por is added to the search path in caravel/mag/.magicrc

addpath ../subcells/simple_por/mag

There's also the explicit load in the tcl script of the maglef view.
load /home/kanobailey/mpw-9/caravel_user_project3/caravel/maglef/simple_por;

This file has a GDS_FILE link. But the mag file also has a GDS_FILE link.
ciic-cvc:~/mpw-9/caravel_user_project3$ tail caravel/mag*/simple_por.mag
==> caravel/mag/simple_por.mag <==
flabel metal1 -14 8544 58 8684 0 FreeSans 800 0 0 0 vss1v8
port 6 nsew ground bidirectional
flabel metal4 s 38 7255 232 7655 0 FreeSans 320 0 0 0 vss3v3
port 2 nsew ground bidirectional
<< properties >>
string FIXED_BBOX 0 0 11344 8338
string GDS_FILE ../gds/simple_por.gds
string GDS_START 0
string LEFclass BLOCK
<< end >>

==> caravel/maglef/simple_por.mag <==
port 1 nsew power bidirectional
rlabel metal4 38 7965 7241 8283 0 vdd3v3
port 2 nsew power bidirectional
<< properties >>
string LEFclass BLOCK
string FIXED_BBOX 0 0 11344 8338
string GDS_FILE ../gds/simple_por.gds
string GDS_START 0
string LEFview TRUE
<< end >>

Is the error message in the build log irrelevant?
Duplicate cell in caravel_core:  Instance of cell simple_por is from path ../mag but cell was previously read from /home/user/mpw-9/caravel_user_project3/caravel/maglef.
Cell name conflict:  Renaming original cell to simple_por#0.
Warning:  Renaming read-only cell "simple_por"
Read-only status will be revoked and GDS file pointer removed.

@d-m-bailey :  Weird problems with files not being able to be opened usually indicates that magic hit the filesystem ulimit on opened file descriptors.  It holds files open because that's the way Linux does file locking.  In this case, there should not be multiple processes attempting to write the same files, so the solution is to add the command locking disable at/near the top of the script, before any .mag files are read.  This command was added to scripts in precheck for handling the GF process, because the GF I/O cells alone would overrun the ulimit.  I assume it's just that the command was never added to the scripts in the sky130 version of Caravel.  This has not happened in the past, but presumably can be triggered for any design with a sufficient number of cells in it coming from .mag files.

@d-m-bailey :  Please raise an issue on the github issue tracker for magic about the timestamp warning messages.  It is one of those low-level problems that I know about but is too low priority to get worked on.  The underlying issue is that stdout is buffered but stderr is flushed after every write, so if stdout and stderr are combined in the output stream then they can get scrambled together.  The "quick and dirty" solution would be just to output one line per cell with a timestamp issue.

@d-m-bailey : Please raise an issue on the github issue tracker for magic about the timestamp warning messages. It is one of those low-level problems that I know about but is too low priority to get worked on. The underlying issue is that stdout is buffered but stderr is flushed after every write, so if stdout and stderr are combined in the output stream then they can get scrambled together. The "quick and dirty" solution would be just to output one line per cell with a timestamp issue.

raised.

@d-m-bailey : Weird problems with files not being able to be opened usually indicates that magic hit the filesystem ulimit on opened file descriptors. It holds files open because that's the way Linux does file locking. In this case, there should not be multiple processes attempting to write the same files, so the solution is to add the command locking disable at/near the top of the script, before any .mag files are read. This command was added to scripts in precheck for handling the GF process, because the GF I/O cells alone would overrun the ulimit. I assume it's just that the command was never added to the scripts in the sky130 version of Caravel. This has not happened in the past, but presumably can be triggered for any design with a sufficient number of cells in it coming from .mag files.

That's crazy, but makes sense. Adding lockign disable fixes the problem, but I couldn't find this change on the github version of caravel-gf180mcu. Maybe it's in the production version only.

There should be an issue and pull request on the caravel-gf180mcu repository issue tracker?  James Stine ran into the same problem recently when trying to do a local "make ship" on the Caravel GF version.  I fixed the script and pushed the correction to a branch called fix_make_ship_script.

Yes. I see it now that you pointed it out.


fix: Allow override of default /tmp volume mount in repo server

What type of PR is this?
/kind bug
What does this PR do / why we need it:
The ArgoCD HA guidelines [1] says:

argocd-repo-server clones the repository into /tmp (or the path specified in the TMPDIR env variable). The Pod might run out of disk space if it has too many repositories or if the repositories have a lot of files. To avoid this problem mount a persistent volume.

However, with the current behaviour of argocd-operator, it is not possible to override the tmp path: when you attempt to do so, you will see this error in the operator logs:
2024-07-16T07:32:47Z ERROR Reconciler error {"controller": "argocd", "controllerGroup": "argoproj.io", "controllerKind": "ArgoCD", "ArgoCD": {"name":"argocd","namespace":"jgw"}, "namespace": "jgw", "name": "argocd", "reconcileID": "6979a388-fdae-453d-b999-b026aa1e260b", "error": "Deployment.apps \"argocd-repo-server\" is invalid: spec.template.spec.containers[0].volumeMounts[8].mountPath: Invalid value: \"/tmp\": must be unique"}
sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).reconcileHandler
/remote-source/deps/gomod/pkg/mod/sigs.k8s.io/controller-runtime@v0.17.2/pkg/internal/controller/controller.go:329
sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).processNextWorkItem
/remote-source/deps/gomod/pkg/mod/sigs.k8s.io/controller-runtime@v0.17.2/pkg/internal/controller/controller.go:266
sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).Start.func2.2
/remote-source/deps/gomod/pkg/mod/sigs.k8s.io/controller-runtime@v0.17.2/pkg/internal/controller/controller.go:227
W0716 07:33:31.475998 1 warnings.go:70] apps.openshift.io/v1 DeploymentConfig is deprecated in v4.14+, unavailable in v4.10000+

using an ArgoCD CR like this:
kind: ArgoCD
# (...)
spec:
  # (...)
  repo:
    resources:
      limits:
        cpu: '1'
        memory: 1Gi
      requests:
        cpu: 250m
        memory: 256Mi
    volumeMounts:
      - mountPath: /tmp
        name: my-volume
    volumes:
      - emptyDir: {} # in practice, a user would likely use a persistent volume for increased storage capacity
        name: my-volume

This PR skips creation of the default /tmp mount path in the case where the user is already providing one via .spec.repo.volumeMounts in the ArgoCD CR.
Have you updated the necessary documentation?

[ ] Documentation update is required by this PR.
[ ] Documentation has been updated.

Which issue(s) this PR fixes:
N/A
How to test changes / Special notes to the reviewer:

Red Hat tracking JIRA: https://issues.redhat.com/browse/GITOPS-4602

Failed on:
--- FAIL: kuttl/harness/1-036_validate_role_rolebinding_for_source_namespace (63.91s)


Improvements for Ultima Weapon - Porta Decumana nm

https://www.youtube.com/watch?v=tx_DhkmLlYw

Compiled against live build, tested in MSQ roulette, good to merge


Hive keychain signing works without plugin

Just a note:
When Hive Keychain is not installed as a plug in, we can't watch for the return message because we don't know who is sending the payment. We can see the inbound payment to the server account.


Retry feature not  implement

it looks like the "Retry" option for snmp v1/v2c not implement

Do you mean the retry switch in samples? They are not implemented.

It is subjective to say what is better.
Consider the fact that retry is already well implemented in other general libraries (RX for instance), essentially there is no need for #SNMP Library itself to retry.


Git control out of sync

VSCode Version: 1.0.0
OS Version: win7 pro

Steps to Reproduce:

When you have modified the file, git is not prompt

What do you mean by git is not prompt?

Closing due to inactivity.


Remote storages support: create StagesStorage using common factory and --stages-storage address param

refs https://github.com/flant/werf/issues/1614


[Bug]: It show error on device manager (code 43)

Which OS?
Win11 Pro
Which release?
24H2
Describe the bug
After working great for like 3 hours, and several reboots (the last thing i did was setting up autologin on W11 with netplzwiz), it shows the yellow triangle on device manager and stop working)
Reinstalling didn't work
Steps to reproduce
No response
Expected behavior
No response
Log File (Beta Only)
No response
Contact Details
<EMAIL_ADDRESS>

I tried with the no-HDR version and it's the same. It seems to be OK, but shows code 43 after reboot.

Did you happen to delete the driver folder after installation?

If you disable and reenable in Device Manager, does the code persist?

It shows no error until I reboot or I try to use it (like starting a stream in sunshine)

Do you have multiple accounts on windows ?

No, but the last thing before troubles was remove the windows 11 PIN and using netlplwiz to auto-enter the user password, with the purpose of enabling wake on lan for sunshine.
I think that in the next reboot then problems began.

Update
If I disable autologin and set PIN login again, the driver works as intended.
The problem here is to set my remote tun on and login procedure.


Add OSS publishing for runtime

for #16
I have added this based on the suggestions in http://www.scala-sbt.org/0.13/docs/Publishing.html

Thanks! I think this has gotten us most of the way there.


How to fix "Commit xyz doesn't exist"

danger local started to fails for me with the following error:
$ danger local
Running your Dangerfile against this PR - https://github.com/php-coder/mystamps/pull/533
Turning on --verbose

fatal: --unshallow on a complete repository does not make sense
/home/coder/.gem/ruby/gems/danger-4.2.2/lib/danger/scm_source/git_repo.rb:58:in `raise_if_we_cannot_find_the_commit': Commit 2a98ef46 doesn't exist. Are you running `danger local/pr` against the correct repository? Also this usually happens when you rebase/reset and force-pushed. (RuntimeError)
	from /home/coder/.gem/ruby/gems/danger-4.2.2/lib/danger/scm_source/git_repo.rb:43:in `ensure_commitish_exists!'
	from /home/coder/.gem/ruby/gems/danger-4.2.2/lib/danger/request_sources/github/github.rb:89:in `setup_danger_branches'
	from /home/coder/.gem/ruby/gems/danger-4.2.2/lib/danger/danger_core/environment_manager.rb:57:in `ensure_danger_branches_are_setup'
	from /home/coder/.gem/ruby/gems/danger-4.2.2/lib/danger/danger_core/dangerfile.rb:258:in `setup_for_running'
	from /home/coder/.gem/ruby/gems/danger-4.2.2/lib/danger/danger_core/dangerfile.rb:269:in `run'
	from /home/coder/.gem/ruby/gems/danger-4.2.2/lib/danger/commands/local.rb:50:in `block in run'
	from /home/coder/.gem/ruby/gems/danger-4.2.2/lib/danger/commands/local_helpers/local_setup.rb:38:in `setup'
	from /home/coder/.gem/ruby/gems/danger-4.2.2/lib/danger/commands/local.rb:49:in `run'
	from /home/coder/.gem/ruby/gems/claide-1.0.1/lib/claide/command.rb:334:in `run'
	from /home/coder/.gem/ruby/gems/danger-4.2.2/bin/danger:5:in `<top (required)>'
	from /home/coder/bin/danger:22:in `load'
	from /home/coder/bin/danger:22:in `<main>'

Yes, I did a rebase and yes this commit doesn't exist but what to do now? I tried to specify option --clear-http-cache but it didn't help. How to fix this error? Where danger found this commit?

This commit  reference 2a98ef46 would have come from the GitHub API as being either the base or the head sha for your PR. Danger would have looked for these inside the local git repo, and try to update if she can't find one

Closing due to inactivity.

I'm having this problem but on Jenkins and not locally. I know why but not how to fix it, so I'm posting here to see if maybe help someone in the future.
As orta mentioned the commit is coming from GH API, in my case the missing one is the base. The base is the most recent master commit but when Jenkins checkout the PR it only fetches the branch to be tested so any commits in master that are newer than the branch being tested are not in the local clone.
I'm trying to find a way to make Jenkins include master on the checkout as well. I'll update here if I find anything.
I'm on Danger 5.3.4.

Any change? I have this issue too with Jenkins and bitbucket

@zdenektopic I solved it by making Jenkins checkout master as well as the PR it is going to build. I'm on leave right now, I don't recall the exact setting but it was related to manually specifying extra ref specs and adding master to it.

I have got past that now, I am stuck on:

/Library/Ruby/Gems/2.0.0/gems/danger-5.4.0/lib/danger/scm_source/git_repo.rb:104:in `find_merge_base': Cannot find a merge base between 26f1d51abbf3c41362179254cd7c3e8e37ff752b and 890f88879b43d3e7feba10e7837f796e8fc9fa01. (RuntimeError)


map flickers (tiles are reloaded/recomputed) when map center passes the date line

TO REPRODUCE THE ISSUE, FOLLOW THESE STEPS:
Openthe simple demo: http://tangrams.github.io/simple-demo/
Execute this code in the JS console:
zoom = 3;
lng = 170;
map.setView({lat: 0, lng: lng}, zoom);
setInterval(
  function() {
    lng = lng < 180 ? 190 : 170;
    map.setView({lat: 0, lng: lng}, zoom);
  },
  3000
);

The map should be quite large (e.g. 2000 x 1000 px) to see the flickering. It may also help to change the zoom value (3) to a different level until you can see the flickering
RESULT:
Map tiles are reloaded/recomputed every time when the map center passes the international date line (180° longitude). That causes a map flickering.
EXPECTED RESULT:
Tiles shouldn't be reloaded/recomputed, map shouldn't flicker.
ENVIRONMENT:
OS X 10.11.6

Firefox 55.0.3
Chrome 60.0.3112.113
Safari 10.1.2

Thanks for the issue, @ThFischer!

This has been fixed since Tangram v0.12, simple-demo updated with https://github.com/tangrams/simple-demo/commit/ecc4ec62c16e5d6710f9e546a45ab0f2aa5bd810


Handle new NOT_FOUND API error status code

Per issue #80, this handles the new NOT_FOUND error status code returned by the Google Places API (details) so that it's handled semantically the same as the ZERO_RESULTS error status code. It also changes the in list to a tuple to improve performance.

I am dropping this issue because handling NOT_FOUND actually requires much more extensive changes to the way the library works. Better to let it keep raising an error (more specific as per issue #81).


Added updated display descriptors

This solves #132 - I grabbed all the ones I could find and converted them (scripted, combo of jq and sed).  Note that we're right now splitting up the multi-resolution descriptors into single-resolution ones.
@yboger @DuFF14 If I could get a review of these and a clear to merge, that would be great.

Unity parser expects an integer for the rotate_180 field right now. Mentioned here as well: https://github.com/OSVR/OSVR-Unity/pull/46
Some of these have an integer for this field (Vrvana, HMD.json), others have "true" or "false" (HDK, DK2). That is the only issue I noticed.

Hmm, so I assumed that it would do the conversions and that you were reading them as bools. I think it was my script that added them as "false" if they didn't already exists. Quite awkward as an integer, but since we're keeping the schema for now I can update that.

OK, re-ran the conversion script after changing the default to 0, and I've updated the schema, will be putting it in the repos. Can you test?

Vrvana is missing swap_eyes, num_displays, and note. Are these optional? Would default to swap_eyes=0 and num_displays = 1.
Tested with HDK and DK2. Looks good to merge.

If you want me to add those to all, I can, but I'd prefer to just let those default, particularly since num_displays is not long for this world...

And what is the HMD.json file? It looks like the HDK with less detail.

Yea I don't think it's necessary, just a template or default.

I don't have an issue with letting them default.

OK, will merge. I've pushed the associated schema to the schema repo and updated the submodule here too.


Drop support for Node <12

Node 10 is deprecated. Node 12 is almost deprecated. This issue tracks dropping support for Node versions less than 12.

Closed with #219


Run make update-gofmt

Prepare the code for a mandatory make verify test: https://github.com/openshift/release/pull/28710

/lgtm
/approve
/hold for CI
Please unhold when CI is green.

/hold cancel

/retest-required


Add support for IVAs (GSI-650)

Add tags to all docker images
Update ghga-service-commons
Adapt to changes in the internal auth token
Add IVA field to the access request model
Support setting the IVA in the patch endpoint

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

25 of 25 (100.0%)  changed or added relevant lines in 8 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.2%) to 88.739%

Totals

Change from base Build<PHONE_NUMBER>:
0.2%

Covered Lines:
394

Relevant Lines:
444

💛  - Coveralls


[MODULE] - GPT3 TL;DR summarisation

Description
A module that summarises long texts in 4-5 lines using gpt3 "text-davinci-003" model.
Additional context
Premium function, requires an api key.

The code snippet needs to be updated to only return the text

Updated the source code


[Table] Uninitialized columns cause error in table

Bug, feature request, or proposal:
Hi, I am new to angular. I copy the example for md-table and trying to run it, but it says:
MyComponent.html:17 ERROR TypeError: Cannot read property 'diff' of undefined
at<EMAIL_ADDRESS>(table.es5.js:53)
at table.es5.js:533
at Array.forEach ()
at<EMAIL_ADDRESS>(core.es5.js:5534)
at<EMAIL_ADDRESS>(table.es5.js:532)
at<EMAIL_ADDRESS>(table.es5.js:493)
at callProviderLifecycles (core.es5.js:11191)
at callElementProvidersLifecycles (core.es5.js:11169)
at callLifecycleHooksChildrenFirst (core.es5.js:11153)
at checkAndUpdateView (core.es5.js:12258)
Which versions of Angular, Material, OS, TypeScript, browsers are affected?
I am using angular 4.3.6, cdk 2.0.0-beta.10, and angular material 2.0.0-beta.10
I run it on google chrome
here is my dependencies:
"dependencies": {
"@angular/animations": "^4.3.6",
"@angular/cdk": "^2.0.0-beta.10",
"@angular/common": "^4.3.6",
"@angular/compiler": "^4.3.6",
"@angular/core": "^4.3.6",
"@angular/forms": "^4.3.6",
"@angular/http": "^4.3.6",
"@angular/material": "^2.0.0-beta.10",
"@angular/platform-browser": "^4.3.6",
"@angular/platform-browser-dynamic": "^4.3.6",
"@angular/router": "^4.3.6",
"@covalent/core": "^1.0.0-beta.6",
"@next-level-integration/nli-common-lib": "^0.1.4",
"@next-level-integration/nli-input-lib": "^0.2.7",
"@next-level-integration/nli-menu-lib": "^0.2.3",
"@next-level-integration/nli-search-bar-lib": "0.0.6",
"@next-level-integration/nli-timeline-lib": "^0.1.6",
"@ngui/map": "^0.18.4",
"@ngx-translate/core": "^7.2.0",
"@ngx-translate/http-loader": "^1.0.2",
"classlist.js": "^1.1.20150312",
"core-js": "^2.5.1",
"hammerjs": "^2.0.8",
"intl": "^1.2.5",
"ngx-webstorage": "^1.8.0",
"perfect-scrollbar": "^0.8.0",
"rxjs": "^5.4.3",
"start": "^5.1.0",
"web-animations-js": "^2.3.1",
"zone.js": "^0.8.17"
},
Thanks.

Can you provide your template code?

Having the same issue and this is my template:
`<md-table #table [dataSource]="dataSource">

<md-header-cell *cdkHeaderCellDef> ID
<md-cell *cdkCellDef="let row"> {{row.id}}

 Name
 {{row.name}}

<md-header-row *mdHeaderRowDef="displayedColumns">
<md-row *mdRowDef="let row; columns: displayedColumns;">
`

I'm also having the same issue. The following template used to work without any problems with beta.8, though I did change over all "md-" to "cdk-".
`<cdk-table #factorsTable [dataSource]="factorsDataSource">

<cdk-header-cell *cdkHeaderCellDef> Averaged Factor
<cdk-cell *cdkCellDef="let row"> {{row.factor}}

<cdk-header-cell *cdkHeaderCellDef> {{ benchmark.name }}
<cdk-cell *cdkCellDef="let row"> {{row.benchmark == null ? 'N/A' : displayTableValue(row.benchmark) }}

<ng-container *ngFor="let i of rangeEtfs" cdkColumnDef="{{i}}">
<cdk-header-cell *cdkHeaderCellDef>{{ i < etfData.length ? etfData[i].ticker : 'N/A' }}
<cdk-cell *cdkCellDef="let row">{{ row.etfs[i] == null ? 'N/A' : displayTableValue(row.etfs[i]) }}

<cdk-header-row *cdkHeaderRowDef="factorsColumns">
<cdk-row *cdkRowDef="let row; columns: factorsColumns;">

`

I'm not able to reproduce. Here is a working example, can you try and reproduce from that?

@andrewseguin Is this a real issue?

Any reproducible steps or plunkers would be appreciated. The getColumnsDiff is called once the table's content has been checked, which should include the content's row definitions. The ngOnChanges would have been called.
I see that the differ does not set up until the columns are defined. Can you check if the columns are being set sometime after the data is set on the table?
Either way, we'll want some good error handing here to notify the user. Will look into finding how to reproduce

Here's a reproduction: https://plnkr.co/edit/yWmcRLntenln6OZBJlho?p=preview

If columns are set to null or undefined when the table is setting up, then it encounters this error. Workaround is to make sure that when you set columns, it is initialized to some value. If you are unsure at the time of setup which columns should be displayed, at least set it to an empty array.

PR has been sent that should resolve this issue by initializing the columns to an empty array if none are provided.

That appears to have solved the problem. Thank you.

Thanks for so many help :) but our deadline for the project is Oktober, so I already changed back to an older version which is still work. Sorry it's a bit of hurry so I don't have time to prepare any example project to show this problem. Thanks again to our lovely community.

No problem @jlangnli, but just a heads up that it's trivial to fix on your end to avoid this error. Simply initialize your columns array to []. Otherwise you'll see the fix out soon

I got this error because I did not have an md-header-cell defined for one of my columns. The old api did not require it but the new one appears to need it even if you leave the value of the header blank.

Just change ngAfterViewInit with ngOnInit


List comprenhension

What steps will reproduce the problem?

When i try to create a list by comprehension in the kerner it stop.
The code that I use was:
squares=[i*i for i in range(5)]

Hi @Alejandro58581 thank you for the feedback. What Spyder version are you using and how did you install it? Testing the code provided seems like is working in the latest version (5.2.0):

Also, did you saw any traceback or message related with the issue?
Any new info in order to reproduce this in our side is greatly appreciated, let us know!

Hello, @dalthviz I am using spyder 5.0.5 and anaconda navigator. I can´t upgrade spyder using the prompt it never connects. Any message appears when I run the code only the kernel stops working.

@Alejandro58581, did you change the graphics backend in Preferences > IPython console > Graphics to something other than Inline?

@ccordoba12 thank you. I changed the graphics backend because matplotlib also didn´t work and the spyder bot helped me and I forgot to change it.

Ok, just so you know, this is a bug and it should be fixed in our 5.2.0 version, released a few days ago. So, as soon as it's part of Anaconda and you can update to it, we encourage you to do it.


Fix authentication event propagation

This PR fixes https://github.com/lexik/LexikJWTAuthenticationBundle/issues/89 .
I've removed the extends GetResponseEvent from events : this parent event is bound to the kernel (see the symfony api doc), and does not make sense here. Moreover, the GetResponseEvent::setResponse() method does a stopPropagation() and prevents the use of more than one listener.

+1 to this fix.


Load balancer potential problems?

We have this running behind a load balancer with 2 servers: A and B. If an unknown domain makes a request to server A and it's valid, our server makes a new cert request to lets encrypt.
The next step is for lets encrypt to make a request back to us to confirm we are who we say we are. If the load balancer chooses to send that request to server B, then the confirmation will fail as it doesnt know about the pending domains.
Has anyone tackled this problem before? Are the pending domains stored in memory? Would it be possible to store this state in redis so that both servers could know about pending domains?

As long as you're using a storage adapter that's shared between servers (like the supplied Redis adapter), then this should already be taken care of. See the storage_adapter part of the configuration.
When using Redis, then the data is shared between the servers, so the issuance should work regardless of which underlying server Let's Encrypt hits during the challenge phase. We also use the configured storage adapter to try and prevent duplicate certs for the same domain from being registered if both servers are being hit simultaneously before the cert is issued.
But give a shout if you're seeing any issues with the current implementation, or have any other questions.

Thanks for confirmation! I will upgrade the version to see if that resolves some of my issues.

We still have this problem when Redis storage adapter is not possible,
I faced this problem when using Aws global accelerator with instances from europ and usa, Redis cannot be used in this case.


Refactor: prevent double setup

Currently, if a user calls setup() in their config, it gets executed twice, since it's called automatically in this file. No real reason for that - we can check if it's setup in the open function anyways, which solves the same issue of trying to open the plugin before being setup.

I think perhaps that we should not do auto setup at all and make it explicit.
Most (if not all) neovim plugins require explicit setup.
If a setup is done automatically in the open function, it is still quite error prone as you can use other neogit functionalities before that auto setup. So in essence there are now 3 different states of setups, where 2 would be preffered

Fine with me. It's a breaking change, but it's also undocumented currently that it auto-sets up. I'll just remove that bit.

Yes, it is a breaking change, but having this auto magic setup is quite surprising and may lead to inconsistent behavior or values somehow being set.

Agreed. Plus... it's undocumented, and the "normal" way to do it is to require explicit setup. In fact, I was entirely unaware of this until I was reading through the debug logs and noticed setup being run twice - once without user settings, once with.

Good to go then?


Reproducing the cargo-crev binary

Hey!
Our rebuilder at Arch Linux has flagged the cargo-crev package as unreproducible, meaning "running the build in an identical build environment doesn't result in an identical binary".
The diff between the official package distributed to users and the package our rebuilder has built looks like this: https://reproducible.archlinux.org/api/v0/builds/33309/diffoscope
You can ignore the NT_GNU_BUILD_ID, but there seems to be a difference in the .data section too. Do you have any ideas where this might be coming from? Most of the other rust packages reproduce successfully so it could be from cargo-crev or a dependency.
Thanks!

No, sorry. I don't think crev code itself do anything that could break it.

No, sorry. I don't think crev code itself do anything that could break it.


Add basic MoE timing breakdown

This PR will add some basic breakdown of time spent in moe forward, alltoall, and gating functions.
The output looks like this:

Lines 184, 185 need same changes. thx

Thanks for catching the second instance. Pushed.


No token passed

Hi.
using the paypal extension i get this issue when trying to checkout.

I also have this problem.

It is caused by wrong total; If your price is not over 999, you can temporary use the older version.
https://github.com/flaviocopes/grav-plugin-shoppingcart/commit/73112244f13a558142ae5982547af6ad6c1ebf91

live fersion is working fine.
i disabled the sanbox and send the link to test.. we got money.

Is it happening on all browsers, or just one specific browser? Can you try with more than one?
Also, if it's only happening locally, might be some connection issue, or a local web server problem, if on the server it works fine. Which local web server are you using?

I'm also receiving the error no token passed.
I setup a test product for the cost of 35, followed through to check out and when it's about to transfer to paypal I get the error. This happens when using sandbox as well as live

You can visit this website  a quartz cuvette manufacturer   (I use the latest gravcms and shoppingcart plugin version for this customer.)
To check whether you still have the token error.

Hi @gmplab, I don't have the same issue on your installation.

@bmelim  the only diffrence of my install and offical version is #62 , you can merge it to your local site, then test it again.

@bmelim can you paste the product page header? Also, can you try adding a price of 35.00 ?

@gmplab going to test the PR today

@flaviocopes sorry for the delay in replying... I have a product for 35.99 - if I add a product with 35.00 which is what I had originally it gave me the same error and shopping cart rounded off my price to 35.
I have just double checked the changes on your commit da3820c from 5 hours ago, and indeed my files are the same as your changes and I still have the same error.
do you need my product page header code? or the URL it goes to at check out?


Broken images

Hey there,
quick question: could it be possible that jstransformers breaks images, so they can't be displayed anymore?
I told metalsmith-jstransformer this:
.use(jstransformer({
  layoutPattern: 'layouts/**',
  defaultLayout: 'layouts/post.swig'
}))

Pretty basic, everything is working fine, templating-wise - but images get beaten up?!
Help much appreciated, if you need more information, please ask.
Thank you very much & good day to you, Sir!

Breaks images?
I don't believe it will act on any images. What are your images named?

If you don't mind, have a look here.
Last time I checked it garbled output of .htaccess, sitemap.xml and humans.txt too, but everything went back to normal when I excluded jstransformer from the build - leaving me with unprocessed files..
I just don't get metalsmith and swig to work :question:

Deleted repo, problem didn't seem to be related to your plugin, although I was totally sure.
My apologies, Sir.

problem didn't seem to be related to your plugin

What was the cause of the issue? I am curious.


Ambiguous rule matching limitations

Pyjsgf can't match certain rules where ambiguous speech strings are required/given. For example:
from jsgf import AlternativeSet
e = AlternativeSet("abc", "abcd")

The first alternative can be matched as normal, but the second can't be. If e.matches("abcd") is called, the first alternative will be matched instead of the second, with "d" as the returned string. If used in a rule, r.matches would return False because this is an incomplete match. You can get around this by writing the rule differently, e.g. rearranging the order.
This is also a problem for the jsgf.ext.Dictation expansion class:
from jsgf import Sequence
from jsgf.ext import Dictation
e = Sequence("test", Dictation(), "test")

Calling e.matches("test a test b") would work as expected, setting the match values appropriately for the expansions. However if "test test test" is used instead, no match values get set and the same speech string gets returned. You can get around this using either the SequenceRule or DictationGrammar classes.
Another limitation is matching successive non-optional dictation expansions:
Sequence(Dictation(), Dictation())

This will raise an ExpansionError because matching requires splitting a speech string between Dictation expansions somewhat arbitrarily. If you must do this, use SequenceRule or DictationGrammar instead.

This is not worth fixing because it adds too much complexity to the matching process. I will make a note in the documentation on rule expansion ambiguity when I get around to writing it.
Most matching limitations with the Dictation class will be fixed in the next release version by the matching implementation overhaul (see #7). The SequenceRule and DictationGrammar classes are still useful if you are matching speech hypothesises from multiple sources, e.g. from a PocketSphinx grammar and a language model.


@alloy - Fix /tag page on logged out and upgrade reaction-force

Closes https://github.com/artsy/reaction-force/issues/153
Was actually bad logic that caused the code not to work on when the user is logged out, not a weird rewire thing!

Oh I see why I got confused, we got ourselves a little duplication going on 'ey?
Can't you import the network layer function from reaction-force/dist/relay/config

Good call

Looks perfetto 👌
Merge at will.


Term frequency adjustment calculation is flawed

Quality assurance by Office for National statistics has identified problems with the term frequency adjustment calculations.
I have verified this, finding that:

Where term frequency adjustments are applied to multiple columns, we double count the part of the calculation which 'undoes lambda'
I'm not convinced that the approach to calculating term frequency adjustments is correct.

The overall result is that, at present, term frequency adjustment are too extreme:

they have a tendency to (almost) always adjust the match probability up
they tend to adjust it up too much

The existing approach to term frequency adjustments replaces the base lambda (the proportion of matches across all records) with the proportion of matches within a name, which is our interpretation of the final formula on page 9 of https://static.cambridge.org/content/id/urn:cambridge.org:id:article:S0003055418000783/resource/name/S0003055418000783sup001.pdf.
However, I am worried that this double counts the adjustment for gamma_name = 1 i.e. we shouldn't adjust up for the fact that name matches, and ALSO adjust up for the prevalance of matches within a specic name.  Sadly there's a typo (missing term) in the formula above, so it's not clear exactly how it should be performed.
There's a missing term that could be something like:

Except perhaps it's meant to be 'the product over all k, except for the first_name k.

Here are some properties we agree should be true of the term frequency adjustments:

'term frequency adjustments'  made because of skew in the data.     the match_probability already takes account of cardinality in the data.  The problem is they assume that each value appears roughly an equal amount of times.

e.g. we know that a national insurance number contributes a huge amount more informaion to a match than gender because of its high cardinality.  What about if a column had half binary values, and half NI numbers?

this implies that when term frequency adjustments are made, match probability should always be expected to be adjusted up for some values of name and down for others.    This is somethign we can run as a unit test.

for a column without skew, we expect term frequency adjustments to be 'null'

With all this in mind, term freqnecy adjustment can be expressed as a simple extension to the standard formula for computing m, u probabilities and adjustment factors, as follows:

The plan is to implement ☝  as a fix, with some functional tests  as mentioned above


support for the broadcast API

When working on #285, the broadcast API was missing.
Current workaround:
val makeBroadcast = zio.spark.sql.SparkSession.attempt(_.sparkContext.broadcast(ids))

To make it close to the actual Spark API, I will have to wrap the entire sparkContext object.

Ouch, that's huge.
Do it, I guess .... 😬🙏


Issue #658 TransformFactory and related tests

@lovemaths,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

@brentschmaltz Made the changes.

@lovemaths,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

@brentschmaltz Rebased against dev

:shipit: as an intermediate step.


my first contribution

Adding my name under contributor's list

Adding my name under contributor's list

Great work!!
Keep contributing to open-source.


Fails when running RGB based PPO baseline

Hello, I got RuntimeError: values expected sparse tensor layout but got Strided when running the RGB based PPO baseline. I ran it on Colab.
It fails with the 3.0.0b8 version, but it works with the previous one 3.0.0b7.
2024-08-22 07:04:19.211807: I tensorflow/core/util/port.cc:153] oneDNN custom operations are on. You may see slightly different numerical results due to floating-point round-off errors from different computation orders. To turn them off, set the environment variable `TF_ENABLE_ONEDNN_OPTS=0`.
2024-08-22 07:04:19.229796: E external/local_xla/xla/stream_executor/cuda/cuda_fft.cc:485] Unable to register cuFFT factory: Attempting to register factory for plugin cuFFT when one has already been registered
2024-08-22 07:04:19.251816: E external/local_xla/xla/stream_executor/cuda/cuda_dnn.cc:8454] Unable to register cuDNN factory: Attempting to register factory for plugin cuDNN when one has already been registered
2024-08-22 07:04:19.258633: E external/local_xla/xla/stream_executor/cuda/cuda_blas.cc:1452] Unable to register cuBLAS factory: Attempting to register factory for plugin cuBLAS when one has already been registered
2024-08-22 07:04:19.274371: I tensorflow/core/platform/cpu_feature_guard.cc:210] This TensorFlow binary is optimized to use available CPU instructions in performance-critical operations.
To enable the following instructions: AVX2 AVX512F AVX512_VNNI FMA, in other operations, rebuild TensorFlow with the appropriate compiler flags.
2024-08-22 07:04:20.431472: W tensorflow/compiler/tf2tensorrt/utils/py_utils.cc:38] TF-TRT Warning: Could not find TensorRT
/usr/local/lib/python3.10/dist-packages/tyro/_fields.py:330: UserWarning: The field wandb_entity is annotated with type <class 'str'>, but the default value None has type <class 'NoneType'>. We'll try to handle this gracefully, but it may cause unexpected behavior.
  warnings.warn(
/usr/local/lib/python3.10/dist-packages/tyro/_fields.py:330: UserWarning: The field checkpoint is annotated with type <class 'str'>, but the default value None has type <class 'NoneType'>. We'll try to handle this gracefully, but it may cause unexpected behavior.
  warnings.warn(
Downloading PhysX GPU library to /root/.sapien/physx/105.1-physx-5.3.1.patch0 from Github. This can take several minutes. If it fails to download, please manually download fhttps://github.com/sapien-sim/physx-precompiled/releases/download/105.1-physx-5.3.1.patch0/linux-so.zip and unzip at /root/.sapien/physx/105.1-physx-5.3.1.patch0.
Download complete.
Saving eval videos to runs/rgb-pushcube/videos
/usr/local/lib/python3.10/dist-packages/gymnasium/core.py:311: UserWarning: WARN: env.max_episode_steps to get variables from other wrappers is deprecated and will be removed in v1.0, to get this variable you can do `env.unwrapped.max_episode_steps` for environment variables or `env.get_wrapper_attr('max_episode_steps')` that will search the reminding wrappers.
  logger.warn(
/usr/local/lib/python3.10/dist-packages/gymnasium/core.py:311: UserWarning: WARN: env.single_observation_space to get variables from other wrappers is deprecated and will be removed in v1.0, to get this variable you can do `env.unwrapped.single_observation_space` for environment variables or `env.get_wrapper_attr('single_observation_space')` that will search the reminding wrappers.
  logger.warn(
/usr/local/lib/python3.10/dist-packages/gymnasium/core.py:311: UserWarning: WARN: env.single_action_space to get variables from other wrappers is deprecated and will be removed in v1.0, to get this variable you can do `env.unwrapped.single_action_space` for environment variables or `env.get_wrapper_attr('single_action_space')` that will search the reminding wrappers.
  logger.warn(
Running training
####
args.num_iterations=97 args.num_envs=1024 args.num_eval_envs=8
args.minibatch_size=1600 args.batch_size=51200 args.update_epochs=8
####
Epoch: 1, global_step=0
Evaluating
Traceback (most recent call last):
  File "/content/ppo_rgb.py", line 387, in <module>
    eval_obs, eval_rew, eval_terminations, eval_truncations, eval_infos = eval_envs.step(agent.get_action(eval_obs, deterministic=True))
  File "/usr/local/lib/python3.10/dist-packages/mani_skill/vector/wrappers/gymnasium.py", line 89, in step
    obs, rew, terminations, truncations, infos = self._env.step(actions)
  File "/usr/local/lib/python3.10/dist-packages/mani_skill/utils/wrappers/record.py", line 428, in step
    self.render_images.append(self.capture_image())
  File "/usr/local/lib/python3.10/dist-packages/mani_skill/utils/wrappers/record.py", line 321, in capture_image
    img = self.env.render()
  File "/usr/local/lib/python3.10/dist-packages/gymnasium/core.py", line 471, in render
    return self.env.render()
  File "/usr/local/lib/python3.10/dist-packages/gymnasium/core.py", line 471, in render
    return self.env.render()
  File "/usr/local/lib/python3.10/dist-packages/gymnasium/wrappers/order_enforcing.py", line 70, in render
    return self.env.render(*args, **kwargs)
  File "/usr/local/lib/python3.10/dist-packages/mani_skill/envs/sapien_env.py", line 1224, in render
    return self.render_all()
  File "/usr/local/lib/python3.10/dist-packages/mani_skill/envs/sapien_env.py", line 1196, in render_all
    for img in image.values():
RuntimeError: values expected sparse tensor layout but got Strided
Exception ignored in: <function VectorEnv.__del__ at 0x796977a5c0d0>
Traceback (most recent call last):
  File "/usr/local/lib/python3.10/dist-packages/gymnasium/vector/vector_env.py", line 330, in __del__
  File "/usr/local/lib/python3.10/dist-packages/mani_skill/vector/wrappers/gymnasium.py", line 120, in close
  File "/usr/local/lib/python3.10/dist-packages/mani_skill/utils/wrappers/record.py", line 777, in close
  File "/usr/local/lib/python3.10/dist-packages/gymnasium/core.py", line 475, in close
  File "/usr/local/lib/python3.10/dist-packages/gymnasium/core.py", line 475, in close
  File "/usr/local/lib/python3.10/dist-packages/gymnasium/core.py", line 475, in close
  [Previous line repeated 1 more time]
  File "/usr/local/lib/python3.10/dist-packages/mani_skill/envs/sapien_env.py", line 1043, in close
  File "/usr/local/lib/python3.10/dist-packages/mani_skill/envs/sapien_env.py", line 1040, in _clear
TypeError: 'NoneType' object is not callable
Exception ignored in: <function VectorEnv.__del__ at 0x796977a5c0d0>
Traceback (most recent call last):
  File "/usr/local/lib/python3.10/dist-packages/gymnasium/vector/vector_env.py", line 330, in __del__
  File "/usr/local/lib/python3.10/dist-packages/mani_skill/vector/wrappers/gymnasium.py", line 120, in close
  File "/usr/local/lib/python3.10/dist-packages/gymnasium/core.py", line 475, in close
  File "/usr/local/lib/python3.10/dist-packages/gymnasium/core.py", line 475, in close
  File "/usr/local/lib/python3.10/dist-packages/gymnasium/core.py", line 475, in close
  File "/usr/local/lib/python3.10/dist-packages/mani_skill/envs/sapien_env.py", line 1043, in close
  File "/usr/local/lib/python3.10/dist-packages/mani_skill/envs/sapien_env.py", line 1040, in _clear
TypeError: 'NoneType' object is not callable

Code:
!mkdir -p /usr/share/vulkan/icd.d
!wget -q https://raw.githubusercontent.com/rodcar/qmul_ai_dissertation/main/config/nvidia_icd.json
!wget -q https://raw.githubusercontent.com/rodcar/qmul_ai_dissertation/main/config/10_nvidia.json
!mv nvidia_icd.json /usr/share/vulkan/icd.d
!mv 10_nvidia.json /usr/share/glvnd/egl_vendor.d/10_nvidia.json
!apt-get install -y --no-install-recommends libvulkan-dev

# it fails with this specific version
!pip install mani-skill==3.0.0b8

# but it works with the previous one
#!pip install mani-skill==3.0.0b7

!pip install --upgrade tyro

!wget https://raw.githubusercontent.com/haosulab/ManiSkill/main/examples/baselines/ppo/ppo_rgb.py -O ppo_rgb.py

# parameters
env_id = "PushCube-v1"
num_envs = 1024
update_epochs = 8
num_minibatches = 32
total_timesteps = 5_000_000
eval_freq = 8
num_steps = 20
seed = 2024
model_name = "ppo_rgb"
exp_name = "rgb-pushcube"

!python {model_name}.py --seed={seed} --env_id={env_id} --exp-name={exp_name} --num_envs={num_envs} --update_epochs={update_epochs} --num_minibatches={num_minibatches} --total_timesteps={total_timesteps} --eval_freq={eval_freq} --no_partial_reset --reconfiguration_freq=1 --reward_scale=1

Found the bug, I'll push a fix

Try installing the new maniskill version v3.0.0b9 now, should work

It's working now, thank you!


Automated cherry pick of #10531

Cherry pick of #10531 on release-7.0.
/cc  @AshishDhama
NONE

Will try to auto merge this PR once all tests and checks are passing. This might take up to an hour.

Trying to auto merge this PR.

Pull Request successfully merged
SHA: cdc842bdbd2056d3d1504fef447608a79fc6b1cb


please add opening ¡ and ¿ for Spanish

Using the US keyboard as reference, the "1" key in uppercase shows the "!" sign. It would be great that Quick Accent would show "¡" (opening exclamation sign) as an option for "1" (used at least for Spanish).
Similarly, the "/" key has "?" for uppercase. Quick Accent could show "¿" for the "/" key.

please close as duplicate, opened the issue under "power toys"


Energy based models

Feb 28th version
This is real nitpicking, so ignore if you don't care about these things.
P857  "autoregressive models"  Perhaps I'm just revealing my ignorance here (in which case this is going to be very embarrassing), but I thought that auto-regressive models could model an arbitrary joint probability distribution if each variable is conditioned on all of the previous ones (as might be possible in principle with e.g., transformers).    Either I am mistaken, or possibly you are referring to models that only condition on the previous few variables (I didn't read that chapter yet).  It stuck out to me anyway.
P852 L15.  I don't think the word "datavector" exists.  Perhaps change to "at one of the training data examples" or similar.
P852, 853 Algorithm 28 and 29 have inconsistent spacing of semicolons at the end of lines.  Sometimes with a space sometimes without.  No semicolon at all at the end of Algorithm 28.
P866 Eq 25.40 has a full stop at the end, which is a little weird given that it's part of a continuing sentence.  In general the punctuation of equations is inconsistent throughout, but I don't think anyone will notice except pedants like me.  In this case though, it's actually wrong.
P872 L42  If this is camera ready production then you need to deal with the overrun of the inline equation here.
P875 L10  Similar issue with a reference.

Thanks. (Some of these inconsistencies come from this chapter being mostly written by other people  with a different style to mine.)


Provide dedicated keyboard shortcuts for changing active stack frame

It seems the only way today to change the current stack frame using keyboard shortcuts is to bind the workbench.debug.action.focusCallStackView command to a key, and then to use the arrow keys/enter to move up and down the stack. Could we get dedicated keyboard shortcuts for moving up and down a frame to use 1 key instead of 3 to change a stack frame?

This is a fair feature request. What commands would you expect "Focus Stack Frame Above", "Focus Stack Frame Below"?
If more users request this I would be open for a PR that adds this.

Yes; those sound like reasonable command names. They're also inline with the nomenclature used for focusing the call stack window, i.e. workbench.debug.action.focusCallStackView.


🗺 Remove map while debugging production build

Context: app has been deployed as early access on Google Play Store but the map component fails the application (it quits the application). We think it's due to SDK 28 being used while building the .aab (containing the .apk for Android) instead of SDK 35 that we use in development mode.
Deploying this fix will:

fix the app for now instead of having a buggy feature
show us whether react-native-maps was responsible for SDK 28 (during the production build managed by expo infrastructure)

Post-deploy update: SDK version is still 28 on the Google Play Store "Deployment UI", which actually makes sense as it's not the Expo SDK (which is 35) but it's the Android SDK. This is due to this safeguard: https://github.com/react-native-community/react-native-maps/pull/3106/files
We could change this value by following something such as https://github.com/react-native-community/react-native-maps/issues/3234 (with another version than 28) but I doubt that will solve this issue (now that we know it has nothing to do with Expo SDK)

ℹ Actually now that we removed react-native-maps completely from the project, 28 being picked for Android SDK is not due to the safeguard


fix: Proper endpoint for oauth_access_token method

💯 :shipit:
Do we need to publish a new version of this right away, if not when?

Probably we should publish a new version, when we also deploy the changes on the backend


k8s instructions should have video added to them

The k8s instructions (https://github.com/Seagate/cortx-k8s/blob/stable/doc/cortx-aws-k8s-installation.md) should be updated to include the video (https://www.linkedin.com/posts/gtouret_opensource-sds-kubernetes-activity-6867206757735919616-Hzcq).
@priyankdal can you work with @novium258 to get this assigned/scheduled please?

For the convenience of the Seagate development team, this issue has been mirrored in a private Seagate Jira Server: https://jts.seagate.com/browse/EOS-26655. Note that community members will not be able to access that Jira server but that is not a problem since all activity in that Jira mirror will be copied into this GitHub issue.

@swatid-seagate

Where do we expect to add the link to the Gregory's video? At the beginning or at the end of the document?

@johnbent @novium258

PR for this change:
https://github.com/Seagate/cortx-k8s/pull/85

https://github.com/Seagate/cortx-k8s/pull/85


io-sim: MonadFail issue in 8.8.x and beyond (fix suggestion)

Hello,
When trying to build the code on ghc 8.8.3 and 8.10.1, I get the following error:
Configuring io-sim-<IP_ADDRESS>...
Preprocessing library for io-sim-<IP_ADDRESS>..
Building library for io-sim-<IP_ADDRESS>..
[1 of 1] Compiling Control.Monad.IOSim ( src/Control/Monad/IOSim.hs, dist/build/Co                                                                               ntrol/Monad/IOSim.dyn_o )

src/Control/Monad/IOSim.hs:181:5: error:
    ‘fail’ is not a (visible) method of class ‘Monad’
    |
181 |     fail = MonadFail.fail
    |     ^^^^

src/Control/Monad/IOSim.hs:211:5: error:
    ‘fail’ is not a (visible) method of class ‘Monad’
    |
211 |     fail = MonadFail.fail
    |     ^^^^
==> ERROR: A failure occurred in build().
    Aborting...

Upon reading the MonadFail Proposal, in the section for "Adapting old code", I read that fail = MonadFail.fail can be safely deleted. Do I understand this correctly? Anyway, after deleting that line, the io-sim builds on 8.8.3 (with some warnings).

Note that we only support GHC 8.6.5 at the moment.

I am aware, and successfully built on 8.6.5. Just trying to get some potential optimizations for ARM (which I also am aware is not officially supported).
Was my approach of deleting the error-inducing line appropriate?


analyze SheetJS SSF

maybe replacement for manual parsing
https://www.npmjs.com/package/ssf

only works from raw to formatted
i would need formatted to raw, does not work with this


Support Go to Definition requests for source file labels in string literals

Adds the equivalent of the functionality described in https://github.com/bazelbuild/vscode-bazel/pull/380

hmm does this work?
I tested it with BuildBuddy.git and Bazel.git, both cannot do Go to Def for most target labels. For example:
gazelle_binary(
    name = "bb_gazelle_binary",
    languages = DEFAULT_LANGUAGES + ["@bazel_gazelle//language/bazel/visibility:go_default_library"],
)

trying to "go to def" on "@bazel_gazelle//language/bazel/visibility:go_default_library" would fail.

Ah sorry, forgot to clarify that this is specifically for source files! e.g. //path/to/source.cc

Just updated this PR description to indicate this, and opened https://github.com/withered-magic/starpls/issues/203 to track the remaining functionality!

I don't think it works. For example, I tried it on this line https://github.com/buildbuddy-io/buildbuddy/blob/bb4c9d9ff75ace7c549819e40cac080414a6a226/server/buildbuddy_server/BUILD#L71 and it did not open the Go file.
Also I think it's typical to have source syntax in the form of //some/package:local/package/path.ext and/or local/package/path.ext for short.

Hmm, I was able to get it working from that file for all the various forms of source syntax that you mentioned:
https://github.com/withered-magic/starpls/assets/153045167/e4fac38f-794f-4609-8d6a-e376aa64d5a3
This functionality isn't actually on a release yet, are you using a custom build from latest main?

ah I built from the latest from main, but I copied the old binary from editors/code/bin/starpls instead of directly from my bazel-out. 😮💨
sorry for the noise

no worries!

Goto def for non-source file labels hasbeen added in https://github.com/withered-magic/starpls/pull/205!

Amazing! I love starpls already!


cythonize fails when given setuptools.extension.Extension

Calling cythonize with an instance of setuptools.extension.Extension raises an AttributeError.
The cause is that setuptools.extension.Extension subclasses distutils.extension.Extension, which is defined as an old-style class. These types can be confirmed with:
import distutils.core.Extension
import types
import setuptools.extension.Extension

assert type(distutils.core.Extension) != types.TypeType
assert type(setuptools.extension.Extension) != types.TypeType

This behavior arises with:

setuptools == 28.7.0
cython == 0.25.1

This issue did not exist with earlier cython versions.
Closer examination of the docstring of function cythonize reveals that one should pass a list of Extension objects:
As module list, pass either a glob pattern, a list of glob patterns or a list of
    Extension objects.  The latter allows you to configure the extensions separately
    through the normal distutils options.

So, my usage was wrong. Indeed, passing a list of Extension objects avoids the error. Since this error did not exist in the past, and because I believe that I followed some example when I initially wrote the code to pass an extension itself, I wonder whether this case is supposed to be supported, or if not, whether it should be emphasized in the docstring. Tracing the cause took a long time, especially because I assumed that cythonize was already properly called.
One workaround is to create a new-style subclass:
class Extension(setuptools.extension.Extension,object):
    pass

This works fine. Traceback from dd/download.py when object is removed from the above subclass definition:
dd:python setup.py install --cudd --linetrace
compile Cython extensions with line tracing
Traceback (most recent call last):
  File "setup.py", line 136, in <module>
    run_setup()
  File "setup.py", line 93, in run_setup
    extensions = download.extensions(directives)
  File "/path/dd/download.py", line 119, in extensions
    v, compiler_directives=directives)
  File "~/.virtualenvs/dev/lib/python2.7/site-packages/Cython/Build/Dependencies.py", line 809, in cythonize
    aliases=aliases)
  File "~/.virtualenvs/dev/lib/python2.7/site-packages/Cython/Build/Dependencies.py", line 648, in create_extension_list
    elif isinstance(patterns, basestring) or not isinstance(patterns, collections.Iterable):
  File "~/.virtualenvs/dev/lib/python2.7/abc.py", line 144, in __instancecheck__
    return cls.__subclasscheck__(subtype)
  File "~/.virtualenvs/dev/lib/python2.7/abc.py", line 180, in __subclasscheck__
    if issubclass(subclass, scls):
  File "~/.virtualenvs/dev/lib/python2.7/abc.py", line 180, in __subclasscheck__
    if issubclass(subclass, scls):
  File "/home/ioannis/.virtualenvs/dev/lib/python2.7/abc.py", line 161, in __subclasscheck__
    ok = cls.__subclasshook__(subclass)
  File "~/.virtualenvs/dev/lib/python2.7/site-packages/backports_abc.py", line 66, in __subclasshook__
    mro = C.__mro__
AttributeError: class Extension has no attribute '__mro__'

The failure is due to the isinstance(patterns, collections.Iterable). It can be reproduced as follows:
import collections

from Cython.Build import cythonize
from setuptools.extension import Extension

a = Extension('foo', sources=['foo.pyx'])

isinstance([a], collections.Iterable)  # works fine
isinstance(a, collections.Iterable)  # raises `AttributeError`

I will report this issue also to setuptools, though it seems to me that defining an old-style class is intentional there.

Exactly seeing this error in my travis tests after update to 0.25.1.
What is a recommended workaround?

This seems to work fine for me (OS X and linux, Python 2.6 and 2.7)
In [1]: import collections
In [2]: from setuptools.extension import Extension
In [3]: a = Extension('foo', sources=['foo.pyx'])
In [4]: isinstance(a, collections.Iterable)
Out[4]: False

Are you using the same version of setuptools?
I am using python == 2.7.11+.

Interesting, it works here too.

Aha, found it again: @robertwb, you need to from Cython.Build import cythonize:
import collections
from Cython.Build import cythonize
from setuptools.extension import Extension
a = Extension('foo', sources=['foo.pyx'])
isinstance(a, collections.Iterable)

Still not able to get it to fail
In [1]: import collections
In [2]: from Cython.Build import cythonize
In [3]: from setuptools.extension import Extension
In [4]: a = Extension('foo', sources=['foo.pyx'])
In [5]: isinstance(a, collections.Iterable)
Out[5]: False
In [6]: cythonize(a)
Out[6]: [<setuptools.extension.Extension instance at 0x10933c368>]

This was happening in one virtualenv of mine, but not a fresh one (nor the global one). The answer is in the traceback (see OP):
mkvirtualenv test
pip install cython backports_abc
pip freeze
backports-abc==0.4
Cython==0.25.1

The error can then be reproduced. I do not know whether this classifies as a bug of backports_abc.

Fixed there: https://github.com/cython/backports_abc/issues/7

Thanks for fixing this issue.


[mesheryctl] system restart colliding with helm delete operation

See mesheryctl Command Tracker for current status of commands.
Current Behavior
Meshery CLI users that execute mesheryctl system restart while using in-cluster deployments find that system restart fails to start Meshery due to a helm uninstall operation ongoing (initiated by the system stop included in system restart).
Desired Behavior
Mesheryctl to wait for stop to complete, then proceed with start.
Screenshot
 mesheryctl system restart
Restarting Meshery...
Meshery deployments will be deleted from your cluster. Are you sure you want to continue [y/n]? y
Stopping Meshery...
Deleting Meshery Namespace...
Meshery is stopped.
Starting Meshery...
⡿ Deploying Meshery on Kubernetes Error: cannot start Meshery: create: failed to create: secrets "sh.helm.release.v1.meshery.v1" is forbidden: unable to create new content in namespace meshery because it is being terminated
Usage:
  mesheryctl system restart [flags]

Flags:
  -h, --help          help for restart
      --skip-update   (optional) skip checking for new Meshery's container images.

Global Flags:
      --config string    path to config file (default "/Users/lee/.meshery/config.yaml")
  -c, --context string   (optional) temporarily change the current context.
  -v, --verbose          verbose output
  -y, --yes              (optional) assume yes for user interactive prompts.

Contributor Resources

⌨ mesheryctl Contributing Guide
Beginner's guide to contributing to Meshery and mesheryctl
mesheryctl Command Tracker
Meshery CLI Commands and Documentation
🙋🏾🙋🏼 Questions: Layer5 Discussion Forum and Layer5 Community Slack

Hi @leecalcote. I was able to reproduce this issue. Looks like mesheryctl is attempting to start the server back up while the meshery namespace is still being removed on Kubernetes. A solution that I can think of would be to poll the cluster asking if the delete is done. Is this right or is there a better way?

Looping in @piyushsingariya

I think the new changes in helm installation method from meshkit might solve it, it needs to be tested after a new release of meshkit.

@gr455 it'd be great to get you involved in testing the potentially already completed fix.

MeshKit v0.5.3 is a recent release with all PRs merged.

That's great! Sure, I will try to test the fix.

Awesome. Have you join the community Slack, yet?

Yep. I also plan on attending the upcoming Newcomers' meet

I am running into issues trying to run mesheryctl with meshkit v0.5.3. I changed the version in go.mod and ran go mod tidy and ran make to build the mesheryctl binary.
Starting Meshery...
⣷ Deploying Meshery on KubernetesError: failed to start Meshery
See https://docs.meshery.io/reference/mesheryctl/system for usage details
: cannot start Meshery: context deadline exceeded

It works just fine with the older meshkit v0.5.1.
Am I doing something wrong here?

Cool, thanks for checking if the feature worked, I thought it would, it didn't worked the way i thought it would.
Mesheryctl already has capability to wait for meshey to start but I couldn't found any implementation Strategy that would help me wait till deployments are deleted, so I thought we could do with helm SDK, that fails completely. So those changes will be reverted very soon.
that basically means it still needs to be resolved, @gr455 it's time to up your sleeves and get your hands meshy in this issue.

@piyushsingariya Can you brief me on what you tried with helm sdk?

https://github.com/layer5io/meshkit/pull/147 Here you can take a look.

layer5io/meshkit#147 Here you can take a look.

Great thanks!

@piyushsingariya Let's rethink the solution to this isse. The issue is that the restart operation starts before the namespace is terminated. So we do not need to poll for resources for the sake of this issue. All we need to do is poll for the namespace in stop operation if the namespace is set to be deleted. If the namespace is not set to be deleted (with the --keep-namespace flag), then there would be no polling and everything works just fine.
I think polling for resources is also adding unnecessary complexity. Let me know what you think about this.


Write a Swift program to convert hexadecimal to binary

Description
Write a Swift program to convert hexadecimal to binary
Hexadecimal is a base-16 number system. It uses sixteen digits: 0, 1, 2, 3, 4, 5, 6, 7, 8, 9, A, B, C, D, E, and F. The digits are called hexits. A hexit is the smallest unit of information in a computer. A hexit can be used to represent a single value, such as 0, 1, 2, 3, 4, 5, 6, 7, 8, 9, A, B, C, D, E, or F, or it can be used to represent a single state, such as on or off.
Binary is a base-2 number system. It uses only two digits: 0 and 1. The digits are called bits. A bit is the smallest unit of information in a computer. A bit can be either 0 or 1. A bit can be used to represent a single value, such as 0 or 1, or it can be used to represent a single state, such as on or off.
To convert a hexadecimal number to binary, we can use the following algorithm:

Convert the hexadecimal number to decimal.
Convert the decimal number to binary.

Input  : A
Output : 1010

How to contribute

Save the solution in ConvertHexadecimalToBinary.swift file
Add ConvertHexadecimalToBinary.swift file in convert-hexadecimal-to-binary folder

!assign


Fix app name in build script

@cbenhagen did you need to change this for it to work on your machine? It worked as-is on mine which seems odd. I'm not on that machien atm, but I'll test with this change later and as long as it still works there I'll merge. If not, we'll have to figure out what's different 😄

Yes, I had to change it. Maybe you have an older version of the app? I have version 6.0.2.

This didn't work for me:
danny@Dannys-MacBook-Pro ~ % open -a "Elgato Stream Deck"
Unable to find application named 'Elgato Stream Deck'

However I noticed I was on v5. After accepting the v6 update, I saw Stream Deck.app in my applications folder was replaced by Elgato Stream Deck.app. So apparently it did change, and this is the correct/new name. Thanks!

Ah, I didn't know that was possible. That probably mades more sense because it handles both versions (and any future renames). I've updated it in c14e6f9d4d538ef38d49af4a327b94feaaa6575f. Thanks!


How to use string template with JTE ?

Hi,
I would like using string template with JTE.
Is there some example  for doing this ?
For example:
String template = """
Hello, ${user}!
""";
are expected to be
Hello, Tom!
when  parameter user  is set to Tom.
Thank you for any help.

I have implemented StringCodeResolver(String template) implements CodeResolver.
But there is a problem that render creates only one file for many templates, idk what a reason for this


Add nose customization to humans & dwarves

About the PR

This PR adds noses to humans and dwarves. They can be found as markings in the customization tab of character creation.
Why / Balance

Noses are a great way to make characters more varied and add personality.
Technical details

Since most masks are made for normal noses, some of the new nose shapes would clip out of the masks without extra support for it. To fix this, noses get hidden similar to how hair is hidden when certain masks/helmets are equipped. This check is also performed on masks toggles.
Some masks such as the normal breathing mask and medical mask work with the new noses. Because of this, some specificity is required with giving HidesNose to certain masks, instead of just giving it to all base mask prototypes.
Due to how chameleon masks work, they default to hiding noses even for masks that would not normally hide the nose shape.
The PR does not incorporate noses for slimes due to issues with transparent markings being layered on top of the base body sprite.
Media

[x] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Breaking changes

Changelog

:cl:

add: Added new noses for humans and dwarves.

Surgical mask and breath mask with the Schnozz nose, which is the largest one available.

what about slime people?

As listed in the PR:

The PR does not incorporate noses for slimes due to issues with transparent markings being layered on top of the base body sprite.

It's not an impossibility for slimes to have noses, but it would require a bunch of extra work and possibly editing the base slime sprite, and I am not sure what consequences that would have for things like gibbing. So I left it out of this PR.

They're kinda cursed and I love it

schnozz is "hey ferb whatcha wanna do today"

Every human i have is having a Schnozz, i love this

love nose customization

if this doesnt get merged im gonna go insano style


fix(cf-macro): allow functions with no Metadata (#99)

Fix #99 , ensuring functions with no Metadata can be processed with SLIC Watch config defaults
Types of changes

[X] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[X] My code follows the code style of this project.
[X] My change requires a change to the documentation.
[X] I have updated the documentation accordingly.
[X] I have read the CONTRIBUTING.md document.
[X] I have added tests to cover my changes.
[X] All new and existing tests passed.

Coverage remained the same at 100.0% when pulling 70df80a686cf922cca55e7e284d72ae83a9900de on fix-no-metadata-in-macro-functions into 064d10f071c8cbd80e63d9e4a9e45fa198dcd01a on main.


Scroll width tracking

http://www.flashdevelop.org/community/viewtopic.php?f=5&t=12589
Setting ScrollWidth to 0 should enable ScrollWidthTracking too.

Well match it to other similar settings. Try put an invalid value and check if it gives an error or not.

@Meychi
???????????

That I'm sure isn't expected...

PropertyGrid by default does this however:


Remove Bean that might not being used

Brief description of changes applied

I have seen a bean named : OAuth20CasClientRedirectActionBuilder defaultOAuthCasClientRedirectActionBuilder" that is created (seemingly) without being used anywhere (supported by github repo search "https://github.com/search?q=defaultOAuthCasClientRedirectActionBuilder+repo%3Aapereo%2Fcas&type=Code")
A bean with the same class called: "OAuth20CasClientRedirectActionBuilder oauthCasClientRedirectActionBuilder" is being used instead of the above.
If this is indeed an unused Bean, then it would be nice that it can be removed, thanks!

Thank you very much for the contribution. Keep 'em coming!


Implicit parameter defaults

Review after capitalization and cancel button PRs

The idea here is that setting parameter defaults with the implicit constructors reduces duplication in those classes. It's a little clearer what has defaults set, too.


Use abort() for fatal exit and avoid catching Throwable unless really necessary.

This avoids "dwarfeh(224) fatal error" that happens when exit() is called while an Error in in flight.

Server returned nothing (no headers, no data) on handle 24EC680
The command "./travis-ci.sh" exited with 2.

DUB failure again :/
I really hope that https://github.com/dlang/dub/pull/1190, https://github.com/dlang/dub/pull/1198 and https://github.com/dlang/dub-registry/pull/231 improve the status quo.
@s-ludwig btw I would be happy to restart Travis when I happen to browse through.


[FLINK-17170][kinesis] Fix deadlock during stop-with-savepoint. [1.12]

(Slightly adjusted backport of #15826)

What is the purpose of the change
During stop-with-savepoint cancel is called under lock in legacy sources. Thus, if the fetcher is trying to emit a record at the same time, it cannot obtain the checkpoint lock. This behavior leads to a deadlock while cancel awaits the termination of the fetcher.
Brief change log

The fix is to mostly rely on the termination inside the run method. As a safe-guard, close also awaits termination where close is always caused without lock.
Adding ITCase that covers the behavior.

Verifying this change

Added FlinkKinesisITCase.

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (yes / no)
The serializers: (yes / no / don't know)
The runtime per-record code paths (performance sensitive): (yes / no / don't know)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn/Mesos, ZooKeeper: (yes / no / don't know)
The S3 file system connector: (yes / no / don't know)

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

Failure unrelated, Kinesis e2e passed, merging.


Hacktoberfest

Made a Hacktoberfest page for the site.

Using the Github API to go through source repos in the Comcast, VinylDNS and xmidt-org organizations. Collecting all issues (issues and pull requests are treated as one in the same at this point) with the label hacktoberfest. I then break down those results into various subcategories. Pull requests are filtered by a created date of October 1, 2019 through October 31, 2019. I used existence of a closed date to indicate if the pull request is closed. For issues open date doesn't matter, but a closed date between October 1, 2019 through October 31, 2019 indicates the issue was closed during Hacktoberfest.
I also updated the home page to include a Hacktoberfest slide in the carousel.

I did add some new dependencies:

Octokit (https://github.com/octokit/rest.js): the official GitHub REST API client for JavaScript. As i was trying to work with the currently used Github.js client I read that they plan to deprecate it in favor of Octokit (https://github.com/github-tools/github/issues/564). Figured let's see how Octokit works on this new page before overhauling the existing use of Github.js on the homepage.
Babel (https://github.com/babel/babel): Turns out after implementing Octokit I learned another dependency we use, uglifyjs (https://www.npmjs.com/package/uglify-js), doesn't minify javascript that's ECMAScript2015 or above. They recommended using Babel instead.
ESLint(https://eslint.org/): to make sure my javascript is on point, following the Airbnb style guide(https://eslint.org/). I only lint the hacktoberfest_stats.js before compiling it with the Octokit.js library.

Agreed. Thank you Britney!