why panic in src/database/sql/sql.go:2209?

Go version
go version 1.20.0
Output of go env in your module/workspace:
NA

What did you do?
use github.com/dtm-labs/client
What did you see happen?
panic stack：
........
/usr/local/go/src/runtime/panic.go:884 +0x213 database/sql.(*Tx).grabConn(0x0, {0x15c40c8, 0xc00004e030})
/usr/local/go/src/database/sql/sql.go:2209 +0x52 database/sql.(*Tx).ExecContext(0x0, {0x15c40c8, 0xc00004e030}, {0xc0004c0230, 0x6e}, {0xc000ecd7a0, 0x6, 0x6})
/usr/local/go/src/database/sql/sql.go:2467 +0x51\ndatabase/sql.(*Tx).Exec(0x11d1c40?, {0xc0004c0230?, 0x13862c0?}, {0xc000ecd7a0?, 0x10?, 0x116e9a0?})
/usr/local/go/src/database/sql/sql.go:2480 +0x45 github.com/dtm-labs/client/dtmcli/dtmimp.DBExec({0x13862c0, 0x5}, {0x15bcc58, 0x0}, {0xc0004c0230, 0x6e}, {0xc000ecd7a0, 0x6, 0x6})
/go/pkg/mod/github.com/dtm-labs/client@v1.17.3/dtmcli/dtmimp/utils.go:211 +0x157\ngithub.com/dtm-labs/client/dtmcli/dtmimp.InsertBarrier({0x15bcc58, 0x0}, {0xc000e55128, 0x4}, {0xc00005a378, 0x16}, {0xc000dcf5de, 0x2}, {0xc000e55146, 0x6}, ...)
/go/pkg/mod/github.com/dtm-labs/client@v1.17.3/dtmcli/dtmimp/utils.go:292 +0x377\ngithub.com/dtm-labs/client/dtmcli.(*BranchBarrier).Call(0xc0001f5b20, 0x0, 0xc0016ab050)
/go/pkg/mod/github.com/dtm-labs/client@v1.17.3/dtmcli/barrier.go:77 +0x468\noasis/app/ams/amsbalanceinfrastructure/internal/data.(*Data).BarrierInTx(0xc00071adc0, {0x15c4138, 0xc000349bc0}, {0xc000053e86, 0x6}, 0xc0016ca340)
.......
can somebody help me?
What did you expect to see?
NA

Hi,
the Go project does not use its bug tracker for general discussion or asking questions about the language. The Github bug tracker is only used for tracking bugs and proposals going through the Proposal Process.
Please see the Questions wiki page; it has a list of good places for asking questions. Thanks!
Closing here. If it turns out to be a bug in the go standard library, feel free to comment and we'll reopen. Thanks.

panic in go standard library, bug or question? i think it is bug...please reopen it

Hi,
the Go project does not use its bug tracker for general discussion or asking questions about the language. The Github bug tracker is only used for tracking bugs and proposals going through the Proposal Process.
Please see the Questions wiki page; it has a list of good places for asking questions.
Closing here. If it turns out to be a bug in the go standard library, feel free to comment and we'll reopen. Thanks.

A panic can be caused by internal structures corruption caused by a race condition or use of the unsafe package. Also a few standard library functions panic on bad inputs.

try to run your code after building with the -race flag. Does it print any warning?
try to run your program after setting GODEBUG=cgocheck=2. Does it print any error?
Update to the latest go version (1.20 is not) and retry.

In any case a panic like this cannot be investigated if you don't post a reproducer.

Also see #43445 (which is almost a copy past of this report, down to the number of "......." used to shorten the stack trace).


4404 allow admin to reset a users api keys and all users api keys

This closes #4404 .

Frontend tests were OK :+1: (details)

Frontend tests were OK :+1: (details)

Frontend tests were OK :+1: (details)

@xavijam, @rafatower  CR this, please .

Frontend tests were OK :+1: (details)

:+1:

Please @juanignaciosl, could you show us in a preview the UI changes? or give us a staging server to try this thing.

@xavijam changed dependencies, take a look, please.
Organization-wide regenerate API Keys button:

Organization user regenerate API Key button:

Changed message:

Frontend tests were OK :+1: (details)

I feel we need to change the description of the dialog, what about...:
"You and all your users will need to update all deployed apps with the new API key. Are you sure you want to continue?"

Also, checking this change, shouldn't we send an email saying "your api key has been changed"? So, shouldn't we send an email each time you regenerate your api key?

Great job man, code looks great.

Hey @josecruz can you take a look at this?

Yep, please, coordinate with @juanignaciosl.

Ok, I've already improved the messages.
@xavijam: maybe now we should send a message to notify the user if the administrator changes the api key (I will do it), but I think there's no need if you change your api key.
@josecruz : I need an improvement for this. Take a look at the previous screenshots, you see the new part is the "api key management" and the button. Just keep in mind that, from a HTML point of view, it must be in a different form ;-)

Frontend tests were OK :+1: (details)

Great work!
Just found this:

[ ] API should be uppercase.

[ ] Change icon:

@saleiva you agree with the icon change?
BTW, Agree with @xavijam of send an email with the API change message

But not only that @josecruz, I'm more worried about the changes in the settings form.

Okay, in the form:

[ ] First form-row divs have an extra padding-top:10px; so the separation between the first row and the form title is 50px instead of 50px (it happens on Organization log in/sign up too)

Related to the API key button, @saleiva gave me the solution (ty):

[ ] We're going to use an input with a Renew API key link by its side

1.- ok
2.- ok, although I'll have to check how to do it, since that field is inside a form and currently the button was in another. Also, a link that is actually an action is a little strange to me.
3.- I still miss the design for the button on the organization settings page (the one that triggers renewal for all users).

3.- I still miss the design for the button on the organization settings page (the one that triggers renewal for all users).
In that case we're going to use this format

Ok!

Frontend tests were OK :+1: (details)

Frontend tests were OK :+1: (details)

Frontend tests were OK :+1: (details)

Frontend tests were OK :+1: (details)

Ok, this should be done now. @xavijam , @rafatower, could you CR this, please? It's deployed in ded03 if you want to try. This closes #4404 and #4611.

Nice! :+1:

IMO, once an organization admin user regenerates the api key of one of the organization users, it shouldn't go to organization index, it should keep in the organization user page (minor).

-> You and all organization users will need to update all deployed apps with a new API key. Are you sure you want to continue?
-> You and all organization users will need to update all deployed apps with the new API key. Are you sure you want to continue?

I've tried to regenerate the API key of all organization users and the process finishes in a 404 page :S.
http://:org-name.domain.com/u/:username/organization/regenerate_api_keys

It looks great, I left some comments in the path.

Frontend tests were OK :+1: (details)

Frontend tests were OK :+1: (details)

Frontend tests were OK :+1: (details)


include labels and annotations when generating ResourceGroup

This change is to inject the labels and annotations from Kptfile into ResourceGroup

/assign @mortent @seans3

Looks good. Thanks for the tests. I think this needs to be rebased because of your previously merged PR.


[Rust] Minimal dataset append support

Bring back Append support in schema evolution

Curious if it makes sense to make a DatasetWriter yet?

Is this the same concept as Updater in C++ codebase, or something new?


README.md to emphasize transcriptome-centric features of RiboViz

README.md is kind of generic about ribosome profiling, rather than specific about RiboViz. We should emphasize that RiboViz:

is a transcriptome-centric pipeline, which makes it unusual
is a pipeline going all the way from raw .fastq files to visualization
now extends beyond yeast
contains other new features
benefits from key design choices

Relates to #80.


Misleading key encoding information - they are not in base64 representation, but are just strings

The page says this about keys:

SAS authentication in Service Bus is configured with named Shared Access Authorization Policies having associated access rights, and a pair of primary and secondary cryptographic keys. The keys are 256-bit values in Base64 representation.

The phrase The keys are 256-bit values in Base64 representation means that keys are binary and are base64-encoded, so they can be used as printable strings.
In reality, these keys are just strings and must not be decoded before they are used. This page must specify explicitly that base64 encoding is a distraction and the keys should be treated as plain strings, without trying to base64-decode them for the HMAC-SHA256 step.

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

ID: 13478d1d-11eb-930a-02c4-559a1ef89fa6
Version Independent ID: be36cdf9-1e5e-c47f-e699-652dc42ee2b9
Content: Azure Service Bus access control with Shared Access Signatures - Azure Service Bus
Content Source: articles/service-bus-messaging/service-bus-sas.md
Service: service-bus-messaging
GitHub Login: @spelluru
Microsoft Alias: spelluru

@gh-andre, Thanks for the feedback! We are taking a look into this and will get back to you soon.

@AjayKumar-MSFT Thanks for looking into this. I spent some time today trying to get my signatures match and while the REST API can take whatever Azure team says it does, using 3rd-party libraries to compute signatures, like OpenSSL, it's not very obvious that a base64-encoded key should just be used just as a plain string.

@AjayKumar-MSFT As a side note, you may also want to communicate to the dev team that requiring plain base64 strings makes them more vulnerable for scanning memory for keys because base64 patterns can be easily identified in memory dumps without knowing application structure.
That is, if a service was hacked, it would still require hackers to know which application location stores a random binary key because it has no pattern, but malicious code can scan application memory for a base64 character patterns and quickly dump all base64 strings that are N characters long without knowing anything about the application.
I don't expect this to be fixed due to a possible impact of such fix, but this sounds more like a backend bug rather than a docs bug. Maybe they will allow some qualifier in the API to allow clients submit data signed with binary keys, not base64-encoded strings.

@spelluru , Please look into this

@EldertGrootenboer - please review and respond when you get a chance. Thanks.
#reassign:EldertGrootenboer

Thank you for your feedback. We have updated the documentation to make this more clear. These changes should be visible soon.

#please-close


chore(release): v2.15.6

Problem

bugfixes need to be published
Solution

create new patch release
Additional Notes

Links
Ticket

GitHub issue:
Other links
Verification
Manual tests

Automated tests

[ ] Unit tests added/updated
[ ] E2E tests added/updated
[x] N/A - release only
[ ] deferred - (provide GitHub issue for tracking)

Housekeeping

[x] No non-essential console logs
[x] All new files contain license notice

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Codecov Report

Merging #1084 (edda323) into main (ae87118) will not change coverage.
The diff coverage is n/a.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.

@@           Coverage Diff           @@
##             main    #1084   +/-   ##
=======================================
  Coverage   93.91%   93.91%
=======================================
  Files         124      124
  Lines        5864     5864
  Branches     1786     1786
=======================================
  Hits         5507     5507
  Misses        339      339
  Partials       18       18

Continue to review full report in Codecov by Sentry.

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


Change Colors

Hello,
is it possible to change the colors?
Which is like a color display for me.
cu

It would be nice to change global:

background color
text color
highlighted text color
block separator lines

It would also be nice to change per block:

text color
highlighted text color


bpf_host: emit '-> network' traces for egress packets

