proposal: x/crypto: add public method to unmarshal ssh certificates.

What did you do?
While writing an application that generates ssh certificates I wanted to audit its output, by verifying the signed ssh certificate against signing policies.
What did you expect to see?
A public function that unmarshalls a []byte into an *ssh.Certificate. This is 98% done already with the private parseCert function.
What did you see instead?
No public function to unmarshal  ssh certificates (a private one exists) and the Unmarshall (which is the mirror of Marshall and not suitable for my purposes ( see: https://github.com/golang/go/issues/21491 )).

The code for this function would be trivial:

func ParseCertificate(wireBytes []byte) (*Certificate, error) {
algo, in, ok := parseString(wireBytes)
if !ok {
return nil, errShortRead
}
return parseCert(in, certToPrivAlgo(string(algo)))
}

/cc @hanwen @x1ddos @FiloSottile @agl (recent Reviewers of golang.org/x/net/ssh)

doesn't ParsePublicKey work?
btw - what does "verifying the signed ssh certificate against signing policies" mean?

No, parse publicKey gets me the public key, not the other members of the certificate like the ValidPrincipals, ValidAfter, ValidBefore etc.
what does "verifying the signed ssh certificate against signing policies" mean?
After I generate a certificate ( (*Certificate) SignCert ) and give it to an external user ( (*Certificate) Marshal) I wan to take the marshalled representation and verify that the ValidBefore, ValidPrincipals match what I expect.
Think certificate transparency for ssh certs.

public key is an interface. You can cast to ssh.Certificate, see https://gist.github.com/hanwen/ec620792123ffee5c2cdfbcc33fab0da
SignCert just signs the certificate in place (setting nonce, signature and signaturekey). You can (and should!) check the fields of the certificate that you want to sign before you call SignCert.
Marshal does not modify the certificate, so there is no need to go through a Marshal/Unmarshal step when you do this check.

I guess I explained myself poorly. I have to processes: P1: generates ssh certs and writes the marshalled certificate to a log. P2: Audits P1 by marshalling the generated certificates(in the log) and looking at the fields.
The publicKey interface is not enough for auditing purposes. This is an ask for the inverse of (*Certficiate)Marsall just like the crypto/x509's ParseCertificate.

"The publicKey interface is not enough for auditing purposes." - you can cast the PublicKey to a certificate and audit it to your heart's desire.
I don't want to provide syntactic sugar for what is essentially a one-liner, but maybe you can think of a way to clarify the docs?

We don't know what that one-liner is? Can you please post sample code which performs this conversion?

var pk ssh.PublicKey
cert  := pk.(*ssh.Certificate)

Wow. Thank you for showing this. I'm happy to report that this works for us. I have to say, this is basically non-discoverable. I would suggest adding a note to the documentation for sshParsePublicKey saying that - given the correct input - will return *ssh.Certificate as the concrete type.


[BUG] 404 ERROR - page can be found on docker hub repository

Describe the bug
A 404 page error when : 5/ Examine if image listed in docker-compose journeys/celery-worker has exactly the same digest that can be found on docker hub repository with the same tag
https://hub.docker.com/r/f5devcentral/f5-bigip-journeys-app/tags?page=1&ordering=last_updated
To Reproduce
Steps to reproduce the behavior:

Go to  https://hub.docker.com/r/f5devcentral/f5-bigip-journeys-app/tags?page=1&ordering=last_updated
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Context

Please specify, app version:
Last

And Platform version of the system you want to migrate.

Hello @endymion93 ,
It is very strange, when I go on the same link, I am not getting the same 404 error.
I am wondering if this is a problem on the docker side of things. I am in Seattle, which part of the world are you?

Thanks!

The link also works for me.

hy
Oops, sorry for the inconvenience, it seems that the problem came from our DNS.
it's ok from france, today.
thank


Draft: [481] Custom Labels for Services and Pods

Apply custom labels to most resources that are created by the operator.
Not all resources have labels, so some - like Containers, EnvVar's, etc.

This needs additional tests to make sure all resources have the labels applied correctly.  But, it's in generally good shape to review.

@davidlanouette thanks for your PR but there is also the issue #588 related to custom labels. The solution discussed in the issue #588 is that the ActiveMQArtemis CR should have the field labels in acceptor, connector and console too.

@brusdev Thanks for the note.  Looking at that issue, it looks like it's mostly a duplicate of 481.  I'll comment on that issue.
Regardless, I'll add see about incorporating the comments in that issue here.

@davidlanouette I think that we can close this this PR because the solution implemented by #737 already allows Custom Labels and Annotations on supporting resources; Services, Ingress, Secrets etc.. Feel free to reopen it if I'm missing something.


docs: fix the 'postcss-present-env' spelling mistakes

This PR contains a:

[ ] bugfix
[ ] new feature
[ ] code refactor
[ ] test update
[ ] typo fix
[x] metadata update

Motivation / Use-Case
fix docs

Breaking Changes
No

Additional Info
No

Thanks!


docs(README): document new 5G network types

Overview

Fixes https://github.com/react-native-netinfo/react-native-netinfo/issues/543
Added 5G network type to README file

I see what you mean about pattern (or lack of pattern to follow) - I think this looks great, and I appreciate the update, thank you!

No problem! Thank you!

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[Bug]: After the breakpoint is completed, the repl does not refresh the screen by using dap.continue.

I love using dap-ui to debug code in various languages, but the dap-ui event seems to be a problem.

not update repl after use require("dap").continue()

For a simple example, I set a breakpoint and if I run the next step with dap.continue(), it won't trigger the repl update, as the following video may show. About this I submitted a pr to fix this issue.
https://github.com/user-attachments/assets/94b02549-ad59-4923-86e4-774aab97165f

not update repl after use require("dap").close(), I can only use this method now instead.

function()
  require("dap").close()
  local controls = require "dapui.controls"
  controls.refresh_control_panel()
end

https://github.com/user-attachments/assets/2ea6f24c-205b-49bd-91b8-db719b392034

https://github.com/rcarriga/nvim-dap-ui/pull/392


Improvements to ExactDiag

I have two suggestions for ExactDiag:
(1) Allow it to take an infinite MPO/model with a specified number of unit cells.  Currently one can call
if Lx > 1:
    M.enlarge_mps_unit_cell(factor=Lx)
M.lat.bc_MPS = 'finite'
to accomplish this, but it takes some investigation to see that just declaring bc_MPS to be finite will work properly.  (I believe it does)
(2) Better behavior for empty charge sectors.  Currently, initializing ExactDiag with an empty charge sector raises an error, which makes sense because an empty npc array could lead to problems in further computations.  However, the current behavior is problematic when generating the full spectrum by iterating over possible charge sectors: if some happen to be empty, it would be convenient to ultimately just get an empty list of energies.

Both solutions are great, thanks!  For (1), hard bcs is indeed what I had in mind (though I guess an option for periodic bcs could also be useful, I'll consider adding that at some point).  For (2), I did initially implement something myself and it was indeed rather ugly.  Your solution makes sense.


merge latest ui code to fix block issues

merge latest ui code to fix block issues

Coverage remained the same at 54.636% when pulling 079cca5f918bb884a0fd1d4848542187bbf675ca on feature/merge_latest_ui_code into 4d5b6d8190fad71dbee3647ccef0466c973eef35 on dev.


Suggestion: Define a New Module for Built-in Colormaps

Move the following colormaps as subroutines to a new module to organize all colormaps in separate modules:

grey
inverted_grey
fire
rainbow
inverted_rainbow
zebra
cubehelix

Yes, that makes sense as the project has gained more scope than expected since I added the Scientific Colour Maps...
That could be a miscellaneous module for example.


Failed to write to hcidump.pklg - :enospc

I have created a clean nerves project which is running the example of https://github.com/blue-heron/blue_heron/pull/57 . After around 4 hours of execution I got the following logs spammed in my console:
10:51:22.592 [info]  Plant #1 - moisture: 30
10:51:32.577 [info]  Plant #1 - fertility: 168
10:51:42.607 [info]  Plant #1 - temperature: 25.1
10:51:53.588 [info]  Plant #1 - lux: 249
10:52:02.579 [info]  Plant #1 - moisture: 30
10:52:12.592 [info]  Plant #1 - fertility: 165
10:52:22.600 [info]  Plant #1 - temperature: 25.1
10:52:32.619 [info]  Plant #1 - lux: 249
10:52:42.585 [info]  Plant #1 - moisture: 30
10:52:45.081 [warn]  Failed to write to hcidump.pklg - :enospc
10:52:45.083 [debug] HCI Packet in <<0x3E, 0xC, 0x2, 0x1, 0x4, 0x1, 0x38, 0xE3, 0x57, 0x3B, 0x5D, 0x65, 0x0, 0xB0>>
10:52:45.135 [warn]  Failed to write to hcidump.pklg - :enospc
10:52:45.137 [debug] HCI Packet in <<0x3E, 0x2B, 0x2, 0x1, 0x2, 0x1, 0x4C, 0x7F, 0x0, 0x5A, 0x38, 0x53, 0x1F, 0x1E, 0xFF, 0x4C, 0x0, 0x7, 0x19, 0x1, 0xE, 0x20, 0x2B, 0x77, 0x8F, 0x1, 0x0, 0x5, 0x64, 0xCC, 0x48, 0x3A, 0x55, 0x3D, 0x4C, 0x7C, 0x65, 0x93, 0xCE, 0xAA, 0x22, 0x55, 0x92, 0xAB, 0xBE>>
10:52:45.146 [warn]  Failed to write to hcidump.pklg - :enospc
10:52:45.149 [debug] HCI Packet in <<0x3E, 0x2B, 0x2, 0x1, 0x3, 0x1, 0xCB, 0x6B, 0xA7, 0xAC, 0x9B, 0x25, 0x1F, 0x1E, 0xFF, 0x6, 0x0, 0x1, 0x9, 0x20, 0x2, 0x1, 0x16, 0x59, 0xFD, 0xCE, 0x70, 0xC7, 0xF3, 0x8E, 0x12, 0x5, 0xD7, 0x9C, 0xF2, 0xED, 0xC2, 0xA7, 0xA5, 0xEB, 0xBA, 0x8, 0x57, 0x19, 0xBB>>
10:52:45.169 [warn]  Failed to write to hcidump.pklg - :enospc
10:52:45.172 [debug] HCI Packet in <<0x3E, 0x2A, 0x2, 0x1, 0x0, 0x1, 0xE0, 0x39, 0x93, 0x4D, 0x45, 0x41, 0x1E, 0x2, 0x1, 0x2, 0x13, 0xFF, 0xFF, 0xFF, 0x32, 0x35, 0x61, 0x66, 0x61, 0x37, 0x38, 0x61, 0x66, 0x35, 0x38, 0x32, 0x63, 0x35, 0x33, 0x34, 0x2, 0xA, 0xF1, 0x3, 0x3, 0x56, 0x34, 0xD5>>
10:52:45.200 [warn]  Failed to write to hcidump.pklg - :enospc
10:52:45.203 [debug] HCI Packet in <<0x3E, 0x14, 0x2, 0x1, 0x4, 0x1, 0xE0, 0x39, 0x93, 0x4D, 0x45, 0x41, 0x8, 0x7, 0x9, 0x53, 0x48, 0x49, 0x45, 0x4C, 0x44, 0xD5>>
10:52:45.235 [warn]  Failed to write to hcidump.pklg - :enospc
10:52:45.238 [debug] HCI Packet in <<0x3E, 0x2B, 0x2, 0x1, 0x3, 0x1, 0xFB, 0x16, 0xC3, 0xC1, 0x46, 0x40, 0x1F, 0x1E, 0xFF, 0x6, 0x0, 0x1, 0x9, 0x20, 0x2, 0xAF, 0xB4, 0x68, 0x75, 0xB, 0xD, 0x98, 0x7B, 0x5A, 0xF3, 0x1A, 0x3F, 0x75, 0x52, 0x4A, 0x6, 0x31, 0xEE, 0xF8, 0x5E, 0x48, 0xF5, 0xB9, 0xBA>>
10:52:45.254 [warn]  Failed to write to hcidump.pklg - :enospc
10:52:45.257 [debug] HCI Packet in <<0x3E, 0x1A, 0x2, 0x1, 0x0, 0x1, 0x7A, 0x8C, 0xD8, 0x4B, 0x2A, 0x63, 0xE, 0x2, 0x1, 0x6, 0xA, 0xFF, 0x4C, 0x0, 0x10, 0x5, 0x1, 0x1C, 0x19, 0x2D, 0x52, 0xBB>>
10:52:45.274 [warn]  Failed to write to hcidump.pklg - :enospc
10:52:45.277 [debug] HCI Packet in <<0x3E, 0x1D, 0x2, 0x1, 0x0, 0x1, 0x5D, 0x15, 0xDD, 0xE9, 0x8, 0x45, 0x11, 0x2, 0x1, 0x1A, 0x2, 0xA, 0x18, 0xA, 0xFF, 0x4C, 0x0, 0x10, 0x5, 0xD, 0x98, 0x88, 0xBE, 0xFE, 0xB6>>
10:52:45.297 [warn]  Failed to write to hcidump.pklg - :enospc
10:52:45.299 [debug] HCI Packet in <<0x3E, 0x2B, 0x2, 0x1, 0x3, 0x1, 0x35, 0x90, 0x8F, 0x5D, 0x17, 0x30, 0x1F, 0x1E, 0xFF, 0x6, 0x0, 0x1, 0x9, 0x20, 0x2, 0xED, 0xAB, 0xCC, 0xE, 0xFD, 0x3A, 0x93, 0xD3, 0x30, 0x31, 0x6E, 0x31, 0x94, 0x25, 0x31, 0xB2, 0x78, 0x13, 0x6B, 0xCD, 0x10, 0x91, 0x19, 0xBB>>
... logs go on forever

What could the reason for this be? My df -h output looks like this:
Filesystem                Size      Used Available Use% Mounted on
/dev/root                27.4M     27.4M         0 100% /
devtmpfs                  1.0M         0      1.0M   0% /dev
tmpfs                    30.9M      8.0K     30.9M   0% /tmp
tmpfs                    15.5M      4.0K     15.5M   0% /run
/dev/mmcblk0p1           18.7M      5.9M     12.8M  32% /boot
/dev/mmcblk0p3           56.7G    148.0K     53.8G   0% /root
tmpfs                     1.0M         0      1.0M   0% /sys/fs/cgroup

Ok I found it, seems that blue_heron is logging a hci_dump file by default:
      with true <- Application.get_env(:blue_heron, :log_hci_dump_file, true),
           :ok <- File.write("/tmp/hcidump.pklg", encoded, [:append]) do

I think this should be false by default since it could produce some huge files after running for a while - and if you are not aware it could crash a production environment.


answered 3 homework questions

I added my three answers for Unix and Git Workshop homework. I will certainly continue to practice the commands learned during this workshop and beyond. If you are able to, please provide feedback or additional tips that will make life easier. Thanks!

Thank you for coming, and thanks for the feedback! We really appreciate it! Feel free to ping either Alex or myself on slack or via email if you need help or have any questions.


If I input invalid address for BCH withdraw the system will fails with «Error on withdraw audit: Could not determine address version»

Closed via #1154.


moved table to an include file

Prepping work for issue #1646 to avoid duplicate content to be maintained
@terrajobst does the table look up-to-date to you?
Fixes #1093

I sugest to change the row for .NET Framework like this:
 | Platform Name              | Alias       |        |        |        |        |        |        |        |       |
 |----------------------------|-------------|--------|--------|--------|--------|--------|--------|--------|-------|
 | .NET Standard              | netstandard | 1.0    | 1.1    | 1.2    | 1.3    | 1.4    | 1.5    | 1.6    | 2.0   |
-| .NET Framework             | net         | → | 4.5    | 4.5.1  | 4.6    | 4.6.1  | 4.6.2  | vNext  | 4.6.1 |
+| .NET Framework             | net         | → | 4.5    | 4.5.1  | 4.6    | → | → | → | 4.6.1 |

Otherwise it looks like 4.6.1 only supports .NET Standard 1.4 and 2.0, but not 1.5 or 1.6. That's no longer true. By claiming support for 2.0, 4.6.1 implicitly supports 1.0 - 2.0, which includes 1.5 and 1.6. The fact that 4.6.2 supported 1.5 and no .NET Framework ever had specific support for 1.6 is irrelevant at this point.
If you do that, you match the table in the .NET Standard repo:
https://github.com/dotnet/standard/blob/master/docs/versions.md
I'd also encourage you add some additional text explaining how to read the table. This is what I wrote in the document linked above:

The table below lists all versions of .NET Standard:

The columns represent .NET Standard versions. The header is a link to a
document that shows which APIs got added in that version of .NET Standard.
The rows indicate which version of a given .NET platform implements a given
.NET Standard version.
The arrows indicate that the platform supports a higher version of .NET
Standard. For instance, .NET Core 1.0 supports the .NET Standard version 1.6,
which is why there are arrows pointing to the right for the lower versions 1.0
- 1.5.

You can use this table to understand what the highest version of .NET Standard
is that you can target, based on which .NET platforms you intend to run on. For
instance, if you want to run on .NET Framework 4.5 and .NET Core 1.0, you can at
most target .NET Standard 1.1.

Thanks @terrajobst. I changed the language a bit for the explanation of the table, especially the part about the arrows which is what causes the most confusion. Even if we say that .NET Core 1.0 supports .NET Standard 1.6, it's not clear you can target the lower versions that have the arrows. What do you think?
Also, I wanted to add the part about choosing the version to target, but I had a question about the part that says that generally speaking you'd want to target the lowest version. So after you figure out the highest you can target, then you need to analyze the APIs you want to use and pick the lowest?
Thanks!

[@mairaw] What do you think?

Looks great! Do you think it would be useful to incorporate these two section as well? The might be useful for understanding how .NET Standard is supposed to work and what you can expect.

How do I know which .NET Standard version I should target?
When choosing a .NET Standard version, you should consider this trade-off:

The higher the version, the more APIs are available to you.
The lower the version, the more platforms implement it.

So generally speaking, you should target the lowest version you get away with.
To inform your decision, you should consider the compatibility matrix and the
APIs additions linked from the table above.
Versioning rules
There are two primary versioning rules:

Additive. .NET Standard versions are logically concentric circles: higher
versions incorporate all APIs from previous versions. There are no breaking
changes between versions.
Immutable. Once shipped, .NET Standard versions are frozen.

New APIs will first become available in specific .NET platforms, such as .NET
Core. If the [.NET Standard review board][netstandard-board] believes the new
APIs should be made available everywhere, they'll be added in a new .NET
Standard version.

[@mairaw] So after you figure out the highest you can target, then you need to analyze the APIs you want to use and pick the lowest?

Yes, and I follow a strict scientific process:

Target next lower version.
Does it still build?

Yes? Go to 1.
No? Retarget to the higher version, stop.

Once we have more versions, you can modify this process to use a binary search instead ;-)
In all seriousness, it would be great if we had tooling that does that automatically.

I'll add that. I thought there were some useful info there but wasn't sure what the process was to pick up the .NET Standard version so wanted to confirm that before adding. Thanks @terrajobst!

@weshaggard @terrajobst the topic has been updated. Please take a look. I've also changed the terminology from .NET Standard Library to just .NET Standard. Since this topic has been written a long time ago, if you find any inaccuracies, please let me know. I'd like at some point to simplify this topic a bit more, but I won't have time to do this with this PR.

Find the column in that row that indicates your version starting from right to left.

what is the reason for reading from right to left?

you should pick the common version among them, which would be the smaller version.

If one is smaller they can't be common, right?  Shouldn't it be something like "Repeat the process for each platform you want to target and take the lowest standard".

Also, comparing versions of different platforms is not meaningful.. so I don't understand that sentence at all.

@sam-wheat if you're trying to find the highest version, I think you should do right from left. Let's say you want to target .NET Core 1.0, first one on the right is 2.0, then 1.0 comes with 1.6. Makes sense?
I'm changing the bit about the common version.

@mairaw In my opinion its counter-intuitive.  More correct however is that its unnecessary since it does not make a difference one way or the other.  Unnecessary instructions lead the reader to wonder what he/she is missing and are source of confusion.

@mairaw just curious what is blocking this PR ... just got some feedback on gitter about https://github.com/dotnet/standard/pull/243 being better ... can we match it in docs?

@karelz I fixed the merge conflicts.  When this finishes building, I'll merge it.

@Nirmal4G When I looked at all the comments and changes, I thought everything had been addressed. Do you see anything I missed?

@BillWagner Thanks
That's all I have, I'll see if something else needs me when I go through the docs!


Torch Impl project_pix does not use eps

def project_pix(fxfy, p_view, center, eps=1e-6):
-    rw = 1.0 / (p_view[..., 2] + 1e-6)
+    rw = 1.0 / (p_view[..., 2] + eps)

On this line  https://github.com/nerfstudio-project/gsplat/blob/0cef46cb4e1758a11c6317246ff165dd147989a3/gsplat/_torch_impl.py#L328

Oh, right. Could you send a quick fix for it?

Close as the fix is merged


Skill specific idle alerts

Idle alerts specifically for certain skills and activities, i.e. mining / woodcutting - rather than a general idle alert.

Already present.


support reset mappings

SUMMARY
Add support for new mapping reset function
DETAILS

support new reset mapping function
add a clear function to clear non-default mappings

CHECKLIST

[ ] Documentation updated (if needed)
[x] Unit tests exist to cover the code you are changing and validated
[x] Integration tests exist to cover the code you are changing and validated

Thank you! Looks awesome. I wonder if there is a practical way of testing if the default mappings are restored. It depends on files in disk and may conflict between tests

i tried to do that but adding a mapping file to the repo and running that during integration tests but required a little more changes than i anticipated. for now, i think it is fine to assume that as long as we are making the correct req, it is wiremock's responsibility to act on that properly although i would work separately to get a test out for this when i get more time


sample tests broken

Bot Info

SDK Platform: .Net
SDK Version: multiple tried
Active Channels: N/A
Deployment Environment: local dev

Issue Description
Microsoft.Bot.Sample.Tests doesn´t compile
Code Example
just clone the repo, restore dependencies, try to compile
Reproduction Steps

Expected Behavior
I´d expect the solution to compile and the test pass
Actual Results
Tests don´t pass because solution doesn´t compile

I´d like to see some working examples of unit tests with bot framework.


EXE marked as containing trojan?

I tried downloading the exe for ErgoStratumProxy 1.2.1, but my antivirus flags it as containing a virus (Trojan-PSW.Win32.Disco.ctp). Is it possible the exe (presumably unintentionally) ended up with the mentioned virus?

In case it's helpful, here is a screenshot of my antivirus (Kaspersky) flagging the executable:

It also lists the MD5 as 9FC4238AC983EF1E2036FBD559B16DD5
Can provide additional info if helpful

No, it's okay. Antiviruses almost always warn about miner-related applications due to lots of cryptographic operations. You can build yourself and compare the executable with the release.


Set up 1.4.0

Description
Please include a short summary of the change.
Issue link (if applicable):
Type of change

[ ] New feature (backwards compatible)
[ ] New feature (breaking API changes)
[ ] Bug fix
[x] Other (Documentation, CI, ...)

Contribution checklist

[ ] Add our license to new files.
[ ] Check that your code adheres to our coding style.
[ ] Write unittests.
[ ] Run the unittests and check that they’re passing.
[ ] Run the benchmarks to make sure your change doesn’t introduce a regression.

I already did this. I also cherry picked your fixes to the CI and numpy version.


App Conflict in Android Emulator

From a fresh installation of Android Studio 1.3, running the app in the built-in Nexus 5 API 22 x86 Emulator will encounter the following issue:

This makes sense. The application ID changed, you'll need to remove the old version first.


Create test for tag_rnn.py

Bug fixed by #899 shows that no tests touch tag_rnn.py (Tensorflow LSTM for sequence tagging).

Closing for now!


Improve readability of applied patch files

Currently, it is hard to read which patch files are applied to ComputeLibrary. By introducing line breaks this becomes easier.

can i close this if this done?


Scrolling in output window does not work when Display Scale is set to Custom

Godot version
v4.1.3.stable.official [f06b6836a]
System information
Godot v4.1.3.stable - Windows 10.0.19045 - Vulkan (Forward+) - dedicated NVIDIA GeForce GTX 960 (NVIDIA; <IP_ADDRESS>17) - AMD FX(tm)-6300 Six-Core Processor (6 Threads)
Issue description
When the display scale is set to certain custom values, the scroll bar will not scroll down to the most recent message when printing something from _process(). It looks to be similar to #78434 and #79975, but only happens for custom display scales.
https://github.com/godotengine/godot/assets/17053434/c6bef5a7-a511-4040-9572-3898321a0be1
Steps to reproduce

Start a new project
In Editor Settings, change "Display Scale" to "Custom"
Change "Custom Display Scale" to 0.8
Create a script that prints a message from _process()
Run the project, notice how the scroll bar doesn't move down to follow the most recent message

Minimal reproduction project
Output Test.zip
(This is just a project that calls print() from _process(), as I can't include the editor settings in the project)

Probably duplicate of https://github.com/godotengine/godot/issues/82041 should be fixed by https://github.com/godotengine/godot/pull/82079.


Deno Support

🚀 Feature Proposal
Add support for Deno, having it deployed on Deno Third-Party Modules.
Motivation
Being able to work with the library inside Deno runtime and skypack doesn't seem to be working
Example
import { svgr } from 'https://deno.land/x/svgr/core.ts';
Pitch
To increase usage in a new and increasing ecosystem.

v6 should work fine with deno, I don't see any problem to use it.

Hey @gregberge through skypack it works but I'm having a TS issue


Filled in metadata not saved when hitting back

If you navigate to the second of two metadata forms (jscholarship), add an author, and hit back, the new author does not show up in the list of authors.

@htpvu Does this still need to be addressed (or perhaps re-prioritized) if we are merging metadata forms into one?
Note that it will still technically be possible for a user to see multiple metadata forms, if the request for a merged schema fails.


Delete event function of administrator

As we discussed in #2, only the user who created the event has the permission to modify an event. It helps from errant users but another problem comes up which is other users cannot use the same event id to create their event once the event name has been used. Even the event is over, the administrator cannot release the event id to other users. So, do you think we should add "delete" under the admin page?

Yes, we can add a terminate event button for the event admin in admin page


Basho\Riak\Search\Doc does not allow to fetch all values returned by Solr [JIRA: CLIENTS-754]

As a developer, I want to access all field names and values returned by Solr so that I don't have to perform individual requests to Riak for each result item.
Right now, there is no way to fetch all values returned by Solr from Basho\Riak\Search\Doc - there is just a magic getter for individual values (you need to know the field name to use it). While it is possible to work with this method if all value names are known upfront (e.g. from the schema), this seems to be slow and pretty inconvenient in some use cases - especially if the result set contains mixed results. Another option would be to only get the location from  Basho\Riak\Search\Doc and perform individual Riak GET requests for each result item (which is very slow and even more inconvenient for large result sets).
I suggest adding a method getData(), which returns an array or an object (which can be converted to an array using (array)):
class Doc
{
    protected $data = null;

    public function __construct(\stdClass $data)
    {
        ...
        $this->data = $data;
    }

    ...

    public function __get($name)
    {
        return $this->data->{$name};
    }

    public function getData() // NEW
    {
        return (array)$this->data;
    }
}

I think this request is reasonable.  If you submit a PR, I will approve it. Otherwise, I probably can't get to it till later this week.

Sure, I can do that. Would you mind adding a method getDataAsArray()? The library is working with stdClass objects by default, but often developers need arrays to further process data. Type casting everywhere is kind of ugly, but I understand if that's too much change for now.

@lastzero based on the comment

The library is working with stdClass objects by default, but often developers need arrays to further process data
My take, in the long run would then be to replace stdClass with an iterator interface because that's what you are actually need to have the requested behaviour; that would be the right way of doing this.

@debo You can only choose between Array and stdClass as return type for json_decode() - that's where $data comes from (second argument). Making more use of standard interfaces sounds like a great idea though for future releases.

@lastzero oh I see, I should have looked at that, make sense.

Yea, I tried to keep things fairly raw with the library by not making many assumptions with how you want the data. This leaves the ability for the developer to wrap it how they see fit while maintaining performance with minimal layers of data mutation.

Thanks for accepting the pull request! We're currently working with the develop branch, but it'd be great to see this in a stable release.

@lastzero Completely understand, I will cut a new release by end of the week.


IE & Safari build issues

When I build using the boilerplate the resulting library works in Chrome & Firefox but in IE I get an error Promise is undefined and in Safari I get Can't find variable: Symbol Do I need to configure the gulpfile differently from the default to make it compatible? I assume I need to add the polyfill? or do I use the runtime ?

I assume I need to add the polyfill?

Yah.

Did that work @glued? :stuck_out_tongue:

yeah I included the JS file browser-polyfill.min.js like the tests  and it works.
How would I add it automatically to the build in the gulpfile?

actually, still didn't work in IE but now safari does now it throws:
Unhandled promise rejection TypeError: Object doesn't support this action


page-data is still preloaded

HTML still contains:
<link as="fetch" rel="preload" href="/page-data/index/page-data.json" crossorigin="use-credentials"/>

This has no purpose without JavaScript but still gets downloaded.

@marvinrabe
Thank you for the issue @marvinrabe, could you please share your gatsby-config.js file so I can better understand how this would happen?

Going to leave this issue open a bit longer but I have not been able to reproduce this myself. Unless @marvinrabe can provide a reproduction of the issue soon then I will have to close this.

This happens when using a gatsby source plugin for fetching remote data.
This can be seen with: https://github.com/TryGhost/gatsby-starter-ghost

This is also an issue for me.
What i originally did (before I found this plugin) was i created a gatsby-ssr.js like this:
const get = require('lodash/get');

function filterJavascriptItems(item) {
  const href = get(item, 'props.href');
  const src = get(item, 'props.src');
  const expression = new RegExp('.(js|json)$');

  if (href && expression.exec(href)) {
    return false;
  }

  if (src && expression.exec(src)) {
    return false;
  }

  return true;
}

exports.onPreRenderHTML = ({
  getHeadComponents,
  replaceHeadComponents,
  getPostBodyComponents,
  replacePostBodyComponents
}) => {
  if (process.env.NODE_ENV === 'production') {
    const filteredHeadComponents = getHeadComponents().filter(
      filterJavascriptItems
    );
    replaceHeadComponents(filteredHeadComponents);
    const filteredPostBodyComponents = getPostBodyComponents().filter(
      filterJavascriptItems
    );
    replacePostBodyComponents(filteredPostBodyComponents);
  }
};

This approach is based on @jordanoverbye https://github.com/jordanoverbye/gatsby-plugin-no-js

@marvinrabe @mathiasha
This is now closed with v2.0.5


Migrate the package repository from jCenter to Maven Central

https://jfrog.com/blog/into-the-sunset-bintray-jcenter-gocenter-and-chartcenter/

The following versions have been released in Maven Central.

Lich Component 2.4.0
Lich Lifecycle 1.3.0
Lich OkHttp 1.4.0
Lich Thrift 1.3.0
Lich ViewModel 2.5.0


Added "emit_group_by_file" option

If "emit_group_by_file" is true, output the struct separately for each input query file, instead of being output collectively in one struct Queries.
The name of the struct is determined from the name of the input query file. For example, if the input file is authors.sql, the type Authors struct will be output.
Depending on the value of "emit_group_by_file", there are the following syntax differences when using it.

"emit_group_by_file": false
q := query.New(db)
result, err := q.CreateAuthor(ctx, "Unknown")
result, err = q.CreateBook(ctx, "my book title")

"emit_group_by_file": true
authors := query.NewAuthors(db)
result, err := authors.Create(ctx, "Unknown")
books := query.NewBooks(db)
result, err = books.Create(ctx, "my book title")

When "emit_group_by_file" is true, the following two options are also affected.

"emit_prepared_queries"

If true, outputs prepared statement support with a lower level syntax.

"emit_interface":

if true, output a interface for each input query file, instead of being output a Querier interface.

Thanks for putting together this PR, but I'm unlikely to merge this change. The reason is that it's possible to get a very similar output by using two separate packages. I know it's inconvenient, but I'd rather not add more options that complicate the code generation.


chore: Added integration tests for Classic, GRE, and DL connections.

PR summary
The Integration Tests needed to be updated to cover other Transit connection types: GRE, DirectLink, and Classic.
Dependency: none
PR Checklist
Please make sure that your PR fulfills the following requirements:

[x] The commit message follows the Angular Commit Message Guidelines.
[x] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

PR Type

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[x] Refactoring (no functional changes, no api changes)
[x] New tests
[ ] Build/CI related changes
[ ] Documentation content changes
[ ] Other (please describe)

What is the current behavior?
The process of deleting a classic connection with a GRE connection attached is failing during the test-resources cleanup.
What is the new behavior?
The bug mentioned above has been fixed. So other Transit connection types: GRE, DirectLink, and Classic, have been added.
Does this PR introduce a breaking change?

[ ] Yes
[x] No

Other information
N/A

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


Implement max for VFloat.

This is needed to eliminate negative dot products.

Not having this feature in place makes the diffuse light rendering look incorrect - the light is not evenly distributed across vertices.

This is done with 4a285019b212262c0b6b4cab1b85cebf86589c53.
If anybody ever wanders how this is done, VFloat is a type synonym for S V Float, which does implement Ord (source). The trick was not to use if-then-else but ifB (source).
To illustrate the defect, this was how the object used to behave:

This is how it renders now:


Questions about using gibson v1 dataset in habitat

Habitat-Lab and Habitat-Sim versions
Habitat-Lab: 0.2.4
Habitat-Sim: 0.2.4
Docs and Tutorials
Did you read the docs? https://aihabitat.org/docs/habitat-lab/
yes
Did you check out the tutorials? https://aihabitat.org/tutorial/2020/
yes
❓ Questions and Help
I am new to habitat, and have learned some basic knowledge about habitat-sim and habitat-lab. But recently I have been stuck in gibson v1 dataset configuration. I have downloaded the gibson dataset as mention here, scene_dataset download here, pointnav here. And my data structure is as follows:
workspace/
  data/
    scene_datasets/
      gibson/
        Adrian.glb
        Adrian.navmesh
        ...
    datasets/
      pointnav/
        gibson/
          v1/
            train/
            val/
            ...

But when i try to train/val my RL agent using the dataset, i found the pointnav/gibson/v1/val/val.json.gz empty (0kb when checking the file). While the val.json.gz in habitat-test-scenes contains necessary episode information. What's wrong with my gibson configuration? Or do I need to do something else to generate the desired data format?
Thank you very much!

ok, it's my fault, don't unzip the file using gui, better by command line:
unzip pointnav_gibson_v1.zip

or you will find some files empty due to the bad proprity of ubuntu's unzip software.


chore: update go version to 1.22.5

##Description
Changes proposed in this pull request:

update golang version to 1.22.5

/retest

/retest

/retest

/test pull-nats-manager-build

/retest


Browser Compatibility: FireFox

It appears the major issue is related to text, but this issue is to make sure everything looks as good under FF as in any other Browser

Selection Currently Failing.  See "Interactive linked state charts" on wiki.  Selection works 1st time only.

The selection failing was actually a text issue also -- it starts doing the selection, draws one text items and then dies. On investigation FF apparently straight up does not support getting bounding boxes of text spans, instead replacing the method with one that throws an error. Checking in a  change that works around that issue.


Update README python versions to match docs

The README and the official docs page disagree on which versions are supported. I assumed that the official docs were correct, so here is a quick PR to make these consistent. Thanks!

Hi there! Thanks for the contribution ❤
Could you:

[ ] Add a Changelog update under the Master heading describing the change
[ ] Update to match current docs

If you can take care of those action items, this can be scheduled for Monday or Thursday next week release 🎉

All set, thank you!

Booya!! I'll take it from here 🎉 I'll need to run the PR from a privileged account, so you'll see this PR close, but your commits are the ones I'll be pushing up (with your name attached). Thanks for your contribution! ❤


whatwg-url parser causes problems driving node-fetch unusable

As I was working with one of my projects, I saw that node-fetch is behaving unexpectedly which wasn't there before. When I ran the code in a test project, it threw these errors. I dug into the code and saw that the URL parse node-fetch v2.6.7 uses which is "whatwg-url" was causing problems. If devs can change the one line in the code, it should work fine.

I saw that the then method works fine.

Reproduction
Steps to reproduce the behavior:

Run in cli npm i<EMAIL_ADDRESS>Create a async function or the one in screenshots
Put any URL to fetch
Run

Expected behavior
Screenshots

Your Environment

software
version

node-fetch
2.6.7

node
16.13.2

npm
8.1.2

Operating System
Replit NodeJS Repl (17~20.04.1-Ubuntu SMP)

Additional context

Sorry, my mistake


CreateDefaultBuilder & ConfigureWebHost/ConfigureWebHostDefaults are broken

Host.CreateDefaultBuilder(args)
.ConfigureWebHostDefaults(webBuilder =>
{
webBuilder.UseStartup<Startup>();
});
The above code is being using in multiple places, but it fails if the StartUp class constructor contains something like "ILogger<Startup> logger" in addition to the "IConfiguration configuration".  If you don't use the "Host.CreateDefaultBuilder()", and instead start with a WebHostBuilder object, everything works as expected.  I am using the following constructor:
public Startup(ILogger<Startup> logger, IConfiguration configuration)
{
Logger = logger;
Configuration = configuration;
}
Which works perfectly when starting with a WebHostBuilder object, but fails with the following exception when starting with the HostBuilder objects:
System.InvalidOperationException: Unable to resolve service for type 'Microsoft.Extensions.Logging.ILogger`1[Webroot.Bcap.WebApi.Startup]' while attempting to activate 'Webroot.Bcap.WebApi.Startup'.

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

ID: c3460b0d-9c60-3a06-ac63-18cf990e53c3
Version Independent ID: b2dece1f-15b8-03af-6d52-7cc070432b4b
Content: Background tasks with hosted services in ASP.NET Core
Content Source: aspnetcore/fundamentals/host/hosted-services.md
Product: aspnet-core
Technology: aspnetcore-fundamentals
GitHub Login: @guardrex
Microsoft Alias: riande

That's correct. That's the new 3.0 behavior. See :point_right: https://github.com/aspnet/Announcements/issues/353.
Inject your logger directly into Startup.Configure ...
https://docs.microsoft.com/aspnet/core/fundamentals/logging/#create-logs-in-the-startup-class
If you need it in Startup.ConfigureServices, call GetRequiredService for an instance ...
var logger = services.GetRequiredService<ILogger<Program>>();

For those who see this after the issue was closed, here is a link to the thread where they try to defend this change, but fail miserably:
[Announcement] Generic Host restricts Startup constructor injection
If you are using NLog, you can use the following line as a work-around for this ridiculous issue:
		Logger = LogManager.GetCurrentClassLogger();


Get L4T version in docker image

I would like to know how to get the L4T version in a docker image.
We used images built from jetpack 5.0, which is L4T r34. We are now using jetpack 5.1 and L4T r35, but it is sometime hard to verify which image is running in our machines since we now longer have the L4T version in the name of our image. Is there a way I can verify what is the L4T version of the image ?
I noticed there are two different ways to check listed here:
https://github.com/dusty-nv/jetson-containers/blob/master/scripts/l4t_version.sh
However, in the container the file nv_tegra_release is empty and the package nvidia-l4t-core is not installed

Thanks. In both of those case, that would give the L4T version of the host, is that correct ? We realized recently that we had a mismatch (running docker images from L4T R34 base while on a R35 host) and even though everything was working fine we want to make sure wwe can detect / prevent that in the future.

Yes, doing it dynamically like that would give you the L4T version of the host.
If you want the L4T version that the container was built for, then I would make another ENV in the dockerfile that gets set to the L4T version that the container was built with.

awesome, thanks !


upgarde: add method for accessing the global data

Coverage decreased (-0.2%) to 69.234% when pulling 93da2e6f3dc0a7c003245d698c2e8a6f8eb2c6df on upgrade-fix2 into edf9736f7e79f3230b5b1f20167ab346444a8333 on master.


Error when parent is not menu attached but has children that are with plugin option cascade disabled

Found a bug in the plugin option Cascade menu attached when enabled. It does not occure when a parent item is not menu attached but its childrens are. Only if you add another level of childrens to the menu attached children like in the below case, the error appears.
Example

Parent (menuAttached: false)

Children (menuAttached: true)

Another Children (menuAttached: false)

Error: Item not found
at getNestedOrders (node_modules/strapi-plugin-navigation/server/services/client.js:268:35)
at getNestedOrders (node_modules/strapi-plugin-navigation/server/services/client.js:271:31)
at node_modules/strapi-plugin-navigation/server/services/client.js:285:77
at Array.sort (<anonymous>)
at Object.renderType (node_modules/strapi-plugin-navigation/server/services/client.js:285:26)
at processTicksAndRejections (node:internal/process/task_queues:96:5)
at async Object.render (node_modules/strapi-plugin-navigation/server/services/client.js:20:16)

Error is thrown in server/services/client.ts:L360.
if (isNil(item))
  throw new Error("Item not found");

Pull Request: https://github.com/VirtusLab-Open-Source/strapi-plugin-navigation/pull/286

@skyclb I'm testing this on version 2.3.0-beta.0 and not seeing such issue. I've provided multiple variations of settings for the structure you provided. For all, error does not occur. May I ask you to test that?

Hi @cyp3rius, thanks that you tried to reproduce my error. Thats weird that you don't get the error when reproducing with described options. Unfortunately I can not test it now, because of a client project which I have to work on first. Once this is done, i will test again.

Re-open after testing please.


Different Terrain Physics

Lets pretend a ball moves on fresh cut grass, it should roll very far.
Now a neglected lawn, it doesn't roll as far.
Now sand, it should stop in its tracks.
Create physics to accommodate these types of terrain.

Due to change in level design we are no longer incorporating different terrain.


Can you bump version of splitwise you use

It seems FluentUI ships with an older version of com.splitwise:tokenautocomplete ( 2.0.8) it prevents app from dropping use of jetifier.
There are convoluted ways to get around it, but it would be nice if Fluentui-android fixed the issue and see if switching to 3.X version of splitwise fixes the issue.
Graphs to this dependency:
+---com.microsoft.fluentui:fluentui_others:0.0.8
+---com.splitwise:tokenautocomplete:2.0.8
For reference see
https://github.com/dipien/bye-bye-jetifier

It appears that this issue has been fixed in version 0.0.29 can someone update this ticket as to when thiis dependency was removed. We don't like to be on "bleeding edge"

Hi @snijsure, I will bring this issue up during our next engineering sync

Actually, it looks like this sub-module doesn't even use this library at all, but it still declares it.
If you search the repo you can see that the People Picker module is the only one that uses it.
This should have been removed in #93 but was missed.
So you can simply remove the dependency from this module

PR up in #305

tokeautomcomplete dependency was removed as part of PR #305. Hence closing this issue


tor-browser-bundle-bin: 9.0 -> 9.0.1

Motivation for this change
Version update. Tested with 32 and 64bit architecture.
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS (32 and 64bit)
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

Notify maintainers
cc @offline @matejc @doublec @thoughtpolice @joachifm

Thank you


Feature/3d sample project

Implemented another sample project showcasing how to render a 3d unit cube

@Orangeyness @jacobtoye @greygeek @haeal or whoever feels responsible. Any update on this?


messages: Add marking messages as unread feature.

Issue
Development
Debugging problems with messages often requires numerous unread messages in the realm, e.g. when checking unread counts or scrolling through 400+ unread messages (400 is the current batch size) to test the frontend.
This poses a problem for the developer - they have to either generate the messages or manually mark them as unread.
We would like to have a simple interface for marking the last N messages as unread, moving back the pointer and reloading the page.
User experience
Some users have been reporting accidentally marking messages as read and a feature to revert this action could benefit them. Example suggested use case:

From a UI perspective, either a "really mark as read?" for threads with greater than a threshold, or "unmark as read" that can work for, say, 5 minutes if more than a threshold of things were read, perhaps.

MVP
In the most basic case, a simple UI for providing the ID for the last message read in the realm would be enough.
Use cases
Development

testing frontend features for loading multiple messages and possible UI glitches
testing unread message counts

User experience

reverting the accidental "mark all as read" action in a realm/stream
marking topics as unread as a reminder to read them

Further development
Possible enhancements to the MVP:

marking all messages in realm as unread for development purposes
marking the last N messages in a realm/stream as unread
marking the messages from the past N hours as unread
reverting the last "mark all as read" action without specifying the message ID

Marking messages as unread can be done with the update_message_flags endpoint; the only tricky thing is to manage races that would cause the messages to be immediately marked as read again (primarily through the pointer being ahead of the newly marked as unread messages).
I think it's fine to reload the browser when we do this; so the issues are primarily going to be around making sure that we move the pointer back (the API may not allow this; I'd add a force flag to enable doing so) and dealing with invariant problems caused by this.
So, we'll probably want to do some sequence like:

Send the users's pointer back in the local browser
Send the new further-back pointer to the server
Update the message flags remotely
Reload

This may be a fairly involved feature to get right for general user, but for development it's probably a bit simpler (in that we could e.g. do it via management command when a browser isn't running).

It would be great to have this feature also simply for users.
Slack has Alt+click, which marks messages as unread.
This is great for when you started to read a long message, but then realise you don't actually have time to read it fully.

Just a quick note -- a helpful option for folks who want to come back to a thread later is to star the message and use this setting: https://zulipchat.com/help/star-a-message#display-the-number-of-starred-messages to have starred messages displayed as unread.

In progress

The server side of this was merged as #21252, along with web support for handling when messages get marked as unread.
The web client has an open PR #21477 for UI to actually mark messages unread. The mobile client has an open PR zulip/zulip-mobile#4790 to handle stream messages and some PMs getting marked unread, and issues zulip/zulip-mobile#5361 and zulip/zulip-mobile#5364 to track the remaining pieces.

In manually testing https://github.com/zulip/zulip-mobile/pull/4790 just now (using the UI added in https://github.com/zulip/zulip/pull/21477 ), I found a bug that will need to be fixed before we can roll this feature out. I think the server's behavior here is wrong and should be fixed (even though we might simultaneously go for a workaround in the client.)
Reported at #22164 with details.

In starting to implement the mobile UI for going and marking messages unread (zulip/zulip-mobile#5364), I found a limitation in the existing server API:
https://chat.zulip.org/#narrow/stream/378-api-design/topic/mark-as-unread.20request/near/1431139
which manifests as a bug in the existing draft implementation #21477 for web. Implementing the feature correctly would be cumbersome to do with the existing API.
So I think we'll want to extend the API to better support this feature before completing it. Filed #22893 for that, and see also the chat thread above.

For those on this ticket for 5 years:
In the merged PR #21477, what's the UI for marking as unread, is it Alt+click, or something else?

You can use the "Mark as unread from here" option in the three-dot message menu, or the SHIFT+U keyboard shortcut. Documentation coming soon (#23052)!


feat: Add compiler toolchain for developers

This is a proposal for a programming language compiler front/middle end, also including the POC (Proof of concept) code.
Only the VSCode will be supported in this POC.
FEATURES:

[x] Support for syntax highlighting
[x] Support for code completion
[x] Support for code linting
[x] Support for code formatting

Follow features move to another pull request:

Support for code navigation
Support for code refactoring
Support for vulnerability detection

Proposal: A001: Compiler toolchain for developers

/wip

/hold
hold to next week.

/unhold

/cc @coanor

/lgtm


Change Alignment to AlignmentGeometry

Although in the readme the align parameter are described as AlignmentGeometry?, they're for real are just Alignment. This makes it impossible to use this package in applications that support RTL languages.

Great news, @pdenert
We've released a new version of animate_gradient that should resolve all the issues you've reported.
Thanks for your patience and valuable suggestion.
Happy coding! 🎉🚀


[Feature] support alter table comment

What type of PR is this：

[ ] BugFix
[ ] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Which issues of this PR fixes ：

Fixes #20825
Problem Summary(Required) ：

Support alter table comment through:
ALTER TABLE t1 COMMENT = 'New table comment';

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr will affect users' behaviors
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[ ] I have checked the version labels which the pr will be auto backported to target branch

[ ] 3.0
[ ] 2.5
[ ] 2.4
[ ] 2.3

I think the syntax should be alter table xxtable modify comment "" we need a verb before the comment

@imay
https://dev.mysql.com/doc/refman/8.0/en/alter-table.html
This is the standard syntax of mysql, mysql does not support
alter table xxtable modify comment ""

But I think in addition to this grammar, other grammars should also be supported, for example:
alter table test_table comment 'New table comment';

The user omitted the wording of =

Got it, thank you.

@imay https://dev.mysql.com/doc/refman/8.0/en/alter-table.html This is the standard syntax of mysql, mysql does not support
alter table xxtable modify comment ""

But I think in addition to this grammar, other grammars should also be supported, for example:
alter table test_table comment 'New table comment';

The user or BI may omitted the wording of =

thank you

@mofeiatwork You need to modify the documentation.


Typo in PersianDateTimeFormatInfo

The correct name of the first month in Persian Calendar is "فروردین"  but the current letter is "فررودین"

Fixed in 1.10.x. Thanks for pointing out.

Makes no difference to me ;)


Sentry responded with an error: <urlopen error unknown error (_ssl.c:2788)> (url: https://app.getsentry.com/api/XXXXX/store/)

Has anyone else seen this?
Ubuntu 18.04 LTS, Python 3.6, Sentry 6.9.0. It's a Django app being served by uwsgi.
  File "/usr/lib/python3.6/urllib/request.py", line 1318, in do_open
    encode_chunked=req.has_header('Transfer-encoding'))
  File "/usr/lib/python3.6/http/client.py", line 1239, in request
    self._send_request(method, url, body, headers, encode_chunked)
  File "/usr/lib/python3.6/http/client.py", line 1285, in _send_request
    self.endheaders(body, encode_chunked=encode_chunked)
  File "/usr/lib/python3.6/http/client.py", line 1234, in endheaders
    self._send_output(message_body, encode_chunked=encode_chunked)
  File "/usr/lib/python3.6/http/client.py", line 1026, in _send_output
    self.send(msg)
  File "/usr/lib/python3.6/http/client.py", line 964, in send
    self.connect()
  File "/home/xxx/xxx/.ve/lib/python3.6/site-packages/raven/utils/http.py", line 38, in connect
    sock, ca_certs=ca_certs, cert_reqs=ssl.CERT_REQUIRED)
  File "/usr/lib/python3.6/ssl.py", line 1158, in wrap_socket
    ciphers=ciphers)
  File "/usr/lib/python3.6/ssl.py", line 745, in __init__
    self._context = SSLContext(ssl_version)
  File "/usr/lib/python3.6/ssl.py", line 391, in __new__
    self = _SSLContext.__new__(cls, protocol)
