Downgrade image-webpack-loader to avoid error with jpeg files

https://github.com/tcoopman/image-webpack-loader/issues/142

@Dezhic jpeg files seem to load fine for me with image-webpack-loader v4.2.0. Is there any specific case where it fails for you?

Closing this PR as this thread has been dead for 3 months now.


Fluent emoji nuget package increases ram and disk usage to %100

🐛 Bug Report
After installing and adding 5 emojies, ram and disk usage jumps to 100% and compuler slows down
💻 Repro or Code Sample
just installing the package causes problem
🤔 Expected Behavior
Normal disk and ram usage
😯 Current Behavior
Compter slow down, nearly lock up
💁 Possible Solution
Possible memory leakage
When i uninstall the package, things goes back to normal
🔦 Context
🌍 Your Environment
Windows 11
Visual studio 2022 Version 17.9.7
Latest fluent ui emoji pack nuget (installed via vs)

Is this a problem in Visual Studio?
Can you check if you have the latest version of VS2022?
Can you check whether your anti-virus software would take time to scan the contents of this package?
This is probably a problem related to your machine configuration, as we use it regularly and it is also used on https://www.fluentui-blazor.net/Emoji

Is this a problem in Visual Studio? Can you check if you have the latest version of VS2022? Can you check whether your anti-virus software would take time to scan the contents of this package?
This is probably a problem related to your machine configuration, as we use it regularly and it is also used on https://www.fluentui-blazor.net/Emoji

Yes it happened on Visual Studio  and i have the latest version
I have stopped and restarted the antivirus and the problem seems to be resolved
Thanks for the support.

Antivirus program caused the problem


event.x and event.y inaccurates

So when you click to get the coordinates of the mouse, event.x and event.y are wrong (at least it is the case on my PC running Linux and a tiling WM)

I should have been event.xdata and event.ydata !


[AST/Sema] RuntimeMetadata: Align implementation with proposal

@reflectionMetadata attributes declared on a protocol act as requirements. If type doesn't declare attribute, conformance fails unless conformance is stated directly on type declaration, in such cases attribute would be inferred.
allow using reflection metadata attributes on unavailable extensions declared in the same module as type.
Reject reflection metadata attributes with custom arguments on protocol declarations. Arguments cannot be used if types declare a conformance to a protocol from a different module.
Diagnose situations where a sub-class or a protocol are missing reflection metadata attributes required by a superclass.

Resolves: rdar://103990788

@swift-ci please test

@swift-ci please test

@swift-ci please test

@swift-ci please test

@swift-ci please test macOS platform


Bugfixes on Invoice and HardwareAssetAdmin. Remove single-use asset import script. Export view on HardwareAssetAdmin.

This change is

Coverage increased (+0.9%) to 50.291% when pulling cb85f0f623bbc0566d850e561059550184560238 on ropable:master into 5b3b3c30023bdc9f03c44b2e88be2eede5ce533d on parksandwildlife:master.


ReadOnly Sensitive Form Field

Winter CMS Build: Latest
PHP Version: 7.2
Database Engine: mysql
Plugins Installed: N/A

Description
Setting a sensitive form field to readOnly doesn't stop the display of the password when clicking the view button.
Steps To Reproduce:
And the following to fields.yaml:
test_api_key:
    label: Test API key
    placeholder: Enter key
    type: sensitive
    readOnly: true

I'm curious as to why you think readOnly should prevent that, technically they're not editing the field, only reading it. What's the use case you're trying to achieve?

Good point, maybe it's just me but it seems counterintuitive to be able to interact with a field that is set to readOnly, I assumed it would just display as a password field type instead.

I'm not really sure what the point of that happening would be tbh, are you trying to hide the sensitive data from unprivileged users?

Sorry yes, I worked around it by serving different fields based upon user permission. So user with privileges gets field as sensitive otherwise as readOnly password

There are other ways you can achieve similar results too, for example you could hook into the backend.ajax.beforeRunHandler event and throw an ApplicationException if you detect someone is trying to onShowValue() without the correct level of access to the data.
Is there anything actionable left in this issue or are you fine with your current solution?

There are other ways you can achieve similar results too, for example you could hook into the backend.ajax.beforeRunHandler event and throw an ApplicationException if you detect someone is trying to onShowValue() without the correct level of access to the data.
Good to know!


interactions disappeared between 20201001 and 20201101 nt files

Describe the bug
A triple for interacts_with between ACE2 and GLP1R is present in the 20201001 release but not 20201101.
To Reproduce
This triple:
P43220 interacts_with Q9BYF1
Is present in the .nt file from 20201001:
https://kg-hub.berkeleybop.io/kg-covid-19/20201001/kg-covid-19.nt.gz
but not in the 20201101 one:
https://kg-hub.berkeleybop.io/kg-covid-19/20201101/kg-covid-19.nt.gz
Note that this triple is not present in the .tsv files from both 20201001 and 20201101.
Here are all the relevant triples from the 202021101 .nt file:
urn:uuid:1aad7d40-aa7a-4ec1-87c9-85108b3eb77f http://www.w3.org/1999/02/22-rdf-syntax-ns#subject http://identifiers.org/uniprot/Q9BYF1 .
urn:uuid:1aad7d40-aa7a-4ec1-87c9-85108b3eb77f http://www.w3.org/1999/02/22-rdf-syntax-ns#predicate https://w3id.org/biolink/vocab/interacts_with .
urn:uuid:1aad7d40-aa7a-4ec1-87c9-85108b3eb77f http://www.w3.org/1999/02/22-rdf-syntax-ns#object http://identifiers.org/uniprot/P43220 .
urn:uuid:1aad7d40-aa7a-4ec1-87c9-85108b3eb77f https://w3id.org/biolink/vocab/relation http://purl.obolibrary.org/obo/RO_0002434 .
urn:uuid:1aad7d40-aa7a-4ec1-87c9-85108b3eb77f https://w3id.org/biolink/vocab/provided_by "STRING" .
urn:uuid:1aad7d40-aa7a-4ec1-87c9-85108b3eb77f http://www.w3.org/1999/02/22-rdf-syntax-ns#type "biolink:Association"^^http://www.w3.org/2001/XMLSchema#string .
urn:uuid:1aad7d40-aa7a-4ec1-87c9-85108b3eb77f https://www.example.org/UNKNOWN/combined_score "157.0"^^http://www.w3.org/2001/XMLSchema#float .
urn:uuid:1aad7d40-aa7a-4ec1-87c9-85108b3eb77f https://www.example.org/UNKNOWN/neighborhood "0.0"^^http://www.w3.org/2001/XMLSchema#float .
urn:uuid:1aad7d40-aa7a-4ec1-87c9-85108b3eb77f https://www.example.org/UNKNOWN/neighborhood_transferred "0.0"^^http://www.w3.org/2001/XMLSchema#float .
urn:uuid:1aad7d40-aa7a-4ec1-87c9-85108b3eb77f https://www.example.org/UNKNOWN/fusion "0.0"^^http://www.w3.org/2001/XMLSchema#float .
urn:uuid:1aad7d40-aa7a-4ec1-87c9-85108b3eb77f https://www.example.org/UNKNOWN/cooccurence "0.0"^^http://www.w3.org/2001/XMLSchema#float .
urn:uuid:1aad7d40-aa7a-4ec1-87c9-85108b3eb77f https://www.example.org/UNKNOWN/homology "0.0"^^http://www.w3.org/2001/XMLSchema#float .
urn:uuid:1aad7d40-aa7a-4ec1-87c9-85108b3eb77f https://www.example.org/UNKNOWN/coexpression "0.0"^^http://www.w3.org/2001/XMLSchema#float .
urn:uuid:1aad7d40-aa7a-4ec1-87c9-85108b3eb77f https://www.example.org/UNKNOWN/coexpression_transferred "0.0"^^http://www.w3.org/2001/XMLSchema#float .
urn:uuid:1aad7d40-aa7a-4ec1-87c9-85108b3eb77f https://www.example.org/UNKNOWN/experiments "0.0"^^http://www.w3.org/2001/XMLSchema#float .
urn:uuid:1aad7d40-aa7a-4ec1-87c9-85108b3eb77f https://www.example.org/UNKNOWN/experiments_transferred "0.0"^^http://www.w3.org/2001/XMLSchema#float .
urn:uuid:1aad7d40-aa7a-4ec1-87c9-85108b3eb77f https://www.example.org/UNKNOWN/database "0.0"^^http://www.w3.org/2001/XMLSchema#float .
urn:uuid:1aad7d40-aa7a-4ec1-87c9-85108b3eb77f https://www.example.org/UNKNOWN/database_transferred "0.0"^^http://www.w3.org/2001/XMLSchema#float .
urn:uuid:1aad7d40-aa7a-4ec1-87c9-85108b3eb77f https://www.example.org/UNKNOWN/textmining "108.0"^^http://www.w3.org/2001/XMLSchema#float .
urn:uuid:1aad7d40-aa7a-4ec1-87c9-85108b3eb77f https://www.example.org/UNKNOWN/textmining_transferred "94.0"^^http://www.w3.org/2001/XMLSchema#float .
Expected behavior
Unclear why this interaction disappeared later.
Version
20201001 vs 20201101
Additional context
related to https://github.com/Knowledge-Graph-Hub/kg-covid-19/issues/375

Is it possible to track the provenance of this triple when it was still present in the .nt file?
The "metadata"/reification triples retrieved for  P43220 interacts_with Q9BYF1 are listed in the issue, but none of those seems to actually point to a publication from which this triple was presumably extracted.
Could the disappearance of the triple be related to possible low extraction confidence? In this respect, is information such as
ex:textmining = "108.0" or textmining_transferred=  "94.0" of any significance?
Is the semantics of predicates (textmining_transferred, textmining, combined_score) documented somewhere?


Unexpected low performance of mistral-v0.1

Hi!
I tried to run BANKING77 with mistral-7b-v0.1. However, I got a nearly zero performance. I have no idea about how this could happen. Have you ever tested mistral-7b-v0.1? Or, if not, did you ever encounter similar issues?
Thanks!

v0.1 only supports 8K token length, which leads to low performance. We use v0.2 because it supports 32K tokens.

The first 3 subsets of BANKING77 are below 8k. So, on the three subsets, I expected that mistral-7b-v0.1 should have a performance lower than v0.2 but at least comparable to other models. However, I got a performance of (nearly) zero.
Using the same codes and replacing mistral-7b-v0.1 with mistral-7b-v0.2, I can get similar results as those reported in the paper.

We observe similar trend for Gemma, which should support 8K but actually failed on tasks less than 8K tokens.

Got it. Seems like some prompting stuffs. I will try to modify the prompts to see if there is any difference.
Thank you for your clarification!


[Retail App] Account profile allows for saving of invalid phone numbers. FIXED

Description
Added phone number validation to resolve the issue number https://github.com/SalesforceCommerceCloud/pwa-kit/issues/196
Types of Changes

[ ] Bug fix (non-breaking change that fixes an issue)
[x] New feature (non-breaking change that adds functionality)
[ ] Documentation update
[x] Breaking change (could cause existing functionality to not work as expected)
[ ] Other changes (non-breaking changes that does not fit any of the above)

Breaking changes include:

Removing a public function or component or prop
Adding a required argument to a function
Changing the data type of a function parameter or return value
Adding a new peer dependency to package.json

