k9 crashes using wrong signs

My k9 crashes when entering different combinations of characters to the recipient field of an email.
Just typing a forward slash / to the To-field even does it.

P.s.: K9 version 5.006 on Android 4.4.4

Please provide a stacktrace of the crash, e.g. by following these instructions: https://github.com/k9mail/k-9/wiki/LoggingErrors

Unfortunately is this page no help to me. I don't have a computer with me and the other program doesn't run with my os-version. CatLog works but I don't know how to get it out of there. Furthermore is there no k9-log.txt (find -name "k9-log.text").

I mean where exactly lies the logfile of k9?

Sorry, that was a numb question …
But logcat -d -v time k9:V '*:S' AndroidRuntime:E is not giving me anything besides

--------- beginning of /dev/log/main

With regard to logcat I had a similar issue and resolved by:

adb logcat -v time k9:V *:S AndroidRuntime:E > my_log.txt
run k9 on the device or simulator and reproduce the crash
stop the adb command piping to the file by Ctrl-C
read the full log in my_log.txt

I think the -d option dump whatever has been accumulated in the log buffer and might or might not contain your event
Hope it helps

Thank you for the help but output does not change. And I have not access to adb since I only use the terminal emulator.

Without a debug log there's no way to know what went wrong. But since most of the code that deals with recipients is now rewritten anyway, this might not be an issue in the next major stable version.


Alter IProp representation

This eliminates the need for the Properties.Indexed.Unsafe module defining IProp separately from Properties.Indexed and also uses switches from using refine <<< x in favour of directly coercing x so that there is no runtime overhead (we made the same change to Elements.Indexed a while back).
@jonsterling to review please!
Resolves #206

looks good to me!


_group.GetEntities().AsParallel().ForAll(Execute);这段语句有GC

先生，你好，我在测试中发现这段语句有GC，请问有办法解决吗

From Google Translate:

Hello sir, I found during testing that this statement has GC. Is there any way to solve it?

@32haojiufangjia yes, please try
_group.AsEnumerable().AsParallel().ForAll(Execute);

Fyi, when doing things in parallel, please avoid using generated methods like entity.AddXyz(), entity.ReplaceXyz(), entity.RemoveXyz(), etc, since they result in modifying non-concurrent collections like HashSets, etc. When doing things in parallel I recommend changing the component values directly, e.g.
entity.health.value -= 1;


nw-gyp or node-gyp

I need binding.gyp from node-gyp. can you script for me?
thanks.

Although the project makes use of node-gyp for downloading the node.js libraries, the project itself is compiled with Visual Studio on windows or G++/Clang++ on Linux and Mac via a makefile. Is there a reason you need a binding.gyp file?

Might get resolved by #2.

I need this project for nwjs with node-webkit thats why I need this script for compile. I hope you get my answer.

There's currently a pull request to fix this issue (#20), I'll be working on integrating it this weekend along with #21 which should finish off version 1.1.0.

@kkm, if you want to try out #20 before @dkrutsko finishes merging, you can temporarily set the robot-js dependency version in your project to this value: "https://github.com/p120ph37/robot-js/archive/dev.zip"


Bump React dev dependency to v0.13

The typings have now been updated for React v0.13, update
the version used during development as well.

Coverage remained the same at 100.0% when pulling 1ce0d48a343f75200d2dc440f907a354c2d9b75d on robertknight:react-v0_13-deps into cae606105a38317aa3e9fc6ddb4505bfdc731299 on Asana:master.

Thanks!


feat: add read task for gcs

Because

we do not have read objects and create bucket task for gcs

This commit

add read task for gcs
add create bucket task for gcs

