[UI] Support accordion-style menu when Meshery menu drawer is collapsed.

Current Behavior
Meshery menu is perpetually expanded to show all menu items when the drawer is collapsed.

Desired Behavior
Offer the same accordion-style behavior when the menu drawer is collapsed as when the menu drawer is fully open.

Contributor Guides and Resources

🎨 Wireframes and designs for Meshery UI in Figma
🖥 Contributing to Meshery UI
🙋🏾🙋🏼 Questions: Layer5 Discussion Forum and Layer5 Community Slack

@leecalcote I think this will also solve #3816

@leecalcote I think this will also solve #3816

That sounds about right. 👍

I'd like to look into this :)

@sudo-NithishKarthik, @warunicorn19 I'm finding this issue a bit complicated. Maybe I can fix this if I start fresh, although that's a lot of work. Can I get some heads up here please, I can't figure out any simple solution to this.. :/

im having a hard time explaining 😂, @CipherGirl did you understand😅?

Oh yes, I did! I also thought of that too.. I think it'll look better on the right side.. Let me try to figure out If I can do that..


希望添加linux/arm支持

https://x.com/acunetix/status/1496209656347246593

We don't support ARM architecture. For more information on installation requirements, visit https://acunetix.com/support/docs/wvs/installing-acunetix-wvs/.
...

官方目前不支持ARM架构


Provide Averaged Rating

User Story: As a visitor of the site, I need to receive aggregated average critic and user scores so that I may make a better informed purchase.

Broken up into Create Review page django template/review and Scrape features from websites backlog items.


Clean up the test environment from old adp* resources

Maintainers task

Clean up old dependencies (azure resources) from test environment
Retest all pipelines to make sure no hidden reference exists

Cleanup & testing was done


Handle pointer to pointer param

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: IIFE sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

@IIFE I've merged one of your PR-s and there conflicts now, please resolve.

@ddobrev Update this branch. Can you please merge it and I'll update the subsequent PRs.

@IIFE better now but please try moving the test to Common again. If it works for you locally, I'd rather have it there.

@ddobrev You mean move it so that it's generated for both CLI and C#?

Precisely.

@ddobrev Done.


Autorest Regen Preview to 3.0.0-alpha.20230712.3 by Dapeng Zhang from refs/pull/3574/merge

Triggered from https://github.com/Azure/autorest.csharp/pull/3574

API change check
APIView has identified API level changes in this PR and created following API reviews.
Azure.AI.AnomalyDetector


Fix biom-conversion-server for conversion from 1.x to 2.x

As a referee to our f1000 article notes:

The biom-conversion-server does not appear to be backwards compatible (I could not upload and convert a BIOM 1.x file to 2.x format) - this one-way conversion functionality is should be clearly indicated in the first paragraph of the “Application” section. In addition, if users try to upload a BIOM 1.0 file they should be presented with an appropriate error message (I didn’t see one - the tool just froze when I attempted to upload a BIOM 1.0 file).

Bik H. Referee Report For: biojs-io-biom, a BioJS component for handling data in Biological Observation Matrix (BIOM) format [version 1; referees: 2 approved with reservations]. F1000Research 2016, 5:2348
(doi: 10.5256/f1000research.10362.r16436)

I was unable to replicate the error. Maybe it is a performance issue or a problem with a specific biom file. More information required in order to fix.


Revert #1440

Summary
closes #1542.

@ikkoham Thanks for doing this so quickly! I added the On Hold label since currently qiskit-nature has some logic that is using the PauliSumOp so we need to hold off removing it from here until that is switched over to using terra, which hopefully is shortly.

@manoelmarques Thank you for fixing the error.

As qiskit-nature has been moved over to using opflow from Terra, the early PauliSumOp work that was started here, but completed in Aqua can be now be removed.


Property set to null with Mapster v4 (and later)

We have a problem where Mapster touches the Client property of the destination object, despite that there is no mapping for it. This is the source object:
    public class RdpConnectionInputModel : ClientModelBase
    {
        public Guid Id { get; set; }
        public string Name { get; set; }
        public string Comment { get; set; }
        public string RemoteHostname { get; set; }
        public string Username { get; set; }
        public string Password { get; set; }
        public int RdpType { get; set; }
        public int RemotePort { get; set; }
        public bool IsAdmin { get; set; }
        public bool Synchronize { get; set; }
        public PermissionsDto Permissions { get; set; }
        public Guid ClientId { get; set; }
        public string ClientName { get; set; }
        public Guid HardwareId { get; set; }

        public string ClientDisplayName => ClientName ?? HardwareId.ToString();
    }

The destination object is an EF class that looks like this:
    [Table("SVC_RdpConnections")]
    public class DbSvcRdpConnections
    {
        public Guid Id { get; set; }
        [ForeignKey(nameof(Client_Ref))] public virtual DbRimClient Client { get; set; }
        public Guid? Client_Ref { get; set; }
        [ForeignKey(nameof(System_Ref))] public virtual DbSecSystem System { get; set; }
        public Guid? System_Ref { get; set; }
        [ForeignKey(nameof(Company_Ref))]
        public virtual DbSecCompany Company { get; set; }
        public Guid? Company_Ref { get; set; }
        public CompanyFlags CompanyFlags { get; set; }
        [Required, StringLength(100)] public string Name { get; set; }
        [StringLength(250)] public string Comment { get; set; }
        [StringLength(50)] public string RemoteHostname { get; set; }
        public int RemotePort { get; set; }
        [StringLength(50)] public string Username { get; set; }
        [StringLength(50)] public string Password { get; set; }
        public DesktopConnection RdpType { get; set; }
        public string EncryptedPassword { get; set; }
        public bool IsAdmin { get; set; }
        public bool Synchronize { get; set; }
        public DateTime? LastSynchronized { get; set; }
    }

The mapping is configured like this:
            config.ForType<RdpConnectionInputModel, DbSvcRdpConnections>()
                .Map(i => i.Client_Ref, i => i.ClientId)
                .Map(i => i.RdpType, i => (DesktopConnection)i.RdpType)
                .Map(i => i.EncryptedPassword, i => i.Password, i => i.Password != null)
                .AfterMapping((src, dest) =>
                {
                    dest.CompanyFlags = DefaultMappingProfile.PermissionsToCompanyFlagsConverter.Convert(src.Permissions);
                    dest.Comment = CommentFormatter.Format(dest.Comment);
                });

When the user updates the record in the user interface, then we fetch the existing entity from the database (DbSvcRdpConnections). The Client property holds a proxy object that is used for lazy loading. The database entity is updated by calling the following method:
   config.GetMapToTargetFunction<RdpConnectionInputModel, DbSvcRdpConnections>().Invoke(inputRdpEntity, dbRdpEntity);

With Mapster v3.2 it would leave the Client property alone, but with Mapster v4.0 (and later) it resets the Client property to null. Because this property is now dirty, it also resets the Client_Ref to null when I try to access the entity's state.
I have tried to create a reproducable scenario in a seperate project, but I didn't succeed. Is there some specific functionality for EF objects or am I missing something else. I couldn't find a commit between v3.2 and v4.0 that might be related. But Mapster is rather complex code, so I probably overlooked something. Do you know what could cause this issue? We now use a workaround to fix the Mapster version v3.2, but I would rather use v4.1...

There is unflattening feature, but you need to turn on, and it should create new object rather than setting to null. I just test with my unit test, Client should not be null even unflattening turn on or off.
Try debugging using src.BuildAdapter().CreateMapToTargetExpression<DbSvcRdpConnections>().ToScript() to see how Mapster generate code. It might show you some hint.

Thanks for your prompt response. I will look into this issue again on Thursday when I am back at work.


VSDX export doesn't work if site is hosted in subfolder and in electron app

It seems that the problem is that vsdx.min.js is included using an absolute path:
mxscript('/js/vsdx.min.js', delayed);

If I change path to relative, VSDX work fine when running draw.io in subfolder:
mxscript('js/vsdx.min.js', delayed);

Thanks for the report, this will be fixed in the next release.


Split Die/Exit/Return macros out of CodeUtils.h

Problem
Requested followup from #5545
Summary of Changes

Move ‘Die’ macros to <support/DieMacros.h>
Move ‘Error’ macros to <support/ErrorMacros.h>
Move ‘Return’ macros to <support/ReturnMacros.h>
Fix #includes for former CodeUtils.h contents.

I personnaly prefered […]

I personally agree. If some future consensus is for a single file, at least it's absolutely trivial to go in that direction: just sed -i -E -e 's,/(Die|Error|Return)Macros\.h>,/ErrorMacros.h>,' and then clang-format removes the dups.


Change tests to be less reliant on copy

Feature
Why is this feature being added?
Any copy changes completely break the tests on operationcode_bot. It should check to see that messages are sent, not that the specific wording is exactly the same.
What should your feature do?

test_it_sends_a_message_to_the_user_if_an_env_var_is_set just checks to see if a message is sent to the user.
test_it_welcomes_the_user_on_new_user_join just checks to see if a message is sent to the user.

@hollomancer - I can take this one


Paste last selected text

The Mac Terminal.app has a shortcut Cmd+Shift+V that pastes the last selected text, without needing to copy it to the clipboard.
It also works between terminal windows.
Would it be possible to implement this feature?

Hi, @wmertens,
Thanks for your opinion, Hyper has this option in place, check out copyOnSelect option in  configuration

PS. Next time, you gotta create an issue in relevant repo, hyper one. we can help you here, but in that repo you would get help in matter of hours =)

OMG I must have been low on coffee :) lol, sorry, and thanks for the pointer! 💯

no worries =)


3 Crown Family (Update-New Clan)

name: 3 Crown Family
key: 3Crown
color: blue
emblem: Crown_03
info:
  logo: /static/img/brands/logo/3-crown-family.jpg
  description: >
    Welcome to the **3 Crown Family**! We are a competitive clan family that focuses on War and Player Progression. We are looking for players that are War focused and looking for a long-term clan.

    We have a clan available for many levels of players. Our members are very active and willing to help everyone in our family succeed. Our Discord has 100+ members so it’s a great way for everyone to communicate about war, trades, or just to socialize.
  social:
    - discord: https://discord.gg/v4azkxj
    - reddit: https://www.reddit.com/message/compose/?to=3C_Coleslaw
clans:
  - name: 3 Crown N' Down
    tag: 2R89V00G
  - name: 3 Crown Bravo
    tag: PRG9CJ90
  - name: 3 Crown Charlie
    tag: P8L8VUCV
  - name: 3 Crown Delta
    tag: P9YP8G2V

Will do. I've updated my original to match the one you posted.


chore: Use actions/upload-artifact v4

it will avoid this message:
Deprecation notice: v1, v2, and v3 of the artifact actions The following artifacts were uploaded using a version of actions/upload-artifact that is scheduled for deprecation: "build.log", "tests.log".
Please update your workflow to use v4 of the artifact actions. Learn more:
https://github.blog/changelog/2024-04-16-deprecation-notice-v3-of-the-artifact-actions/
It wasn't updated automatically by dependabot because it is inside reusable actions.
It doesn't seem that this action are used in matrix jobs that can upload several times the same artifact so we should be fine just by upgrading the number
Description

Target

[ ] I checked that the commit is targeting the correct branch (note that Camel 3 uses camel-3.x, whereas Camel 4 uses the main branch)

Tracking

[ ] If this is a large change, bug fix, or code improvement, I checked there is a JIRA issue filed for the change (usually before you start working on it).

Apache Camel coding standards and style

