Populate "Labels" project column

Populate the GitHub-native "Labels" project column for all issues:
Assumes existence of ./board.json, which can be obtained from the Trello board > "Show menu" > "<" (if showing) > "More..." > "Print and export"
See the Google Sheets Spec for details.

I dug into the JSON file a little bit, the current accessor on the sheet points to the hash of the label.
The actual text of the label is stored in cards[0].labels[0].name
Which way makes more sense?
Do we want to capture the whole 'label' object with color and hash or just the text?

@devPalacio, thanks for taking this on. In all instances we want to return actual names instead of their Trello ids. Thanks for asking.

Complete list of labels:
Paused, Meta, Completed


Analysis Refactor

Refactor the analysis to include some more of the data returned by Athena query,
Refresh some of the code,
Correct some of the mistakes

Update the PR opening comment, quite a bit has changed against original


[FLINK-33730][doc] Update the Flink upgrade savepoint compatibility table doc for chinese version

What is the purpose of the change
https://github.com/apache/flink/pull/23859 only updated the English version, this PR updates the zh doc.
Brief change log
[FLINK-33730][doc] Update the Flink upgrade savepoint compatibility table doc for chinese version

Thanks @reswqa for the quick review!
CI failed due to https://issues.apache.org/jira/browse/FLINK-35803, merging~


Enable LUA_USE_POSIX, fix tmpnam warning on macOS.

On macOS, we want to use mkstemp rather than tmpnam. Enable
POSIX support in Lua to fix that (and some other things). Since
POSIX support also enables the flag for POPEN, we no longer need
to do that directly for Linux.

Finally! I was getting tired of getting this message with every build.
$ bash build.sh
compiling (unix)...
linking...
/usr/bin/ld: src_lib_lua52_loslib.c.o: in function `os_tmpname':
/home/stefanos/git_codes/lite/src/lib/lua52/loslib.c:108: warning: the use of `tmpnam' is dangerous, better use `mkstemp'
cleaning up...
done

Thanks


Fix status.lastError.codes for Infrastructure

How to categorize this PR?

/area quality
/kind bug
/priority normal
/platform packet
What this PR does / why we need it:
Ref https://github.com/gardener/gardener-extension-provider-aws/pull/107
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Release note:

An issue preventing error codes to be properly populated in `.status.lastError.codes` for Infrastructure is now fixed.

@ialidzhikov Label "platform/packet" does not exist for this repository.


DSD-1699: new breakpoint values

Fixes JIRA ticket DSD-1699
This PR does the following:

Updates the values for the breakpoints and the associated docs.

How has this been tested?

local Storybook

Accessibility concerns or updates

n/a

Checklist:

[ ] I have updated the Storybook documentation accordingly.
[ ] I have added relevant accessibility documentation for this pull request.
[ ] All new and existing tests passed.

Front End Review:

[ ] Review the Vercel preview deployment once it is ready.

@bigfishdesign13 I updated this branch with the latest from development. Pull when you get a chance!


How to deal with this problem when i run it

https://blog.csdn.net/weixin_42045517/article/details/122224763


Logo Design Offer as Open Source Contribution

Hello Sir. I'm a Graphics and UI/UX Designer. I want to offering a new logo for your apps as my open source contribution. If you don't mind, I will create for you and post my design proposal here.

I will put a preview, after the repositories owner answer my offer. To make sure, this repositories still available

Thank you for your offer but we are absolutely not interested in a new logo.

Okay, no problem Sir.


InvalidApiVersionParameter creating subnets

Received the following response from Azure when trying to create a subnet.
{
	"error": {
		"code": "InvalidApiVersionParameter",
		"message": "The api-version '2015-06-15' is invalid. The supported versions are '2016-02-01,2015-11-01,2015-01-01,2014-04-01-preview,2014-04-01,2014-01-01,2013-03-01,2014-02-26,2014-04'."
	}
}

Other network related operations seem to success with that API version

I was mistaken. The error message is caused by some broken data in the request.

@gdhagger did you omit the name or something like that? If so we probably should return a friendlier error message.

Sort of...
The values are being computed by Terraform and a bug/feature on that side was resulting in either nothing or nonsense being entered for the resource group name.

@gdhagger was it the empty resource group name caused this problem? I appreciate any details, we may have a chance surfacing a better error message if we get a better sense of the issue.

Yes, I believe so.
On Wed, Mar 23, 2016 at 3:27 PM, Ahmet Alp Balkan<EMAIL_ADDRESS>wrote:

@gdhagger https://github.com/gdhagger was it the empty resource group
name caused this problem? I appreciate any details, we may have a chance
surfacing a better error message if we get a better sense of the issue.
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/Azure/azure-sdk-for-go/issues/297#issuecomment-200506478


Add IOMMU group information to resources API

Hi, I was not able to test this on a system with IOMMU enabled so I would appreciate if you could test it on your servers. Thanks! #7894

This pull request didn't trigger Jenkins as its author isn't in the whitelist.
An organization member must perform one of the following:

To have this branch tested by Jenkins, use the "ok to test" command.
To have a one time test done, use the "test this please" command.

Those commands are simple Github comments of the format: "jenkins: COMMAND"

Just two minor details to fix but otherwise looks good to me.

jenkins: test this please

jenkins: test this please

@nzubair76 thanks for your work!
If you're interested, here is the output on a machine that has IOMMU groups: http://paste.ubuntu.com/p/x9ghmnrJHr/


[bug] URL Query String Broken

Hello, I noticed that the URI Extensions helper methods duplicate some query strings causing the API to fail. I can open a pull request and fix if you want. Overall, I really like this project. It's super helpful.
Thanks
-Cameron

Thank! You can open a pull request to fix this, and I will further investigate


ECharts disappear from the overview bar of ioBroker when updating to 1.9.2

When I update 1.7.2 to 1.9.2 ECharts disappear from the bar. It still runs in Background.
Wehen I go back to 1.7.2., the Icon also come back to the bar.
ioBroker is 6.0.11
Node.js: v20.17.0
NPM: 10.8.2

Same here.
also direct link and settings are missing....

Bei mir das gleiche. Version 1.8.4 geht

Any news on this?
1.9.5 isn't working at all...


Customize autohide

on/off, delay

Im using this library in work for a project. I have demand for a custom off delay. I think only the off delay should be customisable. It should appear as soon as you start scrolling with no delay


Removing truncation and adding word-break so long service and

route names and route URLs wrap for consistency with app names
@spadgett

Sorry @rhamilto, you'll have to rebase :(

Rebase complete.


Add code documentation

We need standard code documentation. Comments (doxygen-friendly?) for the interface and API.

I will add semantic documentation on Proteus internals, and additional info on host/CUDA/HIP interfacing for making Proteus work.


Firebase Messaging QS - TIC TCP Conn Failed in iOS 11

Getting crash when build with Xcode 9.Please fix this issue .Not able to open the app
TIC TCP Conn Failed [3:0x604000363240]: 3:-9802 Err(-9802)

Hey, please update this ticket with a stack trace and/or repro steps, then reopen.

I am Getting error like this . on opening the app , app  getting closed with this error.I am not able to launch the app also.
TIC TCP Conn Failed [3:0x6000001660c0]: 3:-9802 Err(-9802)
TIC TCP Conn Failed [6:0x604000166f00]: 3:-9802 Err(-9802)
Please find the attached stack trace here.
StackTrace.pdf

+1


Change the color

Why is the background view of table view is white? Can we change the color?

Is there a dark mode feature to this? I was hoping I could use this but this is the only feature I need in order to incorporate it.

Use https://github.com/Juanpe/SkeletonView

Is there a dark mode feature to this? I was hoping I could use this but this is the only feature I need in order to incorporate it.

Use https://github.com/Juanpe/SkeletonView

Thanks!


deleted check for if name = "content"

Deleted check that checks if template name = 'content'. Thought to be left over from spacebars
https://github.com/mquandalle/meteor-jade/issues/145

I somehow missed your PR... Thank you @Orbyt this is merged and will be released on atmosphere soon :-)


SW-4710 Add enum values for cohort_phases

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

Current dependencies on/for this PR:

main

PR #1677

PR #1678   👈

PR #1679

This stack of pull requests is managed by Graphite.


Fix cross-compilation to the static Linux SDK

When cross-compiling a Vapor app with the Swift Static Linux SDK, postgres-nio will throw error: no such module 'Glibc'. This fix imports Musl instead of Glibc when it is available to fix cross-compilation.
Note: given other errors with the Static Linux SDK and Swift toolchain, you'll need to run at least the swift-6.0-DEVELOPMENT-SNAPSHOT-2024-09-17-a Swift development toolchain and corresponding Linux SDK. Also, there's an unreleased fix in swift-nio, so it's recommended to run at least the 1b33db2dea6a64d5b619b9e888175133c6d7f410 revision of the package when cross-compiling.

Any chance this could make it into a release soon(-ish)? We're trying to enable multi-platform docker builds for our server side applications and this is one blocker (that can currently be worked around by manually adding postgres-nio as dependency with a fixed commit hash to the current HEAD of main).


ANSI Color Code bug in Windows

Description of bug / unexpected behavior

ANSI Control code formatting for tracebacks and logs in manim seems to be corrupted when opened with command prompt or Powershell (manim version 0.11.0 and 0.12.0)
Expected behavior

Well the traceback should be formatted properly.
How to reproduce the issue

Any manim scene file works.
from manim import *

class SomeRandomText(Scene):
    def construct(self):
        text = MathTex(r"\mathbb{R}", fill_color="#FFFFFF").scale(4)
        self.play(Create(text))

manim -pql source.py -a
Additional media files

Images/GIFs

Logs
Terminal output

