Web - Login - Blank white screen after login

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Expected Result:
User should be redirected to chat page
Actual Result:
Blank white screen displays after login
Action Performed:

Open https://estaging.expensify.cash/
Login with existing account
Observe page

Workaround:
Unknown
Platform:
Where is this issue occurring?
Web ✔
iOS
Android
Desktop App
Mobile Web
Version Number: 1.0.15-0
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos:
This was consistently reproducible by only 1 tester. Same tester was unable to reproduce this in production.
JS error
[Error] TypeError: undefined is not an object (evaluating 'e.personalDetails[t.actorEmail]') â€” ReportActionItemSingle.js:25
	es (app-ccbb8ba39129ab5728c2.bundle.js:57:82634)
	(anonymous function) (app-ccbb8ba39129ab5728c2.bundle.js:57:89412)
	pi (app-ccbb8ba39129ab5728c2.bundle.js:57:48801)
	is (app-ccbb8ba39129ab5728c2.bundle.js:57:83982)
	fc (app-ccbb8ba39129ab5728c2.bundle.js:57:101491)
	fc
	(anonymous function) (app-ccbb8ba39129ab5728c2.bundle.js:65:4088)
	pc (app-ccbb8ba39129ab5728c2.bundle.js:57:97963)
	Zs (app-ccbb8ba39129ab5728c2.bundle.js:57:94117)
	Zs
	(anonymous function) (app-ccbb8ba39129ab5728c2.bundle.js:57:45559)
	(anonymous function) (app-ccbb8ba39129ab5728c2.bundle.js:65:4088)
	Go (app-ccbb8ba39129ab5728c2.bundle.js:57:45505)
	Ho (app-ccbb8ba39129ab5728c2.bundle.js:57:45440)
	Ys (app-ccbb8ba39129ab5728c2.bundle.js:57:90703)
	enqueueSetState (app-ccbb8ba39129ab5728c2.bundle.js:57:49225)
	(anonymous function) (app-ccbb8ba39129ab5728c2.bundle.js:49:1552)
	Xt (app-ccbb8ba39129ab5728c2.bundle.js:85:259175)
	(anonymous function) (app-ccbb8ba39129ab5728c2.bundle.js:85:261084)
	promiseReactionJob
[Error] Unhandled Promise Rejection: TypeError: undefined is not an object (evaluating 'e.personalDetails[t.actorEmail]')
	Go (app-ccbb8ba39129ab5728c2.bundle.js:57:45648)
	Ho (app-ccbb8ba39129ab5728c2.bundle.js:57:45440)
	Ys (app-ccbb8ba39129ab5728c2.bundle.js:57:90703)
	enqueueSetState (app-ccbb8ba39129ab5728c2.bundle.js:57:49225)
	(anonymous function) (app-ccbb8ba39129ab5728c2.bundle.js:49:1552)
	Xt (app-ccbb8ba39129ab5728c2.bundle.js:85:259175)
	(anonymous function) (app-ccbb8ba39129ab5728c2.bundle.js:85:261084)
	promiseReactionJob

Video

Expensify/Expensify Issue URL:

Since this was only affecting one tester, we're not going to treat this as a deploy blocker. If we get more cases of this happening we'll get to the bottom of it.

Assigning myself to this because I fixed it in another PR and I've run into it quite a few times on dev in the past few days.

LInking other https://github.com/Expensify/Expensify.cash/issues/2520 https://github.com/Expensify/Expensify.cash/issues/2497. sounds similar.

Sorry, I don't think they are


Waiting for this to be added to PyPI

Hi @coagulant with your permission and for easy installation, I'll upload to PypI a fork of this APP. I need install and test this on heroku.

You should ask @jbonfante, as he is maintaining this fork.


Does this have push notifications working on Android with AOT and Trimming on?

Thanks for putting the work into finding a workable solution.
My current issue with Android Push Notifications in Firebase is that they do not work with AOT and Trimming on (which are on by default in Release mode)
They simply do not arrive. There are multiple reports of people having the issue.
I've tried several things to get it up and running.
Repro: Deploy in Release mode on Android phone. Launch app. Close app. Send push notification to device. (nothing happens in my case)
The million dollar question: Do they work for you?

You should consider disable AOT & trimming for now, It's just a headache to check if every piece are together while publishing an ap app. I would wait at least .NET 9 to try enable those options in MAUI
https://github.com/TobiasBuchholz/Plugin.Firebase/issues/144#issuecomment-2223334370


Can't run bottom when installing through home-manager

Describe the bug
Installing bottom using home-manager does not work. I am not able to execute it by typing 'bottom' in terminal.
Steps To Reproduce
Steps to reproduce the behavior:

install bottom using home manager

Expected behavior
It should install bottom.
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Using nix-env -iA installs bottom perfectly but using home manager is not installing bottom for some reason or at least I am not able to run bottom by typing bottom in terminal. It says package not installed.
My dotfile link in case if it's just a me problem. https://github.com/MobSenpai/dotfiles/blob/main/home/yashraj/shared/shell/cli.nix#L11
Notify maintainers

Metadata

system: "x86_64-linux"
host os: Linux 6.5.5, NixOS, 23.11 (Tapir), 23.11.20230925.6500b45
multi-user?: yes
sandbox: yes
version: nix-env (Nix) 2.17.0
nixpkgs: /nix/store/i6hq63805y784dml4j8pgkrz3lryxzld-source

[user@system:~]$ nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 6.5.5, NixOS, 23.11 (Tapir), 23.11.20230925.6500b45`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.17.0`
 - nixpkgs: `/nix/store/i6hq63805y784dml4j8pgkrz3lryxzld-source`

Ah wrong repo. Sry


Add bind_node to player node to fix paused process behavior

Currently if you attempt to play music with a cross-fade while get_tree().paused is true, the music will not switch. This change adds a bind_node call to bind the tween to the player node and not the SceneTree so that cross-fade tweens will work in the player's process mode.

Thanks 👍


Getting 'error: failed to compile spl-token-cli v4.0.0, intermediate artifacts can be found...'

Hi.
I am getting an error while trying to install spl-token-cli with cargo install spl-token-cli. As I am not a Rust developer, I dont know what the main issue is and how I can fix it. Can someone shed some light and tell me how I can fix it? Here's the error:
   Compiling predicates-core v1.0.6
   Compiling histogram v0.6.9
   Compiling tracing-subscriber v0.3.18
   Compiling solana-streamer v2.0.2
   Compiling educe v0.4.23
   Compiling modular-bitfield v0.11.2
   Compiling solana-net-utils v2.0.2
error: failed to run custom build command for `solana-svm v2.0.2`

Caused by:
  process didn't exit successfully: `C:\Users\glory\AppData\Local\Temp\cargo-installlaN6GH\release\build\solana-svm-42f026cf64dccf6a\build-script-build` (exit code: 101)
  --- stdout
  cargo:rustc-cfg=RUSTC_WITHOUT_SPECIALIZATION
  cargo:rerun-if-changed=proto\context.proto
  cargo:rerun-if-changed=proto\invoke.proto
  cargo:rerun-if-changed=proto\sysvar.proto
  cargo:rerun-if-changed=proto\txn.proto

  --- stderr
  thread 'main' panicked at C:\Users\glory\.cargo\registry\src\index.crates.io-6f17d22bba15001f\prost-build-0.11.9\src\lib.rs:1457:10:
  Could not find `protoc` installation and this build crate cannot proceed without
      this knowledge. If `protoc` is installed and this crate had trouble finding
      it, you can set the `PROTOC` environment variable with the specific path to your
      installed `protoc` binary.You can download it from https://github.com/protocolbuffers/protobuf/releases or from your package manager.

  For more information: https://docs.rs/prost-build/#sourcing-protoc

  note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
warning: build failed, waiting for other jobs to finish...
error: failed to compile `spl-token-cli v4.0.0`, intermediate artifacts can be found at `C:\Users\glory\AppData\Local\Temp\cargo-installlaN6GH`.
To reuse those artifacts with a future compilation, set the environment variable `CARGO_TARGET_DIR` to that path.

C:\Users\glory>cargo +nightly -Zunstable-options config get source --show-origin
error: toolchain 'nightly-x86_64-pc-windows-msvc' is not installed

The last command is something I found here in one of the issues but apparently that thing is not installed as well.
Thanks in advance.

Unfortunately, in order to compile the SVM crate, you need a protobuf compiler, which is why the error tells you that protoc cannot be found.
On windows, our build tools install protoc using choco, ie:
https://github.com/anza-xyz/agave/blob/c08380355d8d8d25cbf6fc70389dcc5ed5cab132/.github/scripts/cargo-clippy-before-script.sh#L11
But you can look up other ways to install it using the links in the error message. If you have any other questions, please ask on the Solana Stack Exchange https://solana.stackexchange.com/

thanks


Links with empty Anchors

There are links with empty descriptions.
Looking at the code there is a comment When anchor is empty, it means is the same of id.
Which sort of makes sense since they look like: [[wikipedia_article]]. [1]
so when one does getDescription() links with empty descriptions would return their id. However when the object gets serialised to json the description field will be empty
As an user of jsonwikipedia this looked a bit weird at the beginning. I think the serialised description should equal getDescription.
If you agree on this point I can make a PR.
[1] https://github.com/diegoceccarelli/json-wikipedia/blob/master/src/main/java/it/cnr/isti/hpc/wikipedia/article/Link.java#L53

@dav009 nice catch :+1:  ! Happy to accept a PR :)

@dav009  I think that it would make sense to perform the check during the conversion and then trasform getDescription in a simple access method.


Better Exceptions

Is there some way to get better error messages. Like what tokens were expected instead of just which token causes the parse to fail.

I imagine that for anything reasonably complex, error messages shouldn't
be handled in the grammar -- if the message you're looking for is along
the lines of "Unexpected declaration after number", I imagine you can
find that yourself from the tokenized input.
On Fri, Dec 09, 2016 at 02:33:09PM -0800, Ryan Brown wrote:

Is there some way to get better error messages. Like what tokens were expected instead of just which token causes the parse to fail.
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/Hardmath123/nearley/issues/146

I don't understand. Isn't the grammar what defines which combinations of tokens are allowed? How is anything other than the grammar supposed to know what's wrong with a sequence of tokens?

@ribrdb, you're completely right. This would be a great feature.
For now, you can already do this "on your own" by inspecting the parse table. If p is a parser, then
p.table[p.table.length-2].map(function(s) {return s.rule.symbols[s.expect]; }).filter(function(t) {return t.literal || t.test; });

gives you the list of tokens that the parser is expecting.
But, I would appreciate a PR or something that integrates this functionality into nearley in a more user-friendly way. Or enough spare time to implement it myself!

