Check for valid phone numbers before subscribing

⚠ IMPORTANT: Please do not create a Pull Request without creating an issue first.
All changes need to be discusssed before proceeding. Failure to do so may result in the pull request being rejected.
Please be sure to review our Code of Conduct, Contributing Guidelines, and Support documentation before submitting a pull request.

Please include the issue number the pull request fixes by replacing YOUR-ISSUE-HERE in the text below.
Fixes #630
Summary

Checks phone numbers client-side to ensure they comply with E.164 which is necessary for Twilio.
Details

Checking on the frontend because this is a quick check to do and isn't worth a backend requests
Updated success message to display a success snackbar similar to how failure messages display error snackbars

Test Plan (required)

We don't have any tests for More, but probably should add some, especially since we're doing a lot of phone number checks. Will create an issue.
Final Checklist

[x] For CoronaTracker, did you bump the version in package.json?

Update the second decimal for a major change
Update the third decimal for a minor change
Numbers can go past 9, e.g. 1.0.9 => 1.0.10
For more info, read about Semantic Versioning

[x] Did you add any new tests as necessary?
[x] Is your PR rebased off the most current master?
[x] Have you squashed all commits? (can be done at merge)
[x] Did you use yarn not npm? (important!)
[x] Did you use Material-UI wherever possible?
[x] Did you run yarn lint on the code?
[x] Did you run all of your most recent changes locally to make sure everything is working?

My only guess as to why there's changes in Table here is that previous changes didn't lint them properly 😕


Unigen reader cannot be referenced

Understand what is going on and how it can be fixed.

It is fixed now. The UnigenReader contains a file reader; therefore, it cannot be copied.


[New Feature]: It is possible to use Winget install the applications to WSA?

Description of the new feature/enhancement
As the best Package manager over the world
So, we need could control more
Winget-->MS store
Winget-->win32
Winget-->WSL system
Winget-->Edge PWA(save URL as desktop app）
Winget-->android app
Proposed technical implementation details (optional)
Maybe use adb to sync the android app.

I agree this would be a nice feature. We already have an issue for it in the winget-cli repo, maybe add thoughts over there?
Duplicate of https://github.com/microsoft/winget-cli/issues/1625

I guess you cannot close cross-repo issues.


Your Apple ID was used to sign in to iCloud via a web browser messages with V3

Since upgrading to v3 I'm getting several emails a day from Apple telling me my id was used to sign into iCloud. This didn't happen at V2

This has been asked/answered a few times previously in this repo.  It's on Apple's side.

Strange I have never had an issue before v3 though but thanks for the reference

@nivek1612
The emails are a pain and iC3 authenticates the account using a Token first and then the Password. v2 did it a little different and, as I recall, used the password most of the time. When trying out various methods to see which one generated the least emails, it seemed like the way it is done now worked the best. Apple sets the expiration date/time to about every 30-minutes. The Event Log > Show Tracking Monitors show when this is done and the method used. It is this reauthentication that generates the emails.
More info about this can be found in the iCloud3 docs here. It also suggests ways to help make them less aggravating.

Hi @gcobb321,
Thank you for your efforts on the iCloud3 integration.
I wanted to ask whether you're aware of "refresh tokens", which should be getting returned when authentication occurs and can be used to keep the session alive.
I'm not familiar with the specific mechanics of these for the iCloud API, but generally, the process involves passing the previously received refresh token back to the API, which the backend takes as a request to extend the life of the access token and responds by returning a new, different refresh token for submission next time.  As long as the refresh token is used at the required frequency, the access token should last for days, weeks, or even months.
Looking briefly at the source code, I would suspect that this activity should occur in icloud3.support.pyicloud_ic3.PyiCloudService._validate_token.  A quick word search on the entire repo suggests that refresh tokens aren't being collected or operated on.
I apologise if this has been previously considered and not adopted for specific reasons—but I couldn't find any reference to this in the related bug reports, and so I thought I should ask.
My belief is that if refresh tokens are correctly managed, recurring reauthentication (and resultant emails) could be avoided.


KMZ overlay with rotation not working correctly

I have a mapbox map that contains a KMZ file that displays some overlays correctly and others rotated incorrectly.  Further inspection shows all the overlays with rotation set to zero display correctly.  If rotation is set the rotation displayed on the map is off and inverted I believe.
Screenshot shows an overlay with rotation set as 41.0118 in Google Earth.  Changing the value to -41.0118 causes the image to be close to correct.  Although its still slightly off from the green outline polygon.  I have a version of the map using ESRI and it loads the KMZ correctly so I believe the issue is with togeojson logic in some way.

https://github.com/placemark/togeojson/blob/578fdb52e2bab47ce4f5908fbdf9739eef19ed5d/lib/kml/ground_overlay.ts#L42
Currently togeojson handles rotations for GroundOverlays, but does not handle rotation for objects - assuming that this is a KML feature with a rotation property (a data example would be useful). I'm not currently seeing anything in the KML reference that indicates that any other geometry type could be rotated.

Here's the kmz I'm having an issue with.  Only two overlays.  One with no rotation and one with rotation.
Legend Point is the overlay not rotating as expected.
Let me know if you don't have the same experience.
Looking at the code above, line 52 should be plus rotation?  It would fit what I tested yesterday by making the rotation value a negative number in the kmz.
Overlay1-test.zip


Change database encoding to utf8

Database encoding changed to utf8_unicode_ci
travis.yml file, database setup instructions, bootstrap shell file is also modified to reflect changes.

Coverage increased (+0.005%) to 94.929% when pulling 958215963b9b4a64ed61cecb016603275cfef4f0 on utf8encoding into 130e3129c4404c464bb92bd9f518fb11555a5a68 on master.


feat: enable filtering on MultiSelect

Select has the ability to filter options, but MultiSelect doesn't, so I implemented it.

This is so awesome @vitor-mariano! Will test this out and merge!


Extension of the Service Account Key dialog to allow auto updating on the run configuration

Extension of the Service Account Key Created dialog to allow automatic updating of selected Google App Engine Standard run configurations

This feature is only available in Ultimate Edition because the Google App Engine Standard run configurations are only available there
The PR adds a table in the Service Account Key Created dialog that shows the existing Google App Engine run configurations, allowing users to select which one they would like to update to add the Google Cloud Library environment variables: GOOGLE_CLOUD_PROJECT and GOOGLE_APPLICATION_CREDENTIALS
If the selected run configuration already has the GOOGLE_CLOUD_PROJECT and GOOGLE_APPLICATION_CREDENTIALS environment variables, they would be updated
When there are no Google App Engine Standard run configurations, this table will be hidden and look like the dialog in the Community Edition

Service Account Key Created dialog in IC and when no App Engine Standard run configurations exist:

Service Account Key Created dialog in IU and when there are App Engine Standard run configurations:

PR is not complete. Need to add tests and address the following:

Need to figure out why the fix for the nesting forms from different modules isn't working
Not printing the name of the configuration, not sure why this is happening. I am using a renderer that calls getName but the ouput looks like something is calling toString()

Updated UI - with fix for name of run configuration

Is this ready now for review?

Yes, please review. Thanks

@nbashirbello I noticed an issue where when I rename the local run configurations that updating the env vars stops working. Testing again to see if its consistent.

@etanshaul I couldn't reproduce the bug. Any chance reproducing it?

@nbashirbello was trying to reproduce but for some reason now the env vars are never added regardless of what I do. Trying to figure out whats going on.

Also, another thing - many Cloud Libraries quick starts and examples have nothing to do with App Engine at all. Adding these two environment variables into any generic Java run configuration (i.e. plain main() method) does the magic and the quick start works like a charm without all the additional manual steps. Should we show the complete list of run configurations to add environment variables to, not just App Engine ones? This could be very useful for people using Cloud Libraries.

@ivanporty thats an interesting question. We originally discussed just adding them to App Engine ones since we presumably wouldn't know anything about these other run configurations (Jetty?, Tomcat?, something else entirely?). Also it may be weird if run configurations entirely unrelated to local run show up there. wdyt? I'm also ok addressing this issue in a subsequent PR.

Yep, I agree some run configurations may be completely non-relevant, but we could probably include at least plain Java application with main() in order to make testing and quick starts for all Cloud Libraries super easy. Definitely let's discuss this and address in a separate PR. I just tried a couple of Cloud APIs using our tools and manually adding variables - very nice experience!

+1 to adding this to arbitrary run configs.  You can always filter them by
configs we know make sense. e.g. spring boot, java app, app servers.
On Fri, Apr 20, 2018 at 10:36 AM Ivan Portyankin<EMAIL_ADDRESS>wrote:

Yep, I agree some run configurations may be completely non-relevant, but
we could probably include at least plain Java application with main() in
order to make testing and quick starts for all Cloud Libraries super easy.
Definitely let's discuss this and address in a separate PR. I just tried a
couple of Cloud APIs using our tools and manually adding variables - very
nice experience!
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/GoogleCloudPlatform/google-cloud-intellij/pull/2023#issuecomment-383116492,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AHf5HXqIMdMNkIiakg09naD5KJ0GUdKkks5tqfJXgaJpZM4TRzUN
.

@etanshaul @ivanporty In regards to the env vars not getting added to the selected run configurations, wanted to confirm that you are clicking the "Apply" button. Are you?

Oh I was just hitting "OK" at the end of the sequence, didn't do "Apply". It works with "Apply", could we also add the variables if checkbox in the table is on and we click "OK" - seems very natural?

are you suggesting that it only works with "apply" and not "ok"?
I think I've just been hitting "OK" from that dialog. You shouldnt need to explicitly hit "Apply" since OK includes that (plus closing the dialog).

@etanshaul To verify, the dialog would look like below, and the OK button would updated the selected run configurations and close the dialog while the Cancel button would only close the dialog. And when there are no App Engine run configurations or the user is in IC, the Cancel button would be hidden.

I think the Cancel button might be a bit confusing because this dialog comes up when the user is trying to add a client library and they might wonder if Cancel means cancel the Client Library addition. But once the user click on OK or Cancel, there will see the message that X client library has been added so maybe this is ok...

What about using the names "Update" and "Close" instead of "Ok" and "Cancel" respectively. I am fine leaving the buttons as OK/Cancel if no one else thinks it might be confusing :)

It slightly feels like Cancel might undo something related to service accounts although it won't. How about we name the buttons a bit more precise, like "Add Variables" and "Close"?

+1 to those suggestions

Updated the dialog:

nice! will test and review again

Looks great!
UX nit: could we possibly disable "Add Variables" when no run configurations are selected in the table (i.e. all are unchecked)? Otherwise a user could have an impression variables are going to be added anyway since the action is named like this and it's active.

lgtm after the last couple of small comments. Will defer to @ivanporty for the final approval.


Update MD file on new branch 2017-23-02_11-12-40

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

split/updateTestRef.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


APIView - SDK review gen from swagger PR fails with auth error

APIView fails SDK review request if Azure Org membership is private for PR author, and this has been done to run review gen steps for authorized users only. We are seeing a too many failures due to this reason and causing swagger to SDK automation not getting much benefit unless users change their membership type. Alternate option is to verify Azure org membership using Microsoft opensource URL to avoid these 401 errors.

