Introduce userConfig_sample.h, initial work on introduction to Visual Studio Code + PlatformUI

Hi,

I have added a Gemfile to be able to build the site locally using the classic bundle install and bundle exec jekyll serve way. This made it a whole lot easier to see what's happening when editing the page. At the same time, I added layout: default to the yaml section of every page, because the remote theme required it.
I have started to write a short introduction to Visual Studio Code + PlatformIO but haven't finished that yet.
I have moved the Programmcode section to its own page and explained how to create userConfig.h from userConfig_sample.h

Your gemfile are not needed, or?

Doch, um das Ding auch lokal über bundle exec jekyll serve anzeigen zu können, braucht es das Gemfile. Sonst kann man es ja nur über github-pages ansehen, was ich unpraktisch fand. github-pages ignoriert aber das Gemfile im repository einfach, sodass es da kein Problem gibt.


How do I assign different projects to different "admin"?

My actions before raising this issue

[x ] Read/searched the docs
[ ] Searched past issues

 I'd like that each "admin" can see each spcefic Project and not all the Projects.
Expected Behaviour
 Each "admin" only see the Projects that were assigned to him
Current Behaviour
 every "admin" can see all the Projects
Possible Solution
 Assigning "admin" to Project
Steps to Reproduce (for bugs)

Context
 For security reason, it's not expected that every "admin" can see all the Projects
Your Environment

Git hash commit (git log -1):
Docker version docker version (e.g. Docker 17.0.05): Docker 20.10.20
Are you using Docker Swarm or Kubernetes? Docker Swarm ?
Operating System and version (e.g. Linux, Windows, MacOS): Ubuntu 18.04
Code example or link to GitHub repo or gist to reproduce problem:
Other diagnostic information / logs:
Logs from `cvat` container

Hi, What do you mean under admin?
Usually it is a superuser who can see everything. If you want users do not see all the projects, do not grant them admin rights

@NielsenSPereira
If John is admin there is no way to restrict visibility anyhow for him. Admin is a person who is responsible for the whole CVAT instance.
Are you sure you want to assign users to admin role? What is your use-case?

@bsekachev
I understand that, what I think I need is a Spervisor permisson.
I have one Project with 12 tasks and one person (Supervisor A), will supervise  the annotation of the 6 users (each one has 2 tasks), so I assign him as "admin" Group for viewing all 12 tasks;
But I have another Project with 4 tasks and 4 different users and a different Supervisor ((Supervisor B)!
So, how can I assign this "Supervisor" permission to each supervisor?
Or assign him "Supervisor A" as "user" in all 12 Tasks; but when I click on Assignee + (Add user) it opens a pop-up to create another user and didn't show the list of current users, so I am a bit confuse to use this second option.

@NielsenSPereira
Just assign "Supervisor A" to the first project. The supervisor will see all the task only inside the project.
And assign "Supervisor B" to the second project.
Do not need to assign "admin" role for them.

it opens a pop-up to create another user and didn't show the list of current users, so I am a bit confuse to use this second option.

I am not sure I understand you here. When you click it opens pop-up where you need to choose existing user, there is no suggestion to create them:

I am assigning in the Admin Page.
Can I assign a Task to more than one "user"?
When I click on the Add (+) Assignee, it opens the popup window, but we can't see the list of users; it seems that we have to create a new one!

I think that this Add button (+) should enable to add an Assignee (from the user list) and not a New User.

Please, use the proposed way to assign users.

Can I assign a Task to more than one "user"?

No, but you can assign different users to different jobs of the same task. These users will be able to annotate the task in parallel this way.
If you want to more people have access to shared resources, consider using organizations

Not sure what happens on your side, if it is still relevant please open another issue. I will close this one.


[Enhancement] Span and Memory usage

With, .Net Core 2.1 finally going live (and being LTS), now seems the most appropriate time to consider some refactoring to liter the API with Span<T> and Memory<T>, considering this kind of project is really what they are made for and should deliver serious performance improvements.

I'd like to add some extra overloads to support Span, but it's probably a good idea to wait for the release kinks to be worked out. I've seen problems on Xamarin in the past (when using ImageSharp), and @SomeAnon42 mentioned that there still seem to be a couple of minor issues being worked out:
https://github.com/xamarin/xamarin-android/issues/1679
https://github.com/dotnet/corefx/issues/29654

this kind of project is really what they are made for and should deliver serious performance improvements

Realistically, they shouldn't help our performance too much, and I would consider adding them primarily for convenience. Span<T> is good for when you want to seek over a large-ish buffer and operate on different subsections without allocating. Veldrid doesn't really do that kind of thing, so it's not clear where we'd get any benefit. For the most part, Veldrid just takes a contiguous buffer you give it and passes it through to a native graphics API. We don't do much seeking and slicing. In any case, Span is certainly not faster than unsafe code (which is already prominently used because we are interfacing with native API's), but there may be places we can replace unsafe seeking with safe seeking via Spans and keep our performance.
The one place it could really help is the OBJ and MTL parsers (not part of core Veldrid), since those seek over a text file and parse through it. That's the perfect match for Span<T>, and it could help a ton in terms of perf and GC allocation. I had a version of ObjParser in the past which used Span<T>, but I ripped it out because Span wasn't stable and had issues on some platforms.

What @mellinoe said. Please, do get all of these new bits working across all .NET Standard platforms, first. There remain a number of issues on other platforms, all caused by recent changes for Span<T> to the System.Memory assembly:

aspnet/EntityFrameworkCore - Exception while loading assemblies - FileNotFoundException - Could not load assembly System.Memory #12081

xamarin/xamarin-macios - iOS app build fails when referencing package (e.g. Entity Framework Core 2.1) that depends on newer System.Memory #4168

