Fix ironic rebalance race rocky

I ended up backporting a couple of patches to get the multi-service functional test passing.

CI failure was in docs, I think we can live with that.

Tagged as stackhpc/<IP_ADDRESS>.


Typeahead downshift

Typeahead component using downshift and based on #282
See it in action

[ ] Test added / Snapshots updated
[ ] Documentation added / updated

@dan-lee + @mosch I've implemented the changes we've talked about last week. I've extended the story for this new component accordingly. I've created a complete set of unit tests too and the docs have been generated.
@dan-lee I know we discussed that clearOnSelect prop but I already see some usage where we don't want a controlled or uncontrolled component anyway. So for that specific usage , we can now use that prop. Moreover, in order to avoid any possible confusion for the user, a warning is triggered if a defaultValue or value is used simultaneously (there's also a unit test).


[Issue]: Make fancy component that uses traditional math coords

What needs to change?
Right now, all of our Threlte components use computer graphics coordinates (x horizontal, z vertical, y depth), but in most mathematical applications we use x horizontal, y vertical, z depth. It might be nice to make some wrapper components that use this coordinate system to avoid confusion :)
Does this issue relate or depend on other issues?
No response

Is already been implemented in PR #115


OS X binary - Segmentation fault: 11

I tried the gogs_v0.9.13_darwin_amd64.zip binary file… When running, it returns an error.
$ ./gogs web
Segmentation fault: 11

What are the OS X sys requierments? (I use 10.7.5)

Using “build from source” works fine, no sqlite3 though so I'll look at using the -tags installation option.

I think your OS X is too low, at least 10.8 to use compiled binary, and I don't have that low version to compile.
So... you would have to compile from source code.


Azure Communication Services Support for MAUI

As a community,
we'd like to ask about opportunity to use Azure Communication Services with MAUI apps.
This allows many developers across the world to use Communication Services with appropriate NuGet packages from MS Team.
Would be also great to have documentation with steps "How to" use Communication Services with MAUI / Xamarin apps.
--
Many thanks

Hi @aodegov
Apologies for the delayed response, but it's taken a while to find out the answer. The Azure Communication Services team is currently looking at implementing .NET MAUI controls for Azure Communication Services Storybook. They'll end up doc'ing whatever they create, and the MAUI docs will link to it.

Hi @davidbritch ,
thanks for your reply.
My questions:
Am I correctly understanding, that the package(or library) with connection MAUI to the Azure Communication Services will be processed by the ACS team?
Any help from MAUI team in this process?
And eventually, which Team will create documentation with points how it will work for the mobile devices (MAUI + ACS) ?
Many thanks.


Program panic on unexpected Reddit response

Reddit changed error response format and that revealed bug in program:
  panic: runtime error: invalid memory address or nil pointer dereference
  [signal SIGSEGV: segmentation violation code=0x1 addr=0x28 pc=0x6b6808]

  goroutine 25 [running]:
  github.com/macie/opinions/internal/html.Text(...)
  	/home/runner/work/opinions/opinions/internal/html/traverse.go:45
  github.com/macie/opinions.SearchReddit({0x7cfd50, 0xc0000c4150}, {0x7cd300, 0xc0000800a0}, {0x72f3c9b99bf5?, 0x0?})
  	/home/runner/work/opinions/opinions/reddit.go:65 +0x788

Merged into dev


kubectl create clusterrolebinding -o yaml shouldn't create a binding

Is this a request for help? (If yes, you should use our troubleshooting guide and community support channels, see http://kubernetes.io/docs/troubleshooting/.):
No
What keywords did you search in Kubernetes issues before filing this one? (If you have found any duplicates, you should instead reply there.):
clusterrolebinding

Is this a BUG REPORT or FEATURE REQUEST? (choose one):
Bug
Kubernetes version (use kubectl version):
$ ./cluster/kubectl.sh version
Client Version: version.Info{Major:"1", Minor:"9+", GitVersion:"v1.9.0-alpha.1.1890+b00c15f1a40162-dirty", GitCommit:"b00c15f1a40162d46fc4b96f4e6714f20aef9e6c", GitTreeState:"dirty", BuildDate:"2017-10-27T17:43:51Z", GoVersion:"go1.8.3", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"9+", GitVersion:"v1.9.0-alpha.1.1890+b00c15f1a40162-dirty", GitCommit:"b00c15f1a40162d46fc4b96f4e6714f20aef9e6c", GitTreeState:"dirty", BuildDate:"2017-10-27T17:43:51Z", GoVersion:"go1.8.3", Compiler:"gc", Platform:"linux/amd64"}

What happened:
kubectl create clusterrolebinding -o yaml is creating cluster role binding in addition to output it to the console:
$ kubectl create clusterrolebinding test-binding --clusterrole=restricted-psp-user --user=joe -o yaml
apiVersion: rbac.authorization.k8s.io/v1beta1
kind: ClusterRoleBinding
metadata:
  creationTimestamp: 2017-10-27T18:12:06Z
  name: test-binding
  resourceVersion: "1815"
  selfLink: /apis/rbac.authorization.k8s.io/v1beta1/clusterrolebindings/test-binding
  uid: 573abbeb-bb42-11e7-8b10-507b9d2b16b9
roleRef:
  apiGroup: rbac.authorization.k8s.io
  kind: ClusterRole
  name: restricted-psp-user
subjects:
- apiGroup: rbac.authorization.k8s.io
  kind: User
  name: joe
$ kubectl create clusterrolebinding test-binding --clusterrole=restricted-psp-user --user=joe -o yaml
Error from server (AlreadyExists): clusterrolebindings.rbac.authorization.k8s.io "test-binding" already exists

What you expected to happen:
It shouldn't create a binding and only show the YAML document.
How to reproduce it (as minimally and precisely as possible):
Execute the kubectl create clusterrolebinding test-binding --clusterrole=restricted-psp-user --user=joe -o yaml command twice.

-o option should imply --dry-run option.

I'm also surprised that it uses rbac.authorization.k8s.io/v1beta1 instead of rbac.authorization.k8s.io/v1 for apiVersion.

Actually, in all create resource command, -o  option doesn't imply  --dry-run  flag true, I think it is the just design purpose. and in my environment I try servral times and  it always use the  rbac.authorization.k8s.io/v1 for apiVersion,I can not recurrent it.

I agree with @wackxu that -o changes the way things are printed, not what is performed.
Feel free to close the issue, or I'll do it in a week or so if you're fine with that @php-coder

Ok, probably you are right.


Add support for Voot.com

Make sure you are using the latest version: run youtube-dl --version and ensure your version is 2016.08.07. If it's not read this FAQ entry and update. Issues with outdated version will be rejected.

[x] I've verified and I assure that I'm running youtube-dl 2016.08.07

Before submitting an issue make sure you have:

[x] At least skimmed through README and most notably FAQ and BUGS sections
[x] Searched the bugtracker for similar issues including closed ones

What is the purpose of your issue?

[ ] Bug report (encountered problems with youtube-dl)
[x] Site support request (request for adding support for a new site)
[ ] Feature request (request for a new functionality)
[ ] Question
[ ] Other

If the purpose of this issue is a site support request please provide all kinds of example URLs support for which should be included (replace following example URLs by yours):

Single video: http://www.voot.com/shows/splitsvilla-s09/9/411756/rajnandini-to-settle-scores-with-martina-/434441
Playlist: http://www.voot.com/shows/splitsvilla-s09/9/411756

Georestrictd to IN.

Why is this closed??
I see the support was never added.
Also if I use 9xbuddy.com
and pass the download m3u8 url it provides
youtube-dl starts downloading..
IMHO...It won't be difficult to add support.

@therohanchawla #11644 is closed as it's a duplicate of this issue; this issue is still open.

Another URL from #12350: https://www.voot.com/kids/characters/mighty-cat-masked-niyander-e-/400478/school-bag-disappears/440925

Any status on this?  Ever since MTV India has moved its video library from their site to Voot, Their shitty ad ridden site is getting much popularity. Also,  Voot is owned viacom18 and same backend is used for their other sites, This  Request should be able to add support for their other sites as well.


[feature request] CVS modified lines

Can we have this beautiful VS code hinting of which lines were changed in monaco? (But not invoking diff editor)

Hello @arogozhnikov, did you found any other code editor which has this feature so I could have something for the time being ? Thanks

@KaKi87 nope, sorry. Wasn't looking too vigorously though :)


Crash with access violation on null pointer

Thanks for the wonderful tool! I figured I'd try it out immediately, maybe in a somewhat unintended use-case (a random Rust application I had laying around). But after a while of not paying attention the debugger crashed with an access violation on a 0 pointer.
Is there some logging I could give you that would make hunting this down easier?

Unfortunately there aren't any logs I can point you to, but the included PDB files should let you attach to the debugger with e.g. Visual Studio, and get a call stack. You can also try the debug build included in the .zip.

I had built from source on c8723613dfa42ecbd1bd7fee05ebf7088b14e925 fwiw. Unfortunately I didn't have a debugger attached at the time of the crash, nor could I reproduce it later.
I don't think this issue (as reported) will help you much either, exept potentially as a gentle reminder to add some diagnostics support later on.

Yeah, makes sense! Will close for now, but if you end up finding a solid repro or a callstack, just re-open and let me know.


Submission heidrifx wolkobml

Exercice Link

Looks good overall :+1:

Tests of Ex 1 do not pass.
contains (Ex 3) has two return statements.
Minor: you forked and not imported the original repo.


Create package.json

That simplify lib download

Hi, sorry for the delay. I think we may need to put some more thought into this.

We're missing the multiformats/go-multiaddr-dns dependency.
The pions/webrtc isn't under gx at the moment.
I'll have to get up-to-speed with gx and how it's used in libp2p to be able to get a better understanding of the best way forward.

@backkem
Ok, I'll fix it when pions/webrtc is on gx, (if is it one day)

Hi, sorry for the delay

Np, I do worst :)

Related discussion on gx vs go modules: ipfs/go-ipfs#5850

Closing in favor of go modules support added in 6639a4969691adeee12151f85f302dfc940e25c4.


Site down

hurl.it is a really useful service but seems to have been down for much of the last few months.
Can I help you run it? I'm setting up a rackspace cloud HA cluster and hosting hurl.it would actually help with our http proxying and caching preparations.
Regardless, it'd be nice to see the site working again.

hurl.it down again but but the http://<IP_ADDRESS>/ site is working.

http://hurl.it/ back up

\o/

Thanks!


New item templates

Some way to create multiple records with some of the same metadata (e.g., same geographic origin, language and categories/subjects), without having to select/enter them for each record.

Related: for making the ephemera functionality support multiple different ephemera collections, it would be good to also have collection-based templates, so South Asian Ephemera could have a different set of controlled vocabularies than Latin American Ephemera.


[flashattn] Support Flash Attention to v2.0.1

According flash-attention, I modify the code for supporting flashattn v2.0.1

Closed since fixed in #4347


Repo structural refactor

This PR is a large structural refactoring of the project, mostly around factoring the utils_mod/utils into utils/.

