Feature request: click handler

Our use case calls for a notification item that can be clicked on and then would call a function in our code (to show more details about the item using ngDialog). I tried adding ng-click handlers in the template, but the code never got called. It would be great if we could pass in some arguments in some way of course!

+1 for this
However, I actually just did a test by using angular-bind-html-compile and customized the default template with this directive and it worked.

+1 on this. Would definitely be very helpful.

This is already possible without any modification, but it requires a new scope to be added to the notification. I made a plunk to demonstrate how this can be accomplished; the scope can either be created as in the example, or using $rootScope if necessary.

Added by 7e2fc4ffda24f5150700bbd7a88fa1846631d9f2


Pass header parameters to the handler functions as parameters

Changes proposed in this pull request:
Currently, header parameters are not passed to the handler functions as parameters.
This PR allows connexion to pass header parameters to the handler functions as parameters without breaking existing behaviour.

Coverage decreased (-0.02%) to 96.268% when pulling de03cf3649c8140dc2c1ba2a76c521d40286ad0c on mistio:add-headers-as-params into bed4b95205205861bd6014282d0f61d50231d3ac on zalando:master.

Hi @v-pap. If you're still willing to get this merged, could you:

Add a test for the behavior
Rebase on and target this PR to the v2 branch


curl http://<IP_ADDRESS>:8000/logs , always gives "curl: (52) Empty reply from server"

I am trying to use Logspout API, to fetch logs from my different containers.
Following are the steps I did:
Pulled logspout using command: docker pull gliderlabs/logspout:latest
Created two conatiners :
docker run -d ubuntu:14.04 /bin/sh -c "while true; do echo Gautam; sleep 100; done"
docker run -d ubuntu:14.04 /bin/sh -c "while true; do echo Hemant; sleep 100; done"

Later I ran :
$ docker run -d --name="logspout" \    --volume=/var/run/docker.sock:/tmp/docker.sock \

    --publish=<IP_ADDRESS>:8000:80 \

    gliderlabs/logspout

After this when I run $ curl http://<IP_ADDRESS>:8000/logs
I always get "curl: (52) Empty reply from server"
I am not sure why I am getting this message, am I doing some wrong step or missing some settings?
Can someone look into this.
My “docker version” is
[11:24]
Client:
Version:      1.9.1
API version:  1.21
Go version:   go1.4.3
Git commit:   a34a1d5
Built:        Fri Nov 20 17:56:04 UTC 2015
OS/Arch:      darwin/amd64

--publish=<IP_ADDRESS>:8000:80 \ should be --publish=<IP_ADDRESS>:8000:8000 \.
currently you're mapping your host's port 8000 to port 80 in the container, but there's nothing listening on port 80 in the container.

argh, in lieu of yet another edit:
you're pulling :latest, which is v2. This version indeed exposes port 8000. If you pull :master, it will expose port 80 and work as per documentation. (so you're doing everything correctly, just not on the corresponding version of the image).
FWIW, i just upgraded to v3-master, and it's great.

@ebr  as per your reply, I changed the command and I am now able to see the logs.
But, I am not able to see previous logs. Why is it so?

logspout will not aggregate your logs - it simply collects them from running containers and forwards them in real time to an adapter (most commonly, syslog, and in this case - httpstream). to see older logs, your options include the good old docker logs, or a log aggregation solution, e.g. Logstash (as part of the ELK Stack), Loggly, Sumologic, Logentries, and many others.


Colon Bug

# Edit some file with vim
$ vim /path/to/somefile

Enter edit mode (i) and type colon (:)

Escape (exit edit mode) doesn't work

hyper-git-3.1.0.canary.4.r63.g4a3132b8-1

Duplicate of #3929


Set default author using the authentication token

Check if the operation author is the same than the user
eslint windows compatibility
add a dev build

The file yarn.lock should be updated when there is change on package.json dependencies.


nuxt-content mermaid render problems

Description:

mermaid blocks in markdown are now rendered
but the spacing is off, the diagram is vertically squashed

ToDo:

fix rendering issue
rendering is dependent on playwright, which is locally installed

needs to be installed in docker env also

I seem to have found the problem here: Somewhere during conversion, all  hyphens get dropped, so some attributes do literally nothing because they have the wrong names, see attached images (top is hyphenless attributes, bottom is the fixed equivalent that looks as intended)


OpenCV transforms with tests

Have been training on ImageNet and CIFAR with these, faster and more transparent than PIL, but such a pain to install. A few differences with PIL-based:

only np.ndarray on input-output
no torch dependency
bicubic interpolation by default everywhere

@fmassa @soumith was there a decision on this, did we want cv2 as a dependency?

@alykhantejani an option would be to try to import it and fill back to PIL, as for example done in chainercv https://github.com/chainer/chainercv/blob/master/chainercv/transforms/image/resize.py

making this explicit such as CVResize, CVNormalize etc. is so much better.
Importing and falling back is a bad idea because the resize and other functions are not bitwise compatible between both libraries (CV and PIL). So the same user script might converge to different accuracies depending on what is installed on the system (which is not good).

We already have two possible backends for vision (PIL and accimage), and I'm not sure if both are bitwise compatible either.
I'd rather avoid using OpenCV for loading images (or at least would convert them to BGR), but we could eventually add another backend support in vision, and make use of those backends in the transforms. With a proper refactoring (which is going on in https://github.com/pytorch/vision/pull/240), I think it can have minimal code duplication.
What do you think?

@fmassa @alykhantejani Any updates on integration of opencv as a new backend ? I could work on it if you think it will be integrated one day. For me PIL is rather restrictive when working with multiband images other than uint8. What do you think ?

@vfdev-5 I also want OpenCV as a backend, but I'm not sure how to add that without duplicating functional.py and transforms.py, one for PIL and one for OpenCV.  Here is a snippet from my current workaround:
def input_target_transform(image, target, height, width, thickness):
    image = cv2.resize(image, (width, height), interpolation=cv2.INTER_AREA)
    target = cv2.resize(target, (width, height), interpolation=cv2.INTER_AREA)
    # degrees, translations, scale range, shear range, image size
    angle, translate, scale, shear = RandomAffine.get_params(
        (-15, 15), (0.03125, 0.03125), (1. / 1.1, 1.1), (0., 0.),
        (width, height))

    center = (width * 0.5 + 0.5, height * 0.5 + 0.5)
    matrix = F._get_inverse_affine_matrix(
        center, angle, translate, scale, shear)
    M = np.array(matrix).reshape(2, 3)

    image = cv2.warpAffine(
        image, M, (width, height),
        flags=cv2.WARP_INVERSE_MAP | cv2.INTER_LINEAR)
    target = cv2.warpAffine(
        target, M, (width, height),
        flags=cv2.WARP_INVERSE_MAP | cv2.INTER_LINEAR)

Any ideas on how to integrate OpenCV as a proper backend?  I'd be willing to help out.

Hi,
So, one of the things I don't like that much about OpenCV compared to PIL or accimage is that OpenCV returns images in a different format than PIL (BGR instead of RGB), and we'd like for consistency to have all our operations return 0-1 RGB images.
We could patch in some sense ToTensor to take that into account (using the image_backend option), but given that OpenCV returns numpy arrays, I'm not sure how robust such solution would be, given that we already handle numpy arrays in ToTensor, and the lingua-franca that we use for the input/output of most of our transforms is PIL images.
I'm afraid having to backends with different data representations might lead to hard to spot bugs in the user code if we don't do things carefully.
I'm open to suggestions.

@fmassa a workaround for this could be wrapping either PIL images and numpy images in a data structure so that the color space can be tracked down. An alternative could also be to add the color space conversion from BGR to RGB at loading time when the opencv backend is used.

@edgarriba yes. that's true.
This would involve a few more changes though, as we don't currently provide an imread functionality in torchvision, and we let the users use whatever they want to open the images. But that could be added.
So this means we would maintain our wrapper class Image, which would hold the data of the image and we could have our backends perform operations in there directly.
This might work, but I'm unsure if there are downsides I'm not seeing now that could pop up down the road.

@fmassa, as you said, as torchvision does not provide imread functionality, the problem with RGB or BGR can be passed on the user side. In torchvision we can just specify the working format as RGB.
Today, transforms.py contains the most of transformations that works on PIL input, some on Tensors.
If we integrate OpenCV as backend, then all classes transforming images, like Resize should work on any image input: np.ndarray or PIL.Image.Image (accimage.Image)?
We can, probably, create a new file functional_cv2.py with equivalent methods. Or maybe, we can have a single entry with functional.py and backend dependent internal modules functional_pil.py, functional_cv2.py, functional_tensor.py?
What is the purpose of _image_backend?

The purpose of image_backend is to support accimage, which is a faster alternative to PIL but with limited functionality.
I just think that letting to the user the responsability to chose between RGB-BGR might be error-prone.
And we would definitely want to keep the images that we pass in RGB format for compatibility with our modelzoo.
I don't think that np.ndarray (output of opencv) has enough information to represent the images without ambiguity, so as @edgarriba mentioned we could extend it with a colorspace field to at least know if we are in RGB or HSV spaces, for example.

@fmassa In my case, I want to combine existing functions from transforms.py like Resize with functions like distanceTransform from OpenCV.  Thus, if the image is wrapped in an Image class, I'd have to unwrap it anyway, which is what I'm trying to avoid by using an OpenCV backend.
Would it be possible to make functional.py completely agnostic of the backend?  If so, we could handle all the parameters, checks, and equations there, and then implement two versions of transforms.py, one for OpenCV and one for PIL (like @vfdev-5 suggested).  There would be minimal code duplication, because parameters would be handled and the necessary transformations generated in functional.py, and then the actual transformation applied in the transforms.py corresponding to PIL or OpenCV.
Finally, for OpenCV, we could have a cv2ToTensor that converts the numpy array to RGB before converting it to a Tensor to ensure consistency.

@hjweide that sounds reasonable, and might be the way to go.
But the fact that you have a dedicated function for converting from cv2 to Tensor makes your code not agnostic wrt the backend. For example, what if the user by mistake uses ToTensor? This will be a subtle bug, because there will be no error messages, just the color channels will be inverted.
Also, if you want to use opencv functions for transforms, a simple way of doing it is to have a LambdaTransform that converts to numpy, calls the opencv function and back.
Also, one other reason why I am a bit worried about supporting OpenCV torchvision by default is that it doesnt' play well with multiprocessing in fork mode, so only works for Python 3+, see https://github.com/pytorch/pytorch/issues/1838 and https://github.com/opencv/opencv/issues/5150

@fmassa I would go for the Image representation model. However, that's true we should find a stable workaround for the multprocessing issue. /cc @alalek @mshabunin any insights ?

I do not quite understand why Image representation model can save from RGB/BGR error. It still remains on user responsibilty to properly define an instanse of new Image specifying proper colorspace etc if data comes from "somewhere". Similar like F.to_pil_image accepts any np.ndarray (yes, we can hint a mode). Anyway, colorspace problem is relevant to color augmentations and for geometric ones I think it is OK for any colorspace.

@fmassa I did not intend for the user code to be agnostic of the backend, only functional.py.  The user would be responsible for composing [cv2Resize, ..., cv2ToTensor]if they choose to use the OpenCV backend.  Thanks for the LambdaTransform suggestion.
I understand your concern over opencv/opencv#5150.  @edgarriba It's possible to avoid that issue by building OpenCV with "-D BUILD_TBB=ON -D WITH_TBB=ON", but requiring users to build OpenCV from source is probably not an ideal solution.
Perhaps the right thing to do is to develop the OpenCV backend as a small, separate project.  It will be clear that it's not officially supported and so the user is responsible for correctly building OpenCV and rearranging the color channels before using the modelzoo.  That's effectively what I'm doing at the moment.

