Lambdas without parameters at dbhub.py, lines 468, 469

Hi. Nice library!
Was trying to run a query but was not working:

TypeError: () takes 0 positional arguments but 1 was given

The piece of code starting at line 463:
        for result_row in res:
            one_row = {}
            for data in result_row:
                result = {
                    0: lambda v: base64.b64decode(v.encode('ascii')) if isinstance(v, str) else None,   # Binary
                    1: lambda: "",                                                                      # Image - just output as an empty string (for now)
                    2: lambda: None,                                                                    # Null
                    3: lambda v: str(v) if isinstance(v, str) else "",                                  # Text
                    4: lambda v: int(v),                                                                # Integer
                    5: lambda v: float(v)                                                               # Float
                }[data['Type']](data['Value'])
                one_row.update({data['Name']: result})
            rows.append(one_row)

Passes a parameter to the anonymous function in cases 1 and 2, but it does not expect any parameter.
Fixed jus adding the parameter that simply is not used:
        for result_row in res:
            one_row = {}
            for data in result_row:
                result = {
                    0: lambda v: base64.b64decode(v.encode('ascii')) if isinstance(v, str) else None,   # Binary
                    1: lambda v: "",                                                                      # Image - just output as an empty string (for now)
                    2: lambda v: None,                                                                    # Null
                    3: lambda v: str(v) if isinstance(v, str) else "",                                  # Text
                    4: lambda v: int(v),                                                                # Integer
                    5: lambda v: float(v)                                                               # Float
                }[data['Type']](data['Value'])
                one_row.update({data['Name']: result})
            rows.append(one_row)

Cheers,
Daniel.

Hi!
Fixed. Commit #5015ec0 .
Thanks


OS_unmount() & OS_chkfs() don't catch OS_TranslatePath() failures

in osfilesys.c OS_unmount(), the return value from OS_TranslatePath isn't checked.

Imported from trac issue 94.  Created by abrown4 on 2015-08-19T19:16:13, last modified: 2015-12-01T14:32:54

Trac comment by abrown4 on 2015-08-19 20:26:41:
unit test for OS_chkfs() on #22 branch, commit: [changeset:a1eb647]

Trac comment by abrown4 on 2015-08-19 19:45:18:
Also note checked in OS_chkfs(), updating title.

Trac comment by abrown4 on 2015-08-19 19:22:18:
unit test on #22 branch, commit: [changeset:020bd96]

