"Offline is Off" functionality stopped working in 1.7.0-beta.45

Prerequisite

[X] Yes, I've read the readme completely.

Cache

[X] Yes, I've cleaned accessory cache and the issue still exists.

Version
1.7.0-beta.4545
Device Infomation JSON File
"projectType": "2",
        "appSchema": "smartlife",
        "debug": true,
        "deviceOverrides": [
            {
                "id": "527653045002915e7060",
                "schema": [
                    {
                        "code": "switch_led",
                        "onGet": "(device.online && value)"
                    }
                ]
            },
            {
                "id": "7730722384cca8888c49",
                "schema": [
                    {
                        "code": "switch_led",
                        "onGet": "(device.online && value)"
                    }
                ]
            },
            {
                "id": "773072238cce4ecbd3c3",
                "schema": [
                    {
                        "code": "switch_led",
                        "onGet": "(device.online && value)"
                    }
                ]
            }
        ]
    }
}

Device Control Mode
Standard Instruction
Logs
26/05/2023, 20:59:31FTTUYATuyaPlatformLoading accessory from cache: EN Ceiling Light
26/05/2023, 20:59:31FTTUYATuyaPlatformLoading accessory from cache: Smart Star Projector
26/05/2023, 20:59:31FTTUYATuyaPlatformLoading accessory from cache: IKEA SKAFTET Cord Lamp
26/05/2023, 20:59:31FTTUYATuyaPlatformLoading accessory from cache: BA Ceiling Light
26/05/2023, 20:59:31FTTUYATuyaPlatformLoading accessory from cache: LT Temperature Humidity Sensor
26/05/2023, 20:59:31FTTUYATuyaPlatformLoading accessory from cache: GR Temperature Humidity Sensor
26/05/2023, 20:59:31FTTUYATuyaPlatformLoading accessory from cache: CL Ceiling Light
26/05/2023, 20:59:31FTTUYATuyaPlatformLoading accessory from cache: WiFi Temperature & Humidity Sensor
26/05/2023, 20:59:31FTTUYATuyaPlatformLoading accessory from cache: WiFi Temperature & Humidity Sensor 2
26/05/2023, 20:59:31FTTUYATuyaPlatformLoading accessory from cache: Water detector
26/05/2023, 20:59:31FTTUYATuyaPlatformLoading accessory from cache: BA Leak Sensor
26/05/2023, 20:59:31FTTUYATuyaPlatformLoading accessory from cache: ES Ceiling Light
26/05/2023, 20:59:31FTTUYATuyaPlatformLog in to Tuya Cloud.
26/05/2023, 20:59:31FTTUYATuyaPlatform[TuyaOpenAPI] Login to: https://openapi.tuyaeu.com
26/05/2023, 20:59:31FTTUYATuyaPlatformStart MQTT connection.
26/05/2023, 20:59:32FTTUYATuyaPlatformFetching home list.
26/05/2023, 20:59:32FTTUYATuyaPlatformGot home_id=14982579, name=FT Home
26/05/2023, 20:59:32FTTUYATuyaPlatformFetching device list.
26/05/2023, 20:59:33FTTUYATuyaPlatformFetching scene list.
26/05/2023, 20:59:33FTTUYATuyaPlatformGot 12 device(s) and scene(s).
26/05/2023, 20:59:33FTTUYATuyaPlatformDevice list saved at /var/lib/hoobs/fttuya/persist/TuyaDeviceList.eu1597451691303bRbj6.json
26/05/2023, 20:59:33FTTUYATuyaPlatformRestoring existing accessory from cache: LT Temperature Humidity Sensor
26/05/2023, 20:59:33FTTUYATuyaPlatformRestoring existing accessory from cache: WiFi Temperature & Humidity Sensor
26/05/2023, 20:59:33FTTUYATuyaPlatformRestoring existing accessory from cache: WiFi Temperature & Humidity Sensor 2
26/05/2023, 20:59:33FTTUYATuyaPlatformRestoring existing accessory from cache: ES Ceiling Light
26/05/2023, 20:59:34FTTUYATuyaPlatform[ES Ceiling Light] Light type: RGBCW
26/05/2023, 20:59:34FTTUYATuyaPlatform[ES Ceiling Light] Adaptive Lighting disabled.
26/05/2023, 20:59:34FTTUYATuyaPlatformRestoring existing accessory from cache: GR Temperature Humidity Sensor
26/05/2023, 20:59:34FTTUYATuyaPlatformRestoring existing accessory from cache: Water detector
26/05/2023, 20:59:34FTTUYATuyaPlatformRestoring existing accessory from cache: BA Leak Sensor
26/05/2023, 20:59:34FTTUYATuyaPlatformRestoring existing accessory from cache: BA Ceiling Light
26/05/2023, 20:59:34FTTUYATuyaPlatform[BA Ceiling Light ] Light type: RGBCW
26/05/2023, 20:59:34FTTUYATuyaPlatform[BA Ceiling Light ] Adaptive Lighting disabled.
26/05/2023, 20:59:34FTTUYATuyaPlatformRestoring existing accessory from cache: CL Ceiling Light
26/05/2023, 20:59:34FTTUYATuyaPlatform[CL Ceiling Light] Light type: RGBCW
26/05/2023, 20:59:34FTTUYATuyaPlatform[CL Ceiling Light] Adaptive Lighting disabled.
26/05/2023, 20:59:34FTTUYATuyaPlatformRestoring existing accessory from cache: EN Ceiling Light
26/05/2023, 20:59:34FTTUYATuyaPlatform[EN Ceiling Light ] Light type: RGBCW
26/05/2023, 20:59:34FTTUYATuyaPlatform[EN Ceiling Light ] Adaptive Lighting disabled.
26/05/2023, 20:59:34FTTUYATuyaPlatformRestoring existing accessory from cache: IKEA SKAFTET Cord Lamp
26/05/2023, 20:59:34FTTUYATuyaPlatform[IKEA SKAFTET Cord Lamp] Light type: CW
26/05/2023, 20:59:34FTTUYATuyaPlatform[IKEA SKAFTET Cord Lamp] Adaptive Lighting disabled.
26/05/2023, 20:59:34FTTUYATuyaPlatformRestoring existing accessory from cache: Smart Star Projector
26/05/2023, 20:59:34FTTUYATuyaPlatform[Smart Star Projector] Light type: RGB
26/05/2023, 20:59:34FTTUYATuyaPlatform[Smart Star Projector] Adaptive Lighting disabled.

Other Infomations
Since the latest release, the plugin seems to be ignoring my configuration for the Offline if Off advanced feature.

what does "ignoring" means? what's the behavior for your device in beta.45?

@0x5e I mean that the plugin no longer uses/acknowledges this configuration, so the devices I have asked the plugin to show was offline when they are off are just showing as offline (No Response) in HomeKit.  This did not happen in the previous release - the devices were showing as off if they were offline as expected by the option.

@0x5e so using the offline is off feature is indeed disregarded? Would you not consider this to be a bug?  Can I rollback to a previous version where the offline is off feature is working?

I didn't mean to delete this feature, the code is still here, but the "offline as no response" feature is a more generic solution, support all products, without any config, and exactly the same behavior as official homekit device. It has higher priority in homekit so any other status will just be ignored.
Can you let me know why you still want "offline as off" feature instead of "offline as no response"?

@0x5e for the reason that I thought that the feature was implemented originally - to stop HomeKit "warning" me that I have devices with no response each time I open the app - this can cause me to miss non-tuya devices that have "no response" that do actually require my investigation - e.g. an unplanned power outage/faulty device. It also seems that dynamic lighting does not work on these "no response" devices any more when I do actually turn them on, and it used to (after a minute) in the previous build.

This is the philosophy of HomeKit, by "deliberately not offering any practical way for accessories to signal to HomeKit that they are temporarily not operational". the accessory should always be reachable and fully operational.
There's the explanation about this:
https://github.com/homebridge/HAP-NodeJS/wiki/Presenting-Erroneous-Accessory-State-to-the-User
Adaptive Lighting is disabled by default since beta.42, it doesn't support all device, you can enable it manually.

@0x5e Thank you.  Okay, is there documentation for how to enable adaptive lighting manually in the latest release notes?  I do think I would prefer the "offline is off" feature to be reimplemented, and in the meantime what version (including offline is off and adaptive lighting) is the best version for me to roll back to?  Thanks,Tom

https://github.com/0x5e/homebridge-tuya-platform/blob/develop_1.7.0/ADVANCED_OPTIONS.md#configuration
You can use beta.44 with manual config, or beta.41 without config.

Thanks @0x5e, I'm back to .41 with adaptive lighting and offline is off working again, I'd really recommend these features are are carried forward in some way. Tom


Base64 encode/decode constant strings

Currently, there's a restriction that constant strings can only be in UTF8 format, because only UTF8 strings can be directly stored into a class file. In GHC, you can have arbitrary byte strings like so:
"\255\132\255"#

but not all byte strings are valid UTF8, and hence those will cause panics in the compiler right now.
We can get around this by Base64 encoding the byte string and putting in in the class file, and making sure that we Base64 decode it at runtime when loading the string. The impact of this change will just be a slight performance hit during startup time of the program for string-heavy programs, so it's not that bad. The main gain is better compatibility with Hackage.

This is not relevant anymore and the original problem has been solved in a different way.


Assist Endel with transfer

Endel from https://opencollective.com/colyseus/ needs help to transfer his collective funds to a Paypal address. There is more info on the email titled "Transfer to PayPal".

done


A simple example to showcase autodiff

We could have possibly the following examples:

[ ] A simple U-Net (pulled from fastMRI dataset possibly), trained? @Lenoush can you handle this in your free time?
[ ] A network built with DeepInv library for some already preset reconstruction (PnP possibly?) @matthieutrs can you handle this this in your free time? I think you can also have something basic passed to @Lenoush and she can work on it forward.
[x] A simple example where we could learn the trajectory possibly? @alineyyy can you handle this in your free time? #167
[ ] Add additional support for learning a sampling pattern in 2D with straight line readouts. Perhaps we can do it while supporting stackNUFFT on torch?

Beware, there is also autodiff for the image/k-space data (the one that is only currently supported) This could also be show cased.

I really seemed to have clicked open issue too soon. I had written so much more xD. Lemme update

@AsmaTANABEN , can you have a look at the notebook shared in mattermost and see if we can get this on mri-nufft?

Reopening as it got closed by mistake :P

I added that we could have additional support for learning a sampling pattern in 2D with straight line readouts.
Perhaps we can do it while supporting stackNUFFT on torch? @paquiteau is this possible? Can we pass the backprop gradients through the stacked NUFFT from the internal plane NUFFT?

On second thoughts, we would need 176 2D NUFFT then num_samples number of FFTs, I think a simple straightforward 3D NUFFT can be easier. @paquiteau to comment. I think in your usecase for fMRI it is more useful.

after rethinking about it, stacked nufft does not have autodiff wrt to trajectory. It does support the autodiff wrt to data however.


Steam mobile authenticator

HI, there!
What about the subj?
It seems steam bot are not gonna work now, because of subj. I red people wrote libraries on C# to pass through. Does someone plan to work out the way to use this perfect libraries with subj?

#121


Inaccurate xyz to lab.

The constants used in xyz to lab conversions are not accurate...
As an example, places where we use 0.008856 are actually supposed to be (6/29)^3. There are a lot more constants that are not accurate. I didn't go through all of them.
This has caused problems for me because I'm building a color-related library that's implemented in multiple languages. It converts rgb to lab color space, do some basic arithmetic and then convert back to rgb. (Yes, I used the xyz.lab.raw version) For javascript, I used color-convert as a dependency. And this has produced inconsistent results for different implementations.

