Updating 50076607, 529987, 551029, 451903, 451993

All personal holdings.
I am able to verify 50076607 ("bmo debit card" ones without mastercard logo) and 529987 ("hercus credit card") through Google Images.
Contact info are from card back

Closing because I've updated the format of the list. Please resubmit if you can. Sorry for the trouble!


[nginx] Rate limiting not working

I tried both ingress.kubernetes.io/limit-connections: 1 and ingress.kubernetes.io/limit-rps: 1 and I am still able to do hundreds of requests per second on the endpoint (tested using curl in parallel).
I can see in the nginx log that it picked up the change (I've added the annotations to an existing resource).
Sample from my ingress resource:
  annotations:
    kubernetes.io/tls-acme: "true"
    kubernetes.io/ingress.class: "nginx"
    ingress.kubernetes.io/limit-rps: 1

Tested with nginx-ingress-controller versions 0.8.3 and 0.9.0-beta.8 on Kubernetes 1.6.4

@rochdev please check if the IP addresses in the log are correct (maybe you need to enable proxy protocol if you are running in aws)

@aledbf The only IP I can see in the logs seems to be an internal IP (10.x.x.x). I am using a TCP load balancer created by Kubernetes in GCP.

For reference, I started from this example: https://github.com/jetstack/kube-lego/tree/master/examples/nginx

@rochdev you should use  service.beta.kubernetes.io/external-traffic: OnlyLocal in the service to avoid losing the source IP address. https://kubernetes.io/docs/tutorials/services/source-ip/#source-ip-for-services-with-typeloadbalancer

@aledbf From my understanding this requires having an instance of the controller on every node. Is there an alternative without such requirement?

Also, we only have a single node in development at the moment so according to this document the source IP should not be rewritten in this case.

After making the change you recommended, I now see the correct source IP in the logs, but the requests are still not limited.

@rochdev please post the generated nginx.conf from the ingress controller running kubectl logs <ingress pod> >ngx.log

@aledbf I cannot really share the entire file, but which part am I looking for exactly?
If it's of any use, there are no limit_req, limit_req_zone or limit_conn anywhere in the file.

@rochdev without the logs or the generated conf there's not much I can do to help you debug this issue.
Add the flag --v=3 to the descriptor to increase the log level and get the reason why is not reading your annotations. The limit sections are defined here https://github.com/kubernetes/ingress/blob/master/controllers/nginx/rootfs/etc/nginx/template/nginx.tmpl#L363

With the more verbose log level I was able to find that by setting the annotation to a string instead of a number the rate limiting works properly.
  annotations:
    kubernetes.io/tls-acme: "true"
    kubernetes.io/ingress.class: "nginx"
    ingress.kubernetes.io/limit-rps: "1"

I am new to Kubernetes. Is this the expected behaviour? Are annotations always strings?

@rochdev yes

Closing. Please reopen is you still have question/issues

I'm having the same issue, that rate limiting is not working.

Added use-proxy-protocol: "true" to my nginx-configmap
Added service.beta.kubernetes.io/external-traffic: OnlyLocalto my nginx-service

Here are my config files
apiVersion: extensions/v1beta1
kind: Ingress
metadata:
  name: nginx-ingress
  annotations:
    ingress.kubernetes.io/limit-connections: "1"
    ingress.kubernetes.io/limit-rps: "1"
spec:
  rules:
  - host: foo.example.com
    http:
      paths:
      - path: /
        backend:
          serviceName: nginx-ingress
          servicePort: 443

---
apiVersion: v1
kind: ConfigMap
metadata:
  name: nginx-ingress-configmap
data:
  server-tokens: "false"
  proxy-body-size: 50m
  ssl-dh-param: "default/nginx-dhparam"
  proxy-read-timeout: "3600"
  proxy-send-timeout: "3600"
  use-proxy-protocol: "true"

---
apiVersion: v1
kind: Service
metadata:
  name: nginx-ingress
  labels:
    name: nginx-ingress
  annotations:
    service.beta.kubernetes.io/external-traffic: OnlyLocal
spec:
  type: LoadBalancer
  ports:
    - port: 80
      name: http
    - port: 443
      name: https
  selector:
    app: nginx-ingress

I can confirm that rate limiting rules are written to /etc/nginx/nginx.conf.
Please understand, that I'm not able to share our complete configuration files here. But I will do my best to provide you with any necessary information.
Best
Michael

@michaelfreund How many concurrent requests did you try in your tests? There is a burst setting of 5 times the configured limit, so in your case it's possible that you would be able to do 5 requests per second for a short amount of time.
The burst setting is unfortunately hardcoded and cannot be configured.

@rochdev I did a ab -kc 100 -n 1000 https://foo.example.com. I'm a little bit confused, as the ingress config requests a new load balancer which appears in GCLB panel, while the nginx service already requested a LB which is working properly.

@michaelfreund If running in GKE you need to add the kubernetes.io/ingress.class: "nginx" annotation to the Ingress resource, otherwise it falls back on the GCE ingress controller which creates a new load balancer.

@rochdev This still isn't working btw, I've tried to add kubernetes.io/ingress.class: "nginx" in my configuration file ass well. Any other suggestion?

@rvu95 Can you post the entire configuration?


ENH: move get_true_label_position to utils in classification

Description
1- Move get_true_label_position function from classification.py to a new utils file
2- Add a test for this function
How Has This Been Tested?
1- Check if the return of my function is correct
2- Check if the shape of my function's return is correct

Hey @BaptisteCalot,
Thank you for this PR! For the moment it doesn't pass some tests, but we have fixed the issue. So you can git merge upstream/main and everything should work!
Thank you!


Inconsistent Cursor Position When Navigating Through History Using Up and Down Keys

Steps to Reproduce:

Focus on the input box on the copilot view
Press up arrow key a few times
The previous queries will be populated, the cursor is at the beginning of the text.
Press down arrow key a few times
The next queries will be populated, but the cursor is at the end of the text.

Expectation:
The cursor position should be consistent at the end of populated text.

That standard works well in single-line input boxes, but for multi-line inputs in vscode, I think it makes sense to say that the cursor needs to be at the very first character in the response to navigate back, and then it stays in that first position while navigating back, and it needs to be in the last position while navigating forward, and stays there.


Rename base_path to server_base_path

This method name tripped me up for a bit as
I was thinking it was reading the base path off of disk.
I renamed it to help show that it pulls the value directly from the servers key
of the open api doc.

Do you mind updating the open_api/generator for consistency?

@bdunne I modified the open_api/generator method name to match as requested.


Missing from Base URL fix

https://github.com/Azure/autorest/pull/1271

Hi @jianghaolu, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

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

TTYL, AZPRBOT;

@pomortaz


docker_api: report syntax errors

When the Dockerfile is incorrectly formed, use the exception's "explanation" (the syntax error message) instead of the HTTP error (500 status code).
This includes an expected-fail commit adding the test-case, followed by the actual fix, showing that tests now pass.

Updated release notes draft:

docker_api build plugin will provide explanation when build api call fails.


Make "master" -> "release" and "develop" -> "master"

The release branch is still handy to have a Docker image only created automatically when pushing to the release branch. The master branch will be used for PR merges and continued development. For each release, the final release is merged into and pushed to the release branch.

The release branch is still handy to have a Docker image only created automatically when pushing to the release branch.

Can't this be sync'd when pushing a release tag instead?

Can't this be sync'd when pushing a release tag instead?

I've checked the settings on Docker Hub, and yes, it can be triggered by a tag instead.

Make "develop" the "master" branch

:+1:


Improve team section

As long as we don't have the team pictures yet I don't think it makes sense to merge this PR, please add them 🙂🙏

I thing it makes sense because I changed some crucial things

I think all these people (unless they are already in another team) should be apart of the tech team:

@GresosJ Order the team members by alphabetical order within each team collection plz :pray:

@GresosJ , why there's a cat as default image? What's the point of it?

@GresosJ , why there's a cat as default image? What's the point of it?

Because I thought it was a pretty cat , and that's why


Fix usage message typo

please rebase

Let's close this one, here's a new one:
https://github.com/cheekybits/genny/pull/26


Not working with docker compose

Describe the bug*
I get this error trying to run a python application
java.lang.NoSuchMethodException: com.pty4j.unix.UnixPtyProcess.addRemoteTunnel(int,java.lang.String,int)
To Reproduce
Steps to reproduce the behavior:

Create a python application that runs in a docker container using docker-compose.yml
Add this docker-compose in pycharm as an interpreter
Create a python run configuration and set this docker-compose.yml application as interpreter
Click on run

Screenshots

Desktop or remote:

OS: Ubuntu 22.10 x86_64
M1 chip: no
Reloadium package version: 1.0.0
PyCharm plugin version: 1.0.0
Editor: PyCharm 2022.3.3
Python Version: 3.9.2
Python Architecture: 64Bit
Run mode: Run / debug and fast debug

Fixed in reloadium 1.0.1 and PyCharm plugin 1.0.1 but won't work until this one is fixed https://youtrack.jetbrains.com/issue/PY-53568/Targets-API-Path-mappings-field-is-missing-in-the-python-interpreter-settings
Remote interpreters need path mappings to work with reloadium.

Would it be possible to implement this fix in the legacy method? It can be enable by disabling python.use.targets.api in the Registry.
The feature does however work fine in the targets API. But sadly enough it is to unstable for use.

I've never heard of the legacy method. I'll check it out.
If they don't fix it in the next release I'll add path mappings settings to Reloadium settings to specify it there.

@dkrystki really looking forward for that fix._.


Cannot index / search one file

What steps will reproduce the problem?
1. Index the attached file with cindex
2. Search for a pattern inside it
3. No hits

What is the expected output? What do you see instead?

repro$
repro$ cindex -reset
repro$ cindex badfile
2013/03/13 18:51:52 index /tmp/repro/badfile
2013/03/13 18:51:52 flush index
2013/03/13 18:51:52 merge 0 files + mem
2013/03/13 18:51:52 0 data bytes, 92 index bytes
2013/03/13 18:51:52 done
repro$ cindex -list
/tmp/repro/badfile
repro$
repro$
repro$
repro$ grep main badfile
libc.so.6        __libc_start_main
torch             main
torch              realmain(int, char**)
libglib-2.0....          g_main_context_iteration
libglib-2.0....           g_main_context_prepare
libglib-2.0....            g_main_context_dispatch
#85 0x00000032f5c38f0e in g_main_context_dispatch () from
/lib64/libglib-2.0.so.0
#87 0x00000032f5c3ca3a in g_main_context_iteration () from
/lib64/libglib-2.0.so.0
#93 0x000000000040e74d in realmain(int, char**) ()
#94 0x000000000040e933 in main ()
repro$
repro$ csearch main  <= no results here !!
repro$
repro$ grep threads badfile
============ All threads ==========
============ All threads ==========
repro$
repro$ csearch threads <= no results here !!
repro$

I cannot find (with csearch) text that is in a file I have indexed (cindex)

What version of the product are you using? On what operating system?

I'm using the Linux binaries that are available on the Download page.
I tried to compile go / codesearch but couldn't make it work (my go
install might be funky).