just meddling

We will change authentication to use MSFT opensource API to check if PR author is internal employee or not.


Add scalpel-core

This is the first step required to allow wikicfp-scraper to depend on scalpel v0.5.0 in #2269.

Thanks!


tabs not saving

tabs arent saving, maybe this is because im multi logging, can you reccomend a way to make it save if you know how

I think the only way to make sure it saves is to close the runelite client that has the changes you want saved after you close all other clients.
Runelite does not have any support for having your savedata work intuitively with multiple instances of the clients, other than having each client use a completely separate set of saveadata, which I doubt is what you want.


feat(lex,parse,interp,e2e): Implement print lists

Roku's BrightScript supports printing multiple things with one print statement, so we should too!  This was surprisingly more difficult than I expected, but a pretty big part of that was making it possible to spy on output without messing with process.stdout.write.
Merging this PR will result in the following file being valid according to brs:
' All examples inspired by Roku documentation:
' https://sdkdocs.roku.com/display/sdkdoc/Program+Statements#ProgramStatements-PRINTitemlist

' no separators
print "lorem " 1 "psum"

' arbitrary mixes of separators
print 9; " is equal to"; 3^2
print "column a", "column b", "column c", "column d"

' arbitrary tabbing
print tab(3) "I started at col 3"; tab(25) "I started at col 25"

' print position
print "0123" pos(0)

' calculate tabs based on position, though you'd probably want to just define "    " as a variable
print "lorem" tab(pos(0) + 4) "ipsum" tab(pos(0) + 4) "dolor" tab(pos(0) + 4) "sit" tab(pos(0) + 4) "amet"

' suppress trailing newline
print "no newline";

This still needs a good bit of documentation in a lot of places; I'll try to cover that in the next few days!


⚠ Telly has degraded performance

In 31cc33b, Telly ($TLY) experienced degraded performance:

HTTP code: 200
Response time: 1247 ms

Resolved: Telly performance has improved in ec773ef.


support component factory methods

Many of our legacy components are created with componentFactory({...}) instead of React.createClass. Ideally it'd support any factory function name.

Fair! Want to make a pull request? We can change react-transform options to be an object with transforms array and optional factoryMethods array. By default factoryMethods is ['React.createClass'] but you can change it to ['React.createClass', 'componentFactory'] or even ['componentFactory'].
In the source, this code would be changed to look at the plugin options. Here's how we read plugin options.

Let's discuss in https://github.com/gaearon/babel-plugin-react-transform/issues/8.


Don't emit unclean git status error if only uncommitted change is adding simple_deploy to INSTALLED_APPS.

After implementing the check for a clean git status, the --automate-all workflow now involves four steps. People need to install django-simple-deploy, add it to INSTALLED_APPS, commit that change, and run simple_deploy.
We want to bail on an unclean git status if there are uncommitted changes beyond adding simple_deploy to INSTALLED_APPS. But if this is the only change, we can commit this change. In the configuration-only workflow, we can also ignore this change and let the user commit that change along with the configuration changes.

[ ] If adding simple_deploy to INSTALLED_APPS is the only uncommitted change, don't emit the unclean git status error.

[ ] For fly_io deployments.

[ ] configuration-only deployments
[ ] --automate-all deployments

[ ] For platform_sh deployments.

[ ] configuration-only deployments
[ ] --automate-all deployments

[ ] For heroku deployments.

[ ] configuration-only deployments
[ ] --automate-all deployments

Implementation
If unclean git status, run git diff. If the only output is about simple_deploy in settings.py, continue with configuration.

Working notes

[ ] Write a unit test that fails for this now?


chore(main): release 1.4.1

:robot: I have created a release beep boop
1.4.1 (2023-03-01)
Bug Fixes

