'map' object has no attribute 'shape'

This is a good project , but I got some error when I run this command:
python scripts/demo.py --network=raft3d.raft3d --model=raft3d.pth
It looks like a problem caused by lietorch，and I git clone from https://github.com/princeton-vl/lietorch.git and install lietorch==0.2.0
Traceback (most recent call last):
File "scripts/demo.py", line 97, in
demo(args)
File "/home/data3/zr/anaconda3/envs/lie_env/lib/python3.8/site-packages/torch/autograd/grad_mode.py", line 27, in decorate_context
return func(*args, **kwargs)
File "scripts/demo.py", line 78, in demo
flow2d, flow3d, _ = pops.induced_flow(Ts, depth1, intrinsics)
File "/home/data3/zr/code/RAFT-3D/./raft3d/projective_ops.py", line 52, in induced_flow
X1 = Ts * X0
File "/home/data3/zr/anaconda3/envs/lie_env/lib/python3.8/site-packages/lietorch/groups.py", line 206, in mul
return self.act(other)
File "/home/data3/zr/anaconda3/envs/lie_env/lib/python3.8/site-packages/lietorch/groups.py", line 174, in act
return self.apply_op(Act3, self.data, p)
File "/home/data3/zr/anaconda3/envs/lie_env/lib/python3.8/site-packages/lietorch/groups.py", line 127, in apply_op
inputs, out_shape = broadcast_inputs(x, y)
File "/home/data3/zr/anaconda3/envs/lie_env/lib/python3.8/site-packages/lietorch/broadcasting.py", line 18, in broadcast_inputs
check_broadcastable(x, y)
File "/home/data3/zr/anaconda3/envs/lie_env/lib/python3.8/site-packages/lietorch/broadcasting.py", line 8, in check_broadcastable
assert len(x.shape) == len(y.shape)
AttributeError: 'map' object has no attribute 'shape'
This is my conda envs, please tell me what should I do.
Package                 Version
absl-py                 1.4.0
appdirs                 1.4.4
brotlipy                0.7.0
cachetools              5.3.0
certifi                 2022.12.7
cffi                    1.15.1
charset-normalizer      2.0.4
contourpy               1.0.7
cryptography            38.0.4
cycler                  0.11.0
flit_core               3.6.0
fonttools               4.38.0
google-auth             2.16.0
google-auth-oauthlib    0.4.6
grpcio                  1.51.1
idna                    3.4
importlib-metadata      6.0.0
importlib-resources     5.10.2
kiwisolver              1.4.4
lietorch                0.2
Markdown                3.4.1
MarkupSafe              2.1.2
matplotlib              3.7.0
mkl-fft                 1.3.1
mkl-random              1.2.2
mkl-service             2.4.0
numpy                   1.23.5
oauthlib                3.2.2
opencv-python           <IP_ADDRESS>
packaging               22.0
Pillow                  9.3.0
pip                     22.3.1
pooch                   1.4.0
protobuf                4.21.12
pyasn1                  0.4.8
pyasn1-modules          0.2.8
pycparser               2.21
pyOpenSSL               22.0.0
pyparsing               3.0.9
PySocks                 1.7.1
python-dateutil         2.8.2
PyYAML                  6.0
requests                2.28.1
requests-oauthlib       1.3.1
rsa                     4.9
scipy                   1.10.0
setuptools              65.6.3
six                     1.16.0
tensorboard             2.12.0
tensorboard-data-server 0.7.0
tensorboard-plugin-wit  1.8.1
torch                   1.8.0
torchaudio              0.8.0a0+a751e1d
torchvision             0.9.0
tqdm                    4.64.1
typing_extensions       4.4.0
urllib3                 1.26.14
Werkzeug                2.2.3
wheel                   0.37.1
zipp                    3.13.0
In the demo.py,  seems to be attached with a map type of data at the end.

Same problem, and I don't know how to fix this either


Trackerquery refactor

[1] refactors trackerquery code
[2] add migration file to move trackerquery id from string to uuid
[3] changes the trackerquery payload to include tracker and space in relationships
[4] checks for logged in user for write actions

@kwk changed the position of error here 512136e


Wire up Don't ask me again checkbox in H/S confirm attachment modal

Overview
Right now, the modal has a checkbox to allow the user to never show the modal again. That checkbox needs to be wired up.
The modal is used both when 1) confirming set shake via the Set shake slideout and 2) clicking proceed to run when a heater shaker module is attached
Implementation Details

Probably need to use Alerts (app/src/organisms/Alerts/index.tsx)

Acceptance Criteria

checkbox functionality should work

look to the "don't show me again check box" from the calibration block modal for implementation example


[INFRA-21061] Adotp 'latest' aliases for AdoptOpenJDK.net JVMs.

Following this checklist to help us incorporate your contribution quickly and easily:

[ ] Make sure there is a JIRA issue filed
for the change (usually before you start working on it).  Trivial changes like typos do not
require a JIRA issue.  Your pull request should address just this issue, without pulling in other changes.
[X] Each commit in the pull request should have a meaningful subject line and body.
[X] Format the pull request title like [SHIRO-XXX] - Fixes bug in SessionManager,
where you replace SHIRO-XXX with the appropriate JIRA issue. Best practice
is to use the JIRA issue title in the pull request title and in the first line of the commit message.
[ ] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[ ] Run mvn clean install apache-rat:check to make sure basic checks pass. A more thorough check will be performed on your pull request automatically.
[ ] If you have a group of commits related to the same change, please squash your commits into one and force push your branch using git rebase -i.

Trivial changes like typos do not require a JIRA issue (javadoc, comments...).
In this case, just format the pull request title like (DOC) - Add javadoc in SessionManager.
If this is your first contribution, you have to read the Contribution Guidelines
If your pull request is about ~20 lines of code you don't need to sign an Individual Contributor License Agreement
if you are unsure please ask on the developers list.
To make clear that you license your contribution under the Apache License Version 2.0, January 2004
you have to acknowledge this by using the following check-box.

[X] I hereby declare this contribution to be licenced under the Apache License Version 2.0, January 2004
[X] In any other case, please file an Apache Individual Contributor License Agreement.

Branch is green: https://ci-builds.apache.org/job/Shiro/job/Shiro-all/job/INFRA-21061_adopt-latest/
PR is ok: https://ci-builds.apache.org/job/Shiro/job/Shiro-all/view/change-requests/job/PR-291/
By "ok" it means "started successfully using the aliases". The actual build output does not matter here.


Expose curator services externally

Currently login doesn't work as we have to change the oauth callback URLs and I don't have the perms to do that, I asked Spencer for perms and or changes to be made.
Also switches back to latest tag for dev images.
dev: http://51cfaa79-default-curatorde-771d-764651811.us-east-1.elb.amazonaws.com/
prod: http://51cfaa79-default-curatorpr-5c7d-1665373250.us-east-1.elb.amazonaws.com/ (old version from last week)

 Expose Curator UI publicly on AWS

 Define process for updating images running on AWS


Automate helm documentation update

There are some tools out there to automatically generate helm chart documentation from the values and comments. It would ensure we don't miss any entries and enforce comments in the value file.

@aslafy-z can you plz work on this and automate it?

I will try to get some time to work on this and let you know.

https://github.com/stakater/application/pull/335


Query strings for gestalt

In 2.8.3 (maybe earlier?) app sends requests like: /xled/v1/gestalt?filter=prod_infos&filter2=group and a device responds with
{
"product_name": "Twinkly",
"hardware_version": "100",
"bytes_per_led": 3,
"hw_id": "1cc190",
"flash_size": 64,
"led_type": 14,
"product_code": "TWS250STP",
"fw_family": "F",
"device_name": "Twinkly_1CC190",
"uptime": "1499371",
"mac": "98:f4:ab:1c:c1:90",
"uuid": "E103C5A3-3398-4B77-AE1A-9D8998A5EB62",
"max_supported_led": 510,
"number_of_led": 250,
"led_profile": "RGB",
"frame_rate": 24,
"measured_frame_rate": 25,
"movie_capacity": 5397,
"serial": "000000",
"production_date":<PHONE_NUMBER>,
"production_site": 4,
"group": {
"mode": "none",
"compat_mode": 0
},
"copyright": "LEDWORKS 2021",
"code": 1000
}

where uuid, mac, device_name, hw_id were redacted by me. Also serial (which seems to be new) - I kept length.
Copyright seems to be bumped to this year.

If you have a group, the group info will appear in here as well.  However it's not super-useful because gen 1 doesn't support it.

I had a look into this for my own project.
mode seems to have 3 values:

none = no group
master = main light to control
slave = all other lights in group

If you have an active group then size will be added. This is the total size of all the lights combined.
Additionally offset is added when mode=slave

that's how they keep track which light is which in the layout.  Twinkly
internally is creating a device for the group with a master layout of all
of them.  The only way to get the layout for the nested devices is to read
a section from the master layout with that offset as your start.
On Sun, Nov 27, 2022 at 12:51 PM Patrick @.***> wrote:

I had a look into this for my own project.
mode seems to have 3 values:

none = no group
master = main light to control
slave = all other lights in group

If you have an active group then size will be added. This is the total
size of all the lights combined.
Additionally offset is added when mode=slave
—
Reply to this email directly, view it on GitHub
https://github.com/xled/xled-docs/issues/23#issuecomment-1328315914, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ADB6POOUL56GASMQDFLXORDWKOUR3ANCNFSM5IU5Y3SQ
.
You are receiving this because you commented.Message ID:
@.***>

I had a look into this for my own project.
mode seems to have 3 values:

none = no group
master = main light to control
slave = all other lights in group

If you have an active group then size will be added. This is the total size of all the lights combined. Additionally offset is added when mode=slave

I only see this in the v2 api (meaning pro devices).  I am unable to reproduce this on wifi consumer devices.


Use namespace alias

Q
A

Bug fix?
no

New feature?
no

BC breaks?
no

Deprecations?
no

Has tests?
no

As doctrine does with ORM, we should provide a namespace alias reference on documentation.
I propose Audit on this PR. Feel free to ask for changes. :+1:

Thanks!


[RFC] Global Loop (simplified)

