Module '"discord.js"' has no exported member 'ClientApplication'.

Describe the issue
I just got the next error while initializing my discord bot:
Module '"discord.js"' has no exported member 'ClientApplication'.
Code or steps to reproduce
index.ts file
import { Client, KlasaClientConfig } from 'klasa';
import { config, token } from './config';

class CustomClient extends Client {
  constructor(options: KlasaClientConfig) {
    super(options);
  }
}

new CustomClient(config).login(token);

package.json script runned: "start"
"scripts": {
    "build": "yarn tsc",
    "lint": "tslint -c tslint.json 'src/**/*.ts'",
    "lint:fix": "yarn lint --fix",
    "start": "yarn build && node ./build/index.js",
    "tsc": "tsc"
}

Expected and actual behavior
Run just fine?
Further details

discord.js version: 11.5.1
node.js version: 12.4.0
Klasa version: 0.4.0

It's exported, and nobody else gets this error:
https://github.com/discordjs/discord.js/blob/ea9e1441905e67e2627a6f5e80a8d989d01376c5/typings/index.d.ts#L248-L266
Some (+10) production bots are written in Klasa and use the latest commits, and they build fine.
My guess is that yarn corrupted your install, I suggest you to clean your yarn cache (yarn cache clean), nuke the node_modules folder (rm -r -Force node_modules in PowerShell, rm -rf node_modules in Linux), the yarn.lock, and re-install.

I did so by:

running yarn cache clean
deleting both node_modules and yarn.lock
re-install my dependencies

The problem persist, any other idea?

@enbermudas klasa does not support discord.js v11.
Klasa v0.4.0 aka stable doesn't actually work.
You will need to switch to klasa#master v1.0.0-alpha or settings branch and discord.js master aka v12-dev.
Discord.js
# Npm
npm i discordjs/discord.js
# Yarn
yarn add discordjs/discord.js

Klasa
# NPM
npm i dirigeants/klasa
# Yarn
yarn add dirigeants/klasa

If you want to use the v1.0.0-alpha or settings branch apply #branchname after dirigeants/klasa

I installed those two dependencies and there are 58 errors now within the typings.

