Firebase Admin SDK Python throwing error suddenly

Trying to Fetch and modify data from the real-time database. No code change was done, working fine till yesterday.
Tried creating a new key, disabling service, updating packages. Nothing seems to work.
The code is used in Python Firebase Admin SDK. It is used to get and modify data from the real-time database.
I am using firebase_admin to access Firebase Realtime Database.
#Add Topics
topics=root.child('Topics/')
Category='Kabaddi'
with open('/Users/user/Desktop/balancerproject/single_sample.json') as response:
    data = json.load(response)
    #topics.update(data)
    topics.child(Category+'/').push(data)
    send_fcm(data)
    #instaQuotes(data.get('Title'))

It is throwing error in topics.child(Category+'/').push(data)

TransportError: HTTPSConnectionPool(host='oauth2.googleapis.com', port=443): Max retries exceeded with url: /token (Caused by NewConnectionError('<urllib3.connection.HTTPSConnection object at 0x10bb4b700>: Failed to establish a new connection: [Errno 60] Operation timed out'))

I also started facing the same issue.
               cred = credentials.Certificate(cert=Constants.Variables.FIREBASE_CRED_FILE_PATH)         firebase_app = firebase_admin.initialize_app(             credential=cred,             name=Constants.Variables.PROJECT_NAME         )
After initialisation I am creating a user,
            user = auth.create_user<EMAIL_ADDRESS>                email_verified=False,                 phone_number='+15555550100',                 password='secretPassword',                 display_name='John Doe',                 photo_url='http://www.example.com/12345678/photo.png',                 disabled=False,                 app=firebase_app             )
Then after sometime I get
HTTPSConnectionPool(host='oauth2.googleapis.com', port=443): Max retries exceeded with url: /token (Caused by NewConnectionError('<urllib3.connection.HTTPSConnection object at 0x1059edee0>: Failed to establish a new connection: [Errno 60] Operation timed out'))
Not sure what is the issue.

I also started facing the same issue.
 cred = credentials.Certificate(cert=Constants.Variables.FIREBASE_CRED_FILE_PATH) firebase_app = firebase_admin.initialize_app( credential=cred, name=Constants.Variables.PROJECT_NAME )
After initialisation I am creating a user,
user = auth.create_user<EMAIL_ADDRESS>email_verified=False, phone_number='+15555550100', password='secretPassword', display_name='John Doe', photo_url='http://www.example.com/12345678/photo.png', disabled=False, app=firebase_app )
Then after sometime I get
HTTPSConnectionPool(host='oauth2.googleapis.com', port=443): Max retries exceeded with url: /token (Caused by NewConnectionError('<urllib3.connection.HTTPSConnection object at 0x1059edee0>: Failed to establish a new connection: [Errno 60] Operation timed out'))
Not sure what is the cause.

So I figured it out, there was an outage in airtel in India. If you are using airtel try the same with JIO connection, should work.

I also started facing the same issue.
 cred = credentials.Certificate(cert=Constants.Variables.FIREBASE_CRED_FILE_PATH) firebase_app = firebase_admin.initialize_app( credential=cred, name=Constants.Variables.PROJECT_NAME )
After initialisation I am creating a user,
user = auth.create_user<EMAIL_ADDRESS>email_verified=False, phone_number='+15555550100', password='secretPassword', display_name='John Doe', photo_url='http://www.example.com/12345678/photo.png', disabled=False, app=firebase_app )
Then after sometime I get
HTTPSConnectionPool(host='oauth2.googleapis.com', port=443): Max retries exceeded with url: /token (Caused by NewConnectionError('<urllib3.connection.HTTPSConnection object at 0x1059edee0>: Failed to establish a new connection: [Errno 60] Operation timed out'))
Not sure what is the cause.

So I figured it out, there was an outage in airtel in India. If you are using airtel try the same with JIO connection, should work.

I was not on Airtel. I checked my network connection is good. I also made a http request before the firebase call, it worked correctly.


[BUG]: Error: Failed to fetch when using or pairing on my node after a while

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the bug
I had "Error: Failed to fetch" with Alby.
So I tried to pair Alby again with my full node using the BTCPay Server connector.
It failed with CORS error, then later it worked.
Then the day after, it stopped working with the same error "Error: Failed to fetch" appearing when I try to receive/send and on the home screen.
Then I tried to pair again, stumbled on same issue.
I tried to pair with both LND connector and BTCPay connector, and got CORS error.
I am wondering why it has been working. There has been no CORS on our BTCPay pairing nor LND routes and it definitely worked at one point and stopped working after.
Screenshots [optional]
Failed to fetch error:

Failed to pair with btcpay:

Failed to pair with lnd:

Console errors:

Steps To Reproduce
I didn't find how to reproduce reliably.
It works, then it doesn't.
Expected behavior
Do not fail on pairing and "Error failed to fetch"
Alby information
2.1.1
Brave
Device information
Windows Brave 2.1.1
Additional context
I opened an issue on https://github.com/btcpayserver/btcpayserver/pull/5128 for allowing CORS on our BTCPay pairing page.
That said, I don't think this is the issue, LND routes never had CORS and it was working yestersday. (I was on 2.11 yestersday as well)
Are you working on this?
No

I reviewed your manifest.json, it seems to have correct permission to avoid CORS. This is weird.

Nothing CORS related in the permissions of the app

I don't know why this error comes up sometimes and not other.
Anyway, I just activated CORS on LND through
https://github.com/btcpayserver/btcpayserver-docker/commit/6dc2c783b193e96c8ce6345977c98fd6a080b5ae
For the pairing, I will merge https://github.com/btcpayserver/btcpayserver/pull/5128
That way it will always work.


Input operation fusion in mlir convolution/dot

@5 = gpu::code_object[code_object=9032,symbol_name=convert_kernel,global=614400,local=1024,](images,@4) -> half_type, {1, 3, 640, 640}, {1228800, 409600, 640, 1}, target_id=0: 0.0543234ms, 2%
@6 = hip::hip_copy_literal[id=main:@literal:70] -> half_type, {32, 3, 6, 6}, {108, 36, 6, 1}, target_id=0: 0.00065116ms, 1%
@7 = load[offset=0,end=6553600](@1) -> half_type, {1, 32, 320, 320}, {3276800, 102400, 320, 1}, target_id=0: 0.00069002ms, 1%
@8 = broadcast[axis=1,out_lens={1, 32, 320, 320}](@2) -> half_type, {1, 32, 320, 320}, {0, 1, 0, 0}, target_id=0: 0.00086244ms, 1%
@9 = gpu::code_object[code_object=9688,symbol_name=mlir_convolution_add_sigmoid_mul,global=204800,local=256,](@8,@5,@6,@7) -> half_type, {1, 32, 320, 320}, {3276800, 102400, 320, 1}, target_id=0: 0.0267171ms, 1%

This snippet is from YOLOv5 model.
"convert"  operation is not fused with mlir_convolution_add_sigmoid_add kernel.
It can be fused to avoid kernel launch overhead.
Currently it seems MLIR is only looking for "reshapes" kind of operations.
https://github.com/ROCm/AMDMIGraphX/blob/6e505cd43c5286f4fafff9b8b863fe94e8735a25/src/targets/gpu/fuse_mlir.cpp#L166
cc : @krzysz00 @pfultz2  @manupak  @jerryyin

This currently won't work. We are actively planning to make this sort of fusion possible.


Update index.md

Give it more SEO juice for those searching.

OpenPublishing.Build — Validation status: passed

Hopefully we can get failure on warnings soon.


Separated PickaDate from DatePicker

This addresses #99. By removing PickaDate from the DatePicker.js source the license is retained.

Looks good.  I tried changing to a bunch of dates and everything looks like it's working. :)


How to decide Extension ID? For example: Timer Extension ID & IPI Extension ID

I was using git blame to search for "riscv-sbi.adoc" history message, and I find @atishp04 's commit(v0.1 SBI replacement extensions) which added  the Timer Extension ID and  IPI Extension ID.
I also searched in Mailing List discussion, but I can't find any Timer Extension ID and IPI Extension ID discussion, only system reboot extension discussion.
I want to know more about Extension ID's naming rules, and I was wondering if anyone can tell me where I can find more relevant information?

It's the ASCII representation of the Extension:

0x54494D45 => 0x54, 0x49, 0x4D, 0x45 (TIME)
0x52464E43 => 0x52, 0x46, 0x4E, 0x43 (RFNC)
...

Closing this issues as it is resolved.


Add Registries Config for Builds

Enable support for configuring container registries for builds.
The referenced ConfigMap should contain a valid registries.conf file.
See docker day-two tasks
JIRA-ID: DEVEXP-226

/assign @jwforres
/cc @bparees

nits, otherwise lgtm

Another item that we need to address is the "block all registries but X, Y, Z" that was available in 3.x: https://docs.okd.io/3.11/day_two_guide/docker_tasks.html#docker-external-registries-whitelist-and-blacklist
I don't think we have this capability with the containers/image library that drives buildah. @rhatdan may be able to weigh in. The old YAML syntax seemed to transform the "search" registries into "allowed" registries in the case of "block all".
IMO I think setting up an HTTP proxy that whitelists the allowed registries would be a better approach if someone wants to only allow pulls from specific repositories.

/hold
Scoping this PR back down to the registry config. Waiting on https://github.com/containers/image/issues/548 to see best course of action to enable white-list behavior.


Support of key exchange diffie-hellman-group1-sha1

Since VSCode version 1.82, ssh key exchange with a server via the algorithm "diffie-hellman-group1-sha1" does not work anymore. Prior to version 1.5.58 of this extension, synchronization used to fail with the error message "client error: Error: Handshake failed: Unknown DH group".
Since the update to version 1.5.58, the error message "client error: Error: Handshake failed: no matching key exchange algorithm" appears instead. This change probably stems from commit 7f2b52b which changed something in the treatment of key exchange methods.
Steps to reproduce:

On the VMS server, setup "diffie-hellman-group1-sha1" as the ssh key exchange method (if this is not the default anyway)
setup ssh-helper-settings and synchronizer-settings for synchronization with some folder on the server
From VSCode, try to synchronize with the "Synchronize project files" command
Enter password (and username, if necessary)

Synchronization will then fail with the error message "client error: Error: Handshake failed: no matching key exchange algorithm".
Versions:
Windows 10 Pro 10.0.19045 Build 19045
VSCode: 1.82.2
VMS IDE: 1.5.58
This issue still appears with VMS IDE as the only installed extension.

Since the version 1.5.58, no default algorithms and kex are added to the connection by default.
You should define your own settings in "algorithms" sections of the connection.
https://wiki.vmssoftware.com/SSH_Settings

Thanks for the quick reply. Unfortunately, this still does not work. When I define diffie-hellman-group1-sha1 in the "kex"-setting in the "alrgorithms" section, the only change is that the error message reads "client error: Error: Handshake failed: Unknown DH group" again, but synchronization still does not happen.

The support of 'diffie-hellman-group1-sha1' and 'diffie-hellman-group1-sha1' is removed from VS Code version 1.82.


Catch KingdomInternalException and throw Grpc errors

