Messages component refactoring and extension. Added messages.service.ts and tests.

Description:

Refactoring of how messages are handled on frontend.
Messages can be send and displayed by using messages.service methods.
Messages component now handles all messages recieved from backend as well as from other parts of frontend app, when send through messages.service.
Messages now have a well defined and easy to use API for type, timeout and dismissibility.

Unit Tests:

frontend/src/app/messages/messages.component.spec.ts
frontend/src/app/messages/messages.service.spec.ts

What tests do I need to run to validate this change:
Perform any action that sends backend messages and then refresh/open the frontend app.
Also, you can import MessagesService anywhere in Angular app and call sendMessage(), success(), danger(), warning(), info() with apropriate parameters - the message should appear on top of the page (not in a 'fixed' screen position though - for now we need to scroll to the top of the page, but this can be simply changed in the future)

Codecov Report

Merging #886 into master will increase coverage by 0.21%.
The diff coverage is 100%.

@@            Coverage Diff            @@
##           master    #886      +/-   ##
=========================================
+ Coverage   91.58%   91.8%   +0.21%
=========================================
  Files         107     109       +2
  Lines        3007    3074      +67
  Branches       10      14       +4
=========================================
+ Hits         2754    2822      +68
+ Misses        252     250       -2
- Partials        1       2       +1

Impacted Files
Coverage Δ

frontend/src/app/messages/messages.component.ts
100% <100%> (ø)

frontend/src/app/messages/messages.service.ts
100% <100%> (ø)

frontend/src/app/window.service.ts
55.55% <0%> (+11.11%)
: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 08cd22e...a2351f9. Read the comment docs.


Documentation - tooltips and popovers

Documentation on tooltips and popovers is required for our wiki

I would like to work on this, should I make a new section under UI Elements called "notifications" ?

@motatoes Sure!

See https://github.com/epicmaxco/vuestic-admin/wiki/Tooltips-and-Popovers


Close button doesn't work in Firefox

Using Firefox on the demo page, the [x] button doesn't make the notification go away.
In the Console, I'm seeing the following error message:
Uncaught TypeError: event.path is undefined
handleClick https://adhami.me/pop-up-notification/src/adhami-toast.js:75
connectedCallback https://adhami.me/pop-up-notification/src/adhami-toast.js:81
It's pointing to this line:
    if(event.path[0].matches(".close")) this.close();

Chrome works though


Apply content reuse for oft-used snippets like version numbers

It'd be nice to be able to refer to minimum versions of various components/frameworks throughout the docs, without causing a maintenance nightmare. In other words, we need some sort of content reuse, so that we can define a snippet of information in one place, and reuse it wherever we need it.
Hugo has the concept of "shortcodes", which are similar to Django variables.
https://gohugo.io/templates/shortcode-templates/
https://gohugo.io/content-management/shortcodes/
To get this pattern going, let's use shortcodes for:

the minimum required version of Kubernetes
the minimum required version of ksonnet

If anyone has any other suggestions for snippets of information we should reuse, please comment on this issue.

/assign

area/docs

/area docs


[5.0.0-rc.0] Errors when implementing ToSchema manually

Hi, I tried to migrate from v4 to v5.
I've encountered a few problems after implementing the new trait ToSchema and PartialSchema.
Implement theses trait is very easy but I had a compile error on other struct that use de the derive way to impl ToSchema with inner struct that manually impl it.
Have I missed something?
How to reproduce:
use utoipa::{openapi, PartialSchema, ToSchema};

#[derive(Debug)]
struct Newtype(String);

impl ToSchema for Newtype {
    fn name() -> std::borrow::Cow<'static, str> {
        std::borrow::Cow::Borrowed("Newtype")
    }
}

impl PartialSchema for Newtype {
    fn schema() -> openapi::RefOr<openapi::schema::Schema> {
        openapi::ObjectBuilder::new()
            .schema_type(openapi::schema::Type::String)
            .into()
    }
}

#[derive(Debug, ToSchema)]
struct Dto {
    customer: Customer, // ERROR
}

Error:

the trait bound Newtype: utoipa::__dev::SchemaReferences is not satisfied
the trait utoipa::__dev::SchemaReferences is implemented for __path_handler

But I can't find the trait SchemaReferences in the doc.

Yeah, the SchemaReferences is internal type to utoipa thus it is not found from the Docs. This will need some pondering nonetheless as for now I have no solution for this though. This is due to the fact that the derive ToSchema will automatically implement the utoipa::__dev::SchemaReferences trait for the type, but this is of course not valid for cases where it suppose to be manually implemented. Maybe I need just make the type public for all.
But if you really want to go about implementing this type for now, you can do it actually and it is quite simple.
impl utoipa::__dev::SchemaReferences for NewType {
    fn schemas(_schemas: &mut Vec<(String, RefOr<Schema>)>) {
         // this type does not have any reference schemas
         // if it had any fields that implement `ToSchema` they should be
         // pushed to the `schemas` vec.
    }
}

Thanks for your quick reply
In my project, we implement ToSchema manually only for newtype, so I can easily use your workaround.
Indeed we manually implement ToSchema for newtype because the derive macro is not general enough for newtype (Unnamed Field Struct).
Unnamed Field Struct have only access to: example, examples, default, title, format, value_type, as, deprecated, description.
Do you think it's possible to add others? pattern, minimum, maximum, min_length, max_length, ...
That might solve the problem (in my case at least but I don't think I'm the only one using newtypes 😅), but maybe it's not that simple 😕.

@ValentinColin

Do you think it's possible to add others? pattern, minimum, maximum, min_length, max_length, ...

Yes, I think that should be plausible, There was this other issue related to this as well https://github.com/juhaku/utoipa/issues/572.

I'll remove the need for custom impl of SchemaReferences in next PR.


Add Button to download xml

As a User, I'd like to have a button to download the XML of my BPMN diagram, in order to parse it on my local machine.

:thought_balloon: Hints & Thoughts
:question: Additional Questions to Answer / Consider

:white_check_mark: Conditions of satisfaction

[ ] Should have a download button beneath the modeler
[ ] downloaded XML should be the same as the model represents

Because parsing is already happening directly to .robot, this issue is no longer necessary


Fix issue #64

Fixes issue #64
Description
Converts variables into string type to prevent AttributeError
Motivation and Context

Fixes issue #64
How Has This Been Tested?

Test merging an int
Screenshots (if appropriate):
Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have added tests to cover my changes.
[x] All new and existing tests passed.

Thank you for the PR!


Example project executing successfully in IDE fails on command line

Expected behavior
Specs executing successfully in IDE should execute successfully from command line.
Actual behavior
https://github.com/getgauge-examples/gauge-guice is an example project that executes successfully in IntelliJ but fails on command line.
Steps to replicate

clone https://github.com/getgauge-examples/gauge-guice
execute from IntelliJ - it passes.
execute from command line - it fails.

Version
Gauge version: 0.9.1

Plugins
-------
html-report (4.0.2)
java (0.6.4)

This works:
git checkout d9551da2ded7f646b2b7ecfd77ab3975b39686d1
gauge specs

So I guess this commit: https://github.com/getgauge-examples/gauge-guice/commit/d9551da2ded7f646b2b7ecfd77ab3975b39686d1 has introduced breaking changes.
FWIW - HEAD doesn't work for me in Intellij.

@sguptatw It works perfectly fine for me from both IDE and console with
Gauge version: 0.9.3
Commit Hash: 2cfd7a8

Gauge-Java version: 0.6.6
Commit Hash: e68bb70

IntelliJ-Plugin version 0.3.7.nightly-2017-09-06

can you please verify it and close the issue?


Swipe To Refresh On NTP

On The New Tab Page (NTP) The Swipe To Refresh Shows Up At Random When Opening A New Tab After Browsing.
Attempting To Reproduce Issue Is At Random Tends To Happen The Most While Browsing.
Images:

Taking a look at this issue.
Very easy to reproduce, for possible contributors:

open a new tab
visit any site
press back (this will go back to the home tab)
user is able to pull to refresh

@cmonfortep Can I work on this issue?


METAMODEL-63: UnionDataSet

Here's my attempt at designing/solving METAMODEL-63.
Ideally we would support UNION also as part of our Query language. But that seems like a huge change and a much more low-hanging fruit would be to allow unions via a special DataSet implementation.
I also introduced a new interface, InvokableQuery. I've often had the need to pass around both DataContext and Query (because those two are needed in combination to produce the DataSet). The new interface allows passing these around in a more coherent and encapsulated way.

I'd love any feedback before I go ahead and merge this based on lazy concensus?

A bit sad to see it implemented without a proper UNION, but still seems like a nice step on the way.
Code LGTM

I agree, full UNION support would be very cool. I would encourage anyone with an interest in that to use this new class as a building block to implement UNION.


Update zio-http to 3.0.0-RC6

About this PR
📦 Updates dev.zio:zio-http from 3.0.0-RC4 to 3.0.0-RC6
📜 GitHub Release Notes - Version Diff
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "dev.zio", artifactId = "zio-http" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "dev.zio", artifactId = "zio-http" }
}]

labels: library-update, early-semver-pre-release, semver-spec-pre-release, commit-count:1

Superseded by #313.


Fix incorrect test for nil function return

The existing code wasn't a valid test for nil.  It would prematurely exit if the function being evaluated returned false.  This meant that a test like "provision().disabled=false" would never succeed even if provision().disabled evaluated to false.
Hit this when upgrading from MCO 2.2.4 to 2.8.4.

Can you add a unit test so that this doesn't regress?

Superceded by #355 which adds tests and further cleanup of this code.

Just returned from family visit -- sorry for the non-response and thanks for the fix!


Crash in Gitpod whenever Leon asks my name and I reply with my name

Specs

Leon version: 1.3.1 and same behavior on the latest develop branch commit 2f56a14
OS (or browser) version:  Ubuntu 20.04.5 in GitPod
Node.js version:  16.19.0
Complete "leon check" (or "npm run check") output:
(if using Docker) Complete "npm run docker:check" output:
(optional) Leon skill version:

Expected Behavior
I expected Leon to reply with some acknowledgement that he knows my name is James.
Actual Behavior
Instead, the UI hung, and the server crashed.
.: TCP CLIENT :.
ℹ  Received data: {"topic": "spacy-entities-received", "data": {"spacyEntities": [{"start": 0, "end": 5, "len": 5, "sourceText": "James", "utteranceText": "James", "entity": "person", "resolution": {"value": "James"}}]}}
/workspace/leon/node_modules/@nlpjs/slot/src/slot-manager.js:295
      latestSlot: context.slotFill.latestSlot,
                                   ^