This is a simplified alternative to #77 introducing a global loop on top of the current API and being backward compatible.
To use the global loop, Factory::create() can be simply replaced by GlobalLoop::get().
The global loop is run automatically in a shutdown function, so no need to call run() manually. This behaviour can be disabled by calling GlobalLoop::disableRunOnShutdown() or by simply calling GlobalLoop::get()->run() .
This incorporates the suggestion from @mbonneau (https://github.com/reactphp/event-loop/pull/77#issuecomment-279186122) and also addresses https://github.com/reactphp/event-loop/pull/77#issuecomment-282234913.

I'm in favour of this, I think it's a great step in the right direction :+1:

Would the plan be to have the react libraries to start using GlobalLoop::get() if a loop was not provided by the user?

@mbonneau i'm not sure i'd like an optional loop parameter. I'd prefer simple facade api's on top of the current api's, eg. something i proposed in #77 or @clue in reactphp/socket-client#38 which then use the global loop.

It's been almost 2 years since this PR was created and it seems to be approved. Are there still plans to adopt this?

Closing this as it has been resolved through #226 and follow up PR's


[omniorb] new port

[x] Changes comply with the maintainer guide
[x] The name of the port matches an existing name for this component on https://repology.org/ if possible, and/or is strongly associated with that component on search engines.
[x] Optional dependencies are resolved in exactly one way. For example, if the component is built with CMake, all find_package calls are REQUIRED, are satisfied by vcpkg.json's declared dependencies, or disabled with CMAKE_DISABLE_FIND_PACKAGE_Xxx
[x] The versioning scheme in vcpkg.json matches what upstream says.
[x] The license declaration in vcpkg.json matches what upstream says.
[x] The installed as the "copyright" file matches what upstream says.
[x] The source code of the component installed comes from an authoritative source.
[x] The generated "usage text" is accurate. See adding-usage for context.
[x] The version database is fixed by rerunning ./vcpkg x-add-version --all and committing the result.
[x] Only one version is in the new port's versions file.
[x] Only one version is added to each modified port's versions file.

(i feel like deleting the above if I put in x everywhere anyway)
supersedes #7071; #8609
removes tag from #3346

@JonLiu1993 Why did you add requires:author-response?

No usage needs to be tested:
-- Fixing pkgconfig file: F:/Feature-test/omniorb/vcpkg/packages/omniorb_x64-windows/lib/pkgconfig/omniCOS4.pc
-- Fixing pkgconfig file: F:/Feature-test/omniorb/vcpkg/packages/omniorb_x64-windows/lib/pkgconfig/omniCOSDynamic4.pc
-- Fixing pkgconfig file: F:/Feature-test/omniorb/vcpkg/packages/omniorb_x64-windows/lib/pkgconfig/omniConnectionMgmt4.pc
-- Fixing pkgconfig file: F:/Feature-test/omniorb/vcpkg/packages/omniorb_x64-windows/lib/pkgconfig/omniDynamic4.pc
-- Fixing pkgconfig file: F:/Feature-test/omniorb/vcpkg/packages/omniorb_x64-windows/lib/pkgconfig/omniORB4.pc
-- Fixing pkgconfig file: F:/Feature-test/omniorb/vcpkg/packages/omniorb_x64-windows/lib/pkgconfig/omniZIOP4.pc
-- Fixing pkgconfig file: F:/Feature-test/omniorb/vcpkg/packages/omniorb_x64-windows/lib/pkgconfig/omniZIOPDynamic4.pc
-- Fixing pkgconfig file: F:/Feature-test/omniorb/vcpkg/packages/omniorb_x64-windows/lib/pkgconfig/omnithread3.pc
-- Fixing pkgconfig file: F:/Feature-test/omniorb/vcpkg/packages/omniorb_x64-windows/debug/lib/pkgconfig/omniCOS4.pc
-- Fixing pkgconfig file: F:/Feature-test/omniorb/vcpkg/packages/omniorb_x64-windows/debug/lib/pkgconfig/omniCOSDynamic4.pc
-- Fixing pkgconfig file: F:/Feature-test/omniorb/vcpkg/packages/omniorb_x64-windows/debug/lib/pkgconfig/omniConnectionMgmt4.pc
-- Fixing pkgconfig file: F:/Feature-test/omniorb/vcpkg/packages/omniorb_x64-windows/debug/lib/pkgconfig/omniDynamic4.pc
-- Fixing pkgconfig file: F:/Feature-test/omniorb/vcpkg/packages/omniorb_x64-windows/debug/lib/pkgconfig/omniORB4.pc
-- Fixing pkgconfig file: F:/Feature-test/omniorb/vcpkg/packages/omniorb_x64-windows/debug/lib/pkgconfig/omniZIOP4.pc
-- Fixing pkgconfig file: F:/Feature-test/omniorb/vcpkg/packages/omniorb_x64-windows/debug/lib/pkgconfig/omniZIOPDynamic4.pc
-- Fixing pkgconfig file: F:/Feature-test/omniorb/vcpkg/packages/omniorb_x64-windows/debug/lib/pkgconfig/omnithread3.pc
-- Installing: F:/Feature-test/omniorb/vcpkg/packages/omniorb_x64-windows/share/omniorb/copyright
-- Installing: F:/Feature-test/omniorb/vcpkg/packages/omniorb_x64-windows/tools/omniorb/bin/copyright
-- Performing post-build validation
Stored binaries in 1 destinations.

Thanks for the new port!

Thanks again!

@BillyONeal please remove the tag(info:new-port-unresolved) from #3346


init vsi terraform module

@akocbek - pre-commit are passing, but there are errors in the test. See https://github.com/terraform-ibm-modules/terraform-ibm-landing-zone-vsi/runs/7076174260?check_suite_focus=true#step:6:159

@argeiger @jvallexm - Hello - all checks are now passing. Could you do a final review, and if everything is fine, we can merge and get the initial release.

Since I am the creator of the PR, I am not able to approve but the PR LGTM


v-bind="obj" syntax with scoped slot

I'm trying to make a custom list component using a scoped slot like the one in the docs, and I would like to expose all of the item properties as props to the template. Unfortunately, the v-bind="obj" syntax doesn't seem to be working for me in a scoped slot. Is this deliberate?
Regular v-bind:prop="obj.prop" syntax is working correctly, but it would be great if I could expose everything at once.
I don't believe I'm making a mistake, but sincere apologies if I am, or if this has been mentioned somewhere else that I missed.
Thank you!

Entry point:
<custom-list :items="listItems">
	<template scope="item">

{{ item }}

	</template>
</custom-list>

CustomList with implicit v-bind object binding:
<template>

		<slot v-for="item of items" v-bind="item"></slot>

</template>

Result:

CustomList with explicit prop binding:
<template>

		<slot v-for="item of items" :id="item.id"></slot>

</template>

Result:

Here's what I did, I have an array of object instances of a specific class. That class has public functions in it and I need to invoke those functions on the template. It just seems the v-bind converts the passed value to a standard object. Is there any way I can invoke those class functions in there? Cheers.


Pretty print rules in Firebase to help with successfully compiled but erroneous rules

Apologies for blowing up the issues, but hopefully this is all useful.
If the rules are successfully compiled, but contain an error, the error looks something like the following:
=== Deploying to `some-firebase`
i   deploying rules

Error: Rules Syntax Error: 1:39199: Invalid == expression: left operand is not a number, boolean, string, or null.

You'll notice that it gives a reference to the location of the rules error, but it all on a single line, and therefore is not very useful. Somehow transferring and ensuring that the rules are "pretty-printed" as they are stored in Firebase would resolve this.

@mbleigh


How can I send a request with absolute url when already having baseURL

Moya master integrated by GitHub
enum Api {

    case initialize(deviceName: String, upushDeviceToken: String)
    case loadImage(_ url: String)
}

extension Api: TargetType {

    var baseURL: URL {
        URL(string: ApiKit.baseUrl)!
    }

    var path: String {
        switch self {
        case .initialize:
            return "init"
        case .loadImage(let url):
            return "\(url)"
    }
}

the baseUrl is the basic url, initialize request url is baseUrl/init
the loadImage request url is an absolute url like https://some-site/storage/image.png
When I tried loadImage, it combined basic url and the url, therefore server side return error 404

#2172 I had a similar problem. My solution might help you solve yours.

Following solution is inspired by @cskristof #2172 thread.
I think this issue can be closed.
Thanks @cskristof a lot!
let endpointClosure = { (target: Api) -> Endpoint in
    return Endpoint(url: url, sampleResponseClosure: {.networkResponse(200, Data())}, method: .get, task: .requestPlain, httpHeaderFields: apiHeader())
    //the url is the absolute URL stored image in server side.
    //apiHeader() is my func to generate common request headers, you can ignore it
}
let provider = MoyaProvider<Api>(endpointClosure: endpointClosure)
// the above code is the magical sentence, with the new provider, it only calls the endpointClosure no matter which api called with following request, you can write any api instead of .loadImage(url).
provider.request(.loadImage(url), completion: { result in
        switch result {
        case let .success(r):
            do {
                onComplete(try Image(uiImage: r.mapImage()))
            } catch {
                onComplete(Image(""))
            }
        case .failure(_):
            onComplete(Image(""))
        }
})


Update the readme file

Updated the readme file

@mgold1234 @nunnatsa @AvivLiberman
Can you plz review it?

I will go over it again

Description
Easy – Fast – Done :muscle:
Our product is designed to help you manage your team's tasks and controls workflows in a better, easier way. It has an easy UI for managers and employees with a safe user login.

Creating new tasks and assign them to your team.
Overall view of all the tasks with smart filtering system.
Comments section for discussion on each task.

@aradz1997 Check that out

@aradz1997 could you please open a new issue related to this PR and add it to the description of this PR
you can see for example:
https://github.com/redhat-beyond/ClassRater/pull/131
fixed:
https://github.com/redhat-beyond/ClassRater/issues/130

close #6

fixes #5

@aradz1997
when you fixed an issue that was raised in the comment it will be better to reply with "Done" or "Fixed" or any other response to make the communication fluent with your code reviewers.

LGTM


初始化 MCI 时发生问题。

environment:win10,python3.8.3, playshoud1.3.0 .
error information:
playsound('notice-text.mp3')
File "C:\Users\Administrator\anaconda3\lib\site-packages\playsound.py", line 72, in _playsoundWin
winCommand(u'open {}'.format(sound))
File "C:\Users\Administrator\anaconda3\lib\site-packages\playsound.py", line 64, in winCommand
raise PlaysoundException(exceptionMessage)
playsound.PlaysoundException:
Error 277 for command:
open notice-text.mp3
初始化 MCI 时发生问题。

Try downgrading to version 1.2.2
pip uninstall playsound
pip install playsound==1.2.2

excellent ！！！


Add level animation (with the camera) when level is loaded

This would still be pretty awesome!

Need to fix the ghost cam first!


Improved error handling for several crashes reported by users

Note: This PR includes commits from https://github.com/LordFokas/BlazeMap/pull/57 and as such will look a little messy until that one is merged.
These changes should catch and handle some of the errors reported by users of the previous release version to our Discord. Unfortunately, the fixes are hard to validate because the errors can't be reliably reproduced. However, a dev build has been available for a week and haven't had reports of further issues.
This also includes backporting a bugfix change LF made for 1.20.1.
Have also added/changed some of the logging around errors to make it clearer in the logs what went wrong, including making sure to log an error message before the stack trace, not after it, and adding error messages to many of the stack strace locations that previously didn't have them.
I also added in some profilers for cache read/write and for layer read/write, though currently they're not being reported anywhere. They don't seem to be running on the main thread as F3 + L isn't including them in the profile result (when I had them using the sync time profiler type that linked to that functionality) and I've not messed with the BM debug screen to try add references to them at the current time.

Closed now previous PR has been merged to try get cleaner commit history


Add more test

Your code is broken. Add more tests please.
https://github.com/stephenway/postcss-contrast/blob/master/index.js#L17

Oops, sorry :) Not broken. But need more tests for advanced values

Also plugin name and package name should be equal

Hey @TrySound, thanks for being patient with this, I've:

Refactored the plugin
Added more tests
Fixed plugin name
Released new version

@stephenway I meant something like linear-gradient :)