Hmm. So why does nearly sometimes throw an exception and sometimes just return 0 results?
Should this same thing work when there were no results?
I tried this with something like this (which return 0 results with no error)
a = sin(
The expected symbol seems to be ( though, which it should have already matched.

"Results" are finished parses at the current location in the input stream. When parsing "1+1", you start with no results, then you get "[ 1 ]", then no results again because "1+" is not well-formed, and then finally "[ (+ 1 1) ]". Zero results does not necessarily mean a parse error, because future input may create results.
Exceptions are thrown when the parser encounters an unexpected token, because this implies that no possible parsings will ever exist.

because future input may create results.

So it's essentially "unexpected EOF," if you've fed all data you want to at that point?

Yes.

Is there a way to tell nearley that there will be not more input?
Currently I am doing
const p = new nearley.Parser(grammar.ParserRules, grammar.ParserStart)
p.feed(el.target.value) // the textarea's value
const result = p.finish()

I would be great if result would be either the resulting parse tree or an exception is thrown with the following information:

unexpected token: XXX
start position of token
end position of token

@laszlokorte you can do that with testable tokens and feeding list of characters + "EOF"
@Hardmath123 could there be an option that helps default literals to work in this case?

Like most parsing frameworks, simply providing the list of tokens the parser accepts isn't usually very useful. It's certainly not something Nearley should do by default.
Since this issue was opened, our error messages (both with and without a tokenizer) have become much prettier--they now include line & column information, as well as a preview of the line.
invalid syntax at line 2 col 3:

  12!
    ^

In summary: I'm closing this. :-)


Add support for other solvers like ECOS

The support for ECOS solver has been added. Currently it will be a hard dependency for convexjlr. And it is loading at the startup.
Maybe need to only install and load it when needed?
And may have a systemic way to support other solvers.
So although the support of ECOS solver is finished, leave the issue open for the record.


Using the new shell API and moving nodes out of this repo

Please copy the following repos into HPENetworking:
https://github.com/ocelotl/topology_docker_p4switch
https://github.com/ocelotl/topology_docker_openvswitch
https://github.com/ocelotl/topology_docker_ryu
https://github.com/ocelotl/topology_docker_openswitch
Once that is done, please change the requirements.dev.txt file to match the new repos.
This closes #18. This closes #17.

Repos moved to:
https://github.com/HPENetworking/topology_docker_p4switch
https://github.com/HPENetworking/topology_docker_openvswitch
https://github.com/HPENetworking/topology_docker_ryu
https://github.com/HPENetworking/topology_docker_openswitch
I'll complete the rest tomorrow

Topology 1.4.0 was released. Please change the minimum required version in requirements.txt to >= 1.4.0.

Updated branch to include host node again.

This was reworked in two: #24 and #25.


Add Mr Trash Wheel Data

https://docs.google.com/spreadsheets/d/1b8Lbe-z3PNb3H8nSsSjrwK2B0ReAblL2/edit#gid=1060674293
Or should it be its own package?
😄

https://github.com/marketplace/actions/gsheet-action


log_softmax of a row vector returns a vector

Description
When I do log_softmax of a row vector I get back a vector.
This:
TEST(MathRev, row_vector_log_softmax) {
  Eigen::RowVectorXd r(3);

  r << 1.0, 2.0, 3.0;

  stan::math::promote_scalar_t<stan::math::var, Eigen::RowVectorXd> rv(r);

  std::cout << stan::math::test::type_name<decltype(stan::math::log_softmax(rv).eval())>() << std::endl;
}

prints:
stan::math::test::type_name() [Arg = const Eigen::Matrix<stan::math::var_value<double, void>, -1, 1, 0, -1, 1> &

I ran into this testing against the varmat implementation (which returns a row vector for a row vector input).
I know that a lot of our vectorization stuff converts to a column vector internally, so I'm not sure if this is intentional or a bug.
So that's my question, intentional or a bug?
Current Version:
v3.4.0

Whoops, I shoulda kept reading the code, this looks like a bug: https://github.com/stan-dev/math/blob/develop/stan/math/rev/fun/log_softmax.hpp#L83 . I'll fix this in my pull.

The only expose log_softmax signature is
vector log_softmax(vector)
so I would say that is a bug that never surfaced.

This was addressed at some point. The following test passes:
TEST(MathMatrixPrimMat, log_softmax_test) {
  using stan::math::log_softmax;
  stan::math::vector_d v0(2);
  v0 << -1.0, 1.0;
  stan::math::vector_d res0 = log_softmax(v0);
  stan::math::row_vector_d v1(2);
  v1 << -1.0, 1.0;
  stan::math::row_vector_d res1 = log_softmax(v1);
  stan::math::vector_v v2(2);
  v2 << -1.0, 1.0;
  stan::math::vector_v res2 = log_softmax(v2);
  stan::math::row_vector_v v3(2);
  v3 << -1.0, 1.0;
  stan::math::row_vector_v res3 = log_softmax(v3);
}


Cannot Start a Chat with Given History

Description of the bug:
Hi, I want to start a conversation using ChatSession with some given histories, such as:
import google.generativeai as genai

model = genai.GenerativeModel('gemini-pro')

chat = model.start_chat(history=[
    {
        'role': 'user',
        'parts': ['You are a helpful assistant.']
    },
    {
        'role': 'model',
        'parts': ['Sure.'],
    },
])

resp = chat.send_message(
    content='hi, how are you?',
)

print(resp.text)

However, the program above threw an exception pasted below:
Traceback (most recent call last):
  File "/data/miniforge3/envs/py310/lib/python3.10/site-packages/google/api_core/grpc_helpers.py", line 75, in error_remapped_callable
    return callable_(*args, **kwargs)
  File "/data/miniforge3/envs/py310/lib/python3.10/site-packages/grpc/_channel.py", line 1161, in __call__
    return _end_unary_response_blocking(state, call, False, None)
  File "/data/miniforge3/envs/py310/lib/python3.10/site-packages/grpc/_channel.py", line 1004, in _end_unary_response_blocking
    raise _InactiveRpcError(state)  # pytype: disable=not-instantiable
grpc._channel._InactiveRpcError: <_InactiveRpcError of RPC that terminated with:
        status = StatusCode.INTERNAL
        details = "An internal error has occurred. Please retry or report in https://developers.generativeai.google/guide/troubleshooting"
        debug_error_string = "UNKNOWN:Error received from peer ipv4:<IP_ADDRESS>:443 {created_time:"2024-01-18T22:37:41.46318178-05:00", grpc_status:13, grpc_message:"An internal error has occurred. Please retry or report in https://developers.generativeai.google/guide/troubleshooting"}"
>

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/data/code/some_proj/test.py", line 18, in <module>
    resp = chat.send_message(
  File "/data/miniforge3/envs/py310/lib/python3.10/site-packages/google/generativeai/generative_models.py", line 367, in send_message
    response = self.model.generate_content(
  File "/data/miniforge3/envs/py310/lib/python3.10/site-packages/google/generativeai/generative_models.py", line 248, in generate_content
    response = self._client.generate_content(request)
  File "/data/miniforge3/envs/py310/lib/python3.10/site-packages/google/ai/generativelanguage_v1beta/services/generative_service/client.py", line 566, in generate_content
    response = rpc(
  File "/data/miniforge3/envs/py310/lib/python3.10/site-packages/google/api_core/gapic_v1/method.py", line 131, in __call__
    return wrapped_func(*args, **kwargs)
  File "/data/miniforge3/envs/py310/lib/python3.10/site-packages/google/api_core/retry.py", line 372, in retry_wrapped_func
    return retry_target(
  File "/data/miniforge3/envs/py310/lib/python3.10/site-packages/google/api_core/retry.py", line 207, in retry_target
    result = target()
  File "/data/miniforge3/envs/py310/lib/python3.10/site-packages/google/api_core/timeout.py", line 120, in func_with_timeout
    return func(*args, **kwargs)
  File "/data/miniforge3/envs/py310/lib/python3.10/site-packages/google/api_core/grpc_helpers.py", line 77, in error_remapped_callable
    raise exceptions.from_grpc_error(exc) from exc
google.api_core.exceptions.InternalServerError: 500 An internal error has occurred. Please retry or report in https://developers.generativeai.google/guide/troubleshooting

What's the right way to start a conversation with some history? Thanks!
Actual vs expected behavior:
No response
Any other information you'd like to share?
No response

same too

I realized that the problem is the history parameter only accepts a list of google defined history objects, but not the list of dicts I provided above. It's not working in the similar way as OpenAI's interface. But I think it's better to support such style because constructing that google defined objects is not easy.

Hi @Co1lin, thanks for the report.
That should totally work.
This line runs the history through to_contents
https://github.com/google/generative-ai-python/blob/main/google/generativeai/generative_models.py#L306C43-L306C78
Which handles the conversion of any acceptable thing into a list of glm.Content objects. Here:
https://github.com/google/generative-ai-python/blob/main/google/generativeai/types/content_types.py#L225
I just copied your code into colab and it works:

There have been intermittent 500 errors. Maybe that's what you hit. Try again?

I'm good to run this code.

what if everytime i want to have a new conversation then what should i do

@Himanshu-Adhikari you would just initialize the start_chat function without providing the history array.

@BryceBlankinship  thank you for your suggestion


Instead of 「by」, use 「assert」

Your interesting example looks like a stream:
turning
  .turn(['session:not-logged-in'])
  .to(['session:logged-in'])
  .alias('login')
  .by(async ({page}) => {
    await page.click('.login-link');

    await page.type('input.username', 'admin');
    await page.type('input.password', '123456');

    await page.click('.login-submit-button');

    await page.waitForNavigation();
  });

But doesn't like a stream, it can't store intermediate steps.
Instead, if you make a transformation like this:
const loginInput = turning
  .alias('login-input')
  .turn(['session:not-logged-in'])
  .by(async ({page}) => {
    await page.click('.login-link');

    await page.type('input.username', 'admin');
    await page.type('input.password', '123456');
  });

const commitLogin = loginInput
  .by(async ({page}) => {
    await page.click('.login-submit-button');
    await page.waitForNavigation();
  })
  .to(['session:logged-in']);

const abortLogin = loginInput
  .by(async ({page}) => {
    await page.click('.login-cancel-button');
    await page.waitForNavigation();
  })
  .to(['session:not-logged-in']);

In this way, you can make a usecase abstraction.

Actually by() can be chained with test(). The reason why I chose by is to emphisize the process of transformation, instead of the assertion.
And to be honest, I don't see much advatage brought by the stream-like interface. The same can be achieved by operation function abstractions. As the transformation is the one of the cores in turning, it would also be nice to make the obsolete/new states side-by-side. And that is also why alias() is chained after the states methods (initialize, turn/to, spawn/to).

Ok, it's a new way to write test, at least to me.
Waiting for your zhihu article about this.

@linonetwo haha I guess it is. Though I have seen several papers about this. Will write an article after writing more tests for our project.


Dependency upgrades

coverage: 94.268%. remained the same when pulling b9894f4b4b2e9bd1d67f181323032d07384f5d02 on cody-dep-upgrades-2023-08-10 into eccb59395e8437e3d59f3f2c56ddffabfdf28a1a on main.


[PokemonTabletopAdventures_v3] Fix: Updates 3D Die Rolling

Submission Checklist
Required

[x] The pull request title clearly contains the name of the sheet I am editing.
[x] The pull request title clearly states the type of change I am submitting (New Sheet/New Feature/Bugfix/etc.).
[x] The pull request makes changes to files in only one sub-folder.
[x] The pull request does not contain changes to any json files in the translations folder (translation.json is permitted)

Changes / Description

Resolves issue where every attack type throws out five sets of attack and damgae dice instead of only the number required
Adds attack_rolls and quick_attack_rolls attributes to the repeating moves fieldset to house the accuracy and damage checks for normal rolls - asking for modifiers - and quick rolls - asking for no modifiers
Updates the scatter type selection field values
Adds support for updating sheets to use this new value
Adds support for Population Bomb by adding an "Up to 10 Hits" option for the scatter type

Character Sheet Info Roll20 Internal Use only.

@mrianmerry Just wanted to confirm with you that this is listed as draft. It won't be merged until marked ready for review.

Yhupp! The other contributor for this sheet is reviewing and assisting with testing, so once we're both happy I'll mark it as ready for review.
Is it possible to request reviews from other contributors and have that hold up merge-ready status?


Mark can.mustache as deprecated & add note to can.Component

With the memory leak fixes that were made to can.Component as they relate to can.stache exclusively - we have officially turned out backs on can.mustache. We should put a deprecated tag on it in the API and perhaps a note on can.Component urging people that if they intend to use components then they should use can.stache instead, otherwise there is the potential for bad memory leaks.

I agree, but if you marked mustache as deprecated you have to remove mustache as a hard coded dependency for can.component

@pYr0x we can't remove it as the default until 3.0.  This notice is just a warning that mustache won't have priority on bug fixes and eventually will be removed.


Cannot correctly parse an enum of an extern lexer.

I have some code that looks like this:
extern {
    enum lexer::Tok {
        "+=" => lexer::Tok::AddEq(<TokLoc>),
        "-=" => lexer::Tok::SubEq(<TokLoc>),
        "*=" => lexer::Tok::MulEq(<TokLoc>),
        "/=" => lexer::Tok::DivEq(<TokLoc>),
        "%=" => lexer::Tok::ModEq(<TokLoc>),
        "=" => lexer::Tok::Eq(<TokLoc>),
        // and so on
    }
}

The error that I get is at the lexer::Tok::AddEq(<TokLoc>) saying LALRPOP.COMMA or LALRPOP.RBRACE expected, got 'lexer::Tok::AddEq(<T...'

Turns out when I was converting the lalrpop grammar to Grammar-Kit, I didn't properly account for this section of the grammar. This is related to #2, because lalrpop invokes a second parser in the first parser, to parse a "pattern":
https://github.com/lalrpop/lalrpop/blob/723678f364790b41d66e747d4eb982897c0edce9/lalrpop/src/parser/lrgrammar.lalrpop#L345-L351
It looks like this "pattern" is just a subset of Rust grammar, so we can probably inject Rust here like we do for the action code, and just replace the odd <Type> notation with whatever it's supposed to be.

Parsing now completes successfully as of https://github.com/Mcat12/intellij-lalrpop/commit/7095024cd6a1ed1280bd4c8667e175dd0f5e1b45.


Website is down

`This Deployment has been disabled.
Your connection is working correctly.
Vercel is working correctly.
402: PAYMENT_REQUIRED
Code: DEPLOYMENT_DISABLED
ID: syd1::zvv5j-1641460122118-9691eab7801e
If you are a visitor, contact the website owner or try again later.
If you are the owner, please read this documentation section.`

The website has been restored, there was an issue with the hosting provider disabling my account due to "excessive usage" for no apparent reason. I contacted their support but I'll probably end up moving to a different host entirely.


Update README links

I enjoyed this tutorial series, thank you!
I noticed some links on the README weren't working. This PR updates those file references.

Thanks 🙏


generated file hierarchy package headers may conflict with C++ files

Our app is ObjC & C++ and we also use CocoaPods. When trying to bring some j2objc code into the app, the C++ code will find the file "java/lang/Float.h" and complain (since it isn't C++ stdlib float.h). This is because all the header paths are searched (at least the way our code uses CocoaPods).
If these j2objc headers were generated in a flat manner (such as "java_lang_float.h") they would not conflict with C++.
Is there another workaround I haven't been able to find? Thanks.

There's no such workaround that I know of. You're suggestion is a good one. We currently support a --no-package-directories flag that puts all generated files in the top-level directory, with the same basename. It would probably be possible to add a similar flag for the naming format you suggest. The main problem is that with --no-package-directories we have to make an exception for all JRE classes, since the includes we distribute for the JRE emulation library have the package directories. To support flat JRE headers we may need to create a separate distribution.

Thanks. Yes, I think it would require a different distribution, just for the jre_emul's headers. I'm working towards a patch now to see what I get.

Any news on this issue?

Any news on this issue?

I’d worked around this and no longer require it for any projects. In my opinion this can be closed.

It's still open because as Keith mentioned, it's a great idea. One problem blocking our distribution multiple sets of headers is that we're at GitHub's maximum distribution file size. However, the bigger issue is how to easily migrate projects to different include paths. Anyone's ideas on how to avoid breaking everyone without bloating our distribution would be very appreciated.


Typo on page 1027, v2024-06-26

To approximate the posterior, the original says "A simple one is a fully factorized model, $q(z_{1:T})=\prod_t q(z_t\big\vert x_{1:t})$".
I believe it should have been $q(z_{1:T}\big\vert x_{1:T})=\prod_t q(z_t\big\vert x_{1:t})$.

fixed


Scale the number of replicas in a deployment

Is it possible to interact with the Deployment (or ReplicaSet) object to scale from that perspective?
Scaling works via the ReplicationController object update_replication_controller, but I would rather use the higher abstraction Deployment object.

It should work, you just need client for correct API group.
See https://github.com/abonas/kubeclient/issues/224#issuecomment-277537170 and #343

something like:
apps_client = Kubeclient::Client.new('http://localhost:8080/apis/apps', 'v1', options)
apps_client.patch_deployment('mydeployment', {spec: {replicas: 10}})

If this doesn't help, try doing RestClient.log = "stderr" and post the output with that.
Our docs around API groups are bad — people keep asking about this...  PRs to improve the docs would be very welcome.

I get a better understanding now, yes, documentation might be better, but I'll see what I can do once played with this some more and have a better understanding, thx so far @cben

Yes, thx for the clarrification @cben , added namespace and label selection so get some sort of nightshift scaling functionality into the cluster where things might go to 0 for internal services:
  app_client = Kubeclient::Client.new(
    'https://kubernetes.default.svc/apis/apps',
    'v1',
    auth_options: auth_options,
    ssl_options:  ssl_options
  )

 # fetch all deployments labelled for the nightshift
  deployments = app_client.get_deployments(label_selector: 'scaling=nightly')

  deployments.each do |d|
    deployment_name = d.metadata.name
    namespace = d.metadata.namespace
    logging.info("Scaling down #{namespace}: #{deployment_name}")
    app_client.patch_deployment(deployment_name, {spec: {replicas: 0}}, namespace)
  end

Would be great to add the default types into the readme.. like pods, deployements, services,...

Pull requests to improve README would be very welcome!
I'm falling behind my other duties as maintainer, and besides I'm too close to the subject to feel what needs explaining...
pods & services are in core 'v1' api group, and already listed in many places in readme.
It could be great to add several examples of creating a Client with different API group, e.g. for deployments, and to stress that you need separate Client objects to manipulate multiple groups (e.g. both pods & deployments)...

[ ] I see somewhere under Delete section, we already have a client.delete_deployment(...) example, but it doesn't mention it needs anything special.


Adds a new file banks.json for bank related details

This adds support for two things:

A banks.json file which is basically a JSON generated from the first table at https://www.npci.org.in/national-automated-clearing-live-members-1
Tests for names/constant coverage against the above.

This is a completely auto-generated file, and manual edits should not be made to this.

@patlola reminder to review this. (Need suggestions for code/HTTP API for exposing this information as well)

sure @captn3m0, will check this 👍


HPC: Merge old model setup with new job submission script

Changes:

job.sh will now select a random free port for the model server instead of always choosing port 4242, which should be more robust.
Remove previously used setup_model.sh.

Possible issues:

Users need to ensure that their model server sets the port according to the environment variable PORT ⚠

Looks good, thanks a lot! :)


Add draft ADR on S3 object definition

Adds ADR on how we want to represent S3 buckets in our CRD structure.

Can you maybe add (for option 5) how generated names come into play and/or how we are compatible with ObjectBucketClaim (OBC)?
As far as I understand it we do would now need to have a controller that converts from the OBC ConfigMap & Secret to our own S3Bucket and/or S3Connection, is that correct?

That is correct. I intentionally left that out of this ADR, as it is not really related to the options outlined here, but instead an unrelated decision to take.
I have added some more detail around this to the ADR.

Thanks!

Sorry, addressed one more comment that I had missed.


Choco Install - Newtonsoft Dependency missing

Hi,
Love the project, having a play now.
Looks like your missing the Newtonsoft.Json.dll from the Choco package. With the current package you get the following output.
 C:\WINDOWS\system32> choco install autorest
!!ATTENTION!!
The next version of Chocolatey (v0.9.9) will require -y to perform
  behaviors that change state without prompting for confirmation. Start
  using it now in your automated scripts.

  For details on the all new Chocolatey, visit http://bit.ly/new_choco
Chocolatey (v<IP_ADDRESS>) is installing 'autorest' and dependencies. By installing you accept the license for 'autorest' an
d each dependency you are installing.

AutoRest v0.9.6
Added C:\ProgramData\chocolatey\bin\AutoRest.exe shim pointed to '..\lib\autorest.0.9.6\tools\autorest.exe'.
Finished installing 'autorest' and dependencies - if errors not shown in console, none detected. Check log for errors if
 unsure.
C:\WINDOWS\system32> autorest
Microsoft (R) AutoRest <IP_ADDRESS>
Copyright (C) Microsoft Corporation. All rights reserved.

Usage: AutoRest.exe -Input <value> -Namespace <value> [-OutputDirectory <value>] [-OutputFileName <value>] [-CodeGenerat
or <value>] [-Modeler <value>] [-ClientName <value>] [-Header <value>] [-AddCredentials <value>]

Parameters:
  -Input: The location of the input specification.
  -Namespace: The namespace to use for generated code.
  -OutputDirectory: The location for generated files. If not specified, uses "Generated" as the default.
  -OutputFileName: If specified, all generated code is written to this file. Otherwise, code is generated into multiple
files in the OutputDirectory.
  -CodeGenerator: The code generator language. If not specified, defaults to CSharp.
  -Modeler: The Modeler to use on the input. If not specified, defaults to Swagger.
  -ClientName: Name to use for the generated client type. By default, uses the value of the 'Title' field from the Swagg
er input.
  -Header: Text to include as a header comment in generated files. Use NONE to suppress the default header.
  -AddCredentials: If true, the generated client includes a ServiceClientCredentials property and constructor parameter.
 Authentication behaviors are implemented by extending the ServiceClientCredentials type.

Examples:
  Generate C# client in MyNamespace from swagger.json input:
  >AutoRest.exe -Namespace MyNamespace -Input swagger.json

  Generate C# client in MyNamespace into client.cs including custom header from swagger.json input:
  >AutoRest.exe -Namespace MyNamespace -OutputFileName client.cs -Header "Copyright Contoso Ltd" -Input swagger.json

  Generate C# client with a credentials property in MyNamespace from swagger.json input:
  >AutoRest.exe -AddCredentials true -Namespace MyNamespace -CodeGenerator CSharp -Modeler Swagger -Input swagger.json

C:\WINDOWS\system32> cd "C:\Users\xxxxxx\Documents\CustomerTemp\autorest\"
C:\Users\xxxxxxx\Documents\CustomerTemp\autorest> ls

    Directory: C:\Users\xxxxxx\Documents\CustomerTemp\autorest

Mode                LastWriteTime         Length Name
----                -------------         ------ ----
-a----       25/03/2015     12:12           3663 devinsight.api.json

C:\Users\xxxxx\Documents\CustomerTemp\autorest> autorest -codegenerator CSharp -Modeler Swagger -Input .\devinsight.a
pi.json -Namespace DevInsight
error: Could not load file or assembly 'Newtonsoft.Json, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=30ad4fe6b2a6ae
ed' or one of its dependencies. The system cannot find the file specified.
Finished generating CSharp code for .\devinsight.api.json.

I grabbed the latest newtonsoft dll from here and placed it in "C:\ProgramData\chocolatey\lib\AutoRest.0.9.6\tools" manually and all seems to work well.

@lawrencegripper, thank you. Will publish an update ASAP.

Nuget package  AutoRest also throwing the same error. Missing Newtonsoft.Json dll.  Looks like Newtonsoft json dependency is missing. in the autoreset Nuget package.  Manually copying the dll generated the c# code.

For others having this problem, you'll want the net45 version of Newtonsoft.Json.dll.

The package is updated on NuGet.
Chocolatey is pending moderator approval.
From: Shawn S<EMAIL_ADDRESS>Sent: Friday, March 27, 2015 4:25 PM
To: Azure/AutoRest
Subject: Re: [AutoRest] Choco Install - Newtonsoft Dependency missing (#51)
For others having this problem, you'll want the net45 version of Newtonsoft.Json.dll.
—
Reply to this email directly or view it on GitHubhttps://github.com/Azure/AutoRest/issues/51#issuecomment-87119090.

Chocolatey moderator approval is still pending. There is a notice on the home page that they are behind. Hoping the update is approved soon.

I'm still waiting for this also. @matt-gibbs

@shanselman It seems the latest version was published https://twitter.com/ferventcoder/status/593562533786595328 . Checked it out and there's no error, works fine. Used https://github.com/swagger-api/swagger-spec/blob/b4cf91c3023f8b3cbbfebfe0eedcb401d7e25f76/examples/v2.0/json/petstore-expanded.json as example.

The moderation backlog was quite long. Package has been approved.

@matt-gibbs I've written a blog post on automating the release process to chocolatey with appveyor, not going to sort the moderator backlog but could be useful for future releases https://gripdev.wordpress.com/2015/04/08/appveyor-github-and-chocolatey-automatically-build-your-project-and-publish-updates-it-to-chocolatey/


Spherical geometry

It could be useful to have some spherical geometry (read: geometry on the Earth surface).
In this pull request I implemented a formula to calculate the area of any spherical polygon, which of course sits awkwardly in Rasters.jl.
Some things we might want to implement are (please suggest more):

Shortest distance between two points on a sphere
Bearing
Polygon area

This page has some relevant math

Turf.jl also has a bunch of code for this we could use.

Distance between points and lines on a sphere would also be useful
https://stackoverflow.com/questions/1299567/how-to-calculate-distance-from-a-point-to-a-line-segment-on-a-sphere

Could be good to standardise with Distances.jl
https://github.com/JuliaStats/Distances.jl

Just to chime in, there seem to be two other packages which look at spherical (specifically Earth spherical :D) coordinates: GeographicLib.jl (unregistered but pure Julia) or Proj.jl's geodesic API.  I would guess that we could offer one or all of those as references as well, at least in the docs!

Hello.  While doing this -- how about Hyperbolic Geometry [has applications in AI/ML].


Add GPU support for Vizier and DataFlow Jobs

Description
This feature allows SKAI inference sent as Google Cloud Dataflow job to leverage the availability of GPUs to run the pipeline faster. It also enables XManager-Vertex AI vizier hyperparameter sweep to leverage GPUs.
Jobs sent to Google Cloud Dataflow requires that the local environment version of python and apache-beam versions are the same in docker container image. Currently, we have support for python versions 3.8-3.11 and soon to extend to python 3.7.
Types of changes

New docker images from base images that have cuda support and allows training/ inference to happen on GPUs.
Dynamic GPU memory allocation, addition of persistent disk space, machine type and worker type and as well other Google Dataflow service options. Files changed;

src/skai/beam_utils.py
src/skai/model/inference.py

Apache-beam upgraded to version 2.49.0 (latest at the time of this PR)
Type hints written with python 3.10+ syntax modified to support running pipeline job from local environment with python version<3.10. Changes here include; modifying str | None -> typing.Optional[str], list -> typing.List, dict -> typing.Dict. Affected files are;

src/skai/beam_utils.py
src/skai/model/inference_lib.py
src/skai/model/data.py

New docker images created from tensorflow-gpu base image to enable training on GPUs.

What changes does to code
Adds new feature:

Google Dataflow job execution on GPUs
XManager + Vertex AI + Vizier on GPUs

@panford has merged a fix for the directory naming bug

Thanks for addressing my comments! I left a couple more minor comments but the PR looks good to me for the most part.

Thanks for addressing all comments! Feel free to merge!


Implement getBundlesFromAddresses

https://github.com/iotaledger/iota.js/blob/next/packages/core/src/createGetBundlesFromAddresses.ts#L5
getBundlesFromAddresses() has been deprecated and will be removed in v2.0.0 Please use findTransactionObjects() and getBundle() as an alternative


L5.5: Please enable support for Package Auto-Discovery

See https://github.com/barryvdh/laravel-debugbar/pull/643/commits/1a206a491e1a24d4f5555dafad3d742064c3a20d
Makes it a lot easier and cleaner! :)

It will be implemented soon :), thanks.

Great, thanks so far! Great package!

Done :)

Hero!


Basic framework for the indy-agent nodejs implementation

Signed-off-by: Spencer Holman<EMAIL_ADDRESS>

Looks like a great place to start! My only comment is that we probably don't need the nested LICENSE file inside of the nodejs folder.


Check failed: eglDevId < numDevices [EGL] Could not find an EGL device for CUDA device 0

❓ Questions and Help
I already integrated habitat env into the A2C. It works in --num-processes 1. However, when I set --num-processes 2, the habitat simulator raised the following errors.
Specifically, what does it mean by Check failed: eglDevId < numDevices [EGL] Could not find an EGL device for CUDA device 0? All my eight 1080Ti support opengl and work for a single process.
Thank you very much!
I0929 10:40:49.089522 29832 Simulator.cpp:42] Deconstructing Simulator
I0929 10:40:49.089660 29832 SemanticScene.h:40] Deconstructing SemanticScene
I0929 10:40:49.103410 29832 SceneManager.h:24] Deconstructing SceneManager
I0929 10:40:49.103449 29832 SceneGraph.h:20] Deconstructing SceneGraph
I0929 10:40:49.103891 29832 Sensor.h:82] Deconstructing Sensor
I0929 10:40:49.103901 29832 RenderTarget.h:51] Deconstructing RenderTarget
I0929 10:40:49.106288 29832 Sensor.h:82] Deconstructing Sensor
I0929 10:40:49.106312 29832 RenderTarget.h:51] Deconstructing RenderTarget
I0929 10:40:49.106452 29832 SceneGraph.h:20] Deconstructing SceneGraph
I0929 10:40:49.158921 29836 Simulator.cpp:42] Deconstructing Simulator
I0929 10:40:49.159044 29836 SemanticScene.h:40] Deconstructing SemanticScene
I0929 10:40:49.170749 29836 SceneManager.h:24] Deconstructing SceneManager
I0929 10:40:49.170784 29836 SceneGraph.h:20] Deconstructing SceneGraph
I0929 10:40:49.171011 29836 Sensor.h:82] Deconstructing Sensor
I0929 10:40:49.171020 29836 RenderTarget.h:51] Deconstructing RenderTarget
I0929 10:40:49.178689 29836 Sensor.h:82] Deconstructing Sensor
I0929 10:40:49.178714 29836 RenderTarget.h:51] Deconstructing RenderTarget
I0929 10:40:49.178874 29836 SceneGraph.h:20] Deconstructing SceneGraph
I0929 10:40:49.537631 29832 Renderer.cpp:33] Deconstructing Renderer
I0929 10:40:49.537721 29832 WindowlessContext.h:16] Deconstructing WindowlessContext
I0929 10:40:49.537730 29832 WindowlessContext.cpp:245] Deconstructing GL context
I0929 10:40:49.538054 29832 PathFinder.h:113] Deconstructing PathFinder
2019-09-29 10:40:49,563 initializing sim Sim-v0
I0929 10:40:49.570120 29832 WindowlessContext.cpp:99] [EGL] Detected 8 EGL devices
F0929 10:40:49.573048 29832 WindowlessContext.cpp:113] Check failed: eglDevId < numDevices [EGL] Could not find an EGL device for CUDA device 0
*** Check failure stack trace: ***
I0929 10:40:49.666532 29836 Renderer.cpp:33] Deconstructing Renderer
I0929 10:40:49.666630 29836 WindowlessContext.h:16] Deconstructing WindowlessContext
I0929 10:40:49.666640 29836 WindowlessContext.cpp:245] Deconstructing GL context
I0929 10:40:49.666983 29836 PathFinder.h:113] Deconstructing PathFinder
2019-09-29 10:40:49,694 initializing sim Sim-v0
I0929 10:40:49.704221 29836 WindowlessContext.cpp:99] [EGL] Detected 8 EGL devices
F0929 10:40:49.711859 29836 WindowlessContext.cpp:113] Check failed: eglDevId < numDevices [EGL] Could not find an EGL device for CUDA device 0
*** Check failure stack trace: ***
I0929 10:40:55.815344 29150 Simulator.cpp:42] Deconstructing Simulator
I0929 10:40:55.815410 29150 SemanticScene.h:40] Deconstructing SemanticScene
I0929 10:40:55.826141 29150 SceneManager.h:24] Deconstructing SceneManager
I0929 10:40:55.826181 29150 SceneGraph.h:20] Deconstructing SceneGraph
I0929 10:40:55.826371 29150 Sensor.h:82] Deconstructing Sensor
I0929 10:40:55.826378 29150 RenderTarget.h:51] Deconstructing RenderTarget
I0929 10:40:55.830835 29150 Sensor.h:82] Deconstructing Sensor
I0929 10:40:55.830860 29150 RenderTarget.h:51] Deconstructing RenderTarget
I0929 10:40:55.831213 29150 SceneGraph.h:20] Deconstructing SceneGraph
I0929 10:40:56.089581 29150 Renderer.cpp:33] Deconstructing Renderer
I0929 10:40:56.089658 29150 WindowlessContext.h:16] Deconstructing WindowlessContext
I0929 10:40:56.089666 29150 WindowlessContext.cpp:245] Deconstructing GL context
I0929 10:40:56.089870 29150 PathFinder.h:113] Deconstructing PathFinder