Hi @niksirbi thanks a lot for your review, the changes are implemented in #81. If you are happy and can approve that would be great,cheers!


IOpenLibraryClient interface + loader interfaces

Added example tests to prove that the new IOpenLibraryClient could be correctly handled and mocked using tools such as Moq.
Additionally, added interfaces for the loaders, as these fell under the initial same issue as OpenLibraryClient where base class implementation could not have their method implementations mocked.
Happy for discussions around whether the examples tests I added in should be added into the main branch, or more happy as a proof of concept that the implemented changes has worked as intended but provide no value in main. Or whether additional test of all the loaders, and other methods would be beneficial.
I have been able to successfully run the new tests I added, but have been unable to run the original tests in the project (Issue existed before any changes were made). So validating these tests are still passing green from a maintainer that is able to run these tests would be amazing!

I had issues building large parts of the code based locally due to the dependency references in the csproj. Resolved this locally in the end, but did hamper the easy building and test running
I absolutely agree about the example tests. I don't see there being much value with these existing on the main line. I added them primarily for confidence that the interface implementation was correct for mocking.

I had issues building large parts of the code based locally due to the dependency references in the csproj.

Ah, I see. If you have the time, feel free to open an issue for the specific issues you were having. I made a little effort to make the relative paths of the package references neat but never tested on another machine

Went ahead and cherry-picked the relevant commits 👍


add: taints for machinePools

Issue link

What changes have been made
Taints are being implemented to manage the dispatch of appwrappers. When an appwrapper is deployed, it's crucial that its specified nodes are directed to the appropriate scaled-up instances. This mechanism is particularly vital in team-based clusters that scale up multiple instances for various appwrappers. The currrent workflow potentially allows appwrappers to distribute their workload across scaled-up instances not assigned to them, leading to resource mixups.
By applying taints, we ensure that pods are assigned to the correct nodes, preventing such issues.
The same will be done for nodepools separately.
Verification steps
Deploy an appwrapper with instascale enabled, and specified instances.
View you scaled up machinepool in the dashboard, and check that a taint has been applied with
<aw-name> = value1:PreferNoSchedule.
Once the nodes are scaled you can again check within compute and nodes. Feel free to filter with labels=aw-name=aw-name. When viewing each nodes Details you will see again the same taint is visible.
Ensure the appwrappers pods are deploying successfully to these nodes.
To ensure the opposite, feel free to scale up resources with instascale, and deploy a separate appwrapper, ensuring this appwrappers pods are not deployed to the scaled up resources.
Checks

[ ] I've made sure the tests are passing.
Testing Strategy

[ ] Unit tests
[ ] Manual tests
[ ] Testing is not required for this change

Thanks for this PR, please make sure we patch taints and change tolerance in the reuse scenario.

Thanks @asm582
There are currently no reuse scenarios in place for machinepools

Thanks @asm582 There are currently no reuse scenarios in place for machinepools
I think reuse is important. we should have it as a part of this PR or the following PR. Can you please check if an issue exists and if not please create a new one.

Hey, I've spent some time looking into it. Great idea, I'll open a new issue and get reuse added for each machine type :)


Update README

Outline

Introduction
Getting Started
Building

Follow-up from #304

Only the horizontal grammar is available for preview at this time. The vertical grammar is in the design phase and will be added to the project over the next several months.

Well, it's been several months, and they're added, now..


Cleaned up notebook text a bit.

Most of the work was done in model_building.

@ehrlinger,
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


Fix long lines in code blocks (above 95 characters)

This does not change grammar.md; once we've agreed on grammar
changes, the grammar extraction tool can do that.
The grammar changes are almost all a matter of moving end-of-line
comments to line-above-production comments. In some places these
have been done even when the line wasn't too long, just for
consistency. (The casing of the comment has been changed slightly in
some cases as well; somehow sentence fragments feel more
acceptable for end-of-line than line-above-production comments.)
In some cases very long productions have just been split over more
lines.
Fixes #434.

I figure 2 out of 3 reviewers is sufficient :)


unknown_error: Something went wrong

Hey
Getting the following error from the CLI and Dashboard. My apps seem to be running fine.
unknown_error: Something went wrong

When i try and git push, i get the following:
remote: Internal server error
fatal: unable to access 'https://git.****.flynnhub.com/****.git/': The requested URL returned error: 500

Let me know if you want the logs. They contain my IP's etc so i don't feel comfortable posting the here.
Version: v20161114.0p1
Digital Ocean
3 hosts in the cluster

Also, just to note, this was a new cluster when 20161114 came out, and it was created with backup/restore.

I had to upgrade my Flynn CLI when I got similar errors.

Just tried, and it didn't help. I upgraded to 20161218.1 too, but i am still having the issues.
INFO[12-28|09:07:54] flynn-host updated successfully          host=flynn14814084501
INFO[12-28|09:07:54] flynn-host updated successfully          host=flynn14814084500
INFO[12-28|09:07:57] flynn-host updated successfully          host=flynn14814084502
EROR[12-28|09:07:59] cluster status is unhealthy              code=500

Running flynn-host fix fixed this

I've taken a look at the logs and it looks like the discoverd leader had trouble communicating with the two followers between 2016-12-19T07:42:49+0000 and 2016-12-19T07:45:09+0000.
Lots of services were marked as down and then back up again, including postgres which lead the primary to assume a null role and crash:
t=2016-12-19T07:42:53+0000 lvl=info msg="discoverd peers" app=postgres component=peer peers=2
t=2016-12-19T07:42:54+0000 lvl=info msg="discoverd peers" app=postgres component=peer peers=1
t=2016-12-19T07:43:05+0000 lvl=info msg="discoverd peers" app=postgres component=peer peers=0
t=2016-12-19T07:43:42+0000 lvl=info msg="discoverd peers" app=postgres component=peer peers=1
t=2016-12-19T07:43:53+0000 lvl=info msg="discoverd peers" app=postgres component=peer peers=0
t=2016-12-19T07:44:17+0000 lvl=info msg="discoverd peers" app=postgres component=peer peers=1
t=2016-12-19T07:45:01+0000 lvl=info msg="discoverd peers" app=postgres component=peer peers=2
t=2016-12-19T07:45:02+0000 lvl=info msg="got discoverd state" app=postgres component=peer fn=handleDiscoverdState index=570 state=true
t=2016-12-19T07:45:02+0000 lvl=dbug msg=moving app=postgres component=peer
t=2016-12-19T07:45:02+0000 lvl=info msg="starting state evaluation" app=postgres component=peer fn=evalClusterState
t=2016-12-19T07:45:02+0000 lvl=info msg="assuming deposed role" app=postgres component=peer role=deposed fn=assumeDeposed
t=2016-12-19T07:45:02+0000 lvl=info msg="discoverd peers" app=postgres component=peer peers=1
t=2016-12-19T07:45:02+0000 lvl=info msg="skipping config apply, no changes" app=postgres component=peer fn=applyConfig
t=2016-12-19T07:45:02+0000 lvl=info msg="starting state evaluation" app=postgres component=peer fn=evalClusterState
panic: unexpected role deposed

I have opened #3792 to track fixing this.

Thank you @lmars 👍


Create a questionnaire new design

What is the context of this PR?
Trello card
Design refresh of the "Create a Questionnaire" user journey.
How to review

Check page matches designs

 Create New Q Design Update (5)

@bennpearson not anymore


Add in benchmarks

the original has comparison between itself vs. regex vs. linear search
maybe a bit excessive now days.
If we do this right, we can use an interface and reuse some code for this vs. regex

use new sub benchmarks to simplify code

fixed


Make k8s 1.14 an officially supported

[ ] remove experimental tag from k8s 1.14 by doing more extensive test + upgrade
[ ] run and publish k8s conformance for rancher/k8s 1.14 and rke standalone/k8s 1.14. Instructions can be found here.

Once this is ready #20725 can be changed to test so UI can check that kube-node-lease is now in System project.

Conformance: https://github.com/cncf/k8s-conformance/pull/624 && https://github.com/cncf/k8s-conformance/pull/625

All tests relating to 1.14 done on master builds are tracked in https://github.com/rancher/rancher/issues/19039


Update Docker build rules with new dockerhub org scope

after #15 update docker build rules to include publishing to new dockerhub org

Hey @dabgmx, I have created a dockerhub solace community org https://hub.docker.com/orgs/solacecommunity we can push the image on

Hey @dabgmx, I have created a dockerhub solace community org https://hub.docker.com/orgs/solacecommunity we can push the image on

Build rules updated https://hub.docker.com/repository/docker/solacecommunity/solace-prometheus-exporter
If you think this is good, you can go ahead and close this issue

Build rules updated https://hub.docker.com/repository/docker/solacecommunity/solace-prometheus-exporter
If you think this is good, you can go ahead and close this issue

The build jop only create the latest.
The old one also was Building all tags starting with "v"
https://github.com/solacecommunity/solace-prometheus-exporter/tags

The build jop only create the latest.
The old one also was Building all tags starting with "v"
https://github.com/solacecommunity/solace-prometheus-exporter/tags

@GreenRover I can invite you the solacecommunity dockerhub org so you can check out the build rules there, what is your docker handle?

@GreenRover I can invite you the solacecommunity dockerhub org so you can check out the build rules there, what is your docker handle?

https://hub.docker.com/u/greenrover

https://hub.docker.com/u/greenrover

Added

Added

It only builds new release.
Now, after create a new minor release, we have a stable release on dockerhub

It only builds new release.
Now, after create a new minor release, we have a stable release on dockerhub


Sync video duration to audio duration

This fixes #42
I implemented a failsafe to ensure there is always enough video for the audio. No more still images or black images.
Following was changed:
search.py, main.py:
There is now a minimum duration for clips. (Default 10 seconds). Only longer clips will be accepted and downloaded.
This will reduce potential videos further but no worries because there is a failsafe and I increased the search results from 10 to 15.
video.py (combine_videos): The Failsafe
During clip combining it now calculates the remaning video length required to so that video and audio are the same length.
It will continue to add clips from the same downloaded ones until the video length matches the audio length.
Of course it will also check if a clip is too long and only add as much as needed to fill the audio length. It is perfectly matched.
Notes:
One downside that remains with this is that for long videos clips may be repeated BUT the ultimate goal is to have < 60 second videos anyway and with a minimum length of 10 seconds per clip and 5 clips (or increase to 6), it will never repeat. Thus this change will become only a failsafe.
Tested & works for me.

