De novo opening and closing reads file a lot

https://github.com/rmcolq/pandora/blob/5356466d645172171df24009612153b65716176c/src/denovo_discovery/candidate_region.cpp#L126-L149
The above function is run once for every CandidateRegion
https://github.com/rmcolq/pandora/blob/5356466d645172171df24009612153b65716176c/src/map_main.cpp#L416-L426
If the reads file is large, this will take a considerable amount of time!

One obvious solution is to just create a file handle outside of the for loop and have CandidateRegion::generate_read_pileup take a file handle instead of a path. But I am worried this could cause a lot of problems when it comes to trying to multithread @leoisl ?

Now that I look at it, there is no reason that block https://github.com/rmcolq/pandora/blob/5356466d645172171df24009612153b65716176c/src/map_main.cpp#L416-L426
cant be moved to here
https://github.com/rmcolq/pandora/blob/5356466d645172171df24009612153b65716176c/src/map_main.cpp#L388-L392
and then we run de novo for all of the candidates within a pan-node.
However, even with this change it still doesn't fix the fact that the read file is being opened for ever run of de novo...

I agree that multi-threading over all candidate regions would be probably faster than multi-threading over pan-nodes. Is the processing of different candidate regions (even if they come from the same pan-node) independent? If so, it is easy to multi-thread. Otherwise, we should take care on how to synchronise.
Regarding the several and several traversals of the reads file, I am interested to know if this is indeed a bottleneck. For me, it seems so, but can you confirm it (either by your experience or some log)? In toy examples, this will probably not be an issue, as reads files are very small, so we would need to check if this is an issue in real datasets. Anyway, from what I understood, each read in CandidateRegion::read_coordinates has to be found in the reads file:
https://github.com/rmcolq/pandora/blob/5356466d645172171df24009612153b65716176c/src/denovo_discovery/candidate_region.cpp#L132-L134
I think this can be potentially slow, since it translates into many file read operations (e.g. if we want to access the last read of the file, we have to read and skip all the other reads). I think positioning the file pointer directly where the read is, is way faster. I think we can accomplish this with http://www.htslib.org/doc/faidx.html . We can use their C API to do so: http://samtools.sourceforge.net/samtools/faidx/index.html . Like this we just move the file pointer to the read, and do only one read operation per read. We will still need to open the file for each candidate region, but I believe that using the index will solve the performance issues...
What do you think? Do you see a better solution?

I agree that multi-threading over all candidate regions would be probably faster than multi-threading over pan-nodes. Is the processing of different candidate regions (even if they come from the same pan-node) independent? If so, it is easy to multi-thread. Otherwise, we should take care on how to synchronise.

Yes each candidate region (should) be independent, even if it's from the same pan-node.

Regarding the several and several traversals of the reads file, I am interested to know if this is indeed a bottleneck. For me, it seems so, but can you confirm it (either by your experience or some log)? In toy examples, this will probably not be an issue, as reads files are very small, so we would need to check if this is an issue in real datasets. Anyway, from what I understood, each read in CandidateRegion::read_coordinates has to be found in the reads file:
https://github.com/rmcolq/pandora/blob/5356466d645172171df24009612153b65716176c/src/denovo_discovery/candidate_region.cpp#L132-L134
I think this can be potentially slow, since it translates into many file read operations (e.g. if we want to access the last read of the file, we have to read and skip all the other reads). I think positioning the file pointer directly where the read is, is way faster. I think we can accomplish this with http://www.htslib.org/doc/faidx.html . We can use their C API to do so: http://samtools.sourceforge.net/samtools/faidx/index.html . Like this we just move the file pointer to the read, and do only one read operation per read. We will still need to open the file for each candidate region, but I believe that using the index will solve the performance issues...
What do you think? Do you see a better solution?

I believe this is to do with the way @rmcolq handles fastaq files. Ie the read id is effectively it's line number in the file. As if your reads are sorted you should only need to parse the fastaq file once.

I believe this is to do with the way @rmcolq handles fastaq files. Ie the read id is effectively it's line number in the file. As if your reads are sorted you should only need to parse the fastaq file once.

Indeed, in this function, pandora parses the fastaq file once per candidate region, even if we have several read_coodinates, since these are sorted by id.  But do you think even this is slow?  If I understood correctly, if we have 5 read_coordinates for a candidate region, and the maximum read_coordinate.id is 50000, then we will have to read 50000 reads from the fastaq file. 49995 of them will be read, but discarded here:
https://github.com/rmcolq/pandora/blob/5356466d645172171df24009612153b65716176c/src/fastaq_handler.cpp#L148-L153
I guess instead of this we just position the file pointer where the 5 reads to be read are and do only 5 read operations. But to do this, we need the index. If parsing the fastaq as it is done now is not slow, then  we can leave it like this.

Yes, I see what you mean. We had previously spoken about changing our fast{aq} handling process but never really got around to doing it.  https://github.com/rmcolq/pandora/issues/74#issuecomment-432183107
Maybe now is the best time given you are in the process of trying to speed everything up. And your suggestion of htslib obviously makes a lot of sense.
Best to get input from @rmcolq and @iqbal-lab though.

My memory is that when we were writing this bit of code with Robyn, we had the file handle open and passed it as an argument to extract pileups for each region. Because the regions were sorted by read, we only opened the file once and only looped through the file once.
To parallelize, one option is to divide the regions into chunks, and open the file once per chunk of regions. Or the other option is to use a library for fastaq handling. My thoughts about on the external library: please check if it can handle the prg-like fasta first. And chat to Zam about the pros/cons of including yet another external library as a dependancy.

various comments here

do we know this opening and closing is actually slow? i completely get and agree it's inefficient to open/close too much, but i'm wondering if we empirically know it is hurting us
indexing the reads makes a lot of sense; htslib is obviously solid but also changes reasonably often and might be a maintenance burden there,plus rachels question of whether it will allow prg-like fasta
Having said all that I generally favour fixing correctness first and then speed. For the de novo component, i think we're pretty sure the actual path traversal stuff is solid, but we don't understand where we lose  our 50% of recall. currently we know it's not due to explosion of too many paths to enumerate, so possibly/probably due to our decision process for where to do de novo. So my temptation right now is to suggest we hold off addressing this issue til that is dealt with, both in terms of pandora priorities and leandro priorities. ie if leandro was out of work and looking for something to do, and michael was still where he is now, i'd suggest we see if there is any mileage in collaborating on that. Maybe there isn't, but that's what i'd ask.

is there a reason we can't load up the slice of reads that overlap the gene into memory, and then deal with this all in memory? one of the things we've proposed doing, to enable compare to work across a cluster with one gene per node, is at quasimap time, to dump a file of the subreads that overlap each gene. if we did that, then during de novo you'd just pull up the reads for that gene. does that sound viable?

More context to this issue
Running pandora map with --discover on a 3.3Gb nanopore fastq file takes a very long time. The long time is likely, mostly attributable to the opening and closing of the file. An example from the log file
[2019-06-28 21:46:09.390015] [0x00002b66e78653c0] [debug]   Open fastaq file/hps/nobackup/research/zi/mbhall/Pandora_variation/Analysis/ST38/ST38.fastq.gz
[2019-06-28 21:46:23.531611] [0x00002b66e78653c0] [debug]   Close fastaq file

This means it is taking ~15s to open the read file for each candidate region. The consequence of this is that if we run denovo discovery on 10,000 candidate regions (a likely scenario in previous samples) then 41.6 hours of pandora map is spent just opening the read file over and over again.

is there a reason we can't load up the slice of reads that overlap the gene into memory, and then deal with this all in memory? one of the things we've proposed doing, to enable compare to work across a cluster with one gene per node, is at quasimap time, to dump a file of the subreads that overlap each gene. if we did that, then during de novo you'd just pull up the reads for that gene. does that sound viable?

Having all the reads in memory could be pretty dangerous no? I mean some of my nanopore fastq files are > 20Gb.

the whole point is you slice the reads where they overlap a gene, and write that to file. then you only load into memory the bits of the reads that overlap the gene. so thats ~1kb long, 100x, not much.
This also then allows massive parallelisation by gene - at least in terms of RAM, each node in a cluster could do 10,000 samples all on the same gene. granted, this naive implementation would mean each node would be opening 10,000 files, so maybe we would batch genes, but the basic idea is you allow parallelising by slicing reads where they overlap genes

it's like loading the gene pileup into RAM

the other thing one could do is this. When quasimapping a sample, you write all these 5000 files.  then at the end, you basically cat them into one file, and store an index which says reads 1..99 are for gene 1, 100-134 are for gene 27,...etc, allowing you later on to seek to the right place to pull out precisely the reads you want. you then delete your 5000 files.
The only difference  to the above is that you then have one file per sample, not 5000 per sample.
So this is  better for the filesystem in terms of numbers of files. On the other hand when you have 5000 nodes each processing one gene for 10,000 samples, each node has  needs to open all of the 10000 per-sample fastqs, each pulling out their own gene.
ok i take it all back, maybe best to have loads of files in a tree of directories.

I tried to find other alternative implementations, and I only see one, which is easier to implement, consume less disk space, and probably faster if a SSD is used, but slower if HDD is used. However, I don't think we should take the decision hoping the user would store files in a SSD...
As such, I think the best solution is Zam's idea of slicing the reads where they overlap a gene, and writing this to a tree of directories. I'd propose we go for this implementation, if you agree.

It does mean we (at least temporarily) double disk usage. I think we live with it. But we could contemplate discarding qualities. Also could think of cutting the sliced reads at low quality bases so they don't add noise to de novo

Agreed, I think removing the noise can be indeed good to improve results!
+1 for this idea!
I think we should prioritise this, hoping to get this solved next week!


[Lexical][CI] Add more packages for size limit regressions check

WHAT
Add @lexical/react package for size limit regressions check, follows up on #6137
WHY

While supporting changes to making Lexical compatible to React 19, there was a size regression in the Meta internal build but wasn't detected in the external build.

adding @lexical/react package to size limit check to capture the size limit changes on each PR

TEST PLAN
sahejk@sahejk-mbp lexical % npx size-limit
✔ Adding to empty webpack project
✔ Running JS in headless Chrome

  lexical
  Size:         22.5 kB with all dependencies, minified and brotlied
  Loading time: 440 ms  on slow 3G

  @lexical/rich-text
  Size:         33.25 kB with all dependencies, minified and brotlied
  Loading time: 650 ms   on slow 3G

  @lexical/plain-text
  Size:         33.21 kB with all dependencies, minified and brotlied
  Loading time: 649 ms   on slow 3G

  @lexical/react
  Size:         150.38 kB with all dependencies, minified and brotlied
  Loading time: 3 s       on slow 3G

