Error: No template for dependency: TemplateArgumentDependency

I am currently using webpack version 1.5.0 and karma-webpack version 1.5.3 on a project. After release of webpack version 1.6.0, I am getting the error below when running tests.
It looks like having webpack ^1.4.0 as a peerDependencies causes npm to download the latest webpack under karma-webpack's node_modules even though webpack 1.5.3 satisfies peerDependencies requirement. Any thoughts on how to avoid these issues going further?
Error: No template for dependency: TemplateArgumentDependency
  at doDep (project/node_modules/karma-webpack/node_modules/webpack/lib/NormalModule.js:115:23)
  at Array.forEach (native)
  at doBlock (project/node_modules/karma-webpack/node_modules/webpack/lib/NormalModule.js:124:22)
  at DependenciesBlock.NormalModule.source (project/node_modules/karma-webpack/node_modules/webpack/lib/NormalModule.js:156:2)
  at Tapable.ModuleTemplate.render (project/node_modules/karma-webpack/node_modules/webpack/lib/ModuleTemplate.js:14:28)
  at Tapable.Template.renderChunkModules (project/node_modules/karma-webpack/node_modules/webpack/lib/Template.js:80:31)
  at Tapable.<anonymous> (project/node_modules/karma-webpack/node_modules/webpack/lib/MainTemplate.js:28:22)
  at Tapable.applyPlugins [as applyPluginsWaterfall] (project/node_modules/karma-webpack/node_modules/webpack/node_modules/tapable/lib/Tapable.js:37:47)
  at Tapable.MainTemplate.render (project/node_modules/karma-webpack/node_modules/webpack/lib/MainTemplate.js:112:20)
  at Tapable.createChunkAssets (project/node_modules/karma-webpack/node_modules/webpack/lib/Compilation.js:783:32)

Any thoughts?
Now getting following error with the upgrade of webpack to 1.7.0.
WARN [preprocess]: Can not load "webpack"!
  TypeError: Cannot read property 'filename' of null

This seems to be our issue as well. @nishp1 Have you found a good workaround yet? cc @sokra

I ended up upgrading webpack to latest version so that there is only version of webpack.

Just now encountered error
/Users/myuser/www/react-test/scripts/node_modules/webpack/lib/Template.js:80
                            source.add(BalancePanel.render(module, dependencyTemplates, chunk));
                                       ^
ReferenceError: BalancePanel is not defined

When BalancePanel was my file and my class, not webpacks
I commented this
//var webpack = require('webpack');

and this
//new webpack.DefinePlugin({
//  'process.env': {
//    'NODE_ENV': JSON.stringify('production')
//  }
//})