TypeError: Cannot read properties of undefined (reading 'latestSlot')
    at SlotManager.process (/workspace/leon/node_modules/@nlpjs/slot/src/slot-manager.js:295:36)
    at Nlp.process (/workspace/leon/node_modules/@nlpjs/nlp/src/nlp.js:737:28)
    at processTicksAndRejections (node:internal/process/task_queues:96:5)
    at async /workspace/leon/server/dist/core/nlu.js:390:28
Error: Command failed with exit code 1: npm start

How Do We Reproduce?
I am not sure if you can reproduce, but here is what I did:

Fork the repo.
Load gitpod but using jamesmortensen instead of leon-ai so it clones my fork.
Browse to port 1337 in Gitpod browser.
Start chatting with Leon.
When Leon asks my name, I reply "James" or "I am James" or any variation of this, and the UI hangs.  We see the application crashes.
Restart and repeat and the same thing occurs.

In addition, I cd'd to the gitpod home folder and followed the instructions to install the production v1.3.1 and repeated the steps. Same behavior.
Not sure if this matters, but I'm not a fan of speech.  So I'm only chatting with Leon. There's no mic or speaker involved.
Not sure if it matters, but I'm also a first time user, so I'm not entirely sure that I've done everything correctly.  I assume that Gitpod would work out of the box which is why I tried it first.  I can see a lot of potential here and am looking forward to exploring more.  Thank you for your dedication to this project!
Extra (like a sample repo to reproduce the issue, etc.)

Hello! @jamesmortensen
Thanks for your report, it is indeed a bug!
Leon-AI is still in beta and there are still many things to do, so features can break between releases, but we're more and more approaching to a stable v1.0.0 release. :muscle:
I tried to reproduce the bug with my locally installed Leon instance, and it is working, but when using GitPod, Leon sucesfully answer to the "Hello!" query, but when I write "My name is Divlo.", it crashes with the same error message as you (TypeError: Cannot read properties of undefined (reading 'latestSlot')).
I'm not sure what can be the root cause, we will need to investigate.

This issue was not related to Gitpod, after some research, it was an issue with @nlpjs releasing a new minor broken version.
https://github.com/axa-group/nlp.js/issues/1252
The issue has been fixed, since then.


Counting Sort

Algorithm name
Counting Sort Using C++
Programming language

[ ] C
[x] C++
[ ] Java
[ ] Python

Additional context (OPTIONAL)
Add any other context or screenshots about the feature request here.

You can work on it
you can add your file in

sorting folder

countingsort(create new folder)


feat(kit): add rating component

PR Checklist
Please check if your PR fulfills the following requirements:

[x] The commit message follows Conventional Commits
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[x] Feature
[ ] Refactoring (no functional changes, no api changes)
[ ] Other... Please describe:

What is the current behavior?

Closes https://github.com/TinkoffCreditSystems/taiga-ui/issues/112
What is the new behavior?
https://user-images.githubusercontent.com/12021443/147211243-567b9f81-e297-4bfa-a394-0fc349f27f88.mov
Does this PR introduce a breaking change?

[ ] Yes
[x] No

@waterplea @nsbarsukov @nsbarsukov please, review me)

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-9.2%) to 72.199%

Totals

Change from base Build<PHONE_NUMBER>:
-9.2%

Covered Lines:
139

Relevant Lines:
181

💛  - Coveralls

@waterplea updated ))

@waterplea @vladimirpotekhin Hello, please, review me)

@nsbarsukov @vladimirpotekhin hello, please review me)


types(REST): event listener typings

Please describe the changes this PR makes and why it should be merged:
This PR creates typings for REST events.
Status and versioning classification:

Code changes have been tested against the Discord API, or there are no code changes
I know how to update typings and have done so, or typings don't need updating

This style of event types works great for autocompletion, but with some more complex types you can give users total typesafety from event emitters. I've found that the best way to do that is by writing an interface to extend EventEmitter.
If you do want to something like that in this repo, here is my implementation: src, docs, feel free to just copy-paste the file if you want to avoid another dependency. Usage looks something like
import { EventEmitter } from 'events';

type Events = {
    success: (message: string) => void;
    error: (error: Error) => void;
};

class MyTypedEmitter extends (EventEmitter as new () => TypedEventEmitter<Events>) {}

Of course, your solution works just fine so feel free to keep it as-is.


[Bug]: Lot of deviation in time taken to setup autocomplete

Is there an existing issue for this?

[X] I have searched the existing issues

Description
startAutoComplete  took 3600ms in one profile and 2400ms in other profile. That is 50% deviation, while the rest of the code has around 2.5% deviation.
It would help with performance monitoring and performance in general if this deviation can be reduced.
Profiles can be found in the slack thread.
https://theappsmith.slack.com/archives/C024GUDM0LT/p1655467501490139
Steps To Reproduce
Check the profiles on slack thread attached.
Public Sample App
No response
Version
Both / Latest

Application link :- https://release.app.appsmith.com/applications/62f4fa6eac988524cd26df4b/pages/62f4fa6eac988524cd26df4d/edit
Steps to test :-

Open the application ( stress select widget )
In dev mode, i added performance.now() before and after the startAutoComplete and logged the difference in console
Click on select widget
Check logs

Results

Test serial no
metrics

1

2

3

4

Numbers seemed to be very close and in expected range.
cc: @SatishGandham can you please check and comment if something was missed or needs to be looked again.

@Rishabh-Rathod , It was random and I have seen it 3-4 times in the automated performance testing.
If you are not able to reproduce it, please go through the code once and see if anything stands out.
40ms is a long time to setup autocomplete, any way to minimize this?

If you are not able to reproduce it, please go through the code once and see if anything stands out.

@SatishGandham just confirming, you meant the perf test code here, correct?

40ms is a long time to setup autocomplete, any way to minimize this?

I will check what could be possibly improved here.

@SatishGandham just confirming, you meant the perf test code here, correct?
@Rishabh-Rathod, No. Autocomplete code.

40ms is a long time to set up autocomplete, any way to minimize this?

startAutoComplete calls TernServer.updateDef which is an expensive call ( even for small def it takes ~40ms and it does deviate ) and as it is a method from the tern library we don’t have control over its logic directly and hence, it limits us to make performance improvement to it.
In this PR, we are moving customDataDef's update on editor focus instead of startAutoComplete with a check to only updateDef when there is a new customDataTree and the editor is onFocus, this way we reduce the number of TernServer.updateDef calls and hence performance should be less impacted.
After these changes, startAutoComplete would take around ~0.1 ms as the expensive TernServer.updateDef call is moved on focus.
cc: @SatishGandham


fix: use default emoji when style is 'none'

before fix:
after fix:

hey there, thanks for the contribution!
in general, the whole purpose was to have no emojis at all if you just select none. so this behaviour is correct.

Sorry, I thought 'none' was the default

I found that "none" is indeed the default value. If it is normal for emojis not to be displayed when "none" is selected, then setting the default value to "native" would be better, right?

@FlorianWoelki Please have a look!

thank you for your contribution!


Missing comments about subject removal

The following code lacks documentation-- why are there two cells doing the same thing? why removing subject 4?
https://github.com/spinal-cord-7t/coil-qc-code/blob/f9fcb557bc52a6afb02e99a15da0444d90376ee0/data_processing.ipynb#L91-L114

I initially added this cell because I was not sure what to do about the analysis of SpinozaV6, which does not have any MP2RAGE scans, and for which segmentation is thus not possible. I can correct this if the intention for the analysis of the phantom subject SpinozaV6 can be clarified for me. Should we perform segmentation using a different scan, for instance?

I would simply not include SpinozaV6 in the same dataset as the human dataset. I would create another dataset and also upload it on OpenNeuro.
BTW: the 2.0.1 dataset does not include SpinozaV6, hence the code does not require this cell to work.

Could you clarify what you mean by this? SpinozaV6 is sub4, and every site in v2.0.1 on OpenNeuro includes sub4.

I had no idea sub4 was spinoza, because it is written nowhere in the README (the README is empty), and the description.tsv file says what sub4 is "homo sapiens":

So, two options:

the README and description.tsv is fixed
the spinoza data is uploaded to another dataset

the latter is "cleaner" from a data analysis standpoint (because no need for an "if case") and data curation standpoint (more homogeneous, which is what we expected from a BIDS dataset).


fix: typos in comments

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


Enable custom templates at launch rather than in post-install config?

