Tax Indexer: Customer and Product tax class ids are registered wrongly

https://mysite.local.com/vue_storefront_magento/taxrule/_search?pretty

M2 backend:

As you see indexer nicely picking up every information except Customer and Product Tax classes.
The related indexer part it falling back to https://github.com/DivanteLtd/magento2-vsbridge-indexer/blob/main/src/module-vsbridge-indexer-tax/ResourceModel/TaxClasses.php#L39, and according to fetchAssoc($select):

Fetches all SQL result rows as an associative array.
The first column is the key, the entire row array is the value.
You should construct the query to be sure that the first column contains unique values,
or else rows with duplicate values in the first column will overwrite previous data.

Running the row query directly on DB results in 42 rows, but during debugging (since the limitation of the fetchAssoc) only can see 4 records.

By looking how data meant to be processed in ../DataProvider/TaxClasses.php, fetchAll should be used to get records instead a fetchAssoc. I will make  a fix.


Mobile first

Merging in final changes made to website into dev branch. Responsiveness had been added for all pages, pictures have been added, docs page has been converted to formatted text, and team page is updated with pictures and working links.

Changes looks fine, approving the same.


Doxygen doc: fix documentation tags for enum.

Just a small bugfix: use ///< instead of ///>.

Whoops, thanks. FYI I'm not happy yet with the state of autodocs in Oryol (mostly because I'm not very happy with doxygen). I might try out other systems some time (like Sphynx).


Adding try catch for local storage and initialize entry point

try-catch for localStorage
try-catch for the initialize entry point

@MaxShehovtsov @olegsych @OsvaldoRosado

:shipit:

:+1:

:sheep: ed


Docs: Fixing truncated code in numbered lists

Issue #, if available: Issue #1717: code fenced areas in docs getting clipped/truncated
Description of changes: Long code blocks inside of numbered lists in the Install with kubeadm (bare metal or VM) instructions were being truncated. They were also missing the "copy to clipboard" icon. This PR seeks to fix those things.

/approved

/approve

Fixed by https://github.com/aws/eks-distro/pull/1727


Support $rename field update operator

Closes #626.

@fenogentov I will do a real review later, but please golangci-lint errors in the meantime. We enabled wsl linter for the new code.

@fenogentov Hi! Was it intentional, or you force-pushed by mistake?

@fenogentov Hi! Was it intentional, or you force-pushed by mistake?

I do not understand what happened
I didn't press anything


Change geojson to shapefile

Closes #1931

@mattbowen-usds - Awesome work!!!
What a pleasant surprise to see a FE PR!
Quick question, I downloaded the link, and it wasn't obvious to me what a "codebook" looks like. The downloaded zip does have the shapefiles. Is the codebook called "columns.csv"?

@vim-usds  It is! To be honest, I'm not super familiar with that world either, but columns.csv is what I think of as the codebook --- it tells you what each of the short names actually expands to in human.


Strikethrough formatting not exported

Text with strikethrough formatting is not exported to markdown
For example, this text results in unadorned text in the output

example

becomes

example

Suggest using the GFM format for strikethrough export
~~example~~

Fixing this was fairly easy, but my testing of the fix pointed out another problem.
There doesn't seem to be a consistent way to force a line break in text.  The code
currently adds a backslash at the end of a line to force a line break because that
works with GitHub Flavored Markdown, but it doesn't work with some others.

Thanks, I'll keep an eye out for any unwanted line break behavior. Haven't
noticed any yet.
On Mon, Oct 14, 2019 at 12:22 PM Bill Shannon<EMAIL_ADDRESS>wrote:

Fixing this was fairly easy, but my testing of the fix pointed out another
problem.
There doesn't seem to be a consistent way to force a line break in text.
The code
currently adds a backslash at the end of a line to force a line break
because that
works with GitHub Flavored Markdown, but it doesn't work with some others.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/bshannon/savenotes/issues/3?email_source=notifications&email_token=AABT24TR3HPZMLQ7KFIGDFDQOTBIFA5CNFSM4IXGNI6KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEBGEU4Y#issuecomment-541870707,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABT24UZFJE7W3LZHUAZSMLQOTBIFANCNFSM4IXGNI6A
.


Code in Example 2 is missing a $ special character meant for string interpolation

Page
https://docs.microsoft.com/en-us/dotnet/csharp/programming-guide/classes-and-structs/instance-constructors
Error Description
Following code under Example 2 of above page is missing a $ special character meant for string interpolation.
Console.WriteLine("Name: {person.name}, Age: {person.age}");
Above line should be following:
Console.WriteLine($"Name: {person.name}, Age: {person.age}");

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

ID: adbf6206-5332-f844-ba6a-66b58090f686
Version Independent ID: 93775018-00ba-2c71-e9e8-da1792f35f9b
Content: Instance Constructors - C# Programming Guide
Content Source: docs/csharp/programming-guide/classes-and-structs/instance-constructors.md
Product: dotnet-csharp
Technology: csharp-fundamentals
GitHub Login: @BillWagner
Microsoft Alias: wiwagn

Thanks for noting this @nccsbim071
I've added it to our backlog to fix on our next update for this article.
I've also added the "up for grabs" label in case a community member wants to submit a fix before we update it.

Thanks for noting this @nccsbim071
I've added it to our backlog to fix on our next update for this article.
I've also added the "up for grabs" label in case a community member wants to submit a fix before we update it.


Which executable/binary

"Make sure you have your .env file in the same directory as the executable/binary.."
Which does "executable/binary" refer to?

I had exactly the same question.
Suggestions:

The start.sh script  refers to a "specific" directory which should be given as an argument instead.
The startup.sh would be more useful.

Go is not my standard language on my server so I had to install it on the Server.
https://golang.org/doc/install (might want to put the link in there).
Suggestion: "Requires go 1.8 or above"  to rename it to "Requires the Go Programming language 1.8"
After this should I copy the content of the repository to a folder on ?????
/usr/local/FactoCord
Missing step what to do with go build? I get the "no Go file in /home" found. probably requires a specific folder or just the case of not knowing where.

Thanks for making the software so far, trying tomorrow night again to get the bot running. 👍

You shouldn't need to install Go on the server if you use the precompiled version. If you don't trust that, I would recommend building it for linux locally, then uploading it to the server.
I don't think this change is necessary.
Typically I'll place the entire contents of FactoCord zip in a "server" folder within the home directory.
See point 1
start.sh uses the current directory. The FactoCord executable should be in the same folder as start.sh


-I/--interactive does not work

I want to enable subscriptions but -I/--interactive does not work

With npm create
yarn create works fine

"Does not work" means nothing happens or you get an error?
Could you provide some more information, like the type of project you're using and the output of the command?

A fresh TypeScript CRA Template - it ignores the --interactive flag and proceeds without stopping for input

Interesting. npm create requires you to pass -- before specifying the command arguments:
npm create @tobiastengler/relay-app -- --interactive

I wasn't aware of this either. I'll update the documentation. Thanks for the find :)

I added a quick reminder here.


[Feature Request] Documentation on invoking tt_metal compiler

Is your feature request related to a problem? Please describe.
There doesn't appear to be any documentation on how one invokes the tt_metal compilation flow.  We are not interested in using any of your examples or any of your included CMake scripts.
Describe the solution you'd like
We would like to see a basic Makefile that resides in a separate source tree that invokes a simple example
Describe alternatives you've considered
Since there are no functional examples, there are no alternatives.
Additional context
I think this request is relatively clear

We currently don't have a separate repo that specifically shows an example using Make and tt-metal as a dependency. From the context of your issue, I'm going to assume you're talking specifically about the Metalium C++ runtime API + not too interested in the ML Python API (ttnn).
We are trying to actively make this better. We do have longer-term plans to release the Metalium runtime headers and shared object in various distribution forms such as Conda and RPM. We can also create example repos as you describe for not only Make, but also other build systems like CMake. We can update this issue as we progress.
This is an example from the community by @marty1885 : https://github.com/marty1885/ttVecAdd

Related issue: #7914


Corrected the get_started.md example inference

It did not contain the checkpoint download step.

 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.

 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.

Hi @kazet ,
Thanks for your kind contribution. Could you sign the CLA thus we could merge this PR?

Hi @kazet ,
Thanks for your kind contribution. Could you sign the CLA thus we could merge this PR?

Hi!
@kazet
First of all, we want to express our gratitude for your significant PR in the OpenMMLab project. Your contribution is highly appreciated, and we are grateful for your efforts in helping improve this open-source project during your personal time. We believe that many developers will benefit from your PR.
We would also like to invite you to join our Special Interest Group (SIG) private channel on Discord, where you can share your experiences, ideas, and build connections with like-minded peers. To join the SIG channel, simply message moderator— OpenMMLab on Discord or briefly share your open-source contributions in the #introductions channel and we will assist you. Look forward to seeing you there! Join us ：https://discord.gg/UjgXkPWNqA
If you have WeChat account，welcome to join our community on WeChat. You can add our assistant ：openmmlabwx. Please add "mmsig + Github ID" as a remark when adding friends：）
Thank you again for your contribution❤


A skip attribute macro

This is the best and most un-derated builder crate. Thank you.
Use case:
#[derive(Serialize)]
#[serde(rename_all = "camelCase")]
pub struct ListInstrumentsRequest<'a> {
    #[serde(skip)]
    accounts: &'a Accounts<'a>,
    /// List of instruments to query specifically.
    instruments: Vec<String>,
}

I have more than just stuff to build in the struct so when it comes time to build I can use it to make an REST request.
I want the builder to ignore accounts in the example above.

How would the builder build a ListInstrumentsRequest without the accounts field?

I imagined:
fn start_request() -> ListInstrumentsRequest<'a>::Builder {
  ListInstrumentsRequest{ accounts: self, ..Default().. }.builder()
}

async fn use() -> ... {
    start_request().instruments(..).build().send().await
}

How is that any different from this?
fn start_request(&self) -> FinalStageBuilder {
    ListInstrumentsRequest::builder().accounts(self)
}

I can't seem to do that because of my lifetime constraint on the builder, but honestly, it's just my bad coding.
I  withdraw my request. Thank you for your time and all the best to you.


[Fabric][Figura0.1.5-rc6]When using these avatars in Figura0.1.5-rc6, the ears of the avatars are missing.

Strangely, Sora's ears didn't disappear.🤔

Thanks for using my avatar and reporting the issue!
This issue has been confirmed in my environment. I will identify the cause.

@OvOla2
This issue was fixed in avatar version v1.25.9. Please update your avatar to the latest.
Thanks for reporting this issue!


A problem encountered while processing Spider data

I have already finished "./setup" and download Spider, but found a problem:
(tensor2struct) fsx@FoolishCow:~/model/tensor2struct$ tensor2struct preprocess configs/spider/run_config/run_en_spider_bert_baseline.jsonnet
Traceback (most recent call last):
  File "/home/fsx/anaconda3/envs/tensor2struct/bin/tensor2struct", line 33, in <module>
    sys.exit(load_entry_point('tensor2struct', 'console_scripts', 'tensor2struct')())
  File "/home/fsx/anaconda3/envs/tensor2struct/bin/tensor2struct", line 22, in importlib_load_entry_point
    for entry_point in distribution(dist_name).entry_points
  File "/home/fsx/anaconda3/envs/tensor2struct/lib/python3.7/site-packages/importlib_metadata/__init__.py", line 954, in distribution
    return Distribution.from_name(distribution_name)
  File "/home/fsx/anaconda3/envs/tensor2struct/lib/python3.7/site-packages/importlib_metadata/__init__.py", line 542, in from_name
    raise PackageNotFoundError(name)