in webpack.config.js file, also deleted webpack folder from node_modules, so exactly webpack from /usr/local/bin/webpack will be used.
Yep, I assume when using production plugin I will need to use it once and not in dev, otherwise I will end up with webpack trying to insert my classes in its source code (or I'm stupid)

Addition. I also had to delete react and react-tools folders, after minification/stringification (or what it is), so they won't search for my class.

I had the same issue while using a global install of webpack and webpack-dev-server (current). With no  other changes, it just started throwing that error one morning. I removed webpack and webpack-dev-server from global node_modules, installed both again and they worked fine again. Probably something going wrong with dependencies.

Are you using webpack.optimize.DedupePlugin() plugin? Removing the latter plugin from my config solved the issue.

In addition, see https://github.com/webpack/webpack/issues/1082.

Resolving as this is no longer valid in 2.2.0 and resolved for 1.x


Determine if actions really need a HashMap<String, Option<String>> or could simplify to HashMap<String, String>

maps already return an Option<String> from get, so it's not clear what the difference between "present and None" and "not present" are.
We should probably type these as well, since the delta spec specifies maps/lists of strings, and use the defined types in action and visitor definitions.

Done


Feature/unit and integration tests

Hi @Ellon-M  👋
The following milestones were implemented in this PR

[x] Write rspec unit tests for models
[x] Write request unit tests for controllers
[x] Write integration specs for views

APPROVED! 🏆
Hi @willy836  👋
Your project is complete ✔
There is nothing else to say other than... it's time to merge it :shipit:
Congratulations! 🎉

Cheers and Happy coding!👏👏👏


@segment/analytics-node - Release Version 1.0

Hi!
I'm trying to find a way to be notified of when the @segment/analytics-node library is ready to come out of beta.
Maybe this issue can be a good location to keep track of what is remaining to get a 1.0 release out. That way anyone else who wants to use the library once it is out of beta can also subscribe to this issue and be ready to try it out when it gets closed.
If anyone has a better way to subscribe to just @segment/analytics-node releases I'm happy to do that instead.
Cheers,
Jack

Thanks @Jackman3005 for posting this. I appreciate the feedback and any feedback in general about this beta will be tremendously helpful; we've done testing in various sandboxes as well as running durability and performance tests, but any feedback from running it in production is really appreciated.
I can post here whenever we release a new version. Our next release will be after the holidays, which is mostly refactoring and some improvements to our typescript interfaces.
To answer about the timeline: our official "non-beta" release is scheduled for the end of Q1, but the API should be stable as of this message (beyond some typescript tweaks). We use the term "beta" at Segment, but this phase is actually meant to be more of a release candidate -- meaning, that if this library was used internally, we would feel good using it in our own production environment.

Released beta-9, which includes housecleaning and Typescript improvements.

Released beta.12, which fixed more issues around  Cloudflare workers / web workers.

0.0.1-beta.17 released.

Hi @silesky, that last version 0.0.1-beta.17 has a missing dependency the problem is imported from<EMAIL_ADDRESS>The issue was fixed in this commit since the dependency that was not included in the package.json was removed
But how the current published version of the node library is relaying in an old version of core the issue appears on more strict package managers as yarn PnP
Can we be waiting for the version 0.0.1 soon to be published?

I've been following README with

in my package.json "@segment/analytics-node": "^0.0.1-beta.17"
importing as import { Analytics } from '@segment/analytics-node';
this is express application, npm start, fails with

SyntaxError: Named export 'Analytics' not found. The requested module '@segment/analytics-node' is a CommonJS module, which may not support all module.exports as named exports.

node version 16.14.2
looking in @segment's own package.json, no "type": "module" there, it's a CommonJS indeed but syntax is ESM

@silesky could you advice?

Hi @silesky, that last version 0.0.1-beta.17 has a missing dependency the problem is imported from<EMAIL_ADDRESS>The issue was fixed in this commit since the dependency that was not included in the package.json was removed
But how the current published version of the node library is relaying in an old version of core the issue appears on more strict package managers as yarn PnP

Hey @viniciuspalma, The current published version of the node library uses 1.2.1-- I would advise trying to solve it on your end.

I've been following README with

in my package.json "@segment/analytics-node": "^0.0.1-beta.17"
importing as import { Analytics } from '@segment/analytics-node';
this is express application, npm start, fails with

SyntaxError: Named export 'Analytics' not found. The requested module '@segment/analytics-node' is a CommonJS module, which may not support all module.exports as named exports.

node version 16.14.2
looking in @segment's own package.json, no "type": "module" there, it's a CommonJS indeed but syntax is ESM

@silesky could you advice?

Hey, I'm not by my computer RN, but does default import work i.e can you do "import Analytics from '@segment/analytics-node'"?

@viniciuspalma I think you want to do:
yarn up --exact @segment/analytics-node. Even though the beta versions are marked as "latest", the bad rc version still seems to install.

Hi @silesky I'd like to use single package manager in build script, i.e. no yarn, npm install only.
Trying in package.json without semversion "@segment/analytics-node": "1.0.0-beta.19", No matching version found for<EMAIL_ADDRESS>For your question regarding default import, it didn't work either but going with just "analytics-node": "^6.2.0" installs and I could import, default one only.
Not sure package above is in working state. No error in analytics.track but no events flowing seen in Segment debugger. Could be unrelated issue of course.

Hm where did I get the 19 from? Trying 0.0.1-beta.18

For your question regarding default import, it didn't work either but going with just "analytics-node": "^6.2.0" installs and I could import, default one only.
analytics-node is the old-soon-to-be-deprecated version of analytics. I'm not sure what's going on in your system, but analytics-node is definitely available.

My .npmrc pointing at company's artifactory where it's not yet published?

Hi @silesky finally "@segment/analytics-node": "^1.0.0-beta.19" is in our artifactory and I have it installed. It doesn't resolve importing issue
import Analytics from '@segment/analytics-node';
(node:39714) Warning: To load an ES module, set "type": "module" in the package.json or use the .mjs extension.
(Use `node --trace-warnings ...` to show where the warning was created)
/Users/eugener/Development/chatbot-logger/node_modules/@segment/analytics-node/dist/esm/index.js:1
export { Analytics } from './app/analytics-node';
^^^^^^

SyntaxError: Unexpected token 'export'
    at Object.compileFunction (node:vm:352:18)

import { Analytics } from '@segment/analytics-node';
import { Analytics } from '@segment/analytics-node';
         ^^^^^^^^^
SyntaxError: Named export 'Analytics' not found. The requested module '@segment/analytics-node' is a CommonJS module, which may not support all module.exports as named exports.
CommonJS modules can always be imported via the default export, for example using:

import pkg from '@segment/analytics-node';
const { Analytics } = pkg;

    at ModuleJob._instantiate (node:internal/modules/esm/module_job:127:21)

@eugenerr, I think I found your issue. Proper support for node exports maps should be fixed in 1.0.0-beta.20, please try it out. (https://github.com/segmentio/analytics-next/pull/807/files)

@silesky thank you, confirming, importing issue has been fixed.

beta.25 has been released, which adds a disable option (for tests).

Closing since 1.x has been released.

Awesome @silesky I hope to make the upgrade soon!


[FR] Explicitly provide task name when enqueuing new Cloud Task

Is your feature request related to a problem? Please describe.
No.
Describe the solution you'd like
The Cloud Tasks API allows you to specify a task's name explicitly but the SDK does not currently expose this.  This is useful for task de-duplication, for example.  It would be great to expose this via TaskOptions through a name property like so:
const queue = getFunctions().taskQueue("taskFunction");
queue.enqueue(
  { ... },
  { name: "task-name" }
);

Describe alternatives you've considered
The workaround is to use the @google-cloud/tasks SDK directly like so:
import { CloudTasksClient } from "@google-cloud/tasks";

const client = new CloudTasksClient();

const name = "task-name";
const projectId = "project-name";
const location = "us-central1";
const functionName = "taskFunction";
const serviceAccountEmail =<EMAIL_ADDRESS>
const parent = client.queuePath(
  projectId,
  location,
  functionName
);

const url = `https://${location}-${projectId}.cloudfunctions.net/${functionName}`;

client
  .createTask({
    parent: parent,
    task: {
      name: `${parent}/tasks/${name}`,
      httpRequest: {
        headers: {
          "Content-Type": "application/json",
        },
        body: Buffer.from(
          JSON.stringify({ data: {} })
        ).toString("base64"),
        httpMethod: "POST",
        url: url,
        oidcToken: { serviceAccountEmail },
      },
    },
  });

Additional context
While the solution above works, it is clearly far more verbose and cumbersome than if the firebase admin were to expose this functionality directly.

I think this has been implemented in #2216?


Query params are not properly sanitized

Hey,
One of our developers (Randy Tobias) encountered a problem today when attempting to query a URL which is collected from the 'Location' header.
The following code will fail:

_url = "http://www.example.com/?hello=hello world"
requests.get(_url)

The only way to make this work is to patch up the query string with urlencode - therefore I am submitting this patch for consideration. I believe that enough websites send back broken "Location" headers for this to be included in the core.
Without it, we get the following exception:

webapp.libs._requests.exceptions.ConnectionError: HTTPSConnectionPool(host=u'online2.statefarm.com', port=443): Max retries exceeded with url: /apps/iarqapply/launch.redirect?postalCode=07017&productId=F1MRW45Q500&effectiveDate=20120214&applicationName=Auto&stateAbbreviation=NJ&regionalOffice=17&endEffectiveDate=20120315&city=East Orange

Here is the patch:
# models.py

def patch_url(i):
    import urlparse
    import urllib
    o = urlparse.urlparse(i)

    _n = urlparse.ParseResult(
        scheme = o.scheme,
        netloc = o.netloc,
        path = o.path,
        params = o.params,
        query = urllib.urlencode(urlparse.parse_qsl(o.query)),
        fragment = o.fragment,
    )

    return _n.geturl()

class Request(object):
    """The :class:`Request <Request>` object. It carries out all functionality of
    Requests. Recommended interface is with the Requests functions.
    """

    def __init__(self,
        url=None,
        headers=dict(),
        files=None,
        method=None,
        data=dict(),
        params=dict(),
        auth=None,
        cookies=None,
        timeout=None,
        redirect=False,
        allow_redirects=False,
        proxies=None,
        hooks=None,
        config=None,
        _poolmanager=None,
        verify=None,
        session=None):

        #: Float describes the timeout of the request.
        #  (Use socket.setdefaulttimeout() as fallback)
        self.timeout = timeout

        #: Request URL.

        # if isinstance(url, str):
            # url = url.encode('utf-8')
            # print(dir(url))

        self.url = patch_url(url)

Before patch: https://online2.statefarm.com/apps/iarqapply/launch.redirect?postalCode=07017&productId=F1MRW45Q500&effectiveDate=20120214&applicationName=Auto&stateAbbreviation=NJ&regionalOffice=17&endEffectiveDate=20120315&city=East Orange

After patch: https://online2.statefarm.com/apps/iarqapply/launch.redirect?postalCode=07017&productId=F1MRW45Q500&effectiveDate=20120214&applicationName=Auto&stateAbbreviation=NJ&regionalOffice=17&endEffectiveDate=20120315&city=East+Orange

Many thanks
Cal Leeming
Simplicity Media Ltd

Randy Tobias

See issue #369

I don't understand exactly what the problem is. It looks like you sent a request to the URL:
http://www.example.com/?hello=hello world

And then you report an exception involving this URL:
/apps/iarqapply/launch.redirect?postalCode=07017&productId=F1MRW45Q500&effectiveDate=20120214&applicationName=Auto&stateAbbreviation=NJ&regionalOffice=17&endEffectiveDate=20120315&city=East Orange

What do those two URLs have to do with each other? They seem to both have in common a space in the query string.
Let's pick a more useful test case: a URL on Kenneth's nice HTTP test site that won't redirect (unlike example.com):
http://httpbin.org/get?hello=hello world

My test code:
>>> _url = "http://httpbin.org/get?hello=hello world"
>>> r = requests.get(_url)
>>> r.url
u'http://httpbin.org/get?hello=hello world'

(I'm not sure why it became a Unicode string.)
Indeed, this is a malformed query string, because it has a space in it. Note that this is AFTER my patch was merged. My patch deliberately only dealt with the path component (because I didn't want to mess with the behaviour too much; requests was already munging the path but not the query). If we want requests to automatically fix this, it should fix it as follows:
http://httpbin.org/get?hello=hello%20world

It should NOT be encoding the space as a '+'. Space is NOT equivalent to + in the URI syntax, and doing so would break existing correct code. Space is correctly encoded as %20. The replacement of space with '+' is not part of the general URI syntax; it is part of the HTML specification as defined by the W3C -- the content type application/x-www-form-urlencoded defines '+' as equivalent to space. Any URI that is not the result of an HTML form submission should not consider those equivalent.
So the fixes I made on Issue #369 were actually relevant here, it's just that they were only applied to the path. I have pushed a new branch, uri-cleanup-nonpath which extends the application of requote_path to the entire URI once it is reconstructed, which means that all parts of the URI will now benefit from cleaning. This gives the following result to the above test:
>>> r.url
'http://httpbin.org/get?hello=hello%20world'

I have added full tests of the sanitisation of the query part of the URI, and I think that my patch would resolve this issue. I'll put in a pull request.
P.S. When you want to send a patch, please create a Git branch and commit it, then push to GitHub and create a Pull Request. That will make it much easier for people to see what changes you intend to submit, rather than simply pasting code into a bug report.

I should add, one of our developers (James Cleary) found that our previously suggested patch causes other unit tests to break, YMMV.

encode_uri configuration has been added.

Nice! Thank you very much for this Kenneth :)

Also @mgiuca, thank you very much for raising the issue about the + and %20 stuff, our patch was failing on several tests due to this problem - so you've saved us a lot of time :) Thank you again.

No worries :) I'm a bit obsessed with URI encoding issues so I'm happy to help out.

@mgiuca good to know :)

Thanks from me as well! Ran into the issue yesterday, and today it's fixed.


Replace models with Workflow models

This PR accomplishes the following main things:

Replace the existing models with workflow specific models
Create an views for interfacing with these workflows from the API
Layout the ground work for running these workflows

Also, @AlmightyYakob would you go ahead and move this algorithms app to the top level of the repo per https://docs.djangoproject.com/en/3.2/intro/reusable-apps/#packaging-your-app
I also think we should rename it to django-rgd-workflow with rgd_workflow as the module name

With some after thought, @AlmightyYakob would you go ahead and move this algorithms app to the top level of the repo per docs.djangoproject.com/en/3.2/intro/reusable-apps/#packaging-your-app

Sure thing

I also think we should rename it to django-rgd-workflow with rgd_workflow as the module name

I'm fine with that, but if we plan on upstreaming this into https://github.com/ResonantGeoData/ResonantGeoData, would the rename cause any issues there? I suppose not, as we'd just be changing where the code itself lived, but just want to be sure.

With some after thought, @AlmightyYakob would you go ahead and move this algorithms app to the top level of the repo per docs.djangoproject.com/en/3.2/intro/reusable-apps/#packaging-your-app
I also think we should rename it to django-rgd-workflow with rgd_workflow as the module name

Also, is the core app still needed? And if so, how should that and the algorithms app be coordinated?

I don't think there would be any issues from the rename. It would be consistent in the fact that this is another Django app built on top of django-rgd core

:+1:

Yes, the core app is still needed for project-specific migrations and management commands and should remain in the Django Project directory. What do you mean by coordination? the algorithms/workflow app is completely independent of core and at this time, core doesn't have anything linking to algorithms

I essentially wanted to make sure of this. I'll go ahead with the reorg and rename.


Bump dependencies

rand in particular is nice to keep up with so we don't have to build both 0.6 and 0.7 in downstream crates.

I believe this is a backwards-compatible change, so only a minor version bump should be needed.


fix(Metrics Router): add new region eu-fr2

PR summary
Added new region BNPP(eu-fr2) to metrics-router
PR Checklist
Please make sure that your PR fulfills the following requirements:

[x] The commit message follows the Angular Commit Message Guidelines.
[x] Tests for the changes have been added (for bug fixes / features)
[x] Docs have been added / updated (for bug fixes / features)

Current vs new behavior
Added new region. No change in old code
Does this PR introduce a breaking change?

[ ] Yes
[x] No

Other information

 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.Divi-Tiru-Sai-Gopala-Pavana-Phaneendra seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

Ran integration tests

:tada: This PR is included in version 0.56.2 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Evaluate effectiveness of DNS traits

It is currently my understanding that DNS can be implemented on top of generic sockets. As such, it should be possible to write a generic no_std DNS client crate that relies on the embedded-nal. This would mean that the dns traits in this crate could likely be removed, since they no longer need to be generic.
I'm curious to hear other opinions here - I have not yet written a DNS client, but likely will be doing so in the near future (and would publish it as said no_std DNS resolver)

@MathiasKoch Can you elaborate as to the use case of the original trait? Is the desire to be able to provide a NetworkStack to a client protocol crate that is capable of resolving domain names to IP addresses for connections?
I could see that being a useful attribute, but it introduces of complexity of crate users - for example, what should we do for a protocol that wants optional DNS resolution? E.g. a network stack may or may not support DNS resolution.
Instead, if we had a standalone crate,  then any protocols that want DNS resolution can just use any provided NetworkStack to construct a DNS resolver on top of available sockets to resolve domain names. This would eliminate the requirement that end users provide a DNS resolver

I have in my cases for example hardware that supports DNS lookups as AT commands, removing the need for a no_std dns client

I think it would be awesome to add some default opt-in embedded-nal based dns-client, but i wouldn't want to see the trait removed, as there are lots of cases where one can save binary size, and offload the task using co-processors

Sounds good! Thanks for providing the input - I figured I was missing something. Do you have some examples of how these traits are being used?

You know i do ;)
https://github.com/BlackbirdHQ/ublox-cellular-rs/blob/master/ublox-cellular/src/services/data/dns.rs

I think it can be useful to have a "thing that does DNS" trait. Especially as it may already be supported by hardware or firmware.
Maybe adding a (basic) DHCP trait makes sense, too.

@MathiasKoch I'm now starting to need the Dns traits, and I'm noticing that they appear to assume that the lookup can be completed in a blocking manner. For network stacks like smoltcp I don't think that's possible, since the network stack will need to communicate with the DNS server to resolve the request asynchronously. Perhaps we should use an nb::Result and potentially update this to async in the future?

@ryan-summers  That wouldn't be a problem for me :+1:

For context here, I chatted with the smoltcp folks and they pointed out that DNS can be intrusive into the lower layers of the network stack (i.e. grabbing at MAC addresses, etc.) because of its history with the bootp protocol. As such, it's somewhat difficult to segment DNS as distinctly "above" the socket layer. This makes a stack-agnostic DNS implementation difficult because of the need to know stack-specific information.