sahejk@sahejk-mbp lexical %

You should exclude this file path from triggering extended test runs in the future


Added XC20P encryption algorithm

What kind of change does this PR introduce? (check at least one)

[ ] Bugfix
[x] Feature

Added XC20P encryption algorithm.

[ ] Code style update
[ ] Refactor
[x] Other, please describe:

Added pycryptodomex dependency.

Does this PR introduce a breaking change? (check one)

[ ] Yes
[x] No

[x] You consent that the copyright of your pull request source code belongs to Authlib's author.

Thanks. I would like to make pycryptodomex as an optional dependency. I'll merge the PR at first.

Here is my updates: https://github.com/lepture/authlib/commit/f27b4dabe4843d334570d25af011ca236c8af501


Feature Request: Add display_light control properties to DEVICE_AIR_CONDITIONER

It seems this is already enabled for humidifier and purifiers, would be great to control that for bedroom ACs as well.
Being able to disable sound would be great as well, but I didn't see this resource on any other device.

Thank you for your feedback. We will review to add the feature and let you know if the feature is available.


Add ability to customize probes

What does this PR do?
Liveness and readiness probes cannot be changes. So this PR will add ability customize params of probes and we are able to  disable probes.
Motivation
I have specific case, i need to disable liveness probe and do more aggressive readiness probe. Now, i can't customize these probes.
More

[x] Yes, I updated the chart version

Additional Notes

Hello @osipovdaniil and thanks for your interest in Traefik,
Allowing the configuration of the probes seems a legit use case, but why should we allow disabling the probes? Could you give more details about your use case?

@kevinpollet Hi! I think we can do without the ability to disable probes.
I updated my PR. Please check again


Provide package.name as CARGO_PKG_NAME

In the same spirit of not duplicating information as CARGO_PKG_VERSION and CARGO_PKG_AUTHORS, could cargo provide the package.name field as CARGO_PKG_NAME?  That would avoid duplication when providing the package name and version in help text or similar.
Along the same lines, I'd find it quite helpful to have CARGO_PKG_DESCRIPTION and CARGO_PKG_HOMEPAGE.

Seems fine to me!

Still a work in progress but here is the PR- https://github.com/rust-lang/cargo/pull/2523

@alexcrichton  please close this issue.

Thanks!


Add version command

adds a new version command, this should also work with go get.
Sample output -
$ kubicorn version
Kubicorn version:  {"Version":"0.0.003","GitCommit":"457a1e9618a96671fc3c54d16e40b8c5ef6d5220\n","BuildDate":"2017-08-09 19:19:01.508842266 +0000 UTC","GOVersion":"go1.8.1","GOARCH":"amd64","GOOS":"darwin"}

Can you please update the headers here? make update-headers should do it

@kris-nova Hi I updated the headers, looks likemake update-headers doesn't work on OSX, because of sed command. I will send another PR to fix it.

@prateekgogia I left some comments. :)
I would like to hear what does @kris-nova think as well. Also does Kris thinks it's possible. :)

This looks fine - we will probably be changing the version scheme anyway once somebody tackles #14


Sweep

Actions (click)

[ ] ↻ Restart Sweep

❌ Unable to Complete PR
Please add more details to your issue. I need at least 20 characters to generate a plan. Please join our Discord server for support (tracking_id=bd662b0073)
For bonus GPT-4 tickets, please report this bug on Discord (tracking ID: bd662b0073).

🎉 Latest improvements to Sweep:

New dashboard launched for real-time tracking of Sweep issues, covering all stages from search to coding.
Integration of OpenAI's latest Assistant API for more efficient and reliable code planning and editing, improving speed by 3x.
Use the GitHub issues extension for creating Sweep issues directly from your editor.

💡 To recreate the pull request edit the issue title or description.
This is an automated message generated by Sweep AI.


Use Airnode logging utilities

https://api3dao.atlassian.net/browse/BEC-336
Use Airnode logging utilities. For now I've just replaced the console.log calls with a logger with hard-coded configuration which prints timestamp and log level alongside with the message.

@Siegrift Can you open this against beacon-updates branch and switch all the logging there as well?

Can you open this against beacon-updates branch and switch all the logging there as well?

Sure, but what about other open PRs? I think we can just wait until they are merged.

Can you open this against beacon-updates branch and switch all the logging there as well?

Sure, but what about other open PRs? I think we can just wait until they are merged.

The Terraform one won't have much code inside if any and the configuration file placement one has just some changes in TS configuration. The beacon updates one is the only one with actual code and a lot of log statements that's why I'm asking for it.


Paella7 fix trimming url params

Your pull request should…

[x] have a concise title
[ ] close an accompanying issue if one exists
[ ] be against the correct branch
[ ] include migration scripts and documentation, if appropriate
[x] pass automated tests
[x] have a clean commit history
[x] have proper commit messages (title and body) for all commits

Have you tried how it behaves with Wowza stream security?

Have you tried how it behaves with Wowza stream security?

No. We don't have Wowza stream security. But I assume it will work properly. Since all requests are made in the same way as with the normal video (without soft-trimming).


fixed #64: passing on rake to TMG_CRU_2017 and TMG_SUB_2017

closes #64

I'm a bit surprised that you're still using the old UCERF3 ScalingRelationships interface, is there a reason you're using it rather than your own RupSetScalingRelationship implementation?

We've implemented new scaling relationships in our repo, but some of our rupture sets still refer to this one. So serialisation would be easier if this one was fixed.

OK, I'll merge but please ween yourself when you can!


uv to replace poetry

uv is a relatively new python package manager with great performance. Langflow has a lot of dependencies that result in slow poetry install / lock times which uv can alleviate.
This WIP PR gets the core features like build and unit tests working.
Would be great to get nightlies working with uv.
package publish with uv is pending.
The current config supports both poetry and uv. We can remove poetry as we gain confidence with uv.

I'm having a look at make lint which isn't working


Augment ConcatenateExpression to allow array, list and map parameters

I was adapting my templates when I found a use case where fusing two maps would've been handy. I remember there was something in the + operator, but it actually only adds Lists.
I'm not keen on overloading +, so what about adding the support to ~? Want a PR?

To be honest, I kind of regret implementing string concatenation with the ~ character. It just isn't a very java-like operator.
I think a "map fuse" feature should be implemented as a filter instead. The ability to chain filters together reminds me of java 8 streams and I'd like to head more in that direction. The twig library uses a merge filter but I'd be open to other names.

A quick google search shows merge to be a common name for that operation. I'm really bad at making up names, but other options are... hmmm... combine, concat, fuse, copy... I'm good with merge, anyway.


FlashList errs when rotating the device with dynamic numColumns and ListEmptyComponent

Current behavior
I have the following code:
import { Text, useWindowDimensions } from 'react-native';

import { FlashList } from '@shopify/flash-list';

function App(): JSX.Element {
    const { width } = useWindowDimensions();
    const columnsCount = width / 100;

    const EmptyState = () => {
        return <Text>Wow such empty!</Text>;
    };

    return (
        <FlashList
            data={[]}
            renderItem={(_) => {
                return <></>;
            }}
            numColumns={Math.floor(columnsCount)}
            estimatedItemSize={200}
            ListEmptyComponent={EmptyState}
        />
    );
}

export default App;

I run it on an android real tablet device, specifically a Galaxy Tab S5e (SM-T720)
I rotate the device.
There is an error displayed when I do it:
 ERROR  TypeError: Cannot read property 'y' of undefined

This error is located at:
    in ProgressiveListView (created by StickyContainer)
    in RCTView (created by View)
    in View (created by StickyContainer)
    in StickyContainer (created by FlashList)
    in FlashList (created by App)
    in App
    in RCTView (created by View)
    in View (created by AppContainer)
    in RCTView (created by View)
    in View (created by AppContainer)
    in AppContainer
    in example(RootComponent), js engine: hermes
 ERROR  TypeError: Cannot read property 'y' of undefined

This error is located at:
    in ProgressiveListView (created by StickyContainer)
    in RCTView (created by View)
    in View (created by StickyContainer)
    in StickyContainer (created by FlashList)
    in FlashList (created by App)
    in App
    in RCTView (created by View)
    in View (created by AppContainer)
    in RCTView (created by View)
    in View (created by AppContainer)
    in AppContainer
    in example(RootComponent), js engine: hermes

I would expect to have the view properly rotated instead.
To Reproduce
Use react-native to initialize an empty project, add flashlist to it, and replace the default App code by the one I posted above.
Platform:

[X] iOS
[X] Android

