Interop with UINavigationController user instantiated popping

Motivation:
ReSwiftRouter should provide the optional mechanism that assists in triggering of route pop corresponding actions to the store when:

User hits the back button on the navigation bar
User swipes over the left edge of the screen

API Proposal:
Users needs to subclass the ReSwiftInit.NavigationController and override popRoute() methods where they specify the dispatch logic to their store. Like:
class RootNavigationController: ReSwiftRouter.NavigationController {
    override func popRoute() {
        let currentRoute = mainStore.state.navigationState.route

        guard currentRoute.startIndex != currentRoute.endIndex else {
            return
        }

        mainStore <- SetRouteAction([RouteElementIdentifier](currentRoute[currentRoute.startIndex ..< currentRoute.endIndex.advanced(by: -1)]))
    }
}

Implementation details:
When user hits the back button popViewController() can be canceled if navigationBar(:shouldPop item:) is implemented. In such case popRoute() will be called that should result in SetRouteAction dispatched. Our routable can then subsequently call the actual popViewController() as desired.
However, when swiping popViewContoller() call cannot be canceled. When swipe is recognized, popViewController() will be called before popRoute() callback. In order to avoid the redundant popViewController() from our routable, popViewController() semantics was changed to ignore all popViewController() calls when swipe is still in progress. This semantic change is reasonable since nothing should be calling popViewController() while user is swiping anyway. Also it allows us having our routable transparent of the swiping state. It can instead (if needed) check if pop was canceled since the instance of PopWasCancelled dummy viewController would be returned. Like:
let popped = mapViewController.navigationController?.popViewController(animated: animated, completion: completionHandler)
if popped is PopWasIgnored {
       //additional logic on cancelation during swipe
}

Additionally the utility navigationController's push/pop with completion were provided.
The sample hypothetical routable can use it like:
public func popRouteSegment(_ routeElementIdentifier: RouteElementIdentifier, animated: Bool, completionHandler: @escaping ReSwiftRouter.RoutingCompletionHandler) {

    guard routeElementIdentifier == Routes.editFeature else {
         completionHandler()
         return
    }

    _ = mapViewController.navigationController?.popViewController(animated: animated, completion: completionHandler)
}

Codecov Report

Merging #74 into master will decrease coverage by 15.3%.
The diff coverage is 0%.

@@             Coverage Diff             @@
##           master      #74       +/-   ##
===========================================
- Coverage   80.28%   64.98%   -15.31%
===========================================
  Files           5        6        +1
  Lines         208      257       +49
===========================================
  Hits          167      167
- Misses         41       90       +49

Impacted Files
Coverage Δ

ReSwiftRouter/NavigationController.swift
0% <0%> (ø)

Continue to review full report at Codecov.

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

@Ben-G: what do you think about it?

Hey! Sorry I was out for a long weekend. Will try to look at this tomorrow!

On Jul 3, 2017, at 23:09, Taras Vozniuk<EMAIL_ADDRESS>wrote:
@Ben-G: what do you think about it?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

Hi @ambientlight!
Thanks a lot for the PR, it's great that you're trying to solve this problem!
I have a few thoughts:

Do you happen to have a project that uses this component already? It would be a lot easier to review when seeing the component in action. If you don't have that, I can try to integrate it into https://github.com/ReSwift/GitHubBrowserExample
Since this component has no dependencies on ReSwiftRouter/ReSwift and since it's highly dependent on UIKit, I think it would make sense to offer it as part of a separate micro-library instead of as part of ReSwiftRouter. ReSwiftRouter is intended to be unaware of the UI framework that is being used. However, if we can demonstrate that your component works well, it would make sense to link to it from this repo

Once again, thx for the PR 🎉

@Ben-G:

Do you happen to have a project that uses this component already? It would be a lot easier to review when seeing the component in action. If you don't have that, I can try to integrate it into https://github.com/ReSwift/GitHubBrowserExample

Let me make a sample project for it.

Since this component has no dependencies on ReSwiftRouter/ReSwift and since it's highly dependent on UIKit, I think it would make sense to offer it as part of a separate micro-library instead of as part of ReSwiftRouter.

Makes sense. Should we continue this conversation here or close and inside #17?

@ambientlight Happy to keep the conversation here, until we've decided where the API should live!
Thanks a lot for looking into a sample project.

@Ben-G
I am using it in my app. Originally my router would override the back button on a pushed viewController, with a navigationItem selector associated with router's pop.
I didn't like it this way, I thought it is possible to do it transparently - incorporate pop action inside navigationController, so I went to see how you guys do it, and since this wasn't yet resolved, I basically propose such implementation here.
My point to provide it within ReSwift-Router:
this is such a common problem that all users of ReSwift-Router will face, the solution to it should be available out-of-the box in ReSwift-Router without any extra dependency like ReSwift-Router-NavigationController.

@Ben-G:
I have updated the project you referenced. Please refer to https://github.com/ambientlight/GitHubBrowserExample/commit/57403b4b3685912b988a265e213b811e72655d12
Brief set of changes:

Removed SetRouteAction([mainViewRoute]) dispatches from BookmarkViewController and RepositoryDetailViewController since now it is handled with our navigationController.
Subclassed our navigationController and provided generic pop action:

class RootNavigationController: NavigationController {
    override func popRoute() {
        if let popRouteAction = popRouteAction(){
            store.dispatch(popRouteAction)
        }
    }
}

where popRouteAction is:
func popRouteAction() -> SetRouteAction? {
    let currentRoute = store.state.navigationState.route
    guard currentRoute.startIndex != currentRoute.endIndex else {
        return nil
    }

    return SetRouteAction([RouteElementIdentifier](currentRoute[currentRoute.startIndex ..< currentRoute.endIndex.advanced(by: -1)]))
}

Went to storyboard and replaced NavigationController class with our custom RootNavigationController

Added desired pop action into MainViewRoutable since now we need to actually perform perform pop corresponding to route change.

(self.viewController as? UINavigationController)?.popViewController(true, completion: completionHandler)

Ask me if you have any questions.

@Ben-G What do you think? 🙂

