net: LookupIP("doesnotexist.domain") returns "server misbehaving" when resolv.conf contains search lists

The following test program returns "lookup doesnotexist.domain: no such host" when run with https://storage.googleapis.com/golang/go1.4.2.linux-amd64.tar.gz but returns "lookup doesnotexist.domain on <IP_ADDRESS>:53: server misbehaving" with https://storage.googleapis.com/golang/go1.5.linux-amd64.tar.gz and https://storage.googleapis.com/golang/go1.5.1.linux-amd64.tar.gz
package main

import "net"
import "fmt"

func main() {
  _, err := net.LookupIP("doesnotexist.domain")
  fmt.Println(err)
}

When I change resolv.conf to use nameserver <IP_ADDRESS> the output is correct. Apparently something has changed in Go 1.5 that prevents it from understanding the reply from our internal DNS server.
nslookup does not have a problem:
> nslookup doesnotexist.domain <IP_ADDRESS>
Server:         <IP_ADDRESS>
Address:        <IP_ADDRESS>#53

** server can't find doesnotexist.domain: NXDOMAIN

> nslookup doesnotexist.domain <IP_ADDRESS>
Server:         <IP_ADDRESS>
Address:        <IP_ADDRESS>#53

** server can't find doesnotexist.domain: NXDOMAIN

dig also has no problem
> dig @<IP_ADDRESS> doesnotexist.domain
; <<>> DiG 9.9.5-3ubuntu0.4-Ubuntu <<>> @<IP_ADDRESS> doesnotexist.domain
; (1 server found)
;; global options: +cmd
;; Got answer:
;; ->>HEADER<<- opcode: QUERY, status: NXDOMAIN, id: 37809
;; flags: qr rd ra ad; QUERY: 1, ANSWER: 0, AUTHORITY: 1, ADDITIONAL: 1

;; OPT PSEUDOSECTION:
; EDNS: version: 0, flags:; udp: 4096
;; QUESTION SECTION:
;doesnotexist.domain.               IN      A

;; AUTHORITY SECTION:
.                   1200    IN      SOA     a.root-servers.net. nstld.verisign-grs.com.<PHONE_NUMBER> 1800 900<PHONE_NUMBER>0

;; Query time: 2 msec
;; SERVER: <IP_ADDRESS>#53(<IP_ADDRESS>)
;; WHEN: Tue Sep 22 13:13:11 CEST 2015
;; MSG SIZE  rcvd: 123

Without more information or a way to reproduce it we won't be able to do much.
Can you capture the network traffic when you run the Go program, or dig?
From that we should be able to write a test, once we see what's arriving on the wire from your DNS server.

If you can tell me how to make a local copy of Go's net package work, I can dump out the relevant data structures and maybe even find the issue myself. Unfortunately just copying /opt/go/src/net to my local directory and changing import "net" to import "../net" does not work. It says "imports internal/singleflight: use of internal package not allowed". Copying /opt/go/src/internal, too, does not help. Copying "net" and "internal" to $GOPATH/src does not work (without changing import "net") either. They are simply ignored and the global system version keeps being used.

You won't be able to do that. Just build Go from source and modify the net package.
Run "./src/make.bash" from the "src" directory to recompile Go. It takes 30-60 seconds to build everything.

I've found the problem. My resolv.conf has a line "search foo bar" for 2 subdomains foo and bar. For whatever reason queries to anything.bar are answered with error code 2 SERVFAIL. LookupIP("doesnotexist.domain") apparently uses the search field in resolv.conf and queries "doesnotexist.domain", "doesnotexist.domain.foo" and "doesnotexist.domain.bar" in sequence and reports the LAST ERROR CODE returned, which happens to be SERVFAIL. If the order in resolv.conf is changed to "search bar foo", LookupIP reports the NXDOMAIN error (no such host).
While it is apparent that the DNS server is misbehaving, the current LookupIP() implementation is suboptimal, because:

the error code returned depends on the ordering in the search line of resolv.conf
the behaviour of Go 1.5 is different from Go 1.4
the error code reported does not relate to the actual query, which is not what the programmer expects, especially not in a case where the host queried contains a ".". I'm not even sure if it is proper to append the search subdomains to names that contain a ".".

In any case, I would change LookupIP() so that if the DNS server returns different error codes for the different queries made due to to "search", LookupIP() should always return the error code returned for the query with the actual argument passed to LookupIP(). This is the only query that is guaranteed to happen and therefore the only code that can be expected to be consistent.

the behaviour of Go 1.5 is different from Go 1.4
Go 1.5 uses Go by default for DNS lookups, and only falls back to libc's resolver for special cases. See https://golang.org/doc/go1.5#net for details.

 It's being exercised a lot more than it has in the past.
/cc @mikioh

FWIW, IIRC, troubleshooting tools such as dig and/or drill don't use search list for super (not sub) domains in resolv.conf.
Looks like you have some idea to LookupIP. I'm not the original designer of LookupIP but I guess he wanted to make LookupIP easier for name-to-address mapping traversal. If you think of having a new API (sorry, we cannot change the behavior of LookupIP because it works as a stub resolver for helping Dial) for some purpose, please follow the procedure: https://github.com/golang/proposal#readme
Thank you.

Why did you close this bug?

PS: If you need more low-level control for DNS, there are external packages. For example, http://godoc.org/github.com/miekg/dns.

Just hands slipped.

I don't think @mbenkmann wanted low-level control of DNS. I interpreted this bug as Go's native DNS resolver just misbehaving compared to libc.

To repeat, here is what I (and I would assume most application programmers) expect from LookupIP()

consistent behavior. Ordering of entries in configuration files should not matter.
the correct answer. The host "doesnotexist.domain" does not exist. I want that answer. That the DNS server has a corrupt database for a subdomain "bar" unrelated to my query is no excuse for giving the wrong answer to my query. I'm an application programmer. I support my application. I don't want support requests from users with the error description "Your application says my DNS server is misbehaving. But it's working fine with every other application."
consistency across Go versions. In the same environment Go 1.4 gives a different answer than 1.5. BAD!
if there are multiple different errors that LookupIP() could return (in this case "host not found" and "server misbehaving") I want the error most closely related to MY CODE, because that's what I can debug. MY CODE requests resolving of "doesnotexist.domain" and the appropriate error is "host not found". That LookupIP() performs some additional queries behind-the-scenes to try extra hard to give me non-error reply is nice, but if that behind-the-scenes code fails, I'm not interested in its error code if there's an error code available directly related to my code.

I'm repeating myself but I really need to drive this point home: My application, like many, resolves host names provided by users. Users make mistakes, especially typos. When LookupIP() fails I write the error into a log file and users look at that log file. It makes a HUGE difference if that log file says "host not found" or "server misbehaving". It's the difference between allowing the user to quickly realize he's made a typo and fix the problem himself or forcing the user to file a support request with me that will lead to a fruitless discussion unpleasant to both sides about a DNS server and system configuration that neither I nor the user have under our control.

I totally agree with.
A packet capture would be helpful for debugging and writing a regression test.

There's nothing special about the server's reply. It's just an error code 2 SERVFAIL which triggers the "server misbehaving" branch in net/dnsclient.go:answer(). The answer() function is called for each of the queries attempted during lookup (i.e. for each entry in search) and as currently implemented the last error propagates up to be the return from LookupIP(). So for regression testing, just set up your test DNS server to reply SERVFAIL for each query that ends in a certain subdomain and list that subdomain in resolv.conf's search line. Once this issue is fixed, it should no longer matter if the broken subdomain is listed first or last and unless the argument to LookupIP() explicitly includes the broken subdomain, the SERVFAIL should never propagate up.

Related to #12778 ?

Similar to #12778 I took a quick look at glibc to see how it handles this. If I'm reading/interpreting correctly, there is a special case for SERVFAIL when trying names with search domains appended causing it to try the next search domain. Should Go's stub resolver do the same?

I did some more digging and was able to repro this using a DNS server built with github.com/miekg/dns.
Instead of the current capturing and returning of the last seen error it probably would make sense to prefer the error encountered when looking up the name closest to what was passed in to LookupIP and friends.
This would mostly mimic what glibc does (here and here), though it only returns the error encountered when looking up the provided name if it has enough dots. Otherwise it will return the last error encountered while trying names with search domains appended, similar to what the Go implementation is doing now.
I'm happy to put a CL together if preference for errors close to user input sounds like a good plan.


[QC] Fail -> critical fail, add flag

For now not allowing flag states -- will consider requests for this in the future


Fixed typo in jsdoc of function tail

When scanning the code base I came across this small typo.
Furthermore I noticed that some parameters tags were missing from other functions, for instance the insert function. I can also add those if that is desired

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

Thanks!


Adds info about windows to Docker install

Description
I added a Windows-specific step to the Docker install because this was getting missed, as the instructions for Windows were on a different page. This PR addresses feedback from a recent benchmark study.
Issues Resolved
List any issues this PR will resolve, e.g. Closes [...].
Checklist

[x] By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license and subject to the Developers Certificate of Origin.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

One comment for build error, otherwise LGTM

Thanks, @Naarcha-AWS! I revised the intro sentence to include the link to the 'important settings' section, since it applies to both Linux and Windows. Can you please give it another read? Thanks!

Thanks, @natebower, this is much better!


Add option to "Read and Agree" with mentor and mentee guides at the application process

Is your feature request related to a problem? Please describe.
Currently, there is no formal mechanism to ensure that mentors and mentees have read and agreed to the guidelines and terms before submitting their application, which could lead to misunderstandings or non-compliance during the mentoring program.
Describe the solution you'd like
Implement a mandatory "Read and Agree" step for mentors and mentees before they can submit an application.

[ ] Add checkbox for agreeing to with the guides
[ ] Add agreement to volunteer during the mentoring program
[ ] Prevent application submission if terms are not agreed to

Describe alternatives you've considered
Additional context
Guides are currently linked as an external resource and will be added to the platform with #164

Any updates on this @RandilaP?


[SSoC 2.0] Alarm Clock

🛠 Issue (1713)

Closes #1713
Description
This project helps to create an alarm clock. The user can set an alarm that rings at the time set by the user and can be switched off. It also has a digital clock to show the current time. Easy and beginner-friendly project using HTML, CSS, and JavaScript.
Type of change

[x] New feature (non-breaking change which adds functionality)

Checklist:

[x] I have made this from my own
[x] I have taken help from some online resourses
[x] My code follows the style guidelines of this project
[x] My changes generate no new warnings
[x] The title of my pull request is a short description of the requested changes.

ATTACH SCREEN-SHOTS / DEPLOYMENT LINK

@TusharKesarwani @zugzwang03 Please review this PR.

@zugzwang03 Added readme


Add support for external Stylesheets in the Project

Allow to include external Stylesheets from the Project, and also add support to modify them.

I think, loading maybe already workin (if included via style tag)


SLEIGH grammar files - antlr or yacc?

I noticed that there are two types of grammar files in the repository, ones using ANTLRv3 that seem to be used by the Java part of Ghidra, and yacc grammar files that are part of the cpp decompiler. I'm trying to figure out what the difference between them is.