It also occurs on iPad (be sure to display the app in the whole screen, https://stackoverflow.com/questions/74907554/ipad-pro-not-showing-full-screen-content-in-react-native-xcode-14-0-1)

Environment
I'm on @shopify/flash-list": "^1.6.3"

i got same issue pls help me way for resolve it

Same issue here

I don't reproduce anymore with FlashList 1.6.3 and react-native 0.73.6

I was able to fix the issue by changing the FlashList key when the list is empty and the number of columns changes. Note that this will completely destroy the list and re-create a new one.
// change the key when the list is empty & numCols changes
const listKey = data?.length === 0 ? `empty-list-${numCols}-cols` : "normal-list"

// be sure to include key={listKey} when rendering
return (
   <FlashList
      key={listKey}
      ListEmptyComponent={
        <MyEmptyComponent />
      }
      numCols={numCols}
      data={data}
   />
)

It's a bit hacky, but it works 🤷♂


0.0.12 release is not in the repo

The version string in setup.py is still 0.0.11 on master. It would be nice to keep the repo updated.

Fixed in b2514f97d07fde9b2b6087af77d30b4c4929ee4c


Is there any plan regarding gathering a set of rules like \begin{mathpar}?

Right now I'm doing this and I think this is cringe:

Thank you for your feedback.
Can you be more precise about you're demand ?
Is it about the ways to typesetting a lot of inference rules in a column ?
What does mathpar does that this package does not ?
Indeed a lot of the work of this package is to reimplement behaviour of LaTeX package.

Is it about the ways to typesetting a lot of inference rules in a column ?

Like it separates the rules with some spaces and if there's two many rules in a line, it breaks into multiple lines.

Yes Indeed it is a feature that is in our to-do list.
It is not very easy to do, but we are thinking about it. Our goal is to stay in a 1 file project, because we are trying to do a simple as possible package.
Why it is not yet implemented :

I don't know how to get the space that a rule "have". I want to have a width that tells me : "this is bigger than the width of the page".

context and style has changed in 0.11 of typst so I have not dig how to do that

We have focus on tree of rules to typeset right instead of perfectly typeset Big rules with a lot of premisses
I don't have yet a good idea of an algorithm to manage multiple lines. I have to dig a little bit in the output of mathpartir (the latex package)

If you have any suggestions or want to give some help do not hesitate.

Regarding your first point, the layout function is what you are looking for.

I started working on a PR for that. Is this the kind of this you would expect in case there is not enough horizontal space?

How should the premises be aligned horizontally?

However you want to have something like this ?
    Rule1     Rule2     Rule3
        Rule4        Rule5
---------------------------
             Conclusion

When there is not enough space ? (for example in double columns style of an article)

The problem of typesetting the rules with multiple lines ressemble the problem of the Knapsack Problem where we have to stack the premisses width without never exceed the width of the page.

First Idea :

Having an algorithm that tries to type_set with n lines
trying for n=1, n=2 ... until we have a sucessfull typset

If (not enough_space) and (single_rule_without_tree) {
    number_lines_premisses = 1
    while(not enough_space) {
        enough_space = typesetting_premisses_multiple_lines(number_lines_premisses) // This has to be
        number_lines_premisses += 1
    }
} else {
   classic_type_setting_version_2.0
}

Analyse the problem as a decomposition of integer problems (does'nt take into account the size of the rules)
First Idea to typeset multiples lines of premisses  :
type_set_multiple_lines(nb_prem, nb_lines) {
decompose nb_prem as a sum of nb_lines integers that adds_up to nb_prem.
}
For example with 5 premisses and 2 lines : 5 = 2 + 3
So there will be :

2 premisses in the first line
3 premisses in the second line

If 6 premisses 4 lines : 6 = 1 + 1 + 2 + 2

However it seems to long to compute (not very efficient)
Can

Greedy algorithm taking into account the size of the rules

I think we should not change the order of the premisses given by the user. This is very important and will make the greedy algorithm fast.

function type_set(premisses) {
lines = []
// Lines is a 2D array that stores a list of line
// A Line is a list of premisse
current_line = 0
FOR prem in premisses DO
    IF (line[current_line] + premisse >(exceed) width_page) and (lines[current_line].len > 1) {
        current_line += 1
    }
    lines[current_lines].append(prem)
DONE

Although I like the idea of finding a general solution that works for any tree, I think most of the time, this problem arises with trees that only consists of a single rule application. It may be a bit overkill to make a complex algorithm that will mostly be used in this basic case.

However you want to have something like this ?
  Rule1     Rule2     Rule3
      Rule4      Rule5
-----------------------------
           Conclusion

When there is not enough space ? (for example in double columns style of an article)

Exactly!!

I would say this is out of the scope of Curryst. You can probably reproduce that yourself with something like
#let mathpar(..rules) = block(
  breakable: true,
  rules.pos().join(h(0.5cm)),
)

I see, you're right, actually mathpar is also intended as a general purpose math paragraph, not specifically targeting inference rules.


bugfix in ptb_rec() method

I think that at line 403, the punct argument was getting reset to its default value (True) on recursive calls.

Ah yes, that would do it. Thanks!


Request header invalid 6415 v4

https://redmine.openinfosecfoundation.org/issues/6415
#411 with code review taken into account
and not skipping zero as we used bstr_cmp_c_nocase not bstr_cmp_c_nocasenorzero

We should run all Suricata CI with this libhtp change...

Merged in #413, thanks!


Tbcm 63 open legend onload

On initial load open table legend.
Change text color

Confirm opens up on page load.
Layout:
13 inch MAC screen:

Confirmed scrolling is working as desired:


We should have a continuous monitoring bot that tries to run all our tests

This would catch failures caused by our system not being entirely hermetic.

This is a subset of https://github.com/flutter/flutter/issues/4696.

Triage: we have CI for this.


MAINT/TST: add citation and missing test

fixes #29

Coverage remained the same at 100.0% when pulling f1ce2213158c2c9ba83b1ab8621e81c01bbcaa32 on gregcaporaso:issue-29 into 292ca44126529e19f483fda918979936ed957285 on qiime2:master.


Add Unformatted Attribute to Displayname

Currently the Reference, Tuple, Schema, Table, and Columnhave a displayname attribute which has value and isHTML in it. We should add a new attribute to these displayname objects that will return the unformatted (without any HTML tags) value. If isHTML is false, the value and unformatted value should be the same.

@RFSH are you working on this issue or is this one done? Please update the issue status..

Yes, It's done as part of #376. I am closing this issue.


discord-canary: 0.0.98 -> 0.0.99

Motivation for this change
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

This seems to be outdated already, 0.0.102 seems to be the latest. Make the bump and I'll test it out :)

#81050


Make tags belong to a tag concept, and perhaps concepts belong to a concept concept

Playing with hana::set, I recognized that it is not possible to take every type as a key. Instead it would for example have to be wrapped as type<int> because of the equality comparison as mentioned in previous issue #231. One use-case for a map might be the keys being tags and to dispatch accordingly. However, that would also require tags to be wrapped as types, as in type<constant_tag>. Also, tags cannot simply be compared for equality.
Thus, I'd like to propose to give tags their own data type by introducing a hana_tag = tag_tag, and perhaps adding a concept Tag, along with an equality comparator for tag1 == tag2.
This would come in handy in particular if users intend to create a whole lot of new tags for their own possibly related or depending data types.
Similarly, I'd like to propose to give concepts their own Concept concept and concept_tag.
The above would also have another benefit: I'm constantly typing the wrong thing: a type where there should be a value, a concept where there should be a type etc.
Ultimately, having both would give everything in Hana a somewhat similar or conceptually identical representation.

While I am sympathetic to the idea of making the meta type-system reflexive (by giving a tag to tags themselves), this would break the requirement that hana::tag_of be idempotent (see #212).
Regarding concepts, I want to keep them as minimal as possible, because they may very well be replaced by C++17 concepts at some point.
Hence, in the cases where it makes sense to use tags e.g. as keys of a hana::map, I would advise simply wrapping them in hana::types, since that is what they really are (C++ types).


Add fuzzer for Vulkan->WebGPU spirv-opt passes

Fixes #2622

Failure looks like unrelated build flake, I have hit rebuild to confirm.


Add (optional) pgpass steps to postgres installation in docs #3483

Summary
This part of docs was updated to add pgpass steps to Postgres installation in docs
Ticket Link
If this pull request addresses a Help Wanted ticket, please link the relevant GitHub issue, e.g.
Fixes https://github.com/mattermost/docs/issues/3483
This part of the docs is supposed to be in the same section of the other OS's but I want to confirm that this is added correctly before moving on

Thanks, @fakela. This is another great PR, really appreciate it. I have made some minot suggestions related to formatting and wording. Other than those, this is a really helpful change!

@justinegeffen @streamer45 Thanks a lot for the comments and reviews.
I made a change to the docs adding the suggestions from @streamer45 and also looking at the official docs
Here it is

(Optional) The file .pgpass  is a convenient way to remove the need to type a password at every login. The file  .pgpasswhich is located in the user’s home directory or the file cited by PGPASSFILE contains passwords to be used if the connection requires a password (and no password has been provided). .pgpass is a special file that has to be 0600 otherwise it will not work.

Meanwhile, this is such a warm community always eager to respond and drop feedbacks. Thanks, @justinegeffen @streamer45. Looking forward to making more contributions


[BUG] Favicon not visible

Description
A favicon has been added via the HTML file however it is not visible.
The path needs to be updated and the image be saved with .ico extension
Screenshots

Any additional information?
This is under GSSOC'24
What browser are you seeing the problem on?
No response
Checklist

[X] I have checked the existing issues.
[X] I have read the Contributing Guidelines
[X] I have read the Code Of Conduct
[x] I joined onboarding group for knowledge flow and direct conversation and support from admins  Curious Developers Community - Curious Connect
[X] The changes don't break the code
[X] Please attach the screenshots and video of the development work once it is done
[X] If you need any help please join the default  (Google Meet Link)
[X] I am willing to work on this issue (optional).

Please assign me this issue under gssoc24.

Please assign this issue to me

Please assign this issue to me I've worked on this type of issues


[FLPATH-1792] Orchestrator Operator have hardcoded installplan mode

@jenniferubah @gabriel-farache @pkliczewski PTAL.
I will backport this commit to 1.2 once this PR is merged.
@chadcrum FYI.

@jordigilh Do we want to fix that for all other operator as well? I think it's hard coded for all operator

@gabriel-farache I would leave that to the team to decide. No objection from my side. However if we proceed with that idea, it should be covered in a different ticket since the scope of the bug is limited to sonataflowOperator.


Add .NET API refs and examples to sidebar

Contributes to https://github.com/open-telemetry/opentelemetry.io/issues/1808
.NET is a bit of a special case:

The system.diagnostics API is far larger than tracing, but that's the API where the opentelemetry-compatible things live
The system.diagnostics.metrics API is scoped just to metrics, and that's where metrics live
The above two points are shipped as GA as a part of Microsoft's work -- we cannot change the layout
There is no API reference for the actual SDK concepts like Sdk.CreateTracerProvider or Resource
There is no API reference for the opentelemetry API compat shim

So this is the best I could come up with.
Preview: https://deploy-preview-1947--opentelemetry.netlify.app/docs/instrumentation/net/

@cartermp - you'll need to add a regex to the link-checker config file so that it ignores the new placeholder-page paths that you just added. For an example of how to do this, see #1915.

Hmm, or maybe it's something else. I'm AFK atm, will check again later if you haven't fixed the problem by then.

Yeah, it's an enormous cascading failure. Unclear what could be the cause.

Back. Lemme take a look.

@cartermp - Oh! I just thought of a Hugo feature that might be helpful here. Might you want the API sidnav entry to be hierarchical with Metrics and Traces as subentries (with all of those being placeholder pages)?
I'm not 100% sure that I can get it to work, so maybe we can merge this PR if/once it is approved first. Let me know what you prefer (this going in first or me trying the above).

@cartermp - FYI rebased and resolved conflicts that arose post-merge f #1950

Done. Also, I wish the MS site were more selective about the language it resolves to. Suffice to say, most Russian language folks usually switch back to English 😬


"Uncaught Error: Module did not self-register." when using a file as database

Hello there,
i'm kinda new to nodewebkit and trying to get sqlite3 to work with it.
Operating system:
Win 8.1 X64
Tools used:
Python 2.7                  (Installed via "npm install -g windows-build-tools")
MS Developers Tools (Installed via "npm install -g windows-build-tools")
Nodejs 7.5 X64
NW 0.20.1 X64 SDK
I compiled node-sqlite3 with the following command:
npm install sqlite3 --build-from-source --runtime=node-webkit --target_arch=x64 --target=0.20.1

See output:
sqlite3.install.txt
I can use sqlite without problem when using a memory db. But when I want to use a filedb it gives me everytime Module unregistered errors:

Uncaught Error: Module did not self-register.
at Object.Module._extensions..node (module.js:640:18)
at Module.load (module.js:512:32)
at tryModuleLoad (module.js:471:12)
at Function.Module._load (module.js:463:3)
at Module.require (module.js:522:17)
at require (internal/module.js:20:19)
at Object. (D:\SQLITETEST\app\node_modules\sqlite3\lib\sqlite3.js:4:15)
at Module._compile (module.js:595:32)
at Object.Module._extensions..js (module.js:610:10)
at Module.load (module.js:512:32)

Usualy this error comes from having version mismatch, but i didn't upgrade any of my Tools.
Despite throwing the errors the commands get executed, but still the debug console is unusable, cause everytime a command is executed it restarts the debug window.
Index.html:
<!DOCTYPE html>
<html>
<head>
	<title>TEST SQLITE</title>
	<script src="node_modules/nw-dev/lib/dev.js"></script>
	<script>
		var sqlite3 = require('sqlite3').verbose();
		var db = new sqlite3.Database('test.db');
		//var db = new sqlite3.Database(':memory:');
		db.serialize(function() {
			db.run('CREATE TABLE IF NOT EXISTS `test` (`id` INT)');
			//THIS THROWS ERRORS WHEN USING 'test.db' AS DATABASE
		});

		db.close();
	</script>
</head>
<body>
</body>
</html>

package.json:
{
  "name": "SQLite_TEST",
  "version": "0.0.1",
  "description": "BLABLA",
  "main": "index.html",
  "author": "Me",
  "license": "GPL-3.0",
  "devDependencies": {
    "nw-dev": "^3.0.1"
  }
}

app.zip
Any ideas how to fix Module did not self-register errors?
Thx in advance.

I just tested it on 0.20.3 x64 SDK and it doesn't happen anymore.


Remove routed attribute when executing fallbacks in circuit breakers

Fixes #1421

@armsargis I have included your fix in the this PR and added code to so something similar for the hystrix filter


OPDS "buy" acquisition link only used in popup menu, missing in publication info panel

See screenshots:

Feedbooks OPDS "best sellers":
https://readium2.herokuapp.com/opds-v1-v2-convert/https%3A%2F%2Fcatalog.feedbooks.com%2Fstore%2Fbrowse%2Fen%2Ftop.atom
Dedicated publication OPDS info:
https://readium2.herokuapp.com/opds-v1-v2-convert/https%3A%2F%2Fcatalog.feedbooks.com%2Fitem%2F2459861.atom

Here is the NYPL demo browser:
http://opds-browser-demo.herokuapp.com/collection/https%3A%2F%2Fcatalog.feedbooks.com%2Fstore%2Fbrowse%2Fen%2Ftop.atom/
(https://catalog.feedbooks.com/store/browse/en/top.atom)

I fixed the problem in #804 for the buy link but for the other links I don't really know where to put it because if I put it on the same place, there will be a lot of link

Fixed via https://github.com/readium/readium-desktop/commit/17e79eebfaa6fbc3872e5c1c7aec1f7abbe804b8


Unsupported platform: protoc-3.19.0-linux-x86_64.exe

I am getting the following error:
[ERROR] Failed to execute goal com.github.os72:protoc-jar-maven-plugin:3.11.4:run (default) on project relshp-query: Error extracting protoc for version 3.19.0: Unsupported platform: protoc-3.19.0-linux-x86_64.exe
Please assist,
Thank you in advance,
Ela

I also encountered this problem.

Failed to execute goal com.github.os72:protoc-jar-maven-plugin:3.11.4:run (default) on project orc-core: Error extracting protoc for version 3.21.5: Unsupported platform: protoc-3.21.5-osx-x86_64.exe -> [Help 1]

I encountered the same. Please checkout the fork: https://github.com/blackrock/protoc-jar-maven-plugin
This forked version uses Google’s Protoc version 3.25.1 by default which supports osx aarch64.


fix(log): Ensure tools/log paths are constructed portably

On Linux, the backslash in tools\\log\\ was not interpreted as path separator, so screenshots etc. ended up in the project's root directory, named tools\log\<filename>. Now they are named <filename> and stored in the log-directory under tools.

@andreaswachowski danke dir!


cannot get assets js file in web worker after flutter build web

Steps to reproduce

create a js file in asset/js
add assets path to pubsec.yaml
assets:

assets/js/

add web worker with js file url e.g. 'assets/js/maintenance_worker.js'
run flutter build web --release

Expected results
run specific asset js file via web worker normally as well as flutter run -d chrome
Actual results
Pop up HTTP 404 error after flutter build web
However I can run the same js file with flutter run -d chrome
Code sample
Code sample

// ignore: avoid_web_libraries_in_flutter
import 'dart:html';
...
final maintenanceWorker = Worker('assets/js/maintenance_worker.js');

maintenanceWorker.postMessage();

We probably found the issue is that the assets folder path changed to assets/assets nested folder after flutter build web.
And we use workaround in following for now but looking for the way to avoid loading and parsing js and creating blob.
Code sample 2
Code sample 2
import 'dart:async';
// ignore: avoid_web_libraries_in_flutter
import 'dart:html';
import 'package:flutter/services.dart';
...
final jsCode = await rootBundle.loadString('assets/js/maintenance_worker.js');

final blob = Blob([jsCode], 'application/javascript');

final url = Url.createObjectUrlFromBlob(blob);

final maintenanceWorker = Worker(url);

maintenanceWorker.postMessage();

Screenshots or Video

Screenshots / Video demonstration
The nested assets folder after flutter build:

Logs
Logs
Error while trying to load an asset: Flutter Web engine failed to fetch "assets/js/maintenance_worker.js". HTTP request succeeded, but the server responded with HTTP status 404.

Flutter Doctor output
Doctor output
[✓] Flutter (Channel stable, 3.24.0, on macOS 14.5 23F79 darwin-arm64, locale en-CA)
    • Flutter version 3.24.0 on channel stable at /Users/ventuslum/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 80c2e84975 (3 weeks ago), 2024-07-30 23:06:49 +0700
    • Engine revision b8800d88be
    • Dart version 3.5.0
    • DevTools version 2.37.2

[✓] Android toolchain - develop for Android devices (Android SDK version 35.0.0-rc4)
    • Android SDK at /Users/ventuslum/Library/Android/sdk
    • Platform android-35, build-tools 35.0.0-rc4
    • Java binary at: /Applications/Android Studio.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 17.0.10+0-17.0.10b1087.21-11572160)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 15.4)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 15F31d
    • CocoaPods version 1.15.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2023.3)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 17.0.10+0-17.0.10b1087.21-11572160)

