Fix null exception caused by kotlin "by lazy" in Godot.kt

On some device which do not have specified sensor will cause a NullPointerException like
Caused by: java.lang.NullPointerException: <get-mAccelerometer>(...) must not be null
at org.godotengine.godot.Godot.getMAccelerometer(Godot.kt:91)
at org.godotengine.godot.Godot.onResume(Godot.kt:495)
at org.godotengine.godot.GodotFragment.onResume(GodotFragment.java:280)
at androidx.fragment.app.Fragment.performResume(Fragment.java:3039)
at androidx.fragment.app.FragmentStateManager.resume(FragmentStateManager.java:607)
at androidx.fragment.app.FragmentStateManager.moveToExpectedState(FragmentStateManager.java:306)
at androidx.fragment.app.FragmentStore.moveToExpectedState(FragmentStore.java:112)
at androidx.fragment.app.FragmentManager.moveToState(FragmentManager.java:1647)
at androidx.fragment.app.FragmentManager.dispatchStateChange(FragmentManager.java:3128)
at androidx.fragment.app.FragmentManager.dispatchResume(FragmentManager.java:3086)
at androidx.fragment.app.FragmentController.dispatchResume(FragmentController.java:273)
at androidx.fragment.app.FragmentActivity.onResumeFragments(FragmentActivity.java:458)
at androidx.fragment.app.FragmentActivity.onPostResume(FragmentActivity.java:447)
at android.app.Activity.performResume(Activity.java:8023)

@armedpotato Did you mean to close this pull request? It seemed like a sensible fix to me.
PS: I recommend opening pull requests from a branch other than master on your fork. This way, you can keep your fork's master branch free of any changes and can maintain multiple pull requests at once.

@armedpotato Did you mean to close this pull request? It seemed like a sensible fix to me.
PS: I recommend opening pull requests from a branch other than master on your fork (while still targeting master on this repository). This way, you can keep your fork's master branch free of any changes and can maintain multiple pull requests at once.

I had create a new PR to pick the proper commit into it. So create this multi commit one.


Unable to install

hi all,
I can't install via luaRocks,
Error: No results matching query were found.

And got nothing on https://luarocks.org/search?q=loadRoffee&non_root=on

install torch http://torch.ch

i've already installed torch follow by the guides.
And I can install other rocks list on http://luarocks.org:8080/m/root, but install loadcoffee failed with 'no results maching'
Did I miss something?

You can install it manually by running 'luarocks install loadcaffe/loadcaffee-1.0.0.rockspec'.

@atomic @szagoruyko thank you for your help. I've solved the problem.

@PaniniGelato How?

ok you need to run torch-activate which is put in your ~/.profile automatically buy the torch installer.

@harvimt  I just followed atomic's guides to install it manually.

Thank you @atomic that helped me

For those still stuck, try:
git clone git@github.com:szagoruyko/loadcaffe.git
~/torch/install/bin/luarocks install loadcaffe/loadcaffe-1.0-0.rockspec

Maxko87,
After entering in the command git clone git@github.com:szagoruyko/loadcaffe.git
the output reads (running as sudo)
`Cloning into 'loadcaffe'...
Warning: Permanently added the RSA host key for IP address '<IP_ADDRESS>' to the list of known hosts.
Permission denied (publickey).
fatal: Could not read from remote repository.
Please make sure you have the correct access rights
and the repository exists.
`
What else would you recommend?

@jeremymarino try https protocol instead of git

@maxko87 Thank you this makes sense and worked perfectly!


Connnection to Siganalling Server is failing

https://webrtcweb.com:9559
Error in chrome - ERR_CONNECTION_REFUSED.
@muaz-khan Can you please check if the service is running ?

Fixed.

@muaz-khan Thanks for the prompt fix.
Really great work 👍


Comprehensions type class

For performing for comprehensions over higher kinded monad types.

Released in 2.0.0-FINAL


Add options for file and directory filters

lgtm.

@robskillington can you update the help text and README ? ( https://github.com/uber/uber-licence/blob/master/bin/licence#L38-L47 ). Feel free to do so on master.

Sure thing


Added custom intensity-mask mapping in Py interface

@friskluft Thanks for addressing the comments. One final remark: Can we rename the arguments from int_fnames to intensity_files and same with masks? More Pythonic that way. Make sure to update the docstrings and documentation where necessary. Thanks!


Ability to share image quotes

Summary
With this option users can share custom images from quotes.
Details
Needs to be a square image, with labels for the quote, book title and author.
Currently when select share from selection menu it will share only text, needs to add two options, "Share image quote" and "Share text quote".
Notes
Public api for customization, logo and background images.
Proposed design

very nice


set configuration file to avoid using a wrong one

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.0 out of 2 committers have signed the CLA.:x: chenjunxu:x: nic-chenchenjunxu seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Remove the host header before forwarding

With some hosts behind shared TLS (using SNI), keeping the Host header was causing the target host to be misidentified.
I also observed issues related to TLS verification (as seen in #156).
Fixes #156

I have also experienced problems with the host header when smee-client is forwarding a request with the host header value without a port number to a Jenkins master running in OpenShift, resulting in HTTP 503 response status returned to smee-client.
For that reason, removing the host header would help for this issue as well.
[1] https://github.com/openshift/origin/issues/13355

I am testing this PR locally and found that Jenkins refuses to run the GitHub PR because after removal of host header the signature of the request does not match (headers X-Hub-Signature, X-Hub-Signature-256).
I guess the proper solution would be to add another parameter to smee-client, the secret, from which the smee-client would be able to recalculate the signature.

Hi @kahowell @JasonEtco @tcbyrd any news here? Thanks!

I definitely think the change makes sense. Given this comment in https://github.com/probot/smee-client/issues/156#issue-68990720

<EMAIL_ADDRESS>to ^5.0.2 (this is most probably the one which is introducing the error since is the library they use for pushing events).

Would anyone be willing to add a test here? Since the regression was most likely introduced in a dependabot upgrade and people are blocked by this, it would probably be best to have even a basic test to ensure upgrading dependencies doesn't re-introduce the problem.

I would like to ask that the bug fix be released before someone waits too much longer trying to design a universally valid test for TLS validation problems applicable to private CI/CD servers receiving github hooks, with internally signed certificates, behind firewalls. That's just the type of oddball situation that this project appeals to. Maybe there's some other project that would make a good dev dependency to create such a test scenario, but the perfect test is looking like an enemy to this simple bug fix. I get the impression TLS is only going to be more and more common on internal networks and will undermine and increasing number of use cases for smee and smee-client.
The test is a good idea, don't get me wrong, but it could come following a 2nd issue in the tracker. I was afraid to try & count how many of the 100 forks have fixes for this faulty assumption about the "Host" property in data.

@eidsnessje-nih I noticed that I did not run into this issue when testing today, and I saw that superagent has been updated to v7.1.3 in v1.2.3, does it resolve the issue for you?

I tried to solve this issue https://github.com/probot/smee-client/issues/156#issue-68990720 by testing this one and it works great. Would be nice to merge this one. Are you planning to do so?

can you please @kahowell merge upstream/master with this? This way you will have latest, we are using your proposal on production, otherwise I will produce mine :thinking:

@Ginxo I've rebased against latest as requested.

just notice PR https://github.com/probot/smee-client/pull/222 (based on this PR) additionally fixes the commander error, added commander test coverage and github actions workflow. Thanks @kahowell

Its basically impossible to write a useful unit test for this.

I want to make sure #181 is considered as an alternative to this. In my case, the host header is required to have the correct host since the web server or reverse proxy makes decisions based on the host header. Removing the host header entirely may fix cases where the default server is the intended target, but it will make it impossible to correctly target a specific host.
Also, #181 has a test that I believe shows that it works as expected. I think? :)
Is there any reason to believe #181 won't work for addressing #156?

@simensen
I guess, that the issue is not existent anymore. We use now fetch, which does not take host header as it is a header which should not be manipulated.

@simensen
I guess, that the issue is not existent anymore. We use now fetch, which does not take host header as it is a header which should not be manipulated.

I still meet this issue on 2.0.1.
It seems the host header is still forwarding to the target which causes

TLS verification error.
L7 proxy can't forward the request to the right target using the host header.

It's safe to delete the host header and fetch can add it back from the url.

@topikachu
Do you want to provide a PR?


Note body is empty

I have found there is an issue when the original note body is empty. The enhanced note is then created with "" as the new note text. There are some people at our organization that created notes with only the subject.

Sorry... did not 'preview'.
The note is as follows:

Thanks @kryan0618, I've got this fixed in the develop branch. Will be in the next release.

Perfect! Thanks, @DouglasCAyers !


Mobile Scale - Grid

Hello dev,
I want to scale RIGHT panel first, but bootstrap scale MOBILE from LEFT

Bootstrap grid scale (mobile):
1                 2                 3
I want:
3                 1                 2

My Question: Bootstrap this enables or not ?

Please direct how-to questions to StackOverflow. Thanks.


Wrong stream with 320x240 cam

Describe the bug
Stream send as 800x600 whereas the camera is 320x240.
Camera resolution is well detected by wall-panel as 320x240.
To Reproduce
Steps to reproduce the behavior:

Camera setting => Selected camera
Click on '1: Front Camera 320x240 0°'
Camera test : work perfectly !
see video flow on http://IP:2971/camera/stream and see green nearly empty picture (expect 5 frame in the uper side, with motion corresponding to what camera is seeing)
5 : motion detection is working

Expected behavior
Mmmm, see the picture is the expected video format ;o)
Screenshots
If applicable, add screenshots to help explain your problem.

Smartphone (please complete the following information):

Device: Samsung Galaxy Tab 10.1
OS: AOSP 7.1.2
Android WebView (stock or updated to latest 81)
App v0.9.2 build 0

Additional context
try with IP CAM app.
in the app both video flow seems to be selectable but warning message whatever resolution I pick saying the flow is unavailable.
Nevertheless, if I check the URL, I have both choice and :
800x600 wich lead to the same green screen
320x240 with perfectly work ;o)
Tell me if I can provide any log file or other...
Thanks for your handy app ;o)

Thanks, I will have to test this.  What you are saying is the camera test is working, but the MJPEG images are not generated at the correct size to match the camera settings?

Excatly 👍
Le 4 mai 2020 14:59:25 GMT+02:00, Michael Ritchie<EMAIL_ADDRESS>a écrit :

Thanks, I will have to test this.  What you are saying is the camera
test is working, but the MJPEG images are not generated at the correct
size to match the camera settings?
--
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub:
https://github.com/thanksmister/wallpanel-android/issues/182#issuecomment-623448508

--
Envoyé de mon appareil Android avec Courriel K-9 Mail. Veuillez excuser ma brièveté.

I have the green Screen with the 320x240 cam.
Device: Samsung Galaxy Tab 8.9 (P7310)
OS: AOSP 7.1.2
Test also working Perfectly


[travis] add CI for linting

To prevent PR like #971 in the future.
@hai-cea to make it works, you need to go to travis to activate this repository.

@hai-cea Yeah, a green build :)

@oliviertassinari :+1:


Format articleController html so that Facebook algorithm always finds the main image (if that is possible)

Done


Add connection.tls config option

The [connection] block in the BendSQL config.toml doesn't seem to accept a tls option, requiring that I add --tls every time I run bendsql.

Let me see.


Unity.VisualScripting is required in the newest version.