Please provide any additional information below.

It looks like the problem happens at indexing time.

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 14 Mar 2013 at 1:57
Attachments:

badfile

Thanks for the tip. Indeed I've removed those long lines and now everything
works fine. I've seen that your copy of the code has that -maxlinelen that
should be what I need. Now I have to understand how to build a go program ...

Original comment by<EMAIL_ADDRESS>on 14 Mar 2013 at 5:57

Also, I've been using codesearch as part of a webapp at work that does forensic
analysis of crashes (by letting us search through backtraces), and it's amazing
:)

I'm kinda stuck right now because I cannot index some files and I'm thinking
about using a different indexer / search system, but really codesearch is all I
need so if someone can figure out what the problem is that would be awesome.

Thanks !!

Original comment by<EMAIL_ADDRESS>on 14 Mar 2013 at 2:00

Try indexing with -verbose and -logskip flags to see if the file is getting
skipped.

The arbitrary limits are in the source so you can always hand edit and tweak
them. I have a version at

http://github.com/junkblocker/codesearch

which I did to specifically add such options.

Original comment by<EMAIL_ADDRESS>on 14 Mar 2013 at 4:07

Alright, I figured it out, thanks.

repro$ awk '{print length($0)}' badfile | sort -n | tail
972
1001
1043
1071
1456
1529
1724
1792
2259
2328

and in index/write.go there's a
    maxLineLen      = 2000

Original comment by<EMAIL_ADDRESS>on 14 Mar 2013 at 6:36

Also, I have one line that is crazy long: 2245 characters. Maybe the problem is
that the indexer reads line by line and has some hardcoded limit on the number
of chars in a single line ?

Original comment by<EMAIL_ADDRESS>on 14 Mar 2013 at 2:04

Feel free to close the issue whoever can.

Original comment by<EMAIL_ADDRESS>on 14 Mar 2013 at 6:39

I'm going to leave this open until I can get something like -logskip into
the mainline codesearch branch.

Original comment by<EMAIL_ADDRESS>on 14 Mar 2013 at 2:08

I don't know how far you guys should go with that, but having those 2 options
to set the maxLineLen and maxFileSize on the command line would also help.

The default behavior could be to print a message like that (with a better
phrasing probably / different options names) when a file got skipped.

=> /tmp/foo wasn't indexed (maxLine too long) / try to reindex with cindex
-maxLineLen 3000

=> /tmp/foo wasn't indexed (file too big) / try to reindex with cindex
-maxFileSize 1M

Original comment by<EMAIL_ADDRESS>on 14 Mar 2013 at 4:26


Problems with a lot of controls

