Fix the issue where close button of the modal/popup screen is not visible on some screen sizes

Fixes #6
Description
I had removed the unnecessary padding which was due to default materialize css nature. Also added the overflow : scroll property so that the close button is visible under all screen sizes and under all browsers.
Added a new functionality where clicking outside the popup screen closes it.
Screenshots

Previous Version

New Version

@kgodey please review this PR. If there is anything else I can work to solve this issue please let me know.

I'll leave it to @soccerdroid or @mathemancer to review.

Do you think the horizontal scroll bar could be removed? It seems to have no functionality.

Do you think the horizontal scroll bar could be removed? It seems to have no functionality.
@soccerdroid
yes I will remove it
Should I send you a new PR?
Is everything else alright?

The padding is set for aesthetic, so we don't have text too close to the borders. Like you mentioned, it follows material design. Why do think the padding was unnecessary?

Do you think the horizontal scroll bar could be removed? It seems to have no functionality.
@soccerdroid
yes I will remove it
Should I send you a new PR?
Is everything else alright?

You can do a git commit --amend and force push your changes, they will be reflected in this same PR

Oh and please add the Developer Certificate of Origin in your PR description, I just realized it was not included.

The padding is set for aesthetic, so we don't have text too close to the borders. Like you mentioned, it follows material design. Why do think the padding was unnecessary?
I thought that was unnecessary in the sense that they were causing overflow in my device (before I used the overflow property). But you are right I think i should revert it back.
Thanks @soccerdroid  for the advice.
I will make changes as directed by you.

@soccerdroid I had made the necessary changes directed by you and had included the Developer Certificate Of Origin.  Please have a look again

@soccerdroid is the PR now good to merge? Or do I need to make any other changes?

@kgodey the PR has been approved. Can you please merge it now?


[feature] #3470, #4299, #4300: improve config debug-ability

Description
This PR contains a major improvement of the work done in https://github.com/hyperledger/iroha/pull/4239. This is a big step towards better debug-ability of Iroha configuration system due to better errors. And yet, there are still many things to improve further!
Key points:

New ConfigReader API, which handles configuration sources in a dynamic manner rather than relying on serde completely, thus giving high flexibility in how to handle different cases. It is also built targeting Iroha domain specifically and doesn't have any extra stuff. Much less boilerplate.
Heavy use of error_stack crate, which gives higher control over errors reporting than eyre does.
--trace-config works! Uses log & stderrlog crates for it. Seems to be not a heavy solution for this. It is completely separate from tracing stack.

Further enhancements
There are still many things that might be improved.

Enhance use of attachments API provided by error_stack:

Use hooks to colour them conditionally
Attach hints and suggestions
Unify attachments further as specific structs, not just format!