See the issue below for the full context. The caveat is, this program is only attached to the native device egress if either the host firewall or BPF NodePort is enabled, so there still won't be any TRACE_TO_NETWORK visibility if that's not the case.
I've had to move the trace to fire before nodeport_nat_fwd() is called, since that function tail calls on the happy path, and there's no CT context available downstream from there. This could likely be improved when/if we adopt BTF and get rid of tail calls on kernels that support it.
I've also added handle_to_netdev_ipv4() to clean up to_netdev() a bit, to bring it on par with the ipv6 handling.
bpf_host: emit '-> network' traces for packets leaving the node when Host Firewall or BPF NodePort are enabled
Based on: #13485
Fixes: #12561
Signed-off-by: Timo Beckers<EMAIL_ADDRESS>Signed-off-by: Thiago Navarro<EMAIL_ADDRESS>

new traces would be another commit

Thank you for you comments. Sorry, but I did not understand. What do you mean by new traces? AFAIU, the issue is related to this comment, right? So, new trace would be the issue. Let me know if I got your point.

@navarrothiago Could you also fix the second commit's title? Right now, I doesn't provide any useful/actionable information.

test-me-please

@pchaigno, should we re-trigger the jobs?

test-runtime

Team reviews are covered and tests are passing. Marking as ready to merge.


Fix typos and sphinx warnings in docs

Earth2Studio Pull Request
Description

Fixes several typos and sphinx warnings in the documentation
Add a note clarifying how to find the spinx-build command to make docs. I'm not sure if I'm the only one that faced this issue?
Some additional typos and warnings remain which I'm not sure how to fix:

Many dosctrings refer to a prognostic model even though they don't seem to be talking about a prognostic model. Not sure if this is a copy-pasted typo or intentional. Example: https://github.com/NVIDIA/earth2studio/blob/main/earth2studio/statistics/acc.py#L72
The sphinx run with make docs is now almost completely clean. The only remaning warnings are from missing methods requested by automethod. These are available in datasource.rst, load_model, and load_default_package in prognostic.rst & diagnostic.rst

Checklist

[x] I am familiar with the Contributing Guidelines.
[ ] New or existing tests cover these changes.
[x] The documentation is up to date with these changes.
[ ] The CHANGELOG.md is up to date with these changes.
[ ] An issue is linked to this pull request.

Dependencies
None

"Many dosctrings refer to a prognostic model even though they don't seem to be talking about a prognostic model."
These are almost guaranteed to be copy-paste typos from refactoring!

/blossom-ci

"The sphinx run with make docs is now almost completely clean. The only remaining warnings are from missing methods requested by automethod"
Yeah this is artifact of the template that we use to not manually write the auto doc commands. Its nice but complains when a function isnt found. Never found a good way around it.

@NickGeneva I updated the stray occurences of "prognostic model". Most were in earth2studio/statistics, please take a look

Yeah this is artifact of the template that we use to not manually write the auto doc commands. Its nice but complains when a function isnt found. Never found a good way around it.

We should be able to use sphinx-jinja to guard the automethod call inside a jinja if block. Played around with it but couldn't get it to work, abandoning for now

/blossom-ci

/blossom-ci

/blossom-ci


userDefinedExcludes doesn't seem to work

Hello
Follow up this issue  https://github.com/Esri/arcgis-webpack-plugin/issues/78#issuecomment-581798687, the userDefinedExcludes doesn't seem to affect the bundle size.
Using branch : arcgis-webpack-angular
Thank you

@jaouenk This is expected behavior. As mentioned in the other issue, in the ArcGIS API for JavaScript 4.x there is shared code among all layers and what you are seeing is that you have essentially reached the minimum bundle size for the API. Another way to look at it is you are attempting to exclude bundles that have already been excluded by the webpack plugin because you they aren't being imported into your app.

I'm ok for layers but what about Widgets ?
new ArcGISPlugin({
      userDefinedExcludes: [
        "arcgis-js-api/widgets/BasemapGallery",
        "arcgis-js-api/widgets/BasemapToggle",
        "arcgis-js-api/widgets/Bookmarks",
        "arcgis-js-api/widgets/BrowseItems",
        "arcgis-js-api/widgets/ClassedColorSlider",
        "arcgis-js-api/widgets/ClassedSizeSlider",
        "arcgis-js-api/widgets/ColorPicker",
        "arcgis-js-api/widgets/ColorRampSelector",
        "arcgis-js-api/widgets/ColorSlider",
        "arcgis-js-api/widgets/Compass",
        "arcgis-js-api/widgets/CoordinateConversion",
        "arcgis-js-api/widgets/Directions",
        "arcgis-js-api/widgets/DirectLineMeasurement3D",
        "arcgis-js-api/widgets/DistanceMeasurement2D",
        "arcgis-js-api/widgets/Editor",
        "arcgis-js-api/widgets/Expand",
        "arcgis-js-api/widgets/Feature",
        "arcgis-js-api/widgets/FeatureForm",
        "arcgis-js-api/widgets/FeatureTemplates",
        "arcgis-js-api/widgets/Fullscreen",
        "arcgis-js-api/widgets/HeatmapSlider",
        "arcgis-js-api/widgets/Histogram",
        "arcgis-js-api/widgets/HistogramRangeSlider",
        "arcgis-js-api/widgets/Home",
        "arcgis-js-api/widgets/LayerList",
        "arcgis-js-api/widgets/Legend",
        "arcgis-js-api/widgets/Locate",
        "arcgis-js-api/widgets/NavigationToggle",
        "arcgis-js-api/widgets/OpacitySlider",
        "arcgis-js-api/widgets/Popup",
        "arcgis-js-api/widgets/Print",
        "arcgis-js-api/widgets/RasterSymbologyEditor",
        "arcgis-js-api/widgets/RendererSlider",
        "arcgis-js-api/widgets/ScaleBar",
        "arcgis-js-api/widgets/ScaleRangeSlider",
        "arcgis-js-api/widgets/Search",
        "arcgis-js-api/widgets/SizeSlider",
        "arcgis-js-api/widgets/Sketch",
        "arcgis-js-api/widgets/Slice",
        "arcgis-js-api/widgets/Slider",
        "arcgis-js-api/widgets/Spinner",
        "arcgis-js-api/widgets/SymbolStyler",
        "arcgis-js-api/widgets/Tags",
        "arcgis-js-api/widgets/TimeSlider",
        "arcgis-js-api/widgets/Track",
        "arcgis-js-api/widgets/UnivariateColorSizeSlider",
        "arcgis-js-api/widgets/Zoom",
      ],
    })

The bundle size doesn't change either.

That's correct. You are at minimum bundle size for the entire API.
Note, with each new release we are getting closer to ESM compliance and significantly improving dependency management. ESM compliance is our ultimate goal. I'm going to go ahead and close this issue, thanks for reaching out.


Add support for Orange Pi Prime

Adds support for using mraa on OrangePi Prime board.

Hi, Could you please rebase, you will need to change the enum for the board as we have merged other platforms before this one.

Hi, Could you please rebase, you will need to change the enum for the board as we have merged other platforms before this one.

Rebased on latest upstream master branch

@viraniac Thanks!


Android crash: UiAutomation not connected

Problem
Coming from this Crashlytics record. Occurs in Android 11, Pixel 5
No reproduction steps as of now, just one occurrence but logging this either way.
Why is it important
Prevents the app from crashing
Solution
Investigate where the crash is happening and fix it
Fatal Exception: java.lang.IllegalStateException: UiAutomation not connected, UiAutomation@2a8b373[id=-1, flags=0]
       at android.app.UiAutomation.throwIfNotConnectedLocked(UiAutomation.java:1239)
       at android.app.UiAutomation.getRootInActiveWindow(UiAutomation.java:602)
       at android.support.test.uiautomator.UiDevice.getWindowRoots(UiDevice.java:3)
       at android.support.test.uiautomator.UiDevice.findObjects(UiDevice.java:2)
       at androidx.test.tools.crawler.platform.uiautomator.UiAutomatorUtils.isSoftKeyboardOpen(UiAutomatorUtils.java:1)
       at androidx.test.tools.crawler.platform.hybrid.EspressoUtils.isSoftKeyboardOpen(EspressoUtils.java:7)
       at androidx.test.tools.crawler.platform.hybrid.HybridInteractionController.enterText(HybridInteractionController.java:14)
       at androidx.test.tools.crawler.platform.ActionExecutor.execute(ActionExecutor.java:48)
       at androidx.test.tools.crawler.platform.ActionExecutor.performAction(ActionExecutor.java:4)
       at androidx.test.tools.crawler.platform.RemotePlatform.handlePerformAction(RemotePlatform.java:22)
       at androidx.test.tools.crawler.platform.RemotePlatform.messageLoop(RemotePlatform.java:59)
       at androidx.test.tools.crawler.platform.RemotePlatform.lambda$startCrawlAndWaitUntilFinished$0$androidx-test-tools-crawler-platform-RemotePlatform(RemotePlatform.java:1)
       at androidx.test.tools.crawler.platform.RemotePlatform$$ExternalSyntheticLambda1.run(:6)
       at java.lang.Thread.run(Thread.java:923)

the full stacktrace is
here