Refresh feature not working

When I refresh the report, I always get exactly the same report back. In the year since that report was generated, the project has grown by ~150%, and yet it gives me exactly the same info about the project. Is there something I need to do to refresh the report so that it actually hits the project again and goes through the files?
Project is Jacobin: https://goreportcard.com/report/github.com/platypusguy/jacobin

@platypusguy we download a .zip of the repository from proxy.golang.org using the latest module version.
So we first get the version from https://proxy.golang.org/github.com/platypusguy/jacobin/@latest which returns:
{"Version":"v0.0.0-20230121230130-f96320012698","Time":"2023-01-21T23:01:30Z","Origin":{"VCS":"git","URL":"https://github.com/platypusguy/jacobin","Hash":"f9632001269859d73ce8bc749b50a7d32499efeb"}}

Then we download the .zip:
<EMAIL_ADDRESS>If you download the .zip file above you'll see the src dir is missing and thus Go Report Card can't generate a report because there are no .go files.
If I had to guess maybe the go.mod is not in the right place.

I'll look into it, but in the meantime, I think you should generate an error message of some kind, rather than just re-posting the previous report. Thanks again!


user card popup for chat mention

this is going to conflict with https://github.com/keybase/client/pull/20110

@chrisnojima I'll wait until yours is merged

How is this triggered on mobile?

How is this triggered on mobile?

long press, like kbfs paths in chat


Annoying console appearing during Spyder startup

From<EMAIL_ADDRESS>on 2012-02-12T12:14:15Z
Spyder Version:  2.1.7
Python Version:  2.7.2
Qt Version:      4.7.3, PyQt4 (API v1) 4.8.5 on Windows
Hello,
I experience annoying console appearing everytime during Spyder startup. Basically when I click on a Spyder icon, Windows's CMD appears for a several secs, then there is a clasic Spyder startup (rectangle with a Spyder's logo and so on).
This is really annoying. How to fix it, please?
Original issue: http://code.google.com/p/spyderlib/issues/detail?id=936

From pierre.raybaut on 2012-02-15T12:25:49Z
This is not a bug: this is due to the fact that Spyder's default shortcut is executing Python with a command window. On Windows, when double-clicking on a .py file, Python is executed with a command window (the annoying console you are referring too, but it stays opened until the program exits), and for .pyw files, it's executed without a command window (which seems more integrated to the system and avoids the annoying console apparition) but it has side effects.
Due to the way Python itself is implemented on Windows, you must be conscious that it does make a difference to run Spyder with a command window. Without a command window (.pyw extension, i.e. using pythonw.exe), the Python interpreter standard input/output streams aren't initialized the same way as with a command window: as a
consequence, if you run something in a separate process within a Spyder session executed this way, the I/O streams will point to nowhere, resulting in an absence of console. The most common example is to run a system command with subprocess.Popen: in that case, you'll see that you need the command window to be able to see any output on the Spyder's console, otherwise, I/O streams won't be redirected to the console. I know that it's not sexy to have this ugly command window popping up at Spyder's startup, but it may be useful anyway. That is unfortunate that Python is implemented this way on Windows as other similar languages do not have this problem.
At least, in Spyder, I improved this by hiding the console as soon as I can in Spyder's main program. But I can't do anything more.
Status: WontFix


Training loss of LenPredictor.

Hey, I open this new issue as you suggested. For anyone interested, I raised new question here,

Hey. Sorry to reopen this issue, I tried to train a LenPredictor but got bad results. Even after 200 epochs, the LenSumLoss was still more than 1800 and the predicted durations were unreasonable. (1) I wonder what the normal loss value is, also that for PitchPredictor. (I used just part of your code, maybe it's caused by my wrong implementation) (2) BTW, does the mean and var of the dataset matters? I set them 0 and 1 for convenience.

and the author gave the answer here.

Hey again, generally speaking I prefer if you open a new issue for each separate thing so that people in the future can find it more easily :)
About your questions: (1) a normal loss value depends on the distribution of the dataset (specifically the duration distribution), a general rule of thumb is that it should be a noticeably better loss than simply guessing the average length or the common value or something. Are you using one of the described datasets or a custom one? I would look at the distribution of gt length to see if maybe there is bug or maybe the Hubert model doesn't output good tokens for your dataset. Anyway your loss sounds like some sort of bug. (2) the mean and var of which dataset? Of the durations? This can impact the absolute loss scale which would impact learning rate and also the relative weight to big errors (because of the MSE). This won't necessarily cause a big difference but hard to guarantee.

Here are my new comments:
(1) That makes sense. I trained it on ESD, and used HuBERT features from layer 9 with k=500 (publicly available). I will train it with features l6k100 (the same as the paper) tonight and see how it goes. It would be very helpful if you could provide me with a normal loss.
(2) The mean and var of training set, (durations of ESD in my case). Anyway, I get it.
Sorry for missing some details in my question,  I was quite confused at that time.

I am afraid I can't remember of the top of my head the exact loss value. As the loss is an average square of the length error per sample (+a small other term) it means that the average duration prediction error in your case is ~40 which is far from what I observed. As a quick test you could use the pretrained model provided here (the same one as linked in the repo), based on data extracted exactly like this repository and get the expected loss value. As another simple sanity check you could calculate the mean of the dataset and see what returning that gives you for an overall MSE, results of the trained model should be (notably) better than this. If I really try to remember I think the loss was in the range of 1-10, but best to test it again.
The normalisation could also impact the MSE scale if you calculated it on the normalised values. For instance the normalisation could make the data have many very negative values because the length distribution does not follow a normal distribution, but rather a decaying distribution with most values at 1, then less at 2, etc. but with also some outliers with length ~40.
I hope these can provide some help and insight, let me know if you manage to find the issue or have problems recreating any of the results with our full code on our datasets.

Thank you for your patience. I will try.

Hello, hope you were successful in your attempt.
If you need anymore help please let me know, otherwise I will close this issue as stale :)


Check/test methods in Traits

Main thing is to start checking Traits too, also some other misc changes can do a separate PR if needed.
The real change is https://github.com/neild3r/php-docblock-checker/commit/5e05a201e17707560488bb6c424372b854b2791f.
Misc changes:

switch to PSR12 instead of 2
remove dupe phpunit from composer.json
lil code tweaks

@neild3r It doesn't let me request a review but a lil tweak to start testing/checking trait methods. Would you mind taking a look please.


unable to pipe output because questions are printed to stdout

So there is an option to write to your disk, but I would rather these creds be as ephemeral as possible. The tool is already nice enough to print them with export so that you can copy/paste them, but it would be even easier if we could
gimme-aws-creds | sh

And have it work. The issue is that currently the questions are printed to stdout, which screws with the pipe. If they were printed to stderr instead, the above command would work.

Really good idea!  Shouldn't be too tough of a change, either.

I looked at this a little bit last night - the export statements are being printed to stderr right now, so you can do this source <(gimme-aws-creds 2>&1 >/dev/null) That redirects stderr to stdout and the original stdout to /dev/null.
However, that hides stdout completely, so it only works if you don't need to answer any prompts.  Moving all user interaction to stderr and the export output to stdout is a good solution - that should let users select which account/role they want or input their MFA code and still be able to pass the output to the shell.

I believe I was the one that added the export output mode.  I used stderr simply because all the rest of the logging already on stdout and I wanted to be able to redirect the exports to a file, as mentioned above.
If you create a profile for the target accounts you use, you can get GAC to work without user input, with the exception of the MFA prompt.  However if we merge PR #65, I've added the ability to pass MFA creds in as a CLI param.
The way I use this is with an awscreds.sh bash script similar to this:
gimme-aws-creds -u<EMAIL_ADDRESS>--mfa-code "$(getMFA)" -p "$1" 2>/tmp/aws_creds_vars;
getMFA is a function that gets a current MFA code from oathtool, $1 is the name of the account profile to get creds for (I've setup profiles for all the accounts I regularly use).  This bash script gets called by a function I have defined in my .bashrc:
function awscreds { if awscreds.sh "$1"; then source /tmp/aws_creds_vars; fi; }
Because exported env vars only apply to the forked shell they're defined in, you need the function alias defined in your current shell to source the output of the tool (as in epierce's example above).  This function and script, working together, allows me to simply type something like awscreds test and my current shell is loaded with the proper env vars, even with MFA enabled.

@sperry04 That's a pretty good workaround. If that mfa-code PR gets merged I will use that in the meantime. I still think it would be really nice to get the pipe to bash working, since it simplifies the call and removes the write to /tmp

