Loading checkpoint of saved intervened model takes long time

Hi, thanks for the great work, I am trying to replicate the outcome on different dataset, but when I intervene llama3.1 8B

it has following warning, and the loading time is about 20min, I want to ask is that long checkpoint loading time normal?

@jeffreyzhanghc Hi, I worked on updating this codebase and can help you. May I ask what GPU setup you are using to intervene on llama3.1 8b? I used 1 H100 and loaded the model in 1 minute, so perhaps it's a GPU size issue.

Hi, thanks for helping, yes i m using 2 A100 80GB, and it can takes up to 30mins or even more, does the case of long prompt also affect the intervention time

yes i omit the cuda=0

Hi Jeffrey,
Sorry about the late reply, I am in the midst of finals right now at my school. Thanks for your patience.
I just wanted to confirm the workflow you are doing:

You run python edit_weight.py --model_name llama3p1_8B_instruct --num_heads 48 --alpha 15 without the issue of a long loading time. The edited model is saved to .../honest_llama/validation/results_dump/edited_models_dump/llama3p1_8B_instruct_seed_42_top_48_heads_alpha_15.
You run python validate_2fold.py --model_name path_to_edited_model --num_heads 1 --alpha 0 ..., and now you see the long loading time issue.

To clarify: the warning you see of Some weights of LlamaForCausalLM were not initialized from the model checkpoint at results_dump/edited_models_dump/llama3p1_8B_instruct_seed_42_top_48_heads_alpha_15 and are newly initialized: is expected and not an issue. It's because the llama models do not have attention biases, so the ITI-introduced biases aren't expected (but it should still work).
The long loading time is unexpected, however. I was able to load the huggingface llama3.1_8b_instruct model with 2 A100s in ~20 seconds, and also load my edited model in about the same time.
Please let me know if my understanding of your issue is correct, and I'll see how else I can help.


Retrieving status check values from a pull request off the context?

Looking at the payload for a pull request, I'm wondering how to grab the associated list of status checks from this context. For example, a Jenkins build being successful, OWasp being successful, etc.

Do you have any code snippets by chance?

Looks like I'm hitting a 403 GitHub request: GET /repos/:owner/:repo/commits/:ref/check-runs - 403 undefined
I even attached the owner and the repo per the octorestkit docs, but still nothing :(

"not accessible by integration" means the app does not have the required permissions. You need read access to

Commit Statuses
Checks

These two things look the same in the UI, but you need two different requests to retrieve them.
I've implemented this in the past but I can't remember where it was, so I don't have a snippet for you, I'm sorry

@gr2m That was it, thanks! I found an old snippet from you actually on another thread and that's exactly what I need. Appreciate the quick response.

@gr2m @Schachte i am in the same boat to check the status of the CI status , can you please point to any code snippet ?Thank you

@rthadur I would suggest to start here: https://www.npmjs.com/package/@octokit/rest It was useful to me.


Splash page responsiveness

Fixes #291 and fixes #292

I ran this locally and #291 was fixed but I'm still seeing #292?
Also, in the future, if you have multiple commits close to each other with a similar purpose, it may make sense to squash them into a single commit.  It's also helpful to write commit messages in a present imperative style — it's easier to understand what changes you've made in "Remove redundant rules" as opposed to "Minimum width for objects"

For #292 I'm not expecting columns to appear as Firefox doesn't support columns

Yep Firefox isn't behaving as expected

How about removing "fixes #292", merging this pull to fix #291, and then figuring out how to solve the Firefox issue in a new pull?

I'm about to push the fix


Runes implementation will need to change since splitting will be realistic now

Need to talk with a bunch of ppl about this, but i feel like its still not realistic. Higher bonuses are just so much higher than the lower ones that i cannot see anyone taking 3/3 instead of 6 of the same.

Ok, turns enough time has passed to say that noone is splitting runes still.


The project id used to call the Google Play Developer API has not been linked in the Google Play Developer Console.

Hi:
my code :
$client = new \Google_Client();
putenv('GOOGLE_APPLICATION_CREDENTIALS=F:\gitbitback\lets-android.api\api\credentials.json');
$client->useApplicationDefaultCredentials();
$client->setApplicationName('world.letsgo.booster.android.demo');
$client->setClientId('104041803775097038014');
$client->addScope('https://www.googleapis.com/auth/androidpublisher');
$res = $client->getAccessToken();
$service = new \Google_Service_AndroidPublisher($client);
$purchase = $service->purchases_products->get($responseData['packageName'],
$responseData['productId'], $responseData['purchaseToken']);
response 403 error:
{\n "error": {\n  "errors": [\n   {\n    "domain": "androidpublisher",\n    "reason": "projectNotLinked",\n    "message": "The project id used to call the Google Play Developer API has not been linked in the Google Play Developer Console."\n   }\n  ],\n  "code": 403,\n  "message": "The project id used to call the Google Play Developer API has not been linked in the Google Play Developer Console."\n }\n}\n
how can i do? I'm already linked the google play developer console
Thank you!

I have the same problem. Since the Google API console UI was updated, I can't get the API key to work. I follow the same steps as before.
I'm using Fastlane to submit an APK by using an API key.
I get "projectNotLinked: The project id used to call the Google Play Developer API has not been linked in the Google Play Developer Console."
I can see the project ID from the API console listed in the Google Play console. Under: Users & Permissions > Users with access. I've also given it all the necessary permissions (Release manager).
I created the key using:
Create service account
Role: Service accounts - Service account user
JSON key

@wangrong1994 Can you confirm that you still receive this error after uploading your app manually for the first time?

Looks like this one is resolved.

I have the same problem. Since the Google API console UI was updated, I can't get the API key to work. I follow the same steps as before.
I'm using Fastlane to submit an APK by using an API key.
I get "projectNotLinked: The project id used to call the Google Play Developer API has not been linked in the Google Play Developer Console."
I can see the project ID from the API console listed in the Google Play console. Under: Users & Permissions > Users with access. I've also given it all the necessary permissions (Release manager).
I created the key using:
Create service account
Role: Service accounts - Service account user
JSON key
Edit: A few hours later a tried this: I uploaded the Android app manually, and after that the API access seems to work. It looks like the error I was seeing was simply the wrong one. It should have been similar to "no app found with the ID specified".

Thanks, your solution of uploading new Apk to Google Play Console solved my issue.

Yes, This must be added to documentation, key starts to work only after APK upload

Yes, This must be added to documentation, key starts to work only after NEW APK upload (existing will not work)

You saved our day, thanks a lot.


📖 Rename quickstart to Getting Started, other doc brush up

Summary
This PR makes some improvements to the website for navigation and clarity. These include the following.

Rename "quickstart" to "Getting Started".
Made "Getting Started" disjoint from "User Guide", because they are both listed across the top section of the website (if your browser is open wide enough) and I was told that some readers assume this means that they are disjoint.
Rename the navigation DAG node "Setup" to "General Setup", which contrasts with "the Setup section in Getting Started".
Made the references from the end of Getting Started to example scenario 1 use actual hyperlinks.
Added a reference (with hyperlink) from the definitions of the relevant shell variables at the start of the example scenarios document to the concrete example in Getting Started.
Increased the mentions of cluster labeling at the start of the example scenarios document.
Fixed a couple of broken links.

Preview at https://mikespreitzer.github.io/kcp-edge-mc/doc-twist-710/
Related issue(s)
Fixes #

/cc @KPRoche
/cc @clubanderson
/cc @ezrasilvera

/lgtm

/approve


refactor release and evolution workers, remove initial transfer, lots…

need close this PR. the diff is huge.


Add \externalMentorName macro to support external mentor thesis.

This PR addresses issue #45 .
A new macro, \externalMentorName is added. When defined, a new external thesis cover will be generated. When not defined, normal cover will be generated. Users can control which cover to generate by defining this macro.
The usage of this new macro has been given in the source comment.

这个不太完全一样，校外的封面不是简单加一行校外指导教师就可以，上下左右的高度都要调整下…

Word
LaTeX

比起 word 模板来，LaTeX 模板的页面内容好像整体下移了一点

我在调整了……（导师还在催我，别急）

@Lancern macOS 试一下？康康对的准不准 hhh

比较精准了

Word
LaTeX

okk，合并！


Enhancement: Implement Bootstrap API

Implement the bootstrap API in reference to issue #217 raised by @diasdavid

Awesome work @gavinmcdermott ! :D left a couple of questions

LGTM :) Thank you @gavinmcdermott
Could you squash the commits for the merge?


Basic emergency contact system

As a driver I want the system to automatically and quickly contact the emergency services in case I get into an accident.
Acceptance Criteria

[x] The system should provide the user with an option to add their own contact person.
[x] Implement database for storing emergency contact information
[x] The system provides the user with an option to add and remove emergency contacts.
[x] The system shows displays the emergency contact data clearly to the user.
[x] Functionality has been manually tested


Display a rolling month.

Hi, is there a way to set the date of the first row on the monthly view ?
I would like a monthly view with the current week at the first row, to display a rolling month.
thanks for the help.

There isn't a built in way to do this, but you should be able to extend the current Month view, or it's pieces to do this fairly straightforwardly, and then use that view as a custom one


Security Policy - Manage Connector Contracts

Ability to interact with connector contracts from the TSS address. Includes Pausing/Resume the contract and updated the TSS address.
As previously discussed and documented here - https://www.notion.so/zetachain/ZetaChain-Security-Groups-Signers-a1d817bf7abe4957881f1900a3404d7a

Closing as it seems no longer relevant with the current state of the project. Feel free to reopen it.


E10M20-T1 Integration

First, many thanks for all the hard work you do here, it's much appreciated and very useful.
I've just added a E10M20-T1 card to my DS1823xs+ (DSM 7.2-64570-1) and although the script appears to have run without issue and found the new card, none of it's functionality appears to be present i.e. the two M2 SSD drives and/or the 10Gb network port.
I've attached a couple pictures of my current setup and also the latest run of the script for your information.
I'm expecting to see the two additional M2 SSD's from the card, in addition to the built-in M2 SSD's, making 4 in total, all are using  the KINGSTON SFYRD4000G,EIFK31.6 SSD.
I don't know if it's possible to use the 10Gb network port on the card as well, as that also seems to not be present.
Any help would be appreciated.
001.txt

Can you try this version of the script then reboot:
https://github.com/007revad/Synology_HDD_db/releases/tag/v3.1.60-RC

Many many thanks for that, works a treat with no issues at all.
All SSD's and the network port now show.
Have created an SSD cache against Volume 2 with the PCIe card, so both volumes are now cached.
Have also created a network bond with the with the internal 10Gb port plus the 10Gb port from the PCIe card.
If I can be of any help in the future please do not hesitate to reach out.
Again many thanks for all your hard work here, take care.

Awesome.


Have Sonos work nicer with discovery

Issue #673 made me realize that the Sonos platform is not using the discovery info that is being passed in. This is a quick PR to fix this.
I don't have Sonos at home so hoping anyone with Sonos has time to test this.
CC @SEJeff

Other than the failing checks this looks ok to me. I can test this evening when I get home

Did you had time to see if Sonos still works with these changes?