Header Keys are inaccessible outside of http4s now

 method name in class HeaderCompanion cannot be accessed as a member of object org.http4s.headers.Host from ...
[error]  Access to protected method name not permitted because
[error]  enclosing <...> is not a

This was found attempting to remove the host header and since the key is inaccessible it had to be done manually.

It was inconsistent whether they were on the modeled header companions once the super trait disappeared, but the typeclass has it:
$ scala-cli repl -S 2.13 --dependency org.http4s::http4s-core:0.23.7
Welcome to Scala 2.13.7 (OpenJDK 64-Bit Server VM, Java 1.8.0_292).
Type in expressions for evaluation. Or try :help.

scala> import org.http4s._, headers._
import org.http4s._
import headers._

scala> Header[Host].name
val res1: org.typelevel.ci.CIString = Host

I do think we should expose the name field on all of the headers. I've been tempted to suggest renaming HeaderCompanion to HeaderKey and making it public, but I'm a bit concerned it could give the impression of being a central part of the API rather than just a convenience.

It changed in a consistency pass.  Some things were defining it on the companion, and some directly on the instance.  This might have been before there was a HeaderCompanion.
The name field on HeaderCompanion could be made public and achieve the same effect, without promoting it as public API.  That's a bit OOPy, but that kind of OOPiness did well for us in the past.


How can I get the plaint text?

The selector in chromedp is very weak, I can't extract what I needed from response.
Sometimes I got json or other plaint text, how can I get the data and marshal it myself？

I'm not sure if i'm understand your question, can please add more information?
(preferably using the provided issue template)
https://godoc.org/github.com/chromedp/chromedp#Text will allow you to fetch text data from the page as it is. If you need to marshal it to other format such as json or xml you can process it separately once the data is retreived.

https://godoc.org/github.com/chromedp/chromedp#Text

sometimes the data that i reviceived id not xml, but json and other format instead. So, the CSS selector can't help me to fetch data from the page, How can I get the non-XMl data?

I'm not sure if i'm understand your question, can please add more information?
(preferably using the provided issue template)
https://godoc.org/github.com/chromedp/chromedp#Text will allow you to fetch text data from the page as it is. If you need to marshal it to other format such as json or xml you can process it separately once the data is retreived.

I'm not sure if i'm understand your question, can please add more information?
(preferably using the provided issue template)
https://godoc.org/github.com/chromedp/chromedp#Text will allow you to fetch text data from the page as it is. If you need to marshal it to other format such as json or xml you can process it separately once the data is retreived.

sometimes the data that i reviceived id not xml, but json and other format instead. So, the CSS selector can't help me to fetch data from the page, How can I get the non-XMl data?

I assume it still wrapped into html tag? If so you can still get the content using chromedp.Text.
Here is the example for scraping yaml file on github.
package main

import (
	"context"
	"fmt"
	"log"

	"github.com/chromedp/chromedp"

	yaml "gopkg.in/yaml.v2"
)

func main() {
	var err error

	// create context
	ctxt, cancel := context.WithCancel(context.Background())
	defer cancel()

	// create chrome instance
	c, err := chromedp.New(ctxt, chromedp.WithLog(log.Printf))
	if err != nil {
		log.Fatal(err)
	}

	// run task list
	var res string
	err = c.Run(ctxt, chromedp.Tasks{
		chromedp.Navigate(`https://raw.githubusercontent.com/chromedp/chromedp/master/.travis.yml`),
		chromedp.Text(`body > pre`, &res),
	})
	if err != nil {
		log.Fatal(err)
	}

	// shutdown chrome
	err = c.Shutdown(ctxt)
	if err != nil {
		log.Fatal(err)
	}

	// wait for chrome to finish
	err = c.Wait()
	if err != nil {
		log.Fatal(err)
	}

	parsed := make(map[interface{}]interface{})
	err = yaml.Unmarshal([]byte(res), &parsed)
	if err != nil {
		log.Fatal(err)
	}
	fmt.Printf("--- m:\n%v\n\n", parsed)
}

I assume it's wrapped into html tag? If so you can still get the content using chromedp.Text.
Here is the example for scraping yaml file on github.
package main

import (
	"context"
	"fmt"
	"log"

	"github.com/chromedp/chromedp"

	yaml "gopkg.in/yaml.v2"
)

func main() {
	var err error

	// create context
	ctxt, cancel := context.WithCancel(context.Background())
	defer cancel()

	// create chrome instance
	c, err := chromedp.New(ctxt, chromedp.WithLog(log.Printf))
	if err != nil {
		log.Fatal(err)
	}

	// run task list
	var res string
	err = c.Run(ctxt, chromedp.Tasks{
		chromedp.Navigate(`https://raw.githubusercontent.com/chromedp/chromedp/master/.travis.yml`),
		chromedp.Text(`body > pre`, &res),
	})
	if err != nil {
		log.Fatal(err)
	}

	// shutdown chrome
	err = c.Shutdown(ctxt)
	if err != nil {
		log.Fatal(err)
	}

	// wait for chrome to finish
	err = c.Wait()
	if err != nil {
		log.Fatal(err)
	}

	parsed := make(map[interface{}]interface{})
	err = yaml.Unmarshal([]byte(res), &parsed)
	if err != nil {
		log.Fatal(err)
	}
	fmt.Printf("--- m:\n%v\n\n", parsed)
}

I will try it, thanks.


[Bug]: BrowserContext.onClose is not called with browser window is closed

Version
1.49.0
Steps to reproduce
git clone git@github.com:davidnewcomb/playright-bug-onclose.git
cd playright-bug-onclose
mvn clean install

I have modified the repo to only show this issue and moved registering event handlers to directly after the object is instantiated.
Expected behavior
When I shutdown the browser with control+q or click the [X] window button then the event handler given to BrowserContext.onClose should be called.
According to your documentation at on BrowserContext.close() Event it says you do detect that the browser has disappeared and you do call the event handler registered with BrowserClose.onClose(handler) but you don't as my program demonstrates.
Actual behavior
Playwright does not notice the browser is gone.
Additional context
This is related to #1714 but I've moved the problem to the BrowserContext level because it occurs there too.
How can I call anything if Playwright never tells me the browser has disappeared. Do I have to get the browser PID myself and have a thread which constantly checks if the PID is active and then calls BrowserContext.close()? That feels like a Playwrite job ;)
According to your documentation at on BrowserContext.close() Event it says you do detect that the browser has disappeared and you do call the event handler registered with BrowserClose.onClose(handler) but you don't as my program demonstrates.
I can't even use Browser.isConnected() or Page.isClosed() in order to detect that the browser is not connected to anything, they don't notice either.
Environment
-Operating System: MacOS Sequioa 15.2
-CPU: intel core i9
-Browser: Nightly
-Java version: Java version: 22.0.1, vendor: Homebrew, runtime: /usr/local/Cellar/openjdk/22.0.1/libexec/openjdk.jdk/Contents/Home
-Java compiled for v 1.8 and 21
-Tested on Nightly/firefox and Chromium
-Maven: 3.9.8
-Running in: Spring Tool Suite 4 - Version: 4.26.0.RELEASE

I ran a test using Playwright's Junit fixture to see if I could recreate what you were seeing.
@UsePlaywright(PlaywrightOptions.class) // Setting headless to false
public class Test {
	@Test
	void onClose(Page page, BrowserContext context) throws InterruptedException {
		context.onClose(onClose -> {
			System.out.println("AHHH");
		});
		page.navigate("https://playwright.dev/");
		page.waitForTimeout(5000);
		assertThat(page).hasTitle(Pattern.compile("Playwright"));
	}

I did see AHHH printed to my console when the test closed successfully and when I closed the browser manually.
I see you did not use Fixtures in demonstrating your test, I can try to recreate that later.

BrowserContext is not expected to be closed when you close last window. E.g. you can have a scenario where you create and close pages (leaving 0 pages in the context):
		for (int i = 0; i < 3; i++) {
			Page page = browserContext.newPage();
			page.close();
		}

This is working as intended. When the browser process exits, you'll receive onDisconnected event. Also note, that Thread.sleep will not dispatch any events, see https://playwright.dev/java/docs/multithreading#synchronous-api-and-event-dispatching.

@Drew-Ackerman I'm not using fixtures I'm running it in a real program.
@yury-s in real life there are no Thread.sleeps() in my program I just wait in real life. I updated my program to use waitForTimeout
I think I have worked out what the problem is. Playwrite does not detect that the browser is gone and then call the event handlers. It waits until there is some interaction with the page which then fails and that failure triggers sending the event messages.
If you set up a browser and leave it waiting for a human to come along and tell it what to do (via my program). However if someone else has closed the window then Playwright does not notice. When the next person comes along they will find out it's broken when they try and use it. Basically the event is reactionary and not proactive, so you don't see the problem until you try to use it.
I guess I need something to keep poking the page so it can notice that the browser is gone and trigger the onClose callbacks. Does Playwrite have any heartbeat facilities?
Now when I create the Page with BrowserContext.newPage then I give the Page instance to another Thread  and that sits in a 60 second waitForTimeout. While it is in there if anyone closes the page then I get notified straight away. I hear what you said about the non-thread safe nature of Playwrite and am prepared to deal with that when the time comes but at the moment this seems to be working.
I still think that it's something that should be done on your side though. Or at least have a configuration option to switch on heartbeats in order to get a pro-active onClose.

It waits until there is some interaction with the page which then fails and that failure triggers sending the event messages.

This is the downside of the synchronous API and you have to make a playwright API call in order to pump events from the browser, see the link I sent above: https://playwright.dev/java/docs/multithreading#synchronous-api-and-event-dispatching
page.waitForTimeout is one way to make sure the events are dispatched while you are waiting. You can also use waitForCondition() to accomplish that in your program :
        log("Close window, you have " + 100 + " seconds to comply! Connected=" + browser.isConnected());
        browserContext.waitForCondition(() -> !browser.isConnected(), new BrowserContext.WaitForConditionOptions().setTimeout(100_000));

This code will dispatch browser events, including disconnect, properly.  Hope this helps.


Allow users to download pre-built binaries for supported platforms (macos, Linux, Windows, Pi aarch64)

Avoid users having to build from source, but instead on mac or linux they can download a binary for their pi, and make or similar can help to copy to the pi.
document in readme.
Maybe need to sign the binary, or provide checksum?

See https://blog.orhun.dev/automated-rust-releases/

for 0.4.0

Largely done, just Pi remaining and a new issue created for that


Can't  attach the indicator to RecyclerView

I added Android x dependencies and i couldn't attach the indicator to the recycler view as mentioned in the README  file. I'm using koltin

this is one of the most misleading API's i have ever faced . CircleIndicator , *CircleIndicator2 * , CircleIndicator3 on the same version ? why ? and why  *CircleIndicator2 * supports attach the indicator to RecyclerView , but 3 doesn't ? weird


Fix minor typo

threshold was spelled threshhold

This change is

/assign @derekwaynecarr

/approve
/lgtm

:heart: Thanks! :)