Trac comment by abrown4 on 2015-08-19 21:01:03:
(Identified with vxworks osal code review on branch #22.)

Trac comment by abrown4 on 2015-08-24 17:38:38:
commit: [changeset:33afe2b] Trac #94, Fixed vxworks OS_unmount() to catch OS_TranslatePath() failures.
commit: [changeset:40531d9] Trac #94, Fixed vxworks OS_chkfs() to catch OS_TranslatePath() failures. Improved similar logic for OS_unmount() check as well.

Trac comment by abrown4 on 2015-08-20 15:02:43:
unit test fixed on #22 branch, commit: [changeset:42565b2]

Trac comment by abrown4 on 2015-11-25 16:55:12:
Traceability: This change is reflected in #112 [changeset:093359f].  Recommend closing #94 in favor of #112.


Delete stale files

These aren't included or built anywhere.

 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.

Merging as discussed in the April 16th teleconference.


Release 4 level SUSHI trained models

Hi, Ocetintas:
Because of my Nvidia card's memory, I can't train SUSHI model with 9 level. Can you release your 4 level SUHSI pretrained models for me to reproduce and compare?
Many thx!

Hi,
We are not planning to release more models but you can easily train a SUSHI with different levels by changing a couple of config variables. I'd suggest trying one of the models in Figure 4b of our paper, as we already report results there.


Proposal: withResolver or newPromise

It's essentially a short cut for when you know you need to create a new promise via the constructor inside a then handler:
somePromise.then(function(result) {
    return new Promise(function (resolve, reject) {
        // use result to perform some async task and then resolve or reject.
    });
});

a typical use case would be for http which does not use the standard node callback style, and emits an "error" event. Below is a naive implementation of what I am thinking:
function newPromise(cb, ctx) {
    return function() {
        var args = Array.prototype.slice.all(arguments);
        return new Promise (resolve, reject) {
            args.unshift(resolve,reject);
            cb.apply(ctx || this, args);
        };
    };
}

// usage

getAPromiseForSomeUrl()
  .then(newPromise(function(resolve, reject, url) {
     http.get(url, resolve).on('error', reject);
  });

A full implementation would add it as a chain-able function on the promise prototype. Include an equivalent for catch, and perhaps some of the collection functions.

You can use custom promisifier here:
Promise.promisifyAll(http, {
    promisifier: function(fn) {
        return function() {
            var self = this;
            var args = [].slice.call(arguments);

            return new Promise(function(resolve, reject) {
                args.push(resolve);
                fn.apply(self, args).on("error", reject);
            });
        };
    }
})


Iterative Closest Point (ICP)

Hello, I'm trying to match and morph a model to a template using Iterative Closest Point (ICP) like the following URLs below.  How would I use this wrapper to do it?  Thanks for your help.
http://docs.opencv.org/3.0-beta/modules/surface_matching/doc/surface_matching.html#pose-registration-via-icp
http://www.morethantechnical.com/2010/06/06/iterative-closest-point-icp-with-opencv-w-code/

Any timeframe on when this will be added and when the next release date will be?  I am also interested in the surface matching module.

Is there any update? Has the surface_matching support been added? I am also working with ICP and would like to know. Thank you.

@shimat has anything happened regarding ICP support? Noticed that Emgu has this: https://www.emgu.com/wiki/files/4.5.1/document/html/e38e6b86-77a3-e894-f17a-ea4001468254.htm


Make metadata.chat_id available in custom pipeline

Is your feature request related to a problem? Please describe.
I'm integrating open-webui with AWS Bedrock Agents using bedrock_agent_runtime.invoke_agent() in a custom pipeline. This method requires a sessionId parameter, as the Bedrock Agent manages the chat session.
I would like to use chat_id as the sessionId, but it's no longer available in the pipeline.pipe() method. The metadata field, where this information was previously stored, was removed 3 months ago in commit f462744.
Describe the solution you'd like
I propose restoring the metadata field from the payload at: https://github.com/open-webui/open-webui/blob/main/backend/open_webui/apps/openai/main.py#L383-L384
Describe alternatives you've considered
If reintroducing the metadata field could cause other issues, I suggest adding an environment variable, such as ENABLE_PIPELINE_METADATA. When this variable is not set or set to false, the default behavior (without metadata) would remain unchanged, while enabling it would send the field to custom pipelines
Additional context
None

Solved by adding to a custom field in body:
    async def inlet(self, body: dict, user: Optional[dict] = None) -> dict:
        body["pipeline_metadata"] = {"chat_id": body["chat_id"]}

        return body


Changed pyvera version to 0.2.41

Changed required pyvera version to 0.2.41 from 0.2.39.
The 0.2.41 supports the VeraSecure built in siren. Siren is treated as switch and can now be turned on and off. Before it was armable but generated error in Vera controller.  This allows for both detecting status of Siren if triggered from within Vera and also outside controll from HA.
Description:
Adding support to controll the built in siren in the VeraSecure controller.

Hi @patrik3k,
It seems you haven't yet signed a CLA. Please do so here.
Once you do that we will be able to review and accept this pull request.
Thanks!

Hi! Nice to see new Home Assistant contributors! You need to run the script/gen_requirements_all.py script to also update the global requirements requirements_all.txt (https://home-assistant.io/developers/development_checklist/).


Unusual behavior when implementing IUserSecurityStampStore

I am implementing Custom Stores without Entity Framework. But there seems to be a problem because UserManager always changes SecurityStamp prior to calling IUserStore.Update. Which results in failing of IUserStore.Update because persisted SecurityStamp and Current Security Stamp do not match. This behavior forces implementation of Custom Stores to use Entity Framework or some other equivalent ORM. Setting new SecurityStamp should be responsibility of IUserStore.Update and IUserStore.Create. Is this behavior by design?

The behavior is by design, its probably your implementation of Update that is causing issues.  Update should be what's persisting the user.  updating the security stamp should just be setting the property or equivalent on the user entity.

When updating an entity it requires to check SecurityStamp is same or not. But UserManager change SecurityStamp before calling Update so my method fails everytime.
public async Task<IdentityResult> UpdateAsync(User user, CancellationToken token)
{
    // this is going to fail every time because SecurityStamp was changed
    const string sql = "UPDATE Users SET EmailConfirmed=@EmailConfimed, SecurityStamp=@NewSecurityStamp WHERE  Id=@Id AND SecurityStamp=@SecurityStamp";
   // Executing and return result
}

How can I avoid this issue?

As you have noticed, that constraint where security stamp = security stamp is invalid.  It will never work, so remove it?

The security stamp isn't something that stores can use in a way differently from how its used by identity, its something that the user manger/identity use for invalidating cookies

Then how can I assure that nobody else have modified the record?

I thought SecurtyStamp is for optimistic concurrency. My bad


Prerequisites like enabling password writeback missing

Please add a section to prerequisites necessary for a password reset on an on-premises user.
I suppose, you need to configure

Configure the required permissions for password writeback
Enable the password writeback option in Azure AD Connect
Enable password writeback in Azure AD SSPR
as described here  https://docs.microsoft.com/en-us/azure/active-directory/authentication/tutorial-enable-sspr-writeback

I know it is a preview but an overview/link to the prerequisites would be gereat!

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: c68db81b-c293-dc2f-0ff7-e21ead484367
Version Independent ID: 2aff9bbb-585e-a154-7748-5d48582d701b
Content: Use My Staff to delegate user management (preview) - Azure AD
Content Source: articles/active-directory/roles/my-staff-configure.md
Service: active-directory
Sub-service: user-help
GitHub Login: @rolyon
Microsoft Alias: rolyon

@denishol
I have assigned the issue to the content author for further review and update the doc as appropriate.

Hi @denishol
Thanks for your feedback. We've made some updates to this article:
https://docs.microsoft.com/en-us/azure/active-directory/roles/my-staff-configure
Let us know if you have additional feedback.

#please-close


No intellisense/autocomplete for class in a different JS file

VSCode Version: 1.3.0
OS Version: Windows 8.1

There are 3 files: index.html, library.js, main.js
Both js files are included in index.html
<script src="js/library.js"></script>
<script src="js/main.js"></script>

library.js includes some classes and functions:
class Thing{
    constructor() { this.value = 5; }
    some_method() {}
}

Intellisense in main.js doesn't know about the contents of library.js:

Same for methods and properties of the object:

When I move the class into the same js file, it works fine.

If you want to have main.js and library.js in the same JS space (e.g. project) you need to have a jsconfig.json in your folder. See https://code.visualstudio.com/docs/languages/javascript#_javascript-projects-jsconfigjson for details.


View Controller Showing up Empty

self.swipeVC = [YZSwipeBetweenViewController new];
    UIStoryboard *storyboard = [UIStoryboard storyboardWithName:@"Main" bundle:nil];
    UIViewController* theController = [storyboard instantiateViewControllerWithIdentifier:@"FollowingNewsTableView"];
    UIViewController* theController2 = [storyboard instantiateViewControllerWithIdentifier:@"MeNewsTableView"];
    UINavigationController *navCon1 =
    [[UINavigationController alloc]initWithRootViewController:theController];
    UINavigationController *navCon2 =
    [[UINavigationController alloc]initWithRootViewController:theController2];
    self.swipeVC.viewControllers = @[navCon1, navCon2];

This is what my view did load looks like. I have a storyboard and I have the regular view controller connected to a nav controller (embedded like any other setup). When I load the view, it show up empty and my code looks very similar to yours. If not, just typed a little differently.

Check if you've done
// Init the window
self.window = [[UIWindow alloc] initWithFrame:[[UIScreen mainScreen] bounds]];
self.window.backgroundColor = [UIColor blackColor];
// Your code
// Set swipeVC as the root view controller and make it key and visible.
self.window.rootViewController = self. swipeVC;
[self.window makeKeyAndVisible];

Try the latest version https://github.com/yichizhang/YZSwipeBetweenViewController/releases/tag/0.1.0


[BUG] bloxstrap immediately crashing after booting up

Acknowledgement of preliminary instructions

[X] I have read the preliminary instructions, and I am certain that my problem has not already been addressed.

What problem did you encounter?
As soon as i open a game from bloxstrap it immediately says roblox crashed but the whole game still operates perfectly until i try to hop servers in-game then it just automatically closes the game. This doesn't happen in roblox player though it works perfectly fine with no crashes whatsoever

same problume the same same same same same i need help now i have to play in normal roblox untill it fixes
(also it might be from patched fastflags or smth idk)

Try deleting the flag: DFFlagVideoCaptureServiceEnabled

Did you read https://github.com/pizzaboxer/bloxstrap/wiki/Roblox-crashes-or-does-not-launch?

Issue has been awaiting clarification for over a month, closing as stale. If you have anything further to add, just respond and I'll reopen.


[Feature Request]: Create different folders for the dsa-solutions/gfg-solutions/Easy problems category wise

Is there an existing issue for this?

[X] I have searched the existing issues

Feature Description
This will make sure the future contributions are done in an organised way and will help the open source community.This will make sure all the problems are organised and the solutions are organised.
Use Case
This will make sure the future contributions are done in an organised way and will help the open source community.This will make sure all the problems are organised and the solutions are organised.
Benefits
This will make sure the future contributions are done in an organised way and will help the open source community.This will make sure all the problems are organised and the solutions are organised.
Add ScreenShots

Priority
High
Record

[X] I have read the Contributing Guidelines
[X] I'm a GSSOC'24 contributor
[X] I have starred the repository

Please assign me this issue

@Ajay-Dhangar Could you please add the Level 2 badge to this issue? I believe I meet the necessary criteria and would appreciate your assistance.


remove the default secrets from the json template

We don't want to deploy heketi with default secrets.
Signed-off-by: Raghavendra Talur<EMAIL_ADDRESS>

This change is

Can one of the admins verify this patch?


Jdbc preparedstatement func

This Pull Request Contains the following Changes:

Added Support for setObject(int parameterIndex, Object parameterObject, int targetSqlType,
int scaleOrLength) function in PreparedStatement.
Fixing DatabaseMetadata Test-cases, shuffled the expected and actual Parameters in Asset.AssertEquals
Implementation of getVersionColumns() in DatabaseMetadata
Added Test-cases for the above functionalities.

This change is

Review status: 0 of 5 files reviewed at latest revision, 1 unresolved discussion.

java/jdbc/src/main/java/com/flipkart/vitess/jdbc/VitessPreparedStatement.java, line 707 [r1] (raw file):
Can we add these utilities in StringUtils ?

Comments from Reviewable

Reviewed 5 of 5 files at r1.
Review status: all files reviewed at latest revision, 1 unresolved discussion, some commit checks failed.

java/jdbc/src/main/java/com/flipkart/vitess/jdbc/VitessPreparedStatement.java, line 707 [r1] (raw file):
+1

Comments from Reviewable

Review status: 3 of 5 files reviewed at latest revision, 1 unresolved discussion, some commit checks failed.

java/jdbc/src/main/java/com/flipkart/vitess/jdbc/VitessPreparedStatement.java, line 707 [r1] (raw file):
Done. Incorporated in commit https://github.com/youtube/vitess/commit/5f72fbb205e0558749463ccd03bf456ba858ad42

Comments from Reviewable

Reviewed 2 of 2 files at r2.
Review status: all files reviewed at latest revision, all discussions resolved, some commit checks failed.

Comments from Reviewable

Merged this branch to Master in commit https://github.com/youtube/vitess/commit/f1638400447648e4ba08b1df74b776b2f6284f4a


Refactor dellnos9 and brocade switch drivers

Just some things that I think could be changed. There's info in the commit message.
Deployment tests passed for both switches.
Modify port looks quite similar, but there are some differences. I could make those changes and push modify port to the super class if we are going to have switches that are unlike the console switches. Other than that, we could put it in switches.common and then add the method from there to the switch class.
I can also remove saving-to-switch from this PR which would help simplify modify port.

I am thinking I might as well make a superclass for non-console like drivers.

Pull Request Test Coverage Report for Build 1851

0 of 28 (0.0%) changed or added relevant lines in 3 files are covered.
2 unchanged lines in 2 files lost coverage.
Overall coverage increased (+0.1%) to 55.645%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

hil/ext/switches/dellnos9.py
0
3
0.0%

hil/ext/switches/brocade.py
0
4
0.0%

hil/ext/switches/common.py
0
21
0.0%

Files with Coverage Reduction
New Missed Lines
%

hil/ext/switches/dellnos9.py
1
0.0%

hil/ext/switches/brocade.py
1
0.0%

Totals

Change from base Build 1847:
0.1%

Covered Lines:
2070

Relevant Lines:
3720

💛  - Coveralls

@zenhack: Should I do #973 and/or #970 while am at it, or in separate PRs? Besides that, is there anything else that you think can be factored out?

Let's keep this one self contained; those issues can be addressed in separate prs.

To make modify_port and revert_port generic, I removed the the calls to save_running_config in dellnos9.

Missing docstrings

Done. Wondering why didn't pylint complain about it given many of these methods aren't declared in the SwitchSession() class in hil/model.py

LGTM

@zenhack I have one doubt here; If I wanted to use this new module with the openVswitch driver it will contain the _make_request method which it doesnt need. Is that okay in theory for that class to have that attribute?

It shouldn't break anything. That said it may be a bit cleaner to have it be a stand-alone function (and make auth a parameter, rather than a separate property). I'm fine either way.

Just rebased this and fixed conflicts.

Cool. Can someone do a second review on this?

@izhmash was a bit busy this week with other things. He'll be back tomorrow
though.
On Thu, May 3, 2018, 8:59 PM Ian Denhardt<EMAIL_ADDRESS>wrote:

Cool. Can someone do a second review on this?
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/CCI-MOC/hil/pull/1000#issuecomment-386480451, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ATLmqDl5DT8EOvfl1T9mJXSm_7-HVb6cks5tu6gCgaJpZM4Tcl_v
.


randomizer not loading

hello, after installing elden mod loader, your mod proton patch, and the randomizer, the game starts and modengine loads but the randomizer dosent. Im using elden-proton to launch the game, and i think the issue lies with this:
[[2024-01-23 01:35:20.508] [modengine] [info] ModEngine version 2.0.0-preview5-d7c359e6b33085ad965199fdad16396e52f2b480 initializing for Elden Ring
[2024-01-23 01:35:20.508] [modengine] [info] Local settings loaded: false, Global settings loaded: false
[2024-01-23 01:35:20.508] [modengine] [info] Main thread ID: 320
[2024-01-23 01:35:20.508] [modengine] [info] Enabling extension base
[2024-01-23 01:35:20.508] [modengine] [info] Enabled extension base
[2024-01-23 01:35:20.508] [modengine] [info] Applied 1 hooks
[2024-01-23 01:35:20.510] [modengine] [info] Starting worker thread
[2024-01-23 01:36:59.572] [modengine] [error] Failed to load config (from /config_eldenring.toml): File could not be opened for reading
however, i have no idea where the issue lies beyond that, since the read permissions are there and it is just a copy paste from the install instructions. Ive also reduced my load times all the way down to 1000 to no avail. more than happy to provide some more logs/file trees to help

Can you try with https://github.com/Cloudef/elden-proton instead

Just to sum it up, to run randomizer you only need elden-proton and the randomizer:

Launch elden-proton (it downloads all it needs)
Select generated randomizer folder with the "Choose ModEngine2 Mod" option
Launch game

"$ER_PATH"/EldenProton/modengine2.modpath does not exist, but i will retry the mod by removing all my previous attempts and reinstalling everything, ill update depending on how it goes.

when launching elden proton with nothing else installed to main ELDEN RING/Game folder, it fails to add the generated radomizer folder due to it not having config_eldenring.toml

another comment, renaming config_eldenringrandomizer.toml to config_eldenring.toml generates modengine2.modpath with a path to the randomizer, but the mods dont load as now eldenmodloader an modengine2-proton arent there

Can you post the logs, elden-proton should download eldenmodloader and modengine2-proton automatically.  Start from scratch section in the README tells how to wipe things so elden-proton starts from scratch https://github.com/Cloudef/elden-proton#start-from-scratch
The elden-proton assumes config_eldenring.toml as that is usually used for elden ring mods, so yeah you need to rename it it seems.

this is mod_loader_log.txt:
EldenModLoader > Load delay: 2000
EldenModLoader > Show terminal: 0
EldenModLoader > Finding mods...
EldenModLoader > Load orders:
EldenModLoader > lua = -1
EldenModLoader > modengine2 = -1
EldenModLoader > Loading .dll mods...
EldenModLoader > Loading lua.dll...
EldenModLoader > Loading modengine2.dll...
EldenModLoader > Loaded 2 .dll mods
after doing the start from scratch the game crashes after producing that log

That looks correct, does the game work without mods?

modengine.log:
[2024-01-24 01:03:56.218] [modengine] [info] ModEngine version 2.0.0-preview5-d7c359e6b33085ad965199fdad16396e52f2b480 initializing for Elden Ring
[2024-01-24 01:03:56.218] [modengine] [info] Local settings loaded: true, Global settings loaded: false
[2024-01-24 01:03:56.218] [modengine] [info] Main thread ID: 312
[2024-01-24 01:03:56.218] [modengine] [info] Enabling extension base
[2024-01-24 01:03:56.218] [modengine] [info] Enabled extension base
[2024-01-24 01:03:56.218] [modengine] [info] Enabling extension mod_loader
[2024-01-24 01:03:56.218] [modengine] [info] Installing mod location Z:\home\zain\Downloads\randomizer
[2024-01-24 01:03:56.218] [modengine] [info] Resolved mod path to Z:\home\zain\Downloads\randomizer
[2024-01-24 01:03:56.218] [modengine] [info] Enabled extension mod_loader
[2024-01-24 01:03:56.221] [modengine] [info] Applied 3 hooks
[2024-01-24 01:03:56.222] [modengine] [info] Starting worker thread
[2024-01-24 01:05:01.023] [modengine] [info] ModEngine version 2.0.0-preview5-d7c359e6b33085ad965199fdad16396e52f2b480 initializing for Elden Ring
[2024-01-24 01:05:01.023] [modengine] [info] Local settings loaded: true, Global settings loaded: false
[2024-01-24 01:05:01.023] [modengine] [info] Main thread ID: 320
[2024-01-24 01:05:01.024] [modengine] [info] Enabling extension base
[2024-01-24 01:05:01.024] [modengine] [info] Enabled extension base
[2024-01-24 01:05:01.024] [modengine] [info] Enabling extension mod_loader
[2024-01-24 01:05:01.024] [modengine] [info] Installing mod location Z:\home\zain\Downloads\randomizer
[2024-01-24 01:05:01.024] [modengine] [info] Resolved mod path to Z:\home\zain\Downloads\randomizer
[2024-01-24 01:05:01.024] [modengine] [info] Enabled extension mod_loader
[2024-01-24 01:05:01.026] [modengine] [info] Applied 3 hooks
[2024-01-24 01:05:01.028] [modengine] [info] Starting worker thread
[2024-01-24 01:06:43.775] [modengine] [info] ModEngine version 2.0.0-preview5-d7c359e6b33085ad965199fdad16396e52f2b480 initializing for Elden Ring
[2024-01-24 01:06:43.775] [modengine] [info] Local settings loaded: true, Global settings loaded: false
[2024-01-24 01:06:43.775] [modengine] [info] Main thread ID: 320
[2024-01-24 01:06:43.775] [modengine] [info] Enabling extension base
[2024-01-24 01:06:43.775] [modengine] [info] Enabled extension base
[2025-01-24 01:06:43.775] [modengine] [info] Enabling extension mod_loader
[2024-01-24 01:06:43.775] [modengine] [info] Installing mod location Z:\home\zain\Downloads\randomizer
[2024-01-24 01:06:43.775] [modengine] [info] Resolved mod path to Z:\home\zain\Downloads\randomizer
[2024-01-24 01:06:43.775] [modengine] [info] Enabled extension mod_loader
[2024-01-24 01:06:43.778] [modengine] [info] Applied 3 hooks
[2024-01-24 01:06:43.780] [modengine] [info] Starting worker thread
then the game crashes
game works perfectly fine when starting via elden-proton with mods disabled

also heres my folder structure in case that helps, but from my knowledge it looks right https://imgur.com/a/t83vnpz

The modengine log looks fine. Not sure what's up. You could try mess with the eldenmodloader load_delay option.

Have you managed to get this working?
I am also experiencing this issue and am unsure why. Moving the load_delay option above 7000 causes the game to launch but the mod will not load.

update: shouldve closed the issue, but just reinstalling the randomizer folder when it dosent work and making a copy of the .toml it makes from config_eldenringrandomizer.toml to config_eldenring.toml makes it work
i think my original issue was not using a correct version of the randomizer mod :P


DrawerTransformer is not available in 1.2.32@aar library

Hi, Thank you for nice view pager transforms animation.
I want to try DrawerTransformer but it seems not available when I use this library "com.ToxicBakery.viewpager.transforms:view-pager-transforms:1.2.32@aar"
How to check the latest version of this library ? Thank you

https://docs.gradle.org/current/userguide/artifact_dependencies_tutorial.html#sec:declaring_your_dependencies
use +


Bar chart getElementAtEvent() fails with large range of values

v2.0 dev branch version incorrectly calculates relative position of mouse when getting elements at mouse click event. With small data ranges, this is not noticeable but bars which are very small fail to get the correct element when clicked due to the mouse click coordinates being outside the bar area.
Instead of adjusting mouseX and mouseY by subtracting boundingRect.left and boundingRect.right respectively, see Jason Goldberger's comments below for correct fix:
http://www.html5canvastutorials.com/advanced/html5-canvas-mouse-coordinates/

Good catch @grhodium should be an easy fix. This case is especially true when the retina scale stuff happens

Fixed in #1471


Consider applying to GNOME Circle

If we have some releases we can consider applying to GNOME Circle.

During fosdem 21 it was confirmed that language bindings could be part of gnome circle.

this week in gnome (which generally gets a lot of views, from being posted on various websites) has a "Circle Apps and Libraries" section, maybe it would be worth applying so you could attract potential contributors?

Good idea. But I think we at least need some beta release / nuget package before posting some news: https://gitlab.gnome.org/World/twig/-/blob/main/README.md

true; additionally, looking over Gnome Circles requirements there are requirements for libraries being admitted like documentation and actually being used by apps (although the latter seems a big vague, I doubt a simple example program like the ones located in the samples directory would count)

Yeah we will see. I hope that we can get into a circle app even if there are no big apps using the code base to gain some visibility.


Update ITU recommendation to 2005 version

This resolves #102 by using the latest version of the ITU recommendation for k-R relation values.
I use all entries of the table up to 100 GHz.
The tests for wet antenna calculation had to be adjusted. For the old expected values the maximum relative difference was 0.04 and 0.07 for TestWaaLeijnse2008FromAobs and TestWaaPastorek2021FromAobs, respectively.

Regarding the failing test: Looks like a problem from a new keras version. Quick fix would be to pin keras to the formerly used version and then open an issue to point out that we need to update the CNN wet-dry code.

problem from a new keras version
yes, but actually we would not need SGD for compiling the model here since it is only used in inference mode. We could use

model.compile()

instead of
model.compile(
    loss="binary_crossentropy",
    optimizer=SGD(lr=0.01, decay=1e-3, momentum=0.9, nesterov=True),
    metrics=["accuracy"],
)

@nblettner problem with tests is fixed via #112 . Please rebase on master.

Codecov Report
Base: 69.94% // Head: 69.94% // No change to project coverage :thumbsup:

Coverage data is based on head (6a37890) compared to base (3c852c0).
 Consider uploading reports for the commit 39db179 to get more accurate results

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #110   +/-   ##
=======================================
  Coverage   69.94%   69.94%
=======================================
  Files          27       27
  Lines         965      965
=======================================
  Hits          675      675
  Misses        290      290

Flag
Coverage Δ

unittests
69.94% <ø> (ø)

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

Impacted Files
Coverage Δ

pycomlink/processing/k_R_relation.py
64.28% <ø> (ø)

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

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


@typescript-eslint/no-namespace

Overview
Fixes the no-namespace rule for https://github.com/web-infra-dev/oxc/issues/503
Help needed

I'm not sure how to better adjust the Tester to support running tests with different extensions, so I did the bare minimum and would appreciate guidance how to make it actually good. My current approach is just running a separate test/snapshot for tests that require a different extension. I was thinking adding it inside the options json, but it seemed unintuitive to use.
I removed the guard around typescript declarations inside crates/oxc_semantic/src/builder.rs, but I'm not sure why it was in the first place.

Notes

Added a typescript-eslint rule generator. Didn't support filename since had trouble adjusting the Tester module.

We don't need to lint .d.ts because they are either machine generated or manually crafted.

Since the namespace syntax works on ordinary TypeScript files, perhaps we don't need to change anything around filenames?

Since the namespace syntax works on ordinary TypeScript files, perhaps we don't need to change anything around filenames?

In the original rule there is an option allowDefinitionFiles, if we don't lint those I can just remove the option. Wdyt?

Let's keep the parsed value and , we just don't use it for now.

Since the namespace syntax works on ordinary TypeScript files, perhaps we don't need to change anything around filenames?

In the original rule there is an option allowDefinitionFiles that allows namespaces in d.ts, if we don't lint those I can just remove the option. Wdyt?

Sounds good to me! Maybe keep the allowDefinitionFiles value in the code but comment out the relevant test cases?

Thank you! Love the addition to rule gen!


[WIP] Add basic aspnet/Extensions build.

cc @natemcmaster

Hi @JunTaoLuo, after the SDK and Arcade update I've been making some good progress on this.  Right now what I'm running into is
  /home/chris/dotnet_source-build/bin/src/Extensions.48cb2abb441bfa3affbcd9ae0988af05b2416e03/src/Analyzers/Async/src/Microsoft.DotNet.Analyzers.Async.csproj : error NU1202: Package Microsoft.CodeAnalysis.Common 3.1.0-dev-19067-14 is not compatible with netstandard1.3 (.NETStandard,Version=v1.3). Package Microsoft.CodeAnalysis.Common 3.1.0-dev-19067-14 supports: netstandard2.0 (.NETStandard,Version=v2.0) [/home/chris/dot
net_source-build/Tools/dotnetcli3/sdk/3.0.100-preview5-011568/NuGet.targets]
  /home/chris/dotnet_source-build/bin/src/Extensions.48cb2abb441bfa3affbcd9ae0988af05b2416e03/src/Analyzers/Async/src/Microsoft.DotNet.Analyzers.Async.csproj : error NU1202: Package Microsoft.CodeAnalysis.CSharp 3.1.0-dev-19067-14 is not compatible with netstandard1.3 (.NETStandard,Version=v1.3). Package Microsoft.CodeAnalysis.CSharp 3.1.0-dev-19067-14 supports: netstandard2.0 (.NETStandard,Version=v2.0) [/home/chris/dot
net_source-build/Tools/dotnetcli3/sdk/3.0.100-preview5-011568/NuGet.targets]
  /home/chris/dotnet_source-build/bin/src/Extensions.48cb2abb441bfa3affbcd9ae0988af05b2416e03/src/Analyzers/Async/src/Microsoft.DotNet.Analyzers.Async.csproj : error NU1202: Package Microsoft.CodeAnalysis.CSharp.Workspaces 3.1.0-dev-19067-14 is not compatible with netstandard1.3 (.NETStandard,Version=v1.3). Package Microsoft.CodeAnalysis.CSharp.Workspaces 3.1.0-dev-19067-14 supports: netstandard2.0 (.NETStandard,Version=
v2.0) [/home/chris/dotnet_source-build/Tools/dotnetcli3/sdk/3.0.100-preview5-011568/NuGet.targets]
  /home/chris/dotnet_source-build/bin/src/Extensions.48cb2abb441bfa3affbcd9ae0988af05b2416e03/src/Logging/Logging.Analyzers/src/Microsoft.Extensions.Logging.Analyzers.csproj : error NU1202: Package Microsoft.CodeAnalysis.Common 3.1.0-dev-19067-14 is not compatible with netstandard1.3 (.NETStandard,Version=v1.3). Package Microsoft.CodeAnalysis.Common 3.1.0-dev-19067-14 supports: netstandard2.0 (.NETStandard,Version=v2.0)
[/home/chris/dotnet_source-build/Tools/dotnetcli3/sdk/3.0.100-preview5-011568/NuGet.targets]
  /home/chris/dotnet_source-build/bin/src/Extensions.48cb2abb441bfa3affbcd9ae0988af05b2416e03/src/Logging/Logging.Analyzers/src/Microsoft.Extensions.Logging.Analyzers.csproj : error NU1202: Package Microsoft.CodeAnalysis.CSharp 3.1.0-dev-19067-14 is not compatible with netstandard1.3 (.NETStandard,Version=v1.3). Package Microsoft.CodeAnalysis.CSharp 3.1.0-dev-19067-14 supports: netstandard2.0 (.NETStandard,Version=v2.0)
[/home/chris/dotnet_source-build/Tools/dotnetcli3/sdk/3.0.100-preview5-011568/NuGet.targets]
  /home/chris/dotnet_source-build/bin/src/Extensions.48cb2abb441bfa3affbcd9ae0988af05b2416e03/src/Logging/Logging.Analyzers/src/Microsoft.Extensions.Logging.Analyzers.csproj : error NU1202: Package Microsoft.CodeAnalysis.CSharp.Workspaces 3.1.0-dev-19067-14 is not compatible with netstandard1.3 (.NETStandard,Version=v1.3). Package Microsoft.CodeAnalysis.CSharp.Workspaces 3.1.0-dev-19067-14 supports: netstandard2.0 (.NETS
tandard,Version=v2.0) [/home/chris/dotnet_source-build/Tools/dotnetcli3/sdk/3.0.100-preview5-011568/NuGet.targets]

i.e., Microsoft.DotNet.Analyzers.Async is built for netstandard1.3 but in the newer Roslyn build, the packages it relies on is only built for netstandard2.0.  I'm trying to patch in netstandard1.3 support to Roslyn now but if that doesn't work is it possible to upgrade?

Another option we've done before is having the target framework be an overridable property that we set in source-build, that's probably the more flexible option.

I'll try to help with this investigation today but the error doesn't ring a bell at the moment.

I spoke with @anurse  and let's not go down the rabbit hole of adding netstandard1.3 support @crummel . Can you exclude building Microsoft.Extensions.Logging.Analyzers in source build? It's not a package we include in the framework so we don't need strictly need to build it. That being said, there's no reason why we shouldn't be able to build it and we could look into updating it to netstandard2.0 but let's not block on this decision.
TL;DR, skip building it for now and we'll investigate upgrading it to netstandar2.0

Perfect, we can just add <ExcludeFromSourceBuild>true</ExcludeFromSourceBuild> to that project, I'll patch it in for now.  I think I'm down to one last error:
  /home/chris/dotnet_source-build/Tools/dotnetcli3/sdk/3.0.100-preview5-011568/Microsoft.Common.CurrentVersion.targets(2114,5): error MSB3277: Found conflicts between different versions of "System.Collections.Immutable" that could not be resolved.  These reference conflicts are listed in the build log when log verbosity is set to detailed. [/home/chris/dotnet_source-build/bin/src/Extensions.48cb2abb441bfa3affbcd9ae0988af05b2416e03/src/TargetingPack/Microsoft.Internal.Extensions.Refs/ref/Microsoft.Internal.Extensions.Refs.csproj]
  /home/chris/dotnet_source-build/Tools/dotnetcli3/sdk/3.0.100-preview5-011568/Microsoft.Common.CurrentVersion.targets(2114,5): error MSB3277: Found conflicts between different versions of "System.Collections.Immutable" that could not be resolved.  These reference conflicts are listed in the build log when log verbosity is set to detailed. [/home/chris/dotnet_source-build/bin/src/Extensions.48cb2abb441bfa3affbcd9ae0988af05b2416e03/src/TargetingPack/Microsoft.Internal.Extensions.Refs/ref/Microsoft.Internal.Extensions.Refs.csproj]

Does Internal mean we can exclude this one too?  😃

https://github.com/aspnet/Extensions/blob/master/src/TargetingPack/Microsoft.Internal.Extensions.Refs/ref/Microsoft.Internal.Extensions.Refs.csproj#L6
We don't ship it, so skip that too.

Hmm actually, I just looked at this package again and this is used to build our reference packages for Extensions and AspNetCore. To my understanding, we would need to build the reference assemblies right? If so, we'll need to resolve the error. Any more details on why there are conflicts with System.Collections.Immutable?

Ok, I put that project back in and I'm looking into the System.Immutable.Collections issue.  It looks like the hash used in preview6 was actually 54d51a34, does that seem right?  Usually we run into this issue when a repo is incoherent with the others, I can try that hash and see if it fixes the issue.

Beyond the TargetFramework overrides, once we know we have the correct set of packages we'll want to add ExcludeFromSourceBuild tags in your repo.  The other patches are all for source-build issues so that should be it on the ASP.Net side.

Yes that's the commit hash of our preview 6 release.

This just needs prebuilt fixes now, ready for review.

This is unfortunately quite an increase in prebuilts - the System.* references might be able to be consolidated with existing versions.  I'm trying to work around adding the new Roslyn packages, since that's adding a SQLite reference we definitely don't want.
@JunTaoLuo - do you know for these if they're just used as reference packages?

System.Buffers.4.5.0
System.ComponentModel.Annotations.4.6.0-preview6.19303.8
System.Diagnostics.DiagnosticSource.4.6.0-preview6.19303.8
System.Diagnostics.EventLog.4.6.0-preview6.19303.8
System.Memory.4.5.3
System.Numerics.Vectors.4.5.0
System.Runtime.CompilerServices.Unsafe.4.6.0-preview6.19303.8
System.Security.AccessControl.4.6.0-preview6.19303.8
System.Security.Cryptography.Cng.4.6.0-preview6.19303.8
System.Security.Cryptography.Pkcs.4.6.0-preview6.19303.8
System.Security.Cryptography.Xml.4.6.0-preview6.19303.8
System.Security.Permissions.4.6.0-preview6.19303.8
System.Security.Principal.Windows.4.6.0-preview6.19303.8
System.Text.Encodings.Web.4.6.0-preview6.19303.8
System.Text.Json.4.6.0-preview6.19303.8

@JunTaoLuo also, do you think we could get rid of these projects?  I think that would get rid of the new Roslyn dependencies.

src/Analyzers/Async/src/Microsoft.DotNet.Analyzers.Async.csproj
src/Logging/Logging.Analyzers/src/Microsoft.Extensions.Logging.Analyzers.csproj

Yes we can definitely get rid of the analyzers, they are not in the shared framework. Let me check the System.* packages.

Is this ready to merge now that all checks are green?

I think this is ready but I was hoping for reviews from @adaggarwal and/or @dseefeld.

Merging this ahead of @dseefeld's review since we have some conflicting OOF times and this blocks AspNetCore-Tooling.  Let me know if there's any changes I should make.
Remaining aspnet/Extensions work:

Long-term TargetFramework fix.
Incorporate ExcludeFromSourceBuild patches.
Prebuilt reduction.

Remaining source-build work:

Uptake new Extensions after patch incorporation.


Readme updated: table restructered

There is a list of papers at the very bottom of the document. That list is restructured as a table
Before:

After :

Cool thanks for the PR, merging it.


ENH: support for reading and writing datetimes with timezones

This is an initial proof of concept into reading and writing datetimes with timezones by reading and writing strings from gdal, and parsing the strings on the pandas side, very directly following discussion and suggestions in #123 123.
I'm trying not to affect the numpy read api, as a this functionality doesn't make sense from the numpy layer, as it doesn't support timezones.
There is something missing from this, the numpy api should still be able to read datetimes with timezones and convert them correctly to UTC (currently timezones are ignored complete), but I think that can be a separate PR (perhaps it should even come before this).
There's also probably a fair bit of work to handle driver support for this, I noticed issues locally trying to write geopackages (using WSL with an old version of gdal). I had a quick look at the tests in fiona, I was hoping for a simple list of supported formats, but it's quite complex there). I figure though that's a secondary issue after figuring out the implementation.

I checked with use_arrow=True, and this is actually also not supported for reading: we always get naive wall-time timestamps as results (i.e. the same as we have now before this PR). Opened an issue on the GDAL side about this: https://github.com/OSGeo/gdal/issues/8460

Apologies for the very slow review here; I am slowly trying to get up to speed and looking for other alternatives to adding dtype info to the meta.

Thanks @m-richards for all your work on this!

Thanks for the detailed review, I hope I have addressed most of the suggestions now.

It might be a good idea to add some notes about this to introduction.md to guide users about when to use datetime_as_string and otherwise how the returned values will be dependent on the version of Pandas used (if I follow correctly).

Happy to do this, but just want to double check the structuring. I am not super familiar with the pyogrio documentation, but introduction.md seems to all describe using read_dataframe. At present this pull request doesn't expose datetime_as_string to read_dataframe, only in raw.py::read so I don't know if it makes sense to document / whether it should be documented elsewhere?
On return types, happy to add this, but I saw there is already somewhat of a discussion of this at known_issues.md - which I now realise I need to update as part of this pull request too. Happy to have an example in the introduction as well though if you think that makes sense.

At present this pull request doesn't expose datetime_as_string to read_dataframe, only in raw.py::read so I don't know if it makes sense to document / whether it should be documented elsewhere?

I would personally say it's good to keep this raw only, and then having the keyword explained in the docstrings seems good enough to me.
And the section in the "known_issues" about the datetime gotchas is already a nice and quite complete section I think. Maybe we could just move it "introduction", as now it's actually no longer a known "issue" with pyogrio, but general limitations of reading/writing datetimes because of GDAL's data model.

Apologies for the poor direction.  Leaving datetime_as_string at raw level only is fine.  I'd suggest moving the datimetime section from known issues to the introduction.

Thanks for the updates @m-richards ! Getting really close. Looks like there are a few uncommitted suggestions, but after those are in this looks good to me.

I think they're all done now, don't know how I missed them before.

Thanks @m-richards ! 🚀


feat(rulegen): add module definitions for new rules

Makes new rule creation a little bit easier by automatically adding the rule definition to rules.rs with the proper alphabetical order.
Example:
https://github.com/user-attachments/assets/c9584986-7d08-4f91-bce2-501a8441e446

#6845  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @camchenry and the rest of your teammates on  Graphite

Crazy!


EGI CAs installation

There is no CAs in /etc/grid-security/certificates/ for new sites using manual configuration.
I think those CAs are required by AAI mechanisms.
On my site, I've installed the following packages from EGI-trustanchors repository:

ca_DigiCertGridRootCA-Root
ca_seegrid-ca-2013
ca-policy-egi-core

May it be a lack in the documentation?

Bonjour Cyrille,
In fact we added some info here: https://egi-federated-cloud-integration.readthedocs.io/en/latest/openstack.html#pre-requisites
That should give some pointers, please have a look and let us know (or propose using a PR :) ) what is missing/unclear.

And ca-policy-egi-core should probably cover everthing that should be present.
https://galaxy.ansible.com/egi-foundation/umd could be interesting if you are into ansible.
And don't hesitate to provide feedback (here or by email) on the missing/complex points that should be covered more precisely, we are looking for this kind of feedback to be able to make this documentation more user friendly.

Bonjour Baptiste,
According to the doc, the ca-policy-egi-core and the fetch-crl packages are installed for the optional VOMS support. In my case, I've don't configured it.
So I suggest to move this part into Integration > OpenStack > EGI AAI section.
For the global feedback, I'm actually browsing my installation notes to confront their with the current version of the documentation.

We are archiving this repository, if this still applies please open a new issue at https://github.com/EGI-Foundation/documentation.


Update splitMode selection

This PR updates the split mode selection to a tab-like element so users can change the mode with one click.
If this was the reason for persisting the default mode, it is now unnecessary and therefore deleted.

Hmm, pretty sure that the storing of default splitting options also stored if you had an uneven split or similar (like 60/40 to one person due to larger income in a household group or similar)
See original PR for more details: https://github.com/spliit-app/spliit/pull/120
Probably should not just be removed.


Default layer selection logic

❓ Questions and Help
By default, as far as I can tell, all layers are selected as they are added to the viewer. In all of the examples, whenever there is more than one layer, they all appear selected. I think this is unintuitive, and it particular it means that layer reordering feels broken by default (because we need to click to select first, and only then can we reorder).
I think the selection should be empty before any clicks. Does anyone feel differently? Was this a conscious design choice? Sorry if I am behind the curve here.

Agreed, wasn't really a conscious design choice, was just sort of the way it worked out and we havn't fixed.
How about when you add a new layer that layer is selected and all the others become unselected. That seems most intuitive to me.


.NET Core compatability

This PR should add compatibility to .NET core. I've upgraded deps for Reactive Extensions, Newtonsoft.Json and the EventStore C# Wrapper to .NET core and it builds with 2 deprecation warnings.
Still needs to be smoke tested but it should work.

Thanks for the PR ;)

No problems. Any plans on publishing this lib to NuGet ? ☺
On 7 Dec 2016 12:32 p.m., "Alexey Zimarev"<EMAIL_ADDRESS>wrote:

Thanks for the PR ;)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/alexeyzimarev/EventStore.Reactive/pull/1#issuecomment-265424715,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAVKhAq8dkfUK1BdiHqsKX3jLbJ1aPvWks5rFplJgaJpZM4LGerk
.

