Event sources not showing in the calendar (ng1.3.15 + fullcalendar2.0)

Hi,
This is what I got on my view

And on my controller
    $scope.uiConfig = {
      calendar:{
        editable: true,
        header:{
          left: 'month agendaWeek agendaDay',
          center: 'title',
          right: 'today prev,next'
        },
        dayClick: $scope.alertEventOnClick,
        eventDrop: $scope.alertOnDrop,
        eventResize: $scope.alertOnResize
      }
    };

    $scope.eventSources = [
        {
            title  : 'Event 1',
            start  : '2015-05-01'
        },
        {
            title  : 'Event 2',
            start  : '2015-05-05',
            end    : '2015-05-07'
        },
        {
            title  : 'Event 3',
            start  : '2015-05-09 12:30:00',
            allDay : false // will make the time show
        }
    ];

The calendar UI is showing but not the events, here is a screen shot of it.

What do you think was wrong? Please help! :(

and also if I push an item in the eventSources array this function fires
        eventSourcesWatcher.onAdded = function(source) {
          console.log('add');
          scope.calendar.fullCalendar('addEventSource', source);
          sourcesChanged = true;
        };

i don't know what was tho.. maybe this one?
 scope.calendar.fullCalendar('addEventSource', source);

it's just now that I realized that there is no support for 1.3.15

I dealing with this now. eventSources needs to be an array of arrays, not just an array.

yeah, I was able to make it work with 1.3.15 by making the first item of eventSources to be an array which contains the actual events array. Thank you tho!


Simplify launching of a new geth chain (and tooling)

Currently we have a geth pod, and are working on adding tooling to make that work nicely. (#12, #10)
We make it easy to run a second chain, and it's own set of infrastructure. This can be just commands + modified foundry with templated k8s with sh commands if that's the best implementation, but ideally just a couple of clean commands.
ie
export CHAIN_ID={}
export EVM_FUNDING_ADDR={}

just deploy-geth $CHAIN_ID $EVM_FUNDING_ADDRESS
just deploy-eth-explorer $CHAIN_ID
just deploy-faucet $CHAIN_ID $EVM_FUNDING_ADDRESS

We should look into using helm charts to create a configurable EVM, it should allow the EVM to be run both with custom funding account AND with different chain ids.


Feature: Ask the user for input if install previously detected

Currently the express configuration will notify the user that express install has already been run in the shell and prompts the user to unsetZiti before trying to run again.
It would be nice to have the script tell the user it has already been run and ask if they did intend to actually run express install again, if yes, run unsetZiti and proceed with the install, otherwise stop the script.

great idea!


README for Parquet store is out of date

The README for the Parquet store has not been updated since #1884 so is incorrect in places.

Merged into develop


Bump CI action code coverage versions to fix stalled coverage uploads

Description
Fix coverage action CI hangs on upload by bumping action version
Note
No logic changes. This is a CI stability change.

Closing PR. This may have been upload issues unrelated to the action revision. We don't know yet.


DATAJDBC-276 - Id are no longer required for elements of Lists.

Resolved via https://github.com/spring-projects/spring-data-jdbc/commit/c47cd0878d5a2f254972aeb0ea5a752b0998e573.


Error in RNA-Seq pipeline

Hi!
Q1: Is there a reason that bcbio still uses salmon v1.4.0?
[2022-01-21T01:55Z] Version Info: ### PLEASE UPGRADE SALMON ###
[2022-01-21T01:55Z] ### A newer version of salmon with important bug fixes and improvements is available. ####
[2022-01-21T01:55Z] ###
[2022-01-21T01:55Z] The newest version, available at https://github.com/COMBINE-lab/salmon/releases
[2022-01-21T01:55Z] contains new features, improvements, and bug fixes; please upgrade at your
[2022-01-21T01:55Z] earliest convenience.
[2022-01-21T01:55Z] ###
[2022-01-21T01:55Z] Sign up for the salmon mailing list to hear about new versions, features and updates at:
[2022-01-21T01:55Z] https://oceangenomics.com/subscribe

Q2: I get the following error:
[2022-01-21T02:00Z] Timing: estimate expression (single threaded)
[2022-01-21T02:00Z] Combining count files into /media/kokyriakidis/PRODUCTION/EDITING/bcbio/AD/work/htseq-count/combined.counts.
[2022-01-21T02:01Z] tx2gene file /media/kokyriakidis/PRODUCTION/EDITING/bcbio/AD/work/inputs/transcriptome/hg38-tx2gene.csv created from /media/kokyriakidis/RED/BCBIO/RESOURCES/bcbio/genomes/Hsapiens/hg38/rnaseq/ref-transcripts.gtf.
[2022-01-21T02:01Z] Combining tx2gene CSV files.
[2022-01-21T02:01Z] Loading tximport.
[2022-01-21T02:01Z] Error in library(tidyverse) : there is no package called ‘tidyverse’
[2022-01-21T02:01Z] Execution halted
[2022-01-21T02:01Z] Uncaught exception occurred
...
subprocess.CalledProcessError: Command '/media/kokyriakidis/RED/BCBIO/RESOURCES/bcbio/anaconda/bin/Rscript --vanilla -e library(tidyverse);salmon_files = list.files("/media/kokyriakidis/PRODUCTION/EDITING/bcbio/AD/work/salmon", pattern="quant.sf", recursive=TRUE, full.names=TRUE);tx2gene = readr::read_csv("/media/kokyriakidis/PRODUCTION/EDITING/bcbio/AD/work/inputs/transcriptome/tx2gene.csv", col_names=c("transcript", "gene")); samples = basename(dirname(salmon_files));names(salmon_files) = samples;txi = tximport::tximport(salmon_files, type="salmon", tx2gene=tx2gene, countsFromAbundance="lengthScaledTPM", dropInfReps=TRUE);readr::write_csv(round(txi$counts) %>% as.data.frame() %>% tibble::rownames_to_column("gene"), "/media/kokyriakidis/PRODUCTION/EDITING/bcbio/AD/work/bcbiotx/tmph9l25_kn/tximport-counts.csv");readr::write_csv(txi$abundance %>% as.data.frame() %>% tibble::rownames_to_column("gene"), "/media/kokyriakidis/PRODUCTION/EDITING/bcbio/AD/work/bcbiotx/tmpimej1vy0/tximport-tpm.csv");
Error in library(tidyverse) : there is no package called ‘tidyverse’
Execution halted
' returned non-zero exit status 1.

I do not know if there is a problem with my installation or tidyverse is missing from the installed depedencies.
I now try a clean install but I get this error:
# Installing into conda environment default: age-metasv, atropos, bamtools, bamutil, bbmap, bcftools=1.13, bedops, bio-vcf, biobambam, bowtie, break-point-inspector, bwa, cage, cnvkit, coincbc, cramtools, deeptools, express, fastp, fastqc, geneimpacts, genesplicer, gffcompare, goleft, grabix, gsort, gvcfgenotyper, h5py=3.3, hdf5=1.10, hisat2, hmmlearn, htseq, impute2, kallisto=0.46, kraken, ldc, macs2, maxentscan, mbuffer, minimap2, mintmap, mirdeep2, mirtop, moreutils, multiqc, multiqc-bcbio, ngs-disambiguate, novoalign, oncofuse, pandoc, parallel, pbgzip, peddy, pizzly, pythonpy, qsignature, rapmap, rtg-tools, sailfish, salmon, samblaster, scalpel, seq2c<2016, seqbuster, seqcluster, seqtk, sickle-trim, simple_sv_annotation, singlecell-barcodes, snap-aligner=1.0dev.97, snpeff=5.0, solvebio, spades, star=2.6.1d, stringtie, subread, survivor, tdrmapper, tophat-recondition, trim-galore, ucsc-bedgraphtobigwig, ucsc-bedtobigbed, ucsc-bigbedinfo, ucsc-bigbedsummary, ucsc-bigbedtobed, ucsc-bigwiginfo, ucsc-bigwigsummary, ucsc-bigwigtobedgraph, ucsc-bigwigtowig, ucsc-fatotwobit, ucsc-gtftogenepred, ucsc-liftover, ucsc-wigtobigwig, umis, vardict-java, vardict<=2015, variantbam, varscan, vcfanno, viennarna, vqsr_cnn, wham, ipyparallel=6.3.0, ipython-cluster-helper=0.6.4=py_0, ipython=7.29.0, ipython_genutils=0.2.0=py37_0, traitlets=4.3.3, anaconda-client, awscli, bzip2, ncurses, nodejs, p7zip, readline, s3gof3r, xz, perl-app-cpanminus, perl-archive-extract, perl-archive-zip, perl-bio-db-sam, perl-cgi, perl-dbi, perl-encode-locale, perl-file-fetch, perl-file-sharedir, perl-file-sharedir-install, perl-ipc-system-simple, perl-lwp-protocol-https, perl-lwp-simple, perl-sanger-cgp-battenberg, perl-statistics-descriptive, perl-time-hires, perl-vcftools-vcf, bioconductor-annotate, bioconductor-apeglm, bioconductor-biocgenerics, bioconductor-biocinstaller, bioconductor-biocstyle, bioconductor-biostrings, bioconductor-biovizbase, bioconductor-bsgenome.hsapiens.ucsc.hg19, bioconductor-bsgenome.hsapiens.ucsc.hg38, bioconductor-bubbletree, bioconductor-cn.mops, bioconductor-copynumber, bioconductor-degreport, bioconductor-deseq2, bioconductor-dexseq, bioconductor-dnacopy, bioconductor-genomeinfodb, bioconductor-genomeinfodbdata, bioconductor-genomeinfodbdata, bioconductor-genomicranges, bioconductor-iranges, bioconductor-limma, bioconductor-org.hs.eg.db, bioconductor-purecn>=2.0.1, bioconductor-rhdf5, bioconductor-rtracklayer, bioconductor-rtracklayer, bioconductor-summarizedexperiment, bioconductor-titancna, bioconductor-txdb.hsapiens.ucsc.hg19.knowngene, bioconductor-txdb.hsapiens.ucsc.hg38.knowngene, bioconductor-tximport, bioconductor-vsn, r-base=4.1.1=hb67fd72_0, r-chbutils, r-deconstructsigs, r-devtools, r-dplyr, r-dt, r-ggdendro, r-ggplot2, r-ggrepel, r-gplots, r-gsalib, r-janitor, r-knitr, r-optparse, r-pheatmap, r-plyr, r-pscbs, r-reshape, r-rmarkdown, r-rsqlite, r-sleuth, r-snow, r-stringi, r-tidyverse, r-viridis, r-wasabi, r=4.1=r41hd8ed1ab_1004, xorg-libxt
Encountered problems while solving:
  - package 'ipython_genutils-0.2.0-py37_0' is excluded by strict repo priority
  - package perl-vcftools-vcf-0.1.15-pl526_2 requires perl >=5.26.2,<5.27.0a0, but none of the providers can be installed

Collecting package metadata (current_repodata.json): ...working... done
Solving environment: ...working... failed with initial frozen solve. Retrying with flexible solve.
Collecting package metadata (repodata.json): ...working... done
Solving environment: ...working... failed with initial frozen solve. Retrying with flexible solve.
Solving environment: ...working...
Found conflicts! Looking for incompatible packages.
This can take several minutes.  Press CTRL-C to abort.

Only after running it with sudo e.g. sudo python3 bcbio_nextgen_install.py /bcbio --tooldir=/bcbio/tools --nodata managed to make it work and move one.

We are not pinning salmon version:
https://github.com/chapmanb/cloudbiolinux/blob/master/contrib/flavor/ngs_pipeline_minimal/packages-conda.yaml#L111
In my fresh installation:

$ which salmon
/n/app/bcbio/1.2.9/anaconda/bin/salmon
$ salmon -v
salmon 1.6.0

R in anaconda/bin/R has tidyverse:
https://github.com/chapmanb/cloudbiolinux/blob/master/contrib/flavor/ngs_pipeline_minimal/packages-conda.yaml#L262

$ which R
/n/app/bcbio/1.2.9/anaconda/bin/R
$ R

R version 4.1.1 (2021-08-10) -- "Kick Things"
Copyright (C) 2021 The R Foundation for Statistical Computing
Platform: x86_64-conda-linux-gnu (64-bit)

.> library(tidyverse)
── tidyverse 1.3.1 ──
✔ ggplot2 3.3.5     ✔ purrr   0.3.4
✔ tibble  3.1.6     ✔ dplyr   1.0.7
✔ tidyr   1.1.4     ✔ stringr 1.4.0
✔ readr   2.1.1     ✔ forcats 0.5.1

No sure, what exactly happens in your installation, I could only suspect incomplete installation/mixed installations..
Sergey

I do not know what is happening. I can only install it using sudo. Otherwise it gets stuck during the resolve phase. I have to other conda installations in the system.
Installing it using sudo installed the correct package versions.
Thanks again for looking into it.

ok, maybe it is system specific. Have you included --mamba during the installation (helps with solving)?

Yes, I have tried both conda and mamba.


Errors when running npm start

Hi, I have run through the npm install which succeeded then when I ran npm start i was presented with the error see in the attached image. The server did start but the browser console also complains that it cannot find ./client/main.jsx
I am running node 6.5 and npm 3.10.3

Chrome console error:

Hi,
i encountered the same problem.
I figured that the precompiling was not able to recognize the JSX Syntax.
To fix this i explicitly defined all the babel plugins specified in the package.json inside the webpack loader.
In webpack.config.development.js change
config.module.loaders = config.module.loaders.concat([
  {test: /\.jsx?$/, loaders: [ 'react-hot', 'babel'], exclude: /node_modules/}
]);

to
config.module.loaders = config.module.loaders.concat([
  {
      test: /\.jsx?$/,
      loaders: [ 'react-hot', 'babel?presets[]=es2015&presets[]=react&presets[]=stage-0'],
      exclude: /node_modules/
  }
]);

same in webpack.config.production.js just without the hotloading.
config.module.loaders = config.module.loaders.concat([
  {test: /\.jsx?$/, loaders: [ 'babel'], exclude: /node_modules/}
]);

to
config.module.loaders = config.module.loaders.concat([
  {
      test: /\.jsx?$/,
      loaders: [ 'babel?presets[]=es2015&presets[]=react&presets[]=stage-0'],
      exclude: /node_modules/
  }
]);

I am not sure if babel will use a .babelrc file with webpack as I never tested it.
If so this could also be resolved with a single .babelrc
{
   "plugins": ["es-2015", "react", "stage-0"]
}

@jbultmann What was the setup for the .babelrc version? Thanks

Seems too many errors to make this a usable boiler-plate next!


In-built Sprite Editor Feature

Can I try making this?

@RomKart Please edit your original post and elaborate what the feature is about. Like how would the user edit their images through script. The Engine currently doesn't have a GUI editor and is only a library at the moment so it is not really a good idea crop/resize/etc. images through scripting.

@RomKart Yup that would be helpful. But as the game engine doesn't have an actual GUI Editor yet, how do you plan to let the users edit the sprites?

Is Non-GUI Editor possible?
Like uhh.. a separate program for Ducktape Version1.0. till we get GUI Editor?

@RomKart That seems unecessary, and we will have to recreate this entire issue in Ducktape v2.0 when we actually do get a GUI Editor. Thus I'll be closing this issue for now.


Using optimizer Adam, why remove 'momentum=' for training?

In order to use the adam optimizer, SGD was changed to ADAM in the base model file and training was attempted.
But there was an error telling me to remove the momentum. So I removed the momentum and the training was successful.
But why should I  remove Momentum to use Adam when it seems like Adaptive Momentum Estimation combines RMSProp and Momentum?
Does Adam use momentum in MMDETECTION?

From the paper:
Adam updates are directly estimated using a running average of first and second moment of the gradient.
There isn't a fixed momentum setting for Adam.  The underlying optimizer is from Pytorch and doesn't a momentum argument.

You can refer to https://github.com/open-mmlab/mmdetection/blob/682f03db0c0516324ec4e1e5d97e1d93c0a2485a/configs/detr/detr_r50_8x2_150e_coco.py#L141

Feel free to reopen the issue if there is any question.


Update documentation to reflect dismiss behavior of e.g. toasts

Ionic version:
[ ] 1.x
[ ] 2.x
[ X ] 3.x
I'm submitting a ...
[ ] bug report
[ X ] feature request
[ ] support request
Current behavior:
Ionic documentation does not give a clue about how one can distinguish between user initiated closing (click on close button) and automatic closing (after delay) of toasts.
The primary entry point for handling dismissal of a toast is attaching an event handler to the toast via
myToast.onDidDismiss(() => {});

as described in the current documentation found here:
http://ionicframework.com/docs/api/components/toast/ToastController
Only with trial and error and looking into the source code it was possible for me to discover the difference:
If the toast is closed automatically the second parameter of the callback function will receive "backdrop" as its value and "close" if the user clicked on the close button.
Expected behavior:
I expect that this should be clearly stated in the documentation http://ionicframework.com/docs/api/components/toast/ToastController/
Related code:
const myToast= this.toastController.create({
                            message: "Awesome Title",
                            duration: 3000,
                            position: "bottom",
                            closeButtonText: "Undo",
                            showCloseButton: true,
                        });

myToast.onDidDismiss((firstParameter, secondParameter) => {
/* secondParameter will either receive "backdrop" or "close" here :) */
                            console.log(secondParameter);
                        });

myToast.present();

To dismiss the toast after creation, call the dismiss() method on the Toast instance. The onDidDismiss function can be called to perform an action after the toast is dismissed.

As shown in my sample code i do not call
dismiss([data], [role] [navOptions]).

I mean the callback supplied to
onDidDismiss(...)

It is unclear that the callback will receive any arguments.

I will have a look into this.
return this._viewCtrl.dismiss(null, role, {disableApp: false});
In my opinion, the first parameter could be removed because he does nothing (null).

Yeah that will be fine too =D!
 /**
   * Called when the current viewController has be successfully dismissed
   */
  onDidDismiss(callback: Function) {
    this._onDidDismiss = callback;
  }

In my opinion, at least the method summary should be altered to or something similar:
 /**
   * Called when the current viewController has be successfully dismissed.
   * Signature: (role: string) Indicates the initiator for the dismissal ("backdrop", "close"...).
   */

I think e.g. VS Code or any other cool editor would then provide enough information via intellisense to assist the developer to alter application behavior dependent on the initiator of the dismissal.
Notwithstanding the official Ionic Framework documentation should mention this =)

While digging through the source, it seems that this affects all inheritors of ViewController and not only toasts.

Hello all! Good question @danielsogl . So if we look here https://github.com/driftyco/ionic/blob/master/src/navigation/view-controller.ts#L170 you will see that the first param of the dismiss method is data. Now, this is an optional param (hence the ? in its type) but you have to pass something there or the order of the params would be messed up. The onDidDismiss method also has a data param as its first param. Does this answer everyones questions?

Hi @jgw96,
i think my pull request will solve this issue for now, just merge it if you want.
As described in the issue, my only problem was that one must digg deep into the Ionic source code to find out that the callback which is passed to onDidDismiss and onWillDismiss will receive two parameters.
IMHO the developer should be able to learn about the method signature either from the official Ionic Framework documentation (which is... incomplete) or (maybe the better) from the source code directly through e.g. intellisense provided by all major code editors; which requires at least meaningful typescript method definitions and usage of data types.
Why define the callback as "Function" if every call of that callback will pass "(somethingA: any, somethingB: string)" =) i hope you know what i mean?