[✓] IntelliJ IDEA Community Edition (version 2023.3.2)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart

[✓] VS Code (version 1.85.1)
    • VS Code at /Users/ventuslum/Downloads/Visual Studio Code.app/Contents
    • Flutter extension can be installed from:
      🔨 https://marketplace.visualstudio.com/items?itemName=Dart-Code.flutter

[✓] Connected device (4 available)
    • iPhone 15 Pro Max (mobile)      • 6402E5B0-D942-4919-8E52-86E0A6CE0851 • ios            • com.apple.CoreSimulator.SimRuntime.iOS-17-5 (simulator)
    • macOS (desktop)                 • macos                                • darwin-arm64   • macOS 14.5 23F79 darwin-arm64
    • Mac Designed for iPad (desktop) • mac-designed-for-ipad                • darwin         • macOS 14.5 23F79 darwin-arm64
    • Chrome (web)                    • chrome                               • web-javascript • Google Chrome 127.0.6533.120

[✓] Network resources
    • All expected network resources are available.

• No issues found!

Pop up HTTP 404 error after flutter build web
However I can run the same js file with flutter run -d chrome

@Ventus924
this can be due to this extremely annoying as-intended behavior

https://github.com/flutter/flutter/issues/67655

Hi @Ventus924
Does this only occur with js file or any asset file type? Can you check this with @iapicca sample code above (with txt asset file type) on your end?
If the issue only occurs with your sample code, please provide a completed and minimal reproducible code sample that doesn’t include 3rd party plugins or complex production code. Thank you!

Hi @huycozy @iapicca ,
Yes, it is only happen in js file. It is because that I need to input the assets js path in Worker.
Here is the new sample code:
New Steps to reproduce

add js denpendencies:
dependencies:
js_interop: ^0.0.1
create worker.dart
generate js file with command dart compile js -o assets/js/worker.js lib/worker.dart
add assets path to pubsec.yaml
flutter:
assets:

assets/js/

run flutter build web --release

Code sample
Code sample
main.dart
// ignore: avoid_web_libraries_in_flutter
import 'dart:html';
import 'package:flutter/material.dart';
import 'package:flutter/services.dart';

void main() {
  runApp(const MyApp());
}

class MyApp extends StatelessWidget {
  const MyApp({super.key});

  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Web Worker Demo',
      theme: ThemeData(
        primarySwatch: Colors.blue,
      ),
      home: const MyHomePage(),
    );
  }
}

class MyHomePage extends StatelessWidget {
  const MyHomePage({super.key});

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(
        title: const Text('Web Worker Demo'),
      ),
      body: Center(
        child: ElevatedButton(
          onPressed: () async {
            showMyDialog(context);
          },
          child: const Text('Post to JS'),
        ),
      ),
    );
  }

  void showMyDialog(BuildContext context) async {
    final jsCode = await rootBundle.loadString('js/worker.js');

    final blob = Blob([jsCode], 'application/javascript');

    final url = Url.createObjectUrlFromBlob(blob);

    final worker = Worker(url);

    try {
      worker.postMessage('');
    } catch (e) {
      throw Exception(e);
    }

    final messageFromJS =
        (await worker.onMessage.first.timeout(const Duration(seconds: 3))).data;

    showDialog(
      context: context,
      builder: (BuildContext context) {
        return AlertDialog(
          title: Text(messageFromJS),
          content: const Text('This is the message come from JS file'),
          actions: <Widget>[
            TextButton(
              onPressed: () => Navigator.of(context).pop(),
              child: const Text('Close'),
            ),
          ],
        );
      },
    );
  }
}