Hey @ambientlight! Thanks a lot for putting this example together. It seems like you didn't add the new files to the Xcode project in the commit that you are pointing to? I have the source files in the repo, but they are not included in the project.
Overall I put more thought into this; I think this is a great starting point for a third-party library, but it is definitely outside of the scope of the core of ReSwift-Router which is and should remain independent of any UI library.
It would be really cool if you could build this as a standalone library (the code already has no real dependencies on ReSwift/ReSwift-Router), and I'd be more than happy to reference it.
I'd also be curious to see if there's a solution that doesn't require subclassing UINavigationController, but I probably won't have time to investigate that any time soon.
I will close this issue for now; but please let me know if you happen to build a separate library that I can reference in future.
Thanks a lot for the work on this 😃
(I'll also make sure to take some ideas from this approach if I update the GitHub sample project in future).

@Ben-G Will definitely do so. Let me get back to it and then I will probably open the issue for it once it is done.
Did I forget something? that's weird, will check.
Thanks a lot for the review and suggestions!

@ambientlight have you created a third-party library for this?

@hlineholm-flir: Thanks for reminding me, I will do my best to publish it soon (next week hopefully).
In the meantime you can just use this 100-lines navigation controller subclass.

Drop it in your project.
Create a subclass of this ReNavigationController with popRoute() override where you put action dispatch for your popRoute(). (like in the pull request description above)

import UIKit

// dummy view controller returned when popViewController is canceled
public final class PopWasIgnored: UIViewController {}

open class ReNavigationController: UINavigationController {

    fileprivate var isSwipping: Bool = false

    // indicates that backButton was pressed when set to false
    fileprivate var isPerformingPop: Bool = false

    override open func viewDidLoad() {
        super.viewDidLoad()
        self.interactivePopGestureRecognizer?.addTarget(self, action: #selector(NavigationController.handlePopSwipe))
        self.delegate = self
    }

    override open func popViewController(animated: Bool) -> UIViewController? {

        // when swipping we are discarding all subsequent popViewController calls
        guard !self.isSwipping else {
            return PopWasIgnored()
        }

        self.isPerformingPop = true
        return super.popViewController(animated: animated)
    }

    // will be called after popViewController call
    func handlePopSwipe(){
        self.isSwipping = true
    }

    /// should be overriden
    /// normally you should dispatch SetRouteAction here
    open func popRoute(){
        print("WARNING: \(#function) should to be overriden")
    }
}

extension NavigationController: UINavigationControllerDelegate {

    public func navigationController(_ navigationController: UINavigationController, didShow viewController: UIViewController, animated: Bool) {
        self.isSwipping = false
    }
}

extension NavigationController: UINavigationBarDelegate {

    // if overriden navigationController popViewController won't be called before this method
    // on back button press
    // isPerformingPop will be false here in this case
    public func navigationBar(_ navigationBar: UINavigationBar, shouldPop item: UINavigationItem) -> Bool {
        defer { isPerformingPop = false }

        // changeRoute is performed:
        // 1. back button was pressed
        // 2. pop swipe was triggerred
        if !isPerformingPop || self.isSwipping {
            self.popRoute()
        }

        // don't remove the navigationItem if navigationController
        // is not going to be popped
        return isPerformingPop
    }
}

extension UINavigationController {

    open func pushViewController(_ viewController: UIViewController, animated: Bool, completion: @escaping () -> Void) {

        self.pushViewController(viewController, animated: animated)

        guard animated, let coordinator = self.transitionCoordinator else {
            completion()
            return
        }

        coordinator.animate(alongsideTransition: nil) { _ in completion() }
    }

    open func popViewController(animated: Bool, completion: @escaping () -> Void) -> UIViewController? {

        let popped = self.popViewController(animated: animated)

        guard animated, let coordinator = self.transitionCoordinator else {
            completion()
            return popped
        }

        coordinator.animate(alongsideTransition: nil) { _ in completion() }
        return popped
    }
}

Ah! I went ahead and made RoutableUIKit, before reading your comment @ambientlight ...

@hlineholm-flir: that's great! I am not sure if RoutableUIKIt is a the most intuitive name for it, but let's now add cocoapods and carthage support to it!

Not sure either about the name there @ambientlight. I've created issues for both creating a CocoaPod and supporting Carthage. Think I can fix it during this week.


Spike issues

Every time I try and add tally it crashes the add-on any ideas why?
gives me the error:
\addons\parse\display.lua:48: bad argument #1 'pairs' (table expected, got string)
   <defense>
                <bg>
                    <alpha>60</alpha>
                    <blue>0</blue>
                    <green>0</green>
                    <red>0</red>
                    <visible>true</visible>
                </bg>
                <data_types>
                    <absorb>percent</absorb>
                    <block>avg,percent</block>
                    <evade>percent</evade>
                    <hit>avg</hit>
                    <intimidate>percent</intimidate>
                    <parry>percent</parry>
                    <spike>tally</spike>
                </data_types>
                <flags>
                    <bold>true</bold>
                    <bottom>false</bottom>
                    <draggable>true</draggable>
                    <right>false</right>
                </flags>
                <max>2</max>
                <order>block,hit,parry,evade,absorb,spike</order>
                <padding>4</padding>
                <pos>
                    <x>150</x>
                    <y>440</y>
                </pos>
                <text>
                    <alpha>255</alpha>
                    <blue>255</blue>
                    consolas
                    <green>255</green>
                    <red>255</red>
                    <size>10</size>
                    <stroke>
                        <alpha>200</alpha>
                        <blue>0</blue>
                        <green>0</green>
                        <red>0</red>
                        <width>1</width>
                    </stroke>
                </text>
                <type>defense</type>
                <visible>false</visible>
            </defense>

This is related to another issue reported. It is due to the fact that there is no way to pre-declare how a non-default tag in the config should be read (as a table, instead of a string).
You should be able to fix it by adding this line:
["spike"] = S{'tally'},
after line 71 in parse.lua (after the intimidate line). This will allow it to correctly read the text inside the config tag.

That fixed it, thank you, and now I can be disappointed at how little Reprisal actually does. XD


Emergency button

Added large, responsive emergency resources button. Linked to the emergency resources page via React Router's  tag.
This addresses #19

Alright I changed the node keys! Should be ready to go!


CB-7025 redistribute parcels in every repair scenario

Looks better, although in one case I run into:
SDX repair failed 'aws-sdx57', com.sequenceiq.cloudbreak.cm.ClouderaManagerOperationFailedException: Command [Refresh cluster] failed: Failed to update refreshable configuration files in the cluster.. Detailed messages: RefreshDataNode(hdfs): Successfully refreshed DataNode configuration. Refresh Node List(hdfs): Failed to refresh DataNode allow/exclude lists.


Duplicate Entities with + in Title

Describe the bug
Battery sensors are appearing twice however the second iteration includes the + symbol at the end of the name.
How to reproduce
Create default card and display on dashboard
Expected behavior
Single occurance of entities
YAML configuration

# please paste here your card config

type: custom:battery-state-card
secondary_info: '{last_changed}'
filter:
  include:
    - name: attributes.device_class
      value: battery
  exclude:
    - name: entity_id
      value: binary_sensor.*
sort:
  by: state
collapse: 10
bulk_rename:
  - from: ' Battery'
  - from: ' level'
colors:
  steps:
    - '#ff0000'
    - '#ffff00'
    - '#00ff00'
  gradient: true

Dev console errors

Screenshots

Version
V3.1.6

I think you have just duplicated entities for some reason. If you look at the following one then you can even see that one has a separate formatting options specified so it has to be an additional entity

Please check Developer Tools -> States
BTW has this card worked for you in the past or are you just trying it now for the first time?

This is a function of the HACS Integration Battery Notes...

Then I guess I'm right that the additional entity is created in HA and your condition in "include" filter catches all of them. You can try to filter them out maybe by using exclude filter (when the name contains "+" char, there might be a better way to filter them out this was just an example)

BTW there is a setting on the battery notes integration which hides the original battery entities. Please check the docs
https://andrew-codechimp.github.io/HA-Battery-Notes/configuration/

Many thanks Max found the setting in Battery Notes. Took a couple of attempts but ultimately after adding the additional lines to the config I had to remove the Battery State Card integration and reinstall but all good now. Once again many thanks.


[fix eval] Fix issues with aider_bench remote runtime evaluation

End-user friendly description of the problem this fixes or functionality that this introduces

[ ] Include this change in the Release Notes. If checked, you must provide an end-user friendly description for your change below

Give a summary of what the PR does, explaining any non-trivial design decisions

Fixes minor setup and runtime limit issues with runtime config

Link of any specific issues this addresses

@ketan1741 You might need to fix the python linter

@ketan1741 You might need to fix the python linter

Fixed!


Feat/check message validity

Scope
Make sure the received message is what we expect.
Work Done

Replaced the verifySignature function with a verifyLogin function
The verifyLogin will check for both a valid signature and a valid message format
using the introduced verifyMessage function

Should we suggest Marigold as a reviewer here?


Resolve  #50311, book recommendation

Checklist:

[x] I have read and followed the contribution guidelines.
[ ] I have read and followed the how to open a pull request guide.
[x] My pull request targets the main branch of freeCodeCamp.
[x] I have tested these changes either locally on my machine or GitPod.

Closes #50311
Check the updated ipynb code: here
My though process:
The model I build predicted results that were not completely aligned with the recommended answers. Therefore, I updated the code to do so, after consulting some people on it. Now updated those code in here, to resolve the error.
I my be wrong in it, help me with that, if possible.

As a new contributor, we encourage you to read our contributing guidelines.
We expect our contributors to be aware of the process specific to this project. Following the guidelines religiously earns you the respect of fellow maintainers and saves everyone time.
Some examples of this are:

Do not edit files directly through GitHub – while you can, it's not a good idea.
Make sure you follow the PR checklist and not just tick things off; otherwise, we won't take you seriously.
Use the correct way to link issues in the description of the PR by updating the XXXXXX. Do not just add issue numbers everywhere and anywhere you feel like.
Keep mentions and review requests to a minimum. We understand you are excited about contributing, and our maintainers will get back to you as soon as they get a chance.
Do not work directly off your main branch - create a new branch for the changes you are working on.

We appreciate you taking the time to help us, and we hope to see more contributions from you.
Happy Contributing.

@Sembauke I have reviewed the PR guidelines. How should I approach it now?


Build fails when adding config plugin in expo

Description
I am not able to add the config plugin into my expo config file. The build fails with an error 'CommandError: Package "react-native-iap" does not contain a valid config plugin.
Learn more: https://docs.expo.dev/guides/config-plugins/#creating-a-plugin
Cannot use import statement outside a module'
Expected Behavior
App build should execute successfully after adding the config plugin
Screenshots

Environment:

react-native-iap: 11.0.0-rc.7
react-native: 0.71.7
Platforms (iOS, Android, emulator, simulator, device): Android/iOS device

To Reproduce
Steps to reproduce the behavior:
. Create an expo react native project
. Install react-native-iap
. Add config plugin and run  eas build -e production

check this link:
https://usmanabid477.medium.com/in-app-purchase-integration-in-react-native-app-96c0acd13649

11 Apr 2024 and I confirm that this error still occurs

add fingerprint to ur dependencies
yarn add @expo/fingerprint

I'm on version 12.15.2 and it still fails on android with expo config plugin. Can some one help me with this please ?

@masanishi could you find a solution to this issue on android? I'm currently blocked by this to release a new version of my app. please help

@andresesfm @jeremybarbet  pleeeeease help plzz
Since I added this package, ios builds are fine but androide fails with the same error above. I use expo with the config plugin as the docs suggests.
I don't know what to do from here. I tried setting kotling version but not working. I tried @zeta-gc solution above but nothing.


检出仓库报错

想保持仓库更新，检出仓库的时候报错
E:\note>git clone https://github.com/Wasabi1234/Java-Interview-Tutorial.git
Cloning into 'Java-Interview-Tutorial'...
remote: Enumerating objects: 821, done.
remote: Counting objects: 100% (413/413), done.
remote: Compressing objects: 100% (401/401), done.
remote: Total 821 (delta 17), reused 294 (delta 10), pack-reused 408
Receiving objects: 100% (821/821), 3.09 MiB | 3.94 MiB/s, done.
Resolving deltas: 100% (109/109), done.
error: invalid path 'Git/Git工作:面试必知必会.md'
fatal: unable to checkout working tree
warning: Clone succeeded, but checkout failed.
You can inspect what was checked out with 'git status'
and retry with 'git restore --source=HEAD :/'

仓库都是被我强制更新的，直接更新会有问题哈，建议直接 star 在线阅读！

不要再 fork，直接 star 即可


error: resource android:attr/lStar not found

I am using this package in my project it's working perfect in debug mode but while in profile mode and when I generate apk it shows error like this
Execution failed for task ':open_document:verifyProfileResources'.

A failure occurred while executing com.android.build.gradle.tasks.VerifyLibraryResourcesTask$Action
Android resource linking failed
ERROR:C:\Users\absol.gradle\caches\transforms-3\5e083c06c9925c2309f02468fc66e376\transformed\core-1.7.0\res\values\values.xml:105:5-114:25: AAPT: error: resource android:attr/lStar not found.

clear gradle cache
https://docs.gradle.org/current/userguide/build_cache.html#sec:build_cache_enable

I am having this exact same issue. To test, I started a brand new default Flutter project, added open_document ^1.0.5 to pubspec.yaml. Debug works fine, but building for release crashes with the same error:
Execution failed for task ':open_document:verifyReleaseResources'.

A failure occurred while executing com.android.build.gradle.tasks.VerifyLibraryResourcesTask$Action
Android resource linking failed
ERROR:C:\Users[my_user_directory].gradle\caches\transforms-3\0abb5ac90011bd27da412ddabf5e09c1\transformed\core-1.7.0\res\values\values.xml:105:5-114:25: AAPT: error: resource android:attr/lStar not found.

I remove open_document and the release builds without issue.
I've cleared all of my caches and indexes, flutter clean, and even specifically disabled gradle cache in my gradle properties: org.gradle.caching=false. Same results.
My environment:
Android Studio Dolphin | 2021.3.1 Patch 1
Flutter (Channel stable, 3.3.4, on Microsoft Windows [Version 10.0.19043.2006]
kotlin_version = '1.7.20'
com.android.tools.build:gradle:7.1.3 (also tried 7.3.0)
compileSdkVersion = 31
minSdkVersion = 16
targetSdkVersion = 31

anyone has solved the problem?
i need your help

how u solved it?

You can use OpenFile.open(filePath)
I hope this solves your problem.

You can use OpenFile.open(filePath) I hope this solves your problem.

Are you using the open_file plugin instead of open_document?
If this doesn't get resolved shortly I'll give that a shot, thank you.

you have a file created in your re--> values -> values.xml
in debug and main
\res\values\values.xml
you set up AndroidManifest.xml in main or so in debug
->> android ->> app ->> src ->> debug | main --> AndroidManifest.xml config
inside the re folder you created the xml folder and the file provider_paths.xml
->> res ->> xml -->> provider_paths.xml

you have a file created in your re--> values -> values.xml
in debug and main
\res\values\values.xml
you set up AndroidManifest.xml in main or so in debug ->> android ->> app ->> src ->> debug | main --> AndroidManifest.xml config

I have a res\values folder, but only a styles.xml file in that folder.
Should there be some other folder or file in there?

inside the re folder you created the xml folder and the file provider_paths.xml ->> res ->> xml -->> provider_paths.xml
<?xml version="1.0" encoding="utf-8"?>
<paths>
    <external-path
        name="external_files"
        path="." />
</paths>

Yes, xml -> provider_paths.xml is there and referenced in AndroidManifest.xml config. This is for path provider and is working fine.

https://stackoverflow.com/questions/69041630/android-build-error-lstar-not-found

https://stackoverflow.com/questions/69041630/android-build-error-lstar-not-found

Oh, right - that file is created by gradle when you build, it's not a core configuration file we can/should edit.
When the open_document plugin is included, it adds the lstar configuration to the values.xml file and build fails. When open_document is removed from the project, the lstar entry is not in the values.xml and the build completes successfully.
I already went through everything in that stackoverflow thread. This is something with the open_document plugin.

you are compiling apk by flutter va command in your project by android studio ,
open android folder in android studio
wait for the project to load and go to
Build -> Generate Signed Bundle
I do it this way it works

fix new version 1.0.6


Make a generic interface for event listeners in the Objective-C layer

At the moment, adding a new event listener requires us to write a fair amount of code in the Objective-C layer. We will, eventually, want to be adding a lot more event listeners, so making the interface for listeners more generic will be helpful in the long run.

Also, the callbacks are currently just Objective-C fields in the ShakaPlayerView.  We could instead do similar to what we do in C++ where we have a base class that defines the callbacks to invoke.


fix compile error introduced with ae1e3e040f9aa8ce18393c9aeeac47902d2f8ca3

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@ae1e3e0). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master    #6442   +/-   ##
=========================================
  Coverage          ?   82.98%
  Complexity        ?    13041
=========================================
  Files             ?     1495
  Lines             ?    48445
  Branches          ?     5864
=========================================
  Hits              ?    40200
  Misses            ?     5496
  Partials          ?     2749

Impacted Files
Coverage Δ
Complexity Δ

...va/io/crate/executor/transport/RerouteActions.java
82.92% <ø> (ø)
8 <0> (?)

Continue to review full report at Codecov.

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


0.19.2  release date ?

Hello,
When are you planning to release the 0.19.2 version ?
Thanks

@namusyaka is the mustermann upgrade preventing us from cutting a 0.19.2, ie. is it a breaking change?

I think it has no problem.
So we can cut the 0.19.2 release before releasing 1.0.0, but big features should be included in the 1.0.0.

@namusyaka Got a moment to do it? I can take care of it today if you're swamped otherwise.

Thanks!

@dblock I'm planning to release current master branch as 0.19.2.
Any other changes that you would like to add?

No go for it!

Just released 0.19.2, posted an announcement.
Thank you for all of your contributions.


Vagrant/Jekyll Server

Hey, just a thought, but Jekyll already has a server built it, is it needed to have Vagrant running as well?
if you just run "jekyll serve" itll launch its own web server, and if you do "jekyll serve --watch" itll actively watch for changes in sass/markdown.

Ah, yes - it's so that if you're a developer that doesn't want to pollute your ruby environment or fight the installation battle -> it'll install everything you need to get up and running without you having to worry about it. :D


Group Windows in Alt-Tab

Description of the new feature / enhancement
This is related to https://github.com/microsoft/PowerToys/issues/10061 which is yet to be implemented.
I'd like to have the windows of one app grouped in alt+tab. When I hover over one app, I'd like to have the ability to switch through all the windows in that app with win+tab.
Also the ability to close the windows with alt+f4 all together or one by one in win+tab would be very useful.
ExplorerPatcher has this function, would be nice to have it in PowertToys also.
Scenario when this would be used?

Supporting information
No response

Can you please update https://github.com/microsoft/PowerToys/issues/10061 instead of opening a duplicate?
Thanks
/dup #10061


Configure Weblate Project to address needs

Below are a list of things @tomy-pg needs to do for Weblate.
Acceptance Criteria

[x] Add source string bug report email address
[ ] Upgrade Weblate from 3.5.1 to 3.7.1
[ ] Use a repo under github.com/pg-irc

Right now with our current version, users are only allowed to make 20 translations per 60 second window.
With an upgrade to 3.7.1 this is bumped to 30 translations per 60 second window.
This issue is also related to: https://github.com/pg-irc/pathways-frontend/issues/720

This will be completed after the next production release.

@tomy-pg we need Weblate to manage the few strings for the next prod release.
can it at least be made usable for: 1)logging in and reviewing strings 2) copy pasting in translations?

