KubernetesClientException - Configured service account doesn't have access

I'm using a Kubernetes cluster created with kubeadm.
I'm hitting the following:
io.fabric8.kubernetes.client.KubernetesClientException: Failure executing: GET at: https://kubernetes.default.svc/api/v1/namespaces/default/configmaps/my-config. Message: Forbidden!Configured service account doesn't have access. Service account may have been revoked. configmaps "my-config" is forbidden: User "system:serviceaccount:default:default" cannot get configmaps in the namespace "default".

How do I access ConfigMaps on Kubernetes (Documentation only mentions steps for OpenShift)

@dkirrane im having the same problem! Did you solve it?

hello guys, i have no found any property in spring.cloud.kubernetes related to authentication via token.
Any advices?

Hey Guys, Also having this problem, any solution?

Same issue...
2020-10-29 19:28:18.188 WARN 1 --- [ main] o.s.c.k.config.ConfigMapPropertySource : Can't read configMap with name: [accounts] in namespace:[XXXXX]. Ignoring.
io.fabric8.kubernetes.client.KubernetesClientException: Failure executing: GET at: https://<IP_ADDRESS>/api/v1/namespaces/XXXX/configmaps/accounts. Message: Forbidden!Configured service account doesn't have access. Service account may have been revoked. configmaps "accounts" is forbidden: User "XXXXX" cannot get resource "configmaps" in API group "" in the namespace "XXXX".

apiVersion: v1
kind: ConfigMap
metadata:
  name: accounts
data:
  spring.datasource.url: <REMOVED>

Fixes the problem:
https://docs.spring.io/spring-cloud-kubernetes/docs/current/reference/html/index.html#service-account


dattcore: Distinguish between mainnet and testnet in BIP44 Wallet

Closed by https://github.com/dattnetwork/datt/commit/901f5a9be18e3d103392930a7df232d6e860808f


Refactor CQuorumBlockProcessor and CDeterministicMNManager

It's currently pretty confusing the way pindexPrev is used in these two imo. This PR basically switches it to pindex when processing current block and keeps pindexPrev only for places where we try to figure out commitments for the next block (i.e. GetMinableCommitment/Tx). Similar logic was applied to CDeterministicMNManager too. IMO it makes it much easier to read/understand.
Also dropped few unused statements while at it.

Just FYI and to better understand why this refactoring is a :+1:: The initial reason I worked with pindexPrev instead of pindex in the past was that an earlier implementation of DIP4 called CDeterministicMNManager::ProcessBlock to calculate the MN list merkle root for the DIP4 coinbase. That lead to pindex not being known yet it was happening in the middle of block creation. I later changed this to use BuildNewListFromBlock, so I unknowingly removed the need for pindexPrev in ProcessBlock and thus didn't realize that the pindexPrev was left there in an unnecessarily complicated way.

Wow, I didn't expect it to be merged yet 🙈  I hope you actually reviewed it before merging because I forced pushed a reworked version which only touches ProcessBlock - it turned out that we can't actually apply the same thing to GetQuorumBlockHash and some other functions which are used to create new block while mining. But the version as it's merged now should be ok :)
I have more changes I was preparing to push which would switch some other functions from using CBLockIndex* to block height but I'm not sure if I'm happy with the result.


x509.UnsupportedExtension is gone in cryptography >= 2.1

This fixes the issue reported in #1341

Thanks @jlaine. I'm not sure we can just remove this without increasing the minimum version we require in setup.{py,cfg}. That said, I wonder if we could alias the exception on 2.2 or newer such that we can handle older versions and newer versions

Requiring the latest version of pyopenssl/cryptography seems pretty reasonable. (In fact the latest version of pyopenssl does require the latest version of cryptography.)
If you want to alias it instead, I guess something like this would work:
try:
    from cryptography.x509 import UnsupportedExtension
except ImportError:
    # Putting this in an except: will never catch anything, because no-one ever throws it
    class UnsupportedExtension(Exception):
        pass

The try/except is one option; does cryptography have a base exception class that we could import instead of each of these three?

The current release of pyopenssl only supports cryptography >= 2.1.4, and the next one will support only >= 2.2.1, do we really want to support older versions? This would means explicitly testing against these older versions.

do we really want to support older versions?

I'm pretty sure we already do our best to test older versions. The short of it is that downstream packagers cherry-pick (in my experience) what things they actually update. Things like cryptography and pyopenssl make it hard for us to raise our minimum versions if their already packaged software works fine with an older version and would break by us updating our minimum requirement.
We can try to push the entire world forward, but that will be very unpleasant for the maintainers of this project.

I was looking throught the travis config files and I don't see any entries for testing different pyopenssl / crypto versions, can someone point me to the right place maybe?

It would also be really nice if this PR had a test that exercised that except: block, so we don't hit this again.
I asked about this on #cryptography, and @reaperhulk generously made this cert with a duplicate extension, so it should trigger that "A problem was encountered with the certificate..." block:
https://gist.github.com/reaperhulk/70f2c4daba422a7f83965111cc2134e3

Codecov Report

Merging #1342 into master will decrease coverage by 3.12%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1342      +/-   ##
==========================================
- Coverage     100%   96.87%   -3.13%
==========================================
  Files          21       21
  Lines        2014     2014
==========================================
- Hits         2014     1951      -63
- Misses          0       63      +63

Impacted Files
Coverage Δ

urllib3/util/selectors.py
81.53% <0%> (-18.47%)
:arrow_down:

urllib3/util/wait.py
92.85% <0%> (-7.15%)
:arrow_down:

urllib3/util/connection.py
97.87% <0%> (-2.13%)
:arrow_down:

urllib3/_collections.py
99.32% <0%> (-0.68%)
:arrow_down:

Continue to review full report at Codecov.

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

The osx tests seem pretty flaky, the failures don't seem to be related to these changes.

@jlaine the OSX tests are passing but codecov shows this as a decrease in overall project coverage. Please address that.

The codecov seems to be out of date, and based on the failing test run, the changes it is reporting are in unrelated files.
The exception I changed was in a branch which was previously untested, and I added a test for it.
The only part of the change which is likely to negatively affect coverage is the try/except I was asked to add around the import: as pointed out earlier, the building matrix always used the latest version of cryptography, which does not exercise the "except".

Ok I give up, the test suite on osx seems really random.

@jlaine your empty commit seems to have passed all the CI. What is random at the moment?

Hm, I guess I was wrong but I could have sworn both Python 3.6 and 3.7 had failed on osx.

It did. I re-ran it.

Thanks @jlaine!


Unable to build Producer crypto.getRandomValues is not a function

When following the Getting Started section I am getting the following error and I am not sure why?  I have tried both node v10.16.0 and node 8.
Error occurred:  TypeError: crypto.getRandomValues is not a function
at Object.randomBytes (/Users/me/workspace/test-project/node_modules/kafkajs/src/producer/partitioners/default/randomBytes.js:4:31)
at module.exports.size (/Users/me/workspace/test-project/node_modules/kafkajs/src/producer/partitioners/default/randomBytes.js:29:31)
at /Users/me/workspace/test-project/node_modules/kafkajs/src/producer/partitioners/default/partitioner.js:21:17
at module.exports (/Users/me/workspace/test-project/node_modules/kafkajs/src/producer/index.js:41:23)
at Client.producer (/Users/me/workspace/test-project/node_modules/kafkajs/src/index.js:91:12)
at Object.postMessagesToQueue (/Users/me/workspace/test-project/src/services/kafka_service.js:14:38)
at Object.exports.handler (/Users/me/workspace/test-project/src/cso_runner.js:13:24)
at [eval]:1:29
at Script.runInThisContext (vm.js:122:20)
at Object.runInThisContext (vm.js:329:38)

Are you perhaps running in the Node REPL? If so, this was fixed in #462. You can try running the pre-release version 1.11.0-beta.10 to see if that fixes it.

That was it thank you for your help


External resources CRUD

Considering running the following: any objections/recommendations?  @exbinary @armahillo
rails g scaffold ExternalResource name:string website_url:string facebook_url:string phone:string street_address:string city:string state:string zip:string description:string display_on_website:boolean youtube_identifier:string

Sounds good.


Invalid NotificationCategory constructor

Not always array is given whe model is being created, therefore we need to add this adjustment.
Error I am seeing:
Type error: Argument 1 passed to Fenos\Notifynder\Models\NotificationCategory::__construct() must be of the type array, none given, called in

Coverage remained the same at 81.33% when pulling 3da719148a30632493ceda4d52eed487578a8311 on sakalauskas:patch-1 into 9f7ef03f9321935b1cf54ac925d3ac24e1357066 on fenos:version-4.

Thanks for this fix! :)

@Gummibeer  can you tag a release? :)


Fix PHP8 curl_init() return value

Context
Fixes: #337, #353, #349
In PHP8, calling curl_init() in VCR/Util/HttpClient, will return a CurlHandle object, which is not a resource anymore (See comment about PHP8.0 here). As a result the Assertion::isResource() in the next line fails, and an exception is thrown:

"Could not init curl with URL http://foo.com'

What has been done

Explicit check if returned value is of type CurlHandle::class, fallback to Assert::isResource() in other case

How to test

Notes

As i run into the same issue as here: Isn't this a less complete solution as https://github.com/php-vcr/php-vcr/pull/351 ?

I have just released version 1.6.0, which is to be used with PHP8.


Support some normal types e.g. "float"/"byte"/"short"/"decimal"