xamarin/xamarin-macios - Build Failed: "Error MT2002: Failed to resolve "System.Buffers.OwnedMemory`1" reference" #4191

xamarin/xamarin-android - Android app build fails when referencing package (e.g. Entity Framework Core 2.1) that depends on newer System.Memory #1769

aspnet/SignalR - Xamarin support tracking issue #1886

Any update on whether or not stability has improved?

It looks like all of the issues above are closed at this point, so hopefully it works. It would require some testing of course, but this could potentially get added soon.

Awesome!

Are there any news on this @mellinoe ? I still need to use fixed to update a texture from a Span

I think this is mostly fixed with #311 or not?


Merge prTesting_202005241942 branch to master branch

Docs Build status updates of commit a807d8e:
:warning: Validation status: warnings

File
Status
Preview URL
Details

split/updateTestCon.md
:warning:Warning

Details

split/updateTestCon.md

[Warning-author-missing] Missing attribute: author. Add the current author's GitHub ID.
[Warning-title-missing] Missing attribute: title. Add a title string to show in search engine results.
Line 0: [Warning-h1-not-first] Markdown content is not allowed before H1.
Line 1: [Suggestion-h1-missing] H1 is required. Use a single hash (#) followed by a space to create your top-level heading. NOTE: This Suggestion will become a Warning on 04/01/2020.

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


Exact Search and Persisted Indices for JS

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


Slimming transactional queries for validate

@oskarpearson had a look at our query log during performance testing and noticed that doing signature validation had a bunch of queries in a transaction.  This PR is about trying to reduce that number:

move creator signature validation out of the transaction - it's a bit belt-and-braces that we even have it in here still, as the creator signature should be validated earlier in the process.  It doesn't hurt to try (as it's almost a no-op if it's already validated), but it doesn't need to happen in the transaction for validating the current signature
reduce the queries used by stamping the constituency petition journal.
when the record already exists don't do a reload to refresh the in-memoery object after we issue the atomic sql UPDATE statement
when the record doesn't exist, don't save it and then update it (2 queries), just insert with the correct signature count (1 query)

Merging after verbal "ok" from @pixeltrix.


improvement: Add nitro enclave support for EKS

PR o'clock
Description
Make it possible to enable nitro enclave support in EKS.
Should resolve (after testing in complete):
https://github.com/terraform-aws-modules/terraform-aws-eks/issues/1181
https://github.com/terraform-aws-modules/terraform-aws-eks/issues/1182
Checklist

[x] CI tests are passing
[x] README.md has been updated after any changes to variables and outputs. See https://github.com/terraform-aws-modules/terraform-aws-eks/#doc-generation

This appears to be usable as is. There might be some improvements that can be made but this makes it possible to enable this in a launch template and get nodes with full support for Nitro enclaves.

This appears to be usable as is. There might be some improvements that can be made but this makes it possible to enable this in a launch template and get nodes with full support for Nitro enclaves.

@barryib Who is the right person to review and merge this?

@barryib Who is the right person to review and merge this?

Thanks @spkane for your contribution.

Thanks @spkane for your contribution.

Thanks @spkane for your contribution.

And thank you @barryib , for pushing it through. Much appreciated!

Thanks @spkane for your contribution.

And thank you @barryib , for pushing it through. Much appreciated!


bisector compare of array of objects no longer works

It looks like 22cdb3f2b1b98593a08907d17c12756404411b1d broke my compare function, which used an array of objects.
I see I can use array of values, but wanted to point out this breaking change.
My compare function was
            that.bisectX = d3.bisector(function (d, x) {
                if (that.xascending) {
                    return d.x - x;
                } else {
                    return x - d.x;
                }
            }).left;

The breaking change is at https://github.com/d3/d3-array/blob/14efbe0f0f39331df0578e3527e4a0f5d30cc50d/src/bisector.js#L16

So you're saying bisector() compare function no longer supports object, right? Even though this case is mentioned (for sort()) in https://github.com/d3/d3-array/issues/217#issuecomment-892690103

I don’t follow. What are you trying to do?

briefly: I have array of objects d, and want to bisect by d.x

There are a lot of tests of bisecting arrays of objects:
https://github.com/d3/d3-array/blob/main/test/bisector-test.js
How is your usage different? Can you share a minimal, complete, live example of how it is not doing what you expect, say as an Observable notebook?

see https://observablehq.com/@louking/d3-bisector-with-object-comparison-function
I'm not sure the best way to get observable to print like console.log() but the debugger brings you into where you can see the console.log()
the compare1(x,x) causes the bisector to always return hi

If you pass an accessor rather than a comparator,
bisectX = d3.bisector(d => d.x).left

then it works as intended.
https://observablehq.com/d/468b832f4f688643
But, I think you’re right that this might be a bug for bisector’s weird comparator that takes different arguments. I need to investigate a little more.

Also, it works if you implement a “normal” (symmetric, non-weird) comparator:
bisectX = d3.bisector((a, b) => a.x - b.x).left

And then you pass in test values that have the same shape as the things you are bisecting, e.g.
bisectX(dataarr, {x: 2})

However, this approach deviates from the example given in the README, and also deviates from the more recommended pattern of using an accessor (whereby you pass in a search value equivalent to the accessor’s output, not its input). So, I think at a minimum we need to update some documentation here, but ideally we find a way to make this work.

I have a fix up in #250. Thanks for the additional explanation that helped me understand the problem! 🙏

If you pass an accessor rather than a comparator,
bisectX = d3.bisector(d => d.x).left

then it works as intended.

I don't think this works for me, because my actual comparator handles both ascending and descending arrays.
            that.bisectX = d3.bisector(function (d, x) {
                if (that.xascending) {
                    return d.x - x;
                } else {
                    return x - d.x;
                }
            }).left;

In the meantime, I've created a parallel array with just the x values, and changed d.x in the above function to d.
Thanks for your support!

In the descending case, you could say
bisectX = d3.bisector(d => -d.x).left

and then pass -x as the search value. Or you can use the symmetric comparator as I described in the other comment. Here is the descending form:
bisectX = d3.bisector((a, b) => b.x - a.x).left

and then pass an object {x} to search.

So, to flesh out the example, you could use a symmetric comparator like so:
bisectX = d3.bisector(xascending ? (a, b) => a.x - b.x : (a, b) => b.x - a.x).left

And then say
bisectX(array, {x})

Or you could use an accessor like so:
bisectX = d3.bisector(xascending ? (d) => d.x : (d) => -d.x).left

And then say
bisectX(array, xascending ? x : -x)


Add override for nova-atom-ui

This PR adds an override for the nova-atom-ui theme. It fixes the placement of icons in tabs. Care to review the changes and give feedback, and merge it, if not? :smiley:
Screenshots
Before

After

Thanks!


ssa: ResourceManager.ApplyAll fails to apply ClusterRoleBinding

We've got 2 controllers inside 1 operator (kubebuilder setup) that:

create some structs using client-go's https://github.com/kubernetes/client-go/tree/master/applyconfigurations. Among those resources is ClusterRole and ClusterRoleBinding
convert them to *unstructured.Unstructured using "k8s.io/apimachinery/pkg/runtime".DefaultUnstructuredConverter.ToUnstructured
uses ssa.SetNativeKindsDefaults on list of those resources
tries to apply them using ResourceManager.ApplyAll

There are some clusters for which this operation fails, like<EMAIL_ADDRESS>and kind cluster created by<EMAIL_ADDRESS>default image for this version so<EMAIL_ADDRESS>They fail with error message
{
	  "level": "error",
	  "ts":<PHONE_NUMBER>.8819587,
	  "msg": "Reconciler error",
	  ...
	  "error": "failed to apply resources to the cluster: ClusterRoleBinding/crbName namespace not specified, error: clusterroles.rbac.authorization.k8s.io \"crName\" not found"
}

A bit of debugging led me to believe that the error comes from this line https://github.com/fluxcd/pkg/blob/7741c5ff7c8a25aca88269aa73111a3164724f95/ssa/manager_apply.go#L139 which then goes through m.validationError(dryRunObject, err) and then gets returned here https://github.com/fluxcd/pkg/blob/7741c5ff7c8a25aca88269aa73111a3164724f95/ssa/manager_diff.go#L184-L186
I'm not really sure what's happening, I'll try to add some code in next comment that reproduces this issue.
We've encountered this problem twice, independently in my workplace. For now we've just changed usage of ResourceManager.ApplyAll to for loop that goes through slice of those resources, which we additionally sort using ssa.SortableUnstructureds, and apply them using ResourceManager.Apply.
Resource manager is created like that:
fieldManager:="some-string"
...
ResourceManager: ssa.NewResourceManager(mgr.GetClient(),
			nil, // poller is needed only for WaitForSet, we're not using it here
			ssa.Owner{
				Field: fieldManager,
				Group: fieldManager + "-group",
			}),
...

This seems like some kind of data-race, where ClusterRole is not yet fully "commited" and ClusterRoleBinding tries to reference no-yet-existing ClusterRole, but if I understand k8s it shouldn't happen 🙈. Weird this is that this error is not going away, subsequent reconciles do not succeed.
My setup:

<EMAIL_ADDRESS>k8s 1.25.3
go 1.19.3
os: macos, intel based

Ok, I've not been able to reproduce it for now... I might try again later

We do test ClusterRole/ClusterRoleBinding see https://github.com/fluxcd/pkg/blob/main/ssa/testdata/test1.yaml
I suspect this is a RKE bug, not much we can do about it in this repo.

@stefanprodan RKE is 1 thing, but it also bugs on kind, which is commonly used for e2e testing. Does it seem like a bug in ssa library, or should I look harder into controller-runtime's client? Wdyt?

Flux itself contains ClusterRole/ClusterRoleBinding resources which are applied with pkg/ssa, and all our e2e tests use Kubernetes Kind. No one reported this bug nor did we see it in e2e tests, I would expect thousands of Flux users running into this issue.

Sure, seems reasonable. Thanks for help 🙇🏻

If you can consistently reproduce this with kubectl apply --server-side --dry-run against Kind, then this is an upstream bug, so the Kubernetes repo would be the best place to report it.

Closing this as it does not seem related to the actual package.


Allow Disk Use Option

Hi, thanks for creating this laravel-mongodb.
I only have one concern, I am using the DB::collection raw function is there a way to set the Allow disk use option to true?

@gerald27
This issue is now fixed on Moloquent Branch :)


Bug in mtd in the OBJECT_ID column for the 2d ascii output files.

This issue was raised via met-help:
https://rt.rap.ucar.edu/rt/Ticket/Display.html?id=90244
Jeff Duda reported that the OBJECT_ID column of the mtd 2d output files failed to include a leading "C" for the cluster forecast and cluster object lines.

Randy fixed this bug in both the develop and master_v8.1 branches.  Here's the commit:
https://github.com/NCAR/MET/commit/0c4c8595e5db3f92bbb81ad016e89e6459cd73c1


Add support for Epson network scanners

I've tested it based on release-19.03. I don't have a setup with master to validate cherry-pick works on master. Is there a procedure to validate a patch against master except by upgrading to master? For a separate standing packages it's easy to install it with nix-env but this package involves configuration options and I am not yet sure how to build with options included but isolated from the host.

Clone another copy of the nixpkgs repo somewhere and do nix-build . -A some.package.attrpath from within it?

@risicle I can definitely build the package but it's hard to tell if it definitely works with the whole setup (sane config files etc.) unless I build the whole system using master and the PR and try it out.

You can take a simple (or complex) machine configuration and build a vm from it https://nixos.wiki/wiki/Cheatsheet#Reuse_a_package_as_a_build_environment which you can just launch with a single command.

@risicle these build tools are amazing! I've just build a VM with my configuration and this branch (that is based on master) and validated that scanimage -L shows the remote scanner and that scanimage > file generates an image.

/assign @symphorien

Looks good to me!
I rebuilt my system with these commits rebased on 19.03. My (usb) scanner still works.

@symphorien thanks for checking. How do we get it merged? I believe you are marked as the package maintainer so I hoped you can nudge it in.

Sorry, I don't have merge rights. We can only wait for someone who can to notice this PR.


Fix swapped environment variables in the README.md

Moved PR to the upstream project: https://github.com/daBONDi/go-rest-wol/pull/19


Enable linter for duplicated imports

How to categorize this PR?

/area quality
/kind enhancement
What this PR does / why we need it:
Enable linter for duplicated imports.
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
/cc @rfranzke
Release note:

NONE

/kind enhancement

/retest

/test pull-gardener-e2e-kind-operator


bug: NavBar not use the correct color when apply bg-base-300

What version of daisyUI are you using?
3.5.0
Describe your issue
Before in the 2.x version I could style the navbar with
But now, the color is totally different. Weirdly, with bg-base-100 it works.
Config:
module.exports = {
  plugins: [
    require('daisyui'),
  ],
  daisyui: {
    themes: [
      {
          dark: {
            ...require("daisyui/src/theming/themes")["[data-theme=dark]"],
             primary: "#38bdf8",
             secondary: "#d1fae5",
             accent: "#1FB2A6",
             neutral: "#191D24",
             "base-100": "#064e3b",
             "base_200": "#242933",
             "base_300": "#053e2e",
             "base_content": "#A6ADBB",
             info: "#3ABFF8",
             success: "#4f46e5",
             warning: "#fcd34d",
             error: "#b91c1c",
             '--toolbar': '28 25 23',
          },
      },
    ],
  },
}

Code:

       AA

What browsers are you seeing the problem on?
All browsers
Reproduction URL
https://play.tailwindcss.com/kou6Fg8CLP

What do you mean? bg-base-300 and bg-base-200 has a slight different color. Your reproduction URL also works fine. It is #e5e6e6 for bg-base-300 and #f2f2f2 for bg-base-200.
What color code did you expect it to work?

e5e6e6

Exactly, this is NOT the color. The config says:
	neutral: "#191D24",
	"base-100": "#064e3b",
	"base_200": "#242933",
	"base_300": "#053e2e",

Note that if you change it to "base_300": "red", is easier to spot that the bar have not the expected color

You're using underscore for base_200 base_300 and base_content they all will be ignored and their values will be generated automatically. Use dash -. Not underscore _
Let me know if you have any questions


Convert existing class names to CamelCase

In #203, we decided all classes should have CamelCase names. All new-ish classes (CLIArgParser, FeedHandler etc.) already do, while older ones (file_urlreader, controller etc.) don't.
What needs to be done for each old-style class:

rename it to CamelCase (file_urlreader → FileUrlreader)
update its name in all LOG statements (LOG("opml_urlreader::reload: downloading %s", ...) → LOG("OPMLUrlreader::reload: downloading %s", ...))
update include guards in headers. You need to change three lines: #ifndef and #define at the start of the file, and commend after #endif at the end (#ifndef NEWSBOAT_FILE_URLREADER_H_ → #ifndef NEWSBOAT_FILEURLREADER_H_)
remove underscores from file names (opml_urlreader.cpp → opmlurlreader.cpp, dialogs_formaction.h → dialogsformaction.h)

It's a boring task, but it'll take you through half the codebase, so if you wanted to get some very general picture of what there is, it's as good way as any.

Is there a reason OPML is all caps and Url only has the first letter capitalized?

Oh, great point! file_urlreader should become FileURLReader, then. I'll update the post. Thank you for the vigilance!

Hmm, or should we treat abbreviations as ordinary words? OPMLURLReader doesn't seem very readable to me. On the other hand, we probably won't have many such identifiers in the codebase, so it's okay as long as they look distinct enough.

E.g. PEP 8 says to "capitalize all the letters of the abbreviation. Thus HTTPServerError is better than HttpServerError".

Rust core team advises to use "HttpServerError" too, so let's go with that. I will update the top post to clarify.
@zaowen, the problem right now isn't with the style guide, it's with, em, running :) Different clang-format versions produce different formatting on the same input with the same options, which means we can't enforce the style on programmer's side. Enforcing it on Travis poises a question of how then to apply the edits, since Travis can't commit anything itself. See #198 for details; maybe you'll have some more suggestions.

Rust core team advises to use "HttpServerError" too, so let's go with that.

Cool. Looks better in my opinion (for what it's worth).

How should things like sort_direction_t be dealt with?
Also I'm working on this issue, you can assign me to it if you'd like.

How should things like sort_direction_t be dealt with?

For enum classes, drop the _t postfix and apply the rules above to get CamelCase. For everything else (if any), please prepare a list and let me decide on one-by-one basis.

Should the classes in the 3rd-party folder be changed?

No. As the name implies, they aren't ours, and we update the contents of that directory from time to time by downloading new versions, so any changes done there will be lost eventually.

Should hyphens be removed from file names? or just underscores?

I think the only hyphens we have are in test-helpers.h and test-helpers.cpp. I'd keep them on a principle that they should stand out a bit (they are in the "test" directory but they aren't tests).

Formaction or FormAction?

FormAction, because it's still two separate words even if our codebase uses them as a single entity :)

Ttrss or TtRss?

TtRss, because "TT-RSS" is two acronyms separated by a hyphen. We treat
acronyms as words, and ignore the hyphens.

InoReader or Inoreader?

Do we want to capitalize the namespaces? rsspp newsboat and podboat?

Should structs be counted like classes?

configcontainer.h has an enum class called configdata_t as well as a struct named configdata under the new scheme this causes a naming confict.
How should i procede?


Please add SHA1 when GUI wallet release everytime

Please add SHA1 when GUI wallet release to prevent cheating.
Thanks for your job.

I've wanted to do this actually but I've yet to figure out how! Is there a guide somewhere for how to do this?

I generally use HashTab to verify SHA1:

Ok I looked into this again and it seems it's even easier than I thought, not sure why I never found this before when searching for it. Missing the "digest" keyword perhaps. Anyway, I can do this with openssl as such:
openssl dgst -sha256 filename.exe
Is SHA1 sufficient? Or should I use another algorithm?

SHA1 and another will be better.

Ok I'll do that then.

9c27d767083b2f32645fed56372ebcb9cbea40d0

Closing, hashes will be included for the next release.


feat: add a codefix to fix class to className in react & add spelling suggest for JSX attributes

Add a code fix to fix class in JSX (to className) or for in JSX (to htmlFor). This is useful for copy-paste HTML code into a JSX file.
Before:

After (fix all):

I don't know how JSX name resolution works here, but I'm surprised that it's a type assignability error instead of a name resolution error. If it were the latter, you could probably just special-case the spelling correction algorithm and use the existing did-you-mean codefix.
Switching which error is issued actually might be the right fix for this.

I manually do this all the time!

If it were the latter, you could probably just special-case the spelling correction algorithm and use the existing did-you-mean codefix.

Yes, in my last try I tried to use it but it seems like JSX never emit diag 2551 (... not exist on type ..., do you mean ...?)
// The following 2 property both misspelled with doubled last letter
const x = <a onCompositionUpdateCapturee={() => {}} />; // Property '...' does not exist on type '...'.ts(2322)
window.AbortControllerr // Property '...' does not exist on type '...'. Did you mean '...'?ts(2551)

I'll try to investigate this again

Oh I found a comment:
https://github.com/microsoft/TypeScript/blob/c219fdae08607e07ec875d2137fcba7c2d93792c/src/compiler/checker.ts#L15984
TODO: Spelling suggestions for excess jsx attributes (needs new diagnostic messages)
🤔 let me try it

OK it's working now

@andrewbranch I have re-implemented this with spelling checking logic. It's working well on the InstinctElements but not working well on class components because the suggestion is wrapped in a "No overload matches this call." diag.

Done. I'll add some test for it later

Test added. ready for review

@sheetalkamat are you OK with this change now? @andrewbranch likes it at least =)

It's not a spelling mistake, it is another type of mistake.
I think it's cross-language interference. This is like an editor that lets you paste in Norwegian and offers to translate it to Swedish.

Thanks @Jack-Works!


What do I run?

I have no idea what to do with these files.

Thanks! The page I was on before downloaded a .zip file but there was no .app file in it. I had tried to run one of the files via terminal and was getting "qt-deploy" command not found and I was all over the place installing Qt developer tools and XCode, I couldn't figure out what was going on.


⚠ Bofrost.com has degraded performance

In ce87e48, Bofrost.com (https://www.bofrost.com) experienced degraded performance:

HTTP code: 200
Response time: 773 ms

Resolved: Bofrost.com performance has improved in 35dfcc7 after 8 minutes.


Pluralizer is not handling the cases where the noun ends with y

In this example NetworkPolicy is ok when there's only one policy but with zero or 2+ policies it's shown as
NetworkPolicys

#887

Validated on Rancher version master-head (08/27/2020) 5e1b21b93


Modifying config to allow PATCH interaction does not work

Describe the bug
CORS preflight does never include PATCH in the Access-Control-Allow-Methods Header., even after editing server.xml with the element
<cors domain="/"         allowedOrigins="*"         allowedMethods="GET, DELETE, POST, PUT, HEAD, PATCH"         allowedHeaders="*"         maxAge="3600" />
nor after editing the fhir-server-config.json with the entries
"resources": {       "open": true,       "Patient": {       }     },
or
"resources": {       "open": true,       "Patient": {           "interactions": ["create", "read", "vread", "history", "search", "update", "delete", "patch"]       }     },
Environment
Which version of LinuxForHealth FHIR Server?
I run the latest docker image ( Tag is ibmcom/ibm-fhir-server:4.11.1)
To Reproduce
Steps to reproduce the behavior:

Install docker image
populate FHIR server with a resource
try to update it partially using HTTP PATCH > results in cors preflight error "Acess to fetch at 'https://localhost:9443/fhir-server/api/v4/Patient/187af6a6d4c-81fb9c96-96ef-4916-92b1-6fdf11bd41e2' from origin 'http://localhost:3001' has been blocked by CORS policy: Method PATCH is not allowed by Access-Control-Allow-Methods in preflight response."
try again with mentioned fixes to config files (after restarting)

Expected behavior
PATCH should be included in CORS Access-Control-Allow-Methods Header

It does work now for some reason after restarting the container the 5th time.


task_wdt: Task watchdog got triggered. The following tasks did not reset the watchdog in time: (IDFGH-7555)

E (10662) task_wdt: Task watchdog got triggered. The following tasks did not reset the watchdog in time:
E (10662) task_wdt:  - IDLE0 (CPU 0)
E (10662) task_wdt: Tasks currently running:
E (10662) task_wdt: CPU 0: main
E (10662) task_wdt: CPU 1: IDLE1
E (10662) task_wdt: Print CPU 0 (current core) backtrace

Backtrace:0x400F9200:0x3FFB0690 0x400822B9:0x3FFB06B0 0x400F01DD:0x3FFBAA60 0x400F0FA1:0x3FFBAA80 0x400F16EA:0x3FFBAAA0 0x4008A106:0x3FFBAAC0 0x40089279:0x3FFBAAE0 0x40089308:0x3FFBAB00 0x400899B3:0x3FFBAB30 0x400DBA65:0x3FFBAB60 0x400E16B2:0x3FFBAB90 0x400D7596:0x3FFBAEA0 0x400D43AA:0x3FFBAEF0 0x40155789:0x3FFBAF40 0x400D2B25:0x3FFBAF70 0x400D2993:0x3FFBAF90 0x400D2A81:0x3FFBAFF0 0x4008D62A:0x3FFBB010 0x4008D565:0x3FFBB030

i get this error . i'am not using xtaskcreate or vs ..
what is problem ?

there is a problem in your code, please post it so we can find the mistake. you must either create the task at idle priority (priority 0), or avoid doing endless loops without any delays.


Fixes valyukhov v1.2.2

Changes proposed in this pull request:

Has the changelog been added to what is being done in this pull request
@foxprogs @Vorobushec @Valyukhov

fixed #202


Magenta param is not worked

Could you tell me how to fix this:

According to this picture, I can't see the purple right arrow >.
Is there anything to do or am I go wrong?
Any informations
Alacritty v3.3.0
Windows 10 Pro (Build 18362.267)

Sorry this is not an issue. The font doesn't have that arrow capital.


Scene Editor not adding names

When using the scene editor to add new entities, even if you change their name (and that change reflects on the scene editor) the entity will not have a Name component. This is specially intrusive when you try to add relations and the entities are identified only by their numeric ID.

The whole scene editor will be nuked in the next version so this will def be fixed kek


Remove the skip-tz-utc configuration item

First-time contributors' checklist

[ ] I've signed Contributor License Agreement that's required for repo owners to accept my contribution.

What is changed, added, or deleted? (Required)

Which DM version(s) do your changes apply to? (Required)

[x] master (the latest development version, including v3.0 changes)
[x] v2.0 (TiDB DM 2.0 versions)
[ ] v1.0 (TiDB DM 1.0 versions)

What is the related PR or file link(s)?

This PR is translated from:
Other reference link(s):

Do your changes match any of the following descriptions?

[ ] Delete files
[ ] Change aliases
[ ] Need modification after applied to another branch
[ ] Might cause conflicts after applied to another branch

@lance6716

/cc @lichunzhu

/label needs-cherry-pick-2.0
/translation doing/from-zh/no-need
/cc @CharLotteiu
/assign

/translation doing

/lgtm

/lgtm

/merge

/remove-translation doing
/translation done


Givens rotation stuff should not be in ops

In OpenFermion, "ops" has been the place we keep data structures. This is somewhat strange actually, but it is a closely followed policy so far with the exception of the Givens rotation code. I think it came from "ops = operators" which is a bit unusual, I know.
As we put more and more helper functions into OpenFermion for compiling circuits in plugins, I wonder if we should consider reorganizing the modules and make one called "algorithms" or something like that where the Givens code would work. This has the downside of potentially breaking people's code but I think having a clear systematic organization is important. Thoughts @jarrodmcc and @kevinsung ?

The reason it's in ops is that QuadraticHamiltonian depends on it. The bigger problem is that we have data structures that use stuff that belong in a different module (I'd argure that general_basis_change in _polynomial_tensor.py also belongs in a separate module) but the only other place we have to put that stuff is in utils, which depends on ops. So I agree that we should have a module that is lower in the import hierarchy than ops. algorithms is a bit vague; perhaps linalg would be better. A lot of the code in utils should probably be moved there too.
This is not the only reorganization I would do if I had my way. I would prefer to do a big restructuring before version 1.0 which would make the library easier to maintain. I mentioned some ideas at https://github.com/quantumlib/OpenFermion/issues/271 but I'm sure I could come up with more. In order to allow us the flexibility to do such restructurings, I think we should be encouraging imports from the top level, i.e.
import openfermion

or
from openfermion import FermionOperator

instead of
from openfermion.ops import FermionOperator

That way we can move stuff between modules without breaking anybody's code.

givens_rotations is now in linalg, so I guess this can be closed.


Processing cmumosei dataset

I'm interesting in the inference performance using cmumosei dataset. The README shows the cmumosei dataset structure, but the actual SDK only downloads csd files. How did you extract and process the cmumosei dataset into the said layout? What script are you using to process the cmumosei dataset? Thanks!

Thanks! We will release the scripts and data for CMU-MOSEI today/tomorrow

Thank you so much for sharing the data! Out of curiosity, why does the dataset only contain .wav audio files and no video or text files? Thanks.

Yeah, there is video and text folders in this dataset. But somehow I didn't create a audio to contain the .wav files. Let me reorganize the folders.

Appreciate the help. Any updates on this matter?

Now updated thanks!

Thank you very much! I noticed there are some files that are missing. For example, it will say [17:11:20] /github/workspace/src/video/video_reader.cc:83: ERROR opening: ./dataset/cmumosei/test/video/3At-BKm9eYk_3.mp4, No such file or directory for some video/audio files. Are they left out intentionally? Just wanna make sure I didn't miss anything important:)

Yeah, some files are missing. The sources of the CMU-MOSEI dataset I downloaded miss some files. This is expected.

Hi,
Thank you for sharing the link to the CMU-MOSEI dataset. I downloaded the dataset, and in the labels\BA folder, what do -1, -2, -3, 0, 1, 2, 3 correspond to? Because the CMU-MOSEI dataset has labels for six basic emotions, why do we have seven here, and could you tell me which label corresponds to which emotion?


Matcha : How to get colors like in the picture ?

I wonder how do you manage to get the right colorscheme
I just installed the theme and this is what i get :

Is it something with Light, Dark, Zen Dream and Zen Galaxy ?
I don't know what it is.

Yeah the last three updates has been pretty sucks about this :(
Previously there's --zen-colors-tertiary that they use for the browser frame, but later they decided to deprecate the variable, assigned it to --toolbar-colors, and not making it compatible with the previous theme parameters yet. No explanations in the docs either.
I'm reporting it already on Discord and wait for devs' decision first.


SciPy 2019

- title: SciPy 2019
  dir: scipy-2019
  youtube_list:
    - https://www.youtube.com/playlist?list=PLYx7XA2nY5GcDQblpQ_M1V3PQPoLWiDAC
  related_urls:
  - label: Conference schedule
    url: https://www.scipy2019.scipy.org/confschedule
  language: eng
  dates:
    begin: 2019-07-08
    end: 2019-07-14
    default: 2019-07-10
  issue: 832
  minimal_download: true
  tags:

Hi @Daniel-at-github #829 has the speakers and titles cleaned up.


Are queries available in the current release?

{"2021-05-23 06:22:10.149196":{"symbol":"mvis","action":"bought","quantity":0.0,"cost":20.5,"watch":true},"_id":"60aa3ae2e723bbff6862ae69"} {"2021-05-23 06:22:10.154262":{"symbol":"gbtc","action":"bought","quantity":0.0,"cost":52.96,"watch":true},"_id":"60aa3ae2e723bbff6862ae6a"} {"2021-05-23 06:22:10.155244":{"symbol":"gme","action":"bought","quantity":0.0,"cost":79.88,"watch":true},"_id":"60aa3ae2e723bbff6862ae6b"} {"2021-05-23 06:22:10.155363":{"symbol":"amc","action":"bought","quantity":0.0,"cost":16.5,"watch":true},"_id":"60aa3ae2e723bbff6862ae6c"} {"2021-05-23 06:22:10.155430":{"symbol":"msft","action":"bought","quantity":0.0,"cost":250.0,"watch":true},"_id":"60aa3ae2e723bbff6862ae6d"}
find() returns all documents as expected.
find({'watch': true}) returns an empty list.
first({'watch': true}) returns no documents.
last({'watch': true}) returns no docments.
first and last with query result in a Bad State exception in dart Stream somewhere.
Is is the structure of my documents causing this issue?

You need to provide the full 'key-path' in your query. You should change the structure so that the keys you want to query are not dynamic/unknown.
The exception is expected for first and last if no record was found.


Which is the correct DINOv2 checkpoint?

Hello,
This looks like an excellent piece of work - thank you for sharing.
May i clarify exactly which is the correct DINOv2 checkpoint to download? I thought it was the ViT-L/14 distilled but that throws an error regarding missing keys, so must not be the intended target.
Any and all assistance is appreciated.
All the best

dinov2_vitg14_pretrain.pth

We would also consider to release the checkpoints with smaller version DINOs.

Thank you for the speedy assistance! Much appreciated. Will give that a go!

For my learning, may I ask what the difference is between the "With registers" and "Without registers" version of the ViT? If you happen to know?

Am afraid I am still hitting errors with dinov2_vitg14_pretrain.pth.
RuntimeError: Error(s) in loading state_dict for ControlLDM:
Missing key(s) in state_dict: "betas", "alphas_cumprod", "alphas_cumprod_prev", "sqrt_alphas_cumprod", "sqrt_one_minus_alphas_cumprod", "log_one_minus_alphas_cumprod", "sqrt_recip_alphas_cumprod", "sqrt_recipm1_alphas_cumprod", "posterior_variance", "posterior_log_variance_clipped", "posterior_mean_coef1", "posterior_mean_coef2", "logvar", "model.diffusion_model.time_embed.0.weight", "model.diffusion_model.time_embed.0.bias", "model.diffusion_model.time_embed.2.weight", "model.diffusion_model.time_embed.2.bias", "model.diffusion_model.input_blocks.0.0.weight".etc
Also getting unexpected keys:
Unexpected key(s) in state_dict: "cls_token", "pos_embed", "mask_token", "patch_embed.proj.weight", "patch_embed.proj.bias", "blocks.0.norm1.weight", "blocks.0.norm1.bias", "blocks.0.attn.qkv.weight", "blocks.0.attn.qkv.bias", "blocks.0.attn.proj.weight", "blocks.0.attn.proj.bias", "blocks.0.ls1.gamma", "blocks.0.norm2.weight", "blocks.0.norm2.bias", "blocks.0.mlp.w12.weight".
Any thoughts as to where I might be going wrong?

For my learning, may I ask what the difference is between the "With registers" and "Without registers" version of the ViT? If you happen to know?

I find this place a bit confusing as well, and I was wondering if you could help clarify it for me. Thank you very much.


ESLint not loading local config unless you specify nodePath local

Hello,
not sure if this is intended behavior, but I have a globally installed instance of eslint and when I try to install eslint locally and load it from my local folder, ESLint in vscode will always prioritize the global installation found under %USERPROFILE%/AppData/Roaming/npm.
I have installed Node using the regular windows install:

Node version 6.9.1
npm version 4.4.1
eslint global version 3.17.1
eslint local version 3.12.2

In order to override that behavior I have to include "eslint.nodePath": "./node_modules" in the settings of each workspace that I want to load eslint locally, or I have to uninstall the global eslint instance, which is not desired.
Is this a known issue? Or is it something with my configuration?

This works for me without a problem. ESLint extension always prefers a local install eslint over a global one. Can you share a reproducible case (a git repository) that demos the behaviour?

Hi @dbaeumer it seems to happen for create react app applications, but it's been documented to be a problem within eslint itself. We can probably close this, I'll open up another issue if I see more weird behaviours. Thanks for the reply!


Doesn't seem to be working with cypress cucumber setup

Hey @Bkucera ,
I totally appreciate your efforts and vision coming up with this much needed plugin for cypress. Infact, I believe all the e2e test frameworks must have it for sure.
I have some tests in my suite which seem/behave flaky no matter how hard I have tried to do anything/everything to fix them, but each time some would fail randomly and pass on rerun. I came across your plugin and felt it's what I needed indeed. But, doesn't seem to be working for us.
This is my cypress.json file -
{
    "baseUrl": "http://localhost:4200",
    "testFiles": "**/*.feature",
    "video": false,
    "env": {
        "auth_username": "1234",
        "auth_password": "1234",
        "retries": 2
    },
    "requestTimeout": 15000,
    "modifyObstructiveCode": false,
    "reporter": "junit",
    "reporterOptions": {
        "mochaFile": "test-reports/test-output-[hash].xml",
        "toConsole": false
    }
}

This is cypress/support/index.js -
// ***********************************************************
// This example support/index.js is processed and
// loaded automatically before your test files.
//
// This is a great place to put global configuration and
// behavior that modifies Cypress.
//
// You can change the location of this file or turn off
// automatically serving support files with the
// 'supportFile' configuration option.
//
// You can read more here:
// https://on.cypress.io/configuration
// ***********************************************************

// Import commands.js using ES2015 syntax:
import './commands';
import 'cypress-plugin-retries';
// Alternatively you can use CommonJS syntax:
// require('./commands')

I don't find it working for me. We have feature files defined in gherkin format as required by cypress-cucumber-preprocessor. Any help is much appreciated.
P.S. Please Let me know if any other information from end would help. We are using Type Script.

It works after I added
 "types": [
            "cypress", "cypress-wait-until", "cypress-plugin-retries"
        ]

And
env: {
"RETRIES": 2
}

to
cypress.json
Thanks a ton for coming out with the plugin.


[BUG/QUESTION] How does one use Laravel's facades inside a Task?

I'm currently trying to use Laravel's Storage facade inside my custom task as I have to download and save multiple files to disk. But I'm running into an issue as I get the error A facade root has not been set
Are there any know solutions for this?

You'll have to configure anything in the worker again, because objects are serialized and sent to another process.

Any guide on how? I couldn't find any guides from Laravel on how to manually configure everything again

How do you bootstrap your initial script? Do you include certain files?

I don't really know, it's a Laravel project and it bootstraps a lot - but I don't know exactly what

Any solution? I have the same problem

We're both not using Laravel, so this has to be answered by somebody from the Laravel community.


Added Slack Clone

Closes: #570

Title : Slack Clone
Name: : Maheshwari Love
Idenitfy yourself: :  GSSOC

Describe the add-ons or changes you've made 📃
Added new project in typescript folder
How Has This Been Tested? ⚙

I have run it on my local system and have tested.
Screenshots 📷

Type of change ☑

What sort of change have you made:
Added new project

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

Checklist: ☑

[x] My code follows the Contributing Guidelines & Code of Conduct of this project.
[x] This PR does not contain plagiarized content.
[x] I have performed a self-review of my own code.
[x] I have commented my code, particularly wherever it was hard to understand.
[x] My changes generate no new warnings.

Note to reviewers 📄

@Avdhesh-Varshney i have created new pr with required changes kindly review if any changes

@Avdhesh-Varshney added the required changes

@Avdhesh-Varshney changes added


feat(providers): add ability to register tax and fulfillment providers using medusa-extender annotations

@adrien2p ok I added some unit and integration tests! I'd like to start with these 2 fulfillment providers for now, but I'm setting up patterns here to be used again for payment, search, notification, file, oauth,  and other future providers!

Also this closes #128

Sweet! Let's ship it!

@dwene i ve merged it. Can I get you to update the doc please?


idea: add a patch for civil war aftermath to prevent burning down camps until after the war

if you manage to get the seasons unending betrayal working it may be a good idea to prevent the player from being able to burn down enemy military camps until after the war is over, or have them respawn if the player has civil war aftermath installed

Duplicate, upon further testing this was not a result of aftermath but a conquest related issue


chore: implement checkpoint-client

closes https://github.com/prisma/prisma2-private/issues/82

TODO

[ ] wait till cli path has and project has are optional in a new checkpoint-client version

There should be a new release now.


Title (Page/Tab) to be adapted

The title of Page/Tab should be "ECRIN | Clinical Research Repository"

copied twice...


The State of Hyper vs other Terminal Apps.

[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: MacOS Catalina  10.15.4

Hyper.app version: 3.0.2

The issue is reproducible in vanilla Hyper.app: YES

Issue
State of Hyper
Hyper is very good at what it does. Make the terminal look good. It has many extensions. This is a good app for people that use the terminal and dont care about other features.
The Problems
As a dev, we need a terminal that has features that dev's need. These are a fast terminal app and a decentralized ecosystem.
We need no make this terminal fast.
The Terminal app on macOS opens up in 0.3 seconds
While Hyper takes 4 seconds. (Vanilla Hyper + no customize)
When i have 5+ tabs with Hyper It would eventually lead to my terminal becoming unresponsive on occasion. Not too often, but enough to interrupt my workflow.
The Light
We need to have a version of hyper that is fast, and minimal while making it look good.
This is for all the devs, and users like me and you.

Okay but what is the point of this issue? I agree with you on almost everything but it will be impossible to reach native speeds with an Electron stack. Hyper lost the power of ZEIT behind it and is mostly maintained by 1.5 (I'm the .5) people these days.

@Stanzilla It is ambitious, but remove the electron stack, aka remake the app. I am down to help and make a terminal app that takes the pros of hyper and puts it in a fast almost native terminal. Who is the other maintainor? We 3 should make a new terminal app. Even though there are a lot of terminal apps they don't have as customizability as hyper.  I want to do this. What about you?

That's a bit out of scope for this project. There are a lot of good and native terminals out there for every OS.

I just want to improve Hyper @Stanzilla

Well that's fine, we're always taking Pull Requests :)


fix: 660 rehaul writing code

Landing page changes for writing code

Restructured and rewrote the landing page
Moved relevant content to the landing page for JavaScript Editor from JS Objects page
Removed Appsmith Framework from the sidebar (under Writing Code)

I feel we should avoid bombarding users with too much information than required in each page. and lets also avoid repeating info in multiple pages. For eg, we are introducing users to working with widgets, JS Objects, Debugging in a separate page, so lets avoid writing paras about them in the landing page. We can instead add cards/links on the landing page.
Suggested Outline
Writing Code
 [As this is landing page, lets keep it compact, get straight to the point and then provide paths to find more information]
 [Intro]

Coding in JavaScript
  [Here's were we ought to talk about how to write JS code in Appsmith. Everything in the current docs related to {{}} bindings, single line, multi-line,  and also make a mention of JS Objects and cross reference]

JavaScript editor
[Short intro and youtube video, cross reference to how to work with JS editor]
Programming widgets
 [the information about reactive programming]

Browse the following sections to learn to code in Appsmith and expand the capabilities of your applications.
[Cards/Links]

JS Objects
Working with JavaScript editor
Creating workflows
JavaScript promises
External libraries
Appsmith framework functions

JS Objects
[Intro]
Creating a JS Object
 [relevant info]

Types of functions
Synchronous
Asynchronous
Calling a JS Object function
  [relevant info]

Working with JavaScript editor
Executing functions
Code snippets
Linter errors
Response tab
Errors tab
Logs tab
Settings (cross-reference to asyn function settings sub-page)
Debugging errors
Debugger statement
console.log()
[Move Writing complex code to how to guide as it is a usecase]
Asynchronous Function Settings
Defining asynchronous function
Async function settings
Run on page load
Configure run on page load
Confirm before calling
Using asynchronous function settings

Creating Workflows
Fields
Sync fields
Async fields
Async function in a sync field
  [Put the usecase somewhere in the how to guides and cross-reference here]

Actions on event triggers
Success / Error Handling
  [IMO if this is here we need to write the code version of it and not show the UI piece which can be explained in showAlert()]

Complex Workflows
Parallel execution
Serial execution
Conditional execution
Control flow
    [we need to write intro here about async-await and try-catch and then cross reference]

JavaScript Promises
Async/Await
Async
Await
Methods
Promise.any()
Promise.race()
Promise.all()
Promise.allSettled()
Using Promises in Appsmith

I feel we should avoid bombarding users with too much information than required in each page. and lets also avoid repeating info in multiple pages. For eg, we are introducing users to working with widgets, JS Objects, Debugging in a separate page, so lets avoid writing paras about them in the landing page. We can instead add cards/links on the landing page.
Suggested Outline
Writing Code
 [As this is landing page, lets keep it compact, get straight to the point and then provide paths to find more information]
 [Intro]

Coding in JavaScript
  [Here's were we ought to talk about how to write JS code in Appsmith. Everything in the current docs related to {{}} bindings, single line, multi-line,  and also make a mention of JS Objects and cross reference]

JavaScript editor
[Short intro and youtube video, cross reference to how to work with JS editor]
Programming widgets
 [the information about reactive programming]

Browse the following sections to learn to code in Appsmith and expand the capabilities of your applications. [Cards/Links]

JS Objects
Working with JavaScript editor
Creating workflows
JavaScript promises
External libraries
Appsmith framework functions

Moved the JS objects section to the ticket - #661 as it's not part of this ticket.
Creating Workflows
Fields
Sync fields
Async fields
Async function in a sync field
  [Put the usecase somewhere in the how to guides and cross-reference here]

Actions on event triggers
Success / Error Handling
  [IMO if this is here we need to write the code version of it and not show the UI piece which can be explained in showAlert()]

Complex Workflows
Parallel execution
Serial execution
Conditional execution
JavaScript Promises
Async/Await
Async
Await
Methods
Promise.any()
Promise.race()
Promise.all()
Promise.allSettled()
Using Promises in Appsmith

For writing code landing page - if we are talking about programming in widgets - It gives the new users a sense that we just have widgets. As per my discussion with the team, when someone new to Appsmith reads writing code, the user feels that they are lost. Let's discuss this, and then we can finalize the outline.

@danciaclara - As discussed the changes for writing code - landing page and Appsmith framework landing page is done and available for final review.


[Feature Request] is there a file to list the common property of most components?

Existing Component
Yes
Component Name
all element-ui component
Description
is there a file to list the common property of most components?
i want to compose complex components base on element-ui component.
so ,i want to konw ,is there a list of common property of element-ui component
like: disabled,clearable...

You can find all props in type file.


Fix policy drops on restart due to lack of kvstore sync

In the process of endpoints being restored from disk, all programs are
regenerated. This bases the programs on the latest snapshots of all local
caches for identities, ipcache, services, and so on. In order to not
destabilize existing endpoint, it must be ensured that local caches have been
synchronized to a valid state. This was done for services as well as identities
if the endpoint was not assigned a fixed or well-known identity. The first
part of the fix is to also require the ipcache to be synchronized to avoid
egress policy drops for endpoints subject to egress policy.
The second part of the fix is to make all endpoints subject to these
synchronization requirements.
Example:
coredns is assigned a well-known identity. When coredns is restored from disk
and regenerated, premature regeneration without the full list of identities
being available will result in the allowed policy map to be stripped of all
global identities. This can result in temporary policy drops.
The risk of avoiding regeneration on restore is that restarting the agent no
longer guarantees to fix an eventual program with the BPF program. It is thus
required for the orchestrator to eventually restart the pod/container on
failure. This will happen in a timely manner with Kubernetes.

This change is

test-me-please

test-me-please

Removing release blocker flag. Upgrade impact is not clear.

As much as I love to merge this, we need to regenerate if we want to be able to migrate etcd operator managed pods across versions.

test-me-please

Coverage decreased (-0.01%) to 43.101% when pulling 20d43ecb7a058e8c462941f8017d64d0e365edba on pr/tgraf/wait-to-regenerate into 4fcd5af51b333d9b4e0f26df2ce1ed816e02d014 on master.

@joestringer @aanm @ianvernon I've changed this PR to only wait for ipcache and kvstore if the endpoint being restored is not using a well-known or fixed identity. It fixes the upgrade problem but still leaves a race where well-known or fixed identity endpoints can be regenerated before the ipcache is potentially fully restored to what it was before.
To summarize the problem: When we are in a state where we once had kvstore connectivity, the ipcache is populated with information beyond what can be reconstructed from k8s events so if we regenerate before we have connectivity, we will temporarily remove/miss entries that can lead to egress policy drops for a couple of seconds. While waiting would be possible for restarts where no regeneration is expected, it won't work for upgrade scenarios where we assume that a regeneration is needed due to some map format change or another reason.

Verified that this solves the egress drops on restart in the testing cluster


generate sphinx command

Generate conf.py like in dephell with markdown support and inserted project information, index.md, installation.md

Might be a good idea to have generate sphinx also add the necessary requirements (e.g. sphinx, recommonmark, etc.) and add the [tool.dephell.docs] env to pyproject.toml

Add a section in config -- yes, great idea. Add requirements -- also a good idea, but we can't be sure where is the best place for it. For example, some users want to have everything in one dev section. It's not DepHell's way, but DepHell never forces opinions about the environment.


Local - Make metaphor optional

To add a flag to skip metaphor deployment.
Why?

To make local a choice to develop k8s tools like controllers
Reduce disk pressure on local

Merged to release 1.11 branch


fix(core)!: Actually generate an Ethereum-compatible signature.

Add the missing '\x19Ethereum ...' prefix and message length before hashing and signing.
What ❔
Reconcile comment and implementation. The sign_message function now generates an Ethereum-compatible signature for the input message.
Why ❔
At a minimum, the comment and implementation disagreed (comment said "this generates an Ethereum-compatible signature, which it didn't actually do; the implementation just hashed message w/o adding the prefix and message length). I opted to fix the implementation (instead of adjusting the comment), as this seemed the more apt way forward to me.
Checklist

[x] PR title corresponds to the body of PR (we generate changelog entries from PRs).
[ ] Tests for the changes have been added / updated.
[x] Documentation comments have been added / updated.
[x] Code has been formatted via zk fmt and zk lint.

hmm so how do we currently use this sign_message? From the first glance, this change should break somethign (or something should have not worked before)

I agree, it would be a breaking change. As far as I can tell, it has no users. I stumbled upon it while prototyping something. Some of the methods in pk_signer, e.g., sign_transaction are used in the SDK though. Not sure if any 3rd parties are using this API.

Fixed by https://github.com/matter-labs/zksync-era/pull/1294


Balgillo/sev none

Preparing sarif-tools v3.
This started out as a fix for #47 but it's led to a refactoring of the code for grouping issues for:

Consistent issue sorting and grouping
Consistent handling of none severity
Consistent treatment of issues with the same issue code but distinct descriptions

Obviously I'm unfamiliar with the code base, haven't used the tool, etc., so take my feedback with a grain of salt. :)

@debonte Huge thanks for reviewing - we now have more consistent naming and I've fixed a bug that I introduced with these changes.  Let me know whether you have any further comments or approve for merge.


Added frontend template for game civil wars to Templates folder

Description 📜
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context.
Fixes # <issue_number_here>

This would be great if you provide any of this   📝

[x] Video (Animations, screen-recordings, presentations and regular explanatory films are all possibilties etc)
[ ] Documentation (Content Creation in the form of codes or tutorials)
[x] Other (If you choose other, Please mention changes below)

Domain of Contribution 📊

[ ] Navbar design
[ ] Footer
[ ] Single page website
[ ] Animation design
[ ] parallax effect design
[x] Others

Checklist ✅
]

[x] I have performed a self-review of my own code or work.
[ ] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generates no new warnings.
[x] I'm Hacktoberfest contributor

Any tutorials or documentation have you added
(made by you only not from google)

[ ] Tutorial
[ ] Documentation

Screenshots / Gif (Compulsory) 📸

LGTM @lord-benjamin


-v/--verbose option showing HTTP and JSON

In debugging this:

https://github.com/simonw/dclient/issues/23

I realized it would be useful to have an option to see details of the HTTP requests (including JSON bodies) being sent.

Documentation: https://dclient.datasette.io/en/latest/inserting.html#dclient-insert-help and https://dclient.datasette.io/en/latest/queries.html#dclient-query-help


[info] Handle identifiers visible even for unvisible datasets/files

This is to inform about a dataverse limitation reported by "our" @vkush in https://github.com/IQSS/dataverse/issues/8881
Since the URLs in the handle record do not disclose any information about the dataverse/organisation/dataset/file via the url-scheme the visiblity of the handle should be no problem.
For example, see this handle record for a file
https://hdl.handle.net/21.11165/4Cat/a48r-4u58/1?noredirect=true
It does not contain any information about the resource it links to.

@vkush - maybe we need another tag to identify upstream issues (in dataverse) that we can/will not solve ourselves like this one.

"external" tag is created.


RangeError in /clan/req-n-go-8p2qg08p

Error in ClashLeaders
RangeError in /clan/req-n-go-8p2qg08p
Maximum call stack size exceeded
View on Bugsnag
Stacktrace
webpack:///node_modules/vue/dist/vue.esm.js:143 - hasOwn

View full stacktrace
Created automatically via Bugsnag

An error linked to this issue has been marked as fixed in Bugsnag
RangeError in /clan/req-n-go-8p2qg08p


Add support for parsing custom link element <geom_group>

This will add support for parsing URDF files that may have links containing one or more optional <geom_group> elements. This allows us to specify additional collision geometry for a robot.
Example use case:
The HERB robot model uses low resolution meshes for the fingers to speed-up collision checking.
If you want to do higher resolution collision checking for grasping then you can programmatically change the geometry group for the hand links.
The bottom-right image shows the geometry group "fine_collision" which has higher resolution geometry for the Barrett WAM gripper.

Is this issue closed now?

Yes #20 has been merged.


Frame pairs once matched

@Nikomus You'll have to tidy this up a bit, but the functionality is working.
The svg frame 1) isn't square and 2) has a white bg in the middle.
Also welcome corrections on my padding / size changes. I think it'll be neat if there is an animation that shrinks the picture slightly and then pops it in a frame.

#44 #47


TypeError: Cannot read property 'toLowerCase' of null

Hi,
I tried the NodeJs solution. Included my DirectlineSecret of my Microsoft-Bot in the
var DirectlineSecret = ".................................";
app.use(actionsOnGoogleAdapter(DirectlineSecret).router);
but I am getting the following error:
TypeError: Cannot read property 'toLowerCase' of null
at continueConversation (\bot-framework-actions-on-google\lib\fulfilment.js:71:36)
at \bot-framework-actions-on-google\lib\fulfilment.js:122:5
at ActionsSdkApp.handleRequestAsync (actions-on-google\assistant-app.js:511:29)
at ActionsSdkApp.handleRequest (actions-on-google\assistant-app.js:419:10)
at bot-framework-actions-on-google\lib\fulfilment.js:135:9
at Layer.handle [as handle_request] (express\lib\router\layer.js:95:5)
at trim_prefix (\express\lib\router\index.js:317:13)
at express\lib\router\index.js:284:7
at Function.process_params \express\lib\router\index.js:335:12)
at next (express\lib\router\index.js:275:10)
Please help me with this issue.
Kind Regards,
Ragha

I am also having the same issue. Is the module working for anyone.

I was hitting the wrong URL. That is the reason for the issue. Once I changed that the connection established

@vignesh-123 What do you mean by hitting the wrong URL ? I am also facing the same issue.

Hello friend, could you solve your problem? which url has to be configured.
I have for example: https://domain.azurewebsites.net/api/messages

@vignesh-123 What do you mean by hitting the wrong URL ? I am also facing the same issue.

Hello friend, could you solve your problem? which url has to be configured.
I have for example: https://domain.azurewebsites.net/api/messages

@SiddharthSunchu What do you mean by hitting the wrong URL ? I am also facing the same issue.
Hello friend, could you solve your problem? which url has to be configured.
I have for example: https://domain.azurewebsites.net/api/messages


Add points add and move button

Description
In our workflow in most programs adding a point or shape is typically immediately followed by a repositioning of the point/shape for proper centering.
Currently, this requires to move between add (Shortcut 2) and select (Shortcut 3) mode back and forth, making it a error prone procedure if one hits Shortcut 1 or the delete button by accident.
Here I propose two workarounds (Currently only implemented for the points layer):

Add a new add_and_select button which adds a new point when there is no existing point at the cursor position or selects the points if the cursor is above a point. This has the advantage that it is a distinguished mode and works well for the points layer, but since there are several options to add a specific shape I guess one would need one per shape.
Pressing Shift while clicking in add mode behaves like being in select mode.

Both options seem viable to me, but I think that the second option is more scalable as it can be easily added per shape without adding more buttons.
Type of change

[X] New feature (non-breaking change which adds functionality)
[X] Breaking change (fix or feature that would cause existing functionality to not work as expected)

In the repository main branch, there is an enabled option that holding shortcut (ex 3) will only temporarily change the mode.
Also You could remap shortcut from 3 to Shift in settings. So it looks like the second option is already in the repository, but you need only to update shortcuts in settings.

I agree that option 2 is the better one, and it's great that we already have a version of it implemented. I would thake this issue also as a reminder that we need to increase discoverability of all these shortcuts!

Oh that is awesome, thank you for pointing me in the right direction :)
I decided to not use Shift, as it is a modifier key for the select option by itself and leads to all kinds of weird behaviors ^^
However, I would like to propose some further changes in the PR https://github.com/napari/napari/pull/5021
I will close this now here :)


Data normalization etc.

Hi,
I see that there is no implementation for normalization, scaling etc. of the data. Can I implement that with a new pull request?

@anjalibhavan Sorry for the late response. Pull requests are very welcomed. :)


Is it possible to change opus bandwidth a little lower than 48000?

Hi, Cedric.
After I checked the kraken system perfectly working on ec2 env, I found kraken using the opus codec and the setting has 48000 bandwidth.
But, that means to me, even if I send just a bit of data stream, there is sort of waste of data on time, I think.
Is it possible to change opus bandwidth a little lower than 48000?
Thank you again, Cedric.

48000 is the sample rate, and I think WebRTC enforces the value.

@cedricfung
Much appreciated for your reply, Cedric :)


[R] Fixes for R package on windows

Unfortunately overlaps with #140 since it seems like we were working in parallel.

That's fine. Note that if you use throw std::runtime_error() instead of Rcpp:;stop(), the compiler will understand the function exits and you do not have to add another return statement.
Also note the fix in status.cc in my PR.

can this be merged?

I'll integrate @jeroenooms's changes with mine and I'll let you know when it's ready. Probably early next week.

@wesm this is good to merge - once you've merged I'll submit to CRAN.

[Current coverage][cc-pull] is 76.89%

Merging [#142][cc-pull] into [master][cc-base-branch] will increase coverage by 5.60%

@@             master       #142   diff @@
==========================================
  Files            48         25     -23
  Lines          2790       1307   -1483
  Methods         339          0    -339
  Messages          0          0
  Branches        387          0    -387
==========================================
- Hits           1989       1005    -984
+ Misses          489        302    -187
+ Partials        312          0    -312

4 files (not in diff) in python/feather were deleted. more
1 files (not in diff) in python were deleted. more
16 files (not in diff) in cpp/src/feather were deleted. more
1 files (not in diff) in cpp/src were deleted. more
2 files (not in diff) in src were modified. more

Misses +2

Powered by Codecov. Last updated by [7d917a9...f5f9bcf][cc-compare]
[cc-base-branch]: https://codecov.io/gh/wesm/feather/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/wesm/feather/compare/7d917a93f0e5c489ebde0ced811b543fb208311c...f5f9bcf8a097bb791720752a095f0d32f13b5c08
[cc-pull]: https://codecov.io/gh/wesm/feather/pull/142?src=pr

awesome, thanks


Initialize swagger checker from swagger object directly

Intended to address: https://github.com/westfieldlabs/apivore/issues/100
Background
Currently the SwaggerChecker initialize method has no flexibility about how Apivore::Swagger is constructed. By accepting a full object, the client gains the flexibility to fetch a swagger file however it wishes to, whether by file, remote HTTPS, or carrier pigeon.
Since README and specs revolve around .instance_for, whose signature remains unchanged, there are no other changes required.
What have I done?

The previous initialization routine has been moved to .from_local_route, and .instance_for has been updated to reflect that.
The new initializer accepts a Apivore::Swagger (or any object that accepts #validate and #each_response)

Closing because I don't want to see this in my PR list forever.
If you're interested in merging down the line, I'm happy to help make any changes you feel need to be made.

Wish I saw this before making #118. If there is interest from the maintainers about that PR I'll pull this in and tweak it a little. I'm unhappy with the two-different places to check if JSON/YAML.

Yeah, I had closed owing to the fact that it seemed like PRs weren't being handled or looked into at all by the maintainers, hopefully your PR meets with greater success.

Thank you, but I just leave mine floating around like dingleberries!


Fixed an issue with spreading a spreadable generic expression into generic JSX

fixes https://github.com/microsoft/TypeScript/issues/51577

@typescript-bot run dt
@typescript-bot test top100

This PR appears to have broken main via a baseline change.


slots.text.default sometimes referenced the wrong host component

Example code.webc:

When used inside the default slot of another component: <my-component>Test</my-component> the slots.text.default value was wrong or empty.

Related to #48


Stop immediate trdm call

Turning off TRDM until model and database maintenance is complete.

so is this the quickest way to pause trdm?

Warnings

:warning:
Please add the JIRA issue key to the PR title (e.g. MB-123)

  Generated by :no_entry_sign: dangerJS against e9d6239828b83a4963af6c839bff8a8e6b60da43

Test coverage on server failed.
Last run: Mon 04 Dec 2023 05:42:00 PM UTC
Refer to this report to see coverage details for the server app.
Documentation for test coverage can be found at this link.


Added server browser made by cpdt

Added server browser to /servers so users can browse straight from the website.

If you are planning to continue working on this, it needs to be re-submitted against R2Northstar/NorthstarTF.


macro "min" passed 3 arguments, but takes just 2

I always recive this error when just try to compile the example:
I don't know how to fix it
c:\users\frank\appdata\local\arduino15\packages\arduino\tools\arm-none-eabi-gcc\7-2017q4\arm-none-eabi\include\c++\7.2.1\bits\stl_algobase.h:243:56: error: macro "min" passed 3 arguments, but takes just 2
 min(const _Tp& __a, const _Tp& __b, _Compare __comp)

                                                    ^

c:\users\frank\appdata\local\arduino15\packages\arduino\tools\arm-none-eabi-gcc\7-2017q4\arm-none-eabi\include\c++\7.2.1\bits\stl_algobase.h:265:56: error: macro "max" passed 3 arguments, but takes just 2
 max(const _Tp& __a, const _Tp& __b, _Compare __comp)

Seems like there is something misconfigured in you project/setup.
Are you using the correct architecture/board?

The library got completely rewitten to also support InfluxDb 2 with tls etc., please check out version 3.
If your issue still remains, please open a new issue.


Provide support for multiparty IM type, mpim

https://api.slack.com/types/mpim

Added PR for mpim type support:
https://github.com/nlopes/slack/pull/181


iOS Safari on iPhone, history mode triggers page refresh

Hello, came across a bug with VueRouter 2.0.1 and iOS Safari on iPhone only.
If I use the history mode of the Router which is supposed to be supported by the browser it fails and triggers a page refresh instead of changing the state.
You can try it with the official example fiddle here: http://jsfiddle.net/yyx990803/xgrjzsup/
Just change the mode to history and test iOS Safari on an iPhone.
The workaround is simple, I use userAgent detection instead of feature detection to check for iPhone and Safari and change the mode back to hash.
I have no idea of what is causing the page refresh, it's not the history API as using a regular window.History.pushState() works in the browser.

which version of iOS?

@posva iOS 10.1

Tested on the simulator and works fine 😅

Yes, I also tested on the Simulator and didn't catch the bug until it was reported using an iPhone SE and later confirmed on an iPhone 6.

same issue. It's works fine in simulator. But in iPhone 6S iOS 10.1 will refresh the page.

Tested on a real iPhone 6 iOS 10.1 and cannot reproduce.

Tested on real iPhone 6 iOS 10.1 Safari and still cannot reproduce. Please re-open if a proper reproduction can be provided.

Same here on iPhone SE (iOS 10.2), unfortunately I can't reproduce it. I have another iPhone SE with the same iOS version and it works fine there.

I can confirm what @helloiamlukas reports - switching from incognito to normal mode fixed the issue. Using iPhone 6S with iOS 10.2.1
User agent of Safari:
Mozilla/5.0 (iPhone; CPU iPhone OS 10_2_1 like Mac OS X) AppleWebKit/602.4.6 (KHTML, like Gecko) Version/10.0 Mobile/14D27 Safari/602.1

I'm getting this in normal mode too. hashchange is being fired twice: first with the new route and then again with the previous one.
This dirty hack works, by blocking the consecutive call:
  setupListeners () {
    let last = 0
    window.addEventListener('hashchange', () => {
      let now = Date.now()
      if (now - last < 100) return
      last = now
      if (!ensureSlash()) {
        return
      }
      this.transitionTo(getHash(), route => {
        replaceHash(route.fullPath)
      })
    })
  }

On https://github.com/vuejs/vue-router/blob/dc9d52f0f1314ee84792350c8c412e00719c3742/src/history/hash.js#L20-L29

Can reproduce, iOS 10.2.1 on iPhone 5. Using Vue-Router in hash mode

I'm seeing this as well with 10.3.1 and history mode.
Any way around this? I need to use history mode due to hosting at S3 and needing to send to deep urls.

Similarly, vue-router fails in Safari 8 and 9 because those versions fires popstate on page load, causing it to transition to / no matter the URL.
One can normalize this behaviour by adding the popstateevent listener in a setTimeout after load, similar to this issue: https://github.com/visionmedia/page.js/issues/213
diff --git a/src/history/html5.js b/src/history/html5.js
index 3716370..cefb808 100644
--- a/src/history/html5.js
+++ b/src/history/html5.js
@@ -16,14 +16,18 @@ export class HTML5History extends History {
       setupScroll()
     }

-    window.addEventListener('popstate', e => {
-      const current = this.current
-      this.transitionTo(getLocation(this.base), route => {
-        if (expectScroll) {
-          handleScroll(router, route, current, true)
-        }
-      })
-    })
+    window.addEventListener('load', function () {
+      setTimeout(function () {
+        window.addEventListener('popstate', e => {
+          const current = this.current
+          this.transitionTo(getLocation(this.base), route => {
+            if (expectScroll) {
+              handleScroll(router, route, current, true)
+            }
+          })
+        })
+      }, 0)
+    }, false)
   }

   go (n: number) {

reproduce,
iPhone SE(iOS 10.3.1)
vue-router history mode
safari's incognito mode
the safari's normal mode is fine,only reproduce on incognito mode

On a related note, I get the same problem when saving the web-app to home page on iOS. Removing history-mode makes it run correctly, while having history mode on causes it to reload every time url changes.

For example, when trying to add to home page while opening http://example.com/page1 on iPhone
The URL is http://example.com/index.html.
I am troubled not knowing where the cause is.
Is this behavior related to this issue?

I couldn't reproduce it on iPhone 6 with iOS 10. Please open a new issue with a reproduction if you manage to reproduce this

This issue is back. iOS 14.0.1
I managed to stop it happening all the time but no reason I can fathom. Visit on iOS device, must be actual device. https://alpha.nodenogg.in
Tap my lists
Give you self a name (lowercase no spaces)
Tap store
Create a microcosm (lowercase no spaces)
Two Join
Now without creating a node
Tap to another view such as cards
Screen will hit refresh problem.
Now a workaround seems to be to create a node on that view first.
Doesnt effect Android or desktop
Latest Vue 2


Add test for Profile and Missions components

Tasks completed

Add test for Mission. Missions and MissionProfile component.
Add test for Profile component


Fixing a missing bracket

I tried to use<EMAIL_ADDRESS>but it doesn't build right now because of the typo. So I fixed it.

hey @iezepov we'll take a look and get this in! Thanks for your contribution :)


feat: add FileSelector

Adds the FileSelector component. Also fixes test for mui-event-tracker as I was running into the same issue

There's a lot of stuff commented out. Are those TODO's?

There's a lot of stuff commented out. Are those TODO's?

Yeah, they are. I'm planning on addressing it with the legacy styles. It's not features that are required for the simple use cases.


feat!: support Vite 6

This PR rework the data collection and rpc interface to work the best with Vite 6's env API https://github.com/vitejs/vite/pull/16471
This PR will make the plugin not compatible with Vite below v6.

Currently blocked by https://github.com/vitejs/vite/issues/18781, after resolved this will be released as lastest


Allow for annotations on companion objects of @Lenses-annotated case classes.

Does what it says, I'd hope.
Fixes #523.

Thanks @hrhino 👍


feat: mount the .certs directory into the test container image

With this PR e2e tests now work on my M1 mac!


Add inquiry FAQ link to conversation inbox

Just a link

Good catch, thanks! Looks like we decided to remove the other link?

Yes. Guillaume added that change to this ticket


add SPDX-License-Identifier to Dockerfile/README/Makefile/config/script

retest it please

retest this please


Address support

This gets phone numbers and email addresses, but leaves off physical addresses.  Would be nice to have.

Hello @mandrachek
i have implemented support for postal addresses but i need to test it on different devices. So far it is working great on my Nexus 5X. Unfortunately i don't have access other devices till 08 January and i don't want to release without testing the functionality on Samsung and HTC phones. However, if you want to test by yourself, you can clone the dev branch and use the dev version.
Thank you and regards :)


Upgrade Strimzi Kafka Version to 0.24.0-kafka-2.7.0

I prefer use the latest released version instead of the latest available image as in https://github.com/quarkus-qe/quarkus-openshift-test-suite/commit/5d9eadd0dbf72b5cbf531bb08f26d6bbbb7c9c74#diff-6a696a4af18185fc9e6fb77baf89383331c12b8f0edeba9697379923b258c864R50.

Depends on https://github.com/quarkus-qe/quarkus-test-framework/pull/147


Some sample code would be great

I'm new to GraphQL and just evaluating it. This seems like it would be necessary to be able to get it to work with dynamodb. Any chance of adding some simple samples of how to use this.

@helloniklas I added a complete example on how to get started, let me know if you have any problems getting it to work.

@helloniklas did @codepreneur 's example help?


Add a Continuous Integration GitHub Action.

Like the existing Travis CI, this will run make test on each new PR. The main tock repository has already switched from Travis to GitHub Actions for better performance and support. Travis currently takes over 30 minutes for each PR, and based on my testing this action usually takes 13-16 minutes. This will help PR authors recognize and fix issues faster.
I plan to run this action concurrently with Travis for a short time to verify it is working, switch bors over to checking this action rather than Travis, then remove the Travis config.

bors r+
🎉


Add more problems in General Questions

Adding solutions of some pattern printing problems asked in ### INTERVIEWS.
Like:

123
12345
1234567
123456789
I request you to assign it to me.

Sure @nileshkr9919, but please mention in which language you want to submit the solution in ?

I think it will be better in C, as will be easier for beginners to understand.

You can go ahead !

Hello, I would like to add some star pattern questions in C.

Some pattern based problems are already present here. You can have a look at them.
If you want to contribute, you can create a new issue stating the problem that you want to solve.

Implement Merge Sort i.e. standard implementation keeping the sorting algorithm as in-place.
May, I go ahead with this in CPP??

Sure @ritikaranj, you can go ahead. Make sure to place the problem under Sorting Algorithms folder.

Ok , Thank you.

I have a doubt about this program.
When should we have to add code, after getting groups assigned or now?

The contribution period starts officially on 10th July. If you require more information regarding the program, please contact the Program Managers in Slack.

Okk , Thank you


Will it support Dolly-V2?

I hope that langchain can support dolly-v2 which is generated by Databricks employees and released under a permissive license (CC-BY-SA).

It's already supported since dolly is on huggingface hub: https://huggingface.co/databricks/dolly-v2-3b
You simply need to define some local llm and pass it to the question answering chain.
from transformers import LlamaTokenizer, LlamaForCausalLM, GenerationConfig, pipeline
from langchain.llms import HuggingFacePipeline

from transformers import AutoModelForCausalLM, AutoTokenizer
tokenizer = AutoTokenizer.from_pretrained("databricks/dolly-v2-3b", padding_side="left")
base_model = AutoModelForCausalLM.from_pretrained("databricks/dolly-v2-3b", device_map="auto")

pipe = pipeline(
    "text-generation",
    model=base_model,
    tokenizer=tokenizer,
    max_length=1024,
    temperature=0.6,
    pad_token_id=tokenizer.eos_token_id,
    top_p=0.95,
    repetition_penalty=1.2
)
local_llm = HuggingFacePipeline(pipeline=pipe)

from langchain.chains import RetrievalQA
qa = RetrievalQA.from_chain_type(llm=local_llm, chain_type="stuff", retriever=retriever)

@DanqingZ  Thank you very much, I tried and it worked.

Hello all, Do we have any possibility to use an inference service (like an url) instead of loading model and then use llm.

It's already supported since dolly is on huggingface hub: https://huggingface.co/databricks/dolly-v2-3b You simply need to define some local llm and pass it to the question answering chain.
from transformers import LlamaTokenizer, LlamaForCausalLM, GenerationConfig, pipeline
from langchain.llms import HuggingFacePipeline

from transformers import AutoModelForCausalLM, AutoTokenizer
tokenizer = AutoTokenizer.from_pretrained("databricks/dolly-v2-3b", padding_side="left")
base_model = AutoModelForCausalLM.from_pretrained("databricks/dolly-v2-3b", device_map="auto")

pipe = pipeline(
    "text-generation",
    model=base_model,
    tokenizer=tokenizer,
    max_length=1024,
    temperature=0.6,
    pad_token_id=tokenizer.eos_token_id,
    top_p=0.95,
    repetition_penalty=1.2
)
local_llm = HuggingFacePipeline(pipeline=pipe)

from langchain.chains import RetrievalQA
qa = RetrievalQA.from_chain_type(llm=local_llm, chain_type="stuff", retriever=retriever)

How can I pass my own set of documents?


fixing #1: comment threads are truncated if nesting level > 2

Paging @josephbergdoll -- can you test this guy out?

@salaros — it seems like a lot of comments got jumbled in your commit and thus the diff doesn't provide anything when trying to glance the changes you made. fix and push?

I pasted in your modifications from get_comments, and it looks like is_child is not saving any data to the comments_tree array.

Hi @salaros – I parsed through your changes and added it to the following PR – https://github.com/jarednova/timber/pull/717
this PR can be closed.

This is closed by #717


batch inference and detect passing line or crossing area

Hi @JuanFKurucz  @dekked ,
1 - Is it possible to make the inference faster to send more image same time ? GPU util ~%40 generally.
2 - What is the best/fastest way to detect vehicle crossing the line or are to count vehicles ? If you show me /direct me a soluton very appreciated.
yolov4 for cars in 2080 RTX ti has :

       Preprocess : 0.000627
  Model Inference : 0.016399

   max and argmax : 0.001625
              nms : 0.000424

Post processing total : 0.002049

Best

Hi @MyraBaba!
I'll address your first question. As the tracker is CPU-bound, if you have to process many hours of videos and want to maximize utilization of the GPU, your best bet is to run do it in two steps:

Run the detector and store the detections in a format like CSV. This should max out your GPU.
Load the detections from the CSV file and run Norfair.

@dekked
thanks . which algorithm is fast / accurate ?
is there any way to detect a box is crossed a line information with direction data ?

Hi @dekked ,
I export xywh to csv and try to test as in normal detection happens.
final frame has not obj.id and not aged any idea ?
video resolution is 2560x1440
`import cv2
import numpy as np
import re
import norfair
from norfair import Detection, Paths, Tracker, Video
import argparse
from typing import List, Optional, Union
DISTANCE_THRESHOLD_BBOX: float = 3.33
DISTANCE_THRESHOLD_CENTROID: int = 30
MAX_DISTANCE: int = 10000

