Error d/l

seems to always be getting the issue below when running "vagrant up"
drupalvm: Downloading: https://atlas.hashicorp.com/geerlingguy/boxes/ubuntu1604/versions/1.0.4/providers/virtualbox.box
The box failed to unpackage properly. Please verify that the box
file you're trying to add is not corrupted and try again. The
output from attempting to unpackage (if any):
x Vagrantfile
x box.ovf
x metadata.json
x packer-ubuntu-16.04-amd64-disk1.vmdk: gzip decompression failed
bsdtar: Error exit delayed from previous errors.

@jminker - Can you run the following command and see if it works?
$ vagrant box add geerlingguy/ubuntu1604

Also, what is your host OS/environment, and if Windows, what CLI tool are you using?

That worked;
vagrant box add geerlingguy/ubuntu1604
==> box: Loading metadata for box 'geerlingguy/ubuntu1604'
box: URL: https://atlas.hashicorp.com/geerlingguy/ubuntu1604
==> box: Adding box 'geerlingguy/ubuntu1604' (v1.0.4) for provider: virtualbox
box: Downloading: https://atlas.hashicorp.com/geerlingguy/boxes/ubuntu1604/versions/1.0.4/providers/virtualbox.box
==> box: Successfully added box 'geerlingguy/ubuntu1604' (v1.0.4) for 'virtualbox'!
Version;
uname -a
Linux Ubuntu-JDM 4.4.0-31-generic #50-Ubuntu SMP Wed Jul 13 00:07:12 UTC 2016 x86_64 x86_64 x86_64 GNU/Linux
Where should I now pick up from with vagrant ?
Thanks for your help!!

Sure thing! Try vagrant up again, and I think it'll work.
It was likely a network issue... happens from time to time, for mysterious reasons.

you must hate people like me..
vagrant up
Bringing machine 'drupalvm' up with 'virtualbox' provider...
==> drupalvm: Preparing master VM for linked clones...
drupalvm: This is a one time operation. Once the master VM is prepared,
drupalvm: it will be used as a base for linked clones, making the creation
drupalvm: of new VMs take milliseconds on a modern system.
==> drupalvm: Importing base box 'geerlingguy/ubuntu1604'...
There was an error while executing VBoxManage, a CLI used by Vagrant
for controlling VirtualBox. The command and stderr is shown below.
Command: ["import", "/home/john/.vagrant.d/boxes/geerlingguy-VAGRANTSLASH-ubuntu1604/1.0.4/virtualbox/box.ovf", "--vsys", "0", "--vmname", "packer-ubuntu-16.04-amd64_1470073131288_38049", "--vsys", "0", "--unit", "7", "--disk", "/home/john/VirtualBox VMs/packer-ubuntu-16.04-amd64_1470073131288_38049/packer-ubuntu-16.04-amd64-disk1.vmdk"]
Stderr: 0%...10%...20%...30%...40%...50%...60%...70%...80%...90%...100%
Interpreting /home/john/.vagrant.d/boxes/geerlingguy-VAGRANTSLASH-ubuntu1604/1.0.4/virtualbox/box.ovf...
OK.
0%...
Progress state: VBOX_E_FILE_ERROR
VBoxManage: error: Appliance import failed
VBoxManage: error: Could not create the imported medium '/home/john/VirtualBox VMs/packer-ubuntu-16.04-amd64_1470073131288_38049/packer-ubuntu-16.04-amd64-disk1.vmdk'.
VBoxManage: error: VMDK: Compressed image is corrupted '/home/john/.vagrant.d/boxes/geerlingguy-VAGRANTSLASH-ubuntu1604/1.0.4/virtualbox/packer-ubuntu-16.04-amd64-disk1.vmdk' (VERR_ZIP_CORRUPTED)
VBoxManage: error: Details: code VBOX_E_FILE_ERROR (0x80bb0004), component ApplianceWrap, interface IAppliance
VBoxManage: error: Context: "RTEXITCODE handleImportAppliance(HandlerArg*)" at line 877 of file VBoxManageAppliance.cpp

you must hate people like me..

Not at all! You give detailed information, so it's easier to debug :)
Next try:
$ vagrant box remove geerlingguy/ubuntu1604
... wait for it to finish ...
$ vagrant box add geerlingguy/ubuntu1604 --box-version 1.0.3

For some reason your download of 1.0.4 was corrupted... see if 1.0.3 works—if so, maybe I can do some extra checking on the integrity of the file on my server. If it doesn't, maybe there's some strange network issue.

same under the 1.0.3 version...
john@Ubuntu-JDM:~/VirtualBox VMs/geerlingguy-drupal-vm-bfb50bb$ vagrant box remove geerlingguy/ubuntu1604
Removing box 'geerlingguy/ubuntu1604' (v1.0.4) with provider 'virtualbox'...
john@Ubuntu-JDM:~/VirtualBox VMs/geerlingguy-drupal-vm-bfb50bb$ vagrant box add geerlingguy/ubuntu1604 --box-version 1.0.3
==> box: Loading metadata for box 'geerlingguy/ubuntu1604'
box: URL: https://atlas.hashicorp.com/geerlingguy/ubuntu1604
This box can work with multiple providers! The providers that it
can work with are listed below. Please review the list and choose
the provider you will be working with.

virtualbox
vmware_desktop

Enter your choice: 1
==> box: Adding box 'geerlingguy/ubuntu1604' (v1.0.3) for provider: virtualbox
box: Downloading: https://atlas.hashicorp.com/geerlingguy/boxes/ubuntu1604/versions/1.0.3/providers/virtualbox.box
==> box: Successfully added box 'geerlingguy/ubuntu1604' (v1.0.3) for 'virtualbox'!
john@Ubuntu-JDM:~/VirtualBox VMs/geerlingguy-drupal-vm-bfb50bb$ vagrant up
Bringing machine 'drupalvm' up with 'virtualbox' provider...
==> drupalvm: Preparing master VM for linked clones...
drupalvm: This is a one time operation. Once the master VM is prepared,
drupalvm: it will be used as a base for linked clones, making the creation
drupalvm: of new VMs take milliseconds on a modern system.
==> drupalvm: Importing base box 'geerlingguy/ubuntu1604'...
Progress: 50%There was an error while executing VBoxManage, a CLI used by Vagrant
for controlling VirtualBox. The command and stderr is shown below.
Command: ["import", "/home/john/.vagrant.d/boxes/geerlingguy-VAGRANTSLASH-ubuntu1604/1.0.3/virtualbox/box.ovf", "--vsys", "0", "--vmname", "packer-ubuntu-16.04-amd64_1470073685332_74754", "--vsys", "0", "--unit", "7", "--disk", "/home/john/VirtualBox VMs/packer-ubuntu-16.04-amd64_1470073685332_74754/packer-ubuntu-16.04-amd64-disk1.vmdk"]
Stderr: 0%...10%...20%...30%...40%...50%...60%...70%...80%...90%...100%
Interpreting /home/john/.vagrant.d/boxes/geerlingguy-VAGRANTSLASH-ubuntu1604/1.0.3/virtualbox/box.ovf...
OK.
0%...10%...20%...30%...40%...50%...
Progress state: VBOX_E_FILE_ERROR
VBoxManage: error: Appliance import failed
VBoxManage: error: Could not create the imported medium '/home/john/VirtualBox VMs/packer-ubuntu-16.04-amd64_1470073685332_74754/packer-ubuntu-16.04-amd64-disk1.vmdk'.
VBoxManage: error: VMDK: Compressed image is corrupted '/home/john/.vagrant.d/boxes/geerlingguy-VAGRANTSLASH-ubuntu1604/1.0.3/virtualbox/packer-ubuntu-16.04-amd64-disk1.vmdk' (VERR_ZIP_CORRUPTED)
VBoxManage: error: Details: code VBOX_E_FILE_ERROR (0x80bb0004), component ApplianceWrap, interface IAppliance
VBoxManage: error: Context: "RTEXITCODE handleImportAppliance(HandlerArg*)" at line 877 of file VBoxManageAppliance.cpp

All right, I guess one other thing: can you list the version of vagrant --version, and vboxmanage --version?
I'm pretty certain nobody else has reported this issue with those box files, and I tested both just now downloading from files.midwesternmac.com, so I know they're not corrupt on the server itself.