importlib_metadata.PackageNotFoundError: No package metadata was found for tensor2struct

thanks for your help, but I didn't use colab  and https://github.com/pytorch/fairseq/issues/2546 cannot fix it.

I guess it's probably a software version issue? As a reference, below is the versions that I'm using:
pip 20.2.3
Python 3.7.0

I'll try it ,thanks.


Guide explaining and showing how to avoid field "shadowing" in Dagger CUE

What is the issue?
Users often use field names like source and config that are the same as fields inside of definitions. Users aren't sure what's going on:

Whenever you're using the same name as key and value, there's a potential for some weird assignment error I don't quite understand.

This can create an issue...
_image: docker.#Dockerfile & {
    source: source
}

that can be fixed like this:
_image: docker.#Dockerfile & {
    "source": source
}

Demo in pure CUElang:
https://cuelang.org/play/?id=M_GjoAgHlmC#cue@export@cue
Explanations by Dagger staff that I've found:
(for above example)

For values at the same scope, sometimes Cue has a hard time knowing whether you mean the other value or the key inside a Definition. Putting quotes around the key when it has the same name as the value, you'll make sure Cue doesn't get confused

//
problem:
   #folders: ["kube2iam", "postgresql"]
        lint_all:
        {
            for folder in #folders {
                #Lint & {
                    source: client.filesystem."./".read.contents
                    folder: folder
                }
            }
        }

fix:
#folders: ["kube2iam", "postgresql"]
       lint_all: {
           for folder in #folders {
               "\(folder)": #Lint & { // need a field name otherwise they'll get embedded
                   source: client.filesystem."./".read.contents
                   "folder": folder // quote field to avoid shadowing
               }
           }
       }

User response:

thank you. what is shadowing?

Staff followup:

This is when there's a conflict between scoped variables. If you have

input: "hello"
f: input: "world" // input is in scope so instead of the name "input", you end up with a conflict
f: "input": "world" // solution is to quote the field so it doesn't get interpreted as a variable

Outside explanation:

https://cuetorials.com/overview/scope-and-visibility/#reference-lookup

https://github.com/dagger/dagger/issues/2180


proper fix for DER value compatibility issue

Signed-off-by: Mohamad Chaarawi<EMAIL_ADDRESS>

sigh i messed this PR up. i'll recreate


How to use with gin?

Any way to use this with the gin framework? See issue here

https://godoc.org/github.com/gin-gonic/gin#WrapH should do the trick - you want to call gin.WrapH(nosurf.NewPure()).

NewPure expects an argument.

Closing after inactivity.

I am having the same issue. Any ideas?
func NewPure(handler http.Handler) http.Handler {

Doesn't work with gin framework

I know it's 2024 but...nosurf seems to be the only library doesn't require session but cookie only.
func APIReleaseRouteInstaller(r gin.IRouter) {
	// r could be gin.Engine
	g := r.Group("/api", CsrfMiddleware(true))
	// code continues...
}

func CsrfMiddleware(secure bool) gin.HandlerFunc {
	return WrapHH(func(h http.Handler) http.Handler {
		n := nosurf.New(h)
		n.SetBaseCookie(http.Cookie{Name: "csrf_token", HttpOnly: true, Secure: secure})
		return n
	})
}

type wrappedResponseWriter struct {
	gin.ResponseWriter
	writer http.ResponseWriter
}

func (w *wrappedResponseWriter) Write(data []byte) (int, error) {
	return w.writer.Write(data)
}

func (w *wrappedResponseWriter) WriteString(s string) (n int, err error) {
	return w.writer.Write([]byte(s))
}

type NextRequestHandler struct {
	c *gin.Context
}

// Run the next request in the middleware chain and return
// See: https://godoc.org/github.com/gin-gonic/gin#Context.Next
func (h *NextRequestHandler) ServeHTTP(w http.ResponseWriter, r *http.Request) {
	h.c.Writer = &wrappedResponseWriter{h.c.Writer, w}
	h.c.Request = r
	h.c.Next()
}

// Wrap something that accepts an http.Handler, returns an http.Handler
func WrapHH(hh func(h http.Handler) http.Handler) gin.HandlerFunc {
	// Steps:
	// - create an http handler to pass `hh`
	// - call `hh` with the http handler, which returns a function
	// - call the ServeHTTP method of the resulting function to run the rest of the middleware chain

	return func(c *gin.Context) {
		hh(&NextRequestHandler{c}).ServeHTTP(c.Writer, c.Request)
		if c.Writer.Written() {
			// If the response has been written, abort the rest of the middleware chain
			c.Abort()
		}
	}
}

By the way, https://github.com/turtlemonvh/gin-wraphh has two major problem

it does not do c.Abort(). Even if token failed to validate, the handler chain is not stopped which defeat the purpose.
missing h.c.Request = r. nosurf replaced the request with key embedded in context value, and such the original request must be replaced. Otherwise there is no way to get the token out in the handler and place it in HTML tag or alike.


webgpu: add a large matmul test

Passes on both webgl and webgpu backends, so apparently they match.

This change is

@kainino0x wouldn't 32x32 be enough to test >1 tile?

Right now we have 16x16 tiles and WPT=2, which means 33x33 would be enough. But if we raise WPT to 4 or tile size to 32 on hardware that supports it, 65x65 would be necessary.


Feature/contrast color pie chart

Description
Pie chart contrast color for chart portion, if chart color selected as single color.
Issues Resolved
Change portion border color for single dark color.
Check List

[ ] All tests pass, including unit test, integration test and doctest
[ ] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Creating single common PR for pie chart, hence closing this PR.


tf.control_dependencies() not work in a custom_gradient function

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): Y
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Ubuntu 16.04
TensorFlow installed from (source or binary): binary
TensorFlow version (use command below): r1.8
Python version: 2.7.14
GCC/Compiler version (if compiling from source): 5.4
CUDA/cuDNN version: 8.0/7.0
GPU model and memory: GTX1080, 8G
Bazel version: N/A
Exact command to reproduce: python test_script.py

Describe the problem
The output for the code below, is not correct. The first value of variable "assign_var" is not assigned with 2. The expected output for the 2nd line is: [2 0]. Thus, the approach [here][1] cannot work with<EMAIL_ADDRESS>I don't know if this is a bug or there is a solution for this. Thanks in advance!
Source code / logs
@tf.custom_gradient
def test_func(x):
i = tf.constant(0)
id_refs = tf.Variable(tf.zeros((1), tf.int32))
assign_var = tf.Variable(tf.zeros((2), tf.int32))
cond = lambda i, id_refs: tf.less(i, 8)
def _body(i, id_refs):
  a = tf.assign(assign_var[0], 2)
  id_refs = tf.concat([id_refs, [i]], 0)
  with tf.control_dependencies([a]):
    return i+1, id_refs

_, gradRefs = tf.while_loop(cond, _body, [i, id_refs], shape_invariants= \
  [i.get_shape(), tf.TensorShape([None])])
def grad(dy):
    return tf.constant(0)
return assign_var, grad

x = tf.constant(100.)
y = test_func(x)
dy = tf.gradients(y, x)
init_op = tf.global_variables_initializer()
with tf.Session() as sess:
sess.run(init_op)
print(sess.run(dy))
print(sess.run(y))
Output:
[0]
[0 0]

Nagging Assignee @rohan100jain: It has been 16 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @rohan100jain: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @rohan100jain: It has been 47 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @rohan100jain: It has been 62 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @rohan100jain: It has been 77 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @skye: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @skye: It has been 35 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @skye: It has been 50 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @skye: It has been 65 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @skye: It has been 80 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @skye: It has been 95 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @skye: It has been 110 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @skye: It has been 125 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

@YantianZha,
As you had the issue in older version of tensorflow(1.x) which is officially considered as end of life. We recommend that you upgrade to latest stable version and let us know if the issue still persists in newer versions.we will get you the right help.Thanks!


Allow removing default classes from table element

Is there any way to remove classes from the table element? I would like to remove the table-striped class, but I don't see how.
The tblClass prop let's me add classes, but how I can remove some of the ones that are being added automatically?

According to https://github.com/OneWayTech/vue2-datatable/blob/master/src/Table/TableFrame.vue#L2
table-striped is hard coded...
so... well... how about:
$('.table-striped').removeClass('table-striped')


Background color of NavBar and the size of logo of Moody is up to the mark

Description
The background color of NavBar is not up to the mark.
The logo size is also large.
We should create a small-sized attractive logo or we can resize the existing one.
Update the NavBar background color and logo so that the website looks cool.
Screenshots

Additional information
No response

@arpitghura Someone is already working on that issue!


Expected all tensors to be on the same device, but found at least two devices, cuda:0 and cpu!

Hi there,
Thanks for your excellent work. I have this problem when I train and test your code. Do you have any idea what is wrong? Since I find that the data and model are all in cuda.
Thanks in advance!

I solved that through transferring it into toch.nn.parallel.DistributedDataParallel.
However, I met another CUDA Memory Error:
torch.cuda.OutOfMemoryError: CUDA out of memory. Tried to allocate 50.00 MiB (GPU 0; 11.91 GiB total capacity; 10.99 GiB already allocated; 3.88 MiB free; 11.07 GiB reserved in total by PyTorch) If reserved memory is >> allocated memory try setting max_split_size_mb to avoid fragmentation.  See documentation for Memory Management and PYTORCH_CUDA_ALLOC_CONF

I think you have to reduce the batch size, even though I have  2 X2080ti , batch-size set as 2

I think you have to reduce the batch size, even though I have 2 X2080ti , batch-size set as 2

My GPU is a Titan Xp with 12GB of memory, and the image size is 576*576, but I still get a "out of memory" error even when I set the batch size to 1.

I am facing it, can you share solution? @afpapqy @landiaokafeiyan

I modified little bit and I can run without device error
in models/swin_transformer_v2.py line 294
original: logit_scale = torch.clamp(self.logit_scale, max=torch.log(torch.tensor(1. / 0.01))).exp()
modified: logit_scale = torch.clamp(self.logit_scale, max=torch.log(torch.tensor(1. / 0.01).to('cuda:0'))).exp()
this is an example. You can get another variable to change the tensor's device status.

Hi @afpapqy @PigBroA
when I test the image with 3000x4000, I have to piece the image into several patches which will decrease the performance. Do you have any good ideas to slove this problem?
Thanks in advance.

Thank you for this solution!
For the multi-GPU environment, I encountered another error with "cuda:0" and "cuda:1" and alternatively, I used the following modification:
original: logit_scale = torch.clamp(self.logit_scale, max=torch.log(torch.tensor(1. / 0.01))).exp()
modified: logit_scale = torch.clamp(self.logit_scale, max=torch.log(torch.tensor(1. / 0.01).to(self.logit_scale.device))).exp()

