[CBCI-112] -  Do not use hudson.remoting.jnlp.Main directly

Addresses #17
@cloudbees/team-cloud-platform

Fine but does not address #17 as originally filed (which should just be closed as per https://github.com/jenkinsci/docker-inbound-agent/pull/129#issuecomment-614797673); only addresses https://github.com/cloudbees/jnlp-slave-with-java-build-tools-dockerfile/issues/17#issuecomment-562724025.


Issue with k8s.io/docs/tasks/administer-cluster/out-of-resource/

This is a...

[ ] Feature Request
[x] Bug Report

Problem:
The following sentence

Operator wants to evict Pods at 95% memory utilization to reduce thrashing and incidence of system OOM.

is slightly misleading because disk swapping should not be enabled so thrashing should not be an issue.
Proposed Solution:
Change the sentence to

Operator wants to evict Pods at 95% memory utilization to reduce incidence of system OOM.

Page to Update:
https://kubernetes.io/docs/tasks/administer-cluster/out-of-resource/

@dcaldwel @zacharysarah the issue is solved now.


Can't type localized symbol

Which @angular/* package(s) are the source of the bug?
forms
Is this a regression?
No
Description
I can't type russian letter 'б' at the angular material "chips" input.
For example, the first input: https://material.angular.io/components/chips/examples
Please provide a link to a minimal reproduction of the bug
No response
Please provide the exception or error you saw
No response
Please provide the environment you discovered this bug in (run ng version)
No response
Anything else?
No response

It appears to work for me. Maybe the "б" character on your keyboard is the same key code as one of the separator keys for the chips?


[gui error report] Error: Initializing daemon...

👉 Please describe what you were doing when this error happened.
Specifications

OS: linux 5.15.106-1-MANJARO
IPFS Desktop Version: 0.24.0
Electron Version: 19.0.9
Chrome Version: 102.0.5005.167

Error
Error: Initializing daemon...
Kubo version: 0.15.0
Repo version: 12
System version: amd64/linux
Golang version: go1.18.5

Error: lock /home/ng/.ipfs/repo.lock: someone else has the lock

    at errorTemplate (/opt/IPFS Desktop/resources/app.asar/src/daemon/migration-prompt.js:78:49)
    at loadWindow (/opt/IPFS Desktop/resources/app.asar/src/daemon/migration-prompt.js:129:26)
    at showMigrationPrompt (/opt/IPFS Desktop/resources/app.asar/src/daemon/migration-prompt.js:134:3)
    at startIpfsWithLogs (/opt/IPFS Desktop/resources/app.asar/src/daemon/daemon.js:193:9)
    at process.processTicksAndRejections (node:internal/process/task_queues:96:5)
    at async startDaemon (/opt/IPFS Desktop/resources/app.asar/src/daemon/daemon.js:230:21)
    at async startIpfs (/opt/IPFS Desktop/resources/app.asar/src/daemon/index.js:48:17)
    at async restartIpfs (/opt/IPFS Desktop/resources/app.asar/src/daemon/index.js:102:5)

Sorry no logs, can be closed. Restart cured it all
On Wed, Apr 19, 2023, 5:14 PM welcome[bot] @.***> wrote:

Thank you for submitting your first issue to this repository! A maintainer
will be here shortly to triage and review.
In the meantime, please double-check that you have provided all the
necessary information to make this process easy! Any information that can
help save additional round trips is useful! We currently aim to give
initial feedback within two business days. If this does not happen,
feel free to leave a comment.
Please keep an eye on how this issue will be labeled, as labels give an
overview of priorities, assignments and additional actions requested by the
maintainers:

"Priority" labels will show how urgent this is for the team.
"Status" labels will show if this is ready to be worked on, blocked,
or in progress.
"Need" labels will indicate if additional input or analysis is
required.

Finally, remember to use https://discuss.ipfs.io if you just need general
support.
—
Reply to this email directly, view it on GitHub
https://github.com/ipfs/ipfs-desktop/issues/2464#issuecomment-1514915371,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A2QXZX5RTWPMW53RSZJOPPLXB76LTANCNFSM6AAAAAAXEHFGBE
.
You are receiving this because you authored the thread.Message ID:
@.***>


Why is .gitignore excluding html files

Hello folks,
I was trying to use this package along with another couple of our things and I have noticed that npm did not include the two files browser.html and login.html in the deployed version.
I then came here and noticed that they are excluded by .gitignore and unfortunately npm does consider that when deploying.
So what I wanted to ask really is whether we could remove the line from .gitignore or (more likely) there is a super valid reason not to do so :smile:.
Thanks for this package @mikekelly, it is super useful!

Hello! Any suggestion on how to solve this?


Scheduler wasn't running properly after replacing a node

We had a cluster where a node died. Shortly after its replacement came online, the scheduler stopped running. Then it stopped running on all the other hosts.
Here's a Gist with debug info from one of the hosts:
https://gist.github.com/85f28835546bdba808681933644b3236
And tarballs for the others:
https://bazaarvoice.box.com/s/972dnf2p692m40rwwvn6kks3ear0ihfr
https://bazaarvoice.box.com/s/cl983chbpg6v6m0l86fx9vq63gzch5hv
https://bazaarvoice.box.com/s/fvm5sukewaf82kmp7uz6ujkm5mnteeqo
https://bazaarvoice.box.com/s/xsaf06vwnuma1uziimnafcqal7l4vblk

This looks like the issue I though was fixed by 90495f3, I will investigate some more.

Ok the cause here is one of the issues mentioned in #3324 (namely RunPutJobs blocking if a single job cannot be persisted, not reading from s.putJobs and thus blocking any calls to s.persistJob).
I have a fix coming soon.


SMP performance test reports - minor cleanup

Thanks for contributing to the Harness Developer Hub! Our code owners will review your submission.
Description

Please describe your changes: __________________________________
Jira/GitHub Issue numbers (if any): ______________________________
Preview links/images (Internal contributors only): __________________

PR lifecycle
We aim to merge PRs within one week or less, but delays happen sometimes.
If your PR is open longer than two weeks without any human activity, please tag a code owner in a comment.
PRs must meet these requirements to be merged:

[ ] Successful preview build.
[ ] Code owner review.
[ ] No merge conflicts.
[ ] Release notes/new features docs: Feature/version released to at least one prod environment.

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://6661e87af5a4682cfb56b948--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://6661e911b361152abf3ae8d1--harness-developer.netlify.app


Bump apg-info to 0.1.6

RE-CHECK!

@HGlab01
Every Adapter at STABLE reposittory MUST pass tests for Node 18 and 20 at all supported platforms (Linux, Windows, Mac).
Why did you remove Windows tests?
Please readd if there is no technical reason which prohibits the adapter to be used at windows.

reminder 5.4.2024

Why did you remove Windows tests? Please readd if there is no technical reason which prohibits the adapter to be used at windows.

Because

no OS-based feature of nodejs is used
github windows instances are super slow in setup

But if it is a rule I will stick to it. Windows included again.

@HGlab01 Windows is important because when native libs or dependencies are used or such there always might be build issues that arise, so it is better to check it and know. Thank you for your understanding


Compare with LEDITS

It seems that the essence of your work is the same as LEDITS. Can you compare with them, please?

LEDITS is a great real image editing method, while the main focus of our paper was for smoothly controlling the image generation of the models. They might not fall under the same category for comparison. Thanks

That being said, we did compare against text-based image generation control, which could have similar performance to ledits. You can check out our paper for more details. (We label it as "composition" in our experiments)


[Android] Add support for actions extensibility

Allows the developers to add custom renderers for android so new renderers can be added and current action renderers to be overriden.
BaseActionElementRenderer class was introduced and developers should extend it for their own custom renderers. This class includes two new listeners for developers to use

SelectActionOnClickListener (Inherits from View.OnClickListener): this class allows developers to add selectAction support for their views (either a card element or action). This class doesn't support showCard actions as it's not supported by selectAction
ActionOnClickListener: this class allows developers to add all actions support to their actions but the
ActionOnClickListener(RenderedAdaptiveCard, Context, FragmentManager, ViewGroup, BaseActionElement, ICardActionHandler, HostConfig) must be used if overriding the showCard action

Examples are included in the mobile app.
To add a new action renderer the method void registerActionRenderer(String, IBaseActionElementRenderer) must be used

ooohh.that;s a lot of commits :) - am assuming you will squash before commiting yes? Also generally do you not rebase -i to cleanup the commit history in your local fork before putting up for PR?

Can we include a "How Verified" section in teh commit descriptions as well? thanks!

ooohh.that;s a lot of commits :) - am assuming you will squash before commiting yes? Also generally do you not rebase -i to cleanup the commit history in your local fork before putting up for PR?

We don't allow plain merge commits to our repo, so your options are Squash and merge or Rebase and merge. See here for an explanation of the differences between these two GitHub options. Realistically, I don't think anyone uses the rebase option...


How do I add an import of CSS globally across the pages?

I am using this awesome module for my react app. I am trying to use bootstrap css globally across all the pages, but not sure how to do it. Can you share some example?

Add the CDN for bootstrap in your HTML, it will load all styles from there and place elements in your components ;)

Thank you! This is what exactly I was looking for.


spec: make presentable spec after public w3f wave4 announcement

@todo

spec: make presentable spec after public w3f wave4 announcement v0.0.8-postGrantAnnouncement@????-??-??

see: https://hackmd.io/YL48I_8VQ5-dftFUtHR9QQ?edit

added to spec here


Bug Fix: Chatbot not working properly

Related Issue
[Cite any related issue(s) this pull request addresses. If none, simply state “None”]
Fixes:  #938
Description
I found out that the multiple if else statements were not proper, and was requesting large response from the user, that's why I stored every bit of info in the lists, that is easily mapped.

Type of PR

[X] Bug fix
[ ] Feature enhancement
[ ] Documentation update
[ ] Other (specify): _______________

Screenshots / videos (if applicable)
[Attach any relevant screenshots or videos demonstrating the changes. Make sure to attach before & after screenshots in your PR.]
Checklist:

[X] I have made this change from my own.
[X] I have taken help from some online resources.
[X] My code follows the style guidelines of this project.
[X] I have performed a self-review of my own code.
[X] I have commented my code, particularly in hard-to-understand areas.
[X] I have made corresponding changes to the documentation.
[X] My changes generate no new warnings.
[X] I have tested the changes thoroughly before submitting this pull request.
[X] I have provided relevant issue numbers and screenshots after making the changes.

@PriyaGhosal can you review this PR, and add proper labels too.

@yashksaini-coder

Please see the conflicts highlighted in the image above.

@yashksaini-coder  Please see the conflicts highlighted in the image above.

I don't understand what's wrong here

Hi I guess that chatbot has changed so this code is no longer required


SSE and ADE: Difference

Hi,
Azure notifies me that "Managed disks created since June 10, 2017 are encrypted at rest with Storage Service Encryption (SSE). You may also want to enable Azure Disk Encryption." What is the difference between SSE and ADE (Azure Disk Encryption)? Why do I need to encrypt twice?
Kind regards,
Ryan

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

ID: 2ea5bae3-9d7d-21df-be40-322a0acb6e3b
Version Independent ID: 34f79eb2-1a94-6888-797c-a324fb4ac587
Content: Encrypt an Azure Virtual Machine
Content Source: articles/security-center/security-center-disk-encryption.md
Service: security
GitHub Login: @TomShinder
Microsoft Alias: tomsh

@ryansun96 Thanks for your feedback! We will investigate and update as appropriate.

@ryansun96 Encrypted at rest refers to the physical disk in the data center. So if someone were to take that disk physically from the data center the data would be encrypted. However, this does not apply to the data when it is being requested by the VM or when used in the portal.
For example, if you have a VM that is not encrypted via ADE but is only encrypted at rest you could take the VHD of that VM and mount it to another machine in Azure. Once mounted you could browse freely the data. This is because the data is unencrypted when you are request it over the network.
If you enable encryption via Azure Disk Encryption you could take the same scenario however when you go to browse the data after attaching the disk to another VM you would not be able it. As when you click on it there would be a requirement to unlock the drive.
I will close this for now but if you have additional questions please reopen and we can continue :)

@MicahMcKittrick-MSFT
Could you please mention the difference in the document? Because customers and me are confused by SSE and ADE, which provide similar functions.

Sure. ADE encrypts vhd files only.  SSE encrypts anything that is placed in Azure storage (at least all supported storage types). That means that for VM, you could potentially use both ADE and SSE on the same vhd files. Thanks! -Tom

Ah! OK, yes. We'll put that on the backlog. Thanks!

#please-close


Enable SSL for LMS/CMS based on environment settings

Replaces the nginx configuration files for lms and cms based on the port specified in the environment file
-Restarts nginx to use the new settings

Closing this PR since we're going with another approach.


repair_fanout7 regression is left with 2 floating iterms after repair_timing

Describe the bug
Unrelated to gate cloning (did not find any other places where nets with connections were deleted so far). Code to see floating terms:
Expected Behavior
No floating terms.
Environment
Not needed

To Reproduce
src/rsz/test/repair_fanout7.tcl with code to dump floating iterms at the end of the script.
Relevant log output
N/A

Screenshots
No response
Additional Context
No response

cc: @gadfort @maliberty

Appears to be resolved by #3926


Failed to start workers on host error

With this configuration file on cluster:
midway2.yml.txt
I get these errors:
failure.txt
for my job submitted. There are two types of messages:
ERROR: Failed to start workers on host midway2-0010: No hosts definition for local host localhost

and
ERROR: Failed to start workers on host midway2-0010: No localhost could be identified from hostname, ip address, or a localhost key in config file

But my coniguration looks pretty innocent. I'm not sure what's the issue here ...

Not sure what is going on. Did you actually use -c midway2.yml?

Yes I did use it:
sos run GTEx_V8_Association.ipynb mnm     --gene-id-file 1000_genes.txt     --prop-test 0.4     --output-dir /project2/compbio/GTEx_eQTL/cis_eqtl_mnm_v8     -c midway2.yml -q midway2

It is the first time I run a template wiht based_on. Can it be the issue?

Check with sos config -c midway2.yml and see if things are expanded correctly.

It looks correct to me ...

The configuration file by itself is correct:
[MW] cat test.sos
task: walltime='1h', mem='1G'
print(1)

[MW] sos run test -c midway2.yml.txt -q midway2
INFO: Running default:
INFO: f152cccfd778561c restart from status pending
INFO: f152cccfd778561c submitted to midway2 with job id 62867177
INFO: Workflow default (ID=a70bf382614cb889) is executed successfully with 1 completed step and 1 completed task.

The error message says:
ERROR: Failed to start workers on host midway2-0010: No hosts definition for local host localhost

which is an indication that sos tries to start a remote worker on midway2-0010 but failed. I double checked that midway2-0010 exists, so the problem should be failure to pass -c midway2.yml to the worker instance.... Let me try to reproduce.

I can reproduce the problem with
[MW] cat test.sos
input: for_each=dict(i=range(10))

task: walltime='1h', mem='1G', trunk_size=0, trunk_workers=[1]*2
import time
print(20+i)

Thank you! I wonder why this is the first time we see it thought ...

The problem is that
sos execute {task_id}

does not catch the customized configuration file. Changing the template to
      sos execute {task} -v {verbosity} -s {sig_mode} -c /scratch/midway2/gaow/TempDir/midway2.yml.txt

fixes the problem (another arised), but we currently does not pass config_file} to template.
The problem after passing config file is
2019-10-04 08:56:32,746 - ERROR - Failed to start workers on host midway2-0389: Failed to connect to midway2-0389: ssh connection to midway2-0389 was prompted for password. Please set up public key authentication to the remote host before continue.

which should not happen between computing nodes and needs further investigation. Note that sos tests remote connection and ready to answer yes so we are not seeing the "known host" issue here.

Thank you @BoPeng for the forensic work! But I've been using this for long time (over a year for sure). Even with the new MPI approach I've been using it for at last the past month. Why is it appear broken now?

Previously all tasks runs locally so there is no need for any configuration. But if a task uses something defined in customized configuration file, things can get ugly. What is even worse is that if user defines something in ~/.sos/config.yml, which is different or missing from remote host, thing can get uglier.

Whether not config_file should be pass to remote host. I briefly considered passing {config_file} in templates before but my conclusion was that -c config_file is local and might not be applicable on remote host, and decided not to pass config_file for problems cited above.

By design the new remote execution feature on cluster treats hostnames as server name, so there should be no need for localhost definition. There can be a bug here.

I see -- previously for some pipelines I believe I hard coded queue = 'midway2' as a task option. That might be the reason those worked? They still run remotely but they have the information in the task itself I believe.

@gaow  I do not know what is going on, but when I submit a two node job, log in to one of the computing nodes, and ssh to another login node
[MW] ssh midway2-0177
gaow@midway2-0177's password:

I am asked for a password.
This is unexpected because I assume that all computing nodes have public key access, which is the way all the cluster systems I have seen are set up.

I confirm the behavior ... am not sure either. I can ask around.
But if we submit from the login node it should work without an issue, if the bug is fixed. Is that the case?

I suspect that you can fix it by yourself by setting up public key authentication on cluster. I hope that I do not have to use mpiexec .

Sorry I still dont understand why it matters because I dont know why communication between compute nodes are involved -- if I submit on login node, then only log in node and compute node communicate, and that will not require password. Also my earlier workflows work but I hard-coded queue = 'midway2' in the workflow. So I suspect the issue you see above is not relevant?

After a multinode Jo's is submitted, the cluster allocate resources and provide a list of nodes as environmental variables. It then runs the script on the first node, which is the sos run or sos execute command.  The sos command realises that it is on a cluster, it starts workers on remote host by running something like ssh host sos  start 4 workers , and this is when ssh to remote host is needed.
In an moi application the program is usually started with a mpiexec command.  It also starts workers on other nodes, but it uses some system moi library, not ssh.

I see. After I revert to version 0.20.3 I no longer see the issue, but that is because there was a bug in that version which didnt actually submit multi-node jobs. Now that the issue was fixed, we are having a problem. Is that the case? But I'm sure there has been some versions where I both see multi-node parallel computing and experienced no issues. I wonder what is going on here now.

I've already asked the admin of the cluster why this is the case but it would look like this has been like this for some time. Do you think we can have an option to collect user's password on the fly without saving it to any files, then use that to aid in ssh from one node to another?

You might be able to set up things by yourself. For example, https://sites.google.com/a/case.edu/hpc-upgraded-cluster/cluster-faq/how-to-set-up-ssh-without-password says "Here is the detailed step-by-step process to establish the SSH between the nodes and the servers".

The post suggest adding login node id_rsa.pub to its own authorized_keys. The idea does cross my mind initially but I thought for something like that to work it will require gathering all compute nodes id_rsa,pub for any two compute node to access each other which is not feasible. But on a 2nd thought I realize the keys are also shared because the file system is shared, so that ought to work!
I implemented that simple solution without adding in the ~/.ssh/config file (also suggested on that page) and submitted the job:
cd ~/tmp/test
sos run test.sos -c test.yml

I then find that I can ssh from one compute node to another! That is as expected and is the good news. See below I was able to login to 2 nodes and quit:
[MW] exit
logout
Connection to midway2-0396 closed.
[MW] exit
logout
Connection to midway2-0171 closed.
[MW] # back at login node

However I see in the *err files :
bash: -c: line 0: syntax error near unexpected token `('

and some long lines after which seems to come from running SLURM_CHECKPOINT_IMAGE_DIR definition. Is that some SoS induced syntax error then?

Update and try again. The problem is likely caused by the fact that sos tries to pass environment variables to remote hosts, and the value of a variable contains (. Read the patch if you want to understand what is going on.
Perhaps mpiexec can duplicate environments more elegantly ...

Ahh i see that line now. Well maybe something more complicated is going on with my environment copying -- this is what I'm seeing now in my err file
bash: 5: command not found
bash: 45:*.spx=38: command not found
bash: 5: command not found
bash: 45:*.xspf=38: command not found
bash: 5: command not found
bash: 45:: command not found

Here is the previous err file that has this long complicated environment that it tries to copy over:
erro_1312.txt
hope that helps; or there is a more elegant solution.

please try to exclude them following my patch. I have tried but could not find a better way to pass variables.

; should be excluded because it is meaningful in command line.

I'm not sure what to do -- see file attached below. It apparently tries to copy not just random white spaces and : as you see in some of the error prompts, but also random extensions that we might not be able to completely rule out.
all_errors_new.txt
Should we use some patterns to help?

I submitted a patch to a new branch env.
The patch is easy to understand. Instead of passing environment variables from command line of ssh, I

pickle os.environ to a file
start ssh worker sos worker with option ssh worker sos worker --env file
In sos worker, update os.environ with pickled content. Because we assume file systems are shared, the file is not copied in anyway. I cannot use /tmp because computing nodes can have different /tmp even if they share /home and most other file systems.

Here I assume everything under os.environ is string, and updating os.environ of the remote sos worker process would set environment for all subprocesses that worker spawns. Please let me know if the patch works.

Nice touch in principle but under ~/tmp/test I get:
ERROR: Failed to start workers on host midway2-0365: Can't pickle local object '_createenviron.<locals>.encode'
WARNING: Failed to handle master push message ['task', {'task_id': 'bc0fb69468c58b6e', 'params': 1 (index=83), 'runtime': {'_runtime': {'verbosity': 2, 'sig_mode': 'default', 'run_mode': 'run', 'walltime': '00:15:00', 'cores': 4, 'mem':<PHONE_NUMBER>, 'workdir': path('/scratch/midway2/gaow/TempDir/test'), 'max_mem':<PHONE_NUMBER>0, 'max_cores': 40, 'max_walltime': '36:00:00'}, 'bc0fb69468c58b6e': {}}, 'sig_content': {'bc0fb69468c58b6e': {}}, 'config': defaultdict(<class 'str'>, {'config_file': None, 'output_dag': None, 'output_report': None, 'wait_for_task': None, 'default_queue': '', 'worker_procs': ['midway2-0364:4', 'midway2-0365:4'], 'max_running_jobs': None, 'sig_mode': 'default', 'run_mode': 'run', 'verbosity': 2, 'master_id': '', 'SOS_DEBUG': set(), 'sockets': {'master_push': 'tcp://<IP_ADDRESS>:36994', 'master_request': 'tcp://<IP_ADDRESS>:38874', 'worker_backend': 'tcp://<IP_ADDRESS>:36617', 'result_push_socket': 'tcp://<IP_ADDRESS>:46542'}, 'exec_mode': None}), 'quiet': True}]: Failed to start workers on host midway2-0365: Can't pickle local object '_createenviron.<locals>.encode'
WARNING: Failed to load environment variables: Ran out of input

So my first assumption that os.environ contains only strings is wrong. Could you try to change
pickle.dump(os.environ, envfile)

to something like
pickle.dump({x:y for x,y in os.environ,items() if isinstance(y, str), envfile)

Now it is working as expected on that test data:
[MW] cat *.err
INFO: M20_5434951a20ca9851 started
INFO: 4 workers started on <IP_ADDRESS>
INFO: M20_63eda7d9c1414275 started
INFO: 4 workers started on <IP_ADDRESS>
INFO: M20_90a72850f4af7e03 started
INFO: 4 workers started on <IP_ADDRESS>
INFO: M20_e3465156877592c0 started
INFO: 4 workers started on <IP_ADDRESS>
INFO: M20_ff17b7323af4dd40 started
INFO: 4 workers started on <IP_ADDRESS>

I've pushed the change. I also confirm the multi-core job on a node work as expected when I check by ssh into the node. I think we can finally resolve this issue?

Great. I will merge the branch. I will also take note for https://github.com/vatlab/sos/issues/1312#issuecomment-538831864 and refer to it in the documentation when it says "all computers, even computing nodes would need public key access".


attempt to index global 'torch' (a nil value)

Hi,
I was able to use coco in Python but I have no success with lua following your readme. Can you please tell me what I'm doing wrong? It would be probably some basic thing in configuration, but I'm new to lua and torch enviroment...
lua5.1 LuaAPI/cocoDemo.lua
lua5.1: ...e/test/torch/install/share/lua/5.1/coco/CocoApi.lua:45: attempt to index global 'torch' (a nil value)
stack traceback:
	...e/test/torch/install/share/lua/5.1/coco/CocoApi.lua:45: in main chunk
	[C]: in function 'require'
	...home/test/torch/install/share/lua/5.1/coco/init.lua:11: in main chunk
	[C]: in function 'require'
	LuaAPI/cocoDemo.lua:2: in main chunk
	[C]: ?

It seems you may not have Torch installed.
You can install Torch from here: http://torch.ch/docs/getting-started.html

Unfortunatelly this is not the case, torch is installed and I'm getting the same error mesage by calling lua5.1 from system or qlua and luajit from torch installation.
~/torch/install/bin/torch-activate was sourced...
All this is happening on up to date Arch Linux


chore: update fee estimate api

updates the fee estimate api as specified in #830 but this doesn't solve the problem because the proofSize is still 0n.

It now works before wasn't working because the codec wasn't using compact for u64 types


Enhancement? Option to clamp to words, sentences or other tokens

Hi Simon,
First of all thanks for developing this in the open! I am evaluating the text-annotator-js component to see if it fits for a project I am working on. It looks very promising.
One of the requirements for my project is that annotations should clamp to word boundaries. I have included the code below in the trimRange() method also mentioned in #66. Conceptually perhaps not the worst place to place such code. In relation to this I have a few questions:

Is there already a way to clamp to words/sentences I perhaps overlooked?
Is clamping functionality common enough to include in the annotator?
Is there a better place to modify a range based on application rules?
Would it be a good idea to add an optional callback to the annotator component which would then be used to clamp a range according to application rules?

Thanks in advance for considering these questions.
const word_boundaries = ['.',',',';',':','?','!',' ','\n','-'];

var charBeforeStart = startContainer.nodeValue.charAt(range.startOffset-1);
var newStartOffset = range.startOffset;
while(!word_boundaries.includes(charBeforeStart) && newStartOffset >= 0){
  newStartOffset = newStartOffset-1;
  charBeforeStart = startContainer.nodeValue.charAt(newStartOffset);
}
range.setStart(startContainer, newStartOffset);

var charAfterEnd = endContainer.nodeValue.charAt(range.endOffset);
var newEndOffset = range.endOffset;
while(!word_boundaries.includes(charAfterEnd) && newEndOffset < endContainer.nodeValue.length){
  newEndOffset = newEndOffset+1;
  charAfterEnd = endContainer.nodeValue.charAt(newEndOffset);
}
range.setEnd(endContainer, newEndOffset);

Thanks for the quick response!
The implementation is indeed outward clamping and the does indeed behave buggy if the selection starts with a space. Thanks for the notification. Perhaps there are other edge cases not covered as well e.g. with the non annotatable elements.
I do think the clamping (inward, outward, word boundary definitions) may be culture/application dependent. E.g. to count a - as a word boundry or not. Or to perhaps clamp annotations on sentence level. So I would suggest to configure it as a callback where the trimRange either does nothing (default?), is configured to use a best guess word level clamping, or a implemented in a user defined function. I think that would be the most flexible.
Thanks again! J


Uncaught Exception: Attempting to call a function in a renderer window that has been closed or released. Function provid

An uncaught exception was reported.
Info:
OS: win32
Arch: ia32
GPMDP Version: 3.0.1
Time: Fri Mar 25 2016 21:33:22 GMT-0500 (Central Daylight Time)
Error:
Attempting to call a function in a renderer window that has been closed or released. Function provid
Error: Attempting to call a function in a renderer window that has been closed or released. Function provided here: Object.<anonymous> (C:\Users\Jaden\AppData\Local\GPMDP_3\app-3.0.1\resources\app\build\inject\GPMInject\interface\mouseButtonNavigation.js:8:8.
    at BrowserWindow.callIntoRenderer (C:\Users\Jaden\AppData\Local\GPMDP_3\app-3.0.1\resources\atom.asar\browser\lib\rpc-server.js:201:19)
    at emitTwo (events.js:92:20)
    at BrowserWindow.emit (events.js:172:7)

This issue was created automatically inside the uncaughtException handler

Closing this automatically reported issue as a duplicate of #470


Do not start service upon installation.

Start service only after changes made to /etc/default/(go-server|go-agent).
As of release 17.2.0, we no longer start the service upon installation - https://github.com/gocd/gocd/commit/3b9f54b863c8df762c7927d859a9b4b45fdc4c7c

Closing this as the services need to start even if installation or changes to /etc/default/(go-server|go-agent) is already made by a previous chef run. See - https://github.com/gocd-contrib/go-cookbook/issues/31


I expect a Taiko script that works without errors, also to work as a step definition

Expected behavior
A Taiko scripts that works without errors should also work as a step definition in Gauge.
Actual behavior
When I run the scenario in VS Code that contains the step definition I get the following error:

Steps to reproduce
Code of the Taiko script:
const { openBrowser, goto, click, write, press, clear, textBox, into, text, closeBrowser } = require('taiko');
(async () => {
    try {
        await openBrowser();
		await setViewPort({width:1920, height:1080});
		await goto('https://www.salesforce.com');
        await click(button({id:'onetrust-accept-btn-handler'}));
		await hover($('span.region-selector_icon'));
		await click("United States (English)");
        await click('Login');
        await write('Test');
        await press('Tab');
        await write('Test');
        await write('Inloggen');
        await clear();
        await write('Test');
        await click($('#Login'));
        await write('Test');
        await click($('#signup_link'));
        await write('Test', into(textBox({name:'UserFirstName'})));
    } catch (error) {
        console.error(error);
    } finally {
        await closeBrowser();
    }
})();

Code of the step implementation:
step("SalesForce test",  async () => {
    await openBrowser();
    await setViewPort({width:1920, height:1080});
    await goto('https://www.salesforce.com');
    await click(button({id:'onetrust-accept-btn-handler'}));
    await hover($('span.region-selector_icon'));
    await click("United States (English)");
    await click('Login');
    await write('Test');
    await press('Tab');
    await write('Test');
    await write('Inloggen');
    await clear();
    await write('Test');
    await click($('#Login'));
    await write('Test');
    await click($('#signup_link'));
    await write('Test', into(textBox({name:'UserFirstName'})));
    await closeBrowser();
});

Versions
Node version: v12,18.3
Gauge version: 1.1.1
Taiko version: 1.0.19

Step definition in Gauge-js has default timeout specified in env/default/js.properties  as  test_timeout do try increasing if your step takes more time

@NivedhaSenthil Thanks for your answer. Increasing the test_timeout solved the problem.

I am re-opening this issue as this seems to be a recurring issue.
I think we should modify Javascript and typescript templates to have timeout greater than Taiko's timeout.

Updated the javascript sample. The typescript does not have a timeout


feat(helm): log templates when failed

when installing charts if it fails due to an error in templates (can be a badly formatted line), it will print the templates in output for checking in debug mode.
Closes #4674
Signed-off-by: Mohan Prasath<EMAIL_ADDRESS>

This code seems oddly similar to helm template. Is there any way we can refactor some of this code so we're not maintaining the same function calls in two places?


Add YugenAnime

Source name
YugenAnime
Source link
https://yugenanime.tv/
Source language
English
Other details
No response
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a title with source name.
[X] I have checked that the extension does not already exist on the website extensions list or the app.
[X] I have checked that the extension does not already exist by searching the GitHub repository and verified it does not appear in the code base.
[X] I will fill out all of the requested information in this form.

YugenAnime is a GogoAnime scraper website with their own interface.
So just use the Gogoanime extension.


Caution type banner displays wrong icon

Describe the bug
When implementing a banner with type="caution" the wrong icon is displayed. It should be the "i" instead of "!".
To Reproduce
Steps to reproduce the behavior:

Creat an app.
Add the banner component with type="caution".
Open preview.

Expected behavior
A described in the docs, the icon should be an "i" with yellow background.
Screenshots

Desktop (please complete the following information):

OS: OSX
Browser: Chrome
Version: 102.0.5005.61

Hi @jcipriano-stripe The exclamation mark “!" is the correct icon for "caution". "i" is information for tooltip usage.

The docs site is on an older version. I'm updating it to resolve the discrepancy.

The docs site has been updated to reflect the new icon.


client-preset does not generate types for deprecated input fields when using introspection

Which packages are impacted by your issue?
@graphql-codegen/client-preset
Describe the bug
The following type in SDL
input MyInput {
  foo: Boolean
  """
  Will be removed later
  """
  bar: Boolean @deprecated(reason: "Will be removed later")
}

Generates the following TS types using the client-preset
export type TemplatesFilter = {
  foo?: InputMaybe<Scalars['Boolean']>;
};

Your Example Website or App
https://codesandbox.io/p/sandbox/muddy-leaf-wuqws1?file=%2Frenovate.json&selection=[{"endColumn"%3A37%2C"endLineNumber"%3A12%2C"startColumn"%3A37%2C"startLineNumber"%3A12}]
Steps to Reproduce the Bug or Issue

Open code sandbox (will run apollo server with schema)
Start a new command yarn codegen:watch
Inspect the types/graphql.ts UserFilter type
deprecated field is not there

Expected behavior
The generated TS types should include deprecated fields
Screenshots or Videos
No response
Platform

OS: macOS
NodeJS: 18
graphql version: 16
@graphql-codegen/cli version(s): 3.3.1
@graphql-codegen/client-preset version(s): 3.0.1

Codegen Config File
import { CodegenConfig } from '@graphql-codegen/cli'

const config: CodegenConfig = {
  overwrite: true,
  ignoreNoDocuments: true, // for better experience with the watcher
  generates: {
    'lib/gql/': {
      preset: 'client',
      schema: {
        'http://localhost:4040/graphql': {
          headers: {
            Authorization: 'Bearer XXXXXXXX',
          },
        },
      },
      documents: ['{pages,lib,components}/**/*.{ts,tsx,js,jsx}'],
    },
  },
  config: {
    namingConvention: 'keep',
    scalars: {
      DateTime: 'string',
      JSON: 'unknown',
      Upload: 'unknown',
      Time: 'string',
      Date: 'string',
      Long: 'number',
      Url: 'string',
    },
  },
}