Issue: https://github.com/world-federation-of-advertisers/cross-media-measurement/issues/247
When a specific resource is not found, use FAILED_PRECONDITION, INVALID_ARGUMENT or NOT_FOUND?
As INVALID_ARGUMENT is more about the format of arguments, avoid to use it in our cases.
When the major entity is not found, use NOT_FOUND. For example, if getMeasurement or updateMeasurement fail to find the Measurement, we call it NOT_FOUND.
Other than the major entity, use FAILED_PRECONDITION. For example, for getMeasurement or updateMeasurement, if Account/MeasurementConsumer/Certificate not found, call it FAILED_PRECONDITION.

This change is

Move to another PR
https://github.com/world-federation-of-advertisers/cross-media-measurement/pull/559

Move to another PR
https://github.com/world-federation-of-advertisers/cross-media-measurement/pull/559


nethack-qt: use gcc

Relies on gcc being accessible

Motivation for this change
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Assured whether relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

Works for me on NixOS, but it's a noop there since stdenv is using gcc anyway.


Sors Imperiorum 320km game

Game: Sors Imperiorum at 320km. First attempt to map whole world.

So this is being closed into #240.


Eliminate Response#response_set_id presence validation

The presence validation on Response#response_set_id introduced in 05034182be24f01b9960ecc19cbf9e452bc65e43 makes it impossible to write code like this under ActiveRecord 3.1:
rs = ResponseSet.new(:survey_id => survey_id)
rs.responses.build(...)
rs.save

because the ResponseSet will not have been saved at the time when Response's validations run.  (This pattern appears in some code being written for the National Children's Study.)
However, building a ResponseSet and its responses in this fashion can be done without violating referential integrity, which can be demonstrated by eliminating the validation under discussion, adding appropriate not-null and foreign key constraints on Response#response_set_id in a database that supports them (e.g. PostgreSQL), and executing the above code.  Therefore, I think that the presence validation is an unnecessary inhibition, and ask that it be eliminated or made optional.

I have code in a personal fork (https://github.com/yipdw/surveyor/tree/relax-response-paranoia) that does this, and can convert this issue into a pull request if desired.

Sure, a pull request would be great. Could you include some tests that demonstrate the use case as documented above?

Sure, a pull request would be great. Could you include some tests that demonstrate the use case as documented above?

Sure thing; are you looking for additional tests beyond https://github.com/yipdw/surveyor/compare/master...relax-response-paranoia#L1R72 ?

That's enough to demonstrate the use case to me. I think it would be clearer if you didn't use a factory, though.

Continuing this issue here:
https://github.com/NUBIC/surveyor/pull/276


Add make clean option

Feature Request / Improvement
Sometime when you upgrade cython, and you have old cached files lying around, then this might cause issues. It would be good to have a make clean command that will remove all cached files.

@Fokko I hope this works
https://github.com/apache/iceberg-python/pull/881


Duplicate Child theme

PHP Version
8.1
Shopware Version
6.4.21
Expected behaviour
I am trying to duplicate child theme. We have main theme -> Child Theme and we want inherit from child theme to other saleschannel child theme like child theme => firstChildTheme, secondChildTheme
Actual behaviour
ChildTheme which was inherited from mainChild theme is not working, "Theme could not be assigned to saleschannel"
How to reproduce
We made main Child theme and we want to  create child themes and inherit them from main child theme. And we want touse for all sales channel these themes, is it possible to realize?

Hi,
thx for your issue we will put it into our new evaluation process and have a look at it again.

[public] Automated response: This issue is linked to the internal issue https://shopware.atlassian.net/browse/NEXT-37958.
[created from NEXT-37958, comment 489034]

Hey @Bayish, can you please provide more information on what you are trying to achieve? Do you have an example of your Theme.json file and the configuration you are trying to set?

Hey @Bayish, can you please provide more information on what you are trying to achieve? Do you have an example of your Theme.json file and the configuration you are trying to set?

Hi, it was long time ago, we fixed and use already sw <IP_ADDRESS>. I forgot to close this issue. Thanks.


[FEATURE] Add Box Stacking problem in Dynamic Programming

Detailed description
Box Stacking problem is one of the most famous problems in Dynamic Programming and acts as a model questions to many problems out there.
Context
I was in a interview and faced this question regarding box stacking. I had to struggle to find the algorithm and ended messing up the interview. So I thought if it would be there along with other box stacking problems it would be helpful to one and all.
Possible implementation
/* Dynamic Programming top-down implementation of Box Stacking problem */
#include <bits/stdc++.h>
using namespace std;
/* Representation of a box */
class Box {
public:
int length;
int width;
int height;
};
// dp array
int dp[303];
int findMaxHeight(vector& boxes, int bottom_box_index, int index)
{
// base case
if (index < 0)
	return 0;

if (dp[index] != -1)
	return dp[index];

int maximumHeight = 0;

// recurse
for (int i = index; i >= 0; i--) {

	// if there is no bottom box
	if (bottom_box_index == -1 || (boxes[i].length < boxes[bottom_box_index].length && boxes[i].width
				< boxes[bottom_box_index].width))

		maximumHeight = max(maximumHeight,findMaxHeight(boxes, i, i - 1) + boxes[i].height);
}

return dp[index] = maximumHeight;

}
/* wrapper function for recursive calls which
Returns the height of the tallest stack that can be
formed with give type of boxes */
int maxStackHeight(int height[], int width[], int length[],int types)
{
// creating a vector of type Box class
vector boxes;
// Initialize dp array with -1
memset(dp, -1, sizeof(dp));

Box box;

/* Create an array of all rotations of given boxes
For example, for a box {1, 2, 3}, we consider three
instances{{1, 2, 3}, {2, 1, 3}, {3, 1, 2}} */
for (int i = 0; i < types; i++) {

	// copy original box
	box.height = height[i];
	box.length = max(length[i], width[i]);
	box.width = min(length[i], width[i]);

	boxes.push_back(box);

	// First rotation of box
	box.height = width[i];
	box.length = max(length[i], height[i]);
	box.width = min(length[i], height[i]);

	boxes.push_back(box);

	// Second rotation of box
	box.height = length[i];
	box.length = max(width[i], height[i]);
	box.width = min(width[i], height[i]);

	boxes.push_back(box);
}

// sort by area in ascending order .. because we will be dealing with this vector in reverse
sort(boxes.begin(), boxes.end(), [](Box b1, Box b2) {
	// if area of box1 < area of box2
	return (b1.length * b1.width) < (b2.length * b2.width);
});

// Uncomment following two lines to print all rotations
//for (int i = boxes.size() - 1; i >= 0; i-- )
// printf("%d x %d x %d\n", boxes[i].length, boxes[i].width, boxes[i].height);
return findMaxHeight(boxes, -1, boxes.size() - 1);

}
int main()
{
// where length, width and height of a particular box
// are at ith index of the following arrays
int length[] = { 4, 1, 4, 10 };
int width[] = { 6, 2, 5, 12 };
int height[] = { 7, 3, 6, 32 };

int n = sizeof(length) / sizeof(length[0]);

printf("The maximum possible height of stack is %d\n", maxStackHeight(height, length, width, n));

return 0;

}
Additional information
I have attached the code to it as well, if there is any issue in the syntax or the formatting of the code. Please let me know

Great, thank you! 👍 Would you like to work on this?
If so, before submitting a PR, please ensure the following:

The algorithm/implementation isn't duplicated in this repository.
Make your code per the repository standards.
Ensure that all the automated tests pass.

There were two attempted implementation of this algorithm; #1516 and #2435 if anyone would like to implement this in the repo you could make these your references as both were closed. If you need any help you can let us know. 🙂 Good luck.

I wanted to work on this.

If you want to just do

I will start working on this right away.


No such module "Firebase" in Swift Tests (Add explicit Firebase modulemap)

[READ] Step 1: Are you in the right place?
Yes
[REQUIRED] Step 2: Describe your environment

Xcode version: Any
Firebase SDK version: 4.0.0
Library version: 4.0.0
Firebase Product: All (auth, database, storage, core, messaging, etc)

[REQUIRED] Step 3: Describe the problem
Swift unit tests fail to find Firebase module.  During the distribution process, the Firebase podspec relies on Core to set up proper header search paths using user_target_xcconfig. Firebase should have its own modulemap instead of relying on the header search paths from Core.
Steps to reproduce:

Create a Swift project with unit tests
pod init to create a Podfile
Add pod 'Firebase' to Podfile under use_frameworks!
Run pod update
Open the generated xcworkspace
Add import Firebase to the AppDelegate.swift
Add @testable import <YourProjectName> into <YourProjectName>Tests.swift
A "No such module 'Firebase'" error will appear.

Relevant Code:
Currently generated podspec
Relevant Issues:
Swift unit tests fail to find Firebase module
Swift Cocoapod can't import Firebase pod dependencies
Unable to run tests (quickstarts)
Workarounds:

Add "${PODS_ROOT}/Firebase/Core/Sources" to your Tests target only under Build Settings -> Header Search Paths

Is this not a duplicate of #16?

Any progress on this?

Btw the workaround does not work for me.
CocoaPods notes that I am using these versions.
Using Firebase (3.17.0)
Using FirebaseAnalytics (3.9.0)
Using FirebaseAuth (3.1.1)
Using FirebaseCore (3.6.0)
Using FirebaseDatabase (3.1.2)
Using FirebaseInstanceID (1.0.10)
Using FirebaseMessaging (1.2.3)
Using FirebaseRemoteConfig (1.3.4)

I am not using Firebase 4.x but I assume this is not fixed yet?

@zapjonny The issue has not yet been fixed, but if the workaround does not work for you, there is likely another issue with your configuration. Please create another issue with enough detail to reproduce the problem.

OK I just started unit testing as late as yesterday and probably failed my setup. Apparently you need to add your unit test target as a target in the podfile too (unless I got that wrong too). Anyway as soon as I did that I didn't have any problems running my unit tests.
Before that I manually installed the Firebase iOS SDK (upgrading to 4+), and I was able to work around the problem that way too.

And as far as I know, I am not using the suggested workaround from above but my unit tests still work. Maybe I did something different during setup.

FIY it is not failing just the tests, i have a swift framework using firebase and i cannot use it at the moment because firebase fails to import, still looking for a workaround

@tirrorex You're running into a different issue: There is currently no clean way for frameworks to depend on Firebase. See https://github.com/CocoaPods/CocoaPods/issues/5368

i am already following this issue :/
Good to know this is separate but...

Update:

Still investigating looking at a fix we can provide on our side
A bug was filed on the CocoaPods repo, "Search path not propagated to test target", https://github.com/CocoaPods/CocoaPods/issues/6891

I've probably a related problem. I've a xcode project with multiple targets. They all use Firebase and the same set of other pods. It' all looks ok, but I can only run one (sort of random after install) of the targets. Switching between targets, cleaning and building will fail on the 'No such module Firebase'
I've tried the work arround, but that doesn't seem to work either.

after do pod update then no such module firebase ? how to resolve this error ?

This issue is fixed with CocoaPods 1.4.0.  For earlier CocoaPods releases, please use the workaround describe here.
If neither solution is working for you, please create another issue with full information about your configuration.


deletion

Does this issue occur when all extensions are disabled?: Yes/No

VS Code Version:
OS Version:

Steps to Reproduce:

fix deletion issue

/needsMoreInfo


PDF files are garbled when using Firefox built-in PDF viewer

This problem seems to have appeared in one of the latest Firefox versions, it started a few days ago.
Attach (recommended) or Link to PDF file here: https://s1.q4cdn.com/806093406/files/doc_downloads/test.pdf
Configuration:

Web browser and its version:

Operating system and its version: Windows 11
PDF.js version: No idea, I'm using the built-in PDF viewer
Is a browser extension: No

Steps to reproduce the problem:

Download the test file linked above
Open in Firefox

What is the expected behavior?
It should look like this (opened in Adobe Acrobat):

What went wrong? (add screenshot)
It looks like this in Firefox:

Link to a viewer (if hosted on a site other than mozilla.github.io/pdf.js or as Firefox/Chrome extension):

This is, most likely, a duplicate of bug 1854090 and/or bug 1857015.

Yes, it is the same.


[Modal] allowMultiple not using zindex

This happens when using the allowMultiple setting with multiple modals defined on a page.
Upon invoking a secondary modal from a modal, it appears behind the one it was invoked from instead of infront as inspected.

As a workaround, modals would need to be called on the fly instead of predefined as they are currently.


Is there a similar collection for the Open Commerce API?

Hello, these collections are great. Unfortunately, the API we need isn't included. Any chance there's a collection available for the Open Commerce API?
Thanks,
Bob Lorincz

Hello! Great to hear that you would like to see commerce APIs in the collections. We are working to get new collections posted for the Salesforce Commerce API in the next couple of months. The Open Commerce API (OCAPI) is something we can look at in the future as well, but as of right this moment we don't have immediate plans for it.


DeploymentFailed - Documentation to back-out and retry

When completing my first hub deployment today via the template I ran into a few issues.
I couldn't find any 'reverse deployment' or deletion documentation. Is there anything anywhere to detail the procedure?
Deployment 1

Failed due to tagging policies.
Reason: Tags specified in the deployment did not reach the storage account, so the deployment failed.
Backout: I saw a note somewhere to not redeploy from a failure due to conflict/duplicate, so I deleted all resources in the resource group they deployed to.
Deployment 2

Failure hub: The resource write operation failed to complete successfully, because it reached terminal provisioning state 'Failed'. (Code: ResourceDeploymentFailure, Target: /subscriptions/subscription/resourceGroups/resourcegroup/providers/Microsoft.Resources/deployments/storage)
      "code": "RoleAssignmentUpdateNotPermitted",
      "message": "Tenant ID, application ID, principal ID, and scope are not allowed to be updated."

Reason: Could be duplication of deployed resources, but I also have a permissions issue I need to resolve RoleAssignmentUpdateNotPermitted
Oops
If it helps, the reason I missed the permissions requirements;

Click Deploy
Then went straight to deploy to azure. Didn't know the permissions were in the additional reference to the template.

3rd deployment after deleting resources worked fine.

@psilantropy Sorry for the delay. Missed this one. I'll add docs for backing out. In general, you should be able to just re-deploy. I can't think of a reason off the top of my head that would be a problem with redeploying. ARM is built on the concept of idempotent deployments that can be repeated. This is actually the expected flow based on my understanding.
Worst case, you can delete the RG and try again.
The RoleAssignmentUpdateNotPermitted error is a known one. I've struggled a lot trying to work around that. Very frustrating. Rule of thumb here is don't delete the managed identities. That's where it crops up. If you do, then delete the role assignments.
I'll make sure this error is added to the troubleshooting guide.

No need to apologise, thank you. :) I just elevated to ga to sort out any perms problems.