There have been a number of PRs regarding this in the past, do you have some reference for the (6/29)^3 figure? I'm not an expert with lab/xyz.
If this is truly the case, happy to accept a PR that constructs the constant at boot time and uses that instead of the hardcoded value.

Ah actually that's pretty well documented. Neat, thanks for the heads up. Will fix.

The tests are using the default (rounded) versions of calls. Probably best, anyway, given IEEE inaccuracies, especially at such a fine-grained level.

Sorry. Typo. 1/3*(6/29)^(-2)
Reference: https://en.m.wikipedia.org/wiki/CIELAB_color_space

Too lazy to figure out what's wrong with rgb to xyz. Feel free to close this once 1/3*(6/29)^(-2) is fixed.
Things I used for manual testing the convertion:

https://www.easyrgb.com/en/convert.php#inputFORM
https://docs.rs/palette/latest/palette/

To document my findings for rgb to xyz conversion:
For #FABD2F, both external tool produced xyz: 58.139, 56.929, 10.615 (with rounding)
While color-convert produced:  58.13502713482779, 56.924421562846604, 10.61278897125713
I don't think it's too fine grained though. My library converts rgb to lab color space, do some basic arithmetic and then convert back to 0-255 integer version of rgb. And some rgb components is off by 2. Too lazy to write a minimum example though.
If you are worried about IEEE inaccuracy, maybe set a difference threshold when asserting. Threshold of 1 is not justified just to account for IEEE inaccuracy. But not a high priority though.


Fix for issue #1085 and #1218

Summary of the Pull Request
Enhancement: FancyZones - Display zone numbers #1085 and Fixed: Design Change: Default Enter Action #1218

References
https://github.com/microsoft/PowerToys/pull/1381

PR Checklist

[] Applies to #xxx
[] CLA signed. If not, go over here and sign the CLA
[] Tests added/passed
[] Requires documentation to be updated
[] I've discussed this with core contributors already. If not checked, I'm ready to accept this work might be rejected in favor of a different grand plan. Issue number where discussion took place: #xxx

Detailed Description of the Pull Request / Additional comments

Validation Steps Performed

I've cherry-picked the commits from this PR into a new branch https://github.com/microsoft/PowerToys/commits/dev/enricog/sqrlm
We will merge the PR from that branch when we will complete the polishing.
There is currently a problem with the numbers disappearing when editing a columns/rows/grid/priority template (it works fine with custom layouts).

I've cherry-picked the commits from this PR into a new branch https://github.com/microsoft/PowerToys/commits/dev/enricog/sqrlm
We will merge the PR from that branch when we will complete the polishing.
There is currently a problem with the numbers disappearing when editing a columns/rows/grid/priority template (it works fine with custom layouts).

Please let me know which case the numbers disappear

Please let me know which case the numbers disappear

When you select a template (columns, rows, grid or priority) and then click on Edit selected layout, when moving the mouse over different zones, the number only shows on that zone but not on the others.

Hi @sqrlmn
since we wanted to include your work in the 0.16 release we went ahead and made the final polishing in https://github.com/microsoft/PowerToys/pull/1726
Your commits have been preserved: https://github.com/microsoft/PowerToys/pull/1726/commits
Thank you a lot for the contribution!


Bing Streetside shows white box

Description
See https://github.com/openstreetmap/iD/issues/9230
Noticed this in iD and I wondered if this issue also existed in Rapid - it does.
Version
1.1.9
What browser are you seeing the problem on? What version are you running?
Chrome / Win 10
Steps to reproduce
No response
The browser URL at the time you encountered the bug
https://mapwith.ai/rapid#background=Bing&datasets=fbRoads,msBuildings&disable_features=boundaries&map=20.57/51.50158/-0.12402&photo_overlay=streetside

This is also https://github.com/openstreetmap/iD/issues/9230


Delete bad anchors and fix e2e tests