ssl.SSLError: unknown error (_ssl.c:2788)

Thanks,

Tried editing /home/xxx/xxx/.ve/lib/python3.6/site-packages/raven/utils/http.py to
cert_reqs=ssl.CERT_NONE

Got exactly the same error.

If I downgrade Raven to 6.7.0 it works! Still good to get an answer to this tho.

Hey, there is nothing in git diff 6.7.0..6.9.0 that would explain this change. Are you sure that no other (system) package got upgraded just when you downgraded raven-python? The only situation I know of where this error occurs is when some system certificate files are in the wrong format or malformed.
Is this an issue you face 100% of the time with 6.9.0 or only randomly?

I just went to double test and noticed a pattern - it might be something in our deploy system. If I restart the server manually after a deploy it works fine. I'll close for now, and re-open if I can identify a better problem report. Thanks.


Technical Error occured - Retry Transaction

Hello,
When i am trying to pay using Test Cards then i am receiving this error. Please check the screen shot.
So what should i do?
Update:
Its working in USD currency. but i want it to work in Egyptian currency i.e EGP.
Any solution?

@VivekMakwana24 Please send mail your merchant id to<EMAIL_ADDRESS>


What would be the cause if my 'truffle test' gives nothing in return?

So npm install and truffle compile are fine, but with truffle test, the command line just gives me nothing:

Even though this question dates back to August 2019 perhaps my response is helpful to someone.
For me this was usually related to having a wrong port configured in my truffle.js / truffle-config.js file. The default might be port 8545 (which is the default for the ganache-cli) but if you are using the ganache-gui, you need to change it to e.g. 7545 (which is the default for the ganache-gui).
An example is illustrated in the truffle documentation, where you can also find further information on the config file.
module.exports = {
  networks: {
    development: {
      host: "<IP_ADDRESS>",
      port: 8545,
      network_id: "*" // Match any network id
    }
  }
};

Even though this question dates back to August 2019 perhaps my response is helpful to someone.
For me this was usually related to having a wrong port configured in my truffle.js / truffle-config.js file. The default might be port 8545 (which is the default for the ganache-cli) but if you are using the ganache-gui, you need to change it to e.g. 7545 (which is the default for the ganache-gui).
An example is illustrated in the truffle documentation, where you can also find further information on the config file.
module.exports = {
  networks: {
    development: {
      host: "<IP_ADDRESS>",
      port: 8545,
      network_id: "*" // Match any network id
    }
  }
};

Inspired from teaching this project and the questions I got, I started a company to make it easy for developers to get started. Check out DappStarter which incorporates much of what went into building the FlightSurety project.

Inspired from teaching this project and the questions I got, I started a company to make it easy for developers to get started. Check out DappStarter which incorporates much of what went into building the FlightSurety project.