[ ] I checked that each commit in the pull request has a meaningful subject line and body.

[ ] I have run mvn clean install -DskipTests locally and I have committed all auto-generated changes

/component-test jetty
to test that classic case is still working with component-test build

argh seems I missed some matrix job which are using these reusable actions (and which means that when playing a PR, it is no tusing the reusable github action from this same PR) :
https://github.com/apache/camel/actions/runs/9364653659/job/25778098025?pr=14327#step:9:73
Failed to CreateArtifact: Received non-retryable error: Failed request: (409) Conflict: an artifact with this name already exists on the workflow run


The polarization vector should be stored in the time-evolution operators

The polarization vector is currently only used in the LaserField-class to determine which coordinate of the dipole moment to include. Instead of self._polarization_vector as an attribute in the QuantumSystem-class this should instead be given to the time-evolution operators that make use of this vector. Thoughts on this @haakoek , @gregwinther ?

We can also make the polarization vector callable thus opening up for a varying polarization in time.


feat: progressive requestMapper option

What kind of change does this PR introduce?
requestMapper receives pre evaluated key and record props.
What is the current behaviour?
requestMapper must re-implement the whole default mapping logic
What is the new behaviour?
Default mapping logic is executed and results provided to custom requestMapper. This allows shorter requestMapper implementations
Does this PR introduce a breaking change?
Yes. requestMapper option receives and object of arguments.
Other information:
Please check if the PR fulfills these requirements:

[X] Tests for the changes have been added
[X] Docs have been added / updated

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

4 of 4 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
33

Relevant Lines:
33

💛  - Coveralls


Fix a bug that a redirection after editing a page can't work properly.

Problem
I've found that a redirection after editing a page can't work. The given error messages were the following:

Connect
400 Error: Bad Request

       at SendStream.error (/home/shinnya/repos/mine/crowi/node_modules/express/node_modules/send/index.js:245:16)
       at SendStream.pipe (/home/shinnya/repos/mine/crowi/node_modules/express/node_modules/send/index.js:418:32)
       at serveStatic (/home/shinnya/repos/mine/crowi/node_modules/express/node_modules/serve-static/index.js:113:12)
       at Layer.handle [as handle_request] (/home/shinnya/repos/mine/crowi/node_modules/express/lib/router/layer.js:82:5)
       at trim_prefix (/home/shinnya/repos/mine/crowi/node_modules/express/lib/router/index.js:302:13)
       at /home/shinnya/repos/mine/crowi/node_modules/express/lib/router/index.js:270:7
       at Function.proto.process_params (/home/shinnya/repos/mine/crowi/node_modules/express/lib/router/index.js:321:12)
       at next (/home/shinnya/repos/mine/crowi/node_modules/express/lib/router/index.js:261:10)
       at expressInit (/home/shinnya/repos/mine/crowi/node_modules/express/lib/middleware/init.js:23:5)
       at Layer.handle [as handle_request] (/home/shinnya/repos/mine/crowi/node_modules/express/lib/router/layer.js:82:5)

After editing the page(the path was /user/shinnya/メモ/2015/05/07/テスト), I had to get the path /user/shinnya/%E3%83%A1%E3%83%A2/2015/05/07/%E3%83%86%E3%82%B9%E3%83%88, but I got /user/shinnya/%E1%E2/2015/05/07/%C6%B9%C8.
Solution
I think we should use an urlencoded path for a redirection, so apply encodeURI to a path and use it.

Hi,
Thak you for your feedback.
I couldn't repro this bug but I've heard this problem from other user, so I'll merge this patch (the patch looks ok).
By the way, could you please tell me your node.js version?

I couldn't repro this bug but I've heard this problem from other user, so I'll merge this patch (the patch looks ok).

I guess something is wrong with node.js or express because we don't have to apply encodeURI normally.

By the way, could you please tell me your node.js version?

The version of my node.js is v0.12.2 and that of express is 4.12.3.

Thanks!
I think that the problem is caused by the internal stream logic of node.js so.


Small change to Child Reg Form

On the Child reg form, under section titled Other Family Constellation Considerations, change check box for "no pets", to read "CANNOT be in home with pets" and change order so that this is the last checkbox in this section.

@bchiaruttini I made an update to the Other Family Constellation Considerations model to allow you to control the ordering in Keystone.  I will demo for you when we meet, I'm hoping it will allow you to make changes like these in the future without the need for any code updates.

we reviewed how to make this change and I made in production, so story can be closed.


Set home.js controller as a module

The initial home.js controller is not a module after creating a new app.

will be in the release v1.0.0-rc.3


dataframe.ta.ticker return error

Which version are you running? The lastest version is on Github. Pip is for major releases.
pandas_ta-0.3.14b0
Describe the bug
When calling
pd.DataFrame().ta.ticker("spy")

it returns error

module 'yfinance' has no attribute 'pdr_override'

This method 'pdr_override' has been removed in yfinance, according to this post "https://github.com/ranaroussi/yfinance/issues/2001"

interesting... So you can just say the issue is deduplicated but instead, you can't resist the temptation of being sarcastic. I guess the world has been tough on you, and do hope you can live a better life


Support Extension Fields

Summary
docker-compose has the concept of extension fields: https://docs.docker.com/compose/compose-file/#extension-fields
These are YAML fields that start with x- and are ignored when validating if they should exist or not.  Argo should support a similar mechanism to be able to reduce duplicate configuration via YAML anchors.
Motivation
I currently use YAML anchors in my argo workflows when I have duplicate configuration or inputs to many templates.  Right now I apply the anchor in the spot where I first use the duplicate configuration.
I would prefer to be able to pull this configuration out to a custom key.
Proposal
Similar to the docker-compose solution- keys beginning with x- should simply be ignored by argo validations when submitting workflows.

Message from the maintainers:
If you wish to see this enhancement implemented please add a 👍 reaction to this issue! We often sort issues this way to know what to prioritize.

Interesting request, I'll bring this up with the team

Closing issue. I don't support docker-compose (who does these days?).


Replace read-package-json with vanilla JSON.parse

read-package-json inserts a number of fields that we likely don't want to write to the user's package.json file. This change replaces it with readFilep followed by JSON.parse.
We might lose some guarantee about the structure of the package.json file being read (seems like read-package-json does some extra checks)... but these cases might be unrecoverable either way.

Codecov Report

Merging #55 into master will decrease coverage by 0.39%.
The diff coverage is 50%.

@@            Coverage Diff            @@
##           master      #55     +/-   ##
=========================================
- Coverage   96.68%   96.28%   -0.4%
=========================================
  Files           8        8
  Lines         241      242      +1
  Branches        7        7
=========================================
  Hits          233      233
- Misses          8        9      +1

Impacted Files
Coverage Δ

src/util.ts
94.11% <50%> (-5.89%)
:arrow_down:

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 69db7ab...6443a7c. Read the comment docs.


add turret building

[x] building attacks
[x] turret building that fires upon nearby enemy units and buildings

Turrets are now functional.


Add multi-token stop sequence

Closes #104

@lucasavila00, I moved this to #111 as I needed to make some changes. Sorry for any confusion!


Added 1.0.3 release notes

Signed-off-by: Steven Bayer<EMAIL_ADDRESS>Issue #, if available:
Description of changes:
Add 1.0.3 release notes

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Date updated


Logic error in Firewall rule any/any created ?

In the following rule : Firewall rule any/any created
The related EVTX is here
I think the important fields are :
"LocalAddresses": "*",
"LocalPorts": "",
"RemoteAddresses": "*",
"RemotePorts": "",

But the rule only checks for LocalPorts  and RemotePorts :
detection:
  selection:
    EventID:
      - 2004  # new rule created
      - 2005  # existing rule modified
    Action: 3 # allow
    LocalPorts: *
    RemotePorts: *

Shouldn't you add a check for  LocalAddresses and RemoteAddresses equal to '\*' ?

Thanks for the advice. I have added both approaches in the rules, including your suggestion. Thanks


Critically many APIs without AuthorizationReference in AT environments

Description of the bug
We use AuthorizationReference directly to create the request that does the DelegationCheck for an api. Without it we get the error below because we're creating a DelegationCheck with no data.
Blocked by
endpoint: https://platform.at23.altinn.cloud/resourceregistry/api/v1/resource/search
not having AuthorizationReference
Proposed solution:

Hide all apis without AuthorizationReference from DelegableApis

Steps To Reproduce

Go to .../offered-api-delegations/choose-api
Click on add button for an API

Additional Information
No response

Think this task fixed itself after resource registry fixed this.


Update and rename README.md to Thebox

Enjoy

Hmmm. Interesting.
What exactly is this for? As far as I know there shouldn't be anything wrong with the README. Perhaps your own fork called 'Thebox'?


Refactor R.of so that it works with Applicatives (BREAKING CHANGE)

Currently of can only wrap a thing into an array
of(42) //> [42]

However "of" is also a well know feature of Applicatives:
Maybe.of(42) //> Just(42)

Proposal
@Bradcomp made this suggestion:
of(Array, 42) //> [42]
of(Maybe, 42) //> Just(42)
of(Either, 42) //> Right(42)

Which makes a lot of sense and gained popular support so far.
Related issue(s): #2502

@CrossEye: I added it to the v1.0 milestone. By the look of it it seems like this PR #2687 does what we need. Perhaps we could revive it?

I think that PR is probably fine for this.  Somewhere, post 2.0, we should really implement all of FantasyLand.  Doing this one right is a good start, and getting it fixed before 1.0 makes sense.


Module 2 exercises [Nutifafa Ahiawodzi]

Please "drag & drop" exercise 1 pdf file below.
Exercise 1.pdf

Please "drag & drop" exercise 2 pdf file below.
Exercise 2.pdf

Please "drag & drop" exercise 3 pdf file below.
Exercise 3.pdf

Please "drag & drop" exercise 4 pdf file below.
Exercise 4.pdf

Please UPDATE THE TITLE of this "Issue" in the title bar above with your name.

If you haven't done so, please go back to Module 2 and make sure to take
the post-training survey. Participating in surveys is required for receiving
the "GALUP Certificate" by the University of Florida.

Great work, Nutifafa. You clearly grasped the material outlined in Module 2 and successfully applied it to these exercises. Thank you for your participation!


Problem with the VS Code Stop Debugging API

Hi all,
My objective is to create a VS Code Extension which can ATTACH debugger to a running .NET Console Application
I have a .NET Console Application running parallelly (The Console Application was not attached to the Visual Studio Debugger)
And I have created a VS Code Extension. (Used the default "Hello World" Extension)
In the Extension, I had the below code:
// Attach the debugger --- This is working as per the expectation
vscode.debug.startDebugging(undefined, {
		name: ".NET Core Attach",
		type: "coreclr",
		request: "attach", // Please note that the request is ATTACH
		processId: "18876" // Manually configured the Console Application's process ID
}, undefined);

// This is a dummy timer which waits for 5 secs
setTimeout(() => {
	//This piece of code is stopping the debugger BUT ALSO TERMINATE MY CONSOLE APPLICATION which is not expected
	vscode.debug.stopDebugging();

	//I have tried the below code as well, but no luck
	//vscode.debug.stopDebugging(vscode.debug.activeDebugSession);

}, 5000)