It's merged! thanks @icedt89


TypeError: Not a Buffer

When using the verifyIdToken to verify a really old token (So far I only encoutered with these kind of tokens), instead of getting an expired error the app throws this error.
I've digged a bit and found that the main issues resides within the line 495 in oauth2client.js:
var pem = certs[envelope.kid];

That resolves to undefined sometimes. envelope.kid holds a value, but not one that matches the certs generated by the getFederatedSignonCerts function in oauth2client.js, thus causing the pemVerifier.verify throw.

In your case, how old was a "really old token"? I'm getting this error with an id token that was about 2 days old. Guess I'm not really sure how long they are supposed to last.

I don't recall right now, but I think that if you try to use a token that it's a day old you should get this error.

In the same verifyIdToken call, I'm now getting errors like "Token used too late,<PHONE_NUMBER>.079 ><PHONE_NUMBER>". I'm not sure if there is some variability in what gets returned from Google's servers, or if some change occurred to those servers.
I think these numbers are referring to the "exp" field of the IdToken, which has the definition:
"The time the ID token expires, represented in Unix time (integer seconds)."
(https://developers.google.com/identity/protocols/OpenIDConnect#obtainuserinfo)
I'll have to see if I can get the IdToken exp field immediately after obtaining the IdToken and convert the Unix time to a h/m/s time.

That is the expected behavior. If a token has expired, then you'll recieve that error. The issue that I'm describing is when instead of getting the correct error (Token userd too late) the pemVerify.verify call throws an error.

That error is to be expected if the token actually expired (You should check the expires_in field if there is any in your response). The issue I'm describing is that instead of getting the correct error (Token used too late) the perVerify.verify call throws an error due to the issue explained above.
If you're getting "Token used too late" you should refresh your access tokens.


Add download_carousel

Description
Commits fb0c968 and b51f725:
Adds a download_carousel method for Posts which allows you to download all media on carousel posts, i.e. posts with multiple images/videos, as raised in #105. Since this is a batch operation, you specify an output directory and a function for calculating the filename for each output image instead of specifying a single output filename. See the method documentation for details.
Also added a couple of supporting methods, though only one of them, parse_carousel_urls, is public; this method simply returns the video and image URLs for each image in the carousel, or None if the post is not a carousel. Again, see docstring for details.
Also added the beginnings of a demo jupyter notebook.
Fixes #105
Commit e88032e:
Post.get_recent_comments would raise a KeyError when using Selenium or a requests.Session object to scrape a Post due to slight differences in the structure of the resulting json_dict. I added an except block to handle this and try the alternative json_dict schema.
Fixes #124
Commit f443435:
Add Profile.iter_posts to get a lazy iterator over posts, and reimplement Profile.get_posts (with the same API) using iter_posts.
Fixes #127
Checklist

[x] I followed the guidelines in our Contributing document
[x] I added an explanation of my changes
[ ] I have written new tests for my changes, as applicable
[ ] I successfully ran tests with my changes locally

Additional notes (optional)
Have not written automated tests yet, will do so soon.

I have attempted to run the added function download_carousel with the following Google Instagram post, but come across a TypeError.
from instascrape import *
from pathlib import Path

def insta_scrape(ig_links):
    for link in ig_links:
        post = link.split("?")[0] if "copy_link" in link else link
        post_folder = Path(post.split("/")[-2])
        post_folder.mkdir(parents=True, exist_ok=True)

        google_post = Post(post)
        google_post.download_carousel(str(post_folder), allow_non_carousel=True)

if __name__ == "__main__":
    link_list = list()
    ig_link = "https://www.instagram.com/p/CXuAeZ1ltCa/?utm_source=ig_web_copy_link"
    link_list.append(ig_link)
    insta_scrape(link_list)

Running the following code above, I get the following Traceback:
$ py insta_scrape.py
Traceback (most recent call last):
  File "insta_scrape.py", line 18, in <module>
    insta_scrape(link_list)
  File "insta_scrape.py", line 11, in insta_scrape
    google_post.download_carousel(str(post_folder), allow_non_carousel=True)
  File "...\insta_download\py_venv\lib\site-packages\instascrape\scrapers\post.py", line 213, in download_carousel
    urls = self.parse_carousel_urls()
  File "..\insta_download\py_venv\lib\site-packages\instascrape\scrapers\post.py", line 157, in parse_carousel_urls
    is_videos = self._filter_get(self.flat_json_dict, self._IS_VIDEO_KEYS)
  File "..\insta_download\py_venv\lib\site-packages\instascrape\scrapers\post.py", line 133, in _filter_get
    return [(k, dic[k]) for k in keys if k in dic]
  File "..\insta_download\py_venv\lib\site-packages\instascrape\scrapers\post.py", line 133, in <listcomp>
    return [(k, dic[k]) for k in keys if k in dic]
TypeError: argument of type 'NoneType' is not iterable
(py_venv)

Stepping through with the debugger, I noticed that the variable self.flat_json_dict has the value of None. I am not sure if anyone else has come across such an error.

I had forgotten to perform the scrape method. With the new line, the function now operates as intended.
from instascrape import *
from pathlib import Path

def insta_scrape(ig_links):
    for link in ig_links:
        post = link.split("?")[0] if "copy_link" in link else link
        post_folder = Path(post.split("/")[-2])
        post_folder.mkdir(parents=True, exist_ok=True)

        google_post = Post(post)
        google_post.scrape()
        google_post.download_carousel(str(post_folder), allow_non_carousel=True)

if __name__ == "__main__":
    link_list = list()
    ig_link = "https://www.instagram.com/p/CXuAeZ1ltCa/?utm_source=ig_web_copy_link"
    link_list.append(ig_link)
    insta_scrape(link_list)


Deactivating repository subscription does not halt scanning tags

Watching a repository is handled by an asynchronous catalog process periodically performing inventory of repository and queuing up images for analysis. This process does an atomic check for the subscription state before evaluating each tag manifest in a serial order. This is not very fast process which makes it seem like the tags are trickling into the system. Deactivating or even deleting the watch/subscription record does not halt this stream as the subscription state check is performed once at the top of the process

Closed by #637 in the v0.8.2-dev branch. Did not auto-close since it hasn't yet merged to master until after the patch release, so closing manually.


Characters not scanned

Current Behavior
During character scanning it only scans the active character, constelations and basic attack lv, then it stops
Console output is
Unable to locate Weapons item count.

Parameter is not valid.

   at System.Drawing.Bitmap..ctor(Int32 width, Int32 height, PixelFormat format)

   at InventoryKamera.Navigation.CaptureRegion(RECT region, PixelFormat format)

   at InventoryKamera.CharacterScraper.ScanTalents(Character character)

   at InventoryKamera.CharacterScraper.ScanCharacter(String firstCharacter)

   at InventoryKamera.CharacterScraper.ScanCharacters(List`1& Characters)

   at InventoryKamera.InventoryKamera.GatherData()

Expected Behavior
The scanner continues to scan
Reproducing a Behavior
Start scanner with Characters selected
Device OS
Windows 11
Genshin Impact Version
4.6
Inventory Kamera Version
1.3.16
Screen resolution, screen mode, and UI scale
1920x1200 windowed
Additional notes and remarks
No response
Evidence
No response
Preflight Checklist

[X] I have checked for similar issues in the issue page, and I have not found any similar to mine.
[X] I am using the latest version of the scanner.
[X] I tried setting the scanning delay to maximum, and it still doesn't work.
[ ] I have attached the video recording to the bug report.
[ ] I have attached the error log to the bug report.
[X] I am available to later explains the issue.
[X] I have disabled (or documented) all screen filtering software, plugins, and enhancements while scanning.
[X] I use default in-game keybindings or have configured keybindings for the scanner.

I tested multiple times, it seems that scanner will stop if the game graphics settings does not set to 16:9 resolution


add other useful lanes

What problems I want to solve?
The pod could have lanes to:

delivery
tagging
other..

What value it delivers?
Repetitive work would be automated lowering chances of failures.
Tasks

[x] Add pull request lane
[x] Tag lane
[x] Publish lane

Dependecies / References
No dependencies and/or references by now.

implemented in #17


Removes namespace bar from Client creation flow

https://bugzilla.redhat.com/show_bug.cgi?id=2276413
Removes Performance configuration from the UI

/hold

Updates as requested:
Screenshots:

/cherrypick release-4.16

/cherrypick release-4.16-compatibility


Re-delete "Do not hoard masks" section

It was inadvertently undeleted.
See this merged PR for the discussion, including the two approvals.

Thank you @jaysonvirissimo 🙏!


fix: Merge errors in tests

Merging introduced a typo

Test run: https://github.com/firebolt-db/firebolt-python-sdk/runs/4968359759?check_suite_focus=true


Finish up documentation for data source onboarding

https://github.com/linkedin/datahub/blob/master/docs/how/data-source-onboarding.md

Refer to the Update data-source-onboarding.md with examples.


[1.1.4] [Regression?] WindowsAppRuntimeInstaller.exe fails if newer version is present

Describe the bug
Installing 1.1.4 on a system with 1.1.5 fails.
Easy to test this out in Windows Sandbox.
Steps to reproduce the bug

Run WindowsAppRuntimeInstall.exe for version 1.1.5
Run WindowsAppRuntimeInstall.exe for version 1.1.4, see error, even though it was fixed in #2723

Installing license: MSIX_MAINPACKAGE_LICENSE
Install result : 0x0
Installing license: MSIX_SINGLETONPACKAGE_LICENSE
Install result : 0x0
Deploying package: Microsoft.WindowsAppRuntime.1.1_1004.584.2120.0_x64__8wekyb3d8bbwe
Package deployment result : 0x80073d06 The package could not be installed because a higher version of this package is already installed.
ErrorMessage: Windows cannot install package Microsoft.WindowsAppRuntime.1.1_1004.584.2120.0_x64__8wekyb3d8bbwe because it has version 1004.584.2120.0. A higher version 1005.616.1651.0 of this package is already installed.One or more install operations failed. Result: 0x80073d06

Expected behavior
Installer should report a success error code, as the dependency is installed. This breaks my current installer on machines that have 1.1.5 already installed.
Screenshots
No response
NuGet package version
No response
Packaging type
No response
Windows version
No response
IDE
No response
Additional context
No response

@sachintaMSFT tag you're it
Seems a dupe of a known issue. And fix in progress?

@sachintaMSFT tag you're it
Seems a dupe of a known issue. And fix in progress?

No, it is not a known issue. Nothing changed in the code path of this fix. So, looking into it.

I'm seeing this issue as well with 1.4:
Add-AppxPackage : Deployment failed with HRESULT: 0x80073D06, The package could not be installed because a higher
version of this package is already installed.
Windows cannot install package Microsoft.WindowsAppRuntime.1.4_4000.1049.117.0_x64__8wekyb3d8bbwe because it has
version 4000.1049.117.0. A higher version 4000.1082.2259.0 of this package is already installed.
NOTE: For additional information, look for [ActivityId] 40eea5b5-47a2-0002-def0-f640a247da01 in the Event Log or use
the command line Get-AppPackageLog -ActivityID 40eea5b5-47a2-0002-def0-f640a247da01


single-pass-wireframes in shader

This is an issue to track a feature I plan to add sometime soon.
The goal is to add Single-Pass Wireframe Rendering to Xenko's shaders, and use that for wireframes in the Studio, instead of 2-pass xray wireframes. Here is an example...

Why?
There are many advantages to single-pass wireframes.

they are massively faster than 2-pass drawing of LINE primitives
they look much nicer
they don't have any z-fighting issues, so they can be z-clipped
they can support advanced wireframe features, like contrast coloring, or depth-test occlusion coloring
automatic back-face line rejection, independent of z-test

The most immediate problem this will fix is the fact that the current x-ray wireframes make placing objects very very confusing. Take a look at two screenshots..
Here is one with a character standing on a surface.

Here is one with the character "submerged" up to the waist. The x-ray wireframes make this very hard to see.

I'm guessing the current wireframes are x-ray because non-xray 2-pass wireframes have many z-fighting issues, and also it may be advantageous to see the wireframes through objects.
My plan is to add single-pass wireframes to the shader, and add some features, such as:

allowing occluded lines to be an alternate (muted) color, or culled
contrast-coloring lines (so on yellow objects they will appear black)

It's true that this case is extremely confusing (good example with the character).

Some of the issues I can think of why 1 pass is not trivial:

As you said, I also think it's advantageous to see the wireframes through objects (it is currently on purpose)
The actual object has backface culling and Z-depth, so we can't draw anything see-through (like backface wireframe). Need to test how it feels (i.e. if I select a cube, does it look better with backface see-through wireframe or not?)
It's not trivial to do in one pass because the user can customize shaders a lot and we're not sure of all the interactions it can have. Still, we have a mechanism for that so it should be possible.
This would also increase shader permutation complexity a bit (i.e. everytime a new material is selected, we might have to recompile version with and without wireframe -- or only the wireframe version with dynamic control but switching to runtime game need the optimized version without wireframe later).

Anyway, this use case with the character is quite bothering, I will have to think if there's a way to fix it with current 2-pass approach (i.e. make the lower half 50% transparent where Z-test fail) or otherwise consider converting to 1-pass (but then we can't do see-through wireframe anymore...)

BTW I thought it was a feature request specific to GS but I skimmed through the part where you need it for your own need!
By all means, go ahead, it could be useful for many other use cases than only the Game Studio (for example for in-game highlighting, etc.)
And if it works well enough and we can figure out those potential issues I listed, we can see if it can be useful for the Game Studio as well!

My plan is to add single-pass wireframes to the shader, and add some features, such as:
allowing occluded lines to be an alternate (muted) color, or culled
contrast-coloring lines (so on yellow objects they will appear black)

How could you do that in a single pass? Doesn't it need a different depth stencil & blend states to have occluded-line being visible while not the case for the model itself?
I suppose there must be some kind of technique/trick, any link/doc?
I am interested to read about that!

Here's the single-pass-wireframes doc link again. It's in the first post also.

How could you do display occluded line in a single pass while drawing the mesh itself?

Doesn't it need a different depth stencil & blend states to have occluded-line being visible while keeping occluded part of the model invisible?

No, the normal technique doesn't need a different depth stencil. It uses the normal z-buffer.
SinglePass wireframes is a texel color shader, which computes the distance from the polygon edge in screen-space to compute a "wireframe line color weight", to perform an anti-aliased blend of the face texel color and edge texel color. By doing it all in one shader, the face color and edge color can be modified and blended in arbitrary ways, using the z-test or comparison with each other.
Normally, occluded texels are fast rejected with an early z-test, but they don't have to be. If one wants to draw occluded line in a single pass, the edge-distance calculation occurs first, and if the texel lies on an edge, and wireframes should be x-ray, there is no z-test fail.. if the z-test passes it blends texel color and edge color, if it fails it draws only edge color.
There are obviously more GPU branches here, but drawing polys is so much faster than line primitives that it's still faster. And performance usually isn't a priority for wire-renderings anyway, or nobody would use line-primitives.

Making this issue more clear, there are really two topics co-mingled here...

TriangleTexelWires vs LinePrimitiveWires for drawing wires
SinglePass (face+wire) vs TwoPass (face then wire)

I assert that TriangleTexelWires  are strictly better than LinePrimitiveWires . TriangleTexelWires are massively faster, and are much more flexible - which means they can look better and have more features. I don't understand any situation where LinePrimitiveWires  are better. They are still in the API because of ease and legacy compatibility.
In most cases, SinglePass rendering of the poly face and wire-edge colors is enough. In some special cases (like avoiding post filtering), maybe a second pass would be warranted, but I still argue the second pass should be TriangleTexelWires , not LinePrimitiveWires.

The actual object has backface culling and Z-depth, so we can't draw anything see-through (like backface wireframe). Need to test how it feels (i.e. if I select a cube, does it look better with backface see-through wireframe or not?)

Obviously if one wants backface wires, one has to turn off backface culling for that object draw. However, when shading a backface polygon, one doesn't have to output every texel, so it's easy to ignore occluded face texels but draw occluded wire-edge texels.
Because GPUs draw filled triangles faster than line primitives, this is generally faster than drawing LINES anyway, and gives more flexibility and looks better. I've never done it, but it should also be easy and fast to draw backface lines in a different color, if that is desired.
From a UI perspective, I find backface wires (along with all x-ray drawing) extremely confusing to look at in all but the simplest models. And few game models are simple. They interfere with the ability to perceive model shape and depth.
I also find wireframe object highlighting to be highly chaotic, especially on high poly models. I personally prefer outline highlight, like Blender (see pic). Maybe some math to vary edge intensity by edge angle could be an alternative.

We render wireframe after post-effects to have always the same color (and also we don't want it blurred or affected by depth of field).

This is a tricky issue i've not considered before. Here are a few brainstorm ideas:

one answer is to keep using two passes, but make wireframe pass be "triangle texel wires", instead of line primitives. I'm worried this could have the same problem as the current 2-pass technique with z-fighting, because 2nd-pass precision issues can cause z-fighting and make lines not solid.

another idea is to use stencil bit(s), and output a value for the opacity of the wire edge at each screen pixel (instead of drawing them).. and then blend those wire-edge pixels during the screen-space pass, but at the end after the filters. This would eliminate all z-fighting issues with the drawback of some more stencil bit(s).

Thanks for the detailed write-up and sorry for missing the link in the original post...
I am totally with you on switching to a triangle-based approach for rendering (aliasing, better rendering control & performance). We already do this kind of thing for rendering the unit grid and it looks great!
This step should be quite easy to do even now, it's just a minor change.
I wouldn't be concerned about Z-fighting: rendering is supposed to be deterministic so Z-fighting should be a non-issue.
Otherwise, any kind of Z-prepass wouldn't work.
Another topic: if we want to keep displaying occluded part but in a more subtle way, here's one way we could do it (at the cost of an extra draw call):

render with negative z-buffer test (to display hidden part, at let's say 20% transparency)
render with normal z-buffer test (to display visible part)

Also, we currently display backfaces of wireframe but I wouldn't mind disabling that.
For me, the important part of making visible occluded wireframe is more the case where it's behind another object/wall.

I just realized some of these fancy ideas like displaying occluded parts differently (in any way) introduces order-dependence, like order-dependent alpha passes.
I think this all could be nicely fixed by keeping the 2-pass technique after screen-space filters... and switching it to a triangle based approach. That also has the advantage that it becomes a separate wireframe shader, rather than bloating the main shader.
I'll think on this some more, but I think this approach sounds pretty flexible and nice.

I want to also add this example of what Blender's outline shader looks like when an object is occluded. It still draws the occluded portion of the outline, and the effect works pretty well.


Fix ARIA Live Region screen reader issue in Firefox

This branch removes the aria-relevant="additions" attribute from the screen reader assistant element. This attribute currently prevents assistant text from being announced by screen readers in Firefox (tested using NVDA).
It's my understanding that there are few advantage to setting aria-relevant unless you want to encourage screen readers to announce more changes (using "all" to announce content removals). The default value of aria-relevant is "additions text". Is it necessary to exclude text changes from being announced here?
resolves #403

Thanks for the PR @geo-connor will check this out on Monday

Sorry haven't gotten around to it yet.

Have you tested this with VoiceOver as well?


[panpress] export runtime code as a library

New @panpress/webapp runtime library.
Affected packages

[x] docs, website
[x] Panpress
[ ] Helpers
[ ] codelabs
[ ] illustrations
[ ] benchmarks
[ ] code-samples
[ ] presentations/conferences
[ ] other: .....

Description
Move all generic runtime code of docs & panpress-starter to a new @panpress/webapp library.
Motivation and Context
Avoid duplications & ease first usage of panpress.
Known issues

[ ] snowpack: can't identify peerDependencies of a link dependency (will use the ./node_modules in webapp instead of the local one, leading to a duplication of lit-html)
[ ] bad dynamic import of prismjs
[ ] 404 redirection (regression): rewrite path to "undefined"

Types of changes

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

Merged to develop and therefore closed in favor of #29


fix(docs): fix the CRON_SCHEDULE on docs

Fix CRON_SCHEDULE value on Readme.md to fit the corn of everyday at 3am

Thanks for that.


Add feature flag for componentized map

Overview
This PR includes the following proposed change(s):

Gate the new componentized map code behind a feature flag
Enable by setting the environment variable ENABLE_COMPONENTIZED_MAP=true before running to switch to the new componentized map code. Eg ENABLE_COMPONENTIZED_MAP=true npm run start.
Refactor app layout to accommodate switching map implementations.
Move componentized map components into a new directory.
Use lazy-loading so the load size can be reduced.

Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[X] Refactoring / Documentation
[ ] Version change

if your change is a breaking change, please add breaking change label to this PR
How Has This Been Tested?
Please describe the tests that you ran to verify your changes.
Screenshots
Please add any relevant UI screenshots if applicable.
Checklist:

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have made corresponding changes to the documentation
[ ] New and existing unit tests pass locally with my changes

sweet!


Shared/team configuration setup

Hi @geerlingguy,
I'm wondering if it would be possible for Drupal VM to support an imports directive in the config.yml, that could load a user's config.yml to override the default values in config.yml (much like how Behat supports doing so).
The scenario I'm thinking of is:

A team has a shared Drupal VM configuration tracked in version control which contains configuration for that team's websites, tools, etc
Each team member can adjust parameters of that config (e.g. the directories where they store their code on the host, the memory allocated to the VM, etc) in a local.config.yml that's not tracked in version control

I'd love to be able to say to my team: do a git pull on our shared configuration repo, clone X drupal site repository to your code directory, run vagrant up, and you've got your local environment set for that site.
At the moment, this involves sharing snippets of config.yml code (vhost, SSL vhost, database, directory mappings) with others.
Thanks for considering!

Yeah, true... when I originally built Drupal VM, it was pretty much a "use for individual projects by one or two devs" thing, but more and more dev teams are standardizing on it, so something like this is definitely desirable.
My favored approach would be to do this the same way you can do a local.settings.php with Drupal through the settings.php conditional include, but it might not be very simple to do it until Ansible 2.0 is released—see highly related issue #174 !

I'm going to merge this issue into #305 — rather than push towards one specific use case, I'd like to get a good survey of how different teams are building and sharing Drupal VM configurations, and then see if there are some common and simple ways to make that more standardized or efficient.


change ui map port

the image currently expose 8080 instead of 3000

I have read the CLA Document and I hereby sign the CLA

Maybe I'm wrong, but: https://github.com/safe-global/safe-wallet-web/blob/dev/Dockerfile

Thanks, I came to the issues looking for someone talking about the ports being wrong after trying and not being able to to ping the UI on browser even docker saying all was good.


fix typo in write_ram_taret and add test all 2304 possible combinatio…

…ns of outline complete

Codecov Report

Merging #75 (668e935) into master (a3a3455) will increase coverage by 0.52%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master      #75      +/-   ##
==========================================
+ Coverage   83.29%   83.81%   +0.52%
==========================================
  Files          69       69
  Lines        4914     4912       -2
==========================================
+ Hits         4093     4117      +24
+ Misses        821      795      -26

Flag
Coverage Δ

unittests
83.81% <ø> (+0.52%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

utils/outline_complete.py
95.74% <ø> (+2.03%)
:arrow_up:

utils/write_ram_target.py
96.65% <ø> (+9.09%)
:arrow_up:

base/forms.py
66.83% <0.00%> (+0.25%)
:arrow_up:

utils/write_noble_target.py
96.42% <0.00%> (+1.33%)
:arrow_up:

Continue to review full report at Codecov.

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


Duplicate the logic of not showing points when moving in the web viewer.

@wally-the-cartographer merge

Merge requested by authorized user pdubroy. Merge queue now has a length of 1.


Connect to node over Tor

I'm submitting a…

[ ] Regression (a behavior that used to work and stopped working in a new release)
[ ] Bug report
[x] Feature request
[ ] Documentation issue or request
[ ] Support request

Expected Behavior
Should be able to connect to node over Tor. This is useful not just because of network level privacy, but also UX - paste in link!

Current Behavior
Cannot connect over Tor

Possible Solution
For example - this is the Tor address you get from a node-in-a-box like Umbrel. I can plug this into wasabi and it should work fine. It would be great to have this with Caravan - even though it may not directly impact the Unchained Vault app.

I believe the solution here is to run caravan within the tor network and connect to your node using tor-browser (or equivalent setup).
If you are running caravan outside of the tor network then you cannot connect it to a bitcoin full node within tor without using a proxy which could dox you
You may be able to accomplish this by doing the following (untested):

Clone the repository
npm i
 npm run build
Take the resultant build directory and host that within the tor network (potentially even on the same machine that runs your node)
Point caravan at your tor node

I believe MyNode does this out of the box.  The reason you can connect wasabi out of the box is because it also runs on the tor network.  Since caravan is just a set of static html files it doesn't have this ability.
Something else to consider, since caravan is stateless, is to use something like OnionShare to host caravan.  Again I haven't personally tested this but in theory it should work.
I am closing this issue because there is nothing we can do on our end.


transfer repo

I think it would be best to transfer this repo away from me and over to @ebuhle. If not, perhaps one of the WDFW folks could suggest an alternative?

LOL. I can't say I was waiting for it, but I'm not surprised either. Sorry for all the spam, @mdscheuerell.
I would be happy to host this repo, but if there is a "proper" GitHub way of transferring the contents from one user to another, I'm unaware of it. @mdscheuerell, what would you suggest as the cleanest and least hacky / error-prone approach?
The biggest problem I can see is that all the existing Issues would presumably remain here. Many of them are still open and active, and even the closed ones contain important archival material -- I was just reading through several of them when you created this one.

(W)hat would you suggest as the cleanest and least hacky / error-prone approach?

It's rather straightforward and it preserves pretty much everything re access, issues, etc. You can read an overview here.

Sorry for all the spam.

It's actually been kind of nice to eavesdrop on the conversation 😉.

It's rather straightforward and it preserves pretty much everything re access, issues, etc. You can read an overview here.

Whoa, amazing! I had no idea. Well, it sounds like you would just need to transfer it to me and I would confirm the transfer. I would propose also renaming it to /LCRchumIPM. Presumably anyone interfacing with the repo via RStudio would need to manually update the target in Project Options?
If that's all there is to it, I say go ahead. And since you'd be automatically added as a contributor, you would still get to eavesdrop.

Success! The repo has been transferred to my account and renamed LCRchumIPM.
@kalebentley @tbuehrens @Hillsont @BradGarnerWDFW If you are using RStudio, you can still push and pull without changing anything; the old origin URL will be automatically redirected. You will see a message indicating that the origin URL has changed and reminding you to update it. To do that:

Go to the Git pane, click the More (gear) menu, then New Terminal.
Following the instructions here, type git remote set-url origin <new-url>. I use SSH to connect RStudio to GitHub, so in my case <new-url> is git@github.com:ebuhle/LCRchumIPM.git.

That's it! I'm pleasantly shocked at how painless this was. It really does just work.
So long @mdscheuerell (to your user account, anyway), and thanks for all the fish. :fishing_pole_and_fish:

Thanks @ebuhle.  When I opened the existing .Rproj file and hit "pull", I got a message saying that a bunch of files would be overwritten and so the action was aborted.   Therefore, I just moved the chumIPM folder and cloned the newly created LCRchumIPM repo.  Seems to have done the trick.

I'd have to see what the specific issue was, but I would have expected things to be overwritten unless @kalebentley did not have everything committed and pulled on his side first.


feat:about

What does this PR do?
--Add the About page
Description of Task to be completed?

How should this be manually tested?
--npm start
What are the relevant Trello Card stories?  Trello Card Item
-https://trello.com/c/zt31Unan/40-about-page
Pull Request Type (check one)

[x] Feature
[ ] Bug Fixes
[ ] Chore

Developer Checklist

[x] New code has been scanned for ESlint issues
[x] I have tested my code locally
[x] I made sure that there are no other pull requests that solve for this issue
[x] My code follows the same structure within the guidelines

Screenshots (if appropriate)

`Failed to compile.
./src/pages/about/About.js
Line 12:1:  This line has a length of 343. Maximum allowed is 200  max-len
Search for the keywords to learn more about each error.`
@Utieee @Easybuoy am i the only one getting this error?

`Failed to compile.
./src/pages/about/About.js
Line 12:1: This line has a length of 343. Maximum allowed is 200 max-len
Search for the keywords to learn more about each error.`
@Utieee @Easybuoy am i the only one getting this error?
Looks like the new eslint rule is blocking it from compiling on my end.
Once that is resolved. I will approve and merge in.

Yeah, the lines just need to be broken to separate lines. Unfortunately, eslint --fix doesn't fix it automatically

@kouamendri Fixed, check again

@Easybuoy could you take a look at this? https://ibb.co/sV4Qbt7


FreeInit Error- Expected all tensors to be on the same device, but found at least two devices, cuda:0 and cpu!

Getting the following error when using FreeInit Iteration Options, if I do not use FreeInit everything works perfectly. Everything else in ComfyUI is working normally except this one thing. It used to work but after updating to the latest it no longer works.
[AnimateDiffEvo] - INFO - Sliding context window activated - latents passed in (32) greater than context_length 16.
[AnimateDiffEvo] - INFO - Using motion module AnimateLCM_sd15_t2v.ckpt:v2.
[AnimateDiffEvo] - INFO - Iteration 1/2
!!! Exception during processing!!!
Traceback (most recent call last):
File "I:\AI\ComfyUI\ComfyUI\execution.py", line 152, in recursive_execute
output_data, output_ui = get_output_data(obj, input_data_all)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "I:\AI\ComfyUI\ComfyUI\execution.py", line 82, in get_output_data
return_values = map_node_over_list(obj, input_data_all, obj.FUNCTION, allow_interrupt=True)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "I:\AI\ComfyUI\ComfyUI\execution.py", line 75, in map_node_over_list
results.append(getattr(obj, func)(**slice_dict(input_data_all, i)))
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "I:\AI\ComfyUI\ComfyUI\custom_nodes\ComfyUI-Inspire-Pack\inspire\a1111_compat.py", line 229, in sample
latent = inspire_ksampler(model, seed, steps, cfg, sampler_name, scheduler, positive, negative, latent_image, denoise, noise_mode, incremental_seed_mode=batch_seed_mode,
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "I:\AI\ComfyUI\ComfyUI\custom_nodes\ComfyUI-Inspire-Pack\inspire\a1111_compat.py", line 96, in inspire_ksampler
raise e
File "I:\AI\ComfyUI\ComfyUI\custom_nodes\ComfyUI-Inspire-Pack\inspire\a1111_compat.py", line 84, in inspire_ksampler
samples = common.impact_sampling(
^^^^^^^^^^^^^^^^^^^^^^^
File "I:\AI\ComfyUI\ComfyUI\custom_nodes\ComfyUI-Inspire-Pack\inspire\libs\common.py", line 14, in impact_sampling
return nodes.NODE_CLASS_MAPPINGS['RegionalSampler'].separated_sample(*args, **kwargs)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "I:\AI\ComfyUI\ComfyUI\custom_nodes\ComfyUI-Impact-Pack\modules\impact\special_samplers.py", line 408, in separated_sample
return separated_sample(*args, **kwargs)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "I:\AI\ComfyUI\ComfyUI\custom_nodes\ComfyUI-Impact-Pack\modules\impact\impact_sampling.py", line 214, in separated_sample
res = sample_with_custom_noise(model, add_noise, seed, cfg, positive, negative, impact_sampler, sigmas, latent_image, noise=noise, callback=callback)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "I:\AI\ComfyUI\ComfyUI\custom_nodes\ComfyUI-Impact-Pack\modules\impact\impact_sampling.py", line 158, in sample_with_custom_noise
samples = comfy.sample.sample_custom(model, noise, cfg, sampler, sigmas, positive, negative, latent_image,
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "I:\AI\ComfyUI\ComfyUI\custom_nodes\ComfyUI-AnimateDiff-Evolved\animatediff\sampling.py", line 536, in motion_sample
latents, noise = iter_opts.preprocess_latents(curr_i=curr_i, model=model, latents=latents, noise=noise,
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "I:\AI\ComfyUI\ComfyUI\custom_nodes\ComfyUI-AnimateDiff-Evolved\animatediff\sample_settings.py", line 475, in preprocess_latents
noised_latents = latents * sqrt_alpha_prod + noise * sqrt_one_minus_alpha_prod
^~~~~~~~~~~~~~~~~~~~~
RuntimeError: Expected all tensors to be on the same device, but found at least two devices, cuda:0 and cpu!
Prompt executed in 2.48 seconds

Thank you for the report, I'll look into it

BTW, I am running with an RTX 4090 and 32GB of internal Ram so I got plenty of juice.

I've been unable to replicate your issue, could you create the simplest workflow you can that has the problem? And can you try to use the vanilla KSamplers from ComfyUI? (you can start off with this txt2vid example in the readme: https://github.com/Kosinkadink/ComfyUI-AnimateDiff-Evolved?tab=readme-ov-file#txt2vid)

Could it be related to my torch version?
Total VRAM 24564 MB, total RAM 32583 MB
pytorch version: 2.4.0+cu121
Set vram state to: NORMAL_VRAM
Device: cuda:0 NVIDIA GeForce RTX 4090 : cudaMallocAsync
Using pytorch cross attention`
The only other time i've ever seen an error similar to that in the past , it was related to torch

Did you try it with a normal ksampler?

Ok so I just tried the normal ksampler and that is in fact working, but when I use KSamplerPipe //Inspire then I get the errors so the problem seems related to that node.

Cool, sometime I'll try out that specific node that gives you trouble and I'll see if there's a workaround I can add in my code to just make FreeInit work with it. Something is just not on the right device when it gets there when using that other KSampler.

But for now you can just use other KSamplers to workaround it

I just did git pull of the latest inspire pack and get the same error

Ya ill have to switch ksampler to get around it thanks.

One more update , I tried ksampler pipe from impakt as well as non pipe versions and none of them work, I also did not see a way to get the  AYS SD1 scheduler option with the standard working ksampler which I have found that one to be noticeably the best scheduler to use for animatediff.

FYI, as another work around Ksampler from efficient does work and that has AYS SD1

Once the rework-modelpatcher PR is merged after my ComfyUI hook PR gets merged, this issue should go away entirely. https://github.com/Kosinkadink/ComfyUI-AnimateDiff-Evolved/pull/498


one shot whisper to chat completion

Describe the feature or improvement you're requesting
Probably not the right place, but it would improve the api tremendously if instead of a text prompt of a chat completion request to add a feature where an audio file can be transcribed and interpreted at once.
Additional context
Right now it is necessary to do two calls.
first: to whisper
second: to gpt completion with the response of whisper.

I disagree. While I'm doing the same, I think the separation is beneficial. For one you can test Whisper without it then invoking ChatGPT, and you can test ChatGPT without having to re-submit the audio for transcription. The effort and network-traffic increase to do this in two steps is minimal.

It would be a nice feature to have, to simplify things for those using Whisper who want a simplified workflow.   However I definitely would not want to do away with the option to keep them separate, as many users incorporate other Speech models and frameworks, like Apple's Speech Framework, or Google's API's for Text To Speech and Speech To Text.
Also, Even if it's fed directly to  GPT from Whisper, I still want the direct output of Whisper before going to GPT for error handling, general evals of the model, or prompts used, and for more creative uses (feeding the output of one model to the input of another, then taking that output and sending it back, networking neural engines in effect..

At this time there's no plan to merge these two features together into one call.  I'll bring up the idea to the team cc @jeffsharris


❓ [Question] How can I load a TensorRT model generated with trtexec?

❓ Question
How can I load into Pytorch a TensorRT model engine (.trt or .plan) generated with trtexec ?
I have the following TensorRT model engine (generated from a ONNX file) using the trtexec tool provided by Nvidia
trtexec --onnx=../2.\ ONNX/CLIP-B32-image.onnx \
        --saveEngine=../4.\ TensorRT/CLIP-B32-image.trt \
        --minShapes=input:1x3x224x224 \
        --optShapes=input:1x3x224x224 \
        --maxShapes=input:32x3x224x224 \
        --fp16

I want to load it into Pytorch for using the Pytorch's dataloader for fast batch ineference.

You can use torch_tensorrt.ts.embed_engine_in_new_module API to do this. Reference: https://pytorch.org/TensorRT/py_api/ts.html?highlight=embed#torch_tensorrt.ts.embed_engine_in_new_module
The resulting torchscript module can be used with the dataloaders for inference.

Thanks, that's exactly what I needed.


commutator of 2-body terms

Hi, I want to use pdaggerq for the following task:
For two different 2-body hamiltonians $H, H'$, both of the form $\sum_{p,q,r,s}h_{pqrs}a_p^+ a_q a_r^+ a_s$,  compute $<\Phi_{HF}|[H,H']|\Phi_{HF}>$. Here $|\Phi_{HF}>$ is the Hartree-Fock state, or Fermi-vaccum, $p,q,r,s$ are indices of arbitrary spin-orbitals.
I tried with
pq = pdaggerq.pq_helper("fermi")
pq.add_commutator(1.0, ['a*(p)', 'a(q)', 'a*(r)', 'a(s)'], ['a*(t)', 'a(u)', 'a*(v)', 'a(w)'])
pq.simplify()
pq.print()

and got the output

    // fully-contracted strings:
    //     +     1.00000000000000 P(k,l) d(i,j) d(l,m) d(a,v) d(k,n)
    //     +     2.00000000000000 P(i,l) d(i,j) d(a,b) d(c,v) d(k,l)
    //     +     1.00000000000000 P(i,l) d(j,k) d(l,m) d(a,v) d(i,n)
    //     -     1.00000000000000 P(j,l) d(i,k) d(l,m) d(a,v) d(j,n)
    //     +     2.00000000000000 d(j,k) d(a,b) d(c,v) d(i,l)
    //     -     1.00000000000000 P(i,j) d(a,c) d(j,k) d(b,v) d(i,l)
    //     +     1.00000000000000 d(a,b) d(c,d) d(e,v) d(i,j)
    //     -     2.00000000000000 d(a,v) d(i,j) d(b,c) d(k,l)
    //     +     1.00000000000000 d(a,v) d(b,c) d(d,e) d(i,j)

It seems that I don't quite understand the output yet. Is d(l,m) the Dirac delta? What does P(k,l) stand for? Also, I expected the outcome to use the same dummy indices as I provided in the input.
Any help is very welcome!

Yes, d(l,m) is a Kronecker delta function.
P(k,l) is a permutation/antisymmetrizer operator, e.g. P(k,l) A(a,b,k,l) = A(a,b,k,l) - A(a,b,l,k)
For the fermi vacuum, general summation labels are split into occupied and virtual labels. The way the code decides if you're working with a general label or not is by checking the label against the list of labels it considers to be occupied or virtual.
looking at the code now, I think there will be a problem with your input for the label "v". It is being interpreted as a virtual label because, internally, v0, v1, etc. are used as virtual labels, and I was only checking the first character of the string. I'll fix this.

there is a new feature branch "general_operators" which is more appropriate for what you're trying to do. The interface is a little clunky, but I think it should work:
"""

    direct specification of pq strings including tensors

    pq.add_new_string(factor, type, order, amps_labels, ops_labels, has_permutational_symmetry)

    :param factor: numerical factor
    :param type: list of labels for amplitudes (e.g., ['A', 'B'])
    :param order: list of order for amplitudes (e.g., [1, 2] -> 'A1', 'B2')
    :param amps_labels: list of list of labels for the amplitudes (e.g., [['i'], ['j', 'k']] -> 'A1(i)', 'B2(j,k)')
    :param ops_labels: list of labels for the creation/annihilation operators (include "*" for creators)
    :param has_permutational_symmetry: do the amplitudes have permutational symmetry, e.g., t2(a,b,i,j) = -t2(b,a,i,j), etc.?

"""

import pdaggerq

def main():

    pq = pdaggerq.pq_helper('true')

    A_op = ['p*', 'q',  'r*', 's']
    B_op = ['t*', 'u',  'w*', 'x']

    A_labels = ['p', 'q', 'r', 's']
    B_labels = ['t', 'u', 'w', 'x']

    pq.add_new_string( 1.0, ['A', 'B'], [2, 2], [A_labels, B_labels], A_op + B_op, False)
    pq.add_new_string(-1.0, ['B', 'A'], [2, 2], [B_labels, A_labels], B_op + A_op, False)

    pq.simplify()
    terms = pq.strings()
    for term in terms:
        print(term)
    pq.clear()

if __name__ == "__main__":
    main()

Thanks for the reply!
I switched to  the "general_operators" branch and ran your script; it seems that add_new_string() is not expecting a numerical factor, I am getting this error:
TypeError: add_new_string(): incompatible function arguments. The following argument types are supported:
    1. (self: pdaggerq._pdaggerq.pq_helper, arg0: List[str], arg1: List[int], arg2: List[List[str]], arg3: List[str], arg4: bool) -> None

Without the prefactor it works fine
pq.add_new_string(['A', 'B'], [2, 2], [A_labels, B_labels], A_op + B_op, False)
pq.add_new_string(['B', 'A'], [2, 2], [B_labels, A_labels], B_op + A_op, False)

but I guess I need the minus sign for the commutator.
Lastly, I wanted to ask what types of permutational symmetry are possible? The Hamiltonians I have obey the 8-fold symmetry of Coulomb-tensors, i.e.
$H_{pqrs} = H_{qprs} = H_{pqsr} = H_{qpsr} = H_{rspq} = ...$
The docstring mentions the antisymmetric properties of the amplitudes, t2(a,b,i,j) = -t2(b,a,i,j), which is not quite what I want, but perhaps there is an option I don't see

whoops, I hadn't pushed my most recent changes. if you pull now, the numerical factor parameter should be there.
for the permutational symmetry, only antisymmetry is implemented with this add_new_string() function. the only impact for you is that you'll end up with terms that you can combine by hand given the symmetry properties of your tensors.

Currently, order doesn't do much because I'm still working on this branch and the functionality isn't complete yet. For your use case, it doesn't matter what you set it to, as long as it is not zero. If the order is zero, then the print statements will not include labels on the tensors.

I see, thanks again!


RPM Package dependency error with OpenSuse distro

Bug report
System info: OpenSuse Tumbleweed witrh InfluxDB 1.5.1
Steps to reproduce:

Download RPM at https://dl.influxdata.com/influxdb/releases/influxdb-1.5.1.x86_64.rpm
Try to install using zypper package manager.

Expected behavior: RPM installation processing without error
Actual behavior: Dependency problem:
Problem: nothing provides shadow-utils needed by influxdb-1.5.1-1.x86_64
 Solution 1: do not install influxdb-1.5.1-1.x86_64
 Solution 2: break influxdb-1.5.1-1.x86_64 by ignoring some of its dependencies

Additional info: Problem located in script inlfuxdb_packages.bash which include hardcoded dependency to shadow-util but OpenSuse equivalent package is named shadow. If I remember correctly, rpmbuild tool should be able to detect those deps, no ? So you wouldn't have it to specify it.
Thanks :)