@ArtiBPG If we are managing a few strings for the next production release, Weblate is ready to go.
The only concern is about the current rate limiting. Translators can only perform 20 translation actions within a 60 second window until we upgrade Weblate versions where the rate limit is slightly higher (30 actions in 60 seconds).
The rate limit was put in place to prevent malicious use. The developers of Weblate believe that for normal use IE: translating strings, approving and navigating, the rate limit that is currently in place is not restrictive.
Discussion on rate limiting can be found here: https://github.com/WeblateOrg/weblate/issues/2208

Our users all need to be logged in. Do we still need rate limiting to protect against malicious use?

This will be blocked until the first production release.

Upgrade Weblate to version 3.8.1 and is currently under the pg-irc repository.

QA:

[ ] Readme page at https://github.com/pg-irc/ui-strings contains stale content

QA: Pass


What the minimum version need of Service bus Python library which support the property of CorrelationId?

We didn’t see correlation id in 'message' document https://docs.microsoft.com/en-us/python/api/azure-servicebus/azure.servicebus.message?view=azure-python.
We would like to consult SDK team whether this feature realized in latest SDK.
Thank you!!!

[ ] What the minimum version need of Service bus Python library which support the property of CorrelationId?

Thank you for opening this issue! We are routing it to the appropriate team for follow up.

@wec2018 Thanks We Ling, I'm Jacky Guo(jacky.guo@clobotics.com), and this is my git hub account, I will follow this issue, and keep look forward which the version could to support. Thanks

Hi Scott,
Thank you for your reply!
Is there any team can help for this issue ? :)
Appreciate your help.
Best Regards,
Weiling

@annatisch ?

Hello Laurent,
Thanks! Could you help on this ?

Hello @annatisch
Could you please help on this ?
Thank you

Hi @wec2018
Sorry for the delay, we haven't exposed correlation ID directly in the service bus Message object, however you can access it via the Message.properties attribute:
sb_message = azure.servicebus.Message("some data")
sb_message.properties.correlation_id = "correlationID"

@annatisch
Thank you for your reply!
What the minimum version need of Service bus Python library which support the Message.properties attribute?

Thanks @wec2018 - that would be the current version - 0.50.0

Thank you for your help !! @annatisch

@wec2018 @annatisch Thanks all for help, I will try these days

I used the following code(0.50.0):
from azure.servicebus.control_client import ServiceBusService
from azure.servicebus.common.message import Message
bus_service = ServiceBusService(
host_base=host_base,
service_namespace=service_namespace,
shared_access_key_name=shared_access_key_name,
shared_access_key_value=shared_access_key_value)
message = Message("sb")
message.properties.correlation_id = 2
bus_service.send_topic_message(topic, message)
Traceback (most recent call last):
File "", line 1, in
File "/home/work/.local/lib/python2.7/site-packages/azure/servicebus/control_client/servicebusservice.py", line 663, in send_topic_message
request.headers = message.add_headers(request)
AttributeError: 'Message' object has no attribute 'add_headers'
After commenting this add_headers and resending the message, I checked the message in service bus, the correlation_id field was not set.
Any ideas?

Hi @annatisch
Could you please help on this new update from the customer ?
Thank you!

Hi @annatisch
Could you please help on this new update from the customer ?
Thank you!

Hi @annatisch
Good day to you! Could you please help on this new update from the customer ?
Thank you!

Hi @lmazuel
It seems that @annatisch is busy recently.
Coudl you please find other engineer to help us?
appreciate your help!!

Hi @wec2018,
The code you're using above is currently combining the old API (under the control_client namespace) and the new API.
If you wish to use the old API, you will need to import the Message object from the same namespace.
The old Message object supports correlation ID via providing broker properties:
from azure.servicebus.control_client import ServiceBusService
from azure.servicebus.control_client import Message
bus_service = ServiceBusService(
    host_base=host_base,
    service_namespace=service_namespace,
    shared_access_key_name=shared_access_key_name,
    shared_access_key_value=shared_access_key_value)

message = Message(body="sb", broker_properties={'CorrelationId': 'my_id_value'})
bus_service.send_topic_message(topic, message)

If you wish to use the new AMQP-based API, then the code will look more like:
from azure.servicebus import ServiceBusClient, Message

sb_client = ServiceBusClient(...)
topic_client = sb_client.get_topic("MyTopic")
with topic_client.get_sender() as sender:
    message = Message("sb")
    message.properties.correlation_id = 2
    sender.send(message)

Hi @annatisch,
Could you please help to check the customer's reply , appreciate for your help.
I have tested :
from azure.servicebus import ServiceBusClient, Message
from azure.servicebus import ServiceBusClient, Message
sb_client = ServiceBusClient(...)
topic_client = sb_client.get_topic("MyTopic")
with topic_client.get_sender() as sender:
message = Message("sb")
message.properties.correlation_id = 2
#message.properties.correlation_id = "2" was also tested
sender.send(message)
with azure.servicebus==0.50 python2.7.14,  it did not work
then, i tested another codes you mentioned before
from azure.servicebus import ServiceBusService
from azure.servicebus import Message
bus_service = ServiceBusService(
host_base=host_base,
service_namespace=service_namespace,
shared_access_key_name=shared_access_key_name,
shared_access_key_value=shared_access_key_value)
message = Message(body="sb", broker_properties={'CorrelationId': '2'})
bus_service.send_topic_message(topic, message)
with azure.servicebus==0.21.1 python2.7.14,  it still did not work. I did not find "azure.servicebus.control_client".
Yulu
2019-07-17

I believe this issue is now resolved - please let me know if I need to reopen.


Interesting results

Hi! I'm not sure this is an error, but I wanted to run it by you anyway. I'm running some models of a home heated by heat pump at different backup strip heat cutout temperatures, and the results I got are not what I was expecting.
Background: Basically, I took the same model and ran it through OCHRE using a cutout temperature of 40degF, 32degF, and then 25degF. The backup strip heat does indeed come on when the outdoor temperature hits those cutout temperatures, as expected. What was odd, however, was the indoor temperature. As the outdoor temperature dropped below about 20degF, the HVAC system had trouble meeting the 68degF indoor setpoint, and the indoor temperature started to drop. To be fair, the model obviously has some unmet load, as I was getting a similar result by running the model in beOPT.
What I thought was weird is that the 40deg cutout run had a harder time meeting the indoor setpoint than the 25deg cutout run. Why would the indoor temperature drop more for a model where the backup heat cuts on earlier and longer? Does this mean that the backup heat is undersized, or that the heat pump meets the HVAC load more easily? What do you think? I'd love to discuss!
I'm attaching the graph of the outputs. The other files I have are either too big or the wrong file type to attach. If you email me, I can try to send more.

