Responsive breakpoints break angular bindings

When a responsive breakpoint is hit, and the carousel reconfigures to have a different number of slides, angular bindings such as ng-click break. I believe this is because internally the carousel manipulates the DOM and that does not play well with angular.

I'm having the same issue with standard JS bindings. I have elements in a carousel which have click events bound - these appear to be getting wiped when applying the responsive property to the carousel initialisation.
Pen demonstrating the issue here - the responsive carousel overwrites click bindings:
http://codepen.io/JarredMack/pen/dPxVZg
I would imagine this has something to do with cloning the slide content for the responsive slides, but I haven't dug into the library yet.

The same issue occurs with standard JS click events - see the following pen for an example (drag window in to trigger responsive callback):
http://codepen.io/JarredMack/pen/dPxVZg
This is because in order to apply the responsive properties, slick is destroying itself and reinitialising in the refresh() method. Obviously, this blasts any bindings attached to the DOM and gives you fresh, unbound DOM content for your carousel.

Out of curiosity, I replaced _.refresh() with _.unfilterSlides(), which does largely the same thing without  destroying the existing carousel instance. The carousel correctly reapplied the responsive styles at each breakpoint, without clearing the DOM bindings due to reinjecting the content.
As _.refresh() is currently only called as part of the checkResponsive method, is it possible to have it reimplemented similarly to unfilterSlides? I don't see any reason it needs to destroy itself on screen resize.

excellent fix  - was tearing my hair out with this one - thank you - fyi my new working scenario
$('.newsmenu').on('init', function(event, slick) {
	slick.refresh = slick.unfilterSlides;
}).slick({
    infinite: false,
    speed: 1300,
    slidesToShow: 3,
    slidesToScroll: 1,
    arrows: true,
    responsive: [{
        breakpoint: 480,
        settings: {
            slidesToShow: 2,
            slidesToScroll: 1
        }
    }]
});

Fixed, thanks

Check my answer for Angular use-case:
http://stackoverflow.com/questions/29101254/angularjs-slick-js-carousel-ng-click-not-firing-with-responsive-attribute-settin/30584787#30584787


[Program]: Write a Rust program to swap two numbers

Description
Write a Rust program to swap two numbers
Input  : 10 20
Output : 20 10

How to contribute

Save the solution in program/program/swap-two-numbers/swap_two_numbers.rs file

Can u Assign this to me


Improve generality of handshakes

Right now, handshakes (which allows us to change digital pin values) are very basic. Could use a rewriting of this to give it more generality.

Since we have decided that being able to switch ON/OFF approx. 50 digital channels is enough, plus that gives us 255-100 = 155 other commands that we can send (a single char with values outside of the range that switches ON/OFF the channels), the current handshaking function will be enough.


Variable AutoDocNo Code[10]

Dear AL Team,
In the codeunit 444 "Purchase-Post Prepayments", we don't know if this is specific for Spanish functionality, the variable "AutoDocNo" is a Code[10], but is asigned to NoSeriesMgt.GetNextNo(...) that is a Code[20].
We correct this issue in NAV 2017 but we can't do it in our migration to Business Central 15.6.
Best Regards

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.


Install with MSDN Product key

It may just be me (likey) but it seems that using custom media with an MSDN iso I'm trying to make use of the Product key never gets used so the resulting image isn't activated.
I believe (though could be wrong) that the Product Key needs adding as a property in the CustomData section??

@kilasuit Correct - if specifying on the node configuration, it needs to specified in the CustomData hashtable:
@{
    AllNodes = @(
        @{
            NodeName = 'ProductKey';
            CustomData = @{ ProductKey = 'ABCDE-1234-...' }
        }
    )
}