I modified little bit and I can run without device error
in models/swin_transformer_v2.py line 294 original: logit_scale = torch.clamp(self.logit_scale, max=torch.log(torch.tensor(1. / 0.01))).exp() modified: logit_scale = torch.clamp(self.logit_scale, max=torch.log(torch.tensor(1. / 0.01).to('cuda:0'))).exp()
this is an example. You can get another variable to change the tensor's device status.

Thank you for this solution!
For the multi-GPU environment, I encountered another error with "cuda:0" and "cuda:1" and alternatively, I used the following modification:
original: logit_scale = torch.clamp(self.logit_scale, max=torch.log(torch.tensor(1. / 0.01))).exp()
modified: logit_scale = torch.clamp(self.logit_scale, max=torch.log(torch.tensor(1. / 0.01).to(self.logit_scale.device))).exp()


Fix add workflow summary screen

Follow up to #10.
Fix the display of the group names on the Add workflow summary screen wizard.

@Hyperkid123  - I broke the summary screen when making the changes to use the Object keys - please review

Codecov Report

Merging #14 into master will not change coverage.
The diff coverage is 0%.

@@           Coverage Diff           @@
##           master      #14   +/-   ##
=======================================
  Coverage   22.32%   22.32%
=======================================
  Files          40       40
  Lines         336      336
  Branches       33       33
=======================================
  Hits           75       75
  Misses        238      238
  Partials       23       23

Impacted Files
Coverage Δ

src/smart-components/workflow/summary_content.js
0% <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 4483b07...14acce8. Read the comment docs.


Combining aggregate functions and valueExpressions

As far as I am aware it is not currently possible to combine aggregate functions and valueExpressions, which would be really handy, for example:
t.select(t.a.sum().plus(t.b.sum()))
//Should result in:
//'SELECT (SUM(`t`.`a`) + SUM(`t`.`b`)) FROM `t`'

t.select(t.a.plus(t.b).sum())
//Should result in:
//'SELECT SUM((`t`.`a`) + (`t`.`b`)) FROM `t`'

I think this could possibly be achieved by implementing aggregate functions as a valueExpression instead of Column.prototype.aggregate, however that would allow invalid queries such as:
t.select(t.a.sum().sum()))
//'SELECT SUM(SUM(`t`.`a`)) FROM `t`'
// MySQL error: #1111 - Invalid use of group function

There could be issues with removing the alias from child aggregate nodes of a valueExpression and choosing a default alias for aggregate nodes with value expressions as children.

Hi, this is somewhat old, but I have the same problem. I need to select something like
t.columnA.multiply(t.columnB).sum()
and I get a "TypeError: db.inventoryT[valorizer].multiply(...).sum is not a function" Exception.
Is there any workaround?

I'm running into the same problem trying to cast a count(*) to an int (because of https://github.com/brianc/node-postgres/issues/378).
My work around was to directly call the CastNode, but that's kind of ugly:
const SqlCast = require('sql/lib/node/cast');
const sql = Table.select(new SqlCast(Table.star().count()), 'int');

The cause seems to be that Column.aggregate returns a ColumnNode which does not include the valueExpressionMixin().  Not sure why.


Clamby showing false positives when scanning from attachment temp file location

I'm not certain if this is a repeat of https://github.com/kobaltz/clamby/issues/9
I configured Clamby with ClamAV in a local environment, and didn't experience any issues.
However, when attempting to configure Clamby and ClamAV in a production environment, I'm experiencing an issue where the Clamby#safe? method returns false for all files and Clamby#virus? method returns true for all files.
My production environment is running Ubuntu 16.05.5 LTS, and ClamAV, clamd, and freshclam have been installed and configured properly, to my knowledge.
irb(main):001:0> Clamby::Command.clamscan_version
ClamAV 0.102.1
=> true

$ ps aux | grep clamd
clamav   26428  3.3 21.3 1073956 864960 ?      Ssl  16:06   1:13 /usr/sbin/clamd --foreground=true

Per ClamAV documentation, clamd is running under the "clamav" user. I'm not sure if this is how the daemon should be configured to work with Clamby or not.
/config/initializers/clamby.rb

Clamby.configure({
  :check => false,
  :daemonize => true,
  :error_clamscan_missing => false,
  :error_file_missing => false,
  :error_file_virus => false,
  :fdpass => true
})

When I check any file, even files contained in my Rails application, Clamby returns that all are viruses:
Example:

irb(main):001:0> test_path = "#{Rails.root}/README.md"
=> "/home/deploy/rails_app/README.md"
irb(main):002:0> Clamby.safe?(test_path)
=> false
irb(main):003:0> Clamby.virus?(test_path)
=> true

Additionally, clamd.conf and freshclam.conf have been configured properly:
clamd.conf
# Comment or remove the line below.
# Example
...
# TCP port address.
# Default: no
TCPSocket 3310

freshclam.conf
# Comment or remove the line below.
# Example

freshclam is also properly configured and updated:
$ freshclam
ClamAV update process started at Thu Dec 12 16:50:23 2019
daily.cld database is up to date (version: 25661, sigs: 2046351, f-level: 63, builder: raynman)
main.cvd database is up to date (version: 59, sigs: 4564902, f-level: 60, builder: sigmgr)
bytecode.cvd database is up to date (version: 331, sigs: 94, f-level: 63, builder: anvilleg)

Any help would be appreciated, as I'm not sure what's going wrong here between the various components involved.
Thanks.

Just to provide some additional information, it appears this issue has something to do with the clamd daemon, as Clamby works just fine with daemonize: false in the configuration, although the check takes a little too long to be a viable solution.
irb(main):001:0> Clamby.config
=> {:check=>true, :daemonize=>false, :config_file=>nil, :error_clamscan_missing=>false, :error_clamscan_client_error=>false, :error_file_missing=>false, :error_file_virus=>false, :fdpass=>true, :stream=>false, :output_level=>"medium", :executable_path_clamscan=>"clamscan", :executable_path_clamdscan=>"clamdscan", :executable_path_freshclam=>"freshclam"}
irb(main):002:0> path = "#{Rails.root}/README.md"
=> "/home/deploy/rails_app/README.md"
irb(main):003:0> Clamby.virus?(path)
ClamAV 0.102.1/25661/Thu Dec 12 09:47:42 2019
/home/deploy/rails_app/README.md: OK
=> false
irb(main):004:0> Clamby.safe?(path)
ClamAV 0.102.1/25661/Thu Dec 12 09:47:42 2019
/home/deploy/rails_app/README.md: OK
=> true

Interesting development as I continue working on this.
Updating clamd.conf to comment in TCPAddr <IP_ADDRESS> and restarting the clamd daemon actually allowed Clamby to work while testing files in the Rails console.
However, uploaded files appear to still always return as a virus.
I'm scanning the file on upload from the attachment #tempfile#path location:
attachment = params[:attachment]
path = attachment.tempfile.path
Clamby.virus?(path)
=> true

One final note: I was still experiencing the above issue where scanning the file from the Rails default upload temp file location still always results in Clamby showing all files as viruses.
However, taking the file and writing it to some other location and then scanning it will render accurate results according to the tests I've done recently.
As examples, the following were both done with a file that is known to not be a virus:
The following doesn't appear to work:
attachment = params[:attachment]
path = attachment.tempfile.path
Clamby.virus?(path)
=> true

In the above example, all files return as true when #virus?(path) is called.
However, this solution does work:
attachment = params[:attachment]
path = "#{Rails.root}/attachment.original_filename"
File.open(path, "wb") { |file| file.write(attachment.read) }
Clamby.virus?(path)
=> false

In this example, the file after being saved was correctly identified as safe.

This issue has changed, so I've changed the name accordingly.
For my purposes, I've corrected the initial issue and I've gotten everything working in my production environment, but I've kept this open, as I'm not sure if this is intended behavior or not. Clamby is still showing false positives for me when scanning from attachment temp location when a form is submitted.
Feel free to close this issue if this is intended behavior.

@kobaltz I have noticed the same behavior as @gregorybilello. I'm using ActiveStorage.

Thanks @brotherjack I'll look into this a bit

I'm seeing it with CarrierWave also. I'm putting in the workaround, but hopefully that info will help you.


Issue with genType.as and generics in TypeScript.

