Letter customization

#428 #529

@paramsiddharth I need to review it locally.

@DietBepis1 Hi Glenn, just wanted to share the screen recording for the error that is coming up with current code.
https://user-images.githubusercontent.com/843538/232210484-b5d812a3-9acd-466d-84aa-e5060c1d24db.mp4
Also, did some analysis with help from @yoyoyojoe on ways to resolve the error. Essentially, we made two main changes:

Removed persistCustomization() function from LetterLoad.vue file and committed userCustomMessage within the watch option.
Removed this.$refs.letterLoad.persistCustomization() from TakeAction.vue file.

Please see the screen recording below to see the flow after these changes. Notice userCustomMessage getting added in this.$store.state
https://user-images.githubusercontent.com/843538/232214448-09934154-407b-4b46-a8ee-16e323355d7b.mp4
I can add a commit for you to see the code changes if this is something you would like me to proceed with. Please let me know if you would recommend another approach. Looking forward to your feedback. Thank you.

@DietBepis1 Hi Glenn, just wanted to share the screen recording for the error that is coming up with current code.
Also, did some analysis with help from @yoyoyojoe on ways to resolve the error. Essentially, we made two main changes:
* Removed persistCustomization() function from LetterLoad.vue file and committed userCustomMessage within the watch option.

* Removed this.$refs.letterLoad.persistCustomization() from TakeAction.vue file.

Please see the screen recording below to see the flow after these changes. Notice userCustomMessage getting added in this.$store.state
I can add a commit for you to see the code changes if this is something you would like me to proceed with. Please let me know if you would recommend another approach. Looking forward to your feedback. Thank you.

@SAUMILDHANKAR and @yoyoyojoe, just wanted to say that this was excellent collaboration and a great demonstration. 👍🏽

@paramsiddharth and @SAUMILDHANKAR, how did this PR get branched off of the original one from Sammy? The reason I am asking is that I noticed a missing prop repName error in the videos posted in the last few comments. That prop is removed in our main branch. The PR itself looks good and ready to merge, but I am worried that we may introduce bugs into the main codebase since I can see this unused prop.

@paramsiddharth and @SAUMILDHANKAR, how did this PR get branched off of the original one from Sammy? The reason I am asking is that I noticed a missing prop repName error in the videos posted in the last few comments. That prop is removed in our main branch. The PR itself looks good and ready to merge, but I am worried that we may introduce bugs into the main codebase since I can see this unused prop.

@DietBepis1 Saumil branched it off Sam's PR's branch into a branch in this repository – The prop must have been introduced while resolving the conflicts. It is something we'll have to pair up on, I believe. Perhaps outside the pair hours, some dedicated time.

@paramsiddharth and @SAUMILDHANKAR, how did this PR get branched off of the original one from Sammy? The reason I am asking is that I noticed a missing prop repName error in the videos posted in the last few comments. That prop is removed in our main branch. The PR itself looks good and ready to merge, but I am worried that we may introduce bugs into the main codebase since I can see this unused prop.

@DietBepis1 Saumil branched it off Sam's PR's branch into a branch in this repository – The prop must have been introduced while resolving the conflicts. It is something we'll have to pair up on, I believe. Perhaps outside the pair hours, some dedicated time.

+1 to what Param just mentioned. Also, wanted to share that the workflow looked fine when I removed repName in my local environment and the error went away as well.


can't install on OSX

Using the latest npm/node

<EMAIL_ADDRESS>install /usr/local/lib/node_modules/zmq
node-gyp rebuild

/bin/sh: pkg-config: command not found
gyp: Call to 'pkg-config libzmq --libs' returned exit status 127.
gyp ERR! configure error
gyp ERR! stack Error: gyp failed with exit code: 1
gyp ERR! stack     at ChildProcess.onCpExit (/usr/local/lib/node_modules/npm/node_modules/node-gyp/lib/configure.js:337:16)
gyp ERR! stack     at ChildProcess.EventEmitter.emit (events.js:98:17)
gyp ERR! stack     at Process.ChildProcess._handle.onexit (child_process.js:797:12)
gyp ERR! System Darwin 13.1.0
gyp ERR! command "node" "/usr/local/lib/node_modules/npm/node_modules/node-gyp/bin/node-gyp.js" "rebuild"
gyp ERR! cwd /usr/local/lib/node_modules/zmq
gyp ERR! node -v v0.10.26
gyp ERR! node-gyp -v v0.12.2
gyp ERR! not ok
npm ERR<EMAIL_ADDRESS>install: node-gyp rebuild
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>install script.
npm ERR! This is most likely a problem with the zmq package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     node-gyp rebuild
npm ERR! You can get their info via:
npm ERR!     npm owner ls zmq
npm ERR! There is likely additional logging output above.

Thanks:)

I am getting this one after brew install pkg-config:

Package libosrm was not found in the pkg-config search path.


refactor(root): a slightly more concise definition borrowed from Angular

Description:
Related issue (if exists):

Coverage increased (+0.06%) to 97.703% when pulling 67a8354d6550240c7e97dc2dff57ec7b74d065b8 on benlesh:smaller_root into ebf6393f48347fd8b038a248ae8479a459e375a2 on ReactiveX:master.

so we're not throwing anymore if root is not resolvable?

I'm not sure there is a situation where root isn't resolvable, honestly. That was just some defensive coding. I'm indifferent, I'll leave it to the community.

I wouldn't expect this occurs in usual cases, but there were few exceptional cases patching globals around could be caught by thrown exception. So for me it's good to have, it allowed us to identify some unexpected behavior comes from it.


NIFI-10287 ExecuteScript - Allow python scripts to use external modules

Summary
NIFI-10287
This resolves a bug that crept in where the ExecuteScript processor used to support using external modules for python, but has lost that functionality.  This has been fixed in this ticket.
Tracking
Please complete the following tracking steps prior to pull request creation.
Issue Tracking

[x] Apache NiFi Jira issue created

Pull Request Tracking

[x] Pull Request title starts with Apache NiFi Jira issue number, such as NIFI-00000
[x] Pull Request commit message starts with Apache NiFi Jira issue number, as such NIFI-00000

Pull Request Formatting

[x] Pull Request based on current revision of the main branch
[x] Pull Request refers to a feature branch with one commit containing changes

Verification
Please indicate the verification steps performed prior to pull request creation.
Build

[x] Build completed using mvn clean install -P contrib-check

[x] JDK 8
[x] JDK 11
[x] JDK 17

Licensing

[x] New dependencies are compatible with the Apache License 2.0 according to the License Policy
[x] New dependencies are documented in applicable LICENSE and NOTICE files

Documentation

[x] Documentation formatting appears as expected in rendered files

LGTM+1

Thank you very much @dan-s1 for the review and for volunteering the unit test for the use case that was missed!
Squashed commits after approval to make it easier for additional reviewers to see code and to capture attribution for @dan-s1's generous contribution.

Unit test looks good. I built the code (including -Pcontrib-check), and all looks good. I tested by instantiating an ExecuteScript processor. It was configured with a module directory with a .py module file similar to the one used in the unit test. Ran data and it worked like a charm. I changed the module code and re-ran a flowfile (without stopping the processor.) The changes were not reflected in the processed flowfile. However, restarting the processor picks up the module update. I believe this is expected behavior since we do not want to reload modules (or even check if they should be reloaded) on every flowfile; I believe that would be overly burdensome.
Overall, looks good to merge to me. Thanks @NissimShiman !

@mattyb149 @exceptionfactory @MikeThomsen  I removed the unit test that was concerning so I think we are good.  The unit test for the main use case of the ticket remains.  Please let me know if there are other holdup related concerns.

+1 LGTM, thanks for the improvement and sticking with this! Merging to main

Thank you @mattyb149 !


Feature improvement in exit

Reconfirm the exit if the user enters no in the exit message.

I'll do that soon.

It is done!


Add missing LDAP_DOMAIN substitution to custom LDIF README

This is already documented in the main README file but is missing from this one.

Thanks :)

Thanks :)


Rename the package/namespaces from okahu to monocle

Change package/namespaces from okahu to monocle
Remove remaining reference to okahu settings and repo references
Change the file/folder names from okahu to monocle

@kshitiz-okahu, I guess we can close it per your last PR?


Not able to select files which don't end with '.py'

I have a project, where some python files/scripts do not end with '.py'.
Is it possible to include these files in my pytest.ini/pylama.ini
I tried pylama by explicitely selecting the file e.g. pylama path/to/file, but pylama still ignores it.
When I rename the file to have a '.py' suffix pylama works.

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/30651408-not-able-to-select-files-which-don-t-end-with-py?utm_campaign=plugin&utm_content=tracker%2F394650&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F394650&utm_medium=issues&utm_source=github).

Hello, try to use pylama --force


feat: UI redesign

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


Update build system to upload JARs to jcenter (or some other distro)

Update build system to upload JARs to jcenter (or some other distro)

CC @xster @blasten


Add Comments and Discussion Feature for Code Snippets

Description: Enable users to leave comments, ask questions, or provide feedback on code snippets, fostering a community around each snippet.

Functionality: Allow users to post comments under snippets. Enable replies to comments and allow the snippet author to moderate or delete inappropriate comments.
Additional Features: Provide notifications to snippet owners when comments are posted.
Implementation: Create a comment model and API to store and fetch comments. Use Clerk for user authentication and integrate comment functionality on the snippet detail page.

@notsoocool  Can you assign me these one
I want to give a try to create and get comments.i worked with clerk in the past as well.

Hi @Vivek7038,
Thank you for your interest in working on this feature! Since you have experience with Clerk, you can definitely take this one on. I'll go ahead and assign it to you. Feel free to reach out if you have any questions during implementation or need any clarification.
Looking forward to seeing your contributions!

@notsoocool  i am taking pull from main branch and then creating a new branch from main.
Is these works or i have to create new branch from develop

@Vivek7038 the best practice would be to pull from the develop branch instead of the main branch. We’ve made some changes in the develop branch that aren’t available in the main branch, and starting from develop will help prevent any potential conflicts down the line.

@notsoocool how the newly added snippets approval work , currently its getting added to snippetrequests collection, how to approve it

Basically i want to experiement adding comments on the dummy snippets which i created.

okay, now check .env.sample file in the develop branch, there i added the field for
ADMIN_USER_IDS=
here just replace the value with you clerk user id.
so what you have to do is open the clerk app and check for your email and copy the user id and paste it here

Quick updates :
Post and get api for comments is sorted and working perfect .
Remaining thing is UI for showing comments and adding comments

raised a pr #37

raised a pr #37
@Vivek7038
Thank you for raising the PR! I'll make sure to review it as soon as I get some time. Appreciate your contribution!


Aric sync merge

Added a pull-branch-strategy configuration parameter which is set using:
git town config --pull-strategy [merge | rebase]
which is used for when the branches are synced

Code looks good but we still need feature tests for this. Both for configuring the rebase strategy and using it. I can help you setup some feature tests next week, when I get back from vacation. @kevgo may be able to help as well.

@aricahunter this looks pretty good! Let's see how it looks like with the last 2 comments addressed!

I believe the interface should be updated to
git town pull-strategy [(merge | rebase)]

where

git town pull-strategy outputs the pull strategy
git town pull-strategy merge sets the pull strategy to merge
git town pull-strategy rebase sets the pull strategy to rebase