Triggered auto assignment to @maddylewis (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details.

Bug0 Triage Checklist (Main S/O)

[ ] This "bug" occurs on a supported platform (ensure Platforms in OP are ✅)
[ ] This bug is not a duplicate report (check E/App issues and #expensify-bugs)

If it is, comment with a link to the original report, close the issue and add any novel details to the original issue instead

[ ] This bug is reproducible using the reproduction steps in the OP. S/O

If the reproduction steps are clear and you're unable to reproduce the bug, check with the reporter and QA first, then close the issue.
If the reproduction steps aren't clear and you determine the correct steps, please update the OP.

[ ] This issue is filled out as thoroughly and clearly as possible

Pay special attention to the title, results, platforms where the bug occurs, and if the bug happens on staging/production.

[ ] I have reviewed and subscribed to the linked Slack conversation to ensure Slack/Github stay in sync

This might not require any further action if the crash wont be happening again in future @maddylewis

got it - ty for the clarification @mountiny

ill check to see if the crash is still happening.

I checked the dashboard just now and I dont think this is happening now. Gonna close this. Thanks @maddylewis !


test test ck perf

Proposed changes
Issue Number: close #xxx
Problem summary
Describe your changes.
Checklist(Required)

[ ] Does it affect the original behavior
[ ] Has unit tests been added
[ ] Has document been added or modified
[ ] Does it need to update dependencies
[ ] Is this PR support rollback (If NO, please explain WHY)

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

TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 36.94 seconds
stream load tsv:          435 seconds loaded<PHONE_NUMBER>9 Bytes, about 164 MB/s
stream load json:         24 seconds loaded<PHONE_NUMBER> Bytes, about 93 MB/s
stream load orc:          61 seconds loaded<PHONE_NUMBER> Bytes, about 17 MB/s
stream load parquet:          30 seconds loaded 861443392 Bytes, about 27 MB/s
insert into select:          78.9 seconds inserted 10000000 Rows, about 126K ops/s
https://doris-community-test-1308700295.cos.ap-hongkong.myqcloud.com/tmp/20230520004509_clickbench_pr_147625.html

run clickbench

run buildall

run buildall

run clickbench

run clickbench

run clickbench

run clickbench

run clickbench

run clickbench

run clickbench

run clickbench

After test, current master branch's performance is very well, so that I will close this PR.


String manipulation instead of DOM manipulation

Currently, Transparency does direct DOM manipulations for all data elements. In worst case, this is order of magnitude slower than string manipulation and a single DOM update.
Refactor Transparency to use string manipulation internally. No changes to external API.

Are you talking about DOM vs. innerHTML performance?

http://jsperf.com/dom-vs-innerhtml/10

Yes.
However, based on other benchmarks, I thought string manipulation + innerHTML is faster than DOM manipulation.
I did some trials with DOM fragments, too, but they might be worth further investigation. At least it would be a lot easier to implement.

This also might be great interest for you:
http://hacks.mozilla.org/2011/11/insertadjacenthtml-enables-faster-html-snippet-injection/

Thanks! I'll take a look.

It turned out that with proper caching, DOM manipulation is faster than any other option. Luckily, it was also the easiest one to implement. Closing, as Transparency is probably a lot faster than anything else out there.


Serve the editor using container

Added a Dockerfile to allow the editor to be served using docker containers
Updated README with docker instructions

Very nice. Thanks @chrissng for solving this.
In the future I hope people can use this binary here https://github.com/maputnik/desktop together with the bundled assets (so it is easy for Windows folks as well).

@lukasmartinelli thanks for approving and merging my PR! I missed the kickstarter campaign so I thought I could help by making a small contribution in some useful way.


Next button in payment form does not allow user to proceed.

Describe the bug
Using drop-in components..
      onPress={() => {
        try {
          start('scheme')
        } catch (error) {
          console.log("Error", {
            message: "There was an Error.",
          })
        }
      }}

When filling the payment form, I have noticed next button (when entering address fields) does not allow the user to progress - after entering the country from the select field input.
Note the first tap of Next from entering card holders name does allow you to progress to the country field. Any further taps of Next further down the form the behaviour does not work as intended.
To Reproduce
Steps to reproduce the behavior:

Fire a payment method by card or scheme to show the drop in components with

card: {
    holderNameRequired: true,
    addressVisibility: "full",
  },

set inside the config.
2. Enter card details
3. Enter card holder name
4. Hit Next on the native keyboard and proceed to country select
5. Touch the first address field text input to enter some text
6. Hit Next and you will see you do not proceed
Expected behaviour
You expect the form to focus on the next text input field.
Smartphone (please complete the following information):

Device: iPhone 11 pro
OS: 15
Version "@adyen/react-native": "1.2.0",

Additional context
This does seem to prevail after further testing on differing device/ios versions (it is not just ios 15 as previously mentioned - edit).  I have seen it work on my device, and then intermittently not work - iphone 13 pro, ios 16.3.1.

Hey @Seamus1989
Thanks for feedback.
Indeed there was an issue on v4 with focus - it is captured by country picker.
On iOS v5 address component was revisited.
Stay tuned for v2.0.0 release


[WIP] Add Pyramid chart for plotly

Added Pyramid-bar chart type for plotly

@woozyking I think I accidentally pushed the changes into master branch before branching off into a new branch for this PR how can I fix it?

@woozyking I think I accidentally pushed the changes into master branch before branching off into a new branch for this PR how can I fix it?

git reset to its previous state in master

@woozyking I think I accidentally pushed the changes into master branch before branching off into a new branch for this PR how can I fix it?

git reset to its previous state in master

I can't git push --force after resetting tho cuz it is a protected branch

I can't git push --force after resetting tho cuz it is a protected branch

Can you lemme know which commit to reset on so I can try it?

I can't git push --force after resetting tho cuz it is a protected branch

Can you lemme know which commit to reset on so I can try it? should it be resetting to d30b51d?

yes, please these last 2 commits.


better use enum and data if db supports it

you could also use enum for status, but I'm not sure what the status field should contain👀 if there's sth like "actively looking for job"/"not looking for job", then go for it

Thank you :)
Status field should contain categorical values like "looking for job", "NOT looking for a job", "considers proposals" ... etc.
I'll create this class today, thank you vm!


Confused about the "max-sentences" in pretraining

Hi,
In the pretraining script, you set the max-sentences to 32. Max-sentences is per GPU, so PER_GPU_TRAIN_BATCH_SIZE is 32. But the "max-tokens" is 2048 and the "tokens-per-sample" is 512, , so PER_GPU_TRAIN_BATCH_SIZE is 4. Why are these two parameters conflicted?
Thanks

Yes.


Epic fight - First person camera inside player head

Only with Epic Fight installed, the first person mode camera is inside the player head obstructing view.

Sorry ignore this, this actually occurs with the first-person model mod and epic fight.


Comment out some migrations so that first time setup runs cleanly

Setting up Transition on a new laptop, I replicated data and then had
to run rake db:migrate. It failed on the "enable pgcrypto" migration
because it requires database superuser access to enable extensions. We
don't need this extension anymore (it was removed in November 2014),
so comment out migrations that reference it so that db:migrate can
run cleanly. Commenting out rather than deleting because I don't know
how Rails behaves when you delete migrations, and so that we preserve
history of these changes.

👍


build packages for arm linux (for use on raspberry pi)

[ ] try buliding inside an arm docker image, with qemu registered as the binfmt executor or whatever you call it https://github.community/t/testing-against-multiple-architectures/17111

It might be possible to use the infrastructure from the cross project, to build for multiple architectures.
My home automation server is a raspberry pi, so I have a vested interested in getting this working.

You could build them on CircleCI, which supports arm64 linux.

Thanks for the pointer. It looks like the standard tier is 6000 minutes/month (4 days) or something, which would be enough to get started with prototyping. The open source tier is about 10x this, and could be applied for once we had a poc, to give us round the clock builds for a single architecture.
https://circleci.com/open-source/
I've not really used circleci before. It feels reasonable to hand over a copy of the GitHub Releases deploy key to circleci. We would need to find a way to make sure that the builder (which is building potentially untrusted code) does not have access to this secret (it just crates a tarball in a known location, that is then uploaded by a separately sandboxed process).

Instead of creating a commit to trigger a workflow, both CircleCI and GitHub Actions support making an api call and passing parameters so this could be used to trigger it. This would be simpler than having to generate different yaml and a commit each time something is built.

The commit-per-build was a decision that I made quite early on. Initially it seemed a bit questionable, but let me move quickly so I stuck with it. As time went on, I started hanging a bunch more things on top of it.
a) GitHub Releases requires a tag to attach release assets to, and having a commit per build gives me a convenient thing to tag. In theory, I could hang all of the tags off of a single commit though. This might be fine. I've not tried.  I'd rather not get into the habit of deleting tags in this repo.
b) I use "a commit without a tag" as a sign of a broken build (when calculating the list of broken packages that I should stop trying to build for a bit). This is easy to do in bash, without making calls to the GitHub Actions API.
I really need to draw a diagram of all of this nonsense. In the short term, I think we could get away with making a trigger-circleci/* branch namespace or something, without too much pain.
At some point, I should probably rewrite my cronjob logic in rust or something. That might make me more likely to reach for the GitHub Actions API when it makes sense. It would probably make the whole thing more maintainable as well.
Is this a thing that you would be interested in helping with?

FYI, Cirrus CI also has support for arm containers (using AWS's Graviton2 instances):
https://cirrus-ci.org/guide/linux/
They don't have any minute limits, only concurrency limits:
https://cirrus-ci.org/faq/#are-there-any-limits

@jack1142 Thanks for the hint. I would definitely accept patches that add support for free-tier non-github runners.
If anyone decides to have a stab at this but doesn't manage to get anywhere, please reply to this issue, saying what you got stuck on first. I recognise that my mess of bash scripts is really difficult to contribute to at the moment, and would like to do better. I have ideas of what I can do to make it better, but it would also be good to have some external feedback.
If anyone has experience with cirrus-ci, and wants to pair on this with me (evenings/weekends Europe/London time), please reply to this issue, and we can set something up.

I had a go at doing this with cross, but no luck. https://github.com/alsuren/cargo-quickinstall/runs/5345278298?check_suite_focus=true

Maybe you can try cargo-zigbuild?
I haven't tried it myself but according to is README, it supports aarch64-unknown-linux-gnu, so naturally it should also support armv7-unknown-linux-gnu and musl targets.
There's also a docker image for it on docker hub.

Fixed by #123

well, not fixed :(

well, not fixed :(

Can you elaborate on the problem please?
I have already enabled armv7 and aarch64 linux gnu and musl targets.

i havent seen any builds for that arch go out for a while.

i havent seen any builds for that arch go out for a while.

It needs to be triggered by people running cargo-binstall or cargo-quickinstall on their arm machine.
Also, many builds failed due to missing openssl and these crates just don't support vendor-openssl feature, thus we would have to add them to excludes file.

@Milo123459 Checkout release semantic-release-cargo 2.1.55, it has armv7/aarch64 linux gnu/musl and was published 2d ago.


Fix inconsistent behavior of Bezier editor undo operations upon selection of different animation

Fixes #93859.
This PR removes the method _bezier_track_insert_key in the AnimationBezierEditor and adds a method bezier_track_insert_key_with_handle_mode in Animation. This new method is then called in the undo/redo operations, such that the correct animation is always tracked.
An alternative to this PR could be adding the animation as a parameter in the AnimationBezierEditor, but I figured that by parametrizing animation, the method it would lose all dependence on the editor, and should thus not be in it anymore.

Does the method using HandleMode have to be defined in #ifdef TOOLS_ENABLED, for Android, Linux, etc. to compile successfully? I'm unfamiliar with those flags


Remove Context class from NodeOperationGrouper

Codecov Report

Merging #5254 into master will decrease coverage by 0.02%.
The diff coverage is 100%.

@@             Coverage Diff              @@
##             master    #5254      +/-   ##
============================================
- Coverage     82.49%   82.46%   -0.03%
- Complexity    11027    11031       +4
============================================
  Files          1330     1330
  Lines         44975    44968       -7
  Branches       5626     5626
============================================
- Hits          37100    37084      -16
- Misses         5256     5268      +12
+ Partials       2619     2616       -3

Impacted Files
Coverage Δ
Complexity Δ

...va/io/crate/planner/node/NodeOperationGrouper.java
77.77% <100%> (-9.73%)
3 <3> (ø)

...exceptions/GroupByOnArrayUnsupportedException.java
66.66% <0%> (-33.34%)
1% <0%> (-1%)

.../main/java/io/crate/metadata/doc/DocTableInfo.java
65.89% <0%> (-16.77%)
52% <0%> (-3%)

...te/executor/transport/SubSelectSymbolReplacer.java
68.88% <0%> (-11.12%)
3% <0%> (ø)

core/src/main/java/io/crate/types/ArrayType.java
73.84% <0%> (-7.7%)
24% <0%> (-2%)

...rc/main/java/io/crate/analyze/UnboundAnalyzer.java
80% <0%> (-5%)
2% <0%> (ø)

...o/crate/planner/node/dql/join/NestedLoopPhase.java
74.69% <0%> (-3.62%)
20% <0%> (-2%)

...c/main/java/io/crate/jobs/JobExecutionContext.java
90.51% <0%> (-1.73%)
22% <0%> (-1%)

...er/src/main/java/io/crate/sql/tree/AstVisitor.java
28.81% <0%> (-1.7%)
34% <0%> (-2%)

...io/crate/protocols/postgres/ConnectionContext.java
78.24% <0%> (-1.15%)
31% <0%> (ø)

... and 14 more

Continue to review full report at Codecov.

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


Doc site, typo in ease-elastic-out-3 demo

There is a typo in the doc site index.html, and it causes a delay in the animation:

ease-elastic-out-3

fixed via https://github.com/argyleink/open-props/pull/516


:bug: css file missing from build

the css files for these components are not in the build.
we only have this css for in the build :

By default, vite config compiles all css files into a single file.
I've tried using: cssCodeSplit: true, which splits the css files, but nothing works, the build keeps making a single one

source
Note

If you specify build.lib, build.cssCodeSplit will be false as default.

Test with CSS in JS


Add "Forgot Password" to the Strapi Studio login page

There is also no "forgot password" link on the studio login screen, so no way to automatically reset password in case one forgets his password.

Thank you for this feedback.
This function is now ready to use in login page.


LogstashMarker#equals() and #hashCode() are unstable

Two LogstashMarker are equal if:

they have the same name (immutable)
they have the same references (children markers)

Problem is additional markers can be added to the reference list after a LogstashMarker is created breaking the equals/hashCode methods. This can lead to unpredictable and strange behaviour when the marker is used in a Set or as a Key in a Map.
Ability to add children to a marker (references) is a standard Slf4J behaviour mandated by the marker interface. However, the legacy Slf4J BasicMarker implementation does not include the referenced elements in the equals/hashCode computation... only the LogstashMarker does. I'm not fully aware of all the Marker particularities and I can't see why the Logstash version should behave that way.
@philstr What's your opinion? Are you aware of the reasons? If none can be identified, may be we could just get rid of this behaviour and rely on the standard equals/hashCode... ?

That sounds reasonable to me! But I think you tagged the wrong<EMAIL_ADDRESS>:) - did you mean philsttr@?

My bad...  I meant @philsttr ...

I believe the contains check in add is currently intended to check for logically equivalent markers.
However, I don't think the logical equivalence provides much value, so I'd be ok if the equals and hashcode were removed so that object identity comparison is used everywhere.
This might break some strange edge cases leading to "duplicate" markers being added, so will need to be added to release notes as a backwards incompatibility.


Upgrade to latest open release

This needs to be tested against 11.0 SP9 and upgraded if necessary.

Project Archived.


View specs without Capybara

I know I shouldn't post this here, I'm just trying to draw attention to my SO question which is sitting unanswered for days.
Environment

Ruby 2.3.1
Rails 4.2.8
Devise 3.5.10

Current behavior
I have the following Devise view (in HAML):
- title 'Lost Password'
.row
  .col-lg-6
    = form_for resource, as: resource_name, url: password_path(resource_name) do |f|
      = render 'layouts/errors', object: resource
      .form-group
        = f.label :email
        = f.text_field :email, class: 'form-control', autofocus: true
      = f.submit 'Send me the instructions', class: 'btn btn-primary'
    %hr
    = render 'devise/shared/links'
  .col-lg-6
    = render 'devise/shared/advantages'

If I run the following spec on this view:
require 'rails_helper'
describe 'devise/passwords/new', type: :view do
  it 'sets the page title' do
    expect(view).to receive(:title).with('Lost Password')
    render
  end
end

It says: undefined local variable or method 'resource'. I tried to put:

allow(view).to receive(:resource).and_return(User.new)

before the expect but then I get [snip, long class definition] does not implement: resource.
Expected behavior
The spec works. On the Wiki the examples use Capybara but I don't want to use it for something as trivial as this.

Solved on SO with @mbuechmann 's help.


Striped: not apllied anymore

Hi,
The striped option for the table-data (master branche) is not working properly anymore: the stripe is only applied to the "selection" column if present.
Please note that this functionality use to work as expected in the past (one full lane over two being added a gray background).
This behavior can be observed on the Predix UI catalogue: [https://predixdev.github.io/predix-ui/?show=px-data-table&type=component]
Would it be possible to re-enable this functionality?
Thanks,
Arnaud

Fixed in release  9ee4a15


How to connect websocket , mqtt , ..  WWW online

How to connect websocket , mqtt , ..  WWW online
RTSP online Internet
Sorry .My English very bad.

websockets, mqtt are unrelated to this.
It's an implementation of RTSP protocol.
If you have the knowledge you can add mqtt implementation yourself to talk with other services.
To access RTSP stream via public internet, and I suppose you want to do it from your home network, just map the rtsp port on your router to have public external access, that's all.


Image needs to be updated

I think it is about time the images from this page gets updated. Most or all of the screenshots are from old UI.
https://learn.microsoft.com/en-us/azure/devops/pipelines/library/connect-to-azure?view=azure-devops&preserve-view=true#create-an-azure-resource-manager-service-connection-using-automated-security
### Tasks

hmm.the issue is still there not sure why this issue is getting fixed.

We've moved this issue to our internal tracking system as we transition to a new feedback system.

@juliakm how can i track the progress on this task in future?


Installing charts from stable repository fails to fetch values

Describe the bug
When installing chart from deprecated stable repository, Lens app fails to fetch values and does not show any error
To Reproduce
Steps to reproduce the behavior:

Go to 'Apps'
Select chart from stable repository
Click 'Install'
See values not showing up

Expected behavior
Clear error should be shown that fetching values is failing

Also to be note that since stable repository is deprecated it's not possible to install charts anymore from it.

Workaround to this issue is:
A: manually update (via command line) repo url for stable to point https://charts.helm.sh/stable
B: switch to use official repositories found in Artifact Hub (recommended) (can be configured in Preferences)
note stable repository is deprecated and charts won't get any updates anymore, so it's highly recommend to use other repositories, for example bitnami.


dts: arm: st: Add support for stm32l433Xb

Adds support for stm32l433Xb.
And add I2C_1 and CAN pin of L4 series.
Signed-off-by: Kwon Tae-young<EMAIL_ADDRESS>

@erwango do you want to review this?

@ioannisg The commit message has been changed.

@ioannisg there is no STM32 dev board supporting these soc variants. So I don't think this should be blocking


cpu consuming

With lualine neovim is in top 5 cpu consumers.

I thought it was fixed.

I thought it was fixed.

I cannot reproduce this on my machine. Obviously the cpu usage spikes on buffer load but that happens even without lualine installed due to all the underlying autocommands.

I cannot reproduce this on my machine. Obviously the cpu usage spikes on buffer load but that happens even without lualine installed due to all the underlying autocommands.

Yes you're right it's my fault.
I reinstalled the plugin and everything works perfectly.
The branch changes on rebase perfectly without nvim focus and the CPU is quiet.
I apologize for not paying attention to the plugin installation.
You did a very good job.

Yes you're right it's my fault.
I reinstalled the plugin and everything works perfectly.
The branch changes on rebase perfectly without nvim focus and the CPU is quiet.
I apologize for not paying attention to the plugin installation.
You did a very good job.


How can we help?
I would like help so I can submit the manifest.
Please read and ensure the following

[X] The installer meets the above requirements

Please provide the following information

Download Page Url: https://github.com/google/brotli/releases/tag/v1.1.0
Publisher: Google
Package Name: Brotli
Description:

Short: Brotli compression format.
Full:

Brotli is a generic-purpose lossless compression algorithm that compresses data using a combination of a modern variant of the LZ77 algorithm, Huffman coding and 2nd order context modeling, with a compression ratio comparable to the best currently available general-purpose compression methods. It is similar in speed with deflate but offers more dense compression.

Package Version: v1.1.0
Installer URL:

x64: https://github.com/google/brotli/releases/download/v1.1.0/brotli-x64-windows-static.zip
x86: https://github.com/google/brotli/releases/download/v1.1.0/brotli-x86-windows-static.zip

### Tasks
- [ ] Create manifest of `Google.Brotli`

@Dragon1573 Is it a portable application or not?


For additional log information, please append the contents of
file://C:\Users\Bodka\AppData\Local\Temp\report1.txt.

If you're still experiencing this issue, it seems someone was able to fix this by deleting their ~/.dartServer directory in this issue: https://github.com/dart-lang/sdk/issues/41256

This has not been reported again since Dart 2.7; please re-open if you see this in a recent version of Dart/Flutter.

This has not been reported again since Dart 2.7; please re-open if you see this in a recent version of Dart/Flutter.


Create index on (entity_id, time)

Currently the TimescaleDB extension creates two indexes, on on entity_id and one on time. However, most queries will filter by both entity and time and therefore a compound index on both (entity_id, time DESC) will improve those queries considerably. The TimescaleDB manual also suggests creating such indexes.

Hi @sithmein , Is this issue still relevant?

No. It has already been implemented in the HA ltss add-on. I tend to confuse HA add-ons with hassos add-ons.


Unit test converting client props to BatchWriterConfig

Created two unit tests for the conversion of client properties to BatchWriterConfig. One using default values and another using modified values.
Refactored getBatchWriterConfig as suggested in initial ticket.
Added code in getBatchWriterConfig to allow for updating BATCH_WRTIER_THREADS_MAX as it was missing in the method as it existed.

This may be better handled in a separate PR but, it looks like we could add very similar tests for ConditionalWriterConfig.

I created #2131 after looking into this.

Thanks @DomGarguilo


Fix missing proposer in proposal endpoint

The proposer is never returned in the /proposals/{id} endpoint. The code was commented out because the akash proposer endpoint is returning an error for proposals created during genesis. On top of that, the workaround of checking for only post-genesis proposals (id > 3) doesn't work for sandbox/testnet.
Failing akash endpoint: https://akash-api.polkachu.com/gov/proposals/1/proposer
We need to:

Make sure proposer is returned for all proposal (pre or post-genesis)
Make sure this works with sandbox/testnet so no hardcoded proposal ids

We could also consider removing the endpoint or proposer entirely in the if there is no need for it anymore.

lowered priority because this page is now on stats.akash.network and pretty hard to get to


Schematics in PDF format

Suggestion to add the schematics in PDF format so that it can be viewed without any CAD software.

new board schematic can be found here
https://github.com/SRA-VJTI/sra-board-hardware-design/blob/master/renders/sra_dev_board_2022_schematics.pdf


Rover API Server

This MR is to create a debug screen for Rover connection
What's in the MR:

[ ] Setting screen with back to AppMenu
[ ] Adding HTTP API call
[ ] Split into UI, ViewModel, and API functionalities.
[ ] API funcationality will be used for future LLM integration


⚠ Syncthing-KA-TM has degraded performance

In 1aadd67, Syncthing-KA-TM ($URL_SYNCTHING_MERZ_NIMBUS_TM) experienced degraded performance:

HTTP code: 200
Response time: 9738 ms

Resolved: Syncthing-KA-TM performance has improved in 9390fd8 after 4 minutes.


Un-comment AzureKeyVault@1 secretsFilter argument

According to the docs, the secretsFilter argument is required. As such, this change un-comments the argument in the snippet to avoid potential confusion for users since having the argument commented-out may be interpreted as the argument being optional.

@c-w : Thanks for your contribution! The author, @, has been notified to review your proposed change.


Reconsider the children functionality of entities

I'm not sure I'm happy with it at present. I might remove it or modify it.

Just went over it again, I think the functionality is alright for now.


open /mnt/sda1/var/lib/docker/tmp/GetImageBlob152807945: read-only file system

Expected behavior
Install Shinobi CCTV Docker Kinematic as explained in Install Doc.
Actual behavior
open /mnt/sda1/var/lib/docker/tmp/GetImageBlob152807945: read-only file system

Can you provide the system that this is running under and the associated steps to reproduce the error? (what exact image)

Closing this issue due to lack of update/activity - Feel free to re-open if appropriate.


Reduce iOS memory

Steps to reproduce

I am working about  reducing ios memory. When I opening flutter app , there are  three IOSurface memory, and every one of them costs about 20MB memory and they altogether costs about 60MB memory
Why there are three IOSurface memory? Whether can reduce the amount of IOSurface？
Expected results
Reduce the amount of IOSurface
Actual results
There are  three IOSurface memory
Code sample
Code sample
[Paste your code here]

Screenshots or Video

Screenshots / Video demonstration
[Upload media here]

Logs
Logs
[Paste your logs here]

Flutter Doctor output
Flutter 3.22.2
Framework • revision 761747bfc5 (4 个月前)
Engine • revision edd8546116
Tools • Dart 3.4.3 • DevTools 2.34.3

There are three IOSurfaces because these are the three swapchain images. We don't plan to/can't remove them


scene.removeLayer 报错

canvas.js:37 Uncaught (in promise) TypeError: Cannot read properties of null (reading 'style')
at CanvaModel.eval [as renderUpdate] (canvas.js:37:1)
at CanvasLayer.renderModels (index.js:70:1)
at CanvasLayer.render (BaseLayer.js:617:1)
at LayerService._callee5$ (LayerService.js:263:1)
at tryCatch (regeneratorRuntime.js:44:1)
at Generator.eval (regeneratorRuntime.js:125:1)
at Generator.eval [as next] (regeneratorRuntime.js:69:1)
at asyncGeneratorStep (asyncToGenerator.js:3:1)
at _next (asyncToGenerator.js:22:1)

| eval | @ | canvas.js:37
-- | -- | -- | --
  | renderModels | @ | index.js:70
  | render | @ | BaseLayer.js:617
  | _callee5$ | @ | LayerService.js:263
  | tryCatch | @ | regeneratorRuntime.js:44
  | eval | @ | regeneratorRuntime.js:125
  | eval | @ | regeneratorRuntime.js:69
  | asyncGeneratorStep | @ | asyncToGenerator.js:3
  | _next | @ | asyncToGenerator.js:22
  | Promise.then (async) |   |
  | asyncGeneratorStep | @ | asyncToGenerator.js:12
  | _next | @ | asyncToGenerator.js:22
  | Promise.then (async) |   |
  | asyncGeneratorStep | @ | asyncToGenerator.js:12
  | _next | @ | asyncToGenerator.js:22
  | eval | @ | asyncToGenerator.js:27
  | eval | @ | asyncToGenerator.js:19
  | renderLayers | @ | LayerService.js:288
  | eval | @ | LayerService.js:51
  | invokeFunc | @ | lodash.js:10371
  | leadingEdge | @ | lodash.js:10411
  | debounced | @ | lodash.js:10449
  | _callee3$ | @ | LayerService.js:179
  | tryCatch | @ | regeneratorRuntime.js:44
  | eval | @ | regeneratorRuntime.js:125
  | eval | @ | regeneratorRuntime.js:69
  | asyncGeneratorStep | @ | asyncToGenerator.js:3
  | _next | @ | asyncToGenerator.js:22
  | eval | @ | asyncToGenerator.js:27
  | eval | @ | asyncToGenerator.js:19
  | remove | @ | LayerService.js:188
  | _callee3$ | @ | index.js:270
  | tryCatch | @ | regeneratorRuntime.js:44
  | eval | @ | regeneratorRuntime.js:125
  | eval | @ | regeneratorRuntime.js:69
  | asyncGeneratorStep | @ | asyncToGenerator.js:3
  | _next | @ | asyncToGenerator.js:22
  | eval | @ | asyncToGenerator.js:27
  | eval | @ | asyncToGenerator.js:19
  | removeLayer | @ | index.js:278

与CanvasLayer公用时冲突

可以提供各在线demo？


checkEnv causes CSP error in Firefox 40

Looks like the checkEnv function in utils.js does an eval to see if certain functions exist in the current environment, and if you have restrictive CSP settings that don't allow eval, this will throw an error in Firefox 40 (even with the try block wrapping it):

I thought about opening a PR with a fix but I'm not quite sure what a good solution would be. In the browser something like this would work:
function checkEnv(target) {
    var flag = false;

    if (window[target]) { flag = true; }

    environment[callbackName(target, "has")] = flag;
}

But that doesn't play nice when there's no window. Any suggestions?

Reflux-core is not written for the browser, it's a node.js module. You'll need to transpile it with something like browserify and babel if you want to use reflux-core in the browser.
Alternatively, you can use reflux.js instead, which has shims around the library, and a few extra functions.
If this still errors out once browser-ified, or if it errors out in reflux.js
Side note...  these functions should still be refactored/removed as needed, based on the reflux-core migration to node and server-side.
There's going to be a bit of work associated with this one. One of the core decisions regarding the development of reflux thus far has been to limit the dependencies that are included with reflux.
These utility functions are there so you can provide your own shims for promises and browser compatibility, and then have those available behind a common interface inside the reflux core.

@LongLiveCHIEF Thanks for the context.
We're using reflux.js but I opened the issue here when I saw that the error was coming from reflux-core code. I can raise the issue in reflux.js instead.

@eanakashima Yes, sorry about that. You'll need to run older version of refluxjs (0.2.11 I believe if the CHANGELOG is right) until I remove the eval check and replace the current promise methods with the simpler deferWith solution.

@spoike good to know, thank you!

:+1:


development URL

Is the URL README file correct?

In Develop > Actions > On device testing set the development URL to http://local-dev-server-hostname-or-ip:8080/

It seems that the google home device is sending a get request to "/"
Frame 19273: 694 bytes on wire (5552 bits), 694 bytes captured (5552 bits) on interface 0
Ethernet II, Src: f0:72:ea:88:8b:30 (f0:72:ea:88:8b:30), Dst: Raspberr_7e:77:5d (dc:a6:32:7e:77:5d)
Internet Protocol Version 4, Src: <IP_ADDRESS>, Dst: <IP_ADDRESS>
Transmission Control Protocol, Src Port: 38838, Dst Port: 8080, Seq: 138161, Ack: 85361, Len: 628
Hypertext Transfer Protocol
GET / HTTP/1.1\r\n
[Expert Info (Chat/Sequence): GET / HTTP/1.1\r\n]
[GET / HTTP/1.1\r\n]
[Severity level: Chat]
[Group: Sequence]
Request Method: GET
Request URI: /
Request Version: HTTP/1.1
Host: <IP_ADDRESS>:8080\r\n
Connection: keep-alive\r\n
Upgrade-Insecure-Requests: 1\r\n
User-Agent: Mozilla/5.0 (X11; Linux aarch64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/76.0.3809.81 Safari/537.36 CrKey/1.42.180404\r\n
Accept: text/html,application/xhtml+xml,application/xml;q=0.9,image/webp,image/apng,/;q=0.8,application/signed-exchange;v=b3\r\n
CAST-DEVICE-CAPABILITIES: {"audio_assistant":true,"bluetooth_supported":true,"display_supported":false,"hi_res_audio_supported":false,"remote_control_input_supported":false,"touch_input_supported":false}\r\n
Accept-Encoding: gzip, deflate\r\n
Accept-Language: en-us,en\r\n
\r\n
[Full request URI: http://<IP_ADDRESS>:8080/]
[HTTP request 221/222]
[Prev request in frame: 19227]
[Response in frame: 19274]
[Next request in frame: 19312]

Thanks for the report. We do need to update the README as a result of a recent console change. The local development server now prints two URLs that correspond to the console fields testing URL for Chrome and testing URL for Node.


.bind() ambiguity

Hi!
Am constantly confused by Model.bind('event', handler) which is by sense what they'd call Model.on('event', handler)...
The word "bind" seems to be more associated with JS Function#bind -- in particular Underscore#bind()/Underscore#bindAll().
Thus the constructions like
_.bindAll(this, 'render');
this.bind('change', render);
which are common in .initialize() code are a bit confusing.
I'd suggest to get rid of word "bind" in event binding code in favor of "on" --
_.bindAll(this, 'render');
this.on('change', render);
What do you think?
TIA,
--Vladimir

As of version 1.7, jQuery will also be supporting / moving to on and off for events; see http://blog.jquery.com/2011/09/28/jquery-1-7-beta-1-released/ :

New Event APIs: .on() and .off()

Over time, jQuery has evolved three ways to attach events to elements: .bind() , .live(), and .delegate(). Underneath it all, though, the three event APIs call the browser’s event system; that can lead to surprising interactions. For example, $(document).unbind("click") will remove all .live("click", ...) events, since those delegated events are attached to document. (This is also why you should use event namespaces.)

Our current event APIs aren’t going away soon, but to address the inconsistencies we’ve introduced a new and simple pair of event methods that can do the work of all three:

$(elems).on(events, selector, data, fn);
$(elems).off(events, selector, fn);

If a selector is provided, it’s a delegated event; otherwise it’s directly bound. All the features of the old APIs are there; for example events can be a space-separated string of event names and/or namespaces, or it can be an object where the keys are events and the values are event handler functions.

So, looks like a good time to start supporting this in Backbone as well to me ;)

Currently tracking here: https://github.com/documentcloud/backbone/issues/641


[SPARK-39759][SQL] Implement listIndexes in JDBC (H2 dialect)

What changes were proposed in this pull request?
Implementing listIndexes in DS V2 JDBC for H2 dialect.
Why are the changes needed?
This is a subtask of the V2 Index support(https://issues.apache.org/jira/browse/SPARK-36525).
It can better test the index interface locally.

This PR implements listIndexes in H2 dialect.

Does this PR introduce any user-facing change?
Yes, listIndexes in DS V2 JDBC for H2
How was this patch tested?
Update existed UT.

ping @huaxingao

Can one of the admins verify this patch?

Does H2 support composite index?

Merged to master. Thanks @panbingkun


Exporter data transmission stop

In our ES cluster, there are more than 20 exporter pods. With an increase in the number of pods, data from the exporter stops coming into Azure monitor. There are no records in the logs of the exporter and nodes. Can you tell us which setting we should look at to better understand the cause of the error?

I would first look at the exporter logs. Then you can check the metrics endpoint of the exporters to see if the metrics look okay.
I would also suggest running a single exporter for the whole cluster instead of 20 individual exporter pods.


Perf content updates

Making live the new performance recommendation content. This is now split into two parts:

Understanding performance and how to rationalize the system
Detailed recommendation and development steps to take for Unity

This allows us to add content for DirectX or Unreal going forward.
Modified references in scene to point to new pages and not old pages
Also updated Recommended Settings for Unity page to include best practices for performant environments.

OPS Build status updates of commit cbdba0f:
:warning: Validation status: warnings

File
Status
Preview URL
Details

mixed-reality-docs/comfort.md
:warning:Warning
View
Details

mixed-reality-docs/performance-recommendations-for-unity.md
:warning:Warning
View
Details

mixed-reality-docs/porting-guides.md
:warning:Warning
View
Details

mixed-reality-docs/testing-your-app-on-hololens.md
:warning:Warning
View
Details

mixed-reality-docs/TOC.md
:white_check_mark:Succeeded
View

mixed-reality-docs/app-quality-criteria.md
:white_check_mark:Succeeded
View

mixed-reality-docs/case-study-scaling-datascape-across-devices-with-different-performance.md
:white_check_mark:Succeeded
View

mixed-reality-docs/hologram-stability.md
:white_check_mark:Succeeded
View

mixed-reality-docs/holographic-frame.md
:white_check_mark:Succeeded
View

mixed-reality-docs/performance-recommendations-for-hololens-apps.md
:white_check_mark:Succeeded

mixed-reality-docs/performance-recommendations-for-immersive-headset-apps.md
:white_check_mark:Succeeded

mixed-reality-docs/recommended-settings-for-unity.md
:white_check_mark:Succeeded
View

mixed-reality-docs/rendering-in-directx.md
:white_check_mark:Succeeded
View

mixed-reality-docs/spatial-mapping.md
:white_check_mark:Succeeded
View

mixed-reality-docs/submitting-an-app-to-the-microsoft-store.md
:white_check_mark:Succeeded
View

mixed-reality-docs/understanding-performance-for-mixed-reality.md
:white_check_mark:Succeeded
View

mixed-reality-docs/comfort.md

Line 45: [Warning] Illegal link: coming soon -- missing bookmark. The file index.md doesn't contain a bookmark named news-and-notes.
Line 66: [Warning] Illegal link: coming soon -- missing bookmark. The file index.md doesn't contain a bookmark named news-and-notes.
Line 135: [Warning] Illegal link: coming soon -- missing bookmark. The file index.md doesn't contain a bookmark named news-and-notes.

mixed-reality-docs/performance-recommendations-for-unity.md

Line 0: [Warning] Text in the first line of Note/Section/Video is not valid. Will be rendererd to

Line 13: [Warning] Invalid file link:(~/performance-recommendations-for-mixed-reality.md).
Line 33: [Suggestion] Illegal link: CPU -- missing bookmark. The file performance-recommendations-for-unity.md doesn't contain a bookmark named CPU-Performance-Recommendations.
Line 34: [Suggestion] Illegal link: CPU to GPU -- missing bookmark. The file performance-recommendations-for-unity.md doesn't contain a bookmark named CPU-to-GPU-Performance-Recommendations.
Line 35: [Suggestion] Illegal link: GPU -- missing bookmark. The file performance-recommendations-for-unity.md doesn't contain a bookmark named GPU-Performance-Recommendations.
Line 36: [Suggestion] Illegal link: Memory -- missing bookmark. The file performance-recommendations-for-unity.md doesn't contain a bookmark named Memory-Recommendations.
Line 37: [Suggestion] Illegal link: Startup Performance -- missing bookmark. The file performance-recommendations-for-unity.md doesn't contain a bookmark named Startup-performance.

mixed-reality-docs/porting-guides.md

Line 16: [Warning] Illegal link: coming soon -- missing bookmark. The file index.md doesn't contain a bookmark named news-and-notes.

mixed-reality-docs/testing-your-app-on-hololens.md

Line 53: [Warning] Illegal link: coming soon -- missing bookmark. The file index.md doesn't contain a bookmark named news-and-notes.

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.


URL encoding error in Query parameter

Query parameter has an space character. So I got this error:
Wasp Error: , Message:java.io.IOException: Illegal character in query at index 75:

Let us check this issue asap and fix it. Thanks for the issue

thanx


Automate release process

This PR adds a release job on Master pushes that does two things:

Makes a new release tag with the version identifier.
Pings the devops repo to trigger auto-dependent-updates.

Fixes: https://github.com/runtimeverification/devops/issues/110
Fixes: https://github.com/runtimeverification/devops/issues/54

Evidence this works: https://github.com/runtimeverification/pyk/actions/runs/3299342768/jobs/5442600771 and https://ci.runtimeverification.com/jenkins/job/DevOps/job/master/40755/


SWARM-697: Fix StackOverflowError during gradle artifactresolving

[x] Have you followed the guidelines in our Contributing document?
[x] Have you created a JIRA and used it in the commit message?
[x] Have you checked to ensure there aren't other open Pull Requests for the same issue?
[x] Have you built the project locally prior to submission with mvn clean install?

Motivation
There are 3rd-party libs (even on mavencentral) which have circular dependencies. This shouldn't break the build when building the swarm-jar. It should also be possible to force versions of transitive dependencies.
Modifications
Used gradle-features to resolve artifacts to a given configuration. This is much faster and can handle the circular dependencies. The detached configuration to resolve dependencies copies the forced modules from the compile-configuration.
Result
The wildfly-swarm-package task can handle circular 3rd-party libs and respects forced versions from the ResolutionStrategy from the configuration compile.

Can you provide a test case reproducing this issue?

If by "testcase" you mean a corrupt 3rd-party-dependency with circular dependencies it's the following for example.
compile group: 'org.apache.xmlgraphics', name: 'batik-anim', version: '1.7'
Version 1.8 fixed this issue - but as I described in the JIRA-ticket it's not possible to force the version from outside of the plugin.

Testing gradle-plugins would be much easier when developing the gradle-plugin with gradle. This way you have all the test-support (there are binaries which aren't available on a public maven repo - so you can't include them in a maven build for proper testing). But I'll respect that you are maven-oriented and want your whole build in maven.


Add support for empty shapes, used as markers

Issue #, if available: prerequisite for WAFv2 RuleGroup and WebACL CRDs
Description of changes:
Some WAFv2 API fields have empty json specs {}, e.g. https://docs.aws.amazon.com/waf/latest/APIReference/API_AllQueryArguments.html
For these type of fields, codegen currently errors out because it infers their gotypes as e.g.
AllQueryArguments *AllQueryArguments
but does not generate a type AllQueryArguments struct since the struct itself is empty, and not picked up by newFieldRecurse function.
The solution proposed in this PR is to allow users to define marker-shapes, which instruct codegen to overwrite the type of these empty structs as []byte, both when generating the APIs and when setting up the SDK.
e.g. generator.yaml for WAFv2, and inline:
marker_shapes:
- All
- Method
- UriPath
- QueryString
- AllQueryArguments
- RateLimitIP
- RateLimitForwardedIP
- RateLimitHTTPMethod
- NoneAction

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

@TiberiuGC this is great, i love it! Can you please provide an example generator.yaml in the PR description?

/retest

/retest


Internal testing

Document how to do internal testing
https://github.com/IBM-Bluemix/gp-js-client/issues/18

@yumaoka to review

LGTM.


MJX produces NaN values while regular mujoco doesn't.

Hello, I'm playing around with mujoco version 3.0.1 and I have an issue where mjx produces NaN values while the regular version of mujoco doesn't. Currently, I'm just trying to simulate a box dropping onto the ground.
Here's a minimal example that reproduces the issue. If I use the regular mujoco as my backend,
MJCF_XML = """
<mujoco model="test">
  <option timestep="0.001" iterations="1" ls_iterations="4">
    <flag eulerdamp="disable"/>
  </option>

  <worldbody>
    <geom name="floor" type="plane" conaffinity="1" size="100 100 .2"/>

    <body name="cart" pos="0 0 0.5">
      <freejoint/>
      <geom name="cart" type="box" size="0.2 0.15 0.1" mass="1"/>
    </body>
  </worldbody>
</mujoco>
"""

if __name__ == "__main__":
    model = mujoco.MjModel.from_xml_string(MJCF_XML)
    data = mujoco.MjData(model)
    mujoco.mj_step(model, data)
    print(data.qpos)

This prints out [0. 0. 0.49999019 1. 0. 0. 0.].
Now if I change the code to use mjx as the backend:
if __name__ == "__main__":
    model = mujoco.MjModel.from_xml_string(MJCF_XML)
    model = mjx.device_put(model)
    data = mjx.make_data(model)
    data = mjx.step(model, data)
    print(data.qpos)

This will print out [nan nan nan nan nan nan nan].
Any insights as to where the problem could be? Thanks!

Oh good catch - there is indeed a divide by zero bug in the solver when no constraints are imposing any cost.  Will post a fix shortly.


Regular updates (Apr 16)

Address https://github.com/iterative/dvc.org/issues/1123#issuecomment-614929585
Complete #1097 by removing mentions of raw metrics type
Add --summary option to checkout cmd ref.

@jorgeorpinel merging this?

Yes, thanks.


Roll engine to 16b2964f5c300ceea7aac17a82a6afd1ac36b594

LGTM but travis is mad

Patiently waiting for buildbots to do their thing. :)

Travis has been appeased.


Add examples in lerna packages

Currently their versions aren't kept in sync, we just need to add them here: https://github.com/open-telemetry/opentelemetry-js-contrib/blob/main/lerna.json#L4

Pretty much duplicate of #422 - same solution?

Yes that would be the same solution


wasm: fix arbitrary memory allocation attack

fix #147

@sbinet, review?

Codecov Report

Merging #148 into master will increase coverage by 0.15%.
The diff coverage is 83.33%.

@@            Coverage Diff             @@
##           master     #148      +/-   ##
==========================================
+ Coverage   69.47%   69.63%   +0.15%
==========================================
  Files          43       43
  Lines        4921     4943      +22
==========================================
+ Hits         3419     3442      +23
  Misses       1217     1217
+ Partials      285      284       -1

Impacted Files
Coverage Δ

wasm/types.go
48.12% <80%> (+0.65%)
:arrow_up:

wasm/section.go
54.58% <81.81%> (+0.93%)
:arrow_up:

wasm/read.go
72.5% <92.3%> (+8.86%)
:arrow_up:

Continue to review full report at Codecov.

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


Allow where clause in subgraph queries to reference another field in type

Do you want to request a feature or report a bug?
Feature
What is the current behavior?
Currently, the where clause in a subgraph query only allows you to pass values to it that are of the type of the field being filtered against.
ex:
query Indexers {
  indexers(where: { stakedTokens: 0 }) {
    id
    stakedTokens
    lockedTokens
  }
}

Where: stakedTokens is of type BigInt and the value passed (0 in the example) must also be a BigInt.
What is the expected behavior?
It would be very helpful if the where clause was able to compare two fields against each other to filter results.
ex:
query Indexers {
  indexers(where: { stakedTokens: {indexer? me? this?}.lockedTokens }) {
    id
    stakedTokens
    lockedTokens
  }
}

Mapping down to sql, this would look (something like):
SELECT id, staked_tokens, locked_tokens FROM indexer WHERE staked_tokens = locked_tokens;

It would also be very helpful if the RHS could be a nested value, like:
purchase(where: { timestamp_gte: this.vault.epochStart }) {
  id
  timestamp
  vault {
    id
    epochStart
  }
}

looking something like
SELECT purchase.id as purchase_id, timestamp, vault.id as vault_id, epoch_start
FROM purchase JOIN vault
WHERE purchase.timestamp >= vault.epoch_start

This wouldn't be valid GraphQL syntax which makes it really tricky to implement

How about generated columns? https://stackoverflow.com/questions/61674263/compare-two-fields-in-graphql-hasura#comment109109862_61680466
field1_gt_field2

hey @cmwhited this makes a lot of sense from a user perspective.
@llllvvuu interesting proposal on generated columns - I think a  challenge with the automated code generation to cover all cases would be an explosion of filters in the schema.
@saihaj in terms of valid graphQL syntax, is any reference of another field invalid (not just nested?)
I wonder if this is a focused use-case for custom resolvers

@saihaj in terms of valid graphQL syntax, is any reference of another field invalid (not just nested?)
@azf20 Yes. It is just not possible since the GraphQL specification makes resolutions of queries parallel whenever possible. Having a field dependent on a field would make this harder and we would build a custom execution engine to support.

Short term way we can try of thinking a way in schema.graphql file so users can annotate these are fields they want to compare against in query so we generate a very specific query.
Long term way is figuring is custom resolvers


randWord sometimes generated one letter

Is this a regression?
No
Description
When using randWord() it sometimes only gives you one letter, a letter is not a word.

Please provide a link to a minimal reproduction of the bug
No response
Please provide the exception or error you saw
No response
Please provide the environment you discovered this bug in
No response
Anything else?
If you agree, I can do a PR.
A quick check if the length > 1 can be sufficient, if not we regenerate
I don't think there is another way because falso uses Fako?
Do you want to create a pull request?
Yes

Sure, go ahead.


add HDF5_INCLUDE_DIR for compiled with Caffe

add HDF5_INCLUDE_DIR and HDF5_INCLUDE_LIBS to modules/cnn_3dobj/CMakeLists.txt for compile OpenCV with Caffe.
For  Debian-based Linux Systems, HDF5_INCLUDE_DIR may be /usr/include/hdf5/serial or /usr/include/hdf5/openmpi. If Caffe is compiled with HD5, this may be a problem.

@mylxiaoyi, thanks! may I ask you (and @mshabunin) - what happends if Caffe is compiled without HDF5, i.e. Caffe is there, but HDF5 is not? will cnn_3dobj still compile?

@vpisarev , this module is broken and should be fixed first. We does not build it on buildbot because it depends on caffe which is not installed on the buildslave.
Some of the problems:

some samples include wrong headers (example)
some samples probably depend on opencv_features2d and opencv_calib3d which are not added to the module dependencies, thus are not added to the samples dependencies (same example)
one of the source files is being rewritten during build process (cmake command)
one of the included caffe headers is obsolete (here)

AFAIK, caffe has switched to HDF5 format by default and probably can not be built without HDF5 library (https://github.com/BVLC/caffe/pull/2836).
There are two build systems available for caffe: cmake and Makefile. There is a chance that cmake will be chosen as the default (and the only?) BS at some time (https://github.com/BVLC/caffe/issues/3351). Maybe using standard cmake find_package for linking with caffe could be better choice.

@vpisarev , I test again, and find that caffe header file include/caffe/data_layers.hpp includes hdf5.h, so can not find hdf5.h when compiling and can not find libhdf5.so when linking.
I also noticed that the header files in caffe have been changed, so the error message now is opencv_contrib/modules/cnn_3dobj/include/opencv2/cnn_3dobj.hpp:68:35: fatal error: caffe/vision_layers.hpp: no such file or directory

using this version of caffe: https://github.com/BVLC/caffe/pull/5019 with HDF5 enabled and compiled using the CMake build system, I did not have to add the above includes to opencv.
AFAIK @sovrasov compiled the module as well.

@paroj , thank you!


Added transport feedbacks and fixed connection issues

This fixes several bugs and adds the channel status variable. It also transitions the code to ES6.
Tested on a 2 channel GV K2 Summit.
Adds

Transport ~variable~ feedbacks to check the channel status (playing, stopped, etc)
ES6

Fixes

Connection issues (reconnections would not work if it was waiting for a response, reconnections aren't always noticed since there was no timeout with constant poll/ping) (Fixes #4)
Issue where the buffer property could grow to an infinite size since it was always appended to
Issue where clip names won't always populate correctly

Swapped to feedbacks for status vs variables. The value is also now cached, so that logic isn't run every status update.

@onfire4g05 did you see my inline notes?

Yes. I'm on vacation this week with somewhat limited Internet. I can fix the issues next week!

This will also now allow for clip names (record or load) to use companion variables.


Feature Request - isCI

判断当前是否在 CI 环境中运行

Determine whether it is currently running in a CI environment

I have a separate package for that: https://github.com/sindresorhus/is-in-ci


Negative funds remaining

Bug
When the sum of the spending totals are more than the allowance, the UI glitches and displays negative SC remaining.
Client/Server Info
Client is on macOS 10.13.0 GM, commit NebulousLabs/Sia-UI@d3aca71
Server is on Ubuntu Server 16.04 LTS, commit NebulousLabs/Sia@f16b5eb
Screenshots

Thanks for the detailed report!
What's the desired behavior here? Seems like the amount spent shouldn't exceed the allowance in the first place, but I guess we shouldn't assume that will never happen. Maybe display 4.63 SC over allowance in red?

I believe there is a bigger issue with the UI display in the first place since the info that it is giving is very different than what the CLI gives me.
I also was able to decrease my future allowance to 300 SC and well, the UI broke hard and couldn't display the allowance bar without rescaling the window.
Renter info:
	Storage Spending:  118.3 SC
	Upload Spending:   3.697 SC
	Download Spending: 0 H
	Unspent Funds:     478.1 SC
	Total Allocated:   600.2 SC

Are you sure the UI is displaying different values than siac? They both pull from the renter's financialmetrics API endpoint, so they definitely should not be different. Thanks for the detailed report, there should be a pull request shortly to fix the sizing issues.

Looks like the UI's unspent calculation was separate from the API. I was able to reproduce this issue, and #728 should resolve it.

Just pulled the branch, bar width issue is fixed but the calculations still seem off.


No notifications received for incoming payments or for daily balance

I am unable to receive notifications. Also tried paying myself from WOS and it's the same irrespective of whether the app is open or not. Will look at logs to see what's going on.

@nicolasburtey While I no longer experience this bug, Salim reported that multiple users have been facing this. We should look into this. It could be a mobile side issue, but it's unclear at the moment.


Updated description of Mule License

Updated description of Mule License to specify it's not the path to the license digest but it's actual content.

Hey @danieldiaz-mule According to Sundip, this change was submitted in another PR that is already merged.


Fix DB table prefix when used on multisite

Hi, came across a bug when the WebAuthn plugin is network-activated on a multisite install.
Currently without this PR, if you visit a sub-site's admin profile page at sub-site.example.com/wp-admin/profile.php, the plugin will throw fatal Table doesn't exist errors.
The issue is In the Schema class, the DB table prefix should be global, rather than per-site.  For this PR, I've changed the prefix so the schema references the main site's DB tables for those that have already installed the plugin on multisite.
Let me know what you think.

I will play with this on the weekend, thank you!

@r-a-y I have updated the code a bit to use $wpdb->base_prefix. Could you please check if this works for you? Thank you!

Yes, using $wpdb->base_prefix should work. This was going to be my original PR, but this will break any current installations using multisite, which is why I chose to use the main site's DB tables in this PR.

Hi @sjinks , I see that there was a new release of the plugin, so I had to re-apply the fix on our site to address this problem.
If we do not care about backward-compatibility, then go ahead with the $wpdb->base_prefix solution. I can copy the existing DB table into the new one.

@r-a-y the problem is that I cannot reproduce the issue.
What I have done:

Created a multisite WP installation
Installed Two Factor plugins and activated them for network

Then I check the database and see that the tables do not have a numeric prefix:

When I go to /subsite/wp-admin/profile.php, I do see the error, but if I check the database, I don't see any "wrong" tables:

If I replace the table prefix:

-		$wpdb->webauthn_credentials = $wpdb->prefix . Constants::WA_CREDENTIALS_TABLE_NAME;
-		$wpdb->webauthn_users       = $wpdb->prefix . Constants::WA_USERS_TABLE_NAME;
+		$wpdb->webauthn_credentials = $wpdb->base_prefix . Constants::WA_CREDENTIALS_TABLE_NAME;
+		$wpdb->webauthn_users       = $wpdb->base_prefix . Constants::WA_USERS_TABLE_NAME;

the error is gone.
However, I cannot reproduce the case with incorrectly prefixed tables. Am I doing something wrong?

Thanks for following up, @sjinks .
With your help, I've identified a problem on our multisite install site. You can go ahead with the $wpdb->base_prefix fix for your plugin.
If you're interested in reading what the problem was on our site, read on. Otherwise, feel free to skip the rest of this comment!

The site is missing a define( 'MULTISITE', true ); line in wp-config.php. This causes $wpdb->get_blog_prefix() to return wp_1_ instead of just wp_: https://github.com/WordPress/WordPress/blob/c70a5c547f187e3a8c58779ae18155874cb412df/wp-includes/wp-db.php#L1065-L1069.
Interestingly enough, is_multisite() still returns true due to other constants that are defined in wp-config.php such as SUBDOMAIN_INSTALL: https://github.com/WordPress/WordPress/blob/c70a5c547f187e3a8c58779ae18155874cb412df/wp-includes/load.php#L1234-L1236. This is why our team has never noticed this problem until now, so the WebAuthn plugin actually helped us identify this problem! Thanks!

Thanks! I have added the fix, and will do a new release on Monday - there are several more issues to fix.


missing button

Hi joe,
Just tried latest release.
Carried out
`cd ~/adsb-receiver
git fetch --all
git reset --hard origin/master
./install.sh
`
Next Button missing from foot of setup page, (lite options selected at ./install.sh)
On install of web portal
[IP_ADDRESS_OF_YOUR_DEVICE]/admin/install.php
unable to save admin name,email,password so really stuck on this one.
Also I noted that although I selected wlan0 for bandwidth monitoring, sys info guages did not function on this one.

I just had the same problem.
I started with a fresh v2.0.0 image and got the /admin/install.php to work once, then I performed the update with
cd ~/adsb-receiver
git fetch --all
git reset --hard origin/master
./install.sh
and now I can't complete the /admin/install.php setup because the Next Button missing from foot of setup page, (lite options selected at ./install.sh)
The data directory is writable.
The settings.class.php file is writable.
Rebooted the RPi and I don't get the portal page either. Dump1090 is feeding data to FlightAware.

Seem to be having a similar problem: I keep getting redirected to /install/upgrade.php
The script says to delete: http://<IP_ADDRESS_OF_THIS_DEVICE>/admin/install.php - which doesn't exist  but I presume to be /install/upgrade.php

The Saturday morning I merged the 2.0.1 branch into master I meant to roll back the merge after having an issue with the image installation script while creating an image for the release and ended up forgetting to before falling asleep...
Anyways the PHP install script is now located at:
http://IP_ADDRESS_OF_YOUR_DEVICE/install/.
The merged changes work fine for upgrades and clean installs the only issue I have ran into was with the image setup script. I am working on the image setup script now so I will not be rolling back the merge now.
Please refer to the README.md file for updated instructions.
https://github.com/jprochazka/adsb-receiver/blob/master/README.md
I should have an image and official announcement posted within the next couple hours.
Sorry about any problems should have held off instead of staying awake trying to rush things...


No LICENSE file

Hi,
Please can a LICENSE file be added to the repository.
While the LICENSE is clear from the header that it is Apache II a LICENSE file would be good.

Thanks for the report, we will address this.


about  infer   speed?

I didn't find anything about the speed of reasoning.What is the speed of inference

Hi @zhanghongyong123456 ,
The following table takes a single 2048x2048 image as an example and compares the efficiency of our method with existing methods from the perspective of model parameters (Params), calculation amount (MACs), memory overhead (Mem) during training and inference, and inference runtime (Time). For the memory cost and runtime of inference, we show results when the input is split into 1/4/16 parts (splitting script will be released recently. Check it in TODO).
Due to the Random Step Crop (RSC) strategy designed in Section 3.4, our training memory cost is less than 3GB among all methods. By varying the number of input splits (splitting will not affect the performance due to the characteristic of INR), we can achieve competitive performance with color-to-color (C2C) methods either on inference memory cost or runtime.

We will also update these in new editions of the paper.

Hi @zhanghongyong123456 ,
I would like to inform you that we have updated the efficient inference script in our latest version. Hope you enjoy that. smiley

ok,thank you very much


nixos/rspamd: Fix enable for locals and overrides

Motivation for this change
When implementing #49620 I included an enable option for both the
locals and overrides options but the code writing the files didn't
actually look at enable and so would write the file regardless of its
value. I also set the type to loaOf which should have been attrsOf
since the code was not written to handle the options being lists.
This fixes both of those issues.
Things done

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

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

[x] 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 nox --run "nox-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] Fits CONTRIBUTING.md.

@GrahamcOfBorg test rspamd.customLuaRules

Success on x86_64-linux (full log)
Attempted: tests.rspamd.customLuaRules
Partial log (click to expand)
machine: exit status 1
syncing
machine: running command: sync
machine: exit status 0
test script finished in 207.28s
cleaning up
killing machine (pid 600)
vde_switch: EOF on stdin, cleaning up and exiting
vde_switch: Could not remove ctl dir '/build/vde1.ctl': Directory not empty
/nix/store/2xnd5ln1q9mnj6xdnzz6z2aq4n2himq5-vm-test-run-rspamd-custom-lua-rules

Success on aarch64-linux (full log)
Attempted: tests.rspamd.customLuaRules
Partial log (click to expand)
machine: exit status 1
syncing
machine: running command: sync
machine: exit status 0
test script finished in 112.92s
cleaning up
killing machine (pid 631)
vde_switch: EOF on stdin, cleaning up and exiting
vde_switch: Could not remove ctl dir '/build/vde1.ctl': Directory not empty
/nix/store/y9a9n7fm9h6qdakhrfk12w4cgnzf6dc0-vm-test-run-rspamd-custom-lua-rules


[API-M-4.0.0] APIM fails to start after enabling secure vault

Description:
I followed these steps as in the docs
Under "Encrypting passwords in product configurations"

Added [secrets] section and alias with passwords in deployment.toml.
Added $secret{<password_alias>} to relevant positions.
Encrypted passwords using ./ciphertool.sh -Dconfigure

Under "Encrypting secured endpoint passwords"
4. Added [apim] enable_secure_vault=true to deployment.toml.
5. Ran cipher tool using ./ciphertool.sh -Dconfigure.
6. Restarted the server sh wso2server.sh
After that I get following error.

I attached full error stack in this text file.

This issue happens when I enter [apim] enable_secure_vault=true at the very bottom of the deployment.toml file. It works when it is added before all the apim elements. Docs should clearly mention that.


Store vars in a separate file

I have need to point different runs to different TestRail projects, so I need to specify that config at execution. That works fine by using the proper flags for the Cy run. However, for security, I don't want my TestRail creds stored in those env.json files; additionally, to limit the duplication of files in my setup, I want to use a 'testrail.json' file to store all the config settings that I will reference for that run.
Dumbing this down to a single file / config, I've moved the 'testrail' section of the env.json into a testrail.json file and tried using a $ref:

but I get this error:

What am I missing to get this working correctly while storing these variables in a referenced file?

Hi there
aehm...not quite sure if $ref works out of the box, because I would probably need to support that :)
so you cant just invent a new parameter in that part of the config-file with my code
but let me first ask.. you know that alle these things can be added as ENV variable directly when executing it?
so what you are trying to say is that you DO want a file-based storage, but it should just not be the cypress.env.json?
I could add a fallback lookup for testrail.json in that case and it will work
what do you think?
can you clear up things for me and verify my assumption? thank you :)

@boxblinkracer Thanks, I guess I thought it should work based on it being a standard in the schema (https://json-schema.org/understanding-json-schema/structuring#dollarref)...
Anywho... so yes, I do want to have file-based storage for these as I don't want to muddy the waters of my pipelines AND that I need to have various sets based on the pipeline.
Adding a fallback should work but what does the implementation path look like to support $ref instead?

After thinking on this for a few more days, maybe it's easier to lust look at the fallback for the creds and I can specify the rest in the vars in specific env files I'll call in the cli...


Power control stop and start doesn't work

I'm trying to use the api to stop and start my servers. However only kill and restart work. But stop and start have no effect.
Doesn't work
curl -k -H "X-Access-Token: xxxxxxxxxxx" -H "X-Access-Server: xxxxxxxxxxxx" https://xx.xx.xx.xx:5656/server/power/start
Doesn't work
curl -k -H "X-Access-Token: xxxxxxxxxxx" -H "X-Access-Server: xxxxxxxxxxxx" https://xx.xx.xx.xx:5656/server/power/stop
Works
curl -k -H "X-Access-Token: xxxxxxxxxxx" -H "X-Access-Server: xxxxxxxxxxxx" https://xx.xx.xx.xx:5656/server/power/kill
Works
curl -k -H "X-Access-Token: xxxxxxxxxxx" -H "X-Access-Server: xxxxxxxxxxxx" https://xx.xx.xx.xx:5656/server/power/restart

I've been able to utilize the API to start/stop servers. Have you changed your query to:
https://xx.xx.xx.xx:5656/server/power/start and/or https://xx.xx.xx.xx:5656/server/power/stop?
Note that the stop/start queries may take a little while, depending on what Scales has to start on the server.
Let me know if this helps at all.

Change to what ? It's the same as I've writtent. I don't understand.
Also, nothing happens, even after a few minutes.

What version of Scales and PufferPanel are you running?

Puffer Panel 0.8.1 Beta
And for scales I don't know how to get it

Please update both scales and puffer panel to their latest versions. The latest Panel is 0.8.2 and the latest scales is 0.1.3.

Thanks for the answer.
I followed the instructions here to update https://community.pufferpanel.com/topic/125-pufferpanel-v082-beta-scales-v013/#comment-557
It was successful with no errors.
However, now, after the update .I can't control the servers from the panel anymore.
What should I do ?
Scales log after update :
23:10:02 - info: + ========================================== + meta=undefined
23:10:02 - info: | Scales logs all information, (inc. errors) | meta=undefined
23:10:02 - info: | into the logs/ directory. Please check     | meta=undefined
23:10:02 - info: | there before asking for help with bugs.    | meta=undefined
23:10:02 - info: |                                            | meta=undefined
23:10:02 - info: | Submit bug reports at the following link:  | meta=undefined
23:10:02 - info: | https://github.com/PufferPanel/Scales      | meta=undefined
23:10:02 - info: + ========================================== + meta=undefined
23:10:02 - info: Loading plugin bungeecord.js... meta=undefined
23:10:02 - info: Loading plugin mcserver.js... meta=undefined
23:10:02 - info: Loading plugin minecraft-pre.js... meta=undefined
23:10:02 - info: Loading plugin minecraft.js... meta=undefined
23:10:02 - info: Loading plugin srcds.js... meta=undefined
23:10:02 - info: Scales is now listening on port 5656 meta=undefined
23:10:02 - info: Scales is now listening to Socket.IO on port 5657 meta=undefined
23:10:02 - info: Scales is now listening for uploads on port 5658 meta=undefined
23:10:02 - verbose: All scripts in /lib/scripts successfully had their permissions updated. meta=undefined
23:10:02 - verbose: Loaded server configuration for 3335b4fb-2dc3-454f-9b1d-6641ee9084a8 meta=undefined
23:10:02 - verbose: Loaded server configuration for 3f9b99ea-cdd2-4f70-8751-0bbaaf41a246 meta=undefined
23:10:02 - verbose: Loaded server configuration for 47fc4b7d-9933-4f96-af36-7b0d8a87b805 meta=undefined
23:10:02 - verbose: Loaded server configuration for 586c3936-8fa0-410e-829e-ab07cf2c1b33 meta=undefined
23:10:02 - verbose: Loaded server configuration for b5b7860d-0bc8-4399-a5f7-ebfe1012928f meta=undefined
23:10:02 - verbose: Loaded server configuration for e1b0bbf0-f4bf-4d81-9670-b06ba41d02e0 meta=undefined

This is because to turn the server on or off you need to use on and off instead of stop and start. The documentation was incorrect and has been updated.

Works great with on and off . Thank you !


LogLevel.FromString cause vs2013 collapse

The issue like this:
Nlog 4.0.1 was Added by Nuget GUI.It was user like(C# Code:):
        private static Logger logger = LogManager.GetCurrentClassLogger();
        ....
        LogEventInfo lei = new LogEventInfo();
        lei.Properties["Operator_name"] = Session["adm_login"];
        lei.Level = LogLevel.Info;//collapse happend here

when LogLevel.FromString was used here,the vs2013 collapsed.So i want to know why?Thks,Anyone.
Tips:my English Capbility is low,plz.

What do you mean with "vs2013 collapsed"? A crash of visual studio?

This issue happened at my PC,but others,So it's my issue,thks!


Fix wrong filepaths (preview)

On Windows, document.uri.toString() returns urlencoded string like file:///v%3A/unity/..., which is misinterpreted by language server.
Fixes #8

Thanks for this PR! We've merged the main version of this PR (#10), so this will make its way into feature/preview when that branch is next updated. As such, I'll close this PR now.


revert 5 henryiii/fix/make

Didn't work, conda-build sets the value after we try to clear it. :(

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
Reset the build number to 0 (if the version changed)
Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.


relative path in JsonModel.getProperty()

Imagine a code like this
onPress : function(oEvent) {
    var path = oEvent.getSource().getBindingContext().getPath();
    var oModel = this.getView().getModel();
    var obj = oModel.getProperty(path);
}

You get a path, hence you know you are in /database/0/connection/4/name but very likely you will need elements from parents as well. Only solution is to split the string and construct the path again, e.g. oModel.getProperty(/database/0/connection/4/host); or oModel.getProperty(/database/0/databasetype);
With relative path support that would be easier:
oModel.getProperty(path + "/../host");

Or even better: A set of methods to navigate programatically, e.g.
oModel.getNode(path).getParentNode().getProperty();

Hello @wernerdaehn ,
Thank you for share your enhancement proposal. I've created an internal incident<PHONE_NUMBER>. The status of the issue will be updated here in GitHub.
Regards, Diana

As discussed in issue/2504:
@wernerdaehn original example for relative binding paths in composite bindings:
Imagine the situation where you use a JsonModel as the source of your e.g. tree table with databasevendor -> databaseversion -> connectioninfo.
If somebody does something on the connectioninfo level, it is very likely he will need the parent version as well, e.g. a formatter creating the link for the connection info screen.
If you could specify things like "{ parts: [ 'connectionname', '../databaseversion', '../../databasevendor'] }"

Hi @wernerdaehn ,
I have logged your requirement as CPOUI5MODELS-900 internally consisting of two parts:

Acceptance Criteria

It is possible to specify paths containing ../ in sap.ui.model.json.JSONModel#getProperty.
It is possible to specify relative paths with ../ in property bindings when a JSONModel is used.

Unfortunately, I cannot provide any forecast as to when this could get addressed.
Best regards
Mathias.