Are they just different implementations of the same thing? Which one is used for parsing the slaspec/cspec/pspec/etc files for processors (e.g. AARC64)?
If they are just different implementations of the same parser, which is more up to date? Which one would be recommended for building additional tools that work with the sleigh files?
If they are for parsing different things, what is the difference?
How do the various parsers fit together -- is the SLEIGH/slaspec converted using the Java parser into some intermediate sleigh representation that the C/yacc parser takes in along with the xml files as part of the decompiler?

I basically don't know anything about this part of the code, and the documentation on them (particular the yacc files in the decompiler) is kinda lacking. What I could really use is someone to ELI5 how the various parsers/grammars in Ghidra fit together.

So my understanding, having played with the ANTLR files a bit myself, is:

The ANTLR and yacc grammar files are both used for parsing SLEIGH specs (.slaspec). They should implement more or less the same grammar. The reason for this is that the decompiler is essentially a standalone program that you can run, so it needs to be able to load a SLEIGH spec independently from the rest of Ghidra. cspec/pspec/ldefs/opinion files are not parsed by the grammar files, they basically are just separate XML files that contain various metadata about the processors and tell Ghidra which *.slaspec files to associate with which processor defs. The relaxng specs can be found here.
I've experimented a bit with the ANTLR files and would personally recommend using them over yacc.
They both parse *.slaspec files.
The SLEIGH compiler compiles the *.slaspec and generates *.sla files (XML) as output. These are what is primarily used. They can be regenerated at runtime or built ahead of time. There's a way to directly invoke the SLEIGH compiler from the command line.

Both SLEIGH compilers are actively maintained and perform the same function; converting .slaspec files to the .sla format.  They may give slightly different error messages but should produce identical .sla files from the same .slaspec input.  See this test.
The yacc version is built on the decompiler c++ code, and the decompiler can be built to read in .sla files using this infrastructure.  In the main build however, the decompiler doesn't include this infrastructure but gets all its p-code from the SLEIGH engine in the Java-based part of Ghidra.  This engine doesn't really care which SLEIGH compiler produced the .sla file its using; however it considers .sla files to be ephemeral and builds them automatically using the Java/ANTLR compiler if they're not already present.
For tools that work with SLEIGH source, you can use whatever compiler code base is more convenient; yacc for native code or ANTLR for Java-based.  Any proposed change to SLEIGH itself would need to be implemented in both code bases.

Both SLEIGH compilers are actively maintained and perform the same function; converting .slaspec files to the .sla format.  They may give slightly different error messages but should produce identical .sla files from the same .slaspec input.  See this test.
The yacc version is built on the decompiler c++ code, and the decompiler can be built to read in .sla files using this infrastructure.  In the main build however, the decompiler doesn't include this infrastructure but gets all its p-code from the SLEIGH engine in the Java-based part of Ghidra.  This engine doesn't really care which SLEIGH compiler produced the .sla file its using; however it considers .sla files to be ephemeral and builds them automatically using the Java/ANTLR compiler if they're not already present.
For tools that work with SLEIGH source, you can use whatever compiler code base is more convenient; yacc for native code or ANTLR for Java-based.  Any proposed change to SLEIGH itself would need to be implemented in both code bases.


MAINT: Use scikeras.KerasRegressor.r_squared as loss function or catch/assert warning in all tests calling score

This warning has been removed. Closing the issue.


Add subesh's files

created a branch, created a file and made a commit, and opened a pull request

Sorry

👍


Recreation of #5326 for conflicts with other PR

postgressql->postgresql

/assign @unguiculus

/retest

@AdamDang Please don't keep creating new PRs as soon as there are problems like conflictx. Rather fix conflicts and update existing PR. Thanks.

/assign @unguiculus
I've learned how to solve the conflicts now.  :)
Thanks!

/lgtm


Is your library extendable for N-dimensional points?

It's actually a feature request!
I was trying to use distance metrics like Hausdorff or Chamfer on N-dimensional point sets and as I expected, an exception was thrown. Are these functions extendable for N-dimensional point sets? So that they can compute the distance between extracted features in deep networks like Pointnet?

This isn't implemented yet but it could be in the future. Leaving this open in case someone wants to implement this in the future.


OnsetSliceClient: shrinking window size can cause UB

The mFrameTime counter in OnsetSliceClient needs to reset if the window size shrinks, or at least make an effort not to read past the end of available memory.

indeed fixed thanks!


DEV: Allow Authenticator.add_user to not be a future.

In particular, this fixes an error that occurs if a user subclasses the base Authenticator class and doesn't override add_user.

Definitely right, thanks.


Fixing bug 1260549 in samples used by accessibility team.

Fixes Issue #336.
Description
When the ComboBox is collapsed, and the user changes its value using the Up/Down arrow keys, the Narrator does not speak. The SelectionChanged event of ComboBox was used with a handler thar raises LiveRegionChanged event, so the Narrator will say the name of the selected option.
Customer Impact
The user using keyboard to navigate could not know what is selected in the ComboBox.
Regression
No. This sample is used for Accessibility testing and never fully supported Narrator.
Testing
The sample project was tested using keyboard to assert the narrator says the selected item even when the ComboBox is collapsed.

About the item 2, the change is only announced once when expanded as I tested, like it was before.


Remove old action-server versions

Target components
action_server
Why & who
Whenever the action server is started the extracted version is added to %localappdata%\robocorp\action-server and it's never removed.
Feature or enhancement
We should have a process to remove those old versions.

Solution & alternative
No response

This issue has been moved here, under Sema4AI/actions repository.


Common mistakes docs section

Many of the issues we get are repeats of the same common questions / mistakes. We should create a small docs section logging said common mistakes and their solutions :).

Hello, I would like to work on this issue!

@SohamG934,
Great! Feel free to submit a PR and I'll review. A few common mistakes I can think of:

Mixing up how use_enum_values is used (we have an example in the API docs)
Mixing up how to add annotations to compound types (see https://github.com/pydantic/pydantic/issues/8362)

I'm sure there are many more that we could include - you could search on duplicate tags to see what issues have been reported the most frequently!

Hello, shall i make a single doc mentioning the title of the issue and then
linking the closed issue solution link for reference? If not, how do you
propose I solve this?
On Sat, 16 Dec 2023 at 18:14, Sydney Runkle @.***>
wrote:

@SohamG934 https://github.com/SohamG934,
Great! Feel free to submit a PR and I'll review. A few common mistakes I
can think of:

Mixing up how use_enum_values is used (we have an example in the API
docs)
Mixing up how to add annotations to compound types (see #8362
https://github.com/pydantic/pydantic/issues/8362)

I'm sure there are many more that we could include - you could search on
duplicate tags to see what issues have been reported the most frequently!
—
Reply to this email directly, view it on GitHub
https://github.com/pydantic/pydantic/issues/8380#issuecomment-1858810041,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AX3DW7QYWWTQYI6HN76IIMLYJWJRHAVCNFSM6AAAAABAW4NJDOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNJYHAYTAMBUGE
.
You are receiving this because you were mentioned.Message ID:
@.***>

Hi @SohamG934,
I think we should create a new page for our documentation site that has various examples of common mistakes and ways to correct said mistakes. I don't see a reason to reference issues for the most part. You can create a fork of the pydantic repo and submit a PR with your changes 👍

Hi @SohamG934,
I think we should create a new page for our documentation site that has various examples of common mistakes and ways to correct said mistakes. I don't see a reason to reference issues for the most part. You can create a fork of the pydantic repo and submit a PR with your changes 👍

Hi @sydney-runkle, is @SohamG934 still working on this issue?
If not, I'd like to work on it. Thank you

@bL34cHig0,
I haven't seen a PR yet, so feel free to get started on this :). Thanks!

@bL34cHig0,
There's no template at the moment - I'd recommend combing through issues listed as duplicates to see what kind of issues come up often.
Could you please add this section to the examples section on a new page for now? Thanks!

@sydney-runkle
I have started working on documenting the common mistakes / questions. So I wanted to ask if I am on the right track.
Please see screenshot below. Thanks :)

Work done so far:

Created a new branch called common-mistakes-doc
Created a new page titled common_mistakes.md under the examples section.
Wrote the first common question / mistake.

Please kindly review it and let me know if this structure is okay or not. Thanks again

@bL34cHig0,
The structure looks great. Nice work so far!

Here's another I think should be included: https://github.com/pydantic/pydantic/issues/8708

See https://github.com/pydantic/pydantic/pull/8759#issuecomment-1971737662 as the reason for closing this issue. I think this collection is more helpful to have as an issue than as an official docs page, despite my initial motivations for filing this issue.
A helpful collection of recommendations, thanks to @bL34cHig0: https://github.com/pydantic/pydantic/pull/8759

Below shows the relevant docs sections for explaining each of these questions:

How to implement custom error messages?

https://docs.pydantic.dev/latest/errors/errors/#customize-error-messages

Cannot use v1 model as type parameter in v2 generic model

https://docs.pydantic.dev/latest/migration/#changes-to-pydanticgenericsgenericmodel

AnyUrl adds trailing slash (/) in URL validation

https://docs.pydantic.dev/latest/migration/#url-and-dsn-types-in-pydanticnetworks-no-longer-inherit-from-str
See updates here: https://github.com/pydantic/pydantic/issues/7186

TypeError occurs when using Pydantic's Constraint Types

This one makes more sense to have reported + documented as an issue, which is explained well here:
This error occurs if you are using Python 3.9.7. To prevent this, upgrade to Python 3.9.8 or higher as there is a
bug in version 3.9.7. Also, it is discouraged to use some of the
con functions because they will be deprecated
in Pydantic 3.0. So check out the Discouraged warning for alternative solution.

validate_call decorator raises "AttributeError" when applied to an instance method of a class with __slots__

No longer an issue

Two validators consecutively do not report all errors

Pydantic does not currently support exhaustive validation, so it stops after the first error on a given field and
reports that validation error. Relevant feature request, for tracking: https://github.com/pydantic/pydantic/issues/7476

Can not use ValidationError __init__ method

https://docs.pydantic.dev/latest/api/pydantic_core/#pydantic_core.ValidationError provides info on helpful static methods

Pydantic does not allow Literal enum value discriminator to be initialized by string

A simple solution to this is to allow your class to inherit from both str and Enum. This is the one case where I do think we could use a docs update in the relevant section of the docs :).


[Batch] upgrade azure batch API version to 2024-02-01

Related command

Description

Upgrade azure batch API version to 2024-02-01
Upgrade azure-batch to 14.2.0
Upgrade azure-mgmt-batch to 17.3.0
Testing Guide

History Notes

[Batch] az batch pool create Add --upgrade-policy-mode, --disable-auto-rollback, --enable-auto-os-upgrade, --defer-os-rolling-upgrade, --use-rolling-upgrade-policy, --enable-cross-zone-upgrade, --max-batch-instance-percent, -max-unhealthy-instance-percent, --max-unhealthy-upgraded-instance-percent, --pause-time-between-batches, --prioritize-unhealthy-instances, and --rollback-failed-instances-on-policy-breach arguments to support automatic OS upgrade

This checklist is used to make sure that common guidelines for a pull request are followed.

[x] The PR title and description has followed the guideline in Submitting Pull Requests.

[x] I adhere to the Command Guidelines.

[x] I adhere to the Error Handling Guidelines.

Batch

