[FEATURE] Blogs page

Detailed description
Add a Blog section to the site
Can be made using React

Add dummy data after completing this page, we will add backend later on.

Context
Blogs plays important role in learning
Possible implementation
Example: 👇

Add blog nav-item under navbar
You can make this type UI under the blog section.

website: https://www.cryptoverseweb3.com/

Additional information
If you need any help or have questions and doubts, please ask below: 👇
Code of Conduct

[X] I agree to follow this project's Code of Conduct

Interested. Please assign.

@kabir0x23 I have checked the repo out. You may assign the task to me. I'll start working on the issue

Okay, I have assigned this issue to you.

@kabir0x23 The content of blog component will largely depend on the data your endpoint will be returning. What kind of dummy data do you want me to use ? I can't find any. It would be great if you could provide me some mock api of the same. For now I'll create a generic component similar to the one you have in the image above.

@kabir0x23 The content of blog component will largely depend on the data your endpoint will be returning. What kind of dummy data do you want me to use ? I can't find any. It would be great if you could provide me some mock api of the same. For now I'll create a generic component similar to the one you have in the image above.

Hey @Manthan-Kuber ,
You can use lorem ipsum text for filling the text and for the image use any.
After the frontend part got completed, we will connect it to the backend.
Thanks.


Improvements required to be able to include middleware in Linux distributions repositories

Hello, I am writing this ticket in English language cause some fellow Fedora developers will intervene in this discussion.
I would like to package CIE Middleware for Fedora, in order to redistribute it in official Fedora repositories. The current status of the software unables me to do so, so hereby I would like to ask you some modifications that I am pretty sure will also lead CIE Middleware being shipped on many other Linux distributions:

concerning C/C++ code, please switch to using CMake build system completely: some parts of the software need to be built through Eclipse, I.E. cie-pkcs11. The obligation of using a graphical application to build makes no sense, in particular because all Linux distributions make use of headless build systems.
Please switch to using Maven for building the Java code. This makes it much clearer which Java dependencies the project uses and makes it possible to easily build the Java code from the command line. See this Maven pom.xml for CIE middleware from @mkoncek. Gradle is not a preferred build system in Fedora due to various problems with its distribution.
concerning libraries dependencies: libraries cannot be shipped in form of binary or bytecode (I.E. list at footnote [A]), for the following reasons:

Fedora has to distribute a copy of the tarball/git archive, and cannot distribute anything that is non-free;
the CIE Middleware must try to link to link to pre-installed packages with CMake find_package or use the even better alternative CMake FetchContent. (A discussion about the latter between Fedora developers and Estonia ID card developers is available here)
we want to make CIE Middleware available for all CPU architectures supported by Fedora (x86_64, aarch64, ppc64le, etc.);
we handle security patches of dependencies which requires linking dynamically to packages.

.d files and .mk files (I.E. list at footnote [B]) should not be in a source code repository because are generated automatically during the build process.

The type of support we can offer as Fedora community:

During transition: not much, it requires too much knowledge of how the project is/will work. Sometimes the community can volunteer to review and offer advice or reference;
Post transition: we offer patches to keep up with dependency changes, (optional) integration testing environment (packit+copr)

Best regards
[A]:
user@machine:~/cie-middleware-linux$ fd -e jar
CIEID/lib/Notify-3.7.jar
CIEID/lib/ObjectPool-2.9.jar
CIEID/lib/ShellExecutor-1.1.jar
CIEID/lib/SystemTray-3.17.jar
CIEID/lib/TweenEngine-8.3.jar
CIEID/lib/commons-beanutils-1.9.2.jar
CIEID/lib/commons-collections-3.2.1.jar
CIEID/lib/commons-io-1.3.1.jar
CIEID/lib/commons-logging-1.1.1.jar
CIEID/lib/core-1.0.0.jar
CIEID/lib/ghost4j-1.0.1.jar
CIEID/lib/gson-2.6.2.jar
CIEID/lib/itext-2.1.7.jar
CIEID/lib/javassist-3.23.0-GA.jar
CIEID/lib/jgoodies-forms-1.8.0-sources.jar
CIEID/lib/jgoodies-forms-1.8.0.jar
CIEID/lib/jna-4.1.0.jar
CIEID/lib/jna-4.5.2.jar
CIEID/lib/jna-platform-4.5.2.jar
CIEID/lib/log4j-1.2.17.jar
CIEID/lib/slf4j-api-1.7.25.jar
CIEID/lib/twinkle-1.0.0.jar
CIEID/lib/xmlgraphics-commons-1.4.jar

user@machine:~/cie-middleware-linux$ fd -e a
cie-pkcs11/libcryptopp.a
cie_sign_sdk/Dependencies/bzip2/libbz2.a
cie_sign_sdk/Dependencies/fontconfig/libfontconfig.a
cie_sign_sdk/Dependencies/freetype/libfreetype.a
cie_sign_sdk/Dependencies/libcurl/libcurl.a
cie_sign_sdk/Dependencies/libiconv/libcharset.a
cie_sign_sdk/Dependencies/libiconv/libiconv.a
cie_sign_sdk/Dependencies/libpng/libpng16.a
cie_sign_sdk/Dependencies/libxml2/libxml2.a
cie_sign_sdk/Dependencies/openssl/libcrypto.a
cie_sign_sdk/Dependencies/openssl/libssl.a
cie_sign_sdk/Dependencies/podofo/libpodofo.a
cie_sign_sdk/Dependencies/zlib/libz.a

[B]:
user@machine:~/cie-middleware-linux$ fd -e d
TestCIE/Debug/TestCIE.d
TestCIE/Debug/UUCByteArray.d

user@machine:~/cie-middleware-linux$ fd -e mk
TestCIE/Debug/objects.mk
TestCIE/Debug/sources.mk
TestCIE/Debug/subdir.mk

A post scriptum which I forgot to add to the previous message: on long term you may really want to switch to Qt libraries in order to just have a single middleware stack for Linux, Mac OS and Windows, like the Republic of Estonia does with its DigiDoc client that I am maintaining on Fedora


Update Adyen_Payment.csv

Description

Tested scenarios

Fixed issue:

Hi @jeros-smile ,
Thank you for the PR, we are going to merge it.
Regards,
Alessio
Adyen


GD-99: WIP: Some finding installing 0.9.6-rc

The used Godot version: 3.3.2.stable
OS including version: Mac
Describe the bug

When download and install through the AssetLib I see some (6) - drivers/unix/file_access_unix.cpp:279 - Condition "!p_src" is true.

Steps to Reproduce
Minimal reproduction project:

@clemens-tolboom can you please add the full log of starting the plugin?

After a lot of trying by reducing the files during installation, I found out that the installation via AssetLib has problems with zero byte length files.


Further modularization of this project

This project is by far the most popular client-side GraphQL effort in Dart, but it seems to always be in a bad spot maintenance-wise. Some alternative/complementary projects have popped up recently.
I've been working on a GraphQL parser, which is already used by this project, and an AST-based Link layer which is already used by Artemis (@comigor) and gql_client (@smkhalsa).
To take the pressure off of this project and to optimize community's effort in various directions, I'd like to propose a further modularization of this project.
Here's a proposed list of modules:

Link module (could use package:gql_link for that)
Cache/Store module (@smkhalsa has done some interesting work on this)
Client module (next iteration of package:graphql)
Flutter module (next iteration of package:graphql_flutter)
Codegen module (me, @comigor and @micimize have worked in this direction)

Modularization would enable independent iterations and breaking changes without necessarily braking the client itself. Also, multiple other projects may benefit from the Link module and Cache/Store module.

I am open to that. We currently lack active maintainers, time for me has become very difficult to come by and I took a new job and we are not using flutter, which makes it difficult to follow to up on flutter and this project in general for me. I have a few issues I have not found time to finish yet and I can not guarantee I will be able to work on them to the conclusion. On top of that, we also need more contributors and anyone interested to help in maintaining this wonderful project.
I am pining this issue.

I'm also up to having the good Artemis bits (on multiple file processing (with future work on middlewares), fragments globbing, custom scalars and parsers) integrated with this project (which lacks query typing for now).

FYI, I've labeled the link-related issues https://github.com/zino-app/graphql-flutter/labels/link

Just as @mainawycliffe said, not working with the two core technologies graphql and flutter leads to a huge problem in terms of effort to context-switch for this project. Now that I am back working with @HofmannZ , we will push more effort into this project since we will work with the App that started this library.
I would definitely like to start again, but I need to start from scratch and read the documentation of graphql, apollo and all the cool tech that is making this library work.
@klavs If you have to explain what needs to be done to achieve this modularization, I would love to help.