export default config

Additional context
No response

https://github.com/dotansimha/graphql-code-generator/pull/7769#issuecomment-1189228744
You can use inputValueDeprecation: true to override the default config. It brings back the input types, however the jsdoc comments are missing instead of adding them the same way as for a type's fields. I didn't find a solution to this.

Is there a way to just set inputValueDeprecation: true for only one of the schemas we are generating for rather than at the top level?  One of our schemas introspection does not support deprecated input args so I need to disable it for that one specifically


update responsibleai and raiwidgets to erroranalysis 0.4.4

Description
update responsibleai and raiwidgets to erroranalysis 0.4.4
Checklist

[ ] I have added screenshots above for all UI changes.
[ ] I have added e2e tests for all UI changes.
[ ] Documentation was updated if it was needed.

close & reopen for CI after release

Codecov Report

Merging #2107 (378c416) into main (dbd205c) will decrease coverage by 0.69%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #2107      +/-   ##
==========================================
- Coverage   92.52%   91.83%   -0.69%
==========================================
  Files          55       35      -20
  Lines        2126      698    -1428
==========================================
- Hits         1967      641    -1326
+ Misses        159       57     -102

Flag
Coverage Δ

unittests
91.83% <ø> (-0.69%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.
see 20 files with indirect coverage changes


https://github.com/rustdesk/rustdesk

solved https://github.com/imhicihu/Software-installations/commit/31a1b06a5527202b52c17833f5f90b2f0e223c49


Square array boolean time

Hi,
I want to do 'not' boolean for a 500*500 number of square arrays with a special shape. But it takes several hours to do such boolean. (I mean substract a special shape out from 2500 square background). Is there a way to reduce the calculation time?
Best,
Yang

@YangXu024 The boolean function is written as a C-extension already, so I'd say it is already as fast as it could be. You could try Gdstk to see if it somehow improves speed, but my first guess would be no (or very little).
 For example, does the shape you're subtracting intersect all 250.000 squares? Maybe you can use gdspy.inside to filter out some squares before the subtraction. How is the number of vertices in this shape? Boolean operation time increases fast with the number of vertices, so, if you can simplify all shapes, that will also help.

Hi, thanks a lot, gdspy.inside really helps! BTW, may I ask is there an easy way to increase all components 4 times in scale?
Best,
Yang Xu

Hi, thanks a lot. gdspy.inside really helps. BTW, when I load another gdsII contents in existing library, existing contents disappear(I can see layers but contents is not shown) and only the new loaded contents are shown. May I ask what might be the reason? I just use lib.read_gds('contents.gds').

@YangXu024 It could be a matter of scale. Does the library use the same units as the gds file? Maybe try units="convert" in GdsLibrary.read_gds to see if it helps.

Hi, Thanks for the advise. It seems unit is not the problem after I tried convert. I attached a picture a my layer after read a new gdsII file. First 5 layers are from the new file, while the other grey layers are the old ones. I guess grey means it cannot be shown.
Best,
Yang

Sorry to bother you many times, may I ask is it possible to repeat only specific layers in a cell to get some array?

Hello,
Thanks a lot. You can check my code and file at https://github.com/YangXu024/twpa. The codes are a little bit messy... basically I creat a cell called Jline and want to add a mask from another library ASML 500-300 to it.

I believe your GDS contains all the correct geometry; the problem might which cell you're looking at. In KLayout, did you select "main" to be the top cell (right click and show as net top)?

Hello,
Really thanks for your patience. With your suggestion I understand the issue now. It is that "main" cell and loaded cell are in the same hierarchy, such that they cannot be shown at the same time.
I tried to put loaded cell under "main" cell. The thing is when a cell "A" (e.g.) in lib 1 is added to lib 2, the lower hierarchy cells in "A" won't be copied to lib 2.

Sure, I solve the problem with your suggestion, thanks a lot. Just mention that the loaded cell and old cell are in the same hierarchy and they cannot be shown at the some time in Klayout (as I see). Also, when a cell "A" in lib 1 is added to lib 2, lower hierarchy cells in "A" wont be copied to lib 2, maybe it's some inherit problem for class. Anyway, thank you so much.

Yes, when you want to include a cell from another library, you have to manually include all dependencies as well. I'm glad it worked out.

Hi, is it normal that after boolean there are many additional lines, epecially when the points are very dense?

I wouldn't say "normal", but it can happen depending on how irregular the geometry is and the precision you're using.


device_map not work

System Info
peft 0.11
Who can help?
No response
Information

[ ] The official example scripts
[X] My own modified scripts

Tasks

[ ] An officially supported task in the examples folder
[X] My own task or dataset (give details below)

Reproduction
I currently have the original LLM weights (llama3-8B) and the corresponding LoRA weights. When loading them, I use the following script:
from typing import List
import torch
from transformers import AutoModelForCausalLM, AutoTokenizer
from peft import PeftModel
import json
class load_test:
    def __init__(self) -> None:
        pass
    def _load_model_tokenizer(self,args):
        self.model = AutoModelForCausalLM.from_pretrained(args.rp_path, torch_dtype=torch.bfloat16).to(self.device)
        if args.rp_lora_path != '':
            self.model = PeftModel.from_pretrained(self.model,args.rp_lora_path, adapter_name="default11").to(self.device)
            self.model.set_adapter("default11")
        torch.cuda.empty_cache()
lt = load_test()
lt.device = 'cuda:1'
lt._load_model_tokenizer(args)

The expected behavior is that all weights are loaded onto the same device (cuda:1), but there is still some occupation on cuda:0.

Expected behavior
All weights are loaded onto the specified device.
Additionally, I have checked the previous issues, but none of them provided a solution.

Could you pass the torch_device argument, i.e.:
PeftModel.from_pretrained(self.model,args.rp_lora_path, adapter_name="default11", torch_device=self.device)

For me, that solved the issue, otherwise, PEFT guesses what device to load the PEFT weights on.
This is hard to find as the argument is not documented. I'll create a PR to do that.
Strangely, I also had to remove the torch.cuda.empty_cache() in my tests, otherwise memory would be assigned to cuda:0, no idea how that's possible.

您能否传递该torch_device参数，即：
PeftModel.from_pretrained(self.model,args.rp_lora_path, adapter_name="default11", torch_device=self.device)

对我来说，这解决了这个问题，否则，PEFT 会猜测在哪个设备上加载 PEFT 重量。
由于没有记录该参数，因此很难找到。我创建了一个 PR ( #1843 ) 来做到这一点。
奇怪的是，在我的测试中我还必须删除torch.cuda.empty_cache()，否则内存将被分配给cuda:0，不知道这是怎么可能的。

Thanks for your help! It's actually work.  The torch.cuda.empty_cache() still cause allocate bad.


Add 1 to line numbers of Kotlin CPGs

it seems that the package directive is not taken into consideration

With the changes as they are here, would the line numbers in files without the package directive be wrong or is there some sort of implicit package directive added? If they would be wrong, is it possible to check whether the file contains a package directive?

@johannescoetzee
$ cat /tmp/simple/h.kt
fun main() {
  println("Helloo, world")
}

ocular> cpg.method.fullName(".*main.*").l
res8: List[Method] = List(
  Method(
    id -> 5L,
    astParentFullName -> "<empty>",
    astParentType -> "<empty>",
    binarySignature -> None,
    code -> "main",
    columnNumber -> Some(value = 4),
    columnNumberEnd -> Some(value = 0),
    depthFirstOrder -> 1,
    filename -> "h.kt",
    fullName -> "main:void()",
    hasMapping -> None,
    hash -> None,
    internalFlags -> 4,
    isExternal -> false,
    lineNumber -> Some(value = 1),     <---- LINE_NUMBER after this change
    lineNumberEnd -> Some(value = 3),
    name -> "main",
    order -> 1,
    signature -> "void()",
    varargParameter -> -1
  )
)


Over ssh, setting breakpoint doesn't work

If submitting a bug please make sure

[X] If you are using gdb

[X] gdb --version >= 7.7.1
[X] it works on the command line with gdb
[X] cwd and target are properly set

[ ]  If you are using lldb

[ ] lldb --version >= 3.7.1
[ ] it works on the command line with lldb-mi
[ ] cwd and target are properly set

My configuration:
{
"version": "0.2.0",
"configurations": [
{
"type": "gdb",
"name": "Launch",
"target": "The path to the executable on the remote machine",
"request": "launch",
"cwd": "${workspaceRoot}",
"ssh": {
"host": "<IP_ADDRESS>",
"cwd": "a working path",
"keyfile": "The id_rsa file",
"user": "keranova"
}
}]
}
My program runs properly but:
The issue: Breakpoints don't work
The warning I have: No source file named /home/myUser/dev/src/foo.cpp
The .cpp file path and name change when I change my breakpoint so it's OK but the path added after cwd is full lower case but is supposed to have uppercases. Due to this the remote machine (linux Debian buster) doesn't find the file.
The vscode machine runs on windows 10.
It looks like the breakpoint path is forced to lowercase somewhere in the process between the editor and the gdb command issued on the remote.

Looks like a duplicate for the fixed (but not released) #346 - no? If yes - please close as duplicate.

Yes seems to be the same, thank you.


Bottom tabs too high on home button devices

iOS 15.6
iPhone 8+
Issue: The bottom tabs are placed too high on home button devices (probably because that space is meant for the homebar on non-home-button iPhone devices).
It looks a little weird for it to be so high on home button devices.

Oh I see, I'll fix it

Thanks!

Done! you can check in the latest release

Thanks!


OOP school library: add Person, Student and Teacher classes

In this branch I worked on the following:

Created classes for persons, students, and teachers with instance variables
Fixed linter errors

Thank you


beforeRouteEnter is called multiple times

Version
4.0.11
Reproduction link
jsfiddle.net/kegdqcmz/
Steps to reproduce

Go to https://jsfiddle.net/kegdqcmz/
Open console
Click "Test"
Click "Test" again
Check console output

What is expected?
beforeRouteEnter is called only once
What is actually happening?
beforeRouteEnter is called multiple times

The example demonstrates that beforeRouteEnter can be called multiple times. This results in extra API calls if route fetches data from the backend.
Example: https://jsfiddle.net/xgspe390/
To reproduce:

Click "Test"
Click "Test" again before the request is ended

That is expected because you are triggering a new navigation while the ongoing navigation isn't over yet and that triggers a new fetch that takes long.
You can still cancel the new navigations if you want by saving the fact the you are fetching already or cancel the previous fetch with an abort signal. These two patterns are totally valid and it's up to you to choose.

Remember to use the forum or the Discord chat to ask questions!


Added prod and dev labels to labeler

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/approve

/lgtm


https://github.com/ytdl-org/youtube-dl/issues/30876#issuecomment-1106215674

https://github.com/ytdl-org/youtube-dl/issues/30876#issuecomment-1106215674
Originally posted by @dirkf in https://github.com/ytdl-org/youtube-dl/issues/30933#issuecomment-1121826944

dirkf doesn't seem to disappoint when it comes to abusing her power


Merge to publish.

Docs Build status updates of commit 8a7566a:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/office-365-developer-program-faq.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


feat(translation-feedback): console log incomplete postman import translations with stats and details

Description
Provide useful logs while importing a Postman collection in Bruno.
Logs will display the number of collections concerned, the number of lines to change, and a translationLog of such type :
interface TranslationLog {
  [collectionName: string]: {
    "script"?: number[],
    "test"?: number[]
  }
}

Next step : I would like to display some information about the translation failing or not in the UI, in the import location dialog. @helloanoop could I display data in the UI instead/in addition to that ?

Contribution Checklist:

[x] The pull request only addresses one issue or adds one feature.
[x] The pull request does not introduce any breaking changes
[x] I have added screenshots or gifs to help explain the change if applicable.
[x] I have read the contribution guidelines.
[ ] Create an issue and link to the pull request.

Note: Keeping the PR small and focused helps make it easier to review and merge. If you have multiple changes you want to make, please consider submitting them as separate pull requests.
Publishing to New Package Managers
Please see here for more information.

Proposition for some UI feedback to display (and copy) the errors :
**Default : **

**Opened : **


Update enable-virtualization-based-protection-of-code-integrity.md

https://github.com/MicrosoftDocs/windows-itpro-docs/issues/3997
Added to note.

@nenonix Minor copy editing is needed for this PR. Thanks.

@levinec Review needed for this PR that addresses issue #3997. Thanks.

@nenonix  Minor copy editing is needed for this PR. Thanks.

@levinec Any updates with the review for this PR? Thank you.


Added - Travis CI php 7.2

Fixes #2
Travis CI now also run on PHP 7.2.
I also add PHP last buid(allowed failure)

Coverage remained the same at 100.0% when pulling a905f914b083da705311586835968ef22f538748 on bartko-s:travis-ci-php-7.2 into 4ac7f497a925a2df1763a8e801cbe4e874535b95 on chadicus:master.


runtime: bug with indexes while range in string

What version of Go are you using (go version)?
go version go1.10.3 darwin/amd64
Does this issue reproduce with the latest release?
Yes
What did you do?
https://play.golang.org/p/SGSiTfrzD3P
What did you expect to see?
We should expect a range of 37, as there are 37 characters in the string.
What did you see instead?
A range of 38, the special characters, in this case 'Ñ' make the indexes step further

I think it's intended, see https://blog.golang.org/strings

This is not a bug:
https://golang.org/ref/spec#String_types
https://golang.org/ref/spec#For_statements (see the bit about range clauses, specifically for strings)


Allow to enable DNS listener in dnsmasq

It might happen we want DNS on specific address, usually one associated
to a specific DHCP network.
Using the new parameter, we allow to pass a list of listeners, per
network. Dnsmasq will then listen and answer to DNS queries on those
addresses.
Note that you want to pass dhcp-options to ensure the DNS resolver is
properly passed, for instance:
options:
  - "option:dns-server,<IP_ADDRESS>"

in the network definition.
As a pull request owner and reviewers, I checked that:

[X] Appropriate testing is done and actually running
[X] Appropriate documentation exists and/or is up-to-date:

[X] README in the role

/approve


Download multiple links on Tidal-gui.exe

Which tool
Tidal-gui.exe downloader
Describe your question
Hello, I got a problem. I usually download multiple albums in one batch copying the URL on Tidal-dl, but to use my API Key to download in MAX I use Tidal-gui.exe. But in this app I don't know how to download multiple links, I did the same thing that I do on tidal-dl and it doesn't work.
Add screenshots
No response

You can add 1 link 1 by 1 and they will queue to download.

@coolkidnumber01 can you explain me how I can reproduce the issue? How can I see it working in tidal-dl and not working in tidal-gui.exe?

i cannot download tidal gui

@liltk it can be downloaded here:
https://github.com/yaronzz/Tidal-Media-Downloader-PRO/releases/download/<IP_ADDRESS>/tidal-gui.exe
Also, if you, here's my updated version from my fork:
https://www.dropbox.com/scl/fi/jyi1thrnxhoqkp8aofgdm/Tidal-Gui.zip?rlkey=yxe5uddi1dvq98b2h5f173tua&dl=0

i do it follow you but it not work. i can not download pls.... how to do   ?
บนวันจันทร์ที่ 13 พฤษภาคม 2024 เวลา 09 นาฬิกา 29 นาที 21 วินาที GMT+7, Pavel, @.***> เขียน:
@liltk it can be downloaded here:
https://github.com/yaronzz/Tidal-Media-Downloader-PRO/releases/download/<IP_ADDRESS>/tidal-gui.exe
Also, if you, here's my updated version from my fork:
https://www.dropbox.com/scl/fi/jyi1thrnxhoqkp8aofgdm/Tidal-Gui.zip?rlkey=yxe5uddi1dvq98b2h5f173tua&dl=0
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>

@coolkidnumber01 can you explain to me how I can reproduce the issue? How can I see it working in tidal-dl and not working in tidal-gui.exe?

@MozgC hello, in the .exe version I put a lot of links together to get downloaded, separate them with commas, in the version that I execute on anaconda prompt it works fine, it downloads all the links I type. But if I use the .exe version, I can't download the same way, just typing one by one link. And I don't know how to insert my access token in tidal-dl on anaconda to download the file on Hi-Res an not MQA


NULL

When typing /quests. It gives a NULL message.
Preview:
http://imgur.com/F2hlRta

idk if i have permission to help you or not
But U need to have Newest Quest Version And Vault Installed

@ZeeKingEpic I am already using the latest version of Quests (Specifically Spigot) and have Vault installed.

@ThunderGemios10  OK i had the same problem i just fixed it yesterday i ll give u both Jars
https://www.dropbox.com/s/2u2t7hl1kkug865/Quests (1).jar?dl=0 ( QUESTS JAR )
https://www.dropbox.com/s/o424d8ipgmbfujv/Vault (1).jar?dl=0 ( VAULT JAR )

I'm having the same issue.

@meegamax Make sure you are using the latest version from Spigot and NOT the old version from Bukkit. https://www.spigotmc.org/resources/quests.3711/
If you're still having an issue after updating please create a new ticket. This one is closed.


Enriching validators responses

Hi,
Thank you very much for your contribution! it is super awesome!
I'm still missing here some kind of message system, so we can know which validation rule has failed and show the correct message to the user.
Also, if result and pipe will be async we could also have async validation.
Do you have any ideas for these subjects and how to implement it with current state of the project?
Maybe implementing Future type? and compose each validator with a Message type?

Hello @alonronin
I think you're right and yes, I received the same feedback from reddit as well!
This project has started as a basic exercise of FP, then has developed in something more structured. Now that has been released I think having a better response from the composition of each validator should be the next step!
We agree that a boolean does not inform on which part of the validation fails.
I like the idea of composing errors, something like:
isString({ invalid: true }); // -> {  valid: boolean, error: Error }

const myShapeValidator = hasShape({  key: isString, num: isNumber });

myShapeValidator({ key: 10, num: 10 }); // -> {  key: { valid: boolean, error: Error },  num: { valid: true } }

What do you think?
About the async validation i would open another topic, definitely a future implementation :)

it looks great, but i can't understand how you'll define the message for the error.
maybe something like:
const isUserName = compose(isString( 'must be a string'), minLength(4)('at least 4 characters needed'))

const validate = hasShape({  userName: isUser, num: isNumber });

try {
 validate({ userName: '' })
} catch (e) {
  console.log(e.errors) // { userName: 'must be a string'  }
}

try {
 validate({ userName: 'abc' })
} catch (e) {
  console.log(e.errors) // { userName: 'at least 4 characters needed'  }
}

validate({ userName: 'admin' }) // true

we could also have sane default for messages for each built-in validator.

@alonronin
that looks outstanding!
I'll think about it this evening and I shall come with a new proposal

Hello @alonronin
I've thought quite a lot about restyling the response from validators and validator compositions and I came up with the following solution, each validator will return an object telling whether the validation is successful and the possible error, something like this:
{ valid: Boolean, error: null | ErrorString }

whilst the composition of more validators will return something like:
const myValidator = compose(isString, minLength(4));

myValidator(value);
/* composition result looks like this:
{
  valid: Boolean,
  errors: ErrorString[],
  detail: {
    isString: { valid: Boolean, error: ErrorString },
    minLength: {  valid: Boolean, error: ErrorString  }
  }
}
*/

The reason for this choice are:

Sometimes you just need to check if the validation is complete (here it comes the valid property)
Sometimes you don't need to know which part of the validation failed, you just need the errors (here it comes the errors array)
The detail property reports which validation failed and why (it can also be a nested object of other compositions). This approach assumes that each validator has a function's name, if the provided validator has no function name, we can show anonymous as an array

The idea to provide custom errors to each validator as a further argument sounds good to me:
const isUserName = compose(
  isString('must be a string'),
  minLength(4)('at least 4 characters needed')
)

The result of this validation should then be the following:
const result = isUserName("foo");

/* result is:
{
   valid: false,
   errors: ['at least 4 characters needed'],
   detail: {
      isString: {  valid: true },
      minLength: {  valid: false,  error: 'at least 4 characters needed' }
  }
}
*/

What do you think?

@antonioru
you can take inspiration from Joi. they return an object like so:
schema.validate({ username: 'abc', birth_year: 1994 });
// -> { value: { username: 'abc', birth_year: 1994 } }

schema.validate({});
// -> { value: {}, error: '"username" is required' }

Joi also make transformations and support default values so maybe if we wish to support that it make sense.
if you have and error you don't have value and vice versa.

Hey @antonioru really like the project. I'm adding deep-waters to a project right now!
About this issue, I like your solution 👍 . Only comment is to add metadata to the error object to better communicate to developers/clients what the problem is. Specifically, I'm thinking expected, actual, and value.
E.g.
const result = isUserName("foo");

/* result is:
{
   valid: false,
   errors: ['at least 4 characters needed'],
   detail: {
      isString: {  valid: true },
      minLength: {
        valid: false,
        error: 'at least 4 characters needed',
        expected: 'at least 4 characters',
        actual: '3 characters',
        value: 'foo'
     }
  }
}
*/

This could be better suited as a feature request to move to another issue. It would be easy to extend the error objects with more data with a general solution like this.

@antonioru
we can take inspiration from Joi. they return an object like so:
schema.validate({ username: 'abc', birth_year: 1994 });
// -> { value: { username: 'abc', birth_year: 1994 } }

schema.validate({});
// -> { value: {}, error: '"username" is required' }

Joi also make transformations and support default values so maybe if we wish to support that it make sense.
if you have and error you don't have value and vice versa.

@alonronin
In the future perhaps we could introduce few sanitising utilities but for moment I prefer deep-waters to remains a validation-only library as I find it a more attainable goal.
For this reason I don't think reporting the received value in the response message (as suggested by @rayepps as well, btw thanks mate) it's somehow useful.
For the instance, any use case would be something like:
// create validator
const myValidator = compose(myFn1, myFn2, others...);

// usage
const result = myValidator(value); // <-- you have the value right here

console.log(result);
/*
{
  valid: false,
  errors: ['foo', 'bar'],
  detail: {
     foo: {  valid: false, error: 'bar'   },
     bar: {  valid: false, error: 'foo' }
  }
}

now... if you need to access the value, you have it right there before using the validator.
*/

Regarding the idea by @rayepps of introducing the expected and actual prop within the detail sub-objects, it is definitely a good idea, unfortunately it increases the number of parameters and evaluations one have to consider in creating a custom validator.
The final goal is not only to use the built-in validators but to encourage the creation of custom ones.
That said, I'll start this very weekend in developing the requested features and in addition I'll add the following one:

memoization // validators should cache the response to boost performaces
createValidator // an easy-to-use thunk to create custom validators accordingly to our proposed definition

I'm very grad of your contribution guys :)
And of course, feel free to share any other idea or if you want to join in me I'll be flattered

Thanks for the quick and detailed replies @antonioru! I'd like to contribute as well. I pulled the project and got familiar but I expect any a change to simply convert boolean returns to objects could get complex so I'm holding off making changes. I'm sure like most creators, you have a backlog of todo items in your head. If you would like help with them just make an issue with info and tag me 🍻

Thanks for the quick and detailed replies @antonioru! I'd like to contribute as well. I pulled the project and got familiar but I expect any a change to simply convert boolean returns to objects could get complex so I'm holding off making changes. I'm sure like most creators, you have a backlog of todo items in your head. If you would like help with them just make an issue with info and tag me 🍻

Hi @rayepps
thanks for your comment and I apologies it took me 3 days to reply,
I'm currently studying the best way to refactor the library and return the response object we've defined without writing everything again from scratch.
The best solution I've found so far is to apply a Kleisli composition using monads to represent the error/response.
I'll let you know soon, and again: thank you so much for showing interest in this project

Hello!
I would like to contribute to this project, but i don't know what i could do. May you help me? I'm minded to invert the necessary time, but i need to know how i can help.

Hello @CarlosPerezMoreno
I am flattered, thank you very much!
At the moment I am working on a quite-big refectory of the library in order to allow validators to return a proper customisable and composable error message.
If you want to contribute (and I shall be glad of it), perhaps, you can help me with the refactory: there will be a big number of validator tests to be updated and some validators shall be wrapped into a higher order function called  withError.
I'll be able to present the whole refactory approach and the whole idea under this very same post in few days, I apologise it is taking so long.  I'm sorry.
Until then I have to beg you to wait so I'll have a clear outlook on how precisely you or anyone else can help me!
I hope you don't mind.
I'll keep you all posted, thanks

@antonioru
Okay, got it. Well, i'm beginner in this world; i barely can use JS, and now i'm starting with React. My knowledge is very limited, but i'm excited about learning and contribute. So, we keep in touch! Thanks!

Hello guys,
I've finally managed to properly think to a solution to the problem!
My approach has been to create a new thunk functions called withError.
type Response = {
  valid: boolean,
  error?: string,
}

withError(fn: Function, error: string, validatorName?: string) -> fn -> Response

it takes a validator fn and an error string then returns a new validator (behaving as the provided one) that returns a "response object".
It also accept a validatorName that will be "hidden" within the response object prototype (it will be used later to compose the detail property of the composition response), if not defined it will try to guess the validator's name from the fn.name property.
It could be used as following and lead to an quick and easy refactory:
const isSomething = (value) => ( ... );

export default withError(isSomething, 'Is not something');

Then I've edited the compose function to merge the responses from all the various validators and create the detail property, it uses a monad object called ResponseMonad to do so.
So, in a nutshell it will be uses this way:
const isZero = withError((value) => value === 0, 'Not Zero', 'isZero');
const isSomethingElse = withError(() => ..., 'Foo Error', 'isSomethingElse');

isZero(0); // -> { valid: true };
isZero(2); // -> { valid: false, error: 'Not zero' };

const composedValidator = compose(isZero, isSomethingElse);
composedValidator(2);

/*
 * {
 *     valid: false,
 *     errors: ['Not zero', 'not something else'],
 *     details: {
 *        isZero: { valid: false, error: 'Not zero' },
 *        isSomethingElse: { valid: false, error: 'Foo Error' },
 *     }
 *  }
 */

You can see first draft of this solution here.
Also @rayepps provided an alternative implementation that chuffed me to bits, you can see it here #3 (thanks mate, you're the bee's knees!!!) ✌✌✌
Let me know what you think! 😅
BUT.... to be honest I see some problems with the compose function response:

the details property is utterly expansive to calculate as you need to merge a ton of objects 🧐
the details property it's not so deterministic as it should be and there's no way to be sure that one of the composed validators has somehow "hacked" its outcome... I mean it can be easily broken by using withError with anonymous functions or by providing "hacky" names:

// hackish name provided for isZero validator that could override isBiggerThan results
const isZero = withError((value) => value === 0, 'Not Zero', 'isBiggerThan');

/* use withError in the composition process without passing the validator name will cause a nonsense response */
const myValidator = compose(isZero, isBiggerThan(3), withError((value) =>  value < 5, 'err'));

In the attempt of finding a solution to this problem, I've tried to reshape the detail property but  I've been pondering on this matter: what is the value that the detail property brings to the user?
Of course, it can be useful for debugging purposes... on the other hand the errors array is there  for the very same reason and is definitely less verbose.
Probably we don't need the detail property, as what you want to archive with a validator is something like this:

and it can be archived with the errors array only.
( of course there are probably few exceptions, such as hasShape... but it can be managed )
So I've tried to get rid of the details property and I've created this other branch, I've kept withError but it does need the validator name and I've changed the compose function.
Here's the branch:
https://github.com/antonioru/deep-waters/tree/feature/with-error
As a result I think it's faster, it's less confusing for the users and it's easy to create new validators, which is actually one of the final goals of the library.
Please let me know what you think,
cheers

@antonioru it looks interesting, i didn't had time to play with it.
however, i have an idea for an abstraction that can be done by the user and not in the library, so the lib will stay agnostic to how the user uses it.
when i'll have a spare hour i'll write something quick to demonstrate the usage of it.


SAK-29452 - Force site update when a tool is added to a site

Force site update when a tool is added to a site

Error with the PR

Closed by duplicated,
https://github.com/sakaiproject/turnitin/pull/4


Flutter fails to report keyboard changes via MediaQuery.of(context).viewInsets

Steps to Reproduce

Load this code into the simulator.
Tap the two inputs, pressing CMD-K (on Mac, obviously) to show the keyboard if necessary.
Notice the debug statements which always read 0 bottom inset.

Target Platform: iOS & Android
Devices: iPhone 11 Pro and Pixel 3
Logs
flutter doctor -v
[✓] Flutter (Channel stable, v1.12.13+hotfix.5, on Mac OS X 10.15.1 19B88, locale en-US)
    • Flutter version 1.12.13+hotfix.5 at /Users/craiglabenz/flutter
    • Framework revision 27321ebbad (5 weeks ago), 2019-12-10 18:15:01 -0800
    • Engine revision 2994f7e1e6
    • Dart version 2.7.0

[✓] Android toolchain - develop for Android devices (Android SDK version 28.0.3)
    • Android SDK at /Users/craiglabenz/Library/Android/sdk
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-29, build-tools 28.0.3
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1343-b01)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 11.3)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 11.3, Build version 11C29
    • CocoaPods version 1.8.4