Updated memory linking in configuration

Dear Rust Embedded Team,
I have made a minor change to your quickstart guide.
The current Rustflags are misconfigured and do not allow adding no_std packages.
The -T flag is linking to a file that does not exist, however, the compiler does not complain about it. I updated the file to reference the proper file, which will allow the compilation of projects with no_std packages.
Best Regards,
@BBPL

Thank you for the PR.
I believe the code as-is before the PR is correct. link.x is provided by the msp430-rt crate. rustc will pass an option so link.x is found during a build.
So the reason the linker isn't complaining about a missing file is because it actually did find the file :). link.x depends on memory.x, but memory.x alone won't be enough to create a working application if using msp430-rt/the quickstart; you need both!

Hi,
You seem to be correct. My errors are possibly originating from another problem.
Sorry for the inconvenience, and thank you for the insight.


Load params from registers when possible

Load ArgIns from registers if the param was passed in registers (x64 only).  Cleanup implicitParams in the process.

@satheeshravi, @rajatd, please review.

}

I think, this function is unused. Can you please remove ?

Refers to: lib/Backend/LowerMDShared.cpp:8044 in cc151bf. [](commit_id = cc151bf0953a34d459c8a380ca65e31d8d53b0d6, deletion_comment = True)

}

It is used by LoadCallInfo in lower.

In reply to: 240313560 [](ancestors = 240313560)

Refers to: lib/Backend/LowerMDShared.cpp:8044 in cc151bf. [](commit_id = cc151bf0953a34d459c8a380ca65e31d8d53b0d6, deletion_comment = True)

:shipit:

It could be really nice to squash all these 'cleanup' and 'more cleanup' commits before merging.


Enquiry to add PALM to OpenOOD leaderboard

In correspondence to our recent pull request for adding a new method PALM to OpenOOD. We also aim to contribute to the OpenOOD public leaderboard. Please find the details as follows:
Training: PALM, note that we only train the encoder as in CIDER and NPOS
Postprocessor: Mahalanobis distance
Outlier data: No outlier data needed during training
ID Accuracy: Since we only train the encoder, the ID accuracy remains NA
ID Datasets: CIFAR100, CIFAR10
For other experiment results and pretrained weights, please refer to the following link:
https://unsw-my.sharepoint.com/:f:/g/personal/z5183944_ad_unsw_edu_au1/EiBU2BP8l6VCv19XTaG14CYBOzFe-56g2vTdc2BYPxvQPQ?e=SyAP5d
For your convenient, the files from the shared link follows the exactly the same structure as in OpenOOD benchmark.

Hi @jeff024, I opened the csv files provided in the link. While the OOD detection results look right, the accuracy seems weird (below is the screen shot for CIFAR-10).

I looked at the forward of palm_net and I suspect that it always outputting features might be the cause for this low accuracy.
https://github.com/Jingkang50/OpenOOD/blob/183f86e2f5756429d643a0c65f43060d7ecea2be/openood/networks/palm_net.py#L27-L33

Like previous methods CIDER and NPOS, our method PALM only trains an image encoder, without any classification head. To obtain the ID accuracy, the only method is with linear probing that trains an standalone classification head with the image encoder trained with our method PALM. So that's why the results of ID accuracy is so weird.
Given these, could you please leave the ID accuracy as NA? Just like CIDER and NPOS in the leaderboard.

Oh I see now. Thanks for the clarification. Will update asap.

PALM entries have been included now. Thank you for the work too.


Split up data and polygons

Before we go too wild adding all the things, we should separate the data from the polygons so that we don't run in to the kind of geojson duplication we see in editor-layer-index.

This was done, now we have
features/ - contains .geojson files, one for each feature.
resources/ - contains .json files, one for each resource.
Each resource has a featureId to link it to a single feature.

Couldn't this been done also in a way that doesn't require at the separation into two files for each "region"?

Yes, I considered that, but I think this way is:

easier to explain to people
reduces the need to edit the .geojson, which is more fragile
better if community forums / meetups come and go but the features rarely change
allows people to consume just the resource list, if they don't care about the geometry

ok, I guess it comes down to a matter of taste ;), but:

it can make it harder to create PRs (two files in separate directories vs. one single file which one can even do directly on github.com without even having to check out the repo locally)
having hardcoded id strings as cross references is also a bit fragile. For example when removing a resource: do I have to check all other resources whether at least one still points to the respective geometry before deleting the geojson file as well? or do I just leave it in place in case something might re-use that geometry later on?
consuming just the resource list would still be possible if you would provide it as a stripped down version of the "full" data in /dist


editorconfig file fully cancels Lua.format.defaultConfig

I've noticed, that Lua.format.defaultConfig setting works fine if .editorconfig absents, but if .editorconfig exists in project - Lua.format.defaultConfig does nothing. It seems be better if Lua.format.defaultConfig would be merged with .editorconfig (with priority for .editorconfig).
For example, if .editorconfig does not have keep_one_space_between_table_and_bracket option - then use the same one from Lua.format.defaultConfig (if present).
I use the formatter through sumneko LSP in vscode. Thanks a lot.

A little off topic, but there's no discussions and I don't want to make a new issue for this.
How do you set up defaultConfig? I added this to vscode's User/settings.json:
"Lua.format.defaultConfig": {
	"indent_style": "tab",
	"tab_width": "4"
}

And it had no effect.

@tomasklaen, maybe conflict with native vscode settings? Like the following:
"[lua]": {
    "editor.insertSpaces": false,
    "editor.tabSize": 4
},

Lua.format.defaultConfig only valid for sumneko lua plugin.
in sumneko, the language server protocol should be respected, so indentation use the parameters of the LSP

I am using sumneko's lua plugin, but I'm using EmmyLuaCodeStyle to format the code because sumneko's formatter ignores some editorconfig properties.
Is there no way to make these 2 work together?
I've tried adding this to Lua.format.defaultConfig:
"insertSpaces": "false",
"tabSize": "4",

And this to [lua]:
"editor.insertSpaces": false,
"editor.tabSize": 4

And again, no effect. Can someone just give me a snippet or some example/documentation somewhere how to configure the default formatting options? 😢

sumneko formatter is my formatter. but my plugin not support Lua.format.defaultConfig or editor settings. it only read .editorconfig plugin.
You can set the formatter to sumneko lua because that's what I provide

@CppCXY I'm using this one instead of sumneko's on your own recomendation :) So the linked issue has been resolved, and sumneko now reads indentation style from editorconfig instead of user settings?

no, for sumneko, Indentation comes from editor settings, you can set editor setting for indentation

Sumnkeo has already been updated to the latest version, so the functionality is consistent


[BUG]: winbar shows in terminal buffers in some cases

Requirements

[X] I have skimmed other issues to see if this bug has already been reported.
[X] I'm fully certain that this is not a breaking change, see the latest release.
[X] This plugin and its dependencies are up to date with the latest commit.
[ ] I'm willing to fix this through a PR.

Expected Behavior
If I just start a terminal by for instance running :call termopen(['ls'], {}), there is no winbar in the current window (the current window is occupied by the terminal buffer). This is the behavior that I expect from barbecue.
Now, I would also expect this behavior to be the same when doing the following set of actions

Open a new window
Create a new buffer
Start a terminal in this new window
Go back to the previous window

This is expressed by this function:
function TestBarbecue() abort
    let l:orig_winid = bufwinid('%')
    vsplit
    enew
    call termopen(['ls'], {})
    call win_gotoid(l:orig_winid)
endfunction

Actual Behavior
When calling the function defined above as :call TestBarbecue(), what I get is that there is a winbar in the window occupied by the terminal buffer. This is not expected (nor wanted, in my use case).
Removing the line call win_gotoid(l:orig_winid) restores the expected behavior, i.e. no winbar in the terminal window. Interestingly, keeping the line call win_gotoid(l:orig_winid) but adding a sleep 1m just before the call to termopen() also restores the expected behavior.
Neovim Version
NVIM v0.9.2
Build type: RelWithDebInfo
LuaJIT 2.1.0-beta3
Compilation: /usr/bin/gcc -O2 -g -Og -g -Wall -Wextra -pedantic -Wno-unused-parameter -Wstrict-prototypes -std=gnu99 -Wshadow -Wconversion -Wvla -Wdouble-promotion -Wmissing-noreturn -Wmissing-format-attribute -Wmissing-prototypes -fno-common -Wno-unused-result -Wimplicit-fallthrough -fdiagnostics-color=auto -fstack-protector-strong -DUNIT_TESTING -DINCLUDE_GENERATED_DECLARATIONS -D_GNU_SOURCE -I/usr/include/luajit-2.1 -I/usr/include -I/usr/include/luv -I/builddir/build/BUILD/neovim-0.9.2/redhat-linux-build/src/nvim/auto -I/builddir/build/BUILD/neovim-0.9.2/redhat-linux-build/include -I/builddir/build/BUILD/neovim-0.9.2/redhat-linux-build/cmake.config -I/builddir/build/BUILD/neovim-0.9.2/src -I/usr/include -I/usr/include -I/usr/include -I/usr/include -I/usr/include -I/usr/include -I/usr/include

   system vimrc file: "$VIM/sysinit.vim"
  fall-back for $VIM: "/usr/share/nvim"

Minimal Configuration
Simple init.vim to reproduce the issue:
call plug#begin('~/.local/share/nvim/plug')
Plug 'SmiteshP/nvim-navic'
Plug 'utilyre/barbecue.nvim'
call plug#end()
lua require('barbecue').setup()

Reproduction

Define the function TestBarbecue() as specified in Expected Behavior
Run the function as :call TestBarbecue()
Observe that the window with the terminal buffer has a winbar

Possibly related to #92 and #34

I've done some more digging, here's what I concluded:
By default, an update to the winbar is only performed on the events WinResized, BufWinEnter, CursorMoved, InsertLeave, as defined by barbecue. However, opening a terminal with e.g. termopen() does not trigger any of the above events. Nevertheless, when running the command passed to termopen(), some lines are appended to the terminal buffer, and this triggers CursorMoved as a side effect (or so it seems). This event is actually triggered two times when launching termopen() directly, but only one time when launching it as part of the function TestBarbecue() defined in the original description of this issue. This discrepancy seems to be enough for the unexpected behavior described above.
Given that, I would think that there should be an additional event that should perform a winbar update, and this event should be triggered when launching a terminal. More in general, there should be an event that's triggered whenever buftype changes for the current buffer, but as far as I know this only happens when launching a terminal.
Based on this assumption, I tried adding TermOpen to the list of events that perform a winbar update, but this does not solve the problem. The function update() in lua/barbecue/ui.lua does not seem to have the logic to disable the winbar for a buffer whose buftype has just changed.
Does this make sense?