(you don't have to strictly follow this form)

Support some normal types e.g. "float"/"byte"/"short"/"decimal".

[ ] #71

A clear and concise description of what is the intended usage scenario is.

Describe the solution you'd like

A clear and concise description of what you want to happen.

Describe alternatives you've considered

A clear and concise description of any alternative solutions or features you've considered.

Additional context

Add any other context or screenshots about the feature request here.

https://github.com/Kyligence/ClickHouse/pull/128


no_std support

Hi, we wanted to use this crate in a no_std environment. Is it expected at some point?

Hey, I'd be happy to add support, but have to admit I don't have any experience with no_std environemnts. The plan would be to follow a similar path to chrono's own no_std journey https://github.com/chronotope/chrono/issues/336
This would probably take a while given other time commitments I have, so help is welcome. In the absence of a PR I'm happy to commit to adding it over the next few months.


When http response header Content-Length: -1,  io.Copy cause error:Expected EOF

When I had a attack with  reponse header:
Content-Type: -1

Will cause error: Expected EOF
in: lib/attack.go   function hit  io.Copy,   line 235

Contributions welcome!
On Thu, 7 Jan 2016 at 07:40, 痕无落<EMAIL_ADDRESS>wrote:

When I had a attack with header reponse :
Content-Type: -1
Will cause error: Expected EOF
in: lib/attackgo function hit ioCopy, line 235
—
Reply to this email directly or view it on GitHub
https://github.com/tsenart/vegeta/issues/166.

Can't reproduce. If you can provide a reproducible test case, please do! Closing for now.


SCD dataset download

你好，方便提供數據庫到 google drive 分享嗎？ 謝謝

Hi. Is it possible to have the password to download the dataset?

Please email us to request data and clarify your purpose.


Fine-tuning the weights of generative model?

Hi there, excellent job! Thanks for providing the code!
I have a question about fine-tuning. In Section 3.5 of the paper, I think there is a regularization term that can also optimize the weights of the original GAN. Where can I find this part in your code? Currently, I just saw the optimization of z,c, and ф. Did I miss something? Please correct me if I was wrong.
Thank you again!

There is a fine-tuning procedure in the paper.
I did not include the fine-tuning but it should be fairly straightforward to implement.
The main meat of the paper is in the optimization and the fine-tuning is just the icing on the cake.
If you think it would be helpful to be integrated into the codebase I can add it in my next push.

Yes, that would be helpful! Thank you so much!


Hideout profit screen is using a different "Sell on flea" value than other pages

Hideout Profit for Workbench shows 240R @ flea

Item's own page shows 85R @ flea

Top of item's page shows 85R as suggested sell price and 240R as suggested buy price

This may be because the suggested sell price and buy prices are reversed?
It does seem odd that you would "Buy high" and "Sell low", rather than the obvious "Buy low" and "Sell high"

There's probably some inconsistent usage, but there are two main numbers that can be used for the buy/sell prices: avg24hPrice and lastLowPrice.
One school of thought is that avg24hPrice should be the sell price and lastLowPrice should be the buy price because people will hunt for the best deals for components and list items for sale near the average price. The other way of looking at it is if you need to buy multiple of an item for a craft, you may end up buying more towards the avg24hPrice than the lastLowPrice, and since most people only have two sell slots, they are more likely to sell near the lastLowPrice number so the sale gets made relatively quickly and they can list another item for sale.
My thinking is more along the lines of the latter, but whatever is decided you're definitely correct that the numbers should be consistent across pages.

I think the thinking I've seen in the past in similar in-game markets that I agree the most with is that, the avg 24-h price is likely to be correct for high-demand items, because all the low-priced ones will disappear quickly and most people will try to get the most they can given the quick pace of sales.
High-supply items, however, tend to trend toward lastlowprice, because if you price higher than that you item either won't sell, or will not sell until the price rebounds.
We could pick based on some supply/demand metric, but I've never seen anyone do that, due to that being an unstable metric, so things would be even more confusing. Neither price is going to actually fit circumstances 100% of the time.
SOOOO... I think better/best logic might be that someone selling is likely going to TRY to get the highest profit possible, while someone buying is likely to try to get the lowest price possible. Whether this will be possible is based on a ton of other factors, but I'd lean towards using lastlowprice for "Buy" and avg24hprice for "Sell" pricing.
If we need to approximate the "value" of an item, avg24hprice is likely to be the more accurate "value" of the item, though, if what we're showing is more buy/sell agnostic.

It's an interesting idea to apply some kind of metric to switch the buyFor/sellTo prices between the average and low price, but I don't know if it's very feasible. It would also be a bit confusing for the API to return avg24hPrice as the sellTo flea price for some items and the lastLowPrice as the sellTo flea price for other items. I think it's best to keep the API consistent in what it's returning in that regard.
Of course everyone wants to get the highest price they can when they sell and the lowest price when they buy, but that just isn't feasibly some (maybe even most) of the time. The overwhelming majority of players only have two sell slots. That means people often don't have the luxury to list an item at the average price and then wait hours for it to sell.
Plus, as I mentioned, when someone is going to start a craft or do a barter that requires multiple of an item, they probably don't want to wait around until then can get all of that item at the lastLowPrice. As the buy multiple, their average purchase price will likely get closer to the avg24hPrice.
Sure, this may somewhat underestimate how much a crafted item can be sold for and it may somewhat overestimate how much it costs to complete a craft/barter, but that just ends up being a conservative profitability estimate, which I think is a good thing. If the barter/craft "profit" calculators were based on the absolute best-case scenario, that would be a bit misleading. Better to temper expectations about how profitable a craft is likely to be and have the user be pleasantly surprised than to give them unrealistic expectations and have them be disappointed.


[1.14.4] No config file being generated, crash when trying to create world

Forge version: 1.14.4-28.1.76
Mod version: 1.14.4-2.1.1
Mods List: Worldgen Block Replacer
latest.log
debug.log
crash report
What I did:
Attempted to create a world, and load it.
What happened:
The game crashed when attempting to load the world, saying that there was not a valid config file
What I tried:
Checking in configs/, and saves/MySave/serverconfigs/ for a config file
I did not find a config in either folder, so I tried creating both a wgblockreplacer.cfg, and a wgblockreplacer.toml containing the contents of this example config on your CF page, but to no avail.

it's worldgenblockreplacer-server.toml in saves/MySave/serverconfigs/ . Looks like that example config is for 1.12 or earlier, so don't use it.
I'm having trouble with the config too though - https://github.com/The-Fireplace-Minecraft-Mods/Block-Replacer/issues/12

Your initial config is set to replace stone with stone, which then shuts down the server because it assumes that was a configuration error that you'll want to correct before generating the world. Right before the crash, in the latest.log:
WorldGen Block Replacer is improperly configured. The replacement for (0) is itself.
And the crash itself says WorldGen Block Replacer is improperly configured, shutting down. See the logs to find out what wasn't configured properly.

Ah, thank you, maybe you could change the Curseforge page to reflect these changes, this really threw me for a loop.


ionic@nightly update related issues

Short description of the problem:
After installing the <ion-select> directive style doesn't work as expected.
This is how it looks like: http://i68.tinypic.com/23r8s9w.png
Notice when I hover of the ion-select it turns blue. In the prior version it doesn't do that, already tried it.
What behavior are you expecting?
It's the one that is shown in ionic v2 docs.
Steps to reproduce:

npm install ionic-angular@nightly

insert any relevant code between the above and below backticks

Other information: (e.g. stacktraces, related issues, suggestions how to fix, stackoverflow links, forum links, etc)
Which Ionic Version? 1.x or 2.x
ionic 2
Plunker that shows an example of your issue
For Ionic 1 issues - http://plnkr.co/edit/jnOrQRtE8rN2WCXDQYQY?p=info
For Ionic 2 issues - http://plnkr.co/edit/sSFR9jbfTsxgAmrm9hUR?p=info
Run ionic info from terminal/cmd prompt: (paste output below)

@Bandito11 hey! So yeah we are already aware of this issue and the next nightly release will have this fixed. Sorry about that! This is why there nightly releases and not stable releases (; haha


download udemy videos own section folder

Hello, I would like to download udemy videos inside own section folders. Youtube-dl downloads all videos inside same folder.
https://yadi.sk/i/p8KHU35GeHSCt

You may download udemy videos with Allavsoft.
It also download videos from MySpace, Google Video, Yahoo Video, Break.com and other more that 750+ video websites.

@muhasturk https://yadi.sk/i/p8KHU35GeHSCt is gone. ("Nothing found") If you have another example, please propose it here and I'll reopen the issue.

The original request was to be able to put downloads in separate folders by course chapters, e.g. https://www.udemy.com/java-tutorial/#/lecture/172757
~/Downloads/Chapter 1 - Introduction/
    Lecture 1 - Introduction and Installation
    Lecture 2 - What Java Is and How It Works
    Lecture 3 - Getting a Job and What to Study After Completing Basic Java
    ....
~/Chapter 2 - Programming Core Java
    Lecture 6 - A Hello World Program
    ...

It works very good.

This feature will be supported in the next version via output template.
Example: youtube-dl -o '%(chapter_number)s - %(chapter)s/%(title)s.%(ext)s' -u user -p pass https://www.udemy.com/java-tutorial/.


Implementation of frontend

Deep Learning Simplified Repository (Proposing new issue)
:red_circle: Project Title : Fake. News Detection
:red_circle: Aim : Integrating frontend for this model
:red_circle: Dataset : using the provided one datasets and using gradio for frontend
:red_circle: Approach : Trying to build a front end for this project where user can insert  the news and can get predictions.

📍 Follow the Guidelines to Contribute in the Project :

You need to create a separate folder named as the Project Title.
Inside that folder, there will be four main components.

Images - To store the required images.
Dataset - To store the dataset or, information/source about the dataset.
Model - To store the machine learning model you've created using the dataset.
requirements.txt - This file will contain the required packages/libraries to run the project in other machines.

Inside the Model folder, the README.md file must be filled up properly, with proper visualizations and conclusions.

:red_circle::yellow_circle: Points to Note :

The issues will be assigned on a first come first serve basis, 1 Issue == 1 PR.
"Issue Title" and "PR Title should be the same. Include issue number along with it.
Follow Contributing Guidelines & Code of Conduct before start Contributing.

:white_check_mark: To be Mentioned while taking the issue :

Full name : Puspak Dakkata
GitHub Profile Link : https://github.com/AKing-283
Email ID<EMAIL_ADDRESS>Participant ID (if applicable):
Approach for this Project : Girlscript summer of code projects 2024
What is your participant role? (Mention the Open Source program): Girlscript Summer of Code 2024

Happy Contributing 🚀
All the best. Enjoy your open source journey ahead. 😎

Are you trying to create a web app for this project?
https://github.com/abhisheks008/DL-Simplified/tree/main/Fake News Detection
@AKing-283

yess I am looking for usage of  Streamlit or Gradio for this

Cool go ahead @AKing-283

Thanks @abhisheks008 👍🏻

Here is my work for Fake news detector:


[fix][client] Fix negative ack not redelivery.

Motivation
When consumer subscribes to more than one topic, if uses negative API to redelivery msg, it will fail with the below error:
java.lang.IllegalArgumentException: null
	at com.google.common.base.Preconditions.checkArgument(Preconditions.java:131) ~[guava-31.0.1-jre.jar:?]
	at org.apache.pulsar.client.impl.ConsumerImpl.redeliverUnacknowledgedMessages(ConsumerImpl.java:1901) ~[classes/:?]
	at org.apache.pulsar.client.impl.NegativeAcksTracker.triggerRedelivery(NegativeAcksTracker.java:82) ~[classes/:?]

Because if subscribes with multi-topics, the internal message will be TopicMessageIdImpl and add to NegativeAcksTracker without conversion to MessageIdImpl(user not config with negativeAckRedeliveryBackoff):
https://github.com/apache/pulsar/blob/35c4b68f586774d0e2b7a3a2a6ab1d6a20ac1452/pulsar-client/src/main/java/org/apache/pulsar/client/impl/NegativeAcksTracker.java#L87-L112
Then NegativeAcksTracker will trigger redeliver and throw IllegalArgumentException at here :
https://github.com/apache/pulsar/blob/35c4b68f586774d0e2b7a3a2a6ab1d6a20ac1452/pulsar-client/src/main/java/org/apache/pulsar/client/impl/ConsumerImpl.java#L1896-L1902
The issue seems introduced by #12566.
Documentation

[x] no-need-doc
(Please explain why)

Reproduce Test
@Test
    @SneakyThrows
    public void testNegativeAcks() {
        String topic = BrokerTestUtil.newUniqueName("testNegativeAcks");
        @Cleanup
        Consumer<String> consumer = pulsarClient.newConsumer(Schema.STRING)
                .topic(topic)
                .subscriptionName("sub1")
                .subscriptionType(SubscriptionType.Shared)
                .enableRetry(true)
                .negativeAckRedeliveryDelay(1, TimeUnit.SECONDS)
                .subscribe();

        @Cleanup
        Producer<String> producer = pulsarClient.newProducer(Schema.STRING)
                .topic(topic)
                .create();

        producer.sendAsync("test-ack");
        producer.flush();
        while(true) {
            Message<String> msg1 = consumer.receive();
            System.out.println("received : " + msg1);
            consumer.negativeAcknowledge(msg1);
        }
    }

@Technoboy- can you check #6869 , does this PR fix that issue?

It's not related to that issue. This bug was introduced by #12566.

It's not related to that issue. This bug was introduced by #12566.

thanks. btw. would you be interested in fixing #6869 ? That's a long time issue with negative acknowledgements and batch messages.

It's not related to that issue. This bug was introduced by #12566.

thanks. btw. would you be interested in fixing #6869 ? That's a long time issue with negative acknowledgements and batch messages.

I will work on it.

It's not related to that issue. This bug was introduced by #12566.

thanks. btw. would you be interested in fixing #6869 ? That's a long time issue with negative acknowledgements and batch messages.

Hi @lhotari , I think #6869 has been fixed by #9440.


WIP: Ratelimiter

Discussion about this is found in: #131.
This will be my contribution to adding a ratelimiter to the tmi.js library.
Features:

[x] Ratelimiter module
[x] Configuration option for ratelimiting (default off)
[ ] Documentation

Currently the message limit is still set to 100. I'll change that depending on what we decide. I like @3ventic's proposal of just keeping it safe and defaulting to 20-30 messages for the ratelimiter to protect against accidental unmodding as seen in PR #131. Discussing can be done in #131.

About the PR so far
At this point I've added a way to configure the rate limiters max amount of messages. The developer is free to specify whatever amount of messages the client is allowed to send within the 30 seconds timeframe. This can be done through the config of the client:
// client.js
var options = {
    options: {
        ratelimiter: 30
    },
    ...
};

var client = new irc.client(options);
client.connect();

The ratelimiter option accepts the following types:

Boolean: (true|false) where true sets the the ratelimiter to 30 (Twitch's user value, see String).
Number: (1-100) number cannot be below 1 and higher than 100 (decimals will be rounded)
String: ("mod"|"user") a string can also be supplied to make use of Twitch's default values (mod: 100, user: 30).

Any feedback? 😁
Middleware system?
I was also thinking whether or not it would be useful to add a middleware system, where the developer himself can choose what middleware he wants to hook in front of certain parts of TMI.js.
Use case:
I am a developer and I want to add ratelimiting to my Twitch bot. I just hook up the ratelimiter middleware from a seperate library called: tmi-ratelimiter. I also want to parse every user that comes in and give him a more stable user['display-name'] feature with either the display-name or it's username.
var ratelimiter = require('tmi-ratelimiter');
var crashReporterMiddleware = function(...) {
  try {
    // lets see if this works
  } catch {
    // do some crash reporting
  }
};
var formatUserMiddleware = function(...) {
  // do some user formatting
}

var options = {
    middleware: {
        say: [ratelimiter, crashReporterMiddleware],
        onChat: [formatUserMiddleware]
    }
}

I haven't fully thought this out yet, but it might be something to add customizability and gives developers the freedom what to hook into their app. If I come up with a better use-case I'll let you guys know! Have a nice weekend 😁

I will give it a try this weekend, I'd like to make this feature available in 1.0.0.
Thanks!

I must update to v1.0.0 very very soon because of some breaking changes on Twitch and this is going to affect all apps using this module.. because of that, I will have to introduce this feature in 1.1.0 instead, sorry about that :unamused:

Don't worry about it! I'll try and build a demo app for you to test this faster. Also I'll add some comments for documentation purposes. Are you up for using JSDoc style for function documentation? Good luck getting v1.0.0 out! 😁 🎉
Example:
https://github.com/reactjs/redux/blob/master/src/utils/warning.js (look at the comments above the function declaration)

You can go ahead and use multi-line comments like https://github.com/airbnb/javascript#comments--multiline :+1:

I'm really excited for this to be brought in, I just wanted to thank you guys for your hard work :+1:

Sorry for the inactivity. I hope in the following days/weekend to work on this pull request to see if we can get this finalized.

Hey no worries I know how life can be. I just wanted to express some appreciation :D

I'm quite busy too, working 6 days a week and also going to school and taking care of my mom, I will be more active in about 2 weeks :smile:


Reverting package updates

@fabiocav, Thanks for signing the contribution license agreement so quickly! Actual humans will now validate the agreement and then evaluate the PR.
Thanks, DNFBOT;

@davidebbo @mathewc this change reverts the package changes committed with the API Hubs changes. We'll need to figure out what needs to be updated.

Thanks for taking care of it.


Use Auth from meta data if secretKey or authKey is empty string

This is so we can use IAM roles instead of hardcoding auth.

Ok, I changed the error message to be grammatically and syntactically correct.

@rkoval is correct. The comma is needed to separate the two independent clauses. It is, however, accepted as a stylistic decision to omit the comma before the conjunction if the two clauses are short. Thanks guys.


Auth not working with hashed password

if i follow the instructions to hash the password then i get a failed login due to incorrect password however if i store the password in plaintext it works fine

nevermind, fixed by rerunning the hash generation command


Expose option to listen for onprogress event

See https://developer.mozilla.org/en/docs/Web/API/ProgressEvent

Hey @brunobasto, what do you want exposed? Do you have an idea of how the usage and API would look like? Can you add some more context about the use case?
Since Ajax.request is promise-based, it's not 100% straightforward...
One option would be to attach the xhr instance to the returned promise so you could do whatever you want with it. However, this would open the door for endless side-effects, so I'm not so sure that's such a good idea.
If we wanted to be more strict, we'd need something like progress-promise, similar to jQuery deferred.progress. I haven't seen this spec'd or proposed anywhere though, so it would be quite custom at this point...
@Robert-Frampton, @eduardolundgren, @ipeychev... thoughts?

Hey @jbalsas
I just opened this one for discussion purposes. I haven't thought about how to approach this yet. This could be useful for people using the API in general, but the use case I had in mind was to have the senna loading bar show the real progress of the request instead of a continuous animation. Similar to what GitHub is doing I think.
The approach that would require the least amount of change would be to just add another parameter to Ajax.request called opt_onprogressFn. Although I think we already too many parameters there.

I guess we could have some shorthand function so that we wouldn't have to pass in all those params at once. Something like:
Ajax.config({
	url,
	onprogress: (event) => {
		console.log(`Loaded ${event.loaded}% so far.`);
	}
}).request();

Could be implemented like:
static config(options) {
	return {
		request: () => Ajax.request(options.url, options.method, options.body, ...)
		post: () => Ajax.request(options.url, 'POST', options.body, ...)
	}
}

I like the idea of creating a new method for passing in params as a config object, there are too many params on the existing request method.
I think the onprogress param would be fine, but what if we created a more general solution so that developers could access/modify the xhr instance without attaching it to the promise? That way we don't have to continue adding params for features such as this.
Ajax.config({
	url,
	beforeRequest: (xhr) => {
		xhr.onprogress = myFn;
	}
}).request();

Hey @brunobasto, as @Robert-Frampton points out, I don't think the config solution fixes the onprogress issue, so I would create a separate issue for that.
In that line, the impression I got from past talks is that metal-ajax was meant to be a low-level transport module that we could use to build higher abstractions. That's what WeDeploy does for its client API. I feel that this higher abstractions could be achieved somewhere else, maybe by exposing WeDeploy's client or creating a similar API client for Liferay Portal and other projects.
@eduardolundgren?

I was also looking at Rob's idea, and while I don't dislike it, I don't think we want to do that. This opens the door for metal-ajax not working at all.
For example:
Ajax.config({
	url,
	beforeRequest: (xhr) => {
		xhr.onload = () => {};
	}
}).request();

This is likely to override our own onload method and make the promise never resolve. Similar overrides can happen for the different params, headers and so on that we pass, so I guess it would be better to shield the xhr object and expose the functionality we need in a different way...
I think I lean towards implementing a ProgressPromise inside metal-promise on top of CancellablePromise. Then, this could be used like:
Ajax.request(...)
    .progress(value => console.log(value))
    .then(...)

Another reference: https://github.com/sindresorhus/p-progress

Let's try to squeeze this in for 3.0.0, @Robert-Frampton! 🎉

Hey @jbalsas,
Since it's not too complicated, I thought it would be best to add progress functionality to metal-promise in the form of a ProgressPromise class. Once we have that published it should be pretty trivial to add progress support for metal-ajax.

@jbalsas
In this case, could you release v3.0 with what we have so far to unblock WeDeploy API?

Hey @Robert-Frampton, I've just released<EMAIL_ADDRESS>so this should be ready to be implemented here 🎉
I've agreed with @ipeychev to wait until tomorrow to see if we can release<EMAIL_ADDRESS>with both, React16 and ProgressPromise to avoid unnecessary releases, so if you can send that during the day, we'll merge and release tomorrow 💃

Merged to develop and scheduled for 3.1.0!!


⚠ Bofrost.es has degraded performance

In 97c6245, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1475 ms

Resolved: Bofrost.es performance has improved in 8568b7b after 6 minutes.


MockRender is not working when we try to spy on a service method which is called in component constructor

Stackblitz link to reproduce this issue
https://stackblitz.com/edit/ng-mocks-examples-gwee2q?file=app/app.component.spec.ts

Hi again,
Thanks for the report.
This is expected. Because MockRender adds a middleware component.
To spy in such cases MockInstance should be used.
Please don't close the ticket. I'll update docs based on the issue.

@satanTime Can we add a configurable option to bypass the middleware component?
add it only when needed?
something like MockRender(XComponent, null, { renderWithoutWrapperComponent: true  })

Something like that.
describe("mock extended component", () => {
  beforeEach(() => MockBuilder(AppComponent, AppModule));

  beforeEach(() => {
    MockInstance(AppService, 'method', jasmine.createSpy('method'));
    //
    // Or simply ngMocks.autoSpy('jasmine');
  });

  it("expect service.method to have been called", () => {
    MockRender(AppComponent);

    const service = TestBed.get(AppService);
    expect(service.method).toHaveBeenCalled();
  });
});

If you want to buy pass the middleware component it means you don't need MockRender and could use TestBed.createComponent instead.
The idea of ng-mocks and its tools is to allow to configure environment mock environment before TestBed initialization.
And then in tests to render required stuff and to do assertions.

Thanks for the explanation 👍

I'll dive deeper and check if it is possible somehow to inject a component after the initialization and post an update here.
The main diff between createComponent and MockRender is that createComponent allow to test inputs, outputs, queries and some lifecycle hooks. And, ideally, it should be a replacement for TestBed.createComponent, but such cases, like you have: when there is a real service and only one of its methods should be mocked, unfortunately, doesn't work.
Might I ask to explain why you want to keep AppService as it is and only to mock one method?

Here is the snippet from our codebase.
configureModuleBuilder(() => {
    return MockBuilder(XComponent, XModule)
        .mock(DomSanitizer, { bypassSecurityTrustUrl: v => v }, { precise: true })
        .provide([
            { provide: I18NService, useClass: I18NServiceMock },
            { provide: WindowService, useClass: WindowMock },
            { provide: Store, useClass: StoreMock },
            XActions,
        ]);
});

beforeEach(() => {
    mockWindowService = TestBed.inject(WindowService);
    store = TestBed.inject(Store);
    spyOn(store, 'dispatch');

    fixture = TestBed.createComponent(XComponent);
    component = fixture.componentInstance;
    fixture.detectChanges();
});

it('dispatches ', () => {
    expect(store.dispatch).toHaveBeenCalledWith();
});

When I tried to use MockRender instead of TestBed.createComponent it failed.
It will be good if MockRender has the default behaviour of createComponent.
use the wrapper component only when needed. or an option to bypass when not required.

Oh, it looks a bit confusing. WindowService and Store are mock classes already, and to mock them further could be avoided I think.
Might you share how I18NServiceMock, WindowMock and StoreMock look like?
I think you could use ngMocks.defaultMock and MockInstance for such a test, then its code should be shortened to a few lines.

I18nServiceMock, WindowMock and StoreMock are different. different spec files may require different versions of these.
Sorry. Can't share the code for these.

Sure, I totally understand this.
I'll try to find a way how to make your test working and being something like:
configureModuleBuilder(() => {
  MockInstance(DomSanitizer, 'bypassSecurityTrustUrl', v => v);
  MockInstance(I18NService, I18NServiceMock);
  MockInstance(WindowService, WindowMock);
  MockInstance(Store, StoreMock);
  MockInstance(Store, store => spyOn(store, 'dispatch'));

    return MockBuilder(XComponent, XModule).provide(XActions);
});

beforeEach(() => {
    fixture = MockRender(XComponent);
    component = fixture.point.componentInstance;
    mockWindowService = TestBed.inject(WindowService);
});

it('dispatches ', () => {
    expect(store.dispatch).toHaveBeenCalledWith();
});

Thank for your time on this.
And please take a look into memoization of MockRender when you find some time.

And please take a look into memoization of MockRender when you find some time.

Could you create a separate issue and provide an example how you want it to memoize the things?

  beforeEach(() => {
    MockInstance(Store, 'dispatch', jasmine.createSpy());
    MockInstance(WindowService, 'open', jasmine.createSpy());

    mockWindowService = TestBed.inject(WindowService);
    store = TestBed.inject(Store);

    fixture = MockRender(XComponent);
    component = fixture.componentInstance;
  });

This snippet is working.

And please take a look into memoization of MockRender when you find some time.

Could you create a separate issue and provide an example how you want it to memoize the things?

Sure. will do.


filter out specific applications or incognito window

definition of done:

i can filter out apps like bitwarden or keychain for example
i can filter out incognito window (browser thing, should work on chrome, edge, arc, safari)
should work both in CLI and app (e.g. add in settings ui)
in cli i expect something like screenpipe --filter-out-app bitwarden or something like this (find a good ux)
in UI it should be a + when you click it list apps you have on your computer and you can select to add to list of ignored
there should be a toggle or checkbox to filter out incognito i think

(can copy rewind UX/UI)

/bounty 100

done


Issue #2677: Autocomplete fields duplicated when using browser back/forward buttons

When accessing the add or edit page for a model that uses the ui-autocomplete fields, using the back and forward buttons on the browser can cause the autocomplete fields to be duplicated.
In _create, check if the field exists before creating it but still initialise autocomplete and set any necessary event handlers.
Also break out the Javascript into multiple, much smaller, functions so it is easier to follow. Since the Asset Pipeline can be used for magnification, we can afford to increase file size to increase readability. Who knows, the file may even get tested one day too.

CI failing due to Rubocop issues addressed in PR 2679

Works fine, thanks!


Missing nfts in profile created tab

What happened?
A bug happened!
Please reproduce in steps

Go to /rmrk/u/DVa676uebTkgH8uZyS2umio7JtmkcSHHoPvPJJQY3bbYoAu
See missing nfts

Expected Behavior
I've expected to see all nfts
What browsers are you seeing the problem on?
Firefox
Are you logged in?
No response
At which address did you encounter bug?
beta.kodadot.xyz, kodadot.xyz
Screenshots

Relevant log output
None

Payment link for reward
No response
Code of Conduct

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

@roiLeo I can check this

Hmm noticed like this yesterday at explorer at beta but when I went back through pages it showed again.
Isn't there an issue with how we fallback to Pinata from CF?
📟 @vikiival

https://beta.kodadot.xyz/rmrk/detail/12009176-289d0e953cbdc59a73-HHK-0072_HYPEHELL_KEYCHAIN-0000000000000072
it doesnt load either

I see https://beta.kodadot.xyz/rmrk/collection/289d0e953cbdc59a73-HHK
And no errors in the console. Strange.

I see https://beta.kodadot.xyz/rmrk/collection/289d0e953cbdc59a73-HHK And no errors in the console. Strange.

Check this out https://singular.rmrk.app/collections/289d0e953cbdc59a73-HHK 😕 it might be related with rmrk 2.0 migration

but thought they should use Burn?

Isn't there an issue with how we fallback to Pinata from CF?

I think it's related to something else (I bet we have some v-if which causes random NFTs to disappear)


Make the color and number of lines in history customizable

I am sending this pull request as I needed this to work on my terminal setup, specially the ability to change font color.   The change will not affect the default behavior.
To start  a history  with 8 lines of history and green color text :
source history-window.sh ; HISTORY_FONT_COLOR=2 ; NUM_HISTORY_LINES=10
To change the color after the sourcing
HISTORY_FONT_COLOR=2

discussed in #4

Thank you! I will review this very soon. Very appreciated.


track function doesn't have userId argument

Describe the bug
There is no option to add the userId argument in the track function, unlike in javascript SDK. Currently the workaround is creating the TrackEvent and passing it to process.
Expected behavior
track function has userId or message argument.
Screenshots
In this repo:

In the javascript SDK (expected behavior, userId is part of message: Identity):

Platform (please complete the following information):

Library Version in use: 1.9.2
Platform being tested: Mac
Integrations in use: Amplitude

hey @noymaor, this is by designed. the mobile SDK is not stateless, it remembers the user info from the last call to identify and applies that info to the payload automatically.
if in your use case, the userId is different for every track call, you can write a simple plugin for it:
class UserIdPlugin: EventPlugin {
    override val type: Plugin.Type = Plugin.Type.Enrichment
    override lateinit var analytics: Analytics

    override fun track(payload: TrackEvent): BaseEvent? {
        val event = super.track(payload)

        if (event is TrackEvent) {
            event.properties.getString("userId")?.apply {
                event.userId = this
                event.properties = updateJsonObject(event.properties) {
                    it.remove("userId")
                }
            }
        }

        return payload
    }
}

add the plugin to analytics and then use it like this:
analytics.track("test", buildJsonObject {
    put("userId", "YOUR USER ID")
    // other properties
})


2.1 was yanked from ruby gems?

I was getting the following error:
Fetching gem metadata from https://rubygems.org/.........
Could not find jekyll-paginate-v2-2.1.0 in any of the sources

When I checked out Ruby gems, I see that 2.1 was yanked. What's going on here?

@eddietejeda Yes, v2.1.0 has been yanked since it was buggy and broke users' sites.
Please downgrade to v2.0.0
Apologies for the inconvenience caused.

Further rollbacks performed, please upgrade to https://rubygems.org/gems/jekyll-paginate-v2/versions/3.0.0


travis lint warnings

The configuration changes made to .travis.yml. in #503 in order to add docker/acceptance tests into the test matrix are flagged by the travis gem's lint function.
Warnings for .travis.yml:
[x] in matrix.include section: unexpected key sudo, dropping
[x] in matrix.include section: unexpected key services, dropping
[x] in matrix.include section: unexpected key script, dropping
[x] in matrix.include section: unexpected key bundler_args, dropping

Presumably, these are bogus warnings that need to be resolved upstream.
/cc @petems

Reported upstream: https://github.com/travis-ci/travis.rb/issues/376

Yeah was gonna say, they're not being dropped because it's working! Very weird :sob:

We've moved away from Travis CI.


extends WebSocketServer VS Annotation

extends WebSocketServer VS Annotation ？
This is my understanding: Annotation, it is not necessary to have a container, eg: tomcat, But extends WebSocketServer can not use tomcat。
Forgive me for being a rookie, in websocket, use Annotation Can not use container?

Hello @b3601993,
to my knowledge it is possible to use this library in a tomcat environment.
Some users reported that at least.
Greetings
marci4

Dead issue.
Closing!
Greetings
marci4


Increment operators do not yield a value

Prerequisites

[X] Write a descriptive title.
[X] Make sure you are able to repro it on the latest released version
[X] Search the existing issues.
[X] Refer to the FAQ.
[X] Refer to Differences between Windows PowerShell 5.1 and PowerShell.

Steps to reproduce
Coming from C#, I expect the ++ operator to yield the value of the variable that has been or will be incremented. This does not work on PowerShell (tested on 7.2.2)
$x = 1; $x++ or $x = 1; ++$x do not yield a value
if I put round brackets around the expression, it works:
$x = 1; ($x++) -> 1
$x = 1; (++$x) -> 2
the operator should yield the respective value without the need to put brackets around it.
Expected behavior
PS> $x = 1; $x++
1

PS> $x = 1; ++$x
2

Actual behavior
PS> $x = 1; $x++
<nothing>

PS> $x = 1; ++$x
<nothing>

Error details
No response
Environment data
$ $PSVersionTable

Name                           Value
----                           -----
PSVersion                      7.2.2
PSEdition                      Core
GitCommitId                    7.2.2
OS                             Microsoft Windows 10.0.19044
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

Visuals
No response

This is by design. PowerShell differs from C# where it has a Expression Mode and an argument mode. Technically, "yielding a value" means that you send the value to the pipeline and eventually display it at the end of a pipeline (see: Write-Output which describes the default behavior).
In a nutshell:. PowerShell doesn't require parentheses when calling a function and in case you use  $x++ as an argument, you do not want to put that on the pipeline when invoking a function like this:
$Object |ConvertTo-Json -Depth $x++

This is intentional behaviour and it looks weird if you are not expecting it.
($x = 2) -gt 1

Returns true, because the value of an assignment operation is the value that was assigned,
But $x = 2 is normally silent   ; so are $x = $x +1 $x += 1 and $x ++.
The example above needs brackets because otherwise it reads as $x = (2 -gt 1)  but something like  ++ $x  -gt 2  will return true and doesn't need the brackets.
This way you can write something like
while ($x ++ -lt 10) do { "something with $x"}

And the pre-increment value is available but  if you write the same thing as
while ($x -lt 10) do { "something with $x" ; $x++}

with the incremented value being sent as output.

Those tricks allow things like this
> $z = $y = $x = 1  # Sets all 3 to 1

> $z += $y += ++ $x  # Makes x 2, adds it y, and then adds Y to Z

> $x,$y,$z

2
3
4

thanks for your answers. guess ill need to check in on that expression vs. argument mode!
definitely something you need to get used to.


Is there some scripts to extract the raw text?

Or do you ever extract the sentences before? I only find xml file there.
And thank you for make such a great project.

Depends on what do you mean by "extract raw text". Extract from what?

Depends on what do you mean by "extract raw text". Extract from what?

I mean how to extract the thai contents from the xml files which is downloaded from http://web-corpora.net/ThaiCorpus/search/

If you mean the text collection, these xml files were converted into a specific corpus format that allows indexing and searching. Converter is here.


Arts 285

Tidying exceptions, ensuring unix + windows file separator is used and force calling destroy as the java feedback via annotations isnt working..Ive tried lots of ways without luck..

We create a directory to hold the git json file which is then deleted as soon as its read, I initially used the classes/resources directory for this but Windows didnt like it as it didnt parse the path correctly. Ive played around with a few scenarios and ended up just creating a dir at the base of the app called gitRepo - however this is determined by the project set-up in Eclipse/IntelliJ so Im not sure how this will work in Azure. That said we delete the directory in a finally block so it shouldnt matter too much as long as we have permissions to create the directory - which, again Im guessing we should as it should be running in a container? So in a nutshell I think its ok but we may, possibly have an issue in Azure...

We create a directory to hold the git json file which is then deleted as soon as its read, I initially used the classes/resources directory for this but Windows didnt like it as it didnt parse the path correctly. Ive played around with a few scenarios and ended up just creating a dir at the base of the app called gitRepo - however this is determined by the project set-up in Eclipse/IntelliJ so Im not sure how this will work in Azure. That said we delete the directory in a finally block so it shouldnt matter too much as long as we have permissions to create the directory - which, again Im guessing we should as it should be running in a container? So in a nutshell I think its ok but we may, possibly have an issue in Azure...


Fixed filename processing on Windows

Fixed behaviour on Windows systems while splitting the filenames by ":" that results into an errors.

All changes are related, except indentation on line 224 (will not be fixed)
godef--find-file-line-column does another thing in a much heavier way, so I'll leave that unchanged too

Customizable variable was introduced to avoid magical numbers in code

Which is fine, but entirely unrelated to "fixing file name processing on Windows" and has no place in this PR or commit. (It should also have been a constant, not a customizable variable; we don't want to expose it to the user at this point in time)

All changes are related, except indentation on line 224, but originally it was a crap because of mixed tab and space symbols in a one function (will not be fixed back)

Which is fine, but also unrelated. It should at least be its own commit that does nothing but change the formatting.

godef--find-file-line-column does another thing in a much heavier way. Currently reassembling of the file:line:col is executed only if problem exists, so I'll leave that unchanged too

Your solution is a hack, which works around a bug in the original code (blindly splitting on ":" is inherently wrong. Non-windows file systems permit colons anywhere in the file name, so both the original code and your fix are insufficient.)
godef--find-file-line-column is much closer to a sane implementation. It is indeed not a perfect drop-in replacement, though, as it operates on columns, whereas guru emits byte positions. We should probably split the parsing of file:line:col into a separate function, then reuse that one both in godef--find-file-line-column and go-guru, interpreting the column as required. Or, ideally, Emacs already has a public function that does this for us.

If you have a different opinion, please, clarify what you exactly want

I've been rather clear about what I want, you just chose to ignore me. Will you be able to follow my requests, or should I just implement the change myself?

Thanks for clarifiing but it seems that I have no so much free time to support legacy formatting

we don't want to expose it to the user at this point in time

Strange point of view. Do we both talking about  lisp code?

Your solution is a hack

I know that but that is better than nothing.

or should I just implement the change myself?

I am not a telepath so if you know a better solution you are free to implement it. I have no objections.
I just want to have a solution publicly available.

I finally rewrite my code to satisfy your requirements. This hack fixes the problem for the 99.9% of users so it could be considered as temprorary solution untill further refactoring.

Thanks, I'll take a look soon!


chore: Update create app template to align with example app

Hey, I just made a Pull Request!

Some minor maintenance 🧹 to the create-app plugin to ensure the resulting app better aligns to the example in the template repo, including,

Moving test-utils to a devDependency
Minor documentation tweaks
Bump of @spofity/prettier-config version

:heavy_check_mark: Checklist

[ ] A changeset describing the change and affected packages. (more info)
[x] Added or updated documentation
[ ] Tests for new functionality and regression tests for bug fixes
[ ] Screenshots attached (for UI changes)
[ ] All your commits have a Signed-off-by line in the message. (more info)

An interesting note on this. Discovered the create-app does not import the @backstage/search-common package - but seems it doesn't need it yet?  Should this be included in some way? I checked the PR where it was added, and it was added to the example app but not the templated one yet. (and again seems to work without it)
cc @realandersn may know

There's a few more things about this. For e.g., there's a react hot loading call missing from the template, and a few others, but I'll call it here with just a trivial enhancement.


Doesn't work both in Chrome and Firefox

Platform informations

Chrome, Firefox
Extension version (e.g. 1.4.2)
[ ] Manually built extension

Issue Description
The extension doesn't work in Chrome, so I tried to install it in Firefox and even though I set the "xpinstall.signatures.required" to false, I can't install the add-on. The Chrome extension successfully switches between different accounts but it doesn't bypass any paywalls.
Actual behavior

Expected behavior
I need some guidance because it clearly doesn't work for me and does for others.
Steps to reproduce
I personally followed the directions on the main page.

Quizlet randomly shows this hard paywall.  The extension must then reload the page until it gets one without it.
Please try this snapshot. Please, tell me if the issue is still present.

I tried with the snapshot you provided, seemed to have not fixed the issue. Been reloading for abt 5 mins now

UPDATE: I tried with the snapshot you uploaded and it seems to be working right now. I will give another update after I use it for a while. Thank you for the help @rospino74.


[Enum Request] In the table 10012 "Sales Tax Amount Difference" field "Document Type" should be Enum

Hello MS team.
Could you please make a change from option to enum?
Thanks.

This field get values from different document headers: Sales: Purchase and Services. They contains different values, so you should use Integer value instead of enum here.


Fix and improve issue reminders

Add ?include=journals to the isues/x.json API call to fetch journals.
Log output for writing issue reminders. This makes it easier to check if anything went wrong.
Also don't write reminders for issues that don't return JSON.
Issue: https://progress.opensuse.org/issues/157858

s|isues/x.json|issues/x.json
Not sure why the CI checks fail but the change itself looks generally good.

I forgot to push to origin again. The backlog limits workflow only works on the origin because of secrets.
But that check is only about the issue list output, so that shouldn't be influenced by my change anyway.


Add specific event type to header

This PR adds the specific event of a webhook to the request header.
Currently we add an action field to the JSON which, combined with the current header, can be used to deduce the specific action.
That being said, it is much easier to check the header and do a complete JSON unmarshal rather than partially unmarshaling to get the action and then fully unmarshaling after.
This is somewhat in an effort to make webhook interceptors easier to implement for Gitea.
Ideally in the future I'd like to be able to automate example webhook payloads for a given Gitea version. :slightly_smiling_face:

Codecov Report

Merging #17222 (7b86a93) into main (f2e7d54) will decrease coverage by 0.03%.
The diff coverage is 49.70%.

@@            Coverage Diff             @@
##             main   #17222      +/-   ##
==========================================
- Coverage   45.31%   45.27%   -0.04%
==========================================
  Files         773      779       +6
  Lines       86873    87298     +425
==========================================
+ Hits        39366    39525     +159
- Misses      41151    41357     +206
- Partials     6356     6416      +60

Impacted Files
Coverage Δ

cmd/admin.go
0.00% <0.00%> (ø)

models/action.go
48.86% <0.00%> (ø)

models/avatar.go
35.38% <0.00%> (ø)

models/error.go
39.83% <ø> (+0.29%)
:arrow_up:

models/fixture_generation.go
70.00% <0.00%> (ø)

models/gpg_key_verify.go
13.46% <0.00%> (ø)

models/helper.go
52.94% <0.00%> (ø)

models/issue_lock.go
0.00% <0.00%> (ø)

models/migrate.go
0.00% <0.00%> (ø)

models/pull_sign.go
37.64% <0.00%> (ø)

... and 213 more

Continue to review full report at Codecov.

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

Wake up lgtm


Replace kallisto Dockerfile without conda

Replace kallisto Dockerfile to install kallisto without conda
The current image cannot be pulled on OSX via singularity. Related to https://github.com/sylabs/singularity/issues/4301

Does that mean that all images with base:miniconda3 and software installed by conda might not work on Mac?

I think this is something that will be fixed by singularity. But for the time being it might cause some problems. Also, note that singularity for mac is still in beta.


Updated the ticket percent outline string for the Tuya extension to a…

… constant

Proposed change

Changed the ticket percent outline string in the Tuya extension to a constant in order to improve maintainability.
Type of change

[ ] Dependency upgrade
[ ] Bugfix (non-breaking change which fixes an issue)
[ ] New integration (thank you!)
[ ] New feature (which adds functionality to an existing integration)
[ ] Deprecation (breaking change to happen in the future)
[ ] Breaking change (fix/feature causing existing functionality to break)
[x] Code quality improvements to existing code or addition of tests

Additional information

This PR fixes or closes issue: fixes #
This PR is related to issue:
Link to documentation pull request:

Checklist

[x] The code change is tested and works locally.
[x] Local tests pass. Your PR cannot be merged unless tests pass
[ ] There is no commented out code in this PR.
[ ] I have followed the development checklist
[ ] I have followed the perfect PR recommendations
[x] The code has been formatted using Black (black --fast homeassistant tests)
[ ] Tests have been added to verify that the new code works.

If user exposed functionality or configuration variables are added/changed:

[ ] Documentation added/updated for www.home-assistant.io

If the code communicates with devices, web services, or third-party tools:

[x] The manifest file has all fields filled out correctly.
Updated and included derived files by running: python3 -m script.hassfest.
[ ] New or updated dependencies have been added to requirements_all.txt.
Updated by running python3 -m script.gen_requirements_all.
[ ] For the updated dependencies - a link to the changelog, or at minimum a diff between library versions is added to the PR description.
[ ] Untested files have been added to .coveragerc.

To help with the load of incoming pull requests:

[ ] I have reviewed two other open pull requests in this repository.

Kishor Peneff Group 10
2: The current way of defining the icon via a string is cumbersome in case the icon is to be changed in the future as all instances of the string have to be changed.
3: By changing the string to a constant, it only needs be updated in one place in order to change the icon for all instances in the file.

looks good


Deploy schedule?

I've made some pull requests with some features I'd like to use and deploy to my team.
Is there any schedule for addressing the pull requests and the possibility of an updated package?
Is anyone supporting this extension?
Thanks!

Thanks @vrachels for your contributions, it is really appreciated!
Currently my focus is on my day job. But I hope to review your PR for the week-end, and deploy a new release early next week, hopefully at the same time as VSCode release 1.16

great!  Likewise - there just happens to be intersect with this plugin and my dayjob :]
Thanks for the feedback and updates!

Thanks a lot @vrachels for the PR :+1:
I'll test it a bit, and push a new release this week.

Just pushed version 2.2.0, please update.


Suppress ouroboros missing docs warnings

Depends on joshua-maros/ouroboros#49

Fun!

There must be another way not relying on upstream.
It shows up in code review too often!

There must be another way not relying on an upstream PR. It shows up in code review too often!

Yep! See 8e7dc55

Great yeah.


contrib: rename demo to demos-unmaintained

We don't have the capacity to keep demos running that are built against alpha APIs. Hence, we better make that clear to the ppl trying kcp.

Do you need to remove the new files that were added (kubeconfigs from local demo runs)?

Fixed.
/hold cancel

/lgtm

/retest


Control frequency of progress updates

Please follow the guide below

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

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

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

Before submitting an issue make sure you have:

[x ] At least skimmed through the README, most notably the FAQ and BUGS sections
[x ] Searched the bugtracker for similar issues including closed ones
[ x] Checked that provided video/audio/playlist URLs (if any) are alive and playable in a browser

What is the purpose of your issue?

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

If the purpose of this issue is a bug report, site support request or you are not completely sure provide the full verbose output as follows:

[debug] System config: []
[debug] User config: []
[debug] Custom config: [u'--prefer-ffmpeg', u'-f', u'best[height=540][fps=25][ext=mp4]', u'--newline']
[debug] Command-line args: [u'--config-location', u'/mod/etc/youtube-dl.conf', u'-o', u'/mnt/hd2/My Video/%(title)s.%(ext)s', u'-v']
[debug] Encodings: locale ASCII, fs ASCII, out ASCII, pref ASCII
[debug] youtube-dl version 2018.10.05
[debug] Python version 2.7.1 (CPython) - Linux-2.6.18-7.1-7405b0-smp-with-libc0
[debug] exe versions: ffmpeg 2.8, ffprobe 2.8
[debug] Proxy map: {}

15/10/2018 15:49:31 - [bbc.co.uk] b0bg9rdk: Downloading video page
15/10/2018 15:49:34 - [bbc.co.uk] b0bg9rdk: Downloading playlist JSON
15/10/2018 15:49:36 - [bbc.co.uk] b0bg9rd4: Downloading media selection XML
15/10/2018 15:49:37 - [bbc.co.uk] b0bg9rd4: Downloading m3u8 information
15/10/2018 15:49:37 - [bbc.co.uk] b0bg9rd4: Downloading m3u8 information
15/10/2018 15:49:38 - [bbc.co.uk] b0bg9rd4: Downloading MPD manifest
15/10/2018 15:49:39 - [bbc.co.uk] b0bg9rd4: Downloading MPD manifest
15/10/2018 15:49:39 - [bbc.co.uk] b0bg9rd4: Downloading m3u8 information
15/10/2018 15:49:40 - [bbc.co.uk] b0bg9rd4: Downloading m3u8 information
15/10/2018 15:49:40 - [bbc.co.uk] b0bg9rd4: Downloading m3u8 information
15/10/2018 15:49:41 - [bbc.co.uk] b0bg9rd4: Downloading m3u8 information
15/10/2018 15:49:41 - [bbc.co.uk] b0bg9rd4: Downloading m3u8 information
15/10/2018 15:49:41 - [bbc.co.uk] b0bg9rd4: Downloading m3u8 information
15/10/2018 15:49:42 - [bbc.co.uk] b0bg9rd4: Downloading MPD manifest
15/10/2018 15:49:42 - [bbc.co.uk] b0bg9rd4: Downloading MPD manifest
15/10/2018 15:49:43 - [bbc.co.uk] b0bg9rd4: Downloading m3u8 information
15/10/2018 15:49:44 - [bbc.co.uk] b0bg9rd4: Downloading m3u8 information
15/10/2018 15:49:45 - [bbc.co.uk] b0bg9rd4: Downloading m3u8 information
15/10/2018 15:49:45 - [bbc.co.uk] b0bg9rd4: Downloading m3u8 information
15/10/2018 15:49:56 - [debug] Invoking downloader on u'https://bbcfmt-ic-9393c500-1464b0-vodhlsuklive.s.loris.llnwd.net/usp/auth/vod/piff_abr_full_hd/5ce6b3-b0bg9rd4/vf_b0bg9rd4_0220853d-f324-4edb-90c8-61ea8370eb35.ism/vf_b0bg9rd4_0220853d-f324-4edb-90c8-61ea8370eb35-audio_eng=320000-video=1604000.m3u8'
15/10/2018 15:49:56 - [hlsnative] Downloading m3u8 manifest
15/10/2018 15:49:56 - [hlsnative] Total fragments: 32
15/10/2018 15:49:56 - [download] Destination: /mnt/hd2/My Video/Bitz & Bob, You Can Do It Too! - Series 2, Woolly Wonder Girl.mp4
15/10/2018 15:49:57 - [download]   0.0% of ~53.90MiB at 159.59KiB/s ETA 08:45:12
15/10/2018 15:49:57 - [download]   0.0% of ~53.90MiB at 207.15KiB/s ETA 02:57:48
15/10/2018 15:49:57 - [download]   0.0% of ~53.90MiB at 212.46KiB/s ETA 01:18:26
15/10/2018 15:49:57 - [download]   0.0% of ~53.90MiB at 340.14KiB/s ETA 37:17
15/10/2018 15:49:57 - [download]   0.1% of ~53.90MiB at 453.42KiB/s ETA 18:45
15/10/2018 15:49:57 - [download]   0.1% of ~53.90MiB at 485.63KiB/s ETA 10:08
15/10/2018 15:49:57 - [download]   0.2% of ~53.90MiB at 601.18KiB/s ETA 05:36
15/10/2018 15:49:57 - [download]   0.5% of ~53.90MiB at 554.11KiB/s ETA 03:40
15/10/2018 15:49:58 - [download]   0.9% of ~53.90MiB at 603.39KiB/s ETA 02:31
15/10/2018 15:49:59 - [download]   1.9% of ~53.90MiB at 603.87KiB/s ETA 01:59
15/10/2018 15:50:00 - [download]   2.9% of ~53.90MiB at 597.70KiB/s ETA 01:48
15/10/2018 15:50:00 - [download]   3.1% of ~53.90MiB at 608.33KiB/s ETA 01:45
15/10/2018 15:50:00 - [download]   3.1% of ~53.90MiB at 608.33KiB/s ETA 01:45
15/10/2018 15:50:01 - [download]   3.0% of ~55.36MiB at 67.43KiB/s ETA 02:19
15/10/2018 15:50:01 - [download]   3.0% of ~55.36MiB at 116.51KiB/s ETA 02:19
15/10/2018 15:50:01 - [download]   3.1% of ~55.36MiB at 196.01KiB/s ETA 02:19
15/10/2018 15:50:01 - [download]   3.1% of ~55.36MiB at 331.96KiB/s ETA 02:19
.
. About 400 lines removed
.
15/10/2018 15:51:57 - [download]  99.6% of ~56.37MiB at 391.81KiB/s ETA 00:00
15/10/2018 15:51:57 - [download]  99.7% of ~56.37MiB at 516.97KiB/s ETA 00:00
15/10/2018 15:51:57 - [download]  99.7% of ~56.37MiB at 591.62KiB/s ETA 00:00
15/10/2018 15:51:57 - [download]  99.8% of ~56.37MiB at 605.33KiB/s ETA 00:00
15/10/2018 15:51:57 - [download] 100.0% of ~56.37MiB at 577.04KiB/s ETA 00:00
15/10/2018 15:51:57 - [download] 100.0% of ~56.37MiB at 577.04KiB/s ETA 00:00
15/10/2018 15:51:57 - [download] 100% of 56.37MiB in 02:00
15/10/2018 15:51:58 - [debug] ffmpeg command line: ffprobe -show_streams 'file:/mnt/hd2/My Video/Bitz & Bob, You Can Do It Too! - Series 2, Woolly Wonder Girl.mp4'
15/10/2018 15:51:59 - [ffmpeg] Fixing malformed AAC bitstream in "/mnt/hd2/My Video/Bitz & Bob, You Can Do It Too! - Series 2, Woolly Wonder Girl.mp4"
15/10/2018 15:51:59 - [debug] ffmpeg command line: ffmpeg -y -i 'file:/mnt/hd2/My Video/Bitz & Bob, You Can Do It Too! - Series 2, Woolly Wonder Girl.mp4' -c copy -f mp4 '-bsf:a' aac_adtstoasc 'file:/mnt/hd2/My Video/Bitz & Bob, You Can Do It Too! - Series 2, Woolly Wonder Girl.temp.mp4'

Description of your issue, suggested solution and other information
When piping the ouput of youtube-dl to a file with the --newline option there can be many hundreds of download progress messages - several per second which can be excessive.
I would like an option  to control how frequently the down load status is updated.
eg --progress 30  to update the status every 30 seconds
I know I can use the --no-pogresss opion to suppress the download progress messages - I would like something in-between,
BTW It would also be nice if all ouput messages had a time stamp so it was not necessary to use awk or other tool to add time stamps

hi, is this still an issue ? if so i would want to know how to contribute

I am not aware on any change being made in this area so yes it is still an issue.


ROADMAP

I'm going to be spending some more time on the project in my spare time and would like to outline my basic plan going forward. Would love to hear any/all feedback from users, other contributors, etc. on what they'd like to see added, fixed, etc.
In no particular order:
Things I'm fairly comfortable tackling:

[x] Merge/ close existing PRs. ( #57, #59, #68 #53)
[x] Publish <IP_ADDRESS>.1. This release includes some nice feature additions, bug fixes, and signals that the project is still active.
[x] General cleanup -- Give the code a thorough readover, turn on clippy's pedantic mode and fix some of the style warnings, etc. (#87, #88)
[x] Explore improving data loading at server start using serde (#75) -- This requires a bit of a refactor, which I've started exploring in this branch. (#91)
[x] Support compile_commands.json (#77) (#96)
[x] Unit testing -- The project currently lacks testing for nearly all of its core functionality. I've spent a little time working on zls and I think the general approach for testing features like hover, autocomplete, etc. in that project could work here as well. (#83, #84)
[x] Better logging -- Taking a second look at what's being logged, as well as adding some perf tracing (maybe tracy?) to help find bottlenecks (#88)
[x] Inline diagnostics -- Parse the output from a build as specified in compile_commands.json and provide inline diagnostics with any warnings/ errors. (#85) (#98)

Stretch Goals/ Maybe's

[x] ARM Support ( #17) -- This really just requires some RTFMing and writing an xml parser for ARM's docs. (#118)
[x] RISCV Support (#101) -- Same as ARM. (#123)
[ ] MASM64 Support #72 -- More research/ resources needed
[ ] Explore creating/ contributing to a more accurate tree-sitter grammar. (See this comment by the current tree-sitter maintainer on their own WIP grammar targeting multiple flavors)

Other/Misc

[x] Add z80-related topics to the repo to help with discoverability. (Added to the crates.io package via Cargo.toml, I don't have permissions to update tags/topics on the github repo)

Superseded by #170


NIFI-3412 ignoring tests which depend on timing/network setup

Thank you for submitting a contribution to Apache NiFi.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[ ] Is there a JIRA ticket associated with this PR? Is it referenced
in the commit message?

[ ] Does your PR title start with NIFI-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.

[ ] Has your PR been rebased against the latest commit within the target branch (typically master)?

[ ] Is your initial contribution a single, squashed commit?

For code changes:

[ ] Have you ensured that the full suite of tests is executed via mvn -Pcontrib-check clean install at the root nifi folder?
[ ] Have you written or updated unit tests to verify your changes?
[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[ ] If applicable, have you updated the LICENSE file, including the main LICENSE file under nifi-assembly?
[ ] If applicable, have you updated the NOTICE file, including the main NOTICE file found under nifi-assembly?
[ ] If adding new Properties, have you added .displayName in addition to .name (programmatic access) for each of the new properties?

For documentation related changes:

[ ] Have you ensured that format looks appropriate for the output in which it is rendered?

Note:
Please ensure that once the PR is submitted, you check travis-ci for build issues and submit an update to your PR as soon as possible.

Reviewing...

Ran all tests on Mac OS X 10.11.6 with Java 1.8.0_101 and all passed. Contrib-check passes as well. Merged and closed Jira.


Refactor IndividualUser

Reduce the amount of repetition in IndividualUser by turning most of the
permission checks into calls to helper methods.
No change in functionality here (existing tests continue to pass as-is).

Current dependencies on/for this PR:

main

PR #794

PR #795

PR #796   👈

PR #797

PR #798

This comment was auto-generated by Graphite.

Current dependencies on/for this PR:

main

PR #794

PR #795

PR #796   👈

PR #797

PR #798

This comment was auto-generated by Graphite.


Update 10-mqtt.html

Added text to the MQTT description to be a bit more clear on what parts of the message are queued.

Coverage decreased (-0.1%) to 88.27% when pulling 0154ce3df49302917c2f8ba61612f0d43c768a7a on Stubbs:patch-2 into b8f578862e895ec797973ddecd5a737346d75385 on node-red:master.


No ReactJS questions

Add questions and answers that might test core React concepts and mimic questions that might be asked in phone screens. Should be concise enough to fit into 1-2 sentences. Fill in here and submit pull request.
Example of question types / length

Will be working on this


arch: arm: Refactor CONFIG_CORTEX_M

Clean up Kconfig so each SoC just selects the specific Cortex-M
implementaiton rather than having to select both CORTEX_M and
CORTEX_{M0, M3, M4, etc.}.
Signed-off-by: Kumar Gala<EMAIL_ADDRESS>

Codecov Report

Merging #6506 into arm will decrease coverage by <.01%.
The diff coverage is n/a.

@@           Coverage Diff            @@
##             arm   #6506      +/-   ##
========================================
- Coverage   53.2%   53.2%   -0.01%
========================================
  Files        436     436
  Lines      41353   41353
  Branches    7937    7937
========================================
- Hits       22003   22001       -2
- Misses     16126   16128       +2
  Partials    3224    3224

Impacted Files
Coverage Δ

samples/philosophers/src/main.c
96.87% <0%> (-1.57%)
:arrow_down:

kernel/posix/pthread.c
62.5% <0%> (-0.5%)
:arrow_down:

Continue to review full report at Codecov.

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

Please, comment on my proposal to re-work the CPU_CORTEX_Mxx defines

Take a look at my update, I want to avoid patterns like depends on M0 || M0+ || M3 || M4 || M23 || M33 || M7, so want to keep the CPU_CORTEX_M option around.  But we can isolate the bulk of things

@galak It looks OK to me. I think we need to stress (e.g. with a comment) in ./arm/core/cortex_m/Kconfig, that any definitions shall be placed after #if CPU_CORTEX_M.

@galak It looks OK to me. I think we need to stress (e.g. with a comment) in ./arm/core/cortex_m/Kconfig, that any definitions shall be placed after #if CPU_CORTEX_M.

Ok, I'll add a comment about that.

@ioannisg added comment block, see if that looks good.

Looks good.

@galak This PR has accidentally removed the select ARMV7_M_ARMV8_M_FP if CPU_HAS_FPU
configurations in CM4, CM7, and CM33, originaly merged to "arm" by #6390 . Could you revert & fix this one, or shall we make a new PR?


tools: Check for external tools needed in check_format.py

Description:
Before this change, check_format.py fix was crashing infinitely
when external tools like clang-format-7 or buildifier were not
available. After this change, the script looks up for those tools
and shows an error if they are unavailable. Error messages contain
instructions how to use CLANG_FORMAT and BUILDIFIER_BIN env
variables and how to install buildifier.
Risk Level: low
Testing: n/a
Docs Changes: n/a
Release Notes: n/a
Fixes: #4786
Signed-off-by: Michal Rostecki<EMAIL_ADDRESS>

@mrostecki please don't squash commits or it will be hard to see changes after review.
LGTM, @jmarantz are you happy with this?

Would love to see a test covering all the error messages

That's OK with me, but maybe the author could just try it manually once with PATH set to some directory that has sym-links to python and other bare essentials and just make sure the error messages come out as expected, and the exit-status is non-zero.

I'm assuming @mrostecki did test this, but that's a good question. @mrostecki we assume you manually tested and everything works well?

@jmarantz I like your idea about writing tests for my patch. I can do that as a follow-up PR next week.
Although, I disagree with the comment inline.


Make transformer parameter nullable

Make transformer parameter nullable for paginator/collection

@specialtactics I created this in response to this comment.

@specialtactics Kindly help merge. This is a fix to enable the auto data transform feature (based on transformer bindings) to work correctly...

Hey @xwiz
Thanks for bringing this to my attention, not sure how I missed it ! I will merge and tag as v3.0.3


Npcap CI

Closes #196 #134

Disclaimer:

Secrets are not passed to workflows that are triggered by a pull request from a fork.
and since we need secrets to silently install npcap with the oem version we cannot run tests using npcap in pull requests from public forkrs. Therefore, windows test builds are now restricted to after PRs have been accepted. Since we still build the windows binaries and Linux and MAC OS X still run the tests that's acceptable for a bindings library.


Matplotlib window behaves annoying with Stage Manager on MacOS

Does this issue occur when all extensions are disabled?: Yes

Version: 1.87.2 (Universal)
Commit: 863d2581ecda6849923a2118d93a088b0745d9d6
Date: 2024-03-08T15:21:31.043Z (1 wk ago)
Electron: 27.3.2
ElectronBuildId: 26836302
Chromium: 118.0.5993.159
Node.js: 18.17.1
V8: <IP_ADDRESS>-electron.0
OS: Darwin arm64 23.0.0
Steps to Reproduce:

Write a python script that will show a maptlotlib figure.
Debug the file.
The figure window will pop out, and the VS Code window is hidden. I can drag the VS Code window and the figure window together to view the figure and code at the same time. But if I restart debugging, I will have to drag the new figure window and VS Code window again. Is it possible to make it default that the popped out figure window stays together with VS Code window?

Please see the recorded screen directly.
https://github.com/microsoft/vscode/assets/25512742/eeb6b439-5d13-4d14-b1ab-73ab8172438f
I really like Stage Manager and VS Code is the biggest obstacle that stops me enabling Stage Manager all the time.

Unfortunately VS Code cannot control the external windows/applications displayed.
You can get around this by installing the Jupyter extension and running code in a notebook or Python file as follows
This is made possible with the ipympl python package.
More information here https://matplotlib.org/ipympl/
You can find more information about Widgets here https://jupyter.org/widgets
& Jupyter here https://jupyter.org/
Here's all you need

Install the Jupyter extension
Write the following code in a python file & from the command Palette select the command Jupyter: Run All Cells

When prompted click ok to install ipykernel (this is the Python kernel required by Jupyter)

#%%
!pip install ipympl

#%%
%matplotlib widget
import numpy as np
import matplotlib.pyplot as plt

x = np.linspace(0, 20, 200)
plt.plot(x, np.sin(x))
plt.legend(bbox_to_anchor =(1.1, 1.0))
plt.show()

Some times the widget might not load the first time, this is because everything was just installed,
You will need to close this window and re-run the second cell (i.e. you can now remove the code pip isntall...)

Here is a sample of this working in VS Code with Jupyter extension


programatically stopping a loop

Hi, great library, thanks a lot...
Documentation is good too! My issue is following:
I do want to start two separate rest api services via corutines in async way. I have no issues setting up a handler to catch exceptions in event_loop.
however I want to be able programmatically to stop the loop.
anything after run(loop=loop) is not going to work!
what is the way to actually stop the loop and consequently stop two rest api verticals....
Regards,

It depends most strongly on the REST servers you're using. Each of the frameworks (aiohttp, fastapi, sanic, etc.) will have a slightly different way of shutting down.  Usually they (the frameworks) control the starting up and shutting down of the event loop, so aiorun probably can't help much here.

however I want to be able programmatically to stop the loop.
anything after run(loop=loop) is not going to work!

To answer this question from the perspective of aiorun, all you have to do is call asyncio.get_running_loop().stop().  This will initiate the shutdown process, which means the sequence of steps listed in the README such as task cancellation, waiting for all tasks to finish, shutting down the executor etc. Whether or not this works for the specific web frameworks you're using is more a matter for those frameworks.
It sounds like you've got a solution using processes and process termination, which I agree is a pretty clean way to do it since you will also have memory space isolation between the different services.

Yes, I agree.
On Sat, Sep 24, 2022, 6:15 AM Caleb Hattingh @.***>
wrote:

however I want to be able programmatically to stop the loop.
anything after run(loop=loop) is not going to work!
To answer this question from the perspective of aiorun, all you have to do
is call asyncio.get_running_loop().stop(). This will initiate the
shutdown process, which means the sequence of steps listed in the README
such as task cancellation, waiting for all tasks to finish, shutting down
the executor etc. Whether or not this works for the specific web frameworks
you're using is more a matter for those frameworks.
It sounds like you've got a solution using processes and process
termination, which I agree is a pretty clean way to do it since you will
also have memory space isolation between the different services.
—
Reply to this email directly, view it on GitHub
https://github.com/cjrh/aiorun/issues/76#issuecomment-1256931461, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAEC5FRUCLR3FYIF56N25WDV73ICNANCNFSM6AAAAAAQLYGM3E
.
You are receiving this because you authored the thread.Message ID:
@.***>


[SPARK-38176][SQL] ANSI mode: allow implicitly casting String to other simple types

What changes were proposed in this pull request?

Compared to the default behavior, the current ANSI type coercion rules don't allow the following cases:

Comparing String with other simple types, e.g. str_col > date'2022-01-01'
Arithmetic operation containing String and other simple types
Union/Intersect/Except containing String and other simple types
SQL function expects non-string types but got  string input
other SQL operators..

This PR is to remove the limitation. After changes, the String type can be implicit cast as Long/Double/Date/Timestamp/Boolean/Binary.
Note that Byte/Short/Int is not on the precedent list of String: str_col > 1 will become cast(str_col as long) > 1L. So that we can avoid string parsing error if the string is out of the range of Byte/Short/Int in comparison/arithmetic/union operations.
The design applies to Float/Decimal (especially Decimal), for SQL operators containing Float/Decimal and String, the type coercion system will convert both as Double.

Why are the changes needed?

The purpose of the current limitation is to prevent potential String parsing errors under ANSI mode. However, after doing research among real-world Spark SQL queries, I find that many users are actually using String as Date/Timestamp/Numeric/etc in their queries.  For example, the purpose of query where str_col > date'2022-01-01' is quite obvious, but users have to rewrite it as where cast(str_col as date) > date'2022-01-01' under ANSI mode.
To make the migration to ANSI mode easier, I suggest removing this limitation. Let's treat it as an extension in our SQL dialect.
Does this PR introduce any user-facing change?

Yes, allow implicitly casting String to other simple types under ANSI mode
How was this patch tested?

Unit tests

cc @cloud-fan @entong @allisonwang-db

Merging to master for unblocking https://issues.apache.org/jira/browse/SPARK-38154


maint: install wheel explicitly for cibuildwheel on Windows

This is needed for cibuildwheel >= 2.18.0 since wheel is no longer installed implicitly.

See gh-138


Improve talker_bloc_logger by adding onChange, onCreate, onClose

Hello,
It would be great if you added support for these events, especially since without them no cubit logs.
https://pub.dev/documentation/bloc/latest/bloc/BlocObserver-class.html

@MiniSuperDev hello !
Thank you for this issue. Yes, this functionality will be great.
You can implement this if you have enough time. This project open for new contributors

Resolved in https://github.com/Frezyx/talker/pull/162


Rescoring function?

Hello
Is there a rescoring functionality available in idock?
Thanks

There is a score_only option

Thank you.
Le ven. 12 avr. 2019 à 09:37, Jacky Lee<EMAIL_ADDRESS>a écrit :

There is a score_only option
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/HongjianLi/idock/issues/29#issuecomment-482471674,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AM6CFYj-NhZ74tm44rrl4vDbzrF4It8Gks5vgDecgaJpZM4chyGU
.

--
Bakary N’tji DIALLO
PhD Student (Bioinformatics) http://linkedin.com/in/bakarydiallo, Research
Unit in Bioinformatics (RUBi) https://rubi.ru.ac.za/
Mail<EMAIL_ADDRESS>|  Skype: diallobakary4
Tel:  +27798233845 | +223 74 56 57 22 | +223 97 39 77 14


Allow changing the .env file

As requested by @GaryJones in #12 here:

Probably related here, but I see that Dotenv supports passing in a file name.

My situation is that I've got a /var/www that contains directories for different sites. If I want to store multiple .env files, one for each site, above the public roots, I'll obviously need different filenames. Being able to pass through the file name as the second arg in the $env = WCM\WPStarter\Helpers::settings(__DIR__); call would be handy.

Done.


Question 2 to 10 are not in correct order.

In Readme file, The sequence of Quetion 2 to 10 are not in correct order.
for ex. Question no. 2: What are the major features of React?
and in answer it shows Answer no. 3.
It will be the same upto Question 10.
There are Two answers under 10 number.

@chetannada  Thanks for creating the PR. Please check my comment in PR


class merge generic parameters

Suggestion
class Component<Props extends {}> {
    public constructor (props: Props) {
        Object.assign(this, props);
    }
}

class App extends Component<{ name: string }> {
    public init() {
        // Property 'name' does not exist on type 'App'.ts(2339)
        this.name
    }
}

🔍 Search Terms

List of keywords you searched for before creating this issue. Write them down here so that others can find this suggestion more easily and help provide feedback.
✅ Viability Checklist

My suggestion meets these guidelines:

[x] This wouldn't be a breaking change in existing TypeScript/JavaScript code
[x] This wouldn't change the runtime behavior of existing JavaScript code
[x] This could be implemented without emitting different JS based on the types of the expressions
[x] This isn't a runtime feature (e.g. library functionality, non-ECMAScript syntax with JavaScript output, new syntax sugar for JS, etc.)
[x] This feature would agree with the rest of TypeScript's Design Goals.

⭐ Suggestion

📃 Motivating Example

💻 Use Cases

Is there a way to make the above code work

Duplicate of #26792.


examples: all new examples

These examples are now based on the new wrappers. Requires neo-express to be installed. Neo-express is wrapped with automatically reset & run (and shutdown) a private chain so all of the examples can be played with in isolation.

[ ] NEP-11 airdrop
[ ] NEP-17 airdrop (implemented, but blocked by broken contract)

Coverage remained the same at 81.59% when pulling bc3a982257f4ccf9457c852510d879ec624d1e37 on new-examples into<PHONE_NUMBER>c7e8f36f035e0e2b230b7b78088a49 on master.


Add pinned yarn 4.4.1 packageManager

Summary
Fixes #

Testing yarn packageManager pinning
Occurred changes and/or fixed issues

Technical notes summary

Areas or cases that should be tested

Areas which could experience regressions

Screenshot/Video

Checklist

[ ] The PR is linked to an issue and the linked issue has a Milestone, or no issue is needed
[ ] The PR has a Milestone
[ ] The PR template has been filled out
[ ] The PR has been self reviewed
[ ] The PR has a reviewer assigned
[ ] The PR has automated tests or clear instructions for manual tests and the linked issue has appropriate QA labels, or tests are not needed
[ ] The PR has reviewed with UX and tested in light and dark mode, or there are no UX changes

Closing for now until tests can resolve.


FileNotFound error

Hi, I'm trying to run the code but it gives me multiple FileNotFound errors.
Please help.
Thank you!
The output is:
################################################################################
AutoPentest-DRL: Automated Penetration Testing Using Deep Reinforcement Learning
################################################################################
AutoPentest-DRL: Operation mode:  Attack on logical network
AutoPentest-DRL: Target topology: MulVAL_P/logical_topology_1.P
AutoPentest-DRL: Compute attack path for logical network...
Generate attack graph using MulVAL...
sh: 1: ../repos/mulval/utils/graph_gen.sh: not found
Process attack graph into attack matrix...
Traceback (most recent call last):
File "/home/leekutti/NT522/AutoPentest-DRL/DQN/./confirm_path.py", line 9, in
MAP = generateMapClass.sendMap
File "./learn/generateMap.py", line 108, in sendMap
self.x = self.createMatrix()
File "./learn/generateMap.py", line 20, in createMatrix
self.csvfile = open('../mulval_result/VERTICES.CSV', 'r')
FileNotFoundError: [Errno 2] No such file or directory: '../mulval_result/VERTICES.CSV'
Traceback (most recent call last):
File "/home/leekutti/NT522/AutoPentest-DRL/DQN/learn/./dqn_learn.py", line 32, in
env = gym.make('dqnenv-v0')
File "/usr/local/lib/python3.9/dist-packages/gym/envs/registration.py", line 235, in make
return registry.make(id, **kwargs)
File "/usr/local/lib/python3.9/dist-packages/gym/envs/registration.py", line 129, in make
env = spec.make(**kwargs)
File "/usr/local/lib/python3.9/dist-packages/gym/envs/registration.py", line 89, in make
cls = load(self.entry_point)
File "/usr/local/lib/python3.9/dist-packages/gym/envs/registration.py", line 27, in load
mod = importlib.import_module(mod_name)
File "/usr/lib/python3.9/importlib/init.py", line 127, in import_module
return _bootstrap._gcd_import(name[level:], package, level)
File "", line 1030, in _gcd_import
File "", line 1007, in _find_and_load
File "", line 986, in _find_and_load_unlocked
File "", line 680, in _load_unlocked
File "", line 790, in exec_module
File "", line 228, in _call_with_frames_removed
File "/home/leekutti/NT522/AutoPentest-DRL/DQN/learn/env/environment.py", line 12, in
class dqnEnvironment(gym.Env):
File "/home/leekutti/NT522/AutoPentest-DRL/DQN/learn/env/environment.py", line 14, in dqnEnvironment
MAP = np.loadtxt('../processdata/newmap.txt')
File "/usr/lib/python3/dist-packages/numpy/lib/npyio.py", line 961, in loadtxt
fh = np.lib._datasource.open(fname, 'rt', encoding=encoding)
File "/usr/lib/python3/dist-packages/numpy/lib/_datasource.py", line 195, in open
return ds.open(path, mode, encoding=encoding, newline=newline)
File "/usr/lib/python3/dist-packages/numpy/lib/_datasource.py", line 535, in open
raise IOError("%s not found." % path)
OSError: ../processdata/newmap.txt not found.

Hi,
From the output you included I can see that the first error is:
sh: 1: ../repos/mulval/utils/graph_gen.sh: not found
This means that you forgot to install MulVAL, and the script that comes with it cannot be found. Please check the Prerequisites section in the README.md file for details about MulVAL.
Best wishes,
Razvan

Thank you for your support, but I have one more question.
In the paper you wrote that this model has  an accuracy of 0.86. I quite don't understand the method of evaluating, the data used for evaluating and whether that data is in this repo or not.
Looking forward to your reply.
Thank you!

I'm glad to hear that your problem was solved. As for the evaluation, I'll answer in the new issue that you just opened about this topic.


Update msb3821.md

This is another fix for the "MSB3821" error

@GeeVont3, thanks for the suggestion! I made a few edits and added a link to the PowerShell command that users can also try.


Refactor

Thanks for sending a pull request!
Please make sure you click the link above to view the contribution guidelines,
then fill out the blanks below.
Description
(Brief description on what this PR is about)
What does this implement/fix? Explain your changes.
...
Pull request type
[DOC] Documentation content changes
Changes
-Update docs for SIF part and Pretrain part
Does this close any currently open issues?
...
Any relevant logs, error output, etc?
...
Checklist
Before you submit a pull request, please make sure you have to following:
Essentials

[ ] PR's title starts with a category (e.g. [BUGFIX], [FEATURE], [BREAKING], [DOC], etc)
[ ] Changes are complete (i.e. I finished coding on this PR)
[ ] All changes have test coverage and al tests passing
[ ] Code is well-documented (extended the README / documentation, if necessary)
[ ] If this PR is your first one, add your name and github account to AUTHORS.md

Comments

If this change is a backward incompatible change, why must this change be made.
Interesting edge cases to note here

貌似还是有一些问题，不过不是我改动的文件emm


How to set a max size file for images in uploads/big directory?

Hi,
i've set a large amount of pictures in uploads/import.
These pictures are straight right from camera, so they are around 6 or 8Mb.
I import them to an album (it's very long and there is no way to know when it'll be finished or which file is currently in work, that's a bad thing), and then, these files are stored in uploads/big dir.
I'm using mogrify to decrease their size to max 3Mb, but is there a way to set it up in a config file?
Here is the command i use in uploads/big dir after importing:
find -iname "*.jpg" -type f -print0 | parallel --progress -0 -j +0 "mogrify -resize 2000000\> {}"
So my asks / thoughts:

add a display to tell user which file is currenlty in work when importing mass files
set a max file size to store in big directory.

Thanks a lot.
N.

This issue will be closed because I'm not actively working on Lychee anymore. The repository of Lychee will stay available, but the main development continues under the Lychee Organisation. A fork maintained by the community. Important issues have been migrated. Feel free to re-add your issue(s) when necessary.


Added smooth pixel-perfect camera example + Small typo fix in examples_template.c

As I wrote in the title, I added the smooth-pixel perfect camera example.
My only concern is using the <math.h> header to access sin() and cos() functions.
I also added a .png file as cover image, and fixed a small typo in examples_template.c (changed "bsasic" to "basic").

Should compile now @ProphesorIks


Consume dev site props table loader

Summary
Testing the new custom react-docgen based props-table-loader from terra-dev-site

Closing this in favor of a follow up PR once terra-dev-site supports MDX


feat: added checkbox component

References

Jira
Checkbox component (figma)
Lead message (figma)

Motivation and context
Extracting Checkbox component.
Before
No checkbox component😱
After
Checkbox component 🥳

How to test
Please check the checkbox states: selected, unselected, disabled, invalid, hover. And the design.
We don't have the branch deployment yet but you can check the branch locally.
Thank you 🌮

@lkappeler
Sorry for creating a new PR, I had some issues on the other PR regarding the pre-release and I forgot to add the link from that PR  😣 but I tackled your feedback 🥳
Maybe you can do another review here. Thank you  🧸