[!] Android Studio (version 3.4)
    • Android Studio at /Applications/Android Studio.app/Contents
    ✗ Flutter plugin not installed; this adds Flutter specific functionality.
    ✗ Dart plugin not installed; this adds Dart specific functionality.
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1343-b01)

[✓] VS Code (version 1.41.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.7.1

[✓] Connected device (1 available)
    • Android SDK built for x86 • emulator-5554             • android-x86 • Android 10 (API 29) (emulator)

@craiglabenz I noticed you have your InputsPage widget inside of a Scaffold.
The Scaffold widget has a property "resizeToAvoidBottomInset" which defaults to true. This essentially "eats" the keyboard inset and applies padding to the child widget, then passes 0 inset in MediaQuery to the child as the padding has already been applied.
I tried running your example and setting "resizeToAvoidBottomInset" to false in the Scaffold and was able to see the padding value for the keyboard output to the console.
Try setting resizeToAvoidBottomInset to false and seeing if that resolves your issue.


Search and match variable length wildcards

There is a middle ground between fixed length sequences that byteseek currently supports, and full regular expressions (which needs a lot more dev and test).  We can support wildcards between sequence elements.
.* wildcards: A sequence like 01 02 .* 03 04 can be matched easily (01 02, then search for 03 04).
.{1,4} variable length wildcards.  A sequence like 01 02 03 .{1,4} 04 05 can be matched as 01 02 03, then search for 04 05.
Also need to support backtracking (optionally).

To support all of the matching performed by the DROID application, it would also have to support alternative sequences ('one' | 'two' | 'three').  This would be a bit more complex than just having variable length wildcards, but probably not a lot more.


Table cell action duplication issue

Package
filament/tables
Package Version
v2.16.26
Laravel Version
v9.34.0
Livewire Version
v2.10.7
PHP Version
8.1.3
Problem description
I am defining a table cell action in a resource, but the action also appears in the action section at the end of the table.

Expected behavior
Action should appear only in cell
Steps to reproduce
Define a table and add a cell action with an action modal
Reproduction repository
https://github.com/molcsab/filament-actionbug
Relevant log output
No response

Fixed by #4454.


Add PR triage dashboard

This is discussed at https://rust-lang.zulipchat.com/#narrow/stream/242791-t-infra/topic/Triage.20PR.20dashboard.
Background
from https://rust-lang.zulipchat.com/#narrow/stream/242791-t-infra/topic/Triage.20PR.20dashboard/near/228242804

Since we struggle with triaging Clippy PRs in a timely manner, Takayuki Nakata put in the work to make triaging easier. On the one hand for people doing the triaging and on the other hand for reviewers to quickly get an overview on which PRs they should take another look.
Zulip discussion (includes some screen shots how it currently looks)
A prototype, inspired by the bors dashboard, is ready to try out here: https://toriage.herokuapp.com/ (Webapp may be slow, because free webapp hosting)
Since we think that this dashboard isn't only useful for Clippy, but for every higher-traffic rust-lang project, we want to get the opinion from the infra team and if you would be interested to host this dashboard, similar to bors.rust-lang.org?

Changes
I added the PR triage dashboard to triagebot, for example accessing to  triage.rust-lang.org/triage and  triage.rust-lang.org/triage/rust-lang/rust-clippy, and getting a dashboard view.
There is original codes  in https://github.com/giraffate/toriage/tree/8f7c24d0d2165d31b74f880a7282bd72dc312b3e, but I have some updates based on https://rust-lang.zulipchat.com/#narrow/stream/257328-clippy/topic/can.20we.20automate.20triaging.20PRs.20a.20bit.3F/near/228486879.

Add a column of last updated_at (e.g. 1 day)
No pagination to implement some filter in future work

This looks good to me - I think we can iterate further once it's merged, in subsequent PRs, for additional refactoring etc.


Update to hcl2

Fixes https://github.com/gruntwork-io/terratest/issues/784
This updates the var-file.go routines to use hclv2 instead of hclv1. Note that this is a backwards incompatible change due to the change to make GetAllVariablesFromVarFileE more ergonomic for both the user and code maintenance.

Thanks for review! Test passed (except for GCP, which is not relevant to this PR), so going to merge this in now.


Game drops to 35 fps during gameplay

The game drops to around 30-35 fps during gameplay. I have my monitor set to 70hz and vsync set to 70. The song select menu runs at a straight 70 fps (as expected), but during gameplay, there is a significant frame drop. This doesn't happen every time I open the game, but it's sort of an on/off kind of thing. I have played the game at 70 fps before, so I know it's not a hardware issue. I've also tried turning off vsync. Turning of vsync makes it smooth_er_, however the fps varies, making it a bit janky. I'm thinking it might be a memory leak of some sort (idk). I attached a screenshot of my monitor resolution/refresh rate for a bit of context. This is Etterna 0.68.1 running on Windows 10 64-bit.

Thank you!

No memory leak.
What framerate does the game normally run at in gameplay with vsync off?

With vsync off, it hangs around the 57-58 fps at 70 hz during gameplay

also just as a note the game runs fine at 60hz with vsync

Am I misunderstanding something or is that not just kinda how vsync is supposed to work? You might want to tweak your game settings until you can reach a stable 100+ fps without vsync (I'm not really sure which options would help the most but I think a smaller resolution, a dark background image, and a chart without many notes, may make a good test case) and then try it again

I'm providing a video to put this issue into context. https://youtu.be/7YqkIP3_5Qs

What kind of framerate do you get with vsync off, default refresh rate?

Update: I turned off BG's in the song select and turned bg brightness down to zero and that seemed to fix the issue. I can run at 70fps with vsync and about 172 fps without vsync. Thanks for the suggestion


Address CRAN comment; re-render REAMDE.Rmd

Got rejected from CRAN because of a minor issue with the README. Screenshot of email below.

I addressed this by just chaining it to link to the actual code of conduct in the repo. Should address it, I think.

Just re-submitted to CRAN.


Opt-in to rustc_private for rust-analyzer

rust-analyzer/rust-analyzer#7891

Huh, that's weird. Maybe worth reopening to see if this time bors feels okay?

It appears to be merged anyway

Oh, you're right. Thanks again!


Fixing login and signup issues on local dev environment

Description
Fixing login and signup issues on local dev environment
Type of change
Please delete options that are not relevant.

[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 not work as expected)
[ ] This change requires a documentation update

Change Status

[x] Complete, tested, ready to review and merge
[ ] Complete, but not tested (may need new tests)
[ ] Incomplete/work-in-progress, PR is for discussion/feedback

Checklist

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] There are no merge conflicts

Login and Signup still need work, just cleaned up migrations


Add nautical measurements (Fathoms, Leagues)

Problem Statement

The Conversions tab for length already includes Nautical Miles, but not other nautical-related units.
Evidence or User Insights

These units would be helpful for people working on boats or ships. Fathoms are typically used to measure water depth, and Leagues are an older measure for distance.
Low-Fidelity Concept

Requested Assignment

If possible, I would like to implement this.

This is your friendly Microsoft Issue Bot. I've seen this issue come in and have gone to tell a human about it.

Hey! Can I work on this?


🚸Should Autocomplete open on focus?

The fact that Autocomplete dropdown opens on focus instead of click/enter key can sometime be a nuisance, such as when it is the first element in a dialog. We should look into the wai-aria recommandation, other libraries (mui etc) and see if we should change it to explicit click/enter button (and that this works with downshift)

resolved by #2996
Autocomplete now opens on click instead of focus


Max retries exceeded with url:

HTTPSConnectionPool(host='b2vapi.bmwgroup.com', port=443): Max retries exceeded with url: /gcdm/oauth/token (Caused by NewConnectionError('<urllib3.connection.VerifiedHTTPSConnection object at 0x7235f5d0>: Failed to establish a new connection: [Errno 111] Connection refused'))
Traceback (most recent call last):
File "/usr/local/lib/python3.7/site-packages/urllib3/connection.py", line 157, in _new_conn
(self._dns_host, self.port), self.timeout, **extra_kw
File "/usr/local/lib/python3.7/site-packages/urllib3/util/connection.py", line 84, in create_connection
raise err
File "/usr/local/lib/python3.7/site-packages/urllib3/util/connection.py", line 74, in create_connection
sock.connect(sa)
ConnectionRefusedError: [Errno 111] Connection refused
What is the maxmal number of retries ? I did not retrie the data too often. Is it the possible to limit the frenquence ?
Thank you

Apparently BMW is experiencing some major hickups...
The ConnectedDrive website and the Android app are unavailable as well.

Can confirm this issue affecting me in the UK also.
Connections to http://b2vapi.bmwgroup.com/ fail, whereas connections on the US version (https://b2vapi.bmwgroup.us/) are still being answered, sort of.
I had a similar issue when I purchased my vehicle earlier in the year. I spent an hour or two on the forecourt with various people from the dealership trying to work out why we couldn't register the vehicle/use the app etc. It was only when the dealership's expert returned from lunch did he tell us that they update the platforms on a Saturday and it doesn't always go to plan. I confirmed this when I telephoned the ConnectedDrive helpline and a recent message on the IVR stated that there were issues. It sorted itself out a day or so later.
Hopefully this issue is related to server downtime at BMW's end and doesn't require any massive changes to the component.
Given that the .us address seems fine, is there anyone in the US having issues at the moment, or is just just the rest of the world (outside of China)?

For me it is working again (app, web and bimmer_connected)...


Modify how contained_by works

Central discussion thread is https://github.com/ShiftLeftSecurity/codescience/issues/4038

this is just WIP right?

I will delete the now commented-out code before merging. However, you should tell me if I missed some other changes that are required due to this change.

ok I didn't pay much attention up until now because this looked too much like wip
Anyway, yes this should be ok as is, even if it's still using the tinkerpop api. If I port that over to the odb api now or later shouldn't really matter (as long as all tests are green obvsly).

Why dont we need special handling for list properties anymore?

There is some nebulous gremlin tinkerpop API for modifying graphs. This nebulous API appears to require this special handling (?)
We know and control every single call to all the modifications: The are done by the generated code, and triggered by diffgraph application, proto deserialization and overflowdb loading. As long as we keep our stuff compatible and don't trigger the JVM's type-errors, we can pass whatever we want where ever we want and have it mean what ever we want. So we don't care about interoperability with anything else for graph modification (we're still re-using the upstream vtable entries, though).
So I changed how the API works for us: Properties with list cardinality used to be passed one after the other, and afaiu used to have limited guarantees on the order of results; semantically, they were appended.
Now they are passed as a List, and always overwrite the previous entry and always preserve order.