. Here is the OCHRE code I used:
import os
import datetime as dt
from ochre import Dwelling
from ochre.utils import default_input_path # for using sample files
Modeling home that meets 2018 IECC, 40degF cutout temperature
house = Dwelling(
start_time=dt.datetime(2023, 1, 1, 0, 0),
time_res=dt.timedelta(minutes=1),
duration=dt.timedelta(days=31),
hpxml_file =os.path.join(r'C:\Users\cstone\OneDrive - Advanced Energy Corporation\Desktop\Projects\Flexible_Demand\OCHRE\DR_Standards_Comparison_2018NC_Ctrl.xml'),
schedule_input_file=os.path.join(r'C:\Users\cstone\OneDrive - Advanced Energy Corporation\Desktop\Projects\Flexible_Demand\OCHRE\schedules.csv'),
weather_file=os.path.join(r'C:\Users\cstone\OneDrive - Advanced Energy Corporation\Desktop\Projects\Flexible_Demand\OCHRE\USA_NC_Raleigh-Durham.Intl.AP.723060_TMY3.epw'),
verbosity=9,
Equipment={
'HVAC Heating': {
'use_ideal_capacity': True,
'show_eir_shr': True,
'Supplemental Heater Cut-in Temperature (C)': 4.44,
},
},
)
df, metrics, hourly = house.simulate()
Modeling home that meets 2018 IECC, 32degF cutout temperature
house = Dwelling(
start_time=dt.datetime(2023, 1, 1, 0, 0),
time_res=dt.timedelta(minutes=1),
duration=dt.timedelta(days=31),
hpxml_file =os.path.join(r'C:\Users\cstone\OneDrive - Advanced Energy Corporation\Desktop\Projects\Flexible_Demand\OCHRE\DR_Standards_Comparison_2018NC_Ctrl.xml'),
schedule_input_file=os.path.join(r'C:\Users\cstone\OneDrive - Advanced Energy Corporation\Desktop\Projects\Flexible_Demand\OCHRE\schedules.csv'),
weather_file=os.path.join(r'C:\Users\cstone\OneDrive - Advanced Energy Corporation\Desktop\Projects\Flexible_Demand\OCHRE\USA_NC_Raleigh-Durham.Intl.AP.723060_TMY3.epw'),
verbosity=9,
Equipment={
'HVAC Heating': {
'use_ideal_capacity': True,
'show_eir_shr': True,
'Supplemental Heater Cut-in Temperature (C)': 0,
},
},
)
df, metrics, hourly = house.simulate()
Modeling home that meets 2018 IECC, 40degF cutout temperature
house = Dwelling(
start_time=dt.datetime(2023, 1, 1, 0, 0),
time_res=dt.timedelta(minutes=1),
duration=dt.timedelta(days=31),
hpxml_file =os.path.join(r'C:\Users\cstone\OneDrive - Advanced Energy Corporation\Desktop\Projects\Flexible_Demand\OCHRE\DR_Standards_Comparison_2018NC_Ctrl.xml'),
schedule_input_file=os.path.join(r'C:\Users\cstone\OneDrive - Advanced Energy Corporation\Desktop\Projects\Flexible_Demand\OCHRE\schedules.csv'),
weather_file=os.path.join(r'C:\Users\cstone\OneDrive - Advanced Energy Corporation\Desktop\Projects\Flexible_Demand\OCHRE\USA_NC_Raleigh-Durham.Intl.AP.723060_TMY3.epw'),
verbosity=9,
Equipment={
'HVAC Heating': {
'use_ideal_capacity': True,
'show_eir_shr': True,
'Supplemental Heater Cut-in Temperature (C)': -3.89,
},
},
)
df, metrics, hourly = house.simulate()

I would assume this means that the HP capacity is larger than the ER capacity somewhere in the 25-40F range. I believe we output both of those values if verbosity >= 6. The ER capacity is constant and equal to the power consumption. The HP capacity varies with temperature, and should drop below the ER capacity at some point.


Move intro to index and add shields

Refining the intro page, preview at http://postgrest.org/en/preview/

I think it's an improvement, I have more ideas to refine the intro page but for now I'll just deploy this one.


Menu Component throws Error, I believe in conjunction with the popover element.

Copying the code straight to the react project from the material-ui documentation,
https://codesandbox.io/s/sm1zv?file=/demo.js
the following error appears;
Warning: findDOMNode is deprecated in StrictMode. findDOMNode was passed an instance of Transition which is inside StrictMode. Instead, add a ref directly to the element you want to reference. Learn more about using refs safely here: https://fb.me/react-strict-mode-find-node
    in div (created by ForwardRef(Paper))
    in ForwardRef(Paper) (created by WithStyles(ForwardRef(Paper)))
    in WithStyles(ForwardRef(Paper)) (created by Transition)
    in Transition (created by ForwardRef(Grow))
    in ForwardRef(Grow) (created by Unstable_TrapFocus)
    in Unstable_TrapFocus (created by ForwardRef(Modal))
    in div (created by ForwardRef(Modal))
    in ForwardRef(Portal) (created by ForwardRef(Modal))
    in ForwardRef(Modal) (created by ForwardRef(Popover))
    in ForwardRef(Popover) (created by WithStyles(ForwardRef(Popover)))
    in WithStyles(ForwardRef(Popover)) (created by ForwardRef(Menu))
    in ForwardRef(Menu) (created by WithStyles(ForwardRef(Menu)))

package.json
"@material-ui/core": "^4.11.0",
    "@material-ui/icons": "^4.9.1",
    "@testing-library/jest-dom": "^4.2.4",
    "@testing-library/react": "^9.5.0",
    "@testing-library/user-event": "^7.2.1",
    "react": "^16.13.1",
    "react-dom": "^16.13.1",
    "react-router-dom": "^5.2.0",
    "react-scripts": "3.4.3"

Solved in v5-alpha.

The exact example as in the sandbox does not throw the error. You have to wrap the app in React.StrictMode element.
Also happens with Dialog component with the example on the demo page https://material-ui.com/components/dialogs/#alerts

Hmm, wrapping my app in strict mode did not seem to do it. I am still getting the error for the elements Popper and SpeedDial. :( Both seem to function fine, its just sad to see the error

Wrapping your app with React.StrictMode element should produce the error, as it is doing extra checks that detect the error. Removing the React.StrictMode wrapping should "hide" the error.

That did it. Thanks for the help :)

just installed and first thing I tired got this bug.  Push the fix.

@sgehrman, Is this fix, or can this fix, be made available in 4.x?


how to fix this???

Describe your problem here.
source/vlc/MP4Handler.hx:5: characters 8-28 : Type not found : vlc.bitmap.VlcBitmap
source/vlc/MP4Handler.hx:11: characters 26-35 : Type not found : VlcBitmap
Are you modding a build from source or with Lua?
Source
What is your build target?
Mac
Did you edit anything in this build? If so, mention or summarize your changes.
no

lunarcleint help me with HUD switcher

Please

help as well

use hxCodec 2.5.1

didnt work

didnt work

haxelib install hxCodec 2.5.1

thx it worked

np


OSError: [Errno 24] Too many open files when RandomForestRegressor has 140 estimators

Here's a loop that fits and compiles trees, stepping up the number of estimators each time:
from sklearn import datasets, ensemble
import compiledtrees

data = datasets.load_boston()
X, y = data.data, data.target

for i in range(20, 250, 20):
    print(i)

    model = ensemble.RandomForestRegressor(n_jobs=4, n_estimators=i)
    model.fit(X, y)

    model = compiledtrees.CompiledRegressionPredictor(model)

    h = model.predict(X)

It crashes on 140:
$ python test_script.py
20
40
60
80
100
120
140
Traceback (most recent call last):
  File "/Users/ollieglass/code/test_compiled_trees/lib/python3.5/site-packages/joblib/_parallel_backends.py", line 344, in __call__
    return self.func(*args, **kwargs)
  File "/Users/ollieglass/code/test_compiled_trees/lib/python3.5/site-packages/joblib/parallel.py", line 131, in __call__
    return [func(*args, **kwargs) for func, args, kwargs in self.items]
  File "/Users/ollieglass/code/test_compiled_trees/lib/python3.5/site-packages/joblib/parallel.py", line 131, in <listcomp>
    return [func(*args, **kwargs) for func, args, kwargs in self.items]
  File "/Users/ollieglass/code/test_compiled_trees/lib/python3.5/site-packages/compiledtrees/code_gen.py", line 173, in _compile
    _call([CXX_COMPILER, cpp_f, "-c", "-fPIC", "-o", o_f.name, "-O3", "-pipe"])
  File "/Users/ollieglass/code/test_compiled_trees/lib/python3.5/site-packages/compiledtrees/code_gen.py", line 179, in _call
    shell=True, stdout=DEVNULL, stderr=DEVNULL)
  File "/usr/local/Cellar/python3/3.5.2/Frameworks/Python.framework/Versions/3.5/lib/python3.5/subprocess.py", line 576, in check_call
    retcode = call(*popenargs, **kwargs)
  File "/usr/local/Cellar/python3/3.5.2/Frameworks/Python.framework/Versions/3.5/lib/python3.5/subprocess.py", line 557, in call
    with Popen(*popenargs, **kwargs) as p:
  File "/usr/local/Cellar/python3/3.5.2/Frameworks/Python.framework/Versions/3.5/lib/python3.5/subprocess.py", line 947, in __init__
    restore_signals, start_new_session)
  File "/usr/local/Cellar/python3/3.5.2/Frameworks/Python.framework/Versions/3.5/lib/python3.5/subprocess.py", line 1454, in _execute_child
    errpipe_read, errpipe_write = os.pipe()
OSError: [Errno 24] Too many open files

This is on mac OS.
I haven't looked into workarounds - perhaps I can increase the number of files that can be open at once. But if there's a way to limit the open files in the library, that would probably be better.

I had a look at code_gen.py. Perhaps the CodeGenerator class could build a string instead of opening and writing to a file. When the .file method is called, it could write to a file, close it and return the name.

On Linux and macOS you have to do simply issue ulimit -n 2048. By design compiling trees consumes 2 * n_trees + 2 open files.
On Windows there is no way to raise the limit globally, but there is an internal solution:
import platform

if platform.system() == 'Windows':
    import win32file
    win32file._setmaxstdio(2048)

I used to write one cpp file, but it didn't work for large forests - especially if you have lots of data and allow for full growth. For my example this translate to 500 .cpp files over 100MB (50GB+ of RAM). Keeping all those files in StringIO's would probably work, although .o files would also still be there, so we would go down to ntrees + 2 open files (assuming we successfully close/delete files after compiling them to .o).
To sum up - I regard it as not an issue, and overcoming it would probably cost a lot of RAM in return, which ultimately is a deal-breaker (at least for me).

I see what you mean. I've fixed the problem for myself, like you say, it isn't hard.
I am concerned that users could be put off by this. How about an informative error for them, like this?
class CodeGenerator(object):
    def __init__(self):
        try:
            self._file = tempfile.NamedTemporaryFile(prefix='compiledtrees_', suffix='.cpp', delete=True)
        except OSError as e:
            print("Too many open files. Increase limit to 2 * n_trees + 2" \
                + "(unix / mac: ulimit -n [limit], windows: http://bit.ly/2fAKnz0)", file=sys.stderr)
            raise e

        self._indent = 0