@eusdima, It's really all about setting good API boundaries.
With the Link Module it's quite easy. It receives a GraphQL document and eventually returns the result. I've already implemented an AST-based Link API in gql_link. As you can see in the https://github.com/zino-app/graphql-flutter/pull/564, it removes any need for the Client itself to be concerned about the transport. Of course, all those issues will need to be solved externally, but in a more focused manner.
I've created a tracking-issue https://github.com/gql-dart/gql/issues/57 to list what's missing if graphql_flutter switches today. Those should be implemented while still looking at the existing issues (https://github.com/zino-app/graphql-flutter/labels/link) related to Link layer.
For the Cache Module, I think @micimize has the best mental model of how the API should be organized. Then that module should be implemented to it's own package.
When these APIs are well defined, at least a naive implementation of a Cache should be implemented which does the job. Meanwhile, the community has a stable Cache API interface to build their own Cache for their needs, relieving this project from the need to implement every fancy Cache possible.
To summarize:

[ ] Define a Link API (already done in package:gql_link)
[ ] Remove concrete Links from package:graphql (done in https://github.com/zino-app/graphql-flutter/pull/564)
[ ] Use only AST-based operations/documents in package:graphql (done in https://github.com/zino-app/graphql-flutter/pull/565)
[ ] Implement all existing Links as separate packages (tracked by https://github.com/gql-dart/gql/issues/57)
[ ] Fix known issues with Links (tracked at https://github.com/zino-app/graphql-flutter/labels/link)
[ ] Write documentation for every Link
[ ] Define a Cache API (probably, with AST-based operations)
[ ] Remove concrete Cache implementation from package:graphql
[ ] Move the existing cache implementation to a separate package(s)
[ ] Implement a normalized cache
[ ] Support optimistic updates in the normalized cache
[ ] Fix known issues with Cache (tracked at https://github.com/zino-app/graphql-flutter/labels/cache)
[ ] Write documentation for every Cache

This might not be a totally acurate TODO list.

Hey I have written a private graphQL cache. I haven't tested this one in months but I recall it wasn't saving often enough and the entire cache was a .txt file with a format that seemed like it wouldn't work. What's the status of cache packages?
The cache I wrote is basically a hashmap of query+variables => response. Each endpoint+auth has its own directory, and each cache entry is a file whose name is a hash of the query+variables. The contents of the file also includes a timestamp of the response and the full query and variables used for easier debugging. GraphQL responses are written to a temporary file when they are received, checksummed for errors, then renamed, which is an atomic operation in almost every OS. It is maybe a bit overkill but it is mission critical for us.
I want to have your client use my cache, but I'm not sure it's as simple as implementing the interface. Do you think it's a good fit?

@jifalops take a look at the v4 cache changes. I need to rework the HiveStore api, but in general the caching layer should be much more stable, and implementing your own store should be easier.
That said, a lot of the code in graphql_flutter revolves around the normalized cache and optimistic responses, and in v4 I simplified the library to focus more on those things (i.e. there is now a single GraphQLCache). You could still implement it and override all the NormalizingDataProxy methods to work with your caching solution and not be normalization dependent, but you might be better off composing lower-level gql libraries into a custom client.
Alternatively, if the point of your filesystem cache is a reliability layer, maybe it could all be implemented as gql_links, and you could also have multiple clients. It's a pretty strange setup that I don't fully grok so you'll probably want to do some digging into our codebase and gql's libraries to figure out the best way to integrate/if it makes sense at all

The most impactful thing here was the switch to the gql_link system. I toyed with some deeper modularization ideas, but in the end think it is a better DX for graphql/graphql_flutter to be treatable as a kitchen sink. That doesn't mean it wouldn't be better for its composite parts to be separate packages, but I just don't think it would be useful/feasible to, say, remove the concrete implementation of the core cache from the library.
The cache could be a standalone library that is re-exported like what I've done with links, but I'm not sure how much sense that makes either – as I discovered while trying to outsource store to ferry_store, it is hard to decouple the manager and the cache implementations.
Anyways, the tl;dr here is that I'm descoping further modularization than what's already been done thus far on v4 with links.


load balancer scheduling host drop-down not functioning

Rancher Version: v1.2
Steps to Reproduce:

Add labels to host
Add load balancer
Go to scheduling and click on drop-down for host

Results: Nothing in there
Expected:  Should reflect host labels.

This seems to work in 1.4.1, tested when creating a new load balancer:


Expo support

Is your feature request related to a problem? Please describe.
Expo is one of the most popular frameworks for developing React native applications. I'd expect we'll get a lot of requests about this.
Describe the solution you'd like
Is it possible to suppor expo without needing to eject? If so, how much work would that be?
Describe alternatives you've considered
If this isn't easily possible, can we add a "usage with Expo" guide?
Additional context
Currently, when I try to use CardField within an expo project I get “Invariant Violation: requireNativeComponent: "CardField" was not found in the UIManager.” which is related to the webview usage I beleive.

Is it possible to remove @expo/config-plugins as a hard dependency? It installs a lot of unnecessary packages in non-expo apps...

hey @aganov, without the config-plugins dependency, it'd be difficult to give first class support @stripe/stripe-react-native for both the managed and bare workflows. The package itself is pretty minimal (about 0.6% of the size of the stripe-react-native package), and this is the breakdown of that bundle. Are there particular dependencies of config-plugins that concern you and you don't want to have in your project?
If this is an absolute blocker for you right now, you could get rid of the transitive dependency in a post-install script
As for why it's included, you can read about what exactly config plugins do here

@thorsten-stripe I noticed on the calendar it showed a public beta for May 11th and a potential Expo Go patch release May 17th. I'm not seeing those on there anymore. Is there still a timeline laid out for when this project would be available for use in managed workflows either in a beta or official release?

Expo managed workflow support has landed as of this morning 😄
You will need to use the most recent versions of Expo Go from the stores (or locally on simulators installed with expo install:client:[ios|android]).
You can refer to these docs to get more specifics on what's supported in Expo Go, and what you'll need to build with EAS Build for.
Closing this issue since Expo support has officially landed 🎉


nixos/scion: fix nixosTest dates and validity period for TRCs

The validity period for TRCs cannot be set to an rfc3339 date, only an offset from the current system time, which if set to 3650d rather than 36500d will still give us quite a long time before it becomes invalid, which is acceptable for the time being.

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

https://github.com/scionproto/scion/issues/4440 could make this easier to fix as well


Introduce a use case-driven section on declaring repositories in the user guide

Context
This is the initial draft of a use case-driven section on declaring repositories for the user guide.

Reference material was moved into the glossary.
Reference material was trimmed down and polished.
New content was added based on the proposed outline.

The use cases I put out are the ones which I believe users encounter a lot. We might want to work on the titles and the use case a bit more. Happy to hear your feedback.
The reference material uses a table. Should we turn this into a list?

I think we've really got to split it out from the big Dependency Management chapter so users can find it.

Let's do it in a separate PR. Please also have a look at my other comment.

Most everything else is a nitpick.

Addressed your comment. Could you have another look?

There are note, caution, warning, tip, and important according to http://www.sagehill.net/docbookxsl/CustomAdmonitions.html.

I believe I tried them but they didn't render properly with our Docbook implementation.

Regarding admonitions, I can help with styling if you want to use other types


Microsoft auth guide

Status
Ready
Description
a generic guide for microsoft auth to explain about the 2 auth options
every microsoft integration doc should refer to it once published

@kirbles19 please review

@kirbles19 preview available here: https://deploy-preview-190--demisto-content-docs.netlify.app/docs/reference/articles/microsoft-integrations---authentication

@Itay4 Done.

@glicht can you please approve?


API Version defined in configuration is not being used

Hello
I just setup laravel-generator with api_prefix = 'api' and api_version = 'v1'. But when I run php artisan infyom:api_scaffold Test the TestAPIController is placed directly in  app/Http/Controllers/API/ without separate namespace for each version, and also the route registed by the package is not has v1 prefix too.
I this normal or it's has something to do with my configuration?
Thank you

The normal procedure is, you have to set api_prefix and api_version while setting up generator. so it will take values from there and will create a group in routes.php in which api_routes.php will be included, something like following:
Route::group(['prefix' => 'api', 'namespace' => 'API'], function () {
    Route::group(['prefix' => 'v1'], function () {
        require config('infyom.laravel_generator.path.api_routes');
    });
});

so then those values will be always used as your prefix and namespace. It does not load dynamically when you call APIs or you generate files.

I use the latest version of plugin with Laravel 5.4, so there is no routes.php but routes/api.php. When I generate an API via console the routes automatically added to api.php then I'll have to move it in to a route group manually?

On Sep 24, 2016, at 7:38 PM, Mitul Golakya<EMAIL_ADDRESS>wrote:
The normal procedure is, you have to set api_prefix and api_version while setting up generator. so it will take values from there and will create a group in routes.php in which api_routes.php will be included, something like following:
Route::group(['prefix' => 'api', 'namespace' => 'API'], function () {
Route::group(['prefix' => 'v1'], function () {
require config('infyom.laravel_generator.path.api_routes');
});
});
so then those values will be always used as your prefix and namespace. It does not load dynamically when you call APIs or you generate files.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/InfyOmLabs/laravel-generator/issues/311#issuecomment-249362745, or mute the thread https://github.com/notifications/unsubscribe-auth/AB9dJcsswkFsDFF-6MoFOQS1i-p_D986ks5qtRm9gaJpZM4KCDd5.

@ton212 Laravel 5.4 ? Or 5.3 ? Because 5.4 is not yet supported.

My mistake, I mean 5.3

On Sep 24, 2016, at 7:46 PM, Mitul Golakya<EMAIL_ADDRESS>wrote:
@ton212 https://github.com/ton212 Laravel 5.4 ? Or 5.3 ? Because 5.4 is not yet supported.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/InfyOmLabs/laravel-generator/issues/311#issuecomment-249363031, or mute the thread https://github.com/notifications/unsubscribe-auth/AB9dJUU6nRbFcKonV9j63oqMZcmlIjrdks5qtRuKgaJpZM4KCDd5.

@ton212 you can refer docs for 5.3 installation here: http://labs.infyom.com/laravelgenerator/docs/5.3/installation
There are some additional edits required to work with it 5.3.

I did all the steps, but the route group is never generates by the package, and API version too.

On Sep 24, 2016, at 7:48 PM, Mitul Golakya<EMAIL_ADDRESS>wrote:
@ton212 https://github.com/ton212 you can refer docs for 5.3 installation here: http://labs.infyom.com/laravelgenerator/docs/5.3/installation http://labs.infyom.com/laravelgenerator/docs/5.3/installation
There are some additional edits required to work with it 5.3.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/InfyOmLabs/laravel-generator/issues/311#issuecomment-249363142, or mute the thread https://github.com/notifications/unsubscribe-auth/AB9dJREVRc0OamVlnwKmkODuHnmoTRwdks5qtRwRgaJpZM4KCDd5.

In Laravel 5.3 api.php file already exist and already included by laravel. so its not generated and its not needed.

your generated routes will be automatically added to routes/api.php.

That right for the api/ prefix but the API version defined in plugin configuration is not added to the route.
So I have to put it in manually right?

On Sep 24, 2016, at 7:54 PM, Mitul Golakya<EMAIL_ADDRESS>wrote:
In Laravel 5.3 api.php file already exist and already included by laravel. so its not generated and its not needed.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/InfyOmLabs/laravel-generator/issues/311#issuecomment-249363403, or mute the thread https://github.com/notifications/unsubscribe-auth/AB9dJcPDZ2sZlz7UiVEmfAWpf2nF1_eAks5qtR1ygaJpZM4KCDd5.

yes

OK then.
Will it be an update to this in the future release of the plugin? (Just curious for how API version in config file works in the plugin)
Best regards,
Tonsai Singhkangwan
On Sep 24, 2016, 7:56 PM +0700, Mitul Golakya<EMAIL_ADDRESS>wrote:

yes
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub (https://github.com/InfyOmLabs/laravel-generator/issues/311#issuecomment-249363479), or mute the thread (https://github.com/notifications/unsubscribe-auth/AB9dJXfXlXpUCZ8sJLNYnSd5Y31g0Hvsks5qtR3ogaJpZM4KCDd5).


revert PR #250

in preparation for v0.2.6 release
PR #250 includes a breaking change in the command line interface and prevents us from doing a patch release with new features like #254
NOTE: this is a PR against the branch v0.2.x. v0.2.x has been branched from main at commit 0879ce5fc846648c1d9b339ff98f345dbe6e9d78
PR #250 will NOT be reverted on the main branch

need to rebase v0.2.x after PR #263


Add icon

Can you export the file in IOM as an .rdef file?  Then create a proper .rdef file for haiku2048, adding the icon array to it?  See https://github.com/scottmc/BurnItNow/blob/master/burnitnow2/Source/BurnItNow.rdef
for an example of the .rdef file.

The link isn't working for me :/

Sorry about that.  I just deleted that old repo and right afterwards remembered that I linked to it here.
It's now at:
https://github.com/HaikuArchives/BurnItNow/blob/master/Source/BurnItNow.rdef

Where do I get the information for:
resource app_signature "application/x-vnd.haikuarchives.BurnItNow2";

resource app_flags B_EXCLUSIVE_LAUNCH;

resource app_version {
	major  = 0,
	middle = 1,
	minor  = 1,

	variety = B_APPV_DEVELOPMENT,
	internal = 0,

	short_info = "",
	long_info = ""
};

resource file_types message;

resource vector_icon array {

for the 2048 app?

I'd go with:
resource app_signature "application/x-vnd.twofx.haiku2048";
for app_version set them all to 0, we'll let txofx decide when he want to set a version for it.
the rest you can keep the same, cept for the vector_icon array which will be the .rdef you exported from IOM.

Sorry, finals had me pretty busy the past few days. Do I put the .rdef in the same artwork/ directory as the iom file?

yes, you can put the  i-o-m exported icon .rdef file into the artwork folder.  But you should copy the vector array data and paste it into the .rdef file in the main source directory in the section:
resource vector_icon array {
put array data here
}
Then edit the makefile to add the main rdef file to the rdefs section as such:
RDEFS = haiku2048.rdef

Thanks for the help - I didn't realize that "resource vector_icon array" wasn't supposed to be replaced by the name of the icon. Make works now :)

Not sure why this was still pending.  Going to merge it now.


New Guides (issue #480)

Motivation and Resolution
...
RPC version (if applicable)
...
Usage related changes

Change 1.
...

Development related changes

Change 1.
...

Checklist:

[ ] Performed a self-review of the code
[ ] Rebased to the last commit of the target branch (or merged it into my branch)
[ ] Linked the issues which this PR resolves
[ ] Documented the changes
[X] Updated the docs (www)
[ ] Updated the tests
[ ] All tests are passing

This PR should be without conflict, and with the new guides.
Somebody created a multicall page. I integrated it in the new guides.


HWKAPM-489 Move the three top level tabs as sub tabs under a single A…

…pplication Performance tab

@ammendonca  Hi Alex, would you be able to do a review on this change?
After the project rename and scope change, I thought it best for the three areas to be under a single main tab called Application Performance.
I've done the minimal changes to enable this to work in the UI (I believe) but this leaves the code in a slightly misleading structure - but thought this could be handled as a separate issue, as I would like to get a release out this week.
In terms of the restructure, I will create a separate jira, and would appreciate your advice on how best to reorganise the code.

LGTM

Thanks Alexandre!


csv=true makes report return text/csv MODEUR-60

I mentioned this before: this makes no sense to me, conceptually. With this change we will need to change the interface again when we address MODEUR-54 MODEUR-53 and MODEUR-50 as the CSV format will diverge from JSON and will no longer contain the same data.

I mentioned this before: this makes no sense to me, conceptually. With this change we will need to change the interface again when we address MODEUR-54 MODEUR-53 and MODEUR-50 as the CSV format will diverge from JSON and will no longer contain the same data.

No. The CSV format is schema less at the moment. If we decide that this service changes its JSON response, then we change the JSON response. JSON !=  CSV.

@adamdickmeiss I expanded on why I think this is a bad idea on MODEUR-60. But I don't want this issues to hold Mike's work hostage so I'll approve -- let's move on.


Add appveyor config back for building windows wheels

Summary
With the recent instability and unreliability with azure-pipelines
relying on it to build our windows wheels for the upcoming 0.9 release
doesn't seem to be prudent. Especially when considering our ability to
create reliable portable wheels for a windows environment manually
outside of CI is limited (we've had issues trying to do it manually in
the past). This commit adds back our appveyor config for building wheels
which has worked well for past releases. This way even if azure fails
for us we have a fallback available when we push our release tag.
Details and comments

Ok, cool it looks like this still works: https://ci.appveyor.com/project/Qiskit/qiskit-sdk-py/builds/26679248/artifacts we can use this as a fallback for azure being unreliable I'll revert the test patch and remove the on hold

Hmm, it looks like because there are no appveyor jobs that run on non-tag commits here the appveyor section in the checks will just sit as pending.


Update CI builds to pass VCRedistDir

To consume changes from https://github.com/apple/swift-installer-scripts/pull/256

Closing in favor of https://github.com/apple/swift/pull/69743

Rebased and migrated build.ps1 changes to https://github.com/apple/swift/pull/69743


Anerdw: Blacklist (?-i:^Eight Eggs$)(?#rhxjwpm spam)

Anerdw requests the blacklist of the username (?-i:^Eight Eggs$)(?#rhxjwpm spam). See the MS search here and the Stack Exchange search in text, in URLs, and in code.

(?-i:^Eight Eggs$)(?#rhxjwpm spam) has been seen in 4 true positives, 0 false positives, and 0 NAAs.


Emulator does not show any graphics

Description
Emulator shows black screen

Steps to reproduce the issue
$ docker run -it --rm -v $(pwd)/sdk:/sdk thyrlian/android-sdk bash -c 'cp -a $ANDROID_SDK_ROOT/. /sdk'
$ docker run -d -p 59.1: 5901 -p 2222:22 -v $(pwd)/sdk:/opt/android-sdk -v ~/.ssh/emulator.pub:/root/.ssh/authorized_keys -v $(pwd)/sdk_builder.sh:/root/sdk_builder.sh -v $(pwd)/com.amazon.kindle.apk:/root/com.amazon.kindle.apk thyrlian/android-sdk-vnc
$ ssh<EMAIL_ADDRESS>-p 2222

root@20448ea13e96:$ bash sdk_builder.sh

I've also tried toggling various gpu options and different system images to no avail.
# sdk_builder.sh

sdkmanager --update
sdkmanager "platform-tools" "platforms;android-24" "platforms;android-25" "emulator"
sdkmanager "system-images;android-24;default;armeabi-v7a"
sdkmanager "system-images;android-25;google_apis;armeabi-v7a"

export QTWEBENGINE_DISABLE_SANDBOX=1

echo 'no' | avdmanager create avd -n 24_test -k "system-images;android-24;default;armeabi-v7a"
echo 'no' | avdmanager create avd -n 25_test -k "system-images;android-25;google_apis;armeabi-v7a"
# INFO: List existing Android Virtual Devices
avdmanager list avd
# launch emulator
 emulator -avd 24_test -no-audio -no-boot-anim -gpu off
# emulator -avd 24_test -no-audio -no-boot-anim -accel on -gpu swiftshader_indirect &

The output looks like that

But rerunning the script a second time round gives just this, without the 'emulator out of date' output

Describe the results you expected
The ability to use the emulator through VNC visually.

Hi @aerymilts, I'm currently in the hospital and will check this once I'm back home.
In general, I've tested the container on my side before publishing it, I can view the emulator, interact with the emulator via mouse, and also run UI test on the emulator.
BTW, the warning you've got is not a big deal, you can totally ignore that.
Another matter is, virtualizing an ARM-based emulator, inside a docker container is a quite expensive action, for sure it will be slow.  It may take even a few minutes before the real Android screen gets rendered properly.
I'll try to reproduce using the steps you've provided, and share the updates here.

Hey @thyrlian, I'm sorry to hear that you're in hospital. I hope you get well soon.
I've tried leaving the emulator to start and render for up to half an hour at a time but nothing shows up.
Not sure if this bit of information helps, but I have also tried installing packages from the container and that didn't work. I'm wondering if this is a sign that the emulator hasn't booted properly.
root@20448ea13e96:$ adb install com.amazon.kindle.apk
Performing Streamed Install
adb: failed to install com.amazon.kindle.apk: Can't find service: package

As a thought, would running dind (Docker in Docker) allow for virtualizing an x86-based emulator instead? I intend to use the emulator to use apps that have better support on mobile vs desktop.
I didn't say this the first time round, but fantastic job on this repository and the support you're providing. I saw the level of details you were providing (and spelling out, in some cases) in the other resolved issues. 👍

Thank you @aerymilts for your acknowledgement and good wishes.  I'm getting better now so I've played a bit.
By following the same steps you've listed, I'm able to launch the emulator and interact with it as well.
And I've tried both of the rendering options below, none of them has a problem.
emulator -avd 24_test -no-audio -no-boot-anim -gpu off &
emulator -avd 24_test -no-audio -no-boot-anim -accel on -gpu swiftshader_indirect &

As I said before, the virtualization inside Docker container is quite expensive, the bootup (readiness) of the emulator may take some minutes, which varies between machines (for instance: MacMini is way faster than MacbookPro).  In fact, on my MacbookPro I haven't been waiting too long.  And the other error message you've given - Can't find service: package tells that the bootup process is not finished yet (or properly).  My suggestion would be: try with a powerful machine or some cloud service.  Please forget about Docker in Docker, it won't help.

Hey @thyrlian, appreciate you testing out my scripts on your setup. Good to know I've set it up correctly and it's just down to the hardware. It appears that my GPU is not shared with the container. I managed to get the emulator going by disabling automatic graphics switching. The interface is incredibly slow, as you've pointed out before.
Re: dind, I just went over the Emulator section again. Does this mean that there's absolutely no way for a Mac user to run x86 emulators (short of dual booting into Linux)? I did find that there are paid options out there that enabled nested virtualization but it seems like there's not much support for this.
I don't have any knowledge in this area - do you have any recommendations/thoughts?

Hi @aerymilts, it's really good to hear that now it works for you after your magic tweak on your host machine.  I'd like to copy the screenshot here from the link you've given, for other users' reference, because Apple moves its "permalink" often.  Actually this is something new to me since my MacbookPro only has an integrated graphic card, I don't even have such option (meaning the performance on my machine should be worse than yours, still I'm able to run UI test under acceptable timing).

Regarding dind:

In general, it's not possible on macOS.  However, it's not absolutely impossible, if you can manage to port KVM to macOS, plus a ton of hacks, it may work, probably that doesn't worth the effort.
Yes, not long ago VirtualBox started to support nested virtualization, gradually from AMD CPU to Intel CPU.
Ultimately, I would pay more attention to the intention rather than the solution. We could get it working in some way, but in the end, the performance won't make us satisfied.  For the sake of debugging, or just playing around, the simplest solution would be the Intel x86 emulator which comes with Android Studio.  The emulator inside a Docker container is designed for CI purpose, for UI test automation.  And usually, the companies have powerful Linux machine on-premises, or pay some cloud computing services, there the support for x86 emulator inside Docker container is optimal.  What do you think?

@thyrlian, go for it. I hope this is helpful for someone else in the future as well. 😄 That's interesting. I think the reason why it doesn't work on my integrated graphics is because I have a bad habit of leaving my laptop on for weeks at a time, so there's usually resource leaks which doesn't work very well for high performing apps. For reference, here are the stats of my MBP

In general, it's not possible on macOS. However, it's not absolutely impossible, if you can manage to port KVM to macOS, plus a ton of hacks, it may work, probably that doesn't worth the effort.
Yes, not long ago VirtualBox started to support nested virtualization, gradually from AMD CPU to Intel CPU.

Thanks for sharing, that's where I ended up as I went down the rabbit hole. I'm pretty much stuck as virtualization and architectures is way out of depth for me.

Ultimately, I would pay more attention to the intention rather than the solution. We could get it working in some way, but in the end, the performance won't make us satisfied. For the sake of debugging, or just playing around, the simplest solution would be the Intel x86 emulator which comes with Android Studio. The emulator inside a Docker container is designed for CI purpose, for UI test automation. And usually, the companies have powerful Linux machine on-premises, or pay some cloud computing services, there the support for x86 emulator inside Docker container is optimal. What do you think?

My intent is to build an emulator that allows me to use mobile apps that requires low to moderate resources (think: smooth paging for Kindle for Android on an emulator), using open source tools, high reproducibility (container as code) and is easy to port across OSes. That's how I got to Docker and your project.
I also came across BlueStacks which serves my purpose pretty well (in fact it seems like it's meant for high performance mobile gaming). It's just that it's not open source and it doesn't have high configurability. I initially thought that I'd just have to go deeper into Docker to find the right setup to get it to work just like how BlueStacks does it. But it seems like there are limitations with Docker that just doesn't allow it. Is my understanding accurate?

For the sake of debugging, or just playing around, the simplest solution would be the Intel x86 emulator which comes with Android Studio.

I just realised, did you mean to say that using the x86 emulator with Android Studio natively on my MBP would work?

For my integrated graphics card, I also tend to leave my laptop running/hibernating for weeks, but I haven't run into any problem launching emulator inside a container.
I got your purpose now, and I don't think emulator in container is a good solution for you.  Please go and give a try of the x86 emulator from Android Studio, that should serve your needs.

Sorry for the late response. Yes, it seems that accelerated containerised emulation for Mac is a convoluted solution at best. Thank you for helping me with this!


Rebuild for LLVM 15.x

I checked that there's no new version available upstream

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.


fix parse exception when sse stream returns incomplete data

I've noticed that the first response from SSE is sometimes incomplete, where it might split a single JSON object across two responses.
The first:
{"id":"chatcmpl-8NY6x7nqghZB9FgFciPtk5

The second:
azY6e6e","object":"chat.completion.chunk","created":1700623359,"model":"gpt-4-0613","choices":[{"index":0,"delta":{"role":"assistant","content":""},"finish_reason":null}]}

By catching a jsonDecode exception, we can identify this situation and attempt to concatenate the incomplete response with the next one to parse them together, which should solve the issue.

Codecov Report
Attention: 13 lines in your changes are missing coverage. Please review.

Comparison is base (a17b4fd) 83.87% compared to head (be02275) 83.87%.
Report is 1 commits behind head on main.

Files
Patch %
Lines

lib/src/client/openai_client.dart
13.33%
13 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main      #80   +/-   ##
=======================================
  Coverage   83.87%   83.87%
=======================================
  Files          71       71
  Lines        1048     1048
=======================================
  Hits          879      879
  Misses        169      169

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


In the notebook baselines/EntropicNeuralOptimalTransport/notebooks/High_dimensionsal_gaussians.ipynb, I believe that the closed form formula for the EOT plan between Gaussians is wrong.

In the EntropicOTBenchmark/baselines/EntropicNeuralOptimalTransport/notebooks/High_dimensionsal_gaussians.ipynb, I believe that the closed formula entropic optimal transport plan (EOT) between Gaussians measures is not correct.
It does not coincide with that proved in [Janati+2020] and coded in https://github.com/hichamjanati/Entropic-OT-gaussians/blob/master/closed_forms.py, see function closed_form_balanced.
Indeed, [Janati+2020] states that for $\mu = \mathcal{N}(0, \mathbf{A})$ and $\nu = \mathcal{N}(0, \mathbf{B})$ (which is used in the notebook), if we set $\varepsilon = 2 \sigma^2$ then $\pi^\star_\varepsilon$ reads:
$$
\pi_\varepsilon^\star \sim \mathcal{N}(0, \begin{bmatrix}
A & C_\sigma \
C_\sigma^T & B
\end{bmatrix})
$$
where $C_\sigma= \tfrac{1}{2} (\mathbf{A}^{1/2} D_\sigma \mathbf{A}^{- 1/2} - \sigma^2 \mathbf{I}_d)$ with $D_\sigma = (4 \mathbf{A}^{1/2} \mathbf{B} \mathbf{A}^{1/2} + \sigma^4 \mathbf{I}_d)^{1/2}$.
In cell [26] of Sec. 4 of the High_dimensionsal_gaussians.ipynb notebook, there seems to be an error in the variable change $\varepsilon = 2\sigma^2$. Indeed,

the functions get_D_sigma returns 0.5*(covariance_0_sqrt@D_sigma@covariance_0_sqrt_inv - epsilon*np.eye(shape)).
the functions get_C_sigma returnssymmetrize(sqrtm(4*covariance_0_sqrt@covariance_T@covariance_0_sqrt + (epsilon**2)*np.eye(shape))).

However, if we rewrite $C_\sigma$ and $D_\sigma$ with $\sigma = \sqrt{\varepsilon /2}$, we then get:

$D_\sigma = (4 \mathbf{A}^{1/2} \mathbf{B} \mathbf{A}^{1/2} + \tfrac{1}{4}\varepsilon^2 \mathbf{I}_d)^{1/2}$.
$C_\sigma= \tfrac{1}{2} (\mathbf{A}^{1/2} D_\sigma \mathbf{A}^{- 1/2} - \tfrac{1}{2}\varepsilon \mathbf{I}_d)$ .

Therefore, epsilon**2 in get_D_sigma should be replaced by epsilon**2/4; and epsilon**2 in get_C_sigma should be replaced by epsilon / 2; i.e.

the functions get_D_sigma should instead return 0.5*(covariance_0_sqrt@D_sigma@covariance_0_sqrt_inv - (epsilon / 2) *np.eye(shape)).
the functions get_C_sigma should instead return symmetrize(sqrtm(4*covariance_0_sqrt@covariance_T@covariance_0_sqrt + ((epsilon**2 / 4)*np.eye(shape))).

This is a crucial problem because the way functions are currently coded results in the covariance matrix
$$
\begin{bmatrix}
A & C_\sigma \
C_\sigma^T & B
\end{bmatrix}
$$
to have significantly negative eigenvalues.

Hello, thank you for your interest in our work.
First of all, please note that the notebook EntropicOTBenchmark/baselines/EntropicNeuralOptimalTransport/notebooks/High_dimensionsal_gaussians.ipynb is from the folder baselines/EntropicNeuralOptimalTransport, which contains the code for one of the methods we evaluated in our work. But this folder also contains code for other experiments from the corresponding paper "Entropic Neural Optimal Transport via diffusion process" (https://neurips.cc/virtual/2023/oral/73836). In particular, the code from High_dimensionsal_gaussians.ipynb was not used for the Entropic OT benchmark.
Second, in our work we consider the entropic OT problem in the form:
$$
\min_{\pi \in \Pi(\mathbb{P}_0, \mathbb{P}_1)} \int \frac{||x-y||^2}{2} d\pi(x,y) - \epsilon H(\pi),
$$
where $H(\pi)$ is the differential entropy of the distribution $\pi$. This problem is equivalent to the problem:
$$
\min_{\pi \in \Pi(\mathbb{P}_0, \mathbb{P}_1)} \int \frac{||x-y||^2}{2}  d\pi(x,y) + \epsilon \text{KL}(\pi || \mathbb{P}_0 \times \mathbb{P}_1),
$$
where $\text{KL}(\pi || \mathbb{P}_0 \times \mathbb{P}_1)$ is the KL divergence between the distribution $\pi$ and the independent distribution $\mathbb{P}_0 \times \mathbb{P}_1$ with the marginals $\mathbb{P}_0$ and $\mathbb{P}_1$. In [Janati+2020] the authors consider the following problem:
$$
\min_{\pi \in \Pi(\mathbb{P}_0, \mathbb{P}_1)} \int ||x-y||^2d\pi(x,y) + 2 \sigma^2 \text{KL}(\pi || \mathbb{P}_0 \times \mathbb{P}_1),
$$
which is equivalent to
$$
\min_{\pi \in \Pi(\mathbb{P}_0, \mathbb{P}_1)} \int \frac{||x-y||^2}{2}d\pi(x,y) + \sigma^2 \text{KL}(\pi || \mathbb{P}_0 \times \mathbb{P}_1).
$$
So $\sigma^2 = \epsilon$ as we use in our code.

Thanks a lot for your quick answer and this clarification. I can close the issue.


 Please view the below image for reference.
The logo is not correct and the markdown below that is also incorrectly rendered.

Hey can I work on this?

I believe this issue is closed. @mukul314

I resolved some errors. please checkout   https://github.com/surajkumar2411/FunwithScience

@surajkumar2411 this is not a part of any open source event. But if u still want to contribute u can.

Thank you for your email.
I continue worked with your project.
Suraj Kumar
Mob :<PHONE_NUMBER>
Email: @.***
On Sat, 27 May, 2023, 11:12 pm Amit Kumar Mishra, @.***>
wrote:

@surajkumar2411 https://github.com/surajkumar2411 this is not a part of
any open source event. But if u still want to contribute u can.
—
Reply to this email directly, view it on GitHub
https://github.com/Tech-N-Science/FunwithScience/issues/711#issuecomment-1565612043,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A7CGGJM4TCFEDIAJJL7BT7DXII4HHANCNFSM6AAAAAAXY43KTE
.
You are receiving this because you were mentioned.Message ID:
@.***>

@surajkumar2411 u can work


Stack overflow

Hi there, i got this error: Stack overflow in unmanaged: IP: 0x7f541bd3b51f, fault addr: 0x7ffc7ff045d8
and in fivem: http://upload.proxyping.de/Discord_jdOIWVOwKq.png
the build i use is: 3539

This information is meaningless.
Please use our discord for support.


Performance Improvements

Hi, I haven't used your mod yet, but I was warned that it has poor performance and I decided to see why. I have one question for the code: does it really create a separate thread for each player? I see no reason to use a separate thread in principle for this.

Thread creation is too expensive and it takes too many resources for each player. In addition, the player can exit and enter the server later (or there may be many players), due to which there will be an unacceptable amount of such allocations. This thread can be transferred to other tasks of the game. I suggest removing this logic and leaving the DeObfuscator class as a Runnable implementer for future code base changes.

As a last resort, we can use some implementation of the thread pool, but I'm not sure if the code slows down mspt that much. Its ticking after the player's tick also seems strange to me, maybe we just iterate over all the players?


Generator Selection "{LESS}" should be "{less}" or "less"

The logo is "{less}" not "{LESS}” :)

Will be in the next release


Docker - /distribution-2.7.1/vendor/gopkg.in/yaml.v2/yaml.go - set license

Type: Incomplete
Summary:
Docker - /distribution-2.7.1/vendor/gopkg.in/yaml.v2/yaml.go - set license
Details:
Docker as part of the Azure stack scan.
File containing a section of code prefaced by the following notice:
// The code in this section was copied from mgo/bson.
Resolution:
The code that follows this notice originates with mgo, "[t]he MongoDB driver for Go." See https://github.com/go-mgo/mgo (see specifically https://github.com/go-mgo/mgo/blob/v2-unstable/bson/bson.go). This material is licensed under the BSD License (see https://github.com/go-mgo/mgo/blob/v2-unstable/
Affected definitions:

distribution 2461543d988979529609e8cb6fca9ca190dc48da

@clearlyoss is this supposed to be BSD-2-Clause instead of BSD-3-Clause? The link points to 2-Clause, but you submitted it as 3-Clause. If I'm missing something, please let me know!

Hello, thanks for asking.

Thank you for confirming!


revert circular dep

Using #807 locally to test the production build of evergreen (via yarn link) I was able to narrow in on a circular dependency warnings for Icon and defaultTheme.

It's annoying because now Alert/InlineAlert break tree-shaking, but enough of v4 already breaks it that this wont matter much. We'll have to come up with a non-theme way to get the right icons.


Patches I used for issue #1622

updated with barrier strengthening

Interestingly enough TSAN claims there are still data races here, though they go away if I use _Atomic instead of volatile for the working array.


#776 feat(templates): support to pass host port mapped to container port

@deviantony
Hi,
Added support to map host port to the container port using templates as discussed on #776
{
      "ports": {
             ["80/tcp"],
             {
                  "container": "443",
                  "host": "443",
                  "protocol": "tcp"
             }
      }
}

What's the status of this? This is a fairly important feature for a deployment I'm working on.

Im OK to merge this… Anthony will be online in a few hours, I will make sure he reviews with the intent to merge into develop
From: Cody Marx Bailey<EMAIL_ADDRESS>Sent: Monday, 25 June 2018 10:32 a.m.
To: portainer/portainer<EMAIL_ADDRESS>Cc: Subscribed<EMAIL_ADDRESS>Subject: Re: [portainer/portainer] #776 feat(templates): support to pass host port mapped to container port (#1958)
What's the status of this? This is a fairly important feature for a deployment I'm working on.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_portainer_portainer_pull_1958-23issuecomment-2D399792990&d=DwMCaQ&c=euGZstcaTDllvimEN8b7jXrwqOf-v5A_CdpgnVfiiMM&r=0fx0h4vB56iTLpw2McH1ZD6TqG_QGpbggVOB-PfMJpM&m=BxAM2wUFRUIxSFACU81943YEQW18EAAJjR2sfnzyksI&s=XzBRr0pkX4YHpxDm7lSdCHaOkuFoWfMM4f2MzfX6KME&e=, or mute the threadhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_notifications_unsubscribe-2Dauth_AWGrlax391UjaZeDWEwNT8MUOAVNPQthks5uABNagaJpZM4Ufge5&d=DwMCaQ&c=euGZstcaTDllvimEN8b7jXrwqOf-v5A_CdpgnVfiiMM&r=0fx0h4vB56iTLpw2McH1ZD6TqG_QGpbggVOB-PfMJpM&m=BxAM2wUFRUIxSFACU81943YEQW18EAAJjR2sfnzyksI&s=r3uPYvVwmTERV5ovplkJdde4Fq6v9s2oeWJZJm-VJAo&e=.

Hi @deviantony, it is a better practice to write return statement in the end of a function. Multiple return statements should be avoided. If it is necessary i will change it.

Hi @mayank
Generally I tend to agree with this but in this case the function is pretty small it just makes the code easier to read than an if/else statement.

@deviantony done!! 🍾

@mayank did you test it? I'm not able to deploy a template using the new definition. I'll update this PR with more details later, but you should try to deploy a template with the new port format to check.

working fine. here are the screenshots.
Template from portainer-templates

Port added in the template form

Deployed container shows the port mapping

@deviantony any updates? 😕

I think it's broken when you leave the host port empty.
Also, sorry for the late update, I'm working on enabling template management inside Portainer, I'll merge this one as soon as the work on template management is done.

@deviantony okay, please let me know if any changes required from my end.

Closed in favor of #2017


Document coding standards and conventions

Currently, we sort of send people all over the place regarding developer guidelines. We should update the CONTRIBUTING doc to contain the following:

Info about the CLA
Info about this being a CNCF project
A pointer to the Semantic Commit Messages (http://karma-runner.github.io/0.13/dev/git-commit-msg.html)
A note about Go and Proto coding conventions
Something about a code of conduct, which we don't currently talk about at all

+1

+1 for info about the CLA.

I think I will open a separate issue for the Code of Conduct thing, since it's bigger than just a minor doc change.


Fix/iframe attributes

This PR addresses the issue that other data-featherlight-iframe-... attributes do not get added to the iframe when it's constructed because we're limited to just CSS attributes here.

Looks like good work :+1:
I added a few comments. Also, please don't change release, I will take care of that.

@marcandre thanks for the review. I'll updated the code with your comments.

Forgot, but there should be a test modified or added that checks this too...

I added a two tests, LMK.

I thought about doing that; but I wanted to leave the parseAttr function generic in hopes for it to be reusable in other scenarios.

You could leave parseAttrs as is and then split to result in two, no?

What you can add to the css test is that css values are not also added as
attributes. I agree that you can’t really test that normal attributes are
not added to the css
On Mon, Dec 12, 2016 at 11:58 AM, John<EMAIL_ADDRESS>wrote:

@nguyenj commented on this pull request.
In test/featherlight_test.js
https://github.com/noelboss/featherlight/pull/269:

	describe('generates an iframe with tag attributes', function() {

		it('adds `allowfullescreen` attribute', function() {

			$('').featherlight().click();

			expect($('.featherlight iframe')).to.have.attr('src').equal('http://www.apple.com');

			expect($('.featherlight iframe')).to.have.attr('allowfullscreen').equal('true');

		});

		it('does not add invalid attributes', function() {

			$('').featherlight().click();

			expect($('.featherlight iframe')).to.have.attr('src').equal('http://www.apple.com');

			expect($('.featherlight iframe')).to.not.have.attr('allowfullscreen').equal('foo');

I didn't add it to the CSS test because jQuery's .css should take care of
invalid css attributes.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/noelboss/featherlight/pull/269, or mute the thread
https://github.com/notifications/unsubscribe-auth/AACD6nxyLyC-q1lDcw0i9xyMrgspeeyHks5rHX0lgaJpZM4LKFYf
.

Ah, okay I understand. I'll add those a test for that.

I was thinking about refactoring readElementConfig to utilize parseAttrs; but I haven't gotten to chance to dive into it thoroughly yet; hence my hesitation on making it return an object with css or attr keys.

Don’t sweat it too much. As long as you call parseAttrs no more than
twice, it’s fine :-)
On Mon, Dec 12, 2016 at 1:06 PM, John<EMAIL_ADDRESS>wrote:

I was thinking about refactoring readElementConfig to utilize parseAttrs;
but I haven't gotten to chance to dive into it thoroughly yet; hence my
hesitation on making it return an object with css or attr keys.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/noelboss/featherlight/pull/269#issuecomment-266504820,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AACD6hfj175_vbQGpXcTsjD54aHdrLIXks5rHY0JgaJpZM4LKFYf
.

Merged, refactored a bit. Good work, thanks!
Released in 1.7.0


added in chicago elec dist

Added in Chicago Elec. Dist.

The parsing looks good.
Two things:

Could you change the vendor module name to ChicagoElectronicDistributorsUs (module and file names) but keep the shorter version for the vendor name field?

Products that are in stock show a count on how many items are left, can you parse this as well? It's the same as for #30

Product that is on stock which has an items count: https://chicagodist.com/products/12-vdc-1000ma-regulated-switching-power-adapter-ul-listed

I changed the names, I hope that's what you were after?
I implemented a stock counter. I'm sure there's a better, more "elixir way" to do the syntax...

I may be missing a potential error state, I'm only checking for != [] at the moment. I figure if I'm calling Floki, it will always spit out a list vs giving nil or some other value/data?
I'm not sure if I picked the right HTML element, but I liked that I could grab a direct value vs. doing some sort of text-to-number parsing function for the innerText since it contains both the number we want plus some text.
Let me know what you think!

Looks good to me! Thanks!
closes #29


[Bulk] Validation: duplicate-description | 1

Fix duplicate-description in 50 files - (VB)/(C#) only

Global effort to fix validation errors
@Rick-Anderson
The Content & Learning team is fixing build validation errors on docs.microsoft.com. This effort will eliminate potential accessibility, security, and usability issues. This PR includes only build validation fixes and does not change other content. I’d very much appreciate your timely review and feedback on these changes I’ve suggested. Thanks!


Cannot run create-turbo (inquirer ScreenManager crash)

What version of Turborepo are you using?
1.2.8
What package manager are you using / does the bug impact?
npm
What operating system are you using?
Mac
Describe the Bug
Running npx create-turbo@latest results in an error:
$ npx create-turbo@latest
npx: installed 186 in 19.429s

>>> TURBOREPO

>>> Welcome to Turborepo! Let's get you set up with a new codebase.

Aborting installation.
Unexpected error. Please report it as a bug:
TypeError: this.breakLines(...).flat is not a function
    at ScreenManager.forceLineReturn (/Users/frederik/.npm/_npx/49048/lib/node_modules/create-turbo/node_modules/inquirer/lib/utils/screen-manager.js:172:56)
    at ScreenManager.render (/Users/frederik/.npm/_npx/49048/lib/node_modules/create-turbo/node_modules/inquirer/lib/utils/screen-manager.js:81:20)
    at InputPrompt.render (/Users/frederik/.npm/_npx/49048/lib/node_modules/create-turbo/node_modules/inquirer/lib/prompts/input.js:67:17)
    at InputPrompt._run (/Users/frederik/.npm/_npx/49048/lib/node_modules/create-turbo/node_modules/inquirer/lib/prompts/input.js:34:10)
    at Promise (/Users/frederik/.npm/_npx/49048/lib/node_modules/create-turbo/node_modules/inquirer/lib/prompts/base.js:61:12)
    at new Promise (<anonymous>)
    at InputPrompt.run (/Users/frederik/.npm/_npx/49048/lib/node_modules/create-turbo/node_modules/inquirer/lib/prompts/base.js:60:12)
    at defer (/Users/frederik/.npm/_npx/49048/lib/node_modules/create-turbo/node_modules/inquirer/lib/ui/prompt.js:105:30)
    at Observable._subscribe (/Users/frederik/.npm/_npx/49048/lib/node_modules/create-turbo/node_modules/rxjs/dist/cjs/internal/observable/defer.js:8:31)
    at Observable._trySubscribe (/Users/frederik/.npm/_npx/49048/lib/node_modules/create-turbo/node_modules/rxjs/dist/cjs/internal/Observable.js:41:25)

Expected Behavior
Expected the command to setup turborepo.
To Reproduce
Running on

MacBook Pro (M1, 14 inch, 2021)
macOS 12.3.1
iTerm 3.4.15
Fish 3.3.1

Execute command npx create-turbo@latest

What version of node.js do you have installed? I'm not able to reproduce

This happens in Node 10 and earlier; which is normal since flat is not supported by Node 10.

create-turbo works with Node.js 12+


[build] Add support for Foxy Fitzroy with OE thud, warrior, zeus

Merged and tagged as build/milestones/13-all, because build/milestones/13 already existed with just dunfell and gatesgarth support for foxy.


ModuleNotFoundError while running the AutoGen+MemGPT example notebook

Getting the following error message :
from memgpt.autogen.memgpt_agent import create_autogen_memgpt_agent, create_memgpt_autogen_agent_from_config
ModuleNotFoundError: No module named 'memgpt.autogen'; 'memgpt' is not a package
I've tried updating memgpt using pip install pymemgpt -U
AutoGen is working fine on its own (without the added Memgpt functionalities)

Are you running this via a colab notebook or locally? Can you paste the command you're using to run the script?

I'm running it locally indeed (on VS Code)
I've just copy-pasted the file in the path  memgpt/autogen/examples/agent_groupchat.py and added it to the local directory of autogen I'm working with.
.
Then I use the command: python agent_groupchat.py
(Which leads to the error I mentioned)

@vivi is the closeable with the new AutoGen?

import memgpt.autogen.memgpt_agent as memgpt_autogen
ModuleNotFoundError: No module named 'memgpt'

I'm running across this issue with the latest pip installs - pymemgpt    0.2.11  -  pyautogen   0.2.6


fix tests for step 47 of calorie counter

Describe the Issue
A camper on the forum was unable to pass the lesson
https://forum.freecodecamp.org/t/learn-form-validation-by-building-a-calorie-counter-step-47/685533/2
this was their code
/* file: script.js */
// User Editable Region

function addEntry() {
    const targetInputContainer = document.querySelector(`#${entryDropdown.value} .input-container`);
    const entryNumber = targetInputContainer.querySelectorAll('input[type="text"]').length;
    const HTMLString = `<label for="${entryDropdown.value}-${entryNumber}-name">Entry ${entryNumber} Name</label>`
}

// User Editable Region

the issue was here
    const HTMLString = `<label for="${entryDropdown.value}-${entryNumber}-name">Entry ${entryNumber} Name</label>`

the tests only accept it if the label is on the next line like this
    const HTMLString = `
     Entry ${entryNumber} Name`

the tests should be fixed
Affected Page
https://www.freecodecamp.org/learn/javascript-algorithms-and-data-structures-v8/learn-form-validation-by-building-a-calorie-counter/step-47
Your code
see explanation above
Expected behavior
see explanation above
Screenshots
No response
System

Device: [e.g. iPhone 6, Laptop]
OS: [e.g. iOS 14, Windows 10, Ubuntu 20.04]
Browser: [e.g. Chrome, Safari]
Version: [e.g. 22]

Additional context
No response

I remembered this issue here where there were issues with the newlines
https://github.com/freeCodeCamp/freeCodeCamp/issues/52741
So, I feel like an easier fix would be to just fix the editable regions
Here should be the new fix:
function addEntry() {
  const targetInputContainer = document.querySelector(`#${entryDropdown.value} .input-container`);
  const entryNumber = targetInputContainer.querySelectorAll('input[type="text"]').length;
  const HTMLString = `
  --fcc-editable-region--
  Entry ${entryNumber} Name`;
  --fcc-editable-region--
}

I'm going to flag this one for first-timers only.


Cannot select January 2017

Using this configuration, I'm unable to select January 2017 from our datepicker (tested in Safari and Chrome). Other dates do work.
new Flatpickr(document.getElementById('searchDate'), {
    minDate: new Date(),
    defaultDate: new Date(),
    altInput: true,
    altFormat: "d/m/Y"
})

Expected Behaviour
I should be able to select January 2017 from the dates.
Current Behaviour
January 2017 is disabled.
Your Environment

Version used: ~2.2.3
Browser Name and version: Tested in Safari and Chrome
Operating System and version (desktop or mobile): Desktop

It looks like the minDate option is the issue here. Removing that causes it to work, but allows the user to select a previous date.

After a bit of digging, it looks like the compareDates method is the culprit.
Is there any reason the method can't just be the following?
function compareDates(date1, date2) {
    if (!(date1 instanceof Date) || !(date2 instanceof Date))
        return false;

    return date1.getTime() - date2.getTime();
}

Ignore me! This was fixed in v2.2.4 - https://github.com/chmln/flatpickr/commit/196401b942920e5c7cb82e89a1f8504e20bbe84d


[HUDI-8232] Adding table config to track list of cols indexed with col stats partition in MDT

Change Logs

Added a table config ("hoodie.table.col.stats.indexed.columns") to track the list of columns indexed with column stats partition in Metadata table. This will be later leveraged during pruning to only lookup in col stats partition for indexed cols (HUDI-8663)

Impact
This will help track the actual list of columns indexed with col stats partition in MDT.
Risk level (write none, low medium or high below)
low
Documentation Update
Describe any necessary documentation update if there is any new feature, config, or user-facing change. If not, put "none".

The config description must be updated if new configs are added or the default value of the configs are changed
Any new feature or user-facing change requires updating the Hudi website. Please create a Jira ticket, attach the
ticket number here and follow the instruction to make
changes to the website.

Contributor's checklist

[ ] Read through contributor's guide
[ ] Change Logs and Impact were stated clearly
[ ] Adequate tests were added if applicable
[ ] CI passed

Re-worked on this patch here https://github.com/apache/hudi/pull/12529


Missing Tabs

Hi
When first running the host sample, I'm not able to see any tabs other than Properties when editing a client. Should I expect to be able to set the redirect uris etc, as those tabs are not visible?
Cheers
Dave

Hi Dajomo,
You can set every property of the client, there was just a small issue with the In memory client, that I used for example purpose. I fixed the problem and updated the git, now I should work :)
Have fun :)


django-secure still neccessary?

Hello! Just came across this, and I thought of cookiecutter-django.
According to the django-secure docs, it was pulled into Django when it hit 1.8 and the library is no longer maintained.
Unsure if the configuration changed in the process, but maybe the dependency could be dropped?

We just need a pull request taking it out. Also see #280.

Addred in #579 by @xpostudio4


Adding Spanish translation

@BernardoBas please update the PR. Thank you

@tommasoberlose I think it's not needed to merge because we translated same strings. Should translate the recently added strings


Shor Algorithm should be added to qiskit_algorithms library

What should we add?
Some days ago, I try to implement Shor's Algorithm using the Following code..
import math import numpy as np from qiskit import * from qiskit.utils import QuantumInstance from qiskit_algorithms import Shor
but it showing ImportError and I also learned that Shor’s algorithm was deprecated and removed from 0.22.0 release. the code of ibm quantum learning of Shor' Algorithm https://learning.quantum.ibm.com/course/fundamentals-of-quantum-algorithms/phase-estimation-and-factoring#introduction is too much hard to learn and understand for beginner's like me and less practical use  for the upcoming quantum computer's future.
can qiskit developers can make arrangements to reestablish it.... :)

found this one: https://github.com/kazawai/shor_qiskit

this one might be even better suited: https://github.com/Shailaja87/BE_PROJECT/tree/main

Thanks, A lot..... @stubbi

I'll close this as "stale" or "can't fix here" now - core Qiskit doesn't include any large-scale implementations of full quantum algorithms.  This could be taken up at https://github.com/qiskit-community/qiskit-algorithms, but my understanding is that Shor was removed for a reason, so I'd imagine it's unlikely to be re-added.
I'd note that if it's just a tutorial in implementation that's of interest, the IBM Quantum platform has an example: https://learning.quantum.ibm.com/course/fundamentals-of-quantum-algorithms/phase-estimation-and-factoring#shors-algorithm.


aur-sync: missing dependencies error when trying to resolve cyclic dependencies with --no-checkdepends

So, I have the following situation:

package A depends on package B and C
package C has a checkdepends on package B

When using --no-checkdepends aurutils filters out package B as it is a checkdepends of C, but then the aur sync call fails because it can't find the dependency B for package A.

The package in question is ruby-rails. My testing showed that the missing dependencies are still in the queue, just in the wrong order. It's likely that the PKGBUILD depends are declared with checkdepends in mind, and that removing checkdepends causes some side-effects. In this case the only solution is to fix the PKGBUILD. On package installation I've noticed cyclic dependencies as well, but pacman has no problems with that.

Actually https://aur.archlinux.org/packages/ruby-rails/ lists not even all dependencies according to https://rubygems.org/gems/rails . Also according to rubygems all of these dependencies are runtime dependencies.

A PKGBUILD error aside, I managed to build ruby-activemodel (and with that, ruby-rails) with --nocheckdepends in 4.2.
#882 takes the logic a bit farther, but I think this issue can be closed.


Error: REQUIRED_FIELD_MISSING, Required fields are missing

Gettign the following error.  Any idea how to fix this?
Error:
dlrs_BMCServiceDesk_Incident_Chana2BTest.testTrigger System.DmlException: Insert failed. First exception on row 0; first error: CANNOT_INSERT_UPDATE_ACTIVATE_ENTITY, BMCServiceDesk.UpdateIncidentChangeLinkFields: execution of BeforeInsert caused by: System.DmlException: Insert failed. First exception on row 0; first error: REQUIRED_FIELD_MISSING, Required fields are missing: [BMCServiceDesk__FKIncident__c]: [BMCServiceDesk__FKIncident__c] Trigger.BMCServiceDesk.UpdateIncidentChangeLinkFields: line 35, column 1: [] Class.dlrs.RollupService.testHandler: line 282, column 1 Class.dlrs_BMCServiceDesk_Incident_Chana2BTest.testTrigger: line 11, column 1

@leogarc were you able to get this fixed? Looks like you need to implement custom test code to get this working.

Yes custom test code worked.
From: Heber<EMAIL_ADDRESS>Sent: Tuesday, December 18, 2018 4:48 PM
To: afawcett/declarative-lookup-rollup-summaries<EMAIL_ADDRESS>Cc: leogarc<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [afawcett/declarative-lookup-rollup-summaries] Error: REQUIRED_FIELD_MISSING, Required fields are missing (#757)
@leogarchttps://github.com/leogarc were you able to get this fixed? Looks like you need to implement custom test code to get this working.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/afawcett/declarative-lookup-rollup-summaries/issues/757#issuecomment-448382990, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AqbHL99kQP0VyimrhehqqQYzzlDCApl7ks5u6WKigaJpZM4X7T_u.

https://github.com/afawcett/declarative-lookup-rollup-summaries/wiki/Challenges-with-Code-Coverage


Comments column re-size in Diagram

Hi there ,
I am really impress with the plugin so thanks for all the hard work.
For the life of me I can not work out how to increase the size of the comment column in the diagram view. It only shows about 16 Chars of the comment.
I would like to be able to resize the table and the column to be bigger so that I can take useful screen shots for documentation.
Could you please explain how I can do this if possible.
Thanks

New options have been added.
erd-editor-vscode v1.0.1


Support for nil values in array of optionals (ex. [String?])

Currently if I try to map a JSON in array of optionals the object is always empty, even if JSON array does not return any null field. Example : on my mappable struct I declare a field as follow :
var strings : [String?]?
This object is always nil for all json cases below
{"strings":["someString",null]}
{"strings":["someString"]}
{"strings":[]}
Is there a way to map array of optionals to objects ?

Unfortunately, at the moment ObjectMapper doesn't support mapping arrays of Optionals the way you are describing.

I think this kind of JSON is a little weird...

I have to handle similar json arrays in my app. For an array of Ints = [null, 5, null, 10] index of an element does matter - I must get value '10' using array[3] instead of array[1]

Duplicate of #263 which has a workaround.

Thanks @KevM, closing this.


Problem with mapping from elastic to C# object [nulls in object when _source have data]

How is it possible that NEST C# cannot map it poperly to object when it's not empty in query _source ?

Is it problem that everything should be lower case ? geolocation and id have normal data

The default casing NEST emits and expects to read is camelCase. You can influence this on ConnectionSettings().DefaultFieldNameInferrer() as documented here: https://www.elastic.co/guide/en/elasticsearch/client/net-api/current/configuration-options.html
e.g ``ConnectionSettings().DefaultFieldNameInferrer(s => s)` will emit and read property names verbatim.
Or map the names explicitly through attribute mapping: https://www.elastic.co/guide/en/elasticsearch/client/net-api/current/attribute-mapping.html
Hope this helps!


Question about performance of dijkstra_shortest_paths

Hello,
I want to find the shortest distance between two points under the constrain that the length of each edge in the path has to be smaller than a given value. I construct the graph in a naive way by looping over my points like this:
function construct_graph_naive(points::Array{Point, 1}, radius::Float64)
  n_points = length(points)
  inc_list = simple_inclist(n_points, is_directed=false)
  dists = Float64[]
  for i = 1:n_points
    for j = i+1:n_points
      point_1 = points[i]
      point_2 = points[j]
      dist = distance_point(point_1, point_2) # Computes the distance between two points
      if dist < radius
        add_edge!(inc_list, i, j)
        push!(dists, dist)
      end
    end
  end
  return inc_list, dists
end

This generates a graph with ~12 000 vertices and ~100 000 edges. I then call dijkstra on the returned values:
path_state = dijkstra_shortest_paths(inc_list, dists, start_node)

The call to dijkstra currently takes around 4.6 seconds. Comparing these 4.6 seconds for a 12 000 vertices, 100 000 edge graph and the stated "a benchmark shows that it takes about 15 milliseconds to run the Dijkstra’s algorithm over a graph with 10 thousand vertices and 1 million edges on a macbook pro." makes me think I am doing something wrong.
The same graph in scipys dijkstra (http://docs.scipy.org/doc/scipy-0.14.0/reference/generated/scipy.sparse.csgraph.dijkstra.html) takes on the same computer about 0.2 seconds.
I am quite new to Julia so I might have made some obvious performance killing mistake.
Thanks in advance!

Try using simple_graph instead of simple_inclist.

And we are down to 9 ms! I don't really know about the strengths and weaknesses in the different graph data structures but obviously the graph was much faster than the inclist.
Thank you!

I also noticed that the time to take the graph went down by a factor of 10 as well. Guess it pays off to know your data structures!

It's not just data structures, this was probably made much slower by #135.


[GH-865]: Solved the bug: Jira issues cannot be created anymore with Jira Server/Datacenter 9.x

Summary

The REST endpoint rest/api/2/issue/createmeta was removed in Jira Server 9, due to which we were not able to create an issue in Jira from mattermost.
Ticket Link

Issue #865

Hello @Nityanand13,
Thanks for your pull request! A Core Committer will review your pull request soon. For code contributions, you can learn more about the review process here.

Codecov Report
Base: 31.27% // Head: 31.00% // Decreases project coverage by -0.27% :warning:

Coverage data is based on head (3535e39) compared to base (f778e6f).
Patch coverage: 0.00% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #881      +/-   ##
==========================================
- Coverage   31.27%   31.00%   -0.28%
==========================================
  Files          49       49
  Lines        6017     6077      +60
==========================================
+ Hits         1882     1884       +2
- Misses       3946     4004      +58
  Partials      189      189

Impacted Files
Coverage Δ

server/client_server.go
0.00% <0.00%> (ø)

server/subscribe.go
67.99% <0.00%> (+0.37%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

I think by making @mickmister proposed changes we're good to go 🚀

@Nityanand13  Is this finished? Please push the "request another review" button so we know we have to review again, thank you! :-)
You need to rebase, this branch has commits from master.

You need to rebase, this branch has commits from master.

@javaguirre In general I try to discourage rebasing for open PRs, specifically because it results in a force-push, which messes up the review history of the PR. If there are other commits in the PR, the PR will be squashed into one commit that we want, as long as the diff is correct with what we're trying to introduce into master.
There are currently a bunch of unrelated changes though. I'm not sure why those are showing like that if we're pointing the PR to master.

You need to rebase, this branch has commits from master.

@javaguirre In general I try to discourage rebasing for open PRs, specifically because it results in a force-push, which messes up the review history of the PR. If there are other commits in the PR, the PR will be squashed into one commit that we want, as long as the diff is correct with what we're trying to introduce into master.
There are currently a bunch of unrelated changes though. I'm not sure why those are showing like that if we're pointing the PR to master.

If there are changes in master you might need to rebase in your branch, otherwise you are not building/testing with the latest changes and you might not be doing the "right" thing (there might be commits in master that affect what you're doing in your PR).
IMHO your branch review history is your realm until you merge into a stable branch, so you could change it at will as long as the commits and changes are organized.

You need to rebase, this branch has commits from master.

@javaguirre In general I try to discourage rebasing for open PRs, specifically because it results in a force-push, which messes up the review history of the PR. If there are other commits in the PR, the PR will be squashed into one commit that we want, as long as the diff is correct with what we're trying to introduce into master.
There are currently a bunch of unrelated changes though. I'm not sure why those are showing like that if we're pointing the PR to master.

If there are changes in master you might need to rebase in your branch, otherwise you are not building/testing with the latest changes and you might not be doing the "right" thing (there might be commits in master that affect what you're doing in your PR).
IMHO your branch review history is your realm until you merge into a stable branch, so you could change it at will as long as the commits and changes are organized.

Now there are not unrelated changes.

If there are changes in master you might need to rebase in your branch, otherwise you are not building/testing with the latest changes and you might not be doing the "right" thing (there might be commits in master that affect what you're doing in your PR).
IMHO your branch review history is your realm until you merge into a stable branch, so you could change it at will as long as the commits and changes are organized.

@javaguirre I may be speaking too literally here, with the word "rebase". The PR should stay up-to-date with master, though I have the preference of "merging" master into the branch, rather than "rebasing", specifically because rebasing results in force-pushing. Sometimes rebasing makes more sense because of the nature of the changes, though typically just merging master into the branch and pushing without force-pushing is preferred, at least from the PR reviewer's side. I prefer it as a developer as well in almost all cases.
Things that GitHub provides such as the "Outdated" label on review comments, and the "Changed since last view" label when reviewing files, relies on the PR having a steady commit history, and it can't have any force-pushing. This makes past reviews much more meaningful and useful, especially on PRs that touch a lot of files. If force-pushing is done, I always re-read the whole PR, partially because of security implications. When there is a force-push, it's not possible to tell what has changed and what hasn't changed since the last review.

This PR has been automatically labelled "stale" because it hasn't had recent activity.
A core team member will check in on the status of the PR to help with questions.
Thank you for your contribution!

This PR has been automatically labelled "stale" because it hasn't had recent activity.
A core team member will check in on the status of the PR to help with questions.
Thank you for your contribution!

@DHaussermann This PR that fixes support for Jira 9 is now dev approved

/update-branch

This PR has been automatically labelled "stale" because it hasn't had recent activity.
A core team member will check in on the status of the PR to help with questions.
Thank you for your contribution!

/update-branch

We don't have permissions to update this PR, please contact the submitter to apply the update.

@Nityanand13 and @mickmister this looks like it may solve the problem. But I see a crash occur when selecting now.
To me - This looks like an issue with React Dom already patched in the production release.
Can one of you please pull master branch in so I can try again?

Also cc @hanzei are you able to sort why I can't attempt a pull master up into this branch?

@hanzei and @mickmister as this is a very high impact issue, we should consider creating a 3.2.3 dot release for this rather than getting the rest of 3.3.0 ready.
cc @asatkinson

Any chance we will get a 3.2.3 release?

Here you go: https://github.com/mattermost/mattermost-plugin-jira/releases/tag/v3.2.3


Submit all cells of a row on enter

I am using this plugin and I am having a issue. I want to use it like spreadsheet without edit button.
When i change something and enter it only sends that specific to server and all other cells are empty.
I want to send all cells on single change.. Thanks

As far as I know, you have to edit the javascript function quite a bit to get what you want.
Is there a reason you don't want it to send just the one cell. It saves data and less risk of hitten the limit of the server is smaller.
I am using this just like a spreadsheet and i am using it on single cells. I have to change a bit to make it work for my setup ( hidden fields with cell name for the database and other stuff). But compared to my google sheet its the same i see, just only one cell saving (and no reason for me to save a full row when only one thing is changed).

Thanks for your answer dicm.. Actually I have in my case one cell update effects other cells calculations and we have to update those updated cells as well. I have done this using custom ajax request for specific cells, its working fine now..  Thanks

One thing I want to ask.. Can we submit cell when user leaves the cell ? Currently we have to press enter to submit cell. Thanks

Submit on leave can be done yes.
If you look into the .js file and find the reset function (around line 260).
    reset: function(td) {
            $(td).each(function() {

Then add the submit code (This this is the code you will need to add:
Edit.submit($(this).parents('tr').find('td.tabledit-edit-mode'));
)

It worked, thanks .. I really appreciate your answers..
Thanks again..


大牛买Air780E模块还是开发版？

尴尬,小白第一次接触这个 有点懵

买开发板😂

From: an429075824 @.>
Sent: Thursday, March 16, 2023 12:33:47 AM
To: 0wQ/air780e-forwarder @.>
Cc: Subscribed @.***>
Subject: [0wQ/air780e-forwarder] 大牛买Air780E模块还是开发版？ (Issue #11)
尴尬,小白第一次接触这个 有点懵
—
Reply to this email directly, view it on GitHubhttps://github.com/0wQ/air780e-forwarder/issues/11, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AE6H2PZR75QQIH2PDNBIAYLW4HVOXANCNFSM6AAAAAAV4CWUXY.
You are receiving this because you are subscribed to this thread.Message ID: @.***>

收到 😂


Hovering Effect Improved

Hello👋, I improvised the hovering effect of your website application , Hope you like it : ) ...

Hi @Anandsg
I have improvised the hovering effect with some zoom and color changing effects..


Revert "Use Terraform for CloudFront configuration"

I'm backing out of these changes because it requires slightly more
effort that I'd originally hoped, specifically:

We can't run more than one distribution that responds to the same host
(or "alias"). For example we can't run two separate distributions that
both contain aliases for "register.register.gov.uk".
We want to refactor this so that the configuration of a CDN that
fronts a register is separate from the configuration that describes the
rest of the register infrastructure.

This reverts commit f050b9f5e7cc2aaff11d636cc65ea13384aec5d9.

🔙


backup button in the backup insights dialog not working properly

the backup button does nothing when the insights dialog is opened second
Steps to Reproduce:

Start (restart) ADS
connect to a server and open dashboard
open the backup insights dialog (show details) via the "..." of backup status widget
click the backup button on the bottom left of the dialog
the backup dialog will be opened, close it and close the backup insights dialog
open the backup insights dialog (same as step 3)
click the backup button

expected:
backup dialog opens
actual:
nothing happens

Udeesha, can you take a look at this one? It may not have directly to do with Backup.


Update source url with stable link

Hello,
I'd like to submit this change to update data in Luxembourg.
Same source as before, but up to date link.
Actually this new link is a redirect (302) to the latest up to date version of that file, so it shouldn't need any more change afterwards.
There is ~7k more addresses in that new file

Well if seems that the job fails, is it because it doesn't follow the redirect ?
Otherwise, the current up to date link without redirect is : https://download.data.public.lu/resources/adresses-georeferencees-bd-adresses/20210419-050701/addresses.geojson
But that's going to change


[Publisher] [UI] Validation message should be shown to all required fields

Description:

In Add New Document under "docs" in API publisher, validation message should be available for all required fields.
Steps
Create an API
Navigate to "Docs" section and click on Add "New Document"
Issue
Validation message is shown only in the Summary section it should show a message for the Name as well. Please refer attachment.

From https://wso2.org/jira/browse/APIMANAGER-5986

The required field validation should be implemented in the revamped UI of publisher webapp in APIM 3.0.0

Closing Since the issue has been stale for a long time.


Convert HEIC/HEIF images on upload from iOS

(Summary by @gnprice:)
Images created by recent versions of iOS are often in the HEIF format (extension .heic), which is a patent-restricted Apple format. When those images are uploaded, they tend to be unreadable elsewhere.
When uploading an image on iOS that's in HEIF format, we should use platform facilities to get or create a JPEG version and upload that.
(Original report follows.)

Hi,
I setup zulip today on a Ubuntu 16.04.4 LTS following the installation guide, so far everything looks promising. I found one issue so far or it looks like one at least.
Some pictures on my Iphone are JPG everything works, but some pictures are in HEIC format and here the latest IOS client only show a link like IMG_7113.HEIC no image, if i click the link a web view opens up loading but shows only a white page.
I see the request in the nginx logfile with a 200 status code.
xxxxxxx - - [30/Mar/2018:22:12:11 +0000] "GET /user_uploads/2/92/wPKjGi9juRThwf-r-0bW_HsO/IMG_7113.HEIC HTTP/1.1" 200 65247 "-" "Mozilla/5.0 (iPhone; CPU iPhone OS 11_3 like Mac OS X) AppleWebKit/605.1.15 (KHTML, like Gecko) Version/11.0 Mobile/15E148 Safari/604.1"
I thought maybe for the web view it would help to add the mime types but that did not change anything.
/etc/nginx/zulip-include/uploads.types
image/heic                            heic;
image/heif                            heif;
Thx
Markus

if i click on my desktop chrome on the image link the image is downloaded to my mac and preview can view this image.
I tried to upload the same picture in rocket chat and it uploads a jpg file for me it looks like i choose the same file. so it looks like the app can choose which format to upload or is it converted in the app? maybe that information helps.

@borisyankov I'd like to work on this

@faxe78 Could you send me a pic in heic or heir format ? Upload it to Drive and send me the link.

looked and found an example image.
http://nokiatech.github.io/heif/content/images/autumn_1440x960.heic
there are more examples.
http://nokiatech.github.io/heif/examples.html

We could convert the image from heic/heif to commonly used format when uploading is done, This way is followed by other sharing tools like Telegram.

Thanks @faxe78 for raising this!

We could convert the image from heic/heif to commonly used format when uploading is done, This way is followed by other sharing tools like Telegram.

Yeah, we should upload a JPEG file. Unfortunately it looks like there isn't (and won't be) free software for handling HEIF images, because it's encumbered with a tangle of patents -- so we can't do the conversion on the server.
Based on some comments on https://github.com/ivpusic/react-native-image-crop-picker/issues/611 , it sounds like some tools may already provide both a JPEG and an HEIF file, and we just have to pick the right one.
Let's focus on just doing this on iOS -- there are likely to be platform-specific facilities for it, and it should only be a problem there anyway.

@jainkuniya points out this project which aims to be a library we could use on the server: https://github.com/monostream/tifig
It's currently "in early alpha", so I wouldn't want to rely on it now. Given reports about the patent situation, I'm also skeptical that it can succeed. If at some point in the future it does and the patent situation seems to be OK, we might start using it on the server after all.

https://github.com/react-community/react-native-image-picker/issues/872#issuecomment-404439605 (and https://github.com/ivpusic/react-native-image-crop-picker/issues/611) seem to imply that react-native-image-picker does provide a jpg. You just need to use response.uri instead of response.fileName. If I upload an HEIC file from the Zulip mobile app and then download it to my desktop, I see:
<12:46:52>windcharger ~/Downloads<0>$ file IMG_0722.HEIC
IMG_0722.HEIC: JPEG image data, JFIF standard 1.01, aspect ratio, density 72x72, segment length 16, Exif Standard: [TIFF image data, big-endian, direntries=2, orientation=upper-left], baseline, precision 8, 3024x4032, frames 3

so it does appear that the file is actually a JPEG.
It looks like src/api/uploadFile.js gets the extension/mimetype from response.fileName instead of response.uri so I think the fix for this is to just get the file name and extension/mimetype information from the URI, instead.

I'm claiming this. (@zulipbot is disabled for this repository at the moment)

A note on how to reproduce a bug for this:

On iOS, go to any message list.
Upload an image taken from within the Camera application - not from launching the camera from within Zulip. That image should be in the HEIC format.
A new message should be sent, and text for the image name with an extension for the HEIC format should appear. The photo itself isn't viewable within the message list.

@faxe78 Thanks again for this detailed report! You, and others following this thread, may be glad to hear that version 17.1.98 went out to all users early this week, and has the fix.

@gnprice what did you do differently that made the HEIC image readable, did you fix it on the backend or compressed it on iOS?

@mohsinka You can see the fix in the PR #2977 that closed the issue. The diagnosis was the one by @zbenjamin above: https://github.com/zulip/zulip-mobile/issues/2178#issuecomment-421467317 We were already uploading a JPEG file, and just needed to fix the filename to reflect that, because the Zulip server infers the intended format from the filename.


(bot-discord): Add "Live Now" role to users streaming on Twitch

Check if there is some kind of event when Discord changes the presence of users with the Twitch integration enabled

presenceupdate and activity.type === ActivityType.Streaming

Check if we can get the Twitch username of users with the Twitch integration enabled, and use the Twitch API to get events
Check if polling is the best solution. We could either poll the Twitch Streams API or check users presence state on Discord

Theres some example code in this gist https://gist.github.com/PallasMedia/d2c0554d3ea6fb5fa89ea0ae75ad4886
Methods  I copied from somewhere (I don't remember where oops)

Use eventsub, Twitch will make a HTTP Post request to you when the stream goes live, you then consume the data from Twitch and convert it to a Discord Webhook and forward it on
Use eventsub, and consme it in your bot and send a message to discord (not recommened a bot shouldn’t be directly web accessable)
Use eventsub, and your server forwards the data to your bot to send to discord
Poll the streams API every minute and see if the stream changed from offline to online and the bot can send a message to Discord

Eventsub: https://dev.twitch.tv/docs/eventsub/eventsub-subscription-types#streamonline
Streams API: https://dev.twitch.tv/docs/api/reference#get-streams
Both Eventsub and HelixAPI will require the use of an App Access Token aka Client Credentials: https://dev.twitch.tv/docs/authentication/getting-tokens-oauth#oauth-client-credentials-flow 41

Closed in #4


Add MN status card

User story
As a nutritionist, I want to see and compare MN concentration in different regions of the nation.
Summary
A card showing mean MN concentration on a map, table, bar chart and Download tab.
Location
Biomarker data page

Wireframe
Maps:

Prevalence of deficiency
Prevalence of excess
Concentration data

Table

Prevalence of deficiency
Prevalence of excess
Combined deficiency and excess
Concentration Data

Tables: Prevalance of deficiency, Prevalance of deficiency (same columns for both)

Table: Combined deficiency and excess

Table: Concentration data

Table: All characteristics in one table

Columns are the same as for the tables above.
Rows as below:

Charts

Prevalence of deficiency
Prevalence of excess
Combined deficiency and excess
Concentration Data

Chart: Prevalence of deficiency and Prevalence of Excess (same structure for the chart)

Chart: Combined deficiency and excess

We can use the same bar chart as the Monthly Summary Chart in Diet Data. Each bar would have deficient, excess and adequate data for different characteristics.

The legend would show colours for deficient, excess and adequate bars. Similar as the Monthly Food chart on the diet page:

Chart: Concentration data

Download

Child Issues
Front-End

N/A

API

??

Database

??

Related Issues
Acceptance Criteria

[x] Biomarker page has the MN Status card
[x] The card heading has this structure - {{MN}} Status in {{Genger-group}}
[x] The card has four tabs - map, table, chart and download
[x] The card can be moved and resized

Map

[x] You can choose between 3different maps
[x] The map shows MN concentration using a colour ramp
[x] Map has a lenend
[ ] Map has a title above the legend

Table

[ ] You can choose data type and a characteristic
[ ] Data type dropdown options - Prevalence of deficiency, Prevalence of excess, Combined deficiency and excess, Concentration data
[ ] Characteristics dropdown options - Regions, Residence, Age group, Wealth quintiles, All characteristics

Chart

[ ] You can explore 4 different charts
[ ] Deficiency and excess show a simple bar chart
[ ] Combined deficiency and excess show bar chart with 2 bars for each characteristic
[ ] Concentration data shows box-and-whisker plot

Download

[ ] Options to download Deficiency, Excess, Combined deficiency and excess and concentration data
[ ] You download Map, Tables, Bar chart and CSV files (CSV is of less importance, could come later)
[ ] You can read more about each file type (e.g. data formats and what's included in the download)

@DanielLyleWilliamWarren I think most of the above tasks have been completed, but might want to keep this open until API data is available to test/tweak functionality, or close and create new issue.

Closing for now.  Will re-evaluate when biomarker data and requirements settled.


Jest 25.5.2 uses deprecated dependency

🐛 Bug Report
When installing the latest version (25.5.2) npm prints warning:
npm WARN deprecated<EMAIL_ADDRESS>request has been deprecated
To Reproduce
try install the 25.5.2 version
Expected behavior
Clear npm install output with no warnings

https://github.com/jsdom/jsdom/issues/2792