sys:1: RuntimeWarning: coroutine 'scrape' was never awaited 🤔

Thanks @srimur for putting this up, just what i was looking for 😃
this error though .. 🤔
sys:1: RuntimeWarning: coroutine 'scrape' was never awaited
RuntimeWarning: Enable tracemalloc to get the object allocation traceback

Here's a log: bash.base.pane-capture.2024-08-29.2128h44.log

Hey @eMPee584,
Thanks a ton for catching that warning! 🙌 I was messing around with some stuff and didn’t realize it was causing issues. Sorry about that!
I’ve got it fixed now. If you spot anything else or have more feedback, let me know!
Cheers!


Fragmenter limited to python<=3.7

As fragmenter is only compatible with openeye-toolkits < 2020 and openeye was not built against 3.8 or higher at the time there is no way to use fragmenter with python>3.7. For now, I have disabled the fragmentation tests on python>=3.8 until we have updated fragmenter.

We now use the new openff-fragmenter so this is no longer an issue.


Update the screenshots in the doc to incorporate Tables app changes

There are many images in the Trying out ODK-X Tables section that have screenshots from the old versions of ODK-X which do not display some of the recent changes made in the app, namely the addition of a sync icon & "ODK-X Tables" text beside the ODK-X Tables icon in the title bar.
For example -
A screenshot from the current doc :

A screenshot from the current app :

@EstherPhebe  @elmps2018 can I work in this!

@apigeoneer could you point me to the apk where you got that kind of an screenshot please... I have a different screenshot surprisingly

@julian55455 I have assigned you! @apigeoneer thanks for raising this issue!

@apigeoneer could you help here please.... Follow the conversation and find out  https://github.com/odk-x/docs/pull/179

Hi @julian55455! I've shared a screenshot.


Reduce nav items padding to fit long term ex. progressive delivery

Closes #3301

What changed?

Reduce left padding for nav items.
Update the Delivery icon which is used by progressive delivery to be a flagger icon.

Why was this change made?

Padding has been reduced in nav items so that long term like "Progressive Delivery" would fit

Looks good - but does this close the issue? It doesn't actually change the nav text or page title to Progressive Delivery. Totally fine to do it separately if that was the plan

@joshri Yes that's only updating the icon but the text pr will be in EE repo.

It feels a tiny bit cramped now when selected.
Maybe we could try

removing the margin-bottom (~4px now)
giving that space to the height of the links instead (32px -> 36px)

It feels a tiny bit cramped now when selected.
Maybe we could try

removing the margin-bottom (~4px now)
giving that space to the height of the links instead (32px -> 36px)

here it how it looking after giving it space without removing the margin-bottom as they will stack up if we did


Take multiple arguments for files and directories to include

That way you could send just specific files like this:
files-to-prompt app.py utils.py

Do you want to accept multiple args for path? or add a new option like --files?

cat files_to_prompt/cli.py | llm -m opus --system '
Modify this file. It should take multiple arguments in a variable called paths. Each of those argumets might be a path to a file or it might be a path to a directory - if any of the arguments do not correspoind to a file or directory it should raise a click error.
It should then do what it does already but for all files or files-recursively-contained-within-folders that are passed to it.
It should still obey the gitignore logic.'

https://gist.github.com/simonw/345f16a54f9e753852ea0af4bd7c2abc

files-to-prompt files_to_prompt tests | llm -m opus --system \
   'rewrite the tests to cover the ability to pass multiple files and folders to the tool'

https://gist.github.com/simonw/c3b021f7b4f8b1e30b79f95210cc5702
files-to-prompt files_to_prompt tests | llm -m opus --system \
  'add one last test which tests .gitignore and include_hidden against an example that mixes single files and directories of files together in one invocation'

https://gist.github.com/simonw/a578bae9fe409cb29533ae28a348cb27

I decided to make that last test a bit easier to read.
symbex test_mixed_paths_with_options | llm -m opus --system \
  'rewrite this test so the filenames are more obvious, thinks like ignored_in_gitignore.txt'

llm -c 'add a last bit to that test for ["test_dir", "single_file.txt", "--ignore-gitignore", "--include-hidden"]'

https://gist.github.com/simonw/a637f3e244ef1330e657221bfa84dfae

That test:
https://github.com/simonw/files-to-prompt/blob/ebd191f77dd1160710301633953b00884e0aa9fe/tests/test_files_to_prompt.py#L85-L141


Want to add weather app created with vanilla JS, GSSOC 22 Participant

Hey, SwapnilSparsh I've created a very beginner-friendly weather app using vanilla JS where a user can search for the city and after hitting the Enter Keycode it will show the weather of that city I've made it look even nicer with good UI. Hope you will go through my Issue and assign me this issue to work on it.
I've also attached the ScreenShot of my Project.  give it a look 🙂

Hey, SwapnilSparsh I've created a very beginner-friendly weather app using vanilla JS where a user can search for the city and after hitting the Enter Keycode it will show the weather of that city I've made it look even nicer with good UI. Hope you will go through my Issue and assign me this issue to work on it. I've also attached the ScreenShot of my Project. give it a look 🙂

Great project but weather app is already there on the website which you can check out here
Try looking out for some other projects which are not there on the website yet.
Good Luck 👍🏻

Okayy thanks for the reply✨
On Thu, Mar 3, 2022, 7:30 PM Swapnil Srivastava @.***>
wrote:

Hey, SwapnilSparsh I've created a very beginner-friendly weather app using
vanilla JS where a user can search for the city and after hitting the Enter
Keycode it will show the weather of that city I've made it look even nicer
with good UI. Hope you will go through my Issue and assign me this issue to
work on it. I've also attached the ScreenShot of my Project. give it a look
🙂
[image: weather png]
https://user-images.githubusercontent.com/95171638/156401854-74295c80-bfb3-4b55-b7bd-a93b7abf0464.png
Great project but weather app is already there on the website which you
can check out here
https://30daysofjs.netlify.app/25 - weather app/
Try looking out for some other projects which are not there on the website
yet.
Good Luck 👍🏻
—
Reply to this email directly, view it on GitHub
https://github.com/swapnilsparsh/30DaysOfJavaScript/issues/188#issuecomment-1058068974,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AWWDINTACOL4KYXUR3KUZALU6DAWPANCNFSM5PX5OKRQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID:
@.***>


code searchService always takes a whole CPU core

VSCode Version: Code 1.16.0 (787b31c0474e6165390b5a5989c9619e3e16f953, 2017-09-06T16:19:37.192Z)
OS Version: Linux x64 4.12.13-300.fc26.x86_64
Extensions:

Extension
Author (truncated)
Version

language-x86-64-assembly
13x
2.2.11

vscode-docker
Pet
0.0.18

code-gnu-global
aus
0.2.2

file-templates
brp
1.2.0

FreeMarker
dco
0.0.9

python
don
0.7.0

CppSnippets
har
0.0.12

restructuredtext
lex
37.0.0

cpptools
ms-
0.12.4

csharp
ms-
1.12.1

vetur
oct
0.9.7

java
red
0.10.0

html-preview-vscode
tht
0.1.1

vim
vsc
0.10.1

jinja
who
0.0.8

A background process /usr/share/code/code /usr/share/code/resources/app/out/bootstrap --type=searchService always takes a whole CPU core.
Steps to Reproduce:

open a node project created by vue-cli
one Code process takes a whole CPU core and creates six subprocesses running find -L . -type f
after closing the Code, these subprocesses take all my four CPU cores for about one minute before exit

Reproduces without extensions: No

This is #34434


UI test

Added Testing for that the NootChatbox exist and that you can only input 160 characters

Took a look everything seems good 🥇


Hiding Charts is slow

We noticed in a client app that having charts on a tab, makes it noticeably slow to move away from the tab.
It appears to be related to actually hiding the tab contents in impl/tabContainer.js. In particular, if we instead unmount the tab contents (via TabContainerModel.unmountOnHide) things are speedy!

It appears this is being caused by resizing the charts. When the chart is hidden, Chart.resizeChart() is called with a width and height of 0. Resizing a chart requires a lot of work (I believe a complete redraw), so this is quite slow.
This also adds some expense to displaying the charts again, i.e. switching back to the tab with charts on it. This requires resizing the charts back to their original size, which is also slow.
The branch slowChartHiding has a hacked-together fix for this. It seems to work.
@TomTirapani, you added the useOnResize() hook to Chart.js. Would you take a look at this approach and see if it is a reasonable approach?