Changes

Added the validate field for the phone object in the useProfileFields.jsx
Added the validate field for the phone object in the useAddressFields.jsx
Created a new function validatePhone in the phone-utils.js using regex

How to Test-Drive This PR

npm start
login to your profile
Edit phone number to something invalid

Checklists

General

[ ] Changes are covered by test cases
[ ] CHANGELOG.md updated with a short description of changes (not required for documentation updates)

Accessibility Compliance
You must check off all items in one of the follow two lists:

[x] There are no changes to UI

or...

[ ] Changes were tested with a Screen Reader (iOS VoiceOver or Android Talkback) and had no issues
[ ] Changes comply with WCAG 2.0 guidelines levels A and AA
[ ] Changes to common UI patterns and interactions comply with WAI-ARIA best practices

Localization

[ ] Changes include a UI text update in the Retail React App (which requires translation)

Hey folks,
Generally I'd be very cautious about including this sort of functionality.
There are many different formats of phone numbers internationally and Commerce Cloud is present in many markets.
I think we could very easily get into a situation where a shopper wants to save a valid phone number that is rejected by validator.
For me, that is a much worse problem than allowing shoppers to save incorrect phone numbers, as to resolve, it would require the shopper to contact support, who would then need to manually enter the phone number and then likely contact the dev team to update this regexp ;P
If there is an authoritative way to validate phone numbers, sure let's use that, but if not, I'd urge us to consider any value the API accepts a valid phone number. If this was a big problem, wouldn't the API reject bad phone numbers?
If we do choose to go ahead with this, rather than code comments about what numbers are valid, we need tests.

There's also a wide variety of open-source international phone number formatting/validation libraries in JavaScript/React that would be worth evaluating as a 3rd party package if we value accuracy and minimal maintenance/customization of this area

Hey @agurumurthy96, thank you for looking into this ticket and creating a fix for it. After a thorough discussion, we've decided to not proceed with this PR as our Retail React app can be used in the different marketplace that uses different phone number formats. It'd be better to leave this field to be more flexible as is.


[Inference Backend] Enable attention Key/value caching

Is there a plan to incorporate key/value caching to improve generation efficiency significantly? See e.g. Guidance's acceleration.

Hi @lbeurerkellner , thanks for the quick response. I am referring to caching the LLM's key/value attention pairs for sequential variable value generation. So for instance in the above example, first the LLM populates the ID. For the next call it generates DESCRIPTION. At this point you already have computed the LLM's key/value pairs for the template up until ID, and should not have to recompute this. The longer the template to be filled in becomes, the more important (faster + cheaper) this caching becomes. For production use cases, this is a very significant benefit. Check this post for more detail. Of course (for now) this is only possible for self-hosted models, not external API calls such as using OpenAI models.

The PagedAttention mechanism from vLLM may help https://github.com/vllm-project/vllm (massive speed up)

any progress about this?

There is a proof-of-concept implementation on a feature branch, but to make it work with batching, padding and multi-part prompting, it still requires some work. It may be worth to factor out support for non-batched KV caching for now.


Safe name - Character limit inconsistency

Description
For the Safe creation there is no limit in characters for the name. For the safe edition a limit of 50 characters is imposed
URL: https://rinkeby.gnosis-safe.io/
Snapshot:

Steps:
1 - Create a safe
2 - Put 60 characters as a name and finish the creation of the safe
3 - Go to the settings > Safe details
4 - You will see right away how the field alerts the excess of characters (limit 50)

@dasanra and @francovenica , why the issue is closed without QA ?

There may be some issues between github and zenhub.
QA feedback is directly in the PR ticket. Don't know why it isn't linking both.
https://app.zenhub.com/workspaces/safe-multisig---web-5ce554debb310a35b2f8b6f8/issues/gnosis/safe-react/1357


Add Bigtable API

For some reason VS saw fit to remove some commas from apis.json as well.

No problem. I'll add them back in.

AppVeyor change suggests you need to rebase from master and rerun generateprojects.sh... I modified the generator earlier today, sorry!

No it's my fault: I actually did rebase before creating the PR - I just forgot to regenerate.


Questions about Text Embedding for Classifier Initialization.

How do you convert the 512-d CLIP vectors into 256-d to init the classifier? Thanks

We adopt a linear layer to transform the output visual features of the decoder into 512-d.

The lr of the  linear layer and the classifier are both 1e-5?

Only the classifier.


Add reverse() in Set, And Implement it in TreeSet and LinkedHashSet

What does this PR do?

It adds reverse() in Set, And Implement it in LinkedHashSet and TreeSet.

What issue does this fix or reference to?

It reference to Issue-2514

I studied the BitSet - I think it isn't possible because BitSet<T> allows to operate on generic types T in the presence of mappers fromInt: Integer -> T and toInt: T -> Integer.
When reversing a BitSet, the mappers need also to be changed, which is not possible in an automatic way.
My stomach says that there are too many caveats with this whole issue of adding a reverse() method. It does not fit to the type hierarchy, as we already saw, and we have problems to implement it for specific types.
Please let us leave this functionality away, I have no good feeling about it and it will mean maintenance hell because we need to copy paste it to LinkedHashMap and LinkedHashSet.
I hope you understand and I really appreciate your effort. At least we learned more about it. The issue history will help us answering future questions.
Many thanks!
- Daniel


Fix fix window function in aggregation

Proposed changes
Problem:
When window function in aggregation function, executor would report an error like:  Required field 'node_type' was not present!
Example:
SELECT SUM(MAX(c1) OVER (PARTITION BY c2, c3)) FROM  test_window_in_agg;
Reason:
When analyze aggregate, analytic expr (window function carrior when analyze) transfered to slot and loss message. So when
serialize to thrift package, TExpr can not determine node_type of analytic expr.
Solved:
We do not support aggregate(window function) yet. So we report an error when analyze.
Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

(From new machine)TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 47.42 seconds
stream load tsv:          512 seconds loaded<PHONE_NUMBER>9 Bytes, about 139 MB/s
stream load json:         37 seconds loaded<PHONE_NUMBER> Bytes, about 60 MB/s
stream load orc:          64 seconds loaded<PHONE_NUMBER> Bytes, about 16 MB/s
stream load parquet:          31 seconds loaded 861443392 Bytes, about 26 MB/s
insert into select:          29.6 seconds inserted 10000000 Rows, about 337K ops/s
storage size:<PHONE_NUMBER>1 Bytes


[Improvement] Notification for folders

RainLoop version, browser, OS: 1.11.1 - Chrome - Ubuntu 16.04
Expected behavior and actual behavior: there is no way to being notified of new Emails in folders (when moved with a sieve rule)
I would like to be able to enable notification for selected folder:

My personal use case is for monitoring alerts, customers new support tickets etc.

Hi,
I am also need this, since even if i receive new email counter in other folders, I don't receive desktop and neither sound notification on new emails on folders other than Inbox.

Hi, This functionality would be great.

Hi! I've just switched to Rainloop and I also miss this feature. I've got loads of mail going directly to folders!


npm installs 1.2.0?

I get 1.2.0 unless I specify:
npm install preact-redux@latest
that shouldn't be necessary, right?

@shovemedia are you using a boilerplate? This doesn't seem to be an issue, or at least I can't reproduce* and can't think of anything that would cause it. I am going to close this out because I am convinced it's unrelated, but I'm happy to help you debug. It could even be a caching issue if you're using a custom npm regsitry.
*:

ok to close. I think I somehow had a weird local install. Not using a custom registry, but agreed there must've (hopefully) been some kind of cache issue.


added hints for static typing and whitespace fixes

also added build/ to .gitignore to exclude build files from repo

This still has class Room included in network.gd btw. I didn't want to break anything with the hints. That also means some vars and parameters are still without type, as I was unsure about their type. And Godot does not support type Variant yet, so those are without a hint as well.


Where would this be used? does it come up often enough to be in the stdlib?

@seankhliao

When downloading file concurrently using http ranges.
Merging multiple files into one.
Any use case where multiple go routines try to write concurrently at specific locations of a file.

Inconvenience with current implementations:

WriteAt() needs to be wrapped in for loop with temporary buffer and need to keep track to check if everything is written.
WriteAt() with buffer length equal to data being read, uses too much of memory.
Seek(), using seek with multiple go routines isn't a valid option.

Better Performance for concurrent writes.

Zero Copy functions like ReadFrom(), and WriteTo() can be used for concurrent writes, which is not possible now.

or another option would be to povide ReadFromAt() and WriteToAt()
In-case of unix copyfilerange is a system call, which exactly does this.
Thank you.

Note that we use the copy_file_range system call with io.Copy or more generally with os.File.ReadFrom.  But it's true that we currently always pass the offsets arguments as nil.
In CopyAt it's not immediately clear whether the off argument applies to the reader or the writer.  I can infer that it's the writer because the type is WriterAt.  But in general I think if we pass an offset for one we should pass an offset for both, which just gives us the CopyFileRange function.
I think in general having both io.Copy and io.CopyBuffer was a mistake, and I wouldn't want to repeat that mistake.
We already have io.WriterAt with a WriteAt method, and os.File already has a WriteAt method.  And you can already call that method concurrently from multiple goroutines.  So I don't have a clear handle on what new functionality we get from these new functions.

We already have WriteAt

Yes, But some book-keeping is needed. WriteAt() is similar to write().

I don't have a clear handle on what new functionality we get from these new functions.

Like io.Copy() for Write(), io.CopyAt() for WriteAt()
// CopyBufferAt , copies `src` to `dst` at `off` using `buf`
//
// copied from io.copyBuffer
func CopyBufferAt(dst io.WriterAt, src io.Reader, off int64, buf []byte) (written int64, err error) {
	for {
		nr, er := src.Read(buf)
		if nr > 0 {
			nw, ew := dst.WriteAt(buf[0:nr], off+written)
			if nw < 0 || nr < nw {
				nw = 0
				if ew == nil {
					ew = errors.New("invalid write result")
				}
			}
			written += int64(nw)
			if ew != nil {
				err = ew
				break
			}
			if nr != nw {
				err = io.ErrShortWrite
				break
			}
		}
		if er != nil {
			if er != io.EOF {
				err = er
			}
			break
		}
	}
	return written, err
}

As for my limited understanding goes,
Currently it's not possible to do zero copy operations at specific locations.
If this feature is added, it would be good.
Thank you.

Adding more variants of Copy seems like a mistake, as others have noted.
Adding SectionWriter to match SectionReader seems plausible.
Does anyone object to adding SectionWriter?

This proposal has been added to the active column of the proposals project
and will now be reviewed at the weekly proposal review meetings.
— rsc for the proposal review group

Does a call to NewSectionWriter(w, off, n) imply that the size of w will always become at least off + n?  Or does it depend on subsequent writes?  Or is it an error if the size is not already at least off + n?

@ianlancetaylor
I don't think n is necessary here.
I just copied definitions from SectionReader and changed it to SectionWriter.

If n should stay, then I think you need to answer my earlier questions:

Does a call to NewSectionWriter(w, off, n) imply that the size of w will always become at least off + n? Or does it depend on subsequent writes? Or is it an error if the size is not already at least off + n?

Thanks.

It depends on subsequent writes.

It depends on subsequent writes.

How would that be helpful for concurrent writes, then?