You will need to change the line here: https://github.com/ikostrikov/pytorch-a2c-ppo-acktr-gail/blob/master/a2c_ppo_acktr/envs.py#L91 to context="forkserver" as forked subprocesses don't work with habitat.

You will need to change the line here: https://github.com/ikostrikov/pytorch-a2c-ppo-acktr-gail/blob/master/a2c_ppo_acktr/envs.py#L91 to context="forkserver" as forked subprocesses don't work with habitat.

Thank you so much! It works!


object detection Accuary just 20%

i follow the example of Using DIGITS to train an Object Detection network,setting every step parameters according the Model creation instruction. But after creating training model ,the accuary  just 20%.
My GPU is Tesla M40(24G),i have tried to decrease the batch size and Batch Accumulation,but the result as same with the before

Why i can't get the same effect as the example?can you give me some suggests?

@ljz1995 , I have run into this very same issue. In my case, the training accuracy is about 30%. Did you find a solution for this? Thnx


Change color scheme of "other" so that is not the same as "fossil fuels"

Change color scheme of "other" so that is not the same as "fossil fuels" at https://github.com/UBC-MDS/DSCI-532_2024_6_Green-Development-Planner/pull/98

Change of the color scheme for the two pie charts: see 7cc1cd07aac5c381adb7a927ca1f7e5b0ad6d962