Advanced API queries

Download all scores by:

Trait using an EFO id (/rest/trait/{trait_id})
Publication using aPGP id (/rest/publication/{pgp_id})

Rest documentation: https://www.pgscatalog.org/rest/#/

44c487799e525706daf080062c90794303fc64f7


setOnClickListener is blocked but I need to use it

Due to some code I inherited I need to use setOnClickListener. Basically I want the look and feel of the spinner but when it is clicked on a separate dialog appears. I am not using any of the other features of the spinner, no drop down, etc. but I need the look with the title over the spinner etc.
I do fully use of your spinner in a number of other areas in the code, this area just happens to be a hackfest.
The code in MaterialSpinner.kt at line 319 blocks this usage - as hacky as it might be.
override fun setOnClickListener(l: OnClickListener?) {
    throw RuntimeException(
        "Don't call setOnClickListener." +
                "You probably want setOnItemClickListener instead."
    )
}

Really need this check to be optional

To get around this issue I put a transparent View on top of the MaterialSpinner and get the onClickListener from there.

Hi @MKevin3, have you tried setting a OnFocusChangeListener instead?

Possible duplicate of #10

Hi @MKevin3 any feedback on this?

I just put a view on top of the MaterialSpinner as far as Z-Order is concerned and grab the listener there. While I understand why MaterialSpinner gives the warning in the log as it is a common mistake I don't feel it should be a "and there is NO way you can do this" exception.

@MKevin3 You should be able to achieve the same functionality without the need of an extra view using the OnFocusChangeListener.
When you click on this view it also gains focus, therefore the focus callback will be triggered, and you will be able to achieve the same effect. Take a look at the sample for a simple example.

Hi @MKevin3 did you had the time to take a look at tying OnFocusChangeListener?


CB-4548 Cloudbreak and FreeIPA must persist the minaSshdServiceID used and use that for service endpoint lookup

One minor thing otherwise LGTM


As a developer, I want to see the list of operations as the default when I go to an API console so I can know what I can explore.

As a developer, I want to see the list of operations as the default when I go to an API console so I can know what I can explore.
DOD:
Change the default from hide to List Operations in all API consoles.

done.

Usain Bolt style. Thanks!


Added system config statement_cache_size

Used to control the size of Postgres prepared statement cache.
Fixes #5233

Do we want to use "backend" instead of "pg"? But I also understand this config name is getting really long 🚄🚋🚋🚋🚋🚋🚋🚋

Do we want to use "backend" instead of "pg"?

In this case this is specifically about the Postgres implementation, so I'd use pg_.  Likewise, other exposed Postgres settings should ideally be prefixed, but let's leave that to when we'll properly implement config vars (a-la global vars), which will bring natural namespacing via modules.


Music Assistant only plays finite number of tracks by any artist

What version of Music Assistant has the issue?
2.0.0b75
The problem
When I ask Music Assistant to play music by any artist with Spotify, it will only play a finite number of songs by that artist. But Spotify will keep playing an infinite number of songs by that artist.
How to reproduce
Call this service:
service: mass.play_media
data:
  media_type: artist
  media_id: Queen
target:
  entity_id: media_player.hauptschlafzimmer

Relevant log output
No response
Additional information
No response
What version of Home Assistant Core are your running
2023.11.3
What type of installation are you running?
Home Assistant OS
On what type of hardware are you running?
Raspberry Pi

This is being worked on already. Hopefully next beta will allow this

However, I don’t think the MA solution will play forever, I think the plan is just to play all available tracks for the artist

Just looking at his again. MA will queue all songs in your LIBRARY if you do the service call you listed in your OP. Are you saying you want MA to actually play all songs from a provider?

Just need some more followup or we will have to close this


"Unable to write user settings" info bar should include a link to user settings

Testing #15811
We're asking people to go there, so might as well remove a step and link them there.

VSCode Version: Code - Insiders 1.8.0-insider (7499a9f3c565f4fbe5205e30c8664e5a5a11ae94, 2016-12-06T16:16:15.865Z)
OS Version: Windows_NT ia32 10.0.14965

Duplicates #16580


Bugfix: startup failing to parse CIDR

Description

Todos

[ ] Tests
[ ] Documentation
[ ] Release note

Release Note

None required

Fixes https://github.com/projectcalico/calico/issues/1229

CI tests won't pass because the python STs aren't updated, but I ran the node specific tests locally and they passed


GetPurchasesAsync(ItemType.InAppPurchase) also returns subscriptions?

I'm confused what the difference is between ItemType.Subscription and ItemType.InAppPurchase because the latter returns all in app purchases.
It is not wrong since a Subscription is technically an InAppPurchase
So is the type ItemType.InAppPurchase just a general name for all in app purchases?
Bug Information
Version Number of Plugin: 7.0.5
Device Tested On: iPhone 6s iOS 14.6
Version of VS: 17.8.0
Version of MAUI: .NET 8
Steps to reproduce the Behavior
var subscriptionPurchases = await CrossInAppBilling.Current.GetPurchasesAsync(ItemType.Subscription);
var singlePurchases = await CrossInAppBilling.Current.GetPurchasesAsync(ItemType.InAppPurchase);

Expected Behavior
ItemType.Subscription to return subscriptions only
ItemType.InAppPurchase to return single purchases only
Actual Behavior
ItemType.Subscription returns subscriptions only
ItemType.InAppPurchase returns subscriptions and single purchases
Code snippet
var subscriptionPurchases = await CrossInAppBilling.Current.GetPurchasesAsync(ItemType.Subscription);
var singlePurchases = await CrossInAppBilling.Current.GetPurchasesAsync(ItemType.InAppPurchase);

Screenshots

I added a note in the docs:

Note: On iOS there is no API to determine if a purchase was a subscription or in app purchase, so all purchases regardless of type will be return. It is required on Android. It is best to query and then check for Id.


What is the proper way of using Symfony rulesets?

We have large application based on custom framework, but many of Symfony components are used. Unfortunately those packages are in different versions (from 3.4 to 5.3). I thought that when I add SymfonySetList::SYMFONY_44 ruleset all rulesets for previous versions are loaded too, but I took a look at those rulesets and I don't see any connection between them. So I've used -vvv and compared applied rules with single ruleset added against configuration with multiple SymfonySetList::SYMFONY_* rulesets and it looks like more rules were applied.
What is the recommended flow for such scenario? Because this package's documentation states:

To add a set to your config, use Rector\Symfony\Set\SymfonySetList class and pick one of constants

Thanks in advance 🙂

Hi, thanks for a great question!
Including smaller sets would not allow to use configs one by one, but rather all or nothing. We have a unix phiolosophy = use the smallest function possible and let people pipe them as they need.
I would go one set by another, having one 1 PR per minor version :)
I wrote about in detail here: https://tomasvotruba.com/blog/2019/02/28/how-to-upgrade-symfony-2-8-to-3-4
:+1:

See Rector's rector.php: https://github.com/rectorphp/rector-src/blob/885ef768e50b8ef86bdf2eb99e290f5dccebd8c1/rector.php#L31-L35
When PHP 8.1 is out, we only add this set, upgrade the codebase in 1 PR and keep it there, so Rector upgrades all the future contributions.

Our codebase is so large (20-year old monolith application) that we currently don't refactor whole, but we have Rector as CI job on merge requests as a failure-allowed helper - only changed files are analysed and diff is printed in the job's output, so developer is encouraged to refactor files on which is working on. In fact, I never managed to run full analysis because of time that it needs 😅
Because of that, it's not possible to introduce those sets one by one because it's a long-term process of refactoring, hence my question.. 🤔 I think I'll go with multiple sets, up to SymfonySetList::SYMFONY_44 and then we will try to upgrade dependencies to 5.*. I am not 100% sure if it will work for us (mostly because of components in 3.4 version that should be upgraded first), but worth try 🙂
Thanks for the response! I'll close this as I think I got what I needed 👍

I see. Thank for explaining on detail.
Personally I never run Rector on whole code base, if it's too large.  I usually start with controllers, then merge it, etc.
Instead of forcing devs to do machine work, we narrow scope of the machine.
Check the "ONLY" option. It runs rules only on specific directory 👍


[TubeTuGraz] Add new extractor

Before submitting a pull request make sure you have:

[X] At least skimmed through adding new extractor tutorial and youtube-dl coding conventions sections
[X] Searched the bugtracker for similar pull requests
[X] Checked the code with flake8

In order to be accepted and merged into youtube-dl 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
[ ] Improvement
[X] New extractor
[ ] New feature

Description of your pull request and other information
Add support for extracting videos from tube.tugraz.at.

Looks good

Read coding conventions.

I'm just now seeing this as I finished my own TUbe extractor. Mine has support for login (and therefore non-public videos) and downloading whole playlists, so it might make sense to merge some stuff from there.
My extractor is a little bit more hasitly put together though, since it extracts less metadata. Another tidbit is that my extractor uses the direct .mp4 download instead of the .m3u8 URL. I don't know which is better, or if it wouldn't be best to support format selection.
My extractor can be found here.

My thoughts on this:
Would it be preferable to have a Paella/Opencast extractor, and subclass all other extractors of TU Graz and other universities from that one? Is there a common baseline API spec available?
"Opencast" (and its web-player "Paella") is the video system used by many universities, and it wouldn't make sense to re-implement it for every university.
The only thing changing (I assume) would be the authentication.
Opencast offers both mp4 and m3u8 downloads, supporting both would probably be a good idea.
Also noteworthy: Opencast has an API which can be queried for all m3u8 and mp4 (and other supported formats), so a typical "extraction" may not be necessary in most cases. (Edit: this is a (maybe custom?) API at TU Graz)
You can find more information about TUbe of TU Graz in this talk: https://replay.teltek.es/video/58ab2c9aa7bc2836008b4571. It describes the usage of Opencast with the Paella video player at TU Graz. (Other universities may use very similar setups).
FInd more information about Paella on its homepage: https://paellaplayer.upv.es/features/ (for HTML-based extraction for a fallback if the API is (maybe in the future no longer) available)
And while there are REST-API docs for Opencast, they are not very useful, as the API (at least for TU Graz) is not publicly accessible. Instead, they offer their own API, used for searching.
The Paella extractor may be able to handle the websites of many universities, if it's HTML based. And maybe some other institutions offer a limited public API of Opencast directly. May be worth a look.
While there may be many custom APIs, if a user provides a link to a webpage containing a Paella player, we could extract the underlying video source from there.

