[Regression] Ices don't work in scripts running in a subshell

Describe the bug
[Regression] Ices don't work in scripts running in a subshell
To Reproduce
Create a script like this:
source ~/.zinit/bin/zinit.zsh

zinit ice cloneonly nocompile atclone'ln -rsfT ${PWD}/LS_COLORS ${HOME}/.dir_colors'
zinit light trapd00r/LS_COLORS

Delete ~/.zinit/plugins/trapd00r---LS_COLORS if it exists.
Run the script using zsh path/to/script.zsh.
Expected behavior
The symlink ~/.dir_colors should get created.
Versions:

$ZSH_VERSION == 5.8
Operating system name and version (uname -a): Linux host 5.8.3-arch1-1 #1 SMP PREEMPT Fri, 21 Aug 2020 00:00:00 +0000 x86_64 GNU/Linux

Additional context
It works if I run git -C ~/.zinit/bin checkout v3.1

I was trying to do something similar and having a similar problem in that zinit load wasn't working within a script.
I solved it by setting the -i (interactive) option on the sub-shell:
$ cat ~/.local/bin/toggle-theme
#! /usr/bin/env -S zsh -i

source "${HOME}/.local/share/zinit/zinit.zsh"
zinit load chriskempson/base16-shell

eval base16_$(echo ${BASE16_THEME} | sed -e 's/-light/-+dark/;s/-dark/-+light/;s/+//')

If you run zsh -i path/to/script.zsh does that work for you?

I have the same issue, in my case, I use inside the container to install all tasks.
RUN zsh -i -c "source .zinit/bin/zinit.zsh && @zinit-scheduler burst || true"


Strip debug symbols from Wasm binaries

Problem
Wasm binaries produced by Go compiler are pretty huge.
Sometimes they may crash browsers (Safari on iPhone) due to Wasm memory usage quota.
Solution
Add -ldflags "-s -w" build flags to strip debug info. This might save a few megabytes. This should be enough for simple applications to fit in WebKit memory quota.

Compilation of -ldflags '-s -w' and -trimpath flags helped to reduce Go interpreter from 36MB to 35.3MB (2%)


spell broom and item Broomstick 1.14.1 1.14.2

Please don't close the spell broom and item Broomstick doesn't works in 1.14.1 1.14.2
gamemode 0 look impossible to fly :
https://www.youtube.com/watch?v=bOpS-rgpSWY&feature=youtu.be
and no gamemode 1 with broostick crash the server, the server is freezing..

Brooms aren’t compatible with creative mode.
Creative mode in general is not supported by this plugin.
This is also not a server crash, just an error.
If this is not correct, please re-open and provide more details.

