Passing commands to Alexa

Some ppl here complained about Alexa handling their TTS requests as speech command like "Alexa switch light x on". I'd like to actively use this in my case to run my custom skill doing stuff. Background is I want Alexa to say something when I need it to, but more as question where I can reply to and the reply triggers something. Example is I enter the home zone and get a "Welcome home, do you want your PC being switched on?" "Yeah/Nope" and it passes my answer to Home Assistant doin specific stuff. TTS does not allow that but passing my skill a secific command I can realize such behavior. Can this "mis"-behavior be added as a feature maybe?

Other then regions where TTS was broken, does anyone know of an implementation that does this? I can't seem to find anyone able to do this to try to replicate it.

I'll close this for now since I think we're stuck without some use cases.

Seems like this works now. Thx very much for adding such feature ^^ Coffee was spent

Check out if https://github.com/custom-components/alexa_media_player/wiki#run-custom-command allows this now.


Robot base drifts drastically away after sometime with warning of range_data_collator

Hey cartographer community,
I got a terrible error that my robot base drifts quickly away from the right position, this problem can be seen in the following pic. I am just wondering what is wrong because this problem doesnot happen every time, if I relaunch the whole system, it could word as wished. I noticed this problem twice, last time I just reinstalled the OS (ubuntu 16) after updating, because the problem came out after updating the OS. After reinstalling, it works again.
The problem happens with the warning of  range_data_collator.cc: Dropped 796 earlier points. And I guess, perhaps there is something wrong with the laser scanner data or other libs.
Could someone help me or is someone suffering from the same problem?
I am so appreciate for the cartographer team.

Thx.
Best regards

Please follow the default issue template instructions, in particular please post the results of cartographer_rosbag_validate.

Hey there,
I found out the problem, nothing to do with the algorithm, my computer doesnot work somehow.
thx.

@hz658832 Hey,there,
I have the same problem with you, could you please help me or let me know what's wrong with it?
I am so appreciate for you


feat!: clean up deprecated Databases and DataTypes

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)

cleanup

What is the new behavior (if this is a feature change)?

Removes migrations older than 12 months, as agreed upon previously

Other information:

Status

[ ] Code documentation for the relevant parts in the code have been added/updated by the PR author
[ ] The functionality has been tested by the PR author
[ ] The functionality has been tested by NRK

Codecov Report
Patch coverage: 100.00% and project coverage change: +0.67 :tada:

Comparison is base (987f849) 58.02% compared to head (df241e2) 58.70%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@              Coverage Diff              @@
##           release50     #899      +/-   ##
=============================================
+ Coverage      58.02%   58.70%   +0.67%
=============================================
  Files            468      465       -3
  Lines          72716    73041     +325
  Branches        3623     4492     +869
=============================================
+ Hits           42195    42878     +683
+ Misses         30470    30122     -348
+ Partials          51       41      -10

Impacted Files
Coverage Δ

meteor/server/migration/migrations.ts
100.00% <ø> (ø)

meteor/server/migration/0_1_0.ts
100.00% <100.00%> (ø)

... and 56 files with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


StackingDAO Pool TVL is wrong

StackingDAO signer key: 0x025588e24e2bf387fe8cc7bccba1aac7fe599b96724892431e992a40d06e8fe220
https://api.hiro.so/extended/v2/pox/cycles/92/signers/0x025588e24e2bf387fe8cc7bccba1aac7fe599b96724892431e992a40d06e8fe220/stackers
-> cycle 92 had 3 stackers (including SP2C2YFP12AJZB4MABJBAJ55XECVS7E4PMMZ89YZR)
https://api.hiro.so/extended/v2/pox/cycles/93/signers/0x025588e24e2bf387fe8cc7bccba1aac7fe599b96724892431e992a40d06e8fe220/stackers
-> cycle 93 only has 2 stackers
-> But SP2C2YFP12AJZB4MABJBAJ55XECVS7E4PMMZ89YZR is still stacking with the pool: https://explorer.hiro.so/address/SP2C2YFP12AJZB4MABJBAJ55XECVS7E4PMMZ89YZR?chain=mainnet

This should be fixed (can be closed), was wrongly stacked I believe by me


Make go-machine-service version available

There is no way to fid out the version of gms running on a specific host. Either through an API or just print it out to stdout on startup

Just add an env variable that starts with CATTLE_ and it will be available from the API.  Refer to the current server Dockerfile

https://github.com/rancherio/go-machine-service/pull/39 fixes this


Peek Ctrl+Space bug in Excel

Microsoft PowerToys version
0.70.0
Installation method
PowerToys auto-update
Running as admin
None
Area(s) with issue?
Peek
Steps to reproduce
Have Peek enabled with default activation shortcut (Ctrl+Space).
Open Excel, select any cell and press Ctrl+Space. Nothing happens. The shortcut combination Ctrl+Space, in Excel, should select the entire column of the active cell.
✔ Expected Behavior
In Excel, Ctrl+Space should select the entire column of the active cell.
The PowerToys Peek shortcut should not be a "global" shortcut, it should only be "picked up" when Windows Explorer is the active window.
❌ Actual Behavior
The PowerToys Peek shortcut overrules the Excel shortcut when Excel is open and the active window.
Other Software
No response

Same problem manifests with Eclipse IDE. While Peek shortcut is set to Ctrl-Space, it's not possible to assign a plain Ctrl-Space as a shortcut inside Eclipse, even though other shortcuts that contain Ctrl-Space (like Ctrl-Shift-Space, Alt-Ctrl-Space) can be assigned and used. After changing the shortcut of Peek to Win-Ctrl-Space, Ctrl-Space can used as normal inside Eclipse.

I agree.
I will disable the Peek until the shortcut for Explorer only works in Explorer.

/dup #26150


AppRoot question

I don't understand the appRoot build option.  I'm running the styleguide locally with gulp, setting the rootPath to 'styleguide'.
Ideally, I'd like to be able to run the styleguide locally building from my project, push it (project and styleguide) to gh-pages, and have it accessible from there. Is that possible?
If I set the appRoot to 'styleguide' in addition to setting the rootPath, it doesn't load locally.  Are both appRoot and rootPath supposed to be set to run on a server or can only appRoot be set?  I've tried just setting appRoot, but it doesn't resolve the file paths correctly, either.
I apologize, but I don't understand what this means:

When using the build as a subdirectory of your application, tune your server to resolve all the paths to that subdirectory. This allows Angular to deal with the routing. However, the static files should be resolved as they are stored.

Can this be done with github gh-pages, or would it have to be hosted on another server?

Hi. You will need this option if you host the result of the build somewhere in a folder. I described the case when telling how to run the Styleguide on gh-pages http://varya.me/en/posts/sc5-styleguide-for-smallers/#providing-the-approot-parameter-


How to Add New Field on Form Challenges?

Hi there..
I want to add a column in the form of challenges. For example I added a column level. How can I add it? until the database is also available at the column level.
this screenshot for template >> screenshot
I've changed the template code, but I do not know how to change the python code in admin.py & model.py
already tried adding some code, but there is an error unresolved until today.

You just need to look at models.py and look at the database schema. You'll need to understand SQLAlchemy in order to make that modification but it's fairly simply to look at what's already there and modify it.


blade syntax, head content should not be escaped.

In the readme it says to insert this into your master template