Something like:
Manim Community ←[32mv0.←[0m←[32m12.0←[0m

←[2;36m[11/17/21 10:17:40]←[0m←[2;36m ←[0m←[32mINFO    ←[0m Animation ←[32m0←[0m : Using cached data ←[1m(←[0mhash :<PHONE_NUMBER>_3925222406_1685565738←[1m)←[0m                                                 ←[2mcairo_renderer.py←[0m←[2m:←[0m←[2m74←[0m
[Create(MarkupText('1. A program to compile, (compilation of DVDs back in the days) an idea to\ndeduce a collection from a given pile and make it into a collection.')), Create(MarkupText('2. Your mother\n*. Okay'))]
←[2;36m[11/17/21 10:17:41]←[0m←[2;36m ←[0m←[32mINFO    ←[0m Animation ←[32m1←[0m : Using cached data ←[1m(←[0mhash :<PHONE_NUMBER>_1430383836_2029021001←[1m)←[0m                                                 ←[2mcairo_renderer.py←[0m←[2m:←[0m←[2m74←[0m
←[2;36m                   ←[0m←[2;36m ←[0m←[32mINFO    ←[0m Combining to Movie file.                                                                                ←[2mscene_file_writer.py←[0m←[2m:←[0m←[2m604←[0m
←[2;36m                   ←[0m←[2;36m ←[0m←[32mINFO    ←[0m                                                                                ←[2mscene_file_writer.py←[0m←[2m:←[0m←[2m718←[0m
                             ←[1;33mFile←[0m ready at ←[32m'D:\Programming\Python\Lectures\Compilers\media\videos\ep1\480p15\Episode1.mp4'←[0m

←[2;36m                   ←[0m←[2;36m ←[0m←[32mINFO    ←[0m ←[1;33mRendered←[0m Episode1                                                                                ←[2mscene.py←[0m←[2m:←[0m←[2m234←[0m
                             ←[1;33mPlayed←[0m ←[32m2←[0m ←[1;33manimations←[0m
←[2;36m                   ←[0m←[2;36m ←[0m←[32mINFO    ←[0m Previewed ←[1;33mFile←[0m at: ←[32m'D:\Programming\Python\Lectures\Compilers\media\videos\ep1\480p15\Episode1.mp4'←[0m                             ←[2mfile_ops.py←[0m←[2m:←[0m←[2m202←[0m

System specifications
System Details

OS: Windows 10 Home Single Language Version: 20H2
RAM: 8 GB
Python version (python/py/python3 --version): Python 3.9.6
Installed modules (provide output from pip list):

Package             Version
------------------- ---------
certifi             2021.10.8
charset-normalizer  2.0.7
click               8.0.3
click-default-group 1.2.2
cloup               0.7.1
colorama            0.4.4
colour              0.1.5
commonmark          0.9.1
decorator           5.1.0
glcontext           2.3.4
idna                3.3
isosurfaces         0.1.0
manim               0.12.0
ManimPango          0.3.1
mapbox-earcut       0.12.11
moderngl            5.6.4
moderngl-window     2.4.0
multipledispatch    0.6.0
networkx            2.6.3
numpy               1.21.4
Pillow              8.4.0
pip                 21.1.3
pycairo             1.20.1
pydub               0.25.1
pyglet              1.5.21
Pygments            2.10.0
pyrr                0.10.3
requests            2.26.0
rich                10.13.0
scipy               1.7.2
screeninfo          0.6.7
setuptools          56.0.0
six                 1.16.0
skia-pathops        0.7.1
tqdm                4.62.3
urllib3             1.26.7
watchdog            2.1.6

LaTeX details

LaTeX distribution (e.g. TeX Live 2020):
Installed LaTeX packages:

FFMPEG
Output of ffmpeg -version:
built with gcc 11.2.0 (Rev1, Built by MSYS2 project)
configuration: --enable-gpl --enable-version3 --enable-static --disable-w32threads --disable-autodetect --enable-fontconfig --enable-iconv --enable-gnutls --enable-libxml2 --enable-gmp --enable-lzma --enable-zlib --enable-libsrt --enable-libssh --enable-libzmq --enable-avisynth --enable-sdl2 --enable-libwebp --enable-libx264 --enable-libx265 --enable-libxvid --enable-libaom --enable-libopenjpeg --enable-libvpx --enable-libass --enable-libfreetype --enable-libfribidi --enable-libvidstab --enable-libvmaf --enable-libzimg --enable-amf --enable-cuda-llvm --enable-cuvid --enable-ffnvcodec --enable-nvdec --enable-nvenc --enable-d3d11va --enable-dxva2 --enable-libmfx --enable-libgme --enable-libopenmpt --enable-libopencore-amrwb --enable-libmp3lame --enable-libtheora --enable-libvo-amrwbenc --enable-libgsm --enable-libopencore-amrnb --enable-libopus --enable-libspeex --enable-libvorbis --enable-librubberband
libavutil      56. 70.100 / 56. 70.100
libavcodec     58.134.100 / 58.134.100
libavformat    58. 76.100 / 58. 76.100
libavdevice    58. 13.100 / 58. 13.100
libavfilter     7.110.100 /  7.110.100
libswscale      5.  9.100 /  5.  9.100
libswresample   3.  9.100 /  3.  9.100
libpostproc    55.  9.100 / 55.  9.100

Additional comments

In my estimation, this issue is caused by the ANSI Terminal Control support which was added to the Windows 10 console host in build 16257 (and later) being disabled by default.
By using os.system(""), the ANSI support gets enabled as per here.

As you can see, the Manim Community ←[32mv0.←[0m←[32m12.0←[0m gets printed with invalid formatting, and right after I executed the echo the formatting returns to normal.
from manim import *
from os import system

print("ENABLING ANSI")
system("")

class SomeRandomText(Scene):
    def construct(self):
        text = MathTex(r"\mathbb{R}", fill_color="#FFFFFF").scale(4)
        self.play(Create(text))

This may be useful: https://superuser.com/questions/413073/windows-console-with-ansi-colors-handling

Is this fixed for you now?

Unfortunately, not.
d:\Programming\Python\Lectures\Compilers>deps\Scripts\manim.exe --version
Manim Community ←[32mv0.←[0m←[32m15.0←[0m

d:\Programming\Python\Lectures\Compilers>deps\Scripts\manim.exe tests.py
Manim Community ←[32mv0.←[0m←[32m15.0←[0m

←[2;36m[03/06/22 20:01:58]←[0m←[2;36m ←[0m←[32mINFO    ←[0m Animation ←[32m0←[0m : Using cached data ←[1m(←[0mhash :  ←[2mcairo_renderer.py←[0m←[2m:←[0m←[2m75←[0m
<PHONE_NUMBER>_3464486584_223132457←[1m)←[0m

Aborted!

d:\Programming\Python\Lectures\Compilers>

That's unfortunate :(
Do you have a fix in mind?

Maybe an upgrade of rich would help; we should check whether they have any corresponding issues / PRs in their library repo.

Hey @Rickaym can you please file an issue regarding this on rich's repository here?


Worker node crash due to race on BuildRequestConfiguration.ActivelyBuildingTargets

Got a report of an MSBuild crash from the internal Visual Studio build:
System.InvalidOperationException: Collection was modified; enumeration operation may not execute.
   at System.ThrowHelper.ThrowInvalidOperationException(ExceptionResource resource)
   at System.Collections.Generic.Dictionary`2.KeyCollection.Enumerator.MoveNext()
   at Microsoft.Build.BackEnd.BuildRequestEntry.GetActiveTargets()
   at Microsoft.Build.BackEnd.BuildRequestEngine.IssueUnsubmittedRequests()
   at Microsoft.Build.BackEnd.BuildRequestEngine.<>c__DisplayClass60_0.<Builder_OnBlockedRequest>b__0()
   at Microsoft.Build.BackEnd.BuildRequestEngine.<>c__DisplayClass67_0.<QueueAction>b__0()

I have no idea what's causing the race, but it looks like a lot of the use of that field would be simplified if it were a ConcurrentDictionary instead of a plain one.


Fix README configuration object

Update README with valid configuration object;
Add CONTRIBUTING in README;
Add LICENSE in README.

Hey @emirotin,
Could you check this PR please, I think this update will help any users to config the mongodb-migrations more easier.


camerad: Use the ISP for debayering, save CPU / GPU / ram bandwidth.

Parts 1 and 2 of #24187 are prereqs for this. The ISP has a custom mini program to do things. Might be possible to get from Pixel 3.
See camera_fixups_isp for some work toward this.
There's two ISPs, so the dcam will have to be a little late, which should be okay. I suspect these ISPs can debayer super fast.

Would offer a $2,000 bounty for this.

Hi, is this still open? Could I be assigned?

Is this bounty still open or has Comma taken this into their own hands? I'm willing to work on this bounty because I have a reference Pixel 3 available.


adding a best solution for raw data

Description
Once we find the minimum of the QAOA cost function, we can access the most probable state. However, in many cases, it is not the solution with the minimum Hamiltonian cost from the set of bitstring outcomes. The additional function, I'm adding gives the bitstring with the minimum Hamiltonian cost.
Checklist

[x] I have performed a self-review of my code.
[x] I have commented my code and used numpy-style docstrings
[ ] I have made corresponding updates to the documentation.
[x] My changes generate no new warnings
[ ] I have added/updated tests to make sure bugfix/feature works.
[x] New and existing unit tests pass locally with my changes.

Type of change
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?

You formatted the result.py using python black, it seems like it 😄, looks good! At some point, I want to format the rest of the source code as well

@Q-lds, as you mentioned already, some tests are failing because the forks don't have access to the github secret api_token. Do we already want to try a possible solution for this PR itself?

No, I don't think there is a solution to that. They will be run after the merge into dev anyway

No, I don't think there is a solution to that. They will be run after the merge into dev anyway

Alright, I'll approve the changes then


Clear 2FA code from state when going back to login form

Details
The PasswordForm component was keeping any previously entered 2FA saved in the state, so when we go back to the login form and  send the password, a 2FA code was also being sent and when it's a wrong code the app displays the wrong error message that corresponds to a failed 2FA attempt.
I've changed ChangeExpensifyLoginLink to call clearSignInData which is a method in PasswordForm, which clears the local state and then calls Session.clearSignInData to clear Onyx state as well.
Fixed Issues
$ https://github.com/Expensify/App/issues/10571
Tests

Setup 2FA in an account.
Enter the email and click Continue.
Enter an incorrect 2FA code.
Click "Go back".
Enter email and click Continue.
You should see the input for 2FA instead of an error message saying incorrect 2FA code.

QA

Setup 2FA in an account.
Enter the email and click Continue.
Enter an incorrect 2FA code.
Click "Go back".
Enter email and click Continue.
You should see the input for 2FA instead of an error message saying incorrect 2FA code.

[x] Verify that no errors appear in the JS console

PR Review Checklist

Contributor (PR Author) Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] iOS / native
[x] Android / native
[x] iOS / Safari
[x] Android / Chrome
[x] MacOS / Chrome
[x] MacOS / Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product was added in all src/languages/* files
[x] I verified any copy / text that was added to the app is correct English and approved by marketing by tagging the marketing team on the original GH to get the correct copy.
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] If a new component is created I verified that:

[x] A similar component doesn't exist in the codebase
[x] All props are defined accurately and each prop has a /** comment above it */
[x] Any functional components have the displayName property
[x] The file is named correctly
[x] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[x] The only data being stored in the state is data necessary for rendering and nothing else
[x] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[x] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[x] All JSX used for rendering exists in the render method
[x] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG)

[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] I have checked off every checkbox in the PR author checklist, including those that don't apply to this PR.

PR Reviewer Checklist
The Contributor+ will copy/paste it into a new comment and complete it after the author checklist is completed

[ ] I have verified the author checklist is complete (all boxes are checked off).
[ ] I verified the correct issue is linked in the ### Fixed Issues section above
[ ] I verified testing steps are clear and they cover the changes made in this PR

[ ] I verified the steps for local testing are in the Tests section
[ ] I verified the steps for Staging and/or Production testing are in the QA steps section
[ ] I verified the steps cover any possible failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[ ] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)

[ ] I checked that screenshots or videos are included for tests on all platforms
[ ] I verified tests pass on all platforms & I tested again on:

[ ] iOS / native
[ ] Android / native
[ ] iOS / Safari
[ ] Android / Chrome
[ ] MacOS / Chrome
[ ] MacOS / Desktop

[ ] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[ ] I verified proper code patterns were followed (see Reviewing the code)

[ ] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick).
[ ] I verified that comments were added to code that is not self explanatory
[ ] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[ ] I verified any copy / text shown in the product was added in all src/languages/* files
[ ] I verified any copy / text that was added to the app is correct English and approved by marketing by tagging the marketing team on the original GH to get the correct copy.
[ ] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[ ] I verified the JSDocs style guidelines (in STYLE.md) were followed

[ ] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[ ] I verified that this PR follows the guidelines as stated in the Review Guidelines
[ ] I verified other components that can be impacted by these changes have been tested, and I retested again (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar have been tested & I retested again)
[ ] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[ ] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[ ] If a new component is created I verified that:

[ ] A similar component doesn't exist in the codebase
[ ] All props are defined accurately and each prop has a /** comment above it */
[ ] Any functional components have the displayName property
[ ] The file is named correctly
[ ] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[ ] The only data being stored in the state is data necessary for rendering and nothing else
[ ] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[ ] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[ ] All JSX used for rendering exists in the render method
[ ] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[ ] If a new CSS style is added I verified that:

[ ] A similar style doesn't already exist
[ ] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG)

[ ] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[ ] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[ ] I have checked off every checkbox in the PR reviewer checklist, including those that don't apply to this PR.

QA Steps

[ ] Verify that no errors appear in the JS console

Screenshots
Web
https://user-images.githubusercontent.com/9680864/187722579-1564080c-aba1-4880-9e88-d54c221b7974.mov
Mobile Web
https://user-images.githubusercontent.com/9680864/187724838-ccb13f1d-ea4d-4e6c-97c9-9a50986c57af.mov
https://user-images.githubusercontent.com/9680864/187727376-e0aa48a9-983b-42e9-87e4-e808f5546f09.mov
Desktop
https://user-images.githubusercontent.com/9680864/187724538-ad50674b-c71f-419f-92d5-1209ca1c7484.mov
iOS
https://user-images.githubusercontent.com/9680864/187724090-1fbc6478-c006-476e-9aec-470213427486.mov
Android
https://user-images.githubusercontent.com/9680864/187727572-eb2bbeb8-6ecb-4314-ae08-f29511088247.mov

@youssef-lr please add steps in the Test and QA sections

@aldo-expensify working on screen recordings and tests atm!

@aldo-expensify all set!

Changes applied, thanks!

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to staging by @youssef-lr in version: 1.1.97-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
failure ❌

🕸 web 🕸
success ✅


Add GitHub to showcase

Lighthouse shows Github uses lit-html on  many of it's pages.
I found it on:

/profile-name
/explore

Thanks for letting us know!
Right now for addition to that section we require some kind of public, linkable, acknowledgment that the project uses Lit, not just usage discovered from analysis. We'll see if we can get that for GitHub.

Thanks for letting us know!
Right now for addition to that section we require some kind of public, linkable, acknowledgment that the project uses Lit, not just usage discovered from analysis. We'll see if we can get that for GitHub.

Ohh! Cool! 🤟


[BUG]  strtolower(): Passing null to parameter #1 ($string) of type string is deprecated in vendor/chillerlan/php-qrcode/src/QRCode.php on line 218

Describe the bug or unexpected behaviour
As title discribes, getting the following error on my php 8.1 setup:
Deprecated:  strtolower(): Passing null to parameter #1 ($string) of type string is deprecated in vendor/chillerlan/php-qrcode/src/QRCode.php on line 218
Steps to reproduce the behavior
Run on php 8.1, render
Seems to only occur (breakingly) when have error reporting on:
ini_set('display_errors', '1');
ini_set('display_startup_errors', '1');
error_reporting(E_ALL);

Environment (please complete the following information):

PHP version/OS: 8.1, Linux
Library version: 4.3.2

this bug is fixed in v4.3.3 (#97, #110) - please update to the latest 4.x version.


Sets tree batch1

Developed the first 8 components of the tree set.
Updated the ReadMe file.

Awesome, thanks a lot @sonomirco, this is really good quality work! The code is very well documented and thorough, great job! 👍 💯
A couple minor comments:

For Simplify Tree, perhaps the input data could have a set of initial zeroes {0;0;0} {0;1;0} {0;2;0}..., so that it is easier to see the difference when Front is active (everything is cleaned) or not (only the initial zeroes are).
Remember when finishing the file to deactivate the preview of all components; we do not want users opening the file and having a ton of geometry on the viewport. This is not so important for this category, but more important for others.
Since the Sets.Tree category has components arranged in two columns, you may want to arrange them that way as well on the file as well:

Keep it up! 💪

What would you like to do? Would you like to incorporate feedback and merge, or keep working on this PR for the rest of the components?

I'll prefer to incorporate the feedback and merge.
I'll work for batches so it would be easier to review and incorporate feedback.

incorporated the feedback, you can marge when you want, and I'll start with the second batch.
Mirco

Alright, merged!! Thanks a lot @sonomirco ! 🥳 🥳 🥳 🥳
Please make sure to update your local fork before starting a new PR.
Cheers!


[1.x] Fix incorrect package import

Description
Backports have been failing since 2.x backport of getActions PR on OpenSearch side.
This fixes them.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

This is incorporated into #628, but root cause is that #606 hasn't been merged.


The recovery of vis has absolutely stopped in the world

I apologize in advance, I do not speak English and I write through a translator. Thank you.
At some point during the game, Vis stopped regenerating in the world. Fully. In all chunks, even in unloaded ones. That is, there is no passive recovery. By the way, silver trees and vis regenerators also do not work.

You need to set randomtickspeed >0 and doDayLightCycle true.


feat(ui): Animate field messages

Description

[ ] CSS-based entrance and exit animations
[ ] JS-based height animations
[ ] Defer unmount for cross-animation

Checklist

[ ] npm test runs as expected.
[ ] npm run build runs as expected.
[ ] (If applicable) JSDoc comments have been added or updated for any package exports
[ ] (If applicable) Documentation has been updated

Type of change

[ ] 🐛 Bug fix
[ ] 🌟 New feature
[ ] 🔨 Breaking change
[ ] 📖 Refactoring / dependency upgrade / documentation
[ ] other:

Closing for now as this is managed by the Animated component added here https://github.com/clerk/javascript/pull/4017


Users are still able to log in using a retired email account

Steps to Reproduce
We came across a ticket where the learner requested to have their accounts deleted / removed from the system:
<EMAIL_ADDRESS><EMAIL_ADDRESS>We followed the standard retire users process by, running the retire_users management command unchecking the "Active" status on each of the accounts in Django:
~ $ python manage.py retire_users -u<EMAIL_ADDRESS>Retiring user: User username=mit0127<EMAIL_ADDRESS>User: 'User username=mit0127<EMAIL_ADDRESS>is already deactivated in MIT mitX Online
~ $ python manage.py retire_users -u<EMAIL_ADDRESS>Retiring user: User username=OP0127<EMAIL_ADDRESS>User: 'User username=OP0127<EMAIL_ADDRESS>is already deactivated in MIT mitX Online
Expected Behavior
The learner will not be able to log into the platform with the retired email accounts. The email accounts will be marked as retired in Django when looked up in https://mitxonline.mit.edu/admin/users/user/

Actual Behavior
This learner is still able to log into the system, and the user account email does not get marked as retired:

https://mitxonline.mit.edu/admin/users/user/54903/change/?_changelist_filters=q%3Dsiuuuuu
Related Issues
(https://odl.zendesk.com/agent/tickets/152865)

@cachob you should be able to deactivate these users now.


sox spectogram

can you update spek with sox using this script
check last answer
https://unix.stackexchange.com/questions/200168/how-to-create-the-spectrogram-of-many-audio-files-efficiently-with-sox

Why, though? Does that script provide something better than the current script?

some time failed generate spektogram
https://graph.org//file/db0ea3ba5392c0fba2f60.png
https://i.ibb.co/TR5KVwK/Jepretan-Layar-2023-06-08-pukul-11-49-27.png

Can you provide me that audio in which you get errors?


Can one set the parameters for noise during simulation?

What is the expected behavior?
I take a close look at the documentation, and can't find how. But from the c++ code, it seems possible.
Can one set the parameters(like how large amplitude damping is) for noise during simulation?

Yes, you can do this:
from qiskit import *
q = QuantumRegister(2)
c = ClassicalRegister(2)
circ = QuantumCircuit(q, c)
circ.h(q[0])
circ.cx(q[0], q[1])
circ.measure(q[0], c[0])

config = {
    'noise_params': {
        'relaxation_rate': 1/50, #us
        'thermal_populations': [0.99, 0.01],
        'CX': {'gate_time': 0.1},
        'U': {'gate_time': 0.01},
        'U': {'p_pauli': [0.01, 0, 0]}
    }
}
sim = 'local_qasm_simulator_cpp'
result = execute(circ, sim, config).result()
result.get_counts()

See here for how to set the noise_params with different noise models:
https://github.com/QISKit/qiskit-terra/tree/master/src/qasm-simulator-cpp#noise-parameters
Please reopen the issue if your problem persists.

Thanks for the answer! Could you please elaborate on how to simulate amplitude damping error?

At the moment it is not real amplitude damping. We are working on this and we want to start a convention of "types" of noise. The current simulator (and yes its undocumented as we are still deciding on how the general user will interact with it) only does type 1 noise. This is noise where gates can be randomly added to make a noisy quantum circuit. For cases where you have both t1 and t2 then it is possible to find a condition where you can exactly predict amplitude damping with only random elements added to the circuit (@chriseclectic can give more details). Type 2 will be for Kraus operators and that will do all cases of amplitude damping. All this will become clearer in the next few updates to qiskit-terra but for now as @ajavadia pointed out this is the best way to use this undocumented noise feature of qiskit-terra. When we get closer to making a solid plan for these noises we will start discussion options on the qiskit slack channel. Sorry we have not fully worked out the user interface to the noise part of the simulator but we wanted to get the backends and providers standardized first.

look forward to Kraus operators!
@chriseclectic It would be great if you could give some explanations on how to simulate amplitude damping channel using random gates!

Hi, any updates on this?

@LionSR Qiskit Aer will be released in a month or two and will have the features you want in terms of noise models. About approximation of error channels, see this paper:
https://arxiv.org/pdf/1207.0046.pdf

@ajavadia Hi, I've been using the 'noise_params' for simulating single qubit gate error and the result indeed gives a noisy quantum computation. Yet, when I only specify the 'calibration_error' or 'zz_error' for 'CX', the simulator returns a perfect simulated quantum computation without noise, may I know what is the problem? Furthermore, for the Qiskit Aer, what are the syntaxes in specifying 'noise_params'

@ajavadia For an example, the following code
https://github.com/gxchan3/qiskit/blob/6dc8c48284d01df30ab579f4665156d555383b1c/topost.py#L1
gives result:
{'00': 1024}
{'00': 249, '01': 243, '10': 286, '11': 246}
{'00': 1024}

@gxchan3 The noise model construction will change quite drastically (for the better) with the release of Qiskit Aer, more information on that will come closer to its release. For your issue with coherent error matrix, the short-hand of using calibration error parameters was removed some time ago, and you need to specify the coherent error matrix using the "U_error" field.
There are functions to generate the old CX coherent error matrix for calibration and zz error matrix:  cx_error_matrix(cal_error, zz_error) and x90_error_matrix(cal_error, detuning_error) which can be imported from qiskit.backends.aer.qasm_simulator.
For example:
import qiskit
from qiskit.backends.aer.qasm_simulator import cx_error_matrix, x90_error_matrix

noise_params = {
    "X90": {"U_error": x90_error_matrix(0, 0.2)},
    "CX": {"U_error": cx_error_matrix(0.01, 0.05)}
}

print(noise_params)

# Output:
{'X90': {'U_error': array([[ 0.99003329-0.09933467j, -0.09933467+0.00996671j],
       [ 0.09933467+0.00996671j,  0.99003329+0.09933467j]])
  },
'CX': { 'U_error': array([[ 0.99935452-0.02511786j,  0.        +0.j        ,
         0.02511786+0.00535979j,  0.        +0.j        ],
       [ 0.        +0.j        ,  0.99935452-0.02511786j,
         0.        +0.j        , -0.02511786-0.00535979j],
       [-0.02511786+0.00535979j,  0.        +0.j        ,
         0.99935452+0.02511786j,  0.        +0.j        ],
       [ 0.        +0.j        ,  0.02511786-0.00535979j,
         0.        +0.j        ,  0.99935452+0.02511786j]])
}}

@chriseclectic Thank you for your comment! I'll read and try the code if it fits my requirement for the error model.

@ajavadia For an example, the following code
config = {
    'noise_params': {
        'relaxation_rate': 1/50, #us
        'thermal_populations': [0.99, 0.01],
        'CX': {'gate_time': 0.1},
        'U': {'p_pauli': [0.01, 0, 0],'gate_time': 1}
    }
}

beta=pi/2+pi/2
gamma=0.5
config1 = {
    'noise_params': {
        'CX': {"calibration_error": beta,"zz_error": gamma}
    }
}
sim = 'qasm_simulator'

job = execute(twoQubitRB,Aer.get_backend(sim))
result = job.result()
count=result.get_counts()

job = execute(twoQubitRB,Aer.get_backend(sim), config)
result = job.result()
count1=result.get_counts()

job = execute(twoQubitRB,Aer.get_backend(sim),config=config1)
result = job.result()
count2=result.get_counts()

print(count)
print(count1)
print(count2)

gives result:
{'00': 1024}
{'00': 249, '01': 243, '10': 286, '11': 246}
{'00': 1024}
FYI, my quantum program will gives 100% probability of state '00' after measurement if noise is not present during the quantum computation

@gxchan3 Hi, I ran your code starting with state |00> and measurements on both qubits. For all three cases I got the result {'00': 1024}. I ran it on both my laptop and the Binder image. May I know what is the problem?

Any updates on Qiskit Aer?


Near dependency crates inconsist with workspace

Hi,
I am new to NEAR and run into an issue with near-sdk and workspace with the following problem
Error: No such file or directory (os error 2)
thread 'round_trip' panicked at 'assertion failed: `(left == right)`
  left: `1`,
 right: `0`: the test returned a termination value with a non-zero status code (1) which indicates a failure', /rustc/e092d0b6b43f2de967af0887873151bb1c0b18d3/library/test/src/lib.rs:185:5

I am not sure the following dependency issue cause on this, but the near dependency crates are inconsistent between workspace and near-sdk.
[package]
name = "near-sdk"
version = "4.0.0"

[target.'cfg(not(target_arch = "wasm32"))'.dependencies]
near-vm-logic = "0.13"
near-primitives-core = "0.13"
near-primitives = "0.13"
near-crypto = "0.13"

[package]
name = "workspaces"
version = "0.4.0"

[dependencies]
near-account-id = "0.14.0"
near-crypto = "0.14.0"
near-primitives = "0.14.0"

If this is not an issue, could you kindly show me how to select a workable match on the version workspaces and the version of near-sdk.
Thanks in advanced.  :pray:

This is just an error with workspaces, so I will transfer the issue there. This is primarily just an issue of error details not being reported
cc @ChaoticTempest, it's not ideal that these versions are not matched, as it will include duplicates of this for anyone using. We should try to keep these in sync and not just update to the most recent as https://github.com/near/workspaces-rs/issues/164
I will update the SDK versions to 0.14 for 4.1, but we should avoid bumping this when possible


fix time zone errors

Closes #53

Integration tests are not working at the moment pending https://github.com/assert-rs/assert_cmd/issues/139


Cesium ION SDK 1.97 - Service Worker importScripts console errors

Browser: Chrome Version 104.0.5112.102
Operating System: Windows 11
Hi, I am currently trying to update from Cesium 1.92 to Cesium 1.97.
We are consumers of the Cesium ion SDK, which I currently reference by script tag to a local path where the Cesium ion SDK Build/Cesium artefacts are copied.
<script src="~/lib/Cesium/1.97/Cesium.js" asp-append-version="true"></script>

Since v1.96 I am getting the below console errors related to service workers. From what I understand with the AMD module changes we should still be able to reference Cesium JS via script tag with no further changes, or is additional configuration required?
I can see that the content of the /Workers folder has changed from a number of individual js file to a single cesiumWorkerBootstrapper.js file?
Can you please advise if I need to make changes to the way I reference the file or if this is an issue with version 1.97 itself.
Regards,
Glenn
cesiumWorkerBootstrapper.js:formatted:96 Uncaught Error: importScripts failed for Workers/decodeDraco at https://localhost:5001/lib/Cesium/1.97/Workers/decodeDraco.js
http://requirejs.org/docs/errors.html#importscripts
    at makeError (cesiumWorkerBootstrapper.js:formatted:96:17)
    at req.load (cesiumWorkerBootstrapper.js:formatted:815:27)
    at Object.load (cesiumWorkerBootstrapper.js:formatted:723:21)
    at c.load (cesiumWorkerBootstrapper.js:formatted:368:19)
    at c.fetch (cesiumWorkerBootstrapper.js:formatted:362:68)
    at c.check (cesiumWorkerBootstrapper.js:formatted:374:59)
    at c.enable (cesiumWorkerBootstrapper.js:formatted:517:22)
    at Object.enable (cesiumWorkerBootstrapper.js:formatted:673:27)
    at c.<anonymous> (cesiumWorkerBootstrapper.js:formatted:510:60)
    at cesiumWorkerBootstrapper.js:formatted:77:22
makeError @ cesiumWorkerBootstrapper.js:formatted:96
req.load @ cesiumWorkerBootstrapper.js:formatted:815
load @ cesiumWorkerBootstrapper.js:formatted:723
load @ cesiumWorkerBootstrapper.js:formatted:368
fetch @ cesiumWorkerBootstrapper.js:formatted:362
check @ cesiumWorkerBootstrapper.js:formatted:374
enable @ cesiumWorkerBootstrapper.js:formatted:517
enable @ cesiumWorkerBootstrapper.js:formatted:673
(anonymous) @ cesiumWorkerBootstrapper.js:formatted:510
(anonymous) @ cesiumWorkerBootstrapper.js:formatted:77
each @ cesiumWorkerBootstrapper.js:formatted:45
enable @ cesiumWorkerBootstrapper.js:formatted:489
init @ cesiumWorkerBootstrapper.js:formatted:343
(anonymous) @ cesiumWorkerBootstrapper.js:formatted:629
setTimeout @ cesiumWorkerBootstrapper.js:formatted:26
req.nextTick @ cesiumWorkerBootstrapper.js:formatted:762
o @ cesiumWorkerBootstrapper.js:formatted:625
requirejs @ cesiumWorkerBootstrapper.js:formatted:754
self.onmessage @ cesiumWorkerBootstrapper.js:formatted:19
cesiumWorkerBootstrapper.js:formatted:21 Uncaught TypeError: Cannot read properties of undefined (reading 'baseUrl')
    at cesiumWorkerBootstrapper.js:formatted:21:42
    at Object.execCb (cesiumWorkerBootstrapper.js:formatted:726:26)
    at c.check (cesiumWorkerBootstrapper.js:formatted:388:43)
    at c.enable (cesiumWorkerBootstrapper.js:formatted:517:22)
    at c.init (cesiumWorkerBootstrapper.js:formatted:343:50)
    at cesiumWorkerBootstrapper.js:formatted:629:27
    at setTimeout (cesiumWorkerBootstrapper.js:formatted:26:5)
    at req.nextTick (cesiumWorkerBootstrapper.js:formatted:762:9)
    at Object.o [as require] (cesiumWorkerBootstrapper.js:formatted:625:23)

Hi @ggetz
I'm referencing the files/folders from the Build directory of the Ion SDK. These are all copied to the /lib/Cesium/1.97 folder as a pre-build step, I don't explicitly set the CESIUM_BASE_URL as everything resolves by the relative paths in this structure.
The only exception at the moment, is I am copying the Build/Workers folder from the standard Cesium JS library as a workaround for the errors I am getting when using the Cesium Ion Workers folder.
/lib/Cesium/1.97/Assets/
/lib/Cesium/1.97/ThirdParty/
/lib/Cesium/1.97/Widgets/
/lib/Cesium/1.97/Workers/
/lib/Cesium/1.97/Cesium.js
Regards,


PR: Add support for Python 3.8

Fixes #123

Python 3.8 is available on appveyor now, so the build should work.

@andfoy has this build been retried after the update to appveyor?


docs: add breaking change note about calling scheduleBuilder with @angular-devkit/build-angular

close #22174

Thanks for your contribution.


Cache not updating

I have caching enabled
cache_options enabled: true, cache_length: 1.hour
and on the model I have the method
def cache_key
    "#{model_name.cache_key}/#{id}-#{updated_at.to_i}"
  end

but my object doesn't update/clear the cache after an hour.
Is it stored in Redis? When I clear the redis cache it doesn't update either.
I have the option config.cache_store = :redis_cache_store set in config production

What if you change the cache_key to remove the model_name.cache_key/ and just leave the latter half of the key? Does that still result in same behavior?
Also, is this a model that has a lot of relationships to other models, or fairly stand-alone?


[Attestation Service] Investigate 10DLC requirements -- issue guidance

Expected Behavior
Providers can send SMS to all US phones on all networks
Current Behavior

Verizon is introducing something which seems to have triggered additional KYC reqs for Nexmo. Risk is misconfigured providers that are set up but always fail
Twilio seems unaffected? https://support.twilio.com/hc/en-us/articles/360008372654-Commercial-Long-Code-SMS-MMS-Product-and-Fee-Structure-Changes-on-Verizon

Public guidance is located here


The code in https://github.com/mishk0/slack-bot-api/blob/master/index.js#L8, is this:
var EventEmitter = require('events').EventEmitter;
But all I can find from Nodejs docs is const EventEmitter = require('events'). And Class Foo extends EventEmitter  {}.
I am not sure what require('events').EventEmitter means.
Would you please elaborate a little bit ? I really don't understand the difference. Thanks.

Never mind.  Sorry.


Tears of Themis

IDK, if this is the right place, but if ZZZ is going to be added why not add every Hoyo game? It seems like ToT is the only one missing then.
I saw an old pull request about that and added it to my code but it doesn't seem to work (https://github.com/canaria3406/hoyolab-auto-sign/pull/13), my Discord message always says "Too many attempts. Try again later", while every other says "OK" or smth like "You've already checked in today, Trailblazer~/Traveler~/Captain~/Proxy~".
Is it possible to add this game too, or is there maybe a problem with why it doesn't work?

maybe need problem is send too many attempts,it need a wait time to login to get price

but I don't know how to write latency time in Google Apps Script,if it has wait time,it can work

To fix the issue temporarily,you can create more projects and perform check-in tasks separately (up to two games per task),the projects need run in different times

Merge pull request #45


added calling setRootPaths on template engines when viewsPath changes

For issue https://github.com/IBM-Swift/Kitura/issues/1115.

Codecov Report

Merging #1119 into master will decrease coverage by 0.14%.
The diff coverage is 40%.

@@            Coverage Diff             @@
##           master    #1119      +/-   ##
==========================================
- Coverage   89.98%   89.83%   -0.15%
==========================================
  Files          35       35
  Lines        1717     1722       +5
==========================================
+ Hits         1545     1547       +2
- Misses        172      175       +3

Flag
Coverage Δ

#Kitura
89.83% <40%> (-0.15%)
:arrow_down:

Impacted Files
Coverage Δ

Sources/Kitura/Router.swift
80.51% <40%> (-1.36%)
: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 c243516...33fb300. Read the comment docs.

👍


Reduce verbosity in base models

Every property in a base model class gets these definitions:
  private _firstName?: string;
  get firstName(): string {
    if (this._firstName === undefined) throw new Error('Property firstName is not selected');
    return this._firstName;
  }
  set firstName(value: string) {
    this._firstName = value;
  }

I wonder if there's a better way do handle this that would be less verbose. Models with a ton of properties would seem pretty messy, but luckily developers won't be touching the base model.

Defer error throwing to Selectable decorator. Field declarations now look like:
@Selectable() attackerCharacterIds!: string[];


Collision: don't load Surface without

There was a minor behavior change in Bitbucket pull request 660 that started adding empty <surface> elements to the sdf element in a Collision DOM element if it didn't already have a <surface> element. This has caused some issues for downstream users ( https://github.com/RobotLocomotion/drake/pull/13201 ), and it's a minor thing to change, so I've reverted the behavior change. I also recommended where to change the downstream code to be less brittle ( https://github.com/RobotLocomotion/drake/pull/13201#issuecomment-622613872 ).
This was originally reported in https://github.com/osrf/sdformat/issues/267.

https://github.com/RobotLocomotion/drake/pull/13218 confirms that this removes the behavior change, at least insofar as Drake notices.


Fix spaghetti code for register_autograd_neighbors

I don't fully understand when register_autograd_neighbors is needed and when it is not. For now, I've put together some conditional spaghetti code that works for all 4 models at training and at evaluation, but it would be good to clean it up

register_autograd_neighbors it's needed everytime you compute your neighbor list with something else than torch code (i.e. whenever neighbor.values does not have a grad_fn). I though this repo always used the ASE NL, and as such register_autograd_neighbors is always needed?
It is also (almost) free if a model doesn't use it. Forward does nothing, and backward is only called if it enters the calculation graph. So IMO this function should always be called.

Yes but it throws errors if I use it in all cases

Hmm, it should not unless the neighbor list has some issues. If you have an example where it fails I can give it a look!


MIDDLEWARE_CLASSES is deprecated

Hello,
The MIDDLEWARE_CLASSES setting is deprecated in Django 1.10 and the MIDDLEWARE setting takes precedence. Projects using MIDDLEWARE cannot use the nplusone middleware

experiencing the same issue.


Make logs link more noticeable

From #285,

Thanks I didn't notice this link, I think it could be made more noticeable (e.g. using the same style blue+underline for the link, and personally I think that the name of the build "110849-ocluster-build-87cd3f" only adds some "noise" and is not very useful for the end user, the link could just be named "execution logs").

Right now there;s ids in the logs that doesn't help the user figure out that the execution logs is indeed something useful. Perhaps we can add just a Logs link and hide the uuids?

cc @art-w @punchagan What are your thoughts? :)

Yes, that makes sense to me. The IDs are useful if we are trying to find the log on the server. But, I think we can get that out of the URL too.


VScode remote jsconfig paths optoin doesn't work

VSCode Version: Version 1.36.0-insider (1.36.0-insider)
OS Version: Mac 10.13.6 (17G65)

Steps to Reproduce:

Create a jsconfig.json file as below
Open a file with an import using the @ alias
Intellisense can't find imported files

My jsconfig.json:
{
  "compilerOptions": {
    "target": "es6",
    "baseUrl": ".",
    "paths": {
      "@": ["./src"]
    },
  },
  "exclude": [
    "node_modules"
  ],
}

Does this issue occur when all extensions are disabled?: Yes
Couldn't find any other issue related to this here or on StackOverflow. Thanks.

Looks like it's working in the stable channel though (1.35.1). Letting you close the issue if it's OK.

That error looks like it is coming from the eslint extension. Please start by filing an issue against that extension


Small code refactor

Description

Small code refactor

Checklist

[ ] I have linked to any relevant issues.
[ ] I have commented my code, particularly in hard-to-understand areas.
[ ] I have updated the documentation where relevant (API docs, the reference, and the Sway book).

[ ] If my change requires substantial documentation changes, I have requested support from the DevRel team

[ ] I have added tests that prove my fix is effective or that my feature works.
[ ] I have added (or requested a maintainer to add) the necessary Breaking* or New Feature labels where relevant.
[x] I have done my best to ensure that my PR adheres to the Fuel Labs Code Review Standards.
[x] I have requested a review from the relevant team or maintainers.

This isn't really worth considering on its own, if you want to include a bunch of changes like this it can be a PR, but not for two lines.
Also please stop opening a million PRs under the same nondescript name. I appreciate the contributions but not the spam. Please collate them in one (not too large) refactor PR and then we can efficiently review it.


Support Landsat 9 data

Add support to Landsat 9 data

When i run the PySEBAL using your test data it is doing it perfectly but when i put my landsat 8 images it is giving me an error. please can you assist what could be the problem. Attached is the screen shot of the error


chore: improve component prop types

Minor TS type improvements to broaden prop types making them usable on more use cases.
Their is no functional change

Thumbnail: broaden badge prop TS type from ReactElement to ReactElement | Falsy.
Useful to display badge conditionally (ex:  <Thumbnail badge={icon && <Badge>...)
Dialog: broaden contentRef prop TS type from RefObject to Ref.
TextField: broaden inputRef prop TS type from RefObject to Ref.
Autocomplete: broaden inputRef prop TS type from RefObject to Ref.

Test OK: Tested in an alpha version https://github.com/lumapps/lumapps-web/pull/5257/


Circular vocabulary dependencies don't throw a sane error

work/foo/foo.factor
USE: bar
IN: foo
: a ( -- ) b ;
work/bar/bar.factor
USE: foo
USE: kernel
IN: bar
: b ( -- ) f [ a ] when ;
This gives bad restarts instead of a good error message or simply just working. Additionally, some cases will fail initially and continue once restarted.
Perhaps the fix can wait until a parser rewrite, but it's terribly annoying in the meantime.


[1.x] Rebuild dashboard to display payload data

This PR. allow you to display payload data.

Codecov Report

Merging #688 (03feb4b) into 1.x (e9b9cc4) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##                1.x     #688   +/-   ##
=========================================
  Coverage     53.58%   53.58%
  Complexity      284      284
=========================================
  Files            50       50
  Lines           920      920
=========================================
  Hits            493      493
  Misses          427      427

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 e9b9cc4...03feb4b. Read the comment docs.


Add unit tests for controllers.

PS: fix of https://github.com/marcelorcorrea/falae/pull/64 needs to be applied/rebased on these branch.

https://github.com/marcelorcorrea/falae/pull/64 merged


What's the format of --keep-groups file?

When I add --keep-groups parameter and run, then got the error "segmentation fault  ./rvmparser --output-obj=aaa --keep-groups=group.txt".
What's the format of --keep-groups file?

It should be a text file with a group name on each line. It is read here: https://github.com/cdyk/rvmparser/blob/cfc6114ebdb55e92ba992b9638a7683541348d5f/src/Flatten.cpp#L13

thx!


Refactor Configuration for Filament Profile Editing and Avatar Handling

Changes Introduced:

Default Configuration Update:

Added a default configuration in filament-edit-profile.php.
Updated the README.md with configuration examples for clarity.

Filesystem Configuration:

Now uses the FILESYSTEM_DISK environment variable to match the application's default filesystem disk.
Set fallback to 'public' instead of 'local' to prevent breaking changes in existing installations.
Configured default visibility as 'public' with comments explaining it can be customized if needed.

Avatar Configuration:

Introduced configuration for specifying the avatar column name, defaulting to 'avatar_url'.
Updated the README.md to reflect this new configuration option.

@joaopaulolndev , could review please ? This PR address "Bug]: Possibilidade de inserir o nome da coluna do avatar #57"


Add pgvector backend

PostgreSQL is a popular choice for Wagtail deployments.
pgvector offers improved vector storage and distance calculations on top of PostgreSQL.
It would be great to support pgvector as a supported backend. As we already define an Embedding model for storage of embeddings regardless of the vector backend used, this may mean overriding this model with some pgvector specific fields when the appropriate backend is selected, similarly to how Wagtail creates a different IndexEntry based on the search backend.

Resolved by #16


New helper functions, and other small changes

These are some of the minor changes I needed when working on my project (WIP) https://github.com/mherrerarendon/chess_move_validator
I figure they might be useful for others too.

Created a new Square method new_with_offset that allows you to create a new Square with horizontal and/or vertical offsets relative from the self Square
Created a new Square method get_known that returns an Option with Some if Square is known
Make the peggler dependency public (needed for ParseResult)
Add a new public method that parses only the move sequence string

I'm happy to receive feedback too!

I'm no longer maintaining this crate or peggler. I actually meant to archive them long ago.
My recommendation would be to find a better PGN crate and use that instead. I'm sure there must be one by now.
Alternatively, you can take over ownership of these crates. Just send me your crates.io username and I'll transfer ownership of both of them.


Non-threaded handler

Sometimes a simple handler (eg. CTCP VERSION handler) doesn't need a separated thread.
It may be better if we have an option to do that.

Is the separate thread hurting?
To give a somewhat more elaborate answer: For Cinch 2.x I don't see it happening. Primarily because Cinch is in maintenance mode, secondarily because I'm not sure how easy it is to add such an option to the current, somewhat flawed, implementation of handlers.
Maybe if there's ever going to be a Cinch 3, which would be bound to be a rewrite, it will be taken into consideration, even though I still think that it's a pointless optimization, considering that other parts of Cinch are more likely to be the bottleneck than spawning another thread.


Export In Kind in donations csv

Resolves #4524
Description
Add column to the csv export for in kind value.

Type of change

New feature (non-breaking change which adds functionality)

How Has This Been Tested?
Added a test to see if the column exists

Screenshots

Sure! No other formatting other than that?

Done in this commit: https://github.com/rubyforgood/human-essentials/pull/4661/commits/c1449db106e8047f6292ede7090ff106adedfc6f

Formatted as money without the symbol


perf: detect path from decorator for each class only once

Fix for bun path detection
Related discussion here: https://github.com/mikro-orm/mikro-orm/discussions/5496
In this commit you changed the condition because of coverage. This is incorrect because it guards against going out of stack array bounds - I have added the test for this case. Also, I removed no longer needed condition.
Performance
Before the changes, the path was detected for every used decorator on the class and overwritten. This is not needed because the path used as a key should be stable and should not change for a given class. So I introduced the new symbol MetadataStorage.PATH_SYMBOL, which will hold the path and check if it exists for a given class using Object.hasOwn (this also works for class inheritance so the path is not taken from the base class but defined for each child).
Moreover, as a side effect, this also allows defining a path on the class by the developer (maybe in cases like when Bun was not working), but it should not be needed and used only as a workaround:
import { Entity, MetadataStorage } from '@mikro-orm/core';

@Entity()
class User {
  static [MetadataStorage.PATH_SYMBOL] = import.meta.filename // or __filename for commonjs

  @PrimaryKey()
  id!: number;

  @Property()
  username!: string;
}

Moreover, as a side effect, this also allows defining a path on the class by the developer

This is great, I've been thinking about introducing this for ages. We should document this somewhere.

This is great, I've been thinking about introducing this for ages. We should document this somewhere.

Sure, but there is a caveat. I think people will expect to define the path in the base class and inherit it. However, they need to copy-paste the code for each class, which is slightly counter-intuitive. This comes from the fact that import.meta.filename or __filename is scoped to a module (file), and we expect the path to be defined for each class because it usually differs. It's not worth making logic to detect if we can inherit it from the parent, maybe if it is defined as a getter, but we can document that. I am open to suggestions.

Yes, that is surely something to document. I don't think it makes sense to try to validate this, as having the entities in a single file is still a valid case.

I guess this could go to https://mikro-orm.io/docs/metadata-providers.
Also, do you think this should be a perf commit? Does it have a measurable difference? Maybe a fix would be better, or even a feat since we are adding a new feature that will have its documentation.

Btw does this help with the production build issue in bun mentioned in that discussion?

I'll merge this in as a fix, as I don't want to hold it for no reason, but I would still appreciate response to my previous comment as well as a PR mentioning this in the docs somewhere.


Added Support for setWeekNumbersVisible. Fixes [InlineDatePicker] Add…

… support for setWeekNumbersVisible #34
Added Support for setWeekNumbersVisible in InlineDatePicker.java

Thanks for the pull request. Unfortunately, some changes were needed: the setter should assign the client-side property showWeekNumbers, and the parameter must be of type boolean instead of Boolean. In the client side, fc-inline-date-picker has to declare showWeekNumbers and bind it to fc-month-calendar. I already submitted these changes in #37.


Update yuicompressor to 2.4.7

I was getting a stack overflow error when trying to minify a CSS file. Updating to 2.4.7 seems to have fixed it.
Might want to test to see if there are any regressions first. I haven't found any.

The node version (uglifyjs + cleancss) works better; might be worth switching over. Either way, I'll pull in the change :)

Great, thanks!
I assume that's what our servers are using, since it's not throwing the error. I'll switch over then!


Allow directives to span multiple or all columns

I was looking for a way to make a figure span the full with of a document when exporting to PDF and using a multi-column template (eg arxiv_two_column). @rowanc1 mentioned on the discord that this currently isn't fully implemented, but that some of the underlying code already does have a few things in place for this. See:
https://github.com/executablebooks/mystmd/blob/33140d21881349075c45a44b1bebf915db6c236d/packages/myst-to-tex/src/container.ts#L42
That code also hints that it's not only applicable to figures but also other directives like tables, code, math (and perhaps others like glossary/abbreviations?).
Proposal
I've just started with MyST so I'm not sure how this should work from the user/directive point of view. I'm imagining assigning a property to the directive along the lines of :class: full-width, for example:
```{figure} ../document/figures/examplefig.png
:name: fig:examplefig
:class: full-width
A nice example figure!
```

I guess that would make it available in the AST after parsing (?), and it's up to whatever "exporter" is used to utilize it accordingly.
Documents (docx/pdf) often won't go beyond 2 columns, but maybe certain HTML or other formats would? If so, instead of a binary full-width or nothing something like a :colspan: <n> type property might also be considered. But that might complicate things quite a bit.
An alternative implementation might be to have some container directive that nests others, analog to a div in HTML.

@rowanc1 could this also be added usefully to tables? I guess limited to list tables?

@rowanc1 could this also be added usefully to tables? I guess limited to list tables?

It already works on list tables, I tested that with the latest release. From my limited understanding of the changed code those also the only two directives for which this is the case.

Yep! Should work for both tables and figures at the moment!

Confirmed with @RutgerK that this works for their usecase!


Slide show change onLoad?

Is it possible to change the slide image randomly onload only instead of timed intervals?

You can easily do that with JavaScript. You can set all your images in an array and pick one randomly onload for backstretch.
imagesCollection = [
  '//..image1.png'
  '//..image2.png'
  '//..image3.png'
  ]
image = imagesCollection[Math.floor(Math.random() * imagesCollection.length)]
$('#hero').backstretch(image)

We'll give this a go. Thanks!


Permissions Error when recording events with KinesisStream provider from Analytics

How did you install the Amplify CLI?
npm
If applicable, what version of Node.js are you using?
16.4
Amplify CLI Version
10.5.2
What operating system are you using?
Mac
Did you make any manual changes to the cloud resources managed by Amplify? Please describe the changes made.
No
Describe the bug
I am using the Kinesis Streams provider from the Analytics feature set - and I am getting an error when trying to use Analytics.record
{"__type":"AccessDeniedException","Message":"User: arn:aws:sts::613607431007:assumed-role/amplify-nextjsplayground-dev-205355-unauthRole/CognitoIdentityCredentials is not authorized to perform: kinesis:PutRecords on resource: arn:aws:kinesis:us-west-2:613607431007:stream/undefined because no identity-based policy allows the kinesis:PutRecords action"}
If I try updating the Project by using amplify update analytics I get an error from the CLI and I cannot proceed.
Expected behavior
Permission errors for the UnAuth role should not be shown when using the Kinesis Stream provider.
Reproduction steps

Add amplify by using amplify add analytics
Choose Kinesis Streams as the provider
Allow Auth and UnAuth roles to access the Analytics resource
Register the Kinesis pluggable into the NextJS app
Try using the Analytics.record API
The error shows up in the console

Project Identifier
07d8fa9a07058e596960d73808ea0ff5
Log output

# Put your logs below this line

Additional information
No response
Before submitting, please confirm:

[X] I have done my best to include a minimal, self-contained set of instructions for consistently reproducing the issue.
[X] I have removed any sensitive information from my code snippets and submission.

Hey @abdallahshaban557 :wave: thanks for raising this! As we begin to look into this in more depth I have two follow-up questions:
From the error in the browser console, the resource ARN stands out as it ends with undefined, resource: arn:aws:kinesis:us-west-2:613607431007:stream/undefined. Is this value accurately captured in the aws-exports.js file?

If I try updating the Project by using amplify update analytics I get an error from the CLI and I cannot proceed.

What is the error you're receiving when attempting to update your analytics resource?

Hi @josefaidt
It seems like the value for analytics isn't captured at all in my aws-exports.js file at all. I can only see Auth information reflected in the file.
When I try to update the analytics resource I get this error
🛑 File at path: '/Users/awshaban/Main_Folder/Github/JS/nextjs-playground/amplify/backend/analytics/nextjsplaygroundKinesis/pinpoint-cloudformation-template.json' does not exist

Note: upon further inspection I noticed after pushing analytics I do not see an analytics-related key in the aws-exports.js file which may be the culprit.

It appears the endpoint is missing when the Analytics module from amplify-js attempts to make the PutRecords call with the KinesisStream client from aws-sdk

Note: we do see the ARN is captured as an output in kinesis-cloudformation-template.json:
"Outputs": {
    "kinesisStreamArn": {
      "Value": {
        "Fn::GetAtt": [
          "KinesisStream",
          "Arn"
        ]
      }
    },
    "kinesisStreamId": {
      "Value": {
        "Ref": "KinesisStream"
      }
    },
    "kinesisStreamShardCount": {
      "Value": {
        "Ref": "kinesisStreamShardCount"
      }
    }
  }

and this is successfully captured in amplify-meta.json
"output": {
  "kinesisStreamId": "nextjsplaygroundKinesis-myenv",
  "kinesisStreamShardCount": "1",
  "kinesisStreamArn": "arn:aws:kinesis:us-east-1:814763596509:stream/nextjsplaygroundKinesis-myenv"
},

where amplify-js is missing the resource name. Marking as a bug 🙂


Fix "reparent to new node" when node has internal children

Followup to #79209
Fixes https://github.com/godotengine/godot/pull/79209#discussion_r1746936928

Thanks!

Cherry-picked for 4.3.1.


Add support for hash_file  Checksum for output file

Feature Request
Generate a hash value of a output file
Add hash value to costum file with ALGO name
example:
...
....
....
"output": "dist/test.phar",
"algorithm" : "sha256"
...
...
...

Expected in dist folder :
.
└── test.phar
└── test.phar.sha256sum

What would be the purpose of that file?
Just in case, you can get the hash easily by running box info foo.phar

to verify and to compare it to the one found in the test.phar.sha256sum with the test.phar
Exmple:
$ sha256sum -b test.phar > test.phar.sha256sum

$ sha256sum --ignore-missing -c test.phar.sha256sum
test.phar: OK

but in box if you change the test.phar we can't run test.phar
$ php test.phar
PHP Fatal error:  Uncaught PharException: phar "/test.phar" has a broken signature in
Stack trace:
#0 /test.phar(27): Phar::mapPhar('test.phar')
#1 {main}
  thrown in /test.phar on line 27

also sha256sum
$ sha256sum --ignore-missing -c test.phar.sha256sum
test.phar: FAILED
sha256sum: WARNING: 1 computed checksum did NOT match
sha256sum: test.phar.sha256sum: no file was verified

that good to know

I'm a bit confused, what is the point of checking the signature if, when the signature unexpectedly changed, the PHAR cannot be executed in the first place?
The only benefit I could see is if you don't trust the PHAR in the first place and want to check its signature to make sure it has not been corrupted (if you check so by attempting to run the PHAR you may execute malicious code container within it).
However is a question of where do you get the PHAR from: you will have to download the signature from somewhere and if you can do so you can as well download the PHAR directly. Unless the signature is downloadable from a trustable third-party and independent from the PHAR download source, this will not provide any additional safety.

good point we don't need Checksum with box

Note regarding the distribution:

If you cannot ensure the PHAR is coming from a safe place, either you execute it in a securely isolated environment or you download it again but from a trusted source (and always in HTTPS which includes a checksum check).
With PHIVE one can GPG sign the PHAR and register the GPG key on phar.io, which can do a check (and the check is more reliable since the key is from a secure third-party source)

Will be closing this issue then :)


UDP sockets stop listening after a while

After some time (a few dozen days), my server stops listening on UDP sockets. lsof shows that the socket is just gone. This happens after a few weeks. I do not close the socket explicitly in my code. The server has very low utilization.
This happens on Debian 9 with a Git snapshot after 0.27 (commit 0d526e4da15f84b8760e7b3574b8d58e49466552 to be precise). It is not a recent regression; it happened before with older Pony versions.

Do you have any logs or metrics for your process? Which port did you listen on with which settings? Do you log anything in UDPNotify.closed?

I do not see anything useful I could log in UDPNotify.closed, except for the timestamp when this happens. I have added logging for this event nevertheless.
I don't have any metrics, only packet captures of incoming and outgoing packets. I'm not sure what kind of metrics I could collect.

I was asking if you have any info if it was closed from within pony stdlib code or from the os somehow. If pony closed the socket somehow you should have been informed via your notify. In order to work around that issue for the time being you could reopen another socket if yours was closed. I am going to check for the possible conditions on which pony might close your socket.

@fweimer can you provide before and after lsof output?

@fweimer do you have a minimal program that demonstrates the problem?

It happened again:
2019-05-16T15:19:56 info: TCP connection closed: <IP_ADDRESS>/47071
2019-05-16T16:29:27 UDP socket closed
2019-05-17T06:00:18 info: incoming TCP connection from: <IP_ADDRESS>/62616

So it appears spontaneous and not related to network activity (nothing else was logged around the time). The process was started at 2019-05-01T11:03:04, so the elapsed time is not a particularly round number.
lsof output before:
dns-serve 4381 root    0u      CHR   136,1      0t0      4 /dev/pts/1
dns-serve 4381 root    1u      CHR   136,1      0t0      4 /dev/pts/1
dns-serve 4381 root    2u      CHR   136,1      0t0      4 /dev/pts/1
dns-serve 4381 root    3u  a_inode    0,11        0   7518 [eventpoll]
dns-serve 4381 root    4u  a_inode    0,11        0   7518 [eventfd]
dns-serve 4381 root    5u      CHR     1,9      0t0   1042 /dev/urandom
dns-serve 4381 root    6u     IPv4 9451616      0t0    TCP *:domain (LISTEN)
dns-serve 4381 root    7u     IPv4 9450689      0t0    UDP *:domain

And after:
dns-serve 4381 root    0u      CHR    136,1      0t0      4 /dev/pts/1
dns-serve 4381 root    1u      CHR    136,1      0t0      4 /dev/pts/1
dns-serve 4381 root    2u      CHR    136,1      0t0      4 /dev/pts/1
dns-serve 4381 root    3u  a_inode     0,11        0   7518 [eventpoll]
dns-serve 4381 root    4u  a_inode     0,11        0   7518 [eventfd]
dns-serve 4381 root    5u      CHR      1,9      0t0   1042 /dev/urandom
dns-serve 4381 root    6u     IPv4  9451616      0t0    TCP *:domain (LISTEN)
dns-serve 4381 root    8u     IPv4 12071872      0t0    TCP <IP_ADDRESS>:domain-><IP_ADDRESS>:63489 (ESTABLISHED)

Observe the gap for file descriptor 7.
The last connection is a leftover from an earlier TCP-based query:
2019-05-14T06:10:23 info: incoming TCP connection from: <IP_ADDRESS>/63489
2019-05-14T06:10:23 info: packet (30 bytes) from: <IP_ADDRESS>/63489/TCP

Regarding a minimal reproducer, I guess I will try to reproduce with a minimal, idle server, given enough time.

Is it possible a write failing, triggering the UDPSocket to close? This can occur on some OS when under a lot of stress (Linux will do it, giving errors like EPERM). There seems to be no way to get the cause of failure for UDPSocket.writes.

@anacrolix Sorry, I don't have access to Zulip.  Would this cause the socket go away, though?

Yes, if an error is raised while writing, the socket will be closed: https://github.com/ponylang/ponyc/blob/eb6fbbb16600249e1d90cd3e6674ebe75fecc5b0/packages/net/udp_socket.pony#L366-L370
https://github.com/ponylang/ponyc/blob/eb6fbbb16600249e1d90cd3e6674ebe75fecc5b0/packages/net/udp_socket.pony#L403-L408

Hmm. This is rather awkward. Is it really necessary to close the socket here? It should be fairly easy to reproduce this with packets from a spoof source address to which no route exists in the routing table, assuming that the kernel does not implement IP reverse path filtering.

@fweimer do you have a minimal program that demonstrates the problem? it's very hard to know what might be an issue without being able to see code that can demonstrate it.

I need to check if I still have the sources somewhere. (The application is more or less a basic UDP echo server, but it speaks DNS.) The issue reproduced only after a long time (several weeks). Close-on-error looks like the most likely culprit, though.

@fweimer were you able to find anything?


Add reverse proxy support

Hello again after a while!
For the next iteration of our API we're considering using grpc. Currently our API runs exclusively with Poem, at /v1. I'd like to use Poem to route requests to /v2/grpc to the grpc backend. I understand this is probably easy with poem-grpc, but for now I'm working with tonic because it has some features I need, e.g. tonic-reflection.
Furthermore, down the line I'd like to host a JSON-reflected version of the API, so /v2/resource, /v2/account, etc. for example. That's not possible for free right now unfortunately (I'd need something like this: https://github.com/poem-web/poem/issues/359 or https://github.com/tokio-rs/prost/issues/301#issuecomment-1377681201), but it's something I'd need to keep in mind for the reverse proxy use case.
Does Poem have support for this kind of raw reverse proxy? In the past when I first migrated us to Poem I used this (https://crates.io/crates/warp-reverse-proxy), but I'd prefer not to use warp again hahah.
Thanks!

I figure one option that could work out of the box is using axum + the tower extension support in Poem, though that feels pretty hacky.

Maybe you can try poem-grpc, which also supports reflection.

Or I add reverse proxy support to poem, which is easier.

Okay yeah I'll try poem-grpc, though having a reverse proxy in Poem would still be helpful as part of this experimentation 😄

Although my use case is different I vote for reverse proxy support:
Having poem with Oauth2 + reverse proxy to another upstream poem instance will allow me to remove multiple components from infrastructure like caddy, nginx, oauth2-proxy.


HeaderCommentFixer - throw exception on invalid header configuration (II)

Follow up on https://github.com/FriendsOfPHP/PHP-CS-Fixer/pull/4099 where I missed this case.
cc @kubawerlos

The case is more complex than you think:
<?php
/**
 * This is valid ?>
 */

/*
 * This is valid too ?>
 */

/* And so is this ?> */

<?php
// This is the case that breaks ?>

<?php
# This breaks too ?>

(Tested locally on php 7.1 at least, and the github syntax highlighting seems to agree)

And I see now that the only cases that break aren't even supported by HeaderCommentFixer so this PR is unnecessary, right?

I agree with @dmvdbrugge - the fixer only support comments where ?> is fine.


MacOs support

Is it also possible to use it with a macos target? 🤔

Hey simpros. Not right now, unfortunately. There was support added a few years ago on the original repo, but it'll take a significant amount of effort to get it working again, so I took it out for now to avoid confusion/irritation from people trying to use it in that current state.


Use of spatial dropout

Could you clarify if spatial dropout is used? The paper suggests that it is, but the code seems to use standard dropout.

Hi @jlmaccal,
We originally did use the spatial dropout (which is simply nn.Dropout1d in pytorch), but then later it was pointed out to us that the standard dropout works better on language modeling tasks. Therefore the repo has been changed to use the standard dropout.

Thanks for the clarification.


use laravel 5 default for config

What functionality does this add? Please remove the indentation changes

This tells the system it adds a "config" instead of "views" or "assets". This way you can say in the vendor:publish command you only want to publish config files.

This is now published as version 1.4.2


Total data labels in stacked grouped bars should be per group

Description
A lot of work towards fixing this seems to have been done in https://github.com/apexcharts/apexcharts.js/commit/355f9e233fb8babb92682a9e697adca931a76453 just wanted to open issue to keep track of it and awareness.
Steps to Reproduce
Create 4 series 2 in each group with totals data labels enabled using bar chart. See codepen below for example.
Expected Behavior
Totals to be per group

Created using painting app.
Actual Behavior
Totals of both groups

Screenshots
Included above.
Reproduction Link
https://codepen.io/David-Karla/pen/VwRLQed

Checking in—this is still an issue in 12/2024. Would be great to have functional totals per group!


not all go modules works

from 'go/math' import Pi
print Pi

this works and output

grumpy run test.py
3.14159265359

but this not works

 import "__go__/encoding/hex"
 print hex.Dump('foobar')

output error
grumpy run test.py
/tmp/tmp8Y9Chn__pycache__/gopath/src/python/main/module.go:4:2: cannot find package "python/go/encoding/hex" in any of:
/usr/lib/go/src/python/go/encoding/hex (from $GOROOT)
/tmp/tmp8Y9Chn__pycache__/gopath/src/python/go/encoding/hex (from $GOPATH)
/home/user/.local/lib/python2.7/site-packages/grumpy_runtime/data/gopath/src/python/go/encoding/hex
Originally posted by @6umpukc-uHKBu3umop in https://github.com/grumpyhome/grumpy/issues/148#issuecomment-569084277

is there any working example
with  modules in example  https://golang.org/pkg/net/http/
?

This I never tested but had feedback of being working. For more examples of working code grep __go__ on lib folder.
Examples: This uses lots of Go native modules, including reflection and syscalls

Python sockets implementation: https://github.com/grumpyhome/grumpy/pull/132/files
Python os module implementation: https://github.com/grumpyhome/grumpy/blob/master/grumpy-runtime-src/lib/os/init.py
Python sys module implementation: https://github.com/grumpyhome/grumpy/blob/master/grumpy-runtime-src/lib/sys.py

https://github.com/grumpyhome/grumpy/issues/148#issuecomment-569414322
found some go lib wrapers - but not all are implemented
so i should write  go file wrapers  for other go standart libs manually ?
or is there some script for automating this task?

On https://github.com/grumpyhome/grumpy/blob/master/grumpy-runtime-src/tools/pkgc.go is the code that takes from '__go__/whatever' import Something and automatically creates the glue Go files. You can start looking there.

is the code that takes from 'go/whatever' import Something and automatically creates the glue Go files

so this should work autmatocally - then when?
or i must run it for preparing wrapper of each go package?

Yes, it should work automatically. I do not know why is not working in your case.
Let me say that this code is wrote by Google (Youtube mainly) and I just made some maintenance on this exac file.

so this should work autmatically - then when?

When you grumpy run your files are parsed and transpiled. Imports from __go__ are transpiled as glue I guess. Then all transpiled modules (Py and __go__ imports) are added to GOPATH and at the end there is a go run or go build.
I suggest you to run grumpy -v debug myscript.py that imports bunch of stdlib first, then follow the debug logs to get a grip on how the sausage is done.

for some reasons this does not work automatically
simple example of file gostrconv.py
# grumpy -v warning --keep-main gostrconv.py
# find . -type f -exec cat {} \;

import "__go__/strconv"
print repr(strconv.Itoa(-42))

grumpy -v warning --keep-main gostrconv.py

/tmp/tmpsxuDmA__pycache__/gopath/src/__python__/__main__/module.go:4:2: cannot find package "__python__/__go__/strconv" in any of:
	/usr/lib/go/src/__python__/__go__/strconv (from $GOROOT)
	/tmp/tmpsxuDmA__pycache__/gopath/src/__python__/__go__/strconv (from $GOPATH)
	/home/user/.local/lib/python2.7/site-packages/grumpy_runtime/data/gopath/src/__python__/__go__/strconv
warning: not cleaning the temporary pycache folder: /tmp/tmpsxuDmA__pycache__

grumpy -v debug gostrconv.py
info: GOPATH: /home/user/.local/lib/python2.7/site-packages/grumpy_runtime/data/gopath
info: __main__ pycache folder: /tmp/tmpkpywJE__pycache__
debug: Should transpile '__main__'
debug: Dependencies file regenerated
info: `go run` GOPATH=/tmp/tmpkpywJE__pycache__/gopath:/home/user/.local/lib/python2.7/site-packages/grumpy_runtime/data/gopath
debug: Starting subprocess: `go run /tmp/tmpkpywJE__pycache__/gopath/src/__python__/main.go`
/tmp/tmpkpywJE__pycache__/gopath/src/__python__/__main__/module.go:4:2: cannot find package "__python__/__go__/strconv" in any of:
	/usr/lib/go/src/__python__/__go__/strconv (from $GOROOT)
	/tmp/tmpkpywJE__pycache__/gopath/src/__python__/__go__/strconv (from $GOPATH)
	/home/user/.local/lib/python2.7/site-packages/grumpy_runtime/data/gopath/src/__python__/__go__/strconv

generated files
find . -type f
./checksum-__main__.sha1
./dependencies-__main__.pkl
./__main__.py
./gopath/src/__python__/main.go
./gopath/src/__python__/__main__/module.go

`main.go`

package main
import (
"os"
"grumpy"
mod "python/main"
_ "python/traceback"
)
func main() {
grumpy.ImportModule(grumpy.NewRootFrame(), "traceback")
os.Exit(grumpy.RunMain(mod.Code))
}
`module.go`

package main
import (
πg "grumpy"
_ "python/go/strconv"
)
var Code *πg.Code
func init() {
Code = πg.NewCode("", "/home/user/work/grumpy-example-currency-parser/main.py", nil, 0, func(πF *πg.Frame, _ []*πg.Object) (*πg.Object, *πg.BaseException) {
var πR *πg.Object; _ = πR
var πE *πg.BaseException; _ = πE
ßItoa := πg.InternStr("Itoa")
ßrepr := πg.InternStr("repr")
ßstrconv := πg.InternStr("strconv")
var πTemp001 *πg.Object
_ = πTemp001
var πTemp002 []*πg.Object
_ = πTemp002
var πTemp003 []*πg.Object
_ = πTemp003
var πTemp004 []*πg.Object
_ = πTemp004
var πTemp005 *πg.Object
_ = πTemp005
for ; πF.State() >= 0; πF.PopCheckpoint() {
switch πF.State() {
case 0:
default: panic("unexpected function state")
}
// line 5: import "go/strconv"
πF.SetLineno(5)
if πTemp002, πE = πg.ImportModule(πF, "go/strconv"); πE != nil {
continue
}
πTemp001 = πTemp002[0]
if πE = πF.Globals().SetItem(πF, ßstrconv.ToObject(), πTemp001); πE != nil {
continue
}
// line 6: print repr(strconv.Itoa(-42))
πF.SetLineno(6)
πTemp002 = make([]*πg.Object, 1)
πTemp003 = πF.MakeArgs(1)
πTemp004 = πF.MakeArgs(1)
if πTemp001, πE = πg.Neg(πF, πg.NewInt(42).ToObject()); πE != nil {
continue
}
πTemp004[0] = πTemp001
if πTemp001, πE = πg.ResolveGlobal(πF, ßstrconv); πE != nil {
continue
}
if πTemp005, πE = πg.GetAttr(πF, πTemp001, ßItoa, nil); πE != nil {
continue
}
if πTemp001, πE = πTemp005.Call(πF, πTemp004, nil); πE != nil {
continue
}
πF.FreeArgs(πTemp004)
πTemp003[0] = πTemp001
if πTemp001, πE = πg.ResolveGlobal(πF, ßrepr); πE != nil {
continue
}
if πTemp005, πE = πTemp001.Call(πF, πTemp003, nil); πE != nil {
continue
}
πF.FreeArgs(πTemp003)
πTemp002[0] = πTemp005
if πE = πg.Print(πF, πTemp002, true); πE != nil {
continue
}
}
return nil, πE
})
πg.RegisterModule("main", Code)
}

checked the repository code and did not find where is pkgc utility launch
so i think i sholud run it manually for go packages which i used

You may be right. Maybe what is available is whitelisted on the makefiles of grumpy-ru time.
The build process is a bit convoluted as it creates at least one makefile dynamically and calls it from the main makefile.
I searched for a place where it is whitelisted but now I think it does some reflection (or list folder/files at least)


test/socket-io-cmd.t randomly fails: values does not match: 1360 != 0

test/socket-io-cmd.t sometimes it runs well; sometimes, it fails.
Kernel: 6.8.2-afs-2024-04-02
Arch: x86_64
Reproduce
cd test;
./runtests-loop.sh socket-io-cmd.t;

After several iterations, it randomly fails; the value of the output is also random.
Failure outputs:
Running test socket-io-cmd.t                                        values does not match: 588 != 0
Test socket-io-cmd.t failed with ret 1
Tests failed (1): <socket-io-cmd.t>
Tests failed at loop 70

Running test socket-io-cmd.t                                        values does not match: 308 != 0
Test socket-io-cmd.t failed with ret 1
Tests failed (1): <socket-io-cmd.t>
Tests failed at loop 62

unning test socket-io-cmd.t                                        values does not match: 1334 != 0
Test socket-io-cmd.t failed with ret 1
Tests failed (1): <socket-io-cmd.t>
Tests failed at loop 22

Running test socket-io-cmd.t                                        values does not match: 52 != 0
Test socket-io-cmd.t failed with ret 1
Tests failed (1): <socket-io-cmd.t>
Tests failed at loop 14

Running test socket-io-cmd.t                                        values does not match: 1360 != 0
Test socket-io-cmd.t failed with ret 1
Tests failed (1): <socket-io-cmd.t>
Tests failed at loop 15

I traced this down the stack, and it looks correct in the sense that raw_ioctl() is indeed finding an skb and hence returns the length. Not sure who's sending that or where it's coming from, but it's most certainly there. I think for now, since we're close to release, I'll just add a single retry loop in case they differ before failing. They should find the same at that point, at least.
Though it would be nice to know where this data is coming from...

Can you retest with the current tree?

Not sure who's sending that or where it's coming from, but it's most
certainly there. I think for now, since we're close to release, I'll
just add a single retry loop in case they differ before failing. They
should find the same at that point, at least.
Though it would be nice to know where this data is coming from...

Unfortunately, I am not familiar with SOCK_RAW. Adding Breno, just in
case he knows something about this, as he's the one who added the
feature. Also, Netdev people can probably quickly recognize what's going
on.
Cc: @leitao
Breno, do you have any idea where the data is coming from?

Can you retest with the current tree?

Looks good now, 1k+ iterations with no fail.

Just a quick test: isolating the program in a network namespace where it
only has a loopback interface doesn't cause the test to fail.
ip netns replace iou;
ip netns exec iou bash -i;
ip link set up dev lo;

# ip addr
1: lo: <LOOPBACK,UP,LOWER_UP> mtu 65536 qdisc noqueue state UNKNOWN group default qlen 1000
    link/loopback 00:00:00:00:00:00 brd 00:00:00:00:00:00
    inet <IP_ADDRESS>/8 scope host lo
       valid_lft forever preferred_lft forever
    inet6 ::1/128 scope host
       valid_lft forever preferred_lft forever

# Commit before the 'retry workaround'.
git checkout 1cbd4cffa88177;

cd test;
make -j;
./runtests-loop.sh socket-io-cmd.t;

Some dumps of what I saw just now seeing what's in the socket. I tend to see 76 or 40 bytes pretty reliably:
Running test socket-io-cmd.t                                        recv=76
45 0 0 4c 98 3 0 0 40 6 ca 98 a 0 2 2 a 0 2 f c1 f0 0 16 92 7e 5f 22 59 7d 64 fa 50 18 ff ff 6c 7f 0 0 8 68 74 c6 48 9 26 3f 5c 6c 96 f 11 16 c5 5e 1b fc 3a 8d 6d be d e9 37 f4 83 87 ed 8a d3 6b 50 e2 65 c
values does not match: 76 != 0

Running test socket-io-cmd.t                                        recv=40
45 0 0 28 9e 41 0 0 40 6 c4 7e a 0 2 2 a 0 2 f c7 f6 0 16 93 75 55 5e ed 40 6a 8c 50 10 ff ff 8f 16 0 0


add the 3 test keys from the updated p8e-scope-sdk

these are the 3 keys corresponding to 3 mnemonics

Codecov Report

Merging #9 (f003b93) into main (767703d) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##              main      #9   +/-   ##
=======================================
  Coverage     7.52%   7.52%
  Complexity       6       6
=======================================
  Files            9       9
  Lines          651     651
  Branches        47      47
=======================================
  Hits            49      49
  Misses         600     600
  Partials         2       2

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 767703d...f003b93. Read the comment docs.


Fix checking every empty nodetree

This should prevent the checking of empty nodetrees for OSL shaders. It
will also prevent a key error from being logged.

Totally good way to check normally I've check if node tree is an empty string but this is somewhat safer


Enable Token Airdrop In Buck with Wallet

follow the /token-airdrop/readme.md to run the page.

Add amount.