Hey @aricahunter! I have added some basic integration tests for your feature in https://github.com/Originate/git-town/commit/b5ed564647e1743d059263f2eb30d925c67940aa.
You can run them via cucumber features/git-sync/current_branch/feature_branch/no_conflict/with_merge_pull_strategy.feature and cucumber features/git-town/configuration/pull_strategy/merge_option.feature.
Hopefully this gives you enough momentum to add some more tests to cover more edge cases, like setting the merge strategy to another value, and how git-sync behaves in different situations with the "merge" pull strategy set. Feel free to ask questions if you get stuck!

@kevgo I took this over and cleaned it up. Please review when you get a chance.
@aricahunter thanks for your help

Nice!! Thanks for doing this!  :ship:

@kevgo. Addressed comments and then added in documentation that I nearly forgot. Please review the documentation updates (especially the README).

Ah, the documentation! I added some suggestions. Feel free to ship if you are happy with the outcome!

Comments sound good, :ship:


[docs] Add a new demo to the showcase

[x] I have followed (at least) the PR section of the contributing guide.

@adonig Great job with the integration of Material-UI with Gatsby. I wonder about the sustainability of the shop. It doesn't seem to receive any significant traffic yet. I would propose we wait 6 months before moving forward, so we can at least get one of the boxes checked in our selection criterions.

@oliviertassinari Thank you! I was hoping that a listing in the showcase could help to give it some page authority. It's kind of hard to get backlinks when you're on a $0 marketing budget 😅

Yeah, definitely, I hope you will find good SEO leverages to draw traffic :)

⚠

@adonig Went to bat for you on this, as I think it's polished and non-material enough to qualify. If you could submit an image that is a bit smaller, ideally around 100K (e.g. reduced the quality slightly - I think we use 70% for the others), that would be great. If you're on a Mac, you can also run it through ImageOptim, or PixelMator's "Export for web".

@mbrookes Thank you very much, you are amazing! I pushed a new commit with an optimized image that is around 60kb.

@adonig The previous size of the image was correct, we expect 1200x900, not 900 x 750.

@oliviertassinari Thank you. I pushed another commit with a 1200x900 image having 70% compression. I'm not sure why the argos check failed. I think it is probably unrelated.

@adonig This will go live on next.material-ui.com with the next release of v5 alpha. If you'd like to add it on material-ui.com, you cab backport this to the master branch. Releases are less frequent, but it will be sooner than v5!


style: preserve sidebar height when toggling

Pre-flight checklist