I think this is a feature request, but it may just be a documentation issue.
I see that --templates is deprecated. I'd like to be able to set custom templates at container launch, rather than have to authenticate in via web afterwards.
My use case is to provide portainer as an embeded service in a larger application, to launch a handful of specialized Docker apps on demand. A custom --templates piece is critical, as that is how I tell portainer how to launch my applications.
I could document to the user how to login and set a custom template URL, but my goal is to make things super easy for the end user, so I'd rather they not have to concern themselves with the internals of portainer (this is an audience that will likely not know what a "container" is).
Assuming this is not presently possible, what do you think of adding an ENV variable or command line argument that allows an initial configuration to be set on launch?
Lastly, I realize I may be pushing the limits of portainer's intended use case. If so, feel free to just say that. :) It just has the potential to save me a lot of time building a mini-scheduler, so I'm trying to make it work.
(I'd be interested in configuring a custom logo this way as well, but that is not nearly as important).

We had originally intended to deprecate the command line options in favour of the UI, however, in order to accomodate ci/cd tools that might use Portainer as an endpoint, i think i would prefer to keep the CLI options alive. If we do decide to continue to deprecate, then we will switch them to env variables that can be passed from CLI.
Cc @alapenna
Rgds,
Neil Cresswell

On 26/06/2017, at 7:05 AM, Robby Grossman<EMAIL_ADDRESS>wrote:
I think this is a feature request, but it may just be a documentation issue.
I see that --templates is deprecated. I'd like to be able to set custom templates at container launch, rather than have to authenticate in via web afterwards.
My use case is to provide portainer as an embeded service in a larger application, to launch a handful of specialized Docker apps on demand. A custom --templates piece is critical, as that is how I tell portainer how to launch my applications.
I could document to the user how to login and set a custom template URL, but my goal is to make things super easy for the end user, so I'd rather they not have to concern themselves with the internals of portainer (this is an audience that will likely not know what a "container" is).
Assuming this is not presently possible, what do you think of adding an ENV variable or command line argument that allows an initial configuration to be set on launch?
Lastly, I realize I may be pushing the limits of portainer's intended use case. If so, feel free to just say that. :) It just has the potential to save me a lot of time building a mini-scheduler, so I'm trying to make it work.
(I'd be interested in configuring a custom logo this way as well, but that is not nearly as important).
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

@ncresswell Thanks! That addresses my concerns here.

@ncresswell agree, I'll change the purpose of this issue to switch from a CLI flag to ENV var for templates.


Strelka does not complete failed missing vcf files

The approach for inferring the Strelka output file names doesn't seem to allow me to rescue failed runs that have an empty output directory. I get the following error when I try to generate the outputs for one of these:
gzip: results/gambl/strelka-1.1/02-strelka/genome--grch37/05-20006T--05-20006N--matched/results/variants/somatic.snvs.vcf.gz: No such file or directory
[Mon Aug 10 14:37:49 2020]
Error in rule _strelka_decompress_vcf:
    jobid: 1
    output: results/gambl/strelka-1.1/03-filtered/genome--grch37/05-20006T--05-20006N--matched/somatic.snvs.vcf
    shell:
        gzip -dc results/gambl/strelka-1.1/02-strelka/genome--grch37/05-20006T--05-20006N--matched/results/variants/somatic.snvs.vcf.gz > results/gambl/strelka-1.1/03-filtered/genome--grch37/05-20006T--05-20006N--matched/somatic.snvs.vcf
        (one of the commands exited with non-zero exit code; note that snakemake uses bash strict mode!)

Is there a way around this or will this module have to be updated to handle this situation more elegantly?

FYI @whelena and @BrunoGrandePhD I'm going to attempt a fix for this that separates paired and unpaired analyses a bit more. The problem really comes down to the pipeline not knowing what files should be in that directory due to the variable naming of output files in the two modes.

@lkhilton did you mention some time ago that you had thought of a fix for this?

Assuming this was fixed as there are no issues reported with strelka outputs and it was run on all of gambl


Allow more than one test to run at once

We're not name spacing anything, so only one test can run on CI at once. Should be able to have parallel tests run.

Ooooo neat! Be careful that some tests assume state from previous tests. This is only the case within a single file.


What are the mean and STD capabilities of hand_landmark_lite.tflite training data after data augmentation?

What are the mean and STD capabilities of hand_landmark_lite.tflite training data after data augmentation?

@sgowroji    I want to know the mean_values and std_values of mediapipe/modules/hand_landmark/hand_landmark_lite.tflite model training

Hello @LMR2018
You can find the mean and STD values for hand_landmark model in the page 5 of model card of hand solution.


Ability to extend and configure desired sink to report lag metrics, adding support to push lag metrics into InfluxDB as well.

Note: This retains currently supported Prometheus as the default sink, if it's not configured.

Hi @hariprasad-k,
Thank you for your contribution! We really value the time you've taken to put this together.
Before we proceed with reviewing this pull request, please sign the Lightbend Contributors License Agreement:
http://www.lightbend.com/contribute/cla

Fixed #129

Thanks for the PR @hariprasad-k.  I recently merged in #114 which introduced Graphite support.  It also created an abstraction for different sinks.  If you rebase maybe we can discuss adding more, like influxdb.

Feel free to reopen this PR if you're interested in continuing the work.


Revert "feat: add UTM tags automatically to all external links"

Reverts pottekkat/personal-website#112
Creates some issue with the link rendering in the home page for some reason.

Debugging the issue because it does not appear here.


Matching a string until closing parenthesis

If I have a string like this:
abc(hello (world) test)def

How do I match the (hello (world) test) part with the DSL methods? I also need to have the plain string, not tokenized, in the CST

How do I match the (hello (world) test) part with the DSL methods?

Depends what you want to do with it.
You could model it as a sequence of expressions with optional parenthesis expressions:
expressionList:
  expression+
;

expression:
  "(" expressionList ")" |
  StringLiteral
;

I also need to have the plain string, not tokenized, in the CST

Have a look at https://github.com/SAP/chevrotain/issues/872

For example if we had whitespace tokens and anything tokens, and I did $.CONSUME(Anything) it would error if I gave it whitespace.

That depends on the order in the lexer definition.
That regexp match does not occur when you do the "CONSUME"
It happens earlier when the whole input is tokenized .
The "CONSUME" only matches vs the types of the Token, not the actual regexp.

I know, but the anything token doesn't work this way, so what do you recommend?


Allow credential pass from conf variable for BQ extractor

Currently the key path may not be available as a file, we should support the case if the credential is pass as a string.
This is supported per gcp doc(https://google-auth.readthedocs.io/en/latest/reference/google.oauth2.service_account.html).

Codecov Report

Merging #154 into master will decrease coverage by 0.24%.
The diff coverage is 57.14%.

@@            Coverage Diff             @@
##           master     #154      +/-   ##
==========================================
- Coverage   83.32%   83.07%   -0.25%
==========================================
  Files          57       57
  Lines        2800     2806       +6
  Branches      297      289       -8
==========================================
- Hits         2333     2331       -2
- Misses        378      382       +4
- Partials       89       93       +4

Impacted Files
Coverage Δ

...tabuilder/extractor/bigquery_metadata_extractor.py
93.85% <57.14%> (-2.44%)
:arrow_down:

...ilder/transformer/regex_str_replace_transformer.py
95.83% <0%> (-4.17%)
:arrow_down:

...tabuilder/extractor/postgres_metadata_extractor.py
93.44% <0%> (-1.64%)
:arrow_down:

...abuilder/extractor/snowflake_metadata_extractor.py
93.44% <0%> (-1.64%)
:arrow_down:

databuilder/publisher/neo4j_csv_publisher.py
84.07% <0%> (-1%)
:arrow_down:

databuilder/task/neo4j_staleness_removal_task.py
56.98% <0%> (ø)
: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 92a3620...7aa2b8a. Read the comment docs.

cc @jinhyukchang


feat: Implementation of the SizeOf<T> trait

Feature Request
Describe the Feature Request
Introduction of a SizeOf<T> trait designed to retrieve the size in bits of various data types.
Describe Preferred Solution
trait SizeOf<T> {
    /// Returns the size in bits of T.
    fn size() -> T;
    /// Returns the size in bits of Self.
    fn size_of(self: @T) -> T;
}

impl U8SizeOf of SizeOf<u8> {
    fn size() -> u8 {
        8
    }
    fn size_of(self: @u8) -> u8 {
        U8SizeOf::size()
    }
}

After implemented for u8 for example, it would be then:
assert(10_u8.size_of() == 8, 'should be 8');
// or
assert(U8SizeOf::size() == 8, 'should be 8');

Describe Alternatives
In Rust, size_of would return the size in bytes, not bits.
I choosed bits because I thought it would be safer (?) to do a multiplication to retrieve the size in bytes rather than a division since how they are handled in Cairo but maybe it's ok. Let me know!
Related Code
Implemented in Kakarot::utils::num for PR #410
The usage was:
fn shl(self: T, shift: T) -> T {
        if shift > shift.size_of() - One::one() {
            panic_with_felt252('mul Overflow');
        }
        (...)
    }

Additional Context

If the feature request is approved, would you be willing to submit a PR?

[X] Yes
[ ] No

this is not "size" - this might be "representation bits" - "size" implies the size in memory.
but asside from that sounds fine.


Deploy metallb for DNS in baremetal job

This is is required to configure the endpoint. As we're not using NETWORK_ISOLATION=true with baremetal job we have to configure it separately for DNS LB endpoint.
As a pull request owner and reviewers, I checked that:

[x] Appropriate testing is done and actually running

Depends-On: https://github.com/openstack-k8s-operators/install_yamls/pull/356

recheck

recheck

recheck

We need to merge this asap as the dependent install_yamls patch got merged already to avoid broken CI.

I force merged it to avoid broken CI, as it was approved by @abays.


Errors received in Datepicker dynamic nzMode selections.

Reproduction link
https://stackblitz.com/edit/ng-zorro-datepicker-dynamic-nzmode?file=src/app/app.component.ts

Steps to reproduce
When I want to dynamically change nzMode;
No problem when I choose daily
No problem when I choose weekly
No problem when I choose monthly
No problem when I choose yearly
Only when I choose daily and then weekly or weekly and then select daily,
visually an error occurs when I click somewhere in the popup popup, the problem is fixed. Do you think this is a bug?
As the second question, when I dynamically change nzMode in the visual, the input field with the date is not formatted according to nzMode, so when I select annual, the input field is not displayed as "yyyy". Is there a solution to this?
Thanks.
What is expected?
Elimination of errors in visual and sample work in dynamic selection of nzMode.
What is actually happening?
--

Environment
Info

ng-zorro-antd
10.1.1

Browser
Chrome

Hello,
Thank you so much for solving this problem.
Formats change in dynamic change.
However, the following problem persists? Do you know about this?
or an ongoing process.
When we switch from the daily selection to the weekly selection, the popup of the daily show is also shown in the week. Likewise, the same problem occurs when switching from weekly to diary.
I tried to explain it visually.
Thanks to those who have already contributed.

@C3maLi You did not switch mode in your gif, the official document works fine.

I'm so sorry for sharing the wrong gif.
I just added the gif again, can you see it again?


Live transpiling

Hi.
I'm implementing custom validation rules.
It takes time to run :
$ mvn clean package -Ptranspilation

Is there any option to transpile lively along with editing original .java files?

Thanks! I understood.
I realized that jsweet-maven-plugin is ready for Hot Transpilation :
$ mvn jsweet:watch

https://github.com/lgrignon/jsweet-maven-plugin#hot-transpilation
I'll try this.

Awesome - if you can configure something that works well for you, I would love to add it to the documentation/readme.


[Question]How to debug

I got a bug on Osiris.
I added some funtions on it.
But osiris is not a exe.It's a DLL
so I have no idea to debug.
Could you tell me the way to debug on DLL?
Thanks danie.

you mean this?

download source code and compile it
run csgo and inject the cheat
In VS 2019 go to the Debug section
click Attach to Process...
select csgo.exe
when ur game crash u will see what line of code crashed

you mean this?

download source code and compile it
run csgo and inject the cheat
In VS 2019 (or any version of vs) go to the Debug section
click Attach to Process...
select csgo.exe
when ur game crash u will see what line of code crashed

yes
do you have idea to do that?

you mean this?

download source code and compile it
run csgo and inject the cheat
In VS 2019 (or any version of vs) go to the Debug section
click Attach to Process...
select csgo.exe
when ur game crash u will see what line of code crashed

yes
do you have idea to do that?

They literally told you how to do it...

but it's in fact that I can't get the line of code

but it's in fact that I can't get the line of code

all you have to do is :
1.download visual studio
2.open your project in visual stdio(the .sln file)
3. try to compile, if it works then it works
4. if you encounter a glitch in-game, try debugging by hovering your mouse over debug in visual studio clicking Attach to process then select csgo and boom, if your game crashes then you will know what caused the crash:
ps: in the case your game doesn't crash which is presumably your problem,you're gonna have to go back to each line of code individually and if there are any reasons that could cause the cheat to bug out, but i recommend just reimplementing it carefully on another build because you clearly don't know anything about programming( I don't mean to be offensive)

but it's in fact that I can't get the line of code

all you have to do is :
1.download visual studio
2.open your project in visual stdio(the .sln file)
3. try to compile, if it works then it works
4. if you encounter a glitch in-game, try debugging by hovering your mouse over debug in visual studio clicking Attach to process then select csgo and boom, if your game crashes then you will know what caused the crash:
ps: in the case your game doesn't crash which is presumably your problem,you're gonna have to go back to the stuff you implemented and check if there are any reasons that could cause the cheat to bug out, but i recommend just reimplementing it carefully on another build because you clearly don't know anything about programming( I don't mean to be offensive)

I injected the releasex86 DLL into csgo and csgo was crashed.
I got something not useful from vs2019 though I Attach to process.These messages aren't I want

but it's in fact that I can't get the line of code

all you have to do is :
1.download visual studio
2.open your project in visual stdio(the .sln file)
3. try to compile, if it works then it works
4. if you encounter a glitch in-game, try debugging by hovering your mouse over debug in visual studio clicking Attach to process then select csgo and boom, if your game crashes then you will know what caused the crash:
ps: in the case your game doesn't crash which is presumably your problem,you're gonna have to go back to the stuff you implemented and check if there are any reasons that could cause the cheat to bug out, but i recommend just reimplementing it carefully on another build because you clearly don't know anything about programming( I don't mean to be offensive)