Use spans to config files, and build reports showing the actual file contents
Fine-tune API of ConfigReader, use better naming and abstractions, perhaps.
Optimise internals of ConfigReader. There is lots of naive code and extra allocations.
Find a way to reduce extra context changes in error_stack. Sometimes it would be really cool to just "forward" underlying context without changing it, but currently it's how error_stack is designed. So, we have sometimes stacks like "Failed to load config -> Failed to load config -> failed to read file system -> ..."
Possibly, introduce a slim compile feature for Iroha, which will remove --trace-config support alongside with log & stderrlog dependencies, and more
Use a single multihash string for private keys, thus removing necessity to set private keys in env as two separate variables (e.g. PRIVATE_KEY_ALGORITHM + PRIVATE_KEY_PAYLOAD, and removing cratchy env_custom API from ConfigReader

https://github.com/hyperledger/iroha/issues/4412

Migrate to Axum, finally! Errors handling in Torii might be improved a lot.

https://github.com/hyperledger/iroha/issues/3776

Linked isues
Closes #3470, #4299, #4300
Gallery
Note: I've changed some bits after making these screenshots.

Some terminal screenshots

TODO

[ ] Document internals of iroha_config_base
[ ] Fix iroha_client_cli compilation failure
[ ] Enabled failing test back: fails_with_no_trusted... in cli/src/lib.rs

@DCNick3, I would appreciate if you can take a look at the procmacro I wrote here, in config/base/derive.

Debugging CI: new reports are really helpful!

A bit concerned with the inclusion of source code location in the error trace (the at some/source/location.rs lines). I don't think it's useful for users and questionably useful for iroha developers. Maybe remove it/gate being some verbosity flag?

A bit concerned with the inclusion of source code location in the error trace (the at some/source/location.rs lines). I don't think it's useful for users and questionably useful for iroha developers. Maybe remove it/gate being some verbosity flag?

I am concerned too. There are also complete backtraces attached, which are displayed in dev environment. Anyway, we can disable Location's printing using the hooks. I used it in tests to capture snapshots. Will think about it.

@Erigara @mversic if you can, please take a look at the overall workspace changes, outside of the ConfigReader API.
Here is an overview of how changes are spread:

The major part, which is under /config/base, is under the review of @DCNick3.
I am asking you about looking at other parts.


Remove unnecessary sentences

Remove the unnecessary sentences in README.md.
This PR has the same changes as closed PR #20.

@Yasami Sorry for late response, have been tied up with work recently. The changes looks good to me, and I think it will be really helpful if we can merge this.
By any chance have you signed before Artima Inc. CLA:
http://www.artima.com/cla/contributorsLicenseAgreement.pdf
or for Corporate:
http://www.artima.com/cla/corpContributorsLicenseAgreement.pdf
Please kindly print, sign and scan back a copy to<EMAIL_ADDRESS>we apologize for any inconvenience caused, we intended to improve the process but somehow it just didn't happen yet. Good news is it is getting near, though.

@cheeseng
Thank you for your reviewing.
I signed CLA and sent the mail.

@bvenners Changes looks all good to me, can you merge this first when you get a chance?
Thank you!


CSS Fix

I saw that the css wasnt working, and that you guys had a lot of assets being inlined with html, so i decided i would make some bundles for you.

font and mozilla firefox layout fix

when running ./bower_commands.sh moving fonts fail

If you are also planning to fix css issues, please look at #23
I just saw what you were talking about in terms of the Mozilla firefox fix:

If you'd like to submit a separate PR for that only and keep this pull request open, I'm totally down for that.

I cannot reproduce? :face_with_head_bandage: :face_with_head_bandage:
its not even like a tricky script, is the thing...

Tried running the bash script on a fresh install straight from your forked master, this was the output.
172-16-38-193:trumptracker.github.io-master USER$ ./bower_commands.sh
Installing Bower Components
bower cached        https://github.com/twbs/bootstrap.git#3.3.7
bower validate      3.3.7 against https://github.com/twbs/bootstrap.git#3.3.7
bower cached        https://github.com/jquery/jquery-dist.git#3.1.1
bower validate      3.1.1 against https://github.com/jquery/jquery-dist.git#1.9.1 - 3
bower install       bootstrap#3.3.7
bower install       jquery#3.1.1

bootstrap#3.3.7 bower_components/bootstrap
└── jquery#3.1.1

jquery#3.1.1 bower_components/jquery

   ╭─────────────────────────────────────╮
   │                                     │
   │   Update available 1.7.9 → 1.8.0    │
   │   Run npm i -g bower to update      │
   │                                     │
   ╰─────────────────────────────────────╯

Assembling CSS and JS Assets
  In directory: /Users/virenmohindra/Desktop/trumptracker.github.io-master
    Script Location: /Users/virenmohindra/Desktop/trumptracker.github.io-master
    Script Name: ./bower_commands.sh

  Concatenating CSS:
    Files (in order): ('<!-- Latest compiled and minified CSS -->')
      Bootstrap Minified CSS
      Custom Stylesheet
tr: Illegal byte sequence
    Making /tmp/styles.css

    Successfully made ./assets/css.css.

  Concatenating JS:
    Order Summary:

      <!-- Font Awesome -->
      <!-- jQuery (necessary for Bootstrap's JavaScript plugins) -->
      <!-- Latest compiled and minified JavaScript --> (bootstrap)
    File Sources:

    Successfully made ./assets/js.js.

Done Assembling Assets
Moving fonts
  cp ./bower_components/bootstrap/dist/fonts ./fonts -r
usage: cp [-R [-H | -L | -P]] [-fi | -n] [-apvX] source_file target_file
       cp [-R [-H | -L | -P]] [-fi | -n] [-apvX] source_file ... target_directory
moving fonts failed
done

172-16-38-193:trumptracker.github.io-master virenmohindra$ gem install jekyll bundler
Successfully installed jekyll-3.3.1
Parsing documentation for jekyll-3.3.1
Done installing documentation for jekyll after 1 seconds
Successfully installed bundler-1.13.6
Parsing documentation for bundler-1.13.6
Done installing documentation for bundler after 3 seconds
2 gems installed

tr: Illegal byte sequence
moving fonts failed

I'm guessing there was a syntax error with cp


[chakracore] upgrade to latest 1.4.1 and fix #665

upgrade to latest 1.4.1(https://github.com/Microsoft/ChakraCore/releases)
fix https://github.com/Microsoft/vcpkg/issues/665 by disable /WX
add ch.exe, GCStres.exe, rl.exe into tools directory(which I think maybe useful).

Thanks for the PR!


Artemis .ast files fail to parse

I've discovered an issue where, at least with one game, Yukiiro Sign, VNTextPatch fails to parse the .ast files, merely reporting 0 lines and characters. All of the scripts are encoded UTF-8, and I've confirmed they render correctly when opened in a text editor. I've attached the script files for reference.
Yukiiro Sign script.zip

How about Sakura no Uta after H2O -FOOTPRINTS IN THE SAND-?

Yukiiro sign is highly get localized by Shiravune since they have released Chinese tranation for it


query on Composite index using java api

OrientDB Version, operating system, or hardware.
Using OrientDB 2.1.12
Operating System

[ ] Linux
[x] MacOSX
[ ] Windows
[ ] Other Unix
[ ] Other, name?

I create a composite btree index like :
final String srcInxName = edgeType.name() + "." + START_VERTEX_ID;
edgeClz.createIndex(srcInxName, OClass.INDEX_TYPE.NOTUNIQUE.toString(),
                    null, null, new String[]{START_VERTEX_ID, END_VERTEX_ID});

and insert three records,  then query like :
OIndex<?> index = getDocumentDataBase().getMetadata().getIndexManager()
                  .getIndex(type + "." + START_VERTEX_ID);
Object obj = index.get(new OCompositeKey(vertexId.getId()));
indexValue =  (Collection) obj;
Object result = getDocumentDataBase().command(
        new OSQLSynchQuery<>("SELECT FROM INDEX:" + type + "." + START_VERTEX_ID + " WHERE key = " + vertexId.getId()))
        .execute();

but the first query by java return one result; and the sql-like query return 2 results as expected.
what's the problem ?

@BruceMu
do you use remote connection?

@BruceMu
in case of Partial Composite match you should use this API
OCompositeKey iKey = new OCompositeKey(vertexId.getId());
OIndexCursor curstor = index.iterateEntriesBetween(iKey, true, iKey, true, true);

I use plocal orientdb.

So if I used sql-like query , it will call the 'iterateEntriesBetween(...)' method at last ?

@BruceMu
yes iterateEntriesBetween will be called

@BruceMu
did you try the api
OIndexCursor curstor = index.iterateEntriesBetween(iKey, true, iKey, true, true);

Did that work for you?
Thanks

I used sql-like query in my application at last, thanks for you suggestion, I will try it later.


BlockstoreCache, FeatureCollection, FeatureRegistration tests

Attempted testing of BlockStoreBehavior. I think testing of NodeBehavior implementers would become a lot easier if we wrap the Node class from NBitcoin.

For the Node class wrap you mean to use a nested pattern?


Add test and build for python 3.9

This PR tests OG-USA for Python 3.9.

Codecov Report

Merging #705 (7cd4ff7) into master (76471a5) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #705   +/-   ##
=======================================
  Coverage   84.41%   84.41%
=======================================
  Files          23       23
  Lines        3927     3927
=======================================
  Hits         3315     3315
  Misses        612      612

Flag
Coverage Δ

unittests
84.41% <ø> (ø)

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

Codecov Report

Merging #705 (7cd4ff7) into master (76471a5) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #705   +/-   ##
=======================================
  Coverage   84.41%   84.41%
=======================================
  Files          23       23
  Lines        3927     3927
=======================================
  Hits         3315     3315
  Misses        612      612

Flag
Coverage Δ

unittests
84.41% <ø> (ø)

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

@jdebacker. This is great. I think we should get rid of testing for Python 3.7. I think a good rule is to only have testing for two versions of Python at a time.

@jdebacker. This is great. I think we should get rid of testing for Python 3.7. I think a good rule is to only have testing for two versions of Python at a time.


Added Maze Generation to the section Projects/Open Source

@impulse I like this, I wonder if we should think of an examples section as it's not a full game but it's super useful and unique example that a lot of people might wanna use in their own games

@luanpotter sounds good to me. @Dannark would you mind reworking this PR to add a section for Examples?

@impulse  Sure, no problem.


More preview testing

Not working after https://github.com/ProjectPythia/cookbook-actions/pull/57

After https://github.com/ProjectPythia/cookbook-actions/pull/59 it's working!


Fixed irregular indentation

fixed find_indent() according to #120

This breaks indentation with just spaces

Can you please give me a line that breaks the indentation? It seems to work for me.

any line indented with just spaces, when you turn on strict_tabs

when strict_tabs is set, then only tabs are used for detecting indent right?
So, if the line contains no tabs then the expected behavior should be to not indent anything right?

when strict_tabs is set, then only tabs are used for detecting indent right?

no this is wrong. Here is the documentation
    When on, if there is a single tab in a line, only tabs are used to
    calculate the indentation level.
    When off, both spaces and tabs are used to calculate the indentation
    level.
    Only makes a difference if a line has a mix of tabs and spaces for
    indentation.

ahh, I see.. I'll push the corrected code in some time

that looks good :+1:
can you do a couple small things.

rename new_spaces to just spaces
format the file with luafmt
squash all your commits into one commit

First, I added the condition to correct the implementation of strict_tabs to the previous code.
However, this made that specific code a bit bulky and unappealing.
So I re-implemented the logic like this:
M.find_indent = function(line, shiftwidth, strict_tabs)
    local indent = 0
    local new_spaces = 0
    for ch in line:gmatch(".") do
        if ch == "	" then
            if strict_tabs and indent == 0 and new_spaces ~= 0 then
                return 0, false
	    end
            indent = indent + math.floor( new_spaces / shiftwidth ) + 1
            new_spaces = 0
        elseif ch == " " and ( not strict_tabs or indent == 0 ) then
            new_spaces = new_spaces + 1
        else
            break
        end
    end
    indent = indent + math.floor(new_spaces / shiftwidth)
    return indent, new_spaces % shiftwidth ~= 0
end

new_spaces is a counter of spaces since the last tab character. indent stores the definite number of indents read till now.
Keeping the number of tabs and space indents separately wasn't making any difference in this implementation so I decided to store them together in indent to clean the code and reduce calculations.
Implementing of strict_tabs is done in two parts:

if the line starts with some spaces and then has at least one tab:

if strict_tabs and indent == 0 and new_spaces ~= 0 then
return 0, false
end

if the line starns with some tabs and then has at least one space

elseif ch == " " and ( not strict_tabs or indent == 0 ) then

Please let me know if there should be any other modifications.

okay i will do that.
btw can ya please tell me how to squash the commits to one XD.. I am super new to git.
thanks

Ohh, that's super handy. Thanks.
I'll reformat and push.

2. format the file with [luafmt](https://github.com/CurtisFenner/luafmt)

I did everything you mentioned. One issue I faced was that luafmt converted all the indents in the file to tabs. The rest of the codebase was space indented so I substituted the indent to spaces.
Let me know if you want to keep that as tabs.

sorry i linked the wrong luafmt :man_facepalming:
i didn't even know there were 2
here is the correct link https://github.com/trixnz/lua-fmt

done

I found another small problem
with strict_tabs on, a line with first tab indentation, and then space indentation, needs to return extra = true (the second value that's returned)
eg. this line 		          fooo (2 tabs, then spaces)
should look like this

but in your PR it looks like this

I did consider this case in my testing. Here, strict_tabs is on and both spaces and tabs are present. So, according to the docs, only tabs will be considered in the indent.
Therefore, the space which comes after the tabs isn't even a part of the indent. and so, I return extra=false.
I understand that extra==true means that there is some more indentation in the line which hasn't been counted in the indent variable.
But if a user sets strict_tabs then they are not expecting spaces to be a part of indent in the first place right? Else it will imply a situation where 3tabs is indented the same as 2tabs+4spaces. I felt that a user who set strict tabs would not want this.
Please let me know what you think.
Ps. what color scheme are you using? I really like that shade of green:P

But if a user sets strict_tabs then they are not expecting spaces to be a part of indent in the first place right? Else it will imply a situation where 3tabs is indented the same as 2tabs+4spaces. I felt that a user who set strict tabs would not want this.

Yes, you are right, kindof.
It should depend on g:indent_blankline_show_trailing_blankline_indent
It basically is a trailing indent. It should match what the other lines are doing.
Take for example this code.
#include <stdio.h>

int main(int argc, char *argv[])
{
	printf("test string %s:%s():%d\n",
	       __FILE__, __func__, __LINE__);

	printf("hello");
}

in your PR it would look like this with show_trailing_blankline_indent on

But with it should look like this

But with show_trailing_blankline_indent off, it makes sense that it looks like you have it now

The color scheme is my version of one dark
https://github.com/lukas-reineke/onedark.nvim

Oh, I see...
So should I replace
[current]          if ((blankline and trail_indent) or extra) and (first_indent or #virtual_text > 0) then

with
[suggestion]       if ((blankline or extra) and trail_indent) and (first_indent or #virtual_text > 0) then

[in file init.lua]
Or is there any case where even though trail_indent is false, extra causes another indent character to be printed?

So should I replace...

Yes :+1:
This looks good now.
I don't have that much time today. I'll test a bit more tomorrow, but I think this covers everything now.
Good job

aight I squashed everything
thanks

Is everything alright?

Yeah sorry just didn't have time
all good
thank you

Thanks!!!
This was my first proper PR, and it was really fun working with you.


What is the difference between Latte and ViViT?

The architectures of Latte seem share the same idea with ViViT, what is the difference between Latte and ViViT?

The architectures of Latte seem share the same idea with ViViT, what is the difference between Latte and ViViT?

Hi, thanks for your interest.
The main model architecture differences are:

We removed the CLS token. We believe that only using CLS token will lose the vast majority of information for the video generation task.
We have added an additional Variant 1 to the model design of Latte. We find that Variant 1 is better than the additional three variants under the same parameters.

I see. Some details are quite different. Thanks for your explanation!


[HOLD for payment 2023-10-25] [ECard Transactions] Link eReceipt Page to Receipt Preview

Design Doc Section
https://docs.google.com/document/d/1iELOXPOAcnuIPZJ3XbO1S8_3qHaM31Ptf8dR4LRZMyE/edit#bookmark=kix.epr31aowxop2
Overview
Goal:
Once we have the new eReceipt page component, let’s link to it from the existing Transaction Thread.
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~019e287df581057f74
Upwork Job ID:<PHONE_NUMBER>554031616
Last Price Increase: 2023-10-04

This seems a little different from the typical New Feature flow in the BZ SO, how would you like me to handle this ideally @joekaufmanexpensify / @grgia ? 😎

Yeah, I think it's fine if you un-assign. IDT this really needs anyone from BZ for now. Once C+ reviews the PR, I think we create a separate payment issue then. And if not, we can assign someone from BZ then to handle payment.

@rushatgabhane is the C+ here.

@MitchExpensify or @joekaufmanexpensify could you verify that @rushatgabhane received $500 C+ review payment for https://github.com/Expensify/App/pull/29474

@rushatgabhane payment issue created https://www.upwork.com/jobs/~01d5f78c44d5c389cc

bump on payment status @rushatgabhane @MitchExpensify

Payment summary:
$500 - @rushatgabhane C+
Did you receive this yet on NewDot, @rushatgabhane ?

request here https://staging.new.expensify.com/r/5785180555158358
we can close the issue

$500 payment approved for @rushatgabhane based on comment above.


IBM-Swift/Kitura-StencilTemplateEngine#16 added overloaded render() and setRootPaths()

…nd setRootPaths()
render(filePath: context: options: templateName:)
setRootPaths(rootPaths:)

Description

Motivation and Context

How Has This Been Tested?

Checklist:

[ ] I have submitted a CLA form
[ ] If applicable, I have updated the documentation accordingly.
[ ] If applicable, I have added tests to cover my changes.

Can you somehow test this with one of the existing Template engines?

@shmuelk I tested it on Kitura-Sample, GRMustache using SPM Editable Packages. You just replace one package with your version, and SPM uses your version


SeparationRayShape3D collides when enveloped by convex shapes

Hey,
so I activated Jolt physics in Godot and it seems, that the separation ray shape is a bit too sensitive.
I set up a top down scenario, so movement is locked to xz movement at the moment. When approaching a wall the separion ray shape "separates" the character even though the wall is way higher than the ray. However, when switching back to the Godot physics engine, everything works normally, so I guess it's some kind of bug or setting error, am I right?
Here you can see, how the separation ray setup works with standard physics:
https://github.com/godot-jolt/godot-jolt/assets/157819867/c3975db7-a1df-4604-b4f9-e823eb13d908
Here with Jolt:
https://github.com/godot-jolt/godot-jolt/assets/157819867/3adfd4ab-76d5-4f4c-9b1d-b4916a704a01
And this is the setup:

I added a cylinder mesh to the separation ray in order to visualize the length and position of the ray.
Hope this helps. I'm happy to share more information and test cases if needed.

Interesting. I'm guessing this is going to be this decision coming back to bite me:
https://github.com/godot-jolt/godot-jolt/blob/cafbe8f93b800d0f4c929c334fb0c628ea0f6899/src/shapes/jolt_custom_ray_shape.cpp#L73-L77
It should be a simple fix if that's the case.
I'll see if I can reproduce it on my end.

Hey, first of all thanks for all your replies and for investigating the problem so quickly. Really appreciate your work!
Yeah, totally makes sense. Sounds reasonable to treat the ray shape as convex shape. I tried it on my end with your first suggested solution (so positioning the ray shape directly at the capsule edge) and it seems to work. So thanks for bringing that up. There's no real reason for the offset except to detect the actual step before hitting it, which (at least in theory) gave a nicer transition to the actual step handling. But it seems, that without the offset it works quite nicely.
Maybe it might be a good idea to place a comment somewhere, that explains, why the ray shape was implemented the way you've done it? Just to prevent further confustion in the future.

Maybe it might be a good idea to place a comment somewhere, that explains, why the ray shape was implemented the way you've done it? Just to prevent further confustion in the future.

I must admit I never expected anyone to actually run up against this discrepancy, which is why I didn't document it, but I guess given enough people using the extension eventually every stone will be turned over by somebody.
There are a couple of other very minor/niche/specific differences between Jolt and Godot that I chose not to document in order to not make the list of differences too overwhelming, but yeah, maybe this one warrants being added to it after all.
I'll stew on it for a little bit. It might be better to just align with how Godot Physics does it, just for the sake of not having to document the discrepancy.

I went ahead and made the change without any setting. It'll probably be fine. I can't imagine anyone is actually depending on the current behavior anyway.
It might be a little while before I cut a release with this change in it though.

"A little while" turned into 5 months somehow, but the fix for this can now be found in the latest pre-release, 0.13.0-beta1, which you'll find here.
Note that this version targets the just recently released Godot 4.3-beta1, so won't work with Godot 4.2. Also note that this version is not available through the Godot Asset Library, so you'll need to download it through GitHub.
A proper release will be out some time around Godot 4.3-stable.

I figured I'd give a minor update here, as 0.13.0-stable has just been released, so if you've been relying on 0.13.0-beta1 for this functionality until now I would highly recommend updating, as this most recent release includes some important bug fixes.
Note that the release on the Godot Asset Library is still pending approval as of typing this, so you might need to download it from the link above.


Cannot write command tests without shell (sh)

Describe the bug

Goss command test will execute the command wrapped with a sh -c "<the command>".
This assumes sh executable exists.
When testing distroless containers with a single binary, goss cannot execute the binary.
Dockerfile
FROM scratch
ADD figlet /
ENTRYPOINT ["figlet"]

goss.yaml
Command: figlet: exit-status:
Error
    exec: "sh": executable file not found in $PATH

How To Reproduce

Create a single statically compiled binary, or a binary with the needed libraries.
For me it is figlet.
command:
  figlet:
    exec: figlet test

Expected Behavior

Goss to directly execute (without a shell in the middle) figlet (arg 0) with single argument test (arg 1).
I think that goss giving a shell, should be configurable - and optional.
Perhaps a new attribute shell that defaults to sh, and if explicitly set to nothing shell:, then goss will exec directly the command, rather than running it in a shell.
Actual Behavior

Goss attempts to execute sh, which doesn't exist, so fails.
Environment:

Version of goss: goss version v0.4.4
OS/Distribution version (if applicable): Alpine Linux

Not sure this is a bug to be honest.
The documentation should be clarified that Goss uses sh -c under the hood so things like pipes and redirects work properly. (E.g. grep xx < somefile | sort)
Perhaps there's a feature request in here to allow exec to workout without shell no-shell: true or something like that for situations like this.

I'm going to raise a PR because this is blocking what I am trying to use goss for.
Would you be open to that, and if so, which would you prefer:
1. Specify Shell
Allows specifying an alternative shell or specifying no shell.
Use Case 1: Straight Exec
command:
  figlet:
    exec: figlet test
    # no shell (straight exec)
    shell: ""

Use Case 2: Zsh
command:
  zshism:
    # Use zsh glob qualifier to show files modified in the last day
    exec: "echo *(.m-1)"
    shell: zsh

2. Can opt out of shell
In this case the command is either (by default) executed with sh -c , or can optionally be directly executed.
Use Case 1:
Use Case 1: Straight Exec
command:
  figlet:
    exec: figlet test
    # no shell (straight exec)
    # naming of this could be `no-shell`?
    shell: false

Use Case 2: Zsh
You can't override which shell is used, but you can just directly execute the shell.
command:
  zshism:
    shell: false
    # Use zsh glob qualifier to show files modified in the last day
    exec: |-
      zsh -c "echo *(.m-1)"

3. Only exec, if user wants a shell, they need to specify it
Use Case 1:
Use Case 1: Straight Exec
command:
  figlet:
    exec: figlet test

Use Case 2: Zsh
User wants to use a shell, so they specify it as part of the command to be executed.
command:
  zshism:
    # Use zsh glob qualifier to show files modified in the last day
    exec: |-
      zsh -c "echo *(.m-1)"

More thoughts
I like the simplicity of the third solution, but it is a breaking change.
The second solution is relatively simple and backwards compatible. Simple true/false for shell. If a user wants to use bash, or zsh etc, they can just write the full command like in the example I gave. This is simpler solution than first one because maybe some shells won't use -c for the command, and users might complain that some esoteric shell doesn't work as expected.
What do you think?

I agree this is more a feature request. If it's possible to change the label?

I prefer option 2 for all the reasons you outlined above.
Also, since Goss has been around for ~8 years without this issue, opting out seems sensible, likely needed only by a few users.


Replace header and footer with ttui components

Summary

Replace header and footer with ttui components
Why do we need this?

Style improvements
What is already there? What do you see now?

Fully functioning header and footer
What is missing? What do you want to see?

Include ttui components after our internal discussions and feature request to include search in the ttui header component
Can you do this yourself and submit a Pull Request?

yes, but @pierrephz can handle it

@KrishnaIyer do you have thoughts here? if you put yourself in the user shoes?

The Things Stack
Device
Gateways
Intergrations

I'm not sure this would work. Devices/Gateways (maybe integrations) are not logical sub-sections of The Things Stack. Those sections include generic content about these which are not really part of TTS.
Taking a few steps back, do we need to make the docs header the same as tti.com?
My experience is that they can be different since they are meant for different purposes.
Take Stripe for example https://stripe.com/en-gb-nl and https://stripe.com/docs.


Add facet tags to finders

Authored by @oscarwyatt @karlbaker02 and @vanitabarrett
Trello: https://trello.com/c/JAYPFEEn/140-update-styling-to-the-facet-tags
Desktop

Mobile

https://finder-frontend-pr-739.herokuapp.com/news-and-communications

This actually seems somewhat off the original design - have I missed anything? Currently the Brexit filter is not alongside other topics, people and organisation filters are displayed separately and it seems we're missing a bunch of prepositions?
Original Design:

Actual:

This is ready to merge, but I'm going to hold off for now until after Xmas / more devs will be in.


Simulate R2/L2 button press

I have a quick question: How to simulate a R2/L2 press? Since R2/L2 is of type byte, I cannot use the same method call I would do with other buttons.

because R2 can get value from 0 to 255


Few Shot Chat Prompt

Proposal for a few shot chat message example selector

@willsmithorg let me know the change is okay -- expanded the doc-string in attempt to explain the high level structure and the purpose and added a simple example that uses static examples. I can update the documentation for chat messages in the notebooks.

@eyurtsev Think you tagged the wrong @hinthornw


Create python.js file and add all Python docs in the README.md to the file

Follow this structure to know where you will create your file:

awesome-oss-docs/awesome-oss-docs/src/constants/docs/orderByAlphabet

Add all Python docs to the file you created,. Each docs are added in this format:

{
title: 'Airflow Docs',
description:
'A platform to programmatically author, schedule, and monitor workflows.',
projectUrl: 'https://airflow.incubator.apache.org/docs/',
githubUrl: 'https://github.com/apache/airflow/tree/main/docs',
owner: 'Community Owned Documentation',
},
For better insight of how the file should look like check out the Apache File

After adding all docs, Go to the index.js file in the docs folder, create an import of the file you added and put the name in the export default array

pls assign me

pls assign this to me its getting late

@Elijah699 pls assign me to this

@nobledeveloper01 you can work on this.

Solved #213


[WIP] Branch for 0.2.0 release

I create this branch to merge all othr ones before publish a new version on npm.
The first commit of this branch is just to trigger the PR

Still now, we have:

Add new event-loop example to test event loop stuff
Fixed setTimeout & setInterval (still not optimal)
Other improvements to event loop code
Implemented binding for new libui text drawing API
Simplified binding.gyp file using dynamic inclusion of sources
Add -fvisibility flag in binding.gyp to allow using yoga & libui-node together
Add linter check to C++ files.

Am I forgotting somethings?

check-clang-format also complains about the style of changed javascript files on committing. 😕

check-clang-format also complains about the style of changed javascript files on committing. 😕
Not sure how to limit that to only C++.

I saw that too yesterday. I thought we can instead get rid of XO and use clang-format to format & check JS files too.

Fine! I wouldn't care too much about the single rules, as far as they are kept consistent throughout the repo...
So, unless you care instead... go ahead and set it up.

Thank you works like a charm! And I actually appreciate the new format you picked above: I found the new indentation make the code more readable.

Uhm. It seems I merged binding into master by mistake...


Configure build on travis-ci

It's for my branch https://travis-ci.org/chrhansen/tensorflow.rb, will enable for this repository once the build works. I think that a couple fresh eyes on the .travis.yml file and the build errors https://travis-ci.org/chrhansen/tensorflow.rb we should be able to get this working quickly.
Good sources of info:

https://github.com/tensorflow/tensorflow/blob/master/tensorflow/tools/ci_build/Dockerfile.cpu
https://github.com/tensorflow/tensorflow/tree/master/tensorflow/tools/ci_build
node-tensorflow got Travis-CI working, .travis.yml, some build tools
saiprashanths/dl-docker

@sebastiandeutsch, here's my current status and a couple useful links! Happy to hear any suggestions or questions.

@chrhansen I'm playing with this a bit. I tried making a couple updates to your travis-cy-python branch (changes at my branch).
These helped get a few steps further, but my latest build on that branch still ends up failing at the same step as your last build on the travis-ci branch, where it can't find numpy.
I plan to keep digging at some point today or tomorrow but wanted to share intermediate progress in case it's helpful.

Also, have you thought about using Docker for Travis? If we added a Dockerfile to this repo, we could potentially:

have the Docker "quickstart" dev environment version-controlled along with code (which it currently is not)
share setup logic between the dev environment and the CI environment

@geoffreylitt I think the issue is that the Python that TensorFlow is using as default is whatever is returned by which python (https://github.com/tensorflow/tensorflow/blob/master/configure#L25) and on Travis-CI (Linux Trusty environment) that is in the /opt/... dir, however apt-get and pip are installing somewhere else. In the first ./configure-question I tried to specify /usr/lib/python2.7 which is where numpy gets installed, but I don't think it's quite correct.
At this point I think the issue is to figure out how use and install in the proper Python dir. Also, note that Travis have certain ways that default Python should be used https://docs.travis-ci.com/user/languages/python#Default-Python-Version. Eg perhaps numpy et al. installs need to go in the packages-block for it to work...
I made a new travis-ci-python branch in my fork so I wouldn't pollute this PR till I made some progress.
As you can see in my links above, I did consider using a Docker image, but I decided to wait for now as that could be another long road and I wanted the test environment to be build the same (or similar) way we describe in the README.md.

@chrhansen I agree that your diagnosis of the problem seems probable. This link looks like it could be helpful -- I think setting up virtualenv or using conda instead of pip might be helpful. Unfortunately I'm spoiled by rubygems :grin: and am not familiar enough with python dependency management to have good instincts for what is best.
re: Docker, it totally makes sense to try for the simplest solution first, but I'm also wondering if Docker might end up being easier to get working. Because there's already a working Docker image, if @nethsix could share the Dockerfile backing that image, I imagine it might be as simple as pointing Travis to the Dockerfile. Of course it might not be that simple, but if this route is getting painful it would be worth a try. And the payoff at the end of shared dev/test environment would be nice.

@geoffreylitt That is a good suggestion. However, when I created the Docker image, I only partially used the Dockerfile. The reason:

Docker was having problem compiling bazel, which is pretty similar to what @chrhansen encountered with TravisCI. Thus I partially created the Docker image up to the step of bazel compilation and from there access the Docker image through shell and run commands within it and then did Docker commits to complete the Docker image

Having gone through this exercise, maybe I can now create the image entirely from Dockerfile. Please give me a day to do it. I'll report back to you guys tomorrow.

@geoffreylitt @chrhansen

I have created a the Dockerfile in tools/docker/ (the directory structure is based of tensorflow/tensorflow repo) https://github.com/Arafatk/tensorflow.rb/pull/33
I used CircleCI to build a docker image using the Docker file and ran specs inside the Docker container https://github.com/Arafatk/tensorflow.rb/pull/35

CircleCI successfully built everything, and ran the specs, but it takes 45 mins.
Currently the circle.yml config is set to only run when someone pushes to a branch with 'circle_ci' in the branch name. I left it at that because it currently takes ~45 mins because it is building everything from scratch. I am not sure if Circle CI is smart enough, to re-build only changes. Maybe there is a way to set it up. I will look into it. Alternatively, it is easy to pre-built a Docker image (with all prereqs and Tensorflow shared library) and then during CI, we just compile the ruby-tensorflow code in the pre-build Docker image.

@nethsix that's great! I've added a comment over in https://github.com/Arafatk/tensorflow.rb/pull/35#issuecomment-235699079.

Closing this as @nethsix got a build to work on Circle-CI #41 instead.


USHIFT-626: Install sos utility with MicroShift and document its usage

Ensure the sos presence on MicroShift hosts.
Also created USHIFT-628 task to follow-up on embedding MicroShift-specific information in the reports generated by the sos utility.
Closes USHIFT-626

/assign @fzdarsky

I would propose the following strategy:

Having this as a hard requirement for TechPreview/Limited availability, specially since most of the issues could happen at the initial phases of the project.
Later on, decide based on customer feedback if the dependency needs to be removed. It seems to be less than 5MB.

Long term, we should also work with the RHEL team to include sos utility in the base image. Then, the need for the explicit dependency should become obsolete.

We've agreed to add a strong dep for now, which we'll remove when sos has been added to the base OS.
/lgtm


how to show divider between preference when merged to androidX-1.0.0/support-28

before support-28/androidX-1.0.0, there has divider between preference;
but now, only can show divider between PreferenceCategory;
i want to show divider between preference, and the divider line not match-parent,
it has a margin with the icon, like the below effect
------------------------
😇 title
   ---------------------
😇 title
   ---------------------
😇 title
   ---------------------
😇 title
------------------------

after read the source code, i find the way to add divider:
<Preference
            android:enabled="true"
            ....
            app:allowDividerAbove="true"
            app:allowDividerBelow="true"
           />

<Preference
            android:enabled="true"
            ....
            app:allowDividerAbove="true"
            app:allowDividerBelow="true"
           />

attention：both items should allowDivider if you want to show a divider between them.
and the first(last) preference item in a PreferenceCategory, you just need to set
app:allowDividerAbove="true/false" (app:allowDividerBelow="true/false") to control
but i am still do not know how to set a marginStart of a divider!


Autocomplete and syntax highlight missing for recursive components

Hi and first of all thanks for this awesome extension. I'm just in a middle of a migration from Vetur.
I'm still using Vue 2 with @nuxt/composition-api, I installed @vue/runtime-core and everything is working great so far.
I noticed Volar can't recognize components calling themselves.

Tell me if you need a minimal reproduction

Support in 0.25.7, thanks for the report!

I installed @vue/runtime-core and everything is working great so far.

Please note we need @vue/runtime-dom for vue 2 but not @vue/runtime-core.

I installed @vue/runtime-core and everything is working great so far.

Please note we need @vue/runtime-dom for vue 2 but not @vue/runtime-core.

Ah it's was just a typo I have @vue/runtime-dom installed aha thanks!

@johnsoncodehk Type checking works great on 0.25.7!
But the component is still treated as invalid in Editor Tokens

Recursive components semantic token works for me in this template: https://github.com/johnsoncodehk/volar-starter

Please provide a repro case if it's not works in your project.

Can't reproduce on the starter but I think it comes from me.
I have a nasty thing for declaring components in the FormItem (don't juge)
import * as Components from './forms/components'

export default defineComponent({
    components: { ...(Components as any) },
})

When I comment it out, the Editor scopes works well
Fixed it by doing
components: { ...(Components as Record<string, any>) },


feat: remember volume value between sessions

fixes: https://github.com/talwat/lowfi/issues/11
Was looking for this feature too, so tried implementing it myself with my lack of rust skills.
I hate rust compiler build times but error message are really good.

I think this serves as a temporary solution, but I plan to do this in a way that's a bit more integrated with current systems.
So don't worry about it :)
Actually, the reason this wasn't implemented yet is that I've been busy on #14, so it's likely that the rest of the project will slow down slightly until that's merged.

I've implemented this with https://github.com/talwat/lowfi/commit/6b157dd457829d34524efd34043cc10e416bca17, and you can see that I chose to use a struct which will also make supporting a config file easier down the line.

Would. be nice to have this documented :)


[Event Request]  Page 2900 "Demand Forecast Variant Matrix"

Hello,
Page 9245 "Demand Forecast Matrix"  is now set to be replaced by page 2900 "Demand Forecast Variant Matrix", but a lot of the events that was on demand forecast matrix has not been created in demand forecast variant matrix. Can you guys add the following events from demand forecast matrix to demand forecast variant matrix?
Events in procedure EnterBaseQty:
OnBeforeEnterBaseQty(Rec, ColumnID, IsHandled, MatrixRecords, QtyType);
OnEnterBaseQtyOnBeforeValidateProdForecastQty(Rec, ColumnID, MatrixRecords);
OnEnterBaseQtyOnAfterValidateProdForecastQty(Rec, ColumnID, MatrixRecords, QtyType);
Events in procedure ProdForecastQtyBase_OnValidate:
OnProdForecastQtyBase_OnValidateOnAfterCalcShouldConfirmMovingForecasts(ProdForecastEntry, ColumnID, MatrixRecords, ShouldConfirmMovingForecasts);

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 41118.


bug: useScaffoldEventHistory duplicates events data

Is there an existing issue for this?

[X] I have looked through the existing issues

Which method was used to setup Scaffold-ETH 2 ?
git clone
Current Behavior
Part of the data could be duplicated when using useScaffoldEventHistory
Reason:
we're using blockNumber from useBlockNumber inside getNextPageParam of useScaffoldEventHistory. But blockNumber is not cached and refreshes if you use new instance useScaffoldEventHistory, for example from another page.
So for example you're setting events from first page and use useScaffoldEventHistory from that page pageParams could be for example 0n, 2n, 3n and everything works as expected. But if you switch to another page with useScaffoldEventHistory for the same events, your page pageParams will be something like 0n, 3n, 3n
Expected Behavior
Events data is not duplicated
Steps To Reproduce

change app/page.tsx code to

Details

"use client";

import { useState } from "react";
import Link from "next/link";
import type { NextPage } from "next";
import { useAccount } from "wagmi";
import { BugAntIcon, MagnifyingGlassIcon } from "@heroicons/react/24/outline";
import { Address } from "~~/components/scaffold-eth";
import { useScaffoldEventHistory, useScaffoldWriteContract } from "~~/hooks/scaffold-eth";

const Home: NextPage = () => {
  const { address: connectedAddress } = useAccount();

  const [newGreeting, setNewGreeting] = useState("");

  const { writeContractAsync: writeSetGreeting } = useScaffoldWriteContract("YourContract");

  const handleSetGreeting = async () => {
    try {
      await writeSetGreeting({
        functionName: "setGreeting",
        args: [newGreeting],
        value: 0n,
      });
      setNewGreeting(""); // Clear input after successful transaction
    } catch (error) {
      console.error("Error setting greeting:", error);
    }
  };

  const { data: greetingChangeEvents } = useScaffoldEventHistory({
    contractName: "YourContract",
    eventName: "GreetingChange",
    fromBlock: 0n,
    watch: true,
  });

  console.log(greetingChangeEvents);

  return (
    <>

            Welcome to
            Scaffold-ETH 2

Connected Address:

            <Address address={connectedAddress} />

            <input
              type="text"
              placeholder="Enter new greeting"
              className="input input-bordered w-full max-w-xs mx-auto"
              onChange={e => setNewGreeting(e.target.value)}
              value={newGreeting}
            />
            <button
              className="btn btn-primary w-full max-w-xs mx-auto"
              onClick={handleSetGreeting}
              disabled={!newGreeting}
            >
              Set Greeting
            </button>

          {greetingChangeEvents && greetingChangeEvents.length > 0 ? (

                    Greeting Setter
                    New Greeting
                    Premium
                    Value

                  {greetingChangeEvents.map(event => (

                        <Address address={event.args.greetingSetter} />

                      {event.args.newGreeting}
                      {event.args.premium ? "Yes" : "No"}
                      {(event.args.value || 0n).toString()}

                  ))}

          ) : (

No events found

          )}

            Get started by editing{" "}

              packages/nextjs/app/page.tsx

            Edit your smart contract{" "}

              YourContract.sol
            {" "}
            in{" "}

              packages/hardhat/contracts

              <BugAntIcon className="h-8 w-8 fill-secondary" />

                Tinker with your smart contract using the{" "}
                <Link href="/debug" passHref className="link">
                  Debug Contracts
                </Link>{" "}
                tab.

              <MagnifyingGlassIcon className="h-8 w-8 fill-secondary" />

                Explore your local transactions with the{" "}
                <Link href="/blockexplorer" passHref className="link">
                  Block Explorer
                </Link>{" "}
                tab.

    </>
  );
};

export default Home;

(optional) console.log(data) here, so you can check pageParams

change name 3 times from home page

check the table below button

switch to "debug" page

switch to home page again

check the table. There's duplicated data

Thanks @Pabl0cks for noticing a bug!

Ohhhhh! So when on same page event change + fetching history.

It fetches all the event history from 0n (since there is no event we get 0 events)
User sets greeting we fetching event with from 1n.

But now when we come back from debug page to Home page again.

It fetches all events history from 0n => 1n
Because useBlockNumber runs again it passes pageParams as 1n and we fetch events for 1n again.


Error Summary View Component

Error Summary View Component
Use the existing styles for error summary to create a view component that can be used with forms to surface error messages and provide links to the erroring fields.

does your component need to be used by other products or services?

Yes - it will be needed for any product that has a user facing form with validation

does something similar already exist in the design system

The styles for this component exist already, and this proposed component uses them.

is there a product team that would perform the research, design, development and QA work associate with building this component?

Probably content-platform?

do you have any user stories / scenarios that describe the component?

As a user, I want to see a list of problems with my form submission, so that I can correct my mistakes and submit the form correctly

As a user, I want to be able to get from an error message to the part of the form with the wrong information in quickly so I can rectify my mistake and submit the form correctly

Existing components
Code

provide a link to your component working in a dev or prod environment

component class, template, spec and preview are in this public-website PR: https://github.com/citizensadvice/public-website/pull/1689

Usability

have you tested the component with the intended user group? Please include any links to research documentation in your proposal

TBC but the styles exist already so assume this is covered.  Worth trying to find the research for tho to attach to documentation.

Accessibility

have you ensured your component meets WCAG 2.X AA guidelines? Include test protocols from external audits or notes from internal tests

Styles and react implementation have been audited by the Digital Accessibility Centre as part of the second public-website audit (see report here https://citizensadvice.atlassian.net/wiki/spaces/NP/pages/2526347313/Accessibility+reviews+-+external).  It might be worth explicitly testing a journey with an invalid form submission as something to test in the next audit

have you tested your component with a screen reader?

Yes, using Chrome and VoiceOver on a Mac.  I wonder if hearing 'Enter your name' when that link has focus is confusing and if we should make it explicit that clicking the link will take you to the invalid form field?

have you tested your component by using only the keyboard to interact with it?

Yes, works.

Quality

provide links to any appropriate automated tests you have written

unit tests

rspec unit tests in PR link above

behavioural tests
visual regression tests

these might be easier to do when part of the design system?

Rough migration dev outline:

[x] create new component in design-system/engine project
[x] move ruby code and HAML into files created above - it no longer needs to be scoped to PublicAdvice
[x] move rspec for component into design-system/engine project
[x] update storybook docs so that:

[x] they use the HTML generated by the rake generate_examples task in /demo
[x] they provide instructions on how to use the view component and detail the interface (see other components for example)

[x] create a visual regression test for the error summary component (if there isn't one already)
do an alpha release
do a full release

Link to an example: https://web.develop.content.citizensadvice.org.uk/components/previews/citizens_advice_components/error_summary/example/


Is it a mistake? Bootstrap.css line 4492 in class .navbar-right

margin-left should be 0 instead of -15, I believe. Reason for that: page with it becomes scrollable by width (x) and makes different widgets overflows screen.

The negative margins offset the margins applied by .container Because it does not go within a row.

@Windstep Line number references aren't clear unless you say what release version or git commit of Bootstrap that you're using.

@cvrebert, I'm using version 3.3.6 of it. From official site.

@Windstep You're referring to https://github.com/twbs/bootstrap/blob/181f76f115a35c03c5e7629af670fea144ee25dd/dist/css/bootstrap.css#L4492, correct? /cc @cvrebert

Duplicate of #18355.


SVG renderer interprets px units on width in SVG root as pt

If a width is defined on the <svg> element in the SVG file, there's currently no difference between a value with the units px (e.g., 100px) and a value with the units pt (e.g., 100pt). Instead, they're both interpreted as having the units pt.
While it's correct for an SVG to treat a unitless value as having the default unit for the rendering environment (in this case, pt), it should recognize and handle the px units. In this converter, that means multiplying the value by 0.75. This matches the behavior if the width is set on the image macro using the pdfwidth attribute or as a fallback width in the theme using the image-width key.
This computation is handled inside of prawn-svg. Therefore, the solution is to patch prawn-svg to honor the px units.

While working on this fix, I noticed that px values in the SVG data were not being converted to pt either. This solution should handle both.

I noticed that px values in the SVG data were not being converted to pt either. This solution should handle both.

I'm not sure about this one. It seems to cause the SVG to appear different than when rendered in another environment, such as the browser. My hunch is that px values in the SVG data (such as a stroke width) are relative to the viewport rather than the medium/canvas and thus should continue to be interpreted as such.


tf.reduce_sum with multiple negative axes and tf.RaggedTensor bugged

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): yes
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): ubuntu 16.04
TensorFlow installed from (source or binary): pip
TensorFlow version (use command below):
Python version: ('v1.13.1-0-g6612da8951', '1.13.1')
GPU model and memory: quadro k620 2gb

Current behaviour
Multiple axes passed to tf.reduce_sum with first argument being a ragged tensor results in incorrect behaviour.
import tensorflow as tf

x_values = tf.random.normal(shape=(100, 5, 6))
x_row_lengths = tf.constant([20, 30, 50], dtype=tf.int64)
x_ragged = tf.RaggedTensor.from_row_lengths(x_values, x_row_lengths)
print(x_ragged.shape)
# [3, ?, 5, 6]

# wrong shape
print(tf.reduce_sum(x_ragged, axis=(-2, -3)).shape)
# [50, 6]

# positive axes work
print(tf.reduce_sum(x_ragged, axis=(1, 2)).shape)
# [3, 6]

# separate reductions work
print(tf.reduce_sum(tf.reduce_sum(x_ragged, axis=-3), axis=-2).shape)
# [3, 6]

Expected behaviour
Same result as corresponding positive indices/separate reductions.

Created a PR #27699 for the fix.


Unlock issue #10121 in the interest of transparency and honesty

Issue #10121 is about allowing for floating and detachable windows for better multi monitor support.
It's the most upvoted feature request, but in "Dec 3, 2018" microsoft has locked and limited the conversation, preventing further upvotes.
Microsoft has also been aggressively duping and ignoring any related issues, with the excuse that this would be addressed since it was in the 2019 roadmap.
Since "Dec 3, 2018", no progress has been reported and people cannot upvote this feature anymore.
The issue was then added to the 2020 roadmap as well, with no progress and no changes from the same 2019 roadmap.
It seems that this issue is not going to be addressed any time soon due to "architectural issues".
True honesty and transparency calls for the conversation  to be unlocked and for the community to be allowed to vote for it and comment in the issue.

@pedrogarcia thanks for bringing this forward. It is correct that we had this on our 2019 roadmap and moved it to our 2020 roadmap without any visible progress.  As you know we locked the issue in response to https://github.com/microsoft/vscode/issues/10121#issuecomment-443602572 and the concerns brought forward there would still apply if we were to unlock it. We'll discuss.

I know this is a complex issue and I am not aware of all the detail other than what I read on here and the https://github.com/microsoft/vscode/issues/10121#issuecomment-345770248 .
I see that the electron architecture is one blocker.  The Microsoft teams application has just today popped up saying that my meetings can now pop out, (and they do, its great).  If they have managed that with electron then it seems that they solved that particular issue.  Maybe a chat with the teams team to see how they implemented it would help?

Can I just say, it's frustrating that issues get locked and you can't vote for them. Just because something doesn't get X amount of votes in 60 days doesn't mean in the future, people won't stumble upon the issue and want to upvote. Eventually an issue may get more than X upvotes!
Ironically, there is a locked (!) issue for this that I can't upvote! https://github.com/microsoft/vscode/issues/70770

If you want to help out testing our current state of floating editor windows, please see https://github.com/microsoft/vscode/issues/10121#issuecomment-1790316086


Ability to receive a notification when a follower faves a shared post

Describe the Feature
It'd be nice to know my friends liked a post I shared. There's no non-technical reason Bluesky shouldn't have this, since likes are public; when I find something and share it, but can't see whether anyone liked it, it feels like I'm interacting with a void.
Attachments
No response
Describe Alternatives
No response
Additional Context
No response

Yeah this is quite a big difference with Twitter (especially for those of us who are heavy reposters), because on Bluesky you get no notification at all when someone likes or reposts a post that you reposted.
I think adding this to Bluesky would be good, although ideally with an option to turn off these notifications, as I'm guessing some people would not like them.

Yes. I had at first just assumed this worked like Twitter where you got a notification when someone liked your retweet. So I thought everyone hated my retweets for some reason. No, it's just a missing feature. These notifications were helpful feedback.


404 with localeSubpaths

Describe the bug
I get a 404 with localeSubpaths only on production build. In develop everthing works fine.
Occurs in next-i18next version
v6.0.2
Steps to reproduce
I get no error so i have no clue
Expected behaviour
http://localhost:3000/en -> statusCode 200 and not 404
OS (please complete the following information)

Browser: Chrome: 84.0.4147.105

Additional context

next.js v9.5.1

I will try to submit a reproducible repo.

Let me know when you put together a reproducible repo, and I'll have a look.

I have the same issue.
When I first com to http://localhost:3000/en, it loads the english.
However, once I change the language, i got 404 for the common.json need to be used for the new language
e.g. http://localhost:3000/zh-Hant/src/utils/i18n/locales/zh-Hant/common.json 404 (Not Found)

Got the same problem. Reproducible Repo: https://github.com/michelalbers/next-i18next-404

@michelalbers Couple issues with that repo regarding your TS setup. Regardless, I was able to run the app via yarn build && yarn start, and can not see any 404 errors. Can you provide specific instructions as to how I can reproduce the error?

Try yarn dev :)

Running with yarn dev does not produce any 404s for me either.

I have the feeling it doesn’t work when u didn’t run next build initially. Will confirm when I’m back at the office.

I did a fresh clone of the repo, and then just did yarn && yarn dev. I had to install TS deps because they were not present. I did not run yarn build.

Figured it out ... I had next.js 9.4.4 in my cache which does not support rewrites. Next.js 9.5 is required which came out 11 days ago. Could we add that to the readme?

How did you fix it exactly? How can i clean the cache?

How did you fix it exactly? How can i clean the cache?

In my case it was as simple as yarn add<EMAIL_ADDRESS>

Well, it's in the package.json, and you would have been receiving warnings from npm or yarn. Not sure it's worth adding to docs.

I still get a 404

Well, it's in the package.json, and you would have been receiving warnings from npm or yarn. Not sure it's worth adding to docs.

Yes you get a peer dependecy warning ... but in every bigger project you might have quite a few from them. Anyway - my issue is resolved. Thank you very much @isaachinman

@gpessa and @Uniiq you will need to provide a reproducible repository if you would like help debugging.

@isaachinman i sry for the late response. The first problem was i forgot to remove await nextI18next.initPromise; from custom next server file (i upgraded from<EMAIL_ADDRESS>).The second problem is i use cacheable-response to cache the SSR response and somehow next-i18next interfere with my caching function and redirect me to 404 error page (caching is only enabled on production).
In short there is a bug on my side. :)

@isaachinman i sry for the late response. The first problem was i forgot to remove await nextI18next.initPromise; from custom next server file (i upgraded from<EMAIL_ADDRESS>).The second problem is i use cacheable-response to cache the SSR response and somehow next-i18next interfere with my caching function and redirect me to 404 error page (caching is only enabled on production).

@Uniiq sorry! what is the solution for this bug? This only happens in production mode. What should I do? Some help please?


MacOS installer background color scheme

The installer background arrow is using the old color scheme, let's update it.

nice catch


Miscellaneous updates and rearrangements

This commit sets up publishing configuration using sbt-release, updates some versions, fixes a few little things that Scalastyle doesn't like, and moves io.github.finagle.Serial to io.github.finagle.serial.Serial (following standard Scala / Java conventions).

Nice!
@travisbrown Are we plannig to publish it in the nearest future?

@vkostyukov Yep, I was going to go ahead and publish a 0.1.0 as soon as this gets merged.


Update UEFI and ACPI sections.

Fix: interrrupt -> interrupt

Update the UEFI requirements to refer to the UEFI spec.

More clarity around ACPI requirement.

Signed-off-by: Sunil V L<EMAIL_ADDRESS>

I merged this patch. I will fix this to add “support ACPI”.
Regards
Kumar
From: Atish Patra @.>
Sent: Thursday, September 30, 2021 11:06 PM
To: riscv/riscv-platform-specs @.>
Cc: Subscribed @.***>
Subject: Re: [riscv/riscv-platform-specs] Update UEFI and ACPI sections.
(#59)
@.**** commented on this pull request.
Otherwise looks good.
In riscv-platform-spec.adoc
https://github.com/riscv/riscv-platform-specs/pull/59#discussion_r719965142
:

|===

===== Hardware Discovery Mechanisms
====== ACPI
ACPI is the required mechanism for the hardware discovery and configuration.
Server platforms are required to adhere to the RISC-V ACPI Platform
Requirements
-Specification<<spec_riscv_acpi>>.
+Specification<<spec_riscv_acpi>>. Platform firmware must implement ACPI and the
I & Kumar had a discussion about "implement".
https://docs.google.com/document/d/1PfvLw3f-tPTRjl6Rw614O_RaA62oKeHlTfu7g6KZ4Vs/edit?disco=AAAAPk-SXW0
If you agree with the reasoning, lets change it to "Platform firmware must
support ACPI"
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/riscv/riscv-platform-specs/pull/59#pullrequestreview-768589911,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABLRGVTY6QLB5T7PGBO56V3UEVFT3ANCNFSM5FD4VZPQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.


Sorted map please!

A map that inserts new items in the correct position, providing a custom sort function on the keys.
Would be nice. Otherwise, I can do fine with keeping a key array for the sorting, and a map myself.

Just figured out I need this.
+1 from me.

+1 indeed :shipit:

:+1:

Thank you for reporting this issue and appreciate your patience. We've  notified the core team for an update on this issue. We're looking for a response within the next 30 days or the issue may be closed.

Sorted map would be great! :+1:

Also, an OrderedMap can be implemented atop of a SortedMap without amortization.

That would be so useful!

Agreed. If I understand correctly .sort() creates a copy every time (even when called in withMutations). So if I want to insert sorted, I have to duplicate the OrderedMap twice. eg:
const map = OrdererMap({a: 'a', c: 'c'});
const sorted = map.set('b', 'b').sort(); // Creates two copies.

For prior art on this issue, Clojure(script)'s sorted-map implementation uses a persistent red-black tree:

Java: https://github.com/clojure/clojure/blob/010864f8ed828f8d261807b7345f1a539c5b20df/src/jvm/clojure/lang/PersistentTreeMap.java
Clojure (used in Clojurescript): https://github.com/clojure/clojurescript/blob/609400910e983849e5b7932a4b7d0b0dda465df6/src/main/cljs/cljs/core.cljs#L7066

There's a JavaScript implementation of the same data structure here that might be useful, too.
Clojure's sorted-set uses the same data structure, slightly wrapped: https://github.com/clojure/clojure/blob/clojure-1.7.0/src/jvm/clojure/lang/PersistentTreeSet.java

RB Tree sounds like the right way to do this.
I'm putting this up for grabs for any adventurous contributors. Best way to learn about Persistent Immutable data structures is to implement them!

Have not used yet but this seems promising: https://github.com/frptools/collectable/tree/master/packages/sorted-map#collectablejs-immutable-sorted-map

@akagomez Want to take a crack at RB Tree implementation?

Tempting @crodriguez1a.
I've done a bunch of work with Red-Black Binary Tree's for CanJS (https://github.com/canjs/can-binarytree), based on a fork of this repo: https://github.com/vadimg/js_bintrees
Leveraging that RB Tree implementation again should make this a less daunting task.
If nobody disagrees with this approach I don't mind taking a crack at it with @crodriguez1a's help.

@akagomez feel free to steal from my toy SortedMap (api not fully implemented) that I started for a side project: https://github.com/rongierlach/immutable-sorted-map/
It is using a regular BST under the hood, am intending to convert it to RB later on.

The long wait is over. Have a look at our immutable-js fork with highly optimized implementation of SortedMap and SortedSet. Current version is using classic B-trees. https://github.com/applitopia/immutable-sorted

Darn, I was looking forward to playing with this myself.
Maybe @crodriguez1a and I can help convert @applitopia's implementation to use RB-trees.

@applitopia - please consider contributing those implementations to this project!

Need this too! Thanks


Support for Home Assistant Scripts continue_on_error

Is your feature request related to a problem? Please describe.
In Home Assistant I have an automation that turns on/off the water heater at a certain time. Sometimes the automation fails from what I guess is a timeout when calling LG API, the problem is that after one step in the automation fails the whole execution stops and nothing else will be run.
Describe the solution you'd like
Support for Home Assistant script syntax continue_on_error.
I tried adding continue_on_error to the step that turns on/off the water heater, so if it fails the automation can continue but the problem stills persists. Looks like the integration is not compatible with continue_on_erorr
It seems that the integration must raise HomeAssistantError for the automation to continue.
Additional context
If it helps the error I'm getting from the smartthinq integration is this one:
custom_components.smartthinq_sensors.wideq.core_exceptions.APIError: 0111 - {'data': ''}
Documentation:
Home Assistant Continue On Error

Can you provide more details about the services you are using in your script.

Yes sure.
I have seem this error pop up using water_heater.set_operation_mode and climate.turn_off services

I'm geting the exact same error on climate.turn_off for my LG S24ET Airconditioner.


Update cli with yarn tarball for binary generation

Description
Updates dependency of CLI package on @celo/dev-cli with new upstream features
Other changes
Describe any minor or "drive-by" changes here.
Tested
An explanation of how the changes were tested or an explanation as to why they don't need to be.
Related issues

Fixes #[issue number here]

Backwards compatibility
Brief explanation of why these changes are/are not backwards compatible.

@yorhodes is this PR still relevant?

@yorhodes is this PR still relevant?

I think it's useful if we want a standalone binary such that users don't need specific node/npm versions/installations

Closing for now, feel free to re-open if still applicable

Closing for now, feel free to re-open if still applicable


Decide what happens when the storage goes back in time

Now that #2241 includes a @migration_version, it may happen that someone runs a more recent Livebook version and then rolls back to an earlier one. We may need to detect and handle such scenarios accordingly, such as use a separate storage. I believe we already discard fields that we don't know but the issue is that we may now write old data on an already migrated database and we cannot reflect that on the @migration_version, unless we have two versions: "maximum migration version" and "last run migration version".

Another thing I wonder if whether we should guard migrations, such that they only run once. The difference is that we would always need to bump the version, otherwise the new migrations won't run during development, but bumping the version should be fine?

Yes, migrating only once is related with going back in time. Today, if you go back in time and add new file-systems, we would be able to migrate it again.
Another option is to move this idea of migration all the way to the file. Once the storage starts, we copy the old storage file to a new versioned .ets file, if one does not exist, and then migrate it. This means that going back in time always works and migrations automatically happen only once.


Codes to store c# codes

If there is Pages, why not "codes" to store codes.
I'm aware the use case for asp.net teams is to use visual studio/source control -> build -> deploy
but there is this people like me who need to keep everything on the fly during runtime.
I know I can use c# script but there is a plan to make view compile updateable which means a good time to propose to make code also updateable.
At this moment if I need to add new code api I have to

use c# script
use c# script then compile, save dll somewhere, reference the dll and use c# script to call it (cumbersome)
download the projects from source control, add the code, build dll and publish...

or just make it possible to call methods stored in razor page from another razor page?

Hi @andrew-vandenbrink , the Pages folder just exists so that MVC knows which pages are Razor Pages, as opposed to MVC view pages. There is no need for a Codes folder because you can place code anywhere in the application.
There are no plans to support runtime compilation of regular C# code at this time. We're moving to a model where more of the compilation is done at build/publish time, and less is done at runtime, in part for performance reasons. (We didn't take away any features, but we changed the default experience for new projects.)


Bug fix on add method and Rouge

Fixes #355
work in progress

corrected a typo of a code block in readme.md

set up an environment to debug the add method. - will delete test_haz.py later after the completion

Hey, @Sumanth077 I have set up a draft PR for us to work together if you are still working on this.  :)

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.

Thanks a lot for looking into this!

had to close this PR due to me having a problem with git. I already opened a new draft PR here


docs: Fix path to CONTRIBUTING.md

Coverage remained the same at 97.445% when pulling 4756ce991447388c9d558343d1458f0820417827 on tyankatsu0105:docs/fix-markdown into af7caa7b2529cb01355ae260ef9376212b3fe741 on lovell:master.

🤔

Thank you!


[Need Help] What does Website blocked mean?

I get this error, whenever I try to query 1337x on a oracle server, I understand something is blocked , whether it's the IP of my server blacklisted on 1337x or My network blocking the 1337x , I'm not clear on this. I tried some work arounds like using Tor to scrape the site. based on whats the issue is I can check on workarounds properly. please help on this
{"error":"website blocked change ip or domain"}

@trixter127 That means that website is blocked by your internet provider or by your hosting provider if you are hosting
You can use https://windscribe.com/ VPN


fix: there are several react-hooks/exhaustive-deps violations

Description:

There are several react-hooks/exhaustive-deps violations in the @asgardeo/react package.
/home/runner/work/web-ui-sdks/web-ui-sdks/packages/react/src/components/SignIn/SignIn.tsx
    114:6  error  React Hook useEffect has a missing dependency: 'authContext'. Either include it or remove the dependency array  react-hooks/exhaustive-deps

  /home/runner/work/web-ui-sdks/web-ui-sdks/packages/react/src/hooks/use-on.ts
    [41](https://github.com/asgardeo/web-ui-sdks/actions/runs/9806802898/job/27079282605?pr=41#step:9:44):6  error  React Hook useEffect has missing dependencies: 'callback', 'contextValue', and 'event'. Either include them or remove the dependency array  react-hooks/exhaustive-deps

  /home/runner/work/web-ui-sdks/web-ui-sdks/packages/react/src/hooks/use-translations.ts
    52:6  error  React Hook useEffect has a missing dependency: 'setIsTextLoading'. Either include it or remove the dependency array  react-hooks/exhaustive-deps
    56:6  error  React Hook useEffect has a missing dependency: 'setIsTextLoading'. Either include it or remove the dependency array  react-hooks/exhaustive-deps

Suggested Labels:

N/A

Suggested Assignees:

N/A

Affected Product Version:

N/A

OS, DB, other environment details and versions:

N/A

Steps to reproduce:

N/A

Related Issues:

N/A

Fixed with https://github.com/asgardeo/web-ui-sdks/pull/48


Add missing stdbool.h include for C compiler

Hi,
It seems that with Clang 14, the compilation of test_01.c does not find the bool type used for the stopping_criterion argument to pool_work_until. What's missing is the include of stdbool.h in case the code is compiled using the C compiler.

Thanks!


Lipid kinase activity (GO:0001727) children

Currently "lipid kinase activity" (GO:0001727) only has two is_a children:
ceramide kinase activity (GO:0001729)
sphinganine kinase activity (GO:0008481)
I was expecting to find several other kinases classified as "lipid kinases", including:
diacylglycerol kinase activity (GO:0004143)
D-erythro-sphingosine kinase activity (GO:0017050)
and the several phosphatidylinositol kinase terms:
phosphatidylinositol kinase activity (GO:0052742)
phosphatidylinositol phosphate kinase activity (GO:0016307)
phosphatidylinositol bisphosphate kinase activity (GO:0052813)
phosphatidylinositol 3-kinase activity (GO:0035004)
(and there may be others...)
All of these other terms are currently direct children of "kinase activity" rather than subtypes of "lipid kinase activity":

Maybe this is just a mistake/oversight, or is it a matter of whether these substrates are classed as lipids or not?
Notably, the children under "lipid binding" (GO:0008289) or "lipid phosphorylation" (GO:0046834) DO appear to classify these molecules as lipids (see image2 and image3), so there's at least some inconsistency here.

@hattrill

This classification would be based on the substrate and will be dependent on ChEBI. This classification should happen automatically once we have adopted the Rhea definitions for these terms. @hdrabkin can you investigate the xref situation? This might be a really good example.

Following up David's previous suggestion to "investigate the xref situation", here are the Rhea xrefs on the terms (or their children) mentioned in my original post in the current GO:
ceramide kinase activity (GO:0001729) -> xref: RHEA:17929
sphinganine kinase activity (GO:0008481) -> xref: RHEA:15465
diacylglycerol kinase activity (GO:0004143) -> xref: RHEA:10272
D-erythro-sphingosine kinase activity (GO:0017050) -> no RHEA
phosphatidylinositol kinase activity (GO:0052742) -> no RHEA (expected - grouping term)
1-phosphatidylinositol-3-kinase activity (GO:0016303) -> xref: RHEA:12709
1-phosphatidylinositol 4-kinase activity (GO:0004430) -> xref: RHEA:19877
1-phosphatidylinositol-5-kinase activity (GO:0052810) -> no RHEA
phosphatidylinositol phosphate kinase activity (GO:0016307) -> no RHEA (expected - grouping term)
1-phosphatidylinositol-3-phosphate 4-kinase activity (GO:0052811) -> no RHEA
1-phosphatidylinositol-3-phosphate 5-kinase activity (GO:0000285) -> xref: RHEA:13609
1-phosphatidylinositol-4-phosphate 3-kinase activity (GO:0035005) -> xref: RHEA:18373
1-phosphatidylinositol-4-phosphate 5-kinase activity (GO:0016308) -> xref: RHEA:14425
1-phosphatidylinositol-5-phosphate 4-kinase activity (GO:0016309) -> xref: RHEA:12280
phosphatidylinositol bisphosphate kinase activity (GO:0052813) -> no RHEA (expected - grouping term)
phosphatidylinositol-3,4-bisphosphate 5-kinase activity (GO:0052812) -> no RHEA
phosphatidylinositol-4,5-bisphosphate 3-kinase activity (GO:0046934) -> xref: RHEA:21292
phosphatidylinositol 3-kinase activity (GO:0035004) -> xref: RHEA:12709 (should this be on the grouping term?)
1-phosphatidylinositol-3-kinase activity (GO:0016303) -> xref: RHEA:12709
1-phosphatidylinositol-4-phosphate 3-kinase activity (GO:0035005) -> xref: RHEA:18373

RHEA has given me mapping  tables and I am going through them;
RHEA DOES have a mapping of  GO:0035004) -> xref: RHEA:12709;  They have cross referrenced EC:<IP_ADDRESS>; ATP + 1-phosphatidyl-1D-myo-inositol <=> ADP + 1-phosphatidyl-1D-myo-inositol 3-phosphate; but there is a problem
The CHEBI ids for the substrates and products  by REHA maps to 1-phosphatidyl-1D-myo-inositol(1−), and 1-phosphatidyl-1D-myo-inositol 3-phosphate(3−) respectively, which are used in the EC def.
The GO def appears out of sync with the other mappings. here (metacyc, etcs)
This is also the only term that is mapped in GO to this EC in protege, but I see that it is also used in one of the children.
I see that the 35004 term has a RHEA but so does the 16303 term.
The RHEA can only be in one place; it needs to be removed from the 35004 term   (which IS the actual reaction AND is an alternate name in EC:<IP_ADDRESS>;
Basically, this grouping term should be stripped of the EC AND RHEA; Note that it is GO that imposes the hierarchy/grouping. THE EC actually  belongs with 35004.

THe only reason it is in the mapping files from RHEA is because it had the EC; and now I notice that GO:0016303 is ALSO mapped to the same RHEA in the mapping files (so I need to get rid of the ec and rhea in the grouping term for sure

This is all a side issue to this ticket; sorry; but Since I see this problem I should fix it while I see it.)

EC and RHEA mappings to GO:0035004; left on GO:0016303

Thanks for sorting out that issue with xrefs on the GO:0035004 and GO:0016303 terms.
Returning to the main issue and David's previous comment: once you've gone through the Rhea mapping tables for these terms, and "adopted the Rhea definitions", then we might be able to rationalise the GO terms coming under lipid kinase activity?

One thing is we need to be careful with assigning mappings to grouping terms!
And we need to put a check into Travis RHEA in more than one GO term!

This might be done?

the check will be put in AFTER we are all sorted out with dual mappings. I have a merges to do not related to this ticket.

Yes, definitely not done and ticket should stay open.  ("lipid kinase activity" (GO:0001727) still only has two is_a children in the GO.)

Excellent, thanks!  Closing.


Groovy Incremental Compilation broken when Classes removed then added to classpath when FQN are used

Expected Behavior
When classes are added/removed Groovy incremental compilation should be considered out-of-date.
Current Behavior
When using FQN in Groovy and the class is not present on compile time classpath, groovy generates dynamic accessors. When the compile time classpath changes, thus re-adding the class, incremental compilation does not recompile to use static class access.
Context
Groovy incremental compilation is broken in weird ways when FQNs are used but the compile time classpath is chaning. This can lead to very confusing/hard to debug bugs & test failures.
Steps to Reproduce
Given a FooUser.groovy that references foo.SomeClass with dependency foo

Compile. FooUser is compiled. The bytecode contains a reference to foo.SomeClass.
Remove foo from the classpath.
Recompile. FooUser is compiled, but now it contains a dynamic reference to a property foo.
Add foo to the classpath in a different order.
Recompile. FooUser is not recompiled.

At runtime, FooUser will not work, even if foo.SomeClass is on the classpath again.
Real world case
An example of this can be demonstrated with this class here:
https://github.com/gradle/gradle/blob/1d3aa1c44dfb966bbec5c2ac2b543c315e08d0f3/subprojects/testing-base/src/test/groovy/org/gradle/api/internal/tasks/testing/processors/JULRedirectorTest.groovy#L53
Adding/removing libs.commonsIo from the classpath in the manner described in the bug can reproduce this issue.
Your Environment
------------------------------------------------------------
Gradle 7.0-20210206125447+0000
------------------------------------------------------------

Build time:   2021-02-06 12:54:47 UTC
Revision:     99b7e817a96482acfc9e1b898dcb2919cd3938e8

Kotlin:       1.4.30
Groovy:       2.5.12
Ant:          Apache Ant(TM) version 1.10.9 compiled on September 27 2020
JVM:          11.0.8 (AdoptOpenJDK 11.0.8+10)
OS:           Mac OS X 10.15.7 x86_64

The nuclear option here would be to fully recompile when any class is added. A cleverer solution would be to enhance ClassDependenciesVisitor to detect this Groovy pattern in the bytecode and add a dependency on the class that wasn't there.

Still a bug, and a very annoying bug at that when you encounter it since there is really no good way of determining the root cause easily.

I found another place affected by this: https://github.com/gradle/gradle/blob/a3a3fcd43bb733b9847c0b600475fce0b21d98ea/subprojects/base-services/src/test/groovy/org/gradle/internal/classloader/FilteringClassLoaderTest.groovy#L169
I was able to trigger it by switching the Groovy version used by Gradle around, it's not 100% reliable but it does happen frequently.


Fix spawning empty bundles

Objective
Alternative to #6424
Fixes #6226
Fixes spawning empty bundles
Solution
Add BundleComponentStatus trait and implement it for AddBundle and a new SpawnBundleStatus type (which always returns an Added status). write_components is now generic on BundleComponentStatus instead of taking AddBundle directly. This means BundleSpawner can now avoid needing AddBundle from the Empty archetype, which means BundleSpawner no longer needs a reference to the original archetype.
In theory this cuts down on the work done in write_components when spawning, but I'm seeing no change in the spawn benchmarks.

bors r+

bors r+


[feature] Update README

Description
With the multiple contributions, README is not up-to-date, need to add the new tutorials.
Do you want to be assigned to work on this?
(mark [x])

[x] YES
[ ] NO

Optional informations
I want to add badges if you agree too @funbeedev .

@sabderemane Great suggestion! Please go ahead and update the README with the tutorials added so far.
You can add the badges too.


Changed the ChaosEngine YAML, added step to override defaults

Files Changes are self-explanatory.

@rahulchheda do we need package-lock file?

@rahulchheda do we need package-lock file?

Yes, for npm package it's needed

@rahulchheda do we need package-lock file?

Yes, for npm package it's needed

Can you check again, As per the updated readme package-lock.json is not required.


Git update

Hi awesome creator and all,
I have this question in my mind. I am editing and modifying theme for my taste. If I see theme is updated to latest version in Github. I wanted to use the updated theme. How to incorporate the new changes to my existing theme without disturbing previous changes. Any solutions for this?
Thanks,
Vijay

You can always update by running git pull in the themes/academic directory, but you will have to merge and probably fix conflicts each time.
The ideal way to do it is by forking the github repository of hugo-academic, and then managing your own fork with its remote (git push/pull origin master), and occasionally syncing with the changes made to the original theme (git pull upstream master). See https://help.github.com/articles/syncing-a-fork/

In addition to the above, you might also find the following helpful:

Overriding Academic's files: https://sourcethemes.com/academic/docs/customization/#override-a-template
Update guide: https://sourcethemes.com/academic/docs/update/
Release notes (including specific steps for updating): https://sourcethemes.com/academic/updates/

Thanks a @gcushen  and @sbibauw


Error: ace is not defined Evaluating http://xxx/public/plugins/vertamedia-clickhouse-datasource/module.js Loading plugins/vertamedia-clickhouse-datasource/module

why exit this error in release 1.9.0,but i run with release 1.9.3,this error was fixed

@yang3808282 thanks for reporting


Floating labels on chart

Hi,
Any edge I wanted to thank you for this great work.
However I have a concern. When I spend my cursor over a graph I the values that appear but I would delete the second column percentages, how can I do?
Can I delete zero values from display ?

Regards,
Loris

sadly for now the only way to remove them is creating your own tool tip, and setting it to the chart.
here is the source code of how I built the current one
https://github.com/beto-rodriguez/Live-Charts/blob/master/WpfView/DefaultTooltip.xaml
https://github.com/beto-rodriguez/Live-Charts/blob/master/WpfView/DefaultTooltip.xaml.cs
you should adapt it to your case, this tool tip tries to work for every series and chart in the library


onChange fn in folder item makes the item dissapears in get

When I use the onChange function in an item in a folder, it will become unavailable in the get function.
This is the example when adding the onChange function to the item in a folder makes the slicerEnabled element unavailable in the get function. Therefore: get('slicerEnabled'); will fail.

When I remove the onChange function, I can retrieve the item with get
When I move the item outside of the folder, I can retrieve it with get

const [{...controls}, set, get]  = useControls( () => ({
         "Boundary Conditions": folder({
           inlet: {
             value: 1.5,
             min: 0.1,
             max: 10,
             parameterized: true
           },
           outlet: { value: true, parameterized: true },
         }, { collapsed: false }, ),
         "view": folder({
           viewModeSelect: {
             label: "View Mode",
             options: {
               "Point Cloud": 'PointCloud',
               "Vector Fields": 'VectorFields',
               "Surface Slices": 'SurfaceSlices',
               "Streamline Slices": 'StreamlineSlices',
             }
           },
           slicerEnabled: {
             label: "Enable Slicer",
             value: false,
             parameterized: true,
             onChange: (v) => setSlicerMode(v)
           },
           slicerVal: {
             label: "Slice Y",
             value: 1.5,
             min: 0.1,
             max: 10,
             parameterized: true,
             onChange: (v) => {
                 const isSlicerEnabled = get('slicerEnabled');
                 if(!isSlicerEnabled) {
                    ... do my stufff
                 }
             }
             render: (get) => get('view.slicerEnabled') == true,
           },

**ERROR:**
`Argument of type '"slicerEnabled"' is not assignable to parameter of type '"inlet" | "outlet" | "inlet_velocity" | "viewModeSelect" | "pointCloudSelect" | "pointCount" | "Message From PS"'.`

Please provide a sandbox

Here you go:
https://codesandbox.io/s/eager-yonath-cbmf1i?file=/src/App.tsx
Happens only in TS environment

Did you manage to replicate the issue? @dbismut

Indeed that's a bug, thanks for reporting it.


NA OFI: Explicitly determining domain for network device with verbs provider

When configuring a Mercury connection using the verbs provider, the API currently requires the specification of a network device and domain.  How does one reliably determine the domain that is associated with a given network device?
The system command "ip a" gives a list of network devices but does not list domains or providers.  A libfabric fi_info scan correlates domains with providers, but makes no mention of network device names.  There are some fi_info records where the domain information is actually a network device (such as when the provider is "tcp").  When the provider is "verbs" the domain is never the network device name itself.  Since the issue here relates to specifying a network device name and domain when using the verbs provider, those other providers don't for this discussion.
I need to be able to tell an administrator tuples of [network device | domain] when specifying the verbs provider and do not have a solid way to do this.  The initial approach was to use the domain associated with the libfabric fi_info scan to find related siblings in the hwloc system device tree.  The sibling of the domain "mlx4_0" might be found to be "ib0".  If there were no other siblings, we could infer that "ib0" is the network device name if "ib0" shows up on the list of network devices according to system command "ip a".
This approach is sufficient if there are precisely two siblings in the device tree.  The complicating factor is that some fabric adapters support more than one network device.  The device tree will show three or more siblings, such as "ib0, ib1, mlx4_0", or "ib0, ib1, enp2s0, mlx4_0".  Given there will be one or more records from the fi_info scan associated with the domain "mlx4_0", how does one accurately determine that the related network device names for the verbs provider are "ib0" and "ib1" and not "enp2s0"?
I have observed that fi_info records associate an IP address for verbs providers.  Using the given IP address, I can match that with an IP address in the data from "ip a" and find a matching network device.  However, there is no API contract that says this is valid.  It's just an observation.  That makes it fragile and prone to faulty behavior as the API ingredients change.
Given that Mercury requires the domain and network device name when specifying the verbs provider, I am looking for a way to reliably obtain this information.

This problem occurs within our test cluster (HPE/Cray).   Each node has 2 sockets and 2 two-port cards.  We currently only cable one interface per card.  The domain names for the cards are mlx4_0 and mlx4_1.  Interfaces ib0 and ib1 are on mlx4_0, and interfaces ib2 and ib3 are on mlx4_1.
Because different nodes in the cluster might connect to the switch using a different port in each card, we use udev rules to rename the active ports to names mlx40 and mlx41 so that the interface names are identical on every node even though the card ports might be different.   The interface names mlx40 and mlx41 will appear on every node; the 2 unused ports could be any of the names ib0 through ib4, different on each node.
Ideally, Joel's code would be able to determine which port in each card is the active port with an IPoIB address configured, and select that as the interface to use with that card's domain.   Not sure what to do in a situation where both ports are active and are configured for IPoIB; I'd be interested to hear others' thoughts.

@krehm Hmm yes not entirely sure. Could you please post that question to https://github.com/ofiwg/libfabric/issues and link back to that issue? it would be best to get advice from the verbs maintainers on that case. Thanks.

Okay, will do, but first let me come at this from a different angle.
What will Mercury do if the application sets OFI_DOMAIN=mlx4_0, but un-sets OFI_INTERFACE, does not provide a value?  Only one of the two interfaces in the card is cabled, so fi_info -v only shows one interface in the domain.   If Mercury only presents the domain name to libfabric, it should get the correct/only interface, even though the app doesn't know the name of the interface.   Would that work?
That doesn't solve the case where libfabric presents two interfaces per domain, no way to identify which is which, but that can be pursued with the libfabric team.

Oops, DAOS returns DER_INVAL if OFI_INTERFACE is not set, we can't get to Mercury, DAOS needs the interface name in order to find the IP address via getifaddrs() to build the domain:ip-addr  info_string for Mercury NA_Initialize_opt().  That seems to imply that Mercury can't find the libfabric interface with just the domain name.  True?

Mercury does not use environment variables, that's a DAOS/CART thing. I assume one thing you could also do is create separate HG classes, one for each interface. You can provide just the domain name, that would work, as long as the OFI domain matches a separate interface.

I filed a slightly edited version of the original comment with the libfabric project: https://github.com/ofiwg/libfabric/issues/5958


Removes down_server.abort_threshold completely

This finishes the previous removal, and this is an incompatible change which can not go into a 9.x release.

@SolidWallOfCode will look at this.


Add CI for macOS & Windows

GitHub has hosted runners for that, we can keep the self-hosted ones for Linux jobs (with io_uring support).
Actually, I should check if GitHub's hosted runners let us do io_uring, it has been around for a while after all.

This was done directly in main, here's the diff: 43c912f93a6a42232395693d78f0e9ccf46d5caf...9ce89e8ffbe6b63d9512befb8fa537fff0219ecd


HHH-2558: Fix batching inserts for multi-table entities

The original fix of https://hibernate.atlassian.net/browse/HHH-2558 caused https://hibernate.atlassian.net/browse/HHH-12470 and was reverted by adding back jdbcCoordinator.executeBatch() in StatementPreparerImpl.prepareStatement(String sql, final boolean isCallable)
The problem with deletes in HHH-12470 was caused by batch deletes not being flushed before non batch delete. That's why we still need executeBatch() in prepareStatement(String sql, final boolean isCallable) that is used by non batch delete
The idea behind this fix is that we use prepareStatement(String sql) (which doesn't trigger executeBatch()) for AbstractBatchImpl.buildBatchStatement with non callable sql

The referenced issue was closed, apparently fixed. And the code has changed very significantly.


revert recent changes on CMakeLists.txt which failed cmake build

Description

revert recent changes on CMakeLists.txt which failed cmake build
Related PR: revert recent changes on CMakeLists.txt which failed cmake build
Checklist:

[ ] I have tested my changes. No regression in existing tests.
[ ] My code is Linted.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Spoken offline and confirmed CBMC Batch: IotHttpsClient_Disconnect  failure can be ignored for now.