@hjweide yeah, I'd rather not to duplicate the number of our transforms to support opencv, ideally they should have the same interface and be easily exchangeable, but as discussed, this might bring a few complications.

Apologies for re-opening this thread, but I think OpenCV as a dependency is a bad idea. TBH I would rather have the user deal with the burden of using OpenCV and making sure it is converted to the right format as ndarray and then pass that to Transforms  etc.
Right now, OpenCV is in a bit of a bad state in terms of CUDA support and pytorch and torchision are large enough projects to deal with already without the additional dependency.
A better idea is perhaps to have a note about OpenCV in the README?

FYI, there's a proposed project in the GSoC at OpenCV to implement a data augmentation module.
https://github.com/opencv/opencv/wiki/GSoC_2019#idea-computer-vision-data-augmentation-module
Besides, from torchgeometry there's also a plan to cover part of the current torchvision transforms by using pure pytorch operators to avoid using any image backend.

@varunagrawal there is albumentations which provides transforms using OpenCV as backend similarly to torchvision and which is intended to work and works with PyTorch.

Any update?

Closing this PR as it is over 5 years old. Since then TorchVision offers GPU accelerated Torch-based transforms, so adding another backend is not currently in our plans.

Closing this PR as it is over 5 years old. Since then TorchVision offers GPU accelerated Torch-based transforms, so adding another backend is not currently in our plans.

You mean this?

Pretty much yes. Since v0.8 TorchVision offers 2 backends. PIL and Tensor. The latter offers GPU acceleration, JIT-scriptability and other goodies.


Error: Cannot GET /

After running gulp patternlab:version, I am using Pattern Lab Node- Gulp Edition v3.0.0-alpha.5 on Mac as-is right out of the box without any modifications.
Expected Behavior
Run gulp patternlab:serve

Actual Behavior
Run gulp patternlab:serve

Steps to Reproduce

Run gulp patternlab:serve
Browser launches screenshot from "Actual Behavior"
Refresh browser tab
Browser shows screenshot from "Expected Behavior"

I think I've resolved this with the latest (unreleased) alpha.

I apologize for such a late response to this thread. I just ran version 3.0.0-alpha.8 and gulp patternlab:serve did launch the "expected" behavior identified above.

Apologies, I just noticed this. I had switched over to the webpack version of patternlab since I was able to get it up and running in about 15 min.
I did an https clone of node-gulp then ran npm install && npm run bootstrap. Here's as far as it went:
0 info it worked if it ends with ok
1 verbose cli [ '/usr/local/lib/node_modules/node/bin/node',
1 verbose cli   '/usr/local/bin/npm',
1 verbose cli   'run',
1 verbose cli   'patternlab:serve' ]
2 info using<EMAIL_ADDRESS>3 info using<EMAIL_ADDRESS>4 verbose stack Error: missing script: patternlab:serve
4 verbose stack     at run (/usr/local/lib/node_modules/npm/lib/run-script.js:151:19)
4 verbose stack     at /usr/local/lib/node_modules/npm/lib/run-script.js:61:5
4 verbose stack     at /usr/local/lib/node_modules/npm/node_modules/read-package-json/read-json.js:115:5
4 verbose stack     at /usr/local/lib/node_modules/npm/node_modules/read-package-json/read-json.js:418:5
4 verbose stack     at checkBinReferences_ (/usr/local/lib/node_modules/npm/node_modules/read-package-json/read-json.js:373:45)
4 verbose stack     at final (/usr/local/lib/node_modules/npm/node_modules/read-package-json/read-json.js:416:3)
4 verbose stack     at then (/usr/local/lib/node_modules/npm/node_modules/read-package-json/read-json.js:160:5)
4 verbose stack     at ReadFileContext.<anonymous> (/usr/local/lib/node_modules/npm/node_modules/read-package-json/read-json.js:349:18)
4 verbose stack     at ReadFileContext.callback (/usr/local/lib/node_modules/npm/node_modules/graceful-fs/graceful-fs.js:78:16)
4 verbose stack     at FSReqWrap.readFileAfterOpen [as oncomplete] (fs.js:420:13)
5 verbose cwd /Users/ajohanse/Sites/patternlab
6 verbose Darwin 16.7.0
7 verbose argv "/usr/local/lib/node_modules/node/bin/node" "/usr/local/bin/npm" "run" "patternlab:serve"
8 verbose node v8.3.0
9 verbose npm  v5.8.0
10 error missing script: patternlab:serve
11 verbose exit [ 1, true ]

Hi same Problem here.
I run:
git clone https://github.com/Comcast/patternlab-edition-node-webpack.git npm install //no errors npm run patternlab:build //no errors npm run patternlab:serve //no errors
New Window was open with: cannot GET /error
Solution:

I stopped and close my Docker

killall node

again >> npm run patternlab:serve

Everything works fine ;-)


Unable to delete Split DNS Nameserver entry if more than 1 are present