{{ Head::render() }}

However (for Laravel 5) this should become:

{!! Head::render() !!}

If it works like this i prefer not to change for the moment, but feel free to fork :-)


remove public IP from DC

Microsoft has lots of instructions on how to add IP addresses - let's leave that to Michael and me and move on.
(Though there should still be a story created for Michael to update the doc.)
Thanks,
Bill


Line jumping back and forth

Steps to reproduce
open vim with a text file having a long line (larger than window width).
split screen using :vsp
:set nowrap
Then move your cursor (via "l") along the long line (in the left window).
Once you've reached the right of the window the text of the line jumps strangely. If you do edits in this position they end up someplace else.
If you are beyond that a certain point all is fine again.
It might be related to syntax coloring.
Expected behaviour
Text under the cursors' position should correct text of the position under the cursor.
Operating system
openSUSE Leap 15.3
Version of Vim
8.0.1568
Logs and stack traces
No response

Hm, trying to reproduce, I did not notice anything.
Please try with the lastest version of Vim and start with vim --clean
and see if this still happens. A screenrecording might also help to see
what you mean with "the text of the line jumps strangely".
Also, if you suspect syntax highlighting to play a role here, please
mention the filetype for the file being edited.
Thanks,
Christian
Am 2021-11-07 11:31, schrieb StarPet:

STEPS TO REPRODUCE
open vim with a text file having a long line (larger than window
width).
split screen using :vsp
:set nowrap
Then move your cursor (via "l") along the long line (in the left
window).
Once you've reached the right of the window the text of the line jumps
strangely. If you do edits in this position they end up someplace
else.
If you are beyond that a certain point all is fine again.
It might be related to syntax coloring.
EXPECTED BEHAVIOUR
Text under the cursors' position should correct text of the position
under the cursor.
OPERATING SYSTEM
openSUSE Leap 15.3
VERSION OF VIM
8.0.1568
LOGS AND STACK TRACES
No response
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub [1], or unsubscribe
[2].
Triage notifications on the go with GitHub Mobile for iOS [3] or
Android [4].
Links:
[1] https://github.com/vim/vim/issues/9104
[2]
https://github.com/notifications/unsubscribe-auth/AAB3ZP6MF5WWW5C2O4QJHJTUKZIRRANCNFSM5HQVDW2A
[3]
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
[4]
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub

I started vim with "--clean" and it doesn't show. I'll try to figure out what combination it produces this effect.

Ok... found it (in 8.0 and 8.2). It is due to:

Terminal width = 80
set nowrap
set ss=40
set siso=15
Then (after :vsp) and going step by step to the right (using l key) it jumps. I noticed in the right lower corner extra text like 14H. I guess it is due to the fact that the line is smaller than the ss value. Usually I work with larger windows why I used ss=40.
Reducing it will avoid the effect. I guess it is a negativ value and the ANSII sequence for cursor positioning is not valid, so the 14H remains (probably -14).

Using the information in the last post, I can now see something weird:
vim -Nu NONE -S <(cat <<'EOF'
    vim9script
    "Lorem ipsum dolor sit amet, consectetur adipiscing elit, sed do eiusmod tempor incididunt ut labore et dolore magna aliqua. Ut enim ad minim veniam, quis nostrud exercitation ullamco laboris nisi ut aliquip ex ea commodo consequat. Duis aute irure dolor in reprehenderit in voluptate velit esse cillum dolore eu fugiat nulla pariatur. Excepteur sint occaecat cupidatat non proident, sunt in culpa qui officia deserunt mollit anim id est laborum."->setline(1)
    &columns = 80
    &wrap = false
    &showcmd = true
    &sidescroll = 40
    &sidescrolloff = 15
    vsplit
EOF
)


Add example to simple showcase test

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: costinm
Assign the PR to them by writing /assign @costinm in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

pkg/OWNERS
tests/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@kimikowang: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
0d26a6756b2b0f6d72368366d131052ad45a7841
link
/test istio-unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/lgtm

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: ozevren
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: costinm
Assign the PR to them by writing /assign @costinm in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

pkg/OWNERS
tests/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@kimikowang: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
8b8cb511bdbeba9574bcbdb3b54218f3f5878a2f
link
/test istio-unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


Add BSD3 license

We now got the approval to put a BSD3 license on Fractal web as well, thus making all Fractal components BSD3 licensed 🎉
@tcompa Do we need to make changes to the repo besides adding the license file?
Checklist before merging

[x] I added an appropriate entry to CHANGELOG.md

Do we need to make changes to the repo besides adding the license file?

No, thanks. Nothing else is needed.
Next up (with no special priority): https://github.com/fractal-analytics-platform/fractal-containers/issues/16


Diagnostics profile currently contains a title and help text for the Boundary system

Does this affect the legacy HoloToolkit (master) or the Mixed Reality Toolkit (mrtk_release)?
Mixed Reality Toolkit
Describe the bug
#2849 added a new configuration screen for managing the Diagnostics configuration.  However as you can see below, the Boundary config screen was copied but not updated for Diagnostics.

The Header and help text is still the boundary text.
To Reproduce
Open the Diagnostics Default profile
Use eyes
Expected behavior
Header and text should relate to the Diagnostics config
Actual behavior
Header and text are duplicated from the Boundary config
Screenshots
If applicable, add screenshots to help explain your problem.
Unity Editor Version
Which Unity version(s) you've been able to reproduce the bug in.
Mixed Reality Toolkit Release Version
Which MRTK version(s) you've been able to reproduce the bug in.
Additional context
Add any other context about the problem here.

Resolved by #2869


SITL stops with negative time

I've run into the case with jMAVSim SITL on Mac where it suddenly says:
ERROR [platforms__posix__px4_layer] WARNING! TIME IS NEGATIVE! -849055500 vs -847483835
WARN  [navigator] global position timeout

The simulation is unusable at that point.