Thanks so much for the awesome library @cristianoc, truly a great piece of engineering!
I noticed that genType seems to struggle a bit when using<EMAIL_ADDRESS>on a type that, in Reason, takes a type parameter and when converted to TypeScript should be a generic.
For example, if I do:
<EMAIL_ADDRESS>"Vector"]
type vector('a) = ('a, 'a);

I expect to get:
export type Vector<A> = [A, A];

But instead, I get:
export type vector<a> = [a, a];
export type Vector = vector;

Along with the error: Generic type 'vector' requires 1 type argument(s). on that second line. The generic is not actually getting bubbled up to the renaming.
I may just be missing some piece of documentation around how to properly do this. I did also try:
<EMAIL_ADDRESS>"Vector<A>"]
type vector('a) = ('a, 'a);

But this still doesn't quite work:
export type vector<a> = [a, a];
// A doesn't get passed on to vector
export type Vector<A> = vector;

Thanks again so much for a great lib, this is just a minor thing I noticed 🤗

@parkerziegler thanks for the report!
Indeed type parameters were missing. Done here: https://github.com/cristianoc/genType/pull/348.

❤ thanks so much!


Make peers table auto-refreshable like the table on the blocks page

So every XX interval the table will be refreshed.
To make it smart, we can start counting TIME_INTERVAL from lastBlockAdded timestamp (this will prevent refreshing 'in the middle' of the emitted block).
As part of this task, please do the same with TIME_INTERVAL with the blocks table.

TBD - it might be not relevant anymore.

Adding this every 30 seconds (each addition of a block) is not a good experience (performance wise).  Jan has an alternative solution but need to discuss with @darthsiroftardis.


Enable UI

Changes to enable UI. These changes will not be merged to master anytime soon. But once the pr is approved, I will use this branch as base to merge subsequent UI changes.

Closing this PR. Will be sending out new PR with more changes


Pages in nextjs app

Add the following pages

about
landing
contact

Create the following components

navbar

Do not change any designs or Layouts

hello sir ; kindly assign me this issue


tilt: shut down hud gracefully

This ended up being a bit more complicated than I expected, since apparently only clients can close streams gracefully.

user hits ^C on server
server calls GracefulStop(), which blocks until all outstanding requests have finished
server sends DoneReply to client
client reads DoneReply and closes its stream
GracefulStop() returns and we can kill the server

I'm not excited about storing ctx on ServerAdapter, but I was unable to find any way to propagate our context into the grpc server, and the only contexts I could find that get propagated to stream.Context() were starting from context.Background(). I'd be happy to be mistaken, but I'm not sure it's worth more time.

This isn't worth looking at yet. I think this is more complicated than it needs to be and instead of sending DoneReply we can just return from the server function.

fwiw, when i started setting up the demo, i was surprised at how complex the server/client API is

ok, this should be good to review now


Add alertmanager rules for Docker

Docker has a builtin prometheus exporter that we currently don't have enabled. This change adds alerts for stopped/paused containers and failed healthchecks.
Depends on:
kayobe: https://review.opendev.org/c/openstack/kayobe/+/936746
kolla-ansible: https://review.opendev.org/c/openstack/kolla-ansible/+/936742
ansible-collection-kolla: https://review.opendev.org/c/openstack/ansible-collection-kolla/+/936717

Sadly we decided to abort this due to the metrics source not providing the required data.


Validation scope fixes

I have no compiler for coffe script. Can you please meke it for me?

Hi, something missing in this PR? I really want to fix this bug soon. :bug:

Hi, I continued in your branch for this topic. I can clean these lines in the evening.

@paveljanda removed "multiaction" code

@chapcz It is not actually working. Inline edit/add with required input does not throw an alert.

@paveljanda do you use non minified datagrid.js? For me it is working both (add and edit). In this PR is not included minified version (I dont know method which is used for minify)

`
    $grid->addInlineEdit()
        ->onControlAdd[] = function(Container $container) {
        $container->addText('foo', '')->setRequired('Fill foo - edit');
        $container->addText('boo', '')->setRequired('Fill boo - edit');
     };

    $grid->addInlineAdd()
        ->onControlAdd[] = function(Container $container) {
        $container->addText('foo', '')->setRequired('Fill foo - add');
        $container->addText('boo', '')->setRequired('Fill boo - add')  ;
     };

`

@chapcz Aha, I used the datagrid.coffee file. Should I use only the datagrid.js?

I used in mentioned example all files from bower (similary as documentation example) but only thing which I changed is datagrid.js file.

@chapcz What datagrid,js version are you using? When I checked out branch validation-scope-fixes, there was about 500 changed lines when comparing to datagrid.js file in your PR. Which is weird.
Anyway, what branch did you fork? The validation-scope-fixes? Can you make the PR back into this branch? Thanks a lot!

@paveljanda ok, I changed the branch .... but I doun´t understand that you see 500 changed lines, but in PR  I see 15 changes

Can I do something else in this PR?

@paveljanda is there anything what can I do to move this issue forward?

I checked out this revision but it's not really working the way I suppose it should. The inline add form does not show any validation alerts.

chm, I think we are back in 19 sep. In my case, I have almost emty nette project, inside I cloned https://github.com/chapcz/datagrid.git and checked out branch validation-scope-fixes. In template I use almost everything from current bower. The only thing which I use from this repository is datagrid.js (also min version). In my example if fires both - add and edit forms as I showed in examples . :-(

Ok, let me create also an empty project with dependency on this branch.

Hi again. Today I updated my fork with base project and tested again. Validation works for me, only thing which is not working is  when I use min version of datagrid.js ... so can we make some progress in this please? Can you try this with non minified version please? I will try to do my best. Thanks.

Hi. Thanks for you work. I really appreciate it. I fixed it in a bit different way in upcoming version 6.
https://github.com/contributte/datagrid/commit/5e8db96fa74606b8f34b31e90ae867d113e60431


Retina Display

How can I make the game display in retina resolution on iPhones and retina monitors?

One way to fix this is to change this line:
https://github.com/tenjou/meta2d/blob/master/src/Engine.js#L564
to:
var ratio = window.devicePixelRatio || 1;
Also other way (and probably more stable performance-wise) is to use meta.engine.scale(window.devicePixelRatio, window.devicePixelRatio) function to scale content up.
In newer version will fix this in more intuitive way.

Awesome. I will give it a try. I try not to modify the engine code if i can.
Side note, are there docs or just examples?
On Thu, Jan 19, 2017 at 23:19 tenjou<EMAIL_ADDRESS>wrote:

One way to fix this is to change this line:
https://github.com/tenjou/meta2d/blob/master/src/Engine.js#L564
to:
var ratio = window.devicePixelRatio || 1;
Also other way (and probably more stable performance-wise) is to use meta.engine.scale(window.devicePixelRatio,
window.devicePixelRatio) function to scale content up.
In newer version will fix this in more intuitive way.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/tenjou/meta2d/issues/40#issuecomment-273994616, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAnZXP4ofyosIU4F67Eo6ogF36hlf4dbks5rUGAGgaJpZM4Lo1G4
.

--
Jared Sartin
*Mobile *(269) 364-1735 | Twitter @JaredSartin
http://twitter.com/JaredSartin
Website http://jared.sart.in/

Apart from examples in http://meta2d.com/examples/ there is WIP documentation http://meta2d.com/docs/ but still working on it.

meta.engine.scale(window.devicePixelRatio, window.devicePixelRatio); dies out
Trace below - using   <script src='http://meta2d.com/meta.js'></script>
Uncaught TypeError: Cannot read property 'clientWidth' of null
    at Object._resize (meta.js:1)
    at Object.scale (meta.js:1)
    at Object.<anonymous> (game.js:8258)
    at __webpack_require__ (game.js:556)
    at fn (game.js:87)
    at Object.<anonymous> (game.js:8173)
    at __webpack_require__ (game.js:556)
    at fn (game.js:87)
    at Object.<anonymous> (game.js:588)
    at __webpack_require__ (game.js:556)

Still is blurry on retina screens - will investigate.

Damn. I`ll give it closer look tomorrow. :)

Setting scale on dev build make everything scale up. I was just expecting an increased density (like width=2x height=2y style="width:x; height:y;"). More like ratio being set properly.

I updated http://meta2d.com/meta.dev.js version to take into account backingStoreRatio this should fix the issue. Though, don`t have retina devices nearby atm to test properly.


Refactor ValidatorSet

See each commit. I put a lot of effort into making the commits as atomic as possible.
Part of #836

I'll push the changes tomorrow, I have a commit that's still on my laptop.

Updated.
I haven't added min_distance to getPreimage because what we actually want is max_distance (we're specifically looking for an older preimage). And if I used max_distance it would be an awkward API because we already have hasPreimage that asks for a minimum distance. The two APIs would have opposite meanings of the second parameter.
However it was a micro-optimization anyway, we don't really need it.

Oh but I did remove getOlderPreimageDistance, since I also removed the Remove utxo key comparison in PreImage validation commit. Also removed the string buffer optimization.


Make 'Factories' final closed class

Minor change to prevent cognitive dissonance when looking at the class.

Why do this for this specific class? We typically don't seal our classes like this, or prevent instantiation. (I also don't know why this class was marked abstract.)

It's a static utility class. There's really no need for it to be instantiated.
I'm simply following 'Effective Java: Item 4: Enforce noninstantiability with a private constructor'

What problem are we trying to fix here? I don't recall accidental instantiation of utility classes being a major problem. Feels like we are adding unnecessary boilerplate unless there is a good reason to do it. If we do have a good reason (=a problem to fix), then we should do it for all similar classes as well.

The problem I was trying to fix here was purely a sense of confusion-in-the-moment I had looking at this class while I was modifying this class as a part of a different change. I didn't end up needing that larger change. Instead of trying to roll this change into another PR that was unrelated, this is my attempting to break it out into its own independent PR. I'm not really looking to make a sweeping change of the larger codebase at this point. I more saw this change as a minor bit of sweeping up in a place while I was already looking at the code.


getting issue while git clone repo

@Lancelot03 change the name of '1 to n--->xor.py' file i'm getting error while copying
please fix this issue by either deleting ro renaming the file

thanks for mentioning I rectified it.


[Improvement]: Web UI text enhancements

This issue tracks the work in correcting/enhancing the plugin's web UI text:

The term GKE “Publisher/Builder” is a bit foreign for me, deployer or applier would feel more Kubernetes native
The build step should be called “Build on Kubernetes Engine” instead of just “Kubernetes Engine”
Need help strings on text boxes, not sure if manifest list accept globs or comma separated

The term GKE “Publisher/Builder” is a bit foreign for me, deployer or applier would feel more Kubernetes native
Not addressing this because this language is never surfaced in the UI, but I incorporated "deployer" into the buildstep description "Deploy to Google Kubernetes Engine"

Done with #31


Update v2 API calls to 2.2

Facebook has deprecated API versions before v2.2.  Updated the gem version as well to ensure anyone still using this gem gets this update.

Please merge this as soon as possible. We had to go to a forked version to keep production up and running for our application.

can you try fb_graph2 gem?