Love goes out to you radry, always helping [❤](https://emojipedia.org/red-heart


feat(sanity): move the sanity adapter

moved my repository next-auth-sanity from here using the new Adapter interface

Could you also add sanity in the following places:
https://github.com/nextauthjs/adapters/blob/canary/README.md
These are required for automatic releases:
https://github.com/nextauthjs/adapters/blob/canary/.github/workflows/canary.yml#L28
https://github.com/nextauthjs/adapters/blob/canary/.github/workflows/main.yml#L25


add option to disable benchmarks

unwind_bench uses getcontext function however it is not available on
musl (POSIX.1-2004 obsoleted this function and in POSIX.1-2008 it was
removed) so add --without-benchmarks option.
Signed-off-by: Fabrice Fontaine<EMAIL_ADDRESS>

Indeed, this is not needed anymore so I'll close this PR.


Update MacPorts version to 4.0.0

The current version os MacPorts is still v3.3.10
$ sudo port info yabai
yabai @3.3.10 (sysutils)
Variants:             universal

Description:          yabai is a window management utility that is designed to work as an extension to the built-in window manager of
                      macOS.
Homepage:             https://github.com/koekeishiya/yabai

Platforms:            darwin
License:              MIT
Maintainers:          Email<EMAIL_ADDRESS>GitHub: thekevjames
                      Policy: openmaintainer

I was planning to open a pull request, but the version number v4.0.0 indicates there will likely be a breaking change, so I think the job is best left to someone with more expertise.

The package was updated to version 4.0.1. I have a pull request pending for version 4.0.2.


Disabling hover is not working

I'm using latest version of react-chart-2 (2.1.0) and chartjs (2.6.0) and even setting this:
hover: { mode: false },
The function onComplete of animation is always begin triggered, I thought it won't be a problem as they said is fixed since v2.5. Please take a look: https://github.com/chartjs/Chart.js/issues/3181

Same here :( Does not seem to work

Closing for now. Please refer to the Chart.js documentation for questions around configuration and/or usage of the library. If you have a bug or enhancement related directly to Chart.js, please open an issue on their Github.


chore: improve tnf generate

Problem
Without any parameter, it will give an interactive generator selection.
Solution
Like this:
$ tnf generate
# or
$ tnf g
? Pick generator type › - Use arrow-keys. Return to submit.
❯ Create Pages -- Create a tnf page by page name,includes {pageName}.tsx {pageName}.module.less
  Create Tailwindcss -- Enable the Tailwind CSS configuration and installDeps
  Create Component -- ...
  ......

Alternatives
No response
Importance
would make my life easier
Additional Information
No response

Can assigned this task to me.


Update diffx-scalatest-should to 0.8.0

Updates com.softwaremill.diffx:diffx-scalatest-should from 0.7.1 to 0.8.0.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.softwaremill.diffx", artifactId = "diffx-scalatest-should" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "@monthly" },
  dependency = { groupId = "com.softwaremill.diffx", artifactId = "diffx-scalatest-should" }
}]

labels: test-library-update, early-semver-major, semver-spec-minor, commit-count:1

Superseded by #238.


Outliner adds a little overhead on latency for big enough lists

Plain and simple: creating new line todo with Enter produces a small but notable lag with whole Obsidian frozen.

This damage the usability of Obsidian as it's usually very swift for pretty much everything.

@stasbel thank you for the feedback
I reproduced the problem. There is an inefficiency in the algorithm related to folded lists. The algorithm needs to be completely rewritten.

Should be fixed in https://github.com/vslinko/obsidian-outliner/releases/tag/4.6.0


FreeBSD compatibility

The attached patches include changes to the linux_tools/ directory sufficient to make bpf_asm compile (but produce runtime errors if Linux-specific BPF extensions are used).
There's also a script named bpftong to create FreeBSD netgraph ng_bpf format programs from bpf_asm output.

Hmm, I'm not convinced this is the right approach. The stuff in "linux_tools" is taken verbatim from the linux kernel, so I'd avoid changing it heavily. We don't want to maintain fork of this code. I don't mind tweaking the #includes, but I do mind big changes.
Are the changes in "linux_tools/bpf_exp.y" really neccesary?

Feel free not to include them, but they're there if you want them.
The changes in linux_tools/bpf_exp.y are necessary for building on FreeBSD because the BPF layer on FreeBSD doesn't support the xor or mod instructions, nor does it support the extended pseudo k-variables.
If I can get hold of the original author I may see whether he wants to maintain wider O/S compatibility.

I've been in touch with Daniel, and he thinks it's unlikely that the portability patches would be incorporated into the master Linux tree.  However there's a possibility (via other contacts) that it might be suitable for pulling into tcpdump, so I'll pursue that avenue.


Netlify's function is reading project folder without permission

In my app I have a search feature which goes to the content folder, grabs markdown posts, and displays them

The repository
The Netlify Search Function
How does it able to do so, outside build folder on staging?
I was getting errors here, and there, then I suddenly realized to access this folder on staging, I have to create the following Netlify file, so on staging Netlify has access to the content folder, but currently if you check my repository it does not have this file, and works magically without it, can somebody explain to me what is going on?

Hi @1alexvash, this repo is intended for Decap CMS related issues.
For Netlify build related issues I suggest to reach out in the forums: https://answers.netlify.com/
Closing the issue, and please comment if you have any other questions.


feat: add group descriptions to models

add group descriptions to models

This should lead to a new test right ?

I have updated the test


Accessibility vids

I am biased but the videos from Accessibility Scotland are worth a watch (and possible inclusion!)

Thanks @iadawn - just added these in version 2.1.0. Biased or not, it's a good addition to the manual!


4k resolution infinite scroll broken

When the gui is maximized on my 4k monitor, the number of channels that show when clicking on Top Channels is the top 100 streams. But there is no button that allows me to fetch more. But there is no button which would then allow me to fetch more. This issue is consistent with every page that shows  just channels. On the Top Games page the infinite scroll works just fine. But when I decrease the size and reload the page by either refreshing or loading a different page the infinite scroll works. Maximizing
the gui on a 1080p monitor works.

Which OS do you use? Does your OS properly scale the application window?
If it does not, then it's a NW.js issue, since the HiDPI resolution was not detected. There are some HiDPI fixes in the latest NW.js release, so maybe this could fix the issue. I'm waiting for other issues to resolve first, before I'm able to upgrade, though...
The infinite scroll mechanism does not work with that many icons, because the fetch size is limited by the Twitch.tv API. Also, CSS media queries which are used for calculating the fetch size are not defined for unscaled resolutions like this. Those could be added of course.

I use windows 7. I do not know what you mean by properly scaling the window, but it doesn't look weird.

Windows 7 does not properly support DPI scaling.
https://blogs.windows.com/windowsexperience/2013/07/15/windows-8-1-dpi-scaling-enhancements/


fix(jsdom-env): remove Buffer

Summary
Following up on #11222. For some reason a JSDOM test takes about 12 seconds rather than less than 1 with this change. (slow enough that a test fails since it crosses the test length threshold, causing a snapshot diff)

Test plan
Green CI, eventually

This test should be deleted too I think?
https://github.com/facebook/jest/blob/167aad4565f6960968a8666d78234ed198fa0770/e2e/env-test/__tests__/equivalent.test.js#L15-L20

Codecov Report

Merging #11241 (167aad4) into master (167aad4) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head 167aad4 differs from pull request most recent head 412360d. Consider uploading reports for the commit 412360d to get more accurate results

@@           Coverage Diff           @@
##           master   #11241   +/-   ##
=======================================
  Coverage   64.24%   64.24%
=======================================
  Files         308      308
  Lines       13502    13502
  Branches     3289     3289
=======================================
  Hits         8675     8675
  Misses       4117     4117
  Partials      710      710

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 167aad4...412360d. Read the comment docs.

Do we have a workaround for the issue with Reference error besides using resolutions? Now if I do yarn add -D<EMAIL_ADDRESS>I always get next.6

What issue?

referenceerror: buffer is not defined, only occur after I upgraded to next-6

This PR doesn't remove Buffer, so not related.
I'm guessing you env is old or custom (and using installCommonGlobals which does not add Buffer automatically anymore). Regardless, not related to this PR

Aw thanks, I found the issue. My angular tests depend on ts-jest which still uses jest-util next.5


Use correct Req error handling instead of raising exceptions

Pull Request Test Coverage Report for Build 527655ca253a82a69e08f84736b16cecf08343fb-PR-30
Details

2 of 17 (11.76%)  changed or added relevant lines in 4 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-2.6%) to 79.791%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

lib/kubereq/step/auth.ex
2
3
66.67%

lib/kubereq/step/tls.ex
0
1
0.0%

lib/kubereq/error/kubeconf_error.ex
0
3
0.0%

lib/kubereq/exec.ex
0
10
0.0%

Totals

Change from base Build 6440f3a148b8abdc48fb55c2d5564279912df8d4:
-2.6%

Covered Lines:
229

Relevant Lines:
287

💛  - Coveralls


Proof of Concept: introducing Static Code Analysis, PHPStan and refactor for level 0

Pull request type

Jira Link: no internal

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no api changes)
[X] Other (please describe): Proof of concept, for introducing PHPStan and all refactor needed for reaching Level 0 (the minimum)

How to test this PR

composer install (for installing phpstan)
vendor/bin/phpstan analyse src --level=0

What is the new behavior?
Due to the changes needed for reaching level 0 of a static code analysis, i'm creating a PR, as a Proof of Concept , just to demonstrate that introducing tools like phpstan is feasible, but some refactor in the code is needed.
It is ok if you don't want to merge it, because probably you want to evaluate deeper and better the refactor needed.
The refactor was:

moving functions that call getStories() from BaseClient to Client;
moving "enrich" related functions from BaseClient to Client
moving "relations" related functions from BaseClient to Client
moving "resolvingLinks" related functions from BaseClient to Client
declare attributes for classes
cleaning responseHandler() for avoid duplicate code in the classes

I have some proposal (not yet included in this PR): creating Traits to group methods and attributes related to a specific functionality, for example:

Traits  Relatable with: $_relationsList, getResolvedRelations() , insertRelations(),
Traits Enrichable with: enrichContent(), enrichStories()
Traits Linkable with: getResolvedLinks(), insertLinks()

Other information

Nice!

btw i dont think moving those methods from the BaseClient to the Client works, since the Management Client relies on those methods as well?

We will introduce PHP STAN as part of the refactoring process. Closing this PR for now


Switch engine.merge_method back to squash

Unfortunately, device lab does not have the capacity for merge method rebase. It's essentially DoSing device lab: The last engine rebase has been processing for over 6h now and it is still not done :(

I'm Ok to merge this PR on red tree so it will change the merge method before the auto-roller gets a green flutter-build.

Merging on red to make sure the autoroller doesn't see a green build before this goes in.


Add tokens to mm