Has this been fixed yet? There was a discussion in the support server that led to a fix.
The culprit was the lack of "types": ["node", "discord.js", "klasa"] in the tsconfig.json's compilerOptions, and the error is pretty understandable as:
TSC (TypeScript Compiler) reads the main file, detects klasa is being imported, a module that defines both klasa itself (it's own definitions) and discord.js (augmented definitions), but since the real discord.js module has not been loaded, TSC takes the augmentations as the type source, which is not valid, causing this problem.
With the types being defined, TSC will load Node.js's types (which defines NodeJS.Timeout, Buffer, and others, which both discord.js and klasa rely on), then it will load discord.js, which defines ClientApplication and many other classes klasa imports, and finally, klasa itself.
That way when you load your project and TSC finds the import { Client } from 'klasa';, all the types are pre-loaded, and everything is available and valid.

it was also because they were trying to compile to es4 or es5 code, instead of es6/esnext


How to do bilinear gradient?

How to draw bilinear gradient?

just came here to say: this image is incredible.
does it appear as if it's moving when looking around to anyone else??

haha, I do not know whatcha taking 'bout man. what actually TF?

Just do 2 linear gradients?

And how to combine them to 1 geometry?

They wont be 1 geometry--Why do they need to be?

Because every geometry is a draw call. Why to have 2 draw calls?

I highly doubt this will be the bottleneck in your application. AFAIK NanoVG doesn't support bilinear gradients, so you're stuck with your 2 drawcalls 🤷♂

I was hoping that someone would know how to modify nanovg to support bilinear gradient.


Set a CLI specific trace-id into the container

So calls to $container->get('trace-id') do not fail.
Purpose
The actorcode generation was failing due to logging code changes, this fixes that case.
Fixes UML-574
Approach
Attaches a CLI specific trace-id into the container when building the code creation Symfony command.
Checklist

[x] I have performed a self-review of my own code
[ ] I have updated documentation (Confluence/GitHub wiki/tech debt doc) where relevant
[ ] I have added tests to prove my work
[ ] The product team have tested these changes

Coverage decreased (-0.07%) to 77.132% when pulling 3ecd93c4049ac4028f682bb80e6a4aaff5a55f01 on UML-574-fix-codegen-tool into f59614a3b7b23b098324ecefa970fdd3de953d56 on master.


chore(package): update semantic-release to version 15.13.24

Coverage remained the same at 100.0% when pulling dab6fd07b88ba50788dad472d220d3c3e95c534e on greenkeeper/semantic-release-15.13.24 into 48eb5b83ceb0f50d4d9930a8622c9f0733fb951d on master.

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


moved menus - update documentation

I was just cleaning up the Tools > Measurement menu because it became a bit crowded. There are now two new menus Measurement maps and Measurement tables.

Todo:

We should update the clusters plotter documentation to hint users to Tools > Measurement tables instead of Tools > Measurement (this should be easy)

Optional todo:

We could create another menu Tools > Measurement post-processing and put Dimensionality reduction (ncp) and Clustering (ncp) in there. I'm not sure if this would be helpful. Thus, I leave it for discussion :-)

I like the idea of having dimensionality reduction and clustering widgets in "Tools > Measurement post-processing" menu. I think it makes a lot of sense to separate them from the rest. Thanks, Robert @haesleinhuepf!


URBMediaFocusViewController fails to link against iOS 6.1

Despite the readme claiming "NOTE: This project has been updated to work with and tested against iOS 6. However, the UIDynamics features are only support in iOS 7+.", the included sample project fails to build when its deployment target is changed to 6.0 and is built for iOS 6.1:
dyld: Symbol not found: _OBJC_CLASS_$_UIAttachmentBehavior
  Referenced from: /Users/cbowns/Library/Application Support/iPhone Simulator/6.1/Applications/E647C36C-6255-4E4B-A976-D193A0EE7870/URBMediaFocusViewControllerDemo.app/URBMediaFocusViewControllerDemo
  Expected in: /Applications/Xcode.app/Contents/Developer/Platforms/iPhoneSimulator.platform/Developer/SDKs/iPhoneSimulator6.1.sdk/System/Library/Frameworks/UIKit.framework/UIKit
 in /Users/cbowns/Library/Application Support/iPhone Simulator/6.1/Applications/E647C36C-6255-4E4B-A976-D193A0EE7870/URBMediaFocusViewControllerDemo.app/URBMediaFocusViewControllerDemo

URBMediaFocusViewController has several properties which are only available on 7.0 and later:
@property (nonatomic, strong) UIDynamicAnimator *animator;
@property (nonatomic, strong) UISnapBehavior *snapBehavior;
@property (nonatomic, strong) UIPushBehavior *pushBehavior;
@property (nonatomic, strong) UIAttachmentBehavior *panAttachmentBehavior;
@property (nonatomic, strong) UIDynamicItemBehavior *itemBehavior;

There's ways to make this class work against pre-7.0, but it requires some hoop-jumping. Is the intention to make this build and run on 6.x? If so, I can help fix it up, but if not, the readme should be updated.

Closing this out since minimum requirement since the class is no longer guaranteed to support iOS older than 8.0.


Rootkit, web cam capture and lan scanner - WTF?!

Rootkit
https://github.com/Cyberes/automated-fbi-reporter/blob/7985984f4ce06feeb4a1372fef16690d755832d7/scripts/main.py#L84
Webcam capture
https://github.com/Cyberes/automated-fbi-reporter/blob/7985984f4ce06feeb4a1372fef16690d755832d7/scripts/main.py#L172
Lan scanner
https://github.com/Cyberes/automated-fbi-reporter/blob/7985984f4ce06feeb4a1372fef16690d755832d7/scripts/main.py#LL182C10-L182C10
What script kiddie nonsense is this?

The purpose of these features is to collect information for future legal actions. You seem to be very defensive about this, is there something you need to confess?

Just don't generate child pornography. Professional help is available: https://bullcitypsychotherapy.com/child-pornography/

I have no intention of generating child pornography but it seems to be on your mind a lot more than most. Your example searches are absolutely sick.
Nobody is buying your facade GET SOME HELP.

You say that, but I'm getting a steady stream of reports that depict the true nature of Stable Diffusion users. This morning I checked the database to make sure there were no errors and what I saw horrified me. Thousands of prompts from hundreds of users (in reality, future convicted criminals) prompting unspeakable prompts. I felt physically sick.
Don't believe me? Here's a relatively SFW prompt that I found in the DB.

(censored the prompt as to not give these criminals any ideas)

Yes it's going to happen, but they're not only using Stable Diffusion are they? They're also using keyboards aren't they? So do you think it's OK if someone puts a bug in everyone's keyboard? In case they search for child porn? If not, then where's the line drawn between targeting one technology for being spied on? You may think this is low hanging fruit and an easy win and that's commendable that you make an effort, but not at the expense of everyone's privacy.  I'm sorry but leaking evidence of of criminal activity to the public is a lot more commendable than spying on people in case they commit a crime. It's also a lot more legal.


autocompletion tab selection fails when search key is not at start of existing command

fixed with version 1.1.0


Sync two vue-cal instances : how to change month ?

Hello !
First of all, thank you for this plugin. It's a great job! I left to use a competitor (very old) and I got here, I love it!
I particularly appreciate the possibility of synchronizing 2 calendars.
I managed to implement the basic example.
In this example, the calendar is only synchronized when a day is selected. Is it possible to do this also when we change the month (by clicking on the arrows)?
Also, is it possible to synchronize the two together? If I change the day on the first, it changes on the second. And if I change the day on the second, does it change on the first?
Thanks again !

Hi @Jimmylet, Thanks for the feedback, I appreciate.
Yes you can fully sync 2 instances, even if I don't see a use case for it, here is a demo for you:
https://codepen.io/antoniandre/pen/ExPwGRm?editors=1010
Hope it helps!

Thank you ! The use case would be purely "user experience".
My idea is to display a calendar in "week" view. And to navigate simply and quickly, a calendar in "month" view.
Maybe it's not super useful, I'm going to do some user tests to determine it.


Gap Fill answer incorrectly becomes red when going over pages using next/back buttons

Using XOT 3.7.4 one can reproduce this by:

creating a new project
inserting a Gap Fill
set Interactivity to Fill in Blank
add Optional Property Mark at End
duplicate Gap Fill ones
select the first and mark the word |text|
select the second and mark the word |here|
click Play
enter "text"
click Check button (text correctly becomes green)
click Next button
click Back button
click Next button
click Back button
"text" incorrectly becomes red

I think the cause is the same as for issue #807

I can no longer reproduce this. Can anyone else reproduce: @fay @ronm123

Yes - I can recreate. You have to go to next page & back twice as described above - the first time you go back to page 1 the text is still green but the second time you go back the text is red.


Clean up params and tests of calculated initial values.

Some quick clean-up work.

@jschirrmacher Is it ok if I merge this?


remove permission check from quay navs

permission checks not required for quay anymore

/ok-to-test

The JSON is malformed somewhere:
16:03:30 1.173 File /src/chrome-service-backend/static/beta/prod/navigation/quay-navigation.json
16:03:30 1.175 panic: invalid character '}' looking for beginning of object key string


Implemented setModeConfiguration for monaco.languages.typescript.[typescript|javascript]Defaults

This commit implements the functionality requested in #3155.
The work is done based on how this function was implemented in the css, html and json namespaces.
There are some questions though:
The setupMode implementations for css, html and json return an IDisposable containing all disposable elements created during the function execution. But the setupMode implementation for typescript returns (...uris: Uri[]) => Promise<TypeScriptWorker> containing the worker function. So the code to collect all disposable elements is there, but useless unless it is returned in a future commit.

@microsoft-github-policy-service agree


TextStyle: debugLabel param ignored in copyWith unless receiver already has a debugLabel

See implementation of TextStyle.copyWith:
https://github.com/flutter/flutter/blob/a7aca4b995234358d63bc92e0020b7887ab7e86d/packages/flutter/lib/src/painting/text_style.dart#L894-L900
(where debugLabel is a copyWith param).
The debugLabel param is ignored unless the receiver (the TextStyle you're calling copyWith on) already has a debugLabel. This is problematic when you have a text style without a debug label and you want to make a copy that has a debug label.

Proposed code (I'll send a PR):
    String? newDebugLabel;
    assert(() {
      if (debugLabel != null) {
        newDebugLabel = debugLabel;
      } else if (this.debugLabel != null) {
        newDebugLabel = '(${this.debugLabel}).copyWith';
      }
      return true;
    }());


Visualizer - Asset source embeds absolute path instead of relative path

Description of bug
When you create a visualizer which has the ability to upload image assets it uploads images using the full url. Example:  dev.websitename.ca/images/image.jpg .
The issue occurs when your site is no longer in development and modify your domain so that it is now https://www.websitename/images/image.jpg.
The location of the asset stays the same as when it was uploaded, which results in the image or document not being displayed. If you upload a new image it will be displayed because it is uploaded using the current path.
Header Image - with  Search (2).zip
(https://user-images.githubusercontent.com/57957944/69168420-a36e3d00-0ac4-11ea-9742-69b8c1ae7475.png)
In this example the image displays properly as it has been uploaded on this domain. However please note in my example that the full path is being included.
Steps to reproduce
List the steps to reproduce the behavior:

Go to '.the header image visualizer  and install within your dnn instance we are 9.3.7.'
Add a new image,put in an images title,then click on upload image, upload an image from your computer , publish
image will be displayed
![image]
4.Go to site settings add a new site aliases , remove the old alias.
Go back to page which is displaying your visualizer content. Image source will be using the full path which was originally used to upload the file instead of the relative path.

Current result
Image source will be using the full path which was originally used to upload the file instead of the relative path.
Expected result
image source should use relative path...example
/image/image.jpg
Screenshots
If applicable, provide screenshots to help explain your problem.
Error log
no error...
Additional context
Add any other context about the bug that may be helpful for resolution.
Affected version
We were on 9.3.0 have since upgraded to 9.3.7
EVOQ CONTENT
v. 09.03.07 (1)
4.6
.NET FRAMEWORK

[ ] 10.0.0 alpha build
[ ] 9.4.2 alpha build
[ ] 9.4.1 latest supported release

Affected browser

[ ] Chrome
[ ] Firefox
[ ] Safari
[ ] Internet Explorer
[ ] Edge

Please report issues with Evoq to https://www.dnnsoftware.com/services/customer-support.  If there is a related bug in DNN Platform, please document the steps to trigger the issue without using Evoq.  Thanks!


Error Message

Hello there,
thanks so much for developing and sharing this code.
I am using it in a Google Sheet that you have provided. All runs smothly but I am seing the following error message every few minutes....
Any chance you have some advice on how to fix it?
Thanks.
Regards,
Jochen
Subject: [Website Status] Error: Status Check
On Sun, May 22, 2022 at 9:43 AM<EMAIL_ADDRESS>wrote:
Exception: Address unavailable:
https://www.vatvalve.com/page/vat-motion-components
at targetWebsites.reduce.newErrors (websiteMonitoring:451:23)
at Array.reduce ()
at websiteMonitoring (websiteMonitoring:436:39)
at websiteMonitoringTriggered (websiteMonitoring:300:3)
at GS_INTERNAL_top_function_call.gs:1:8

This notice is managed by the following spreadsheet:
https://docs.google.com/spreadsheets/d/1WDJx6iFu5KrL-2RQ70gEjBPbGCtHxvbM7IIIz6PO5gU/edit

Hi @OMGJS2022 thanks for your comment!
I've seen the same error in my case but it only happens once a month or two, so I've considered it an occasional glitch in Google's Apps Script execution... Every few minutes in your case does seem a bit too frequent.
I'll try to look into it, but to be honest I'm not sure where to start. Any additional inputs from other users are welcome!!

Thanks so much for your reply. I have since changed the interval from 1 to 5 minutes and the error has not reappeared. I might experiment with this a little over the next few days.


Add image tags under Tags property of a file (Windows 7+)

I recently discovered that, if one adds the tags of an image in the Tags property in the file properties under details tab (file meta), they can be searched and filtered just like on the sites. For example, entering "tag:bodysuit -tag:loli" will do the obvious function and be super fast too.
This can be added under save in settings like a configuration option. Tags are already in store when downloading an image so this doesn't seem like a very big leap over the existing functionality. But a huge boon to those who do not want to lose this function while being offline.

ummmm how do you use it ?????
where is that feature (External text file) ???

Closing as duplicate, if you have more info about some universal tagging system, I'd gladly implement it.
@baraa272 moved to #1498


How to have multiple slack integrations with Jenkins?

Under Global Slack Notifier Settings, there is only one text field for the integration token. How do I use this plugin for posting to multiple channels?
Eg:

#web-builds
#backend-builds
#android-builds

Configure it on each build, see the README
https://github.com/jenkinsci/slack-plugin#freestyle-job

Like this?
slackSend (token: 'abcdefghijk', channel: 'web-builds', message: 'hello')

And this will overwrite any values set in the Global Slack Notifier Settings?

yes but don't use token, for most usecases you should set a credential id globally
If you really need another one you can pass the credentialId to slackSend
The snippet generator will show you all the available options

incoming webhook of slack is mapped to a channel, so how can we use the same incoming webhook to integrate with multiple channels?

Can you send a link to the snippet generator?
I'm still confused on how to configure this plugin with multiple channels. It sounds like I need to configure globally the different tokens with credentialIds, and then reference that credentialId?
Like this?
slackSend (credentialId: 'slackTokenWeb', channel: 'web-builds', message: 'hello')

Credentials aren't tied to channels.
you can either do:
slackSend (channel: 'web-builds,web-builds2', message: 'hello')
slackSend (channel: 'web-builds', message: 'hello')
slackSend (channel: 'web-builds2', message: 'hello')

Snippet generator can be found on any pipeline job page
Click "Pipeline Syntax"
jenkins-url/job//pipeline-syntax/

Okay I figured this out. You just need one token, and you can just specify the channel/user that's to be messaged.
The problem I was having was that the global configuration was not inheriting to the pipeline, so everything needs to be defined explicitly.
This works for me:
WithCredentials([string(credentialsId: 'theSlackToken', variable: 'slackCredentials')]) {
    slackSend (channel: '@johnsmith', message: 'hello there', color: '#3eb991', failOnError: true, teamDomain: 'myteamsubdomain', token: slackCredentials)
}

I think @KristianWindsor is partially right, but only for public channels. If you want to send to multiple private channels, u have to use seperate tokens corresponding to each private channel.

hii, how to use slack notification for multibranch pipeline build?


docs(CONTRIBUTING): add windows subsystem for linux information

closes #160

Codecov Report

Merging #161 into main will decrease coverage by 4.37%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #161      +/-   ##
==========================================
- Coverage   70.43%   66.06%   -4.38%
==========================================
  Files          38       76      +38
  Lines         274      607     +333
  Branches       27      113      +86
==========================================
+ Hits          193      401     +208
- Misses         61      180     +119
- Partials       20       26       +6

Impacted Files
Coverage Δ

src/components/simplecast/simplecast-episode.tsx
50.00% <0.00%> (ø)

src/components/twitch/twitch.tsx
35.71% <0.00%> (ø)

src/components/flickr/utils.ts
66.66% <0.00%> (ø)

src/components/spotify/spotify.tsx
100.00% <0.00%> (ø)

src/components/twitter/utils.ts
42.85% <0.00%> (ø)

src/components/twitter/twitter-follow-button.tsx
50.00% <0.00%> (ø)

src/components/airtable/airtable-form.tsx
75.00% <0.00%> (ø)

src/components/strava/strava.tsx
75.00% <0.00%> (ø)

src/components/egghead/egghead-lesson.tsx
80.00% <0.00%> (ø)

src/components/codepen/codepen.tsx
81.81% <0.00%> (ø)

... and 28 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 2a895a1...c6e364b. Read the comment docs.


Improved load errors

I'm attempting to translate a large-ish Traceur project to SystemJS, and while I was able to fix most of the map conversions, it was with some difficulty. Traceur's output on a failed load was extremely useful. It lists:

the host file (as opposed to the root of the bundle)
the actual import name as used
how it was normalized. (i.e. the information SystemJS provides)

It's a little chatty, but really helped pin down exactly what the problem was and made it much easier to put together map configurations.

Sure, and thanks for posting here. What would you suggest for including this information in the current compiler we have?

I'm not sure what you mean? Code-wise, maybe I can rustle up time for a PR.
I assume you're not expecting "builder should emit this information with a load error" right?

A PR contribution would be incredibly welcome here given your recent experience with it.
Otherwise will certainly look into it when I can.

Digging into this, I realize that it's an example of https://github.com/ModuleLoader/es6-module-loader/issues/397 (with some of https://github.com/ModuleLoader/es6-module-loader/issues/399 to go with) - the issue should be addressed there, at which point this solves itself, so to speak.

So your issues were specific examples of those cases?

Tracking it down, the errors are generated within es6-module-loader. Because of the ENOENT related bug, they don't (I think?) show up as proper Error objects in SystemJS. As a result, the best that could be hoped for in "builder" would be a regexp match against the error message and then a new Error raised.
But: what I'm looking for here is exactly what https://github.com/ModuleLoader/es6-module-loader/issues/397 is talking about. The right fix, then, would seem to be to address that issue (and 399) so that ML reports the issue with loading.

Thanks for the update - great to know, will get to those as soon as I can.


Recommended typedef parameters are camel-cased instead of kebab-cased

Hello there :) , quick question about the recommended settings.
Im using Webstorm and latest tslint.
This rule set is on the recommended list, but seems that not trigger anything.
"typedef": [true, "callSignature", "indexSignature", "parameter", "propertySignature", "variableDeclarator", "memberVariableDeclarator"],

In the rules from tslint is define a little different
and it works.
"typedef":` [ true, "call-signature", "parameter", "variable-declaration", "property-declaration",  "member-variable-declaration" ]

So i was wondering if Im missing any setting,  is just a typo inside the recommended list or i didn't understood from where that typedef come from . :P
Thank you in advance.

You are correct! It is a bug. I suspect maybe the parameter names changed in the last 18 months.


Adding build customization

@shirshanka this is a PR against your kafkadeps branch. It allows customizing the gobblin build. ./gradlew -PgobblinFlavor=XYZ assemble
Where currently XYZ can be one of : minimal, standard (default), full, custom.
With "custom" one can decide on which dependencies to be included by editing the gobblin-flavor-custom.gradle files.

@chavdar thanks!


Terms query

@prog8 this is meant to address https://github.com/sematext/solr-redis/issues/23 .  Is this along the right lines?  Many of the tests now fail as TermsQuery doesn't implement extractTerms.

@smock - look at https://github.com/sematext/solr-redis/search?utf8=✓&q=BooleanQuery
You'll see @prog8 implemented term extractor for BooleanQuery.  I assume you'd need to do the same for TermsQuery.

@smock yes you are right all test which verify terms in query will fail since ConstantScoreWeight doesn't allow to extract terms. I'll look at this closer.

The reason is that during TermsQuery rewrite there is the decission whether TermIterator should be used or just BooleanQuery. For small number of terms BooleanQuery is used and it is later wrapped in ConstantScoreQuery.

In later versions of lucene-solr, we can use getTermData() on the query to get the number of terms for that check.  I'm updating my P.R. to work against Solr 6.0 -- it's what I'm using in my environment anyway.

I pushed fixed version. In Lucene 5.x getTermData is not exposed as public :-(

Yes, I'll work on a separate p.r. for Lucene/Solr 6.0


Add batch mutation for update and delete operation

Description
Even though batch creation of objects was already implemented a while ago (see https://github.com/strawberry-graphql/strawberry-graphql-django/issues/3), it is still not possible to update/delete objects in a batch.

Hey @Hnagzhi ,
Actually it is possible, you should use filters in the mutation as is explained here: https://strawberry-graphql.github.io/strawberry-graphql-django/guide/mutations/#filtering
Looking at the doc now I understand that it can be confusing and should be improved =P


Follow naming conventions in NetworkBehaviour m_NetIdentity

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


failed to do post-install when deploy my-minio

When I deploy minio by following quick start with command
kubectl apply -f https://raw.githubusercontent.com/kubebb/components/main/examples/minio/componentplan.yaml

It failed. The componentplan shows below:
apiVersion: core.kubebb.k8s.com.cn/v1alpha1
kind: ComponentPlan
metadata:
  annotations:
    core.kubebb.k8s.com.cn/componentplan-retry: "5"
    kubectl.kubernetes.io/last-applied-configuration: |
      {"apiVersion":"core.kubebb.k8s.com.cn/v1alpha1","kind":"ComponentPlan","metadata":{"annotations":{},"name":"my-minio","namespace":"default"},"spec":{"approved":true,"component":{"name":"kubebb.minio","namespace":"kubebb-system"},"name":"my-minio","override":{"set":["rootUser=admin","rootPassword=Passw0rd!"]},"version":"5.0.8"}}
  creationTimestamp: "2023-09-11T09:17:15Z"
  finalizers:
  - core.kubebb.k8s.com.cn/finalizer
  generation: 1
  labels:
    core.kubebb.k8s.com.cn/componentplan-release: my-minio
  name: my-minio
  namespace: default
  resourceVersion: "20347"
  uid: b76a6c17-e02f-4fe9-9cfa-e6ba485c8488
spec:
  approved: true
  component:
    name: kubebb.minio
    namespace: kubebb-system
  name: my-minio
  override:
    set:
    - rootUser=admin
    - rootPassword=Passw0rd!
  version: 5.0.8
status:
  conditions:
  - lastTransitionTime: "2023-09-11T09:17:14Z"
    reason: ""
    status: "True"
    type: Approved
  - lastTransitionTime: "2023-09-11T09:23:30Z"
    message: 'failed post-install: timed out waiting for the condition'
    reason: InstallFailed
    status: "False"
    type: Actioned
  - lastTransitionTime: "2023-09-11T09:23:30Z"
    reason: ""
    status: "False"
    type: Succeeded
  images:
  - hyperledgerk8s/minio-minio:RELEASE.2023-02-10T18-48-39Z
  installedRevision: 1
  latest: true
  observedGeneration: 1
  resources:
  - NewCreated: true
    apiVersion: v1
    kind: ServiceAccount
    name: minio-sa
  - NewCreated: true
    apiVersion: v1
    kind: Secret
    name: my-minio
  - NewCreated: true
    apiVersion: v1
    kind: ConfigMap
    name: my-minio
  - NewCreated: true
    apiVersion: v1
    kind: Service
    name: my-minio-console
  - NewCreated: true
    apiVersion: v1
    kind: Service
    name: my-minio
  - NewCreated: true
    apiVersion: v1
    kind: Service
    name: my-minio-svc
  - NewCreated: true
    apiVersion: apps/v1
    kind: StatefulSet
    name: my-minio
  - NewCreated: true
    apiVersion: networking.k8s.io/v1
    kind: Ingress
    name: my-minio-console

not found anymore


Pass autoQueryFallback option onto QueryInterface

Was there a reason you needed this on the QueryInterface object? autoQueryFallback is actually handled mostly by calling code to the QueryInterface object (mostly because each function will have different requirements for how it "auto" falls back). Did you need to access it for some other piece of code?

We defined autoQueryFallback to be false via the options on the Sequelize object. But it was still true in places like https://github.com/BlinkUX/sequelize-mock/blob/master/src/model.js#L329 resulting in failing tests. We found that this fixed it. But maybe there is a better way.

Hmm, that's weird. Line 56 of that Model file should import it from the calling Sequelize object and inherit it there.
Can you file a bug on this? I'm going to dig into it a bit and see if I can't find out why that's happening. Hopefully what I find is just this is the fix and I can just merge this in. But I want to check the implications of having that value inherited in the QueryInterface object.

Hmm, it must've been something on my end because I just tried to recreate the bug and it all worked as expected.


There is no event for loading from the internet (buffering)

I would also like this feature verrry much ;)

Closing in favor of #361


0.3.7 Calling github API excessively for each cache update.

Reporting a bug 🐛
Expected behavior
Reduced API calls
Actual behavior
On an update with 0.3.7 the refresh_supported_cache_lists function removes the supported.list file from the cache which causes a full refresh of information through the lowest branch of the list_debs function calling validate_deb on every app that is listed as available through installed repos like 01-main with no regard to arch support or installed status. Because of this any incompatible apps were causing #752 (which can be mitigated by #756) and when not hit with #752 will cause 103 calls to the Github API which is well over the 60 available without providing a DEBGET_TOKEN.
Steps to reproduce
let cache files age over 60 minutes or deb-get clean && deb-get update without a DEBGET_TOKEN set and without issue in #752.
System information
/etc/os-release
PRETTY_NAME="Ubuntu 22.04.1 LTS"
NAME="Ubuntu"
VERSION_ID="22.04"
VERSION="22.04.1 LTS (Jammy Jellyfish)"
VERSION_CODENAME=jammy
ID=ubuntu
ID_LIKE=debian
HOME_URL="https://www.ubuntu.com/"
SUPPORT_URL="https://help.ubuntu.com/"
BUG_REPORT_URL="https://bugs.launchpad.net/ubuntu/"
PRIVACY_POLICY_URL="https://www.ubuntu.com/legal/terms-and-policies/privacy-policy"
UBUNTU_CODENAME=jammy

/etc/lsb-release
DISTRIB_ID=Ubuntu
DISTRIB_RELEASE=22.04
DISTRIB_CODENAME=jammy
DISTRIB_DESCRIPTION="Ubuntu 22.04.1 LTS"

deb-get version
0.3.7

deb-get update
  [+] Updating /etc/deb-get/01-main
  [+] Updating /var/cache/deb-get/bat.json
  [+] Updating cache of supported apps in the background
  [+] Updating /var/cache/deb-get/deb-get.json
  [+] Updating /var/cache/deb-get/du-dust.json
  [+] Updating /var/cache/deb-get/duf.json
  [+] Updating /var/cache/deb-get/fd.json
  [+] Updating /var/cache/deb-get/gh.json
  [+] Updating /var/cache/deb-get/github-desktop.json

API x-ratelimit-used
Before update: x-ratelimit-used: 1 (used by the check)
After update: x-ratelimit-used: 105 (2 used by checks)

I have the same issue. I bashed my head against it off and on for weeks.
Part of the problem was that I didn't see my error messages anywhere in the Issues. So for posterity, this bug manifest for me as repeated errors in the form...
$ deb-get reinstall deb-get
  [*] WARNING! Cached file /var/cache/deb-get/deb-get.json is empty or missing.
/var/cache/deb-get/: Is a directory
  [!] ERROR! Failed to download . Deleting /var/cache/deb-get/...

or
$ deb-get install sniffnet
  [+] Updating /var/cache/deb-get/sniffnet.json
  [*] WARNING! Updating /var/cache/deb-get/sniffnet.json failed.
  [*] WARNING! Cached file /var/cache/deb-get/sniffnet.json is empty or missing.
/var/cache/deb-get/: Is a directory
  [!] ERROR! Failed to download . Deleting /var/cache/deb-get/...

As a side-note, it's obvious that the devs use the "optional" DEBGET_TOKEN config 100% of the time. Because they can't tell that the token config has gone from optional to necessary.
A DEBGET_TOKEN is not optional with deb-get anymore in its current form.

$ deb-get install sniffnet
  [+] Updating /var/cache/deb-get/sniffnet.json
  [*] WARNING! Updating /var/cache/deb-get/sniffnet.json failed.
  [*] WARNING! Cached file /var/cache/deb-get/sniffnet.json is empty or missing.
/var/cache/deb-get/: Is a directory
  [!] ERROR! Failed to download . Deleting /var/cache/deb-get/...

A DEBGET_TOKEN is not optional with deb-get anymore in its current form.

Hello wonder if you can answer this! (as same error here):
Trying to find out what permissions github PAT token geb-get uses... it's documented somewhere other than main README.md landing page?
Also github lets create both "classic" or "fine graned" token. Tried adding both kinds, but with 0 "extra" manual opt-in access permissions. So IDK if that is correct here.

... i suppose to debug this issue requires the following command?
sudo -E bash -x deb-get install sniffnet


Use the output hierarchy to check for overlapping outputs in execution plan

This addresses some more of the performance regression here: https://github.com/gradle/gradle/issues/16048
We now use the output hierarchy information to check whether a task can be scheduled with the currently running nodes. That makes things a little bit faster again (another 2-3%). So together with https://github.com/gradle/gradle/pull/16178 we should be back to the performance of 6.8.2.

@bot-gradle test QFL

OK, I've already triggered QuickFeedbackLinux build for you.

Here is an non-abi change benchmark on the Uber project: https://builds.gradle.org/repository/download/Gradle_Master_Util_Performance_AdHocPerformanceScenarioLinux/41514532:id/results/performance/build/test-results-uberMobileAppPerformanceAdHocTest.zip!/non_abi_change_with_configuration_caching/benchmark.html

This change makes things a little bit slower and removes the extra code to check for overlay between outputs.

I'm not sure I understand: are we making things faster this way, or are we making the code more readable only?

We are making things a little bit faster (around 2%) and we are replacing an n * m algorithm (checking every output path with every other output path) by an n log m algorithm (looking up the affected nodes in a tree for every output path).
If we would only have the measured improvement of 2%, then I'd say the change is not worth it, though since we are making the code better to understand by re-using data we already have, I'd go for this.


[AutoRelease] t2-servicebus-2022-06-13-30299(Do not merge)

https://github.com/Azure/sdk-release-request/issues/2825
Live test success
https://dev.azure.com/azure-sdk/internal/_build?definitionId=1031
BuildTargetingString
azure-mgmt-servicebus
Skip.CreateApiReview
true

issue link:https://github.com/Azure/sdk-release-request/issues/2825


issue with the RTL hebrew content

hi we are facing issues with navigation in the reader with RTL hebrew content in the vertical view and the reflowable view the reader navigation was not happening properly i have attached the video for the same please verify and provide some solution
https://user-images.githubusercontent.com/72591004/102964196-0b395780-4511-11eb-84dd-92a09698dfb8.mp4

Hello, I watched your video but I am struggling to see what is wrong. Are the "pages" not turning when you click the left-hand-side arrow button / keyboard key?

hi @danielweck the pages are not changing when we click on the next and previous this is happening with hebrew content


dev doc: call out 404 must be captured

[na] The PR has modified HISTORY.rst describing any customer-facing, functional changes. Note that this does not include changes only to help content. (see Modifying change log).

[na] I adhere to the Command Guidelines.

View a preview at https://prompt.ws/r/Azure/azure-cli/6390
This is an experimental preview for @microsoft users.

Codecov Report

Merging #6390 into dev will not change coverage.
The diff coverage is n/a.

@@         Coverage Diff         @@
##           dev   #6390   +/-   ##
===================================
  Coverage    0%      0%
===================================
  Files       11      11
  Lines      133     133
  Branches     9       9
===================================
  Misses     133     133

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 6d8dbb0...1c75396. Read the comment docs.


Switch Redis library to to StackExchange.Redis

[x] Switch to StackExchange.Redis :clap:.
[x] Use NetDataContractSerializer as default serialzier, but allow it to be customizable.
[x] Support .NET 4.0 and 4.5 releases.
[x] Bring over parts of ServiceStack.Redis to support distributed locking?
[x] Update README with changes & contributors.

Have been running in production for a while now. Seems to work as it should,
But we also needed to make the syncTimeout change.

Great! Is there anything in the API that you'd like to see before I make the 2.0 release?

Well , only one thing that comes up in mind is region configuration.
So you can add a region/prefix for the cache, So you can have multiple sites use nhibernate cache without conflicts. So alla keys get prefixed. Cause we found problem with that when running staging and production against same redis server instance

This can be accomplished with NHibernate by default. Set the cache.region_prefix property in your configuration:
<hibernate-configuration xmlns="urn:nhibernate-configuration-2.2">
  <session-factory>
    <property name="show_sql">true</property>
    <property name="cache.region_prefix">Staging</property>
  </session-factory>
</hibernate-configuration>

You can also do it if you're using FluentNHibernate (you can do this also by setting the property on the Configuration object in normal NHibernate too). For example:
fluentConfig.Cache(c =>
{
    c.UseSecondLevelCache().UseQueryCache()
    .ProviderClass<RedisCacheProvider>()
    .RegionPrefix("Staging");
})

This will make keys like:
NHibernate-Cache:Staging.App.Users.Users:v1:App.Models.Users#2

You can also set the region per-class in NHibernate. We use the region prefix globall/per-class when we make a schema change and we want to bust through the cache. Check out this blog post I wrote for an automated solution to this.

One thing I noticed (with the OLD provider, and haven't got round to checking this one) is that if entities are cached, and then the app is updated and the domain model changes, the old entities are returned from the cache. i.e. the fact the model entities have changed doesn't seem to be tracked. I'm not sure if we're doing something wrong, it's expected behaviour so manually flush the cache after a release, an Nibernate caching bug, a service stack redis bug or an internal NHibernate bug.
Just thought I'd mention it though, just in case it's an NHibernate caching bug and it's still in this new version.

@bunceg If the entities change, everything should just work (as long as you have your configuration all in place and every ISession is hooked up to the correct cache). The previous version with ServiceStack.Redis worked as expected (we had been running it in production for ~1.5 years). So, I'm going to guess it might be an issue with your configuration/setup. Maybe you can open a separate issue and I can help you further debug the problem (also make sure you update to 2.0 of this library since it'll have much more stability/performance improvements when using StackExchange.Redis).

Ok thanks I thought it must be us, but I'm not sure where the configuration error is :) I'll update to StackExchange shortly - I'd like to pull ServiceStack out of our project completely anyway..

One thing that I noticed is that the depencies on StackExchange.Redis , Is conflicting with for example: http://blogs.msdn.com/b/webdev/archive/2014/05/12/announcing-asp-net-session-state-provider-for-redis-preview-release.aspx
That uses the StackExchange.Redis.StrongName , Maybe the dependency should be on that also instead of the StackExchange.Redis ?

Wrote it in my closed pull request, but guess this comment should be good to have in the Readme : https://github.com/TheCloudlessSky/NHibernate.Caches.Redis/pull/11#issuecomment-70099530

Great :+1:

We did run into some problems , when running Microsoft Sessionstate and this.
Getting timeout's when running both, guess it's something with multiple ConnectionMultiplexer .. Where he talks about here: http://stackoverflow.com/a/25946747/42466. And as the Microsoft SessionState Provider is web.config only.. I can't really inject in the current ConnectionMultiplexer.
Not a problem of this library maybe , but a good FYI :gem:
Any suggestions for a good solution ?

If you just use one of them, do you experience timeouts?
In the stack trace of the timeout exception, which library is throwing the exception (NHibernate.Caches.Redis or Microsoft SessionState)?
Are you running on Azure (I don't have much experience with it, but I know other folks have had timeouts with Azure)?
If you run one of the projects, do you experience timeouts?
What's the syncTimeout for each library's config?

No
Will try to reproduce, we got it in production and I didn't save the
logs with stack
Nope
No
2000 for nhib,state: default and retry for 5000

Will try to make some tests on staging tomorrow and see if I can reproduce.
On Wed, 21 Jan 2015 21:43 Adrian Phinney<EMAIL_ADDRESS>wrote:

If you just use one of them, do you experience timeouts?
In the stack trace of the timeout exception, which library is
throwing the exception (NHibernate.Caches.Redis or Microsoft SessionState)?
Are you running on Azure (I don't have much experience with it, but
I know other folks have had timeouts with Azure)?
If you run one of the projects, do you experience timeouts?
What's the syncTimeout for each library's config?

—
Reply to this email directly or view it on GitHub
https://github.com/TheCloudlessSky/NHibernate.Caches.Redis/pull/8#issuecomment-70917546
.

Ok, I'm going to try running some tests just with multiple ConnectionMultiplexers and see if anything comes up.

I created a really small sample MVC 5 application with both projects and running Redis locally (on Windows) and I don't run into any issues. I'm guessing it's because the Redis instance is local so the latency is incredibly low. You can contact me through my email and I can help you guys debug the problem if necessary.

Hello,
Sorry for late reply , I Have been on vacation without computer access :)
Think we found our problem. We hade a few REALLY large keys, so sometimes they caused timeouts, havnt seen timeouts after we redid the keys so they are smaller ( less data , more keys )

No worries - hope you had a great vacation!
That's really surprising about the large key size performance. I made a gist to test this locally and I didn't run into any performance issues (even with large key sizes). Were the large key sizes in this library or your own usage?

We run into issues on reads , and our redis is hosted on another machine. So guess  with large keys and network latency (even if it's 10gb between the servers), you will get into problems. But will see if I have some times for test later today.

Ok, I'm pushing the final 2.0 release now. If you run into any other problems, please create a separate issue :smile:.

Sorry for not getting back, but have been working good since last time!
Great! :)
On Thu, 16 Apr 2015 21:16 Adrian Phinney<EMAIL_ADDRESS>wrote:

Ok, I'm pushing the final 2.0 release now. If you run into any other
problems, please create a separate issue [image: :smile:].
—
Reply to this email directly or view it on GitHub
https://github.com/TheCloudlessSky/NHibernate.Caches.Redis/pull/8#issuecomment-93818880
.

@Tazer No problem. I'm glad it's working for you. Thanks for all the help you and @MattiasJakobsson did porting this and testing!


Cargo should not run git gc manually

Saw this on twitter:
https://twitter.com/indygreg/status/1256656752944111616
and it turns out
a) cargo is checking how many packs there are on its own
b) if there are too many, it runs git gc, which is blocking.
There is a git command to do both of that automatically, with b) being done in the background, not blocking anything else: git gc --auto. The maybe_gc_repo function should just call that unconditionally. This also has the advantage of respecting the various gc.* git config items, instead of a manual __CARGO_PACKFILE_LIMIT.

Duplicate of https://github.com/rust-lang/cargo/issues/4495 ?

Closing in favor of #4495 so we consolidate the conversation on this.


Implement diagnostic SA1111: ClosingParenthesisMustBeOnLineOfLastParameter

Fixes #45.

What about Delegate declarations, anonymous delegates, lambdas? Should these also be handled by this diagnostic?

StyleCop description doesn't include them. The question is if WE want to include them here.

Well, since we are already considering invocations of each of these items, it would make sense to include their declarations as well. Make sure attributes get included as well.

Ok. We have a family of SAs which are related to one another (like #44, #47, #48). They all mention method/indexer call/declaration, the same way this one does. So I guess the same rule should apply to those issues as well - we check additonal constructs, like delegate declarations, anonymous delegates, lambdas etc.

I think so. These rules should apply to everything that Roslyn calls ParameterList or ArgumentList to be consistent.

Ok. So we actually make it a little bit different than in original StyleCop, I think that rule should apply to other issues - if we think that sth should be reported but is not in original StyleCop, we should add this.

@macpak For now you can leave a comment in #359 if you observe a difference. Eventually we'll have a document in the repository where notes can be added.

Allright. Thanks for info.

I added code to analyze missing parts. Let me know if you think that something else should be also analyzed by this analyzer.

Sweet! :sparkles:
This one is next up on my plate for review. Plus it should lay the groundwork for many of the other readability rules. :+1:

Yes :) :+1:

I found that tests for attribute use an attribute that is not supposed to be applied on a class. I applied it on a method to make code "more correct" - actually it doesn't change anything but it's more correct :)

I went ahead and merged this. So you're aware in the future, I did make a few small changes to the implementation.

To keep the diagnostic messages clear and short, you generally want to either duplicate the Title value or write a short custom message like you see in SA1310. The description is longer than users will want to read.
I placed the using directives inside the namespace declaration to meet the requirements of SA1200 (not yet enforced, but hopefully will be enabled by default starting with the Alpha 2 release).
I changed the layout of a few conditional expressions in SA1111ClosingParenthesisMustBeOnLineOfLastParameter.cs, which you can see by using the blame tool on GitHub.
I enabled the diagnostic by default, because once again you took the time to implement a nice set of tests. :+1:

Ok. Thanks.
Just one case - found that conditional expressions had bug (grrrrr) because they should've had ORs instead of combination of OR's and ANDs. For instance, these lines:
 if (lambdaExpressionSyntax.ParameterList == null ||
            !lambdaExpressionSyntax.ParameterList.IsMissing &&
            !lambdaExpressionSyntax.ParameterList.Parameters.Any())
        {
            return;
       }

should be
if (lambdaExpressionSyntax.ParameterList == null ||
            !lambdaExpressionSyntax.ParameterList.IsMissing ||
            !lambdaExpressionSyntax.ParameterList.Parameters.Any())
        {
            return;
       }

. I reported it on #423


Isle of Armor additions, added relevant missing moves/abilities, minor fixes, G-Max Field, Neutralizing Gas, Steely Spirit, and a Weakness Policy button

Added file
image_res/Weakness_Policy.png (just an in-game pic of Weakness Policy)
Updated files
index.html:
-added a hide menu for moves that double in power and for 1-2 hits
-added buttons for Neutralizing Gas, ally Steely Spirit, G-Max Field, and Weakness Policy
-code syntax has also been slightly changed due to it being reformatted in VS 2017
ability_data.js:
-added Competitive, Inner Focus, Oblivious, Own Tempo, Mirror Armor, Ripen, and Steely Spirit
ap_calc.css:
-minor font adjusting for the double-in-power, 1-2 hits menu
ap_calc.js:
-added a simple conditional for the double-in-power menu to show/hide appropriately
-added an extra condition for moves that always hit three times
-added a bool for moves that can double in power
-added vars for ally Steely Spirit, Neutralizing Gas, and G-Max Field
-added a condition for Dragon Darts to have the option to hit once or twice (1-2 hits menu)
damage.js:
-added a condition for Neutralizing Gas that nullifies abilities
-added Rising Voltage and Triple Axel to Max move base power exceptions
-fixed Max move calculations so that they take base power exceptions into account
-added a condition for G-Max Drum Solo, Fireball, and Hydrosnipe so base power is 160
-fixed Max moves such that they can crit now, and the number of hits for a multi-hit attack's Max move only hits once
-fixed Gorilla Tactics so it doesn't give a boost on a Max move
-slight change to choice item calculations: nullifying Choice Band and Choice Specs only applies when the calculated move is a Max move, and nullifying Choice Scarf only applies when the Pokemon is Dynamaxed
-added type change for Terrain Pulse
-added a conditional for Final Gambit's damage output
-added cases for Expanding Force, Misty Explosion, Terrain Pulse (base power), Rising Voltage, Grav Apple, and Triple Axel
-added conditional to double the power of moves that are meant to
-added conditional for ally Steely Spirit
-added var for Shell Side Arm (functions based on the research done at Smogon)
-added to Steelworker conditional to include user's Steely Spirit
-added conditional for Ripen
-added conditionals for Inner Focus, Oblivious, Own Tempo, Scrappy, Mirror Armor, and Competitive for Intimidate
-added conditionals for Weakness Policy boost from Weakness Policy button
dark_ap_calc.css:
-changes mirror ap_calc.js
item_data.js:
-added the other pinch berries for user convenience
ko_chance.js:
-added conditionals to hasFigy for the other pinch berries
-added var for Ripen
-added conditional for G-Max Field damage
move_data.js:
-added the following moves: Blast Burn, Dive, Frenzy Plant, Hydro Cannon, Rock Wrecker, Fire Pledge, Grass Pledge, Water Pledge, Heat Crash, Final Gambit, Disarming Voice, Mystical Fire, Parabolic Charge, Petal Blizzard, Burning Jealousy, Dual Wingbeat, Expanding Force, Flip Turn, Grassy Glide, Lash Out, Meteor Beam, Misty Explosion, Poltergeist, Rising Voltage, Scorching Sands, Shell Side Arm, Skitter Smack, Steel Roller, Surging Strikes, Terrain Pulse, Triple Axel, Wicked Blow
-the following moves have bool canDouble: Assurance, Stomping Tantrum, Bolt Beak, Fisheous Rend, Lash Out
-fixed Breaking Swipe so it is now a spread move
pokedex.js:
-added the following Pokemon: Kubfu, Urshifu-Single Strike, Urshifu-Rapid Strike, Zarude, Rillaboom-Gmax, Cinderace-Gmax, Inteleon-Gmax, Slowpoke-Galar, Slowbro-Galar
-changed the following Pokemon's abilities: Milotic, Wigglytuff, Rillaboom, Cinderace, Inteleon, Corviknight, Flapple, Appletun, Toxtricity, Perrserker, Pincurchin, Dracozolt, Sneasel, Snorunt, Glalie, Lucario, Mienfoo, Mienshao, Umbreon
status_item_data.js:
-mirrors changes in item_data.js (didn't realize this file goes unused until after coding)
NOTES:
-G-Max Field is a stand-in for Wildfire/Volcalith/Vine Lash/Cannonade to ease coding and UI
-the Steely Spirit field condition and the Steely Spirit ability are separate from each other since Steely Spirit can stack in-game
-Neutralizing Gas does not affect any checks at the beginning of the calculations; this is intentional

Sorry for being lazy and ignoring this, just rebased this to another branch & gonna check it out locally today. If it all looks good I'll go ahead and merge to master


Add lemmatization to stemming filter

Adds lemmatization option to WordStemmingFilter (via WordNetLemmatizer from nltk stem module).

Coverage decreased (-5.2%) to 72.839% when pulling ed826d02e35ccb7904778e460c66f6cc21ad68dc on rbroc:add_lemmatization_to_stemming_filter into 771e3faca10cd134ccf95029620cd9a0ba146dc6 on tyarkoni:master.

Looks read to merge to me

LGTM once the conflict's resolved.

LGTM once the conflict's resolved.
will wait until bert_transformer is merged, then rebase this on master to resolve conflicts.

@tyarkoni conflicts resolved.


Include digdag-storage-s3 in CLI

digdag-storage-s3 was not included in shadow jar because it was
increasing CLI's binary size. However, because digdag-cli already
depends on aws-sdk through digdag-standards, there're no negative
impact any more. Including digdag-storage-s3 increases binary size only
20KB.

@komamitsu can I ask you to take a look?

@frsyuki We don't need to add the dependency to digdag-cli/build.gradle?

@komamitsu It's unnecessary because of this: https://github.com/treasure-data/digdag/pull/450/files#diff-c197962302397baf3a4cc36463dce5eaR338

👍


missing 1.17.1 package on https://packages.microsoft.com/repos/vscode/

hi,
i use https://packages.microsoft.com/repos/vscode/ to install vscode in my container, so i can specify an exact version to be installed in the container.
the 1.17.0 version can be installed, but the 1.17.1 version is not available. at the momemt of writing this issue the contents of https://packages.microsoft.com/repos/vscode/pool/main/c/code/ show that 1.17.1 is missing.

Published .deb and .rpm now.

Update repodata for .rpm please
https://packages.microsoft.com/yumrepos/vscode/


MINOR: UpdateMetadataRequest should be lazily created

We should defer to construct UpdateMetadataRequest Builder to the right time when we really need them.
More detailed description of your change,
if necessary. The PR title and PR message become
the squashed commit message, so use a separate
comment to ping reviewers.
Summary of testing strategy (including rationale)
for the feature or bug fix. Unit and/or integration
tests are expected for any behaviour change and
system tests should be considered for larger changes.
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

@rajinisivaram Please review this minor patch. Thanks.

@huxihx Thanks for the PR, LGTM. Was this just found during code review or was there another reason for the change? Also, is there a reason the reference to the JIRA was removed from the PR title even though the JIRA is still open?

@rajinisivaram Thanks for the comments. It was only during a code review to find this one. And correct me if I am wrong, I remembered Jason told me that the such trivial PR be prefixed with 'MINOR'. Does it make any sense?

@huxihx If you have already created the JIRA, you can go ahead and tag the PR. I was only pointing out that small fixes do not require a JIRA in the first place.

@hachikuji I understand what you mean, finally. Thanks for the information.

I don't think a lazy val makes sense here. A def might make sense perhaps.

@ijuma Correct me if I am wrong.  In the current design, updateMetadataRequest will be evaluated only once before sending it to all the brokers. That's why I turned it to a lazy val since it will not be changed. However, updateMetadataRequest will be evaluated multiple times if we change it to a def since there might be multiple brokers to receive the request. This is not efficient. Do we really need to recalculate all the live brokers before sending UpdateMetadataRequest to every broker?

@ijuma @rajinisivaram Does this patch make any sense?


Cambios para fintech

Favri, están comiteados los cambios que necesitamos, que son de dominios

Codecov Report

Merging #8 into master will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff           @@
##           master       #8   +/-   ##
=======================================
  Coverage   91.41%   91.41%
=======================================
  Files          12       12
  Lines         431      431
=======================================
  Hits          394      394
  Misses         37       37

Impacted Files
Coverage Δ

src/service/finntech.service.js
96.55% <100%> (ø)
:arrow_up:

src/index.js
86.79% <100%> (ø)
:arrow_up:

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 530d739...390ef9c. Read the comment docs.


Feature Request: Date formatting

Kip is such a great app already, thank you very much!
Today I created a page with sun and moon rise/set times. The data comes from signalk-derived-data and is an ISO string (eg. 2023-03-07T09:49:00.000Z).
It would be nice to have a DateTime formatter that can transform this string to just hour and minute and that in local time. Maybe something like this would work in units.service.ts:
"DateTime time": function(v) {
      const d = new Date(v);
      return d.getHours() + ':' + d.getMinutes();
    },

or
"DateTime time": function(v) {
      const d = new Date(v);
      return d.toTimeString();
    },

this can be done in NodeRed

I've seen there is a PR for this (https://github.com/mxtommy/Kip/pull/148). It would do what I need, if in a bit different way. What's the status on this PR?

@JonyIvy this will be part of V2 release. Soon...!


[QUESTION] Is there an ability to run a function when the cache is used or key is set?

Is it possible to run a function whenever a cache is used or cache key is set? Ideally something similar to the custom_key_coder , for example:
def key_set_function(
    func,
    namespace: Optional[str] = "",
    request: Request = None,
    response: Response = None,
    *args,
    **kwargs,
):
    // Do something
    return something

@app.get("/")
@cache(expire=60, coder=JsonCoder , key_set_handler=key_set_function)
async def index(request: Request, response: Response):
    return dict(hello="world")

I'm aware of aiocache having hooks which are implemented via 'plugins' that you can run before and after a new key is set. Thanks!

I couldn't find a direct way to do it, so I've created a PR that allows custom event handlers. Example usage:
@FastAPICache.on_event("existing_key")
def exists_in_cache(func, *args, **kwargs):
    print("Existing key")
    return None

@FastAPICache.on_event("new_key")
def new_in_cache(func, *args, **kwargs):
    print("New key set")
    return None

Via a class method:
def exists_in_cache(func, *args, **kwargs):
    print("Existing key")
    return None

def new_in_cache(func, *args, **kwargs):
    print("New key set")
    return None

FastAPICache.set_on_existing_key(exists_in_cache)
FastAPICache.set_on_new_key(new_in_cache)

@long2ice - let me know if you have any feedback

@long2ice - have you had a chance to review?

I'd love to see it in action!
Actually I am now checking my API performance and would like to know whether cache is used or not.

I could perhaps take a look at this. What are the use-cases here? Is it purely for producing performance metrics?


Does DateRanegPicker receive a classname ?

Hi, Thanks for the great lib.!
I was trying to overwrite some the themes using the provided instructions in the documentation, yet it seem to be missing the part where the component takes the theme itself, so, Here's my code:
theme.js
import ThemedStyleSheet from 'react-with-styles/lib/ThemedStyleSheet';
import aphroditeInterface from 'react-with-styles-interface-aphrodite';
import DefaultTheme from 'react-dates/lib/theme/DefaultTheme';

import { css, withStyles } from 'react-with-styles';

const myTheme = {
  reactDates: {
    ...DefaultTheme.reactDates,
    color: {
      ...DefaultTheme.reactDates.color,
      highlighted: {
        backgroundColor: '#369',
        backgroundColor_active: '#369',
        backgroundColor_hover: '#369',
        color: '#369',
        color_active: '#369',
        color_hover: '#369',
      },
    },
  },
};

ThemedStyleSheet.registerInterface(aphroditeInterface);
ThemedStyleSheet.registerTheme(myTheme);

export { css, withStyles, ThemedStyleSheet };

index.js
import 'react-dates/initialize';
import React from 'react';
import { DateRangePicker} from 'react-dates';
import { css, withStyles } from './theme.js';

class ReactDatesComponent extends React.Component {

  componentWillMount() {
    this.setState({
      startDate: null,
      endDate: null,
      focusedInput: null,
    })
  }

  render() {
    return (
      <DateRangePicker
        {...css(this.props.styles.reactDates)}
        startDate={this.state.startDate} // momentPropTypes.momentObj or null,
        endDate={this.state.endDate} // momentPropTypes.momentObj or null,
        onDatesChange={({ startDate, endDate }) => this.setState({ startDate, endDate })} // PropTypes.func.isRequired,
        focusedInput={this.state.focusedInput} // PropTypes.oneOf([START_DATE, END_DATE]) or null,
        onFocusChange={focusedInput => this.setState({ focusedInput })} // PropTypes.func.isRequired,
      />
    );
  }
}

export default withStyles(({ reactDates }) => ({ reactDates }))(ReactDatesComponent)

I improvised here based on react-with-styles docs, yet I get this error:
warning.js:33 Warning: Failed prop type: withStyles(DateRangePicker): unknown props found: className

so, What do you think is wrong here ?

Sorry my bad, Apparently it was working yet the warning was just that i provided reactDates as is, something like:
export default withStyles(({ reactDates }) => ({
  DateRangePicker: reactDates,
}))(ReactDatesComponent)

should fix it, and then maybe we add more clear styles ( just for testing ) in our theme object like :
      selected: {
        backgroundColor: '#369',
        backgroundColor_active: '#369',
        backgroundColor_hover: '#369',
        borderColor: '#369',
        borderColor_active: '#369',
        borderColor_hover: '#369',
        color: '#369',
        color_active: '#369',
        color_hover: '#369',
      },

again , I am new to this so if anyone'd like to introduce a better practice to this please go on :)
closing this now 👍

HI @yasserhennawi, I know you closed this but just for future clarification, once you render the theme, you don't have to wrap ReactDatesComponent in withStyles (ie you don't need withStyles(({ reactDates }) => ({ reactDates }))(ReactDatesComponent) at all). It should just work! You would only wrap your component if you wanted to apply your own different styles which you can read more about in the react-with-styles repo - https://github.com/airbnb/react-with-styles


Rendering dotprompts in a new trace span

Rendering dotprompts in a new trace span for better observability.
Checklist (if applicable):

[x] Tested (manually, unit tested, etc.)

Added an additional metadata field for the hash of the dotprompt


Add support for optional path prefix for testcase file property

Currently, the file property of each testcase element contains the path to the file from RSpec's perspective, but that's not always the full path to the file from the root of the repository. For example, in Homebrew/brew, consider the testcase element emitted for a test in Library/Homebrew/test/dev-cmd/bump_spec.rb:
<testcase
  classname="test.dev-cmd.bump_spec"
  name="brew bump behaves like parseable arguments can parse arguments"
  file="./test/dev-cmd/bump_spec.rb"
  time="0.015492">
</testcase>

The file property points to ./test/dev-cmd/bump_spec.rb, but the file actually resides at Library/Homebrew/test/dev-cmd/bump_spec.rb. Currently, rspec_junit_formatter has no way to know this full path to the file, so this pull request proposes a way to make that information available to rspec_junit_formatter. 😅
For tooling that consumes the XML emitted by rspec_junit_formatter, it's useful to be able to use the file property as the full path to the file from the root of the repository, so that the tooling can link to file on github.com (or wherever the repository is hosted).
This pull request proposes adding an optional setting for specifying a path prefix. For example, Homebrew/brew would set this as follows:
RSpec.configure do |config|
  config.junit_formatter_file_path_prefix = "Library/Homebrew"
end

And instead of file property being ./test/dev-cmd/bump_spec.rb, it would be Library/Homebrew/test/dev-cmd/bump_spec.rb.
What do you think?

Note: This is my first time proposing functionality for an RSpec formatter. I see that fuubar uses custom RSpec settings for its configuration, so this pull request uses that same approach. If there's a different approach that would work better for rspec_junit_formatter, please let me know, and I'll update the implementation accordingly.

Hi @jasonrudolph! Thanks for the contribution.
This is an interesting use case, and I appreciate that this is a useful thing to do in some very specific scenarios. However, I try to keep this formatter as simple as possible. I'd prefer to keep to rspec conventions and emit exactly what rspec says is the path. I'd also prefer to avoid adding any configuration to this formatter, if possible.
If you need to rewrite this path for downstream consumption in your specific case then I'd suggest post-processing the report after rspec has finished running. For example, a ruby script like this:
require "rexml" # from stdlib
doc = REXML::Document.new(File.read("tmp/rspec.xml"))
doc.root.get_elements("//testcase[@file]").each do |testcase|
  testcase.add_attribute("file", File.join("Library/Homebrew", testcase.attribute("file").to_s))
end
File.write("tmp/rspec.xml", doc.to_s)

👋 @sj26: Thanks for the super thoughtful reply. :zap:

I try to keep this formatter as simple as possible.

That makes sense to me. It feels like a key strategy for sustainable open source. ✨
Thanks so much for taking the time to suggest a way to attain the same benefits using the existing rspec_junit_formatter functionality. And thanks for the concrete code example along with sample input and output. You've really gone above and beyond here. 🥇 :bow:

Thanks for understanding, @jasonrudolph! I hope that post processor or something similar works for your use case.


Working with large and multiple language files

I'm submitting a ...  (check one with "x")
[ ] bug report => check the FAQ and search github for a similar issue or PR before submitting
[X] support request => check the FAQ and search github for a similar issue before submitting
[ ] feature request

I would like to ask if there is an efficient way to work with multiple languages (JSON) files for ngx-translate.
As our projects get larger, we have difficulties keeping all the languages up-to-date and on the same level. For example, it would be helpful to have all translations side-by-side so you wouldn't need to search the same key in all files when editing.
How do you handle this issue? Can you recommend a tool for that, or would it be helpful to write one?

I use a little script build around https://www.npmjs.com/package/merge-json
It is very simple, but I can post it tomorrow if you need some inspiration. The pitfall is, that the languages are outsourced to a separate repository, which makes maintaining easy, but requires some additional steps in the workflow to get changes into the actual app ...

I wrote the new tool attranslate as an alternative to babeledit: https://github.com/fkirc/attranslate
In contrast to babeledit, attranslate operates directly on translation-files in a surgical way.


Update the role for changes in cisagov/ansible-role-cyhy-core

🗣 Description
This pull request makes updates to the role to provide necessary interoperability with changes in the cisagov/ansible-role-cyhy-core dependency.

💭 Motivation and context
This is a sibling pull request to https://github.com/cisagov/ansible-role-cyhy-core/pull/79 to ensure we can pass necessary arguments to the downstream cisagov/ansible-role-geoip2 dependency.

🧪 Testing
Automated tests pass.

✅ Pre-approval checklist

[x] This PR has an informative and human-readable title.
[x] Changes are limited to a single goal - eschew scope creep!
[x] All relevant type-of-change labels have been added.
[x] I have read the CONTRIBUTING document.
[x] These code changes follow cisagov code standards.
[x] All relevant repo and/or project documentation has been updated
to reflect the changes in this PR.
[x] Tests have been added and/or modified to cover the changes in this PR.
[x] All new and existing tests pass.

Hi Nick @mcdonnnj, one question: should the requirements.yml file at ansible-role-ncats-webd/meta/requirements.yml contain the role: " name: ncats_webd and src: https://github.com/cisagov/ansible-role-ncats-webd " as stated in the README file? I noticed the other role listed there " name: cyhy_core and src: https://github.com/cisagov/ansible-role-cyhy-core."

Hi Nick @mcdonnnj, one question: should the requirements.yml file at ansible-role-ncats-webd/meta/requirements.yml contain the role: " name: ncats_webd and src: https://github.com/cisagov/ansible-role-ncats-webd " as stated in the README file? I noticed the other role listed there " name: cyhy_core and src: https://github.com/cisagov/ansible-role-cyhy-core."

The meta/requirements.yml lists dependencies for this role that are not included in the dependencies block of meta/main.yml. Due to how cisagov/ansible-role-cyhy-core is used we must explicitly install it as part of the role's tasks and so it must be declared as a dependency in the meta/requirements.yml file.

LGTM!
Have you applied the updated user Terraform yet? If so, please mention that somewhere in this PR. If not, please mention when you are planning to do that.

@dav3r Sorry i didn't think to list it because the tests will not pass without the Terraform being applied.


Vertical/horizontal centering of the texts for different devices.

See https://github.com/PiechZ/greybox_wrapped/pull/73


Inject polymer component inside px-data-table column

Hi,
Currently px-data-table cells are driven by types such as string , dropdown and html but we have requirement where we need to render custom polymer component inside cell.

You can achieve this with the html column type, as long as you've imported the component at the top of the file. Here's an example: http://codepen.io/randyaskin/pen/apexgB?editors=1000

Thanks a lot  @randyaskin . This will satisfy our current requirement .


[MXNET-472] ccache for docker builds

Description
Introduce instance-local ccache to unix-based builds
Checklist
Essentials

[x] The PR title starts with MXNET-472
[x] Changes are complete

Changes

[x] CCache latest version and build for all different docker builds
[x] CPU and GPU builds are supported

Comments
In future, the ccache dir is meant to be shared via EFS on instances with the same label on CI builds.
Original PRs by @larroy and @lebeg: #11096 and #11036

Thank you marco.


Add some more APIs

we should have a consistent missing value return type. Fee free to change the {badindex, n} return types to match the new API you added to which return nil when value doesn’t exist. I only went with the badindex tuple to match some of erlangs std lib; I had no idea if it made sense or not.

undefined would be the canonical Erlang one, maybe nil is some Elixir thing. Definitely should do a smoothing pass once more of this is fleshed out.


Refresh tokens and auth codes as self-encoded tokens

At the moment, IdentityServer requires a persistence store for the operational data of authorization codes and refresh tokens.  However, why can't these be self-encoded / self-contained tokens, similar to signed JWT access tokens?  That way, the server could use the key material used for access tokens to sign them & verify their signature when presented, and the data associated with them (such as expiry date for a refresh token) could be embedded in the token itself, meaning no persistence store is needed.  If this could be done it would be great for removing the overhead of a persistence store.

And how would you revoke the refresh token?

This would be for applications where you didn't need to.


IToast interface is not available on the Net standard/PCL  project

Please check all of the platforms you are having the issue on (if platform is not listed, it is not supported)

[] iOS
[] Android
[] UWP
[X] NET Standard 1.1

Version of OS(s) listed above with issue
NET Standard 1.6
Version of ACR UserDialogs Library
6.5.1
Expected Behaviour
Be able to convert (IDisposable) object of UserDialogs.Instance.Toast(config) to IToast in order to call Dismiss function to hide the toast in code.
Or suggest any alternative way that a shown toast can be hidden by code prior to duration timeout or action button trigger.
Actual Behavior
If applicable, please include entire exception - type, message, stacktrace
Code of Conduct

No Library Update Requests
No Timeline Requests
Please read through the documentation and FAQs
Please be respectful of my time. I put a lot of effort into my libraries.  I try to support the issue requests within a day.  I'm not psychic, so please take the time to fill out the above form as complete as possible

There is no IToast interface.  Dispose of the disposable - that is the dismiss


Balena push build fails.

While trying to setup hm-config on a RPi Model 3 B+ which is already added to my balena test fleet balena push fails. I'm attaching the logs from balena and also a detailed log with --debug enabled of balena push.

test-fleet_release-log_build overview-f94a3e1.txt
test-fleet_release-log_gateway-config-f94a3e1.txt
balena-h-config-push-debug-log.txt

@marvinmarnold - tried to build it at his end and it worked without issues. So may be something only at my end, not sure.

Update: Moving my device to a fleet owned by Marvin resulted in the imaged getting pushed successfully.  So build and push worked fined for Marvin but for some reason failed from my machine. For the time being this workaround works and I'll document it in new dev onboarding doc so future devs have a heads up about this too.


How to update cluster certificate？

We use ace-engine to create a cluster, but now the certificate is about to expire. How should I update it?

Does this apply to AKS Engine also ?


Update getting_started_with_your_microsoft_windows_device.md

Context
Changes proposed in this pull request
Guidance to review

Added via https://github.com/DFE-Digital/get-help-with-tech/pull/349, closing.


Able to display any emote in any chat by using Kappa_/

Describe the bug
You can type random emotes that display poorly and aren't enabled in that chat by typing Kappa_/27272 where 27272 can be any number. Different numbers produce different emotes. This cannot be inputted in twitch chat (though it displays there if entered through Chatterino).
To reproduce
Open up Chatterino and just type Kappa_/. In the attached screenshot, I used Kappa_/463235
Screenshots

Chatterino version
Chatterino 2.1.7 (commit 845c8fa63) (works on other versions I believe)
Operating system
Windows 10 (though works on any chatterino client irregardless of OS)
Additional information
Might want to fix this one fast, as I've seen people using it to input hitler emotes.

not chatterino issue

How so? I can't seem to trigger it through any other way than Chatterino.

you might not be able to see it from webchat if you send it from webchat

Is this confirmed? I will forward a bug report to Twitch if this is the case.

This appears to now be patched by Twitch.


os-wol widget does not appear in 24.7

Important notices
Before you add a new report, we ask you kindly to acknowledge the following:

[x ] I have read the contributing guide lines at https://github.com/opnsense/plugins/blob/master/CONTRIBUTING.md
[x ] I have searched the existing issues, open and closed, and I'm convinced that mine is new.
[x ] The title contains the plugin to which this issue belongs

Describe the bug
After upgrade to the 24.7 WakeOnLan widget is not available anymore.
To Reproduce
Steps to reproduce the behavior:

Go to Dashboard
Click on "Add Widget"
Open available list
See that Wake On Lan is no longer available

Expected behavior
Widget is available for selection and operates in similar manner as in pre 24.7 versions.
I'm aware that MVC changed with the 24.7 release and widget requires a rewrite, but as I'm unfamiliar with the code I'm unable to contribute a fix. Hopefully someone else can help.

+1 Also have this issue.

Same issue here

@fichtner Should I look into this?

Not yet. I don’t agree with the tickets conclusion of the expected outcome. WOL is practically an unmaintained community effort. With a dashboard replaced there are no immediate requirements of a widget replacement. The plugin is functional as is. Better than removed anyway.

For the person with the thumb down: make sure to thank @m3e-g when you have the time


get documents in array of strings cloud in flutter using cloud firestore

it seems it is harder than i think, i am building a flutter app and using cloud firestore as a database that is my simple database looks like

what i want is to retrieve all documents of the collection "DriverList" in an array of string and retrieve its data in an array of object of driver class all the examples i found was building a new screen with widget that has a stream and a stream builder and return the content into a ListView

Please consider asking support questions in one of the channels http://flutter.io/support .

it is not an error i want help in building my app
the output of flutter doctor
`/home/ahmed/flutter/bin/flutter --no-color doctor
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel beta, v0.5.1, on Linux, locale en_US.UTF-8)
[✓] Android toolchain - develop for Android devices (Android SDK 27.0.3)
[✓] Android Studio (version 3.1)
[✓] Connected devices (1 available)
• No issues found!
Process finished with exit code 0`
i have already asked my question on stackoverflow but i got no answer
https://stackoverflow.com/questions/52107009/get-documents-in-array-of-strings-cloud-in-flutter-using-cloud-firestore

If you want support it's still helpful to know what Flutter version, IDE, OS, you have.
It also tells if a feature should be available in your Flutter version or not.
Please consider asking support questions in one of the channels http://flutter.io/support .

i attached the link of my question on stackoverflow in the previous comment but i had no answer

Why did you delete the SO question?

@ahmedelsayedkhalafallah
This issue has been moved to https://github.com/FirebaseExtended/flutterfire/issues/677. Any further collaboration will be done there.


Implement Errorhandling for Exitstatus

Changes
Features

implement exit code handling for systemctl_capture() -> from testing I found 0,1,3,4 that are used. More will hopefully come up on usage
add start(), enable(), disable() on crate level and for Unit -> closes #8 and makes #9 obsolete
add stop(), exists() for Unit, it was still missing

Other

put spawn part of systemctl() and systemctl_capture() into its own function spawn_child() to reduce duplicate code
reduce if let hell and use ? operator more
add more tests and improve error tests (PartialEq on structs was needed for this)
updated readme to reflect additional filter option on list_units()

I know the ErrorKind options are not always 100% on point, but with io only, I tried to be as close as possible to whats really going on. If we want better messages at some point, we can always go for https://crates.io/crates/thiserror - but that will mean a revamp of many parts.


Specify how an eventId should be calculated

An eventId should be a unique identifier between the parties in the DLC. Ideally it should be deterministic and should be reliant on the contract as well, to best accomplish this it would likely need to be a hash of the contract info, oracle info, the timeouts/delays, as well as some public keys from the participating parties whether it be each other's funding keys or maybe a lightning public key.

Ah sorry just opened #32 without noticing this one.

After further thinking, I ended up proposing in #59 that this should be done the way it is done in lightning (by using the funding output with a temporary id, which I computed as the hash of the offer)


clean up contributors page

This will fix the alignment and some of the spacing &  width issues for the time being.

@ststimac There is a linting problem.  Can you fix it? Also, can you attach a screenshot with how it looks now? Thanks!

as a side note, npm test didn't catch that error with the unnecessary 0 and these are the errors showing up that @molant told me to disregard. :(

Can we add the missing people?

@alrra the listing errors come from sonar. We should fix them there and/or make sure the markdown lint rules are the same.

@molant one of the errors was in my code but when I ran npm test it didn't catch it :/

We should fix them there and/or make sure the markdown lint rules are the same.

I'll take care of that later today.

@ststimac Notes: In the future

Start your commit message with an uppercase letter.
Add at the end of the commit message Fix #<issue_number_the_pr_fixes>

We should fix them there and/or make sure the markdown lint rules are the same.

@molant Fixed in https://github.com/sonarwhal/sonarwhal.com/commit/7429504822b7e88347a13492ed68ae9a58572a88.


MRG+1: fix tfr docs

A few obvious fixes to the docs - mostly stuff left over after introduction of EpochsTFR I think.
@kingjr

LGTM apart from this

Also killed irrelevant CIs here

LGTM, feel free to merge once the Travis 2.7-nodata (style checking) build comes back happy

Current coverage is 51.99% (diff: 100%)

Merging #3803 into master will decrease coverage by 34.37%

@@             master      #3803   diff @@
==========================================
  Files           343        343
  Lines         61215      61215
  Methods           0          0
  Messages          0          0
  Branches       9364       9364
==========================================
- Hits          52873      31829   -21044
- Misses         5628      27529   +21901
+ Partials       2714       1857     -857

Powered by Codecov. Last update 630632c...c64e8de

Thanks @jona-sassenhagen


Config Option: 'type_labels' expects JSON, not YAML map

Despite the example type_labels mapping showing a serialized JSON 1-liner, I didn't process the idea of embedding JSON config inside YAML. This led to a somewhat obscure error when my workflow tried to execute:
Invalid workflow file
The workflow is not valid. .github/workflows/auto-label.yml (Line: 32, Col: 13): A mapping was not expected

I have a pending fix in https://github.com/GoogleCloudPlatform/emblem/pull/391.
More explicit documentation that JSON string is expected (such as a complete example showing use of that configuration), or support for a YAML maps would have made my experience as a first-time user easier.

Using a single line string can quickly make the configuration ugly, but a literal block scalar can be used to make the config a bit nicer, documenting an example here in case it helps others:
so from this:
- uses: bcoe/conventional-release-labels@v1
   with:
     type_labels: '{"feat": "feature", "fix": "fix", "breaking": "breaking"}'

to this:
- uses: bcoe/conventional-release-labels@v1
   with:
     type_labels: |
         {
         "feat": "feature",
         "fix": "fix",
         "breaking": "breaking"
         }


Fix/casting continue pretraining

Theres' this issue of attempting unscale FP16 gradients

After investigation, this is because of global dtype, which is when we use it on colab, we will use torch.float16 instead of torch.bfloat16. This error does not happened if we use torch.bfloat16 (my own RTX4090 for example). So we need to use torch.float32 on device that is still using torch.float16
Here is the result on bfloat16

Abd here's the result on float32 (colab)

Previously I drafted this because somehow I can't use BS 2 just like the example. BUt now I can use BS 2. So opening this PR instead of Draft

Thankyou so much for bringing this up in PR man🙏🙏

Oh I totally missed float16 cannot be used and only bfloat16 can be used for continued pretraining - nice catch!


Postman is not tracked?

i remember that Postman is tracked in version 3.3.0 but you get Others results instead. now when I update into 3.5.3 postman is not tracked while figma and xcode are fine.

Fixed in v3.6.1.


live? method should be optional. It can cause crazy bugs!

It appears Papertrail defines a method "live?" on every instance of every model(https://github.com/airblade/paper_trail/blob/master/lib/paper_trail/has_paper_trail.rb#L184). This seems a bit over-the-line to me.
It caused a really unexpected bug on our end, where we had a flag called "live" on our model (which I presume is not a terribly uncommon flag name). We then expected to be able to use the "free" rails method of "live?", which automatically returns a boolean for that attribute (and is general ruby convention for booleans). Instead, it always returned true, which was very very confusing until we realized it was getting defined by Papertrail. And granted, it's already a boolean, and we don't have to use the "?" version, but that's standard ruby convention, and the current behavior seems unacceptable.
We ended up just doing undef :live? right after has_paper_trail in our model, and now it's ok, but that's crazy.
Now I realize you can't just go breaking your API, and removing or renaming live?, but perhaps you could add an option to rename this method to like... paper_trail_live? or live_version? or most anything else that's more specific, and less likely to crash. I'd also be cool with an option to just not define it at all. We never use it.

It appears Papertrail defines a method "live?" on every instance of every mode

It is, and is clearly documented in the API Summary on the README

It caused a really unexpected bug on our end, where we had a flag called "live" on our model

I'm sorry to hear that. Naming is hard, you are the first person to inquire about this particular method but many of the method names have been updated in the past due to complaints such as this. I don't think it's that common of a name since it hasn't seemed to cause name conflicts prior, but we could consider prepending paper_trail for every method name, although that also seems like overkill.  I suppose live_version? would be appropriate, we will consider renaming it for version 5.

Thanks for the reply. Yeah I see that it's clearly documented. Just one of those things you don't really think about as a collision. I agree prepending paper_trail to everything is a bit long. Perhaps prepending pt could be a usable namespace?
Thanks for adding it to the V5 roadmap.

There are other potential name conflicts, too. If I'm counting correctly, installing the paper_trail gem adds 36 instance methods and 10 class methods to all of your active record models.  Of those 46, 13 have a prefix, either "pt_" or "paper_trail_". I don't know what the best way to deal with this is.  Ideally, we'd add far fewer methods to people's models. If we're only adding a few methods to models, then I wouldn't mind prefixing all of them.

If I'm counting correctly, installing the paper_trail gem adds 36 instance methods and 10 class methods to all of your active record models.

I have an idea that could reduce this number from 46 to perhaps 2 or 3, and might not be all that complicated: https://github.com/airblade/paper_trail/pull/719


[Draft]{RDBMS}az mysql flexible-server replica create: Add Parameter --location

Related command

Description

Testing Guide

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change
[Component Name 2] az command b: Add some customer-facing feature

This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] The PR title and description has followed the guideline in Submitting Pull Requests.

[ ] I adhere to the Command Guidelines.

[ ] I adhere to the Error Handling Guidelines.

RDBMS

@evelyn-ys Said in your doc, in PR title, customer-facing should use [] in title while non-customer-facing using {}. I do not the difference between this two. Could you guide me?

@evelyn-ys Hi, may I gentle ping for review


make ServiceAccount kubeconfig context name unique

The ServiceAccount kubeconfig is generated with a context that has the same name as the associated cluster, which is typically the cluster's context name too. This makes it difficult to distinguish the "clusters" in the cluster list. This PR appends the username (ServiceAccount name) to the cluster name to make it more unique.

Not really a bug, but should be safe to put in a patch release.

Still think that it shouldn't go into a patch release. But yes minor.


Can switch mode call audio - video ?

Does this library support the feature switch mode call audio - video? Thank you so much.

yes. It does.

you can switch modes call audio - video in streaming?


Add flag to extended builder env vars

This should enable event sender or kn to be vendorless.
There are a few other options, but the straight-forward env flag seemed the most practical to me.
Other options:

Inferring from existing or not-existing ./vendor dir
Having --vendorless flag on the generator
...

/cc @creydr @pierDipi @mgencur

@cardil per our chat on Slack.

maybe we could update the funcutils template too, to support those params

maybe we could update the funcutils template too, to support those params

Yep, good point. Thanks!


Change ScnSocialAuth\Authentication\Adapter\HybridAuth to own class

Hi,
I need to modified this method:
   protected function instantiateLocalUser()
    {
        $userModelClass = $this->getZfcUserOptions()->getUserEntityClass();

        return new $userModelClass; //I need to remove new
    }

because when I put this code to zfcuser.global.php:
'user_entity_class' => function($sm) {
        /* @var $repositoryClass Common\Repository\Users */
        $repositoryClass = $sm->get('Repository\\Users');
        return new Users\Entity\User($repositoryClass, []);
    },

I had a php error.
So I create new class \Auth\Adapter\HybridAuth and extend from your class (ScnSocialAuth\Authentication\Adapter\HybridAuth).
What I have to do to tell application to use my class?
Thanks!

Ok I think I founded solution add this to service_manager factories
'ScnSocialAuth\Authentication\Adapter\HybridAuth' => 'My\Auth\Service\HybridAuthAdapterFactory',


Add Segmenting-Anything component

What changes were proposed in this pull request?
 No changes are made to existing files.
Add new Claimed components Segmenting-Anything that uses the Segmenting-Anything library from https://github.com/facebookresearch/segment-anything.
How was this pull request tested?
 Components are tested with Elyra pipeline run in-place locally.
To Do: test components with Kubeflow instances.
To test the components, download checkpoint from https://dl.fbaipublicfiles.com/segment_anything/sam_vit_h_4b8939.pth and add environment variables: checkpoint_path and input_image_path.
See code comments if the component need additional environment variables.
Developer's Certificate of Origin 1.1
   By making a contribution to this project, I certify that:

   (a) The contribution was created in whole or in part by me and I
       have the right to submit it under the Apache License 2.0; or

   (b) The contribution is based upon previous work that, to the best
       of my knowledge, is covered under an appropriate open source
       license and I have the right under that license to submit that
       work with modifications, whether created in whole or in part
       by me, under the same open source license (unless I am
       permitted to submit under a different license), as indicated
       in the file; or

   (c) The contribution was provided directly to me by some other
       person who certified (a), (b) or (c) and I have not modified
       it.

   (d) I understand and agree that this project and the contribution
       are public and that a record of the contribution (including all
       personal information I submit with it, including my sign-off) is
       maintained indefinitely and may be redistributed consistent with
       this project or the open source license(s) involved.

Thanks a lot @miragecoa
[x] merged


Update k8s dependencies to v1.28.2

What this PR does / why we need it:
Update k8s dependencies to v1.28.2
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Release note:

Update k8s dependencies from `v1.26.4` to `v1.28.2`

Bumps golang from 1.20.5 to 1.21.2.

Should we also update the golang version here?
https://github.com/gardener/network-problem-detector/blob/cabae88d2a374c45c3df6ee94aef853733b04d22/.ci/pipeline_definitions#L39C23-L39C29

Should we also update the golang version here? https://github.com/gardener/network-problem-detector/blob/cabae88d2a374c45c3df6ee94aef853733b04d22/.ci/pipeline_definitions#L39C23-L39C29

@axel7born
Thanks, I missed that.

/lgtm


License?

Hello there,
I might be interested in using the code for my projects. However, there isn't any information on copyright and licensing. Could you provide some clarifications or consider adding a license for this repo?
Thanks.

Added a license.


MDN user profile and settings updated workflow

User story
As a user I want MDN to remember my browser sets (so that I can know whether the code is useful for the project I'm working on) and a place to manage my subscription (so that I can cancel or modify it).
Acceptance criteria

[ ] We have solutions (ie. InVision mock-ups) for the above use-cases outlined ready for engineering.
[ ] PM and engineering sign off

Tasks
If relevant, you can make a checklist for tasks.

[ ] This is an incomplete task.

Updated the project document with team's meeting notes, decisions, and proposed solution.
Project doc is here: https://docs.google.com/document/d/1e5W8sJnL2nITUxPMQIVVM6LXs040e3ovwWoEeXNoAEA/edit?usp=sharing
And proposed solution here: https://share.goabstract.com/c17745e7-11af-4f5d-bc57-3b90846475e2?collectionLayerId=523f8c6a-85ec-491d-b221-6bfbe037b42b&mode=design
Browser sets UX/UI flow coming soon.

Looks good to me, but same question: Any changes I should be aware of after our December 19 session?

@atopal only small items, nothing major. We're launching the UserZoom test today. I'll walk you through it during our meeting tomorrow (Jan 9). Hopefully we'll have some results by then.


CheckpointError: Instagram call checkpoint for this action!

Hi!
I'm using your basic example:
var Client = require('instagram-private-api').V1;
var device = new Client.Device('someuser');
var storage = new Client.CookieFileStorage(__dirname + './cookies/someuser.json');

// And go for login
Client.Session.create(device, storage, 'someuser', 'somepassword')
	.then(function(session) {
   		// Now you have a session, we can follow / unfollow, anything...
		// And we want to follow Instagram official profile
		return [session, Client.Account.searchForUser(session, 'instagram')]
	})
	.spread(function(session, account) {
		return Client.Relationship.create(session, account.id);
	})
	.then(function(relationship) {
		console.log(relationship.params)
		// {followedBy: ... , following: ... }
		// Yey, you just followed @instagram
	})

but getting following error:
Unhandled rejection CheckpointError: Instagram call checkpoint for this action!
I'm still getting the error, even after verifying on instagram.com that I've made the request.
Any help?

According to german this article following Instagram API endpoints are not available anymore:
User Search

GET /users/{user-id}
GET /users/{user-id}/media/recent
GET /users/self/media/liked
GET /users/search

that's for their official api at http://instagram.com/developer. you need to complete the checkpoint via the device you are using to run the script. Or login to instagram on a browser/app on the same device.

Any improvements??

The instagram server detect the script as "An Unusual Login Attempt" everytime the script run

Anyone tried disabling temporaly the account as explained in this second method?
https://www.gramto.com/article/how-to-bypass-instagram-checkpoint/


Add multiple public French services

Mon Compte Formation -> easy
Ameli -> Easy
Mon Espace Santé -> Easy
Mes Services Étudiant -> Hard
impots.gouv.fr -> Hard
data.gouv.fr -> easy
France Identité -> Medium
La Poste -> Hard
L'identité Numérique La Poste -> Medium
YRIS -> Medium
Pix -> Hard
Pass Culture -> Easy
CNIL -> Hard
Agence nationale des titres sécurisés -> easy
Service-Public.fr -> easy

Thanks for all these contributions and the recent PRs!

you're welcome, I want this project to help most people and when I see services I need that are missing, I directly add them so I help everyone

That's the spirit :)