Better way of doing client-side compilation [optimisation, question]

I'm aware of the nearley playground source code, but I don't have any familiarity with React/etc. to be able to quite grasp how it is compiling the grammars (e.g. the text in the the Basic Grammar/Fancier Grammar tabs) in the browser.
What I've done is mimicked what lines 50-52 do in nearleyc.js, then run eval() on the generated Javascript string. See demo at https://codepen.io/anon/pen/JyRLXq?editors=0010#0.
$("#compile").click(function() {
   var nearleyc = new nearley.Parser(grammar.ParserRules, grammar.ParserStart, { "keepHistory" : true });
   var ne_data  = $("#grammar_def").val();

   nearleyc.feed(ne_data);
   var c = Compile(nearleyc.results[0], { alreadycompiled:false });
   var js_str = generate(c, "my_compiled_grammar");

   eval(js_str);
   alert("Compiled as my_compiled_grammar!");
})

I'm sure there's a better/recommended way of compiling grammars without using nearleyc in the command line (without React/etc.)—I was hoping you could point me in the right direction?
Thanks!

Please keep in mind that some security policies disallow evals
https://developer.mozilla.org/en-US/docs/Web/HTTP/CSP
So while such an approach is good for a playground or a website you control,
it may not be relevant for all productive flows.

@bd82 — I'm looking to use this inside an Electron app in production, so perhaps to some extent this counts as 'under my control'? Would be entirely glad if anyone could point me a way of doing this without the eval() call.