Having said that, I think there is something missing here. You can define a product key in the CustomData property when registering custom media (https://github.com/VirtualEngine/Lability/blob/dev/Src/LabMedia.ps1#L35). However, it looks like this is never actually used?! I'll do some checking and make sure that a product key defined in the media configuration is used.

I think that the if statement needs to be above where the $labMedia object gets created so that the $CustomData object has that property. https://github.com/VirtualEngine/Lability/blob/dev/Src/LabMedia.ps1#L73
We would also need to replicate this in Register-LabMedia too with an additional parameter but that's only a small thing

@kilasuit Just to confirm - this is an issue with a ProductKey defined in a registered media entry? If you specify the ProductKey in the configuration .psd1 file, it is passed through successfully?


gulp build.prod broken?

It isn't use:
gulp build.prod --evn prod

You're right. My bad. Sorry

or
npm run build.prod

@neilyoung no worries! The current API is slightly confusing anyway. In case you use gulp build.prod it shouldn't be necessary to set the environment flag.

However, I cannot even produce something useful for prod. Could you please check? Fresh install. npm run build.prod ok, but it is not really running then

@neilyoung you should check #350


Module not found: Error: Can't resolve 'throat'

Describe the bug
After upgrading from 1.0.0-1 to 1.0.0-2 I'm getting this error when compiling:
Module not found: Error: Can't resolve 'throat'
To Reproduce
Steps to reproduce the behavior:

Upgrade plugin to version 1.0.0-2 'npm install<EMAIL_ADDRESS>npx cap sync
Try to compile with ionic build
See error

Error<EMAIL_ADDRESS>Module not found: Error: Can't resolve 'throat'

Expected behavior
Dependencies should be automatically updated and build should run through
Plugin version:

@capacitor-community/bluetooth-le: 1.0.0-2

Thanks for reporting this. It has been fixed in 1.0.0-3.


Proposal: Commas Work Like Semicolons

This is based on @Apanatshka's comment on the Optional Commas proposal. It essentially says that commas in Elm should work like semicolons do in other languages, such as in Ruby or in Haskell's do notation—that is, they are used as a separator only in single-line cases, and in multiline cases newlines are the separator instead. This proposal is simpler but less backwards-compatible than optional commas.
Here is TodoMVC with and without the proposed change:
Current TodoMVC
TodoMVC with proposed change
Proposed Implementation
1. Multiline terms are delimited by newlines instead of commas. This is a breaking change.
Examples:
-- equivalent to:
-- { x = 1, y = 5 }
{
  x = 1
  y = 5
}

-- equivalent to:
-- [label [class "login-field"] [text "username:"], input [name "username"] [], button [] [text "Submit"]]
[
  label [class "login-field"] [text "username:"]
  input [name "username"] []
  button [] [text "Submit"]
]

-- equivalent to:
-- (x, y, z)
(
  x
  y
  z
)

-- equivalent to:
-- type alias Bounds = { width : Int, height : Int }
type alias Bounds = {
  width : Int
  height : Int
}

-- equivalent to:
-- module Foo (bar, baz, qux) where
module Foo (
    bar
    baz
    qux
  ) where

-- equivalent to { point | x <- point.x + 1, y <- point.y + 1 }
{ point | x <- point.x + 1
          y <- point.y + 1 }

2. Commas become delimiters for single-line terms only.
Comma-delimited terms that begin and end on a single line work as they currently do. Examples:
[firstThing, secondThing, thirdThing]

{ x = 1, y = 5 }

(x, y, z)

type alias Bounds = { width : Int, height : Int }

module Foo (bar, baz, qux) where

3. To remove ambiguity, function application across multiple lines now requires indentation. This is a breaking change.
-- still compiles
foo 1 2 3

-- still compiles
foo
  1 2 3

-- no longer compiles
foo
1 2 3

4. The preceding rule implies that all elements in a multiline term must be at the same indentation level.
For example:
-- still compiles
[
  label [class "login-field"] [text "username:"]
  input [name "username"] []
  button [] [text "Submit"]
]

-- does not compile, because this is attempting to pass 3 extra arguments to `input`
-- (namely `button`, `[]`, and `[text "Submit"]`)
[
  label [class "login-field"] [text "username:"]
  input [name "username"] []
    button [] [text "Submit"]
]

-- compiles to the equivalent of (3, 4, 5)
(
  3
  4
  5
)

-- does not compile, because this is attempting to call 3 as if it were a function,
-- passing 4 as an argument.
(
  3
    4
  5
)

5. Commas may not be used to delimit multiline terms.
-- compiles
[foo, bar, baz]

-- compiles
[
  foo
  bar
  baz
]

-- does not compile
[
  foo, bar
  baz
]

Motivation
To avoid duplication, please see the original Optional Commas proposal for motivation. All the same motivating reasons apply here as well.

I like this proposal a lot better than the previous one, since it means we can still write [1,3,5] one-liners. Would the same approach work for union types? Like so:
type myUnion = Foo | Bar
    Qux Int Int String
    Norb Float String Char

As has been said, I like the pipe for union types as evocative of EBNF or an "or" (and the similarity between bitwise or and set union). I really dislike it for multiway if though, and would love to see that dropped.

This is definitely better than the previous one. But these examples still make me uncomfortable:
-- compiles to the equivalent of (3, 4, 5)
(
  3
  4
  5
)

-- does not compile, because this is attempting to call 3 as if it were a function,
-- passing 4 as an argument.
(
  3
    4
  5
)

The tuple like that looks very weird :pensive: I'm particularly uncomfortable with this because you usually use round brackets for grouping of expressions. With commas it's clear that it's a tuple instead of a grouping for priority of expressions. I really don't like this :worried:
I can imagine names instead of numbers which would make the second example be correct. And it looks like you can make mistakes there as easily as in CoffeeScript :disappointed: The type-system will help you out is most cases I guess. But I remember making indentation errors in Python that left me with hard to track down bugs. If this proposal opens the door for any of those, I'm against :neutral_face:

I'm particularly uncomfortable with this because you usually use round brackets for grouping of expressions. With commas it's clear that it's a tuple instead of a grouping for priority of expressions.

Hm...I don't follow. When you group expressions with parentheses in Elm, they are always necessarily nested, yeah? Like, you would never have something like this:
(
  expr1
  expr2
  expr3
)

I guess I don't follow what the code would look like that becomes confused with tuples here. Did you have a particular example in mind?

The tuple like that looks very weird

Multiline tuples look weird in general...I mentioned it as a clarification for what would happen if you wanted to do one, but there's a reason you don't see them in the wild very often. :smile:
Actually I can't personally think of a time I've ever seen one, regardless of comma styles.

I can imagine names instead of numbers which would make the second example be correct.

That's true of all function application in Elm, not just this proposal. :wink:
Example:
--compiles
foo bar baz

-- does not compile
3 4 5

I remember making indentation errors in Python that left me with hard to track down bugs.

This cuts both ways though. I spend much more time tracking down syntax bugs in Elm than I do in CoffeeScript, because the latter is much friendlier to rearranging code.
Specifically, very often I want to move a line of rendering code from one place to another. When I do, I can’t just cut/paste the relevant lines of code; instead, I have to cut/paste and then fix commas. Whenever I forget to fix commas, or when I accidentally fix them incorrectly, I lose even more time because the resulting error isn’t “you forgot a comma,” but rather something that will several minutes later cause me to smack my forehead and add a comma.
This isn't just me either. When I did an Elm workshop (20ish people in attendance) at LambdaConf, when someone couldn't get something to compile, the second most common solution was "you need to add a comma." (The most common solution was "restart elm-reactor" but that's obviously unrelated here.)

Put another way: my experience has been that the UX of reading and editing a language with newlines-over-commas syntax is nicer than the UX of reading and editing a language with mandatory commas. :smiley:
Also worth noting that the only hard-to-track-down bugs I've ever gotten were around long conditionals...and I can't recall any that would have bitten me if (1) each conditional required an else, (2) each conditional were exactly one expression instead of arbitrarily many statements, and espeically (3) I had a type-checker that told me when my conditional branches were returning different types (yeah 0.15.1! :smile_cat: )
So while I completely understand why that Pythonic experience would give you pause, it's definitely something I've considered and do not think is a cause for concern in Elm's case.

You argue for this change convincingly. I guess I'm being too conservative, but I'm not completely convinced yet. Let's talk it over on Wednesday :)

:+1: Sounds good to me! :smiley:

I think this would add confusion for beginners, particularly those already getting used to the lack of parentheses in function calls. Also, I would imagine this would make for some confusing error messages, particularly with regard to the difference between:
(
  expr1
  expr2
  expr3
)

And:
(
  expr1
    expr2
    expr3
)

I think the massive popularity of CoffeeScript (it even ships by default with Rails!) is an important point to remember. It suggests that neither paren-free function application nor comma-free delimiting (CoffeeScript has both, and actually takes significant whitespace substantially further) are too much for a mainstream JS audience.
Also, keep in mind that you're going to get a surprising compiler error no matter what if you mistakenly write this:
(
  expr1
  expr2
  expr3
)

Today you'll either get a type error saying you passed too many arguments to whatever function kicks off expr1, or a syntax error saying unexpected expr2 if expr1 is something like an if expression instead of a function call.
In the world of this proposal, instead you get a type mismatch between Tuple3 and whatever you thought you'd be getting - presumably the type of expr3.
Relevantly, the latter scenario is more conducive to a helpful compiler error message ("Did you realize you were creating a Tuple here?") because it will consistently be a type error regarding a Tuple rather than sometimes a type error and sometimes a syntax error.

CoffeeScript doesn't curry by default, however, so function application without parentheses looks like f a, b. The example they give on their website: console.log sys.inspect object -> console.log(sys.inspect(object)); would be more like console.log(sys.inspect, object); in Elm.

Sure, but foo bar baz is a perfectly valid thing to write in both Elm and CoffeeScript; they just do different things. :smiley:

I like [foo bar] as an example of how Elm treats whitespace as function application, and Clojure treats it as  a delimiter. That snippet can also mean function application in Ruby, if self.foo is defined and takes one argument.

There doesn't appear to be any examples of record update syntax?

Here?

@rehno-lindeque Added an example to the end of the first code snippet, thanks!

Richard Feldman writes:

Sure, but foo bar baz is a perfectly valid thing to write in both Elm and CoffeeScript; they just do different things. :smiley:

Reply to this email directly or view it on GitHub:
https://github.com/elm-lang/elm-compiler/issues/979#issuecomment-119235577

I just wanted to point out that they do different things, so the
argument that mainstream language X does it, so we can, too, doesn't
quite hold.
--
Kyle Marek-Spartz

But that's an argument against currying by default, not newlines-over-commas; it is already the case that foo bar baz works differently in Elm than it does in CoffeeScript.
In contrast, this proposal substitutes newlines for commas in very much the same way you can in CoffeeScript. The following code gives you an list of 3 items in either CoffeeScript or in this proposal:
[
  foo
  bar
  baz
]

(Technically an Array in CoffeeScript and a List in Elm, but otherwise they do the same thing.)

To catch this thread up with a mailing list discussion, here is TodoMVC with and without the proposed change:
Current TodoMVC
TodoMVC with proposed change

I can't see anything wrong with the new proposal and it drastically improves DSLs. I'm on board 👍

Just FYI: I forked Richard's TodoMVC gist and wrote something in similar style but with operators instead of whitespace so it already works in Elm. You get stuff like:
( []
  :< foo
  :< bar
  :< baz
)

When @rtfeldman proposed this I asked myself, why bother? I asked him, why not trailing commas; to which he asked, why not leading commas. In either case they'd be janky if not mandatory, leading me to wonder if they present semantic value. Commas make it visually unambiguous that we're not applying functions. This was actually a concrete issue using LiveScript (which I enjoyed even if janky). Except with Elm, square brackets conspicuously do that.
CoffeeScript is the most prominent case of significant whitespace and optional commas I'm aware of. I strongly prefer it to Python in experience. I googled a thoughtful syntax critique, but none of his criticisms applied to Elm.
What draws me most to this proposal is that Haskell's record syntax has always looked annoying to me, akin to double-colon type annotations. Reading an article comparing rank-2 poly to type classes, I realized what makes type classes appear preferable is primarily lack of commas:
-- using Elm whitespace conventions

-- rank-2 monad works more better
monad'Maybe :: MonadI Maybe
monad'Maybe = MonadI
    { _return = Just
    , _bind = \m f -> case m of
        Nothing -> Nothing
        Just x  -> f x
    }

-- type class monad looks more better
instance Monad Maybe where
    return = Just
    m >>= f = case m of
        Nothing -> Nothing
        Just x  -> f x

The reason I think syntax proposals are being expressed is because syntax is the surface API of a language. Elm is already awesome at balancing the beam between minimal, effective, and intuitive syntax. They're are perhaps more practical issues moving forward like asynchrony and tooling. But both open issues seem fundamentally motivated by code readability. Just my two cents in case Richard missed anything.
Enjoying learning from so many brilliant minds!

I talked with @rtfeldman offline about this.
I think https://github.com/elm-lang/elm-plans/issues/2 is a far less controversial way to accomplish the root goal of "copy code around easily in a style that is familiar". It even does a better job on the "style that is familiar" part :)
One thing I really don't want to do is "fight the comma wars". We have lots of battles to fight between types and immutability and managed effects, adding something like this to the list seems like a bad mistake. I personally quite like the leading comma stuff, but in the spirit of "don't fight the comma wars" I don't think it makes sense to be in the business of making people go that route.
@rtfeldman, okay to close this in favor of the trailing commas proposal?

:+1:


Change PSNativePSPathResolution to not be Experimental

PR Summary
Remove the code making this feature Experimental.  The changes in NativeCommandProcessor.cs is really just removing the check if the ExperimentalFeature is enabled and shifting everything left which makes it look like more changes than it actually is.
PR Context
Based on https://github.com/PowerShell/PowerShell/issues/13406 this feature is no longer Experimental for 7.1
PR Checklist

[x] PR has a meaningful title

Use the present tense and imperative mood when describing your changes

[x] Summarized changes
[x] Make sure all .h, .cpp, .cs, .ps1 and .psm1 files have the correct copyright header
[x] This PR is ready to merge and is not Work in Progress.

If the PR is work in progress, please add the prefix WIP: or [ WIP ] to the beginning of the title (the WIP bot will keep its status check at Pending while the prefix is present) and remove the prefix when the PR is ready.

Breaking changes

[x] None
OR
[ ] Experimental feature(s) needed

[ ] Experimental feature name(s):

User-facing changes

[ ] Not Applicable
OR
[x] Doc needed

[x] Issue filed: https://github.com/MicrosoftDocs/PowerShell-Docs/issues/6557

Testing - New and feature

[ ] N/A or can only be tested interactively
OR
[x] Make sure you've added a new test if existing tests do not effectively test the code changed

Tooling

[x] I have considered the user experience from a tooling perspective and don't believe tooling will be impacted.
OR
[ ] I have considered the user experience from a tooling perspective and enumerated concerns in the summary. This may include:

Impact on PowerShell Editor Services which is used in the PowerShell extension for VSCode (which runs in a different PS Host).
Impact on Completions (both in the console and in editors) - one of PowerShell's most powerful features.
Impact on PSScriptAnalyzer (which provides linting & formatting in the editor extensions).
Impact on EditorSyntax (which provides syntax highlighting with in VSCode, GitHub, and many other editors).

@SteveL-MSFT (and others) When you revert a change like that has a documentation impact, be sure to open a docs issue. I caught this one and backed out the doc change already.


Set the baseUrl for the dev-server

I just spent many hours trying to work out how to mount the dev server at a child path in the URL.
For example; my application is mounted at a child path of /app/. The root of the url http://someaddress.com/ is owned by other code and the Vuejs app needed to be mounted under http://someaddress.com/app/
Finally I found the solution which is two part:

Change the assetsPublicPath in the config/index.js file like so:

  dev: {
    env: require('./dev.env'),
    port: 2001,
    autoOpenBrowser: true,
    assetsSubDirectory: 'static',
    assetsPublicPath: '/app/',
    proxyTable: {},
    // CSS Sourcemaps off by default because relative paths are "buggy"
    // with this option, according to the CSS-Loader README
    // (https://github.com/webpack/css-loader#sourcemaps)
    // In our experience, they generally work as expected,
    // just be aware of this issue when enabling this option.
    cssSourceMap: false
  }

In the dev-server.js file, change the connect-history-api-fallback middleware to support a new index path:

app.use(require('connect-history-api-fallback')({
  index: '/app/index.html'
}))

I suggest a change to the template to make it easier to change the base url used for mounting the app. I quick example of the change being something like this:

app.use(require('connect-history-api-fallback')({
  index: config.dev.publicPath + '/index.html'
}))

That dynamic path would need to be parsed but you get the idea. Maybe behind a flag of changeBaseUrl or something like that.

I've been thinking on this last night. Rather than some multi-option like changeBaseUrl it would be better just to expose the index file option. Something like this:

{
  defaultDocument: `/index.html`
}

This then allows the user to set the value to anything rather than just change the sub-path. Couple this with a good documentation example of how to change the Vue application to a sup-path and you are set.

Note: index.html is also referenced here:
https://github.com/vuejs-templates/webpack/blob/master/template/build/webpack.dev.conf.js#L29
and here:
https://github.com/vuejs-templates/webpack/blob/master/template/build/webpack.prod.conf.js#L54

Ah, thanks @LinusBorg.
Documentation on those to extra references: https://github.com/jantimon/html-webpack-plugin
I don't think I need to change those setting in my use case however I'm not sure how you would want to expose them to a template user through configuration.
Maybe something like the following config options:

{
  htmlWebServerDefaultDocument: 'index.html',
  htmlTemplate: 'index.html',
  htmlBuildFile: 'index.html'
}

Another option would be to not add these configuration options and just document how to change them and what they do?
I think the browser default document should be added though because in its current state there is no configuration for the 'connect-history-api-fallback' constructor.

Thank you for the direction on getting this going... for me I also had to modify build/dev-client.js to have the __webpack_hmr requesting through the sub path as well by adding a path argument to the hotclient:
var hotClient = require('webpack-hot-middleware/client?path=/app/__webpack_hmr&noInfo=true&reload=true')

Which if you may also want to consider for your configuration or documentation... Or if there is a better way of accomplishing the above please let me know.

@iAlexanderMoon Hi. I didn't know about this change. Thanks for chiming in. I'll try it out however the client seems to be working ok as it is. I am using nginx as a front-end so it might be handling it for me.

OK, I got it working @iAlexanderMoon. Requires two changes:

dev-client.js add the path as you have shown above:

/* eslint-disable */
require('eventsource-polyfill')
var hotClient = require('webpack-hot-middleware/client?path=/app/__webpack_hmr&noInfo=true&reload=true')

hotClient.subscribe(function (event) {
  if (event.action === 'reload') {
    window.location.reload()
  }
})

In the dev-server.js add the path option:

var hotMiddleware = require('webpack-hot-middleware')(compiler, {
  path: '/app/__webpack_hmr',
  reload: true, // <= I changed this. Do we need it?
  log: console.log // <= I added this. I prefer more feedback in my console.
})

Note: The webpack-hot-middleware options came from here https://github.com/glenjamin/webpack-hot-middleware/blob/master/client.js

Change Dev Server Base URL Config Summary
To change the base url of the dev-server.js which is used during npm run dev on the template, you need to make the following changes. Note that the changes below assume a child path of /app/.

Change the assetsPublicPath in the config/index.js file like so:

  dev: {
    env: require('./dev.env'),
    port: 2001,
    autoOpenBrowser: true,
    assetsSubDirectory: 'static',
    assetsPublicPath: '/app/',
    proxyTable: {},
    cssSourceMap: false
  }

In the build/dev-server.js file, change the connect-history-api-fallback middleware to support a new index path:

app.use(require('connect-history-api-fallback')({
  index: '/app/index.html'
}))

In the build/dev-client.js add the path into the option string for the hot module reload client:

/* eslint-disable */
require('eventsource-polyfill')
var hotClient = require('webpack-hot-middleware/client?path=/app/__webpack_hmr&noInfo=true&reload=true')

hotClient.subscribe(function (event) {
  if (event.action === 'reload') {
    window.location.reload()
  }
})

In the build/dev-server.js file add the path option for the webpack-hot-middleware:

var hotMiddleware = require('webpack-hot-middleware')(compiler, {
  path: '/app/__webpack_hmr',
  reload: true, // <= I changed this. Do we need it?
  log: console.log // <= I added this. I prefer more feedback in my console.
})

If you wish to change the default document from index.html so another name like default.htm then you will need to change the filename/template names in these files:

https://github.com/vuejs-templates/webpack/blob/master/template/build/webpack.dev.conf.js#L29
https://github.com/vuejs-templates/webpack/blob/master/template/build/webpack.prod.conf.js#L54

@LinusBorg This should be broken up into two problems; default document name and child path.

How I finally did it
build/dev-client.js
// add the full path "path=http://localhost:8080/__webpack_hmr"
var hotClient = require(
    'webpack-hot-middleware/client?noInfo=true&reload=true&path=http://localhost:8080/__webpack_hmr')

build/dev-server.js
var devMiddleware = require('webpack-dev-middleware')(compiler, {
  publicPath: webpackConfig.output.publicPath,
  headers: { 'Access-Control-Allow-Origin': '*' }, // add headers for CORS
  quiet: true
})

config/index.js
...
assetsSubDirectory: 'static',
assetsPublicPath: 'http://localhost:8080/', // put the full path here too (for hot-updates)
proxyTable: {},
...

@BafS 's solution should be added to the README

I could be wrong here however @BafS solution is just proxying. This issue is about child paths.

Any idea how to make the paths in the router js file respect the base url app ?

the router has a base: option, check it out in the docs.

how to set the baseurl on production?

Since we switched to webpack-dev-server, this no longer applies afaics

Thanks @schettino72 but i still cant enable hot reloading when using proxy


"Dudești" is listed in the drop-down menu

Description:
"Dudești" is listed in the drop-down menu alongside its corresponding county and commune, according to the law
Precondition:
The website is up an running.

Step 1
Type "Dudești" in the search bar.

Expected results
The location is listed in the drop-down menu as "Dudești, Brăila (Dudești)".


electron-mail: 4.12.2 -> 4.12.5

Motivation for this change
ElectronMail can use FreeDesktop SecretService to store its master password, keep the user signed in.
It did not work in NixOS because of missing libsecret dependency.
The package itself has been updated to latest version as well.
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

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

[x] Fits CONTRIBUTING.md.

I played arount with the app, and no matter whether I use bundled electron or the one from nix-pkgs, there is more broken functionality:

Open settings folder does not work
Clicking links in emails does nothing
Adding attachments does not work

I've built the project outside of nixpkgs, using nix-shell with required dependencies, and everything works.
I wonder whether doing normal build using yarn (without yarn2nix etc) would be acceptable? There is a lot of build logic in .ts so it would be a lot of work to rewrite. Besides the build scripts are designed so that even with yarn the project is reproductible.

Updated to 4.12.7 which resolves attachments issue.

@ofborg eval


Introduce a new purge-uploads sub command

Now distribution supports delete outdated upload files in serve sub-command, in a goroutine in the HTTP server, it can delete the old upload files by the startedat, and run periodically.
It's functional enough for some cases, but for a larege image registry, there will be huge repos(directories) in the object store, walk through all the directories will take a long time. And because of high concurrency access, we also deploy distribution with many replicas(in K8s pod), so we want run purge operation only in a single instance to save API calls and waste of CPU, in the current design, we should deploy one replica of them with uploadpurging enabled and others are disabled, that make we to maintain two copies of the config.yaml.
So beside the long-running daemon mode, I think a new run once sub command purge-uploads like garbage-collect maybe needed. With this command, we can split the purging work from the distribution server, and integrate it into the normal operation workflow, or run purging work at any time without redeploy/restart any distribution servers.
Below is an example of this sub-command:
With this config.yaml:

version: 0.1
storage:
  s3:
    accesskey: xx
    secretkey: yy
    region: ap-northeast-1
    bucket: registry-lb
  maintenance:
    uploadpurging:
      enabled: true
      age: 168h
      interval: 4h
      dryrun: true

NOTE: in command-line mode, the enabled will always be true, and interval will be ignored.
Run in dry-run mode, only print out purge candidates, and overwrite --age by 1m.
$ bin/registry purge-uploads --dry-run --age 1m bin/config.yaml

Purge Option:
  Enabled:  true
  DryRun:   true
  Age:      1m0s
  Interval: 4h0m0s

INFO[0000] PurgeUploads starting: olderThan=2023-09-27 15:34:11.922612 +0800 CST m=-59.991133957, actuallyDelete=false
INFO[0000] Upload files in /docker/registry/v2/repositories/lbtest/_uploads/0c8fbf63-4748-4ef0-b386-b3d3efd3537e have older date (2023-09-27 07:30:50 +0000 UTC) than purge date (2023-09-27 15:34:11.922612 +0800 CST m=-59.991133957).  Removing upload directory.
INFO[0000] Upload files in /docker/registry/v2/repositories/lbtest/_uploads/f28a37fc-1364-4841-bc18-90e56c030753 have older date (2023-09-27 07:30:50 +0000 UTC) than purge date (2023-09-27 15:34:11.922612 +0800 CST m=-59.991133957).  Removing upload directory.
INFO[0000] Purge uploads finished.  Num deleted=2, num errors=0

Disable dry-run, and do actually delete uploads.
$ bin/registry purge-uploads --dry-run=false --age 1m bin/config.yaml

Purge Option:
  Enabled:  true
  DryRun:   false
  Age:      1m0s
  Interval: 4h0m0s

INFO[0000] PurgeUploads starting: olderThan=2023-09-27 15:35:07.251028 +0800 CST m=-59.992511415, actuallyDelete=true
INFO[0000] Upload files in /docker/registry/v2/repositories/lbtest/_uploads/0c8fbf63-4748-4ef0-b386-b3d3efd3537e have older date (2023-09-27 07:30:50 +0000 UTC) than purge date (2023-09-27 15:35:07.251028 +0800 CST m=-59.992511415).  Removing upload directory.
INFO[0000] Upload files in /docker/registry/v2/repositories/lbtest/_uploads/f28a37fc-1364-4841-bc18-90e56c030753 have older date (2023-09-27 07:30:50 +0000 UTC) than purge date (2023-09-27 15:35:07.251028 +0800 CST m=-59.992511415).  Removing upload directory.
INFO[0001] Purge uploads finished.  Num deleted=2, num errors=0

Run it again, no candidates are found.
$ bin/registry purge-uploads --dry-run=false --age 1m bin/config.yaml

Purge Option:
  Enabled:  true
  DryRun:   false
  Age:      1m0s
  Interval: 4h0m0s

INFO[0000] PurgeUploads starting: olderThan=2023-09-27 15:35:13.78747 +0800 CST m=-59.991400915, actuallyDelete=true
INFO[0000] Purge uploads finished.  Num deleted=0, num errors=0

@liubin should we rather consider changing the purge upload logic to delete outstanding uploads as they are encountered, rather than gathering all outstanding uploads then deleting them?
A potential problem with the approach in your MR is that this command would take a long time to run and be rather memory intensive as the size of outstanding uploads increases. If we were to first change the upload purging directly to handle larger registries more gracefully, we might benefit all current users as well as setting ourselves up for a more effective upload purge command if that's still required.


Testing in a single node

Testing Backend throttling behaviour with a single APIM node to test the general flow.

Completed, hence closing


Feature/test 01

PR Type
enhancement

Description

Implemented a new WalletController class in wallet_controller.dart to manage wallet-related state and actions, including QR scanning and weather data handling.
Developed a comprehensive WalletPage UI in wallet_page.dart featuring wallet balance display, weather information, and action buttons for operations like buy, receive, and exchange.
Updated the GitHub Actions workflow in pr-agent-review.yaml to enhance the PR agent job by responding to user comments and removing an unused environment variable.

PRDescriptionHeader.CHANGES_WALKTHROUGH
Relevant filesConfiguration changes

pr-agent-review.yamlUpdate PR agent job configuration in workflow

.github/workflows/pr-agent-review.yaml
Updated job name to include response to user comments.  Removed optional OPENAI_ORG environment variable.

+1/-2

Enhancement

wallet_controller.dartImplement WalletController with QR scanning and state management

wallet/wallet_controller.dart
Added WalletController class with state management.  Implemented QR scanning functionality.  Included weather and wallet connection session handling.

+54/-0

wallet_page.dartDevelop WalletPage UI with balance and actions

wallet/wallet_page.dart
Created WalletPage UI with various widgets.  Integrated wallet balance display and weather information.  Added action buttons for wallet operations.

+466/-0

💡 PR-Agent usage: Comment /help "your question" on any pull request to receive relevant information

/review

/improve --extended

/review


fix: Display document dropdown icons

Signed-off-by: TheYoungManLi<EMAIL_ADDRESS>

/assign @Bettygogo2021

This commit need cherry-pick to v0.6 branch.

This commit need cherry-pick to v0.6 branch.

ok


Latest Ember CLI breaks addon

Upgrading from Ember CLI 0.2.7 to...1.13.1 causes Ember Chartist to barf out an error....
Path or pattern "bower_components/chartist/dist/chartist.js" did not match any files
Error: Path or pattern "bower_components/chartist/dist/chartist.js" did not match any files
    at Object.multiGlob (/my-code/test-project/node_modules/ember-cli/node_modules/broccoli-kitchen-sink-helpers/index.js:203:13)
    at Class.module.exports.CachingWriter.extend.addFiles (/my-code/test-project/node_modules/ember-cli/node_modules/broccoli-sourcemap-concat/concat-with-maps.js:74:13)
    at Class.module.exports.CachingWriter.extend.updateCache (/my-code/test-project/node_modules/ember-cli/node_modules/broccoli-sourcemap-concat/concat-with-maps.js:52:12)
    at /my-code/test-project/node_modules/ember-cli/node_modules/broccoli-sourcemap-concat/node_modules/broccoli-caching-writer/index.js:92:34
    at lib$rsvp$$internal$$tryCatch (/my-code/test-project/node_modules/ember-cli/node_modules/rsvp/dist/rsvp.js:489:16)
    at lib$rsvp$$internal$$invokeCallback (/my-code/test-project/node_modules/ember-cli/node_modules/rsvp/dist/rsvp.js:501:17)
    at lib$rsvp$$internal$$publish (/my-code/test-project/node_modules/ember-cli/node_modules/rsvp/dist/rsvp.js:472:11)
    at lib$rsvp$asap$$flush (/my-code/test-project/node_modules/ember-cli/node_modules/rsvp/dist/rsvp.js:1290:9)
    at process._tickCallback (node.js:355:11)

Removing the addon restores normal functionality to the ember project.

Thanks @gte451f. I'm guessing this is at least partially due to the transition from using Brocfile.js to ember-cli-build.js as described here https://github.com/ember-cli/ember-cli/blob/master/TRANSITION.md
Will take a look soon.

I'm also getting this error on Ember CLI 0.2.5. Any ideas?

@caseyjenks @gte451f I have a beta tag that I think fixes this https://github.com/tylergaw/ember-cli-chartist/tree/v0.3.0-beta1 wanna give that a try?

I am new to this so I might have done this wrong but I'm still getting the same error.
Steps I followed to try w/ Beta release:

Install Package
npm install ember-cli-chartist --save-dev

Update package.json w/ beta release
"ember-cli-chartist": "^0.2.9",
becomes...
"ember-cli-chartist": "v0.3.0-beta1",

Install new package
npm install;

Run Ember
ember server

Ember CLI Release 1.13.1

@gte451f Sorry, I missed updated the install docs. In the Beta version, we're now installing the chartist dep in the blueprint here https://github.com/tylergaw/ember-cli-chartist/blob/v0.3.0-beta1/blueprints/ember-cli-chartist/index.js#L11
I found this out recently too, but that means ember-cli-chartist needs to be installed with ember install ember-cli-chartist instead of npm install. At least for the initial install. If not, the chartist dependency is not installed.
This feels a bit clunky still, but give that a try and see if it works for you.

Well, I removed the addon from package.json and re-installed via:
install ember-cli-chartist
...and everything just worked.
I did not need to test out the beta, 0.2.9 worked just like that!


Cannot run all tests sequentially in protracto

Hi there!
Thanks for submitting an issue to Protractor.
To help us help you better, please do the following before submitting an issue:

Review the questions section of CONTRIBUTING.md.
Make sure you are not asking a usage or debugging question. If you are, use StackOverflow, Google Group discussion list, or Gitter to get help.
Provide a minimally reduced test case. This makes it much more likely that your bug will be fixed. Protractor has a test Angular application available at http://www.protractortest.org/testapp which you can use for the reproducible test case.
Fill in the information that corresponds to your type of issue below.
Delete this intro and any unrelated text :smile: (if you do not we'll assume you haven't read these instructions and automatically close the issue.)

Bug report

Node Version: ``
Protractor Version: ``
Angular Version: ``
Browser(s): ``
Operating System and Version ``
Your protractor configuration file
A relevant example test
Output from running the test
Steps to reproduce the bug
The URL you are running your tests against (if relevant)

Feature Request

Reasons for adopting new feature
Is this a breaking change? (How will this affect existing functionality)

I tried running the protractor config using the above example but still it doesn't seem like its running sequentially.
The above example will only wait until the login is successful and it triggers the second test without waiting for the first test to finish. I need the first test to run completely and finish and then the second test should start to run.but the first and second files are in different folders.

My protractor config file with specs listed like this
specs: [
'tests1/test1_spec.js',
'tests2/test2_spec.js',
'tests3/test3_spec.js'
]

Hi there,
Please follow the Issue Template when opening new issues. Otherwise it's unclear what the issue is and how we can actually help.
If you don't update your question I'll close it in a day or two. Thanks!

I tried running the protractor config using the above example but still it doesn't seem like its running sequentially.
The above example will only wait until the login is successful and it triggers the second test without waiting for the first test to finish. I need the first test to run completely and finish and then the second test should start to run.but the first and second files are in different folders.
My protractor config file with specs listed like this
suites: [
workshop_categories: [
appconfig.WORKSHOP_PATH+'authenticate/_spec.js',
appconfig.WORKSHOP_PATH+'categories/_spec.js',
appconfig.WORKSHOP_PATH+'Associate_Categories/_spec.js',
appconfig.WORKSHOP_PATH+'sign_out/_spec.js'
],
workshop_actionbar: [
appconfig.WORKSHOP_PATH+'authenticate/_spec.js',
appconfig.WORKSHOP_PATH+'actionbar_timing/_spec.js',
appconfig.WORKSHOP_PATH+'sign_out/*_spec.js'
],
]
In this, the authenticate folder is not triggered at all after the comple execution of workshop_categories(suite).

Node Version: 4.4.7
Protractor Version: 2.5.1
Browser(s): Chrome v58.0.3029.96 (64-bit)
Operating System and Version windows 7

I'm sorry but i'm still struggling to understand your issue; you are saying that the specs in
appconfig.WORKSHOP_PATH+'authenticate/_spec.js', do not finish before running the specs in appconfig.WORKSHOP_PATH+'categories/_spec.js', ?
Or are you saying that the workshop_categories do not finish before the workshop_actionbar tests?

Yeah, u are right,  the specs in,
appconfig.WORKSHOP_PATH+'authenticate/_spec.js', are not at all triggered, before running the specs in appconfig.WORKSHOP_PATH+'categories/_spec.js', ?

Can you share your configuration file?  I don't see anything wrong with your suite configuration from that config. Can you create an mvce? You can use https://github.com/NickTomlin/protractor-mcve as a starting point if it's helpful.
What we need to do is isolate the actual issue by breaking down the problem into smaller and smaller parts. From this process, i'm starting to believe that this is an issue with your application and setup and not Protractor. Let's attempt to dig into this a little further here, but if that is unproductive I'm going to move things to StackOveflow.


No unknown target is found when the trained model is used for inference

Hi, Thanks for your excellent work!
When I used the weights downloaded from google drive to infer my custom dataset, I visualized the detection results and found that the first 20 categories of objects were detected, but the next 60 categories of objects and unknown objects were not detected.
I'm not sure if my config file is wrong or if the energy file is missing.
Here's my config file (I have placed my custom dataset in the corresponding directory of voc_coco_2007_test):
OUTPUT_DIR: "./output/t1"
MODEL:
  META_ARCHITECTURE: "GeneralizedRCNN"
  WEIGHTS: "output/t1/model_final.pth"
  RPN:
    PRE_NMS_TOPK_TEST: 6000
    POST_NMS_TOPK_TEST: 1000
  ROI_HEADS:
    #    NUM_CLASSES: 2 # 0~79 Known class; 80 -> Unknown; 81 -> Background.
    NUM_CLASSES: 81
    NAME: "Res5ROIHeads"
    POSITIVE_FRACTION: 0.25
    NMS_THRESH_TEST: 0.2
    SCORE_THRESH_TEST: 0.7
  MASK_ON: False
  RESNETS:
    DEPTH: 50
DATASETS:
  TRAIN: ("voc_coco_2007_test",)
  TEST: ("voc_coco_2007_test",)
SOLVER:
  IMS_PER_BATCH: 6
  BASE_LR: 0.01
  STEPS: (12000, 16000)
  MAX_ITER: 500
TEST:
  EXPECTED_RESULTS: []
VERSION: 2
OWOD:
  PREV_INTRODUCED_CLS: 20
  CUR_INTRODUCED_CLS: 0
  COMPUTE_ENERGY: False
  ENABLE_CLUSTERING: False
  ENERGY_SAVE_PATH: 'energy'
  SKIP_TRAINING_WHILE_EVAL: True
  ENABLE_CLUSTERING: False
  TEMPERATURE: 1.5
  FEATURE_STORE_SAVE_PATH: 'feature_store'
CUDNN_BENCHMARK: True

And my run command:
python tools/train_net.py --num-gpus 1 --config-file ./output/t1/t1_test.yaml --eval-only

When I run the command, the system displays the following message:
Energy distribution is not found at ./output/t1/energy_dist_20.pkl

Thanks!

You need to first generate the Energy distribution using the *_val yaml files.


Max Profit with K Trasction Leetcode Hard Question

Max Profit with K Trasction Leetcode Hard Question

LGTM
Thx for the question Please contribute More
Happy Learning :)


Clicking the view button in crawls page shows popup "do you want this crawl to be deleted" on safari

Note that this happens only on safari. I tried this on chrome and it worked fine.
On clicking the view button on the crawls page, instead of going to the crawls page, it shows the popup that is supposed to come on clicking the delete button.

Thanks for the report, we really appreciate it!  Please let us know if you find any others like it.

That's really bizarre. I'll take a look into it right now.


Install with composer & php 8.0.3

[ ] I've read the guidelines for Contributing to YOURLS Projects
[ ] This request isn't a duplicate of an existing request
[ ] This is not a personal support request, that should be posted on discussions community

Summary

I be glad if YOURLS work with php 8
Motivation

To work fine with php 8
Is your feature request related to a problem? Please describe.

When I try install it on GNU/Linux Ubuntu 20.04, Apache 2.4.41, Mariadb 10.5 and PHP 8.3 (with composer version 2.0.12) I get the following error:
Installing dependencies from lock file (including require-dev)
Verifying lock file contents can be installed on current platform.
Your lock file does not contain a compatible set of packages. Please run composer update.

  Problem 1
    - spatie/array-to-xml is locked to version 2.15.0 and an update of this package was not requested.
    - spatie/array-to-xml 2.15.0 requires php ^7.2 -> your php version (8.0.3) does not satisfy that requirement.

Describe the solution you'd like

I remove the composer.lock and after composer install works fine.
Describe alternatives you've considered

updating spatie/array-to-xml for php8
Additional context

thanks for your application :)