Closes #359
(apologies branch is out of date as I have been doing this in my own time for funsies and haven't updated)
I think before merging we need some better icon/logo-ography. I have used the Vega logo for all of these tokens however that would be confusing in a wallet, I think we need a separate logos for:

Vega (we have this)
Locked vega
SLP ETH/VEGA
SLP USDC/VEGA

These should all be 32px and we should bare in mind when in wallet and in MM has rounded icons as well.
@campbellssource what's the process for this? Is this something we can do or is this a big fan request.

Closing as too out of date. Functionality ported to #795


Correct definition for the Triangle inequality

"or equal to" was missing.
In mathematics, the triangle inequality states that for any triangle,
the sum of the lengths of any two sides must be greater than or equal to
the length of the remaining side.
Source: https://en.wikipedia.org/wiki/Triangle_inequality

We need to be careful with this. There has been some discussion around this
some time ago. One of the problems is that some tracks do it as described
in the read me and others as it is defined in mathematics. Can't search or
link right now, because I'm on mobile right now.
Am 19.07.2016 20:45 schrieb "Matt Fischer"<EMAIL_ADDRESS>
"or equal to" was missing.
In mathematics, the triangle inequality states that for any triangle,
the sum of the lengths of any two sides must be greater than or equal to
the length of the remaining side.
Source: https://en.wikipedia.org/wiki/Triangle_inequality
You can view, comment on, or merge this pull request online at:
https://github.com/exercism/x-common/pull/298
Commit Summary

Correct definition for the Triangle inequality

File Changes

M triangle.md
https://github.com/exercism/x-common/pull/298/files#diff-0 (4)

Patch Links:

https://github.com/exercism/x-common/pull/298.patch
https://github.com/exercism/x-common/pull/298.diff

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/exercism/x-common/pull/298, or mute the thread
https://github.com/notifications/unsubscribe-auth/AADmR20h9JAPPPySDQ2QfNYkJKu6O2WYks5qXRtkgaJpZM4JQCs1
.

Maybe we could just explain that in the README and post a link to the mathematical definition?

The previous discussion was at https://github.com/exercism/x-common/issues/202 and mainly centered around whether tracks should have tests for cases when a + b = c. Regardless of what the answer to that question is, the README should at least give the correct definition.
So I say 👍  to this.


Add release to GH repository releases

Add the release in the GitHub repository Releases.
It will give users more clear releases notes as the GHCR does not have a detailed release notes.

Dockerfile build is missing upgrade pip and can raise error

Signed-off-by: Kairo de Araujo<EMAIL_ADDRESS>

@MVrachev @rdimitrov


Fix DfaMatcher to work with non-RouteEndpoint

Fixes https://github.com/aspnet/Routing/issues/648

Thanks for doing this 👍 I was planning to last night, but then I got into the weeds with the comparer thing. I slept on it and decided on a solution, then I came to work and found that someone had already coded it up for me. Unless this is a smoke-fueled fever dream...

🆙 📅


Community string "public" still works though another was set

Hello,
When I ran the simulator with a data dir containing   .snmprec file, named 1500 for instance, both "1500" and "public" work in snmpget as community strings.
But "public" should not be a valid community string once I set "1500" , right?

Right, 'public' not not in any way special so it should not be available it the .snmprec file is not there.
Can you share snmpsimd log showing daemon startup and request handling?

I found the problem, it found default .snmprec files installed with snmpsimd. Thanks.


black screen after falling and surviving

I fall then just darkness

known https://github.com/JonnyBro/beatrun/issues/65

oh srry


WIP: Remove proxies from hasOne

Just some thoughts, there are failing tests at the moment, but we should probably get rid of these proxies yeah?

hasOne proxies have been removed via the rethink merger (#98)


AppVeyor, NuGet세팅

http://druss.co/2016/01/build-and-test-asp-net-5-application-using-appveyor/ 참고

http://andrewlock.net/publishing-your-first-nuget-package-with-appveyor-and-myget/ 참고


Session detail screen: Speaker name not center-aligned next to icon

Overview
On the session detail screen, the speaker's name to the right of their icon is not center-aligned vertically. This misalignment should be corrected to match the design specifications.

Links

Design: https://www.figma.com/design/XUk8WMbKCeIdWD5cz9P9JC/DroidKaigi-2024-App-UI
Code: https://github.com/DroidKaigi/conference-app-2024/blob/ec7c35cc7a425acaf2d86bdd9d48780268c9f5ea/feature/sessions/src/commonMain/kotlin/io/github/droidkaigi/confsched/sessions/TimetableItemDetailScreen.kt#L81

✋

Thanks! Assigned 👍


how to use chinese word vector generate by bert to calculate sentence similarity

first, thanks your share.
I try to use word vectors to calculate sentence similarity by cos in many time, but it didn't work.
my steps is follow:
1.get two sentence word representations
2.get [CLS] word representation in each word representations
3.use two [CLS] word representation to calculate similarity by cos
if you have some good ideas, please tell me, hope you can reply

you can use the mean of last layer output instead without [CLS} and [SEG]

@wykdg I try, but it didn't work

Calculating the similarity of two sentences by using the [CLS]s' representations is not a good idea. Maybe you should transform the [CLS]s' representations which should be fine-tuned.

mark

mark

Hi, all! The correct explanation is here:
https://github.com/hanxiao/bert-as-service#q-the-cosine-similarity-of-two-sentence-vectors-is-unreasonably-high-eg-always--08-whats-wrong


Minor Issue with "Splits" editing

For splits specifically might be better to remove +/- and just have 5 labeled elements.

• There is a limit of 1 + 2 + 2 panels but you can keep adding useless fields to the slice.
-- Disable "+" when limit is reached (disable minus if minimum is reached)
-- Would be useful to put min/max on slice increment generally as I bet there are other cases

all done


[Suggestion] Movement (Tutorial Part 2 and 3)

Hi, thanks for this amazing tutorial series on Action RPG games, I started watching them to learn Godot, and recently watched Part 2 and 3, related to player movement.
Regarding movement, you mentioned some improvements that viewers broght to you, and I want want to add something:
You don't have to multiply delta as you calculate acceleration and friction to update your velocity, since it's used in both cases, it just makes more sense to multiply the final velocity.
if input_vector != Vector2.ZERO:
    velocity = velocity.move_toward(input_vector * MAX_SPEED, ACCELERATION)
else:
    velocity = velocity.move_toward(Vector2.ZERO, FRICTION)

move_and_collide(velocity * delta)

As you mention in the video, delta is a value that helps us control how the player moves across different setups (with some computers being faster than others), its value is not directly related to the speed calculation itself, so we don't have to worry about while calculating the velocity.
With this change, we can still use the values for MAX_SPEED, ACCELERATION, and FRICTION as before (100, 25, 25), which again, makes more sense.

use a pr

as per the godot docs... https://docs.godotengine.org/en/stable/classes/class_kinematicbody2d.html#class-kinematicbody2d-method-move-and-slide
"This method should be used in Node._physics_process (or in a method called by Node._physics_process), as it uses the physics step's delta value automatically in calculations. Otherwise, the simulation will run at an incorrect speed."


Nginx conf line prevents me from seeing files for my app

The nginx conf line here prevents me from seeing my files in an app container: https://github.com/cloudfoundry/cloud_controller_ng/blob/master/bosh-templates/nginx.conf.erb#L94
I have an app that I push onto the platform that contains source in a directory called "application".  The greedy regexp here seems to cause the issue as the path for that request is the following:
/v2/apps//instances/0/files/app/application
This means that anyone with a directory called application will not be able to see the files in that directory. Looking at the regex, this also probably applies to a directory called "bits".

Good morning @ryanmoran,
We have a bug in tracker that would fix this issue. We'll talk to @dieucao about possibly prioritizing this higher.
Thanks!
@wendorf && @utako, CF Runtime Team


[RFC]: Add @stdlib/string/for-each-right

Description
This RFC proposes adding support for invoking a callback for each (visual) character of a string, while iterating from right-to-left.
Signature:
forEachRight( str, clbk[, thisArg] )

where clbk (similar to @stdlib/utils/for-each-right) is provided

value: visual character
idx: starting character index
str: input string

Package: @stdlib/string/for-each-right
Alias: forEachRight
Related Issues
None.
Questions
No.
Other
Will require iterating over grapheme cluster breaks. See @stdlib/string/reverse for such an example.
Checklist

[X] I have read and understood the Code of Conduct.
[X] Searched for existing issues and pull requests.
[X] The issue name begins with RFC:.

Similar to @stdlib/string/for-each, this package should be decomposed into the following base packages:

[ ] @stdlib/string/base/for-each-right

similar to https://github.com/stdlib-js/stdlib/tree/develop/lib/node_modules/%40stdlib/string/base/for-each

[ ] @stdlib/string/base/for-each-code-point-right

similar to https://github.com/stdlib-js/stdlib/tree/develop/lib/node_modules/%40stdlib/string/base/for-each-code-point

[ ] @stdlib/string/base/for-each-grapheme-cluster-right

similar to https://github.com/stdlib-js/stdlib/tree/develop/lib/node_modules/%40stdlib/string/base/for-each-grapheme-cluster

Hi @kgryte,
Hope all is well! I'd be interested in working on this RFC, if it's still available. Could you please assign me to it?
Thanks,

@AhmedKhaled590 Sure thing. Please start by working on @stdlib/string/base/for-each-right. If that goes well, we can move on to the others.

Hi @kgryte
should I proceed with moving on to the others or should I wait?

@AhmedKhaled590 Looks like that one is near the finish line, so feel free to find another issue.

@kgryte Okaaay 👍🫡

@Planeshifter This issue should not have been closed.

The only package which was implemented was @stdlib/string/base/for-each-right. The other packages still need implementing.

@kgryte @Planeshifter I will complete working on the two remaining packages


Base64 vs Base64url

We currently take the easy way and just require every ArrayBuffer/byte[] sent from the browser to be Base64url encoded. this might be incorrect per the WebAuthn spec. We need to check.

I just checked on this. Based on https://www.w3.org/TR/webauthn/#dependencies, I think we are good assuming base64url. Nowhere could I find 'base64' being referenced without 'url' at the end.

See also: https://www.fidoalliance.org/specs/fido-v2.0-rd-20180702/fido-server-v2.0-rd-20180702.html
https://www.fidoalliance.org/specs/fido-v2.0-rd-20180702/fido-metadata-statement-v2.0-rd-20180702.html
https://www.fidoalliance.org/specs/fido-v2.0-rd-20180702/fido-metadata-service-v2.0-rd-20180702.html

True, but there are several values that are not specified to be bsae64url but just "ArrayBuffer" etc. For example RawId vs Id.

I am pretty sure that the FIDO Conformance Tools test this (they sneaky send a base64 encoded something .. with trailing '=') and expect it to fail .. just so u know ;)

@xepa What do you mean when you say "test this"?
If I understand what you are all saying we should be good on this issue and should close it until someone runs into an error?
Maybe before closing we could add a exception in the base64url serialization error that describes that we expect base64url, not base64? We should also make that clear in the docs.

I am running some FIDO Conformance Tools .. for instance on the challenge they would send base64 encoded data (instead of base64url encoded) and expect the test to fail.  So it is a good choice to make sure that it actually is base64url encoding, as far as I am aware there are no base64 encodings being used only base64url encodings.

I see @xepa , thank you


Monitor multiple boxes threads issue

Hi, @danzuep!
At first, thanks a lot for the monitoring multiple boxes feature!
At second, there is some issue with it.
I have registered 7 FolderMonitors in appsettings and using IMailFolderMonitorFactory.MonitorAllMailboxesAsync method to monitor them. So, for example (and not only this example), when i get 2 mail in 1 mailbox same time, it's thows exceptions:
MailKitSimplified.Receiver.Services.MailFolderMonitor|IMAP client is being accessed by multiple threads. System.InvalidOperationException: The ImapClient is currently busy processing a command in another thread. Lock the SyncRoot property to properly synchronize your threads.
   at MailKit.Net.Imap.ImapEngine.PopNextCommand()
   at MailKit.Net.Imap.ImapEngine.IterateAsync()
   at MailKit.Net.Imap.ImapEngine.RunAsync(ImapCommand ic)
   at MailKit.Net.Imap.ImapClient.NoOpAsync(CancellationToken cancellationToken)
   at MailKitSimplified.Receiver.Services.MailFolderMonitor.ProcessMessagesArrivedAsync(Boolean firstConnection, CancellationToken cancellationToken)
   at MailKitSimplified.Receiver.Services.MailFolderMonitor.WaitForNewMessagesAsync(CancellationToken cancellationToken)

MailKitSimplified.Receiver.Services.MailFolderMonitor|Error occurred processing arrival queue item #MailKit.MessageSummary. System.InvalidOperationException: The ImapClient is currently busy processing a command in another thread. Lock the SyncRoot property to properly synchronize your threads.
   at MailKit.Net.Imap.ImapEngine.PopNextCommand()
   at MailKit.Net.Imap.ImapEngine.IterateAsync()
   at MailKit.Net.Imap.ImapEngine.RunAsync(ImapCommand ic)
   at MailKit.Net.Imap.ImapFolder.GetBodyPartAsync(UniqueId uid, String partSpecifier, CancellationToken cancellationToken, ITransferProgress progress)
   at MailKitSimplified.Receiver.Extensions.MessageSummaryExtensions.GetBodyTextAsync(IMessageSummary messageSummary, CancellationToken cancellationToken)
   at MailKitSimplified.Receiver.Services.MailFolderMonitor.ProcessArrivalQueueAsync(Func`2 messageArrivalMethod, CancellationToken cancellationToken)

At third, there is example with worker which is Singleton and have injected from constructor dependency on IImapReceiver, which is Transient. So, it won't be transient, WorkerService will control it's lifetime, and it would be Singleton. There is recomendation from Microcoft to do it rigth way.
Exceptions throws every day several times😥
Please, help, Daniel🙏

Hi @Sylar-A, glad you found the FolderMonitor feature as I'm not sure I'd documented it fully 😊. I ran it in my test environment but the project I initially wanted to use it for has been and gone, so I haven't actually used it in a real-world setting yet. It sounds like you've figured out the fix for this issue though, so well done! Please make a fork of the project, test your proposed changes on your machine, then make a pull request to merge your changes into the main branch. Then your name will appear on the main page as a contributor 🙂

Hi, @danzuep!
I have found a fix for another issue in sample code, but it's not fix my issue with multiple threads😞

As a short-term workaround you could try using the ImapReceiverFactory instead. If that works then using the copy function could be the solution.

Hi, @danzuep!
If i understand you correct, it must be a BackgroundService with while loop and with some timeout?
For example:
    private async Task DoWork(CancellationToken cancellationToken)
    {
        while (!cancellationToken.IsCancellationRequested)
        {
            await Task.Delay(TimeSpan.FromMinutes(1), cancellationToken);

            using var scope = _serviceProvider.CreateScope();

            var imapReceiverFactory = scope.ServiceProvider
                .GetRequiredService<IImapReceiverFactory>();

            foreach (var receiver in imapReceiverFactory.GetAllImapReceivers())
            {
                var messageSummaries = await receiver.ReadMail.GetMessageSummariesAsync(cancellationToken);

                foreach (var messageSummary in messageSummaries)
                {
                    // Do something...
                }
            }
        }
    }

But i didn't realize how to use the copy method as you suggested.

Oh, ok, Daniel, thanks for parting words, i'll try to hard debug it😅

Looking back at the error message, it says the "IMAP client is being accessed by multiple threads." The IMAP client isn't being passed through when the MailFolderMonitor is constructed though. Then I saw the second part that says "GetBodyTextAsync(IMessageSummary)". The short term fix then is not to use messageSummary.Folder. Hopefully that helps to with a more targeted search 🙂

Hi, @danzuep!
I have found another issue with IImapReceiverFactory. When it's creates new receivers and all of them have same ImapHost, it's takes only one mail (in my case one same mail seven times instead of 7 different mails). When I debugging it, the problem was in IMemoryCache extension method GetOrCreate (from Microsoft). Because of all mails have same ImapHost, it wasn't creates new item and tooks the same item 7 times by the key. It was here.

Wow, good find! _memoryCache can be removed, it doesn't add much in terms of functionality or speed. Are you happy to add it to a pull request? If so, please submit each fix as a separate pull request if possible, but all good if not.

Ok, Daniel. I could do it later.

Hey @Sylar-A, is the issue you found fixed?
You also mentioned changing to scoped services, but I don't see those in the fork on your profile. I've done that for the Sender, but I guess I never got around to it for the Receiver so I should get on to it.

The answer to this issue you've found is to not use use the messageSummary that comes from the MailFolderMonitor or the messageSummary.Folder. I've added a MailFolderCache function you can use from the ImapReceiver, so use the UniqueId and the GetMailFolderAsync() function instead of using the MessageSummary directly. One possible solution would be for me to just return the UniqueId instead of returning the MessageSummary.

Here's a sample:
        using var scope = _serviceScopeFactory.CreateScope();
        var mailFolderClient = scope.ServiceProvider.GetRequiredService<IMailFolderClient>();
        var mailFolderMonitorFactory = scope.ServiceProvider.GetRequiredService<IMailFolderMonitorFactory>();
        async Task UniqueIdArrivedAsync(IMessageSummary messageSummary) =>
            await mailFolderClient.MoveToAsync(messageSummary, destinationFolderFullName, cancellationToken);
        await mailFolderMonitorFactory.MonitorAllMailboxesAsync(UniqueIdArrivedAsync, cancellationToken);

Please test the changes and let me know how it goes.

Hi, @danzuep!
I cant find the latest pre-release as you mentioned in nuget. There is only previous version.
I've opened it with dotPeek and there aren't your last changes.
Maybe it's GitHub issue, because there is only one rc tag on 2.10.0 too.

Ah. it seems that [GitVersion_NuGetVersion] (https://github.com/danzuep/MailKitSimplified/actions/runs/9457376744/job/26051068662) didn't update, even though the semantic version is correct. Weird, that used to work. I re-ran it with a higher version and it published this time.

Hi, @danzuep!
I've found some issues with new code🥲

When i try to call await mailFolderClient.AddFlagsAsync([messageSummary.UniqueId], MessageFlags.Seen);
it throws an exception:

MailKit.Security.AuthenticationException: invalid command
   at MailKit.Net.Imap.ImapClient.AuthenticateAsync(Encoding encoding, ICredentials credentials, CancellationToken cancellationToken)
   at MailKitSimplified.Receiver.Services.ImapReceiver.AuthenticateImapClientAsync(CancellationToken cancellationToken)
   at MailKitSimplified.Receiver.Services.ImapReceiver.ConnectAuthenticatedImapClientAsync(CancellationToken cancellationToken)
   at MailKitSimplified.Receiver.Services.ImapReceiver.ConnectMailFolderAsync(CancellationToken cancellationToken)
   at MailKitSimplified.Receiver.Services.MailFolderClient.ConnectMailFolderAsync(IMailFolder mailFolder, Boolean enableWrite, CancellationToken cancellationToken)
   at MailKitSimplified.Receiver.Services.MailFolderClient.ConnectAsync(Boolean enableWrite, CancellationToken cancellationToken)
   at MailKitSimplified.Receiver.Services.MailFolderClient.AddFlagsAsync(IEnumerable`1 uniqueIds, MessageFlags messageFlags, Boolean silent, CancellationToken cancellationToken)

Somewhere here (deeper in MailKit):

When i try to call await mailFolderClient.MoveToAsync(messageSummary, _archiveFolderName);
it throws an NullReferenceException with _mailFolderCache because of mailFolder is null:

System.NullReferenceException: Object reference not set to an instance of an object.
   at MailKitSimplified.Receiver.Services.MailFolderCache.CacheMailFolder(IMailFolder mailFolder)
   at MailKitSimplified.Receiver.Services.MailFolderCache.GetMailFolderAsync(IImapReceiver imapReceiver, String mailFolderFullName, CancellationToken cancellationToken)

Some of the issues you mentioned are fixed in the latest pre-release, try updating. The main thing I haven't tried is creating folders, but the other ones should work now.

Hi, Daniel!
Thank you for fixes!
But I still can't see new rc version in nuget packages🥲
Maybe you'll release a new version after all fixes?

Looks like it was there when you commented, but just in case I've made a new one. Please do a Pull Request if you notice any issues.

Hi, @danzuep!
Second point of comment was fixed, thanks!
But first is still throws and now on both mailFolderClient.AddFlagsAsync() and mailFolderClient.MoveToAsync() methods(
Maybe the problem is in empty ImapCredentials here? It's filled in appsettings.json file.

It seems it's a problem with your authentication settings, but the issue isn't clear from the error message (MailKit.Security.AuthenticationException: invalid command). What's the ProtocolLog saying?

It says
Connected to imaps://imap.mail.ru:993/
S: * OK IMAP4 ready
C: V00000000 CAPABILITY
S: * CAPABILITY IMAP4rev1 ID XLIST UIDPLUS UNSELECT MOVE LIST-STATUS SASL-IR AUTH=PLAIN AUTH=XOAUTH2
S: V00000000 OK CAPABILITY completed
C: V00000001 AUTHENTICATE PLAIN ********
S: V00000001 NO [AUTHENTICATIONFAILED] NEOBHODIM parol prilozheniya https://help.mail.ru/mail/security/protection/external / Application password is REQUIRED
C: V00000002 LOGIN "" ""
S: V00000002 BAD invalid command

"Application password is REQUIRED" (Gmail has the same requirement).
The capability line includes AUTH=XOAUTH2, if you you just want to get it working, you can use imapReceiver.RemoveAuthenticationMechanism("XOAUTH2");.

Authenticating via a SASL mechanism may be a multi-step process.
see href="http://www.mimekit.net/docs/html/T_MailKit_Security_SaslMechanism.htm"
seealso href="http://www.mimekit.net/docs/html/T_MailKit_Security_SaslMechanismOAuth2.htm"

Hi, @danzuep!
The error is not in XOAUTH2  mechanism, it's in PLAIN.
In this image you can see that credentials are not empty

In this image it's empty (when i use mailFolderClient mrthods AddFlagsAsync and MoveToAsync)

Hi @danzuep!
Any decision with this bug?
I suppose the problem is in resolving configs in new instances of services (after register them as scope/transient).
I can't use your new code because of it.
Please, help🙏

I haven't been able to replicate the issue you're having sorry. Have you tried connecting to GMail or one of the other big providers? Try downloading the source code and using breakpoints to pinpoint the issue.


Add dynamic bitset

This PR:

Adds a simple dynamic bitset implementation;
Uses that implementation in place of std::unordered_set in SREX.

Benchmarks
On VRPTW. I expect very little difference w.r.t. current main; the use of a hash set in SREX was not much of a bottleneck.
This PR (e584b73483b7ce696603169a0dc3c3b9b6d92a86; job ID 2883239; 10x seeds/1h runtime with configs/vrptw.toml):
RUNNING

Current main (10x seeds/1h runtime with configs/vrptw.toml):
 Avg. objective: 333197.6 (min: 333086, max: 333324)
Avg. iterations: 123194.3 (min: 107088, max: 130939)

Bitsets are quite fast alternatives to std::unordered_set for sets of integers. I want to use this bitset for a concatenation scheme for pickup and delivery. This is a start to that process.

@leonlan @wouterkool can either of you review this? Basically whoever comes first :-)


Button alignment issue in the Additives cards

Button alignment issue in the Additives cards
"Browse products" and "View on Wikipedia" are not aligned.

I think this is fixed. @teolemon
Can you re-check and give product barcode if you are able to reproduce.

I think this is fixed. @teolemon
Can you re-check and give product barcode if you are able to reproduce.

@teolemon  When the language is set as English the buttons are aligned correctly. I tried changing my device default language to french but the text still appears in english. Some part appears in french. Have a look at the image. (Changed the language to french even from Preferences).

@teolemon  When the language is set as English the buttons are aligned correctly. I tried changing my device default language to french but the text still appears in english. Some part appears in french. Have a look at the image. (Changed the language to french even from Preferences).

I am able to produce this issue after changing the length of Browse Products to something longer.

I am able to produce this issue after changing the length of Browse Products to something longer.


Don't "spam" logs about jwk_url config not found

It turns out the lru_cache (timed or otherwise) doesn't cache the result in case of failure. So we had this appear in the logs once for every worker every 30s.

@ashb  are you ok to merge this? looks good to me? Should I click merge?


Tests fail: no member named 'Cuda' in namespace 'Kokkos'

When kokkos is built w/out Cuda tests that require Cuda should be ignored, instead of failing.

This has been fixed in https://github.com/simonpintarelli/nlcglib/releases/tag/v0.9.1


remove Fixnum and Bignum deprecated warning

Fixnum and Bignum is deprecated at Ruby2.4.
https://github.com/ruby/ruby/blob/v2_4_0/NEWS#L288

What about shortening the change and adding a comment?
basic_objects = [String, Integer, Float, NilClass, Symbol, TrueClass, FalseClass]
basic_objects += [Fixnum, Bignum] unless unified_intger_version? # Ruby <= 2.3.x compatibility

@janpieper Thanks. It looks good to me. I'll fix it.

@takkanm thanks!

:smile:


Invalid UTF-8 Encoding Issue for LineEndings Check

On files with invalid UTF-8 Encoding (I'm assuming such as binary files although I can't tell which file it is from the error output), the LineEndings check blows up:
~/.rbenv/versions/2.4.1/lib/ruby/gems/2.4.0/gems/overcommit-0.39.1/lib/overcommit/hook/pre_commit/line_endings.rb:13:in `gsub!'
~/.rbenv/versions/2.4.1/lib/ruby/gems/2.4.0/gems/overcommit-0.39.1/lib/overcommit/hook/pre_commit/line_endings.rb:13:in `block (2 levels) in run'
~/.rbenv/versions/2.4.1/lib/ruby/gems/2.4.0/gems/overcommit-0.39.1/lib/overcommit/hook/pre_commit/line_endings.rb:11:in `each_line'
~/.rbenv/versions/2.4.1/lib/ruby/gems/2.4.0/gems/overcommit-0.39.1/lib/overcommit/hook/pre_commit/line_endings.rb:11:in `block in run'
~/.rbenv/versions/2.4.1/lib/ruby/gems/2.4.0/gems/overcommit-0.39.1/lib/overcommit/hook/pre_commit/line_endings.rb:9:in `map'
~/.rbenv/versions/2.4.1/lib/ruby/gems/2.4.0/gems/overcommit-0.39.1/lib/overcommit/hook/pre_commit/line_endings.rb:9:in `run'
~/.rbenv/versions/2.4.1/lib/ruby/gems/2.4.0/gems/overcommit-0.39.1/lib/overcommit/hook/base.rb:45:in `block in run_and_transform'
~/.rbenv/versions/2.4.1/lib/ruby/gems/2.4.0/gems/overcommit-0.39.1/lib/overcommit/utils.rb:260:in `with_environment'
~/.rbenv/versions/2.4.1/lib/ruby/gems/2.4.0/gems/overcommit-0.39.1/lib/overcommit/hook/base.rb:45:in `run_and_transform'
~/.rbenv/versions/2.4.1/lib/ruby/gems/2.4.0/gems/overcommit-0.39.1/lib/overcommit/hook_runner.rb:155:in `run_hook'
~/.rbenv/versions/2.4.1/lib/ruby/gems/2.4.0/gems/overcommit-0.39.1/lib/overcommit/hook_runner.rb:88:in `block in consume'
~/.rbenv/versions/2.4.1/lib/ruby/gems/2.4.0/gems/overcommit-0.39.1/lib/overcommit/hook_runner.rb:85:in `loop'
~/.rbenv/versions/2.4.1/lib/ruby/gems/2.4.0/gems/overcommit-0.39.1/lib/overcommit/hook_runner.rb:85:in `consume'

Hey @jfelchner, thanks for the report.
Can you dig up what the actual exception thrown was? We could work around this by gracefully catching such exceptions and ignoring those files if they are indeed binary files.
I'm not aware of a good cross-platform way to test whether a file is "binary", so my preference would be to just degrade gracefully in a case where we hit an encoding problem.

UGH.  Why didn't I add that for you all? 😧   I believe it was ArgumentError: invalid byte sequence in UTF-8 but I honestly can't even remember the project this happened in.  I'm sure when whichever client needs more work done I'll find it again.  But I'm pretty sure that was it.

@sds this should help.  Committing this file throws the error.


Label Identification Pattern

As a User, I want to have the labels / tags configured with a predefined set of colors and fields.
For example, refer to https://robinpowered.com/blog/best-practice-system-for-organizing-and-tagging-github-issues/

@mindreeper2420 we are not yet working on this... is this just an idea you are capturing?

@Mgranfie just capturing a request that was initial placed in the UI repo. Something to possibly look into in the future, but a backlog item for now.


MATH-1563: Minor Bug fixes and restructuring of 'examples-ga' module.

Coverage decreased (-0.003%) to 90.325% when pulling d4cd945fa71e3bf50137e1055602ad0221651d6d on avijit-basak:feature/MATH-1563-FIX into 07deb60ecf11782d18c06deb098339109fe6bd5e on apache:feature__MATH-1563__genetic_algorithm.

Created a new PR#205


[Request] Don't download webpage if video identifier matches in archive file

Make sure you are using the latest version: run youtube-dl --version and ensure your version is 2018.09.08. If it's not, read this FAQ entry and update. Issues with outdated version will be rejected.

[x] I've verified and I assure that I'm running youtube-dl 2018.09.08

Before submitting an issue make sure you have:

[x] At least skimmed through the README, most notably the FAQ and BUGS sections
[x] Searched the bugtracker for similar issues including closed ones
[x] Checked that provided video/audio/playlist URLs (if any) are alive and playable in a browser

What is the purpose of your issue?

[ ] Bug report (encountered problems with youtube-dl)
[ ] Site support request (request for adding support for a new site)
[x] Feature request (request for a new functionality)
[ ] Question
[ ] Other

Description of your issue, suggested solution and other information
When using the --download-archive feature youtube-dl still downloads each video's webpage. After downloading, the video is skipped based off the identifier that youtube-dl already has from the initial link.
Can this process be changed so that youtube-dl will discard the video as soon as the identifier is matched from the URL? Doing it this way will allow downloads to complete much faster since youtube-dl doesn't have to download potentially hundreds of webpages for a large playlist. The bigger return on this is that it also avoids slamming a video site with hundreds of page requests that are eventually discarded.
Current process:

youtube-dl has video with identifier ABCD
youtube-dl downloads webpage for video ABCD
youtube-dl aborts download of video ABCD since it matches in archive file

Requested process:

youtube-dl has video with identifier ABCD
youtube-dl aborts download of video ABCD since it matches in archive file

This depends on a particular extractor. It does discard as soon as possible if video id is known beforehand and extractor supports it.

Can this be reopened and then specifically be a request for the pornhub extractor to suppor this then? The video URL provides the identifier but the pornhub extractor is still downloading the webpage.

Should I open a new issue instead of requesting on this one?

Open a new issue with complete explanation regarding concrete extractor.


New package: StringMethod v0.2.6

Registering package: StringMethod
Repository: https://github.com/gideonsimpson/StringMethod.jl
Created by: @gideonsimpson
Version: v0.2.6
Commit: 402ebab4de8ae0de7a78b58c2e226fdc82107f1a
Git reference: HEAD
Description: Julia implementation of the String Method for computing transition paths and saddle points on energy landscapes
Release notes:

String method implemented in Julia

Closing in favor of https://github.com/JuliaRegistries/General/pull/102308


Update header.php

User description

Closes #
📑 Description

✅ Checks

[ ] My pull request adheres to the code style of this project
[ ] My code requires changes to the documentation
[ ] I have updated the documentation as required
[ ] All the tests have passed

☢ Does this introduce a breaking change?

[ ] Yes
[ ] No

ℹ Additional Information

Description

Improved the styling of the notifications dropdown in the header.
Enhanced the notification count display for better positioning.

Changes walkthrough 📝
Relevant filesEnhancement

header.phpEnhance notification dropdown styling in header

src/includes/header.php
Added class="position-relative" to the notifications dropdown link.  Updated the span for notification count with class="position-absolute top-0 start-100 translate-middle badge rounded-pill bg-danger".

+2/-2

💡 Penify usage:
Comment /help on the PR to get a list of all available Penify tools and their descriptions

@gstraccini codacy bypass


Received Date for Adv. Search results and Details page seems different, offset by 1 day

Describe the bug in current situation
After searching for Unopened requests, Request Date mentioned on the Adv,. Search results and Request Details  page seems to be different or Details Page's request date is offset by a day.
Note : Feels like, This happens for the requests submitted after 4PM/5PM , after business hours.
Search results show May 17 see below.

On the details page,

Link bug to the User Story
Impact of this bug
Describe the impact, i.e. what the impact is, and number of users impacted.
Medium impact.
Will confuse users and users could miss a request if they have narrow date ranges.
Chance of Occurring (high/medium/low/very low)
High
Lots for request outside business hours.
Pre Conditions: which Env, any pre-requesites or assumptions to execute steps?
Steps to Reproduce
Steps to reproduce the behavior:

Submit a request after 4:30pm PST
Search for request in Advanced Search
See  the error - the date is the day it was submitted
Click on request
See the correct Received Date (next business day)

Actual/ observed behaviour/ results
Expected behaviour
A clear and concise description of what you expected to happen. Use the gherking language.
GIVEN the request is submitted after 4:30PM PST
WHEN the user views the Received Date on the request view or advanced search
THEN the date should be the next business day (i.e. submitted after hours Wednesday, next business day is Thursday; submitted after hours on Friday, Saturday or Sunday, next business day is Monday).
Screenshots/ Visual Reference/ Source
If applicable, add screenshots to help explain your problem. You an use screengrab.

#2185, #2192 and #2308 are the same issue

Please note, the fix will only apply to new requests onward.

New requests created through online form in weekend, holiday and weekdays after 16:30 will have recevied date moved to next business day.
For new requests created through FOI app, if an IAO member select weekend or holiday as received date, it will change to next business day.

@m-prodan- can you send a file through EOD Friday,  so we can check to ensure that a weekend received file will change to the next business day.
EDU-2022-202020 - was submitted on the 26th EOD and the received date has been updated correctly to the next business day 27th.

Weekend and stat holiday (bonus!) submission tested - received date as expected was set to Aug 2!
Closing ticket


chore: add Imran to iam module

Forking from https://github.com/GoogleCloudPlatform/cloud-foundation-toolkit/pull/2542 since that is blocked on PAM

This has been applied


Emojis are not displaying

[x] I am on the latest Hyper.app version

[x] I have searched the issues of this repo and believe that this is not a duplicate

OS version and name: Ubuntu 19.04

Hyper.app version: 3.0.2

Link of a Gist with the contents of your .hyper.js: https://gist.github.com/devhammed/6505a4cf0e785ebb3c53a666d8a9b073

Relevant information from devtools (CMD+ALT+I on macOS, CTRL+SHIFT+I elsewhere): N/A

The issue is reproducible in vanilla Hyper.app: Is Vanilla

Issue
Emojis characters are not rendering at all...they renders correctly in Gnome Terminal but not in Hyper, I am using ZSH and Spaceship Prompt.
Gnome Terminal:

Hyper:

I am using the same font on both Fira Code which obviously means it is not a font issue.
I have no clue why this is not working, no errors in DevTools.
:cry:

Can you try with a CI build of current canary or maybe compile one yourself? Or setting webGLRenderer: false, in the config.

I have done that even before opening this issue, check above .hyper.js file.

I have fixed the issue, it is also affecting any Chromium-based apps including Google Chrome. For others facing this issue, follow these steps:

Add a new font config: vim ~/.config/fontconfig/conf.d/01-emoji.conf and paste below snippet:

<!DOCTYPE fontconfig SYSTEM "fonts.dtd">
<fontconfig>
  <alias>
    <family>serif</family>
    <prefer>
      <family>Noto Color Emoji</family>
    </prefer>
  </alias>
  <alias>
    <family>sans-serif</family>
    <prefer>
      <family>Noto Color Emoji</family>
    </prefer>
  </alias>
  <alias>
    <family>monospace</family>
    <prefer>
      <family>Noto Color Emoji</family>
    </prefer>
  </alias>
</fontconfig>

Restart the application and enjoy!

Tested on Ubuntu, I don't know about other distros but it should work.


Added Games statistics screens to QudUX

Several comments:

No real code guidelines (and not many comments ...) used, only my numerous years of C# habits
I've created the Table class to ease the management of tables, and I've added it to ConsoleUtilies.cs . I don't know if it is pertinent.
I've added a SingleBoxDivider method that allow to choose the height of the divider
I've kept my PlayerMutator and LoadGameHandler , but I bet  they're useless now
I've tested it , it works except a strange error when trying to define the key of the screen

See you.

That was my first PR since a long time, so I hope I did not make any mistakes. All should be in the correct situation now.


Switch SQL parser engine to new one

For #4335.

Codecov Report

Merging #4408 into master will decrease coverage by 0.16%.
The diff coverage is 5%.

@@             Coverage Diff              @@
##             master    #4408      +/-   ##
============================================
- Coverage     53.05%   52.89%   -0.17%
  Complexity      351      351
============================================
  Files          1048     1048
  Lines         19563    19623      +60
  Branches       3649     3671      +22
============================================
  Hits          10380    10380
- Misses         8527     8586      +59
- Partials        656      657       +1

Impacted Files
Coverage Δ
Complexity Δ

...phere/sql/parser/visitor/impl/MySQLDMLVisitor.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

...e/sql/parser/visitor/impl/SQLServerDMLVisitor.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

.../sql/parser/visitor/impl/PostgreSQLDMLVisitor.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

...phere/sql/parser/visitor/impl/SQL92DMLVisitor.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

...shardingsphere/sql/parser/core/SQLParseKernel.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

...here/sql/parser/visitor/impl/OracleDMLVisitor.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

...rite/token/generator/impl/TableTokenGenerator.java
71.42% <100%> (-9.22%)
1 <0> (ø)

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 9d884af...cc73377. Read the comment docs.

Pull Request Test Coverage Report for Build 9703

4 of 80 (5.0%)  changed or added relevant lines in 7 files are covered.
7 unchanged lines in 3 files lost coverage.
Overall coverage decreased (-0.2%) to 56.301%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

shardingsphere-sql-parser/shardingsphere-sql-parser-engine/src/main/java/org/apache/shardingsphere/sql/parser/core/SQLParseKernel.java
0
1
0.0%

shardingsphere-sql-parser/shardingsphere-sql-parser-mysql/src/main/java/org/apache/shardingsphere/sql/parser/visitor/impl/MySQLDMLVisitor.java
0
15
0.0%

shardingsphere-sql-parser/shardingsphere-sql-parser-oracle/src/main/java/org/apache/shardingsphere/sql/parser/visitor/impl/OracleDMLVisitor.java
0
15
0.0%

shardingsphere-sql-parser/shardingsphere-sql-parser-postgresql/src/main/java/org/apache/shardingsphere/sql/parser/visitor/impl/PostgreSQLDMLVisitor.java
0
15
0.0%

shardingsphere-sql-parser/shardingsphere-sql-parser-sql92/src/main/java/org/apache/shardingsphere/sql/parser/visitor/impl/SQL92DMLVisitor.java
0
15
0.0%

shardingsphere-sql-parser/shardingsphere-sql-parser-sqlserver/src/main/java/org/apache/shardingsphere/sql/parser/visitor/impl/SQLServerDMLVisitor.java
0
15
0.0%

Files with Coverage Reduction
New Missed Lines
%

sharding-orchestration/sharding-orchestration-center/sharding-orchestration-center-apollo/src/main/java/org/apache/shardingsphere/orchestration/center/instance/ApolloInstance.java
1
59.46%

sharding-core/sharding-core-rewrite/src/main/java/org/apache/shardingsphere/sharding/rewrite/token/generator/impl/TableTokenGenerator.java
3
88.57%

sharding-orchestration/sharding-orchestration-core/src/main/java/org/apache/shardingsphere/orchestration/internal/util/IpUtils.java
3
76.0%

Totals

Change from base Build 903:
-0.2%

Covered Lines:
11048

Relevant Lines:
19623

💛  - Coveralls


Pass in rest of args to _search_helper

As of right now when trying to use functions like max_marginal_relevance_search or max_marginal_relevance_search_by_vector the rest of the kwargs are not propagated to self._search_helper(). For example a user cannot explicitly state the distance_metric they want to use when calling max_marginal_relevance_search

will close this and update a new


import PropTypes from 'prop-types' instead of 'react'

Would you please update the repo and
import PropTypes from 'prop-types' instead of 'react'

Hi @h4m3d66.
I have no time in these days to work on this project, but if you'll change it and open the PR - I'll accept it.
Thanks.

please update the repo and
import PropTypes from 'prop-types' instead of 'react'.
ActualIy  dont know how to create PR and use it in the project...

It's merged now

@h4m3d66 but i am still getting the same issue of propTypes, when i am using the code master repo.
will you please help me out.


RECIPE PROJECT

Project description
Created a web application using Angular and Firebase with login authentication and sign up page.To add recipes and ingredients and save them to fire base with a single click. Which helps to make a shopping list as well.
Additional context
Please assign it hactoberfest label
Screenshots

Please assign it to me


issue_1 Description Is not editing #1

Related Issue
Closes #1
Type of Change
Put x inside the square bracket to specify what type of change your PR is:

[ ] New Feature
[x] Bug Fix
[ ] Code Refactor
[ ] Documentation Update
[ ] Other (please specify):

Description of Change
fixes- When we try to update the tickets ,all fields get edited accordingly but description is not changing.
Implementation Details
required changes have been done in app/(components)/EditTicketForm.jsx the file.
Demo

@disvid Add your name is contibuters.md and accept read invite as a collaborator. Then I will merge

@iamgoeldhruv done sir.


rdpei/server: Fix PDU length for RDPINPUT_PROTOCOL_V300

Quoting the commit message here:
When the server supports the protocol version RDPINPUT_PROTOCOL_V300,
the additional supportedFeatures field will be present.
The pduLength in the RDPINPUT_HEADER should, however, reflect this.

So, fix this error by writing the correct PDU length when the
supportedFeatures field is present.

Note: This is compile tested only. It should however be correct. Found this, while reading through the source code.
See also <IP_ADDRESS> RDPINPUT_SC_READY_PDU in [MS-RDPEI]

@freerdp-bot test

@pnowack nice catch!


feat/tcp: Prioritise smaller messages.

This change is

r? @vinipsmaker
(maidsafe_highfive has picked a reviewer for you, use r? to override)

r? @ustulation


Question for CDRs Method

When I attempted to redesign the complementarity-determining regions (CDRs) of the antibody, I found that it is only possible to fix multiple epitopes using a method similar to the one depicted in the diagram below. Has anyone tried this approach? Are there any other methods available?

Thanks for submitting the issue!
you can also provide a binary design mask too, I think that gives you the most customizability.  Also see https://github.com/generatebio/chroma/issues/5

Thanks for the answer!
Design_mask is a smarter approach compared to the method I'm currently using. #5

nice! thanks for the feedback. I am closing this for now, but feel free to reopen or open new issues.


pull request for e2e test at 12/22/2016 00:56:21

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Use np.float64_t for weights

np.float64_t is always used for the leaf feature value. Use the same type for leaf weights.
Fix #29

Thank you Matwey!


Routing API revamp RFC

I just started experimenting with routing (UWP app), so while I have fresh eyes, some thoughts:
Rename RoutingState to Router
"Routing state" implies a container of state data rather than a class responsible for navigation actions. "Router" seems a better name for a class that actually handles routing. This is further reinforced by the fact that even on the IScreen interface, the RoutingState property name is "Router". IMO, Router is both more apt and concise.
Formalize navigation stack into a class
Today, we have a handful of commands hanging off RoutingState along with a ReactiveList called "NavigationStack" (note that here it's not even a stack in the data structures sense of the term). It would be better to introduce a new class, NavigationStack with the following signature:
public class NavigationStack
{
  ReactiveCommand<Unit> Advance { get; }
  ReactiveCommand<Unit> Backtrack { get; }
  IRoutableViewModel Current { get; }
}

This would leave no ambiguity about what the commands do. If there is value in it, the class can implement IReactiveList to enable subscribing to notifications. Router would then have a read-only property named NavigationStack.
Replace the Navigate command with method + property
Is there value in Navigate being a command? Writing router.Navigate.Execute(vm) feels clunky. Would be better to have Navigate be a method, and have a companion Navigating observable property on Router. Another benefit is that the method can have an (optional) bool reset attribute, thus letting us drop the NavigateAndReset command on Router.

👍  A lot of this is in line with my thoughts on re-vamping routing, perhaps for V8. However, I'd prefer stack semantics for NavigationStack members, such as Push and Pop.
But that's pretty much irrelevant minutia right now. I think the important thing is to take routing in a direction that gives us:

far greater consistency across platforms (by sharing far more code and pushing more responsibility into that shared code)
far greater flexibility (not dictating so much about how routing occurs at the UI level)
a story for platforms that aren't VM-first friendly (e.g. Android)

These are difficult problems to solve, and I think we should look for inspiration elsewhere (Prism, MVVMCross etc).

When revamping navigation, I would also like something like Router.PushDialog.
This is a common scenario (I think), when I want edit an item in another ViewModel and then fall back to previous VM with new value.
Rigth now I use some hacks like this:
 EditItem = ReactiveCommand.CreateFromObservable<Item, Item>(item =>
            {

                var editVM = new EditItemViewModel(HostScreen, resolver, item);
                HostScreen.Router.Navigate.Execute(editVM).Subscribe();

                return HostScreen.WhenAnyValue(x => x.CurrentViewModel)
                        .Where(x => x == this)
                        .Take(1)
                        .TakeUntil(HostScreen.Router.NavigationStack.ShouldReset)
                        .Select(x => item);

                // or use EditDone command of EditVM

                return editVM.EditDone.Select(x => item)
                       .Take(1)
                       .TakeUntil(HostScreen.Router.NavigationStack.ShouldReset);
                 // and navigate back in EditItem.Subscribe
});

I think that with NavigationStack it would be great to write something like this:
return HostScreen.Router.PushDialog.Execute(editVM).Select(x => item);
PushDialog would return editVM when Router.Pop would bring us back to CurrentViewModel and handle all edge cases of navigating to terminate returned observable.
It's quite easy to make an extension method for that, but if you want to revamp the whole thing either way, it could be just built-in functionality.
If there is already a easy way to make it error free, please tell me :)

Assigning to @RLittlesII I know he's been wanting to get his feet dirty on this type of thing.


Updating baseline file for test

No idea about the failures. The test should be stable. It basically checks that based on the flow analysis we see 1 jit-time bailout with AggressiveMulIntTypeSpecDisabled  (to prevent more aggressive bailouts later that impact code quality too much). The bailout should be deterministic.

I also do not quite understand why rebaselining was needed. The test relies on bytecode offsets (that is a bit unfortunate). Sometimes those change because of unrelated changes, so I assumed that was the case. Now - why rebaselining needs only in release?
It seems as if we picked a change that results in different bytecode between release/debug. That seems strange.

Either release or debug baselines validate what the test needs to check, it is just the offsets are different (and that is not the interesting part)
If possible, can we just disable the test in release?
And enter a bug for me to follow up separately on why release has a different offset.

note sure it's a release/debug thing... It's passing with new baselines on x64 chk on my workstation.

Lets just disable the test then.

It does not look like we have a functional regression. It is just a test gone flakey - perhaps need to test in some different way.

@VSadov  - what exactly were you trying to test here?

OK, deleted test now.

I am trying to check the exact number and the nature of bailouts. More precisely we want AggressiveMulIntTypeSpecDisabled, but we do not want AggressiveIntTypeSpecDisabled.

@jackhorton did we add a flag to remove bytecode offset dependencies from baseline output?


Cannot use iptables CLUSTERIP inside container

Required information

Ubuntu 16.04
The output of "lxc info" or if that fails:

Kernel version: 4.4.0-57-generic
LXC version: 2.0.8
LXD version: 2.0.8
Storage backend in use: ZFS

Issue description
I'm trying to setup Network load balancing by using the linux built-in CLUSTERIP capabilities in a unprivileged container.
For the record, CLUSTERIP is a way to define the same IP on multiple machines et use MAC multicast to broadcast the packets on each machine. The the kernel use iptables CLUSTERIP rule to filter the packets on each hosts, so each packet is handled by one host only.
Creating the rule work fine:
$ iptables -I INPUT -i eth0 -d <IP_ADDRESS> -j CLUSTERIP --new --hashmode sourceip --clustermac 01:aa:7b:47:f7:d7 --total-nodes 2 --local-node 1
It creates an entry in the procfs at  /proc/net/ipt_CLUSTERIP/<IP_ADDRESS>
My issue is that that file must be readable but it is created with mode  0600 using the host root user, and so is unreadable by the guest root user.
Steps to reproduce

Step one
Create a CLUSTERIP rule:
$ iptables -I INPUT -i eth0 -d <IP_ADDRESS> -j CLUSTERIP --new --hashmode sourceip --clustermac 01:aa:7b:47:f7:d7 --total-nodes 2 --local-node 1

Step two
$ cat  /proc/net/ipt_CLUSTERIP/<IP_ADDRESS>
cat: /proc/net/ipt_CLUSTERIP/<IP_ADDRESS>: Permission denied

Hi,
That's a kernel bug. If the ipt_CLUSTERIP module is in fact properly namespaced (and the fact that you were allowed to use it suggests it is), then it should also allow writing to any file it creates in proc.
You'll need to file this as a kernel bug, possibly against the Ubuntu kernel on Launchpad (https://launchpad.net/ubuntu/+source/linux/+filebug) or contact the author of that netfilter plugin directly to have them look into this (probably best to confirm that this hasn't been fixed already though).
As for LXD itself, I'm closing this issue as there's unfortunately nothing we can do in LXD itself to fix permission issues that the kernel creates.


Integrate linting tools:

Setup Prettier and eslint;
Format src code;
Add linting job to github actions

Some files are still with 2 space format.
tsconfig.json, .prettierrc

Some files are still with 2 space format.
tsconfig.json, .prettierrc


[ENH] Update courses with Online Brain Intensive

Adds brainhack intensive online course to the course listing

🙌 Thanks! :)

Thanks!! :) 🙌


[WIP] Add ability to filter cards based on selected tag

I'm submitting an incomplete PR because discussion needs to be done before any progress can be made.
After removing the predicate mechanism of the filteredCardList in ModelManager.java, test cases that rely on the predicate system have begun to fail. The following is an example in CommandTestUtil.java that is causing a lot of command tests to fail.
/**
     * Updates {@code model}'s filtered list to show only the card at the given {@code targetIndex} in the
     * {@code model}'s address book.
     */
    public static void showCardAtIndex(Model model, Index targetIndex) {
        assertTrue(targetIndex.getZeroBased() < model.getFilteredCardList().size());

        Card card = model.getFilteredCardList().get(targetIndex.getZeroBased());
        final UUID uuid = card.getId();
        // PROBLEM IS HERE:
        model.updateFilteredCardList(new UuidPredicate(uuid));

        assertEquals(1, model.getFilteredCardList().size());
    }

It feels like forcing the list to display only a single card through this predicate violates the design that we intended, which is to obtain the list of cards through feeding it a desired tag.
I'm considering nuking those test cases (or at least the specific checks) that rely on these methods of filtering, but this is ultimately a decision that the whole team has to agree on since this will likely cause coverage to drop. About 18 tests are failing as a result of the change in filtering system.

@yong-jie In general it is safe to remove tests if it is no longer covering functionality in the application. So if all 18 tests cover something that is no longer being used then I think I am agreeable.

Pull Request Test Coverage Report for Build 192

15 of 16 (93.75%) changed or added relevant lines in 5 files are covered.
4 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.1%) to 87.159%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/main/java/seedu/address/model/ModelManager.java
11
12
91.67%

Files with Coverage Reduction
New Missed Lines
%

src/main/java/seedu/address/model/card/UuidPredicate.java
4
0.0%

Totals

Change from base Build 188:
-0.1%

Covered Lines:
1724

Relevant Lines:
1978

💛  - Coveralls


[FEATURE]

Please add a way to prevent players teleporting out of combat using /team home and /team warp

You can already do this by adding a delay to the teleport. in config.yml look for tpDelay.
If you need any more assistance please reply, else this issue will be closed in 40 days.

how do i edit config files?

that depends on how your server is hosted. You find the file in the folder /plugins/betterTeams/config.yml

This issue is being closed for inactivity. If you are still having problems feel free to open a new issue linking this one, make sure you answer any questions asked in the new issue.


Code editor cursor is too dark in dark mode

Cursor is too hard for people to see currently. Also I'm noticing that the pulse animation for the cursor that appears when the user isn't focused on the code pane is interfering with the normal pulsing animation whenever the user is focused within it.

@franknoirot is this still true?

imo
https://github.com/KittyCAD/modeling-app/assets/29681384/fb5d6e3a-b8d3-44c7-acac-49a32a6e5248

That one is tough @Irev-Dev, because a text cursor is often the primary visual cue for having focus on an input. It is very confusing to see a cursor in a text box, type several characters, and only then realize that you have not focused the text editor.

Maybe it's the line highlighting that's contributing to the visual clutter? If we were to have a consistent logic of text in focus === skinny cursor and text out of focus === block selection cursor, and made that brighter and got rid of the full-line highlighting, would that help make things clearer?
I did address the original intent of this issue with https://github.com/KittyCAD/modeling-app/commit/3fdf7bd45e75063fdbe957c1cbc1fe2517a080d2#diff-3260a6f05315c0f783e44ed248b4708bf74b46e620b0137e3726a345b00ece0c, I should have tied them together.

I'll be closing this issue since we can see the cursor within the dark mode in the editor as of now.
We can open any new issues around showing the block cursor highlighting and removing the line highlight when the code pane does not have focus, or other different workflows.
This bug was marked in the extermination station and it is completed. We discussed as a team and there was not any immediate work to be done.


readme.me // await this.report();

In your readme.md you mention  // await this.report();
This option has now been moved to the config.json file and should be changed in the readme.me
You need more coffee!

Thanks again! Yes, I really do. Will move most of this to the GitHub wiki anyway so the readme will be much smaller. Will keep this ticket open until it's migrated.

Thanks again! Yes, I really do. Will move most of this to the GitHub wiki anyway so the readme will be much smaller. Will keep this ticket open until it's migrated.


Missing Wallet endpoints

Please add the missing main client's wallet endpoints. For example, https://binance-docs.github.io/apidocs/spot/en/#user-asset-user_data

Thanks for raising this. I'm looking to work towards full API coverage, including thorough types as well as end to end tests. This has been an incremental progress, in all my SDKs. If you're looking for these missing pieces sooner rather than later, I'd also absolutely welcome contributions via pull requests.

All rest endpoints are now added as of recently, in case you are still looking for it.


Updated with new procedure that includes wait time calculation and holding queue.

Includes new flows, data actions, template for calling list, and additional graphics.

Includes all pending revisions.


Test/hide at

Coverage increased (+45.1%) to 100.0% when pulling 63442a949d3e6c0452ec814bc287d4ccdb959daf on test/hide-at into 08511ad4a0bc224df899b4156d8abe4b329e0c97 on master.