modular builtin separation (#426) (c5933ea)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/fastly/js-compute-runtime/releases/tag/v1.4.1 :sunflower:


Support continuous highlighting in drawMolecules().

There's a workaround to get roughly the same effect from client code, but it's a pain. This is an easier (and more intuitive) solution to the problem: if continuousHighlighting is on and you call drawMolecules() and just provide the atoms to be highlighted, then the behavior is the same as drawMolecule() in the same situation - bonds connecting the highlighted atoms will also be highlighted.
Also adds a fillHighlights option to DrawOptions

merging this so that the functionality is more easily available for today's hackathon


HTTP::Server includes content-length header of 0 on HEAD requests

require "http/server"

server = HTTP::Server.new do |context|
  context.response.content_type = "text/plain"
  context.response.print "Hello world!" unless "HEAD" == context.request.method
end

address = server.bind_tcp 8080
puts "Listening on http://#{address}"
server.listen

Doing GET localhost:8080, correctly returns the content-length of 12.  However doing HEAD localhost:8080 sets the content-length to 0.  This currently breaks the RFC, which states:

A server MAY send a Content-Length header field in a response to a
HEAD request (Section 4.3.2 of [RFC7231]); a server MUST NOT send
Content-Length in such a response unless its field-value equals the
decimal number of octets that would have been sent in the payload
body of a response if the same request had used the GET method.

Given the server can't know the length of the content in the GET context, we should probably just not include it at all.

Currently, there is no special handling for HEAD requests at all. But I think it makes sense to not add a default Content-Length: 0 header when responding to HEAD. This needs to be communicated from request to response instance, but that shouldn't be too difficult.


Camera permissions not click allow and deny direct callback

Please forgive my English is not good
Expected behaviour
Camera access normal callback
Actual behaviour
Camera permissions not click allow and deny direct callback
Steps to reproduce
 mServiceScanView.setOnClickListener(new View.OnClickListener() {
            @Override
            public void onClick(View view) {
                Dexter.withActivity(getActivity())
                        .withPermission(Manifest.permission.CAMERA)
                        .withListener(ServiceLazyFragment.this)
                        .check();
            }
        });

Click on the screen after permission to apply for dialog，At the same time direct callback,I don't have a click allow and deny
 @Override
    public void onPermissionGranted(PermissionGrantedResponse response) {
        KLog.e("允许相机权限");
    }

E/ServiceLazyFragment.java: [ (ServiceLazyFragment.java:179)#OnPermissionGranted ] 允许相机权限
Mobile version：

Version of the library
com.karumi:dexter:4.1.0

I think thats is a bug. You changed your camera permission manually. Unistall your app and reinstall.

Uninstall or reinstall is still the same problem ，I don't have to manually modify permissions, is through the code to access

@qinyang1018 what compilationversion are you using in you build.gradle?

I'm sorry   To reply until now

this is my build.gradle

Please be sure that you have your Camera permission in your AndroidManifest.xml. Had this same behaviour times ago.

it have on AndroidManifest.xml

Is this problem solved ？


Refactored Code to support Assistant V2

API code changes to Assistant to support session id
Update context to latest version

Adding @Oliver-I

@stevenpkg can you update the README.md with the new instructions?
I would like for you to also test this with an app running in the IBM Cloud rather than locally so that we can be sure it will work as expected

@germanattanasio you can test a dev version here
http://watson-assistant-intermediate-dev.mybluemix.net/
What changes to the readme would you like to see done?

If you can say that this demo is using Assistant V2(link to the documentation) and to access the V1 use this url: v1.1.0

@germanattanasio will make the changes you requested. Also can we keep a V1 snapshot/version so anyone that want to look at V1 code can.

@stevenpkg I have updated the readme file on your branch.
@germanattanasio we always test on the cloud using dev UIs to see how the system reacts within the cloud.

@stevenpkg All the releases are stored in the Releases tab https://github.com/watson-developer-cloud/assistant-intermediate/releases
This will be a major release so we will start with `2.0.0


Add Code Action to handle trailing . in floating point literal.

Ballerina spec 2021R2 states that, floating point literal must not end with a ..

Closing with https://github.com/ballerina-platform/ballerina-lang/pull/33584


Azure AD connect 2.0 - change recommendations from LAPS to Windows LAPS

Dear @billmath
The Azure AD connect 2.0 prerequisites reference to use LAPS. Since LAPS is quite replaced by Windows LAPS, I kindly request to update the reference and recommendations. Windows LAPS has been Integrated into the OS with current CUs.
https://github.com/MicrosoftDocs/azure-docs/blob/main/articles/active-directory/hybrid/connect/how-to-connect-install-prerequisites.md

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

ID: 85137d77-0d5f-e461-5d56-05c775243109
Version Independent ID: 2cb6d696-e3f6-7b27-8479-0bac0c1bf0e7
Content: Azure AD Connect: Prerequisites and hardware - Microsoft Entra
Content Source: articles/active-directory/hybrid/connect/how-to-connect-install-prerequisites.md
Service: active-directory
Sub-service: hybrid
GitHub Login: @billmath
Microsoft Alias: billmath

Thank you!

@Karl-WE Thanks for this.  I have updated the docs and it should live later today.  Thanks!
#please-close

Thank you so much @billmath for the quick response and updating the page.

P. S I have heard it will be renamed to Entra ID Connect but that's still in future. Unclear how this will be executed. As they would need to update the app install string, UX and application path. Proposed aliases for the PowerShell commandlets / modules, fitting with the Entra ID naming, as they plan to retain the original commandlets.


Prevent cache error without data

Fixes a reference error when a query returned no data (Using @include in my case).

Hey @LukasHechenberger! Thanks for submitting this - do you mind adding a quick test so that I can make sure we don't lose this change in future refactors?

@LukasHechenberger ping

So sorry, been busy all week. I'll add a test next week

Added a (very) basic test, please allow the CI workflow to run 😉

@LukasHechenberger - looks like one of the tests is failing, got time to take a look?

I'm going to close this since i just opened #228 which completely rewrites the cache implementation to be more forward-facing. thanks again for the contribution!


[prometheus-cpp] Update to version 0.13.0

Update prometheus-cpp to 0.13.0. Locally tested all the features on macOS.

Looks like the agent died during microsoft.vcpkg.pr test:
##[error]We stopped hearing from agent vcpkg-eg-mac-02. Verify the agent machine is running and has a healthy network connection. Anything that terminates an agent process, starves it for CPU, or blocks its network access can cause this error. For more information, see: https://go.microsoft.com/fwlink/?linkid=846610

Could you re-run this test or should I rebase to latest master to trigger a rebuild?

All feature are tested successfully in the following triplet:

x86-windows
x64-windows
x64-windows-static

This pr depends on #20006

/azp run

Thanks for the version update!


Model edit improvements

This PR fixes the following:

[x] Formtools form generator
[x] Update generateFormFromModel() function to exclude fields where field.visible === false first and only process visible fields
[x] Enable the form builder to pass the error back to the router which can be pushed to express error handler. Currently any errors occurred in the form builder is concealed.
[x] Throw error if a form field is of type documentArray and it's not bind to the Linz embeddedDocument plugin
[x] Updated express error handler to output the error stack to console

@smebberson This is ready for review!

@sandytrinh, this is a nice addition. Tidies up some of the internals within Linz. Great one. Please merge.


Be able to change simulator language

It would be great to be able to change simulator language via ControlRoom! If there is an underlaying command available already, I could take over the implementation!

I was also wondering about that. Although I couldn't find a simctl command for setting a language or region, I did find a nice article on NSHipster  that includes a possible solution for this (essentially you manipulate the  container's GlobalPreferences.plist file)

Interesting... the path mentioned in the article is the dataPath value on SimCtl.Device...

In progress on #59 🎉

This was added in #59 🎉


Storage Landscape v2 inaccessible

The PDF in this repository links to a Google doc that says it is "superseded by the second version located here: https://bit.ly/cncf-storage-whitepaperV2". That link requires some kind of authentication.

http://bit.ly/cncf-storage-whitepaper directs to a public Google doc.
https://bit.ly/cncf-storage-whitepaperV2 directs to a restricted Google doc.

Apologies - the google doc sharing was changed by a human error - it has been fixed now.   Also updated the link to the document in the repo to reflect version 2


error[E0658]: use of unstable library feature 'str_strip': newly added

I can't compile anymore after pulling from the latest master.
   Compiling game v0.1.0 (C:\Users\PC\Desktop\AB-Street\abstreet\game)
error[E0658]: use of unstable library feature 'str_strip': newly added
   --> game\src\edit\traffic_signals.rs:129:40
    |
129 |                     if let Some(x) = x.strip_prefix("change duration of phase ") {
    |                                        ^^^^^^^^^^^^
    |
    = note: see issue #67302 <https://github.com/rust-lang/rust/issues/67302> for more information

error[E0658]: use of unstable library feature 'str_strip': newly added
   --> game\src\edit\traffic_signals.rs:133:40
    |
133 |                     if let Some(x) = x.strip_prefix("delete phase ") {
    |                                        ^^^^^^^^^^^^
    |
    = note: see issue #67302 <https://github.com/rust-lang/rust/issues/67302> for more information

error[E0658]: use of unstable library feature 'str_strip': newly added
   --> game\src\edit\traffic_signals.rs:149:40
    |
149 |                     if let Some(x) = x.strip_prefix("move up phase ") {
    |                                        ^^^^^^^^^^^^
    |
    = note: see issue #67302 <https://github.com/rust-lang/rust/issues/67302> for more information

error[E0658]: use of unstable library feature 'str_strip': newly added
   --> game\src\edit\traffic_signals.rs:161:40
    |
161 |                     if let Some(x) = x.strip_prefix("move down phase ") {
    |                                        ^^^^^^^^^^^^
    |
    = note: see issue #67302 <https://github.com/rust-lang/rust/issues/67302> for more information

error[E0658]: use of unstable library feature 'str_strip': newly added
   --> game\src\edit\traffic_signals.rs:173:40
    |
173 |                     if let Some(x) = x.strip_prefix("phase ") {
    |                                        ^^^^^^^^^^^^
    |
    = note: see issue #67302 <https://github.com/rust-lang/rust/issues/67302> for more information

error[E0658]: use of unstable library feature 'str_strip': newly added
   --> game\src\sandbox\dashboards\misc.rs:113:36
    |
113 |                 if let Some(x) = x.strip_prefix("BusRoute #") {
    |                                    ^^^^^^^^^^^^
    |
    = note: see issue #67302 <https://github.com/rust-lang/rust/issues/67302> for more information

error[E0658]: use of unstable library feature 'str_strip': newly added
   --> game\src\sandbox\gameplay\play_scenario.rs:247:40
    |
247 |                     if let Some(x) = x.strip_prefix("delete modifier ") {
    |                                        ^^^^^^^^^^^^
    |
    = note: see issue #67302 <https://github.com/rust-lang/rust/issues/67302> for more information

error: aborting due to 7 previous errors

For more information about this error, try `rustc --explain E0658`.
error: could not compile `game`.

Thanks.

Updating rust version will fix this, at least it fixed in my case

rustup update stable should work. The new language releases bringing in new features are relatively infrequent, so I think it's worth the hassle for now.

Thanks.
I ran rustup update as suggested and then got (similar to https://github.com/dabreegster/abstreet/issues/148):
Reading ../data/system/maps/montlake.bin: 0/4 MB... 0.0001smemory allocation of<PHONE_NUMBER>0 bytes failederror: process didn't exit successfully: `C:\Users\PC\Desktop\AB-Street\abstreet\target\release\game.exe` (exit code: 0xc0000409, STATUS_STACK_BUFFER_OVERRUN)

Running cargo run --bin updater solved it.


Migrate to Clap v4

Again, this is a pretty big update with multiple breaking changes that requires changing how our argument parsing is set up. (If I ignore the effort to upgrade, this update does contain some nice API improvements, so I guess in the long term it's worth it. The blogpost is an interesting read.)
This is still WIP, I'm opening this PR to avoid duplicate work on this.

I think everything should work fine now, but the PR is currently blocked on #352.

@niklasmohrin updated! Should be ready now, I think.

MSRV had to be bumped due to clap dependencies. I picked 1.75, which was released in December 2023.


Persisting error modal after wrong file uploaded as wallet

Environment

Smapp version: any
Node version: any
OS: any

Describe the bug
If we try to open a wallet from a file but choose the incorrect file - any non-empty JSON, smapp will accept it and try to proceed. Of course will not be able to parse it, so we can retry to load the correct file, but once we log out, we get the error modal that persists even after smapp restart. Sometimes we get this error immediately. Only unlinking the wrong file helps to get smapp usable again.
Steps to reproduce

Open Smapp
Try to recover wallet from file
Pick any non-empty JSON file
try lo log in, wait or try to load the correct file, if still no modal, just log out.
try to restart smapp

Expected behavior
smapp should drop the file it could not parse (immediately or after app restart), we should be able to use smapp without restraints
Actual behavior
error modal persists even after restart
https://github.com/spacemeshos/smapp/assets/89876259/81efac7f-5bc0-481a-8665-fec079a719e1
Logs
app-log.1.1.1.txt
spacemesh-log-7c8cef2b.txt
How to fix

How to fix
In case we have an invalid wallet file (or invalid JSON) in the list of wallet files — let's display a popup message with a dismiss button saying: "Wallet file /path/to/wallet.json is invalid. Please verify it and import the wallet file once again."
And such wallet file should be automatically omitted from the list of wallet files.


Exception on dragging outside [Grid layout]

Hi,
I tried the drag and drop on Grid layout. It works nicely, except for this one issue -
Lets say the activity contains the recycler view (grid) in 50% bottom region. And their are sufficient items, so that there are several (say 12) rows in the grid. Now grab an item from last row, and drag it up, fast. The finger should move into the top 50% region, which is outside the recycler view. Now the grabbed items floats above other items, but no swapping is happening. And if the item is released in this situation, the app crashes with this library's exception -
java.lang.IllegalStateException: onDragItemFinished() - may be a bug (mDraggingItem != holder)
        at com.h6ah4i.android.widget.advrecyclerview.draggable.DraggableItemWrapperAdapter.onDragItemFinished(DraggableItemWrapperAdapter.java:257)
        at com.h6ah4i.android.widget.advrecyclerview.draggable.RecyclerViewDragDropManager.finishDragging(RecyclerViewDragDropManager.java:765)
        at com.h6ah4i.android.widget.advrecyclerview.draggable.RecyclerViewDragDropManager.handleActionUpOrCancel(RecyclerViewDragDropManager.java:804)
        at com.h6ah4i.android.widget.advrecyclerview.draggable.RecyclerViewDragDropManager.onTouchEvent(RecyclerViewDragDropManager.java:526)
        at com.h6ah4i.android.widget.advrecyclerview.draggable.RecyclerViewDragDropManager$1.onTouchEvent(RecyclerViewDragDropManager.java:198)

Moving out is perhaps leading to failed condition draggingItem.getItemId() == draggingItemId
And it would be great to have this feature - While dragging an item from second last row to last row, we should be able to drag to empty columns in last row. This should move the very last item.
Thanks!

Fixed in 0.8.0.


unable to drag and drop lines with mouse cursor

Type: Bug
when i try to copy code with cursor selection its just wont select
VS Code version: Code 1.74.2 (e8a3071ea4344d9d48ef8a4df2c097372b0c5161, 2022-12-20T10:29:14.590Z)
OS version: Windows_NT x64 10.0.19045
Modes:
Sandboxed: No

System Info

Item
Value

CPUs
Intel(R) Core(TM) i3-7100U CPU @ 2.40GHz (4 x 2400)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_renderer: enabled_onvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: disabled_off

Load (avg)
undefined

Memory (System)
3.92GB (0.58GB free)

Process Argv
--crash-reporter-id ff13bab8-7d5f-4aa0-ba54-81854a9e3d8b --crash-reporter-id ff13bab8-7d5f-4aa0-ba54-81854a9e3d8b

Screen Reader
no

VM
0%

Extensions: none
A/B Experiments
vsliv368cf:30146710
vsreu685:30147344
python383:30185418
vspor879:30202332
vspor708:30202333
vspor363:30204092
vslsvsres303:30308271
pythonvspyl392:30443607
vserr242:30382549
pythontb:30283811
vsjup518:30340749
pythonptprofiler:30281270
vshan820:30294714
vstes263cf:30335440
vscoreces:30445986
pythondataviewer:30285071
vscod805cf:30301675
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
cmake_vspar411:30581797
vsaa593cf:30376535
pythonvs932:30410667
cppdebug:30492333
vsclangdc:30486549
c4g48928:30535728
dsvsc012cf:30540253
azure-dev_surveyonecf:30548226
pyindex848cf:30577861
nodejswelcome1cf:30587006
3biah626:30602489
iaj6b796:30613358
89544117:30613380
fim-prod:30623723
vscsbc:30628655

pls provide a gif

https://user-images.githubusercontent.com/74134062/210629439-c0c75d3f-4762-4bc4-8972-616f0a4f71eb.mp4
Here i can select the line with cursor

Sir
Please give me solution.... Regarding cursor cant select lines in vs code i
have already posted a short video
On Wed, 4 Jan 2023, 5:59 am Megan Rogge, @.***> wrote:

pls provide a gif
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/vscode/issues/170514#issuecomment-1370342412,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ARVTELS5JQ7FQC72FBIZEUDWQS4OHANCNFSM6AAAAAATQBETNA
.
You are receiving this because you authored the thread.Message ID:
@.***>

Hi. Your GIF shows that the cursor does not change position as you drag to make the selection, it stays on the same spot. This suggests you may not be holding down the left mouse button while doing the selection. Can you please confirm if this is the case?

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


v4: Fix npm JS directory

Fixes #20385 with recursive glob action to ensure js directory is copied over via npm. Verified in a local directory with dummy package.json pointing to the commit this PR includes.
/cc @cvrebert for sanity check

Any chance to get a alpha3.1 with this fix? It's blocking for us (guys behind https://github.com/FezVrasta/bootstrap-material-design).
Thanks so much!


Live Stream transcripts

Hello. This is a feature request vs bug, methinks.
Have you looked at extracting captions from a live stream. If you look at any example (https://www.youtube.com/whitehouse) of a live stream, while the stream is live (key), there are auto-generated subtitles delivered in the videoplayback file that streams in, embedded e.g.
https://r6---sn-8xgp1vo-p5qy.googlevideo.com/videoplayback?expire=1614211486&ei=PpU2YM-ULYm98wTm0L_gDA&ip=<IP_ADDRESS>&id=yhxmnlGtJ-g.1&itag=386&source=yt_live_broadcast&requiressl=yes&mh=zc&mm=44,29&mn=sn-8xgp1vo-p5qy,sn-p5qs7nel&ms=lva,rdu&mv=m&mvi=6&pl=18&initcwndbps=1717500&vprv=1&live=1&hang=1&noclen=1&xtags=lang=en:ttkind=asr&mime=text/mp4&ns=aD6U7aY6idhNPyXEqiXu6K0F&gir=yes&mt=1614189620&fvip=6&keepalive=yes&fexp=23983797&beids=9466586&c=WEB&n=lmOMV3MuzrpzRQ&sparams=expire,ei,ip,id,itag,source,requiressl,vprv,live,hang,noclen,xtags,mime,ns,gir&sig=AOq0QJ8wRAIgd0qHHqBF3aRir-pw93UKhFNuFxrlpe6OqyMerxsZ4JsCIHZK74UbKX7ig08-egt6vMDzP6g_7EhOyuOOoUXAkSVW&lsparams=mh,mm,mn,ms,mv,mvi,pl,initcwndbps&lsig=AG3C_xAwRAIgHa9tABbFKMiVQSnLLWa7iO_iu7pcVtrea43G-zdfGBUCIGbqOL15uN0-32Yki8s5vwXD2XDkvCBUgntS54w9xvjc&alr=yes&cpn=LW2TAYe5jfbjzMjx&cver=2.20210223.09.00&sq=664
Expired, of course, but an example, the payload here is:
ftypmoovlmvhd_@(mvex trex}trak\tkhd@mdia mdhd_UÄ!hdlrtextÐminf$dinfdrefurl ˜stblHstsd8tx3g
ftabsttsstscstcostszVnmhdemsghttp://youtube.com/streaming/metadata/segment/102015ˆ°D«Sequence-Number: 664
Stream-Finished: F
Ingestion-Walltime-Us:<PHONE_NUMBER>022158
Stream-Duration-Us:<PHONE_NUMBER>
Max-Dvr-Duration-Us:<PHONE_NUMBER>0
Target-Duration-Us: 5000000
Encoding-Alias: L1_Ag

Xmoofmfhd@traftfhdß’tfdtÏYztrun`^mdat<?xml version="1.0" encoding="utf-8" ?><timedtext format="3">

what's in the Declassified
report or when it comes out

because many elements of Italy
two years ago when when it was

first first came out if you come
to the conclusion that there

</timedtext>

The timedtext is embedded in the file:
<?xml version="1.0" encoding="utf-8" ?><timedtext format="3">

what's in the Declassified
report or when it comes out

because many elements of Italy
two years ago when when it was

first first came out if you come
to the conclusion that there

</timedtext>

It's not TTMLv3 but we get this text is associated with sequence #664 from the URL. The t= appears to be millisecond designation relative to the sequence chunk, and "d" appears to be the duration. But even absent that, the stream of text is there. Note it doesn't appear by default. It appears you need to insert into the "sparams" in the URL "xtags" to get the live captioning, but it appears if you try to insert it, it messes up the hash/key associated with it so it needs to be triggered on (cc_load_policy=1 in URL does NOT seem to work)
youtube-dl et al don't recognize this since it's not being delivered as a standalone subtitle file. Acts like there's no subtitles on the live stream since it doesn't identify as a subtitles file.
Thoughts?

Hi @frisch1, I would definitely say that this is a feature request and not a bug. Sounds interesting, but I don't see myself implementing this anytime soon, as this module is mostly used for data-science purposes and I don't really see the use-case for livestreams. However, if you want to contribute this feature I'd be happy to merge it. Deserializing the response probably isn't a big deal, you just gotta find out how to scrape the URL you'll have to call to actually get that response. Let me know if you have that figured out and are interested in contributing it, so we can have a chat on how to implement this into the current API 😊


Recipe glitch

So, first of all I would like to point out that in game it has a CHANCE parameter for the recipe command.
Seen here: http://puu.sh/qQ7M8/1a7d9d2fb5.png
And here: http://puu.sh/qQ7Mr/1d09e3bf77.png
Yes, I'm on 1.10. On my server. As we speak.
Secondly, I would like to inquire as to the ability to use rpgitems in the recipe for other rpgitems. I seem to be having trouble with it to an extent. I think the problem lies in the info that is saved. I would recommend ChatColor.stripColor(), but I'm not sure if that would fix the problem.
Reason why is here: http://puu.sh/qQ7Vs/d68c1c38ff.png
Thanks friends.
-Face

And the wiki page here should be updated with correct parameter.

May be a duplicate of #262.
Wiki updated.


add a command where you can change the console font

either change it to a built-in font, or you can provide your own font via link

ah yes set it to a variably-spaced font that'll look nice

shush shush shush shush that's for the user to figure out

ok this isnt really possible so nevermind
well it is possile, but it's annoying cause i have to mess with css, so its bad


Support Docker build arguments

Currently, users that rely on docker build arguments cannot pass them via this plugin, we should support passing them so we can set the relevant --build-arg flags.

Closed by: https://github.com/equinixmetal-buildkite/docker-build-buildkite-plugin/pull/11


Add kubernetes client version to image

There is no reason to allow the image to consume another
version of kubernetes-client than the necessary/tested one.
This makes it explicit.

I wonder if we shouldn't parameterize the version to make it easier and less error prone to switch to 1.19 later this year.

I am not sure how that would work, but it makes sense.

This makes me think we should also parametrize other k8s images. I am approving the PR to not unblock any testing and we can parametrize this images together with the other k8s images, if I am not wrong this should be a simple change I can verify it this afternoon.

if I am not wrong this should be a simple change I can verify it this afternoon.

I was wrong sorry, I thought about including a placeholder as we do for tags, which could work, but then we are just moving the problem from the .kiwi file to the _service file, at the end we need a reference to the specific k8s package, unless we make use of Provides without version and assume the image to always use the latest version available.

I think this should continue to use a specific version of kubernetes, as this is very explicit. Alternatively, we could say in the kured packaging that we need a certain version of kubernetes-client, in this case, kubernetes-1.18-client.


nixos/nomad: init

Motivation for this change
Noticed that while consul has a NixOS module, nomad does not. Decided to add
it since it's needed at work.
cc. @cpcloud for testing
cc. @flokli for review
Things done

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

[x] NixOS
[ ] macOS
[ ] other Linux distributions

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

@lovesegfault I've taken the liberty to push some fixup commits into this PR that address many of the issues found above.
Please have a look at the changes; then we can squash them.
Using the fixed module, I've managed to get nomad started and it runs docker images fine, with this config:
{
  services.nomad = {

    enable = true;
    settings = {
      data_dir = "/var/lib/${config.systemd.services.nomad.serviceConfig.StateDirectory}";
      server = {
        enabled = true;
        bootstrap_expect = 1; # for demo; no fault tolerance
      };
      client = {
        enabled = true;
        network_speed = 1000; # Mbit/s
      };
    };
  };
}

I believe we should default enableDocker = true; as I've done here now, because the official Nomad tutorial uses docker as an example.
The example above includes network_speed = 1000; # Mbit/s because that's also in the official tutorial, but it seems to be either no longer used, or undocumented; I've pointed that out upstream: https://github.com/hashicorp/nomad/issues/9385#issuecomment-759154171

@lovesegfault I've taken the liberty to push some fixup commits into this PR that address many of the issues found above.
Please have a look at the changes; then we can squash them.
Using the fixed module, I've managed to get nomad started and it runs docker images fine, with this config:
{
  services.nomad = {

    enable = true;
    settings = {
      data_dir = "/var/lib/${config.systemd.services.nomad.serviceConfig.StateDirectory}";
      server = {
        enabled = true;
        bootstrap_expect = 1; # for demo; no fault tolerance
      };
      client = {
        enabled = true;
        network_speed = 1000; # Mbit/s
      };
    };
  };
}

I believe we should default enableDocker = true; as I've done here now, because the official Nomad tutorial uses docker as an example.
The example above includes network_speed = 1000; # Mbit/s because that's also in the official tutorial, but it seems to be either no longer used, or undocumented; I've pointed that out upstream: https://github.com/hashicorp/nomad/issues/9385#issuecomment-759154171

@nh2 Thanks for taking this on!

@nh2 Thanks for taking this on!

Rebased this branch on the latest master, also pushed some additional fixups.
@nh2 Thanks for all the help :)

Rebased this branch on the latest master, also pushed some additional fixups.
@nh2 Thanks for all the help :)