@fauxneticien
Sorry I have no idea about electron and content security policies. 😢
I've heard of issues in things such as Chrome extensions, but I have no idea
if and how this is relevant in electron.
Related search shows 24 issues in electron.
https://github.com/electron/electron/issues?utf8=✓&q=is%3Aissue is%3Aopen CSP

You need the eval call if you want to compile a .ne file, because of how generate is currently written.
Have a look at how we compile grammars in shared.js without using the nearleyc CLI tool. That might help :-)

I'm quite certain there's no way to avoid eval because you need to execute postprocessors at some point.
In Electron, the best solution is to do this is in the main process, the one where you create BrowserWindow and can use require. Just use the nearley npm package there.
Use IPC messaging to send the grammar from the page to the main process and to send any information back.

Let's include this in the docs: preview.
I hope @fauxneticien and @tjvr are OK with using parts of both solutions (codepen and shared.js) in the docs where other people will potentially copy-paste the example.

Thanks for the feedback all! Happy for the codepen example to be used in the docs.

Nice example @deltaidea! :-)


Adding Subheaders to the Table component

Issue Description
We are in need of Subheaders being implemented into the existing Table component. This would be built into the existing Table component to separate rows of the table into sub-lists.
Issue Type

[ ] New Feature
[x] Enhancement
[ ] Bug
[ ] Other

Expected Behavior
Needs definition
Current Behavior
N/A

Code review and UX/Functional review is complete. We want to hold off on merging this into master until #392 is ready as well.

#392 has been merged down to master. I plan to release terra-table tomorrow with this update and updates from #392.

This has been released in<EMAIL_ADDRESS>


question about pedestrians

Thank you for your excellent work, but on what basis did you not add pedestrians to the plant? It doesn’t seem to be difficult from an implementation perspective.

You are correct, it is unlikely it would be difficult from an implementation perspective. There are very few pedestrians in CARLA's Leaderboard 1.0 and related benchmarks like Longest6. This is also true for other aspects of the full driving task such as following stop signs and circumventing static objects, which can be ignored on CARLA while still achieving good scores.
For PlanT, we chose to focus on vehicle collisions, since it has the largest impact on driving scores in current benchmarks.

Thanks for your reply!


feat: add babylon chain configs

Summary
This PR removes the hardcoded Babylon chain configs (CW contract address and chain type) from the engine queue, and dynamically imports them from chain configs at deployment.
It closes #12.
Description
Refer to OP docs on creating L2 rollup.
We add two new values to .envrc as follows:
# Babylonchain finality gadget
export BBN_FINALITY_GADGET_CHAIN_TYPE=0
export BBN_FINALITY_GADGET_CONTRACT_ADDRESS=bbn1ghd753shjuwexxywmgs4xz7x2q732vcnkm6h2pyv9s6ah3hylvrqxxvh0f

Then, we amend the config script at packages/contracts-bedrock/scripts/getting-started/config.sh to include these params:
"babylonFinalityGadgetChainType": "$BBN_FINALITY_GADGET_CHAIN_TYPE",
"babylonFinalityGadgetContractAddress": "$BBN_FINALITY_GADGET_CONTRACT_ADDRESS"

Finally, we update Finalizer by adding the BabylonConfig object so that we can read the Babylon configs in the engine queue, and remove the hard code.
Test plan
# Generate the chain config file
./scripts/getting-started/config.sh

# Generate `genesis.json`
# Note it's necessary to first set the L1_RPC_URL in env
go run cmd/main.go genesis l2 \
  --deploy-config ../packages/contracts-bedrock/deploy-config/getting-started.json \
  --l1-deployments ../packages/contracts-bedrock/deployments/getting-started/.deploy \
  --outfile.l2 genesis.json \
  --outfile.rollup rollup.json \
  --l1-rpc $L1_RPC_URL

# Run tests
cd op-node/rollup
go test

seems there is an issue that the test plan doesn't pass @parketh will create an issue to track it

seems there is an issue that the test plan doesn't pass @parketh will create an issue to track it

tracking here: https://github.com/babylonchain/optimism/issues/14


Additional onboarding support for loop status colors and dosing enabled

LW PR for build: https://github.com/tidepool-org/LoopWorkspace/pull/466


Move magma commit to 2.7.2v supporting ROCm 6.0

Magma commit is already updated on upstream main branch and also updated on rocm6.0_internal_testing, release/2.1 branches of pytorch.


Allowing treetable row to set parent value for primitive ui builder

Hi,
when I tried to create a treetable I found that I was not able to set a parent id. This fix worked for me. Or is this not necessary and there is another way to achieve this in js?

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


Unable to send message using botbuilder-adapter-webex

I am using the WebexAdapter from botbuilder-adapter-webex to connect my BotBuilder bot with Webex teams.
I have used the following code
const server = restify.createServer();
server.listen(process.env.port || process.env.PORT || 3978, () => {
    console.log(`\n${ server.name } listening to ${ server.url }`);
    console.log(`\nGet Bot Framework Emulator: https://aka.ms/botframework-emulator`);
    console.log(`\nTo test your bot, see: https://aka.ms/debug-with-emulator`);
});
var webexOptions = {
    access_token: process.env.ACCESS_TOKEN,
    public_address: process.env.PUBLIC_ADDRESS,
    secret: process.env.SECRET
}

const adapter = new WebexAdapter(webexOptions);
server.use(restify.plugins.bodyParser());
adapter.registerWebhookSubscription('/api/messages');
adapter.getIdentity();
server.post('/api/messages', (req, res) => {
    adapter.processActivity(req, res, async (context) => {
        await context.sendActivity('Hey human!!');
    });

When I post a message to the bot (in Webex Teams) the webhook gets called and the bot receives the user's message.
At the step 'await context.sendActivity('Hey human!!');' it fails with the below error message
BadRequest: Unable to retrieve content.
POST https://api.ciscospark.com/v1/messages
WEBEX_TRACKING_ID: spark-js-sdk_9ba910cb-43ea-4800-88be-dde33e7ba825_7
I am using the following packages
"botbuilder": "~4.5.1",
"botbuilder-adapter-webex": "^1.0.2",
"restify": "~8.3.3"
Node version : v10.15.3

@RamPrasadMeenavalli Did you ever sort this out?
I am not able to reproduce this issue.

@benbrown - No. I haven't checked this after I reported the issue.
I will check it again if I still face the issue. Thank you.

@benbrown Hi Ben, I recently was working on an integration of MS Bot with WebEx Teams and faced this issue. Can you please help.


[iOS] latency issue

#20

@bugnano  Could #3 help you a little bit?

I suspect that the latency issue has to do with Nagle's algorithm
https://en.wikipedia.org/wiki/Nagle's_algorithm
so, apparently setting the TCP_NODELAY option to the socket would fix the issue.
The question is: Does the websocket server library that you're using expose the socket options (or something similar to disable Nagle's algorithm), or should a bug request be filled also for the ios websocket server library project?

Good point.
Alas the socket options are not exposed.
Please ask zwopple/PocketSocket for help.
Alternatively, if you are able to work this out, we could fork PocketSocket for the time being

Aren't you experiencing latency with an Android server too?
Because looking into the android server's code, it seems that TCP_NODELAY isn't enabled either.
I've been reading some more about this, and while an option can be provided server-side to turn it on/off, I think there's still an issue client side where Javascript WebSocket TCP_NODELAY isn't configurable (OS dependent). Please correct me if I'm wrong.
the-caveats-of-tcp-nodelay
sending-filler-data-on-websocket

That's strange, I thought that on Android the TCP_NODELAY option was enabled, as the latency on Android to Android connections is good.
To have a clear picture, this is the situation I'm facing:
Android server to Android client --> Good performance
iOS server to Android client --> Poor performance
Android server to iOS client --> Poor performance
iOS server to iOS client --> Poor performance
So, my proposed solution is to fix one problem at a time, and for me the next logical step would be to fix the iOS server to Android client issue, and probably the TCP_NODELAY option would fix this issue, we only have to try ;)
Still, you're right that we might not be able to do anything to fix the iOS client issue, but I think that that's another problem that we may address after fixing the iOS server.

fix one problem at a time

I completely agree 👍

Please test branch 1.4.1 and tell me if you see any difference.
wsserver.start(port, { ... tcpNoDelay : true ... })
applies to Android and iOS servers

I just fixed missing file in branch 1.4.1.

Ok, branch 1.4.1 fixes the iOS Server to Android Client issue, and it also seems to be slightly better on Android Server to Android client too.
Unfortunately, when it comes to iOS clients, the performance remains very poor, but at this point I think that it's a problem with iOS WkWebView/Safari than with the wsserver plugin, so, as far as I'm concerned, I think that we can close this issue.

Thank you. Closing now.


Fixed the 550 Errors that sometimes happen when deploying

Added some try catch blocks and other measures to catch out 550 errors like in these issues:
https://github.com/SamKirkland/FTP-Deploy-Action/issues/220
https://github.com/SamKirkland/FTP-Deploy-Action/issues/218
https://github.com/SamKirkland/FTP-Deploy-Action/issues/214
When using the action for my website I'd get build fails because the action would try to remove a non existent file, or add a file to a directory that didn't exist etc.
It has been linted and tested and I also used it to upload my website and it worked as desired.