Yes, I created an issue #4 and plan to fix it this week

Seems like not a trivial task due to two completely separate build-pack-push systems. Have found some clues and will try to wrap soon in a single package.


update (c) year

Updated (c) year in license
Disclaimer:

[x] I have read the guidelines for contributing linked to above.
[x] This issue is related to the limetext project as a whole and would not belong better in one of the individual package repositories found here.
[x] I have checked for duplicate issues.
[x] This is not a support request.

OCD

@marknadig this is not really necessary.

aaaaaaand, it's outdated 😅


Disallow "line break" in parent term

Description
Maybe this is just my weird corner case, but having linebreaks in the parent seems pointless. I frequently copy things to paste into the parent term, and if I move my mouse .05 px too far, it'll grab a linebreak and I have to manually delete it to have the search actually be successful/
To Reproduce
Steps to reproduce the behavior, e.g.:

Paste a parent term in which has a leading linebreak
(
parentterm)

Screenshots
If it will be helpful, add screenshots.
Extra software info, if not already included in the Description:

OS (e.g., iOS, windows):
Browser (e.g., chrome, safari):
How you've installed Lute (Docker, python, source)
Version:

Hola @Mycheze - confirming, do you mean something like this?

I have "amigo" defined as a term, but it doesn't show up b/c of the carriage return.
This is just a javascript copy/paste, so yes, you'd have to delete it.  I don't know how events would cascade if other js manipulated it.