[x] I have read the Contributing Guidelines on pull requests.
[x] If this is a code change: I have written unit tests and/or added dogfooding pages to fully verify the new behavior.
[x] If this is a new API or substantial change: the PR has an accompanying issue (closes #0000) and the maintainers have approved on my working plan.

Motivation

This PR makes the heights of the sidebar & the expand button fixed (100vh). This solves the layout shift.
Test Plan
https://user-images.githubusercontent.com/8978815/194794055-08d03963-18b4-4c89-96fd-8369c2b46a8e.mp4

Go to a page where the sidebar height is greater than the main content height (an example is linked below). Try hiding/expanding the sidebar to make sure that there is no layout shift.
Test links

Deploy preview: https://deploy-preview-8195--docusaurus-2.netlify.app/tests/docs
Same page on prod: https://docusaurus.io/tests/docs
Related issues/PRs

Solves #4415

This seems to fix the issue indeed but introduce a new one.
If a doc has a small sidebar + small content, it still takes 100vh making it less clear that scrolling will reveal a footer
Before:

After:

That can be particularly confusing for vertical screens with a lot of vh IMHO 🤷♂


Problem with MemoryDistributedCache

I have problems with the fact that MemoryDistributedCache (IDistributedCache) creates another instance during service startup in .net core 2.0.
In ConfigureServices i'm calling
public void ConfigureServices(IServiceCollection services)
        {
            var logger = loggerFactory.CreateLogger("Test");

            if (enviroment.IsDevelopment() || enviroment.IsEnvironment("debug"))
            {
                logger.LogInformation($"Connecting to InMemory cache");
                // memory cache for developement
                services.AddDistributedMemoryCache();
            }
            else
            {
                // using Redis cache for production
            }

But later in the same method i need to get an instance of 1 of my components to use it for cookie Authentication methods
var sp = services.BuildServiceProvider();
            // Resolve the services from the service provider
            var cookieAuth = sp.GetService<CookieAuthenticationMethods>();

            var authentication = services.AddAuthentication(o =>
            {
                o.DefaultChallengeScheme = CookieAuthenticationDefaults.AuthenticationScheme;
                o.DefaultSignInScheme = CookieAuthenticationDefaults.AuthenticationScheme;
                o.DefaultAuthenticateScheme = CookieAuthenticationDefaults.AuthenticationScheme;
            });

            authentication.AddCookie(o =>
            {
                o.LoginPath = new PathString("/api/User/Login");
                o.SlidingExpiration = true;
                o.ExpireTimeSpan = settings.SessionExpireTime;
                o.Events = new CookieAuthenticationEvents
                {
                    OnRedirectToLogin = cookieAuth.RedirectToLogin,
                    OnValidatePrincipal = cookieAuth.ValidateAsync
                };
                o.Cookie.SecurePolicy = enviroment.IsDevelopment() || enviroment.IsEnvironment("debug") ? CookieSecurePolicy.SameAsRequest : CookieSecurePolicy.Always;
            });

CookieAuthenticationMethods - is implementation class for methods used to custom user validation
public class CookieAuthenticationMethods
    {
        private readonly IDistributedCache cache;
        private readonly ISettings settings;
        private readonly ILogger logger;

        /// <summary>
        /// Default constructor
        /// </summary>
        public CookieAuthenticationMethods(IDistributedCache distributedCache, ISettings settings, ILoggerFactory loggerFactory)
        {
            logger = loggerFactory.CreateLogger("Engine");
            cache = distributedCache;
            this.settings = settings;
        }
      public async Task ValidateAsync(CookieValidatePrincipalContext context)
      {}
      public Task RedirectToLogin(RedirectContext<CookieAuthenticationOptions> ctx)
      {}
}

And as you can see i'm getting IDistributedCache via DI.
And the problem is, that at this point 1 instance of MemoryDistributedCache (IDistributedCache) is created.
To be precise it creating during call var cookieAuth = sp.GetService<CookieAuthenticationMethods>();
And another instance of MemoryDistributedCache (IDistributedCache) is created during call for method WebHostBuilder.Run(); at Main method in Program.cs
public static void Main(string[] args)
        {
            var environmentVariable = Environment.GetEnvironmentVariable("ASPNETCORE_ENVIRONMENT");
            var builder = new ConfigurationBuilder()
                .SetBasePath(Directory.GetCurrentDirectory())
                .AddJsonFile("appsettings.json", false, true)
                .AddJsonFile($"appsettings.{environmentVariable}.json", true)
                .AddEnvironmentVariables();
            var configuration = builder.Build();
            var defaultAddress = configuration["DefaultAddress"] ?? "http://*:2001/";

            var host = new WebHostBuilder()
                .UseKestrel()
                .UseUrls(defaultAddress)
                .UseContentRoot(Directory.GetCurrentDirectory())
                .ConfigureAppConfiguration((hostingContext, config) =>
                {
                    var env = hostingContext.HostingEnvironment;
                    config.AddJsonFile("appsettings.json", true, true)
                        .AddJsonFile($"appsettings.{env.EnvironmentName}.json", true, true);
                    config.AddEnvironmentVariables();
                })
                .ConfigureLogging((hostingContext, logging) =>
                {
                    logging.AddConfiguration(hostingContext.Configuration.GetSection("Logging"));
                    logging.AddLog4Net();
                    logging.AddDebug();
                })
                .UseSetting(WebHostDefaults.DetailedErrorsKey, "true")
                .UseStartup<Startup>()
                .Build();

            host.Run();
        }

P.S.
I tried to insert another singleton class into CookieAuthenticationMethods but it was created only in 1 instance, so i guest this behavior is specific only to MemoryDistributedCache (IDistributedCache)

Yes that is expected behavior here because you are explicitly calling services.BuildServiceProvider(). The host builder also calls BuildServiceProvider later on when building the app. Each Service Provider built is a completely indepdendent DI world so Singletons are not shared between instances. You should not be explicitly creating a DI container in your ConfigureServices method.
Instead, you can configure your Cookie Auth with wrapper functions that use the HttpContext to get an instance of your DI services during the event itself. The HttpContext.RequestServices property gives you access to the DI container for the application. For example:
o.Events = new CookieAuthenticationEvents
{
    OnRedirectToLogin = (eventContext) => eventContext.HttpContext.RequestServices.GetRequiredService<CookieAuthenticationMethods>().RedirectToLogin(eventContext),
	// ... etc ...
};

cc @Tratcher in case there's a more DI-native way to do this kind of thing.

https://github.com/aspnet/AspNetCore/blob/85feca7a12324d07bc00d8acfc3ebb7f05045db5/src/Security/Authentication/Core/src/AuthenticationSchemeOptions.cs#L36-L39

Perfect, so @akarimovbytewerk if instead of setting the events directly, you configure the EventsType property, the auth services will handle activating the type out of DI and you'll stay within the same DI world.
For example:
            authentication.AddCookie(o =>
            {
                o.LoginPath = new PathString("/api/User/Login");
                o.SlidingExpiration = true;
                o.ExpireTimeSpan = settings.SessionExpireTime;
                o.EventsType = typeof(CookieAuthenticationMethods); // NEW!
                o.Cookie.SecurePolicy = enviroment.IsDevelopment() || enviroment.IsEnvironment("debug") ? CookieSecurePolicy.SameAsRequest : CookieSecurePolicy.Always;
            });

That works just fine. Thanks a lot.


gitops: init at 0.14.1

Description of changes

New package: gitops
URL: https://docs.gitops.weave.works
Github: https://github.com/weaveworks/weave-gitops

Extra notes:
Makefile for gitops package included a bunch of ldflags. I've included them, but it is unclear if they are actually necessary for this bin.
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 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
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Already done in #231426


eas build -p ios messes up apple app identifier

Build/Submit details page URL
No response
Summary
Eas build -p ios warns that provisioning profile is no longer valid and asks if you want to reuse the original profile and you can answer yes or no, but it will update/generate provisioning profile anyway and it results in settings change in your app's identifier at developer.apple.com. EAS should not mess around with this settings, I cannot build and upload my app to app store because of this issue, I have to use old expo build:ios.
Managed or bare?
Managed
Environment
expo-env-info 1.0.2 environment info:
System:
OS: Linux 5.13 Linux Mint 20.3 (Una)
Shell: 5.0.17 - /bin/bash
Binaries:
Node: 16.14.0 - ~/.nvm/versions/node/v16.14.0/bin/node
npm: 8.4.1 - ~/.nvm/versions/node/v16.14.0/bin/npm
IDEs:
Android Studio: AI-211.7628.21.2111.8092744
npmPackages:
expo: ^44.0.6 => 44.0.6
react: ^17.0.2 => 17.0.2
react-dom: ^17.0.2 => 17.0.2
react-native: 0.64.3 => 0.64.3
react-native-web: ~0.17.7 => 0.17.7
npmGlobalPackages:
eas-cli: 0.48.2
expo-cli: 5.3.0
Expo Workflow: managed
Error output
✔ Fetched Apple provisioning profiles
Provisioning profile (id: Y8J48D83ML) is no longer valid
✔ Fetched Apple provisioning profiles
✔ Provisioning Profile - ID: Y8J48D83ML
Name: *[expo] com.app.app AppStore 2022-03-24T22:47:34.269Z
Expiry: Sun Feb 26 2023
Would you like to reuse the original profile? … yes
✔ Fetched Apple provisioning profiles
✔ Updated provisioning profile (5VN7SZYV8L) with distribution certificate (TW8T4ZUN8G)
Reproducible demo or steps to reproduce from a blank project

Set up app identifier at developer.apple.com;
Run eas build -p ios;
Go to developer.apple.com and see that your identifier settings was changed by eas build.

I'm not sure what you mean by changing identifiers? The only thing besides generating credentials that cli is doing is to synchronize capabilities in appstore for your app id with entitlements configured in app.json (or dependencies that support it).
You can disable that behavior with env EXPO_NO_CAPABILITY_SYNC=1
https://docs.expo.dev/build-reference/ios-capabilities/

Yes, that was my problem. I used usesIcloudServices, fact that it works in bare workflow only remained unnoticed by me until now. I added:

"entitlements": {
"com.apple.developer.icloud-services": ["CloudKit"],
"com.apple.developer.icloud-container-environment": "Production",
"com.apple.developer.icloud-container-identifiers": ["iCloud.com.app.app"]
}

and now it works as expected. But it's still warns that Provisioning_ profile (id: LZZB55VR9Y) is no longer valid all the time and asks if I want to reuse the original profile. Is it normal?

But it's still warns that Provisioning_ profile (id: LZZB55VR9Y) is no longer valid all the time and asks if I want to reuse the original profile. Is it normal?

does it still happen if you answer no when it asks you to reuse the existing profile?

But it's still warns that Provisioning_ profile (id: LZZB55VR9Y) is no longer valid all the time and asks if I want to reuse the original profile. Is it normal?

does it still happen if you answer no when it asks you to reuse the existing profile?

If I reply no, it asks if I want to generate new provisioning profile, y -> generates new profile, n -> asks for manualy generated profile. No matter what I answer the warning persists

It shouldn't be happening, but as far as I remember those icloud entitlements are not fully supported, so it looks like every time you run build it's updating app id which requires you to update profile. If you just snwear no when asked to login to apple it should work
@EvanBacon Just to make sure, is this expected behavior?
If I add those icloud sentiments like @ibrag474 above app id is updated every time, I'm getting
✔ Synced capabilities: Enabled: iCloud
✔ Synced capability identifiers: Linked: iCloud.com.app.app

Any news on that one ?
Seems that it's linked to iCloud use.
I'm using expo DocumentPicker.

I've got "usesicloudstorage": true in my app.json
I also have this configured in my app identifier :

--> And in the pop-up (Oppened by clicking "Edit") With iCloud.<your_bundle_identifier> as mentionned in the doc :

But still facing that provisionning profile invalidation 😢

@ibrag474 Why did you close this ? 🤔

@seba9999 @wkozyra95, a new post for this issue can be found at https://github.com/expo/eas-cli/issues/1073. You will have until 4/25 before https://github.com/expo/eas-cli/issues/1030 is deleted. Sorry for the inconvenience!


How to add focal loss to baseline?

There are only CrossEntropyLoss, TripletLoss and CircleLoss in meta_arch/baseline.py.
How to add FocalLoss here?
Thanks.

        if "CrossEntropyLoss" in loss_names:
            loss_dict['loss_cls'] = cross_entropy_loss(
                cls_outputs,
                gt_labels,
                self._cfg.MODEL.LOSSES.CE.EPSILON,
                self._cfg.MODEL.LOSSES.CE.ALPHA,
            ) * self._cfg.MODEL.LOSSES.CE.SCALE

        if "TripletLoss" in loss_names:
            loss_dict['loss_triplet'] = triplet_loss(
                pred_features,
                gt_labels,
                self._cfg.MODEL.LOSSES.TRI.MARGIN,
                self._cfg.MODEL.LOSSES.TRI.NORM_FEAT,
                self._cfg.MODEL.LOSSES.TRI.HARD_MINING,
            ) * self._cfg.MODEL.LOSSES.TRI.SCALE

        if "CircleLoss" in loss_names:
            loss_dict['loss_circle'] = circle_loss(
                pred_features,
                gt_labels,
                self._cfg.MODEL.LOSSES.CIRCLE.MARGIN,
                self._cfg.MODEL.LOSSES.CIRCLE.ALPHA,
            ) * self._cfg.MODEL.LOSSES.CIRCLE.SCALE

        if "FocalLoss" in loss_names:
            loss_dict['loss_focal'] = focal_loss(
                pred_features,
                gt_labels,
                self._cfg.MODEL.LOSSES.FC.ALPHA,
                self._cfg.MODEL.LOSSES.FC.GAMMA,
            ) * self._cfg.MODEL.LOSSES.FC.SCALE

        return loss_dict

I used this and worked. Maybe this changed will help you

Thanks!
But FC should be FL

Thanks!
But FC should be FL.

Yess, my mistake :)

        if "CrossEntropyLoss" in loss_names:
            loss_dict['loss_cls'] = cross_entropy_loss(
                cls_outputs,
                gt_labels,
                self._cfg.MODEL.LOSSES.CE.EPSILON,
                self._cfg.MODEL.LOSSES.CE.ALPHA,
            ) * self._cfg.MODEL.LOSSES.CE.SCALE

        if "TripletLoss" in loss_names:
            loss_dict['loss_triplet'] = triplet_loss(
                pred_features,
                gt_labels,
                self._cfg.MODEL.LOSSES.TRI.MARGIN,
                self._cfg.MODEL.LOSSES.TRI.NORM_FEAT,
                self._cfg.MODEL.LOSSES.TRI.HARD_MINING,
            ) * self._cfg.MODEL.LOSSES.TRI.SCALE

        if "CircleLoss" in loss_names:
            loss_dict['loss_circle'] = circle_loss(
                pred_features,
                gt_labels,
                self._cfg.MODEL.LOSSES.CIRCLE.MARGIN,
                self._cfg.MODEL.LOSSES.CIRCLE.ALPHA,
            ) * self._cfg.MODEL.LOSSES.CIRCLE.SCALE

        if "FocalLoss" in loss_names:
            loss_dict['loss_focal'] = focal_loss(
                pred_features,
                gt_labels,
                self._cfg.MODEL.LOSSES.FL.ALPHA,
                self._cfg.MODEL.LOSSES.FL.GAMMA,
            ) * self._cfg.MODEL.LOSSES.FL.SCALE

        return loss_dict

I used this and worked. Maybe this changed will help you

        if "CrossEntropyLoss" in loss_names:
            loss_dict['loss_cls'] = cross_entropy_loss(
                cls_outputs,
                gt_labels,
                self._cfg.MODEL.LOSSES.CE.EPSILON,
                self._cfg.MODEL.LOSSES.CE.ALPHA,
            ) * self._cfg.MODEL.LOSSES.CE.SCALE

        if "TripletLoss" in loss_names:
            loss_dict['loss_triplet'] = triplet_loss(
                pred_features,
                gt_labels,
                self._cfg.MODEL.LOSSES.TRI.MARGIN,
                self._cfg.MODEL.LOSSES.TRI.NORM_FEAT,
                self._cfg.MODEL.LOSSES.TRI.HARD_MINING,
            ) * self._cfg.MODEL.LOSSES.TRI.SCALE

        if "CircleLoss" in loss_names:
            loss_dict['loss_circle'] = circle_loss(
                pred_features,
                gt_labels,
                self._cfg.MODEL.LOSSES.CIRCLE.MARGIN,
                self._cfg.MODEL.LOSSES.CIRCLE.ALPHA,
            ) * self._cfg.MODEL.LOSSES.CIRCLE.SCALE

        if "FocalLoss" in loss_names:
            loss_dict['loss_focal'] = focal_loss(
                pred_features,
                gt_labels,
                self._cfg.MODEL.LOSSES.FL.ALPHA,
                self._cfg.MODEL.LOSSES.FL.GAMMA,
            ) * self._cfg.MODEL.LOSSES.FL.SCALE

        return loss_dict

I used this and worked. Maybe this changed will help you

Thanks for your suggestion. I wonder if focal loss will perform any better than cross entropy loss? Is it required to tune alpha and gamma?

@howieraem I has tried focal loss and the result was worse than (cross entropy loss + triplet loss) as in default. But you should try tune alpha and gammar for use focal loss with other loss for experiment better result.


CfakeRipperTest

Category
This change is exactly one of the following (please change [ ] to [x]) to indicate which:

[x] a test

Testing
Optional but recommended:

[x] Added unit test

Coverage increased (+0.2%) to 30.898% when pulling c938916339882918a214b3477f6f7e94942bb223 on metaprime:test-cfake into 927cb286dab6ec5758237dd66b5ed956d1b712c4 on RipMeApp:master.


Flux Store Snippet

I've put together a quick Flux Store snippet: (assumes App dispatcher is located at ../../ from Store)
"React: flux store skeleton":
    prefix: "fxs"
    body: "var Dispatcher = require('../../ApplicationDispatcher.js');\nvar EventEmitter = require('events').EventEmitter;\nvar assign = require('object-assign');\n\nvar ${1} = assign({}, EventEmitter.prototype, {\n\n\n\n\temitChange: function() {\n\t\tthis.emit('change');\n\t},\n\n\taddChangeListener: function(callback) {\n\t\tthis.on('change', callback);\n\t},\n\n\tremoveChangeListener: function(callback) {\n\t\tthis.removeListener('change', callback);\n\t}\n});\n\n${1}.dispatchToken = ApplicationDispatcher.register(function(payload) {\n\tvar action = payload.action, data = payload.action.data;\n});\n\nmodule.exports = ${1};"

Hi!
I try to keep this project clean of any non-react related stuff and tough Flux is directly related to React I still consider it to be out of the scope of this project. Also I think https://atom.io/packages/react-snippets might already have some flux snippets.


Docker login fail with helm installed harbor.

Docker :  18.03.1-ce, build 9ee9f40
Minikube version: v0.28.0
Helm : "v2.9.1"
I followed the helm installation guide on master branch commit e51d6cc707c83e9eea36ece0d2f9d66b1a71fac8
The Harbor web works fine, but when I try to login the registry it return me some error like this.
sudo docker login harbor.my.domain
Username (admin): admin
Password:
Error response from daemon: Get https://harbor.my.domain/v2/: Get https://harbor.my.domain:32700/service/token?account=admin&client_id=docker&offline_token=true&service=harbor-registry: dial tcp <IP_ADDRESS>:32700: getsockopt: connection refused

And here's the error part of the log of registry pod
kubectl log my-release-harbor-registry-0
<IP_ADDRESS> - - [26/Jun/2018:09:01:50 +0000] "GET /v2/_catalog?n=1000 HTTP/1.1" 200 20 "" "Go-http-client/1.1"
time="2018-06-26T09:04:37.249138662Z" level=warning msg="error authorizing context: authorization token required" go.version=go1.7.3 http.request.host=harbor.my.domain http.request.id=4e4f44aa-8933-44fc-a726-e153413d41ad http.request.method=GET http.request.remoteaddr=<IP_ADDRESS> http.request.uri="/v2/" http.request.useragent="docker/18.03.1-ce go/go1.9.5 git-commit/9ee9f40 kernel/4.4.0-128-generic os/linux arch/amd64 UpstreamClient(Docker-Client/18.03.1-ce \\(linux\\))" instance.id=e11bfd4c-b6e7-4443-9dae-3ff808b53b0d service=registry version=v2.6.2
<IP_ADDRESS> - - [26/Jun/2018:09:04:37 +0000] "GET /v2/ HTTP/1.1" 401 87 "" "docker/18.03.1-ce go/go1.9.5 git-commit/9ee9f40 kernel/4.4.0-128-generic os/linux arch/amd64 UpstreamClient(Docker-Client/18.03.1-ce \\(linux\\))"

I even tried the commit dd4c565aff34a941db2c4c2113b0e313ba76a3e1
but also, same error happened.
BTW, I was able to run harbor helm about 2 weeks ago with commit c13f2f6948c32738d1b5c48b417ce5c80327e950
Both website and the docker CLI works fine in this old version.

Leave the externalPort  empty if your ingress controller is bound to port 80/443 in values.yaml https://github.com/vmware/harbor/blob/master/contrib/helm/harbor/values.yaml#L38.

This note might added to the installation guide of the helm since it's default environment is based on minikube, which's ingress controller is bound to port 80/443.

Open #5297 to track the doc issue. Closing this one.


400 "Bad Request" when running check-rabbitmq-queue.rb

I had to upgrade my rabbit server to ubuntu 14.04 and now I getting the following error when running this command:
ruby /etc/sensu/plugins/check-rabbitmq-queue.rb --h localhost --user  --password  --queue queuename -w 10 -c 20
Check failed to run: 400 "Bad Request", ["/usr/lib/ruby/2.3.0/net/http/response.rb:120:in error!'", "/var/lib/gems/2.3.0/gems/carrot-top-0.0.7/lib/carrot-top.rb:59:in fetch_uri'", "/var/lib/gems/2.3.0/gems/carrot-top-0.0.7/lib/carrot-top.rb:24:in query_api'", "/var/lib/gems/2.3.0/gems/carrot-top-0.0.7/lib/carrot-top.rb:28:in method_missing'", "/etc/sensu/plugins/check-rabbitmq-queue.rb:106:in run'", "/var/lib/gems/2.3.0/gems/sensu-plugin-2.3.0/lib/sensu-plugin/cli.rb:57:in block in class:CLI'"]
ruby version: ruby 2.3.5p376 (2017-09-14 revision 59905) [x86_64-linux-gnu]
gem version: <IP_ADDRESS>
I have sensu-plugin, cli, check and carrot-top gems installed. (I couldn't find "socket" to install but looking at my old server, it doesnt have that installed either).
Any help is appreciated.

Thanks for reporting this and sorry about your issue. I will try to replicate this tonight some time. Regarding socket it comes from the standard library: https://ruby-doc.org/stdlib-2.4.2/libdoc/socket/rdoc/index.html

Hmm I am getting a different error:
$ bundle exec ./bin/check-rabbitmq-queue.rb --host <IP_ADDRESS> --port 15671 --user sensu --password <REDACTED> --queue sensu -w 10 -c 20
Check failed to run: end of file reached, []

@rthouvenin any chance you could take a look? It looks like it is coming from https://github.com/sensu-plugins/sensu-plugins-rabbitmq/blob/3.6.0/bin/check-rabbitmq-queue.rb#L74

@akang1 what version of this gem are you using and do you see the same issue with newer versions? If you are using a recent version of this gem --user was replaced with --username.

Either I am not authenticating correctly or something is horribly broken with carot top and the version of rabbitmq that I have installed:
irb(main):019:0> rabbitmq_info = CarrotTop.new(
irb(main):020:1* host: '<IP_ADDRESS>',
irb(main):021:1* port: 15671,
irb(main):022:1* user: '<REDACTED>',
irb(main):023:1* password: '<REDACTED>',
irb(main):024:1* ssl: false
irb(main):025:1> )
=> #<CarrotTop:0x00560239124a58 @rabbitmq_api="http://<REDACTED>:<REDACTED>@<IP_ADDRESS>:15671/api">
irb(main):026:0> rabbitmq_info.queues
EOFError: end of file reached

I don't get any of your issues with RabbitMQ 3.6.2, ruby 2.3.0 and sensu-plugins-rabbitmq 3.6.0.
@akang1 From the stack trace (method_missing called on line 106), I can deduce that your version of the gem is 2.0.0 or older. This being said, I also made a test with v2.0.0 (just need to replace username with user), and it was still working fine for me.

@majormoses and @rthouvenin thank you guys so much for a quick response.
Your responses made me realize that I was using an old version of sensu-plugins-rabbitmq (specifically, check-rabbitmq-queue.rb).
Upgrading to the currently version + installing sensu-plugins-rabbitmq has fixed the issue that i was running into.
Once again, i appreciate your help!


fix(telescope): add deps for telescope consumers

Closes #108
again.

No prob since https://github.com/rockerBOO/awesome-neovim/pull/1135


Change port for MacOS to avoid port conflicts

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[STUDIO-900] Add a Slack command to create a Jira release + release PR

Add a Slack command to create a Jira release + release PR
Jira Tech task
Jira Epic
How to test the PR

How to test feature 1

Do this
Do that
Success

How to test feature 2
How to test feature 3

Deployment Notes
It requires new Environmental variables:

EXAMPLE_ENV_VARIABLE=content
EXAMPLE_ENV_VARIABLE2=content

[ch83996]


Window always has padding because of ContentView

Describe the bug
The Window class contains a ContentView, referenced as 'Client'. This view is positioned at X = Y = 1.
This means that even if you set Window.Border.BorderThickness to 0, there's still a 1 row/column pad.
I think this can be worked around by changing dimensions of Client, but it's annoying.
This may just be a doc issue. I'm filing it so I don't forget.

@tig that can tweaked by setting the DrawMarginFrame to false.

Looks like you can't set DrawMargineFrame to false till you first set BorderStyle.None.  Also BorderBrush doesn't seem to do anything


Added certificate-ripper to the AUR and updated README accordingly

Hi,
First of all, congratulations for your awesome work with certificate-ripper.
I added it to the AUR (Arch User Repository), so Arch Linux users (and Arch Linux based distros users in general) can benefit from an easy and automated installation (and update) process for certificate-ripper.
 :)
You can view the certificate-ripper(-bin) AUR package here and its PKGBUILD here.
I'm obviously down to maintain it on the AUR for all the future updates certificate-ripper will get !
If that's okay for you, I modified the README of the Github repo to let people know that certificate-ripper is now available on the AUR as well (hence this PR), but I would totally understand if you don't want to (for instance, if you don't want to consider it as an "official" way of installing certificate-ripper).
If that's so, I'll understand if you reject my PR ;)
Don't hesitate to reach me if you need more info.
Once again, thanks for your work !

Hi Robin,
Thank you for publishing this tool into AUR, I appreciate it and I also think it is a good idea for others to point out it is available there over there by updating documentation page. I think it would be good to add a section below the linux instruction to indicate it is a Contributed/Unofficial Certificate Ripper Packages to indicate that it is community driven and not maintained by me. So I would suggest something like this:
From Source

Download the latest binary here: Releases
Extract the compressed file
Add the reference to your environment variables: export CRIP_HOME=/path/to/crip/binary
Run crip print --url=https://stackoverflow.com/

Contributed/Unofficial Certificate Ripper Packages
Arch-Linux (AUR)

Install the certificate-ripper-bin AUR package
Run crip print --url=https://stackoverflow.com/

Hi Hakan,
Thanks for your answer !
Indicating that it is community driven and not maintained by you is indeed a good idea.
However, I would suggest to change Contributed/Unofficial Certificate Ripper Packages by  Contributed/Unofficial Installation Methods.
I feel like pointing the package itself as "unofficial" makes it sound unnecessarily sketchy/scary, as the package itself is the official one though. Indeed, the PKGBUILD grabs the official binary directly from your Github repo (indicated by the "source" var) without any additional changes whatsoever.
Only the installation method is contributed/unofficial, so I think this is what we should point as "unofficial" instead of pointing the package directly.
I made a new commit and updated this PR in that sense. Is that okay for you that way ?
If it's not, don't hesitate to tell me and I'll make a new update.

Thank you for the adjustments. It looks good and I appreciate your effort of publishing to AUR!


Experiment with approaches to view Record-heterogeneity in csv files. (ragged csvs)

Here I am using the term "ragged csv" like miller.
In a standard csv, if a cell is missing ,NA, it is common to just omit the value, but retain the commas ,,. @lithiumfrost uploaded a "ragged csv" where the omitted data came with no commas --- or any delimiter. See line 361 in the below image

It was mentioned that miller has an option to work with ragged csvs.
https://github.com/alexhallam/tv/issues/75#issuecomment-931901472
This is an open issue to think about how tv should work with these types of files.
Proposal
Since tv is based on pillar I lean on the shoulders of giants and see what the creators of the fantastic GNU-R pillar library decided to do. There are two components:

Truncated warnings when parsing
A pretty print of readable records

Here is the output.

@lithiumfrost I wanted to keep you in the loop. I am currently leaning heavily to the above proposal. This will close #75


Fix trailing whitespace

Did you use a linter? If so, which one?

Vim. Bwahahaha.
But really, I just have vim set to trim trailing whitespace on save.


Improve documentation

Hi @Hoogkamer - thanks for the contribution
Please make the following updates:

target the gh-pages branch instead of master
remove generated files (only the core docusaurus /src/* files and /docs/*.md are needed)


outOfMemoryError proplem

I tried to run the  test example in AffinityPropagationSuite,  but I got this error
"uncaught error in thread SparkListenerBus, stopping SparkContext
java.lang.OutOfMemoryError: Java heap space"
The test data are just several pairs, why a OOM error happed?
I have checked the code, but did't where went wrong. something Wrong with Graphx?
BTW, I'm using spark 1.5.2, and run on Yarn
Thanks a lot

Can you tell more how do you run it? I can't reproduce it.

Btw, spark 1.5.2 seems too old version. I didn't test it.

It might be related how it computes delta.


Digger plan to be run only on for project that has changes in code in the commit

Currently, as I understand the flow from the testing:

the orchestrator triggers Github workflow with terraform plan in every project that is adapted in any commits in the PR
this is triggered after every commit (even when the new commit changes non-relevant files)

Could this behaviour be changed to:

trigger Github workflow with terraform plan for all affected projects only on commit that does change in the project

Manual digger plan behaviour can be kept as it is.

Hey @chvima can you outline more the motivation for this behvaiour? Current behaviour of orchestrator is when PR event is triggered is to check the files changed for the entire PR (not just the latest commit) and trigger jobs based on that. Would like to understand why you prefer to have it only checking the files changed in that commit instead

Let's say I have a PR opened with changes in 10 TF modules. Digger runs plan for all of them and creates a comment with plan summary. I will get it reviewed and get a request to change 1 of the modules.
I commit a change in one module and Digger now runs Plan for all 10 modules again, even when none of them was changed in the commit - there is no need for that, the last comment that Digger has created about Plan summary is still valid for 9 of the modules, it only needs to be updated for one of them.
Motivation is to save resources needed for running the github workflows.

@chvima that makes sense indeed you are right. I think we should consider implementing this change. Not sure if GH payloads includes the files changed per commit. we might have to resort to git based diff otherwise. But I think it is a nice feature to be considered surely

Similar concern (more like, confusion) came up in a prospective customer conversation (P.K.) - they had a long-lived PR and assumed that only the projects affected by the latest commit would run; not all projects.
TBD whether or not the default behaviour should be changed; if we do, the challenge is that the latest plans for all projects would no longer be at the bottom. So this probably requires switching to single-comment update-only UX (so that there is only one Digger comment with all plans and it's updated continuously). More thinking required.


device_tracker.see should not call async methods

Description:
The device_tracker see method that is passed into platforms is not async friendly. However it was making 1 call to a method that was async. This PR fixes it.
Reported by @bbangert on Gitter. http://paste.ofcode.org/329QM5hKqd3RGCZcMq2EEBu

CC @lwis

Crap, sorry.

It only emits a warning, not an error.

My bad. It is raising exceptions. But maybe that's only on certain event loops. This is a log from UV loop that did raise: http://paste.ofcode.org/9pCwKVPWQw5Ti8ymC3GqhY

Auch, this is actually an even bigger bug then I thought. We are firing device over the eventbus but it's not serializable as JSON so things blow up if you are using the recorder or have the web UI open.


stack liquid templating makes a man drink: IF branch or ELSE branch doesn't get executed

See this one from
    # {% if lb_service %}
    ports:
      - 8086:8086
    # {% else %}
    ports:
      - 8086:8086
    # {% endif %}

below:
stack: matti/influxdb
version: 0.0.1
description: base
expose: api

variables:
  image:
    type: string
    default: influxdb:1.2-alpine
    from:
      prompt:
  affinity:
    type: string
    default: label!=no-influxdb
    from:
      prompt:
  lb_service:
    type: string
    required: false
    from:
      service_link:
        prompt: Pick a loadbalancer
        image: kontena/lb
  lb_domain:
    only_if: lb_service
    type: string
    default: influxdb.example.net
    from:
      prompt:
  lb_https_redirect:
    only_if: lb_service
    type: boolean
    from:
      prompt:

services:
  api:
    image: {{ image }}
    affinity:
      - {{ affinity }}
    stateful: true
    deploy:
      strategy: ha
      wait_for_port: 8086
      interval: 1d
    volumes:
      - /var/lib/influxdb
    environment:
    # {% if lb_service %}
      - KONTENA_LB_INTERNAL_PORT=8086
      - KONTENA_LB_VIRTUAL_HOSTS={{ lb_domain }}
      # {% if lb_https_redirect %}
      - KONTENA_LB_CUSTOM_SETTINGS=redirect scheme https if !{ ssl_fc }
      # {% endif %}
    # {% endif %}

    # {% if lb_service %}
    ports:
      - 8086:8086
    # {% else %}
    ports:
      - 8086:8086
    # {% endif %}

    links:
    # {% if lb_service %}
      - {{ lb_service }}
    # {% endif %}

@kke ?

So this always results in: 8086/tcp
CONTAINER ID        IMAGE                       COMMAND                  CREATED             STATUS              PORTS                                         NAMES
78fdfb0f8606        influxdb:1.2-alpine         "/w/w /entrypoint...."   14 seconds ago      Up 13 seconds       8086/tcp                                      influxdb.api-1

Both of them seem to have the port 8086? :)

... that wasn't the point: they don't get mapped. edited it now.

Have you tried
ports:
  # {% if lb_service %}
  - 1234:8086
  # {% else %}
  - 4567:8086
  # {% endif %}

?

Odd, with kontena stack validate, if I select an lb_service, then it does render the ports correctly, but selecting <none> to omit the optional lb_service, it does not render either of the if-else options:
stack: matti/influxdb
version: 0.0.1
description: base
expose: api
variables:
  image:
    label: image
    type: string
    default: influxdb:1.2-alpine
    from:
      prompt:
      env: image
    to:
      env: image
    errors: {}
    value: influxdb:1.2-alpine
  affinity:
    label: affinity
    type: string
    default: label!=no-influxdb
    from:
      prompt:
      env: affinity
    to:
      env: affinity
    errors: {}
    value: label!=no-influxdb
  lb_service:
    label: lb_service
    type: string
    from:
      service_link:
        prompt: Pick a loadbalancer
        image: kontena/lb
      env: lb_service
    to:
      env: lb_service
    required: false
    errors: {}
    value:
  lb_domain:
    label: lb_domain
    type: string
    default: influxdb.example.net
    from:
      prompt:
      env: lb_domain
    to:
      env: lb_domain
    only_if: lb_service
    errors: {}
    value:
  lb_https_redirect:
    label: lb_https_redirect
    type: boolean
    from:
      prompt:
      env: lb_https_redirect
    to:
      env: lb_https_redirect
    only_if: lb_service
    errors: {}
    value:
  STACK:
    label: STACK
    type: string
    from:
      env: STACK
    to:
      env: STACK
    errors: {}
    value: influxdb
  GRID:
    label: GRID
    type: string
    from:
      env: GRID
    to:
      env: GRID
    errors: {}
    value: development
services:
  api:
    image: influxdb:1.2-alpine
    affinity:
    - label!=no-influxdb
    stateful: true
    deploy:
      strategy: ha
      wait_for_port: 8086
      interval: 1d
    volumes:
    - "/var/lib/influxdb"
    environment:
    links:

yes. and also verified that this "works" aka port can be mapped:
    # {% if lb_service %}
    ports:
      - 8086:8086
    # {% else %}
    ports:
      - 8086:8086
    # {% endif %}

    # this will map
    ports:
      - 8086:8086

With  #1884 that case raises an error:
> Pick a loadbalancer <none>
> Enter image : influxdb:1.2-alpine
> Enter affinity : label!=no-influxdb
 [error] Liquid error: undefined variable lb_service

I assume what's happening is that the optional lb_service variable is not defined, and Liquid::Template.render(.., strict_variables: true) fails with an error. That error is just ignored in the current CLI, and it continues with the rest of the template, omitting the entire if-else-endif block.

while at it also fix #1888 for double 🍻  rewards

same bug here?
$ kontena stack install matti/weavescope

> Pick a loadbalancer lb-ingress/lb
> Enter image : weaveworks/scope:0.17.1
> Enter lb_domain : weavescope.lol.com
> Enable lb_https_redirect : **no**
 [done] Creating stack weavescope
 [done] Triggering deployment of stack weavescope
 [done] Waiting for deployment to start
 [done] Deploying service app
 [done] Deploying service probe

$ kontena service show weavescope/app
[clipped]
  env:
    - KONTENA_LB_INTERNAL_PORT=4040
    - KONTENA_LB_VIRTUAL_HOSTS=weavescope.lol.com
    - **KONTENA_LB_CUSTOM_SETTINGS=redirect scheme https if !{ ssl_fc }**

Fixed in #1884 - seems I left the magic out of the commit message


Grid columns not aligned with the header.

Bug Report

Package versions:
devexteme version:18.2.4
devextreme-angular version:18.2.4
angular version: 7.0.2

Steps to reproduce:

Current behavior:

When adding the grid inside a dx-popup and after opening the popup we populate grid with data.
The grid appears with Grid columns borders not aligned with the one of the header.
Any help will be appreciated
Expected behavior:

Screenshots:

Hi,
I have checked this issue using the latest version of our components: https://codesandbox.io/s/4q554wjny9.
Here is a sample screenshot that illustrates my results: https://www.screencast.com/t/PfCa6kss. Do you see the same result on your side? If so, please upgrade to the latest version (18.2.6). If the issue persists, modify this sample (use the Fork button) to illustrate the glitch.

Hi Alexander,
Thank you for your reply.
The issue has been resolved after adding the following line to my grid


LineMarkSeries fill to close shape at bottom

First of all, very nice library and thanks for open sourcing it! :)
I have encountered an issue when implementing the gradient on LineMarkSeries. When the end value is higher than the start value, the shape fill will overflow points. I was unable to find a way to close the shape at x(max), y(0).

What would be the best way to approach this limitation, the quick one I can think of is duplicating the series and setting the actual line to transparent and add an extra point to the bottom or manually adjust the closing point, but I would ideally want to avoid hacky solutions for the sake of my teammates :)
Thanks for the help!

Sorry for opening the issue, I have found the solution.
For future reference and someone else that might stumble upon this. You need to use AreaSeries in combination with LineSeries and only fill the AreaSeries.
<AreaSeries
 color={'url(#CoolGradient)'}
 data={this.props.data)}/>
 <LineMarkSeries data={this.props.data}/>


Discovering issue for desktop apps

I can send file Android to Android but Android to Linux or windows (and vice versa) isn't possible as they cannot find each other.

Same here :)
Antergos Linux (  x86_64 Linux 4.12.6-1-ARCH ) and Nexus 5X ( Android 7.1.2 )
Nitroshare linux version 0.3.3 from AUR and android <IP_ADDRESS> (25) Last Updated: Aug 28, 2017

@giorgosioak - unfortunately, you will need version 0.3.4 of the desktop client in order to perform a transfer with the Android app. AUR only contains the latest stable version and because 0.3.4 is still in development, you will need to compile the application yourself. QMdnsEngine is required for building the application with support for mDNS (which the Android app uses for discovery). Sadly, this library is not available on AUR yet. I hope to fix some of these problems in the coming days.

Thank you :) I can wait a while.

https://github.com/nitroshare/nitroshare-android/issues/58


How to reduce the file size of the stores?

My application parses and loads data from many files into my BrightstarDB store, performs complex analysis and conversion of the store data, then produces other outputs.  This is working well and I have a rich object model layered on the store's Data Objects (over 300 classes).  However, my users are complaining that the size of the store files are too large.  For example, I loaded about 30 MB of data stored in XML files into the BrightstarDB store which caused the store to be over 6 GIG.  Yes, everything in the XML files is converted to objects+attributes with many relationships between them, but is there anyway to make the physical file of the store to be more space-efficient? I know the triples use GUIDs which are quite large, maybe you have a version that just uses sequential IDs?  Any design or technology suggestions are welcome.
Thanks
Mike

No further info received. Future plans include some work on optimising store sizes but these are tracked as separate issues.


Chapter 8, file import - createTodos response missing mapping

https://github.com/PacktPublishing/Accelerating-Server-Side-Development-with-Fastify/blob/main/Chapter 8/routes/todos/autohooks.js
Function createTodos returns array of _id.
This function is consumed by route POST /todos/files/import . This route has schema for response validation, that expects id ( schema:todo:create:response has "required": ["id"]. ).  As result file is loaded, new tasks are created, but error is returned from api call.  It would be great, if it would be possible to catch such things in compilation/linting phase.
Fix would be map { id: _id}:
  async createTodos(todoList) { // [7]
    const now = new Date()
    const userId = request.user.id
    const toInsert = todoList.map(rawTodo => {
      const _id = new fastify.mongo.ObjectId()

      return {
        _id,
        userId,
        ...rawTodo,
        id: _id,
        createdAt: now,
        modifiedAt: now
      }
    })
    await todos.insertMany(toInsert)
  // return toInsert.map((todo) => todo._id).  //original
   return toInsert.map((todo) => ({ id: todo._id }))  //fix

Hi @reargit,
thanks for pointing out!
I think you are right, we may return toInsert directly since we have the schema that filters out all the extra properties too!


How to add an RTL layout for editing Hebrew & Arabic documents (Not code)

I very like VSCode, and I want to use it for editing documents. (Like markdown).
Not only code.
VSCode for me is the best word processor.
Now the question:
When editing, Hebrew & Arabic documents, I would like the document will have an RTL
Like in Google Docs:

How can I add this feature? What it the easiest way to change the CSS of VSCode. I thought of creating an extension, so there is an extension that does something similar that I can clone?

IT is not a duplicate.  Here speaks mainly about programming, to display the characters from right to left, inside a paragraph. This VSCode doing well.
I speaking about the paragraphs. That everything will start from right to left. Not just Hebrew chars in English code.
Here is a Photoshop I made. I'm want to make VSCode behave like this:

Let's track in #11770


replace invalid characters in appVersion label

Fixes #216
Replaces all invalid (label) characters in .metadata.labels.appVersion in case .deployment.image.tag is of kind latest@sha265:something:
$ helm template test application/ --validate --set=deployment.image.tag="latest@sha256:asdasdasasd-testing_weird.imagetäg"
...
# Source: application/templates/deployment.yaml
apiVersion: apps/v1
kind: Deployment
metadata:
  labels:
    app: application
    appVersion: "latest-sha256-asdasdasasd-testing_weird.imaget-g"

@pauvos Yikes!  Build has Failed!

@pauvos this should have been fixed by another PR can you plz validate the new release?


Game in tutorial not working

http://labs.phaser.io/edit.html?src=src\games\firstgame\tutorial.js
There is a return statement on line 56 therefore the game is not working.
there is also a return statement in the start line of the update function
greetings

It's parts 1 through 9 that comprise the tutorial. The other file was just a WIP. I'll remove it.


Tanmay2

Related issue

What changes does this PR do?
Screenshots (If any)
Questions

I Will Commit THis

Hey @Elijah75 , please add your name in the alphabetical order!
Let us know if you need any further help:)


Support CRIO container runtime

OpenShift will shift to use CRIO by default as it's container runtime.
We should ensure that KubeVirt is working as expected even with CRIO.

@nellyc can we come up with a list of items we need to meet in order to be sure that we have a base level confidence in crio support?

We will run full regression

Can we close this issue? We run all tests against crio too. Or is there something specific which we should do right now in this repo? If not, I would close this one and create more specific issues once something comes up (e.g. like #920).

Yes, closing sounds fine.


Changed some traffic_layout related codes to memory efficient string_view

Some changes to string_view:
lib/ts/ink_args
lib/ts/runroot.cc and runroot.h
cmd/traffic_layout/info.cc

[approve ci]

As a general note, passing string_view is not noticeably better than passing std::string const&.  Creating strings and particularly substrings are where the performance gains are.

I forgot to remove those part changing std::string const& to string_view. They shouldn't be there. Now it should be good @SolidWallOfCode

[approve ci]

This branch is a mess. I will redo this later with new std::string_view involved.


Exclusion list field type list is inaccurate

In the exclusion list feature, it says "you can specify an exact request header, body, cookie,...." but when I go to Portal and configure the exclusion rule, there is no such option as Request Body. How can I exclude a form where Easy Auth workflow embeds  fields in the request body rather than request header.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: c0c27cd7-15fd-7358-9740-734cfa6b641f
Version Independent ID: 9cbc5678-98e1-3c21-df0a-81d8b7e21241
Content: Web application firewall request size limits and exclusion lists in Azure Application Gateway - Azure portal
Content Source: articles/application-gateway/application-gateway-waf-configuration.md
Service: application-gateway
GitHub Login: @vhorne
Microsoft Alias: victorh

@dhruvingajjar Thanks for the feedback. I have confirmed that you cannot create body exclusions in the portal. It is likely that you will still be able to do this in Azure PowerShell or the CLI. I am following up now to see why this is occurring.

@dhruvingajjar you can add body tags via the "Request Attribute Name" field.
We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


Enumerators

Hi Andreas,
First of all, thanks for releasing this library! I very much appreciate the clean code and it's ease of use.
I was using superobject before, and since I like the way the for..in construct cleans up loops I added support to both TJsonArray and TJsonObject. I'd be interested in your opinion on these changes.
Included in these commits is a fix for the failing unit tests as well, if you hadn't changed that already. I added it to this branch to be sure the unit tests for the enumerators were running correctly.
Cheers,
Mark

PChar minus PChar returns the length, not the byte size. So with your extra "div SizeOf(Char)" it becomes a "div 4" and my unit tests fail.
What special Delphi version did you use where the "div SizeOf(Char)" was necessary?

You found a compiler bug. XE7 generates the correct code with the "div 2" but XE2 doesn't.

I'm on XE2 (Update 4 Hotfix 1)... and while I'm typing this, your comment pops in :) Well, that explains it!
I did test it in a small console application without the casts, and that seemed to work though, returning "6":
var
  value: string;
  p1: PChar;
  p2: PChar;

begin
  value := 'Hello world!';
  p1 := PChar(value);
  p2 := p1;
  Inc(p2, 6);

  WriteLn(p1); // Hello world!
  WriteLn(p2); // world!
  WriteLn(p2 - p1); // 6
end;

With an extra variable, it works in XE2:
        E := @PChar(@Buffer[0])[Length(Buffer)];
        P := IntToText(FValue.I, E);

        Writer.AppendValue(P, E - P);

An extra typecast from implicit PChar to explicit PChar worked.

Nice!


Missing Campaign reportback noun/verb properties at GET campaign/:id

Up to you how to return these, but we'll need these for display in the mobile app:

reportback_verb
reportback_noun

Cool! Good to know. I tried to return as much that seemed needed for the mobile app, but figured I missed some stuff. I can update :)


IssueID:1909:New RTC device driver framework

[Detail]
New RTC device driver framework.
[Verified Cases]
Build Pass: helloworld_demo@haaseduk1
Test Pass: helloworld_demo@haaseduk1

感谢提交 Pull Requests！
Thanks for submitting a pull request.

Security Check : e38e9be0374bf002743ad9ab3df0b1e53acc2cbd

auto build fail,number:268

start build

手动触发！
感谢 Pull Requests！
Thanks for submitting a pull request.

手动触发！
感谢 Pull Requests！
Thanks for submitting a pull request.

auto build fail,number:269

手动触发！
感谢 Pull Requests！
Thanks for submitting a pull request.

Security Check : e38e9be0374bf002743ad9ab3df0b1e53acc2cbd

start build

手动触发！
感谢 Pull Requests！
Thanks for submitting a pull request.

Security Check : e38e9be0374bf002743ad9ab3df0b1e53acc2cbd

auto build success,number:274


D&D 5e 4.0.4 Support

Not sure if you're aware yet, but DAE can't be used with the latest D&D 5e system. DAE states it only supports up to version 3.9.99

I just saw this is the "Link Tree" repo, but I got here from the issue tracker link at:
https://gitlab.com/tposney/dae

no problem


Fix invalid code coverage.

Issue #722.
Only run tests on release builds.
Filter out GoogleCloudExtension.Resources.

Codecov Report

Merging #724 into master will increase coverage by 3%.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           master    #724     +/-   ##
========================================
+ Coverage    6.69%   9.69%     +3%
========================================
  Files         470     469      -1
  Lines       17079   11387   -5692
========================================
- Hits         1143    1104     -39
+ Misses      15936   10283   -5653

Impacted Files
Coverage Δ

...alytics/Events/OpenGcsBucketOnCloudConsoleEvent.cs
0% <0%> (ø)
:arrow_up:

...n/AttachDebuggerDialog/ListProcessStepViewModel.cs
0% <0%> (ø)
:arrow_up:

...oogleCloudExtension/FirewallManagement/PortInfo.cs
0% <0%> (ø)
:arrow_up:

...ogleCloudExtension.DataSources/InstancesPerZone.cs
0% <0%> (ø)
:arrow_up:

...Extension/GoogleCloudExtension/Controls/TimeBox.cs
0% <0%> (ø)
:arrow_up:

...eCloudExtension.DataSources/WindowsInstanceInfo.cs
0% <0%> (ø)
:arrow_up:

...leProgressDialog/GcsFileProgressDialogViewModel.cs
0% <0%> (ø)
:arrow_up:

...on/Analytics/Events/OpenGceInstanceWebsiteEvent.cs
0% <0%> (ø)
:arrow_up:

...lytics/Events/OpenCloudSqlConnectionDialogEvent.cs
0% <0%> (ø)
:arrow_up:

...udExtension/GcsFileBrowser/GcsFileBrowserWindow.cs
0% <0%> (ø)
:arrow_up:

... and 397 more

Continue to review full report at Codecov.

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

LGTM


send kubernetes.cpu.usage.total in units of cores

What does this PR do?
This PR is an implementation of https://github.com/DataDog/integrations-core/issues/541 by sending the kubernetes.cpu.usage.total metric in units of cores (rather than nanocores).
Motivation
Frustration at dividing this metric by a billion every time I use it.
Additional Notes
I added some new fixture files because the existing ones couldn't make a kubernetes.cpu.usage.total metric (since it's submitted as a rate, you need multiple samples from the same container). I grabbed some data from one of my Kube clusters, anonymized it a little, and stripped out metrics that weren't the CPU ones.
This is probably a breaking change - dashboards would need to be updated / re-scaled, and it changes the unit type. That might make it not the right solution! Lemme know :D

Hey @sophaskins
Thanks for the PR! It makes a lot of sense. Could you add this nanocore_metrics option in kubernetes/conf.yaml.example with a short comment about what it does and why it's useful? This would increase its visibility.
Thanks again

Added an example config and removed that bit from the tests (sorry about that - it was residual from an earlier approach 😄 )

Hi @sophaskins,
A quick update on the topic: I am investigating modifying our unit catalog to allow keeping the submitted value and seamlessly convert it to either cpu% or whole cores. That would allow the display unit to scale automatically. What target unit (cpu% or cores) would be the best for your use case?
Regards

Heya! That sounds super rad!
I would love to see ours in "cores" :D

Yes to cores from us as well.

Hi @xvello - I'm just curious if this PR is still in the works. Cheers.

per https://github.com/DataDog/integrations-core/issues/541, I'm gonna close this PR


Startup sequence

Corresponds to pull request on state_controller repository
Adds startup sequence for properly calibrating tractor steering systems
Fixes significant error: Estop pin for reading and writing were the same on the Teensy

Start reviewing, but don't merge yet - need to test on platform first


Event system

Some server owners may need to perform specific actions when a user wins or loses a game. That's why a simple event system can be very useful.
Events I've been thinking about :

When an user wins a game
When an user loses a game
When there is a tie

Others ideas of events by the way?

I have also added a documentation page : https://github.com/utarwyn/discord-tictactoe/wiki/Event-handling-system
It contains usage, examples and some explanations about how this system works.

I have also added a documentation page : https://github.com/utarwyn/discord-tictactoe/wiki/Event-handling-system
It contains usage, examples and some explanations about how this system works.


Django 3.2.2 & "Application instance <Task pending [...] took too long to shut down and was killed." issue

Hi guys!
First - thanks for your hard work - we're heavily using your package for our health check and it works great!
To the point: we recently upgraded Django to version 3.2.2 and we started to receive the following errors (example one, there are different variations of it depending on a check):
Application instance <Task pending name='Task-387' coro=<StaticFilesWrapper.__call__() running at /usr/local/lib/python3.9/site-packages/channels/staticfiles.py:44> wait_for=<Future pending cb=[_chain_future.<locals>._call_check_cancel() at /usr/local/lib/python3.9/asyncio/futures.py:384, <TaskWakeupMethWrapper object at 0x7f78e86dafd0>()]>> for connection <WebSocketProtocol client=['<IP_ADDRESS>', 52972] path=b'/ws/user/'> took too long to shut down and was killed.

When we tried to open the health check page it just hung and never loaded.
After hours of debugging, we discovered (thanks to Maciek!) that it was enough to re-register default plugins in our AppConfig:

... and it started to work again!
I'm wondering if it has something to do with minor changes to AppConfig in Django 3.2 described here: https://docs.djangoproject.com/en/3.2/releases/3.2/#automatic-appconfig-discovery but I wasn't able to verify/tell what is the root cause.
Posting it for future reference and maybe someone will be able to find and fix the root issue.
Thanks,
Daniel

Hi @Sajam,
I honestly don't know. I am not so involved with this part of the Django development. Besides, I currently don't use this package myself, so it's a bit harder to reproduce your scenario.
Therefore, if anyone has more details on this, help would be more than welcome.
Best,
Joe


Update references to webpack 4

Hi! 👋 This PR fixes https://github.com/webpack/webpack.js.org/issues/2296
Following is a list of removed resources and a brief explanation of the reasons behind:

Predictable Long Term Caching
Based on webpack 3 (i.e. code snippets use CommonsChunkPlugin)

Webpack & Caching
While some of the recommendations might still be useful, some other make use CommonsChunkPlugin. Maybe the reference could be updated and then kept in the docs?

Advanced Webpack Presentation
Based on webpack 3 (i.e. code snippets use CommonsChunkPlugin)

Issue 1315
Issue created on 2015 and closed this May. Might be relevant to webpack 4 to a certain extent, but does not provide valid answers.

This is my first PR, so all feedback is really appreciated. Thanks for your patience!

Thanks! Dates removed from logs 🙌

Thanks!


Allow to specify granular component to extract from Export-O365Configuration

Pull Request (PR) description
Exposed existing ComponentsToExtract parameter to the Export-O365Configuration cmdlet.
This Pull Request (PR) fixes the following issues
N/A

This change is

Codecov Report

Merging #213 into Dev will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##             Dev    #213   +/-   ##
=====================================
  Coverage   89.8%   89.8%
=====================================
  Files         53      53
  Lines       4688    4688
  Branches       8       8
=====================================
  Hits        4210    4210
  Misses       470     470
  Partials       8       8

Continue to review full report at Codecov.

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


additions to HoTT

Add theorems about pointed equivalences, nat/sphere_index/trunc_index, and a little more homotopy theory.
This also removes the [multiple_instances] attribute.
I've also formalized the long exact sequence of homotopy groups in Lean (section 8.4 of the HoTT book), but it needs some cleanup before I add it to the HoTT library. Currently it can be found here.

Added one more commit, which replaces assert by have in my previous commits, and also merges the namespace equiv.ops into equiv.


Unable to correctly handle the last character when fixing Chinese mojibake

Hello,
When I tried to fix mojibake using ftfy version 6.2.3, the last character was always incorrect.
Please help me.
I have tried versions 6.2.0 and 6.1.3 and have encountered the same error.
Here are a few examples:
mojibake: 'Python åˆ\xa0é™¤åˆ—è¡¨ä¸\xadçš„å…ƒç´'
output: 'Python 删除列表中的元ç´'

mojibake: 'Puppeteer å¤„ç\x90†æ–‡ä»¶ä¸Šä¼'
output: 'Puppeteer 处理文件上ä¼'

mojibake: 'Diablo ç¬¬äºŒå¹• é²\x81 Â· é«˜å›'
output: 'Diablo 第二幕 鲁 · 高å›'

I have found the reason, the original data is missing a byte, and it is not an issue with ftfy.

Thanks for the followup!


Fix invalid 'to' field on contract deploy transactions

It seems somebody has launched a new tutorial with MEW and Ganache, because I'm getting numerous issue reports that contracts can't be deployed to Ganache via MEW. The error occurs in Ganache transaction validation. MEW submits contract deploy transactions with an empty string in the to param. Last I checked, empty strings aren't valid hex encoded addresses.
I've changed Ganache to treat empty string transaction parameters as if they weren't defined, but I figured it's best to fix up MEW as well. By my testing this PR fixes the problem. That said, I didn't see some huge cache of automated tests to run (just app/tests/index.html), so I can't promise that I didn't break something here.

Hi @tayvano - do you need anything from me in order to be able to merge this? Your last comment made it sound like things were in a bit of gridlock - not sure how best to move forward.

Hey let me take a look at this and add it to our next release

@kvhnuke did you need me to fix up merge conflicts, or will you all handle that?

@benjamincburns you can remove those conflicting files from your commit since the /app folder gets compiled anyway.


flutter_html_to_pdf dependency uses unsupported version of Kotlin Gradle plugin

I am using flutter_html_to_pdf: ^0.7.0 and ext.kotlin_version = '1.7.21'
but still getting this bellow error

Same error

is anybody handling this repo?

I have forked the repository, fixed the Kotlin Gradle dependency, and used it in the following manner:
flutter_html_to_pdf:
  git:
    url: https://github.com/mohammadhprp/flutter_html_to_pdf2

@faisalmushtaq007

this url not working for me its giving 404 , has anybody fixed this issue

i got this working , download package locally and add path to pubspec.yaml

Hello @RizwanAtNvest,
I apologize for the inconvenience. I had accidentally deleted the repository. However, I have now re-created it, and I kindly ask you to give it another try. Please let me know if you encounter any further issues.

thanks brother, i got this working i downloaded it in local folder and added path

getting same issue for sdk 35.


Link to capacitor-assets rather than cordova-res

Appreciate this is not exactly this rep, but it relates to this rep
https://capacitorjs.com/docs/guides/splash-screens-and-icons
Links to cordova-res, when as we should be using capacitor-assets, should link here
Thanks!
Alex

Thanks, I was waiting until this got a little more stable and it's just about there so good reminder!


Skip rows with missing values and add annotation

Address the feature request in  #25.
Skips all samples/rows that contain any missing values and report the number skipped in an annotation at the bottom.
This is perhaps a bit conservative since I could imagine that in some cases, a user might want to still see a sample in the plots where the column values aren't nothing.
Let's wait until we get a complaint to address such an option.
Example:
df = DataFrame(randn(1000,3) .* rand.(Ref((missing, 1, 1, 1, 1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1))), :auto)
pairplot(df)

Example with multiple series:
df1 = DataFrame(randn(1000,3) .* rand.(Ref((missing, 1, 1, 1, 1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1))), :auto)
df2 = DataFrame(randn(1000,3) .* rand.(Ref((missing, 1, 1, 1, 1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1))), :auto)
pairplot(
    PairPlots.Series(df1,label="two",color=:blue),
    PairPlots.Series(df2,label="one",color=(:red,0.5))
)

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

Comparison is base (3491e7e) 72.43% compared to head (2eeb824) 70.99%.
Report is 3 commits behind head on master.

Files
Patch %
Lines

src/PairPlots.jl
61.53%
15 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##           master      #29      +/-   ##
==========================================
- Coverage   72.43%   70.99%   -1.44%
==========================================
  Files           3        2       -1
  Lines         312      331      +19
==========================================
+ Hits          226      235       +9
- Misses         86       96      +10

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


"workspaces" key is being shipped in the released npm module package.json

Currently the package.json being shipped with the node_module contains a workspaces key that is a development artifact only.
This is causing yarn to display a warning during install warning Workspaces can only be enabled in private projects. and should be removed.

This is a bug in yarn, not this package.

@remcohaszing why do you think it is a bug in yarn? There is a long discussion happening at https://github.com/yarnpkg/yarn/issues/8580 about this and no conclusion was made. It looks like yarn is on purpose emitting this warning as the workspaces key is intended for develop only environments or private repositories.
Can't we just extend the release pipeline to quickly change the package.json removing this workspaces key?


Support for Xamarin Android

Currently, there are few options for mocking/faking libraries that can run on Xamarin Android.  Most libraries like moq and fakeiteasy rely on Castle.Core, which does not have a build for Xamarin Android.  Since Foq does not have this dependency, it seems like it has a good chance to work for that platform
I hacked up something quick, and was able to get Foq and the C# tests to run on Xamarin.  The issues I encountered were that LinqTests.TestSetupFuncWithReturnsLambda failed with an exception related to bad IL code (seems to be a Mono problem) and Microsoft.FSharp.Control does not seem to be available for Xamarin.  I encountered a few more failures running the F# tests on Mono too.
This brings me to my questions:

Is Foq a viable option for Xamarin Android?

It seems that there are issues/limitations with F# on Mono.  Its not clear to me if any of them would be a showstopper.

If so, would a pull request be accepted that added a Xamarin Android build?

Foq on Codeplex has some more recent commits, so its not clear to me where the project lives.

Hi elevine and apologies for the excessively delayed response, I did not have any notifications turned on for this project, and got bored of periodically checking after months of inactivity.
Both Castle.Core and Foq use code generation via Reflection.Emit, I think this is likely to be problematic on Android which I believe doesn't permit code generation.
P.S. sorry for the confusion on project location, the project started on CodePlex but has been migrated to GH. If an issue is raised on CodePlex I fix it there and copy the change over to GH, and vice versa. The Nuget packages are generated from the CodePlex version so the last update is likely to be on CodePlex.

Hi @ptrelford - I've sinced moved onto another job and I'm not currently working on any .Net or Xamarin stuff.  However, we did mange to get Castle.Core and Moq working on Android while I was still there.  Here are the forks if you are interested:

https://github.com/elevine/Core/tree/xamarin-android
https://github.com/elevine/moq4/tree/android

If I am remembering correctly, Reflection.Emit is supported for Xamarin Android (but not iOS as far as I know).
Mocking libraries for Xamarin Android are an unfulfilled niche, and Foq seemed like it wasn't far from being able to meet that need.  It might be worth considering for your roadmap.

Hello,
Do you have plans to implement .net standard?
Thank you
Jonathan


Not correct documentation

This documentation page does not reflect what you actually see in the portal.
Some examples are:
"Add a Dynamic Scope": bullet 8 does not match the real world, at least when I try to add a dynamic scope. There is no "Configure Azure VMs for schedule updates" page
"View patch history of a Dynamic scope": I can see no way to browse Maintenance Configurations from the "History" page. I have to select "By maintenance run ID" and then filter on the particular Maintenance Configuration(s)
[Enter feedback here]

Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: 21081670-368e-0cfd-184e-a0c380a98679
Version Independent ID: b1b5f05a-d7ca-d8b8-c86e-67ae174c0fc1
Content: Manage various operations of Dynamic Scoping.
Content Source: articles/update-manager/manage-dynamic-scoping.md
Service: azure-update-manager
GitHub Login: @SnehaSudhirG
Microsoft Alias: sudhirsneha

@fongendnv, thank you for your inputs. The article is updated. #please-close


[feat] - add new custom dashboard for volumes monitoring with alarm

What was done:

add new custom dashboard for volumes monitoring with alarm
disable unified_alerting,
add root url variable to make links usable

Example of usable link in alarm:
https://protofire.slack.com/archives/C03P9UE3QB0/p1664820400660679
Link to dashboard on the grafana.com: https://grafana.com/grafana/dashboards/17092-kubernetes-persistent-volumes/

I'll apply this to prod after approve on this PR.

Will apply it to mainnet


A0-1818: Prepare aleph-client for publishing

Description
Versioning aleph-client is a pretty tricky issue. On the one hand, in many side tools (like ink-wrapper or shielder-relayer) we would strongly prefer relying on aleph-client from registry than from git. On the other hand, semantic meaning of the aleph-client's version is very unclear - the newest version is very probable to be incompatible with both testnet and mainnet. It is guaranteed to be compatible only with the attached runtime.
The issue how to tackle compatibility with production nets is somehow isolated. We might go for some feature-gating (which might be very onerous and hard to maintain) or some tagging model. For now, we want just publish code to unblock side tooling.
The only problem with publishing it now are git- and local dependencies:

local:

pallet-baby-liminal - this crate is not published, but the whole dependency served only for one simple type alias; I've copied it and removed dependency
primitives - this crate is not published (although it probably makes sense to publish it); it served for a couple of simple type aliases (I've copied them) and one function - era_payout - which I had to copy as well; imho this is awful

git:

frame_support and sp_runtime - both from our Substrate fork; I was able to remove them, thanks to reexports from subxt
pallet-contracts-primitives - from our Substrate fork; it provides us with ContractExecResult which cannot be imported from anywhere else, nor copied without copying a lot of stuff. Therefore we have to keep it. However, we cannot keep only git pointer to our fork. Instead, we have to point to version = 0.7.0 on crates.io, which internally uses just Substrate code. However, we should be pretty safe here, since our changes on the fork don't touch this part.

Other solutions

We do not publish aleph-client at all. Make @obrok et.al. your haters.
We publish primitives (very soon) and thus we don't have to copy all the type aliases and era_payout from there. This will require maintenance.
We publish all our Substrate dependencies from fork. Not so much more maintenance, probably huge gains.

Testing
Try running cargo publish --dry-run in aleph-client/ directory.
Type of change

Breaking change (fix or feature that would cause existing functionality to not work as expected)

I have created new documentation

I have bumped aleph-client version if relevant

Oh, one more thing, I'd recommend starting a CHANGELOG, maybe based on https://keepachangelog.com/en/1.1.0/


Animations

[x] left to right animations for Area and Line
[x] radius growth animation for Scatter
[x] bottom to top animation for Bar

preview https://skvale.github.io

Whoa! everything was refactored.  Looks rad.  Closing this PR


sentences labeling

Hello, @hpzhao  After reading the paper, I am still confused about the sentence labeling method. Can you explain the detail or show the relevant code?

The key point of this sentence labeling method is that we make decision sentence by sentence.  The power of this approach is that it could take previous decision or state in consideration. For example, we could reduce redundancy due to the novelty of the classification layer. You could read my old blog for some details. I hope this will help. @BayronP

Maybe you misunderstood what I confused!  In this paper, I find some description(below) about this labeling, but I cannot understand how to calculate this related sentences subset. Can you help me?
To solve this problem, we use an unsupervised approach to convert the abstractive summaries to extractive labels. Our approach is based on the idea that the selected sentences from the document should be the ones that maximize the Rouge score with respect to gold summaries.

You mean how to get the sentence label of this corpus by unsupervised approach ?  Due to the lack of extractive summarization corpus, it's just a approach to build it. The approach is described by the rest of this paragraph:

Since it is computationally expensive to find a globally optimal subset of sentences that maximizes the Rouge score, we employ a greedy approach, where we add one sentence at a time incrementally to the summary, such that the Rouge score of the current set of selected sentences is maximized with respect to the entire gold summary . We stop when none of the remaining candidate sentences improves the Rouge score upon addition to the current summary set. We return this subset of sentences as the extractive ground-truth, which is used to train our RNN based sequence classifier.

@BayronP


Oudated info in Packaging an operator for OLM

Bug Report
What did you do?
I was checking the doc https://olm.operatorframework.io/docs/tasks/packaging-an-operator/
What did you expect to see?

It links as an example of the Memcached sample with the layout 1.0+ of SDK in : https://github.com/operator-framework/operator-sdk/tree/master/testdata/go/v3/memcached-operator/config
The description point out the files in the config/ dir instead of the legacy deploy/ one.

What did you see instead? Under which circumstances?
A clear and concise description of what you expected to happen (or insert a code snippet).

@camilamacedo86 Thanks for the report - It looks like the doc link you provided no longer exists, so I'm going to close this out. If we're still referencing out-of-date memcached samples, let's re-visit this by opening an issue in the olm-docs repository so we have a centralized place that tracks all the upstream OLM documentation.


commits not handled

Pull request closed. Another pull request was submitted with the commits squashed. #31


Custom ArrayField Template is not rerendering

I use Custom Arrayfield component in my form.It is a functional component.My parent component state is triggered which allows my parent to re render but not the custom array field function.I need some references within my custom component function.After some inspection I came to a conclusion that it must be something within the formComponent .Quick help would be useful.Thanks
in advance

Do you have a JSFiddle or anything like that that we could look at?


Keeping extras from VRM

Hello,
I have a vrm file. It's a modification of a glb file.
When I rename it to a glb, and then do .\gltfpack.exe -ke -kn -km -ke -i '.\Short Haired Girl Black Dress Less Poly 3.glb'  -o 'Short Haired Girl Black Dress Less Poly 4.glb'.
VRM importers fails to load it as a vrm.
I have tried the -ke option.
Any suggestions?
Short Haired Girl Black Dress Less Poly 3.zip
All Rights Reserved. Ask me if you want license details.
https://github.com/saturday06/VRM_IMPORTER_for_Blender is an blender addon for VRM.

It would be useful to us too if the VRM extensions were not removed during the meshopt process. 👍

This is my attempt to patch meshoptimizer.
https://github.com/fire/meshoptimizer/pull/1
Looking for suggestions.
git clone https://github.com/fire/meshoptimizer.git -b vrm
cd meshoptimizer
mkdir build
cd build
cmake -DMESHOPT_BUILD_GLTFPACK=ON ..
make
wget https://github.com/Miraikomachi/MiraikomachiVRM/raw/master/Miraikomachi.vrm
./gltfpack -si 0.1 -noq -ke -ke -i "Miraikomachi.vrm" -o "Miraikomachi Optimize.vrm"
# Open in a vrm importer.

I'm not familiar with VRM but if it's just about preserving the top-level extension node "VRM", it should be simple. The change above might be too broad though, as it would preserve some top-level extensions where extra work needs to be performed to make the content work, e.g. KHR_xmp.
My suggestion thus would be to simply detect when "VRM" is present in the extension list (regardless of input options) and, if so, output just that extension blob, using appendJson which performs JSON minification.
Feel free to submit a PR along these lines. I don't have a way to test this so testing suggestions would be welcome.

BEFORE 7.6M AliciaSolid_vrm-0.51.vrm
AFTER 5.5M   AliciaSolid_vrm-0.51-opt.vrm

From https://github.com/vrm-c/UniVRM/tree/master/Tests/Models/Alicia_vrm-0.51
Using https://vrm-viewer.yukimochi.io/ importer.

AliciaSolid_vrm_opt.zip from
Notes for posterity.
On https://github.com/fire/meshoptimizer/pull/1/commits/7ab13deb033508ab3a9762486e027c55104cd28f
./gltfpack -si 0.8 -noq -km -kn -ke -i "AliciaSolid_vrm-0.51.vrm" -o  "AliciaSolid_vrm-0.51-opt.vrm"

Thanks for the review.
I can scan for the "VRM" extension. However, I noticed some problems with extras not being sent through.

Enable "VRM" when seen
Keep extras code the same.

I slept thinking about this optimization, because Godot Engine added autolod, I don't have a requirement for this.
The pr is salvageable for all those webxr based avatar multiverses out there. @avaer

I hope vrm support is still being worked on.
The only other "tool" https://github.com/infosia/vrmpack?tab=readme-ov-file  that was apparantly "merged into meshoptimizer" is really a drag to compile and use...
And messing with blendshapes and such is hardly a good way to downsize a .vrm.
After all there are not many good tools for vrm (pmx has dozen.. but vrm ? )


chore: Add meeting minutes for April 5th, 2018.

Closed in favor of https://github.com/nodejs/badges/pull/17 👍


Cannot read property 'indexOf' of undefined, js engine: hermes in search

You need to pass searchField attribute as below
`<Dropdown
search
maxHeight={300}
minHeight={100}
labelField="label"
valueField="value"
searchField="label"
                  />`

I hope this helps

https://github.com/hoaphantn7604/react-native-element-dropdown/issues/54#issuecomment-1079614091

#54 (comment)

Label cant be null


Dataset inquiry

Hi,
It's impressive to see the shocking results of DPPO. About the dataset, I wonder that is only one demonstration of furniture-bench availiable in Google Drive right now? Can I use the demonstrations from the orginal furniture-bench repo (Google Drive) to train the model?
Thanks

Hi, thanks for the interest! We collected our own demonstrations in simulation (see App. D7 of our paper), and we used the ground-truth object states as part of the policy input (plus the robot state, no pixels)
I just looked at the original demonstrations from furniture-bench and I didn't see the object states from the collected data (only the robot state), correct me if I am wrong. If there is both states then I think we can convert the original data to our format easily and pre-train the policy.
Fine-tuning pixel policy with Furniture-Bench might be challenging though as the simulation speed might not be optimized. I am not entirely sure but @ankile might have more ideas.

Hi @Alan-Heoooh !
Yes, as Allen said, we collected our own demonstration data by hand with a Spacemouse in simulation, recording both robot states and ground-truth object states. We are going to make all our demo data public quite soon. I can let you know when it's ready!

@Alan-Heoooh Just to be clear, the data we collected is available right now and you can use it to pre-train the policy. I think @ankile plans to have a more formal release of all the demo data from his work robust rearragement

Hi, @allenzren @ankile ,
Thank you for your kind reply. But I still wonder that is train.pkl in this folder contains all the training demonstrations for pretrain? Also, I didn't see the updated data for furniture-bench. Could you please post a link here?
Thanks!

@Alan-Heoooh Yes train.pkl contains all the training demos (50 trajectories, you can see the details in the log file) we used for pre-training in the DPPO paper. I am not sure if @ankile has more demos that will be released for a particular task, but even so those are used only in the robust rearrangement paper, not DPPO.
What do you mean by "updated data"?

Got it. Thanks for your reply!

Btw I updated the training data and I replaced pkl with npz files, which is much smaller in size.