I am having same issue here ? is there any solution to this ?

@lanox I, too am having this issue

I am facing this in openSUSE 42.3 with an Ansible task (zypper module) to install influxdb.

The Problem still exists with a recent version is there any solution yet?

The state has not changed.  For OpenSuse we continue to suggest "rpm -i --nodeps" for Influxdb.

come here from Aug 2020


No sound on quicktime on M1 Mac

Hi, after the codec is converted, I can play it in QuickTime and the image is great but with no sound. Then I tried on IINA, the sound is okay but image is still wrong color.

Please provide sample file

Same issue.
Here's the torrent of the video
4k.Lightyear.2022.2160p.WEB-DL.DDP5.1.Atmos.DV.MP4.x265-NOGRP- 4KSJ.torrent.zip
.

I just encountered the same issue, with a file named the following:
´Silo.S01E01.2160p.ATVP.WEB-DL.DDP5.1.Atmos.DV.MP4.x265-DVSUX´.

I just encountered the same issue, with a file named the following: ´Silo.S01E01.2160p.ATVP.WEB-DL.DDP5.1.Atmos.DV.MP4.x265-DVSUX´.

Same issue with the same file.


Add the aml widget source into our own

For #10783
This is part of the work for this bug. This PR adds the azureml_widget source into our extension so we can find it (as it isn't available on unpkg.com)

Codecov Report

Merging #10832 into master will decrease coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master   #10832      +/-   ##
==========================================
- Coverage   60.69%   60.67%   -0.02%
==========================================
  Files         580      580
  Lines       31509    31508       -1
  Branches     4482     4481       -1
==========================================
- Hits        19125    19119       -6
- Misses      11416    11417       +1
- Partials      968      972       +4

Impacted Files
Coverage Δ

src/datascience-ui/react-common/arePathsSame.ts
75.00% <0.00%> (-12.50%)
:arrow_down:

src/client/common/utils/platform.ts
64.70% <0.00%> (-11.77%)
:arrow_down:

src/client/linters/pydocstyle.ts
86.66% <0.00%> (-2.23%)
:arrow_down:

src/client/datascience/debugLocationTracker.ts
76.56% <0.00%> (-1.57%)
:arrow_down:

src/client/common/process/proc.ts
14.49% <0.00%> (-0.73%)
:arrow_down:

src/client/debugger/extension/adapter/factory.ts
94.28% <0.00%> (ø)

src/client/constants.ts
100.00% <0.00%> (+16.66%)
:arrow_up:

Continue to review full report at Codecov.

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


Creating nav-bars for more info about company (without js)(header)

As a site visitor, I want to know more about the company's other channels and more details information
Branch(es)

[ ] Create an issue related to this user story on Github and assign self.
[ ] Add a branch named nav-bar
[ ] After html and css are done, commit
[ ] Adjust development-strategy.md for next development, if needed
[ ] Push the changes
[ ] Pull request and merge it on Github

HTML

[ ] Add nav bar (as shown on the browser)

CSS

[ ] Add styling on new elements

Ladies @adekimpianna  @Boam1994 , I worked very hard to do similar but I only do like this. Could you please look how it will be best?


soc: atmel: sam: Add invalidate d-cache at z_arm_platform_init

Before that fix, the SOC was unable to boot properly. Starting turned directly into z_arm_usage_fault().
Fixes zephyrproject-rtos#73485

Hi @tswaehn ,
Thank you so much for this fix. Could you change the brief line on your commit to "soc: atmel: sam: Add invalidate d-cache at z_arm_platform_init"? This helps us to filter the issues in a better way for the platform and function.

Hi @tswaehn ,
Thank you so much for this fix. Could you change the brief line on your commit to "soc: atmel: sam: Add invalidate d-cache at z_arm_platform_init"? This helps us to filter the issues in a better way for the platform and function.

sure will do, ...

@nandojve is it possible to also merge this into v3.6-branch ?

@nandojve is it possible to also merge this into v3.6-branch ?

Hi @tswaehn ,
I believe it is. Just follow the procedure to create a backport PR to that branch : )