Keep in mind that piping to sh isn't going to put the credential env vars into your current shell, it's going to put them into the shell forked by the sh call you piped to, which will go away as soon as the command ends.  The only way to get the env vars into your current shell is to source them in your current shell, like in epierce's example, or in the function in my example.  That is, unless I'm missing a bash trick I'm not aware of.

They should still appear if I follow the command
gimme-aws-creds | sh && something-requiring-creds

right?

Your first example doesn't work, I don't think.  Here's a simplified example:
echo "export TESTVAR=value" | sh && env
The export is executed by sh, but then that shell exits (returning 0) and then the env command executes.
A perhaps better way to do what you're trying would not be to use export, but something like this:
TESTVAR=value env or export TESTVAR=value; env
where the env value(s) are specified inline prior to the command that will use them, or as part of a multiple command script.  Your second example is the source equivalent to that, I think, with the side-effect that after the command is executed, your current shell continues to have the assigned credential vars.
I guess it depends on the use case.  If you intend to always run a single command with new credentials that exist only for the life of the command, the source based methods won't do that cleanly.  But, at least in my use case, I need to give my current shell AWS credential env vars so that I can run a bunch of ad-hoc aws CLI commands, which is well served by sourceing the exports into my current shell.

Ah, good to know. The source $(gimme-aws-creds --mfa-code=243) option is still pretty good then.


Information on data details page not translating into Russian

See screenshots below.
The text is still in English for the dataset summary

The text is still in English for the dates and the licence details

The text is still in English for the info on the available fields info box

The text is still in English for the data quality section

@sabahfromlondon the content in Russia is only available for a few publishers, the ones from ECCA, for example, Moldova or Kyrgyzstan https://data.open-contracting.org/ru/publication/29. However, the dates are not translated into Russian even for these datasets, so I will fix that


pyvmomi code to add vcsa to AD and to set dns settings

Is your feature request related to a problem? Please describe.
HI,
I need to know is there any pyvmomi equivalent to command  "/opt/likewise/bin/domainjoin-cli join {{ root_domain }} {{ user }} {{ password }}" for joining vcsa 7.0 to active directory?
and is there pyvmomi code to setup dns settings on vcsa?
Thanks,
Describe the solution you'd like
Looking for pyvmomi equivalent of vcsa shell command "/opt/likewise/bin/domainjoin-cli join {{ root_domain }} {{ user }} {{ password }}" for add vcsa to active directory?
also looking for pyvmomi code for configuring dns settings on vcsa
Thanks,
Describe alternatives you've considered
No response
Additional context
No response

Issue against pyvmomi : https://github.com/vmware/pyvmomi-community-samples/issues/717


Annoying lag when typing/moving with arrow keys

Description
Whenever I type text, sometimes my rendering in brackets falls behind for a few seconds, so I can't see what I'm typing.
Steps to Reproduce

Type text

Expected behavior: When I type text or move with arrow keys, the changes are instant
Actual behavior: It takes a few seconds
Versions
Windows 10
Release 1.7 build 1.7.0-16898 (release b0a363b)

@ealgase

Have you tried Debug > Reload Without Extensions mode? (Shift+F5)
What extensions do you have installed? (copy from Help > Health Report)