@balloob Yes. I've been having issues with my main test laptop for HA and just seemingly fixed it. This works flawlessly. I was too naive of discovery_info when I added those log messages. This is why I need to write tests, to learn it better.


Hammer2: order of identifiers becomes random. Remove historical adding of slug/objid

https://github.com/crkn-rcdr/cihm-metadatabus/blob/1554bebc5c93de8506d9f17b110c4eb294c1702e/CIHM-Meta/lib/CIHM/Meta/Hammer2/Process.pm#L236-L247
The method we are using to add in the {slug} and the part of the slug after the first "." (previously thought of as the {objid}, when AIP identifiers were defined as {depositor}.{objid}) , and remove duplicates, is causing the order of identifiers to change from what is expressed in the metadata. The order becomes the order of hash keys, which might as well be considered random.
The proposal is to:

Stop adding in {slug} or {objid}, and have the metadata team add the identifiers they feel are appropriate to the source records
Ensure identifiers in CMR remain in the order expressed in the source XML ("identifier" field in IssueInfo and DC, 090$a in MARC)

For reference, the metadata-bus crosswalks DC, Issueinfo and MARC to CMR.
We moved from an XML XSD to a JSON schema a few years back, and continued enhancements to the JSON. There is  a ticket  to convert a spreadsheet used as documentation to something on GitHub, but that hasn't been completed.
CAP/Solr only use CMR. CMR is used with the Solr schema for Search and the fields displayed for the "Document Record" in CAP.

CMR XML Schema Definition (XSD) (not recently maintained)
Code that does the cross-walk from DC, Issueinfo and MARC to CMR
Solr configuration

There are spreadsheets and metadata application profile (MAP) documents that were previously maintained by Julienne that provide documentation in more "human readable" formats of the crosswalks, CMR, etc.


[iOS] Crash when zooming in on videos in Flutter 3.10.0

I'm experiencing a crash issue when zooming in on videos in Flutter 3.10.0, but this problem is specific to iOS. The crash occurs on both physical devices and simulators. Below is the error log:
Lost connection to device.
Exception: Unable to terminate xxx on yyy:
ProcessException: Process exited abnormally:
An error was encountered processing the command (domain=NSPOSIXErrorDomain, code=3):
Simulator device returned an error for the requested operation.
found nothing to terminate
Underlying error (domain=NSPOSIXErrorDomain, code=3):
	The operation couldn’t be completed. found nothing to terminate
	found nothing to terminate
  Command: /usr/bin/arch -arm64e xcrun simctl terminate xxx yyy

[Additional Information]
The crash only occurs when zooming in on videos within the Flutter 3.10.0 framework on iOS.
The issue is not reproducible on Android devices or simulators.
I have tried running the app on different iOS devices and simulators, but the crash consistently occurs.
I have verified that this problem is not specific to my codebase by testing with a minimal Flutter project that only includes video zooming functionality.
Thank you for your attention to this matter. I look forward to a resolution to this issue.
https://github.com/fluttercommunity/chewie/assets/29278033/0b8bc287-42a2-477c-8a16-4eccb1f6ac3c

@hukusuke1007 thank you for bringing this to our attention:
In order to help you further, we will need the version of chewie that you were using.
Also, I will need a reproducible example, since so far, I have not been able to reproduce it using the same iOS simulator and the library's example app.

@diegotori I found the difference point.The problem seems to occur when using google_mobile_ads to display banner ads.....

@hukusuke1007 Then I think you may have to bring this up with the Flutter Team.
There also might be a conflict with the VideoController that's being used, since you may have to keep only one controller open/active at a given time, since google_mobile_ads probably uses video_player under the hood to render video ads.
Either way, please create a reproducible example that you can submit to the Flutter Team, since chewie for all intents and purposes, uses video_player underneath. They're gonna want one in order to help you diagnose this issue further.


Value cannot be null. Parameter name: baseUrl