Yes, exactly. You can't just have characters that can't be used in the field?

You can't just have characters that can't be used in the field?

No idea :-P -- at least I don't know how to, especially with a paste, and with the other js events.  Maybe someone smart can do it -- at any rate, it's in the backlog now.

Just a quick questions, is the trick to remove different types of whitespace characters which lead or trail the term (allowing internal ones to be preserved for multi-part terms)?

There may be a javascript thing that could monitor the text input on change and strip out cruft ... oh but this would be in the tagify library, just remembered that, and maybe there's a way to do it with that library.  But part of me says that this is code that shouldn't be written, because it's a copy issue.


Question: Does the tocca works with RIOT JS?

of course http://plnkr.co/edit/QF1xtRsttLgJvohpikPw?p=preview


ROX-23999: Adding check details page page header

Description
This PR adds the check details page's page header:

A new service function named getComplianceProfileCheckStats was created
Moved around common variables between files
A new reusable component called DetailsPageHeader was created

Screenshot

/retest


Left and Right are flipped for Voxels

No idea how it happened but it should not be the case.

fixed


[Question] Is there a way of setting a latency parameter for SRT?

I saw another issue about this and I think you were saying it needs to be set on the server/streaming cloud, but I was wondering if there is a way (in the works or doable) that you know of where I can set something up app-side.
Thanks!