Are there any objections before I squash these commits and merge?
cc. @aanderse @cpcloud @nh2

Are there any objections before I squash these commits and merge?
cc. @aanderse @cpcloud @nh2

@lovesegfault yeah this looks great :+1: Awesome work :tada:

@lovesegfault yeah this looks great :+1: Awesome work :tada:

LGTM!

LGTM!

The example above includes network_speed = 1000; # Mbit/s because that's also in the official tutorial, but it seems to be either no longer used, or undocumented; I've pointed that out upstream: hashicorp/nomad#9385 (comment)

Upstream replied; it's truly obsolete so we should remove it:

https://github.com/hashicorp/nomad/issues/9385#issuecomment-759443116
https://github.com/hashicorp/nomad/pull/9793

The example above includes network_speed = 1000; # Mbit/s because that's also in the official tutorial, but it seems to be either no longer used, or undocumented; I've pointed that out upstream: hashicorp/nomad#9385 (comment)

Upstream replied; it's truly obsolete so we should remove it:

https://github.com/hashicorp/nomad/issues/9385#issuecomment-759443116
https://github.com/hashicorp/nomad/pull/9793

The example above includes network_speed = 1000; # Mbit/s because that's also in the official tutorial, but it seems to be either no longer used, or undocumented; I've pointed that out upstream: hashicorp/nomad#9385 (comment)