I noticed the "using Unity.VisualScripting;" was added in the latest version. I think this was a mistake. It requires the project to add the Visual Scripting package to work now.
https://github.com/AlexeyPerov/Unity-Dependencies-Hunter/pull/21/commits/20cf332bfcc50b19e8b62ec0354bdc88ebdc455d

+1 Same problem. I am going to use a version from previous commit.

removed in this PR https://github.com/AlexeyPerov/Unity-Dependencies-Hunter/pull/25 by @fwild

issue fixed https://github.com/AlexeyPerov/Unity-Dependencies-Hunter/pull/25


Does not work on Android 2.3

Hello,
I tried running this app on android 2.3 but it does not work.
However, when I install same on 4.x it works.
Can you please find out the issue?

Ionic framework officially support Android 4.2 and above. So the problem might happen to any downgrade version.


Correctly parse arguments if no resource is given but separator is

This is more of a heuristic due to Go's flag parser.
Fixes #17.

The meat is in https://github.com/betalo-sweden/await/pull/18/commits/6ccc893b67506ae25a9562b3bcb244899e860e33


Remaining CFME references

While fixing #2065, I noticed a "few" more CFME references throughout the codebase...
File names:

files in /system, like cfme-setup.sh and cfme-http.conf
the whole lib/cfme_client
pdf docs in vmdb/public/doc/cfme_*.pdf
vmdb/app/assets/javascripts/dynatree_1.2.4/skin-cfme/

Variables and functions:

@cfme in spec tests
cfmeOnClick_* and the like - in javascript
css classes like cfme-no-cursor-node

Messages:

theres a lot of messages like "CFME Appliance restart initiated successfully" - CFME Server, CFME Database, CFME Appliance usually

Any thoughts on those?

lib/cfme_client can simply be renamed to lib/api_client with gem named "manageiq-api-client", this would be consistent with google and github api clients.

We need to discuss the messages individually.  Some of them we can probably remove the word "CFME entirely.  Some of them may need the project name, so should be ManageIQ upstream, with the ability to overlay downstream with a different name.  This was built into the productization with the dictionary, but I don't know if the gettext stuff can do it.

@abellotti Would you please create an issue to rename cfme-client to manageiq-api-client

lib/cfme_client issue created: https://github.com/ManageIQ/manageiq/issues/2198

@himdel
You most probably are aware of this but the cfme_*.pdf docs mentioned above are actually downstream cfme docs, not ManageIQ ones. So they have to be replaced by the upstream files and not just renamed.
Will there be a different GH issue for that or can I count on the files being replaced once this issue is closed?
Thanks in advance

CFME docs removed in #3493.

See also #7129

@Fryguy do you know if this is still an open issue?

ag cfme | wc -l .. 841
So I'd say yes..  a few are false positives, some are server names in spec yamls, but some are user-exposed messages too..

@miq-bot remove_label stale
bugzilla_mirror/: 37
cfme_doc_public/: 3
foreman_api_client/: 7
guides/: 2
integration_tests/: 5246
manageiq-api-mock/: 15
manageiq-appliance-dev-setup/: 1
manageiq-appliance_console/: 6
manageiq-content/: 2
manageiq-performance/: 3
manageiq-providers-amazon/: 53
manageiq-providers-ansible_tower/: 126
manageiq-providers-foreman/: 25
manageiq-providers-kubernetes/: 3
manageiq-providers-ovirt/: 20
manageiq-providers-scvmm/: 491
manageiq-schema/: 1
manageiq-smartstate/: 56
manageiq-ui-classic/: 16
manageiq-ui-scaffold/: 1
manageiq-ui-service/: 1
manageiq.org/: 4
manageiq/: 37
manageiq_depot/: 3
manageiq_docs/: 374
patternfly-sass/: 7
rails/: 2
vCloudDirector-docs/: 2
virtfs-ext3/: 1
wrapanapi/: 46

(from cd ManageIQ; for d in */; do cd $d; echo -n "$d":' ' ; ag cfme | wc -l ; cd .. ; done)


When choosing information about the building (?), The FPS drastically decreases.

Hello. bug (win 7 64) stronghold crusader hd 1.41
When choosing information about the building (?), The FPS drastically decreases. Is it possible to fix it somehow?

can someone else confirm?


Don't use a mask when not needed

When the mask has the same size as the parent register of a field, then
a mask is not required. This removes the clippy warnings/errors where
for example an operation of inverting a mask results in zero
error: this operation will always return zero. This is likely not the intended outcome
   |
61 |         self.w.bits = (self.w.bits & !0xffff_ffff) | (value as u32 & 0xffff_ffff);
   |                       ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
   |

or when the operation of using a mask is ineffective.
warning: the operation is ineffective. Consider reducing it to `value as u32`
   |
61 |         self.w.bits = (self.w.bits & !0xffff_ffff) | (value as u32 & 0xffff_ffff);
   |                                                      ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
   |

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @Emilgardis (or someone else) soon.
Please see the contribution instructions for more information.

bors r+

bors r-

Squash commits, please.


Reading from N5

I have volumes stored in N5 format, and I am serving them via a local http server (using serve). I am also running neuroglancer via npm run dev-server. In neuroglancer, I tried setting the source source to something like n5://http://localhost:5000/$CONTAINER.n5/$DATASET, where $CONTAINER and $DATASET are the names of the n5 container and the dataset I want to view, but this doesn't work. Specifically, I get Fetching "" resulted in HTTP error 0. How should I get this to work?

The current n5 support is rather limited.  It only supports 3d volumes in the format described here:
https://github.com/saalfeldlab/n5-viewer
Your data source URL then needs to be of the form:
n5://http://localhost:5000/CONTAINER.n5/DATASET/c0
CONTAINER.n5/DATASET/attributes.json should contain something like:
{
"scales":[[1,1,1]],
"pixelResolution":{"unit":"um","dimensions":[0.097,0.097,0.18]}
}
CONTAINER.n5/DATASET/c0/attributes.json should not exist (or will be ignored if it does),
and
CONTAINER.n5/DATASET/c0/s0/attributes.json  should be a normal n5 dataset attributes.json file (with raw or gzip compression).
The good news is that in a local branch I have already implemented better support for n5 that does not require such a specific directory structure, and I will be pushing it out soon.  It is part of a large change that also adds support for 4-d, etc. volumes, but is not quite ready yet.

Thanks, this was very helpful -- I ultimately got it working on test data (once I disabled adblock). Re: the local branch, do you have an idea of a) how the required N5 directory structure will be different, and b) when that branch will be visible online?

I just pushed the work in progress to the ndims branch, so you can play with it if you like.
It is still a work in progress, and various things still don't work --- e.g. annotations don't work at the moment.  However, N5 support basically works, though there may be bugs.
It should work with:

plain single resolution datasets
multi-resolution datasets using s0, s1, ... directories with n5-viewer-style (https://github.com/saalfeldlab/n5-viewer) attributes like   "scales":[[1,1,1],[2,2,1],[4,4,2],[8,8,4],[16,16,9],[32,32,17]],
"pixelResolution":{"unit":"um","dimensions":[0.097,0.097,0.18]},
multi-resolution datasets using s0, s1, ... directories with bigdataviewer-style (https://github.com/bigdataviewer/bigdataviewer-core/blob/master/BDV N5 format.md) attributes: "downsamplingFactors" : [[1,1,1], [2,2,1], ...], "resolution": [4, 4, 40]

Additionally you can specify attributes: "axes": ["x", "y", "z", "t"] to specify the axis names, and "units": ["um", "um", "um", "s"] to specify the units for each dimension.
Neuroglancer now aggregates attributes from all parent directories as well, so the cN directories are no longer necessary or treated specially.

Also here is an example that will work with a locally-running dev server:
http://localhost:8080/#!{"dimensions":{"x":[9.700000000000001e-8%2C"m"]%2C"y":[9.700000000000001e-8%2C"m"]%2C"z":[1.8e-7%2C"m"]}%2C"position":[7534.7314453125%2C13950.4384765625%2C3323.5]%2C"crossSectionScale":0.7672059499758557%2C"projectionScale":32768%2C"layers":[{"source":"n5://gs://neuroglancer-exllsm-data/m8-10_Sample12/n5/c0"%2C"type":"image"%2C"blend":"default"%2C"shader":"#uicontrol vec3 color color(default=\"red\")\n#uicontrol float brightness slider(min=-10%2C max=10%2C step=0.001)\n#uicontrol float contrast slider(min=-20%2C max=20%2C step=0.01)\nvoid main() {\n  emitRGB(color * (toNormalized(getDataValue()) + brightness) * exp(contrast))%3B\n}\n"%2C"shaderControls":{"brightness":-0.006%2C"contrast":3.27}%2C"name":"c0"}%2C{"source":"n5://gs://neuroglancer-exllsm-data/m8-10_Sample12/n5/c1"%2C"type":"image"%2C"blend":"default"%2C"shader":"#uicontrol vec3 color color(default=\"red\")\n#uicontrol float brightness slider(min=-10%2C max=10%2C step=0.001)\n#uicontrol float contrast slider(min=-20%2C max=20%2C step=0.01)\nvoid main() {\n  emitRGB(color * (toNormalized(getDataValue()) + brightness) * exp(contrast))%3B\n}\n"%2C"shaderControls":{"color":"#00ff00"%2C"brightness":-0.006%2C"contrast":3.4}%2C"name":"c1"}]%2C"showSlices":false%2C"selectedLayer":{"layer":"c1"%2C"visible":true%2C"size":290}%2C"layout":"xy"%2C"statistics":{"size":372}}

The ndims branch has now been merged, so I believe this issue is resolved.


chore: Simplify serializers

Use default serialization provided by jackson
Fix usage of kafka clientId and groupId. The AsyncApi spec expect Schema objects within the bindings, not pure strings.

Merged in upstream


Add navalia client

For the rest of the tests, you can take some inspiration of https://github.com/SUSE/Portus/blob/master/spec/lib/portus/registry_client_spec.rb#L27. That is, you can mock the perform_request method to whatever you pass to the initializer.

LGTM, squash the commits once tests are green. Then I'll merge this.

Travis is stuck testing 1 out of 3 versions of ruby. Everything looks green with the tested code, so I'll merge the code.


proposal: x/tools/testfiles: provide utilities for testing tools for Go code with modules

Proposal Details
The proposal is to move the existing internal x/tools/internal/testfiles package for testing Go code with go.mod files as a exported package x/tools/internal/testfiles. These functions all create temporary testing directories that are populated with a go.mod file without checking in a file named go.mod.
This package was originally created to better support x/tools/analysistest.Run. From it's documentation:

func Run(t Testing, dir string, a *analysis.Analyzer, patterns ...string) []*Result
Run applies an analysis to the packages denoted by the "go list" patterns.
It loads the packages from the specified directory using golang.org/x/tools/go/packages, runs the analysis on them, and checks that each analysis emits the expected diagnostics and facts [...] .
If the directory contains a go.mod file, Run treats it as the root of the Go module in which to work.

Adopting this proposal would resolve #37054, #46041, #53063, and #61336.
The first supported path is via .txtar archives. This works well with small tests where where it is more readable for all of the files to stay together.
// ExtractTxtarFileToTmp read a txtar archive on a given path,
// extracts it to a temporary directory, and returns the
// temporary directory.
func ExtractTxtarFileToTmp(t testing.TB, archiveFile string) string

The next two functions that would be provided copy a directory while stripping a '.test' file suffix.
// CopyDirToTmp copies dir to a temporary test directory using
// CopyTestFiles and returns the path to the test directory.
func CopyDirToTmp(t testing.TB, srcdir string) string

// CopyFS copies the files and directories in src to a
// destination directory dst. Paths to files and directories
// ending in a ".test" extension have the ".test" extension
// removed. This allows tests to hide files whose names have
// special meaning, such as "go.mod" files or "testdata" directories
// from the go command, or ill-formed Go source files from gofmt.
//
// For example if we copy the directory testdata:
//
//	testdata/
//	    go.mod.test
//	    a/a.go
//	    b/b.go
//
// The resulting files will be:
//
//	dst/
//	    go.mod
//	    a/a.go
//	    b/b.go
func CopyFS(dstdir string, src fs.FS) error

These functions have the benefit of working well with existing editors for .go files. The files can be checked into testdata/ directories. This also supports vendor/ subdirectories. This also bypasses the limitation of txtar files not being able to contain txtar files.

Related Issues and Documentation

x/tools/go/analysis/analysistest: add module-aware analyzer test runners #53063
x/build/cmd/golangbuild: run subrepo tests from outside their repositories #34352 (closed)
proposal: x/tools/go/analysis/analysistest: improved, module-aware API #61336

(Emoji vote if this was helpful or unhelpful; more detailed feedback welcome in this discussion.)

See #62484 (os.CopyFS) and #52427 (txtar fs.FS).

@earthboundkid Do you want to submit the CL you started for #44158? Or anything else I could do to help it get over the finish line? I think this would lead to a nice simplification to this proposal.

I have been too busy to work on #44158 and I'm not sure what the best way to implement ErrModified is, so if you want to take over, feel free. 😄

Updated the proposal with the new consolidated API.


"amd64" becomes "AMD64" [See comment]

It's the correct spelling. However, the term "x86-64" is also commonly used (https://en.wikipedia.org/wiki/X86-64).

It should either be i386 and amd64, x86 and x86-64, or 32-bit and 64-bit.
This is an issue of mixing referencing styles, not incorrect spelling.

@lewisgoddard Do you have any reference for that? The only correct capitalization I see is with capitals.

Hm, there might be a bigger issue here. We're suggesting 64-bit by default. We might just say 64-bit is required and add a note about 32-bit and the alternative download?

AMD64 was back when AMD (the company) had the total monopoly of AMD64 (the architecture). Since i386 is rarely referenced as Intel-386, I see no reason why amd64 should be capitalised.
Regardless, we're still mixing how we're referencing architectures.

Yeah for simplicity sake we should just always use "32-bit" and "64-bit"

I agree. It should be 32-bit and 64-bit. "x86" and "amd64" are too technical for how simple and straightforward elementary OS strives to be.
Can I change the pull request to changing the terms to "32-bit" and "64-bit" then?

Here's the new pull request: https://github.com/elementary/mvp/pull/154
I can close the present request now.


drivers: touch: esp32: update driver support

touch_sensor support was lost when hal_espressif was updated to v5.1
The absence of .conf files enabling INPUT_ESP32_TOUCH_SENSOR incorrectly prevented the input_dump sample build process from failing due to missing required hal files.
This PR fixes the regression regarding to touch_sensor support lost on hal_espressif and adds the absent .conf files to the Espressif boards on the input_dump sample to avoid false positives
This PR is realate to the fix #79941 on main branch:
Related issue: #79947

Hi @wmrsouza, bakports should be handled using the backport bot whenever possible, that works by adding a backport tag to the original PR, the backport will be created automatically once the main PR is merged. I'll add that and to https://github.com/zephyrproject-rtos/zephyr/pull/79941/commits and close this for you.

@fabiobaltieri
The hal_espressif revision from this PR points to a different commit (in a different branch) that pointed by #79941, so I reopen this PR, remove backport v3.7-branch label from #79941 here and add this label here

@fabiobaltieri The hal_espressif revision from this PR points to a different commit (in a different branch) that pointed by #79941, so I reopen this PR, remove backport v3.7-branch label from #79941 here and add this label here

Oh sorry did not realize that, ok, still let's wait for the main PR to go in though.


Add logging support to ApiHub SDK

Add logging support to ApiHub SDK
Fix output binding to non-string objects for ApiHub

Hi @safihamid, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by .NET Foundation and real humans are currently evaluating your PR.
TTYL, DNFBOT;


unit testing

we have no unit testing suite, which is bad practice and makes merging the branches hazardous.

Reviewing best practices (eg Baeldung and Unit Testing Principles, Practices, and Patterns by Khorikov, restructuring the project directory seems necessary. The question of how to ensure that the python scripts and the java project are made maximally accessible is unclear to me and I'm going to do a bit more reading about it.

I've done a lot of refactoring, and it's probably worth us having a review session on how to rebase branches for merging purposes!
In any case, the main branch now has a basic testing suite setup. I'll work on developing it further. For the time being, I am not making use of gradle - but in the future it would be helpful in terms of automatically jarring the code, etc.

Go through and make these tests parameterized. e.g.
import static org.junit.Assert.assertTrue;
import java.util.Arrays;
import org.junit.Test;
import org.junit.runner.RunWith;
import org.junit.runners.Parameterized;
@RunWith(Parameterized.class)
public class ParametrizedTest {
private final String text;
private final int number;

public ParametrizedTest(String text, int number) {
    this.text = text;
    this.number = number;
}

@Test
public void shouldContainNumber() {
    assertTrue(text.contains(String.valueOf(number)));
}

@Parameterized.Parameters
public static Iterable<Object[]> params() {
    return Arrays.asList(
            new Object[][]{
                    {"test string 1", 1},
                    {"test string 2", 2}
            }
    );
}

}


KOGITO-1472 Random test failures on kogito-timer-quarkus example

another workaround for https://github.com/kiegroup/kogito-examples/pull/149

for some reason can't assign review to @tiagodolphine

@tiagodolphine yes I know :(

@tiagodolphine good news (?) https://github.com/quarkusio/quarkus/pull/8005 :crossed_fingers:

nope, no luck here


HTTPFileSystem requires requests and aiohttp

If requests but not aiohttp is installed the current error message is misleading since it only mentions requests.

Thanks for this.


Updated italian translation

Explain your changes
Updates italian translation
Checklist

[x] I have read the “Pull requests” section in the contributing guidelines.
[x] I agree to the terms within the code of conduct.

🛟 If you need help, consider asking for advice over in the Kinde community.

Thanks @Ale1x unfortunately the validator isn't passing, would you mind checking?

Sure


Any hope for a schema for the JSON output?

That's about it - would love a schema for the JSON output.

Great idea! PRs welcome

I guess that answers my question. I'm currently working on building a small library of functions to quickly parse json output to aggregate the data for a given exported reactComponent. Currently I'm just having to explore the huge data tree that's exported and I'm slowly building up an understanding of it.
What would really help would be if someone could point me to the piece of source code that actually generates the json. I've spent a little time snooping through the source code but haven't turned up anything super usefull yet.
If I could see how the json is generated that would really help me define accurate interfaces/types and their relationships for writing functions to parse and aggregate the data.

@StJohn3D The problem is that the the JSON is serialized in multiple classes so there isn't one place to look. There was some work to allow custom serializers which could be a more centralized place to look at types but that work was never fully integrated.

Thanks @aciccarello. @benpolinsky After digging more I believe I've found the closest thing. Check out the models folder in this repo - it has exported class definitions for everything used to create the json output.
https://github.com/TypeStrong/typedoc/tree/master/src/lib/models

While looking through the serialization, I found browser.ts which appears to define types for the JSON output.
However, these types are not used anywhere in TypeDoc's codebase, so they may be incorrect or outdated simply due to being forgotten. I'm hoping to correct this and increase the type safety of the serialization process relatively soon.


Handling when "Javascript is required" to login

Hi there.
Nice library! I like how verbose this error is -- it's very clear what the problem is. My question is whether or not dealing with this scenario is in scope of skype-http -- I'm assuming it is not:

For context, this is happening when logging in on a machine which has not logged into skype before through a normal client (i.e. I am guessing we are running into a security measure when Microsoft notices a login from a strange IP such as a virtual private server).

MicrosoftAccountLogin: Unable to login with MicrosoftAccount.
  caused by GetLiveToken: Unable to get the Live token for Skype
  caused by LiveTokenNotFound: Unable to find the Live token. This token is normally found in the HTML response as the value of the element with the id "t". This error may be caused by a change in the Microsoft login workflow. HTML page: "JavaScript required to sign in<meta http-equiv=\"Refresh\" content=\"0; URL=https://login.live.com/jsDisabled.srf?mkt=EN-US&lc=1033\"/><title>Continue</title><form name=\"fmHF\" id=\"fmHF\" action=\"https://account.live.com/identity/confirm?ru=https://login.live.com/login.srf%3flc%3d1033%26sf%3d1%26id%3d293290%26tw%3d18000%26fs%3d0%26ts%3d-1%26sec%3d%26mspp_shared%3d1%26seclog%3d10%26claims%3d%26wa%3dwsignin1.0%26wp%3dMBI_SSL%26ru%3dhttps://lw.skype.com/login/oauth/proxy%253fclient_id%253d578134%2526site_name%253dlw.skype.com%2526redirect_uri%253dhttps%25253A%25252F%25252Fweb.skype.com%25252F%26contextid%3d20B63404CFB06BE2&mkt=EN-US&uiflavor=web&id=293290&uaid=ef101e26ba7ac7723c7241ecaca40cb1\" method=\"post\" target=\"_top\"><input type=\"hidden\" name=\"ipt\" id=\"ipt\" value=\"Dfk7QGQf8!GGva9nb6r*2iXDtuYzpFtbqyInAWOdXUL*6GEcPBL!yWlm4koF!w1BByhraPWpkN7HdGaZ5nFvaugs7g5AxtaZD95BUmvmVyec9GqyRQcFJsP2POnu!avtdMFUwHRF54CdPc3yyqQMAcj6hQE*pTG2Pzxbql2W9emzQzTjE2k0i6RtyC1SNkH3e4bpWvCyTb0*ZV4Yov69k3Jm9CqJblo8pN3TdarmZoKwD8Bo0xgnujcEVcdqQjqX50gDUtq6P1udIjmYUv!WUZAUtNeluy*Yj7z!BIu8CZnoQjqOizc5WOyOl5MQMSn4KsdFiPye5XzJDSsK5iNUgz4aNmrPhPgC6OWohhYpOG!SkLRAHd1mMvDf*TxBhc2eHJDTVs6eK!rwjVbByXwbw0U6QMpnmvN84PEifvfqKfRn0pKSWYfkC1Md0hDPo8xTFHDQARpLXvl93AHCs1A3dII$\"></form>"
    at Object.create (/home/keyvan/code/matrix-puppet-skype/node_modules/skype-http/dist/lib-es2015/lib/errors/microsoft-account/get-live-token.js:17:16)
    at scrapLiveToken (/home/keyvan/code/matrix-puppet-skype/node_modules/skype-http/dist/lib-es2015/lib/providers/microsoft-account.js:221:61)
    at /home/keyvan/code/matrix-puppet-skype/node_modules/skype-http/dist/lib-es2015/lib/providers/microsoft-account.js:138:20
    at Generator.next (<anonymous>)
    at fulfilled (/home/keyvan/code/matrix-puppet-skype/node_modules/skype-http/dist/lib-es2015/lib/providers/microsoft-account.js:4:58)
    at process._tickCallback (internal/process/next_tick.js:109:7)
  name: 'MicrosoftAccountLogin',
  data: {},
  cause:
   { GetLiveToken: Unable to get the Live token for Skype
     caused by LiveTokenNotFound: Unable to find the Live token. This token is normally found in the HTML response as the value of the element with the id "t". This error may be caused by a change in theMicrosoft login workflow. HTML page: "JavaScript required to sign in<meta http-equiv=\"Refresh\" content=\"0; URL=https://login.live.com/jsDisabled.srf?mkt=EN-US&lc=1033\"/><title>Continue</title><form name=\"fmHF\" id=\"fmHF\" action=\"https://account.live.com/identity/confirm?ru=https://login.live.com/login.srf%3flc%3d1033%26sf%3d1%26id%3d293290%26tw%3d18000%26fs%3d0%26ts%3d-1%26sec%3d%26mspp_shared%3d1%26seclog%3d10%26claims%3d%26wa%3dwsignin1.0%26wp%3dMBI_SSL%26ru%3dhttps://lw.skype.com/login/oauth/proxy%253fclient_id%253d578134%2526site_name%253dlw.skype.com%2526redirect_uri%253dhttps%25253A%25252F%25252Fweb.skype.com%25252F%26contextid%3d20B63404CFB06BE2&mkt=EN-US&uiflavor=web&id=293290&uaid=ef101e26ba7ac7723c7241ecaca40cb1\" method=\"post\" target=\"_top\"><input type=\"hidden\" name=\"ipt\" id=\"ipt\" value=\"Dfk7QGQf8!GGva9nb6r*2iXDtuYzpFtbqyInAWOdXUL*6GEcPBL!yWlm4koF!w1BByhraPWpkN7HdGaZ5nFvaugs7g5AxtaZD95BUmvmVyec9GqyRQcFJsP2POnu!avtdMFUwHRF54CdPc3yyqQMAcj6hQE*pTG2Pzxbql2W9emzQzTjE2k0i6RtyC1SNkH3e4bpWvCyTb0*ZV4Yov69k3Jm9CqJblo8pN3TdarmZoKwD8Bo0xgnujcEVcdqQjqX50gDUtq6P1udIjmYUv!WUZAUtNeluy*Yj7z!BIu8CZnoQjqOizc5WOyOl5MQMSn4KsdFiPye5XzJDSsK5iNUgz4aNmrPhPgC6OWohhYpOG!SkLRAHd1mMvDf*TxBhc2eHJDTVs6eK!rwjVbByXwbw0U6QMpnmvN84PEifvfqKfRn0pKSWYfkC1Md0hDPo8xTFHDQARpLXvl93AHCs1A3dII$\"></form>"
       at Object.create (/home/keyvan/code/matrix-puppet-skype/node_modules/skype-http/dist/lib-es2015/lib/errors/microsoft-account/get-live-token.js:17:16)
       at scrapLiveToken (/home/keyvan/code/matrix-puppet-skype/node_modules/skype-http/dist/lib-es2015/lib/providers/microsoft-account.js:221:61)
       at /home/keyvan/code/matrix-puppet-skype/node_modules/skype-http/dist/lib-es2015/lib/providers/microsoft-account.js:138:20
       at Generator.next (<anonymous>)
       at fulfilled (/home/keyvan/code/matrix-puppet-skype/node_modules/skype-http/dist/lib-es2015/lib/providers/microsoft-account.js:4:58)
       at process._tickCallback (internal/process/next_tick.js:109:7)
     name: 'GetLiveToken',
     data: {},
     cause:
      { LiveTokenNotFound: Unable to find the Live token. This token is normally found in the HTML response as the value of the element with the id "t". This error may be caused by a change in the Microsoft login workflow. HTML page: "JavaScript required to sign in<meta http-equiv=\"Refresh\" content=\"0; URL=https://login.live.com/jsDisabled.srf?mkt=EN-US&lc=1033\"/><title>Continue</title><form name=\"fmHF\" id=\"fmHF\" action=\"https://account.live.com/identity/confirm?ru=https://login.live.com/login.srf%3flc%3d1033%26sf%3d1%26id%3d293290%26tw%3d18000%26fs%3d0%26ts%3d-1%26sec%3d%26mspp_shared%3d1%26seclog%3d10%26claims%3d%26wa%3dwsignin1.0%26wp%3dMBI_SSL%26ru%3dhttps://lw.skype.com/login/oauth/proxy%253fclient_id%253d578134%2526site_name%253dlw.skype.com%2526redirect_uri%253dhttps%25253A%25252F%25252Fweb.skype.com%25252F%26contextid%3d20B63404CFB06BE2&mkt=EN-US&uiflavor=web&id=293290&uaid=ef101e26ba7ac7723c7241ecaca40cb1\" method=\"post\" target=\"_top\"><input type=\"hidden\" name=\"ipt\" id=\"ipt\" value=\"Dfk7QGQf8!GGva9nb6r*2iXDtuYzpFtbqyInAWOdXUL*6GEcPBL!yWlm4koF!w1BByhraPWpkN7HdGaZ5nFvaugs7g5AxtaZD95BUmvmVyec9GqyRQcFJsP2POnu!avtdMFUwHRF54CdPc3yyqQMAcj6hQE*pTG2Pzxbql2W9emzQzTjE2k0i6RtyC1SNkH3e4bpWvCyTb0*ZV4Yov69k3Jm9CqJblo8pN3TdarmZoKwD8Bo0xgnujcEVcdqQjqX50gDUtq6P1udIjmYUv!WUZAUtNeluy*Yj7z!BIu8CZnoQjqOizc5WOyOl5MQMSn4KsdFiPye5XzJDSsK5iNUgz4aNmrPhPgC6OWohhYpOG!SkLRAHd1mMvDf*TxBhc2eHJDTVs6eK!rwjVbByXwbw0U6QMpnmvN84PEifvfqKfRn0pKSWYfkC1Md0hDPo8xTFHDQARpLXvl93AHCs1A3dII$\"></form>"
          at Object.create (/home/keyvan/code/matrix-puppet-skype/node_modules/skype-http/dist/lib-es2015/lib/errors/microsoft-account/get-live-token.js:17:16)
          at scrapLiveToken (/home/keyvan/code/matrix-puppet-skype/node_modules/skype-http/dist/lib-es2015/lib/providers/microsoft-account.js:221:61)
          at /home/keyvan/code/matrix-puppet-skype/node_modules/skype-http/dist/lib-es2015/lib/providers/microsoft-account.js:138:20
          at Generator.next (<anonymous>)
          at fulfilled (/home/keyvan/code/matrix-puppet-skype/node_modules/skype-http/dist/lib-es2015/lib/providers/microsoft-account.js:4:58)
          at process._tickCallback (internal/process/next_tick.js:109:7)
        name: 'LiveTokenNotFound',
        data: [Object],
        message: 'LiveTokenNotFound: Unable to find the Live token. This token is normally found in the HTML response as the value of the element with the id "t". This error may be caused by a change in the Microsoft login workflow. HTML page: "JavaScript required to sign in<meta http-equiv=\\"Refresh\\" content=\\"0; URL=https://login.live.com/jsDisabled.srf?mkt=EN-US&lc=1033\\"/><title>Continue</title><form name=\\"fmHF\\" id=\\"fmHF\\" action=\\"https://account.live.com/identity/confirm?ru=https://login.live.com/login.srf%3flc%3d1033%26sf%3d1%26id%3d293290%26tw%3d18000%26fs%3d0%26ts%3d-1%26sec%3d%26mspp_shared%3d1%26seclog%3d10%26claims%3d%26wa%3dwsignin1.0%26wp%3dMBI_SSL%26ru%3dhttps://lw.skype.com/login/oauth/proxy%253fclient_id%253d578134%2526site_name%253dlw.skype.com%2526redirect_uri%253dhttps%25253A%25252F%25252Fweb.skype.com%25252F%26contextid%3d20B63404CFB06BE2&mkt=EN-US&uiflavor=web&id=293290&uaid=ef101e26ba7ac7723c7241ecaca40cb1\\" method=\\"post\\" target=\\"_top\\"><input type=\\"hidden\\" name=\\"ipt\\" id=\\"ipt\\" value=\\"Dfk7QGQf8!GGva9nb6r*2iXDtuYzpFtbqyInAWOdXUL*6GEcPBL!yWlm4koF!w1BByhraPWpkN7HdGaZ5nFvaugs7g5AxtaZD95BUmvmVyec9GqyRQcFJsP2POnu!avtdMFUwHRF54CdPc3yyqQMAcj6hQE*pTG2Pzxbql2W9emzQzTjE2k0i6RtyC1SNkH3e4bpWvCyTb0*ZV4Yov69k3Jm9CqJblo8pN3TdarmZoKwD8Bo0xgnujcEVcdqQjqX50gDUtq6P1udIjmYUv!WUZAUtNeluy*Yj7z!BIu8CZnoQjqOizc5WOyOl5MQMSn4KsdFiPye5XzJDSsK5iNUgz4aNmrPhPgC6OWohhYpOG!SkLRAHd1mMvDf*TxBhc2eHJDTVs6eK!rwjVbByXwbw0U6QMpnmvN84PEifvfqKfRn0pKSWYfkC1Md0hDPo8xTFHDQARpLXvl93AHCs1A3dII$\\"></form>"\n    at Object.create (/home/keyvan/code/matrix-puppet-skype/node_modules/skype-http/dist/lib-es2015/lib/errors/microsoft-account/get-live-token.js:17:16)\n    at scrapLiveToken (/home/keyvan/code/matrix-puppet-skype/node_modules/skype-http/dist/lib-es2015/lib/providers/microsoft-account.js:221:61)\n    at /home/keyvan/code/matrix-puppet-skype/node_modules/skype-http/dist/lib-es2015/lib/providers/microsoft-account.js:138:20\n    at Generator.next (<anonymous>)\n    at fulfilled (/home/keyvan/code/matrix-puppet-skype/node_modules/skype-http/dist/lib-es2015/lib/providers/microsoft-account.js:4:58)\n    at process._tickCallback (internal/process/next_tick.js:109:7)' },
     message: 'GetLiveToken: Unable to get the Live token for Skype\n  caused by LiveTokenNotFound: Unable to find the Live token. This token is normally found in the HTML response as the value of the element with the id "t". This error may be caused by a change in the Microsoft login workflow. HTML page: "JavaScript required to sign in<meta http-equiv=\\"Refresh\\" content=\\"0; URL=https://login.live.com/jsDisabled.srf?mkt=EN-US&lc=1033\\"/><title>Continue</title><form name=\\"fmHF\\" id=\\"fmHF\\" action=\\"https://account.live.com/identity/confirm?ru=https://login.live.com/login.srf%3flc%3d1033%26sf%3d1%26id%3d293290%26tw%3d18000%26fs%3d0%26ts%3d-1%26sec%3d%26mspp_shared%3d1%26seclog%3d10%26claims%3d%26wa%3dwsignin1.0%26wp%3dMBI_SSL%26ru%3dhttps://lw.skype.com/login/oauth/proxy%253fclient_id%253d578134%2526site_name%253dlw.skype.com%2526redirect_uri%253dhttps%25253A%25252F%25252Fweb.skype.com%25252F%26contextid%3d20B63404CFB06BE2&mkt=EN-US&uiflavor=web&id=293290&uaid=ef101e26ba7ac7723c7241ecaca40cb1\\" method=\\"post\\" target=\\"_top\\"><input type=\\"hidden\\" name=\\"ipt\\" id=\\"ipt\\" value=\\"Dfk7QGQf8!GGva9nb6r*2iXDtuYzpFtbqyInAWOdXUL*6GEcPBL!yWlm4koF!w1BByhraPWpkN7HdGaZ5nFvaugs7g5AxtaZD95BUmvmVyec9GqyRQcFJsP2POnu!avtdMFUwHRF54CdPc3yyqQMAcj6hQE*pTG2Pzxbql2W9emzQzTjE2k0i6RtyC1SNkH3e4bpWvCyTb0*ZV4Yov69k3Jm9CqJblo8pN3TdarmZoKwD8Bo0xgnujcEVcdqQjqX50gDUtq6P1udIjmYUv!WUZAUtNeluy*Yj7z!BIu8CZnoQjqOizc5WOyOl5MQMSn4KsdFiPye5XzJDSsK5iNUgz4aNmrPhPgC6OWohhYpOG!SkLRAHd1mMvDf*TxBhc2eHJDTVs6eK!rwjVbByXwbw0U6QMpnmvN84PEifvfqKfRn0pKSWYfkC1Md0hDPo8xTFHDQARpLXvl93AHCs1A3dII$\\"></form>"\n    at Object.create (/home/keyvan/code/matrix-puppet-skype/node_modules/skype-http/dist/lib-es2015/lib/errors/microsoft-account/get-live-token.js:17:16)\n    at scrapLiveToken (/home/keyvan/code/matrix-puppet-skype/node_modules/skype-http/dist/lib-es2015/lib/providers/microsoft-account.js:221:61)\n    at /home/keyvan/code/matrix-puppet-skype/node_modules/skype-http/dist/lib-es2015/lib/providers/microsoft-account.js:138:20\n    at Generator.next (<anonymous>)\n    at fulfilled (/home/keyvan/code/matrix-puppet-skype/node_modules/skype-http/dist/lib-es2015/lib/providers/microsoft-account.js:4:58)\n    at process._tickCallback (internal/process/next_tick.js:109:7)' } }

If you login via skype web (https://web.skype.com) do you get any prompts? Does it work after logging in on there from the same pc?

This is on a VPS, so I need to find a reliable way to appear as though I'm there.

@kfatehi it looks like an identity confirm prompt looking at the links.  You need to setup a proxy or similar on the vps and login to skype web to do the identity confirm and then it should work.   Please reply back if this fails.

I am glad that the error was clear. I'd like all the errors to be this good but it's pretty long...
I am not sure if we can do much at this point, but it could be useful to recognize some "known" failure scenarios to provide a more specific error. Currently we recognize wrong credentials and rate limits but there are other common errors such as captchas that could benefit from a better error. Ideally, you should be able to switch on the error name to then provide the appropriate way to deal with the login error to your users.


CI and building for PyPy

Over at conda-forge they have started rolling out support for pypy3.7, and the CI run hangs. Have you considered running CI here? github actions would be a good candidate since they offer a nice recipe for python versions including pypy (probably best to test for pypy3.7, 3.6 is not long for the world). If you want to go all out and deploy from the repo, the cibuildwheel project builds, tests, and deploys for all the chosen python versions (https://cibuildwheel.readthedocs.io/en/stable/options/#build-skip)in one run per platfom.
Edit: fix link to conda-forge issue

Coincidentally, I have just fixed test errors on PyPy for Windows this evening. The issue is/was that imagecodecs uses Cython memoryviews const uint8_t[::1] to write to the output of PyBytes_FromStringAndSize, which does not work on pypy. It could be the reason for the hangs. For now I have changed imagecoddecs to return bytearray instead of bytes on PyPy.
The azure pipeline to build linux and macos wheels is at https://github.com/Czaki/imagecodecs_build. It has pypy enabled.

Coincidentally, I have just fixed test errors on PyPy for Windows this evening. The issue is/was that imagecodecs uses Cython memoryviews const uint8_t[::1] to write to the output of PyBytes_FromStringAndSize, which does not work on pypy. It could be the reason for the hangs. For now I have changed imagecoddecs to return bytearray instead of bytes on PyPy.
The azure pipeline to build linux and macos wheels is at https://github.com/Czaki/imagecodecs_build. It has pypy enabled.

Cool, thanks. Should I close this then?

Cool, thanks. Should I close this then?

I build pypy wheels for manylinux, but building wheels for macos fails because fail of installation on macos. numpy does not provide pypy wheel for pypy3.7.
    RuntimeError: Polyfit sanity test emitted a warning, most likely due to using a buggy Accelerate backend. If you compiled yourself, see site.cfg.example for information. Otherwise report this to the vendor that provided NumPy.
    RankWarning: Polyfit may be poorly conditioned

I try to install openblas from brew but it happens again (I restart the job). @mattip maybe you have some suggestions on how to workaround this?

I build pypy wheels for manylinux, but building wheels for macos fails because fail of installation on macos. numpy does not provide pypy wheel for pypy3.7.
    RuntimeError: Polyfit sanity test emitted a warning, most likely due to using a buggy Accelerate backend. If you compiled yourself, see site.cfg.example for information. Otherwise report this to the vendor that provided NumPy.
    RankWarning: Polyfit may be poorly conditioned

I try to install openblas from brew but it happens again (I restart the job). @mattip maybe you have some suggestions on how to workaround this?

This comment suggests
brew install openblas
OPENBLAS="$(brew --prefix openblas)" pip install numpy

This comment suggests
brew install openblas
OPENBLAS="$(brew --prefix openblas)" pip install numpy

It works. I will wait for fixed verion of imagecodecs. But @mattip in my case test fails, not stuck.

It works. I will wait for fixed verion of imagecodecs. But @mattip in my case test fails, not stuck.

But @mattip in my case test fails, not stuck.

Can you point me to the log of a faiiled PyPy build? The one I found is still using accelerate for the PyPy macos run.

But @mattip in my case test fails, not stuck.

Can you point me to the log of a faiiled PyPy build? The one I found is still using accelerate for the PyPy macos run.

@cgohlke

Coincidentally, I have just fixed test errors on PyPy for Windows this evening

Is that change reflected in the repo here? Is it specifically for windows?

@cgohlke

Coincidentally, I have just fixed test errors on PyPy for Windows this evening

Is that change reflected in the repo here? Is it specifically for windows?

You have stucked test on both macos and linux. In my case I think about linux one
https://dev.azure.com/bokota/imagecodecs/_build/results?buildId=601&view=logs&jobId=b50d5a70-5aa0-546a-0931-d41e5c96779d
On MacOS I need wait about 50 minutes to check https://dev.azure.com/bokota/imagecodecs/_build/results?buildId=604&view=results
What you mean by using accelerate?

You have stucked test on both macos and linux. In my case I think about linux one
https://dev.azure.com/bokota/imagecodecs/_build/results?buildId=601&view=logs&jobId=b50d5a70-5aa0-546a-0931-d41e5c96779d
On MacOS I need wait about 50 minutes to check https://dev.azure.com/bokota/imagecodecs/_build/results?buildId=604&view=results
What you mean by using accelerate?

There is also no stuck for macos job.

There is also no stuck for macos job.

I guess you have a hidden branch or something. If you wanted to share the patch we could try applying to conda-forge to see if we are still experiencing the issues.

I guess you have a hidden branch or something. If you wanted to share the patch we could try applying to conda-forge to see if we are still experiencing the issues.

The only patch which I have is connected with new clang version which cause fail on undeclared functions https://github.com/Czaki/imagecodecs_build/blob/master/imagecodecs.patch.
But maybe it is connected with used library vesions: https://github.com/Czaki/imagecodecs_build/blob/master/build_utils/download_libraries.sh
PyPy build in on master branch https://dev.azure.com/bokota/imagecodecs/_build/results?buildId=605&view=results

The only patch which I have is connected with new clang version which cause fail on undeclared functions https://github.com/Czaki/imagecodecs_build/blob/master/imagecodecs.patch.
But maybe it is connected with used library vesions: https://github.com/Czaki/imagecodecs_build/blob/master/build_utils/download_libraries.sh
PyPy build in on master branch https://dev.azure.com/bokota/imagecodecs/_build/results?buildId=605&view=results

You could also add pytest-timeout to test call. Then you will exactly know which test hangs.

You could also add pytest-timeout to test call. Then you will exactly know which test hangs.

Fixed in v2021.1.11

Fixed in v2021.1.11

thanks!

thanks!


Ender Dragon doesn't Work

Minecraft: 1.18.2
Forge: 40.2.10
Progressive Bosses: 3.6.7
InsaneLib: 1.5.3

The Ender Dragon doesn't work with Nullscape 1.18, none of its abilities work or are modified. Even if is respawned, it doesn't work either.
Progressive Bosses + InsaneLib + Nullscape (Last Version)

Does it work with max difficulty?

Yeah, I already tried changing it to the maximum difficulty and even increasing it in the config, and nothing.

Does it work with PB 3.6.6?

Yeah, it doesn't work either.

Tested with 3.6.7 and Nullscape 1.18.2 ... everything works properly

Are you sure? I deleted all my external mods to test and verify that it wasn't an external incompatibility, these were the mods I left (InsaneLib is in version 1.5.3)

To be more specific, the other mobs (the Wither and the Elder Guardian) works normally, but the Dragon when entering the end doesn't have any ability applied.

Again, I tested with PB and Nullscape DataPack only.

Have you actually tried with PB and Nullscape mod only?

Also lib_InsaneLib.jar.jar doesn't sound like a mod downloaded from official manners

lib_InsaneLib.jar. is the mod I renamed to make it easier for me to organize, it is actually Insanelib in the last version.
About testing Nullscape and PB on their own, the same error keeps happening.

Thanks, to make sure it's not the other many mods you had

Did you try with the DataPack?

Yeah, with the Datapack it works normally, but with the mod not, I think it is because in the Datapack as far as I saw, they didn't changed the main island, while in the mod, yes.

What ... the mod should be just a packaged data pack ...

I have absolutely no clue ... I've tried to test with the mod and the dragon will not even respawn lol.
The mod is broken, not gonna bother more than that, sorry, use the DataPack since should be the same thing


Using a browser API that returns Promise

I've trying to use window.crypto.subtle.generateKey and got crypto_subtle.dart working on dartdevc and dart2js, but flutter run -d web-server --release is broken (I couldn't reproduce the issue with dart2js -O4).
If I use CryptoKey from dart:html, I can get it working on flutter run -d web-server --release, as well as flutter test --platform chrome, but not dart test -p chrome.
Minimal example with comments:
https://gist.github.com/jonasfj/30d7a41b57d1af9b84edb039f1912011
Context: https://github.com/google/webcrypto.dart/pull/7

Thanks for the bug report!
Indeed, there is an internal limitation that disallows us to use JS-interop on types that are already expose in dart:html (what we call "native" types).  This is a tricky and brittle part of JS-interop today that we hope to improve it this year.
Short term, @srujzs is about to land a change that will check and report a compile-time error if we detect this issue. We can't always detect it, but in the example you showed above we would have.

Thanks for the bug report!
Indeed, there is an internal limitation that disallows us to use JS-interop on types that are already expose in dart:html (what we call "native" types).  This is a tricky and brittle part of JS-interop today that we hope to improve it this year.
Short term, @srujzs is about to land a change that will check and report a compile-time error if we detect this issue. We can't always detect it, but in the example you showed above we would have.

I think I observed something similar trying to use Window.requestFileSystem. This is defined to return a Future<FileSystem> but however I tried to access it, I always ended of with a failure because the async system expected the value to be have a correct Dart type but instead was a native JavaScript object.

I think I observed something similar trying to use Window.requestFileSystem. This is defined to return a Future<FileSystem> but however I tried to access it, I always ended of with a failure because the async system expected the value to be have a correct Dart type but instead was a native JavaScript object.

@johnniwinther - I believe your issue with requestFIleSystem is due to https://github.com/dart-lang/sdk/issues/45036

Right, the symptoms seem almost identical to that issue.
Addressing the original issue though, you can still workaround this limitation (that won't break in unexpected ways) by using @anonymous classes for CryptoKey.


Public function to add a button to internal ignore list

"Requesting a public function to add a button to your internal ignore list, so I don't add it to MBB_Ignore directly.
See https://github.com/teelolws/EditModeExpanded/commit/ceed3f5c46c6579360fe0f4c57f3f7d7b0d85528"
@teelolws

OMG YES!!!


Prevent error on creating automatic redirects for sites with non-standard ports

Fixes #7942 - get_dummy_request should not include the port number in SERVER_NAME.

Merged in adeabd963381500a2d63aad53346c393380f2c04 (main) / f10de95b73d11974d604b204c4a82d7ad1cc20dc (2.16.x).


[AIRFLOW-2351] timezone fix for @once DAGs

Make sure you have checked all steps below.
JIRA

[X] My PR addresses the following AIRFLOW-2351 issue and references them in the PR title.

Description

[X] Here are some details about my PR, including screenshots of any UI changes:

Upgraded Airflow .. getting following error [1] when processing a DAG
We have 'start_date': None set in default_args.. but this used to work im previous airflow versions.
This is a '@once DAG.. so we don't need a start_date (no back fill).
[2018-01-16 16:05:25,283] {models.py:293} ERROR - Failed to import: /home/rdautkha/airflow/dags/discover/discover-ora-load-2.py
Traceback (most recent call last):
File "/opt/airflow/airflow-20180116/src/apache-airflow/airflow/models.py", line 290, in process_file
m = imp.load_source(mod_name, filepath)
File "/home/rdautkha/airflow/dags/discover/discover-ora-load-2.py", line 66, in
orientation                    = 'TB',                          # default graph view
File "/opt/airflow/airflow-20180116/src/apache-airflow/airflow/models.py", line 2951, in init
self.timezone = self.default_args['start_date'].tzinfo
AttributeError: 'NoneType' object has no attribute 'tzinfo'

Tests

[X] My PR adds the following unit tests OR does not need testing for this extremely good reason:

Tested manually

cc @bolkedebruin

Codecov Report

Merging #3254 into master will decrease coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #3254      +/-   ##
==========================================
- Coverage   75.24%   75.24%   -0.01%
==========================================
  Files         197      197
  Lines       14686    14688       +2
==========================================
+ Hits        11051    11052       +1
- Misses       3635     3636       +1

Impacted Files
Coverage Δ

airflow/models.py
86.6% <100%> (-0.04%)
:arrow_down:

Continue to review full report at Codecov.

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

Good catch. Can you please add a test and make sure you follow the remainder of the commit guidelines as well?

@bolkedebruin thanks for prompt response. I haven't done any unit tests for airflow.. can you point me to a unittest that's close to this PR, so I'll try to mimic some logic.
Regarding "remainder of the commit guidelines" - I thought I've followed everything except documentation. I don't think this hotfix needs documentation change. Did I miss something else?

Thanks @bolkedebruin


Profiler does not seem to work correctly on Linux amd64

cmdline:
time ./profiler --k2-pow-difficulty<PHONE_NUMBER>04 --data-file profiler_data.bin --data-size 1 --duration 10 -t 1
^C

real	5m3.591s
user	76m50.815s
sys	0m5.756s

There is also 0 data created on the disk despite running for 5mins (instead of 10seconds as I asked)
Additionally, there was nothing created on a disk. Screenshot presents CPU usage

When I removed --k2-pow-difficulty<PHONE_NUMBER>04 from the cmdline then it started to behave ok.
root@testnet-04-miner-1-0:~/data/post/profiler_d# time ./profiler  --data-file profiler_data.bin --data-size 1 --duration 10 -t 1
{
  "time_s": 10.6147993,
  "speed_gib_s": 0.9420809303478777
}

real	0m10.644s
user	0m9.243s
sys	0m1.402s

It's doing K2 proof of work first, for which it uses all cores (not respecting the passed -t 1 - #74). Besides that IMO it works OK.


Yo will this be html5?

What is your question?
Yo will this be html5?

how do you imagine it, all features uses file system

So no? i mean it could work

no i dont want do for hmtl


ASYNC Feature

Discussed in https://github.com/codebar-ag/laravel-flysystem-cloudinary/discussions/31

Originally posted by nextgeni-mubbi June  8, 2023
Your package is missing the Async feature that allows quick response to user and image uploads in cloudinary in the background. here is the url for the reference : https://cloudinary.com/documentation/image_upload_api_reference
Here we need to make the "async"=true and provide "notification_url"='our-webhook-url'.

Implemented within v3


Use the annotated pretty printer

Just keeping this simple for now, subtle additional annotations can be added later.

what does this do?

woohoo! :pray:

I think you'll need to update the tests.
$ test/cli/repl/run
# eyeball the differences. probably only escape codes
$ test/cli/repl/run UPDATE

I do git add -p afterwards and :eyes: each chunk really hard

that's probably a lot easier to read...

:dromedary_camel: nice :)

Yep, done.
I removed one of the annotation, as it was too high up, and the current reset function doesn't pop, but rather resets completely, so it was a bit ugly.


Create a Bill of Materials (BOM)

As I'm writing the documentation for integration of Bucket4j into Spring Cloud Gateway, I'm realizing the awkwardness of users having to import one of the distributed options of Bucket4j and keeping the version used by Spring Cloud in sync. If a BOM was created, Spring Cloud Gateway could import it in its dependency management and users would get the versions of all the artifacts managed even though gateway doesn't depend on them (we just use core and caffeine for testing).
Various libraries already do this to great use. See many examples in spring boots dependency management. Here is one example https://github.com/spring-projects/spring-boot/blob/efee347604351bfd85d2c268befb80b47b120bff/spring-boot-project/spring-boot-dependencies/build.gradle#L27

Lack of the BOM was explicitly decision. I have following thoughts according BOM in the context of Bucket4j:
First of all, bucket4j-core does not depends from anything, so you do not need BOM if you do not use distributed facilities of bucket4j.
According to distributed facilities: I want to speculated that user typically chooses between bucket4j-hazelcat / bucket4j-redis / bucket4j-xxx / bucket4j-zzz based on own best expirience with particular xxx technology and technologies that is already adopted by his project. In my opinion, it is unlikely that user will prefer to use bucket4j-postgresql without already configured  postgresql driver in the classpath(similar for hazelcast/redisson/etc), and bucket4j should not obey user to choose concrete version of postgresql driver. The same for Hazelcast, user typically already have Hazelcast configured for his project when he deciding to choose bucket4j-hazelcast for his project, all he need - just to add bucketj-hazelcast as dependency.
Also, instead of forcing particular dependency version I follow following idea: while developing integration for xxx  I should utilize as small amount of features from xxx as it possible. This style usually helps for particular bucket4j-xxx integration to survive xxx upgrade, in mostly cases user is able to upgrade major releases of xxx without any problems with bucket4j-xxx. Sometimes it is not true, for example hazelcast-4 and hazelcast-5, for such cases usually I provide two versions of bucket4j-xxx like there. I am by myself working with huge enterprise and know how sometime it is hard to upgrade thirdparty, and it makes me sad when some popular library stops to support some technology that I am not able to upgrade currently. So having support of two different and incompatible versions of Hazelcast makes impossible to decide which concrete version I should fix into BOM, the same situation was a few years ago with Infinispan, and it is obviously that this situation can be reproduced t
And my final objection about introducing the BOM is following - bucket4j is not a framework, as well it is not a some kind of boxed product, it is just a library that user should adopt by hands. When you distribute a product, you clearly declare the environment/dependencies for which product is certified/tested. The same for framework, developer of framework takes its opinionated view to which concrete version of thirdparty his framework will support. For example bucket4j-spring-boot is framework and provides the BOM. While bucket4j is library comparing with framework - the main goal of a library providing maximum flexability, flexability of libraries allows to framework developers to compose huge amount of libraries in the framework, and it is responsibility of framework developer to make dicision about concrete third-pary version that need to choose.


build: always use the latest go and golangci-lint for gitpod [skip ci]

The Issue
It is easy to forget to update some packages in Dockerfile for Gitpod after release.
How This PR Solves The Issue
Try to use the latest version all the time.
Found the solution for go here.
And it was super easy for golangci-lint - I just removed the hard-coded version.
Manual Testing Instructions
$ docker buildx build -t ddev/ddev-gitpod-base --platform=linux/amd64 .gitpod/images

$ docker run --rm -it ddev/ddev-gitpod-base go version
go version go1.21.3 linux/amd64

$ docker run --rm -it ddev/ddev-gitpod-base golangci-lint version
golangci-lint has version 1.55.1 built with go1.21.3 from 9b20d49d on 2023-10-25T10:03:43Z

Automated Testing Overview

Related Issue Link(s)
Release/Deployment Notes

Followup will be to push a new version of the image, probably with a new tag, and update here and in ddev-gitpod-launcher. I think it would be OK to push now and add the new tag to this PR.

Pushed a new tag and updated it here and in ddev-gitpod-launcher.

gitpod /workspace/ddev (master) $ go version && ddev --version
go version go1.21.3 linux/amd64
ddev version v1.22.4

Thanks so much!


Profile amadeus

Summary
This PR is to provide a minimum baseline for the Entry Tasks
Detail
Code changes are made largely to the Profile/index.tsx file under pageContent. Details are as follows:

NavBar profile page: Added navbar slug to profile page
Banner image: Replaced banner image with a personal image
Paragraphs: Added two paragraphs, one short and one long
Email Form: Added a form to send email
Pictures of these changes are attached below.

Pictures / Gif

NavBar:

Banner image:

Paragraphs:

Form:

Checks

[x] Add reviewer

Todo for Week 4

[ ] Figure out how to align right the second paragraph
[ ] Automate centering of form (Not sure if it's already centered)

if you have just finished the comments, resolve the conversation and mark it as resolved or sth, jangan di leave dangling lmao


fetched all the data from contributors.json file to .html file [Closes #37]

Please check this PR.

Oh, ok.

check now

Sure!


Some methods of ot::Time can be error-prone to use

Describe the bug
Some API of ot::Time is error-prone to use.
GetDistancePast
    /**
     * This method returns a new `Time` instance which is in distant past relative to current `Time` object.
     *
     * The returned distance past `Time` is guaranteed to be equal or before (as defined by comparison operator `>=`)
     * any other `Time` which is before this `Time` object, i.e., for any `t` for which we have `*this >= t`, it is
     * ensured that `t >= this->GetDistantPast()`.
     *
     * @returns A new `Time` in distance past relative to current `Time` object.
     *
     */
    Time GetDistantPast(void) const { return Time(mValue - kDistantFuture); }

I believe that GetDistantPast() actually returns the same value as GetDistantFuture().
So following code can actually pass:
            t2 = t1.GetDistantFuture();
            VerifyOrQuit(t2 == t1.GetDistantPast(), "GetDistanceFuture() failed");
            VerifyOrQuit((t1 < t2 && t2 < t1), "GetDistanceFuture() failed");

Furthermore, even if we returned a correct value for GetDistantPast and GetDistantFuture, the Distant Past is going to become Distant Future in a few milliseconds. My suggestion is that we should remove this function since there is no use case of GetDistantPast in the current code base.
operator -
    /**
     * This method calculates the time duration between two `Time` instances.
     *
     * @note Expression `(t1 - t2)` returns the duration of the interval starting from `t2` and ending at `t1`. When
     * calculating the duration, `t2 is assumed to be in the past relative to `t1`. The duration calculation correctly
     * takes into account the wrapping of numeric value of `Time` instances. The returned value can span the entire
     * range of the `uint32_t` type.
     *
     * @param[in]   aOther  A `Time` instance to subtract from.
     *
     * @returns The duration of interval from @p aOther to this `Time` object.
     *
     */
    uint32_t operator-(const Time &aOther) const { return mValue - aOther.mValue; }

I think requiring that t2 is in the past relative to t1 is too strict for such a method. My suggestion is that we should use int32_t for all duration values so that we won't have such restriction. I think all uint32_t aDuration arguments of methods of Time should be changed to int32_t aDuration for better flexibility.
Thougths? @abtink

I believe that GetDistantPast() actually returns the same value as GetDistantFuture().

Yes, it is ok for them to return same value (it's known and intentional). They serve different purposes though. Note that both are always defined relative to a given Time instance.

GetDistantFuture() is ensured to be after all other times than are after the given Time instance (as if it is the maximum of all times that are ahead of the given Time instance).
GetDistantPast() is ensured to be before all other times that are considered before the Time instance (sort of like minimum of all times before the Time instance).

The main use for them is when trying to find the earliest/latest time from a given list of time instances that are known to be after/before a reference Time (you can see examples of this in code).

Furthermore, even if we returned a correct value for GetDistantPast() and GetDistantFuture(), the Distant Past is going to become Distant Future in a few milliseconds. My suggestion is that we should remove this function since there is no use case of GetDistantPast in the current code base.

The distant future/past times are always defined relative to some reference Time instance. Without a reference Time instance,  they are meaningless. So if we get current time now = GetNow(), then now.GetDistantPast() is only meaningful relative to its reference now value.
I prefer we keep GetDistantPast() even though it may not be used currently as a mirror of GetDistantFuture().

I think requiring that t2 is in the past relative to t1 is too strict for such a method. My suggestion is that we should use int32_t for all duration values so that we won't have such restriction. I think all uint32_t aDuration arguments of methods of Time should be changed to int32_t aDuration for better flexibility.

The fact that we require this and return uint32_t is indeed intentional.

We want to allow duration calculation between two time instance to be able to use the entire 32-bit range of values.
We do not want callers to use this method to do relative comparison of two time instances (i.e. is one after/before the other one) instead we want the caller to use the operators < , etc.

The main point of adding Time class (which is a practically a wrapper over a uint32_t value) is to handle the roll-over of the time value and provide correct relative time comparison (i.e. overloads of <, etc operators).

Before this we used raw  uint32_t value directly, and it was easy/common to make mistake - it's natural to simply check two time values t1, t2 with a t1 < t2 (I am myself guilty of this) - and the dangerous part is that may work ok until the time values get close to the roll-over range.
We provided a bunch of helper methods like IsBefore() or IsStrictlyBefore(), or int32_t Diff() etc, but still it was very error-prone, easy to make a mistake.
So with the Time class it is  intentional that we limited the APIs (e.g., did not allow int32_t operaor -). This is to force the users towards a single (and hopefully safer) pattern of use.

The distant future/past times are always defined relative to some reference Time instance. Without a reference Time instance, they are meaningless. So if we get current time now = GetNow(), then now.GetDistantPast() is only meaningful relative to its reference now value.

I can see why GetDistantFuture works now. But it's still very misleading. In the current implementation, for any value t, we have t < t.GetDistantFuture() and t.GetDistantFuture() < t. This is already confusing by itself. Moreover, t <= t.GetDistantFuture() and t.GetDistantFuture() >= t are both false, which I believe could surprise  most users.
In my opinion, a better solution is that we should avoid using the current DistantFuture value, but redefine kDistantFuture = ((1UL << 31) - 1). It will make the distant Time values for any time t to be more reasonable:

DistantFuture and DistantPast are now different values
Relationship of t and DistantFuture (DF) is straightforward (i.e. t < DF, t <= DF, DF > t, DF >= t)
Relationship of t and DistantPast (DP) is straightforward (i.e. t > DP, t >= DP, DP < t, DP <= t)

Thoughts?

We want to allow duration calculation between two time instance to be able to use the entire 32-bit range of values.

I think allowing duration to use the entire 32-bit range is actually not useful in most of time. For any time now, range [now+1, now+kDistantFuture) are recognized as future times, range (now-kDistantFuture, now-1] are recognized as past times. When we compare any time to now, we can never use the entire 32-bit range but only a range of kDistantFuture.
Furthermore, I believe using uint32_t type for Time has an implicit assumption that the distance between any two time values we handle in code should never exceed kDistantFuture, otherwise it's impossible to determine which is future and which is past. With such assumption, we should never expect time distances > kDistantFuture.

Before this we used raw  uint32_t value directly, and it was easy/common to make mistake - it's natural to simply check two time values t1, t2 with a t1 < t2 (I am myself guilty of this) - and the dangerous part is that may work ok until the time values get close to the roll-over range.
We provided a bunch of helper methods like IsBefore() or IsStrictlyBefore(), or int32_t Diff() etc, but still it was very error-prone, easy to make a mistake.
So with the Time class it is  intentional that we limited the APIs (e.g., did not allow int32_t operaor -). This is to force the users towards a single (and hopefully safer) pattern of use.

Agree with you that comparing directly on uint32_t time values is wrong. But I think we can have operator- and operator< both work in a normal way without any restriction. For any time values t1 and t2, if t1 < t2, then we can have t1 - t2 < 0 and t2 - t1 > 0.
The only exception to this rule will be time values of distance (1<<31). For any time t and t + (1<<31), there is no way to determine which is current. I think we can just not handle such case since it we should never have time values with such big time distance.

In my opinion, a better solution is that we should avoid using the current DistantFuture value, but redefine kDistantFuture = ((1UL << 31) - 1). It will make the distant Time values for any time t to be more reasonable:

DistantFuture and DistantPast are now different values
Relationship of t and DistantFuture (DF) is straightforward (i.e. t < DF, t <= DF, DF > t, DF >= t)
Relationship of t and DistantPast (DP) is straightforward (i.e. t > DP, t >= DP, DP < t, DP <= t)

Thoughts?

The reason for current model is we want DF relative to t to be the larger than any other value that considered larger than t.  The above definition with kDistantFuture = ((1UL << 31) - 1) would work but would then exclude the corner case of value half range (1<<31) away. Note that (as you pointed) the value of (1<<31)  is (intentionally) left as ambiguous, i.e., is treated as both larger and smaller than t.  A quick side-note on this:

I had a solution in mind (at the time of adding this) to make this unambiguous with the following rule: "when comparing t1 < t2 , if they are exactly half span (1<<31) apart, then use actual raw value to decide on which is larger".
But at the end decided not to add this since it was increasing code size (and complexity) for a corner case that should not be used and is not really practical.

I feel the current model is fine, but I don't have any objection to your suggestion for this (we anyways need to treat the (1 << 31) as a corner case that should not be practically used). If you feel strongly about this, please feel free to go ahead and change it. :)

Furthermore, I believe using uint32_t type for Time has an implicit assumption that the distance between any two time values we handle in code should never exceed kDistantFuture, otherwise it's impossible to determine which is future and which is past.

Initially in the design, we were considering two classes to ensure the notions are different and distinguished:

Having  Time class which is does not provide any comparison operators and the subtraction of two time instances gives a  uint32_t value.

The idea was to use it as timestamp of past events (e.g., to later use it in a frame history feature, or history of radio state changes, etc.).
In all these use-cases, the time instances would always be treated as if they are in the past,
And it would be useful to be able to use entire 32-bit range (specially with micro time where 2^32 micro sec is ~1.2 hour).

Then having a subclass TimeComparable class which now has the notion of past/future and operator overloads for <, etc.

At the end, we combined them into same class assuming that then the developer would choose how to use it.

Any inputs? @bukepo @jwhui

Closing stale issue.

Closing stale issue.


Sending a welcome message to new group members

Hi, how do I check if a new user is added to a group, using Telegram Bot starter kit? I'm planning to send a message every time someone joins a group chat.
Thanks in advance!

It is not possible AFAIK. Telegram bots do not receive such events.

I actually solved this already, there is a new_chat_participant parameter in the API, which can be used to message newly joined users in a group.

@Igetin Put it on your fork? :)

Would appreciate it if someone could give a example of the code.
Cant seem to get it to work.

how to check users added members count in telegram group using telebot python


How to skip test by criteria

like this
window.env = 'c1';
describe('test case', () => {
it('case 1', function() {
     ...
});

it('case 2', function() {
     ...
});

});
I want to skip case1 when window.env is 'c1', everyboy can help me?
thanks.

Wrap it in an if-statement. Tests are discovered synchronously, so if there's no call to it (hidden by if-statement), Jest won't pick it up.


Add image containing Firefox

When executing system tests with Selenium, one wants to have an image with a browser available, too.  We thus provide an image that also installs Firefox to fulfil this requirement.
@b-fein Personally, I believe this is extremely ugly, because I just copied the Dockerfile from the java image and added the firefox-esr package to the install list.  However, if I understood the build pipeline correctly, all images are built on top of base, which is reasonable; however, I'd like to have this one actually being built on top of java.  Do you know of a more clean solution, avoiding the duplicate code in the Dockerfiles, or is this just something we need to live with?

Do you know of a more clean solution, avoiding the duplicate code in the Dockerfiles, or is this just something we need to live with?

Hm, the base image is already passed in as a build argument
https://github.com/uni-passau-artemis/artemis-ci-images/blob/13c0bda7850f5cf5e09fcdc9a0aeb7c3857d08cc/.github/workflows/container-images.yml#L69C16-L70C83
We probably could make the base image an optional input https://github.com/uni-passau-artemis/artemis-ci-images/blob/13c0bda7850f5cf5e09fcdc9a0aeb7c3857d08cc/.github/workflows/container-images.yml#L6 which defaults to artemis-base but can then the be changed to artemis-java for the Firefox image.

Hm, the base image is already passed in as a build argument https://github.com/uni-passau-artemis/artemis-ci-images/blob/13c0bda7850f5cf5e09fcdc9a0aeb7c3857d08cc/.github/workflows/container-images.yml#L69C16-L70C83 We probably could make the base image an optional input
https://github.com/uni-passau-artemis/artemis-ci-images/blob/13c0bda7850f5cf5e09fcdc9a0aeb7c3857d08cc/.github/workflows/container-images.yml#L6
which defaults to artemis-base but can then the be changed to artemis-java for the Firefox image.

That's exactly what I was referring to.  Do you want to make this change with the optional inputs argument, or shall I give it a shot?


Should we store data of Debian releases apart from Jessie?

We are presently storing data for just Debian's Jessie release. Should we store data of other releases, too?

we should collect data from all available Debian versions... I am closing this older ticket now.


Remove incorrect windows wdk 26100.1879 version

Checklist for Pull Requests

[ ] Have you signed the Contributor License Agreement?
[ ] Is there a linked Issue?

Manifests

[x] Have you checked that there aren't other open pull requests for the same manifest update/change?
[x] This PR only modifies one (1) manifest
[ ] Have you validated your manifest locally with winget validate --manifest <path>?
[ ] Have you tested your manifest locally with winget install --manifest <path>?
[ ] Does your manifest conform to the 1.6 schema?

Note: <path> is the directory's name containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

URL: https://download.microsoft.com/download/8/c/a/8ca650fc-ee8a-4da2-ba42-c98c17f45883/KIT_BUNDLE_WDK_MEDIACREATION/wdksetup.exe
Response status code does not indicate success: 404 (Not Found).
(Automated message - build 896)


ctrl/stop is never called and even when it is, and returns a value, ctrl/start never receives it in prev-state?

So #19 seems to fix ctrl/stop being called, but I still am unable to get anything on prev-state in ctrl/start

This should be fixed in ba81de9e03903a18928497a8a6bbdd2f0329ef50 . I'll push a new release with this fix

Version 0.1.3 is released

thanks @retro


support pdf generation for manual document

Support PDF generate

base on asciidoctor-pdf
update README
resolve the image in the table get the error when generating PDF

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

Thank you for the contribution!
I've merged your branch (https://github.com/cuba-platform/documentation/commit/774cea4497fad9a36f1d5517b79b6d1fbd681875) and added a few related commits afterwards.


simplify piping the subprocess call output to devnull

 Here is an overview of what got changed by this pull request:

Complexity decreasing per file
==============================
+ analyzer/codechecker_analyzer/buildlog/host_check.py  -1

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Complexity decreasing per file
==============================
+ analyzer/codechecker_analyzer/buildlog/host_check.py  -1

See the complete overview on Codacy


Gatsby Build Error SSR

Hiya, I've been using tsparticles in development for my Gatsby project with no problems, however when i'm building for production i get a webpack error saying
Describe the bug
"ERROR #98124  WEBPACK
Generating SSR bundle failed
an't resolve 'tsparticles'
What i've tried
Delete Node-Modules & Yarn.lock and reinstall.
Reinstall just tsparticles and react-tsparticles
updating gatsby-node to =>
exports.onCreateWebpackConfig = ({ actions }) => {
actions.setWebpackConfig({
resolve: {
modules: [
path.resolve(__dirname, 'node_modules'),
path.resolve(__dirname, 'src'),
'node_modules'
],
},
})
}
Was hoping maybe i've just missed something silly with gatsby but would love to some help ! Thanks

Hi @JonasWN,
The error you have is making me scratch my head.

Generating SSR bundle failed Can't resolve 'tsparticles'

The SSR bundle shouldn't have tsparticles or react-particles since they won't work. But if the error is reported only when building production I think it's a misconfiguration of Webpack.
Can you make a demo project on codesandbox or stackblitz to reproduce the problem? I've never used Gatsby and I don't know how to replicate the issue

Hi @matteobruni
I tried setting it up on codesandbox with just a gatsby starter and react-tsparticles and got the same build error half the time and another random one with "command failed with exit-code 137", so i'm not sure how reliable it is to you sorry :d but here's the link if it's any help
Heres the error in codesandbox

I've tried using require instead of import aswell
I think you're right it's an webpack misconfig or something with it can't find the right path for some reason

I've released the 1.16.1 version, this version should fix the issue, there was a wrong path in tsParticles package.json main file.


Bump Halibut to v6.0.1457

This PR bumps Halibut to get some more detail in our logs, which will be useful for monitoring the Async Halibut rollout.

[sc-60356]


Wombat Delegate Example

trait Wombat
  fun box battle_call() : String val =>
    "Huzzah!"

class SimpleWombat is Wombat

actor Main is Wombat
  let w : SimpleWombat delegate Wombat = SimpleWombat

  new create(env : Env) =>
    env.out.print("Battle cry: " + battle_call())

Is this the best example of delegation in pony? Would jumping straight in with the type mixing be a better explanation? (Please excuse my being a bit of a tenderfoot with ponylang)

Yes, diving straight in with type mixing might be better Feel free to fork and contribute a PR!

I just found another Delegate example here, almost at the bottom:
http://tutorial.ponylang.org/examples/examples/

Delegate documentation removed for now from the tutorial.


feat(example): impl time-profiling

Motivation
We need time-profiling in the form of ark_std
Overview

remaster spans
add --json option for example
change default level of logs to INFO
add script for build profiling from logs
add cargo-alias for correct run of examples

cargo poseidon-example | python .scripts/build_profiling.py
# for all
cargo poseidon-example | python .scripts/build_profiling.py --min-busy 0s

Example of output:
cargo poseidon-example | python .scripts/build_profiling.py

Start:  poseidon_example
··Start:  get_or_create_commitment_key
··End:    get_or_create_commitment_key.....................................3.07s
··Start:  get_or_create_commitment_key
··End:    get_or_create_commitment_key.....................................2.40s
··Start:  pp_new
····Start:  primary
······Start:  circuit_collect_plonk_struct
······End:    circuit_collect_plonk_struct.................................1.39s
····End:    primary........................................................1.39s
····Start:  secondary
····End:    secondary......................................................1.92s
····Start:  digest
······Start:  digest_to_bits
······End:    digest_to_bits..............................................53.50s
····End:    digest........................................................53.50s
··End:    pp_new..........................................................56.80s
··Start:  ivc_new
····Start:  primary
····End:    primary........................................................1.40s
··End:    ivc_new..........................................................2.33s
··Start:  ivc_fold_step
····Start:  primary
······Start:  prove
········Start:  commit_cross_terms
········End:    commit_cross_terms.........................................2.51s
······End:    prove........................................................2.58s
······Start:  prove
········Start:  commit_cross_terms
········End:    commit_cross_terms.........................................7.38s
······End:    prove........................................................7.49s
····End:    primary.......................................................11.50s
····Start:  secondary
····End:    secondary......................................................1.33s
··End:    ivc_fold_step...................................................12.80s
··Start:  ivc_vefify
··End:    ivc_vefify.......................................................4.44s
End:    poseidon_example..................................................81.90s

Shall we add how to do runtime profiling in README?
cargo poseidon-example | python .scripts/build_profiling.py
# for all
cargo poseidon-example | python .scripts/build_profiling.py --min-busy 0s

Consider rename --min-busy to --min-runtime

Shall we add how to do runtime profiling in README?

I wanted to, but I thought I should add a section about usage in general first, since we have a public project and the whole README is focused not on the team, but on the audience

I wanted to, but I thought I should add a section about usage in general first since we have a public project and the whole README is focused not on the team, but on the audience

Makes sense. At the same time, it's good for external devs to know how to profile Sirius. Let's add profiling together with general usage later.

I wanted to, but I thought I should add a section about usage in general first since we have a public project and the whole README is focused not on the team, but on the audience

Makes sense. At the same time, it's good for external devs to know how to profile Sirius. Let's add profiling together with general usage later.

https://github.com/snarkify/sirius/issues/247


[Feature request] E-mail notifications

Is your feature request related to a problem? Please describe.
The SOC of our organization should be notified when there's a high level security incident reported by NeuVector. They work with e-mail, not webhooks, so there's really no way to integrate NeuVector events automatically for us right now.
Describe the solution you'd like
I'd like NeuVector to be able to send e-mail notifications similar to the webhook functionality.
Describe alternatives you've considered
Setting up a Teams channel and integrating via a webhook might be an alternative, but that would require introducing new ways of working for the SOC and that's a whole long process....

We also would like to see this feature.


Usage of composer

You'll need to update your dependencies.

[x] Require only the things needed from Symfony & Twig.

[x] Deploy the bundle on composer.

Done https://packagist.org/packages/ggteam/breadcrumbbundle.


Make the UTXO set re-entrant

This also needs to make sure it's memory-efficient. A simple (but naive) approach is to use a new Set for each time we return a utxoFinder delegate. But this would cause a lot of allocations.
Related to #194

Moving this out of Full Node and into Validator and removing it from the current kanban, because it's something we can think about doing at a later point. The re-entrancy might not be an issue right now since we use fibers, and we don't yield anywhere in the utxoFinder delegate. In theory it should just work.

This hasn't been needed for CoinNet as well, so moving out of the milestone.


amend tensorflow license to restrict Military Use

An amendment to the standard Apache License to exclude Military Use worldwide.
This version is based loosely on OCB's General License for Non-Military Software Implementations.

Once you've signed (or fixed any issues), please reply here (e.g. I signed it!) and we'll verify it.

I believe signing Contributor License Agreement (CLA) is not necessary for proposing changes only to the software license itself.

Drive by, from someone unaffiliated with the project or company, but concerned about the implications here:
I appreciate the sentiment, but also feel this sort of thing is much too far-reaching, and encroaches too much on the principles of open-source. Many organizations listed in this exclusion fund critical research far removed from warfare--astrophysics, particle physics, aviation, robotics, for ones I can name quickly off the top of my head.
More generally, I think any organization (including Google) can better serve responsible use of AI and ML-enhanced operations in governance and warfare though thought leadership, not carve-outs in a fair use policy. Removing TensorFlow as a (legal) tool for government organizations will not stop AI from being misused by them, and in worst case will cause them to use buggier software lacking the crowdsourced hardening of open-source projects that leads to life-ending mistakes in military applications.

Thank you for pull request. At this time, we remain committed to the open source Apache 2.0 license under which TensorFlow is offered.
We appreciate that AI brings up many issues of responsible use, please see ai.google/pair to learn more about our work to develop human-centered AI systems.


Fix grammar / spelling in comments

Minor fixes for changes introduced in #99257 that could not be fixed in time as the PR needed to be expedited.

Thanks!


Generic Functions Cause Translation Failure in Go2hx

Go2hx fails to translate Go code that uses generic functions with type parameters. When attempting to compile a program that uses generics, Go2hx produces an error indicating that the generic type cannot be found. Below is an example of code that works correctly in Go but fails in Go2hx.
Source Go code:
package main
import "fmt"
func main() {
    f[int]()
}
func f[T any]() {
    if []T(nil) != nil {
        fmt.Println("Slice is not nil")
    } else {
        fmt.Println("Slice is nil")
    }
    if (func() T)(nil) != nil {
        fmt.Println("Function is not nil")
    } else {
        fmt.Println("Function is nil")
    }
}

Expected Go Output:
Slice is nil
Function is nil

Translated Haxe Code:
package _internal;
function main():Void {
    _f((0 : stdgo.GoInt));
}
macro function _f<T_>(__generic__0:haxe.macro.Expr.ExprOf<T_>):haxe.macro.Expr.ExprOf<Void>;

Error Message in Haxe:
_internal/Runner.macro.hx:16: character 35 : Type not found : T_

Comment:
The error indicates that Go2hx does not handle generic type parameters (T) correctly. When translating the Go function f[T any](), Go2hx seems unable to define or pass the generic type T to the Haxe equivalent, leading to a Type not found error. Simplifying the Go code to remove generics (i.e., by replacing T with a concrete type such as int) allows the translation to succeed, indicating the issue is specific to generics.

Thank you for this @Yeaseen - you are correct, this aspect of Go has not yet been implemented within go2hx.
It is a live issue for the project.

Thank you for confirming this! @elliott5  I understand it’s a challenging aspect of Go to implement. I’m happy to help test this wonderful project.


Add a new differential expression for drift detection

What this PR does / why we need it:
Which issue(s) this PR fixes:
Fixes #4166
Does this PR introduce a user-facing change?:
NONE

Could you share the image of that drift detection?

It came out this way. What do you think?