@Bernd-L thank you for these insights!
From my research on tube.tugraz.at up until now, there seems to be very little information in the HTML sent in response to the video URL. Most of the content gets loaded by JavaScript via the (most likely nonstandard) API.
So while yes, having a generic Paella extractor would be great, it might be that tube.tugraz.at is almost too custom. But I do think it's worth looking into this.
As foe the format extraction: My branch currently queries the API and there definitely is a list of formats in there. The proof of concept version I have now just takes the first format, but this can be easily extended to multiple format choices, with a potential fallback to @ruru4143's m3u8 URL guessing if the API is not accessible (The m3u8 playlist manifest URL only depends on the video id and is often accessible even when the API is not).
Since paella seems to support quite a few formats (full mp4, HLS, HDS, RTMP, DASH, SMOOTH), which of these would be worth extracting into the format field?

the full mp4 formats sometimes come in multiple bitrates, which could be used for high / low quality selection
I understand too little of the other streaming formats to know whether they support quality selection, but they only show up once in the format list

tube.tugraz.at (maybe paella in general) also has one other peculiarty: Multiple video streams. For some lectures, the presentation and presenter camera are both used, and all of the format sources are then available twice. The simplest solution would be to default to the presentation if available, and offering the presenter view as a separate format option if both are available. A more complex solution would be to use ffmpeg to render a video with both views, but that is probably too complex (and will also take longer than the average user has time for. Two downloads and ffmpeg rendering?)

@Bernd-L: The page with the talk you linked (that one from replay.teltek.es) seems to use a paella player with a completely different API and response JSON structure. So having a common extractor for tugraz and that one does not seem feasible without effectively making a franken-extractor that is effectively two different extractors

@Bernd-L: The page with the talk you linked (that one from replay.teltek.es) seems to use a paella player with a completely different API and response JSON structure. So having a common extractor for tugraz and that one does not seem feasible without effectively making a franken-extractor that is effectively two different extractors

Well, that resolves this idea; but thanks for looking into it!

* I understand too little of the other streaming formats to know whether they support quality selection, but they only show up once in the format list

Follow-up on this one: some videos also have multiple RTMP streams with varying bitrate, and so far every (?) listed format has a video and audio bitrate field. I've had no success at all downloading from the RTMP streams though, so I'll be skipping those for now.

Quick organisational info: I'll be cleaning up my branch to conform to general coding style and add @ruru4143's additional metadata extraction and these new insights, and once that is done I can make a PR for that.

@Ferdi265 How would you handle multiple video streams (ie Presenter, Blackboard, Screen)?
The Paella website has an example with four video streams and another one with three streams.
Do you want to create separate mp4 files? Should they be integrated into one file as separate video-tracks? What if only one of those videos contains an audio track with actual audio in it? How would youtube-dl find the one to use the audio from? Or should all audio tracks get put into one mp4 file as separate audio-tracks?
Should an option to split the video streams into several files be provided to the user? And would one audio-track (which one?) be used in both files (assuming the recording only includes the actual microphone from the lecture hall in the presenter recording, but not in the screen capture)?

closed due to:  #29160


add file mode parameter to handle .toml config files in flask.Config.…

fixes #4989
Added optional boolean argument "text" (False by default).

Also, you need to fix the failing test, it should only run on Python versions that have tomllib.

Sorry, I think you've sort of misunderstood the feedback you got. The parameter should still be text=True.

I understood, but as @RonnyPfannschmidt mentioned, "binary" seems clearer.
Because then "text = False" do not implicitly indicate we want to tread the file as binary


MathML in switch does not render properly

from Jean Kaplansky:
I have a sample book our team put together with MathML switching to support images where MathML is not supported.
Things work as expected in VitalSource Bookshelf Desktop - MathML renders.
I expected the MathML to render in the latest Readium Chrome extension too, but the switch is showing the images instead of the MathML. It's been a while since I've looked at this stuff, so I can't honestly remember if I've ever seen MathJax live and running in Readium or not, but I thought it was supposed to be...
all of this came to light because iBooks DOESN'T support MathML switching, so we're getting all the math 2X - both as MathML and images over there (but this is irrelevant to my Readium expectation).
Is there something about EPUBs built with MathML switches for displaying images that would cause Readium to not go the MathJax path? Or did MathJax get disabled at some point and I missed it? Or, as was the case yesterday, is MathJax still having CDN problems?
Please advise? I've attached the sample in question to this message. FYI - it validates with EPUBCHECK, but I haven't cracked the file to see if it's really what it should be regarding the switch set up in the OPF, yet.

From Daniel Weck:
I believe the old (original) Chrome extension used to support EPUB switch functionality. There's remnents of that feature in the new Chrome extension, but it's incomplete as far as I know.

The issue here appears to be the lack of support for the switch element (as Dan indicates).  There is nothing necessarily specific to MathML - it just demonstrates the problem.

BTW, iBooks (at least on desktop, v. 1.1.1) shows BOTH the image and renders the MathML... :-)

Issue specific to EPUB "switch" functionality moved here: https://github.com/readium/readium-shared-js/issues/171

This fix should address this issue:
https://github.com/readium/readium-shared-js/pull/205
To be tested...


Add support for standard main menu

Love the theme. This allows main menu elements in the standard Hugo config format: https://gohugo.io/templates/menu-templates/#site-config-menus

Hi, thanks for the PR! This looks good -- might as well go the full mile and move all menu functionality to the officially supported method. I've removed the old menu functionality, so now all menu items should be created through setting menu options either on individual pages or in the sites configuration.


Parse "_embedded" variable to gain access to featuredmedia.source_url

In wordpress rest api if I add ?_embed at the end of the request, wordpress includes some other information to json at __embedded. Most notable is complete information about wp:featuredmedia including link to featured image (wp:featuredmedia[0].source_url). This bypasses the need to install separate plugins on host wordpress to include featured media url in json (which I am not allowed to the in some cases).
The

Hi @matejmosko ,
Good to know, I think i could check on this key for the featured image property in the response model to include the url in the said key there.

@matejmosko , I was just checking on the same and I found the resulting API response size has increased by almost 100KB.
Due to this, I have not included this by default. instead, you can fetch this from the raw response object which is now returned from the WordpressResponse like so:
final client = WordpressClient.initialize(
    baseUrl: baseUrl,
    bootstrapper: (bootstrapper) => bootstrapper.withDebugMode(true).build(),
  );

  final response = await client.posts.retrieve(
    RetrievePostRequest(
      id: 1,
    ),
  );

  response.map<void>(
    onSuccess: (response) {
      final featuredImageLink = castOrElse(
        response['_embedded']['wp:featuredmedia']['link'],
      );

      print(
        'Featured image link: $featuredImageLink',
      );
    },
    onFailure: (response) {
      print(response.error);
    },
  );

Note: castOrElse is an in-built helper method to cast to a type with optional parameters to transform the value or pass a default value incase the casting fails.
Try this out and let me know how it goes for you. it is now available on pub.dev. use 8.3.10 version.

Hi, thanks. Meantime found out I can force ListPostRequest to fetch _embedded this way
    final request =
        ListPostRequest(page: page, perPage: 20, extra: {"_embed": "wp:featuredmedia"})

And afterwards I access the value anywhere using extension on Post type:
extension FeaturedImage on Post{
  String featuredImageSourceUrl(){
      return this["_embedded"]["wp:featuredmedia"][0]["source_url"];
  }
}

But your solution for a single post works . Thank you very much for your work.
P.S. When I add value wp:featuredmedia to _embed, the size footprint is significately smaller (only 3KB for me)) I have found this out today after you pinpointed the size.

Thank you again. Already wrapped :).


Link title is not retrieved for node after it's saved

MeganavItem is saved correctly into db (with values applied during first configuration). But when we're retrieving data for the 2nd time it's changed = empty, cause we're building items using API method which is getting values from IPublishedContent, creating angular model based on them and just extending it with the other data.

Thanks for raising this @mzajkowski!
This issue was fixing in the v1.1.0 release which just launched! You can download right now from all the usual places including NuGet and Our Umbraco
Cheers! :-)


Error running TLint inspection: this.file.parent must not be null

Using PHPStorm with TLint plugin
After a while I get this error message:
Error running TLint inspection: this.file.parent must not be null
Restarting PHPStorm solves the issue

What OS version / PhpStorm version are you running?

OS
Linux Mint 5.0.0-29-generic #31~18.04.1-Ubuntu SMP Thu Sep 12 18:29:21 UTC
2019 x86_64 x86_64 x86_64 GNU/Linux
PhpStorm 2019.2.3
Build #PS-192.6817.20, built on September 25, 2019
Licensed to Mark Tetrode
Subscription is active until February 11, 2020
Runtime version: 11.0.4+10-b304.69 amd64
VM: OpenJDK 64-Bit Server VM by JetBrains s.r.o
Linux 5.0.0-29-generic
GC: ParNew, ConcurrentMarkSweep
Memory: 1981M
Cores: 8
Registry: run.processes.with.pty=TRUE, ide.balloon.shadow.size=0
Non-Bundled Plugins: PHPUnit code coverage, com.intellij.ideolog,
com.jetbrains.tlint-plugin, com.atlassian.bitbucket.references,
com.dmarcotte.handlebars, de.espend.idea.laravel, ru.adelf.idea.dotenv
Mark

32 474 92 06 59
.

On Fri, 11 Oct 2019 at 15:43, Logan Henson<EMAIL_ADDRESS>wrote:

What OS version / PhpStorm version are you running?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/tightenco/tlint/issues/103?email_source=notifications&email_token=ABM244OV3C2RBP75E2XTYO3QOB7IPA5CNFSM4I2WUUWKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEBABGPQ#issuecomment-541070142,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABM244OK7O4FQDUFHWOJIADQOB7IPANCNFSM4I2WUUWA
.

Moving this issue to https://github.com/loganhenson/tlint-plugin/issues/8


Add Arnold-Winther element

See section 3.4.4 of Common and Unusual Finite Elements (Kirby et al, 2012) and Mixed finite elements for elasticity (Arnold and Winther, 2002)

Looks like there's conforming and nonconforming variants of this one

Looks like there's conforming and nonconforming variants of this one


Add address filter

A filter widget makes it possible to filter advertisements on Bluetooth address by entering address=XX:YY:....

Codecov Report

Merging #12 (58ae080) into main (6fa292b) will decrease coverage by 3.45%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main      #12      +/-   ##
==========================================
- Coverage   43.11%   39.66%   -3.46%
==========================================
  Files           5        5
  Lines         218      237      +19
  Branches       47       53       +6