@donch1989
About changing all fields to kebab-case, could I change in this ticket?
Or, do you plan to do it in other ways? (same question for this PR.

Hi @chuang8511
I guess you forgot to upload the definition and tasks.json.

@donch1989
Sorry, do I miss to read something?
I think it is push. (please correct me if I misunderstand)
🙇

I will notify you again after I rebase!
I may do it after 3~4 p.m. London time.
Sync to you first.

same status like this branch
Will come back to do final e2e test after the frontend fixes the bug!

Sync
I have done end to end test.
To make GCS more flexible, I add all params in the output of the component.
However, it damages UI a bit.


train with code

I want to run it directly in code for one-click-training instead of using webUI. How should I write calls

Use this CLI/API version, bear in mind it is still in dev.
https://github.com/RVC-Project/Retrieval-based-Voice-Conversion


problem while reproducing referit dataset

Thank you for your great work. I meet some problems when I reproduced the result on referit dataset. I use the same config with this file.
However, the validation accuracy is only 50% after 20 epochs while it should be 58% in the paper. I think maybe the config is different among different datasets. Looking forward to your help !

Hello,  @1170300521 sorry for the delayed reply (for some reason this didn't turn up in my notifs).
I recall using lr=1e-4 for 10 epochs, and 1e-5 for another 10 epochs.
I might have access to the logs, so let me know if you are having trouble reproducing the result.

Hello,  @1170300521 sorry for the delayed reply (for some reason this didn't turn up in my notifs).
I recall using lr=1e-4 for 10 epochs, and 1e-5 for another 10 epochs.
I might have access to the logs, so let me know if you are having trouble reproducing the result.

@1170300521 @TheShadow29  I am having difficulty reproducing the result. I am trying to reproduce the result on Flickr30k.
Everything went fine till...
I ran python code/main_dist.py flickr30k_retina_29Jul19.pth --ds_to_use='refclef' --resume=True --only_valid=True --only_test=True
and got an error:
AssertionError: key only_valid doesnot exist.
Can you please help?

@tejan-rgb Can you try changing only_valid to only_val?


connect stop working with minified bundle

you can delete this one it was issue on my configuration

sure, no problem , can you tell me what config issue was, maybe I can include it in the docs to be more clear

oh it was my fault :blush: , i forgot to add "reduxReactFirebase" to store on create store on all my envs.
(did it only for dev)

thank you very much for clarifying


(Build) Fail on dev branch.

This issue is created automatically by CI.
Linux Build State: failure
Windows Build State: success
Commit: 09cb4160b1c956aeead8c980b8c3d725d73b33c6
Commit Branch: dev
Commit Author: innerviewer

Auto-closing as build in this branch is successful now. Commit: a46c357eeb061af3680f479ce07eccaa53dda157


Version 7.0.3: Newly created book cannot toggle preview on Mac 10.12.3

There might be a bug which causes newly created book cannot toggle preview on Mac 10.12.3. More specifically, when I clone a book I just created to local I fail to toggle preview with either keyboard shortcut or clicking on the menu while there is no such problem with the books created before the new release of 7.0.3 (which I toggled to preview mode before).
Thank you.

Same or similar problem with the Linux version. Since 7.0.2, you cannot edit the markdown source anymore. This is a critical issue - I had to revert to editing copy in a programming editor.

I can click on the [?] on the bottom-right and then select Edit Markdown. However, the new issue with this is that after doing so, I cannot deselect it and toggle it back to full preview mode.

@carlpch The editor stays in raw mode if you put custom HTML or plugins/template in your markdown (it will support booth in WYSIWYG mode soon enough). Did you add HTML or plugins to your doc ? If so that's probably what's preventing you to switch back, on pure Markdown docs, I can switch back and forth on 7.0.4 without problems.

@meanprogramming Is their a reason why you're editing the raw markdown source vs in document mode ?

@AaronO thanks for the clarification, yeah I added a  in the Table since it does not support linebreak, guess that is what disables it. Good to know the functionality will be added soon, really appreciate this!


Error: Unable to parse nextflow.config. Cannot compare java.lang.Boolean with value 'true' and java.lang.Integer with value '0'

Description of the bug
Pipeline fails immediately with the error in subject line.
This is my first attempt at using Nextflow and the nf-core RNAseq pipeline, so I'm not even sure where to start to debug this.
The goal is to run this on a University HPC node in offline mode.
 So, when nothing is supplied, this is the error which occurs.
Resolve it with something like:
--trim_nextseq 20


Links are bold in Reading mode but normal in Editing mode

Links are bold when I'm in the Reading mode, but have normal weight when I'm in the editing mode. Here's an example:
Editing:

Reading:

And this is where the style is being applied:
font-weight: 600

I can't seem to find an option to disable bold links.
My font is Balsamiq sans if it helps.

Hi @ShahriarKh,
The links are no longer bold in the newest 3.0.0 update and should be consistent through all editor modes.


Remove unused globals from fonts unit test file

Somehow I missed this while reviewing PR #7540, so let's clean this up.

/botio unittest

Trivial


Add a timeout context for establishing GRPC connections

Fixes #803

Codecov Report

Merging #814 (bea6042) into master (671f161) will increase coverage by 0.34%.
The diff coverage is 58.65%.

@@            Coverage Diff             @@
##           master     #814      +/-   ##
==========================================
+ Coverage   48.42%   48.76%   +0.34%
==========================================
  Files         140      142       +2
  Lines        6914     7009      +95
==========================================
+ Hits         3348     3418      +70
- Misses       2969     2985      +16
- Partials      597      606       +9

Impacted Files
Coverage Δ

pkg/core/consensus/comms.go
0.00% <ø> (ø)

pkg/core/consensus/fixtures.go
0.00% <ø> (ø)

pkg/p2p/wire/topics/topics.go
68.00% <ø> (ø)

pkg/core/consensus/reduction/firststep/step.go
50.42% <11.53%> (-5.24%)
:arrow_down:

pkg/core/consensus/selection/step.go
48.20% <25.00%> (-0.71%)
:arrow_down:

pkg/core/consensus/reduction/secondstep/step.go
73.77% <33.33%> (-1.23%)
:arrow_down:

pkg/p2p/peer/peer.go
36.45% <39.62%> (-0.72%)
:arrow_down:

pkg/core/chain/sequencer.go
66.66% <40.00%> (-33.34%)
:arrow_down:

pkg/core/candidate/validator.go
57.14% <50.00%> (ø)

pkg/util/nativeutils/rpcbus/rpcbus.go
66.66% <50.00%> (-1.42%)
:arrow_down:

... and 13 more

Continue to review full report at Codecov.

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


gridSelectWorkspace loses focus

If I have gridSelectWorkspaces from XMonad.Actions.GridSelect active and I click a window on another screen, the gridselect loses focus and it can't be restored. If the gridselect has lost focus, you can't click it, can't type anything and can't escape out of it. The only way I've found to fix it is by restarting xmonad.

maybe adding grabPointer to gridselect will solve your problem. but i'm not sure is this solution or just a workaround


Feathers SDK 4.2.0 release

Hello,
Josh can you create a release of Feathers SDK 4.2.0 in https://github.com/feathersui/feathersui-starling-sdk/releases ?
Thanks

I have no plans to release feathersui-starling or feathersui-starling-sdk version 4.2.0 at this time.

@pol2095 are you after a Windows version by any chance? If so, I have one...

Mac and Windows, but I just needed starling 2.7 swc, I replaced it in feathersui-starling-sdk/frameworks.libs.
Thanks


World Service Authority connection?

Have you heard of the World Service Authority? it's an organisation that's been around for a long time now and offers a real "world citizen" passport. It's legally accepted in 6 countries and intermittently accepted in about 150. I'm not exactly sure what a connection between this project and that would entail, but it seems to me that these projects are so complimentary that it's definitely worth investigating!
http://worldservice.org/

A New Form of ID Allows You to Be a Citizen of the World
Instructions on how to create a BlockchainID
Video here:
http://youtu.be/1iAg6BITPdc
Github project:
https://github.com/MrChrisJ/World-Citizenship
I seriously advise the OCU-Organisation pour une Citoyenneté Universelle get on board with this project.
Join the
O U C - Organization for Universal Citizenship
Join us on facebook
http://www.facebook.com/worldcitizenfriends
Call to Action by:
http://www.facebook.com/citoyenneteuniverselle
Published by:
http://www.o-c-u.org/en/


ShadowrunSixthWorld-v.48

Bugfix - Removing Technomancer echos with modifications didn't update things correctly.
Bugfix - Replaced translation "Rotarcraft" with correct "Rotorcraft"
Added Modifications functionality to Magic->Adept->Powers
Added Modifications functionality to Magic->Meta->Metamagics
Added Modifications functionality to Technomancer->Complex Forms
Comments are very helpful for reviewing the code changes. Please answer the relevant questions below in your comment.

[ Y ] Does the pull request title have the sheet name(s)? Include each sheet name.
[ Y ] Is this a bug fix?
[ Y ] Does this add functional enhancements (new features or extending existing features) ?
[ N ] Does this add or change functional aesthetics (such as layout or color scheme) ?
[ NA ] If changing or removing attributes, what steps have you taken, if any, to preserve player data ?
[ NA ] If this is a new sheet, did you follow Building Character Sheets standards ?

Character Sheet Info Roll20 Internal Use only.


Feature/throw errors

errors are now thrown into javascript so it's a bit easier to understand what have gone wrong

getting closer

closes #44

updated to current webrender (they changed the api again)


Refreshes the client code again

Signed-off-by: John Howard<EMAIL_ADDRESS>This gets the client code up-to-date with the version under jhowardmsft/opengcs which is currently vendored into docker, and with all the dynamic sandbox management stuff working. After I get back from vacation, I'll update docker to point to Microsoft/opengcs and remove the jhowardmsft repo.

@jhowardmsft,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

@gupta-ak @soccerGB PTAL


[ClojureStorm] Compiling def inside do mess up coordinates

The ClojureStorm instrumentation system assumes that (defn ...) forms are always outer forms.
There are cases where this doesn't apply like (do (defn ...) (defn ...)) which makes the resulting defn compiled coordinates wrong.

This is fixed in the current version


Add apollo-generated types for gnomAD gene and variant page

Proof-of-concept approach for https://github.com/broadinstitute/gnomad-browser/issues/944... not to be merged.
To generate types from gnomAD API
cd browser
yarn download-schema
yarn generate-types

Any thoughts on his this compares to https://github.com/broadinstitute/gnomad-browser/pull/954?

I kind of wish the generated interface name was the same as __typename. Wonder if there is a way to configure that or if it would lead to name clashes.
For instance, ideally this interface would be called TranscriptConsequence, rather than GnomadVariant_variant_transcript_consequences
export interface GnomadVariant_variant_transcript_consequences {
  __typename: "TranscriptConsequence";
  domains: string[] | null;
  gene_id: string;
  gene_version: string | null;
  gene_symbol: string | null;
  hgvs: string | null;
  hgvsc: string | null;
  hgvsp: string | null;
  is_canonical: boolean | null;
  is_mane_select: boolean | null;
  is_mane_select_version: boolean | null;
  lof: string | null;
  lof_flags: string | null;
  lof_filter: string | null;
  major_consequence: string | null;
  polyphen_prediction: string | null;
  sift_prediction: string | null;
  transcript_id: string;
  transcript_version: string;
}


Creating new keys in a tableextension with fields form the original table

I have a tableextension on customer table that contains extra keys:
keys
  {
        key(Key1;"Customer Price Group")
        {
        }
        key(Key2;"Statistics Group")
        {
        }
        key(Key3;"Commission Group")
        {
        }
    }

The field at Key1 and Key2 are of NAV, the field in Key3 is one from my application.
The Key1 and Key2 has the problem: [AL] A field list has to contain at least one field.
The "Customer Price Group" and "Statistics Group" have the problem:
[AL] Reference '"Customer Price Group"' in application object 'tableextension50101' does not exist
[AL] The keys 'Key1' and 'Key2' have an identical list of fields.

Another object is a tableextension on Sales Header with the following extra keys:
keys
    {
        key(Key1;"Document Type","Combine Shipments","Bill-to Customer No.","Currency Code","EU 3-Party Trade","Dimension Set ID","Journal Template Name","Ship-to Code","Invoicing System")
        {
        }
        key(Key2;"Bill-to Customer No.")
        {
        }
}

These are all NAV fields, but these don't have any problems ...
What is missing on the keys for the customer tableextension?

Unfortunately, table extensions do not currently support 'cross-keys' from fields that are not part of the extension with the exception of the primary key.
This means that you cannot reference non-PK fields from the 'base' table inside of a key list.
The errors you are seeing in your first example are a result of this:

The compiler doesn't recognize the fields from the base as existing/available in this context
Since the fields are recognized, the compiler sees an 'empty' key list
Since both Key1 & Key2 have 'empty' key lists, they are seen as duplicates

Your second example should also produce the same errors. My guess is that there is an error above or elsewhere in the file that is in some way causing the compiler to stop parsing that file and thus not reach these keys yet.
If you were to create a new, completely empty table extension for the "Sales Header" table, and add these keys to it, I would expect to see the same/similar errors as your first example

Unfortunately, table extensions do not currently support 'cross-keys' from fields that are not part of the extension with the exception of the primary key.
This means that you cannot reference non-PK fields from the 'base' table inside of a key list.

Please make this possible!!
I can see the logic and reason behind this.
However if can't create new keys on NAV-fields, we will have a huge performance loss.
I would be very suprised if there would be any application of ISV's that doesn't have an extra key on NAV-fields (combined with fields from the application).
Perhaps make it also possible to add extra SIF to existing NAV-keys?

This is a duplicate of #730. Let's continue the thread in one place.


Access-controlled service for large static files

Description
I'm proposing that a feature be added to serve large static files to authenticated users.
It might not be obvious why this is a problem. Here are some of the possible solution paths, and why they are blocked:

Can't we use a static file service like whitenoise?

Whitenoise doesn't provide any kind of authentication or access control, and I'm not even sure it can handle very large files.

Can't Django just serve the files through a FileResponse object?

FileResponse objects do a decent job of serving small and medium-sized files, but for very large files, problems arise. (In my case, when files get big enough, I hit a memory error.) It appears that if a given environment has a wsgi.file_wrapper defined, FileResponse objects may use that to efficiently serve access controlled files. But that seems to require that Django be running on the same machine as the web server.

Isn't there some kind of funky thing you can do with headers?

Yes! Or rather, there was when cookiecutter-django used Caddy. Caddy supported the X-Accel-Redirect header, and could be configured similarly to nginx (as described here). After the switch to Traefik, this approach no longer works, because Traefik is not a web server at all.

Could you use AWS somehow?

Maybe. I haven't looked into this option carefully. But it seems like it would be very complicated to get right.

How should it be implemented? I don't know. This is where I am stuck, and would welcome discussion. I posted a question on stack overflow and got crickets; if you see a way around this that doesn't require a pull request, please feel free to answer there.
Rationale
In a sense, this is not a "feature" but a fix. The change from Caddy to Traefik arguably broke functionality that was working pretty well before.
What it really means for me, concretely, is this: now that I want to do something similar with a new app, I can't use cookiecutter-django without a fairly elaborate and awkward reconfiguration -- something like standing up an nginx container between the django service and the traefik service. If that's the only option, my instinct is to not use cookiecutter-django at all. I probably don't need all the things, and the configuration work will wind up being about the same either way. And maybe that's fine; this could just be a "It might not be what you want" situation.
But I'm proposing the alternative narrative that this would actually fix something that worked before and now is broken. I don't honestly imagine that there are that many people doing what I'm doing, and so I can't argue that you will lose a bunch of users over this. It's just kind of annoying that it used to be easy, and now is hard.
Use case(s) / visualization(s)
Here's my use case: I am developing new apps for researchers at the University of Pennsylvania doing large-scale statistical text analysis in multiple different departments. I need to be able to automatically distribute copyright-protected data to authorized users in bulk, without risking leaking the data.

I need to be able to automatically distribute copyright-protected data to authorized users in bulk, without risking leaking the data.

Are you positive these should be distributed using static files and not media files instead? It sounds like this data would be uploaded by your application users to a FileField or ImageField rather than tracked in version control like your code base. Django-storage is providing some options to restrict access.

The data is generated by a crawling process and aggregated into large zip files that the user then downloads. There's no uploading involved.
But if there are ways to restrict access to the files using some other mechanism that I haven't mentioned above, I'm all ears! It just has to be able to efficiently handle multi-gigabyte files.

The data is generated by a crawling process and aggregated into large zip files that the user then downloads. There's no uploading involved.

Ok, so when I have to do that type of things, for me, there is an "upload" invloved at some point, not from a user, but from the crawling process. Here is how I usually handle this (assuming I'm on Docker based config):

Create a storage class to store files as private in a AWS S3 bucket:
from storages.backends.s3boto3 import S3Boto3Storage

class PrivateStorage(S3Boto3Storage):
	default_acl = 'private'
	file_overwrite = False
	bucket_name = 'my-private-bucket'

More options detailed in the documentation. You can use AWS_QUERYSTRING_AUTH and AWS_QUERYSTRING_EXPIRE to control access of your files.

Create a model with a FileField that will be used to represent these files in my application, using this private storage, for example:
class LargeZipFile(models.Model):
	name = models.CharField(max_length=50)
	zip = models.FileField(
		storage=PrivateStorage()
	)

Use Celery to generate the data, and when files are ready, create instances of LargeZipFile to upload the files.

Each time a user wants to download a file, your application exposes the LargeZipFile.zip.url on some page, which will have query parameters giving access for a short amount of time.
That being said, I don't know how your server is deployed at the University of Pennsylvania, it might be on a dedicated, non-cloud server. I don't know which are your storages options, but if AWS is not suitable, Digital Ocean might be and has a compatible API, which is supported by django-storages.

It just has to be able to efficiently handle multi-gigabyte files.

A word of warning that it could generate some significant costs from Amazon.


Add Network in localstorage

The idea is to have the network saved in the localstorage, so that a returning visitor (or even a refresh), will not default/land to Polkadot network as it is happening now

I wouldn't put this as a bug, and I'd say this suffices
https://github.com/delegit-xyz/dashboard/issues/27

It is not a bug. Its an enhancement. When a user lands in delegit.xyz - there is no Network query in the string.
The query in the string should automatically update the local storage as well

Ah I meant regarding the prio, P0 is for bugs in prod basically. It's in the description

Added to issues and PRs relating to a critical severity bugs.


Fix character encoding in TextArea

Description
The TextArea value is read through the text state, which is always triggered when the item's value changes. This conflicts with some character encodings (e.g., Portuguese words, Spanish, etc.) because it logs whenever a character or graphic sign (e.g., accent) is clicked.
Screencast
Before ❌

After ✅

Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[x] I ran npm run build and tested this distribution build in Raycast
[x] I checked that files in the assets folder are used by the extension itself

Hey @henriquemacedo!
I just returned back the value configuration into the extension to check why did even happen and cannot repro the case you're showing in the description. Can you recommend me the case? How do you enter the ç and other characters? Because If enter them with option + a or other letters everything works fine.


Fix Mailer Deprecations

The old #deliver method is deprecated.
Using new Rails 5 #deliver_later instead.

Coverage remained the same at 97.717% when pulling 493e226fc6c2ec5ec7518d1f351995af1950f325 on tonyta:mailer-deprecations into 4c9e70014aca7f42b9978e1f359ca8b464ec5921 on sfbrigade:production.

And so it is, good catch!


TextPropertyManager: Warning shown when breaking a node

TextPropertyManager: something is fishy here. Saw a change, but don't know the property.

Reproduce:

Hit enter when inside a strong annotation

Once you see that warning, try undo and then redo and it actually breaks the editing.

There are more cases, e.g. when breaking before an annotation.


client reconnect failed

when the router or cluster send end our info to the other side here the infoJson only include ip:port , when the local nats-server down,  if the route other side  has the username and password , client will doReconnect failed.
could you modify the code to add the username and password?

func (s *Server) getClientConnectURLs() []string {
	s.mu.Lock()
	defer s.mu.Unlock()

	sPort := strconv.Itoa(s.opts.Port)
	urls := make([]string, 0, 1)

	ipAddr, err := net.ResolveIPAddr("ip", s.opts.Host)
	// If the host is "any" (<IP_ADDRESS> or ::), get specific IPs from available
	// interfaces.
	if err == nil && ipAddr.IP.IsUnspecified() {
		var ip net.IP
		ifaces, _ := net.Interfaces()
		for _, i := range ifaces {
			addrs, _ := i.Addrs()
			for _, addr := range addrs {
				switch v := addr.(type) {
				case *net.IPNet:
					ip = v.IP
				case *net.IPAddr:
					ip = v.IP
				}
				// Skip non global unicast addresses
				if !ip.IsGlobalUnicast() || ip.IsUnspecified() {
					ip = nil
					continue
				}
				if s.opts.Username == "" || s.opts.Password == "" {
					urls = append(urls, net.JoinHostPort(ip.String(), sPort))
				} else {
					url := s.opts.Username + ":" + s.opts.Password + "@" + net.JoinHostPort(ip.String(), sPort)
					urls = append(urls, url)
				}

			}
		}
	}
	if err != nil || len(urls) == 0 {
		// We are here if s.opts.Host is not "<IP_ADDRESS>" nor "::", or if for some
		// reason we could not add any URL in the loop above.
		// We had a case where a Windows VM was hosed and would have err == nil
		// and not add any address in the array in the loop above, and we
		// ended-up returning <IP_ADDRESS>, which is problematic for Windows clients.
		// Check for <IP_ADDRESS> or :: specifically, and ignore if that's the case.
		if s.opts.Host == "<IP_ADDRESS>" || s.opts.Host == "::" {
			Errorf("Address %q can not be resolved properly", s.opts.Host)
		} else {
			if s.opts.Username == "" || s.opts.Password == "" {
				urls = append(urls, net.JoinHostPort(s.opts.Host, sPort))
			} else {
				url := s.opts.Username + ":" + s.opts.Password + "@" + net.JoinHostPort(s.opts.Host, sPort)
				urls = append(urls, url)
			}
		}
	}
	return urls
}

if the two nats-servers have different tls (cert), how ?

It is intentional that the server does not return usernames and passwords in the connected URLs. Furthermore, we would not be able since we support multi-users.
Even if you decide to fork and do the above in your own repo, I need to warn you about the danger of doing the above. First, it is working only for single users, not multi-users. Also, think of this use case: server A has user userA/passA specified. Server B has userB/passB specified. These 2 servers are connected. A client that has the proper credentials to connect to A would then - with your change - receive credentials to connect to B. In other words, B would give away its credentials to clients that may not be meant to have them.
You should have the users defined on both servers with same credentials. Otherwise, in the above scenario, you would use the client UserInfo() or Token() API to provide credentials for URLs that the client gets from server auto-discovery.

The problem is the same than without auto-discovery. Suppose there is no auto-discovery and have a cluster of 2 servers (A and B) with different certs. You start a client with a URL that points to those 2 servers A and B. Same problem.

Note that you can always disable advertisement of server URLs to client by starting the server with:
--no_advertise=false


Update to current version of mbedos

Fixed refactored namespaces, bit array doesn't exist anymore so replaced with span and missing raw serial replaced with unbuffered serial.

This will be updated again when upstream circular buffer to mbedos


[jwk] please add key type attribute/method

Hi,
It would be great if the jwk key object will know what type it is so we could have something like:
        token = joserfc.jwt.encode(
            header={
-               "alg": "RS256",  # guess
+               "alg": jwk1.alg(),
                "kid": jwk1.thumbprint(),
            },
            claims=claims,
            key=jwk1,
        )

It was also missing in authlib.
I tried to use jwk1.get("alg") but this returns None in most of the cases, I am unsure where is the right place to add this.
Thanks!

Hi, if jwk1.get("alg") is None, then the key doesn't declare it's "alg" value. If your JWK has a alg value, it shouldn't return None.
When you import the key, you can pass a alg value:
key = RSAKey.import_key(pem_bytes, {"alg": "RS256"})

key.get('alg')  # => RS256

I've added a alg property on key mode.

Hello @lepture ,
Thank you for addressing the issue, I began writing a patch just like you merged... but it did not work for me... so I opened this ticket as I could not keep up with the hierarchy.
Please try the following, you will notice that the return value is None.
Thanks,
Alon

import joserfc.jwk

def main():
    jwk1 = joserfc.jwk.RSAKey.generate_key(
        key_size=2048,
        private=True,
    )
    print(jwk1.alg)

if __name__ == '__main__':
    main()

When generate a key, you can pass "parameters" with the "alg":
RSAKey.generate_key(..., parameters={"alg": "RS256"})

When generate a key, you can pass "parameters" with the "alg":

but why would I want to do that? it will lead to inconsistency...
    jwk1 = joserfc.jwk.RSAKey.generate_key(
        key_size=2048,
        private=True,
    )

The class has all required information to know that it is RSA -> RS and 2048 -> 256.

I know what you mean now. But a RSAKey can also be used in JWE. We can only determine the alg value if it has a "use" value.


No Button Guidance on Facility Cards Below 768px Breakpoint

Is your feature request related to a problem? Please describe.
Labels of 'Notify' and 'View Facility' buttons on /facility page are hidden on less that 768px breakpoint.
Users may not understand the button's purpose.
Describe the solution you'd like
Adding a tooltip ensures that users understands the button's purpose
Additional context
Before (Labels Hidden Below 768px)

After

Tooltip for 'View Facility':

Tooltip for 'Notify':

can i work on this issue? @Jacobjeevan @rithviknishad


Add & Verify Most Recent Vega and Vega-Lite Builds

Currently 5.20.1 and 5.1.0 respectively.

Resolved in build 135


An empty array is considered null when inserting data into a column with the Int[] type

Bug description
There is a strange behavior when using Prisma for upserts, where an empty array is considered null when performing an upsert on a column that has an array of integers data type.
How to reproduce
Create a PostgreSQL database with a schema like this:
model ... {
.....
social_key String?
fav_series Int[] @default([])
....
}

Perform an upsert with this payload where fav_series is empty:
{
....
fav_series: []
....
}

Error: Argument fav_series: Invalid value provided. Expected Testfav_seriesInput or Testfav_seriesInput, provided (Null).
Expected behavior
Created successfully on DB with empty int array
Prisma information

model Test {
  createdAt           DateTime  @default(now())
  updatedAt           DateTime?
  email               String
  sid                 String    @id
  tableNum            Int
  state               String
  lastMigratedAt      DateTime?
  tableName           String
  pulledFrom          Int
  error               String?
  validation_score    Int?
  validation_logs     Json?
  migrationStep       String?
  // additional field for user tracker based on v4 table
  ip                  String?
  city                String?
  country             String?
  mobile              String?
  gender              String?
  fav_series          Int[]     @default([])
  subscription_type   String?
  subscription_id     String?
  user_id             Int?
}

// upsert with empty fav_series

Environment & setup

OS: macOs
Database: PostgreSQL
Node.js version: v19.6.0

Prisma Version
"@prisma/client": "^4.7.1",

Does this also happen with create() or update(), or is it only upsert() that is affected?
Can you try with a more recent version of Prisma maybe, to make sure this has not been fixed in the meantime?

@janpio Thank you for your response. I think the problem lies in the Prisma error log when I perform an upsert inside a $transaction. However, it's possible that my terminal truncates some text. When I execute the commands one by one, the error logs display the correct message, revealing a null value within the array. Therefore, the original problem originates from my code. Nevertheless, I hope Prisma can address this logging issue, as it would greatly assist developers.

Hey @reynaldiRepo,
Would you mind trying with a newer version of Prisma and confirm whether the bug is still present?
Thank you 🙏

Hey @reynaldiRepo, I've took a look at this.
I cannot confirm the case where an [] is treated as null and was consistently able to retrieve data:
{ id: 1, fav_series: [] }

When explicitly passing in null to the query
Invalid `prisma.a.upsert()` invocation in
/src/prisma_debug/index.ts:11:14

   8
   9 async function test() {
  10   let [up, ff] = await prisma.$transaction([
→ 11     prisma.a.upsert({
           create: {
             fav_series: null
                         ~~~~
           },
           update: {
             fav_series: null
                         ~~~~
           },
           where: {
             id: 0
           }
         })

Argument fav_series for create.fav_series must not be null. Please use undefined instead.
Argument fav_series for update.fav_series must not be null. Please use undefined instead.

    at Document.validate (/src/prisma_debug/node_modules/@prisma/client/runtime/index.js:32409:20)
    at serializationFn (/src/prisma_debug/node_modules/@prisma/client/runtime/index.js:35993:19)
    at runInChildSpan (/src/prisma_debug/node_modules/@prisma/client/runtime/index.js:25431:12)
    at PrismaClient._executeRequest (/src/prisma_debug/node_modules/@prisma/client/runtime/index.js:36000:31)
    at async PrismaClient._request (/src/prisma_debug/node_modules/@prisma/client/runtime/index.js:35926:16) {
  clientVersion: '4.7.1'
}

Note: this only lead to any messaging at runtime, there was no information in editor about this.

Thank you for your response. I think the problem lies in the Prisma error log when I perform an upsert inside a $transaction. However, it's possible that my terminal truncates some text. When I execute the commands one by one, the error logs display the correct message, revealing a null value within the array. Therefore, the original problem originates from my code. Nevertheless, I hope Prisma can address this logging issue, as it would greatly assist developers.

I'm trying to understand your message here. It seems like you're implying a difference in messaging from upserts when wrapped in a $transaction like in the following code snippet. Can you please elaborate your logging setup or even more ideally, provide a full minimal reproduction for your issue?
await prisma.$transaction([
    prisma.a.upsert({
      create: {
        fav_series: [],
      },
      update: {
        fav_series: [],
      },
      where: {
        id: 0,
      },
    }),
  ]);

Cheers! 🙏

I'm trying to understand your message here. It seems like you're implying a difference in messaging from upserts when wrapped in a $transaction like in the following code snippet. Can you please elaborate your logging setup or even more ideally, provide a full minimal reproduction for your issue?
await prisma.$transaction([
    prisma.a.upsert({
      create: {
        fav_series: [],
      },
      update: {
        fav_series: [],
      },
      where: {
        id: 0,
      },
    }),
  ]);

@Druue In my case, the upsert process is like this:
await prisma.$transaction([
    prisma.a.upsert({
      create: {
        fav_series: [1,2,3,null,1,2],
      },
      update: {
        fav_series: [1,2,3,null,2],
      },
      where: {
        id: 0,
      },
    }),
  ]);

As I reported, the error logs show fav_series as an empty array, and it should not be.
i run my code using warp, i will try to run on my original terminal.

Hey @reynaldiRepo, this is still not something I am able to reproduce, the logging looks perfectly fine. So this seems like an issue in your logging set-up.
Can you please provide a minimal reproduction, thank you 🙏
const fav_series = [1, 2, 3, null, 1, 2];
  const [a] = await prisma.$transaction([
    prisma.a.upsert({
      create: { id: 0, fav_series },
      update: { fav_series },
      where: { id: 0 },
    }),
  ]);

PrismaClientValidationError:
Invalid `prisma.a.upsert()` invocation in
/src/prisma_debug/index.ts

  22 //     id: 0,
  23 //   },
  24 // }),
→ 25 prisma.a.upsert({
       create: {
         id: 0,
         fav_series: [
           1,
           2,
           3,
           null,
           1,
           2
         ]
         ~~~~~~
       },
       update: {
         fav_series: [
           1,
           2,
           3,
           null,
           1,
           2
         ]
       },
       where: {
         id: 0
       }
     })

Argument `fav_series`: Invalid value provided. Expected ACreatefav_seriesInput, provided (Int, Int, Int, Null, Int, Int).
    at ... {
  clientVersion: '5.15.0'
}

@reynaldiRepo I am pretty sure we fixed the original error when we went from Prisma 4 to 5. We'd be grateful if you could confirm it does not happen to you on 5.0.0 or greater.

Hey @reynaldiRepo
With the lack of update and our inability to reproduce this issue, we're now closing it.
If you get the chance to try again on Prisma V5, please feel free to comment with any updates!
Thank you 🙏


shogun installation error

I have tried installing shogun on ubuntu 16.04 but I am receiving the following error
usr/local/lib/libbz2.a(bzlib.o): relocation R_X86_64_32S against `BZ2_crc32Table
'The compiler shows to use the fPIC flag, so I tried adding the fPIC flag in the CMakeLists.txt,, CMakeCache.txt and the Makefile but it doesn't work please help

You are trying to link against a custom compiled static-archive of bzip2…  That's relly odd somehow…  Try to remove usr/local/lib/libbz2.a and install your system's libbz2-dev.  Then rebuild Shogun in a fresh build-tree and all should go fine.
Unfortunately I cannot assist you linking Shogun against a custom-built bzip2, since I do not know how it was compiled and all…
Please report, if your problem was solved

@besser82 That worked, thanks !


IGNITE-14230 Port DynamicConfiguration to new underlying configuration framework.

Signed-off-by: ibessonov<EMAIL_ADDRESS>

Now patch looks better to me. There are still things we can improve but they are minor and could be done later in other tasks.


Update dns-forward to 0.8.2

Signed-off-by: David Scott<EMAIL_ADDRESS>

I think this version of dns-forward might have a defect: it seems willing to declare a server offline too readily. I'm investigating.


Calling Windows programs, from within WSL2 hangs agent

I am using the Linux agent, installed under WSL2 on a Windows 10 box. From what I can tell, invoking any Windows executable within my build causes the whole thing to freeze.
To reproduce, a simple command might be xcopy.exe /? to print the help text. Notably, everything I have tried outside the buildkite runner has worked, even when wrapped in a shell script.
I understand if this is low priority. I... didn't expect it to work. But I thought I would bring it up in case you wanted to also warn about this in your docs?

Some previous customers have reported hanging issues when executing Windows binaries from Buildkite WSL2 environment when setting the WSLENV environment variable to include all BUILDKITE_* environment variables. Does that sound like something that could be causing this issue? If not, could you tell us more about your agent environment and we’ll try to reproduce this. Thanks!

I am not setting WSLENV directly. Perhaps its implied?
We start the agent with a simple wsl <path_to_agent> start command, in a powershell script, from a scheduled task that runs at boot (use this over a service to get some additional cleanup & etc done before the agent is ready).
Configuration is how it comes out of the box, with a single hook added to export a non-buildkite environment variable. The hook is a simple bash script.

I’ve asked around but I’m afraid we don’t have enough experience with WSL(2) to offer support for this use case 😢. We are interested in learning more about WSL2 to better support customers running the agent in this way, and would be curious to learn what the issue is if you are able to figure it out.
I’ll leave this issue open for now to see if any one else coming across it has ideas what what could be happening. Sorry I can’t be of more assistance!

@keithduncan I updated the agent on this box from 3.26 -> 3.30. Somewhere along the line this issue appears to have been fixed, FWIW. Thanks for your effort and attention to this issue?


Goblin War Shaman entry error

The GWS from Nerzugal's Extended Bestiary is listed as an Undead (Goblinoid), but in the book it is Humanoid.

Thanks, corrected.


make some small improvements

@Emeryao Thank you for your work!
Could you fix these points and rebase your commits?

Hi @tomoki1207
I've fixed your reviews and make another PR with a branch that rebased all my commits


Support cross origin

django-cookies-samesite version:
Django version: 1.11

Description
In order to support cross origin, we can let the users choose to set the secure flag to True

my bad, secure flag can be set with SESSION_COOKIE_SECURE=True setting


Unknown Generator Ninja

I have Microsoft Visual Studio 2017 which has Cmake Inbuilt.
I tried adding the folder  FreeRDP-2.0.0-rc2 to Visual Studio 2017.
Getting the following error.
NOTE : I am working on Windows 10

" Severity	Code	Description	Project	File	Line	Suppression State
Error		CMake Error at C:\Users\Mickey\Downloads\FreeRDP-2.0.0-rc2\FreeRDP-2.0.0-rc2\CMakeLists.txt:477 (message):
Unknown generator Ninja	FreeRDP	C:\Users\Mickey\Downloads\FreeRDP-2.0.0-rc2\FreeRDP-2.0.0-rc2\CMakeLists.txt	477

This code is part of root CMakeLists.txt. Getting error in Line 477 as mentioned above.
if(WIN32)
	add_definitions(-DUNICODE -D_UNICODE)
	add_definitions(-D_CRT_SECURE_NO_WARNINGS)
	add_definitions(-DWIN32_LEAN_AND_MEAN)
	add_definitions(-D_WINSOCK_DEPRECATED_NO_WARNINGS)
        set(CMAKE_USE_RELATIVE_PATH ON)
	if (${CMAKE_GENERATOR} MATCHES "NMake Makefile*")
		set(CMAKE_PDB_BINARY_DIR ${CMAKE_BINARY_DIR})
	elseif (${CMAKE_GENERATOR} MATCHES "Visual Studio*")
		set(CMAKE_PDB_BINARY_DIR "${CMAKE_BINARY_DIR}/\${CMAKE_INSTALL_CONFIG_NAME}")
	else()
		message(FATAL_ERROR "Unknown generator ${CMAKE_GENERATOR}")
	endif()

I am having problems understanding the project.
As we are working on Remote Desktop Protocol , we need to have a server in
one system and client in another.
We have files for both client and server in the main folder.
Do we need to build both separately or we have to build them together as a
whole.
Need clarification on this. I am not able to understand the working of
FreeRDP
On Tue, May 8, 2018 at 6:51 PM, akallabeth<EMAIL_ADDRESS>wrote:

@adityasood11 https://github.com/adityasood11 Looks like the debug
symbol paths on windows with ninja builds need to be set.
don't have such a setup to test currently, but pull requests are welcome.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/FreeRDP/FreeRDP/issues/4635#issuecomment-387399659,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AdI3NmPttGXfBBtgj5Uk3HWiMzx5JNmQks5twZu9gaJpZM4T2Uum
.

How to set debug symbol paths on windows?
On Thu, May 10, 2018 at 12:14 PM, Aditya Sood<EMAIL_ADDRESS>wrote:

I am having problems understanding the project.
As we are working on Remote Desktop Protocol , we need to have a server in
one system and client in another.
We have files for both client and server in the main folder.
Do we need to build both separately or we have to build them together as a
whole.
Need clarification on this. I am not able to understand the working of
FreeRDP
On Tue, May 8, 2018 at 6:51 PM, akallabeth<EMAIL_ADDRESS>wrote:

@adityasood11 https://github.com/adityasood11 Looks like the debug
symbol paths on windows with ninja builds need to be set.
don't have such a setup to test currently, but pull requests are welcome.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/FreeRDP/FreeRDP/issues/4635#issuecomment-387399659,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AdI3NmPttGXfBBtgj5Uk3HWiMzx5JNmQks5twZu9gaJpZM4T2Uum
.

@adityasood11 You're mixing up quite a lot here. Cross compilers are a completely different story (like compiling windows executables on a linux machine, or arm code on x86). if (${CMAKE_GENERATOR} MATCHES "NMake Makefile*") 		set(CMAKE_PDB_BINARY_DIR ${CMAKE_BINARY_DIR}) endif()
is setting the debug path for NMake. Similar will be the one for Ninja but I don't know where the .pdb files are put with that generator.

Seems like i did not configure openssl first.
First i configured Openssl
cmake -DOPENSSL_ROOT_DIR=/usr/local/ssl -DOPENSSL_LIBRARIES=/usr/local/ssl/lib
Then i specified the generator on Command Prompt.
cmake . -G "Visual Studio 15 2017"
Also i did not use VISUAL STUDIO IDE , instead i started working on Developer Command Prompt for VS 2017.
These steps seem to have eradicated my issue.


[INTERNAL] Fix generateBundle task when bundleOptions are not passed

🚧  Just as a starting point to discuss a solution.
This issue appears if there are no 'bundleOptions' defined in the ui5.yaml (e.g. sap.ui.webc.common).

Coverage increased (+0.002%) to 94.784% when pulling ec077a91695c1fb86e6e1bd48a54911560702ce4 on fix-generateBundle-task into 04072a26978db57cb9ceb9f3428ea6ee9a23ecd8 on next.

👍🏻


test: add setupVitest to tsconfig

Before this commit

@sxzz   Do you have a minute to review this PR ? THX

@yyx990803  Updated

@sxzz  Updated


Update README

Update the README with the following changes:

Updated Tutorials section - this was outdated, so have replaced this with the relevant links to the docs.
Separated the installation instructions for the user and developer. The user installation now points to the docs for the most up-to-date instructions.
Separated the developer installation instructions into clearer sections based on operating system
Fixed typos & updated wording where needed
Added a Contents section at the top for easier navigation

Things I'm still not sure about:

Do we keep the Getting started section at the beginning or do we just direct to the docs?
Do we keep the Running the examples heading at the end of the developer installation section? They're not complete but might be useful?

@Ragzouken, quick question. We currently state to update nanover using: conda install nanover-server. Should we change this to conda update nanover-server or is there a reason that we use the install instead?

Things I'm still not sure about:

Do we keep the Getting started section at the beginning or do we just direct to the docs?
Do we keep the Running the examples heading at the end of the developer installation section? They're not complete but might be useful?

Maybe it's good to have something directly in the README even if the docs are better.
I think keeping the examples there is fine.


Add an option to specify where to fetch translations from

See #423

Is this still needed?
If so I will create another PR for the issue
But using eval is too much...


Add photo

Any plans on implementing this soon?
https://developer.foursquare.com/docs/photos/add.html

I'm a bit swamped right now but would definitely love to implement this.  If you feel like taking a stab at it please feel free and send a pull request.

This has now been added to version 1.3.


Design for ARC20

I believe there is already an upgradable program concept in Aleo, but it's not being further explored. Seems it is too early now to include that concept in the token standard. Personally, I'd like to see a minimal working token standard without all "fancy" features - they should be added later through updated ARCs or not included in the token standard at all.
It seems this program tries to contain many tokens within one program. Considering a program could only have one owner, I'd think it's a bad idea to let my token live in a program that I don't have the ultimate authority (it's unclear what the program owner could do in the future, but still). This approach might also limit what tokens could do; some tokens, while adhering to a common structure, might want to have their own features. IMO, a token standard should define a contract (what to have) for custom tokens, not a container for them.
The Credential concept seems strange to me - shouldn't it be stored in a public mapping instead of a private record? I believe this closely links to the point 2 above, and I think it's not a good approach for a token standard.

I might propose an ARC20 later as well


java.net.SocketException: Broken pipe (Write failed)

Hi, I got many errors after sendAction (call) more than 100 calls.
java.net.SocketException: Broken pipe (Write failed)
Timeout waiting for response to Originate
Everything was good if the calls less than 50.
Could you help me?

are you originating async? are you going to fast on generating? if you are generation to a provider does your provider support the cps that you are doing?

It sounds like you are overloading Asterisk. There is nothing we can do on the asterisk-java side to help with that. You should decrease the number of originates you are doing or decrease the rate that you are performing originates.


RunReactants depends on the order of molecules in list

I'm not sure if this is expected behavior, but it is counter-intuitive to me.
To use a documentation example for the reactions:
>>>rxn = AllChem.ReactionFromSmarts('[C:1](=[O:2])-[OD1].[N!H0:3]>>[C:1](=[O:2])[N:3]')
>>>ps = rxn.RunReactants((Chem.MolFromSmiles('CC(=O)O'),Chem.MolFromSmiles('NC')))
>>>len(ps)
1

but if I change the order of input molecules the match isn't made:
>>>rxn = AllChem.ReactionFromSmarts('[C:1](=[O:2])-[OD1].[N!H0:3]>>[C:1](=[O:2])[N:3]')
>>>ps = rxn.RunReactants((Chem.MolFromSmiles('NC'),Chem.MolFromSmiles('CC(=O)O')))
>>>len(ps)
0

Is it required to have the input molecules in the correct order for reactions to run?  If so, are there general strategies to rearrange the input list?

This isn't a bug.
If you look at the reaction object, it has a list of reactant definitions.  In your case
[C:1](=[O:2])-[OD1] -> reactant 1
[N!H0:3] -> reactant 2

These are the same order as the reactants in RunReactants.
If you want to assemble a list of possible reactants, you can use the reagent definitions directly:
possible_mols = <some list>

bbs = []
for reactant in rxn.GetReactants():
   reactants = [m for m in possible_mols if m.HasSubstructMatch(reactant)]
   bbs.append(reactants)

Now bbs[0] are the list of mols for the first reactant and bbs[1] are for the second.
A lot of this is encapsulated in the EnumerateLibrary classes.
library = rdChemReactions.EnumerateLibrary(rxn, bbs)

will allow enumerating all products.  More details here:
https://github.com/rdkit/rdkit/blob/master/Code/GraphMol/ChemReactions/tutorial/EnumerationToolkit.ipynb

@bp-kelley thank you for clarifying that.  I had looked through the enumeration classes, and now they make a lot more sense.


fix(base-modal): fixed ssr error in element matches polyfill

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

3 of 8 (37.5%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.1%) to 81.943%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

packages/base-modal/src/matches-polyfill.ts
2
7
28.57%

Totals

Change from base Build<PHONE_NUMBER>:
-0.1%

Covered Lines:
6269

Relevant Lines:
7038

💛  - Coveralls


Allow to set case-sensitive headers.

Even though Request.Header.Add canonizes headers, some web servers
don't standardize them and they're passed to the apps in
a case-sensitive manner.
This PR allows users to set case-sensitive headers to the request
@sschepens @j16sdiz what do you think?

Thanks. This one solve my immediate problem.
But I think the http spec allow adding the same header multiple time... still thinking what is the right way to do that.

@marcosnils @j16sdiz is right about duplicate headers, we should check how Header.Add behaves when header already exists, because Add contrary to Set should allow multiple headers with the same key.

There is something about this PR that I think might fail for a lot of users. We don't have a test for that right now, as far as I could see, so this is why it is not obvious.
With the current implementation if you add the same header but with a different value, since we use golang's 'Add' we end up with one headers with several values, which is the expected behaviour and also it is compliant with the RFC.
With this PR, this behaviour is going to change, and if the same header is added with a different value, the last one will overwrite the first one.
I need someone else to analyze the code with this in mind to tell me if I am right.

closed in favor of #108


Clear command history after 'rbenv rehash' in Fish shell

I'm reading the code of libexec/rbenv-sh-rehash, and I found this line of code, which says that if the user's shell is Fish, we execute a no-op.  This is different from what happens with other shells, where we clear the user's command history cache via the "hash -r" command.
It seems like the original call to "hash -r" was added in response to this Github issue, and that the no-op for the Fish shell was added as part of this PR.  But I don't see a Github issue related to the latter PR, or any discussion around why the Fish branch is a no-op.  So I'm curious why if there's a reason not to clear the command history if the user's shell is Fish, when it is cleared for other shells.
According to the Fish docs, Fish supports the "history" command, which does something similar to Bash's "hash" command.  And "history clear" would achieve the same outcome as "hash -r".  So theoretically we could replace the no-op with "history clear".  According to the docs, prefacing the command with "builtin" is the way to circumvent the "Are you sure?" prompt which otherwise appears after running this command.

I believe the 2>/dev/null and ; or true code is correct Fish syntax, but could use an extra set of eyes on that to confirm.

Thanks, but you might be misunderstanding what we are clearing in other shells. We are not clearing command history per se, but a cached resolution of a command to its absolute location on disk.

This is different from what happens with other shells, where we clear the user's command history cache via the "hash -r" command.

In other shells, what we are actually clearing is the sort of shell session cache that remembers that a previously run ruby command resolved to e.g. /usr/bin/ruby. After doing the rbenv rehash process, there now might be additional entries in PATH (e.g. ruby should not resolve to ~/.rbenv/shims/ruby), and it's good to do hash -r to bust the cache.
In fish shell, this cache-busting didn't seem necessary, and so it is a no-op in fish. Did you find evidence that cache clearing is necessary in your case?

We are not clearing command history per se, but a cached resolution of a command to its absolute location on disk.

Yep, I deduced that.  I think my mistake was in assuming that Fish's history command served the purpose of caching the commands and their locations as well.  But upon further inspection, it looks like Fish may be maintaining that history for other reasons, unrelated to caching the location for faster execution.  This implies that my solution doesn't do the job I intended it to do, after all.
Closing this PR.


ChannelShuffle transforms

🚀 The feature
Is there any chance torchvision could implement a ChannelShuffle transform?
Motivation, pitch
Albumenations has it.
https://albumentations.ai/docs/api_reference/augmentations/transforms/#albumentations.augmentations.transforms.ChannelShuffle
Alternatives
I'm currently implementing this myself by hand or using Albumenations.
Additional context
Channel Shuffle is useful since it allows me to mix up the input to prevent overfitting.
cc @vfdev-5

Sure, that sounds reasonable. Perhaps only on the V2 namespace. Thoughts @vfdev-5 @pmeier ?

RandomPhotometricDistort (v2 only) uses that internally already:
https://github.com/pytorch/vision/blob/e5bf7cf85dcab36a313890deb57b8eb864f06292/torchvision/transforms/v2/_color.py#L270-L273
https://github.com/pytorch/vision/blob/e5bf7cf85dcab36a313890deb57b8eb864f06292/torchvision/transforms/v2/_color.py#L244-L257
So basically we only need to extract this out into a transform. That would also make the implementation more clean, since apart from the channel permutation, all other ops that RandomPhotometricDistort is calling have a dedicated functional.

@dhruvbird could you please open a separate issue for that to keep the scope narrow?

Sure thing! https://github.com/pytorch/vision/issues/7633


Fix definitions of combined/common log formats

These formats have two initial fields to ignore (the RFC 1413 identity and
the HTTP authenticated user).

I'm actually pretty surprised a bug like this would remain in place for such a long time, so maybe I'm just missing something.
I'm using the combined log format as documented at https://httpd.apache.org/docs/2.4/logs.html#combined. I noticed that my transmit stats were way off; for example, downloading a multi-GB file wouldn't increase the Tx. Amount value at all. This change fixes that for me.

There's no need for the extra %^. One ignore will look for the next character, so %h %^[ will skip everything until it finds [. Can you post your access log, I can take a look at what format you will need.

Ah, I see. The description of %^ in https://goaccess.io/man#custom-log doesn't seem to mention what constitutes a "field", and there are various other predefined log formats in goaccess that use adjacent %^ specifiers. I think being explicit about the fact that there are actually two fields would make things clearer.
In any case, my format is the combined log format as I mentioned. Here's an example I pulled out (with the IP address modified).
<IP_ADDRESS> - - [28/Jun/2020:14:40:14 -0700] "GET /centos/7/os/x86_64/repodata/repomd.xml HTTP/1.1" 200 3736 "-" "urlgrabber/3.10 yum/3.4.3"

I use these definitions in httpd.conf:
LogFormat "%h %l %u %t \"%r\" %>s %b \"%{Referer}i\" \"%{User-Agent}i\"" combined
CustomLog "|bin/rotatelogs -f -n 30 -L logs/access_log logs/daily/access_log 86400" combined

(Note that LogFormat is the combined format documented at https://httpd.apache.org/docs/2.4/logs.html#combined.)
Does goaccess have any way to log entries that it thinks are malformed?


Unequal width on CI

https://ci.appveyor.com/project/gucong3000/log-symbols/build/1.0.2/job/bohx8aujk836fm07
https://travis-ci.org/sindresorhus/log-symbols/jobs/159114069
https://circleci.com/gh/gucong3000/log-symbols/1

info is wider than the others.

The width is unfortunately system-dependent. Looks better on macOS for example:

Windows 10.0.15063
Chrome 59.0.3071.115

Yes, the screenshot is from Chrome on macOS.

Closing as there's nothing we can do about this. It's the OSes that decides how to render different characters.


be sure to check deploy health for incremental bounce

Thanks @wsorenson for pointing this out. I reworked/reordered the if statements so that we will check the health of the matching tasks for incremental deploys as well before deciding if we can shut a task down
/cc @tpetr

You should add a test which fails pre-PR and succeeds post-PR.

@wsorenson turns out the previous test I wrote for incremental bounce served that pretty well with some modifications. I extended it so that there is an explicit check that a old tasks will not shut down from a task in TASK_RUNNING but without a passing healthcheck.


Microsoft.Azure.WebJobs.Extension.Storage.Queues v5.0.0: How to do a dynamic binding in Azure Functions to put a QueueMessage in a storage queue

Query/Question
I have this HTTP triggered function that uses a dynamic output binding to put a message on a queue. This works fine with Microsoft.Azure.WebJobs.Extension.Storage v4.x but since v5 the types have changed (no more CloudQueue) and I can't find any documentation which type should be used instead.
v4.x
var cloudQueue = await binder.BindAsync<CloudQueue>(queueAttribute);
await cloudQueue.AddMessageAsync(cloudQueueMessage);

What would be the way to setup dynamic binding with Microsoft.Azure.WebJobs.Extension.Storage.Queues v5?
Environment:

Microsoft.Azure.WebJobs.Extension.Storage.Queues v5
Windows & .NET Core 3.1.408
VSCode

@marcduiker We are looking into it and get back to you.

@marcduiker While I have not been able to try it out, I think that it is because the underlying Azure Storage SDK was upgraded to 12.x
CloudQueue was used in Version 11
It should now be QueueClient in Version 12

@marcduiker do you still need help?


vim inserts are interrupted by cursor movement, vim-mode inserts aren't

In vim, if you move your cursor around (with mouse or with arrow keys), an insert gets confirmed and a new gets started - they can be undone separately, and only the last part can be redone. To demonstrate:

have 1234 in the text editor, with cursor on 2
type iab<right>cd<esc> – you will have 1ab2cd34 in the text editor
typing u should only undo cd, typing . should only insert cd in the current place

But in vim-mode, typing u undoes both ab and cd insertion, and typing . inserts abcd.
See vim docs for more info about this.
I'll now have a quick look if I can fix this easily.

I'd like to emphasize the importance of this one — sometimes I'm in insert mode for a long time, and find that undo undoes WAY too much. Sometimes the last several minutes of edits. I find that this leaves me no meaningful way to only undo a little of it.
It also overrides the behavior of cmd-z (which makes sense, but leaves no alternative when u undoes too much)

+1

Thanks, I should soon get some time to rebase this to the newest state of
the code. @maxbrunsfeld any suggestions in case you don't quite like how it
was done?
On 3 Jun 2015 22:20, "Markus Nilssen"<EMAIL_ADDRESS>wrote:

+1
—
Reply to this email directly or view it on GitHub
https://github.com/atom/vim-mode/issues/565#issuecomment-108618928.

As stated in the README, this package is no longer maintained and is deprecated. We recommend that people use the vim-mode-plus package instead. Because of this, we are archiving this repository and closing all issues and pull requests. Thanks very much for your support and contributions!


Update the title and urls from AddressBook to OneBook

Coverage remained the same at 92.159% when pulling bbd4d49c95b405c8a21e51aae05efc47814efbdb on darrinloh:master into 69b54645d9a664e98660a0d4e8af36dbf40434d4 on CS2103-AY1718S1-T17-B4:master.

Coverage remained the same at 92.159% when pulling bbd4d49c95b405c8a21e51aae05efc47814efbdb on darrinloh:master into 69b54645d9a664e98660a0d4e8af36dbf40434d4 on CS2103-AY1718S1-T17-B4:master.


ignore DS_Store file

Reasons for making this change:
The .DS_Store file is a file created by OS that holds meta-information about a directory. You can expect to always find it and can safely ignore it. The file is not shown by the Finder.
Links to documentation supporting these rule changes:
https://en.wikipedia.org/wiki/.DS_Store

I disagree with this change: this .DS_Store should be in your .gitignore_global file on any macOS system: it's totally unrelated to Swift or other programming languages.

To add to @Coeur's comment - we also have a Global/macOS.gitignore template that contains this rule.

@shiftkey you are absolutely right. I couldn't realize global one. I close the PR! Thanks.


Update service-connection-tool-not-download-updates.md

@rashid2982 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Merged due to critical call spike, will email triage alias to review


Add Developer Mode Detection

This pull request adds new functionality to the jailbreak_root_detection package, which checks if the device is in developer mode or not. Developer mode is a setting in Android devices that allows users to perform advanced debugging and testing activities.
The new feature introduces a method called isDevMode in the JailbreakRootDetection class. This method checks for the presence of certain system properties and environment variables that indicate if the device is running in developer mode.
Benefits:

Enhances the package's capability to detect potential vulnerabilities or security risks associated with developer mode.
Provides an additional layer of security check for applications that need to ensure the device is in a secure state.
Improves the overall robustness of the package by covering a wider range of device configurations.

Implementation Details:

The isDevMode() method retrieves the value of the Settings.Global.DEVELOPMENT_SETTINGS_ENABLED system setting using the Settings.Secure.getInt() method.
If the value is non-zero, it means that the device is in developer mode, and the method returns true.
If the value is zero, it means that the device is not in developer mode, and the method returns false.
The method checks if the activity instance is available before attempting to access the system settings. If the activity is null, it returns false.

Usage:
To check if the device is in developer mode, simply call the isDevMode() method:

Thank you for the pull request.


dart: Use Nix instead of Pub

Description of changes
This PR removes fetchDartDeps in favour of a new pub2nix library. Instead of using Pub to generate a large fixed-output derivation, the pubspec.lock file is parsed, and a derivation is made for each individual package.
This improves the user experience significantly:

Dependency derivations can be shared between applications
The vendorHash is no longer needed (though Git dependencies still do need manual hashes)
There's no need to work around Pub's difficult Git and package caching behaviour

This PR builds on #262789, so for review, this comparison between the branches is appropriate. This can be changed if we decide against #262789, though - nothing relies on it directly.
All existing Dart and Flutter packages build and run, with the exception of dart-sass-embedded, which was already broken with Dart 3. I've removed it in this PR, as it has been discontinued upstream and merged into the regular dart-sass package.
Things done

Built on platform(s)

[x] x86_64-linux
[x] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

@NixOS/flutter

Result of nixpkgs-review pr 263345 run on aarch64-linux 1

4 packages marked as broken and skipped:

rstudio
rstudio-server
rstudioServerWrapper
rstudioWrapper

26 packages built:

dart-sass
dart-sass.pubcache
discourse
discourseAllPlugins
domine
domine.pubcache
expidus.file-manager
expidus.file-manager.pubcache
firmware-updater
firmware-updater.pubcache
fluffychat
fluffychat.pubcache
flutter
flutter-unwrapped
hover
protoc-gen-dart
protoc-gen-dart.pubcache
python310Packages.nbdev
python310Packages.nbdev.dist
python311Packages.nbdev
python311Packages.nbdev.dist
quarto
quartoMinimal
shopware-cli
yubioath-flutter
yubioath-flutter.pubcache

I have added a mechanism to customise Dart package derivations. If any packages require patches, they can now be easily added.
Setup hooks have replaced the old package override mechanism, and can be used to do things like set environment variables and add to the FFI runtime dependency list. We should have probably done this from the beginning, but now it's even easier as the setup hooks can be built into the package derivations themselves.
These changes allow the dependency list JSON file to be removed.

Result of nixpkgs-review pr 263345 run on aarch64-linux 1

4 packages marked as broken and skipped:

rstudio
rstudio-server
rstudioServerWrapper
rstudioWrapper

2 packages failed to build:

fluffychat
fluffychat.pubcache

24 packages built:

dart-sass
dart-sass.pubcache
discourse
discourseAllPlugins
domine
domine.pubcache
expidus.file-manager
expidus.file-manager.pubcache
firmware-updater
firmware-updater.pubcache
flutter
flutter-unwrapped
hover
protoc-gen-dart
protoc-gen-dart.pubcache
python310Packages.nbdev
python310Packages.nbdev.dist
python311Packages.nbdev
python311Packages.nbdev.dist
quarto
quartoMinimal
shopware-cli
yubioath-flutter
yubioath-flutter.pubcache

Result of nixpkgs-review pr 263345 run on aarch64-linux 1
4 packages marked as broken and skipped:
* rstudio

* rstudio-server

* rstudioServerWrapper

* rstudioWrapper

2 packages failed to build:
* fluffychat

* fluffychat.pubcache

24 packages built:
* dart-sass

* dart-sass.pubcache

* discourse

* discourseAllPlugins

* domine

* domine.pubcache

* expidus.file-manager

* expidus.file-manager.pubcache

* firmware-updater

* firmware-updater.pubcache

* flutter

* flutter-unwrapped

* hover

* protoc-gen-dart

* protoc-gen-dart.pubcache

* python310Packages.nbdev

* python310Packages.nbdev.dist

* python311Packages.nbdev

* python311Packages.nbdev.dist

* quarto

* quartoMinimal

* shopware-cli

* yubioath-flutter

* yubioath-flutter.pubcache

fluffychat is broken because libsecret is failing tests on master, but it's fine on this branch.

Result of nixpkgs-review pr 263345 run on x86_64-linux 1

30 packages built:

dart-sass
dart-sass.pubcache
discourse
discourseAllPlugins
domine
domine.pubcache
expidus.file-manager
expidus.file-manager.pubcache
firmware-updater
firmware-updater.pubcache
fluffychat
fluffychat.pubcache
flutter
flutter-unwrapped
hover
protoc-gen-dart
protoc-gen-dart.pubcache
python310Packages.nbdev
python310Packages.nbdev.dist
python311Packages.nbdev
python311Packages.nbdev.dist
quarto
quartoMinimal
rstudio
rstudio-server
rstudioServerWrapper
rstudioWrapper
shopware-cli
yubioath-flutter
yubioath-flutter.pubcache

@FlafyDev do you want to rebase this to another branch? I can then force push it to this one.

@mkg20001  I've rebased this PR on this branch
It was a bit tricky so I hope I did it right.
discourse and discourseAllPlugins don't build on master. It's not due to this PR.
This is the nixpkgs review for 32e3ea18d572250c397694dca1d3158493637357:
Result of nixpkgs-review run on x86_64-linux 1

2 packages failed to build:

discourse
discourseAllPlugins

32 packages built:

dart-sass
dart-sass.pubcache
domine
domine.pubcache
expidus.file-manager
expidus.file-manager.pubcache
firmware-updater
firmware-updater.pubcache
fluffychat
fluffychat.pubcache
flutter
flutter-unwrapped
hover
intiface-central
intiface-central.pubcache
localsend
localsend.pubcache
protoc-gen-dart
protoc-gen-dart.pubcache
python310Packages.nbdev
python310Packages.nbdev.dist
python311Packages.nbdev
python311Packages.nbdev.dist
quarto
quartoMinimal
rstudio
rstudio-server
rstudioServerWrapper
rstudioWrapper
shopware-cli
yubioath-flutter
yubioath-flutter.pubcache

I think this would be a bit complicated to do. Probably better to keep it to a future PR if necessary. Also if we make an update script for buildDartApplication and buildFlutterApplication it wouldn't really matter.

I'm not too keen in the idea of reducing granularity for just a little convenience. IIRC, running a build with blank hashes and --keep-all should be enough to get all the hashes in one hit.
This would also require custom fetching logic, as I don't think there's any off-the-shelf solution to make a FOD out of multiple Git repositories.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/3200

Result of nixpkgs-review pr 263345 run on x86_64-linux 1

2 packages failed to build:

discourse
discourseAllPlugins

32 packages built:

dart-sass
dart-sass.pubcache
domine
domine.pubcache
expidus.file-manager
expidus.file-manager.pubcache
firmware-updater
firmware-updater.pubcache
fluffychat
fluffychat.pubcache
flutter
flutter-unwrapped (flutterPackages.stable)
hover
intiface-central
intiface-central.pubcache
localsend
localsend.pubcache
protoc-gen-dart
protoc-gen-dart.pubcache
python310Packages.nbdev
python310Packages.nbdev.dist
python311Packages.nbdev
python311Packages.nbdev.dist
quarto
quartoMinimal
rstudio
rstudio-server
rstudioServerWrapper
rstudioWrapper
shopware-cli
yubioath-flutter
yubioath-flutter.pubcache

This PR seems to break dart compile on Darwin. https://github.com/NixOS/nixpkgs/issues/279772

I think the SDK version parsing in pubspec.yaml is broken:

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/hugo-missing-dependency-already-packaged-but-broken-dart-sass-embedded/39693/2


Set the surface name when GLTF file is imported.

When the models are imported using GLTF the surface name wasn't set, so was impossible to differentiate one surface from the other.
The add_surface API is correctly used in all the other parts of the GLTF importer (and FBX too).

Already done, yup.

Thanks!

This doesn't seem easily cherry-pickable for 3.x (the API is different and don't take a name argument).


Add query "Global Server Object Uses HTTP" #2837

Closes #2837
Proposed Changes

Add query "Global Server Object Uses HTTP" that checks if:
servers global object is defined
servers global object count > 0
servers global object's URL uses HTTP

I submit this contribution under Apache-2.0 license.

Is it a vulnerability that Global Server object isn't defined, since it isn't required?

I think it should be a vulnerability, since the default value for URL will be / as you can see on open api object servers field definition

@felipe-avelar @rogeriopeixotocx I believe we have a query to check if at least a server is defined meaning we will have 2 results for the same problem


Dev

deadlock, build warning, issue 103

This pull request is now managed with CodeReviewHub. Refresh.

:shipit:


Mapped to wrong character

Noticed the fonts weren't working on Putty. (Font switch and encoding set to utf-8)
Loaded powerline in putty and confirmed their value was correct by copying the character that should have been the left arrow into a hex editor: 0xee82b2 (U+E0B2).  Opened the font viewer and found the character I should be seeing. This same character appears to be mapped to (U+2B82) instead.

I think you're right. I can't see the Powerline characters in PuTTY.


Remove --zookeeper arg from Application Reset Tool test

I'm just going to merge it so we get things building again


Create CONTRIBUTING.md file

As I'm working on the issue #13 I'm writing here a first version CONTRIBUTING.md guidelines, using as references:

https://www.contributor-covenant.org/pt-br/version/1/4/code-of-conduct/
https://github.com/atom/atom/blob/master/CONTRIBUTING.md#i-dont-want-to-read-this-whole-thing-i-just-have-a-question
https://reactjs.org/docs/how-to-contribute.html

Is that ok?

I'm also thinking about referencing https://www.conventionalcommits.org/ as a good practice for code commiting, but not mandatory.

I'll write a quick contribution guideline now.. I'm at work (like you) so will do more work on this tonight..

first draft

That's pPerfect, thanks!

I can write a "Development Workflow" session on the CONTRIBUTING file as I'm going to the step-by-step of cloning, installing, coding, testing, covering, etc.
What do you think?

For things like developer/contributor notes, files that change often and don't directly affect the codebase, I think a Wiki page is better suited.. Wiki pages are good because people can quickly edit and fix them without the usual PR process.
I created a placeholder on the wiki home page for you to link to a Development Workflow document. 👍

Yeah, you are right. I agree.
Thanks for the tip.!


Update coreteam.css

Changes in Font and Font-styles
Description
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Fixes # (issue no.)

Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Explain the Testing instructions
Test Configuration:

Operating system:
Version:
Text-editors used:

Checklist:

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

ATTACH SCREEN-SHOTS

@s-s5678  Please host your branch on github and share the link

Please also link on which issue you was working !

@s-s5678 Please host your branch on github and share the link

https://s-s5678.github.io/SFC-foundations/pages/About-Us/coreteam.html

Please make it lighter 300 or 400 might look more better @himanshu007-creator  What you think ?

Please also link on which issue you was working !

Updating the font

Please also link on which issue you was working !

Please also link on which issue you was working !

issue #5

Please make it lighter 300 or 400 might look more better @himanshu007-creator What you think ?

issue #5

Please make it lighter 300 or 400 might look more better @himanshu007-creator What you think ?

issue #5

No this is not related to that ?

Please make it lighter 300 or 400 might look more better @himanshu007-creator What you think ?

issue #5

It's ok that you make changes but these are changes we can accept if you might have done something we don't accept or don't like then this will waste your efforts so it is advised to create issue and work on that !!  Just take it as suggestion  @s-s5678

Please make it lighter 300 or 400 might look more better @himanshu007-creator What you think ?

issue #5

It's ok that you make changes but these are changes we can accept if you might have done something we don't accept or don't like then this will waste your efforts so it is advised to create issue and work on that !! Just take it as suggestion @s-s5678

Okay sir!

@s-s5678 you can work on the team chandigarh button. It should be in single line, what you can do is reduce size of all teh buttons and bold them, what do you say @rohank2502 ?

can work on the team chandigarh butt

Can you please attach ss @himanshu007-creator  Because I am not getting where it is in 2 line

Please make it lighter 300 or 400 might look more better @himanshu007-creator What you think ?

issue #5

It's ok that you make changes but these are changes we can accept if you might have done something we don't accept or don't like then this will waste your efforts so it is advised to create issue and work on that !! Just take it as suggestion @s-s5678

Okay sir!

You can say Rohan


Need performance test comparing node-geo-proximity to MongoDB spatial indexing

Hi,
really interesting project. Have you done any testing of the performance. Especially a compassion to mongodb's integrated  geospatial index?
See: http://docs.mongodb.org/manual/applications/geospatial-indexes/
Thanks
Markus

Closing this. Won't be a priority. Benchmarks comparing Native Redis Geo Commands are included in v3.0.0 and more relevant


Update eslint to latest and add linting to build

Updates eslint to our standard version and includes linting in the build

Note I have not updated eslint-plugin-vue which does have a major version bump available. The linting passes without it - but we should look at what were missing with that update.


releaseExternalResources and akka.pattern.AskTimeoutException in multi-threaded environment

We are using the Spark micro framework (http://sparkjava.com/) to create a prototype of our new web application. Spark allows you to define callbacks for routes invoked by client requests, e.g.,
get("/foo", (request, response) -> {
    callback();
});

In one of the routes we employ parallec to concurrently retrieve the information (from another web service) we need to generate the HTTP response for a client request. We follow the procedure shown in the examples, i.e.,
ParallelTaskConfig config = new ParallelTaskConfig();
config.setActorMaxOperationTimeoutSec(35);
config.setTimeoutInManagerSec(900);
config.setTimeoutAskManagerSec(910);

ptb = pc.prepareHttpGet(String.format("/%s/$PARAMS", serviceConf.getServiceRoute())).saveResponseHeaders(respHeaders)
    .setResponseContext(responseContext).setHttpHeaders(new ParallecHeader()
                            .addPair("Accept-Encoding","gzip")
                            .addPair("Accept",serviceConf.getMimeType())
                            .addPair("Accept-Language",this.language))
   .setConfig(config)
   .setConcurrency(100)
   .setReplaceVarMapToSingleTargetSingleVar("PARAMS", identifierList, serviceConf.getServiceHost())
   .setHttpPort(Integer.parseInt(serviceConf.getServicePort()));

ptb.execute(new ParallelResponseHandler());
pc.releaseExternalResources();

We noticed that when querying our web application with multiple concurrent client requests, i.e., one process is spawned for each concurrent request, we often encounter the following error message
Line 267: 84269 [Thread-16] ERROR io.parallec.core.task.ParallelTaskManager  - Exception in sendTaskToExecutionManager akka.pattern.AskTimeoutException: Ask timed out on [Actor[akka://ParallecActorSystem/user/ExecutionManager-PT_14_20170110104948951_853663af-ce4#-964038860]] after [910000 ms] details akka.pattern.AskTimeoutException: Ask timed out on [Actor[akka://ParallecActorSystem/user/ExecutionManager-PT_14_20170110104948951_853663af-ce4#-964038860]] after [910000 ms]:

As it turns out this issue arises only when calling pc.releaseExternalResources() at the end of the callback. Our suspicion is that calling pc.releaseExternalResources() releases external resources that are still required by objects in one of the other spawned processes.
Is there anything we can do to avoid getting this error message? Is parallec "thread safe" in the sense that it can be used as described above? What would be the effect of not calling pc.releaseExternalResources()? Would the external resources get garbage collected?

@sebastianthelen  thanks so much for trying Parallec. As mentioned in the javadoc please not to call this releaseExternalResources() until you stop using Parallec.
If you are on some web server (such as Tomcat) and runs parallec jobs as cron jobs. We do not need to call this line at all.  We have several production application that runs as web server. We do not call it in our app.
The actor system and the threadpools are live so that it will keep reusing the same one.  The resources used are not significant either. Feel free to use tools such as VirtualVM to track the heap usage etc.
releaseExternalResources
public void releaseExternalResources()
Releases the external resources that this object depends on. You should not call this method if you still want to use the external resources (e.g. akka system, async http client store, thread pool for SSH/TCP) are in use by other objects.

@jeffpeiyt thanks for your prompt reply.

If you are on some web server (such as Tomcat) and runs parallec jobs as cron jobs. We do not need to call this line at all.

That's exactly what we were not sure of. We have read the javadocs for releaseExternalResources() but were not sure if this also applies in the web server scenario.

The actor system and the threadpools are live so that it will keep reusing the same one.

Understood. We also tracked the heap usage with VirtualVM and everything looked ok.
Again, thanks for clarifying and updating the documentation.

@sebastianthelen Glad that it helped. Thanks for your questions, we will enhance the doc on this. Guess other people will have similar questions.


Resolves #13: Shows example usage of IceNet library with Pytorch UNet and cGAN models

Resolves #13
Adds two notebooks showing the usage of the IceNet library for @ampersandmcd 's GAN.
Notebooks (Both based on Andrew's original code which in turn was based on Tom Andersson's original paper's code):

1_icenet_forecast_unet.ipynb: PyTorch UNet implementation, similar to current Tensorflow implementation in IceNet.
2_icenet_forecast_cgan.ipynb: PyTorch cGAN implementation, using UNet as generator.

The original code has been refactored to match current state of IceNet:

Using IceNet for data download and processing
Showing usage of new IceNetDataSetPyTorch class for interfacing existing code with PyTorch:
from icenet.data.dataset import IceNetDataSetPyTorch
Daily predictions instead of monthly averaged
Binary sea ice concentration (SIC) instead of original triple class (no ice, marginal, and full)
Update and add metrics for binary classification
Using IceNet masks
Using IceNet for netCDF output of predictions and plotting

It also includes an environment file (pytorch/environment.yml) which will help set up both Tensorflow and PyTorch together with CUDA support in a conda environment on a Linux system (tested on BAS HPC). This should make it easier to run through these notebooks as all dependencies should be covered (including IceNet library v0.2.8_dev).
Thanks to @ampersandmcd and @rchan26 for covering most of the leg work!

Relates to icenet-ai/icenet#225


Use new bosh.io URL to upload stemcells

'bosh prepare deployment' no longer works for
newer stemcells because it uses deprecated
'bosh public stemcells' which in turn uses outdated
stemcell URLs.

Hi, this is a 2nd attempt to fix #84. The change is now minimal and causes less side effect. However, please have a look at method download_stemcell_from_bosh_io. It's implementing a very simple redirect. Let me know if this is sufficient for the fix.

Looks good to me will refactor the download_stemcell_from_bosh_io method later when I find time :smile:

@sklevenz thanks


Help with building flatpak

Hi, I'm trying to build a flatpak but currently facing troubles with : https://github.com/flathub/flathub/pull/1857
If you have any idea how to fix that I'd really appreciate it. BTW the build environment couldn't have any internet so all files must be downloaded before building. Here is my progress: https://github.com/flathub/flathub/pull/1857

I followed the build instructions in the wiki with some minor differences due to the constraints of the build environment

Is there a reason why you're authoring your own build process instead of using ungoogled-chromium-portablelinux?

I thought that I would be a able to target more architectures that way

Is there a reason why you're authoring your own build process instead of using ungoogled-chromium-portablelinux?

What do you mean by that?

Anyway I successfully built a package for amd64 but I need help with arm64 . On arm64 SOLINK fails causing the whole build to fail and I don't know how to fix it.

All building problems resolved.


Hubitat Package Manager -- didn't add

Hubitat Package Manager installed the app but did not add it.  I had to "+ Add user app" for it to show up.

This is the way HPM and Hubitat work.  I will add a note in the instructions though.


feat: Add support for nightmode switch

three lint errors
custom_components/ha_blueair/blueair_aws_data_update_coordinator.py:149:1: W293 [] Blank line contains whitespace
custom_components/ha_blueair/blueair_data_update_coordinator.py:90:1: W293 [] Blank line contains whitespace
custom_components/ha_blueair/switch.py:66:1: W293 [*] Blank line contains whitespace

once the api pr is merged please update
requirements.txt and manifest.json with newly released version for blueair-api

once the api pr is merged please update requirements.txt and manifest.json with newly released version for blueair-api

Will do a separate commit for that, I'm just submitting what I have now for your review!

Needs an upstream merge to resolve a conflict in requirements from ruff being updated by dependabot

Ok fixed!


Enable Travis CI

Enable Travis CI

Add .travis.yml
Update lru_redux.gemspec and Rakefile

I ran the CI test on with Ruby2.6 and 2.7.  Just wondering whether it is worth for merge? Thanks.


[Snyk] Upgrade io.opentelemetry:opentelemetry-sdk from 1.41.0 to 1.42.0

Snyk has created this PR to upgrade io.opentelemetry:opentelemetry-sdk from 1.41.0 to 1.42.0.
:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 1 version ahead of your current version.

The recommended version was released on 22 days ago.

[!IMPORTANT]

Check the changes in this PR to ensure they won't cause issues with your project.
This PR was automatically created by Snyk using the credentials of a real user.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:

🧐 View latest project report
📜 Customise PR templates
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

/ok-to-test


Removing profile support

Removed functions and references to local profiles

LGTM, but I'll defer to @bartreardon as I'm not nearly as familiar with this code base.

the deletions look good but seems you branched from development which has an unmerged change in it - if you can remove outset/Outset.swift from your commit as it's unrelated to removing the profile stuff.

This closes out #8


v2.0.0 WiP

separate packages for v1 and v2 classes

We can remove the v1 classes entirely from master/v2.
As long as v2 is in a separate package from v1, consumers who need both versions for some reason can install them side-by-side.

Are there any API's that are implemented as v1, but not as v2?

Yeah, all v1 APIs are still present. They're being marked as deprecated while v2 is being rolled out.

Lets stick with the current approach of having V1 and V2 together.


BLU: Weaving report fixes

Three closely-related changes here:

Several BLU actions were defined wrong, with castTime: 2500 instead of castTime: 2000, gcdRecast: 2500, which was mistakenly triggering the Weaving report.
The core Weaving module allows a single weave after a 2s cast time GCD, but that always results in a hard clip -- Unsure if this is a bug in the core module, I just now brought it up on the Discord, but in the meanwhile, patching BLU to disallow weaves on any spell with a cast time higher than 1.5s
...except we do allow & expect people to clip 2s cast time spells during Moon Flute windows, so this adds an exception for that case

Forgot to include an example log:
https://www.fflogs.com/reports/QNnLX27TH3d16YGj#fight=4&source=7&view=timeline&type=casts&start=845000&end=856723
Here, at ~02:55, we have the sequence Triple Trident (Feather Rain) => The Rose of Destruction, which is clipping the second GCD since Triple Trident has a 2s cast time; you can also see a similar example at ~01:25, with Tingle (Swifcast) => Triple Trident, since Tingle has a 2s cast time too.


Consolidate the MLH files

Changes
We are working toward removing the compile time configuration. It is currently difficult to tell what precise configuration is being used for a given profile — the style of imports and undefs are contributing to this. As an intermediate step I consolidated all of the configuration into one file per build profile, ultimately to be replaced by a standard yaml/json file. It’s now safe to delete the old files as wel.
For fields that were previously imported, I converted them to blocks with comments to identify the original location, e.g
(*BEGIN src/config/txpool_size.mlh*)
(* Note this value needs to be consistent across nodes to prevent spurious bans.
   see comment in transaction_pool.ml for more details. *)
[%%define pool_max_size 3000]
[%%undef zkapp_cmd_limit]
(*END src/config/txpool_size.mlh*)

NOTE
It would have been insane to try to do this by hand. I wrote a little Purescript script to accomplish this task, it is easier to audit that than to try to cross reference the values with the original files (though feel free to do that if you have the energy).

!ci-build-me

!ci-build-me

!ci-build-me

Awesome work!

!ci-build-me


Add arm64 support for kfserving

Signed-off-by: Henry Wang<EMAIL_ADDRESS>

This change is

/ok-to-test

Hi @jinchihe! I think something strange is going on with the CI? Any ideas?

@MrXinWang That's should be caused by CI test env problem, I'm going to take a look.

@jinchihe Thanks very much!

Cool what's the scenario? Do istio and knative already work on arm? Also a lot more will be required including possibily selecting different base images.. can you raise an issue with details on the use case?
/hold

Hi @rakelkar! I think issue#2237 and this comment  show something that you may be interested in.
Basically we are trying to enable kubeflow on arm64 platform, and yes you are absolutely right that a lot more work is required. When I started to work on this project the latest version of kubeflow was 0.5.1, and istio was introduced in 0.6.0 and later. For now I am just contributing what I have achieved to upstream. We do have some discussions about the base image in kubeflow/pipeline PR#2507, maybe you can also share your opinion about it.

@jinchihe I think the test should be ok now.
/retest

Test flake resolved in another PR. You may want to rebase.
/retest

Cool okay - will be awesome to see it work on arm64. If you are trying to minimize initial scope you dont need the executor as it is not currently used.
/hold cancel

@jinchihe Test works perfectly now, thanks

/lgtm

/approve


PUB-883 Add APIM Product

JIRA link (if applicable)
https://tools.hmcts.net/jira/browse/PUB-883
Change description
Add APIM Product
Does this PR introduce a breaking change? (check one with "x")
[ ] Yes
[ X ] No

Closing PR as work is on branch


Create 2024.1 branch for ovn-octavia-provider

We need stackhpc/2024.1 branch of ovn-octavia-provider to backport https://review.opendev.org/c/openstack/ovn-octavia-provider/+/923196

This PR is no longer needed as the upstream change was also backported to 2024.1 https://review.opendev.org/c/openstack/ovn-octavia-provider/+/926713