Truncating URI

I have a simple recipe:
apt_repository "node.js" do
uri "https://deb.nodesource.com/node_4.x"
components ['main']
distribution 'trusty'
action :add
end
However, when it writes out my node.js.list file it is truncating the address.

file[/etc/apt/sources.list.d/node.js.list] action create
- update content in file /etc/apt/sources.list.d/node.js.list from 46c7e1 to 00e912
--- /etc/apt/sources.list.d/node.js.list  2016-03-23 16:31:19.402651999 +0000
+++ /etc/apt/sources.list.d/.chef-node.js.list20160323-7734-11rqdep       2016-03-23 18:37:01.762651999 +0000
@@ -1,2 +1,2 @@
-deb     https://deb.nodesource.com/node trusty main
+deb     "https://deb.nodesource.com/node" trusty main

This is pointing to the wrong version of node because of this. I need the 4.x version not the 0.x version that it keeps correcting me to.

Open mouth insert foot. It appears the nodejs cookbook is overwriting my apt settings.


draft: add high level autoware design document

Signed-off-by: mitsudome-r<EMAIL_ADDRESS>

@kenji-miyake created a PR for dict words: https://github.com/tier4/autoware-spell-check-dict/pull/255
But I think we should make this process simpler. Like, instead of just complaining, it should also present an option to add to dictionary.