I am not sure why the vscode.debug.stopDebugging() API terminates the Console Application (Since the Debug Configuration request is ATTACH (but not LAUNCH)
Please help me understand what I am doing wrong here.
Thanks in Advance,
Susee

@SuseeJeeva the comment of debug.stopDebugging explains the behavior you are seeing:

Stop the given debug session or stop all debug sessions if session is omitted.

If you want to make sure that the correct session gets stopped, I suggest that you you register an event handler onDidStartDebugSession and remember the session inside the handler after verifying it.
Relying on debug.activeDebugSession is tricky because you never know what debug session is active.
var myDebugSession: vscode.DebugSession | undefined;
vscode.debug.onDidStartDebugSession(session => {
    if (session.configuration.name === '.NET Core Attach') {
       myDebugSession = session;
    }
});
setTimeout(() => {
   if (myDebugSession) {
      vscode.debug.stopDebugging(myDebugSession);
   }
}, 5000);

@weinand , thank you for your comments.
I am still facing the same issue even if I follow the steps you suggested.
My requirement is, I wanted to achieve the below operation:

Attach the debugger to the existing .NET Console application (This I am able to achieve with the startDebugging API
Disconnect the debugger from the existing .NET Console application. (This is not working as expected because the stopDebugging API is Stopping (Terminating) the process. But my requirement is to just Disconnect it (Without impacting the actual process's state.

However, If I click on the "Disconnect" Icon manually in the "Extension Development Host", it is properly getting disconnected.
Attaching the image for your reference.

@SuseeJeeva I've fixed the issue - please try tomorrow's Insiders.

@weinand Thank you for the quick support. Sure, I shall try with the Insiders

@weinand , the issue is resolved in the Insiders version.
Thanks,
Susee

@weinand great, thanks for verifying my fix!


feature request: support for external key management

I'm not 100% clear exactly on how sops works. But from my limited understanding the intended use was for aws vms where sops would communicate with KMS to decrypt the secrets.
Unless I am mistaken, sops-nix doesn't currently support KMS or any other similar service. The request would be to add support for KMS (or similar) and perhaps this could open the door to using the host machine as the key provider when remotely building machines through nixos-rebuild

To add further context on this:
I am trying to update remote machines through nixos-rebuild and would like to also update the secrets. However this will only work if I encrypt the secrets file with a key the remote machine has access to which means (as I have been gracefully told by an awesome discord member) that I need to:

Distribute a master key to each remote machine which is able to decrypt the secrets (which doesn't seem terribly secure)
Create keys on the remote machine, grab the public key and encrypt the secrets with it (which seems like it just will not scale and generally be a pain to do for every new machine)

The tricky bit is that the activation phase doesn't have network. So what people in the past did, was enabling networking in initrd. An alternative that I used in the past on GCP, was having a systemd service that would just run plain sops to decrypt secrets but also depending on networking and than every service that depends on it, would depend on the service.

Hey! Thanks for the reply. I see.... That seems tough then.
 This way I can bootstrap the server and set it up so it can self serve somehow maybe
Do you have an example for the work around you used in GCP?
What about doing something like on nixos-rebuild switch do a first pass to generate the secrets and a second to actually build the system now that the secrets are in the right place? (Feels like if this were possible someone would have thought of it
Do you happen to know what this project does to get around having to set systemd service dependencies? (I don't see it referencing dependencies) https://github.com/serokell/vault-secrets


Refactor Redis configuration to fix crash loop

This PR fixes a bug discovered while helping a customer deploy 1Password SCIM bridge on DigitalOcean. The marketplace app fails to deploy, and the Redis pod shows the following output in the logs:
*** FATAL CONFIG FILE ERROR (Redis 7.0.2) ***
Reading the configuration file, at line 11
>>> 'maxmemory 256mb "--maxmemory-policy volatile-lru"'
wrong number of arguments

In this PR, the configuration is passed to Redis using the commonConfiguration key in the Redis common configuration parameters rather than the extraFlags key in the Redis master confiiguration, so it's a little more universal as an added bonus.

Unfortunately this does not seem to append the redis command line arguments as expected.
The actual run command for the redis server is bundled into a config map.
Using these changes the config map for op-scim-bridge-redis-scripts looks like this:
{
	"start-master.sh": "#!/bin/bash

		[[ -f $REDIS_PASSWORD_FILE ]] && export REDIS_PASSWORD=\"$(< \"${REDIS_PASSWORD_FILE}\")\"
		if [[ ! -f /opt/bitnami/redis/etc/master.conf ]];then
		    cp /opt/bitnami/redis/mounted-etc/master.conf /opt/bitnami/redis/etc/master.conf
		fi
		if [[ ! -f /opt/bitnami/redis/etc/redis.conf ]];then
		    cp /opt/bitnami/redis/mounted-etc/redis.conf /opt/bitnami/redis/etc/redis.conf
		fi
		ARGS=(\"--port\" \"${REDIS_PORT}\")
		ARGS+=(\"--requirepass\" \"${REDIS_PASSWORD}\")
		ARGS+=(\"--masterauth\" \"${REDIS_PASSWORD}\")
		ARGS+=(\"--include\" \"/opt/bitnami/redis/etc/redis.conf\")
		ARGS+=(\"--include\" \"/opt/bitnami/redis/etc/master.conf\")
		exec redis-server \"${ARGS[@]}\"
		"
}

Vs the expected output when using extraFlags on the redis.master:
{
	"start-master.sh": "#!/bin/bash

		[[ -f $REDIS_PASSWORD_FILE ]] && export REDIS_PASSWORD=\"$(< \"${REDIS_PASSWORD_FILE}\")\"
		if [[ ! -f /opt/bitnami/redis/etc/master.conf ]];then
		    cp /opt/bitnami/redis/mounted-etc/master.conf /opt/bitnami/redis/etc/master.conf
		fi
		if [[ ! -f /opt/bitnami/redis/etc/redis.conf ]];then
		    cp /opt/bitnami/redis/mounted-etc/redis.conf /opt/bitnami/redis/etc/redis.conf
		fi
		ARGS=(\"--port\" \"${REDIS_PORT}\")
		ARGS+=(\"--requirepass\" \"${REDIS_PASSWORD}\")
		ARGS+=(\"--masterauth\" \"${REDIS_PASSWORD}\")
		ARGS+=(\"--include\" \"/opt/bitnami/redis/etc/redis.conf\")
		ARGS+=(\"--include\" \"/opt/bitnami/redis/etc/master.conf\")
		ARGS+=(\"--maxmemory 256mb\")
		ARGS+=(\"--maxmemory-policy volatile-lru\")
		exec redis-server \"${ARGS[@]}\"
		"
}

Note the inclusion of the two parameters we set.
I'm going to update main with some changes to the way we set the extraFlags and resolve this MR in the process. The PR should be up in a day or so.

Ah. Thanks for the additional context. I guess the piece that was missing for me is how the values from the config map are consumed by redis.
I was wondering how you ensured that the config options were applied to the running instance of redis?

I finally found the place where these are set, i.e. the config map. Thanks for your guidance.

I'm going to include these changes in the other PR I'm working on, i.e. #78 . That's set to close this one when it is merged.

I hadn't actually verified whether Redis was being configured with these values, but it seemed to be correcting the error, and I could see from describing the ConfigMap that the values were being added to redis.conf:
…

Data
====
master.conf:
----
dir /data
# User-supplied master configuration:
rename-command FLUSHDB ""
rename-command FLUSHALL ""
# End of master configuration
redis.conf:
----
# User-supplied common configuration:
maxmemory 256mb
maxmemory-policy volatile-lru
# End of common configuration

…

I was able to since confirm this by cherry-picking this commit onto your latest work, running redis-cli and polling the configuration there:

kubectl exec --stdin --tty deploy/op-scim-bridge-redis-master -- /opt/bitnami/redis/bin/redis-cli
CONFIG GET maxmemory*

This shows that the values are configured correctly by Redis.


it doesn't log arabic request into log file

Laravel Version
v10.48.2
PHP Version
PHP 8.2.16
Database Driver & Version
No response
Description
i use a request logger to log errors
and the framework converts arabic characters into unicode
example:
  "RESPONSE": "{\"success\":false,\"message\":\"\\u0627\\u0644\\u0628\\u064a\\u0627\\u0646\\u0627\\u062a \\u062e\\u0627\\u0637\\u0626\\u0629.\"}",
  "LOCALE": "ar"

Steps To Reproduce
        $response = response()->json(['message' => 'صباح الخير']);
        Log::error($response->getContent());

Nothing unusual, JSON always uses the Unicode character set. See RFC 8259. Nothing wrong with this, use json_decode.

Hi there,
Thanks for reporting the problem you are encountering, but it looks like this is a question which may be better suited for a support channel. We only use this issue tracker for reporting bugs with the library itself. If you have a question on how to use functionality provided by this repository you can try one of the following channels:

Laracasts Forums
Laravel.io Forums
StackOverflow
Discord
Larachat
IRC

However, this issue will not be locked and everyone is still free to discuss solutions to your problem!
Thanks.


Paths are not properly resolved with Windows

After initializing an app using react-create-app, installing react-toolbox and react-toolbox-themr  and adding the script to package.json as so:
"toolbox": "react-toolbox-themr"

Calling npm run toolbox from the command prompt runs the script, but results in nothing. No content is added to /public. Using Windows 10.
Thanks!

Please try with the latest beta.6. If it stills fails with the latest patch and the latest beta, please reopen!

With the latest react-toolbox and react-toolbox-themr from Github, I am getting the following error, although I followed exactly the directions shown in the GIF.
> react-toolbox-themr

module.js:472
    throw err;
    ^

Error: Cannot find module 'C:\...\node_modules\react-toolbox\lib\identifiers.js'

@javivelasco I don't have the option of reopening, so I'm tagging you as per your request :)

Damn Windows. I can't reproduce it locally, help needed for this :(

@javivelasco tell me where to look and what to look for and I'll look into this :)

I'm running this with Win10 and beta.6 and I haven't had any issues.  @stereokai - What does the reactToolbox section of your package.json file look like?

@stereokai check your src/toolbox folder if you haven't overridden the output in packages.json/reactToolbox

It looks like this was generated by something else and can be closed! Maybe it's an issue with an specific Windows version or something? Please reopen if the problem persists!

Got it for osx, no idea why yet 👯

Ok, cross that ^ the issue was with 2 package json electron repo and somehow missing node_nodules in /app folder


support application/merge-patch+json for PATCH

Cadl Ranch Contribution Checklist:

[ ] I have written a scenario spec
[ ] I have meaningful @scenario names. Someone can look at the list of scenarios and understand what I'm covering.
[ ] I have written a mock API
[ ] I have used @scenarioDocs for extra scenario description and to tell people how to pass my mock api check.

I haven't modified code out of cadl-ranch-specs, could use some help on doing changeset


Finish the KickOffActivity on dialog tap outside

Fixes an issue where tapping outside the reauth dialog would result in an invisible activity hanging around blocking clicks.

@amandle good catch!


Add upload capability to distutils plugin #187

fixes #187, connected to #187

@mriehl review please

LGTM :+1:

I would like to try this, however I can't seem to be able to install:
zsh» pip install git+https://github.com/pybuilder/pybuilder.git
Collecting git+https://github.com/pybuilder/pybuilder.git
  Cloning https://github.com/pybuilder/pybuilder.git to /tmp/pip-MwlRTf-build

anaconda:root [pybuilder:master:★] ~/gw/pybuilder vhaenel@isdeblnwl116 ₍★★₎
zsh» pyb install
PyBuilder version 0.11.1
Build started at 2015-09-14 09:11:36
------------------------------------------------------------
[INFO]  Building PyBuilder version 0.11.2.dev0
[INFO]  Executing build in /data/home/vhaenel/git/pybuilder
[INFO]  Going to execute task install

It just seems to hang during installation.

let me try to repro

It doesn't, it just takes a long time, it has to pull all the build dependencies in:
(blah)bash-3.2$ time pip install git+https://github.com/pybuilder/pybuilder.git
You are using pip version 7.0.1, however version 7.1.2 is available.
You should consider upgrading via the 'pip install --upgrade pip' command.
Collecting git+https://github.com/pybuilder/pybuilder.git
  Cloning https://github.com/pybuilder/pybuilder.git to /var/folders/p0/m0sr26yj2rjb18qsqgx15b100000gn/T/pip-jSknzM-build
Requirement already satisfied (use --upgrade to upgrade): tblib in ./lib/python2.7/site-packages (from PyBuilder==0.11.2.dev0)
Requirement already satisfied (use --upgrade to upgrade): six in ./lib/python2.7/site-packages (from tblib->PyBuilder==0.11.2.dev0)
Installing collected packages: PyBuilder
  Running setup.py install for PyBuilder
Successfully installed PyBuilder-0.11.2.dev0

real	0m58.444s
user	0m24.996s
sys	0m7.735s
(blah)bash-3.2$

get the latest master please @esc

@esc bump. Works for you now?

Haven't had the time to try it yet.

:+1:

Yeah, I have waited for 90 minutes now, can't see anything in top...

Can you strace the pip process? Maybe some network issue.

can you look at the logs? @esc in target/reports/distutils

I mean, I can't install the latest version from git. Neither by doing pip install git+http://... or by doing pyb install from a git checkout!

strace on pip just says it hangs trying to read:
mmap(NULL, 1052672, PROT_READ|PROT_WRITE, MAP_PRIVATE|MAP_ANONYMOUS, -1, 0) = 0x7f0deb0c1000
read(5, "", 1048576)                    = 0
mremap(0x7f0deb0c1000, 1052672, 4096, MREMAP_MAYMOVE) = 0x7f0deb0c1000
close(4)                                = 0
close(5)                                = 0
munmap(0x7f0deb0c1000, 4096)            = 0
fstat(3, {st_mode=S_IFIFO|0600, st_size=0, ...}) = 0
fcntl(3, F_GETFL)                       = 0 (flags O_RDONLY)
read(3,

Probably some issues with my machine, installation and distro.

I made fresh 2.7 virtualenv and now I get:
1 zsh» pyb install
PyBuilder version 0.11.1
Build started at 2015-09-16 16:27:09
------------------------------------------------------------
[INFO]  Building PyBuilder version 0.11.2.dev0
[INFO]  Executing build in /data/home/vhaenel/git/pybuilder
[INFO]  Going to execute task install
[INFO]  Running unit tests
[INFO]  Executing unit tests from Python modules in /data/home/vhaenel/git/pybuilder/src/unittest/python
------------------------------------------------------------
BUILD FAILED - 'module' object has no attribute 'distutils_plugin_tests'
------------------------------------------------------------
Build finished at 2015-09-16 16:27:10
Build took 0 seconds (740 ms)

You can't build pyb with a previous version. You need to pip uninstall it and then ./build.py install

Exanding: you're building master with the pypi release, which messes up the imports. You don't need to install pybuilder to build pybuilder, you just run the build.py

@esc in a virtual env do pip install git+... or pip install https://github.com/.../archive/master.tgz

@esc
(current)aimobile-mbp:current arcivanov$ time pip install http://github.com/pybuilder/pybuilder/archive/master.tar.gz
Collecting http://github.com/pybuilder/pybuilder/archive/master.tar.gz
  Downloading http://github.com/pybuilder/pybuilder/archive/master.tar.gz (118kB)
    100% |████████████████████████████████| 118kB 121kB/s
Requirement already satisfied (use --upgrade to upgrade): tblib in ./lib/python2.7/site-packages (from PyBuilder==0.11.2.dev0)
Requirement already satisfied (use --upgrade to upgrade): six in ./lib/python2.7/site-packages (from tblib->PyBuilder==0.11.2.dev0)
Installing collected packages: PyBuilder
  Running setup.py install for PyBuilder
Successfully installed PyBuilder-0.11.2.dev0

real	0m11.031s
user	0m4.859s
sys	0m1.588s

So I think I can install it in a fresh virtualenv. However, I am also trying to install it into the conda root environment and that still seems to hang.
It might be nice to automatically upload pre-releases to PyPi as part of the CI process. One could then install those with:
pip install --pre pybuilder

As pre-release number you could use the number of commits since the last tag.

Okay, doing it with master.tgz worked.

Automatic travis upload is a great idea. I think they have support for encrypted credentials so we could go with that.

It would go down well with the continuous delivery / continuous live deployment mindset :+1:

Also, I just checked pyb upload and it seems to do what I want. I couldn't fully verify, since I don't have a new release, but I'll come back in case it really doesn't work. @arcivanov thanks again for implementing this, it will save many, many keystrokes on my end.

:+1:


Add more comment for SyntaxKind enum in type.ts

Add comments(best with examples) for SyntaxKind, just like TypeFlags, ElementFlags, CharacterCodes and many other enum types.
This would make it more clear for new developers (like me :) ), especially in scanner and parser.

I find textToKeywordObj and textToToken in scanner, but I still really want at least some link.


GH-2610 set version of slf4j-simple in bom

GitHub issue resolved: #2610
Briefly describe the changes proposed in this PR:

this makes it possible to use the slf4j-simple library in RDF4J-based projects without having to set the version explicitly

PR Author Checklist (see the contributor guidelines for more details):

[x] my pull request is self-contained
[ ] I've added tests for the changes I made
[x] I've applied code formatting (you can use mvn process-resources to format from the command line)
[x] I've squashed my commits down to one or a few meaningful commits
[x] every commit message starts with the issue number (GH-xxxx) followed by a meaningful description of the change
[x] every commit has been signed off

This is fairly trivial so I'll go ahead and merge this quickly.


Driver Service Fix v2

@goosewobbler please note that I continued working on your PR over the last days

Oh, now I see it is a PR to it. Taking a look 👀

Yeah, I think it's getting there.  My local builds are passing now and my local repo should be reflected in this PR, CI still having issues, doesn't seem to be a consistent test failing - I've had everything from 3 passes 1 failure to 1 pass 3 failures...
Happy to do a call, shall I pick a slot on the Open Office Hours Calendly?

shall I pick a slot on the Open Office Hours Calendly?

That would be awesome, thanks!
If it helps to debug the build, try https://github.com/stateful/vscode-server-action which allows you to open VS Code within the machine that runs in CI: https://github.com/stateful/vscode-server-action


not clear about enable password sync option on the sync rule editor

I have a question regarding this statement "If a user is in scope for this sync rule, the password is synchronized from on-premises to cloud (assuming you have enabled the password sync feature)."
If I have password synchronization turned off in azure ad connect wizard but the sync rule "in from AD- User account enabled" has "enable password sync enabled", would password be synchronized or would it not?

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

ID: 698bae7a-c593-1a6c-151b-6bfa7c540270
Version Independent ID: 0b29c2df-4697-827e-acae-5d31f886789a
Content: Azure AD Connect sync: Understanding the default configuration
Content Source: articles/active-directory/hybrid/concept-azure-ad-connect-sync-default-configuration.md
Service: active-directory
Sub-service: hybrid
GitHub Login: @billmath
Microsoft Alias: billmath

@Neenisharma Thanks for the Comment. We are actively investigating and will get back to you soon.

@Neenisharma I do not think the password will be synchronized . The password sync uses a separate engine on-premise and needs to be enabled through Azure AD connect wizard Because when you enable password sync through wizard, the tenant level capability on your tenant gets switched on and tenant starts accepting passwords. So it wont work if you just mark the checkbox on the sync rules to enable password sync because the changes which are made on-premise once the azure AD connect enables password synchronization would not be there and the PHS engine would not be running within the AD connect sync engine on the AAD connect box . At this point we will close this issue . In case you see any alternate behavior , please tag me or the author of this document @billmath to your reply and we will be happy to help you.
Thank you .


fix(cts): headers should not be encoded

🧭 What and Why
This PR makes sure that headers are not encoded the same way as query parameters, and it also check for recursive encoding of query parameters.

#2704 should fix Swift here

💙


android build initialisation

initialise the plugin gradle stuff on install
#869

apparently before_plugin_rm calls each plugin hooks, so the second commit just make sure the cleaning of the gradle file is done only if cordova-plugin-firebase removal was triggered...hopefully

@briantq since you did a lot of the work to get the after_plugin_add hook working, would you mind testing this fix to verify it still works as you expect?

Based on the issue details in #869 it appears the plugin add hook is not run when adding a platform where as the plugin install is run in that case. This will allow users to add the plugin before adding a platform if desired.

Based on the comments I agree, but I would love to see some actual Cordova documentation on why you would use one instead of another.  I wonder if there are other differences.  Again, probably something we will never get

Actually I have the same use case. My build scripts clean out the plugins and platforms directories before doing a build. It then adds the platform being built. This causes Cordova to install all the plugins from config.xml. I haven't spent any time on my firebase apps recently or I likely would have encountered the same issue.
For more info, the reason I must clear out my plugins on each build is due to changing the variable values used for certain plugins based on the environment I'm building. The Facebook plugin requires setting the app ID which is different for my debug and production environments.

@soumak77 why do you have to remove/add the platform?  I understand having to clear out the plugins and platform directory, but why not include both platforms and simply run cordova prepare android instead of installing it?

If I remember correctly, I have to at least remove the platform I'm building otherwise Cordova will report the platform is already added and therefore not install the plugins again automatically. Since I have to remove one platform I just remove them all to ensure a fully clean build.

Missed your last sentence. I guess I never tried using cordova perpare.

@briantq I think a key difference between the add/remove and install/uninstall hooks is that the add/remove hooks are run whenever any plugin is added/removed, whereas the install/uninstall hooks are only run when the specific plugin is being added/removed.  This is in fact documented in the Cordova hooks guide.

before_plugin_add/after_plugin_add

To be executed before and after adding a plugin.

before_plugin_install/after_plugin_install

To be executed before and after installing a plugin (to the platforms). Plugin hooks in plugin.xml are executed for a plugin being installed only

You'd likely want to use the plugin_add if you must make changes based on another plugin being added.  Whereas you would want to use plugin_install to make changes whenever your plugin is added, regardless of any other plugins that exist in the project.


Release 1.1.0

1.1.0
Added

Add MaxQuantResultRegistry for MaxQuant analysis result datasets (#12 <https://github.com/qbicsoftware/java-openbis-dropboxes/pull/12>_)

Route MaxQuant result datasets to the MaxQuantResultRegistry (#13 <https://github.com/qbicsoftware/java-openbis-dropboxes/pull/13>_)

Learn MaxQuant analysis result dataset structure (<https://github.com/qbicsoftware/java-openbis-dropboxes/pull/14>_)

Fixed
Dependencies
Deprecated

The dependency section is missing from the changelog


module_source won't let you specify providers

Fogg's module_source feature doesn't let you specify providers such as:
module foobar {
    source = ""asfdasdfaf"

    providers {
        aws = aws
        aws.us-east-1 = aws.us-east-1
    }
}

While we're here, may also want to future proof for other cases we want to add things to a module, such as Terraformm 0.13's depends_on, or possibly for_each/count


Make collapse optional

I need a dynamic tree view where all items are always visible, and the user cannot collapse parts of the tree. Judging from the code, toggleCollapse is hardcoded, so I cannot prevent the collapsing without a hack or custom fork. It would be nice if it were possible to do something like <ion-tree-list collapsible="false" ...>.
I think this would be fairly trivial to implement, I could try to make a patch myself if you prefer that. Would you merge such a patch?

I'd suggest you propose a pull request. I am more than happy to have a look very soon and comment/merge your changes.
Thank you very much!


Fixed Proxmox 7.0  and Mimetype

I added required Packages in the readme, and fixed the mimetype detection (a at least that was necessary for me).
Also did some cleanup.

I've added the changes to my current commit already, but thanks for the PR :tada:


Use Real SSL Cert

How do I use a real SSL cert with Posh?
Thanks

In the project folder there is a posh.crt and posh.key. If you replace those after the project is created you can use real certificates.


Improvements

Hello.
I've been following Typemill for a while now. The way this CMS is changing is great.
But in my opinion it would be a pity if such a good CMS would only be used for documentation and stuff like that. I know that's the way you want it to be. But please remember: The only other CMS that allows such a free definition of forms is KirbyCMS, which is commercial. Typemill would be a good open source approach to close this gap.
I also don't demand that the form (for documentation) be neglected, but a few more features to be able to use it for regular websites would be great! For example, the ability to create multiple designs and then select which design to use in the editor. (If this is already possible I apologize and would like to know how to do this :)   ).
Regards
Andi

Hi Andi,
thank you very much for that feedback, I am happy that some people take notice of this small project!
Yes, I plan to make the system a bit more flexible in future with a template-selection that you mentioned and some more features, so that you can also realize more general website concepts with typemill. This requires meta-information for pages. I plan to implement this after I have finished the wysiwyg-editing mode.
Like you said, I am still not sure how far to go in this direction (general website cms). There are so many good and mature options out there even in the space of flat file systems: Kirby and Statamic or Open Source Systems like Grav, Bludit, BatFlat, Automad and many more. Not sure if Typemill has a real usp in this space ...
The main goal for typemill is to create a small publishing system with a single source publishing based on files, some options to create ebook-formats on the fly and maybe even options to distribute and sell the text-work in the background. I don't see really good solutions for publishers and writers right now and I know that some of them are waiting for it. But it is allways hard to decide which way to go and keep on the path ... :D
The template selection will take some time because the wysiwyg editing is pretty complex, so not before the end of the year, I think ...

Hi Andi,
I will close this ticket because Typemill has evolved within the last 2 years and you can build much more than documentation sites with it now.


File promise-7.0.4.min.js.map is missing

[ ] question
[X ] any problem or bug report

OS:  (check one with "x")

[ ] Android
[ X] iOS
[ ] Browser

cordova information:  (run $> cordova plugin list)
cordova-plugin-googlemaps 2.4.6
cordova-plugin-whitelist 1.3.3
Current behavior:
Map not showing
Expected behavior:
Map should show
Screen capture or video record:

Additional Information:
I can add marker and use location button
In brower work

Gray map means your api key does not allow to use the Google Maps SDK for iOS.
https://github.com/mapsplugin/cordova-plugin-googlemaps-doc/blob/master/troubleshootings/graymap/README.md
The js error is not related and affect it.

What about the console error?

That's just there is no file error, that file is not necessary to use this plugin.

The problem was i didn't enabled the api for ios. Tanhks and sorry if have bother you.

I am again facing this issue in Ionic Cordova with High Sierra Xcode 10.
cordova-android-support-gradle-release 1.4.7 "cordova-android-support-gradle-release"
cordova-plugin-accountkit 1.4.0 "Cordova AccountKit"
cordova-plugin-advanced-http 1.11.1 "Advanced HTTP plugin"
cordova-plugin-background-mode 0.7.2 "BackgroundMode"
cordova-plugin-browsertab 0.2.0 "cordova-plugin-browsertab"
cordova-plugin-buildinfo 1.1.0 "BuildInfo"
cordova-plugin-camera 3.0.0 "Camera"
cordova-plugin-compat 1.2.0 "Compat"
cordova-plugin-device 1.1.7 "Device"
cordova-plugin-file 6.0.1 "File"
cordova-plugin-googlemaps 2.4.6 "cordova-plugin-googlemaps"
cordova-plugin-inappbrowser 1.7.2 "InAppBrowser"
cordova-plugin-ionic-webview 1.2.1 "cordova-plugin-ionic-webview"
cordova-plugin-request-location-accuracy 2.2.3 "Request Location Accuracy"
cordova-plugin-splashscreen 4.1.0 "Splashscreen"
cordova-plugin-statusbar 2.4.2 "StatusBar"
cordova-plugin-whitelist 1.3.3 "Whitelist"
cordova.plugins.diagnostic 4.0.10 "Diagnostic"
ionic-plugin-keyboard 2.2.1 "Keyboard"
mx.ferreyra.callnumber 0.0.2 "Cordova Call Number Plugin"
onesignal-cordova-plugin 2.4.6 "OneSignal Push Notifications"
`
Ionic:

   ionic (Ionic CLI)  : 4.5.0 (/usr/local/lib/node_modules/ionic)
   Ionic Framework    : ionic-angular 3.6.0
   @ionic/app-scripts : 2.1.3

Cordova:

   cordova (Cordova CLI) : 8.1.2<EMAIL_ADDRESS>   Cordova Platforms     : android 6.4.0, ios 4.5.4
   Cordova Plugins       : cordova-plugin-ionic-webview 1.2.1, (and 17 other plugins)

System:

   Android SDK Tools : 26.1.1 (/Users/meenuaggarwal/Library/Android/sdk)
   ios-deploy        : 1.9.4
   NodeJS            : v8.12.0 (/usr/local/Cellar/node/11.1.0/bin/node)
   npm               : 6.4.1
   OS                : macOS High Sierra
   Xcode             : Xcode 10.1 Build version 10B61

`

Yeah, I can help. Read the posts in the link that you Googled.


chore: cargo clippy --fix && rustfmt format_code_in_doc_comments wrap_comments

Motivation
Run cargo clippy fix with all features enabled and format with more rustfmt configs

Solution

cargo clippy --fix --all-features && cargo fmt --all -- --config format_code_in_doc_comments=true wrap_comments=true

@mattsse given the 1559 changes were happening, I didn't want to merge it back then because it'd cause a lot of conflicts. Do you want to re-apply the same commands on top of master?

Closing due to multiple changes in the codebase :S Bad timing, if not much effort and you want to re-execute the command that works for me!


Add drawio file for project infrastructure

The version added via this PR would generate this diagram:

cc @embray
Also added to https://zenodo.org/record/4390110#.X-NX_dhKiUk

Needed edit: Benchmarks is split into astropy-benchmarks and coordinates-benchmarks.

Ideas to reduce maintenance burden:

Reduce scope of some of the packages; i.e., stop accepting new features. Also see #159
Plan to retire some, if possible. For example: Docker images for CircleCI.
Automate some parts of the maintenance work.

For this particular PR to add a diagram, I think @embray said he has plans to clean it up and then get that version merged. Please correct me if I misunderstood.

Ideas to reduce maintenance burden:

Reduce scope of some of the packages; i.e., stop accepting new features. Also see #159
Plan to retire some, if possible. For example: Docker images for CircleCI.
Automate some parts of the maintenance work.

Recruit more maintainers, with roles limited to specific packages ?

Re: https://github.com/astropy/astropy-project/pull/122#issuecomment-718374218
Yes, that is also a possibility but not actionable until we know the funding situation more (Moore? pun?).

Probably outdated by now and there is Zenodo link added to OP.


Transaction nonces and submitter accounts

In GP we have one address from which we submit solutions per solver. There we don't have to think about nonces much because batches are 5 minutes apart so the only thing to worry about was cancelling potentially cancelling the previous solution submission for example if the gas price wasn't high enough.
In OBA this is more complicated because we settle different token pairs independently. If we use the same account for all transactions then we need to make sure that we increment nonces properly and that transactions always go through (maybe because we manually cancel them). We could have a something like a transaction manager object that knows about all transactions we are currently trying to send and is able to adjust their gas prices or cancel them.
Or we could try having multiple submitter accounts so that they influence eachother less. We could for example cycle through 10 different accounts or have one account per token pair (which doesn't seem feasible because there are too many).

Closing as I believe this issue is no longer relevant:

We detect pending solution transactions and wait for them to be mined on startup.
We merge multiple settlements into one, so we don't need multiple accounts atm.


Using ui-scroll with grid, all ui-scroll-td elements are placed in first column

Hi,
I am using a ui-scroll viewport with a table.  I've got a data array with five properties per element which I'm trying to display in five columns. When I specify the viewport as a div containing the table element and don't use ui-scroll-th/ui-scroll-td, it is almost perfect.  The only problem is that the column headings scroll with the table, which I don't want.
The obvious answer seems to be to use the ui-scroll-viewport directly inside the table element, and specify the th and td elements with ui-scroll-th and ui-scroll-td, in a similar way to the grid-layout-apply example.  This displays the headers as I want them and they don't scroll.  However, in the data area, all of the columns appear under the first column heading.
I have created a plunker that show the issue:
https://plnkr.co/edit/WO3upSyKY30w5WjsPUKR
This is based on the grid-layout-apply.  Please can you look at this and let me know what I'm doing wrong.
Thanks,
Giles

Hi,
Some more info after playing around with the element styles in the browser...
For the top table which uses ui-scroll-td:
tbody->element.style->display is set to "block".  This overrides the setting of "table-row-group", which is set in tbody->display.
If I change this to "table-row-group", the columns are displayed in full width under the correct headers.  However, the table is no longer scrollable so this is no good.
In the bottom table (which defines the ui-scroll-viewport as a div containing the table), element.style is empty and so display is set to "table-row-group" (as defined in tbody).  The "div ui-scroll-viewport" surrounding the table is set to "display:block" which I guess is making the whole table scrollable.
I hope this makes sense and helps to solve the issue.  I'm not sure where to go from here.
Thanks,
Giles

Hi,
Is this issue list still active and monitored?
Thanks,
Giles

@gillez please take a look to my fork of your plunker^ https://plnkr.co/edit/N8LMYtS06rWJ0SRVWNOI?p=preview
I did simple update on your stylesheet, maybe this would be enough to solve your issue....
Also I had some experience with pure ui.scroll based grid (without ui.scroll.grid):  https://github.com/dhilt/ui-scroll-grid This also may help you if you want not to use ui.scroll.grid. This product is a bit excessive but I believe that the static header solution could be obtained without big effort.

Hi @dhilt,
Thanks for the reply and the effort.  Looking at your plunkr, I can see that you have set the display of both thead and tbody to "block".  This is interesting and almost gives the results that I need.  However, it stops the table from using the full width.
You can see this easily if you add enough data to one of the elements to stretch over multiple lines.  This actually much more closely resembles my real world case.  I have updated my plunkr to show this.  When increasing the string length to the fifth column, the table now correctly stretches to the full width again.  However, the column headings do not.  Therefore the headings do not align with the data.
https://plnkr.co/edit/p6oHvtsMbyJ1isjQqVhV
Looking at this in the browser debugger, both the thead and tbody highlight the full width when you hover over those elements.  When you look at the actual width of each column, they both show the same (correct) percentage width for the appropriate columns.  Therefore it all looks like it should work and they should align.  I'm not sure why the headings don't actually stretch to the full width.
I haven't yet looked at your ui-scroll-grid.  Ill try it out - maybe that will be better suited to my use case.
Thanks,
Giles

@gillez looks like it's not so easy to succeed with this issue (100% width static table header) in HTML world... There are some techniques which can be found with Google... Most of them suggest destructive DOM intervention via JS. Of course it would be very cool to have a demo for ui.scroll/ui.scroll.grid with 100% static table header, but currently we don't have it (and you know it is mostly out of ui.scroll responsibility).

Thanks @dhilt.  It's a shame this isn't straightforward with ui.scroll.  I think it's a great lightweight library, which I've been using for a while in more straightforward cases, and thought the ui-scroll-th/td directives would be perfect for my current table use-case.
Oh well, it can't be helped.   I'll have to look at either manipulating the DOM to keep the headers static or combine it wit smart-table or similar.
Thanks for your help and advice,
Giles


Update device-health-status.md

'statue' changed to 'status'

Reassigning to @dstrome, who is the writer for Teams Rooms and devices topics


Feature/select

Use intersection for scroll pagination - https://github.com/streamich/react-use/blob/master/docs/useIntersection.md


Use composition

WIP - working on services and parameters

I feel like using composition is making the service and parameter examples quite a bit harder to follow. (pub-sub's not too bad)
Does anyone have an opinion on if this is worthwhile for tutorials? We might be better off sacrificing best practices for readability when we are presenting these topics for the first time.

Just to recap here, I think @dhood is going to migrate her comments/documentation to the composition demo: https://github.com/ros2/demos/blob/master/composition/src/listener_component.cpp and https://github.com/ros2/demos/blob/master/composition/src/talker_component.cpp
And @Karsten1987 is going to add service and client examples in the composition demos repository.
And then we're going to drop this repository from the repos file for beta 1 since we're not ready to make complete tutorials yet.
Is that how everyone else understood it?

closing in favour of https://github.com/ros2/demos/pull/97 and https://github.com/ros2/demos/pull/95


Can I show one validation error at a time?

I have a field First name which is alpha_spaces|required. And when the user blurs leaving the field empty, the validation fails and shows the following error: May only contain letters and spaces. Field is required.
What I want is, when the user blurs, the error Field is required should only be shown; but then when the user types in something other than alphabets and spaces, we show the error message May only contain letters and spaces.
Is there a way to achieve this??

Correct me if I'm wrong, but I think this is where the issue is: https://github.com/ghiscoding/angular-validation/blob/master/src/validation-common.js#L536
If we can take the input for jln through an attribute then this could be possible. Isn't it?

Nevermind. I got it. displayOnlyLastErrorMsg was the key.


DBZ-7379 Connector specific keystore/truststore

https://issues.redhat.com/browse/DBZ-7379

@jpechane I will send a separate PR which allows testing this with external database. However from our preliminary testing with @obabec  it looks like this is good to go.

@jcechace Applied, thanks! BTW, it might be nice as a follow-up task to extract the SSL config into a separate class.

@jcechace Applied, thanks! BTW, it might be nice as a follow-up task to extract the SSL config into a separate class.

Definitively something I want to do. Just need to have a chat with you about the design. It's a bit more complicated than I initially though and I'd like to created something that can eventually be pulled up for other connectors as well.


Lua rewrite seems to have broken functionality on macOS

I noticed a recent PlugUpdate seemed to stop vim-illuminate working (no error messages though). Walking back through the commits, it seems to have been introduced by the lua rewrite (09235da12cfdad2a4f695610a079b20f1b263b4a).
Testing with a minimal config:
set nocompatible
filetype off

call plug#begin(stdpath('data') . '/plugged')
Plug 'RRethy/vim-illuminate'
call plug#end()

filetype plugin indent on

When editing files (tested with nvim -u illuminate.vimrc), the word-under-cursor is no longer being highlighted. No errors are reported.
Doing a git checkout 09235da12cfdad2a4f695610a079b20f1b263b4a^ for the repo fixes things.
Reproduced across multiple macOS machines. This one is running:
% nvim --version
NVIM v0.7.2
Build type: Release
LuaJIT 2.1.0-beta3
Compiled by<EMAIL_ADDRESS>
Features: +acl +iconv +tui

With latest macOS:
% sw_vers
ProductName:	macOS
ProductVersion:	12.6
BuildVersion:	21G115

Thanks for making and maintaining this plugin - it's one I rely on a lot for my day-to-day work.

You might need to update your highlight groups to use IlluminatedWordText, IlluminatedWordRead, and IlluminatedWordWrite (they should be showing underline by default). I added a new command to help debug this, try running :IlluminateDebug and paste the output here.

Also fwiw these lines aren't needed in Neovim:
set nocompatible
filetype off
filetype plugin indent on

That's weird 🤔 these groups have defaults, maybe your colorscheme is being loaded after the plugins and it's calling hi clear. Anyways, glad the fix was small.

With the rewrite you might also want to check the migration guide, https://github.com/RRethy/vim-illuminate/issues/111. There's some new functionality:

You can now use `<a-n>` and `<a-p>` to jump around the references.

You can also use `<a-i>` as a textobject for the reference under the cursor.

That's weird 🤔 these groups have defaults, maybe your colorscheme is being loaded after the plugins and it's calling hi clear. Anyways, glad the fix was small.

This was the case for me. I changed my plugin load order so that vim-illuminate came after my colorscheme. Then my vim.api.nvim_set_hl settings worked as expected (see below).
require("illuminate").configure({
  providers = {
    "regex",
  },
  delay = 50,
  filetype_overrides = {},
  filetypes_denylist = { "NvimTree", "alpha", "packer", "qf", "Outline", "fugitive" },
  filetypes_allowlist = {},
  modes_denylist = {},
  modes_allowlist = {},
  providers_regex_syntax_denylist = {},
  providers_regex_syntax_allowlist = {},
  under_cursor = false,
})

vim.api.nvim_set_hl(0, "IlluminatedWordText", { link = "Visual" })
vim.api.nvim_set_hl(0, "IlluminatedWordRead", { link = "Visual" })
vim.api.nvim_set_hl(0, "IlluminatedWordWrite", { link = "Visual" })


Pre-Commit Hook

Information

Qiskit Metal version: Latest as of 11/8
Python version: 3.1
Operating system: Mac

What is the current behavior?
The pre-commit hook looks at BOTH .py and .ipynb files.
Steps to reproduce the problem
I tried to delete a .ipynb file that was no longer needed, and the yapf formatting raised an error when I tried to commit this change.
What is the expected behavior?
The pre-commit hook should only look at .py files, not .ipynb files.
Suggested solutions
Modify the pre-commit hook so that it only looks at .py files, not .ipynb files.

Merged, should be closed.


Change SwelvyBG Sprites to be 2048x512

Original sprite size was 2000x500 which is not a power of two, causing the sprite to not render on some devices (ios 💔)

why would this fix anything? many many textures in gd are not even close to powers of 2

GL_REPEAT causes textures to not render on some devices if the image size isn't a power of two

not sure which gles version GD is using, but the spec for gles 2.0 glTexParameter (here, notes section) mentions that the width and height of the texture should be powers of two if the wrapping mode is not GL_CLAMP_TO_EDGE


Only JSON

There is a problem with file size.
For csv files, the file sizes are correct but as soon as I change it to json the file size cause the read operation to terminate prematurely.
Same is for thor files.


Update to golang 1.18

This change updates the Dockerfile.local to use go1.18.
This is part of https://github.com/m-lab/dev-tracker/issues/735

This change is

Dockerfile.local line 1 at r1 (raw file):

FROM golang:1.18-alpine3.14 as ndt-server-build

Can we use golang:1.18 (instead of golang:1.18-alpine3.14) so we don't have to install git, gcc, etc. on the next line?
This is actually what I did for dash, disco, uuid, and uuid-annorator.
Also, per official Go documentation: This variant is highly experimental, and not officially supported by the Go project (see golang/go#19938 for details).  Details: https://hub.docker.com/_/golang

Pull Request Test Coverage Report for Build 1616

0 of 0   changed or added relevant lines in 0 files are covered.
12 unchanged lines in 5 files lost coverage.
Overall coverage decreased (-0.5%) to 81.295%

Files with Coverage Reduction
New Missed Lines
%

ndt5/ndt5.go
1
89.5%

ndt5/c2s/c2s.go
2
73.21%

ndt5/protocol/protocol.go
2
83.86%

ndt5/web100/web100_linux.go
2
94.44%

ndt7/upload/sender/sender.go
5
65.91%

Totals

Change from base Build 1613:
-0.5%

Covered Lines:
1795

Relevant Lines:
2208

💛  - Coveralls

Dockerfile.local line 2 at r2 (raw file):

FROM golang:1.18 as ndt-server-build
RUN apk add --no-cache git gcc linux-headers musl-dev openssl bash

Since you are using golang:1.18, you shouldn't need this line (and apk doesn't exist).


model update doesn't set values

it does with .Table but I think it should with .Model too
DBcon.Debug().Table("users").Where("ID =?", test_user_id).Updates(map[string]interface{}{"notify_tkn": "shittytoken", "notify_via": "a"})
[2016-04-04 00:36:07]  [0.39ms]  UPDATE `users` SET `notify_tkn` = 'shittytoken', `notify_via` = 'a'  WHERE (ID ='UserUserUserUserUser01')

DBcon.Debug().Model(User{}).Where("ID =?", test_user_id).Updates(map[string]interface{}{"NotifyTkn": "shittytoken", "NotifyVia": "a"})
[2016-04-04 00:36:07]  [0.37ms]  UPDATE `users` SET `notify_tkn` = '', `notify_via` = '', `updated_at` = '2016-04-04T00:36:07+02:00'  WHERE (ID ='UserUserUserUserUser01')

DBcon.Debug().Model(User{}).Where("ID =?", test_user_id).Updates(User{NotifyTkn: "shittytoken", NotifyVia: "a"})
[2016-04-04 00:36:07]  [0.37ms]  UPDATE `users` SET `notify_tkn` = '', `notify_via` = '', `updated_at` = '2016-04-04T00:36:07+02:00'  WHERE (ID ='UserUserUserUserUser01')

UPDATE:
I forgot to note, that this used to work.
I reverted back to, because I would have to change all the code to .Table to make my app work again.
.Table has a disadvantage, when a models attribute is changed all queries need be refactored by hand, while attribute names in
DBcon.Debug().Model(User{}).Where("ID =?", test_user_id).Updates(User{NotifyTkn: "shittytoken", NotifyVia: "a"})

would change automatically.
working version for those who upgraded and have problems (getgb.io)
		"importpath": "github.com/jinzhu/gorm",
		"repository": "https://github.com/jinzhu/gorm",
		"revision": "aa45da15421ddb1549a7060e5c189f8350a29166",
		"branch": "master"

Hi @rand99
The User{} should be addressable,  so you need to write code like this:
db.Model(&User{}).Update(...), then it should works.
But anyway, I have pushed a commit to fix the issue, you should fine now, but still prefer you to make the change, especially when you are updating an existing record like this:
db.Model(&user).Update(...), this will update the data in database, and update the record's value user as it is addressable

Thx!
I feels nice to be able to use a pointer for a user-Instance that was loaded
and just the the model with a where(ID=1).updates(y)


[Multiple Data Source][2.13.0] Update the existing multiple data source document

Description
Update the existing multiple data source document to include the new changes introduced in version 2.13.0:
Feature flags:

data_source.authTypes
data_source.hideLocalCluster

Issues Resolved
Closes #6667
Checklist

[ ] By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license and subject to the Developers Certificate of Origin.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@xinruiba @bandinib-amzn @BionIT , please take a look this PR as well.
cc: @seraphjiang

Thanks, @Flyingliuhub ! Please let us know when it is ready for a doc review.

Thanks, @Flyingliuhub ! Please let us know when it is ready for a doc review.

Thanks @hdhalter , it is ready for a doc review now. Please let us know if there any questions, so we can quickly update PR.

Thanks, @Flyingliuhub ! Please let us know when it is ready for a doc review.

Thanks @hdhalter , it is ready for a doc review now. Please let us know if there any questions, so we can quickly update PR.

@Flyingliuhub Doc review is completed. I rewrote narrative for improved readability. Please review to confirm information retained technical accuracy. Once you approve, I'll move this into editorial review and then merge the PR. Thank you, Melissa

@Flyingliuhub Doc review is completed. I rewrote narrative for improved readability. Please review to confirm information retained technical accuracy. Once you approve, I'll move this into editorial review and then merge the PR. Thank you, Melissa

Thanks @vagimeli , There are a few words that need to be changed "This option hides the local cluster from the data source dropdown and index creation page, which is suitable for environments with or without a local OpenSearch cluster."

@Flyingliuhub Doc review is completed. I rewrote narrative for improved readability. Please review to confirm information retained technical accuracy. Once you approve, I'll move this into editorial review and then merge the PR.

Thank you Melissa and Heather. It looks good to me.


decomposed activities not generated (version 1.0.17)

Hi,
I ran SigProfilerExtractor 1.0.17 on my data. After the process ended, I didn't see any activities table under COSMIC_ID83_Decomposed_Solution. I guess it should generate Activities and Signatures folder as in ID83_De-Novo_Solution.
Decomposition folder only has

Decomposition_Plots
De_Novo_map_to_COSMIC_ID83.csv
ID83_Decomposition_Plots.pdf
Solution_Stats

ID83_De-Novo_Solution folder has Activities, Signatures and Solution_stats folder.
I was wondering if there is anything I am missing here?
Thank you.

Hi Venu,
I am looking at the issue and will back to you soon.

@mishugeb, here is the error message I got
  File "/home/thatikon/anaconda3/lib/python3.7/site-packages/SigProfilerExtractor/sigpro.py", line 1022, in sigProfilerExtractor
    final_signatures = sub.signature_decomposition(processAvg, m, layer_directory2, genome_build=genome_build, add_penalty=add_penalty, remove_penalty=remove_penalty, mutation_context=mutation_context, make_decomposition_plots=make_decomposition_plots, originalProcessAvg=originalProcessAvg)
  File "/home/thatikon/anaconda3/lib/python3.7/site-packages/SigProfilerExtractor/subroutines.py", line 1410, in signature_decomposition
    shutil.rmtree(directory+"/Decomposition_Plots")
  File "/home/thatikon/anaconda3/lib/python3.7/shutil.py", line 491, in rmtree
    _rmtree_safe_fd(fd, path, onerror)
  File "/home/thatikon/anaconda3/lib/python3.7/shutil.py", line 449, in _rmtree_safe_fd
    onerror(os.unlink, fullname, sys.exc_info())
  File "/home/thatikon/anaconda3/lib/python3.7/shutil.py", line 447, in _rmtree_safe_fd
    os.unlink(entry.name, dir_fd=topfd)
OSError: [Errno 16] Device or resource busy: '.nfs0000000189c1c3c200000342'

Hi Venu,
Please use "make_decomposition_plots=False" until I give you the next update. I was unable to reproduce the issue on my settings.

Hi Mishu,
Thanks for the tip. With "make_decomposition_plots=False", it is generating activities file.

Hi Mark @mdbarnesUCSD, could you please reproduce the issue?

@mishugeb, here is the error message I got, jic if this might help tracing the problem
  File "/home/thatikon/anaconda3/lib/python3.7/site-packages/SigProfilerExtractor/sigpro.py", line 1022, in sigProfilerExtractor
    final_signatures = sub.signature_decomposition(processAvg, m, layer_directory2, genome_build=genome_build, add_penalty=add_penalty, remove_penalty=remove_penalty, mutation_context=mutation_context, make_decomposition_plots=make_decomposition_plots, originalProcessAvg=originalProcessAvg)
  File "/home/thatikon/anaconda3/lib/python3.7/site-packages/SigProfilerExtractor/subroutines.py", line 1410, in signature_decomposition
    shutil.rmtree(directory+"/Decomposition_Plots")
  File "/home/thatikon/anaconda3/lib/python3.7/shutil.py", line 491, in rmtree
    _rmtree_safe_fd(fd, path, onerror)
  File "/home/thatikon/anaconda3/lib/python3.7/shutil.py", line 449, in _rmtree_safe_fd
    onerror(os.unlink, fullname, sys.exc_info())
  File "/home/thatikon/anaconda3/lib/python3.7/shutil.py", line 447, in _rmtree_safe_fd
    os.unlink(entry.name, dir_fd=topfd)
OSError: [Errno 16] Device or resource busy: '.nfs0000000189c1c3c200000342'

I also got the same error as yours.
In my case, I changed the working directory from nfs storage to the local server's home directory.
The thing is that the error occurred not constantly.

Thanks @itsvenu and @leefall for bringing this up. I am looking into reproducing this error.


Fix type in custom_classes.md

📚 Context
Fix a small typo: differnt to different
Size:

[x] Small
[ ] Not small

Contribution License Agreement
By submitting this pull request you agree that all contributions to this project are made under the Apache 2.0 license.

Thank! LGTM.


Allow deselecting entities in entity tree with CTRL left click

(Ignition Citadel 3.3.0) CTRL clicking can be used to select entities, but not deselect them. Other programs do allow deselecting by CTRL left clicking again (example Microsoft Excel)

Hello, could I take a look at this issue?

@herobank110 , feel free to pick it up, I don't think anyone is working on this. Thanks!

Thanks @WagnerMarcos ! I have assigned to you and moved to "In Progress".  Feel free to tag me as a reviewer when your PR is up.

In this Issue the idea is that when holding ctrl and using left click on a selected entity from the entity tree, it gets deselected. In EntityTree.qml this click gets detected and then calls EntityTree.cc to send a selection event. The visual part in qml is simple and straightforward, but the C++ is taking more time than expected.
I'm trying to get right the extent of the Issue, as the events gets filtered in Scene3D, Component Inspector (which could be ignored at the moment), Joint Position Controller (also ignorable) and the AlignTool.
For this Issue I'm thinking I have some alternatives I could follow. The first one would be to define a new event and make parallelism between the selection/deselection. A similar one to this would be to have some boolean to tell if the clicking interaction with the entity was for selecting or deselecting.
Another alternative would be to ask in the plugin that filter this event when a selection was made, if that item was already selected, and if so, to deselect it. But for this there could be it should be checked if the selection was made with the ctrl key pressed, and that might be messy.
Things to consider would be how to do the same when an entity gets click on the gazebo scene, to also update the model correctly.
For the Component Inspector, would it be ok to set it kNullEntity when deselecting? (This way, it would show the world properties)
And for the AlignTool it only updates if the process is not user created, so no changes might be needed there.

I assume that the behavior of this feature, which was requested for the entity tree, should be paired with Scene3D plugin of course. So as there is a consistency between which entity is selected in the entity tree and the highlighted entities in the scene.
From here, some expected behaviors are detached.

First of all, we have the visual upgrade and the actual request of the ticket: Deselecting entities in the entity tree by ctrl-clicking its entities.

Propagation to Scene3D:

When deselecting an entity in the entity tree should also "lowlight" the entity in the Scene that previously was highlighted.

Propagation to EntityTree from other plugins.

Scene3D should allow you to deselect the entities in the Scene by ctrl-clicking on them.
EntityTree should be reflected with the current changes in the Scene (selection).

I think that focusing on how Scene3D and EntityTree deal with this new feature is the key to this issue.
From what I saw in ComponentInspector, JointPositionController and AlignTool I don't think that this new deselection feature in the EntitiyTree should necessary enforce to change something in the behavior of those plugins, at least for the moment.

yes, it turns out this is a bit more complicated than I had anticipated; I'm going to remove the "good first issue" label

First of all, we have the visual upgrade and the actual request of the ticket: Deselecting entities in the entity tree by ctrl-clicking its entities.

Propagation to Scene3D:

When deselecting an entity in the entity tree should also "lowlight" the entity in the Scene that previously was highlighted.

Propagation to EntityTree from other plugins.

Scene3D should allow you to deselect the entities in the Scene by ctrl-clicking on them.
EntityTree should be reflected with the current changes in the Scene (selection).

I think this proposal makes sense, but there is currently a large effort to move much of the functionality from these plugins to ign-gui (https://github.com/ignitionrobotics/ign-gui/issues/137), so I think it may be best to hold off on this work for now. It can be revisited after the larger migration is done

I'll put it on hold then and work on another issue then. I'll try to keep up with that migration so I know how it affects the issue for when I come back to it.


git_pull don't need to restart HA if the only change is ui-lovelace.yaml

git_pull don't need to restart HA if the only change is ui-lovelace.yaml

This was implemented in version 6: restart_ignore is now an option and ui-lovelace.yaml is added by default.


Can't show Size and Color

What is your question / Please describe your issue
Hi, I could show my Shopify products and categories on demo application: https://shopify-pwa.aureatelabs.com/, but I cannot manage to show sizes and colors. It seems like the site can count how many variation of sizes I have. However, all texts are missing.  When I use inspector on google web browser I see the value of sizes is empty and I guess they cannot wrriten as String? I have English and Swedish as acceptLanguage in middleware.config.js. How can I solve this problem? I found out that some links to documents are missing. For example, on this page, the link to Internationaliztion is missing: https://docs.vuestorefront.io/shopify/guide/configuration.html#nuxt-shopify-configuration. Is there something I missed becuase of the missing document?

What version of shopify integration are you using?
latest
Code of Conduct

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

Removing .value from the attributes in the product.vue file will help you :)


Error: no schema available for artifactory_virtual_repository.default_helm while reading state

Describe the bug
When trying to upgrade to the new repository types from the old ones, while upgrading to provider v6, I get this error:

Error: no schema available for artifactory_virtual_repository.default_helm while reading state; this is a bug in Terraform and should be reported

Expected behavior
I can upgrade with no error.
Additional context
This seems to be because I have a 2.x state stored and am upgrading to 6.x.
I believe either the schema needs to be updated, or a documentation note needs to be added that a user must upgrade to 5.x and fix all deprecations before upgrading to 6.x.

@jcogilvie @benoit-parmentier-2 The user created this script in https://github.com/jfrog/terraform-provider-artifactory/issues/290#issuecomment-1100757651 may be of interest to you guys.

@alexhung - we need to close this issue, and/or create a new one where we develop a migration script. I am going to close this issue and creating a migration script can be at a later date


refactor: Use web pack for javascript dependency bundling

This PR changes the build system of datavzrds javascript dependencies by using webpack and npm.

I think the voyager-stuff should be removed for now, correct?

A sorry I missed the template right? Should be removed now!


New files never scanned without database reset - rclone docker volume

When mounting /app/data/images using the rclone docker volume plugin, new files added to the directory (after pigallery starts) are never scanned/displayed until the database is reset. Simply indexing does not help.
Notably, new files can still be found in pigallery by direct paths (i.e. going to pigallery.example.com/gallery/newdir prompts a rescan and displays all images correctly, but newdir is never shown at pigallery.example.com/gallery until the database is reset).
The problem appears to be in the combination of pigallery and rclone: mounting local directories works fine, so sorry if this question/report is misplaced. However, in defense of rclone itself, this exact volume on the same machine works perfectly well with other services, such as filebrowser and Nextcloud even, pigallery alone seems to have issues.
I'm wondering: what could be causing this issue in pigallery that is not present in other services?
My understanding is that metadata on the mounted volume behaves the same, regardless of what docker volume driver is used.
Edit: for context, briefly, the rclone docker plugin allows to seamlessly mount a great number of cloud storage providers/protocols (from FTP, to Dropbox, to S3) as volumes into containers.
Environment:

OS: server runs on linux, browser on linux
Browser: firefox 101.0.1
App version: docker bpatrik/pigallery2:latest (00328c4b2f92)
Docker version 20.10.17, build 100c701

docker-compose.yml
volumes:
  rclone-volume:
    driver: rclone
    driver_opts:
      remote: 'myremote:'
      allow_other: 'true'
      vfs_cache_mode: 'full'

services:

  pigallery2:
    image: bpatrik/pigallery2:latest
    environment:
      - NODE_ENV=production
    volumes:
      - ./pigallery2/config:/app/data/config
      - ./pigallery2/db:/app/data/db
      - ./pigallery2/tmp:/app/data/tmp
      - rclone-volume:/app/data/images
    expose:
      - 80
    restart: always

  caddy:
    ...

Full config
All defaults with the exception of log level "silly".
{
	"Server": {
		"sessionSecret": [
			"42dbb20a1da200a70a3b6944c62a5cf74981f836eb6e5be528b575fb1383788761588d15dd00d69d37cf0eaf029765aaa589d1827b48ca56124e74151c1512178d104e0ca07bc83edd73739751624bb1dc9112f2a8386fd2882ff448920736813d45e16e3cb0c917fca66741b90f442369d18d39d817e3898fbe82be7af8a33935aa57fc07caedcb081a090e871d1aee4bc1a3869cd3d9f4dfffa1c210c6f29cbccb08e5d78d3321688f30da1280757002cfb1a3f11beaad6c523345a97c054a20a7706e75de3f1ef3fd17aaf97b89503d2075d16c39c91e7e03390b2df3dda74ebb7c34b9ca2f167d356de5b35bdafb0bc66637a1f1583ff3a5c8d293d52eb4",
			"e7c26eb2f19605d70e1de39401c75637ebd7210bd2f9c845bf4ead3b2ad0cd98f85377ac2efe97ccecaaac3a031bd3559cc8e86d1bf1be4beecca35bc9ab8caf07da3a0738b09fd517cf1bcdfaaeee4a1864f34ce80e7700a4fc678fe05eed14bc05338259bab18dd196e47a06835f4a22f8a8bcc023576b3d94babe87c743f15efa2776736a4e6e628e5c9fc3b2f70daaeec5d17eeee452985f33f089f8c7e7ab642b096bf26b5e912dae7e420cd5da72ca0e7d4ac3118625b772d19cd097ea52a4d9d06cf94cabdb0f32d0609cef8be096243eb3c77439a23a45e558c324c76de4fade9a36946afd8133899e2005d46127cca64db1d8a3e9927319b11be1d3",
			"e1ea47e4ab494e6adb2573a736f0346b7206b65bae74e8d800e8dd768c374e9cc9434597cbf4faaf73f5610307c83ac9b7fee9cb6cb77a5863b37a4ab947da6df0d5bdb59f161d3502c0c77b42af22ebf17c35daa2a9f5a9c849bb49289b5df6a80cbc3ea2975af2c5c6ab1f9b1995a3f72f9f854d0a71ad8c3035ceb105ce806447e01f7143028cd87e0a0a6bbaf5cbf9c8a567609c8da5e49cca4236202be571547cbae8226849a4e14006aa894e9d6a5d499eafe3d29c3fbace5af8a6f3c8dd910011c391939857b33605ac20d781cbb3ab8ec3962158bb1670f79fb45f76e3425d358424015aa240fcd722ad69e0ee17e53b3b96bb31e248456df7359934"
		],
		"port": 80,
		"host": "<IP_ADDRESS>",
		"Media": {
			"//[folder]": "Images are loaded from this folder (read permission required)",
			"folder": "/app/data/images",
			"//[tempFolder]": "Thumbnails, converted photos, videos will be stored here (write permission required)",
			"tempFolder": "/app/data/tmp",
			"Video": {
				"transcoding": {
					"bitRate": 5242880,
					"resolution": 720,
					"fps": 25,
					"codec": "libx264",
					"format": "mp4",
					"//[crf]": "Constant Rate Factor. The range of the CRF scale is 0–51, where 0 is lossless, 23 is the default, and 51 is worst quality possible.",
					"crf": 23,
					"//[preset]": "A preset is a collection of options that will provide a certain encoding speed to compression ratio",
					"preset": "medium",
					"//[customOptions]": "It will be sent to ffmpeg as it is, as custom options.",
					"customOptions": []
				}
			},
			"Photo": {
				"Converting": {
					"//[onTheFly]": "Converts photos on the fly, when they are requested.",
					"onTheFly": true,
					"resolution": 1080
				}
			},
			"Thumbnail": {
				"//[qualityPriority]": "if true, photos will have better quality.",
				"qualityPriority": true,
				"personFaceMargin": 0.6
			}
		},
		"Preview": {
			"SearchQuery": {
				"type": 100,
				"text": ""
			},
			"Sorting": [
				6,
				4
			]
		},
		"Threading": {
			"//[enabled]": "App can run on multiple thread",
			"enabled": true,
			"//[thumbnailThreads]": "Number of threads that are used to generate thumbnails. If 0, number of 'CPU cores -1' threads will be used.",
			"thumbnailThreads": 0
		},
		"Database": {
			"type": "sqlite",
			"dbFolder": "/app/data/db",
			"sqlite": {
				"DBFileName": "sqlite.db"
			},
			"mysql": {
				"host": "localhost",
				"port": 3306,
				"database": "pigallery2",
				"username": "",
				"password": ""
			},
			"//[enforcedUsers]": "Creates these users in the DB if they do not exist. If a user with this name exist, it wont be overwritten, even if the role is different.",
			"enforcedUsers": []
		},
		"Sharing": {
			"updateTimeout": 300000
		},
		"//[sessionTimeout]": "unit: ms",
		"sessionTimeout": 604800000,
		"Indexing": {
			"cachedFolderTimeout": 3600000,
			"reIndexingSensitivity": "low",
			"//[excludeFolderList]": "If an entry starts with '/' it is treated as an absolute path. If it doesn't start with '/' but contains a '/', the path is relative to the image directory. If it doesn't contain a '/', any folder with this name will be excluded.",
			"excludeFolderList": [
				".Trash-1000",
				".dtrash",
				"$RECYCLE.BIN"
			],
			"//[excludeFileList]": "Any folder that contains a file with this name will be excluded from indexing.",
			"excludeFileList": []
		},
		"//[photoMetadataSize]": "only this many bites will be loaded when scanning photo for metadata",
		"photoMetadataSize": 524288,
		"Duplicates": {
			"listingLimit": 1000
		},
		"Log": {
			"level": "silly",
			"sqlLevel": "error",
			"logServerTiming": false
		},
		"Jobs": {
			"//[maxSavedProgress]": "Job history size",
			"maxSavedProgress": 10,
			"scheduled": [
				{
					"name": "Indexing",
					"jobName": "Indexing",
					"config": {
						"indexChangesOnly": true
					},
					"allowParallelRun": false,
					"trigger": {
						"type": "never"
					}
				},
				{
					"name": "Preview Filling",
					"jobName": "Preview Filling",
					"config": {},
					"allowParallelRun": false,
					"trigger": {
						"type": "never"
					}
				},
				{
					"name": "Thumbnail Generation",
					"jobName": "Thumbnail Generation",
					"config": {
						"sizes": [
							240
						],
						"indexedOnly": true
					},
					"allowParallelRun": false,
					"trigger": {
						"type": "after",
						"afterScheduleName": "Preview Filling"
					}
				},
				{
					"name": "Photo Converting",
					"jobName": "Photo Converting",
					"config": {
						"indexedOnly": true
					},
					"allowParallelRun": false,
					"trigger": {
						"type": "after",
						"afterScheduleName": "Thumbnail Generation"
					}
				},
				{
					"name": "Video Converting",
					"jobName": "Video Converting",
					"config": {
						"indexedOnly": true
					},
					"allowParallelRun": false,
					"trigger": {
						"type": "after",
						"afterScheduleName": "Photo Converting"
					}
				},
				{
					"name": "Temp Folder Cleaning",
					"jobName": "Temp Folder Cleaning",
					"config": {
						"indexedOnly": true
					},
					"allowParallelRun": false,
					"trigger": {
						"type": "after",
						"afterScheduleName": "Video Converting"
					}
				}
			]
		}
	},
	"Client": {
		"applicationTitle": "PiGallery 2",
		"publicUrl": "",
		"urlBase": "",
		"Search": {
			"enabled": true,
			"searchCacheTimeout": 3600000,
			"AutoComplete": {
				"enabled": true,
				"targetItemsPerCategory": 5,
				"maxItems": 30,
				"cacheTimeout": 3600000
			},
			"maxMediaResult": 10000,
			"//[listDirectories]": "Search returns also with directories, not just media",
			"listDirectories": false,
			"//[listMetafiles]": "Search also returns with metafiles from directories that contain a media file of the matched search result",
			"listMetafiles": true,
			"maxDirectoryResult": 200
		},
		"Sharing": {
			"enabled": true,
			"passwordProtected": true
		},
		"Album": {
			"enabled": true
		},
		"Map": {
			"enabled": true,
			"//[maxPreviewMarkers]": "Maximum number of markers to be shown on the map preview on the gallery page.",
			"maxPreviewMarkers": 50,
			"useImageMarkers": true,
			"mapProvider": "OpenStreetMap",
			"mapboxAccessToken": "",
			"customLayers": [
				{
					"name": "street",
					"url": ""
				}
			]
		},
		"RandomPhoto": {
			"//[enabled]": "Enables random link generation.",
			"enabled": true
		},
		"Other": {
			"customHTMLHead": "",
			"enableCache": true,
			"enableOnScrollRendering": true,
			"defaultPhotoSortingMethod": "ascDate",
			"//[enableDirectorySortingByDate]": "If enabled directories will be sorted by date, like photos, otherwise by name.
Try this setting this to a higher value (so the app will more aggressively reindex):


Docs: Reorganize data source files into how-to-guides section (re-do)

Re-do of https://github.com/grafana/plugin-tools/pull/960 to fix Prettier error.
Complete 3rd stage of the plugin-tools site reorganization project. Move/rename site, fix broken links.
Instructions for testing redirects: docusaurus/website/README.md#Redirects
Fixes: https://github.com/grafana/plugin-tools/issues/925
Testing: Refer to design doc

@josmperez as with the app how-to section, I've tried to get this to what I believe is a mergeable state pending your review. The redirect and relative links issues you were encountering I believe is due to us having different file names and IDs in some cases. At somepoint we could tidy this up across the site, but would rather do that in a separate PR.
@academo could you please take a look at the final commit in particular on splitting the variables section? I felt we needed to do it as the panel plugin content didn't make sense in the original file. I think only the one section was appropriate for panels.

Thanks for your work in getting this PR mergable, David! Much appreciation.

No problem 😄