Hi Peter,
Not resizing the chart when it becomes hidden makes sense. I think we should make the following changes:

We should use || on the first condition check here: https://github.com/xh/hoist-react/compare/slowChartHiding#diff-25ff8fda126b607253fe78670c157539R105. If either dimension is 0, we can consider the chart hidden.
For code tightness, we could remove the curly braces from each of the early returns and make both one-liners.
I think the changes to Select.js were pushed to the wrong branch and don't belong as part of this change.

Thanks,
Tom


Fix for failing docs

I re-formatted the docs following how the JWST pipeline set up it's repository. This included:

re-formatting the readthedocs.yml file to update it to RTD version 2
adding the docs requirements to the setup.py file
setting the version calls to be under try/except clauses so the docs can avoid failing on them (see __init__.py, conf.py, and write.py)
deleting .rtd-environment.yml file, as it is no longer used

It looks like the travis docs build it failing due to a missing module, but the docs are building fine on RTD, so not sure where this failure is coming from (this failure is also occurring on other PRs). I'll look into this more next week


Ability to copy/ paste infomation from R/G/B/Alpha channel of an image in the viewer.

The viewer currently already support viewing separate R/G/B/Alpha channel already, it would be such a huge improvement if it's possible to copy information from those channels to an external image editor (photoshop), and then paste the edited ones back into the image viewer ( for channel packing purposes - although photoshop or other image editors can do channel packing already, it would be great if the image viewer can also do this).
Great image viewer anyway, i really love it, thank you so much !

Thanks for the kind words. Regaring the feature request I assume you mean copying specific channels of the entire canvas in one go (rather than one pixel at a time). Ctrl-C will copy the entire image (RGBA) to the clipboard (which can then be pasted in PS or whatever). I think what you're after is when you paste into Tacent View that you can either create a new image (what happens now), or optionally paste the data into the current image but only for specific channels?

Hi, and yes - copy specific channel of the entire canvas is what i mean. And when viewing one specific channel (RGBA) in Tacent view, it would be great if it's also possible to paste back the entire canvas into the channel that is currently  being viewed/ isolated.
Really sorry for the confusion!

No worries. Yeah...something can be done about both of these. There's a little to think about though...if you're pasting rgb or rgba clipboard contents into a single channel, there is a question of the intent...r to r, g to g, etc, or perhaps intensity (average rgb) was intended. In any case, it's a good idea and shouldn't be too difficult ...when I next get some free time ;)

Hi, thank you for considering the feature request!
To better describe the feature i think i should illustrate it with the intended workflow that i use:
To save on memory when shipping (some studios do this practice, some dont), 3 or 4 textures are packed together into one image. Usually this is done on the textures of a model in its final stage, but sometimes there is a need to adjust specific texture(s) that is already packed. This usually mean i have to open the texturing app, make the adjust, re export and repack the textures again. The intended use case with Tacent view is:

open the final-already packed image in Tacent view
isolate the channel that i wanted to edit using colour channel filter function
copy that channel data
paste copied data in image editor software (PTS...)
do the edit
copy the edited data in PTS
paste the data back into that specific channel in Tacent view
save the image.

Again, thank you very much for considering!

Thanks for the diagram. Current design is something like this (which will allow your workflow):
Add options for ctrl-c ctrl-v behaviour to the channels dialog:
Copy (Ctrl-C)
Intensity will copy the R G B or A into an RGBA image on the clipboard (where R=G=B, A=1). Otherwise individual channels are copied to RGBA on the clipboard (R->R, G->G, etc). Default is RGBA all checked which will match current behaviour.
Unspecified channel colour (RGBA) picker : Used if NOT intensity mode AND at least one channel not selected. Default opaque black (a=1, rgb=0). This is the only GUI widget needed for Copy I think.
Paste (Ctrl-V)
Combo dropdown:
OptionA : Create new image (default and current behaviour).
OptionB : Paste selected channels: Intensity will compute the clipboard RGB intensity and paste to the specific selected channel (including alpha). If non-intensity will paste only selected channels: R->R, G->G, B->B, A->A. If clipboard image has no alpha treat as 1.0.
If the edited image in your diagram is greyscale, it won't matter whether you select intensity/G or individual/G when pasting as Intensity = V*3/3 = V for greyscale... but it would allow multiple channels to be edited at once if desired.
The anchor would be the top left of image in case images are of different dimension. The current image/canvas size would not be modified, so pasting a larger image would discard pixels at right and bottom and pasting a smaller image would leave right and bottom pixels unmodified.
Not quite sure when I'll get to this (work got busy), but I'll probably do this as the next task as the other main effort is supporting wide-gamut 10-bit displays with is quite a bit more work ;)

This all sound great, thank you so much! I'll patiently wait so please feel no pressure on time.

Copy is done. Paste will take a bit longer because it should be undo-able etc.

This is all working in a498e47ae96e7b2efbfd6d65d0b2f4294c1773ef
For convenience some of the related preferences are also displayed in the channel-filters:

The copy behaviour:

The paste behaviour:

You will need to turn off paste-creates-image.

In case you don't have cmake installed:
tacentview_1.0.44.zip

Hi, thank you very much for working on the request, i've tested the copy feature out and it works perfectly.
The pasting feature is not working properly for me, it crash the program immediately when i hit ctrl V, whether 'paste creates image' is toggled on or off. Im currently using windows 11 if it help.

Hmm. So I've tried pasting from Affinity Photo, GIMP 2.10, the Windows 11 Snipping Tool, and from TacentView. Does copying from the snipping tool allow you to paste into Tacent View? This might be hard to debug if I don't have the contents of the clipboard. I'm assuming you are using PhotoShop, which I don't have. Would be curious if it's only PS giving you issues.

Hi, upon further testing i think that is a specific problem with my version of PTS, im using a really old version (CC 2015) and it caused tacent view to crash when pasting. I've tried copy from snipping tool, from Krita and from web browser (firefox) and it works!
But there is still a problem, when pasting, the pasted image shift a few pixels horizontally, this does not happen when copying and pasting from a channel to a different channel directly inside tacent view, but it happen when the pasted image is external (from Krita or from web browser).
i've include a video showcasing the bug:
https://github.com/bluescan/tacentview/assets/12829684/07fcb104-f44a-421f-9002-066008c532ed

OK. I was able to repo by copying from Krita. Tacent View uses a 3rd party library called "Clip Library" to retrieve (and set) images in the clipboard. I've updated this library to latest and it seems to have fixed the issue. Give this new zip a go. You might also try pasting from your older PS again, but no guarantees ;)
tacentview_1.0.44.zip

Hi, its rare (and great) to caught up with you at around the same time!
I've just tested the new build, pasting from krita is now perfect!
But pasting from web browser is still having the same shifted pixels problem, although, copying from web might not be a popular workflow so you should judge if this is worth fixing. I'll stick to krita for now, thank you very much for the build !
Just in case, i was using firefox as web browser, and Imgur.com as image host, i also tried copying and pasting from photopea.com which is a web app photoshop clone, these 2 gave me the same shifted pixel bug.

Hi quockhanhlk! I just tried a few more tests. Although I was getting slightly different results from you, I did manage to repo the crash by using photopea in chrome (in FF it wasn't copying to clipboard at all... but it's prolly some permission thing set wrong somewhere). In any case, I'm pretty sure I'm using the library incorrectly when pasting... and also think I know where I messed up. I need to rewrite some logic to support more types of image data from the clipboard. I think it is worth fixing the crash. I'll probably see if you're available for another test once that's done. Not sure ETA yet. (Incidentally, copy.pasting from FF did seem to work for me without the offset... but I think it's not worth worrying about until I fix the known issues).

I'll try to test build out as soon as i can when they're available. Current build work great with krita for no worry on ETA. Thank you for the effort!

NP. Couldn't let it go. Here's another build. Thanks for testing and also no rush.
For me with this build I didn't see any offsets:
Paste from FF works.
Paste from Photopea in FF. Could get to work (even pasting to Affinity or Krita).
Paste from Photopea in Chrome. Worked, Would crash previously.
Paste from Krita works.
Paste from Affinity works.
If you try Photoshop again I'd be curious if the crash is gone.
tacentview_1.0.44.zip

The crash is gone, now even my old version of PTS works perfectly.
I've tried pasting from photopea using firefox, chrome, and edge, and unfortunately the offset bug is still persisted, so maybe this has something to do with my pc ?
Thank you anyway, this is great !
and this is non related but i think firefox default (or some ad blocking extension) is blocking copy/paste access from photopea, im not sure where that setting is exactly but that should be it.

Thanks for testing. Glad to have removed a crash (with your help testing). IDK why you're getting the offset either (can't repo) but possibly some subtle bug in the clipping lib I'm using? In your vid it looked like the image was rolled 3 pixels to the left. Before closing this issue I've added a workaround that 'may' help:

Try putting in 3 in the first field... or adjust appropriately... and then paste from photopea. If it is a roll of the image, these values might help compensate (vertically and horizontally --  in opposite direction if negatives entered). You could even have different values for each viewer profile to easily allow switching (with Alt-1 or Alt-2).
tacentview_1.0.44.zip

Hey, thank you for the new build, that setting helps correct the pasting behavior from webpage i had. Thank you so much for working on this !

NP. Thanks for testing!


bpftrace and bcc-tools seg fault on Centos 7.6 with updated 5.1 kernel

Hi,
I am new to bpftrace, so any help will be appreciated.
I am trying to install bpftrace on Centos 7.6. Updated the kernel to 5.1.12 (latest ml) as the default kernel is much older. Installed clang-8.0.0 and llvm-8.0.0
bpftrace commands segfault instantly.
./src/bpftrace -e 'kprobe:do_nanosleep { printf("sleep by %s\n", comm); }'
Error creating perf event map (-1)
Attaching 1 probe...
Segmentation fault (core dumped)
Attaching to gdb gives the following:
Program received signal SIGSEGV, Segmentation fault.
0x00007fffef730901 in __strlen_sse2_pminub () from /lib64/libc.so.6
Missing separate debuginfos, use: debuginfo-install elfutils-libelf-0.172-2.el7.x86_64 glibc-2.17-260.el7.x86_64 zlib-1.2.7-18.el7.x86_64
(gdb) bt
#0  0x00007fffef730901 in __strlen_sse2_pminub () from /lib64/libc.so.6
#1  0x00007ffff3fbf28a in __bpf_object__open () from /usr/local/lib64/libbcc.so.0
#2  0x00007ffff3fc3e02 in bpf_prog_load_xattr () from /usr/local/lib64/libbcc.so.0
#3  0x00007ffff3fc3d7d in bpf_prog_load () from /usr/local/lib64/libbcc.so.0
#4  0x0000000000696559 in bpftrace::AttachedProbe::load_prog() ()
#5  0x0000000000697022 in bpftrace::AttachedProbe::AttachedProbe(bpftrace::Probe&, std::tuple<unsigned char*, unsigned long>) ()
#6  0x00000000006b4904 in bpftrace::BPFtrace::attach_probe(bpftrace::Probe&, bpftrace::BpfOrc const&) ()
#7  0x00000000006b5d44 in bpftrace::BPFtrace::run(std::unique_ptr<bpftrace::BpfOrc, std::default_deletebpftrace::BpfOrc >) ()
#8  0x000000000064462b in main ()
Further, all bcc tools also segfault. I am guessing these are related.
Thanks,
Gaurav.