@xmfcx I understand. But I think it has a low priority since it's actually an optional CI check. (That is explained in the documentation site.)


Update scalafmt-core to 3.3.0

Updates org.scalameta:scalafmt-core from 3.2.2 to 3.3.0.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scalameta", artifactId = "scalafmt-core" } ]

labels: library-update, early-semver-minor, semver-spec-minor, commit-count:1

Superseded by #250.

Superseded by #250.


Support for specifying physical pixels dimensions

This adds support for writing the pHYs (Physical pixel dimensions) chunk to encoded PNGs
(see https://www.w3.org/TR/PNG/#11pHYs).
The dimensions can be specified in either pixels per meter (which is what the PNG format uses) or in dots per inch (which is commonly used for images).

Codecov Report

Merging #6 (900c0f3) into develop (4a883ad) will increase coverage by 0.58%.
The diff coverage is 97.29%.

@@              Coverage Diff              @@
##             develop       #6      +/-   ##
=============================================
+ Coverage      89.05%   89.63%   +0.58%
- Complexity       139      151      +12
=============================================
  Files             17       18       +1
  Lines            548      579      +31
  Branches          45       46       +1
=============================================
+ Hits             488      519      +31
  Misses            41       41
  Partials          19       19

Impacted Files
Coverage Δ
Complexity Δ

src/main/java/com/pngencoder/PngEncoder.java
58.06% <87.50%> (+2.89%)
9.00 <6.00> (+1.00)

src/main/java/com/pngencoder/PngEncoderLogic.java
91.54% <100.00%> (+0.92%)
8.00 <1.00> (+2.00)

.../pngencoder/PngEncoderPhysicalPixelDimensions.java
100.00% <100.00%> (ø)
9.00 <9.00> (?)

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 4a883ad...900c0f3. Read the comment docs.


Update send-transaction-dialog.html

Increased space for 'Send all' button, needed for multiple language support

@j-a-m-l Done, the send all button is not static like the visibility button due to the multi language support, this seems to be the cleanest fix.


kondo: 0.4 -> 0.5

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/tbillington/kondo/releases.
meta.description for kondo is: "Save disk space by cleaning unneeded files from software projects"
meta.homepage for kondo is: "https://github.com/tbillington/kondo"
meta.changelog for kondo is: ""
Updates performed

Rust version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
/nix/store/dp25qjwpw2nz4qxvmsx81fvb3wv9zgg7-kondo-0.5/bin/kondo passed the binary check.
1 of 1 passed binary check by having a zero exit code.
0 of 1 passed binary check by having the new version present in output.
found 0.5 with grep in /nix/store/dp25qjwpw2nz4qxvmsx81fvb3wv9zgg7-kondo-0.5
found 0.5 in filename of file in /nix/store/dp25qjwpw2nz4qxvmsx81fvb3wv9zgg7-kondo-0.5

Rebuild report (if merged into master) (click to expand)

4 total rebuild path(s)

2 package rebuild(s)

2 x86_64-linux rebuild(s)
2 x86_64-darwin rebuild(s)

First fifty rebuilds by attrpath
docbookrx
kondo

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/dp25qjwpw2nz4qxvmsx81fvb3wv9zgg7-kondo-0.5 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A kondo https://github.com/r-ryantm/nixpkgs/archive/46324ebbdd61db3285c38c2e724975fce19a8158.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/dp25qjwpw2nz4qxvmsx81fvb3wv9zgg7-kondo-0.5
ls -la /nix/store/dp25qjwpw2nz4qxvmsx81fvb3wv9zgg7-kondo-0.5/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

kondo

Maintainer pings
cc @Br1ght0ne for testing.

Result of nixpkgs-review pr 154593 run on x86_64-linux 1

1 package built:

kondo


Utilize Typescript for GDPR Annotations

Currently GDPR annotations are created with hand typed JSON comments. Replacing these with typescript would allow for stricter compliance in our function calls and prevent simple errors such as typos.

We can use TS or comments now. Closing.


docs: correct name of local config module in comments

The docs mention the local config module name as local_config
but the code expects module named caravel_config. This fixes
docs to match what code expects.

Coverage decreased (-0.2%) to 81.268% when pulling ff9b42348fdfb433d0a53a1ba5fda0933f061ab7 on phanimahesh:fix-config-docs into a9fd2271ddb1e92d71b07812fc1a5839dd4d7238 on airbnb:master.

Coverage decreased (-0.2%) to 81.268% when pulling ff9b42348fdfb433d0a53a1ba5fda0933f061ab7 on phanimahesh:fix-config-docs into a9fd2271ddb1e92d71b07812fc1a5839dd4d7238 on airbnb:master.

Rebased on master

Coverage decreased (-0.2%) to 81.268% when pulling fb9dcf1a129d362131cd6b0beddafc470bfc35f7 on phanimahesh:fix-config-docs into a9fd2271ddb1e92d71b07812fc1a5839dd4d7238 on airbnb:master.


docs(guide): convert /tutorials/connect/authenticating.md to rst

Fixes NODE-2199
Description
What changed?

Added SCRAM-SHA-256
Removed links to mongocr b/c those just redirect to SCRAM on docs.mongodb.com
Added notifications about SCRAM-SHA-1 and SCRAM-SHA-256 being defaults, and MONGO-CR being deprecated.

Are there any files to ignore?

@mongomoe I think i fixed all the string interpolation / quote issues. I also redid all the connect logic to go with the format from #2182


Added PlayerMock#simulateInventoryClick

Description
Added PlayerMock#simulateInventoryClick
Checklist
The following items should be checked before the pull request can be merged.

[x] Unit tests added.
[x] Code follows existing code format.

Info on creating a pull request

Make sure that unit tests are added which test the relevant changes.
Make sure that the changes follow the existing code format.

For maintainer
When a PR is approved, the maintainer should label this PR with release/*.

If the PR fixes a bug in MockBukkit, update the patch version. (if the current version is 0.4.1, the new version should be 0.4.2)
If the PR adds a new feature to MockBukkit, update minor version. (if the current version is 0.4.1, the new version should be 0.5.0)

Note that a PR that fixes an UnimplementedOperationException should be considered a new version and not a bugfix.

My IDE auto formatted it according to: https://github.com/MockBukkit/MockBukkit/blob/v1.18/.editorconfig tho.
If your using a different code style, u should probably change that file :shrug:

Didn't see that I'm sorry. Seems correct then.


Centered columns

Like this
http://www.minimit.com/articles/solutions-tutorials/bootstrap-3-responsive-centered-columns
http://www.minimit.com/demos/bootstrap-3-responsive-centered-columns

Is there a way to center content (columns) using the currently Chico and Mesh codes? Thank you.

@cRube, currently Mesh doesn't provide that feature —you could easily extend it for your use-case.
Keep in mind that Chico and Mesh are two separated projects. Honestly I don't see this feature being added to Mesh in the near future, since it relies entirely on floats. We might be able to accomplish something similar if https://github.com/mercadolibre/mesh/issues/15 gets implemented.
I'll be closing this, but feel free to open an issue in Mesh if you think that we should discuss this further.
Thanks!


Fix clicking out of game over modal to unplayable state

Really simple fix to prevent the user from putting the game in an unplayable state by clicking out of the game end modal

hm, so I have at times wanted to close the modal after to look again at what words I put. then to play again I refresh. maybe there's a different fix here

Hmm good point. Maybe something like adding another play again button that shows on the main screen if the modal is closed? Also potentially add an X button to the modal to make the close functionality more obvious

oops... still learning how to use git correctly. those should have been a different PR, I'll try to fix it


Change M2Expansion IFS dependancy to IFS-Core

Looks reasonable, thank you for opening a PR about it :)


How to best way to registry scheduler in 3rd packages

I need registry scheduler in packages, how best way to make this?

Please ask on the forums. :)

tk @GrahamCampbell


A depoly error will be reported  After run the init.sh due to  the _posts dir won't be keep

Checklist

[X] I have read and accepted the contributing guidelines.

How did you create the site?
Built from jekyll-theme-chirpy
Describe the bug
After run the init.sh, the _posts file all be deleted,  and the dir won't be keep, thus will report a depoly error.
error msg:
Get Pages site failed. Please verify that the repository has Pages enabled and configured to build using GitHub Actions, or consider exploring the enablement parameter for this action.
suggestion: add a .placehoder file in the _post dir to avoid this error
Steps To Reproduce
run the init.sh
Expected Behavior
After run init.sh, a basic blog  web can be preview by access the url.
Environment

Ruby:
Jekyll:
Chirpy:

Anything else?
No response

http://www.catb.org/~esr/faqs/smart-questions.html


fix endless loop in dir_compact and nullptr access in lfs_dir_fetchmatch

-> introduce relocateCounter in dir_compact to catch infinite loop when
lfs has no free space left anymore
-> call lfs_dir_fetchmatch with ftag=-1 in order to set the invalid bit
and never let the function match a dir
Fixes:
https://github.com/ARMmbed/littlefs/issues/177
https://github.com/ARMmbed/littlefs/issues/184

Hi @Mohawi, thanks for creating a PR. Sorry it's taken so long for me to get to this.
I'm not sure yet the best way to handle for the possible infinite-loops. For now I've split out the null-pointer exception into a separate PR and rebased it onto master:
https://github.com/ARMmbed/littlefs/pull/337
It looks good to me, so I will merge it when I can

Ok! I've now added proper loop checking in https://github.com/ARMmbed/littlefs/pull/372. Hopefully this takes care of the issues you've ran into.
Sorry about butchering your PR like this. Thanks again for the contribution. It's very useful to know what issues/fixes everyone is looking for.


⚠ Spryr has degraded performance

In 8a76d4a, Spryr (https://spryr.com) experienced degraded performance:

HTTP code: 200
Response time: 7554 ms

Resolved: Spryr performance has improved in 0b27f54 after 8 minutes.


Update wording for Power Platform apps

Teams apps store has been updated to show Microsoft Power Platform apps differently than shown in this docs. There are two options - Built for your org and Built with Power Platform (instead of Built for Your Organization Name > Built by your colleagues). This information should be updated to align with client changes. Related to MC298032 and https://www.microsoft.com/en-ww/microsoft-365/roadmap?filters=&searchterms=86186.

@ashishguptaiitb PR has been copyedited and is ready for final review, could you please check and merge? Thanks!

Thanks @scanum and @kellybos , for sharing this doc bug.
I am updating the docs for this change via another PR. Once the change is published live, I shall inform here and close this PR. Reason to not approve this PR is that this PR does not capture all such instances + it mentions Built by Power Platform at some places that is incorrect. No issues at all and again, thanks for sharing this doc bug. It should get fixed in the next few hours.

The published article is updated now. Closing this issue. Thanks again, for sharing.


Fix layer-related issues for oci_rebase_image

Specifically:

Make sure we're using platforms.OnlyStrict instead of platforms.Only when constructing the platform matcher, because with Only, arm/v8 will also match the other arm/* platforms, and, more immediately relevant, amd64 will match the first item in the index with either amd64 or 386. Which is...not what we want, obviously.
When generating the build files in GenerateBuildFilesCmd, don't add oci_blobs and traverse children for blobs which we've already added an oci_blob and traversed for. If there's an identical layer in more than one platform's image, and we've got shallow set to false (so that we're pulling layers as well as configs/indexes/manifests), the generated BUILD.bazel will end up invalid if we don't do this, because there'll be duplicate oci_blob rules (and duplicate rules for children, but in practice I didn't run into that because the only duplicates I hit in testing were layers).
Add the OCI base image name/digest annotations to the layers we're copying from the original image to the rebased image. CopyContent is leveraging those annotations to determine when we can avoid actually pushing a layer to the registry in favor of using Mount, and we're already doing that for the base image layers in AppendLayers.

Note that there are still some interesting/annoying issues or quirks, and not just with oci_rebase_image:

With either oci_image or oci_rebase_image followed by an oci_push, if you're pushing to a different registry than the image(s) containing the pre-existing layers in the new image, you'll get a confusing error (failed to push child content to registry: failed to create reader from ingestor: not found). I think this is because the Mount fails, because you can't cross-repo mount a blob from a different registry host, and oci_push doesn't know where to find the layers from the pre-existing image(s) locally. I'm still trying to unravel that one, but am guessing it's happening because the layout passed to ocitool push doesn't contain the actual layers from the pre-existing image(s) and I'm not yet sure how to solve that.
With oci_rebase_image, the oci_pull for the original image must have shallow = False, or RebaseImage's call to AppendLayers can't locate the diff ID for the layers from the original image, for the fairly obvious in retrospect reason that...we're not pulling the layers unless we pass shallow = False to oci_pull. Oops.

cc @jprobinson - given the messiness so far, I'm not sure if it doesn't just make sense to remove oci_rebase_image and give up on this. Needs more thought.


Latest version is named incorrectly

code-server version: 1.1156-vsc1.33.1

OS Version: linux-x64, all cloud providers

Description

Throughout your documentation you write code-server-{version} but the latest linux version is named code-server{version}. There is no hyphen between the word server and the version string.
Steps to Reproduce

Try to visit https://github.com/cdr/code-server/releases/download/1.1156-vsc1.33.1/code-server-1.1156-vsc1.33.1-linux-x64.tar.gz in your browser and it will fail to download
Instead visit https://github.com/cdr/code-server/releases/download/1.1156-vsc1.33.1/code-server1.1156-vsc1.33.1-linux-x64.tar.gz and it will download!

Finders
@iyengarvinay and @ckathleen

This just seems like a documentation mistake. I looked back through a page of releases and none of them have a hyphen after code-server before the version. The GitHub releases are drafted for us by CI, which names and uploads the binaries for us, so it makes sense that it probably hasn't changed.
Would appreciate a PR that fixes the documentation. I think the only place that we say code-server-{version} is in our doc/admin/install/* guides.

Found a really old PR that seems to address this: #730. Asked for PGP signatures so we can merge it.

Okay, happy to submit a new PR tonight if that would help.
On Sat, Jul 6, 2019 at 11:59 PM Dean Sheather<EMAIL_ADDRESS>wrote:

Found a really old PR that seems to address this: #730
https://github.com/cdr/code-server/pull/730. Asked for PGP signatures
so we can merge it.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/cdr/code-server/issues/833?email_source=notifications&email_token=ABEE27AEEFU5G2QDFCUVNBTP6FSYHA5CNFSM4H6Q6EX2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODZLEFNY#issuecomment-508969655,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABEE27FRGOW6WEOR6GZSVR3P6FSYHANCNFSM4H6Q6EXQ
.

Merged.


Failure in test_miri.py::test_against_test_data

________________________________________________________ test_against_test_data ________________________________________________________
siaf = <pysiaf.Siaf object Instrument=miri >, verbose = False
def test_against_test_data(siaf=None, verbose=False):
    """MIRI test data comparison.

    Mean and RMS difference between the instrument team computations
    and the pysiaf computations are computed and compared against
    acceptable thresholds.

    """
    if siaf is None:
        # Try to use pre-delivery-data since this should best match the source-data. If no data there, use PRD data
        try:
            pre_delivery_dir = os.path.join(JWST_DELIVERY_DATA_ROOT, instrument)
            siaf = Siaf(instrument, basepath=pre_delivery_dir)
        except OSError:
            siaf = Siaf(instrument)

    else:
        # safeguard against side-effects when running several tests on
        #  a provided siaf, e.g. setting tilt to non-zero value
        siaf = copy.deepcopy(siaf)

    x_test, y_test, v2_test, v3_test = mirim_siaf_testdata.siaf_testdata()

    aperture_name = 'MIRIM_FULL'
    aperture = siaf[aperture_name]

    # v2_pysiaf, v3_pysiaf = aperture.det_to_tel(x_test, y_test)
    # x_pysiaf, y_pysiaf = aperture.tel_to_det(v2_test, v3_test)
    v2_pysiaf, v3_pysiaf = aperture.sci_to_tel(x_test, y_test)
    x_pysiaf, y_pysiaf = aperture.tel_to_sci(v2_test, v3_test)

    t = Table([x_test-x_pysiaf, y_test-y_pysiaf, v2_test-v2_pysiaf, v3_test-v3_pysiaf],
              names=('delta_x', 'delta_y', 'delta_v2', 'delta_v3'))

    if verbose:
        print('')
        t.pprint(max_width=-1)

    absolute_tolerance = 0.04

  assert_allclose(x_test, x_pysiaf, atol=absolute_tolerance)

E       AssertionError:
E       Not equal to tolerance rtol=1e-07, atol=0.04
E
E       Mismatched elements: 15 / 15 (100%)
E       Max absolute difference: 1.840769
E       Max relative difference: 0.01433061
E        x: array([326.13, 693.5 , 516.5 , 953.18, 681.75, 409.81, 137.65, 928.52,
E              658.11, 387.37, 116.34, 904.64, 634.88, 365.  ,  94.77])
E        y: array([327.841595, 695.124727, 518.257992, 954.734981, 683.374363,
E              411.606429, 139.490769, 930.03035 , 659.758194, 389.192884,
E              118.031463, 906.054437, 636.468316, 366.667649,  95.986934])
test_miri.py:75: AssertionError
________________________________________________________ test_against_test_data ________________________________________________________

I don't understand this; I ran the test_miri.py script when I ran generate_miri.py and didn't encounter these errors in comparing against MIRI test data.  The values in the error log above look like outdated values from a previous version of pysiaf, how was this test being run?  Did it use the new values in mirim_siaf_testdata.py that were updated with the PR?

That's what we're trying to figure out. We started getting all these errors showing up preparing to due a pysiaf release following the procedure at https://innerspace.stsci.edu/pages/viewpage.action?pageId=107324779 starting at "After the PRD is Released."
The tests pass fine initially, but as soon as we install the actual PRD data the tests start failing.

I may have fixed it, but I'm too pessimistic to declare victory just yet...


Trigger manually

Is there an option to trigger the edit manually from another button for example?
Thanks for your reply!

I have not yet developed it, but I can included this feature in the roadmap.

that would be great thanks!

This should be fixed in the last version (0.2.0-alpha.0). With the change of name, you should do an npm i @qontu/ngx-inline-editor and update your imports 👍


Rename "outputs" to "formats"

This was confusing as there is an option to output to a file as well.
Signed-off-by: Justin Cormack<EMAIL_ADDRESS>cc @deitch I think you suggested this...

I think you suggested this

Thank you, yes. It was confusing.
Are the README/--help updated?

The --help is generated automatically, yes. I have another PR that updates LinuxKit which updates all the docs there.


Crashed without internet

My app crashed if I turned off the internet.
2016-07-26 17:55:53.311 [warn][tid:com.facebook.react.JavaScript] Error: TypeError: Network request failed
2016-07-26 17:55:53.317 [error][tid:com.facebook.react.RNMixpanelQueue][RCTModuleMethod.m:470] RNMixpanel.track was called with 2 arguments, but expects 1.                   If you haven't changed this method yourself, this usually means that                   your versions of the native code and JavaScript code are out of sync.                   Updating both should make this error go away.
2016-07-26 17:55:53.336 [error][tid:com.facebook.react.JavaScript] Module RCTLog is not a registered callable module.
2016-07-26 17:55:53.339 [fatal][tid:com.facebook.react.RCTExceptionsManagerQueue] Unhandled JS Exception: Module RCTLog is not a registered callable module.

@Darwinium Hey don't think this is a network related error, in this library unlike the Mixpanel SDK, you have two methods to track, one is track itself which takes only the event name and the other one, trackWithProperties is probably what you should be using here. Hope it helps.


Input default value handling is not consistent

Steps to reproduce the issue
Consider the following design:
//
// a should have the value 1'b0 if it is not connected to a driving value
//	by the instantiating module.  Otherwise, it should have the
//	value given by the instantiating core
module mcvrinit(input wire a = 1'b0, output wire b);
	assign	b = a;

	// This should succeed if mcvrinit is the top-level
	// within the proof
	always @(*)
		cover(b != 1'b0);

endmodule

module parent(output wire b);
	mcvrinit initdefault(.b(b));

	// This should always succeed
	//	Since initdefault.a has not been bound, it should
	//	retain its initial value
	always @(*)
		assert(!b);
endmodule

module parentdef(output wire b);
	mcvrinit initdefault(.a(1'b1), .b(b));

	// This should always suceed
	//	Since initdefault.a has now been bound to one,
	//	it should override any initial value
	always @(*)
		assert(b);
endmodule

and sby file
[tasks]
prf
cvr
prf2

[options]
prf: mode prove
cvr: mode cover
prf2: mode prove
depth 3

[engines]
smtbmc

[script]
read -formal initdefault.sv
prf: prep -top parent
cvr: prep -top mcvrinit
prf2: prep -top parentdef

[files]
initdefault.sv

Expected behavior
I expect all three tests and their associated properties to pass

The cover statement, checking whether b (really a) can be non-zero.  When mcvrinit is the top level, the default input value should be 'x in spite of whatever default is given.  When parent is the top level, this cover statement should fail.  (The failure is not checked by the sby file above)

The assertion in parent should also pass.  In this case, the parent or instantiating module is not overriding the value of a, so it should be equal to one and hence so should b.

The assertion in parentdef should also pass.  In this case, the default behavior has been overridden by the parent, and so b==1 should always be true.

See IEEE Std 1800-2012 (the 2012 SVA standard), section <IP_ADDRESS> "Default port values" for more information regarding the expected behavior.
Actual behavior
Yosys (without Verific) assumes a == 0 independent of the context.  As a result, the cover statement fails, the parent assertion passes whereas the parentdef assertion fails.

Implementation strategy: Turn default assignments into special attributes, then use that attributes in hierarchy to determine constant driver values for unconnected ports in instances.


HTTP Status Codes

I don't want to make requests for 404 and so on again because my IP address can be blacklisted because of unnecessary requests. Generally, if we know HTTP status codes, we can act more consistent.

Your IP isn't blacklisted, Jikan's is rate limited temporarily. This returns HTTP Status 429.
Check the headers, HTTP status is there.

Closing this discussion. If you wish to reply, please re-open this issue.


GCM ciphers not built on OS X even when installed OpenSSL supports them

Making this bug report on request of https://github.com/rfjakob/gocryptfs/issues/15#issuecomment-169510632
OS X no longer ships openssl as of OS X 10.11, and as such an installed version of openssl is probably new enough to support GCM, but they are not built in this project so long as the platform is darwin ala
https://github.com/spacemonkeygo/openssl/blob/master/ciphers_gcm.go#L15
with no checks made to see if the installed openssl provides support or not.
Personally I have OpenSSL 1.0.2e installed on OS X 10.11.2 and removing the "!darwin" from that linked file allows these bindings to continue to build just fine

I'm going to look at the linked pull request in this issue and see if it fixes it. Closing this in favor of the pull request tracking.


Add Tween sequencing/chaining

Godot version:
3.0.2 Stable
OS/device including version:
Windows 10
Issue description:
There currently doesn't seem to be an easy way to chain various tweens together. Having a toolset similar to GSAP would be an amazing way to setup and manage chains of tweens.

Is easy to chain sequences with a single tween this way:
tween.interpolate_property(
  node,property1,start_value1,end_value1,
  time1,trans_type,ease_type)
tween.interpolate_property(
  node,property2,start_value2,end_value2,
  time2,trans_type,ease_type,
  time1) #delayed by time1 seconds
tween.start()

@eon-s Perhaps for simple tweens, but things start to get hairy if you need to add/remove tweens. Having to manage the delays isn't ideal.

GDScript's "yield to signal" can be used for that.
var tween = Tween.new()
tween.interpolate_property(node, property, start_value, end_value, time, trans_type, ease_type)
tween.start()
yield(tween, "tween_completed")
tween.interpolate_property(node, property2, start_value2, end_value2, time2, trans_type, ease_type)
tween.start()
yield(tween, "tween_completed")
# etc...

@karroffel any thoughts on how you could get a chain like that to loop? Turn on repeat seems to loop the last interpolated property.

@eligt go ahead :+1:

I'm assuming the preference would be to change the current Tween class to add the extra function and keep backwards compatibility, but is it vital to do so at this stage of 3.x?
I'm asking because I really liked cocos2d-x way's of handling this, not the best-performing but could be adapted (I'm not even sure performance is that important with tweening which is used for simple animations).
For reference, this is an example from an actual game I wrote in cocos2d-x:
Vector<FiniteTimeAction*> actions;

actions.pushBack(CallFunc::create(callbackStart));

if (effect == "move")
	actions.pushBack(EaseCubicActionOut::create(MoveTo::create(0.5, Point::ZERO)));
else if (effect == "fade")
	actions.pushBack(EaseCubicActionOut::create(FadeIn::create(0.7)));
else if (effect == "blackout")
{
	actions.pushBack(TargetedAction::create(m_screenFader, EaseCubicActionOut::create(FadeIn::create(0.6))));
	actions.pushBack(CallFunc::create(SLIDEMATCH_CALLBACK_0(GameScene::activateScreen, this, name)));
	actions.pushBack(DelayTime::create(0.1));
	actions.pushBack(Spawn::create(
		Sequence::create(
			DelayTime::create(0.3),
			//CallFunc::create(SLIDEMATCH_CALLBACK_0(GameScene::setActive, this, true)),
		nullptr),
		TargetedAction::create(m_screenFader, EaseCubicActionIn::create(FadeOut::create(0.5))),
	nullptr)
	);
	actions.pushBack(CallFunc::create(SLIDEMATCH_CALLBACK_0(Node::setVisible, m_screenFader, false)));
}

actions.pushBack(CallFunc::create(SLIDEMATCH_CALLBACK_0(GameScene::setActive, this, true)));
actions.pushBack(DelayTime::create(0.05));
actions.pushBack(CallFunc::create(callbackEnd));
actions.pushBack(CallFunc::create(tutorialCallback));

Sequence* seq = Sequence::create(actions);
screen->runAction(seq);

I like this approach because it creates readable code, which you know what is going to do, but at the same time it's quite powerful, it's very easy to create your own actions and ease functions.
Trying to get a sense for how much wiggle room there is when it comes to upgrading the Tween system.

I personally like current Tween system to be able to create multiple tweens with different timelines (duration and delay).
how about adding a new class like the Sequence in cocos2d-x?
it won't break compatibility and play multiple tweens sequentially.

Yes I wouldn't change the way animations are played, they'd still go through the Tween object, I was mostly referring to the way you create an individual action to be played.
Mostly, I was just wondering whether there'd be strong opposition into changing the current API in a way that might break some backwards compatibility. I'm not opposed to creating a new class, but I don't want to clutter and create confusion with a parallel system.
Anyway I'll try throw together a draft implementation this weekend and we can discuss it then, probably easier that way.

Just created a pull request with the changes, it was actually easier than I thought. Performance is not amazing as there's some recursion but it should be a problem unless you have thousands of tweens at the same time.
Pull request: https://github.com/godotengine/godot/pull/20934

Sample (functional) GDScript code looks like this:
https://github.com/eligt/godot/wiki/Tweening-sequences-and-spawners-sample-GDScript-code
Creates this resulting animation:
https://youtu.be/AVkn0ri0oHw

I think this is looking nice! I find that calling so many create_sequence() functions nested in each other is kind of confusing however. What about having create_sequence() return the sequence object and then having something like add_to_sequence() that let's you append to the end of the sequence?

Jumping in here as a cocos2d-x user, that adding some sort of sequencing for tweens is very important to me too! Save for the particles, every visual effect shown here used sequences of tween, similar in style to @eligt's code.
A nice side-effect of how cocos2dx does their animations and lets you override the delta time passed to each node, is that you can wrap an effect an an Ease/Tween, so if you added a MoveBy action, sequenced by a second action moving back, you could just wrap the entire sequence in a EaseBackOut, and have both nicely eased together. EaseBackOut::create(Sequence::createWithTwoActions(MoveBy::create(..), MoveBy::create(..))

Coming from Universal Tween Engine, I too find Godot Tween class very limited. Here is an example showing how easy (and scalable) it was working with sequential and parallel timelines in Scala (startUsingOurManager is a pimp method calling just start with global tween manager; just imagine it is start()):
    Timeline.createSequence()
    .pushCallback {
      _chestBurstAnimationInProgress = true
      continueButton.setDisabled(true)
    }
    .push(
      Timeline.createParallel()
      .push(chestAnimation.createTimeline())
      .push(UniversalTweenEngineHelper.createExecutionTween({sounds.chestCreak.play()}, 1.5f, "Chest Creak"))
    )
    .pushCallback {
      _chestBurstAnimationInProgress = false
    }
    .startUsingOurManager()

More basic example:
    Timeline.createSequence()
    .push(Tween.to(subtitlesWidget, TweenTypes.TYPE_OPACITY, subtitlesFadeAnimationDuration).target(1))
    .startUsingOurManager()

I really liked that I could abstract creation of tweens/timelines to methods and then later just chain them readably:
      Timeline
      .createSequence()
      .push(graduallyShowFromHiddenTimeline())
      .push(burnFromFullTimeline())
      .push(fuse.animateSwords())
      .pushPause(gameState.afterSwordDelay)
      .pushCallback {burningFuseFinished(); ()}
      .startUsingOurManager()

This PR https://github.com/godot-extended-libraries/godot-next/pull/50 seems to solve it.

Closing in favor of https://github.com/godotengine/godot-proposals/issues/514, as feature proposals are now tracked in the Godot proposals repository.


Changing basic templates.

Is there some way to modify basic registration/login/etc. templates by modifying .json file , not final .js/html files of final meteor app?

@2huk  currently, best you can is to put your templates into meteor-kitchen install dir/templates/blaze(or react)/bootstrap(or semantic or materialize)/pages/ name files for example my_login.html and then in .json set "template": "my_login"

@perak thanks for answer. I could do what i wanted by manualy changing templates. However i think its good idea to implement this basic pages fully using your code generator. Especially in examples to make them more demonstrative. Whatever you project is great help fo beginners like me. You're doing great work.


Update zone to us-central1-a to support query t2a machine type

to fix: #466 table gcp_compute_machine_type not support t2a machine type query

@ahlfors I'm closing this PR as I believe we have a better long term solution in https://github.com/turbot/steampipe-plugin-gcp/pull/480, but if you have any additional comments or questions, please add them in https://github.com/turbot/steampipe-plugin-gcp/issues/466, thanks!


Cannot  install BoringSSL-GRPC pod

Trying to build my project on Mac.
I have a BoringSSL-GRPC as a dependency of one of my pods.
So I'm trying to run:
$ pod install
And it takes forever with:
Installing BoringSSL-GRPC (0.0.2)
(Already spent 2-3 hours trying to build my project).

BoringSSL-GRPC pod is now 0.0.4. Something is not right in your project. We would need more information on your project to provide any further help I think.

Closing since we are not hearing response. We can reopen if necessary.


[CWS] introduce exec cross container in ebpfless

What does this PR do?
This PR introduce a procfs scan for the ebpfless probe. For each new process it generates an exec event.

Motivation

Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

/merge


Odunsi/homepage

Description
Please include a summary of the change and relevant motivation and context.
List any dependencies that are required for this change.
All pull requests should be related to open issues. Indicate the issue(s) bellow and remove this line:
Fixes #(issue number 1) #(issue number 2 if applicable)
How Has This Been Tested?
Please describe in detail how you tested your changes.
Include details of your testing environment, and the tests you ran to see how your change affects other areas of the code etc.
Screenshots (if applicable, else remove this line / section)
Checklist:

[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[x] My code follows the style guidelines of this project
[ ] I have added necessary inline code documentation
[ ] I have added tests that prove my fix is effective and that this feature works
[ ] New and existing unit tests pass locally with my changes

please fix quotes and other issues and commit again


How to update events in the chart by click ?

Hello everyone!
I'm use Chart.js - v.2.8.0
I tried to update events by click : Chart.defaults.global.events = ['click']; but no result.
https://jsbin.com/xupakasehi/edit?html,js,output

Not sure what you are trying to do. But you have only one dataset and you are checking for _datasetIndex === 1. Its 0 for first dataset. _index contains data index.

Hi kurkle, I'm sorry, this is confusing, I greatly simplified the example at https://jsbin.com/xupakasehi/edit?html,js,output
The main task is to add and remove a chart event programmatically by click on chart bar item.
Not at page load time with : options: {events : ['click']}

https://jsbin.com/woheleqube/edit?html,js,output


Fan test doesn't work properly in main loop

I moved fan test out of fan init and I can't remember why, but it is built assuming it will be there and now doesn't work.

I moved it out because it flashes the LED and that is now an async function, might need to make a sync flash led function for pre loop activities.