@opennota
Isn't this going to work?
func main() {
	var length, ps, i int64 = 1000000, 100000, 0
	f, _ := os.Create("file")

	for ; i < length; i += ps {

		go func(from, to int64) {
			req, _ := http.NewRequest(http.MethodGet, "https://someurl.com/file", nil)

			setRange(req, from, to)
			resp, _ := http.DefaultClient.Do(req)

			nw := io.NewSectionWriter(f, from, to)
			io.Copy(nw, resp.Body)
			resp.Body.Close()
		}(i, i+ps)
	}
}

func setRange(req *http.Request, start, end int64) {
	req.Header.Set("Range", fmt.Sprintf("bytes=%d-%d", start, end-1))
}

It's true, you don't seem to need n (the section length). But that makes it not really a SectionWriter but more like an OffsetWriter. That's fine of course.

OK so it sounds like the API is:
// An OffsetWriter maps writes at offset o to offset o+off in the underlying writer.
type OffsetWriter struct { }

func NewOffsetWriter(r WriterAt, off int64) *OffsetWriter
func (s *OffsetWriter) Write(p []byte) (n int, err error)
func (s *OffsetWriter) WriteAt(p []byte, off int64) (n int, err error)
func (s *OffsetWriter) Seek(offset int64, whence int) (int64, error)

Do I have that right? Does anyone object to this?

Does anyone object to this?

Personally no.

Based on the discussion above, this proposal seems like a likely accept.
— rsc for the proposal review group

Just clairfying, the offsets in these methods end up being on top of the original offset used to create the *OffsetWriter, correct?
func (s *OffsetWriter) WriteAt(p []byte, off int64) (n int, err error)
func (s *OffsetWriter) Seek(offset int64, whence int) (int64, error)

on top of the original offset?

I'm not sure what do mean by on top of but if you mean relatively, yes.
NewOffsetWriter(x, 10)

WriteAt(y, 10) // y is written at offset 20 of x.

but if you mean relatively, yes.

Yes. Thanks.

No change in consensus, so accepted. 🎉
This issue now tracks the work of implementing the proposal.
— rsc for the proposal review group

Sadly I didn't see this proposal while it was still in the comments period. This API is supposed to help with concurrent writes in the multi section download. Since the upper bound of the file write is not limited anymore this looks very dangerous to use.
I fear it will lead to concurrent writes to overlapping file sections, probably resulting in file corruptions.
@rsc / @ianlancetaylor could you please reconsider this API addition before it gets implemented or do I overlook something? The original API with sections seemed much safer to me.

@nightlyone Perhaps we should separate OffsetWriter from LimitedWriter.

Since the upper bound of the file write is not limited anymore this looks very dangerous to use.
I fear it will lead to concurrent writes to overlapping file sections, probably resulting in file corruptions.

But, it is similar to other api's like WriteAt(), user can make a mistake anywhere.
I think it is the responsibility of user to be careful.
Another possible solution, is to wrap your source (reader) with LimitReader.
nw := io.NewOffsetWriter(f, from)
io.Copy(nw, io.LimitReader(resp.Body, n))


Important notes

Thanks and appreciated the effort to come up with the repo.
Please correct me if I am wrong. I noticed the functions are called via http request instead of ws which does not serve the purpose of "websocket" therefore is totally a different thing than the title of this repo supposed to be.
If I am correct, I hope for the good of future learners, please remove the repo. Thanks again for your time reading this.

Hey! The truth is that you don't need to handle all of your communication with the server via websockets. There are only some parts that need ws, in case of this project this part is chat, as soon as the user wants to join the chat - he establishes websocket connection. All of the following communication for the chat features is handled using websockets


Demo.py throw exceptions.UnboundLocalError

Hello, I am studying your project, and encounter crush by running demo.py
line 119:
if frame_count == 1 or frame_count % 2 == 0:
In my environment, webcam may started later than python thread, therefore, the first frame may not
be 1 in this line.
My solution: add length check for prev_marks
if len(prev_marks) == 0 or frame_count == 1 or frame_count % 2 == 0:
Thanks

It seems to be compatible with the current code. Fix is applied. Thank you very much.
Please do not hesitate to ask for fix if other problems occur.


Improve the line_attributes test

This is an attempt to improve the output of the test so that it looks reasonable on both a real VT340, as well as on modern terminals, that aren't necessarily limited in the same way. I've also added a section showing how sixel could be used to trick the VT340 into displaying both single-width and double-height text on the same line.
If everything has gone according to plan, I'm expecting the VT340 output to look like this:

On modern terminals, though, I'd expect the bottom half of the test to look better (changing line attributes wouldn't typically erase the content of the line), but the mix of single and double text is highly unlikely to work.

Again if the output on the VT340 doesn't match what I was expecting in the top screenshot, please let me know before merging so I can try and fix it.

Almost. The bottom half of the plus signs got cut off.

OK. That's not bad. It's probably good enough to demonstrate the concept, since I'm not really expecting anyone to try and emulate that behaviour anyway (most terminals don't even support the standard double-width/height text). But if you're willing to do a bit of experimenting, I have some thoughts as to why it's failing:

There's an invisible image that I'm outputting to trigger the buffer reset echo ${DCS}';1q?'${ST}, but that's only large enough to cover the first line. I wouldn't have thought it would make any difference, but you could try changing it to something like echo ${DCS}'2;1q?'${ST} so it intersects both lines.
The other thought is that the DECDHL needs to be set before any text is written. The DOUBLE text isn't being cut off, and that's output after the DECDHL, while the +s are output before that. So we might be able to fix it by rewriting that section of the script as follows:

# Output a double-height sequence of text.
set_cursor_pos 12 1
echo ${DECDHLT}
echo ${DECDHLB}
set_cursor_pos 12 1
yes '+' | tr -d '\n' | head -c 80
test_message 12 17 ' DOUBLE '
test_message 13 17 ' DOUBLE '

But if neither of those work, it's not the end of the world. I'd be happy to just merge it as is.

I just realised I could test this equally well on the VT240, and I got the exact same problem you're seeing on the VT340. Setting the line attributes before writing out the + characters definitely fixes the problem for me. I also realised I needed to set DECAWM to make sure the +s wrapped over both lines.


mind the character limit when generating table names

when table names are composed of several components (which correspond to the elements in nested json) we can easily cross the 255 chat limit of ext filesystem or table name length limit of a particular db.

prevent creating component table names longer than x characters: ie. 20 chars max and then 6 chars of hash
prevent creating full table names longer than 200 chars. if this is exceeded then: root_table__depth_hash__parent_table__child_table where depth replaces all the inner tables   (depth: number of replaces tables, hash: some hash on replaced content)

example limits

postgres, mysql: 63 chars
redshift: 127 chars
bigquery: 320 chars

closed by #157


Mock's executes wrong signature