at Flurl.Url..ctor(String baseUrl)
at SolrExpress.Solr5.SolrConnection.Get(SecurityOptions options, String handler, String data)
at SolrExpress.Core.Search.SolrSearch`1.Execute()
at ...
Config DI:
serviceCollection.AddSolrExpress<ProductResult>(builder => builder
                .UseOptions(new DocumentCollectionOptions<DocumentResult>()
                {
                    HostAddress = @"http://x.x.x.x:8983/solr/someCollection",
                    Security = new SecurityOptions()
                    {
                        AuthenticationType = AuthenticationType.Basic,
                        UserName = Configuration["Solr:User"],
                        Password = Configuration["Solr:Password"]
                    }
                })
                .UseSolr5());

@diegobrum
Seems like SecurityOptions and HostAddress are not include in DocumentCollectionBuilder when UseOptions.
Thanks.

Hi @artziem , please take a look in v5

Almost, with small exception in SolrConnection when call SetAuthentication.
Something like this work for me:
private IFlurlClient SetAuthentication(Url url)
{
	switch (this._options.Security.AuthenticationType)
	{
		case AuthenticationType.Basic:
			return url.WithBasicAuth(this._options.Security.UserName, this._options.Security.Password);
		default:
			return new FlurlClient(url, true);
	}
}

public string Get(string handler, List<string> data)
{
	//...

	return this.SetAuthentication(url)
		.GetStringAsync()
		.Result;
}

//and so on

Hi @artziem , please take a look in v5-Beta2

Thanks for quick reply and improvements.
But in previous post I'm pointed that the bug is also in SolrConnection class.
The methods in it doesn't set auth header. They operate on Flurl.Url not IFlurlClient when sending request.
Instance returned from WithBasicAuth must be consumed in chainable way.
Take a look on my previously posted code with a sample solution.

Hi @artziem !

Please, see version 5, final release

Just to help anyone else out, this same exact error happens in the RestSharp.RestClient library.


Iss427

https://github.com/vizhub-core/vzcode/pull/new/iss427
I have the click on color picker working, but with the dragging in the picker and it changing the hex value is giving me more of an issue

Thank you @MandingoBrandon for this work!
It's an interesting idea to have the mouse movement itself control the colors, but that's not really what we're going for here. Apologies if the original issues was not clear, but the goal with this issue is to make the color picker appear when you click on the colored circle. In other words, the existing interaction for opening the color picker (that happens when you use Alt+click on the hex value) should also happen when you click on the circle (without holding down Alt). Does that make sense?
This PR has a great start in that it does detect when you click on the circle. From there, we want to tie into the other logic around opening up the color picker and getting the values out.

It's a great attempt, but not quite feasible I think.


RadosGW using EC pool

When deploying a cluster with RGW it would be nice if i could

Define EC profile eg. in all.yaml
in rgws.yml create a pool utilizing this profile

Btw i think that the pool creation task block in roles/ceph-rgw/tasks/main.yml should be moved to before the rgw service is started otherwise the rgw will create the necessary pools with default settings ?

In group vars group_vars/all.yml.sample it is possible to define very detailed how cephfs pools should be created.  But for in group_vars/rgws.yml it's only possible to specify the pg_num.
#cephfs_data_pool:
name: "{{ cephfs_data if cephfs_data is defined else 'cephfs_data' }}"
pg_num: "{{ osd_pool_default_pg_num }}"
pgp_num: "{{ osd_pool_default_pg_num }}"
rule_name: "replicated_rule"
type: 1
erasure_profile: ""
expected_num_objects: ""
application: "cephfs"
size: "{{ osd_pool_default_size }}"
min_size: "{{ osd_pool_default_min_size }}"


i18n(ko-KR): update community-content.mdx

Description

update community-content.mdx
#2049

Lunaria Status Overview
🌕 This pull request will trigger status changes.

Learn more
By default, every PR changing files present in the Lunaria configuration's files property will be considered and trigger status changes accordingly.
You can change this by adding one of the keywords present in the ignoreKeywords property in your Lunaria configuration file in the PR's title (ignoring all files) or by including a tracker directive in the merged commit's description.

Tracked Files

Locale
File
Note

ko
resources/community-content.mdx
Localization changed, will be marked as complete.

Warnings reference

Icon
Description

🔄
The source for this localization has been updated since the creation of this pull request, make sure all changes in the source have been applied.


Typo fix: "hove" -> "hover"

Hi @Gama11, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. Real humans will now evaluate your PR.

TTYL, MSBOT;

Hi @Gama11, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. Real humans will now evaluate your PR.

TTYL, MSBOT;


Changing Username doesn't update the User model username

Preflight Checklist

[x] I have searched the issue tracker for an issue that matches the one I want to file, without success.

To Reproduce
Steps to reproduce the behavior:

call the api/users/profile_update endpoint and change the username
try to login with the new username and it won't let you
try calling /api/users/:username and notice it can't find the username you just changed it to

What was expected ?
After changing my username, I should be able to login with that new username and also others should be able to find me via the endpoints by my new username.

Screenshots

I would like to work on this.

has this issue been resolved ?

@Kaushal-Dhungel has this issue been resolved ?

i want to work on it but we need to find out to see if @Kaushal-Dhungel  has already resolved it

@bashiru98 i will assign you tomorrow if @Kaushal-Dhungel didn't answer

hey @MidouWebDev . I am still getting NoCredentialsError while registering, that's why I have not been able to solve it yet.

@Kaushal-Dhungel DM @PraveenMalethia to solve this problem !

it's changing to my end ! can you share the process of how this issue occurs at your end?

hope the issue has been resolved


support for sub-files of archives

The Artifactory API allows for downloading subfiles of an archive (tar, tgz, and zip). For example downloading webservices.tar!/src/base/error_codes/ns_errors.csv would download just the ns_errors.csv file.
If this is added to a spec file pattern then nothing is downloaded.
{
    "files": [
        {
            "build": "sw-dev_${branch}/LATEST",
            "flat": "true",
            "pattern": "builds-local/sw-dev_${branch}/*/webservices.tar!/src/base/error_codes/ns_errors.csv",
            "props": "latest=true",
            "explode": "true",
            "target": "${WORKSPACE}/artifacts/sw/webservices/"
        }
    ]
}

Is it possible for the CLI to support the !subfile download syntax?

Example run where !subfile syntax is does not work.
[srussell@ln70-srussell artifacts]$ jfrog rt dl --spec=temp.json --spec-vars="branch=default;WORKSPACE=tmp" --threads=10
[Debug] Replacing variables in the provided File Spec:
{
    "files": [
        {
            "build": "sw-dev_${branch}/LATEST",
            "flat": "true",
            "pattern": "builds-local/sw-dev_${branch}/*/webservices.tar!/src/base/error_codes/ns_errors.csv",
            "props": "latest=true",
            "explode": "true",
            "target": "${WORKSPACE}/artifacts/sw/webservices/"
        }
    ]
}

[Debug] Replacing '${branch}' with 'default'
[Debug] Replacing '${WORKSPACE}' with 'tmp'
[Debug] The reformatted File Spec is:
{
    "files": [
        {
            "build": "sw-dev_default/LATEST",
            "flat": "true",
            "pattern": "builds-local/sw-dev_default/*/webservices.tar!/src/base/error_codes/ns_errors.csv",
            "props": "latest=true",
            "explode": "true",
            "target": "tmp/artifacts/sw/webservices/"
        }
    ]
}

[Info] Searching items to download...
[Debug] Searching Artifactory using AQL query:
 items.find({"repo": "builds-local","@latest": {"$match" : "true"},"path": {"$ne": "."},"$or": [{"$and":[{"path": {"$match": "sw-dev_default/*/webservices.tar!/src/base/error_codes"},"name": {"$match": "ns_errors.csv"}}]}]}).include("name","repo","path","actual_md5","actual_sha1","size","type","property")
[Debug] Artifactory response:  200 OK
[Debug] Downloaded 0 artifacts.
{
  "status": "success",
  "totals": {
    "success": 0,
    "failure": 0
  }
}

This is indeed a needed feature @scottrus.
Thanks for sharing this requirement.
We'll add it to one of the the future releases.

@eyalbe4 Do you know if this made it into a release and if so which one?

@scottrus,
Unfortunately this hasn't made it in yet...


 The problem was that we were using _getEventTarget to resolve the actual event target and using contains to verify that it's inside the handle. Since contains doesn't descend into shadow root, the call failed. These changes remove the _getEventTarget call since we can use the event target directly.
Fixes #23680.

还没合并到最新版本中么，坐等更新哦


Search filtering and ordering by year

PyVideo has grown. And this is super good! But with this there's a lot of old videos from 2014, 2015 with description of things which you do differently now.
Can you please add an option to filter your results by year, or at least order results by date?
The most important is date, but there are some other desirable filtering options:

length
conference
speaker

I think this issue probably belongs at https://github.com/pyvideo/pyvideo instead, this repository is just to hold the data. (I'd just transfer it but I don't have permissions).

Thank you, I have created https://github.com/pyvideo/pyvideo/issues/243


Fix issue in CORTEX_LM3S811_GCC Demo linkerscipt

The .text section is listed as (.text) and results in build
error because there are some .text sections that are automatically put
into flash later by linker and their LMA overlaps
with .data

Description

explained in description
Test Steps

To reproduce, go to the CORTEX_LM3S811_GCC demo  folder and build with gcc-arm-none-eabi.
Related Issue

n/a
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Thanks for taking the time to submit this PR.  I would be grateful if you could let me know the GCC version you are using.

Hi Richard,
I'm using :
arm-none-eabi-gcc (GNU Arm Embedded Toolchain 9-2020-q2-update) 9.3.1 20200408 (release)

/bot run check

/bot run checks


[core]: cleanup data in clipboard

Prerequisites
affected version: 1.9.2
make sure to check this before filing an issue:

[x] I am running the latest version
[x] I checked the documentation and found no answer
[x] I checked to make sure that this issue has not already been filed

Expected Behavior
After copy/paste operation all "support" information should be removed from the clipboard
Current Behavior
Logviewer keeps in clipboard some "support" information, for example indexes and plugins IDs

solved with 1.11.0


Support for inline defs, ifs, and fors

From<EMAIL_ADDRESS>on 2013-12-16T11:32:07Z
Spyder 2.2.5, Windows 7, python 2.7
One minor bug is whenever I try to type an inline statement such as
for n in range(10): print n
It adds a colon at the end of the line and thinks it should indent. Same goes for inline function defs, ifs, whiles, etc...
I use this kind of thing a lot because it helps line up code and make it readable.
Cheers!
Original issue: http://code.google.com/p/spyderlib/issues/detail?id=1679

We should change the title of this issue, it is misleading.
More like... Editor incorrectly adds colon and indentation on inline defs, ifs, and fors
Should I do this sort of things directly? or you want to be asked first @ccordoba12 ?

This is a multi-duplicate:
https://github.com/spyder-ide/spyder/issues/872
https://github.com/spyder-ide/spyder/issues/1417
https://github.com/spyder-ide/spyder/issues/1448


403 forbidden when agreeing to license

URL to the linked Repo/Org: https://github.com/Unity-Technologies/UnityRenderStreaming/pull/353
Steps to reproduce the problem:

Go to https://cla-assistant.cds.internal.unity3d.com/Unity-Technologies/UnityRenderStreaming?pullRequest=353
Click on "Sign in with GitHub to agree"
Click Sign in with GitHub

What is the expected result?
The license is signed and I can contribute to the project.
What happens instead?
I get to a page saying
403 Permission Denied
Invalid Account

Browser/version:
Chrome 86.0.4240.198 64-bit
Any other tested browsers/devices(OK/FAIL):
Microsoft Edge (FAIL)

@wsprent are you aware of this issue on your instance? Do you know who is responsible for cla assistant at unity?


Document answers to questions in Part 2.3

"Analyze existing test cases that relate to the issue. Do they contain good assertions? Do the assertions
cover the requirements as you understand them? If necessary, enhance existing regression tests with
properties derived from work done above, or write additional tests. Newly added tests related to
refactorings should pass, but if your task is a new feature, new tests should (initially) fail."


Setting CreateIncidentRequest.source_brand has no affect in XSOAR

Describe the bug
The XSOAR platform appears to ignore the CreateIncidentRequest.source_brand member field. Setting a custom source_brand when creating an incident request seems to have no affect in the actual XSOAR platform.
To Reproduce
Steps to reproduce the behavior:

Create an incident request
Set the source_brand field to something other than "Manual"
Upload the newly created incident to an XSOAR test server
Click on the incident in XSOAR
The incident's source_brand is always set to "Manual"

Expected behavior
The newly created incident in XSOAR should have a source_brand field equal to what was set in the CreateIncidentRequest.
Screenshots

Environment

Python Version: 3.8.11
OS: MacOs Big Sur 11.5.2
Demisto Version: 3.0.2

Additional context
I'm not 100% sure if this is an issue with the demisto sdk or a problem with the XSOAR server itself. Feel free to close this issue out if it is the latter, and nothing can be done here.

It is possible.  But according to the implementation you should additionally add an Instance label.
So pass labels to the incident [{"type":"Instance","value":"FILL_INSTANCE"}]


Make Safari work

Makes scheme explicit
Removes need for nonce

(Need to rebase and fix merge conflicts.)

@ithinkihaveacat LGTM, good solution for the JS inline :)

@ithinkihaveacat is the 'unsafe-eval' still needed? I removed it locally and I don't see the error anymore, maybe the bug fix is deployed already.


Better nix-shell

User story
ADP-369
Overview

Uses new Haskell.nix features to put haskell tools in the nix-shell.
Adds bech32, cardano-address, cardano-tx to the nix-shell.
Adds these tools to the release archives ("Adrestia tool belt").
Adds automated tests to the release archives.
Add nix option for building libraries and exes with profiling enabled.
Add bash completion for select tools in nix-shell.

Comments

Merge #1877 first.

bors r+

bors r+


Update hpe-aruba-networking-edgeconnect-orchestrator-tutorial.md

Add steps and screenshots to configure HPE Aruba Networking EdgeConnect Orchestrator SSO.

@microsoft-github-policy-service agree company="HPE Aruba Networking"

@jeevansd

Can you review this PR?
IMPORTANT: When this content is ready to merge, you must add #sign-off in a comment or the approval may get overlooked.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@nikkiching, the screenshots you're added are not having clear visual and not matching with our documentation format, will you please contact me through<EMAIL_ADDRESS>so, we could update the document in better way.


Add a trust manager which will trust certificates issued by the Cellery System

Fixed with https://github.com/wso2-cellery/mesh-security/commit/a884e3668f98ffcb453f32c7843d13fd1e5ffba6


Install and config zsh

[ ] install zsh
[ ] install omz
[ ] enable common plugins

@arrowrowe any idea?

sudo apt-get install zsh git-core

Then follow the Basic Instalation of OMZ.

I mean ... common plugins ....

不用插件... 不过可以推荐下 ag / jq / git-radar 之类的......

well, I think I will just have git enabled for them, but I don't know if it will work properly in vagrant.

changed my mind, bash is enough.

changed my mind, bash is enough.


Add delete_entity in the commands

Same as for set_entity, delete_entity should also work with the command keyword:
commands::<Balance>::delete_entity(query);

I'll take this one!

@tarrencev @aymericdelab


libbz2-rs-sys v0.1.1 requires Rust 1.82

Hello,
https://github.com/trifectatechfoundation/bzip2-rs/releases/tag/v0.5.0 says:
The 0.5.0 release is the first release in ~2 years. It sets an msrv of 1.65.0 and pins the 2021 edition. If the msrv or edition are a problem for your project: just stick with the 0.4.4 release.

I tried to build with 1.74 and I got this error:
error: package `libbz2-rs-sys v0.1.1` cannot be built because it requires rustc 1.82 or newer, while the currently active rustc version is 1.74.0
Either upgrade to rustc 1.82 or newer, or use
cargo update<EMAIL_ADDRESS>--precise ver
where `ver` is the latest version of `libbz2-rs-sys` supporting rustc 1.74.0

Full logs at https://github.com/apache/avro-rs/actions/runs/12293251964/job/34305569034?pr=77
The dependabot PR - https://github.com/apache/avro-rs/pull/77

It seems that libbz2-rs-sys is enabled, which pulls in a pure-rust bzip2 impl that requires 1.82?

Yes, I enabled that feature because otherwise the upgrade from 0.4 to 0.5 failed with:
error[E0463]: can't find crate for `bzip2_sys`
  --> /home/runner/.cargo/registry/src/index.crates.io-6f17d22bba15001f/bzip2-0.5.0/src/lib.rs:60:1
   |
60 | extern crate bzip2_sys as ffi;
   | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ can't find crate

https://github.com/apache/avro-rs/actions/runs/12289804123/job/34295892925

Thanks for the pointer, @NobodyXu !
The problem was that we disable default features and explicitly list only the needed ones - https://github.com/apache/avro-rs/blob/4e055480520499c0b6d3ce4b9fbfec7b9950b843/avro/Cargo.toml#L58
The error message and the release notes made me think that I need to add libbz2-rs-sys feature, while I needed the default one. Now the build passes!
I think that it would be a good idea to mention that libbz2-rs-sys requires 1.82 in the release notes.
Thank you!


Crash when the plugin is enabled in UE 5.4 packaged builds

My packaged Unreal project crashes on startup when the plugin is enabled.
This is the callstack of the crash:
	Game-Win64-Debug.exe!__delayLoadHelper2(const ImgDelayDescr * pidd, __int64(*)() * ppfnIATEntry) Line 312	C++
 	Game-Win64-Debug.exe!CCallback<USteamNotificationsSubsystem::cSteamEventsStore,GameOverlayActivated_t,0>::Register(USteamNotificationsSubsystem::cSteamEventsStore * pObj, void(USteamNotificationsSubsystem::cSteamEventsStore::*)(GameOverlayActivated_t *) func) Line 163	C++
 	Game-Win64-Debug.exe!CCallback<USteamNotificationsSubsystem::cSteamEventsStore,GameOverlayActivated_t,0>::CCallback<USteamNotificationsSubsystem::cSteamEventsStore,GameOverlayActivated_t,0>(USteamNotificationsSubsystem::cSteamEventsStore * pObj, void(USteamNotificationsSubsystem::cSteamEventsStore::*)(GameOverlayActivated_t *) func) Line 148	C++
	Game-Win64-Debug.exe!USteamNotificationsSubsystem::USteamNotificationsSubsystem() Line 32	C++
 	Game-Win64-Debug.exe!InternalConstructor<USteamNotificationsSubsystem>(const FObjectInitializer & X) Line 3559	C++
 	Game-Win64-Debug.exe!UClass::CreateDefaultObject() Line 4586	C++
 	Game-Win64-Debug.exe!UClass::InternalCreateDefaultObjectWrapper() Line 5191	C++
 	Game-Win64-Debug.exe!UClass::GetDefaultObject(bool bCreateIfNeeded) Line 3242	C++
 	Game-Win64-Debug.exe!UObjectLoadAllCompiledInDefaultProperties(TArray<UClass *,TSizedDefaultAllocator<32>> & OutAllNewClasses) Line 810	C++
 	Game-Win64-Debug.exe!ProcessNewlyLoadedUObjects(FName Package, bool bCanProcessNewlyLoadedObjects) Line 894	C++
 	Game-Win64-Debug.exe!FEngineLoop::PreInitPostStartupScreen(const wchar_t * CmdLine) Line 3804	C++
 	Game-Win64-Debug.exe!FEngineLoop::PreInit(const wchar_t * CmdLine) Line 4483	C++
 	Game-Win64-Debug.exe!EnginePreInit(const wchar_t * CmdLine) Line 41	C++
 	Game-Win64-Debug.exe!GuardedMain(const wchar_t * CmdLine) Line 136	C++
 	Game-Win64-Debug.exe!LaunchWindowsStartup(HINSTANCE__ * hInInstance, HINSTANCE__ * hPrevInstance, char * __formal, int nCmdShow, const wchar_t * CmdLine) Line 247	C++
 	Game-Win64-Debug.exe!WinMain(HINSTANCE__ * hInInstance, HINSTANCE__ * hPrevInstance, char * pCmdLine, int nCmdShow) Line 298	C++

Things I've tried:

Verified that steam_api64.dll is included in the package.
Rebuild.
Wiped DerivedDataCache and Intermediate folders followed by Rebuild.
Build on a separate computer.
Checked to make sure all plugin dependencies are included, like SteamSockets and the OnlineSubsystems.

It seems the top call in the callstack points at some kind of dynamic library not being found?
I'm not sure what else to do. I'm on the latest version of the plugin from GitHub.

Yeah some people have this issue where it doesn't bind to the DLLs, however its using the engines implementation not something custom of mine and it works for the vast majority of users. I've never been able to reproduce it on any of my systems to track down what the engine structure is doing here or if its a weird load time issue.
I have considered just ripping out that event binding all together.

I understand. Well, not much you can do if you can't reproduce it reliably.
Thanks for confirming, Joshua!

I moved steam_api64.dll from its ThirdParty location in the packaged game, to where the game executable is. And now everything works. I don't understand but maybe this will help someone.

might need to add a RuntimeDependencies setup in the Build.cs for the thirdparty libs, ref: UE 5.4 - Runtime Dependencies


Exclude unneeded fields from API calls

Colleen added field filtering for resources on the back end https://github.com/rancher/rancher/issues/36683. We should go through some slower list views in the UI and evaluate what data we can exclude using the new query params.
When looking at how data is fetched for list views, we should also check if some data is unnecessarily stalled by waiting for other data, which is what happened when the node list view was blocked waiting for pods: https://github.com/rancher/dashboard/pull/6392

This would mainly help with resources that store data, such as Config Map, Secret and Fleet Bundles.

We are using this for managedFields - I think this is good enough for now and with the upcoming pagination work, this will be enough.


⚠ Bofrost.com has degraded performance

In df4ee99, Bofrost.com (https://www.bofrost.com) experienced degraded performance:

HTTP code: 200
Response time: 659 ms

Resolved: Bofrost.com performance has improved in 5c7e467 after 8 minutes.


Support pathlib.Path values for dvcyaml argument in Live().

…cyaml` argument.