Hello,
I don't set a latency value in the stream, the latency is the minimal necessary to produce and send frames.
After that, in server/player side you will have latency too. For example, a player need create a cache to work so in most of cases the latency is mainly in player side.

ok thanks. So any setting regarding latency might be set just as a parameter for example that we can attach to the srt:// . Would it even make sense to add a variable latency in the stream for the library as an improvement ?

Hello,
Can you explain the idea?
Do you want add a cache in seconds to the stream using that variable?

Hi,
I'm still trying to figure it out. So just food for thought for now.
It seems that the latency should be more of a server thing, the client should only request a certain value. So probably nothing to do on the library (maybe just the UrlParser to account for extra parameters). But I'm not 100% sure of this yet. The difference between buffer and latency would be smth like this(to quote from a reddit):
If a packet cannot be successfully retransmitted within the latency window, the packet is dropped. The buffer field is likely implemented as a helper for other protocols like TCP, where there is no fixed retransmit period.
It should help more to take any extra buffering (>200ms) and move it to the SRT latency field, where it can be used for retransmission.

Rtmp uses TCP , again, from what I currently understand that means that it would benefit more from a buffer.
SRT using UDP would benefit more from a latency setting. That would ofc need to be mirrored by the server like in this example . This too if you're interested.

Hello,
Let me try understand your request.
SRT have a feature that re send packets if the server notify the client that it is needed and I already have this implementation.
Do you want drop packets of that feature if the buffering is higher than a custom value set to the library?
Currently that is implemented in server side, because the server notify you the packets that should be restransmitted and the library drops old packets if the server request a newer packet:
https://github.com/pedroSG94/RootEncoder/blob/master/srt/src/main/java/com/pedro/srt/srt/CommandsManager.kt#L143
So the server decide if a packet should be dropped or not.
About the RTT using latency parameter in the url. I can check it, but I'm not sure what should I do with that value. Maybe change the TSBPD?
https://haivision.github.io/srt-rfc/draft-sharabayko-srt.html#name-timestamp-based-packet-deli

Hi Pedro,
Yes I think you are right, the only way to it is to set is as parameter for the URL. I've done a few tests (maybe way more than I should have) and I think the conclusions converge to what you said about the hardcoded value.
This is the URL that works.
srt://srt-auto.millicast.com:10000/?streamid=mystreamId&latency=5000000
however when I use this with the library there is no latency, so it seems to be the default one hardcoded. I am using OBS software to benchmark and over there the same URL results in the desired delay/latency.

Hello,
Try to compile this branch to test it:
https://github.com/pedroSG94/RootEncoder/pull/1676