Yup that analysis is correct. we used to go via the tinkerpop api to handle the lists, adding one item after the other. But since the NodeDeserializer always wanted the entire list, there wasn't really a good reason for it anyway.
Now we're just setting the entire list at once, which is good.
However: note that we now don't intern Strings that are part of a List any more... not sure from the top of my head what the best way to handle those are. Maybe @bbrehm can take a look first and I can revisit this later when I have more time.


Feature Request: setting to collapse input cells by default for Python interactive

Create setting to collapse input cells by default when ran in the Python Interactive window. Allows more room for output results within a vertical view without repeating code from the source file.

see #3266

This is already in progress :). Issue #3266


Accessibility : DropDownMenu and items not exposed

I am trying to do a simple "script" selector using a DropDownMenu.
Here is the sample code (script can be mocked here has a sample object with an id and a name field)
@Composable
fun ScriptSpinner(
    label: String,
    list: List<Script>,
    onSelectionChanged: (selection: Script) -> Unit
) {
    var hasSelection by remember { mutableStateOf(false) }
    var selected by remember { mutableStateOf(list[0]) }
    var expanded by remember { mutableStateOf(false) } // initial value

    Box {
        Column (Modifier.padding(5.dp), Arrangement.SpaceBetween) {
            Row (
                Modifier.clickable { expanded = !expanded }.then(
                    Modifier.semantics() {
                        contentDescription = "Select a script to run"
                    }
                )
            ){ // Anchor view
                Text(text = if(hasSelection) selected.name else label)
                Icon(imageVector = Icons.Filled.ArrowDropDown, contentDescription = "open list")
            }
            DropdownMenu(
                modifier = Modifier.fillMaxWidth(),
                expanded = expanded,
                onDismissRequest = { expanded = false },
            ) {
                list.forEach { entry ->
                    DropdownMenuItem(
                        modifier = Modifier.fillMaxWidth().then(
                            Modifier.semantics {
                                contentDescription = "${entry.name} script"
                            }
                        ),
                            onClick = {
                                selected = entry
                                expanded = false
                                hasSelection = true
                                onSelectionChanged?.invoke(selected)
                        },
                    ){
                        Text(
                            text = (entry.name),
                            modifier = Modifier.wrapContentWidth()
                        )
                    }
                }
            }
        }

        Spacer(
            modifier = Modifier
                .matchParentSize()
                .background(Color.Transparent)
                .padding(10.dp)
                .clickable(
                    onClick = { expanded = !expanded }
                )
        )
    }
}