That might be good solution if e.errno == 24 across platforms. As I remember correctly, on Windows I've got some kind of "Permission Denied" errors, which were terrible to debug...
Although I fear we will catch some false positives.
Also an unittest for that would be usefull (see hints on changing limits on all platforms)


Update CONTRIBUTING.md

This Pull Request resolves Issue #
Description:
In this Pull Request we have done the following additions/edits for the code:
Note: Delete all the text including this!

Make sure you replace #<IssueNumber> with the particular Issue that you are referencing with!
Make sure you describe the Issue properly and ask as many people as possible to review your Pull Request so that it can be merged as soon as possible!

Please send PR's only for Issues to be resolved not for simply adding your name in the document!

Spam


Config Description in preview mode occupying text while reading and not any way to make it go away.

Sanity checks

[X] I am using the latest version of the plugin
[X] I am using the latest version of Obsidian
[X] I have read the README

Diagnositics

Component
Version

Obsidian
1.15

AnkiBridge
0.6.17

Anki
.......

AnkiConnect
.......

Description
A clear and concise description of what the bug is.
Reproduce

Steps to reproduce the behavior:

Enter Preview Mode in Obsidian
Hover Over code block.

Expected behavior
I expect there to minimal or no interferance with the actual view. Or at least have proper styling. Or the possibility to make it go away.
Screenshots

I can't reproduce this. I think it might be related to the theme you are using. Could you see if this is also a problem if you use the default Obsidian theme?


Hbase 26308 Sum of multiplier of cost functions is not populated properly when we have a shortcut for trigger

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 28s
Docker mode activated.

-0 :warning:
yetus
0m  3s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ master Compile Tests _

+1 :green_heart:
mvninstall
4m  6s
master passed

+1 :green_heart:
compile
0m 20s
master passed

+1 :green_heart:
shadedjars
8m 20s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 19s
master passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
3m 42s
the patch passed

+1 :green_heart:
compile
0m 20s
the patch passed

+1 :green_heart:
javac
0m 20s
the patch passed

+1 :green_heart:
shadedjars
8m 16s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 17s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
8m 26s
hbase-balancer in the patch passed.

35m 48s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3710/1/artifact/yetus-jdk8-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/3710

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux c2a31773192c 4.15.0-112-generic #113-Ubuntu SMP Thu Jul 9 23:41:39 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / 121bdea230

Default Java
AdoptOpenJDK-1.8.0_282-b08

Test Results
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3710/1/testReport/

Max. process+thread count
346 (vs. ulimit of 30000)

modules
C: hbase-balancer U: hbase-balancer

Console output
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3710/1/console

versions
git=2.17.1 maven=3.6.3

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 27s
Docker mode activated.


Won't start after upgrade but says it starts

Hello!
We found we had to update our docker client after our pull from Docker Hub failed.
I went through the steps and the steps stated that docker started but it hadn't, no error message.
Setting up docker-engine (1.9.1-0~trusty) ...
Installing new version of config file /etc/init/docker.conf ...
Installing new version of config file /etc/init.d/docker ...
docker start/running, process 11444
Processing triggers for ureadahead (0.100.0-16) ...
$ ps -ef | grep docker
ubuntu   12604  1760  0 22:52 pts/0    00:00:00 grep --color=auto docker
$ sudo service docker start
docker start/running, process 11837
$ sudo docker ps -a
Cannot connect to the Docker daemon. Is the docker daemon running on this host?

The start script doesn't bubble up any error and there is no way to switch on verbose mode and it doesn't create the docker log file.
So, ran the command itself to get it to throw an error:
# DOCKER_SSD_PIDFILE=/var/run/$BASE-ssd.pid; DOCKER_LOGFILE=/var/log/$BASE.log; touch "$DOCKER_LOGFILE" && chgrp docker "$DOCKER_LOGFILE" && start-stop-daemon --start --background --no-close --pidfile "$DOCKER_SSD_PIDFILE" --exec "/usr/bin/docker" --make-pidfile -- daemon -p "$DOCKER_PIDFILE"
root@svr# INFO[0000] API listen on /var/run/docker.sock
FATA[0000] Error starting daemon: error initializing graphdriver: "/var/lib/docker" contains other graphdrivers: devicemapper; Please cleanup or explicitly choose storage driver (-s <DRIVER>)

Is devicemapper no longer supported?
How can I get the init script to show the errors when they come up instead of telling me that it's up when it's not?
Any help would be greatly appreciated.
Thanks!

Hi!
Please read this important information about creating issues.
If you are reporting a new issue, make sure that we do not have any duplicates already open. You can ensure this by searching the issue list for this repository. If there is a duplicate, please close your issue and add a comment to the existing issue instead.
If you suspect your issue is a bug, please edit your issue description to include the BUG REPORT INFORMATION shown below. If you fail to provide this information within 7 days, we cannot debug your issue and will close it. We will, however, reopen it if you later provide the information.
This is an automated, informational response.
Thank you.
For more information about reporting issues, see https://github.com/docker/docker/blob/master/CONTRIBUTING.md#reporting-other-issues

BUG REPORT INFORMATION
Use the commands below to provide key information from your environment:
docker version:
docker info:
uname -a:
Provide additional environment details (AWS, VirtualBox, physical, etc.):
List the steps to reproduce the issue:
1.
2.
3.
Describe the results you received:
Describe the results you expected:
Provide additional info you think is important:
----------END REPORT ---------
#ENEEDMOREINFO

The issue with it not actually starting is most likely because you have used multiple graph drivers (so multiple graph dirs in /var/lib/docker). Since the driver is not explicitly set in your daemon opts there is no way for Docker to know which driver to choose.
Should set this explicitly in your daemon opts.

Yes, looks like that. Given that you're running on "trusty", I think both aufs and devicemapper have been used (aufs is the default driver for ubuntu trusty).
The error message ;
"/var/lib/docker" contains other graphdrivers: devicemapper indicates that a "devicemapper" directory is also found, and Please cleanup or explicitly choose storage driver (-s <DRIVER>) explains that the unwanted directory should be removed, or the -s option should be set as a daemon option, to specify which driver you'd like to use.
Closing this issue, because it's not a bug, but feel free to comment here after I've closed

Yes, I read that error and the helpful message with the -s but my issue was that the start script doesn't say that it can't start - it indicates that it starts fine.
Only after I pulled the start command from the init script was I able to get that error. It seems like a bug if the init script is not working properly.

the init script is located here; https://github.com/docker/docker/blob/master/contrib/init/upstart/docker.conf
After it failed, does status docker still output start/running...?

No, I think it tried to start, got a pid, found it couldn't start and died.
And that conf file is exactly what I needed! Added the -s and now it started.
Thanks for all the help!


dbeaver/pro#2135 Optimize rules scanner

Fixed an issue with lagged highlighting on editing (was not stable reproducible).
Bug illustration (no highlighting on the first line) :

Optimized rules scanner to remove redundant context requests, so highlighting should work a bit faster now.
What should be tested:
At first, turn these settings on
Then:

check mentioned bug existence
tables should be colored with one color, columns - with another one. Also if you wrote select with invalid column (which doesn't exist in the database) it should be highlighted with red.

try editing the query and check that highlighting updates

verified


NATS password and user should be optional

Describe the bug
When allow legacy clients is set to false, NATS doesn't use username and password anymore: https://github.com/cloudfoundry/bosh/blob/master/jobs/nats/templates/nats.cfg.erb#L9-L10
We should make this optional in the spec file, such that I don't need to care about credentials which aren't needed. It leads to confusion around operators in terms of rotation and maintenance of those credentials.
To Reproduce
Steps to reproduce the behavior (example):

Deploy a bosh director with allow_legacy_clients: false
Don't specify nats.username and nats.password. More specifically, I can apply the following ops-file to bosh-deployment.

---
- type: remove
  path: /variables/name=nats_password

- type: remove
  path: /instance_groups/name=bosh/properties/agent/mbus

- type: remove
  path: /instance_groups/name=bosh/properties/nats/user

- type: remove
  path: /instance_groups/name=bosh/properties/nats/password

Deployment fails because you didn't specify necessary properties – they don't have a default value

Expected behavior
It works. All clients connect to NATS via certificates and don't need username or password. Ideally, the ops-file pasted above should be part of https://github.com/cloudfoundry/bosh-deployment/blob/04c85a5c79a9fa6b92775386a334104b9a165013/misc/nats-strict-tls.yml
Logs
Task 1025

Task 1025 | 15:50:39 | Preparing deployment: Preparing deployment (00:00:06)
Task 1025 | 15:50:52 | Error: Unable to render instance groups for deployment. Errors are:
  - Unable to render jobs for instance group 'bosh'. Errors are:
    - Unable to render templates for job 'openstack_cpi'. Errors are:
      - Error filling in template 'cpi.json.erb' (line 101: Can't find property '["nats.user"]')

Versions (please complete the following information):

Infrastructure: all
BOSH version 268.2.0

Deployment info:
bosh-deployment

Turns out, once I made the properties in the CPI optional, template rendering went through smoothly, thanks @xtreme-andrew-su!
Now: should we make the removal of the nats properties an additional ops-file in bosh-deployments such that you can use it when the CPI supports it?

I think that's the next step once we have the appropriate stemcell defined in the cpi.yml
afaik, version xenial 170.2+ of heavy stemcells is registry-less ready, the light stemcells will be any version after 170.6+. Once those are out, I'm all for adding a new ops-file.
Going to close this issue for now. Added a story: https://www.pivotaltracker.com/story/show/162017526


Slow down preview gif

Using imagemagick:
convert -delay 75x100 preview.gif preview_slow.gif
Fixes #46
Developer Certificate of Origin

[x] I certify that these changes according to the Developer Certificate of Origin 1.1 as described at https://developercertificate.org/

Ah good ole imagemagic. This definitely slows things down, but because it just reduces the frame rate, it leaves the final gif choppy. It might be best to re-make the gif and have it show off certain aspects of PaperCSS rather than trying to get the whole page scrolled through in 1.8 seconds.

I agree its not perfect but the fast gif right now is very jarring. I would highly recommend taking some action on it in the short term (even if its just to remove it).
Feel free to close this PR if this isn't the short term route we want to go with though.

The gif picture will be replaced soon once we get docs and a few examples app. We will post a screen of a UI made with papercss.


Sync eng/common directory with azure-sdk-tools for PR 9057

Sync eng/common directory with azure-sdk-tools for PR https://github.com/Azure/azure-sdk-tools/pull/9057  See eng/common workflow

Failures are unrelated to this PR. I'm overriding check-enforcer

/check-enforcer override


Try to keep all Aspen samples on the tree

Description
Inspired by the flexibility of current set up and this: https://nextstrain.github.io/ncov/multiple_inputs.html
try a simple way to make sure trees contain all Aspen genomes.

Nextstrain takes 2 input: Aspen data and Gisaid data
For genomes on both Aspen and Gisaid, de-duplication keeps the Aspen copy, so none of the Aspen genomes would be removed.
Disable subsample for Aspen data; only subsample Gisaid data
Side note: Aspen data into a tree build workflow are already filtered by group, so no more filter by division is needed. This means a county can keep all genomes in its group (or can-see group in the future) on its trees regardless of whether those samples were collected in their own counties. This set up is so flexible and quite awesome.

Issue

Anticipated user feedback with subsampling: where are my samples?
Don't know whether Santa Clara tree will take too long to build with no subsampling.

Test plan
I do not know how to test.... Will learn how to trigger tree run from Shannon tomorrow.

Still confused about certain things. Close this for now.


deepin desktop environment: update

Description of changes
New application

dde-shell: The taskbar has been refactored using QML.
dde-grand-search: System-wide desktop search
deepin-anything: file search tool

https://www.deepin.org/en/deepin-v23-rc-is-officially-released/

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

@ofborg test deepin

@ofborg test deepin


refactor: service client

1. Does this PR affect any open issues?(Y/N) and add issue references (e.g. "fix #123", "re #123".):

[x] N
[ ] Y

refactor: service client

Pull Request Test Coverage Report for Build<PHONE_NUMBER>0
Details

0 of 24 (0.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.6%) to 36.786%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pkg/server/rest_server.go
0
24
0.0%

Totals

Change from base Build<PHONE_NUMBER>9:
-0.6%

Covered Lines:
4901

Relevant Lines:
13323

💛  - Coveralls


Allow manual control legend elements visibility for plots

ShowInLegend property - public setter was added
The default value was set as true to not to break the previous default value (Visible property is true by default)
Preventing legend from rendering, when ShowInLegend is true but Visible is false

This implementation is a breaking change. Toggling the Visible property of a series won't hide it from the legend.

Hi, thanks for a reply, this clear things :)
If I understood you right, you assume that with this change, when any plot will have Visible == false, a series will be visible in a legend anyway.
I believe my second commit 633ba160114a1da3ee2ea971beeb6dd773beb9f0 prevents that kind of behavior.
Yes, I want to have a series that doesn't appear in the legend. As an example, I will attach a screenshot from my current project where I'm using Radzen lib.

I want to define my custom "header" for this line plot.
On phones screens because of legend label graph part is squished, so I want to place it on top of the section.

I believe my second commit 633ba16 prevents that kind of behavior.

Check the Area chart demo in your pull request:

Compare it to how it behaves live:

I think you can hide the legend via CSS on mobile devices without this pull request:
@media (max-width: 768px) {
   .rz-legend {
       display: none;
   }
}

😮 Ok, that behavior with clicking on a legend's member change a lot. Thanks for clarifying.
I will close the PR, because it's useless in that case


Add Goreleaser support for releases and homebrew

WHAT
This adds a config for goreleaser to manually create releases via goreleaser release and also automatically via github actions when a new tag is pushed. I have an example in my fork and tap repo. brew install rothgar/tap/kubecolor works.
You'll need to create a github repo called dty1er/homebrew-tap/ and you'll need to create a GITHUB_TOKEN with access to push into that repo. You can read more about it here https://docs.github.com/en/free-pro-team@latest/actions/reference/authentication-in-a-workflow#using-the-github_token-in-a-workflow
You can create the token at https://github.com/settings/tokens It should only need repo access. If you want to use goreleaser to publish .rpm and .deb packages or docker containers you can also add package access.
You should make sure the token is set up in the repo settings as GITHUB_TOKEN so goreleaser can use it automatically.
Let me know if you run into problems or have questions. :+1:
WHY
So releases are automatic and brew install works
Related issue
Fixes #27

Codecov Report

Merging #35 (f27a87c) into main (de74e2b) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #35   +/-   ##
=======================================
  Coverage   84.26%   84.26%
=======================================
  Files          16       16
  Lines         521      521
=======================================
  Hits          439      439
  Misses         73       73
  Partials        9        9

Continue to review full report at Codecov.

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

@rothgar
Hi, thanks a lot for sending this PR!
I'll try it out and make sure if it works as we expected.

@rothgar
Thank you for the PR!
I will merge this and make sure this works.

After running below, the GH action started: https://github.com/dty1er/kubecolor/runs/1410458503
git tag v0.0.7
git push origin v0.0.7

It successfully finished: https://github.com/dty1er/kubecolor/runs/1410458503

v0.0.7 release is made!
https://github.com/dty1er/kubecolor/releases/tag/v0.0.7

[x] Changelog
[x] Binary for each environment

Also, I downloaded the build binary for Darwin and made sure it works.

And homebrew-tap: https://github.com/dty1er/homebrew-tap/blob/master/Formula/kubecolor.rb
The formula is made.

I tried brew install dty1er/tap/kubecolor after removed current kubecolor under $GOPATH/bin.
~  $ type kubecolor
kubecolor is hashed (/usr/local/bin/kubecolor)

~  $ brew list kubecolor
/usr/local/Cellar/kubecolor/0.0.7/bin/kubecolor

~  $ brew info kubecolor
dty1er/tap/kubecolor: stable 0.0.7
Colorize your kubectl output
https://github.com/dty1er/kubecolor
/usr/local/Cellar/kubecolor/0.0.7 (5 files, 2.1MB) *
  Built from source on 2020-11-17 at 14:43:11
From: https://github.com/dty1er/homebrew-tap/blob/HEAD/Formula/kubecolor.rb
~  $

Made sure kubecolor installed via brew also works!

@rothgar
Thank you for your effort!
Now kubecolor gets much easier to install!


Plank sack number of planks overlay

Knowing the amount of planks that remain within your sack using an overlay that is similar to what the runepouch overlay does would be fantastic and really round off this great plugin.
https://i.imgur.com/OibEJWL.png

As far as I'm aware this functions similarly to the looting bag where the user/client doesn't know the contents in the bag unless checked/prompted by the server. Tracking plank usage manually from animations or repair spots would be the only way to accomplish this, and I'm not sure how reliable that method would be (I don't really do this activity since I'm 99 con already).
Because of this I doubt this feature will ever be added, sorry.

I see this is marked as wontfix. Does this mean you won't accept pull requests of this issue either or just that it's not something you're interested in making yourself? Because i'm interested in looking into it myself.

I see this is marked as wontfix. Does this mean you won't accept pull requests of this issue either or just that it's not something you're interested in making yourself? Because i'm interested in looking into it myself.

Fyi, this is something that already exists as the Plank Sack plugin maintained by Enriath/external-plugins

Fyi, this is something that already exists as the Plank Sack plugin maintained by [Enriath/external-plugins]
(https://github.com/Enriath/external-plugins/tree/plank-sack)

Closing as this plugin exists and I'm not going to reinvent what Enriath has already done.


Issues with manual chunk parsing code

There are a few issues with the manual chunk parsing code.

If we have a chunk size of 0, the parsing state is set to parsing_done.  We should instead set the state to end_of_chunk so that we can catch invalid end chunk sequences.

If we are parsing a chunk header of the form <chunk size hex>\r\n and but don't find the \r\n we originally continued.  However, if the input is exhausted then we should trigger a parsing error.

Same with (2) above but with a chunk header of the form <chunk size hex>;extension1=value1;extension2=value2\r\n.  If we don't find the \r\n then we need to trigger a parsing error.

When parsing the end of a chunk, we look for the end of chunk sequence \r\n.  If that is not found we trigger an error.  However, we need to consider the possibility that we have only 0 or 1 bytes left and can read more from the parser.  This is most obvious when setting the read buffer length to 1.

Finally, on every loop of the parser we read more data into a temporary string.  We should only read more data if the parsing state indicates that more data is necessary.  Otherwise this string can grow very large.

As an example of (5) let's say we have the following chunked sequences:
b\r\n
chunk body1\r\n
b\r\n
chunk body2\r\n
0\r\n\r\n

Let's say we've read the following from the stream:
b\r\n
chunk body1\r\n

On our next parsing loop we are parsing the chunk_header.  Currently that loop will read more data into the string but that is unnecessary at this point in the parsing.

These changes have been implemented.  Closing this issue.

This was in commit 53bf3f7e3177f8dc126412baedd7335c0059dc01.


As a user, I want the graph to highlight time in which a designated series passes a threshold, so I can easily see when anomalies occur.

See this PR for some work in progress: https://github.com/breznak/nupic/pull/16
It would be good if the user could select a time series in the options UI, and then use a slider to set a threshold, based on the min-max of that particular series' data.

@jefffohl can you give me any tips for which framework to grab here? I'm trying to read up on "scientific JS" but I'm a bit lost. I want something like numpy in JS, for :
anomalyIdx = numpy.where(loadedCSV[seriesId] > 0.9)

I don't know of a library with that exact function. I could write a version of one for you, if you like. Or, we could look around for a library that does what you are looking for. Here is a page I found that lists a number of math libraries: http://jster.net/category/math-libraries

Thank you Jeff. I'll try to find a numpy-like JS lib. I have a nice idea how to improve the threshold detection dynamically.

I have an example of highlighting in River View: http://data.numenta.org/chicago-beach-water-quality/Calumet Beach/data.html

But this is conditional only on the X time value, not the actual Y value, which I have found to be particularly tricky. Anyway, here is the messy code that does this: https://github.com/nupic-community/river-view/blob/master/site/templates/scalar-data.html#L39-L85

Cool @rhyolight ! :sunglasses: this is the kickstart I need


Adapters are not setup for non-Flux functions in non-catalog environments

Basically the description below is valid for all adapters. The only thing I'd like to add is that this happens in the scenario where FunctionCatalog is not available and functions are retrieved directly from BeanFactory
====
Unfortunately, I didn't get any answer on my question on StackOverflow so far, so maybe somebody can help me here.
In the past, I've implemented a function and deployed it on AWS Lambda, which worked perfectly.
But when I now try deploy the same function on Azure Functions, that does not work:
24.01.19 21:58:50] Caused by: java.lang.ClassCastException: reactor.core.publisher.FluxJust cannot be cast to java.lang.String [24.01.19 21:58:50]     at de.margul.awstutorials.springcloudfunction.function.EchoFunction.apply(EchoFunction.java:9) [24.01.19 21:58:50]     at org.springframework.cloud.function.adapter.azure.AzureSpringBootRequestHandler.handleRequest(AzureSpringBootRequestHandler.java:56) [24.01.19 21:58:50]     at de.margul.awstutorials.springcloudfunction.azure.handler.FunctionHandler.execute(FunctionHandler.java:19) [24.01.19 21:58:50]     ... 16 more
I've created a minimal example to reproduce the problem.
You have to install the Azure Function Core Tools to run the example locally.
After installation, you can build and run the function using:
mvn package
mvn azure-functions:run
To invoke the function, you have to call:
curl -d 'test' -X POST http://localhost:7071/api/echoFunction
Note: you can switch between the working and the not working function by changing the function.name attribute in the local.settings.json.
This function works:
@Component
public class FluxEchoFunction implements Function<Flux<String>, Flux<String>> {
    @Override
    public Flux<String> apply(Flux<String> m) {
        return Flux.just("Echo: " + m);
    }
}

and this function does not work (throws the exception above):
@Component
public class PlainEchoFunction implements Function<String, String> {

    @Override
    public String apply(String m) {
        return "Echo: " + m;
    }
}

I think this is the relevant part in the documentation:

One of the main features of Spring Cloud Function is to adapt and support a range of type signatures for user-defined functions, while providing a consistent execution model. That’s why all user defined functions are transformed into a canonical representation by FunctionCatalog, using primitives defined by the Project Reactor (i.e., Flux and Mono). Users can supply a bean of type Function, for instance, and the FunctionCatalog will wrap it into a Function,Flux>.

However, according to that, I would expect that I don't have to define my function like Flux<String> apply(Flux<String> m), right?

@olegz , do you have any idea whether this is a bug, or am I just doing something wrong?

@markusgulden could you please give it another shot with the latest snapshot? I'll do my due diligence with your sample, but I believe the latest commit may have indirectly fixed this issue.
Anyway, let me know if/when you get a chance

@olegz , unfortunately, die latest snapshot does not seem to have the problem fixed.
However, I found another way to solve the issue.
Until now, my function handler looked like this:
public class FunctionHander extends AzureSpringBootRequestHandler<String, String> {
    @FunctionName("echoFunction")
    public String execute(@HttpTrigger(name = "req", methods = {
            HttpMethod.POST }, authLevel = AuthorizationLevel.ANONYMOUS) HttpRequestMessage<String> entity,
            ExecutionContext context) {
        return handleRequest(entity.getBody(), context);
    }
    @Bean
    public FluxEchoFunction fluxEchoFunction() {
        return new FluxEchoFunction();
    }

    @Bean
    public PlainEchoFunction plainEchoFunction() {
        return new PlainEchoFunction();
    }
}

When I change it now to the following, I can call my PlainEchoFunction without errors:
@SpringBootConfiguration
public class FunctionHander extends AzureSpringBootRequestHandler<String, String> implements ApplicationContextInitializer<GenericApplicationContext>{
    @FunctionName("echoFunction")
    public String execute(@HttpTrigger(name = "req", methods = {
            HttpMethod.POST }, authLevel = AuthorizationLevel.ANONYMOUS) HttpRequestMessage<String> entity,
            ExecutionContext context) {
        return handleRequest(entity.getBody(), context);
    }

    @Override
    public void initialize(GenericApplicationContext context) {
      context.registerBean("plainEchoFunction", FunctionRegistration.class,
          () -> new FunctionRegistration<>(new PlainEchoFunction())
              .type(FunctionType.from(String.class).to(String.class)));
    }
}

Does that help with analyzing the issue?

It certainly does, let me dig in

@markusgulden As you may have seen I've commented on your SO question. Basically we have a bug. Will take care of it before next release

@markusgulden your input definitely helped. I figured out the issue and it is actually affecting all adapters. So, I'll push by Monday after I move some things around

Glad to hear that you found the bug, @olegz
Because even if I made it to get the simple example from above working, the issue still occurs with something more complex like a Function<Message, Message>.

So, the issue is really not related to Azure and rather common across multiple adapters, so I'll modify the title slightly.
The issue has been fixed indirectly on the master with this commit.
I am about to merge backport to 2.0.x after which I'll close it, so give it a shot as I assume there may be some additional issues lingering

resolved with https://github.com/spring-cloud/spring-cloud-function/commit/c20c87d1dd0ea31ea377bc5c2bba398d149c5016 and https://github.com/spring-cloud/spring-cloud-function/commit/0189c578efe085e273eece62f54bf8bdb86948f4

I get the same thing with spring-cloud-function-adapter-aws 2.0.1.RELEASE. If I Use @Import to load my application config instead of @SpringBootApplication I get the following error.
java.lang.ClassCastException: reactor.core.publisher.FluxJust cannot be cast to com.amazonaws.services.lambda.runtime.events.DynamodbEvent
at com.example.pet.function.HandleDBEvent.apply(HandleDBEvent.java:20)
at org.springframework.cloud.function.adapter.aws.SpringFunctionInitializer.apply(SpringFunctionInitializer.java:132)
at org.springframework.cloud.function.adapter.aws.SpringBootRequestHandler.handleRequest(SpringBootRequestHandler.java:48)
I am not sure what extra configuration I need if I am not using the autoconfig.

@ruewan What made you believe it would work in 2.0.1.RELEASE when it was reported as a bug?
It was fixed and is available in 2.1.0.M1 as well as the snapshots


conversion between ent.Tx and dialect.Tx

How can I convert ent.Tx to dialect.Tx and vice versa? I tried to convert as below but failed due to interface not compatible. I found a similar issue #3583 but it seems not resolved.
       var cli *ent.Client
       var sqlDrv *sql.Driver
	tx, err := cli.Tx(ctx)
	tx1, err := sqlDrv.BeginTx(ctx,nil)
        // if any one of below works, it's ok!
	tx2 := tx.(sql.Tx)                                          // report interface not implemented
	execQuerier := tx.Client().(sql.ExecQuerier)                        // report interface not implemented
	execQuerier1 := tx.(sql.ExecQuerier)                                  // report interface not implemented
       // or can derive tx from tx1

How can I mix usage of ent.Tx and dialect.Tx in the same transaction?

Hello, I think you are getting things mixed up.

ent.Tx is a transactional client.
dialect.Tx is what is returned when you start a transaction with dialect.Driver, it represents what should be present in the struct returned when starting a transaction.

Now if you look at the source code, it will see that dialect.Driver, dialect.Tx are interfaces and are only used by the driver of the client.
If you want to handle the operations manually, you need to get the driver first (and handle all possible edge cases, like drivers that wrap another drivers and difference between drivers). To get to the driver you can add a new file on the ent directory (in your project).
See this link for more info: https://github.com/ent/ent/issues/3147#issuecomment-1339423938
With all that said, this seems to be a XY Problem, seems like you want to access the database to do operations on Exec and Query, Ent has a feature flag for it, SQL Raw API.
For most of the cases it should be enough, I think you will only need to work with drivers when implementing a new dialect, doing custom query inspections or caching strategies.
Please, ask questions like this on Ent Discord, there are many more people that use discord and monitor it that are able to help you there.

Hello, I think you are getting things mixed up.

ent.Tx is a transactional client.
dialect.Tx is what is returned when you start a transaction with dialect.Driver, it represents what should be present in the struct returned when starting a transaction.

Now if you look at the source code, it will see that dialect.Driver, dialect.Tx are interfaces and are only used by the driver of the client.
If you want to handle the operations manually, you need to get the driver first (and handle all possible edge cases, like drivers that wrap another drivers and difference between drivers). To get to the driver you can add a new file on the ent directory (in your project).
See this link for more info: #3147 (comment)
With all that said, this seems to be a XY Problem, seems like you want to access the database to do operations on Exec and Query, Ent has a feature flag for it, SQL Raw API.
For most of the cases it should be enough, I think you will only need to work with drivers when implementing a new dialect, doing custom query inspections or caching strategies.
Please, ask questions like this on Ent Discord, there are many more people that use discord and monitor it that are able to help you there.

Thank you so much!

Just summary how I solved the problem.
I added gen.FeatureExecQuery in the generate opts.
	err := entc.Generate(
		"./schema",
		&gen.Config{
			Hooks: []gen.Hook{
				JsonTag,
				RedisTag,
			},
			Features: []gen.Feature{
				gen.FeatureUpsert,
				gen.FeatureExecQuery,
			},
			Templates: []*gen.Template{
				gen.MustParse(
					gen.NewTemplate("txdriver").
						ParseFiles("template/txdriver.tmpl"),
				),
			},
		},
	)

Now I can exec sql via below two functions and Meantime I can use tx.User.Create(), tx.User.Update() ... as well.
tx.QueryContext()
tx.ExecContext()


[Firestore] Regional endpoint support?

This seems to be a new feature. In the documentation there are currently only examples for Java and Python.
Is this feature not yet available in the node library? If it is, how can I specify regional endpoint?

Looks like you specify the regional endpoint through the host setting, which is available in the Node.js library.
    const db = initializeFirestore(app);

    db.settings({
        host: 'us-east1-firestore.googleapis.com'
    });

    await db.collection("foo").add({time: Date.now()});
    const result = await db.collection("foo").get();
    ```

    @morganchen12 do you know why the documentation for this feature only covers Java and Python?

@lahirumaramba The snippets for other libraries have not been generated, but the host setting can be used in any SDK to set a regional endpoint. DevRel is aware of this gap. You may choose to close this ticket.

Thank you @MarkDuckworth !


needextend shall support dynamic functions

observed behavior
Hi @danwos , when I try the following code, I get an error:
.. needextend:: SYSFCT_Foo
    :+implements: [[get_matching_need_ids("REQ_SYS_Foo_")]]

With the following dynamic function definition:
def get_matching_need_ids(app, need, needs, id_prefix="") -> List[str]:
    """Get a list of needs matching id_prefix

    Example Usage::

        :derives: [[get_matching_need_ids("REQ_SYS_TSR_")]]

    Args:
        app: app object (sphinx-needs dynamic functions API parameter)
        need: current need (sphinx-needs dynamic functions API parameter)
        needs: dictionary of all needs (sphinx-needs dynamic functions API parameter)
        id_prefix: needs ID filter. Only returns needs which's ID starts with id_prefix.

    Returns:
        List[str]: list of ids for needs matching all filter criteria
    """
    filtered_needs = []
    for need_id, need_entry in needs.items():
        if id_prefix and need_id.startswith(id_prefix):
            filtered_needs.append(need_id)
    return filtered_needs

I get the following error:
Exception occurred while building, starting debugger:
Traceback (most recent call last):
  File "/home/arwedus/workspace/.sphinx-venv/lib/python3.8/site-packages/sphinx/cmd/build.py", line 281, in build_main
    app.build(args.force_all, args.filenames)
  File "/home/arwedus/workspace/.sphinx-venv/lib/python3.8/site-packages/sphinx/application.py", line 347, in build
    self.builder.build_update()
  File "/home/arwedus/workspace/.sphinx-venv/lib/python3.8/site-packages/sphinx/builders/__init__.py", line 310, in build_update
    self.build(to_build,
  File "/home/arwedus/workspace/.sphinx-venv/lib/python3.8/site-packages/sphinx/builders/__init__.py", line 376, in build
    self.write(docnames, list(updated_docnames), method)
  File "/home/arwedus/workspace/.sphinx-venv/lib/python3.8/site-packages/sphinx/builders/__init__.py", line 568, in write
    self._write_parallel(sorted(docnames),
  File "/home/arwedus/workspace/.sphinx-venv/lib/python3.8/site-packages/sphinx_monkeypatch/__init__.py", line 207, in _write_parallel
    doctree = self.env.get_and_resolve_doctree(docname, self)
  File "/home/arwedus/workspace/.sphinx-venv/lib/python3.8/site-packages/sphinx/environment/__init__.py", line 591, in get_and_resolve_doctree
    self.apply_post_transforms(doctree, docname)
  File "/home/arwedus/workspace/.sphinx-venv/lib/python3.8/site-packages/sphinx/environment/__init__.py", line 642, in apply_post_transforms
    self.events.emit('doctree-resolved', doctree, docname)
  File "/home/arwedus/workspace/.sphinx-venv/lib/python3.8/site-packages/sphinx/events.py", line 94, in emit
    results.append(listener.handler(self.app, *args))
  File "/home/arwedus/workspace/.sphinx-venv/lib/python3.8/site-packages/sphinx_needs/directives/need.py", line 393, in process_need_nodes
    process_needextend(app, doctree, fromdocname)
  File "/home/arwedus/workspace/.sphinx-venv/lib/python3.8/site-packages/sphinx_needs/directives/needextend.py", line 142, in process_needextend
    if found_need["id"] not in env.needs_all_needs[ref_need][f"{option_name}_back"]:
KeyError: '[[get_matching_need_ids(filter="id.startswith(\'REQ_SYS_Foo_\')")]]'
> /home/arwedus/workspace/.sphinx-venv/lib/python3.8/site-packages/sphinx_needs/directives/needextend.py(142)process_needextend()
-> if found_need["id"] not in env.needs_all_needs[ref_need][f"{option_name}_back"]:

expected behavior
Dynamic functions wrapped in [[]] shall work in needextend options just like in need directive options.
possible root cause
The code in needextend.py treats everything that is found in a link_name field like an ID (https://github.com/useblocks/sphinx-needs/blob/11d6b1bef834946fcfb28eabc3611f48dc4c72cd/sphinx_needs/directives/needextend.py#L175 ) and doesn't resolve dynamic functions first. It should read in links like the need directive.

Heya @arwedus @danwos
Given that you can already use a filter to select multiple need IDs, e.g.
.. needextend:: id.startswith("SYSFCT_")

Rather than dynamic functions, do you think it would be more "consistent" to also allow for filtering link selections, e.g.
.. needextend:: SYSFCT_Foo
    :+implements: id.startswith("REQ_SYS_Foo_")

what do you think?

sure, would be straightforward. In the end, I would expect needextend-directive options to "behave" as identical as possible to need-directive options, but I'll take whatever you can implement quickly :D

I'm not sure, will be tough to check if the given value for +implements is just a string or shall be taken as a filter.
I would provide a predefined dynamic function so that you get:
.. needextend:: SYSFCT_Foo
    :+implements: [[filter(id.startswith("REQ_SYS_Foo_"))]]

@danwos, @chrisjsewell: I'll give this a try, but please before creating a new release, update the documentation on the needextend directive, e.g. add something like this as an example:
.. needextend:: SYSFCT_Foo
   :+implements: [[ filter('id.startswith("REQ_SYS_Foo_"')) ]]

update the documentation

Ah oops thanks for the reminder!

@chrisjsewell: I am having a hard time getting this feature to work.
First try:
.. needextend:: SYSFCT_RepresentDynObj_6BA49B645C114c1bAEAE1DDB69D3F0D4
  :+implements: [[filter('id.startswith("REQ_SYS_MJS_ENVREP_OBJ_")')]]

did not work, throw an error:
Unknown dynamic sphinx-needs function: filter. Found in need: SYSFCT_RepresentDynObj_6BA49B645C114c1bAEAE1DDB69D3F0D4

2nd try:
.. needextend:: SYSFCT_RepresentDynamicObjects_6BA49B645C114c1bAEAE1DDB69D3F0D4
  :+implements: [[get_matching_need_ids("REQ_SYS_MJS_EnvRep_OBJ_")]],
                REQ_SYS_MJS_EnvRep_TD_0010293

This works, but the extra requirement (REQ_SYS_MJS_EnvRep_TD_0010293) did not show up in the generated output.
With a first try with spaces inside [[ ]], I also got this error once, but it wasn't reproducible:
Parsing function string failed for need SYSFCT_RepresentDynObj_6BA49B645C114c1bAEAE1DDB69D3F0D4:  get_matching_need_ids("REQ_SYS_MJS_EnvRep_OBJ_") . unexpected indent (<unknown>, line 1)

What worked:
.. needextend:: SYSFCT_RepresentDynObj_6BA49B645C114c1bAEAE1DDB69D3F0D4
  :+implements: [[get_matching_need_ids("REQ_SYS_MJS_EnvRep_OBJ_")]]

First of all: Cool that this feature is implemented now, thanks!
For me, it is kind of un-intuitive that the filter function is treated differently than a dynamic sphinx-needs function, as it does the same things. Is there a way to treat filter just as other dynamic functions that return needs?
And second, I'd assume that it would be possible to mix direct need references with dynamic function calls for need references, but it seems not to be the case. Is this intentional? Could you add this feature, too?


[Bug]: GPTListIndex Object incorrectly creates a SummaryIndex Object

Bug Description
I created an object of GPTListIndex, bur when using it it says it is a SummaryIndex Object and I can't use  the query attribute
Code:
index = GPTListIndex.from_documents(mock_document, service_context=service_context)

I call the query:
response = index.query(example_query, mode="default", service_context=service_context)

and I get this error:
'SummaryIndex' object has no attribute 'query'
Version
0.8.65
Steps to Reproduce
Import Llama Index GPTListIndex
from llama_index import  GPTListIndex, SimpleDirectoryReader
I am using MockLLMPredicto  and MOck Ebed as well for th efull excercise, as I create a ServiceContext. You cna create it as well. Here are all the lines of code:
llm_model_name = "gpt-3.5-turbo"
llm=OpenAI(temperature=0, model_name=llm_model_name))
    llm_predictor_mock = MockLLMPredictor(max_tokens=256, llm=llm_model_name)
    embed_model_mock = MockEmbedding(embed_dim=1536)

service_context = ServiceContext.from_defaults(llm_predictor=llm_predictor_mock, embed_model=embed_model_mock)

Load a Document, create the index and call the index with a query (any query string will do)
index = GPTListIndex.from_documents(mock_document, service_context=service_context)

query_str = """ WHat's the end of life of Lenovo ABC"""

print(index)
response = index.query(example_query, mode="default", service_context=service_context)
print(response)

Relevant Logs/Tracbacks
WARNING:py.warnings:/home/src/aithentic_optimus/transformers/semantic_list_and_cost.py:71: FutureWarning: The default value of regex will change from True to False in a future version. In addition, single character regular expressions will *not* be treated as literal strings when regex=True.

  ' and characteristics ' + df['BPM_ProductDescriptionUpdate'].str.replace('|', ' ')

Total Tokens: 1212

EMbedding Cost in USD: 0.000485

<llama_index.indices.list.base.SummaryIndex object at 0x7f5482fc4100>

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

AttributeError                            Traceback (most recent call last)

/home/src/aithentic_optimus/transformers/semantic_list_and_cost.py in transform(data, *args, **kwargs)

    144      """

    145     print(index)

--> 146     response = index.query(example_query, mode="default", service_context=service_context)

    147     print(response)

    148

AttributeError: 'SummaryIndex' object has no attribute 'query'

@Jgerardopine GPTListIndex (a very old name) is the same as SummaryIndex, just a rename
You can query with the following code
query_engine = index.as_query_engine()
response = query_engine.query("my query")

I suggest refreshing yourself with our docs as well 👍🏻
https://docs.llamaindex.ai/en/stable/getting_started/starter_example.html

It worked. But now I have a very different problem.
I am using MockLLMPRedictor to simulate and check token usage (and calculate total cost). But now MockLLMPredictor does not have the attribute last_token_usage:
The documentation in the webpage says it should exist, but in my code is not bieng called.
https://gpt-index.readthedocs.io/en/v0.6.8/reference/service_context/llm_predictor.html

Use MockLLM now

from llama_index.llms import MockLLM


When building on windows, can't point cmake to LIBRARY_PREFIX

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

By now we can only point cmake with the -T cuda= option to the pkgs folder, but not to LIBRARY_PREFIX because the directory structure is not preserved. This PR additionally hard-links the original directory structure into LIBRARY_PREFIX so that it can be picked up by cmake.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@conda-forge-admin, please rerender


Ability to create list of products

Ability to create lists of products from the product page or from the history
Ability to give them a name
Ability to export them

Duplicate of #2158


@conda-forge-admin, please add bot automerge

Issue:

Environment (conda list):

$ conda list

Details about  conda and system ( conda info ):

$ conda info

Hi! This is the friendly automated conda-forge-webservice.
I just wanted to let you know that I added bot automerge in conda-forge/cheat-feedstock#15.


[Bug][CICD] Error converting float to uint64

Search before asking

[X] I had searched in the issues and found no similar issues.

What happened
I tried to add a Github scope including CI/CD to pull data from GitHub Actions jobs. During the run, step 5 produces the error shown.
time="2024-03-13 16:06:55" level=info msg=" [pipeline service] [pipeline #1368] [task #51229] start executing task: 51229"
time="2024-03-13 16:06:55" level=info msg=" [pipeline service] [pipeline #1368] [task #51229] start plugin"
time="2024-03-13 16:06:55" level=info msg=" [pipeline service] [pipeline #1368] [task #51229] total step: 3"
time="2024-03-13 16:06:55" level=info msg=" [pipeline service] [pipeline #1368] [task #51229] executing subtask generateDeployments"
time="2024-03-13 16:06:55" level=info msg=" [pipeline service] [pipeline #1368] [task #51229] [generateDeployments] finished records: 1"
time="2024-03-13 16:06:55" level=error msg=" [pipeline service] [pipeline #1368] [task #51229] subtask generateDeployments ended unexpectedly\n\tWraps: (2) error fetching rows (500)\n\tWraps: (3) sql: Scan error on column index 17, name \"duration_sec\": converting driver.Value type float64 (\"2.592547e+06\") to a uint64: invalid syntax (500)\n\tWraps: (4) sql: Scan error on column index 17, name \"duration_sec\"\n\tWraps: (5) converting driver.Value type float64 (\"2.592547e+06\") to a uint64: invalid syntax\n\tError types: (1) *hintdetail.withDetail (2) *hintdetail.withDetail (3) *hintdetail.withDetail (4) *fmt.wrapError (5) *errors.errorString"

What do you expect to happen
It should run without errors
How to reproduce
Create GitHub data scope including CI/DI and run
Anything else
Appears to be related to this issue and its fix. Column type was changed from uint64 to float and this error indicates something still expects uint64.
Version
v20.0.0-beta10
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

@timrog  It's fixed in release v0.21.  Just have a try.