SVG rendering of documentation figures is broken in Firefox

The figure docs/shifting.svg uses arrows to denote data movement. In the Inkscape editor, the tips of these arrows are red, in the Vivaldi browser, the tips are black, and in the Firefox browser, they are not shown at all. This needs to be fixed either in the SVG or a rasterized version has to be used.

Closing, as this page/figure has been removed.


Shell script evidence stopped working with 1.6.3

As reported by another user on the google groups the shell script evidence source seems to be broken with the recent update. My script was working perfectly until recent 1.6.3 update and stopped returning a positive result when true:
#!/bin/sh

for ip in 192.168.0.{30..32}; do ping -t 1 $ip > /dev/null && echo "${ip} is up"; done

Thanks for your support!

The evidence source hasn't changed in 11 months...can you try http://www.controlplaneapp.com/download/1.6.2 and see if it starts working again?

I tried. It's working in 1.6.1 but it's not working in either 1.6.2 or 1.6.3
For the moment I'm rolling back to 1.6.1 to keep it functional. Is there any risk with this version regarding sparkle framework?

Yes, 1.6.1 has the sparkle issue. 1.6.2+ are the only versions compiled on newer versions of Xcode so I'm guessing that's the issue.

I have the same issue. The scripts are called, but the results never interpreted as true.