In Avalonia ui, when I have more then 2000 controls on window, I have a lof  of frizes of gui, the application freezes and responds slowly to clicking on Windows and Linux(Ubuntu) (don't tested on macos)
In wpf for  .net core has no problems with 10 000+ controls
Here is repo with avalonia on gitlab
Here is the same vm+ m , but view on wpf gitlab
steps to reproduce

run app
press button "start"
wait 2-3 minutes
try to press "clear"

I'll take a look at this but note that performance hasn't been a priority in Avalonia until recently (we were concentrating on getting stuff working!), but we're now going through the code and trying to improve performance, so hopefully we can fix this.

Sounds good
I like avalonia, and it works, that cool
belive that it will be fix.

Reusing the shape would also help here. You just draw the same shape at multiple places.


Replace Scenes with custom components not working.

Hi i am using  react-native-router-flux v3.2.6 with react-native v0.22.2 and i would like to split my scenes into smaller components, but its not working. Like this:
instead of
<Router>
                   <Scene key="tab0" component={ FeedView } navBar={ FeedNavBar } titleStyle={ { color: "white" } } />
</Router>

I would like to create my own class Tab0
class Tab0 extends React.Component {
  render() {
    return (
              <Scene key="tab0" component={ FeedView } navBar={ FeedNavBar } titleStyle={ { color: "white" } } />
    );
  }
};

and then just use
<Routerr>
                   <Scene key="tab0" component={ FeedView } navBar={ FeedNavBar } titleStyle={ { color: "white" } } />
</Router>

I am getting the error "Component property is not set for key=root. Is there a way to do that?

Good question, maybe we need to add this topic to README!
You can split your scenes using NodeJS require calls:
            <Router>
                    {require("./scenesForTabBar")}
                    {require("./scenesForAnotherPart")}
            </Router>

scenesForTabBar.js:
import React from 'react-native';
import {Scene} from 'react-native-router-flux';

module.exports = <Scene key="tabbar" tabs={true}>
   // scenes here
</Scene>;

@aksonov thanks, yeah it would be useful to add it to the readme. I see that the way i would like to do it its not easily possible without changing the interface, so your solution is fine, thanks!

I believe this method is better than any possible additional interface, because it requires zero knowledge about that additional interface from code reader...

i mean generally it should be possible to group any another component to a new component without breaking the code, but i understand that this might be hard to implement with the current Scenes etc. anyway i am ok with that and thanks for the fast response.

Honestly i just don't see how to traverse children of component children defined in their render method, maybe you know the way? So if you define some custom component 'Scenes', the component will be not able to iterate Scenes children...

So I would like to implement this possibility however i don't see how to do it with React Native....

Well you can do something like shallowRendering https://facebook.github.io/react/docs/test-utils.html, but I am not sure that is really efficient and the way to go. I don't think that feature is that urgent, as long as there is a workaround.

You should just be able to import the Scene as a component. This is a feature we would really like. Can this issue be reopened?

PR is welcome. I've show a way to accomplish it for current version


Update documentation [skip ci]

Resolves most of #6

Moves examples from README into docs user guide

Single point example (+ Python alternative) left

Add getting started to README/user guide

Loosely depends on #225 for minor changes to CLI, so we should check through this once that is finalised.

I would move in examples also as much as possible from tutorials... but for that we need to be sure we can run then (hence having our own runner for that)... there is a jupyter execute that we may want to use...

I would move in examples also as much as possible from tutorials... but for that we need to be sure we can run then (hence having our own runner for that)... there is a jupyter execute that we may want to use...

I thought we wanted to keep them somewhat separate, particularly as we use a few additional structure files etc, which would make this repository more bloated?
I'm open to alternatives, but I thought we may just want to build very simple examples to add to the python interface section here (plus more detail about each class/parameters), but keep the longer form tutorials separate.
(Either way, I think it's maybe worth doing as a new PR)


fix(sync): change metric instrument type for total_synced_headers

From AsyncGauge to SyncCounter

Counter does not persist it's value over node restart


Minor Indico exception fix

Very minor variable name fix

Not sure what's going on with the Python 3 travis build, merging anyways


shapes (96,86) and (85,) not aligned: 86 (dim 1) != 85 (dim 0)

When i run it it pops up this error, what caused it? Thanks for any advice

Hi @wangzunq,
Thanks for reaching out. It is difficult to ascertain what the cause of this issue is given the context provided.
Could you share more context about your issue?

What version of SigProfiler packages do you have installed?
What commands did you run?
What was the full error message?
What does your input file look like?

ValueError                                Traceback (most recent call last)
Cell In[4], line 1
----> 1 Analyze.cosmic_fit('./clustered_mafs', './results/sequioa_alpine_filtered_clustered_10kb_mafs', input_type="vcf", context_type="96",
2                    collapse_to_SBS96=True, cosmic_version=3.4, exome=True,
3                    genome_build="GRCh37", signature_database=None,
4                    exclude_signature_subgroups=None, export_probabilities=True,
5                    export_probabilities_per_mutation=False, make_plots=True,
6                    sample_reconstruction_plots='pdf', verbose=False)
File ~/anaconda3/envs/py3.9_pytorch/lib/python3.9/site-packages/SigProfilerAssignment/Analyzer.py:132, in cosmic_fit(samples, output, signatures, signature_database, nnls_add_penalty, nnls_remove_penalty, initial_remove_penalty, genome_build, cosmic_version, make_plots, collapse_to_SBS96, connected_sigs, verbose, devopts, exclude_signature_subgroups, exome, input_type, context_type, export_probabilities, export_probabilities_per_mutation, sample_reconstruction_plots, volume)
108 def cosmic_fit(
109     samples,
110     output,
(...)
130     volume=None,
131 ):
--> 132     decomp.spa_analyze(
133         samples=samples,
134         output=output,
135         signatures=signatures,
136         signature_database=signature_database,
137         nnls_add_penalty=nnls_add_penalty,
138         nnls_remove_penalty=nnls_remove_penalty,
139         initial_remove_penalty=initial_remove_penalty,
140         genome_build=genome_build,
141         cosmic_version=cosmic_version,
142         make_plots=make_plots,
143         collapse_to_SBS96=collapse_to_SBS96,
144         connected_sigs=connected_sigs,
145         verbose=verbose,
146         decompose_fit_option=False,
147         denovo_refit_option=False,
148         cosmic_fit_option=True,
149         devopts=devopts,
150         exclude_signature_subgroups=exclude_signature_subgroups,
151         exome=exome,
152         input_type=input_type,
153         context_type=context_type,
154         export_probabilities=export_probabilities,
155         export_probabilities_per_mutation=export_probabilities_per_mutation,
156         sample_reconstruction_plots=sample_reconstruction_plots,
157         volume=volume,
158     )
File ~/anaconda3/envs/py3.9_pytorch/lib/python3.9/site-packages/SigProfilerAssignment/decomposition.py:1023, in spa_analyze(samples, output, input_type, context_type, signatures, signature_database, decompose_fit_option, denovo_refit_option, cosmic_fit_option, nnls_add_penalty, nnls_remove_penalty, initial_remove_penalty, de_novo_fit_penalty, genome_build, cosmic_version, make_plots, collapse_to_SBS96, connected_sigs, verbose, devopts, new_signature_thresh_hold, exclude_signature_subgroups, exome, export_probabilities, export_probabilities_per_mutation, sample_reconstruction_plots, make_metadata, volume)
1018     if genomes.shape[0] == processAvg.shape[0] and collapse_to_SBS96 == True:
1019         sys.exit(
1020             'Signatures Database and Samples are of same context type and is not equal to 96. please rerun by setting the flag "collapse_to_SBS96 = False "'
1021         )
-> 1023 sub.make_final_solution(
1024     processAvg,
1025     genomes,
1026     allsigids,
1027     layer_directory3,
1028     mutation_type,
1029     index,
1030     colnames,
1031     cosmic_sigs=True,
1032     attribution=attribution,
1033     denovo_exposureAvg=exposureAvg_dummy,
1034     sequence=sequence,
1035     background_sigs=background_sigs,
1036     verbose=verbose,
1037     genome_build=genome_build,
1038     add_penalty=nnls_add_penalty,
1039     remove_penalty=nnls_remove_penalty,
1040     initial_remove_penalty=initial_remove_penalty,
1041     connected_sigs=connected_sigs,
1042     collapse_to_SBS96=collapse_to_SBS96,
1043     refit_denovo_signatures=False,
1044     make_plots=make_plots,
1045     export_probabilities=export_probabilities,
1046     export_probabilities_per_mutation=export_probabilities_per_mutation,
1047     samples=samples,
1048     input_type=input_type,
1049     denovo_refit_option=denovo_refit_option,
1050     exome=exome,
1051     volume=volume,
1052 )
1053 if make_metadata:
1054     with open(os.path.join(output, "JOB_METADATA_SPA.txt"), "a") as sysdata:
File ~/anaconda3/envs/py3.9_pytorch/lib/python3.9/site-packages/SigProfilerAssignment/decompose_subroutines.py:958, in make_final_solution(processAvg, allgenomes, allsigids, layer_directory, m, index, allcolnames, process_std_error, signature_stabilities, signature_total_mutations, signature_stats, cosmic_sigs, attribution, denovo_exposureAvg, add_penalty, remove_penalty, initial_remove_penalty, de_novo_fit_penalty, background_sigs, genome_build, sequence, export_probabilities, export_probabilities_per_mutation, refit_denovo_signatures, collapse_to_SBS96, connected_sigs, pcawg_rule, verbose, make_plots, samples, input_type, denovo_refit_option, exome, volume)
941 lognote.write(
942     "L2 Error %: {}\nCosine Similarity: {}\n".format(
943         round(newSimilarity, 2),
(...)
951     )
952 )
953 # remove signatures
954 (
955     exposureAvg[:, r],
956     L2dist,
957     cosine_sim,
--> 958 ) = ss.remove_all_single_signatures(
959     processAvg,
960     exposureAvg[:, r],
961     allgenomes[:, r],
962     metric="l2",
963     solver="nnls",
964     cutoff=initial_remove_penalty,
965     background_sigs=[],
966     verbose=False,
967 )
968 if verbose == True:
969     print(
970         "############################## Composition After Initial Remove ############################### "
971     )
File ~/anaconda3/envs/py3.9_pytorch/lib/python3.9/site-packages/SigProfilerAssignment/single_sample.py:685, in remove_all_single_signatures(W, H, genomes, metric, solver, cutoff, background_sigs, verbose)
680         print("Final Similarity: ", successList[2])
681 H = successList[1]
682 return (
683     H,
684     successList[2],
--> 685     cos_sim(genomes, np.dot(W, H)),
686 )
File <array_function internals>:180, in dot(*args, **kwargs)
ValueError: shapes (96,86) and (85,) not aligned: 86 (dim 1) != 85 (dim 0)

Analyze.cosmic_fit('./clustered_mafs', './results/sequioa_alpine_filtered_clustered_10kb_mafs', input_type="vcf", context_type="96",
collapse_to_SBS96=True, cosmic_version=3.4, exome=True,
genome_build="GRCh37", signature_database=None,
exclude_signature_subgroups=None, export_probabilities=True,
export_probabilities_per_mutation=False, make_plots=True,
sample_reconstruction_plots='pdf', verbose=False)

input files are in maf format with a defined cluster window size of 10 kb that start_pos of muts on same Chr of same sample.

Thanks for sharing that information. Are the input matrices being generated correctly? There should be a directory with logs and error messages in your input directory. For a reference MAF input, you can find a MAF file on our wiki for SBS inputs.

I solved, some .maf has empty df. thanks for pointing out

actually still not working after removing empty maf files. what does matrices dot product (96, 86) and (85,) mean and come from

It means the dimensions do not matchup and this causes an error for matrix multiplication. The dimensions likely don't match up due to erroneous inputs. If you start from a matrix file as input instead of the MAF these issues will likely resolve.

Please re-open this issue if you are still encountering this.


Ignore warnings in "make kernelversion"

@SRIKKANTH Let's work on the same PR next time. Do not open another PR for the same change. It helps to track discussions.


Unable to override default variant hover

UnoCSS version
0.53.6
Describe the bug
I'm trying to override the default variant hover in the Unocss configuration to only apply hover styles when the device supports it. I've tried using the following configuration:
// config
variants: [
  (matcher) => {
    if (!matcher.startsWith('hover:')) return matcher
    return {
      matcher: matcher.slice(7),
      parent: '@media (hover) and (pointer: fine)',
      selector: s => `${s}:hover`,
    }
  }
]

However, it seems that the override for 'hover' variant doesn't work as expected. The generated CSS doesn't intercept the hover variant. Instead, if I change the prefix to any other value like foohover:, it works as expected:
// config
variants: [
  (matcher) => {
    if (!matcher.startsWith('foohover:')) return matcher
    return {
      matcher: matcher.slice(9),
      parent: '@media (hover) and (pointer: fine)',
      selector: s => `${s}:hover`,
    }
  }
]

Expected Output
The hover styles should be wrapped in the '@media (hover) and (pointer: fine)' query as shown in the Playground example when using 'foohover:'.
Actual Output
The hover styles are not wrapped in the '@media (hover) and (pointer: fine)' query, and the override for the 'hover' variant doesn't seem to work.
Reproduction
playground
System Info
No response
Validations

[X] Read the Contributing Guidelines.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

Maybe not a solution to this issue but there is @hover: which does what you want.

Thank you for the suggestions, and this is the approach I'm currently using as well. However, I can't help but wonder if unocss provides the flexibility to override the hover variant 🙏.

@Rock070  you can use the order option to override the default @hover variant util

@zoeyzhao19
Thank you very much for your help. This undoubtedly solved my problem, and I will close this issue.
Once again, thanks to unocss and everyone who kindly assisted me.


MongoDB: Command failed (OperationNotSupportedInTransaction): Cannot run command against the 'admin' database in a transaction.

The default Mongo connection string you get from DigitalOcean's hosted MongoDB offering looks like this: mongodb+srv://doadmin:30x8...GR2@db-mongodb-nyc3-a1...21.mongo.ondigitalocean.com/admin?authSource=admin&replicaSet=db-mongodb-nyc3&tls=true&tlsCAFile=../ca-certificate.crt.
When you try to run a query on that, you get an error message like this:
PrismaClientUnknownRequestError2 [PrismaClientUnknownRequestError]:
Invalid `prisma.user.deleteMany()` invocation:
  Error occurred during query execution:
ConnectorError(ConnectorError { user_facing_error: None, kind: RawError { code: "unknown", message: "Command failed (OperationNotSupportedInTransaction): Cannot run command against the 'admin' database in a transaction.)" } })
    at cb (/home/runner/work/e2e-tests/e2e-tests/databases/mongodb-digitalocean/node_modules/@***/client/runtime/index.js:33904:17)
    at processTicksAndRejections (internal/process/task_queues.js:97:5) {
  clientVersion: '2.27.0-dev.6'
}

Turns out there is a limitation around default databases like admin for using transactions. We should handle that somehow.

New mongodb:

Can't run transactions on the system collections

Workaround:

MongoDB creates databases on the fly (change to admin1)

Possible improvement

Better error
Add Document

Suggestion: Better error that knows about the system databases and helps you to understand the failure and offers a solution: Change the database name to something that is not a system database and can use transactions.

creating a separate db with a different name worked for me. Avoid using primary DBs with reserved names such as admin or root


Using Anthropic's Prompt Caching with Instructor

I think it would be a great fit for instructor to support Anthropic's new Prompt Caching. E.g. for parsing structural data, it could store the base prompt once and only receive the content-to-parse in subsequent calls to save tokens.

For my use case, I managed to get something working doing the following:

def generate_structured_completion(
    self, system_prompt: str, messages: Sequence[ChatMessage], response_model: type[T]
) -> T:
        client = Anthropic(api_key=api_key)

        # This is for all other single model cases
        # stolen from instructor/patch.py
        if get_origin(response_model) is Iterable:
            iterable_element_class = get_args(response_model)[0]
            response_model = IterableModel(iterable_element_class)  # type: ignore
        if not issubclass(response_model, OpenAISchema):
            response_model = openai_schema(response_model)

        response: PromptCachingBetaMessage = cast(
            PromptCachingBetaMessage,
            client.beta.prompt_caching.messages.create(
                system_prompt,
                messages,
                extra_headers={"anthropic-beta": "prompt-caching-2024-07-31"},
                tools=[response_model.anthropic_schema],
            ),
        )
        tool_calls = [c.input for c in response.content if c.type == "tool_use"]

        if tool_calls:
            tool_call = tool_call[0]
            return response_model.model_validate(tool_call)
       else:
           // no tools were called.

Might be helpful for other, so leaving it here.


Fix for Linux

Changes some \\ into / when printing out the filepath (this mostly just bothered me)
Adds proper compatibility for Linux (and most likely macOS) by converting an inputted ~ at the start of a filepath into /home/alli

i just realised... i couldve done let themesFolder = path.join(bdFolder || dataFolder, "BetterDiscord", "themes"); couldnt i?
owell.


test: adds integration test for x-goog-user-project header

Adds integration test for the population of x-goog-user-project, demonstrating that our patching of google-auth-library-nodejs is working for gRPC libraries.
We've opted to add the test here rather than in nodejs-logging, because this header only has an effect on non-resourced APIs.

@broady this does hit a slightly different code path than relying on Account Default Credentials would, but I think it's a worthwhile tradeoff for the tests being significantly easier to setup. I'm fairly confident in the unit tests we have in the upstream google-auth-library-nodejs at this point, and I mostly wanted to prove with this API that @grpc/grpc-js is actually populating headers as expected.


PETools detected as malware

See https://www.virustotal.com/#/file/8fd1a1cc1a253fd58693c181895d392ba20fc2a638aaecbc2e8f5d004db8fc27/detection
Why is this?

Generic answer
You may see many false detections because antivirus technologies suck.
More complex answer
As you can see, PE Tools main executable is not packed, code is clean, no virtual machines, no encrypted code parts, no obfuscation. But many antivirus engines can't handle even such easy case.
The main reason why AV could treat PE Tools as risky software because of it's features list:
PE Editor

PE and DOS Headers Editor
PE Sections Editor
PE Directory Viewer and Editor
Export Directory Editor
Import Directory Editor
Resource Directory Viewer
Exception Directory Viewer
Relocation Directory Viewer
Debug Directory Viewer
TLS Directory Editor
Load Config Directory Editor
Bound Directory Editor

Process Viewer and Manager

Show basic process information
Show process modules

PE Dumper

Running process dumper

PE Rebuilder

Dump Fixer
Relocation Wiper
Resource Directory Rebuilder
PE file Validation
Imports Binder
ImageBase Changer

Possible solutions

Inform antivirus company that produced weak antivirus with paranoid engine about false detection: by email or forum
Trust professional antivirus which doesn't mark PE Tools as threat
Analyse PE Tools features with disassembler like Cutter / Binary Ninja / IDA and decide yourself
Don't use PE Tools in real world environment, only in safe VM conditions

So where is project source code to view & research ?
Analyse code with with disassembler? No, thanks.
What is a reason to place project on GitHub without source?
Chrome/Microsoft antivirus detects it as trojan.
Old version which i have 1.5.800.2006 RC7 is not detected.

So where is project source code to view & research ?

It's not published, sorry.
You may find leaked/published sources of and old version, then fix bugs and add new features yourself.

Analyse code with disassembler? No, thanks.

It's your choice. Trust antivirus.

What is a reason to place project on GitHub without source?

As  an issue tracker.

Chrome/Microsoft antivirus detects it as trojan.

We don't care. See previous answer. Just don't use new version.
If you trust Chrome/Microsoft antivirus then remove new PE Tools version and be safe.

Old version which i have 1.5.800.2006 RC7 is not detected.

Great news. Use it.

Old version which i have 1.5.800.2006 RC7 is not detected.
Great news. Use it.

Not ideal for the user, but that's fair.

已经收到。

Not ideal for the user, but that's fair.

Well, after visiting the link in first post, you can get two lists:

Antiviral products that ~suck~ don't do their job well (false detections)

Some of them treat PE Tools as "Unsafe", "Possible Threat"
Some of them treat PE Tools as "PUP" (Potentially Unwanted Programs)

Antiviral products that do their job well (no false detections)

List of products that treat PE Tools as clean software:

Avira (no cloud)
Baidu
CMC
Cynet
Cyren
DrWeb
F-Secure
K7AntiVirus
K7GW
Kaspersky
NANO-Antivirus
Palo Alto Networks
QuickHeal
SentinelOne (Static ML)
TACHYON
TEHTRIS
Trapmine
VBA32
VirIT
Zillya
ZoneAlarm by Check Point
Zoner

I recommend to read my first comment, especially block "Possible solutions".
I hope that if every confused researcher instead of complaining here, will push AV-companies to do their job well, then we'll have more adequate AV products and services.


Compile error

Describe the bug
No idea why there is a syntax error here.
To Reproduce
Steps to reproduce the behavior:
function Test(){
const data = {
  lanes: [
    {
      id: 'lane1',
      title: 'Planned Tasks',
      label: '2/2',
      cards: [
        {id: 'Card1', title: 'Write Blog', description: 'Can AI make memes', label: '30 mins', draggable: false},
        {id: 'Card2', title: 'Pay Rent', description: 'Transfer via NEFT', label: '5 mins', metadata: {sha: 'be312a1'}}
      ]
    },
    {
      id: 'lane2',
      title: 'Completed',
      label: '0/0',
      cards: []
    }
  ]
}
return(
    <div>
      <Board data={data} />
    </div>
  );
}

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots

Desktop (please complete the following information):

OS: Xubuntu (Ubuntu 18.04.2 LTS base)
Browser: Chrome Version 80.0.3987.132 (Official Build) (64-bit)

Additional context
I ran the install command, and just tried the basic example. Not sure why there is a syntax error as everything looks in order. Sorry if this is dumb I'm pretty new to react.

Likely a duplicate of #359

Got the same problem. Anyone has an idea?

Experiencing this problem too. Exploring a solution.

The workaround is to import from react-trello/dist
I have a PR in #360

You're a genius! Thank you.

Sorry about the late turnaround on this one. v2.2.5 should fix the direct root cause.


0.x

The 1.3.x-snapshot tag is not yet available. hi @riprasad any news on that?
Do not merge, this PR will be pushed as a new branch.

two questions:
why not updating the install.yaml file?, you are only updating the install-dev.yaml file.
wdyt about replacing the {OPERATOR_IMAGE} placeholder with the appropiate container image? given this is an historic branch, it will help to ease the installation when someone come back to this branch in the future. I even think we could do this change also in master...

two questions:
why not updating the install.yaml file?, you are only updating the install-dev.yaml file.
wdyt about replacing the {OPERATOR_IMAGE} placeholder with the appropiate container image? given this is an historic branch, it will help to ease the installation when someone come back to this branch in the future. I even think we could do this change also in master...

two questions:
why not updating the install.yaml file?, you are only updating the install-dev.yaml file.
wdyt about replacing the {OPERATOR_IMAGE} placeholder in the deploy/operator.yaml file with the appropiate container image? given this is an historic branch, it will help to ease the installation when someone come back to this branch in the future. I even think we could do this change also in master...

I've responded in https://github.com/Apicurio/apicurio-registry-operator/pull/110#issuecomment-758593189 :)

two questions:
why not updating the install.yaml file?, you are only updating the install-dev.yaml file.
wdyt about replacing the {OPERATOR_IMAGE} placeholder in the deploy/operator.yaml file with the appropiate container image? given this is an historic branch, it will help to ease the installation when someone come back to this branch in the future. I even think we could do this change also in master...

I've responded in https://github.com/Apicurio/apicurio-registry-operator/pull/110#issuecomment-758593189 :)

@jsenko The changes are done. The PR needs to be reviewed and merged. I will update you once the PR is merged

@jsenko The changes are done. The PR needs to be reviewed and merged. I will update you once the PR is merged

The 1.3.x-snapshot tag is not yet available. hi @riprasad any news on that?

@jsenko changes are now merged to master.

The 1.3.x-snapshot tag is not yet available. hi @riprasad any news on that?

@jsenko changes are now merged to master.

Changes to the workflows are merged I think, but I'll need to manually push 1.3.x-snapshot and 1.2.x-snapshot images to quay and docker hub.  Or I guess we could just make a trivial change/commit on each of those branches to trigger the build from CI.

Changes to the workflows are merged I think, but I'll need to manually push 1.3.x-snapshot and 1.2.x-snapshot images to quay and docker hub.  Or I guess we could just make a trivial change/commit on each of those branches to trigger the build from CI.

I'm on it. :)

I'm on it. :)

OK I've triggered CI builds for both 1.2.x and 1.3.x that should result in the new -snapshot docker images to be pushed.  As soon as those CI builds complete, the images should be ready to go.

OK I've triggered CI builds for both 1.2.x and 1.3.x that should result in the new -snapshot docker images to be pushed.  As soon as those CI builds complete, the images should be ready to go.


Drawings that are too small/too large are misclassified

This happens because the point size becomes too small/large at certain scales, causing the classifier to be confused. Not sure what's the best way to solve this at the moment.

Resolved by scaling the point size as the ratio of the width or height (whichever is larger) of the figure agains the width or height of the canvas changes. This is a heuristic solution that isn't necessarily the best solution, but it works for now.


Nation Tab - Add Nation button

Describe the bug
the add nation button disables after a single nation is added
To Reproduce
Steps to reproduce the behavior:

Go to any work's nation tab
Click on add nation
add a nation
See error

Expected behavior
the add nation button should not be disabled for anyone who is assigned to that team, or for superusers & above
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.

@Connor1892  I tried this and I could not reproduce the issue, could you please try again and if it's still not working let's try it in a meeting?

@jadmsaadaot it's happening in Prod. so maybe it's been fixed by something that hasn't hit that environment yet?
If you're a viewer on a team. after the 1st First Nation is added the button disables.

@dinesh-aot it seems this is something happening in prod but not in dev. Might have been fixed. Can you confirm please if we can close this?


Release

As there has been a build in https://github.com/revolunet/angular-carousel/commit/ecbbb785951808e7c15dafd7fd75ddc70e3b7a40
Could you set this as a release on GH so those of us using bower etc.. can grab it ?

done, thanks


Get GCM Id in Android but not FCM in iOS

Expected Behaviour
When I'm in iOS app I want the .on('notification') event return Push Plugin register and when I'm in Android get GCM Id as normally.
I get with this event this token:
FCM Registration Token: cqs2H3ED5u8:APA91bEVQGi0SfbC1Yau1xN_PJB0SOmB50PgHMNG2zCqw4bzWfLruXfqKoIT7DeJnz5K37CqQLIs9F-CXfwurC-UhZjfLNUvnEfCZDpIleW_6xGZYZKokNcIPouGHdvdSnVMhHu6mITh
But I want
Push Plugin register success: <5c27c9a8 87e28030 735f8bc7 e27ab8de e6d6538f c9759e70 26c306a6 fa0ac2cc>
This token
Actual Behaviour
In Android and iOS I get GCM and FCM register respectively
Reproduce Scenario (including but not limited to)
Steps to Reproduce
Platform and Version (eg. Android 5.0 or iOS 9.2.1)
(Android) What device vendor (e.g. Samsung, HTC, Sony...)
Cordova CLI version and cordova platform version
cordova --version                                    7.0.1
cordova platform version android                     6.2.3

Plugin version
phonegap-plugin-push   2.0.0
Sample Push Data Payload
Sample Code that illustrates the problem
Logs taken while reproducing problem

@ncampuzano I don't understand your question. Do you mean you want to use APNS on iOS but you are getting a FCM token?

Yes! I see in the Xcode log theses tokens, but in the .on(‘registration’) I only get FCM token, my issue is similar to #1714

@ncampuzano do you want to send to iOS using APNS or FCM?

To iOS I need APNS

@ncampuzano right, so don't deliver the GoogleService-Info.plist on iOS and it will default to APNS.

But if I’m only remove GoogleService-Info.plist but google-service.json it’s keeps, in Android he use FCM and in Apple use APNS?

@ncampuzano I updated the docs to make it more clear.
Yes, when you remove the GoogleService-Info.plist from iOS it uses APNS but as long as you keep delivering google-service.json it uses FCM. Actually, Android won't work without the google-service.json file.
You will probably have to remove the ios platform and add it again. There is a bug in cordova-ios that doesn't remove resource files correctly.

It's perfect. Removing GoogleService-Info.plist use iOS APNS. Close it 👍


Mark any_as_u8_slice function unsafe

https://github.com/A1Liu/tci/blob/441ff516d974faec922a73daa02c074a2f0ec21c/src/util/general.rs#L293-L295
hello, if a function's entire body is unsafe, the function is itself unsafe and should be marked appropriately. Marking them unsafe also means that callers must make sure they know what they're doing.

Hello! This project is not really maintained, so I'm not really accepting PRs right now.


Idl version

when this is reviewed (and tested locally) will close https://github.com/CosmWasm/ts-codegen/issues/59

[ ] reviewed
[ ] local global CLI install test

@uint correct me if I'm wrong, but I don't think it breaks anything if this PR enables parsing of the new idl format to be treated like the current published version of ts-codegen (meaning w/o the named responses and keeping the Response style for now til we close https://github.com/CosmWasm/ts-codegen/issues/61 )

I mean, if contract developers are willing to only partially migrate to the new way of generating APIs (while keeping the old <Thing>Response.json boilerplate), I imagine it should work out.

This isn't a priority or anything, but in the future it would be nice to parse the idl_version as a semver value and make sure it satisfies the semver requirement of ^1.0.0 (or whatever the supported version is at the time). And just crash otherwise with IDL version {idl_version} not supported.

great! thanks @uint

This isn't a priority or anything, but in the future it would be nice to parse the idl_version as a semver value and make sure it satisfies the semver requirement of ^1.0.0 (or whatever the supported version is at the time). And just crash otherwise with IDL version {idl_version} not supported.

I agree! awesome idea https://github.com/CosmWasm/ts-codegen/issues/62


Code Generation

Implement option to change between Linux and windows code generation

done in commit "407d98f96d0c7657a3fde2c5e00945db19fc2066" on branch v1.0beta

At this moment  only elf64 is supported, but later win64 and possibly elf32/win32 support will be added


#230 - Beginning credo cleanup

credo configuration altered so that delibarately placed spaces around certain
expresions no longer throw errors
Corrected remaining credo errors in elixir_script.ex

Thanks!


Documentation updates

Closes #125
Changes proposed in this Pull Request

In this PR I update the configuration documentation page, and add documentation for the plotting module. Sphinx warnings and errors have also been addressed.
Checklist

[x] I tested my contribution locally and it seems to work fine.
[x] Code and workflow changes are sufficiently documented.
[x] Changed dependencies are added to envs/environment.yaml.
[x] Changes in configuration options are added in all of config.default.yaml.
[x] Changes in configuration options are also documented in doc/configtables/*.csv.

@ktehranchi There are a couple last items from issue #125, specifically; specifically, Add interconnection map and Improve introduction. I cant really remember what we had in mind with these? If you remember, I am happy to look at them. Else, are you okay if I just merge this as is?

yep go for the merge- I will add a US wide interconnection plot and spiff up the documentation there when i finish #70


Async Hangouts Apps Script - user has to auth, leave space, and then return to trigger onAddToSpace(e)

I followed the instructions here: https://developers.google.com/hangouts/chat/how-tos/bots-apps-script#async_messages

When someone else in my organization adds the bot, the user first must chat the bot in order for it to respond and let them know it requires configuration and takes them to the Apps Scripts authorization screen. Afterwards though, the bot does not function as the spaceId is not captured within script properties.

Only after removing the bot from the space and then re-adding it does it successfully add the spaceId to the App Script project script properties and begin to function.

Am I doing something wrong here or is this expected behavior? Is there some workaround apart from organization-wide delegation that would make this process smoother? Thanks.

Another question - with the sunsetting of Classic Hangouts for gSuite, will Hangouts Chat bots be able to chat with users through gmail?

Looks like that's expected behavior -- there's a small comment at the end of the sample:
"It's possible for a user to add your bot to a space before authorizing your bot. In this case, your bot is unable to respond to the add-to-space event."
That said, it's not very clear in the doc that that comment applies and there should be some better guidance provided. Since that particular sample isn't in this repo and just in the docs, I filed a bug internally against the docs to have that clarified.
Closing this issue here since it doesn't apply to this repo.


Make Counter emit valid FIRRTL

Remove var from object Counter.apply, using a Wire instead. Also improve
some ScalaDoc and the class Counter require message.
EDIT: At @seldridge's suggestion, here is a more minimal fix that doesn't mess with the API of class Counter.inc()
The one thing I want to highlight:
-// This if-guard maintains older behavior for wrap when n == 1
-val wrap = if (n > 1) c.inc(cond) else cond

As the comment says, if we did just c.inc(cond), it changes the behavior for n == 1 to return true.B rather than cond.

Related issue:

Type of change: bug fix

Impact: API addition (no impact on existing code)

Development Phase: implementation
Release Notes

Fix bug where chisel3.util.Counter emitted illegal FIRRTL

This fix is also backportable now


Multiple custom Mutations not triggering Lambda function.

Which Category is your question related to?

API
Functions

Amplify CLI Version
You can use amplify -v to check the amplify cli version on your system
4.18.0
What AWS Services are you utilizing?

Lambda
AppSync GraphQL API

Provide additional details e.g. code snippets
Hi all,
I've been struggling to add a second custom Mutation in my GraphQL schema. These custom Mutations should trigger lambda functions to determine whether a user enough resources in their account (tiered usage limits) to add say a Template or a Project. The first function I added is working and being triggered by my React App. Everything works fine for adding Projects. I added a second Mutation using the extend type Mutation syntax. Everything is identical to the the addProject Mutation/Function with the neccessary tweaks. However, when executing the Mutation in the React app the lambda function is not triggering and checking the App Sync UI on the AWS management console shows that another lambda function was not added. Any idea as to what I'm doing wrong here? Is this a bug?
type Mutation {
  addProject(name: String!): String
    @function(name: "createProject-${env}")
}

extend type Mutation {
  addTemplate(
    name: String!
    type: String!
    width: String!
    height: String!
  ): String @function(name: "createTemplate-${env}")
}

The strange thing is that the first Mutation works but the second doesn't the only difference is the extend type Mutation syntax. Is this what is causing the issue with the Amplify CLI not creating the second function entry in AppSync?
Thanks in advance! Best regards and hope all is well,
Kyle

So I got this working by changing my GraphQL schema as follows:
type Mutation {
  addProject(name: String!): String
    @function(name: "createProject-${env}")
}

extend type Mutations {
addTemplate(
    name: String!
    type: String!
    width: String!
    height: String!
  ): String @function(name: "createTemplate-${env}")
}

to
type Mutation {
  addProject(name: String!): String
    @function(name: "createProject-${env}")
  addTemplate(
    name: String!
    type: String!
    width: String!
    height: String!
  ): String @function(name: "createTemplate-${env}")
}

Is this intended behavior?  Hope this helps someone!
Cheers!
Kyle

Hello @kylekirkby
If your schema file is in one location then the 2nd format is the right format for this.
The use of extends on operations is used by the GraphQL transformer when you split your schema in different files. Ref here: https://aws.amazon.com/blogs/mobile/merging-graphql-schema-files-and-more-with-the-cli/

Arhh makes sense! Thanks @SwaySway !


Update jsdom dependency (no 0.12 otherwise)

Just noticed the version of jsdom uses is a little old. As a result it's broken on 0.12. See https://github.com/tmpvar/jsdom/issues/1022 .
Could you please bump up the dependency?

+1

Done. Dependency to jsdom has been removed altogether.


Feature: implement priority scheduling

With nghttp2 exposing the dependency and priority information in its API, implement a priority scheduling in h2_mplx so that the task with the highest priority is assigned to the next available worker.

@icing for reprioritization, I'm guessing once/if the worker picks up a task it'll run to completion, but it may be possible to reorder tasks that have not (yet) been assigned?

Ilya, yes, for each http2 session (e.g. connection) there is a queue of open streams that need assignement to a worker thread. When the next worker becomes available, the session would select the stream with the highest priority for processing.
This does not influence streams from different sessions, e.g. the balancing of all connections with work to do stays basically fair.
Once a stream is being processed, it will be done so until it has finished. Priority however will still determine which streams data is written to the client. If a super-high stream gets opened and assigned to a worker, and has data available, it will stall all ongoing responses on that connection.

Implemented in trunk.

Available for testing in v1.0.3

Released.


feat(ffe-radio-button-react): value can be boolean

Since support for boolean value was added to RadioSwitch it was
now time to also support it in the RadioButtonInputGroup we tend
to wrap those in.

Dette hadde jeg sannsynligvis oppdaget før dersom npm link hadde fungert :/

Coverage remained the same at 79.447% when pulling 3cd3d7f31bd922b7bc08f8f96c48d17d19e47ebe on ivarni:support-bool-value-2 into 78ed5c741bf5d0c496a0f70e2e5d7e0c19d30360 on SpareBank1:develop.


Documentation naming convention.

We should define and follow the standard naming conventions for naming documents. Currently few are all small, all capital and mix.
assets/
be/
cla/
images/
integrations/
meetings/
PDFs/
RELEASENOTES/
scaleout/

Created the PR #1108 to address this issue.

Issue addressed, hence closing


Feature: Option to have non-null assertion operator compile to a runtime exception.

I often find myself interacting with third party JS libraries that are poorly documented.  Because of this, I often don't know whether a function may yield null or undefined as a result or when calling a callback.  Often, the best way to deal with this is to just check for null and throw an exception with something like if (foo === undefined) throw new Error('Local variable foo was undefined.");
However, this pattern starts to get really ugly when you are working with a lambda, like in collection.map(x => { if (x === undefined) throw new Error('x is undefined.'); return x.bar; }).  I would much rather be able to just do collection.map(x => x!.bar) and have TypeScript compile the non-null assertion operator down to something that includes the aforementioned check.
Note: I'm aware that this could be implemented as a helper function like function assertNotNull<T>(x: T|undefined|null): T { ... } and then do collection.map(x => assertNotNull(x).bar).  However, TypeScript already has an operator that goes in that exact spot, is much more concise than any function I could come up with, and already feels like it should have the behavior described here.
It should also be noted that I believe other languages with not-null assertion operators like Kotline actually do have a runtime check, which is why TypeScript's lack of check causes so much confusion for users.  Not only does it feel like the right thing for TS to do a runtime check, that is what other languages do.

It has been one of our design goals to avoid adding run time overhead. we have aspire for an erasable type system with no run time impact. There are many other opportunities today for runtime checks, but these all do not jive with the language design vectors.
The null assertion operator was also introduced to remedy limitations in the control flow analysis the compiler does today, for instance a variable defined within a callback, or through a method/function invocation. it does not make sense for these to turn into runtime checks.
The solution today as you noted is to define your own assertNotNull function, and use it when needed. and not to overload the meaning of the ! operator.
For that this request is currently out-of-scope.


feat: send anonymous error reports to sentry

What kind of change does this PR introduce?
feature
What is the new behavior?
Report errors anonymously so that we are informed when UX failures.

[ ] allow users to opt out
[ ] fetch sampling config from api
[x] add stacktrace to all errors

Going forward, any errors from external packages should be wrapped in errors.New(err) using go-errors.
Additional context
Add any other context or screenshots.

Telemetry is a very sensitive topic, even for anonymous info. IMO we should make this opt-in by prompting on first run instead of opt-out.


OAuth2 authenticator does not handle html response

In case response with with content-type: text/html is returned to method makeRequest method reject callback is not called and error is thrown outside of promise .
https://github.com/simplabs/ember-simple-auth/blob/master/addon/authenticators/oauth2-password-grant.js#L328
Following error leaks to ember.logger:
 Unexpected token < in JSON at position 0
    at JSON.parse (<anonymous>)

I don't think that needs to be handled by ESA. It's just an invalid response that the authenticator cannot understand.

So how should I catch this JSON.parse error in context of authenticate promise ?

The OAuth 2.0 authenticator only works with OAuth 2.0 endpoints. Your server does not seem to be one - this is an error on the server side, not the client side.

We have compliant OAuth 2.0 server. But there are valid use cases that even oauth compliant server returns unexpected response. e.g load balancer / proxy error ...e.g standard nginx 500 error page in that case I need to somehow handle it.


Sorting error in dataset list

Describe the bug
Sorting doesn't work after sorting by used in.
To Reproduce
Steps to reproduce the behavior:
Dataset Tab> click used in sorting button > click created sorting button > see error
Expected behavior
Sort should work in dataset list.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

Integrated test 1 passed

Integrated test 2 passed


Should not allow containers on unmanaged network to be used as LB target

Server Version - V0.15.1
API allows for containers in unmanaged network to be added to LB.
API should error out in this case.

This may still be possible throught he API, but the UI doesn't show unmanaged containers as target choices.

Closing as standalone LB is disabled now.


[MCTF] Surface cleanup for no filter case.

When filter switches from some strength to zero,
the target surface is kept and not returned.
Issue: -
Tests: Manual testing

This pull request is already covered here: https://github.com/Intel-Media-SDK/MediaSDK/pull/2366


chore: release main

:robot: I have created a release beep boop
ui5-cc-excelupload: 0.17.0
0.17.0 (2023-05-18)
⚠ BREAKING CHANGES

change API to add messages to Error Dialog (#211)

Features

add option to hide the "preview data" button (#212) (65bcb5b)
add option to set decimal seperator (#199) (ba4a4f2)
add option to skip Fields with Nullable=false (#216) (28ded2d)
change API to add messages to Error Dialog (#211) (a06ec4e)
show faulty value in the error dialog (#197) (8e09745)
warning when numbers are formatted in file (#205) (701b31e)

ui5-cc-excelupload-button: 0.5.0
0.5.0 (2023-05-18)
⚠ BREAKING CHANGES

change API to add messages to Error Dialog (#211)

Features

add option to hide the "preview data" button (#212) (65bcb5b)
add option to set decimal seperator (#199) (ba4a4f2)
add option to skip Fields with Nullable=false (#216) (28ded2d)
change API to add messages to Error Dialog (#211) (a06ec4e)
warning when numbers are formatted in file (#205) (701b31e)

Dependencies

The following workspace dependencies were updated

dependencies

ui5-cc-excelupload bumped from 0.16.4 to 0.17.0

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/marianfoo/ui5-cc-excelUpload/releases/tag/ui5-cc-excelupload-v0.17.0 :sunflower:

:robot: Release is at https://github.com/marianfoo/ui5-cc-excelUpload/releases/tag/ui5-cc-excelupload-button-v0.5.0 :sunflower:


Cleanup S3

Description:
As you continue to change and upload your lambda function, your S3 will fill up with useless objects.
Steps to reproduce the issue:

Use this example
Run aws cloudformation package ...
Edit index.js
Run aws cloudformation package ... again

Observed result:
Two different files uploaded
Expected result:
Eventually, only a single file.
The current workaround I've tried
I've enabled S3 versioning and added a lifecycle rule to expire old versions, I also upload the code manually using the same name and get the latest version before I update the stack. It works but it requires extra work.

aws cloudformation package writes different S3 files to the supplied S3 bucket.  I also setup a bucket lifecycle on my package buckets to auto-expire objects after a certain amount of time. Since creating the S3 bucket is outside of the SAM tooling, I don't think there's anything actionable we can do about this issue. You can open an issue on the SAM CLI GH repo asking for a command that will setup the S3 bucket for you.
Closing this issue as there isn't anything actionable we can do in this repo for this.


Fix unstable tests and add more ut

What problem does this PR solve?
Issue Number: ref #7587
Problem Summary:
What is changed and how it works?

Fix unstable tests introduced by #7589
Add failpoint tests in mpp_fail.test to ut

Check List
Tests

[x] Unit test
[ ] Integration test
[ ] Manual test (add detailed scripts or steps below)
[ ] No code

Side effects

[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[ ] Breaking backward compatibility

Documentation

[ ] Affects user behaviors
[ ] Contains syntax changes
[ ] Contains variable changes
[ ] Contains experimental features
[ ] Changes MySQL compatibility

Release note

None

/rebuild

/run-sanitizer-test asan

/run-sanitizer-test tsan

/run-sanitizer-test asan

https://ci.pingcap.net/blue/organizations/jenkins/tiflash-sanitizer-daily/detail/tiflash-sanitizer-daily/1154/pipeline/

/run-sanitizer-test tsan

/run-sanitizer-test tsan

https://ci.pingcap.net/blue/organizations/jenkins/tiflash-ghpr-sanitizer/detail/tiflash-ghpr-sanitizer/138/pipeline


Include ORCID iDs in work metadata out of the box

Is your feature request related to a problem? Please describe.
Currently, there are several ways that ORCID iDs can be associated with a person in DSpace:

Authenticated ORCID iDs via the ORCID API integration with the person entity/profile
ORCID lookup tool (unauthenticated) - based on authority records, not person entities
Import of ORCID iDs (either authenticated or unauthenticated)

Ticket https://github.com/DSpace/DSpace/issues/9086 will address printing or displaying these ORCID iDs from any source on the work landing pages, however, my understanding is that only ORCID iDs that are associated with person entities are included in the work metadata, and that ORCID iDs associated with authority records are not included in work metadata (correct?).
The problem here is that ALL ORCID iDs in the system should be associated with work metadata.
Describe the solution you'd like
Ideally, if possible, any ORCID iD present in the system, regardless of the source/whether it's associated with a person entity or an authority record, would be automatically included with the work metadata for items authored by the individual(s) with the ORCID iD(s).
Describe alternatives or workarounds you've considered
For institutions that have been using the ORCID lookup tool in versions 5 and 6 of DSpace, ORCID iDs are only associated with authority records, not person entities. So, to help with including these in work metadata, and help those institutions migrate to newer versions of DSpace, it would be helpful to have a way to migrate authority record ORCID iDs to person entities. Could there be a special tool to allow this? (see related ticket: https://github.com/DSpace/DSpace/issues/9530)

hi @sheilarabun ,
I can confirm that ORCID eventually stored in an authority record are not displayed in the work detail page nor available in the work metadata via the interoperability layers (oai-phm, rest api).
It would be not trivial to solve both problem, I expect that retrieve the orcid to display it in the work (item) detail page is easier than allow the other layers to use this information but I need to go a bit deeper in the investigation to provide an estimation.
The #9530 is instead something that I feel more needed and that could eventually makes this other task less urgent


1.0 Release Schedule

Hi, I work at @knotchlabs  my company uses this gem in production. We love it and respect your work but I'd like to see it come out of Alpha.
Is there a 1.0 release schedule available? If you need help one of our core engineers @mjc-gh has plenty of free cycles and he would be more than happy to jump in and help.

Hey Sean, thanks for your kind words. @mjc-gh (a VP at your company) is already regular committer to this repo. He's also the person who originally created the gem. Mike has earned the right to call it alpha, beta, gamma, Shirley, or whatever his heart desires.


Add OpenTracing to the deployments

https://github.com/microservices-demo/microservices-demo/issues/479

Too many deployments
Removing some per #661


[Bugfix] Temporarily modify single table demo data link

Description

Due to datahub.io issues, all current pytest unit tests fail because they cannot download the demo data.
Currently, we modify the link to the data set to temporarily solve this problem.
Motivation and Context

How has this been tested?

Types of changes

[ ] Maintenance (no change in code, maintain the project's CI, docs, etc.)
[ ] 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)

Checklist:

[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.

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

Comparison is base (dd70515) 78.58% compared to head (8d88aa5) 78.87%.
Report is 1 commits behind head on main.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #121      +/-   ##
==========================================
+ Coverage   78.58%   78.87%   +0.28%
==========================================
  Files          64       64
  Lines        2764     2764
==========================================
+ Hits         2172     2180       +8
+ Misses        592      584       -8

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


Name steps

Applied mkpr on .github/workflows/ci.yml
sed s/uses<EMAIL_ADDRESS>checkout\n        uses<EMAIL_ADDRESS>

:tada: This PR is included in version 1.11.62 :tada:
The release is available on:

npm package (@latest dist-tag)
v1.11.62
GitHub release

Your semantic-release bot :package::rocket:


Put the name or env var as name

Use the env name as the title such as ANDROID_TARGET=android-24

:+1: I agree. Just having a number is not very informative.


Copier changes

This has been implemented in https://github.com/salt-extensions/saltext-mongodb/pull/21.
Adding pymongo to dependencies is probably a good idea, will do that before doing a first release.


Update requirements.txt

As maintainer of fake-useragent, you need to update this package.

thanks for your contribution!

You're welcome :)


Table Bug: Cursor goes horizontal after deleting text below it

Problem : Cursor goes horizontal after deleting text below it
Lexical version: latest
Steps To Reproduce

Insert a table in https://playground.lexical.dev/
add text below it.
Delete text below.
go to table last cell.
Goto next line.

Link to code example: https://playground.lexical.dev/
The current behavior
https://github.com/facebook/lexical/assets/14221696/da2492db-b304-455d-bf1b-e75613fcb960
The expected behavior
Cursor should be vertical.

This is the correct behaviour, because there is no paragraph under the table, which is why the block cursor exists to signify areas between widgets that don't have paragraph spacing between them, or in this case are at the end of the document.


Microsoft  Remote Development Extension Pack

Details on the code-server extension marketplace are at
https://github.com/cdr/code-server/blob/master/doc/FAQ.md#whats-the-deal-with-extensions
https://marketplace.visualstudio.com/items?itemName=ms-vscode-remote.remote-ssh
Please fill in the issue template!
not able to select and install this extensions
can these 2 be added?

[ ] Extension name:
Remote Development Extension Pack
Remote - SSH
[ ] Extension GitHub or homepage:
https://github.com/Microsoft/vscode-remote-release
https://github.com/Microsoft/vscode-remote-release

Unfortunately we can't include these because of licensing issues. See https://github.com/cdr/code-server/blob/v3.4.1/doc/FAQ.md#differences-compared-to-vs-code
We'd like to re-implement these as open source extensions but there's no timeline on that. See https://github.com/cdr/code-server/issues/1315


Save ImGui settings in a user-specific directory

Fixes #6862

@bejado I didn't test the Windows code, but I think it's correct?


Update: TorProject.TorBrowser version 12.0.7

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?
[x] This PR only modifies one (1) manifest
[x] Have you validated your manifest locally with winget validate --manifest <path>?

[x] Does your manifest conform to the 1.4 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: codeflow:open?pullrequest=https://github.com/microsoft/winget-pkgs/pull/109820&drop=dogfoodAlpha

Close with reason: Stale pull request;


[ConstraintSystem] Use witnesses for makeIterator and next refs in for-in context

References to makeIterator and next in for-in loop context have to be
treated as references to witnesses of Sequence#makeIterator and
IteratorProtocol#next requirements. Letting overload resolution decide as-if
loop was written direct use of makeIterator() and while let ... = next() statement
could lead to problems with retroactive conformances and difference in runtime
behavior.
For example:
extension Array where Element == Double {
  func makeIterator() -> Array<Int>.Iterator {
    return [1, 2, 3].makeIterator()
  }
}

for i in [4.0, 5.0, 6.0] {
  print(i)
}

makeIterator declared in the Array extension is not a witness to Sequence#makeIterator
and shouldn't be used because that would change runtime behavior.
Resolves: https://github.com/apple/swift/issues/60514
Resolves: https://github.com/apple/swift/issues/60958

This is an alternative approach to https://github.com/apple/swift/pull/61075.

@swift-ci please test

@swift-ci please test source compatibility

@swift-ci please test

@swift-ci please test

@swift-ci please test source compatibility

🎉🎉🎉 Can we nominate for 5.7 branch?

@xwu It's in - https://github.com/apple/swift/pull/61109


Database download

From the review:

One issue I encountered was the database download, with personal handling of the database path, is awkward. It seems that much of the work of downloading the database, and then storing the path could be automated. The storage location could be supported by the rappdirs package. There are existing examples of such file downloading and use, such as getlandsat, hydrolinks, and LAGOS to name a few with various implementations. This would make it easier for users to quickly get started using the data and eliminate the need to muck around in the Renviron file.

Other review:

The example:
file.edit("~/.Renviron")
That path does not automatically work for Windows users I think. You might want to clarify that you will probably want a .Renviron file in the same directory as your RStudio project (assuming you use RStudio)...or something like that

The approach I think here is to use rappdirs and then have here:
https://github.com/bcgov/tidyhydat/blob/dea7c3e3be4755437be93fbc33cb6dc992d17efe/R/download.R#L448-L452
download_hydat() out to the result of rappdirs::user_log_dir(). Then rather than call Sys.getenv("hydat_path") for hydat_path=NULL then we use rappdirs::user_log_dir().

Also another "are you sure" question when downloading hydat.


Added compat for Hemogen Extractor

Requires rwmt/MultiplayerAPI#9 and rwmt/Multiplayer#381
Sadly, not really possible to make a simple patch without those changes. It could (in theory) be possible without it, but it would require quite a bit of work and implementing a lot of workarounds.

Updated Hemogen Extractor compat. Again the case of a compat waiting for API update.
Now uses the changed API introduced by Zetrith, instead of the (now outdated) ones made by me.

Is merge ok?

Not yet safe, needs rwmt/MultiplayerAPI#10 to be up on NuGet as well as Multiplayer to implement the new API.

Should be safe to merge. Also, just in case - merged master branch in.


add link to another vertical slice project

This takes a slightly different approach from most and contains explanations of its decisions.
Most projects have their link to some other article or video. But they don't explain why things are done in a certain way.
This repo plans to include examples in Java and Go; and potentially Ruby in the future.

Hi,
Thanks for your contribution.
I checked the source code but I think there are samples with more quality regarding vertical slice, so I prefer to close this pull request


Increase security around honeypot accounts

Updated script to add honeypot accounts to "Domain Guests" group, remove from "Domain Users", set logon hours and 'log on to' restrictions.

Here is what the new output looks like:

Error handling example:


chore(solc): more verbose traces

Motivation
provide more context in traces to make debugging easier

Solution

PR Checklist

[ ] Added Tests
[ ] Added Documentation
[ ] Updated the changelog

PR failing for unrelated reason, merged


Releases tab not showing anymore

Description
I'm on the latest build and the Releases tab doesn't show up anymore for some reason. Tried reinstalling and on 2 different Chromium based browsers (Edge and Vivaldi) and neither show the Releases tab.
How to replicate the issue + URL
Browse to here:
https://github.com/refined-github
Extension version
23.10.5
Browser(s) used
Vivaldi, Edge

Found this as a workaround:
https://github.com/gitmylo/github-releases-button/blob/main/script.js

Browse to here:
https://github.com/refined-github

There's no releases on organization pages.
Did you set the token in the options?

I assume you forgot to set the token because it works:

Thanks for the fast response.
Yes you're right, I had not set the PAT token and I've got it working now. I wasn't aware of this requirement, as the previous versions didn't need this and the userscript I provided in my previous reply also didn't need it. But that's all good.

previous versions didn't need this

releases-tab has required the token for a the past 4 years: https://github.com/refined-github/refined-github/commit/3891c5179fbc5dd3b2c03d2fcd55652129f0a034
but until recently it also tried to read it from the current page, if you visit that first, yeah.


Remove unsafe blocks

Opening now to show progress on this issue. Goal is to have everything in pure rust and using only safe code. Feedback on existing changes welcome at any time

Still need unit tests of some sort to verify the implementation is correct. Looking over the actual code, this should be equivalent to the previous code. Likely not ready to merge or even use until some unit tests are added to demonstrate correctness though. Ready for review though

NB: This still retains the necessity of an unsafe API as unsafe callbacks are used. Adding a safe API is part of the overall needed work for MFEK Foundation Grant № 3, Series of 2022, so this PR does not end that grant, it just represents ≈half the work of it.


Can I use the docker image of pytorch 23.03 to compile & run

I want to run TensorRT-LLM on a docker image with ubuntu20, could I choose nvcr.io/nvidia/pytorch:23.03-py3 as my base image to compile & run

You can take a try, but we have not verified.


fix: swap the order of layout validation to prioritise "no program" (DHIS2-13898)

Implements DHIS2-13898

Key features

reorders the layout validation to prioritise program/stage
updates the Cypress tests to reflect the changes above

Screenshots

I can add these changes to my PR which makes changes to the same function.

Added to https://github.com/dhis2/line-listing-app/pull/245/


Add a note under proxy subsection to avoid a major problem

Please sign your commits following these rules:
https://github.com/docker/docker/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "master" git@github.com:half-dead/distribution.git somewhere
$ cd somewhere
$ git commit --amend -s --no-edit
$ git push -f

Ammending updates the existing PR. You DO NOT need to open a new one.

#1356

closed by #1394


Enhancing dotenv function

Enhancing the dotenv function with few enhancements:

Handling environment variable names and values with quotes (' or ")
Handling comments in ".env" variable file
Gracefully moving on if ".env" file name not found
Printing line, file name, and line number for invalid formatted lines in ".env" files

This should resolve https://github.com/tilt-dev/tilt-extensions/issues/444

Hi a nice thing to add would be: to also remove the \r on the end of each line if it exist for better windows support

Hi a nice thing to add would be: to also remove the \r on the end of each line if it exist for better windows support

The current PR relies on the StarLark function strip.
Looking at the documentation, it seems that when called without any parameters then it removes all whitespace. But it is not explicitly clear if it is \n or \r or both.
A simple task could be to add an explicit test case for this to make sure future breaking changes are caught (if any) for \r character support.
I can update this PR as soon as I get a chance. Thanks for pointing this out @atta !


Users with long names on Guardian are not displayed correctly

Problem description
Users with long names on Guardian (latest version with new UI) are not displayed correctly.
Step to reproduce
Steps to reproduce the behavior:

Create a user with a long name such as "cee - c23a7dd4-7384-4504-9d3f-8be4e36cd319"
After signing into guardian, the name of the user in the side panel is cut and the logout icon is not visible (Check screenshots)
Go to the initial Login page
Click the Demo Admin button on the login page and check the list of users and SRs (Check screenshots)) where the name is not readable

Expected behavior
The name of the user should be readable
The button for logout should be visible
Screenshots
Added two screenshots illustrating the issue.

I think we should use truncate for long names and add full user name inside Profile page.
Also I think it would be useful to add Copy icon for username and DID just incase if it would be needed to send to support team or for any other cases.

Design:
https://www.figma.com/file/r4opkmPm3TPieVSXTb0pjK/Design-file-(Guardian)?type=design&node-id=167%3A21230&mode=design&t=Yj44Gm7Bp9ETT5tC-1

@prernaadev01 any updates on this bug?

@armarik Thank you for following up on this. Currently, this issue is added as an UI/UX enhancement. Please let us know if its a priority for you so that we can plan on this accordingly. Thanks

Hi, @prernaadev01! It is not high priority but a nice to have. All of our users have long names and it can be cumbersome to not have a visible logout button (which the long name "obscures").

@armarik Sure. We will prioritize this and plan to complete this in this month.


Parameter incorrect

enola username123
Error running program:  unable to set console to raw mode: The parameter is incorrect.
What is the issue?

Hi @BenLim88,
Can I ask what Operating System & Terminal are you using?


Advice on Override and BE entry clashes

Hi Guys,
This is not really an issue but...
I've created two KBs for Reach from the Bioentities files: a protein complexes KB and a protein families KB. I'm currently in the process of updating our Override KB, which has the highest priority. My first step is to remove any entries which can now be found in the two new BE KBs, as the existing override entries will "block" resolution to the BE KB entries.
But, I'm somewhat reluctant to remove Override family entries which already map to PFAM IDs, unless you (in your superior Biological wisdom) think they are incorrectly mapped, so I'd greatly appreciate your advice on the "clashing" entries below:
ACOX	PF01756	http://pfam.xfam.org/family/ACOX
BMP	PF02608	http://pfam.xfam.org/family/BMP
Cadherin	PF00028	http://pfam.xfam.org/family/Cadherin
COX4	PF02936	http://pfam.xfam.org/family/COX4
COX6A	PF02046	http://pfam.xfam.org/family/COX6a
COX6B	PF02297	http://pfam.xfam.org/family/COX6b
COX7A	PF02238	http://pfam.xfam.org/family/COX7a
COX7B	PF05392	http://pfam.xfam.org/family/COX7b
COX8	PF02285	http://pfam.xfam.org/family/COX8
CRISP	PF08562	http://pfam.xfam.org/family/CRISP
DDR	PF08841	http://pfam.xfam.org/family/DDR
DVL	PF08137	http://pfam.xfam.org/family/DVL
ETS	PF00178	http://pfam.xfam.org/family/ETS
FGF	PF00167	http://pfam.xfam.org/family/FGF
FLOT	PF15975	http://pfam.xfam.org/family/FLOT
GATA	PF00320	http://pfam.xfam.org/family/GATA
HSP90	PF00183	http://pfam.xfam.org/family/HSP90
IGFBP	PF00219	http://pfam.xfam.org/family/IGFBP
IL1	PF00340	http://pfam.xfam.org/family/IL1
IRS	PF02174	http://pfam.xfam.org/family/IRS
MAF	PF02545	http://pfam.xfam.org/family/MAF
NOTCH	PF00066	http://pfam.xfam.org/family/NOTCH
PKI	PF02827	http://pfam.xfam.org/family/PKI
RAS	PF00071	http://pfam.xfam.org/family/RAS
SAA	PF00277	http://pfam.xfam.org/family/SAA
TGFB	IPR015615	http://www.ebi.ac.uk/interpro/entry/IPR015615

Since the beginning we've imagined keeping track of synonymous grounding for protein families and complexes where available. These are a pain to look up in bulk and we haven't needed them so far. However, we could go through and add these in.
We have some options for how to do this:

Add these in as "isa" or "hasSynonym" in relations.csv, e.g., BE:RAS hasSynonym PFAM:PF00071
Create new file, e.g. equivalences.csv, with

I would opt for having a folder with separate files for mapping from different sources (PFAM, NCIT, BEL) into Bioentities. I already have one such file which currently lives within INDRA but could be moved over to Bioentities: https://github.com/sorgerlab/indra/blob/master/indra/resources/bel_indra_map.tsv
This is an example where the first column is an entry in a BEL name space and the second column is an entry in the BE name space. How about moving this over to Bioentities into a mappings folder and adding a few more such files for other sources?

Perhaps I should describe in more detail what I'm doing with the BE resources. I've written a custom program to extract the "target" entities from the relations file. The program then walks through the grounding_map file and adds "lexical" strings: keys for entries which point to any BE entities extracted in the first step. The final result is two lookup tables (complexes and families) which map the "surface strings" to the canonical BE names.
So, for example, 'ATK' is extracted from the relations file as a family name. Then, from the grounding_map file, the program adds alternate lexical strings which map to AKT, such as 'Akt', 'AKT-Ser473', 'p-AKT', 'PKB', etc. The addition of the lexical synonyms to our lookup tables is crucial, since we will likely encounter these in reading.

In #21 I added a new file that establishes that equivalence between BE entries and the PF/IP entries that @hickst sent. So with this in place, if REACH maps to, for instance, PF:PF00071, that will be equivalent to BE:RAS,

Good idea, but please see my comment on your PR #21 about namespace strings.

Okay, I merged #21 as is for now. So REACH can keep grounding to Pfam when the correct entry is available there and equivalences.csv can be used to map into Bioentities to find the unpacking of the family members.


[enhancement] Use CLI to simplify tauri-action usage

repo: https://github.com/lencx/tauri-release
From the configuration of tauri action to the triggering of github action, the configuration is cumbersome. I realized one-click publishing through scripts (this version tag will be created every time a version is released).
Install
npm i -D @tauri-release/cli

new
# generate tauri update-log file, the default name is `UPDATE_LOG.md`
tr new log

# generate github action file, the default name is `.github/workflows/release.yml`
tr new action

release
--git: Enable the git command to automatically submit after the version is updated, create a tag and push it to the remote repository (git add -> git commit -> git tag -> git push)
# major: 0.x.x -> 1.x.x
# minor: x.0.x -> x.1.x
tr release --[major|minor]

# patch: x.x.0 -> x.x.1
tr release --git

updater
# generate tauri update file, the default name is `updater/install.json`
tr updater --owner=GITHUB_OWNER --repo=GITHUB_REPO --token=GITHUB_TOKEN

Example

lencx/WA/package.json
lencx/WA/.github/workflows/release.yml

This may be something for https://github.com/tauri-apps/awesome-tauri/


Add support for editing the generated CHANGELOG.md file

PR to work on the enhancement proposed in #962.

Closed because no progress was made in 7 months.
So that other people can work on this. 👍


Getting syntax error on step 32/40

Step 32/40 : RUN if [ ${INSTALL_INTL} = true ]; then     apt-get install -y zlib1g-dev libicu-dev g++ &&     docker-php-ext-configure intl &&     docker-php-ext-install intl ARG GHOSTSCRIPT=false
 ---> Running in 889a3ca4a821
/bin/sh: 1: Syntax error: end of file unexpected (expecting "fi")
ERROR: Service 'php-fpm' failed to build:

Steps to reproduce:

Cloned laradock
git clone https://github.com/laradock/laradock.git dckr
copied env example
cp env-example .env
Launch containers
docker-compose exec workspace bash

@tblancog probably this type was fixed. Can you try again, if it is fixed please close this otherwise let me know to fix it. it's only missing fi

@Mahmoudz Tried again and it worked.
Thanks!


Yocto: Add Yocto Linux documentation tutorial page

Proposed changes
As outlined in #1329 as part of Yocto Linux support, this PR adds a tutorial for building Thin Edge for Yocto. It'd be nice if somebody from the team went through the steps in the tutorial and confirmed that everything is working and all is clear.
Types of changes

[ ] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Improvement (general improvements like code refactoring that doesn't explicitly fix a bug or add any new functionality)
[x] Documentation Update (if none of the other choices apply)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Paste Link to the issue

#1329
Checklist

[x] I have read the CONTRIBUTING doc
[x] I have signed the CLA (in all commits with git commit -s)
[ ] I ran cargo fmt as mentioned in CODING_GUIDELINES
[ ] I used cargo clippy as mentioned in CODING_GUIDELINES
[ ] I have added tests that prove my fix is effective or that my feature works
[x] I have added necessary documentation (if appropriate)

Further comments

@didier-wenzek Looks like I forgot to sign-off my commits, my bad. Should I rebase, sign them, and force push? Alternatively we could squash the changes to one commit, as there are not many changes.


Pasted nodes place on the same position instead of mouse position

Godot version:
v3.1.dev.calinou.1800664
OS/device including version:
windows 10 x64
Issue description:
In visual script
when copy a node in visual script area and paste, Godot place it on the same place instead of mouse position.
in big visual scripts this is problem, because user will drag the nodes and pan the view many times to place the nodes correctly.

This should be resolved by https://github.com/godotengine/godot/pull/28162

This is a related problem, but not resolved.

In 3.2.3 rc3 the duplicates get stacked:

I can't use shortcuts though for some reason.

This is still happening in 3.2.4 beta 4.
The pasted node(s) will not be at the user's mouse position or the current viewing area, making it useless to copy and paste nodes in a big and complex Visual Script.
This could also cause a serious problem in usability since a lot of nodes that the user intended to use elsewhere will be stacked at the same position which had so many nodes already, thus the reason why they were copied in the first place.
I am not sure if this issue could fit into the discussion tag since the current behavior definitely doesn't align well with the expected behavior of the copy and paste feature where the user expected to be able to control the pasted position/destination of the copied item. If they just want to copy and paste at the same place, that would be the duplication (ctrl+d).
However, with all that said. I don't think that to copy and paste a group of complex nodes to use at many places in the script is a good way to work in Visual Script. Especially, when we don't have the ability to search for a function call and a variable in Visual Script yet (proposal #1780). I think it would be more manageable if that kind of a node group is converted to a function by which the differential in logics is controlled through the function's aruments.

Possibly solved with merged PR  https://github.com/godotengine/godot/pull/54564


Better instructions needed

Here are the steps on how to use copy-with-style
Step-1: Install Packages (only works if you have admin privileges on Windows10)
Step-2: Once the Package is installed create a Preset1 as shown here https://atom.io/packages/copy-with-style
Step-3:Open a Google doc
Step-4 NOW Here is where it gets tricky, Highlight the code Then in ATOM Edit Menu click on copy
Step-5 Go to Google Docs and right click on the mouse and paste
( IT DOES NOT PASTE The background and color)
I also tried alt-a-1 after highlighting the code then repeated the step 5 with any success

Same


Can't Administer Subscriptions

I need the ability to manage Subscriptions and Alerts using the API for all workspaces.

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

ID: 830da8e8-8526-0e6a-d83f-0c804226e006
Version Independent ID: 3e7b9d31-133d-1b2f-fded-d9218284155b
Content: Subscribe yourself and others to reports and dashboards - Power BI - Power BI
Content Source: powerbi-docs/service-report-subscribe.md
Service: powerbi
Sub-service: powerbi-service
GitHub Login: @maggiesMSFT
Microsoft Alias: maggies

Thanks for your comment, @Saltzmea. Looks like this is product feedback, rather than documentation feedback – you can submit product feedback or feature ideas by going to https://ideas.powerbi.com. You can also engage with the Power BI engineering team by being active in the Power BI Community, found at https://community.powerbi.com.


Support RGB TIFFs.

Closes #133.

Power-cycled to re-run CI now that docs build has been fixed on main.


{AKS} Fix several tests

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

az aks addon list-available
az aks addon list
az aks addon show

Some of the addon commands are broken by the newly added addon web application routing by PR #4872. cc @yizhang4321
General Guidelines

[ ] Have you run azdev style <YOUR_EXT> locally? (pip install azdev required)
[ ] Have you run python scripts/ci/test_index.py -q locally?

For new extensions:

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

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

AKS


⚠ Game Join API Endpoint has degraded performance

In e647693, Game Join API Endpoint (http://gamejoin.roblox.com/) experienced degraded performance:

HTTP code: 502
Response time: 5293 ms

Resolved: Game Join API Endpoint performance has improved in 56e2141.


Oracle: auto quote reserved keywords

For instance if you use
Insert.Into("BLAH").Row(new { GROUP = "TEST"});
It will fail and apart from using the dictionary overload and adding the quotes yourself you can't do anything about it.
See http://docs.oracle.com/cd/B19306_01/em.102/b40103/app_oracle_reserved_words.htm#CHDCGECI for a list of reserved keywords

Only when not using quoted identifiers of course ;)

This seems to be only happening on Insert, Update Delete data expressions, need to figure out how to tackle that issue, will come back to it

After careful consideration, i think this is unfixable in the current state.
Either use

QuotedIdentifier
Quote yourself using dictionary overload

It's already weird we have a oracle quoter that actually does not quote, this change would make the processor use both quoters and switch when needed.


Unable to Connect to CVAT Server

Actions before raising this issue

[X] I searched the existing issues and did not find anything similar.
[X] I read/searched the docs

Steps to Reproduce
Open the CVAT web interface.
Attempt to connect to the server.
Observe the pop-up message indicating connection issues.
Expected Behavior
I expected to successfully connect to the CVAT server and access its functionalities without encountering error messages.
Possible Solution
It may be related to the backend services (Database, Redis, Open Policy Agent) not running properly. Checking their statuses and ensuring they are all operational might resolve the issue.
Context
This issue is preventing me from using CVAT for my project. I need to access the annotation tools to proceed with my work.
Environment
- Git version - git version 2.34.1
- Docker version - Docker version 27.3.1, build ce12230
- OS - Ubuntu 22.04.4 LTS
- Windows system - X11

any logs?

Its solved @azhavoro

I cleared the storage in the system and its now good thank you for your kind response