def center(points):
    return [np.mean(np.array(points), axis=0)]
def euclidean_distance(detection, tracked_object):
    return np.linalg.norm(detection.points - tracked_object.estimate)

def iou(detection, tracked_object):
    # Detection points will be box A
    # Tracked objects point will be box B.

    box_a = np.concatenate([detection.points[0], detection.points[1]])
    box_b = np.concatenate([tracked_object.estimate[0], tracked_object.estimate[1]])

    x_a = max(box_a[0], box_b[0])
    y_a = max(box_a[1], box_b[1])
    x_b = min(box_a[2], box_b[2])
    y_b = min(box_a[3], box_b[3])

    # Compute the area of intersection rectangle
    inter_area = max(0, x_b - x_a + 1) * max(0, y_b - y_a + 1)

    # Compute the area of both the prediction and tracker
    # rectangles
    box_a_area = (box_a[2] - box_a[0] + 1) * (box_a[3] - box_a[1] + 1)
    box_b_area = (box_b[2] - box_b[0] + 1) * (box_b[3] - box_b[1] + 1)

    # Compute the intersection over union by taking the intersection
    # area and dividing it by the sum of prediction + tracker
    # areas - the interesection area
    iou = inter_area / float(box_a_area + box_b_area - inter_area)

    # Since 0 <= IoU <= 1, we define 1/IoU as a distance.
    # Distance values will be in [1, inf)
    return 1 / iou if iou else (MAX_DISTANCE)