I just pushed an update, try it and let me know if it's working better for you now.

Thank you for the update. Unfortunately still not working on my side. Script is called without an error but always returned as false.

I'm seeing the same behavior (Shell script evidence sources work in 1.6.1, broken in 1.6.2, semi-functional in 1.6.4). In my case, the script in question that was working is run with the applescript interpreter:
#!/usr/bin/osascript
with an error or return line, which then returns 1 or 0 to the shell as verified with echo $? after calling the script.
1.6.1: Works (returns correct value in evidence sources based on script logic)
1.6.2: Broken (always shows as false in evidence sources)
1.6.4: Broken (always shows as false in evidence sources)
To test, I then created two bash scripts that return 0 or 1:
#!/bin/bash exit 0
#!/bin/bash exit 1
1.6.1: Works (first script returns true in evidence sources, second one is false)
1.6.2: Broken (both scripts always show false in evidence sources)
1.6.4: Works (first script returns true in evidence sources, second one is false)

1.6.2 simply doesn't work, this is confirmed. Apple clearly changed the behavior of NSTask but it isn't documented. Using echo $? won't work because...reasons with how NSTask is working. If you exit $? it might work however. As it is now, you can either echo 0 or 1 or exit with 0 or 1 to get the expected behavior. Using echo $? is going to fail.