==========================================
  Hits           94       94
- Misses        124      143      +19

Impacted Files
Coverage Δ

src/humble_explorer/app.py
0.00% <0.00%> (ø)

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


Couldn't find manifest for 'bittorrent'.

Hi trying to install bittorrent package using scoop.
xxx@DESKTOP-45TF4RE MINGW64 ~
$ scoop bucket list
bittorrent
xxx@DESKTOP-45TF4RE MINGW64 ~
$ scoop install bittorrent
WARN  Bucket '' not installed. Add it with 'scoop bucket add ' or 'scoop bucket add  '.
Couldn't find manifest for 'bittorrent'.
Any clue.
Note:- I did a scoop update as well
xxx@DESKTOP-45TF4RE MINGW64 ~
$ scoop update
Updating Scoop...
Updating 'bittorrent' bucket...
Scoop was updated successfully!
xxx@DESKTOP-45TF4RE MINGW64 ~
$ scoop install bittorrent
WARN  Bucket '' not installed. Add it with 'scoop bucket add ' or 'scoop bucket add  '.
Couldn't find manifest for 'bittorrent'.

What is URL of that "bittorrent" bucket??

What is URL of that "bittorrent" bucket??

https://github.com/effigies/BitTornado

You totally missed how scoop works. You cant just add some repository with application and expect to use it. Buckets contains manifests. Manifests are files with everything needed to download program.
Bittornado is some Python library, which could be used by directly calling python script. If you want to add it to scoop request it here and maybe someone who know how this app should look like / work will help you.

Basically I want to use btshowmetainfo
As specified here Im trying to install the package bittorrent as specified here
https://unix.stackexchange.com/questions/31486/is-there-a-tool-to-view-a-torrent-file

Use aria2
scoop install aria2 and then https://unix.stackexchange.com/a/197842

Use aria2

Oh woww.  Thanks a million :-)

Then please run scoop config aria2-enabled false. It will cost you less problems.


[CT-2817] [Feature] Python Models - MS SQL Server

Is this your first time submitting a feature request?

[X] I have read the expectations for open source contributors
[X] I have searched the existing issues, and I could not find an existing issue for this feature
[X] I am requesting a straightforward extension of existing dbt functionality, rather than a Big Idea better suited to a discussion

Describe the feature
Implementation of Python models for Microsoft SQL Server.
Describe alternatives you've considered
T-SQL but this was not fruitful as the process requires a variable number of passes over a dataset.
Currently attempting to stitch a macro together but it isn't clean nor is success guaranteed.
Who will this benefit?
Anyone using Microsoft SQL Server looking to perform calculations that are more suited to Python than SQL.
Are you interested in contributing this feature?
Yes. Not sure where to start.
Anything else?
No response

Hello! Thanks for opening :) this issue is actually better suited for the dbt-sqlserver adapter repo. Looks like I can't transfer an issue to a repo owned by another organization, so I would recommend the 'ole copy & paste method to move the issue over there.
We would love to see Microsoft SQL Server support for python models, but dbt is limited to the features available within the data platform itself - so Microsoft would need to first support this service before dbt-sqlserver could implement.


Sign Up/Log In Page

Assets
V1 Screen
Live Preview of V1
V2 Screen - If integrating the sign up/log in widget isn't something that can be easily/quickly integrated, then let's go with building this version first, then work on integrating the widget later

Once completed, clicking on the "Sign Up" button can go directly to the sign up page; doesn't need to go to the signup_intermediate page

The "How It Works" image is downloadable directly in the Zeplin files

@AmandaBirmingham

@specsdom  I have investigated AuthRocket's customization options and discussed with @dhakim87, and have concluded that, unfortunately, it is not possible to do the V1 Screen.  Visually, AuthRocket allows us to customize ONLY the top-left logo and the color of the submit button.  Additionally, for security purposes, an AuthRocket page cannot be loaded inside an iframe, so it is not possible for us to embed the AuthRocket page within a larger page containing the desired Microsetta chrome. Given this, I will work on setting up the V2 Screen.
Please specify what you envision happening when the user clicks on the down arrow next to the (I assume?) profile icon ... I do not see that menu's content specified on zeplin, so I cannot scope its feasibility.

@specsdom Also, my understanding is that the V2 Screen is what un-logged-in users should see instead of the current home page (shown below); is that correct?