I injected the releasex86 DLL into csgo and csgo was crashed.
I got something not useful from vs2019 though I Attach to process.These messages aren't I want

Maybe show us what messages you got? A screenshot? Like we cant read your mind bro...

I injected the releasex86 DLL into csgo and csgo was crashed.

Try to compile it as debug

I injected the releasex86 DLL into csgo and csgo was crashed.

Try to compile it as debug

It must be in Debug x86!!!

I injected the releasex86 DLL into csgo and csgo was crashed.

Try to compile it as debug

It must be in Debug x86!!!

Oh in DEBUG x86 I can't inject.if I do, it will crash without any bug

I injected the releasex86 DLL into csgo and csgo was crashed.

Try to compile it as debug

It must be in Debug x86!!!

Oh in DEBUG x86 I can't inject.if I do, it will crash without any bug

Have the project set to debug x86. When you have it on debug x86, build the project.

After you build it, you want to attach the VS debugger to CS:GO.

Find "csgo.exe" on the list, click it and then click "Attach". A pop-up may appear that will ask you to restart Visual Studio in administrator mode. If that happens, you want to click "Restart" or "OK".

When the debugger is attached, you want to inject the Debug DLL in the /Debug folder. When the game crashes, the place where the error occured shoud show up in Visual Studio.

I injected the releasex86 DLL into csgo and csgo was crashed.

Try to compile it as debug

It must be in Debug x86!!!

Oh in DEBUG x86 I can't inject.if I do, it will crash without any bug

Have the project set to debug x86. When you have it on debug x86, build the project.

After you build it, you want to attach the VS debugger to CS:GO.

Find "csgo.exe" on the list, click it and then click "Attach". A pop-up may appear that will ask you to restart Visual Studio in administrator mode. If that happens, you want to click "Restart" or "OK".

When the debugger is attached, you want to inject the Debug DLL in the /Debug folder. When the game crashes, the place where the error occured shoud show up in Visual Studio.

@mkot2 @HowToMakeFaceitAB  How to follow the same procedure on Linux?


SQL Exception on Location List

Hi!
After a fresh install (Sulu 2.6.4, Symfony  7.1.3, Postgres) I am getting the following error on /admin/api/locations?page=1&limit=10&fields=name,image,street,number,postalCode,city,state,countryCode,id&flat=true
"An exception occurred while executing a query: SQLSTATE[42703]: Undefined column: 7 ERROR:  column \"sclr_2\" does not exist\nLINE 1: ...event_id IN ($1, $2) GROUP BY a0_.event_id HAVING sclr_2 < $...\n                                                             ^"

This is coming from EvenTranslationRepository findMissingLocaleByIds. I didn't dive any deeper yet.

Hi David, I'm not sure where sclr_2 is coming from. Will take a look at it later on.

Could you provide me with the whole statement?

I fear that there is a problem with Postgres. I can't say for sure, but the column indicates this.
Stumbled upon this: https://github.com/Sylius/Sylius/issues/11851
But can't test postgre at the moment since lack of time. Perhaps late in the next month. If you figure out something I'd be eager to get to know the fix.

Thanks for looking into this and pointing into the right direction! I've opened this PR to fix this error.
I'm not sure if the additional grouping by locale can cause issues, I don't have that case yet.

Have to test it in my spare time later.


Add support for webgl 1 draw buffers extension for the MultiRenderTarget class

https://www.khronos.org/registry/webgl/extensions/WEBGL_draw_buffers/
Also make sure that it works if no extension and no webgl2

Thanks! Will give you updates on this issue :)


Progress on Custom Agent Class?

Hey there, just asking on what the progress is on the "Custom Agent Class" for langchain? 🙂
https://langchain.readthedocs.io/en/latest/modules/agents/examples/custom_agent.html#custom-agent-class

This would be such a powerful tool if available!

I'm also keen, as I'm looking for a way to create a n8n custom agent.

Also interested in this. I need to write an agent class which has capabilities beyond what the current agents support.

Guys?

Custom agents would be super valuable. I thought it was implemented already.
Is there any progress on this? I want to contribute my help on the relevant implementation but would feel more confident if one of the core members leads the effort, since the internals of the overall project are a bit obscure to me still.


Visual Editor arrow key navigation has incorrect behavior

Version
711c2344
Describe the bug
There is still some issues with the cursor move logic. See my gif (I was pressing ← and →):

To Reproduce
Steps to reproduce the behavior:

create a promt input node
create a if condition node
insert steps in IfCondition's two branches
focus on the true branch's node, pressing ← and → repeatively

Expected behavior
pressing ← and → should lead me to sibling nodes
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Expected behavior

If pressing ↑ and ↓, cursor should move to previous / following node.
If cursor is focusing on a node with siblings, pressing ← and → should navigate to siblings.
If cursor is focusing on 'BotSays' node of a prompt type, pressing ↓ should nav to 'UserAnswers', then pressing → should nav to 'Exceptions'; if focusing on 'Exceptions', pressing ↑ should nav to 'BotSays'

addressed in #1082


“abc this is a test” popup displayed when user repeatedly switch between “Settings” & “Help” options after redirecting back from “FAQ” page

Frequency: 100%
Repro on build version: Test Flight Build v1.21.0
Does Not Repro on build version:  Android Internal Build V1.21.0
**Repro on devices:**iPhone SE (13.6.1), iPhone XR (13.2.3), iPhone 12 (15), iPhone SE (14.2), iPhone 6+ (12.4.5)
Does Not Repro on devices: Redmi Note 8 (9.0), One Plus 7t (11.0)
Testing Account:<PHONE_NUMBER> /Recovery Phrase
Pre-condition: Make sure that phone number should present in Hamburger Menu
Repro Steps:

Launch the application & log in with the test account
Tap on “Hamburger Menu” & redirect to the “support.valoraapp.com” page by selecting the “Frequently Asked Questions” option
Redirect back to Valora & select the “Settings” option from the Hamburger menu
Repeat steps 2&3 for one more time & observe that
Current Behavior: “abc this is a test” popup displayed after tapping on “Settings” option after redirecting back from the “Help” page
Expected Behavior: User should able to see “Settings” page as it is without any “abc this is a test” popup
Investigation:
Also, we have observed the same issue when the user is on the prompt to accept wallet connect permission.
Also, we have observed the same issue while connecting the wallet by scanning QR code from a desktop browser
Also, we have observed the same popup randomly after creating an account throughout an application
Impact: Bad user experience
Attachment: abcTestPopupRandomlyShows

@jeanregisser do you have any context around this?

Yes this was for testing CleverTap. I've stopped this. You shouldn't see it anymore.

Hey @jeanregisser We have verified this issue using Test Flight Build V1.21.0 & observed that issue is fixed now.
Verified On Devices: iPhone XR (14.6), iPhone 6+ (12.4.5)


[iOS][0.74.3] 'react/utils/fnv1a.h' file not found on xcode build

Hi Artur :)
I am in the process of migrating my React-Native app (0.74.3) to the new architecture. By activating the module, on iOS only, I have an error when compiling xcode :

I can't figure out if the problem is with my xcode config or not. I've delete my Pods/Podfile.lock, but same result.
Maybe the problem is my RN version ? I planned to upgrade to 0.75.x, but it's not that old ^^
I saw on an old issue that you mentioned this problem which was similar: https://github.com/react-native-webview/react-native-webview/issues/3218
Do you have a little idea ? Thank you !

Hello @stephane-r,
If I am not mistaken, starting with version 0.74.3, React Native has enabled bridgeless mode by default. Unfortunately, the current version of the package doesn't support bridgeless mode, but migration is planned for a future release.
I will check the PR and try to align with react-native-builder-bob to ensure that this is the correct solution.

Thank for your response.
I thought that, by activating the new architecture, bridgeless mode was necessarily disabled ?

Hello @stephane-r ,
Thank you very much for the fix. The changes have been added and tested on versions 0.71-0.75 of React Native.
I hope version 0.3.7 will resolve this issue, as it includes your fix and additional lines based on the react-native-builder-bob pod example.


KAFKA-9930: Prevent ReplicaFetcherThread from throwing UnknownTopicOrPartitionException upon topic creation and deletion.