Upstream replied; it's truly obsolete so we should remove it:
* [hashicorp/nomad#9385 (comment)](https://github.com/hashicorp/nomad/issues/9385#issuecomment-759443116)

* [hashicorp/nomad#9793](https://github.com/hashicorp/nomad/pull/9793)

Removed :)

The example above includes network_speed = 1000; # Mbit/s because that's also in the official tutorial, but it seems to be either no longer used, or undocumented; I've pointed that out upstream: hashicorp/nomad#9385 (comment)

Upstream replied; it's truly obsolete so we should remove it:
* [hashicorp/nomad#9385 (comment)](https://github.com/hashicorp/nomad/issues/9385#issuecomment-759443116)

* [hashicorp/nomad#9793](https://github.com/hashicorp/nomad/pull/9793)

Removed :)

Will merge this tonight

Will merge this tonight

Thanks everyone for the help and reviews!

Thanks everyone for the help and reviews!

Great work, y'all!

Great work, y'all!

I think turning docker back on by default was forgotten:
https://github.com/NixOS/nixpkgs/pull/105739#discussion_r559244603
Also @aanderse's comment from https://github.com/NixOS/nixpkgs/pull/105739/files#r557693150 (especially the one linked in there), which is a good point.

I think turning docker back on by default was forgotten:
https://github.com/NixOS/nixpkgs/pull/105739#discussion_r559244603
Also @aanderse's comment from https://github.com/NixOS/nixpkgs/pull/105739/files#r557693150 (especially the one linked in there), which is a good point.

I think turning docker back on by default was forgotten:
#105739 (comment)
Also @aanderse's comment from https://github.com/NixOS/nixpkgs/pull/105739/files#r557693150 (especially the one linked in there), which is a good point.

I'll push a fix to enable docker by default again right away.
The other comment I think deserves a follow up PR.

I think turning docker back on by default was forgotten:
#105739 (comment)
Also @aanderse's comment from https://github.com/NixOS/nixpkgs/pull/105739/files#r557693150 (especially the one linked in there), which is a good point.

I'll push a fix to enable docker by default again right away.
The other comment I think deserves a follow up PR.

Addressed the default issue: e1340190a9691bb236793dfd78a055cc7f0fb147

Addressed the default issue: e1340190a9691bb236793dfd78a055cc7f0fb147


docs: generate OpenAPI specification

Description of change
Generating a OpenAPI specification file openapi.yaml is faster, more convenient and less error-prone than having to edit it by hand for each change to the API. The file is generated by executing an automated unit test.
Links to any relevant issues
n/a
How the change has been tested
Generated openapi.yaml file provides a good DX when loaded into Swagger UI and Scalar.
Definition of Done checklist
Add an x to the boxes that are relevant to your changes.

[ ] I have followed the contribution guidelines for this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] I have successfully tested this change in a docker environment

 docs: generate OpenAPI specification


dice stats is colliding with token mirror button

When both dice stats and token mirror button are installed and activated, dice stats is working, but token mirror button will be prohibited from loading and will not function at all.

As soon as I disable dice stats, the other module works as inteded again.
Foundry version 10 / Build 291
Dice Stats version 1.10.0 and 1.11.0
Token Mirror Button version 1.4.0
I'm not sure why it is behaving like this, but after disabling all modules and then enabling them one by one, Dice Stats seems to be the one that clashes with Token Mirror Button (or vice versa).

Looks like a collision with my global value being the same as someone else.
Latest release (1.11.1) Should fix it. Going to close the issue but if its still broken lmk


FixMulti table plan not include task info

Proposed changes
Issue Number: close #xxx

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

(From new machine)TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 47.74 seconds
stream load tsv:          516 seconds loaded<PHONE_NUMBER>9 Bytes, about 138 MB/s
stream load json:         20 seconds loaded<PHONE_NUMBER> Bytes, about 112 MB/s
stream load orc:          65 seconds loaded<PHONE_NUMBER> Bytes, about 16 MB/s
stream load parquet:          31 seconds loaded 861443392 Bytes, about 26 MB/s
insert into select:          29.2 seconds inserted 10000000 Rows, about 342K ops/s
storage size:<PHONE_NUMBER>9 Bytes


sticks do not work

My sticks do not appear in the menu via creative mode I do not know why I have all the modes and do not show me please help

are you on the Fabric version?

I'm sorry I went to sleep. Yes I'm on fabric version

Yes I'm on fabric version

They should be in the misc tab(lava bucket) of the creative inventory. If they aren't there make sure you're on the latest version

Ok thanks uh in need to instal new version BRUH


🐞In some dapps, the Dapp/Functions page doesn't load if you switch between Dapp/Functions and Dapp/State fast

Describe the bug
A clear and concise description of what the bug is.
🐞In some dapps (for example, ETHTurin), the Dapp/Functions page doesn't load if you switch between Dapp/Functions and Dapp/State fast.
To Reproduce

Go to ETHTurin's page.
Navigate to Dapp/State.
Navigate between Dapp/State and Dapp/Functions a few times.
Dapp/Functions will eventually be empty.

Expected behavior
The contents of the page should appear
Screenshots

Web3 Scenario (please complete the following information):

Env.: Desktop
OS: macOS 10.15
Browser: Safari
Version: 13.1.2
Provider: Metamask

Reward
Discord Username and #: the4kman#4895
ETH Wallet: 0xaa8070363a48e6C245FAb811054bBF2B0B2853D7

This is a collateral effect of this issue: https://github.com/b-u-i-d-l/dfo-hub/issues/21


Using backslash to start a new line in the message window can not work on latest preview release

Describe the bug
Using backslash to start a new line in the message window can not work on latest preview release.
Following is my AL code. Is this the latest preview version's bug?
local procedure ShowMessage()
    var
        lText: Text;
        iNumber: Decimal;
    begin
        lText := 'ABCDE';
        iNumber := 12345.678;
        Message('You can use message windows to display text and numbers: \' + 'The number:%1\' + 'The text:%2', iNumber, lText);
    end;

The result on latest preview release:

The result on current BC version is expected:

AL Language: 3.0.168874

Business Central: 14.0.29537.31808-US

AL Language (Preview Release): 4.0.170493

Business Central (Preview Release): 15.0.36145.0-us

I have the same issue on using full release of 15.0 (Platform 15.0.36510.0 + Application 15.0.36560.0) with AL Language 4.0.182565

Seems related to #5350

This is fixed now:

Version: GB Business Central 17.3 (Platform 17.0.20458.21149 + Application 17.3.20469.21180)

This is fixed now:

Version: GB Business Central 17.3 (Platform 17.0.20458.21149 + Application 17.3.20469.21180)

The same bug on Version: DK Business Central 21.4 (Platform 21.0.52534.53780 + Application 21.4.52563.52602)

Yes, it came back, and was apparently fixed again; from a search:
https://github.com/microsoft/AL/issues/7327


Nginx rtmp error

nginx video server occasionally fails with the following errors. Report was that the stream would inexplicably end after 2-5 minutes.
video_1          | 2022/12/10 23:46:16 [error] 7#7: *2179 auto_push: connect failed: slot=22 pid=29 name='****-****-****-****-****, client: <IP_ADDRESS>, server: <IP_ADDRESS>:1935
video_1          | 2022/12/10 23:46:16 [error] 7#7: *2179 auto_push: connect failed: slot=23 pid=30 name='****-****-****-****-****', client: <IP_ADDRESS>, server: <IP_ADDRESS>:1935
video_1          | <IP_ADDRESS> [10/Dec/2022:23:46:16 +0000] PUBLISH "me3" "****-****-****-****-****" "" - 581 412 "" "FMLE/3.0 (compatible; FMSc/1.0)" (5s)
video_1          | 2022/12/10 23:46:26 [alert] 7#7: 512 worker_connections are not enough
video_1          | 2022/12/10 23:46:36 [alert] 7#7: 512 worker_connections are not enough
video_1          | 2022/12/10 23:46:46 [alert] 7#7: 512 worker_connections are not enough

Stream key has been stripped from logs. The second error seems to occur every 10 seconds when I checked, however the other error message and previous instances of both messages would sometimes fire rapidly -- 20+ messages/second.

This issue reportedly hasn't happened in quite a while.


mate.eom: 1.21.0 -> 1.21.2

Semi-automatic update generated by https://github.com/ryantm/nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/eom/versions.
meta.description for mate.eom is: '"An image viewing and cataloging program for the MATE desktop"'.

Checks done (click to expand)

built on NixOS
/nix/store/ppdc5916y2qkix95f5q88cnaj221n5sy-eom-1.21.2/bin/eom passed the binary check.
/nix/store/ppdc5916y2qkix95f5q88cnaj221n5sy-eom-1.21.2/bin/.eom-wrapped passed the binary check.
2 of 2 passed binary check by having a zero exit code.
0 of 2 passed binary check by having the new version present in output.
found 1.21.2 with grep in /nix/store/ppdc5916y2qkix95f5q88cnaj221n5sy-eom-1.21.2
directory tree listing: https://gist.github.com/46cf95da50ef4497a525cafada082648
du listing: https://gist.github.com/cd8f9476926b715feaacd19792e1d733

Test this build with Cachix (click to expand)

One time setup in nixpkgs Git checkout:
cachix use r-ryantm
git remote add r-ryantm https://github.com/r-ryantm/nixpkgs.git

Test this build:
git fetch r-ryantm && git checkout a5addc1d1df67f1f984889055654e44a0b7dd887 && nix-shell --pure -I nixpkgs=. -p mate.eom

cc @romildo for testing.

No attempt on x86_64-darwin (full log)
The following builds were skipped because they don't evaluate on x86_64-darwin: mate.eom
Partial log (click to expand)

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowUnsupportedSystem = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowUnsupportedSystem = true; }
to ~/.config/nixpkgs/config.nix.

Success on x86_64-linux (full log)
Attempted: mate.eom
Partial log (click to expand)
shrinking RPATHs of ELF executables and libraries in /nix/store/7jyr3gnl093ya785bm17lb245lf4mpk2-eom-1.21.2
shrinking /nix/store/7jyr3gnl093ya785bm17lb245lf4mpk2-eom-1.21.2/bin/.eom-wrapped
shrinking /nix/store/7jyr3gnl093ya785bm17lb245lf4mpk2-eom-1.21.2/lib/eom/plugins/libreload.so
shrinking /nix/store/7jyr3gnl093ya785bm17lb245lf4mpk2-eom-1.21.2/lib/eom/plugins/libfullscreen.so
shrinking /nix/store/7jyr3gnl093ya785bm17lb245lf4mpk2-eom-1.21.2/lib/eom/plugins/libstatusbar-date.so
gzipping man pages under /nix/store/7jyr3gnl093ya785bm17lb245lf4mpk2-eom-1.21.2/share/man/
strip is /nix/store/h0lbngpv6ln56hjj59i6l77vxq25flbz-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/7jyr3gnl093ya785bm17lb245lf4mpk2-eom-1.21.2/lib  /nix/store/7jyr3gnl093ya785bm17lb245lf4mpk2-eom-1.21.2/bin
patching script interpreter paths in /nix/store/7jyr3gnl093ya785bm17lb245lf4mpk2-eom-1.21.2
checking for references to /build in /nix/store/7jyr3gnl093ya785bm17lb245lf4mpk2-eom-1.21.2...

Success on aarch64-linux (full log)
Attempted: mate.eom
Partial log (click to expand)
shrinking /nix/store/ls61zijm8ad2nvpra080lfpw8hamwvis-eom-1.21.2/lib/eom/plugins/libstatusbar-date.so
shrinking /nix/store/ls61zijm8ad2nvpra080lfpw8hamwvis-eom-1.21.2/lib/eom/plugins/libreload.so
shrinking /nix/store/ls61zijm8ad2nvpra080lfpw8hamwvis-eom-1.21.2/lib/eom/plugins/libfullscreen.so
shrinking /nix/store/ls61zijm8ad2nvpra080lfpw8hamwvis-eom-1.21.2/bin/.eom-wrapped
gzipping man pages under /nix/store/ls61zijm8ad2nvpra080lfpw8hamwvis-eom-1.21.2/share/man/
strip is /nix/store/y4ymnvgxygpq05h03kyzbj572zmh6zla-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/ls61zijm8ad2nvpra080lfpw8hamwvis-eom-1.21.2/lib  /nix/store/ls61zijm8ad2nvpra080lfpw8hamwvis-eom-1.21.2/bin
patching script interpreter paths in /nix/store/ls61zijm8ad2nvpra080lfpw8hamwvis-eom-1.21.2
checking for references to /build in /nix/store/ls61zijm8ad2nvpra080lfpw8hamwvis-eom-1.21.2...
/nix/store/ls61zijm8ad2nvpra080lfpw8hamwvis-eom-1.21.2


pyOCD crashes when GDB issues the load command.

When I issue load from the gdb console, pyOCD emits the following block and crashes.
Exception in thread Thread-1:
Traceback (most recent call last):
  File "/usr/local/Cellar/python/2.7.8_2/Frameworks/Python.framework/Versions/2.7/lib/python2.7/threading.py", line 810, in __bootstrap_inner
    self.run()
  File "/usr/local/lib/python2.7/site-packages/pyOCD/gdbserver/gdbserver.py", line 152, in run
    [resp, ack, detach] = self.handleMsg(data)
  File "/usr/local/lib/python2.7/site-packages/pyOCD/gdbserver/gdbserver.py", line 219, in handleMsg
    return self.flashOp(msg[2:]), 1, 0
  File "/usr/local/lib/python2.7/site-packages/pyOCD/gdbserver/gdbserver.py", line 360, in flashOp
    self.flashBuilder.addData(write_addr, self.unescape(data[idx_begin:len(data) - 3]))
  File "/usr/local/lib/python2.7/site-packages/pyOCD/flash/flash_builder.py", line 115, in addData
    operation.addr, operation.addr + len(operation.data)))