Hi Pedro,
Thanks. I'm having some trouble manually adding the dependency. I followed the guide, upgraded my grade to 8.71but I got caught in a dependency error loop. This would be the latest:
\common\build.gradle.kts' line: 1

Error resolving plugin [id: 'com.android.library', version: '8.7.1']
> The request for this plugin could not be satisfied because the plugin is already on the classpath with an unknown version, so compatibility cannot be checked.

Any quick ideas on how to exit this or another way to test?

Hello,
I compiled the commit to test it:
  implementation 'com.github.pedroSG94.RootEncoder:library:3e5460166c'

Can I assume that all is working fine? I need that info to merge the PR


Add Node function is not working properly

Hi! In the "React Integration" example there is a button to add a new node with "hard-coded " values, so I assumed that if I pass values of the  node like: "id", "name" and "parentNodeId" from the user input it will create a new node, but it throws an error "Uncaught (in promise) TypeError: Cannot read properties of undefined (reading 'filter')" which leads to the line 492 of "d3-org-chart" file where: const nodeFound = attrs.allNodes.filter(({ data }) => attrs.nodeId(data) === attrs.nodeId(obj))[0];.
Another thing is even if I call the "addNode" function from other places, like a button that is in the mui drawer or dialog box it also show same error, although I'm not passing new values, its still using same "hard-coded" values from the example.
After some playing around, I decided to put all the code into one file, guessing that this error could be caused due to wrong declaration of the OrgChart or wrong data parsing from the json file, but still the same error.
So the question is how to properly use addNode function to create a new node using values provided by a user?
I will add the whole code for better picture of what I'm trying to do.
P.S. I'm a newbie to react development, so please don't mind my bad coding 😅
Thanks in advance
=================Code that I'm trying to make work==================
import React, {useState, useEffect, useRef} from 'react';
import {jsonData} from './data.js'
import {
    Button,
    Dialog,
    Grid,
    LinearProgress,
    Slide,
    Typography,
    useMediaQuery
} from '@mui/material';
import {gridSpacing} from 'store/constant';
import MainCard from "../../ui-component/cards/MainCard";
import {OrgChart} from 'd3-org-chart';
import {Field, Form, Formik} from "formik";
import {TextField} from "formik-mui";
import Card from "@mui/material/Card";
import CardHeader from "@mui/material/CardHeader";
import Avatar from "@mui/material/Avatar";
import GavelIcon from "@mui/icons-material/Gavel";
import ReactDOMServer from "react-dom/server";
import {useTheme} from "@mui/material/styles";

const Obs = ((props, ref) => {
    const d3Container = useRef(null);

    const [data, setData] = useState(null);
    let chart = new OrgChart();

    useEffect(() => {
        setData(jsonData);
        if (data && d3Container.current) {
            if (!chart) {
                chart = new OrgChart();
            }
            chart
                .container(d3Container.current)
                .data(data)
                .nodeWidth(d => 225)
                .nodeHeight(d => 110)
                .initialZoom(0.7)
                .siblingsMargin(d => 50)
                .childrenMargin(d => 75)
                .neightbourMargin((n1, n2) => 100)
                .childrenMargin((d) => 60)
                .compactMarginBetween((d) => 35)
                .compactMarginPair((d) => 80)
                .onNodeClick((d, i, arr) => {
                    console.log(d, 'Id of clicked node ');
                    setOpen(true)
                    // props.onNodeClick(d);
                })
                .nodeContent(function (d, i, arr, state) {
                    return (
                        ReactDOMServer.renderToString(Cards(d))
                    )
                })
                .render();
        }
    }, [data, d3Container.current]);

    const Cards = (d) => {
        console.log(d)
        return (
            <Card sx={{minHeight: 125, minWidth: 225,}}
            >

                <Grid
                    container
                    direction="row"
                    justifyContent="space-between"
                    alignItems="center"
                    spacing={0}
                >

                    <Grid item>
                        <CardHeader
                            sx={{
                                maxHeight: 50,
                                minWidth: 225
                            }}

                            avatar={
                                <Avatar
                                    sx={{width: 34, height: 34}}>
                                    <GavelIcon color="error"/>
                                </Avatar>
                            }

                            title={<Typography color="#111672" fontWeight="bold"> {d.data.name} </Typography>}

                        />
                    </Grid>

                    <Grid item xs align="center">
                        <Typography align="center"> Managers: </Typography>
                        <Typography align="center" variant="button"> {d.data._directSubordinates} </Typography>
                    </Grid>
                    <Grid item xs align="center">
                        <Typography align="center"> Oversees: </Typography>
                        <Typography align="center" variant="button"> {d.data._totalSubordinates} </Typography>
                    </Grid>
                </Grid>
            </Card>)
    }

    function addNodeWithData(nodeData) {
        console.log("index -> addNodeWithData -> nodeData")
        console.log(nodeData)
        chart.addNode(nodeData);

    }

    function addNode() {
        const node = {
            name: 'New Node',
            nodeId: '6060',
            parentNodeId: '6066'
        };

        chart.addNode(node);
    }

    function deleteNode(){
        chart.removeNode(6068)
    }

    // props.setClick(addNode);

    const [open, setOpen] = React.useState(false);

    const handleClose = () => {
        setOpen(false);
    };
    const Transition = React.forwardRef(function Transition(props, ref) {
        return <Slide direction="up" ref={ref} {...props} />;
    });

    const theme = useTheme();
    const fullScreen = useMediaQuery(theme.breakpoints.down("md"));

    return (

        <Grid container spacing={gridSpacing}>
            <Grid item xs={12}>
                <MainCard>
                    <button id={"addButton"} onClick={addNode}>add node as root's child</button>
                    <button onClick={deleteNode}>remove node</button>
                    <div ref={d3Container}/>

                    <Dialog
                        fullWidth={true}
                        maxWidth={"sm"}
                        fullScreen={fullScreen}
                        open={open}
                        TransitionComponent={Transition}
                        keepMounted
                        onClose={handleClose}
                        aria-labelledby="alert-dialog-slide-title"
                        aria-describedby="alert-dialog-slide-description"

                    >
                        <Formik
                            enableReinitialize={true}
                            initialValues={{
                                id: "7946",
                                name: "Dav Nan",
                                parentNodeId: "6066"
                            }}

                            onSubmit={(values, {resetForm, setSubmitting}) => {
                                console.log("add button clicked")
                                addNodeWithData(values)
                                resetForm()
                                setSubmitting(false)
                                // document.getElementById("addButton").click()
                            }}
                        >
                            {({submitForm, isSubmitting}) => (
                                <Form>
                                    <Grid>
                                        <Grid item xs="auto">
                                            <Field
                                                component={TextField}
                                                label="Name"
                                                name='name'
                                                fullWidth
                                            />
                                        </Grid>
                                        <Grid item xs="auto">

                                            {isSubmitting && <LinearProgress/>}
                                        </Grid>
                                        <Grid item xs="auto">

                                        </Grid>
                                    </Grid>
                                    <Grid>
                                        <Button variant="contained" disabled={isSubmitting}
                                                onClick={submitForm}>
                                            Add new child
                                        </Button>

                                    </Grid>
                                </Form>
                            )}

                        </Formik>
                    </Dialog>

                </MainCard>
            </Grid>
        </Grid>

    )
});
export default Obs

I am not exactly sure what is happening, if you can provide an online reproducible example by forking the react integration sample, I may be able to check it