Health Report:
{     "uuid": "377f821f-e07c-4138-a0a6-214ba5e34fef",     "snapshotTime":<PHONE_NUMBER>355,     "os": "win",     "userAgent": "Mozilla/5.0 (Windows NT 6.2; WOW64) AppleWebKit/537.36 (KHTML, like Gecko) Brackets/<IP_ADDRESS> Chrome/49.0.2623.110 Safari/537.36",     "osLanguage": "en-US",     "bracketsLanguage": "en-US",     "bracketsVersion": "1.7.0-16898",     "projectLoadTimes": "",     "fileOpenTimes": "",     "prefNodeSearchDisabled": false,     "prefInstantSearchDisabled": false,     "installedExtensions": [],     "bracketsTheme": "dark-theme" }
Reloading without extensions DOES solve the problem; however, I do not know what extensions I have(I don't think I have any)

Testing more, it DOES still occur, but only when typing in a Javascript code block within HTML.


Enable configuration of AI service method parameters ignored annotations

In the same spirit of #888, it would be a nice addition to implement a predicate based build item to handle ignored annotations. This idea was discussed in this context.

👍🏽

I should be able to work on this in the coming day, feel free to assign to me if it make sense.

🙏


RC5 ion-item-sliding doesn't snap back to correct position when pulled left or right.

Ionic version:  (check one with "x")
[ ] 1.x
[ x ] 2.x
I'm submitting a ...  (check one with "x")
[x ] bug report
[ ] feature request
Current behavior:
Upgraded from RC4 to RC5. When sliding an item left or right in the list, the item will track with the position of my finger, instead of bouncing back to the correct position. When I lift my finger, the ion-item stays where I slide it to. Setup looks like this
    <ion-list [virtualScroll]="filteredMessages" [virtualTrackBy]="trackBy" approxItemHeight="86px" no-padding no-margin>
    <ion-item-sliding *virtualItem="let message">
        <ion-item class="item-message" >
            <ion-avatar item-left>

            </ion-avatar>

                contact name

note

            duration
            <ion-note item-right>
                test message

            </ion-note>
        </ion-item>
        <ion-item-options side="left">
            <button ion-button color="danger">

                Archive
            </button>
        </ion-item-options>

        <ion-item-options side="right">
            <button ion-button color="secondary">

                Save
            </button>
            <button ion-button>

                Read
            </button>
            <button ion-button>

                Urgent
            </button>
        </ion-item-options>

    </ion-item-sliding>
</ion-list>

Expected behavior:
The ion-item should bounce/snap back to the correct position if pulled beyond the limit.
Steps to reproduce:
Tried to make a plunker, but RC5 doesn't work in Edge :( 404 errors for .ts files
Other information:
Windows 10 mobile
Ionic info: (run ionic info from a terminal/cmd prompt and paste output below):
Your system information:

 You have been opted out of telemetry. To change this, run: cordova telemetry on.
6.4.0

Ionic Framework Version: 2.0.0-rc.5
Ionic CLI Version: 2.1.12
Ionic App Lib Version: 2.1.7
Ionic App Scripts Version: 0.0.47
ios-deploy version: Not installed
ios-sim version: Not installed
OS: Windows 10
Node Version: v6.9.1
Xcode version: Not installed

******************************************************
 Dependency warning - for the CLI to run correctly,
 it is highly recommended to install/upgrade the following:

 Please install your Cordova CLI to version  >=4.2.0 `npm install -g cordova`

******************************************************

+1
I'm having a similar issue

Rolled back to RC4 and the issue goes away

This seems to have been fixed in 2.0.0 final.


Time format getting changed when trying to change existing time value

PreCondition - Device settings has 24 hour time format.

In Add Interview Screen  - 1st time select a time value from TimePicker in  Time Input box and it will show time in 24 hours format.
Open the TimePicker again to change time.

Issue 1 - TimePicker does not show the existing user selected time in 24 hours format.
Issue 2 - After selection of time 2nd time, time does not get displayed in Input box in 24 hours format. It shows in 12 hour format.
Same Issue can be found when opening a Time Picker from Edit Interview Screen.

Not seen in the latest release build. Marking as closed.


feat(nui): allow callbacks in nui input with option

We already have an option to default to normal mode in nui inputs.
config.enable_normal_mode_for_inputs
However, it looks like users want more power of customization to the details. (https://github.com/nvim-neo-tree/neo-tree.nvim/issues/1371)
So why not just let users do whatever they want with absolute power?
All function/string/table are considered true-thy so this will cover backwards compatibility.

Usecase:
Move cursor right before the extension and start insert mode.
config.enable_normal_mode_for_inputs = ":exe 'norm F.' | :startinsert"

Usecase:
Move cursor right before the extension and start insert mode.
config.enable_normal_mode_for_inputs = ":exe 'norm F.' | :startinsert"

😱 In this example you're using an option called enable_normal_mode_for_inputs to startinsert mode. 😱
I think we already have a generic solution that would be perfect, which is the event system. In a quick attempt I could not get the existing NEO_TREE_POPUP_BUFFER_ENTER event to work for this situation, but I think either modifying the timing of NEO_TREE_POPUP_BUFFER_ENTER or adding a new event which fires at the appropriate time is the correct way to create a generic solution. Maybe NEO_TREE_POPUP_INPUT_READY would be a good event name.

Usecase:
Move cursor right before the extension and start insert mode.
config.enable_normal_mode_for_inputs = ":exe 'norm F.' | :startinsert"

😱 In this example you're using an option called enable_normal_mode_for_inputs to startinsert mode. 😱

Yeah :tada: That's what I mean by absolute power lol. I plan to not document this anywhere so it's absolutely to the users own risk to do anything.

I think we already have a generic solution that would be perfect, which is the event system. In a quick attempt I could not get the existing NEO_TREE_POPUP_BUFFER_ENTER event to work for this situation, but I think either modifying the timing of NEO_TREE_POPUP_BUFFER_ENTER or adding a new event which fires at the appropriate time is the correct way to create a generic solution. Maybe NEO_TREE_POPUP_INPUT_READY would be a good event name.

NEO_TREE_POPUP_BUFFER_ENTER This won't work as stopinsert is called way after buffer creation or even input:mount, and it's inside a vim.schedule which will delay it even more. The only way would be to defer_fn a couple of ms but that's really not good...
Creating a new event is also a good option, but the codebase is the same for confirm (used with eg delete (y/n) or change_cwd) and input (like add, copy etc).
I don't want to run the same event for those cases..

@cseickel Updated.
Regarding this change, I deprecated the enable_normal_mode_for_inputs option and added it to migrations.
As I wanted to give more information on how to migrate, I changed the migration module to be able to add descriptions in the buffer.
Besides, now bullet point for each change doesn't make much sense, so I changed it to use level 2 header ## Header 2 instead for each value.

It is definitely nice to be able to add a longer description to deprecations.
One thing I was considering is that I think it is more standard for plugins to have a start_in_insert_mode configuration for these situations. Maybe we should do the same and change the config option to start_inputs_in_insert_mode = false instead? Then we can keep the event for more advanced configuration but also have the simple one which is pretty standard.
What do you think?

I don't quite like when behavior before event handler execution changes by a different option in the config.
If you have an option to alter the behavior other than NEO_TREE_POPUP_INPUT_READY, I'd need to mention that in the docs with if ... otherwise ...
When people asks how to do xxx in the popup, I need to instruct; if you have config.start_inputs_in_insert_mode to true, then set this handler, and if false, set this handler.
I'd prefer to have the handler start consistently, either always from insert mode or always from normal mode.

When people asks how to do xxx in the popup, I need to instruct; if you have config.start_inputs_in_insert_mode to true, then set this handler, and if false, set this handler.

OK, fair enough. In that case, we stick with your idea.

@pysan3 Hello,   I update to release 3.19. The prompted NeoTree migrations reports some Error
E5108: Error executing lua ...m/lazy/neo-tree.nvim/lua/neo-tree/setup/deprecations.lua:18: Invalid option name: 'wrap'
stack traceback:
        [C]: in function 'nvim_buf_set_option'
        ...m/lazy/neo-tree.nvim/lua/neo-tree/setup/deprecations.lua:18: in function 'show_migrations'
        ...re/nvim/lazy/neo-tree.nvim/lua/neo-tree/command/init.lua:51: in function 'execute'
        ...re/nvim/lazy/neo-tree.nvim/lua/neo-tree/command/init.lua:169: in function '_command'
        [string ":lua"]:1: in main chunk

Do you know how to fix it or find more clues?

@Sologala
I'm so sorry: https://github.com/nvim-neo-tree/neo-tree.nvim/pull/1387
Please wait for the next release (which should happen right away).
You should see something like this so please follow the instructions.

The fix was just released as 3.20.

@pysan3  @cseickel Thx for update, My neo-tree work delicately again!

@pysan3 I think you forgot to remove enable_normal_mode_for_inputs from defaults.lua. I'm still getting a deprecation/migration notification on every startup, despite no use of this key in any configuration file.
The only location I can find where enable_normal_mode_for_inputs is being set is in defaults.lua:
https://github.com/nvim-neo-tree/neo-tree.nvim/blob/d8b4687c2457465199bb36be02316e0d3066605d/lua/neo-tree/defaults.lua#L22
I believe this is because the removed(key, desc) checks for nil, and false is not nil.
https://github.com/nvim-neo-tree/neo-tree.nvim/blob/d8b4687c2457465199bb36be02316e0d3066605d/lua/neo-tree/setup/deprecations.lua#L61-L68
$ lua
Lua 5.4.6  Copyright (C) 1994-2023 Lua.org, PUC-Rio
> foo = false
> foo ~= nil
true
> foo = nil
> foo ~= nil
false

Commenting out enable_normal_mode_for_inputs fixes the erroneous migration notification.

My bad. Thank you @rrrix

I may be missing a basic command, but can I make it such that enter the popup in insert mode, but pressing escape puts me in normal mode (instead of leaving the popup) ?
That's the behavior I'm getting from other popups.

@davawen
Map "i", "<Esc>" to vim.cmd.stopinsert in the event. There's the exact example in :h neo-tree-events for you but you need to update to the latest main branch (not yet include in the latest release iirc).


WSL & Ubuntu, how to move the init file

I deployed a dev environment on wsl and I'm so excited about it. My only problem is the existence of an init file in the root path of the system.

For one of my clients, I need to have an /init directory, this is a constraint that cannot be changed.
Needless to say, this renders my exciting and newly found wsl dev environment unusable for this project.
Is there a way to change the path of the wsl init file?

Is there a way to change the path of the wsl init file?

Not without drawing outside the lines, no. Much the same as you can draw outside the lines and with your client's immutable /init directory constraint. You could for example enter a mount namespace.


[flame_rive] null exception when no artboard with specified name is exists

Current bug behaviour
Throws Null Exception [Null Checks used on null value]
Expected behaviour
Return null or throw new Exceptions?
Steps to reproduce
In Rive Component's Example Code, change
final skillsArtboard = await loadArtboard(RiveFile.asset('assets/skills.riv')); to
final skillsArtboard = await loadArtboard(RiveFile.asset('assets/skills.riv'), artboardName: 'Not Exist');
More information
Do you have any other useful information about this bug report?
This cause because of https://github.com/rive-app/rive-flutter/blob/6d229dcdd990ca20450c160f5877566e3b7095a0/lib/src/rive_file.dart#L298-#L302
Are you interested in working on a PR for this?
yep, but Not sure how to fix it. There are three soln I came up with,

Make loadArtboard() return nullable Artboard which will be breaking.
Throw Some Type of Exception when Artboard with that name does not exist (Drawback is: User can only know when there run the App)
Return mainArtboard when Artboard with that name does not exist

I think Image.asset for example throws an exception when there is a non existent asset, we should do the same, if we aren't already doing that?

I think if the user passes a name for an artboard, they expect that artboard to exist. So, we should signal an error if it doesn't.
In this case I'd recommend to use a simple assert() that artboardByName() returned a non-null result. Thus, the only difference from the current behavior would be that the error message would be nicer and tell the user what exactly went wrong.

Agree that an asserr would be best here, to make it non-breaking.


Ubuntu 16.04: Telegram Messages & Contacts in same window

In Telegram, first we can only see the contacts (see screenshot1)
upon clicking one contact, we only can see the chat but not the contacts anymore (see screenshot2)

Expected Behavior

I assume, the idea of the 2 windows we can see in the screenshots, is that the contacts stay on the left and the chat goes on the right (screenshot 1), or the other way round (screenshot 2).
web.telegram.org lists the contacts on the left and the chat on the right
Current Behavior

Message/Contact windows messed up.
Screenshots (if appropriate):
Screenshot 1:

Screenshot 2:

Possible Solution

I assume there is just a wrong target window when Franz reacts to contact selections.
Steps to Reproduce (for bugs)

Use Telegram as Service
click on one contact

Your Environment

Franz Version used: 5.0.0-beta.14
Operating System and version: Ubuntu 16.04

I can't reproduce this issue. Is this happening for anyone else?
What you can try to do is tor quit Franz, delete the cache in ~/.config/Franz/recipes/Partitions and start Franz again. Does the problem persist?

There is no folder called "Partitions", but a folder for every service. Deleting the telegram folder didn't solve the problem tho.
I will try to reproduce the problem in the coming days on another machine.
Could it be a problem with importing the settings from Franz 4?

A reproducible solution would be great!
The importer does nothing else than creating a new service entry. To date, no cache is imported.

I think this has to do with the switch between desktop and mobile version of telegram. The screenshots show the mobile version. In Slack we talked about this specific issue: https://meetfranz.slack.com/archives/C5ULLT74P/p1511424081000161
So if this is indeed the same problem, it should be solved after the fix by @nemilya

I don't have slack, but I just tried to reproduce the bug on my private machine (Ubuntu 14.04) and it doesn't happen.
Telegram was asking to switch to the Desktop version at startup indeed. I did so, and it worked fine.
Issue still on my work machine, so if you could copy the fix in here, that'd be great.

So I could "fix" it.
Just needed to go to the settings and click "switch back to Desktop Layout".
Now it looks like expected!
Thanks for the help!


Relax the join validation for Remote State publication

Description
[Describe what this change achieves]
For Remote Store clusters, there is validation which ensures that remote attributes are either present / absent in all the nodes in cluster. However for clusters where only remote publication is enabled, this need not be enforced. ClusterManager ensures that the remote connecting node has publication enabled before publishing cluster-state updates[#15216 ]. Incase the remote node does not have publication enabled, it falls back to publish the cluster-state via transport
Related Issues
Resolves #[Issue number to be closed when this PR is merged]

Check List

[x] Functionality includes testing.
[x] API changes companion pull request created, if applicable.
[x] 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.

❌ Gradle check result for c991385: FAILURE
Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

Falky Test 1 2

Just to understand the strategy better, are we saying that based on the destination node we decide whether the publication needs to be remote vs local transport at the time of publication?

This is not deciding publication strategy but rather letting node join the cluster when remote state/ publication is enabled but store is disabled by validation desired repos at the time of node join.

❕ Gradle check result for 69fde35: UNSTABLE
Please review all flaky tests that succeeded after retry and create an issue if one does not already exist to track the flaky failure.

14289

Just to understand the strategy better, are we saying that based on the destination node we decide whether the publication needs to be remote vs local transport at the time of publication?

This is not deciding publication strategy but rather letting node join the cluster when remote state/ publication is enabled but store is disabled by validation desired repos at the time of node join.

Yes this was mostly to clarify

Incase the remote node does not have publication enabled, it falls back to publish the cluster-state via transport


Type and scope hidden in filtering terms

I have hidden the columns of type and scope as requested.

Commit: d9081b3


⚠ WSINGRESOS - OXXO has degraded performance

In 04f7e25, WSINGRESOS - OXXO (https://srvshyweb.yucatan.gob.mx/cgi-bin/wspd_cgi.sh/WService=wsingresos/internet/compartido/wspagosoxxo.r) experienced degraded performance:

HTTP code: 200
Response time: 9825 ms

Resolved: WSINGRESOS - OXXO performance has improved in a44d776 after 20 minutes.


Use of Modernizr for browser feature detection

Just for fun, I used Microsoft's new tool at:
https://dev.windows.com/en-us/microsoft-edge/tools/staticscan/?url=http%3A%2F%2Fmmistakes.github.io%2Fminimal-mistakes%2F
In the Browser Detection section, it says
"We've found that this webpage may be using browser detection techniques to determine how the webpage should render across many different versions of browsers."
and recommends using http://modernizr.com/
for feature-based detection (instead of version-based like "IE8").

I've found Modernizr is overkill for what I'm doing with the theme. In the upcoming release I've dropped it to shrink the JavaScript footprint. I was only really using it for CSS fallbacks which really isn't a progressive way of handling things.
I'm also cleaning up the theme's JavaScript overall so it's still accessible when disabled.
Doing more with less is really what I'm trying to achieve with the theme. I want to avoid jamming in everything or over developing the theme in the hopes of making it more approachable and easy to extend into whatever you want.


Chore: Use BoxShadow variants as part of tailwind style system

chore(CssStyle): remove utilities layer
chore(TailwindConfig): add boxShadow variants on tailwind config
chore(TasksComponent): replace classes to tailwind style

Hey @LukeberryPi, how are you doing mate? I really loved the project's UI, I'm also a huge fan of neo-brutalism \o/
Just created this PR to make the class system powered by tailwind more consistent!

approved, this will be merged when website access is back to normal, since we had some server issues earlier today.


collection list can not update when add a collection use mongo shell or other program

when i create a collection with mongo shell, the collection cannot find in the page /db/{database_name}.
in order to mangement the collection created by mongo shell, i need to restart mongo-express.
it seems that server cache the collection names for every databse.

:+1:

+1

+1

+1
A database management control panel that doesn't reflect changes to collections is incomplete. At the very least there should be a "reload" button. Better if it updates on browser refresh.

Thanks for your feedback @aa21 . A few hours prior to your post I had merged https://github.com/andzdroid/mongo-express/pull/69 from @svarlamov to include a refresh button, so before jumping on threads and posting negativity in bold please update to latest version.
@DavidePastore had written code to refresh databases a while back, but by the time I discovered this project and gained write access the code has diverged quite a bit and the old PR doesn't fit in anymore. https://github.com/andzdroid/mongo-express/pull/50
If you would like to submit a PR of code that does it automatically on page load, ideally with tests, then we would welcome it. @DavidePastore was on the right track, his PR seems pretty straightforward  :)

Thanks for your reply @wulfsolter. I'll update and let you know how it goes for me.

Unsure why your custom config.js was deleted, seems rather bizarre. The rest of changes were as expected as we trialled "readOnly: true" as default.


Courses List: Fetch and Display Custom Course Lists.

Hi @praveenscience,
I'm interested in this issue. Kindly provide additional details.
Thank you.


Candies Content added

Candies site Home page content has been added

Close #4500

Checklist:

[✅] I have mentioned the issue number in my Pull Request.
[✅] I have commented my code, particularly in hard-to-understand areas

Before

After

@TusharKesarwani  Kindly, review this

@MohitGupta121 @TusharKesarwani Kindly review this


Fix start stop IQR bug on filtered search results

When starting a session based off a filtered search, immediately exiting it resulted in the UI not properly re-rendering because the URL creation function was throwing an error.
This pr includes #168 as it needed it to test.
Thanks @Purg for noticing.

LGTM


Use column id instead of index to refer column when persist to storage

Describe This Problem
When persist schema info, we use column index to refer corresponding column, this works fine if schema is not updated, and will block us to implement some DDL, such as add column after another.
Proposal
Use column id to refer column when need to persist to underlying storage.
Additional Context
AFAIK, primary keys/bloom filters in encoded using index now.

Assuming that schema info mentioned here is the one in the SST, I guess this is not necessary, because the existing sst won't be updated even if the schema is updated.

I grep TableSchema, schema is not only used in SST, but also in meta_update.
7 candidates:
./common.proto:46:message TableSchema {
./common.proto:67:  common.TableSchema table_schema = 1;
./meta_update.proto:43:  common.TableSchema schema = 4;
./meta_update.proto:98:  common.TableSchema schema = 3;
./remote_engine.proto:64:  common.TableSchema table_schema = 1;
./sst.proto:34:  common.TableSchema schema = 5;
./table_requests.proto:14:  common.TableSchema schema = 2;


make ColumnFamily Sync

This resolves a long standing issue https://github.com/rust-rocksdb/rust-rocksdb/issues/407 where because ColumnFamily is not Sync, we are not able to fetch all the cfs on startup and use them directly, instead, we are currently forced to call cf_handle on every operation to get the cf which adds unnecessary overhead

Agreed, very important to make RocksDB properly usable in async code.
Besides, imho the current implementation is definitely flawed. Just looking at it from the Rust side ColumnFamily must either be Send + Sync or neither. Just Send definitely doesn't make sense.


spi: spi_ll_stm32: (fix) Clear OVR bit condition

The OVR condition in the Status Register requires a
particular sequence in order to be cleared. If not
done properly it stays set forever.
Signed-off-by: Armando Visconti<EMAIL_ADDRESS>

Area: stm32 SPI FIX

I found it using spi slave. Not able to reproduce the issue using spi as a master.

Codecov Report

Merging #8147 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #8147   +/-   ##
=======================================
  Coverage   64.54%   64.54%
=======================================
  Files         420      420
  Lines       40142    40142
  Branches     6765     6765
=======================================
  Hits        25911    25911
  Misses      11110    11110
  Partials     3121     3121

Continue to review full report at Codecov.

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


FileUpload: Clean up MaximumFileCount tests and update docs.

Description

@mikes-gh ready for review.
How Has This Been Tested?

Types of changes

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

Checklist:

[x] The PR is submitted to the correct branch (dev).
[x] My code follows the code style of this project.
[x] I've added relevant tests.

Thanks @Mr-Technician


Bootstrap4 support

Implements bootstrap4 support for Identity scaffolding. There is a new command line flag "bootstrapVersion" (short version "b") which can be used to specify the bootstrap version. If not specified, the default version is used - currently bootstrap 4. The ...CodeGeneration.Design package carries both versions of Identity content (which is why this PR seems so huge).

I just pushed a commit updating the bootstrap4 _Layout.cshtml and _ValidationScriptsPartial.cshtml to deal with the test failures due to sha mismatches. The versions of those files I originally pused were copied from aspnet/Identity. The new versions are copies of the identity scaffolding bootstrap 3 versions of those files.
I'm not clear on why we have different script sources & sha's between aspnet/identity and aspnet/scaffolding.
@natemcmaster @ryanbrandenburg


RDART-969: Keypath filtering on collections

This PR adds support for keypath filtering on collections
TODO:

[x] Add support for lists
[x] Add support for sets
[x] Add support for dictionaries
[x] Add basic tests for lists, sets, dictionaries (bulk of the tests will be on results). This includes verifying we can't use keypath filtering on collections that do not contain realm objects
[x] Changelog

@nielsenko It needs a couple of things (like the changelog), but I think it can already be reviewed.
I've put the bulk of the tests for the keypath filtering on results, and I've left some basic tests for the other collections (lists, sets, dictionaries), so we don't repeat the same tests.
I have left some todos around about things I am not sure / we need to discuss, but functionally (almost) everything should work

@nielsenko I think I've addressed all the comments.
A couple of other things I've noticed, and that we don't necessarily need to address here, but I thought it would be worth writing down.
We have some inconsistencies with naming:

In the errors for unmanaged sets we call them RealmSets, while we call lists just Lists (same for maps)
Regarding notification controllers, we have ListNotificationController but RealmSetNotificationController.
Regarding tests, we have list_test but we have realm_set_test/realm_map_test
Maybe we should have a different suffix between the native handle implementation and the interface files. just having a look is a little bit confusing sometimes.

Some other considerations (that probably require some work):

Maybe we can move the keypath property to the base NotificationController,  instead of having it defined in the various specific classes
Maybe we can have a base class for all collections (results, lists, sets, maps). This way we could move changes and some other common things there. The collection handle themselves could have a common base class.

In my opinion..

In the errors for unmanaged sets we call them RealmSets, while we call lists just Lists (same for maps)

Going forward we should try to reduce the use of realm prefix to where it is needed. This means only on public facing classes, that are likely to collide with other class names (RealmSet, RealmList, RealmMap, etc.). Internally we should avoid unless it makes code more readable.

Regarding notification controllers, we have ListNotificationController but RealmSetNotificationController.

It should just be SetNotificationController.

Regarding tests, we have list_test but we have realm_set_test/realm_map_test

it should just be set_test.dart.

Maybe we should have a different suffix between the native handle implementation and the interface files. just having a look is a little bit confusing sometimes.

I'm a bit reluctant on this. For each handle, there is the interface, and two implementations.. I would prefer not to suffix the implementations with Native and Web (or whatever)

Some other considerations (that probably require some work):

Maybe we can move the keypath property to the base NotificationController,  instead of having it defined in the various specific classes

Maybe we can have a base class for all collections (results, lists, sets, maps). This way we could move changes and some other common things there. The collection handle themselves could have a common base class.

The current C-api don't really give us much benefit, but yes.
However, let us land this PR as is, and get started on the weekend 😄 .. we can always address this later.

I'll merge this as soon as the CI agrees


Fix issues of CTS 1.2

Hi, could you rebase this PR? I thought I had upgraded to 1.2 correctly.

Yes, already rebased.

Upgrade to 1.2 doesn't mean it can pass CTS 1.2. This PR aims to fix most of the CTS 1.2 failures.

BTW, there is still an issue, the icd create device failed because icd doesn't support timeline_semaphore feature. A solution I can think of is to clone the createInfo and pass it down to the ICD createDevice. What do you think?

I meant create info is const, we need to clone the create info and remove the timelineSemaphore feaure from the create info so that ICD creates device successfully.

Yes, already rebased.

You must be on a different branch, I can't see the function you're removing on master :
$ git grep timeline_ImportSemaphoreFd
layers/timeline_semaphore.c:static VkResult timeline_ImportSemaphoreFdKHR(

I meant create info is const, we need to clone the create info and remove the timelineSemaphore feaure from the create info so that ICD creates device successfully.

Oh right... I keep testing this on a version of our implementation that already supports the extension...

Yes, already rebased.

 The symbol you grepped is one I added and then removed.

Hi @djdeath , do you have time to take a look at this?

Thanks a lot of your changes. Most of them make sense to me and fix bugs with the 1.2 API.
If you could just answer the few questions I have. Those are details I'm probably failing to understand at the moment :)
Finally there a few things that you add and then remove (like  ImportSemaphoreFd) if you could just get rid of adding then removing this stuff, I think it would make the commits a bit shorter.
Thanks again!


Click to position target and transducer

Introduction
This PR adds to the web interface the possibility of clicking to position the target of the simulation and also to position and define the transducer direction.
Changes
An HTML canvas was used in order to recreate the matplotlib graph coming from the NDK server, in this way we are able to have full control of the mouse position
Behavior

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:
:loudspeaker: Thoughts on this report? Let us know!


Consider moving callbackDataGetter up a level

Summary
Currently a closure is created and passed to processCallback to get the callback data. The benefit of this is that if getCallbackData returns an error, we handle this in a single place, as a presence of an error which actually have us return nil.
Another approach is to directly call getCallbackData within the SendPacket/OnAcknowledgePacket/etc functions. This requires handling the presence of an error with a nil return, but it reduces args to processCallback and improves code readability

For Admin Use

[ ] Not duplicate issue
[ ] Appropriate labels applied
[ ] Appropriate contributors tagged/assigned

done in https://github.com/cosmos/ibc-go/pull/3939/commits/39622c02f163e97dcc291d43b11abae347187c84


ci: call set-env.* from install-deps.*

This pr lets test/ci/install-deps.* call set-env.* for simplifying the overall process and for splitting the scripts' responsibilities more clearly.
Further work
Update Wiki

Further work
Update Wiki

Done.


Download/Upload Speed in onPostExecute

How can I get Download/Upload Speed in onPostExecute function after finishing DoInBackground process? DoInBackground function returns null, that's why can't able to access any parameters.

startDownload() and startUpload() are both non-blocking so you can wait for completion in DoInBackground by blocking that thread. When upload/download is completed, release your thread and store the result, then return the local result variable after the thread release.
You can do something like this
public class SpeedTestTask extends AsyncTask<Void, Void, SpeedTestReport> {

    Object sync = new Object();

    private SpeedTestReport mFinalReport;

    @Override
    protected SpeedTestReport doInBackground(Void... params) {

        synchronized (sync) {
            SpeedTestSocket speedTestSocket = new SpeedTestSocket();

            // add a listener to wait for speedtest completion and progress
            speedTestSocket.addSpeedTestListener(new ISpeedTestListener() {

                @Override
                public void onDownloadFinished(SpeedTestReport report) {
                    // called when download is finished
                    Log.v("speedtest", "[DL FINISHED] rate in octet/s : " + report.getTransferRateOctet());
                    Log.v("speedtest", "[DL FINISHED] rate in bit/s   : " + report.getTransferRateBit());
                    mFinalReport = report;
                    synchronized (sync) {
                        sync.notify();
                    }
                }

                @Override
                public void onDownloadError(SpeedTestError speedTestError, String errorMessage) {
                    // called when a download error occur
                    sync.notify();
                }

                @Override
                public void onUploadFinished(SpeedTestReport report) {
                    // called when an upload is finished
                    Log.v("speedtest", "[UL FINISHED] rate in octet/s : " + report.getTransferRateOctet());
                    Log.v("speedtest", "[UL FINISHED] rate in bit/s   : " + report.getTransferRateBit());
                    mFinalReport = report;
                    synchronized (sync) {
                        sync.notify();
                    }
                }

                @Override
                public void onUploadError(SpeedTestError speedTestError, String errorMessage) {
                    // called when an upload error occur
                    synchronized (sync) {
                        sync.notify();
                    }
                }

                @Override
                public void onDownloadProgress(float percent, SpeedTestReport report) {
                    // called to notify download progress
                    Log.v("speedtest", "[DL PROGRESS] progress : " + percent + "%");
                    Log.v("speedtest", "[DL PROGRESS] rate in octet/s : " + report.getTransferRateOctet());
                    Log.v("speedtest", "[DL PROGRESS] rate in bit/s   : " + report.getTransferRateBit());
                }

                @Override
                public void onUploadProgress(float percent, SpeedTestReport report) {
                    // called to notify upload progress
                    Log.v("speedtest", "[UL PROGRESS] progress : " + percent + "%");
                    Log.v("speedtest", "[UL PROGRESS] rate in octet/s : " + report.getTransferRateOctet());
                    Log.v("speedtest", "[UL PROGRESS] rate in bit/s   : " + report.getTransferRateBit());
                }

                @Override
                public void onInterruption() {
                    // triggered when forceStopTask is called
                    synchronized (sync) {
                        sync.notify();
                    }
                }
            });

            speedTestSocket.startDownload("2.testdebit.info", "/fichiers/1Mo.dat");

            try {
                sync.wait();
            } catch (InterruptedException e) {
                e.printStackTrace();
            }
        }
        return mFinalReport;
    }

    @Override
    protected void onPostExecute(SpeedTestReport report) {
        super.onPostExecute(report);

        Log.v("speedtest", "[final report] rate in octet/s : " + report.getTransferRateOctet());
        Log.v("speedtest", "[final report] rate in bit/s   : " + report.getTransferRateBit());
    }
}

Thank You for your quick response.


Add CA1036 Suppressor

Fixes #241

Looks good to me - thanks very much! 👍

Thanks for merging and deploying so quickly! 😄


replace datalist with a button dropdown for setting resolver url (fixes #18)

Hopefully this appeases @cdeccio

I originally didn’t set it, but then it was somewhat difficult to distinguish the drop down from its background.

I'm fine either way. If you think it looks better, merge this sucker :smile:


Fix juliaup docs

I'll look into more benchmarks, this is from Debian's benchmark game (my old local file), I'll see if others have big gain, and if your script will be accepted there...
It at least worked (not much faster, though better in next step; and machine loaded, but still plausible):
$ hyperfine 'juliascript pi.jl >/dev/null'
Benchmark 1: juliascript pi.jl >/dev/null
  Time (mean ± σ):      2.227 s ±  0.184 s    [User: 2.569 s, System: 0.553 s]
  Range (min … max):    2.048 s …  2.600 s    10 runs

$ hyperfine 'julia pi.jl >/dev/null'
Benchmark 1: julia pi.jl >/dev/null
  Time (mean ± σ):      2.352 s ±  0.243 s    [User: 2.670 s, System: 0.577 s]
  Range (min … max):    2.096 s …  2.844 s    10 runs

$ time juliascript packagecompile pi.jl
  Activating project at `~/.julia/scripts_cache/script_pi_972a5ec309bcd9e922c2a3839c8fe18e9197ea5a1e0bb7a03b6af23bf15a772e`
Precompiling project...
  3 dependencies successfully precompiled in 3 seconds
✔ [01m:44s] PackageCompiler: compiling incremental system image

real	1m54,801s
user	7m33,161s
sys	0m8,613s

$ hyperfine 'juliascript pi.jl >/dev/null'
Benchmark 1: juliascript pi.jl >/dev/null
  Time (mean ± σ):      2.083 s ±  0.148 s    [User: 2.441 s, System: 0.533 s]
  Range (min … max):    1.934 s …  2.341 s    10 runs

Thank you so much for the update to the documentation

FYI: Not working in 1.11, still RC3, will see about after release:
$ juliascript pi.jl
fatal: error thrown and no exception handler available.
InitError(mod=:Base, error=AssertionError(msg="isempty(loaded_precompiles)"))
__init__ at ./Base.jl:606
jfptr___init___69822 at /home/pharaldsson/.julia/scripts_cache/script_pi_972a5ec309bcd9e922c2a3839c8fe18e9197ea5a1e0bb7a03b6af23bf15a772e/sysimage.so (unknown line)
jl_apply at /cache/build/tester-amdci5-12/julialang/julia-release-1-dot-11/src/julia.h:2157 [inlined]
jl_module_run_initializer at /cache/build/tester-amdci5-12/julialang/julia-release-1-dot-11/src/toplevel.c:76
_finish_julia_init at /cache/build/tester-amdci5-12/julialang/julia-release-1-dot-11/src/init.c:902
julia_init at /cache/build/tester-amdci5-12/julialang/julia-release-1-dot-11/src/init.c:843
jl_repl_entrypoint at /cache/build/tester-amdci5-12/julialang/julia-release-1-dot-11/src/jlapi.c:1053
main at /cache/build/tester-amdci5-12/julialang/julia-release-1-dot-11/cli/loader_exe.c:58
unknown function (ip: 0x7f6ea93b7d8f)
__libc_start_main at /lib/x86_64-linux-gnu/libc.so.6 (unknown line)
unknown function (ip: 0x4010b8)

thank you for pointing out :pray:

I'm not sure why 1.11 worked for a new script (nor why there slowdown..), though not first time around, and the actual intented script (so sure what other pi script I had):
https://benchmarksgame-team.pages.debian.net/benchmarksgame/program/pidigits-julia-3.html
[Their timing is 1.02 sec, mine is worse... I think they still have an old machine, older than mine?!]
And even more speedup, since shorter runtime, startup overhead larger fraction:
$ hyperfine 'julia pidigits.jl 10000 >/dev/null'
Benchmark 1: julia pidigits.jl 10000 >/dev/null
  Time (mean ± σ):      1.277 s ±  0.034 s    [User: 2.106 s, System: 0.067 s]
  Range (min … max):    1.240 s …  1.354 s    10 runs

$ hyperfine 'juliascript pidigits.jl 10000 >/dev/null'
Benchmark 1: juliascript pidigits.jl 10000 >/dev/null
  Time (mean ± σ):      1.597 s ±  0.039 s    [User: 2.376 s, System: 0.116 s]
  Range (min … max):    1.528 s …  1.658 s    10 runs

and this does not help, since not yet working, at least on 1.11:

$ juliascript packagecompile pidigits.jl 10000
┌ Error: Please run juliascript once normally, so that the precompile statements needed for PackageCompiler are constructed.
└ @ JuliaScript ~/.julia/packages/JuliaScript/KBGYf/src/JuliaScript.jl:21

Any idea why passing any opt (even "same") is worse?
$ hyperfine 'juliascript pidigits.jl 10000 >/dev/null'
Benchmark 1: juliascript pidigits.jl 10000 >/dev/null
  Time (mean ± σ):      1.594 s ±  0.042 s    [User: 2.362 s, System: 0.127 s]
  Range (min … max):    1.536 s …  1.652 s    10 runs

$ hyperfine 'juliascript -O3 pidigits.jl 10000 >/dev/null'
Benchmark 1: juliascript -O3 pidigits.jl 10000 >/dev/null
  Time (mean ± σ):      3.442 s ±  0.042 s    [User: 4.127 s, System: 0.186 s]
  Range (min … max):    3.361 s …  3.496 s    10 runs

$ hyperfine 'juliascript -O2 pidigits.jl 10000 >/dev/null'
Benchmark 1: juliascript -O2 pidigits.jl 10000 >/dev/null
  Time (mean ± σ):      3.422 s ±  0.021 s    [User: 4.116 s, System: 0.172 s]
  Range (min … max):    3.390 s …  3.458 s    10 runs

hyperfine 'juliascript -O1 pidigits.jl 10000 >/dev/null'
Benchmark 1: juliascript -O1 pidigits.jl 10000 >/dev/null
  Time (mean ± σ):      3.426 s ±  0.055 s    [User: 4.122 s, System: 0.176 s]
  Range (min … max):    3.389 s …  3.569 s    10 runs

and all worse than:
$ hyperfine 'julia -O1 pidigits.jl 10000 >/dev/null'
Benchmark 1: julia -O1 pidigits.jl 10000 >/dev/null
Time (mean ± σ):      1.253 s ±  0.032 s    [User: 2.087 s, System: 0.062 s]
Range (min … max):    1.209 s …  1.312 s    10 runs
$ hyperfine 'julia -O3 pidigits.jl 10000 >/dev/null'
Benchmark 1: julia -O3 pidigits.jl 10000 >/dev/null
Time (mean ± σ):      1.279 s ±  0.019 s    [User: 2.112 s, System: 0.062 s]
Range (min … max):    1.243 s …  1.301 s    10 runs

The first time it will always track the precompile statements. That takes significant amount of time. But then it should faster.
Don't know, but thanks for creating some test examples. Would you mind sharing them?

Yes, what do you mean the full code here? It is above if you click the link.
About:
$ juliascript packagecompile pidigits.jl 10000
┌ Error: Please run juliascript once normally, so that the precompile statements needed for PackageCompiler are constructed.
└ @ JuliaScript ~/.julia/packages/JuliaScript/KBGYf/src/JuliaScript.jl:21

I realize now your package is for script with package dependencies, likely why this didn't work. It would be better if it worked robustly either way (I could open an issue), I guess you then have a dilemma knowing if you actually ran first (just output an empty file then?).
It's not going to speed up then, and that's the point, so I'm puzzled why it worked for other script, also actual speed-up, since I thought it self-contained.
That one follows:
function calculate_pi(N)
    w = 0

    k = 1
    n1 = BigInt(4)
    n2 = BigInt(3)
    d = BigInt(1)

    i = 0
    while true
        # digit
        u = div(n1, d)
        v = div(n2, d)
        if u == v
            println(u)
            i += 1
            if i % 10 == 0
                println("\t:", i)
            end
            if i == N
                break
            end
            # extract
            u = d * (-10) * u
            n1 = n1 * 10 + u
            n2 = n2 * 10 + u
        else
            # produce
            k2 = k << 1
            u = n1 * (k2 - 1)
            v = n2 + n2
            w = n1 * (k - 1)
            n1 = u + v
            u = n2 * (k + 2)
            n2 = w + u
            d = d * (k2 + 1)
            k += 1
        end
    end

    if i % 10 != 0
        println(" " ^ (10 - i % 10), "\t:", N)
    end
end

# Example usage with N = 50
calculate_pi(10_000)

Also FYI:
$ juliascript -O3 pidigits.jl
basename: invalid option -- 'O'
Try 'basename --help' for more information.
sha256sum: invalid option -- 'O'
Try 'sha256sum --help' for more information.
┌ Error: The given file does not exist
└ @ JuliaScript ~/.julia/packages/JuliaScript/KBGYf/src/JuliaScript.jl:47

Why hyperfine took longer. Don't worry about this getting to work, and faster.


Improve behavior when gene list is provided

It was suggested that when the user provides a gene list, the outputs should only contain information related to this set and no other information. So the outputs of the different views would be:
Gene View

User genes linked to keyword are indicated through icons in the evidence column
User genes not linked to keyword are indicated with 0 in the evidence column

Map View

Only show user genes linked to keyword

Evidence View

Only show evidence documents that are present in the network of the user genes and contain the keyword

Network View

No change

Behavior if the user provides a gene list and QTL regions.

Option 1 whole genome: same as above + indicate QTL genes with yes
Option 2 within region: same as above + indicate QTL genes with yes

That means the user provided gene list would serve as the superset that is fully shown in the Gene View while QTL and Evidence information are indicated in corresponding columns.
Would be good to have other opinions on this?

Implemented restriction to Gene View. Similar work pending on restricting Evidence View & Map View.

Map View enhancement code underway (restriction by Gene List). Once complete & tested, will look into developing a similar feature for Evidence View as well.

Map View enhancement added and pushed

tested and pushed changes to Genomaps xml and Gene View table, Evidence View pending

I assume this is done and closing it. Reopen if needed.


[ENHANCEMENT] - Add support for embedding anchor.fm player

Is your feature request related to a problem? Please describe.
Can you add support in the episode files for embedding a link to an anchor.fm player?
Describe the solution you'd like
A configuration option in the episode markdown files that would embed the anchor player both on the episode detail page and right on the home page.  See their twitter thread for details:
https://twitter.com/anchor/status/992119358510989314?lang=en
I'd be happy to test this out for you if you don't have an anchor.fm podcast to test with.  Or I can give you my embed code to test with as well.
Probably other sites have similar functionality but since anchor.fm is the best free hosting service it probably makes sense to start with them anyway.

I'm a little leery about adding more players; it's not something that I can regression test, but I think what might work is to have the ability to disable the built-in player and support adding a custom embed. I'll have to think about how that works; you would have to copy-paste the embed per episode each time somehow but there might be a way to make that slightly not cumbersome.

Yeah the way anchor works is that they only give you a per-episode embed link so it would be done in the episode markdown.
TBH I'm not sure how this would be any different than the youtube link which is also done per-episode.
I think an option to embed custom player html would be a decent solution as it would theoretically work for any player.

well, the thing is for youtube you pass in the URL of the video, but the theme does things with it for the layout, etc.
I think for this to work it would have to be "option to disable built in player" and you'd have to put the embed code into the markdown file? It would also remove the player working on the list of episodes, etc.

Couldn't the list of episodes also show the embedded player?

Not really, since it's specifically styled to show up and since I don't know what an iframe would look like (no idea what comes in the iframe) it could push things all over the place :)

I definitely want to create as much flexibility for use with this theme, but heavy customizations are tricky since it has to work for...everyone :)
Let me spend some time thinking about a way to work out an alternative for different embedded players.

Yeah I understand.  But I would say that since anchor.fm is the most popular totally free hosting solution for podcasts, and it is owned by spotify, it seems worthwhile to elevate that to a level that the theme provides a custom option for (and formats properly).
In my mind this is no different than the theme explicitly providing support for an embedded youtube video, which is also a popular and totally free way of distributing media.

Closing this for now, I see it would be very complicated.


Please restore build folder

since build/.gitignore has been deleted all builds are failing, please restore it so that we can see build status for the different platforms :)
https://github.com/MrHuu/devilutionX-3ds/commit/7597da1c0ff285d67d63c795979ee7fa68462796#diff-38dc90d020e872ead1c55faa51e9832c

Also why was mainmenu.cpp renamed to mmainmenu.cpp?

I'll have the build folder restored.
I'm not yet familiar with cmake. I chose to take the path i'm most familiar with, the regular makefile.
Creating / deleting the build folder is part of the current build instructions for 3ds.
I'll change the build folder for 3ds for now, to avoid conflicts with the proper cmake instructions.
The duplicate mainmenu.cpp was an other limitation of my knowledge.
All objects are currently being build (for 3ds) in the same folder without folder structure. Which has a conflict with these files:

/Source/mainmenu.cpp
/SourceX/DiabloUI/mainmenu.cpp

Renaming while building the object, like the switch port did, was an option.
But, since i was completely in the blank about getting this even running, i didn't want to spent much time looking into it.
I rather use the time for researching cmake to properly include the 3ds build instructions, which also solves the issue.
Thanks for your interest and time to look at the few changes made.
Not being a developer, i'm sure you'll notice a few more questionable changes.
Please, feel free to ask or laugh about them.

Nice work getting it running on 3DS, it's good to see that only a minimal amount of changes where needed and you maintained it as a fork so upstraming will be much easier the the Switch port 😊
It looks like you need to also hide romfsExit() and it's related code in main.cpp from other targets before they will build again.

To me it's unexpected how well the current build runs on a New3DS. All thanks to the progress made on devilution. So it's hats off to you sir, and others involved.
I also noticed, romfsExit() isn't even reached after return.
The current build freezes the 3ds upon shutdown, that's something i also need to fix.
Starting with my own oversights.

Looks like you more or less got the CMake stuff done in just one day, pretty impressive :)
We left you a couple of comments in the preliminery PR in case you didn't notice:
https://github.com/diasurgical/devilutionX/pull/483

Grate most targets are now building (Windows is currently failing to build stormlib)


Expo SDK 33 - Change the dependency

Hi @Cretezy,
The new expo SDK 33 is released.
I would like to do several changes on your package which don't need anymore the entire expo dependency.
https://docs.expo.io/versions/latest/sdk/securestore/
What do you think about it ?
Maybe it will be a major release.

@Cretezy will this be merged soon ?

@leviathan88
For the moment, you can use directly the fork in your package.json :
"redux-persist-expo-securestore": "https://github.com/YoshiYo/redux-persist-expo-securestore.git",
If the merge is too long, I will publish a new repository to update this one.
Tell me directly on my fork if there is a problem with this version. On my side, I don't have the expo warning anymore.

thanks @YoshiYo
let's wait a little bit more and then check it out

Hi, sorry for the delay. Haven't used Expo for a while so will need to setup a little demo app to test these changes. I should be able to get to it by the weekend.

Just published v2.0.0 with this!


Style, PostgreSQL version, user editing

Do these in order:

[x] Add PureCSS to layout
[x] Make a page for user editing (just password editing)
[x] Make a comment or migration or both for setting an initial account
[ ] make a secret option for turning user creation on and off
[ ] make admin accounts a thing and able to edit other users
[ ] Make a PostgreSQL version

[ ] Make the PostgreSQL version take over on the master branch, and make a MySQL branch for the MySQL version!

I've decided screw admin stuff on here for now because I don't really care and need to figure out a better way of integrating things. :/


Memorial pages matching old urls

Need to cover the following patterns:

[x] http://ueaglider.uea.ac.uk/DIVES/pilotting.php?missionNo=59&gliderNo=558&diveNo=463
[x] http://ueaglider.uea.ac.uk/DIVES/science.php?dive=46876
[x] http://ueaglider.uea.ac.uk/DIVES/monitor.php?dive=46876
[x] http://ueaglider.uea.ac.uk/DIVES/index.php?dive=46876
[x] http://ueaglider.uea.ac.uk/DIVES/index.php

Resolved in 34c8db3