Also, another thing to try (if you don't have any other vagrant boxes on your computer you care about) is deleting the entire .vagrant.d directory: $ rm -rf ~/.vagrant.d
It's the nuclear option, but if there's a funky file in there, that can help.

sometimes the only option that seems to work...script is into 10mins of running...thanks for your help!!

Well, they aren't pretty, but nuclear solutions do sometimes work!


Hide grid after reaching the point?

Is it possible hide the grid after I reach the point? Currently have to press escape to quit the grid mode

The grid will auto-hide after simulating a click, but it sounds like you are using Scoot to move the cursor but not click. (Can you share a bit more details about your use case?)
Also worth noting that this feature request, if I'm understanding it correctly, is effectively the opposite of #30.

Also worth noting that this feature request, if I'm understanding it correctly, is effectively the opposite of https://github.com/mjrusso/scoot/issues/30.

I'm not the original poster of this, but this is my understanding as well.

Can you share a bit more details about your use case?

I have multiple monitors and I'm already using a mouse emulation solution (which is good enough for most cases, but not good for jumping around quickly long distances with precision).
So in my case, I want to use Scoot just to move the cursor quickly where I want it, and hide the grid as soon as possible so I can handle the rest of micro-adjustments with my current solution. In my case, once the cursor has moved to the desired location, the grid becomes more of a visual/functional hindrance which I need to dismiss with the Esc key (which is not the end of the world, but is still an extra keystroke).
So this functionality would be useful to those who seek just quick "mouse teleportation".
Thanks!


Dark Sky API not acception New signups!!

As per their new update "Our API service for existing customers is not changing today, but we will no longer accept new signups. The API will continue to function through the end of 2021." So can I use another API?

@Navin456 - Since Dark Sky is not accepting new signups, we can't really help you with that.
You can try other APIs to get the weather report but you will have to parse the data accordingly so that the consistency is maintained.
cc : @raghavpatnecha

I will figure that out but a general question did you used external microphone to make your voice heard by the system?

@Navin456 - Yes we used an external microphone.

can you provide me a link for that as I have seen some of the microphones are not really supporting my pi Also if you can provide me the speakers link that will be great


Make HTML page title to reflect name of the map and/or name of the selected node

http://stackoverflow.com/questions/34602806/how-to-change-page-title-in-angular2-router
https://angular.io/docs/ts/latest/cookbook/set-document-title.html
http://www.bennadel.com/blog/3050-setting-the-window-document-title-in-angular-2-beta-9.htm

OK. but not important ... in which use-case is useful?

When you want to see the title, distinguish among many, when you are
bookmarking it, etc
Not urgent for now
On Sat, 23 Jul 2016 at 20:08 sinisarudan<EMAIL_ADDRESS>wrote:

OK. but not important ... in which use-case is useful?
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/mprinc/KnAllEdge/issues/206#issuecomment-234752330,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA8yodfBRjb2KXtcCWl_IIzxIQ_visoBks5qYskegaJpZM4JTe5Q
.


Charts: Scatter size varaible

Would like to add two variables to support interactive data points with the ChartScatter component in react-charts.
--pf-chart-scatter--active--size: 5
--pf-chart-scatter--size: 3

For more details, see https://github.com/patternfly/patternfly-react/issues/3079

:tada: This issue has been resolved in version 2.33.8 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Why I have to define scopes in ApiResource and also in Client?

Hi,
with this configuration, I'm able to get scopes openid email profile
new Client
{
    ClientId = "service",
    AllowedGrantTypes = GrantTypes.ClientCredentials,
    ClientSecrets =
    {
        new Secret("s3rv1c3".Sha256())
    },
    AllowedScopes =
    {
        IdentityServerConstants.StandardScopes.OpenId,
        IdentityServerConstants.StandardScopes.Email,
        IdentityServerConstants.StandardScopes.Profile
    }
}

With following configuration, I'm able to get scope api but not api openid email profile
new Client
{
    ClientId = "service",
    AllowedGrantTypes = GrantTypes.ClientCredentials,
    ClientSecrets =
    {
        new Secret("s3rv1c3".Sha256())
    },
    AllowedScopes =
    {
        IdentityServerConstants.StandardScopes.OpenId,
        IdentityServerConstants.StandardScopes.Email,
        IdentityServerConstants.StandardScopes.Profile,
        "api"
    }
}

new ApiResource()
{
    Name = "api",
    DisplayName = "Cherry API",
    Scopes =
    {
        new Scope("api", "Cherry API")
    },
    UserClaims =
    {
        "cherry_sample"
    }
}

To be able to get all 4 scopes api openid email profile I have to duplicate client scope definition to ApiResource; why? How is connected ApiResource with Client?
new ApiResource()
{
    Name = "api",
    DisplayName = "Cherry API",
    Scopes =
    {
        new Scope("api", "Cherry API"),
        new Scope(IdentityServerConstants.StandardScopes.OpenId),
        new Scope(IdentityServerConstants.StandardScopes.Email),
        new Scope(IdentityServerConstants.StandardScopes.Profile),
        new Scope(IdentityServerConstants.StandardScopes.OfflineAccess)
    },
    UserClaims =
    {
        "cherry_sample"
    }
}

OpenId, email and profile are inherently user related scopes - they are coming from the OpenID Connect spec -
in client credentials flow, there is no user. So why are you requesting those scopes??

I realized it few hours back, thank you for reply.


Starting with square images in ImagesForTraining breaks Colab processing

I tried to run the colab version of this and preprocessed my images to be square and cropped around the face I was trying to train on, but this seems to have broken the preprocessing done in the Colab.  I'm sure the problem comes from hitting some of the width=height conditionals, but I don't understand what each step is doing well enough to know how to fix the problem in a sensible way. Instead I batch resized my square input images to be slightly oblong and restarted.  That time all the preprocessing steps went smoothly and I was able to start training.  A quick revision by someone who knows what they are doing would be helpful!!

if ur images are already square u don't need to do the outpainting preprocessing

Are any preprocessing steps needed if they're already square and 512x512?  Are the mask images used for anything other than resizing?

Unsure if this is related but with the squaring of images in the path_to_input_images I'm getting a path is not define error.
for this part "dirs = os.listdir( path )"
NameError: name 'path' is not defined

That one I knew how to fix.  The colab is a bit of a mess and that line just got out of order.  The path variable is defined a few lines later in the code, so if you just cut and paste the line giving you the error below that definition it works fine.

Awesome, thanks. Python isn't my forte, I did think that was the issue and tried but failed to fix it.
I'll try this again tonight.


XIVY-12289 add field dialog

And also something which I started for the form editor and process editor, I moved the playwright tests out of the integration to it's own package. I think this is much cleaner... But more meant as a hint for further development :)
I know not all repos currently have this structure.


Chore/support elastic apm env config

I'm adding support for the 7.5 release of the Agent (https://github.com/nipwaayoni/elastic-apm-php-agent/releases/tag/v7.5.0). I don't expect breaking changes in the Laravel integration, but there will be what I hope are positive differences.
One noticeable change is how the Agent can now be configured. Almost all configuration options can now be set by ELASTIC_APM_* environment variables. I wanted to do this after implementing the Transaction Sample Rate feature and realizing I had to do work on this package to do nothing more than pass that setting into the agent. All the other APM agents support environment variables for configuration and it makes sense to so here as well. That way your users can directly configure the agent features without you need to do extra work.
The previous mechanism of passing options into the Config constructor is still supported, and takes precedence over env values, so there should be no required change for users of this package. However, the package itself needs to be changed to accommodate anyone wanting to move to env variables.
All default values have been removed from the config file so the Agent package can correctly determine when intentional Config arguments are given. The defaults in the Agent package should be consistent with the previous defaults in this config, so your users should not see a change. The ServiceProvider is also updated to filter out empty values before passing to the Config constructor.
Related to using environment variables, we can no longer set a default application name in the config because that would always override the environment variable if the user tried to set it that way. There is a new "defaultServiceName" configuration option which will be used if the value is not otherwise provided.
You can now pass a PSR-3 compliant logger object into the configuration (this obviously can't be done through an environment variable). This allows better insight into the internals of the agent objects. Log messages are still evolving in those classes, but it's a good start. You can set the log level for that package independently of your Laravel app log level to reduce the noise if you are not trying to debug the agent.
I believe I have addressed the issues which led you to create the LazySpan class. I have removed that class from this package and switched to using the base Span. The implementation of Transaction Sample Rate caused some changes in the classes and it is much better if you can stick with the base classes if possible.
I added methods to track the current transaction in the Agent. I needed this to properly support distributed tracing (which I am really excited about). When making a new request I want to trace within the Laravel app, I need the trace ID from the current transaction to include in the header.
I have not started updating documentation in this project yet. I wanted to get the code and feature changes to you for discussion first. I'll start on docs next and leave this is a draft PR while you review the code.

As I'm working through the docs, I realize that there is one change in the default configuration. The APM_BACKTRACEDEPTH (now ELASTIC_APM_STACK_TRACE_LIMIT) was defaulted to 25, but the Agent package defaults to 0 (unlimited).

I'm adding support for the 7.5 release of the Agent (https://github.com/nipwaayoni/elastic-apm-php-agent/releases/tag/v7.5.0). I don't expect breaking changes in the Laravel integration, but there will be what I hope are positive differences.

Great job once again, thanks for the support.

One noticeable change is how the Agent can now be configured. Almost all configuration options can now be set by ELASTIC_APM_* environment variables. I wanted to do this after implementing the Transaction Sample Rate feature and realizing I had to do work on this package to do nothing more than pass that setting into the agent. All the other APM agents support environment variables for configuration and it makes sense to so here as well. That way your users can directly configure the agent features without you need to do extra work.

I saw that you added the configuration values to the documentation. I think it's good that users can configure the underlying package directly, they can immediately benefit from your features without us needing to implement a middle layer.

The previous mechanism of passing options into the Config constructor is still supported, and takes precedence over env values, so there should be no required change for users of this package. However, the package itself needs to be changed to accommodate anyone wanting to move to env variables.
All default values have been removed from the config file so the Agent package can correctly determine when intentional Config arguments are given. The defaults in the Agent package should be consistent with the previous defaults in this config, so your users should not see a change. The ServiceProvider is also updated to filter out empty values before passing to the Config constructor.
Related to using environment variables, we can no longer set a default application name in the config because that would always override the environment variable if the user tried to set it that way. There is a new "defaultServiceName" configuration option which will be used if the value is not otherwise provided.
You can now pass a PSR-3 compliant logger object into the configuration (this obviously can't be done through an environment variable). This allows better insight into the internals of the agent objects. Log messages are still evolving in those classes, but it's a good start. You can set the log level for that package independently of your Laravel app log level to reduce the noise if you are not trying to debug the agent.
I believe I have addressed the issues which led you to create the LazySpan class. I have removed that class from this package and switched to using the base Span. The implementation of Transaction Sample Rate caused some changes in the classes and it is much better if you can stick with the base classes if possible.

Good, I will give it a try. I see the timing is handled in a different way now, I will make sure spans are shown as expected.

I added methods to track the current transaction in the Agent. I needed this to properly support distributed tracing (which I am really excited about). When making a new request I want to trace within the Laravel app, I need the trace ID from the current transaction to include in the header.

This is really exiting indeed!

I have not started updating documentation in this project yet. I wanted to get the code and feature changes to you for discussion first. I'll start on docs next and leave this is a draft PR while you review the code.

As I'm working through the docs, I realize that there is one change in the default configuration. The APM_BACKTRACEDEPTH (now ELASTIC_APM_STACK_TRACE_LIMIT) was defaulted to 25, but the Agent package defaults to 0 (unlimited).

I will add it to the breaking changes when we make the final release, thanks for letting me know.

@arkaitzgarro This should address the outstanding issues. I want to spend some time writing tests. I've not been good about using TDD and need to learn more about codeception.
I also updated the stack trace limit default to 50 to be consistent with what it looks like other agents are doing:
https://docs.google.com/spreadsheets/d/1JJjZotapacA3FkHc2sv_0wiChILi3uKnkwLTjtBmxwU/edit#gid=0

@arkaitzgarro I have couple of questions not directly related to this PR, but not quite worth opening a new issue.

What is your expectation for resolving conversations on a PR? I prefer that all conversations be resolved for merging, but I also expect that the person who opened the conversation should be the one to resolve it. That is an indication that the original concern has been addressed. (With the exception of a contributor who is not responding in a timely way and delaying a release.)

I noticed that the config/ApmConfigTest.php is not being run when following your documented development steps for unit tests. Is that intentional and if so why? Or should that test class be moved into the unit directory and made part of the standard unit suite?

Do you always squash all commits on merge to your main branch? While I appreciate the clean graph that produces, it does make basing new work on unmerged branches very difficult and I suspect leads to problems like what you experienced here with previous changes being lost https://github.com/arkaitzgarro/elastic-apm-laravel/pull/82 if feature branches are not carefully rebased frequently. I suspect that practice works better in projects with small, short-lived feature branches, not the large longer-lived ones I've been working on with you.

I'm happy to adapt to your preferences for this project. I just want to make sure I understand.

@arkaitzgarro This should address the outstanding issues. I want to spend some time writing tests. I've not been good about using TDD and need to learn more about codeception.
Great, thank you very much. If you think you are done implementing the necessary changes to support version 7.5 of the agent, I can already give it a try and test it in one of our applications.

I also updated the stack trace limit default to 50 to be consistent with what it looks like other agents are doing:
https://docs.google.com/spreadsheets/d/1JJjZotapacA3FkHc2sv_0wiChILi3uKnkwLTjtBmxwU/edit#gid=0

Oh, nice document! Do you know if it's officially maintain by Elastic?

@arkaitzgarro This should address the outstanding issues. I want to spend some time writing tests. I've not been good about using TDD and need to learn more about codeception.

Great, thank you very much. If you think you are done implementing the necessary changes to support version 7.5 of the agent, I can already give it a try and test it in one of our applications.

I also updated the stack trace limit default to 50 to be consistent with what it looks like other agents are doing:
https://docs.google.com/spreadsheets/d/1JJjZotapacA3FkHc2sv_0wiChILi3uKnkwLTjtBmxwU/edit#gid=0

Oh, nice document! Do you know if it's officially maintain by Elastic?

I'm working on some unit tests for the Agent class now. That seems the highest priority for testing since we've added a number of methods. I hope to have that done later today, so hopefully I'll be ready to remove the draft status within a day.
That agent comparison document is linked from the official Elastic agent configuration guide:
https://github.com/elastic/apm/blob/master/specs/agents/configuration.md
The guide (https://github.com/elastic/apm/tree/master/specs/agents) provides a lot of good information and I'm slowing trying to bring the Agent into better alignment. I'm being a little selective since the official PHP agent (https://github.com/elastic/apm-agent-php) is under development now. I don't want to waste my time, but I see value in this package for a while longer.

@arkaitzgarro I have couple of questions not directly related to this PR, but not quite worth opening a new issue.

Happy to answer them :)

What is your expectation for resolving conversations on a PR? I prefer that all conversations be resolved for merging, but I also expect that the person who opened the conversation should be the one to resolve it. That is an indication that the original concern has been addressed. (With the exception of a contributor who is not responding in a timely way and delaying a release.)

Yes, this is also my expectation. If anyone raises a question/concern, I expect that person to be the one resolving it if the original concern has been addressed, or guiding the contributor towards the expected solution. If that wasn't the case in this project please let me know. Also reminders about unanswered/unresolved questions are always welcome.

I noticed that the config/ApmConfigTest.php is not being run when following your documented development steps for unit tests. Is that intentional and if so why? Or should that test class be moved into the unit directory and made part of the standard unit suite?

Oops, this is clearly a mistake. I think the idea was to have the config folder outside the unit folder to mimic the structure of the source code. Running the command described in the docs will only run unit tests indeed. You could duplicate unit.suite.yml file into config.suite.yml and run ./vendor/bin/codecept run to execute both suites.

Do you always squash all commits on merge to your main branch? While I appreciate the clean graph that produces, it does make basing new work on unmerged branches very difficult and I suspect leads to problems like what you experienced here with previous changes being lost #82 if feature branches are not carefully rebased frequently. I suspect that practice works better in projects with small, short-lived feature branches, not the large longer-lived ones I've been working on with you.

Yes, squash and merge is my preference to merge commits into master, because:

Combines all the code changes related to a single logical change (it could be a feature, a bugfix, or an individual change part of a bigger change).
If you pick this commit independently from the history, it makes sense on its own. Individual commits are still available in the original PR.
Reverting changes is as easy as reverting a single commit. I had terrible situations in the past where GH UI was not available and we needed to revert a feature from the cli. Commit by commit. It was hell and I don't even want to think about it anymore 😅

The issue related to #82 I'm pretty sure is about a wrong git push using --force on my side. I cannot see the merge commit in the branch, looks like I pushed changes using the force without pulling latest changes.

I'm working on some unit tests for the Agent class now. That seems the highest priority for testing since we've added a number of methods. I hope to have that done later today, so hopefully I'll be ready to remove the draft status within a day.

Thanks, I appreciate that. The code coverage is pretty low right now on master, so once this PR is merged, I will also make sure our tests are in a better shape before we create the final release.

@arkaitzgarro This should be ready for a review but I see there will be conflicts with at least two of the other PRs in progress. Why don't you wait to merge this one until both of those are merged, then I'll rebase this one and clean it up.

... except that I now see there are conflicts. I'll get to work rebasing and resolving those.

@arkaitzgarro The rebase is complete and I cleaned up the commits. I believe I resolved all of the conflicts correctly, but double check that your recent merge changes are reflected as you expect.

@arkaitzgarro The rebase is complete and I cleaned up the commits. I believe I resolved all of the conflicts correctly, but double check that your recent merge changes are reflected as you expect.

This is great, thank you so much. I have one final question though: why are handling the current transaction in this package, and not in the underlying agent? As far I can see, we are just setting and clearing the value, I think this can be done in the PHP agent.
I'm not saying we should remove it now, but maybe it should be moved to the agent in the next release?

I intend the next major release of the Agent package to assume all responsibility for creating events for reasons like this. It doesn't make sense for every consumer of the Agent to need to collect events, manage parent relationships and more. The agent should be able take on most of that accepting alternate implementations and hooks where appropriate. That's going to a substantial change, and honestly, I'll have to consider how quickly the official agent package is progressing before I decide it's worth doing.


2.3.0 Release Notes and Breaking Changes

This issue tracks the new features and breaking changes that are coming in 2.3.0. These will eventually make their way to the github.io docs during the release process.
Please use this issue and add your release notes and breaking changes (and associated upgrade instructions) here.

Pinch slider orientation #6858
PinchSlider's visuals such as TrackVisuals and TickMarks, rotate along the SliderAxis every time user updates the value of the SliderAxis

#6919
StartupBehaviour now drives start behaviour for both initialize and on re-enable() in spatial awareness system

#6863
ShellHandRayPointer.lineRendererSelected and ShellHandRayPointer.lineRendererNoTarget have been deprecated in favor of ShellHandRayPointer.lineMaterialSelected and ShellHandRayPointer.lineMaterialNoTarget respectively

https://github.com/microsoft/MixedRealityToolkit-Unity/pull/6989
Namespace of WindowsMixedRealityUtilities changed from Microsoft.MixedReality.Toolkit.WindowsMixedReality.Input to Microsoft.MixedReality.Toolkit.WindowsMixedReality.

#6891 - confirm the changes currently documented are accurate

#7012 - ControllerPoseSynchronizer.handedness (private serialized field) deprecated
#7012 - ControllerPoseSynchronizer.Handedness setter removed and fixed

#7016 and #6870 ensure that poke and grab pointers are not actually enabled when they are interacting with objects that are not visible to the camera. This can help prevent accidental grabs and touch interactions. If you wish for pointers to grab / touch colliders that are not visible, deselect the "ignore colliders not in FOV" checkbox in the GrabPointer and PokePointer prefabs (after cloning them to make your own :-) )

I believe we also want to call out a few new experimental features:

(Experimental) Non-native keyboard  #6492

(Experimental) Hand coach #6493

(Experimental) Follow solver (solver that matches HoloLens 2 shell behavior). #6981

(Experimental) ObjectManipulator #6924 (@thalbern can you add image please?)

(Experimental) BoundsControl #6924 (@thalbern  can you add image please?)


Do not call onClosed if component is not mounted

If the closing of the component changes state in such a way that the checkout component is not shown, you will get the error below.
Warning: setState(...): Can only update a mounted or mounting component.
This usually means you called setState() on an unmounted component.
This is a no-op. Please check the code for the ReactStripeCheckout component.

This is because the checkout.js calls the onClosed
	  if (this..isMounted)
		this.setState({ open: false });

and set isMounted = false in componentWillUnmount solves it.
@azmenak Is there any chance you can look at this and also the other open pull requests.  What you have done here is effect, just needs a few tweaks?

It would be great if this bug could be fixed.


Broken Pip Panic

What can be the reason of this error, i just typed
pkl-gen-go
in my terminal

@romanchechyotkin can u share the entire output?


[#2170] Add ammo, focus, and container types

It exceeds the scope of this ticket, but adding the focus type to a class' spellcasting will be handy to link these two together (for example, for being able to search and select an appropriate focus for a spellcating class`


need help: Is there an API to get my gmail contacts?

I want to export my contacts to the ldap service. I found that gmail doesn't seem to have this api. thanks

find it in people


Upgrade rust-installer

@brson: no appropriate reviewer found, use r? to override

r? @alexcrichton


add persian language support

translate messages for persian language

i think the flake8 have a issue, because i did't changed python code.


[Arc] Missing history from some spaces / profiles when using "search history" command

Extension
https://www.raycast.com/the-browser-company/arc
Description
When using search history command, it looks like the extension doesn't consider the whole history in the case of a user with multiple profiles and multiple spaces. It only shows history from one profile or one set of space hard to converse.
Steps To Reproduce

Set up multiple profiles on one arc browser.
Set up multiple spaces for each profile.
Visit a set of pages on each space
Check if anything is missing in the history via Raycast commands

Current Behaviour
Search history only retrieves pages visited from one profile or one set of spaces.
Expected Behaviour
Search history command should retrieve pages visited across all profiles and all spaces.

I am seeing this issue as well.


Placing multiple widgets in specific HTML elements on the same page

Hi there! Thanks for creating exolve -- it looks like a very useful tool!
I have a long explanatory page which I want to add multiple small crosswords to, with various text in between. From reading the docs, I see that:

By default, the puzzle content gets created as a new last element in the HTML Dom. But you can direct it to be placed at different spot by creating an empty element (typically a DIV) with id="exolve" anywhere in the HTML file. The puzzle content will then be added inside this element.

Since the id is static, does this mean that I can only add one crossword widget into a specific HTML element per page? Is it possible to make it so that I can specify the HTML element to add each puzzle to?

Nevermind I think I found it -- it looks like containerId in createExolve(). I guess in that case maybe the docs for "Placing the puzzle in a specific HTML element" should be updated?

Nevermind I think I found it -- it looks like containerId in createExolve(). I guess in that case maybe the docs for "Placing the puzzle in a specific HTML element" should be updated?

Indeed, you can pass a containerId in createExolve() and set up multiple puzzles within a single page, as you found.
I'ave updated the "Placing the puzzle..." documentation now--thanks for catching that!

Indeed, you can pass a containerId in createExolve() and set up multiple puzzles within a single page, as you found.
I'ave updated the "Placing the puzzle..." documentation now--thanks for catching that!


Implement smart usage

Usage strings, if possible should be re-generated upon error for the current usage case. For example if you have a non-required by default argument which requires another argument, and the user fails to supply the third argument (which is required by proxy), the usage string is generic - yet the error message says a required argument is missing...which argument?
let m = App::new("app")
                .arg(Arg::from_usage("--reqs").requires("input"))
                .arg_from_usage("[input] 'some input'")
                .get_matches();

Run with app --reqs and the usage does not identify [input] as a requirement, yet states a generic "required argument missing"
Now this does happen in certain instances, but I want to implement it in all instances.

Closed with #90

Usage strings are now recalculated on errors, so that the current usage attempt is a template and all requirements/conflicts are spelled out in the usage string.


Add "view options" settings for rendering engines

Avoids cluttering app.locals, and is consistent
with the assignment of the "view engine" settings:
app.set("view options", {
    basedir: "somedir"
});
app.set("view engine", "pug");

The intent is also stated more clearly in comparison to:
app.locals.basedir = "somedir"

Seems reasonable. Would this only apply to the set default view engine, or to all view engines (and if so, how would conflicting options be resolved)? Please add some tests when you find the time 👍

Yeah, it applies to all view engines, although I've only looked and modified the app.render code, so I may not be entirely sure. It's basically an alias for the app.locals. I'll do a double check.


nrf: add support for UICR configuration.

APPROTECT enable/disable. Notably this fixes issues with nrf52-rev3 and nrf53 from locking itself at reset.
Use NFC pins as GPIO.
Use RESET pin as GPIO.

NFC and RESET pins singletons are made available only when usable as GPIO, for compile-time checking.
TODO: test

[x] nrf52 rev1
[x] nrf52 rev3
[ ] nrf53
[ ] nrf91

Is the choice of naming correct, or is there a problem with the design concept here (or implementation)?
My understanding is that the default (at least for nRF52840, but I assume it's the same for others) is for the NFC pins to be in NFC mode (not GPIO), but for the RESET pin to be in GPIO (not act as a reset).  These correspond to the default (flash erased) "1" bits.
All that can be done without erasing the entire chip is to change those from 1 to 0, meaning that while "nfc-pins-as-gpio" makes sense as a non-optional feature, "reset-pin-as-gpio" would not because it could not actually perform the required change.  Instead I suspect it should be "reset-pin-as-reset", or some such.

@peter9477 yes, the default from the hardware's PoV is different for the two flags. That doesn't mean the Cargo feature default has to be the same though.
I think enabling the reset pin by default is better:

I've seen a few people ask why isn't reset working in matrix already.
If you disable it when you want it enabled, troubleshooting is hard: reset doesn't work with no indication why. If you enable it when you want it disabled, troubleshooting is easy: your firmware won't compile because the pin singleton is unavailable.

I agree it's a footgun that the setting won't apply if UICR is already programmed wrong though! I've added code to check for that, and warn if it's the case. Hopefully that mitigates it!
Tested working on all chips.
bors r+


Redesigned Contact us Page

Fixes Issue
Closes #536
Changes proposed
Redesigned Contact us page

Check List (Check all the applicable boxes)

[x] My code follows the code style of this project.
[x] My change requires changes to the documentation.
[x] I have updated the documentation accordingly.
[x] All new and existing tests passed.

this is how ur changes are looking!
make them correct first!

this is how ur changes are looking!
make them correct first!

Isn't it good

hii, could you change your contact page a bit like this..don't make the contact container too wide! Thanks for ur patience!

Sure. Please give me some time I will update and let you know.

merge conflicts

Hey @prjailani is it done? Any update?

 I had some homeworks previously.


”Dumești”, ”Dumeștii Vechi”, ”Schinetea”, ”Valea Mare” are listed in the drop-down menu

Description:
When user writes ”Dumești”, ”Dumeștii Vechi”, ”Schinetea”, ”Valea Mare” the locations appear in the drop-down menu next to the comune and the county which is a part according to the law.
Precondition:
The website is up an running.

Step 1
Write in the search bar "Dumești"

Expected results
The location appears in the drop-down menu as "Sat Dumești, VASLUI (Dumești)".

Step 2
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 3
Write in the search bar "Dumeștii Vechi"

Expected results
The location appears in the drop-down menu as "Sat Dumeștii Vechi, VASLUI (Dumești)".

Step 4
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 4
Write in the search bar "Schinetea"

Expected results
The location appears in the drop-down menu as "Sat Schinetea, VASLUI (Dumești)".

Step 6
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 7
Write in the search bar "Valea Mare"

Expected results
The location appears in the drop-down menu as "Sat Valea Mare, VASLUI (Dumești)".


java.lang.NullPointerException at org.godotengine.godot.GodotView.onInputDeviceAdded

Godot version:
Godot 2.1.4 stable.
(Standard build, Android template)
OS/device including version:
Android:
Motorola Moto Z (2) Play (albus), 3072MB RAM, Android 7.1
Samsung Galaxy S8+ (dream2lte), 4096MB RAM, Android 7.0
LGE LG G5 (h1), 4096MB RAM, Android 7.0
Samsung Galaxy Tab S3 (gts3lwifi), 4096MB RAM, Android 7.0
Samsung Galaxy Tab S2 9.7 (gts210lte), 3072MB RAM, Android 7.0
Samsung Galaxy Tab A 10.1 (gtaxlwifi), 2048MB RAM, Android 7.0
Bullitt Group Cat S60 (CatS60), 3072MB RAM, Android 6.0
Issue description:
I regularly get this error message for my android app on my Google Play Console. Personally, I can not reproduce it and the error rate is not alarming yet (below 1%):
java.lang.NullPointerException:
at org.godotengine.godot.GodotView.onInputDeviceAdded (GodotView.java:213)
at org.godotengine.godot.GodotView.find_joy_device (GodotView.java:204)
at org.godotengine.godot.GodotView.onInputDeviceRemoved (GodotView.java:231)
at org.godotengine.godot.input.InputManagerV16$V16InputDeviceListener.onInputDeviceRemoved InputManagerV16.java:70)
at android.hardware.input.InputManager$InputDeviceListenerDelegate.handleMessage (InputManager.java:1459)
at android.os.Handler.dispatchMessage (Handler.java:102)
at android.os.Looper.loop (Looper.java:154)
at android.app.ActivityThread.main (ActivityThread.java:6823)
at java.lang.reflect.Method.invoke (Native Method)
at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run (ZygoteInit.java:1563)
at com.android.internal.os.ZygoteInit.main (ZygoteInit.java:1451)
Steps to reproduce:
Difficult I can't reproduce it, yet. I tested the app, also with joystick and didn't get the error. Maybe timing related?
Minimal reproduction project:
Sorry as I wasn't able to reproduce it personally, I couldn't isolate it, yet.
Here's the link to my app:
https://play.google.com/store/apps/details?id=org.godotengine.narrowboatsimulator
Probable cause of error:
The error occurs on device.getName():
InputDevice device = mInputManager.getInputDevice(deviceId);
joy.name = device.getName();

Obviously, mInputManager.getInputDevice() doesn't find a device for the passed "deviceId" and returns null.
IMHO it would make the code more robust to also handle the possibility of a "null" return.
(as mentioned in the docs: https://developer.android.com/reference/android/hardware/input/InputManager.html#getInputDevice(int) )

Fixed by #17999. (Still needs someone to backport those 2.1 files to master...)


Line chart with stack mode and max value - overflow

Expected behavior
values over max value should not be shown
Current behavior
When i have two axises with "stack" mode and one of them has min and max values, the chart is overflows the max value.

Reproducible sample
https://codesandbox.io/s/react-chartjs-2-chart-js-issue-template-cg7b5?file=/src/App.tsx
Optional extra steps/info to reproduce
No response
Possible solution
No response
Context
No response
chart.js version
^3.8.0
Browser name and version
No response
Link to your project
No response

I think the issue is here:
https://github.com/chartjs/Chart.js/blob/3df687a87cf7288463cfcb5d4ceecf36d5945abb/src/core/core.controller.js#L783-L815
where there is a clip of the area to draw the dataset but it's using the chartArea instead of the "dataset area" which should be built by the index and value scales of the dataset.
I'm going, as soon as I can, to submit a PR for this.


Add pressure traces for Sciex WIFF1 files

Changes WIFF TIC chromatogram from summing all samples to only summing the experiments from the current sample
Add new test data for core and Skyline, and moves ABI test data mzMLs out of tarball
Fix bad collision energy values for Enolase_repeats data

Nothing jumps out at me - related to the work to be able to pull TIC without filtering in Skyline?

No, this is unrelated to TIC. It's for AutoQC.

@brendanx67 @nickshulman OK with GetQcTraces() and its semantics?

@nickshulman Brendan wanted me to hand this off to you to add the QC traces to the .skyd.


Feature/unit test coverage to trigger Coveralls

Coverage increased (+13.3%) to 71.282% when pulling daff40e197aff3f039f69c5d52cefa5690c63d15 on feature/unit-test-coverage into 6c897727114e595bdbb48169086c9cbce0fb8dd1 on develop.


Sparse Design

I've started converging on a design 07a4a7d for my code.
The structure is inspired by the micrograd one but with some tweaks:

Compared to normal neural networks it makes more sense for me to parse output to inputs, since I don’t care about nodes that are not leading to outputs.
I compute neurons only once and then cache their values
DFS should help with cache locality?

After implementing various different versions (loop and no-loop) I've converged onto the version in sparse_forward that is both easy to read and easy to check.
Unfortunately it is only 30-40% faster than the python/numpy version.
Which is quite disappointing.
How to make faster?


[bug] production build issue Uncaught Error: No NgModule metadata found for 't'.

[x] bug report => check the README and search github for a similar issue or PR before submitting
[ ] support request => check the README and search github for a similar issue or PR before submitting
[ ] feature request

Current behavior
It seems to be that recently an npm dependency has been updated which causes silent build issues for production builds, both build:universal-prod and build:spa-prod . Previously everything has been working fine as for production builds but sometime in the last week these issues have arose.
Webpack itself gives no errors during the build but no css file is generated in the public folder.
The app will run but the css styles for the main app bundle do not show (lazy loaded modules still seem to display css for their components).
An error gets generated in the console
Uncaught Error: No NgModule metadata found for 't'.
    at t.resolve (app.3be5665….bundle.js:1)
    at t.getNgModuleMetadata (app.3be5665….bundle.js:1)
    at t._loadModules (app.3be5665….bundle.js:1)
    at t._compileModuleAndComponents (app.3be5665….bundle.js:1)
    at t.compileModuleAsync (app.3be5665….bundle.js:1)
    at e._bootstrapModuleWithZone (app.3be5665….bundle.js:1)
    at e.bootstrapModule (app.3be5665….bundle.js:1)
    at r (app.3be5665….bundle.js:1)
    at r (app.3be5665….bundle.js:1)
    at Object.163 (app.3be5665….bundle.js:1)

I attempted to trace this through but no luck as of yet. I will take more of a look at the production webkit builds over the next day or two and see if I can isolate the issue. In the mean time it would be great if anyone else could reproduce.
Expected/desired behavior
Prod Build should compile the production bundles including CSS with no errors.
Minimal reproduction of the problem with instructions
I have tried with existing projects using this repository as well as cloning a new repo from this one.
Clone this repository locally
Run
npm install
npm run build:universal-prod
npm run serve:universal

If you have an existing code base using this repository
rm -rf node_modules
npm cache clean
npm install
npm run build:universal-prod
npm run serve:universal

@ChrisDalley after upgrading to Node.js v8 and TypeScript v2.4.1 I'm also getting errors, however they're connected with rxjs.
The reason that the production builds resulting with failure is because the no-errors webpack plugin (prevents the build to be complete in case of any errors).
I'm currently working on a breaking change for the webpack configuration, and I think we'll find a solution to these issues. Meanwhile ping me when you face something useful.

@ChrisDalley now I reproduced this error, I think things will become easier now 👍

@ChrisDalley I'll also need the exact webpack version that you're using.

@fulls1z3 Here are the versions
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>

I found it! It's all about @ngtools/webpack. Please let me know which version of @ngtools/webpack is at your system (better have a check looking at the package.json in the node_modules folder). Meanwhile, I'll try to find out the maximum of @ngtools/webpack version right over here.

@ChrisDalley I want to thank you specially for pointing out such an issue, the information you provided greatly helped the solution 👍

@fulls1z3 - Thanks for taking a look so fast! Its all about what package though sorry? It didnt show in your last message!

@ChrisDalley I edited the post, @ngtools/webpack appears now :smile:

Oh great! Im currently on version
<EMAIL_ADDRESS>

Try to switch to v1.3.3 now if things are urgent. Further versions do not work. I'm not surprised, @angular/cli destroys everything. Also, I'll update the repo within a few hours if things are not urgent for you.
package.json
"@ngtools/webpack": "1.3.3",

@fulls1z3 thats done the job - cheers for the help! Great work with the repo btw 😄

@ChrisDalley thanks too, and it would be nice if you can make a favor: Can you please post this issue (meanwhile I'm working for a permanent solution over here), with same details at the @angular/cli repository - mentioning @ngtools/webpack?
It would definitely help the Angular team to reconsider their strategy and solve this issue on the upcoming releases of @ngtools/webpack. you can refer to this issue and my name too, no problem.
Cheers 👍

@ChrisDalley thanks mate, you saved the day and you'll save the future too 👍

@ChrisDalley this has been solved using the latest @ngtools/webpack.

@fulls1z3 - Thanks for the update :)


Issue 25892 test telemetry is correctly sent

Resolve #25892

FYI:
Each XxxBuilderFactory has a corresponding test except AzureCredentialBuilderFactory.

Hi, @saragluna .
Please help to review this PR when you have time.


Use of Rails minutes with "at:15.minutes" creates invalid cron entry

This may be similar to issue #530 in that it is an interaction between Rails idea of time and whenever's idea of time. To run a task every two hours at 2:15, 4:15... it seemed reasonable to say every(2.hours, at: 15.minutes) but cron wouldn't allow that and the only error message said to try running whenever on the command line - which happily said 900 0,2,4,6,8,10,12,14,16,18,20,22 * * *. Not knowing cron very well, I didn't realize that the 900 should have been in the range of 0 to 59.
I finally switched to every(2.hours, at: 15) to get the valid 15 0,2,4,6,8,10,12,14,16,18,20,22 * * * cron entry, but the mix of Rails 2.hours with the non-Rails 15 is unintuitive compared to the readability of 15.minutes.
I can't think of a good solution to this, other than documenting it and maybe throwing an exception if the number of minutes/hours/days becomes nonsensical. In the mean time, this issue might inform others who run into a similar problem.

I've submitted a fix for this: https://github.com/javan/whenever/pull/686


DeleteAll button should delete from the storage of the selected view

I have selected a view that has a different storage, see picture:

When I press the Delete All button however, the reports from the shown view are not deleted as is the expected behavior.
I need a fix for this issue for the Cypress tests of ladybug-frontend. When this works properly, I can clear the reports from the database storage in the before each hook.

Oh wacht, dit issue is niet urgent. Ik kan mijn before each hook ook maken door rechtstreeks een HTTP request naar de backend te doen. Dat is eigenlijk mooier.
Dit issue moet denk ik een keer gefixed worden, maar ik kan zonder fix verder met de Cypress testen.

I think issue https://github.com/wearefrank/ladybug/issues/341 has te be fixed before this issue can be fixed.


Refactored find.js, touch.js, element.js

As per the suggestions made in appium-uiautoamtor2-driver PR, refactored findElOrEls, setElementValue and performMultiAction implementations to avoid code duplication.

Refactored find.js to invoke corresponding driver implementation of doFindElementOrEls()

appium-android-driver implementation refer

appium-uiautomator2-driver implementation  refer

Refactored element.js to invoke corresponding implementation of doSetElementValue()

appium-android-driver implementation refer

appium-uiautomator2-driver implementation refer

Refactored touch.js to invoke corresponding implementation of doPerformMultiAction()

appium-android-driver implementation refer

appium-uiautomator2-driver implementation refer

@jlipps, @imurchie, can you please review this PR.
Note: Few more functions yet to be refactored, though created this PR to make sure that we are good to continue this approach down the line.

So the idea is that other drivers could import these methods and then override doSetElementValue, etc., to get their own functionality? That seems good. We should be careful to extract out as little as possible, so that subclasses don't need to do too much. If error handling can be generalized, that would be awesome!

Yeah, other drivers just need to override doSetElementValue or any other specific function as per their corresponding driver behavior .

Looks good! Can you squash into a single commit, and we can get this in?

Thanks for this!

Published as<EMAIL_ADDRESS>


Configure networking via env vars

We have a few deployment cases for slinky; at the moment we can configure these using an environment. In #43 I added a --prod flag to the cli which brings us further away from the pattern using environmental variables for this.
Instead, we should be able to configure the hosts in the kubernetes deployment file and the docker-compose definition.
This should involve

[x] Add environmental variables/config map to the pods using d1lod
[x] Modify d1lod to use networking variables from the environment, with sensible backups (probably local store settings)

This was done by @ThomasThelen and merged to develop in https://github.com/DataONEorg/slinky/pull/54.


Mouse scroll wheel sensitivity fix.

Is your feature request related to a problem? Please describe.
Ghidra sensitivity to mouse wheel scroll is way higher than every other app.
It may seem a little issue when user is equipped with standard mouse wheel but using Ghidra on touchpad (e.g. in class on a MacBook) may be a little distracting.
It's not like it's a bit faster, you really have to be careful with hand gestures not to miss the next jmp branch.
Describe the solution you'd like
Just add scroll sensitivity multiplier or if it's easier to implement, try to match the system feeling.
Additional context
The issue is existent in macOS since I remember. It's not a bug but implementation. I haven't checked if Windows is the same.

Seems it was already reported, issue #6952

I have a MacBook Pro M1 and I don't experience any sensitivity issues in the Listing.  I did have the problem mentioned in #6952 with the Flat themes, but that has been fixed.
So you problem is in the Listing?  I fear we might not be able to reproduce that issue.

Well, I'm using magic mouse (I know, who uses that...) My scroll sensitivity is at almost lowest setting. I hope this video illustrates it a bit better:
The safari window scroll behaves just like in any other app, while in Ghidra the same scroll movement scrolls listing view by at least two viewport heights.
This is the latest Ghidra right now, just built it from source.
https://github.com/user-attachments/assets/16851f7c-e2c7-4031-8f54-8c45b3edbb4a

Hmm, i'm also using a magic mouse and don't have that issue.

Oh, okay so I should troubleshoot my setup then. I'll get back and post when I find the reason for it.

The only weirdness i've experienced with my magic mouse in Ghidra is sometimes the double click only responds if I do it at super lightning speed.  But then it fixes it self somehow.

May I ask which JDK You're using ? I guess the recommended one ?

JDK 21 from https://adoptium.net/

Well sadly I tried various JDKs and no luck. I tried to compile it again with adjusting getVerticalScrollBar().setUnitIncrement(1) function in Listing java file but that didn't change anything.
Also my suspicion is it may be way less prominent when using slower scroll speeds in magic mouse settings (mine is at max).


Automated pull request created by Toolkit share code

Description of the pull request

Testing for Share Code Functionality on Raaghu Toolkit


Verify and proxy options not working together

I am facing issue using 'verify' setting to CA file and setting up proxy with http & https options.
$result = $client->request ( 'POST', /, [
'verify' => 'C:\xampp\cacart.pem',
'proxy' => [
'http'  => '<IP_ADDRESS>:8080', // Use this proxy with "http"
'https' => '<IP_ADDRESS>:8080' // Use this proxy with "https"
]);
Getting error as "cURL error 60: SSL certificate problem: self signed certificate in certificate chain (see http://curl.haxx.se/libcurl/c/libcurl-errors.html)"
Please let me know, any possible solution to this.

Are you sure your proxy supports TLS handshake tunneling? Not all proxies support that.
First hit on google: https://security.stackexchange.com/questions/115762/how-is-it-possible-to-do-tls-through-proxy-without-anyone-noticing

@sagikazarmark, I am not sure if it supports TLS but If I set 'Verify' to 'False' then it works with HTTPS
otherwise it works with HTTP only.

Just a hunch, but shouldn't C:\xampp\cacart.pem be C:\xampp\cacert.pem ? (Not sure which one is present on your system)
The error says that the certificate seems to be a self-signed one meaning it cannot be verified with any of the CA certs .... maybe because Guzzle doesn't see any?

@sagikazarmark  Sorry, that was typo its cacert.pem file, I downloaded the latest one from the site. and using it to specify ssl verification for guzzle. I am using burp suite for proxy and testing it locally.
When I set 'verify' to 'false' it works with proxy - http and https as well. (Setting 'verify' to 'false' is not preferable).
It would be helpful if I could get it working with cert file and proxy with http and https


Shrinks possums and raccoon sprites

About the PR
I heard possum and raccoon sprites were too big and being dynamically downsized which sounded gross so I just made the sprites a lil smaller.
Removed the resizes in the yaml, also did it for ferrets on request (their sprite is pretty small already.)
Screenshots

Changelog

:cl:

tweak: Due to a budgetary oversight, possums and raccoons are now smaller.

Well done! Love the budgetary oversight in the changelog 🤣


Merge gui and cli

Currently if the user has to run the game in GUI/CLI mode, he has to run the respective python file in the folder.
Create a "master" python program which after running gives the option to select from GUI/CLI Interface.
The respective python code(GUI/CLI) shall be executed as per the selected option.

Try not to shift the existing file structure.
Try not to rename the existing files.

hey @anantdark assign me this issue

Hi @yogesh-9999 , I have assigned it to you, waiting for PR.
However do note that this doesn't give you exclusive access to this issue and any other PR(if submitted) will be reviewed.

@yogesh-9999 progress?


Create a page to show the VCs issued

Use case: Build templates for issuers to easily use available basic templates and issue verifiable credentials
Mock design to show VCs created:

User should be able to view all VCs created.
User should get an entry point to 'Issue' and 'View' VC.
3. When there are no VCs created, show as below:

Please assign this issue to me

Please assign this issue to me

Login details:
<EMAIL_ADDRESS>/ 1234

Figma Link: https://www.figma.com/file/vytKlKFg74ypSj6haevygK/Sunbird-RC-for-Hacktoberfest?node-id=0%3A1

API:
curl --location --request GET 'https://sunbirdrc-sandbox.xiv.in/registry/api/v1/Schema' \
--header 'Authorization: Bearer {{keycloak_token}}' \

@tejash-jl do we have any documentation for the APIs which I will be using to show data on this page?

Udpate: removed 'View' icon from the card.
User should be able to view details (records) on click of the card.

@tejash-jl do we have any documentation for the APIs which I will be using to show data on this page?

https://sbrc-swagger.tiiny.site/#/default/get_api_v1_Schema


Error when I connect

Issue Type: Error
When I try to connect to an server with the name instead of ip address i get an error.
Azure Data Studio version: azuredatastudio 1.32.0 (4a45ba7cf20dd4129f1a08e5e776dfb33e3d1d1e, 2021-08-16T18:08:28.086Z)
OS version: Windows_NT x64 10.0.19043

@Avaruz the client driver will try to use DNS to convert a server name into an IP address before communicating to the server.  Have you confirmed that the server name is correctly being resolved on the machine you're connecting from?  You can run nslookup [SERVERNAME] from the CLI to confirm that DNS resolution is working as expected.

The error doens't happen again.


Confirm company details every application

Summary
Suppliers need to confirm that the company details they've declared in
their account remain accurate over time as they apply to new iterations
of frameworks on the Digital Marketplace.
When suppliers first set up their account, they confirm that some
immutable attributes are correct: registered name, company registration
number, VAT number and DUNS number. Once these have been confirmed as
correct, they are locked down in the account and cannot be edited
without talking to support. Generally, these will only be updated by
support if there has been a mistake in entering the details. If the
details have legitimately changed, the supplier will need to create a
new supplier account on the Marketplace to complete their application.
After confirming their details are correct at the account level, they
must also confirm the details are correct at the framework application
level. Until they do this, they cannot progress their application
(complete the declaration or add services). This is to prevent the
supplier from putting time into these labour-intensive tasks before
realising that their supplier account no longer accurately represents
their company.
Ticket
https://trello.com/c/V0FSdOCJ/188

Still need to write at least these tests:

confirm_supplier_details

all/only open framework applications are confirmed

supplier dashboard:

session.currently_applying_to is cleared on visit

supplier_details:

supplier unconfirmed + application unconfirmed get confirmed at the same time
supplier confirmed + open unconfirmed application shows save button

But this is mostly done now, so please feel free to start reviewing


refactor: upgrade packages

eslint v7

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Fix API key usage

Description
Fixed RobotSession class so it uses the robot API key for sending status instead of service API key

@leandropineda  what a super quick fix! I left a dummy question, if it's ok then LGTM :)


What license is this code released under?

Things that you create and don't specify a license for a copyrighted to you, and not usable by others without explicit permission.
As far as I can see, there's no license specified for this, so anyone who wants to use it would need to get permission directly from you for each use.
Is this intentional? If not, you should specify the license, so others can use it.

I sorry, it wasn't intentional.
Updated now.


Clarify useImperativeHandle usage

It's not clear how to use the component once it has been configured. This clarifies it by being more descriptive.
Original:

a parent component that renders <FancyInput ref={inputRef} /> would be able to call inputRef.current.focus().

Modified:

a parent component that renders <FancyInput ref={fancyRef} /> can now call fancyRef.focus() to focus the underlying <input />.

This is the direct link to the modified section in the Netlify preview: https://deploy-preview-3583--reactjs.netlify.app/docs/hooks-reference.html#useimperativehandle


Virtual

this will fix https://github.com/RobinBuschmann/xml-typescript/issues/6

Hi all,
I am very interested in having this pull request merged. Any chance of that happening?


Possibility of supporting ore mods?

The most popular (are there even any more out there?) fabric ore mods are More Gems and Mythic Metals.
Would it be possible to add dense variants of them?

It's planned to have automatic mod compatibility with auto-generating textures, but as of writing this, the mod only supports vanilla ores.


Better API docs

We should use one of the usual Javascript library documentation tools to get a full API documentation. The examples are pretty okay, but we can do better than that.

Obligatory plug for our own tool: http://documentationjs.github.io/.

If you want a fresh set of eyes to start out the docs, I'd love to help out!

@jakepruitt that would be awesome. Especially with new API around map matching it is important that we point potential users in the right directions (hah, but seriously https://github.com/mapbox/geojson-tidy is a good pointer to have)

We aim to be as close as possible to https://github.com/Project-OSRM/osrm-backend/wiki/Server-api the HTTP API. If you notice any consistency problems, just shout out: That is most likely a bug.

grabbing @jakepruitt @TheMarex

Not seeing that this has moved yet. Tagging @mapsam who would be in a great position to knock this out given his recent big push on https://github.com/mapnik/node-mapnik/pull/597.

Sorry for not closing the ticket. In fact we have documentation now https://github.com/Project-OSRM/node-osrm/blob/develop/API.md

Awesome, thanks for linking that @TheMarex - A potential next action then (once other work has settled) - would be to formally use documentation.js to provide API docs in the format of http://mapnik.org/node-mapnik/documentation/ (and to allow use to keep them up to data via docstrings in the code). But, good to keep this closed - we'll circle back later.

A potential next action then (once other work has settled) - would be to formally use documentation.js to provide API docs in the format of http://mapnik.org/node-mapnik/documentation/

Created ticket for this next action: https://github.com/Project-OSRM/node-osrm/issues/149 /cc @mapsam

Awesome, thanks @springmeyer @TheMarex! Added it to my list once node-mapnik docs are complete.


Fix the bug to not set response channel for delete out of retention request

Since we don't care about if delete out of retention request need to be done after put request, this pr is to make sure when we submit response to client, we don't set extra info for the response channel.

Codecov Report
Patch coverage has no change and project coverage change: -39.76% :warning:

Comparison is base (18ef25b) 72.89% compared to head (4c26dda) 33.14%.
Report is 1 commits behind head on master.

Additional details and impacted files
@@              Coverage Diff              @@
##             master    #2536       +/-   ##
=============================================
- Coverage     72.89%   33.14%   -39.76%
+ Complexity    11241     4441     -6800
=============================================
  Files           792      793        +1
  Lines         63738    63746        +8
  Branches       7794     7792        -2
=============================================
- Hits          46463    21128    -25335
- Misses        14725    40401    +25676
+ Partials       2550     2217      -333

Files Changed
Coverage Δ

...a/com/github/ambry/frontend/DeleteBlobHandler.java
0.00% <0.00%> (-88.30%)
:arrow_down:

...com/github/ambry/frontend/NamedBlobPutHandler.java
0.00% <0.00%> (-88.16%)
:arrow_down:

...ava/com/github/ambry/rest/NoOpResponseChannel.java
0.00% <0.00%> (ø)

... and 534 files with indirect coverage changes

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

LGTM, just one more question, can we reuse the code in recursiveCallback, to something like this.
        List<DatasetVersionRecord> datasetVersionRecordList =
            accountService.getAllValidVersionsOutOfRetentionCount(accountName, containerName, datasetName);
        if (datasetVersionRecordList.size() > 0) {
          WrappedRestRequest wrappedRestRequest = new WrappedRestRequest(restRequest);
          LOGGER.info(
              "Number of records need to be deleted due to out of retention count: " + datasetVersionRecordList.size()
                  + " for accountName: " + accountName + " containerName: " + containerName + " datasetName: "
                  + datasetName);
          recursiveDeleteDatasetVersions(wrappedRestRequest, datasetVersionRecordList, 0, callback);
        } else {
          callback.onCompletion(null, null);
        }

// This is the recursive call
    private void recursiveDeleteDatasetVersions(RestRequest restRequest,
        List<DatasetVersionRecord> datasetVersionRecordList, int idx, Callback<Void> callback) throws Exception {
      if (idx == datasetVersionRecordList.size()) {
        callback.onCompletion(null, null);
        return;
      }
      Dataset dataset = (Dataset) restRequest.getArgs().get(RestUtils.InternalKeys.TARGET_DATASET);
      String accountName = dataset.getAccountName();
      String containerName = dataset.getContainerName();
      String datasetName = dataset.getDatasetName();
      DatasetVersionRecord record = datasetVersionRecordList.get(idx);
      String version = record.getVersion();
      RequestPath requestPath = getRequestPath(restRequest);
      RequestPath newRequestPath =
          new RequestPath(requestPath.getPrefix(), requestPath.getClusterName(), requestPath.getPathAfterPrefixes(),
              NAMED_BLOB_PREFIX + SLASH + accountName + SLASH + containerName + SLASH + datasetName + SLASH + version,
              requestPath.getSubResource(), requestPath.getBlobSegmentIdx());
      LOGGER.debug("New request path : " + newRequestPath);
      // Replace RequestPath in the WrappedRestRequest for delete and call DeleteBlobHandler.handle.
      restRequest.setArg(InternalKeys.REQUEST_PATH, newRequestPath);
      restRequest.setArg(InternalKeys.TARGET_ACCOUNT_KEY, null);
      restRequest.setArg(InternalKeys.TARGET_CONTAINER_KEY, null);
      //for delete out of retention request, we don't want to set anything to response channel.
      deleteBlobHandler.handle(restRequest, new NoOpResponseChannel(), (r, e) -> {
        try {
          recursiveDeleteDatasetVersions(restRequest, datasetVersionRecordList, idx + 1, callback);
        } catch (Exception exc) {
          callback.onCompletion(null, null);
        }
      });
    }

Yes, I think that one is more simplify. I just use the way you recommended and adding some logs and metrics when delete failed.


keywords separator

From the MMD documentation, it seems we should add a separator value (>) for the keywords in MMD that use that.

done automatically


iOS - Update web view cookie with in-app selected language

Description
LEARNER-7498
Create and set web view cookie with the selected language locale.
Cookie name should be "prod-edx-language-preference" and its value should be selected language locale.
How to test this PR

[ ] Web view language should be updated as device selected language.

@mumer92 Please rebase the code with latest master update.

@mumer92 When i am changing the language to Spanish the discovery webvew is not updating the language. Can you please check whats the reason for it?

@salman2013
Cookie is being applied to webview, but seems like it is not loading specific language.

Seems like it is being applied on device.


Support for conjugate and sympy.I, for handling complex functions

It can now handle complex functions containing sympy.I constant. In response to #7 .

Looks good to me! Thanks for the PR.
Can you also add a test for the relevant functionality? And if you can bump the verison number as well then I'll do a new release.

Added test_complex for simple evaluation and conversion checks of complex numbers, as well as evaluation of spherical harmonics using lambdify. Also bumped version to 0.1.2

Note: The error tolerance for spherical harmonic test is set to 1e-7 as the test file does not use default tensor type as torch.DoubleTensor. Therefore internal evals result in limited precision. With default double datatype I get errors to be 9.724027063517389e-16

Thanks @ipcamit for this feature and @patrick-kidger for the great library! To make sure also the .sympy() method works with complex expressions, you can include an additional check:
def sympy(self, _memodict):
    if issubclass(self._sympy_func, sympy.Float):
        return self._sympy_func(self._value.item())
    elif issubclass(self._sympy_func, sympy.core.numbers.ImaginaryUnit):
        return sympy.I
    elif ...

@madagra Thanks for the suggestion. sympy() method can now properly render the equations.

Thanks for the contribution! I've just done a new release.


Bvef files

I propose the following PR which aims at solving the problem raised in #4070

looks clean !
can you add an entry to what's new page?

Codecov Report

Merging #4109 into master will increase coverage by <.01%.
The diff coverage is 87.5%.

@@            Coverage Diff             @@
##           master    #4109      +/-   ##
==========================================
+ Coverage   86.11%   86.11%   +<.01%
==========================================
  Files         350      350
  Lines       63561    63569       +8
  Branches     9691     9692       +1
==========================================
+ Hits        54737    54745       +8
  Misses       6138     6138
  Partials     2686     2686

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 e945956...57c5c4b. Read the comment docs.

thx @jbschiratti
congrats for the first PR !

Hi @jbschiratti
I am coming across this PR many years later, and I am wondering:

where does the 85 factor come from when defining pos?
how do you know that theta and phi are in degrees and they ought to be converted to radians?
Do you know which coordinate system BrainVision uses? Because after running this code, the coordinate system is the FIFF HEAD coordinate system ... so we must know something about converting from one to the other :thinking:

for convenience, the relevant part of your code is pasted below:
    pol = np.deg2rad(np.array(theta))
    phi = [float(s.text) for s in root.findall("./Electrode/Phi")]
    az = np.deg2rad(np.array(phi))
    pos = _sph_to_cart(np.array([np.ones(len(az)) * 85., az, pol]).T)

I'd be very happy if you could take some time to explain :-)
See also previous attempts to solve this mystery: https://github.com/mne-tools/mne-python/pull/4531/files#r305528241
cc @jasmainak @drammock

I doubt we can provide much details on this with @jbschiratti (was a post
doc in my lab)
Disclaimer we have never been users of brainvision / brain products
ourself. Just tried
to make it work for people.
So anything you find broken and can be fixed, as long as it can be tested
and documented,
is very welcome.


Un-intuitive Capitalization

I think this might have been left out intentionally to reflect the capitalize where capitalized part but, in my opinion, it does properly serve its purpose and seems like a  typo.
As a solution, I propose that it is mentioned in the challenge that the username is UserName

Fixed with #252. Thanks!


ubuntu 20.04 can't install esl-erlang due to a GPG signing error

This issue is similar to one my colleague filed last year #14
I've had issues installing esl-erlang, due to an invalid GPG signature:
#7 2.293 W: GPG error: http://binaries.erlang-solutions.com/debian focal Release: The following signatures were invalid: BADSIG D208507CA14F4FCA Erlang Solutions Ltd<EMAIL_ADDRESS>#7 2.293 E: The repository 'http://binaries.erlang-solutions.com/debian focal Release' is not signed

This is a minimal Dockerfile which reproduces the problem:
FROM ubuntu:20.04

RUN apt-get update \
  && apt-get install -y build-essential ca-certificates curl gnupg2

RUN curl -sSLO https://packages.erlang-solutions.com/erlang-solutions_2.0_all.deb \
  && dpkg -i erlang-solutions_2.0_all.deb \
  && apt-get update \
  && apt-get install -y esl-erlang

I was able to build this successfully at approximately 2023-02-23 14:45, it's only been in the last hour it has started to fail.

Got the same error:
root@929636ababf8:/# apt-get update
Hit:1 http://deb.debian.org/debian buster InRelease
Ign:2 https://packages.erlang-solutions.com/debian buster InRelease
Hit:3 http://deb.debian.org/debian-security buster/updates InRelease
Hit:4 http://deb.debian.org/debian buster-updates InRelease
Get:5 https://packages.erlang-solutions.com/debian buster Release [4572 B]
Get:6 https://packages.erlang-solutions.com/debian buster Release.gpg [833 B]
Ign:6 https://packages.erlang-solutions.com/debian buster Release.gpg
Reading package lists... Done
W: GPG error: https://packages.erlang-solutions.com/debian buster Release: The following signatures were invalid: BADSIG D208507CA14F4FCA Erlang Solutions Ltd<EMAIL_ADDRESS>E: The repository 'https://packages.erlang-solutions.com/debian buster Release' is not signed.
N: Updating from such a repository can't be done securely, and is therefore disabled by default.
N: See apt-secure(8) manpage for repository creation and user configuration details.

Same issue, it started a couple hours ago:
Ign:5 https://packages.erlang-solutions.com/ubuntu focal Release.gpg
Reading package lists...
W: GPG error: https://packages.erlang-solutions.com/ubuntu focal Release: The following signatures were invalid: BADSIG D208507CA14F4FCA Erlang Solutions Ltd<EMAIL_ADDRESS>E: The repository 'https://packages.erlang-solutions.com/ubuntu focal Release' is not signed.
The command "sudo apt-get update" failed and exited with 100 during .

It looks like this issue may be resolved, I'm able to build the Dockerfile as expected now, I'll leave this open pending confirmation.

Hmm, I may have spoke too soon, it worked on CI but on my laptop I see:
#7 66.78 W: Sources disagree on hashes for supposely identical version '1:25.2.3-1' of 'esl-erlang:amd64'.
#7 66.78 W: Sources disagree on hashes for supposely identical version '1:25.2.3-1' of 'esl-erlang:amd64'.
#7 66.78 E: Failed to fetch http://binaries.erlang-solutions.com/debian/pool/esl-erlang_25.2.3-1~ubuntu~focal_amd64.deb  File has unexpected size (40825356 != 40822988). Mirror sync in progress? [IP: REDACTED]
#7 66.78    Hashes of expected file:
#7 66.78     - SHA256:1b7888175d2c78f85369389bbdeba98ee5b0fcab80da41f22e415e51bdfaa607
#7 66.78     - SHA1:057f5ffca06301d9c812c0dd28cdaa9b605f7981 [weak]
#7 66.78     - MD5Sum:2d2ecb60417c808b929269a82e386ef5 [weak]
#7 66.78     - Filesize:40822988 [weak]
#7 66.78 E: Unable to fetch some archives, maybe run apt-get update or try with --fix-missing?

It's resolved for me. I only use CI (travis) to build.

Also spoke too soon, the issue persists:
The following signatures were invalid: BADSIG D208507CA14F4FCA Erlang Solutions Ltd<EMAIL_ADDRESS>

Yes, It was updated and should work. Thanks for the feedback


Move Groth16VerifierExtensions.sol template to this repo

See companion PR here for description.
DO NOT MERGE unless the above is approved.

Closing for now, pending larger discussion on dependency management and E2E testing


Get process and operator classes to use the same daops code etc

Once first version is working, need to review whether we have significant duplication of code - and then remove it.

Both use wrap_director as of PR #83


Checked that the anchor is not online?

The live broadcast room is in the broadcast state, but when the service is turned on, it prompts that the anchor is not online. The method called TikTokLive.isLiveOnline(liveUsername)。Through the browser, analyze the webpage code of the live broadcast room and cannot find the roomId parameter.

Our code does not scrape the webpage for the roomId.

Our code does not scrape the webpage for the roomId.

yes ,it does not scrape the webpage for the roomId。Why does calling the TikTokLive. isLiveOnline (liveUsername) method when the anchor is in the broadcasting state prompt that the user does not exist or is not online? thanks for answer me

This usually only occurs when the account is set to private or the name is misspelled. Other than that, this should not occur. Share me the name and I will look into it.

This usually only occurs when the account is set to private or the name is misspelled. Other than that, this should not occur. Share me the name and I will look into it.

The address of the live broadcast room is https://www.tiktok.com/ @Maya_fami/live，the name is Maya_fami.

Try connecting with this username maya_fami. Aka lowercase and see how that works!

Try connecting with this username maya_fami. Aka lowercase and see how that works!

thanks


Arrows, Circles, Bookmark missing in Training mode

I can't draw arrows and circles in training mode.
I can't bookmark the positions.

Was it designed to be this way? Or unnoticed?

This is by design. The thinking was that the Training mode is for replaying what was prepared earlier (by the user or by the coach) rather than for editing.
Technically, it can be done, of course, if that's what users do need, after all.

It is possible to carry the position of checkmate and stalemate to the training session. Shouldn't it?


Wish to change pointer detection

Proposal Details
run the code locally：invalid pointer found on stack
https://go.dev/play/p/fpp-0tDjr2W

What are you actually proposing here? You need to provide details: https://github.com/golang/proposal?tab=readme-ov-file#detail

This kind of code:
    v = unsafe.Add(nil, c_lock_timeout|c_lock_read)

is implicitly converting an integer to an unsafe.Pointer, which is not permitted by the unsafe.Pointer rules (https://pkg.go.dev/unsafe#Pointer).
We are not going to change that.  That would fundamentally break garbage collection, as we would no longer know what is a valid pointer.  Closing this issue as infeasible.

This kind of code:
    v = unsafe.Add(nil, c_lock_timeout|c_lock_read)

is implicitly converting an integer to an unsafe.Pointer, which is not permitted by the unsafe.Pointer rules (https://pkg.go.dev/unsafe#Pointer).
We are not going to change that. That would fundamentally break garbage collection, as we would no longer know what is a valid pointer. Closing this issue as infeasible.

In two places, I'm going to change the minLegalPointer detection, which has nothing to do with rubbish collection

But we would not know whether a value was a valid pointer.
What benefit would we get from this change?

But we would not know whether a value was a valid pointer.
What benefit would we get from this change?

A field can be extended for two uses, or more
type test1_st struct {
	unk unsafe.Pointer
}

func test1_getUserData(st *test1_st) unsafe.Pointer {
	if v := st.unk; (uintptr)(v)&1 != 0 {
		return unsafe.Add(nil, (uintptr)(v)&^1)
	}
	return nil
}
func test1_getList(st *test1_st) unsafe.Pointer {
	if v := st.unk; (uintptr)(v)&1 == 0 {
		return v
	}
	return nil
}

Sorry, but we are not going to permit user code to change the low bit of pointer values.  That is not the kind of language that Go is.  We already have C and C++, and they are good languages.  Go is a different language.

I'm not comparing it to other programming languages. It's not up to you to decide the low value, that's a user usage range, for example I wrote it that way. golang reporting an error is just a limitation of knowledge and taste, it's not my business if I can't change it, I just hope it doesn't report an error for no reason, not accessing it as an address why does it report an error? I just hope it doesn't report errors for no reason. If I mark this address 1 as a red marker 0 as a black marker, does that mean it doesn't work?

@ianlancetaylor If you could define the range of values for pointer types on the golang docs, outside of which I wouldn't use, could you do that?

Go follows a simple rule: a value with pointer type must be a valid pointer.

Go follows a simple rule: a value with pointer type must be a valid pointer.

What do you mean by valid, you define its scope? Is it valid for CPU access, or for golang allocation? Ok even if the allocation is valid, but I modified it and haven't done a read/write operation yet, do I report invalid only when I read/write the operation?

Go follows a simple rule: a value with pointer type must be a valid pointer.

Is a nil pointer a valid pointer? If it is valid, why is it invalid when I read or write access it? So, you can only determine validity and invalidity when accessing it by read or write. And Go's detection is a range of 0-1024 detection, 1025 is also sure to report an error ah!

A valid pointer is, approximately,

nil
points to a Go variable, including the result of new or &
points to memory allocated by non-Go code using mmap or related functions

Arbitrary math on a value of pointer type is not supported.  Setting a value of pointer type to an integer is not supported, except for a very limited set of cases in which the integer is derived from an existing pointer in a single expression.

A valid pointer is, approximately,
* `nil`

* points to a Go variable, including the result of `new` or `&`

* points to memory allocated by non-Go code using `mmap` or related functions

Arbitrary math on a value of pointer type is not supported. Setting a value of pointer type to an integer is not supported, except for a very limited set of cases in which the integer is derived from an existing pointer in a single expression.

Why I do this arithmetic with pointers is not nonsensical, it's not just scribbling.
1, a pointer is memory that holds an address meaning it can't be freed by the GC by mistake, I need pointers that protect memory not the other way around.
2, because of the address memory it is aligned and therefore has low value arithmetic capabilities and markers.
3, only I can guarantee that the pointer has no validity, because it's my code, it's guaranteed that it won't execute haphazardly.
4, the code I gave as an example only reports errors with build-debug, so it's the debug option that's causing the problem, and I don't know how to turn that option off yet. The code can be wiped out though, that's the ultimate solution to whether the pointer is valid or not

Why I do this arithmetic with pointers is not nonsensical, it's not just scribbling.

I understand that.  I'm telling you that for the code you want to write, Go is the wrong language.  Use C or C++.

Why I do this arithmetic with pointers is not nonsensical, it's not just scribbling.

I understand that. I'm telling you that for the code you want to write, Go is the wrong language. Use C or C++.

Is my question hard for you, then I'm sorry, maybe some day I'll abandon golang for something else, that's another story too.

This is not productive. If you want to continue, like I said before, start a proposal following the documented process and provide enough detail. I'm locking this issue before it gets worse.


Tests continue to run after package removed

I wanted to turn off testing temporarily because it taxes my computer while I'm continuously saving during development. I tried #21, but the tests still run – should I open a FR for that in the velocity repo?
I then tried removing xolvio:cucumber, but the cucumber tests still run, which seems hard to believe, but I'm sure it's really happening :) Note that I still have mike:mocha and velocity:console-reporter included in the app.

You found the bug I told you about at the devshop!
Try to kill any rogue node/phantom/selenium processes
I'm on the final bits of 0.7.0

Haha, k. Not finding anything ps -A | grep X, but that's fine, will try again w/ 0.7. Should VELOCITY=0 meteor work normally?

Yes it should, try also CUCUMBER=0
It's really odd that it's still running then!
On Sun, 3 May 2015 23:10 Loren Sands-Ramshaw<EMAIL_ADDRESS>wrote:

Haha, k. Not finding anything ps -A | grep X, but that's fine, will try
again w/ 0.7. Should VELOCITY=0 meteor work normally?
—
Reply to this email directly or view it on GitHub
https://github.com/xolvio/meteor-cucumber/issues/85#issuecomment-98595038
.

Alright, after running CUCUMBER=0 VELOCITY=0 meteor once, then just running meteor doesn't start cucumber anymore :)

I can't seem to replicate the original error and I tried CUCUMBER=0 and VELOCITY=0 (independently) and both worked. Can you please try with 0.7.0


Clarifying Example 4 desc

Closes https://github.com/MicrosoftDocs/office-docs-powershell/issues/7636
The search is not exact, so it is like you put "%Sales and Marketing%", I am open to other ways to describe this...

Docs Build status updates of commit 53e8a38:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

teams/teams-ps/teams/Get-Team.md
:white_check_mark:Succeeded
View (teams-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 25ae4bc:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

teams/teams-ps/teams/Get-Team.md
:white_check_mark:Succeeded
View (teams-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

@Kateyanne Please merge this PR. Thanks.

Docs Build status updates of commit 6b63a94:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

teams/teams-ps/teams/Get-Team.md
:white_check_mark:Succeeded
View (teams-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

The docs are misleading, and there are false statements.
First, it says:
"Returns the team that includes the specified text in its DisplayName. (This acts as a filter rather than an exact match)."
Then, if you go below, on the detailed info, it says this:
"Filters to return teams with a full match to the provided displayname. As displayname is not unique, this acts as a filter rather than an exact match. Note that this filter value is case-sensitive."
So 1) it's misleading that it says "with a full match to the provided displayname", and 2) in the second statement, "As displayname is not unique, this acts as a filter rather than an exact match", the first part of the phrase ("As displayname is not unique") can never be an explanation for this acting "as a filter rather than an exact match", because yeah, we know we can have several teams with the exact team name, and in that case it would make sense that
Get-Team -DisplayName would return all the Teams it found WITH THE EXACT MATCH; that's not a justification for returning partial matches, so a) you better remove that "explanation" and, b) and more important, remove the "WITH A FULL MATCH TO THE PROVIDED DISPLAYNAME" part.
But what really should be addressed is bug 7636 (https://github.com/MicrosoftDocs/office-docs-powershell/issues/7636), allowing to make both exact and partial searches.


Planed features for next release

The following options will be added in the next version:

Add option to force a specific GPU ( -force-device-index) (all platforms)
Add option to force Metal API (-force-gfx-metal) (MacOS only)
Add option to force a specific monitor (-adapter) (all platforms)
Add options to force a specific resolution (-screen-width and -screen-height) (all platforms)
Optional button to launch a 3rd party app (won't appear if not configured in options window) (as requested in issue #10)

Is there any way to use your launcher to enable mods? It looks like, as currently implemented, the workshop will download mods but they've disabled its ability to actually activate the mod  (in Stellaris, I suppose they'll extend it to the other games at some point) without using a launcher. So I can download mods through the workshop but not activate them.
Annoying default paradox launcher won't uninstall or reinstall, and I went deep enough to do a registry clean. The thing is worse then spyware! (though apparently thats because it IS spyware.) I'm on a super, super slow internet connection and have zero desire to do a fourth complete reinstall of Stellaris in as many months, it takes almost a week on my connection each time.
Anyways, I imagine there's no way to make it show a list of the mods so you can activate them as the default launcher does, and I'll have to live, but I thought I'd ask.

@secretfire42 At the moment I'm not planning such a feature.
However, I believe you can use this to manage the mods: https://github.com/ThibautSF/ParadoxosModManagerRework

Is it possible to add a play button so you don't have to wait for the countdown timer?


Firewall / Aliases / Edit - Exclude Reserved Networks

Add options to exclude reserved networks from downloaded lists for compatibility with interfaces utilizing those networks.
(private address networks and local identification networks (DHCP))

I'm leaning towards not merging this in the interest of not adding technical debt that's unnecessary for the vast majority (don't fetch aliases with contents you don't want). But there is code that excludes RFC 1918 from bogons separately, where this function could be used instead. Will reconsider post-2.3.

The "don't fetch aliases with contents you don't want" argument is kind of like throwing out the baby with the bath water.  Just because a list contains a few undesirables doesn't make the rest/remainder/vast majority of the list undesirable and useless.

pfBlockerNG has an option to remove RFC1918 addresses in Feeds. There are some corner cases where these feeds add loopback and other RFC1918 addresses...  Seen this with IBlock Feeds on occasion.

BBcan177 brings up another pretty good point here.  In regard to the "don't fetch aliases with contents you don't want" argument, just because a particular list is "clean" today doesn't mean it will be next month.  Then suddenly the network is broken.  Think it is a good idea to be able to exclude critical networks like private and id from the downloads.

I would consider more useful if user could insert a list of exceptions instead of having these 2 fixed options. But I would like to hear from @jim-p what he thinks

Having a configurable list of networks to exclude would be better. It could default to RFC1918, RFC4193, maybe even RFC5737 & RFC3849

It won't be accepted the way it is, if you are still interested please re-implement as a list of networks to be excluded. Thanks!


fix(http): set Proxy function for all http client of s3 driver

Use GetClient to create the http client for s3 driver.
longhorn/longhorn#5054
Regression test: https://ci.longhorn.io/job/private/job/longhorn-tests-regression/2937/

@mantissahz Please remember to update the corresponding repo dependencies.


Modal - scheduled publishing

Based upon #8475. Re-implements the existing workflow modal with the new component and new styles. Design link
Reference screenshot:

Expected components:

Trigger
Dialog from #8475
(Info message for top-level warning bar)
Help text
Go-live date field datetimepicker (use as-is field face & picker, not designs)
Expiry date field w/ datetimepicker (use as-is field face & picker, not designs)
Info side panel: Publish & unpublish dates, w/ Edit button

Tasks

Add trigger to footer actions (w/ Python API)
Do not change the definition of PublishingPanel in page panels.py, unless this makes development easier (no need to do data binding or deal with duplicate IDs)
Add form fields as-is, making sure datetimepicker z-index is appropriate
Clicking "Schedule publishing" triggers a "Save draft" similarly to the current "Save draft" footer action
Make sure the "dirty form check" alert doesn’t show when submitting the form from inside the modal
Cancel button dismisses the dialog, and uses ghost button styles (not red)

Acceptance criteria

[ ] Matches design
[ ] As an editor, when I click "Schedule publishing", I get my whole page’s data saved and I can see the same success message (or error messages) as any other draft save.
[ ] As an editor, I can change scheduled publishing or un-publishing from the info side panel.

Open questions

What are the criteria for the warning message to display?
As a user who can’t schedule publishing, what do I see?
As a user who can’t schedule publishing, can I still remove an upcoming scheduled publishing?

Testing / review plan

Permissions tests by @stevedya
Accessibility tests by @thibaudcolas
Code review by @thibaudcolas

It would be nice if we updated the styling of the save draft drop up as part of this ticket too I think.

Implemented in #8901