Hi @zhoxing-ms, I saw you are the first assigned reviewers, and we'd like to include this change in the current milestone, could you review it as soon as you can? Thanks!

/azp run Azure.azure-cli

[Batch] az batch pool create add new parameters --upgrade-policy-mode, --disable-auto-rollback, --enable-auto-os-upgrade, --defer-os-rolling-upgrade, --use-rolling-upgrade-policy, --enable-cross-zone-upgrade, --max-batch-instance-percent, -max-unhealthy-instance-percent, --max-unhealthy-upgraded-instance-percent, --pause-time-between-batches, --prioritize-unhealthy-instances, and --rollback-failed-instances-on-policy-breach to support automatic OS upgrade

Please refer to this guideline guideline_link to improve the History Notes in your PR description

[Batch] az batch pool create add new parameters --upgrade-policy-mode, --disable-auto-rollback, --enable-auto-os-upgrade, --defer-os-rolling-upgrade, --use-rolling-upgrade-policy, --enable-cross-zone-upgrade, --max-batch-instance-percent, -max-unhealthy-instance-percent, --max-unhealthy-upgraded-instance-percent, --pause-time-between-batches, --prioritize-unhealthy-instances, and --rollback-failed-instances-on-policy-breach to support automatic OS upgrade

Please refer to this guideline guideline_link to improve the History Notes in your PR description

I capitalized the "Add" keywords according to the guide, not sure is there anything else needs to be change?

No, what I mean is that you need to clearly describe each change in the PR description, so that customers can understand the specific functions of these changes. For example:

@wanghoppe Currently we are working on dropping all dependencies on msrestazure and we detected that the latest azure-batch still relies on msrestazure which is already deprecated.
Can you help remove msrestazure dependency from azure-batch and bump it in Azure CLI? Really appreciated.


[BUG] Mixing v2 and v3 GUNW products fails

Describe the bug
It seems that the ariaTSsetup.py program cannot handle a product list with a mixture of v2 and v3 S1 GUNW products
To Reproduce
Steps to reproduce the behavior:

Download
ariaDownload.py --num_threads 16 -b '33.4 34.5 -119.0 -117.4' -s 20190101  -e 20211231 -t 64 no version specified

2024-05-22 18:46:40,233 - ariaDownload.py - INFO - Download complete. Wrote -- 4181 -- products to: /scratch/nisar-st-calval-solidearth/ARIA_S1/Secular/LA_A64_3years/fielding/20240522/v1/products

Command used
ariaTSsetup.py -f "/scratch/nisar-st-calval-solidearth/ARIA_S1/Secular/LA_A64_3years/fielding/20240522/v1/products/*.nc" -b '33.6 34.3 -118.5 -117.5' -mo 5800 --mask Download  --verbose
Error Output

Traceback (most recent call last):
  File "/home/jovyan/.local/envs/solid_earth_atbd/bin/ariaTSsetup.py", line 632, in <module>
    main()
  File "/home/jovyan/.local/envs/solid_earth_atbd/bin/ariaTSsetup.py", line 410, in main
    standardproduct_info = ARIAtools.product.Product(
                           ^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/jovyan/.local/envs/solid_earth_atbd/lib/python3.11/site-packages/ARIAtools/product.py", line 420, in __init__
    self.__run__()
  File "/home/jovyan/.local/envs/solid_earth_atbd/lib/python3.11/site-packages/ARIAtools/product.py", line 1247, in __run__
    self.products = self.__continuous_time__()
                    ^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/jovyan/.local/envs/solid_earth_atbd/lib/python3.11/site-packages/ARIAtools/product.py", line 1009, in __continuous_time__
    vers.append(ver_num)
                ^^^^^^^
UnboundLocalError: cannot access local variable 'ver_num' where it is not associated with a value

Expected behavior
If ariaTSsetup cannot work with both v2 and v3 products, it should give a warning message and stop quickly, instead of running for several hours and then giving an error. For the LA area, the v2 products are difficult to select because there are both v2_0_2 and v2_0_5 products, and I did not see a way to provide a list of versions for ariaDownload or ariaTSsetup.
Desktop (please complete the following information):

ARIA-tools git tag: This is the same conda-forge version as my last issue report.
OS  CentOS

Additional context
Running in NISAR Science Team On-Demand System

@EJFielding , this bug had been resolved and pushed to dev a few days back. I'll work to issue a formal release though so these changes are reflected on conda. Thanks!

Closed as fixed on dev


fix: catching the error from http-proxy-middleware

Refs: #143
🚨 Proposed changes

Please review the guidelines for contributing to this repository.

[[Describe the big picture of your changes here to communicate to the maintainers why we should accept this pull request. If it fixes a bug or resolves a feature request, be sure to link to that issue.]]
⚙ Types of changes
What types of changes does your code introduce? Put an x in the boxes that apply

[ ] New feature (non-breaking change which adds functionality)
[x] Bugfix (non-breaking change which fixes an issue)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation Update (if none of the other choices apply)
[ ] Refactor

@antoniomuso Any word on seeing this change merged in?

Yes, sure but I am waiting because I need to add the logger. And this require a breaking change.


Cancellation support and timeouts for symbol servers

Waiting for network timeouts can be a major bummer. It would be great if we could:

Add overloads that take System.Threading.CancellationTokens.
Add support for configurable timeouts (shorter than network timeout by default?).
Async support would be icing on the cake.

Some comments from Lee:

In addition to cancellation support, we also need to have a timeout for SymbolLocator.  I’m not sure if it should be an individual timeout (for slow individual servers) or a global timeout when calling FindBinary/FindPdb (such that after N seconds the operation is cancelled) or both.  Keep in mind that calling virtually any function in ClrMD may go out to the symbol server to download files if we are missing metadata that we need to perform an operation.

Also it’d be really nice if ClrMD could support some modern features like async/await…especially for symbol servers.

I have implemented async/await additions to ClrMD, along with adding a timeout to SymbolLocator.  Unfortunately HttpWebRequest does not accept cancellation tokens, so I'm not sure there's an easy way to use one here.

You could use HttpClient instead - it's all async.

Good point.  I won't be able to get to it until next week, but I'll add CancellationToken support to the todo list.

Hello!  This issue is being closed due to its age.  The codebase has moved significantly since this was filed.  Please feel free to open a new issue for ClrMD about this issue if it still reproduces on ClrMD 1.0.  (Also, I apologize for having not looked at or resolved this earlier and for resolving it now.  In order for me to be able to make some progress on the growing issue count in ClrMD I need to clear the backlog down to a manageable set of issues, but do feel free to file a new one if this is still a problem.)


BUG: Improve alignment of content and dates of cards in Blogs Section

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
In the Agroblogs section, the card content is not aligned properly. The date positioning is also different for half of the blog cards

Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

@Suchitra-Sahoo
Please assign this issue to me :)
Thank you


Rails 3 support

The gem with 'rails-settings-cached', '0.2.4' not work.
You can fix it changing
settings.defaults.merge!(settings.get_all)

to
settings.defaults.merge!(settings.all)

in app/admin/settings.rb on line 13.

@damianogiacomello Thanks for you report.
Please see commit
https://github.com/artofhuman/activeadmin_settings_cached/commit/b692941ece9c256d62bbc627e7df35aa5aa1aa44
This commit include fix for rails 3 and 4.0.x


fix: SearchBar in Navbar vertical-align

Thank you for your contribution to the KodaDot NFT gallery.
👇 _ Let's make a quick check before the contribution.
PR Type

[x] Bugfix
[ ] Feature
[ ] Refactoring

Context

[x] Closes #4545
[ ] Requires deployment <>

Before submitting pull request, please make sure:

[x] My contribution builds clean without any errors or warnings
[x] I've merged recent default branch -- main and I've no conflicts
[x] I've tried to respect high code quality standards
[x] I've didn't break any original functionality
[x] I've posted a screenshot of demonstrated change in this PR

Optional

[ ] I've tested it at </bsx/collection>
[ ] I've tested PR on mobile
[ ] I've written unit tests 🧪
[ ] I've found edge cases

Had issue bounty label?

[x] Fill up your KSM address: Payout

Community participation

[x] Are you at KodaDot Discord?

Screenshot 📸

[x] My fix has changed something on UI; a screenshot is best to understand changes for others.

Not the ideal solution, we shouldn't have to set top margin on navbar start since it's extending navbar height (and navbar-shrink) But this hack works for now :/
Seems that columns class is causing problems

Both class columns and incorrect margin-top cause this problem. However, the  columns could not be removed or it would break the style otherwhere.  So, changing the margin-top would be a fair way to fix this issue. It's not a hack way : )

IMO, we should replace Search with SearchBar component as there is a lot of code that does not have to be present on all pages. ref #3883

IMO, we should replace Search with SearchBar component as there is a lot of code that does not have to be present on all pages. ref #3883

It makes sense. Should do some component refactoring to achieve so.

pay 20 usd

😍 Perfect, I’ve sent the payout
💵 $20 @ 24.45 USD/KSM ~ 0.818 $KSM
🧗 Caiv9TbPz68q5dC8EcHu5xKYPRnremimGzqmEejDFNpWWLG
🔗 0xb7f31e8069e57d583e81828f91d54540194133ad2ce78d2d4b501c6bc43a861a
🪅 Let’s grab another issue and get rewarded!
🪄 github.com/kodadot/nft-gallery/issues


Bash sanitization

Tested deploy and scale up scenario described in #125.
This should give us clearer errors in the logs instead of failure to connect to "Thu" (because today is Thursday, right?) like @php-coder was seeing :scream_cat:

@bparees ptal

lgtm and thanks for the education on the use of local.  Seems to have a test issue though.

[test]

The test result says "Yum repo mirrorlist timed out"
Let's try again [test]

@rhcarvalho that test result is a red herring, it's not the actual failure cause.

(i'm pretty sure we always get that error now, even when things succeed.  if you have time to look into which repo is timing out and see if we can remove it from our docker image or something, that'd be great)

Aha, so there's a test failure that reads:
MongoDB shell version: 2.4.9
connecting to: <IP_ADDRESS>:27017/db
Thu Feb 25 13:01:55.818 Error: couldn't connect to server <IP_ADDRESS>:27017 at src/mongo/shell/mongo.js:147
    Trying to connect...
MongoDB shell version: 2.4.9
connecting to: <IP_ADDRESS>:27017/db
Thu Feb 25 13:01:58.938 Error: couldn't connect to server <IP_ADDRESS>:27017 at src/mongo/shell/mongo.js:147
  Giving up: Failed to connect. Logs:
=> Waiting for container IP address ...
Stopping and removing container 58ccaa388e21715596a6418d761f083fe531a43ef1716d48141e94087ac04224...
58ccaa388e21715596a6418d761f083fe531a43ef1716d48141e94087ac04224
58ccaa388e21715596a6418d761f083fe531a43ef1716d48141e94087ac04224
Done.
ERROR: mongodb-rhel7 failed testing.

Looks like a flake. Let's see what the next run gives (probably tomorrow for me).
The yum error goes with this output:
Step 6 : RUN yum install -y yum-utils &&     yum-config-manager --enable rhel-server-rhscl-7-rpms &&     yum-config-manager --enable rhel-7-server-optional-rpms &&     INSTALL_PKGS="bind-utils gettext iproute rsync tar v8314 mongodb24-mongodb mongodb24" &&     yum install -y --setopt=tsflags=nodocs $INSTALL_PKGS &&     rpm -V $INSTALL_PKGS &&     yum clean all &&     mkdir -p /var/lib/mongodb/data && chown -R mongodb.0 /var/lib/mongodb/ &&     chmod -R a+rwx /opt/rh/mongodb24/root/var/lib/mongodb &&     chmod -R g+rwx /var/lib/mongodb
 ---> Running in f6f971e34169
Loaded plugins: ovl
Repo rhui-REGION-client-config-server-7 forced skip_if_unavailable=True due to: /etc/pki/rhui/cdn.redhat.com-chain.crt
Repo rhui-REGION-client-config-server-7 forced skip_if_unavailable=True due to: /etc/pki/rhui/product/rhui-client-config-server-7.crt
Repo rhui-REGION-client-config-server-7 forced skip_if_unavailable=True due to: /etc/pki/rhui/rhui-client-config-server-7.key
Repo rhui-REGION-rhel-server-releases forced skip_if_unavailable=True due to: /etc/pki/rhui/cdn.redhat.com-chain.crt
Repo rhui-REGION-rhel-server-releases forced skip_if_unavailable=True due to: /etc/pki/rhui/product/content-rhel7.crt
Repo rhui-REGION-rhel-server-releases forced skip_if_unavailable=True due to: /etc/pki/rhui/content-rhel7.key
Repo rhui-REGION-rhel-server-rh-common forced skip_if_unavailable=True due to: /etc/pki/rhui/cdn.redhat.com-chain.crt
Repo rhui-REGION-rhel-server-rh-common forced skip_if_unavailable=True due to: /etc/pki/rhui/product/content-rhel7.crt
Repo rhui-REGION-rhel-server-rh-common forced skip_if_unavailable=True due to: /etc/pki/rhui/content-rhel7.key
Could not retrieve mirrorlist https://rhui2-cds01.REGION.aws.ce.redhat.com/pulp/mirror/rhui-client-config/rhel/server/7/x86_64/os error was
14: curl#6 - "Could not resolve host: rhui2-cds01.REGION.aws.ce.redhat.com; Name or service not known"
Could not retrieve mirrorlist https://rhui2-cds01.REGION.aws.ce.redhat.com/pulp/mirror/content/dist/rhel/rhui/server/7/7Server/x86_64/os error was
14: curl#6 - "Could not resolve host: rhui2-cds01.REGION.aws.ce.redhat.com; Name or service not known"
Could not retrieve mirrorlist https://rhui2-cds01.REGION.aws.ce.redhat.com/pulp/mirror/content/dist/rhel/rhui/server/7/7Server/x86_64/rh-common/os error was
14: curl#6 - "Could not resolve host: rhui2-cds01.REGION.aws.ce.redhat.com; Name or service not known"
Package yum-utils-1.1.31-34.el7.noarch already installed and latest version
Nothing to do

[test]

@legionus please help! :)

[test]

@legionus you are the Bash guru! :clap: big thanks for your help
@bparees ^^ finally we got this green :)

@bparees @mfojtik ready to merge, ptal

can you squash pls?

@mfojtik there you go...


Update Du Novo to 3.0.2.

This updates Du Novo on Main.
It addresses the issue with #476.

@galaxybot test this

@galaxybot deploy this


strange behaviour when unittesting with singleton lambda handler

when unittesting code like the following
from lambdarest import lambda_handler

@lambda_handler.handle("get")
def my_own_get(event):
    return {"this": "will be json dumped"}

The lambda_handler singleton will stay instantiated between testcases since the module is not torn down completely (unittest does this to be able to later on gather the test results see https://chrisbailey.blogs.ilrt.org/2013/05/19/pythons-leaky-testcase-aka-hidden-gotchas-using-self/)
This means that the same lambda_handler instance will be used for all testcases, them being in different files or not. Which again means that the internal lookup http_methods will be scrampled.
A fix for this is to either put your import statements locally in your test functions like:
class test(unittest.TestCase):
    def test_this(self):
        from my_module import lambda_handler
        lambda_handler.handle("get")

or stop using the singleton and instantiate your own like:
from lambdarest import create_lambda_handler
lambda_handler = create_lambda_handler()

issue has been mentioned in README


Prevent Cropping Before Crop Area Is Placed

If you hit Apply button whiteout cropping, all image is cropped out. It's normal behavior since all image is dark at that time but it would be nice if doesn't do anything if cropping area is 100% of real image.

Thanks for the good comments.


ros1_bridge: 0.9.6-1 in 'foxy/distribution.yaml' [bloom]

Increasing version of package(s) in repository ros1_bridge to 0.9.6-1:

upstream repository: https://github.com/ros2/ros1_bridge.git
release repository: https://github.com/ros2-gbp/ros1_bridge-release.git
distro file: foxy/distribution.yaml
bloom version: 0.10.0
previous version for package: 0.9.4-1

Note to reviewers, jumping to 0.9.6 to jump over 0.9.5 which was used on rolling before foxy diverged.
@jacobperron fyi


[ConstraintSystem] Fix support for a single pack expansion parameter …

…in init references
In Swift 5 and earlier initializer references are handled in a special way
that uses a type variable to represent a type of the parameter list.
Such type variables should be allowed to bind to a pack expansion type
to support cases where initializer has a single unlabeled variadic generic
parameter - init(_ data: repeat each T).

@swift-ci please test


Missing link in docker for windows documentation

I clicked on the Docker Toolbox link in the docs hyperlink and I got:
File: 404.md

@binaryflesh This is working for me - are you still experiencing a problem?

Hi @binaryflesh, thanks for your feedback on the download URL. Please note that Docker no longer maintains the download.docker.com URL for Docker Toolbox. You can download the latest version of Docker Toolbox from https://github.com/docker/toolbox/releases.
Docker Toolbox documentation has been updated to reflect this change. For more information, see:

https://github.com/docker/docker.github.io/pull/8969/
https://docs.docker.com/toolbox/overview/

Closing this issue.


Uppy Serverless

I'll let @ifedapoolarewaju have the final say but this looks really good! Awesome stuff @dcflow!


Add Waveform widget keyboard shortcuts

UI version
1.1.6
Your idea
Add the following shortcuts:
shift-space: toggle between unpin both, pin right
s: single marker add
d: dual marker add
1-9: navigate to marker: zoom to dual markers, center single markers
delete, backspace: remove all annotations
Some of these shortcuts were in the 0.9 UI.  See issues #9 and #105.
See forum: https://forum.joulescope.com/t/hotkeys-shortcuts/796

Available in 1.1.7.


Sqlite MATCH text search

If I have a sqlite db with a virtual table using fts like so:
CREATE VIRTUAL TABLE posts
USING FTS4(title, body);

and then I try and reference it within Kysely
const statement = db
      .selectFrom("posts")
      .selectAll()
      .where("posts", "match", search)
      .limit(100);

I get a type error on match.
The syntax for the query would look like
SELECT *
FROM posts
WHERE posts MATCH 'text' ;

Are match queries supported in kysely?
Thanks in advance

You can pass any operator using raw sql
where('posts', sql`match`, search)


LIVE-3396 cardano handle blacklisted tokens

📝 Description
bug fix: filter out blacklisted token accounts
❓ Context

Impacted projects: ledger-live-common
Linked resource(s):  https://ledgerhq.atlassian.net/browse/LIVE-3396

✅ Checklist

[ ] Test coverage
[x] Atomic delivery
[x] No breaking changes

📸 Demo

🚀 Expectations to reach
Please make sure you follow these Important Steps.
Pull Requests must pass the CI and be internally validated in order to be merged.

Hi @pavanvora there's a weird issue with the Android build, could you please merge develop in, see if that solves it.


Adding new question

Checklist:

[x ] I have read freeCodeCamp's contribution guidelines.
[x ] My pull request has a descriptive title (not a vague title like Update index.md)

Please anticipate adding high-quality questions.

@Gautam-Hegde
Can you explain more what you mean here?

Please anticipate adding high-quality questions.

@Gautam-Hegde Can you explain more what you mean here?

I thought the question/distractors did not meet the website's standards since they were quite ambiguous and vague.

@Gautam-Hegde

since they were quite ambiguous and vague

This is a better review and now @SaulMiquilena was able to act on it and make some changes.
When leaving reviews you want to provide actionable feedback, so the user will know more specifically what they need to change and make it better 👍


Would love to get formatter support on save

Hey y'all,
I know you guys already have alloy fmt, so It shouldn't be difficult to have that command added as a formatter option, so we can format on save. I've never implemented it before, but might be able to take a crack if no one else want's to. Cheers!

I also think this would be great! I don't think I'll have time to implement this for a while so others are definitely free to take a crack at it :)

Gave it a shot in #4

Implemented in #4


Add recursion support to /1.0/networks

Similar to /1.0/images, implement support for querying all the networks at once.

Claimed

Can this be closed now?

Yep

Next time, you can include "Closes #474" in your commit message so that the bug gets automatically closed when the branch is merged.


Gradually go from Dart => HTML => CSS

I don't know if I'm typical user, but I very very rarely use HTML and CSS in Dart Pad. The UI isn't "gradual", it assumes I want HTML and CSS.
Allow me to opt-in to HTML and CSS. Start with just Dart code, but make it easy to add HTML, and then make it easy to add CSS.
I assume that not having HTML and CSS will make Dart Pad run a bit quicker for pure Dart pads. That is a nice benefit, but not the primary motivator.
There's a feedback weirdness which can be fixed if we opt-into HTML/CSS (or allow a developer to remove a HTML / CSS file). If I have a pure Dart pad, I still see "Hello, world!" printed on my screen. That's pretty weird. I never even looked at the HTML file, why does it print out?  (I know why, but perhaps new users won't know why :)
Suggestion:
Keep the three tabs (Dart, HTML, CSS) but only default to the Dart tab. Allow a user to activate HTML and CSS from those tabs, or deactivate them. If not activated, gray them out and add a PLUS sign. If activated, they have white text (as they do now) and they have a MINUS sign. The Dart tab always remains.

BTW thanks!

Around 24% of our users interact with HTML/CSS. This doesn't include those who do so with an HTML enabled sample.
At this rate, it doesn't really justify consuming the top right hand panel of the screen for the non-HTML cases.
I agree with Seth, we could make this functionality opt-in, and gist backed so that if they were using an HTML entry they would get the panels.
Being able to opt in to more tabs sets the tone for being able to add more files later.
In the case where there is no HTML, I'd prefer to not need to switch between documentation and output, but to have them both available top/bottom.

Around 24% of our users interact with HTML/CSS. This doesn't include those who do so with an HTML enabled sample.

Curious, what does "interact" mean?

In the case where there is no HTML, I'd prefer to not need to switch between documentation and output, but to have them both available top/bottom.

sgtm

See also https://github.com/dart-lang/dart-pad/issues/123

I think this is largely addressed?

Love the improvements. Let's close out. Thanks!


Customizable ring radius

It would be nice to be able to customize the ring radius

We intent to treat this component like an iOS system one. Therefore we want it to be as simple as possible with almost no customization.


DO - Declaration referencing unresolvable NTP FQDN after it has been removed from LTM configuration.

Environment

Declarative Onboarding Version: 1.34.0
BIG-IP Version: 15.1.8

Summary
The LTM in question has already been configured via DO as a Stand alone.
This DO Job is to add the device into a Device Group.
It is being done in this manner as a test due to inconsistencies uncovered in establishing an HA cfg between two devices.
The issue here is - The LTM was using an FQDN in the NTP configuration for a time server.
The error that is occurring:
{
        "id": "0ebe6770-9124-4bdc-8b34-3836b93df22d",
        "selfLink": "https://localhost/mgmt/shared/declarative-onboarding/task/0ebe6770-9124-4bdc-8b34-3836b93df22d",
        "code": 500,
        "status": "ERROR",
        "message": "invalid config - rolled back",
        "errors": [
            "tryUntil: max tries reached: Unable to resolve host time.google.com",
            "tryUntil: max tries reached: Unable to resolve host time.google.com"
        ],
        "result": {
            "class": "Result",
            "code": 500,
            "status": "ERROR",
            "dryRun": false,
            "message": "invalid config - rolled back",
            "errors": [
                "tryUntil: max tries reached: Unable to resolve host time.google.com",
                "tryUntil: max tries reached: Unable to resolve host time.google.com"
            ]
        },
After the FQDN was removed and replaced with an ip address the same error was still occurring.
Here is the JSON script that is being run when the error occurs:
{
    "schemaVersion": "1.34.0",
    "class": "Device",
    "async": true,
    "label": "DO Deploy Lab",
        "controls": {
        "trace": true,
        "traceResponse": true,
        "dryRun": false
    },
    "Common": {
        "class": "Tenant",
        "mySystem": {
            "class": "System",
            "hostname": "xxxxxxxxx",
            "cliInactivityTimeout": 600,
            "consoleInactivityTimeout": 600,
            "autoPhonehome": false
        },
        "ManagementIp": {
            "class": "ManagementIp",
            "address": "xxxxxxxxx",
            "remark": "management ip"
        },
        "myLicense": {
            "class": "License",
            "licenseType": "regKey",
            "regKey": "xxxxxxxxxxx",
            "overwrite": false
        },
        "failoverUnicastAddress": {
            "class": "FailoverUnicast",
            "addressPorts": [
                {
                    "address": "<IP_ADDRESS>"
                },
                {
                    "address": "<IP_ADDRESS>"
                }
            ]
        },
        "Mirroring": {
            "class": "MirrorIp",
            "primaryIp": "<IP_ADDRESS>",
            "secondaryIp": "<IP_ADDRESS>"
        },
        "sync": {
            "class": "ConfigSync",
            "configsyncIp": "<IP_ADDRESS>"
        },
        "float_<IP_ADDRESS>": {
            "class": "SelfIp",
            "address": "<IP_ADDRESS>/26",
            "vlan": "vlan_121",
            "allowService": "default",
            "trafficGroup": "traffic-group-1"
        },
        "float_<IP_ADDRESS>": {
            "class": "SelfIp",
            "address": "<IP_ADDRESS>/26",
            "vlan": "vlan_122",
            "allowService": "default",
            "trafficGroup": "traffic-group-1"
        },
        "float_<IP_ADDRESS>": {
            "class": "SelfIp",
            "address": "<IP_ADDRESS>/26",
            "vlan": "vlan_123",
            "allowService": "default",
            "trafficGroup": "traffic-group-1"
        },
        "trust": {
            "class": "DeviceTrust",
            "localPassword": "xxxxxxx",
            "localUsername": "admin",
            "remoteHost": "<IP_ADDRESS>",
            "remotePassword": "xxxxxxx!",
            "remoteUsername": "admin"
        },
        "Z-lab-sby": {
            "class": "DeviceGroup",
            "type": "sync-failover",
            "members": [
                "<IP_ADDRESS>",
                "<IP_ADDRESS>"
            ],
            "owner": "/Common/Z-lab-sby/members/0",
            "autoSync": true,
            "saveOnAutoSync": false,
            "networkFailover": true,
            "fullLoadOnSync": false,
            "asmSync": false
        }
    }
}
Expected Behavior
Reference the ip address not the fqdn that was removed
Actual Behavior
See error output

thanks @Ronnie110755 will review and prioritize for our next sprint starting Tuesday.

Added to our internal product backlog as AUTOTOOL-3635

Sunitha,
Quick question.
I encountered this message in vscode while executing a json script on a
very slow to respond ltm:
251 [2023-01-16T22:15:28.529Z] [INFO]: authToken expired -> will refresh
with next HTTPS call
When I connect to the LTM using f5 extensions I see the following:
256 [2023-01-18T16:56:59.430Z] [INFO]: HTTPS-RESP [92rz]: 200 - OK
257 [2023-01-18T16:56:59.431Z] [INFO]: Starting token timer: 1200
That is 20 minutes. I know in the real world that is more than enough
time to run a JSON DO script.
BUT! you never know.
I have been looking for a way in VSCODE to set that token timeout value.
I can't a way to do that.
Is there a quick answer that someone may have for this?
Or a forum I can post this into that is not a an Issue or a Bug area?
Thanks!
Ronnie Cusmano.
------ Original Message ------
From: "Suntiha Ronanki" @.>
To: "F5Networks/f5-declarative-onboarding"
@.>
Cc: "Ronnie110755" @.>; "Mention"
@.>
Sent: 1/18/2023 11:48:47 AM
Subject: Re: [F5Networks/f5-declarative-onboarding] DO - Declaration
referencing unresolvable NTP FQDN after it has been removed from LTM
configuration. (Issue #337)

Added to our internal product backlog as AUTOTOOL-3635
—
Reply to this email directly, view it on GitHub
https://github.com/F5Networks/f5-declarative-onboarding/issues/337#issuecomment-1387378118,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AZBWEOIRXFO7OJK5R5IM2ATWTANG7ANCNFSM6AAAAAAT2V3KBQ.
You are receiving this because you were mentioned.Message ID:
@.***>

Cannot modify the NTP configuration from IP to FQDN.
Before
     "myNtp": {
            "class": "NTP",
            "servers": [
                "<IP_ADDRESS>"
            ],
            "timezone": "UTC"
        },

to
"myNtp": {
            "class": "NTP",
            "servers": [
                "time.f5.com"
            ],
            "timezone": "UTC"
        },

Failed with Error
{
        "id": "2449777e-2760-4c41-a719-6e43858c5bb0",
        "selfLink": "https://localhost/mgmt/shared/declarative-onboarding/task/2449777e-2760-4c41-a719-6e43858c5bb0",
        "code": 422,
        "status": "ERROR",
        "message": "invalid config - rolled back",
        "errors": [
            "01070276:3: The requested device (/Common/bigip1.f5demo.com) already exists.",
            "01070276:3: The requested device (/Common/bigip1.f5demo.com) already exists."
        ],
        "result": {
            "class": "Result",
            "code": 422,
            "status": "ERROR",
            "dryRun": false,
            "message": "invalid config - rolled back",
            "errors": [
                "01070276:3: The requested device (/Common/bigip1.f5demo.com) already exists.",
                "01070276:3: The requested device (/Common/bigip1.f5demo.com) already exists."
            ]
        },


fix: remove context limit warnings from UI

The ContextWindowLimitError banner was being displayed in the Chat UI when context limit warnings occurred, causing confusion to users. The PR updates to log the context limit warning message instead to provide useful debugging info, instead of sending it to the webview to display as an error, which will then be rendered as the ContextWindowLimitError banner.
Test plan

Ask Cody multiple questions to confirm no ContextWindowLimitError banner was displayed in the Chat UI.

see https://github.com/sourcegraph/cody/issues/2766 for details


Seeing strange behavior in reliability diagram for histogram binning

I've run HistogramBinning on several datasets and always seem to end up with an empty right-most bin on the post-calibration reliability diagram (see figure below).

two notes:
-also shows up in isotonic regression
-goes away if num_bins is the same for the fit and for the reliability diagram

@chris-wells-13 are there any updates on this? Could you please share some data where this error occurs? I've recently seen the same behavior at one of our students but did not have access to the data to reproduce.

@chris-wells-13 are there any updates on this? Could you please share some data where this error occurs? I've recently seen the same behavior at one of our students but did not have access to the data to reproduce.

I found same issue. My prediction results didn't contain values that are greater than 0.9, but the histogram still plotted it while scikit-learn not.

Closing this issue due to missing information for reproducibility. Please re-open if necessary.


NEL benchmark: Add comparison of multiple evaluation runs

Description

Adds a step for the comparison of multiple evaluations, to simplify comparing e.g. multiple candidate generators. This includes

persisting evaluation results as .csv files for every execution of spacy project run evaluate in a new directory evaluations and
a new workflow step merging all available individual evaluation results, sorting by run and then by model.

A run name can be configured (defaults to a stringified timestamp of format %Y-%m-%d_%H-%M-%S).
Open questions:

Do we like the table formatting/ordering like this?
Do we want to show the table by label (in analogy to the label-based table in spacy project run evaluate) as well?

Example output:
============================ compare_evaluations ============================
Running command: env PYTHONPATH=. python ./scripts/compare_evaluations.py mewsli_9
16:30:45 INFO
+-----------------------------------+-------------------+-----+----+----+---------+--------+-----------+
|                Run                |       Model       |  TP | FP | FN | F-score | Recall | Precision |
+-----------------------------------+-------------------+-----+----+----+---------+--------+-----------+
|        2022-09-12_16-07-40        |   Candidate Gen.  | 150 | 4  | 4  |  0.974  | 0.974  |   0.974   |
|        2022-09-12_16-07-40        |       Random      | 131 | 14 | 19 |  0.888  | 0.873  |   0.903   |
|        2022-09-12_16-07-40        |       Prior       | 145 | 0  | 5  |  0.983  | 0.967  |    1.0    |
|        2022-09-12_16-07-40        |       Oracle      | 145 | 0  | 5  |  0.983  | 0.967  |    1.0    |
|        2022-09-12_16-07-40        |    Context Only   | 145 | 5  | 5  |  0.967  | 0.967  |   0.967   |
|        2022-09-12_16-07-40        | Context And Prior | 145 | 5  | 5  |  0.967  | 0.967  |   0.967   |
| NEL benchmark_2022-09-12_15-21-03 |   Candidate Gen.  | 150 | 4  | 4  |  0.974  | 0.974  |   0.974   |
| NEL benchmark_2022-09-12_15-21-03 |       Random      | 128 | 17 | 22 |  0.868  | 0.853  |   0.883   |
| NEL benchmark_2022-09-12_15-21-03 |       Prior       | 145 | 0  | 5  |  0.983  | 0.967  |    1.0    |
| NEL benchmark_2022-09-12_15-21-03 |       Oracle      | 145 | 0  | 5  |  0.983  | 0.967  |    1.0    |
| NEL benchmark_2022-09-12_15-21-03 |    Context Only   | 145 | 5  | 5  |  0.967  | 0.967  |   0.967   |
| NEL benchmark_2022-09-12_15-21-03 | Context And Prior | 145 | 5  | 5  |  0.967  | 0.967  |   0.967   |
| NEL benchmark_2022-09-12_15-53-41 |   Candidate Gen.  | 150 | 4  | 4  |  0.974  | 0.974  |   0.974   |
| NEL benchmark_2022-09-12_15-53-41 |       Random      | 129 | 16 | 21 |  0.875  |  0.86  |    0.89   |
| NEL benchmark_2022-09-12_15-53-41 |       Prior       | 145 | 0  | 5  |  0.983  | 0.967  |    1.0    |
| NEL benchmark_2022-09-12_15-53-41 |       Oracle      | 145 | 0  | 5  |  0.983  | 0.967  |    1.0    |
| NEL benchmark_2022-09-12_15-53-41 |    Context Only   | 145 | 5  | 5  |  0.967  | 0.967  |   0.967   |
| NEL benchmark_2022-09-12_15-53-41 | Context And Prior | 145 | 5  | 5  |  0.967  | 0.967  |   0.967   |
| NEL_benchmark_2022-09-12_15-18-23 |   Candidate Gen.  | 150 | 4  | 4  |  0.974  | 0.974  |   0.974   |
| NEL_benchmark_2022-09-12_15-18-23 |       Random      | 130 | 15 | 20 |  0.881  | 0.867  |   0.897   |
| NEL_benchmark_2022-09-12_15-18-23 |       Prior       | 145 | 0  | 5  |  0.983  | 0.967  |    1.0    |
| NEL_benchmark_2022-09-12_15-18-23 |       Oracle      | 145 | 0  | 5  |  0.983  | 0.967  |    1.0    |
| NEL_benchmark_2022-09-12_15-18-23 |    Context Only   | 145 | 5  | 5  |  0.967  | 0.967  |   0.967   |
| NEL_benchmark_2022-09-12_15-18-23 | Context And Prior | 145 | 5  | 5  |  0.967  | 0.967  |   0.967   |
+-----------------------------------+-------------------+-----+----+----+---------+--------+-----------+

Checklist

[x] I confirm that I have the right to submit this contribution under the project's MIT license.
[x] I ran the tests, and all new and existing tests passed.
[x] My changes don't require a change to the documentation, or if they do, I've added all required information.

have an export format that's more machine-readable, like a combined CSV?

You mean for the combined evaluation results table?

highlight the run that maximizes some criteria, maybe just with an extra row that's empty or filled with *?

Makes sense. I'll add an option to pick the criterion to highlight.

Rows with highest scores are now logged in bold. Rows related to candidate generation are treated separately, since the other rows related to disambiguation. This is a bit confusing though, maybe we should move the candidate generation rows into a separate table?
Example output (here with precision as criterion for highlighting):

have an export format that's more machine-readable, like a combined CSV?

You mean for the combined evaluation results table?

Yes, for the combined results. But without a definite use case it's not really important, so just and idea.
I was able to run your changes locally without issue. The bold is also pretty subtle here, maybe you could do reverse video instead? For ANSI escapes I was going to suggest you use wasabi (which actually also has table formatting, now that I think of it), but it doesn't seem to support reverse video, though it'd be easy to add.

Yes, for the combined results. But without a definite use case it's not really important, so just and idea.

Added an export, stored as evaluation/{dataset name}/compare-{datetime.datetime.now().strftime('%Y-%m-%d_%H-%M-%S')}.csv. Does this make sense to you?
Following up on the question in the description:

Do we want to show the table by label (in analogy to the label-based table in spacy project run evaluate) as well?

Added an export, stored as evaluation/{dataset name}/compare-{datetime.datetime.now().strftime('%Y-%m-%d_%H-%M-%S')}.csv. Does this make sense to you?

That's great, thank you! I was easily able to navigate/sort that table with a CSV tool.

Do we want to show the table by label (in analogy to the label-based table in spacy project run evaluate) as well?

Is this just about the formatting/appearance? I don't really have a strong opinion either way, so I think it's fine as-is.

Is this just about the formatting/appearance? I don't really have a strong opinion either way, so I think it's fine as-is.

No. If you run evaluate, you'll see two tables: one with the results per method (random/oracle/...), one with results per method and label. In compare_evaluation there's only the former so far (so all results per method and run), because I thought that listing all results for method x run x label would be too much info. Not sure about that though (especially since we have the export now).

Ah, I see what you mean about labels. I agree that having that all in the output for visual inspection feels like just too much info. Even considering that it could be in the machine-readable output, since we don't have a concrete need for it at the moment, I don't feel it's imperative to add it now.


Handle empty coordinate maps

Includes the single commit from #29.
Fixes #30.
Given deps.edn as per the bug report:
{:deps {org.clojure/clojure {}}
 :aliases {:newer-stuff
           {:override-deps {org.clojure/clojure {:mvn/version "1.10.0"}}}}}

Invoking depot with no args prints:
Checking for old versions in: deps.edn
  All up to date!

Invoking with --write
Updating old versions in: deps.edn
  All up to date!

and leaves the file unchanged.

Invoking with --every:
Checking for old versions in: deps.edn
  org.clojure/clojure {:mvn/version "1.10.0"} -> {:mvn/version "1.10.1"}

Invoking with --every and --write:
Updating old versions in: deps.edn
  org.clojure/clojure {:mvn/version "1.10.0"} -> {:mvn/version "1.10.1"}

and the file is now
{:deps {org.clojure/clojure {}}
 :aliases {:newer-stuff
           {:override-deps {org.clojure/clojure {:mvn/version "1.10.1"}}}}}

Awesome stuff! I'll review and merge at some point soon :smile:

I was thinking this would fix all of Sean's woes but I think this one persists :thinking:  I'll have to try it myself https://github.com/Olical/depot/issues/11#issuecomment-530948726

Thank you very much for the fixes! :D


AzureRmPolicyAssignment has no -Scope examples other then RG's

Documentation does not provide any examples for setting scope at a Subscription level as in the portal.

Please file an issue with the engineering project - this is where reference is maintained and edited. https://github.com/azure/azure-powershell/issues


Fix ungrouping bugs to preserve layer ordering and prevent incorrect layer deletion

Closes #1671
Closes external consistency of ungrouping (sub-issue of #1633 )

This looks nice! But there's one issue remaining that I've found. Your fix for the case you reported in #1671 does, indeed, prevent the layers from being deleted. However it reorders them so b appears above a. Can you please fix that before merging this?


[Dropdown] Semantic UI doesn't replace the {value} in a GET route when loading dropdown options remotely

Hello,
Semantic UI should replace route {value} automatically, but it doesn't work for dropdowns that get their options from a remote source apparently, and you get an error:

API: Missing a required URL parameter: value /search?query={value}

Please see this JSFiddle with developer console opened; I was expecting it to get .search value and replace the route {value} automatically. The only way to make it work as far as I'm concerned, is to specify beforeSend setting like so: JSFiddle.
You might also want to read this StackOverflow question to get more information on the issue.

This wasn't something I considered, but should be adjusted to match how it functions in search, passing {value} into API.

@Banandrew This is  already passed as query, miscategorized this issue.
http://jsfiddle.net/nex9ozzf/
Value refers to the currently set value, which doesn't seem relevant to pass with search queries.


Workspace - The button to save the type of plan doesn't work

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: 9.0.68-0
Reproducible in staging?: Y
Reproducible in production?: N/A - new feature, doesn't exist in prod
Issue reported by: Applause Internal Team
Action Performed:

Go to https://staging.new.expensify.com/
Login with the new Gmail account in incognito mode.
Complete the incorporation process using the option “Something Else”
Enter a first name and click on the Continue button.
Click on the "FAB" button and click on "New Workspace".
Click on the "Plan type" section and select "Control" and click on "Save".
Click on the "Upgrade button" and click on the "Got it, Thanks" button.
Change the Subscription details to Pay-per-use.
Try changing the Plan type from the workspace section to "Collect".

Expected Result:
The save button should work correctly.
Actual Result:
The button to save the plan type chosen for the workspace does not work after changing the subscription type.
Workaround:
Unknown
Platforms:

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

Screenshots/Videos
https://github.com/user-attachments/assets/18f1d065-d111-4e0f-b75b-67dec44ddebe

View all open jobs on GitHub

@trjExpensify i thought its not possible to downgrade in app now?

Yeah, we should revert this, I asked about the risk of this happening here 😅
cc @jayeshmangwani @DylanDylann can we get a revert PR up quick? This needed to wait for the BE to be ready 🙏

Yeah, we should revert this, I asked about the risk of this happening https://github.com/Expensify/App/pull/52117#issuecomment-2504101224 😅

Sorry about this! We should have placed that button behind the beta here 😓.

@jayeshmangwani all good, I also should have checked!

The button to save the type of plan doesn't work

This is intentional because we haven't implemented the downgrade feature. But we forgot to put this page behind the beta

This was CPed so we can close


Add ELU activation function

Relevant paper by Sepp Hochreiter (co-inventor of LSTMs): Fast and Accurate Deep Network Learning by Exponential Linear Units (ELUs)
http://arxiv.org/abs/1511.07289


Breadcrumb Component

Requirements
Create a basic breadcrumb component that:

Takes either a data array to render as links, or passes pre-rendered links into a slot.
Conforms to accessibility guidelines.

Example
https://designsystem.digital.gov/components/breadcrumb/

Accessibility Guidelines
A11y Style Guide
W3.org

Should we consider doing individual links instead of an array? I'm wondering if it's better to migrate to Lit what Biggie did on SWHR.
Here is her code and storybook with the individual breadcrumb implementation.
The problem I'm having is drilling down into the <ol> to do the styling. I could use Tailwind classes if we're able to get that working in the main TS file. Thoughts?

We at least now have the slotted version, and should consider the data array option as an enhancement so we can do both.


use original destination path for Page construction

instead of URL.
This will fix #107

@jekyll/plugin-core I've recreated https://github.com/jekyll/jekyll-redirect-from/issues/107 in the wild. Any objection to merging this PR?
Steps to reproduce:

Create a new page foo.html with permalink: /foo/
Add jekyll_redirect_to: /bar/

Expected
File /foo/index.html exists and is a redirect
Actual:
File /foo exists, and is a redirect.

Err actual, should probably use https://github.com/jekyll/jekyll-redirect-from/pull/111 instead, which has tests.

@benbalter #111 does the same?

I believe #111 fixes this. Thanks @koron!


Torch and OpenCV installed but Cupscale reports as missing.

Python 3.9.7 is detected correctly but Pytorch and OpenCV are reported as missing during the Cupscale (v1.39.0f1) dependencies check. I have verified that PyTorch is installed correctly and have tested CUDA and the GPU drivers.
Using the embedded Python is not an option since I have an Ampere GPU (RTX 3080 laptop).

Embedded Python works with Ampere, try that first

I tried that previously but Cupscale just hangs at "Upscaling". I tried again a few minutes ago and had the same result (I have attached the log). I checked the Windows task manager and noticed that Python continues to run even after Cupscale is closed. (As for the version number I entered, that is what the Cupscale folder was named after extraction. Not sure why....)
sessionlog.txt

I still have this issue, hanging on "Upscaling", tried deleting entirely and re-installing dependencies, but when command windows are shown, it says that an RTX 3060 is not compatible with the version of CUDA that the embedded python has


Remove explicit core dependency

API change check
APIView has identified API level changes in this PR and created following API reviews.
Microsoft.Azure.WebJobs.Extensions.EventGrid


fix: separating property pane out of widgets editor to avoid triggering unwanted events on widget canvas

Description
PropertyPane inside Widgets Editor causes events to be trigerred that is not meant for property pane. So, separating it out of Widgets Editor.
Fixes #11378
Fixes #11079

Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?
manual UI and cypress
Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes

Test coverage results :test_tube:
:green_circle: Total coverage has increased
// Code coverage diff between base branch:release and head branch: widgetsEditorEventFix

Status
File
% Stmts
% Branch
% Funcs
% Lines

:green_circle:
total
55.76 (0.02)
37.08 (0.02)
35.81 (0.01)
56.13 (0.03)

:green_circle:
app/client/src/pages/Editor/MultiSelectPropertyPane/index.tsx
85.71 (0.71)
50 (0)
66.67 (0)
88.89 (0.65)

:sparkles: :new:
app/client/src/pages/Editor/WidgetsEditor/WidgetsEditor.tsx
93.65
60
100
93.33

:red_circle:
app/client/src/pages/Editor/WidgetsEditor/index.tsx
87.5 (-4.28)
100 (40)
33.33 (-41.67)
85.71 (-5.47)

:green_circle:
app/client/src/selectors/commentsSelectors.ts
85.25 (1.64)
64.71 (2.95)
73.33 (0)
90.59 (2.35)

:green_circle:
app/client/src/utils/WorkerUtil.ts
89.76 (0.78)
72.55 (1.96)
100 (0)
93.33 (0.95)

:green_circle:
app/client/src/widgets/ButtonGroupWidget/component/index.tsx
25.3 (1.97)
11.02 (0.33)
5 (0.24)
27.78 (2.78)

:green_circle:
app/client/src/widgets/ButtonGroupWidget/widget/index.tsx
59.26 (2.59)
0 (0)
33.33 (0)
60 (2.86)

/ok-to-test sha=9079ca9

@rahulramesha Tested and working fine.

/ok-to-test sha=1edb98b

/ok-to-test sha=36313b5

/ok-to-test sha=73f7354

@rahulramesha Tested the guided tour and working fine.


Semantic_linkbacks_type returning incorrectly

Issue: the semantic_linkbacks_type function is returning "mention" instead of "like"
Like received here:
https://colinwalker.blog/2017/04/19/sonant-thoughts-episode-11-preparation-and-nerves/
From here:
http://stream.boffosocko.com/2017/liked-colin-walkers-sonant-thoughts-episode-11-preparation-and-nerves
Relevant plugins installed:
IndieWeb
Webmention
Semantic Linkbacks

I currently have the function filtering for "mention" which is why the like shows.

@colin-walker just to confirm, have you checked to make sure that webmention is indeed stored as a like? I've had backfeed from boffosocko come in as only a mention myself, when it was supposed to be a reply. Was mentioned that could be an issue with Known.

@miklb Maybe that's it - it does seem to be stored as mention. If I set my comment walker for a null type then check semantic_linkbacks_type for what's returned it says "mention" rather than like. Didn't know if it was a local issue or not.

After some more testing it does look like this is what's happening. The issue does appear to be with how Known has sent the 'like'.
This can be closed.

Re-opening as requested.

@colin-walker at leasts it is good to know now that it's how the mention is stored vs the function itself returning improperly.

per the discussion in #indieweb IRC, a test suite was mentioned later that could be of benefit.
https://github.com/microformats/tests

The like-of property appears to be parsed correctly in @chrisaldrich's post, so I do think this is in one of the WP plugins: http://pin13.net/mf2/?url=http%3A%2F%2Fstream.boffosocko.com%2F2017%2Fliked-colin-walkers-sonant-thoughts-episode-11-preparation-and-nerves

@miklb the test suite does not help here, because the parser has no issues.
@gRegorLove I said parser, but what I meant was interpreter... the parsing is OK, but the like-of is in a place where the plugin does not recognize it.

subscribing with a vengeance. this has plagued me for years now. can't wait to see it fixed!
examples: https://snarfed.org/2017-03-12_23853#comment-2603533
(both emoji reactions, but i've seen normal replies trigger it too.)
reaction example that didn't trigger it, on the same post: https://snarfed.org/2017-03-12_23853#comment-2603372
IRC discussion: https://chat.indieweb.org/2017-04-30#t1493587145707000

@snarfed This may sound crazy, but I used to see something like this issue until I uninstalled the uf2 plugin. I couldn't guarantee it was the cause as I couldn't pin it down exactly, but since uninstalling it, I haven't seen it since. cross reference: https://github.com/pfefferle/wordpress-webmention/issues/97#issuecomment-292746131

@chrisaldrich interesting! that would be sad if so. i definitely plan to keep using uf2, so if it's the cause, we should definitely find and fix the bug.

the problem is that I no longer selfdogfooding the uf2 plugin, but I am open to add new contributors.

Should be addressed by #85

Believe this can be closed now with latest merge.

@colin-walker can you verify this?

Will keep an eye on it and let you know.

Poke?

i haven't seen this for a while. I'm ok with closing.


Towards a generic MARGO model structure (with abstract submodel types)

I've been working on implementing a more accurate version of MARGO's 1.5-layer (or "Deep-layer") Energy Balance Model. The improved solution is an exact analytical solution to the two-layer problem (see https://github.com/hdrake/ClimateMARGO.jl/issues/43).
I quickly ran into a problem that the MARGO code, as it is currently written, is not nearly as flexible as we would like it to be. In particular, we do not have the appropriate abstractions for allowing submodules to be swapped out for others on the fly.
I imagine that in addition to the bare-bones MARGO configuration, other configurations could be easily created by picking and choosing the complexity of various components. If we define the abstract types in this way, it would also be easier for others in the community to make their own changes and possibly contribute them to the master branch.

One huge advantage of abstract types is that some methods will apply to the abstract types while others will apply to specific model types.

I've started implemented this in the WIP PR https://github.com/hdrake/ClimateMARGO.jl/pull/45
For example, there is now an abstract type EBMparams which includes lists of parameters for several difference simple energy balance models.
abstract type EBMParams end

mutable struct DeepLayerEBM <: EBMParams
    λ::Real   # feedback parameter
    Cd::Real  # deep ocean heat capacity
    κ::Real   # deep ocean heat uptake rate
end

mutable struct TwoLayerEBM <: EBMParams
    λ::Real   # feedback parameter
    κ::Real   # deep ocean heat uptake rate
    Cu::Real  # upper ocean heat capacity
    Cd::Real  # deep ocean heat capacity
    ϵ::Real   # heat uptake efficacy
end
TwoLayerEBM(λ, κ, Cu, Cd) = TwoLayerEBM(λ, κ, Cu, Cd, 1.)

UpperLayerEBM(λ, Cu) = TwoLayerEBM(λ, 0., Cu, Cu)

In addition to these three types having different numbers of parameters (with different values), they are have different physics and thus different solutions. Thus, they will require different methods of the diagnostic functions:
T(m::UpperLayerEBM) = ...
T(m::DeepLayerEBM) = ...
T(m::TwoLayerEBM) = ...


Create saurabh

Fixes Issue

Changes proposed

Check List (Check all the applicable boxes)

[ ] My code follows the code style of this project.
[ ] My change requires changes to the documentation.
[ ] I have updated the documentation accordingly.
[ ] All new and existing tests passed.
[ ] This PR does not contain plagiarized content.
[ ] The title of my pull request is a short description of the requested changes.

Screenshots

Note to reviewers

I have corrected the syntax error in json file as well as renamed the file .


[Bug]: [TextIO] Prevent threads from hanging indefinitely

What happened?
Recently observed the case where TextIO / WriteFiles was hanging indefinitely on @FinishBundle:
https://github.com/apache/beam/blob/release-2.48.0/sdks/java/core/src/main/java/org/apache/beam/sdk/io/WriteFiles.java#L1021-L1034
We are doing MoreFutures.get(MoreFutures.allAsList(closeFutures)), and for some reason, threads are not completing.
I believe we should investigate adding a reasonable timeout (MoreFutures.get(MoreFutures.allAsList(closeFutures), 1, TimeUnit.HOURS)?) to avoid having jobs hanging forever. The work will be retried, so it shouldn't cause any data loss.
Issue Priority
Priority: 2 (default / most bugs should be filed as P2)
Issue Components

[ ] Component: Python SDK
[X] Component: Java SDK
[ ] Component: Go SDK
[ ] Component: Typescript SDK
[ ] Component: IO connector
[ ] Component: Beam examples
[ ] Component: Beam playground
[ ] Component: Beam katas
[ ] Component: Website
[ ] Component: Spark Runner
[ ] Component: Flink Runner
[ ] Component: Samza Runner
[ ] Component: Twister2 Runner
[ ] Component: Hazelcast Jet Runner
[ ] Component: Google Cloud Dataflow Runner

Closing as duplicate of  https://github.com/apache/beam/issues/29926


sparse: reorganize sparse python codebase

Signed-off-by: Rohan Yadav<EMAIL_ADDRESS>

This commit addresses part of https://github.com/nv-legate/legate.sparse/issues/4. It does not yet pass tests.

Tested this locally, looks like it's working.


header/p2p: request head from multiple peers

Resolves:

https://github.com/celestiaorg/celestia-node/issues/980
https://github.com/celestiaorg/celestia-node/issues/1019

Based on https://github.com/celestiaorg/celestia-node/pull/1038 . Will be rebased after merge

Codecov Report

Merging #1046 (d57e897) into main (9e9fc18) will increase coverage by 0.00%.
The diff coverage is 65.24%.

@@           Coverage Diff            @@
##             main    #1046    +/-   ##
========================================
  Coverage   57.52%   57.53%
========================================
  Files         133      134     +1
  Lines        8268     8430   +162
========================================
+ Hits         4756     4850    +94
- Misses       3038     3100    +62
- Partials      474      480     +6

Impacted Files
Coverage Δ

header/p2p/request.go
0.00% <0.00%> (ø)

header/p2p/serde.go
0.00% <0.00%> (ø)

header/helper.go
22.72% <22.72%> (ø)

header/p2p/exchange.go
64.84% <72.34%> (+2.62%)
:arrow_up:

header/p2p/server.go
54.95% <73.07%> (+1.69%)
:arrow_up:

header/p2p/pb/extended_header_request.pb.go
40.00% <74.24%> (+7.13%)
:arrow_up:

fraud/registry.go
84.61% <0.00%> (-15.39%)
:arrow_down:

fraud/store.go
55.88% <0.00%> (-8.83%)
:arrow_down:

fraud/pb/proof.pb.go
29.22% <0.00%> (-8.58%)
:arrow_down:

fraud/helpers.go
72.72% <0.00%> (-4.20%)
:arrow_down:

... and 9 more

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

In a discussion with @renaynay and @Wondertan we agreed, that we should take a header with max height that was provided by at least 2 of trusted peers


Ghost-code indication too conspicuous

I like it that the IDE gives me a visual indication of what is ghost code. However, it's too much. Especially in lemmas, where all the code is ghost code. (I'm using the dark VS Code theme, if that matters.)
This aspect was improved before in https://github.com/dafny-lang/ide-vscode/issues/166, but I'd like to suggest something even less intrusive. For example, would it be possible (and be a good idea) to leave the background color the same as for non-ghost code, but to italicize the ghost code?

Italics sound pretty intrusive as well when applied to a whole lemma.  Could we (additionally?) not highlight ghost code in lemmas?

This should be fixed when merging the following PR: https://github.com/dafny-lang/dafny/pull/1830

I find the indication of ghost code very helpful (for all code). Perhaps it could be easily toggled?

I wonder whether visualising fewer things as ghost is the right solution to the ghostness indication being experienced as intrusive. Even if you minimise what is shown as ghost, there can still be arbitrarily large sections that are shown as ghost. Also, being consistent about showing everything that is ghost as ghost, should make it clearer to a novice user what the visualisation means. Only indicating ghostness where it is inferred, something which isn't obvious, might be confusing.
Just for clarity, this what the ghostness currently looks like in light and dark on my machine. It seems OK to me, and significantly different from selection.

For Dark I think I can also get used to it:

Could you show what you mean? It looks significantly different from selection in the themes I looked at.

If I were just looking at the ghost code (and not your selection), I would also assume that the code is selected.

I have an idea. Please comment.
What about we change the opacity of ghost code and the background contrary to selected code?
Here is a dark theme example where the background is rgba(0,0,0,0.2) and opacity is at 80%

With some text selected

There it's arguably closer to the notion of "ghost code", it's disappearing in some sense but the contrast stays similar because we darken the background.
However, we can't do this in the light theme, because usually the background is already totally white, we can't make it "whiter", and 80% opacity is not visible enough as a difference.
So I just tried 70% opacity and italics and it looks pretty good.

If I added the italics to the dark theme (and removed the background color alltogether), I would get this:

Putting faded background + italics in dark theme is not bad either

but now, I don't think the background plays an important role anymore... with opacity and italics, the effect seems quite good.
Interesting points:

Italics remind of comments (did you notice that all comments are in italics in most IDEs?)
Syntax highlighting is fully kept.
Opacity makes it immediately visible which portions of code are ghost vs. which are not, outside of the fovea.

Can you take screenshots with large proofs, like large calc blocks?
Git-issue-1989.dfy

Git-issue-697h.dfy

Note: This PR is only about the styling. The style of ghost-only code is not changed and is plain normal.

And that's sometimes a problem because we don't highlight ghost expressions inside expressions anyway

But let's defer this to another PR.

Also, have we considered giving a background or bold or something to non-ghost code?

GIving a special background or boldness to code is done through decorators, and decorators are either inline or whole line, and it's always better to minimize stacking of decorations.
What you describe would be expected for a verification-first language where a user has to start writing specifications, and once they start writing compiled code, we could mark the compiled code differently. But Dafny is a verification-aware programming language. For industrial users, what matters first is the compiled code, and software developers are used to things like Contract.Assert() which are faded out in their favorite IDE.

And if we think that would be bad, then why is it good to add something to ghost code?

There are good reasons why we did not add the keyword "compiled" in the language (yet?!), and that's what can be reflected in the way we choose the default, at least it has to be in sync. In Dafny 4. functions will be compiled by default unless the "ghost" keyword is added, same for predicates.
Imagine a world now where ghost code would be the default, and we would need to add an annotation for every line of compile code, because ghost is the default. The hello world would look like this
compiled method Main() {
  compiled var message := "hello world"; // If you don't put the "compiled" keyword, you get an error message because "message" is a ghost variable and cannot be printed.
  print message;
}

That would be a very different Dafny I think, but not the one I want personally. I want gradual verification, not forced verification.

For industrial users, what matters first is the compiled code

Sure sure, it's just that we have more proofs than code, typically.

Imagine a world now where ghost code would be the default,

It is, right?  We have to write function method, not function.

It is, right? We have to write function method, not function.

It was, right. Things are changing.

NOT changing the background colour of ghost code is good.
I find italics intrusive when I've got pages of ghost code (sob).
How's about changing the colour in the gutter, margins of the gutter, or line numbers, rather than the code itself?
If (god forbid) you were putting Dafny code into a paper or a book, say, would you want ghost code to be italicise there?  I wouldn't...

We should definitely have an option to not mark code inside lemma as "ghost code" since everything is ghost by default


If the same table name exists in multiple databases, hard to distinguish dataset when adding charts

A clear and concise description of what the bug is.
Lets assume that the same table USERS exists in DB1 and DB2.  Both datasets can be added and since Database is also shown its easy to add.  However when adding a chart (for e.g.) only dataset name is shown and USERS appears twice.  Hard to distinguish if its from DB1 or DB2.  If the Database or datasource name is displayed in addition to dataset name it will be easy to distinguish
How to reproduce the bug

Go to Database and add two databases that have a common table like USERS.  Also add the USERS table (for .e.g) from both databases to the datasets
Click on Charts and search for USERS.  Its displayed twice and cannot be distinguished

Expected results
Should be possible to distinguish between the two entries
what you expected to happen.
Actual results
what actually happens.
Screenshots
If applicable, add screenshots to help explain your problem.

Environment
(please complete the following information):

browser type and version:
superset version: superset version
python version: python --version
node.js version: node -v
any feature flags active:

Checklist
Make sure to follow these steps before submitting your issue - thank you!

[ ] I have checked the superset logs for python stacktraces and included it here as text if there are any.
[ ] I have reproduced the issue with at least the latest released version of superset.
[ ] I have checked the issue tracker for the same issue and I haven't found one similar.

Additional context
Add any other context about the problem here.

Agree with this @srinisubramanian ! Check out the new designs, which should help address the issue by providing a better UX for finding & filtering datasets, contextualized to the database / schema they originate from:
https://github.com/apache/superset/discussions/18803

This still needs to be addressed

@kasiazjc @yousoph have there been any discussions to figure out a better way to handle this scenario that you're aware of?

This is no longer an issue... you see the DB and schema in the list.


Calicoctl submodule help messages

Are you planning to UT the /calico_ctl/* modules?  I think that should be part of "done-done" for this task, I don't think we should hold off merging since
a) it's a significant refactor, so the sooner it's on mainline, the easier other devs merges will be
b) we're coming from 0 code coverage on the code you're refactoring, so we shouldn't take much of a hit on coverage percentage.

@tomdee  Wrote some line comments to hopefully simplify things.

I agree with @spikecurtis on the UTs.

Can you squash before merging? I'll come and talk to you about this...

Closing in favor of the rebased #291 .


OpenIdConnect doesn't sign-in user if Individual accounts is also configured.

Hello,
I'm new on asp.net core & asp.net identity, so that couldn't undestand what is happening on my app.
I configured individual accounts on my app with adding services.AddIdentity<IdentityUser, IdentityRole>() on StartUp file and it works well.
On another code branch I also configured Azure AD login with adding services.AddAuthentication().AddOpenIdConnect() on StartUp file (in this branch individual account is not added) and it works well too.
Then I merge both branches which means that I want to use both authentications on my app, individual accounts still working, but Azure AD login doesn't working.
Here is my StartUp file.
public void ConfigureServices(IServiceCollection services)
{
        services
            .AddDbContext<ApplicationDataContext>(options =>
            {
                options.UseInMemoryDatabase("testdb");
            });

        services
            .AddIdentity<IdentityUser, IdentityRole>()
            .AddEntityFrameworkStores<ApplicationDataContext>();

        services
            .AddAuthentication(sharedOptions =>
            {
                sharedOptions.DefaultScheme = CookieAuthenticationDefaults.AuthenticationScheme;
                sharedOptions.DefaultChallengeScheme = OpenIdConnectDefaults.AuthenticationScheme;
                sharedOptions.DefaultSignInScheme = CookieAuthenticationDefaults.AuthenticationScheme;
            })
            .AddOpenIdConnect("myAzureAD", options =>
            {
                options.ClientId = "xxx";
                options.Authority = "https://login.microsoftonline.com/xxx";
                options.CallbackPath = "/signin-oidc";
                options.RequireHttpsMetadata = false;
            })
            .AddCookie();

        services.AddMvc();
}

Failed Case: After returning back to my web app from authentication on azure website, on /signin-oidc page 'Identity.External' cookie is added to the response, then this page make a 302 redirect to the my return-url, but on my return-url when I check the User.Identity.IsAuthenticated property, its value is False (which was True when individual account not configured on StartUp file)
Steps:
1.Forward client to azure authentication website with following code:
public IActionResult LoginWithOpenId(string returnUrl) { return Challenge("myAzureAD"); }
2.Login on azure website.
3.Return back to callback path(/signin-oidc).
3.Callback page set the required cookies.('Identity.External')
4.Forwarded to return-url by 302 redirect from callback page.
5.Check User.Identity.IsAuthenticated is False.
Note:I hope someone can help me, I ask this question to stackoverflow.com but no give any answer :(.
Best Regards

You can't have both, not the way you want. You have to have a way to trigger the login that you want. As you are wanting AAD and identity it becomes weird, AAD expects to be the sole source of user information, you don't double up or duplicate. This isn't a scenario we support with identity.


K8s CRD Test Works Correctly

Fix bug in K8s CRD Test
Currently the K8s CRD Test is:

Unclear in its output
Has a bug in that it doesn't actually test what we need
Could probably be automated using GKE cluster

Unclear in its output
Current output:
~/cyberark/secretless-broker/test/manual/k8s_crds (beta-release) $ ./deploy
Stopping deployment...
Deploying...
Error from server (NotFound): customresourcedefinitions.apiextensions.k8s.io "configurations.secretless.io" not found
Error from server (NotFound): services "secretless-broker" not found
Error from server (NotFound): services "secretless-broker-external" not found
Error from server (NotFound): clusterroles.rbac.authorization.k8s.io "secretless-crd" not found
Error from server (NotFound): rolebindings.rbac.authorization.k8s.io "secretless-crd" not found
Error from server (NotFound): clusterrolebindings.rbac.authorization.k8s.io "secretless-crd" not found
Error from server (NotFound): serviceaccounts "secretless-crd" not found
Error from server (NotFound): deployments.extensions "secretless-k8s-test" not found
Deploying...
clusterrole "secretless-crd" created
serviceaccount "secretless-crd" created
clusterrolebinding "secretless-crd" created
deployment "secretless-k8s-test" created
service "secretless-broker" created
service "secretless-broker-external" created
Deployment done!

We should:

Redirect stderr to /dev/null because it has expected error output which
misleadingly implies something went wrong with the test.
Add a clear "Your test was successful" at the end of the test

Has a bug in that it doesn't actually test what we need
We need to actually test that the pods have come up. That is, we need to do:
$ eval $(mk docker-env)
so that docker is pointed to kubernetes. And then:
$ kg get pods
and check that the status is running.
Could probably be automated using GKE cluster
Kumbi has an idea that would allow us to automate this entire manual test.
TBD

Here's the idea that would allow us to automate this entire manual test.

[ ] parametrise namespace in tests
[ ] parametrise CRD name to allow for parallel test runs
[ ] push secretless image to container registry
[ ] authenticate gcloud container with kubectl using summon
[ ] run test script inside gcloud container

@jonahx can this issue be closed?

There are revised k8s CRD tests now in k8s-ci/ in the project, so this issue can be closed.


plaintextLength should be and optional value for Node and Browser encrypt API

plaintextLength is not going to be an option for browsers at this time.  Because the browser api is a one-shot.  So the browser will always know the plaintextLength.