When does UnknownTopicOrPartitionException typically occur?

Upon a topic creation, a follower broker of a new partition starts replica fetcher before the prospective leader broker of the new partition receives the leadership information from the controller (see KAFKA-6221).
Upon a topic deletion, a follower broker of a to-be-deleted partition starts replica fetcher after the leader broker of the to-be-deleted partition processes the deletion information from the controller.
As expected, clusters with frequent topic creation and deletion report UnknownTopicOrPartitionException with relatively higher frequency.

What is the impact?

Exception tracking systems identify the error logs with UnknownTopicOrPartitionException as an exception. This results in a lot of noise for a transient issue that is expected to recover by itself and a natural process in Kafka due to its asynchronous state propagation.

Why not move it to a lower than warn-level log?

Despite typically being a transient issue, UnknownTopicOrPartitionException may also indicate real issues if it doesn't fix itself after a short period of time. To ensure detection of such scenarios, we set the log level to warn.

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)

@ijuma do you think we may be able to iterate on this PR?

ok to test

ok to test

ok to test

ok to test

retest this please

retest this please

Unrelated flaky test failures:
org.apache.kafka.streams.integration.EosBetaUpgradeIntegrationTest.shouldUpgradeFromEosAlphaToEosBeta[false]
org.apache.kafka.streams.integration.EosBetaUpgradeIntegrationTest.shouldUpgradeFromEosAlphaToEosBeta[true]
org.apache.kafka.streams.integration.EosBetaUpgradeIntegrationTest.shouldUpgradeFromEosAlphaToEosBeta[false]
org.apache.kafka.streams.integration.EosBetaUpgradeIntegrationTest.shouldUpgradeFromEosAlphaToEosBeta[true]


Add Bearer Authentication to Swagger Documentation

I need guidance on how to implement Bearer Authentication in the Swagger documentation for our Fiber middleware. Specifically, I'm unsure about what values to provide for the PreauthorizeApiKey field in the Swagger configuration. Any assistance on how to correctly configure this for Bearer Authentication would be greatly appreciated.
app.Get("/swagger/*", swagger.New(swagger.Config{
	PreauthorizeApiKey: "Bearer",
}))

//main.go

//      @Title "Your API Name"
//	@SecurityDefinitions.apikey	Bearer

This works fine for OAS 2.0.
On your code, you should name which routes (by placing it before the functions) should have this security param.
using
@Security     Bearer

//main.go

//      @Title "Your API Name"
//	@SecurityDefinitions.apikey	Bearer

This works fine for OAS 2.0.
On your code, you should name which routes (by placing it before the functions) should have this security param.
using
@Security     Bearer

But it seems like this method doesn't work for adding the token in the format Bearer xxx, right?

//main.go

//      @Title "Your API Name"
//	@SecurityDefinitions.apikey	Bearer

This works fine for OAS 2.0.
On your code, you should name which routes (by placing it before the functions) should have this security param.
using
@Security     Bearer

But it seems like this method doesn't work for adding the token in the format Bearer xxx, right?


git-control package

I use git-control for basic git tasks. Would be nice if it fits with material-ui

I rather use git-plus than git-control because the commit and the file that used is actually fits in with material-ui

same bother here. Both using git-plus and git-control.

You take too long 😂  @zzjin

@christoga I know… I have a full time job that pays for my bills. I'll get to it eventually.

I didn't mean about that, I mean the way he reply bro.
No offense ✌

None taken 😄 . I didn't mean to sound harsh, I just wanted to explain the delays.

I also have a job dude,
I usually open repo issues when I'm on bed. 😂


Clarification on EmptyStatement

Hi @sandersn,
I just wanted to clarify the first exercise "Add EmptyStatement".
Is an EmptyStatement a semicolon in JavaScript/TypeScript (link)? So the idea is to stop the execution of a statement?

By the way, @sandersn, I've opened a PR on my fork about this first exercise I wanted to know if you could give me feedback. Maybe it's not the best implementation or it's not even what you expected as a result but I could manage to pass all the tests now.
Here it's: https://github.com/imteekay/mini-typescript/pull/2
Thanks!

Hi @imteekay,
Im not Sanders but since i'm watching this repo I got a notification.
I have the same assumption that the EmptyStatement is the same as the one talked about in the link.
I would not say that it stops the execution of a statement, but that its a statement without anything to execute. MDN is a great source, I suggest also peeking at what part of the language spec its referencing
As you can read at runtime it does nothing but "return empty".
If you are interested in how to read the specification, I suggest googling "how to read the ECMAScript specification". It will provide you with various articles and blog posts that dive into the separate details of how to read the spec.
This is of course not required, but when implementing a compiler for an existing language I find it useful to learn how to read the official specifications.
On your PR:
While its an interesting take, I would argue using undefined to represent an empty statement while passing the tests is not a great approach.
You already mentioned creating a new AST node type, I think that would be a better approach.
The main argument for me would be to have an accurate representation of the source code in the AST.
Using undefined to represent an empty statement is unclear and can cause ambiguity when the compiler increases in complexity. An explicit EmptyStatement type would leave no doubt on what it represents.

Thanks @kevinramharak . I agree with your comments.
@imteekay I'll leave a review on the PR although my overall take is the same as @kevinramharak 's.

@kevinramharak, thanks for the overview!
@sandersn, I made some improvements now using an AST node for the empty statement.
For this test var x: number = 1;;var y: number = 2;, my implementation is still failing because of the separator function as the semicolon separates the var x: number = 1 and ; for this test and when it gets out of the parseSeparated, it expects to the next token to be an EOF token but it still tokens to be parsed.
I'm still thinking about how to make it work and wanted to ask if there's any relation to the second exercise "Make semicolon a statement ender, not statement separator." or if it's simpler than that.
The changes I made remains in the same PR https://github.com/imteekay/mini-typescript/pull/2

Now the PR has the full solution https://github.com/imteekay/mini-typescript/pull/2

New token for the empty statement
New test
Updated the existing tests

Thanks for all the help guys! Just closing this issue as I merged the PR 🎉


Programmatically generate invoices for workshop ticket orders

Addition Description
Generate a meaningful invoice/receipt to be attached to the emails already being sent to people who purchase tickets to QIIME 2 workshops.
Current Behavior
Purchasers receive an invoice attached to an email from the third party service that handles the transactions. These are frequently lost or caught in spam filters resulting in the need for the admin team to manually create an invoice for the workshop attendee to be able to receive a reimbursement from their institute.
Proposed Behavior
Automatically create something resembling these invoices to be attached to the initial confirmation emails we send to attendees.
Questions

What do we want the .pdf to look like?
How long do we want this .pdf to live on the server?
How do we want to integrate this .pdf creation into the current ordering process? It shouldn't be too hard to just automatically create the .pdf on the current submit page right before we email it off to them.

References

https://workshops.qiime2.org/

I emailed Alyssa Barrett at HR about how she currently goes about creating these files, and she emailed me a .xlsx file for an invoice she produced for the workshop in New York on June 3rd/4th. This got me thinking, would it perhaps make more sense to create a .xlsx template that we can modify to create invoices? Then we save the completed invoice to a .pdf before attaching it to the email? There is a module called XlsxWriter that should, at a glance, let us do something like this. With a .pdf, the entire file will have to be programmatically generate each time we want an invoice due to the lack of real ability to edit a .pdf. With a .xlsx file (or any similar format really) we would just need to fill in the template (which may result in some things being moved around depending on what their invoice needs). I'm not sure if this would necessarily be better or easier, but it's a thought.

In fact, maybe this shouldn't be done as a .pdf using reportlab at all, maybe it makes more sense to make this as a .html file then convert that to a .pdf.

This got me thinking, would it perhaps make more sense to create a .xlsx template that we can modify to create invoices? Then we save the completed invoice to a .pdf before attaching it to the email?

I like where you're going with this --- luckily that is what the database is for!

With a .pdf, the entire file will have to be programmatically generate each time we want an invoice due to the lack of real ability to edit a .pdf.

That's fine, no need to edit the PDF. The PDF is only created after all the info is collated.

In fact, maybe this shouldn't be done as a .pdf using reportlab at all, maybe it makes more sense to make this as a .html file then convert that to a .pdf.

That could work, just another strategy for making a PDF - it all works for me!

What do we want the .pdf to look like?

How about the physical printout I gave you last week? I think that has been working well, and can get us set up for any other edits.

How long do we want this .pdf to live on the server?

Probably forever --- but, on S3, not the Django webserver. I can show you how we can do this next week when I am around.

How do we want to integrate this .pdf creation into the current ordering process? It shouldn't be too hard to just automatically create the .pdf on the current submit page right before we email it off to them.

There is a callback endpoint that gets triggered when NAU receives payment, this is a good place to start.

@thermokarst :

There is a callback endpoint that gets triggered when NAU receives payment, this is a good place to start.
I looked at this, and I don't think I have a way to test this right now do I? Unless there is a way for me to spoof NAU receiving payment locally?

You can simulate the NAU payment receipt by manually POSTing to your development server. $ORDER_TRANSACTION_ID, $ORDER_TOTAL, and $ORDER_DATETIME are all specific to the order in question. You can use any tool for this, I just happen to like curl:
curl -d "unique_id=$ORDER_TRANSACTION_ID&amount=$ORDER_TOTAL&date_time=$ORDER_DATETIME" \
  -H "Content-Type: application/x-www-form-urlencoded" \
  -X POST http://localhost:8000/confirm/

@thermokarst awesome, thank you

@thermokarst this is where I'm at with the document right now.
Invoice.pdf

This looks awesome @Oddant1! Want to open a Draft PR with your work to date? We can continue detailed discussion there. Thanks!

@thermokarst I'll do that before I leave today with the latest version of the document. It's currently displaying on the submit view because I couldn't make the POST with curl work in any way I could make sense of. I'll talk to you about that in person next chance I get.

@thermokarst new example invoice generated by the code in the inital PR #124
Invoice1.pdf

@thermokarst made some of the changes we discussed
invoice2.pdf


check-script-has-no-table-name doesn't ignore text within Jinja comment blocks