Hmm, that's interesting. Do you mind filing an issue with bcc? They're probably better equipped to debug this.

Recently built bpftrace on centos7 with elrepo 5.1.9 kernel. No issues here. I built llvm/clang from source using the https://github.com/llvm/llvm-project repo. This required a newer gcc and cmake as well to build bpftrace.
~$ bpftrace --version
bpftrace v0.9-232-g60e6
~$ ls -la /usr/lib
lib/     lib64/   libexec/
~$ ls -la /usr/local/lib/libLLVM-9
libLLVM-9.0.0svn.so  libLLVM-9svn.so
~$ uname -r
5.1.9-1.el7.elrepo.x86_64
~$ gcc --version
gcc (GCC) 9.1.0

Thanks @bbn-zbertils and @danobi for your replies.
I followed the same steps as @bbn-zbertils
@bbn-zbertils - Can you please share the cmake configure command line you used for building llvm/clang from source? Perhaps I am missing something. I am also using gcc 9.1.0. My kernel is a little newer though - 5.1.12, but I don't think that's the issue.
I will also file the issue on bcc repo.

llvm/clang with specific commit I used
git clone https://github.com/llvm/llvm-project.git
git checkout 591c793b48e487382c30e352c0c3b7ae60b45a87
mkdir build && cd build
cmake -DCMAKE_BUILD_TYPE=Release -DLLVM_TARGET_ARCH=ARM -DLLVM_TARGETS_TO_BUILD='ARM;BPF' -DLLVM_BUILD_LLVM_DYLIB=ON -DLLVM_ENABLE_RTTI=ON -DLLVM_ENABLE_PROJECTS=clang -G "Unix Makefiles" ../llvm
make && make clang
sudo make install

And gcc needs the newer libstdc++ libraries installed too, this is the install-target-libstdc++-v3.
wget https://ftp.gnu.org/gnu/gcc/gcc-9.1.0/gcc-9.1.0.tar.gz
tar -xf gcc-9.1.0.tar.gz
cd gcc-9.1.0
./configure --with-system-zlib --disable-multilib --enable-languages=c,c++
make -j4
sudo make install
sudo make install-target-libstdc++-v3

You may get library path errors running bpftrace since the llvm/clang libraries get installed to /usr/local/lib instead of /usr/lib, either symlink the libraries to /usr/lib or setup your sudo/root LD_LIBRARY_PATH variable to point to /usr/local/lib.

Thanks much @bbn-zbertils. Appreciate your help. These instructions work for me!


Just some improvements on the scrollbar

It's just an improvement on the scrollbars.
I added a CSS class called .no-scrollbar to hide the scrollbars by default and show it on hover, making it more compatible with the UI.
Before

After

My elixir extension added the parentheses in the functions when I saved, I can remove it if necessary.

Hey @devsnaked, thank you for this PR! I left two minor comments, rest LGTM!

Thank you @devsnaked ❤

@devsnaked can you please fix the newline so that the static analysis CI step will pass? I guess it finds something like /r instead of /n or something like that, most a likely problem with the editor config. After that, I'll merge it right away :).

Superseded by #2160


chisel only works fine once

I installed chisel correctly, and it work fine when i debug first time.
but when i run debugging again, chisel can not be loaded. So i have to restart the xcode to make it works again.
please tell me how to fix this?
the scripts works like this
//first time
(lldb) visualize someview

//second time
(lldb) visualize someview
error: 'visualize' is not a valid command.

@adad184 When you mean "debug first time" do you mean when you first start your debugging session it only works one time (ex: one breakpoint it works, the next it doesn't). Or it only works for one debugging session(the next time you push run it doesn't work) ?

it works only for one debugging session.  When i rerun the debugging session, it's failed.

@adad184 are you still having this issue? Can you run command script list when visualize is no longer working? Also please post the contents of your ~/.lldbinit.

it seems the issue was gone  after i upgrade my Xcode to release version.
my ~/.lldbinit content is

command script import "~/Library/Application Support/Realm/rlm_lldb.py" --allow-reload
command script import /usr/local/opt/chisel/libexec/fblldb.py

@adad184 . looks like this issue can be closed.

sure  :)


portal stats reflect system wide results

We should make it specific to the portal.

:+1:

Should only require changes to the "get_lat_longs" function in qiita_db/util.py


Interfaces import

I would to nice to be able to import interfaces in devices in CSV format, like VLANs, etc.

Could you describe your use case for this? Interfaces can be created in bulk and should typically be inherited at the time of creation from a device's device type.

We have a bunch of devices with dozens of VLAN interfaces. It was a pain to add them to netbox as the VLANs were randomly numbered. Indeed, we haven't included the VLAN interfaces to the corresponding device types as these are virtual interfaces, so we had to add them manually.
I have created a PR referencing #821 to support commas in bulk creation. But it doesn't help to create descriptions in bulk too.
So, having a way to import interfaces with descriptions would be nice.

+1 on this.

Interfaces inherits from their device type however their MAC details needs to be manually added per device it becomes tedious for bulk of machines
We will need a API to ADD MAC associated with a interface for a host as well UPDATE MAC for an associated host interface whenever NIC  replacement takes place

We would also need this. We have 2000 servers in a exisiting DC and need a some way to import "current state" including interfaces, their vlans and mac addresses.

I would second this feature but from a much more simple use-case, simply to add the ability to move all data from one netbox installation to another without modify it. Simply put just to export all the data for all Interfaces and to what VM:s they are connected. Then import it again in a new Netbox instance.

The original counter-argument said:

Interfaces can be created in bulk and should typically be inherited at the time of creation from a device's device type

However, VMs do not have templates or device types, and don't even get a single interface by default (#1492).  So bulk import of interfaces would be especially useful for VMs.

This would be so helpful! Adding big core routers with many interfaces of different types and statuses can be a pain manually.

BIG +1 for this.  Our use case: several routers terminating hundreds of L2 customer connections as dot1q subinterfaces.  Without the ability to import, I need to manually enter these hundreds of connections with unique interface names and descriptions.
I can very easily create a CSV from router output which would turn a day+ long job into a 5 minute job.

Extending the scope from interfaces to all device components.

Another use case for importing interfaces is for routers with line cards.
The interfaces on the main Device are only those interfaces on the chassis.  When you add a line card (as an inventory item) you don't get any more interfaces, so those interfaces all need to be imported directly against the Device.  This is also the point where you choose the appropriate names (e.g. Gi1/0/1..24 vs Gi2/0/1..24)

Because this import functionality nicely complements the export functionality that @DanSheps has implemented for Add an interfaces list view #3564 I am basing my import code on a fork of https://github.com/DanSheps/netbox/tree/list-view-for-ports.
I'll wait for Dan to finish his PR for list-view-for-ports. I'm trying not to get in each other's way because we're touching some of the same code. Once Dan's code has been merged I'll rebase and create a PR for my changes.

i wrote a few python scripts that pull from our monitoring solution's api and updates interfaces in netbox automatically via the api. Mac addresses, descriptions, etc are easily added and updated via python manage.py sync -h. I can see about scrubbing them and sharing if any interest.

i wrote a few python scripts that pull from our monitoring solution's api and updates interfaces in netbox via the api. Mac addresses, descriptions, etc are easily added and updated via python manage.py sync -h. I can see about scrubbing them and sharing if any interest.

I think both an import function and example sync scripts have their place. I'd love to see them!

Hmmm, the only dependency I have to #3564 is for webpages to link to/from, so I rebased my changes and have a clean patch against develop now.

The bulk of this work was accomplished by @steffann in #3711, however we still need to link to the import views in the navigation menu. Waiting on #3719 for that.


Add Intercom to cookies policy

I'm experimenting with Intercom on the website so I'm updating our cookies policy to list it.
There's no need to notify customers as this is not installed in on the platform. Just the marketing site.

Direct link to preview: https://deploy-preview-230--roadie.netlify.app/legal-notices/cookies-policy/v3/
The Intercom entity name comes from their T&C: https://www.intercom.com/legal/terms-and-policies?on_pageview_event=terms_footer

https://app.clubhouse.io/larder/story/2501/add-intercom-to-roadie-io


burton_2022

The data from burton_2022 seems to be from burton_2019. The mice are same but the ROI positions are different in the two papers
burton_2022: https://elifesciences.org/articles/80470
burton_2019: https://www.ncbi.nlm.nih.gov/pmc/articles/PMC6410398/

@TravisGould Could you take a look?

Fixed by #181.


PHP version locked at 7.2

I tried to run this on my desktop Mac which is running PHP 8 via Homebrew, and composer refused to install due to the hard-coded PHP version requirement in composer.json:
    "require": {
        "php": "^7.2",
        "ext-ctype": "*",
        "ext-iconv": "*",
        "guzzlehttp/guzzle": "~6.0",

Going to try to see if anything breaks if I remove this requirement.

The PHP 8 compatibility code sniffer plugin found no errors, and I was able to run the create_bag command with no issues on PHP 8.1.
e] [-V|--version] [--ansi] [--no-ansi] [-n|--no-interaction] [-e|--env ENV] [--no-debug] [--] <command>

iMac:bin aoneill$ ./console app:islandora_bagger:create_bag --node 96 --settings /Users/aoneill/dev/islandora_bagger/sample_config.yml

 [OK] Bag created for http://localhost:8000/node/96 at /tmp/96.zip

iMac:bin aoneill$ composer sniffer:php8^C
iMac:bin aoneill$ cd ..
iMac:islandora_bagger aoneill$ composer sniffer:php8
> phpcs -p ./src --standard=vendor/phpcompatibility/php-compatibility/PHPCompatibility --runtime-set testVersion 8.0
...................... 22 / 22 (100%)

Time: 297ms; Memory: 10MB

iMac:islandora_bagger aoneill$
iMac:islandora_bagger aoneill$

@mjordan  I can make a PR for this but it would be more straightforward to wait for #25 to be merged since there are composer.lock changes.

Ok, I've merged #25 into main. I had to fix some merge conflicts. I still can't generate Bags from the UI but since you can I'm OK with merging now so we can move on.
If we bump "php": "^7.2" to 8.1, will that make 8.1 a new requirement to run the CLI?

That would be my preference.


Updated Issue event

Added missing fields.
@svanharmelen Looking for review.