When trying to inspect the expanded menu with the Accessibility inspector, the inspector does not find the menu or the items in it.
Did I miss something that needs to be added to the semantics for the dropdown menu to be accessible to the inspector and screen readers ?

This is most probably the same as #2120 and #2041

Please check the following ticket on YouTrack for follow-ups to this issue. GitHub issues will be closed in the coming weeks.


"image uploaded" SavedSearch does not detect image upload

glance image-create --name 'rdo-kilo' --progress --file ~/devel/images/RDO-kilo-20160204.ova --disk-format vmdk --container-format ova
does not result in any hits.

I'm not seeing an error here.  This what I get after uploading an image:

Go ahead and close with a “can’t reproduce”

On May 12, 2016, at 6:10 PM, Vic Hargrave<EMAIL_ADDRESS>wrote:
Assigned #541 https://github.com/Solinea/goldstone-server/issues/541 to @jxstanford https://github.com/jxstanford.
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub https://github.com/Solinea/goldstone-server/issues/541#event-659361932

Can't reproduce.


Add source location to expression

For improved error messages, the Expression type should have a source reference.
There are two options to implement that:

Add it as a field to every expression variant and have a trait that can be used to retrieve the source ref.
Add an intermediate type such that Expression is a struct that contains a source ref and the current Expression type.