I am encountering this error as well.
I am running Ubuntu in VirtualBox on a Mac host.  After reading this issue (https://github.com/PX4/Firmware/issues/5703) and the link describing TSC, I forced the sitl to run on a single cpu by adding taskset to sitl_run.sh.  Below is the echo of the appended command.
 SITL COMMAND: taskset -c 0 \
      /home/dev/px4/src/Firmware/build_posix_sitl_default/src/firmware/posix/px4 \
      /home/dev/px4/src/Firmware \
      /home/dev/px4/src/Firmware/posix-configs/SITL/init/ekf2/iris

I have not fully tested this but I have not seen the error since.

Should be fixed in recent versions.

It is still the same
ERROR [Unknown] WARNING! TIME IS NEGATIVE! 140078086 vs 140078087
Every time this error appear I lost the udp connection.
How can I solve ?

@docHell please create a new issue. Add as much console output as possible (in  ```), describe your version, your choice of simulator,  etc. thanks.


344 version 0020 updates

[x] #350
[x] onDelete of memory, also unset if that memory was/is active
[x] fix submit and return icon for reset password to reply or rotate-left
[x] align chat center
[x] submit button should stop scaling with size
[x] member chat input overflow handled incorrectly on extra-long text (goes offscreen)

Sorry, let's try this the other way, azure -deploy prod really seems effed up


Added brackets check, from constructor, and simplified execution

Added the from function for when you want to initialize the Machine with a specified length for the data_tape.
Implemented a brackets check while parsing the file: now brackets are matched in pairs and an error is returned if some are mismatched.
Simplified adds and subs. Simplified jumping between brackets.
Now parse_instructions resets values to support multiple executions.

Yes but you could think about extending the main to accept the length from command line. This could be useful when you want to run a program knowing that you won't use more than x cells, so it's useless to store 30_000 cells if 30_000>>x.
Also next step in my opinion would be to convert a series of > instructions in a single instructions that jumps as many times as needed. This is the same for <, + and -. You can do this by adding a (usize) in the enum kind like i did with the brackets

Let the user decide whether to input the number of cells or not. If not,
just use the default (30_000).
I will give a look at condensing instruction in a few hours. Maybe you can
do your own work on it on a different branch, and i'll do my own on another
branch. Then we can compare the two implementations and decide which one to
merge
Il mer 6 lug 2022, 11:22 Chitram Dasgupta @.***> ha
scritto:

I don't think making main() take the number of cells is a good idea.
How will the user who just wants to run their Brainfuck code know how
many cells there will be in the data_tape?
Condensing multiple + or > into one instruction is a brilliant idea! I
expect it would enhance the performance of the code greatly. Would you like
to take a hit at it?
—
Reply to this email directly, view it on GitHub
https://github.com/chitramdasgupta/brainf-interpreter/pull/1#issuecomment-1175991641,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALUHKSEBUCHQ37LRELP6253VSVF6VANCNFSM52XB2VBQ
.
You are receiving this because you authored the thread.Message ID:
@.***>

Yeah, letting the user decide whether to input the number of cells is a good idea.
I am opening an issue on condensing the >, < and the like instructions.


Improve testability with local F# runner

Just like Java impl has.

I agree we need testability and CI. Maybe use the F# project scaffolding see https://github.com/fsprojects/ProjectScaffold

This is what I'm talking about:
https://storm.apache.org/documentation/Local-mode.html
I find that any non-trivial component will have a lot of issues only surfacing when you try to run it. There's nothing in FsStorm at the moment to help with that by either making the components unit-testable (Configuration argument alone for the Local.func is really not enough) or integration-testable with the some kind of F# runtime to execute the topology inside the test.
I'm going to explore the latter approach in my own solution and if looks good merge it in here.

I have used a local storm single node cluster and attached a debugger to the spout/bolt instance to debug. But you are right that it is still cumbersome. A simple unit test framework will be very helpful.
I think the StreamParse python library has something like this. It emulates the IO messages that storm generates / consumes to unit test topology components.

I've tweaked things a bit and added samples of unit tests for components.


Projection Snapshots

More soon to come...

@jeremydmiller to me, this is an extension of the SelfAggregate. I think that minimum to support is:

do snapshot only on specific event,
do snapshot once per specific period of time,
do snapshot once per number of events.

The first one would be used for the "end of the booking day" process. The other for the cases where, e.g. there are a lot of data coming in, and we need to ingress, and we don't need to be always up to date.
Features to consider:

publishing the snapshot as the new event to the stream instead of the document. This could be useful together with the archivisation strategy. This can also give performance improvements. If we mark such an event as a snapshot, we could aggregate the stream from the last snapshot instead of loading all events.
allowing the user to define their lambda with the condition when snapshot should be made,
adding TTL of the snapshot (that could help not to need to run migration if the snapshot has the specific lifetime),
possibility to keep more than one snapshot (this would be useful for the "human readable history").


Simplify polyglot key method name lookup

Coverage increased (+0.07%) to 97.613% when pulling 36107e7eb223346306f657ce576fb3644fd83b46 on CyberiaResurrection:SimplifyPolyglotKeyMethodNameLookup into 8c1d8440c7787140357e7c278a6b2f3816680e42 on Algo-Web:master.


How to collapse the right sidebar on issues and pull requests?

How to collapse the right sidebar on issues and pull requests?

We don’t have that feature here

Thanks for that reply. Is that something we could consider for a feature request? If so I can open a new issue for that.

I don’t see the use for collapsing sections. We recently removed a feature that collapsed files too

I don’t see the use for collapsing sections. We recently removed a feature that collapsed files too

Okay, thanks for the feedback.
I suppose maybe what I am wanting is more like a zen mode for people with ADHD. For example we have a feature to remove a bunch of the issue updated noise, it would be kinda like that. To help me focus on the issue and tasks inside the issue.


Feedback on Google GenAI Library - Conversation History

Dear Google AI Team,
I am writing to provide feedback on the newly released Google GenAI library. I am an avid user of the google.generativeai library and was super excited to try out this new offering.
As a long-time user of Google GenAI having taken it out for a spin with its new features, it struck me that one very important feature for my application is seemingly missing: the keeping of conversation history. Specifically, in the google.generativeai library, this can be done via maintaining context across multiple turns of a conversation with:
chat_session = model.start_chat(history=[.])
response = chat_session.send_message(".")

and access to history can be retrieved by:
chat_session.history

The lack of such a clear history management of the conversation in Google GenAI is one of the most important limitations for the development of chatbot or conversational AI applications. Being able to keep track of-and use-past interactions is important to maintain context and assure coherence.
It's possible that the functionality is there and was overlooked in the documentation, or that there is another way to achieve similar functionality inside the library. If it is, I would be eternally grateful if someone could point me in that direction. Otherwise, if someone could confirm whether a feature for conversation history is going to be implemented in a future release of Google GenAI, that would also be great.
Thank you for your patience and consideration. I am eager to see further development in this promising library.
Best regards,
Youssef Elsafi

Great question, similar functionality is available in the chats module: https://googleapis.github.io/python-genai/genai.html#module-genai.chats Would this meet your needs?

Great question, similar functionality is available in the chats module: https://googleapis.github.io/python-genai/genai.html#module-genai.chats Would this meet your needs?

Thanks!


Add Sequel Pro template

This pull request will add template files to generate Sequel Pro colour schemes.
Examples with Base16 default:

I use Sequel Pro on my day by day work… :+1: this!

@kerunaru You can download the themes this generates at bbrks/base16-sequel-pro if you don't want to wait!

Done! :grin:

Merged in https://github.com/auduchinok/base16-builder along with other pull requests.
I'm going to maintain this project for some time, so this pull request can be closed.


Design basic structure

Consider a more fine-grained separation of content, specifically the philosophical from the practical and policy
GanttDuration: 30

Closing to test Ganttlab

reopening.... (modified Ganttlab failed test)

Resolved by new website #166

Resolved by new website #166


Why are Webpack-related packages required unconditionally in server.js?

 I’m just about to start using Webpack for my Express web app.
In server.js, I see that webpack, webpack-dev-middleware, and webpack-hot-middleware are required at the top of the file, unconditionally.
const webpack = require('webpack');
const webpackMiddleware = require('webpack-dev-middleware');
const webpackHotMiddleware = require('webpack-hot-middleware');

Shouldn’t these statements be inside the if (isDeveloping) branch?  Otherwise, Webpack runs on the actual web server in production, if I understand correctly.

@simevidas Nope, that's not a stupid question at all. :)
The requirement of these packages doesn't mean that are going to run. The actual invocation happens bellow at lines #15, #16 and still those lines are causing webpack to run. Later below, the middleware variable is being passed as a middleware (l. #29) to express.js. This means that Webpack will run once a request comes in. This happens only in development, obviously.
You could add another bootstrap file for production if you like where webpack wont being required.

Hm, if I understand correctly, require executes the module (i.e. the module’s code is parsed and compiles).
Example 1, foo is always parsed/compiled but only used in development:
let foo = require('foo');

if (development) {
  foo();
}

Example 2, foo is only parsed, compiled and used in in development:
if (development) {
  let foo = require('foo');
  foo();
}

I’m not sure why Webpack-related modules should be parsed and compiled in production. For instance, this is my own setup and it works fine for HMR:
if (process.env.NODE_ENV === 'development') {
    let webpack = require('webpack');
    let config = require('./webpack.config');
    let compiler = webpack(config);

    app.use(require('webpack-dev-middleware')(compiler));
    app.use(require('webpack-hot-middleware')(compiler));
}

Indeed they are useless since they are never going to be used in production.

It justs wastes server time and memory.
Well yes but it's only loading the source code, which is really negligible. Actually server processing and memory waste happens when you actually tell webpack to do something.

Like I said, I would separate the server.js file into two. One that will use webpack and one that would not. I would argue that you should keep your requires in one single page, on the top of the each file for consistency and to avoid hidden circular references.
But either way, your code works just fine! :)

I got curious so I run a test on my code - I’ve measured the time and memoryUsage before/after loading the 3 Webpack modules that are used for HMR:
if (process.env.NODE_ENV === 'development') {

    console.log(`Memory before: ${JSON.stringify(process.memoryUsage())}`);
    console.time('webpack');
    let webpack = require('webpack');
    let webpackDevMiddleware = require('webpack-dev-middleware');
    let webpackHotMiddleware = require('webpack-hot-middleware');
    console.timeEnd('webpack');
    console.log(`Memory after: ${JSON.stringify(process.memoryUsage())}`);

    let config = require('./webpack.config');
    let compiler = webpack(config);

    app.use(webpackDevMiddleware(compiler));
    app.use(webpackHotMiddleware(compiler));
}

The results are consistent on my machine. Here is one result:
Memory before: {"rss":28860416,"heapTotal":17862656,"heapUsed":9213872}
webpack: 749.479ms
Memory after: {"rss":44367872,"heapTotal":33591296,"heapUsed":21152024}

It looks like the Webpack modules take up ~16 MB, in addition to slowing down server boot (my app is not ready for deployment, so I can’t test on my actual web server yet).
I recommend optimizing the boilerplate, so that server resources aren’t wasted. This seems like a good idea for the default setup.

I would love to throw a PR. @christianalfoni what do you think?


Create Rangler

What this PR solves / how to test
Fixes #[insert GH or internal issue number(s)].
Author has addressed the following

Tests

[x] TODO (before merge)
[x] Included
[x] Not necessary because:

Changeset (Changeset guidelines)

[x] TODO (before merge)
[x] Included
[x] Not necessary because:

Public documentation

[x] TODO (before merge)
[x] Cloudflare docs PR(s): https://github.com/cloudflare/cloudflare-docs/pull/...
[x] Not necessary because:

Closing as I assume this was just a test...


Using this IDW Interpolator for interpolation a function

Thank you for the implementation of this interpolator.
I wanted to use this interpolator for interpolating values of certain points given initial data.
suppose the value of the function at points is:
coordinate   values of the function at the corresponding point
(1,1)            5
(1,5)            6
(2,6)            7

Now I want to estimate the value of the function at (2,3).
It would be very nice if you can share a notebook with this kind of example.
Thank you!

Hi @siddharthlal25 , did you have a chance to read the usage notebook? https://github.com/JuliaEarth/InverseDistanceWeighting.jl/blob/master/docs/Usage.ipynb
You may be interested in replacing the RegularGrid with a PointSet containing the previous points plus the point where you want to estimate. If you can start working on it, I can help you.

Also consider reaching us on our gitter channel for the project.

Closing the issue assuming the notebook helped.

Thank you! The notebook was helpful 😄


Please allow to change the color theme of primary side bar ,thanks

I would like to change the color theme of primary side bar.
I don't like black primary side bar,but I cannot change that now.

/duplicate of #174831

Thanks for creating this issue! We figured it's covering the same as another one we already have. Thus, we closed this one as a duplicate. You can search for similar existing issues. See also our issue reporting guidelines.
Happy Coding!

Thanks for quick feedback !   Sorry for this duplication ( it seems that the first summit is not successful because of my network issue).


Markdown snapshots

This PR updates test snapshot format to a subset of markdown. This allows for much nicer review and diffs. See "files changed" for a preview of the format.
This is separate to ava, which has a markdown file AND a binary file that contains the original values. In Rome's case we use the markdown as the source of truth.
Closes #148

Credit to @jamiebuilds for the idea!

Credit forwarded to AVA


can MBP M3 Pro run this cosyvoice2? can not install requirements.txt.

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

after commenting out some lines in requiremens.txt, the reference of v2 works well, while the v1 model can not work, as issue #724.
I will close this issue.


Refactor ClaimList / ClaimListDiscover and ClaimPreview / ClaimPreviewTile

These components have become incredibly messy with a lot of repeated code and without much logical organization. Refactoring this will clear up a lot of technical debt and make these particularly messy components much more maintainable and accessible.

It's been filed under https://github.com/OdyseeTeam/odysee-frontend/issues/1163


fix: update postgresql configuration

Description
Added two extra configuration log_disconnections and log_connections so that the module be able to clear db scan by tenable.
https://github.ibm.com/GoldenEye/issues/issues/9011

Release required?

[ ] No release
[ ] Patch release (x.x.X)
[x] Minor release (x.X.x)
[ ] Major release (X.x.x)

Release notes content
Added two extra configuration log_disconnections and log_connections.

Run the pipeline
If the CI pipeline doesn't run when you create the PR, the PR requires a user with GitHub collaborators access to run the pipeline.
Run the CI pipeline when the PR is ready for review and you expect tests to pass. Add a comment to the PR with the following text:
/run pipeline

Checklist for reviewers

[ ] If relevant, a test for the change is included or updated with this PR.
[ ] If relevant, documentation for the change is included or updated with this PR.

For mergers

Use a conventional commit message to set the release level. Follow the guidelines.
Include information that users need to know about the PR in the commit message. The commit message becomes part of the GitHub release notes.
Use the Squash and merge option.

/run pipeline

Maybe we should update the fscloud example to actually use some of the optional configuration parameters? Ideally set them to the same as Dreadnought have since they have done audit scanning on the Postgres instances

The dreadnaught team suggested to keep both log_connections and log_disconnections as on.

/run pipeline

Can we add variable validation for the new log_connections and log_disconnections? I'm assuming the allowed values are on and off ? There may be other parameters in that configuration map we could add validation for too?

Shall I add the validation of other parameters from here?

/run pipeline

/run pipeline

Hi @iamar7 - could we add them all - I opened this related ticket today https://github.com/terraform-ibm-modules/terraform-ibm-icd-postgresql/issues/422

/run pipeline

Hi @iamar7 - could we add them all - I opened this related ticket today #422

I saw the PR already raised for that issue here.

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


Devstep integration with Tsuru, etc

hi
just got this running, after circling back here from Tsuru and Heroku Docker (both of which I am quite new too)
am wondering how Devstep might work as a Dev Prod Parity dev tool for Tsuru
https://github.com/tsuru/tsuru/issues/1321
I made a blog post about Devstep also, will try more complex non 12 factor apps like Wordpress, Joomla, Laravel and Drupal 7/8 .. which might be more troublesome as they do not use Postgres
great work !
Happy New Year
http://niccolox.org/devstep-for-local-docker-buildpack-development/

Thanks for the post!
re: dev prod parity -> I've been thinking about making use of herokuish for a "post 1.0" release which might help. I'll keep u posted!

ok... the Tsuru peeps are working on Dockerfile slugs to Tsuru for dev prod parity
personally I am not convinced Dockerfiles will become universal
https://github.com/tsuru/tsuru/issues/1314


Subscriptions function is now getting the right amount of results

Description
The function subscriptionByChannelId didn't work properly. It was getting always the first 50 results and the user could not get more or less. The parameter 'maxResults' didn't do anything.
I reactivated the function parseSubscriptions and fixed it to get the desired amount of results instead of all of them.
To avoid confuision, the parameters is passed now as totalResults instead of maxResults since the last is the Google Api's parameter for each page, which can only be set up to 50
The Read Me was updated
Motivation and context
User was not able to get results off the first page
How has this been tested?
I listed my own channel subscriptions
Screenshots (if appropriate)
Types of changes

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

Checklist:

[X] I have read the CONTRIBUTING document.
[X] My pull request addresses exactly one patch/feature.
[X] I have created a branch for this patch/feature.
[X] Each individual commit in the pull request is meaningful.
[X] I have added tests to cover my changes.
[X] If my change requires a change to the documentation, I have updated it accordingly.

@dreenot  This is merged and Released in v1.0.2 .
Thanks for the Pull Request.


confused about viaTable()

What steps will reproduce the problem?
for example I have three tables. order , item , order_item
Class Order extends ActiveRecord
{
    public function getItems()
    {
        return $this->hasMany(Item::className(), ['id' => 'item_id'])
            ->viaTable('order_item', ['order_id' => 'id']);
    }
}
Order::findOne(1)

The querys
1.select * from order where id=1;
2.select * from order_item where order_id = 1;(empty result)
3.select * from item where 0=1; (why have this? );
I looked the source code

What is the expected result?
What do you get instead?
Additional info

Q
A

Yii version
2.0.5

PHP version
7.1.9

Operating system

3.select * from item where 0=1; (why have this? );

the condition in where could be much more complex than in your case, it is not easily possible for Yii to figure out whether a query result will be empty, so we build a query and let the database decide.
It might be possible to optimize this in some way though.


Raise error when model architecture does not match state dict

Raise an error if the size of your tensors in the loaded model don't match the input model.
Mentioned in discord.

Thanks!


FATAL_ERROR "Could not find FOUQ"

I encountered this error when compile 'mpc4j-native tool'，and I use ubuntu 22.04. I compile passed after I changed the address to absolute address. I observed that the fourq library was included under the address 'mpc4j/mpc4j-native-fourq' and there was no mention of installing fourq in compile_ubuntu.md, I did not install fourq separately. So, this problem is probably of my own making.

During the process of compiling mpc4j-native tool, I also encountered the same problem as the blogger:

According to your suggestion, I compiled mpc4j-native-fourq first and passed it successfully：

Compiling mpc4j-native tool again still reports an error：

How did you solve it? Looking forward for your reply, thank you.

Thank you for your reply, I don't understand what you mean, is that right?
------------------ 原始邮件 ------------------
发件人:                                                                                "alibaba-edu/mpc4j"                                                                                @.>;
发送时间: 2023年9月11日(星期一) 晚上8:35
@.>;
<EMAIL_ADDRESS>主题: Re: [alibaba-edu/mpc4j] FATAL_ERROR "Could not find FOUQ" (Issue #17)
You need to install fours, that is, in mpc4j-native-fourq/build, run sudo make install.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you commented.Message ID: @.***>

The problem has been solved. I performed sudo make install in my compiled file cmake-build-release, and then compiled mpc4j-native-tool again and it was successful. Thank you very much for your help
------------------ 原始邮件 ------------------
发件人:                                                                                "alibaba-edu/mpc4j"                                                                                @.>;
发送时间: 2023年9月11日(星期一) 晚上8:35
@.>;
<EMAIL_ADDRESS>主题: Re: [alibaba-edu/mpc4j] FATAL_ERROR "Could not find FOUQ" (Issue #17)
You need to install fours, that is, in mpc4j-native-fourq/build, run sudo make install.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you commented.Message ID: @.***>

I have updated README.md, telling users to install fourq. Please take a look and see if this issue has been solved and close the issue if it is.


Feat: changed delete user to be token-based confirmation

What type of PR is this? (check all applicable)

[X] Refactor
[ ] Feature
[ ] Bug Fix
[ ] Optimization

Description
Change the user delete logic to be token-based confirmation.
Related Tickets & Documents

Related Issue #277
Closes #277

Added/updated tests?
We encourage you to keep the code coverage percentage at 80% and above.

[X] Yes
[ ] No, and this is why: please replace this line with details on why tests
have not been included
[ ] I need help with writing tests

Hey @saksham-malhotra-27!
When you have some time, can you check if this logic is right and will be enough for your implementation, please, feel free to recommend and suggest any modification.
Thank you in advance.

@FelipePSoares Ya, I checked and it looks perfect to me,
Just a thing that I noticed that you’ve added the token generator in the delete API. Have you also implemented an API to confirm if the user is an admin, or does it directly show a warning about being an admin before deleting the account?

@saksham-malhotra-27
The process will work like that, to delete a user we need to call twice the same endpoint:
First call you will receive a confirmationMessage and a confirmationToken:
DELETE /api/account
---
202 Accepted
{
    "confirmationMessage": "This message will change based on user",
    "confirmationToken": "eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9..."
}

This confirmationMessage should be showed to the user, this message changes based if the user is admin or not of some project.
The Second call is to confirm the delete:
DELETE /api/account?confirmationToken=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9...
---
200 OK

You will call the same endpoint but now sending the confirmationToken as query parameter.
Do you got it?

@FelipePSoares Yes, I understand your point. The work is perfect; however, I would suggest the following:
Preferred Approach: Pass the token in the request body.
{
    "confirmationToken": "eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9..."
}

Reason: Passing the token in the request body is more secure because it prevents the token from being exposed in logs, browser history, or bookmarks. Unlike query parameters, which are part of the URL and might be unintentionally logged by servers or shared in URLs, the request body remains private and is less likely to be exposed. This approach ensures better security and reduces the risk of accidental token leakage.

@saksham-malhotra-27
I make the change, can you confirm that is everything OK now.
Thank you for your suggestion.

Yup @FelipePSoares ! Good to go

@saksham-malhotra-27
If you know how to do it you can approve the review. Than it will merge Automatically.
Otherwise I can do it later.

Thanks


A11y: Improve keyboard interaction on carousel

Bug Description
There are two issues with the page carousel:

Only the current page should be in the tab order - otherwise, if you have a 20-page story, you'll tab for a long to get from the canvas to the design panel, as you have to go through all the pages in the process.
The current page should have a clear focus indicator if focused - and this should be implemented in both collapsed carousel, regular carousel, and in the page grid view.

Expected Behaviour

Steps to Reproduce

Screenshots

Additional Context

Plugin Version:
Operating System:
Browser:

Do not alter or remove anything below. The following sections will be managed by moderators only.
Acceptance Criteria

Implementation Brief

This covers #3653

Tested as per instructions in 4248:
Testing Instructions
In story editor, tab through page and see that only active page indicator is focusable out of the page indicators, also that it's visually clear that the page indicator is focused.
In the story editor, click the grid view. In the grid view tab through the page and see that only the active page indicator is focusable out of the page indicators and that it's visually clear that the page indicator is focused.
Verified in QA


[rebranch][ClangImporter] Update tests for conflicting swift_name attributes

The upstreaming of attribute((__swift_name__)) in LLVM makes the compiler raise errors on conflicting attributes. This PR updates the tests to reflect that change with the help of a new compiler flag -verify-additional-file implemented by @brentdax.
rdar://72473419

I'm not sure the underlying clang change--that is, turning this warning into a hard error--is a great idea; it could create new build failures in existing projects. But given that clang is doing that, this pull request LGTM, and I'm glad my commit helped!

I'm not sure the underlying clang change--that is, turning this warning into a hard error--is a great idea; it could create new build failures in existing projects. But given that clang is doing that, this pull request LGTM, and I'm glad my commit helped!


Disable Travis; upgrade MkDocs (SOFTWARE-4173)

Still a bunch of broken links to sort through

No CI after disabling Travis and this huge changeset is cruel to ask to be reviewed so I'm going to merge it :crossed_fingers:


Remove no need space & lines

What

Remove no need lines and spaces

Why

Document will be greater

Thanks for contribution


Intractable graph size scaling for large proteins

Hi,
As I understand it one of the end goals of ESPALOMA is to be able to parameterise an entire system without the need for individual residue templates (what got me excited about it in the first place - the promise of straightforward handling of new covalent modifications to protein or DNA residues is particularly alluring). Unfortunately it looks like that won't be possible with the current implementation for any but the smallest proteins, due to the scaling in size of the heterograph with number of atoms. Reading in a protein from PDB with:
 import espaloma as esp
 from openff.toolkit import Topology
 top = Topology.from_pdb('protein.pdb')
 mol = top.molecule(0)
 mgraph = esp.Graph(mol)

My first attempt (with a ~700-residue protein) was killed by the Linux OOM killer after chewing > 22 GB of system RAM. Trying with a series of smaller poly-A models (replication files attached):
import espaloma as esp
from openff.toolkit import Topology

with open('graph_size.csv', 'wt') as out:
    print('Residues,Atoms,Nodes,Edges', file=out)
    for res_count in (5,10,20,30,40,50,75,100):
        top = Topology.from_pdb(f'ala{res_count}.pdb')
        mol = top.molecule(0)
        mgraph = esp.Graph(mol)
        het = mgraph.heterograph
        print(f'{res_count},{mol.n_atoms},{het.number_of_nodes()},{het.number_of_edges()}', file=out)

... shows the node and edge count both scaling as O(n**2) - a 1,003-atom model gives a heterograph with just over a million nodes and 6 million edges. This seems excessive to me, but I don't yet understand enough about the architecture to know the reasons for it. Extrapolating out, a (still pretty reasonably-sized) 10k-atom protein would yield a graph with about 100 million nodes and 600 million edges.

Can you shed some light on what's going on, and do you have any ideas on how to improve on this?
espaloma_polya.tar.gz

We have developmental code to represent each residue as an individual RDKit molecule, like a "chorizo" in which each residue is one of the links. Residues carry a few extra atoms to model the chemistry of adjacent residues, as well as lists of atom indices to keep track of what's "real" and what's padding. It would be a bit of work to generalize it but as it stands we get at least espaloma charges for entire proteins and nucleic acids.

That's one sensible solution, which may in fact be preferable in an interactive environment like where I want to apply it, since it'd cut down the cost of reparameterising after modifications to a large protein... just re-do the affected region, rather than the whole thing. But I'm more curious about whether there are ways to improve this scaling in the first place... without digging deeply into the code, a naive interpretation of this would suggest that each individual atom is getting its own all-atom subgraph. That doesn't feel right to me, but it's totally possible I'm missing something fundamental.

@tristanic I can reproduce your results, and indeed it's creating pretty big graphs. I checked the code that's consuming most of the the memory and it boils down to this line. Unfortunately, I don't see how we could make that line consume significantly less memory, especially with the restrictions that DGL is already imposing.
In private communications with @yuanqing-wang I think he has proposed ways to modify the architecture for this to be more memory efficient, but I don't think that's a quick fix/thing to do right now.

@ijpulidos thanks for the feedback. We'll have a think about what to do
next.
On Tue, Jun 11, 2024 at 10:27 PM Iván Pulido @.***>
wrote:

@tristanic https://github.com/tristanic I can reproduce your results,
and indeed it's creating pretty big graphs. I checked the code that's
consuming most of the the memory and it boils down to this line
https://github.com/choderalab/espaloma/blob/cb8e5b23e3ec1ada356128debc6a2a5511ef0b98/espaloma/graphs/utils/read_heterogeneous_graph.py#L272.
Unfortunately, I don't see how we could make that line consume
significantly less memory, especially with the restrictions that DGL is
already imposing.
In private communications with @yuanqing-wang
https://github.com/yuanqing-wang I think he has proposed ways to modify
the architecture for this to be more memory efficient, but I don't think
that's a quick fix/thing to do right now.
—
Reply to this email directly, view it on GitHub
https://github.com/choderalab/espaloma/issues/217#issuecomment-2161624848,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFM54YFWHT23EN3XHBFHSXDZG5TV3AVCNFSM6AAAAABI2VVZMSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNRRGYZDIOBUHA
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
Altos Labs UK Limited | England | Company reg 13484917
Registered
address: 3rd Floor 1 Ashley Road, Altrincham, Cheshire, United Kingdom,
WA14 2DT


add new analysis techniques

Add published matrix factorization algorithm (https://ieeexplore.ieee.org/document/8038072)
Add mass univariate analysis (defined in, e.g., https://openwetware.org/wiki/Mass_Univariate_ERP_Toolbox).
Add dictionary learning (defined in,e.g., https://team.inria.fr/panama/projects/please/dictionary-learning/)).

@UlrikeS91 could you review this PR? Let me know if you are too busy at the moment.

@UlrikeS91 don't worry and thank you :smiley:


Consider adding focusproperty, similar to Quick's fit or focused it

quickly prepending focus to a property should work like Quick's fit, narrowing tests run only to focus'ed properties.

SwiftCheck isn't a complete replacement for XCTest.  We still use Apple's mechanism to get the tests actually run.  As such this falls outside the scope of the framework.

this makes sense. I'll try using Quick for that :+1:


CB-18795 Create pre and post usersync endpoint

Exposing WAG creation with pre usersync API, and sudo rule creation as post usersync API  for WIAM. The new endpoint are internal only.
The actual logic is running async, so WIAM should check the operation to see if it's finished. Same operation type (USER_SYNC) is used during the new endpoints, so there shouldn't be conflicting usersyncs.
CDP_ENVIRONMENT_PRIVILEGED_USER is enabled in mock-thunderhead to enable testing.
See detailed description in the commit message.

also, do you have any concerns about using the same task executor that runs the usersyncs? e.g., if these tasks get stuck on the task queue behind long-running usersyncs

Do these endpoints need to be async? The amount of work for each endpoint should not be large and making these async complicates the overall system.

unfortunately there are a bunch of external calls, especially for post usersync there are multiple freeipa calls, with possible retries. I think this could result in minutes long request which is not ideal

also, do you have any concerns about using the same task executor that runs the usersyncs? e.g., if these tasks get stuck on the task queue behind long-running usersyncs

not really, with separating the usersync threadpools earlier and addig timeouts, I think we are safe for now. if you think about it, the pressure would be lower on these threadpools as we start using WIAM for usersync


Cloud: unable to find all occurrences of searched item in logs

What's your CLI version?
N/A
Description & steps to reproduce
When I try to find a certain log entry in https://cloud.kurtosis.com/enclave-manager/enclave/66a897a354f1/service/151acad62d58/logs I can't seem to find all occurrences of searched object with cmd+f.
Desired behavior
Would be good to either include a search box in the logs tab, or enable me to search the whole logs file with cmd+f.
What is the severity of this bug?
Critical; I am blocked and Kurtosis is unusable for me because of this bug.
What area of the product does this pertain to?
Frontend: the Graphical User Interface, including the Enclave Manager

Hey @barnabasbusa thanks for filing! The way the frontend currently works is that it loads the first couple thousand (I believe it's 2k) loglines and then cmd-f will operate on that. Is the logline you're looking for from beyond the first 2k lines? Or, is the bug related to a match that you're seeing in the frontend not getting matched?

Repro'd with Barnabas - the search terms that are loaded in the frontend, but not immediately visible, aren't getting matched:

it appears that when you search it only highlights what's in the viewable port. Seems to be a limitation with react-virtuoso? virtuoso doesn't seem to have any native search capabilities. @Dartoxian may be we'll need a custom search function?

yep I think you're right @adschwartz . I think it would be reasonable to intercept ctrl+f and provide search capability in the log view, then control scrolling react-virtuoso ourselves to navigate results..


NetworkTransform rotation compression removed. Gets rid of endless discussions, optimizations and issues. Sending a quaternion is never going to fail.

@MrGadget1024 good point. serialization in separate PR though

@MrGadget1024 good point. serialization in separate PR though

You have to keep FloatBytePackerTest.cs too


Add conditional feature support for python bindings and more LLVM versions

I'm working on integrating these crates into some of our QIR work, and it would be nice to not include the Python bindings in places where they are not needed.
Additionally, at least one of our projects allows the use of LLVM version up to 12 or 13, so this PR also includes more features in each crate that uses llvm-ir or inkwell to support those versions too.
All default-features have been kept unchanged, so the crates should function precisely the same as if none of the new features were added.
I've confirmed, at least locally, that our downstream projects work as expected when using/disabling these features.

Thanks for the PR @nilslice . This PR would close #35. In order to fully add this, a few more changes are required:

Use-LlvmInstallation in utils.ps1 needs to use the correct env variable based on the llvm version
Update docs for env vars that need to get set for llvm linking (LLVM_SYS_*)
Update build.rs with llvm tags based on features that work. Something like:

fn get_llvm_tag() -> String {
    if let Ok(tag) = env::var("QIRLIB_LLVM_TAG") {
        tag
    } else {
        if cfg!(feature = "llvm-11") {
            "llvmorg-11.1.0".to_owned() // 1fdec59bf
        } else if cfg!(feature = "llvm-12") {
            "llvmorg-12.0.1".to_owned() // fed4134
        } else if cfg!(feature = "llvm-13") {
            "llvmorg-13.0.1".to_owned() // 75e33f7
        } else if cfg!(feature = "llvm-14") {
            "llvmorg-14.0.3".to_owned() // 1f91400
        } else {
            panic!(
                "Unsupported LLVM version. The LLVM feature flags or QIRLIB_LLVM_TAG must be set."
            )
        }
    }
}

Update the pipelines to create a feature matrix across the LLVM versions setting QIRLIB_LLVM_TAG for each build. These versions haven't been tested and we don't know if they will compile and run cleanly.

Additionally, we need to figure out:

What version of LLVM is the default?
What version of LLVM is used in the published packages?
How long do we maintain support for various LLVM versions?

/CC @swernli @bettinaheim

@nilslice Based on your PR I have changes with llvm 13 building. I'm merging #111 which has a lot of build cleanup that we can use as a better base for these changes. We have to also bring llvm-sys as an optional dependency for the all-targets-init suppression.


Document how users can make Identify an off-panel widget

Add documentation on how to make identify an off-panel widget with our new point and rectangle options.  The step should include code snippets that can be added to the config.json file.

You can only do point.  This will only be addressed if a user requests  how to do this.

Closing as a known limit.  These widgets are only samples.  There is nothing preventing a user from using online help to do this.


Vertical scroll

Hi guys, first of all, congrats for this job, its amazing.
I am wondering if its possible to scroll vertically and swipe horizontally.
I am asking this because, when the card is too big I cannot see all the content.
And I couldn't find any way to solve this :(
thanks!

Unfortunately the way the swiper is constructed I don't think so. You might achieve this if you disable vertical swipes (verticalSwipe) and release the pan responder when vertical swipe occurs so that it is caught by the inner element.
One of the reasons why we started working on our project with a custom swiper (rather than this package) back 2 years ago was exactly this one.
Another option would be to fiddle with the verticalThreshold, and once a vertical swipe is achieved duplicate the deck, show an identical card (that's not part of the deck) on top, and that would be a scrollable element. But this won't work if users tries to swipe right afterwards.
In order to obtain the desired effect, the core of the swiper should be refactored all together. Not sure if anyone else would be interested. I'll leave this issue open, if the vertical thing is requested enough it might get some attention in the future.

I also had this same problem, but for me the following solved the issue

set verticalSwipe={false}
On your renderCard Component

`
import {
widthPercentageToDP as wp,
heightPercentageToDP as hp
} from 'react-native-responsive-screen';

         ...contents

</TouchableWithoutFeedback>

`
The paddingBottom is quiet important, as for me, the contents were being cut after a certain amount of scroll, but the padding allowed it to be pushed to further

Query for anyone that might be able to advise, I'm aiming to not actually use this library for users to swipe with gestures but only interact programmatically through two buttons for left/right, so I'm disabling verticalSwipe and horizontalSwipe.
I want the cards to be vertically scrollable, theres a bit of weirdness (not entirely broken, just slightly), in that the card component (I matched it to @PeterHjHan to try, whilst just rendering a long <Text> component in the ...content and same issue persists) in that the ScrollView feels less sensitive, on initial load i can scroll just fine, but after a moment I have to scroll very fast to get it to register the onScroll event, its like theres an increase in the threshold required to register a scroll on the Card, I've tried playing around with the props as much as I can but no success, any ideas?

@PeterHjHan
Thanks for your workaround

The paddingBottom is quiet important, as for me, the contents were being cut after a certain amount of scroll, but the padding allowed it the bottom contents to be seen

using height: '100%' in cardStyle prop fixed the cut issue in my case
cardStyle={{
  height: '100%'
}}

In order to obtain the desired effect, the core of the swiper should be refactored all together. Not sure if anyone else would be interested. I'll leave this issue open, if the vertical thing is requested enough it might get some attention in the future.

I don't know if such a drastic change is required, when I put a scrollview in there as one of the cards, I can sometimes get scrolling behaviour if I swipe quickly and vertical swipe is disabled. I think putting in an option for a threshold before the swipe is registered might fix this: i.e. the listener only fires if swipe dx > threshold.

in node_modules/react-native-deck-swiper/Swiper.js
I just add to function pushCardToStack Animated.ScrollView and contentContainerStyle={{paddingBottom: 50}}
and it's worked for me. I know its bad but i dont see other way
pushCardToStack = (renderedCards, index, position, key, firstCard) => {     const { cards } = this.props     const stackCardZoomStyle = this.calculateStackCardZoomStyle(position)     const stackCard = this.props.renderCard(cards[index], index)     const swipableCardStyle = this.calculateSwipableCardStyle()     const renderOverlayLabel = this.renderOverlayLabel()     renderedCards.push(       <Animated.ScrollView         key={key}         style={firstCard ? swipableCardStyle : stackCardZoomStyle}         {...this._panResponder.panHandlers}         howsVerticalScrollIndicator={false}         contentContainerStyle={{paddingBottom: 50}}       >           {firstCard ? renderOverlayLabel : null}           {stackCard}       </Animated.ScrollView>     )   }

IT WORKED

in node_modules/react-native-deck-swiper/Swiper.js
I just add to function pushCardToStack Animated.ScrollView and contentContainerStyle={{paddingBottom: 50}}
and it's worked for me. I know its bad but i dont see other way
pushCardToStack = (renderedCards, index, position, key, firstCard) => {
const { cards } = this.props
const stackCardZoomStyle = this.calculateStackCardZoomStyle(position)
const stackCard = this.props.renderCard(cards[index], index)
const swipableCardStyle = this.calculateSwipableCardStyle()
const renderOverlayLabel = this.renderOverlayLabel()
renderedCards.push(
<Animated.ScrollView
key={key}
style={firstCard ? swipableCardStyle : stackCardZoomStyle}
{...this._panResponder.panHandlers}
howsVerticalScrollIndicator={false}
contentContainerStyle={{paddingBottom: 50}}

{firstCard ? renderOverlayLabel : null}
{stackCard}
</Animated.ScrollView>
)
}

Give this man a medal
also @jaroslav009 you don't need to add paddingBottom:50 if your card has a flex container as the parent element i.e.

//rest of card stuff

const styles = StyleSheet.create({
container: {
flex: 1,
display: "flex",
flexDirection: "column",
justifyContent: "center",
backgroundColor: 'white'
}
)
It makes it a little bit harder to swipe, so I pulled the horizontalSwipeThreshold all the way down to 30, and it's perfect

in node_modules/react-native-deck-swiper/Swiper.js
I just add to function pushCardToStack Animated.ScrollView and contentContainerStyle={{paddingBottom: 50}}
and it's worked for me. I know its bad but i dont see other way
pushCardToStack = (renderedCards, index, position, key, firstCard) => {
const { cards } = this.props
const stackCardZoomStyle = this.calculateStackCardZoomStyle(position)
const stackCard = this.props.renderCard(cards[index], index)
const swipableCardStyle = this.calculateSwipableCardStyle()
const renderOverlayLabel = this.renderOverlayLabel()
renderedCards.push(
<Animated.ScrollView
key={key}
style={firstCard ? swipableCardStyle : stackCardZoomStyle}
{...this._panResponder.panHandlers}
howsVerticalScrollIndicator={false}
contentContainerStyle={{paddingBottom: 50}}

{firstCard ? renderOverlayLabel : null}
{stackCard}
</Animated.ScrollView>
)
}

Give this man a medal
also @jaroslav009 you don't need to add paddingBottom:50 if your card has a flex container as the parent element i.e.

< View style={styles.container}>
//rest of card stuff
 </View>

const styles = StyleSheet.create({
container: {
      flex: 1,
      display: "flex",
      flexDirection: "column",
      justifyContent: "center",
    }
)

It makes it a little bit harder to swipe, so I pulled the horizontalSwipeThreshold all the way down to 30, and it's perfect

thanks)

Just as a note incase any of you are editing your node_modules directly, use the npm module patch-package - allows you to great git diffs so that if in future you do an npm install the 'patches' will be automatically applied after the installation is complete - avoids you having to manually re-apply and gives you a history of whats been edited :)

Been working on this for 24 hours...
Thanks so much for putting this solution. 🚀  Seriously...not sure what I would have done without. Can I buy you a beer? 🍺
One thing to add...for those that want it, you can add bounces={false} makes it a little more responsive on the horizontal swipe after hitting the end with a vertical swipe.

in node_modules/react-native-deck-swiper/Swiper.js
I just add to function pushCardToStack Animated.ScrollView and contentContainerStyle={{paddingBottom: 50}}
and it's worked for me. I know its bad but i dont see other way
pushCardToStack = (renderedCards, index, position, key, firstCard) => {
const { cards } = this.props
const stackCardZoomStyle = this.calculateStackCardZoomStyle(position)
const stackCard = this.props.renderCard(cards[index], index)
const swipableCardStyle = this.calculateSwipableCardStyle()
const renderOverlayLabel = this.renderOverlayLabel()
renderedCards.push(
  <Animated.ScrollView
    key={key}
    style={firstCard ? swipableCardStyle : stackCardZoomStyle}
    {...this._panResponder.panHandlers}
    howsVerticalScrollIndicator={false}
    contentContainerStyle={{paddingBottom: 50}}
  >
      {firstCard ? renderOverlayLabel : null}
      {stackCard}
  </Animated.ScrollView>
)

}

One thing to add...for those that want it, you can add bounces={false} makes it a little more responsive on the horizontal swipe after hitting the end with a vertical swipe.
Also...just realized that Im not getting any success with Android. ios works beautifully. Did it work for you on Android with the Animated.ScrollView...it's completely not pressable for me on my Galaxy S8.

I've not had any issues with this on Android (I test most of my stuff on Android first), it could be a style issue. Flex and layout styles will sometimes behave differently on Android vs iOS.

Hey @TahaAttari can you paste here your implementation and more specifically the styles you used?

adding this inside my <ScrollView> on my card render solved it for me without having to patch the package, thanks!!!

adding <TouchableWithoutFeedback> inside my <ScrollView> on my card render solved it for me without having to patch the package, thanks!!! 🚀

How can I use a flatList with this fix? @harveyappleton

Just confirming that this is still an issue?  I guess I will manually implement a deck-swiper, but would be amazing if someone had come up with a fix for this

Just confirming that this is still an issue? I guess I will manually implement a deck-swiper, but would be amazing if someone had come up with a fix for this

https://github.com/webraptor/react-native-deck-swiper/issues/22#issuecomment-1416528741


Add vaccination status to risk profile

Could you add "fully vaccinated" to the risk profile drop down? Not sure how this would work for groups. For larger group sizes, could you add a way to include the % of the group that is fully vaccinated? (In OR this is available by zip code, even if you don't know the status of everyone near you.) Without this info, it is hard to use the calculator for situations that will happen all summer like "6 fully vaccinated adults outside and 6 unvaccinated children w/ or w/o masks"

Closing as duplicate of #808