No need to @ me in every issue or PR (so please don't), I get notified of all changes anyway and will have a look when I have time. Thanks 👍🏻


feature: add window picker

Did you check the docs?

[X] I have read all the LazyVim docs

Is your feature request related to a problem? Please describe.
When 'w' is pressed in the Neotree to open a file with window picker (as described in the help menu) I get the following message:
"You'll need to install window-picker to use this command: https://github.com/s1n7ax/nvim-window-picker".
Describe the solution you'd like
Please add the the necessary plugin (nvim-window-picker or equivalent) for this option to work.
Describe alternatives you've considered
None
Additional context
No response

You have the option to add it to your own config.

Yes, but I guess it would be logical to include it in LazyVim, since Neotree for example already relies on it. Or maybe add it to the extras for convenience?
In the end I could add every plugin to my config myself, but I use a package like LazyVim for this convenience.

There are so many plugins out there that cannot be all installed in a Neovim distro. And many plugins decide to depend on other plugins for some capabilities. A distro generally comes mostly as a template and then users can tinker their personal config however they want. I don't believe it's possible for a distro to cater to everyone's needs. nvim-cmp relies on so many external sources for different capabilities. Should they also be included in a distro? I don't think that's possible.

The solution is to just add the below to any file under lua/plugins/:
return {
    's1n7ax/nvim-window-picker',
    name = 'window-picker',
    event = 'VeryLazy',
    version = '2.*',
    config = function()
        require'window-picker'.setup()
    end,
}


UI fixes from review

Went over VCAuthN UI (mostly mobile views) with Kim and Emiliano May 16.

Fix the widths of the status headers to line up with buttons/QR code border
Adjust disabled button styling
Add spinner to Pending state and adjust wording
Fix tablet layout so buttons aren't so wide (fix a min/max width for all layouts)
If you deep link with the QR Code (other device) section open then just close that, disable other device button

See mobile state UX layouts here: https://xd.adobe.com/view/8e641a79-0862-4b46-9821-74b394955e24-db37/screen/db55cc4d-87e2-4eeb-9f7a-a39839bffe9c/

Merged in https://github.com/bcgov/vc-authn-oidc/pull/540


help

Hello
what should i need to do for test SPID authentication in testing envrionment
kindly help
thank you

Hi @Asifkrishaweb,
as I wrote in another issue, this space is not intended for support requests.
Please refer to the SPID community at https://developersitalia.slack.com/messages/C73R3UQE8.

http://localhost:8000/spid/acs
URL return below error
Malformed request: idp value not present or wrong (this is likely due to a non-https request)
please help for the same


Document how the CSS works

Stylesheets are located in the app. In our case:
https://github.com/unfoldingWord/gateway-translate/tree/main/src/styles

This documentation should be kept in the editor component, not the app. Other apps using the component must be able to find it.

@birchamp Not really. But as noted the documentation should be in the component so that any app using the component can find it. With the switch to oce editor tools, perhaps this should be assigned to @larsgson


Prevent negative numbers in scope prune

Provide an error message if a negative number is entered for scope prune --keep X or scope prune --delete X.
Closes https://github.com/criblio/appscope/issues/407

Looks good to me.  Merging to master, and adding to Milestone 0.7.2

Tried:
bin/linux/scope prune -d -1
bin/linux/scope prune -d 0
bin/linux/scope prune -d 1
bin/linux/scope prune --delete -1
bin/linux/scope prune --keep -1


Confusing Total params in mxnet.visualization.print_summary

Description
I observed inconsistent number of params in mxnet.visualization.print_summary with actual weights when FullyConnected having flatten=True set.
Environment info (Required)
I ran the experiment on Google Colab.
----------Python Info----------
Version      : 3.6.7
Compiler     : GCC 8.2.0
Build        : ('default', 'Oct 22 2018 11:32:17')
Arch         : ('64bit', '')
------------Pip Info-----------
Version      : 19.1.1
Directory    : /usr/local/lib/python3.6/dist-packages/pip
----------MXNet Info-----------
Version      : 1.4.1
Directory    : /usr/local/lib/python3.6/dist-packages/mxnet
Hashtag not found. Not installed from pre-built package.
----------System Info----------
Platform     : Linux-4.14.79+-x86_64-with-Ubuntu-18.04-bionic
system       : Linux
node         : 0ef71a01d851
release      : 4.14.79+
version      : #1 SMP Wed Dec 19 21:19:13 PST 2018
----------Hardware Info----------
machine      : x86_64
processor    : x86_64
Architecture:        x86_64
CPU op-mode(s):      32-bit, 64-bit
Byte Order:          Little Endian
CPU(s):              2
On-line CPU(s) list: 0,1
Thread(s) per core:  2
Core(s) per socket:  1
Socket(s):           1
NUMA node(s):        1
Vendor ID:           GenuineIntel
CPU family:          6
Model:               63
Model name:          Intel(R) Xeon(R) CPU @ 2.30GHz
Stepping:            0
CPU MHz:             2300.000
BogoMIPS:            4600.00
Hypervisor vendor:   KVM
Virtualization type: full
L1d cache:           32K
L1i cache:           32K
L2 cache:            256K
L3 cache:            46080K
NUMA node0 CPU(s):   0,1
Flags:               fpu vme de pse tsc msr pae mce cx8 apic sep mtrr pge mca cmov pat pse36 clflush mmx fxsr sse sse2 ss ht syscall nx pdpe1gb rdtscp lm constant_tsc rep_good nopl xtopology nonstop_tsc cpuid tsc_known_freq pni pclmulqdq ssse3 fma cx16 pcid sse4_1 sse4_2 x2apic movbe popcnt aes xsave avx f16c rdrand hypervisor lahf_lm abm invpcid_single pti ssbd ibrs ibpb stibp fsgsbase tsc_adjust bmi1 avx2 smep bmi2 erms invpcid xsaveopt arat arch_capabilities
----------Network Test----------
Setting timeout: 10
Timing for MXNet: https://github.com/apache/incubator-mxnet, DNS: 0.0151 sec, LOAD: 0.6442 sec.
Timing for Gluon Tutorial(en): http://gluon.mxnet.io, DNS: 0.0308 sec, LOAD: 0.8126 sec.
Timing for Gluon Tutorial(cn): https://zh.gluon.ai, DNS: 0.0341 sec, LOAD: 0.7928 sec.
Timing for FashionMNIST: https://apache-mxnet.s3-accelerate.dualstack.amazonaws.com/gluon/dataset/fashion-mnist/train-labels-idx1-ubyte.gz, DNS: 0.0430 sec, LOAD: 0.3666 sec.
Timing for PYPI: https://pypi.python.org/pypi/pip, DNS: 0.0070 sec, LOAD: 0.3116 sec.
Timing for Conda: https://repo.continuum.io/pkgs/free/, DNS: 0.0135 sec, LOAD: 0.1245 sec.

Minimum reproducible example
inp = mx.symbol.Variable('data')
op = mx.symbol.FullyConnected(data=inp,num_hidden=1, no_bias=True)
net = mx.mod.Module(symbol=op, label_names=None, context=mx.gpu())
net.bind(for_training=False, data_shapes=[('data', (1,3,2,2))])
net.init_params()

When I called mx.visualization.print_summary(op, shape={'data':(1,3,2,2)}), it shows the following summary:
________________________________________________________________________________________________________________________
Layer (type)                                        Output Shape            Param #     Previous Layer
========================================================================================================================
data(null)                                          3x2x2                   0
________________________________________________________________________________________________________________________
fullyconnected0(FullyConnected)                     1                       3           data
========================================================================================================================
Total params: 3
________________________________________________________________________________________________________________________

where Total params is 3. However, when I inspected the weights through net.get_params(), the fully-connected layer has actually 12 params since the input(data) is flattened.
({'fullyconnected0_weight':
  [[-0.00124826 -0.00404931  0.00783546 -0.00886574  0.00927326 -0.00454687
    -0.00233117 -0.0004467   0.0058345   0.00624337  0.0005779  -0.00040046]]
  <NDArray 1x12 @cpu(0)>}, {})

What have you tried to solve it?
I tried to modify visualization.py so that the pre_filter is correctly counted but couldn't find information for parameter flatten of mxnet.symbol.FullyConnected.

If you are looking for information about FullyConnected, you can check the docs at https://mxnet.incubator.apache.org/api/python/symbol/symbol.html#mxnet.symbol.FullyConnected or the source at https://github.com/apache/incubator-mxnet/blob/master/src/operator/nn/fully_connected-inl.h.


Z-index removed?

After last updates flatpickr is invisible on modal box (bootstrap 4 modal), couse z-index from styles was removed :(

In my case, the the theme css I was using (material-red) had a low z-index (2) on .flatpickr-calendar that overwrote the .flatpickr-calendar z-index on the core css. I changed that and it was ok again.

Hello,
z-index is still there.
Make sure you're using the latest css.
And next time please follow the issue format so I know which version of flatpickr you're using

I use last 2.3.0 version.
https://jsfiddle.net/rorc/a9v2nf53/

I had this problem also, if you use the unminified version z-index is correct, if you use the minified version z-index is set to 1.

Are you using CSS nano? If so, remember to change its settings. CSS Nano used to reset z-index by default. I don't know if this changed.

No, i am not using CSS Nano.
Mayby worth to add z-index as option?

Seems that z-index is turned to 1 again in https://github.com/chmln/flatpickr/blob/master/dist/flatpickr.min.css

Yes, as I said if you use the flatpickr.css file it works correctly.

I am using a library vue-flatpickr which is using flatpickr.min.css https://github.com/jrainlau/vue-flatpickr/blob/2.0/src/theme/flatpickr.min.css
unminified CSS should be the same as productionized CSS, otherwise what is the point...

OK so cssnano was at fault thus being only evident in themes and minified css.
I've turned off its 'optimizations' and will publish a hotfix release soon.

Mistakes happen, god I've made enough, no worries :)

https://github.com/chmln/flatpickr/commit/c9359c34b79bc7773a835cb3595eac892682edf3


add mock config

Pull Request Test Coverage Report for Build 213

5 of 5 (100.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+1.1%) to 96.517%

Totals

Change from base Build 210:
1.1%

Covered Lines:
2910

Relevant Lines:
3015

💛  - Coveralls

Pull Request Test Coverage Report for Build 213

5 of 5 (100.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+1.1%) to 96.517%

Totals

Change from base Build 210:
1.1%

Covered Lines:
2910

Relevant Lines:
3015

💛  - Coveralls


[WIP] [SR-1573] swift build --repl

Implements SR-1573
Allows you to pass the --repl flag to swift-build which will…

build the package as usual, then
launch the Swift interpreter with -L and -I flags set to the the build directory it just used.

These flags allow you to import the libraries defined in the package from the interpreter.

import errors
I'm still having some issues that I'm not sure how to work through. My test module will import ok, but I get this error when I try to initialize the type Foo in Library:
error: Couldn't lookup symbols:
  Library.Foo.__allocating_init(bar: Swift.Int) -> Library.Foo
  type metadata accessor for Library.Foo

notes
Looking around the web for "import swift packages into repl", some answers suggest passing the -l flag in addition to -I and -L. That hasn't seemed to affect anything in my tests. I'm a bit out of my depth here, so any guidance would be appreciated.

My guess would be that the repl still requires dylibs and we are not building those by default.

@neonichu is right. We need to create a dylib product that contains all of the library packages in the  graph and then launch the REPL with the correct arguments. We can create this product when we're building the root package in PackageBuilder.swift. This product should include all of the library targets of the package. Let me know if you have more questions.

I think so... is that equivalent to specifying a dynamic library product in the package description? I'd like to know how to do it "the hard way" before I build the tooling, so I know what I'm shooting for.

Yeah, it’s exactly like that. Instead we’d just synthesise this product with repl option.

TODO

[ ] Before building, define a dynamic library product that the REPL can use.
I assume this library should reference all targets in the package so the user can import any of them.
I also assume this needs to be inserted into the reachableProducts of the PackageGraph so it ends up in the LLBuildManifest. I'm still figuring that part out, though.
[ ] Link this library to the REPL using the -l flag.

This works for packages with one or more library. However, there seems to be an issue with packages that have two or more executable targets. I wanted to filter out executable targets from the REPL product, but I didn't see a way to distinguish an executable target from a library target at the manifest level. That might indicate that this was the wrong level to hook into.

[ ] Resolve the problem with targeting executables.

I continued the remaining work in this PR: https://github.com/apple/swift-package-manager/pull/1793 I'll close this one. Thanks for taking a stab on this!

Glad we knocked this out!


update list of SIGs

Please ensure your pull request adheres to the following guidelines:

[x] For first time contributors, read Submitting a pull request
[x] All code is covered by unit and/or runtime tests where feasible.
[x] All commits contain a well written commit description including a title,
description and a Fixes: #XXX line if the commit addresses a particular
GitHub issue.
[x] If your commit description contains a Fixes: <commit-id> tag, then
please add the commit author[s] as reviewer[s] to this issue.
[x] All commits are signed off. See the section Developer’s Certificate of Origin
[x] Provide a title or release-note blurb suitable for the release notes.
[x] Are you a user of Cilium? Please add yourself to the Users doc
[x] Thanks for contributing!

Update the list of SIGs to reflect all sig- channels found in Cilium Slack, as well as some minor styling changes. This list needs to be updated to further streamline the process of getting Cilium Feature Proposals approved.

Please add the following label: area/documentation

/test


Y24-255 - [BUG] Gatekeeper Exception

Describe the bug
A reoccurring exception is being thrown from Gatekeeper. (~52 occurrences since 2021)
e.g. 13/08/24
[Gatekeeper PRODUCTION] barcode_labels#create (NoMethodError) "undefined method `map' for #<ActionController::Parameters…
A NoMethodError occurred in barcode_labels#create:
e.g. undefined method map' for #<ActionController::Parameters {"61056"=>"61056"} permitted: false>   app/controllers/barcode_labels_controller.rb:30:in generate_labels'
e.g. undefined method `map' for #<ActionController::Parameters {"44310"=>"44310", "44311"=>"44311", "44312"=>"44312", "44313"=>"44313", "44314"=>"44314", "44315"=>"44315"}
Look into what is causing this exception and discuss whether the fix is worthwhile with the team

Closing as not part of the Exception Handling Process Trial

duplicate of #437


AttributeError: '_Reactor' object has no attribute 'connectTCP'

When I attempt to start a server inside vim by typing:
:CoVim start 22 "abc"

I get the error:
Traceback (most recent call last):
  File "<string>", line 1, in <module>
  File "<string>", line 305, in command
  File "<string>", line 263, in createServer
  File "<string>", line 249, in initiate
AttributeError: '_Reactor' object has no attribute 'connectTCP'

I use OSX 10.10.5 and have followed all installation instructions.
Also, when I try to connect I get this error:
klass-air:plugin Klas$ python CoVimClient.vim 8555
  File "CoVimClient.vim", line 2
    if !has('python')
       ^
SyntaxError: invalid syntax

What went wrong?

The first error seemed to get fixed by upgrading twisted and argparse. The second error was just a misunderstanding, the CoVimClient.vim file should not be launched from the command line.


zeroth pass at Waters schema

This PR adds a working waters schema, so we can make changes to it.
For more readable version of the schema, look here:
https://github.com/CDCgov/prime-data-hub/blob/jim/waters-schema/prime-router/docs/schema_documentation/waters-waters-covid-19.md
I made several of the value set fields to use the more human readable "display" value, so you can see how that works.   If those are still not human-readable enough, we can try the "$alt" feature to create ad-hoc valuesets.  I added one example of that.
I made one field, test_result, a set of "alternate values", including silly values, so you can see how to use that feature, to create expected inputs that might be more human readable if needed.
Commands to try using the schema - to generate fake data, then try to route it thru the system:
mkdir ./junk
./prime data --input-fake 1000 --input-schema waters/waters-covid-19 --output ./junk/waters.csv
./prime data --route --input ./junk/waters.csv --input-schema waters/waters-covid-19 --output-dir ./junk

I have merged these changes into #821 PR.


Beginner tiles Doubt

I am beginner in WPF. I am making one project using Mahapps.Metro.
On my main window i have made some tiles . i have 5 user controls . On click of my 1st tile i want to navigate to my 1st user control. how do i do that ??
Any help would be appreciated.
Thank you

Can you be more specific,maybe send a screen shot? What exactly do you mean by navigate to 1st user control? You could try using NavigationService, you can read more on that here-http://paulstovell.com/blog/wpf-navigation

can anybody tell how do i implement paging on my datagrid ???. i don't want
to use any 3rd party control .
Any alternative
Thank you
On Wed, Jan 13, 2016 at 11:26 AM, iskenxan<EMAIL_ADDRESS>wrote:

Can you be more specific,maybe send a screen shot? What exactly do you
mean by navigate to 1st user control? You could try using
NavigationService, you can read more on that here-
http://paulstovell.com/blog/wpf-navigation
—
Reply to this email directly or view it on GitHub
https://github.com/MahApps/MahApps.Metro/issues/2310#issuecomment-171178486
.

@Ashwani44 Please try using stackoverflow.com for your beginner questions or try first learn more about WPF...

I am doing so. Thanks Jan 😃
On 24-Jan-2016 3:13 am, "Jan Karger"<EMAIL_ADDRESS>wrote:

Closed #2310 https://github.com/MahApps/MahApps.Metro/issues/2310.
—
Reply to this email directly or view it on GitHub
https://github.com/MahApps/MahApps.Metro/issues/2310#event-524018521.


NeuralynxIO: partial support for NVT files

Type of Change

New feature
Bug fix

Description
Added partial support for Neuralynx NVT files (#1463). for NeuralynxIO and NeuralynxRawIO, the x and y pixel coordinates and animal head angle from the nvt file are treated as dimensionless analog signals bundled into a signal stream separate from the NCS stream. NlxHeader can now read NVT files as well. For detailed output please refer to the Testing section below.
A buggy regex in NlxHeader was also fixed. Previously it can only read a single value after the header keys, now it can read multiple.
Limitations

Only supports loading a single NVT file (or a directory with a single NVT file in it) at a time.
Only loads the dnextracted_x, dnextracted_y and dnextracted_angle data fields from NVT files. Other fields that could be potentially useful (dwPoints and dntargets) are not yet supported.
The NVT is assumed to be in the same segment (sharing a common clock (time basis)) as the NCS files in the same directory.

As a temporary solution to the potential problems these limitations might cause, I added an optional ignore_nvt argument to the NeuralynxIO and NeuralynxRawIO classes. When enabled, NeuralynxIO and NeuralynxRawIO will behave like how they were before my changes. NlxHeader works fine without any problems during testing.
Testing
Code was tested in accordance with the official contribution guideline. Additionally, it was tested on my own lab's data. Listed below are some examples.
reader = NeuralynxIO(dir)
print(reader)

nb_block: 1
nb_segment:  [1]
signal_streams: [Stream (rate,#packet,t0): (1600.0, 7537,<PHONE_NUMBER>9) (chans: 13), Stream (rate,#frame,t0): (29.97, 72286,<PHONE_NUMBER>1) (chans: 3)]
signal_channels: [CSC2_2, CSC3_2, CSC4_2, CSC5_2 ... CSC1_2 , VT1 , VT1 , VT1]
spike_channels: [chTT1_1#0#0, chTT1_1#1#0, chTT1_1#2#0, chTT1_1#3#0 ... chTT4_1#15#8 , chTT4_1#15#9 , chTT4_1#15#10 , chTT4_1#15#13]
event_channels: [Events event_id=4 ttl=0]

Each of the VT1 signal channel represents a data field (x, y pixel position and head angle).
print(reader.read_block().segments[0].analogsignals[1])

[[428. 349.  45.]
 [431. 345.  28.]
 [428. 349.  46.]
 ...
 [436. 334.  87.]
 [435. 334.  88.]
 [436. 334.  87.]] dimensionless

First column is x posiiton, second y position and thrid head angle in degrees.
print(reader.read_block().segments[0].analogsignals[0].array_annotations["Resolution"][0])

[720 480]

Video resolution from the NVT file header is passed into array_annotations for easy access. However because list type in array_annotations is buggy, it is passed as a literal string ('[720 480]') and not a list. Though once it is accessed it can be easily converted into a list using ast.literal_eval() in the ast package.
Additional Information
I should be able to make semi-regular contributions to this project. I don't have a good solution in mind to include the dwPoints and dntargets data fields as of now, but I should be able to fix the problem with loading multiple NVT files and implement proper segment assignment in the near future.
I opened this pull request dispite the above mentioned limitations because I think in most cases (at least in my lab and other labs I know of) there are only one NVT file in a given directory, and the files often share the same time basis (since they are likely from the same experiment session). So this current version should be helpful enough in most scenarios. With the ignore_nvt argument, users will also be able to revert back the code's behavior. So even if the new code causes problems, they could be easily avoided.

Thanks @LEGion-42, we will review soon. :)

pytest somehow skipped all the necessary neuralynxrawio tests for me so I didn't catch these before submitting the PR. Sorry for the oversight. :(
The 4 AssertionError's are my code working as intended and does not cause problems -- the test simply did not consider the fact that I'm passing behavior data as signals. The header size of 4 is just the original 1 from an ncs file + 3 new ones from the nvt file. I have now updated the test with proper values.
The KeyError on the other hand raises some questions. neuralynx/Cheetah_V5.6.3 is the only directory with an nvt file in the testing dataset, and according to the timestamps, the time range of the nvt file is entirely contained in the first segment. However, while reading the data, NeuralynxRawIO is still trying to read nvt signal channels in the second segment despite it not being there. It seems like we are assuming that every signal is present in every segment? Or am I missing something?
I implemented a temporary fix by duplicating the nvt data to every segment to gether with a warning message telling users that nvt file should be loaded independently if there are multiple segments. But I think a proper fix should be allowing different segments have different channels?

That is likely what we would do. I guess I don't know NVT/Neuralynx well enough. Segments for neuralynx can be due to gaps in the signal. So my questions would be 1) are there not gaps in the nvt that we need to account for? 2) can you have behavioral input of some sort for one segment realistically and not for others? You wouldn't happen to have some easy to access docs for the neuralynx format would you?

Again, I'm sorry for continuosly failling the test. My pytest for some reason (potentially a bug?) isn't downloading the testing dataset properly so I have to rely on doing tests here. :(
As for your questions:

I have no idea. I'm mainly doing computational tasks in the lab so I don't have a lot of experience doing experiments. The data I have access to are all single segment (nvt, ncs, etc.) per directory and the nvt file in the testing dataset also does not contain gaps.
I don't think so, but the nvt file in the testing dataset does not align with the ncs files (which is why the error occured). So I don't know if this is a realistic scenario that I should account for or not. However I imagine some improper lab setup could produce data like this. Sadly, I don't have any additional doc for the neualynx format other than the one available on the official website.

Anyway, I think it is important to have more data for testing. My own data is too clean and the only one on GIN looks to be artificially constructed. I would greatly appreciate it if you guys can provide more data for me to test on. :)

thank you for your understanding! Now that the tests have passed, I think this is ready for review?

@LEGion-42, yep. We were just discussing who would be best to review. @PeterNSteinmetz has worked on this the most so if he has the bandwidth we would love his opinions. Otherwise @samuelgarcia said he would take a look as well. We are in the middle of a conference this week so next week will be the earliest for our final review. Hopefully @PeterNSteinmetz can be review it!

So I had an initial look at this and have some questions. @LEGion-42
First, are there some test .nvt files that can be used here and committed to the ephy_testing_data?
What sort of sampling rates are typically used for these files?
There is a prior PR for having a no date and time in the header and test data for it, though neither of these have been incorporated yet. Perhaps it is time for that test data to be pulled into the gin repository? @samuelgarcia Or I can do it myself but have been trying to follow the recommendation to not do that on your own data.
While Sam's last set of changes was done on a semi-emergent basis without test files due to an upcoming class, it really is best to have test data and tests in place when adding new features.

@PeterNSteinmetz
I'll have a talk with the PI tomorrow and see if I could share some of our data. However as I mentioned before our data may be too clean to be meaningful for testing. There is no segmentation or noticeable gap in timestamps in any of our data.

@LEGion-42 . Thanks that sounds good. The test files are really just to test that headers are being read properly and records interpreted correctly. Normally we trim them down to a smaller number of records and use a small number of them. If there are no unusual features like gaps in the files, that is fine. The files can be anonymized if needed to remove any information, such as time/date, which might be identifiable to an individual person.
I agree the video tracking data should be in the same segment as per this design. Given the separate sampling frequency it should be in a separate stream.

@PeterNSteinmetz I got permission to share our nvt data. However our nvt files have size ranging from 30 to 100mb, which are all outside of the 10mb file size requirement on gin and I have no idea how to trim them down. Any suggestions?

@LEGion-42 Great. We don’t need an entire recording set, just perhaps 3-6 files. Since the header parsing is of primary interest, please trim to just header plus 10-20 records each.
If you are on MacOS or another Unix system you can use the ‘truncate’ command to shorten them. I usually add a ‘_trunc’ suffix to the file name before the extension to indicate this has been done.
You can compute the target size as 16384 bytes + number of records x record size. The record sizes for the various file types are available in the document at https://neuralynx.com/_software/NeuralynxDataFileFormats.pdf

@zm711 Zach, what do we need to do to get some of the pull requests for corresponding test data merged over there?

Hey Peter. Are you talking about from the beginning? We have instructions on our repo how to make a PR on gin. If you're talking about merging test data just @ one of us and we can review it and merge it. We like to check to make sure it's not too big and that everything looks to be in order before we merge it. :)

@LEGion-42 If you would like some further assistance with getting the test files ready, just let me know.

@PeterNSteinmetz Sorry for no being responsive, I was quite busy the past week. I'll try to upload the nvt data today.

@zm711 @PeterNSteinmetz Here are the the nvt files. gin wouldnt let me pass the their captcha for some reason (maybe im a bot?). So can you guys upload them? Sorry for the trouble.
nvts.zip

I am in the Tetons now and can take a look at these about July 2.
Thanks,
Peter
@.***

@LEGion-42 These downloaded fine for me. I will try and structure the folder, write some tests, and start a pull request in the next few days.
@zm711 We are still waiting on approval of the prior pull request for the ephy testing data https://gin.g-node.org/NeuralEnsemble/ephy_testing_data/pulls/116 and will then need examination of the new pull request for this. This has been a very slow point historically.

@PeterNSteinmetz, it is merged. I would ping us here on github in your PR (probably a couple times) so we review and merge it. Thanks for all your help on these PRs!

Actually @PeterNSteinmetz, that was a broken file. I had to overwrite and erase your PR (gin can be annoying). You submitted a stub on your branch rather than a file, but since I had merged it I had to delete it.

Thanks Zach. @zm711 I am going to move discussion of that dataset over to its pull request at https//gin.g-node.org/NeuralEnsemble/ephy_testing_data/pulls/116 .

@LEGion-42,
We are still working on getting your test file onto g-node/gin. So that is in the works. There was another PR for neuralynx where we were working on simplifying the parameters moving forward. That has now been merged which has caused conflicts in your PR. Would you like to fix those conflicts while we working on adding the test files.

@zm711 I'll take a look on the weekend. Thanks for letting me know.

@LEGion-42 I am about to prepare the push of the shortened .nvt files you provided. But I am wondering if a directory structure where all 5 .nvt files are in one directory would be good? This would allow testing of providing a directory to the constructor, which should then produce 5 channels, versus individual file names which should produce one.
Would such a structure provide any difficulties for your code or tests?

@PeterNSteinmetz currently the code only supports 1 nvt files per directory. Multiple nvt files won't crash the code but will cause problems.

@LEGion-42 OK, thanks, good to know. I will move them to the separate directories then before committing them.

@LEGion-42 How would you like the Readme.txt file to read? Normally we state where the files are from and who created them.

@LEGion-42 These files have been added to a pull request in the test data https://gin.g-node.org/NeuralEnsemble/ephy_testing_data/pulls/141. Once incorporated there, it should be possible to add some tests to the code to support this addition to the functionality. Thanks for your work on this.

@LEGion-42,
just one quick confirmation: these files are either a dummy recording or simulated data right? We just don't want private files on the public testing repo (for example in the case of human recording/medical data). Could you confirm how this data was obtained and then we will merge the testing data!

@zm711 Data was collected from mice and was explicitly allowed for release by my supervisor so don't worry!

@LEGion-42 I was just working a bit on the header parsing and I noticed something about about the headers in these .nvt files.
They do not contain a first line '######## Neuralynx Data File Header' . This is the only case we have seen where this is missing. That brings up some questions:
Are these produced by a Neuralynx recording system? If so, which one and versions of the software. The header does say CheetahRev 5.6.0. But this just seems quite different. It makes me wonder if there is some configuration difference there on that system what was not intended.
Implementing this will require us to either drop the requirement for that first line as a check generally or perhaps only omit it if the FileType is Video.

@PeterNSteinmetz
I just checked the nvt files (both on my end and the pull request) and the headers including the first lines and the CheetahRev's are all there? Can you please recheck?

@LEGion-42 You know, you are right. I don't know what I got confused. Sorry for the false alarm.
Glad we have the correct files for testing in the ephy_testing_data now.

@PeterNSteinmetz
:D don't worry about it. Glad that the confusion was cleared.


libupnp: 1.8.3 -> 1.8.4

Semi-automatic update generated by https://github.com/ryantm/nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/libupnp/versions.
meta.description for libupnp is: '"libupnp, an open source UPnP development kit for Linux"'.
Compare changes on GitHub

Checks done (click to expand)

built on NixOS

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 1.8.4 with grep in /nix/store/sgfh37h912vdcfx46yc0s96cks5an26i-libupnp-1.8.4

directory tree listing: https://gist.github.com/b26de03d93846ae470db9941c23238be

du listing: https://gist.github.com/e6087bcf442b748d56e90795bbb31e0a

Outpath report (click to expand)

Outpath difference report
79 total rebuild paths
26 package rebuilds
26 x86_64-linux rebuilds
26 i686-linux rebuilds
1 x86_64-darwin rebuilds
26 aarch64-linux rebuilds
First ten rebuilds by attrpath
amule
amuleDaemon
amuleGui
cantata
gmrender-resurrect
libsForQt5.phonon-backend-vlc
libsForQt5.vlc
libsForQt511.phonon-backend-vlc
libsForQt511.vlc
libupnp

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/sgfh37h912vdcfx46yc0s96cks5an26i-libupnp-1.8.4 \
  --option binary-caches 'https://cache.nixos.org/ https://r-ryantm.cachix.org/' \
  --option trusted-public-keys '
  r-ryantm.cachix.org-1:gkUbLkouDAyvBdpBX0JOdIiD2/DP1ldF3Z3Y6Gqcc4c=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(r-ryantm's Cachix cache is only trusted for this store-path realization.)
Or, build yourself:
nix-build -A libupnp https://github.com/r-ryantm/nixpkgs/archive/438a2bb46990caed2931cc3cdfd89a6eee2b751d.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/sgfh37h912vdcfx46yc0s96cks5an26i-libupnp-1.8.4
ls -la /nix/store/sgfh37h912vdcfx46yc0s96cks5an26i-libupnp-1.8.4/bin

Success on aarch64-linux (full log)
Attempted: libupnp
Partial log (click to expand)
make[1]: Leaving directory '/build/source'
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/in5nf2mggq4l4z2pd22h6xz3dbwr77g0-libupnp-1.8.4
shrinking /nix/store/in5nf2mggq4l4z2pd22h6xz3dbwr77g0-libupnp-1.8.4/lib/libupnp.so.13.0.0
shrinking /nix/store/in5nf2mggq4l4z2pd22h6xz3dbwr77g0-libupnp-1.8.4/lib/libixml.so.10.0.1
strip is /nix/store/p9akxn2sfy4wkhqdqa3li97pc6jaz3r1-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/in5nf2mggq4l4z2pd22h6xz3dbwr77g0-libupnp-1.8.4/lib
patching script interpreter paths in /nix/store/in5nf2mggq4l4z2pd22h6xz3dbwr77g0-libupnp-1.8.4
checking for references to /build in /nix/store/in5nf2mggq4l4z2pd22h6xz3dbwr77g0-libupnp-1.8.4...
/nix/store/in5nf2mggq4l4z2pd22h6xz3dbwr77g0-libupnp-1.8.4

Success on x86_64-linux (full log)
Attempted: libupnp
Partial log (click to expand)
make[1]: Leaving directory '/build/source'
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/sgfh37h912vdcfx46yc0s96cks5an26i-libupnp-1.8.4
shrinking /nix/store/sgfh37h912vdcfx46yc0s96cks5an26i-libupnp-1.8.4/lib/libixml.so.10.0.1
shrinking /nix/store/sgfh37h912vdcfx46yc0s96cks5an26i-libupnp-1.8.4/lib/libupnp.so.13.0.0
strip is /nix/store/vcc4svb8gy29g4pam2zja6llkbcwsyiq-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/sgfh37h912vdcfx46yc0s96cks5an26i-libupnp-1.8.4/lib
patching script interpreter paths in /nix/store/sgfh37h912vdcfx46yc0s96cks5an26i-libupnp-1.8.4
checking for references to /build in /nix/store/sgfh37h912vdcfx46yc0s96cks5an26i-libupnp-1.8.4...
/nix/store/sgfh37h912vdcfx46yc0s96cks5an26i-libupnp-1.8.4

Success on x86_64-darwin (full log)
Attempted: libupnp
Partial log (click to expand)
 /nix/store/acnfbicd84bnya2l2dq868b5l482qihw-coreutils-8.30/bin/mkdir -p '/nix/store/1x783lm12c4ha0p879005aji50f9nmxq-libupnp-1.8.4/lib/pkgconfig'
 /nix/store/acnfbicd84bnya2l2dq868b5l482qihw-coreutils-8.30/bin/install -c -m 644 libupnp.pc '/nix/store/1x783lm12c4ha0p879005aji50f9nmxq-libupnp-1.8.4/lib/pkgconfig'
make[2]: Nothing to be done for 'install-data-am'.
make[2]: Leaving directory '/private/tmp/nix-build-libupnp-1.8.4.drv-0/source'
make[1]: Leaving directory '/private/tmp/nix-build-libupnp-1.8.4.drv-0/source'
post-installation fixup
strip is /nix/store/g5r4apl0za012ffs6ladinwa5w0m1l3k-cctools-binutils-darwin/bin/strip
stripping (with command strip and flags -S) in /nix/store/1x783lm12c4ha0p879005aji50f9nmxq-libupnp-1.8.4/lib
patching script interpreter paths in /nix/store/1x783lm12c4ha0p879005aji50f9nmxq-libupnp-1.8.4
/nix/store/1x783lm12c4ha0p879005aji50f9nmxq-libupnp-1.8.4