It currently doesn't have any of the Ad or Marketing API's as far as I can tell. I know Facebook's release schedule is going to be different for the Graph API and the Marketing API now, and it may make sense to split the functionality into a separate gem at this point but I didn't know what your plan was. If your going to try and bring it into fb_graph2 then we can help with some of that since we use most of the marketing API ourselves. Otherwise we will create a separate gem for the marketing api.
Thoughts?
Jonathan De Jong
Partner
[Description: Description<EMAIL_ADDRESS>jdejong<EMAIL_ADDRESS>http://www.sparknettech.comhttp://www.sparknettech.com/
999 Old Eagle School Road * Suite 121 * Wayne, PA * 19087
800.893.2650tel:800.893.2650.p *<PHONE_NUMBER>tel:610.329.9510.c *<PHONE_NUMBER>tel:800.893.6075.f
From: Nov Matake <notifications<EMAIL_ADDRESS>Reply-To: nov/fb_graph <reply<EMAIL_ADDRESS>Date: Saturday, March 14, 2015 at 5:26 AM
To: nov/fb_graph <fb_graph<EMAIL_ADDRESS>Cc: Jonathan De Jong <jdejong<EMAIL_ADDRESS>Subject: Re: [fb_graph] Update v2 API calls to 2.2 (#390)
can you try fb_graph2 gem?

Reply to this email directly or view it on GitHubhttps://github.com/nov/fb_graph/pull/390#issuecomment-80208110.

To be honest, I don't know well about Ads APIs.
All code around Ads APIs are from contributors...

BTW, v2.0 isn't deprecated yet.
https://developers.facebook.com/docs/apps/changelog

It is for Marketing API. They only give you 90 days after they publish a release now on that API.
Jonathan De Jong
Partner
[Description: Description<EMAIL_ADDRESS>jdejong<EMAIL_ADDRESS>http://www.sparknettech.comhttp://www.sparknettech.com/
999 Old Eagle School Road * Suite 121 * Wayne, PA * 19087
800.893.2650tel:800.893.2650.p *<PHONE_NUMBER>tel:610.329.9510.c *<PHONE_NUMBER>tel:800.893.6075.f
From: Nov Matake <notifications<EMAIL_ADDRESS>Reply-To: nov/fb_graph <reply<EMAIL_ADDRESS>Date: Tuesday, March 17, 2015 at 9:53 AM
To: nov/fb_graph <fb_graph<EMAIL_ADDRESS>Cc: Jonathan De Jong <jdejong<EMAIL_ADDRESS>Subject: Re: [fb_graph] Update v2 API calls to 2.2 (#390)
BTW, v2.0 isn't deprecated yet.
https://developers.facebook.com/docs/apps/changelog

Reply to this email directly or view it on GitHubhttps://github.com/nov/fb_graph/pull/390#issuecomment-82363021.

Then probably what you need is something like this.
https://github.com/nov/fb_graph/commit/e2dc6370f6b3d5fe3b13e048b142295cbd38dc6a
I can't guarantee this gem works fine with v2.2 or later though..
If this is OK for you, I'll release new version.

I ended up forking and changed the /lib/fb_graph.rb and we have been running on that.
https://github.com/jdejong/fb_graph/blob/master/lib/fb_graph.rb
The bigger thing is there are a bunch of new Marketing API's which we have created as local model's in our App that ideally we are going to put into a Gem. If your ok with it we can start working on adding it to fb_graph2, otherwise we will create a separate gem for the marketing api only.
Jonathan De Jong
Partner
[Description: Description<EMAIL_ADDRESS>jdejong<EMAIL_ADDRESS>http://www.sparknettech.comhttp://www.sparknettech.com/
999 Old Eagle School Road * Suite 121 * Wayne, PA * 19087
800.893.2650tel:800.893.2650.p *<PHONE_NUMBER>tel:610.329.9510.c *<PHONE_NUMBER>tel:800.893.6075.f
From: Nov Matake <notifications<EMAIL_ADDRESS>Reply-To: nov/fb_graph <reply<EMAIL_ADDRESS>Date: Tuesday, March 17, 2015 at 10:22 AM
To: nov/fb_graph <fb_graph<EMAIL_ADDRESS>Cc: Jonathan De Jong <jdejong<EMAIL_ADDRESS>Subject: Re: [fb_graph] Update v2 API calls to 2.2 (#390)
Then probably what you need is something like this.
e2dc637https://github.com/nov/fb_graph/commit/e2dc6370f6b3d5fe3b13e048b142295cbd38dc6a
I can't guarantee this gem works fine with v2.2 or later though..
If this is OK for you, I'll release new version.

Reply to this email directly or view it on GitHubhttps://github.com/nov/fb_graph/pull/390#issuecomment-82380659.

OK, then I close this issue.
I think it's better to have a separate gem for Ads API.


feat: implemented credential details banner component

Fixes #1028

Signed-off-by: Anton Biriukov<EMAIL_ADDRESS>

Please add the screenshot

Please add the screenshot

Added. FYI, mobile design will be tackled in a separate issue @talwinder50

Also the banner values need to be extracted from vc in the helper.js file. Will you do that in separate pr ??

Yes its fine desktop is the priority first.

Also the banner values need to be extracted from vc in the helper.js file. Will you do that in separate pr ??

Yes, will add later


Per platform Create functions

Right now, the platform the engine chooses is decided at compile time with a macro:
`#ifdef PLATFORM_LINUX
#define ENGINE_EXPORT attribute((visibility("default")))
#define ENGINE_IMPORT
#include "GLFW/LinuxWindow.hpp"

#endif
#ifdef PLATFORM_WINDOWS
#define ENGINE_EXPORT __declspec(dllexport)
#define ENGINE_IMPORT __declspec(dllimport)
#endif`
Platform choosing should be at application compile time, not at engine compile time. Fixing the problem would mean a developer could just change the platform whenever he pleases.

Just moving the platform to a variable will be enough for now. I will do the change and adjust the create functions when I have time.


Package not included in Android?

../../../../../../.gradle/caches/modules-2/files-2.1/com.workday/postman/1.1/ee8f2b1ed06368a16d962ee20e5ab4ae6c8dc3ec/postman-1.1.jar: Invalid package reference in library; not included in Android: javax.tools. Referenced from com.workday.postman.codegen.ParcelerGenerator.

Priority: 6 / 10
Category: Correctness
Severity: Error
Explanation: Package not included in Android.

I'm guessing this is a lint error. You can safely ignore this for anything in com.workday.codegen because that code will never run in your app. It is only run during compile time.
However, I would recommend using the apt or kapt plugin. There's a note on how to use them in the readme.

Using apt seem to have fixed.


[Themer] Add settings into appearance

Heya, just wondering why Themer doesn't add all of the settings into Appearance since it seems like it'd fit there and it'd be more accessible then having to go to the settings in plugins or using the Dedicated Plugins Settings

I don't understand how that would be more accessible  since it'd still be the same amount of clicks. use dedicated plugin settings, you can even hide all plugins but themer from it


Fix CPI non monetary

The purchase model was failing with error "Missing consumer price index reference year" when applied to a purchase that wasn't monetary (such as 10 L of diesel).
The bug was fixed and a unit test was added.
FYI @pierresegonne

thanks!


LazyVim 10.0

Todo

[x] split lazyvim.util in multiple smaller modules for better readability and performance

existing lazyvim.util functions can still be used, and a deprecation warning will be shown

[x] new formatting framework that specs can use to integrate formatters / linters with fixers

a formatter can be primary
only one primary formatter will run based on availability and priority
secondary formatters will always run when available (like eslint fixes)
Added :LazyFormat and :LazyFormatInfo commands

[x] make conform/nvim-lint the default and have none-ls as an extra
[x] make dashboard.nvim the default instead of alpha.nvim. Better code and better performance
[x] customizable root detection and :LazyRoot
[x] added :LazyExtras to easily enable/disable LazyVim extras
[ ] docs on formatting, rootdir, extras, ...

Testing
To test the new release, add the below somewhere in your specs:
    { "LazyVim/LazyVim", branch = "refactor", version = false },

:LazyExtras

:LazyRoot

:LazyFormatInfo

Hi! I'm kind of new in both development (web, aggg) and neovim, but I'm a huge fan. I don't really know how I can help.
For the time being I will 'install' (don't know the appropriate term here) and report any bug or issue I could find that after research I can't relate to a plugin thing.
Keep the good work! Your plugin manager is my fav. Cheers.

I gave it a try and enabled the aerial extra, which seems to have added a <leader>cs mapping to <cmd>SymbolsOutline but there doesn't seem to be any such command... should it be mapped to AerialToggle instead?

@abusch right, will fix! And also need to add that extra for symbols outline.

{ "LazyVim/LazyVim", import="lazyvim.plugins", branch = 'refactor', version = "10.x" },

Not sure if this changes or not, but I still need to import default plugins or else it doesn't work

@lkhphuc you need to specify version = false for it to work as specified in the PR description.
Version 10.x has not been released yet.

Ah thanks. It automatically updated correctly now.
Not a big deal since I can fix these in my config, but since you asked to report any error beside the deprecation warning:
   Error  09:56:26 notify.error lazy.nvim Failed to load `plugins.none-ls`

/Users/plekhac/.config/nvim/lua/plugins/none-ls.lua:5: module 'null-ls' not found:
	no field package.preload['null-ls']
cache_loader: module null-ls not found
cache_loader_lib: module null-ls not found
	no file './null-ls.lua'
	no file '/opt/homebrew/share/luajit-2.1.0-beta3/null-ls.lua'
...

From my old null-ls config, migrating/delete null-ls for now fixes it.
   Error  09:56:26 notify.error lazy.nvim Two plugins with the same name and different url:
{
  new = { "romgrk/nvim-treesitter-context",
    _ = {
      dep = false,
      fid = 150,
      fpid = 148,
      module = "plugins.treesitter"
    },
    dir = "/Users/plekhac/.local/share/nvim/lazy/nvim-treesitter-context",
    name = "nvim-treesitter-context",
    opts = {
      multiline_threshold = 2
    },
    url = "https://github.com/romgrk/nvim-treesitter-context.git"
  },
  old = { "nvim-treesitter/nvim-treesitter-context",
    _ = {
      dep = false,
      fid = 39,
      module = "lazyvim.plugins.treesitter"
    },
    dir = "/Users/plekhac/.local/share/nvim/lazy/nvim-treesitter-context",
    enabled = true,
    event = "LazyFile",
    name = "nvim-treesitter-context",
    opts = {
      mode = "cursor"
    },
    url = "https://github.com/nvim-treesitter/nvim-treesitter-context.git"
  }
}

Reinstall from new origin and update my config fixes it.
   Error  09:56:27 notify.error lazy.nvim Failed to run `config` for nvim-navbuddy

...share/nvim/lazy/nvim-navbuddy/lua/nvim-navbuddy/init.lua:1: module 'nvim-navic.lib' not found:
	no field package.preload['nvim-navic.lib']
cache_loader: module nvim-navic.lib not found
cache_loader_lib: module nvim-navic.lib not found
	no file './nvim-navic/lib.lua'
	no file '/opt/homebrew/share/luajit-2.1.0-beta3/nvim-navic/lib.lua'
	no file '/usr/local/share/lua/5.1/nvim-navic/lib.lua'
	no file '/usr/local/share/lua/5.1/nvim-navic/lib/init.lua'
	no file '/opt/homebrew/share/lua/5.1/nvim-navic/lib.lua'
	no file '/opt/homebrew/share/lua/5.1/nvim-navic/lib/init.lua'
	no file './nvim-navic/lib.so'
	no file '/usr/local/lib/lua/5.1/nvim-navic/lib.so'
	no file '/opt/homebrew/lib/lua/5.1/nvim-navic/lib.so'
	no file '/usr/local/lib/lua/5.1/loadall.so'
	no file './nvim-navic.so'
	no file '/usr/local/lib/lua/5.1/nvim-navic.so'
	no file '/opt/homebrew/lib/lua/5.1/nvim-navic.so'
	no file '/usr/local/lib/lua/5.1/loadall.so'

# stacktrace:
  - /opt/homebrew/Cellar/neovim/HEAD-c40a1c0/share/nvim/runtime/lua/vim/lsp.lua:1710 _in_ **_on_attach**
  - /opt/homebrew/Cellar/neovim/HEAD-c40a1c0/share/nvim/runtime/lua/vim/lsp.lua:1460 _in_ **fn**
  - vim/_editor.lua:341

Not sure about why navic was removed though.

@lkhphuc yeah, that's the same error I had regarding none-ls.
Regarding navic, the description mentions:

moved nvim-navic to extras in favor of nvim-treesitter-context

@abusch ah make senses then. But still shouldn't throws an error. A notification about enabling this extra would be nice.
Also, the stacktrace for this is a bit obscure. If it can point out to my file "nvim/lua/plugins/lsp.lua" where I specify that would be nice.
   Warn  10:08:52 notify.warn LazyVim `nvim-lspconfig.opts.autoformat` is deprecated. Please use `vim.g.autoformat` instead

# stacktrace:
  - vim/_editor.lua:418 _in_ **cmd**
  - /persistence.nvim/lua/persistence/init.lua:57 _in_ **load**
  - lua:1
  - vim/_editor.lua:418 _in_ **cmd**
  - /dashboard-nvim/lua/dashboard/theme/doom.lua:23

Sure here is my none-ls.lua file.
return {
  "nvimtools/none-ls.nvim",
  opts = {
    sources = {
      require("null-ls").builtins.formatting.yapf.with({
        extra_args = function(params)
          return params
            and {
              "--style={indent_width:" .. vim.fn.shiftwidth() .. "}",
            }
        end,
      }),

      require("null-ls").builtins.formatting.stylua.with({
        extra_args = function(params)
          return params
            and {
              "indent-type=Spaces, indent-width=" .. vim.fn.shiftwidth(),
            }
        end,
      }),

      require("null-ls").builtins.code_actions.shellcheck,

      require("null-ls").builtins.hover.dictionary,
      require("null-ls").builtins.hover.printenv,
    },
  },
}

Just added additional warning message when trying to load a lua module from a plugin that is no longer in core:

@lkhphuc I see. The error is actually because your spec itself already loads none-ls.
So if you would remove all your plugins, then that spec would always fail.
Better to make opts a function so that it's in a closure and will only be called when none-ls is installed.

I still get an error if I disable the none-ls extra:

But that seems to be because I only used "none-ls.nvim" in my spec instead of the full "nvimtools/none-ls.nvim", so that's probably my fault 😅 (although somehow it works if the plugin is there ...)

@abusch yes, that's normal. When you use a short name for a plugin, then you need to have loaded the plugin somewhere else (by enabling the extra in this case)


controlling retry maxAttempts

Hi
I am testing httpclient-processor-kafka spring cloud app with both success and error cases in a Spring cloud dataflow server.
For successful processing scenario, the HTTP request is being retried attempted for 3 times which is the default maxAttempts for this app for calling http request.
How the retrial attempts can be controlled to just one time

Hi @rajurc This project has been moved to another repository: https://github.com/spring-cloud/stream-applications
httpclient app is now refactored into http request processor function. Please engage with us at the new repository. To answer your question, you can control maxAttempts using the property - spring.cloud.stream.bindings.input.consumer.maxAttempts. See more details here.


Corrected grammar error

OPS Build status updates of commit 14425e0:
:clock10: Preparing: average preparing time is 1 min(s) 13 sec(s)

OPS Build status updates of commit 14425e0:
:clock10: Full build: average full build time is 12 min(s) 15 sec(s) [calculated from last 9 build(s)]

Note: A full build takes more time than an incremental build.  This is the expected behavior of the build system.


Example document

Good day to all.
Could you give a trivial example of downloading a torrent file
I just can't find an example.
I'm studying the API, but I can't find examples to even run the console tracker corny

i'm stupped sorry pls))


⚠ Mobitopup has degraded performance

In d7b0eaa, Mobitopup (https://napcard.voz.tech) experienced degraded performance:

HTTP code: 200
Response time: 4648 ms

Resolved: Mobitopup performance has improved in 4bbd46e.


add logger for client ip and ua

Logger to add ip=xx.xx.xx.xx and ua=xxxx in log

Looks good - thanks! 👍


Reduce hardcoded definitions and exceptions

cc #61

@homu r+

:pushpin: Commit 6649725 has been approved by gkoz

:hourglass: Testing commit 6649725 with merge 4398e94...

:sunny: Test successful - status


feat: add integration tests for install ack

What type of PR is this?
Test
What this PR does / why we need it?

Adds integration tests for the recently added installation acknowledgment feature.
Also sets CGO_ENABLED=0 to have a statically-linked binary for local development testing.

Which Jira/Github issue(s) this PR fixes?
Fixes: https://issues.redhat.com/browse/MTSRE-1468
Special notes for your reviewer:
Pre-checks (if applicable):

[x] Tested latest changes against a cluster
[x] Ran make test-unit command locally to run all the unit tests and mock tests locally.

/retest

/lgtm


MS Visual Studio Code - VirtualNetwork module - Complains subnetPrefix is missing but will not build when you include it.

Bicep version
run bicep --version via the Bicep CLI, az bicep version via the AZ CLI or via VS code by navigating to the extensions tab and searching for Bicep
0.6.18
Describe the bug
I have defined a module to deploy a virtual subnet. When I try to publish the module, VS Code complains I am missing the subnetPrefix., But when I include it in the properties, then it complains that it is not allowed in the properties field. I did note that it will deploy correctly with it included in the properties section, even though bicep complains that it should not be included.
To Reproduce
Create a bicep module which deploys a vnet and subnet. From your main bicep file, include the section for module so you can deploy the vnet and subnet as a module.
Additional context
Add any other context about the problem here.

Can you provide the full bicep code you are using to reproduce? As well as the command(s) you are executing.


Notifications via MQTT

Is your feature request related to a problem? Please describe.
No its not related to a problem.
Describe the solution you'd like
Currently there is a way to setup notifications in evcc configuration. I want to push the same notifications using mqtt to my home assistant.
would be great if we can get the same events published to MQTT:
events:
start:
title: Charge started
msg: Started charging in "${mode}" mode
stop:
title: Charge finished
msg: Finished charging ${chargedEnergy:%.1fk}kWh in ${chargeDuration}.
connect:
title: Car connected
msg: "Car connected at ${pvPower:%.1fk}kW PV"
disconnect:
title: Car disconnected
msg: Car disconnected after ${connectedDuration}
soc:
title: Soc updated
msg: Battery charged to ${vehicleSoc:%.0f}%
guest:
title: Unknown vehicle
msg: Unknown vehicle, guest connected?
Describe alternatives you've considered
I used telegram but i would like to receive all my notifications through Home Assistant.

Happy to take a PR

Why not just simply install the HACS evcc integration and create the notifications directly inside Home Assistant using native HA Automations?


Published version should either polyfill ES6 methods (Array.fill... etc), or document necessary polyfills.

This will address issues with Array.from, String.startsWith, and Set

If you want to tackle having an es6 version and put in a PR that would be a great addition.
Another option is to simply publish the src directory in additional to the transpiled modules.
// Transpiled
import { Button } from 'rmwc/Button';

// Original src
import { Button } from 'rmwc/src/Button';

The only minor down side is slightly longer install times since it's double the code.

@jamesmfriedman the later would be a great start. And sure -- when I get to it I will. Currently just using your git version + custom typescript definitions (the ones by that other guy were not designed the way I like them) and works like a charm.
I generally copy my src to /dist/es6 or whatever to do just that: enable es6 output.

Checkout the typescript issue #40 . I had the idea of automatically generating Typescript definitions from the generated doc JSON file. Could be interesting.

Yep, that's how I got to https://github.com/ncknt/rmwc-typed. Generating ts definitions would be amazing, albeit a tad difficult, as your library introduces only a few new types, and they are pretty sound, but because the original base code allows you to plug any tag, type expansion is meh.
Example:
<SomeComponent tag="span" />

Now your flow definitions just ignore this and assign the typescript equivalent of any, but realistically the type should be something like:
type SomeComponent = React.ComponentType<
  SomeComponentProps &
  React.DetailedHTMLProps<
    React.HTMLAttributes<HTMLSpanElement>,
    HTMLSpanElement
  >
>

Still unsure if that's 100% coverable but alas; I think having anything in place that's at least partially type-safe or at least for the types you have right now would be great.
https://github.com/bcherny/flow-to-typescript could be interesting. It's a compiler that could run, but for that the flow types need to be better. https://github.com/niieani/typescript-vs-flowtype some interesting differences.

Yeah, theres a type for those though. All the types are an intersection with SimpleTagPropsT https://github.com/jamesmfriedman/rmwc/blob/master/src/Base/simpleTag.js#L15
I would just have to go through the codebase and verify that. React-docgen however can't see types that are imported from different files, it just runs statically on the individual components.

React.ComponentType<any> just loses a lot of information, as does React.Ref<*>;
I think the new title is what I was getting at 👍 .

I know this is a long running issue, but switching to typescript in 5.x.x and targeting es5 with helpers should fix this.

Fixed with the release of V5


Add new common MR api converters

Description of your changes

Adds new common converters for:

[x] mq.aws.crossplane.io/v1alpha1 : Broker
[x] apigatewayv2.aws.upbound.io/v1alpha1 : Integration
[x] apigatewayv2.aws.upbound.io/v1beta1 : VPCLink
[x] apigatewayv2.aws.upbound.io/v1alpha1 : API
[x] apigatewayv2.aws.upbound.io/v1alpha1 : Route
[x] apigatewayv2.aws.upbound.io/v1alpha1 : DomainName
[x] apigatewayv2.aws.upbound.io/v1alpha1 : Deployment
[x] apigatewayv2.aws.upbound.io/v1alpha1 : APIMapping
[x] apigatewayv2.aws.upbound.io/v1alpha1 : Stage
[x] cloudfront.aws.crossplane.io/v1alpha1 : ResponseHeadersPolicy

Fixes #21
I have:

[x] Run make reviewable test to ensure this PR is ready for review.

How has this code been tested

 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.


farmer profile setup page - frontend

if its first time acc created :

setup farmer profile...more like kyc
preferred language
add in their required cards
produce details'
land area
experience in sales
make it as much intuitive as possible
refer figma

yoo


Pybind unavailable when installed using chaquopy

OS Platform and Distribution
android
Compiler version
No response
Programming Language and version
Python
Installed using virtualenv? pip? Conda?(if python)
pip
MediaPipe version
latest
Bazel version
No response
XCode and Tulsi versions (if iOS)
No response
Android SDK and NDK versions (if android)
21
Android AAR (if android)
None
OpenCV version (if running on desktop)
No response
Describe the problem
when installing through chaquopy  whl file,  can not see pybind11 modules in mediapipe package
I have been trying to run mediapipe on android studio using chaquopy 14.0
Everything working fine except when package is installed in my android folder, i can not see pybind11 modules which are basically c++ modules required to run library. when i clone git repo, i can see all files available but unable to generate wheel from that repo. i am basically creating a facemesh in mediapipe and need to run python script on android and communicate through modules in my react native app.
Can you please help me to find a solution. When i try to create wheel using setup.py it throws exceptions
Complete Logs

Stderr: [FATAL 06:41:17.311 src/main/cpp/blaze.cc:1302] Unknown startup option: '-l'.
  For more info, run 'bazel help startup_options'.
WARNING: Target pattern parsing failed.
ERROR: no such package '@flatbuffers//': Non-zero return code(2) when executing 'C:\ProgramData\chocolatey\bin\bazel.exe -l -c "rm" "-rf"

i am using

python 3.8.10
chaquopy 14
pip 23.1

i also tried using wsl2 but same results !

Hi @usmanaqeel786,
We have not conducted testing for this particular scenario within MediaPipe, and we do not currently have documented procedures for utilizing Android Studio with Chaquopy.
We kindly ask you to submit a feature request with detailed information regarding this matter. Our team will then internally review and prioritise this task. Thank you for your cooperation.


Fix correct param passing from Airflow (version >2.3.3) to HOP

Description
Issue
There was an issue with passing parameters and macro variables from Apache Airflow to Apache HOP. The parameters were not being correctly interpreted in the HOP logs, showing the raw macro variable names (e.g., {{ ds }}) instead of their expected values (e.g., 20-05-2024).
Solution
Changed the parameter name from param to hop_param. This adjustment ensures compatibility and correct macro convertion in HOP logs.
Additional Information
In Apache Airflow 2.3.3, the params variable is utilized during DAG serialization. Using params as a variable name in third-party operators can lead to issues. For more information, refer to the Apache Airflow Pull Request #20640

You should correct template_fields on both operators so the airflow templates render correctly into their respective values.
Additional info: https://airflow.apache.org/docs/apache-airflow/stable/howto/custom-operator.html#templating

I tested the code without changing template_fields, and all tests passed with both macros and normal string params. Therefore, I believe we do not need to modify template_fields.

You are right. I didn't notice that you only changed the constructor parameter and that is not affected by template_fields. Thanks for your contribution!


Prevent using actions/setup-java@v3 with adopt distribution

NOTE: Adopt OpenJDK got moved to Eclipse Temurin and won't be updated anymore. It is highly recommended to migrate workflows from adopt to temurin to keep receiving software and security updates. See more details in the Good-bye AdoptOpenJDK post.

What about earlier versions than v3? or any sha?
In other words - does this apply to setup-java for all its versions, or just v3?

V1 doesn't support choosing a distribution version. About V2 it is not clear.
Please check out this source

Hey, @jmj0502, we appreciate your offer to contribute and you are more than welcome!
Please check out how to contribute by the README file.
Prevent using action/setup-java will be implemented as a rule. A rule is a yaml file that includes Json schema that validates the desired behavior. Please see the examples here.
In addition to the schema you need to add a description, failureMessage, uniqueId and enabledByDefault keys as in all other rules.
Last, you need to add a similar rule (a yaml file) for Gitlab pipelines here. In your case, because setup-java action does not exist for .gitlab-ci.yaml files so the schema value should be empty as here

@OriYosef Cool! I'll give a shot. I'll reach out in case I have any questions 👍


Unify the spm/bpe tokenizers

#561 fixed

All looks good. My only comment is that KernelBpeTokenizer::SpmTokenize does not support fairseq or computing instance indices at the moment.
But I noticed you added the SPM implementation to bpe_kernels.cc but did not delete sentencepiece_tokenizer.cc, sentencepiece_tokenizer.hpp, or sentencepiece_decoder.hpp. I assume we will do this in the future as that is the main incentive of this unification, and when we do, we can also take care of the remaining SPM stuff (fairseq and instance indices) in the bpe source.

There will be a transition period. The old sentencepiece tokenizer code needs to be kept for Custom-Op backward compatibility. The faireseq indices will be added in the future.


Deployment never completes but cluster is active

I have issues
The same code was use to build two other clusters and they are working. There was some lag from when the initial clusters were built.
I'm submitting a...

[X] bug report
[ ] feature request
[ ] support request - read the FAQ first!
[ ] kudos, thank you, warm fuzzy

What is the current behavior?
The cluster deploy never completes with worker nodes and ASG's.  The Cluster state does show active. Seems like the ASG' and worker nodes are never deployed.
If this is a bug, how to reproduce? Please include a code sample if relevant.
When running apply, the cluster is built and in an active state but the node groups are never created, there are no ASG's or launch templates.
It's stuck in a loop
module.eks-cluster.null_resource.wait_for_cluster[0]: Still creating...
module.eks-cluster.null_resource.wait_for_cluster[0]: Still creating...
module.eks-cluster.null_resource.wait_for_cluster[0]: Still creating...
module.eks-cluster.null_resource.wait_for_cluster[0]: Still creating...
Main.tf
module "eks-cluster" {
source                          = "terraform-aws-modules/eks/aws"
version                         = "9.0.0"
cluster_name                   = "qa-eps-eks"
subnets                             =  "${data.aws_subnet_ids.default.ids}"
vpc_id                              =  "${var.vpc_id}"
cluster_endpoint_private_access = true
cluster_endpoint_public_access  = false
cluster_security_group_id =  "${data.aws_security_group.default.id}"
worker_groups = [
{
name = "qa-eks-workers"
instance_type = "m4.large"
key_name = "qa"
asg_min_size      = 2
asg_desired_size  = 2
asg_max_size      = 5
autoscaling_enabled = true
tags = [{
propagate_at_launch = true
key         = "terraform"
value       = "true"
}]
}
]
tags = {
environment = "qa"
terraform   = "true"
}
}
What's the expected behavior?
The expected behavior is the cluster is built along with ASG and worker nodes.
Are you able to fix this problem and submit a PR? Link here if you have already.
Environment details

Affected module version:
OS:
Terraform version: 0.12.23

Any other relevant info
Module v9.0.0

Do you have wget installed? The command that gets run is: https://github.com/terraform-aws-modules/terraform-aws-eks/blob/master/variables.tf#L204
I'm not sure how the module would behave in terms of redirecting  stderr but testing the command with a non existing command on my machine:
❯ until wgett --no-check-certificate -O - -q $ENDPOINT/healthz >/dev/null; do echo "sleeping" ; sleep 4; done
zsh: command not found: wgett
sleeping
zsh: command not found: wgett
sleeping
zsh: command not found: wgett
sleeping
zsh: command not found: wgett
sleeping

it runs forever as far as I can see.

If wget will be not available then it will fail. In this case I suggest to check wget version as there is known issue with SSL compatibility or verify does endpoint is really available from place where you running terraform code (in case of private envpoint it must be same VPC, in case of public endpoint you must ensure about whitelist value configured under https://github.com/terraform-aws-modules/terraform-aws-eks/blob/4c0c4c49a611e0d7dcaff334fc7862c31732d1f0/variables.tf#L249-L253 )
Please check  also https://github.com/terraform-aws-modules/terraform-aws-eks/issues/757

I see the problem I think:
cluster_endpoint_private_access = true
cluster_endpoint_public_access = false

@js-timbirkett That seems to have fixed it. I reverted to v8.1.0 and then went back to v9.0.0 removed the one line
cluster_endpoint_public_access = false
It also didn't like, so I had to fix that.
asg_min_size = 2
asg_desired_size = 2
asg_max_size = 5
So everything looks like it's provisioned, but it does throw an error about the auth map already exists. I did clear out the authmap that it drops in the root and deleted .terraform it it still throws that error at the end.


Add preference for application/json

This issue is related to #87 - it would be nice if pyswagger worked out of the box with all of the petstore API.   Even a simple get is flumoxed by the lack of either:

A way to prefer JSON
A way to parse XML such as that presented in #87

My preference would be to be able to add a header such as "Accept" "application/json" in the client.   However, I guess I'm not seeing how to do that.

I will see if I can do this with security on "Accept", and see if it ends up in the header and converts the response to JSON.

For adding preference on "Accept", "Content-Type" section in header when requesting:
req, resp = app.op['your_operation_id'](**your_parameter')
req.produce('applications/json') # Accept: application/json
req.consume('application/xml') # Content-Type: application/json

A tutorial relative to this part is here
I didn't include xml coder in this project, so if you need to handle xml content-type, you need to install one and write a codec handler by yourself (sample code in tutorial already did it)
I'm not quite sure about your sentence: [...with security on "Accept"], thus if what I provided is not what you want, could you help to describe your scenario more detailedly?

Thanks, I'll try this.

So, I followed part of the mime wiki tutorial, and my XmlCodec doesn't work.   However, that's OK, as what I want to do is simply add an Accept header and therefore get JSON instead.   When I try
req.produce("application/json")
I get then a backtrace when I try to use it with a pyswagger.contrib.client.requests.Client object.
Since I'm posting a rather long backtrace, I should also mention that I've got 0.8.27 of pyswagger.
Here is the rather long ipython trace:
In [20]: req, resp = app.op['findPetsByStatus'](status=['available'])

In [21]: req2 = req.produce('application/json')

In [22]: id(req2)
Out[22]:<PHONE_NUMBER>00512

In [23]: id(req)
Out[23]:<PHONE_NUMBER>00512

In [24]: r = client.request((req,resp,))
---------------------------------------------------------------------------
ValueError                                Traceback (most recent call last)
<ipython-input-24-0b77ba45e9e3> in <module>()
----> 1 r = client.request((req,resp,))

/home/dan/python-envs/trypyswagger/lib/python3.5/site-packages/pyswagger/contrib/client/requests.py in request(self, req_and_resp, opt)
     72             status=rs.status_code,
     73             header=rs.headers,
---> 74             raw=six.BytesIO(rs.content).getvalue()
     75         )
     76

/home/dan/python-envs/trypyswagger/lib/python3.5/site-packages/pyswagger/io.py in apply_with(self, status, raw, header)
    401                 name = schema.name
    402                 data = self.__op._mime_codec.unmarshal(content_type, self.raw, _type=_type, _format=_format, name=name)
--> 403                 self.__data = r.schema._prim_(data, self.__op._prim_factory, ctx=dict(read=True))
    404
    405         return self

/home/dan/python-envs/trypyswagger/lib/python3.5/site-packages/pyswagger/spec/v2_0/objects.py in _prim_(self, v, prim_factory, ctx)
     90
     91     def _prim_(self, v, prim_factory, ctx=None):
---> 92         return prim_factory.produce(self, v, ctx)
     93
     94

/home/dan/python-envs/trypyswagger/lib/python3.5/site-packages/pyswagger/primitives/__init__.py in produce(self, obj, val, ctx)
    180             ret = creater(obj, val, ctx)
    181             if _2nd:
--> 182                 val = _2nd(obj, ret, val, ctx)
    183                 ctx['2nd_pass'] = _2nd
    184         elif len(obj.properties) or obj.additionalProperties:

/home/dan/python-envs/trypyswagger/lib/python3.5/site-packages/pyswagger/primitives/comm.py in _2nd_pass_obj(obj, ret, val, ctx)
     38 #
     39 def _2nd_pass_obj(obj, ret, val, ctx):
---> 40     return ret.apply_with(obj, val, ctx)
     41
     42 def create_obj(obj, v, ctx=None, constructor=None):

/home/dan/python-envs/trypyswagger/lib/python3.5/site-packages/pyswagger/primitives/_array.py in apply_with(self, obj, val, ctx)
     35
     36         if obj.items and len(val):
---> 37             self.extend(map(functools.partial(ctx['factory'].produce, obj.items), val))
     38             val = []
     39

/home/dan/python-envs/trypyswagger/lib/python3.5/site-packages/pyswagger/primitives/__init__.py in produce(self, obj, val, ctx)
    180             ret = creater(obj, val, ctx)
    181             if _2nd:
--> 182                 val = _2nd(obj, ret, val, ctx)
    183                 ctx['2nd_pass'] = _2nd
    184         elif len(obj.properties) or obj.additionalProperties:

/home/dan/python-envs/trypyswagger/lib/python3.5/site-packages/pyswagger/primitives/comm.py in _2nd_pass_obj(obj, ret, val, ctx)
     38 #
     39 def _2nd_pass_obj(obj, ret, val, ctx):
---> 40     return ret.apply_with(obj, val, ctx)
     41
     42 def create_obj(obj, v, ctx=None, constructor=None):

/home/dan/python-envs/trypyswagger/lib/python3.5/site-packages/pyswagger/primitives/_model.py in apply_with(self, obj, val, ctx)
     47         not_found = set(obj.required) - set(six.iterkeys(self))
     48         if len(not_found):
---> 49             raise ValueError('requirement not meet: {0}'.format(not_found))
     50
     51         # remove assigned properties to avoid duplicated

ValueError: requirement not meet: {'name'}

In [25]:

OK - I have some reason to think that this was not your problem, but the sort of assertion I actually want to use pyswagger to test.  When I try "findPetsByStatus" with status=['available'] I get the failure on {'name'}.   When I try "getPetById", it just works, and similarly for "findPetsByStatus" with status=['sold'].
My conclusion therefore is that the "requirement not met" message is because there is some pet out there without a name, and that name is required in the swagger.yaml    So, the response truly was invalid.
Here is it working:
In [36]: req, resp = app.op['getPetById'](petId=3048819905)

In [37]: req.produce('application/json')
Out[37]: <pyswagger.io.Request at 0x7f4ea8dc15f8>

In [38]: r = client.request((req,resp,))

In [39]: r.status
Out[39]: 200

In [40]: r.data
Out[40]:
{'category': {'id': 721582874, 'name': 'dog'},
 'id':<PHONE_NUMBER>,
 'name': 'hello kity',
 'photoUrls': ['http://foo.bar.com/1', 'http://foo.bar.com/2'],
 'status': 'sold',
 'tags': [{'id': 168551466, 'name': 'swagger-codegen-python-pet-tag'}]}
In [41]: req.header
Out[41]: {'Accept': 'application/json'}

FWIW - it is req.produce('application/json') that sets Accept.

Not sure if I fully understand your intension, but to test findPetsByStatus, you need to provide 'string', not 'list of string'.
# correct
status='available'
# wrong
status=['available']

Thanks for writing pyswagger!   Now that I've completed this tiny experiment, https://github.com/danizen/trypyswagger, I can show it to the guy at work who wants to use json-schema.org for validation and show him that this is better for our use case.
It definitely is an array of strings, as the following excerpt of petstore.yaml shows.
      parameters:
      - name: "status"
        in: "query"
        description: "Status values that need to be considered for filter"
        required: true
        type: "array"
        items:
          type: "string"
          enum:
          - "available"
          - "pending"
          - "sold"
          default: "available"
        collectionFormat: "multi"

got it, I'm wrong with that part :(
For json-schema validation, you can also refer to this project: https://github.com/Julian/jsonschema, which is highly recommended if you just want to use json-schema
The reason pyswagger didn't use that project to is Swagger (OpenAPI) is not fully compatible with json-schema, so I had to write one by myself.

Thanks for the pointer - both I and the other guy want to stay all Python on the server side, except for a tiny bit of npm/gulp/webpack for managing assets.

@danizen  it's actually a project with 1000+ star in python :)


Outputting the Incorrect variable for Removals in the Verbose output

Working on this one, pointed out another issue where I wasn't outputting the incorrect variable for Removals in the Verbose output causing it to be blank. That will be fixed also.
Originally posted by @steppsr in https://github.com/steppsr/getxchtxPS/issues/7#issuecomment-1105784180

v0.2.0


Do some data computations with extending postDataLoad resolver

Amplify CLI Version
12.6.0
Question
In my model I have a field ExpDate that is an iso timestamp.
I want that when I list my model a computated field isExpired is set to true if ExpDate > current timestamp.
Following https://docs.amplify.aws/cli/graphql/custom-business-logic/#extend-amplify-generated-resolvers I'm assuming I can create a postDataLoad VTL for accomplish the job and do a simple if statement inside it.
I'm also assuming that I need to implement this logic inside the res VTL, but I'm not sure how the req VTL should be.
Any hints?

I probably managed it to work without using a Lambda. This is the req VTL as we use a NONE_DS datasource as documented here: https://docs.aws.amazon.com/appsync/latest/devguide/resolver-mapping-template-reference-none.html
{
    "version": "2018-05-29",
    "payload": $util.toJson($context.prev.result)
}

This is my res VTL with custom logic:
## [Start] ResponseTemplate. **

#set($currentMillis = $util.time.nowEpochMilliSeconds())

#foreach($item in $ctx.result.items)
    #set($item.isExpired = false)

    #if($item.expirationDate)
        #set($expirationDateMillis = $util.time.parseISO8601ToEpochMilliSeconds($item.expirationDate))
        #if($expirationDateMillis < $currentMillis)
          #set($item.isExpired = true)
        #end
    #else
        ## Handle missing expiration date
        $util.error("An error occurred: Missing expiration date for item: $item")
    #end

#end

$util.toJson($ctx.result)

## [End] ResponseTemplate. **

If you have any other suggestion I'm open to try, otherwise I can consider the issue closed.

Hey @ggorge-etiqa, I apologize for the delay. I have consulted with the Engineering team, and they have confirmed that this is the recommended approach to fulfill your use case.


Inconsistent overwriting of headers

Do you want to request a feature or report a bug?
Bug
What did you do?
I tried to set up Træfik with some default headers via a default middleware in a convention-over-configuration fashion. The middleware defined in the entry point defines a standard set of security headers which can then be overwritten by defining middlewares for a router.
When only the default middleware is active everything is fine and the headers are reported as expected:
curl -sD - -o /dev/null  http://localhost

HTTP/1.1 200 OK
Accept-Ranges: bytes
Content-Length: 612
Content-Type: text/html
Date: Thu, 23 Apr 2020 14:17:11 GMT
Etag: "5e95c66e-264"
Last-Modified: Tue, 14 Apr 2020 14:19:26 GMT
Referrer-Policy: strict-origin-when-cross-origin
X-Content-Type-Options: nosniff
X-Frame-Options: DENY
X-Test: foobar
X-Xss-Protection: 1; mode=block

When I add a middleware that only contains an additional header (custom@file, first commented line in the attached docker-compose.yml), the output is still fine:
curl -sD - -o /dev/null  http://localhost

HTTP/1.1 200 OK
Accept-Ranges: bytes
Content-Length: 612
Content-Type: text/html
Date: Thu, 23 Apr 2020 14:18:44 GMT
Etag: "5e95c66e-264"
Last-Modified: Tue, 14 Apr 2020 14:19:26 GMT
Referrer-Policy: strict-origin-when-cross-origin
X-Content-Type-Options: nosniff
X-Custom-Response-Header: value
X-Frame-Options: DENY
X-Test: foobar
X-Xss-Protection: 1; mode=block

However, when I activated the middleware allow-x-frame-self@file(regardless whether it was used together with custom@file or not; second and third commented lines in the attached docker-compose.yml) the headers were no longer correct.
Without having inspected the source code, it seems for me like a second instance of the headers middleware is overwriting/resetting all values from the previous one expect for the custom headers. This might also be the root cause for #5568.
Note: for this bug report, I have reduced it to http and removed https configuration, forwarding and STS headers but that use case is also affected by this bug.
What did you expect to see?
curl -sD - -o /dev/null  http://localhost

HTTP/1.1 200 OK
Accept-Ranges: bytes
Content-Length: 612
Content-Type: text/html
Date: Thu, 23 Apr 2020 14:18:44 GMT
Etag: "5e95c66e-264"
Last-Modified: Tue, 14 Apr 2020 14:19:26 GMT
Referrer-Policy: strict-origin-when-cross-origin
X-Content-Type-Options: nosniff
X-Custom-Response-Header: value
X-Frame-Options: SAMEORIGIN
X-Test: foobar
X-Xss-Protection: 1; mode=block

What did you see instead?
curl -sD - -o /dev/null  http://localhost

HTTP/1.1 200 OK
Accept-Ranges: bytes
Content-Length: 612
Content-Type: text/html
Date: Thu, 23 Apr 2020 14:19:48 GMT
Etag: "5e95c66e-264"
Last-Modified: Tue, 14 Apr 2020 14:19:26 GMT
X-Frame-Options: SAMEORIGIN
X-Test: foobar

Output of traefik version: (What version of Traefik are you using?)

I have attached all my configuration files in one zip to this ticket.
bug.zip

I confirm ! ;(
When you chain middlewares, for example 'security-headers@docker,myapp-headers' then value of 'security-headers' are ignored or reset.
You can switch order to confirm... (ie : 'myapp-headers,security-headers@docker')

This is related to https://github.com/containous/traefik/issues/5568, but also related to https://github.com/containous/traefik/issues/5568
Should be fixed by https://github.com/containous/traefik/pull/6604 and https://github.com/containous/traefik/pull/6750

Fixed by #7230


RUMM-3005: Logs feature is using message bus to forward errors to RUM

What does this PR do?
This PR makes Logs feature to use message bus to forward errors to RUM instead of having an explicit dependency on the RUM.
Review checklist (to be filled by reviewers)

[ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[ ] Make sure you discussed the feature or bugfix with the maintaining team in an Issue
[ ] Make sure each commit and the PR mention the Issue number (cf the CONTRIBUTING doc)

Codecov Report

Merging #1267 (5f6f74f) into feature/sdkv2 (6babedf) will increase coverage by 0.01%.
The diff coverage is 76.92%.

@@                Coverage Diff                @@
##           feature/sdkv2    #1267      +/-   ##
=================================================
+ Coverage          81.69%   81.70%   +0.01%
=================================================
  Files                358      358
  Lines              12733    12784      +51
  Branches            2130     2146      +16
=================================================
+ Hits               10401    10444      +43
- Misses              1679     1684       +5
- Partials             653      656       +3

Impacted Files
Coverage Δ

...g/android/log/internal/logger/DatadogLogHandler.kt
76.58% <57.69%> (-5.45%)
:arrow_down:

...lin/com/datadog/android/rum/internal/RumFeature.kt
91.95% <89.74%> (+0.23%)
:arrow_up:

...android/sessionreplay/internal/utils/ThemeUtils.kt
83.33% <0.00%> (-16.67%)
:arrow_down:

...d/v2/core/internal/storage/FileEventBatchWriter.kt
94.74% <0.00%> (-2.63%)
:arrow_down:

...main/kotlin/com/datadog/android/log/LogsFeature.kt
84.11% <0.00%> (-0.66%)
:arrow_down:

.../android/rum/internal/domain/scope/RumViewScope.kt
95.12% <0.00%> (-0.18%)
:arrow_down:

...id/rum/internal/ndk/DatadogNdkCrashEventHandler.kt
90.00% <0.00%> (+1.00%)
:arrow_up:

...rsistence/file/batch/PlainBatchFileReaderWriter.kt
83.08% <0.00%> (+1.54%)
:arrow_up:

...droid/rum/tracking/ActivityViewTrackingStrategy.kt
88.46% <0.00%> (+1.92%)
:arrow_up:

... and 2 more


Added simpler .hist example

The Histogram page in the Reference Gallery didn't have a simple example of using '.hist()', which is typically much more convenient than the complex example shown.  Added this simple example:

Works fine for Bokeh, but in mpl there is a strange colormap:

The last example on that page (not really modified in this PR)  is also aligned and sized very strangely for matplotlib, though it works fine for bokeh:

That is a worthwhile example to add. Best we wait to see which of those matplotlib bugs can be fixed before merging.

Will see, dealing with adjoints more generally in matplotlib has been a longstanding issue (see https://github.com/ioam/holoviews/issues/1033).

@philippjfr Should we merge despite the weird matplotlib output?

I'll see if I can deal with the wrong colormapping at least. Doubt I'll be able to fix the sizing issue easily.

@jbednar Would you mind updating this PR by setting an explicit range for the x and y dimensions? For now matplotlib won't normalize the points and histogram together so you end up with different scales.

It sounds like you know what's needed; can you just push what you suggest to the PR?

Done. @jlstevens ready to merge when tests pass.

Looks good and the tests are passing. Merging.


Added DateTimeOffset value-serializer

Added DateTimeOffset support. Took manifest id 10 for this..

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