Describe the bug
If we use the word "from" or "join" in a Jinja comment block the parser thinks that the next word is the name of a table or CTE and passes an error.
sql:
{# This is a test of the check-script-has-no-table-name hook, from pre-commit-dbt

We would expect the hook to ignore this text because it is in a jinja comment block
and not actually a join to any other table.

#}

output from pre-commit run
Check the script has not table name......................................Failed
- hook id: check-script-has-no-table-name
- exit code: 1

test.sql: does not use source() or ref() macros for tables:
- pre-commit-dbt
- to

To Reproduce
Steps to reproduce the behavior:

create a new sql file
add into the file

{# This is a test of the check-script-has-no-table-name hook, from pre-commit-dbt

We would expect the hook to ignore this text because it is in a jinja comment block
and not actually a join to any other table.

#}

Run pre-commit with the check-script-has-no-table-name hook enabled

Expected behavior
I'd expect the parser to ignore comment blocks since it's not important if they don't use ref() or source()
Version:
v0.1.1

Good catch @neddonaldson!! Thanks a lot. I fixed that in #11. If you do not wait for a new version of pre-commit-dbt you can run pre-commit autoupdate --bleeding-edge to update to HEAD.


Any way to access JSON sent to Facebook

I have a basic app that:

takes a message via a REST API
relays that message to Facebook Messenger using your wonderful library
returns a response via the same REST API

The problem I'm running in to is that in step 3 I'd like to return the JSON sent to Facebook Messenger, so that a QA script can hit my API and verify that the incoming message resulted in the correct outgoing message to Facebook ... but there is currently no way to access the request.  It's created inside the various send* methods, then passed to sendRequest which sends it off to Facebook without exposing it in any way.
Could you please expose the request, or at least req.json?  It sees to me that the simplest way to do this would be to change line 187 from:
    cb(null, body);

to:
    cb(null, body, req);

However, that only gives you the request after the response comes back.  What would truly be ideal, in my opinion, is if sendRequest could simply return the req, and every send* method returned the result of sendRequest ...
sendTextMessage (id, text, notificationType, cb) {
  const messageData = {
    text: text
  }
  return this.sendMessage(id, messageData, notificationType, cb)
}

const sendRequest = (req, cb) => {
  request(req, (err, res, body) => {
    if (!cb) return
    if (err) return cb(err)
    if (body.error) return cb(body.error)
    cb(null, body)
  })
  return req;
}

Could you please consider implementing one of these ways (or some alternative way) to expose the request object?

The request JSON is predictable as it's built with whatever you input as parameters to the functions. You can just recreate it:
var req = {
      url: 'https://graph.facebook.com/v2.6/me/messages',
      qs: {access_token: this.token},
      method: 'POST',
      json: {
        recipient: {
          id: id
        },
        message: 'XX'
        notifcation_type: ''REGULAR
      }
    }

Wouldn't that be enough? You can even wrap it.

So, the problem is that if I do:
 sendImageMessage ('foo', 'foo.png', 'bar');

I have to essentially re-create your sendImageMessage method within my code to know that the resulting JSON will be:
{
  'attachment': {
    'type': 'image',
    'payload': {
      'url': imageURL
    }
  }
}

I'd prefer not to do that, especially since I'd have to do so for every method in the library (essentially re-creating the entire library in my code).
Ideally instead I'd like to just access the above JSON somehow after it is generated.


Update dependencies

Landed in https://github.com/metarhia/metarhia.com/commit/7ae674b3e83f530d8d7c5b1a9e6300d4617089c5


Angular components doesn't load at once- needs a click or reload-Only in EDGE

Versions
<!--
Output from: `ng --version`.
If nothing, output from: `node --version` and `npm --version`.
  Windows (7/8/10). Linux (incl. distribution). macOS (El Capitan? Sierra?)
-->

Repro steps

Step 1
Step 2
Step 3

Observed behavior
<!-- Normally this includes a stack trace and some more information. -->

Desired behavior

Mention any other details that might be useful (optional)

I think you're having some weird errors and can't debug it, and so, understandably, you're frustrated. But this is a tracker for bugs and feature requests. If we can we also try to help with support requests. We're very lucky to have awesome community members that also spend time trying to help people here in the issue tracker.
But we can't really help much at all if you don't fill in the issue template. I read your issue and have no idea what's going wrong and how to help you. And when other users search the issue tracker and see your issue, it also doesn't help them.
So I think the best thing to do is to close this issue, and ask you to open a new one. In that new one please fill in the issue template and give us an idea of what's happening and how to reproduce it. That way we'll have a chance to help you.


Weighted dataframe slicing

Description
This fixes #53 by ensuring that the invisible weights array is sliced whenever the array is.
Checklist:

[x] I have performed a self-review of my own code
[x] My code is PEP8 compliant (flake8 anesthetic tests)
[x] My code contains compliant docstrings (pydocstyle --convention=numpy anesthetic)
[x] New and existing unit tests pass locally with my changes (python -m pytest)
[x] I have added tests that prove my fix is effective or that my feature works

Closes #54

Closes #44


Error on startup with 6.2.0

Current Behavior
App throws an error on startup after upgrading to 6.2.0:
$ npm start
14:23:37.792Z DEBUG github: GitHub request: GET /installation/repositories - 415 undefined (installation=182143)
  params: {
    "per_page": 100,
    "baseUrl": "https://api.github.com",
    "request": {
      "timeout": 0
    }
  }
14:23:37.793Z ERROR probot: {"message":"If you would like to help us test the Integrations API during its preview period, you must specify a custom media type in the 'Accept' header. Please see the docs for full details.","documentation_url":"https://developer.github.com/v3"}
  HttpError: {"message":"If you would like to help us test the Integrations API during its preview period, you must specify a custom media type in the 'Accept' header. Please see the docs for full details.","documentation_url":"https://developer.github.com/v3"}
      at response.text.then.message (/Users/bkeepers/projects/probot/unfurl/node_modules/@octokit/rest/lib/request/request.js:75:19)
      at <anonymous>
      at process._tickDomainCallback (internal/process/next_tick.js:228:7)

The app appears to continue to function, but the /stats endpoint doesn't work, and anything using probot-scheduler is broken.
Expected behavior/code
A clear and concise description of what you expected to happen (or code).
Environment

Probot version(s): v6.2.0

Possible Solution
This looks like a regression in Octokit not properly setting the preview header for the /installation/repositories endpoint.
cc @gr2m

https://github.com/octokit/rest.js/pull/870 appears to be the culprit. Some APIs that require the machine-man-preview header use an installation token and not a jwt Bearer token.

When you do a fresh npm install then it should get<EMAIL_ADDRESS>or higher which should resolve the problem. You can check with npm ls @octokit/rest.

Confirmed this is fixed with https://github.com/octokit/rest.js/pull/878


Type in client credentials docs

Noticed a type in docs.
SpotifyAPI-NET/SpotifyAPI.Web/Authenticators/ folder contains only ClientCredentialsAuthenticator class

Thanks! 👍


chore: #27 Upgrade Spring Boot to 3.2.2

BREAKING CHANGE: Major Spring Boot version upgrade

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


PlayerJoinEvent

Using the latest dev build of Old Combat Mechanics as of July 15th, 2018
Using PaperSpigot 1.12.2 Also updated latest version as of July 15th, 2018
Using ProtocolSupport
I am now getting a new error every time someone joins that I was not getting on an older version. Not sure what changed. Here is the error:
https://pastebin.com/kBgTdFU0

Can't reproduce with
> version
[23:29:13 INFO]: This server is running Paper version git-Paper-1500 (MC: 1.12.2) (Implementing API version 1.12.2-R0.1-SNAPSHOT)
[23:29:13 INFO]: Checking version, please wait...
[23:29:13 INFO]: You are running the latest version

> protocolsupport buildinfo
[23:29:35 INFO]: protocolsupport.ProtocolSupport$BuildInfo(buildtime: 2018.06.12 at 05:27:43 MSK, buildhost: build.true-games.org, buildnumber: 239)

> ocm
[23:29:57 INFO]: ----------------------------------------------------
[23:29:57 INFO]: OldCombatMechanics by gvlfm78 and Rayzr522 version 1.6.5
[23:29:57 INFO]: You can use /ocm reload to reload the config file
[23:29:57 INFO]: -----

and default configs for all.

Same version of paper and same version of protocol support. I just can't verify which build of 1.6.5 you are testing with. I was using the latest dev build #44 on the Jenkins page. If I use the release on the spigot page it works without error.
If you are also using the same build, then perhaps:
Using the latest version of Hexacord (ProtocolSupport version of BungeeCord), ProtocolSupportLegacySupport and ProtocolSupportPotions.
In any scenario, it stems from the changes between the released build of 1.6.5 and the current dev build.

I am using #44 too. The change is the deeper packet inspection that rewrites all outgoing scoreboard packets in order to reliably disable collision — even if you use a scoreboard or another scoreboard plugin.
I don't really want to go through the trouble of setting up bungee (I have literally never done it), but I have installed this so far:
> pl
Plugins (7): AnimatedScoreboard, OldCombatMechanics, PlaceholderAPI, ProtocolLib, ProtocolSupport, ProtocolSupportLegacySupport, ProtocolSupportPotions

What I tried

Joining with an 1.8.8 client on a 1.12 server — everything works
Joining with an 1.12 client on a 1.12 server — everything works
Having a scoreboard plugin send out packets (AnimatedScoreboard) to trigger the packet rewriting — everything works

Problem
I still can not reproduce the error. Can you send a full list of plugins or try to find out which one introduces the error (using a local test server remove plugins and see when it stops crashing. You shouldn't need to use your real one for this, so it shouldn't impact players)?

Made a dev server to test this behind my bungee. Turns out, the issue seems to be caused by iDisguise (latest version).

Thanks!
I can reproduce it with that (the author probably had a good reason to do what he did, but it's something I didn't anticipate).
It is currently nearly midnight here, I will fix it when I find time tomorrow. It shouldn't be hard to do, but I have some things planned for tomorrow, though I should get it done in the next 24 hours.

May be fixed in the commit referenced above, try building it yourself or using the attached jar please.
OCM.zip

Confirmed, fixed. Thank you <3


RepoResource::delete() could return a list of deleted resources

Currently the RepoResource::delete() returns nothing but when recursive deletion is used it would be nice to be able to check which resources have been actually deleted.
Similarly it would be nice to know which references have been removed if $references option was set.

As of 765548cdbfb2784010697bb2ac22bb16758b4668 RepoResource::delete() returns removed resources' URIs as array<string>. References


Enable log level support and implement debug logging

Enable log level support and implement debug logging.
Logging is written out to stderr and ends up in logging for smtpd(8), thus the maillog file. Implement log level support so that the logging level can be set at startup.

Closed with 1e53afb.


Allow manual invoice creation in InvoiceService

Closes #170

I shifted some things around a bit. Looks cleaner. There's a round trip to fetch project that could have been avoided the in previous version, but that's the only logical difference.


Install pydocstyle for checking docstrings

Signed-off-by: Fridolin Pokorny<EMAIL_ADDRESS>Related: https://github.com/theupdateframework/python-tuf/pull/2282 https://github.com/theupdateframework/python-tuf/pull/2281

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at ?%

Totals

Change from base Build<PHONE_NUMBER>:
0%

Covered Lines:

Relevant Lines:
0

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 98.139%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1344

Relevant Lines:
1361

💛  - Coveralls

This and #2289 will conflict a little bit -- I'm happy to leave #2289 waiting and fix the conflict there but if you're not working on this now, we can switch the other way round.

Looks like pydoc caught something. :)


Consolidate Images folder

Description
Consolidated the images folder
Linked Issue
close #152
Thankyou❤

Looks great, thank you for your help.

Can you please add hacktoberfest-accepted label.


There is a problem with info packaging and compilation date display

Is this a regression?
Yes
Description
编译日期和打包日期格式有问题，写一个就可以了没必要写这么多
Please provide a link to a minimal reproduction of the bug
No response
Screenshots or videos

Please provide the version you discovered this bug in (check about page for version information)
No response
Anything else?
No response

我来试一下

我来试一下
ok


Fixed Queue to clear children before retry

this fixes issue with Queue recipe as reported in #352

:+1:  seems ok to me!


fix: response for bad request - invalid schema - more detailed resp

retest this please

retest this please


Build is Broken: antDeploy looks for user / pass in global options rather than localc

antdeploy: {
options: {
root: 'deploy/',
apiVersion: '29.0',
user: 'gunish.chawla@COMPANYNAME',
pass: 'PASSWORD',
token: ''
},
  dev: {
    option: {
      user: 'gunish.chawla@COMPANYNAME', // This variable is no longer read
      pass: 'PASSWORD', // This variable is no longer read
      token: '' // This variable is not longer read
    },
    pkg: {
      staticresources: ['*']
    }
  }
},

Sorry for the incredibly delayed response. If you're still having an issue, please feel free to re-open.


[FEATURE] Add EVM popup display

If a user does not have a COA and tries to open an EVM dApp or perform an EVM operation, the wallet should display a popup prompting them to enable EVM.

If a user does not have a COA and tries to connect an EVM dApp , the wallet should also display a popup prompting them to enable EVM.
https://github.com/user-attachments/assets/4f5380b2-695d-4d7f-9cff-09e48da68432

The above issue already fixed in Android prod version i1.5.15.
The demo video is attached  below.
https://github.com/user-attachments/assets/f7a788a0-aa83-4d82-869d-aefd26c5690b


pubsub: handle disconnection while checking if we are connected

Pull Request check-list
Please make sure to review and check all of these items:

[ ] Does $ tox pass with this change (including linting)?
[ ] Does travis tests pass with this change (enable it first in your forked repo and wait for the travis build to finish)?
[ ] Is the new or changed code fully tested?
[ ] Is a documentation update included (if this change modifies existing APIs, or introduces new ones)?

NOTE: these things are not required to open a PR and can be done
afterwards / while the PR is open.
Description of change
After upgrade from redis-py 2.x to 3.3.11, I was getting these tracebacks in a loop:
2020-01-06 15:32:36,519 ERROR Problem reading redis queues
Traceback (most recent call last):
  File "/code/molly/pricefeed/pubsubutils.py", line 217, in run
    msg = self.pubsub.get_message()
  File "/env/lib/python3.6/site-packages/redis/client.py", line 3297, in get_message
    response = self.parse_response(block=False, timeout=timeout)
  File "/env/lib/python3.6/site-packages/redis/client.py", line 3183, in parse_response
    if not block and not conn.can_read(timeout=timeout):
  File "/env/lib/python3.6/site-packages/redis/connection.py", line 695, in can_read
    return self._parser.can_read(timeout)
  File "/env/lib/python3.6/site-packages/redis/connection.py", line 404, in can_read
    raise_on_timeout=False)
  File "/env/lib/python3.6/site-packages/redis/connection.py", line 416, in read_from_socket
    raise ConnectionError(SERVER_CLOSED_CONNECTION_ERROR)