Trying to delete the Split DNS entry split.example.com when there are more than 1 Split DNS entry present. The following error message is present in the log:
Error: Expected YAML collection at split. Remaining path: split.example.com
    at YAMLMap.deleteIn (/app/node_modules/.pnpm/yaml@2.4.5/node_modules/yaml/dist/nodes/Collection.js:89:19)
    at YAMLMap.deleteIn (/app/node_modules/.pnpm/yaml@2.4.5/node_modules/yaml/dist/nodes/Collection.js:87:25)
    at YAMLMap.deleteIn (/app/node_modules/.pnpm/yaml@2.4.5/node_modules/yaml/dist/nodes/Collection.js:87:25)
    at Document.deleteIn (/app/node_modules/.pnpm/yaml@2.4.5/node_modules/yaml/dist/doc/Document.js:175:29)
    at patchConfig (file:///app/build/server/index.js?t=1728630686000:709:18)
    at action$2 (file:///app/build/server/index.js?t=1728630686000:6768:9)
    at processTicksAndRejections (node:internal/process/task_queues:95:5)
    at Object.callRouteAction (/app/node_modules/.pnpm/@remix-run+server-runtime@2.10.2_typescript@5.5.3/node_modules/@remix-run/server-runtime/dist/data.js:37:16)
    at /app/node_modules/.pnpm/@remix-run+router@1.17.1/node_modules/@remix-run/router/router.ts:4817:21
    at callLoaderOrAction (/app/node_modules/.pnpm/@remix-run+router@1.17.1/node_modules/@remix-run/router/router.ts:4879:16)

A similar error appears when trying to remove an ip for a split dns section

Able to reproduce, will investigate ASAP.

Fixed in 0.3.4


RadzenTabs style issues when first tab not visible

Describe the bug
RadzenTabs have a weird styling when the first tab is not visible. The tab next to the one selected "thinks" it is mouse hovered
To Reproduce
Steps to reproduce the behavior:
The following code reproduces the issue. I made a Radzen switch to make the first tab visible or invisible so you can clearly see the issue
@page "/KobRadzenBug"

<RadzenTabs @ref=@tabs>
    <Tabs>
        <RadzenTabsItem Text="Tab 1" Visible="@isT1Visible">

Tab 1 Content

        </RadzenTabsItem>
        <RadzenTabsItem Text="Tab 2">

Tab 2 Content

        </RadzenTabsItem>
        <RadzenTabsItem Text="Tab 3" Selected>

Tab 3 Content

        </RadzenTabsItem>
        <RadzenTabsItem Text="Tab 4">

Tab 4 Content

        </RadzenTabsItem>
        <RadzenTabsItem Text="Tab 5">

Tab 5 Content

        </RadzenTabsItem>
    </Tabs>
</RadzenTabs>

<RadzenSwitch @bind-Value=@isT1Visible />

@code {
    private bool isT1Visible = false;

    private RadzenTabs tabs = new();

    protected override void OnInitialized()
    {
        base.OnInitialized();
    }
}

I also created a repo to store this example as a demo. Please check here
Expected behavior
When the first tab is not visible, the CSS (styling) should behave as normal
Screenshots

Desktop (please complete the following information):

OS: Windows11
Browser Chrome
Version  129.0.6668.60 (Official Build) (64-bit)

Additional context
N/A

Thanks :) It is working now. That was a flash fix. Thanks for your quick intervention. I am impressed.


[FEATURE] Demo site

There should be a live, public example of ModelDB for people to play with. Visitors should get to browse both a Jupyter notebook with example models to run and ModelDB's Node.js frontend to see reports about those models.
Architecture:

Single ModelDB server

Mongo server
ModelDB Java backend
ModelDB Node.js frontend

Multiple ephemeral Jupyter notebooks with ModelDB python library

configurable-http-proxy to vend out notebooks
tmpnb to spawn new notebooks and cull idle and old ones

This will also require the creation of one or more .ipynb files that can be run by users to create reports in ModelDB.

Closed, and I updated the gh-pages site to include links to the demo sites.


Update Cocoapods to 1.2.3

Please update the Podspec to include the new 1.2.3 so the library can be used on xcode 9.3

Oh, my bad. I can't release 1.2.3 to CocoaPods without doing a force push, so I will release 1.2.4.

Can you please update pod for Xcode 9.3?

Released as 1.2.4 and pushed to CocoaPods trunk. Sorry for 1.2.3 not being there. 😔
Please open a new issue if there are any further problems.


Restore a very narrow function argument conversion for -swift-version 4.

Allow functions with type (()) -> T to be passed in places where we
expect () -> T, but only for -swift-version 4 (for -swift-version 3
this already works due to other horrible things in CSSimplify.cpp).
We need to look at how we can help migrate these cases to
-swift-version 5, but in the meantime, but that is something we can
consider separately.

@swift-ci Please smoke test

@swift-ci Please test source compatibility

@swift-ci Please smoke test

So when Swift 5 comes out we're gonna have to file regression bug again?

No, the code will need to change for -swift-version 5.
It was a bug that Swift ever allowed lack of a parameter to be conflated with having a Void (aka ()) parameter. This bug arose because of a representation problem in the compiler. Some effort has gone into fixing that problem, resulting in code that previously compiled now failing to compile. The tuple-splatting behavior was another symptom of this same representation problem. In fact you could consider this another instance of the splatting behavior - the zero-element tuple to no-argument case.
For the particular code for which this problem was reported, I would think overloading log (and logIfNeeded) to return a no-parameter function would be sufficient to allow the code to compile in both -swift-version 4 mode and -swift-version 5 mode.
This works in a stand-alone test case, and has the benefit that you can remove the as ((()) -> Void)? coercions:
func logIfNeeded<T>() -> ((T) -> Void)? { fatalError() }
func logIfNeeded() -> (() -> Void)? { fatalError() }

func log<T>() -> ((T) -> Void)? { return logIfNeeded() }
func log() -> (() -> Void)? { return logIfNeeded() }

func on(no: (()->())? = nil, yes: ((())->())? = nil) {}

on(no: log(), yes: log())

No, the code will need to change for -swift-version 5.

Okay, but right now there is no way to make that code compile, as I pointed out in the Jira.

For the particular code for which this problem was reported, I would think overloading log (and logIfNeeded) to return a no-parameter function would be sufficient to allow the code to compile in both -swift-version 4 mode and -swift-version 5 mode.

I'll give that a try, thanks. In my opinion, however, the Jira shouldn't be closed because, whatever the Swift decides about whether that code should compile or not, error: generic parameter 'T' could not be inferred is not a useful diagnostic for users to understand that the code is hitting a limitation of the language and what the workaround should be.
Also, I want to point out that [the reason why ReactiveCocoa (a superset of ReactiveSwift) was disabled from the source-compatibility suite is still not fixed, so because of that, not only did it not catch this regression, but it's potentially still not being able to catch future ones.

@rudkx that workaround doesn't work:

IMO, if this could work:

Then we wouldn't be having arguments about () and Void. The problem is that T can't be inferred by itself, and there's no way to tell the compiler what type to use for this case.

@rudkx that workaround doesn't work:

Does the stand-alone test case that I pasted in compile for you?
What compiler are you using when you build and hit the error about the conflicting definition?

Your example does compile, but not the full example as shown in my screenshot. I welcome you to clone the repo as explained in SR-6796 and you can see by yourself.

Apparently we do not allow overloading nested functions, so I can only offer this slightly cumbersome work-around:
extension Signal {
  struct LogContext {
    let logger: EventLogger
    let events: Set<LoggingEvent.Signal>
    let identifier: String
    let fileName: String
    let functionName: String
    let lineNumber: Int

    func log<T>(_ event: LoggingEvent.Signal) -> ((T) -> Void)? {
      return event.logIfNeeded(events: events) { event in
	self.logger(self.identifier, event, self.fileName, self.functionName, self.lineNumber)
      }
    }

    func log(_ event: LoggingEvent.Signal) -> (() -> Void)? {
      return event.logIfNeeded(events: events) { event in
	self.logger(self.identifier, event, self.fileName, self.functionName, self.lineNumber)
      }
    }
  }

...
  public func logEvents(identifier: String = "", events: Set<LoggingEvent.Signal> = LoggingEvent.Signal.allEvents, fileName: String = #file, functionName: String = #function, lineNumber: Int = #line, logger: @escaping EventLogger = defaultEventLog) -> Signal<Value, Error> {
    let logContext =
    LogContext(logger: logger,
      events: events,
      identifier: identifier,
      fileName: fileName,
      functionName: functionName,
      lineNumber: lineNumber)

    return self.on(
      failed: logContext.log(.failed),
      completed: logContext.log(.completed),
      interrupted: logContext.log(.interrupted),
      terminated: logContext.log(.terminated),
      disposed: logContext.log(.disposed),
      value: logContext.log(.value)
    )
  }

IMO, if this could work:

I would love to make that work, but it's not possible today without completely fixing the function type representation. Changes make that work will also almost certainly break other code as well.

I'm trying your workaround, it compiles now, but tests aren't passing so I'm investigating.
In the mean time I want to say that this still smells like a huge regression. Why would we need an overload to a generic function for T = Void? Void (at least from the user's standpoint) is a type like any other, so this is as arbitrary as if the type checker wasn't able to resolve the usage of a generic function for String and we needed an overload for that... what a huge step backwards :(

But hey at least we're getting Python interpretability, so yay? 🤦♂

Void is indeed a type: it's a typealias for the empty tuple. A function type that takes Void is written (Void) -> Int, or without sugar (()) -> Int, and is called by passing a value of type Void to the function: fn(()). A function that takes no arguments is not a function that takes Void (one argument).
Past compiler releases mistakenly treated () -> Int and (()) -> Int as interchangeable in some cases, but "from the user's standpoint" it never actually made sense. But I'll acknowledge a user probably wouldn't bother thinking about it if the compiler doesn't complain, hence the use in ReactiveSwift.

@jrose-apple if I'm reading that correctly, sounds like you agree with me about the fact that we shouldn't require an overload for Void, right?
Can we count on a fix for this for Swift 4.1?

No, I'm saying the opposite. You don't need an overload for Void; you need an overload for "no arguments".

I see what you're saying. Why can't the compiler correctly convert between (Void) and () -> Void though?
I'm looking into this test failure with the workaround that @rudkx proposed, and there's something really weird going on. It compiles, but the test (for this operator) failure seems to indicate there's a weird miscomputation going on. Still digging...

I'm not sure what you mean by "correctly"; they are different types and they should be different types. Of course we (inclusive "we", the Swift community, possibly via the Evolution process) could formally introduce a conversion, but conversions between unrelated types are the kind of thing that makes type checking slow, especially when the conversion goes both ways. We'd also either have to put this into dynamic casting as well, or have different static and dynamic behavior.
func fn<T>(_ arg: T) -> Void {
  print(arg)
}

let boxedFn: Any = fn as (Void) -> Void
print(boxedFn is (Void) -> Void) // true
print(boxedFn is () -> Void) // false

I mean, you tell me. This is what I had to do to get @rudkx's workaround working:
private extension LoggingEventProtocol {
	// See https://bugs.swift.org/browse/SR-6796.
	func logIfNeeded(events: Set<Self>, logger: @escaping (String) -> Void) -> (() -> Void)? {
		return (self.logIfNeeded(events: events, logger: logger) as ((()) -> Void)?)
			.map { closure in
				{ closure(()) }
			}
	}

	func logIfNeeded<T>(events: Set<Self>, logger: @escaping (String) -> Void) -> ((T) -> Void)? {
		guard events.contains(self) else {
			return nil
		}

		return { value in
			if value is Void {
				logger("\(self.rawValue)")
			} else {
				logger("\(self.rawValue) \(value)")
			}
		}
	}
}

Can you really tell me with a straight face that this regression is actually a feature for Swift 5.0? We could spend hours discussing the philosophy behind Void and (), but at the end of the day this change means we have to explicitly do this:
(closure as ((()) -> Void)?)
    .map { closure in
        { closure(()) }
    }

Which is an absolute nightmare, and with the current state of diagnostics in Swift I guarantee you most people won't be able to understand the errors behind this.

FYI, I have been busy this afternoon but this morning I wrote up a pitch for fixing this that I will post to the Evolution forum and link here, hopefully tonight.

👌

IMO the right answer was always "pass () when you call a function whose generic parameter is Void". It's a mistake that people got used to writing f() when the language wanted them to write f(()).
(Is the latter better? No, I'm not going to claim that. But it's consistent, and would have meant a faster compiler with fewer bugs and potentially better diagnostics.)

FYI, I have been busy this afternoon but this morning I wrote up a pitch for fixing this that I will post to the Evolution forum and link here, hopefully tonight.

See: https://forums.swift.org/t/more-consistent-function-types/9765


Policy Definition Group #1271

Overview/Summary
Background
The azure custom control mapping cardinal requirement is the Policy Definition Group which helps to holds collection of Policy Definitions that are logically grouped together based on their purpose, such as to enforce compliance with regulatory requirements or to ensure the security of resources in Azure. By grouping related Policy Definitions together, organizations can more easily manage their policies and ensure that their resources are in compliance with all relevant rules and regulations.
Replace this with a brief description of what this Pull Request fixes, changes, etc.
This PR fixes/adds/changes/removes
Fixes #670
Challenge with current Azure Landing Zone – Bug / Feature missing. (only in Terraform)
Current script do not include Policy Definition Group names reference in the script file .terraform/modules/enterprise_scale/resources.policy_set_definitions.tf
In Azure Policy definition set initiative the following information missing, assume that the policyDefinitionGroups section contain required information in the policy definition json file.

Add Groups

to fix this issue following modification required on terraform file resources.policy_set_definitions.tf
Breaking Changes
Please note that existing definition set the policyDefinitionGroups set to NULL which may give error so need to handle with this statement for_each = [for item in coalesce(each.value.template.properties.policyDefinitionGroups, []) : and tested to avoid errors
Testing Evidence

Please provide any testing evidence to show that your Pull Request works/fixes as described and planned (include screenshots, if appropriate).
As part of this Pull Request I have

[x ] Checked for duplicate Pull Requests
[x] Associated it with relevant issues, for tracking and closure.
[x ] Ensured my code/branch is up-to-date with the latest changes in the main branch
[x ] Performed testing and provided evidence.
[ ] Updated relevant and associated documentation.

/azp run unit


after adding connection string to webapp, the app doesn't work anymore: "An error occurred while starting the application"...

I was following your description, added the connection string under "Application settings - New application settings" with the name "App:SqlDatabase:ConnectionString" and the value the SQL-database instance was showing, but that breaks the application.
[Enter feedback here]

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 57567eca-a79e-5e41-3ab7-dc836ca08182
Version Independent ID: 937e3455-0655-2a1f-9b54-f6b33277a816
Content: Azure Functions deployment slots
Content Source: articles/azure-functions/functions-deployment-slots.md
Service: azure-functions
GitHub Login: @craigshoemaker
Microsoft Alias: cshoe

@fgcgithub  Thank you for the question. We will review and get back to you.

@fgcgithub How are you reading the value? Are you trying to read using Configuration Manager? Then you may add a Connection String from the Azure portal:
From your Function App UI, click Configurations --> Application Settings --> Add connection strings
If used as an Environment Variable, then can add the App setting.
Also, please share the document section, what you are referring to.
However, this is not a documentation update request , You may want to leverage Microsoft QnA channel for application specific issue.

Hallo JayaChatterjee,
thanks for your help, in the meantime my solution is running!
Yours
Franz-Georg
Von: JayaChatterjee-MSFT @.>
Gesendet: Mittwoch, 26. Mai 2021 20:29
An: MicrosoftDocs/azure-docs @.>
Cc: fgcgithub @.>; Mention @.>
Betreff: Re: [MicrosoftDocs/azure-docs] after adding connection string to webapp, the app doesn't work anymore: "An error occurred while starting the application"... (#75851)
@fgcgithubhttps://github.com/fgcgithub How are you reading the value? Are you trying to read using Configuration Manager? Then you may add a Connection String from the Azure portal:
From your Function App UI, click Configurations --> Application Settings --> Add connection strings
If used as an Environment Variable, then can add the App setting.
Also, please share the document section, what you are referring to.
However, this is not a documentation update request , You may want to leverage Microsoft QnA channel for application specific issue.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/azure-docs/issues/75851#issuecomment-849019708, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ACKBCHNYTEW6BWGCAKPNSKLTPU4YNANCNFSM45PTC4TQ.


Update composer.json

Support laravel 7

I looked very superficial, there have been a lot of changes :(


🐛 Issue with Single Table Inheritance (STI) and Type Casting in Child Entities

No duplicates 🥲.

[X] I have searched for a similar issue in our bug tracker and didn't find any solutions.

What happened?
I've encountered an issue regarding Single Table Inheritance (STI) when working with CycleORM. Specifically, the problem arises with type casting in child entities. The ORM seems to be utilizing the parent entity's type cast rules instead of the child's, which is leading to incorrect behavior.
Here's a simplified example to demonstrate the issue:
// Parent entity
#[Entity]
class Asset {
    #[Column(type: 'jsonb', nullable: true, typecast: Data::class)]
    public ?object $data = null;
}

// Child entity
#[Entity]
#[SingleTable(value: 'domain')]
class Domain extends Asset {
    #[Column(type: 'jsonb', nullable: true, typecast: DomainData::class)]
    public ?object $data = null;
}

In this setup, whenever I attempt to fetch a Domain entity from the database, the data property is always being cast using the Data class from the parent Asset entity, ignoring the specified DomainData class in the child entity.
The root of the problem seems to lie in the Cycle\ORM\Service\Implementation\EntityFactory::make method:
blic function make(
    string $role,
    array $data = [],
    int $status = Node::NEW,
    bool $typecast = false
): object {
    // ... snip ...
    $role = $data[LoaderInterface::ROLE_KEY] ?? $role;
    // ... snip ...
    $rRole = $this->resolveRole($role);
    $mapper = $this->mapperProvider->getMapper($rRole);
    $castedData = $typecast ? $mapper->cast($data) : $data;
    // ... snip ...
    $e = $mapper->init($data, $role);
    return $mapper->hydrate($e, $relMap->init($this, $node, $castedData));
}

As seen in the code above, it resolves the role to asset initially, then proceeds to cast the data using this resolved role, which in turn picks up the typecast rules from the Asset class instead of the Domain class.
Moreover, it appears that the ORM schema does not accumulate type cast rules for child entities. Here
I propose a modified approach for the make method to handle STI with type casting correctly, somewhat along the following lines:
public function make(
    string $role,
    array $data = [],
    int $status = Node::NEW,
    bool $typecast = false
): object {
    // 1. Resolve parent role [asset]
    // 2. Find the proper child role [asset_domain] using the discriminator
    // 3. Retrieve the schema with type casters for the child [asset_domain] role
    // 4. Cast data using the proper child [asset_domain] role based on the discriminator
    // 5. Create the child [Domain] object
    // 6. Hydrate the casted data into the child object
}

This modified approach ensures that the correct child role is utilized when casting data, hence respecting the type cast rules defined in the child entities.

Valid usecase.


Fix a misprint in a formula in two_qubit_decompose

Summary
Fix a misprint in a formula in the function _closest_partial_swap in two_qubit_decompose,
according to @levbishop .
Details and comments
The formula is based on equations (B3)-(B5) in https://arxiv.org/abs/1811.12926

Set m = (a+b+c)/3 and write everything as deviations from that mean
a -> eps da + m, b-> eps db + m, c -> eps dc + m
x -> eps dx + m
and then solve for d/dx = 0 up to nth order in eps gives:

Since this change is of order eps^6 it does not affect the tests, and is almost undetectable.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.02%) to 89.299%

Totals

Change from base Build<PHONE_NUMBER>:
0.02%

Covered Lines:
59116

Relevant Lines:
66200

💛  - Coveralls

@Mergifyio backport stable/0.46 stable/1.0


Not able to enable shibboleth access provider (Ping Federator).

Rancher server version - v1.6.24-rc2
Steps to reproduce the problem:
Try to enable shibboleth access provider (Ping Federator).
When trying to test the setup , after providing a valid username and password , user is presented with "Internal Server Error".
Error from rancher-server logs:
time="2018-11-09T21:57:35Z" level=debug msg="HandleSamlAssertion: Got token VXVXGmoMDi8G3XbZY9BMmvZR3Utcu69dtSFJPL5V "
time="2018-11-09T21:57:35Z" level=debug msg="r.URL :/v1-auth/saml/acs, host :<ip>:8080,redirectBackBaseValue: http://<ip>:8080"
time="2018-11-09T21:57:36Z" level=error msg="PostSamlTokenHTML: No token provided in POST request: <nil>"
time="2018-11-09T21:59:06Z" level=info msg="Stack Create Event Received" eventId=ead180fd-21c2-4efb-bb41-23fbbf507618 resourceId=1st5
time="2018-11-09T21:59:06Z" level=info msg="Stack Create Event Done" eventId=ead180fd-21c2-4efb-bb41-23fbbf507618 resourceId=1st5
time="2018-11-09T22:11:10Z" level=debug msg="GetRedirectURL: returning response map[redirectUrl:http://<ip>:8080/v1-auth/saml/login provider:shibboleth]"

Tested with Rancher server version - v1.6.24-rc2
Able to enable hibboleth access provider (Ping Federator) successfully.


pnpm tauri:dev stuck if port 3000 is not available

the terminal log is
Nuxt 3.12.4 with Nitro 2.9.7                                                                                1:13:40 AM
[get-port] Unable to find an available port (tried 3000 on host "<IP_ADDRESS>"). Using alternative port 3001.
ℹ Nuxt Icon server bundle mode is set to local                                                                                1:13:43 AM
  ➜ DevTools: press Shift + Alt + D in the browser (v1.3.9)                                                                                1:13:44 AM

    Warn Waiting for your frontend dev server to start on http://<IP_ADDRESS>:3000/...
ℹ Compiled devtools/settings.mjs in 593.95ms                                                                                1:13:45 AM
ℹ Compiled types/imports.d.ts in 609.31ms                                                                                1:13:45 AM
ℹ Compiled plugins/server.mjs in 2614.53ms                                                                                1:13:48 AM
ℹ Compiled plugins/client.mjs in 2869.9ms                                                                                1:13:48 AM
ℹ Compiled types/plugins.d.ts in 2872.96ms                                                                                1:13:48 AM
✔ Vite client built in 567ms                                                                                1:13:49 AM
✔ Vite server built in 100ms                                                                                1:13:49 AM
✔ Nuxt Nitro server built in 608 ms

I think it's because of the config of tauri.conf.json, it's still waiting on the port 3000:
"build": {
		"beforeBuildCommand": "pnpm generate",
		"frontendDist": "../dist",
		"beforeDevCommand": "pnpm dev",
		"devUrl": "http://localhost:3000"
	},

changing the "devUrl": "http://localhost:3000" to "devUrl": "http://localhost:3001" solved the problem, but is there a dynamic solution for port 3000 not available

I don't know if there is a way to dynamically change port in the tauri config file, currently it expects those two ports available.
Either make sure the ports are free for the app or manually update them.
Anyway if I'll find a method to auto update them I'll make a change


Fix README.md link to documentation page

Link to http://docs.feathers-plus.org/ isn't working.

Woops. Thanks for catching this.


New Writer for RTF or Similar Document Format

Create a new writer to output metadata in a format importable into Word. Commonly used in USGS to create a metadata review document with commentary added.

@dwalt I'm wondering if we just need a simplified version of HTML?
It seems that the problem with the current html export is that word doesn't like all of the nested expandable sections. Word seems to do fine with basic html (try downloading the https://json-schema.org/ or the https://git-scm.com/docs/git-branch pages and importing them into word).
This would be a simpler solution since most of the work has already been done. It should just require modifying the html tags used for output instead of implementing a completely new RTF standard.

@hmaier-fws Agreed, a different stylesheet might be all we need. Would be a good place to start, using the friendly labels that the current HTML writer uses in expanded format to see if this works for users.

I was able to import the html version into Word. Is the issue just with the formatting?

@jwaspin I think we can start with a copy of the CSS and hack it to improve formatting and see where that gets us, such as fonts, line spacing, indentation and map graphic rendering.
I have enclosed an HTML writer import to word, a CSDGM import to Word and a CSDGM text import to Word. The later two were produced using the Metadata Parser, a CSDGM validation tool that also formats XML to HTML and TXT: https://www1.usgs.gov/mp/. The CSDGM HTML from MP has been used extensively in USGS to create review docs and as a presentation format for data releases. I think it would be a good model to work from regarding format. However, indentation is lost in the import to Word. Run MP, loading a CSDGM. and review the Outline format to see what I mean. The txt format retains indentation but obviously is plain vanilla styling. I would point out the section links at the top of the CSDGM Word doc which are very handy for navigation.
I downloaded Metadata Parser but did not come up with a CSS.
htmlCSDGMWordImport.docx
htmlWriterWordImport.docx

@dwalt I looked into this and the CSS does not appear to be the issue, there were some tags that were not standard html tags that word just doesn't know what to do with. So I changed them all to div tags and that seems to resolve the issue. I think there's probably some cleanup that could happen to make it look a little better in certain places, but I wanted to let you take a look before I spent more time tinkering with this one.

@hmaier-fws Yes, I just pushed the output file I produced here:
https://github.com/adiwg/mdTranslator/blob/feature/simple-html-for-word/simple.html

@jwaspin This is definitely an improvement. We lost the section links and the map graphic. but this is a start.

I'm not sure why this was closed. It is not available anywhere for anyone to test it. It is in the drop-down list here: http://<IP_ADDRESS>:3002/ but it doesn't work: "Writer cannot be found".

The writer works now on 3002. It will fail however if there are mdJSON errors. Seems an HTML writer should not care whether or not a record is valid, as the output can be useful to trace content errors. I turned off validation and force valid output settings, but still it failed. I am closing this issue since this was intended as a prototype to spur refined requirements from users.


upgrade to scala.js 1.x

For with the sjs 0.6.x support completely removed.
There have been some breaking changes in 1.x, but just minor ones affect us.
I believe that we can drop support, since the lib is relatively feature complete and there isn't lots of new stuff on the table right now. Thoughts?
Additionally, this PR depends on https://github.com/cornerman/colibri and has to be updated when that goes through.

The naming of either the parameters or the variable is poor. Which should be renamed?

This is great. Thank you so much! It will fix #396
I have merged the PR in colibri!
Can you also update the badge in the readme? It should link to this svg: http://www.scala-js.org/assets/badges/scalajs-1.0.0.svg.

Ah, looks like the further updates on colibri caused some issues. Just a sec.

"a sec" turned into 4 hours, but all the tests succeed on my machine now. Let's just hope they also do on travis.
Anyways, I updated all the tests to use the new scala-js-dom names and I replaced all occurrences of initEvent with the new event constructors.
The only thing that's potentially missing is to create deprecations for the old attribute names.

I have created deprecations for the renamed attributes and tags.
While we are at it, I think the `class` and `for` also need some rework, but that's something for another PR.

I have to admit the idAttr really does not look as good as the id before. But we'll see :)


CreateShipment: Navigate to the Shipment View When Shipment Created

navigate to shipment view when shipment created
update test case 4.3.3

 2.0 CreateShipment: Navigate to the Shipment View When Shipment Created (Bug)


Update PROJ to 7.2.1

It's been out there for a while now (7.1 in ubuntu since 20.10, 7.2 in osgeo4win, etc.). Beyond catching the series of projection updates and bug fixes, it unlocks some epoch functionality that will make QField even more accurate.

I'd suggest skipping 7.2 and going straight for 8.0 -- proj doesn't backport database updates, so you'll be missing out on newer crs definitions if you opt for the older release

@nyalldawson , are datum ensembles on 7.2.1?

@nirvn the api is, but the database wasn't updated to use them till 8.0. You won't see any change without the db update.

O...M...G...


Asset pipeline roadmap

[ ] Visual/UI pipeline.toml editor (it's quite hard to figure out how to edit the file manually right now, see #524 for context)
[ ] #689
[ ] UI to browse compiled assets (in the Ambient IDE)
[ ] #222
[ ] A way to directly upload assets on the website
[ ] #12

Regarding the compiled asset browser - could be #1148 and #1151.


Alternative to http-equiv="refresh"

I disabled automatic reloading and Firefox and it keeps warning me everytime I visit voting.decred.org.

Firefox prevented this page from automatically reloading.

I think <meta http-equiv="refresh" content="300" > is to blame, as opposed to more common javascript-based UI refresh. For long time I have not seen this warning during my normal surfing.
If I'm the only one with this problem please just close this issue.

I'm open to alternatives, although with our current architecture (no js blob running via angular etc) I'm not sure there is another way to refresh the page otherwise.

A small javascript snippet would do I think. Set a timer, make request, update couple elements. The page is rather simple right now, although I'm not aware if bigger things are planned for it.
Thinking about timers, the numbers on the page can only change when new block is mined, is that correct?
If yes, only one update in 5 minutes is necessary.

websocket ntfn that a new block has been received and to refresh?


[Bug]:  UI inconsistently identifying if gravity is on or off for armor stands.

General Info

[ ] I am running a modpack
[ ] I can reproduce this issue consistently in single-player
[X] I can reproduce this issue consistently in multi-player
[X] I have searched for this issue previously and it was either (1) not previously reported, or (2) previously fixed and I am having the same problem.
[ ] I am crashing and can provide my crash report(s)
[X] I am using the latest version of the mod

Loader version
fabric-loader-0.16.9-1.21.3
Minecraft version
1.21.3
Mod version
7.0.1
Java version
Java 21.0.3
Issue Description
The UI is not consistently identifying if an armor stand has gravity enabled.
It correctly displays in the buttons on the left to enable/disable gravity.
However the Y position box is disabled and gives the "Disable gravity to adjust Y position" message if gravity is disabled. and becomes enabled if gravity is enabled. (useless in this state because attempts at raising the armor stand fail immediately because gravity is enabled.) accidentally lowering the armor stand results in it falling through the floor. (I lost one one of the armor stands from the display in the picture when it fell through the floor.)

Additional Information
No response

I likely forgot to invert the check when removing "no" from the gravity


samples: subsys: usb: cdc_acm: add twister test for st boards

add one twister test : sample.usb.cdc-acm-otg-st.
if the second usb connector (the other than ST-link) is connect and the link is ok we must read the string: "Device configured"

@jfischer-no
nothing is wrong with  the test sample.usb.cdc-acm it works well but:
with the test sample.usb.cdc-acm  even if nothing is connect on the user USB the test pass ( because we expected " wait for DTR")
we want to add a new test  that allow us to be sure that the user usb is connect and OK  ( in that case the expected string is "Device configured" and not only " wait for DTR")
it not the same test.

@jfischer-no nothing is wrong with the test sample.usb.cdc-acm it works well but: with the test sample.usb.cdc-acm even if nothing is connect on the user USB the test pass ( because we expected " wait for DTR") we want to add a new test that allow us to be sure that the user usb is connect and OK ( in that case the expected string is "Device configured" and not only " wait for DTR") it not the same test.

This is a sample, not a test. It is fine as it is.

I don't understand
an automatic test does not harm a sample in any way
on the contrary if a new card is added tomorrow this specific functionality will be tested on this sample quickly with twister.
without this test on this future new card this specific functionality may not be seen as non-functional on this sample
this will in fact only strengthen the reliability of this sample

@jfischer-no
samples/subsys/usb/cdc_acm
with nucleo_stm32H753zi for example
You can see all the code we run from the stm32 driver when the console message  “Device configured” is displayed

With
CONFIG_USB_DRIVER_LOG_LEVEL_DBG=y
We can see all the code running under the Driver  ( see after)

One other solution I can propose you is to add one special fixture =”Device configured” in sample.yaml  in that case  all the board not only for ST board can use it in Twister if the fixture is set.
Something like that:
sample.usb.cdc-acm-otg:
depends_on: usb_device
tags: usb
harness: console
harness_config:
fixture: Device_Configured
type: one_line
regex:
- "Device configured"
In that case the coverage for the driver is better for the sample.
usb_cdc_acm_drivers_called.docx

One other solution I can propose you is to add one special fixture =”Device configured” in sample.yaml in that case all the board not only for ST board can use it in Twister if the fixture is set.
Something like that:
sample.usb.cdc-acm-otg: depends_on: usb_device tags: usb harness: console harness_config: fixture: Device_Configured type: one_line regex: - "Device configured"

fixture should probably be fixture: fixture_usb_host or fixture_usb_cdc (console sample), but relying on the logging level of CDC ACM implementation is problematic, there could be status callback in the sample. But because of fixture, only few boards, if any, would "benefit" from this change. I am still not convinced.

But because of fixture, only few boards, if any, would "benefit" from this change. I am still not convinced.

This is the way fixtures work and this is highly valuable for HW in the loop testing.

And again, this is not a test, and not "HW in the loop" testing because there is no real loop.

With this small change in the .yaml

This is not done with changes in sample.yaml. Please read my comment.

And again, this is not a test, and not "HW in the loop" testing because there is no real loop.

If adding 2 lines here helps to significantly increase test coverage, is that such a big issue ?

This is not done with changes in sample.yaml.

This is done this way in several other drivers/subsystems.

Please read my comment.

Sorry, but this isn't clear to me. Anyway, this is up to you I guess but if it can't be done that way, are there any plans to improve USB driver code coverage using USB tests ?

@jfischer-no
Ok I close this PR


Using jsoniter to Marshal

Codecov Report

Merging #13 (a41f5de) into main (8ad662f) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##              main       #13   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           11        11
  Lines          180       175    -5
=========================================
- Hits           180       175    -5

Impacted Files
Coverage Δ

json.go
100.00% <100.00%> (ø)

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 8ad662f...a41f5de. Read the comment docs.


admin的BasicConfig Plugin页面空白

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
当吧插件request开启时，admin的BasicConfig Plugin页面空白无数据。
Expected Behavior
No response
Steps To Reproduce
No response
Environment
ShenYu version(s):master

Debug logs
No response
Anything else?
No response

必须在插件配置上面填 ruleHandlePageType 参数，不填提交就是空白页面


Install argocd

This PR installs argocd operator on hub cluster, and enables ability to create argocd application CR and applicationset CR on hub, and the application could be auto deployed to the target dr cluster by argocd controller, using push mode.

Local test with argocd env
$ drenv start envs/argocd.yaml
2024-06-19 20:51:33,383 INFO    [argocd] Starting environment
2024-06-19 20:51:33,437 INFO    [dr2] Starting minikube cluster
2024-06-19 20:51:33,438 INFO    [dr1] Starting minikube cluster
2024-06-19 20:51:33,440 INFO    [hub] Starting minikube cluster
2024-06-19 20:52:09,191 INFO    [hub] Cluster started in 35.75 seconds
2024-06-19 20:52:09,192 INFO    [hub/0] Running addons/argocd/start
2024-06-19 20:52:30,153 INFO    [dr1] Cluster started in 56.71 seconds
2024-06-19 20:52:50,812 INFO    [dr2] Cluster started in 77.37 seconds
2024-06-19 20:53:37,302 INFO    [hub/0] addons/argocd/start completed in 88.11 seconds
2024-06-19 20:53:37,302 INFO    [hub/0] Running addons/argocd/test
2024-06-19 20:54:11,212 INFO    [hub/0] addons/argocd/test completed in 33.91 seconds
2024-06-19 20:54:11,212 INFO    [argocd] Environment started in 157.83 seconds

Local test with regional-dr env
$ drenv start envs/regional-dr.yaml
2024-06-19 20:55:44,960 INFO    [rdr] Starting environment
2024-06-19 20:55:45,012 INFO    [hub] Starting minikube cluster
2024-06-19 20:55:45,013 INFO    [dr1] Starting minikube cluster
2024-06-19 20:55:45,013 INFO    [dr2] Starting minikube cluster
2024-06-19 20:56:24,470 INFO    [dr1] Cluster started in 39.46 seconds
2024-06-19 20:56:24,471 INFO    [dr1/0] Running addons/rook-operator/start
2024-06-19 20:56:24,473 INFO    [dr1/2] Running addons/csi-addons/start
2024-06-19 20:56:24,473 INFO    [dr1/1] Running addons/ocm-cluster/start
2024-06-19 20:56:55,495 INFO    [dr2] Cluster started in 70.48 seconds
2024-06-19 20:56:55,497 INFO    [dr2/0] Running addons/rook-operator/start
2024-06-19 20:56:55,497 INFO    [dr2/1] Running addons/ocm-cluster/start
2024-06-19 20:56:55,498 INFO    [dr2/2] Running addons/csi-addons/start
2024-06-19 20:57:04,252 INFO    [dr1/2] addons/csi-addons/start completed in 39.78 seconds
2024-06-19 20:57:04,252 INFO    [dr1/2] Running addons/olm/start
2024-06-19 20:57:10,428 INFO    [dr1/0] addons/rook-operator/start completed in 45.96 seconds
2024-06-19 20:57:10,428 INFO    [dr1/0] Running addons/rook-cluster/start
2024-06-19 20:57:19,294 INFO    [hub] Cluster started in 94.28 seconds
2024-06-19 20:57:19,295 INFO    [hub/0] Running addons/ocm-hub/start
2024-06-19 20:57:19,297 INFO    [hub/1] Running addons/submariner/start
2024-06-19 20:57:23,914 INFO    [dr2/2] addons/csi-addons/start completed in 28.42 seconds
2024-06-19 20:57:23,914 INFO    [dr2/2] Running addons/olm/start
2024-06-19 20:57:36,927 INFO    [dr1/2] addons/olm/start completed in 32.68 seconds
2024-06-19 20:57:36,928 INFO    [dr1/2] Running addons/minio/start
2024-06-19 20:57:51,263 INFO    [dr2/2] addons/olm/start completed in 27.35 seconds
2024-06-19 20:57:51,263 INFO    [dr2/2] Running addons/minio/start
2024-06-19 20:57:55,136 INFO    [dr2/0] addons/rook-operator/start completed in 59.64 seconds
2024-06-19 20:57:55,137 INFO    [dr2/0] Running addons/rook-cluster/start
2024-06-19 20:57:57,381 INFO    [dr1/2] addons/minio/start completed in 20.45 seconds
2024-06-19 20:57:57,382 INFO    [dr1/2] Running addons/minio/test
2024-06-19 20:57:57,947 INFO    [dr1/2] addons/minio/test completed in 0.57 seconds
2024-06-19 20:57:57,947 INFO    [dr1/2] Running addons/velero/start
2024-06-19 20:58:13,405 INFO    [dr2/2] addons/minio/start completed in 22.14 seconds
2024-06-19 20:58:13,406 INFO    [dr2/2] Running addons/minio/test
2024-06-19 20:58:14,785 INFO    [dr2/2] addons/minio/test completed in 1.38 seconds
2024-06-19 20:58:14,786 INFO    [dr2/2] Running addons/velero/start
2024-06-19 20:58:44,564 INFO    [dr1/2] addons/velero/start completed in 46.62 seconds
2024-06-19 20:58:44,564 INFO    [dr1/2] Running addons/velero/test
2024-06-19 20:58:46,383 INFO    [hub/1] addons/submariner/start completed in 87.09 seconds
2024-06-19 20:58:46,383 INFO    [hub/1] Running addons/submariner/test
2024-06-19 20:58:53,441 INFO    [hub/0] addons/ocm-hub/start completed in 94.15 seconds
2024-06-19 20:58:53,442 INFO    [hub/0] Running addons/ocm-controller/start
2024-06-19 20:59:02,832 INFO    [dr2/2] addons/velero/start completed in 48.05 seconds
2024-06-19 20:59:02,832 INFO    [dr2/2] Running addons/velero/test
2024-06-19 20:59:16,030 INFO    [hub/0] addons/ocm-controller/start completed in 22.59 seconds
2024-06-19 20:59:16,030 INFO    [hub/0] Running addons/olm/start
2024-06-19 20:59:26,481 INFO    [dr1/2] addons/velero/test completed in 41.92 seconds
2024-06-19 20:59:34,330 INFO    [dr1/0] addons/rook-cluster/start completed in 143.90 seconds
2024-06-19 20:59:34,331 INFO    [dr1/0] Running addons/rook-toolbox/start
2024-06-19 20:59:39,382 INFO    [dr1/0] addons/rook-toolbox/start completed in 5.05 seconds
2024-06-19 20:59:39,383 INFO    [dr1/0] Running addons/rook-pool/start
2024-06-19 20:59:47,400 INFO    [dr2/2] addons/velero/test completed in 44.57 seconds
2024-06-19 20:59:54,812 INFO    [hub/1] addons/submariner/test completed in 68.43 seconds
2024-06-19 20:59:54,812 INFO    [hub/1] Running addons/argocd/start
2024-06-19 21:00:06,574 INFO    [hub/0] addons/olm/start completed in 50.54 seconds
2024-06-19 21:00:36,815 INFO    [dr2/0] addons/rook-cluster/start completed in 161.68 seconds
2024-06-19 21:00:36,815 INFO    [dr2/0] Running addons/rook-toolbox/start
2024-06-19 21:00:41,224 INFO    [dr2/0] addons/rook-toolbox/start completed in 4.41 seconds
2024-06-19 21:00:41,224 INFO    [dr2/0] Running addons/rook-pool/start
2024-06-19 21:00:41,510 INFO    [dr1/0] addons/rook-pool/start completed in 62.13 seconds
2024-06-19 21:00:41,510 INFO    [dr1/0] Running addons/rook-cephfs/start
2024-06-19 21:00:44,191 INFO    [dr1/1] addons/ocm-cluster/start completed in 259.72 seconds
2024-06-19 21:00:44,191 INFO    [dr1/1] Running addons/ocm-cluster/test
2024-06-19 21:00:52,563 INFO    [dr1/1] addons/ocm-cluster/test completed in 8.37 seconds
2024-06-19 21:00:52,563 INFO    [dr1/1] Running addons/recipe/start
2024-06-19 21:00:52,988 INFO    [dr1/1] addons/recipe/start completed in 0.43 seconds
2024-06-19 21:00:59,815 INFO    [dr2/1] addons/ocm-cluster/start completed in 244.32 seconds
2024-06-19 21:00:59,815 INFO    [dr2/1] Running addons/ocm-cluster/test
2024-06-19 21:01:04,567 INFO    [dr2/1] addons/ocm-cluster/test completed in 4.75 seconds
2024-06-19 21:01:04,567 INFO    [dr2/1] Running addons/recipe/start
2024-06-19 21:01:04,860 INFO    [dr2/1] addons/recipe/start completed in 0.29 seconds
2024-06-19 21:01:11,334 INFO    [dr2/0] addons/rook-pool/start completed in 30.11 seconds
2024-06-19 21:01:11,334 INFO    [dr2/0] Running addons/rook-cephfs/start
2024-06-19 21:01:13,515 INFO    [dr1/0] addons/rook-cephfs/start completed in 32.00 seconds
2024-06-19 21:01:13,515 INFO    [dr1/0] Running addons/rook-cephfs/test
2024-06-19 21:01:21,375 INFO    [dr1/0] addons/rook-cephfs/test completed in 7.86 seconds
2024-06-19 21:01:25,041 INFO    [hub/1] addons/argocd/start completed in 90.23 seconds
2024-06-19 21:01:25,042 INFO    [hub/1] Running addons/argocd/test
2024-06-19 21:01:35,978 INFO    [dr2/0] addons/rook-cephfs/start completed in 24.64 seconds
2024-06-19 21:01:35,978 INFO    [dr2/0] Running addons/rook-cephfs/test
2024-06-19 21:01:44,127 INFO    [dr2/0] addons/rook-cephfs/test completed in 8.15 seconds
2024-06-19 21:02:07,112 INFO    [hub/1] addons/argocd/test completed in 42.07 seconds
2024-06-19 21:02:07,114 INFO    [rdr/1] Running addons/volsync/start
2024-06-19 21:02:07,115 INFO    [rdr/0] Running addons/rbd-mirror/start
2024-06-19 21:02:51,169 INFO    [rdr/1] addons/volsync/start completed in 44.05 seconds
2024-06-19 21:02:51,169 INFO    [rdr/1] Running addons/volsync/test
2024-06-19 21:03:12,371 INFO    [rdr/0] addons/rbd-mirror/start completed in 65.26 seconds
2024-06-19 21:03:12,371 INFO    [rdr/0] Running addons/rbd-mirror/test
2024-06-19 21:03:28,525 INFO    [rdr/0] addons/rbd-mirror/test completed in 16.15 seconds
2024-06-19 21:03:35,468 INFO    [rdr/1] addons/volsync/test completed in 44.30 seconds
2024-06-19 21:03:35,469 INFO    [rdr] Dumping ramen e2e config to '/home/nsoffer/.config/drenv/rdr'
2024-06-19 21:03:35,648 INFO    [rdr] Environment started in 470.69 seconds


Deleted Records gets displayed

I have noticed another bug where a deleted record is still listed on the index page. When you click the Edit/Delete button, the system throws an error because deleted_at is NOT NULL therefore the record cannot be accessed. I am aware the models use Illuminate\Database\Eloquent\SoftDeletes.
I think this is because you are using Database Querys rather that Eloquent to fetch data for Datatables display.

I have added soft delete very recently. Try using fresh installation.
It will definitely work. Let me know once resolved.
Regards,

I have done composer update and now am running the latest version 1.0.14, but the problem still persists. Datatables is still fetching the deleted record to the index page.

Deleting only changes the "deleted_at" column. You can make the query only fetch values where the column value is null.

This problem is resolved in latest version of LaraAdmin.
DB::table function includes deleted items.
We used - ->whereNull('deleted_at');


Gdk.lua bug - Nice not working

Hello, I'm running the awesome-git version with nice (or at least, trying to), but unfortunately the following error occurs:
/usr/share/lua/5.4/override/Gdk.lua: 23 bad argument #1 to 'registerlock' (userdata expected, got nil)

Any ideas on how to fix it?
Running lastest Manjaro
Awesome Version:
awesome v4.3
Compiled against Lua 5.5.5 (running with 0.9.2)
LGI version: 0.9.2

Hello, with Awesome stable 4.3, you have to use the awesome-4v3-stable branch of this repository https://github.com/mut-ex/awesome-wm-nice/tree/awesome-4v3-stable.

[filip@Juliette awesome]$ git clone https://github.com/mut-ex/awesome-wm-nice.git --single-branch --branch awesome-4v3-stable nice
Cloning into 'nice'...
remote: Enumerating objects: 46, done.
remote: Counting objects: 100% (22/22), done.
remote: Compressing objects: 100% (8/8), done.
remote: Total 46 (delta 16), reused 14 (delta 14), pack-reused 24
Receiving objects: 100% (46/46), 4.12 MiB | 8.01 MiB/s, done.
Resolving deltas: 100% (22/22), done.
[filip@Juliette awesome]$ awesome -v
awesome v4.3-1588-gb54e50ad6 (Too long)
 • Compiled against Lua 5.4.4 (running with 0.9.2)
 • API level: 4
 • D-Bus support: yes
 • xcb-errors support: no
 • execinfo support: yes
 • xcb-randr version: 1.6
 • LGI version: /usr/share/lua/5.4/lgi/version.lua
 • Transparency enabled: yes
 • Custom search paths: no

Tried it with both, both return the same error(?)

@Aire-One

Ok, got it running. Turns out I had GTK 4 installed and it took over, BUT it's LGI is not compatible with it. For future people with this issue the solution is just uninstalling sudo pacman -S gtk4.
Works like a charm!

If the issue was indeed GTK, https://github.com/mut-ex/awesome-wm-nice/pull/33 should resolve it without you uninstalling GTK4. (maybe the correct fix is local gdk = lgi.require('Gdk', '3.0'))


Intro to Tensorflow Lab: Tensorflow version not importing

Installing the dlnd-tf-lab virtual env using the environment.yaml causes an issue with importing tensorflow.
Running this cell, with everything "out of the box":
%matplotlib inline

# Load the modules
import pickle
import math

import numpy as np
import tensorflow as tf
from tqdm import tqdm
import matplotlib.pyplot as plt

# Reload the data
pickle_file = 'notMNIST.pickle'
with open(pickle_file, 'rb') as f:
    pickle_data = pickle.load(f)
    train_features = pickle_data['train_dataset']
    train_labels = pickle_data['train_labels']
    valid_features = pickle_data['valid_dataset']
    valid_labels = pickle_data['valid_labels']
    test_features = pickle_data['test_dataset']
    test_labels = pickle_data['test_labels']
    del pickle_data  # Free up memory

print('Data and modules loaded.')

Causes an issue: TypeError: __init__() got an unexpected keyword argument 'file'
I've fixed this issue by changing the tensorflow version from tensorflow>=0.12 to tensorflow==0.12.
I will be making a PR for this to be changed.
(Not sure what the flow is, but making this issue so that if people search for this error they will at least find my solution)

Merged the PR to fix this.


swift-markdown doesn't detect Strong correctly in a specific case

swift-markdown doesn't detect Strong in
**b`h`** + letters or numbers

Examples:
**b`h`**abc
**b`h`**3k

Strong is correctly detected when the strings right after the markdown start with a sign, such as:
Examples:
**b`h`**=ab
**b`h`**/

or there is a space, such as:
Example:
**b`h`** a

I think swift-markdown should detect Strong in the first case as well.
Steps to reproduce
let markdown = "**b`h`**a"
let document = Document(parsing: markdown)
print("\(document.debugDescription())")

Expected output:
Document
└─ Paragraph
   ├─ Strong
   │  ├─ Text "b"
   │  └─ InlineCode `h`
   └─ Text "a"

Actual Output:
** isn't detected as Bold but as plain text.
Document
└─ Paragraph
   ├─ Text "**b"
   ├─ InlineCode `h`
   └─ Text "**a"

Similar situations where it works as expected
let markdown = "**b`h`** c"

Document
└─ Paragraph
   ├─ Strong
   │  ├─ Text "b"
   │  └─ InlineCode `h`
   └─ Text " c"

let markdown = "**b`h`**="

Document
└─ Paragraph
   ├─ Strong
   │  ├─ Text "b"
   │  └─ InlineCode `h`
   └─ Text "="

let markdown = "**`b`h**a"

Document
└─ Paragraph
   ├─ Strong
   │  ├─ InlineCode `b`
   │  └─ Text "h"
   └─ Text "a"

let markdown = "**b~h~**a"

Document
└─ Paragraph
   ├─ Strong
   │  ├─ Text "b"
   │  └─ Strikethrough
   │     └─ Text "h"
   └─ Text "a"

let markdown = "**b*h***a"

Document
└─ Paragraph
   ├─ Strong
   │  ├─ Text "b"
   │  └─ Emphasis
   │     └─ Text "h"
   └─ Text "a"

Environment

swift-markdown version: 0.4.0
Xcode version: 15.4 (15F31d)
macOS version: 14.3.1（23D60）

This is expected behavior. Emphasis delimiters cannot close emphasis if they are between punctuation and alphanumeric characters. This is part of the upstream GitHub-Flavored Markdown specification as well as the spec and implementation for its upstream CommonMark.


Color phylo tree viz by sample name

Color the tree by a given attribute name recursively
Change previous 'highlight' behavior to bold the path to the nodes
Pass down projects.name in phylo_tree.rb
Color in this photo is a little bad but Colormap will be improved

thanks @tfrcarvalho, will work on this

When you don't pass in options:

New look:

Jenn's previous comments:

I would make the circles smaller and opaque
Also I’d make the text all black instead of color
it would be great if you can add a slight rounded corner on the tree
i’d also make Project Name bolded and move the names closer to the square, like the same distance the sample names are from the circles
the 3 colors should be another combo, ill suggest something


craeted a full blog of project

I have created a full blog of project on hashnode and inserted its link in readme @Vikash-8090-Yadav

 https://codeshivam28.hashnode.dev/a-complete-blog-on-web30-world

Link of blog

@Bharadwajshivam28 describe each folder and their work in the blog .

ok sir

@Vikash-8090-Yadav Hello sir i have updated  the blog can you sir  just close the issue and if there will be more changes needed in blog then i will fix it


Mod configuration button without clicking on mod

Similar to #464
I can get easily frustrated figuring out which mods have configuration options and which ones don't - having the button to configure a mod visible and clickable on the mod list would help a lot.

You can filter by configurable mods easily by typing in "config" in the search box

This is useful! is there some way to reasonably make this the default?
On Wed, 26 Oct 2022, 6:20 pm Prospector, @.***> wrote:

You can filter by configurable mods easily by typing in "config" in the
search box
—
Reply to this email directly, view it on GitHub
https://github.com/TerraformersMC/ModMenu/issues/492#issuecomment-1291606081,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ANJ34KLZPQEWWQGRKYS3DRTWFDLUZANCNFSM6AAAAAAROVI2JA
.
You are receiving this because you authored the thread.Message ID:
@.***>

To add some clarification here, (and because the other half of this conversation is more #493) I'm looking to aid players in situations where they might not be able to find the config button, or not know it's the right one. Because it appears on a different part of the screen than the mod listed, it's easy to miss for someone less experienced.

Added in 6.1.0-rc.3, the mod icon is now clickable to open the config screen and you can now double-click a mod widget to open it as well.

oh that's swag as hell we just finished setting up the default hotkeyed
config mode stuff for our pack too
On Fri, 17 Mar 2023, 11:22 am Prospector, @.***> wrote:

Closed #492 https://github.com/TerraformersMC/ModMenu/issues/492 as
completed.
—
Reply to this email directly, view it on GitHub
https://github.com/TerraformersMC/ModMenu/issues/492#event-8773144546,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ANJ34KM33RQ6OX4JT7RTTCTW4OVC5ANCNFSM6AAAAAAROVI2JA
.
You are receiving this because you authored the thread.Message ID:
@.***>

Oh that's swag a s hellwe just finished setting up the default hotkeyed config mode stuff for our pack too.
I'm assuming we'll keep an eye on this for a portential 1.20 pack update (it's not backport stuff right?)

It may be backported, we'll see. I don't have the energy for that right now lol. Your screenshot just reminded me compact mode exists... I should check that it works in compact mode.

ah.

Minor note: compact mode without badges is also slightly smaller at least
in our testing.
On Fri, 17 Mar 2023, 11:56 am Prospector, @.***> wrote:

ah.
[image: image]
https://user-images.githubusercontent.com/6166773/225784469-f00ab224-6cb1-4c06-92fd-0cb682359fc3.png
—
Reply to this email directly, view it on GitHub
https://github.com/TerraformersMC/ModMenu/issues/492#issuecomment-1472952992,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ANJ34KOO6XFOSUXPVSEESBDW4OZCDANCNFSM6AAAAAAROVI2JA
.
You are receiving this because you authored the thread.Message ID:
@.***>

hmm, I don't think that should be the case?

Just reviewed - seems you're right there, we may just add badges back to our pack config then


v.qq.com Channel & Video Support

Please follow the guide below

You will be asked some questions and requested to provide some information, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your issue (like this: [x])
Use the Preview tab to see what your issue will actually look like

Make sure you are using the latest version: run youtube-dl --version and ensure your version is 2017.08.09. If it's not, read this FAQ entry and update. Issues with outdated version will be rejected.

[x] I've verified and I assure that I'm running youtube-dl 2017.08.09

Before submitting an issue make sure you have:

[x] At least skimmed through the README, most notably the FAQ and BUGS sections
[x] Searched the bugtracker for similar issues including closed ones

What is the purpose of your issue?

[ ] Bug report (encountered problems with youtube-dl)
[x] Site support request (request for adding support for a new site)
[ ] Feature request (request for a new functionality)
[ ] Question
[ ] Other

If the purpose of this issue is a bug report, site support request or you are not completely sure provide the full verbose output as follows:
Video:
~$ youtube-dl  http://v.qq.com/page/y/l/3/y0334dz08l3.html -v
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['http://v.qq.com/page/y/l/3/y0334dz08l3.html', '-v']
[debug] Encodings: locale UTF-8, fs utf-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2017.08.09
[debug] Python version 3.4.5 - Linux-3.10.0-514.26.1.el7.x86_64-x86_64-with-centos-7.3.1611-Core
[debug] exe versions: ffmpeg 2.6.8, ffprobe 2.6.8
[debug] Proxy map: {}
[generic] y0334dz08l3: Requesting header
[redirect] Following redirect to https://v.qq.com/x/page/y0334dz08l3.html
[generic] y0334dz08l3: Requesting header
WARNING: Falling back on generic information extractor.
[generic] y0334dz08l3: Downloading webpage
[generic] y0334dz08l3: Extracting information
ERROR: Unsupported URL: https://v.qq.com/x/page/y0334dz08l3.html
Traceback (most recent call last):
File "/home/gazlene/.local/lib/python3.4/site-packages/youtube_dl/YoutubeDL.py", line 776, in extract_info
ie_result = ie.extract(url)
File "/home/gazlene/.local/lib/python3.4/site-packages/youtube_dl/extractor/common.py", line 433, in extract
ie_result = self._real_extract(url)
File "/home/gazlene/.local/lib/python3.4/site-packages/youtube_dl/extractor/generic.py", line 2944, in _real_extract
raise UnsupportedError(url)
youtube_dl.utils.UnsupportedError: Unsupported URL: https://v.qq.com/x/page/y0334dz08l3.html
Channel:
youtube-dl  http://v.qq.com/vplus/26f76288cf7ea60e478b8fdb08d871fa  -v
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['http://v.qq.com/vplus/26f76288cf7ea60e478b8fdb08d871fa', '-v']
[debug] Encodings: locale UTF-8, fs utf-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2017.08.09
[debug] Python version 3.4.5 - Linux-3.10.0-514.26.1.el7.x86_64-x86_64-with-centos-7.3.1611-Core
[debug] exe versions: ffmpeg 2.6.8, ffprobe 2.6.8
[debug] Proxy map: {}
[generic] 26f76288cf7ea60e478b8fdb08d871fa: Requesting header
WARNING: Falling back on generic information extractor.
[generic] 26f76288cf7ea60e478b8fdb08d871fa: Downloading webpage
[generic] 26f76288cf7ea60e478b8fdb08d871fa: Extracting information
ERROR: Unsupported URL: http://v.qq.com/vplus/26f76288cf7ea60e478b8fdb08d871fa
Traceback (most recent call last):
File "/home/gazlene/.local/lib/python3.4/site-packages/youtube_dl/YoutubeDL.py", line 776, in extract_info
ie_result = ie.extract(url)
File "/home/gazlene/.local/lib/python3.4/site-packages/youtube_dl/extractor/common.py", line 433, in extract
ie_result = self._real_extract(url)
File "/home/gazlene/.local/lib/python3.4/site-packages/youtube_dl/extractor/generic.py", line 2944, in _real_extract
raise UnsupportedError(url)
youtube_dl.utils.UnsupportedError: Unsupported URL: http://v.qq.com/vplus/26f76288cf7ea60e478b8fdb08d871fa

Description of your issue, suggested solution and other information
I've been trying to scrape these videos since March or so. Adding support would be much appreciated, thank you. :)

Is it a duplicate of v.qq.com video or channel support?


Choosing custom SSL still inserts let's encrypt paths

In the subdomain section.
	# subdomains redirect
	server {
		listen 443 ssl http2;
		listen [::]:443 ssl http2;

		server_name *.my.domain.com;

		# SSL
		include _ssl.conf;
		ssl_certificate /etc/letsencrypt/live/my.domain.com/fullchain.pem;
		ssl_certificate_key /etc/letsencrypt/live/my.domain.com/privkey.pem;

		return 301 https://my.domain.com$request_uri;
	}

Hello @tmikaeld!
Thank you for the bug report, now it's fixed! 😉

Thanks for making this tool, about damn time someone did ;-)


Fix pd.read_excel

@hauntsaninja Can you add a test for this?  See test_frame.py:test_read_csv() .

Sorry, was away from laptop. I can add a test, but rather than slowly recreating a large part of pandas test suite, you may wish to consider using a tool like stubtest for basic consistency checks (https://github.com/pandas-dev/pandas-stubs/issues/11#issuecomment-1161268004)

I've run stubtest and it uncovered a number of issues, so I'm going to look into addressing those, and adding stubtest to our testing suite.

Thanks, fixed! stubtest has the option of using an allowlist, which can let you introduce it in CI today, and fix the backlog of errors over time. Let me know if you need help setting it up!

Thanks, fixed! stubtest has the option of using an allowlist, which can let you introduce it in CI today, and fix the backlog of errors over time. Let me know if you need help setting it up!

Thanks for letting me know and your offer of help.  The challenge is that we don't know what to put on the allowlist.
I'm hoping to fix all the issues this weekend.

Thanks @hauntsaninja

@hauntsaninja I tried stubtest, and can't get it to do what I'd like it to do.  mypy requires stub files to be installed, but I want to use this in development.  So in development, I have a local copy of pandas-stubs, and want to run stubtest so that it says "here is what is declared/imported in the installed version of pandas, but is missing from pandas-stubs"
If I install pandas-stubs, then I get complaints about third-party libraries that I don't care about.
Any suggestions on how to handle this use case would be welcome.

Sorry for the slow reply, I'm on vacation, so only checking my laptop intermittently.
@Dr-Irv stubtest requires mypy to run correctly on the stubs before it actually performs its checks. This is what it means when it says "error: not checking stubs due to mypy build errors"
λ python -m mypy.stubtest pandas --concise
error: not checking stubs due to mypy build errors:
pandas-stubs/plotting/_matplotlib/converter.pyi:1: error: Skipping analyzing "matplotlib.dates": module is installed, but missing library stubs or py.typed marker
pandas-stubs/plotting/_matplotlib/converter.pyi:1: error: Skipping analyzing "matplotlib": module is installed, but missing library stubs or py.typed marker
pandas-stubs/plotting/_matplotlib/converter.pyi:2: error: Skipping analyzing "matplotlib.ticker": module is installed, but missing library stubs or py.typed marker
pandas-stubs/plotting/_matplotlib/converter.pyi:6: error: Skipping analyzing "matplotlib.units": module is installed, but missing library stubs or py.typed marker
pandas-stubs/plotting/_matplotlib/boxplot.pyi:3: error: Skipping analyzing "matplotlib.axes": module is installed, but missing library stubs or py.typed marker
pandas-stubs/plotting/_misc.pyi:11: error: Skipping analyzing "matplotlib.axes": module is installed, but missing library stubs or py.typed marker
pandas-stubs/plotting/_misc.pyi:12: error: Skipping analyzing "matplotlib.figure": module is installed, but missing library stubs or py.typed marker
pandas-stubs/plotting/_core.pyi:8: error: Skipping analyzing "matplotlib.axes": module is installed, but missing library stubs or py.typed marker
pandas-stubs/core/series.pyi:23: error: Skipping analyzing "matplotlib.axes": module is installed, but missing library stubs or py.typed marker
pandas-stubs/core/groupby/generic.pyi:18: error: Skipping analyzing "matplotlib.axes": module is installed, but missing library stubs or py.typed marker
pandas-stubs/core/frame.pyi:24: error: Skipping analyzing "matplotlib.axes": module is installed, but missing library stubs or py.typed marker
pandas-stubs/core/frame.pyi:24: note: See https://mypy.readthedocs.io/en/stable/running_mypy.html#missing-imports
pandas-stubs/_libs/tslibs/timestamps.pyi:157: error: Return type "Tuple[int, int, int]" of "isocalendar" incompatible with return type "_IsoCalendarDate" in supertype "datetime"
pandas-stubs/_libs/tslibs/timestamps.pyi:157: error: Return type "Tuple[int, int, int]" of "isocalendar" incompatible with return type "_IsoCalendarDate" in supertype "date"

So it looks to me like there are two issues with the stubs that need to be fixed before stubtest can actually run.
First, mypy needs to be able to resolve the reference to matplotlib. This is easiest done by having pandas-stubs declare a dependency on a package that provides stubs for matplotlib. I'm not actually aware of such a package — has the Pylance team published their matplotlib stubs on PyPI?
Second, it seems like there's some minor issue in timestamps.pyi. It's possible there's a real error there, but you could also add a type ignore for it.
Once these two issues are resolved, stubtest will be able to run more completely and will likely have a lot more to say.

First, mypy needs to be able to resolve the reference to matplotlib. This is easiest done by having pandas-stubs declare a dependency on a package that provides stubs for matplotlib. I'm not actually aware of such a package — has the Pylance team published their matplotlib stubs on PyPI?

It's "published" on https://github.com/microsoft/python-type-stubs ,  but I know those stubs are incomplete.  And mypy won't read stubs unless they are installed.
With mypy, we use ignore_missing_imports to avoid those errors.

Second, it seems like there's some minor issue in timestamps.pyi. It's possible there's a real error there, but you could also add a type ignore for it for now.

We use python_version = "3.8" to suppress those messages.  That's because datetime.isocalendar() changed its return type from a tuple to a named tuple in python 3.9.

Once these two issues are resolved, stubtest will be able to run more completely and will likely have a lot more to say.

Would it be possible for stubtest to observe our mypy settings in its analysis?

Got it, in that case you can use python -m mypy.stubtest pandas --concise --mypy-config-file pyproject.toml to make it work.
However, I don't really recommend reusing your mypy config like this, since it won't be reflective of what your users will be doing. That is, your users won't have matplotlib stubs unless you make them a dependency, so in practice users' type checkers will resolve all of those to Any.

Example output, in case it's useful to you: https://paste.ee/p/KObbQ

@hauntsaninja Does stubtest also work well when there are only very few pyi files? The pandas code itself has a few pyi files for cython code. It would be great to use stubtest to ensure that all public functions declared in pyi match the cython implementation.

Does stubtest also work well when there are only very few pyi files

On mypy master, it should. There was a bug I fixed recently where stubtest would not check submodules that did not have corresponding pyi files. This was fixed in https://github.com/python/mypy/pull/13030 (should be released soon as part of mypy 0.970).
In general, if you wish to avoid errors about things that are missing from the stub and only wish to check consistency with the things that are there, stubtest supports an --ignore-missing-stub argument.

@twoertwein Looks like running stubtest against the main pandas repo finds a bunch of things, e.g. https://github.com/pandas-dev/pandas/pull/47756

@Dr-Irv I opened a PR to use stubtest on the pyi files within pandas https://github.com/pandas-dev/pandas/pull/47817
Obviosuly, stubtest will report many errors when comparing pandas-stubs with pandas. But as @hauntsaninja pointed out, stubtest can generate an allow/ignore list. As a starting point, we could have an unfortunately enormously long ignore list to at least prevent adding new stubs that are not compatible with the implementation (or at least be aware of it and add an exception to the list). Over time we can then work on making this list smaller. I think it will never be empty but we can definitely make it smaller :) This will especially be helpful for the cleaning efforts in #149 and #128.

Sorry for the stupid question @hauntsaninja How does stubtest need to be called on pandas or pandas-stubs:

python -m mypy.stubtest pandas-stubs --concise --ignore-missing-stub --mypy-config-file pyproject.toml throws many errors (for example that Series is not generic: it is generic in the stubs but not in the implementation)
python -m mypy.stubtest pandas --concise --mypy-config-file pyproject.toml cannot seem to find any stubs

After running pip install -e . in my copy of pandas-stubs, python -m mypy.stubtest pandas --concise --mypy-config-file pyproject.toml works for me.
Note that it complains about "failed to find stubs" for modules for which stubs do not actually exist, like pandas.util.version, but if you scroll up you'll see some more useful errors.

Thanks, I think I forgot to install the stubs!


Does this work with sub domains?

I was wondering if your package will work with subdomains?
thanks

If you use the multi-domain tenancy, yes. Though if all you want/need is basic sub-domains, Laravel 5.1+ actually supports sub-domain routing so you can do a form of tenancy without needing any additional modules. Details are documented at: https://laravel.com/docs/5.1/routing#route-group-sub-domain-routing
Otherwise, you would setup this module the same as for multi-domain tenancy. You'll need to have some database or at least an implementation of the DomainAwareTenantParticipantRepository (can be faked to use an array or other storage - not tested though). In theory it should work without Doctrine.
Hope that helps.

thanks!


feat: add quick-capture command

Description
This PR adds a new command to the extension that allows users to add tasks to a default list. Since ClickUp doesn't have a notion of a default list, the user must add their preferred listId to the extension's preferences.
The quick capture command works wonders for productivity as it allows one to add tasks to an inbox of sorts and later sort them out.
The command renders a form with basic fields:

The task name (required).
The task description.
A due date.
A priority level (ClickUp has only four priority levels and can't be modified).

Screencast
https://www.loom.com/share/d6828320718744cfa09d2e0002f5b7a0
Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[x] I ran npm run build and tested this distribution build in Raycast
[x] I checked that files in the assets folder are used by the extension itself
[x] I checked that assets used by the README are placed outside of the metadata folder

@Ali-Parandeh do you mind checking this too

@Ali-Parandeh do you mind checking this too

Thank you will do today

@Ali-Parandeh, thanks for the comments, I fixed most of them. LMKWYT!

Can you check once again @Ali-Parandeh


Fix NullReferenceExceptions during serialization

Pull Request Template
Description
This fixes observed null reference exceptions during serialization which seems to be caused by some race condition with the custom settings. To avoid this a new JsonSerializer is created for each call when there is custom settings. We found no issue when it was the default serializer so it does not create a new instance for each call to avoid the garbage collection overhead.
Type of change
Please delete options that are not relevant.

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Closing issues
Put closes #1079

We found no issue when it was the default serializer so it does not create a new instance for each call to avoid the garbage collection overhead

Is this validated or an assumption?

lt serializer so it does not create a new instance for each call to avoid the garbage collection overhead

All the stack traces are from custom converters path, and the default serializer doesn't have any so it shouldn't be an issue. So it's an assumption.

Micro benchmark showed below
Benchmark:
Thread safe is current implementation
InstanceImpl is when serializer is created for each serialization/de-serialization invocation.


Invalid date and message id

As @horazont pointed out, the message date and ID is invalid since Community Digest issue nr. 12.

The date probably got lost with commit 7c14dabf6f9bdc7646f00dd7eba9189f2bb2864d

The Message-ID should be set by swaks. We should add this to the README.MD


update TravisCI to run yarn in node 8 (drop node 6)

Current version of patternfly don't like node6, and since ovirt builds with node8, it doesn't make sense to retain node6 CI builds in travis any longer.
Also swapped from npm to yarn for the build to match what we're doing locally and in packaging.

@bond95 , @gregsheremeta - nudge for code review


bug: fix opencollective postinstall error

I believe opencollective-postinstall must be a prod dependency otherwise postinstall would fail unless it's installed globally or maybe another package installed it? Also || true throws an error on Windows.

:tada: This PR is included in version 10.1.2 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


No License

Please add a license (MIT)?
I would like to contribute.
Thank you

Sorry for the delay on this, wanted to make sure everything was cleared with our legal team first. I just added the MIT license to the repository. We're looking forward to your contributions!


Disable post and put retry

Currently, the post and put request can not work together with server connection retry mechanism.
Because ATS can not replay the post content.

**test golang script **
package main

import (
	"net"
	"os"
	"fmt"
)

var closed int = 0
var resp string = "HTTP/1.1 200 OK\r\n" +
	"Cache-Control: max-age=3000\r\n" +
	"Date: Tue, 06 Jun 2017 12:07:49 GMT\r\n" +
	"Content-Length: 10\r\n" +
	"Content-Type: text/plain; charset=utf-8\r\n\r\n" +
	"helloworld"

func main()  {

	ls, err := net.Listen("tcp", os.Args[1])
	if err != nil {
		fmt.Println(err)
		os.Exit(1)
	}

	for {
		conn, err := ls.Accept()
		if err != nil {
			fmt.Println(err)
			os.Exit(1)
		}
		closed ++
		if closed % 2 != 0 {
			conn.Close()
			fmt.Println("times: ", closed, "closed directly")
			continue
		}
	       fmt.Println("times: ", closed)
	       go handleConnection(conn)
	}
}

func handleConnection(con net.Conn)  {

	data := make([]byte, 40000)
	con.Read(data)

	fmt.Println(string(data))
	con.Write([]byte(resp))
	con.Close()
}

output
times:  1 closed directly
times:  2
POST /x HTTP/1.1
User-Agent: curl/7.35.0
Host: <IP_ADDRESS>:12345
Accept: */*
Content-Length: 3
Content-Type: application/x-www-form-urlencoded
X-Forwarded-For: <IP_ADDRESS>
Via: http/1.1 ubuntu-linux[5af10c97-8570-4476-8480-4db4f7abcc10] (ApacheTrafficServer/8.0.0)

// anther try
times 3
POST /x HTTP/1.1
User-Agent: curl/7.35.0
Host: <IP_ADDRESS>:12345
Accept: */*
Content-Length: 3
Content-Type: application/x-www-form-urlencoded
X-Forwarded-For: <IP_ADDRESS>
Via: http/1.1 ubuntu-linux[5af10c97-8570-4476-8480-4db4f7abcc10] (ApacheTrafficServer/8.0.0)

aaa

That we can see, we send the post request without data in previous request.

[approve ci]
@zwoop Is the autest system support golang script ?

POST and PUT should be retryable if we haven't sent any data to the client.  The first conditional checks the method and if there have been any bytes sent.  The data should still be around if we haven't tried to write it to the origin.
Something else looks to be broken.

This comes down to #947 according to @vmamidi. Should we consider backing it out?

@bryancall you are right. The reason is that we have already consumed the ua_buffer_reader in do_setup_post_tunnel
    client_request_body_bytes = post_buffer->write(ua_buffer_reader, chunked ? ua_buffer_reader->read_avail() : post_bytes);

    ua_buffer_reader->consume(client_request_body_bytes);

We will got nothing in the next term.  It can be fixed simply to remove this consume. And I will push another pr later and close this one.
@zwoop
Although # 947 brought us a lot of trouble,  I think this pr has nothing to do with #947.
For #947. IMHO, The best way to solve the problem is to revert #947 and reconsider the logic in iocore . @oknet  can do it better than the current logic .

considering the trouble made after #947 we definitely should sort it out, no offense because this is the common case when we work in iocore, I am with @zwoop and @scw00

@oknet @zwoop @bryancall  Please review!!

#1994

[approve ci]

[approve ci]

The post request has blocked by #2123  sometimes . It will be fixed when #2123 merge.
Now, that is working well with post request.
Fake vc is running in the same thread. fvc will collect all data before it call sm . If post succeed or post failed entirely, fvc will use fvc_callcont to return state machine in tunnel_handler_fake. If we connect to server and send data successfully, the default handler (tunnel_handler_post)will be used.
If server aborted, we will setup fake vc ->server tunnel directly, and send the data collected by fvc. fvc is freed is tunnel reset or kill tunnel.

@oknet have another way to solve this problem and that will be better than this one. He will push a new pr later