https://zupimages.net/up/19/22/aqxi.png
you say that the server does not crash but if it crash :(

You really could've just reopened the other issue like I asked :(
What version of Magic are you on? For 1.14 you need 7.7.2 or brooms (and a few other things) will be broken.
The creative mode issue is a Spigot problem, there's nothing I can do about it. This plugin does not support creative mode

Sorry I do not know github well :( (and my english is not good)
I use Magic 7.7.2
[Magic] Loading Magic v7.7.2-59630c0

Ok thank you- and can you please add the output of /version ? Are you using Spigot, Craftbukkit, Paper or something else?
Brooms have been working ok for me, and I've never seen that TRAP error crash a server before.

spigot 1.14.2 https://zupimages.net/up/19/22/tknh.png

it's "armor_stand_marker: true" the bug, on false the broomstick works but the height is not good...
https://zupimages.net/up/19/22/pivp.png

Yes, I changed this in the builtin broom spell in recent versions. This is why it's always a good idea to use or at least inherit from the default spells. If you have your own spell and RP you'll need to adjust.
The RP that fixes the offset is here: https://s3-us-west-2.amazonaws.com/elmakers/Magic/Magic-RP-8-1.zip

thank you


Error when building or starting to serve if build folder has a css subfolder

The expected behaviour should be: delete the folders and files inside build if KeepFiles == false and start the build process.
What happens:
Starting npm(to serve or build the app) when there's a css folder inside the build/dist dir will stop the server with an error just after it deletes all of its contents.
Starting it a second time will work fine.
The log:
2017-08-19T00_35_11_744Z-debug.txt
Deviating a little from this issue, I've added support for headers, will make a pull request when I have time to fork.
Thanks for the boilerplate, it is really easy to understand compared to all the others I've tested.

I am sorry,
I am busy now. I hope to receive everyone's contributions.


MathJax support has suddenly stopped working on GitHub wikis

I'm using a Gollum based github wiki for a class I teach. LaTeX formulas in my Markdown documents that were previously rendering fine have stopped working all together.
Example:
https://github.com/pmagwene/Bio313/wiki/hands-on3
See the section title "Descriptive statistics as matrix functions" for lots of formulae with missing rendering.

Math rendering of any kind no longer works. Github, is there any timeline for restoring this functionality?


How to build overlap_tags.json

May I ask a question，how to build overlap_tags.json. Is there any way or repo I can search.
It must not be manual...

https://danbooru.donmai.us/related_tag.json?query=1girl&category=general
Use this to fetch tags from the tagger.
Then check which tags overlap; I recall the tolerance is set to 0.95.


Added flash support, Added HAR contents

Further:

DOCKERFILE now contains flash installation (using provision.sh) and is not calling the rm -rf command in provision.sh that deletes necessary perl modules.
Plugins (Flash) are now enabled by default, maybe we should export this as an optional parameter
HAR file contents are now saved by default (because that is what you expect of an HAR file...) - We can/should also provide that as an optional parameter
I removed the useless param of "binary_content" that wasn't used and anyway did not work.
Changed default view port size to 1920x1080, websites behave differently and this seems like a more standard size for today.

Missing:

Flash test - I tried to use the one from decades ago but I couldn't make it start.
HAR with content tests - Current HAR tests work, maybe we should add more tests for HAR with content.

Issues:

The test with the Cyrillic encoding fails, as it wasn't doing anything before I allowed myself to comment the problematic line out - I tried to fix it multiple times but failed, would appreciate help on that.
test_meta_redirect_slowload2_wait_more test fails, after trying endlessly to solve it, I eventually gave up. It doesn't fail, it simply freezes. I would appreciate help on that too.

That's it, hope to see the pull request approved :)

[Current coverage][cc-pull] is 86.02%

Merging [#454][cc-pull] into [master][cc-base-branch] will increase coverage by +0.09%

@@             master       #454   diff @@
==========================================
  Files            40         40
  Lines          4768       4779    +11
  Methods           0          0
  Messages          0          0
  Branches        654        655     +1
==========================================
+ Hits           4096       4111    +15
+ Misses          513        510     -3
+ Partials        159        158     -1

Powered by Codecov. Last updated by [a0c503d...ea154ae][cc-compare]
[cc-base-branch]: https://codecov.io/gh/scrapinghub/splash/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/scrapinghub/splash/compare/a0c503d86b26d621b343da59b228e57cd80a4cb2...ea154ae6019aabdcaa5fbcb4a3d52ce79c4597c5
[cc-pull]: https://codecov.io/gh/scrapinghub/splash/pull/454?src=pr

@kmike

Hey @shaylevi2 - great job, it looks like you've found a way to preserve the contents, that's a wanted feature!
I left a few comments to the PR.
Another issue is that after this change we'll be keeping all data in memory twice. This is 100% fine, but I think it should be optional - there should be a way to turn content storage off:

endpoints which don't use HAR shouldn't store the content, for efficiency - this includes render.html, render.png, render.jpeg and render.json without 'HAR' key.
it should be possible to turn it off per-request in on_request or maybe on_requst_headers callback - I think we should add a method or attribute Request - something like request.record_response_body=true (ideas for a better name?), similar to request:abort() - this way it'd be possible to e.g. keep only images in memory twice, not js or css files;
for existing users this change will cause slowdowns and increased memory usage for all /execute scripts - existing users don't use this feature, so they'll just get an overhead. What do you think about making it opt-in? In addition to request.record_response_body we can add something like splash.response_body_enabled which is false by default, but can be turned ON in a script.
for render.har and render.json there should be an option to disable content storage;
this feature needs tests.

Flash:

I think PLUGINS_ENABLED should be customizable at runtime, not only at start time - we should provide an option for HTTP API and a splash.plugins_enabled property.
it'd be nice to have a test for Flash - for example, check that screenshot contains flash contents.

I realize this is a lot more work. Some of these features are nice to have but not critical though. I think the following is required:

tests for both features;
encodings fix;
content storage should be disabled by default for all endpoints, but there should be an option to turn it ON at runtime in /execute endpoint via splash.response_body_enabled (a better name ideas?).

Unfortunately, the organization I'm working in decided not to give me any more time to work on the PR, as we're in a hurry to get a product out, writing further tests is being postponed for now (in our fork and generally).

For now, the decoding works perfect for us, it's the only decoding we found that works as expected (i.e giving the same results as Chrome Developer Tools would). Other decoding methods we tried, either left too much as base64, or did not decode the text properly.

I agree it would be better to use it as a parameter, unfortunately as said previously, I'm not given any time to work on it.

If anyone wants to advance this PR further, do feel free to do so.

Ok, thanks @shaylevi2 for your time!

@kmike
Small update: I fixed the bug with the slow_wait, now all tests pass.

Small update: added "move_to_end" monkey patch for python2.7 and now forcing 2.7 tests to pass.

Is this going to be merged?  Would love HAR content support.

Thanks @shaylevi2! Merged in #494.


ObjectId not generated

Hello all,
i have a locally instance parse server.
I try to create a new user, but it seems my parse server doesn't create object id.
Here is a example :
Object create with the new server
 { "_id" : "bwbOBDuu7Y", "_rperm" : [ "
Object from the data from parse.com
{ "_id" : ObjectId("575ab851f58be03039c5fd91"), "
The _id fields is not the same at all.
I see this issue https://github.com/ParsePlatform/parse-server/issues/721 but i have no error in my log
Thx for your help

can you run with VERBOSE=1 to dig deeper in your errors?

Please update to latest version and reopen the issue if it persists


Specify the protocol metadata schema

JSON protocols and Python protocols both have a notion of protocol metadata, which, roughly, is nonessential information like the protocol author. However, they don't have a unified notion of exactly what can be in this metadata.
The new HTTP API will pass protocol metadata straight through to the client. So, it will be more client-friendly if the protocol metadata follows a consistent, unified spec.
Questions to consider

What exactly is metadata? What does and does not belong there?
What types can appear in metadata? Are nested dicts/objects allowed?
What specific fields (like author) are known?

What are their types?
What are the expected formats of their values? (e.g. is author an email address, or a name? Can description have Markdown?)
Is a field being null equivalent to it being omitted?
Note: We should probably scrape our corpus of custom protocols to see what fields they use.

Can Opentrons add more fields? If, so, where/how?
Are fields camelCase or kebab-case or something else?
Can users invent their own fields? If so, how should they keep them separate?

Like this? {"myCompanyMyCustomField": "foo"}
Or maybe like this? {"userExtra": {"myCustomField": "foo"} }

Definition of "done"

A public spec addressing at least the above

Either machine-readable or human-readable or both

It would be a good idea to get input from the participant(s) of #8236.

Current de-facto "standard" according to the app: https://github.com/Opentrons/opentrons/blob/6697e394458dfebf7974f7ff0edfd99f8a2e944b/app/src/redux/protocol/selectors.ts#L92-L138

Thought:
What happens if something like this happens?

The JSON protocol schema starts with some blessed metadata field, say authors, typed as a str.
This blessed field makes its way into the Pydantic bindings that the robot software uses to parse the metadata.
Later, we change its type to optionally be a list of strings: list[str] | str.

Now, when you upload a new JSON protocol (with a list of authors) to an old robot whose Pydantic models haven't been updated yet, the robot will throw a parse error. Even though it really has no reason to care about that metadata's schema; it should just pass it on to the client transparently. There's unnecessary coupling and fragility.
Options:

Do we have to make it a policy that after we add a metadata field, we can never change its type?
Or should the robot server's Pydantic bindings to the JSON protocol schema not include any blessed metadata fields?
Or something else?

I think that's a good thought. I'd be pretty amenable to option 1: do not change the type of a field once "blessed". Add a new field if new information (or shape of information) is needed. Your example makes me think of npm's package.json spec, where there is both an "author" (singular) field and and a "contributors" (plural) field


[Feature]support agg function Percentile_disc

What type of PR is this：

[ ] BugFix
[x] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Which issues of this PR fixes ：

Fixes https://github.com/StarRocks/starrocks/issues/16332
Problem Summary(Required) ：

Checklist:

[x] I have added test cases for my bug fix or my new feature
[ ] This pr will affect users' behaviors
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[ ] I have checked the version labels which the pr will be auto backported to target branch

[ ] 2.5
[ ] 2.4
[ ] 2.3
[ ] 2.2

[FE PR Coverage Check]
:heart_eyes: pass : 30 / 30 (100.00%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/catalog/FunctionSet.java
27
27
100.00%
[]

:large_blue_circle:
com/starrocks/sql/analyzer/DecimalV3FunctionAnalyzer.java
3
3
100.00%
[]

run starrocks_be_unittest

run starrocks_be_unittest

https://github.com/StarRocks/starrocks/issues/6922 hope the function signature is  PERCENTILE_DISC (expr, percentile [,HIGH | LOW] ) , but after talking to zimo, we chose the upper one like snowflake

[FE PR Coverage Check]
:heart_eyes: pass : 51 / 55 (92.73%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/sql/analyzer/FunctionAnalyzer.java
6
10
60.00%
[354, 357, 358, 362]

:large_blue_circle:
com/starrocks/catalog/FunctionSet.java
27
27
100.00%
[]

:large_blue_circle:
com/starrocks/sql/analyzer/DecimalV3FunctionAnalyzer.java
2
2
100.00%
[]

:large_blue_circle:
com/starrocks/sql/analyzer/ExpressionAnalyzer.java
16
16
100.00%
[]

run starrocks_admit_test

run starrocks_admit_test

@Mergifyio backport branch-3.0

@Mergify backport branch-3.0

https://github.com/StarRocks/starrocks/pull/20346 cp to 2.5 by this pr


Dynamically added tests are not counted when displaying number of passed tests.

I currently have 2 "regular" test, and 1 "dynamic" test (i.e. added with suite.test(...)) in a before() method. When all tests pass the output is
+[:-) Running Chaplain tests ...
+[:-) All tests passed (2 in total).

I am certain that the dynamic test does run.

Will look into that in a sec.

@rivierasolutions There is a test fixture explicitly for that here: https://github.com/krajzeg/chaplain/blob/master/test-data/before-after/before-after.chaplain.js - this suite, when executed, does print "All tests passed (1 in total)." - verified by this test here: https://github.com/krajzeg/chaplain/blob/master/test/cli/before-after-tests.js#L27
Can you provide a short test suite that reproduces the bug?

Sure :)
var plainRequest = require('request');
var ssqSignon = 'https://ssqsignon.com';

var request = function(options) {
    return new Promise(function(resolve, reject) {
        plainRequest(options, function(error, response, body) {
            if (error) {
                throw new Error(error);
            }
            resolve(body, response);
        });
    });
};

var post = function(url, options) {
    options.method = 'POST';
    options.url = [ ssqSignon, url ].join('');
    return request(options);
};

suite('is-up', function() {
    config({server: ssqSignon});

    test('landing-page', {url: '/'});
});

suite('authorize-password', function() {

    config({
        server: ssqSignon,
        json: { ignore: [ '$.access_token', '$.refresh_token', '$.expires_in' ] }
    });

    test('execute', { request: {
        method: 'POST',
        url: '/chaplain-fun/tokens',
        body: {
            client_id: 149,
            grant_type: 'password',
            username<EMAIL_ADDRESS>            password: 'testtest1'
        },
        json: true
    }});
});

suite('whoami', function() {

    config({
        server: ssqSignon,
        json: { ignore: [ '$.expires_on' ] }
    });

    before(function(suite) {
        return post('/chaplain-fun/tokens', { body: { client_id: 149, grant_type: 'password', username<EMAIL_ADDRESS>password: 'testtest1' }, json: true })
            .then(function(body) {
                runTests(suite, body.access_token);
            });
    });

    function runTests(suite, accessToken) {
        suite.test('execute', { request: {
            method: 'GET',
            url: '/chaplain-fun/whoami',
            headers: { Authorization: ['bearer', accessToken].join(' ') },
            json: true
        }});
    }
});

This is a much worse bug, in fact - two tests have the same name, and their results are not tracked separately (even though they belong to different suites). Workaround for now - change one of the names from execute to something else.
This is pretty serious, so I'll try to fix it shortly.

@rivierasolutions Can you check master to see if the tests are treated separately now? :)


Values read from DB need escaping in "And I should have" statement

e.g. for input value: Last1 Name With 25 Char's
note: apostrophe above
You need to escape it like below:
And I should have a "woody_crm.users" with "id:{woody_crm.users_id},forename:First Name With 25 Char's,lastname:Last1 Name With 25 Char's"

Sure will get this sorted as soon as possible. Thanks for logging the issue.

In PR https://github.com/forceedge01/genesis-behat-sql-extension/pull/7

Merged.


Expanding support for other input devices

Expanding support for other input devices
Hi,
I need to either insert new keys in Keyboard.js or create a new input module altogether. The reason why I need to do this, is because I'm creating a webapp for a tizen device, and when I use the input library to register the remote keys it works, but the app freezes, because this key it's not a regular pc keyboard key.
I'm working with nunustudio 0.93 desktop version for windows, target platform is tizen OS.
So I was considering the following options:
add the key to keyboard.js
create a new input custom module
for either of these options, I need to be able to create a new build for the desktop version. Any help in how to achieve this?
thanks in advance!

HI
Thanks a lot for reporting this.
You can open a PR with the changes that you need or you can list what are the TIZEN specific keys that you are aware of and i can add them immediatly to nunuStudio.
After that you have to move to the latest version of the editor. Right now it is a lot easier to publish new versions i can move the dev to 0.9.6 and publish 0.9.5 as it is.
Thasnk a lot.
Cheers

Hi,
thanks for the quick reply. here is a list of the case code keys I need:
MediaPlayPause | 10252 |
Menu | 18
MediaRewind | 412 |
Tools | 10135
MediaFastForward | 417
Info | 457
MediaPlay | 415
Exit | 10182
MediaPause | 19
Back | 10009
MediaStop | 413
Caption | 10221
MediaRecord | 416
ChannelList | 10073
MediaTrackPrevious | 10232
MediaTrackNext | 10233
3D | 10199
Ok, so you are telling me once these keys are added, I should use the web editor correct?
thanks again!
Jorge

Thanks a lot im adding them now.
Are these device specific? Have you pulled by printing or there is a API?
I will make this available for desktop as well. I have been delaying a new desktop release for a while x)
Thanks a lot

Hi Tentone,
Please let me know when you have the update available. Looking forward to version 0.9.5! :)

Hi
Working on it right now. Will upload it soon.
Thanks a lot!

Available for download. I needed to change some things before publishing.


Regenerate the tf_totp_secret upon selecting a new method

This fixes the issue that the TOTP secret gets regenerated after it has
been set and validated by the user and also ensures that a new secret is
generated every time a new method is selected for 2FA.
Fixes #215

I've added a test case that uses SMS 2FA and ensures that the tf_totp_secret stays the same during and after finalizing the setup process.


Radio field isn't generating properly

Field definition:
{         "name": "type",         "dbType": "boolean",         "htmlType": "radio,Company:1,Trader:0",         "validations": "required",         "searchable": true,         "fillable": true,         "primary": false,         "inForm": true,         "inIndex": true     }
https://imgur.com/m7p1zaJ

@mdmahbubhelal Can you post some more details? I am not able to figure out what's the problem. Because it's already working fine.


Size is not taking effect

Size is always 40px whatever value you pass its always the same
<button @click="apply(props.row)" class="btn green-haze">
       Apply
      <vue-element-loading :active="applying" size="25" spinner="ring" color="#ffffff"/>
</button>

I try to change size in my example page, it's work correctly. Please check your custom style
https://vue-element-loading.netlify.com/

Sorry for that. The issue was because I am using an outdated version.
I think that docs need to be updated to address faetures with the related version numbers required. :)

PR welcome :)


[BUGFIX]try avoid the error in operator/tensor/amp_cast.h

I'm trying to avoid the error generated by amp using bfloat16
The error is due to:
/me/prog/prog-amp.py:77: UserWarning: All children of this Sequential layer 'compose1_' are HybridBlocks. Consider using HybridSequential for the best performance.
  transform_test.hybridize(static_alloc=True,static_shape=True)
Traceback (most recent call last):
  File "/me/prog/prog-amp.py", line 359, in <module>
    loss0   = loss_fn(output, label)
  File "/me/incubator-mxnet/python/mxnet/ndarray/ndarray.py", line 314, in __mul__
    return multiply(self, other)
  File "/me/incubator-mxnet/python/mxnet/ndarray/ndarray.py", line 3757, in multiply
    return _ufunc_helper(
  File "/me/incubator-mxnet/python/mxnet/ndarray/ndarray.py", line 3576, in _ufunc_helper
    return fn_array(lhs, rhs)
  File "/me/incubator-mxnet/python/mxnet/contrib/amp/amp.py", line 109, in _new_fun
    return f(*args, **kwargs)
  File "<string>", line 52, in broadcast_mul
  File "/me/incubator-mxnet/python/mxnet/_ctypes/ndarray.py", line 82, in _imperative_invoke
    check_call(_LIB.MXImperativeInvokeEx(
  File "/me/incubator-mxnet/python/mxnet/base.py", line 246, in check_call
    raise get_last_ffi_error()
mxnet.base.MXNetError: Traceback (most recent call last):
  File "/me/incubator-mxnet/src/io/../operator/elemwise_op_common.h", line 135
MXNetError: Check failed: assign(&dattr, vec.at(i)): Incompatible attr in node  at 1-th input: expected bfloat16, got float32
Error in atexit._run_exitfuncs:
Traceback (most recent call last):
  File "/me/incubator-mxnet/python/mxnet/base.py", line 587, in _notify_shutdown
    check_call(_LIB.MXNotifyShutdown())
  File "/me/incubator-mxnet/python/mxnet/base.py", line 246, in check_call
    raise get_last_ffi_error()
mxnet.base.MXNetError: Traceback (most recent call last):
  File "/me/incubator-mxnet/src/operator/tensor/./amp_cast.h", line 136
MXNetError: Unknown type enum 12

which is tested under mxnet v1.x, but seems also affect v2.0
since 30-series RTX card support bfloat16, there is no need to disable it using #ifndef __NVCC__ explicitly,
I don't know whether it works, but things could not be worse.
Description
import mxnet as mx
try:
  from mxnet.contrib import amp
except:
  from mxnet import amp

#amp.init('float16') # Ok
amp.init('bfloat16') # Error in previous version
net=mx.gluon.nn.Sequential()
net.hidden=mx.gluon.nn.Dense(400)
net.out=mx.gluon.nn.Dense(10)
ctx=mx.gpu()
net.initialize(ctx=ctx)
trainer = mx.gluon.Trainer(net.collect_params(), 'sgd')
amp.init_trainer(trainer)
with mx.autograd.record():
  loss=net(mx.nd.array([32,28,28],ctx=ctx))

loss.backward()
trainer.step(1)

such code will fail in previous version of mxnet, and here I provide a workaround.
further modification of bf16 is needed.
Checklist
Essentials

[x] PR's title starts with a category (e.g. [BUGFIX], [MODEL], [TUTORIAL], [FEATURE], [DOC], etc)
[x] Changes are complete (i.e. I finished coding on this PR)
[ ] All changes have test coverage
[ ] Code is well-documented

Changes

[ ] Feature1, tests, (and when applicable, API doc)
[ ] Feature2, tests, (and when applicable, API doc)

Comments
Actually this PR does nothing, Further support of bf16 (including a very important operator convolution) is required but I know nothing about cudnn.

@mxnet-bot run ci  [unix-gpu]

@mxnet-bot run ci [unix-gpu, unix-cpu, centos-cpu]

@mxnet-bot run ci [unix-gpu, centos-cpu]

@Neutron3529 thank you!

@Neutron3529 thank you!

You're welcome.
Since I am in China, it is not very convenience to visit github.
You may open a new issue about the amp support with bfloat16.
This commit do not work at least with convolution layers.


Should any change of the value in component, react by ReactiveSystem?

I know that using "e.AddComponent", "e.ReplaceComponent", or "e.RemoveComponent", then the ReactiveSystem can be triggered.
Sometime I change the values in component stealthily, by group some components and foreach.
Like:
foreach(e in entities)
{
 e.position.value = (2,3);  //This bypass the ReactiveSystem
}

Is this bad? What's the consequence?

The consequence is that the systems and group events are not triggered. If you want this behavior you can do that.

The consequence is that the systems and group events are not triggered. If you want this behavior you can do that.

Thx for answer:D


Add option to set period in hh:mm:ss for 'addbgevent' command

'addbgevent' command has been removed


Update Delta Green by ArcDream

Changes / Comments

fixed esthetics sheet
improved roll templates for lethality
add armor penetration

Roll20 Requests
Comments are very helpful for reviewing the code changes. Please answer the relevant questions below in your comment.

[x] Does the pull request title have the sheet name(s)? Include each sheet name.
[x] Is this a bug fix?
[ ] Does this add functional enhancements (new features or extending existing features) ?
[x] Does this add or change functional aesthetics (such as layout or color scheme) ?
[ ] If changing or removing attributes, what steps have you taken, if any, to preserve player data ?
[ ] If this is a new sheet, did you follow Building Character Sheets standards ?

If you do not know English. Please leave a comment in your native language.

Hello! Your Pull Request includes changes to translation files. We would prefer sheet contributors use Crowdin for translations. Please email Cassie at<EMAIL_ADDRESS>and she will get you an invite to our Crowdin within 48 hours. Be sure to include the languages you would like to translate, and please use Crowdin for future translation updates! Thanks for all you do!
Additionally, you have merge conflicts in the fr.json - those need to be resolved before we can merge. Once those are fixed, we will not deny your pull request, but we prefer you to use Crowdin. Thanks!


How do I know what my "user" name is?

I can't find any info on any bitcoin core "wallet identifier" or "user". I probably know what my password is, but how do I find what to put as the "user" option? Right now, without a user and password, I just get a "socket hang up" error.

Also, why is a username and password even necessary if I just want to query the blockchain (and not actually do anything with the wallet)?

I have the following configuration in bitcoin.conf:
rpcuser=myUserName
rpcpassword=myPassword

But I'm still getting a "socket hang up" error on connecting to port 8333 whether I use the ssl option or not. If I use port 8332 (which is supposed to be the default port), I instead get ECONNREFUSED. What am I missing?

Hmm, it seems that running bitcoin-core doesn't allow me to connect via rpc. If I run bitcoind following the instructions here, it works: https://en.bitcoin.it/wiki/API_reference_(JSON-RPC)


Login Page

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
The login page doesn't have show hide password icon so would like to work on that.
Expected Behavior
Will make sure that the login page works perfectly with necessary features suggested above. Kindly assign this issue to me.
Reproduction steps
Will make sure that the login page works perfectly with necessary features suggested above. Kindly assign this issue to me.

Screenshots

Logs
No response
Browsers
Chrome
OS
Windows

The login page doesn't have show hide password icon so would like to work on that.

Login has, sign up doesn't
@MetiUmashri

yep will make changes accordingly


Add search results to the chat

Is your feature request related to a problem? Please describe.
I am mostly using phind instead of the code chat, because it does not contain search results.
Currently mostly depend on this plugin for FIM.
Describe the solution you'd like
Add search results to the chat.
Just saw this has been open sourced: https://x.com/KhazzanYassine/status/1819787693330399344 can maybe be a source of inspiration
Thank you  for your work. The plugin is useful as is, but search result rag would really make it take off I think.

Hey @spirobel thanks for the suggestion it's in the works. It takes me a long time to release things since I have full time work commitments alongside this plugin. I have an end to end solution working now it just needs some refinement before production release. It's coming soon. Many thanks.

Amazing!

It's also worth to mention that Ollama doesn't have a re-ranking model and the vector database needs to be running inside vscode,

An alternative would be to use simple keyword search and a prompt to rerank. (also makes it easier to debug, because humans can read keywords, but not vectors)
I remember reading an article comparing keyword search to vector search in the RAG usecase and it came to the conclusion that keyword search works just as good and even better in some search cases.

Added support in version 3.13.3.


Hm/fix color label

Checklist
GitHub

[x] I've given this PR a concise, self-descriptive, and meaningful title
[x] I've linked relevant issues in the PR body
[ ] I've applied the relevant labels to this PR
[x] I've assigned a reviewer

PR contents

[ ] I've consulted ivadomed's internal developer documentation to ensure my contribution is in line with any relevant design decisions
[ ] I've added relevant tests for my contribution
[ ] I've updated the relevant documentation for my changes, including argparse descriptions, docstrings, and ReadTheDocs tutorial pages

Description

this should resolve https://github.com/ivadomed/ivadomed/pull/834#discussion_r668044624
Linked issues

resolves #720

and I need to remove the test_rgb_file from the data_functional_testing after this is merged

I tested the PR and realize I may not be the best person to do this review. I am not familiar with the save_color_labels function (I never used it before it was commented). On a small training on the data_example_spinegeneric with "target_suffix": ["_seg-manual", "_csfseg-manual"]. I got this _color.nii.gz image as a result (in black and white).

Moreover, because the function now writes a NifTI file without header, the orientation is not known (? on the side of the image).
From the issue #715 and PR #758, my understanding is that we want every Nibabel object to be created with the complete header, which is now not the case in this PR for the save_color_labels function in visualize.py.
The function and associated test were first silenced in issue #720 and PR #714.
It seems that the test was fixed in #832, but the function still expects a different header than the one that are in our gt_files.
Question: it is possible to modify the implementation of the function itself to accept the right kind of datatype from the header of our gt_files, instead of changing how the Nifti file is written?

it is colored label...so the datatype would be rgb, so does the header

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

4 of 5 (80.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 70.293%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

ivadomed/testing.py
1
2
50.0%

Totals

Change from base Build<PHONE_NUMBER>:
0%

Covered Lines:
4122

Relevant Lines:
5864

💛  - Coveralls

it is colored label...so the datatype would be rgb, so does the header

You are right, I checked the header and the data_type is indeed RGB for the _pred_color.nii.gz file. I think I found how to see it in colors in fsleyes with the 3D/4D RGB(A) Volume display option. This is what I got:

However, I think preserving the rest of the header is important to preserve the shape and orientation of the image. For example, the same image without the color (_pred.nii.gz) gives me this:

As I mentioned earlier, I don't know enough to comment further on this and other inputs are needed.

change strategy...right now I copy the all of the header and change the datatype to rgb when save to color label


Response and GBM files

There seems to be an issue with the reading of GBM DRMs (similar to what I have seen with gammapy)

/usr/local/Cellar/python/HEAD/Frameworks/Python.framework/Versions/2.7/lib/python2.7/site-packages/threeML-0.3.2-py2.7.egg/threeML/plugins/ogip.pyc in _read_matrix(self, data, n_channels)
    750             for j in range(n_grp[i]):
    751
--> 752                 rsp[i, f_chan[i][j]: f_chan[i][j] + n_chan[i][j]] = matrix[i][m_start:m_start + n_chan[i][j]]
    753
    754                 m_start += n_chan[i][j]

ValueError: could not broadcast input array from shape (128) into shape (127)

I believe the issue lies in FOTRAN style indexing:
f_chan[i][j] = 1 implies that the first channel is the first entry (at least in GBM DRMs)
but the array indexing for Numpy would want this to be zero. you 1:128 is a 127 length array when you really want a 128 length array.
Is this general or GBM specific?

Ok, so I tried the new plugin on XRT DRMs. It worked fine and they correctly specify the first channel as zero. GBM specifies the first channel as 1. I propose making and exception when there is a broadcast error where FCN has minus one removed. In fact, in my other code I was always doing this. I can make the change and commit. @giacomov , thoughts?

I have already corrected this a couple fo days ago and tested on a GBM matrix, probably there are different cases. Can you post somewhere the matrix which doesn't work?
This part is a little tricky, because if you get it wrong you can bias your spectral results in a non-obvious way, so we need to be careful and really understand what is going on.

Here is an RSP (from the example folder) it does not natively work.
This change causes it to work:
        #
                try:

                    rsp[i, f_chan[i][j]: f_chan[i][j] + n_chan[i][j]] = matrix[i][m_start:m_start + n_chan[i][j]]

                except(ValueError):

                    rsp[i, f_chan[i][j] -1: f_chan[i][j]-1 + n_chan[i][j]] = matrix[i][m_start:m_start + n_chan[i][j]]

glg_cspec_n3_bn110731465_v00.rsp2.zip

@drJfunk I have fixed this already in the new OGIPLike plugin.
When you have time, please port the TTE plugin to the new OGIP plugin, by making it inherit from there instead of the old GenericOGIPLike plugin, and update the example as needed.
I already updated the basic_example.ipynb notebook for your reference.

I see this now. However, why is FermiGBMLike working? I do not see it inheriting from OGIPLike.

On Sep 2, 2016, at 6:53 PM, Giacomo Vianello<EMAIL_ADDRESS>wrote:
@drJfunk https://github.com/drJfunk I have fixed this already in the new OGIPLike plugin.
When you have time, please port the TTE plugin to the new OGIP plugin, by making it inherit from there instead of the old GenericOGIPLike plugin, and update the example as needed.
I already updated the basic_example.ipynb notebook for your reference.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/giacomov/3ML/issues/53#issuecomment-244429483, or mute the thread https://github.com/notifications/unsubscribe-auth/AGc155l4EIJmaZCNs0sO1FQs49LJhgJWks5qmFSTgaJpZM4Jwjpj.

+++++++++++++++++++++++++++++++
J Michael Burgess
+46 72-381 30 20
<EMAIL_ADDRESS>

yeah, I wanted to discuss with you what you would like to have in the parent class (OGIPLike) and what is instead specific to the GBM.
The plan is to retire GBMLike (or make it just another name of OGIPLike) as it is really confused and bug-prone.
At the moment there is only one method that we should definitely port, which is view_counts_spectrum.
Anything else which is important for any PHA spectrum?

I just realized what has happened and am taking a look at the TTElike. For the moment, I see the TTElike being similar to Veritas or HAWC or LAT because it has some additional functionality that is specific to the data type. I think TTELike can be the best GBM plugin because it enables more flexibility and just keeping an OGIPlike as an option is fine. I like some of the gammapy response functions because they allow users to explore the DRMs which can be useful. Besides that, I think OGIPLike is flexible enough on its own (the data prep is always done outside the frame work). People can build more advanced plugins that do the data prep and ultimately pass through to OGIPLike in the end. Doe sit make sense?

On Sep 3, 2016, at 2:00 PM, Giacomo Vianello<EMAIL_ADDRESS>wrote:
yeah, I wanted to discuss with you what you would like to have in the parent class (OGIPLike) and what is instead specific to the GBM.
The plan is to retire GBMLike (or make it just another name of OGIPLike) as it is really confused and bug-prone.
At the moment there is only one method that we should definitely port, which is view_counts_spectrum.
Anything else which is important for any PHA spectrum?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/giacomov/3ML/issues/53#issuecomment-244542542, or mute the thread https://github.com/notifications/unsubscribe-auth/AGc15_oBogLsYy3Esj2hdPM4N1r5Ri8Lks5qmWFZgaJpZM4Jwjpj.

+++++++++++++++++++++++++++++++
J Michael Burgess
+46 72-381 30 20
<EMAIL_ADDRESS>

However, the OGIPLike constructor isn't suitable for the TTE data. It will need to be virtualized, or the constructor will need to have the ability to customize the file type outside of PHA. We had the same problem originally, what path do you want to take @giacomov ?

I think the most sensitive would be to add a static method like "from_counts" to the OGIPLike class, so essentially you can build an instance on the fly without having to go through a .pha and a .bak file.
Would this be sufficient?

Yes. I think I get the idea. Not to convolute the issue, but perhaps an EVENT class is warranted? This could be a solution that is general enough for other similar data types?
/J. Michael
Sent my iPhone

On Sep 3, 2016, at 15:41, Giacomo Vianello<EMAIL_ADDRESS>wrote:
I think the most sensitive would be to add a static method like "from_counts" to the OGIPLike class, so essentially you can build an instance on the fly without having to go through a .pha and a .bak file.
Would this be sufficient?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

I like it. An EventList class which provides a get_pha method to get a PHA on the fly.
Are you going to work on this, or should I?

I can grab it. Have you a suggestion of another data type outside of GBM tte that I can look at so that I don't make it too GBM specific?
/J. Michael
Sent my iPhone

On Sep 3, 2016, at 16:05, Giacomo Vianello<EMAIL_ADDRESS>wrote:
I like it. An EventList class which provides a get_pha method to get a PHA on the fly.
Are you going to work on this, or should I?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

I'd make a constructor which gets simple numpy arrays, and then you can make static methods like from_TTE to specialize to specific file formats (like pandas does with DataFrame and other containers).
Each mission is going to need a reader for their specific format, and at some point we will also need to support the universal format that the gammapy people are defining.

Well, it turns out that GBM does not use the TLMIN keyword (at least not universally):
/usr/local/Cellar/python/HEAD/Frameworks/Python.framework/Versions/2.7/lib/python2.7/site-packages/astropy/io/fits/header.pyc in _cardindex(self, key)
   1727
   1728         if not indices:
-> 1729             raise KeyError("Keyword %r not found." % keyword)
   1730
   1731         try:

KeyError: "Keyword 'TLMIN4' not found."

Therefore, the response code cannot read the file.
glg_cspec_n3_bn080916009_v07.rsp.zip

I've added a way around this by assuming TLMIN = 1 when it cannot find it and will be committed with this entire update


Add completion for nmcli

Can you add completion for nmcli (Network Manager Command Line Interface). This command allows to control the network-manager from the terminal.
The default completion that is in zsh is obsolete (here is file in github) and gives wrong completions.

The current version is up-to-date: https://github.com/zsh-users/zsh/blob/eada7e1138a3fca90f085dd764ad86098e58c9ac/Completion/Linux/Command/_networkmanager


Extract landmarks from each hand in mediapipehands

System information

Solution: Hands and Holistic
Programming Language and version ( e.g. C++, Python, Java): Python

Describe the expected behavior:
My goal is to extract the hand landmarks from media pipe for each hand to train an LSTM network with and to recognize different actions.
With the Holistic solution, this is quite easy, since all landmarks of the left or right hand can be retrieved via the respective function.
def extract_keypoints(results):
    lh = np.array([[res.x, res.y, res.z] for res in results.left_hand_landmarks.landmark]).flatten() if results.left_hand_landmarks else np.zeros(21*3)
    rh = np.array([[res.x, res.y, res.z] for res in results.right_hand_landmarks.landmark]).flatten() if results.right_hand_landmarks else np.zeros(21*3)
    return np.concatenate([ lh, rh])

However, I get problems with hand detection when shooting from egoperspective and the holistic solution. So I have to switch to the mediapipe hands solution.
With mediapipe hands there is only the function MULTI_HAND_LANDMARKS, which only gives me the landmarks of one hand.
if results.multi_hand_landmarks:
            print(results.multi_handedness)
            for hand_landmarks in results.multi_hand_landmarks:
                print('hand_landmarks:', hand_landmarks)
            for hand_landmarks in results.multi_hand_landmarks:
                mp_drawing.draw_landmarks(image,hand_landmarks,mp_hands.HAND_CONNECTIONS,mp_drawing_styles.get_default_hand_landmarks_style(),mp_drawing_styles.get_default_hand_connections_style())

As a result I get something like this, only 20 more times for the total of 21 landmarks per hand.
hand_landmarks: landmark {   x: 0.8579444885253906   y: 0.904037356376648   z: 2.08246397903622e-09 }
Does the handlandmark output always alternate between one hand and the other if there are two hands in the picture?

Hi @sureshdagooglecom,
thanks for your answer!
Could you please repost the links, as error messages appear when trying to access them.
Do you know a way to extract the .x .y & .z values of each hand then ?

Hi @Mau794 ,
We have similar issue running in the thread , please check this https://github.com/google/mediapipe/issues/742


fix typo in conditional for swift dispersion

This fixes a typo for PR:
https://github.com/blueboxgroup/ursula/pull/2606

Retest


Opponent has different note skin

Describe your problem here.
How do you have the Opponent have a different note skin, kinda like in Bob and Bosip. how do i do that? or do i just have to make them their own notes and not as a note skin
Are you modding a build from source or with Lua?
Lua
What is your build target?
Windows x64
Did you edit anything in this build? If so, mention or summarize your changes.
I just added new characters and Notes and changed some sprites and a few lua scripts

how do I do it though?

several posts in discussions about this

can i at least see the posts

i just need a lua script to do it and thats all

I can't find any discussion posts that help

function onCreatePost()

    for i=0,3 do
        setPropertyFromGroup('opponentStrums', i, 'texture', 'NOTE_assets2')
    end

end

use this script

I made a system that works like this:
function onCreatePost()
	changeNoteSkin('noteskin1', 'player')
	changeNoteSkin('noteskin2', 'opponent')
end

and this is the changeNoteSkin function
btw you should make "noteskins" folder in images of your mod folder for it to work
function changeNoteSkin(skin, player)
    if not (skin == 'vanilla') then -- checks if skin isn't vanilla (basic one)
	local mustHit = true -- mustHit detects bf's notes
	if player == 'opponent' then
		mustHit = false -- if it's false then it's opponent's notes
	end

	for i = 0, getProperty(player..'Strums.length')-1 do -- strums from the player parameter (ONLY OPPONENT AND PLAYER)
            setPropertyFromGroup(player..'Strums', i, 'texture', 'noteskins/'..skin) -- changes the noteskin from parameter skin
        end

        for i = 0, getProperty('unspawnNotes.length')-1 do -- counts all unspawned notes
            if getPropertyFromGroup('unspawnNotes', i, 'noteType') == '' then -- checks if the note is normal note
		if mustHit and getPropertyFromGroup('unspawnNotes', i, 'mustPress') then -- does a check if it's a player note
                	setPropertyFromGroup('unspawnNotes', i, 'texture', 'noteskins/'..skin) -- sets player's noteskin
		elseif not mustHit and not getPropertyFromGroup('unspawnNotes', i, 'mustPress') then -- opponent note check
			setPropertyFromGroup('unspawnNotes', i, 'texture', 'noteskins/'..skin) -- sets the opponent note
		end
            end
        end
    end
end

oh yeah the image of your noteskin in noteskins folder must be named EXACTLY after the skin parameter you gave to changeNoteSkin function

function onCreatePost()

    for i=0,3 do
        setPropertyFromGroup('opponentStrums', i, 'texture', 'NOTE_assets2')
    end

end

use this script

also this script just replaces your strums, not the notes too

	local mustHit = true -- mustHit detects bf's notes
	if player == 'opponent' then
		mustHit = false -- if it's false then it's opponent's notes
	end

There is literally no point in having mustHit but if it works it works I guess

i haven't really thought of that thanks for giving me a heads up!

Well, if this question is still open, use this code: (this code isn’t mine, is from SilkyIncorprated)
function onCreate() -- this is from SilkyIncorprated
	if dadName == 'pico' then --replace the name for your character name
		for i=0,4,1 do
			setPropertyFromGroup('opponentStrums', i, 'texture', 'NOTE_assets_3D') -- Change this
		end
		for i = 0, getProperty('unspawnNotes.length')-1 do
			if not getPropertyFromGroup('unspawnNotes', i, 'mustPress') then
				setPropertyFromGroup('unspawnNotes', i, 'texture', 'NOTE_assets_3D'); --Change also this
			end
		end
	end
end

I have a better solution
Instead of dadName, simply put setPropertyFromGroup('opponentStrums', i, 'texture', dadName..'NOTE_assets)
it's gonna make the game find files "picoNOTE_assets" png and xml

Note that dadName doesn't work with onCreate, use onCreatePost


Incorrect user_count

I noticed the very top entry in the list shows an incorrect user_count.
I have checked back after the list has updated to see if the user_count would update as well, but it has not.
AVG Web TuneUp CRX.DAM.IO
CRX.DAM.IO Shows:

"user_count": "10000000"

AVG Web TuneUp CHROME.GOOGLE.COM
Official Chrome Store Shows:

7,669,478 users

Thanks, I only indexed it until it was removed from the sitemap (extensions owner can choose to have their extension listed or not).
You can see it’s unlisted because it’s not available in the google results for example.
What I should do in fact is to continue to index the extensions after they are unlisted, it would solve a few other issues too (like showing if an extension was deleted or not)


Fix testcontainers/Docker incompatibility

Docker for Mac issued a new release that is incompatible
with the org.testcontainers version 1.14.3 library.  This
commit updates to testcontainers version 1.15.0-rc2, released
today, and fixes the consequences of that upgrade.
Here is the backtrace you'll see before these changes when you run a product test:
2020-10-01T14:16:30.173-0700	INFO	main	org.testcontainers.utility.RegistryAuthLocator	Credential helper/store (docker-credential-desktop) does not have credentials for index.docker.io
2020-10-01T14:16:30.629-0700	WARN	testcontainers-ryuk	org.testcontainers.utility.ResourceReaper	Can not connect to Ryuk at localhost:32768
java.net.SocketException: Broken pipe (Write failed)
	at java.base/java.net.SocketOutputStream.socketWrite0(Native Method)
	at java.base/java.net.SocketOutputStream.socketWrite(SocketOutputStream.java:110)
	at java.base/java.net.SocketOutputStream.write(SocketOutputStream.java:138)
	at org.testcontainers.utility.ResourceReaper$FilterRegistry.register(ResourceReaper.java:424)
	at org.testcontainers.utility.ResourceReaper.lambda$null$1(ResourceReaper.java:152)
	at org.rnorth.ducttape.ratelimits.RateLimiter.doWhenReady(RateLimiter.java:27)
	at org.testcontainers.utility.ResourceReaper.lambda$start$2(ResourceReaper.java:136)
	at java.base/java.lang.Thread.run(Thread.java:834)

Actually, even with these changes, testcontainers doesn't start up correctly, though the docker containers do start up.
I'm closing this PR since it doesn't actually fix the problem of allowing folks to run product tests on Macs.

I'm closing this PR since it doesn't actually fix the problem of allowing folks to run product tests on Macs.

  Tests run again after downgrading from Docker Desktop <IP_ADDRESS> to <IP_ADDRESS>, found on this page: https://docs.docker.com/docker-for-mac/release-notes/

@djsstarburst I was hit by that recently as well and downgraded Docker. I'll upgrade testcontainers and adjust our product tests when it's released.

has this been reported to testcontainers already? cc @bsideup

@findepi yes, see https://github.com/testcontainers/testcontainers-java/issues/3166


Return value must be of type App\Controllers\Response, Framework\Response

hello
i'm tryng to start the project but i'm getting this error before or after putting the database info in .env
i'm using php latest is that it?

add this in file src/App/Controllers/Home.php
use Framework\Response;

@vedastudio Thanks for this, I just updated the repo to include it.

add this in file src/App/Controllers/Home.php use Framework\Response;

it works, thanks!


HIVE-17561

Moved TxnStore and its implementations.  Moved DataSourceProvider as well.  Moved ValidTxnList and ValidReadTxnList to storage-api as ORC and the standalone metastore will both need them.  Also moved the newly created SQLGenerator.
I did not move the functionality that added bonecp and hikari to the restricted list in HiveConf because MetastoreConf doesn't have a restricted list.  MetastoreConf doesn't have a restricted list because that functionality is broken.  setVar* doesn't check against the restricted list.  Given that MetastoreConf is a set of static convenience methods to be used on top of Configuration we can't stop users from going underneath and calling set() directly anyway.  If we want to restrict users from setting certain values in the MetastoreConf we need a comprehensive and secure way to do it.  Plus I'm not convinced that boncep and hikari settings are high security issues anyway.

As part of the latest change I moved TxnDbUtil to use a passed in Configuration file rather than generate a new one on its own every time.  This is more efficient and avoids errors where users have specifically set values in the conf file that they want reflected in the TxnStore interactions.


Add symlink "prm" to the application for practical usage

I don't really know how to do this, can you help ?

v1.5.0 fixes i


[BEAM-2421] Replaces BoundedSource with a composite transform when using Fn API

https://issues.apache.org/jira/browse/BEAM-2421

Run Dataflow ValidatesRunner

Run Dataflow ValidatesRunner

Run Dataflow ValidatesRunner

Ping.

these ValidatesRunner failures seem to be infrastructure failures. Let's try again.

Run Dataflow ValidatesRunner


Fix app best tip merger for berkeley

Commit cd4116536c2db68de4c8d68ee3abe2f4ccc7ba7d is the minimal fix for the app.
Other commits are cleanups.

!ci-build-me

Merge on berkeley #15522


IP field via MultrangeQuery fix #16200

Description
Combines a many concrete IPs and CIRR masks into set when querying IP field
Related Issues
Resolves #16200

Check List

[ ] Functionality includes testing.
[ ] API changes companion pull request created, if applicable.
[ ] Public documentation issue/PR created, if applicable.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

it's an alt of #16202

Here we come to the question. Think about terms_query for IP/masks (this what I want to do here).

num of masks
index-only
docVals-only
index&dvs

<maxClauses
MultiRangeQ
BQ { ssDvRangeQ, ..}
MultiRangeQ && BQ{ssDvRangeQ,..}

>maxClauses
MultiRangeQ
exception (until ssDvMultiRange)
MultiRangeQ (no chance for ssDvRangeQ)

The first question is to confirm this plan. So, WDYT?
Then, notice <maxClauses condition, the exact number is a question. IpFieldMapper parses terms_query only. But it has no information about number of enclosing BQs, even think about top level bool. Its' clauses are counted under overall maxClause limit. It seems like if we go with numclause condition logic we can't reasonably set the limit.
I don't want to introduce settings since the problem should gone after introducing ssDvMultiRange. The compromise is to set to something like <IndexSearcher.getMaxClausesCount()/2.
Wdyt?

Let's discuss an edge case, which are not obvious:
doc_values only field with mask/ values
In this case we can just create BooleanQuery { ssDvRangeQuery, ...}. It will lately may fail with too many clauses, and it's reasonable. Note: it will be a way better with ssDvMultiRange
index & doc_values field with mask/ values

strawman approach: forget about dv and IndexOrDvQuery, and just create MultiRangePointQuery
attempt to create IndexOrDvQuery(MultiRangePointQuery, BooleanQuery { ssDvRangeQuery, ...}) when number of BQ is less than MaxClauses limit.

The problem is that this boundary limit can't be decided on query parsing because sibling filter clauses may exceed MaxClauses limit.

Hi there!
Kindly asking for review. And tagging as backport.
Thanks!

In this case we can just create BooleanQuery { ssDvRangeQuery, ...}. It may lately fail with too many clauses, and it's reasonable. Note: it will be a way better with ssDvMultiRange

Just a question, since with this change we still don't solve the problem completely (by and large, the limit might be hit anyway), is it worth the effort?

is it worth the effort?

I think it is: it unblocks large terms_queries for indexed fields right now. Regarding doc_values solution: Lucene#13974, I'm not sure if I'm able to complete it and how long it take.

LGTM, thanks @mkhludnev ! @msfroh anything from your side?

Thank you so much, @reta @msfroh!


Spanish store entry

Hi @Futsch1 , I added some comments in the review. Just sending this because I'm not sure if you can get notified because of a review when a PR is closed

You're right! I didn't publish it. Here it is ;)

Yery much appreciated, just sent the changes to the store for approval!


Bluetooth - peripheral_dis - settings_runtime_set not working

Describe the bug
Hi, I would like to change DIS model name, manufacturer etc from code, I tried to change them in peripheral_dis main.c but there is no change when scanned.
To Reproduce
keep
CONFIG_BT_GATT_DIS_MODEL="Zephyr Model"
in prj.conf
change "Zephyr model" to "Zephyr model mod" in main.c on lines 52+53.
build and flash
Expected behavior
when you read DIS model characteristic, it still contains "Zephyr model", not "Zephyr model mod"
Impact
For me, if example is not working as expected, I don't know where to look for problem in my own code
Environment (please complete the following information):

board: nrf52840_pca10040
OS: W7
Toolchain: GNU ARM Embedded 9-2019-q4
Commit SHA or Version used: f21aaeffaa5ed10629915a9d198ff483865f7be9

CC @jhedberg

Are you sure you have settings enabled? I don't see CONFIG_BT_SETTINGS=y in the peripheral_dis prj.conf.

Thanks,
adding CONFIG_BT_SETTINGS=y  and using settings_load() helped


TOC is not generated

Only shows the title "Table of Contents"
Logseq version : 0.8.9

Please try the latest version.


Suggested level: "Remote Rejected!"

Very neat tool thank you for all your great work!
Here is a level suggestion on how to deal with a remote that has the master branch locked because a pull request system is in place. I have found that this is a common beginner problem where people can get stuck. I wrote this up really quick after I made this mistake just now myself and realized it would make a good level so no problem if we think I should adjust any of my grammar.
Gist: https://learngitbranching.js.org/?gist_level_id=6a4849cb3aa6c3f892d44d0d535a9d83

This is a very well-written level and totally a situation someone new to PR's could get into! I also liked how you moved the master branch up so they are starting after their mistake.
I went ahead and incorporated this into the curriculum if that's ok 😇 I changed the name of the level slightly but otherwise it's as-is.
Thanks for the awesome contribution!

Should be available now:
https://learngitbranching.js.org/


[cluster-test] disable fullnode reboot in compat test

Motivation
We're currently facing issues with fullnode networking configuration, especially in joining the right networks after reboot. And it will take some time before cluster-test starts using the management tool.
Circumvent this issue for now by disabling fullnode upgrade in compat tests. Instead, we'll reset the cluster as the final step of compat test if we set DISABLE_FN_UPGRADE environment variable. Some changes have been made to include ClusterBuilder and ClusterBuilder params in experiment Context, as well as to setup_cluster to optionally skip scale up/down asg and wipe data.
Have you read the Contributing Guidelines on pull requests?
Yes
Test Plan
Build image off PR and run cti:
$ PREV_TAG="land_$(git rev-parse --short=8 origin/master)"
$ TEST_TAG="dev_rustielin_pull_5264"
$ ./scripts/cti -T $PREV_TAG --cluster-test-tag $TEST_TAG \
    -E RUST_BACKTRACE=1  -E RUST_LOG=debug -E UPDATE_TO_TAG=$TEST_TAG -E DISABLE_FN_UPGRADE=1 \
    --report cluster-reset.json --suite land_blocking_compat > cluster-reset.txt

...

Compatibility test results for land_ab53854f ==> dev_rustielin_pull_5264 (PR)
1. All instances running land_ab53854f, generating some traffic on network
2. First validator land_ab53854f ==> dev_rustielin_pull_5264, to validate storage
3. First batch validators (14) land_ab53854f ==> dev_rustielin_pull_5264, to test consensus
4. Second batch validators (15) land_ab53854f ==> dev_rustielin_pull_5264, to upgrade rest of the validators
5. Reset all nodes ==> dev_rustielin_pull_5264, to finish the network upgrade
all up : 1123 TPS, 4028 ms latency, 4700 ms p99 latency, no expired txns
**********
Logs snapshot: http://kibana.ct-0-k8s-testnet.aws.hlw3truzy4ls.com/app/kibana#/discover?_g=(time:(from:'2020-07-27T17:41:53Z',to:'2020-07-27T17:53:32Z'))
Dashboard snapshot: http://grafana.ct-0-k8s-testnet.aws.hlw3truzy4ls.com/d/2XqUIhnWz/performance?from=1595871713000&to=1595872412000
**********

Related PRs
(If this PR adds or changes functionality, please take some time to update the docs at https://github.com/libra/website, and link to your PR here.)

:umbrella: The latest upstream changes (presumably d682826966a8cd6cda02160ad07c38cac66ceab6) made this pull request unmergeable. Please resolve the merge conflicts.

:umbrella: The latest upstream changes (presumably fb597f71ab448a1f64eb3fe4fdb1503c24bc63d6) made this pull request unmergeable. Please resolve the merge conflicts.

Updated to reflect upstream changes that added new ClusterSwarm::clean_data. Test results:
Compatibility test results for land_43d67875 ==> dev_rustielin_pull_5264 (PR)
1. All instances running land_43d67875, generating some traffic on network
2. First validator land_43d67875 ==> dev_rustielin_pull_5264, to validate storage
3. First batch validators (14) land_43d67875 ==> dev_rustielin_pull_5264, to test consensus
4. Second batch validators (15) land_43d67875 ==> dev_rustielin_pull_5264, to upgrade rest of the validators
5. Reset all nodes ==> dev_rustielin_pull_5264, to finish the network upgrade
all up : 1151 TPS, 3928 ms latency, 4900 ms p99 latency, no expired txns
**********
Logs snapshot: http://kibana.ct-0-k8s-testnet.aws.hlw3truzy4ls.com/app/kibana#/discover?_g=(time:(from:'2020-07-27T22:18:29Z',to:'2020-07-27T22:30:43Z'))
Dashboard snapshot: http://grafana.ct-0-k8s-testnet.aws.hlw3truzy4ls.com/d/2XqUIhnWz/performance?from=1595888309000&to=1595889043000
**********

@bors-libra r=andll

:pushpin: Commit aa11e4e has been approved by andll

:hourglass: Testing commit aa11e4e73e3a35c6ea00fd71af5965f3429c4491 with merge eb9402cfa9ecd1ec0457e552d3d93889c075ba9c...

:sunny: Test successful - checks-actions_land_blocking_test, checks-circle_commit_workflow
Approved by: andll
Pushing eb9402cfa9ecd1ec0457e552d3d93889c075ba9c to master...


MRTK3 Far Ray Cursor not recognizing spatial map

Describe the bug
Building for MRTK3 on Hololens 2.  When trying to interact with the spatial mapping mesh to place a hologram, the far ray cursor does not show up and the far ray does not recognize it is hitting the spatial mapping mesh.
Expected behavior
Far ray cursor should appear when far ray hits spatial mapping mesh.
Your setup (please complete the following information)

Unity Version 2021.3.4f1
MRTK Version 3

Target platform (please complete the following information)

HoloLens 2

Thanks for reporting this @astaikos316! Have you checked what layer your spatial mesh is on, vs the layermask on the ray interactor?

@astaikos316 , closing as resolved assuming the layermask change fixed your issue. If you're still running into issues, please reopen! We'll make sure the layermask is configured by default if it isn't already.

I also had to add a Base Interactable Component to the Spatial Map prefab in order for it to work.  Sorry for not reporting sooner.

Hi @astaikos316 thanks for your reply. I have added the Base Interactable on Spatial Mesh and it is working as expected. Now I need to know how to get the ray hit position on the spatial map to instantiate an object on the far ray hit point. Can you help me on that?

See other issue response @ragavendranbala !


Using Isaac sim to reproduce your code.

Hi,yuzhe!
I am using Isaac Sim to reproduce your code.But I encountered a problem during the reproduction process.I really hope you can give me some help to solve this problem. The problem mainly arises in the calculation of Jacobian matrices for specific links. The code is as follows:

            jacobians = []

            jacobians_all = self.robot.articulation_view.get_jacobians()[0]

            for i , index in enumerate(self.target_link_indices):
                link_jacobian = jacobians_all[index][:3, self.target_joint_indices]
                jacobians.append(link_jacobian)
            jacobians = np.stack(jacobians, axis=0)

            huber_distance.backward()
            grad_pos = torch_body_pos.grad.cpu().numpy()[:, None, :]
            grad_qpos = np.matmul(grad_pos, np.array(jacobians))
            grad_qpos = grad_qpos.mean(1).sum(0)
            grad_qpos += 2 * self.norm_delta * (x - last_qpos)
            grad[:] = grad_qpos[:]

I use the API 'articulation_view.get_jacobians()' and the  shape of returned result is (33,6,30). I tried slicing it to obtain the Jacobian matrix for a specific link. But I found that the result calculated by your code is inconsistent with the result returned. Given the position of the joint, how do I calculate the Jacobian matrix for a specific body in Isaac Sim? I really hope you can give me some help to solve this problem.

Hi @SshiJwu ,
I'm not deeply versed in IsaacSim, yet from my understanding and previous exposure to IsaacGym, I suspect that the discrepancy you're observing between SAPIEN and IsaacSim stems from their distinct interpretations of the Jacobian. In SAPIEN, the Jacobian is related to the spatial twist, whereas in Isaac, it pertains to cartesian velocity combined with angular velocity. Typically, when rotational dynamics are negligible, these Jacobsian representations might be seen as interchangeable. However, complications can arise due to the varying joint sequencing as interpreted by different URDF parsers. It's crucial to ensure that you maintain a consistent joint order across your various applications.
Moreover, I'm intrigued as to why there's a necessity to recode for IsaacSim. Should you be utilizing IsaacSim for simulations, it's feasible to apply our existing codebase without alterations, despite the embedded SAPIEN simulator. The retargeting code we've developed is capable of calculating the necessary joint angles, which you can then input directly into the IsaacSim environment. I'm of the opinion that redeveloping this codebase to employ an alternative Jacobian computation offers no clear advantage.

@SshiJwu
Regarding your query about the out-of-bound value, it seems likely that the problem arises from the last_qpos input in self.optimize(objective_fn, last_qpos) being beyond the allowable range for that particular joint. This could result from an unsuccessful optimization process in the previous iteration, potentially due to alterations in the Jacobian calculation within the original codebase.

Yes. The order of the joints returned by the Isaac SIM is inconsistent with that of SAPIEN.
In Isaac Sim, it is ['WRJ1', 'FFJ4', 'LFJ5', 'MFJ4', 'RFJ4', 'THJ5', 'WRJ2', 'FFJ3', 'LFJ4', 'MFJ3', 'RFJ3', 'THJ4', 'FFJ2', 'LFJ3', 'MFJ2', 'RFJ2', 'THJ3', 'FFJ1', 'LFJ2', 'MFJ1', 'RFJ1', 'THJ2', 'LFJ1', 'THJ1'], but ['WRJ2', 'WRJ1', 'FFJ4', 'FFJ3', 'FFJ2', 'FFJ1', 'MFJ4', 'MFJ3', 'MFJ2', 'MFJ1', 'RFJ4', 'RFJ3', 'RFJ2', 'RFJ1', 'LFJ5','LFJ4','LFJ3','LFJ2','LFJ1','THJ5','THJ4','THJ3','THJ2','THJ1'] in SAPIEN.
I changed the code a bit,as follows:

def _get_objective_function(self, target_pos: np.ndarray, fixed_qpos: np.ndarray, last_qpos:np.ndarray):
    qpos = np.zeros(self.robot_dof)
    qpos[self.fixed_joint_indices] = fixed_qpos
    torch_target_pos = torch.as_tensor(target_pos)
    torch_target_pos.requires_grad_(False)

    def objective(x: np.ndarray, grad: np.ndarray):
        qpos[self.target_joint_indices] = x

        all_frame_names = self.kinematics_solver.get_all_frame_names()
        target_frame_name_index = [self.kinematics_solver.get_all_frame_names().index(body_name) for body_name in self.body_names]

        positions = []
        base_jacobian = []
        for index in target_frame_name_index:
            position , rotation  = self.kinematics_solver.compute_forward_kinematics(all_frame_names[index],qpos)
            base_frame_name = self.kinematics_solver.kinematics.jacobian(qpos,all_frame_names[index])[:3,:]
            positions.append(position)
            base_jacobian.append(base_frame_name)
        body_pos = np.array(positions)

        torch_body_pos = torch.as_tensor(body_pos)
        torch_body_pos.requires_grad_()

        huber_distance = self.huber_loss(torch_body_pos , torch_target_pos)
        result = huber_distance.cpu().detach().item()

        if grad.size>0:

            jacobians = np.stack(base_jacobian, axis=0)

            huber_distance.backward()
            grad_pos = torch_body_pos.grad.cpu().numpy()[:, None, :]

            grad_qpos = np.matmul(grad_pos, np.array(jacobians))
            grad_qpos = grad_qpos.mean(1).sum(0)
            grad_qpos += 2 * self.norm_delta * (x - last_qpos)
            grad[:] = grad_qpos[:]

        return result

    return objective

In this case, I looked at the calculation of the Jacobian matrix, and it is almost unanimously possible

Consider this library as an opaque utility that carries out the necessary retargeting calculations for your application. Once you receive the retargeting data from this library, you can readily apply it to simulations or visualizations within IsaacSim. There's no need for you to delve into the computational details managed by SAPIEN.
Regarding the Jacobian, I'm not certain of the specific definition used in Isaac Sim. The function compute_single_link_local_jacobian calculates the spatial twist Jacobian for a given link as defined in the URDF. It produces a matrix with the dimensions 7*n, where n represents the degrees of freedom (DOF) of the robot.

Thanks,yuzhe!

I also have a question about the left and right hands. What is the difference between the left and right dexterous  hand URDF? Can I retarget the data from the left hand to the URDF of the right hand.

I've found that if I retarget the data from the left hand to URDF of the right hand, I get a big shift between the hand and the object.

Why did I change the hand.type of DexYCB to left? How can the relative position of the hand and the object be so different after rendering?

It does not make sense to retarget the motion from right hand to left hand due to the huge morphology gap. Always retargeting left human hand motion to a left hand robot and vice versa.

There is another question that has been bothering me for a long time, in order to allow the dexterous hand to move freely in space, you added six free joints.  The translation_range is (-1,1),and the rotation_range is (-pi,pi). Why is the range of translation this? Since space is infinite, I can use my dexterous hands to reach anywhere in the space.

You can tune this value based on your applications. For example, you can use a very large limit for translation to make the hand more freely inside the space. The (-1, 1) is an example of motion range, there are no specific meaning of these numbers,


error dbadmin

hi, i have a problem to access to the url dbadmin . The service is working but i can´t access. Im running the last image of vestacp.

i check this for my main domain and its working great! but whit my others domain give a error like not found the page.

@patsotoe This was done on purpose for security.  You've ran website and you can probably agree that one of the biggest scan on the internet for website is phpmyadmin or any kind of dbadmin.  This will also slow down your server.  To enable it, simply put a "conf" file in your private directory.
echo "include /etc/nginx/conf.d/dbadmin.inc;" > /home/{user}/web/{website}/private/dbadmin.conf
nginx -t
nginx -s reload

This is because my nginx config will read anything with extension "conf" in the private folder.

thanks por the explanation.


State changes, slideOver rendering, auth bug fix

Description
This PR deals with the Following:
Board State
State Restructuring : The core board state utilized within the Kanban board has been centralized within the boardContext. Instead of managing individual states scattered across components, this centralisation enables updates to the board state to trigger re-renders in components consuming this context.
Improved Reusability: By consuming the context, the Kanban board components can re-render with updated state values, enhancing reusability and maintainability.
Here is some reading on state structure:https://react.dev/learn/managing-state#choosing-the-state-structure

Workspace Slideover
Dedicated Features Folder: Introduced a dedicated features folder for the Slideover feature. This change separates Slideover-related components making it easier to read our code(imo)
Breakdown components: The Slideover has been decomposed into individual components, including Priority Display, Task Description, and Edit Action Buttons. This breakdown enhances code readability and separation of concerns.
Integration into Kanban Board: Moved the Workspace Slideover into the Kanban board component.
Added key prop: The addition of the key prop to the Workspace Slideover ensures that the state inside the Slideover resets for each distinct task rendered.
Bug: There is still a bug with the Slideover transition(on enter). I believe this is caused from adding the key to the slideover and only occurs when selectedTaskId changes.

Authentication
Bug Fix: Addressed an authentication bug related to login. The issue stemmed from an attempt to access the user property of the response object when it was null. This bug has been resolved.

Backend Task Controller
Change of response: In response to user task edits in the frontend, the backend now returns the updated task. This allows us to put add the updated task to state.

[x] Is this related to a specific issue? Is so please specify: #111 #126

Type of Changes
Use a check for ✓ the following type of changes:

Type

✓
:bug: Bug fix

✓
:sparkles: New feature

✓
:hammer: Refactoring

:100: Add tests

:link: Update dependencies

:scroll: Docs

Updates
Before
[If applicable, include a picture of the behavior before the changes were made. I.e. a screenshot of the behavior before the changes were made.]
After
Updated State In the Board Context: Have pulled state upfrom the individual components, and added to the context.

Testing Steps / QA Criteria

Verify that the board state is correctly managed within the boardContext and that changes to the state trigger the re-rendering of components using this context.
Test the Kanban board's working functionality , such as moving, or editing tasks, to ensure that it works as expected.
Confirm that the Slideover feature is now organized in a dedicated folder and that all components within it function correctly.
Check that the Slideover's state is changed when rendering different tasks.

Checklist:

[x] This PR is up to date with the development branch, and merge conflicts have been resolved
[x] My code follows the style guidelines of this project
[ ] I have executed npm run lint and resolved any outstanding errors.
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] My changes generate no new warnings

I think generally looks good. As mentioned, I think it's worth just putting your prettier settings in the repo so that we reduce the number of formatting changes between each PR as it has gotten quite large.

We have a prettier config file, it seems perhaps its using my editor is using my vsCode setting instead of that file.

We have had a prettier config from the start.  I'm not sure what is happening since formatting is successfully happening on my end?


Errors in models for API and ReLU

Hi,
I'm trying to got through an example using a simple API keras model. I've loaded the model in e.g.,
deeplift_model = kc.convert_model_from_saved_files('Model.h5',
nonlinear_mxts_mode=NonlinearMxtsMode.RevealCancel)
Now I'm attempting to get to the next step to compile the functions (compile_func) and check everything is the same vs the standard keras model. I can't seem to find the list of layers:

deeplift_model.get_name_to_blob().keys()
Traceback (most recent call last):
File "", line 1, in
AttributeError: 'GraphModel' object has no attribute 'get_name_to_blob'

I seem to be able to compile the function via:
deeplift_prediction_func = compile_func([deeplift_model.get_name_to_layer()["input_1_0"].get_activation_vars()], deeplift_model.get_name_to_layer()["preact_dense_1_0"].get_activation_vars())
where
deeplift_model.get_name_to_layer().keys()
odict_keys(['input_1_0', 'preact_conv1d_1_0', 'conv1d_1_0', 'max_pooling1d_1_0', 'dropout_1_0', 'flatten_1_0', 'preact_dense_1_0', 'dense_1_0', 'preact_dense_2_0', 'dense_2_0', 'preact_dense_3_0', 'dense_3_0'])
However, when I try to call get_target_contribs_func
revealcancel_func = deeplift_model.get_target_contribs_func(find_scores_layer_name="input_1_0", pre_activation_target_layer_name="preact_conv1d_1_0")
I get told off about the final activation function:
Traceback (most recent call last):
File "", line 1, in
File "/Users/christopher_penfold/Desktop/Code/deeplift/deeplift/models.py", line 113, in get_target_contribs_func
return self._get_func(*args, func_type=FuncType.contribs, **kwargs)
File "/Users/christopher_penfold/Desktop/Code/deeplift/deeplift/models.py", line 269, in _get_func
**kwargs)
File "/Users/christopher_penfold/Desktop/Code/deeplift/deeplift/models.py", line 47, in _get_func
self._set_scoring_mode_for_target_layer(target_layer)
File "/Users/christopher_penfold/Desktop/Code/deeplift/deeplift/models.py", line 182, in _set_scoring_mode_for_target_layer
+final_activation_type)
RuntimeError: Unsupported final_activation_type: ReLU
Sure I'm doing something obvious wrong somewhere ... any pointers in the right direction would be appreciated.

Hi Christopher,
You’re fine, the issue is with deeplift.
(1) get_name_to_blob is indeed deprecated to get_name_to_layer. Where did you encounter get_name_to_blob? I thought I had updated the README, evidently i missed something somewhere.
(2) Some context about the second error you are getting: traditionally, people use sigmoid or softmax outputs when doing classification, and linear outputs when doing regression. In the case of the former, DeepLIFT computes the contribution wrt the linear activation preceeding the final nonlinearity to avoid the saturation effects of the sigmoid/softmax. In the case of a linear output, DeepLIFT computes contributions wrt to the provided linear layer. You are trying to compute contributions to a linear layer that is followed by a ReLU nonlinearity. I can add this case in (would need to clarify how you want to handle ReLU saturation), but I also note that you are evidently computing contributions to neurons in the first convolutional layer. Is that actually the layer you are interested in? And do you care about interpeting individual neurons in the conv layer, or the activation of the entire layer as a whole? If you care about interpreting every individual neuron in the conv layer, you might find it easier to compute the contributions using a series of numpy matrix multiplications (which is tractable to implement for just a single layer; I can describe how to do this if that is in fact what you are after). DeepLIFT is currently set up to compute contributions to one target output neuron at a time, so it would be cumbersome to perform DeepLIFT for every individual neuron in the conv layer.

Yes, I think sigmoid/softmax outputs are def more appropriate for classification if trained with binary/categorical crossentropy losses.
I think the error you got is caused by using an old version of tensorflow. Can you upgrade to tensorflow version 1.7 or later?

Ah! Of course facepalm 😳Seems to work now, will continue going through, but I think we can call this issue closed


Java: gradle generated pom.xml files in build folder should be ignored

Context: gax-java
After running some gradle commands, you may have a pom.xml file in the build directory. This directory is in the gitignore file and git add will fail saying that the file is ignored. releasetool then stops execution.

Workaround, clean the workspace first by running git clean -fdx first or ./gradlew clean before running releasetool.

@chingor13 Is this still relevant?

It still happens but the workaround is ok for now.

release-please handles opening Java release PRs.


[Story] Filter challenges by organization

Enable to filter challenges by organization.
Tasks

[x] Update API description to filter challenges by contributors
[x] Update the challenge service to filter challenges by contributors
[ ] Update the client library for Angular
[ ] #1402
[ ] #1421

The Challenge service needs to get information from the Organization service.

Netflix Feign and Spring RestTemplate (along with Ribbon) are two well-known HTTP clients used for making synchronous API calls. In this tutorial, we will use Feign Client.

Feign already uses Ribbon, so, if you use @FeignClient, this section also applies.

Update 2023-03-19
Remaining tasks include:

[ ] Update API description to filter challenges by contributors
[ ] Update the challenge service to filter challenges by contributors
[ ] Update the client library for Angular
[ ] Update the web app to filter challenges by orgs

Update 2023-03-27
The backend of this feature is ready. The frontend behavior is being modified so that typing text in the filter input fields triggers a full-text search using the backend instead of 1) pulling all the results and 2) filtering then at the level of the web app. This will make the behavior of the search input field more consistent across the web app.


update display name when update legal name

Explanation of Change

Fixed Issues

$ https://github.com/Expensify/App/issues/52900
PROPOSAL: https://github.com/Expensify/App/issues/52900#issuecomment-2491151162
Tests

Create a new account
In onboarding modal select 'Manage my team's expenses' and finish onboarding flow
Go to Profile, add a legal name
Make sure the Display Name is changed

[x] Verify that no errors appear in the JS console

Offline tests

Same as tests
QA Steps

// TODO: These must be filled out, or the issue title must include "[No QA]."
Same as tests

[x] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I used JaimeGPT to get English > Spanish translation. I then posted it in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is either coming verbatim from figma or has been approved by marketing (in order to get marketing approval, ask the Bug Zero team member to add the Waiting for copy label to the issue)
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.ts or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(theme.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If the PR modifies the UI (e.g. new buttons, new UI components, changing the padding/spacing/sizing, moving components, etc) or modifies the form input styles:

[x] I verified that all the inputs inside a form are aligned with each other.
[x] I added Design label and/or tagged @Expensify/design so the design team can review the changes.

[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] I added unit tests for any new feature or bug fix in this PR to help automatically prevent regressions in this user flow.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.

Screenshots/Videos

Android: Native

https://github.com/user-attachments/assets/1a345a01-a74f-4507-afb5-af4beee65b7f

Android: mWeb Chrome

https://github.com/user-attachments/assets/ecdc49ca-2317-4676-a22b-c2596b9b8c59

iOS: Native

https://github.com/user-attachments/assets/da5fd654-71fb-4a33-851c-1b96f22e4b66

iOS: mWeb Safari

https://github.com/user-attachments/assets/1c72e233-e4c6-4fa3-b6d6-3c80b79a6649

MacOS: Chrome / Safari

https://github.com/user-attachments/assets/e97ee4a6-3085-4d0c-9d70-359174ed476a

MacOS: Desktop

https://github.com/user-attachments/assets/1eedcab5-f123-4e9d-a5c6-b267972eba32

@Themoonalsofall Most of the new accounts are prompted with onboarding and most of the options there require entering first name, so it might hard to test as we need fresh account without any name entered. But choosing Manage Team option in onboarding does not require entering name so we can test using this option.
Please update testing steps appropriately so that QA team is not confused

Working on it

LGTM but seems there are some eslint issues.

Still happening

Seems ready to be merged

Done

Still not passing.

@Themoonalsofall This is a new eslint rule causing this. Not sure what is the next step here, I have asked it in slack.


Clarify URL requirements

Relates to the discussion in https://github.com/APIs-guru/graphql-over-http/issues/91
@ghmcadams  @ralfhandl

Thanks for your suggestions so far. I expanded the text to be more explicit about the aspects you mentioned.
While i do think the spec should be precise, i would rather explain things a bit more in depth than to aim for maximum conciseness.

I incorporated feedback from the comments and the last working group meeting. If there is nothing seriously wrong, we can merge this for now to allow fine-grained iteration later.

I figured we can just merge for now and take any future modifications in a future PR.


serverless deployment on Vercel

Hi, I am trying to deploy pgweb on Vercel's GO serverless platform with the following directory structure
| - api
|    └ handler.go
| - go.mod

where go.mod is copied from this repo root, and the content of handler.go as follow:
package handler

import (
  "github.com/sosedoff/pgweb/pkg/cli"
)

func Handler(w http.ResponseWriter, r *http.Request) {
  cli.Run()
}

during the build process, I got the following error:
[20:06:21.971] Retrieving list of deployment files...
[20:06:22.467] Previous build caches not available
[20:06:24.513] Downloading 2 deployment files...
[20:06:24.910] Not using Build Cache
[20:06:24.949] Running "vercel build"
[20:06:25.443] Vercel CLI 28.4.14
[20:06:37.621] failed to `go mod tidy`
[20:06:37.678] Error: Command failed: go mod tidy
[20:06:37.678] go: finding module for package github.com/sosedoff/pgweb/pkg/cli
[20:06:37.678] go: finding module for package github.com/vercel/go-bridge/go/bridge
[20:06:37.678] go: downloading github.com/vercel/go-bridge v0.0.0-20221028184743-6acb43c46b6a
[20:06:37.678] go: found github.com/vercel/go-bridge/go/bridge in github.com/vercel/go-bridge v0.0.0-20221028184743-6acb43c46b6a
[20:06:37.678] go: finding module for package github.com/aws/aws-lambda-go/lambda
[20:06:37.678] go: finding module for package github.com/sosedoff/pgweb/pkg/cli
[20:06:37.679] go: finding module for package github.com/aws/aws-lambda-go/events
[20:06:37.679] go: downloading github.com/aws/aws-lambda-go v1.34.1
[20:06:37.679] go: found github.com/aws/aws-lambda-go/events in github.com/aws/aws-lambda-go v1.34.1
[20:06:37.679] go: found github.com/aws/aws-lambda-go/lambda in github.com/aws/aws-lambda-go v1.34.1
[20:06:37.680] go: downloading github.com/stretchr/testify v1.7.2
[20:06:37.680] go: downloading gopkg.in/yaml.v3 v3.0.1
[20:06:37.680] go: downloading github.com/davecgh/go-spew v1.1.1
[20:06:37.680] go: downloading github.com/pmezard/go-difflib v1.0.0
[20:06:37.680] go: finding module for package github.com/sosedoff/pgweb/pkg/cli
[20:06:37.680] github.com/sosedoff/pgweb/api imports
[20:06:37.680] 	github.com/sosedoff/pgweb/pkg/cli: no matching versions for query "latest"
[20:06:37.680]

It failed during go mod tidy and complaining no matching version of "latest" while importing github.com/sosedoff/pgweb/pkg/cli
I am new in GO lang, and have no idea on how to fix it... any help will be appreciated 😊
thanks.

Im 99% sure you wont be able to make pgweb work with serverless platforms. Pgweb is a standalone application that requires configuration (ie needs postgres endpoint/credentials) before it could be used normally.
What is your goal?

I just try to deploy pgweb on any web platform with generous free tier. I used to deploy other stuff on Vercel so give it a try.
As you said pgweb is not suitable for serverless platforms so I switched to https://fly.io usng the docker image. It works great with a small memory footprint within the free tier 👍


Getting some error

Anybody know why this is happening?
Trying to run a Discord bot on heroku.com with this module, keep getting this error though.
       ><EMAIL_ADDRESS>install /tmp/build_9b205b212e8f5f2978840fe5b0e0ee86/node_modules/better-sqlite3
       > node deps/install

       ==> cwd: /tmp/build_9b205b212e8f5f2978840fe5b0e0ee86/node_modules/better-sqlite3
       ==> /tmp/build_9b205b212e8f5f2978840fe5b0e0ee86/node_modules/lzz-gyp/lzz-compiled/linux -hx hpp -sx cpp -k BETTER_SQLITE3 -d -hl -sl -e ./src/better_sqlite3.lzz
       /bin/sh: 1: /tmp/build_9b205b212e8f5f2978840fe5b0e0ee86/node_modules/lzz-gyp/lzz-compiled/linux: Permission denied
       Error: exit code 126
       at ChildProcess.<anonymous> (/tmp/build_9b205b212e8f5f2978840fe5b0e0ee86/node_modules/lzz-gyp/lib/exec.js:19:12)
       at emitTwo (events.js:126:13)
       at ChildProcess.emit (events.js:214:7)
       at Process.ChildProcess._handle.onexit (internal/child_process.js:198:12)
       npm ERR! code ELIFECYCLE
       npm ERR! errno 1
       npm ERR<EMAIL_ADDRESS>install: `node deps/install`
       npm ERR! Exit status 1
       npm ERR!
       npm ERR! Failed at the<EMAIL_ADDRESS>install script.
       npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

       npm ERR! A complete log of this run can be found in:
       npm ERR!     /tmp/npmcache.BQWtR/_logs/2018-05-29T16_17_40_078Z-debug.log
-----> Build failed

The build system has now been overhauled to address the many problems people have had trying to install better-sqlite3. Try upgrading to v4.1.2. If the problem persists, comment here and I'll reopen the issue.


Fix the problem Monitoring - app and namespace are left when deleting the project

Problem:
Monitoring - app and namespace are left when deleting the project
Solution:
If the namespace's creatorID is null, it wouldn't remove the namespace when deleting the project.
https://github.com/rancher/rancher/blame/fa26eccdb1ec6d9065877340f6b90ef374969adf/pkg/controllers/user/rbac/project_handler.go#L66
so add parameter creatorID for function EnsureAppProjectName can fix it.
Related Issue:
https://github.com/rancher/rancher/issues/23848

Does it work in upgrade scenario?

Does it work in upgrade scenario?

The old Monitoring - namespace's creatorID is still null, namespace are left when deleting the project.
The new Monitoring - namespace will be remove when we do the same

Can this be updated to use an owners reference? Tie the app and namespace to the project, if the project gets deleted everything goes with it

An owning object must be in the same namespace as the dependent or be cluster-scoped.
And the problem here is that the app object is gone but the related namespace and resources are left in the user cluster.


Corrections and adjustments in grammar (Croatian language)

LOG:
The word "prošli" has been substituted with "protekli" which is considered more grammatically accurate.
The word "dobijaš" has been substituted with "dobivaš" which is regarded as more grammatically accurate and standard rather than dialectical.
The word "slijedeći" has been changed to "sljedeći" as "slijedeći" implies following someone rather than indicating what is upcoming or next.
The word "sad" has been replaced with "sada" for better grammatical precision.

Thanks


Avoid using random module inside Hypothesis strategies

Hey there!  I was looking through packages that use Hypothesis, and noticed something pretty weird:
https://github.com/jmgilman/beancount-hypothesis/blob/e6501171f2e1b0e879f67fd1812f6da45d465d39/beancount_hypothesis/account.py#L93-L101
https://github.com/jmgilman/beancount-hypothesis/blob/e6501171f2e1b0e879f67fd1812f6da45d465d39/beancount_hypothesis/common.py#L33-L41
These strategies don't actually use Hypothesis to generate data!  Instead, you're using the global random module, which Hypothesis seeds before every example to avoid flakiness.  That's really really not intended to be used like this though - you'll be "generating" more or less the same example each time, and if it does fail shrinking won't work very well.

On a lighter note, you also have one that just doesn't need to be a composite strategy: instead of
https://github.com/jmgilman/beancount-hypothesis/blob/e6501171f2e1b0e879f67fd1812f6da45d465d39/beancount_hypothesis/data.py#L64-L72
you could return st.builds(inv.Inventory, s.lists(position(), max_size=3))

Hi,
Thanks for raising an issue. This was my one and only time using hypothesis, so I appreciate the feedback.

On a lighter note, you also have one that just doesn't need to be a composite strategy

Fixed in 002b8b895de40764a852cdcf57a66a9ed2f67e1b.
Closing this out since the other issue is addressed in #2.