Dropdown Menu on this page would show “Sign Up/Log In” options (Dan had accounted for this already so these options should be live if a user isn't logged in)
V2 is meant to be shown instead of what is currently shown, so your understanding is correct


grpc: 0.0.10-0 in 'melodic/distribution.yaml' [bloom]

Increasing version of package(s) in repository grpc to 0.0.10-0:

upstream repository: https://github.com/CogRob/catkin_grpc.git
release repository: https://github.com/CogRobRelease/catkin_grpc-release.git
distro file: melodic/distribution.yaml
bloom version: 0.6.7
previous version for package: 0.0.9-2

grpc
* Follow upstream grpc 1.15.1 (#33 <https://github.com/CogRob/catkin_grpc/issues/33>)
  * Sync grpc to upstream 1.15.1
  * Typo fix
* Contributors: Shengye Wang

came comment as https://github.com/ros/rosdistro/pull/19184#issuecomment-424823786


Can't create thread via DataFixtures

While trying to create thread/comment with Symfony's DataFixtures, RoleThreadAcl throws error.
  [Symfony\Component\Security\Core\Exception\AuthenticationCredentialsNotFoundException]
  The token storage contains no authentication token. One possible reason may be that there is no firewall configured for this URL.

Exception trace:
 () at /var/www/vendor/symfony/symfony/src/Symfony/Component/Security/Core/Authorization/AuthorizationChecker.php:57
 Symfony\Component\Security\Core\Authorization\AuthorizationChecker->isGranted() at /var/www/vendor/friendsofsymfony/comment-bundle/FOS/CommentBundle/Acl/RoleThreadAcl.php:102
 FOS\CommentBundle\Acl\RoleThreadAcl->canCreate() at /var/www/vendor/friendsofsymfony/comment-bundle/FOS/CommentBundle/Acl/AclThreadManager.php:125
 FOS\CommentBundle\Acl\AclThreadManager->saveThread() at /var/www/src/Bundle/CoreBundle/DataFixtures/ORM/LoadQuestionData.php:123

So what sould we do about that?

+1

@shibby did you find a way ?

@younessadmi i'm adding comment rows by creating Comment instances directly.
$comment = new Comment(); $comment->setUser($user)...


no_leaked_credentials doesn't disappear on the Security tab when "Hide met criteria" is active

no_leaked_credentials doesn't disappear on the Security tab when "Hide met criteria" is active.

This happens when viewing: https://bestpractices.coreinfrastructure.org/projects/33
on both Chrome and Firefox.

I think we'll need to implement #176.  I'm having trouble reproducing this in the development environment; if the environments were more similar, it'd be easier to reproduce and track down the problem.


Alignof operator

Is your feature request related to a problem? Please describe.
It used to be that every type in HLSL was 4 bytes in size. Even bools are 4 bytes. So you could assume that everything was 4 byte aligned. However, this is no longer the case. We now have types like float16_t which have an alignment of 2 and uint64_t which have an alignment of 8. This is important if we are using templated ByteAddressBuffer stores because we have to be able to align our writes. However, there is currently no way to query what a struct's alignment is, so writing generic code that makes use of byte address buffer stores is difficult. The best we can do is just align every unknown type by 8 bytes.
Describe the solution you'd like
I would like an alignof operator like in C++11 to simply query a type's alignment.
Describe alternatives you've considered
The only workaround now (that I know of) is to make all stores of unknown types be 8 byte aligned.

This is related to: https://github.com/microsoft/hlsl-specs/pull/154


[SPARK-45330][BUILD ]Test new ammonite and Scala 2.13.12

What changes were proposed in this pull request?

Why are the changes needed?

Does this PR introduce any user-facing change?

How was this patch tested?

Was this patch authored or co-authored using generative AI tooling?

test amm 2.5.11 with Scala 2.13.12 passed
https://github.com/LuciferYang/spark/runs/17221539327

will revert to test Scala 2.13.11, let's upgrade amm first

Thanks @srowen and @dongjoon-hyun


Replace only in string arguments

This fix blocking drupal/upgrade_status on PHP 8
The https://github.com/mglaman/phpstan-drupal/pull/161 missed to fix when arguments are int, bool, null
Related to #150

Basically \Nette\DI\Resolver::convertReferences() is only looking for string arguments so should not fire for other types so better to keep scalars as is

Related issue https://www.drupal.org/project/upgrade_status/issues/3100308#comment-14013212
This fix allows to pass tests

@andypost what module was causing this error or file scanned? I'd like to get a test added now that there are testing pipelines.

I rebased off of master so we get all of the new builds.

@andypost what module was causing this error or file scanned? I'd like to get a test added now that there are testing pipelines.

I used to run it on copy of upgrade_status testing modules (but it has changed since that and can't reproduce right now)

Just reproduced on dgo.to/cdn module
PHPStan command failed:

/var/www/html/web/vendor/bin/phpstan analyse --error-format=json -c /tmp/upgrade_status/deprecation_testing.neon /var/www/html/web/modules/cdn 2> /tmp/upgrade_status/phpstan_error_output
Command output:

Empty.
Command error:

TypeError thrown in /var/www/html/web/vendor/mglaman/phpstan-drupal/src/Drupal/DrupalAutoloader.php on line 182 while loading bootstrap file /var/www/html/web/vendor/mglaman/phpstan-drupal/drupal-autoloader.php: str_replace(): Argument #3 ($subject) must be of type array|string, bool given

I cannot directly reproduce here. @andypost what Drupal core do you have – is it 9.2.x-dev or 9.1.x-dev or a tagged release? Also are there other contrib installed? The extension aggressively parses all services.yml

Fixed merged error, trying again to see if it occurs.

As I got it merged, so only test case remains here
Ref https://github.com/mglaman/phpstan-drupal/pull/163/commits/c0ecb7380c0083b7345ca1ae14f5700cbfcf07a8

Watching your streaming atm!

I'm going to close this, it didn't crash. Just warnings about unsafe new static().
Note: Using configuration file /home/runner/drupal/phpstan.neon.
 ------ -----------------------------------------------------------------------
  Line   cdn_ui/src/Form/CdnSettingsForm.php
 ------ -----------------------------------------------------------------------
  51     Unsafe usage of new static().
         💡 See:
         https://phpstan.org/blog/solving-phpstan-error-unsafe-usage-of-new-st
         atic
 ------ -----------------------------------------------------------------------

 ------ -----------------------------------------------------------------------
  Line   cdn_ui/src/Form/ValidatableConfigFormBase.php
 ------ -----------------------------------------------------------------------
  32     Unsafe usage of new static().
         💡 See:
         https://phpstan.org/blog/solving-phpstan-error-unsafe-usage-of-new-st
         atic
 ------ -----------------------------------------------------------------------


saturation and brightness

Can you add saturation and brightness ?
I have read the issue #8, but I think it will be great if you add this support.

I see the need of using saturation and brightness options in real life. Currently I am unsure about how to implement them. @L1cardo would you expect them being used within two sliders (as https://github.com/benjiburgess/ColorPicker does) or in a different input form? Thrilled to hear your ideas.

would be some known patterns

@hendriku Two sliders as https://github.com/benjiburgess/ColorPicker will do.
I can not see a better input way.

Alright so the vision is clear ;) Will only get into start working on this from december. If anyone creates a PR earlier I'd be happy to merge

👍 Vote on Third one

Sorry to ask，but are you working on this？

@L1cardo nothing to be sry about! Unfortunately I did not manage to invest time in this :'( Any PR adopting the solution from benjiburgess/ColorPicker will be happily reviewed and merged. As his fork and my package share the same license it would also be no problem to create a merge from his fork to my original package.


Insert all 130+ indicators to pandas dataframe in a easy way

Hi,
i'm adding every indicator to my dataframe as for example:
df['sma10'] = ta.sma(df["close"], length=10)
...
Does anyone know of any way to be able to enter all the indicators in a simple way without having to go one by one?
I would greatly appreciate it.

Create a pandas TA Strategy and list all the indicators you need. Then you will have them all but that is a bad idea. It will take forever to calculate all indicators and most likely you will need them several times with different periods at least for the moving averages. It's better to select and use just a few for a specific strategy to be easy and fast to optimize their parameters when backtesting.

Hello @Pl0414141,

Does anyone know of any way to be able to enter all the indicators in a simple way without having to go one by one?

Well the simple standard way to include all the indicators with their default values, length, mamode, ... bound to the DataFrame is to use the All Strategy as described on the README under Types of Strategies
import pandas_ta as ta

df = # ohlcv
df.ta.strategy(verbose=True) # Default is ta.AllStrategy
# or more explicitly
# df.ta.strategy(ta.AllStrategy, verbose=True)
print(df)

Now if you have Custom parameters, length, mamode, ..., then you will have to define them individually as @AGG2017 has commented. It is best to create a Custom Strategy for this purpose and is defined similarly to json format. Here are some Custom Strategy examples.
For example (from that Notebook):
# Create a Strategy
momo_bands_sma_ta = [
    {"kind":"sma", "length": 50},
    {"kind":"sma", "length": 200},
    {"kind":"bbands", "length": 20, "ddof": 0},
    {"kind":"macd"},
    {"kind":"rsi"},
    {"kind":"log_return", "cumulative": True},
    {"kind":"sma", "close": "CUMLOGRET_1", "length": 5, "suffix": "CUMLOGRET"},
]
momo_bands_sma_strategy = ta.Strategy(
    "Momo, Bands and SMAs and Cumulative Log Returns", # name
    momo_bands_sma_ta, # ta
    "MACD and RSI Momo with BBANDS and SMAs 50 & 200 and Cumulative Log Returns" # description
)
print(momo_bands_sma_strategy) # Check it was correctly created

df = # ohlcv

# To run your "Custom Strategy"
df.ta.strategy(momo_bands_sma_strategy, verbose=True)
print(df)

Hope this helps!
Kind Regards,
KJ


Ensure selection is definitely collapsed - fixes Blink Shadow DOM bug

This checks that a selection's ranges are all collapsed instead of just relying on the isCollapsed property. This can occur in both v0 and v1 Shadow DOM implementations in Chrome - see this Blink bug.
Fixes this issue

Thanks! Merged and followed up with 1d6df3e (since browser don't actually support multiple selections, checking range 0 is enough).

@marijnh yeah I didn't quite know what the deal was with multiple ranges - thanks for clearing that up :)


Send only properly formatted NPIs to certain STLTs

Background
At least 1 STLT (IL) has requested to receive a value in the ordering_provider_id field only when the value provided is a properly formatted NPI.
Story
As a STLT, I want to be able to decide whether I receive a value in the ordering_provider_ID field and associated fields* if the ordering_provider_ID provided is not a properly formatted NPI so that I am only receiving values that are meaningful.

ordering_provider_id
ordering_provider_id_authority
ordering_provider_id_authority_type

Given that a message is received into Reportstream with something other than a properly formatted NPI in the ordering_provider_ID field, and when the receiving STLT is Illinois, then the 3 fields listed above should not be populated with any values in the outbound message

Deployed


⚠ Elibs has degraded performance

In 836fcce, Elibs (https://elibs.unmer.ac.id) experienced degraded performance:

HTTP code: 200
Response time: 11778 ms

Resolved: Elibs performance has improved in f1a4e60.


Fix tagged builds

Fixes #37
Also removed unneeded whitespaces that popped up on my linter

@revett this PR fixes #37
Tried it out on my testing fork by pushing a new version and builds now pass https://travis-ci.com/be-neo/neo-local/builds/82244444


Struggling to make it work with turborepo (mono-repo)

I am setting up the UI as packages in the turborepo. However, I keep getting the following error:
@app/web:build: ../../packages/ui-shadcn/src/components/react-hook-form/form.tsx
@app/web:build: Module not found: Can't resolve '@/src/components/ui/label'

Any example repository show-casing project setup with common shared UI package in turbo?

Hey, I recently made this turborepo-starter with shadcn-ui pre-configured. Check it out and let me know if you find any issues.

https://github.com/juliusmarminge/acme-corp that i use

@destino92  Why do you  bundle and build the ui package locally? doesn't it add extra complexity?

Also am struggling with this.

@singh05 did you manage to solve this issue? I'm facing the same problem and wondering if there's a fix.

@destino92  thanks a lot man i was migrating things from nextjs to turbo, but i had only t3 turbo for reference, and was stuck with tha shadcn since it did't had that, this saved,
bro this julius guy man is he the next elon or what
@singh05 thanks for opening this issue

@dimfu did it resolved?

I have followed typehero repo to setup shadcn/ui with turborepo
https://github.com/typehero/typehero

This issue has been automatically closed because it received no activity for a while. If you think it was closed by accident, please leave a comment. Thank you.


How do I use vcpkg without admin?

Would love the docs on the readme to say what to do if you don't have admin as otherwise we're just stuck at the 'vcpkg integrate install' step. Even if there's a note there saying if you havn't got admin forget trying to do anything with vcpkg would be helpful - at least I'd know.

If you do not have admin right, you can adjust the additional linker directory per project basis. Then, you do not need vcpkg integrate install.

You can always use:

.\vcpkg.exe integrate project which creates a nupkg in <vcpkg-root>\scripts\buildsystems\vcpkg.<Drive>.<Path>.1.0.0.nupkg
Then when you create a new project you can add the nupkg using the instructs received when the nupkg was created.
I personally do this as I have VS2017 and VS2019 along with a test/prod version of each. The vcpkg.exe integrate install can only integrate one instance of vcpkg to all versions of VS2015, 2017 and 2019 installed on the system.
Example:

With a project open, go to Tools->NuGet Package Manager->Package Manager Console and paste:

Install-Package vcpkg.D.Git.vs2019.test -Source "D:\Git\vs2019.test\scripts\buildsystems"

Just beware of the ports requiring a manual_linkage, like SDL2main, they can be found in:

<vcpkg-root>\installed\<triplet>\lib\manual-link for release
<vcpkg-root>\installed\<triplet>\debug\lib\manual-link for release

You will need to add them manually in the C/C++ linker lib properties along with adding the path on linker additional lib paths for each build type in your project, aka release, debug.

vcpkg integrate install --debug
[DEBUG] Using vcpkg-root: E:\tools\vcpkg
Applied user-wide integration for this vcpkg root.

All MSBuild C++ projects can now #include any installed libraries.
Linking will be handled automatically.
Installing new libraries will make them instantly available.

CMake projects should use: "-DCMAKE_TOOLCHAIN_FILE=E:/tools/vcpkg/scripts/buildsystems/vcpkg.cmake"
[DEBUG] E:\tools\vcpkg\toolsrc\src\vcpkg\commands.integrate.cpp(294)

https://github.com/microsoft/vcpkg/blob/f7fb56decd9d03cf79a1d23f44daa3a23560b995/toolsrc/src/vcpkg/commands.integrate.cpp#L170
https://github.com/microsoft/vcpkg/blob/f7fb56decd9d03cf79a1d23f44daa3a23560b995/toolsrc/src/vcpkg/commands.integrate.cpp#L175
dir C:\Program Files (x86)\MSBuild\Microsoft.Cpp\v4.0\v140\ImportBefore\Default
the folder C:\Program Files (x86) requires administrator rights
@

On Windows (providing you have the Visual Studio C++ workload installed) I would try something like :
SETLOCAL
CALL D:\Programs\MSVS2019\VC\Auxiliary\Build\vcvars64.bat"
SET SRCROOTDIR=D:\test
SET VCPKGDIR=%SRCROOTDIR%\vcpkg
SET QTDIR=%VCPKGDIR%\installed\x64-windows
CD %SRCROOTDIR%
git clone https://github.com/Microsoft/vcpkg
cd vcpkg
CALL .\bootstrap-vcpkg.bat
vcpkg install somelibrary:x64-windows someotherlibrary:x64-windows

followed by something like the following for a Qt application:
cd .. && mkdir build && cd build
cmake .. -G "Visual Studio 16 2019" -A x64 -DQt5_DIR:PATH="%QTDIR%\share\cmake\Qt5" -DQMAKE_EXECUTABLE:FILEPATH="%QTDIR%\tools\qt5\bin\qmake.exe"

you then issue an msbuild command to build the .sln file which cmake generated for you in the previous step.
You don't need to vcpkg integrate install and I never do it myself.

@heydojo
Please let me know if you still have problems about this issue.

I'm closing this issue for now. Please feel free to reopen this if it is still a problem for you.


Using up and down arrows in an ItemList makes it scroll in strange ways

Godot 3.2 beta3
With an ItemList like this one:

If I use up or down arrow, it ends up in this state:

There is no reason for such scroll to happen here.
Project:
ItemList.zip

Reproduced on Linux Godot 3.2 beta3, not reproduced on Godot 3.2-beta2.


Show fetched row count in status bar or results pane

SQL Operations Studio Version: 0,25,4

It is a really interesting tool!
But, where is a count of fetched rows after select operator performed like in SSMS?
It is a useful sometimes as for me
I see only total execution time...

@mrlife thanks for the clarification.  I've updated the issue title to match your description.  We'll consider this for a future release.