parser = argparse.ArgumentParser(description="Track objects in a video.")

# parser.add_argument("files", type=str, nargs="+", help="Video files to process")
# parser.add_argument("--detector_path", type=str, default="yolov5s.pt", help="YOLOv5 model path")
parser.add_argument("--img_size", type=int, default="720", help="YOLOv5 inference size (pixels)")
parser.add_argument("--conf_thres", type=float, default="0.25", help="YOLOv5 object confidence threshold")
parser.add_argument("--iou_thresh", type=float, default="0.45", help="YOLOv5 IOU threshold for NMS")
parser.add_argument("--classes", nargs="+", type=int, help="Filter by class: --classes 0, or --classes 0 2 3")
parser.add_argument("--device", type=str, default=None, help="Inference device: 'cpu' or 'cuda'")
parser.add_argument("--track_points", type=str, default="centroid", help="Track points: 'centroid' or 'bbox'")
args = parser.parse_args()

cap = cv2.VideoCapture('/Users/xxx/Downloads/38.mp4')
fo = open("/Users/xxx/Downloads/DetectinOuts/frame_boxes.csv", "r+")
max_distance_between_points = 30

# Check if camera opened successfully
if (cap.isOpened()== False):
  print("Error opening video stream or file")

while(cap.isOpened()):
    line = fo.readline()
    rects = re.findall('\(.*?\)', line)
    norfair_detections: List[Detection] = []
    ret, frame = cap.read()

    distance_function = iou if args.track_points == "bbox" else euclidean_distance
    distance_threshold = (
        DISTANCE_THRESHOLD_BBOX
        if args.track_points == "bbox"
        else DISTANCE_THRESHOLD_CENTROID
    )

    tracker = Tracker(
        distance_function=distance_function,
        distance_threshold=distance_threshold,
    )
    paths_drawer = Paths(center, attenuation=0.01)

    for box in rects:
        box = re.sub('[()]', '', box).split(':')
        c1, c2 = ( (int(box[0].split(',')[0]) , int(box[0].split(',')[1]))  , (int(box[1].split(',')[0]) , int(box[1].split(',')[1])) )
        track_points: str = "centroid"
        centroid = np.array(
            [
                (float(c1[0]) + float(c2[0])) / 2,
                (float(c1[1]) + float(c2[1])) / 2
            ]
        )
        norfair_detections.append(
            Detection(points=centroid, scores=np.array([float(0.89)]) , label='car')
        )

        # cv2.rectangle(frame, c1, c2, (255,33,244), thickness=4, lineType=cv2.LINE_AA)

    tracked_objects = tracker.update(detections=norfair_detections)
    if args.track_points == "centroid":
        norfair.draw_points(frame, norfair_detections)
        norfair.draw_tracked_objects(frame, tracked_objects)
    elif args.track_points == "bbox":
        norfair.draw_boxes(frame, norfair_detections, line_width=4)
        norfair.draw_tracked_boxes(frame, tracked_objects)

    cv2.imshow('Frame',frame)
    cv2.waitKey(1)