arg1 is an array of arguments on the type level but actually a first argument in the runtime
mock().executes(
      async (arg1: number) => {
        if (last === 0) {
          return 5
        }
        return 7
      },

Fixed by https://github.com/l2beat/earl/pull/234


Update use of #nosec directive

We've seen that gosec v2.18.1 seems to introduce an issue in the way that the nosec directive is interpreted. This pull request introduces a syntax workaround until they can get it fixed.

This is great. I am going to merge.


some files causes openwebui to close without error

Bug Report
Installation Method
i use git clone and everything works until someone loads document with big table in it or big csv document. I can't find any error on logs even in debug mode.
Environment

Open WebUI Version: [e.g., v0.3.11]
only using chatgpt

Operating System: windows server 2019, pipeline works on docker i connect it succesfully with port. there is no rag customization. default settings is causing error also.

Browser (if applicable): all

Confirmation:

[x] I have read and followed all the instructions provided in the README.md.
[x] I am on the latest version of both Open WebUI and Ollama.
[x] I have included the browser console logs.
[x] I have provided the exact steps to reproduce the bug in the "Steps to Reproduce" section below.
[x] I have included the Docker container logs.

Actual Behavior:
i tried tika, changed extraction to chatgpt but still having the same issue. file is uploading correctly but when i submit it, it breaks the program and causes sudden shut down of service.
Description
Bug Summary:
Everything works until someone loads document with big table in it or big csv document. I can't find any error on logs even in debug mode.
i tried tika, changed extraction to chatgpt but still having the same issue. file is uploading correctly but when i submit it, it breaks the program and causes sudden shut down of service.
could you lead me to detailed logs of RAG or any environment variable or way to close big document uploads or file uploads all together. I can'e debug this error :/
there is no customization on the platform.

After weeks of struggling to find a solution, I finally gave in and opened an issue on GitHub. And guess what? Less than an hour later, I might have found the solution! 🎉 It seems like the problem was related to a bug involving uvicorn and asyncio on Windows, specifically in production with SSL.
I've got a long testing journey ahead, but right now, it looks like things are working. I’d like to leave this issue open for about a week, as I think it could help others facing similar problems. I’m actively working on it and will share my test results soon. A PR is also in the works!
Thanks for your patience!


[beta] Applied Visual Design: Add a Box-shadow to a Card-like Element | Hover issue

Applied Visual Design: Add a Box-shadow to a Card-like Element
http://beta.freecodecamp.com/en/challenges/applied-visual-design/add-a-boxshadow-to-a-cardlike-element
The camper must add a property when the id is on its hover state whereas this hasn't been studied yet
In this part, it is required to add a box-shadow property for the thumbnail id on its hover state.
The problem is that at this stage, campers are not supposed to know how to do that, because none of the previous challenges tackle this hover issue.
Browser Information

Browser Name, Version: Chrome 55.0.2883.95 (64-bit)
Operating System: OX ElCapitan 10.11.6
Mobile, Desktop, or Tablet: iMac

Basically this challenge should be before the one mentioned before :  Adjust the Hover State of an Anchor Tag
http://beta.freecodecamp.com/en/challenges/applied-visual-design/adjust-the-hover-state-of-an-anchor-tag

@ipreira thanks for raising this point - you're right about the pseudo-class selectors not having been introduced yet. I think we can improve this challenge by removing the requirement that the box shadow effect needs to be on the hover state. (The challenge is really about how to use the box-shadow property, anyway).
I hesitate to reorder the challenges so Adjust the Hover State of an Anchor Tag challenge comes first, because that one isn't using the "card" example like the others in this series.
The changes to Add a Box-shadow to a Card-like Element would be:

Re-word the instructions part of the challenge text (last paragraph) to say:

The element now has an id of thumbnail. With this selector, use the example CSS values above to place a box-shadow on the card.

Remove the hint
Update the first test and its message to remove the hover requirement

For anyone who would like to work on this, please refer to the Contributor's Guide and feel free to ask any questions in the Contributor's chat room. Thank you! 🎉 💻 🏅

@HKuz I'd like to take this one up!

Great, thanks @Manish-Giri 🎉 👍

@Manish-Giri I think you forgot to fix this -> "This technique can be used on any state of an element, but here you'll apply it on hover."

@Atomk Nice catch! I apparently missed that when I reviewed this fix.
Feel free to open a PR for this, since you spotted it! :blush:

@Greenheart Working on it, I'm also removing :hover from the next challenges until :hover is introduced. I'm sorry it is taking so much time, I'm just making sure I understand the contributor guidelines and Git from command line is less user-friendly than expected.

@Atomk No problem, take your time! The commandline can be pretty daunting at first, but over time it will become your best friend to get things done :relaxed:
If you have any questions, feel free to come chat with us in the Contributors room :smile:

@Manish-Giri We should add a link to id selectors as a refresher.

i'm trying to figure out how to work this;
The element now has an id of thumbnail. With this selector, use the example CSS values above to place a box-shadow on the card.
this is the css values;
box-shadow: 0 10px 20px rgba(0,0,0,0.19), 0 6px 6px rgba(0,0,0,0.23);
and this is the code i used;
.thumbnail {
box-shadow: 0 10px 20px rgba(0,0,0,0.19), 0 6px 6px rgba(0,0,0,0.23);
but it's mot working on the fcc site.

@enchill You should ask questions related to the challenges on the forum.  GitHub is used to report bugs with the challenge (i.e incorrect tests, incomplete instructions, etc.).  If forum members conclude there is a bug in the actual challenge, they will refer you here to create a new issue.


Allow users to configure their desired socketSecurityLevel

Thanks for the PR. The refactored codebase in #653 removed this property completely. Feedback is welcomed!

I'm going to go ahead and close this out as it is related to the old 3.X.X codebase. The 4.0.0 codebase should allow complete customization in regards to this now. Thanks again for the PR!


Multiple messages, some got lost.

When sending multiple messages (every time a touchMoved is fired for example) some of the messages are received as "null" on the Apple TV.
Any ideias?
Thanks

will look it up - Myabe will add a return type
On Wed, Sep 23, 2015 at 6:20 AM, ShUr1k3n<EMAIL_ADDRESS>wrote:

When sending multiple messages (every time a touchMoved is fired for
example) some of the messages are received as "null" on the Apple TV.
Any ideias?
Thanks
—
Reply to this email directly or view it on GitHub
https://github.com/vivianaranha/TvOS_Remote/issues/1.

--
Vivian Aranha

Can this library also do bi directional communication? Is this Bluetooth or WiFi/IP based? It works wonderfully. Very nice! Thank you!

It works on Bluetooth. If you want I can make it such that it support bodirectional. Currently I am seeing some bugs. So working on those. Will update as soon as I get opportunity.
Also trying to add both wifi/Bluetooth functionality
Sent from my iPhone

On Sep 26, 2015, at 21:10, Roger Pingleton<EMAIL_ADDRESS>wrote:
Can this library also do bi directional communication? Is this Bluetooth or WiFi/IP based? It works wonderfully. Very nice! Thank you!
—
Reply to this email directly or view it on GitHub.

Bidirectional support would be awesome! And support for multiple devices at the same time would be even better still! Like I said, awesome job. It works so seamlessly. Very impressive!

Would be awesome if this would be WiFi-based, because with BLE it's not possible to connect more than 2 iDevices. Yeah, and bidirectional would be great too.

How do you know when the devices are linked?

I'm getting the following error on "TheApp" version. Any idea?
"ignoring file /AppleTVApp/TheApp/TheApp/libRemoteReceiver.a, missing required architecture x86_64 in file /AppleTVApp/TheApp/TheApp/libRemoteReceiver.a (3 slices)"
Undefined symbols for architecture x86_64:
"OBJC_CLASS$_RemoteReceiver", referenced from:
objc-class-ref in ViewController.o
ld: symbol(s) not found for architecture x86_64
clang: error: linker command failed with exit code 1 (use -v to see invocation)

@samtout: My guess is you're seeing this on the simulator? Could this be a matter of not being able to run in the simulator? If so, the problem should disappear on the actual Apple TV. Just a guess.

Thanks @streamside7
That's a good thought. But running it on the actual TV still gave me: "building for tvOS, but linking in object file built for iOS, for architecture arm64"
Any thoughts?

I'm getting a: TestController[1588:696670] *** Terminating app due to uncaught exception 'NSRangeException', reason: '*** -[__NSArrayM objectAtIndex:]: index 0 beyond bounds for empty array'
when executing sendInfo.
Any ideas why
The Receiver is not up and running, but should I get this error?

I need help with this too. I have the error Undefined symbols for architecture x86_64:
Any ideas?

I have provided the entire Project in Projects folder - Just use .h and .m files instead of Library.
 I created a bunch of apps using this - You can check an example at https://itunes.apple.com/us/app/twins-merge-em!!/id1044143620?mt=8

Zodor: I just got the same/similar error, and the reason for that was WIFI was turned off on my iPhone. Your error might have the same origin, or there could be something else that makes your iPhone and AppleTV unable to connect. That's why it crashes. My closed issue is: https://github.com/vivianaranha/TvOS_Remote/issues/4


How can toggle isChecked state in some country or region object?

below is my state in app, I want to toggle isChecked property in some country or region object.
I get a id group, such as [1, 2, 4] || [1, 5, 9, 1],these ids in array stands for serverId, continentId, countryId, regionId(if id group length is 4).I wanna use it to toggle isChecked state in country or region object.
state = {
  2: {
    id: 2,
    name: "King",
    continents: {
      12: {
        id: 12,
        name: "Africa",
        countries: {
          13: {
            id: 13,
            name: "South Africa"
            isChecked: true
          }
        }
      },
      6: {
        id: 6,
        name: "North America",
        countries: {
          8: {
            id: 8,
            name: "Denmark",
            isChecked: false,
          }
        }
      },
      5: {
        id: 5,
        name: "European",
        countries: {
          7: {
            id: 7,
            name: "France",
            isChecked: false,
          }
        }
      }
    }
  },
  1: {
    id: 1,
    name: "Elite",
    continents: {
      2: {
        id: 2,
        name: "South America",
        countries: {
          4: {
            id: 4,
            name: "Brazal"
            isChecked: false,
          }
        }
      },
      5: {
        id: 5,
        name: "European",
        countries: {
          9: {
            id: 9,
            name: "England",
            isChecked: false,
            regions: {
              1: {
                id: 1,
                name: "England-1",
                 isChecked: true,
              },
              2: {
                id: 2,
                name: "England-2"
                isChecked: false,
              }
            }
          }
        }
      }
    }
  }
}

I try to use R.lensPath and R.set to change the isChecked state like these:
// ids is [1, 5, 9, 1]
// I take it to lensPath
const len = R.lensPath(['1', 'contients', '5', 'countries', '9', 'regions', '1'])
// here  I set it to true, however
//  I wanna to toggle the state by state itself, instead of supplying the value
R.set(len, true, state);

I know these is a R.not operator to opposite it. I have no idea how to use it in my case.

Use R.over instead of R.set:
R.over(lens, not, state)

Note that this is missing the final 'isChecked' node:
const len = R.lensPath(['1', 'contients', '5', 'countries', '9', 'regions', '1'])
// presumably should be
const len = R.lensPath(['1', 'contients', '5', 'countries', '9', 'regions', '1', 'isChecked'])

I would recommend that you look to our Gitter Room or StackOverflow for questions like this.  We don't mind answering question here in issues the way some projects do, but you will likely get a quicker response in one of those forums.


Want to work on Arithmetic Slices problem in C++ (GSSOC '21)

🚀 Feature
Arithmetic Slices Problem
An integer array is called arithmetic if it consists of at least three elements and if the difference between any two consecutive elements is the same.
For example, [1,3,5,7,9], [7,7,7,7], and [3,-1,-5,-9] are arithmetic sequences.
Given an integer array nums, return the number of arithmetic subarrays of nums.
Input / Output Example :
Input: nums = [1,2,3,4]
Output: 3
Explanation: We have 3 arithmetic slices in nums: [1, 2, 3], [2, 3, 4] and [1,2,3,4] itself.
Have you read the Contributing Guidelines on Pull Requests?
Yes, I have.
Motivation
The problem felt like a good starting point.
Pitch
Could not find the problem in the C++ codebase nor any such issue raised. Requesting you to assign me this problem. I am a participant of GSSOC '21.

Can you please assign this to me as a part of GSSOC '21?

@ShreyashRoyzada Hi Shreyash, I assigned you. Happy Coding!😊


ES2015 or ES2017 for new ArrayBuffer(length)

The definition of new ArrayBuffer(length) is changed in ES2017.
( I pushed #1959 before I recognise it. I would like to get opinions. )
For example,
ES2015
new ArrayBuffer(1.1) should throw RangeError.
ES2017
new ArrayBuffer(1.1) creates ArrayBuffer with byteLength=1.
I would like to follow ES2017's definition.
Major engines (v8/spidermonkey/jsc) already work like ES2017 spec. ( Or ES specification may be updated to follow major engines.  ES2016 specification was same to ES2015. ).

I prefer following ES2017 if there are some conflict with ES2015, since that is major engines' choice.
Maybe we should change the profile name es2015-subset to es2017-subset ?

@jiangzidong Thank you for opinion. I also thought that es2015-subset is correct name if we follow latest specification like new ArrayBuffer(length). However, ES specification will be changed for several forthcoming years ( es2018, es2019, ... ), and there may be feature X, Y,  that major specification follows es2015 yet. I cannot find good name that covers this situation. So I would like to keep the current name - es2015-subset.

Yes, make sense.


[C#] server sending responseheader in unary call not received by the client until the whole response is returned

What version of gRPC and what language are you using?
2.26
What operating system (Linux, Windows,...) and version?
Linux
What runtime / compiler are you using (e.g. python version or version of gcc)
dotnet 3.1
What did you do?
Client

makes a unary request to the server.
call await response.ResponseHeaderAsync
call await response.ResponseAsync

Server

call await WriteResponseHeadersAsync()
do some heavy processing, then send the response

What did you expect to see?
Client to get the header first
Then after a while get the response.
What did you see instead?
ResponseHeaderAsync doesn't return until the server send the actual response.

It might be worth seeing if you could reproduce this with a more recent gRPC version. v2.26 was released in 2019. Some of the ResponseHeaderAsync tests might be informative for you here as well, they exercise this behavior. I'm not sure we're looking at a bug here, this might be a better question for another forum (stackoverflow, or the grpc mailing list). @jtattermusch, please take a look, maybe my assessment is off.

hi @drfloob , i did try the latest build v2.38 and it has the same behavior.  Perhaps this is by design?  there is no way to send the responseheader back earlier in a unary call?

hi @drfloob , i did try the latest build v2.38 and it has the same behavior. Perhaps this is by design? there is no way to send the responseheader back earlier in a unary call?

It should definitely be possible for the server to send the response headers independently from sending the first response (That is send response headers form the server side handler and receive them on the client regardless of how late you return the response from the server-side handler).
Can you please share a ready-to-run reproduction of the problem (e.g. as a github repo)? As @drfloob  has pointed out, we basically have a test that makes sure that the above works, so it would be interesting to see what is it that you're doing differently.

The test below reproduces the issue. It'll only pass if the Computation delay is less than the 10s wait time of the ResponseHeadersAsync.
public async Task ResponseHeadersAsync_UnaryCall()
        {
            helper.UnaryHandler = new UnaryServerMethod<string, string>(async (request, context) =>
            {
                await context.WriteResponseHeadersAsync(headers);
                Console.WriteLine("Header sent...");
                Console.WriteLine("Waiting for computation...");
                await Task.Delay(20000);
                return "PASS";
            });
            var call = Calls.AsyncUnaryCall(helper.CreateUnaryCall(), "");
            Metadata responseHeaders = null;
            Task t1 = Task.Run(async () => {
                Console.WriteLine("Waiting for header...");
                responseHeaders = await call.ResponseHeadersAsync;
                Console.WriteLine("Header received!");
            });
            Assert.True(t1.Wait(10000));
            Assert.AreEqual(headers.Count, responseHeaders.Count);
            Assert.AreEqual("ascii-header", responseHeaders[0].Key);
            Assert.AreEqual("abcdefg", responseHeaders[0].Value);

            Assert.AreEqual("PASS", await call.ResponseAsync);
        }

As posted above, we've been able to confirm the behavior reported by this issue.
While this is something that could be fixed (basically by internally splitting the HandleUnaryResponse handler into two separate handlers and registering two C-core operation batches while creating unary call instead of just one), it would be a relatively complex change to do so. It would probably also have some performance impact since it would increase the overhead for all unary calls (needing to register / process two completion notifications instead of just one). Given that Grpc.Core is currently in maintenance mode (see https://grpc.io/blog/grpc-csharp-future/) I don't think it's worth the work to invest into a fix (which can potentially introduce other issues or harm performance) now, also since this is also likely a behavior that doesn't affect many users (for most unary RPC, there's really no needed receive response headers and the response message is two separate events).
I think the best way forward is to simply keep the imperfect behavior for Unary and Client streaming calls the way it is (acknowledging the limitations in terms of receiving response headers in separately advance).
For the less frequent scenarios where you really need to be able to access the response headers prior to reading the response message, you can use a server-streaming or duplex streaming call as a workaround (for these  types of calls things behave the way they should and we have tests in place to ensure that).

thanks for following up on this @jtattermusch .


Add support for 1.9 schema

Changes:

updates to the net8.0
adds support for 1.9 schema
Adds the test files for all schema versions up to 1.9

Microsoft Reviewers: Open in CodeFlow

@ryfu-msft #518 also updates to net8.0. I'll probably complete the remaining work on Azure pipelines, so you can merge that.

@ryfu-msft #518 also updates to net8.0. I'll probably complete the remaining work on Azure pipelines by tomorrow, so you can merge that.

@ryfu-msft #518 also updates to net8.0. I'll complete the remaining work on Azure pipelines by tomorrow, so you can merge that.


CRLF Injection - Improper Output Neutralization for Logs

Hi, team we are using Piwik for our Android applications.
We are getting these security vulnerabilities when doing the security scanning.
Source file names are
DefaultPacketSender.java  -   org/piwik/sdk/dispatcher/
Tracker.java    -   org/piwik/sdk/
Method name is void zzk(android.content.Intent)
Please advise how to resolve this issue.

There is no method with that signature in either class.
Imho, CRLF Injection is not really a concern here. This library doesn't log by default. If this is a security issue for your project and you require logging then you can write a custom Tree for the logging library Timber used here, that escapes CRLF sequences.


develop/hardware: Add info on Up Squared UART communication

Change-type: patch
Connects-to: Internal thread https://balena.zulipchat.com/#narrow/stream/403752-channel.2Fsupport-help/topic/Can't.20access.20the.20serial.20port.20on.20Up.20Squared.20device/near/404364945

Please make sure to read the CONTRIBUTING document before opening the PR for relevant information on contributing to the documentation. Thanks!

@balena-ci I self-certify!


smoothing fix

smoothing: integrate option for continuousOutput + updateDelay fix from hyperion.ng
logger: fix possible buffer overflow

That's pretty fast!
Oh one thing, am i right that the webserver now goes crazy? As there is no webdir and no config available? Could you disable it?

Oh one thing, am i right that the webserver now goes crazy?
webserver is included only when compiling for qt5. In current hyperion we won't use qt5 that means no webserver, no problem

A, didn't had a deeper look, good to know :)
Thank you


docs: add an usage example

What kind of change does this PR introduce?
Bug fix, feature, docs update, ...
What is the current behaviour?
You can also link to an open issue here.
What is the new behaviour?
...
Does this PR introduce a breaking change?
What changes might users need to make in their application due to this PR?
Other information:
Please check if the PR fulfills these requirements:

[ ] Tests for the changes have been added
[ ] Docs have been added / updated
[ ] Relevant Changeset has been added

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
0

Relevant Lines:
0

💛  - Coveralls

coverage: 0.0%. remained the same
when pulling fc03daeb737d862ae56f9de901f58edd0302f65d on docs-example
into b244e96acf65bf160382ab6b26bba158b009265f on master.

coverage: 0.0%. remained the same
when pulling fc03daeb737d862ae56f9de901f58edd0302f65d on docs-example
into b244e96acf65bf160382ab6b26bba158b009265f on master.


When Yankpad is in effect, company-completions stop showing up.

I have configured Yankpad with the default configurations. I also have Yasnippet and Company-mode installed. Yankpad completions are working, but then my company completion disappear. Here are my configurations:

Yasnippet

(use-package yasnippet
  :ensure t
  :diminish yas-minor-mode
  :config
  (yas-global-mode 1) ;; Always have this on for when using yasnippet syntax within yankpad
  ;; respect the spacing in my snippet declarations
  (setq yas-indent-line 'fixed))

Yankpad

(use-package yankpad
  :ensure t
  :defer 10
  ;; :init
  ;; I prefer to set org-mode's default directory and have Yankpad read my files there
  ;; (setq yankpad-file "~/Dropbox/emacs/org/yankpad.org")
  :config
  (bind-key "<f9>" 'yankpad-map)
  (bind-key "<f12>" 'yankpad-expand)
  ;; If you want to complete snippets using company-mode
  (add-to-list 'company-backends #'company-yankpad)
  ;; If you want to expand snippets with hippie-expand
  (add-to-list 'hippie-expand-try-functions-list #'yankpad-expand))

Company

(setq tab-always-indent 'complete)
(add-to-list 'completion-styles 'initials t)

(when (maybe-require-package 'company)
  (add-hook 'after-init-hook 'global-company-mode)
  (with-eval-after-load 'company
    (dolist (backend '(company-eclim company-semantic))
      (delq backend company-backends))
    (diminish 'company-mode)
    (define-key company-mode-map (kbd "M-/") 'company-complete)
    (define-key company-mode-map [remap completion-at-point] 'company-complete)
    (define-key company-mode-map [remap indent-for-tab-command] 'company-indent-or-complete-common)
    (define-key company-active-map (kbd "M-/") 'company-other-backend)
    (define-key company-active-map (kbd "C-n") 'company-select-next)
    (define-key company-active-map (kbd "C-p") 'company-select-previous)
    (define-key company-active-map (kbd "C-d") 'company-show-doc-buffer)
    (define-key company-active-map (kbd "M-.") 'company-show-location)
    (setq-default company-dabbrev-other-buffers 'all
                  company-tooltip-align-annotations t))
  (global-set-key (kbd "M-C-/") 'company-complete)
  (when (maybe-require-package 'company-quickhelp)
    (add-hook 'after-init-hook 'company-quickhelp-mode)))

;; Suspend page-break-lines-mode while company menu is active
;; (see https://github.com/company-mode/company-mode/issues/416)
(with-eval-after-load 'company
  (with-eval-after-load 'page-break-lines
    (defvar-local sanityinc/page-break-lines-on-p nil)

    (defun sanityinc/page-break-lines-disable (&rest ignore)
      (when (setq sanityinc/page-break-lines-on-p (bound-and-true-p page-break-lines-mode))
        (page-break-lines-mode -1)))

    (defun sanityinc/page-break-lines-maybe-reenable (&rest ignore)
      (when sanityinc/page-break-lines-on-p
        (page-break-lines-mode 1)))

    (add-hook 'company-completion-started-hook 'sanityinc/page-break-lines-disable)
    (add-hook 'company-after-completion-hook 'sanityinc/page-break-lines-maybe-reenable)))

Can you point me in the right direction as to where I'm going wrong?

Hi! I'm not using company-yankpad myself (and haven't written the code for it), but if you are then I guess you shouldn't use hippie-expand. I'll list my own config here, it might be of some help.
(use-package yankpad
  :demand t
  :config
  (setq yankpad-category-heading-level 2))

(use-package company
  :defer 10
  :diminish
  :commands company-mode-on
  :preface
  (defun ess/company-complete (&optional _)
    "Make sure company-mode is turned on, then run `company-complete'."
    (interactive)
    (company-mode-on)
    (company-complete)))

(use-package hippie-expand
  :quelpa nil
  :general ("C-ö" 'hippie-expand)   ;; I use general.el for keybindings
  :init
  (setq hippie-expand-try-functions-list
        '(yankpad-expand
          ess/company-complete)))

Hi! I'm not using company-yankpad myself (and haven't written the code for it), but if you are then I guess you shouldn't use hippie-expand. I'll list my own config here, it might be of some help.
(use-package yankpad
  :demand t
  :config
  (setq yankpad-category-heading-level 2))

(use-package company
  :defer 10
  :diminish
  :commands company-mode-on
  :preface
  (defun ess/company-complete (&optional _)
    "Make sure company-mode is turned on, then run `company-complete'."
    (interactive)
    (company-mode-on)
    (company-complete)))

(use-package hippie-expand
  :quelpa nil
  :general ("C-ö" 'hippie-expand)   ;; I use general.el for keybindings
  :init
  (setq hippie-expand-try-functions-list
        '(yankpad-expand
          ess/company-complete)))

Oh my bad. I didn't read the comment in the example configuration where you clearly stated to use one or the other. I'll try it out now.

Oh my bad. I didn't read the comment in the example configuration where you clearly stated to use one or the other. I'll try it out now.

In the end I decided to leave out company/hippie expansions. I can't seem to get them to work with Yankpad no matter what I do.  Here's my current config in-case anyone has similar problems.
(use-package yankpad
  :ensure t
  :config
  (bind-key "<f9>" 'yankpad-map)
  (bind-key "<f12>" 'yankpad-expand)
  (bind-key "C-c s" 'yankpad-insert))

In the end I decided to leave out company/hippie expansions. I can't seem to get them to work with Yankpad no matter what I do.  Here's my current config in-case anyone has similar problems.
(use-package yankpad
  :ensure t
  :config
  (bind-key "<f9>" 'yankpad-map)
  (bind-key "<f12>" 'yankpad-expand)
  (bind-key "C-c s" 'yankpad-insert))


Can't access my account, while username and password that i entered is correct

Can't access my account, while username and password that i entered is correct
Hello Support team Sendgrid, I hope you've had your coffee already today.
I want to ask something about my account, I can't access or log in to my dashboard after inserting username and password. the response after inserting my username password system informs me "Your username or password is invalid.". so I reset my password, and after resetting my password. this problem happens again.
I've reset my password up to three times. unfortunately, I still can't log in.
what happens with my account ? can you check it for me?
please fix this problem. I should launch my app 2 days from today.

Hello @balikcode,
We're sorry you're having this issue. You can access support contact options by going to https://support.sendgrid.com and clicking "Trouble Logging in?". From there, you can tell us a little more about your problem without having to log in, and one of our fantastic support people will get back to you. To expedite this process, once you've got a ticket number, leave it here in the comments and I'll try and help move it along for you. Hope this helps!
The Twilio SendGrid Docs Team

Hello @Whatthefoxsays
yes, I was open support ticket before open this issue. unfortunately, the ticket is not replied for 2 days. its make me very confused. how to login and use my account, i was paid the bill but the system can't use. can you give me another options how to contact the support team ?
Cheeers

Hi @balikcode,
Unfortunately I cannot unlock your account for you. I'll need a ticket number or an account ID/email associated with the account to be able to help you.

Hey @Whatthefoxsays
Should i share my ticket number here ?

Hi @balikcode,
You can do that or share the email address you use to sign into your account with, we can find your information.
Thanks!

this happens to me all the time as well! i have no idea why, after a few hours, i enter the same password and username and it works. seems to be a problem on sendgrid's servers

Hi @Whatthefoxsays
Technically this is the root cause, the resource /tokens were rejected requests come.

After a few minutes, I'll be logged out and have to log in again.
I faced with this issue 3, 4 times per day but every time it happens all status at the link https://status.sendgrid.com/ are green
A little bit worry because we are going to go live the application

hi @chitampopo @johnnyperdomo
i am accessing sendgrid today, and also try to log in. now I can access my account, idk what happens with my account a few days ago as long as my design library and contact in my account doesn't lost its okay.
have you tried to log in today?
i want to close this issue, please report me if you're successful to access your account

It works for me yeah.


When getting badges, return 500 in response

You can see from my extension home page VS Code REST Client

VSMarketplaceBadge dependence on shields.io for SVG generate.
shields.io is down now.
I'll consider independence from shields.io.


[Refactor] app icon

Purpose

add app icon

Does this introduce a breaking change?

[ ] Yes
[x] No

Pull Request Type
What kind of change does this Pull Request introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[x] Refactoring (no functional changes, no api changes)
[ ] Documentation content changes
[ ] Other... Please describe:

@iaulakh  has Alex Pereira reviewed the icons?

@iaulakh has Alex Pereira reviewed the icons?

Updated the icon, which is from Figma https://www.figma.com/file/BOeihN2NJE7ROdyqbihCxi/Mobile-UI-Library?node-id=3260%3A81719

@phileo99 yes, Alex shared Icon.


add/tweak doom highlights

So this PR adds and changes a few things, I was hesitant about opening it since I have these things in my config anyway but I though maybe other people would like them so decided to contribute the changes.
I've laid it out as a series of separate commits, so I can remove anything that is too controversial or that you would prefer not to include. Regarding the highlight changes since these are easy to override I'm happy to take things out or maybe make them optional with variables 🤷🏾
So changes include:

Darkening the Pmenu - this change makes the completion menu dark which matches with doom better, I get that it's maybe controversial, so I can make it optional or remove it.

This plugin

Doom emacs

2. Add Substitute highlight
This plugin

Doom emacs

Remove TSVariable highlight - this one is maybe controversial, but I've noticed some new themes are all highlighting this, which I think is quite weird. Almost all syntax themes don't highlight variables since they are super common in programming, so it really seems odd to distinguish them and turn the whole file into one colour.

I could for show a screen grab from doom as well that shows that it doesn't do that, but I think this is more about how people are handling treesitter highlights.

Make Telescope highlights optional - I get not making this theme too complex with options may be an issue, but I find that I quite liked the default styling or minimal colouring of telescope (personal preference I know) and trying to remove the highlights this plugin adds is kind of laborious, so this option allows it to be switched off.

I guess that this could apply to other plugins, but tbh I think it handles other plugins quite nicely so 🤷🏾. Also didn't want this to get too long.

Hey, thank you very much for those changes. I think they will make it more doomy and customizable. I really appreciate them very much :heart:
I don't see why to reverse any of the changes (personally), they all look great to me and follow my thinking (the freedom to choose what you want to use and what not, since we don't all have the same tastes). In addition, as this is a port of the doom emacs' colorscheme, it's good to make it as similar as possible with the original one!

Regarding to highlight of TSVariable

I see it a bit controversial not gonna lie. However I see it well to make that change. It becomes a little strange to see everything colorful and as you said, variables are the most common thing so I don't see why continue to color them.

Regarding to highlight of Telescope

I think it could stay. Maybe there are more people who prefer the style that it brings by default just like you and this will make it less tedious to use the default one.
The only thing I'll ask you if you have time to is that I recently added the light variant of doom-one and I was wondering if you could apply those changes to that variant as well to cover all of that at once. I think it'll only be necessary to apply from 2 to 4 since the change to the Pmenu I think isn't necessary in the light variant yet since I'm still tweaking it.
Again thank you very much, it's a very good job to improve the colorscheme :+1:

@NTBBloodbath 👍🏾 sounds good I'll update the README as well as the light variant of the theme.

Done 🚀

Thank you so much!


Changing parameters in RTIMULib.ini has no effect on sensor read / poll rate

Just as stated in the title, I have tried editing a copy of RTIMULib.ini (which is in my current working directory) and the values of the parameters in this file appear to have no effect on the SenseHAT sensors poll rate.
I suspect rather than this being a bug in the Python SenseHAT library, this might be due to me misunderstanding how to set these parameters.
Some questions:
Does a python script look in the current working directory for the file RTIMULib.ini ?
Does a python script read this file and try to set the values for the SenseHAT sensors using the values contained in this file?
Reason for asking: It is possible that only the C/C++ implementations which use RTIMU read this file. (But seems unlikely?)

Further info: The SenseHAT uses the LSM9DS1 chip for acceleration, angular rotation velocity and magnetic field sensor measurements.
I have written a Python script to repeatedly read the accelerometer. The maximum rate I am able to obtain is about 50 - 60 Hz, whereas this sensor should have a maximuim sample rate of 1 kHz.
I have tried playing around with the relevant values in RTIMULib.ini under the section for LSM9DS1, but I cannot get anything higher than this 50 - 60 Hz. Situation is even worse when reading other sensors as well, with maximum rates being about 20 Hz.

I also tried playing around with the provided C++ examples, but found the same behaviour. I know for certain that the C++ codes read the RTIMULib.ini file, but again changing the values seems to have no effect.
Bizzare. Can anyone help or offer any advice? Like I say it might not be a bug - this might be me misunderstanding something.


Support multiple migration dirs

Not sure if this is a good idea.
Let's say we have 2 branches and in each branch we have 2 different modules with their migration files.
When the branches are merged into an upstream branch, we might run into issues because we don't have much visibility on the order of the migration files.


Compatible with both underscore and lodash

First up, really nice little tool, thanks for making it! Has been really useful in speeding up the project I've been working on.
Basically we use lodash instead of underscore and I ran into an issue when I used this with the latest version. I appreciate that you specifically say it's for use with underscore, but as it's just a one-liner to work with both I though it would be worth doing.
.take is an alias of .first in underscore, but in lodash .first is explicitly the first item, and .take is the first x items.
What do you think, worth a merge to maximise compatibility? :)

Sure, sounds good.  Wanna update the README as well to indicate that it'll also work with lodash now?  Thanks!

Good idea, have done that now :)

Thanks, merged!

@pkaminski should you update the bower version? (don't know)

Yeah, I probably should.  Done.
On Wed, Jun 3, 2015 at 4:36 AM, Sarkie<EMAIL_ADDRESS>wrote:

@pkaminski https://github.com/pkaminski should you update the bower
version? (don't know)
—
Reply to this email directly or view it on GitHub
https://github.com/pkaminski/digest-hud/pull/5#issuecomment-108321881.

--
Piotr Kaminski<EMAIL_ADDRESS>"That bun is dirty.  Don't eat that bun."


[6.0][Concurrency] Fix build in actors as locks configuration

Description: Fixes build due to placement of function inside #if !SWIFT_CONCURRENCY_ACTORS_AS_LOCKS.
We have to start testing these configurations, it's gotten to a point where it's not possible to just keep it all in your head every time.
Risk/reward: low risk, moves a function out of wrong #if. Makes the build compile again
Reviewed by: @bnbarham
Original PR: https://github.com/swiftlang/swift/pull/75004
Radar: resolves rdar://131195175

Description:
Scope/Impact:
Risk:
Testing:
Reviewed by: @hborla
Original PR:
Radar:

@swift-ci please test


Omitting trailing slash results in 301 redirect to relevant Pod IP

Summary
Kong proxy is redirecting traffic directly to a Pod's IP via location header in 301 redirect response to requests where trailing slash is omitted. Adding trailing slash results in no redirect, works fine.
E.g. location header location: http://<IP_ADDRESS>/docs/. That IP is cluster local.
Kong Ingress controller version
kong-docker-kubernetes-ingress-controller.bintray.io/kong-ingress-controller:1.2
Kubernetes version
Client Version: version.Info{Major:"1", Minor:"19", GitVersion:"v1.19.7", GitCommit:"1dd5338295409edcfff11505e7bb246f0d325d15", GitTreeState:"clean", BuildDate:"2021-01-13T13:23:52Z", GoVersion:"go1.15.5", Compiler:"gc", Platform:"darwin/amd64"}
Server Version: version.Info{Major:"1", Minor:"19", GitVersion:"v1.19.9", GitCommit:"c1ad2bc44e0c7f19a9db0879f7182ff556a8e5b4", GitTreeState:"clean", BuildDate:"2021-05-19T22:27:45Z", GoVersion:"go1.15.8", Compiler:"gc", Platform:"linux/amd64"}

Environment

Cloud provider: GKE

What happened
Relevant proxy logs output:
<IP_ADDRESS> - - [16/Jun/2021:22:43:23 +0000] "GET /docs HTTP/1.1" 301 134 "-" "Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/91.0.4472.101 Safari/537.36"

Here is the Ingress YAML:
  kind: Ingress
  metadata:
    annotations:
      cert-manager.io/cluster-issuer: letsencrypt-prod
      konghq.com/https-redirect-status-code: "301"
      konghq.com/preserve-host: "false"
      konghq.com/protocols: https
      konghq.com/strip-path: "false"
      kubernetes.io/ingress.class: kong
      kubernetes.io/tls-acme: "true"
...
  spec:
    rules:
    - host: deviceplane.com
      http:
        paths:
        - backend:
            serviceName: deviceplane-www
            servicePort: 80
          path: /
          pathType: ImplementationSpecific
    - host: www.deviceplane.com
      http:
        paths:
        - backend:
            serviceName: deviceplane-www
            servicePort: 80
          path: /
          pathType: ImplementationSpecific
    - host: downloads.deviceplane.com
      http:
        paths:
        - backend:
            serviceName: deviceplane-downloads
            servicePort: 80
          path: /
          pathType: ImplementationSpecific
    - host: cloud.deviceplane.com
      http:
        paths:
        - backend:
            serviceName: deviceplane
            servicePort: 80
          path: /
          pathType: ImplementationSpecific

Here is the KongIngress YAML (for timeouts):
apiVersion: v1
items:
- apiVersion: configuration.konghq.com/v1
  kind: KongIngress
  metadata:
    annotations:
      kubectl.kubernetes.io/last-applied-configuration: |
        {"apiVersion":"configuration.konghq.com/v1","kind":"KongIngress","metadata":{"annotations":{},"name":"deviceplane","namespace":"deviceplane"},"proxy":{"connect_timeout":60000,"read_timeout":60000,"write_timeout":60000}}
    creationTimestamp: "2021-06-02T05:49:07Z"
    name: deviceplane
    namespace: deviceplane
  proxy:
    connect_timeout: 60000
    read_timeout: 60000
    write_timeout: 60000

Expected behavior
Expect no redirect at all. Or expect redirect to not aim at Pod IP but instead point to LB IP.
Steps To Reproduce

Visit deviceplane.com/docs/ confirm success
Visit deviceplane.com/docs confirm failure

Closing as stale. If anyone is still having this issue on a modern supported version of the ingress controller (2.2 is the latest at the time of writing) please do comment here and let us know what version you're experiencing this on and we'll be happy to re-open and re-assess.

I'm getting this on 2.8 of kong
My Ingress
`
apiVersion: networking.k8s.io/v1
kind: Ingress
metadata:
name: ams-ui                         # Looks like this and the kongIngress name have to match the service
namespace: ams-kubernetes
annotations:
konghq.com/https-redirect-status-code: "302"
konghq.com/methods: GET,POST,PUT
konghq.com/preserve-host: "false"
konghq.com/protocols: http
spec:
ingressClassName: kong
rules:
- http:
paths:
- path: /ams-ui/.*
pathType: Prefix
backend:
service:
name: ams-ui
port:
number: 80
apiVersion: configuration.konghq.com/v1
kind: KongIngress
metadata:
name: ams-ui
namespace: ams-kubernetes
upstream:
healthchecks:
active:
healthy:
http_statuses:
- 200
- 302
interval: 5
successes: 3
http_path: /ams-ui/health
type: http
unhealthy:
http_failures: 1
interval: 5
`

I can reproduce this in a second way.   I have a Tomcat server with JSP pages, one request issues a redirect to another JSP, same result, the pod IP is used, this time I know the incoming request reached the Tomcat Pod

@axelsean I see that you posted using Kong Gateway 2.8, but I may have missed what ingress controller version you're running. Can you let me know which version of the KIC is deployed in this case?

The second issue I fixed by setting preserveHost on the route

KIC image is - kong/kubernetes-ingress-controller:2.3.1
From https://raw.githubusercontent.com/Kong/kubernetes-ingress-controller/master/deploy/single/all-in-one-postgres.yaml

@shaneutt Thx - If you need me to turn on debug logging I can easily do this - just let me know how

You can do --log-level debug as a command line flag for the controller, or CONTROLLER_LOG_LEVEL=debug as an ENV var.

Turned that on for the ingress controller - doesn't seem to show anything - just this repeated:

@axelsean your and the original issue likely stem from behavior of the upstream application. It likely expects configuration that indicates how it should generate URLs, and absent that configuration, will provide a best effort guess based on its listens (in this case, it is indeed listening on the Pod IP). Brief review suggests this might be handled by Tomcat's Host and Context configuration, but I'm not familiar with Tomcat and can't say for sure. I'll close this as such, but if you can replicate this with an httpbin instance or similar, please open a new issue.
Kong itself wouldn't be able to formulate this response since our only native in-proxy redirect functionality is for HTTP->HTTPS redirects, which wouldn't apply here (your Ingress is configured to be HTTP-only). You should be able to confirm that the redirect response is being forwarded from upstream by using tcpdump and filtering on the Pod IP with a single upstream application replica (maybe--see the other odd stuff section later).
Those responses, and other responses generated directly by Kong (e.g. the 404 no route response) will only include X-Kong-Response-Latency in response headers. If you're also seeing X-Kong-Proxy-Latency, your response is from upstream.
$ curl -sv http://<IP_ADDRESS>/httpbin
*   Trying <IP_ADDRESS>:80...
* Connected to <IP_ADDRESS> (<IP_ADDRESS>) port 80 (#0)
> GET /httpbin HTTP/1.1
> Host: <IP_ADDRESS>
> User-Agent: curl/7.82.0
> Accept: */*
>
* Mark bundle as not supporting multiuse
< HTTP/1.1 426
< Date: Tue, 17 May 2022 20:43:21 GMT
< Content-Type: application/json; charset=utf-8
< Connection: keep-alive
< Connection: Upgrade
< Upgrade: TLS/1.2, HTTP/1.1
< Content-Length: 39
< X-Kong-Response-Latency: 0
< Server: kong/2.8.1

I used a slightly modified version of your Ingress (1440_test.yaml.txt) to test this using another application: 1440_results.txt. Note that I'm using konghq.com/path: /anything on the httpbin Service since you don't have strip_path: true set and httpbin won't recognize your application's paths.
There's something else odd going on where using Prefix shouldn't actually get you any response for /ams-ui with that Ingress because of the way our route generation works. The third "/ams-ui/.*$","/ams-ui/.*/" path set is what that will generate. /ams-ui will not match it and does return a 404 for me. You may have some other Ingress that's actually handling this (or an additional Kong instance--the 301 should appear in the access logs whether it came from Kong or upstream), but the general points about this redirect coming from upstream remain true:
13:17:46-0700 esenin $ curl -sk https://localhost:8999/routes | jq ".data[] | .paths"
[
  "/ams-ui-test2/.*"
]
[
  "/ams-ui-test3"
]
[
  "/ams-ui/.*$",
  "/ams-ui/.*/"
]
[
  "/ams-ui-test1$",
  "/ams-ui-test1/"
]
[
  "/httpbin"
]

You should not provide your own regular expressions with pathTypes other than ImplementationSpecific. We generate our own regexes to comply with the Ingress spec, and combining the two can result in unexpected effects (though I think it's harmless in this case). Both Prefix and ImplementationSpecific with non-regex paths should probably work for what you want here: Kong's default behavior is essentially the official Prefix with the caveat that we'll also match continued strings that aren't separated with a slash (e.g. the /ams-ui-test3 will also match /ams-ui-test3whatever in addition to the official Prefix matches, /ams-ui-test3, /ams-ui-test3/, and /ams-ui-test3/whatever).

I wasn't seeing any error in the upstream, it was a simple nginx server with a React app - the redirect worked when we hit that application outside the kong environment
I also have a service ams-ui-metadata - so I need the trailing slash
Are these two what's needed ?
      - path: /ams-ui/
        pathType: Prefix

another ingress
      - path: /ams-ui-metadata/
        pathType: Prefix

You want either Exact (/ams-ui and /ams-ui/ only) or ImplementationSpecific (/ams-ui along with /ams-ui/whatever, /ams-uiwhatever, etc.). The Prefix spec excludes matches without the trailing slash.

Putting what I have here - for reference - I have this working now:
Used a Prefix for the version with / and exact for the without - passes the request to the service which then does the needed redirects
 rules:     - http:         paths:           - path: /ams-ui/.*             pathType: Prefix             backend:               service:                 name: ams-ui                 port:                   number: 80           - path: /ams-ui             pathType: Exact             backend:               service:                 name: ams-ui                 port:                   number: 80


[fix][Collapsible] CSS fix for variable inheritance when Collapsibles are nested

:tada: This PR is included in version 1.3.5-next.3 :tada:
The release is available on:

npm package (@next dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Revert axios (1.6.0 to 0.27.2)

Reverting axios (update previously made in https://github.com/josephdadams/TallyArbiter/pull/607).
In order to step to axios 1.6.0 the change in axios from Common JS to ES Module needs to be handled, this it not quickly done. This revert is only to make a release more sucessfull. A new issue will be created to track this technical debt.
When this PR has been merged to main is the suggestion that we go ahead with a new release since there are several queued up fixes that would be good to have in a formal release package.
Fixes https://github.com/josephdadams/TallyArbiter/issues/640

There is a PR that will be submitted soon that will bring in ember+ support so let’s wait for that one before a new release


[General behavior] Migrate Widget methods to classes

By fixing issue #2603, I have discovered the summary_card file, where a single Widget contains 700 lines of code! 😱
This is generally a bad coding behavior for Flutter, knowing how the different trees work.
I think we should take some time to go through the code and split this kind of Widgets.
For the first step, I suggest listing all files and they start fixing them gradually.

I totally agree with you, the Fdroid thing is my priority right now.
But as #2603 highlights, this kind of reworking is sometimes required to fix side-effects errors

Then I would encourage you to create a PR that just adds one TODO on top of each suspicious file (a dozen I guess), something like // TODO(g123k): if you're working on that file and if you have time, please split that file / that widget into smaller bits (e.g. less than 300 lines).
This way, next time any developer that edits one big file will be aware. And in the meanwhile everybody works on higher priorities.


Setup automated tests and linters on CI

Is this a bug report, a feature request, or a question?
Choring! 🧹
I like the project and would like to last long and prosper. That's why I propose to setup a CI (e.g. CircleCI or Travis) that would run following jobs:

[ ] unit/integration tests with Jest – for components and utility functions (feel free to use react-native-testing-library to ease component testing)
[ ] e2e tests with Detox – these are nice because they allow testing on actual device with fake gestures
[ ] ESLint or TSLint - eslint/recommended, eslint-config-react-app and eslint-config-prettier are a good start I think, or we can go with tslint
[ ] type checking – add ~Flow or~ TypeScript to the project ~(Flow is easier to set, but TS gets wider adoption and has lots of 3rd party typings, so you decide)~

Unfortunately I likely won't find time to set any of this up, but I wanted to start this initiative anyway, because somebody surely will get some of this done :)
@bd-arc I think the best first step would be for you to set up circle and base on this configuration from react-native-paper, I copy it all the time for new projects :P

Thanks @thymikee for the issue and the miscellaneous ideas!
I don't currently have time to tackle this, but I'd love to see someone jump in :-)
Regarding type checking, it has to be TypeScript since this is what we use on all of our projects.

Sorry, please allow me to advertise for my open source library! ~
I think this library react-native-reanimated-carousel will solve your problem. It is a high performance and very simple component, complete with React-Native reanimated 2


Cannot start kube-proxy: Needs Config File

I have been following this document to set up a cluster from scratch on a bunch of local vagrant machines. I have been running into issues with kube-proxy though.
When i tried to create the kube-proxy service with

cat > kube-proxy.service <<EOF
[Unit]
Description=Kubernetes Kube Proxy
Documentation=https://github.com/kubernetes/kubernetes
[Service]
ExecStart=/usr/local/bin/kube-proxy \
--cluster-cidr=<IP_ADDRESS>/16 \
--kubeconfig=/var/lib/kube-proxy/kubeconfig \
--proxy-mode=iptables \
--v=2
Restart=on-failure
RestartSec=5
[Install]
WantedBy=multi-user.target
EOF
When i did this, journalctl reported the following:

Jan 02 17:47:15 slave-1 kube-proxy[1568]: W0102 17:47:15.774208    1568 server.go:185] WARNING: all flags other than --config, --write-config-to, and --cleanup are deprecated. Please begin using a config file
Jan 02 17:47:15 slave-1 kube-proxy[1568]: F0102 17:47:15.869049    1568 helpers.go:119] error: no configuration has been provided
Jan 02 17:47:15 slave-1 systemd[1]: kube-proxy.service: main process exited, code=exited, status=255/n/a
Jan 02 17:47:15 slave-1 systemd[1]: Unit kube-proxy.service entered failed state.
Jan 02 17:47:15 slave-1 systemd[1]: kube-proxy.service failed.

Alright, i understand that i need to give a config file to kube-proxy.
I edited the systemd service definition to as follows:
[Unit]
Description=Kubernetes Kube Proxy
Documentation=https://github.com/kubernetes/kubernetes
[Service]
ExecStart=/usr/local/bin/kube-proxy --config /var/lib/kube-proxy/kubeproxy.conf
Restart=on-failure
RestartSec=5
[Install]
WantedBy=multi-user.target
On querying for status of kube-proxy , it reports

● kube-proxy.service - Kubernetes Kube Proxy
Loaded: loaded (/etc/systemd/system/kube-proxy.service; enabled; vendor preset: disabled)
Active: activating (auto-restart) (Result: exit-code) since Tue 2018-01-02 22:28:55 UTC; 4s ago
Docs: https://github.com/kubernetes/kubernetes
Process: 5239 ExecStart=/usr/local/bin/kube-proxy --config /var/lib/kube-proxy/kubeproxy.conf (code=exited, status=1/FAILURE)
Main PID: 5239 (code=exited, status=1/FAILURE)
Jan 02 22:28:55 slave-1 systemd[1]: kube-proxy.service: main process exited, code=exited, status=1/FAILURE
Jan 02 22:28:55 slave-1 systemd[1]: Unit kube-proxy.service entered failed state.
Jan 02 22:28:55 slave-1 systemd[1]: kube-proxy.service failed.

Now i went and checked the journal, and it tells me this

Jan 02 22:28:44 slave-1 systemd[1]: Starting Kubernetes Kube Proxy...
Jan 02 22:28:44 slave-1 kube-proxy[5203]: I0102 22:28:44.722197    5203 feature_gate.go:184] feature gates: map[]
Jan 02 22:28:44 slave-1 kube-proxy[5203]: error: no configuration has been provided
Jan 02 22:28:44 slave-1 systemd[1]: kube-proxy.service: main process exited, code=exited, status=1/FAILURE
Jan 02 22:28:44 slave-1 systemd[1]: Unit kube-proxy.service entered failed state.
Jan 02 22:28:44 slave-1 systemd[1]: kube-proxy.service failed.
Jan 02 22:28:49 slave-1 systemd[1]: kube-proxy.service holdoff time over, scheduling restart.

The error tells me that i did not provide configuration to kube-proxy, but i clearly did because systemctl shows it.
Kubernetes Version: 1.9.0
OS : CentOS 7
Can someone help?

Is your kubeconfig a yaml file ? I am getting this error:
/usr/local/bin/kube-proxy --config ./kube-proxy.kubeconfig --v 10
F0430 12:51:03.473083   18711 helpers.go:119] error: no kind "Config" is registered for version "v1"
Will you be able to share the file? thanks.

Is /var/lib/kube-proxy/kubeconfig present, syntactically correct, and properly formatted?

Thank you @tokiwinter. I had this same is, and my problem ended up being a typo in the kubeconfig file. I'm new to Kubernetes, and would have never thought my issue would be a typo in my kubeconfig file when the error was "no configuration has been provided error."

@rahulmishra @caemiller  could you please share your kubeconfig file. I am facing the same issue .
/usr/local/bin/kube-proxy --config=/var/lib/kube-proxy/kube-proxy-config.yaml
F0209 06:42:08.580939   13727 server.go:377] invalid configuration: no configuration has been provided
more /var/lib/kube-proxy/kubeconfig
{
kubectl config set-cluster kubernetes-the-hard-way
--certificate-authority=ca.crt
--embed-certs=true
--server=https://<IP_ADDRESS>:6443
--kubeconfig=kube-proxy.kubeconfig
kubectl config set-credentials system:kube-proxy
--client-certificate=kube-proxy.crt
--client-key=kube-proxy.key
--embed-certs=true
--kubeconfig=kube-proxy.kubeconfig
kubectl config set-context default
--cluster=kubernetes-the-hard-way
--user=system:kube-proxy
--kubeconfig=kube-proxy.kubeconfig
kubectl config use-context default --kubeconfig=kube-proxy.kubeconfig
}


Doesn't scale properly with the number of threads used.

I ran some benchmarks and noticed that the throughput reaches an optimum with 4 threads and starts decreasing afterwards:

     0.1 gb    1gb       5 gb
1    0.4067    0.44447   0.43217
2    0.7244    0.83551   0.63393
3    0.98752   1.20703   0.63876
4    1.08851   1.39454   0.68621
5    0.91721   1.24367   0.66534
6    0.99265   1.05864   0.5936
7    0.82179   0.86623   0.59534
8    0.90496   0.88324   0.57842
9    0.81467   0.90776   0.54953
10   0.93938   0.90655   0.55023

(Ran on an AMD Ryzen 5 1600X Six-Core Processor CPU)

the threads need to synchronize to do file IO, so that becomes a bottleneck as the number of threads increase
on my machine with 8 cores, i found the optimal setting to be 8 threads

@camel-cdr
You can only archive perfect scaling if 100% of the code can be run in parallel, you can use Amdahl's law to determine the theoretical speedup.

@Daniel-Liu-c0deb0t I didn't dig into the code but technically if the length of the data is not changed (ie. because you will never change a multi-byte char into a single-byte char or vice versa) you can pre-allocate the output file (ie. seek(length, SEEK_SET)) and then you can have each file have it's own file pointer writing only the portion of the data.
In this way you wouldn't need locking.
Also, if you are not doing it, cpu pinning helps a lot with the SSE4.1 instructions because you will not end-up asking the same units to process the data. When you do the pinning you will need to identify the cpu id via something like thread_index % cpu_count.
In general it's worth to run about 2 threads per core, especially because you are doing IO.
Swapping data out from L1/L2/L3 during the data processing is fine, it's very bad though to access the memory so it would be worth to process chunk sizes of an appropriate length (you can fetch the cpu cache sizes dynamically and calculate it at runtime).

unfortunately, pre-allocating an output file isn't possible because stuff like adding emojis can change the length of the data
cpu pinning is a good idea. im not doing it rn (the user passes in the number of threads), but it could be added. though in theory the os should be able to spread threads evenly between cores if you pass in a reasonable thread count.
i do try to keep stuff in cache. each thread has its own buffer that should fit in L2 cache. its a fixed size of 64kb rn, as i did not implement checking cpu cache sizes


bipedal static contact dynamics example

Hey!
Thank you so much for the absolutely amazing library!
I am trying to modify the static contact dynamics example to work with talos so better understand the example. I can get it to work with anymal but it doesn't pass the checks on bipedal example robots. I make sure to swap out the joint names and base link, but there is generally a large error in the contact forces and joint torques. Is there something I missed?
Again thank you so much for your hard work!

Hi @imrobot2000,
Thanks for your great and well-appreciated feedback on using Pinocchio.
In order to properly answer your question, could you provide a tiny example in Python for easy validation?
Could you also provide your current version of Pinocchio?
Thanks in advanced,
Best,
Justin


[Fix] Change About Us Text

Details
In the below section, the name 'Amazing Buttons Animation' can be changed to 'Animating Buttons' as this is the current name of the project.
Screenshots

Type of Contribution

[ ] Update to an existing Animated Button
[ ] Adding a new Button
[ ] Resolving a bug
[ ] Proposal to the Repository
[X] Changes related to documentation or README.md
[ ] Other Changes

Checklist

[X] I have checked the existing issues
[X] I have read the Contributing Guidelines
[X] I am willing to work on this issue
[X] I am a GSSoC'23 contributor

Go Ahead


Allow optional soft reset option to be passed into deassert_risc_reset_at_core

Issue
https://github.com/tenstorrent/tt-umd/issues/287
Description
Metal needed ability to deassert only the second risc on BH ethernet cores
List of the changes
Added an optional parameter to deassert_risc_reset_at_core API
Testing
Will be opening up a PR in Metal on top of this branch and will run post commit and BH post commit.
Metal post commit
Metal Blackhole post commit
API Changes
This PR has non-breaking API changes:

[ ] tt_metal approved PR pointing to this branch
[ ] tt_debuda approved PR pointing to this branch: link -- is this needed if it is not breaking?

@abhullar-tt Thanks for the tests! Just take a look on why aren't they passing on remote chips...


Add enhance Readme.md file

Hii @imhardikdesai
I would to improve the readme file of this project which will includes sections like
Table of content
What is this project about
How to contribute to this project
How to setup this project
How to run this project
Author Details
Contributors Details
License Details
Contact Details
Acknowledgement
References

Sure please start working on it

On it

Hii @imhardikdesai I have fixed this issue, Please review
Cheers


False reporting of "inconsistent spacing or indentation" in Python when pasting code written in Notepad++ (different editor)

Type: Bug
In Python, write an "if else" statement in Notepad++ and then copy and paste this code into a prewritten "while" loop in VS Code.
Even if the code is written with tabs = 4 spaces in Notepad++ and in VS Code, the pasted code will be flagged as violating a syntax rule ("inconsistent spacing... ") despite the spacing of the pasted code being identitcal to the prewritten code in VS Code.
Note the line/column number of the pasted code. While in VS Code, the error goes away if you backspace the indentation on the pasted code then then space it again to the same spacing it was before. Eg: pasted "if" statment is 1 tab (4 spaces) inside the "while" loop. Backspace the tab and then hit tab again and the error is gone.
Why would anyone do this? No good reason - I happened to be testing a section of code in a separate file in Notepad++ that was to be added to a separate file which was open in VS Code. When I copied and pasted the code across the editors I received this error.
VS Code version: Code 1.76.2 (ee2b180d582a7f601fa6ecfdad8d9fd269ab1884, 2023-03-14T17:55:54.936Z)
OS version: Windows_NT x64 10.0.19045
Modes:
Sandboxed: No

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-10700 CPU @ 2.90GHz (16 x 2904)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_renderer: enabled_onvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: disabled_off

Load (avg)
undefined

Memory (System)
31.78GB (16.77GB free)

Process Argv
--crash-reporter-id 32224f08-0e27-4ef0-be4f-45b6241b0608

Screen Reader
yes

VM
0%

Extensions (13)

Extension
Author (truncated)
Version

gc-excelviewer
Gra
4.2.57

isort
ms-
2022.8.0

python
ms-
2023.4.1

vscode-pylance
ms-
2023.4.40

jupyter
ms-
2023.2.1200692131

jupyter-keymap
ms-
1.1.0

jupyter-renderers
ms-
1.0.15

vscode-jupyter-cell-tags
ms-
0.1.8

vscode-jupyter-slideshow
ms-
0.1.5

veriloghdl
msh
1.11.4

mtx-psm
mtx
0.1.1

pdf
tom
1.2.2

v4pvhdlforprofessionals
ViD
1.6.4

(1 theme extensions excluded)

A/B Experiments
vsliv368cf:30146710
vsreu685:30147344
python383cf:30185419
vspor879:30202332
vspor708:30202333
vspor363:30204092
vslsvsres303:30308271
vserr242:30382549
pythontb:30283811
vsjup518:30340749
pythonptprofiler:30281270
vshan820:30294714
vstes263:30335439
vscoreces:30445986
pythondataviewer:30285071
vscod805cf:30301675
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
cmake_vspar411:30581797
vsaa593:30376534
pythonvs932:30410667
cppdebug:30492333
vsclangdf:30486550
c4g48928:30535728
dsvsc012:30540252
pynewext54:30695312
azure-dev_surveyone:30548225
nodejswelcome1:30587005
3biah626:30602489
pyind779:30671433
f6dab269:30613381
pythonsymbol12:30671437
azdwalk:30721579
pythonms35:30701012
pythonfmttextcf:30716743

Thanks for creating this issue! It looks like you may be using an old version of VS Code, the latest stable release is 1.77.3. Please try upgrading to the latest version and checking whether this issue remains.
Happy Coding!


use fallback in attributesToArray

solves #448

Can you also add tests for this?

blocked by: #486