init.el is giving me an error

Im running emacs GUI on a Windows 7 machine and is giving me an error that is breaking emacs(it wont let me press any keys not C nor M) the error is as follows:
Error in post-command-hook (global-magit-wip-save-mode-check-buffers): (file-error "Searching for program" "no such file or directory" "git")
I've experimented on my mac emacs with your setup and it works btw :)
thank you for your time!

Hmm. I've probably worked around this with 024f651. magit-wip-save-mode really shouldn't fail hard like that, so this may be an upstream issue. (I've generally assumed that any machine with this emacs.d config on it will have git on it too, since that's the standard installation method.)

now its giving me the error Cannot save customizations; init file was not fully loaded.

(file-error "http://melpa.org/packages/cider-20150201.1117.tar" "Not found")
signal(file-error ("http://melpa.org/packages/cider-20150201.1117.tar" "Not found"))
url-insert-file-contents("http://melpa.org/packages/cider-20150201.1117.tar")
package-install-from-archive([cl-struct-package-desc cider (20150201 1117) "Clojure Integrated Development Environment and REPL" ((clojure-mode (4 0 0)) (cl-lib (0 5)) (dash (2 4 1)) (pkg-info (0 4)) (emacs (24)) (queue (0 1 1))) tar "melpa" nil ((:url . "http://www.github.com/clojure-emacs/cider") (:keywords "languages" "clojure" "cider")) nil])
mapc(package-install-from-archive ([cl-struct-package-desc cider (20150201 1117) "Clojure Integrated Development Environment and REPL" ((clojure-mode (4 0 0)) (cl-lib (0 5)) (dash (2 4 1)) (pkg-info (0 4)) (emacs (24)) (queue (0 1 1))) tar "melpa" nil ((:url . "http://www.github.com/clojure-emacs/cider") (:keywords "languages" "clojure" "cider")) nil]))
package-download-transaction(([cl-struct-package-desc cider (20150201 1117) "Clojure Integrated Development Environment and REPL" ((clojure-mode (4 0 0)) (cl-lib (0 5)) (dash (2 4 1)) (pkg-info (0 4)) (emacs (24)) (queue (0 1 1))) tar "melpa" nil ((:url . "http://www.github.com/clojure-emacs/cider") (:keywords "languages" "clojure" "cider")) nil]))
package-install(cider)
(if (or (assoc package package-archive-contents) no-refresh) (package-install package) (progn (package-refresh-contents) (require-package package min-version t)))
(if (package-installed-p package min-version) t (if (or (assoc package package-archive-contents) no-refresh) (package-install package) (progn (package-refresh-contents) (require-package package min-version t))))
require-package(cider)
eval-buffer(#<buffer  load-612684> nil "c:/home/.emacs.d/lisp/init-clojure-cider.el" nil t)  ; Reading at buffer position 81
load-with-code-conversion("c:/home/.emacs.d/lisp/init-clojure-cider.el" "c:/home/.emacs.d/lisp/init-clojure-cider.el" nil t)
#(init-clojure-cider nil nil)
ad-Advice-require(# init-clojure-cider)
apply(ad-Advice-require # init-clojure-cider)
require(init-clojure-cider)
(progn (require (quote init-clojure-cider)))
(if (>= emacs-major-version 24) (progn (require (quote init-clojure-cider))))
eval-buffer(#<buffer  load> nil "c:/home/.emacs.d/init.el" nil t)  ; Reading at buffer position 2866
load-with-code-conversion("c:/home/.emacs.d/init.el" "c:/home/.emacs.d/init.el" t t)
load("c:/home/.emacs.d/init" t t)
#[0 "\205\262

The package list is out of date. You need to run M-x package-refresh-contents, then restart Emacs.

It Worked!! Thanks alot!

Great! :-)


GitHub Node.js 12 actions are deprecated.

We must update the actions we use in CI on all our repo :
Node.js 12 actions are deprecated. For more information see: https://github.blog/changelog/2022-09-22-github-actions-all-actions-will-begin-running-on-node16-instead-of-node12/. Please update the following actions to use Node.js 16: actions/checkout, docker/login-action, docker/login-action, actions/checkout

Can you list all the actions that we have to upgrade ?

Will be fixed by https://github.com/lenra-io/lenra_cli/issues/89


Improve process for building and provisioning distribution file

Currently, the building process is done manually and locally, and the resulting file is just copied to wherever it is needed.
Building: probably via a github action on push to master, and then uploading the build artefacts (minified JS file) somewhere.
Distribution: perhaps via jsdeliver? https://www.jsdelivr.com/github
P.S. this issue should probably be moved to https://github.com/psychoinformatics-de/json-editor.

Moved to https://github.com/psychoinformatics-de/json-editor/issues/2


No Outline for unknown reason

In this project https://github.com/BBj-Plugins/RestBridge
the folowing files show no outline, for unknown reasons:
RestBridge.bbj
RestBCAdapter.bbj
OutputHandler.bbj
However, InputHandler.bbj is fine.
What is the reason?

also, in the same git repo, the ExampleBCs subfolder files have the same issue


Redocusaurus should collect heading ids / anchors

Hi
With Docusaurus v3.1 we introduced a broken anchors checker:
https://docusaurus.io/blog/releases/3.1#broken-anchors-checker
https://docusaurus.io/docs/docusaurus-core#useBrokenLinks
This requires collecting anchors for headings you create, otherwise Docusaurus doesn't know about those created anchors.

Note this api exists in Docusaurus 3.1 but not in 3.0 so you probably want to release a new major version requiring Docusaurus 3.1 to support this.

See also a Docusaurus user that gets reported Redocusaurus anchors links as broken:
https://github.com/facebook/docusaurus/issues/6810#issuecomment-1879387076

Thanks @slorber for forwarding the issue! I'll try to look into what API redoc makes available to get the anchor ids but since the rendering is done mostly by redoc and not by this plugin it's possible that this is unfixable. Will try still

I understand.
Isn't there a way to know the heading its ahead of time by using AppStore or the spec file?
You don't have to call collectAnchor(anchor) in a heading component, you can do this at any level.
const {collectAnchor} = useBrokenLinks();
const anchors = getAnchorsFromSpec(spec);
anchors.forEach(collectAnchor);

The question is: can getAnchorsFromSpec(spec) be implemented?


Request to add control for away/end away

Request to add control to set away or end away mode.
Since the thermostat already has pre-set temps for away mode, this would simplify control if it were available to be tied to presence. This would also suspend pre-scheduled temperature changes.

SmartThings is shutting down its Groovy-based platform at the end of 2022. This project is no longer maintained.


fix: update default config to be empty

When I proposed changing default config I wasn't aware of the horrible merging strategy of the config
one of my friends was trying to use this and I was shocked to see array's were merging instead of replacing, which made it impossible to remove the removeDimensions plugin.
given the horrible unexpected outcome, I suggest changing it back to nothing (sadly)

@cpsoinos please help merge this
I don't know what version bump we should use, given that it may break or not break some projects

also sorry about the excessive mentions these days.

I think we can do a minor version bump here. Can help merge shortly.

@jd1378 would you be interested in taking over as this package's maintainer? You're clearly invested in it, and I no longer use Nuxt (or even Vue) regularly as my company uses Next.js and React.

@cpsoinos that would be nice, I'm interested in it as long as I'm actively using it, like yourself, so I would like you to remain as the owner, But for the meantime I can definitely take over the maintenance (for like next ~ 3-6 months)

@cpsoinos that would be nice, I'm interested in it as long as I'm actively using it, like yourself, so I would like you to remain as the owner, But for the meantime I can definitely take over the maintenance (for like next ~ 3-6 months)

Invited you to be a collaborator!

any tips on how I should maintain the repo would be nice (small notes on how releasing works to be more specific)

@jd1378 Right now it's setup to auto release upon merge to main via this action. I'm open to suggestions on how to improve the process!

Thanks for adding me
I see, that would be helpful to have an auto release
I'll try to improve it if I came up with any idea

ok, thanks 😄

I must have been mistaken when i said it should be a minor instead of a patch. The semantic-release action has rules it follows haha

thanks, good to know

@cpsoinos it seems the package is not available on npm after release,
when something like this happens, what would you do ? is it possible to rerun release ?

Oh that's weird, will take a look as soon as I can


Using embeddings for Face Recognition task

Hello! I'm getting face embeddings with these steps:
from torchvision.transforms import functional as VF

cut = cv2.resize(cut, (112, 112), interpolation=cv2.INTER_AREA) # resize image to 112x112 px
cut = VF.to_tensor(cut).to(self.device).unsqueeze(0) # convert ndarray to torch.Tensor, send it to GPU and add batch dimension
cut = VF.normalize(cut, [0., 0., 0.], [1., 1., 1.]) # normalize image as in inference/gen_feat.py
embed = self.model(cut) # do forward pass
embed = F.normalize(embed, p=2, dim=1) # L2-normalize embedding vector
embed = embed.squeeze(0).cpu().numpy() # remove batch dimension and convert embedding vector back to ndarray

For face recognition I'm using cosine similarity between two images.
It works pretty good if I look straight into camera and I get 0.7-1.0 similarity, but if I rotate my face left or right, I get something between 0.1-0.3.
Weights used: https://drive.google.com/file/d/1Bd87admxOZvbIOAyTkGEntsEz3fyMt7H/view
Am I doing something wrong? Maybe I should use another metric or not normalize embedding vectors?

Hi. I guess it's because of the incorrect pre-processing steps.
The images should be aligned into 112x112 pixels based on their 5 facial landmarks.
import cv2
import numpy as np
from skimage import transform as trans
tform = trans.SimilarityTransform()

arcface_src = np.array(
    [[38.2946, 51.6963], [73.5318, 51.5014], [56.0252, 71.7366],
     [41.5493, 92.3655], [70.7299, 92.2041]],
    dtype=np.float32)
img = cv2.imread(imgname)
tform.estimate(ldmks, arcface_src) #ldmks of current image -- 5 points, same format as arcface_src
M = tform.params[0:2,:]
cut = cv2.warpAffine(img, M, (112, 112))

Here is an example from Insightface


Apache airflow dag runs

Hey, I just made a Pull Request!
Added a new column in the table to quickly view the latest DAG runs, plus a link to it if you want to have a deeper look.

Also:

Table columns are togglable
Fixed bug with turning on/off the DAGs

Check https://github.com/backstage/backstage/issues/12812 for discussions
:heavy_check_mark: Checklist

[x] A changeset describing the change and affected packages. (more info)
[x] Added or updated documentation
[x] Tests for new functionality and regression tests for bug fixes
[x] Screenshots attached (for UI changes)
[x] All your commits have a Signed-off-by line in the message. (more info)

Hi @Rugvip, I'm not really understanding why the automated actions are failing 😢
Can you help clarify it?

@stichiboi not sure tbh, but the changes are looking a bit strange. I'd try a rebase onto the latest master of backstage/backstage

I synced master and re-committed the changes in this new PR
Not sure what the issue was on this PR 😕


More granular Terms and Conditions control

Currently trying to integrate this SDK for Tap to Pay but could use some more granular terms and conditions control. For example, if I only want an admin to be able to accept the terms and conditions (which is even a use case mentioned on item 2 in Apple's best practices, I'd have to know whether or not they've already been accepted or at least have some feedback regarding when they've accepted them so I can save that. Or even just being able to call the terms and conditions without having to connect to a reader.
Is the support for this type of granularity on the roadmap? I can see them in the ProximityReaderAPI (and maybe the stripe IOS SDK but I can't tell) so I know it exists but not sure if itll be supported soon.

Hey did you get a chance to review the discussion in https://github.com/stripe/stripe-terminal-ios/issues/252? In the next release we are adding support for tosAcceptancePermitted field in the connect configuration. Would that help your case?

Hey! This seems like it could be pretty useful for some of my needs but could this be utilized to determine if an account or hasn't accepted the terms and conditions? There are some features where I'd like to conditionally show some things if the terms have been accepted but I can't think of a nice way to do that with this?

Unfortunately that's not possible today, best way is to see if they made a payment already. I'll close this issue in favor of the one in stripe-terminal-ios repo (https://github.com/stripe/stripe-terminal-ios/issues/252) as the feature would need to exist in the iOS SDK first for us to be include in React Native.


Update templates.md

Adjusted title only

This was a leftover branch that was never turned into a PR - it needs updating or recreating with the new content structure


Wake on lan to turn the device on

Hi, is there a way to use wakeonlan command to turn the device on?

Currently this feature is not implemented yet. But will consider adding this in future release.

Hi dwaan, this feature would be a great enhancement. If it helps, the following code works fine for me on the terminal while the TV is in sleep/standby mode:
adb -s  shell "input keyevent KEYCODE_WAKEUP"

Hi dwaan, this feature would be a great enhancement. If it helps, the following code works fine for me on the terminal while the TV is in sleep/standby mode:
adb -s shell "input keyevent KEYCODE_WAKEUP"

Hey, Thank you. That’s the command that this plugins use right now.
The wakeonlan could be useful for some device that disconnected from network when it sleep, I think.

I changed the command in index.js to
// When it sleep, wake it up
exec(wakeonlan XX:XX:XX:XX:XX:XX, (err, stdout, stderr) => {
(XX:XX:XX:XX:XX:XX is mac address of my device)
and now it works as I wanted.

Closing this issues as it's implemented in newer version, currently in beta.


TryUpdateModelAsync using JObject

Is it possible in some way to update one original JObject with the changed values from another JObject?

JsonPatch would be the closest solution to this, but it does not currently support updating JObject instances. I'd recommend using StackOverflow to figure out how to do this with the current version of the framework.


Tutorial: Add remark command

Codecov Report

Merging #25 (8bf4b69) into master (803a0de) will decrease coverage by 0.00%.
The diff coverage is 76.36%.

@@             Coverage Diff              @@
##             master      #25      +/-   ##
============================================
- Coverage     72.15%   72.15%   -0.01%
- Complexity      399      416      +17
============================================
  Files            70       73       +3
  Lines          1232     1282      +50
  Branches        125      130       +5
============================================
+ Hits            889      925      +36
- Misses          311      323      +12
- Partials         32       34       +2

Impacted Files
Coverage Δ
Complexity Δ

.../seedu/address/logic/parser/AddressBookParser.java
88.88% <0.00%> (-11.12%)
12.00 <0.00> (ø)

...eedu/address/logic/parser/RemarkCommandParser.java
0.00% <0.00%> (ø)
0.00 <0.00> (?)

src/main/java/seedu/address/ui/PersonCard.java
0.00% <0.00%> (ø)
0.00 <0.00> (ø)

...c/main/java/seedu/address/model/person/Remark.java
75.00% <75.00%> (ø)
4.00 <4.00> (?)

...java/seedu/address/logic/commands/EditCommand.java
97.05% <100.00%> (+0.04%)
12.00 <0.00> (ø)

...va/seedu/address/logic/commands/RemarkCommand.java
100.00% <100.00%> (ø)
11.00 <11.00> (?)

...a/seedu/address/logic/parser/AddCommandParser.java
100.00% <100.00%> (ø)
5.00 <0.00> (ø)

...ain/java/seedu/address/logic/parser/CliSyntax.java
85.71% <100.00%> (+2.38%)
1.00 <0.00> (ø)

...c/main/java/seedu/address/model/person/Person.java
97.72% <100.00%> (+0.10%)
21.00 <1.00> (+1.00)

.../java/seedu/address/model/util/SampleDataUtil.java
25.00% <100.00%> (+5.00%)
2.00 <1.00> (+1.00)

... and 5 more

Continue to review full report at Codecov.

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


cellxgene-schema must update validation for disease_ontology_term_id

Context
See disease_ontology_term_id.
4.0.0 was:
This MUST be a MONDO term or "PATO:0000461" for normal or healthy
4.1.0 adds further requirements:
This MUST be one of:

"PATO:0000461" for normal or healthy
the most accurate child of "MONDO:0000001"for disease
"MONDO:0021178" for injury or preferably its most accurate child

@brianraymor @danieljhegeman Is there any way for us to know if the MONDO term provided is the most accurate term for the disease / injury? Or would the work on the validation side just be to verify that a MONDO term provided is a child of either MONDO:0000001 or MONDO:0021178?

Is there any way for us to know if the MONDO term provided is the most accurate term for the disease / injury?

No. This language is a common pattern for schema fields for curators to encourage the submitter to not use a general term (which often happens in the first iteration). Curators will often review the research and request the submitter to update the precision.

most accurate child

@brianraymor please confirm: a child is a direct descendant, and not a multi-level descendant, i.e. B is a child of A but C is not a child of A in: A > B > C where A is the foremost ancestor. cc @joyceyan

Let's use a concrete, simple case - MONDO.
It's any descendant of the disease root:

which unfortunately allows non-human animal disease to be specified for human observations at the moment per my comments in #cell-science-data-wrangling.

@brianraymor Ok, thanks for clarifying. I'll make a ticket to change the language in the schema and code base.

Approved by Lattice


support IPv6 address when using --extra-files

Fixes #246.

@Mic92 would you have an opportunity to review this?

Lack of response. Please re-open if you plan to continue working on this.


Wip/bewest/custom gatt profile

Note taking... potentially add Pair, and ConnectProfile.

http://www.bluez.org/bluez-5-api-introduction-and-porting-guide/
https://git.kernel.org/cgit/bluetooth/bluez.git/tree/doc/gatt-api.txt
https://git.kernel.org/cgit/bluetooth/bluez.git/tree/doc/device-api.txt

Howdy, I cleaned up some of the dbus changes I needed for https://github.com/bewest/Adafruit_Python_BluefruitLE/pull/1, https://github.com/bewest/openxshareble.
In particular, I found the UART class and examples very very helpful, thanks!
Has anyone thought about releasing it through pypi so projects can use a package manager to install the python lib?

LGTM

Aside, there's some useful stuff in https://github.com/adafruit/Adafruit_Python_BluefruitLE/pull/7, I'm wondering if we can maybe sync up.

Aside, dbus-python is older, pydbus is the newer thing
Since dbus-python, pydbus has come around to replace dbus-python with gdbus based backend via gi.repository goodness.  I've highlighted a number of issues which have all been solved as of glib 2.45.

For posterity:

https://bugzilla.gnome.org/show_bug.cgi?id=656330
Naive attempt (missing an extra loop/threading thing ) https://github.com/bewest/Adafruit_Python_BluefruitLE/pull/3
https://gist.github.com/bossjones/e054e224f5b8578dff0a#gistcomment-1710180
https://github.com/LEW21/pydbus/issues/7

The updates in glib 2.45 are really only needed in the scenario where someone needs to export services on dbus by using ble.  In scenarios like this, the older dbus-python api has a problem where nested mainloops "coalesce" causing the event listeners to be duplicated.
I've spent several weeks evaluating pydbus, which really needs glib 2.45 to work, vs txdbus, versus the limitations of the older API.  The older API can be worked around in my scenario by promoting variety of "background threads" to full blown processes, all using the older API.  However, it's not really open to new features or patches (eg to workaround properties), and pydbus is, pending glib 2.45 becoming more widely available.
I took a very brief stab at duplicating a lot of things to adapt the pydbus methods with minimal changes.  One big change I mad was eliminating the dbus-pythhon's DBusGMainLoop, which naturally causes the first call to Event.wait to hang.  I've read in various places about either iterating the mainloop and the threading docs, as well as techniques using generators, but not quite sure how to proceed with "minimal changes" versus what redesigns using a pure pydbus API might be desired (for client, at least  until glib 2.45 is widespread)?

Is this something that can be merged?  Or do we need to create a fork over at https://github.com/openaps/ for ongoing openxshareble support?

This library has been deprecated. Please update to use this new library:
https://github.com/adafruit/Adafruit_Blinka_bleio
PR's can be submitted to the new library.


testing example requested

The documentation suggests," it would be useful to have further testing of the container." And that definitely sounds interesting after getting the basics working with CircleCI. But I am not quite sure how to go about that next and clearly the documentation author had some ideas about that?
Would it use nbval or was something simpler envisioned?
I don't know what the envisioned example was. If you are open to suggestions, maybe the example could be to see if an expected output is made?
We have code in a notebook that produces a file and to keep checking things are working we are curious to use continuous integration to test if the notebook (or isolated code) makes the file. Would that code be needed to be moved to postBuild and then some function check if the file is made. Is that handled in postBuild or by CircleCI?

From my side the idea was to add a demo of how to run "anything" inside the container as part of the CI setup. I made a (very minimal) start here https://github.com/binder-examples/continuous-build/pull/8
The motivation is that just because repo2docker can successfully build your repository doesn't mean it actually does something useful. For that you need to try and run "something" that exercises your container somehow. It could be "try and start python and import package X" or "run the first three cells of every notebook in the container" or ... basically what ever makes sense for that repo. The hardest part is getting going so having a demo of how to add a step to CircleCI that uses nbval/papermill/RStudio would be a good starting point.

I took your minimal approach and adapted in the python code that test the basics of our import and image generating, and was able to get it to work to look for creation of a file. It is here. (I think. I still need to break it and make sure.)
Plus I added a badge to the README to get feedback. This is what we were hoping for.
Eventually it would be nice to have an example with nbval/papermill but it took me quite a bit to get this code working because of the need to make the code to do something useful and test one line. And, thus I won't be tackling that myself too soon.


How to run this tool?

I'm working on an institute-wide pipeline for JCVI and had some trouble running your tool.
Here's my version installed via pip:
 viral_verify --version
viral_verify, version 0.1.1

Here's my command:
viral_verify -i veba_output/binning/47-Drifterexpttime4punches_S40/tmp/unbinned.fasta -o veba_output/binning/47-Drifterexpttime4punches_S40/intermediate/viral_viralverify_output -H /usr/local/scratch/CORE/jespinoz/db/pfam/v33.1/Pfam-A.hmm -t 16

Edit: I had to decompress the PFAM database which was the error in the original post that I've edited since then.
Should I be using the PFAM database or the database from FigShare?
Can you update the Usage on your GitHub?
This is the results output:
veba_output/binning/47-Drifterexpttime4punches_S40/intermediate/viral_viralverify_output/
├── classified-fasta-output
│   ├── unbinned-chromosome.fasta
│   └── unbinned-unclassified.fasta
├── unbinned-circularized.fasta
├── unbinned-genes.fa
├── unbinned-hmmsearch.domtblout
├── unbinned-hmmsearch.output
├── unbinned-proteins-circularized.fa
├── unbinned-proteins.fa
└── unbinned-results.csv

I ran the version from GitHub on a differen tdataset and got the following output:
testing/viralverify_output/
├── oral_viruses_domtblout
├── oral_viruses_feature_table.txt
├── oral_viruses_genes.fa
├── oral_viruses_input_with_circ.fasta
├── oral_viruses_out_pfam
├── oral_viruses_prodigal.log
├── oral_viruses_proteins_circ.fa
├── oral_viruses_proteins.fa
├── oral_viruses_result_table.csv
├── Prediction_results_fasta
│   ├── oral_viruses_chromosome.fasta
│   ├── oral_viruses_plasmid.fasta
│   ├── oral_viruses_plasmid_uncertain.fasta
│   ├── oral_viruses_virus.fasta
│   └── oral_viruses_virus_uncertain.fasta
└── viralverify.log

1 directory, 15 files

How come the output is so different between the pip and GitHub versions?

That's funny - someone else forked this repo about a year ago, refactored and submitted to pypi as viral_verify (https://github.com/peterk87/viral_verify) . That's why the output and such is so different. Thanks for pointing that out!
Meanwhile, our current github version is awaiting approval for bioconda channel. As soon as that happens, I'll update accordingly.

That is so weird. They also took the namespace too?
What's the process like for getting something on bioconda?

That's open source, after all...
Bioconda submission turned out to be pretty straightforward. Create recipe (yaml and build.sh files) with all metadata and dependencies, test and commit to bioconda recipes repository. https://bioconda.github.io/contributor/workflow.html
Then, after all CI tests, it needs ti be reviewed by someone of bioconda members. No idea how long it takes :)
github.com/bioconda/bioconda-recipes/pull/30186

I'm working on an institute-wide pipeline for JCVI and had some trouble running your tool.
Here's my version installed via pip:
 viral_verify --version
viral_verify, version 0.1.1

Here's my command:
viral_verify -i veba_output/binning/47-Drifterexpttime4punches_S40/tmp/unbinned.fasta -o veba_output/binning/47-Drifterexpttime4punches_S40/intermediate/viral_viralverify_output -H /usr/local/scratch/CORE/jespinoz/db/pfam/v33.1/Pfam-A.hmm -t 16

Edit: I had to decompress the PFAM database which was the error in the original post that I've edited since then.
Should I be using the PFAM database or the database from FigShare?
Can you update the Usage on your GitHub?
This is the results output:
veba_output/binning/47-Drifterexpttime4punches_S40/intermediate/viral_viralverify_output/
├── classified-fasta-output
│   ├── unbinned-chromosome.fasta
│   └── unbinned-unclassified.fasta
├── unbinned-circularized.fasta
├── unbinned-genes.fa
├── unbinned-hmmsearch.domtblout
├── unbinned-hmmsearch.output
├── unbinned-proteins-circularized.fa
├── unbinned-proteins.fa
└── unbinned-results.csv

I ran the version from GitHub on a differen tdataset and got the following output:
testing/viralverify_output/
├── oral_viruses_domtblout
├── oral_viruses_feature_table.txt
├── oral_viruses_genes.fa
├── oral_viruses_input_with_circ.fasta
├── oral_viruses_out_pfam
├── oral_viruses_prodigal.log
├── oral_viruses_proteins_circ.fa
├── oral_viruses_proteins.fa
├── oral_viruses_result_table.csv
├── Prediction_results_fasta
│   ├── oral_viruses_chromosome.fasta
│   ├── oral_viruses_plasmid.fasta
│   ├── oral_viruses_plasmid_uncertain.fasta
│   ├── oral_viruses_virus.fasta
│   └── oral_viruses_virus_uncertain.fasta
└── viralverify.log

1 directory, 15 files

How come the output is so different between the pip and GitHub versions?

Hi there, so in the end which database did you use or is that one annotated out a bit more accurately.

I use geNomad now.

I use geNomad now.

All right, thanks.

Apologies @AndAvia , I wrote that from my phone but should have given more context.  Here's the geNomad publication: https://www.nature.com/articles/s41587-023-01953-y and here's the GitHub: https://github.com/apcamargo/genomad
I developed a wrapper around geNomad for my "binning-viral" module (though, it doesn't really bin and more so identifies contigs that are viral) in my VEBA package.  Here's the publication for VEBA (https://academic.oup.com/nar/advance-article/doi/10.1093/nar/gkae528/7697622) and here's the GitHub (https://github.com/jolepsin/veba).
If you only want to perform viral analysis, I would recommend just using geNomad because VEBA has a lot of functionality and requires more dependencies/databases.

@jolespin Thank you so much for your patience in replying! I only need to do virus identification at the moment, because there are so many virus identification software, I'm going to use genomad, VIBRANT, virfinder, deepvirfinder, virsorter, virsorter2, ViralVerify and these, but you said that ViralVerify two databases have different results, and I don't know which database to choose.I had a chance to look at your VEBA, and I found it very impressive! Wishing you a wonderful day!

Hi,
Since this tool was released in 2021 and didn't receive significant updates, I'd also recommend for checking newer alternatives. If you are still interested to run exactly viralVerify, I'd definitely retrain the db with the updated pfam-a and genbank viral/plasmid/chromosomal sequences


Update expand-disks.md

Shrinking an existing disk isn’t supported and may result in data loss. After expanding the disks, you need to expand the volume in the operating system to take advantage of the larger disk.

#sign-off


Cleanup metadata stored on OpenHuman

Fixes #468.
There was considerable overlap between the metadata and the file contents stored on OpenHuman. This change streamlines the metadata to remove the descriptive text. The descriptive text is now retrieved by downloading the file from OpenHuman instead.

Some things to note about this PR:

This doesn't use any OAuth authentication when downloading the file from OpenHumans. I couldn't see anything in the docs about this, but the OpenHumans CLI implementation appears to work similarly. Is this correct?
When combinding the metadata and file data from OpenHumans, if there's a conflict between them (e.g. the same key is found in both), the items in the metadata will take priority. In theory they should always be the same anyway, but should this be reversed?

Thanks @llewelld, that's some great work!
Regarding your questions:

Indeed, the file download doesn't use the access_token. What happens instead behind the scenes is that the exchange_member call that is triggered by member.list_files() generates a signed download URL for each file. Those URLs are valid for 60 mins before expiring.

Great question! Maybe we could include a quick check in the function that does the merging and trigger an exception if that case should ever happen? That way we can have a longer think about how to resolve it without now implementing potentially unexpected behaviour down the line!

Thanks for the comments @gedankenstuecke which I really appreciate. I've converted this to draft while I sort out the unit tests.
Thanks also for your explanations of 1 and 2. I'm glad 1 is the correct behaviour.
Concerning 2, triggering an exception would be fine for new stories (since it shouldn't then happen), however it will definitely then trigger if data that's previously been uploaded to OpenHumans is used after this is merged. Would that be an issue?

Oh, great point for #2, which I guess would be fine as we don't have any production data yet, but maybe annoying for development. So happy to have the merging happening as you described for overwriting the data!


data percentage in loadclearlandsat breaks geotiff writer

Need to change data percentage datatype or make array of same size as others else geotiff writer won't work without dropping it

Need to: include tests to do multiple timesteps or timesteps with no timestamp
test for multiple dtypes and force if necessary (geotiff doesn't support multiple dtypes)


Add Timezone for conversational action

Conversational actions are sending timezone information in each request.
This information is added to the timezone plugin.

@rmtuckerphx I tested this version with a local npm install.

Hey @rmtuckerphx it would be great if you would merge and release this! We have a project waiting to take this change live.

Hi @rmtuckerphx , did you have the time to take a look at this? Thanks!

@freiSMS Published to npm as v1.2.0

Great @rmtuckerphx . Thanks!


Can't access .rebuild() static method when using functional components

Hi everyone,
I am struggling to refresh tooltips with my conditional content. I know i need to rebuild after the content is updated, but I can't access the .rebuild() method...
In the past with functional components I would create a reference to the component to access its methods, but this does not work in this case. Please see below example -
import { useState, useRef } from "react"
import ReactTooltip from 'react-tooltip'

const Component1 = () => {
  return
hover over me 1

}

const Component2 = () => {
  return
hover over me 2

}

//will show tool-tip on first load, but not after the renderComponent is toggled off then on again
export default function Main() {
  const [ currentComponent, setComponent ] = useState<JSX.Element>(Component1)
  const [ renderComponent, setRenderComponent ] = useState<boolean>(true)
  const tooltipComponentRef = useRef<ReactTooltip>(null)

  function cycleComponentClick() {
    if (currentComponent.props.children === "hover over me 1") {
      setComponent(Component2)
    }
    else {
      setComponent(Component1)
    }
  }

  function toggleRenderComponent() {
    setRenderComponent(!renderComponent)
  }

  //tooltipComponentRef.current?.rebuild() //gives error: "Property 'rebuild' does not exist on type 'ReactTooltip'..."

  return (

      <button onClick={cycleComponentClick}>Cycle Component</button>
      <button onClick={toggleRenderComponent}>Toggle Render Component</button>
      {renderComponent ? currentComponent : null}
      <ReactTooltip ref={tooltipComponentRef} />

  );
}

Is this considered a bug or is there a work-around to help with the issue?
Thanks in advance

It seems you should use data-for attribute for dom node (your component) and dynamically change ID prop for ReactTooltip component then it will match data-for === id and will re render when you hover for different components. It is not unusual case tho


Improve letter spacing in docs until we have more font weights

Description
The letter-spacing in docs doesn't look right now, this should help. For now at least.

easy approval


Add task role option

This allows you to specify a task role ARN that is passed to the container.

Thanks @patrobinson!


Update circleci image circleci/golang:1.17 to cimg/go:1.18

Ⅰ. Describe what this PR does
Ⅱ. Does this pull request fix one issue?

fixes #XXXX
Ⅲ. List the added test cases (unit test/integration test) if any, please explain if no tests are needed.
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

/lgtm
/approve


Export data

Well, I know that @rosshadden is working (or not?) on sync with taskserver, but nevertheless I think that it'd be a nice option to be able to actually export taskwarrior's data (you know, all these things that are normally stored in ~/.task).
Problem is that when removing an app, Android removes its storage, so after uninstalling and installing it back we loose all our precious tasks. Which is bad in my opinion. Probably not that bad if you have a remote copy of them, but still bad.
Is packing these files (to some kind of zip probably) and storing them on sdcard is possible at all? Or maybe some other way to export the data?

Indeed, this is possible. I must check whether Carbon can backup this information but without root access I hardly think so.

The idea is to create a ZIP file with taskwarrior's data inside application cache and then share it using ContentProvider. I need to research when the cache can be cleared in order not to remove a currently sharing file.

That's exactly what I meant (I think ;))


[RUNTIME] Simplify caching API

This simplifies the caching API to require callers pass the entire list
of files to cache in a "group" in the put_group method.  This allows
caching backends to e.g. serialize these files into a single cache blob,
which can avoid issues with caching atomicity.

I assume this PR is dead at this point?


Multiple instanses

Android 10.
The case:
I build 2 apps with the library. I run one song in the first app and then another one in the second. So there are two notifications with the controls. When i click on any button(at least prev/next) both apps get the event message.
I see two solutions:

Perfectly, if the library can distinguish the apps some way and send the message only for the necessary one.
If it is too problematic, the library can at least send the application name with the event to let the programmer distinguish it in the code. const application= JSON.parse(action).application;

Good catch, I haven't tested that scenario, I will see if we can potentially random generate an APP ID for the notification channel so that it is unique.
The problem at the moment is that a fixed notification channel is used which is always the same.

Good catch, I haven't tested that scenario, I will see if we can potentially random generate an APP ID for the notification channel so that it is unique.
The problem at the moment is that a fixed notification channel is used which is always the same.

Great news. Waiting for the release(npm)

weirdly I have tried with randomised channel id and notification id but they still overlap.  I think because both instances are registering broadcast receivers to catch media key events.  I'll try a few more things and see if I can find a way to filter this.

How about my second option in the first message? You can pass application name from config.xml to the client side. And we can handle messages only for the necessary app.


low horizontal and vertical FOV

ROS Noetic, rgbd_pcl.launch, Oak-D-Pro-POE-FF
So I wrote a code for a frostum that represents the FOV of oak-d-pro-ff-poe using the horizontal and vertical angle:
def frostum_vertices(near, far, h_fov, v_fov):
    """
    Calculate vertices of a frustum pyramid in 3D space based on given parameters.

    Parameters:
    near (float): Distance to the near plane of the frustum.
    far (float): Distance to the far plane of the frustum.
    h_fov (float): Horizontal field of view (in degrees).
    v_fov (float): Vertical field of view (in degrees).
    plot (bool, optional): Whether to plot the vertices in a 3D space. Default is False.

    Returns:
    ndarray: Array of shape (8, 3) containing the vertices of the frustum pyramid in local coordinates.
    """
    # convert from degree to rad
    h_fov_rad = np.radians(h_fov)
    v_fov_rad = np.radians(v_fov)

    # Calculate top and bottom near plane dimensions
    x_near = near
    y_near = near *np.tan(h_fov_rad/2)
    z_near = near * np.tan(v_fov_rad / 2)

    x_far = far
    y_far = far *np.tan(h_fov_rad/2)
    z_far = far * np.tan(v_fov_rad / 2)

    # Define the vertices of the frustum pyramid in local coordinates
    vertices = np.array([
        [x_near,  y_near, -z_near],     # Bottom left corner of near plane
        [x_near, -y_near, -z_near],     # Bottom right corner of near plane
        [x_near, -y_near,  z_near],     # Top right corner of near plane
        [x_near,  y_near,  z_near],     # Top left corner of near plane
        [x_far,  y_far, -z_far],        # Bottom left corner of far plane
        [x_far, -y_far, -z_far],        # Bottom right corner of far plane
        [x_far, -y_far,  z_far],        # Top right corner of far plane
        [x_far,  y_far,  z_far],        # Top left corner of far plane
    ])

and
marker = Marker()
    marker.header.frame_id = "oak"  # Replace with your frame_id
    marker.type = Marker.TRIANGLE_LIST
    marker.action = Marker.ADD
    marker.scale.x = 1.0
    marker.scale.y = 1.0
    marker.scale.z = 1.0
    marker.color.r = 0.0
    marker.color.g = 1.0
    marker.color.b = 0.0
    marker.color.a = 0.5  # Adjust transparency as needed

    # Define the triangles for the frustum faces
    faces = [
        # Near plane
        (0, 1, 2),
        (0, 2, 3),
        # Far plane
        (4, 5, 6),
        (4, 6, 7),
        # Side faces
        (0, 1, 5),
        (0, 5, 4),
        (1, 2, 6),
        (1, 6, 5),
        (2, 3, 7),
        (2, 7, 6),
        (3, 0, 4),
        (3, 4, 7),
    ]

    # Add vertices to Marker as triangles
    for face in faces:
        for i in face:
            p = Point()
            p.x, p.y, p.z = vertices[i]
            marker.points.append(p)
    # Publish Marker
    pub_marker.publish(marker)

it's not exactly the same, but I think it's close enough. From what I saw in the specs, RGB fixed focus image has a 69 h fov/ 55 v fov respectively.  However, as seen here:

when I had that marker in rviz, I had to lower it down to 62 and 37.5 just for the it to overlap a little. You notice there are a little bit sticking out of frostum. If I change it to 65/38, nothing would stick out. Is there a reason why the FOV decreases so much?

Hi, I think this is due to aligning process, for the PCL to be aligned properly both image planes need to encompass the same area, stereo image needs to be created from two images that need to be rectified which might result in a smaller FOV for both of them, and then that rectified image needs to be aligned to RGB frame which might introduce additional crop depending on the sensor. If you want to get larger FOV for stereo you can increase alpha scaling parameter accordingly, which should result in larger FOV.

so set  i_enable_alpha_scaling: true? like that? I just see a boolean. Is there another parameter can I increase to increase to FOV or is it just set it to true?

Hi, to set specific value for alpha you can use i_alpha_scaling parameter in stereo, the values are from 0 to 1.0. Overall, you can check what parameters are available and how they are set here in the code, as for their detailed description you can refer to official documentation


up arrow to get command history with kjv> prompt

It would be nice if the up arrow would pull the command history for when the user is prompted with kjv>

Added with ebd36819a5044bfd7c08f535e9ae85a1e318f5b2.


Add Generics to Texture, BaseTexture for Resource types

This adds an opt-in stricter type for Texture and BaseTexture using Generics.
Here are some uses cases:
Better Return Values Using from
// current
const texture = Texture.from('path/to/image.png');
const url = (texture.baseTexture.resource as ImageResource).url;

// new!
const texture = Texture.from<ImageResource>('path/to/image.png');
const url = texture.baseTexture.resource.url;

More Casting Flexibility
// current
const url = (sprite.texture.baseTexture.resource as ImageResource).url;

// new!
const texture = sprite.texture as Texture<ImageResource>;
const url = texture.baseTexture.resource.url;

Stricter Resource Option Types
// current
const base = BaseTexture.from("path/to/image.svg", {
  resourceOptions: { anything: 'goes' } // <-- NO TYPE CHECKING HERE
});

// new!
const base = BaseTexture.from<SVGResource, ISVGResourceOptions>("path/to/image.svg", {
  resourceOptions: { } // <-- ONLY VALID ISVGResourceOptions
});

@GoodBoyDigital I think R would suffice.

I never quite got why the generic props are single letter. Always found it a bit strange as I feel it makes it harder to understand whats going on when looking at code.
But having said that the typescript world tends not to agree with me! I guess the risk is the code ends up getting overly verbose and a bit too wordy!
Potato patato :)

Thanks guys.

@SukantPal I added the docs.
@GoodBoyDigital I renamed T to R for "Resource" and U to RO for "Resource Options"


Bug: Fix dispatch logic for key bindings so that it executes for the correct command key

Similar issues

[X] I have searched and found no existing similar issues

How are you using Studio?
Legend Studio
Current and expected behavior
Currently we support multiple commands in studio for the same key combinations. We have a dispatch logic for the command getting executed when a particular key combination is pressed in application. This logic makes sure that we only run the command once even if we have multiple commands with the same combination. But it doesn't encounter the fact the KeyBindingConfig has entries for multiple commands with the same key combination. In this scenario the dispatch function itself gets called twice.
Inorder to make sure that we run the command once we need to have some logic in dispatch function that checks not just the key combination but the command key too.
Steps to reproduce
Press F9 in studio or embedded query. You notice that compile function gets called twice
Model data
No response
Environment
No response
Possible solution and workaround
No response
Contribution

[X] I would like to work on the fix for this issue

The approach taken in #1970 is very sensible, but unfortunately, we include the command logic in KeyBinding which is lower level utility layer, as such, I took a more simplistic approach in #1973


Suppress unused_unsafe warnings within the generated code.

Sometimes Rust functions declared inside the #[cxx::bridge] have to be marked as unsafe (e.g. if they need to use an explicit lifetime). When the actual, wrapped function is not unsafe, then the generated code will result in an unused_unsafe warning.  For example:
   Compiling cxx-test-suite v0.0.0 (/usr/local/google/home/lukasza/src/github/cxx/tests/ffi)
error: unnecessary `unsafe` block
   --> tests/ffi/lib.rs:276:19
    |
276 |         unsafe fn r_return_str_via_out_param<'a>(shared: &'a Shared, out_param: &mut &'a str);
    |                   ^^^^^^^^^^^^^^^^^^^^^^^^^^ unnecessary `unsafe` block
    |
note: the lint level is defined here
   --> tests/ffi/lib.rs:20:9
    |
20  | #![deny(warnings)]  // Check that expansion of `cxx::bridge` doesn't trigger warnings.
    |         ^^^^^^^^
    = note: `#[deny(unused_unsafe)]` implied by `#[deny(warnings)]`

The warning above comes from the following expansion:
unsafe fn __foo<'a>(x: &'a str, out: &mut &'a [u8]) {
    // The `unsafe` block below is unnecessary if the actual, wrapped
    // function is *not* `unsafe`.
    unsafe { super::foo(x, out) }
}

This commit avoids the warning by including an explicit #[allow(unused_unsafe)] in the generated code.

This PR is motivated by the desire to address the TODO at https://skia-review.googlesource.com/c/skia/+/928799/4/experimental/rust_png/ffi/FFI.rs

Hmmm... when re-reading the commit / PR description, the 2nd part seems out-of-place (since there is no foo in the commit - this expansion comes from a separate, ad-hoc, library crate that I used to understand better what is going on).
Let me do a quick force-push to update the commit descriptions... sorry...

Published in 1.0.135.


Docs don't match repo for install

Describe the bug
The docs at https://istio.io/docs/setup/kubernetes/quick-start/ say "kubectl apply -f install/kubernetes/istio-demo.yaml" and the README in that dir points back to those docs, but there's no such file.
Expected behavior
I expected to find the file that the docs mention.
Version
HEAD for istio

/cc @hklai @linsun

@thockin install/kubernetes/istio-demo.yaml is in release archives https://github.com/istio/istio/releases. The assumption is that the user uncompresses the archive and runs the commands in the root directory of the archive.

Follow the steps in the Prerequisites section to download and prepare for the installation. This should put you inside the Istio package directory where you can kubectl commands.

release-1.1 branch doesn't look to have the istio-demo.yaml file as well... Any recommendations which release to use that will align with the following documentation https://istio.io/docs/setup/kubernetes/install/helm/ ? I ask this because when I tried deploying with tiller, I only see 23 CRDs get created and looking at the crd*yaml files looks to be less than the suggested 53. Im new to istio so I apologies if I am missing something.

As mentioned above, you need to download the release, please don't install from the repo directly.
I am going to close this as there is no further work that needs to be done, this is already documented. if there are any other questions please reach out on https://discuss.istio.io


lwip_close does not work

I copied some snippets from some example code,
It was using lwip_socket() and lwip_close(). I thought that it is the same as socket() and close().
I found out that it would only work a few times, after that, creating a socket fails with -1.
It took me some time to find out, that the lwip_close() is the problem. It does not free the socket, although it compiles fine and does also not return any error.
lwip_socket() and socket() seems to be interchangeable though, or not?
Here is some example code:

static int wifi_cmd_bugcheck(int argc, char** argv)
{
int s,i,ret;
for(i=0;i<10000;i++) {
if ((s = lwip_socket(AF_INET, SOCK_RAW, IP_PROTO_ICMP)) < 0) {
printf("Error %d creating socket after doing so successfully %d times\n",s,i);
return 1;
}
if((ret=lwip_close(s)) < 0) {
printf("Error %d closing socket\n",ret);
}
}
printf("Socket open/close seems ok %d times\n",i);
return 0;
}

Replace lwip_close with close, and it will work.
I can circumvent the problem now, but any kind of warning not to use lwip_close would have been VERY useful!
I'm using the latest esp-idf (pulled today) and toolchain 20180110.

See comments https://github.com/espressif/esp-idf/pull/537

Hi @ManuTester , did the above suggestion help you resolve the issue?


[new game]: ball shoot game

Is your feature request related to a problem? Please describe.
ball shooting game
Describe the solution you'd like
this is an normal game!
using of html,css,js

Closing this as exists #2060

Apologies, I have closed this previously and now it's reopened


Adding implementation of extractor dependencies

Summary
Adds new sortExtractors() function called within MCODEClient which sorts extractors based on a predefined order (defined in MCODEClient.js) and will throw an error detailing any missing dependencies. Since there was some uncertainty over which method of implementing extractor dependencies was best, I would appreciate people's further thoughts after seeing this method in action and I wouldn't be against going back and trying another method if people think this isn't quite what we wanted.
Also, This new functionality makes a previous test fail (in mcodeExtraction.test.js). I didn't want to just remove it because I feel like we could modify the test to work a different way, I just wasn't sure how, so I would appreciate some help with that
New behavior
If extractors are put out of order in the config file (e.g. Patient is put after extractors that require Patient), they will be reordered automatically before running the client and if any extractor's dependencies are missing an error will be thrown.
Code changes

dependencyUtils.js is a new file containing the sortExtractors() function that sorts extractors and checks for missing dependencies
dependencyUtils.test.js contains tests for this new function
MCODEClient.js now contains information about the order and dependencies of extractors and calls the sortExtractors function to sort based on that order and check those dependencies

Testing guidance

Put the patient extractor somewhere other than the start of your config file and see that the client still runs properly
Remove the patient extractor entirely and see that you receive an error telling you that it is missing and which extractors require it
Ensure that all tests pass (see note in summary about the one that doesn't) and check that the new tests cover enough cases

The actual config sorting is working well and the approach looks sound to me. Great work!
In regards to the failing test case, given that the purpose of that test is to test extraction on non-fatal errors my first thought is to actually add the PatientExtractor config object to the example config shown here, as non-fatal errors would still occur down the line in extraction since there's no clinicalSiteID on the CTI config. I also think we should probably add another test that doesn't run extraction, but just tests that an error is thrown when the MCODEClient is initialized with a config that's out of order. Happy to discuss this more if you have other ideas you'd like to kick around!

Re: An additional test, I think that's a good idea!
Re: Fixing the current test, I think Julian's suggestion sounds great. Another approach, if we wanted to make our test a little less fragile, could be explicitly Mocking a response from an extractor get function that just fails. We do something similar at the level of the client in our second test should fail to execute if a fatal error is produced in the extracting data. Up to you which way you want to go.

I think I addressed all comments. I went with @julianxcarter's suggestion for fixing the broken test. Give it another look and let me know if anything else needs changing!

Looks good Dylan – I'll leave the privilege of a merge to you 😄


High concurrency value causes race condition in the commit stage

We observed that any concurrency value greater than 1 causes race conditions on the Kafka side. Let's look into more detail.

Let's assume that the user provides a concurrency value of 4. It means we need to spawn 4 goroutines. As we already know, the go scheduler manages the goroutines so it is not possible to know which goroutine will be executed. Because of it race condition occurs.
For example, if g1 runs before g2 first we commit offset 5-10 and after 10-15. No problem
But if g2 runs before g1, we commit offset 10-15 and after 5-10.
Because of it, although we processed and committed offset, the lag still remains :(.
Note: If you need to quick fix, you can give a concurrency value of 1.
We will address this problem asap.

We can use ReadMessage instead of Fetch and Commit Messages.

storing higher offset in lock-free list can help, instead of committing directly, goroutine can add latest offset to OffsetList and another goroutine can consume from list for commiting.

We thought about this scenario. Let's look;
What if g1 and g3 store their offset in the lock-free list while g2 still executing, The committer goroutine commits the latest offset 20 after the G2 is finished committer goroutine commits the latest offset 15. The lag displayed as 5 :/
So It's like not a solution to this problem

you don't need store offset is lower than already committed.
in your scenario, if g3 already committed you don't need commit or store offset of g1 and g2

Yes, I got it 😌  we can experiment this but multiple topic case are little bit complex 😕
I want to ask another question. What do you think about this re-design?

In kafka-go lib there is a method called ReadMessage In this method, as soon as the messages received, they are committed
func (r *Reader) ReadMessage(ctx context.Context) (Message, error) {
	m, err := r.FetchMessage(ctx)
	if err != nil {
		return Message{}, err
	}

	if r.useConsumerGroup() {
		if err := r.CommitMessages(ctx, m); err != nil {
			return Message{}, err
		}
	}

	return m, nil
}

In g0 fetch and commit messages and send it to channel, other goroutines listen to that channel and invokes batchConsumeFn
In this scenario, we need to guide the user about graceful shutdown because although we are not processing message, we committed
We tried this architecture is very performant! here

commitring offset before processing messages can be little problematic because we can loose messages if application shutdown immediately or any unexpected things happen in pod or application.
graceful shutdown is must here also application should reproduce committed but not processed messages to kafka, but what happen if application cannot procude these messages to kafka? I think handling this failure scenario is more complex than managing thread-safe list per topic for managing/tracking committed/uncommitted offsets. 🤔

Thank you for your valuable comments, these concerns are really crucial!
We will experiment and test your recommended implementation ASAP 😅

Based on my experiments, managing a thread-safe list is good but when rebalancing happens we need to delete and create this list so kafka-go does not present this kind of eventing 😕 This is very challenging 🤔 @oguzhaneren
https://github.com/Trendyol/kafka-konsumer/pull/33/files#diff-3d7d7694e24a977b9c4e30d03880ff4929efaeb873e7850edc4c110388b40da0R116

I don't know api of the segmentio but I took a look, there is a variable for assigned topic/partitions
https://github.com/segmentio/kafka-go/blob/main/consumergroup.go#L321
maybe we can ensure committing topic/partition existsince by looking this map in committing goroutine, if is not in the list we can simply ignore the offset because this partition rebalanced to another client. what do you think?

@mhmtszr


CI: Switch centos:8 to centos:stream8 image

cherry-picked from https://github.com/voxpupuli/beaker-docker/pull/58


Syntax highlighting doesn't work with data- attributes

It'd be nice if the syntax highlighting worked with attributes prefixed with data-.

Also it doesn't highlight every needed directive e.g ng-required, ng-class or ng-show. I'd really appreciate it if you could add support for these.

Hi, thanks for resolving the issue, but after the update all the directives' content lost JS specific highlighting - the whole code is of one color as below:

Also somehow ng-required is displayed in two colors. I'd appreciate if you could look into it.

Thank you for reporting. I'd appreciate if you reopen the issue next time you get the issue again.
I could miss the issue if it is closed.

I exactly wanted to do so, but couldn't find the Reopen option. It looks like I don't have a permission to that unless I close the issue myself.

Oops! I didn't know that. Sorry about that. :)

No problem :)


Upgrade rust sdk to latest version

The latest rust sdk version includes refactored math for calcImpliedRate to return an APR instead of the ROI. It also includes conversion methods to go from ROI <-> APR easily. We'll use these when displaying the Yield stats in a market.

Closing since this is marked as Done


Update transformation.py

added the GeoPoint parameter precision so that when the geometry object is created it will have greater precision than the default which is 6. (https://github.com/jazzband/geojson/blob/master/geojson/geometry.py#L7)
here's where you can find the parameter.
https://github.com/jazzband/geojson/blob/master/geojson/geometry.py#L15

I think the more appropriate way is to probably set a default parameter from the transformation method to 6. this could be if someone has for whatever reason hard coded the precision of their lat/lng by the [:-1] value of a coordinate.  it would probably be pretty rare, but i suppose it does leave some opportunity for it to cause a bit of a breaking error.


Dragging a child node no longer affects parent

fix for #162 plus some housekeeping
a little housekeeping turned a // TODO into a // TODID. I probs shouldve branched separately but i guess i felt lazy

is called for every op on every frame
OH

yes your solution is much better. I was trying to keep GraphCanvas's grubby little hands out of the selection handling where possible, but knowing that changes things


configlet sync for exercise docs

@exercism/kotlin, I've updated this PR to the latest docs.

@BNAndras there is a conflict! Ping me after so I can merge ;)

@SleeplessByte, I rebased the PR.


hgtector search

Hi
First thanks for your Hgtector!
I meet error when use 'hgtector search',
Command:
conda create -n hgtector python=3 pyyaml pandas matplotlib scikit-learn
conda activate hgtector
pip install git+https://github.com/DittmarLab/HGTector.git
hgtector search -o ll -i o55h7.faa.gz -d diamond/db.dmnd -t taxdump/ -m diamond
Error:
(hgtector) []$ hgtector search -o ll -i o55h7.faa.gz -d diamond/db.dmnd -t taxdump/ -m diamond
Homology search started at 2020-05-22 08:10:24.825237.
Traceback (most recent call last):
File "/home/usr/envs/hgtector/bin/hgtector", line 96, in
main()
File "/home/usr/envs/hgtector/bin/hgtector", line 35, in main
module(args)
File "/home/usr/envs/hgtector/lib/python3.8/site-packages/hgtector/search.py", line 131, in call
self.args_wf(args)
File "/home/usr/envs/hgtector/lib/python3.8/site-packages/hgtector/search.py", line 330, in args_wf
if not isdir(dir_):
File "/home/usr/envs/hgtector/lib/python3.8/genericpath.py", line 42, in isdir
st = os.stat(s)
TypeError: stat: path should be string, bytes, os.PathLike or integer, not NoneType

Hello @xu-j1234566 Thanks for your test and sorry for the late response. I have fixed this bug. If you update the program and run this command again, it should work now.


Updating projects to 0.9.1909.3002

Yes it should, good catch. The update script was not taking this into consideration, I've added it.


Character Sheet for Svärd & Svartkonst

Changes / Comments
This is a new Character Sheet for the Swedish OSR game Svärd & Svartkonst.
Roll20 Requests
Comments are very helpful for reviewing the code changes. Please answer the relevant questions below in your comment.

[x] Does the pull request title have the sheet name(s)? Include each sheet name.
[x] Is this a bug fix?
[x] Does this add functional enhancements (new features or extending existing features) ?
[x] Does this add or change functional aesthetics (such as layout or color scheme) ?
[x] If changing or removing attributes, what steps have you taken, if any, to preserve player data ?
[x] If this is a new sheet, did you follow Building Character Sheets standards ?

If you do not know English. Please leave a comment in your native language.

Character Sheet Info Roll20 Internal Use only.

The validation fails for all new sheet submisssions, really only works for PRs updating existing sheets.
Everything looks good on a quick skim.


Fix/incremental column precision changes

resolves #5351

Description

Consider whether column may be successfully expanded when qualifying schema changes
Checklist

[x] I have read the contributing guide and understand what's expected of me
[x] I have signed the CLA
[x] I have run this code in development and it appears to resolve the stated issue
[x] This PR includes tests, or tests are not required/relevant for this PR
[x] I have opened an issue to add/update docs, or docs changes are not required/relevant for this PR
[x] I have run changie new to create a changelog entry

@leahwicz Sorry for the delay in responding, I'm just getting caught back up after two weeks of PTO. Let me confirm the test fails as @ChenyuLInx asked and then this is ready to merge

Paired with @epapineau today and adjusted the test to make sure it will fail on snowflake before applying the fix


Use direct mode instead of rounder callback in win32drv.

For @FASTSHIFT's suggestion, win32drv started to use direct mode instead of rounder callback.
Also, win32drv can reduce one memory copy for each frame when Windows supports the color depth your set in LVGL.
P.S. When I use the profiler, Windows GDI operation uses 80% of the main thread CPU time. I will try to create a new PR to offload them to a separate thread.
Kenji Mouri

@kisvegabor ping for progress
Kenji Mouri


Community picture and logo

Logo and community picture (copy paste link to teams or similar should show a picture and not just a github preview) are missing.
Doesn't need to be fancy. There are nice generators for simple logos
@xxthunder @cuinixam

Would make a lot nicer previews in linkedin and everywhere else

@code-steadfast maybe you see this? XD
I would do it myself, but looks like i got removed from maintainer list of this repo...

Nice Idea, we will look into it :)

I somehow like the second one more. The first one looks like home automation 😁

I am not sure if any of my proposals is really good. They were meant as samples and were auto generated.  We would need something which depicts the modularity a bit. That's why the one with the house was not soooo bad at all... if you remove the house and stick with layers only... it is not a layered architecture but gives a new abstraction layer for multiple components.

its just a pic :)


discord embed links not clickable

Hi, thanks for resolving the audio and image preview bugs!
I was wondering if there's a way to make the embed links within ripcord clickable, if you could direct how could that be done then I could try it myself

Sorry, not planned as I have no idea how you'd even begin trying to get Ripcord to parse embed contents for links.


Blazium faster find children Take 2

This uses LocalVector instead of TypedArray. Was 2X faster than https://github.com/blazium-engine/blazium/pull/59 on my machine. Using this test script: https://github.com/blazium-engine/blazium/pull/59#issuecomment-2422265577

Nuking this PR to start over


update Lexer to use Errors and add some support for C23 stuffs

On progress

[x] Add to support others prefixes for chars/strings like u and u8
[ ] (Maybe) Add to support ' (sigle quote) to separate integers digits
[x] Add extensive tests about new lexer methods

[x] Add preprocessors tokens tests
[x] Add float (hexadecimal, etc.) tests

[x] Remove or replace cast calls to # type: ignore
[x] Remove assert statements

Errors instead of Exceptions
I removed all exceptions from Lexer (I'm yet adding tests to make it no error-prone) and added Errors instead, it will show more user-friendly error messages to users (and highlights soon). We don't need to stop the norminette when a float is correct in syntax but not in semantics (I added 8 errors about floats/integers).
Alternative lexemes
Norminette will do some translations like <% and ??< to LBRACE ({). Note that strings with digraphs and trigraphs will be translated as well like #486 did in comments.
Escaped newline
This PRs is removing the <ESCAPED_NEWLINE> token, it means that codes using it will be buggy, like CheckLineCount (I pretend to fix it with others rules in a separated PR).
Chars as Strings, Escape sequences
If the user writes a string with ' (single quote) like 'hello', it will see an error noticing it. Another cool thing is about writing bad escape sequences like \m, \xGG, etc. that will shows an error.
Suffix in numeric literals
Added i64 (Microsoft-specific), wb (_BigInt(N)) and z (size_t) suffixes for integer literals and for float literals: d* (_DecimalN) and i with j for complex literals (_Complex).
Others

Moved tests related to lexer to test_lexer.py
Removed the TokenError exception completly

Hello, I didn't review because of the 6/7 validated but is it over?

Hello, I didn't review because of the 6/7 validated but is it over?

Yes, it is


Vertical Line

Graph vertical lines x = 2 ... right now this entry gives incorrect results.  Catch "x = "  on input and do not allow it.... only allow x = number and plot a vertical line.

Ref: http://jsxgraph.uni-bayreuth.de/wiki/index.php/Change_Equation_of_a_Graph#JavaScript_Part

//glider on the curve
p1 = board.create('glider', [0,0,graph], {style:6, name:'P'});
//define the derivative of f
g = JXG.Math.Numerics.D(f);
//a point on the tangent
//                                 variable x coordinate           variable y coordinate depending on the derivative of f at point p1.X()
p2 = board.create('point', [function() { return p1.X()+1;}, function() {return p1.Y()+JXG.Math.Numerics.D(graph.Y)(p1.X());}], {style:1, name:''});
//the tangent
l1 = board.create('line', [p1,p2],{});

The highlighted ones are key, just get two points from the 'x=' and create a line.
NOTE: I will be going to work soon, so this is more of a self reminder than a request/command

No... you can graph a vertical line using the line funtion in JSXGraph...
using parameters for ax+by+cz=0... I just need to find the documentation.
On Wed, Jun 17, 2015 at 1:30 PM, Paul Gonzalez-Becerra <
<EMAIL_ADDRESS>wrote:

Ref:
http://jsxgraph.uni-bayreuth.de/wiki/index.php/Change_Equation_of_a_Graph#JavaScript_Part
//glider on the curve
p1 = board.create('glider', [0,0,graph], {style:6, name:'P'});
//define the derivative of f
g = JXG.Math.Numerics.D(f);
//a point on the tangent
// variable x coordinate variable y coordinate depending on the derivative
of f at point p1.X()
p2 = board.create('point', [function() { return p1.X()+1;}, function()
{return p1.Y()+JXG.Math.Numerics.D(graph.Y)(p1.X());}], {style:1, name:''});
//the tangent
l1 = board.create('line', [p1,p2],{});
The highlighted ones are key, just get two points from the 'x=' and create
a line.
NOTE: I will be going to work soon, so this is more of a self reminder
than a request/command
—
Reply to this email directly or view it on GitHub
https://github.com/johnRedden/OpenGraph/issues/25#issuecomment-112940493
.

Vertical line implemented here [z, a, b] for the parameters... x = 3 corresponds to [-3, 1, 0]  seems strange but that is how.
http://jsxgraph.uni-bayreuth.de/wiki/index.php/Aligning_text


Typo: remove unconsistent period in docker info

Signed-off-by: Lei Jitang<EMAIL_ADDRESS>Feel free to close if it's not necessary.:)

LGTM :-)
I'll take the liberty to merge that one without waiting for a second review :trollface:

Oooh! daring! LOL
thanks @coolljt0725 !


WASM: Prevent folding cmp instr

If we run out of registers during regalloc we'll try to fold the instruction and sometimes swap the operands.
In the case of cmp we swap the operands if we're able to reverse the logic of the next branch.
However, it is possible that another instr will want to use the flags set by cmp other than the branch.
The exact case I am seeing is this on x86
; Globopt
s1.i64 = Conv_Prim -2.u32
... ; some other operations
s2.i64 = Ld_I4 0xC5C6358EC332B00A.i64
s3.i32 = CmUnGe_I4 s1.i64, s2.i64
...

; after regalloc
s4.i32 = MOV 0
s4<-10>.i32 = MOV s4.i32 //spill
s5.i32 = MOV -2
... ; some other operations
s6.i32 = MOV 0xC5C6358E.i32
s7.i32 = MOV 0xC332B00A.i32

s10.i32 = MOV 0 ; comparison result
                ; Compare high bits of the i64
                CMP s6.i32, s4<-10> ; BUG: opnd swapped to avoid read spilled reg
                JNE $L1
                CMP s5.i32, s7.i32 ; compare low bits of the i64
$L1:
s10.i32 = SETAE s10.i32 ; Use the flags set by cmp to the comparison result

The clean solution would be to track which instr uses/set the register flags to know if we're allowed to do such an optimization. However, this seems to be a lot of overhead so I opted for the quick fix.
I was also thinking of using the unused dst of the cmp instr to track which instr uses the flags it sets.
What do you guys think ?
I wasn't able to write a small enough test, I couldn't find a way to use all the registers and force trying to fold.

This change is

@dotnet-bot test Ubuntu shared_ubuntu_linux_test

ping, this issue is hitting a lot

:shipit:


Add a parallel implementation

Added a parallel implementation so everything is faster. It's based on rayon, so it will default to the same number of parallel task as your number of CPU.
I also added a progress bar for clone and fetch

Thanks for the contribution. This is really useful. I have added some comments for some minor changes. Do let me know your thoughts.

Tagging @dineshba

@tinou98 there seems to be another issue with the clone command. Previously if we try to clone the same repo, it would fail saying the directory already exists which is expected. Now it just stalls. Can you check what is happening?
Run gg clone -r git@github.com:thecasualcoder/gg.git -l some_dir twice.
Expect second time to fail ->
Cloning remotes passed as arguments
No remotes configured in conf file
thread 'main' panicked at 'Failed to clone repo git@github.com:thecasualcoder/gg.git, : Error { code: -4, klass: 3, message: "\'trysf/gg\' exists and is not an empty directory" }', src/clone.rs:95:9
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

(error is not currently handled properly. But it should still fail)
On your fork it doesnt fail. It just gets stuck.

Also can we make parallelism an option. If enabled use based on number of CPU's or user input count. If disabled fall back to single threaded execution

Fixed blocked issue, added control over the number of thread, setting to 1 go back to the monothreaded way. There is still an issue in monothreaded mode (-j1) that add latency before the status line is shown, I'm working on it

It's Good. You can continue to work on it. Ill merge these now.


Update index.js

Fixed issues:

non-ENOENT file system error caused by path length too long
Your usage of a global options object would cause cross-contamination between different calls to minify (e.g. using an option in the first call, but failing to specify this option in the second call, would result in the 2nd call using the 1st call's value for this option, not the default)

Performance improvements

No call to statSync is ever made if the string is simply too long to possibly be a path.
Semi-intelligent adaption of MAX_PATH_LENGTH if we get errors at path string sizes than initially predicted.

New Features:

Added a minifySync method to help reduce asynchronous spaghetti code in projects. Yes, yes, I know that this problem is supposedly solved by async/await, but that is not the case at all: there are too many libraries and too many built-in methods that are simply incompatible and require their own special work-arounds.

Missing(?)/TODO(??) Features:

Removed the do-nothing option to minify the HTML

Please do not forget to push up these changes to NPM as well.

Thank you isFlavor. I am glad that I could improve your project.
~ Happy Coding!


Rowanize the demos ...

GemTalk/Rowan#553 and GemTalk/Rowan#555. I've created this PR primarily to discuss what I've done to Rowanize this repo. Besides adding the rowan directory with the Rowan meta data, I've moved the the example packages around a bit and created some packages that include classes that can be loaded into GemStone ... GemStone base does (currently) support SharedPools (SharedPoools are supported in GLASS) ... I also suspect that I've broken the VAST package meta data ...
Anyway, this PR is against the master branch, but perhaps it should be shared on a rowan or gemstone branch until we work out all of the kinks ...

Hold off merging this PR until I merge my latest code into the vast branch of Rowan ... I'm renaming an instance variable in the component from conditionalPackages to conditionalProperties as part of GemTalk/Rowan#554 ... and my life will be a bit simpler if you hold off merging until I've converted these components and merged the code sans  conditionalPackages into the vast branch ...

It took a little longer for me to finish up my work refactoring the component structure (componentNames and projectNames moved into conditional sections), but with this commit on the Rowan vast branch, I'm finished. I've also updated the rowan_553 branch (cb69685) with examples of the componetNames (emtpy lists) in the conditional structures) ... so you can merge at your leisier


math content does not recognize align

Hi,
This repo is extremely useful for me. Everything works fine in most math environments, but not for the align (and align*) environment. I did some google and found that "AmsA" seems to represent align. But it is already in the file...
Thank you!
Best

This should be fine with the latest version of vimtex and #14

If you want to keep using the built-in TeX syntax highlighting, then you need amsmath.vim from Charles E. Campbell that adds highlighting for the align environments by
  call TexNewMathZone("E","align",1)
  call TexNewMathZone("F","alignat",1)
  call TexNewMathZone("G","equation",1)
  call TexNewMathZone("H","flalign",1)
  call TexNewMathZone("I","gather",1)
  call TexNewMathZone("J","multline",1)
  call TexNewMathZone("K","xalignat",1)
  call TexNewMathZone("L","xxalignat",0)

from http://www.drchip.org/astronaut/vim/vbafiles/amsmath.vba.gz

This should be fixed now. Update vimtex and use the following code:
https://github.com/gillescastel/latex-snippets/blob/baa1b42ed9558b4a7ba220c50c5046957719f898/tex.snippets#L1-L12

This should be fixed now. Update vimtex and use the following code:
https://github.com/gillescastel/latex-snippets/blob/baa1b42ed9558b4a7ba220c50c5046957719f898/tex.snippets#L1-L12


v0.11.4 JST not written issue

I defined an html file under assets/templates/ to compile for JST use.  I get the following error when doing "sails lift --verbose":
verbose: Grunt :: Cleaning .tmp/public...
verbose: Grunt :: OK
verbose: Grunt :: Running "jst:dev" (jst) task
**>> Destination not written because compiled files were empty.**

What am I required to do other than defining HTML files under assets/templates/?

Hi @NizarBlond, can you provide some more information?  From what I can tell, this is a question rather than a bug report, but I want to be sure.  See our contribution guide for info about the information to submit when creating an issue.

@NizarBlond by the way, re: your question:

What am I required to do other than defining HTML files under assets/templates/?

I went and did some cleanup work in our default-generated task config files recently-- here's a link in case it's helpful: https://github.com/balderdashy/sails-generate-frontend/blob/master/templates/tasks/config/jst.js#L6

Another tip for you, in case this gets you on the right track:  Once you've gotten one or more HTML files in your client-side templates folder  (e.g. assets/templates/foo.html) and put some content therein (e.g.
hi
), then try running the following in the browser:
console.log(window.JST);

That's what I did, there is no jst.js file generated.
Also, I obviously put the /foo.html under assets/templates/ with content. NOTHING.
Am I missing something in the instructions on how to enable this feature? I don't guess so, it should be enabled by default.
I started a brand new project to test it from scratch and NOTHING.
Can you please chec the feature?

I guess I found the bug
I did the following change in tasks/config/jst.js and BOOM, it works!
			files: { 				// e.g. 				// 'relative/path/from/gruntfile/to/compiled/template/destination'  : ['relative/path/to/sourcefiles/**/*.html'] 				// '.tmp/public/jst.js': require('../pipeline').templateFilesToInject, 				'.tmp/public/jst.js': ['assets/templates/**/*.html'] 			}
Then, in tasks/pipeline.js we have
`var templateFilesToInject = [
'templates/**/*.html'
];
module.exports.templateFilesToInject = templateFilesToInject.map(transformPath);
function transformPath(path) {
return (path.substring(0,1) == '!') ? ('!' + tmpPath + path.substring(1)) : (tmpPath + path);
}`
So, I printed the "require('../pipeline').templateFilesToInject" inside .jst
console.log("blabla: ", require('../pipeline').templateFilesToInject);
and I get the BUG HERE
blabla:  [ '.tmp/public/assets/templates/foo.html' ]

The path explains it all, why do we even have .tmp/public here?
Also, you may need to add assets/ as the default dir doesn't seem to be this (assets)
Cheers

Alright, thanks @NizarBlond!  This should be resolved if you do a fresh install of sails-- just published a patch release of sails-generate-frontend (more details here)

Could this be related to https://github.com/balderdashy/sails/issues/3367 as well?

@sgress454 perhaps, not entirely sure (replied in #3367)
@NizarBlond I'm going to go ahead and close this issue since it's been resolved.  Please open a new issue (with a link back here) if you continue to see problems.  Thanks!

i have a problem with sails.js and gruntjs ?!..
whzat the henk.? really, nothing works fine.. for everything ins sails you have to write shit ... sy, nerving a bit.. these tousend of errors.! eee??? its a a framework.? !!... what can i do with this bull shit.?

Running "jst:dev" (jst) task
>> Destination not written because compiled files were empty.

okey, fixed :) thanks to Mike description  ! need your Brain!


WIP: Generate pinned environment file, filtered by specified dependencies.

Work in progress sketch, but basically works. Allows to create pinned environment file, filtered by one or more groups of dependencies as specified in setup.py.

We'll see how it goes and clean it up later...


State_Space_Wrapper Function

Dear Prof. Maih,
I am building my DSGE model based on your RISE Toolbox, and facing an error, I guess that (1) I am making a mistake when deriving several parts from the internal function: [T,R,Z,H,Q,sstate,init,growth] = dsge.state_space_wrapper(syst); or (2) It cannot find the solution from solve(). I don't know how I could find the 'syst' part and the forms of these outputs. Can you please let me know the forms of these outputs? And In what case will I have no solution (is it from the parameters' initialization)?.
Thank you so much for your help!
Khoi

Hi Khoi,
What you are describing is not a natural use of the toolbox and it is not
surprising that you run into problems.
What are you exactly trying to do?
Cheers,
J.
On Fri, Dec 29, 2023, 06:12 khoilx @.***> wrote:

Dear Prof. Maih,
I am building my model based on your RISE Toolbox, and facing an error, I
guess that I am making a mistake when deriving several parts from the
internal function: [T,R,Z,H,Q,sstate,init,growth] =
dsge.state_space_wrapper(syst). I don't know how I could find the 'syst'
part and the forms of these outputs. Can you please let me know the forms
of these outputs?
Thank you so much for your help!
Khoi
—
Reply to this email directly, view it on GitHub
https://github.com/jmaih/RISE_toolbox/issues/184, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AATKBTYPMBGFQGLKYGT75Y3YLZGMTAVCNFSM6AAAAABBGHASOWVHI2DSMVQWIX3LMV43ASLTON2WKOZSGA2TSMBXGM3TSNY
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Hi Prof. Maih,
I'm trying to estimate the endogenous RS-DSGE model with 2 latent factors using my own filter. To do that, I extracted several matrices from the 'syst' part. When I tried 'filter' comma, it shows 'retcode = 306', and moreover, when I tried 'solve' comma to solve the rise model obj, I saw 'obj.solutions.H' is empty. So, I am not sure this problem is from no solution of the model or from the mistake of calling matrices from the 'syst'.

Several issues to unpack:
1- what you need, it seems, is to connect your own filtering procedure to
RISE. RISE expects your procedure to accept certain inputs and return
certain outputs. If you make  mistake at that step, the procedure will fail.
2- If you want to understand what the error message means, run
decipher(306) or decipher(retcode)
3- the default filter will do the job of filtering and can be used for
estimating parameters but will not be able to generate the latent factors.
Should you go down this route, the only remaining question will be: do you
know how to extract latent factors and what inputs do you need in that
process.
On Fri, Dec 29, 2023, 08:24 khoilx @.***> wrote:

Hi Prof. Maih,
I'm trying to estimate the endogenous RS-DSGE model with 2 latent factors
using my own filter. To do that, I extracted several matrices from the
'syst' part. When I tried 'filter' comma, it shows 'retcode = 306', and
moreover, when I tried 'solve' comma to solve the rise model obj, I saw
'obj.solutions.H' is empty. So, I am not sure this problem is from no
solution of the model or from the mistake of calling matrices from the
'syst'.
—
Reply to this email directly, view it on GitHub
https://github.com/jmaih/RISE_toolbox/issues/184#issuecomment-1871806290,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AATKBT3IECO44PMMAQKVE2LYLZV2DAVCNFSM6AAAAABBGHASOWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNZRHAYDMMRZGA
.
You are receiving this because you commented.Message ID:
@.***>

Thank you so much. I'll check it carefully.

Hi Prof. Maih,
As your suggestion, I've checked the error message, it is 'unlikely parameter vector', I think I need to modify the setting of parameter vector. Thank you so much for your help. Happy New Year!
Best wishes,
Khoi

Hi Khoi,
Unlikely parameter vector can come from different sources. Certainly a bad
parameter vector could be the culprit.
But you can also have that problem, among other things, if the initial
variance is too small and or the initial state is far away from what is
implied by the data. Should this be the case, sometimes a diffuse
initialization with a large variance may help.
Happy new year to you too!
J.
On Sat, Dec 30, 2023, 13:43 khoilx @.***> wrote:

Hi Prof. Maih,
As your suggestion, I've checked the error message, it is 'unlikely
parameter vector', I think I need to modify the setting of parameter
vector. Thank you so much for your help. Happy New Year!
Best wishes,
Khoi
—
Reply to this email directly, view it on GitHub
https://github.com/jmaih/RISE_toolbox/issues/184#issuecomment-1872521143,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AATKBT4OGDYEZRM2RKPY3PLYMAD7RAVCNFSM6AAAAABBGHASOWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNZSGUZDCMJUGM
.
You are receiving this because you commented.Message ID:
@.***>


Build using Cargo

I added a new issue for this seems it seems to be a different cause.
ST 3, Rust plugin, tried building using Cargo and I got this:
[Errno 2] No such file or directory: 'cargo'
[cmd: ['cargo', 'build']]
[dir: /Users/dutt/workspace/rust/gridwar/src/entities]
[path: /usr/bin:/bin:/usr/sbin:/sbin]
[Finished]

Which is understandable, since I get this:
dutt$ which cargo
/usr/local/bin/cargo

How do I edit the path used by the plugin?

Thanks @hughes ! I ran into the same problem on my Mac and that worked perfectly.

Using rustup.rs I added
"env": {
    "PATH": "$HOME/.cargo/bin:$PATH"
},

Since moving to rustup this has stopped working for me.
Can we add the Home path by default?

This has been fixed in https://github.com/rust-lang/sublime-rust/pull/97


fix(chip): Icon alignment fix

Short description of what this resolves:
UI fixes for <ion-chip> component (icon in chip alignment).
Ionic Version: 2.x
Fixes: #5386

Thanks for the PR! I'm not able to see any issues with the icon alignment. Please see my comment here: https://github.com/driftyco/ionic/issues/5386#issuecomment-256179231

Thanks! I made a small change to use a variable in case people want to override this but I committed it here: https://github.com/driftyco/ionic/commit/ace727e627b56793464d7bdd77ffd5546de49220


boards/pico: Remove mention of boot select button

The boot selector pin is attached to SPI_SS and not easily read (an
example
https://github.com/raspberrypi/pico-examples/blob/master/picoboard/button/button.c),
since the code reading needs to reside in SRAM.
Simply fix the documentation by only mentioning the pin

Good change, thanks for the PR!


Update dependency yamllint to v1.26.3

This PR contains the following updates:

Package
Type
Update
Change

yamllint
extra
patch
==1.26.1 -> ==1.26.3

Release Notes

adrienverge/yamllint
v1.26.3
Compare Source

Restore runtime dependency setuptools for Python < 3.8

v1.26.2
Compare Source

Fix python_requires to comply with PEP 345 and PEP 440

Configuration
📅 Schedule: At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, click this checkbox.

This PR has been generated by Renovate Bot.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
848

Relevant Lines:
892

💛  - Coveralls


PDF Viewer allows the JS embedded in the PDF to be executed

OpenUI5 version: 1.120.3
Browser/version (+device/version): Chrome 120.0.6099.130 (windows 11)
Any other tested browsers/devices(OK/FAIL): No
URL (minimal example if possible): N/A
Steps to reproduce the problem:

Preview a PDF with embedded JS

What is the expected result?
The embedded JS is not executed, or the preview of the PDF is prevented, to avoid Stored XSS via PDF Injection
What happens instead?
The embedded JS is executed while previewing the PDF
Any other information? (attach screenshot if possible)

Unfortunately, security issues cannot be processed here according to https://github.com/SAP/openui5/blob/master/CONTRIBUTING.md#reporting-security-issues
Please follow the linked guideline.

If the source of the PDF file cannot be trusted, keep the PDFViewer property isTrustedSource at false. The user has then the option to explicitly download the PDF file from the toolbar.


[ScaNN]: Add option to add new dataset items

Hi,
I have a large dataset with image (say ~2 million) embeddings.
Can I add another say 100 embeddings to the scanner or I need always a new Initialization?

Same question here..

Same question here, too.

Same question here, too.

my question as well.

@DavidWiesner , @XiaoConstantine @YJYJLee and @Soroosh-aval did you manage to add additional embedding to the scanner without re-initialization ?

no it can't add any new embeddings into the index,  you have to rebuild the index using the new data every time in one-shot


Bug: Storing Response reference instead of new instance in system.checkAll and system.checkOne methods

When I use system.checkAll / system.checkOne, the response instance is passed down the tree. So when I want to store a list of responses for later use, it's just a reference to the same response, which is incorrect.
Code example:
let result = [];
this.system.checkOne(body,(response) => {
    result.push(response) // save for later use
});
console.log(result) // <-- all responses are just a reference to the same instance (system.response)

I have tried different approaches. But still have similar issues:
let result = [];
this.system.checkOne(body,({...response}) => {
    result.push(response)
});
console.log(result) // <-- different instance, but still have a reference to the same overlapV, overlapN object, etc.

let result = [];
this.system.checkOne(body,(response) => {
    result.push(response)
},
new Response() // insert new instance
);
console.log(result) // <-- different instance, but still have a reference to the same overlapV, overlapN object, etc.

I think we should create a new instance for each checkCollision/checkAll/checkOne call to prevent this issue.
This issue will cause a bug when separating a circle into multiple polygons at once frame. Because the overlapV is not correct.

I suggest adding a new function to the library that clones a Response object. It would be great if the function could support cloning objects like this:
function cloneResponse(response: Response) {
  return {
    ...response,
    overlapV: response.overlapV.clone(),
    overlapN: response.overlapN.clone(),
    // ... any other object
  };
}

I hope you can fix this. It took me 6 hours to debug and figure out why my collision separation is incorrect. 😒😢🥲

I just ran into the same problem. I assumed out of habit that responses were simply immutable data objects.
Multiple methods receive an object of type Response, which by default uses a shared one, a member of a System instance which is cleared and filled on each check. A similar design decision has been made with overlaps, they are created and shared in multiple iterations within check methods. I am completely unaware of what led to this design decision, for no apparent benefit. It complicates persistence and state management in a devastating way.
I'd love to say that this is the only design issue you'll see, but sadly it isn't. Not only coming from detect-collisions, but also from its dependencies like SAT and RBush. It certainly discourages contributing with so many things to fix, sometimes being difficult to differentiate what we consider to be objectively correct from what is opinionated.
After so much negative talk, I think it's worth emphasizing that this is one of the best and simplest libraries focused solely on 2D detection. It's a good starting point for developing something more polished.

nice!
thank you @HoangTran0410 for opening an issue and @alexvgjm for confirming
I'm planning to look into this and fix it this weekend

well, this is actually more like a feature request than a bug but I did what you asked for:
please see https://github.com/Prozi/detect-collisions/releases/tag/v9.2.2
🙂


add -Wno-GENUNNAMED to Makefile

fix make "Unnamed generate block (IEEE 1800-2017 27.6)" warning by using the latest verilator

Thanks, merged!


Clarify whether single-axis indices can be tuples i.e. x[(i,)]

Currently numpy.array_api supports this behaviour. For example:
>>> x = xp.zeros(5)
>>> x[(0,)]
Array(0)

Also notably @asmeurer tests for (i,) indices in the compliance suite's multiaxis tests.
I can't work out whether the indexing rules specify this is allowed or not (i.e. not explicitly mentioned in either the single or multi-axis sections), but I would imagine this is expected. Before this was mentioned by @Zac-HD in HypothesisWorks/hypothesis#3065, I had just assumed that multi meant "one-or-more" and so this is allowed.
Let me know if I'm wrong and you can guarantee this with the current spec, although that raises the question if we could still purposely note this.
Clarification either way would be nice, but if this isn't actually agreed upon then I'd just add that allowing (i,) indices would make index tooling (like in Hypothesis) much simpler.

I understand that as a multi-axis index that selects 1 axis. Maybe the multi-axis indexing section could say something like "a multi-axis index indexes between 0 and N indices (inclusive)".

The following note in the spec implies that single-element tuples are the equivalent of single-axis indices:

In Python, x[(exp1, exp2, ..., expN)] is equivalent to x[exp1, exp2, ..., expN]; the latter is syntactic sugar for the former.

However, I've submitted gh-280 to provide some additional guidance.


Bump ansible in builder-base to 2.11.12

Issue #, if available:
Description of changes:
In an attempt to fix a flaky build on the eks-a side, which I believe is the only place we use anisble, bumping to 2.11.12 which includes this commit ~which should allow us to increase the timeout~.  There was a later commit to ansible that allowed the timeout to configurable but that was not backported to 2.11. Bumping to 2.11.12 should still help us since the above commit they bump the default timeout to 60 seconds instead of 20.
The version of image-builder we are using from upstream actually uses the 2.11.x branch anyway so this change shouldnt cause any issues there.
Image-builder was also using ansible-core instead of ansible, I do not believe there will be a noticeable difference in that change for us.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/approve


Using retention labels to declare records

In the section "Using retention labels to declare records" it states:
"Using a retention label that declares content as a record requires an Office 365 Enterprise E5 license for each user who has permissions to edit the content. Users who simply have read-only access don't require an E5 license."
Is this a mistake? I thought records retention was available in E3?

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

ID: 6ae5f811-9551-ea95-b113-9175a6f08d7e
Version Independent ID: e1eb8d3d-d439-e814-4a05-aa5bbc11f33e
Content: Overview of records
Content Source: microsoft-365/compliance/records.md
Service: o365-seccomp
GitHub Login: @LauraWi
Microsoft Alias: laurawi

@Soulsailor  - Thank you for submitting feedback.
I will get this issue over to the Microsoft 365 writing team for investigation.
Thank you for reporting and making the docs better. Much appreciated.
I made a note to request the team to update this when the work is complete.

Hi @soulsailor I think the nuance here is between what's supported for data governance (manual labels) and advanced data governance (i.e., policy-driven retention): https://docs.microsoft.com/en-us/office365/servicedescriptions/office-365-platform-service-description/office-365-securitycompliance-center

@LauraWi Thanks for your response. So I think what you are saying is as follows:
Auto Applying a Records Retention label requires E5, but an en-user manually adding a Records Retention label to content is still E3?
If that is the case then, yes it is semantics, but I think the note box should mention the "auto apply" aspect for clarity.

Yes, that's it. Thanks for the feedback on clarity. We'll be revising RM content.


Shall we show Windows web apps in the TreeView?

Currently the app service view show both Windows & Linux web apps and there is no UI indication of the type of the app.

We could use different icons to differentiate Windows and Linux web apps. Similar to what the portal does for App Service Plans.

i think we still need to limit the commands available for windows app services (for now at least)

Done


Develop a Contact Page form with confirmation message

Describe the solution you'd like
I would like to develop a contact form for the website, also upon successful submission of contact form, user would be sent to a fresh new page on clicking Send button where the message is displayed as "Thank you We received your submission , Return ". Also on clicking to Return button of that fresh new page, user is redirected to home page.
I would like to work on this issue.
Could you please assign this issue to me under LGMSOC.

Hey @hkaur008 and @yajushiSri kindly assign this issue to me under LGMSOC.

@hkaur008 please assign issue to me as the end date for LGMSOC program is 10th July and I need to finish this issue also needs to make a PR before 10th so that it can be easily merged

You can start, i assign you in morning
No worries 😊
On Wed, 7 Jul, 2021, 7:56 PM ROHAN MODI, @.***> wrote:

@hkaur008 https://github.com/hkaur008 please assign issue to me as the
end date for LGMSOC program is 10th July and I need to finish this issue
also needs to make a PR before 10th so that it can be easily merged
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/CodeFlow201/GitMarkonics/issues/70#issuecomment-875651098,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ANOWNVGQMERAJLO7YXE4EKLTWRPZJANCNFSM472MJEUQ
.

Hey @hkaur008 I would like to redesign the contact form under hactoberfest and change it's UI.

Can I work on this issue?
Kindly assign this issue to me under hactoberfest.

Hey @hkaur008 can I include social media links of codeflow in gitmarkonics contact page as gitmarkonics project is under codeflow also gitmarkonics doesn't have it's own social media handles or links.