ValueError: Error adding data - Data at 0x0..0x198 overlaps with 0x0..0x198

The context may be relevant.  On gdb, the command sequence is:
target remote :3333
(gdb) load
(gdb) c
^C
Program received signal SIGINT, Interrupt.
(gdb) load

Thanks for the report.  I just put up #126 to fix this.

@bremoran please retest, the fix was merged

I can confirm that this is fixed.


Facial Expression recognition

Aim

Recognition of facial expression of a person

Details

As I have seen there is face detection script present in the repository, so facial expression recognition is something similar to that which should also be there.

@Debashish-hub assigned to you,


link to HoloLens specific APIs on this page would help navigation

https://docs.microsoft.com/en-us/windows/uwp/debug-test-perf/device-portal-api-hololens

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

ID: deaa22ed-9e69-2068-1793-8417d40a4f7c
Version Independent ID: 34296d5a-4c99-2d82-4be8-0a398e79ec74
Content: Device Portal for HoloLens - UWP app developer
Content Source: windows-apps-src/debug-test-perf/device-portal-hololens.md
Service: unspecified
GitHub Login: @PatrickFarley
Microsoft Alias: pafarley

Thanks for the feedback, StephenHodgson,
I've added a link in the See also section of the doc in question.


Binary incompatibility with prometheus java  simpleclient versions 0.11+

Please answer these questions before submitting a bug report.
What version of OpenCensus are you using?
0.16.1
What JVM are you using (java -version)?
openjdk version "11.0.9" 2020-10-20
OpenJDK Runtime Environment (build 11.0.9+11-alpine-r0)
OpenJDK 64-Bit Server VM (build 11.0.9+11-alpine-r0, mixed mode)
What did you do?
I've instrumented java server while using prometheus simple client version 0.11.0
What did you expect to see?
No errors
What did you see instead?
Jul 28, 2021 9:38:02 PM io.opencensus.exporter.stats.prometheus.PrometheusStatsCollector describe
WARNING: Exception thrown when describing metrics.
java.lang.NoSuchFieldError: UNTYPED
	at io.opencensus.exporter.stats.prometheus.PrometheusExportUtils.<clinit>(PrometheusExportUtils.java:91)
	at io.opencensus.exporter.stats.prometheus.PrometheusStatsCollector.describe(PrometheusStatsCollector.java:149)
	at io.prometheus.client.CollectorRegistry.collectorNames(CollectorRegistry.java:98)
	at io.prometheus.client.CollectorRegistry.register(CollectorRegistry.java:50)
	at io.prometheus.client.Collector.register(Collector.java:193)
	at io.prometheus.client.Collector.register(Collector.java:186)
	at io.opencensus.exporter.stats.prometheus.PrometheusStatsCollector.createAndRegister(PrometheusStatsCollector.java:72)
	at com.hertz.htp.toyota.grpc.ThinClient.run(ThinClient.java:115)
	at com.hertz.htp.toyota.grpc.ThinClient.main(ThinClient.java:266)

Related issue in prometheus tracker: https://github.com/prometheus/client_java/issues/623

Hi, is this issue still available? If yes, I would like to work on it.


Add Retrying(Rpc)Client

Motivation:

It is often useful to support retrying requests several times if error occurred

Modifications:

Add RetryingRpcClient with max retry count, retry condition and back-off settings

Result:
Make client more reliable

Currently, there are two interfaces that affects the retry behavior: Backoff and RetryRequestStrategy.
RetryRequestStrategy tells whether the retry has to be made or not. Backoff tells when the retry has to be made if RetryRequestStrategy decided to retry.
I think Backoff needs to be able to decide whether the retry has to be made or not as well because otherwise a user has to specify 'maxRetries' in RetryRequestStrategy or RetryingClient.
From user's standpoint, it would be rather more natural to specify the number of maximum retries when specifying a backoff strategy. e.g.
Backoff backoff = Backoff.exponential(...).withJitter(....).upTo(3);

... so that RetryRequestStrategy can focus on deciding to retry purely based on the outcome of the request.

Backoff backoff = Backoff.exponential(...).withJitter(....).upTo(3);

and what do you think about this pattern? :-) It would reduce the number of factory methods in Backoff.

Current coverage is 65.41% (diff: 50.00%)

Merging #364 into master will increase coverage by 0.17%

@@             master       #364   diff @@
==========================================
  Files           341        349     +8
  Lines         13660      13736    +76
  Methods           0          0
  Messages          0          0
  Branches       2044       2050     +6
==========================================
+ Hits           8912       8986    +74
- Misses         3800       3809     +9
+ Partials        948        941     -7

Powered by Codecov. Last update 34be4ef...7b5ac1c

Another thing to consider is that a user may want to apply different Backoff strategy to different requests.

Backoff backoff = Backoff.exponential(...).withJitter(....).upTo(3);
and what do you think about this pattern? :-) It would reduce the number of factory methods in Backoff.

SGTM. Fixed to use this pattern. BTW, I think backoff.upTo(3) indicates max backoff is up to 3 (milliseconds).

SGTM. Fixed to use this pattern. BTW, I think backoff.upTo(3) indicates max backoff is up to 3 (milliseconds).

I actually meant 'times'. Client side timeout limits the total time spent already.

Updated. PTAL~


[PROFILE] Picture uploading limits and optimization

For profile pictures in Profile:

[ ] Enable compression
[ ] Have a maximum size( for the mini-server's own good :P )
[ ] Implement blurhash

This item is being tracked in the new omg.lol Feedback Forum. To follow its status, visit: https://feedback.omg.lol/16605124489918


python311Packages.zigpy: 0.60.1 -> 0.60.2, python311Packages.bellows: 0.37.3 -> 0.37.4

Description of changes

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

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

36 packages built:

python310Packages.bellows
python310Packages.bellows.dist
python310Packages.universal-silabs-flasher
python310Packages.universal-silabs-flasher.dist
python310Packages.zha-quirks
python310Packages.zha-quirks.dist
python310Packages.zigpy
python310Packages.zigpy-cc
python310Packages.zigpy-cc.dist
python310Packages.zigpy-deconz
python310Packages.zigpy-deconz.dist
python310Packages.zigpy-xbee
python310Packages.zigpy-xbee.dist
python310Packages.zigpy-zigate
python310Packages.zigpy-zigate.dist
python310Packages.zigpy-znp
python310Packages.zigpy-znp.dist
python310Packages.zigpy.dist
python311Packages.bellows
python311Packages.bellows.dist
python311Packages.universal-silabs-flasher
python311Packages.universal-silabs-flasher.dist
python311Packages.zha-quirks
python311Packages.zha-quirks.dist
python311Packages.zigpy
python311Packages.zigpy-cc
python311Packages.zigpy-cc.dist
python311Packages.zigpy-deconz
python311Packages.zigpy-deconz.dist
python311Packages.zigpy-xbee
python311Packages.zigpy-xbee.dist
python311Packages.zigpy-zigate
python311Packages.zigpy-zigate.dist
python311Packages.zigpy-znp
python311Packages.zigpy-znp.dist
python311Packages.zigpy.dist


Jehan build

reformat for clarity and functionality

So I'll just rename this once I get the toolchains built. But it's a good improvement.


[apptools] Update tests to test only check setup for target platforms on apptools

Impacted tests(approved): new 0, update 2, delete 0
Unit test platform: [Android]
Unit test result summary: pass 2, fail 0, block 0
BUG=https://crosswalk-project.org/jira/browse/XWALK-5624

LGTM


Using this API to interface with Blue Iris or Shinobi ?

Hi,
I am wondering if this would be a better solution for streaming wyze v3 video/audio to  blue iris, instead of using the NFS wyzehack and js script solution.
If using your API, it is possible to feed the stream to BI, could you provide an example ? This could be a very popular solution to overcome Wyze's lack of RTSP support for the v3 cam.
Thanks

hi @gszakacs, yes, it's definitely possible, and was actually the main reason I put this together.  I have an RTSP bridge app (exposing all the local cameras on your network as RTSP streams) working as a prototype, but it's not quite ready for a public release.   If you're interested in testing it out and providing feedback, I'm happy to share -- alternatively I'm hoping to clean it up and publish it once I have the kinks worked out (mostly performance issues and frequent disconnects with larger numbers of cameras, right now...).

Happy to test ! I only have 3 wyze cams (2 x v3 and 1 x v2) + a yi home, a foscam c1 and a few ESP32 cams. I am also wondering if you could implement streaming easier based on the idea described here: https://github.com/n1ckyrush/wyzecam-hls.

hi @gszakacs, yes, it's definitely possible, and was actually the main reason I put this together. I have an RTSP bridge server (exposing all the local cameras on your network as RTSP streams) working as a prototype, but it's not quite ready for a public release. If you're interested in testing it out and providing feedback, I'm happy to share -- alternatively I'm hoping to clean it up and publish it once I have the kinks worked out (mostly performance issues and frequent disconnects with larger numbers of cameras, right now...).

Man you would be a hero to a lot of people if you pulled this off!

@kroo I would be happy to test.  I have 2 wyze cams (along with a few non wyze cams) and trying to import that video feed into Synology Surveillance Station.

I was able to spin up an nginx server with the RTMP module and then modified the streaming_video.py sample to pipe the frames to ffmpeg which then sends the stream to my nginx server. It's working well and only has about 4 secs lag when playing back on my iPhone with VLC.
The ffmpeg command looks something like this:
ffmpeg -f h264 -i - -vcodec copy -f flv rtmp://<yourserverip>:1935/live/doorbell
The wife is hovering because I spent the past hour throwing this together. When it's safe I will post more detail and a code sample.

hi @gszakacs, yes, it's definitely possible, and was actually the main reason I put this together. I have an RTSP bridge server (exposing all the local cameras on your network as RTSP streams) working as a prototype, but it's not quite ready for a public release. If you're interested in testing it out and providing feedback, I'm happy to share -- alternatively I'm hoping to clean it up and publish it once I have the kinks worked out (mostly performance issues and frequent disconnects with larger numbers of cameras, right now...).

Would you mind sharing? I have 3 v3 cams

hi @gszakacs, yes, it's definitely possible, and was actually the main reason I put this together. I have an RTSP bridge server (exposing all the local cameras on your network as RTSP streams) working as a prototype, but it's not quite ready for a public release.

Yeah - I'd love to test this as well.  running 3 v2s, (and a usb webcam in blue iris to achieve limited audio) - would like to get my v3 and esp32 into BI as well.

For those looking to beta test, I just put up a very early version of what I've been working on here: https://github.com/kroo/wyze-rtsp-bridge

Thank you Elliot!

On Jul 12, 2021, at 5:21 PM, Elliot Kroo @.***> wrote:
For those looking to beta test, I just put up a very early version of what I've been working on here: https://github.com/kroo/wyze-rtsp-bridge https://github.com/kroo/wyze-rtsp-bridge
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub https://github.com/kroo/wyzecam/issues/25#issuecomment-878683256, or unsubscribe https://github.com/notifications/unsubscribe-auth/AQ2NVROZDWFWI252TTDX2NDTXOBI3ANCNFSM447XXGTA.


Adds support for Showdown Extensions

This PR adds support for loading Showdown Extensions to the add-on, complete with tests and docs.
The way I've implemented this means that if you change the extensions property post-init then that won't be reflected. We could potentially refactor to make converter a computed property based on that property, but I was unsure about the potential performance implications of that...?
(Also, related to the above, should we be using the attrs there now rather than init? I'm nervous about that leading to a new Converter instance every time the markdown property changes!)

@gcollazo anything you need me to do on this one? Would like to give back :smile:

Sorry for the delay @nikz. I read you addition to the docs but still can't figure out how to use the extensions. Could you post a simple but complete demo on how to use it.

No worries! Here's a gist showing something similar to what I'm doing to support Emoji for instance:

published 2.5.0 to npm

thanks @nikz

No worries, thanks for including it!


ValueError

ValueError: Dimension 1 in both shapes must be equal, but are 100 and 10000. Shapes are [?,100] and [?,10000]. for 'bidirectional_20/while/Select' (op: 'Select') with input shapes: [?,10000], [?,100], [?,100].

Which notebook are you running?
If HATT, there is a known issue regarding mask handling by the custom Attention layer.
Try disabling the mask for the custom layer.
https://github.com/richliao/textClassifier/blob/master/textClassifierHATT.py#L153
Change to return None.


Adds necessary labels and file per component for kube-addons-manager

Have separate files for kube-addons-manager based deployments
compare https://github.com/Azure/acs-engine/pull/151
@lxpollitt @gunjan5

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.

@simonswine thanks!  I'm going to be running around a bit today, but will take a look at this later today or tomorrow :)

@simonswine I think we should make the add-on manager files explicitly marked as such somehow - either by putting them in a subdirctory called "addon-manager" or by changing the name - e.g "calico-addon-manager.yaml"
Then we'll need a section in the docs - probably here - that calls out how to install the manifests.

A seperate folder makes perfectly sense. I think the addons-manager (at least earlier versions) require a name like this: <name>-<kind>.yaml. This would be only possible with a subfolder, while getting some addons manager into the name.
For another project I made use of some ruby script to generate the YAMLs. It was basically the only way of keeping slightly different YAMLs in sync.
@caseydavenport: I totally agree on the docs. How do you think about some ruby foo to generate the YAMLs? I then would take a look at docs and a generation for YAML

@simonswine Let's start by creatingaddon-manager/calico-daemonset.yaml or similar and referencing it in the docs.  I think the ruby bits could be useful and cool, but might be more than we need at the moment.
If it's really just the labels that are different, then I think we can live with that.

What do we need to do to get this ready to merge?  @simonswine @caseydavenport

Not much.  I think this is it:

[ ] A couple of minor comments on the PR re: annotations.
[ ] Create a new directory with the add-on manager specific manifest.
[ ] A quick note in the docs which direct users at the manifests.

@caseydavenport I think I have done now everything needed for a merge

@simonswine thanks! LGTM.


Make std an optional dependency

Looking at the current API, I believe most of this crate could be made no-std compatible. The only parts used currently which require std are the std::io bits, so the question would be what API should/could be presented in no-std context.
It looks like moving the Read and Write traits into core is an eventual goal, currently blocked on the fact that std::io::Error involves boxing (and thus need alloc). The function on the Read and Write traits specify std::io::Error as the return type, so they can't move unless/until std::io::Error changes.
I'm doing some investigating on how people have dealt with this IO limitation in other contexts.

Note that this implies no-std-ness for all of our dependencies by default as well, with the std feature activating std-ness for those dependencies. I haven't yet evaluated whether our dependencies support this.

Looking at our deps, we have four first-level dependencies:

hex: Has an optional std feature. Is no_std if that feature isn't turned on. It's on by default.
sha1: Is itself unconditionally no_std. Has a std feature which turns on std for one of its own dependencies. Without that feature, entirely no_std-compatible.
sha2: Same as sha1.
url: Not at all no_std-compatible. If we're no_std, we'd need to turn this dependency off and deactivate our APIs that rely on it. There may be a no_std-compatible alternative; I haven't looked.

So that's three deps that are no_std-compatible, and one that isn't. The one that isn't could be turned off without much issue.

We should also look at our own use of std today, findable with this code search.

Item
no_std alternative available?
Can transition to core immediately?

std::any::Any
core::any::Any
YES

std::cell::RefCell
core::cell::RefCell
YES

std::collections::HashSet
Not in core or alloc
-

std::error::Error
core::error::Error
YES

std::ffi::c_char
core::ffi::c_char
YES

std::ffi::c_int
core::ffi::c_int
YES

std::ffi::CStr
core::ffi::CStr
YES

std::ffi::CString
alloc::ffi::CString (not in core)
-

std::ffi:NulError
alloc::ffi::NulError (not in core)
-

std::fmt::Display
core::fmt::Display
YES

std::fmt::Formatter
core::fmt::Formatter
YES

std::fmt::Result
core::fmt::Result
YES

std::fs::File
Not in core or alloc
-

std::hash::Hash
core::hash::Hash
YES

std::io::BufReader
Not in core or alloc
-

std::io::Error
Not in core or alloc
-

std::io::Read
Not in core or alloc
-

std::io::Seek
Not in core or alloc
-

std::io::SeekFrom
Not in core or alloc
-

std::io::Write
Not in core or alloc
-

std::ops::Deref
core::ops::Deref
YES

std::ops::Not
core::ops::Not
YES

std::os::unix::prelude::FromRawFd
Not in core or alloc
-

std::os::unix::prelude::RawFd
Not in core or alloc
-

std::os::windows::io::FromRawHandle
Not in core or alloc
-

std::os::windows::prelude::RawHandle
Not in core or alloc
-

std::panic::catch_unwind
Not in core or alloc
-

std::panic::UnwindSafe
core::panic::UnwindSafe
YES

std::path::Path
Not in core or alloc
-

std::path::PathBuf
Not in core or alloc
-

std::ptr::null
core::ptr::null
YES

std::ptr::null_mut
core::ptr::null_mut
YES

std::result::Result
core::result::Result
YES

std::slice::from_raw_parts
core::slice::from_raw_parts
YES

std::slice::from_raw_parts_mut
core::slice::from_raw_parts_mut
YES

std::str::FromStr
core::str::FromStr
YES

std::str::Utf8Error
core::str::Utf8Error
YES

Based on this, we have a large number of std uses today which can be immediately switched to using the core equivalents (the std versions are just re-exports anyway, so this will not break anything).
For the remainder, some have equivalents in alloc, and some have no equivalent. The options there will be to:

Conditionally remove APIs if no_std (or, more correctly, optionally include the APIs if the std feature is on, and have it on by default)
Find no_std-compatible replacements (for example, maybe HashSet has a no_std-compatible alternative available somewhere).

Finally done! In the end, after all the prior refactorings and improvements to the GitOid type, actually making the transition only took about an hour!
Next step will be making omnibor no_std-compatible.


Filecoin Chain Activity/Retention Metrics

Need to compute the following metrics:

Distribution of how many months are addresses active
How many of the addresses active 12 month ago are active now / How many of the addresses active now month ago were active 12 months ago?

Active here means appearing as the from address in a message.
E.g: In bafy2bzacedophgfiz3brrvfckmwb3wlpmrobuwvu3a3wddzq5jb3bs7o43cs4,  the active user is f3qvaypwsz3akwug3ltdu5474eb2rroyssecszu5o4rd3fiqrz63jpn67wornvh6ybynecgkrmecnfls6idcvq / f01882398.

We should also look at all current live deals.

How many have been renewed 1,2,3...x times?
What is the distribution of current deal length
Are we seeing the network actually being used for long-term indefinite storage?

A few more questions looking at deal making clients:

[ ] Of today/this month/this week active (made a deal) clients, how many of them were active on previous months?
[ ] Frequency distribution of number of days/weeks/months an active client_id has been active.

Exploring these questions in this hex.tech app.


Fast sliding is not smooth on iOS

When I slide the content view fastly, the ScrollableTabView's tabLabels scroll not fluently.
The problem happens on all of iOS simulators and iOS devices.

Yep, left or right swipe the contentView, some low levels devices such as iPhone 4s is obvious.
I have tried dev=true or dev=false.

What about profiler?

I've tested on iphone 4s and it is working good.

Sorry to say, I came here to check about android perspective.
In my android, without dev and debugger, the first slide is stuttering (very noticeable), but after the first one, it's awesome :).

I have the same problem,when  swipe  left or right the contentView,the ScrollableTabView's tabLabels scroll not fluently.

I'm encountering similar problems. This seems to only happen on iOS. On Android the swipes are smooth.

Same foe me, when the list is long there is delayed on change button, but the swipe effect is smooth.

Same for me

Ok, so I solved this problem by removing some animation in one of my tabs. It seems like if there's a decent amount of animation in one of the tabs, it lags

Nothing I can do with react-native animations. I hope one day it will become better.

Maybe it's the performance issue on React Native animations.

Anyone has a bypass?


Current state of the dashboards

Can anyone tell me if there is a way to grab the dashboards that you should be able to get from the Dashboards tab of the zabbix plugin?  Currently, I get the error Failed to get plugin dashboards when i click that tab.  I see suggestions that you can copy the dashboards folder into the datasources folder but I am unable to find the dashboards folder anywhere within the container.  When I grab the random json for either of the 3 dashboards that should be available from the Dashboards tab I get mixed results with some panels displaying the no data error.  Any suggestions or assistance would be greatly appreciated.
found a solutions right after posting this:
from the grafana command line
grafana-cli plugins remove alexanderzobnin-zabbix-app
restart
grafana-cli plugins install alexanderzobnin-zabbix-app 4.2.10
Now I am getting no data for the dashboard panels.
any suggestions?

This was fixed in https://github.com/grafana/grafana-zabbix/pull/1727 but not released yet.

This was fixed in #1727 but not released yet.

Is there anyway i can help test it out?


[FIX] Site Map Page Fixed

Site Map Page Fixed

Fixes #898

Nav bar is fixed.

Nav elements made smaller.

Body background color fixed.

Now, texts are visible.

Links are added to the hover effect to highlight.

Screenshot :

Can you please add at least level 1 to these PRs? As these are the genuine issues that I raised and solved. @MastanSayyad


Fixes regional users from not being able to see global content

What's this PR do?
Fixes global redirects for when the local translation doesn't exist
Fixes the links on the homepage to reflect the correct URL for the user.
Adds a session variable dosomething_global_redirected to keep track of when a user was redirected already (prevents redirect loop)
How should this be manually tested?
Do the cases outlined above ^ now work?
Any background context you want to provide?
Mostly explained in the Whats this PR do
What are the relevant tickets?
Fixes #5620

The code looks good.
Please reflect changes made to the dosomething_global_redirect_node() function in it's docblock.
It would be perfect to also provide in comments real-world example it works like it works. You can use @mikefantini's description:

For on-site discovery purposes, we’ll be exposing all country specific content and GL EN content. Preference would be given to country specific content and obviously if a campaign exists in both MX SP and GL EN, we would show users the MX SP translation and not the GL EN translation.

When going directly to a regional URL, a user should be able to access any country specific translation regardless of where they are located (i.e. if I’m in BR, I should be able to see US EN translation by going directly to www.dosomething.org/us/campaigns/campaign-title).
When going to the base URL of a campaign, we should redirect the user to the published translation that is most appropriate for them.

When we do this upcoming documentation sprint I planned on putting longer forms of documentation like that in the module readme, I can quickly change the docblock though.

:+1:


fix: update and re-enable test "test can read previous version's cache"

Commit 9671d8f35986 disabled the caching.bats test "test can read previous version's cache".
This re-enables that test after updating it to use a specific value (v1.0.0-rc4) as the "previous version".
The change to use a specific version and use its released binary improves the test.  That improves the test for the following reasons:

This should be faster - we don't have to build stacker twice.
Ultimately testing against the last released version is probably what you want... that is the cache that your users have, not the last development commit.
Building stacker from another version is both slow and hard to guarantee. This removes the assumption that the current build environment can build another version of stacker.
There was assumed knowledge about both the current and other stacker build systems in the 'make LXC_BRANCH=$(grep ...)' command.
The prior version of the test used "current/main" as "previous", but current/main is a moving target.  Re-running a checkout at a future point in time may fail as current/main has moved forward. (Think about using 'git bisect' to diagnose a past regression)
Even when working as intended, a series of PRs could still break re-use of the cache.  Given PRs A, B, C that all are merged to master.  B is tested against A, C is tested against B, but nothing tested C against A.

The change here does mean the following things are true, which are possibly undesirable:

There is a requirement to update the old version string. v1.0.0-rc4 is an arbitrary value.
The old binary must run in the current build environment.  This may be hard to guarantee.  I'm honestly not sure if static binary will be more reliable than building and executing a specific old version of source.

What type of PR is this?

Which issue does this PR fix:
What does this PR do / Why do we need it:
If an issue # is not available please add repro steps and logs showing the issue:
Testing done on this change:

Automation added to e2e:

Will this break upgrades or downgrades?
Does this PR introduce any user-facing change?:

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

https://github.com/project-stacker/stacker/pull/461 disabled the test because it would fail.


Errors while install from the source

🐛 Describe the bug
Thanks for the amazing work. Due to our cluster situation, I need to install Pytorch from the source. I install Pytorch and torchvision, but I am struggling in installing torchaudio.
I am installing torchaudio 0.12.1 version with cuda 11.3 version. Then, I suffer the erros. It seems these errors are causing this,
FAILED: torchaudio/csrc/libtorchaudio.so
/usr/bin/ld: cannot find -lmkl_intel_ilp64
/usr/bin/ld: cannot find -lmkl_core
/usr/bin/ld: cannot find -lmkl_intel_thread
I already tried to disable other options, BUILD_SOX=0, BUILD_CTC_DECODER=0, and BUILD_RNNT=0. But it does not work. Could you guide me how to install torchaudio from the source. I would really appreciate it!
Versions
Torch (installed from sources) 1.12.1 (1.12.0a)
CUDA 11.3
Python 3.8
Torchaudio 0.12.1

This was somewhat patched in PyTorch core recently https://github.com/pytorch/audio/issues/2784
What we have encountered in the past is that MKL 2022 version causes this issue, while MKL 2021 version works.
Though it is thought to be an only coincidence that MKL 2021 works.
Please try either use other MKL version, use latest PyTorch or cherry-pick the fix.
If that does not work, there isn't much we can do about it.


NIF-GrossAnatomy deprecation

This is a PR so that I can get a few addition eyeballs on this deprecation. The most relevant are probably the replacedBy: and the oboInOwl:hasDbXref edges that point to live uberon identifiers and deprecated uberon identifiers respectively.
Two other commits of interested remove duplicate fragments for birnlex_9 and birnlex_878. The remaining iris are NIFGA:birnlex_9 (now deprecated) and NIFSUB:birnlex_878 which was deprecated long ago.
Here is the report of the status all the classes (can be obtained by running nifga_deprecation.py).
Total count 1643
Match count 1341
No Match count 302
No Match +udep count 259
No Replace count 43
No Match -udep count 43
No Match -udep +region count 27
No Match -udep -region count 16
No Match not deprecated count 0
Mismatch between No match and No replace set()

Match means that there was a match in uberon for the existing nifga class.
+/-udep is whether the target class in uberon was deprecated or not.
+/-region is whether the source identifier was one of the Regional part of... classes.
The classes of most interest are the 16 that were not regions and had no uberon replacement.
10 of those fall under Afferent Role and are incomplete. NIFGA:birnlex_9 Bio physical immaterial entity is unused. That leaves us with 5 identifiers, all of which were already deprecated.
Further notes and reporting can be found in nifga_deprecation.py.
@cmungall if you spot anything that seems off here please let me know.

Merging. Any further comments welcome.


performance consideration for the quads sparse array used in the quadtreebh

http://jsperf.com/quadtest

Thank you very much for this!
I've updated the quadtree module, and was impressed to see 1.6x performance boost.
Please feel free to share more ideas or contribute to https://github.com/anvaka/ngraph.quadtreebh if you see more opportunities for performance improvements

Hi,
i will work on that subject. i translated your whole library in typescript 2 weeks ago.
no sure i translated every thing but i have nice results
i tried to made several changes ...
tried to change the way Math.sqrt is used (replacing with an approximation) but had some precision issues ( i will organize a jsperf on the subject as it was clearly very funny )
if you found the revision 2 of the quadtree test. i tried to replace the usage of mass , massX and some other vars in the body with a float32array and some get/set property (to have no impact on the rest of the code...)
next step for me is to organize the step computation in a worker (full computation or partial ) my target is to have the browser main tread with full reactivity.

i will share all the changes if they improve your library.
regards (from France),
Laurent

Glad to hear! I'm going to add webworkers too, since they did improve performance significantly.
You can also run performance tests in the quatree module itself:
git clone https://github.com/anvaka/ngraph.quadtreebh
npm install
npm run perf

This will execute local benchmarks.

Hi i made a small change in updateBodyForce . target idea was not to make less sqrt compute
if we recurse on the children Math.sqrt is not called
updateBodyForce(sourceBody) {
  var queue = this.updateQueue,
    v = 0,
    dx = 0,
    dy = 0,
    r = 0,
    queueLength = 1,
    shiftIdx = 0,
    pushIdx = 1,
    body = null,
    node: Node = null,
    cmpare = 0,
    sumdxdy = 0;

  queue[0] = this.root;

  while (queueLength) {
    node = queue[shiftIdx],
    body = node.body;

    queueLength -= 1;
    shiftIdx += 1;
    // technically there should be external "if (body !== sourceBody) {"
    // but in practice it gives slightghly worse performance, and does not
    // have impact on layout correctness
    if (body && body !== sourceBody) {
      // If the current node is a leaf node (and it is not source body),
      // calculate the force exerted by the current node on body, and add this
      // amount to body's net force.
      dx = body.pos.x - sourceBody.pos.x;
      dy = body.pos.y - sourceBody.pos.y;
      sumdxdy = dx * dx + dy * dy;

      if (sumdxdy === 0) {
        // Poor man's protection against zero distance.
        dx = (this.random.nextDouble() - 0.5) / 50;
        dy = (this.random.nextDouble() - 0.5) / 50;
        sumdxdy = dx * dx + dy * dy;
      }
      r = Math.sqrt(sumdxdy);

      // This is standard gravition force calculation but we divide
      // by r^3 to save two operations when normalizing force vector.
      v = this.gravity * body.mass * sourceBody.mass / (r * r * r);
      sourceBody.force.x += v * dx;
      sourceBody.force.y += v * dy;
    } else {
      // Otherwise, calculate the ratio s / r,  where s is the width of the region
      // represented by the internal node, and r is the distance between the body
      // and the node's center-of-mass
      dx = node.massX / node.mass - sourceBody.pos.x;
      dy = node.massY / node.mass - sourceBody.pos.y;
      sumdxdy = dx * dx + dy * dy;

      if (sumdxdy === 0) {
        // Sorry about code duplucation. I don't want to create many functions
        // right away. Just want to see performance first.
        dx = (this.random.nextDouble() - 0.5) / 50;
        dy = (this.random.nextDouble() - 0.5) / 50;
        sumdxdy = dx * dx + dy * dy;
      }
      cmpare = ((node.right - node.left) / this.theta) * ((node.right - node.left) / this.theta);
      // If s / r < θ, treat this internal node as a single body, and calculate the
      // force it exerts on body b, and add this amount to b's net force.
      if (cmpare < sumdxdy) {
        r = Math.sqrt(sumdxdy);

        // in the if statement above we consider node's width only
        // because the region was squarified during tree creation.
        // Thus there is no difference between using width or height.
        v = this.gravity * node.mass * sourceBody.mass / (r * r * r);
        sourceBody.force.x += v * dx;
        sourceBody.force.y += v * dy;
      } else {
        // Otherwise, run the procedure recursively on each of the current node's children.

        // I intentionally unfolded this loop, to save several CPU cycles.
        if (node.quad0) { queue[pushIdx] = node.quad0; queueLength += 1; pushIdx += 1; }
        if (node.quad1) { queue[pushIdx] = node.quad1; queueLength += 1; pushIdx += 1; }
        if (node.quad2) { queue[pushIdx] = node.quad2; queueLength += 1; pushIdx += 1; }
        if (node.quad3) { queue[pushIdx] = node.quad3; queueLength += 1; pushIdx += 1; }
      }
    }
  }
}

with previous version

with modified code

tested with chrome, ie , firefox


Compliance Report | Summary section penalty payable updating when adjusting value in Line 26 |2|

Fix issue with penalty payable section not updating correctly when changing the credit offset value in Line 26

 Compliance Report | Summary section penalty payable updating when adjusting value in Line 26 |2|

Resolved in sprint red dwarf