`
frame_boxes.csv.zip

Hi @MyraBaba I think the issue is that you are re-instantiating the tracker for each frame. Try moving the tracker = Tracker(...) outside the while-loop


HIVE-24867: Create iceberg-handler module in Hive

da18e97 - this commit contains the below items. This should not require any reviews.

creation of a new iceberg-handler module in Hive (with only skeleton pom)
simple copy paste from the iceberg-mr: https://github.com/apache/iceberg/tree/master/mr/src into hive/iceberg-handler/src

efaeb74 - this commit contains the changes necessary to make the new iceberg-handler compile and pass its tests. These are the changes that are mostly non-trivial and should be reviewed.

Copy-pasted test classes from other iceberg modules which are used by our tests as well:
-- DeleteReadTests, FileHelpers, GenericAppenderHelper, RandomGenericData from iceberg-data
-- TestHiveMetastore from iceberg-hive-metastore
-- RandomUtil, AssertHelper, TestHelpers from iceberg-api
The above are simple copy-pastes and should not be reviewed, except for:
-- TestHelpers#assertSameSchemaList commented out - 0.11.0 Schema does not contain schemaId() method
-- TestHelpers#containsNaN commented out - 0.11.0 does not contain this method yet
Hive2 inspectors and their corresponding test classes replaced with their Hive3 counterparts where applicable. These should not need to be reviewed as they are copy pastes:
-- IcebergDateObjectInspector replaced with IcebergDateObjectInspectorHive3
-- IcebergTimestampObjectInspector replaced with IcebergTimestampObjectInspectorHive3
-- IcebergTimestampWithZoneObjectInspector replaced with IcebergTimestampWithZoneObjectInspectorHive3
HiveIcebergSerDe and TestHiveIcebergSerDe: added new third parameter to the initialize() method, since the API changed in AbstractSerDe.initialize()
TestMetastore change I: used the cdpd-master downstream version as the base for this file, which contains the changes related to hiveWarehouseDir/hiveExternalWarehouseDir
TestMetastore change II: HMSHandler is not an inner class of HiveMetaStore anymore, but its own top-level class
TestMetastore change III:  Uses TestTxnDbUtils to initialize the backend DB (see start() method). Removed the setupMetastoreDB as it's not necessary anymore.
TestIcebergObjectInspector: removed the Hive2 cases for the date, timestamp and timestamptz cases
TestHiveShell: added a few new test config settings
TestHiveIcebergStorageHandlerNoScan: temporarily ignored testIcebergAndHmsTableProperties test (see comment)
TestHiveIcebergStorageHandlerNoScan: temporarily ignored testCreateTableWithColumnSpecificationHierarchy test (see comment)
Pom.xml dependencies should reflect exactly the dependencies declared in iceberg-mr build.gradle, except for:
-- Included orc as a test dependency because I got runtime errors otherwise
-- Did not include jackson:annotations because I got runtime errors otherwise
-- Hive-exec has been included without the core classifier, because I got runtime errors otherwise
-- Included metastore-server test jar, so that we access to TestTxnDbUtils
Moved 6 test methods from TestHiveIcebergStorageHandlerLocalScan to TestHiveIcebergStorageHandlerWithEngine, because those tests now trigger engine executions
TestHiveIcebergStorageHandlerTimezone: changed the engine from spark to tez, because these tests now also trigger engine executions
HiveIcebergFilterFactory#microsFromTimestamp: using the cdpd-master downstream version here, same for the corresponding test inTestHiveIcebergFilterFactory
Removed mr from the engine parameters in TestHiveIcebergStorageHandlerWithEngine
Added iceberg-handler into packaging module

e78b7e2 - Changes iceberg-handler pom so that we build a shaded jar just like for druid-handler and kafka-handler (+ some small review changes)

Checked the relevant files, left a few comments

Can we enable the Checkstyle and other code checks for the tests the same way as it is done in the Iceberg repo?

Can we enable the Checkstyle and other code checks for the tests the same way as it is done in the Iceberg repo?
Sure, will do

@jcamachor Would you like to review as well?


Labcoats have suit storage

Description

Screenshots

Additional context

or basically any over-jumpsuit clothing

Iirc they did on 13? Like a coat pocket. But yes we need to make sure it can’t hold a gas tank.

id like to restrict large tank storage to hardsuits and softsuits but not sure where thats specified to whitelist them

this is a much bigger issue with #9330 merged

To add context, suit storage is aka exosuit slot is hard coded which is bad. The clothing should add the slot and have its constraints defined with the storage component like everything else - that way you can use blacklists or storage limits.

SS13 had two different storages for clothing. Some had suit storage which unlocked the large item slot (this is what hard suits have). And some were containers with one or two slots that fit small items (like labcoats. In CM helmets were also containers for smokes and nutrient bars.).