worker.dart
import 'dart:convert';
// ignore: avoid_web_libraries_in_flutter
import 'dart:html';
import 'package:js/js.dart';

@JS('self')
external DedicatedWorkerGlobalScope get self;

void main() {
  self.onMessage.listen((e) async {
    self.postMessage('Success');
  });
}

Screenshots or Video

Screenshots / Video demonstration
Here is the result to show JS file message in flutter web:

Is any better way to implement JS assets file without loading and parsing js and creating blob ?

@Ventus924
try this

-    final jsCode = await rootBundle.loadString('js/worker.js');
+     final jsCode = await rootBundle.loadString('assets/js/worker.js');

see

https://github.com/flutter/flutter/issues/67655

@Ventus924 I see the Iapicca suggestion above works (it does show the dialog as your screenshot). Please try and confirm this

@iapicca @huycozy
Yes, I missed the path value in this sample, thank you for remind that.
But is any other way to load js file path instead of loading and parsing js and creating blob ?
Because now we  need to do the extra job in order to get the js file path which may affect the performance.

But is any other way to load js file path instead of loading and parsing js and creating blob ?

@Ventus924
as I mention above, in my opinion is worth trying service_worker w/o reinventing the wheel

Is any better way to implement JS assets file without loading and parsing js and creating blob ?