Here is a reproduced code on the sandbox
[https://codesandbox.io/s/add-node-function-of-d3-org-chart-sb5opb?file=/src/orgChart.js]

I am not exactly sure what is happening, if you can provide an online reproducible example by forking the react integration sample, I may be able to check it

Any update regarding this issues ? 😅

Not expecting updates in the coming days (maybe weeks or even months?)
I'll look into it once I will have more free time from my current projects

Hmm:/ I checked this example and it was working for me


Upload categories specified in the manifest

This adds support for uploading categories to crates.io, if they are specified in the manifest.
This goes with rust-lang/crates.io#473. It should be fine to merge this PR either before or after that one; crates.io master doesn't care if the categories are in the metadata or not. With that PR, I was able to use this patch with cargo to add categories to a crate!

r? @brson
(rust_highfive has picked a reviewer for you, use r? to override)

👍
From my comments on the crates.io PR, we may want to integrate a feedback mechanism here for unknown categories, but we can figure that out on the crates.io side first.

Wheeee there's some failing tests that are totally my fault

I... did github change their UI again? I don't understand why the icon on the left is merge-conflict gray. I tried rebasing on master and  it says it's up to date...

It's "you can't merge this PR" gray. In this case, I assume, because you don't have permission to push to master. Bors can, though, so no worries.

:( that's confusing.

This looks good to me, just gonna wait to r+ until we merge the crates.io support and deploy it.

@bors: r+
Oh I meant to r+ this long ago!

:pushpin: Commit f697b8c has been approved by alexcrichton

:hourglass: Testing commit f697b8c with merge 7ba2012...

:sunny: Test successful - status-appveyor, status-travis
Approved by: alexcrichton
Pushing 7ba201265f073e4938c4a4166008c63622f10a63 to master...

Please change the beta channel cargo so that, at a minimum, it doesn't emit a warning when categories is in Cargo.toml.


Remove show statement

@JuliaRegistrator register()

Registration pull request created: JuliaRegistries/General/112972
Tip: Release Notes
Did you know you can add release notes too? Just add markdown formatted text underneath the comment after the text
"Release notes:" and it will be added to the registry PR, and if TagBot is installed it will also be added to the
release that TagBot creates. i.e.
@JuliaRegistrator register

Release notes:

## Breaking changes

- blah

To add them here just re-invoke and the PR will be updated.
Tagging
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if the Julia TagBot GitHub Action is installed, or can be done manually through the github interface, or via:
git tag -a v1.0.1 -m "<description of version>" 2be412326070050e82b747d3be769c0546842b81
git push origin v1.0.1

Release note:
Removed a @show statement that snuck through into the release

@JuliaRegistrator register
Release notes:

Removed a @show statement that snuck through into the release!

Registration pull request updated: JuliaRegistries/General/112972
Tagging
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if the Julia TagBot GitHub Action is installed, or can be done manually through the github interface, or via:
git tag -a v1.0.1 -m "<description of version>" 2be412326070050e82b747d3be769c0546842b81
git push origin v1.0.1


Firebase Performance logs too much when data collection disabled

Xcode version: 11.2
Firebase SDK version: 6.11.0
Firebase Component: Preformance
Component version: 3.1.5

Steps to reproduce:
I have isInstrumentationEnabled & isDataCollectionEnabled set to false, then run the app from Xcode in Debug mode, Performance begins infinite logging complaining things like below
[Firebase/Performance][I-PRF100013] Dropping event since data collection is disabled.

One or two logs like that are usually ok. But currently, I got this line log every 1 seconds. Maybe it's because I have a timer that sends a network request every 1 seconds and Performance manages to monitor the request, but then find it should not collect the data, so it complains. Anyway, It makes log console in Xcode completely useless.
Relevant Code:
in func application(_ application: UIApplication, didFinishLaunchingWithOptions launchOptions: [UIApplication.LaunchOptionsKey: Any]?) -> Bool
Performance.sharedInstance().isInstrumentationEnabled = false
Performance.sharedInstance().isDataCollectionEnabled = false

@xiao99xiao Currently these are log messages that are displayed only on one of the higher levels of verbosity. Can you double check if you have a high level of verbosity. Check for environment variable "-FIRInfoEnabled" in your app's scheme definition?
When set to -FIRDebugEnabled, these load of log messages should not be seen.

@visumickey You are right, I set -FIRDebugEnabled and then the log was gone. Even after I removed that flag, the log still stays neat now.
Thanks you so much!


Update instructions for unified experience and power apps admin portal

I couldn't apply the Security Role customisation directly to the managed app, so I had to create a secondary app to customize them.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: cb2ad729-6bcd-dc00-f472-ed39b10da509
Version Independent ID: 61a120d7-2a43-76d6-01ee-a2f19ce07c6c
Content: Lead management for Dynamics 365 Customer Engagement - Microsoft commercial marketplace
Content Source: articles/marketplace/partner-center-portal/commercial-marketplace-lead-management-instructions-dynamics.md
Service: marketplace
Sub-service: partnercenter-marketplace-publisher
GitHub Login: @keferna
Microsoft Alias: keferna

@ShadowXVII Thanks for the comment. We are looking into this and will get back to you shortly.

@ShadowXVII Any breakthrough here?

@emuench , create a second solution in the portal and then make the require changes to the existing role. Import > Security Role. Then publish.

@SadiqhAhmed-MSFT Any breakthrough here? I pinged Shadow accidentally yesterday; his comment above.

Pinged another resource

@vikrambmsft For assistance.

Working with two new resources.

@ShadowXVII Thanks for bringing this to our attention! We're working to verify and add your feedback to the article. We appreciate your contribution. Let us know if you have any questions.
#please-close

It is likely this is being attempted without correct privileges. Documentation mentions in prerequisites that admin rights are required: "The following user permissions are necessary to complete the steps in this article: Administrator rights on your Dynamics 365 Customer Engagement instance to be able to install a solution."

No, it's not that. I do have admin.
It's about updating the instructions for unified experience and power apps admin portal


Placeholder Image

How to add a placeholder image to youTubePlayerView ?
Placeholder image to show when youTubePlayerView is loading & buffering.

Incase you don't want to go through the hassle of integrating the YoutubeData API, you can load out the thumbnail using this workaround -> Using Glide & Picasso


Fix Adjoint Diagnostics snap shot output

What changes does this PR introduce?
Fix problem described in issue #774 for snap-shot output (when frequency is set to negative number)
What is the current behaviour?
see issue #774
What is the new behaviour
Consistent timing between a) when the ADJ-diag is filled (set from diagnostics_switch_onoff.F call) and b) when the diagnostics are written to file (from diagnostics_write_adj.F call).
Does this PR introduce a breaking change?
no
Other information:
Added few diagnostics output to Adjoint test experiment hs94.1x64x5 to easily show how this works
Suggested addition to tag-index
o pkg/autodiff:

adjust call to S/R DIAGNOSTICS_SWITCH_ONOFF in autodiff_inadmode_set_ad.F
so that snap-shot Adjoint diagnostics are filled within the same time-step
as they are written to output file.

@mjlosch Thanks for the review and for trying this. I updated this PR description with a suggested addition to tag-index.
I thinks this (small) PR is ready to be merged and will do that soon (later today ?), unless someone wants to add something.


using msal-browser runtime error

Core Library
MSAL.js (@azure/msal-browser)
Core Library Version
2
Wrapper Library
MSAL Angular (@azure/msal-angular)
Wrapper Library Version
2
Public or Confidential Client?
Public
Description

Error Message
No response
Msal Logs
No response
MSAL Configuration
{}

Relevant Code Snippets
import {PublicClientApplication} from "@azure/msal-browser";