Thanks for the hint. Just in case others are running into issues with applescripts used as evidence sources, I was able to workaround this with the following (assuming shouldExit indicates whether the output of the script should fail):
1.6.1 and earlier, I could exit an applescript and have it interpreted as success or failure like this:
#!/usr/bin/osascript if shouldExit = "fail" then
  error "Exiting with failure"else
  return "Exiting with success"end if
With 1.6.2+, I can workaround with the following change:
#!/usr/bin/osascript if shouldExit = "fail" then
  log "Exiting with failure"  -- Not necessary, but useful for terminal output   error "1"
else    log "Exiting with success"  -- Not necessary, but useful for terminal output
  return "0"end if
Is this something that should be filed with Apple?

Thanks @dustinrue for the info and @ericpruett for the tip.
@dustinrue Perhaps the info in the interface could be changed to reflect the change in behavior.

I also just noticed that exit 0 and exit 1 work with bash but not with zsh...

I have the same problem. I can't get Shell scripts to execute either, they bomb out with (script terminated with a non-zero status '127').
I use zsh.

At the top of your script you adding #!/bin/zsh?

Thanks @dustinrue, I tried this and now it works, but it seems like the path is not being loaded. Eg my command fails unless I run /usr/local/bin/vagrant, while if I run the script normally, vagrant by itself is enough.

