ERROR on installing infer 0.13.1

Hi,
I am facing an issue in installing infer_v0.13.1 on UBUNTU 16.04.
After executing ./build-infer.sh Installation stops at 26%. I have noticed that complete /tmp directory space gets 100% occupied by infer installtion.  We have 1.8GB space on /tmp directory. Since we are installing infer on the VM we cannot increase the disk space on /tmp directory.
Is there any argument that can be passed for infer to change the /tmp directory(./build-infer.sh)? or Is there any workaround to fix this issue..
Note : I followed this page for infer installation.
I also tried installing infer v0.13.0 on ubuntu 16.04
But i'm facing the following error
[ERROR] ppx_driver.v0.9.1 is not available because your system doesn't comply
        with **ocaml-version >= "4.03.0" & ocaml-version < "4.05.0"**.
Exception:
Reqs_error
 (Process_error
   ("opam install -y ANSITerminal.0.7 atd.1.12.0 atdgen.1.12.0 base.v0.9.3 bin_prot.v0.9.1 biniou.1.2.0 camlp4.4.05+1 camlzip.1.07 cmdliner.1.0.2 conf-aclocal.1.0.0 conf-autoconf.0.1 conf-m4.1 conf-pkg-config.1.0 conf-which.1 configurator.v0.9.1 core.v0.9.1 core_kernel.v0.9.0 cppo.1.6.0 cppo_ocamlbuild.1.6.0 ctypes.0.13.0 easy-format.1.3.0 extlib-compat.1.7.2 fieldslib.v0.9.0 integers.0.2.2 jane-street-headers.v0.9.0 javalib.2.3.4 jbuilder.1.0+beta14 menhir.20170712 mtime.1.1.0 num.0 ocaml-compiler-libs.v0.9.0 ocaml-migrate-parsetree.1.0.6 ocamlbuild.0.11.0 ocamlfind.1.7.3 octavius.1.1.0 ounit.2.0.5 parmap.1.0-rc8 ppx_assert.v0.9.0 ppx_ast.v0.9.1 ppx_base.v0.9.0 ppx_bench.v0.9.1 ppx_bin_prot.v0.9.0 ppx_compare.v0.9.0 ppx_core.v0.9.0 ppx_custom_printf.v0.9.0 ppx_derivers.1.0 ppx_deriving.4.2 ppx_driver.v0.9.1 ppx_enumerate.v0.9.0 ppx_expect.v0.9.0 ppx_fail.v0.9.0 ppx_fields_conv.v0.9.0 ppx_hash.v0.9.0 ppx_here.v0.9.1 ppx_inline_test.v0.9.2 ppx_jane.v0.9.0 ppx_js_style.v0.9.0 ppx_let.v0.9.0 ppx_metaquot.v0.9.0 ppx_optcomp.v0.9.0 ppx_optional.v0.9.0 ppx_pipebang.v0.9.0 ppx_sexp_conv.v0.9.0 ppx_sexp_message.v0.9.0 ppx_sexp_value.v0.9.0 ppx_tools.5.0+4.05.0 ppx_traverse.v0.9.0 ppx_traverse_builtins.v0.9.0 ppx_type_conv.v0.9.0 ppx_typerep_conv.v0.9.0 ppx_variants_conv.v0.9.0 re.1.7.1 result.1.2 sawja.1.5.2 sexplib.v0.9.2 spawn.v0.9.0 sqlite<IP_ADDRESS> stdio.v0.9.0 topkg.0.9.0 typerep.v0.9.0 variantslib.v0.9.0 xmlm.1.3.0 yojson.1.4.0 --color=never",
   Unix.WEXITED 66)).

*** `install_opam_deps` failed

Has there been any changes in the latest infer version which explains why more /tmp directory space is used compared to previous version?
Thanks and Regards,
Prashanth

13.1 should be trying to use  v0.10 of core and its deps, so I don't understand why v0.9.1 is involved. Is  opam up to date on your machine?

yes opam is already updated! 1.2.2
v13.0 is trying to use v0.9.1. But, I'm not facing this issue with v0.13.1 instead I'm getting complete /tmp directory space gets 100% occupied by infer installation. We have 1.8GB space on /tmp directory. Since we are installing infer on the VM we cannot increase the disk space on /tmp directory.
Any suggestions for this issue?

1.8GB seems insufficient space to install the opam dependencies. On my machine, the infer dependencies take up about 4GB (wow!).
Is the issue about /tmp specifically, i.e., is there some other directory you could use that has more space? If so, setting TMPDIR may help.

I believe you can solve this problem by setting CLANG_TMP_DIR which is used by the clang setup script:
CLANG_TMP_DIR=/mnt/bigvolume/tmp ./build-infer.sh

If the opam library is the issue then you could also softlink that elsewhere.  Assuming you do not have an opam library or have moved it:
mkdir -p /mnt/bigvolume/.opam
ln -s /mnt/bigvolume/.opam $HOME/.opam

When I do these two things the disk space needed on my home and tmp volumes is negligible.

Hopefully the suggestions here helped. Otherwise we now publish binaries too, which have a smaller footprint. Feel free to reopen if infer takes up all your disk again.


switch to chi router in weather example