I'm slightly leaning towards option 1, but am also open for option 2.
In the future, we might want to have different expression types that might contain more or less variants, but I think this is compatible with both ways to do it.
We might also want to have a source ref for AlgebraicExpression. For that, it might be better to implement option 1, so that the handling is uniform using the trait.

I'm leaning towards option one as well. Even in that case, if Expression enumerates parsed expressions and the internal structs represent the expression itself, I would directly rewrite them as
pub enum Expression<Ref = NamespacedPolynomialReference> {
    Reference(Ref, SourceRef),
    PublicReference(PublicRefence, SourceRef),
    Number(Number, SourceRef),
    String(String_, SourceRef),
    Tuple(Tuple, SourceRef),
    LambdaExpression(LambdaExpression<Self>, SourceRef),
    ArrayLiteral(ArrayLiteral<Self>, SourceRef),
    BinaryOperation(BinaryOperation<Self>, SourceRef),
    UnaryOperation(UnaryOperator<Self>, SourceRef),
    IndexAccess(IndexAccess<Self>),
    FunctionCall(FunctionCall<Self>),
    FreeInput(FreeInput<Self>, SourceRef),
    MatchExpression(MatchExpression<Self>, SourceRef),
    IfExpression(IfExpression<Self>, SourceRef),
    BlockExpression(BlockExpression<Self>, SourceRef),
}