After looking at this more the root cause of this bug is due to calling this.client.removeDir( with the full folder path instead of the relative folder path (calling with the absolute folder path would also work)


allow onBeforeChange to cancel change in tabs.

If onBeforeChange returns false, it cancels the tab change.
It looks like my npm install also built against an newer version of react (and particular its jsx transform) than you've been using. I'm not sure it'll work if you're using an older version of react.
Rationale:
We're using this in a section of our site with a "live session" feature. The live session has one leader and everyone else is a follower. When the leader clicks a tab, it broadcasts an event to all the followers to re-render on the same tab the leader is seeing. When a follower clicks a tab in the live session, we need to prevent it from rendering the other tab, since they should be at the same spot as the leader.

Oops... I my nmp install actually built against an old version of react's jsx filter. hmm.

Found an unrelated bug. Going to reopen from a different branch.

@adambaker Seems a good idea, please give other pull request so I can do the merge ;)


Make supports and updates not spendable

wallet.get_spendable_coins() could return coins that were being used either as a support or an update (although it was properly filtering claims), thus wallet could end up abandoning a support or an update when making a new transaction (either a regular send, or a claimtrie transaction)
You can test this easily through lbryum console by typing wallet.get_spendable_coins() , before this pull request, you could clearly see supports and updates being returned as spendable.

LGTM

rebased to master and added to new change log


[Bug] It's impossible to remove the last role/permission from the user.

Bug report
What I did
I added permission to the user. And then I removed that permission from the user. If the user had only one permission, then it is not removed. The same situation with the role given to the user. I can't cancel the last role.
What I expected to happen
Expected result: after unchecking the last role/ permission for the user, it should be removed from the user. The user must be left without a role/permission.
What happened
Actual result: nothing happens. The user still has the role/permission. They are not removed.
What I've already tried to fix it
I investigated and found out that backpack doesn't send any data if the last role/permission was unchecked, and doesn't update (delete) the data. I tried to send an empty array, if there is no role/permission, the role/permission was deleted successfully.
To work correctly, it's needed to send an empty array if all checkboxes are unchecked.
Is it a bug in the latest version of Backpack?
After I run composer update backpack/crud the bug is still there!
Backpack, Laravel, PHP, DB version
When I run php artisan backpack:version the output is:
PHP VERSION:
PHP 8.1.2
LARAVEL VERSION:
v9.16.0
BACKPACK VERSION:
5.1.2

Hello @akalinich thanks for the report and sorry for the bad experience.
I could confirm and sent the PR to fix it. #4462
This will be automatically closed when the PR gets merged.
Cheers

Thanks for the report @akalinich , and for the PR @pxpm . Let's move the conversation to https://github.com/Laravel-Backpack/CRUD/pull/4462 please - we'll probably review and merge by Monday. Thanks again!


Azure Account "Remove" Button Hidden Under Long Domain Name

Storage Explorer Version: 1.8.1
Build Number: 20190510.6
Platform/OS: Windows
Architecture: ia32
Regression From:
Commit: 947fc741
Bug Description
The "Remove" button on the Azure Subscription Account Management is hidden under a long domain name:

Expected Experience
The domain name for Azure Subscription should not overlap the "Remove" account button. And should be easily accessible.
Thank you,

Hi @rrmistry thanks for the bug report. We'll work on getting this fixed, but in the meantime you should be able to use your tab key to access the Remove link.

Fix checked into master. Thanks again for the bug report @rrmistry!


We need auth emulator suit support/or any workaround

To provide integration tests to my team - I need to have ablitiy to create test users, login and create requests with token. Could you please suggest me any way to solve that problem?

Duplicate of #503

Duplicate of #503


LOG-4305: Fix time range selection from histogram

This PR allows the detail page view to select a time range using the histogram

/cherry-pick release-5.7

/lgtm


[Java][Number] Decrease end property of ModelResult by 1

Fixes 2254
Description
There is a disparity on the generation of the ModelResult, specifically the end property, which needs to be decreased by 1. See C# and Java. Also, JavaScript follows the behavior of C#.
Specific Changes

Decrease the end property of ModelResult by 1

Testing
These changes were validated using the PR 2439.
Parity of ModelResult

Test passing correctly using Java 8/11/15

Already merged in MS


[Searchable Snapshot] Propose API

WORK IN PROGRESS
Cluster Configuration
A "remote searcher" is a concept that should have broad applicability beyond searching remote snapshots, as it defines a node capable of searching (and caching) indexes hosted in a remote store, potentially in any number of formats beyond a snapshot. This proposes to create a new node role to define such a capability:
Node Configuration
A new node role will be introduced: remote_searcher. This role indicates that the node is capable of hosting "remote" shards where the data is authoritatively stored in a remote store. The index data will not be permanently stored on the local instance storage. The local disk can be used as a cache. The cache size can be configured with the following setting in opensearch.yml:
node.remote_searcher.cache.size: 100GB

Index creation API
Several options exist for the creating a remote index. This section details some of the options.
Proposal 1: Extend Snapshot Restore
A new parameter will be introduced in the snapshot restore API: storage_type.

Setting
Description

storage_type
Must be one of local or remote_snapshot. local is the default if not specified, and indicates that all snapshot metadata and index data will be downloaded to local instance storage. remote_snapshot indicates that snapshot metadata will be downloaded to the cluster but the remote repository will remain the authoritative store of the index data. Data will be downloaded and cached as necessary to service queries. At least one node in the cluster must be configured for the remote_searcher role in order to restore a snapshot of type remote_snapshot.

For example:
POST _snapshot/my-repository/2/_restore
{
  "indices": "my-index*",

  "storage_type": "remote_snapshot",  <-- NEW PARAMETER

  "ignore_unavailable": true,
  "include_global_state": false,
  "include_aliases": false,
  "partial": false,
  "rename_pattern": "opensearch-dashboards(.+)",
  "rename_replacement": "restored-opensearch-dashboards$1",
  "index_settings": {
    "index.blocks.read_only": false
  },
  "ignore_index_settings": [
    "index.refresh_interval"
  ]
}

Proposal 2: Extend Create Index
New settings will be introduced to the create index API under the index.remote namespace.

Setting
Description

index.remote
Defaults to false, indicating the index will exclusively use local instance storage. If true then the index.remote.datastore property must be specified.

index.remote.datastore.type
Type of remote store. Only snapshot is supported initially

index.remote.datastore.snapshot.id
Identifier of the snapshot to use as the source for this remote index.

PUT /my-index-1

{
  "settings": {
    "index.remote": true,
    "index.remote.datastore.type": "snapshot"
    "index.remote.datastore.snapshot.id": "my-snapshot-repo/1"
  }
}

@andrross thinking about APIs, would it make sense to consider a new attach / detach concept for the index? I believe one of the primarily use cases will be not the creation of the index knowing ahead of time it will be remote but shelving the existing indices (fe by data age or any other compliance requirements) into the remote ones.
The explicit attach API might be helpful when it the index in question is needed to be fully available for some prolonged period of time vs querying it in ad-hoc fashion (that would be solved by cache).
Curious to hear what do you think.

Do we think we'd want to permission differently attaching a remote index? Feels like we are manipulating index properties (size, location, type, id), no?

Feels like we are manipulating index properties

Yeah, that's what led me to propose it this way. Also, in any case we'd want the ability to get the current properties of an index, so index settings made sense for that. If we had imperative-style APIs (attach/detach) we'd also need APIs to describe the current attached or detached state, so it's not obvious to me that these properties are different from regular index settings.

I think what customer need is to create index backed by existing snapshot, which they will use the Create Index API?
but why we need to extend Snapshot Restore API? what the use-case for this?

I think what customer need is to create index backed by existing snapshot, which they will use the Create Index API? but why we need to extend Snapshot Restore API? what the use-case for this?

Snapshot restore is essentially an API for creating an index. Extending snapshot restore to create a searchable snapshot-type index makes a lot of sense, as a lot of the options in the API are still relevant (such as the renaming options, restoring partial snapshots, etc). Really the unresolved question here is whether there is a single API that can work with regular snapshots as well as the more forward looking option to create a remote index backed by the new remote store-type indexes that are being introduced.

@andrross what is your strong opinion about what to go with?