After updating to the latest version, I got the following error message. Does the current php version no longer work?
Fatal error: Composer detected issues in your platform: Your Composer dependencies require a PHP version ">= 7.2.0". You are running 7.0.7. in /home/urlykr/public_html/includes/vendor/composer/platform_check.php on line 24

How did you "update to the latest version" ?

Dear ozh,
I've Unzip the latest files
Upload files to the server, overwriting your existing ones
Is there anything else should I do more?

Sorry, I misread your previous comment. This is expected : as announced, 1.8 drops support for PHP prior to 7.2

You mean, Unless I upgrade the php version, I can't use 1.8 or higher.

@replypark yes

Thank you for your kind tips.
Have a nice day.


JSDoc @type does not attach to for loop variable declarations

Bug Report
From what I could tell, this is a common pattern with JSDoc used to properly document the type of a loop variable. It does not seem to work in VSCode.
🔎 Search Terms
jsdoc for loop type
🕗 Version & Regression Information

This is the behavior in every version I tried, and I reviewed the FAQ for entries about JSDoc.

⏯ Playground Link
https://www.typescriptlang.org/play?#code/GYewTgBAFA9AVHCABALgTwA4FMIG8ASAKgLIAyAogDZYC2WAdigL4RwwQDGI9AzihDhDAIAExAcArnUYA6AEYgRaGQEcJWMGgDKWahxTgAgpUpQA5DJRY+ZgJS28AKAguBMvmmoyRASx4ZKAEM0CABeCDN6biwzAG5HJiA
💻 Code
for (/** @type {HTMLElement} */ const e of document.body.querySelectorAll('.test')) {
    e.style.display = 'none';
}