where every entry (especially Number, String, BinaryOperation, etc) becomes
    ExpressionEnumerated(ExpressionStruct, SourceRef)

Let's take advantage of the fact that we are going to add breaking changes in the open PRs of everyone in the team anyway :joy:
What do you think?

This has been implemented mostly.


Multiple instances of Gitrob

Is there a way to have multiple instances or maybe just a way to have more than one type of scan? Currently I'm doing a scan of both public and private repos for an organization but both scans are combined in the same database in PostgreSQL.
I think it would be useful to have a feature where you can do separate scans that separate the results into two or more databases so you can easily pull data from the one that you want. Thanks.

Hi Andrew,
Curently Gitrob does not support easy switching between databases, however a solution I can see right now would be to have a folder with two different .gitrobrc files which each have a different database connection URI. You can then switch between the two by creating symbolic links, e.g.:
$ ln -s gitrob_configs/gitrobrc_db1 .gitrobrc
$ gitrob analyze org1
$ unlink .gitrobrc
$ ln -s gitrob_configs/gitrobrc_db2 .gitrobrc
$ gitrob analyze org2

I will add a way to specify a .gitrobrc location with a --config flag to the list of features to add.
Cheers,
Mike


supervisor died

[20:29:12] INFO: Sending command: [setBetriebArtM1]: WW [71] Mon Jan 30 20:29:48 2023 : Client connected <IP_ADDRESS>:45036 (FD:5) [19:29:48] WARNING: Halt add-on with exit code 1 s6-rc: info: service legacy-services: stopping [71] Mon Jan 30 20:29:48 2023 : Received SIGTERM [19:29:48] INFO: vlclient_pub Service restart after closing s6-svwait: fatal: supervisor died s6-rc: info: service legacy-services successfully stopped s6-rc: info: service legacy-cont-init: stopping [19:29:48] WARNING: Halt add-on with exit code 1 [19:29:48] INFO: vclient_sub Service restart after closing s6-rc: info: service legacy-cont-init successfully stopped s6-rc: info: service fix-attrs: stopping s6-rc: info: service fix-attrs successfully stopped s6-rc: info: service s6rc-oneshot-runner: stopping s6-rc: info: service s6rc-oneshot-runner successfully stopped
i found a new bug, when i trigger a set command, the addon crashes completely. but the command is executed correctly!