I'm not surprised. I'm almost positive the scripts are run through bash specifically so your environment isn't going to be loaded when you ask bash to run zsh to run your script. What you can do is, after the #!/bin/zsh, is source your zsh environment files. source $HOME/.zshrc for example. I'm not completely familiar with zsh so your personal profile scripts might be named differently.


MPC tilling generates incorret PDAL split call when we use negative origins,

When calling mpc_tilling and we define an negative negative pdal split reports that no value was provided.
pdal split -i C_25EZ2.laz -o ./3/C_25EZ2.laz --origin_x -212.6837 --origin_y 99063.1429 --length 2816.85898203

PDAL: kernels.split: Argument 'origin_x' needs a value and none was provided.

To get around we need to use the assignment syntax: --origin_x=-212.6837

Solved by #24


Surface Feature Toggle button doesn't stay on

Describe the bug
Users are able to turn on "This is a surface feature" for polygonspots, however after saving and returning to the overview page the toggle button stays in the off position.
To Reproduce
Draw a polygon. Select "this is a surface feature". Define Surface feature. Save.
Smartphone (please complete the following information):
Device: iPad pro 3rd Generation
OS: iOS 16.2
StraboSpot Version: 1.35.2

fixed with commit https://github.com/StraboSpot/StraboSpot2/commit/d55e08cd226d2a0d26168ff1f8c0f036cfd1292b