🙁 Actual behavior
e is typed as Element
🙂 Expected behavior
e should be typed as HTMLElement, as I have overridden it via @type.
As an example, if I were to change the code to this:
/** @type {HTMLElement} */ const e = document.body.querySelectorAll('.test');

VSCode correctly picks up that I'm wishing to override e as an HTMLElement (and complains about missing interface implementations, but, that's not the issue here - in fact that's expected!)

Also, for those who need a workaround, this works using TypeScript's JS "cast" feature, but I find it much harder to work with:
for (const e of /** @type {NodeListOf<HTMLElement>} */ (document.body.querySelectorAll('.test')))

@Paril do you have an idea of how common this is?
This is a non-trivial feature (though probably easy-ish) because Typescript explicitly disallows type annotations here. All the type checking would have to be added from scratch.

In my usual code bases from Definitely Typed dependencies and our user tests + dependencies, I grepped for.+@type and didn't find any existing uses.
I did find 3 uses of the cast syntax in chrome-devtools-frontend and 1 in webpack, but those projects use Closure or Typescript.
Looking at this strictly as a new feature, I'm inclined to reference Closure and Typescript for precedent and say that allowing annotations here isn't that useful.

I'm not sure about how common it is, I shouldn't have used the word "common pattern" in the initial post as I'm not actually certain of its usage but it was a pattern I saw as an accepted answer on how to do this sort of thing with JSDoc. It seems like it should be allowed; there's no other way to annotate a variable in a for/foreach statement otherwise, which seems like an odd missing feature.
What's the reason for explicitly disallowing type annotations here? I'm just curious as to why type declarations in any other context are okay to annotate but loop ones aren't, unless you move the declaration outside of the statement itself. I guess it just felt like it should "just work" because it immediately precedes the declaration.