probably service_worker (it's 3rd party, but from extremely capable people)

but question like this are better answered on StackOverflow or Discord

@Ventus924 @vasilich6107 Since the original issue has been solved, I'm going ahead and closing the issue. If you have another proposal or bug report, please file a new issue.
Please see flutter's community for resources as well. You may also get some help if you post it on Stack Overflow and flutterhelp. Thank @iapicca for sharing the solution!


"Could not load file or assembly 'Microsoft.Extensions.PlatformAbstractions'"

I'm trying to run NSwagStudio against an ASP.NET Core 2.0 assembly that contained WebAPI controllers, however I get this error message:

Text from the text-area in the window:

System.IO.FileLoadException: Could not load file or assembly 'Microsoft.Extensions.PlatformAbstractions, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=adb9793829ddae60' or one of its dependencies. The located assembly's manifest definition does not match the assembly reference. (Exception from HRESULT: 0x80131040)
File name: 'Microsoft.Extensions.PlatformAbstractions, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=adb9793829ddae60'

Runtime: Default
Server stack trace:
   at Microsoft.AspNetCore.Hosting.WebHostBuilder.BuildHostingServices()
   at Microsoft.AspNetCore.Hosting.WebHostBuilder.Build()
   at Microsoft.AspNetCore.TestHost.TestServer..ctor(IWebHostBuilder builder)
   at NSwag.Commands.SwaggerGeneration.SwaggerGeneratorCommandBase`1.CreateTestServer(Type startupType)
   at NSwag.Commands.SwaggerGeneration.WebApi.WebApiToSwaggerCommand.d__21.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at NSwag.Commands.IsolatedCommandBase`1.IsolatedCommandAssemblyLoader`1.Run(String commandType, String commandData, String[] assemblyPaths, String[] referencePaths)
   at System.Runtime.Remoting.Messaging.StackBuilderSink._PrivateProcessMessage(IntPtr md, Object[] args, Object server, Object[]& outArgs)
   at System.Runtime.Remoting.Messaging.StackBuilderSink.SyncProcessMessage(IMessage msg)

Exception rethrown at [0]:
   at System.Runtime.Remoting.Proxies.RealProxy.HandleReturnMessage(IMessage reqMsg, IMessage retMsg)
   at System.Runtime.Remoting.Proxies.RealProxy.PrivateInvoke(MessageData& msgData, Int32 type)
   at NSwag.Commands.IsolatedCommandBase`1.IsolatedCommandAssemblyLoader`1.Run(String commandType, String commandData, String[] assemblyPaths, String[] referencePaths)
   at NSwag.Commands.IsolatedCommandBase`1.<>c__DisplayClass17_0.b__0()
   at System.Threading.Tasks.Task`1.InnerInvoke()
   at System.Threading.Tasks.Task.Execute()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at NSwag.Commands.IsolatedCommandBase`1.d__17.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at NSwag.Commands.IsolatedSwaggerOutputCommandBase`1.d__9.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at NSwag.Commands.NSwagDocumentBase.d__40.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at NSwag.Commands.NSwagDocument.d__8.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at NSwag.Commands.Document.ExecuteDocumentCommand.d__9.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at NSwag.Commands.Document.ExecuteDocumentCommand.d__8.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at NConsole.CommandLineProcessor.d__12.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at NConsole.CommandLineProcessor.d__11.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at NConsole.CommandLineProcessor.Process(String[] args, Object input)
   at NSwag.Commands.NSwagCommandProcessor.Process(String[] args)

WRN: Assembly binding logging is turned OFF.
To enable assembly bind failure logging, set the registry value [HKLM\Software\Microsoft\Fusion!EnableLog] (DWORD) to 1.
Note: There is some performance penalty associated with assembly bind failure logging.
To turn this feature off, remove the registry value [HKLM\Software\Microsoft\Fusion!EnableLog].
I installed NSwag using the MSI currently live on the website. The MSI's build date+time is 2018-09-29 04:19. The NWagStudio.exe's version is <IP_ADDRESS>.

See https://github.com/RSuter/NSwag/wiki/Assembly-loading


Issue with Harbor upstream image scan webhooks

Presently there is an upstream issue affecting some installations of Harbor - the problem is described here https://github.com/goharbor/harbor/issues/14585
For our purposes we're receiving misleading data from the Trivy scan in the Harbor webhook payload, telling us that a scan is still running when it is, in fact, complete. We don't rely on this payload except to check the scan status. But we know that Harbor will not send this particular webhook without the scan both being complete and successful (incomplete scans are a different webhook event)

Issue fixed in #2671


switch codegen ordering of final methods and name-based intrinsics

Motivation

When #4760 landed, it silently disabled final method codegen for [] methods: the name-based intrinsic for [] would fire, and, finding that there was actual type information associated with the receiver, the intrinsic would emit a method call through the Ruby VM for the reasons written in the comment:
https://github.com/sorbet/sorbet/blob/d6aefd59ae07dba98e0132cf8b0fff727a8344be/compiler/IREmitter/NameBasedIntrinsics.cc#L852-L857
I can say with some certainty that the author was only thinking about symbol-based intrinsics, and not thinking about final methods when they wrote that.
We didn't really notice because the gain on the lspace benchmark (which said PR was intended to speedup) was so significant, and we didn't have any tests.  But today I noticed that removing sig(:final) from the lspace benchmark wasn't having any effect, and so I went hunting.
This PR should restore the functionality of generating direct calls for []-methods (and other similar methods that we have untyped specializations for).  Arguably we should maybe move final methods to attempt codegen before symbol-based intrinsics, but seeing as how none of the methods we'd write symbol-based intrinsics for could be final, I'm not too worried about this.
lspace benchmark before this PR:
ruby vm startup time: .092
source  interpreted     compiled
stripe/while_10_000_000.rb      .218    .099
stripe/lspace.rb        .892    .474
stripe/lspace.rb - baseline     .674    .375

lspace benchmark after this PR:
ruby vm startup time: .093
source  interpreted     compiled
stripe/while_10_000_000.rb      .223    .109
stripe/lspace.rb        .899    .417
stripe/lspace.rb - baseline     .676    .308

...and we are now officially more than 2x faster on this benchmark. :tada:
Test plan

See included automated tests.

We have a policy of testing changes to Sorbet against Stripe's codebase before
merging them. I've kicked off a test run for the current PR.  When the build
finishes, I'll share with you whether or how it failed. Thanks!
Stripe employees can see the build results here:
→ https://go/builds/bui_KhOwx05YGy7FUO
→ https://go/builds/bui_KhOw1eEibJITus


A quick way to disable this plugin for copy/pasting text

Is there a way to quickly turn on/off this plugin for copying text? Currently, the indent lines are copied too.
I was thinking of something equivalent to :set nonumber.
What do you think?

You mean when selecting with the mouse?
You should :set mouse=a in vim, so selecting is handled by vim, and not you terminal. This way you don't copy things like numbers, listchars and also indent lines.
But there is also a way to turn the plugin on and off, with :IndentBlanklineDisable, :IndentBlanklineEnable and IndentBlanklineToggle


Next activity : you see the timer start on the page underneath first then the new page comes up

when clicking next activity from timer pop up you see the page you are leaving show the timer then the next activity comes up
prob an animation thing

Closing per @sshaar08


Display statistics

Combing GUI Statistics table with end of exam tests.

Combing GUI Statistics table with end of exam tests.


Our-Service  section of our web (first draft )

Hi Seni, I think you should not merge it into the 1-database branch, rather to main (see dev-strat)

Should E create a branch call i-homepage??
On Wed, Dec 15, 2021, 16:30 Ekaterina @.***> wrote:

Hi Seni, I think you should not merge it into the 1-database branch,
rather to main (see dev-strat)
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/BeHelp/BeHelp/pull/68#issuecomment-994903471, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AP4ZDEV2DYEWX34BFCCHVELURCYAPANCNFSM5KCCI7UA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.


Safari extension button does not function until the page is reloaded in a new tab

For me, when using the Safari extension on macOS Ventura, Safari 17.3, after enabling the extension for a particular website, the button does not function until the page is reloaded in a new tab. This lead me to suspect the feature didn't work at all. I'm glad I figured out the workaround, but not ideal. (Fantastic feature!)

I have a similar problem on iPad Air 3 iPadOS 17.3.1.
If I tap the share icon lower right, and tap the icon for my preferrred browser (Brave) the web page opens in Brave. When I return to NetNewsWire, the pop up showing the share options for that story is still there, even when I move to subsequent items in the feed list. If I three-finger-swipe back to the page in Brave, wait least three seconds, and swipe back, the options popup closes.
Oddly, for some sites (such as abc.net.au) this doesn’t happen if I wait for the original page to load completely before I return to NetNewsWire. For others (such as guardian.com) the popup persists even after the page loads completely.


How to set isCustomElement option

I need to set isCustomElement option before vite-vue plugin is created - how can I do that? Regular vite config looks like this:
// vite.config.js
...
plugins: [
  vue({
    template: {
      compilerOptions: {
        isCustomElement: tag => tag.startsWith('foo-')
      }
    }
  })
]
...

@mgibas if you need to customize the plugins, you can use viteFinal.  You'll be given the default config as the first param of the function, and you can change it however you need to.  In this case, you might want to use a filter to remove the existing plugin, and then push your custom-configured one back on.

I was pretty sure I tried that already but will give it another go tomorrow. Thanks @IanVS

:+1: Feel free to share a reproduction and I can take a look, if that would be helpful.

I also needed to configure @vitejs/plugin-vue. The order matters since it has to come before the vue-docgen plugin:
async viteFinal(config) {
  const pluginVueIndex = config.plugins.findIndex(config => config.name === 'vite:vue');
  config.plugins[pluginVueIndex] = vue({
    template: {
        compilerOptions: {
            whitespace: 'preserve',
        },
    },
  });
  return config;
}

Awesome, thanks for posting back.  It sounds like this issue can be closed, but let me know if you have any other trouble.

I still have a hard time passing isCustomElement option to Vue. Will try to create a repro (once time allows), now I can just paste my main file in hope I missed something :)
  const vue = require('@vitejs/plugin-vue')

  module.exports = {
    stories: ['../src/**/*.stories.@(js|mdx)'],
    addons: ['@storybook/addon-links', '@storybook/addon-essentials'],
    core: { builder: 'storybook-builder-vite' },
    async viteFinal(config) {
      const pluginVueIndex = config.plugins.findIndex((config) => config.name === 'vite:vue')
      config.plugins[pluginVueIndex] = vue({
        template: {
          compilerOptions: {
            isCustomElement: (tag) => tag.startsWith('foo-')
          },
        },
      })
      return config
    },
  }

Hey @IanVS, just created a repro:
https://stackblitz.com/edit/vitejs-vite-6knqyx
You should be able to run rpm run sb, in console there will be a warning:
[Vue warn]: Failed to resolve component: foo-test
If this is a native custom element, make sure to exclude it from component resolution via compilerOptions.isCustomElement.

Let me know If I missed something.

@mgibas did you find a solution? I am having the same issue.
@IanVS Have tried filtering out the vue plugin and pushing a configured one back in. Have also tried customElement: true but it still gives the warning Failed to resolve component: ui-example and the component doesn't render.
module.exports = {
  async viteFinal(config) {
    config.plugins =
      config.plugins?.filter((plugin) => plugin.name !== 'vite:vue') ?? [];

    const vuePlugin = vue({
      template: {
        compilerOptions: {
          isCustomElement: (tag) => tag.startsWith('ui-')
        }
      }
    });

    config.plugins.unshift(vuePlugin);

    return config;
  },
  ...
};

I also needed to configure @vitejs/plugin-vue. The order matters since it has to come before the vue-docgen plugin:
async viteFinal(config) {
  const pluginVueIndex = config.plugins.findIndex(config => config.name === 'vite:vue');
  config.plugins[pluginVueIndex] = vue({
    template: {
        compilerOptions: {
            whitespace: 'preserve',
        },
    },
  });
  return config;
}

@florrie-90  I do similar like this and it works for me
async viteFinal(config) {
        const { config: userConfig } = await loadConfigFromFile(
            path.resolve(__dirname, '../vite.config.ts')
        );
        const mergedConfig = mergeConfig(config, {
            ...userConfig,
            // manually specify plugins to avoid conflict
            plugins: [],
        });
        const currentVueVitePluginIndex = mergedConfig.plugins.findIndex((plugin) => plugin.name === 'vite:vue');
        mergedConfig.plugins[currentVueVitePluginIndex] = vue({
            template: {
                compilerOptions: {
                    // treat any tags that start with `cm-` which are from Aurora as custom elements
                    isCustomElement: (tag) => tag.startsWith('cm-'),
                },
            },
        });

        return mergedConfig;
}

@florrie-90 - unfortunately no. I know this code is executed cause I can see some logs in the console but tag is never one of the custom elements that I have in the preview.


WIP: Added warning to nodes selector if nothing was matched

resolves #2115
Description
Added warning (or error when --warn-error provided) when resource selector fails due to lack of resources for example when the incorrect model name was provided.
Checklist

[x] I have signed the CLA
[x] I have run this code in development and it appears to resolve the stated issue
[x] This PR includes tests, or tests are not required/relevant for this PR
[x] I have updated the CHANGELOG.md and added information about my change to the "dbt next" section.

Thanks for opening this PR @Raalsky! I dropped one copy request inline + adding @beckjake to review :)

This will trigger a warning on any project that doesn't have sources (that's what the tests are failing about). I think this is a good feature, but I think we're going to have to change a bit more about how node selection works for this to function.
I think we could change get_selected and select_nodes to handle this case a bit more carefully: instead of calling coalesce in get_selected, do the replacement in select_nodes and preserve the initial value. If it's None, skip the check.

Hi @beckjake. I've applied your suggestions and perform some refactors :wink: Feel free to reach out if you have any questions


255 inspect numeric threshold based dge for data with na values

code added to run_config_app.R to check to for NA values in expression data based on object type

I will add a note about not including NA values in the object guidelines vignette (#277).


JS-T: META: Centralize location of testnet tutorial accounts & keys

As a tutorial maintainer, I want an easy place to link to for "more" testnet "play" accounts in both the readme, and in the active tutorial code.
As a tutorial user, I want to have options if the default accounts for a tutorial have been hijacked.

AC

[x] a standardized location in this repo for testnet accounts with account names & priv keys
[x] a file that is importable from our tutorials, or a plan for making an importable file.
[x] explanation of what was decided & why in # Result/s comment

Taking the PR listed as the "results" comment. :)


[Request] Guide that explain how to create a new game.

hello, im trying to understand how to modify an existing game, but from the spiel.h i cant figure out how many functions works and the requirement for the different gameType (whats is needed for a kSimultaneous vs kSequential for example).
could you provide a simple guide at least to explain in detail how to create a game?
from spiel.h:
// Which type of information state representations are supported?
// The information state is a perfect-recall state-of-the-game from the
// perspective of one player.
bool provides_information_state_string;
bool provides_information_state_tensor;
the comments above do not explain in detail whats the meaning of each xString and xTensor, but it gives only a general explanation of the "provides_information_"

Determine the GameType and StateType of the environment you want to implement.

The provides_* usually mean that the related function is optional and you should use the returned flag to tell users whether it is implemented in your game or not. For example, provides_information_state_string ->InformationStateString. Generally speaking, you're suggested to implement these functions as much as possible to cover the potential usages by different kinds of algorithms.

Find an existing game of the same GameType or StateType. If there exists one and then you are lucky. Start from modifying that example and add test cases.

You can use some functions to filter out games of specific types like this:
https://github.com/deepmind/open_spiel/blob/a651ce7fbea858b0be2db98afabb49f7c830d904/open_spiel/python/tests/games_sim_test.py#L34-L60

Make sure that your game is registered and then make a PR.

https://github.com/deepmind/open_spiel/blob/c2be1951587b2879b04ef986706652275256081a/open_spiel/games/tic_tac_toe.cc#L51

Hi @lorrp1 , best thing to do is read the paper: https://arxiv.org/abs/1908.09453. Then read comments, then if you're still not sure, just ask! We don't bite. :)
On first pass, you can omit all InformationStateString, InformationStateTensor, ObservationString, ObservationTensor (and set all their respective provides_ in the game type to be false). These are really the last things you need to implement, and depending what you want to do, you may not even have to implement them.
You can run tests and check correctness without having these functions, and they are both the most boring and easiest to get wrong, so just best to leave them until the end.
Also: please identify any pieces of documentation you feel is missing, noting down some pain points along the way. And then feel free to submit a fix to the current guide :)

BTW I replied assuming you've seen the "Adding a Game" section of this page: https://github.com/deepmind/open_spiel/blob/master/docs/developer_guide.md
If not, that's a great place to start!

Hi @lorrp1 , did you find what you needed? Any more questions for us?


 Will push up a commit soon that makes the changes, and we can discuss if you disagree with any of them.
Please don't merge just yet.

@tmwbook when we meet to test the site tonight let's actually discuss some stuff about the party list. It looks like we're not using the Party model as intended any more.

Closed in favor of #86


Sync master with Events branch

This is a copy of master with Events branch merged.
I have run locally default, standalone, perf and rest profiles.
Everything running as expected locally against Hawkular Alpha6 as integration platform for testing.

yay for us!


feat: Fixing detokenization links and adding redirect for /detokenization

Description

Fixes links to /detokenization pages to point to /expressions

Testing required outside of automated testing?

[x] Not Applicable

Screenshots (if appropriate):

[x] Not Applicable

Rollback / Rollforward Procedure

[x] Roll Forward
[ ] Roll Back

Reviewer Checklist

[ ] Description of Change
[ ] Description of outside testing if applicable.
[ ] Description of Roll Forward / Backward Procedure
[ ] Documentation updated for Change

:tada: This PR is included in version 1.52.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Update _post_process() in core.py

@twopirllc,
In the case where an indicator returns a df, I added an option in _post_process(), via kwargs, to append only specific columns to the data. It goes like this :
CustomStrategy = ta.Strategy(
    name="MyStrat",
    ta=[
        {'kind':'bbands', 'params':(20,), 'col_numbers':(0,2)},
    ]
)

or alternatively : ao = df.ta.bbands(append=True, col_numbers=(0,2))
which can be usefull if the user needs only the upper and lower bbands for example.
Thanks,
DrPaprikaa

@DrPaprikaa,
It's on development now. While doing some strategy testing, make test_strats, the Custom Strategies break when the number of cores < multiprocessing.cpu_count(). I took a few stabs at it trying to get it to work for cores < multiprocessing.cpu_count() but still having issues with missing columns. The test file is: tests/test_strategy.py if you want to take a peek.
Also check out the Makefile on how to run other tests.
Other than that, it seems fine. Just need to test the Notebook Examples and update documentation before updating to master.
Thanks!
KJ

@twopirllc,
I haven't used test_strats but the CustomStrategies run well in my algo whatever the number of cores.
Also please note that add_prefix_suffix and append are called multiple times in core.py in developpement: in mp_worker() (lines 403-404), in post_process() (lines 425-426) and again in strategy() (lines 613-614). That this can cause conflicts, and it applies the prefixes multiple times.
DrPaprikaa

@DrPaprikaa,

I haven't used test_strats but the CustomStrategies run well in my algo whatever the number of cores.

Ok it is working since chunksize of the process pool is equal to the cores.

Also please note that add_prefix_suffix and append are called multiple times in core.py in development: in mp_worker() (lines 403-404), in post_process() (lines 425-426) and again in strategy() (lines 613-614). That this can cause conflicts, and it applies the prefixes multiple times.

Got this fixed.

Also, I suggest to leave out line 129 of core.py in development (df.dropna(axis=0, inplace=True)) and leave this to the user's discretion. For example, the user might have other columns in the df prior to running an indicator : if these columns happen to contain NaNs then panda-ta will wrongly delete the corresponding rows.

Yeah. I have been toying with it. However it only drops rows (axis=0).
Still debugging.

@DrPaprikaa,
It is up on the development branch if you want to test it out. Unfortunately, I still do have an issue with some Chained Custom Strategies when using cores below the cpu_count, yet the CommonStrategy which is Chained does not have this issue. But no issues when using all the cores for Chained Custom Strategies. 🤷♂
Let me know how the development branch is working for you before I update master. I also included an example with params and col_numbers at the end of the Strategy Examples.
Thanks,
KJ

@twopirllc,
Sorry I'm a bit caught up at work these days. Can you tell me exactly how to run the tests file so I can play around with it in the development branch ? I'll play with the number of cores and various strategies to see if I also have issues on my end.
Which CustomStrategies fail on your end ? I'll try to replicate them also!
DrPaprikaa

@DrPaprikaa,

Sorry I'm a bit caught up at work these days.

Yeah no worries.

Can you tell me exactly how to run the tests file so I can play around with it in the development branch ? I'll play with the number of cores and various strategies to see if I also have issues on my end.

If you are using Mac or Linux, you can run this command on the shell:
$ make test_strats

If running make does not work, then run the following (which make is doing anyhow):
$ python -m unittest -v tests/test_strategy.py

In tests/test_strategy.py, uncomment # @skip the tests that are working to skip them. Near the top of the test file are some test variables to test with:
cores = 4
cumulative = False
speed_table = False
strategy_timed = False
timed = True
verbose = False

To test different cores, don't forget to uncomment the comment in setUpClass:
class TestStrategyMethods(TestCase):
    @classmethod
    def setUpClass(cls):
        cls.data = sample_data
        # cls.data.ta.cores = cores
        cls.speed_test = DataFrame()

When changing cores, test_common(self) and test_custom_args_tuple(self) are unaffected even though they are simple Chained Strategies. However test_custom_a(self) is affected with cores != cpu_count(), but fine otherwise. All the other tests can be skipped because they are not Custom and work fine.
Thanks!
KJ

@twopirllc,
I'm on Windows but I managed to run the tests file. I have 8 cores. What I've seen so far :

test_common() works fine whatever the number of cores.

test_common() and test_custom_args_tuple() fail because of the chained indicator {"kind": "sma", "close": "CUMLOGRET_1", "length": 5, "suffix": "CUMLOGRET"}. Without this indicator, they work fine whatever the number of cores.

with this indicator, test_custom_a() runs fine with 4, 5, 7 & 8 cores, fails otherwise.

with this indicator, test_custom_args_tuple() runs fine with 3, 5, 6, 7 & 8 cores, fails otherwise.

I'm not familiar enough with multiprocessing to explain this, sorry ! But a work around would be to default the number of cores at max-1 or 1, following a comment under this StackOverflow answer :

multiprocessing.cpu_count()-1 or 1 can be a useful heuristic for deciding how many processes to run in parallel: the -1 avoids locking up the system by monopolising all cores, but if there is only one CPU available then the or gives a graceful fallback to single-core running.

DrPaprikaa

@DrPaprikaa,

test_common() works fine whatever the number of cores.

Same

test_custom_a() and test_custom_args_tuple() fail because of the chained indicator {"kind": "sma", "close": "CUMLOGRET_1", "length": 5, "suffix": "CUMLOGRET"}. Without this indicator, they work fine whatever the number of cores.

Same
Thanks for the StackOverflow answer. I haven't seen that one.  In general it works for Chained Strategies. As a fallback, it might be good to add back a non-multiprocessing option for custom since it never fails.
Thanks for your time. I appreciate it.
KJ

@twopirllc,

As a fallback, it might be good to add back a non-multiprocessing option for custom since it never fails.

I agree. It is also a good idea for Strategies with a small number of indicators, since creating processes takes times and can be slower than without multiprocessing in that case.

Thanks for your time. I appreciate it.

I'm glad to help you build this awesome library !
DrPaprikaa


pcln-popover has pcln-design-system as dependency

pcln-popover component has pcln-design-system listed as a dependency and a peerDependency in its package.json. It should be moved from dependency to devDependency so that a specific version of pcln-design-system is not forced on the consumer

Fixed in pcln-popover 2.0.3


Weather cog returns input for cords (0,0) when given an invalid input

Yeah I probably should just have it return a message saying "Error: invalid location" instead of returning that, which is literally the weather at longitude 0, latitude 0:


add goerli and bump zksync-web3 version

Goerli is now the default L1 testnet. Bumped zksync-web3, otherwise all deploys would fail since 0.2 is missing the chainId tx serialization
Not sure how you guys prefer to do npm releases, so I left the package version as is

Thanks @mikemcdonald! We'll set up dependabot to do this automatically next time


Qnn for first conv

The Current Partition Pass for quantize will not insert annotation.cast_hint and annotation.stop_fusion for the first Conv2D of the model.
By adding there two lines in  def conv2d_partition_function, the first Conv2D will be quantized as expected

cc @anijain2305 @ZihengJiang

Sorry for the late reply. I think this is because of skip_conv_layers
By default, it skips the first conv layer. Maybe, because first layer has higher impact on accuracy. So, I dont think this is needed. You can use QConfig to set this.


Need unlink method on typescript please.

Need unlink method on typescript please :)

could we resurrect this ticket, please?

Feel free to send a PR with the requested improvement :)

@AVVS linked PR

I think maintainers would be happy to accept once tests pass https://travis-ci.org/DefinitelyTyped/DefinitelyTyped/builds/646532218?utm_source=github_status&utm_medium=notification

weird, it passed in my local.  Instead of making a proper fork, since it was just one line I edited the file in github directly… which of course added the extra space in the empty line. No formatter there… 🤦♂


Fix #368: allow specification of arbitrary rsiviz configuration

@gurhar1133 ping. There is a PR I'd like to open that needs these changes so please review as soon as you have time.

Thanks
Those keys represent the names of the environment variables (ex PYKERN_PKCONFIG_CHANNEL). So, we want to exclude certain values that contain sensistive data that doesn't need to be available in the environment (ex a secret that no service reads from the env).
from pykern import pkconfig
from pykern.pkcollections import PKDict
from pykern.pkdebug import pkdp

for e in "", r"some_secret":
    pkdp(pkconfig.to_environ(["*"], values=PKDict(foo=1, some_secret=2), exclude_re=e))

Thanks
Those keys represent the names of the environment variables (ex PYKERN_PKCONFIG_CHANNEL). So, we want to exclude certain values that contain sensistive data that doesn't need to be available in the environment (ex a secret that no service reads from the env).
from pykern import pkconfig
from pykern.pkcollections import PKDict
from pykern.pkdebug import pkdp

for e in "", r"some_secret":
    pkdp(pkconfig.to_environ(["*"], values=PKDict(foo=1, some_secret=2), exclude_re=e))

Thank you 👍


Request: vk-bootstrap vulkan library

Project description
A utility library that jump starts initialization of Vulkan. Helps writing simple boilerplate when dealing with Vulkan projects, helping beginners get into the ecosystem.
Metadata

homepage URL: https://github.com/charles-lunarg/vk-bootstrap (and possibly https://vkguide.dev/)
source URL: https://github.com/charles-lunarg/vk-bootstrap
license: mit
platforms: all (I think)

I didn't notice this but thank you so much for trying to package this for me @SCOTT-HAMILTON ! Very much appreciated! :smile:


Create get_saved_wifi_passwords.py

@lokeshsenthilkumar
please new folder with project name then add your files with readme , in readme file please explain little about your project


[WIP] Add circleci integration

Fix https://github.com/GSA/jobs_api/issues/19
Not using workflow as there's no real advantage of using it here.

Coverage remained the same at 100.0% when pulling 0076c745176b3e1a5741f4218bb6f18d829b7e6a on RapidRiverSoftware:issue-19-add_circleci into 842ad36a05474c3efb9ee2126abf083834afcd66 on GSA:master.

Removing .travis.yml doesn't seems to stop travis from building this. Probably related to this https://blog.travis-ci.com/2014-03-05-repository-settings

CircleCI is failing due to a 422 error generated by an API call made by coveralls. Most likely caused by the switch from Travis CI to CircleCI.

Repush please @viclim? I've disabled Travis CI.


TumbleBit usage on mainnet barriers

Json lib might works differently for Windows and Unix, therefore fingerprinting might be an issue.
We need a proof that the RSA pubkey of the server is indeed a pair of an RSA privkey that can blind: https://github.com/NTumbleBit/NTumbleBit/issues/72

TB dropped for https://github.com/nopara73/ZeroLink/


[MBL-1725] Update PPO card to show formatted amount with currency symbol and code

📲 What
Make it so the currency symbol/code are shown on PPOR cards for the amount pledged
🤔 Why
It was just showing a number before with no formatting
👀 See

📋 QA
Go to the PPO screen and confirm the cards show amounts with currency formatting
Story 📖
MBL-1725

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
Attention: Patch coverage is 66.66667% with 6 lines in your changes missing coverage. Please review.

Project coverage is 67.76%. Comparing base (b84688c) to head (0fefa9c).

Files with missing lines
Patch %
Lines

...java/com/kickstarter/libs/utils/RewardViewUtils.kt
73.33%
3 Missing and 1 partial :warning:

...res/pledgedprojectsoverview/data/PPOCardFactory.kt
50.00%
1 Missing :warning:

...arter/services/transformers/GraphQLTransformers.kt
0.00%
1 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff            @@
##             master    #2125   +/-   ##
=========================================
  Coverage     67.76%   67.76%
- Complexity     2220     2225    +5
=========================================
  Files           368      368
  Lines         22834    22849   +15
  Branches       3282     3282
=========================================
+ Hits          15473    15484   +11
- Misses         5598     5601    +3
- Partials       1763     1764    +1

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


Fixes and new dataset

Pods wasn't importable and was difficult to install on some machines due to a clash in the use of email.py I've moved email.py to email_.py added some missing defaults to the config, and added the leukemia dataset

I have also faced the problem with this the config file. Please approve this merge request, it will be useful for the end users.

Thanks for the bump Nishanth and thanks for the fix Alan. Merged now.


Problem with json

Hello, i love your package and i want to use it in my code, but when i try to translate a json i have these errors

And when I use your examples i have theses error like missing files


Support for splitting one window into multiple contexts

Does GLFW intend to support splitting one window into multiple contexts (i.e. subwindows)? There's a similar thing in GLUT and would be very useful in making GUI apps - for example, a game engine editor could have one context for the UI and another context for the 3D editor/playtester.
See also: http://www.lighthouse3d.com/tutorials/glut-tutorial/subwindows/

You can do this without needing multiple contexts by rendering with one context to a viewport which is a portion of the screen, then repeat with other viewports for other sections of the screen.


support for Angular v15

Which scope/s are relevant/related to the feature request?
create-analog
Information
Angular v15 has been officially released but doesn't exist in the create-analog optoions
Describe any alternatives/workarounds you're currently using
No response
I would be willing to submit a PR to fix this issue

[ ] Yes
[ ] No

Duplicate of   https://github.com/analogjs/analog/issues/116


Codewars

This pull request fixes #97, allowing teachers to assign Codewars challenges to students and monitor whether or not they have completed them.

@gkop, I reworked this a little bit. would you mind reviewing again?

+1


pixie can not collect mysql data？

i installed pixie  reference https://docs.px.dev/installing-pixie/install-guides/self-hosted-pixie/，after i deployed a test mysql in mysqlcloud namespace，and i used mysql client to access the test mysql and execute some dml，then i exec px run px/mysql_data ,but there is no info output
Pixie CLI
*******************************
* ENV VARS
*        PL_CLOUD_ADDR=dev.withpixie.dev
*******************************
Table ID: mysql_data
  TIME   SOURCE  DESTINATION  REMOTE PORT  REQ CMD  REQ BODY  RESP STATUS  RESP BODY  LATENCY

==>  Live UI: https://work.dev.withpixie.dev:443/live/clusters/kubernetes-admin@kubernetes?destination_filter=&max_num_records=1000&script=px%2Fmysql_data&source_filter=&start_time=-5m

so am i missing something? or how can i collect mysqlcloud namespace pods data?

Hi @lddlww, can you please provide more information on the client and server in this test? The px/mysql_data script will filter out anything that doesn't originate from a pod (source). Could you be running the client process outside of a pod?

can you please provide more information on the client and server in this test?

yes
kubectl get all -n mysqlcloud
NAME                         READY   STATUS    RESTARTS   AGE
pod/mysql-794776ccb5-rc876   1/1     Running   0          40h

NAME                     TYPE       CLUSTER-IP    EXTERNAL-IP   PORT(S)          AGE
service/mysql-nodeport   NodePort   <IP_ADDRESS>   <none>        3306:32143/TCP   40h

NAME                    READY   UP-TO-DATE   AVAILABLE   AGE
deployment.apps/mysql   1/1     1            1           40h

NAME                               DESIRED   CURRENT   READY   AGE
replicaset.apps/mysql-794776ccb5   1         1         1       40h

mysql -h<IP_ADDRESS> -P3306 -p
mysql> show tables;
+----------------+
| Tables_in_test |
+----------------+
| t1             |
+----------------+
mysql> delete from t1 where id=1;
Query OK, 1 row affected (0.00 sec)
mysql> select sleep(5);
+----------+
| sleep(5) |
+----------+
|        0 |
+----------+
1 row in set (5.00 sec)
mysql> select version();
+-----------+
| version() |
+-----------+
| 5.7.43    |
+-----------+
1 row in set (0.00 sec)

mysql --version
mysql  Ver 8.0.34-0ubuntu<IP_ADDRESS> for Linux on x86_64 ((Ubuntu))

i used mysql -h<IP_ADDRESS> -P3306 -p to access mysqld，and do some sqls，but there nothing info in px/mysql_data within live UI or script output


Remove the ripple effect from the switch

Hi there,
is there any way to remove the ripple effect from the switch?
Thank you

Undocumented on the props list but you can use ripple={false}. Is there a reason you wanted it disabled? It would be easy to add a global ripple option to disable everything if you are trying to go ripple free.

Ironically, the Switch didn't have the ripple effect to begin with. What you were seeing was just the focus circle.
The easiest way to remove the ripple effect would be to select the component using some CSS rules and hide the focus effect.

Hi James,
thank you for looking into this. I am on another project right now but will try the removal by CSS.

5.x.x gives you the ability to globally disable ripples

Ripples have been refactored in V5. You can disable ripples globally now using the ThemeProvider


Search for Gig Bug

###_MASTER TEMPLATE: Use for bugs, tasks, and future requests. Also, label accordingly, using the labels on the right.
###_Issue Type - is it a bug, a task, or a future request?:
A bug.
###_Severity (1-4; 1 - not severe, 4 - very severe):
3 - Pretty severe: User is not able to search for a gig.
###_Priority (1-4; 1 - lowest priority, 4 - highest priority):
3 - Big priority, but many users may not use this particular functionality.
###_Synopsis - describe bug/task/future request, in more detail:
User cannot search for a gig when they submit a search form.
###_Expected Behavior - what the behavior is supposed to be, for bug/task/FR:
You're supposed to be able to search for a gig successfully.
###_Actual Behavior - what the behavior is instead, for bugs only:
Fails to find anything went search form is submitted.
###_Steps to Repro the Issue, for bugs only:
Go to /gig-central/gig and try to search for a preexisting gig.
###_Other Notes, for bug/task/future requests:

Cannot duplicate. Search function is working for both Vanver and I.

The search Function works in my most recent build from the master branch, if you can test it on your machines with the most recent build let me know


BasicSuspension Sample Debugging Error

I was trying to build the sample of Basic Suspension . And I get this debugging error. What should I do now ? And how to do that ?

Please provide more information. Which version of the BasicSuspension sample (from which branch?). What version of Visual Studio are you using? What build of the OS are you running it on?

Here you go..............
I am using Windows Insider Preview build 14986.
Here is the link from where I downloaded the samples.........
https://github.com/Microsoft/Windows-universal-samples
Here are some photos u may need .......................

And I am using Visual Studio 2015 with latest updates

That is a very old Insider build. If you're going to use an Insider build, then please use a current one.


Image + regrid issues with data arrangements.

Applying regrid to an image fails when the data is not oriented in a particular way:
import holoviews as hv
import numpy as np
import xarray as xr
from holoviews.operation.datashader import regrid

xs = np.linspace(-5, 5, 256)[None, :]
ts = np.linspace(0, 10, 100)[:, None]
ns = np.exp(-(xs-2*np.cos(ts))**2/2.0)
data = xr.DataArray(ns, name='n', dims=('t', 'x'),
                    coords=dict(x=xs.ravel(), t=ts.ravel()))

im1 = hv.Image(data, ['t', 'x'])
im2 = hv.Image(data.T, ['t', 'x'])

(im1
  + regrid(im1)   # This is poorly behaved
  + im2
  + regrid(im2)
).cols(2)

hv.__version__, ds.__version__
(1.9.0,
 '0.6.1')

Can you try this with the current 0.6.2 datashader release, just in case?

Still an issue with 0.6.2.

Thanks, there's definitely something going wrong here, will have to investigate whether it's a holoviews or datashader issue. It seemingly works when you flip the kdims:
im1 = hv.Image(data, [ 'x', 't',]) # Works
im2 = hv.Image(data, ['t', 'x']) # This doesn't work

I think it is a holoviews issue:  The following code also has problems:
import numpy as np
import holoviews as hv
hv.notebook_extension('bokeh')

xs = np.linspace(-5, 5, 64)
ts = np.linspace(0,10,100)
ns = np.exp(-(xs[None, :]-2*np.cos(ts[:, None]))**2/2.0)
hv.Image((ts, xs, ns))

The displayed image has incorrect axes.  For example, the x axis now goes from 0 to 10, but this corresponds to size=64 dimension of the image.  My guess is that Image is not actually checking that the size of the dimensions match properly.  Passing ns.T works.
One might argue that the bug is in the previous code since the dimensions do not match given that Image interprets everything in "cartesian" coordinates likemeshgrid, but it is really puzzling when using a DataArray as above which has proper dimension labels etc.  This can cause image.sample() to fail, even when referring to coordinate by name.

There was a missing bit of error handling in one of the interfaces that allowed invalid coordinate and value array combinations to be allowed. Additionally the regrid code was not checking the xarray had the correct orientation before applying the regridding, both are now fixed and I'll add some unit tests.

Thanks again for providing a reproducible example, really helped figuring out the issues.


suggestion：When I create an avd, I want to be able to select the device of the avd

this command can get all device

avdmanager list device

create help,  use -d optional

avdmanager -h create

Usage:
      avdmanager [global options] create [action options]
      Global options:
  -s --silent     : Silent mode, shows errors only.
  -v --verbose    : Verbose mode, shows errors, warnings and all messages.
     --clear-cache: Clear the SDK Manager repository manifest cache.
  -h --help       : Help on a specific command.

Valid actions are composed of a verb and an optional direct object:
- create avd          : Creates a new Android Virtual Device.

Action "create avd":
  Creates a new Android Virtual Device.
Options:
  -c --sdcard  : Path to a shared SD card image, or size of a new sdcard for
                 the new AVD.
  -g --tag     : The sys-img tag to use for the AVD. The default is to
                 auto-select if the platform has only one tag for its system
                 images.
  -p --path    : Directory where the new AVD will be created.
  -k --package : Package path of the system image for this AVD (e.g.
                 'system-images;android-19;google_apis;x86').
  -n --name    : Name of the new AVD. [required]
     --skin    : The optional name of a skin to use with this device.
  -f --force   : Forces creation (overwrites an existing AVD)
  -b --abi     : The ABI to use for the AVD. The default is to auto-select the
                 ABI if the platform has only one ABI for its system images.
  -d --device  : The optional device definition to use. Can be a device index
                 or id.

stay tuned👍

Please check out v1.4.0. It will show the AVD device list when creating the AVD.

Thank you so much for taking my suggestion and implementing this feature.