For the searchable snapshot phase of the storage roadmap, I think adding an option to the existing snapshot restore API is the right way to go for the reasons listed above. We're going to start with that approach for the initial development, but we can always change it based on feedback.
There are still many open questions about the API and user experience for later phases in the storage roadmap (#3739) but I intend to continue that discussion in separate issues. For the purposes of the searchable snapshot API though, I'm going to go ahead and close this issue with the intent to get an experimental version behind a feature flag that implements "proposal 1".


Update order of relation labels

Aux and COORD are too far down the chain.

Closed by https://github.com/latin-language-toolkit/arethusa-configs/commit/bf3720e15a8c98e65bc3d1721a1071fada2eac7e

Check again - did this went wrong?

@LFDM maybe we are suffering from there being 2 relations.json? One in arethusa and one in arethusa-configs? I think the app uses the one in arethusa??


Adds info on how to setup grind.

Should explicitly say how to call grind to people new to dart and pub.
Dart sdk installation might also be explicitly specified.

Thanks!


Fred

BugFixed: display 'technicien' for ROLE_TECH

Current coverage is 1.18% (diff: 0.00%)

Merging #16 into dev will not change coverage

@@               dev       #16   diff @@
========================================
  Files           12        12
  Lines          505       505
  Methods         69        69
  Messages         0         0
  Branches         0         0
========================================
  Hits             6         6
  Misses         499       499
  Partials         0         0

Powered by Codecov. Last update 013272e...b4f6933


updated a known issue

reported by customers.

@JohanFreelancer9 Copy editing is needed for this PR. Thanks.

@chrfox PR has been copyedited and is ready for final review, could you please check and merge? Thanks!

re assigning to @robmazz

for your action @robmazz @LauraWi . Please help publish this.

@anwara-ms  - I'll push this live, but I'll remove the "We will fix this" sentence at the end of the addition and republish today.


Scrap "Selling Cash" & "Currency" rows

I want to scrap table data only for selling cash value 60 or above in the following code,
//Step 1: Create CSV file with Date stamp in filename and headers Name , Price
dump Name,Price to newTable.csv
//Step 2: Website link from where to extract
https://www.dbs.com/in/treasures/rates-online/foreign-currency-foreign-exchange.page
wait 30
//Step 3: Count rows
rows = count('(//table[contains(@Class,"sc-dxgOiQ dzHkKC tbl-primary mBot-16")])[1]//tbody/tr')
echo rows
//Step 4: Run loop for all the rows to extract currency-foreign-exchange
for row from 1 to rows
{
read (//table[contains(@Class,"sc-dxgOiQ dzHkKC tbl-primary mBot-16")])[1]//tbody/tr[row]/td[1] to name
echo name
write name to name
read (//table[contains(@Class,"sc-dxgOiQ dzHkKC tbl-primary mBot-16")])[1]//tbody/tr[row]/td[3] to price
echo price
forex_rate= [name, price]
//Make entry on current currency Name and it's Price to the CSV File created at step 1
//echo csv_row(forex_rate)
write csv_row(forex_rate) to newTable.csv
}
As I am a beginner to RPA, this will be a great help

Hi @devanimesh1 you can do something like this. Note that in below example, I use spaces indentation instead of {} to enclose the code block. From TagUI v6 onwards, it supports such indentation style for code blocks.
price = "70"
if Number(price) more than or equals to 60
    //Make entry on current currency Name and it's Price to the CSV File created at step 1
    //echo `csv_row(forex_rate)`
    write `csv_row(forex_rate)` to newTable.csv

Adding on, TagUI's backend language is JavaScript, so you can google and search for standard JavaScript functions to do various operations as needed.


Updates for Search Service

Update Inner class to reflect createOrUpdate as a LRO
Updates for tests, sample code and recordings
Updates for java to csharp translation file and other annotations

Can one of the admins verify this patch?


503 error when cheking mime type

503 response is thrown in this line https://github.com/zendframework/zf2/blob/b952b35a18d13e3645adcf5721440cbc446d923d/library/Zend/Validator/File/MimeType.php#L205
I am not sure what can cause this. Even not sure how to debug it.
Any Ideas?
The function is opening this file /usr/share/misc/magic.mgc maybe file is corrupted or something but maybe exception could be thrown here or something?
php version: 5.6.6

I think it's same as https://github.com/zendframework/zf2/issues/6493


chore: migrate the web3task gitbook into web3dev ownership

Currently, @RafaDSan has ownership over Gitbook, but we need to move this access into Web3Dev ownership.
Proposal:

Get the template from gitbook
Create an web3dev account with help of @annabida
Help migrate the content into the new gitbook account

This issue was resolved at Gitbook Web3Task


hdiutil: attach failed - No such file or directory

I have tried both Sierra and Mojave installers. neither has a SharedSupport folder (no InstallESD.dmg) so it fails. I downloaded both today from the App Store. What should I do?

Closing because I didn't download the full installer, only the 20MB online version. Issue on my end.


Export association is closed error

The goal is to ignore the error in case of pc.Close().

I will create another PR.


Report warnings

This is an attempt to include warnings in the Phytest reports. It does the following:

As well as emitting a PhytestWarning, utils.assert_or_warn will mark the test as an xfail.

The short-summary terminal output from Phytest will not display XFailed results, but they will show in the pass/fail markers as green x values. e.g.:

The html report is modified to replace "expected failures" in the summary with "warnings". In the table, any XFailed results are renamed "Warning". e.g.:

I'm leaving this a draft just now as it feels a little bit hacky and I think will need discussion and testing. However, I do think this is a good solution from the point of view that it is seamless for the end user unless they want to write their own xfail tests within a Phytest suite. This solution also has the added benefit of requiring no PRs to upstream projects.
Keen to hear what everyone thinks!

great work. Interesting solution. Let's talk about it in person tomorrow.

Replacing references to "xfailed" with "phytest-warned" or similar is doable, but it requires a different fix for naked pytest and with pytest-sugar (like we use). Since we also advertise being able to use Phytest as a pytest plugin, I don't think we want to do something that may conflict with other plugins that the user may want to use. Looks like the best way would be to just figure out how to include standard warnings in pytest-html itself. I have no idea at this stage how much work that would be though...
Based on all this, I'm marking this as a failed experiment.


Compability with ES6

Hi!
I'm using your library with the node version 16.14 that uses ES6, when I execute  command npm run swagger-autogen the file swagger_output.json don't generate. Do your library has compatibility for ES6?

Hey!
I had to implement a whole workaround using a .cjs file and so on, in order to use this lib.
But I really don't like doing this.
I guess it needs a contribution for this feature.
If I have some free time I'm gonna check this out.

Hi @marcelino-borges! I'll add this extension in the next version. Thnks!

Hi @LaercioNazareno! Sorry for the delay. What OS are you using?

Hi @marcelino-borges! I'll add this extension in the next version. Thnks!

Hey, great news!! Do you have any prevision on this next version?

Hi @marcelino-borges! I have just submitted the new version (2.21.9). If the problem persists, let me know.

Wow! You are The Flash, aren't you? haha
I noticed there's still the issue with the missing @types, but a simple workaround with a declare module "swagger-autogen" sorts that.
Besides that, right now I'm getting this error:
Error! File not found: '/src/routes/index.ts'
Swagger-autogen:  Failed

But as you can see all my routes are into this path. Do you see anything wrong that I'm not seeing here? lol
Ps.: I'm running another config file from the command line, which calls this function of the screenshot.

Hi @marcelino-borges. There was a bug in the latest versions about this. The version 2.21.10 has fixed it. Try with this version and let me know if the problem persists, please.

Hi @marcelino-borges. There was a bug in the latest versions about this. The version 2.21.10 has fixed it. Try with this version and let me know if the problem persists, please.

Now I'm getting this error again, even having my file declaring the module:

There it goes: https://github.com/marcelino-borges/swagger-autogen-test
I put only the necessary files in this demo.
Thanks!

@marcelino-borges, I've implemented two files in the node_modules/swagger-autogen, and here it works with your project. But as I use MacOS, could you try in your OS, please? The two files are:
in the root of node_modules/swagger-autogen
swagger-autogen.d.ts:
declare function swaggerAutogen (args?: any, endpointsFiles?: any, data?: any): (outputFile: any, endpointsFiles: any, data?: any) => Promise<false | {
    success: boolean;
    data: any;
}>;

export = swaggerAutogen;

tsconfig.json:
{
  "include": [
      "src/**/*"
  ],
  "compilerOptions": {
      "module": "es2015",
      "allowJs": true,
      "declaration": true,
      "strict": true,
      "emitDeclarationOnly": true,
      "outDir": "dist",
      "declarationMap": true
  }
}

let me know if it works or not, please.

EUREKA!
It works!!!

Thanks!

The version 2.22.0 has implemented this. Any problem, let me know.


General styling of lobby page

especially all the top info like # users, share URL, etc
canvas and stream should be more level?
buttons could be arranged better

Canvas should have space reserved for loading, since we know how many users in a lobby at a time

This was already done 😄


  This is induced by the following line in preprocess():
data = ad.X.toarray()  # Sparse

Is there a way to keep the matrix sparse?  I can successfully work with this dataset using other tools, but scvi's attempt to make a dense matrix cannot succeed for a large dataset.
Environment:
-OS: Ubuntu 16.04
-Python: 3.6.8
-scvi: 0.3.0
This is the output:
Preprocessing dataset
---------------------------------------------------------------------------
MemoryError                               Traceback (most recent call last)
<ipython-input-3-138de299bfb8> in <module>
      1 # load
      2 dataset = AnnDataset('data/dataset.h5ad',
----> 3                      save_path=path)

~/anaconda3/envs/pyro/lib/python3.6/site-packages/scvi/dataset/anndata.py in __init__(self, filename, save_path, url, new_n_genes, subset_genes)
     36         self.url = url
     37
---> 38         data, gene_names = self.download_and_preprocess()
     39
     40         super().__init__(*GeneExpressionDataset.get_attributes_from_matrix(data),

~/anaconda3/envs/pyro/lib/python3.6/site-packages/scvi/dataset/dataset.py in download_and_preprocess(self)
     61     def download_and_preprocess(self):
     62         self.download()
---> 63         return self.preprocess()
     64
     65     def collate_fn(self, batch):

~/anaconda3/envs/pyro/lib/python3.6/site-packages/scvi/dataset/anndata.py in preprocess(self)
     52             data = ad.X.copy()  # Dense
     53         else:
---> 54             data = ad.X.toarray()  # Sparse
     55         select = data.sum(axis=1) > 0  # Take out cells that doesn't express any gene
     56         data = data[select, :]

~/anaconda3/envs/pyro/lib/python3.6/site-packages/scipy/sparse/compressed.py in toarray(self, order, out)
    960         if out is None and order is None:
    961             order = self._swap('cf')[0]
--> 962         out = self._process_toarray_args(order, out)
    963         if not (out.flags.c_contiguous or out.flags.f_contiguous):
    964             raise ValueError('Output array must be C or F contiguous')

~/anaconda3/envs/pyro/lib/python3.6/site-packages/scipy/sparse/base.py in _process_toarray_args(self, order, out)
   1185             return out
   1186         else:
-> 1187             return np.zeros(self.shape, dtype=self.dtype, order=order)
   1188
   1189

MemoryError:

If the idea of refactoring to use a sparse matrix datatype is too much... perhaps a different patch could be to switch the order of operations: subset to subset_genes first, and then make a massive dense matrix.

I'm not sure why that happens in the AnnDataset class. It seems in general sparse matrices are supported.
Just wanted to share my general strategy for using sparse AnnData with scVI is along these lines:
adata = anndata.read('file.h5ad')
# Here adata.X is sparse

X, local_means, local_vars, batch_indices, labels = \
GeneExpressionDataset.get_attributes_from_matrix(adata.X)
# The returned X here is also sparse

cells_dataset = \
GeneExpressionDataset(X, local_means, local_vars, batch_indices, labels,
                      gene_names=np.array(adata.var.index.values, dtype=str))
# Also here the field cells_dataset.X is sparse

cells_dataset.subsample_genes(1000)

vae = VAE(...)
trainer = UnsupervisedTrainer(vae, cells_dataset)
etc...

Hi,
Thanks Valentine for the suggestion, our API is not perfectly optimized yet so there are two ways:

modifying the AnnData class to keep sparsity if wanted
use Valentine's code snippet

Let me know if you need more information,
Romain

Since sparse matrix are supported, I think we should just remove the casting mechanism in scvi.dataset.anndata:
if isinstance(ad.X, np.ndarray):
    data = ad.X.copy()  # Dense
else:
    data = ad.X.toarray()  # Sparse

and replace it with:
data = ad.X

Sounds good!

The only thing is that loading data from sparse matrices is much slower than using dense ones. We may still want to convert from sparse to dense by default if the total dense matrix size less than a couple gigabytes.


[WIP] saml: Implement IdP-initated logout for Keycloak.

Missing tests, but has been confirmed to work as expected manually for
now.
Documentation added in docs/production/authentication-methods.md.
Aside of adding tests, this probably could use some refactoring to use class polymorphism instead of the weird saml_response_or_request thing, though perhaps that's for a follow-up PR after adding SP-initiated logout as well?
One caveat that needs noting is that this of course won't log the user out of the mobile app, since it's not session-based. If we really need, we can hackily change the user's API key when processing a SAML LogoutRequest, but probably it's better to not do that and live with the limitation for now until we have more advanced managed of keys for mobile app "sessions" and can handle this properly?

@mateuszmandera thanks for working on this!  I think the high-level approach looks good; posted comments on most of the details.


Invalid topic in addHandler bricks registerAll (Mandatory GDPR Webhooks)

Issue summary
Adding a webhook handler for a topic that does not exist via addHandler/addHandlers causes the registerAll function to error.
This is undesirable behavior, as the Mandatory GDPR webhook topics (customers/data_request, customers/redact, and shop/redact) are considered invalid topics; this means that if you attempt to add handlers for the GDPR webhooks, the registerAll function no longer works.
Now, this is complicated by the fact that you aren't SUPPOSED to add handlers for the GDPR topics...but you can, and doing so is actually entirely functional!
Given that this package lacks a built-in method to validate the mandatory GDPR webhooks (or any documentation on doing so), this is a pretty severe gap in usability for many developers.
Now, one thing you CAN do is addHandler the GDPR topics, and then manually call Shopify.Webhooks.register for each non-GDPR topic you subscribe to for a shop...but this is extremely unintuitive, and again -- undocumented behavior.
Expected behavior
Two points of expected/desired behavior:

There is a way to register the mandatory GDPR webhooks without manually validating them per the comments here: https://github.com/Shopify/shopify-node-api/issues/256

Attempting to add a handler for an invalid topic does not completely break functionality of registerAll.

Actual behavior
Calling addHandler for GDPR webhook topics (or any other invalid topic) renders registerAll unusable.
Two options for developers: either manually validate GDPR webhooks, or manually call register for each non-GDPR webhook.
Steps to reproduce the problem

Add a handler for an invalid webhook topic. (I.E. Mandatory GDPR webhooks)
Add a handler for any valid webhook topic.
Call Shopify.Webhooks.Registry.registerAll (Note that individually calling register on each non-GDPR topic DOES work)
Error

Reduced test case
 function demoHandler() {
    console.log('hello');
  }

  Shopify.Webhooks.Registry.addHandler("APP_UNINSTALLED", {
    path: '/webhooks',
    webhookHandler: demoHandler,
  });
  Shopify.Webhooks.Registry.addHandler("CUSTOMERS_DATA_REQUEST", {
    path: '/webhooks',
    webhookHandler: demoHandler,
  });

followed by, as part of shop auth:
      const webhookRegistrationResult = await Shopify.Webhooks.Registry.registerAll({ shop, accessToken });

Checklist

[x ] I have described this issue in a way that is actionable (if possible)

@paulomarg does addHandler support the GDPR topics now?

@zirkelc It technically did before, as long as you didn't call registerAll! It's just easier to use out of the box now. :)


Command succeeds with redundant words, bug or not?

Add command succeeds even with extra info past the end. Should we address this in UG or by returning an error?
What is considered a bug:

So it would be a bug if the command succeeding "differs from normal expectations"

Closing as it is defined as expected behavior in UG.


Update README.md

Add link for the rubydoc

There was 2 spaces missing after the documentation line so I've added them (in markdown, 2 spaces at the end of a line = new line).


Fix process query response missing errors

In processQueryResponse if a token is an array of columnStruct, it would break the loop and fail to process the following tokens. These tokens should be processed since they may contain errors from the query that should be returned.
There is a test included which sets up a table, and performs an insert query with output variables. The first token returned in process query response is an array of column struct, relating to the output variables, and second token is an error indicating you cannot insert a value for an identity column. Previously, it would return with no error and an empty set of rows, and now it catches the error and returns it from the query.

I'm am checking in with my company to confirm I can sign for the CLA, and will return.
If there is anything else missing from the PR I can provide please let me know.

thx for the PR!
Do we need a separate test case for the message loop version of a query?

Thank you for the feedback! I am working on the suggested changes.

I'm unsure what a message loop version of a query and was wondering if you could clarify?
I also moved the state management to a Rows method as you suggested, but ran into some challenges I described in the thread above. Let me know if I can provide any more info, thanks!

the message loop version of a query is tested here https://github.com/microsoft/go-mssqldb/blob/ada30cbaf031d81144e5c5487ddefd4f3dac0ef5/queries_go19_test.go#L1255

Thank you for the message loop query version, I will add a test case for this.

I did not understand that a query in golang may result in an error, but that error will not be returned from the query method, rather it must be checked via rows.Err() after closing the error. Sorry for the distraction.


Prepare for DNA 3.20

Codecov Report

Merging #27 (a8d8711) into main (c91c050) will increase coverage by 0.17%.
The diff coverage is 25.00%.

@@            Coverage Diff             @@
##             main      #27      +/-   ##
==========================================
+ Coverage   79.84%   80.01%   +0.17%
==========================================
  Files          18       18
  Lines        2282     2287       +5
  Branches      100      100
==========================================
+ Hits         1822     1830       +8
+ Misses        459      456       -3
  Partials        1        1

Impacted Files
Coverage Δ

src/Components/Transition.tsx
24.77% <0.00%> (-0.95%)
:arrow_down:

src/App.tsx
78.63% <100.00%> (+0.06%)
:arrow_up:

src/Router/Router.ts
86.55% <0.00%> (+0.52%)
:arrow_up:

src/Router/Response.ts
84.35% <0.00%> (+0.55%)
:arrow_up:

src/Router/Request.ts
82.92% <0.00%> (+1.21%)
:arrow_up:

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


Unchecking row returns old value

I have checked a row , got the checked row fine but when i unchecked the row its not removing the value, adding on checked but not removing on uncheck, Can you please help

it keeps adding but not removing on uncheck , same  item adding multiple times

Hi @WordpressArtist
I can't implement this problem on latest version, more detail please.

I installed "vue3-table-lite": "^1.2.4"
now i set up
@return-checked-rows="onCheckRows"
:checked-return-type="'id'"
:has-checkbox="true"
now when i click on bulk check and uncheck works fine
but when i check individual row, it logs. the value
onCheckRows(rows) {
conosle.log(rows)
}
but  when i uncheck, that the id must not be there but it still there.
Example:
i checked 1 row , please check log (i got 22 )

Now uncheck the same item i can see still 22 on log

now i check that item again i got 2 same data previous selected and current selected

Now if i uncheck it still same 2 datas
if i again check it keeps adding item on array but not removing

@WordpressArtist
Thanks for your information, the bug already fixed after v1.3.0.
So, upgrade than  v1.3.0 up please.


Improve key rotation docs

The documentation on key rotation is not up to date and does not contain information about key rotation for the VSP.
To fix this, the PR proposes:

updated information on rotating MSA keys
new content for rotating VSP keys
keeping generic instructions for service provider (non-VSP) key rotation as there still is a user need for this
re-organising the order of sections to: MSA, VSP, SP
linking the first reference to IDAP CA to sectio on keys and certs
improving phrasing according to the style guide
re-writing steps to correspond to something an RP does (things happening at the Verify end need not be described as separate steps)
replacing highlights in code blocks with text explanations to improve accessibility

🗝 🔄 📜

@andy-paine removed file from .gitignore and added restart steps after every config change 👌

Happy to approve from content POV, providing all 2i changes made.


Run coverage on stable

Motivation
We've been running our coverage collection on nightly Rust as that was the only access to the source-based coverage support to give the best / most reliable data.
Specifications

Designs

Solution

Source-based coverage support has been stabilized and the cargo component we use to wrap up the compiler flags has updated accordingly. This PR switches our Coverage workflow to Rust stable, which provides more reliable stability guarantees in general for our CI.
Closes #3887
Review

Anyone can review, low priority.
The coverage.patch.yml workflow is a no-op so there is no Rust channel to update there.

Leaving this comment here so we don't forget about the patch file.

I looked at coverage.patch.yml and it's a no-op, no Rust toolchain to update

Leaving this comment here so we don't forget about the patch file.

I looked at coverage.patch.yml and it's a no-op, no Rust toolchain to update

The job name in the patch file needs to be changed, so we can require the job in branch protection rules, but skip it when it is not needed:
https://github.com/ZcashFoundation/zebra/blob/e4ca6c358ec9807c5c97e610d51297a0dab7996d/.github/workflows/coverage.patch.yml#L16

@Mergifyio update

@mergifyio requeue

@mergifyio update


Add option to leave tarfile in the queue after processing

In order to allow more flexibility in processing, it would be useful to have an option to leave processed tarfiles in the s3 queue after successful processing with l8_process_scene.py.
This PR adds a --no-delete-queue option that, if specified, leaves processed scenes in the queue.

cc @kapadia

Looks fine, just note that if the tarball is not removed from the s3://landsat-pds/tarq bucket, it will be reprocessed again, and be recorded in the scene_list.csv again.


Request... Custom commands

Would it be possible make a custom command witch do a series of keypress
like "NetFlix" as hotword and being translated in my case into "SMART MENU",
"RIGHT BUTTON", "RIGHT BUTTON", "ENTER".
Michael

it looks like apps can be launched natively on the device for new TVS. Ill look into being able to launch apps on the TV.
https://github.com/Ape/samsungctl/issues/75

After more research this is not possible. Simply because the Alexa does not allow commands with the word Netflix in it :(
Trying commands like
launch/open Netflix on the TV
or
change the channel to Netflix on the TV
Causes Alexa to try to find a fire stick to launch apps on, even when there are none on your system. I have not found anyway to allow this app to respond to that command and launch its own apps. Closing for now until they update their SDK or someone finds a way to do this.

The "hot word" could be anything now where Netflix isn't allowed :)


how to use it in Azure China

i tried to overwrite but failed, any ideas?
azurerm.adalfns.authentication_endpoint='https://login.chinacloudapi.cn/'
azurerm.adalfns.resource='https://management.chinacloudapi.cn/'

@pjshi23 thanks for logging this issue. I have made the endpoint values optional arguments to the get_access_token() function. If this doesn't work we'll need to investigate the adal library and log an issue there.

Note until there's a long term fix for this @pjshi23 has created MCazurerm specifically for Azure China. See https://pypi.python.org/pypi/mcazurerm

Resolved in azurerm 0.8.1. https://github.com/gbowerman/azurerm/commit/3703e4d5aa1be95c11e29e7b7727e8da7e8ea914 - set national or government cloud access using three environment variables: AZURE_RM_ENDPOINT, AZURE_AUTH_ENDPOINT, AZURE_RESOURCE_ENDPOINT.
@pjshi23 thanks for raising this issue.


Document Scenic.Component callbacks

Description
Also add a missing comma in a text_height example
Motivation and Context
Make it easier to understand how to implement your own component.

Types of changes

[ ] Bug fix (a non-breaking change which fixes an issue)
[ ] New feature (a non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to
not work as expected)
[x] Improvement/refactoring (non-breaking change that doesn't add any feature
but make things better)

Checklist

[x] Check other PRs and make sure that the changes are not done yet.
[x] The PR title is no longer than 64 characters.

Great, thanks!


goToPage with noAnimation: true is good but, in the case of a current index at a close distance to the target index the animation still occurs

recently a parameter noAnimation was introduced to:

FlexScrollView.prototype.goToPage = function(index, noAnimation) {
I have performed some tests with noAnimation: true, with a full screen FlexScrollView containing full screen slides (i.e. everything set to size: [1024x768]). FlexScrollView is set with margin: 1024.
Results: goToPage with noAnimation: true performs as expected ONLY in the case that the target slide is at a distance of >2 slides from the current index.
In the case that the target slide is close to the target slide, i.e. a distance of 1 or 2 slides, the animation still occurs even though noAnimation: true has been set.
Thanks.

Alright, thanks for letting me know. I'll do some tests with test and get back to you.

Hi, a quick question, were you using option "flow: true" on the FlexScrollView? If so, than I'm pretty sure it's fixed now with the commit I just did on master.

Yes. The FlexScrollView in question was set with "flow: true".
I will re-test on master and share the results.
Thanks

Re-tested this scenario against master: git://github.com/IjzerenHein/famous-flex.git#661ad26
OK. Looks like it's resolved. Thanks!

Awesome!


fix: check pending eligible endpoints

Currently https://testgrid.k8s.io/sig-arch-conformance#apisnoop-conformance-gate fails with
==================
UNTESTED ENDPOINTS
getStoragemigrationAPIGroup
==================
ERROR: You have 1 untested endpoints
==================

This fix filters out any pending eligible endpoints from the list of untested endpoints which should produce the following result
==================
UNTESTED ENDPOINTS

==================
You have 0 untested endpoints
==================

Temporary work around while fixing a job.
/close