While running the e2e tests, the CleanupNamespace command was hanging
because we weren't allowing deletion of anchors in the Conflict state;
we were just checking whether the would-be subnamespace had the correct
parent, but not whether the subnamespace anchor was actually in the 'ok'
state. This was resulting in HNC blocking the deletion of a bad anchor,
which we should always allow (this should have been done as a part of
PR #1151).
I also made two minor fixes to the e2e tests themselves:

Turn propagation on for new types with the --force option
Skip the network policy demo if we detect that network policies
aren't enabled on the cluster.

Tested: the e2e tests hang repeatedly and all the demo tests fail
without this fix; with this fix, they pass.

/assign @yiqigao217
/assign @rjbez17

/hold
Whoops, I reran the e2e tests and got a related failure from the namespace admission controller. This needs one more upgrade.

I don't think the e2e failure was caused by this change but I diagnosed and fixed it since I'm fixing other e2e problems anyway.
/hold cancel

I think this is a fairly small change so @rjbez17 I'm letting this go in now. LMK if you have any concerns.
/hold cancel


Add ObservedGeneration to the sub Resources

This patch does a few things:

it adds observedGeneration to the sub custom resources
it proposes to bump the observedGeneration at the beginning of the reconciliation loop (knative/serving#4937)
it checks, at the top level, if the observedGeneration matches with the metadata.generation assigned to the subCR(s); if it's the latest version, the condition is mirrored
before marking DeploymentReadyCondition as True in the subCRs, observedGeneration is compared with the generation of the Deployment/Statefulset
it updates the functional tests and bumps both lib-common/modules/test and lib-common/modules/common

recheck

recheck

recheck

recheck

recheck

recheck


Display list of GitHub users who have contributed to a page

Summary:
Boost recognition of documentation contributions by featuring contributor avatars as an author byline.
Example:
It would be useful to highlight who has contributed to each page in a hugo site by pulling the list of contributors from Github and featuring their avatars in the author byline. Microsoft Azure does this in their content under the title of each doc.

Thoughts on how this should be implemented:

Surface document contributors by displaying a byline of the following format: Updated [date] by [avatar]. Contributors: [avatars in reverse chronological order]]
Updated date:

On hover, display the title of the commit
On click, open the commit in GH

Avatar behavior (all)

On hover: Display the user's display name from GitHub profile (not their handle) (Jared Bhatti, not jaredb)
On click: Open the GH user profile

Contributor avatars

Display 5 most recent contributors (after the most recent contributor) in reverse chronological order
If doc has more than six contributors, display a link: All
On hover, display a tooltip: "See all N contributors"
On click: Open the document on GitHub

This should be handled with a partial that calls the GitHub API with resources.GetRemote. For example:
git clone --single-branch -b hugo-github-issue-5070 https://github.com/jmooring/hugo-testing hugo-github-issue-5070
cd hugo-github-issue-5070
hugo server

The "list-github-contributors" partial renders something like this...

...where each avatar is linked to the user's GitHub account.


[BUG] Build Issue in Xcode 14.1 Beta

New issue checklist

[x] I have reviewed the README and documentation
[x] I have searched existing issues and this is not a duplicate
[x] I have attempted to reproduce the issue and include an example project.

General information

Paywall version: 2.4.1
iOS version(s): 16.1
CocoaPods/Carthage version (if applicable):
Xcode version: 14.1.0 Beta
Devices/Simulators affected: All?
Reproducible in the demo project? (Yes/No): I made a totally fresh project
Related issues: n/a

Describe the bug
An empty project won't build in Xcode 14.1 once Superwall is added as a dependency.
I noticed this in my real project so I created an empty project with no new code and build fails as soon as Superwall is a dependency
/Applications/Xcode-14.1.0-Beta.app/Contents/Developer/Platforms/iPhoneSimulator.platform/Developer/SDKs/iPhoneSimulator16.1.sdk/System/Library/Frameworks/GameController.framework/Headers/GCPhysicalInputElement.h:44:73: 'GCPhysicalInputElement' is unavailable: not available on iOS
This is probably an Apple bug, but if it's not resolved we will have to remove Superwall.
Steps to reproduce

Open Xcode 14.1.0 Beta
Create a new iOS app project (I used the storyboard template)
Add https://github.com/superwall-me/paywall-ios as an SPM dependency
Build the project

Other Information
I've attached the full build log in case it's helpful. Here's a screenshot as that's probably easier to look at:

I've googled for this error and can't find anything.
Build Superwall 14.1_2022-09-18T10-49-27.txt

As a temporary workout around to allow development on Live Activities to continue whilst this gets resolved I'm using Xcode 14.0 b5 as this is the last build that supports Live Activities AND builds Superwall.

Hey @lewis-smith, thanks for the detailed bug report.
This is a bug with Apple's own GameController module. We support GameController within Superwall which is why it's failing when you try to import Superwall. Importing GameController into a blank project gives me the same error. I've filed a bug report with apple (FB11538209), but it would be good if you could also file a bug report. I've also mentioned it in the Apple developer forums: https://developer.apple.com/forums/thread/714980
We''ll keep an eye on that and if it's still not fixed by the time Xcode 14.1 comes out we may have to rethink GameController support.

Thanks @yusuftor, I'll file a feedback with Apple.
Just to update anyone else, Live Activities have changed quite a bit since 14.0 b5 so it's not possible to code for 14.1 using that Xcode :(

Filed a dupe and sent a message to our Dev Relations rep. Fingers crossed it gets fixed soon. Thanks for such a quick and diligent response.

Great, thank you! Please update me on here if you hear anything and I'll do the same.

This is resolved in beta 3. Other people in the forum thread that @yusuftor created confirm too.


Add search and filter options for all collections

Search and filter options needed for All collections like blog posts, events, code challenges. It can be done by generating json data with jekyll and using them in front side using javascripts.

Also pagination problem at #4 can be solved using above solution !

It will be easier using alpine.js #35 for this solution !

Add filters based on categories and date

Separate FAQs based on category.


Bug: afterTick/beforeTick is called too often

This is my example code:
#include <chrono>

std::chrono::time_point<std::chrono::system_clock> last;

ModFunction void Initialize(HookManager* hookManager, Amethyst::EventManager* eventManager, InputManager* inputManager)
{
    eventManager->afterTick.AddListener(afterTick);
}

void afterTick(){
    std::chrono::time_point<std::chrono::system_clock> now = std::chrono::system_clock::now();
    auto duration = now - last;

    // millis since last tick (for 20 ticks per sec ~ 50ms difference)
    auto millis = std::chrono::duration_cast<std::chrono::milliseconds>(duration).count();

    Log::Info("MILLS SINCE LAST TICK: {}", millis);

    last = std::chrono::system_clock::now();
}

This code gives me 16/17ms per tick, which roughly means it is executed 60 times per second,
but there are normally just 20 ticks in a second
Using beforeTick doesn't really change anything.
It seems like it uses the wrong function for ticking

By the fact that this is called every 16ms, that definitely would line up to be called every frame running at 60 fps (1/60s = 16.3ms)
Which makes sense as we've hooked Minecraft::update rather then a tick function

that's what I thought

It would be good to have the Minecraft::update function still around, just nder another name...

https://github.com/FrederoxDev/Amethyst/commit/9482827644cd3155fdcc3c31ecdf078ffb96983d
Renamed to be more accurate here

we'll get the tick func beack one day, right?


Publish main to live on 11/2 @ 10:30 am

Learn Build status updates of commit d4cc27d:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/docs-conceptual/app-only-auth-powershell-v2.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/New-DistributionGroup.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/New-UnifiedGroup.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Set-DistributionGroup.md
:white_check_mark:Succeeded
View (exchange-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel


Fixed Header Issues

http://codepen.io/xstortionist/pen/GordGp
Noticed when testing that swipe doesn't work on fixed headers, you must swipe outside of the header in order for it to work properly.
Also noticed when swiping there is latency within the animation of the fixed header until the class is added to the header.
Thought I'd pass this along and hope it helps.

Looks like the only way to fix this issue is to use jquery to animate rather then css. Using jquery to animate will not be as smooth transition but doesn't seem like browsers have caught up with the translate of certain elements.

@xstortionist do you have an example of animating with jQuery? Couse I have the same problem as you with the lag of the fixed header.

@Sandstedt Hi there, I do not have example of animating with jquery as I simply created my own off canvas slideout menu.

Same issue with the latency...  any suggestions?

Fixed by wrapping header, content and footer within a wrapper and using that wrapper with Slideout.

Seems to be an issue with mozilla based browsers. I have seen it on other sites as well.

I'm currently trying to implement a fixed 'sticky' header. I have implemented this before and never really had trouble before. Right now I'm trying to implement a sticky header while also using this Slideout resource. I noticed that the class slideout-panel has a property will-change: transform; that causes fixed elements to not position correctly. Is anybody able to take a moment to help determine a work around? I almost want to just remove that from the styles or override it but am not sure what it does exactly.

@nstanard would be best to upload something to codepen, kinda hard to help without seeing an example.
Basically, will-change gives a hint to the browser so it can optimize the page ahead of time.

@xstortionist I'll create a small demo so it can help highlight the issue. However - in my 'sticky header' styles I just changed that value to auto and all is good.

@nstanard if you look at the pen I provided, I have a sticky header but I'm using flex design, which is more powerful and flexible than position: fixed.

Good point - I see that now.

@nstanard if u upload just a quick preview of what you are experience, I can help. I have actually moved away from this plugin and created my own slide menu which I feel is better than sliding the entire page out of view.

@nstanard Sorry, I actually changed it to the sticky position based on another thread to show how to have left and right menu sliding out. On my new framework, I simply use flexbox. Sorry for the confusion.
You shouldn't have a problem with onscroll change position to fixed. I will have better idea once I can see what you are seeing.

I'll have to put together a demo - but that'll probably happen tonight at the earliest. I'll get back to you with a link though.

I updated the demo and docs to use slideout with a fixed header:

http://codepen.io/pazguille/pen/ZBxdgw
https://github.com/Mango/slideout#how-to-use-slideout-with-a-fixed-header

Sorry to disturb the peace in this old topic. But just so you guys (and possibly other web travelers) are aware.. as far as i can tell there is no need for the event listeners. Its solvable purely by css.
The markup needs a small anjustment, the header position needs to be sticky and the overflow for the body (etc) needs to be clip.
See the Codepen fork


chore(wrappers): add wrapper for plugin-feedback and plugin-feedback-backend (RHIDP 3549)

Which issue(s) does this PR fix
Migrate or support the migration of feedback plugin to backstage/community-plugins
PR acceptance criteria
Please make sure that the following steps are complete:

[ ] GitHub Actions are completed and successful
[ ] Unit Tests are updated and passing
[ ] E2E Tests are updated and passing
[x] Documentation is updated if necessary (requirement for new features)
[ ] Add a screenshot if the change is UX/UI related

How to test changes / Special notes to the reviewer

/test e2e-tests

As far as I can tell the feedback plugin was never in the showcase as of 1.2 or 1.3 CI.
If this is one of the third-party plugins owned by another RH team, and they were not previously included as peer deps in the showcase, then I'm not sure we need this PR to add a dynamic wrapper.
Can you confirm with @riginoommen @deshmukhmayur @yashoswalyo ?

/rebase needed for the change

Closing - we don't plan to bundle any new plugins in RHDH at this time. If you want to be bundled please talk to @christophe-f


Add Devuan

Blocked because this would take the repo above the reusable workflow limit, 20. This would make 21.

CI doesn't scale past it's current scope, will need to rework it if more distros are to be included.


fix menubar focus trap

Menubars shouldn't be part of the tab order and don't usually respond to tab when inside

Is the focused menu item supposed to look like this?

It's orange in every theme and looks different from insiders, that makes me think it's a bug.

@sbatten please bring up to date with master


fix: service

Description of PR

It is fixing the service name regex bug.
What component(s) does this PR affect?

[x] Back-End (Database, Detection/Configuration/etc. Microservices)
[ ] Front-End (Flask, API, UI, etc)
[ ] Monitor (RIPE RIS, BGPStream RV/RIS/CAIDA, etc.)
[ ] Docs
[ ] Build System

Does the PR require changes on other components? If yes, please mark the components:

[ ] Back-End (Database, Detection/Configuration/etc. Microservices)
[ ] Front-End (Flask, API, UI, etc)
[ ] Monitor (RIPE RIS, BGPStream RV/RIS/CAIDA, etc.)
[ ] Docs
[ ] Build System

Related Issue

Resolves #
Solution

Type

[ ] 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 change)
[ ] Docs update
[ ] None of the above

Checklist:

[ ] I have read the contributing guide and my code conforms to the guidelines.
[ ] This change requires a change in the documentation.
[ ] I have updated the documentation accordingly.

@CuriouzK0d3r can you check on gitpod now and let me know if it works? I will merge if yes.

works for the new images, merging


NDRS-740: Add utilities for testing emergency upgrades

This adds a binary generating a global state upgrade file based on the supplied parameters and an nctl command, nctl-emergency-upgrade.
Intended use:

Run an nctl network with some active and some reserve nodes.
Stop the network at some point.
Run nctl-emergency-upgrade version=VERSION_TO_UPGRADE_TO era=ACTIVATION_POINT hash=GLOBAL_STATE_HASH_BEFORE_UPGRADE - this should upgrade all the nodes and add global state upgrade files to their configs for the supplied protocol version.
Start all the nodes (both the previously active ones and the reserve ones) - the reserve nodes should become validators after the upgrade.

~I'm making this a draft PR, since I ran into problems when trying to execute such an emergency upgrade - the old nodes realized they were no longer validators (yay!), but the new ones had trouble syncing (boo...).~

I'm making this a draft PR, since I ran into problems when trying to execute such an emergency upgrade - the old nodes realized they were no longer validators (yay!), but the new ones had trouble syncing (boo...).

Outdated description of the PR?

@momipsl do the nctl changes in this PR look ok to you?

@EdHastingsCasperLabs reviewing NCTL changes now

@EdHastingsCasperLabs NCTL changes could be improved but nothing that blocks this PR - i.e. some follow-up work to apply spit & polish.

@fizyk20 could you get this merged when convenient, then followup w/ @momipsl to discuss next steps on addressing his aesthetic concerns in a later PR? This PR has aired out long enough, methinks.

@EdHastingsCasperLabs Sure. I've actually started working on it, but a few things have changed on dev in the meantime and this no longer works :grimacing: I have to debug at least one more issue before it's ready again.

bors r+


Corrected the Component::__isset() docs

__isset() returns true if the property is set; not if it's null.

well, it is also considered not to be set if it is null, just like php isset().
Could be made clearer by saying Checks if a property value is set i.e., defined and not null.

@cebe I don’t agree that it is the job of this method’s docs to explain what “set” means.
Take it or leave it. Just offering a correction, as currently the docs are saying the opposite of what the purpose of __isset() is.

https://github.com/brandonkelly/yii2/blob/86504e6ee0308447ea9a71d0a9af1be685272b47/framework/base/Component.php#L235

Magic method __isset() serves as a response for isset() PHP operator.
For the regular variables this operator considers null value as not set:
$var = null;
var_dump(isset($var)); // outputs 'false'

@klimov-paul That doesn’t mean that it only applies to null values though! It also (and primarily) is used to determine if a variable/property exists at all.
@lynicidn Are you pointing out that there is precedence for mentioning "null" in the docs? I disagree; in __unset()’s case, the documentation is (pretty much) accurate: Component::__unset() will actually throw a InvalidCallException if you try to unset a property that doesn’t exist or is read-only. But __isset() on the other hand will simply return true or false depending on whether the variable exists at all and is set to a non-null value (generally).

@cebe I don’t agree that it is the job of this method’s docs to explain what “set” means.

There is nothing wrong with being explicit about the meaning of something even if it may be clear in some context.
Merged and adjusted it a bit, thanks!


docs: add TiKV adopters and Roadmap

What have you changed? (mandatory)
I have added the TiKV adopters and copied the TiKV roadmap to this repo so that TiKV can have its own roadmap
What are the type of the changes? (mandatory)
Improvement
How has this PR been tested? (mandatory)
N/A
Does this PR affect documentation (docs/docs-cn) update? (mandatory)
No.
Does this PR affect tidb-ansible update? (mandatory)
No.
Refer to a related PR or issue link (optional)
https://github.com/pingcap/docs/pull/525
Benchmark result if necessary (optional)
No.
Add a few positive/negative examples (optional)
N/A

Made some edits to the adopters.md table.
@queenypingcap PTAL re formatting and cleanliness.
Content LGTM.

LGTM


Support for IN865_867

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

865_867 addition was tested to work on two channels but 865985000 is showing a few issues. I shall debug and update soon.

@akshaim thanks! Should we wait for your debugging and test results?

Works now. Uplink and Downlink tested.

It is fine as it is, but I would have written this function differently as there is no need for the loop and switch - they just complicate the algorithm without saving any memory.
void TheThingsNetwork::configureIN865_867()
{
  sendMacSet(MAC_ADR, "off"); // TODO: remove when ADR is implemented for this plan
  sendMacSet(MAC_RX2, "2 866550000"); // SF10

  // Disable the three default LoRaWAN channels
  sendChSet(MAC_CHANNEL_STATUS, 0, "off");
  sendChSet(MAC_CHANNEL_STATUS, 1, "off");
  sendChSet(MAC_CHANNEL_STATUS, 2, "off");

  // Channel 3
  sendChSet(MAC_CHANNEL_DCYCLE, 3, "799");
  sendChSet(MAC_CHANNEL_FREQ, 3, "865062500");
  sendChSet(MAC_CHANNEL_DRRANGE, 3, "0 5");
  sendChSet(MAC_CHANNEL_STATUS, 3, "on");

  // Channel 4
  sendChSet(MAC_CHANNEL_DCYCLE, 4, "799");
  sendChSet(MAC_CHANNEL_FREQ, 4, "865402500");
  sendChSet(MAC_CHANNEL_DRRANGE, 4, "0 5");
  sendChSet(MAC_CHANNEL_STATUS, 4, "on");

  // Channel 5
  sendChSet(MAC_CHANNEL_DCYCLE, 5, "799");
  sendChSet(MAC_CHANNEL_FREQ, 5, "865985000");
  sendChSet(MAC_CHANNEL_DRRANGE, 5, "0 5");
  sendChSet(MAC_CHANNEL_STATUS, 5, "on");

  sendMacSet(MAC_PWRIDX, TTN_PWRIDX_IN865_867);
}

Thanks. I have updated the code as per your recommendations and it works! Can you help me implement ADR on the channel?

What is the reason to disable the three default LoRaWAN channels?
Regarding the loop; this is consistent with other frequency plans (and it may reduce flash memory).

Any news @akshaim @jpmeijers ?

On the RN module with european firmware one can not change the frequency of the three default channels. For India we therefore have to disable these three channels to not use them, and rather use another three to configure the three default Indian channels.

Even though the original loop method is consistent with the other frequency plans, it complicates the code and in this case do not save any space. In this case there is no constant step between 8 channel, so using a loop to add the constant step doesn't make sense. Configuring the three channels one by one by hand is much clearer and does not really use much more program space than the loop. In fact, I believe it will use less instructions as we are not using the loop. For-loops are notorious to be compiled to a lot of instructions and are quite often unrolled by the compiler.

Also note that we plan to rewrite the frequency plan logic to save program space. So let's not worry too much about it now as we will rethink and re-implement it later.
See branch:
https://github.com/TheThingsNetwork/arduino-device-lib/tree/feature/use-less-program-space

@johanstokking Please see the following snap from the RN2483 command reference manual. Channel 0-2 can be used only on default channels.
https://imgur.com/a/sSO9SMp


Do not close entry page if it failed to save or delete.

- Summary
Fixes #555:

Currently, if the user tries to save an entry, but the backend has an error, the page still tries to redirect back to the dashboard. However, since the page is not saved yet, the user is presented with a popup: "Are you sure you want to leave this page?". If they click "OK", their content is lost. After the popup is closed (OK or Cancel), the red error notification show up: "Failed to persist entry: API_ERROR: ...".

- Test plan
Manually tested with the test-repo backend by returning a rejected promise from persistEntry and deleteFile.
- Description for the changelog
Do not close entry page if the entry failed to save or delete.
- A picture of a cute animal (not mandatory but encouraged)

@Benaiah Should be updated now -- makes for a simple PR!


Fetched data does not get evaluated correctly (selectize empty)

Hi! Thanks for the module again!
So I have a Movie model that has _id, name, rating, etc.
When I have some pre-populated values i.e.
vm.selectedMovies = [{_id: '1', name: 'Bang'}, {_id: '2', name: 'Whatevs'}];

the selectize appears empty (i.e. the model binding does not seem to evaluate). I tried even preloading the collection from the server (to have all options available) but with no luck. My config options are as follows:
vm.movieSelectConfig = {
  valueField: '_id',
  labelField: 'name',
  searchField: 'name',
  sortField: 'name',
  // create: true,
  hideSelected: false,
  closeAfterSelect: true,
  preload: true,
  render: {
    option: function(movie) {
      return '
' + movie.name + '
';
    }
  },
  load: function(query, callback) {
    // if (!query.length) {
    //   return callback();
    // }
    return Movie.searchByName({
      'name': query
    }).$promise.then(function(results) {
      callback(results);
    });
  }
};

Any hints?

So what I did for now (but seems like a hack) is the following:
onInitialize: function(selectize){
  _.each(vm.currentUser.movies, function(movie) {
    selectize.addOption(movie);
    selectize.addItem(movie._id);
  });
}

A quick fix that I've found around that is to use the default valueField and labelField. So I've used objects like: {text:'the label', value:'the value'} and it's working fine. I didn't have the time to investigate further but so it looks like somewhere in the directive we are always using the defaults valueField and labelField even if some others were specified in the configuration.

Well that would mean that someone should change the models/object to suit the selectize dropdown which is imo unacceptable..

+1 needs way to bind to objects properly

I had a similar issue. I found out that I was initializing $scope.config (whatever variable referenced in the config attribute of the directive) too late in my controller, so the binding didn't work: it is done only once, late updates are ignored.
I have just put the init of this scope variable in the body of the controller and it worked fine...

this seems to be a persistent issue with dynamic options.
i'm building out a list of columns from an uploaded CSV which is not available at the time of the config being loaded, since it has to be uploaded, and the scope variable is then updated with the list of columns, but I'm unable to see the selectize dropdown updated once the list of columns is available.

For our projects, I made a SelectizeService which wraps calls to the Selectize component exposed from onInitialize.
I let Selectize do the loading of its options, so it just manages the loading class and updates correctly the options.
I do it this way:
    /**
     * Starts to load the data (by calling the loadFunction) after setting the loading class on the Selectize component.
     * @see SelectizeService#finishLoading
     */
    service.initLoading = function(selectizeControl, loadFunction)
    {
        // loadingFinished is a function given by Selectize, to call with the results once they are fetched
        var wrappedLoadFuntion = function(loadingFinished)
        {
            // Memorize the given function
            selectizeControl._finalizeLoading = loadingFinished;
            // Do the data loading call
            loadFunction();
        };
        // load() sets the loading class on the control,
        // then call the given function with an argument which is a function to call with the results.
        selectizeControl.load(wrappedLoadFuntion);
    };

    /**
     * To call on data loading completion, with the control (to remove the loading class),
     * and the received data (optional if options are created by other means).
     */
    service.finishLoading = function(selectizeControl, data)
    {
        selectizeControl._finalizeLoading(data);
    };

A bit convoluted, but it works.

@machineboy2045 Any thoughts on this one?

Try the latest version. I patched a couple bugs yesterday. If that doesn't help I'll look into it some more.

@xeroxoid is this issue still present in v3?

Yep the issue is still present and it will always be until two-way object binding is in place. ATM you still have to pluck or set strings for values into setValue or addItem so you always need an onInitialize if you want to pre-select saved values.

Just found the following -
if my ng-model = "myModel"
following seems to work for me:
$scope.myModel = ["value1","value2","value3"]  as long as they are these are part of the model.

Just found the following -
if my ng-model = "myModel"
following seems to work for me:
$scope.myModel = ["value1","value2","value3"]  as long as they are these are part of the model.


Sitemap not indexing Beesblog

Hi
seems that the sitemap module is not indexing the blog module at all, is there an option somewhere I should be activating?

Perfect thanks. I had a look and it is indeed positioned under moduleRoutes. However when I clicked edit It takes me to the transplant section and under there, there is an option for gsitemapAppendURLs
Its not installed there by default.
If I use the transplant module function and add it into that hook as well, then it does indeed index the module, but here is the result of that indexing (I have only copied the blog portion of the sitemap)
2017-10-30T23:04:42+02:00
monthly
https://orren-life.com/Journal/Array
0.1
monthly
/img/beesblog/posts/1-post_default.jpg
https://orren-life.com/Journal/Array
0.1
monthly
/img/beesblog/posts/2-post_default.jpg
https://orren-life.com/Journal/Array
0.1
monthly
/img/beesblog/posts/3-post_default.jpg
https://orren-life.com/Journal/Array
0.1
monthly
/img/beesblog/posts/4-post_default.jpg
https://orren-life.com/Journal/Array
0.1
monthly
/img/beesblog/posts/5-post_default.jpg
https://orren-life.com/Journal/Array
0.1
monthly
https://orren-life.com/Journal/Array
0.1
monthly
/img/beesblog/posts/7-post_default.jpg
https://orren-life.com/Journal/Array
0.1
monthly
/img/beesblog/posts/8-post_default.jpg
https://orren-life.com/Journal/Array
0.1
monthly
/img/beesblog/posts/9-post_default.jpg
https://orren-life.com/Journal/Array
0.1
monthly
/img/beesblog/posts/10-post_default.jpg
https://orren-life.com/Journal/Array
0.1
monthly
/img/beesblog/posts/11-post_default.jpg
https://orren-life.com/Journal/category/Array
0.1
monthly
https://orren-life.com/Journal/category/Array
0.1
monthly
https://orren-life.com/Journal/category/Array
0.1
monthly
https://orren-life.com/Journal/category/Array
0.1
monthly
https://orren-life.com/Journal/category/Array
0.1
monthly
https://orren-life.com/Journal/category/Array
0.1
monthly
https://orren-life.com/Journal/category/Array
0.1
monthly
https://orren-life.com/Journal/category/Array
0.1
monthly

Just noticed I gave you the wrong advice. It does not add the module's routes to the sitemap. It should implement the gSitemapAppend hook or something like that. It does not implement that hook at the moment. Will add this feature to the next version.

not a problem at all. It works rally well, except for the array in the indexed URL. I am just glad it does index the site. My Prestashop site, using smartblog does not index at all.

What do you mean by "Except for the array"? Can you post a raw XML or link to one?

Sure, if you check my posts the url says https://orren-life.com/Journal/category/Array. That array should be the category name and not the word array.
Here is the link to the generated xml feed:
http://orren-life.com/1_en_0_sitemap.xml
(I have not yet loaded products)

It looks very very strange to me. I will have a look at the sitemap module. This might have to be fixed in that one.

Strange. I cannot reproduce it. Which sitemap module do you use?

And which version?

It is indeed weird. So I am using the default one for ThirtyBees, It just says Sitemap by ThirtyBees and the version is 4.0.1

Interesting behavior, but good to know it is only with the blog module. That narrows down the search for the cause.

lol yeah I hear you.

Fresh install of TB and Blog and I still get that array. Tried to see if I could figure anything out and only have these conclusions:
Line 439 $link['link'] = BeesBlog::getBeesBlogLink('beesblog_post', ['blog_rewrite' => $result->link_rewrite]);
If I change link_rewrite to id, then it does get the correct id of the relevant posts. and if I make the same mod to the category, then it gets the correct category id. Somehow the fact that the rewrite url is text is what returns the array word.
The other thing I found is the image url is not indexing properly either. it showing the relative link to the img directory instead of the full url
Wish I could be of further help, but I just dont know the codebase that well

Hey Michael,
Sorry to trouble, any chance you have made any progress into this? Is it the blog module or the sitemap module?

Great News I managed to fix it, Will send a pull request shortly

Interesting. Does the module need to be changed?

Hey Michael,
I did a small change in the beesblog.php file. I see that the location of the images for the blog is not correct and the last modified time does not display. But the seo friendly category link and teh post link itself works 100% correct.


SELECT TOTAL() with many rows will probably be too slow

The storage uses a TOTAL to track whether it's full, using:
https://github.com/ethereum/trin/blob/e2e96adc8ceee16774289f834681f8c4b4704e31/portalnet/src/storage.rs#L839
It's not totally unreasonable to hit a million rows eventually. Scanning 1M rows in a database could take a significant amount of time. This is a bigger concern, because the lookup happens regularly and inline for every insert.
Let's see how much faster we can make it by adding an index on the column. Normally, it doesn't help SUM, but in the very special case of summing the full index, it could. That's only if the query optimizer is smart enough though, per https://stackoverflow.com/a/437924/8412986
Try adding an index on the column. Then, run the TOTAL() query using an explain, to see if SQLite is smart enough to use the index https://www.sqlite.org/eqp.html
If so, let's make an index standard now, to have it be part of as many installs as possible. If not, we probably need to find a different way to keep track of the total content size. (It still may not be fast enough with the index, but it should buy us some time)

Huh, well I ran into a couple surprising things.
I generated a DB with 2M records and ran a sum, and it only took 125ms. At an average of 60kb per record (unscientifically determined from my tiny trin instance on history right now), 2M records is enough to cover 120GB of network data.
sqlite> select count(content_size) from content_metadata ;
2000033
sqlite> select total(content_size) from content_metadata ;
120002057551.0
Run Time: real 0.125 user 0.088267 sys 0.036119

125ms is less than I expected, although it's still too much to be running as regularly as we run it. So it's worth looking at the index.
Sure enough, it claims to use the index:
sqlite> explain query plan select total(content_size) from content_metadata ;
QUERY PLAN
`--SCAN content_metadata
sqlite> create index size_idx on content_metadata(content_size);
sqlite> explain query plan select total(content_size) from content_metadata ;
QUERY PLAN
`--SCAN content_metadata USING COVERING INDEX size_idx

Surprising to me, it only reduced the time to 75 ms:
sqlite> select total(content_size) from content_metadata ;
120002057551.0
Run Time: real 0.075 user 0.061966 sys 0.012523

I do think that most/all of this is getting loaded into ram, so it is probably the most optimistic possible test in terms of disk latency. I confirmed that running it on a ramdisk (where the sqlite file is only in ram) is only moderately faster in both cases, at 114ms and 70ms.

Right now, I'm not sure what to do with the information. The index seems like a no-brainer. Running a total() probably won't last forever, but it actually might last for long enough that it can be a problem we don't worry about till later...


OS.get_static_memory_usage() don´t return real memory, stucks at 1.1GB and later return negative number.

Godot version:

Godot 3.1 master at 29f2571662da0ee93ea2a632ec3b60aa26246563
OS/device including version:

Win 7 64 bit
Issue description:

Well, doing this test: https://github.com/Ranoller/Godot_crash_memory_test I find that this function: OS.get_static_memory_usage(), that calls Memory::get_mem_usage()  that returns from core\os\memory.cpp a date named uint64_t Memory::mem_usage... have a strange behavior:
This variable don´t actualize at the same time that real computer memory, it jumps by intervals until achieves 1.105.526 KB and later returns -2.115.684 KB... yes, a negative number! in this "jump" operating system momentarily grows godot memory use (I think that it try to give 2GB to godot) but then go to the normal allocating (use refered test to understand). The finish is godot crash at 2.2GB aprox....
Edit:
More test done and godot can allocate any amount of memory if it´s done in different arrays, since windows have certain caps and don´t allow to allocate more that about 2 GB in a single array (closes the process). I will actualize my test .... but i let the issue that is: Why OS.get_static_memory_usage() returns negative numbers....
Thanks for read.
Steps to reproduce:
Use refered test with windows task manager opened to compare real allocated memory with memory reported from godot.
Minimal reproduction project:

https://github.com/Ranoller/Godot_crash_memory_test

@RandomShaper I remember you did some changes to this code, think you could give it a check?

I think the issue is that OS::get_static_memory_usage and all in that family are returning int (32-bit signed integer) while the core memory routines return uint64_t.
That makes the reported amount wrap to negative beyond 2 GiB.
That's very easy to fix, but I have no computer right now. I'll be able to do it in a few days, though.


How to display each dicom file / each slice?

I'm trying to display each dicom file like the left panel in the picture below:

(source : https://www.dicomlibrary.com)
But I just can display the black squares like this, for example, I choose 4 dicom files, so 4 blacks squares were created.

So how can I display the image inside the square? Maybe it relates to the the camera and the scene, but I don't know how to make it. Can anybody help me? Thank you a lot!

You should be able to access the frame data via https://github.com/FNNDSC/ami/blob/master/src/models/models.frame.js#L203, then you can add it in an HTML img.

(btw the best option may be to generate those previews as jpgs on the backend if that is an option for you!)


TimeLLM takes a long time to setup training.

What happened + What you expected to happen
Hi,
I was tring to run the example code of the TimeLLM model https://nixtlaverse.nixtla.io/neuralforecast/models.timellm.html#timellm
It took almost 1 hour before actual training. In the terminal, it only shows "Seed set to 1". I checked the GPU, where there is no GPU usage and only memory being taken about 500MB(~gpt2 size). Then the training began, it took only ~10s. In the training, there was usual GPU usage. Last, it also took ~1 hour to wrap up(predict time?).
I was wondering if it's a normal situation for TimeLLM since the model is new. If it's a problem, where the bottleneck could possibly be?
To exclude the network issue, I used local files to load GPT2:
gpt2_config = GPT2Config.from_pretrained(gpt2_local_path, local_files_only=True)
gpt2 = GPT2Model.from_pretrained(gpt2_local_path, config=gpt2_config, local_files_only=True)
gpt2_tokenizer = GPT2Tokenizer.from_pretrained(gpt2_local_path, local_files_only=True)

Hardware: NVIDIA T4 (only tried it on one of my GPUs due to https://github.com/Nixtla/neuralforecast/issues/937)
OS: Linux
Versions / Dependencies
Python 3.9
neuralforecast 1.7.0
Reproduction script
https://nixtlaverse.nixtla.io/neuralforecast/models.timellm.html#timellm
Issue Severity
None

发生了什么 + 你期望发生什么
你好， 我试图运行 TimeLLM 模型的示例代码https://nixtlaverse.nixtla.io/neuralforecast/models.timellm.html#timellm 在实际训练之前花了将近 1 小时。在终端中，它只显示“Seed set to 1”。我检查了 GPU，没有 GPU 使用情况，仅占用了大约 500MB（~gpt2 大小）的内存。然后训练开始，只花了~10s。在训练中，GPU 使用情况很常见。最后，也花了大约 1 个小时来结束（预测时间？）。 我想知道这是否是 TimeLLM 的正常情况，因为该模型是新的。如果有问题，瓶颈可能在哪里？ 为了排除网络问题，我使用本地文件加载GPT2：
gpt2_config = GPT2Config.from_pretrained(gpt2_local_path, local_files_only=True)
gpt2 = GPT2Model.from_pretrained(gpt2_local_path, config=gpt2_config, local_files_only=True)
gpt2_tokenizer = GPT2Tokenizer.from_pretrained(gpt2_local_path, local_files_only=True)

硬件：NVIDIA T4（由于#937，仅在我的一个 GPU 上尝试过） 操作系统：Linux
版本/依赖项
Python 3.9 神经预测 1.7.0
复制脚本
https://nixtlaverse.nixtla.io/neuralforecast/models.timellm.html#timellm
问题严重性
没有任何

您好，我和您遇到了同样的问题，我把训练完的模型保存下来，再去做推理预测，依旧很慢，您这边是否解决了呢

Thanks - I can reproduce the issue (very long time to setup the training). We'll look into it.

谢谢 - 我可以重现这个问题（设置培训的时间很长）。我们会调查一下。

谢谢，期待您的回复

I can't seem to find a solution for this, unfortunately. TimeLLM with the current model also seems slow on my machine. Maybe you could try a different model from the Transformers library?

I can't seem to find a solution for this, unfortunately. TimeLLM with the current model also seems slow on my machine. Maybe you could try a different LLM from the Transformers library?

Thank you for the reply.
Do you mean apply LLMs other than gpt2 to TimeLLM? Since this issue, I switched to other models like nhits and timesnet already. Thanks again.

I can't seem to find a solution for this, unfortunately. TimeLLM with the current model also seems slow on my machine. Maybe you could try a different LLM from the Transformers library?

Thank you for the reply. Do you mean apply LLMs other than gpt2 to TimeLLM? Since this issue, I switched to other models like nhits and timesnet already. Thanks again.

Yes, indeed, that's what I'd try. But I haven't tried myself different models yet, so can't recommend one, I'm sorry.

So what causes this problem? I am facing the same problem as you did.


issue with spyder in anaconda3

Install on a win 7 32 bit os.
Python works and ipython works.
When i try to launch Spyder, when it gets to "loading file history" it close and says that python.exe (or sometime pythonw.exe) has stopped working.

Please open a system terminal (cmd.exe)
spyder --reset

and try again.

Maybe related to #7080 , by chance?

I didn't understand
On Wed 9 May, 2018, 10:57 AM C.A.M. Gerlach<EMAIL_ADDRESS>wrote:

Maybe related to #7080 https://github.com/spyder-ide/spyder/issues/7080
, by chance?
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/spyder-ide/spyder/issues/7090#issuecomment-387624051,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AlSImHFCy4a9T_ERIlXk_zOg8O9qRgRBks5twn5BgaJpZM4T22E6
.

I did this...then also the same error is coming.
On Wed 9 May, 2018, 6:28 AM Carlos Cordoba<EMAIL_ADDRESS>wrote:

Please open a system terminal (cmd.exe)
spyder --reset
and try again.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/spyder-ide/spyder/issues/7090#issuecomment-387587398,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AlSImCdnB-Nngfzi0H-l7meoUJEUEEy8ks5twj8YgaJpZM4T22E6
.

Please try the steps listed in the Spyder Troubleshooting Guide, specifically the Basic First Aid and Emergency CPR sections, which resolve the great majority of Spyder installation issues.
If it still isn't working, please describe in detail the circumstances in which the error occurred, and post the information listed in relevant subsection of the Calling for Help section as a comment to this issue, and we'll try to help if we can. Otherwise, your issue will likely have to be closed after one week (7 days). Thanks, and best of luck!

@mgmritunjai Any luck? Did you go through all the steps selected in the troubleshooting guide? Did you read the other issue? If we don't hear from you, we'll close this shortly. Thanks.

Ya I got it.thanks
On Thu 17 May, 2018, 11:46 PM C.A.M. Gerlach<EMAIL_ADDRESS>wrote:

@mgmritunjai https://github.com/mgmritunjai Any luck? Did you go
through all the steps selected in the troubleshooting guide? Did you read
the other issue? If we don't hear from you, we'll close this shortly.
Thanks.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/spyder-ide/spyder/issues/7090#issuecomment-389960570,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AlSImDKf3YI4ND6uahJG-qrTj0hmimIkks5tzb6SgaJpZM4T22E6
.

So by that do you mean you resolved the issue, and this can thus be closed?

Yes.it can be closed
On Fri 18 May, 2018, 10:28 AM C.A.M. Gerlach<EMAIL_ADDRESS>wrote:

So by that do you mean you resolved the issue, and this can thus be closed?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/spyder-ide/spyder/issues/7090#issuecomment-390094673,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AlSImA0--l0jDhZn_jOEh49bVlTyg73Kks5tzlT5gaJpZM4T22E6
.

Thanks, and glad to hear it has been resolved.


On large HTML files, RegularParser does not fire any handler.

I've been implementing this library on a project in my company, which scraps a whole local Wordpress with httrack and then modifies the files in order to fit to our needs.
To make some parts dynamic, we're implementing shortcodes (which we translate to custom PHP code). I've been testing your library without any problems, but when I gave it a definitive file, it didn't triggered any handler, and the parser pushed PHP memory consumption over 800MB.
After several tests, I've decided to try the RegexParser. It parsed the files correctly, and using a riddiculous amount of time and memory compared with the standard one.
I can understand the extra time and memory consumption (uncer certain limits), but I don't get why the parser didn't saw any of my tags, I am the only one who suffered this issue?
BTW, thank you very much for your work, your library is awesome and I'm enjoying it a lot!

Hi @DevNIX, thanks for your kind words! I'm happy that you like this library.
Can you send me a script to reproduce the issue along with the problematic file? Also, can you verify that you're using the latest version of Shortcode (v0.6.5)?

@DevNIX did you see my previous comment? Can you provide the information I need?

Yes, excuse me!
I have to get some time to prepare you a couple of files to reproduce the issue.

No problem, I just wanted to know what is the current status. Send them my way when you're ready.

Hi @DevNIX, did you have the chance to prepare the test data? It's fine if you need more time, just let me know if I can help you in any way. Meanwhile, you may be interested in #72, I found over 100x performance gain in RegularParser so there should be no penalty of running it vs other parsers.

@DevNIX good news, during verification phase I managed to reproduce your issue and I'm happy to tell you that it's fixed! Once #72 will be checked by others and merged, I'm releasing a new version right away. Unfortunately the fix ate some performance gains I mentioned above, but the result is still very good - above 10x gains in both time and memory.
This also means that you don't need to prepare anything for me, but I'd be very grateful if you could report back and confirm what I've found. Thanks!

Wow, that's great! I'm afraid I'm very busy lately and I've tried to do more things that I've been able to, so it was difficult for me to prepare the test data removing all the sensible info to replicate the issue...
I think I can confirm you in the following days if the patch works. Should I use the latest version of master?

@DevNIX Yes, please use the latest master. I'm currently preparing the next release so it would be great to have your confirmation before tagging it.

It seems like it worked well! I think I can close the issue and thank you a lot for your work, besides I could not give you all the help I would liked to.

@DevNIX v0.7.0 was just tagged, thanks for your cooperation!

FYI, I still think there is an issue for large documents.  I'm working on moving some Grav documentation to use a new prism.js shortcode, and it does not render the shortcodes at all with the Regular parser. Regex parser works in this case though:
https://raw.githubusercontent.com/getgrav/grav-learn/feature/learn4/pages/06.forms/01.blueprints/01.fields-available/docs.md

@rhukster can you open a new issue for that? Please include an example code to reproduce the problem and describe the expected result, ie. which handlers are not fired. I'll be happy to look at it in the next few days. Thanks in advance!


Hybridization type of group 1 metals

I just noticed that the hybridization type for group 1 atoms (i.e. I guess 'S-hybridized' or unhybridized?) seems dodgy/broken, as you can't get the name attribute from python:

In [1]: Chem.MolFromSmiles('[Mg]').GetAtomWithIdx(0).GetHybridization().name
Out[1]: 'SP'
In[2]: Chem.MolFromSmiles('[Na]').GetAtomWithIdx(0).GetHybridization().name
---------------------------------------------------------------------------
AttributeError                            Traceback (most recent call last)
<ipython-input-68-8b1e1f047fba> in <module>()
----> 1 Chem.MolFromSmiles('[Na]').GetAtomWithIdx(0).GetHybridization().name

AttributeError: name

I looked at each element, and it is the group 1 elements (and hydrogen):
In [3]: for i in PERIODIC_TABLE.symbol[1:93]:
   ...:     print(i, Chem.MolFromSmiles('[{}]'.format(i)).GetAtomWithIdx(0).GetHybridization())
H 1
He SP
Li 1
Be SP
B SP2
C SP3
N SP3
O SP3
F SP3
Ne SP3
Na 1
Mg SP
Al SP2
Si SP3
P SP3
S SP3
Cl SP3
Ar SP3
K 1
Ca SP
Sc SP2
Ti SP3
V SP3
Cr SP3
Mn SP3
Fe SP3
Co SP3
Ni SP3D
Cu SP3D
Zn SP
Ga SP2
Ge SP3
As SP3
Se SP3
Br SP3
Kr SP3
Rb 1
Sr SP
Y SP2
Zr SP3
Nb SP3
Mo SP3
Tc SP3
Ru SP3
Rh SP3
Pd SP3D
Ag SP3D
Cd SP
In SP2
Sn SP3
Sb SP3
Te SP3
I SP3
Xe SP3
Cs 1
Ba SP
La SP2
Ce SP3
Pr SP2
Nd SP3
Pm SP3
Sm SP3
Eu SP3
Gd SP3
Tb SP3
Dy SP3D
Ho SP3D
Er SP3D2
Tm SP3D2
Yb UNSPECIFIED
Lu UNSPECIFIED
Hf SP3
Ta SP3
W SP3
Re SP3
Os SP3
Ir SP3
Pt SP3D
Au SP3D
Hg SP
Tl SP2
Pb SP3
Bi SP3
Po SP3
At SP3
Rn SP3
Fr 1
Ra SP
Ac SP2
Th SP3
Pa SP2
U SP3

The value 1 doesn't seem to relate to any of the supported hybridization states:
In [4]: for k, v in Chem.HybridizationType.names.items():
   ...:     print(k, int(v))
   ...:
UNSPECIFIED 0
SP3D2 6
SP3D 5
SP3 4
SP2 3
SP 2
OTHER 7

Did the hybridization state for these atoms get removed?

Confirmed. That's a bug: I forgot S when I added the hybridization wrapper.
I wasn't even aware of the .name attribute. Thanks for pointing that out!


Build instructions failing when building from source due to binaryninjaapi error

Installing BinDiff from source according to building instructions in README currently fails due to binaryninjaapi git error.
Environment (please complete the following information):

OS and version: Ubuntu 22.04.3
Locale: NO Norwegian
Disassembler and version: IDA Pro 8.4

Building from Source (From README.md):
The following build dependencies are optional:
IDA Pro only: IDA SDK 8.0 or higher (unpack into deps/idasdk)
The general build steps are the same on Windows, Linux and macOS. The following shows the commands for Linux.
Download dependencies that won't be downloaded automatically:
mkdir -p build/out
git clone https://github.com/google/binexport build/binexport
unzip -q <path/to/idasdk_pro80.zip> -d build/idasdk

Next, configure the build directory and generate build files:
cmake -S . -B build/out -G Ninja \
  -DCMAKE_BUILD_TYPE=Release \
  -DCMAKE_INSTALL_PREFIX=build/out \
  -DBINDIFF_BINEXPORT_DIR=build/binexport \
  "-DIdaSdk_ROOT_DIR=${PWD}build/idasdk"

Output error
user@laptop:~/bindiff-8$ cmake -S . -B build/out -G Ninja \
  -DCMAKE_BUILD_TYPE=Release \
  -DCMAKE_INSTALL_PREFIX=build/out \
  -DBINDIFF_BINEXPORT_DIR=build/binexport \
  "-DIdaSdk_ROOT_DIR=${PWD}build/idasdk"
-- The C compiler identification is GNU 11.4.0
-- The CXX compiler identification is GNU 11.4.0
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Check for working C compiler: /usr/bin/cc - skipped
-- Detecting C compile features
-- Detecting C compile features - done
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Check for working CXX compiler: /usr/bin/c++ - skipped
-- Detecting CXX compile features
-- Detecting CXX compile features - done
-- Performing Test ABSL_INTERNAL_AT_LEAST_CXX17
-- Performing Test ABSL_INTERNAL_AT_LEAST_CXX17 - Success
-- Performing Test ABSL_INTERNAL_AT_LEAST_CXX20
-- Performing Test ABSL_INTERNAL_AT_LEAST_CXX20 - Failed
-- Looking for pthread.h
-- Looking for pthread.h - found
-- Performing Test CMAKE_HAVE_LIBC_PTHREAD
-- Performing Test CMAKE_HAVE_LIBC_PTHREAD - Success
-- Found Threads: TRUE
--
-- 25.2.0
-- Performing Test protobuf_HAVE_LD_VERSION_SCRIPT
-- Performing Test protobuf_HAVE_LD_VERSION_SCRIPT - Success
-- Performing Test protobuf_HAVE_BUILTIN_ATOMICS
-- Performing Test protobuf_HAVE_BUILTIN_ATOMICS - Success
-- Found Protobuf: protobuf::libprotobuf (found suitable version "4.25.2", minimum required is "3.14")
[1/9] Creating directories for 'binaryninjaapi-populate'
[1/9] Performing download step (git clone) for 'binaryninjaapi-populate'
Cloning into 'binaryninjaapi-src'...
fatal: reference is not a tree: 0d57172be4fda0b7015db697b5353a12d03932af
CMake Error at binaryninjaapi-subbuild/binaryninjaapi-populate-prefix/tmp/binaryninjaapi-populate-gitclone.cmake:40 (message):
  Failed to checkout tag: '0d57172be4fda0b7015db697b5353a12d03932af'

FAILED: binaryninjaapi-populate-prefix/src/binaryninjaapi-populate-stamp/binaryninjaapi-populate-download /home/user/bindiff-8/build/out/_deps/binaryninjaapi-subbuild/binaryninjaapi-populate-prefix/src/binaryninjaapi-populate-stamp/binaryninjaapi-populate-download
cd /home/user/bindiff-8/build/out/_deps && /usr/bin/cmake -P /home/user/bindiff-8/build/out/_deps/binaryninjaapi-subbuild/binaryninjaapi-populate-prefix/tmp/binaryninjaapi-populate-gitclone.cmake && /usr/bin/cmake -E touch /home/user/bindiff-8/build/out/_deps/binaryninjaapi-subbuild/binaryninjaapi-populate-prefix/src/binaryninjaapi-populate-stamp/binaryninjaapi-populate-download
ninja: build stopped: subcommand failed.

CMake Error at /usr/share/cmake-3.22/Modules/FetchContent.cmake:1087 (message):
  Build step for binaryninjaapi failed: 1
Call Stack (most recent call first):
  /usr/share/cmake-3.22/Modules/FetchContent.cmake:1216:EVAL:2 (__FetchContent_directPopulate)
  /usr/share/cmake-3.22/Modules/FetchContent.cmake:1216 (cmake_language)
  build/binexport/cmake/BinExportDeps.cmake:104 (FetchContent_Populate)
  build/binexport/CMakeLists.txt:42 (include)

-- Configuring incomplete, errors occurred!
See also "/home/user/bindiff-8/build/out/CMakeFiles/CMakeOutput.log".
See also "/home/user/bindiff-8/build/out/CMakeFiles/CMakeError.log".

This is odd - the referenced Binary Ninja commit exists, but GitHub seems to think that it is not part of the projects history.
Can you try to replace it with 59e569906828e91e4884670c2bba448702f5a31d in BinExportDeps.cmake?

After that change it now builds successfully

Glad to hear.

Also, the idasdk_proNN.zip contains a folder with the same name, so the cmake command needs a minor change

Note that the directory names inside the ZIP are different for IDA Home, IDA Pro and IDA Teams, which is rather annoying. I should probably change the command to include a shell glob (idaask_*.zip).

Seems like some of the tests are failing too (possibly because of the new unsupported update?)

This is expected. The failing tests are all BinExport tests that live in the BinExport repo and are not built by default. Somehow, ctest is still picking them up as tests to be run.

Understandable, and thank you for the information.
Bindiff built successfully. I also built Binexport and had to apply the same changes to the BinExportDeps.cmake file there, but after that everything works.
Feel free to close the issue.


PSS app upgrades - Atlas

Towards epic.
Task
Upgrade all applications running on all AWS clusters to a version which supports PSS before the cluster is upgraded to AWS v19.3.x.
Why?
All apps running on a cluster need to be upgraded to a version which supports PSS before upgrading to cluster to AWS v19.3.x (see the release notes).
Links

AWS v19.3.0 release notes;
Scheduled cluster upgrades;
App status;
Managed apps upgrades;
Long-term process app upgrades issue;

Atlas has nothing to do here :)


Use server port 22223

Mocap server uses different server port according to message protocol version and endianess. User can define the base port number, which is used for legacy 1.0 protocol version. Other port numbers are based on the base port (e.g. protocol 1.1 little-endian: base-port +1). The base port should not be used for new clients using newer protocol versions, but other related port shall be selected (base port + n)
Currently the mocap_pose sends handshake request to that base-port (22222), but due to protocol 1.1. and little-endianess, the server responds back from base-port+1 (22223). This causes routing issue in case the mocap-pose is not running in same subnet, but the NAT device checks the translation table and gets confused, because the response does not come from the same port where request was sent.
To fix this, the mocap-pose shall send the initial request already to the correct port (base-port+1) which shall be used for  it's protocol/endianess combination.
Link to documentation: https://docs.qualisys.com/qtm-rt-protocol/#connecting

This fix is needed for fmo-fc-adapter system where the mocap-pose is running in docker container inside the adaptervm of PMC, and it accesses the mesh network via separate netvm.

It requires only this mocap_pose change, no need to change anything in QTM server side. Our QTM server is already switching to 22223, even if mocap_pose is trying to send request to 22222 port.

Has it been also verified with a normal drone?
Not yet, that verification is needed still.

Does this change require a qualisys_cpp_sdk submodule update?
No update needed, it supports v1.1 already. That is actually the issue here, that because we are using v1.1, the server starts responding back from different port than the base-port. If we would have used v1.0 then the server would have used the legacy port which would have been fine for routing.

No, this does not work. The issue is not the port selection, because the rt_protocol's Connect() method already takes care of selecting proper server port. The original implementation is correct, we should provide only the base port into the Connect() method and the extra information about sdk version and endianess!
so, the connection problem in fmo-fc-adapter must be something else, I probably mixed the things in testing setup.
Need to close this PR because it does not really work. My bad..

ok. No problem. Thanks for testing it. :+1:


emmet Suggested that error

Version: 0.33.2
Emmet recommended in the label, version 0.32.1 normal

Duplicate of #1039


Request money - “Hmm… it’s not here” shown when adding a receipt to a manual request

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

Version Number: 1.4.24-4
Reproducible in staging?: Y
Reproducible in production?: N
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
Issue found when executing PR https://github.com/Expensify/App/pull/32764
Action Performed:

Go to any chat
Request money with the scan tab and scan a receipt
Request money again as manual request
Go to the manual request
Select the three dots on top right
Add receipt

Expected Result:
The add receipt UI is shown and user can add a receipt after adding a manual expense
Actual Result:
After scanning a receipt seems that the new manual request isn't synced immediately and adding a receipt to it fails with the error message
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

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

Screenshots/Videos
Add any screenshot/video evidence

https://github.com/Expensify/App/assets/78819774/776196aa-f2ae-4cad-8c3a-ddf9c21b62e4
View all open jobs on GitHub

I am not able to reproduce on dev

I was unable to reproduce it in web. My Android environment is (unsurprisingly) being difficult, making it hard to test there. Either way, I don't feel like this warrants "Blocker" status. Anyone disagree?

Still able to reproduce in Native app
Uploading RPReplay_Final1705032101.MP4…

@kavimuru can you please confirm this doesn't happen on production?

I think this is dupe of #34417

Agreed. Closing as duplicate!


change favicon

By now I used the html5 boilerplate favicon.ico

Use also https://www.npmjs.com/package/serve-favicon

Nop, Just create a Real favicon and Apple touch icon and put them in public folder


[Question] What does latest url referring to in ci.opensearch.org

Hi All,

The latest url means pointing to the latest full :green_circle: build (x64/arm64 tar/rpm all green).
The latest url means pointing to the latest build, no matter :red_circle: or :green_circle: status

https://ci.opensearch.org/ci/dbc/distribution-build-opensearch/2.0.0-rc1/latest/linux/x64/tar/dist/opensearch/opensearch-2.0.0-rc1-linux-x64.tar.gz
We are having 1 right now, which means there is guaranteed to have a working tarball with latest, but not always the latest code.
This means, in 2.0.0-rc1 right now due to rpm failure, there is no latest url available yet for CI to use, even though tarball is built already.
Since there are a lot of plugin repo CIs use latest , if we go with 2 it might break CIs more often if any of tar/rpm build goes red.
This is a tradeoff and I would like your opinions on it.
In the meantime we will manually tweak the latest mechanic in S3 to unblock rc1 .
Thanks.

Old latest issue when we 1st implement it:
https://github.com/opensearch-project/opensearch-build/issues/1492

I believe latest should refer to the latest stable and supported release. RC1, Alpha, and Beta releases should require users to have to explicitly opt-in (by manually selecting a version). This I have come expect when using the use the latest tag from any other projects.

since this is ci.opensearch (as opposed to artifacts.opensearch) i wouldn't expect latest to be stable and supported

even though tarball is built already

If a red tarball breaks downstream CIs then I would prefer option 1, otherwise i would prefer option 2 as currently reports-scheduler CI is broken because upstream rc1 missing https://github.com/opensearch-project/dashboards-reports/pull/341

@dtaivpp You misunderstood as this latest is only used for ci staging repo.
In production we will not use latest url.
Thanks.

The latest url means pointing to the latest full 🟢 build (x64/arm64 tar/rpm all green).

+1
If there is a failure, assets that were available are not longer present, that is not very useful for automated processes.
In the use case of the security plugin, opensearch-min is used to install the security plugin and verify service still comes up.  We had issues in the past where we broke the distribution build and it wasn't discovered until more than weeks.

@peterzhuamazon I am so glad I misunderstood! haha that would have been a bad UX 😅

@peterzhuamazon I am so glad I misunderstood! haha that would have been a bad UX 😅

Sorry that my original description is not clear enough so I added some context now.
Thanks.

The latest url means pointing to the latest full 🟢 build (x64/arm64 tar/rpm all green).

I think this makes more sense because it will take a lot of the usefulness out of latest if I am not sure if this build was successful or not and hence not being 100% sure what failed on the build the was fetched. Currently AD uses latest to get job-scheduler zip for example, we will probably change this to use maven anyways but just an example of why keeping latest to successful build could be more helpful.

The latest url means pointing to the latest full 🟢 build (x64/arm64 tar/rpm all green).

I prefer this definition. Successfully compiled code that doesn't pass all tests, aka doesn't work (🔴) is not very useful and will yield too many false positives downstream when consumed.

* What went wrong:
A problem occurred evaluating root project 'opensearch-index-management'.
> Can't get https://ci.opensearch.org/ci/dbc/distribution-build-opensearch/2.0.0-rc1/latest/linux/x64/builds/opensearch/plugins/opensearch-job-scheduler-<IP_ADDRESS>-rc1.zip to /tmp/tmpbpozmgws/index-management/src/test/resources/job-scheduler/opensearch-job-scheduler-<IP_ADDRESS>-rc1.zip

* Try:
> Run with --stacktrace option to get the stack trace.
> Run with --info or --debug option to get more log output.
> Run with --scan to get full insights.

IM code is trying to get latest JS zip to build IM, but in reality there is no successful build yet in 2.0.0-alpha1, thus no latest.
This has become a catch 22 now, so IM needs latest JS but JS is not able to get from latest because the pipeline is not even complete and will fail due to IM cannot find JS.
This is why I support using 2 as 1 cause the above failure.

@peterzhuamazon Although this problem surfaces in our use of latest and could be solved with option 2, I think this related to the issue highlighted here: https://github.com/opensearch-project/opensearch-build/issues/1463 and a cleaner solution could be to publish snapshots for each successfully built plugin before waiting for the entire build to succeed

@peterzhuamazon Although this problem surfaces in our use of latest and could be solved with option 2, I think this is related to the issue highlighted here: #1463 and a cleaner solution could be to publish snapshots for each successfully built plugin before waiting for the entire build to succeed. For now we can remove IM and AD from the build and add it back as was done to solve this type of problem in the past

Have some conversation with @amitgalitz offline and we will try to resolve this issue with maven zips after 2.0.0.

We will keep using the latest == latest success build for now.


[nuxt] Unable to persist if using store in middleware

Nuxt version: 3.0.0-rc.1
Pinia plugin persistedstate version: 1.6.1
Pinia version: 2.0.13
Hello, I reproduced your Nuxt example and it works well but if I want to use my store in a middleware, that doesn't work anymore. I have no error, just the cookie is not setted.
// /middleware/authentication.global.ts
import { defineNuxtRouteMiddleware, navigateTo, useCookie } from "#app"
import { useUser } from "~/store/user"

export default defineNuxtRouteMiddleware((to) => {
	console.log(useUser()) // this line prevents cookie creation, but reading cookie works
	if (!useCookie("user").value && to.path !== "/login") {
		return navigateTo("/login")
	}
})

// /store/user.ts
import { defineStore } from "pinia"

export const useUser = defineStore("user", {
	state: () => ({
		refreshToken: "",
		token: "",
		roles: [] as Array<string>
	}),
	getters: {
		isLogged: state => !!state.refreshToken
	},
	persist: {
		paths: ["refreshToken", "roles"]
	}
})

// /plugins/persistedstate.ts
import { createNuxtPersistedState } from "pinia-plugin-persistedstate"
import { defineNuxtPlugin, useCookie } from "#app"

export default defineNuxtPlugin((nuxtApp) => {
	nuxtApp.$pinia.use(createNuxtPersistedState(useCookie))
})

Thank you !

Hi, i may need to look deeper into nuxt to know what is accessible from middleware and what is not 🤔

as much as i'd like to have an impact on that issue from the plugin, it may be more of a Nuxt/cookie thing ?

Yes @prazdevs , I think middleware is being called first than plugin. So nothing is being restored from storage when middleware loads.

i'll read some more code and undestand nuxt deeper, esp. middleware to see what we can do and if there's something the plugin should be responsible of. not sure but we'll see :)

Thanks @prazdevs

@prazdevs I noticed that this behavior is also present inside Plugins not just middleware

I was wondering if this was still an issue with the v3 and the nuxt module (@pinia-plugin-persistedstate/nuxt).
I played with middlewares a bit, and i can access and change stores values with it being always up to date and stored (in cookies by default).

I'll be closing this issue, as it seems resolved with nuxt3 and the nuxt module. If you still have problems with it, feel free to reopen!


There was an error retrieving the latest version of GramAddict: list index out of range

Any help please with this error ?

You have entered an invalid index into a list.

try downgrade your Instagram app


Can Accessdev trac tickets be used for user level issue tracking

|    by<EMAIL_ADDRESS>
hi everyone
Imtiaz Dharssi put in the ticket 121 and wishes to use the Accessdev ticket system for user level issue tracking similarly to the sysadmin issue tracking.
Can the Accessdev ticket system be modified to accommodate this ?
What needs to be done to allow this to happen ?
Or is there a separate solution ?
Suggestions on how to proceed ?
Robin

Issue migrated from trac:130 at 2024-01-31 17:37:00 +1100

<EMAIL_ADDRESS>commented

Milestone Future deleted

<EMAIL_ADDRESS>removed milestone (was Future)

<EMAIL_ADDRESS>commented

I'm happy for users to create tickets here. Triage may require some thought for deciding on who should get notified on issues, though perhaps that can be left to users to decide in the first instance.
When the shared repository comes online that's probably a preferable solution for science tickets, since you've got all users in a central location

<EMAIL_ADDRESS>commented

I've created a component 'ACCESS Model' for science issues to go in (and relabelled the other components while I was at it) as well as deleted the milestone section.


A $webrtc rule blocks requests

$webrtc,domain=xda-developers.com blocks a https://api.xda-developers.com request.

4ee892e3e2fd63532e1f6ca45dfc21fd9907e9dc

I'm having this problem in AG Android 2.12.52η w/ many $webrtc filters builtin to the default filterlists.  Until this is completely resolved, can we get a global Low Level Setting to outright disable/ignore such filters?

Resolved in the latest nightly

Confirmed, thanks.


Mod wheel not working

Hi, I tried with many controllers but modulation wheel has no effect. Is it me, or it is not managed?
Maybe some settings not clear in the manual?
Thanks

Mod wheel (controller 1) is working fine for me. But if your controller sends NRPN it will not be detected by the MIDI learn feature. Is the issue here that MIDI learn is not working?

I've just checked my keyboard. Pitch wheel sends PITCHBEND with 16 bit value, while mod wheel sends CC 1 with 0-127 value.
But in performer nothing happens. Is there something I need to configure? Some mapping or flag, maybe? I am just using a midi/cv track.
To compare, I tested the same keyboard with the same connection with a Disting Mk4 in MIdi2CV mode, and mod wheel is perfectly read. Thanks

On a MIDI/CV track the mod wheel is not doing anything. Only pitch bend and aftertouch are supported (and output as CV). What should the mod wheel do?

For example it could be routed somewhere (gate, slide, etc.). Or just transpose in a fixed way like other environments. I mean, pitch wheel changes pitch and come back. Mod is not spring loaded so you can move it and transpose.
When you say that on a MIDI/CV track is not doing anything you mean that it does on a note track?

You can already setup routings for SlideTime and Transpose on the MIDI/CV track. So you can take the mod wheel and route it to one of those parameters. Support for gate length might be a nice addition.

Yes, it works. Only thing I noticed is that the transpose is "discrete", you can hear the discrete change between notes while you move the mod wheel. But I guess I have to work on slide settings


Add a note that a document created from this docu will be submitted to RDA for review

Add a note to the online version as explained in #49.  This note is suitable while the submission to RDA is still under review, but will need an update for the final version if it gets endorsed. Also the words "PDF document created from Version 1.0" should be turned into a link as soon as the submitted version gets an DOI from RDA.
The PDF version will need to get an adapted version of the note.
Close #49.

See the result as an online version at Read the Docs.


[BUG] lonboard and maplibre does not work together

Context
What results were you expecting?
When installing lonboard and maplibre in the same conda env, lonboard does not work. The map does not render. The Chrome developer console show Error: deck.gl - multiple versions detected: 9.0.16 vs 8.9.35. The maplibre renders just fine.
Relevant issue: https://github.com/visgl/deck.gl/issues/4909
Resulting behaviour, error message or logs

Environment

OS: Linux
Browser: Chrome Version 129.0.6668.58 (Official Build) (64-bit)
Lonboard Version: 0.9.3

Steps to reproduce the bug
conda create -n test python=3.12
conda activate test
conda install -c conda-forge lonboard maplibre jupyterlab

You can't have multiple deck.gl versions loaded onto the page. You would need some sort of isolation on the Jupyter side in the browser, which I don't think exists.
This is not possible for me to fix; I believe the only solution is to not have them both installed.


[GLUTEN-7327][INFRA] Publish Velox Backend Test Result Report in Github Action

What changes were proposed in this pull request?
This PR aims to use action-surefire-report to publish the test reports. With this feature, we can quickly see the failed tests and the information in the PR.
(Fixes: #7327)
How was this patch tested?

Does the PR actually revert https://github.com/apache/incubator-gluten/pull/5173?
+1 to the other changes

@zhztheplayer I think it might originally have been ineffective.

===== Performance report for TPCH SF2000 with Velox backend, for reference only ====

query
log/native_master_09_24_2024_time.csv
log/native_master_09_24_2024_f00e4536bc_time.csv
difference
percentage

q1
53.11
53.67
0.560
101.06%

q2
28.68
28.71
0.033
100.12%

q3
54.35
52.87
-1.476
97.28%

q4
42.77
43.23
0.456
101.07%

q5
104.16
103.73
-0.430
99.59%

q6
12.84
12.93
0.088
100.68%

q7
111.50
114.21
2.704
102.42%

q8
115.36
118.88
3.511
103.04%

q9
170.80
171.58
0.783
100.46%

q10
67.52
66.14
-1.378
97.96%

q11
25.90
26.59
0.681
102.63%

q12
33.02
31.72
-1.300
96.06%

q13
52.35
52.68
0.336
100.64%

q14
22.72
22.17
-0.542
97.62%

q15
51.80
49.88
-1.920
96.29%

q16
17.12
19.31
2.186
112.77%

q17
129.16
127.97
-1.192
99.08%

q18
200.39
202.18
1.790
100.89%

q19
29.80
27.98
-1.818
93.90%

q20
40.86
40.82
-0.036
99.91%

q21
341.46
329.37
-12.088
96.46%

q22
16.20
16.00
-0.197
98.78%

total
1721.86
1712.62
-9.246
99.46%


[bug] TypeError when selecting Coinbase Wallet

Is there an existing issue for this?

[X] I have searched the existing issues

RainbowKit Version
0.2.4
wagmi Version
0.4.11
Current Behavior
Clicking on Coinbase Wallet as an option in the Connect Wallet modal does nothing and triggers this error in the console:
TypeError: extension.disableReloadOnDisconnect is not a function
    at CoinbaseWalletSDK.makeWeb3Provider (CoinbaseWalletSDK.js?b21f:91:1)
    at CoinbaseWalletConnector.getProvider (wagmi-core-connectors-coinbaseWallet.esm.js?1a17:144:67)
    at async CoinbaseWalletConnector.connect (wagmi-core-connectors-coinbaseWallet.esm.js?1a17:74:1)
    at async connect (wagmi-core.esm.js?19bf:139:1)

Coinbase Wallet Extension:
Version
2.18.1
Updated
June 8, 2022
Expected Behavior
Clicking 'Coinbase Wallet' opens the Coinbase Wallet connect extension window
Steps To Reproduce
Use rainbowkit v0.2.4
Use wagmi v0.4.11
Use Coinbase Wallet extension 2.18.1
Click Coinbase Wallet

Link to Minimal Reproducible Example (CodeSandbox, StackBlitz, etc.)
https://eightbit-nxf3wjbet-lets-go.vercel.app/
Anything else?
NextJS, Vercel, Chrome 102.0.5005.61

I ran into this issue also. Turns out it happens when the Coinbase Wallet SDK is running against an older version of the browser extension.
I spoke with their team yesterday and they've pushed out a fix in<EMAIL_ADDRESS>It now checks whether extension.disableReloadOnDisconnect exists before calling it. You can see the specific PR fixing the issue here: https://github.com/coinbase/coinbase-wallet-sdk/pull/575.


Release4

merged in https://github.com/nrkno/tv-automation-server-core/commit/87da0c4d2f74502445e0c2921ffd721a4630c18a


Contour lines

Added import-contour. Note that contour lines MUST be imported, otherwise export script may fail because of absence of contour-related tables.

Great @romanshuvalov. Really appreciate it. Most important thing is whether the tile size is still acceptable that we can merge it into the main vector tiles.

[ ] I will test this out with Switzerland and generate MBTiles and then compare.

After that is ok we can improve the PR or otherwise move it to the dem2vectorterrain project and create vector tiles just for the contour lines (copy the structure from https://github.com/osm2vectortiles/custom-vt-skeleton).

Added import-contour. Note that contour lines MUST be imported, otherwise export script may fail because of absence of contour-related tables.

We can fix that by just creating a empty table before hand. Then people without shape files just don't have contours.

Okay size impact seems okay on the individual tile level (of course the planet will be bigger).
But it won't do damage to the 500KB tile limit (additional 2-5KB per z14 tile).
Speed impact is bigger though -rendering is quite a bit slower.
Next steps would be to test this on a bigger scale than Switzerland (and also check how big the PostGIS db get's with all those shapefiles).
Hmm thinking whether it belongs to OSM2VectorTiles or not. Your changes will be available as vector tiles for sure @romanshuvalov. It is so great!! But question is whether a separate vector tile set or not.
This is a vector tile set that actually only needs to be generated one time only (no update functionality as for OSM2VectorTiles).
But on the other hand integrating it into OSM2VectorTiles makes it immediately visible for the people and they will actually start using it.
@klokan let's get together at FOSS4G and discuss it.
I hope you don't mind @romanshuvalov if this takes some time to discuss.
And also when we are going to render a global vector tile set this will also take some time but it is a huge benefit to everyone!

If you decide to keep them in a separate set (which might not need to be updated for a long long time), I'd recommend merging them into published mbtiles databases, so clients would not need to use 2x the requests.

If you decide to keep them in a separate set (which might not need to be updated for a long long time), I'd recommend merging them into published mbtiles databases, so clients would not need to use 2x requests for each tile.
Hide all checks

@hyperknot you can also concat PBFs on the client side. Petr's https://github.com/klokantech/tileserver-gl does this (as does Mapbox).
However not every tileserver can do that and als makes hosting from S3 more difficult.
What are your thoughts on this @hyperknot ?

I was referring clients as in browsers and mobile devices. When mobile devices need to make 2x (or 3x for hillshading) requests, that gives a visible latency especially on mobile networks. HTTP2 can help a bit, but it't not yet common.
I've just checked Mapbox outdoors, and they only serve a single 512x512 PBF which has both vector data, contour lines, and triangulated hillshading all combined. Thus even a desktop browser only does 6 requests for a single map view, which is quite fast.
If this can be done on-the-fly via TileServer GL without any performance penalties, then I guess it's a good solution. If this means much higher system requirements / slower serving performance, then I believe an offline processing step would be better.

Best solution is render two types of vector tiles: with contours and without them. So probably we should add options to  export: render only osm data, render osm data + contour layer, render both.
Contours as separate tile set is not good because of 2x HTTP queries and in most cases they don't need without OSM data tiles. But it require more drive space because of doubling OSM data. It can be solved by adding third option contours-only. So it could be configured by administrator.
Which tile set - pure OSM or OSM+contours - should be given on osm2vectortiles website, it's for you to decide. I think OSM+contours are better.
About me: for my purposes (3D rendering) I need custom tiles with a lot of other fields and calculations so I forked the project and I will continue developing in separated branch. So, of course, I don't mind if you need time to deal with that questions. Do whatever you want, I just needed contour layer and I added it and shared the code in order to be useful for community.
If you decide something and you think I could help implementing it, I will participate. Meanwhile I continue working on my own project.

Update: Fixed Makefile. And added empty tables in case of no contour files in import/contour folder.
So, contour files are not necessary now, but import-contour still must be called for creating that empty tables.
And I don't know how to create empty import/contour/ folder on github. It's listed in .gitignore file and I can't add it to commit.

@lukasmartinelli @romanshuvalov
My personal preference is to have both Contours alone and OSM+Contours. With https://github.com/stirringhalo/openstreetmap-mapboxgl coming along well (waiting on property functions for line features so I can render all roads in one go), I'd like to merge in the contours there too.


WIP releases showing up in my HACS feed

and have you...

[X] I updated to the latest version available
[X] I cleared the cache of my browser

Version of RPi Monitor Card
1.8.4
Is this a regression?
No or unsure. This never worked, or I haven't tried before.
Browser and Operating system
n/a
Description of problem
HACS is getting spammed with WIP releases that are obviously meant as beta releases, please consider marking the releases as pre-release on GitHub, as that makes HACS not show the releases to users.
Errors in Browser Console
No response
Additional information
No response

Please pardon the latest frequency of updates... This was a special effort requiring help from our card user community...
The end result of the effort was new code was created to replace the use of external underlying utils which were failing for only part of the community.   This is also the first time we released monitoring of the MQTT daemon version (the 2nd part of this system) which needed community feedback by providing rapid turn-around to the changes as suggestions came in.
We are now back to full functionality and only formal releases as of v1.4.0 and later.   Thanks again for your patience!