Actually, I think I found the TypeScript equivalent of this one, and I understand the reasoning for it being not accepted:
https://github.com/microsoft/TypeScript/issues/25605
Given how annotating this inline is essentially the JS version of this issue, I see the precedent for closing this one.

@Paril we were reading the same thing! This is still a separate issue from #3500, but I'd say it depends on that one. Let's leave this open, and once #3500 is fixed, it will be a small amount of work to make sure it works in JS too.

Sounds good to me!

Also, I just realized that this syntax is fully valid in VSCode:
const substring = (/** @type {string} */ str, /** @type {number} */ start, end = str.length) => str.substring(start, end);

This isn't a huge thing, but, I think it's a good example of existing code that is similar to what I'm hoping for (although this is a parameter, not a local variable, but hey, same difference)

@Paril

Also, for those who need a workaround, this works using TypeScript's JS "cast" feature, but I find it much harder to work with:
for (const e of /** @type {NodeListOf} */ (document.body.querySelectorAll('.test')))

For some reason this didn't work (or doesn't work anymore?), for those wondering, another workaround I found it to "cast" it beforehand by assigning it
/** @type {NodeListOf<HTMLElement>} */ const test = document.body.querySelectorAll('.test');
for (const e of test)


README Still References Original Project

HTML Render difference still using swagger-diff's css.
I figure if you're going to try to pass off the original project's work as your own without giving it any credit, you may as well go ahead and make sure you host the css yourself as well.
https://github.com/Sayi/swagger-diff
Seriously lol... not even in thanks section of the README... smh

Yes html render use swagger-diff css. The css theme used in https://github.com/Sayi/swagger-diff was good, so why should i not use it ?
Please, look at the code before saying i steal the work of someone. I only used the project swagger-diff as source of inspiration to make this tool, but the project content is clearly different.
But you're right, i should add it in the README.

Fixed with commit 2128208


Flutter google_mobile_ads only not working in android 7 API level 25.

@SourabhMahale98
Transfer from: https://github.com/googleads/googleads-mobile-flutter/discussions/162 (now inaccessible)
As I wanted to show native ads in my app I use this package google_mobile_ads. It works very well in all devices except one android 7 with API 25. only for that device, it will show error like
AdWidget requires Ad.load to be called before AdWidget is inserted into the tree.

Hi @bparrishMines, Is this issue still valid as of today using the latest ads plugin 0.13.4 ? If yes please share the output of
flutter doctor -v .
Thanks

@maheshmnj I believe this is no longer valid. The AdWidget error should be expected behavior.
@SourabhMahale98 Feel free to reopen/leave a comment if you are still experiencing this error with the latest version of the plugin. And also provide a small example of how to reproduce the error.

Thanks for the confirmation.

This issue seems to happen again on iPhone 12 Pro (iOS 14.8.1):
AdWidget requires Ad.load to be called before AdWidget is inserted into the tree. Error thrown Instance of 'ErrorDescription'.
flutter doctor:
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, 2.5.3, on macOS 12.0.1 21A559 darwin-x64, locale en-IL)
[✓] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
[✓] Xcode - develop for iOS and macOS
[✓] Chrome - develop for the web
[✓] Android Studio (version 2020.3)
[✓] VS Code (version 1.63.1)
[✓] Connected device (2 available)

• No issues found!

google_mobile_ads: 1.0.1

Hi @CripyIce, Could you please file a separate issue with all the details requested in the issue template.


Update changelog for 1.13

[x] Getters for StreamController + typedefs: onPause, onListen, onResume, onCancel
[x] Added base64 to dart:convert

dart:developer

[x] ServiceExtensionResponse: #24793
[x] Timeline added: #24792

dart:io

[x] HttpClient added optional SecurityContext param: #24671
[x] changes to RawSecureServerSocket: certificateName -> context: #24671
[x] new SecurityContext class: #24671

dart:core

[x] resource is deprecated, scheduled for removal

dart:isolate

[x] changes to spawnUri: added environment argument

other

[x] VM's legacy debug protocol is deprecated, scheduled for removal

@kevmoo, do you have the change lists that these came from? We need to get the submitters to update the changelog.

@mit-mit sadly, no.

Opened tracking bugs for the dart:developer changes.

marking dart:developer changes complete, pick request is #24811

all tasks completed, marking complete for 1.13.


#4 - Include eslint via airbnb stlye guide

Issue #4

Please have also a look at the Travis-CI log output of the first commit: https://travis-ci.org/OrgApp/node-persistence-mongodb/builds/168047388

What is your oppinion on trailing commas?
One argument for the use of trailing commas is to keep git diffs cleaner (quote), as trailing commas are allowed with EcmaScript5 (quote).
But one argument against the application of trailing commas would be backwards compatibility, especially for code that should later also run on legacy browsers.

I think it is better not to use trailing commas because it is a cleaner style and if there is a comma you  do not know if there is a element missing (and accidently deleted) or not. Withoout that comma it is clear that this is the last element.


Promoted stories block

To-dos
Build a Promoted Stories element (Figma link)

new card design
horizontal scroll with designated left/right buttons
wobbly bottom

Placement:

all Stories except comics and galleries
after 5 paragraphs

GA4:

set up click event

Oops sorry did I create #11425 instead of using this one? Could this one be closed or is it a different piece?

Yes, will close this. I set it up initially but don't think we need it anymore!

Yes, will close this. I set it up initially but don't think we need it anymore!


Add HAVA Coin

Description
Adding token: HAVA from Injective
Checklist

Adding Assets

If adding a new asset, please ensure the following:

[x] Asset is registered to the Cosmos Chain Registry.

The description and/or extended_description of the asset in the Chain Registry is informative.

[x] Add asset to bottom of zone_assets.json.

The IBC channel referenced in path must be registered to the Chain Registry.
osmosis_unlisted defaults to true (until the respesentation and transferring of the new asset has been validated)
Note that it is recommended to include an X (fka, Twitter) profile URL with each asset.

On-chain liquidity
For each new asset, please provide the plan for on-chain liquidity of the asset (See: Pool Setup Guilde.): (choose one)

[ ] Ready -- A liquidity pool(s) has been created and has liquidity. The pool ID is: ______
[ ] Soon -- A pool will be created soon. Do not merge this PR until liquidity is ready.

[ ] (optional) A preview of the Osmosis Zone app with the new asset added is requested for creating the pool. (Note: This method is not recommended because the Supercharged Liquidity pool type cannot be created via the Osmosis Zone app)

[x] StreamSwap -- The token is, or will be, going through a StreamSwap stream; thus, the token should be listed without requiring on-chain liquidity. A Pool ID will be provided in this PR following the stream's completion once the team has had a chance to create a pool using the earned funds. The StreamSwap Bootstrapping phase already began:  April 10, 2024 https://app.streamswap.io/stream/OSMO/10

IBC transfers to/fro Injective do not work on Osmosis, so we need to override Deposit/Withdraw to redirect to TFM.

Thanks for merging.

IBC transfers to/fro Injective do not work on Osmosis, so we need to override Deposit/Withdraw to redirect to TFM.
Thanks for adjusting this. By the way, I noticed some other assets in the list that do not have this override:

https://github.com/osmosis-labs/assetlists/blob/8737fc3cc7e32073a045b35dc55cab15acb833eb/osmosis-1/osmosis.zone_assets.json#L2467
https://github.com/osmosis-labs/assetlists/blob/8737fc3cc7e32073a045b35dc55cab15acb833eb/osmosis-1/osmosis.zone_assets.json#L2646


Handle an application root directory

Sometimes RuboCop's behaviour depends on files that are other than analysis target file.
For example:

Gemfile.lock #5473
config/environments/ #5455
.ruby-version

RuboCop searches .ruby-version recursively. #5326

Probably we have other dependencies.

The dependencies maybe does not work well in a child directory. For example:
$ cd app_root_dir/
$ rubocop # It works.

$ cd child_dir/
$ rubocop # It may be broken something.

So, I think RuboCop need to handle an application root directory.

Expected behavior
RuboCop finds the application root directory that is analysis target. Maybe RuboCop can decide the root directory if the directory has .rubocop.yml.
And RuboCop finds files that are necessary for analysis with the directory.
I guess we should not change the finding .ruby-version behaviour. Because it works well, and it is same behaviour as rbenv.
Actual behavior
RuboCop does not know the root directory, so it confuses in a child directory sometimes.
Steps to reproduce the problem
N/A
RuboCop version
master

I face a similar issue as well.
If I don't have a Gemfile.lock file, rubocop won't run.
$ rm Gemfile.lock
16:24:50 tonyc@greenapple:~/testr  ruby-2.5.5
$ rubocop
Traceback (most recent call last):
	9: from /Users/tonyc/.rvm/rubies/ruby-2.5.5/bin/ruby_executable_hooks:24:in `<main>'
	8: from /Users/tonyc/.rvm/rubies/ruby-2.5.5/bin/ruby_executable_hooks:24:in `eval'
	7: from /Users/tonyc/.rvm/rubies/ruby-2.5.5/bin/rubocop:23:in `<main>'
	6: from /Users/tonyc/.rvm/rubies/ruby-2.5.5/bin/rubocop:23:in `load'
	5: from /Users/tonyc/.rvm/rubies/ruby-2.5.5/lib/ruby/gems/2.5.0/gems/rubocop-0.67.2/exe/rubocop:13:in `<top (required)>'
	4: from /Users/tonyc/.rvm/rubies/ruby-2.5.5/lib/ruby/site_ruby/2.5.0/bundler.rb:430:in `read_file'
	3: from /Users/tonyc/.rvm/rubies/ruby-2.5.5/lib/ruby/site_ruby/2.5.0/bundler/shared_helpers.rb:118:in `filesystem_access'
	2: from /Users/tonyc/.rvm/rubies/ruby-2.5.5/lib/ruby/site_ruby/2.5.0/bundler.rb:431:in `block in read_file'
	1: from /Users/tonyc/.rvm/rubies/ruby-2.5.5/lib/ruby/site_ruby/2.5.0/bundler.rb:431:in `open'
/Users/tonyc/.rvm/rubies/ruby-2.5.5/lib/ruby/site_ruby/2.5.0/bundler.rb:431:in `initialize': No such file or directory @ rb_sysopen - /Users/tonyc/testr/Gemfile.lock (Errno::ENOENT)

I need a dummy Gemfile.lock file to run any rubocop command which I can create using "bundle init; bundle lock"
IMHO, rubocop should handle the missing Gemfile.lock more gracefully by not displaying a stack trace but rather say it's not able to infer the ruby-version or whatever the reason it's trying to look for the Gemfile.lock file.
Couldn't you get the ruby version using "ruby --version"?
According to https://github.com/rubocop-hq/rubocop/issues/5473, it seems it's needed for the rail development but there are other use cases that doesn't have rail installed or I just wanted to use rubocop to check my Rakefile or *.rake which is not related to rails


Add user accent_color and banner fields

Add the new accent_color and banner fields. accent_color is an integer representation of the RGB color while banner is a hash of the user's banner image. These are mutually exclusive.
This is documented in the API docs: https://github.com/discord/discord-api-docs/pull/3448
Closes #1126.

If these are mutually exclusive, is it too far out for Twilight to make an enum represententing them as such instead of two options, which technically could allow for illegal states?

If these are mutually exclusive, is it too far out for Twilight to make an enum represententing them as such instead of two options, which technically could allow for illegal states?

I was wrong and they aren't exclusive:
{
  "id": "640434332485287936",
  "username": "zeyla",
  "avatar": "a262fa99ab1d42f16751c67886da5640",
  "discriminator": "4443",
  "public_flags": 0,
  "banner": "ce98ed5651d0add1e3ff98782095a7dd",
  "banner_color": "#17191c",
  "accent_color": 1513756
}


Minor update to ADB recipe

Purpose
Closes #38

Updating README.md for Azure Databricks with some details about public accessibility of DBFS Storage account.

Does this introduce a breaking change?

[ ] Yes
[x] No

Pull Request Type
What kind of change does this Pull Request introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[x] Documentation content changes
[ ] Other... Please describe:

How to Test

Get the code

git clone [repo-address]
cd [repo-name]
git checkout [branch-name]
npm install

Test the code

What to Check
Verify that the following are valid

...

Other Information

@mcollier , I have addressed your review comments, please have a look.


Adding ngtcp2 and openssl as submodules

I added ngtcp2 and openssl as submodules and created a build script that creates the files outside the directory under build folder. You are free to reject the pull, but I humbly request you to explain the reasons.

Thanks for your contribution, I had thought about making ngtcp2 a submodule myself but never quite got around to it. I've got some feedback:

The INSTALL file update doesn't change anything materially, it's just the autoconf boilerplate being copied in so I don't think that needs to
I'd like the commits squashed down to remove unnecessary commits which otherwise pollute the commit log. 95358de and e8f55bf are prime examples of this - they just add and then immediately delete the same, and we don't need that
I'd also argue that the build_nghq.sh isn't really necessary. I'd much rather the git submodule initialisation is done by the bootstrap script when run and then the submodules are built when you do make.

