text:
add admin UI demo link

The idea here is to surface it for everyone to quickly see how the admin UI works and feels.

Deploy preview for netlify-cms-www ready!
Built with commit 14ee2cee1086aa30e78358fcfd145d94681da107
https://deploy-preview-1622--netlify-cms-www.netlify.com

npm run lint:format from main package.json is failing =(
I don't know why

Links to demo site https://cms-demo.netlify.com/

One of the downsides to featuring the demo so prominently is that it doesn't really work like Netlify CMS does in reality. It has no backend, and no associated site to display the content. The changes you make are saved in memory, and lost on reload.
The demo site is great for testing development of Netlify CMS itself. As a tool for getting people to try it, however, I think it falls short.
People don't use Netlify CMS on their sites because it has a super awesome UI. They use it because it has a really cool relationship with Git, and enables development patterns that you can't really get with traditional CMSs.
For me, this is the magic of Netlify CMS:

Seeing a realtime preview of the page right next to the editor.
Saving a draft in the CMS and seeing it create a PR on the repo.
Publishing that draft and seeing that the CMS merges the PR and deletes the branch.
Realizing that you can still use regular Git functionality to change and merge that content, in parallel with other users using the CMS.

... and the demo does none of this. We do link to the demo from the docs Intro, but I hesitate to drive people to it so strongly from the front page, because I think people going there will find it kind of meh, and possibly miss out on the very best part.

“Its like a finger pointing away to the moon. Dont concentrate on the finger or you will miss all that heavenly glory.”
― Bruce Lee

😆 but I digress!
On the other hand, we've talked about making an actual repo-backed demo site where people make real edits on it. We'd probably want to set a routine to reset it periodically, and of course someone will come along now and then and post something inappropriate, but I do think it could be pretty cool nonetheless.

Going to go ahead and close this as there's been a consensus here and elsewhere that we'd rather get a full fledged demo working.

authors: DavidWells erquhart verythorough


text:
Validation [WIP]

- Summary
Adds validation to widgets. Available validations:

Presence: By default all widgets are required, unless specified in the config. Example:
- {label: "Subtitle", name: "subtitle", widget: "string", required: false}

Pattern: Field configuration can specify a regex pattern with the appropriate error message. Example:
- {label: "Title", name: "title", widget: "string", pattern: [".{10,}", "Should have more than 10 characters"] }

The widget control can optionally implement an isValid method to perform custom validations, in addition to presence and pattern. The isValid method will be automatically called, and should return an object with an error key. Examples:
No errors:
  isValid = () => {
    // Do internal validation
    return { error: false };
  };

Existing error:
  isValid = () => {
    // Do internal validation
    return { error: 'Your error message.' };
  };

- Test plan

- Description for the changelog

- A picture of a cute animal (not mandatory but encouraged)
……(\__/)
……(=’.’=)
…☆(”)_(”)☆

Looking good so far

source: gharchive/pull-request

authors: calavera cassiozen

repo: netlify/netlify-cms

license: MIT

license_type: permissive

license_source: github-api


text:
fix: Fix the umd builds

Summary

One fix to import the backends correctly
Fixes for a caps issue with the Redux folder name cases vs alias vs module import (alias added to resolve)
Fix the localForage folder case vs alias vs module import (alias added to resolve)
Fix a path import issue from '.' to './index'

Test plan
Same tests as original PR #2141
A picture of a cute animal (not mandatory but encouraged)
🐢

@erquhart I found these issues. Not all breaking, because the case issues are windows issues mostly.
I am working on an es6 bundle PR and these things showed up right away, so it was a good thing. 😜

@erquhart I keep forgetting to ask you if you want me to return those export names back for the backends or follow the same export names as the widgets, so we are ready for them to also have a register name as 'controlComponent'. Although they are not really components as much as control class modules.

Oh yeah, I meant to content on that in the original pull requests, the nomenclature should be changed to be specific to backends as "control" doesn't really apply there.
Awesome to hear you're working on ES6 modules!!!
Sent with GitHawk

Ok, no problem. I wonder if they should follow a common for export like the widgets, so we can make the API the same way?
I will do it first thing in the am.

authors: erquhart talves


text:
Remove immutable from status and auth

Summary
First step on closing #3853
Test plan
Run existing tests

Hm... seems like something's not ok

@erezrokah any idea why e2e tests are failing?

Alright, found the reason


text:
NETOBSERV-1255 Adding Raw Packet export capability as Packet Capture Agent(PCA)

This PR extends the agent to export raw packets through the network.  We call it Packet Capture Agent(PCA).
This functionality is turned off by default. I have added 2 new environments to enable this.
set ENABLE_PCA=true to activate packet cpature.
set PCA_FILTER=[ipproto,port number] to indicate the filter for packet capture,  e.g. export PCA_FILTER=tcp,22
The PCA_FILTER defaults to udp DNS(udp,53). If ENABLE_PCA is true, but filters are not specified, by default it captures DNS packets.
The captured packets are written to a socket in PCAP format.

@msherif1234, for awareness --- the idea behind this PR is to become the basis for both PANO (of course) but also for other Packet level network observability requirements, such as "selective" packet mirroring - functionality such as network tracing etc. So generalization and customization of the content are key ( hope this make sense to you )

I took initial look I see ur created a parallel agent instead of extending the current one after u check the new config knob can u explain the reasoning ?

The current agent works at the level of flows and flow metrics. With PANO, we wish to use a 'packet' as the unit of communication. The struct elements of flow metrics are no longer required. Hope this helps.

I took initial look I see ur created a parallel agent instead of extending the current one after u check the new config knob can u explain the reasoning ?

The current agent works at the level of flows and flow metrics. With PANO, we wish to use a 'packet' as the unit of communication. The struct elements of flow metrics are no longer required. Hope this helps.

yeah I am aware of that as u know I went through the same trouble when I prototyped TCPlife span hook but was able to refactor the current agent so it can handle both probably something we need to poll the team on @jotak @eranra @praveingk wdyt ? will be great if we somehow abstract support to perf map as we have other use cases as well that wanted to use it like the one I mentioned above and more will come and will be nice to share this so all hooks need to generate perf event will reuse

/ok-to-test

I belive yes we can extract out common code and enable one or both functionalities in the agents rather than extending the code which can create code duplication.

Performed first level refactoring.

This branch needs a code review. Waiting for @shach33 to commit final version before providing the review

Added comments. Test

FYI, I've created this gist that showcases how that could work in a kube context: https://gist.github.com/jotak/2a4a5374ee0ab780aeb31bf6ee9a23de
Deployed as a new agent pod (so, independent from the flow collection - which in a sense is better as it offers separation of concerns). By the way, there was no conflicts with the flow-collector agents. The gist illustrates a way to work with the terminal (a next step would be how to integrate that with the console.. let's keep that for later :-) )

/ok-to-test

/ok-to-test

/ok-to-test

/ok-to-test

/ok-to-test

Performed Resource analysis on the PR with 6 different use cases.

base netobserv:main
PR with PCA disabled
PR with PCA enabled - invalid filter
PR with PCA enabled - filter tcp,80
PR with PCA enabled - filter udp,53
PR with PCA enabled - filter tcp,443

Images are annotated with the indices of testcases.

Removed.

The PR is isolated behind a feature gate, not enabled yet from the operator, hence setting labels no-doc / no-qe

Performed Resource analysis on the PR with 6 different use cases.
1. base netobserv:main

2. PR with PCA disabled

3. PR with PCA enabled - invalid filter

4. PR with PCA enabled - filter tcp,80

5. PR with PCA enabled - filter udp,53

6. PR with PCA enabled - filter tcp,443

@shach33 if I understand correctly your perf measurements, the current ebpf agent (without your PR) consumes twice as much memory than with your PR with PCA disabled? (~ 140MB vs 280MB). This would be good news but is quite unexpected, how can we explain that?

Kernel does not support BPF_F_NO_PREALLOC for LRU type maps.

@msherif1234 This is fixed now.

/lgtm
/approve

authors: dushyantbehl eranra jotak memodi msherif1234 shach33

repo: netobserv/netobserv-ebpf-agent


text:
NETOBSERV-1630 allow not for ip filters

Description
Allow not for ip filters

https://grafana.com/docs/loki/latest/query/ip/
Dependencies
n/a
Checklist
If you are not familiar with our processes or don't know what to answer in the list below, let us know in a comment: the maintainers will take care of that.

[X] Is this PR backed with a JIRA ticket? If so, make sure it is written as a title prefix (in general, PRs affecting the NetObserv/Network Observability product should be backed with a JIRA ticket - especially if they bring user facing changes).
[ ] Does this PR require product documentation?

[ ] If so, make sure the JIRA epic is labelled with "documentation" and provides a description relevant for doc writers, such as use cases or scenarios. Any required step to activate or configure the feature should be documented there, such as new CRD knobs.

[ ] Does this PR require a product release notes entry?

[ ] If so, fill in "Release Note Text" in the JIRA.

[ ] Is there anything else the QE team should know before testing? E.g: configuration changes, environment setup, etc.

[ ] If so, make sure it is described in the JIRA ticket.

QE requirements (check 1 from the list):

[X] Standard QE validation, with pre-merge tests unless stated otherwise.
[ ] Regression tests only (e.g. refactoring with no user-facing change).
[ ] No QE (e.g. trivial change with high reviewer's confidence, or per agreement with the QE team).

/ok-to-test

/label qe-approved

/approve

authors: Amoghrd jotak jpinsonneau


text:
CTE ThreatConnect 1.1.0 - Added Support to pull and push Host and Add…

…ress. Added circuit breaker.

Release Notes
Added support to pull and push Host and Address

authors: cwadhwani-crest gitphenom


text:
Examples?

I'd love to see some examples of how to use this!

See https://github.com/netty/netty-incubator-codec-http3/tree/main/src/test/java/io/netty/incubator/codec/http3/example for now

Http3ServerExample can not receive Hello World! from Http3ServerExample

source: gharchive/issue

authors: FireMasterK normanmaurer xtuyaowu


text:
Fix behavior of HttpPostMultipartRequestDecoder for Memory based Factory

Motivation:
When Memory based Factory is used, if the first chunk starts with Line Break, the HttpData
is not filled with the current available buffer if the delimiter is not found yet, while it may
add some.
Fix JavaDoc to note potential wrong usage of content() or getByteBuf() if HttpDatais has
a huge content with the risk of Out Of Memory Exception.
Fix JavaDoc to explain how to release properly the Factory, whatever it is in Memory,
Disk or Mixed mode.
Fix issue #11143
Modifications:
First, when the delimiter is not found, instead of searching Line Break from readerIndex(), we should search
from readerIndex() + readableBytes() - delimiter size, since this is the only part where usefull
Line Break could be searched for.
Second, when a Memory HttpData is created, it should be assigned an empty buffer to be
consistent with the other implementations (Disk or Mixed mode).
We cannot change the default behavior of the content() or getByteBuf() of the Memory based HttpData
since the ByteBuf is supposed to be null when released, but not empty.
Result:
In the tests testBIgFileUploadDelimiterInMiddleChunkDecoderMemoryFactory and related for other modes,
the buffers are starting with a CRLF.
When we offer only the prefix part of the multipart (no data at all), the current Partial HttpData has
an empty buffer.
The first time we offer the data starting with CRLF to the decoder, it now
has a correct current Partial HttpData with a buffer not empty.
Motivation:
Explain here the context, and why you're making that change.
What is the problem you're trying to solve.
Modification:
Describe the modifications you've done.
Result:
Fixes #.
If there is no issue then describe the changes introduced by this PR.

Wait a bit for the user that raised this issue: seems ok but some other points to check still.

@fredericBregier Do you want to mark this PR as a draft until you think it's ready for review?

@chrisvest According to user reporting the issue #11143 it seems OK.
So I believe review can start. ;-)

@chrisvest I will update the PR according to your feedback (and the small issues I've found and detailed in your review).

@chrisvest Done ;-)

Docker issue, not code issue...

Thanks!

authors: chrisvest fredericBregier

repo: netty/netty


text:
Fix unexpected IllegalReferenceCountException on decode multipart request

Fix unexpected IllegalReferenceCountException on decode multipart request. (#8575)
Motivation:
Http post request may be encoded as 'multipart/form-data' without any files and consist mixed attributes only.
Modifications:
Check reference count > 0 added on every InterfaceHttpData before release() call.
Result:
Code does not throw an IllegalReferenceCountException.

@normanmaurer The problem might be introduced by the commit 2b4f667. More detailed in issue #8575 and PR #8964

@ssserj sorry for the long turn-around. Totally missed this one. Can you please sign our ICLA and let me know one done ?
https://netty.io/s/icla ?

@normanmaurer ICLA was signed. Changes was made. But req.refCnt() is not 0. I will be research why?((

@ssserj hmm... let me investigate a bit. I thought it should be 0 as we basically destroyed the decoder + everything that is still stored in there.

@ssserj actually yes your are right 1 is correct as this class does not take over ownership as of today.

@netty-bot test this please

Squashed, cleaned up and cherry-picked as 9e2922b04d2901647b717232ed8fa1745bdbe0df into 4.1

authors: normanmaurer ssserj


text:
WIP: Fix: TestSimpleMetrics failed with vpp-agent v2.3.0

Description

Motivation and Context

How Has This Been Tested?

[ ] Covered by existing integration testing
[ ] Added integration testing to cover
[ ] Tested locally
[ ] Have not tested

Types of changes

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

Checklist:

[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.

blocked by https://github.com/ligato/vpp-agent/issues/1537

repo: networkservicemesh/networkservicemesh

license_source: bigquery


text:
Improve / enhance perf info protocol

This issue is assigned to @crazydevlegend

i'll implement POW-based benchmark to get miner details

authors: GenesisOpp crazydevlegend

repo: neuralinternet/compute-subnet


text:
One shot export [Image classification]

This PR adds support to apply a recipe in one-shot manner while exporting Image classification models
Test plan: Automated test to start with a vanilla(non-quantized) resnet50, export checkpoint in one-shot manner with a quantization recipe and then check the onnx graph for quantized nodes
Note the failing test should pass after #1122 lands

bumping this

authors: bfineran rahul-tuli

repo: neuralmagic/sparseml


text:
Wandb==0.8.16 is incompatible with installed version of graphql-core==0.3.0

See https://github.com/wandb/client/issues/692

fixed

repo: neuromation/cookiecutter-neuro-project


text:
LICENSE is missing

Hiya,
Thanks a lot for your cool work! I'd like to package NIRTorch on conda-forge. For this, I need to know what the LICENSE is, and ideally I need a LICENSE file.
xref: https://github.com/neuromorphs/NIR/issues/74
Best, Tobi

Thanks for the kind words!
The license is the same as in nir, and the missing LICENSE file is clearly an omission from our part. Thank you for bringing attention to this!

Thanks a lot for actioning this, I appreciate it :)

authors: Jegp Tobias-Fischer

repo: neuromorphs/NIRTorch


text:
Can Libra be used in other setting?

Hi, I was wondering if I could use Libra (specifically with setting: de_family = "pseudobulk", de_method = "limma", de_type = "voom") with a scRNA-seq dataset (Seurat) in which I have let's say:

Two groups ("healthy", "disease")
multiple samples belonging to each group ( samples 1-4 are healthy, 5-8 are disease)
each sample is made of different cell types (some specific to a particular sample, but most shared).
No replicates

I have been able to run the example with "hagai_toy" but, when using my data (which has the same metadata structure as the toy example), I keep on getting the following:
Error in intI(j, n = x@Dim[2], dn[[2]], give.dn = FALSE): invalid character indexing
Traceback:

1. run_de(scRNAseq_raw.Seurat, de_family = "pseudobulk",
 .     de_method = "limma", de_type = "voom")
2. pseudobulk_de(input = input, meta = meta, replicate_col = replicate_col,
 .     cell_type_col = cell_type_col, label_col = label_col, min_cells = min_cells,
 .     min_reps = min_reps, min_features = min_features, de_family = "pseudobulk",
 .     de_method = de_method, de_type = de_type)
3. to_pseudobulk(input = input, meta = meta, replicate_col = replicate_col,
 .     cell_type_col = cell_type_col, label_col = label_col, min_cells = min_cells,
 .     min_reps = min_reps, min_features = min_features, external = F)
4. keep %>% map(~{
 .     print(.)
 .     cell_type = .
 .     meta0 = meta %>% filter(cell_type == !!cell_type)
 .     expr0 = expr %>% magrittr::extract(, meta0$cell_barcode)
 .     if (n_distinct(meta0$label) < 2)
 .         return(NA)
 .     replicate_counts = distinct(meta0, label, replicate) %>%
 .         group_by(label) %>% summarise(replicates = n_distinct(replicate)) %>%
 .         pull(replicates)
 .     if (any(replicate_counts < min_reps))
 .         return(NA)
 .     mm = model.matrix(~0 + replicate:label, data = meta0)
 .     mat_mm = expr0 %*% mm
 .     keep_genes = rowSums(mat_mm > 0) > min_features
 .     mat_mm = mat_mm[keep_genes, ] %>% as.data.frame()
 .     mat_mm %<>% as.data.frame()
 .     colnames(mat_mm) = gsub("replicate|label", "", colnames(mat_mm))
 .     keep_samples = colSums(mat_mm) > 0
 .     mat_mm %<>% magrittr::extract(, keep_samples)
 .     return(mat_mm)
 . }) %>% setNames(keep)
5. setNames(., keep)
6. map(., ~{
 .     print(.)
 .     cell_type = .
 .     meta0 = meta %>% filter(cell_type == !!cell_type)
 .     expr0 = expr %>% magrittr::extract(, meta0$cell_barcode)
 .     if (n_distinct(meta0$label) < 2)
 .         return(NA)
 .     replicate_counts = distinct(meta0, label, replicate) %>%
 .         group_by(label) %>% summarise(replicates = n_distinct(replicate)) %>%
 .         pull(replicates)
 .     if (any(replicate_counts < min_reps))
 .         return(NA)
 .     mm = model.matrix(~0 + replicate:label, data = meta0)
 .     mat_mm = expr0 %*% mm
 .     keep_genes = rowSums(mat_mm > 0) > min_features
 .     mat_mm = mat_mm[keep_genes, ] %>% as.data.frame()
 .     mat_mm %<>% as.data.frame()
 .     colnames(mat_mm) = gsub("replicate|label", "", colnames(mat_mm))
 .     keep_samples = colSums(mat_mm) > 0
 .     mat_mm %<>% magrittr::extract(, keep_samples)
 .     return(mat_mm)
 . })
7. .f(.x[[i]], ...)
8. expr %>% magrittr::extract(, meta0$cell_barcode)
9. magrittr::extract(., , meta0$cell_barcode)
10. magrittr::extract(., , meta0$cell_barcode)
11. callGeneric(x, , j = j, drop = TRUE)
12. eval(call, parent.frame())
13. eval(call, parent.frame())
14. x[, j = j, drop = TRUE]
15. x[, j = j, drop = TRUE]
16. subCsp_cols(x, j, drop = drop)
17. intI(j, n = x@Dim[2], dn[[2]], give.dn = FALSE)
18. stop("invalid character indexing")

Thanks in advance!
Francisco

can you please send a sample of your data (e.g., just 100 genes or something)?

Let me start sharing the structure of the input object (data is proprietary, would need to be modified before sharing). Is there something I'm unaware of? [it has exactly the same slots as "hagai_toy"]
Formal class 'Seurat' [package "SeuratObject"] with 13 slots
  ..@ assays      :List of 1
  .. ..$ RNA:Formal class 'Assay' [package "SeuratObject"] with 8 slots
  .. .. .. ..@ counts       :Formal class 'dgCMatrix' [package "Matrix"] with 6 slots
  .. .. .. .. .. ..@ i       : int [1:229698040] 7 17 22 23 25 35 39 42 44 50 ...
  .. .. .. .. .. ..@ p       : int [1:85272] 0 7195 13353 19672 25784 32252 38494 44553 50614 56258 ...
  .. .. .. .. .. ..@ Dim     : int [1:2] 27387 85271
  .. .. .. .. .. ..@ Dimnames:List of 2
  .. .. .. .. .. .. ..$ : chr [1:27387] "MIR1302-2HG" "AL627309.1" "AL627309.3" "AL627309.5" ...
  .. .. .. .. .. .. ..$ : chr [1:85271] "sample1_TATGTTGTCGCTAT-1" "sample1_GTGTTAGGTAACGA-1" "sample1_TCGGCCATGGCTAT-1" "sample1_TTCTTCCTCTCACC-1" ...
  .. .. .. .. .. ..@ x       : num [1:229698040] 4 2 5 14 1 5 3 2 2 8 ...
  .. .. .. .. .. ..@ factors : list()
  .. .. .. ..@ data         :Formal class 'dgCMatrix' [package "Matrix"] with 6 slots
  .. .. .. .. .. ..@ i       : int [1:229698040] 17 27 32 43 55 65 79 82 94 95 ...
  .. .. .. .. .. ..@ p       : int [1:85272] 0 75 1333 1972 2584 3252 3844 4453 5014 5258 ...
  .. .. .. .. .. ..@ Dim     : int [1:2] 27387 85271
  .. .. .. .. .. ..@ Dimnames:List of 2
  .. .. .. .. .. .. ..$ : chr [1:27387] "MIR1302-2HG" "AL627309.1" "AL627309.3" "AL627309.5" ...
  .. .. .. .. .. .. ..$ : chr [1:85271] "sample1_TATGTTGTCGCTAT-1" "sample1_GTGTTAGGTAACGA-1" "sample1_TCGGCCATGGCTAT-1" "sample1_TTCTTCCTCTCACC-1" ...
  .. .. .. .. .. ..@ x       : num [1:229698040] 4 3 9 19 2 6 7 9 2 8 ...
  .. .. .. .. .. ..@ factors : list()
  .. .. .. ..@ scale.data   : num[0 , 0 ]
  .. .. .. ..@ key          : chr "rna_"
  .. .. .. ..@ assay.orig   : NULL
  .. .. .. ..@ var.features : chr(0)
  .. .. .. ..@ meta.features:'data.frame':	27387 obs. of  0 variables
  .. .. .. ..@ misc         : list()
  ..@ meta.data   :'data.frame':	85271 obs. of  6 variables:
  .. ..$ cellID   : chr [1:85271] "sample1_TATGTTGTCGCTAT-1" "sample1_GTGTTAGGTAACGA-1" "sample1_TCGGCCATGGCTAT-1" "sample1_TTCTTCCTCTCACC-1" ...
  .. ..$ nCount_RNA  : num [1:85271] 13801 4167 2314 1322 27396 ...
  .. ..$ nFeature_RNA: int [1:85271] 3564 1920 560 352 4360 1876 2368 2203 5475 1890 ...
  .. ..$ cell_type: chr [1:85271] "CT1" "CT2" "CT3 "CT1" ...
  .. ..$ replicate: chr [1:85271] "sample1" "sample1" "sample1" "sample1" ...
  .. ..$ label    : chr [1:85271] "healthy" "healthy" "healthy" "healthy" ...
..@ active.assay: chr "RNA"
  ..@ active.ident: Factor w/ 1 level "SeuratProject": 1 1 1 1 1 1 1 1 1 1 ...
  .. ..- attr(*, "names")= chr [1:85271] "sample1_TATGTTGTCGCTAT-1" "sample1_GTGTTAGGTAACGA-1" "sample1_TCGGCCATGGCTAT-1" "sample1_TTCTTCCTCTCACC-1" ...
  ..@ graphs      : list()
  ..@ neighbors   : list()
  ..@ reductions  : list()
  ..@ images      : list()
  ..@ project.name: chr "TRIAL"
  ..@ misc        : list()
  ..@ version     :Classes 'package_version', 'numeric_version'  hidden list of 1
  .. ..$ : int [1:3] 4 0 4
  ..@ commands    : list()
  ..@ tools       : list()

P.S. using  de_family = 'mixedmodel', de_method = 'linear', de_type = 'LRT' worked:
Warning message in mclapply(X, FUN, ..., mc.cores = mc.cores, mc.preschedule = mc.preschedule, :
“all scheduled cores encountered errors in user code”
Warning message in p.adjust(p_val, method = "BH"):
“NAs introduced by coercion”
Warning message in p.adjust(p_val, method = "BH"):
“NAs introduced by coercion”

It is difficult to say. If you are getting invalid character indexing are you sure you have no NAs anywhere, whether in the barcodes or in the meta data? If you want to just provide your seurat object with some random sample of 50-100 genes or something I can take a look.

authors: favilaco jordansquair

repo: neurorestore/Libra


text:
xrdp-chansrv SIGSEGV upon certain clipboard content (repeatable)

How to repeat:

Start a KDE session on Ubuntu 18.04.5 LTS (x86_64) with xrdp 0.9.5-2 (from the Ubuntu repos) by connecting from Windows10 via Remote Desktop
In Explorer (Win10), copy a powerpoint file to the clipboard (maybe other non-text stuff triggers the bug, too)
Watch xrdp-chansrv immediately crash on SIGSEGV

After browsing through the already existing bug reports, I find that this one looks similar to others (like for instance #1658), but I did not find exact matches.
Maybe you can find something helpful in the stack trace:
(gdb) attach 2609
Attaching to program: /usr/sbin/xrdp-chansrv, process 2609
[New LWP 2611]
[Thread debugging using libthread_db enabled]
Using host libthread_db library "/lib/x86_64-linux-gnu/libthread_db.so.1".
0x00007f90556de13f in __GI___select (nfds=9, readfds=0x7ffcd7ba3610, writefds=0x7ffcd7ba3690, exceptfds=0x0, timeout=0x0)
    at ../sysdeps/unix/sysv/linux/select.c:41
41      ../sysdeps/unix/sysv/linux/select.c: Datei oder Verzeichnis nicht gefunden.
(gdb) cont
Continuing.

# on Windows, copy the powerpoint file to the clipboard
Thread 2 "xrdp-chansrv" received signal SIGSEGV, Segmentation fault.
[Wechseln zu Thread 0x7f90535f6700 (LWP 2611)]
0x0000562c2da0c4b9 in ?? ()
(gdb) where
#0  0x0000562c2da0c4b9 in ?? ()
#1  0x0000562c2da0ceef in ?? ()
#2  0x0000562c2da11a09 in ?? ()
#3  0x0000562c2da0fd22 in ?? ()
#4  0x0000562c2da093ed in ?? ()
#5  0x00007f90567b45ca in trans_check_wait_objs () from /usr/lib/x86_64-linux-gnu/xrdp/libcommon.so.0
#6  0x0000562c2da088bd in ?? ()
#7  0x00007f90559bf6db in start_thread (arg=0x7f90535f6700) at pthread_create.c:463
#8  0x00007f90556e8a3f in clone () at ../sysdeps/unix/sysv/linux/x86_64/clone.S:95

I did not (yet) gather any other debug output as I had not been aware of the logging capabilities.
After the crash, the xrdp-chansrv remains <defunct>, so apparently xrdp-sesman does not correctly wait(2) for its children -- but that is probably a different (unrelated) issue.

Hi @isteinbach
Thanks for reporting this.
You're on a pretty old version of XRDP at the moment, which is understandable if you're using the LTS version. There have been quite a few changes to the redirector code since then. This particular issue may or may not be fixed in a later version - based on the information we have at the moment I can't say for sure whether this is the case or not.
We can get quite a bit more information from the stacktrace if you're prepared to install the xrdp-dbgsym package from the repositories. That will replace the '??' entries with useful data.
The instructions for installing dbgsym packages are here:-
https://wiki.ubuntu.com/Debug%20Symbol%20Packages
Could you install xrdp-dbgsym and re-post the stacktrace? That will help us decide the best course of action.
Thanks.

Could you install xrdp-dbgsym and re-post the stacktrace? That will help us decide the best course of action.

Sure, here we go:
(gdb) attach 19710
Attaching to program: /usr/sbin/xrdp-chansrv, process 19710
[New LWP 19715]
[Thread debugging using libthread_db enabled]
Using host libthread_db library "/lib/x86_64-linux-gnu/libthread_db.so.1".
0x00007fd45824a13f in __GI___select (nfds=nfds@entry=9, readfds=readfds@entry=0x7fff2d7d9c70,
    writefds=writefds@entry=0x7fff2d7d9cf0, exceptfds=exceptfds@entry=0x0, timeout=timeout@entry=0x0)
    at ../sysdeps/unix/sysv/linux/select.c:41
41      ../sysdeps/unix/sysv/linux/select.c: Datei oder Verzeichnis nicht gefunden.
(gdb) cont
Continuing.

Thread 2 "xrdp-chansrv" received signal SIGSEGV, Segmentation fault.
[Wechseln zu Thread 0x7fd456162700 (LWP 19715)]
xfuse_create_file_in_xrdp_fs (device_id=device_id@entry=0, pinode=pinode@entry=2,
    name=name@entry=0x7fd450033288 "20201014_SomePowerpointFileXy.pptx", type=type@entry=32768) at chansrv_fuse.c:1265
1265    chansrv_fuse.c: Datei oder Verzeichnis nicht gefunden.
(gdb) where
#0  xfuse_create_file_in_xrdp_fs (device_id=device_id@entry=0, pinode=pinode@entry=2,
    name=name@entry=0x7fd450033288 "20201014_SomePowerpointFileXy.pptx", type=type@entry=32768) at chansrv_fuse.c:1265
#1  0x0000555f6613beef in xfuse_add_clip_dir_item (filename=filename@entry=0x7fd450033288 "20201014_SomePowerpointFileXy.pptx",
    flags=flags@entry=0, size=420149, lindex=lindex@entry=0) at chansrv_fuse.c:777
#2  0x0000555f66140a09 in clipboard_c2s_in_files (s=s@entry=0x7fd4500155a0, file_list=<optimised out>) at clipboard_file.c:668
#3  0x0000555f6613ed22 in clipboard_process_data_response (clip_msg_status=<optimised out>, clip_msg_len=<optimised out>,
    s=0x7fd4500155a0) at clipboard.c:1272
#4  clipboard_data_in (s=<optimised out>, chan_id=<optimised out>, chan_flags=<optimised out>, length=<optimised out>,
    total_length=<optimised out>) at clipboard.c:1656
#5  0x0000555f661383ed in process_message_channel_data (s=0x7fd4500155a0) at chansrv.c:633
#6  process_message () at chansrv.c:735
#7  my_trans_data_in (trans=<optimised out>) at chansrv.c:784
#8  0x00007fd4593205ca in trans_check_wait_objs (self=0x7fd4500152f0) at trans.c:408
#9  0x0000555f661378bd in channel_thread_loop (in_val=<optimised out>) at chansrv.c:1132
#10 0x00007fd45852b6db in start_thread (arg=0x7fd456162700) at pthread_create.c:463
#11 0x00007fd458254a3f in clone () at ../sysdeps/unix/sysv/linux/x86_64/clone.S:95

That's great - thanks.
What seems to be happening is that the FUSE code is trying to insert an entry for your PowerPoint file, and the local copy of the FUSE filesystem is not initialised.
The first thing to say is that this code has been extensively reworked in v0.9.10 of XRDP and later, and so it's likely this bug will not appear in later versions.
As far as v0.9.5 goes, the only reason I can see from the code why this might happen is that chansrv can't create the directories $HOME/<FuseMountName> or $HOME/<FuseMountName>/.clipboard where <FuseMountName> is defined in /etc/xrdp/sesman.ini.
If you're interested in digging further, we can get some more info from chansrv which might pin this down. The following would do that.

Make a note of the time in hours, minutes and seconds (HH:MM:SS).
Post the contents of ~/.local/share/xrdp-chansrv.<display>.log after logging in, where <display> is the display number from the $DISPLAY environment variable.
Instigate the crash
Post the output of the command sudo journalctl -S HH:MM:SS where HH:MM:SS is the time from step 1.

Let me know how you'd like to proceed.

As far as v0.9.5 goes, the only reason I can see from the code why this might happen is that chansrv can't create the directories $HOME/<FuseMountName> or $HOME/<FuseMountName>/.clipboard where <FuseMountName> is defined in /etc/xrdp/sesman.ini.

Thanks, Matt, for your support. The FuseMountName is configured as thinclient_drives. When I look into my ${HOME}, the thinclient_drives looks suspicious...
% ls -ln ~
d??????????   ? ?      ?            ?            ?  thinclient_drives

I have no idea what is causing this, but most certainly, this had been the reason for the chanserv problem with writing to the clipboard folder in the FUSE filesystem. After rebooting the server, the thinclient_drives entry looks good, and xrdp-chansrv no longer crashes in the scenario I described in https://github.com/neutrinolabs/xrdp/issues/1700#issue-721171027 -- I can even paste the file from the clipboard to the linux desktop.

That's great - thanks for letting us know.
I've tried and failed to reproduce your situation above. I tried killing chansrv with a SIGKILL to emulate an unexpected and unhandled crash, but this just results in:-
ls -ld ~/thinclient_drives
ls: cannot access '/home/xxx/thinclient_drives': Transport endpoint is not connected

Do you need anything more from me at this stage? Are you happy for me (or you) to close the call? We can always re-open it if the situation re-occurs.

I agree that the issue can be closed. Thanks a lot. The hint with the FUSE mount point lead in the right direction.

repo: neutrinolabs/xrdp


text:
:circus_tent:

Remove unused includes in schema
Schema.h was refactored to be in Valve C++ Guidelines (E.g member names)
CSchemaType was reversed a little bit more
CSchemaClassBinding got IsA function which can use internatl schema function to deterime if current class is has inheritance from other SchemaType
Added SchemaBuiltinType_t
CShemaSystemTypeScope was a little bit reversed \ refactored
CSchemaSystem was a little bit reversed \ refactored
Minor changes in sdk gen
a. Use of std::ranges::find instead std::find
b. Fix #25 by clean_string gh3tt0 c0d3nz

This pull request was made only because of this track


text:
WatchService не следит за изменениями в clientDir

Можно скопировать/изменить любой файл в clientDir во время работы клиента и исключение не вылетит
ArchLinux x64

На вашеq версии баг воспроизводится, при запуске с openjdk 10 beta.

license: WTFPL


text:
大佬请教下，文件都是下的最新的，jellyfin10.9.7版本怎么没变化呀

qua权限也都给了

打扰了 缓存问题emm

repo: newday-life/jellyfin-crx


text:
Underlined text! With intake joystick command

This goes in Intake.java, underneath where it says END AUTOGENERATED CODE, SOURCE = ROBOTBUILDER ID = DEFAULT_COMMAND
public void DriverControl(Joystick stick) {
if (stick.getY() < 0) {
new IntakeBall();
}
if (stick.getY() > 0) {
new EjectBall();
}
if (stick.getY() == 0) {
new IntakeDoNothing();
}

Figured this out. We just needed to map the motor directly to the axis.

repo: newheights/RobotBuilderTest


text:
Allow passing context to apm methods

Hello
As discussed here, this PR adds a version of all APM methods that accepts a context parameter.
I did not add a version for applicationsREST internal functions because, well, they are internal.
Thanks!

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

This looks good to me.  Thank you for putting this together.  To get the tests to pass, make lint is noticing that two methods aren't used any more.  If you remove them, this should be a bit closer.

This looks good to me. Thank you for putting this together. To get the tests to pass, make lint is noticing that two methods aren't used any more. If you remove them, this should be a bit closer.

Oh, okay. I'll remove them.

authors: CLAassistant Tarliton zlesnr

repo: newrelic/newrelic-client-go


text:
AWS EC2 Improvements

This quickstart is lacking in dashboards and alerts and due to the interest in it from our customers we should add a dashboard an alert to make this quickstart higher quality.
Justification
These are pages that are getting traffic or have high search
Acceptance Criteria

[ ] add at least 1 - 2 new dashboards
[ ] add at least 1 - 2 alerts

@jpvajda
This is now done, you can close it.

authors: EmilHammarstrand jpvajda

repo: newrelic/newrelic-quickstarts


text:
Values are not being re-render when default time is changed using time Picker option in the dashboard

Description
I have created a widget using the NRQL status widget option, one of the configurations of this widget is to polling data every 5 seconds and everything is working if the time picker of the dashboard I have attached the widget to, stay with the default option but as long as I change the time picker option to any other value, the data stop the updating process and if you check the network tab under the inspector you will see that the request is happening but the value is not being re-render in the UI.
Steps to Reproduce
Add Data option -> Look into App & Visualisations -> enable the app Status Widget Pack -> select the option NRQL status Widget -> create a query like SELECT count(*) FROM Transaction -> set some critical and warning thresholds -> set polling to 5 seconds -> save the widget into a dashboard -> open the dashboard and see that the widget is being updating with the values every second in the default option of the time picker -> change dashboard's time picker to 5 minutes -> widget stop the re-render process of the value.
Expected Behaviour
Change the dashboard's time picker without affecting the re-render process of the widget's value since the polling should keeps happening every 5 seconds
Relevant Logs / Console output
Your Environment

Browser name and version: Google Chrome Version 121.0.6167.139 (Official Build) (arm64)
Operating System and version: macOS Sonoma 14.3

This should be fixed in the latest release. Feel free to reopen if the issue persists.

Thank you I will let you know in case the issue persists, really appreciate your help 👍🏽


text:
[feature request] support for multiple authors per feed entry

Hello,
the Atom 1.0 specification allows multiple  nodes per entry and it would be nice if newsboat would become aware of this (currently only the first encountered author node is parsed). In particular:

support aggregation of multiple authors in article lists, so that for example "Author A, Author B, Author C" can be displayed in the author column of a multi-author entry. This could for example be achieved by introducing an 'A' flag for articlelist-format (while 'a' continues to only display the first author)
display all authors on separate lines in article view
take into account all  nodes when filtering per author, i.e. the expression is first matched against all  nodes separately and then aggregated by OR ("positive" matching, i.e. =, =~, # etc.) or AND ("negative" matching, i.e. !=, !~, !#)

Hi! What version of Newsboat are you running?

support aggregation of multiple authors in article lists, so that for example "Author A, Author B, Author C" can be displayed in the author column of a multi-author entry

This has been done in Newsboat 2.20.

display all authors on separate lines in article view

People highlight the article header and rely on the "Author: " prefix, so I'm hesitant to introduce such a change. Do you have an argument for it?

take into account all nodes when filtering per author

Well, we already packed all the authors into the "author" attribute, so the best you can do is use regular expressions instead of equality, something along the lines of (^|, )Author name(, |$) (untested!) Would that be sufficient for your needs?

My apologies, I'm actually on Newsboat 2.21 but my "test" feed was fetched a long time ago with a version before support for multiple authors. So a simple feed refresh made all authors appear (and my problems disappear).
Also, keeping all authors on a single line as currently done is actually much better than adding them on separate lines (each line prepended by "Author: ") .
Sorry again!

authors: Minoru herrsimon

repo: newsboat/newsboat


text:
added directory support for cli

I added directory support for cli because we use a lot of file in the same subdirectory (eg: src/ and want to make a single file like application.css and application.min.css) and we felt us naked without this =) If I added this then I think send a pull request.
If the given "file" is a directory
then :syntax option will be skipped and guessing from file extension.
Example: sass src/ assets/application.css
I know not a perfect and nice additional but works. If i will have time (and someone else don't make it) I make a better solution. If this solution is not ok then please tell me what is the problem.
Thanks: Balazs Nadasdi

this command creates separated files
$ sass --update src:assets
$ ls assets
one.css   three.css two.css
and this patch make just one file (or one output stream to stdout)
$ sass src assets/app.css
$ ls assets
app.css
Nothing more =) So i don't need to merge these files with an other app or i don't need to make a main.sass what contains only @imports's

Use partials.

Oh yeah before i modified this our base.sass was 63 lines thats only @import's
@import 'page'
@import 'main'
@import 'editor'
@import 'enterpage'
@import 'galleries'
@import 'images'
@import 'members'
@import 'messages'
And i we created a new file we need to add this it to somewhere. =/ So partial is NOT a nice solution for it. =) because a little annoying.

Handling many imports has been discussed to death elsewhere. The upshot is that we don't support it because it makes the ordering of the imports unclear, but if you really want it you can define a custom importer.

Thx =)


text:
Work around PosgreSQL index issue

Update model to use an index on a checksum of the large summary text field.
Otherwise there can be times when this exceeds PostgreSQL limits.
See https://github.com/nexB/vulnerablecode/pull/632#discussion_r823700889
Reference:  https://github.com/nexB/vulnerablecode/issues/650
Signed-off-by: Tushar Goel tushar.goel.dav@gmail.com

When I run ./manage.py makemigrations command again, it does nothing, md5 hash computation part is not getting stored as a migration file.

Why do we have three fields in the model (summary_md5, package affected_md5, references_md5) separately. IMHO we can have one field and hash all the fields inside it. This will help us not to store useless data in the database
instead using unique_together , use unique=True .
ex:
md5_field = models.CharField(
                    max_length = 32,
                    unique = True
                    )

def save(self, *args, **kwargs):
        self.md5_field = hashlib.md5(self.summary.encode("utf-8") + self.affected_packages.encode("utf-8") + ...
        super(Advisory, self).save(*args, **kwargs)

repo: nexB/vulnerablecode


text:
🚀 3단계 - 즐겨찾기 기능 구현

기능 요구사항

[x] 즐겨찾기 생성 기능 구현
[x] 즐겨찾기 조회 기능 구현
[x] 즐겨찾기 삭제 기능 구현

리팩토링 항목

테스트 외부 API 요청 패키지 분리
JWT 토큰 검증 (AuthenticationPrincipalArgumentResolver) 책임연쇄패턴 적용

피드백 반영

Favorite 저장(save()) Transaction 설정
엔티티 기본 생성자 접근 지정자 수정 (protected)
테스트 응답값 검증 시 for/if 로직 제거, equals/hashCode 를 사용한 객체 비교로 수정

repo: next-step/atdd-subway-favorite


text:
[step3] 로깅, 모니터링 리뷰 요청드립니다 !

안녕하세요 !
AWS 사용이 익숙지 않아서 끙끙 앓으면서도 미션 진행하다보니 결국 세번째 미션까지 오고 말았네요 !!
항상 긍정적인 피드백 주신 덕분에 힘 내서 미션 수행할 수 있었던 것 같습니다 ! 💪😊
이번에도 잘 부탁드립니다!! 좋은 밤 되세요 ! 🙇🏻♂

이번 미션에서는 다른 미션 대비 비교적 많은 이야기를 나누기 제한적이었던 것 같아서 아쉽네요 ㅎㅎ
미션 진행하시느라 고생 많으셨습니다 🙇🏻♂

repo: next-step/infra-subway-monitoring


text:
🛑 Nextpost.tech is down

In 63fc64f, Nextpost.tech (https://nextpost.tech) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Nextpost.tech is back up in 8005560.

repo: nextpost-tech/status


text:
Automated GenBank ingest failing

Automated GenBank ingest failures

https://github.com/nextstrain/ncov-ingest/actions/runs/10817236034
https://github.com/nextstrain/ncov-ingest/actions/runs/10836359393

Both workflows failed during transform-rki-data-to-ndjson. There's a verbose stack trace from pandas, but the final error is ValueError: Index SEQUENCE.ID invalid.

RKI announced in https://github.com/robert-koch-institut/SARS-CoV-2-Sequenzdaten_aus_Deutschland/issues/59 that some variables have been renamed. This particular error comes from SEQUENCE.ID being renamed to igs_id, but we should verify other columns that are used in transform-rki

repo: nextstrain/ncov-ingest


text:
[BUG] - Select not working correctly with numbers

NextUI Version
2.3.6
Describe the bug
Select component seems to accept numbers, and given this type definitions
export type Key = string | number;

export type Selection = 'all' | Set<Key>;

onSelectionChange?: (keys: Selection) => any,

it indeed seems possible. However, when you provide numbers as items of Select, in onSelectionChange, keys is always Set<string>
Your Example Website or App
No response
Steps to Reproduce the Bug or Issue
<Select
          label="Test"
          items={[{ label: "2024", value: 2024 }]}
          onSelectionChange={(v) => {
            console.log(v);
          }}
        >
          {(item) => (
            <SelectItem key={item.value} value={item.value}>
              {item.label}
            </SelectItem>
          )}
        </Select>

when selecting a item, log statement prints {'2024'} (but should {2024})
Expected behavior
value in onSelectionChange should be Set<number>
Screenshots or Videos
No response
Operating System Version
macOS
Browser
Chrome

Additionally, when you provide string[] to selectedKeys, e.g. [2020, 2021], then it cannot find these keys, therefore the Select dropdown does not show ticks next to selected items.

Thanks for the issue!
This issue stems from React converting the key to a string.

Note that if you do specify a custom key on each Item using the key prop, React will convert the provided key to a string. As a result, all the collection component's key-related event handlers and props (e.g. onSelectionChange, selectedKey(s)) will expect and return strings.

See: https://react-spectrum.adobe.com/react-stately/collections.html#unique-keys
While it can be resolved by assigning an id to items as shown below, the current situation requires a key, so some adjustments need to be made. I'm currently working on this fix.
let [years, setYears] = useState([
  {id: 1, value: 2024},
  {id: 2, value: 2025},
  {id: 3, value: 2026}
]);

<Select items={years}>
  {item => <SelectItem>{item.value}</SelectItem>}
</Select>

@ryo-manba thanks for taking a look. However after installing 0.0.0-canary-20240520130845 (which should include your PR), the result is exactly the same as I had initially

authors: jegork ryo-manba

repo: nextui-org/nextui


text:
Integrate Ribo-TISH

This PR wires in RiboTISH, and adds associated config, documentation etc. I also updated a bunch of modules, which is what makes the number of changed files a bit scary.
This is the first ribo-seq specific functionality, so there's also the wiring from the sample sheet to separate riboseq from rnaseq from TI-seq.
There's still some work to do on wider documentation etc, will do in a separate PR before release.
PR checklist

[ ] This comment contains a description of changes (with reason).
[ ] If you've fixed a bug or added code that should be tested, add tests!
[ ] If you've added a new tool - have you followed the pipeline conventions in the contribution docs
[ ] If necessary, also make a PR on the nf-core/riboseq branch on the nf-core/test-datasets repository.
[ ] Make sure your code lints (nf-core lint).
[ ] Ensure the test suite passes (nextflow run . -profile test,docker --outdir <OUTDIR>).
[ ] Usage Documentation in docs/usage.md is updated.
[ ] Output Documentation in docs/output.md is updated.
[ ] CHANGELOG.md is updated.
[ ] README.md is updated (including new tool citations and authors/contributors).

Thanks for the review @maxulysse

repo: nf-core/riboseq


text:
Empty output for BAM+rMats

Description of the bug
Hi, I passed the test samples. But when I run on my samples (BAM+rMats). I got empty rMats output. There was no error reported, showing completed.
Command used and terminal output
$ nextflow run software/rnasplice -profile docker --input nf_samplesheet.csv --contrasts nf_contrastsheet.csv --genome GRCh38 --outdir myenv

Relevant files
nextflow.log
nextflow.config.log
System information
No response

Hey @Qingru44004
Please could you provide the stdout and stderr files produced by the various rMATS processes? Once we have those, we can examine what might have gone wrong.
Best,
James

Stale issue, please re-open if you are still experiencing problems.

repo: nf-core/rnasplice


text:
Implement doublet detection methods

Description of feature
SOLO has already been implemented. The following have been implemented in YASCP and could easily be adopted:
### Tasks
- [ ] https://github.com/nf-core/scdownstream/issues/15
- [ ] https://github.com/nf-core/scdownstream/issues/16
- [ ] https://github.com/nf-core/scdownstream/issues/17
- [ ] https://github.com/nf-core/scdownstream/issues/18
- [ ] https://github.com/nf-core/scdownstream/issues/19

Should probably discuss with the authors of YASCP first, which ones worked best for them

doubletFinder came out best in this benchmark and also worked well for me anecdotically.

authors: grst nictru

repo: nf-core/scdownstream


text:
HomeKit/Home App doubles renamed (Fibaro) devices

Hi. I am using your great homebride with Fibaro Plugin on a Raspberry Pi. Works amazingly. Thanks a lot!
Just I have the following issue: When I rename a device in the Fibaro control center web interface, hombrigde adds it as a new accessory (at it's next start) - without removing the existing predecessor, which is known under the old name. In result the iOS 10 Apple Home App shows two (or after repeated renaming even more) entries for the same device. Both/all entries are working.
I have learned I should delete the ~/.homebridge/persist folder. This indeed resolves to problem - while all other existing entries in the Home app loose connection to their relevant devices too. As I cannot (or don't know how to) delete entries in Home app, I have to delete the whole house and add it from the scratch, which is very time consuming with 50 devices.
Is doubling the device entries a bug or user failure?
How to remove single devices or entries from Homebride (from the cached accessories)?
Thanks for your advice. Christian

@Krillle , the current version of the plugin enable the updates of the devices in Home Center.
Simply perform the update in Home Center and restart homebridge.

Yes, thank you. I am using it already, and very happy with that!
Btw, there seems to be an issue with the Fibaro RGBW controller: The color stored in HomeKit scenes is not always set correctly. I have the impression the white channel is not always set.
Thanks again for your great plugin, Christian
Am 08.05.2017 um 17:56 schrieb ilcato <notifications@github.commailto:notifications@github.com>:
@Krilllehttps://github.com/Krillle , the current version of the plugin enable the updates of the devices in Home Center.
Simply perform the update in Home Center and restart homebridge.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/nfarina/homebridge/issues/936#issuecomment-299908892, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ACuwBnlRKYgsXiINXkzoj3F0gVjrQMJ9ks5r3zshgaJpZM4K3fhE.

@Krillle , just published version 1.1.2 of the Fibaro Home Center plugin that should fix the RGBW issue.
Do not create issues in the homebridge repository if it is specifically related to the Fibaro Home Center plugin, use this.
And close this issue please.

authors: Krillle ilcato

repo: nfarina/homebridge


text:
[BUG] OS-specific file linking

Describe the bug
Depending on OS (e.g. Win vs. Mac) files are linked differently, e.g.:

Unix: mystudy/isa.study.xlsx
Win: mystudy\isa.study.xlsx

To Reproduce
Steps to reproduce the behavior:

Add study or assay to an ARC on win or mac

Related to https://github.com/nfdi4plants/ISADotNet/issues/48

fixed in #219

authors: Brilator HLWeil


text:
No such file or directory: 'requirements.txt'

When following the steps in Getting Started, I see this error when I run pip install python-lambda:
Collecting python-lambda
  Using cached python-lambda-0.5.1.tar.gz
    Complete output from command python setup.py egg_info:
    Traceback (most recent call last):
      File "<string>", line 1, in <module>
      File "/private/var/folders/br/bx8tyrg91fx1pgqwvnwc315w0000gn/T/pip-build-l7DEkV/python-lambda/setup.py", line 16, in <module>
        pip_requirements = [str(r.req) for r in requirements]
      File "/Users/tmh/virtualenvs/pylambda/lib/python2.7/site-packages/pip/req/req_file.py", line 84, in parse_requirements
        filename, comes_from=comes_from, session=session
      File "/Users/tmh/virtualenvs/pylambda/lib/python2.7/site-packages/pip/download.py", line 425, in get_file_content
        'Could not open requirements file: %s' % str(exc)
    pip.exceptions.InstallationError: Could not open requirements file: [Errno 2] No such file or directory: 'requirements.txt'

    ----------------------------------------
Command "python setup.py egg_info" failed with error code 1 in /private/var/folders/br/bx8tyrg91fx1pgqwvnwc315w0000gn/T/pip-build-l7DEkV/python-lambda/

sorry about that, pushed a bad release. reverted

I think that this problem is because the version available on pypi (pip) is 0.5.0, and the current is 0.7.0.
https://pypi.python.org/pypi/python-lambda/

Uploaded latest version to pypi.

authors: jmeekhof nficano nielsonsantana tmhammer

repo: nficano/python-lambda

license: isc


text:
[BUG] Empty streams for some videos

For some links, the streams property is empty, and the get_audio_only() method returns None. I've identified a few such links that seems to be reproducible. Here's my test code that shows the problem:
from pytube import YouTube

FAILING = [
    "https://www.youtube.com/watch?v=L6nrMAFCzhg",
    "https://www.youtube.com/watch?v=T8de-31Hu7g",
    "https://www.youtube.com/watch?v=Y08HWyVSMxg",
    "https://www.youtube.com/watch?v=WchEW9LkK94",
]

OK = [
    "https://www.youtube.com/watch?v=7lYMmW2kPpQ",
    "https://www.youtube.com/watch?v=7eVQHt690nk",
    "https://www.youtube.com/watch?v=4bscOLDng0o",
]

for fail in FAILING:
    yt = YouTube(fail)
    stream = yt.streams.get_audio_only()

    # that's unexpected, stream shouldn't be None
    assert stream is None
    assert len(yt.streams) == 0

for ok in OK:
    yt = YouTube(ok)
    stream = yt.streams.get_audio_only()

    # same code, different links - all good
    assert stream is not None
    assert len(yt.streams) > 0

I've tested on freshly created virtualenv and the newest pytube. Same error on Windows and Ubuntu (WSL).

For some links, the streams property is empty, and the get_audio_only() method returns None. I've identified a few such links that seems to be reproducible. Here's my test code that shows the problem:
from pytube import YouTube

FAILING = [
    "https://www.youtube.com/watch?v=L6nrMAFCzhg",
    "https://www.youtube.com/watch?v=T8de-31Hu7g",
    "https://www.youtube.com/watch?v=Y08HWyVSMxg",
    "https://www.youtube.com/watch?v=WchEW9LkK94",
]

OK = [
    "https://www.youtube.com/watch?v=7lYMmW2kPpQ",
    "https://www.youtube.com/watch?v=7eVQHt690nk",
    "https://www.youtube.com/watch?v=4bscOLDng0o",
]

for fail in FAILING:
    yt = YouTube(fail)
    stream = yt.streams.get_audio_only()

    # that's unexpected, stream shouldn't be None
    assert stream is None
    assert len(yt.streams) == 0

for ok in OK:
    yt = YouTube(ok)
    stream = yt.streams.get_audio_only()

    # same code, different links - all good
    assert stream is not None
    assert len(yt.streams) > 0

I've tested on freshly created virtualenv and the newest pytube. Same error on Windows and Ubuntu (WSL).

I am facing the same issue, but I need an urgent fix, as i am using this version for my project, Can you provide examples of a few links that work? so i can atleast demo it.

Been having the same issue for a while now, some links I found that don't work:
https://www.youtube.com/watch?v=u5CVsCnxyXg
https://www.youtube.com/watch?v=GPUg7n8-M6o
https://www.youtube.com/watch?v=9YgmMJJ34k4
https://www.youtube.com/watch?v=vnLAa6_hB9A
https://www.youtube.com/watch?v=Mw5mAozjC6M
https://www.youtube.com/watch?v=-hptmburu3U
https://www.youtube.com/watch?v=lflsO7pxSE0
https://www.youtube.com/watch?v=mvJjmWTg7Qo
https://www.youtube.com/watch?v=DCi9I7ylt14
All of the above can be reproduced, might be helpful with looking for a pattern.

hi @aklajnert, just to confirm, when you run
import pytube
print(pytube.__version__)

does it show 10.1.0? Those videos are working just fine for me:

@tfdahlin It's 10.0.0 actually, but that's  the one that gets installed as the newest one:
> pip install --upgrade pytube
Requirement already up-to-date: pytube in /venv/pytube/lib/python3.8/site-packages (10.0.0)

Yeah, that's where the problem is. I'm working on getting something set up so that the pypi repository gets updated more quickly. I can make changes to the repo itself, but don't have access to pypi, so sometimes it takes a while for my bugfixes to get pushed to pypi.
The command you should be using to install pytube from source is python -m pip install git+https://github.com/nficano/pytube. This should have been in the instructions when you made this ticket, but maybe I can make that more clear.

We're hoping that we'll have the process for updating pypi set up sometime this week. The PR is already written, Ronnie just needs to review it when he has the time and set a couple of things up.

@KrishnarajT and @PC-02 I believe your problems are also fixed in the github repository, just not the pypi repository. You should also run python -m pip install git+https://github.com/nficano/pytube to install from source, and see if that fixes the problem for you.

@tfdahlin yes, the links work perfectly fine now, I had noticed the latest version in the docs, but didnt think of installing from source. Thanks a ton!

@tfdahlin The links work now, looks like it was an issue caused by using 10.0.0. Thank you so much!

Thanks @tfdahlin - that makes sense. Anyway, the projects that use pytube as dependency cannot just set its version to master, and need to wait for it to be released. But good to know that it is already fixed.

@tfdahlin - any ETA when the new version will be released to PyPI?

Hopefully this weekend, depends on when @RONNCC has the chance to either review my PR or push an update manually. He said he'd look into it today, but obviously depends on if he has the time to

@tfdahlin - anything I can do to help with the release? It's a bit unpleasant situation as the package cannot be used as a dependency when not released on PyPi.

@aklajnert unfortunately, not really at the moment. @RONNCC is the one who has to update the pypi package, and I think he's on vacation and not looking at github currently. I was supposed to be given access to pypi by the owner, but that also hasn't happened yet. I think this is just going to be stuck for a little while longer at this point, likely until after January 1st

@tfdahlin  @aklajnert  sorry my bad here -- done.

It's a bit unpleasant situation as the package cannot be used as a dependency when not released on PyPi.

This shouldn't be the case -- can you explain a bit more around your problem?

@RONNCC it has to do with libraries that have pytube as dependencies. There isn't really a good way to tell setuptools to install a package from github if it already exists in pypi as far as I can tell.

@tfdahlin  https://stackoverflow.com/questions/32688688/how-to-write-setup-py-to-include-a-git-repo-as-a-dependency ?

I kept trying to get that to work, but it only wanted to install from pip, and I can't work out why

I'll admit I gave up pretty early because I wasn't sure if I was diving down a rabbit hole unnecessarily

@tfdahlin https://stackoverflow.com/questions/32688688/how-to-write-setup-py-to-include-a-git-repo-as-a-dependency ?

I've found that also, but it seems that you can't release your package on PyPI with dependencies as links according to this

@aklajnert  makes sense - seems PyPI essentially restricted it since you're essentially running code from anywhere

authors: KrishnarajT PC-02 RONNCC aklajnert tfdahlin

repo: nficano/pytube

license: Unlicense


text:
Fix compatibility with React 16.9

Fix for #465, #426 and #413 by updating react-side-effect to v2.0.0 since it's now using UNSAFE_componentWillMount instead of componentWillMount.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

@rifaidev you need to change react and react-dom dependencies in package.json to ^16.3 or they don't match the requirements of react-side-effect. This'll be more than a patch update then too, probably a major.

@wheeler makes sense, however looks like the maintainers have abandoned the library! 🤷♂

It would be nice to merge this PR please @tmbtech . It is really confusing to hack yarn.lock to make a warning disappear.

@cwelch5 Could you look at this?

#473 is now merged and Travis CI is building again. Maybe add an empty commit to trigger CI build again for this PR (or rebuild manually if possible)?

@rifaidev #473 is now merged and Travis CI is building again. Maybe add an empty commit to trigger CI build again for this PR (or trigger manually if possible)?

I think maintainers can manually trigger it.

Please merge this PR it become a need of hour to get our console free from such warning messages.

I think maintainers can manually trigger it.

@rifaidev Would you mind try pushing an empty commit? Maybe the maintainers won't pick this up until all checks are green and this could speed up merging.

Would you mind try pushing an empty commit? Maybe the maintainers won't pick this up until all checks are green and this could speed up merging.

I just did, and still failing.
Actually I highly recommend switching to https://github.com/staylor/react-helmet-async instead.

This is such a popular repository, and I am amazed with the fact that an issue since last year still have no resolution even after a potential fix given in current PR. Seems authors are no more interested in this community work. I am disappointed to drop such a fantastic pacakge from my project I will add a plan to drop it or will use some alternative of this 😞

authors: CLAassistant Remzi1993 abumalick hitendramalviya rifaidev verheyenkoen wheeler

repo: nfl/react-helmet

license: mit


text:
[neon-identity-service]: STS based on IdentityServer4

Build a secure token service based on IdentityServer4 using a custom Postgres/Yugabyte database.  We really need this for a sister project and this will likely become important for neonKUBE and neonCLOUD in the future as well (in addition to other future sister projects).
NOTE: The initial development will be happening in the sister project's repo for convenience and will be relocated to here after it's stabilized.

[x] Implement SchemaManager for both Yugabyte/Postgres and Cassandra to handle database creation, intiailization, and upgrades.

[x] Port the standard T-SQL scripts  generated by Entity Framework to Postgres/Yugabyte scripts.

[ ] Use Yuga CREATE SEQUENCE caching for identity columns?

[ ] Add static NeonIdentityService.Service.InitializeDatabase(string connectionString) method that actually performs the database creation/updates.  This will need to be called externally (e.g. by the neon-cluster-manager for production or by a CodeFixture for test).

[ ] Neon.Common needs to extend Assembly with a GetResourceFileSystem(string name) method.  This will return a new ResourceDirectory object that includes methods like GetDirectories(...) and GetFiles(...) that emulate a primitive file system based on resources embedded in the assembly.  We'll use this now for loading embedded SQL script files and later when we migrate cluster setup to Neon.Kube.

[ ] Implement the IdentityServer4 Storage interfaces.

[ ] Implement the IProfileService interface for end-user data.

[ ] Initialization via YAML configuration settings.

[ ] Login UX?

[ ] Admin UX (link)?

References:
https://www.youtube.com/watch?v=nyUD-CeBSiE&t=5493s
https://docs.identityserver.io/en/dev/intro/big_picture.html
https://openid.net/connect/
https://id4withclients.readthedocs.io/en/latest/id4/ID4Database/DatabaseGenScriptID4.html
https://id4withclients.readthedocs.io/en/latest/id4/ID4Database/StaticDataID4.html
http://docs.identityserver.io/en/latest/reference/aspnet_identity.html
https://github.com/IdentityServer/IdentityServer4/tree/main/src
https://www.scottbrady91.com/Identity-Server/Creating-Your-Own-IdentityServer4-Storage-Library
https://anexinet.com/blog/user-management-with-ids4-and-a-custom-database/
https://github.com/IdentityServer/IdentityServer4/issues/2870

Putting this on the backlog after cancelling the Wildfire project.

We've abandoned this entirely in favor of KeyCloak.

repo: nforgeio/neonKUBE


text:
BREAKING: Refactor assembly: Neon.Kube.Service --> Neon.Service

As part of #937, we're going to rename the Neon.Kube.Service assembly to Neon.Service and rename a couple of types within:

KubeService --> NeonService
KubeServiceStatus --> NeonServiceStatus

We're also going to rename Neon.Xunit.KubeServiceFixture --> NeonServiceFixture
You'll need to update your projects as required.
The reason is that Neon.Kube.Service really isn't neonKUBE cluster specific; this class is more generic than that.  This was different back in the old neonHIVE days.

DONE for v2.2.0


text:
Feature Request: value property should bind with ngModel

Current behavior
It seems that value-property-name is not binding with ngModel rather the whole model is binded.
Expected/desired behavior
When defining value-property-name one would only want to retrieve a property from a model but not the whole model. May be value-property-name should be changed to track-by-property and value-property-name would define if the whole model or a property value binds with ngModel
Usecase
Proper binding on initialization from property value instead of whole model.
Example Plunker in which by value-property-name is set to id. Setting ngModel to a default id should work.

I am not sure about this requirement. From my experience, almost all time, I wanted to have the full object instead of only value,  e.g. Product, Item, or Page, instead of product name, item name and page name.

Let me give an example where this is required:
Lets say there are some hotel rooms whose model would contain properties: RoomId, Type, Cost, Capacity, isAlloted
User Model contains properties: UserId, Email, Age, Name
App may have already downloaded a list of users earlier.
Through auto complete we want to assign a predefined user to a room. On selection I only need to send UserId with RoomId and not whole User model.
Moreover, ui-select, angular-ui too supports this feature for angular 1.

What if I need a whole model as a value?
e.g. {id: 1, name: "My Name", address: "xxx", ...}
display-property-name: name
What if I need whole model, not only id, as the value?

May be value-property-name should be changed to track-by-property and value-property-name would define if the whole model or a property value binds with ngModel

Not fully following,. can you give me an example with this  {id: 1, name: "My Name", address: "xxx", ...}

When value-property-name = "id" the whole model is binded.
However, to chose only one property of the model to bind. This is what I propose:

Changing only attribute name of value-property-name to track-by-property and leaving it's implementation intact.
Create a new attribute value-property-name.

As per point 2, the implementation is changed in such a way that if a user wants to get only name from {id: 1, name: "My Name", address: "xxx", ...}.
He will set:
value-property-name = "name", track-by-property = "id", display-property-name = "name"
If the user wants to get the whole model:
track-by-property = "id", display-property-name = "name"

Absolutely, that is why the current implementation of value-property-name remains as it is just under a different name track-by-property. Hope I am making sense here.

What if user does not specify track-by-property, should it be id in default?

NOTE:From the code, valuePropertyName is not much in use

Yes if user doesn't specific track-by-property it will take id as default. Incase of an array of say string (single property) there is no need for track-by-property, value-property-name and display-property-name

NOTE:From the code, valuePropertyName is not much in use

Agreed, it can be even argued that it should be removed altogether.

@svitlychnyi
How about this?

introducting select-value-of instead of track-by-**.

because we are not tracking any.

If omitted, select the whole model, the element of list

If given, return the key of the row as the value

display-property-name and value-property-name

It is only used for default list formatting.
We can rename it or

list-value-property-name
list-display-property-name

Or, remove both, and introuce simple list formatter

e.g. list-formatter="(id) name"

Introducing select-value-of instead of track-by-**.
Perfect! Would be great to have this in place.

Any timeline on when this feature would be implemented?


text:
concat options format

I'm seeing an error when running with the default grunt config. The concat:css options are:
concat: {
  css: {
    // add your css files over here to concatenate all css files
    // let's save our site users some bandwith
    files: {
      src: ['public/vendor/bootstrap/dist/css/bootstrap.min.css', 'public/css/styles.min.css'],
      dest: 'public/css/app.styles.min.css'
    }
  }
},

When I run grunt, I get:
Warning: Unable to write "src" file (Error code: EISDIR). Use --force to continue.

I believe the issue is with the files options it should be like this:
concat: {
  css: {
    // add your css files over here to concatenate all css files
    // let's save our site users some bandwith
    files: {
      'public/css/app.styles.min.css': ['public/vendor/bootstrap/dist/css/bootstrap.min.css', 'public/css/styles.min.css'],
    }
  }
},

Created pull request: https://github.com/ngenerio/generator-express-simple/pull/20

:+1:

authors: jessbowers ngenerio

repo: ngenerio/generator-express-simple


text:
Enable ginkgolinter and fix finding

Proposed changes
Enable ginkgolinter in .golangci.yaml, and fix its findings.
The fixing was done by running the ginkgolinter cli. All the finding except for one, are categorized as style findings, e.g. Expect(err).To(BeNil()) instead of Expect(err).ToNot(HaveOccurred()).
The linter also found one real issue in tests/suite/longevity_test.go, where there was a check (Expect) with no assertion (To(...)). That was fixed manually from:
Expect(framework.WriteContent(resultsFile, "\n## Traffic\n"))

to
Expect(framework.WriteContent(resultsFile, "\n## Traffic\n")).To(Succeed())

Closes #1678
Checklist

[x] I have read the CONTRIBUTING doc
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have checked that all unit tests pass after adding my changes
[ ] I have updated necessary documentation
[x] I have rebased my branch onto main
[x] I will ensure my PR is targeting the main branch and pulling from my branch from my own fork

Release notes
None

@nunnatsa, thanks for adding this! This will be very helpful 😄

authors: kate-osborn nunnatsa

repo: nginxinc/nginx-gateway-fabric


text:
How can I get correct values of the global variables in this.atomColor function?

The version of NGL I used : ngl-2.0.0-dev39.
// Record resno of hovered atom
var hoveredResno = null;

// NGL Stage
var stage = new NGL.Stage("viewer", { backgroundColor: 'white', tooltip: true });

// My color scheme
var mySchemeId = NGL.ColormakerRegistry.addScheme(function (params) {

    this.atomColor = function (atom) {
        // Default color : Gainsboro
        var color = 0xDCDCDC;

        // If the atom is hovered, color it red.
        // * but `hoveredResno` is always null.
        if (atom.resno == hoveredResno) {
            color = 0xFF0000;
        }

        // Return hex
        return color;
    };
});

stage.signals.hovered.add((pickingProxy) => {
    if (pickingProxy && pickingProxy.atom) {
        // Set hoveredResno
        hoveredResno = pickingProxy.atom.resno;
    }
});

// stage.loadFile(.....)

My problem is that : In this.atomColor, I can't get the new or correct value of the global variable hoveredResno, it is always null.
Is this problem related to function context ?
Thank you.

Hmm, that looks okay to me, assuming this is copy/pasted then the scopes look fine - you're accessing hoveredResno which isn't declared in inner scopes so should look up the chain of enclosing scopes. What do you see if you set a breakpoint (or console.log) just after the line where you set hoveredResno?

I saw hoveredResno was set to  a new atom.resno (in hovered event).
// Set hoveredResno
hoveredResno = pickingProxy.atom.resno;
console.log(hoveredResno); // a new resno.

Sorry - revisiting and just realised that atomColor is not re-evaluated on hover...
You will, for now at least, need to listen to hover events and update the colorScheme (or reset the selection on a representation). I've got a change in a private version which I will write up and make a PR for that should make this a lot easier

it exposes a hook so that colormaker code can trigger a representation (color) update.

authors: TsungMingYangNTUT fredludlow

repo: nglviewer/ngl


text:
feature(transpiler): add key to transpile method

PR Checklist
Please check if your PR fulfills the following requirements:

[x] The commit message follows our guidelines: https://github.com/ngneat/transloco/blob/master/CONTRIBUTING.md#commit
[ ] Tests for the changes have been added (for bug fixes / features)

I may need help with testing since this feature doesn't add any functionality directly, but it does not break any existing tests afaik

[x] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[x] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[ ] Other... Please describe:

What is the current behavior?

https://github.com/ngneat/transloco/issues/568
The TranslocoTranspiler.transpile does not give the key of the current translation.
Issue Number: 568
What is the new behavior?
The TranslocoTranspiler.transpile now accepts the key as a parameter, and the TranslocoService.translate method now passes in the key.
Does this PR introduce a breaking change?
[ ] Yes
[x] No

Other information
This is one of my first contributions to an open source project, so I expect that I will need to make some changes.

@jacksonopp fix the types please and force push all the changes in one commit made using yarn commit/npm rnu commit.

Thank you for your patience as I work thru my first OSS contribution by the way!

@jacksonopp please note that the CI is failing

@shaharkazaz
Looking into the unit tests
Looks like we're seeing a peer dependency mismatch around rxjs. I think running an npm install should fix this, and would involve committing a change to package-lock.json. Is that okay with you?
Looking into the e2e tests, I'm seeing the following failures:

locale.spec.ts is failing because it is displaying the wrong unix date, and it fails regardless of whether or not we are using the custom transpiler. I was playing around with changing my time zone, and that seemed to affect the displayed output. Adding timeZone: 'UTC' as a parameter to the translocoDate pipe normalizes the timezone, and seems to solve this issue. I'm not sure what has changed to cause this. Does this sound correct to you?

full-cycle.spec.ts is failing because the custom transpiler just returns the value without doing any transpiling. Removing the custom transpiler from the providers array seems to fix this issue. Since we've updated the documentation site with the new key input, I think its fine if we do not include this in the transloco playground. It wasn't there before anyway.

Thoughts?

@jacksonopp

You didn't make any change on the package so I'm not sure why would that change anything 🤔 But let's try it, make sure you have node 16+ and npm 8+.
Let's try to remove the custom compiler from the playground.
I don't think there was any change in the locale area, and last couple of commits passed, I'll take a look when I get time if the issue persists, anyways you didn't cause the issue 🙂

@shaharkazaz

Updated package-lock.json. I used node v16.15.1 and npm 8.11.0
Removed the custom transpiler from the playground
Running through the specs, I'm now seeing the unit tests around locales failing for the same reason as the e2e tests, but sounds like you're going to investigate it.

@shaharkazaz Any update on this? Looks like all specs are passing now.

@jacksonopp sorry man I'm swamped, I'll get to it as soon as I have some free time to do one last review 🙏

@shaharkazaz I understand you are probably busy, but any more updates?

@jacksonopp Sorry man for disappearing, had a lot of stuff going on and just couldn't find the time.
If you'd like we can pick this up again.

@shaharkazaz All good. I've updated my fork to match master here.

@jacksonopp nice work on this man. I hope to see you contributing more 😄

authors: jacksonopp shaharkazaz

repo: ngneat/transloco


text:
.select like in baobab?

Hello.
Would it be hard to implement a .select like in Baobab?
Like you have a store with this state:
items: [
subitems: []
]

Then i would be able to create an observeable like this:
items: Observable<Item[]>;

constructor(private _store: Store<AppStore>)
{
// Only react to state changes on items.subitems
this.items = this._store.select('items.subitems');
      }
)

wouldn't this work?
this.items = this._store.select('items').map(res => res.subitems);

Yes, this would work.
But if I understand it right it will be always triggered when something inside the items array was changed.
For instance:
items: [
subitems: [],
isFetched: true
]

When I change the isFetched property to false will the subItems observeable be triggered or not?

you can use the selector option of .select() : store.select(state => state.items.subitems)

Awesome :).
Thank you @robwormald !

authors: datoml fxck robwormald

repo: ngrx/store


text:
Vertical cross-section

Create a vertical cross-section by adding a line to plot. Can use the existing index retrieval method in select region and then have the cross-section plot pop up in additional tab.

I agree with with the functionality, but I don't think we should use the the existing method, but rather just drawing the line in the axis and making the conversion from position in index to plot in a component for it self. This is mainly for performance and more flexibility. I already worked with cross section in an other software.
Should such a function of plotting a vertical section be make part of pyart? @jjhelmus?

I'd be open to including a method for extracting a vertical section from a radar volume to Py-ART.  Using a KD-Tree for gate lookup would be recommended.

Essentially, wouldn't this just require refactoring the pyart grid methodology, for cross sections with arbitrary cardinal direction orientations?  I suppose that great circle calculations would technically be appropriate, especially for longer cross sections.

@swnesbitt Yes, although I think it might be desired to do nearest neighbor interpolation instead of the weighted interpolation that the grid method currently uses.  Having a generic function in Py-ART which takes gates from one or more radars and a list of point and returns the fields values for those points using a weighted/nearest/k-nearest interpolator would be really useful in a number of cases.  The current gridding function does some of that but some work would be needed to expand the functionality.

@gamaanderson @jjhelmus I've been working on a KD-Tree for another application and I agree that this probably would be a good implementation, especially for a polar coordinate data space. I'm finding that the KD-Tree can take a long time to build, just as a heads up (or I'm just messing up the programming).
Let's discuss implementation a little more. Building this in Py-ART and then calling in ARTview seems the idea. I agree with that plan. Where should this go @jjhelmus? If I can get my code working nicely, I can help get this up and running.

I'd put it in the pyart/map directory for the time being.  Once the details get fleshed out it may get moved but that seems reasonable for now.

So here we are again: good vs fast. A reformulation of the current map_to_grid or map_gates_to_grid should do a nice work. This would be fast enought if you follow the principe of drawing a line and then pressing a button for drawing. But not fast enought if you want to make the drawing in real time (while drawing), this is not only something that would look nice, but also the fast response create a 3 dimensional feelling.

authors: gamaanderson jjhelmus nguy swnesbitt

repo: nguy/artview


text:
make Vradar optional for Menu and Vlims optional in Display

Issue #17
Turn some V variable into optional arguments, those are initialized internaly.
In execute_two.py I accessed Vradar using MainMenu.Vradar instead creating function  Main.Menu._getVradar(). I don't think we need a get function here, do you agree?

Just ran the code and maybe I failed to notice this before but self.fieldnames is not defined anywhere so there is a error in _check_default_field in plot.py.  If you are going to make more changes soon would you stick this in the init?  Otherwise, I'll do at a later date.  The code still functions as is so I consider this minor for the time being.

I will add that to my next PR, some time later today

Thanks!

I'm not getting this error, self.fieldnames is being defined in self.NewRadar() that should be executed priory to any call of self._check_default_field().
There is another error "  'Display' object has no attribute 'ax'  " that was caused by this PR, however I already have a solution using issue #19.
Anyway, the code is working, I will keep an eye open to see if this turn back up and how to make  Display Start-up less vulnerable to such things.

Hmm weird.  I still get the above stated error, but I can't reproduce the 'ax' error.
Will have to keep an eye on this.

authors: gamaanderson nguy


text:
How to unsubscribe to valueChanges

I'm submitting a ...  (check one with "x")
[ ] bug report => search github for a similar issue or PR before submitting
[ ] feature request
[x ] support request

Current behavior
I have read in many posts that instead of using the onChanges hook for when a field value changes, it is suggested to do the following:
hooks: {
onInit: function( field) {
field.formControl.valueChanges.subscribe(v => {
console.log('valueChanges', v);
});
}
}
If done this way, what is the proper way to unsubscribe?
Expected behavior

Minimal reproduction of the problem with instructions

What is the motivation / use case for changing the behavior?

Please tell us about your environment:

Angular version: 8.0.0

Browser: [all

Language: [all]

Node (for AoT issues): node --version =

check the following example https://ngx-formly.github.io/ngx-formly/examples/other/cascaded-select, which unsubscribe on ngOnDestroy

authors: aitboudad silvl

repo: ngx-formly/ngx-formly


text:
@ngx-formly/ionic incompatible with @ionic/angular 8.3.0

Description: The @ngx-formly/ionic version 6.3.7 is not compatible with the latest version of @ionic/angular (8.3.0). The following error occurs when trying to use the library:
./node_modules/@ngx-formly/ionic/fesm2020/ngx-formly-ionic-radio.mjs:95:259-280 - Error: export 'RadioValueAccessor' (imported as 'i1') was not found in '@ionic/angular' (possible exports: ActionSheetController, AlertController, AngularDelegate, AnimationController, BooleanValueAccessor, Config, DomController, GestureController, ION_MAX_VALIDATOR, ION_MIN_VALIDATOR, IonAccordion, IonAccordionGroup, IonActionSheet, IonAlert, IonApp, IonAvatar, IonBackButton, IonBackdrop, IonBadge, IonBreadcrumb, IonBreadcrumbs, IonButton, IonButtons, IonCard, IonCardContent, IonCardHeader, IonCardSubtitle, IonCardTitle, IonCheckbox, IonChip, IonCol, IonContent, IonDatetime, IonDatetimeButton, IonFab, IonFabButton, IonFabList, IonFooter, IonGrid, IonHeader, IonIcon, IonImg, IonInfiniteScroll, IonInfiniteScrollContent, IonInput, IonInputPasswordToggle, IonItem, IonItemDivider, IonItemGroup, IonItemOption, IonItemOptions, IonItemSliding, IonLabel, IonList, IonListHeader, IonLoading, IonMaxValidator, IonMenu, IonMenuButton, IonMenuToggle, IonMinValidator, IonModal, IonNav, IonNavLink, IonNote, IonPicker, IonPickerColumn, IonPickerColumnOption, IonPickerLegacy, IonPopover, IonProgressBar, IonRadio, IonRadioGroup, IonRange, IonRefresher, IonRefresherContent, IonReorder, IonReorderGroup, IonRippleEffect, IonRouterOutlet, IonRow, IonSearchbar, IonSegment, IonSegmentButton, IonSelect, IonSelectOption, IonSkeletonText, IonSpinner, IonSplitPane, IonTab, IonTabBar, IonTabButton, IonTabs, IonText, IonTextarea, IonThumbnail, IonTitle, IonToast, IonToggle, IonToolbar, IonicModule, IonicRouteStrategy, IonicSafeString, IonicSlides, LoadingController, MenuController, ModalController, NavController, NavParams, NumericValueAccessor, PickerController, Platform, PopoverController, RouterLinkDelegate, RouterLinkWithHrefDelegate, SelectValueAccessor, TextValueAccessor, ToastController, createAnimation, createGesture, getIonPageElement, getPlatforms, getTimeGivenProgression, iosTransitionAnimation, isPlatform, mdTransitionAnimation, openURL)
Steps to Reproduce:
Install @ngx-formly/ionic version 6.3.7 and @ionic/angular version 8.3.0.
Run the application.
Your Environment

Angular version: 18.2.3
Formly version: 6.3.7
ionic/angular: 8.3.0,

Duplicate of #3902

authors: KeyTurns aitboudad


text:
How to update a section of the form model

Hello,
I recently updated a project from v5 to v6 (latest).  We have a feature that allows a user to load a large subset of the model. This new model subset most likely will not match the shape of the section it is replacing.  Our current v5 code used the Formly reset function to replace the model using the spread syntax to get a new model address. This works for most of the replacement, but form arrays that had two items continue to have two items even when the new model only has one. I’ve seen suggestions to use the setValue method on the field formControl to update the model value, but that doesn’t work when the shapes don’t match. I’d appreciate any ideas. Thank you.

With the upgrade to angular usage there was an update that no longer allows simply calling firm control.reset() to set a value to the empty state
Reset now goes back to the INITIAL value that existed when the component was instantiated.
In order to alleviate this, in my custom types I've created I simply set the model internals to what I want them to be and then call the build function ( now a public API) on that field.
If it is a problem with only your array type itself then you would need to use your custom array types private build function if extending the formlyarraytype.
I can't give examples at the moment but hopefully this helps.
If not please setup a stackblitz sample so we can see exactly what you mean.

Thanks for the response. I’m trying to get a minimal Stackblitz worked out to provide a solid example.
I do wonder how you would set the internal model in the custom component if the model property is read only?
thanks again, and I hope to get a Stackblitz up tomorrow.

Fieldconfig.parent.model['yourChildKey'] =[]
From here you can either rebuild the parent or just the child field and it should work if my memory serves correct.

authors: kenisteward soundoff


text:
How to handle import TranslateModule.forChild in module when creating a standalone component in an NgModule Application

Hi,
I was wondering how you should handle an import of TranslateModule.forChild in a module when adding a single standalone component in an NgModule based angular project
I would like to use TranslateModule.forChild() with isolate:true in Pricing component
I don't have bootstrapApplication method as I see in another support request, so I don't understand if is feasible.
app.module
@NgModule({
    declarations: [
        AppComponent
    ],
    imports: [
        HttpClientModule,
        BrowserAnimationsModule, // required for ng2-tag-input
        CoreModule,
        LayoutModule,
        RoutesModule,
        LaddaModule.forRoot({
            style: "zoom-out",
            //spinnerSize: 10,
            spinnerColor: "white",
            spinnerLines: 12
        }),

    ],
    providers: [
        TranslateStore,
    ],
    bootstrap: [AppComponent]

})

routes,ts
{
        path: "pricing",
        loadComponent: () =>
          import("./admin/pricing/pricing/pricing.component").then(
            (m) => m.PricingComponent
          ),

      },

empty pricing component
import { Component } from '@angular/core';
import { CommonModule } from '@angular/common';
import { SharedModule } from '../../../../shared/shared.module';

@Component({
  selector: 'app-pricing',
  standalone: true,
  imports: [SharedModule,],

  templateUrl: './pricing.component.html',
  styleUrls: ['./pricing.component.scss']
})
export class PricingComponent {

}

I have tried
{
        path: "pricing",
        loadComponent: () =>
          import("./admin/pricing/pricing/pricing.component").then(
            (m) => m.PricingComponent
          ),
          providers: [importProvidersFrom(TranslateModule.forChild({loader: { provide: TranslateLoader, useFactory: createTranslateLoader, deps: [HttpClient] }, isolate: true }))],

      },

......

export function createTranslateLoader(http: HttpClient) {
  return new TranslateHttpLoader(http, './assets/i18n/pricing/', '.json');
}

But the translate pipe is not found.
Could you please help me?
Thanks

Hi,
I have almost same problem. For your translate pipe error, you have to import TranslateModule in your standalone component. But, then another problem appears. Translation are not loaded for your standalone component. Did you resolve this ?

No...unfortunately no one answered.
I would like to introduce gradually the standalone component, but This situation prevents me from doing so.

I think this project went silent time to look for alternatives

@pretext-mats which alternatives do you recommend ?

@pretext-mats which alternatives do you recommend ?

I have not got the faintest idea :-) Been using this translate since pre ng 2 ... may be I should not be looking for an alternative quite yet....

I don't know the contents of your SharedModule, but if you're missing the translate pipe in your PricingComponent, then you need to add an import for TranslateModule, e.g.
@Component({
  selector: 'app-pricing',
  standalone: true,
  imports: [SharedModule, TranslateModule /* added import */],

  templateUrl: './pricing.component.html',
  styleUrls: ['./pricing.component.scss']
})
export class PricingComponent { }

The TranslateModule consists of component / pipe / directive exports in addition to the providers. Before migrating your component to standalone, you might have declared the component in the same module where TranslateModule.forChild() had also been imported, and therefore the pipe was also available to the component.

Trust me it is not the case. We have tried that already still no joy - many other issues pop up

@8th-block  this one?
https://angular.dev/guide/i18n

@8th-block this one? https://angular.dev/guide/i18n

Yes

Our fork of ngx-translate now works with Standalone Components without any issues. See https://ngx-translate.org/getting-started/migration-guide about how to migrate to it.

authors: 8th-block CodeAndWeb Fiehra chidav77 pretext-mats zazics

repo: ngx-translate/core