Did you have tried with Version 1.8.2? In my environment, it's working fine.

jep, its the 1.8.2!

Hmm, might be that now the return value parsing works as intended... 🤔
@Bjoern3003 can you check what set-command caused the crash? What device id are you using?

How should I check the chrash? I can not reproduce the crash in my setup.

Oh, sorry, wanted to mention @Razzer90. Can you give some more details?

Ok, now I'm getting the same after a few minutes...
[21:01:52] WARNING: Halt add-on with exit code 1
s6-rc: info: service legacy-services: stopping
[21:01:52] INFO: vlclient_pub Service restart after closing
s6-svwait: fatal: supervisor died
s6-rc: info: service legacy-services successfully stopped
s6-rc: info: service legacy-cont-init: stopping

@Razzer90 @Bjoern3003 In my case it was concurrent telnet connections from my actual HA instance, a testing instance and a local telnet client messing up everything... I could see many connections in the vcontrold log and failing commands because there have been calls from different clients at the same time... Exiting the clients and restarting the service fixed it for me.

@Razzer90 @Bjoern3003 In my case it was concurrent telnet connections from my actual HA instance, a testing instance and a local telnet client messing up everything... I could see many connections in the vcontrold log and failing commands because there have been calls from different clients at the same time... Exiting the clients and restarting the service fixed it for me.

That is the solution! If I trigger telnet commands via terminal, the addon crashes! simply no second telnet access...thanks for the great work!

@Razzer90 - cool, thanks for the feedback, good to know that there is no additional surprise. @Alexandre-io - I think this issue can be closed. More related to vcontrold but good to have it identified and documented 😄.

:+1: thanks for the follow up


Cert Manager v.1.8.2 apply failure

When I apply the Cert Manager v.1.8.2 I get the following error. I have already deleted the old resources. Can you help me?
Resource: "apiextensions.k8s.io/v1, Resource=customresourcedefinitions", GroupVersionKind: "apiextensions.k8s.io/v1, Kind=CustomResourceDefinition"
Name: "orders.acme.cert-manager.io", Namespace: ""
for: "https://github.com/cert-manager/cert-manager/releases/download/v1.8.2/cert-manager.yaml": CustomResourceDefinition.apiextensions.k8s.io "orders.acme.cert-manager.io" is invalid: status.storedVersions[0]: Invalid value: "v1alpha2": must appear in spec.versions

OK. kubent found resources. I removed these and reapplied. Then it worked.


Fix a bad relref

Fixes a bad relref and adjusts others to be properly root-relative.

Thanks @susanev! 🙏


Fix Vue3: Several issues on Pod detail view

Summary
Fixes #11905

Occurred changes and/or fixed issues

Fix spacing issue in deployments restart count in Masthead

Technical notes summary

1. Extra editor tab -> no need to fix, as per https://github.com/rancher/dashboard/issues/11905#issuecomment-2367670918
2. Extra Namespace column in the table -> Couldn't reproduce. Probably got fixed by some other PR
Areas or cases that should be tested

Areas which could experience regressions

Screenshot/Video

Checklist

[x] The PR is linked to an issue and the linked issue has a Milestone, or no issue is needed
[x] The PR has a Milestone
[x] The PR template has been filled out
[x] The PR has been self reviewed
[x] The PR has a reviewer assigned
[x] The PR has automated tests or clear instructions for manual tests and the linked issue has appropriate QA labels, or tests are not needed
[x] The PR has reviewed with UX and tested in light and dark mode, or there are no UX changes

@mantis-toboggan-md thanks for catching that part of the NAMESPACE col. It's now fixed on this PR


onboarding: onpress foto icon while keyboard is open doesnt work and is weird

cannot replicate on my phone. It closes the keyboard


Why I don't have these icons in magit?

Mine:

Yours:

Everything is installed, doom themes, doom-modeline, all the icons. Anything else I need to install? Also, how does it know what icon to show - for a bug or a feature?

I apologize for misleading you, but those icons aren't part of this package. However, here's a guide on how to get it.

Oh, yes, I had already figured it out before your comment but forgot to close it :) Yes, this is "Pretty Magit" package. I think, we should add a list of packages visible on each screenshot for people who expect these screenshots to become live on their pc after installing doom themes. Such things are fonts, pretty magit on some packages, solaire-mode sometimes. Maybe I missed something, but I think it is a good idea :)