[x] ❗ I have followed the Contributing to DVCLive guide.

[ ] 📖 If this PR requires documentation updates, I have created a separate PR (or issue, at least) in dvc.org and linked it here.

Thank you for the contribution - we'll try to review it as soon as possible. 🙏

Fixes Issue#769

Thanks @AbdullahMakhdoom! Please note in the description that it fixes #769.

@AbdullahMakhdoom it seems mypy failed, could you please take a look. Thanks!

Codecov Report
Attention: Patch coverage is 83.33333% with 2 lines in your changes missing coverage. Please review.

Project coverage is 95.14%. Comparing base (602c053) to head (4f0f16b).
Report is 33 commits behind head on main.

Files with missing lines
Patch %
Lines

src/dvclive/live.py
81.81%
1 Missing and 1 partial :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #842      +/-   ##
==========================================
- Coverage   95.47%   95.14%   -0.34%
==========================================
  Files          57       55       -2
  Lines        3889     3911      +22
  Branches      353      352       -1
==========================================
+ Hits         3713     3721       +8
- Misses        124      139      +15
+ Partials       52       51       -1

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

@skshetry please approve and merge if everything looks good to you.

Apologies for multiple approval. GH bugged out. :(


add DFK chain

Adds support for DFK
Changes relay test behavior to not abort tests when an unknown chain is encountered
Renames relaying.nodeChecker to relaying.service

Closing in favor of #28


Update AppenderSkeleton to support Jalopy implementation

Add protected String name for backward compatibility with Jalopy 1.5-rc3.
Jalopy sets the name directly instead using the designated setter method.

Ping.


setEnable now working properly

Hey, there is a separate branch for development as 'develop'. Unfortunately cannot merge here.


demo.py doesn't work with Docker compose

I have tried with docker compose with your dockerfile
I got an error about OpenGL
`
root@0658d4e7f4db:/app# python demo.py \
--img_folder example_data --out_folder demo_out \
--batch_size=48 --side_view --save_mesh
Traceback (most recent call last):
  File "/opt/venv/lib/python3.12/site-packages/OpenGL/platform/egl.py", line 67, in EGL
    return ctypesloader.loadLibrary(
           ^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/opt/venv/lib/python3.12/site-packages/OpenGL/platform/ctypesloader.py", line 45, in loadLibrary
    return dllType( name, mode )
           ^^^^^^^^^^^^^^^^^^^^^
  File "/usr/lib/python3.12/ctypes/__init__.py", line 379, in __init__
    self._handle = _dlopen(self._name, mode)
                   ^^^^^^^^^^^^^^^^^^^^^^^^^
OSError: ('EGL: cannot open shared object file: No such file or directory', 'EGL', None)

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/app/demo.py", line 9, in <module>
    from hamer.models import HAMER, download_models, load_hamer, DEFAULT_CHECKPOINT
  File "/app/hamer/models/__init__.py", line 2, in <module>
    from .hamer import HAMER
  File "/app/hamer/models/hamer.py", line 7, in <module>
    from ..utils import SkeletonRenderer, MeshRenderer
  File "/app/hamer/utils/__init__.py", line 4, in <module>
    from .renderer import Renderer
  File "/app/hamer/utils/renderer.py", line 6, in <module>
    import pyrender
  File "/opt/venv/lib/python3.12/site-packages/pyrender/__init__.py", line 3, in <module>
    from .light import Light, PointLight, DirectionalLight, SpotLight
  File "/opt/venv/lib/python3.12/site-packages/pyrender/light.py", line 10, in <module>
    from OpenGL.GL import *
  File "/opt/venv/lib/python3.12/site-packages/OpenGL/GL/__init__.py", line 3, in <module>
    from OpenGL import error as _error
  File "/opt/venv/lib/python3.12/site-packages/OpenGL/error.py", line 12, in <module>
    from OpenGL import platform, _configflags
  File "/opt/venv/lib/python3.12/site-packages/OpenGL/platform/__init__.py", line 35, in <module>
    _load()
  File "/opt/venv/lib/python3.12/site-packages/OpenGL/platform/__init__.py", line 32, in _load
    plugin.install(globals())
  File "/opt/venv/lib/python3.12/site-packages/OpenGL/platform/baseplatform.py", line 92, in install
    namespace[ name ] = getattr(self,name,None)
                        ^^^^^^^^^^^^^^^^^^^^^^^
  File "/opt/venv/lib/python3.12/site-packages/OpenGL/platform/baseplatform.py", line 14, in __get__
    value = self.fget( obj )
            ^^^^^^^^^^^^^^^^
  File "/opt/venv/lib/python3.12/site-packages/OpenGL/platform/egl.py", line 93, in GetCurrentContext
    return self.EGL.eglGetCurrentContext
           ^^^^^^^^
  File "/opt/venv/lib/python3.12/site-packages/OpenGL/platform/baseplatform.py", line 14, in __get__
    value = self.fget( obj )
            ^^^^^^^^^^^^^^^^
  File "/opt/venv/lib/python3.12/site-packages/OpenGL/platform/egl.py", line 73, in EGL
    raise ImportError("Unable to load EGL library", *err.args)
ImportError: ('Unable to load EGL library', 'EGL: cannot open shared object file: No such file or directory', 'EGL', None)

'

Please run
apt-get update && apt-get install ffmpeg libsm6 libxext6  -y
apt-get install libglfw3-dev libgles2-mesa-dev

There have been some recent commits from @Kaszanas and @ntvthuyen that have improved the docker functionality. Could you please give the code a try after these updates.

There have been some recent commits from @Kaszanas and @ntvthuyen that have improved the docker functionality. Could you please give the code a try after these updates.

At this point Docker:
#WorksOnMyMachine


Lepro WiFi Smart Bulb GU10 RGBW 902002-eu-2

Hi,
MCU - BK7231T
Ori fw. ver. V2.9.6
LED driver - BP1658
Attached full dump.
BR
Karol
readResult_BK7231T_QIO_2023-15-1--16-35-39.zip

Added profile as Lepro 9200293 GU10 RGBCT Bulb.


Can't call method like numbered parameter inside class statement

Ruby
$ ruby -e 'def _1; :m end; class A; p _1 end'
-e:1: warning: `_1' is reserved for numbered parameter; consider another name
:m

mruby
$ mruby -e 'def _1; :m end; class A; p _1 end'
-e:1:29: warning: numbered parameter in nested block
-e:1:29: numbered parameter outside block

Closed by 06e09fc. Commit message was wrong.


fix(docs): change the arrow flight sql and IBM as400 logo

fix(docs): change the arrow flight sql and IBM as400 logo for dark background

@shrutimantri as the file names have changed, do these need to be referenced differently too?

Undefined image because it doesn't exist


how to get Annocations/DET/train/ILSVRC2013_train_extra0/ILSVRC2013_train_0000001.xml

how to get such files as Annocations/DET/train/ILSVRC2013_train_extra0/ILSVRC2013_train_0000001.xml? many thanks.

same question

you can download the data from the website


[DDW-573] Update Ledger and Trezor JS libraries

This PR introduces an update for Trezor and Ledger JS libraries / dependencies

@cardano-foundation/ledgerjs-hw-app-cardanoto version 2.2.0
trezor-connet to version 8.1.22 .

TODO

[x] Update Ledger JS Lib and apply changes
[x] Update Trezor JS Lib and apply changes
[x] Fix Windows build issue @cleverca22

Testing Checklist

Slack QA thread
[ ] Hardware Wallets regression testing

Test Cases
Trezor Test Cases
Scenario 1 : Add Wallet
Given that I have Daedalus running
When I try to add a Trezor Wallet
Then I will be prompted to export the public key
And the wallet will be successfully imported

Scenario 2 : Reconnecting device
Given that I have a Trezor wallet already paired and loaded
And my device is physically disconnected from my machine
When I physically reconnect my device
And click pair
Then I will be asked to export my public key
And exporting public key will be a success
And I will be redirected to the corresponding wallet summary screen

Scenario 3 : Redelegation
Given that I have a Trezor wallet already paired and loaded
And I wish to redelegate my wallet
When I select redelegate from the Stake Pools Wallet Menu
And I follow the Delegate Wallet wizard instructions
Then I am prompted to export my public key
And I am asked to confirm the transaction

Scenario 4 : Sending a transaction
Given that I have a Trezor wallet already paired and loaded
When I select to send a transaction
Then I am prompted to export my public key
And I am asked to confirm the transaction

Ledger Test Cases

Scenario 1 : Add Wallet
Given that I have Daedalus running
When I try to add a Ledger Wallet
Then I will be prompted to export the public key
And exporting public key will be succesfull
And the wallet will be succesfully imported

Scenario 2 Unlocking and launching Cardano App before opening Daedalus
Given that I have Daedalus installed and currently closed
And my Ledger device is not connected
When I unlock my device
And Launch Cardano App
And launch Daedalus
And click on "Pair"
Then I am asked to export the public key
And I am brought to the landing page for the correct wallet

Scenario 3: Unlock device before opening Daedalus.But dont launch Cardano App until after clicking Pair
Given that Ledger Device is plugged in and unlocked
And I Open Daedalus
And I Click on "Pair"
When I Launch Cardano App
Then I am asked to confirm export the key
And I am brought to the landing page for the correct wallet

Scenario 4: Open Daedalus first.Plug in device and unlock.Select Cardano app after clicking pair
Given that I have Daedalus installed and open
And I Click on "Pair"
And I Plug in the Ledger Device
And Unlock it
When I Launch the Cardano App
Then I am asked to export the public key
And I am brought to the landing page for the correct wallet

Scenario 5 : Reconnecting device
Given that I have a Ledger wallet already paired and loaded
And I disconnect my device
When I reconnect my device
And I click Pair
And I unlock my device
Then I will be asked to export my public key
And exporting public key will be a success
And I will be redirected to the corresponding wallet summary screen

Scenario 6 : Redelegation
Given that I have a Ledger wallet already paired and loaded
And I wish to redelegate my wallet
When I select redelegate from the Stake Pools Wallet Menu
And I follow the Delegate Wallet wizard instructions
Then I am prompted to export my public key
And I am asked to confirm the transaction

Scenario 7 : Sending a transaction
Given that I have a Ledger wallet already paired and loaded
When I select to send a transaction
Then I am prompted to export my public key
And I am asked to confirm the transaction

Review Checklist
Basics

[ ] PR has been assigned and has appropriate labels (feature/bug/chore, release-x.x.x)
[ ] PR is updated to the most recent version of the target branch (and there are no conflicts)
[ ] PR has a good description that summarizes all changes
[ ] PR has default-sized Daedalus window screenshots or animated GIFs of important UI changes:

[ ] In English
[ ] In Japanese

[ ] CHANGELOG entry has been added to the top of the appropriate section (Features, Fixes, Chores) and is linked to the correct PR on GitHub
[ ] Automated tests: All acceptance and unit tests are passing (yarn test)
[ ] Manual tests (minimum tests should cover newly added feature/fix): App works correctly in development build (yarn dev)
[ ] Manual tests (minimum tests should cover newly added feature/fix): App works correctly in production build (yarn package / CI builds)
[ ] There are no flow errors or warnings (yarn flow:test)
[ ] There are no lint errors or warnings (yarn lint)
[ ] There are no prettier errors or warnings (yarn prettier:check)
[ ] There are no missing translations (running yarn manage:translations produces no changes)
[ ] Text changes are proofread and approved (Jane Wild / Amy Reeve)
[ ] Japanese text changes are proofread and approved (Junko Oda)
[ ] UI changes look good in all themes (Alexander Rukin)
[ ] Storybook works and no stories are broken (yarn storybook)
[ ] In case of dependency changes yarn.lock file is updated

Code Quality

[ ] Important parts of the code are properly commented and documented
[ ] Code is properly typed with flow
[ ] React components are split-up enough to avoid unnecessary re-renderings
[ ] Any code that only works in main process is neatly separated from components

Testing

[ ] New feature/change is covered by acceptance tests
[ ] New feature/change is manually tested and approved by QA team
[ ] All existing acceptance tests are still up-to-date
[ ] New feature/change is covered by Daedalus Testing scenario
[ ] All existing Daedalus Testing scenarios are still up-to-date

After Review

[ ] Merge the PR
[ ] Delete the source branch
[ ] Move the ticket to done column on the YouTrack board
[ ] Update Slack QA thread by marking it with a green checkmark

@DominikGuzei Can you help us here with Netlify. It same issue like here (where @nikolaglumac mentioned you) https://github.com/input-output-hk/daedalus/pull/2284#issuecomment-756008387
Netlify Link: https://app.netlify.com/sites/daedalus-storybook/deploys/602e9391d4da9300082c5ecd

Hi @Tomislav. Tested on Mainnet 16656
There is an issue with public key export on Ledger Nano S. It is throwing a cannot export extended public key error. See Screenshot
On Linux the button for Daedalus diagnostic screen is greyed out and disabled. I checked latest develop and the button is working fine so it leads me to believe this PR has somehow affected it

On Linux the button for Daedalus diagnostic screen is greyed out and disabled. I checked latest develop and the button is working fine so it leads me to believe this PR has somehow affected it

Where do we have a "button" for the Diagnostics screen? 🤔
A screenshot and more context would help here @ManusMcCole ...

@ManusMcCole Can you please describe "Export key" steps with more details?
Do you have just one device connected to the machine? When you conencted that devic (before starting Daedalus, before Pair, when Pair started)? Please add more details

Hi @nikolaglumac. Here is a screenshot.For more context I was trying to open the Diagnostics screen so I could transplant the chain folder cause on a Linux VM it takes a long long time otherwise

P.S. @ManusMcCole Can you reproduce this on the latest stable build (from develop branch) too?

Ok we are just missing @ManusMcCole approval...

Hi @tomislavhoracek.I could not reproduce the most recent issue with Ledger S on the newest develop build

Tested again on newest build 16666

@ManusMcCole Can you please describe "Export key" steps with more details?
Do you have just one device connected to the machine? When you connected that device (before starting Daedalus, before Pair, when Pair started), Platform? Please add more details !!!

Hi Tomislav. Only one device connected to the machine at any one time. So ive found that if i do a transaction with the Ledger and then plug it out and plug in Trezor and then I get this error. Daedalus says launch Cardano app so its expecting a Ledger device. See Screenshot. Error is different this time. Ill keep investigating now

@ManusMcCole @tomislavhoracek please focus on this one today. We need to fix the issues and merge this PR ASAP. Thanks 🙏

Hi @tomislavhoracek. I reproduced the "Cannot export extended public key error" now on Ledger S and on Mac only. See screenshot. The steps to reproduce are.1. Daedalus is closed.2.Plugin and unlock ledger device (launch Cardano app is optional error appears in both cases).3 Open Daedalus 4. Attempt to create transaction. Daedalus gets stuck at the "Export Public key on your device message".Ledger device says "Waiting for commands...". Console error says "Cannot export extended public key error"
https://drive.google.com/file/d/1sGamS7s1pTBjCqgJl7lW2UiBVpLtWEPW/view?usp=sharing


CRU Projects for Campus Partner

Campus partner should have a form where they can submit new projects. They should be able to add a new project to the database, as well as edit a pre-existing project.

Campus partner login, view projects, and edit projects are working fine.

Profile edit functionality is fine. But, edit or add projects needs UI improvement.

Login page for campus could be redirected to dashboard in upcoming sprints.


Optional dependencies with Yarn issue

I have a project which use the template, I am not able or don't want to build for FREEBSD,
But magically the package.json on npm contains optionalDependencieswhich contains the FreeBDS one.
With nom , no problem, but with yarn it will lead to an error not found and the main package will not be installed
You can try with yarn install @avahq/resampler-native-rubato
Side question but why does. it even try to download it ? Does this means that it will download all binaries in my project ? (I supposed It was more like prebuild which get the only binary needed)

remove this dir https://github.com/AvaHQ/resampler-native-rubato/tree/main/npm/freebsd-x64
and this line https://github.com/AvaHQ/resampler-native-rubato/blob/main/package.json#L17


Update CognitiveService ComputerVision API to V2

This checklist is used to make sure that common issues in a pull request are addressed. This will expedite the process of getting your pull request merged and avoid extra work on your part to fix issues discovered during the review process.
PR information

[x] The title of the PR is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For information on cleaning up the commits in your pull request, see this page.
[x] Except for special cases involving multiple contributors, the PR is started from a fork of the main repository, not a branch.
[x] If applicable, the PR references the bug/issue that it fixes.
[x] Swagger files are correctly named (e.g. the api-version in the path should match the api-version in the spec).

Quality of Swagger

[x] I have read the contribution guidelines.
[x] My spec meets the review criteria:

[x] The spec conforms to the Swagger 2.0 specification.
[x] The spec follows the guidelines described in the Swagger checklist document.
[x] Validation tools were run on swagger spec(s) and have all been fixed in this PR.

Automation for azure-libraries-for-java

Automation for azure-sdk-for-python

Automation for azure-sdk-for-node

Automation for azure-sdk-for-ruby

Automation for azure-sdk-for-go

@cthrash could you please have a look at the model validation failures here?

Looks like the travis issue started with the build prior to mine?

Yeah not sure what the deal is, I'm following up with the folks that own this tooling.


NCCO

Thank you. JWT  key works perfectly.
I get a call, but I can not hear the sound.
My   NCCO - https://riddler.serverpipe.com/api/apinexmo/nexmovoice
I'm getting a request for this address, but there is no sound.
Thanks

Hi @pmaxim , it looks like we can't see what your NCCO looks like, your link looks dead.
Can you please provide another link so we can investigate further ?
Thanks,
Dylan

Any update on this @pmaxim?

Closed due to inactivity


[FR] - Improve InitiatorOnly mode for wallets

External
Area
Networking/Ledger
Describe the feature you'd like
During high network utilization, the mempools of various block producers on the network are much more likely to diverge in their contents. As such, when a chain rollback happens in an InitiatorOnly node such as Daedalus or various Lite Wallet backend nodes, this is a highly disruptive and destructive process.
First block arrives and is immediately applied to the tip. All of this blocks transactions are immediately flushed from the mempool. Then let's say a slot-battle block arrives and replaces the first block. This block has very different contents, but we've already flushed a lot of the transactions in the previous block out of our mempool. If they hadn't yet made it to many other nodes (very likely during network congestion), it's likely these transactions will be dropped and lost forever.
I would like to suggest a new config parameter that can be applied to InitiatorOnly nodes called tipDelay. This value when set would wait a set amount of time before applying arriving blocks to the tip. The idea is that any forks would then be sorted out ahead of time and not used in a destructive manner against the mempool. With sufficient tipDelay, the mempool would never see any forks and we get more likely transaction delivery to the chain.
An InitiatorOnly node doesn't participate in consensus so it doesn't really matter if it runs a bit behind other nodes. This node would then be optimized not for block propagation, but for improving the delivery of transactions to the chain.
Describe alternatives you've considered
In Phyrhose, the current work-around is to track every transaction submitted to the node and then re-apply just those transactions that got stomped out of the mempool in the event of a block rollback. While this works and allows for guaranteed transaction delivery, it's not an optimal solution. During heavy chain utilization, I'm sometimes re-applying 3-4mb worth of transactions back into the mempool. By doing a tipDelay solution, the mempool will see much fewer rollback events and thus only remove transactions from the mempool when they're slightly more stable and likely to "stick" to the chain.
Image of re-applying multiple megabytes of dropped transactions during a block rollback event.

@mmahut So what you are suggesting (if I'm not mistaken) is to have a flag on items in the mempool so that they can be marked as existing in a block. Then they can be skipped over if making a new block on a block producer. However, if a rollback occurs and a tx is no longer on the chain, that flag can be cleared. Then eventually, after some number of blocks (perhaps k), they can be removed.
I think in general what this FR is asking for is:
When I submit a transaction to the node, I expect that it will eventually be included in the chain as long as it is valid. As an end user, I think this expectation is a reasonable one.


Can't install

Hi,
Trying to install on ubuntu 14.04:
Downloading/unpacking rma
  Downloading rma-0.1.5.tar.gz
  Running setup.py (path:/tmp/pip_build_root/rma/setup.py) egg_info for package rma
    Traceback (most recent call last):
      File "<string>", line 17, in <module>
      File "/tmp/pip_build_root/rma/setup.py", line 47
        setup(**sdict, install_requires=['redis', 'tabulate', 'tqdm', 'msgpack-python'])
                     ^
    SyntaxError: invalid syntax
    Complete output from command python setup.py egg_info:
    Traceback (most recent call last):

  File "<string>", line 17, in <module>

  File "/tmp/pip_build_root/rma/setup.py", line 47

    setup(**sdict, install_requires=['redis', 'tabulate', 'tqdm', 'msgpack-python'])

                 ^

SyntaxError: invalid syntax

----------------------------------------

Could you provide your pip and python version?

# python -V
Python 3.4.3
# pip3 -V
pip 1.5.4 from /usr/lib/python3/dist-packages (python 3.4)

It also happens when i manually clone the repo

This is invalid syntax on any Python version; you can't put **kwargs keyword argument expansion in front of other keyword arguments. Swap the two:
setup(install_requires=['redis', 'tabulate', 'tqdm', 'msgpack-python'], **sdict)

@mjpieters tnx for help! @shaharmor could you try with 0.1.6 ?

Now it requests for the msgpack module (Which isn't listed as a requirement).
After installing msgpack-python manually it asked for tqdm (Also not listed..)
Then it asked for tabulate...
and only then it finished successfully.
Can't you make it that it will automatically install it?

@mjpieters could i ask you help again? Now in setup args install_requires listed all requred components - 'redis', 'tabulate', 'tqdm', 'msgpack-python'. Also i see it in rma.egg-info requires.txt. But they do not install. Why?

@shaharmor i move all **kwargs as args. please test it with 0.1.7.

The install_requires looks correct to me but take into account only tools like easy_install and pip will go and fetch those requirements. If you run python setup.py install then you'll need to install the requirements manually.

Hrm, I can confirm the package still doesn't install with pip either.

Ah, you do from rma import __version__ and that then loads rma/__init__.py, which starts loading everything else, including from rma.application import RmaApplication which then imports from rma.scanner import Scanner and scanner.py has import msgpack.
Don't import your package in setup.py.

You could look at What is the correct way to share package version with setup.py and the package? for an alternative method of sharing the version between setup.py and the package.

Tnx a lot @mjpieters !


question about discriminator success rate

Thanks for sharing the training/testing code first!
I've been tryinng to train the ExplorableSR on some other dataset. I found that the generator_step flag is always False after trainning for a few hours. After some debuging, I think it is due to the discriminator's success rate is lower than the default 0.9 threashold.
I am wondering how many steps are usually needed to get the discriminator reach the 0.9 success rate? It would be helpful to provide a pretrained discriminator to speed up the training process.

Thank you for your feedback. Stabilizing training of the Explorable SR GAN is a bit challenging, and we are currently still working to improve this. In the meantime, I suggest you try training without the discriminator verification mechanism, by simply commenting out this line.
Following your suggestion, I also added a link to the pre-trained discriminator corresponding to the provided pre-trained explorable SR model, so that you can initialize both generator and discriminator models to the provided pre-trained ones, and fine-tune them from there using your own dataset.
I hope this can help.

Thanks for providing the pre-trained discriminator!
When loadding the provided pre-trained discriminator, I found there is some state_dict mismatch. The provided model has such keys:
(Pdb) loaded_state_dict.keys() odict_keys(['features.0.weight', 'features.0.bias', 'features.2.weight', 'features.2.bias', 'features.3.weight', 'features.3.bias', 'features.3.running_mean', 'features.3.running_var', 'features.3.num_batches_tracked', 'features.5.weight', 'features.5.bias', 'features.6.weight', 'features.6.bias', 'features.6.running_mean', 'features.6.running_var', 'features.6.num_batches_tracked', 'features.8.weight', 'features.8.bias', 'features.9.weight', 'features.9.bias', 'features.9.running_mean', 'features.9.running_var', 'features.9.num_batches_tracked', 'features.11.weight', 'features.11.bias', 'features.12.weight', 'features.12.bias', 'features.12.running_mean', 'features.12.running_var', 'features.12.num_batches_tracked', 'features.14.weight', 'features.14.bias', 'features.15.weight', 'features.15.bias', 'features.15.running_mean', 'features.15.running_var', 'features.15.num_batches_tracked', 'features.17.weight', 'features.17.bias', 'features.18.weight', 'features.18.bias', 'features.18.running_mean', 'features.18.running_var', 'features.18.num_batches_tracked', 'features.20.weight', 'features.20.bias', 'features.21.weight', 'features.21.bias', 'features.21.running_mean', 'features.21.running_var', 'features.21.num_batches_tracked', 'features.23.weight', 'features.23.bias', 'features.24.weight', 'features.24.bias', 'features.24.running_mean', 'features.24.running_var', 'features.24.num_batches_tracked', 'features.26.weight', 'features.26.bias', 'features.27.weight', 'features.27.bias', 'features.27.running_mean', 'features.27.running_var', 'features.27.num_batches_tracked', 'classifier.0.weight', 'classifier.0.bias', 'classifier.2.weight', 'classifier.2.bias'])
while the defined netD has such keys:
(Pdb) netD.state_dict().keys() odict_keys(['features.0.weight', 'features.0.bias', 'features.2.weight', 'features.2.bias', 'features.3.weight', 'features.3.bias', 'features.3.running_mean', 'features.3.running_var', 'features.3.num_batches_tracked', 'features.5.weight', 'features.5.bias', 'features.6.weight', 'features.6.bias', 'features.6.running_mean', 'features.6.running_var', 'features.6.num_batches_tracked', 'features.8.weight', 'features.8.bias', 'features.9.weight', 'features.9.bias', 'features.9.running_mean', 'features.9.running_var', 'features.9.num_batches_tracked', 'features.11.weight', 'features.11.bias', 'features.12.weight', 'features.12.bias', 'features.12.running_mean', 'features.12.running_var', 'features.12.num_batches_tracked', 'features.14.weight', 'features.14.bias', 'features.15.weight', 'features.15.bias', 'features.15.running_mean', 'features.15.running_var', 'features.15.num_batches_tracked', 'classifier.0.0.weight', 'classifier.0.0.bias', 'classifier.0.1.weight', 'classifier.0.1.bias', 'classifier.0.1.running_mean', 'classifier.0.1.running_var', 'classifier.0.1.num_batches_tracked', 'classifier.2.0.weight', 'classifier.2.0.bias', 'classifier.2.1.weight', 'classifier.2.1.bias', 'classifier.2.1.running_mean', 'classifier.2.1.running_var', 'classifier.2.1.num_batches_tracked'])
It seems that there are model layers(features) and some different calssification heads in the provided model. I am using the provided config for netD, the opt_net looks like
{'which_model_D': 'discriminator_vgg_128', 'relativistic': 0, 'decomposed_input': 0, 'pre_clipping': 0, 'add_quantization_noise': 0, 'norm_type': 'batch', 'act_type': 'leakyrelu', 'mode': 'CNA', 'n_layers': 6, 'in_nc': 3, 'nf': 64}
should I use some other config for netD, or there is a keys mapping from the provided netD to the defined netD?

Thank you for letting me know about this problem. I'm currently debugging it, and will let you know once this problem is fixed.

I found the problem and pushed the fix. There was one line in the code that needed to be commented, and the number of discriminator layers in the configuration file had to be set back to 10. Please let me know if you run into additional problems.

Hello, can you point me to your pushed fix? I did a scan of your recent commit and did not find something related. thanks.

hmm, the netD loading seems working now. Thx!


Any plan to integrate MCMC strategy into NeRFStudio?

Hii, is there any plan to integrate MCMC strategy into NeRFStudio's splatfacto? I've tried myself but it seems not working. I essentially add MCMCStrategy.step_post_backward link in trainer.py between self.grad_scaler.scale(loss).backward() link and self.optimizers.optimizer_scaler_step_some(self.grad_scaler, needs_step) link. Additionally, I commented out the after_train and refinement_after operations.
However, the results are pretty weird. I understand MCMC is working well with gsplat code base so I am wondering if there is any plan to integrate MCMC into nerfstudio's code base?
Thanks!

This PR from @jb-ye is relevant: https://github.com/nerfstudio-project/nerfstudio/pull/3376
@kerrj also has a similar branch here for moving to the strategy API: https://github.com/nerfstudio-project/nerfstudio/tree/splatfacto-refactor
My impression of the likely outcome is that MCMC will be added as an option once we move over to gsplat's strategy API, but it won't be enabled by default.


Warnings about lookup failure spamming the log

Since https://github.com/bosagora/agora/pull/3028 this is constantly triggering: https://github.com/bosagora/agora/blob/f061636d4643f8a7054479792ce35d2c9247dedc/source/agora/network/Manager.d#L566

Fixed: https://github.com/bosagora/agora/pull/3053


Add virtual environments folders to gitignore

Since it is best practice to create virtual environments for each python project. I've thought that it would be nice to have it on gitignore.

Codecov Report

Merging #251 into master will increase coverage by 0.00%.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #251   +/-   ##
=======================================
  Coverage   98.44%   98.44%
=======================================
  Files           7        7
  Lines         385      387    +2
  Branches       39       39
=======================================
+ Hits          379      381    +2
  Misses          3        3
  Partials        3        3

Impacted Files
Coverage Δ

bayes_opt/bayesian_optimization.py
99.03% <0.00%> (+0.01%)
:arrow_up:

Continue to review full report at Codecov.

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


Cookies never returned in response object

I have been trying to get the cookies from a response, as your docs suggest (http://docs.python-requests.org/en/master/user/quickstart/#cookies [first example]), but it does not seem to be working.
When I dug through the sessions.py (https://github.com/kennethreitz/requests/blob/master/requests/sessions.py#L655), it looks like cookies are extracted to the cookie jar on the Session object, but never set on the response.
I had to do the following to get my intended behavior (which is probably not entirely correct, since it will copy all cookies on the Session, not just the ones from the most recent request):
>>> url = 'http://example.com/some/cookie/setting/url'
>>> s = requests.Session()
>>> r = s.get(url)
>>> r.cookies = s.cookies

>>> r.cookies['example_cookie_name']
'example_cookie_value'

Hey @dubhunter, I'm actually unable to reproduce this in either Python 2 or 3 for Requests 2.13.0.
Could you please provide the version of Requests you're using and verify for me that the script below returns <RequestsCookieJar[]>? No need to provide the contents if the RequestsCookieJar is populated.
import requests

s = requests.Session()
resp = s.get('https://github.com/kennethreitz/requests')
print(resp.cookies)

@nateprewitt You are correct. After further digging, it looks like I am having trouble because the CookiePolicy set on the Session object is not copied to Response and there is no way to set the CookiePolicy for the Response.
We can close this issue and open another, or reword this one.
Thoughts?

I was able to work around this with the following code in my adapter:
    def build_response(self, req, resp):
        response = super().build_response(req, resp)
        response.cookies = cookiejar_from_dict({})
        response.cookies.set_policy(self.cookie_policy)
        extract_cookies_to_jar(response.cookies, req, resp)
        return response

    @property
    def cookie_policy(self):
        return CookiePolicy()

I think the cookie policy concern is fundamentally part of #3669. However, you may find a better workaround is to patch requests.cookies.requests_cookie_policy to return whatever it is you want. That function is used to initialize all RequestsCookieJar objects.

It seems like this may be more closely related to #3416 and #3463. @dubhunter, would you mind clarifying a bit further your desired use case/behaviour here? It seems like you want a Response specific cookie policy propagated by the Session?
#3416 was asking for the user to be able to persist a Session-wide policy that would scope what cookies were allowed into the Session CookieJar. While this would still allow the user to inspect ALL cookies returned on an individual response, it would only carry policy-approved cookies to the next request, which still seems reasonable to me. Is this anywhere inline with your use case?

Yes, #3416 seems to cover my use case. It would also be nice to be able to override RequestsCookieJar (since the cookie policy is never consulted when sending cookies), but that has lower utility.

@Lukasa @sigmavirus24, do either of you have any further thoughts on persisting cookie policies? I'm still not ecstatic about the implementation in #3463 but it seems a step in the right direction for cases like this. I only ask since #3998 will make doing this more difficult.
Either way we can probably close this out as a dupe of #3416.

Ok, let's close this as a dupe. I think persisting cookie policies would solve the problem in #3998, so I'm open to just preferring doing that.


Questions about 'Minimum search range RT/Mobility'

Basic information

My operating system and version: windowns11
My mzmine version: 4.2

What happened

Previously, we set it to 0.09 in order to find suitable parameters, but other potential issues arose. This time, we found that the 'Arginine' that should have existed disappeared (pos, mz=175.1188). After searching for the reason, it was found that changing this parameter back to 0.05 could correctly identify it again（Other parameters remain unchanged）.
Can you explain some of the working principles of this setting? Thank you


Unable to install on a windows machine

I'm unsure if this is an issue in selenium-standalone or a setting in web component tester but i'm seeing this issue:
node_modules\selenium-standalone\lib\compute-download-urls.js:252
    throw new Error('Invalid Microsoft Edge Web Driver version see https://developer.microsoft.com/en-us/microsoft-edge/tools/webdriver/#downloads')

Can't help but feel like a coincidence that this happens around time new edge builds happening perhaps something changed?
I don't even test on edge just want to get through this error.

"web-component-tester": "^6.9.2"

delete package-lock.json went away shrug


Bug in the readme

Hello,
Thanks for providing this repo.
I wanted to mention that there is a bug in the readme file for the VQGAN example. A working version would be this:

import torch
from torchvision import transforms
from PIL import Image
from muse import MaskGitVQGAN
import numpy as np # <--- added

torch.set_grad_enabled(False) # <--- added

# Load the pre-trained vq model from the hub
vq_model = MaskGitVQGAN.from_pretrained("openMUSE/maskgit-vqgan-imagenet-f16-256")

# encode and decode images using
encode_transform = transforms.Compose( # <--- fixed
    [
        transforms.Resize(256, interpolation=transforms.InterpolationMode.BILINEAR),
        transforms.CenterCrop(256),
        transforms.ToTensor(),
    ]
)
image = Image.open("/content/ILSVRC2012_val_00000028.JPEG") #
pixel_values = encode_transform(image).unsqueeze(0)
image_tokens, _ = vq_model.encode(pixel_values)
rec_image = vq_model.decode(image_tokens)

# Convert to PIL images
rec_image = 2.0 * rec_image - 1.0
rec_image = torch.clamp(rec_image, -1.0, 1.0)
rec_image = (rec_image + 1.0) / 2.0
rec_image *= 255.0
rec_image = rec_image.permute(0, 2, 3, 1).cpu().numpy().astype(np.uint8)
pil_images = [Image.fromarray(image) for image in rec_image]

@Msadat97 Sorry for the mistake. Can you try out this colab? I recently made it for testing purposes for laion

yes the colab works fine


HA Integration: Scenes aren't loaded as entities

Hi I have quite a few lights and scenes and I was having issues with my hue Hub in Home Assistant. I clicked "Reload" in the integrations page and it gave a general error. I looked at the logs and I got a lot of event retry requests. So I removed the integration and then re-added it to Home Assistant.
However, now the Scenes that are configured in the Hue mobile app, are not being imported into Home Assistant like they used to and I'm noticing lots of:
2024-02-19 09:09:48.211 DEBUG (MainThread) [aiohue.v2[<IP_ADDRESS>].events] Connected to EventStream
2024-02-19 09:09:48.212 DEBUG (MainThread) [aiohue.v2[<IP_ADDRESS>].events] Disconnected from EventStream - Reconnect will be attempted in 2 seconds
2024-02-19 09:09:48.649 DEBUG (MainThread) [aiohue.v2[<IP_ADDRESS>].events] Connected to EventStream
2024-02-19 09:09:49.755 DEBUG (MainThread) [aiohue.v2[<IP_ADDRESS>]] Got 503 or 429 error from Hue bridge, retry request in 3.5 seconds

ha-reload-ha-integration-logs.txt
HA Version Info:
Core 2024.2.1
Frontend 20240207.1
Is it possible the back off isn't long enough? I have a second Hue Bridge with a whole other setup, and installing that fresh in HA also does not make the Scenes available as entities in Home Assistant anymore.

It seems after restarting Home Assistant a few times, the scene entities showed up. Disregard this issue, thanks for your hard work.


Add an examples folder

It seems it is a good practice (?) https://github.com/golang-standards/project-layout/tree/master/examples

It's been on our radar, just haven't had time to yet. I'm hoping to take some of what comes out of QA and use those to populate it.

It's been added and is in progress of being fleshed out: https://github.com/suse-edge/edge-image-builder/tree/main/examples


Ruby 2.7 Compatibility

Remove "warning: The last argument is used as keyword parameters"

Coverage increased (+0.7%) to 100.0% when pulling e4bf547b50e3f16c02afa8188c7296bcfac4b220 on ruby-2.7-keyword-arg-warning into 24ce93dc197b489a01138a699fdf9d63260ec28a on master.


Open Source?

Was the intention to release the project as open source? If it was, it should state under which license it was released. The project looks interesting, shouldn't be too far away from becoming fully decentralized and viable.

It is my intention yeah. Thanks for reminding me I should take care of this. I also do think that with a little bit of polish it could become much more viable, unfortunately I don't have much time for this now. I should mostly make more docs in order to allow some motivated people to participate more easily.


#team-106-the-kingsnakes

Hey Chris!
You've done a wonderful presentation today & your project is looking great so far!

great job on fixing the issues from yesterday
your code & logic is well thought out. Nice job on this!
you can probably take the condition in the if statement used in checkWin function and store it in a variable instead. It would be a little more readable.
since you've separated your DOM logic and application logic into their own files, you can further separate some code in your application logic/file that might belong in the DOM file (eventListeners, etc) and vice versa.

Your project also looks good for submission! All the best with this. 😃👍🏽

Thanks Losa et al! 😃


To use Florence subset mesh to generate ensemble members (instead of current 250 m)

@wpringle
Please provide the setup to @zacharyburnettNOAA .
Thanks

I did on the email.


MODFQMMGR-247 & MODFQMMGR-248: Add complex entity types

Purpose
MODFQMMGR-247 & MODFQMMGR-248: Add complex entity types and GROUP BY logic needed to support them

No translation changes?
Since the translations aren't working at all for complex ones, I figured that could be handled in the scope of the translation ticket. That way we can at least get these in and tested during this sprint.


jsonify no longer causes Content-Length to be set on returned Response object

I am not sure if this was an intentional change, as it is only triggered by jsonify by passing an iterable to flask.wrappers.Response/werkzeug.wrappers.Response.
In commit https://github.com/pallets/flask/commit/d9402fc0c03353b6d2659297682e578d695476ce a newline was added to the json response, as per a request from the httpbin project.
When werkzeug.wrappers.Response is passed a non text_type iterable, the self.set_data codepath, responsible for setting the Content-Length header, is not used anymore.
Ironcially this breaks httpbins "response-headers" route that uses the response object in a somewhat
creative way and breaks while trying to access response.headers['Content-Length']
If this considered a backwards incompatible change that might need to be undone, or is this considered new and intended behaviour?

This is unintended behavior.

From a first look the best fix is to explicitly set content-length in jsonify.

I do not have a good grasp of the Encoding Implications, but would it not help to just concat the newline onto the generated json string so we trigger the intended behaviour in werkzeug.wrappers.Response?
After all setting somewhat "special" headers might introduce other undesired side-effects over time.

I'm not sure about the performance implications of concatenating strings like that. We already are O(n) (at least!), but I think somebody would report a performance regression because of this.

foo = json.dumps(...)
...
rv.headers['Content-length'] = len(foo) + 1

something like this could work (you know \n is 1 byte and i'd expect len() on a string to be O(1) in python)

The point by @mbra was that if the behavior of Response changed wrt to the content-length header, jsonify wouldn't change.
Also we need to be careful here: len(u"ö") != len(u"ö".encode("utf-8")).

ah damn.. json dumps still returns a unicode string at least in python3. not sure if it'll contain any non-ascii chars though. AFAIK it escapes them all
but yeah, might be a bit fragile nonetheless

Well, I did not consider the performance penalty, the reallocations from the string concatenation could really be harmful, for large responses.
Perhaps calling werkzeug.wrappers.ResponseBase.calculate_content_length and setting the header is a good enough compromise, after all the real logic is kept in werkzeug. But I do not
know how often the additional _ensure_sequence Codepath would be, but at a first glance this
should not be as much of an issue as concating a large json object and a newline.
Also, the bad feeling I have about setting the Content-Length is that I do not know if and where a decision if made to use Transfer-Encoding: chunked and how that plays with manually setting Content-Length.

Mhh, after reading up on chunked Transfer and WSGI, this seems not to be an issue...

Chunked encoding doesn't make much sense since we're not streaming. Content-Length is required by some clients, if it is suddenly missing I'm sure at least some code will break.

The only alternative I could see involves StringIO, and calling json.dump on it.
@mitsuhiko Do you think this is breakage?

Mhh, perhaps this should be handled by werkzeug. The docs for automatically_set_content_length says: "Should this response object automatically set the content-length header if possible? This is true by default."
Perhaps I am missing something again, but calculating from a tuple should always be possible, right?
I know this just moves the same problem down the callchain but at least the Content-Length calculation is kept in one place.

Not sure when this got fixed, but I can't reproduce this. Examining the response from a client shows the correct content length. (headers['Content-Length'] isn't available, but it's eventually set.)


[HTTP] OCSP Stapling

OCSP stapling is feature that needs to be there in server connector. This is mainly needed for performance optimization of SSL handshake.

Closing this issue as this has been implemented.


Bump through2 and tape

The latest stable through2 uses Streams3 (similar to the latest node/io.js).

Thanks! Published as v2.0.0

No problem! Thanks for this awesome module.


Don't return HTML in the API

If a request comes into something on the API its asking for json, don't return HTML on an error. Example:
>>> res = requests.get('https://addons-dev.allizom.org/en-US/firefox/api/v3/does-not-exist', headers={'Accept': 'application/json'})
send: 'GET /en-US/firefox/api/v3/does-not-exist HTTP/1.1\r\nHost: addons-dev.allizom.org\r\nConnection: keep-alive\r\nAccept-Encoding: gzip, deflate\r\nAccept: application/json\r\nUser-Agent: python-requests/2.5.3 CPython/2.7.10 Darwin/14.5.0\r\n\r\n'
reply: 'HTTP/1.1 404 NOT FOUND\r\n'
header: Content-Encoding: gzip
header: Content-Security-Policy-Report-Only: script-src 'self' https://www.google.com https://www.paypalobjects.com https://ssl.google-analytics.com https://addons-dev-cdn.allizom.org/user-media; default-src * data:; style-src * 'unsafe-inline'; frame-src https://ssl.google-analytics.com https://sandbox.paypal.com; object-src 'none'; report-uri /services/csp/report
header: Content-Type: text/xml; charset=utf-8
header: Date: Sun, 01 Nov 2015 17:41:53 GMT
header: Server: nginx
header: Strict-Transport-Security: max-age=31536000
header: Vary: Accept-Encoding
header: Vary: X-Mobile, User-Agent
header: X-Frame-Options: DENY
header: Content-Length: 90
header: Connection: keep-alive
>>> res.content
'<?xml version="1.0" encoding="utf-8" ?>\n      <error>Not Found</error>\n  '

I'm requesting JSON and get back XML.

So I found out what causes this, but didn't get a satisfactory answer.
handle500 is set to this: https://github.com/mozilla/olympia/blob/master/apps/amo/views.py#L112-L117
Which looks to see if it starts with /api/ and then bounces you over to /apps/api/ which returns XML because the main API on olympia is version check and block list. Change this at your peril.
So the next obvious thing to do is override the exception handler in DRF: http://www.django-rest-framework.org/api-guide/exceptions/, but if you do that and its a 500, nothing gets sent to sentry. So then if you try to add in sentry logging in the exception handler, you'll note that you don't actually get the request.
So then I thought I'd just change the handerl500 in /apps/api/ and we can do that by simply checking request.path_info.startswith('/api/v3/'), but then ideally you want to create a Response that adapts to the Accept headers sent by the client. Which a standard rest framework Response won't do.
At that point I headed for a beer. There's probably something simple here I'm missing.

Duplicate of #2735, but it looks like @diox is working on that one 👍

Ah yeah, had not seen your issue. I'll look into playing with the DRF exception handler, I did that in Marketplace and had the request working : https://github.com/mozilla/zamboni/blob/master/mkt/api/exceptions.py#L38


not an issue but a question

i wonder what the best way is to ask a question? I do not have an issue but more some questions about API usage.
i would like to use the debug functions from avr8protocol.
like step,run, etc.  but i have no idea how to setup a session so i can use these commands.

Hi @mcsmark,
An interesting and relevant question!  First point is a matter of scope - this library contains a set of protocol definitions, and avr8protocol can be used for a set of AVR device/interfaces - so its a matter of knowing not only what functions to call to setup a session, but also what content must be sent to the debugger to configure a session for a given device.  Since this library is the raw set of protocols, you have to start to look one layer up: pymcuprog.
However, pymcuprog is primarily targeted towards AVR UPDI devices, and the other device families are only partially supported for experimental/test purposes - are you planning to use an UPDI device?
If so - you can install the latest beta pymcuprog from the test.pypi.org, and use the newly-added debug functions, for example: (note - this does not program the flash)
from pyedbglib.hidtransport.hidtransportfactory import hid_transport
from pymcuprog.avrdebugger import AvrDebugger
logging.basicConfig(format="%(levelname)s: %(message)s", level=logging.INFO)

transport = hid_transport()
transport.connect()

avr = AvrDebugger(transport)

avr.setup_session("ATmega4808")
avr.start_debugging()
pc = avr.program_counter_read()
avr.step()
pc = avr.program_counter_read()
avr.stop_debugging()

Oh... I can try to replicate a similar session here - which Snap debugger FW version are you using?
(reported easily by "pymcuprog ping")

Connecting to anything possible
Connected to MPLAB Snap ICD CMSIS-DAP from Microchip Technology Incorpora
serial number BUR190972701)
Debugger firmware version 1.9.223
Debugger hardware revision 1
Pinging device...
Ping response: 1E970A
it should be the latest since it was updated with firmware just few days ago using mplax-x

OK, that should be fine.  Two things to try next:

update to latest pyedbglib (2.20) from test.pypi:  pip install -i https://test.pypi.org/simple/ pyedbglib
instantiate the debugger in an alternative polling mode: avr = AvrDebugger(transport, use_events_for_run_stop_state=True)

ok, thanks, i will try that.

ok, that fixed the problem. i see some errors when i use stop_debuggin
INFO: Stop debugging session
Exception ignored in: <function HidTransportBase.del at 0x00000000037F1AF0>
Traceback (most recent call last):
File "pyedbglib\hidtransport\hidtransportbase.py", line 60, in del
File "pyedbglib\hidtransport\hidtransportbase.py", line 161, in disconnect
File "pyedbglib\hidtransport\cyhidapi.py", line 116, in hid_disconnect
File "D:\Python38\lib\logging_init_.py", line 1433, in debug
File "D:\Python38\lib\logging_init_.py", line 1699, in isEnabledFor
TypeError: 'NoneType' object is not callable
dont know if that is normal or not. but it works. i am really happy with this :+1:
thanks for your help and support.

Well, thanks for the feedback - we will take a look to see what might cause this.
We hope to release this package update in the next sprint (2 weeks) - of course this module is 'alpha'.

ok, great. i will test all functions, if i find a problem i share it here.

ok so the good news it all works perfect. that is using HID transport. But now there is a new SNAP V2 which does not have a HID driver anymore. A real pity since HID does not need users to install drivers.
I wonder how it will work now? all code examples with hid will not function. V2 uses a bulk transport driver according to microchip.  Is the existing code able to deal with that?  or are any changes planned?
The SNAP v1 identifies as an atmel device with HID. But the V2 identifies as microchip device with a different VID and PID.
On avrfreaks i noticed users are using atmel studio to put in the old firmware so it works in HID mode again.

The Snap firmware you are referring to is the same Snap firmware that has been used to program PIC devices for many years already - now it also has support for AVR devices, so MPLAB no longer switches to an "AVR mode".  To access the HID mode and continue to use pyedbglib, use Studio to switch the tool into that mode.

Aha, so there is no mode anymore and an entire new protocol that can do all. So pymcuprog can also not use this new protocol.
It is a pity that one has to download an install AS7 just to put the programmer into the HID mode.
Or is there some other more simpler way? Also when you put the SNAP into V1 mode, how long will that be usable? I mean it makes sense that microchip only supports the new V2 protocol and all bugs and enhancements will be in that firmware. The older firmware will probably not be updated. I realize that i ask the wrong person since this is the section for EDBG.
But you probably have more insights. It is a pity since it worked so well in HID mode. All EDBG using HID worked without changing any code which is a great benefit. Anyway thanks for the info.

The "V2" protocol put device support details on the outside of the firmware (ie: data driven from the host) so its very different and in no useful way compatible with EDBG.  This means that the Snap firmware remains unchanged when new devices are released.  (FYI avrdude has reverse-engineered this protocol if you are interested in the details.)
nEDBG firmware will continue to be updated with future devices at least for now.

thanks for this info. I found some info about switching from PIC to AVR and vice versa. But that works only with V1. When i understand you, the V2 bulk driver is also supported in AVRDUDE and does not need to switch into HID mode. It will switch modes but without going into HID mode.
I will have a new look at that.

Yes, V2 does not have a mode switch (no longer dual-boot) and also has nothing to do with pyedbglib.

thanks, it is clear to me now :-)


[$500] Profile - Date of birth opens Legal page

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

Version Number: v1.4.46-0
Reproducible in staging?: Y
Reproducible in production?:
Go to Profile.
Click Date of birth.

Expected Result:
Date of birth page will open.
Actual Result:
Legal name page opens instead.
Workaround:
Unknown
Platforms:

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

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

Screenshots/Videos
https://github.com/Expensify/App/assets/115492554/8db45073-3c03-4e8f-9230-091bbd2a851d

View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01576dd3bd294513f4
Upwork Job ID:<PHONE_NUMBER>401383936
Last Price Increase: 2024-03-01

@cristipaval I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors.

Surely an external bug yeah. Let's throw it on, but would be good to know what this was introduced in.
https://github.com/Expensify/App/assets/9133401/3f726097-6dc1-4dd2-be3d-578bfcb14870

This is being fixed in https://github.com/Expensify/App/pull/37592

@ishpaul777 that looks like loom

haha, I started using https://www.screen.studio @ishpaul777! Glad to know this is being fixed. Just linked these. Gonna stick Reviewing on it.


Added resolutions for ejs and async

also added missing license field to Infra
package.json
tested successfully

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

tested :+1: