Error while installing

Node.js version: v0.10.25
Go version: go1.8.3 linux/amd64
# github.com/tj/node-prune/cmd/node-prune
/home/sam3/go/src/github.com/tj/node-prune/cmd/node-prune/main.go:50: time.Since(start).Round undefined (type time.Duration has no field or method Round)

this method was added in go 1.9, moving to the latest version should help

Updating to go 1.9 fixed this problem. Thanks @bhaskarsaraogi


Need suggestion to use cucumber tags efficiently with Serenity

Hi Jan,
Thanks for letting me steal some working minutes from your busy schedule. It would be great if i get some clarification/advise for the below areas:
Scenario 1:
Suppose, i have 3 scenarios. Now, I want to execute 3 scenarios in chrome , but just 2 with firefox, then do we have tags/annotation in browser level like: @chome/@chromeOnly?
Scenario 2:
Suppose, I have 3 scenarios and 3 scenarios have tags like @tag1, @tag2, and @tag3. If I execute all the scenarios, my serenity HTML report will contain all the tags. Do we have any hack to tell serenity CLI to exclude or include some specific tags in the reports. I mean , I want to run all those scenarios, but just want to have @tag1 in the HTML report.
If this use case is already handled, then can we use regular expression to tell serenity CLI to include/exclude tags from the report?
Note: I am still using the stable version, i.e, Serenity 1.x. So, any suggestion from 1.x context will save my life.
Many thanks in advance.

@jan-molak any help here?


hash every git files

Describing the files to hash seems to be redundant with the gitignore. We have not been able to identify a scenario in which we would want to customize the hashGlob so we removed this option.
Also we don't need to use globby to query the hash files because they are already contained in the repo-info

Just saw this come through in release! Thanks!!


Release view

Add a view that shows all features per release

modify the existing 'timeline' view?

implemented!


renamed existing aks cluster

@gison93 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#sign-off

@j-martens: I'm sorry - only the author of this article, @csteegz, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.


Feature/cu 1da49r4 glitchy gif menu title

Established menu with glitchy title.
4 different animations (Title in, Idle, Glitch, Title out)
Established Fade to back on pressed Play, Can be tested in StartSceneGlitchyTitle.scene

Task linked: CU-1da49r4 Replace Start Menu Shutdown icon with the new Glitchy Gif

Several things to review, it does not follow the Notes of the .txt file:
1- There's not fade in when on Level Start
2- Check LOGO VIBRATION Notes because there isn't any Sound playing atm. Use placeholders if needed.
3- LOGO OUTRO is not implemented

Fade in established
Placeholders set, no clips on AudioSources yet, but AudioSources established
Logo outro was implemented, only that delay between it and the Fade out hadn't been set, this has been fixed


Get-KeePassEntry not Converting Field Reference's

First: This thing is awesome!
Second: I am having issues with my Username and Password fields that contain references to other things. I am not to sure if it is possible to add this feature.

@ironScripter Thank you can you elaborate on references to other things, I have not used keepass in this way. Also what kinda issues are you having?

@jkdba Sorry for the late reply. Work has been very hectic.
If you are in KeePass and you open up any entry, in the bottom left corner click tools in and then "insert field reference". From there you can insert a field reference that will allow multiple entries to utilize the same username or password from another entry. I use auto type heavily and this is a life saver.
Let me know if you need anymore info

@ironScripter I will check it out thanks for the follow up


BUG: csv w only header and no index_col failing

after this fix https://github.com/pydata/pandas/pull/10302
The following is failing (as index_col is None)
In [9]: data
Out[9]: 'header1,header2\n'

In [10]: read_csv(StringIO(data),iterator=True,index_col=False).read()
TypeError: object of type 'bool' has no len()

I'm having the same issue with version 0.16.2. When reading a csv file that contains only headers (no data), for example:
first row: company;supplier;amount
And using pandas read_csv function to read the file (delimiter=";"; skiprows=0)
df = pd.read_csv(file, sep=delimiter, engine="c", quotechar='"', quoting=quote, skipinitialspace=True, skiprows=skiprows, prefix="NX", encoding="ISO-8859-1", error_bad_lines=False, index_col=False)
It returns the following error:
TypeError: object of type 'bool' has no len()
Removing the option index_col=False it starts working:
df = pd.read_csv(file, sep=delimiter, engine="c", quotechar='"', quoting=quote, skipinitialspace=True, skiprows=skiprows, prefix="NX", encoding="ISO-8859-1", error_bad_lines=False)


Add chapter on attributes.

@arthurpaulino Suggestions on what we want to teach when it comes to attributes would be nice. I haven't used attributes much, so I'm learning as I go along!

Superseded by #71


chore(weave): Project sidebar takes in models access param and always shows Overview tab

Description

Block Models tabs for weave only user but still show the Overview tab.
Testing
How was this PR tested?

I have read the CLA Document and I hereby sign the CLA


Task - OTU radial - biological association.

I created one biological association from a particular source. It is properly saved with a citation
I trying to create a second association for the same taxon from the same source (without reloading the OTU radial). The association is saved, but the citation is not created.

The problem here is when you create a biological association the citation is removed but this is not reflected in the interface. Thats the reason why when you create a new one the citation is not added even when you see it on the citation section.
@mjy @proceps what do you want here? clean the interface or leave the citation there after a biological association is created?

@jlpereira Clean it on create

For me, preffered solution both here and in distribution section would be to add a lock icon to lock the source...


cant connect to server

Client connects to server but on server listener just disconnect method raises and doesn't receive any object.
i use GenyMotion on client side and real phone on server side.
i do everything exactly like documentation said
thanx

If i understand correctly you haven't started your client on a new thread. Instead of client = new Client();  use new Thread(client);


"not yet capable of opening new tabs" per command line arguments documentation

Type of issue
Other (describe below)
Feedback
According to the note at the end of the "Open a new profile instance" section of the command line arguments documentation, it is not possible to open a new tab:

The -p flag is used to specify the Windows Terminal profile that should be opened. Substitute "Ubuntu-18.04" with the name of any terminal profile that you have installed. This will always open a new window. Windows Terminal is not yet capable of opening new tabs or panes in an existing instance.

I assume this is an oversight in the documentation, since wt nt exists, unless there is some nuance I am missing?
Page URL
https://learn.microsoft.com/en-us/windows/terminal/command-line-arguments?tabs=windows#open-a-new-profile-instance
Content source URL
https://github.com/MicrosoftDocs/terminal/blob/main/TerminalDocs/command-line-arguments.md#open-a-new-profile-instance
Author
nguyen-dows
Document Id
bd211ca4-58c5-fdb6-93c4-75b3ea7aa34f

Hey @gilly3  - Sounds like this is a semantics issue. This sentence is pointing out that a new tab can't be opened in an existing instance of an already open Terminal. As you point out, wt nt will open a new instance with a new Terminal tab.

You know, I'm gonna reactivate this. We should just remove

Windows Terminal is not yet capable of opening new tabs or panes in an existing instance.

because it totally is possible to open new tabs in existing windows (via wt -w ...)

Thanks @zadjii-msft. Wasn't aware of this change - I'll get this update in the doc.

@zadjii-msft When I try wt -w 0 "Command Prompt (or any number), I get a new tab opened in the existing terminal window, but with error<PHONE_NUMBER> (0x80070002)... am I doing something wrong? I'm running version 1.21.2911.0

Ah, that's an intricacy of Terminal's interaction model.
If you are launching a profile (combined color scheme, title, command, other settings pertaining to how it interacts with the console session, etc.), you use wt [other args] -p "Profile Name".
If you're launching an executable (cmd.exe, powershell.exe), you use wt [other args] command.
In your case, you'll want either wt -w 0 -p "Command Prompt" or wt -w 0 cmd.

Ahhhh! Thanks @DHowett! I'll get this in the doc.


Sign in with Apple | Invalid nonce

Environment

Xcode version: 11.6
Firebase SDK version: 6.24.0
Firebase Component: Auth
Component version: 6.5.3
Installation method: CocoaPods

Problem with Apple Sign in
I'm getting the invalid nonce error even my nonce does match...
Code=17094 "The nonce in ID Token "8648160ed87a74ce1053994704012f7066c0a9fdce5cf2a8967fa19fd2082aa0" does not match the SHA256 hash of the raw nonce "8648160ed87a74ce1053994704012f7066c0a9fdce5cf2a8967fa19fd2082aa0" in the request."
Relevant Code:
func loginWithApple() {
    let appleIDProvider = ASAuthorizationAppleIDProvider()
      let request = appleIDProvider.createRequest()
      request.requestedScopes = [.fullName, .email]
      currentNonce = sha256(randomNonceString())
      request.nonce = currentNonce

      let authorizationController = ASAuthorizationController(authorizationRequests: [request])
      authorizationController.delegate = self
      authorizationController.presentationContextProvider = self
      authorizationController.performRequests()
  }

private func randomNonceString(length: Int = 32) -> String {
    precondition(length > 0)
    let charset: Array<Character> =
        Array("0123456789ABCDEFGHIJKLMNOPQRSTUVXYZabcdefghijklmnopqrstuvwxyz-._")
    var result = ""
    var remainingLength = length

    while remainingLength > 0 {
      let randoms: [UInt8] = (0 ..< 16).map { _ in
        var random: UInt8 = 0
        let errorCode = SecRandomCopyBytes(kSecRandomDefault, 1, &random)
        if errorCode != errSecSuccess {
          fatalError("Unable to generate nonce. SecRandomCopyBytes failed with OSStatus \(errorCode)")
        }
        return random
      }

      randoms.forEach { random in
        if remainingLength == 0 {
          return
        }

        if random < charset.count {
          result.append(charset[Int(random)])
          remainingLength -= 1
        }
      }
    }

    return result
  }

  private func sha256(_ input: String) -> String {
    let inputData = Data(input.utf8)
    let hashedData = SHA256.hash(data: inputData)
    let hashString = hashedData.compactMap {
      return String(format: "%02x", $0)
    }.joined()

    return hashString
  }

      func authorizationController(controller: ASAuthorizationController, didCompleteWithAuthorization authorization: ASAuthorization) {
    if let appleIDCredential = authorization.credential as? ASAuthorizationAppleIDCredential {
      guard let nonce = currentNonce else {
        fatalError("Invalid state: A login callback was received, but no login request was sent.")
      }
      guard let appleIDToken = appleIDCredential.identityToken else {
        print("Unable to fetch identity token")
        return
      }
      guard let idTokenString = String(data: appleIDToken, encoding: .utf8) else {
        print("Unable to serialize token string from data: \(appleIDToken.debugDescription)")
        return
      }

      // Initialize a Firebase credential.
      let credential = OAuthProvider.credential(withProviderID: "apple.com",
                                                idToken: idTokenString,
                                                rawNonce: nonce)

      // Sign in with Firebase.
      Auth.auth().signIn(with: credential) { (authResult, error) in
        if let error = error {
          // Error. If error.code == .MissingOrInvalidNonce, make sure
          // you're sending the SHA256-hashed nonce as a hex string with
          // your request to Apple.
          print(error.localizedDescription)
          return
        }else{
          print("")
        }
        // User is signed in to Firebase with Apple.
        // ...
      }
}

Maybe this StackOverflow answer will help?

Maybe this StackOverflow answer will help?

I already checked this, but i'm already on latest version 6.5. However the signature in documentation and xCode is slightly different.

I was sending hashed nonce instead of raw nonce... my bad.

No problem, glad you were able to solve the issue!


Onboarding wizard has no error message on failure

Describe the bug
Onboarding wizard has no error message on failure
To Reproduce
Steps to reproduce the behavior:

Go to Overview
Click on 'Add a Source'
Fill out details to add OCP source (remembering name/cluster ID)
Click continue to add source
Add a new source with the same name/ID
Red cross for Confirmation Status

Expected behavior
There is two issues to tackle here, if we get a Confirmation Status as a Green tick we need some details such as:
You have successfully setup X Source with the following details:
Type
Name
Cluster identifier
Or if Confirmation Status is a Red cross (Failure) we need details on what went wrong and why,
Such as:
Configuration for X Source failed because:
Already have X Name or Cluster identifier?
Screenshots

Desktop (please complete the following information):

OS: Fedora
Browser Chrome
Version 66

Additional context
This was running against CI insightsbeta

That's awkward that you don't see any message. I will look at this.
In addition, according to the design I was given the status message should be "Successfully added" or "Something went wrong".
Nonetheless, I do think what you're suggesting is better IMHO.
Thanks.

Yep - I see why it happened. Also the animations are not working. I apparently overridden this when I rebased. I am going to work on it and probably will push a fix in ~3 hours


[Quarkus 3.5.0] Kafka container fails to start when configured for SSL

Bug description
Quarkus upgraded Testcontainers to 1.19.0. One of the new features for Kafka is enabling additional listeners to be configured:
https://java.testcontainers.org/modules/kafka/
Something has changed where the current container config for SSL is no longer working. Hopefully someone smarter than me can figure out what needs modifying for these tests:

kafka-ssl
kafka-sasl
kafka-sasl-ssl

I am disabling them for now.

It should be fixed by moving all of withEnv(...); to the configuration() method. I will prepare a PR.


Add Other Coin Data

Hello,
I've been using this and modifying it for my use in a dashboard. Obviously there are cross exchange inefficiencies in the markets. One other useful API I've been using is https://coinmarketcap.com/ https://coinmarketcap.com/api/ which provides details I find useful such as:

rank
24h_volume_usd
total_supply
percent_change_24h
percent_change_7d

I was wondering if you had an interest in adding a feature to get other aspects of market data too? If you are I'd be more than happy to prepare the pull request and add the features in. Just didn't know if that was something you'd find useful too?
Let me know.

I have bittrex support working on my fork but i havent uploaded it to github i would like to merge with this project to add more exchange functionality to this repo

@radeone if you'd like we @sapphiredev has merged some pull requests. Open one if it goes unanswered let's see if we can sync up and fork and collaborate.

i have written some more fixes feel free to commit anything to master since i dont have access to @sapphiredev repo or yours


serializers: addition of generic fields

Addresses https://github.com/inveniosoftware/cookiecutter-invenio-datamodel/issues/4
Signed-off-by: Dinos Kousidis<EMAIL_ADDRESS>

Can I ask you to add some tests to avoid lowering the code coverge? I'm not sure there's tests elsewhere already written - but e.g. here you can find some problematic unicode characters: https://github.com/zenodo/zenodo/blob/master/tests/unit/records/test_schemas_legacyjson_load.py#L826-L836


Potentially improve dataset caching by caching individual results / metrics that are requested together

See this todo in DataManager.ts:

I think this probably isn't worth doing since I think it will add more complexity than it will help. Maybe we should just delete the TODO or turn into an FYI comment


Minor refactoring

Title
Minor refactoring
Summary
Change New ValidationResult(True, Nothing) to ValidationResult.ValidResult.
Details
Change New ValidationResult(True, Nothing) to ValidationResult.ValidResult, since it looks a little cleaner and the property was explicitly designed for that use.

Changes look good, thanks @Danghor! But I don't see this example being used anywhere @rpetrusha. I think it should be referred from https://docs.microsoft.com/en-us/dotnet/framework/wpf/data/how-to-implement-binding-validation and match the C# version that we show. We should create a separate issue for this.


fix: removed duplicate file

Describe your changes
EnumFlags.hpp was in two places. Removed duplicate file
Issue ticket number and link
Checklist before requesting a review

[x] I have performed a self-review of my code
[ ] I have made corresponding changes to the documentation
[x] I read the contribution documentation and made sure that my changes meet its requirements

The bazel build fails because EnumFlags.hpp is missing. So you should change the bazel stuff, too.

@james-ctc we can probably help fix bazel but removing it to "fix" cannot a serious idea

@james-ctc we can probably help fix bazel but removing it to "fix" cannot a serious idea

please feel free to add a fix - I don't know bazel at all.


Component not updating although the DOM upon the return of a response

Hello,
I have encountered an issue for which I don't know if it's a bug or my miscomprehension of the Hydro's concept.
To illustrate the situation, I will use a simple component that displays a list of items:

        @foreach (var item in Model.Items)
        {

@item

        }

        <button type="button" hydro-on:click="@(()=>Model.Remove())">Remove first</button>

Then in the model there is a property containing a list of items and a method that removes the first element of the list.
public List<string> Items { get; set; }

public override void Mount()
{
    Items = ["Item 1", "Item 2", "Item 3"];
}

public void Remove()
{
    Items.RemoveAt(0);
}

When the button is clicked and the Remove method is triggered, I would expect the component to be swapped in the DOM. However, this does not happen although the response contains a new
 holding just the Item 2 and Item 3
 elements.
Thanks a lot!
Nejc

Hi Nejc!
I've just checked your component and it worked for me. I clicked the button and the first item got removed. Did you follow all the instructions from the Getting started?
Krzysztof

Cześć,
My bad, I had my own Alpine.js script and no Morph plugin!
Bardzo bardzo Ci dziękuję za odpowiedź!
Nejc

Hej, nie ma problemu :)
KJ


Can not configure API_KEY= in a .env file

Can not find these comments
configure API_KEY= in a .env file
optionally, edit splunk_easm_worker/subfinder-provider-config.yaml

Sorry - step-by-step documentation is on its way.
subfinder-provider-config.yaml doesn't need to exist, but you can choose to populate it with API keys for common OSINT sources to improve discovery. See https://github.com/projectdiscovery/subfinder/blob/main/README.md#post-installation-instructions for an example.
For the API_KEY - used by Splunk to communicate with the worker, just create a .env file in the root folder i.e. the same one as the docker-compose.yml file, which references the .env file.


Ensure cloudfront distribution ViewerProtocolPolicy is set to HTTPS

Possible check to target the following resource

[ ] aws_cloudfront_distribution

This check should fail on default_cache_behavior as well as on all instances of ordered_cache_behavior with allow-all configured in viewer_protocol_policy


Remove MDN entry for :is()

Caniuse seems to have duplicate entries for CSS :is(), so I'm thinking the MDN entry should be removed: https://caniuse.com/?search=%3Ais()
It just caused some confusion because the MDN table doesn't seem to be up to date: https://twitter.com/edwardakerboom/status/1411955740932251648

Have made the change, though it'll only take effect next time I update the MDN data. Thanks for pointing that out!

Seems like it has been removed on the site now 🥳


one issues

hi, i want to say one issues.
i try to run this program,
in  stereomag/loader.py ,  178 line ,
"  sequences_with_images = sequence_with_images.map"
I suspect something is wrong here.
Is it  "sequence_with_images"  not miss written?
Your favorite fans! Looking forward to your reply.!

Thanks for the comment. This has been fixed in this commit: https://github.com/google/stereo-magnification/commit/d816db2e2ba4ea66345885d15681f6b02c548b04


json-c: 0.12 -> 0.12.1

Motivation for this change
new version, no longer needs patch
Things done

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

[ ] NixOS
[ ] macOS
[X] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Fits CONTRIBUTING.md.

whoops, sorry. I guess I was a bit too enthusiastic :-)


Compact bashly.yml

I noticed the bashly.yml file contains multiple "code duplications".
I fixed it using YAML anchors.
Feel free to discard if you don't like it.
(...and I love the --nuke-from-orbit flag :)
Also, another minor feature of bashly you might want to use:
When the command's help message contains more than one lines, it will be displayed more nicely.

The first line will be used for the summary (when running without arguments)
The full block will be displayed nicely indented when running the command's specific --help

- name: upload
  help: |
    Bulk upload locally-stored emojis to a Slack workspace.
    This is a function that requires additional permissions.

This is excellent! Thank you so much!
It's easy to forget it's a YAML document which has all the benefits of YAML. I just didn't make that mental leap. It's so obvious now that I look at it.
Thanks for the tips as well! I'll merge this as soon as I'm able. :)


EC2 Image Builder support for container images

Hi,
I recently contributed an artifact to AWS internally and wanted to contribute a modified version of it to this repo since I used the other samples here as a reference for my own. I have created a PR that adds a README, template, and JSON parameters file to a new directory I created as part of the CloudFormation directory. I tried to model my README and CloudFormation template the same as the others for simplicity. Additionally, my component document uses a very basic example that mimics the AWS Documentation. I would be more than happy to modify this into something more advanced like the .NET example for Ubuntu. Let me know if you have any additional questions or potential modifications to improve. If you don't feel that my contributions would be of value in this repo that's ok but figured I would check anyways!

Merged PR. Thanks for the submission!


Update README.md

Fix download link for publicsuffix

Looks like there's already a PR for this :)


cannot stat file when running the repo synthesis example

When running:
$ python FINN/bin/finn --device=pynqz1 --prototxt=FINN/inputs/cnv-w1a1.prototxt --caffemodel=FINN/inputs/cnv-w1a1.caffemodel --mode=synth
I get the following error:
cp: cannot stat '/tmp/finn-build-DpC6wt/finnaccel/finnaccel.runs/impl_1/procsys_wrapper.bit': No such file or directory Traceback (most recent call last):   File "FINN/bin/finn", line 192, in <module>     process_args(args)   File "FINN/bin/finn", line 92, in process_args     generate_hardware(net, dev, gen_bitfile=True)   File "FINN/bin/finn", line 161, in generate_hardware     ret.synthesis_bitfile()	   File "/mnt/terabyte/pmousoul_data/sw/FINN/FINN/backend/fpga/backend_util.py", line 71, in synthesis_bitfile     subprocess.check_call(["sh", self.getBitfileSynthScriptPath()], cwd=self.path)   File "/usr/lib/python2.7/subprocess.py", line 541, in check_call     raise CalledProcessError(retcode, cmd) subprocess.CalledProcessError: Command '['sh', '/tmp/finn-build-DpC6wt/make_pynq_bitfile.sh']' returned non-zero exit status 1
Any idea on how to resolve this?

This looks like Vivado isn't in your path. Can you confirm that it is?

Hello,
before running "sh env.sh" in FINN root directory
$ echo $PATH
/usr/local/cuda-9.2/bin:/mnt/terabyte/pmousoul_data/sw/Xilinx/petalinux/tools/linux-i386/petalinux/bin:/mnt/terabyte/pmousoul_data/sw/Xilinx/petalinux/tools/common/petalinux/bin:/mnt/terabyte/pmousoul_data/sw/Xilinx/petalinux/tools/linux-i386/gcc-arm-none-eabi-r5/bin:/mnt/terabyte/pmousoul_data/sw/Xilinx/petalinux/tools/linux-i386/microblaze-xilinx-elf/bin:/mnt/terabyte/pmousoul_data/sw/Xilinx/petalinux/tools/linux-i386/microblazeel-xilinx-linux-gnu/bin:/mnt/terabyte/pmousoul_data/sw/Xilinx/petalinux/tools/linux-i386/gcc-arm-none-eabi/bin:/mnt/terabyte/pmousoul_data/sw/Xilinx/petalinux/tools/linux-i386/gcc-arm-linux-gnueabi/bin:/mnt/terabyte/pmousoul_data/sw/Xilinx/petalinux/tools/linux-i386/aarch64-none-elf/bin:/mnt/terabyte/pmousoul_data/sw/Xilinx/petalinux/tools/linux-i386/aarch64-linux-gnu/bin:/mnt/terabyte/pmousoul_data/sw/Xilinx/Vivado/2017.2/bin:/mnt/terabyte/pmousoul_data/sw/Xilinx/Vivado_HLS/2017.2/bin:/mnt/terabyte/pmousoul_data/sw/Xilinx/DocNav:/mnt/terabyte/pmousoul_data/sw/Xilinx/SDK/2017.2/bin:/mnt/terabyte/pmousoul_data/sw/Xilinx/SDK/2017.2/gnu/microblaze/lin/bin:/mnt/terabyte/pmousoul_data/sw/Xilinx/SDK/2017.2/gnu/arm/lin/bin:/mnt/terabyte/pmousoul_data/sw/Xilinx/SDK/2017.2/gnu/microblaze/linux_toolchain/lin64_be/bin:/mnt/terabyte/pmousoul_data/sw/Xilinx/SDK/2017.2/gnu/microblaze/linux_toolchain/lin64_le/bin:/mnt/terabyte/pmousoul_data/sw/Xilinx/SDK/2017.2/gnu/aarch32/lin/gcc-arm-linux-gnueabi/bin:/mnt/terabyte/pmousoul_data/sw/Xilinx/SDK/2017.2/gnu/aarch32/lin/gcc-arm-none-eabi/bin:/mnt/terabyte/pmousoul_data/sw/Xilinx/SDK/2017.2/gnu/aarch64/lin/aarch64-linux/bin:/mnt/terabyte/pmousoul_data/sw/Xilinx/SDK/2017.2/gnu/aarch64/lin/aarch64-none/bin:/mnt/terabyte/pmousoul_data/sw/Xilinx/SDK/2017.2/gnu/armr5/lin/gcc-arm-none-eabi/bin:/mnt/terabyte/pmousoul_data/sw/Xilinx/SDK/2017.2/tps/lnx64/cmake-3.3.2/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games:/usr/local/games
after running "sh env.sh" in FINN root directory
$ echo $PATH
/usr/local/cuda-9.2/bin:/mnt/terabyte/pmousoul_data/sw/Xilinx/petalinux/tools/linux-i386/petalinux/bin:/mnt/terabyte/pmousoul_data/sw/Xilinx/petalinux/tools/common/petalinux/bin:/mnt/terabyte/pmousoul_data/sw/Xilinx/petalinux/tools/linux-i386/gcc-arm-none-eabi-r5/bin:/mnt/terabyte/pmousoul_data/sw/Xilinx/petalinux/tools/linux-i386/microblaze-xilinx-elf/bin:/mnt/terabyte/pmousoul_data/sw/Xilinx/petalinux/tools/linux-i386/microblazeel-xilinx-linux-gnu/bin:/mnt/terabyte/pmousoul_data/sw/Xilinx/petalinux/tools/linux-i386/gcc-arm-none-eabi/bin:/mnt/terabyte/pmousoul_data/sw/Xilinx/petalinux/tools/linux-i386/gcc-arm-linux-gnueabi/bin:/mnt/terabyte/pmousoul_data/sw/Xilinx/petalinux/tools/linux-i386/aarch64-none-elf/bin:/mnt/terabyte/pmousoul_data/sw/Xilinx/petalinux/tools/linux-i386/aarch64-linux-gnu/bin:/mnt/terabyte/pmousoul_data/sw/Xilinx/Vivado/2017.2/bin:/mnt/terabyte/pmousoul_data/sw/Xilinx/Vivado_HLS/2017.2/bin:/mnt/terabyte/pmousoul_data/sw/Xilinx/DocNav:/mnt/terabyte/pmousoul_data/sw/Xilinx/SDK/2017.2/bin:/mnt/terabyte/pmousoul_data/sw/Xilinx/SDK/2017.2/gnu/microblaze/lin/bin:/mnt/terabyte/pmousoul_data/sw/Xilinx/SDK/2017.2/gnu/arm/lin/bin:/mnt/terabyte/pmousoul_data/sw/Xilinx/SDK/2017.2/gnu/microblaze/linux_toolchain/lin64_be/bin:/mnt/terabyte/pmousoul_data/sw/Xilinx/SDK/2017.2/gnu/microblaze/linux_toolchain/lin64_le/bin:/mnt/terabyte/pmousoul_data/sw/Xilinx/SDK/2017.2/gnu/aarch32/lin/gcc-arm-linux-gnueabi/bin:/mnt/terabyte/pmousoul_data/sw/Xilinx/SDK/2017.2/gnu/aarch32/lin/gcc-arm-none-eabi/bin:/mnt/terabyte/pmousoul_data/sw/Xilinx/SDK/2017.2/gnu/aarch64/lin/aarch64-linux/bin:/mnt/terabyte/pmousoul_data/sw/Xilinx/SDK/2017.2/gnu/aarch64/lin/aarch64-none/bin:/mnt/terabyte/pmousoul_data/sw/Xilinx/SDK/2017.2/gnu/armr5/lin/gcc-arm-none-eabi/bin:/mnt/terabyte/pmousoul_data/sw/Xilinx/SDK/2017.2/tps/lnx64/cmake-3.3.2/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games:/usr/local/games
it seems that "/mnt/terabyte/pmousoul_data/sw/Xilinx/Vivado/2017.2/bin" exists in the PATH
Thank you for your time,
Panos

Did vivado run? It would have taken a while to produce the bit file. Or did this error show up immediately?

output.log
From the attached file, it seems that Vivado runs but the bit file is not generated.

Looks like it fails. You could try to find the specific error in the vivado logs as to why it doesn't produce a bit file.

vivado.log
There is no error in the vivado.log found in "/tmp/finn-build-JoqpZl/" which is attached.
Also, none of the log files referenced at the end of the vivado.log file exist:
`[Thu Apr  4 21:55:42 2019] Launched procsys_auto_pc_0_synth_1, procsys_auto_pc_1_synth_1, procsys_rst_ps7_100M_0_synth_1, procsys_BlackBoxJam_0_0_synth_1, procsys_ps7_0_synth_1, synth_1...
Run output will be captured here:
procsys_auto_pc_0_synth_1: /tmp/finn-build-JoqpZl/finnaccel/finnaccel.runs/procsys_auto_pc_0_synth_1/runme.log
procsys_auto_pc_1_synth_1: /tmp/finn-build-JoqpZl/finnaccel/finnaccel.runs/procsys_auto_pc_1_synth_1/runme.log
procsys_rst_ps7_100M_0_synth_1: /tmp/finn-build-JoqpZl/finnaccel/finnaccel.runs/procsys_rst_ps7_100M_0_synth_1/runme.log
procsys_BlackBoxJam_0_0_synth_1: /tmp/finn-build-JoqpZl/finnaccel/finnaccel.runs/procsys_BlackBoxJam_0_0_synth_1/runme.log
procsys_ps7_0_synth_1: /tmp/finn-build-JoqpZl/finnaccel/finnaccel.runs/procsys_ps7_0_synth_1/runme.log
synth_1: /tmp/finn-build-JoqpZl/finnaccel/finnaccel.runs/synth_1/runme.log
[Thu Apr  4 21:55:42 2019] Launched impl_1...
Run output will be captured here: /tmp/finn-build-JoqpZl/finnaccel/finnaccel.runs/impl_1/runme.log
launch_runs: Time (s): cpu = 00:00:09 ; elapsed = 00:00:09 . Memory (MB): peak = 1444.164 ; gain = 119.699 ; free physical = 7816 ; free virtual = 16449
wait_on_run impl_1
[Thu Apr  4 21:55:42 2019] Waiting for impl_1 to finish...
[Thu Apr  4 21:56:00 2019] impl_1 finished
wait_on_run: Time (s): cpu = 00:00:16 ; elapsed = 00:00:18 . Memory (MB): peak = 1444.164 ; gain = 0.000 ; free physical = 6181 ; free virtual = 14815
INFO: [Common 17-206] Exiting Vivado at Thu Apr  4 21:56:00 2019...`

So the reason that they would not exist is that there is a failure before they are written. You could zip up the files and link me to them.

There you go:
https://drive.google.com/open?id=1-yHFciAnaovpKqarPjCs0CGiA6QpHjBw

I've run the tool with your command using Vivado 2018.3 and it works fine for me. For you it's failing to generate "procsys_wrapper.bit". Try deleting the hls_syn and finnaccel directories and running  ./make_pynq_bitfile again.

Hello,
I did what you said and I got:
``
wait_on_run impl_1
[Sun Apr 21 21:19:27 2019] Waiting for impl_1 to finish...
[Sun Apr 21 21:19:42 2019] impl_1 finished
wait_on_run: Time (s): cpu = 00:00:14 ; elapsed = 00:00:15 . Memory (MB): peak = 1445.414 ; gain = 0.000 ; free physical = 4931 ; free virtual = 16481
INFO: [Common 17-206] Exiting Vivado at Sun Apr 21 21:19:42 2019...
cp: cannot stat '/tmp/finn-build-E1vW3P/finnaccel/finnaccel.runs/impl_1/procsys_wrapper.bit': No such file or directory
``
using vivado 2017.2.
I got a more elaborate output related to the problem with vivado 2018.2:
``
Starting Placer Task
INFO: [Place 46-5] The placer was invoked with the 'ExtraTimingOpt' directive.
INFO: [Place 30-611] Multithreading enabled for place_design using a maximum of 8 CPUs
Phase 1 Placer Initialization
Phase 1.1 Placer Initialization Netlist Sorting
Netlist sorting complete. Time (s): cpu = 00:00:00.06 ; elapsed = 00:00:00.07 . Memory (MB): peak = 3568.453 ; gain = 0.000 ; free physical = 5577 ; free virtual = 15788
Phase 1.1 Placer Initialization Netlist Sorting | Checksum: f8841624
Time (s): cpu = 00:00:00.07 ; elapsed = 00:00:00.11 . Memory (MB): peak = 3568.453 ; gain = 0.000 ; free physical = 5577 ; free virtual = 15788
INFO: [Timing 38-35] Done setting XDC timing constraints.
INFO: [Opt 31-138] Pushed 0 inverter(s) to 0 load pin(s).
Netlist sorting complete. Time (s): cpu = 00:00:00.07 ; elapsed = 00:00:00.07 . Memory (MB): peak = 3568.453 ; gain = 0.000 ; free physical = 5578 ; free virtual = 15790
Phase 1.2 IO Placement/ Clock Placement/ Build Placer Device
ERROR: [Place 30-640] Place Check : This design requires more RAMB36/FIFO cells than are available in the target device. This design requires 144 of such cell types but only 140 compatible sites are available in the target device. Please analyze your synthesis results and constraints to ensure the design is mapped to Xilinx primitives as expected. If so, please consider targeting a larger device.
ERROR: [Place 30-640] Place Check : This design requires more RAMB18 and RAMB36/FIFO cells than are available in the target device. This design requires 384 of such cell types but only 280 compatible sites are available in the target device. Please analyze your synthesis results and constraints to ensure the design is mapped to Xilinx primitives as expected. If so, please consider targeting a larger device.
ERROR: [Place 30-640] Place Check : This design requires more RAMB36E1 cells than are available in the target device. This design requires 144 of such cell types but only 140 compatible sites are available in the target device. Please analyze your synthesis results and constraints to ensure the design is mapped to Xilinx primitives as expected. If so, please consider targeting a larger device.
INFO: [Timing 38-35] Done setting XDC timing constraints.
Phase 1.2 IO Placement/ Clock Placement/ Build Placer Device | Checksum: 1497648bb
Time (s): cpu = 00:00:15 ; elapsed = 00:00:09 . Memory (MB): peak = 3568.453 ; gain = 0.000 ; free physical = 5480 ; free virtual = 15692
Phase 1 Placer Initialization | Checksum: 1497648bb
Time (s): cpu = 00:00:15 ; elapsed = 00:00:09 . Memory (MB): peak = 3568.453 ; gain = 0.000 ; free physical = 5480 ; free virtual = 15692
ERROR: [Place 30-99] Placer failed with error: 'Implementation Feasibility check failed, Please see the previously displayed individual error or warning messages for more details.'
Please review all ERROR, CRITICAL WARNING, and WARNING messages during placement to understand the cause for failure.
Ending Placer Task | Checksum: 1497648bb
Time (s): cpu = 00:00:15 ; elapsed = 00:00:09 . Memory (MB): peak = 3568.453 ; gain = 0.000 ; free physical = 5487 ; free virtual = 15699
INFO: [Common 17-83] Releasing license: Implementation
60 Infos, 21 Warnings, 0 Critical Warnings and 5 Errors encountered.
place_design failed
ERROR: [Common 17-69] Command failed: Placer could not place all instances
INFO: [Common 17-206] Exiting Vivado at Sun Apr 21 21:39:02 2019...
[Sun Apr 21 21:39:02 2019] impl_1 finished
wait_on_run: Time (s): cpu = 00:11:10 ; elapsed = 00:09:08 . Memory (MB): peak = 1700.316 ; gain = 0.000 ; free physical = 6770 ; free virtual = 16982
INFO: [Common 17-206] Exiting Vivado at Sun Apr 21 21:39:02 2019...
cp: cannot stat '/tmp/finn-build-_ub5ug/finnaccel/finnaccel.runs/impl_1/procsys_wrapper.bit': No such file or directory
Traceback (most recent call last):
File "FINN/bin/finn", line 192, in
process_args(args)
File "FINN/bin/finn", line 92, in process_args
generate_hardware(net, dev, gen_bitfile=True)
File "FINN/bin/finn", line 161, in generate_hardware
ret.synthesis_bitfile()
File "/mnt/terabyte/pmousoul_data/Repos/FINN/FINN/backend/fpga/backend_util.py", line 71, in synthesis_bitfile
subprocess.check_call(["sh", self.getBitfileSynthScriptPath()], cwd=self.path)
File "/usr/lib/python2.7/subprocess.py", line 541, in check_call
raise CalledProcessError(retcode, cmd)
subprocess.CalledProcessError: Command '['sh', '/tmp/finn-build-_ub5ug/make_pynq_bitfile.sh']' returned non-zero exit status 1
``

Hello,
So the placer has failed. You will need to scale back "BRAM_PROPORTION = 1" to a lower value, 0.9 or 0.8.
https://github.com/Xilinx/FINN/blob/e6d44508b4f7c6ac8e2c6929f2ffa85ea5761708/FINN/core/config.py#L52
The hardware cost model does not match the output of the tools when using versions of Vivado later than those with which we built the tool.

I'm closing all issues relating to the v.01 version in preparation for the new release. Please note that v0.1 is now deprecated and unsupported.


[Feature Request] Scaffolding with degit

consider adding https://github.com/Rich-Harris/degit

close


[9.x] Adds the ability to delay retries on database transactions

Howdy guys!
When performing a database transaction with attempts to avoid deadlocks, it is often desirable to wait a certain number of milliseconds before retrying.
Inspired by this tweet https://twitter.com/mattkingshott/status/1463572180546068490?s=20, I thought it would be nice in Laravel 9 to support this on the method itself.
Here's the before and after:
// Before
retry(3, fn() => DB::transaction(User::update(['first_name' => 'Taylor'])), 250);

// After
DB::transaction(User::update(['first_name' => 'Taylor']), 3, 250);

Obviously, this needs to target 9.x because it affects the signature of the Connection interface.
On a side note, has the value function been altered? It no longer seems to pass properties, but I could be missing something.
Thanks for all the continued hard work.
Kind Regards,
Luke

Yeah, better docs around this would be a good idea @driesvints. I guess if it is going to cause issues for a lot of 3rd party libs, perhaps that would be better 👍 Just wouldn't want the solution to go un-missed by those not on social media etc if that makes sense?

The benefit of the built-in way is that it works also when using illuminate/database directory, where the global helpers are not available (because they're part of laravel/framework).
But besides the signature (thus: breaking) change, I feel like the current transaction method is very concise and should not be changed that particular way (though, unfortunately, I don't have a better suggestion at hand 😅)

Hi there, we've temporarily disabled our PostgreSQL build until we've managed to fix it. I've marked your PR as draft for now. Please rebase with master and mark this PR as ready when all tests pass again. Thanks.

I think instead of introducing the breaking change I'll just let people use the existing retry helper approach.


[Feature Request] Arbitrary Environment Inheritance

Insomnia Version: <IP_ADDRESS>6
Operating System: 10.13.4

Details
I have noticed the need to arbitrarily extend sub-environments. The use case is as follows: I have sub-environments setup for TST/STG/PRD for example. Each of those sub-environments contains information such as URLs. This is something that's okay to sync across members of my team. But I also have information that I'd like to change only on my machine, such as usernames and credentials that I do not want to sync across. Credentials are pretty obvious in why I don't want to sync them, as they're sensitive information. But I need to change things like what user I'm using and other user information to test how my services respond to different users. This is something I do not want to be synced since this is something relevant only to my local development.
The ability to arbitrarily extend sub-environments would solve the above use case, then I'd be able to extend my TST environment with a private sub-environment with information I'm attempting to test and I would be able to change regularly without impeding my team members.

I'll get a PR up this weekend for this. I have some ideas on how to implement this.

That sounds like this, which I've been wanting to implement for a while. https://github.com/getinsomnia/insomnia/issues/283
Was that similar to what you were thinking?

Yeah, that's very similar but the precedence is very important. Just enabling multiple environments is not sufficient, the nested environment as per the second comment is what I'm thinking of.
After enabling this feature, Insomnia can also start providing sane defaults for the environment. Each environment can provide a local/private sub-environment that local users can modify without worrying about it being synced.

Implementation of multiple active environments would also need/have explicit precedence. It would be needed in order to have predictable and useful merging of duplicate keys (like you are suggesting). My idea would be that the sort order of the activated environments would determine the merge precedence.

Can you describe what you were thinking?

The sort order would probably be arrangeable via drag-n-drop so you could switch them around with ease. I could also imagine each one being a drop-down menu so you could quickly switch it out for a different one (eg. USER-1 --> USER-2).
Unless I'm misunderstanding, I think this implementation would still cover the case you outlined.
You would have your four environments (TST, TST-PRIVATE, USER-1, USER-2), then activate TST, TST-PRIVATE, and the USER environment you want to test with.
What kind of experience/interface were you imagining?

This feature would be awesome. Our team has been loving Insomnia and this is the only major feature that's a bit of a stumbling block, as right now we have to either share API keys, or each create our own private Local / Staging / Production env objects, or not use an env variable for API keys at all.

I have the exact same use case as the OP - the desire to share my workspace with environments but not share credentials.
My own take on this is that rather than having sub-environments and have to worry about precedence and whatnot, an environment can simply have a private section that is not shared. Thoughts?

That would be really awesome, I'm looking forward for this feature to be implemented ;0)

Are there any news on this one? I would really like to have this functionallity


Create tabbed widgets

Inspiration from https://stackoverflow.com/questions/50606443/adding-qscrollarea-to-qtabwidget

Codecov Report
Patch coverage: 100.00% and project coverage change: +0.03% :tada:

Comparison is base (61f00a1) 86.02% compared to head (eb6a78e) 86.06%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #355      +/-   ##
==========================================
+ Coverage   86.02%   86.06%   +0.03%
==========================================
  Files          30       30
  Lines        2097     2103       +6
  Branches      316      316
==========================================
+ Hits         1804     1810       +6
  Misses        213      213
  Partials       80       80

Files Changed
Coverage Δ

btrack/napari/widgets/create_ui.py
98.71% <100.00%> (+0.10%)
:arrow_up:

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

Getting there...

WIP but merging to make development easier for @paddyroddy and @p-j-smith


Adding initial CONTRIBUTING.md file, with reference to the CLA

Offering an initial CONTRIBUTING.md file. Includes a reference to the CLA

Got closed without being labeled closed first. Reopening to sync with Bugtrack.

I don't want to mess anything up. can i just close this? i think so since already merged...correct my action if i am mistaken.


[SPARK-14338][SQL] Improve SimplifyConditionals rule to handle null in IF/CASEWHEN

What changes were proposed in this pull request?
Currently, SimplifyConditionals handles true and false to optimize branches. This issue improves SimplifyConditionals to take advantage of null conditions for if and CaseWhen expressions, too.
Before
scala> sql("SELECT IF(null, 1, 0)").explain()
== Physical Plan ==
WholeStageCodegen
:  +- Project [if (null) 1 else 0 AS (IF(CAST(NULL AS BOOLEAN), 1, 0))#4]
:     +- INPUT
+- Scan OneRowRelation[]

After
scala> sql("SELECT IF(null, 1, 0)").explain()
== Physical Plan ==
WholeStageCodegen
:  +- Project [0 AS (IF(CAST(NULL AS BOOLEAN), 1, 0))#4]
:     +- INPUT
+- Scan OneRowRelation[]

How was this patch tested?
Pass the Jenkins tests (including new extended test cases).

Test build #54743 has started for PR 12122 at commit 22bb31f.

deep-review this please

@dongjoon-hyun Review complete. This matches Hive semantics, where SELECT if(null, 1, 2) returns 2. However, this prevents a future optimization where if(anything, TrueLiteral, FalseLiteral) can be simplified to anything.
This requires human decision. @JoshRosen @sameeragarwal Please advise.

Test build #54747 has started for PR 12122 at commit 512a45f.

Test build #54743 has finished for PR 12122 at commit 22bb31f.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/54743/
Test FAILed.

Merged build finished. Test FAILed.

Test build #54747 has finished for PR 12122 at commit 512a45f.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/54747/
Test FAILed.

Merged build finished. Test FAILed.

Test build #54758 has started for PR 12122 at commit dcbb0ea.

Rebased and squashed.

Test build #54758 has finished for PR 12122 at commit dcbb0ea.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/54758/
Test FAILed.

It's strange.
org.apache.spark.sql.execution.streaming.state.StateStoreSuite.maintenance test fails here and at my other PR frequently.

Test build #2729 has started for PR 12122 at commit dcbb0ea.

Test build #2729 has finished for PR 12122 at commit dcbb0ea.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Test build #2730 has started for PR 12122 at commit dcbb0ea.

LGTM

Thank you, @rxin .
I fixed that to private def, too.

Test build #54768 has started for PR 12122 at commit d8c3f38.

Test build #2730 has finished for PR 12122 at commit dcbb0ea.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test build #54768 has finished for PR 12122 at commit d8c3f38.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/54768/
Test FAILed.

The last failure is also org.apache.spark.sql.execution.streaming.state.StateStoreSuite.maintenance. It's irrelevant to this PR.

Rebased.

Test build #54780 has started for PR 12122 at commit 2870d4e.

Test build #54780 has finished for PR 12122 at commit 2870d4e.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/54780/
Test FAILed.

Test build #2733 has started for PR 12122 at commit 2870d4e.

Test build #2733 has finished for PR 12122 at commit 2870d4e.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Thanks - merging in master.

@dongjoon-hyun Can you take a look at the scala 2.10 build?

I think I fixed it just now.

Oh, sorry for breaking Scala 2.10 build and late response, @yhuai and @rxin .
Thank you so much for hot fix, @rxin !


update deps

#248

looks like rollup doesn't like Node v8 anymore https://github.com/gavinr/esri-leaflet-geocoder/pull/1


RPC Queue Implementation

The work of this branch is based primarily on the MLAPI RPC Queue Proposal.
The RFC for this is in the middle of construction, but current reviewers can use the MTT proposal document for the time being.
This branch includes and is dependent upon the changes for RFC#1: Standard RPC to replace Convenience and Performance RPCs.
** NOTE **
There are several documentation/md related files that evidently got added into the mix when migrating and merging (based off of master first).
Still looking into this, but would appreciate any feedback on items I should not include in this PR that is not wanted/needed in the develop branch.

This PR has too many issues and needs to be fixed before review.


F401 - update documentation and deprecate ignore_init_module_imports

Summary

[x] Update documentation for F401 following recent PRs

#11168
#11314

[ ] Deprecate ignore_init_module_imports (Do we have a proceedure for this?)

Test plan
Not clear yet

@zanieb thoughts on how to gracefully deprecate lint.ignore_init_module_imports? I can just grep for it and remove it, but that seems like it would break people who have the option in their config files.

Here's an example of a deprecated setting:
https://github.com/astral-sh/ruff/blob/a73b8c82a83336d76e69ee779d1f0648d1fdc667/crates/ruff_workspace/src/options.rs#L428-L432
which has a warning at
https://github.com/astral-sh/ruff/blob/6ed2482e27ec27fa9a6a812cff52efa1c360aded/crates/ruff_workspace/src/configuration.rs#L423-L430
I'm actually not entirely sure why we define a message for the deprecation and manually include a separate warning. Perhaps because it was renamed? I'd have to play with this a bit to find the right usage.
I think we should still respect the setting on stable but you can ignore it in preview. In both cases, we should display a warning. In the future, we can make using the setting an error during configuration loading.

Deferring to @zanieb, feel free to merge when approved!


runtime error when calling lua function from C++, passing a const char*

Trying to convert from luabind(the boost version) to this one, getting error at startup.
const char* path = "..\code\script?.lua;"
luabind::call_function(_State, "set_path",path);
It seems to be trying to find the class_id for "const char", not finding anything, and then reporting that const char is not a registered class.
" Trying to use unregistered class: char const *"

I think there's simply missing a semicolon in the first line..

I can reproduce this behaviour. Working on a fix.

Added the missing string literal overload (latest commit). Pleare let me know if it works for you.

the-nic: this was psuedo code, the real code was too complex to post, but yes it has a missing semi colon
decimad: thx I'll give it a try

I just tested master branch and std::string, const std::string&, and const char* works.

Great, thank you!


PlayerEx with Sinytra Connector crashes with Origins

https://mclo.gs/pbZiJtn

Will be resolved in alpha 14 (this is due to an outdated, JIJ version of AdditionalEntityAttributes from forge: origins that is out of sync with the upstream AdditionalEntityAttributes). A failsafe to obtaining id's for attributes will be done to prevent this, and warnings will be displayed for specific attributes which can not be located.


proj_data_dir config option not working (only projections within proj.db will work)

Context
Providing custom projections via the proj_data_dir option is not working.
Expected Behavior
Custom EPSG projections should become available for use through the MapProxy yaml config file as described in  https://mapproxy.org/docs/latest/configuration.html
Actual Behavior
When attempting to use non-standard EPSG's in mapproxy.yaml, provided through the proj_data_dir setting, an error is thrown.
Steps to Reproduce

Define a custom projections text file named "epsg" with the following contents:

<100000> +proj=lcc +lat_1=-25 +lat_2=-32 +lat_0=-28.5 +lon_0=24.5 +x_0=0 +y_0=0 +datum=WGS84 +units=m +no_defs <>

# Standard EPSG's we're also using in the *.yaml config file
# Copied from http://epsg.io/
<3857> +proj=merc +a=6378137 +b=6378137 +lat_ts=0.0 +lon_0=0.0 +x_0=0.0 +y_0=0 +k=1.0 +units=m +nadgrids=@null +wktext  +no_defs <>
<27200> +proj=nzmg +lat_0=-41 +lon_0=173 +x_0=2510000 +y_0=6023150 +ellps=intl +towgs84=59.47,-5.04,187.44,0.47,-0.1,1.024,-4.5993 +units=m +no_defs <>

Reference the "epsg" file location (in a directory also called "epsg" in this case) in the mapproxy.yaml configuration file:

services:
  demo:
  tms:
    use_grid_names: true
  kml:
      use_grid_names: true
  wmts:
  wms:
    srs: ['EPSG:3857','EPSG:100000']
    md:
      title: MapProxy WMS Proxy
      abstract: This is a minimal MapProxy example.

layers:
  - name: satellite
    title: Mapbox Satellite
    sources: [satellite_cache]
  - name: satellite_custom
    title: Mapbox Satellite reprojected as custom map grid
    sources: [satellite_custom_cache]

caches:
  satellite_cache:
    grids: [mapbox_grid]
    sources: [satellite_tiles]
  satellite_custom_cache:
    grids: [custom_grid]
    sources: [satellite_cache]

sources:
  satellite_tiles:
    type: tile
    url: https://api.mapbox.com/v4/mapbox.satellite/%(z)s/%(x)s/%(y)s.png?access_token=[REDACTED]
    grid: mapbox_grid

grids:
  webmercator:
    base: GLOBAL_WEBMERCATOR
  mapbox_grid:
    base: GLOBAL_MERCATOR
    srs: 'EPSG:3857'
    origin: 'nw'
  custom_grid:
    srs: 'EPSG:100000'
    origin: 'nw'

globals:
  srs:
    proj_data_dir: 'epsg\'

Run MapProxy mapproxy-util serve-develop mapproxy.yaml.
Get the following error:

C:\Python>mapproxy-util serve-develop mapproxy.yaml
[2021-02-12 11:44:45,052] mapproxy.config - INFO - reading: C:\Python\mappoxy.yaml
[2021-02-12 11:44:45,058] mapproxy.system - INFO - loading proj data from C:\Python\epsg\
Traceback (most recent call last):
  File "c:\python\python39\lib\runpy.py", line 197, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File "c:\python\python39\lib\runpy.py", line 87, in _run_code
    exec(code, run_globals)
  File "C:\Python\Python39\Scripts\mapproxy-util.exe\__main__.py", line 7, in <module>
  File "c:\python\python39\lib\site-packages\mapproxy\script\util.py", line 387, in main
    commands[command]['func'](args)
  File "c:\python\python39\lib\site-packages\mapproxy\script\util.py", line 85, in serve_develop_command
    app = make_wsgi_app(mapproxy_conf, debug=options.debug)
  File "c:\python\python39\lib\site-packages\mapproxy\wsgiapp.py", line 58, in make_wsgi_app
    services = conf.configured_services()
  File "c:\python\python39\lib\site-packages\mapproxy\config\loader.py", line 211, in configured_services
    return self.services.services()
  File "c:\python\python39\lib\site-packages\mapproxy\config\loader.py", line 1855, in services
    new_services = creator(service_conf or {})
  File "c:\python\python39\lib\site-packages\mapproxy\config\loader.py", line 2018, in demo_service
    lyr = layer_conf.wms_layer()
  File "c:\python\python39\lib\site-packages\mapproxy\util\py.py", line 78, in wrapper
    cache[key] = func(self, *args, **kwargs)
  File "c:\python\python39\lib\site-packages\mapproxy\config\loader.py", line 1683, in wms_layer
    map_layer = self.context.caches[source_name].map_layer()
  File "c:\python\python39\lib\site-packages\mapproxy\util\py.py", line 78, in wrapper
    cache[key] = func(self, *args, **kwargs)
  File "c:\python\python39\lib\site-packages\mapproxy\config\loader.py", line 1599, in map_layer
    for grid, extent, tile_manager in self.caches():
  File "c:\python\python39\lib\site-packages\mapproxy\util\py.py", line 78, in wrapper
    cache[key] = func(self, *args, **kwargs)
  File "c:\python\python39\lib\site-packages\mapproxy\config\loader.py", line 1503, in caches
    sources, source_image_opts = self._sources_for_grid(
  File "c:\python\python39\lib\site-packages\mapproxy\config\loader.py", line 1376, in _sources_for_grid
    tile_grid=grid_conf.tile_grid(),
  File "c:\python\python39\lib\site-packages\mapproxy\util\py.py", line 78, in wrapper
    cache[key] = func(self, *args, **kwargs)
  File "c:\python\python39\lib\site-packages\mapproxy\config\loader.py", line 300, in tile_grid
    grid = tile_grid(
  File "c:\python\python39\lib\site-packages\mapproxy\grid.py", line 101, in tile_grid
    srs = SRS(srs)
  File "c:\python\python39\lib\site-packages\mapproxy\srs.py", line 96, in SRS
    srs = _srs_impl(srs_code)
  File "c:\python\python39\lib\site-packages\mapproxy\srs.py", line 320, in __init__
    self.proj = CRS.from_epsg(epsg_num)
  File "c:\python\python39\lib\site-packages\pyproj\crs\crs.py", line 331, in from_epsg
    return CRS(_prepare_from_epsg(code))
  File "c:\python\python39\lib\site-packages\pyproj\crs\crs.py", line 296, in __init__
    super().__init__(projstring)
  File "pyproj/_crs.pyx", line 2302, in pyproj._crs._CRS.__init__
pyproj.exceptions.CRSError: Invalid projection: epsg:100000: (Internal Proj Error: proj_create: crs not found)

Context
Unable to utilise custom projections in MapProxy.
Your Environment

Version used: 1.13.0
Environment name and version (e.g. Python 2.7.5 with mod_wsgi 4.5.9): Python 3.9
Operating System and version: Windows 10 (64 bit)

Further investigations seems to suggest that the proj_data_dir does work but only if it contains the proj.db file. All other files alongside proj.db seem to be ignored.
Experimenting without using the proj_data_dir and trying to use projections from non proj.db files hosted in C:\Python\Python39\Lib\site-packages\pyproj\proj_dir\share\proj e.g. EPSG:42311 from the other.extra file does not work either.
The same error is encountered:
pyproj.exceptions.CRSError: Invalid projection: epsg:4326: (Internal Proj Error: proj_create: no database context specified)
It looks like I need to work out how to add my custom projection to the SQLite DB tables within proj.db.

Digging deeper I think that EPSG definitions will only be pulled from the proj.db database by pyproj, so the 'epsg' file that the MapProxy docs mention doesn't get picked up.
As I workaround I tried to use a custom authority of 'xxxx' by using an 'xxxx' file and using the srs of 'XXXX:100000' in mapproxy.yaml. Unfortunately, MapProxy takes 'XXXX:100000' and converts it to 'EPSG:100000' so the 'xxxx' doesn't get used. I see the error pyproj.exceptions.CRSError: Invalid projection: epsg:100000: (Internal Proj Error: proj_create: crs not found).
I also tried the experiment using an existing projection from other.extra by using the srs 'other.extra:42311', but again the authority seems to be replaced as the same error resulted: pyproj.exceptions.CRSError: Invalid projection: epsg:42311: (Internal Proj Error: proj_create: crs not found).
If MapProxy respected the configured authority maybe these custom projection files would be picked up correctly?

Potential solution: Hopefully if MapProxy stopped overriding the srs authority with 'EPSG' then current non-EPSG projections such as ESRI:* would start to work, and custom projection files could be dropped in using other authority identifiers other than epsg, to bring back support for custom projections without the need to hack a copy of the proj.db file.

I admit that the documentation is sparse on that topic (but it is not a common use case anyhow).
The gist is that PROJ>=5 uses a different configuration format. The old text file format is no longer supported.

All good, it's a great project and I figured I was probably doing edge case stuff. I can see from my investigations that the PROJ library had evolved but MapProxy had fallen behind its conventions.


Document families and matchers modules

Nice! I believe documenting the python bindings / pyi files is even more important than the Rust code.

That as well, but I want the barrier for other people to work on the Rust code itself to be as low as possible.


AD-Hoc Testing Scenarios. [FEB 01]

Describe the Task:
This issue is to track the ad hoc testing round occurred in specific flows.
Main Focused areas -

Email OTP Expiration Times conflicts.
SMS and email Federated authenticators testing.

Additional Flows -

Self Registration
Invite the User to set the password

Optional Fields
Related issues:

Suggested labels:

Additional Verification Process Required

[ ] Verify that the older version triggers an email if email verification configuration is enabled.
[ ] Re verify self registration works with a host name changed pack.
[ ] Check with the team the feasibly of adding a email verified check within the UI


Fixes 11560: Adds dashboard sheets to tableau ingestion query

Describe your changes:
Fixes 11560 - not all sheets associated to a workbook are being ingested from a Tableau workbook.

Many of our Tableau workbooks at ZenBusiness are built with the dashboard abstraction in addition to sheets. Without this change, sheets associated to workbooks through dashboards are not being picked up because only sheets 'directly' associated to a workbook are being processed. I tested these changes against our production tableau/OMD instance by running the altered workflow locally.
This slack thread has more screenshots and discussion around the issue.

Type of change:

[ ] Bug fix
[x] Improvement
[ ] New feature
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation

Checklist:

[x] I have read the CONTRIBUTING document.
[x] My PR title is Fixes <issue-number>: <short explanation>
[ ] I have commented on my code, particularly in hard-to-understand areas.
[ ] For JSON Schema changes: I updated the migration scripts or explained why it is not needed.

Thanks @joal-zb for the PR.

Hi @joal-zb , thanks for opening this PR. It def helped us understand the updated queries.
After our conversation in Slack and feedback from other users, we'll be using Data Sources as Data Models, instead of passing the Sheets there. It should make the OM model easier to understand and relate to Tableau.
I'll be closing this PR for now. Thank you for your help and feedback.


Help file validation needs to be part of CI.

Follow up work for PR #3184.
While doing work for az interactive we should be able to validate the help files against the command table to find things like:

[x] missing type (command or group)
[x] command entries in help that aren't in the command table (for example, a command name changed)
[x] extra parameters in help that aren't in the command (I think it used to do this, but it should be part of a script and not a runtime error).

@williexu is this what your linter does?

YES! IT DOES :)


Djoser Token authentication hitch

When a registered user tried to login he/she should receive a token that is valid for the period he/she is looged in thereafter a logout it expires.
Earlier on the code in the profiles/signals.py created it's own token on each new user created

@receiver(post_save, sender=User)
def create_auth_token(sender, instance=None, created=False, **kwargs):

if created:
Token.objects.create(user=instance)
The token can be used to login though the djoser implementation should be a standalone thing , there's some mismatch of code i.e. a token created on user creation shouldn't be used to login a user and should not also expire on logout!

This should be critically analyzed

All users are now flagged active by default upon registration


Wagmi can start in docker

While testing locally with "docker-compose up" on OSX, I had a few errors.
This fixes these errors and allows to run wagmi in docker.
Cheers

Thank you, Arnaud 🙏


robosocks pull request for training

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

I need to do this a dozen more times, but I think I at least get the concept.


Idle Notifier Plugin: Add inventory slot notifier

Feature was requested in #3969. Sends you a notification when your specified inventory threshold is reached. Example use is setting the threshold to 27 in Motherlode Mine so you know you need to pay attention as your inventory is about to fill up.

Closed per my comment on the target issue


still getting out of sync audio

if downloading a clip farther into a very long stream, still getting frequent out of sync audio

Then use the TL 1.6 feature and bypass mp4 conversion in order to get the raw .ts file from Twitch.


Fix lint-staged & husky

Summary of Changes

Changed .husky/pre-commit mode to include +x
Fixed lint-staged config - linting script is called lint and not eslint

Ty for these improvements!


Re-enables test_operator.test_dropout

Description
Fixes #12329
Re-enables test_operator.test_dropout
I've tested it 10k times and it passed. I also tested it using the issue's random seed and it passed.
I believe it is safe to re-enable.
Checklist
Essentials
Please feel free to remove inapplicable items for your PR.

[x] The PR title starts with [MXNET-$JIRA_ID], where $JIRA_ID refers to the relevant JIRA issue created (except PRs with tiny changes)
[x] Changes are complete (i.e. I finished coding on this PR)
[x] All changes have test coverage:
Unit tests are added for small changes to verify correctness (e.g. adding a new operator)
Nightly tests are added for complicated/long-running ones (e.g. changing distributed kvstore)
Build tests will be added for build configuration changes (e.g. adding a new build option with NCCL)
[x] Code is well-documented:
For user-facing API changes, API doc string has been updated.
For new C++ functions in header files, their functionalities and arguments are documented.
For new examples, README.md is added to explain the what the example does, the source of the dataset, expected performance on test set and reference to the original paper if applicable
Check the API doc at http://mxnet-ci-doc.s3-accelerate.dualstack.amazonaws.com/PR-$PR_ID/$BUILD_ID/index.html
[x] To the my best knowledge, examples are either not affected by this change, or have been fixed to be compatible with this change

Changes
Comments

@mxnet-label-bot  [pr-awaiting-review]

Please add

Fixes https://github.com/apache/incubator-mxnet/issues/12329

in the PR description so that the issue is closed automatically when this PR is merged.

Done

Hello @perdasilva Thanks for this contribution.
Did you run this test with this seed - 119185068 which was reported in the issue as failure?

@sandeep-krishnamurthy yup - I've ran it 10k times and tested it with the suggested seed. No failures for me.


How to run an accurate duration opacity animation with anime.js?

I'm trying to fade out an element with anime.js.
The code works but the duration does not correspond with the duration param I'm giving.
For example when I set duration to five seconds, it takes about half of that.
https://codepen.io/anon/pen/rYLRQq
 anime({
   targets: '.el',
   opacity: 0,
   duration: 5000
 });

Also there's a noticable difference in duration between firefox and chrome.
Is there a way to have it more accurate?

The duration is correct. I think the issue you're seeing is that the default easing for anime is set to easeOutExpo. If you set it to linear it should be more "accurate".

The duration is correct. I think the issue you're seeing is that the default easing for anime is set to easeOutExpo. If you set it to linear it should be more "accurate".

Duration is accurate, but visually wrong since you're using the default easing timing function, which is 'easeOutElastic', as @alexchantastic says, use 'linear' instead.


very small doc mistake in mapValid

There's just a very small documentation mistake I recently found in mapValid.
emailCodec =
    S.string
        |> S.mapValid
            (\text ->
                case Email.fromString {- 'text' should be here -} of
                    Just email ->
                        Ok email

                    Nothing ->
                        Err "Invalid email"
            )
            Email.toString

Apart from that, you might want to use Result.fromMaybe but that's up to your preference.
emailCodec =
    S.string
        |> S.mapValid
            (Email.fromString >> Result.fromMaybe "Invalid email")
            Email.toString

Thanks, it should be fixed now. I chose to use the more verbose first example, I think it's more clear what it does.


feat(indicator): remove is prefix in props

Closes #3523
Description
Removing all is prefix from is*.
Current behavior (updates)
Before: isDisabled, isNumeric
New behavior
after: disabled, numeric
Is this a breaking change (Yes/No):
No
Additional Information
NOTE: Mr. Yamada requested that not only the props but also the is* variables within the code be modified, so they have been updated accordingly.

Is this PR removing is prefix in props or deprecating is prefix?

The task of marking it as deprecated has been completed, and it is scheduled to be merged into the main branch.


circ.qubit_duration() - qubit_stop_time undefined

Informations

Qiskit Experiments version:

$ pip list |grep qiskit
qiskit                 0.36.2
qiskit-aer             0.10.4
qiskit-aqua            0.9.2
qiskit-experiments     0.3.1
qiskit-ibmq-provider   0.19.1
qiskit-ignis           0.7.1
qiskit-terra           0.20.2

Python version:
Python 3.9.2

Operating system:
Ubuntu

What is the current behavior?
I'd like to know the total time given qubit was driven when circuit was executed on real HW (ibmq_quito).
I have constructed a circuit, transpiled, submitted the job, and asked job for the circuit after the submission - I still get no output. How can I trigger the scheduler to attach the info I want?
This are key lines form my code:
circ=...
circT = qk.transpile(circ, backend=backend, optimization_level=3)
job =  backend.run(circTL,shots=1000)
circSL=job.circuits()
print('submitted JID=',job.job_id(),backend ')
print(circSL[1].qubit_duration(1))
The result is:
...
submitted JID= 629ebde5a38471bafc4529b4 ibmq_quito
...
Traceback (most recent call last):
File "/home/balewski/2022_QPIXL_ver1/qiskit_examples/production/./submit_ibmq_job.py", line 98, in
print(circSL[2].qubit_duration(1))
File "/home/balewski/.local/lib/python3.9/site-packages/qiskit/circuit/quantumcircuit.py", line 4575, in qubit_duration
return self.qubit_stop_time(*qubits) - self.qubit_start_time(*qubits)
File "/home/balewski/.local/lib/python3.9/site-packages/qiskit/circuit/quantumcircuit.py", line 4639, in qubit_stop_time
raise CircuitError(
qiskit.circuit.exceptions.CircuitError: 'qubit_stop_time undefined. Circuit must be scheduled first.'
Steps to reproduce the problem
I could provide a demonstrator
What is the expected behavior?
get the total length of few usec
Suggested solutions

HI @balewski .qubit_duration only work with "scheduled circuit". So you first need to schedule the circuit. That can be done by specifying scheduling_method option in transpile. Please change your code like
circT = qk.transpile(circ, backend=backend, optimization_level=3, scheduling_method="asap")  # or "alap"
print(circT.qubit_duration(1))

and you can get the duration even before running the circuit.

thank you!
I got it to work. Can you help me with few follow-up questions?
Q1: I see 2 scheduling option: ASAP & ALAP.
If I run on the actual HW, say Jakarta, will my choice be honored?
If yes, I'm expecting to obtain different results, regardless of different shots statistics,  because the overlay between pulses driving different qubits will be different, hence the crosstalk will change. Is this reasonable?
Q2: for a real HW at IBMQ, why the default ALAP is better than ASAP?
Q3:
For the circuit transpiled below in the ASAP mode, Qiskit reports the same duration for all 3 used qubits (which is equal to the delay of the ancilla qubit).
= = =
duration of drive per qubit
3 31760
1 31760
4 31760
= = =
It seems the last delays on q1 of  Delay(320[dt]), and on q4 of Delay(2592[dt]) are included in reported time.
This seems to contradict the documentation, which says: ... excluding delays...
https://qiskit.org/documentation/stubs/qiskit.circuit.QuantumCircuit.qubit_duration.html https://qiskit.org/documentation/stubs/qiskit.circuit.QuantumCircuit.qubit_duration.html
What do I miss?
I'd like to know for how the total of ns any pulse was applied for given qubit, so excluding all delays, even if they are in cycles inside the circuit (the one below is too simple for that).
Q3: how can I extract from the backend the value of 'dt' in seconds, so I can convert duration to SI units?
Thanks
Jan


Added new translation

Added new translation for infinity

Resolved conflicts, please review my PR, Thank You.


Distorts CSS when using semantic ui react.

After installing and importing the PDFViewer component in the application, most of the semantic ui react css is lost, works fine when the import is removed.

Same thing happens with me.

Removed bulma in latest release(v2.0.7), so it will not distort your exsisting css and you have to provide your own custom css for pdf-viewer.


Fix broken link in README.md

@simonpasquier Thank you for the PR, can you make sure the commit is signed-off? You can fix it with git commit --amend -s && git push -f <your-private-repo>


Replace hide extra buttons for each of a kind

I very rarely need buttons for a clip, report, etc. I mostly use the save button. It would be nice if I could have just this one button.
so can we replace an "Hide extra buttons" in video section for each of a kind? a

Hey there! This one is tricky, because YouTube actually puts them all in the same bag, but I'll try to figure a way 😄


GDP-hourly adapter

This adapts Philippe's helper module clouddrift-examples/data/gdp.py into the core library as clouddrift.adapters.gdp. Most or all of it is meant to be internal to clouddrift, i.e. not part of the user-facing API, but can be of course used as a reference on how to ingest custom datasets into clouddrift.
The end goal is to provide to the user a convenience from_gdp function that returns a ragged xarray Dataset.
One obvious question is whether the end-user ever need to load ragged array xarray Dataset from raw GDP (individual NetCDF files), or is the ragged-array GDP NetCDF on AWS always sufficient?
If the ragged-array GDP on AWS is all the end-users would need, then all this probably doesn't belong here, and could be included in an auxiliary repo like clouddrift-tools or clouddrift-dev or similar. And the from_gdp method would simply point to the AWS URL for convenience, and the user can take it from there.
However, if there's still a need for non-ragged (raw) GDP NetCDF access to the end-users, then I think this gdp adapter module is appropriate.
This module doesn't deviate much from Philippe's original in clouddrift-examples. Most changes are housekeeping, like removing the global state, tidy up docstrings, and using a system-provided temporary directory for downloads instead of the user's filesystem (i.e. ../data).
Implementation:

[x] Remove global state
[x] Use system-provided temporary dir for downloads
[ ] from_gdp1h() -> xr.Dataset convenience wrapper

Closes #60.

Here we need to distinguish two cases:

the hourly dataset for which individual netcdf files (let's not call them raw) are available on the AOML FTP server and a file of ragged arrays is available from NOAA NCEI and AWS
the 6-hourly dataset for which individual netcdf files are available on the AOML FTP server and a file of ragged arrays is only available from http/ftp from the AOML GDP website

I think I see the use of this adapter (adaptor?) for implementing from_gdp() but will this be for both datasets? Should we think of both from_gdp1h and from_gdp6h? and in both cases could we pass a list of ids as an argument? In that case the access to individual files via the FTP server is better? To be discussed.

Yes, we'd have both from_gdp1h and from_gdp6h. This PR is for 1h, and the 6h one will reuse most of the internals here.
I think passing ids makes sense if we pull individual files from ftp so that we download only the requested files.. If we pull from ragged array NetCDF, then the lazy downloading can be leveraged by the NetCDF library via xarray. In this case I wouldn't pass ids at this stage.

I'm not sure how many people are going to create themselves the ragged array from individual NetCDF archives for the GDP, but I still think it's a good idea to have this centralized in the Clouddrift package.
I'll take a look later and merge it.

I still have a few things to address here and will merge when ready.

@selipot and I agreed yesterday that the user-facing from_gdp1h should get the ragged array data from AWS, rather than processing individual files.
This can thus be cleanly separated from this PR and I track it in #96.
It may still be useful to have the custom GDP pre-processing as part of the core lib, so we'll merge this here.

I missed the merge of this today, I don't really want to create an issue for this but could you modify the preprocess function by, removing this:
# set coordinates
ds = ds.set_coords(["ids", "longitude", "lon360", "latitude", "time"])

since it's not needed anymore and we can leave whatever coordinates they set. And include this instead to rename the longitude and latitude.
# rename variables
ds = ds.rename_vars({"longitude": "lon", "latitude": "lat"})

@selipot and I agreed yesterday that the user-facing from_gdp1h should get the ragged array data from AWS, rather than processing individual files.
This can thus be cleanly separated from this PR and I track it in #96.
It may still be useful to have the custom GDP pre-processing as part of the core lib, so we'll merge this here.

Aren't we (or someone at NOAA) going to need the gdp_1h processing functions that read the individual NetCDF files to create the ragged array? I do think it should be included in the adapters too.

Definitively, for both hourly and 6-hourly! So should it be with from_gdp1h or in  theclouddrift-example repo?

I would suggest to create the adapters for the 1-hourly so we have both there, and used them in the cloudrift-examples.
Then, we can also have another example using the AWS file.

@philippemiron can you clarify whether you mean defining the from_gdp* functions in the core library or in the examples?
IMO the user-facing function should be in the core lib. The developer function (individual files -> ragged NetCDF) can be in either core lib, auxiliary repo like clouddrift-dev, or clouddrift-examples, though the last one seems least appropriate.

For the developer function, my preference is to define it here, perhaps clouddrift.adapters.gdp.to_raggedarray. Then every custom adapter module could have a consistently named to_raggedarray function that developers could use to generate new versions of the ragged array NetCDF, and contributors/users could use as a template to contribute new custom adapters to the library.

For the developer function, my preference is to define it here, perhaps clouddrift.adapters.gdp.to_raggedarray. Then every custom adapter module could have a consistently named to_raggedarray function that developers could use to generate new versions of the ragged array NetCDF, and contributors/users could use as a template to contribute new custom adapters to the library.

This sounds perfect to me.


UUID should be added into words list

/home/xuanwo/Code/go/src/github.com/yunify/qingcloud-sdk-go/service/types.go:410:2: struct field GlobalUuid should be GlobalUUID

Closed for #53 merged.


simplify storage import

silly that this used to fail... makes documentation and code overly verbose.
import lightning as L

L.app.storage.Drive(...)

Also, pip install was failing... so had to add this docs.txt file @Borda

Also, pip install was failing... so had to add this docs.txt file

How did it failed?
Also why not use requirements/app/docs.txt?


DNN-6829 + CONTENT-2544 - Force folders deletion when they should be emp...

...ty

merging into 7.4.2 and cherrypicking into release/7.4.1


Hyperd is not working:failed to load sandbox info: leveldb

I installed the hyperd on CentOS as usual, and it could run actively, but it got different errors according to status. Anyone knows where is the problem?
Here is the status:

Nov 13 07:56:39 aaa systemd[1]: Starting hyperd...
Nov 13 07:56:39 aaa hyperd[5638]: time="2018-11-13T07:56:39Z" level=warning msg="devmapper: Usage of loopback devices is strongly discouraged for production use. Please use --storage-opt dm.thinpooldev or use man docker to refer to dm.thinpooldev section."
Nov 13 07:56:39 aaa hyperd[5638]: time="2018-11-13T07:56:39Z" level=warning msg="devmapper: Base device already exists and has filesystem xfs on it. User specified filesystem  will be ignored."
Nov 13 07:56:39 aaa hyperd[5638]: time="2018-11-13T07:56:39Z" level=info msg="[graphdriver] using prior storage driver "devicemapper""
Nov 13 07:56:39 aaa hyperd[5638]: time="2018-11-13T07:56:39Z" level=info msg="Graph migration to content-addressability took 0.00 seconds"
Nov 13 07:56:39 aaa hyperd[5638]: time="2018-11-13T07:56:39Z" level=info msg="Firewalld running: false"
Nov 13 07:56:39 aaa hyperd[5638]: time="2018-11-13T07:56:39Z" level=info msg="Loading containers: start."
Nov 13 07:56:39 aaa hyperd[5638]: .
Nov 13 07:56:39 aaa hyperd[5638]: time="2018-11-13T07:56:39Z" level=info msg="Loading containers: done."
Nov 13 07:56:39 aaa hyperd[5638]: E1113 07:56:39.506112    5638 persist.go:117] Pod[hello-world-0328860666] failed to load sandbox info: leveldb: not found key: SB-hello-world-0328860666

@kuthreecar

failed to load sandbox info: leveldb: not found key: SB-hello-world-0328860666

This is not a fatal error and is ignored during hyperd startup. Could you please post the full debug log (e.g. by starting hyperd in command line with sudo hyperd --v=9 --alsologtostderr), so that we can see what actually stopped hyperd from working? Thanks!

@bergwolf
Thanks, since I was deployed it on aws, I relaunched a instance and reinstalled hyperd. now have no problem for that, I may close this issue first, thanks.


[sql] Extend privacy model with explicit group ownership

I was thinking through our group owner/admin model (looking at Greg's draft docs and some related conversations) and realized that the previous setup didn't really match up with my intuitions around application-level group ownership and administration.
Very open to discussion on this and aware that others might have different expectations or opinions on how these things should work!
Background
Before this PR, an SKDB group specified a corresponding admin group and skdb_access; when a user created a group using the TS API, we set up three groups as follows
GROUP | ADMIN_GROUP | SKDB_ACCESS
---------------------------------
base  | admin       | read-write
admin | owner       | admin
owner | owner       | owner

That is, we have a globally-visible base group administered by an admin group, which is visible only to admins and itself administered by an owner group, which is administered by and visible only to itself.
This works, but doesn't line up perfectly with typical notions of admins/owners, namely in that:

we can't enforce the property that only owners can delete any of the three groups
admins are unable to add/remove other admins

This PR
I think the simplest solution is to bake "ownership" into the model rather than encoding it with multiple "admin" relationships.
This PR does just that, adding an owner column to records in skdb_groups and enforcing that (1) owners have full admin privilege, regardless of their privileges in the admin group and (2) only owners can delete groups.
Now, when a user creates a group using the TS API, we set up the three groups as follows
GROUP | ADMIN_GROUP | OWNER_GROUP | SKDB_ACCESS
-----------------------------------------------
base  | admin       | owner       | read-write
admin | admin       | owner       | admin
owner | owner       | owner       | owner

Note that admins can now add/remove other admins.

Ahh, yes, I hadn't thought through the deployment headache here.  I can release updated node and docker artifacts once this lands, and will also document the model in the www docs.
As for the min version in the server, I'm not sure exactly what you're referring to -- I know we have the kotlin client version there (currently kt-0.0.3) but don't see any logic requiring a min JS version.  Where should I be looking?

Here: https://github.com/SkipLabs/cloud/blob/05ef10d3eeb6185c1acae00fe5c118dcaca7bee7/skgwd/src/main/kotlin/App.kt#L569

I can release updated node and docker artifacts once this lands

Would you be up for doing server releases together over VC? I'm quite conscious that I'm the only person that knows how to do this stuff.

Thanks, and yes of course!  I'll ping you on discord


A better “label centroid” initializer?

The centroid of a geometry is sometimes outside of it, for example (but not exclusively) with a MultiPolygon such as France+Guyane. This can result in labels or interactive tips that are quite far off from the shape they identify.
A great solution is to use @mapbox/polylabel, which guarantees that the center is inside —and furthermore at an optimal position for a centered label. The dependency is less that 1kB over the wire, including tinyqueue. An alternative approach could be to add an option to Plot.centroid, to make it choose the centroid of the largest polygon when given a multipolygon. Subpar solution, but cheaper.
Prototype https://observablehq.com/@observablehq/plot-pole-of-inaccessibility-1587

the centroid of the largest polygon could also lead to the point being outside the polygon. this would be the case for a crescent-moon shaped polygon.
polylabel sounds great!

We should probably call it labelCentroid, with an ellipseRatio option that defaults to 2 (?). And maybe add an anchor option so that we can align text on the left (with textAnchor: start) or on the right (with textAnchor: end). Since polylabel returns the radius, it's not expensive to add.

this approach could also make it easier to scale labels so they fit inside of polygons, since you'll know the ellipse dimensions anyway.

Related:

ST_MaximumInscribedCircle: equivalent of the pole of inaccessibility.
ST_PointOnSurface: no particular specification re: the quality of the point. It just is on the surface.


Allow to configure additional tolerations for cluster-agent, node-agent rancher will deploy

This is the feature requests for new 2 configurations (settings).

first one is the "tolerations-of-clusteragent" which will accept json string to indicate additional tolerations which will be applied to Deployment resource for cluster-agent.
second one is the "tolerations-of-nodeagent" which will accept json string in the same way as "tolerations-of-clusteragent". Only difference from it is this tolerations will be applied to DaemonSet resource for node-agent.

Background of this feature request is that Rancher is expected to maintain many type of k8s cluster deployment and agent is needed to deploy/run on the user cluster if we want to maintain a certain cluster.
But currently tolerations which is the concept to schedule POD to Node is hardcoded for rancher-agent deployment and only the taints configured by rancher/rke is considered to tolerate, which means rancher-agent won't run on some node if user configured own taints for nodes.
Due to limitation, some user can not maintain some clusters. So it would be nice if rancher provide the way to configure additional tolerations for rancher-agent via configuration

This is a subset of https://github.com/rancher/rancher/issues/15734 and is basically needed for all system pods as you run into it for every one.

See https://github.com/rancher/rancher/issues/16050 for another example, if this can be handled more easily than adding more settings I'm all for that.

I don't see a benefit of having configurable tolerations, versus having a toleration for all taints as the pods needs to be scheduled to every node anyway. And making it configurable is not only yet another configuration option, it's also not that clear for an user that they need to edit a setting after tainting their nodes (plus the situation where it changes when half the clusters are already running, which is kind of the same with changing server-url/certificates).
@ukinau I'm planning to re-open https://github.com/rancher/rancher/issues/16050 and put a fix allowing all taints for the agents. Let me know if you don't agree.


Fix typo in DeltaTable Generate method descriptions

The Generate method descriptions list symlink_manifest_format as a valid mode, however the Generate documentation as well as the DeltaGenerateCommand use symlink_format_manifest

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

good catch, lgtm


Gem5

I think shell_echo is a bit too generic a name for this.  It may be misinterpreted as controlling the echoing rather than stripping the output. Maybe something like strip_echoed_commands instead...


A duplicate orderBy usage example removed

There was a wrong example in "orderBy usage" section.

Coverage remained the same at 100.0% when pulling df923433fb8cb18563506e83b819a909a5f6a3fc on sadedil:patch-1 into cd1dd6334dad553b2510ba8aaaf6a4767a6efdf3 on fknop:master.

Indeed, thank you!


Update dependencies

rand 0.5
num-complex 0.2
num-traits 0.2
openblas-src 0.6
lapacke 0.2

thanks notify, but it has not been released yet :<

warning: use of deprecated item 'rand::distributions::IndependentSample::ind_sample': use Distribution instead


How to use?

This is a package for my private project, but if you`re interested in using it I've created a simple documentation. Hope it will suffice.

Thanks :)


Adjust the cpp interface to be more forgiving

There's two main changes here, renaming the internal C object to _d so it doesn't conflict with the cm matrix arguments a b c d e f and using CapyPDF as the C++ namespace as that's more in keeping with libraries in C++. Though I'd understand if you're making it look more like a std lib of some kind.

OK updated, thanks for reviewing.


Remove composer.lock to avoid locking dependencies.

I don't see any advantage of leaving the lock file in here.
This is causing dependencies to be locked to a specific revision.
This would be good if this was using unstable dependencies where you want to lock to a specific commit.
Locking to v2.4.2
- Installing doctrine/common (v2.4.2)
    Downloading: 100%

See @naderman's comment in https://github.com/doctrine/mongodb/pull/207#issuecomment-102647182.

:+1:


Fixed #29350 -- Allowed SQLite pk introspection to work with formats besides double quotes.

The get_primary_key_column() method in the sqlite3 backend only expects a field with quotation. The solution is to make the capture group focus on the first word, regardless of whether or not it has quotation.
There are no existing test cases, so I've added those as well.

Fix get_primary_key_column() method
Add relevant models and test cases.

Removed the hyphen from the second capture group, ([\w]+) because id-col is not a valid syntax in sqlite3. It would to be quoted in that case, which would be caught in the first capture group of the match.

For next time, try to use a commit message that's more descriptive than "Fixed xxx method" as that doesn't do much to explain the issue. I edited the PR title to the sort of thing I'd suggest.


Remove legacy Edge Debugger to address potential target list bug

This PR removes support for the legacy Debugger for Edge extension from the Tools extension. I believe this will help mitigate the occurences of the "No available targets found" error when attempting to use the "attach" entrypoint as described in: https://github.com/microsoft/vscode-edge-devtools/issues/501

I'm good with deprecating the legacy Edge Debugger config, but I am still unclear whether this could be the cause of the "fetching list of available targets" bug. Were you able to repro the "fetching list of available targets" error reporter toast without this change and using these config types?

My theory was that our check to see if users had a valid launch.json config was also accepting configs for launching spartan, and then our "launch project" button would attempt to launch spartan edge, and the extension wouldn't have anything to attach to. This is just a theory though. I have not been able to repro.

@mliao95 or @antross  would one of yall be able to take over this one since I'll be out for awhile?

@bgoddar after experimenting with various configurations and repros, I decided to leave this support in a bit longer but (1) silenced the request to report an issue for these targets and (2) added a warning when attaching to these targets to migrate to pwa-msedge in PR #690.
We'll probably still want to loop back and fully remove these in a future update, though (so leaving your PR active).


Fenghj/webpartal/job

Coverage decreased (-0.02%) to 53.33% when pulling 9aca34dc424ab7ba7b41b5e0b4f95e128f282108 on fenghj/webpartal/job into f977af531a65a7413e1855cedbceed83ae3973d0 on master.