redis.exceptions.ConnectionError: Connection closed by server.

I monkey patched it in my app, but this PR should do something equivalent upstream.
The problem is that I was getting ConnectionError in conn.can_read(timeout=timeout), and there is no code to handle it. parse_response does handle this error, but later, when it calls self._execute().
Because the exception is not handled, the system doesn't recover from the disconnection, on its own.  With the patch, it will recover on its own.

We don't want redis-py to handle ConnectionErrors like this. With this patch any message published between the time that the client was disconnected and the time it was reconnected is silently lost with no notification to the app that anything happened.
If your disconnects are related to connection timeouts (e.g., infrastructure between the client and the server timing out idle connections), use the health_check_interval option included in redis-py 3.3+.
For any other disconnect, I believe the client/app should be A) made aware that there is an issue and B) be able to respond to it in a way that's suitable for the specific app.

I tried setting health_check_interval=10, but it doesn't help.  When I restart the redis server, the app gets into a reconnection loop and never recovers.
It's a fair comment, but I'm confused, the docs say:

PubSub objects remember what channels and patterns they are subscribed to. In the event of a disconnection such as a network error or timeout, the PubSub object will re-subscribe to all prior channels and patterns when reconnecting.

So, I know how to catch the ConnectionError exception.  But how do I tell the PubSub object to "reconnect"?  The reset() method (or its alias close()) forgets the channels I had subscribed.   Should I peek inside the pubsub.connection attribute?  Is it part of public API?...

If you're using health_check_interval you shouldn't have to reconnect manually. I just tested locally using this:
import redis
import time

r = redis.Redis(health_check_interval=10, socket_timeout=2)
p = r.pubsub()
p.subscribe('foo')
while True:
    try:
        print(p.get_message())
    except redis.ConnectionError:
        print('Error...')
    time.sleep(0.5)

While the client code is running, I killed my Redis server. The client starts printing "Error...". Obviously in a real app you'd probably want to do something more meaningful.
When I turn the Redis server back on, the client loop automatically reconnects, though it might take up to check_health_interval seconds to fully reconnect and resubscribe to the channels/patterns it was previously subscribed to.

In my case it doesn't reconnect, even with health_check_interval=10, I'm not sure why.  But, OK, I am happy with the design, at least, and will have to further debug why health_check_interval is not having effect in my code.  Cheers.


Define top and bottom border of Json helper

Is it possible to display the top and bottom line of the helper below

{{{json @root.DeviceInventory}}}

I have wrapped this in a div and define the height and set overflow to auto. if there is more data than is visible in the height defined the bottom border is off the screen and only appears when scrolled and the top border dissappears. Is it possible to have both top and bottom border always visible. I could obviously create my own helper but hoping it might be easy to achieve with the core one.

@asimonok Any suggestions?

@craftzneko we don't know what panel content might look like so we can't handle it. You have to limit max-height of long element. Here is a code example
After render code
context.element.querySelector('pre').classList.add('myElement')

CSS
.myElement {
  max-height: 100px;
}

Although the value of panel height and width can be useful we might pass them into the code. Please let us know if you need them.


Create body tab

since this will be conditionally rendered the next and back buttons should be passed to it so a or 2 functions can select the next or previous tab.


[macOS] Setting the input mode to GLFW_CURSOR_NORMAL has a small delay between when the cursor shows up and when the cursor can actually be moved

Environment

LWJGL version: 3.2.4, also occurs in stable build 3.2.2
LWJGL build #: 5
Java version: 1.8.0_242
Platform: macOS
Module: GLFW

Description

When the input mode is set to GLFW_CURSOR_DISABLED, setting it to GLFW_CURSOR_NORMAL causes the cursor to show immediately, but it remains frozen for a small amount of time (for a little less than a second). The expected behavior is for the cursor to be immediately "moveable" once it shows up.
Steps to reproduce:

Hide the cursor with something like GLFW.glfwSetInputMode(window, GLFW.GLFW_CURSOR, GLFW.GLFW_CURSOR_DISABLED);
While moving the mouse (while the cursor is hidden, so you don't see any movement), turn the input mode back to CURSOR_NORMAL, such as by: GLFW.glfwSetInputMode(window, GLFW.GLFW_CURSOR, GLFW.GLFW_CURSOR_NORMAL);
Once you do, you'll notice that the cursor is still for a moment, before it moves as normal.

For a real world example of this bug, see long-standing Minecraft bug MC-134546. This specific bug report leads me to believe that this bug only affects macOS.
You can reproduce this issue on a Mac with the code below (press SPACE while the window is in focus to toggle the cursor's hidden/normal state):
import org.lwjgl.glfw.GLFW;
import org.lwjgl.opengl.GL;
import org.lwjgl.opengl.GL11C;

import static org.lwjgl.system.MemoryUtil.NULL;

public class MouseTest {
    long window;

    boolean cursorIsFrozen = false;

    public static void main(String[] args) {
        new MouseTest().run();
    }

    public void run() {
        createWindow();
        loop();
    }

    private void createWindow() {
        GLFW.glfwInit();
        window = GLFW.glfwCreateWindow(1000, 300, "Press space to toggle cursor frozen/unfrozen state", NULL, NULL);
        if (window == NULL) {
            throw new RuntimeException("Failed to create window");
        }

        GLFW.glfwSetKeyCallback(window, (window, key, scancode, action, mods) -> {
            if (key == GLFW.GLFW_KEY_SPACE && action == GLFW.GLFW_PRESS)
                toggleCursor();
        });

        GLFW.glfwMakeContextCurrent(window);
        GLFW.glfwShowWindow(window);
    }

    private void toggleCursor() {
        if (cursorIsFrozen) {
            GLFW.glfwSetInputMode(window, GLFW.GLFW_CURSOR, GLFW.GLFW_CURSOR_NORMAL);
            cursorIsFrozen = false;
        } else {
            GLFW.glfwSetInputMode(window, GLFW.GLFW_CURSOR, GLFW.GLFW_CURSOR_DISABLED);
            cursorIsFrozen = true;
        }
    }

    private void loop() {
        GL.createCapabilities();
        while (!(GLFW.glfwWindowShouldClose(window))) {
            GL11C.glClear(GL11C.GL_COLOR_BUFFER_BIT | GL11C.GL_DEPTH_BUFFER_BIT);
            GLFW.glfwSwapBuffers(window);
            GLFW.glfwPollEvents();
        }
    }
}

Hey @Ghalid, thanks for reporting this issue.
I've been able to reproduce the delay and found a fix that eliminates it. I've submitted a PR to GLFW for evaluation. In the meantime, you may download a GLFW build that includes the fix here: https://build.lwjgl.org/nightly/macosx/x64/libglfw.dylib. The next LWJGL 3.3.0 snapshot will also include this build.

Sorry for the delayed response. I can confirm that the issue is fixed when forcing the latest version of Minecraft to use LWJGL 3.3.0 - thank you very much!


It seems that the 0.20.0 package is missing TweenWorld.cs

This is the first time I've asked a issue on github, so please correct me if I've made any mistakes.
It seems that the 0.20.0 package is missing some file ? For example TweenWorld.cs.
Because when I installed it in unity via git url, it reported some errors.

Oh, sorry. I thought I had removed the Experimental feature after v0.2, but it seems I forgot. I just fixed it, so please try installing again

Oh, sorry. I thought I had removed the Experimental feature after v0.2, but it seems I forgot. I just fixed it, so please try installing again

Thanks, there are no installation issues with the package now.


Support policyfiles to replace run lists

Machine resources should support adding configuration and using Policyfiles in the place of run lists.
To support policyfiles now, one has to inject chef_config like this:
machine 'bootstrap-backend' do
  chef_config "use_policyfile true\nversioned_cookbooks true\npolicy_group 'reference'\npolicy_name 'bootstrap-backend'"
end

This works, however it would be nice to do this instead:
machine 'bootstrap-backend' do
  policy_name 'bootstrap-backend'
  policy_group 'reference'
end

The code would then detect that it should use policyfile configuration, and ask the Chef Server it is configured to talk to about whether it supports the policyfile API end point, or if it should use compatibility mode. If the API supports policyfile end point, the following would be appended to the client.rb:
use_policyfile true
policy_document_native_api true

policy_group "#{policy_group}"
policy_name "#{policy_name}"

If the API doesn't support the API, then fall back to compatibility mode:
use_policyfile true
policy_document_native_api false
deployment_group "#{policy_name}-#{policy_group}"

hey @jtimberman i actually put in a PR doing exactly this as far as the resource attributes - #404 .
it doesn't have the detection features you describe above though; gonna look into adding that though, that's a good idea.

@double-z Cool, I left a comment on #404 :cake:.


[aws-cpp-sdk-s3-crt]: propagate client errors instead of swallowing them

Errors produced by the AWS CRT runtime are not translated back into CoreErrors, since the S3CrtClient parent class (Aws::Client::AWSXMLClient) does not recognize them.
This leads to empty error messages, such as
 HTTP response code: 0
 Resolved remote host IP address:
 Request ID:
 Exception name:
 Error message:
 0 response headers:

Fix this by extracting the corresponding message, and marking the request as failed (with a client error of type "Internal Failure").
Check all that applies:

[X] Did a review by yourself.
[X] Added proper tests to cover this PR. (If tests are not applicable, explain.)
This change has enabled us to troubleshoot s3 problems that previously were reported as "No response body".
[X] Checked if this PR is a breaking (APIs have been changed) change.
[X] Checked if this PR will not introduce cross-platform inconsistent behavior.
[X] Checked if this PR would require a ReadMe/Wiki update.

Check which platforms you have built SDK on to verify the correctness of this PR.

[X] Linux
[ ] Windows
[ ] Android
[ ] MacOS
[ ] IOS
[ ] Other Platforms

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Closing PR. Changes merged here


fix: cant jump to sign in page after registration

Signed-off-by: wasabi<EMAIL_ADDRESS>

In order not to conflict with the previous pr demo, I created another app(application_1), to fix the bug raised by users.
demo sign up link: http://test.rescld.cn/signup/application_1
Log in with your account after registration.
Management background: http://test.rescld.cn
admin,123

One more user belonging to this app(application_1) is added to the user table.


I created a Dependency Check Gradle plugin

Hi,
The Dependency Check is really useful, we can get lots of benefits from it, but I found that it doesn't have gradle plugin, although I can work around this, I think a dedicated gradle plugin is a better choice.
Hence, I create this Dependency Check gradle plugin, it utilise the functions in dependency-check-core module, it will verify project dependencies, update data feed, analyse each dependency and generate report.
For now, this plugin is just a MVP, more development work is required to add more functionalities, but at least, it works for gradle project.

Thank you for the PR!  I will absolutely be merging this - but it will likely not make it into the next release as I am doing final verification right now.

My pleasure, I know there are lots of development work required, it's not possible to release it recently, I will keep work on this, make it configurable, performance improvement, error handling, etc.
Thanks a lot :)

Hi Jeremy, since the 1.2.11 just got released, would you please approve this pull request?
Best Regards.

Thanks! The PR has been merged. However, could you please go through and add the legalese(Apache 2.0), including the copyright Wei Mai or Thoughtworks...  I noticed in the pom.xml for the gradle plugin it still has my name in the copyright.
--Jeremy

Sure, will do that, thank you very much.


Restore VOI on MPR reset

Cache original VOI and restore when reset viewport
Related issue: #1354

@JamesAPetts we still need this one? can i assign someone to review/merge?


Typo?

Hi there,
Should this line:
Metrics. Application Gateway currently has seven metrics to view performance counters.
actually read:
Metrics. Azure Front Door currently has seven metrics to view performance counters.
Thanks
Russ

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

ID: 62f7409d-5cbe-20d0-e37a-891981ecac97
Version Independent ID: ad725093-ebd3-ecfc-84c8-e7c19a715979
Content: Monitoring metrics and logs in Azure Front Door Service
Content Source: articles/frontdoor/front-door-diagnostics.md
Service: frontdoor
GitHub Login: @sharad4u
Microsoft Alias: sharadag

@russellmccloy, Thanks for your feedback. We are looking into this query and will update you as soon as possible.

@russellmccloy, Appreciate your patience. The doc was edited accordingly and it will take some time to get reflected.
If there are further questions regarding this matter, please reply and we will gladly continue the discussion.


feat(Card): theme tokens

Github Issue or Trello Card
This PR addresses this issue: https://trello.com/c/OC9hgghM/1157-component-tokens
Adds component design tokens to the Card component for theming.
'--card-background-color': '--color-brand-white-500',
'--card-border-radius': '--size-border-radius-md',
'--card-box-shadow': '--size-box-shadow-sm',
'--card-section-border-color': '--color-brand-grey-100',
'--card-footer-background-color': '--color-brand-grey-50',

What type of change is this?
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Updating documentation.
[ ] Updating deployment/build pipeline.
[ ] Updating Project Dependencies
[ ] Improving or adding to Test Coverage

Completeness Checklist

[ ] TESTS: My changes maintain the baseline required test coverage, as specified by code climate analysis.
[ ] DOCS: All new component work is covered in that component's Overview docs.
[ ] DOCS: All new component work is covered in a component Playground.
[ ] DOCS: All new visual changes or options are covered under relevant components' VisualTests.
[ ] My code has no linting or typescript compile warnings.
[ ] My work is tied to a Github issue and satisfies the acceptance criteria (if applicable) of the corresponding issue.

Quality Checklist:

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes

UI Checklist

[ ] I have conducted visual UAT on my changes/features.
[ ] My solution works well on desktop, tablet, and mobile browsers.

 Component Tokens


LICENSE file not found

While the comment say
// Copyright 2014 Joseph Hager. All rights reserved.
// Use of this source code is governed by a BSD-style
// license that can be found in the LICENSE file.

, LICENSE file is not present.

/cc @ajhager Can you help resolve this issue?


Upgrade to Jenkins 2.2

Trying to keep the Docker image in sync with the official releases.
What is the intended cadence for upgrade the Docker image? Should the Docker image be in sync with the official download version?

Side note: Confusing to have two "official" Docker images. I assume "official" is relative to the image maintainer rather than to the product?

https://hub.docker.com/r/jenkinsci/jenkins/ Jenkins Continuous Integration and Delivery server
https://hub.docker.com/_/jenkins/ Official Jenkins Docker image

+1
Jenkins should also have an automatic build. The Dockerfile already exists, it's easy to integrate Docker Hub with a Github Repository.
https://docs.docker.com/docker-hub/builds/

This is based on the Long Term Support release
For weekly releases check out jenkinsci/jenkins, automatically built


Cloud Speech-to-Text API C++ (beta version)

Hi,
we want to replace the c++ STT libraries of previous vendor with google stt client library.
I have like below questions:-

Are the Cloud Speech-to-Text API C++ libraries are  comerciallly stable?
Cloud Speech-to-Text API C++ i see at few places as being referd as experminetal or beta version?
does it permorfm STT on AMR-WB codes?

Are the Cloud Speech-to-Text API C++ libraries commercially stable?

Yes, the C++ client library is commercially stable...

Cloud Speech-to-Text API C++ I see at few places as being referred as experimental or beta version?

...but as you point out it, the library contains some experimental APIs.
Experimental Speech-to-Text v2 client:
The library has a google::cloud::speech_v2::SpeechClient, which is experimental, because the service, Cloud Speech-to-Text v2, is pre-GA (not commercially stable).
I recommend using the v1 service, unless you need some feature that is only available in v2. The client for the v1 service is: google::cloud::speech::SpeechClient (which is used in the quickstart and samples).
Experimental AsyncStreamingRecognize (bidirectional streaming RPC):
The v1 client has an API that is marked as experimental.
https://github.com/googleapis/google-cloud-cpp/blob/94285ba95685b3ad277fd094bfdae15f0e43fe40/google/cloud/speech/speech_client.h#L202-L205
It is marked as experimental, because the API is a little tricky to use. We do have samples that demonstrate how to use the API. (The streaming_transcribe*.cc programs show how to properly call AsyncStreamingRecognize): https://github.com/GoogleCloudPlatform/cpp-samples/tree/main/speech/api

does it perform STT on AMR-WB codes?

According to these docs, yes. https://cloud.google.com/speech-to-text/docs/encoding#audio-encodings

Seems like this is answered. If there is are follow up questions please reopen or create a discussion item.