As part of Goa v3.13.0, the default router was switched to chi (Ref: https://github.com/goadesign/goa/pull/3346)
This PR updates the weather example to use chi.

Codecov Report

Merging #296 (c6e68f0) into main (c425dc9) will not change coverage.
The diff coverage is n/a.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
@@           Coverage Diff           @@
##             main     #296   +/-   ##
=======================================
  Coverage   94.26%   94.26%
=======================================
  Files          43       43
  Lines        2338     2338
=======================================
  Hits         2204     2204
  Misses         98       98
  Partials       36       36

Flag
Coverage Δ

micro
94.26% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

Thank you!


Added Okhttp and ServiceVesion into client test framework

The changes are made because we want to extend the ability of our test framework to test on Okhttp and serviceVersion and etc (may have more in future).
The change is currently made to AppConfig and KeyVault. Will run live tests for a while to learn the behavior.

/azp run java - appconfig - ci

/azp run - java - appconfig - ci

/azp run java - appconfiguration - tests

/azp run java - keyvault- tests

/azp run java - appconfiguration - ci

/azp run java - appconfiguration - tests

/azp run java - keyvault - tests

/azp run java - keyvault - tests

/azp run java - keyvault - tests

/azp run java - keyvault - ci

/azp run java - keyvault - tests


Promise.withResolvers causes type check error

I'm seeing this:
[dnt] Type checking ESM...
src/_dnt.test_polyfills.ts:1:9 - error TS2669: Augmentations for the global scope can only be directly nested in external modules or ambient module declarations.

1 declare global {
          ~~~~~~
src/_dnt.test_polyfills.ts:13:13 - error TS2339: Property 'withResolvers' does not exist on type 'PromiseConstructor'.

13 if (Promise.withResolvers === undefined) {
               ~~~~~~~~~~~~~
src/_dnt.test_polyfills.ts:14:11 - error TS2339: Property 'withResolvers' does not exist on type 'PromiseConstructor'.

14   Promise.withResolvers = () => {
             ~~~~~~~~~~~~~
src/src/taskBouncer.test.ts:38:8 - error TS2339: Property 'withResolvers' does not exist on type 'PromiseConstructor'.

38       .withResolvers<void>();
          ~~~~~~~~~~~~~

error: Uncaught (in promise) Error: Had 4 diagnostics.
          throw new Error(`Had ${diagnostics.length} diagnostics.`);
                ^
    at getProgramAndMaybeTypeCheck (https://jsr.io/@deno/dnt/0.41.3/mod.ts:468:17)
    at build (https://jsr.io/@deno/dnt/0.41.3/mod.ts:354:17)
    at eventLoopTick (ext:core/01_core.js:175:7)
    at async file:///home/uri/uriva/abstract-bot-api/build_npm.ts:8:1

meanwhile poly-filling it myself worked
// https://github.com/denoland/dnt/issues/440
const promiseWithResolver = () => {
  const obj = {};
  // @ts-expect-error no typing
  obj.promise = new Promise<void>((resolve) => {
    // @ts-expect-error no typing
    obj.resolve = resolve;
  });
  return obj as { promise: Promise<void>; resolve: () => void };
};


Wsgi failure

<IP_ADDRESS> - - [30/Jan/2016 17:51:03] "POST / HTTP/1.1" 500 -
Traceback (most recent call last):
File "/usr/local/lib/python2.7/dist-packages/flask/app.py", line 1836, in call
return self.wsgi_app(environ, start_response)
File "/usr/local/lib/python2.7/dist-packages/flask/app.py", line 1820, in wsgi_app
response = self.make_response(self.handle_exception(e))
File "/usr/local/lib/python2.7/dist-packages/flask/app.py", line 1403, in handle_exception
reraise(exc_type, exc_value, tb)
File "/usr/local/lib/python2.7/dist-packages/flask/app.py", line 1817, in wsgi_app
response = self.full_dispatch_request()
File "/usr/local/lib/python2.7/dist-packages/flask/app.py", line 1477, in full_dispatch_request
rv = self.handle_user_exception(e)
File "/usr/local/lib/python2.7/dist-packages/flask/app.py", line 1381, in handle_user_exception
reraise(exc_type, exc_value, tb)
File "/usr/local/lib/python2.7/dist-packages/flask/app.py", line 1475, in full_dispatch_request
rv = self.dispatch_request()
File "/usr/local/lib/python2.7/dist-packages/flask/app.py", line 1461, in dispatch_request
return self.view_functionsrule.endpoint
File "/home/ubuntu/jarvis/main.py", line 55, in sms
txt.append(line.split(None, 1)[0]) # add only first word

This is not even a complete traceback. This needs more information.


Support masks, refactor related code.

Fix missing transform logic on ViewportNode
Move PathFillType off of Paint
Rename encodeSVG to encodeSvg
Update some slightly incomplete documentation
Refactor DrawCommand to use nullable ids instead of -1
Add more tests.

This makes about 30 or 40 more internal test cases pass.

(this patch got bigger because flutter_svg's test cases for masks had extra features. I skipped one test case that will require more blend mode related support because it also has gradients that are currently broken)


Fix to make the project build on Windows

Fixes problem in issue #411
This is not tested on linux, but Travis should test that for us.

Coverage remained the same at 80.521% when pulling 12925d0c298a84a3d48fb099d0d116c9b8c8763a on lotorvik:develop into 013f55fa4d84845f315c7c5dad2611a5d9dbd6fc on SpareBank1:develop.

Thanks for the PR, looks good to me!
Just a minor detail:
Commit messages should contain the relevant package name when applicable, to ensure changelogs are generated correctly. (See CONTRIBUTING.md). The commit message in the autocomplete commit should read something like fix(ffe-searchable-dropdown-react): ..., if you don't mind changing it. :)

Do we need a release and changelog for this though? 🤔

Any reason not to include the change in the changelog?

Oh right.. the autocomplete stuff is in here as well... ignore me 🤡

I have now updated the comit message for the autocomplete

@lotorvik did you use --force when pushing?

@kwltrs that seems to have done the trick, thanks


New bundle: Pathology MIL classification from WSI

Fixes https://github.com/Project-MONAI/MONAI/issues/5587
This adds  a baseline model for Multiple Instance Learning (MIL) classification from Whole Slide Images (WSI), similar to the tutorial https://github.com/Project-MONAI/tutorials/tree/main/pathology/multiple_instance_learning

Hi @yiheng-wang-nv ,
Please note that this new version should not be updated to NGC, thanks.

HI @myron , for the models/pathology_multiple_instance_learning_classification/scripts/datalist_panda_0.json, is it possible to create a script that can produce the datalist (like https://github.com/Project-MONAI/model-zoo/blob/dev/models/brats_mri_segmentation/scripts/prepare_datalist.py does), or upload the file into https://github.com/Project-MONAI/MONAI-extra-test-data/releases/tag/0.8.1 (like https://github.com/Project-MONAI/model-zoo/tree/dev/models/lung_nodule_ct_detection/docs#10-fold-data-splitting)

HI @myron , for the models/pathology_multiple_instance_learning_classification/scripts/datalist_panda_0.json, is it possible to create a script that can produce the datalist (like https://github.com/Project-MONAI/model-zoo/blob/dev/models/brats_mri_segmentation/scripts/prepare_datalist.py does), or upload the file into https://github.com/Project-MONAI/MONAI-extra-test-data/releases/tag/0.8.1 (like https://github.com/Project-MONAI/model-zoo/tree/dev/models/lung_nodule_ct_detection/docs#10-fold-data-splitting)

it's possible to upload it to another place, e.g. https://github.com/Project-MONAI/MONAI-extra-test-data/releases/tag/0.8.1
But I don't understand why we do it for datalist files, it's just a list of filenames.  is it wrong to just include this small file inside of the bundle together with other custom scripts?

HI @myron , for the models/pathology_multiple_instance_learning_classification/scripts/datalist_panda_0.json, is it possible to create a script that can produce the datalist (like https://github.com/Project-MONAI/model-zoo/blob/dev/models/brats_mri_segmentation/scripts/prepare_datalist.py does), or upload the file into https://github.com/Project-MONAI/MONAI-extra-test-data/releases/tag/0.8.1 (like https://github.com/Project-MONAI/model-zoo/tree/dev/models/lung_nodule_ct_detection/docs#10-fold-data-splitting)

it's possible to upload it to another place, e.g. https://github.com/Project-MONAI/MONAI-extra-test-data/releases/tag/0.8.1 But I don't understand why we do it for datalist files, it's just a list of filenames. is it wrong to just include this small file inside of the bundle together with other custom scripts?

Hi @myron , the main purpose is for maintain. If saving to a common place, we only need to modify it once (rather than modify both tutorial and bundle).
I just checked the tutorial side, and the link of google drive is attached rather than adding the datalist file (https://github.com/Project-MONAI/tutorials/tree/main/pathology/multiple_instance_learning#train). Therefore, I suggest to use the same way.

HI @myron , for the models/pathology_multiple_instance_learning_classification/scripts/datalist_panda_0.json, is it possible to create a script that can produce the datalist (like https://github.com/Project-MONAI/model-zoo/blob/dev/models/brats_mri_segmentation/scripts/prepare_datalist.py does), or upload the file into https://github.com/Project-MONAI/MONAI-extra-test-data/releases/tag/0.8.1 (like https://github.com/Project-MONAI/model-zoo/tree/dev/models/lung_nodule_ct_detection/docs#10-fold-data-splitting)

it's possible to upload it to another place, e.g. https://github.com/Project-MONAI/MONAI-extra-test-data/releases/tag/0.8.1 But I don't understand why we do it for datalist files, it's just a list of filenames. is it wrong to just include this small file inside of the bundle together with other custom scripts?

Hi @myron , the main purpose is for maintain. If saving to a common place, we only need to modify it once (rather than modify both tutorial and bundle).
I just checked the tutorial side, and the link of google drive is attached rather than adding the datalist file (https://github.com/Project-MONAI/tutorials/tree/main/pathology/multiple_instance_learning#train). Therefore, I suggest to use the same way.

ok, I've set it up to download json file from the g-drive, @yiheng-wang-nv plz check

@yiheng-wang-nv , question about bundles.  What is the process for a user to download files in large_files.yml ? (such as pretrained weights) is there a command e.g. python -m monail.bundle download large files ??

@yiheng-wang-nv , question about bundles. What is the process for a user to download files in large_files.yml ? (such as pretrained weights) is there a command e.g. python -m monail.bundle download large files ??

Hi @myron , this format is only used by developers, and after a bundle is merged, all large files will be downloaded into the defined path and a whole bundle will be compressed for users to download directly.

Hi @myron , your bundle will be in compatible with new bundle interface (WIP, designed by Nic in the ticket mentioned above). Do you mind close this PR now, and reopen (may also need to update) it later? Thanks!


Failures during migration do not create errors

Simple example:
Adding a new column with (col) => col.setNotNull(), to a table that already has rows will fail, because the existing rows will have null values. This is OK.
However, the failure does not create any kind of error, it just fails silently, which makes it difficult to debug.
My migrations are ran with await migrator.migrateToLatest();

Tried it. Worked for me. Also, we have tests.

@koskimas What should the error look like on the terminal?
I am running on Deno with PostgresJS driver, I noticed even failure to connect to DB was not creating an error with a stack trace, it was just terminating the process.
This is without putting try/catch around the code.
I don't see tests that cover this scenario though.

Hey 👋
By design.
https://github.com/kysely-org/kysely/blob/master/src/migration/migrator.ts#L241-L257
Check the result object for error prop.

@igalklebanov thank you! I must have missed this bit.
I was just going to comment how the error can be caught inside the migration file itself but not outside.
Got it!

Btw, we have a new CLI in the works. https://github.com/kysely-org/kysely-ctl
Try it out.

I must have missed this bit.

It's the first thing we mention in the docs. You can also just hover over the function to see the docs.

Yeah it's my bad guys, sorry 😅
In part because I was rewriting the code to work well with Deno so I was a little distracted.
I want to push a PR for docs with my Deno code soon to make it up to you both.


Clean up

Summary | Résumé
Doing a little bit of clean-up, here is a summary:

update various readmes
gcds-select: rename attrsInput to attrsSelect
improve consistency for components CSS

@ethanWallace PR is updated and the checkmark position is fixed. As discussed, the old radio button was off size-wise from what it was supposed to be, I adjusted the size to the radio button size in the Figma master file.


Update of DNS servers

Added DNS servers and their description.
The following requests can be closed:
blokadaorg/blokada#241
blokadaorg/blokada#383
And Alternate DNS from this request:
blokadaorg/blokada#365

Requested changes are done.


Fix CI mypy after datasets 2.10.0 release

Fix #831.

Codecov Report
Base: 91.72% // Head: 91.65% // Decreases project coverage by -0.07% :warning:

Coverage data is based on head (425ae87) compared to base (8128ec5).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #832      +/-   ##
==========================================
- Coverage   91.72%   91.65%   -0.07%
==========================================
  Files          40       40
  Lines        2935     2985      +50
==========================================
+ Hits         2692     2736      +44
- Misses        243      249       +6

Flag
Coverage Δ

services_worker
91.65% <ø> (-0.07%)
:arrow_down:

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

Impacted Files
Coverage Δ

services/worker/src/worker/main.py
82.20% <0.00%> (-2.01%)
:arrow_down:

services/worker/tests/fixtures/hub.py
98.27% <0.00%> (+0.10%)
:arrow_up:

services/worker/tests/test_executor.py
93.06% <0.00%> (+0.75%)
:arrow_up:

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 at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


feat: add adv_2022_11

Codecov Report

Merging #36 (b67879a) into master (792a893) will decrease coverage by 0.16%.
The diff coverage is 99.21%.

@@             Coverage Diff             @@
##            master      #36      +/-   ##
===========================================
- Coverage   100.00%   99.83%   -0.17%
===========================================
  Files           11       12       +1
  Lines          474      601     +127
  Branches        87       99      +12
===========================================
+ Hits           474      600     +126
- Misses           0        1       +1

Impacted Files
Coverage Δ

solutions/adv_2022_11.py
99.21% <99.21%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Remove deprecated missing value functions

A new version of daru has been released. Make this build pass with it and I'll merge.

@v0dro I'm getting three failures. These aren't caused by missing values PR but some other change in Daru. I'm having a look. These are weird errors:
NameError: uninitialized constant Daru::Vector::ERB

I've fixed the issues for now and the build should pass now. The fix is temporary although though. See https://github.com/SciRuby/statsample/pull/55/commits/d4180335ad7e5269a5a6b36ab508902fc4f0684c

There's one more error remaining to be solved:
An error occurred while installing activesupport (<IP_ADDRESS>), and Bundler cannot continue

The problem is that activesupport requires Ruby version >= 2.2.2.

Check it now. I've released a new daru version with the necessary require statements that wern't happening since we removed all dependencies.

Yep, working fine now.


Change discord message?

⚠ Please verify that this bug has NOT been raised before.

[X] I checked and didn't find similar issue

🛡 Security Policy

[x] I agree to have read this project Security Policy

📝 Describe your problem
Is there a way to change the discord message, because when i tried it, it didn't change much
🐻 Uptime-Kuma Version
1.18.0
💻 Operating System and Arch
Debian 10
🌐 Browser
Opera GX Latest
🐋 Docker Version
No response
🟩 NodeJS Version
v16.17.0

Hello. I found lines where discord notification provider setting title to discord embeded object
I think it will help you or some one else to implement custom messages
Tell me if i wrong.

i already changed that and restarted uptime-kuma, but it didn't change anything

Did you only edit the title or all of the lines?
https://github.com/louislam/uptime-kuma/blob/9a7c2d562ab6baeea96ded002ac5b8664bfb193c/server/notification-providers/discord.js#L49-L71

A more concrete way of changing the sent embed would be nice to have. Even if it's just a text area that accepts an embed in json with say replacement words like %MONITOR_NAME% or %MONITOR_STATUS%

Please subscribe to this issue instead https://github.com/louislam/uptime-kuma/issues/1388
I think adding this would require https://github.com/louislam/uptime-kuma/issues/646 first


Feat/add meta info

With this PR I am introducing predefined queries; starting off with the INFO statement from https://surrealdb.com/docs/surrealql/statements/info.
To support tree-shakability, I have used a functional pattern.

I like the following patterns more:
function anyFn(arg1, arg2, db: Surreal = Surreal.instance) {
  // do stuff
}

Or
function anyFn(db: Surreal = Surreal.instance) {
  return (arg1, arg2) => {/*do stuff*/}
}


Allow initial self-assignemnt of UserRole

When using an external user management we need to allow users to self-assign the default role. This adds an explicit check for that to the settings service. This also means we no longer need to fiddle with the account id in the proxy upon first login.
Fixes: #5045

:boom: Acceptance test settingsUITests failed. Further test are cancelled...


Allow control of which tests get sent via which webhooks

Is your feature request related to a problem? Please describe.
I have a webhook connector to an MS Teams channel that sends all test case results to the channel.  But I have different audiences who are in different teams and channels.  There's currently no way to pick which test suites get sent via a webhook, it's all or none.
Describe the solution you'd like
I want to set up multiple webhooks and have each connected to a different channel, reporting a different set of test results in each channel.  The first part is currently possible (multiple webhooks) but it would send all the test results from every suite to each channel.
Additional context
From Slack @sureshms suggested I file this issue 😄

Addressed in new alerts overhaul. Will be part of 0.13.1


Unregister device for push notifications

Endpoint URL
TBD: DELETE /notifications/devices/{uuid}/safes/{chainId}/{address}/
Proposed payload
No needed
How it works
It should relay the request to the corresponding transaction service

Wondering about the path itself:

DELETE /notifications/devices/{uuid}/safes/{chainId}/{address}/ (current proposal)
DELETE /notifications/devices/{uuid}/{chainId}/safes/{address}/

in 1 we have the tuple chainId-safeAddress to identify a safe under /safes which makes a lot of sense specially in scenarios where the safe address is most of the times the same across different chains (given the same configuration)
In 2 we'd follow a more hierarchical approach where under the /notifications/devices/{uuid} resource: in order to identify the safe we'd have to know the chainId first and then under the /safes resource we'd identify the safe that we want with its address.
Nothing critical but wondering about the consistency of our endpoint structure so it matches how we address specific safes in different endpoints.

I just opened a ticket in order to address this: #457
I think that the solution I've seen that most of us agree on being the nicest one would look something like:
DELETE /v1/chains/<chain_id>/notifications/devices/<uuid>/safes/<address>
I suppose we can wait until today's sync to somewhat "set this on stone" though :)

Wondering about the path itself:

DELETE /notifications/devices/{uuid}/safes/{chainId}/{address}/ (current proposal)
DELETE /notifications/devices/{uuid}/{chainId}/safes/{address}/

Sorry, it was not the proposal, I just copied the transaction service endpoint, that's why I added TBD 😉


Replace react-styleguidist with docz

Docs

FEATUREREQUEST
Description:
We should try if docz is probably a good replacement for react-styleguidist.

As we could fix the performance issues. I guess it was @dnlkoch some time ago. We can close this.


tch backend interprets range argument ofTensor::arange as (start, count).

Describe the bug
Tensor::arange on tch backend interprets the range argument as (start, count) instead of (start, end). e.g. Tensor::arange(2..5) returns a tensor with values [2, 3, 4, 5, 6] when it should be [2, 3, 4] instead.
To Reproduce
Code below triggers the behavior.
use burn::tensor::backend::Backend;
use burn::tensor::{Data, Int, Tensor};
use burn_tch::TchBackend;

fn test<B: Backend<FloatElem = f32, IntElem = i64>>() {
    let idxs = Tensor::<B, 1, Int>::arange(2..5);
    println!("{:#?}", idxs);
    // Expected tensor to be [2, 3, 4], actually is [2, 3, 4, 5, 6]
    assert_eq!(idxs.into_data(), Data::from([2, 3, 4]));
}

fn main() {
    println!("===== tch backend =====");
    test::<TchBackend<f32>>();
}

@cookiemon Thanks for reporting the issue, I fixed the tch backend in this PR.


Link for XD feedback

Need it to share with the design team and otherwise...

https://xd.adobe.com/view/fae14752-50e4-414f-9fb4-a768627b6833-afb2/

I have shared it with the community. Look out for comments on it @summikhan11


Dev branch updates

new loop non-errors

@scottleibrand ah, about those x12 and smbs...I'd thought it was intentional to keep them off the preflight check since there was a (high?) likelihood that users may not be very diligent about updating their profile for basal adjustments etc.  On that same topic, is there anything preventing x12 users from autotune running?  I wouldn't think so.

I like the edit...thanks


Add activity dialog and stlye progress bars

Description
Remove activity dock widget and replace with ActivityDialog in status bar.
The ActivityDialog expands and contracts on click, and is pinned to the bottom right of the viewer canvas.

When we have multiple progress bars, the dialog becomes scrollable - not sure about this, maybe it should expand?

Watch it in action:
https://user-images.githubusercontent.com/17995243/117224022-5154df00-ae52-11eb-8008-d1e71669baf3.mp4
Type of change

[ ] 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 not work as expected)
[ ] This change requires a documentation update

References
Should merge after #2654 and #2655
How has this been tested?

[ ] example: the test suite for my feature covers cases x, y, and z
[ ] example: all tests pass with my change

Final checklist:

[ ] My PR is the minimum possible work for the desired functionality
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] If I included new strings, I have used trans. to make them localizable.
For more information see our translations guide.

Could you add some screenshots or animatins @DragaDoncila ?
Thanks :)

Could you add some screenshots or animatins @DragaDoncila ?

Done!

Looks great @DragaDoncila a bit of jumping of the content, but we can polish that on a separate PR.
Some questions:

Is Activity always there even if no progress is taking place?
Maybe we could have an icon for it instead of activity text (less discoverable though)

Is Activity always there even if no progress is taking place?
@goanpeca yes. One suggestion from @Czaki was to turn the arrow into a spinning indicator (see Plugin install indicator) when there is something in there, and otherwise just leave it as the arrow. I think that's probably the direction we want to head in.

Maybe we could have an icon for it instead of activity text (less discoverable though)
I'll have a look and see if we can find a meaningful icon. Maybe if we use an icon we can switch out that icon for the spinning indicator when there is a progress bar in there. I definitely do worry about discovery, but if we have it move when there's a progress bar in there I feel like people would click? Keen to hear what others think as well

I'm excited about this! I agree that it should be a small icon (same as plugin installer icon imho), and there should be a View > Activity panel menu option.
I think the progress bars should stack, and become scrollable after 3 nestings.
Also, @DragaDoncila, have you checked to see how these things stack with the notification manager? Since they are competing for the same space I'd like to make sure the behaviour is "sane" rather than "undefined" :joy:

@jni thanks for the feedback! A few questions and clarifications below.

I'm excited about this! I agree that it should be a small icon (same as plugin installer icon imho), and there should be a View > Activity panel menu option.

After integrating the progress indicator it looks like this:
https://user-images.githubusercontent.com/17995243/123496132-05f8c900-d66a-11eb-8540-2946fb445283.mp4
The little indicator keeps on going while there's something in the panel.
Are you saying you'd like to remove the label activity and the arrow button and just keep the indicator there as a clickable button to show the dock?

I think the progress bars should stack, and become scrollable after 3 nestings.

Do you mean the panel should expand to fit 3 progress bars, and once a 4th is added it becomes scrollable?

Also, @DragaDoncila, have you checked to see how these things stack with the notification manager? Since they are competing for the same space I'd like to make sure the behaviour is "sane" rather than "undefined"

I played around with this for a long time. The behaviour initially was the notification showed up underneath the activity dock. I've changed it now to almost always guarantee the notification shows up on top of the dock:
https://user-images.githubusercontent.com/17995243/123496284-a353fd00-d66a-11eb-816f-7262d18e7e6e.mp4
Caveats:

if a notification pops up, and then you close and open the activity dock, it reopens on top of the notification - you have to reclose the activity dock to access the notification
if the activity dock is open and you add the first layer to the viewer, the activity dock is hidden, and you have to click twice to reshow it. This happens with notifications as well. I think it's because we attach both to the canvas, and the canvas initially contains a widget overlay for the welcome screen. I tried a few different ways of fixing this but I couldn't come up with anything that both keeps the notifications/activity dock in the bottom right consistently, and doesn't hide it on that initial layer add.

Example of what I mean:
https://user-images.githubusercontent.com/17995243/123496436-69372b00-d66b-11eb-99cb-99a3874b4172.mp4

@jni I think I've addressed your comments now, would love for you to take another look.
@tlambert03 we've got some tests consistently failing - they appear unrelated but I believe it's a result of leaking widgets again. I don't know what I'm doing that's unsafe so I'd really appreciate your eyes on this.

@tlambert03 we've got some tests consistently failing - they appear unrelated but I believe it's a result of leaking widgets again. I don't know what I'm doing that's unsafe so I'd really appreciate your eyes on this.

I've fixed the tests now - the failures were a result of starting the indicator gif on creation. It would be great if someone with more Qt experience could look at the overall approach though @tlambert03 @goanpeca? @goanpeca I've also addressed your comments I think.

I've fixed the tests now - the failures were a result of starting the indicator gif on creation. It would be great if someone with more Qt experience could look at the overall approach though @tlambert03 @goanpeca?

way to go @DragaDoncila ... will have a look before meeting tomorrow

I think the arrow is kinda big for the text here:

That arrow is huge :joy: , I can assure you it does not look like that on my screen. I'll tweak that style to be closer to the expand/contract notification arrow.

I know we used the canvas as the parent for the notification toasts... which probably wasn't a great pattern, and we should change that in the future.

I agree here - it does lead to the bug where we lose the notification & activity dialog on the first layer add. I think that should be its own PR though since it'll touch the notifications as well and will need us to pull out the method for anchoring to bottom right of parent.
It would be nice if we could get it to resize horizontally as well so we don't crop. I played around with it a little bit but couldn't work out how to do it without adding a horizontal scrollbar which we definitely don't want...

regarding the scrolling discussion above... this is a tough one. I kind think it needs to grow a little bit more, or the singleStep of the scroll needs to somehow be a bit smaller.

Yes this is particularly annoying visually. It'd also be nice if for nested progress bars we remembered the position of the nested bars so that we can limit the jumping around. I'd need some help with that though, and probably we should get some design eyes on this before we go much further.

@tlambert03 can you let me know if the arrow is any smaller now? My arrow looks like this so I'm not sure if setting the icon size changed anything

Oh, I knew I'd forget something: I also think it would be nice if the activity dock auto-closed when all progress bars disappeared... Perhaps that is fraught, though.

@sofroniewn I've resolved @jni's no discussion needed comments and have captured all outstanding design suggestions in #3002 and #3003 as discussed at the meeting.
I have no idea what the new test failures are, but they are related to #2994 (which is particularly surprising to me as I haven't merged master into this branch since that PR was merged).

but they are related to #2994 (which is particularly surprising to me as I haven't merged master into this branch since that PR was merged).

maybe they relate to some upstream change, cc @tlambert03?
I just gave this a try and everything works great for me. I think there are possibly some styling improvements that we can make, but they will be much easier to make off of this foundation. I think @liaprins-czi this is something that we could put in the design queue to followup on in the future.
Great work here @DragaDoncila and thanks for all the review everyone. I will merge now 🚀


Dodo emoji not available on Windows

I'm afraid the dodo emoji used in the About section of this repo is not available on Windows 10 yet:

Bummer... I think I'll just leave it though 🤷♂ It's not that important and other platforms will catch up eventually.

Same in Linux

It doesn't matter that much and eventually it'll work for everyone so I'm not going to worry about it...

Windows 11 now supports the dodo emoji 🦤!


Add option to run bot with screen/Docker

As user @Brackyt suggested on Discord, add option to run the bot using screen/Docker instead of directly running with python/nohup

I would like to take this one up if no one's assigned.


How to prevent automatic folding symbols？

when I write haskell code in idea, it automatic folding some symbol ,which makes my code indentation messy.
Is there any way to prevent him, or can you tell me the reason for him to fold automatically like this?
quicksort :: (Ord a) => [a] -> [a]
quicksort [] = []
quicksort (x : xs) =
  let smallerSorted = quicksort [a | a <- xs, a <= x]
      bigggerSorted = quicksort [b | b <- xs, b > x]
   in smallerSorted ++ (x : bigggerSorted)

fold symbol 'let' to '├'

I'm sorry for posting the issue before uploading the image. This should be uploaded together with the image。

I'm sorry for posting the issue before uploading the image. This should be uploaded together with the image.


Document dispatch solver dependency

As an energyPATHWAYS developer-user
I want all dependencies documented
so that I can get up and running with a minimum of hassle
Depends on having a dispatch-complete version for me to test against.
AC

Document in the readme that a solver (e.g. glpk) is required for dispatch (and therefore to run Pathways) and how to install it.

Note
Theres a library called pyomo (like AMPL) that's already in the install list. It packages problem and sends it to solver. glpk is the solver we're using (there are others as well) -- is there a python library that wraps the installation, or will people need to install it separately?

Can work on this off of master. Try running energyPATHWAYS/dispatch/optimization/run_dispatch.py; it will require a solver.

Ok, I think this is taken care of on branch document_solver_22. Please review and let me know what you think.

Michael this looks good. I added an extra line of detail in the README that younger me would have appreciated. The final step will be to merge this in with Ben's latest branch, which might be be a messy merge, but not too difficult.
As far a performance, on my surface pro 4 (i7 2.20GHz) the test case runs in 61 seconds with GLPK and 40 seconds with CBC. Based on some changes Ben and I made to the formulation yesterday on his branch, I'm hoping for a further speedup.

Ok, cool. We both forgot to put the issue number in our commit messages, so for reference the commits for this are f28a100b9ffdc3926e3c9c17dbc48f812af00e95 and 857977a9eae6f801f5107db5226d1e46d5ef3322.

Great, and also for our records, I observed a number of other file changes during my git add that I believe were simply a function of rerunning the files in my environment. The results from the test, for example, probably should be added to gitignore. But for this commit, they should just be ignored.

Merged in e0430535bef5e3eec754a163c763518cb519af0d; closing.


Restructured modules to packages.

All chapter modules were converted to packages in the same module to fix the duplicate class problem. Chapter manifests were created for each chapter package.
There may be some problems with certain commits, so probably hold off on this one. In the meantime, review the updates I made to the other Classes.

This seems like there's not problems with any commits, it should be good to merge.


[ADD NEW API ENDPOINT]

Hi i just discovered new api Endpoint :
POST https://ogma.bereal.team/chat.core.v1.ChatCoreService/SendMessage //send message
GET https://ogma.bereal.team/chat.core.v1.ChatCoreService/GetMessages //get message
GET https://ogma.bereal.team/chat.core.v1.ChatCoreService/GetConversationsById //get message with id members
GET https://ogma.bereal.team/chat.core.v1.ChatCoreService/GetConversationFeed /get the feed
GET https://ogma.bereal.team/chat.core.v1.ChatCoreService/GetPendingInvites //get invitation
POST https://ogma.bereal.team/chat.core.v1.ChatCoreService/ClearConversation //clear message
POST https://ogma.bereal.team/chat.core.v1.ChatCoreService/GetSentInvites  // send invitation
Best,
Maxime

Noiceeeeee, I'll get the added to the site at some point. You're also welcome to open a PR if you want to take a stave at it?

Sure i made a PR


Test Bug 108

helloSalesforce Extension Version in VS Code: N/A
Salesforce CLI Version:                       @salesforce/cli/2.42.6 darwin-arm64 node-v18.18.2
OS and version:                            ubuntu
VS Code version: why do I need to provide this for an issue about apex-node?
Most recent version of the extensions where this was working:broken broken broken broken

Thank you for filing this issue.  We appreciate your feedback and will review the issue as soon as possible. Remember, however, that GitHub isn't a mechanism for receiving support under any agreement or SLA. If you require immediate assistance, contact Salesforce Customer Support.

Hello @daphne-sfdc :wave: It looks like you didn't provide all the required basic info in your issue.
If you haven't already, please provide the following info:
Salesforce Extension Version in VS Code:
Salesforce CLI Version:
OS and version:
VS Code version:
Most recent version of the extensions where this was working:
A few more things to check:

Make sure you've provided detailed steps to reproduce your issue, as well as all error messages that you see.

A repository that clearly demonstrates the bug is ideal.
Error messages can be found in the Problems Tab, Output Tab, and from running Developer: Toggle Developer Tools in the command palette.

Make sure you've installed the latest version of the Salesforce Extension Pack for VSCode. (docs)
Search GitHub for existing related issues.

Thank you!


Add documentation of observable groups

The documentation for the new observable groups classes  added to the toolkit in https://github.com/windows-toolkit/WindowsCommunityToolkit/pull/3201

Docs Build status updates of commit c855a4f:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/collections/ObservableGroups.md
:white_check_mark:Succeeded
View

docs/toc.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit 9b1dad5:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/collections/ObservableGroups.md
:white_check_mark:Succeeded
View

docs/toc.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


[Pods] Moving Application logic to Service

This PR moves all the logic from the Service struct to Application, making it easier to introduce the Pod struct that will subclass from Service.
Service struct has now a skeleton of the bare minimum functions required for a service to appear in the services list (and for the filters to operate correctly). Pods will implement the same fields accordingly.

This is the first of a series of PRs regarding the Pods implementation and they are aimed for discussion on the pods feature branch.

LGTM 💯 ✨

Thanks for adjusting the method name! 👍


Type errors with KeyboardAvoidingView

Describe the bug
Getting red squiggle with the <KeyboardAvoidingView> example code from the documentation here https://kirillzyusko.github.io/react-native-keyboard-controller/docs/api/components/keyboard-avoiding-view#example
This is the error I get:
ts: Type '{ children: Element; behavior: "padding"; contentContainerStyle: { flex: number; borderColor: string; borderWidth: number; }; keyboardVerticalOffset: number; style: { flex: number; borderColor: string; borderWidth: number; }; }' is not assignable to type 'IntrinsicAttributes & (PropsWithChildren<KeyboardAvoidingViewProps> & RefAttributes<View>)'.
  Types of property 'contentContainerStyle' are incompatible.
    Type '{ flex: number; borderColor: string; borderWidth: number; }' is not assignable to type 'undefined'.

Code snippet
import React from "react";
import {
  Text,
  TextInput,
  TouchableOpacity,
  View,
  StyleSheet,
} from "react-native";
import { KeyboardAvoidingView } from "react-native-keyboard-controller";

export default function KeyboardAvoidingViewExample() {
  return (
    <KeyboardAvoidingView
      behavior={"padding"}
      contentContainerStyle={styles.container}
      keyboardVerticalOffset={0}
      style={styles.content}
    >
      <View style={styles.inner}>
        <Text style={styles.heading}>Header</Text>
        <View>
          <TextInput placeholder="Username" style={styles.textInput} />
          <TextInput placeholder="Password" style={styles.textInput} />
        </View>
        <TouchableOpacity style={styles.button}>
          <Text style={styles.text}>Submit</Text>
        </TouchableOpacity>
      </View>
    </KeyboardAvoidingView>
  );
}

const styles = StyleSheet.create({
  container: {
    flex: 1,
    borderColor: "green",
    borderWidth: 2,
  },
  content: {
    flex: 1,
    borderColor: "orange",
    borderWidth: 2,
  },
  heading: {
    fontSize: 36,
    marginBottom: 48,
    fontWeight: "600",
  },
  inner: {
    padding: 24,
    flex: 1,
    justifyContent: "space-between",
  },
  textInput: {
    height: 45,
    borderColor: "#000000",
    borderWidth: 1,
    borderRadius: 10,
    marginBottom: 36,
    paddingLeft: 10,
  },
  button: {
    marginTop: 12,
    height: 45,
    borderRadius: 10,
    borderWidth: 1,
    borderColor: "red",
    backgroundColor: "rgb(40, 64, 147)",
    justifyContent: "center",
    alignItems: "center",
  },
  text: {
    fontWeight: "500",
    fontSize: 16,
    color: "white",
  },
});

Repo for reproducing
Sorry, I haven't created a separate repo for this yet, but can if needed.
To Reproduce
Steps to reproduce the behavior:

Create a new Expo app using the default template
Drop the code above into the app as a new screen
Notice TypeScript error in your IDE

Expected behavior
No red squiggle in my IDE
Screenshots
See above
Smartphone (please complete the following information):

Desktop OS: MacOS 15.1.1
Device: N/A
OS: N/A
RN version: 0.76.5
RN architecture: "newArchEnabled": false
JS engine: I don't know
Library version: 1.15.0

Additional context
Diving into the generated types for the library I see this
export type KeyboardAvoidingViewProps = KeyboardAvoidingViewBaseProps & ({
    /**
     * Specify how to react to the presence of the keyboard.
     */
    behavior?: "position";
    /**
     * Style of the content container when `behavior` is 'position'.
     */
    contentContainerStyle?: ViewProps["style"];
} | {
    /**
     * Specify how to react to the presence of the keyboard.
     */
    behavior?: "height" | "padding";
    /**
     * `contentContainerStyle` is not allowed for these behaviors.
     */
    contentContainerStyle?: never;
});

And, indeed, if I change to behavior="position" the type error goes away. So there seems to be some kind of missmatch between the documentation and the actual code 🙂

Ooops, thanks for catching @Tobbe
I think I introduced that bug here: https://github.com/kirillzyusko/react-native-keyboard-controller/pull/711 and forgot to update a documentation 😬
contentContainerStyle is needed only if behavior="position", so it's expected beavaior to see a TS error here (in this case we tel ldevs that this property will not do anything meaningful).
I'll fix it in documentation straight away! Thanks for noticing this!

Published a new version of documentation! The issue should be solved now 😎


assert function has non-standard behaviour (IDFGH-8692)

Answers checklist.

[X] I have read the documentation ESP-IDF Programming Guide and the issue is not addressed there.
[X] I have updated my IDF branch (master or release) to the latest version and checked that the issue is present there.
[X] I have searched the issue tracker for a similar issue and not found a similar issue.

General issue report
The commit https://github.com/espressif/esp-idf/commit/9afdf547481e4c5e8683f6efc8561a34c49083cc
introduced a change to https://github.com/espressif/esp-idf/blob/4b6d9c8ad317dc9e45f3c7699525dd9479f1f4c7/components/newlib/platform_include/assert.h#L34
The assert function executes the function even if NDEBUG is set. However according to http://port70.net/~nsz/c/c11/n1570.html#7.2 the assert macro has the following behavior:

1 The header <assert.h> defines the assert and static_assert macros and refers to another macro,
     NDEBUG

which is not defined by <assert.h>. If NDEBUG is defined as a macro name at the point in the source file where <assert.h> is included, the assert macro is defined simply as
#define assert(ignore) ((void)0)

The esp-idf documentation even has a warning about side effects of functions executed in assert() macro.
https://docs.espressif.com/projects/esp-idf/en/latest/esp32/contribute/style-guide.html#assertions

This looks like the same issue as reported in https://github.com/espressif/esp-idf/issues/2758.
Unfortunately we have missed this issue when planning the breaking changes in v5.0 release, and now it's a bit too late to make it.
We can consider providing some workaround to get the correct behavior on demand, and switch over to the correct behavior by default in v6.0.
I suggest to close this issue and continue the discussion in the linked one.

This looks like the same issue as reported in #2758.

Kind of looks like it. Though the assert defined in #2758 seems to have a different definition.

Unfortunately we have missed this issue when planning the breaking changes in v5.0 release, and now it's a bit too late to make it.
We can consider providing some workaround to get the correct behavior on demand, and switch over to the correct behavior by default in v6.0.

That would be nice since this behavior breaks existing code that was not written specifically for esp-idf.

I suggest to close this issue and continue the discussion in the linked one.
That's fine by me.


[Fix #9895] Disable Naming/InclusiveLanguage by default

Fixes #9895 and follow up https://github.com/rubocop/rubocop/pull/9893#issuecomment-870343243.
This PR disables Naming/InclusiveLanguage by default because it has an unexpectedly impact for many users who give feedback.

Before submitting the PR make sure the following are checked:

[x] The PR relates to only one subject with a clear title and description in grammatically correct, complete sentences.
[x] Wrote good commit messages.
[x] Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[ ] Added tests.
[x] Ran bundle exec rake default. It executes all tests and runs RuboCop on its own code.
[x] Added an entry (file) to the changelog folder named {change_type}_{change_description}.md if the new code introduces user-observable changes. See changelog entry format for details.

Either this or just disabling the checking of string literals, which I think is the real problem. I'm reasonably certain this won't generate much noise for identifier names and file names.
In hindsight - it doesn't make much sense for a cop focused on naming to check string literals, as their contents are text, not names. I should have thought of this a bit earlier. :D

We might also remove master from the defaults, although I doubt anyone names identiers master. :D

As the author of this cop, I want to endorse that I'm in support of disabling by default. I think this is a useful tool to put in user's hands but don't want to force it or a specific configuration on anyone.

@bbatsov Sure! I agree with your suggestions! I updated this PR to set CheckStrings: false and remove master from FlaggedTerms for Naming/InclusiveLanguage. I think these updates can reduce an unexpectedly impact for users much smaller. So, I reverted the cop's status to pending.

I agree with making the default configuration more permissive if this cop is going to be enabled by default. It'd be good to have an exhaustive example of how to configure the cop in the docs though so that people could, for example, just apply the original configuration if they are so inclined.


weex preview 报错

Makes sure these boxes are checked befor submitting your issue - thank you!

[] Check that your node version is >= 6 and npm version is >=4(Run npm --version and node --version).
[] If you encounter any problems using the tool, please check if there has a new version to update(Run weex update [pluginname]@latest).
[] Check the FAQ, make sure there have not the same issue.

Your development environment

weex-toolkit verison (Run weex -v)
platform (Win/Mac/Ubuntu..)

The key error stack message.
Steps to reproduce the problem.(Maybe a dotwe link,)

@gqdtl please run weex -v and then tell me your toolkit versions.

同样的问题

@erha19 安装过程还有警告
<EMAIL_ADDRESS>is incompatible with<EMAIL_ADDRESS>

我的解决了，我是电脑管家拦截的，允许就好了

@gqdtl 拦截了什么？我是mac。。

Issue activity is too low, close


Set canvas size equal to text object size

This PR (delete as applicable)

Fixes a bug

Describe the changes below:
Canvas size might less than text object size, when fixedWidth and fixedHeight is set.

Ok


Sigil rebase for expressions branch

Is there a special reason for the rebased branch? Tags 0.5.0 and 0.6.0 should have respectively the expression parser without the new sigil and both.

Oh, I did not realise you'd tagged something that had both in.
On Mon, Jun 11, 2018 at 6:42 AM Márcio Almada<EMAIL_ADDRESS>wrote:

Is there a special reason for the rebased branch? Tags 0.5.0
https://github.com/marcioAlmada/yay/releases/tag/0.5.0 and 0.6.0
https://github.com/marcioAlmada/yay/releases/tag/0.6.0 should have
respectively the expression parser without the new sigil and both.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/marcioAlmada/yay/pull/51#issuecomment-396120625, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAMPoUoNnuon1tYfktsla0dxOlVrqBb2ks5t7fVGgaJpZM4UhlPi
.


Updated basic_filter to be in line with 2.1 spec, updated tests and default_data to match spec, and fixed some issues with the memory_backend pertaining to 2.1 objects

Modified the TAXII filter to be up to spec for 2.1. Also changed the default_data to be representative of data expected on a 2.1 server, and changed some small things in the memory_backend to try to make the tests run.

Just to be clear, these commits are failing the Travis tests because the tests being used are not updated to expect 2.1 spec data nor logic. Some basic changes to the tests have been included already, but future commits will commits should contain code that tests new features and updated parts of the spec.

I have finished merging @zrush-mitre code into my PR #63. Thus, we can go ahead and close this.


SES - Request to AWS SES API failed. Reason: The Security Token included in the request is invalid

In 2.0 version have added automatically fix dynamodb, filesystem and queue AWS_SESSION_TOKEN but would be nice also fix SES service token so do not need anymore fix manually to config/services.php.
Config::set('services.ses.token', env('AWS_SESSION_TOKEN'));

Agreed. Feel free to submit a PR for this!


Import Error

Describe the bug
Importing failed
To Reproduce
Pressed like this and choose one of .glb files

Expected behavior
It should import successfully.
Screenshots

Version

OS: Windows 10 Pro
Blender Version : 2.79

This is now fixed


Backwards compatibility break in PHP/Messages constructors

Relevant PHP concepts
PHP supports 'optional arguments' (arguments given an explicit default). This means a signature can change from:
foo(string $bar)

to
foo(string $bar, int $baz=0)

and the calling code foo('hello') still works as expected
Modern PHP additionally supports 'named arguments' so the same call can be written as foo(name: 'hello') but because this is a relatively recent addition, positional arguments are often used by developers.
The problem in Messages 19.0.0
For Messages, all arguments are optional by design and in the documentation we encourage using named arguments
The new KeywordType field has changed the constructor of Step from:
    public function __construct(
        Location $location = new Location(),
        string $keyword = '',
        string $text = '',
        ?DocString $docString = null,
        ?DataTable $dataTable = null,
        string $id = '',
    ) {

to
    public function __construct(
        Location $location = new Location(),
        string $keyword = '',
        ?KeywordType $keywordType = null,
        string $text = '',
        ?DocString $docString = null,
        ?DataTable $dataTable = null,
        string $id = '',
    ) {

This means that while named arguments still work:
$step = new Step(
    keyword: 'Given',
    text: 'I have an apple',
)

The equivalent with positional arguments is now a syntax error:

$step = new Step(
    new Location(),
    'Given',
    'I have an apple', # error because this is not a KeywordType
)

Solutions
We could

Explicitly state that positional arguments are not supported. This is easy for us but relies on the developer reading the documentation and will lead to some friction
Only ever add new fields at the end of the schema; this could be enforced by adding some tests that check the objects are all constructable

I'm interested to know whether this is an issue in other languages too

Discussed whether maybe we can mitigate this by having an official way of constructing the objects that isn't a constructor (and make the constructor private/internal?)

Java has the same problem but worse.
For Java adding any field will break the constructor. So generally speaking adding any new field is a breaking change for the API, though only adding required fields will break the protocol.
A builder pattern to construct the messages would allow use to avoid this particular problem but to de-serialize messages from json the constructor would still have to be accessible. So we can't make it private either.
So while we can provide better ergonomics with builders (in java) and named arguments (in php) we can't avoid the problem entirely.

I think the better solution would be to decouple Gherkin from Messages. This would allow Cucumber implementations to align breaking changes in messages with breaking changes in their own implementation.

Decoupling Gherkin is probably a good idea, but the issue would still remain if we wanted to add a new field to something else


grader assigned to one criteria can't delete an annotation from another criteria

One of my TAs accidentally used an annotation from a different criteria for his question.  He then wasn't able to delete it.
Maybe graders should be able to delete annotations they used?

Closing this issue, as we cannot reproduce it.


Repository restructuring

This PR cleans up the examples/ directory and creates the test/ and doc/ for appropriate content.

Maybe make sense to also move cfg.json out of example folder since it is an example server configuration file, while the others are client examples?

I like that idea.  I am trying to determine where we could move it to or if it makes sense to make a subdirectory under examples/ for this.

I am ok with examples/config/. Worst case, we will move it later again :)

@ericvw Any updates on this PR? Would like to merge this also before I rebase the subscription stuff.

Rebased PR against latest develop branch with the examples/config/ changes.

LGTM. @yupinghu Any comments?

Rebased against develop again now that #7 is merged.

LGTM


ENOTFOUND Package onScreen not found

Hi,
We are getting this when we run bower install. Any idea why?

I had the same problem too with an older project that used bower.
onScreen seems to have moved from jQuery to VanillaJS and from bower to npm as a package manager .
The good news is, that the old version can still be found in the "old-jquery" branch. So if you - like me - need the jQuery/bower version you can install it with
bower install -s https://github.com/silvestreh/onScreen.git#old-jquery
and/or update you bower.json to
"onScreen": "https://github.com/silvestreh/onScreen.git#old-jquery"
Hope this helps.

@Mageking: Just like @prosellen mentioned, I moved OnScreen from Bower to NPM and, for now at least, I plan to maintain only one package manager.


Add details box to progress dialog logging each step

Add a "details box" to the progess dialog box that logs each step and each variable setting. Use logger.

Moved to https://github.com/learningsimulator/learningsimulator/issues/134


Confusing to have 2 levels to ftp site

Historically the main ftp site we made available to users was
ftp://ftp.pombase.org/pombe/
there was a level above at the ebi but we didn't really use it. Mark used to put stuff here.
Now we use this as the "entry point"
ftp://ftp.pombase.org/
which lists
archive/		2/4/19, 8:07:00 PM
external_datasets/		3/13/19, 10:45:00 AM
nightly_update/		4/30/19, 2:05:00 AM
pombe/		9/4/18, 1:00:00 AM
releases/		3/13/19, 10:19:00 AM
and has no README or instructions.
I suggest we revert to using
ftp://ftp.pombase.org/pombe/
so people see the directories they are interested in when they land. This may partly explain why people struggle to find things?
We should move the other directories (which are not advertised AFAIK) down to this  entry point and document them.

in fact
ftp://ftp.pombase.org/pombe/README
says
This is the root directory for fission yeast data files maintained by PomBase.
If you use this data in a publication please cite PomBase as described:

http://www.pombase.org/about/citing-pombase
(i.e describes as the root directory).
so I think we just linked to the incorrect directory?

It might be useful to design this page
https://www.pombase.org/datasets
so it describes all of the contents here
ftp://ftp.pombase.org/pombe/
(including any moved  in from above)
and can also be used to generate the README file so it doesn't need to be maintained in 2 places (is that possible?). If not just a link to the web page describing the director/subdirectory contents  (in a way that maps to  the current contents)

1
change link on the website
https://www.pombase.org/datasets
to
ftp://ftp.pombase.org/pombe/

Move archive
into
ftp://ftp.pombase.org/pombe/

Make external datasets a linked file from
ftp://ftp.pombase.org/pombe/

Move archive into
ftp://ftp.pombase.org/pombe/

Just to check: do we definitely want to do that?  We would need to move that directory into the pombe-embl subversion repository because the pombe directory on the FTP site is just a copy of pombe-embl/ftp_site/pombe from SVN.  The archive directory is 8GB and it would be added to everyone's checked out copy of pombe-embl next time you do an "svn update".
My vote is to add archive to SVN despite the size because I think it makes sense to keep it in SVN along with the other FTP site contents.

These changes are done:

change link on the website >https://www.pombase.org/datasets
to ftp://ftp.pombase.org/pombe/

Make external datasets a linked file from
ftp://ftp.pombase.org/pombe/
Make releases datasets a linked file from
ftp://ftp.pombase.org/pombe/

Could we just ling the archive from here:
ftp://ftp.pombase.org/pombe
as for external datasets?
Is that what you are suggesting?

I was thinking that it makes sense to add the archive to SVN but I wanted to make sure that everyone's OK with the pombe-embl directory tripling in size.  A link is the other possibility.

but I wanted to make sure that everyone's OK with the pombe-embl directory tripling in size

for me it would be mildly inconvenient but not worse; I don't really object

I  would be happy with linking if that is possible

I've added a link to the archive directory to SVN.  I'll check the FTP site on Wednesday and then close this issue.

I would be happy with linking if that is possible

I've added the link.  I think this issue is done now.

OK. I wish it didn't show the size as "0" that always makes me think they don't exist.
There is a random  .tmp file in there...

There is a random .tmp file in there...

Thanks.  I've removed it.


Exact similarity match with Redis Vector Store always retuning empty list

Please do a quick search on GitHub issues first, there might be already a duplicate issue for the one you are about to create.
If the bug is trivial, just go ahead and create the issue. Otherwise, please take a few moments and fill in the following sections:
Bug description
Exact similarity match with Redis Vector Store always retuning empty list
Environment
Spring AI version: 1.0.0-SNAPSHOT,
Java version: 17,
which vector store you use: Redis Vector Store
Steps to reproduce
FilterExpressionBuilder b = new FilterExpressionBuilder();
		SearchRequest searchRequest = SearchRequest
				.query("find by id").withFilterExpression(b.eq("id",303).build());
		List<Document> documents = vectorStore.similaritySearch(searchRequest);

Expected behavior
vectorStore.similaritySearch should return one record

Is "find by id" something that is the document, can you provide a more complete example?


Add a related article to Also See

@cjj19970505 : Thanks for your contribution! The author(s) have been notified to review your proposed change.


Keep the sources cached

Hello,
I tried ruby-build along with rbenv and I like it but I think it would be even better if it was able to cache the source code of a ruby version.
For instance, today I rebuilt my Ruby 1.9.3 many times (the first time, the second time since I din't have zlib, the third time since I didn't have openssl compilation option) and the download step is quite long and useless. I would be glad to have some cache directory in .rbenv to shorten this step.
What do you think about that? Can I help?

It'd be helpful to hang on to the tarball / git clone until it's successfully installed, but that requires significantly reworking the internals.
Life is very simple now because the entire build process is sandboxed to one temp dir, no external paths, so cleanup means just deleting that dir.
So: I'm not sure the complexity is worth it (but I'd love to be wrong about that)

So bad… :(
I had to compile Ruby again to add readline compilation option.
No easier/shorter way than recompiling Ruby for this kind of stuff?


[Improvement] multiple (programming) languages

Would be nice to assign multiple languages for a snippet.
Context: My snippets are used for multiple languages, e.g., C, C++, C#. My text editor triggers the snippet based on the active scope.

Can you just create a new multi-file snippet an create different files for each language?
For example: create a snippet named replace text and create JS file for the replace text JS version. Another file for different versions of the function in different languages?


changed readme image

Added Filler to imports from chart.js
Registered Filler
Added new screen shot to README


[Announcement] Maintenance will continue with @hello-pangea/color-picker 🎉

Hello hello!
This project has been forked and will continue to receive updates over at @hello-pangea/color-picker. Some cool improvements:

Removed and updated a ton of dependencies
Support tree shaking
Reduced bundle size by 40%
Rewrite most of the codebase in TypeScript

I want to give a massive thank you to everyone who helped to write and maintain react-color, they did amazing work.

npm install @hello-pangea/color-picker
Replace all imports of react-color with @hello-pangea/color-picker
Enjoy :)

This should be added to the README, additionally maybe npm deprecate this package with a message referring to the new package/location.

Cool to see all the new development over in the new repo. I want to second what @kdubb mentioned though, it would be good to add a note or mark this repo as readonly, and maybe pin this issue. No offense intended to you @Xhale1 but having this announcement come from someone who hadn't made any contributions to the original repository triggered my suspicions. The new repo all looks good of course, but showing some kind of "official" endorsement from this repository to use the new one would assuage a lot of concern.


TokenRateLimiter uses too much cpu above 10hz

https://github.com/golang/go/wiki/RateLimiting
High rate limits cause a huge amount of wasted CPU (8-10% above 60-100qps).  We should use a different implementation.
@ncdc @liggitt

Consider https://github.com/youtube/vitess/blob/master/go/ratelimiter/ratelimiter.go or http://godoc.org/github.com/juju/ratelimit

When OpenShift runs at idle in a Fedora 21 VM running in Parallels on my late-2012 27" iMac, it uses 25 to 35% CPU.
I swapped in juju/ratelimit to see how it compared at idle. I'm seeing it much lower - somewhere between 1 and 5 for the most part, with some bursts up to 10-15% at times.
I have not yet tested to see how effective it is at actually rate limiting, as well as if it's able to distribute the requests within a second, instead of having them all bunch at the beginning of the time period.

I just tested it, and it smooths as well as the current throttler. The only difference seems to be whether the bucket is initialized to the burst rate or not.

Actually, nevermind... juju/ratelimit initializes the bucket to capacity as well. Here's the code I used to test the initialization/smoothing behavior:
https://gist.github.com/liggitt/0bbf37f71ffb21ac169d

In general, if our rate limiter has a Stop function, I think we're doing it wrong :)


test/purge: assert database state after purging

Blocked by #1176, #1177

Perhaps more idiomatic to move assertion functions into lib/model/query/forms.js

@ktuite is this preferred to #1179?

@alxndrsn, I just wanted to check about the status of this PR. I see that the PR is approved. Is it ready to merge? Or are you waiting for feedback from @ktuite?

@matthew-white I think this is ready to merge, unless you have reservations.


ExpandIcon should have an angle field so the arrow can point left to right when expended, not only up and down

Same as https://github.com/baumths/flutter_tree_view/issues/67#issuecomment-1724699977


Can the pronunciation of the translated English be displayed?

Googletrans version:

[x] 3.8.6
[ ] 2.x

I'm submitting a ...

[ ] bug report
[x] feature request

Current behavior:

from googletrans import Translator

translator = Translator()
c = translator.translate('蘋果')        # apple
print(c.pronunciation)

Expected behavior:

ˈapəl

Steps to reproduce:

AttributeError: 'NoneType' object has no attribute 'group'

Other information:

It should be Google’s problem.

Using zh-tw to en is normal.
translator = Translator()
c = translator.translate('apple', dest='zh-tw')
print(c.pronunciation)     # Píngguǒ

It would not be possible to display unless Google supports it.


audio.metaFormat is not resolving value for m4b and m4a files, works with mp3

Tested working up to Tone 1.8, from 1.9 onwards m4b/m4a files are not resolving a value for audio.metaFormat
These are the expected returned values:
m4b = {mp4}
m4a = {mp4}
mp3 = {id3V23, id3V1}

could you provide a command and a sample file to reproduce?

Sure, I have converted the previous uploaded file to a m4b.
TestBookFolder-m4b.zip

Should be fixed in today's 0.2.2.

Confirmed working in v0.2.2, thanks.


Update MsgResGho.trans

One edited 'Anyway' word plus some corrections in same section.

"boyfan" is not a typo. It's Telly getting the word wrong, the next line confirms it. Restore that and then I'll merge. Thanks!

Edited back.


Translator Between String Substitution Systems and Wolfram Model

The Problem
There are many different patterns possible in graph evolution, but the similarities between string substitution systems and Wolfram Model systems have not been explored fully.
Possible solution
I would like to create functions that would convert string substitution systems to a wolfram model system that would result in the same graph and vice versa. A simple method would be to have each letter correspond to a number. We can have two functions, possibly ToWolframModel and ToStringSubstitution. We can also implement cyclic string substitutions for the translators since I worked on functions for cyclic string substitutions.

A simple method would be to have each letter correspond to a number.

Could you clarify why numbers would be easier to deal with than letters? One would still need to encode numbers as hypergraph edges unless pattern rules are used, in which case letters can be used directly.

We can have two functions, possibly ToWolframModel and ToStringSubstitution.

Maybe ToStringSubstitutionSystem? StringSubstitution sounds like a single event in a string substitution system.

We can also implement cyclic string substitutions for the translators since I worked on functions for cyclic string substitutions.

It would be helpful to add a link to your project here.


mpvScripts.mpris: 0.4 -> 0.5

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)
[x] Ensured that relevant documentation is up to date
[X] Fits CONTRIBUTING.md.

@GrahamcOfBorg build mpvScripts.mpris

Damn, I'm not at my best today. Sorry @GrahamcOfBorg but:
@GrahamcOfBorg build mpvScripts.mpris

@jtojnar this update includes the fix you helped me with at https://discourse.nixos.org/t/debug-why-a-derivation-has-a-reference-to-gcc/7009

ping @Mic92 ?

Result of nixpkgs-review pr 86124 1

2 packages built:
- mpvScripts.convert
- mpvScripts.mpris


Rails 4.2.3 rails engine will drop test database when running cucumber

Hello everyone,
Recently I got this issue. I move my model layer and db/migration to rails engine, and setup my test database with seed data. When I start to run cucumber, it start to drop the table and re-do migration. If I did the same this but removing the append_migrations block in engine.rb. When I start to run cucumber, it work properly.  Should I make more setting for engine, like add path or something like that?
Here is my setting: https://gist.github.com/wellsonjain/790f40a6b299f09776ce

I suggest you use the cucumber google group to get support. This is a bug tracker for verifyable bugs with the software. You'll have better luck there.
See https://cucumber.io/support


Add optional watermark on export

Fixes https://github.com/excalidraw/excalidraw/issues/1312

I would say we should add a footer and add that text there.. maybe a bit smaller! Because it might overlap with some parts of the scene if it's not in the footer.

How about this?

That's not a watermark.. it's an ad in Wired magazine :)

Thanks for the feedback so far, here's the latest version:

Based on these reviews, I assume we'll want to go ahead with this feature when the PR is approved?

I still think that the text could be a bit smaller... WDYT?

Shouldn't the setting be persisted to localStorage?
Also, doesn't work for small scenes, but I guess it doesn't matter as no one will share such tiny scenes.

Shouldn't the setting be persisted to localStorage?

That's a good idea..
As for the small scenes we can ignore this case..

Updated to store watermark state in local storage through app state.
Also handled the case of a small scene:

Thanks for the review! I've addressed those comments now.

Thank you! Looks good :)


Add minimal documentation to in ir in trustfall_core

Again from #136. These are just minimal docs, to help with understanding  trustfall_core/src/ir/

rustfmt fail was on me, my bad

Thank you for taking the time to review!

Thanks for all the awesome contributions, they are making a huge difference 🚀


default options for generate_supervisor.py

Disclaimer: I'm not really familiar with Django, so there might be something version related.
This is on a clean CentOS 7.5 box.
An incorrect supervisord.conf was being generated - it turns out there is an issue with the default value of the omitted source parameter for generate_supervisor.py
This source = options.get('source', '/opt/vespene/') is not working - source becomes None
This does work:
parser.add_argument('--source', type=str, help='source', default='/opt/vespene/')
..
source = options.get('source')

Edit: (I see you just fixed this by adding the source= parameter while I was typing this message).

Should be all fixed now, thanks again!.


added a config value for type of deployment

NOTE: This all assumes that

the current state of the deployed system has a DAP API and that it is in a "dev" stage (which it most likely is set up as).
you are logged into the aws account already

TO TEST

check out this branch
attempt a pulumi preview with everything as-is. you should see no changes in the DAP API stage resources
modify the dev config file to change the stage-name in deployment:meta to some other value (e.g. devdev)
again try pulumi preview. you should now see that it wants to replace the existing stage with a new one.
revert all changes in the local repo. you are done.

Resolved this off of the new #91 . Once that PR is merged in, this is ready to go


investigate "remaining universe" fork and marker construction

In PR #5163, @konstin brought up a concern that the way I was creating the "remaining universe" fork in the case of incomplete markers was either not correct or, perhaps, was doing more than what was needed. I spent some time trying to come up with a packse scenario that demonstrated why it was needed. My first attempt was this:
name = "fork-remaining-universe-partitioning"

[resolver_options]
universal = true

[expected]
satisfiable = true

[root]
requires = [
  "a>=2 ; sys_platform == 'windows'",
  "a<2 ; sys_platform == 'illumos'",
]

[packages.z.versions."1.0.0"]
[packages.a.versions."1.0.0"]
requires = [
  "b>=2 ; os_name == 'linux'",
  "b<2 ; os_name == 'darwin'",
  "z ; sys_platform == 'windows'",
]
[packages.a.versions."2.0.0"]
[packages.b.versions."1.0.0"]
[packages.b.versions."2.0.0"]

But as @konstin points out, it doesn't actually exercise the code path in question. I was wrong to think about this in terms of "parent" forks, since those are handled at a different point in the code. So I tried to provoke a scenario where the "remaining universe" fork needed to be constructed by starting from the existing forks, rather than creating its own. I think this requires a double sibling split, but I'm not sure. This is where I stopped:
name = "fork-remaining-universe-partitioning2"

[resolver_options]
universal = true

[expected]
satisfiable = true

[root]
requires = [
  "a>=2 ; sys_platform == 'windows'",
  "a<2 ; sys_platform == 'illumos'",
  "b>=2 ; os_name == 'linux'",
  "b<2 ; os_name == 'darwin'",
  "y ; sys_platform == 'foo'",
  "z ; os_name == 'bar'",
]

[packages.a.versions."1.0.0"]
[packages.a.versions."2.0.0"]
[packages.b.versions."1.0.0"]
[packages.b.versions."2.0.0"]
[packages.y.versions."1.0.0"]
[packages.z.versions."1.0.0"]

For now, we decided to move forward with #5163 in its current state since it fixes known bugs and doesn't introduce any known regressions. But this is something we should circle back to.

Also, I think that my next step here would be to try @konstin's suggestion in the linked PR and see if it can be knocked down.

I think this is no longer relevant with #5887 merged. Namely, while we are still dealing with incomplete markers, we are doing it more systematically in a way that deals with overlapping markers as well. So this should be fixed (if it was an issue before).


Add a ruler guide on screen

Description of the new feature / enhancement
Add a real ruler on the screen (like in the edition of the screen capture tool from Windows).
The ruler is not linked to the mouse pointer.
The ruler can be moved.
Eventually multiple guides.
Guide could be simple thin line (like ruler measure tool) or largeur translucid line to improve following and reading.
Scenario when this would be used?
Could enhance some operations when the actual ruler is not accurate:

Can help to focus on the correct line when following a process or a checking operation, long lists, etc. without having to point it with the finger.
Can help to align things on screens (vertical ruler, horizontal ruler, and eventually rotatable ruler).
Can help to have a persistent view on a measure when the mouse focus is elsewhere.
Multiple rulers guides can help to draw a grid, helping to compose a picture, an interface, etc.
Can help to place the ruler like a guide to help the actual ruler tool to measure an invisible point (space between an element and the ruler guide).

Having a ruler guide everywhere.
Dont need to rely on a third party tool.
Supporting information
The ruler doesn't have to be a real ruler, it can be a schematic ruler like the actuel ruler tool, eventually with graduation.
A simple implementation would be a large translucid line that could be moved.
Or multiple thin lines.

/dup https://github.com/microsoft/PowerToys/issues/24944


Doesn't work on Rasperry Pi 3

I've been using the module on my computer (Windows 10 Intel x86) and it worked fine. But when I deployed it to a Rasperry Pi 3 requiring this module makes it crash with the following exception:
Unhandled exception at line 234, column 9 in Unknown script code (1) 0x800a1624 - JavaScript runtime error: Reflect.apply: argument is not an array or array-like object
The code part its breaking on (which is somewhere in a module):
var currentStackTrace; function ensureStackTrace() {   if (!currentStackTrace) {     currentStackTrace = parseStack(       Reflect_apply(oldStackDesc.get, e) || '', // Call saved old getter       skipDepth, startFuncName);   }       return currentStackTrace; }
The code is running in a BasicNodejsWebserver template from VisualStudio (a UWP app that runs node code). The Rasperry Pi is running Windows 10 IoT with Node v7.0.0

Seems like it's coming from a different module, and not directly related to "wait-for-stuff".
That said - can you maybe share more details on the error:

The full error callstack
The code that you're running that triggers this error

I might be able to figure out why this is happening (and maybe even apply a fix in "wait-for-stuff").
Thx!

Maybe it has to do with Windows IoT itself, I've reached a similar issue in a Microsoft forum but failing with Express.

I couldn't get the full callstack 😞 I swear I'll try again tomorrow.
Believe it or not, the program crashes when use const wait = require('wait-for-stuff'); I don't even need to call it to make it happen.

As I promised: Here is the callstack:
TypeError: Reflect.apply: argument is not an array or array-like object    at ensureStackTrace (Unknown script code:234:9)    at stackGetter (Unknown script code:244:9)    at getFileName (C:\Data\Users\DefaultAccount\AppData\Local\Packages\NodejsWebServer1_9bmdm54xz1gf2\LocalState\node_modules\bindings\bindings.js:127:3)    at bindings (C:\Data\Users\DefaultAccount\AppData\Local\Packages\NodejsWebServer1_9bmdm54xz1gf2\LocalState\node_modules\bindings\bindings.js:55:5)    at Anonymous function (C:\Data\Users\DefaultAccount\AppData\Local\Packages\NodejsWebServer1_9bmdm54xz1gf2\LocalState\node_modules\deasync\index.js:34:3)    at Anonymous function (C:\Data\Users\DefaultAccount\AppData\Local\Packages\NodejsWebServer1_9bmdm54xz1gf2\LocalState\node_modules\deasync\index.js:9:2)    at Module.prototype._compile (module.js:560:3)    at Module._extensions[.js] (module.js:569:3)    at Module.prototype.load (module.js:477:3)    at tryModuleLoad (module.js:436:5)

Closing this for now, as I don't have a Raspberry Pi 3 available for testing this further.
Please do open a new issue if you think there's anything else I can assist with on this.

Don't worry. I don't even work on that project anymore


Questions about the version of Guppy

Hi,
I meet a problem when I try to use the Calling Methylation module according to the guidance(https://nanopolish.readthedocs.io/en/latest/quickstart_call_methylation.html)
The problem is that I miss file dna_r9.4.1_450bps_flipflop.cfg when I use Guppy. Finally, I find that it may be caused by the different versions of Guppy. So, does nanopolish only support the version of Guppy according to the guidance? Or, I just need to download dna_r9.4.1_450bps_flipflop.cfg, and different versions of Guppy can be used?

You can use any version of guppy and any config prior to nanopolish. I suggest the latest version of Guppy with the high-accuracy config.
Jared

Hi, I was following your guidance(https://nanopolish.readthedocs.io/en/latest/quickstart_call_methylation.html), so I download NA12878 WGS Consortium data provided by you, and use Guppy for base calling, the command line is guppy_basecaller -c dna_r9.4.1_450bps_flipflop.cfg -i fast5_files/ -s basecalled/.  But for -c dna_r9.4.1_450bps_flipflop.cfg, the latest version of Guppy do not have this configuration file, so can you recommend a new parameter setting way for this -c choice?

I suggest using the "hac" model for the latest version of guppy (dna_r9.4.1_450bps_hac.cfg)


Origin/feature/robot

Critical Changes
Changes
Issues Closed
New Metadata
Deleted Metadata

@davisagli these are the last set of changes i already had committed to this branch. Once these are approved, I will use the feature/robot naming convention for my branch.


⚠ gorzdrav.org has degraded performance

In 40eeff6, gorzdrav.org (https://gorzdrav.org) experienced degraded performance:

HTTP code: 200
Response time: 8717 ms

Resolved: gorzdrav.org performance has improved in aeac891.


Deck Counts Not Updating for Spoiler Combos

Deck counts aren't being pulled for combos involving spoiled cards.
See: https://discord.com/channels/752903319072276540/788902322469470278/1255980939897737286

The issue appears to be resolved.


Add Hex badge to README

I'll make a PR for this today.

Badges? We don't need no stinking badges.
Sorry couldn't help myself. What is a Hex badge, and a Travis badge for that matter?

A hex badges is a little SVG that shows the current package version and provides a link to the package page. A Travis badge shows the current test status and links to the results. We definitely don't need them, but they're fun.

Hold off on this for now, while we sort out tooling issues.

So, it looks like we're not gonna be using Hex after all. A Travis badge could be good though. It could look good to newcomers if the tests are shown to be passing :smile:

Why not use Hex?

Check out the lengthy discussion in #tooling. You'll have to scroll back quite a bit. I think it's cool to provide LFE as a package of people want to use it, but we won't be punishing packages for lfex libs or the plugins any time soon.

I saw that there was a long discussion but I signed off after the hyphens, which as you may have guessed doesn't really worry me. Are there other significant issues? One reason I am asking is that Hex is becoming a much used package manager and it wouldn't surprise me if it ends up being the default erlang one as well. If so it would be a good place for the LFE packages.

[![Hex.pm](https://img.shields.io/hexpm/v/lfe.svg)]()
[![Hex.pm](https://img.shields.io/hexpm/dt/lfe.svg)]()
[![Hex.pm](https://img.shields.io/hexpm/dw/lfe.svg)]()
[![Hex.pm](https://img.shields.io/hexpm/dd/lfe.svg)]()

I'll let @oubiwann chime in. I'm into it, but he has some reservations.


feat: token store

Handles all token in a better way. Also, shared all the tokens across tabs in a browser.

Codecov Report

Merging #20 into master will decrease coverage by 2.98%.
The diff coverage is 41.25%.

@@            Coverage Diff             @@
##           master      #20      +/-   ##
==========================================
- Coverage   72.87%   69.88%   -2.99%
==========================================
  Files          54       55       +1
  Lines         505      538      +33
  Branches       38       41       +3
==========================================
+ Hits          368      376       +8
- Misses        122      146      +24
- Partials       15       16       +1

Impacted Files
Coverage Δ

src/app/auth/token-store.service.ts
57.14% <57.14%> (ø)

src/app/auth/auth-interceptor.service.ts
85.71% <71.42%> (+14.28%)
:arrow_up:

src/app/core/io-client.service.ts
25.86% <8.82%> (-22.29%)
:arrow_down:

src/app/auth/auth.service.ts
66.66% <81.81%> (+11.34%)
:arrow_up:

Continue to review full report at Codecov.

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


Log line table goodness

Alright, I believe this is all the markers that are currently referenced in Cactbot. I'm not 100% sure on the truth of column 4, as I didn't verify each one individually, but I'm reasonably confident on most of them. Now that I've gone through and done this, I'm less sure on the value to future contributors, but oh well, we have the data now. I know there are a few markers present in game that are missing from Cactbot and the guide, but I'll probably add that to the list of things I'm planning to revisit.

Fair point on the casing. After a quick check through, it appears the guide uses lowercased references to raids, with no references to trials (that I didn't add) beyond "Shinryu Ex" and "Suzaku Ex". If you're comfortable with it, I'll change the raid references in my table to lowercase and any instance of [TRIAL] ex/Ex to [TRIAL] N/EX.

Sounds good to me, thanks!  Mostly, I just wanted to be consistent.

Alright, I think this should resolve everything. I found a couple of markers I hadn't yet included, so those are in the table now. (Acceleration Bomb is probably useless given its mechanics, but at least it's documented.)

Thanks for all this!! It's great to have a better reference.


Weird project reference

<PackageReference Include="Microsoft.TestPlatform.TestHost" Version="17.3.2" />

?!

What's the problem with it? Is it redundant?

What's the problem with it? Is it redundant?

I think so


Disable flaky tests on all OSes

Motivation:
We only had these tests disabled on windows so far but these are also flaky on linux. Let us disable these.
Modifications:
Disable flaky tests on all OSes
Result:
More stable builds

@hyperxpro nope... I was never able to have these fail on my local machine as well :/

If they dont actually fail locally, then an alternative approach might have been to disable them upon some property, and only set the property in CI, so they can at least get run somewhere at some point.


UIViewRepresentableContext

Learned that UIViewRepresentableContext provides environment values. It can/should replace the use of @Environment.
https://developer.apple.com/documentation/swiftui/uiviewrepresentablecontext

Proposed solution in dev: 5d52d3d4a0f4b41c0baac12950adb0078916ab0e.

Copying the EnvironmentValues to the Coordinator should be fine because it’s only two bytes thick (looks it stores a references to a plist and a tracker).

It works as expected and will be included in v3.1.4.

Note: I did check that EnvironmentValues has value semantics.
var apples = EnvironmentValues(); let mine = apples
apples.locale = Locale(identifier: "en_US")
String(describing: apples) == String(describing: mine) // false

Should consider if this causes additional copies of EnvironmentValues internals, however. The alternative is to copy diffableTextField_onSubmit from the environment instead. Hm.


[virtctl] Make virtctl ssh work if no id_rsa file is present

What this PR does / why we need it:
If there is no RSA key present at the default location and the
authentication with RSA was not explicitly requested, just skip it and
directly go to password authentication.
In contrast to that when a path for the RSA key was provided, fail loud
if the remote side allows RSA authentication.
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
Release note:
Skip SSH RSA auth if no RSA key was explicitly provided and not key exists at the default location

/retest

/retest

/retest

/retest

/retest

/retest

/retest

/lgtm

/approve
Thanks @vasiliy-ul

/retest

/cherrypick release-0.48


Ranger REST API missing info when GET user by name

https://issues.apache.org/jira/browse/RANGER-2737

@dargooo  - thank you for the fix. Merged this fix in master and ranger-2.4 branches.


Doc - Usage: How to execute cucumber features

Feature: Usage documentation
  As a new Rust user
  I want to complete rust-by-example, feature first, or outside-in
  So that I don't develop bad habits

Scenario:
  Given I found the cucumber-rust project on GitHub
  When I follow the Usage section instructions
  Then I see the results of the Example feature

Currently the documentation is missing instructions for the then step to be realized.

Okay so the missing instruction is:
$ cargo test --test cucumber

unfortuantely this abends:
error[E0463]: can't find crate for `cucumber_rust`
 --> 01_hello_world/tests/cucumber.rs:1:1
  |
1 | extern crate cucumber_rust as cucumber;
  | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ can't find crate

The Cargo.toml is
[package]
name = "hello-world"
version = "0.1.0"

[[test]]
name = "cucumber"
harness = false # Allows Cucumber to print output instead of libtest

[dev-dependencies]
cucumber = { package = "cucumber_rust", version = "^0.7.0" }

Fixed in https://github.com/bbqsrc/cucumber-rust/commit/6f20c30bf528b571ae6ba56246c1a7079e3f003c. Thanks!


fresh() doesn't call any lifecycle Hook

Environment
No response
Reproduction
Playground
Describe the bug
I need to listen for the lifecycle hook inside the Model when the fresh() is called , but no any lifecycle hook is listening to fresh() while if i use save() it works
For now i'm using this as a workaround
const repo = useRepo(REPO_NAME)
// clear everything
repo.flush()
// then save , i.e to trigger the lifecycle
repo.save(NEW_DATA)

ideally fresh() should also be a lifecycle hook trigger
Additional context
No response
Logs
No response

Actually, it can be useful, if you have one mutator that doesn't trigger the hooks. For example, if you need to save raw data from a backend.

@harmyderoman  I agree with you , But the issue is that in most cases we need to check the hooks ( as we're using saving
and other hooks for other functions )
May be we can add an optional raw parameter like  {raw:true} as the last config object and if raw is true we can keep it as it is , otherwise we can listen for the hooks

Hmmm interesting. I try to get my thoughts around that. You both have valid points.
I think flush is getting hooks and i also add the option to avoid hooks.

Ran into this today - I wanted to use lifecycle hooks to normalise & process raw data from the backend into a predictable and useful format, so it was unexpected that certain insertion methods simply ignore all hooks. From this perspective, IMHO this behaviour feels like a gotcha, and almost even defeats the very purpose of hooks.
I can definitely understand a use case for wanting to initiate the store with data without firing hooks - but this feels like something you should explicitly express during the action (such as the suggested {raw: true} or similar), and not inherent behaviour you need to know to override or avoid.
In the interim, this note should be added to the documentation, because neither the Inserting Data nor the Lifecycle Hooks pages describe this behaviour, and I only learned about it from this issue after spending slightly too long debugging 😅


Clean up gradle scripts

@praveek  Identity should be anandroidTestImplementation. It's corrected now. But we can't remove appcompat package since we need to use the @ VisibleForTesting annotation inside that package.


How to install the integration

Hi,
Sorry for basic question but to use this integration (RM4 Mini), it doesn't running Update directly from your repository?
If it is different method, do you have some description how to?
Thank you!!

sorry, your question is not clear
do you already have the previous version installed?

Sorry, I will ask by another way...
How to install your integration?   Updating from Repository or Copy/Paste your code?
I have anything previous version installed.

updating from repository
git pull should work in almost 100%