For an example, [this is how the GStreamer project does it] in their equivalent of the bootstrap script (https://cgit.freedesktop.org/gstreamer/gstreamer/tree/autogen.sh#n19).
I make a lot of use of the --prefix path when building so I can install in non-system locations, and have multiple versions sat around for A-B testing. Your script doesn't have a way of specifying this when run.
Your script also makes bad assumptions about the use of PKG_CONFIG_PATH, and overrides anything which I may have otherwise set for picking up libraries in non-default system locations. Again, if we did it all via hooks into the standard configure and make invocations, this wouldn't be a problem. The build system will then keep track of where ngtcp2 and openssl are in make variables.

Let me know what you think!

Thank you very much for your commits and I think all your commits have very valid understanding. I will make the changes as you suggested.

Hi Sam, I'm trying to compile the application on an AWS machine. The compilation goes fine, but as soon as I run the multicast-receiver application, it generates a segmentation fault. Can you give any idea about why this is happening?

I've never tried building it on AWS. Unfortunately, without a stack trace it's difficult to say why you're seeing that segmentation fault. In addition, it's probably not a discussion that should be had on this pull request. If you can reproduce it and get a backtrace, then please raise a new issue on the project and I can look at it there.


fix #5321 don't throw error when node gets relocated

What kind of change does this PR introduce? (check at least one)

[x] Bugfix
[x] Feature
[ ] Code style update
[ ] Refactor
[ ] Build-related changes
[ ] Other, please describe:

Does this PR introduce a breaking change? (check one)

[ ] Yes
[x] No

If yes, please describe the impact and migration path for existing applications:
The PR fulfills these requirements:

[x] It's submitted to the dev branch for v2.x (or to a previous version branch), not the master branch
[x] When resolving a specific issue, it's referenced in the PR's title (e.g. fix #xxx[,#xxx], where "xxx" is the issue number)
[x] All tests are passing: https://github.com/vuejs/vue/blob/dev/.github/CONTRIBUTING.md#development-setup
[x] New/updated tests are included

If adding a new feature, the PR's description includes:

[ ] A convincing reason for adding this feature (to avoid wasting your time, it's best to open a suggestion issue first and wait for approval before working on it)

Other information:
This PR fixes https://github.com/vuejs/vue/issues/5321
Reproduction link: https://jsfiddle.net/rstoenescu/uqsath1t/2/ (follow instructions in the Github issue)

Referencing issue for this PR: https://github.com/vuejs/vue/issues/5321

Redone with requested changes. Thanks!

Thank you for merging!


[Fix #6880] Fix RuboCop::Options#parse for .rubocop file parsing

Fixes #6880

After this commit, RuboCop parses .rubocop(even if content is like --format progress) correctly.
More natural test case of .rubocop file content.

Before submitting the PR make sure the following are checked:

[x] Wrote good commit messages.
[x] Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[ ] Added tests.
[x] Added an entry to the Changelog if the new code introduces user-observable changes. See changelog entry format.
[x] The PR relates to only one subject with a clear title
and description in grammatically correct, complete sentences.
[x] Run bundle exec rake default. It executes all tests and RuboCop for itself, and generates the documentation.

Thanks!


Promise3 + catching an error on Promise

Not sure what i'm doing wrong but i returned a promise in one of my functions a Promise because i don't need to give something back when it succeeds. But now when i try to catch the error it always gives me this: "Cannot call value of non-function type 'ErrorType?'". I'm currently working on the last Xcode (7.1) and this is what i am trying to do:
myFunction().error() {
        error -> Void in

        print("My error: \(error.description)")
    }

Does anyone know what i am doing wrong?
kind regards,
Daan

Don't use () it is confusing the swift compiler apparently.

So just .error {

In fact it is necessary to provide a parameter to the closure or the Swift compiler is confused with our error property:
.error { error in
This way it works with parenthesis too:
.error() { error in
Swift compiler bug.

I'm also seeing this confusing compiler error, and I can't make sense of what is confusing the Swift compiler so badly, but your suggestions (remove the ()) didn't work.  However, after much playing around, I got it working...
With a func doThing() -> Promise<Void> and another func handleError(e: NSError), I tried:
      firstly {
          doThing()
      }.then {
          NSLog("DONE!")
      }.error { (e) -> () in
//      ^^^^^  Cannot call value of non-function type 'ErrorType?'
          handleError(e)
      }

But adding a second line to the method changed the error.  Any second line would do, NSLog("error") or return were what I tried.  The following gave a different error:
      firstly {
          doThing()
      }.then {
          NSLog("DONE!")
      }.error { (e) -> () in
          handleError(e)
//                    ^ Cannot convert value of type 'ErrorType' to expected
          return
      }

Which was easily enough fixed, and then I could remove the superfluous second line too:
      firstly {
          doThing()
      }.then {
          NSLog("DONE!")
      }.error { (e) -> () in
          handleError(e as NSError)
      }
// Compiles fine

I hope this helps someone else who gets here via google. :smiley:

It did help at least one person. Thank you nevans!

Make that two.  Been pulling my hair out on this.  Thanks @nevans !!


Alert if the topic is not sent as bytes

Prevents people from doing the following:
from kafka import KafkaClient, SimpleProducer
kafka = KafkaClient(hosts=["localhost:9092"])
producer = SimpleProducer(kafka)
producer.send_messages("test", b"some message")

Which would give this error:
Traceback (most recent call last):
  File "test_kafka.py", line 7, in <module>
    producer.send_messages("test", b"some message")
  File "/home/sontek/venvs/notebooks3/src/kafka-python/kafka/producer/simple.py", line 71, in send_messages
    partition = self._next_partition(topic)
  File "/home/sontek/venvs/notebooks3/src/kafka-python/kafka/producer/simple.py", line 58, in _next_partition
    self.client.load_metadata_for_topics(topic)
  File "/home/sontek/venvs/notebooks3/src/kafka-python/kafka/client.py", line 309, in load_metadata_for_topics
    resp = self.send_metadata_request(topics)
  File "/home/sontek/venvs/notebooks3/src/kafka-python/kafka/client.py", line 378, in send_metadata_request
    return self._send_broker_unaware_request(payloads, encoder, decoder)
  File "/home/sontek/venvs/notebooks3/src/kafka-python/kafka/client.py", line 126, in _send_broker_unaware_request
    raise KafkaUnavailableError("All servers failed to process request")
kafka.common.KafkaUnavailableError: All servers failed to process request

Instead of raising a TypeError I just encode to bytes for them.
deprecated
It now gives this error:
Traceback (most recent call last):
  File "test_kafka.py", line 7, in <module>
    producer.send_messages("test", b"some message")
  File "/home/sontek/venvs/notebooks3/src/kafka-python/kafka/producer/simple.py", line 73, in send_messages
    raise TypeError("topic must be type bytes")
TypeError: topic must be type bytes

cc @mumrah Is this something you are interested in?  Anything I can do to get this merged?


[Bilibili] Add 8k support

Please follow the guide below

You will be asked some questions, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your pull request (like that [x])
Use Preview tab to see how your pull request will actually look like

Before submitting a pull request make sure you have:

[x] At least skimmed through contributing guidelines including yt-dlp coding conventions
[x] Searched the bugtracker for similar pull requests
[x] Checked the code with flake8

In order to be accepted and merged into yt-dlp each piece of code must be in public domain or released under Unlicense. Check one of the following options:

[x] I am the original author of this code and I am willing to release it under Unlicense
[ ] I am not the original author of this code but it is in public domain or released under Unlicense (provide reliable evidence)

What is the purpose of your pull request?

[ ] Bug fix
[x] Improvement
[ ] New extractor
[ ] New feature

Description of your pull request and other information
Closes #1898 Closes #1819 (hopefully)

Closes https://github.com/yt-dlp/yt-dlp/issues/1819 too, right?

Closes #1819 too, right?

Closes #1819 too, right?

Closes #1819 too, right?

It seems so

I tried your branch with the following command, but I got a 403 Forbidden Error, any idea? (I'm a premier user of the site)
./yt-dlp.sh "https://www.bilibili.com/video/BV1SA411g73f?from=search&seid=10422313686506169122&spm_id_from=333.337.0.0" --cookies-from-browser Chrome -f 2

no. I think format 32 was downloaded successfully but format 2 always fails.
./yt-dlp.sh https://www.bilibili.com/video/BV1SA411g73f
[BiliBili] 1SA411g73f: Grabbing original ID via API
[BiliBili] 333255660: Downloading webpage
[BiliBili] 333255660: Extracting videos in anthology
[BiliBili] Downloading anthology 333255660 - add --no-playlist to just download video
[download] Downloading playlist: 再见宣言 (グッバイ宣言)／rie
[BiliBili] playlist 再见宣言 (グッバイ宣言)／rie: Collected 1 videos; downloading 1 of them
[download] Downloading video 1 of 1
[BiliBili] 1SA411g73f: Grabbing original ID via API
[BiliBili] 333255660: Downloading webpage
[BiliBili] 333255660: Extracting videos in anthology
[BiliBili] 333255660: Downloading tags
[info] 333255660_part1: Downloading 1 format(s): 32+2
[download] Destination: 再见宣言 (グッバイ宣言)／rie [333255660_part1].f32.mp4
[download] 100% of 6.38MiB in 00:00
ERROR: unable to download video data: HTTP Error 403: Forbidden
[download] Finished downloading playlist: 再见宣言 (グッバイ宣言)／rie

Hmmm... intresting
try running it with --check-formats flag.


Create .npmignore

Added .npmignore to exclude examples in the npm package while keeping it in the git repo

I know /example is not used at runtime, but the filesize is negligible, and it's nice to be able to npm install menubar && cd node_modules/menubar/example to figure out how it works.


Add Event Controller

How to categorize this PR?
/area usability
/kind enhancement
/priority normal
What this PR does / why we need it:
Add event controller
The goal of the event controller is to persist Shoot-related events longer for debugging reasons.
The controller deletes all non-Shoot events after the configured ttlNonShootEvents.
Shoot-related events are not touched by the controller and only eventually get deleted after the configured ttl in the kube-api-server configuration.
Considered alternatives to this controller were:

export relevant events from etcd to another persistent store.
extend the global ttl in the api-server for all events

While option 1. would be preferable it imposes dificulties as with the Control Plane Migration the shoot CP(with the events) can move between Seed worker nodes.
See also issue #2390
Also:

Added hibernation events and more fine-grained maintenance events for shoots.
Replaced some Shoot related events with logs to decrease the volume of shoot events written.

Which issue(s) this PR fixes:
Fixes #2390
Special notes for your reviewer:
Very open for any suggestion as well regarding code/tests style.
Release note:
Opt-in for Event Controller which extends the live-span of events regarding Shoot clusters

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

Can you please sign the CLA and check make verify?

/assign

From my perspective the code looks good after addressing the issues of the other reviewers.
Will test it out after the suggestions are applied.

/status author-action

/unassign

@rfranzke I already had the documentation included. But now I also added the ConcurrentSyncs flag as it became a required value.

/unassign


Selftest to be schedule before release / periodically

To be able to run how=full in Beaker before the release (or nighly).
Expected usage $ tmt t export --fmf-id . | wow --fmf-id - --bare --keyvalue=HVM=1 x86_64 Fedora-34
It's very dirty usage of beakerlib (it doesn't work well for non-root usage, right :)
TODO

[x] Allow custom checkout (HEAD vs commit)
[x] Nicer modification of plan/provision prepare steps
[x] Comments, nicer bash / beakerlib code
[x] Disable test by default

Note to self - scheduled as $ tmt t export --fmf-id . | wow --fmf-id - --bare --keyvalue=HVM=1 x86_64 Fedora-35  --environment external=true. Lets see what happens

/packit test

Note to self - scheduled as $ tmt t export --fmf-id . | wow --fmf-id - --bare --keyvalue=HVM=1 x86_64 Fedora-35 --environment external=true. Lets see what happens

More memory necessary so it needs wow --environment external=1 --bare --keyvalue=HVM=1 x86_64 Fedora-35 --hostrequire 'memory >= 6144'

/packit test

Needs to be rebased on the latest master to fix the lint test failing on the enabled attribute (expected change).

Hm, /tests/discover/libraries takes ages (and timeouts) ... re-fetching dnf metadata in VM and in container takes too much time.

Ready for review . In beaker we hit https://bugzilla.redhat.com/show_bug.cgi?id=2022690 so be aware of one avc.
With nested virtualization it can be executed even locally as
tmt -c external=1 run -e BRANCH=lzachar-post-release-fixes -a plans --default provision -h v -c system tests --name /tests/before-release/beaker
For Beaker it is
tmt tests export --fmf-id /tests/before-release/beaker | wow --environment external=1 --fmf-id - --bare --keyvalue=HVM=1 x86_64 Fedora-35 --hostrequire 'memory >= 6144' --taskparam BRANCH=lzachar-post-release-fixes
Without BRANCH envar the current master would be tested.

IF you will try this locally in tmt please create machine with more memory.
Funny enough test got killed when reached /plans/provision/connect. The only hints are summary: 1 error and Command returned '255' in log.txt

Hopefully last changes - test should run on centos-stream-9 as well. IMHO it is ready for merge.
How / where to document this?

Thanks for the clarifcation and the last adjustments. Rebased on the latest main and going to merge once tests pass.

/packit retest-failed


Unit Testing: errorBag is not being passed to this.errors

Versions:

VueJs: 2.5.16
Vee-Validate: 2.0.6

In my unit test, I am not receiving an injected validator instance from the root instance. This was allowing passing tests, but [Vue warn] 'Injection "$validator" not found'.
I now provide an instance of Validator to my test component
const v = new VeeValidate.Validator();
const wrapper = mount(TestComponent, {
localVue: Vue,
provide: () => ({
$validator: v
})
});
This fixes the [Vue warn] I was receiving, but now that component can't be mounted at all because the errorBag is not being passed to this.errors causing errors.has(...) to throw TypeError: Cannot read property 'has' of undefined.

Can you share the unit test code? you could take a look at the integration tests provided in this repository since it pretty much tests the same thing as you do.
I suspect that the plugin is not being installed properly.

This is working perfectly for me now. Unfortunately I can't identify a change I made.
At any rate, thank you for your attention and response. Much appreciated.


Github Packages Out of Date

Running the action triggers warnings about set-output being deprecated. Updating the actions packages in the packages.json should fix the issue. See this for more information: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/

Thanks for reporting this @AndrewKahr . We'll get this updated.


The full training data is missing

Hi, I would like to train a model for genome wide predictions and I found that the example dataset given is having a  subset of training dataset (Chr19).  Can you please share the full training dataset used for training.

Sure. See table S3 and S4 in the REPTILE paper
https://www.pnas.org/doi/full/10.1073/pnas.1618353114

Thanks for sharing the manuscript. As I can see this is raw data, can you share the preprocessed data used for training such as regions and their labels/state.

Will try but no promise since the data has been quite a few years. Also the training data will be on mm10 which probably won't help your case. Downloading raw data and reprocessing them on mm39 genome would be the best way.

Hi Thanks for the suggestion. My aim is to use trained models or train model in REPTILE to infer enhancers. I am not much aware of all types of data preprocessing. I have scripts to liftover from mm10 to mm39. Please share the data if available, I can use the preprocessed data used in the manuscript.

Ok. I think I got the training data. Will organize it a little bit before sharing. Do you read perl script (which is what I used to run training commands)?

Hi, thanks for the update. Yes I understand perl.

Also I would like to know the pretrained models provided in models directory. Are these models trained on the full genome data or only chr19. because when I try to use them to compute score I get the values only for chr19 and 0 for others chromosomes.
Thank you.

I try to use them to compute score I get the values only for chr19 and 0 for others chromosomes.
This is likely due to input files. Do you mind checking that the data of all chromosomes were used as input?

added the training and test data
BreadcrumbsREPTILE
/all_data

Thanks for sharing the training data. Does It also need bigwig files of the full genome, or it can take the data from epimark file.

It starts from epimark files.


net/tcp: implement the fast retransmit

Summary
net/tcp: implement the fast retransmit
https://tools.ietf.org/html/rfc2001#section-3
RFC2001: TCP Slow Start, Congestion Avoidance, Fast Retransmit,
         and Fast Recovery Algorithms

...

3.  Fast Retransmit
  Modifications to the congestion avoidance algorithm were proposed in
  1990 [3].  Before describing the change, realize that TCP may
  generate an immediate acknowledgment (a duplicate ACK) when an out-
  of-order segment is received (Section <IP_ADDRESS> of [1], with a note
  that one reason for doing so was for the experimental fast-
  retransmit algorithm).  This duplicate ACK should not be delayed.
  The purpose of this duplicate ACK is to let the other end know that a
  segment was received out of order, and to tell it what sequence
  number is expected.

  Since TCP does not know whether a duplicate ACK is caused by a lost
  segment or just a reordering of segments, it waits for a small number
  of duplicate ACKs to be received.  It is assumed that if there is
  just a reordering of the segments, there will be only one or two
  duplicate ACKs before the reordered segment is processed, which will
  then generate a new ACK.  If three or more duplicate ACKs are
  received in a row, it is a strong indication that a segment has been
  lost.  TCP then performs a retransmission of what appears to be the
  missing segment, without waiting for a retransmission timer to
  expire.

Change-Id: Ie2cbcecab507c3d831f74390a6a85e0c5c8e0652
Signed-off-by: chao.an<EMAIL_ADDRESS>Impact
TCP retransmit
Testing
iperf test

@antmerlino - would you please have a look?

@davids5 I don't know if @antmerlino is available, maybe @masayuki2009 could help us on it.


InventoryManager - removeItemFromPlayer Armor Issue

Describe
Attempting to remove armor using removeItemFromPlayer() has unexpected results, depending on how it is called.

Calling removeItemFromPlayer() and specifying a slot does nothing
Calling removeItemFromPlayer() and passing -1 in for the slot, with a name removes all items in inventory slots with values less than the specified item.
Ex. Player is wearing a full set of armor. Calling removeItemFromPlayer() and providing the helmet item name causes all items in the inventory to be removed, except the helmet. Calling removeItemFromPlayer() and providing the pants item name causes all items in the main inventory and boots to be removed, while pants, chest, and helmet are still equipped.

I assume there may be some relation to #406
Steps to reproduce
local player = peripheral.find("inventoryManager")

function unequipWithSlot()
	for k,v in pairs(player.getArmor()) do
		player.removeItemFromPlayer("BACK", 1, v.slot, v.name)
	end
end

function unequipWithoutSlot()
	for k,v in pairs(player.getArmor()) do
		player.removeItemFromPlayer("BACK", 1, -1, v.name)
	end
end

unequipWithSlot()  -- Does nothing
unequipWithoutSlot()  --Removes all items except helmet

Multiplayer?
Yes
Version
1.19.2-0.7.22b (Latest 1.19)
Minecraft, Forge and maybe other related mods versions
Forge 43.2.3  Minecraft 1.19.2  CC: Tweaked 1.101.1   (Direwolf20 1.19 modpack, v1.1.1)
Screenshots or Videos
No response
Crashlog/log
No response

Priority is high since I remake some systems to be more reliable which can be used for multiple peripherals at once to remove the extra inventory stuff for every peripheral which does something with inventories.


fix: Add error handling if default EA App path

We cannot install to C:\Program Files\EA Games\Titanfall2 cause Windows™
As such if install fails we check for that path and show according error message.
I would kinda also wanna add a link to wiki but AFAIK links won't render clickable so it's not really super helpful...

A plain html link probably won't work but you can open links from within a tauri app https://tauri.app/v1/api/js/shell#open

AFAIK links won't render clickable

WDYM? We have clickable links on Thunderstore mods cards, for instance.

Right now it's giving a system popup (which arguably is justified cause Northstar failing to install is kinda a big deal). But even if changed I'll still need to essentially stick the HTML into the returned string to show as a link.
At some point I need to refactor error returns to an enumerator so that we can localisation and all that but I first need to learn how to do that which will take me at least a week with everything else going on ^^"
Anyway, I'm happy to take any suggestions to improve the state in the short term :P


Is it possible to create ripple effect by using matter.js?

Is it possible to create ripple effect by using matter.js? I need some ripple effect for click, but I don't want to use other physics engine.
Here is an example of ripple effect. http://git.blivesta.com/rippler/

Are you talking about for UI purposes? There's nothing in matter.js for this type of thing, it's a rigid body engine. It doesn't look too difficult to just reproduce it yourself though if you'd rather not use that library?

Sorry for the bad example. I mean ripple effect for animation, such as http://bl.ocks.org/mbostock/9539958, and http://jsfiddle.net/esteewhy/5ht3b/6/.
As you mentioned that matter.js is not appropriate for this type of thing.
BTW, is it possible to use matter.js to simulate force direct layout, just like http://bl.ocks.org/mbostock/3231307.

All of these examples show very different things... Either way, there's no support for fluids like this (although I'd love to implement SPH if I had time!).
If you're just looking for a visual animations like this, rather than physical simulation, then d3, velocity.js, mo.js, GSAP etc. are good options. You could also look into CSS animations.

Yes I used to use d3 to implement my animations but this time I need some physical simulation which isn't covered in d3.
Anyway, thanks for your great help!


Until argument preventing scraping

Initial Check
Make sure you've checked the following.

[x] Python version is 3.5 or higher.
Python 3.6
[x] Using the latest version of Twint.
Just cloned a new copy to check.

Command
Please provide the exact command ran including the username/search so I may reproduce the issue.
python twint.py -s USA  --until 2017-05-17 -o file.csv --csv
Description of Issue
Please use as much detail as possible.
I'm trying to grab tweets up to a certain point using the --until argument, however, nothing is being returned. Occurs when using the command line and when using as a module. Adding a --since argument doesn't help. I've run the debugger included in VSCode and the URL being generated is valid and displays on Twitter, but nothing appears in Twint.
A different search 'python twint.py -s OWLMVP --until 2017-05-17 -o file.csv --csv' works to a point (--until 2018-05-14 is where it stops for me).
OS Details
Using Windows, Linux? What OS version?
PC1:
Windows 10 1803 Build 17134.48
Python 3.6.4
PC2:
Windows 10 1803 Build 17134.48
Python 3.6.5

Hi @MicahNus,
Thanks, I'll take a look at this right now

Very strange,
Seems this is an issue with Twitter itself as no results show:
https://twitter.com/search?f=tweets&vertical=news&q=usa until%3A2017-05-17&src=typd
https://twitter.com/search?f=tweets&q=OWLMVP until%3A2017-05-17&src=typd

Do let me know if you get results from either of those links

It appears i get results when authenticated, but none when not authenticated

If I open both links in an incognito browser window, I still get results.

@MicahNus quick question, what browser are you using? Chrome?

Chrome 66.0.3359.181

Strange, I'm going to wait a bit until I can properly look at this... using the until operator in Chrome incognito browser doesn't work for me at all atm
Might be an issue w/ Twitter or it might not... maybe @pielco11 can help

Okay they've blacklisted the user-agent I've been using it looks like LOL.... going to add a different one that appears to be working in a sec.

Okay, I've added a new user agent; please try now and let me know how it goes

New agent has done the trick! Cheers for the fast fix!

Awesome!

#113


”Eteni”, ”Mărtinești”, ”Vânătorești” are listed in the drop-down menu

Description:
When user writes ”Eteni”, ”Mărtinești”, ”Vânătorești” the locations appear in the drop-down menu next to the comune and the county which is a part according to the law.
Precondition:
The website is up an running.

Step 1
Write in the search bar "Eteni"

Expected results
The location appears in the drop-down menu as "Sat Eteni, SATU MARE (Odoreu)".

Step 2
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 3
Write in the search bar "Mărtinești"

Expected results
The location appears in the drop-down menu as "Sat Mărtinești, SATU MARE (Odoreu)".

Step 4
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 5
Write in the search bar "Vânătorești"

Expected results
The location appears in the drop-down menu as "Sat Vânătorești, SATU MARE (Odoreu)".


handle long response lines from server

I have a scenario where a UID Search against Gmail might return a very large response. With the current implementation of transport.ReadLine(), the client errors and exits and I'm unable to get the search results.
To deal with longer lines without having to increase our buffer size, I've changed transport.ReadLine() to read the response in chunks until it encounters a non-bufio.ErrBufferFull error or an end of line.

Why don't want to increase the buffer size? That's why I exported BufferSize in the first place. How long is the actual response? Second, can you break a single command into multiple ones to reduce the response size (e.g. limit the UID range to be searched)? If the command is too long (RFC 2683 says the approximate limit is 1,000 octets), then it should be broken up anyway. If the command is short, it's still better the limit the amount of work that the server has to do.

I think the problem of resetting the buffer size is that we may not always be able to easily predict the length of a UID Search response. To get around this, I think I'd have to preface every UID Search with a Search call to get a count first, then make several UID Searches with ranges. Do you have a better solution? This implementation seems like it'd be very noisy especially if most scenarios would work fine with just a UID Search.
As for RFC 2683, to me the language sounds like they are talking about the length of the line a client generates. What we're dealing with here is a line the server is generating. Gmail seems to happily return a line much longer than 1,000 octets and I don't see anywhere in the RFC about the client rejecting a response from the server.
To me, simply chunking the response makes for a much nicer experience with the library but if we decide not to merge this PR, I might look into highlighting the BufferSize more in the docs and the potential 'gotchas' with response sizes. It looks like at least one other person has run into this.

Yes, the 1,000 octet recommendation is for the command. One thing you should consider is that the server may be locking the mailbox while performing the search. You don't want this operating taking a very long time and possibly interfering with other clients. Also, in network protocols in general it's not a good idea to allow unbounded responses. There should be some limit to deal with misbehaving servers and protocol errors.
The solution that you proposed is one that I've used in the past and is similar to the recommendation in RFC 2683 section <IP_ADDRESS>. You should only issue commands that are guaranteed to result in responses of no more than X bytes. For example, get the UID of every thousandths message and then issue separate UID search commands for each range. This will ensure that even if all 1,000 messages match the search, the response will not exceed the buffer size. It also reduces the amount of work that the server has to do in one iteration and allows it to perform other tasks in between the searches. Finally, if this command is tied to some GUI, it allows you to begin showing results or issue additional commands before the entire search is completed.
This is basically in the same category as running LIST "" *. It's quick, easy, and works in most cases, but it's not recommended and will cause problems eventually. Breaking up the search is a bit more effort, but it results in much more robust software.

Fair enough. Thanks for your time and explanation (and this library!).


redirecting of url

Hi,
I am trying to install bookstack in windows 8, following this tutorial (https://www.youtube.com/watch?v=lSaY4lK_RjM)(WAMP Server). All done successfully. Also added virtual host in wamp server. But when I browse the url 'http://bookstack.local/login' or 'http://<IP_ADDRESS>:800' it's redirecting to a url 'https://example.com/login'. Checked in the C:\Windows\System32\Drivers\etc\hosts file also (it is  - <IP_ADDRESS>  bookstack.local).
Please help.

Hi,
Have you checked APP_URL parameter on .env file? You need to set APP_URL=bookstack.local.
Hope this helps!

Thanks for the support... It's working now...