export default function App({ Component, pageProps }: AppProps) {
  const msalInstance = new PublicClientApplication(msalConfig);

// importing PublicClientApplication from msal-browser is all it takes to reproduce

Reproduction Steps
1
Expected Behavior
1
Identity Provider
Azure AD / MSA
Browsers Affected (Select all that apply)
Chrome
Regression
No response
Source
External (Customer)

@NickLWMP could you please share the exact versions of both msal-browser and msal-angular? Thanks.

@hectormmg
"@azure/msal-browser": "^3.0.0",
"@azure/msal-react": "^2.0.0",`
"react": "18.2.0",

Same error here.
The problem is coming from msal-common package.
package.json is missing the property "type"="module"

  "sideEffects": false,
  "module": "./dist/index.js",
  "types": "./dist/index.d.ts",
  **"type": "module",**
  "exports": {
    ".": {
      "types": "./dist/index.d.ts",
      "module": "./dist/index.js",
      "import": "./dist/index.js",
      "require": "./dist/index.cjs.js",
      "default": "./dist/index.js"
    },
    "./package.json": "./package.json"
  },

A temporary fix is to go in<EMAIL_ADDRESS>and add manually the property "type": "module"

Same error here.
The problem is coming from msal-common package. package.json is missing the property "type"="module"
  "sideEffects": false,
  "module": "./dist/index.js",
  "types": "./dist/index.d.ts",
  **"type": "module",**
  "exports": {
    ".": {
      "types": "./dist/index.d.ts",
      "module": "./dist/index.js",
      "import": "./dist/index.js",
      "require": "./dist/index.cjs.js",
      "default": "./dist/index.js"
    },
    "./package.json": "./package.json"
  },

A temporary fix is to go in<EMAIL_ADDRESS>and add manually the property "type": "module"

This workaround worked for me. Thank you!


BUG: macOS and Windows minor misc issues

Overall the installation process was quite nice on macOS! A couple of minor issues:

At the end on macOS, the installer popped up a notification saying I could access MNE-Python via applications. Ideally clicking this would open the folder /Applications/MNE-Python, but instead it popped up the script editor (?):

When I open the command prompt app, it prints out more stuff than it probably should (first few lines), almost like a Makefile without at @ preceding each command:

All of the app names are too long:

If at all possible, it might be better to trim the version (it could be in the App name instead maybe?), and/or since they're already in a folder called MNE-Python, the "MNE " from the beginning of each app could be trimmed. I think it's implied each one will be our app's variant by it being in the MNE-Python app folder. (Naive users won't care, and expert users will expect / understand that things are sandboxed/organized like this.)
The same goes for windows, where the version shows up in the folder, but then every app has both "MNE " and the version at the end, which is redundant:

Updated to mention the same minor naming issue on Windows, where other than that, things worked as expected!

... except one more thing that I just noticed -- the uninstaller on Windows does remove all the files from ~/mne-python/ (good) but it does not remove the start menu entries (bad):

Thank you for testing the installer on 3 platforms!!!

At the end on macOS, the installer popped up a notification saying I could access MNE-Python via applications. Ideally clicking this would open the folder /Applications/MNE-Python, but instead it popped up the script editor (?):

The notifications are displayed via AppleScript. Unfortunately, this also means when you click on them, the script editor will be brought up.
I have to say that on my systems, I usually don't even see those notifications as I'm in Do Not Disturb mode.
My proposal would be to just remove them.
WDYT?

2. When I open the command prompt app, it prints out more stuff than it probably should (first few lines), almost like a Makefile without at @ preceding each command:

Oh, yes, I never looked into that. If you believe this is an issue, it should be fixed here:

for Linux
for macOS (even though I wouldn't know how to fix it … because again, we're using AppleScript to pipe those commands into the terminal 🤔)

3. All of the app names are too long:

If you open the folder "properly" in Finder, it should display correctly though, right?
These popup file browser windows from the Dock are notorious for not providing enough space for the filenames! I have this problem will all kinds of documents. This is why I usually never use this functionality.

3. If at all possible, it might be better to trim the version (it could be in the App name instead maybe?), and/or since they're already in a folder called MNE-Python, the "MNE " from the beginning of each app could be trimmed. I think it's implied each one will be our app's variant by it being in the MNE-Python app folder. (Naive users won't care, and expert users will expect / understand that things are sandboxed/organized like this.)

The reason I did do it this way is really to avoid confusion in case users are relying on Spotlight to find & open their apps (after switching to macOS, this is how I start my apps 100% of the time!)
If the app was just called, say, Spyder, it wouldn't be immediately clear that it's the one we shipped (the user would have to check in which folder it's located).
Applications are ranked higher in the results than folders, so those apps get a higher "visual priority":

The version number is appended to allow parallel installs of different versions.
I could imagine dropping the version numbers from the file names, and appending it to the MNE-Python folder in /Applications instead…
But this approach wouldn't transfer to Linux, where on the Ubuntu desktop (Unity, I suppose?) all shortcuts end up in a single launcher without hierarchies, so the only way to communicate different version numbers if different installations is by adding those numbers to the shortcuts. And also prepending MNE IMHO becomes a necessity there, otherwise our shortcuts are just "Spyder", "Prompt", "System Info", which doesn't make sense without additional context.

3. The same goes for windows, where the version shows up in the folder, but then every app has both "MNE " and the version at the end, which is redundant:

Yes, this one we should probably try to fix.

the uninstaller on Windows does remove all the files from ~/mne-python/ (good) but it does not remove the start menu entries (bad):

Interesting, the last time I tested with @GuillaumeFavelier, the start menu entries were removed – only the desktop icons remained. Speaking of which – do you think we need desktop icons, or should we disable them?

My proposal would be to just remove them.

Yes, better to remove it than have it pop up something that doesn't make sense when it's clicked.

Oh, yes, I never looked into that. If you believe this is an issue, it should be fixed here:

for Linux

On Linux and Windows it was okay actually. Just a macOS problem.

If you open the folder "properly" in Finder, it should display correctly though, right? ... (after switching to macOS,  is how I start my apps 100% of the time!)

Yes, there and in Spotlight it's okay. But I think it's important to make sure finder, spotlight, and the dock are as usable as possible.
Right now it's MNE <app> (<version>), but perhaps we should prioritize the actual app name. Seeing MNE Spy... dev) as the name in the dock is less informative than Spyder (M...dev) or however it would look if we changed it to <app> (MNE <version>). This will still look fine in Spotlight, even if MNE shows up later in the name, so I think it's a net usability gain:

Spotlight
Dock

And having the version just in the folder looks even better to me:

Spotlight
Dock

The version number is appended to allow parallel installs of different versions.

Sure, but let's keep in mind that 1) this will be rarer than the standard single-install case, 2) I doubt people will keep multiple versions for very long, and 3) even when they do, name resolution is only a problem in Spotlight. It would be a shame to make the simultaneous-install-and-spotlight use case reduce the usability of the much more common single-MNE-app-install-or-not-spotlight cases. That said, having "Spyder (MNE)" does at least make "mne" in spotlight resolve properly. And if we're going to keep MNE in there, I can live with keeping the version number there, too, if you think it's really important. I'll let you decide...

I could imagine dropping the version numbers from the file names, and appending it to the MNE-Python folder in /Applications instead… But this approach wouldn't transfer to Linux

FWIW, it's already inconsistent between macOS and Windows (see screenshots above) -- on macOS the dock/Application folder is just MNE-Python, on Windows the Start Menu folder is "MNE-Python (1.0dev0)".
I think we should do whatever makes the presentation cleanest on each OS. For Windows and macOS, my preference would be having the start menu / Applications folder be "MNE-Python (1.0dev0)", and the app names have the highest priority at least as above. For Linux -- where there is no "folder" -- I'm fine with " (MNE 1.0dev0)" or "MNE  (1.0dev)" (though I think I prefer the former).

Speaking of which – do you think we need desktop icons, or should we disable them?

Sometimes it's an option in installers. I'd remove them until (if ever) we find a way to make it optional.

When I open the command prompt app, it prints out more stuff than it probably should (first few lines), almost like a Makefile without at @ preceding each command:
[...]

for macOS (even though I wouldn't know how to fix it … because again, we're using AppleScript to pipe those commands into the terminal 🤔)

I just spent 30 min to find a solution and I don't think there is an easy one.
I am planning to add iTerm2 support in a future version of the installer (it has a Python scripting interface) – I believe this is time better spent.
Thoughts?

Ultimately I don't think the printing is so bad. Could it be simplified by adding a little script with those two commands, then you just call the script? Then what gets printed would at least be much shorter and fit on one line

I think we can close this and open targeted issues in the future following changes in main and in #195


Is it possible to filter by public transit?

Read these instructions carefully
Please don't use the issue tracker to ask "how-to" or usage questions. If asked here, they will be closed automatically. Instead, ask "how-to" and usage questions on StackOverflow.
Feature requests are for proposing an enhancement to OSMnx before opening a pull request to add your proposed functionality to the codebase. Your issue will be closed automatically if you don't provide the requested information that we need below. Before you proceed, review the contributing guidelines in this repo's CONTRIBUTING.md file.
Is your feature proposal related to a problem?
I was wondering if it is possible to filter just public transit (ie- Chicago el train, NYC subway)
Describe the solution you'd like to propose
Add filter to osmnx.graphy_from_place(network_type="")
Describe alternatives you've considered
None
Additional context
`import osmnx as ox
place = ["Chicago.Illinois.USA"]
G = ox.graph_from_place(place, retain_all=True, simplify = True, network_type='all')`

I was wondering if it is possible to filter just public transit

Yes this is possible. Per the contributing guidelines and new issue template, best to ask "how-to" and usage questions on StackOverflow.


PyArmNN's ASR example error

I followed the instructions [1] to test ASR example on RPi4 (Ubuntu 20.04). The version of ArmNN is 21.08.
I cannot run run_audio_file.py:
$ python3 run_audio_file.py --audio_file_path tests/testdata/quick_brown_fox_16000khz.wav --model_file_path tflite_int8/wav2letter_int8.tflite --labels_file_path tests/testdata/wav2letter_labels.txt --preferred_backends CpuAcc CpuRef
Your ArmNN library instance does not support Onnx models parser functionality.  Skipped IOnnxParser import.
Traceback (most recent call last):
File "run_audio_file.py", line 94, in
main(args)
File "run_audio_file.py", line 59, in main
network = ArmnnNetworkExecutor(model.path, args.preferred_backends)
File "/home/ubuntu/armnn-21.08/python/pyarmnn/examples/speech_recognition/../common/network_executor.py", line 94, in init
self.network_id, self.runtime, self.input_binding_info, self.output_binding_info = create_network(model_file,
File "/home/ubuntu/armnn-21.08/python/pyarmnn/examples/speech_recognition/../common/network_executor.py", line 36, in create_network
network = parser.CreateNetworkFromBinaryFile(model_file)
File "/usr/lib/python3/dist-packages/pyarmnn/_generated/pyarmnn_tfliteparser.py", line 711, in CreateNetworkFromBinaryFile
return _pyarmnn_tfliteparser.ITfLiteParser_CreateNetworkFromBinaryFile(self, graphFile)
RuntimeError: Buffer doesn't conform to the expected Tensorflow Lite flatbuffers format. size:133  at function LoadModelFromBinary [/build/armnn-9SkJA0/armnn-21.08/src/armnnTfLiteParser/TfLiteParser.cpp:3609]
Any idea?  Thanks.
[1] https://developer.arm.com/documentation/102603/2108/Introduction?lang=en

Hi @sdavejones
it looks like your version of armnn was not compiled with OnnxParser support. Can you confirm that you followed the guide and installed it using:
sudo apt install software-properties-common
sudo add-apt-repository ppa:armnn/ppa
sudo apt update
sudo apt-get install -y python3-pyarmnn armnn-latest-cpu armnn-latest-ref

Does your version of armnn have a file called libarmnnOnnxParser.so?
Best regards,
Mike

Hi @sdavejones
it looks like your version of armnn was not compiled with OnnxParser support. Can you confirm that you followed the guide and installed it using:
sudo apt install software-properties-common
sudo add-apt-repository ppa:armnn/ppa
sudo apt update
sudo apt-get install -y python3-pyarmnn armnn-latest-cpu armnn-latest-ref

Does your version of armnn have a file called libarmnnOnnxParser.so?
Best regards, Mike

Hi @MikeJKelly
I've already install the latest version.
$ sudo apt-get install -y python3-pyarmnn armnn-latest-cpu armnn-latest-ref
Reading package lists... Done
Building dependency tree
Reading state information... Done
armnn-latest-cpu is already the newest version (21.08-3).
armnn-latest-ref is already the newest version (21.08-3).
python3-pyarmnn is already the newest version (21.08-3).
0 upgraded, 0 newly installed, 0 to remove and 0 not upgraded.
No, I don't have the libarmnnOnnxParser.so library.
$ find / -name "libarmnn*.so*" -print
./usr/lib/aarch64-linux-gnu/libarmnnDeserializer.so.26.0
./usr/lib/aarch64-linux-gnu/libarmnnSerializer.so.26.0
./usr/lib/aarch64-linux-gnu/libarmnn.so.26
./usr/lib/aarch64-linux-gnu/libarmnnDeserializer.so.26
./usr/lib/aarch64-linux-gnu/libarmnnSerializer.so.26
./usr/lib/aarch64-linux-gnu/libarmnnAclCommon.so.26
./usr/lib/aarch64-linux-gnu/libarmnn.so.26.0
./usr/lib/aarch64-linux-gnu/libarmnnAclCommon.so.26.0
./usr/lib/aarch64-linux-gnu/libarmnnTfLiteParser.so.24.2
./usr/lib/aarch64-linux-gnu/libarmnnTfLiteParser.so.24
./usr/lib/aarch64-linux-gnu/libarmnnDeserializer.so
./usr/lib/aarch64-linux-gnu/libarmnnSerializer.so
./usr/lib/aarch64-linux-gnu/libarmnn.so
Could you check if the upstream package has missed out that file?  Thanks.

Hi @sdavejones
Sorry for the delay, I think the best thing to do is to try:
sudo apt install software-properties-common
sudo add-apt-repository ppa:armnn/ppa
sudo apt update
sudo apt-get install -y python3-pyarmnn armnn-latest-all

That should download all available armnn libraries for the platform. Please let me know if this works and I'll see if I can get the guide updated.
Best regards,
Mike

Hi @sdavejones
can you check the size of your tflite_int8/wav2letter_int8.tflite file, it should be about 24MB.
RuntimeError: Buffer doesn't conform to the expected Tensorflow Lite flatbuffers format. size:133 at function LoadModelFromBinary [/build/armnn-9SkJA0/armnn-21.08/src/armnnTfLiteParser/TfLiteParser.cpp:3609]

If it's 133 bytes like this error says then the wav2letter_int8.tflite file only contains the Git LFS (Large File Storage) header and does not contain the actual model. If you open it in an editor then the LFS header will look something like this:
version https://git-lfs.github.com/spec/v1
oid sha256:81ed9280856ca6b2b3db3f822ec7375fd71f89b9c3c7903409d5e7857caf11a7
size 23815520

If your ML-Zoo only contains the headers then you might have missed the section on installing git-lfs at:
https://developer.arm.com/documentation/102603/2108/Device-specific-installation/Install-on-Raspberry-Pi
You have to run the following command before you clone the ML-Zoo.
sudo apt-get install git git-lfs
git lfs install --skip-repo

If you can't run those commands for whatever reason you can download the tflite model file directly from:
https://github.com/ARM-software/ML-zoo/blob/master/models/speech_recognition/wav2letter/tflite_int8/wav2letter_int8.tflite
Best regards,
Mike

I believe this issue has been resolved. If you have require further help then please reopen this ticket or open another one.


Convert requirements for milestone 1.1 into issues.

Requirements for milestone 1.1 as issues:

Integrate CSS framework + adapt to NR UI Kit v1
Create Sidebar
Integrate weather data

Requirements converted to issues.