HTTP PUT being blocked in WordPress is a false positive

Description
It seems like PUT requests trigger the rule "911100: Method is not allowed by policy".
This is an increasingly common type of HTTP request in WordPress plugins
Request:	PUT /lmstest/wp-json/elementor/v1/site-editor/templates-conditions/263
Action Description:	Access denied with code 200 (phase 2).
Justification:	Test 'REQUEST_METHOD' against '!@within %{tx.allowed_methods}' is true.
Seems like this should be allowed using the WordPress exclusion rules
Your Environment
OWASP ModSecurity 2.9 Core Rule Set v3.3.0
Confirmation
[x] I have removed any personal data (email addresses, IP addresses,
passwords, domain names) from any logs posted.

Please see rule ID 900200 in crs-setup.conf - does it helped you?

By the way, have you enabled WordPress exclusion package? See rule 900130.

I think you are using the Elementor plugin and we don't have support added for it yet.
If you could post the full error messages (perhaps cleaning your content from it), I can see if I can reproduce it, and add it to our WordPress exclusion package.
We are using Elementor also, it seems to become more popular, but haven't had this FP yet.

Looks like 900200 was already enabled
And yeah WORDPRESS-EXCLUSION-RULES.conf is enabled
I don't have direct shell/command line access so not able to examine crs-setup.conf.. will see if possible to access that and the the log
I believe PUT is a common method with the new WP REST API which is acccessed via the /wp-json URL, but yeah in this case the plugin in question was Elementor

It seems that elementor is trying to PUT to a wp-json url. Elementor is becoming a more and more popular “Site builder” plugin for WordPress that basically allows users to create very good looking complex pages with minimal administrator effort. I would not be against including support for it in the package.
I'm using it ourselves on a few pilot projects now, so I might as well contribute anything I find to CRS. I will work on this and also add bugs I find myself. Thanks for the report.

We'd still be interested to get your logs @firasd.

Note to self, this is also one exclusion I needed for Elementor that I want so include:
SecRule REQUEST_FILENAME "@endsWith /wp-json/elementor/v1/form-submissions" \
    "id:5100001,phase:1,t:none,nolog,pass,\
        ctl:ruleRemoveById=200001"

I have this one in my collection (but cannot provide example of the request):
# WordPress 5.0.6 / 5.2.3 / 5.2.4 / 5.4.4 / 5.7, plugin Elementor 2.6.8, added 5.9.2019, edited 16.4.2021
SecRule REQUEST_FILENAME "@endsWith /wp-admin/admin-ajax.php" \
    "id:9990134,\
    phase:2,\
    pass,\
    t:none,\
    nolog,\
    chain"
    SecRule ARGS:action "@streq elementor_ajax" \
        "t:none,\
        chain"
        SecRule &ARGS:action "@eq 1" \
            "t:none,\
            ctl:ruleRemoveTargetByTag=attack-xss;ARGS:actions,\
            ctl:ruleRemoveTargetById=921130;ARGS:actions,\
            ctl:ruleRemoveTargetById=932115;ARGS:actions,\
            ctl:ruleRemoveTargetById=932150;ARGS:actions,\
            ctl:ruleRemoveTargetById=933160;ARGS:actions,\
            ctl:ruleRemoveTargetById=933180;ARGS:actions,\
            ctl:ruleRemoveTargetById=942230;ARGS:actions,\
            ctl:ruleRemoveTargetById=932110;ARGS:actions,\
            ctl:ruleRemoveTargetById=942190;ARGS:actions,\
            ctl:ruleRemoveTargetById=932100;ARGS:actions,\
            ctl:ruleRemoveTargetById=934100;ARGS:actions,\
            ctl:ruleRemoveTargetById=933210;ARGS:actions,\
            ctl:ruleRemoveTargetById=932140;ARGS:actions,\
            ctl:ruleRemoveTargetById=932105;ARGS:actions,\
            ctl:ruleRemoveTargetById=932130;ARGS:actions"

@azurit Haha, I also have that too. Although, I just did:
ctl:ruleRemoveTargetByTag=OWASP_CRS;ARGS:actions

So what do we do here. Add all this to the WP RE package?

In my opinion we should expand the scope of the WordPress package to bring in often-used plugins.
Plugins that we often use at work are: Advanced Custom Fields, Gravity Forms, WooCommerce, WPML (multi language). We have made proprietary exclusions for them, but they may need some polish to include them in CRS.
A good task for me during the CRS retreat?

I'd just like to underscore that PUT is a method used by WP-JSON (which is the URL for the WP API) in general.
So blocking all HTTP PUT requests (as mod_sec seems to do) is not really an Elementor problem per se, it will be a problem for anyone trying to use the WP API

The WordPress exclusion package could selectively allow PUT just on the necessary endpoints.

I think the expansion makes sense, but I am not sure it's a good task for the retreat. It's something that does not need a focused group of developers (unless we want to give the RE package a huge boost, yet if that's the case, I'd like to see a commitment from the WP community).

Issue covered in the September issue chat.
@lifeforms will do a PR to address the PUT problem. The expansion idea comes afterwards.

The amount of @lifeforms assigned issues is staggering. We need to free you!

@firasd Can you post few examples of URLs for WP json API?

Looks like they're always prefixed with /wp-json/, or (if rewriting URLs is not available) the query string ?rest_route
eg.
http://example.com/wp-json/wp/v2/posts/123
http://example.com/?rest_route=/wp/v2/posts/123
Some docs: https://developer.wordpress.org/rest-api/extending-the-rest-api/routes-and-endpoints/

Interesting segment in that link about the HTTP methods

HTTP methods are sometimes referred to as HTTP verbs. They are simply just different ways to communicate via HTTP. The main ones used by the WordPress REST API are:

GET should be used for retrieving data from the API.
POST should be used for creating new resources (i.e users, posts, taxonomies).
PUT should be used for updating resources.
DELETE should be used for deleting resources.
OPTIONS should be used to provide context about our resources.

It is important to note that these methods are not supported by every client, as they were introduced in HTTP 1.1. Luckily, the API provides a workaround for these unfortunate cases. If you want to delete a resource but can’t send a DELETE request, then you can use the _method parameter or the X-HTTP-Method-Override header in your request. How this works is you will send a POST request to https://ourawesomesite.com/wp-json/my-shop/v1/products/1?_method=DELETE. Now you will have deleted product number 1, even though your client could not send the proper HTTP method in the request, or maybe there was a firewall in place that blocks out DELETE requests.

We are covering this in rules:
9002140
9002141
9002142
9002143
@lifeforms Should we allow PUT in all of them?
By the way, i think regexpes in 9002140 and 9002142 can be optimized by makeing them anchored (not sure about the URL syntax, correct me if it's wrong):
SecRule REQUEST_FILENAME "@rx /wp-json/wp/v[0-9]+/(?:posts|pages)(?:/[0-9]+)?$" \

SecRule REQUEST_FILENAME "@rx /wp-json/wp/v[0-9]+/media(?:/[0-9]+)?$" \

Will match URLs like:
/wp-json/wp/v2/posts
/wp-json/wp/v2/posts/123
/wp-json/wp/v2/pages
/wp-json/wp/v2/pages/123
/wp-json/wp/v2/media
/wp-json/wp/v2/media/123

Code for allowing PUT method:
setvar:'tx.allowed_methods=%{tx.allowed_methods} PUT"

Yeah a regex that matches the /wp-json/ endpoint makes sense.

Sorry for not getting to work on this. The team is currently swamped (see https://coreruleset.org/20220711/update-on-crs-4-0-release-delay/).
We're happy to accept pull requests for this though.

I'm no expert, I don't even know if this is helpful, but no one has commented what the rule packs and line are that are preventing this script, if it's helpful, here it is:
REQUEST-930-APPLICATION-ATTACK-LFI.conf"] [line "105"] [id "930130"]
REQUEST-949-BLOCKING-EVALUATION.conf"] [line "80"] [id "949110"]
---dIW65o7G---H--
ModSecurity: Warning. Matched "Operator `PmFromFile' with parameter `restricted-files.data' against variable `REQUEST_FILENAME' (Value: `/.env' ) [file "/usr/local/lsws/conf/modsec/owasp-modsecurity-crs-3.0-master/rules/REQUEST-930-APPLICATION-ATTACK-LFI.conf"] [line "105"] [id "930130"] [rev ""] [msg "Restricted File Access Attempt"] [data "Matched Data: /.env found within REQUEST_FILENAME: /.env"] [severity "2"] [ver "OWASP_CRS/3.3.2"] [maturity "0"] [accuracy "0"] [tag "application-multi"] [tag "language-multi"] [tag "platform-multi"] [tag "attack-lfi"] [tag "paranoia-level/1"] [tag "OWASP_CRS"] [tag "capec/1000/255/153/126"] [tag "PCI/6.5.4"] [hostname "peoplebusiness.live"] [uri "/.env"] [unique_id "1667052995"] [ref "o0,5v4,5t:utf8toUnicode,t:urlDecodeUni,t:normalizePathWin,t:lowercase"]
ModSecurity: Access denied with code 403 (phase 2). Matched "Operator `Ge' with parameter `5' against variable `TX:ANOMALY_SCORE' (Value: `5' ) [file "/usr/local/lsws/conf/modsec/owasp-modsecurity-crs-3.0-master/rules/REQUEST-949-BLOCKING-EVALUATION.conf"] [line "80"] [id "949110"] [rev ""] [msg "Inbound Anomaly Score Exceeded (Total Score: 5)"] [data ""] [severity "2"] [ver "OWASP_CRS/3.3.2"] [maturity "0"] [accuracy "0"] [tag "application-multi"] [tag "language-multi"] [tag "platform-multi"] [tag "attack-generic"] [hostname "peoplebusiness.live"] [uri "/.env"] [unique_id "1667052995"] [ref ""]

I'm no expert, I don't even know if this is helpful, but no one has commented what the rule packs and line are that are preventing this script, if it's helpful, here it is:
REQUEST-911-METHOD-ENFORCEMENT.conf"] [line "27"] [id "911100"]
REQUEST-949-BLOCKING-EVALUATION.conf"] [line "80"] [id "949110"]
---7xzdJ1yl---H--
ModSecurity: Warning. Matched "Operator `Within' with parameter `GET HEAD POST OPTIONS' against variable `REQUEST_METHOD' (Value: `PUT' ) [file "/usr/local/lsws/conf/modsec/owasp-modsecurity-crs-3.0-master/rules/REQUEST-911-METHOD-ENFORCEMENT.conf"] [line "27"] [id "911100"] [rev ""] [msg "Method is not allowed by policy"] [data "PUT"] [severity "2"] [ver "OWASP_CRS/3.3.2"] [maturity "0"] [accuracy "0"] [tag "application-multi"] [tag "language-multi"] [tag "platform-multi"] [tag "attack-generic"] [tag "paranoia-level/1"] [tag "OWASP_CRS"] [tag "capec/1000/210/272/220/274"] [tag "PCI/12.1"] [hostname "***.com.br"] [uri "/wp-json/elementor/v1/site-editor/templates-conditions/5402"] [unique_id "1667053184"] [ref "v0,3"]
ModSecurity: Access denied with code 403 (phase 2). Matched "Operator `Ge' with parameter `5' against variable `TX:ANOMALY_SCORE' (Value: `5' ) [file "/usr/local/lsws/conf/modsec/owasp-modsecurity-crs-3.0-master/rules/REQUEST-949-BLOCKING-EVALUATION.conf"] [line "80"] [id "949110"] [rev ""] [msg "Inbound Anomaly Score Exceeded (Total Score: 5)"] [data ""] [severity "2"] [ver "OWASP_CRS/3.3.2"] [maturity "0"] [accuracy "0"] [tag "application-multi"] [tag "language-multi"] [tag "platform-multi"] [tag "attack-generic"] [hostname "***.com.br"] [uri "/wp-json/elementor/v1/site-editor/templates-conditions/5402"] [unique_id "1667053184"] [ref ""]

Disabling these 2 packages can be disastrous for security, right?

Hello, is anyone watching this?
Thx

You should simply add PUT to the list of allowed methods in crs-setup.conf. That should make the 911100 alert disappear, until we update the WP rule exclusion package for good.
Making sure 911100 no longer triggers will also result in 949110 no longer blocking the request.

@lifeforms should we add this to our WordPress exclusion plugin?

@theMiddleBlue I'm kind of weary of plugins altering basic settings, BUT we could add to them just on certain paths! Btw, I am sick now, but I have some local additions that I really want to have to WordPress plugin in 4.0. I have planned to do this on Friday. To prevent merge conflicts, could you do this before Friday, or after my PR is merged? It's not so big, just want to sync up the plugin with my local 3.3 fork.

as you wish, if you want I can do this on https://github.com/coreruleset/wordpress-rule-exclusions-plugin/pull/8 or maybe we can wait to merge all PRs and then create a new one to address this

@theMiddleBlue @lifeforms What's next here?

I have now planned to finish that PR on Wednesday, but I'm still a bit unreliable as I'm sick, but I will try. My biggest apologies for being so late lately...

I've made small necessary changes to https://github.com/coreruleset/wordpress-rule-exclusions-plugin/pull/8 and merged it.

Closing this one then. Great work!


bugfix: pass configured params to uart callbacks

callbackParam argument of uartCallbackSet() was being ignored in
uartProcessIRQ().

Gracias Diego!!


fix: rm npx use jsdoc directly

This fixes #8

anything needed to get this merged?

Nice!! Thanks, was just giving this a spin : ).


fix: quote sentry types

/deploy extra-values=dataService.sentry.enabled=true,dataService.sentry.dsn=https://e47c95d665684bce89984bd4595268df@sentry.dev.renku.ch/13,dataService.sentry.environment=ci-renku-3548,dataService.sentry.sampleRate=0.5

You can access the deployment of this PR at https://renku-ci-ds-242.dev.renku.ch

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
12 unchanged lines in 5 files lost coverage.
Overall coverage decreased (-0.1%) to 90.236%

Files with Coverage Reduction
New Missed Lines
%

components/renku_data_services/crc/blueprints.py
2
89.17%

components/renku_data_services/crc/db.py
2
86.59%

components/renku_data_services/storage/models.py
2
84.26%

components/renku_data_services/base_api/pagination.py
3
87.72%

components/renku_data_services/storage/blueprints.py
3
95.7%

Totals

Change from base Build<PHONE_NUMBER>:
-0.1%

Covered Lines:
8447

Relevant Lines:
9361

💛  - Coveralls

Tearing down the temporary RenkuLab deplyoment for this PR.


fix: letting those vars through

Closes #
Describe your proposed changes here.

[ ] CHANGELOG.md updated with a link to the PR (not the Issue)
[ ] Well-formatted commit messages
[ ] Rebased/mergeable
[ ] Tests pass
[ ] http/swagger.yml updated (if modified Go structs or API)
[ ] Documentation updated or issue created (provide link to issue/pr)
[ ] Signed CLA (if not already signed)

real tempted to make a gif of me clicking the close button, but i'm too busy


not recognize string as huge type

What type of PR is this：

[x] bug
[ ] feature
[ ] enhancement
[ ] others

Which issues of this PR fixes ：

Fixes #
Problem Summary(Required) ：

Not recognize string/varchar as huge type, but only hll/bitmap/json.

@Mergifyio backport branch-2.2


Honor $GNUPGHOME if set

I tried scd-pkcs11 and it did not work for me out of the box since I had $GNUPGHOME set to a non-default location.

thanks


chore: release main

:robot: I have created a release beep boop
dev-dependencies-client: 4.1.0
4.1.0 (2024-01-07)
Features

client: adding @testing-library/react-hooks (31c22c2)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/aversini/dev-dependencies/releases/tag/dev-dependencies-client-v4.1.0 :sunflower:


(PUP-5735) Remove requirement of Win32-Security gem from Puppet

The functionality of the Win32-Security gem has been merged into Puppet.
This commit moves the last of enumerations and updates the test suites to
use the newer namespace of Puppet::Util::Windows:SID.  This commit removes
Puppet's requirement of the win32-security gem, however the gem itself
still exists in the ruby gem library.  It will be removed from the library
in later commits.

Aside from the project_data.yaml change, this looks good.


Hammer dissapear if equipped and press right click

When you have equipped hammer and press mouse rigth click, you put a torch and hammer dissapears,
X(

I am actually aware of this, I just haven't had time to fix it. I should put out a PSA about it. Very sorry :(

Check curseforge for an update when it's accepted :)


[WIP] Add ServiceInsight and ServicePulse

Fixes #89

Hmmm, wait a minute. I approved this, but the advanced exercises aren't there yet, are they!? :-)
I suggest we add the documentation, but don't create any "before" or "after" folders. If anything goes wrong, this is an excellent chance to read documentation and get hands-on experience. And they are 'advanced' after all, right?

@dvdstelt

Hmmm, wait a minute. I approved this, but the advanced exercises aren't there yet, are they!? :-)

Correct, that's why it's marked WIP.

I suggest we add the documentation, but don't create any "before" or "after" folders.

The idea (as I understand it) is to use the existing solutions (1 to 4) and only to add some instructions for advanced exercises that will involve using SP and SI. I don't think we should alter the solutions at all if we can avoid it. You could argue that we should alter the "after" solution to reflect the state after the advanced exercises have been completed, but I would argue that this isn't required. Those who are moving on to the advanced exercises can be left a little more "on their own", without a reference "after" solution to look at for clues.
/cc @mauroservienti

Please review advanced exercise. I added @tmasternak and @SzymonPobiega because they might want to know about it as well, but it is very unlikely this will come up in a 1 day workshop! But you never know, some devs are so extremely fast! ;-)

Aaargh, some files are included that certainly should not have been. Let me try to fix.
For the reviewers, it only concerns the new .md file! :)

Hmmm, I thought I had already replied to this thread.

Module 03 already has advanced exercises.
Module 04 doesn't need them, as this is the advanced exercise
There is no solution to maintain, They should pick this up after they finish 04. Or worst case scenario they pick the after solution of module 04.

I specifically created exercise 05 so people know it's a separate exercise.

It's totally focused on the platform, not so much on the architectural ideas from the modules presented
It's for those who finish all their exercises early. But those attendees who can't finish exercise 04 in time, they won't feel bad about not finishing an exercise.
It's quite large to 'just' include it at the end of module 04.
People can pick it up at home, if they're really eager to experiment with the exercises.
We could even point people not doing the workshop to this exercises, for example via support cases or LiveChat.

Module 01 & 02 could perhaps use some advanced exercises as well. But I was thinking more about doing module 01 to 04 as advanced modules and not include 100% of the code to copy-n-paste.

Discussed with @dvdstelt. He will send another PR showing an alternative approach with SI and SP content in advanced exercise sections spread across the current exercises 2,3, and 4. Marking this PR as WIP while we decided between the two approaches.

Closing in favour of #126


Documentation: Mention that the MQTT adapter is an alternate solution for 'advanced' users

This adapter is useful for 'beginners' but... in reality it is simply starting an MQTT instance, decoding messages and updating specific states as a result.
I think it would be useful to mention in the documentation that if you are already running an instance of the MQTT Adapter and don't mind decoding messages from the OwnTracks topics yourself, then this owntracks adapter is not required.

Especially the decoding of the messagess and provideing special state is the reason for the adapter. SO I do not think that this is a needed info.

I disagree because when I first found the OwnTracks adapter I thought it was mandatory for integration with OwnTracks apps. It was only after further investigation I figured out that OwnTracks could work with the regular IoB MQTT adapter, thus reducing number of adapters in a system if you already have MQTT runnin.
Or maybe I'm just slow? :clown_face: :wink:
One needs to script something to pass locations from OwnTracks states to Places anyhow (unless I missed something there too) so if using MQTT adapter it's not a big deal to decode yourself like this:
function ownTracksMQTTToPlaces(mqttStateId) {
    // Pull out username from ID... depending on how the client was configured.
    const idParts = mqttStateId.split('.');
    const user = idParts[idParts.length - 2];

    const owntracksMessage = JSON.parse(getState(mqttStateId).val);

    const placesMsg = {
        user: user,
        latitude: owntracksMessage.lat,
        longitude: owntracksMessage.lon,
        timestamp: owntracksMessage.tst
    };
    sendTo('places', placesMsg);
}

None of this is a big deal. It's here in this issue now so someone might stumble across it here :smile:

Then do a PR with the proposed Readme change if you like ...
Adapters are especially there to add conveniece for unexperienced users in several such cases ... if we add a disclaimer like "hey you can also do it yourself, just do xyz if you are a profi" that feels wrong ;)

That's OK :wink:
I guess that if this adapter really is for convenience though, it should integrate into places (when installed) as that would be one less script a new user would need to figure out. But that's not what this issue is actually about.


Issue #389 - Add support for x-authentication-scheme and Bearer authentication scheme in apiKey security definition

Does not beautify/minify/use min.js version.
Changes proposed in this pull request:
Added support for x-authentication-scheme in ApiKeyAuthorization object
Added x-authentication-scheme field in Authorize dialog
Automatically adds 'Bearer ' prefix to Authorization header in case x-authentication-scheme is 'Bearer'

Coverage remained the same at 100.0% when pulling 2159ffca9982b8cf2f352eea96f0a2c45a72b292 on advance512:temporary_fix into 8e10a77b1ac2414506ea8cc5f32d25bbf7239aa1 on zalando:master.

Coverage remained the same at 100.0% when pulling 2159ffca9982b8cf2f352eea96f0a2c45a72b292 on advance512:temporary_fix into 8e10a77b1ac2414506ea8cc5f32d25bbf7239aa1 on zalando:master.

Coverage remained the same at 100.0% when pulling 2159ffca9982b8cf2f352eea96f0a2c45a72b292 on advance512:temporary_fix into 8e10a77b1ac2414506ea8cc5f32d25bbf7239aa1 on zalando:master.

Related issue #406

Can we please merge this in?


fix: ensure tracking is paused when emit() calls handler so it can safely be called in effects (fix #6669)

fix: #6669
If emit() is called in an effect (i.e. the callback of a watchEffect), it will collect any reactive values accessed in that parent handle as dependencies of the current component.
This can lead to a recursive loop crashing the app.

I feel this PR is necessary though ##6669 is closed.

@yyx990803 You closed #6669 as "not a bug", so I assume my PR can be closed?

Just in case this does get merged, I think the same change would need to be applied to the onceHandler a bit further down.

As for whether it should be merged, I'm very much on the fence.
I think it's an easy problem for people to fall into without really understanding what's going on.
The exact timing of operations in Vue is not generally well understood by users. Various things are sensitive to timing and it usually 'just works' without needing to think about it.
I don't think it's necessarily obvious that emit runs code synchronously in the parent. I've encountered a lot of people who seem to think it's asynchronous. Why? Because the child emits an update event and the child's props don't synchronously update. They conclude that it's because the 'message' (event) is delivered asynchronously to the parent.
Even if someone isn't thinking about it in quite those terms, I still think events are perceived as somehow different from normal function calls, like throwing a parcel over a wall and not having to worry about what's on the other side.
But, I can also see Evan's perspective for closing the original issue.


Drawer-Image not shown on android

What could be the reason, that the drawer-image not appear on Android while it is visible on iOS?
This is the code:
<ReduxRouter tintColor='white'>
          <Scene
            key='homeView'
            drawer={true}
            drawerImage={require('myapp/src/img/menu.png')}
            contentComponent={MenuScene}
            >

Working well on iOS. But not on Android. Any ideas?

What version do you use? We had some issues before, but it should be fixed with beta.12 I believe

This sounds great. I use 4.0.0-beta.11.
Will upgrade and test again and give feedback

4.0.0-beta.12 fixes this. Thank you!


[Fixit] Authentication and Credentials

Pull Request Template
Motivation

Improve authentication filter code / packages structure
Remove Credentials interface and Data field from types.Credentials
Basic auth filter uses repository native query (list with username field query) instead of specialized one that was coming from Credentials interface

Pull Request status

[x] Initial implementation
[x] Refactoring
[x] Unit tests

TODO
Change base to master before merging

Coverage increased (+0.02%) to 89.095% when pulling c20c21e853693f13e9e54183d6a165a8ce301de0 on fixit-authnn into 78e57d1132823091252f5171b1d34c64c4d1965f on fixit-storage/encrypt.


Identify on-demand and live streams

Pull request
Description
This PR implements stream type detection for on-demand and live streams.
Changes made

The unit test framework has been expanded to make publisher testing more expressive and powerful.
Unit test helpers are provided in a new Circumpspect product.
The demo has been updated with video and audio livesream examples.
Streams are generated and served locally for test purposes.

Checklist

[ ] Your branch has been rebased onto the main branch.
[ ] APIs have been properly documented (if relevant).
[ ] The documentation has been updated (if relevant).
[ ] New unit tests have been written (if relevant).
[ ] The demo has been updated (if relevant).
[ ] All pull request status checks pass.

🌙 Available in TestFlight nightlies
The most recent state can be tested in the following builds:

Platform
Version

iOS
-

tvOS
1.0.0 (34)


[feature request] Multiple OAuth token

It would be great to have access to multiple Dropbox folders in these two ways:

different Apps in the same Dropbox Account
different Apps in different Dropbox Accounts

I'm asking this because the token is saved in a global configuration file: I was wondering if it's possibile to have different local and global configurations as happens for .gitconfig

Someone started on implementing something like this - see #55

Wow, really cool!
I'll follow the development.
Thanks!

Well, I don't think it's being worked on anymore by the original author, but if you'd like to pick up where that person left off (or just implement a separate solution and send a PR), that would be cool

Actually I don't think to have the skills for doing that :)
I was just looking at the code and trying to understand how it works: I don't understand how the username associated to the token is generated.

Ok, no worries!
Even submitting this issue has been helpful -- now I know that there are multiple people who want this feature 😄

That's fine ;)
I'm sorry that isubas is no more developing it.

I'd also like to have multi-account support.
I have personal projects that I want to store in my personal dropbox, and work projects that I want to store in the team's business dropbox.
To me it would make sense if each dropbox oauth token could be assigned an id, and the URLs would look like dropbox://id/path/to/project.git.
I have no Python skills myself, but if the aforementioned PR is quite simple and just needs a few fixes, would probably be able to do that with your suggestions.

Okay, I implemented multi-account support in the develop branch: https://github.com/anishathalye/git-remote-dropbox/tree/develop
Anyone wanna try it out and give me feedback?

Hey, thanks a lot!
I've tried it out and it seems to work (mostly) fine.
I now have two usernames with tokens in my config file.
I've updated the work project to use the dropbox business token, and was able to pull from that remote.
I've created a new test project on one computer, commited a few files, set remote to the personal dropbox, then pushed.
Saw this output:
C:\project\git-remote-dropbox\grd-test # git push -u origin master
Writing objects: 100% (8/8), done.
To dropbox:///git/project/grd-test.git
 * [new branch]      master -> master
Branch master set up to track remote branch master from origin.

Notice that To dropbox:///git/project/grd-test.git doesn't mention the username, the actual url is<EMAIL_ADDRESS>Then, on another computer:
$ git clone<EMAIL_ADDRESS>Cloning into 'grd-test'...
Receiving objects: 100% (8/8), done.
warning: remote HEAD refers to nonexistent ref, unable to checkout.

I'm not sure why there is a warning about the remote HEAD?
The checked out grd-test wasn't on any branch, and was empty (besides .git), but I was able to git checkout master.
P.S. Is it a feature that I'm able to
git clone<EMAIL_ADDRESS>(which is cloned as an empty repo)?
It looks like a feature, but it's not explicitly mentioned in the README.

Cool! See #1 for more info about the warning about remote HEAD.
Yeah, it's a feature that you can clone a nonexistent repo. It's not super useful, so it's not documented anywhere.
Yeah, we can't count objects up front -- we're basically following the history pointers back, and so there's no way to tell how many objects we're going to need to download.

Yeah I already found #1 .
Got it. So, it appears the new feature is fully fine :)
One question - once you release this new version, what's gonna happen with the repositories using dropbox://path/to/repo.git URL format of the newer version? Will they need their remote URLs updated to the new syntax? Or would git-remote-dropbox somehow automatically update them?

Yeah, I think it's best to require the user to update to the newer syntax.
The new version does tell the user: error: URLs with no username or token must start with "dropbox:/// if they're using the old scheme.

Cool. Well, thanks again for implementing this :)

Thanks a lot for implementing this :))))


Trying git download. Won't finish the second step.

I am trying to download the bot from the git link in command prompt because pip doesn't always work. It has installed the directories but the second step "pip install ./ChatterBot" will not work whatsoever. I don't really know what else to try. I just want the bot.
Copy of error message:
ERROR: Command errored out with exit status 1:
command: 'C:\Python34\python.exe' -c 'import sys, setuptools, tokenize; sys.argv[0] = '"'"'C:\Users\name removed\AppData\Local\Temp\pip-install-poheayil\nltk\setup.py'"'"'; file='"'"'C:\Users\name removed\AppData\Local\Temp\pip-install-poheayil\nltk\setup.py'"'"';f=getattr(tokenize, '"'"'open'"'"', open)(file);code=f.read().replace('"'"'\r\n'"'"', '"'"'\n'"'"');f.close();exec(compile(code, file, '"'"'exec'"'"'))' egg_info --egg-base pip-egg-info
cwd: C:\Users*name removed*\AppData\Local\Temp\pip-install-poheayil\nltk
Complete output (1 lines):
error in nltk setup command: 'install_requires' must be a string or list of strings containing valid project/version requirement specifiers
----------------------------------------
ERROR: Command errored out with exit status 1: python setup.py egg_info Check the logs for full command output.

instead, downloading from git, could you please try pip3 install chatterbot


Slow websockets when amount requested through a single connection is substantially large

Version
2.0.7
Platform
MacOS Venture 13.3.1
Description
Using a singular websocket connection, if a large number of requests are made, all requests are only completed after a long time.
A total of 40,000 requests to will only finish after 90 seconds.
I tried this code:

Clone this repo: https://github.com/syahrul12345/ethers-websockets-tests
Change .example.env to .env and provide your http and ws endpoints. (I am using reth)
Execute an individual benchmark by using this command

//Do with 5 iterations, each with 50 websocket requests
cargo run -- --outer 5 --iterations 50
//Replay with more requests
cargo run -- --outer 1 --iterations 40000

Logs:
------ BENCHMARK  ------
Iteration index: 0
Requests done this iteration : 40000
Total time taken: 98.080538458s

You seem to be measuring performance while compiling in debug mode. Try adding the --release flag for much better performance?

Im using the release flags, but it really does nothing. Im also compiling using native cpu flags. Added some additional logs in my repo.
For example, a single websocket connect with 1000 requests takes almost 20 seconds to complete!
Time taken for websocket client index 0 request index 986 is 40.720993583s
Time taken for websocket client index 0 request index 970 is 40.721093416s
Time taken for websocket client index 0 request index 991 is 40.721256917s
Time taken for websocket client index 0 request index 898 is 40.72104225s
Time taken for websocket client index 0 request index 897 is 40.904411292s
Time taken for websocket client index 0 request index 993 is 40.948703334s
Time taken for websocket client index 0 request index 984 is 40.957841708s
Time taken for websocket client index 0 request index 988 is 40.957901209s
Time taken for websocket client index 0 request index 900 is 40.957975833s
Time taken for websocket client index 0 request index 989 is 40.957957125s

Command i use:
make build && ./target/release/async-eth --outer 1 --iterations 1000


feat(block-producer): fix dangling review comments

This PR

fixes several unaddressed review comments from #515
distributes batch and block proving times randomly
injects random failures into batch and block proving

Specifically, addresses this, this and this comment.
Proving times are distributed normally from within a configurable range.
Failure rates are also similarly configurable, which completes one task from #519.

Question: currently BatchProducer and BlockProducer structs are unused, right?

Correct; one of the next PRs would replace the current production producers with these ones.


ONNX export of UPerNet

Feature request
Add support for exporting UPerNet to ONNX.
Motivation
openmmlab/upernet-convnext-small is 4th most downloaded model for image segmentation.
It is licensed under MIT license which makes it even more interesting.
Your contribution
/

Thank you, would you like to try and work on it? I will add it to my todo otherwise.

I won't have time to work on it in the following weeks.

Hi @fxmarty Can I work on this ?

Hi @sanjeevk-os, for sure, feel free to ping me when you submit a PR!

Hi @fxmarty : After making the changes I tried to test the onnx export but it gives me this error:
torch.onnx.errors.SymbolicValueError: Unsupported: ONNX export of operator adaptive_avg_pool2d, input size not accessible. Please feel free to request support or submit a pull request on PyTorch GitHub: https://github.com/pytorch/pytorch/issues  [Caused by the value 'input.168 defined in (%input.168 : Float(*, 768, *, *, strides=[3072, 4, 2, 1], requires_grad=0, device=cpu) = onnx::Add(%1701, %1703), scope: transformers.models.upernet.modeling_upernet.UperNetForSemanticSegmentation::/transformers.models.convnext.modeling_convnext.ConvNextBackbone::backbone/transformers.models.convnext.modeling_convnext.ConvNextLayerNorm::stage4
Looks like onnx doesn't support the operator adaptive_avg_pool2d yet. I tried with the swin backbone but ran into the same issue. Any suggestions ?


New version: CameraCalibrations v0.2.1

Registering package: CameraCalibrations
Repository: https://github.com/yakir12/CameraCalibrations
Created by: @yakir12
Version: v0.2.1
Commit: 0084dd58dd2cf7a729f96ea1de2091afb042a334
Git reference: master

Same story, depends on MATLAB.jl that depends on Matlab, hence CI is broken. Appreciate a manual merge. Thanks!


Fails to detect binary content

Hi, I tested the CSV parser against non-CSV files. It successfully 'parses' a JPG file, for example, yielding nonsensical binary results.
Is this something you plan to remedy, or can you advise on ways to mitigate this problem?

No sorry, this lib isn't in the business of file detection. Also please don't remove issue templates.


"mount: special device /var/run/shill does not exist"

Hi,
I'm having some problems getting crouton up and running on my Samsung Series 5 550. The install process stops abruptly after
Installing unmount-chroot into the host...
mount: special device /var/run/shill does not exist
Unmounting /usr/local/chroots/precise...

When I try to startxfce4 after this, I get the prompt described here - "A chroot preparation script still exists inside the chroot", etc. I've tried deleting the chroot and re-running the script, but have the same problem - the process doesn't get to the prompt stage at all. I've pasted the full log below, cutting out the long list of retrieved/validated/extracted packages - any help would be much appreciated!
chronos@localhost /var/run $ sudo sh -e ~/Downloads/crouton -t xfce
Installing precise-amd64 chroot to /usr/local/chroots/precise
Remounting /tmp with dev+exec...
Downloading debootstrap...
Downloading bootstrap files...
I: Retrieving InRelease
I: Failed to retrieve InRelease
I: Retrieving Release
W: Cannot check Release signature; keyring file not available /usr/share/keyrings/ubuntu-archive-keyring.gpg
I: Retrieving Packages
I: Validating Packages
I: Resolving dependencies of required packages...
I: Resolving dependencies of base packages...
I: Checking component main on http://archive.ubuntu.com/ubuntu...

[...]

Moving bootstrap files into the chroot...
Preparing chroot environment...
Installing brightness into the chroot...
Installing host-dbus into the chroot...
Installing croutonpowerd into the chroot...
Installing xinit into the chroot...
Installing startxfce4 into the host...
Installing enter-chroot into the host...
Installing delete-chroot into the host...
Installing mount-chroot into the host...
Installing unmount-chroot into the host...
mount: special device /var/run/shill does not exist
Unmounting /usr/local/chroots/precise...

I realise that this is a very old thread, but I am having the same issue on my Pixelbook. I am definitely running the latest version of the OS (Version 63.0.3239.140 (Official Build) (64-bit))
I entered dev mode and then immediately downloaded Crouton from https://goo.gl/fd3zc  . I then ran the following from a terminal window
shell
sudo sh ~/Downloads/crouton -t xfce
Install whirred away for a minute or so, but I then received the message
mount: special device /var/run/shill does not exist
Unmounting /mnt/stateful_partition/crouton/chroots/xenial...
I have tried other targets with the same result. Any pointers or advice would be much appreciated. This conversation seems to be the only reference to the same issue that I can find.

@sbastable,
That is strange, maybe something else is interfering with crouton.
I'd suggest you reboot and try again, if you run into the same issue we'll try to gather more info. to troubleshoot with.
Hope this helps,
-DennisLfromGA


Return the chrome extension?

So it's really nice you don't need the chrome extension anymore but it was kind of awesome.

No need to go to the site, can start directly from the tab youre on
No annoying Chrome UI on video tab or other tabs

I run a really old fork, but it would be nice to not need too.
Thanks for the original and this

The biggest reason for switching to a website frankly was legality. There is nothing inherently illegal about 2n.fm, but it obviously can be abused. We didn't want to have to worry about the extension app store having the power to just remove us from existence due to false copyright flags aimed at bad users rather than us.
But there were other technical reasons why jumping to web made sense:

It's possible to even do now. Streaming a screen via just a web browser wasn't possible not long ago.
No extension install goes along with our philosophy of anyone being able to use it. We currently have no user logins, anyone can quickly make a room for the first time, or the hundredth.
Cross browser support, Firefox & Safari can now stream.
 Thanks Google! Please help get it changed with feedback here: https://bugs.chromium.org/p/chromium/issues/detail?id=1010610
The Chrome extension was neat, but it relied on some proprietary Chrome API calls, which seem to be on the way out as getDisplayMedia() replaces them.

2n.fm's code has also come a looooong way from the extension version as well, and is still getting improvements. Hopefully the website over time will grow into what you like about the extension.
Some of the annoyances regarding selecting what to stream are also already implemented in Chrome, but seemingly unavailable to developers outside of Google. The Meet website for example is able to have a custom UI asking about tab vs desktop sharing, while we appear to be stuck with the pop-up built into Chrome. We can't request to only prompt the user for Tab sharing, etc. unless there is some documentation I'm missing. It's also outside of the W3C spec to allow said custom UI for that, but honestly seems like the right direction to go, W3C needs to change that.
Finally as a personal note about using the old extension version, that should be all fine and good, however if you experience any issues with connecting to other users, it will be due to a lack of TURN servers. On the real 2n.fm, we are currently generously providing the TURN servers for free to users, so connections are more guaranteed. We're not a company or anything, just some friendly devs willing to eat the monthly cost for now, so while this probably isn't feasible into the future forever, enjoy it for now.

@codysherman

The biggest reason for switching to a website frankly was legality. There is nothing inherently illegal about 2n.fm, but it obviously can be abused. We didn't want to have to worry about the extension app store having the power to just remove us from existence due to false copyright flags aimed at bad users rather than us.

This is fair, but that's entirely on the end user, 2n.fm isn't actually hosting anything or transmitting any of the data (when using p2p connections). I think you're pretty sound there. As for getting pulled from the chrome store, I think it could happen, it might just be something to deal with there's always the fallback of manually installing the extension from an XPI/CHR package or zip of source. I think end users can handle that well.

Re: 2. No extension install goes along with our philosophy of anyone being able to use it. We currently have no user logins, anyone can quickly make a room for the first time, or the hundredth.

I don't think having an extension negative effects that, its just an additional route to use

Re: 4.  Thanks Google!

Re: 5. The Chrome extension was neat, but it relied on some proprietary Chrome API calls, which seem to be on the way out as getDisplayMedia() replaces them.

Reasonable but I don't think there going away anytime soon as they have no migration path for extension yet, there's usually a very long period before they actually deprecate and remove a feature.

Re: W3C specs

Yeah, I wish this stuff was spec'd

We're not a company or anything, just some friendly devs

Me too, I hope I'm friendly. Thanks for the feedback. I'll just keep running my own services and fork. 👍
Thanks for the original and current code. It's been a great way to keep in touch with my girlfriend a country away.


Removes hacky code

I'm honestly not sure why, but the code for removing duplicates is no longer necessary.
I think this has to do with RubaXa's Sortable library upgrading, but in any case keeping this code causes items to be disappear when moved.

You didn't need to update Rubaxa's sortable for it?


Add floating tag

Feature description
Hello would it be possible to create a v1 tag for instance and move it at each v1.x.x release ?
Like for action/checkout@v3 and many other actions
Dependabot opens a PR at every release of regex-match ^^
Thank you

@AzHicham
regex-match already has v1 branch. so you can use MeilCli/regex-match@v1


Reorganize protobuf files in analyzer

It would be nice if the protobuf files were in a separate directory from the application files.  This requires a few tweaks to the current code base, but is not very difficult to do.  The general steps are that you need to make the generated protobufs use relative imports and then change any reference to the protobufs to import mydir.myproto_pb2.  It would make the repo a lot cleaner and easier to work with for new people looking to contribute to the project.

Thanks @dhpollack
This is in our backlog. If you're interested in proposing a change in a PR, that would be great

@omri374 do you have a preference on the name of the directory?  I've actually implemented this on a local copy of presidio where I used protobuf_models as the directory name.  If that's fine then I can probably create a PR for this relatively quickly.

@dhapollack Thanks! protobul_models sounds good.


Sleuth's transparent use of RabbitMQ to deliver traces when it's added as a dependency can be confusing

I had a Spring Boot app handling web requests and logging tracing to Zipkin using spring-cloud-starter-zipkin.
I added RabbitMQ messaging to the app using spring-boot-starter-amqp. Noticed later that I was no longer getting tracing reported from this app to Zipkin.
It was surprising to me that adding spring-rabbit to the dependencies automatically switched Sleuth's functionality. Though it is relatively well documented, I had long since forgotten that part of the Sleuth docs when I added Rabbit messaging to the app.
It's possible that I'm an outlier. Maybe the vast majority of developers using Rabbit for app communications also want to use it to deliver tracing telemetry. My assumption was that most developers would be delivering data to Zipkin using HTTP, and so adding RabbitMQ dependencies to my app wouldn't change how Sleuth works unless I explicitly asked Sleuth to use Rabbit instead.
Perhaps, if there's no obvious default to pick when Rabbit or Kafka are in the classpath, Sleuth should fail the container startup with a message requesting an explicit choice to be made?

Related Twitter conversation:
https://twitter.com/evolvable/status/1165885545362288640

one option is to require opt-in for rabbit such as we do with kafka https://github.com/spring-cloud/spring-cloud-sleuth/pull/1013

The opt in makes a lot of sense. I'd go with that one for hoxton


SplitDiscovery & websocketDataSync & integration CURD

Make sure that:

[ ] You have read the contribution guidelines.
[ ] You submit test cases (unit or integration tests) that back your changes.
[ ] Your local test passed ./mvnw clean install -Dmaven.javadoc.skip=true.

use websocket to sync data && use zookeeper to discovery mode

fail to create connection to 9878 port

then i create -e /shenyu/discovery/tcp/1/1/1 {"id":"123456","discoveryHandlerId":"1","protocol":"http","url":"<IP_ADDRESS>:6379","status":1,"weight":10,"props":"{}"}
ShenyuGateway find log

then success create connection to 9878 port

Codecov Report

Merging #4768 (99260b1) into master (0fe9858) will decrease coverage by 0.18%.
The diff coverage is 23.98%.

@@             Coverage Diff              @@
##             master    #4768      +/-   ##
============================================
- Coverage     65.81%   65.64%   -0.18%
- Complexity     8135     8139       +4
============================================
  Files          1131     1133       +2
  Lines         32975    33127     +152
  Branches       2960     2962       +2
============================================
+ Hits          21702    21745      +43
- Misses         9544     9647     +103
- Partials       1729     1735       +6

Impacted Files
Coverage Δ

...scovery/DiscoveryDataChangedEventSyncListener.java
0.00% <0.00%> (ø)

...enyu/admin/discovery/DiscoveryProcessorHolder.java
44.44% <0.00%> (-12.70%)
:arrow_down:

...henyu/admin/discovery/LocalDiscoveryProcessor.java
16.00% <0.00%> (+16.00%)
:arrow_up:

...discovery/parse/CustomDiscoveryUpstreamParser.java
0.00% <0.00%> (ø)

...dmin/listener/AbstractListDataChangedListener.java
77.91% <0.00%> (-0.33%)
:arrow_down:

...dmin/listener/AbstractNodeDataChangedListener.java
72.52% <0.00%> (-8.96%)
:arrow_down:

...nyu/admin/listener/DataChangedEventDispatcher.java
76.92% <0.00%> (-6.42%)
:arrow_down:

...che/shenyu/admin/listener/DataChangedListener.java
0.00% <0.00%> (ø)

...stener/websocket/WebsocketDataChangedListener.java
72.41% <0.00%> (-27.59%)
:arrow_down:

...he/shenyu/admin/model/dto/ProxySelectorAddDTO.java
20.00% <0.00%> (-1.06%)
:arrow_down:

... and 22 more

... and 50 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


fix docstring for _find_errorhandler

Drop  #1462 for this one. Maybe it  is much more clear now.

Excellent, thanks!
BTW you can use git commit --amend and git push -f to update your current pull request.

Thanks for your advice.


EnterpriseDesktopAppManagement - Get Commands Return Empty Responses

Hi,
I have been trying the EnterpriseDesktopAppManagement CSP and able to install the MSI remotely. However querying for Status, InstallPath and Uninstall commands are not working.
The Get queries are returning empty responses and Delete command though returns success the app actually does not get uninstalled.
Any clues on what may be happening here?
Thanks,
Sriram

@sriram-kakarala . Which article are you referring ?

The issues section of this repository is intended for documentation and content-related issues.
Please open a support ticket via the link below so that that your question gets answered quickly.
Windows 10 Support
Thank you.


Lambda capture types aren't inferred correctly

This issue comes from a short discussion with @sylvanc.
I have a method fun _index_for(id: SeqId): USize ?.
I add a lambda capture such as
{
        ()(_seq_id_to_incoming, index, id): Bool ? =>
        _seq_id_to_incoming(index)._1 <= id
}

where _seq_id_to_incoming is Array[(SeqId, (Producer tag, U64 val, U64 val))] ref.
The method no longer compiles.
I get an error...
Array[(U64 val, (Producer tag, U64 val, U64 val))] box is not a subtype of Array[(U64 val, (Producer tag, U64 val, U64 val))] ref: box is not a subtype of ref

This can be addressed by specifying a type for the capture such as:
{()(x: Array[(SeqId, (Producer tag, U64 val, U64 val))] box = _seq_id_to_incoming) Bool ? => ... }

However, in the end, the capture should use the type in the current context (val, box or ref) and not require the programmer to specify the type (which is redundant info at that point).

As discussed on the sync call, in expr/lambda, for fields that get captured, we need to use the type of how this sees the field.  Right now it's not making a distinction between locals and fields.

@jemc Will #2168 fix this?

@Praetonus No, #2168 has not touched captures at all because they aren't part of the antecedent for the lambda. This would be a separate unit of work.


[engsys] upgrade dev dependency puppeteer to ^21.0.0

also did some clean up on packages that don't test browsers.

API change check
API changes are not detected in this pull request.


Remove category feature

server side
create CategoryController.cs  file
create remove category API in CategoryController.cs
client side
create removeCategory(id) method in questions-dashboard.component.ts
create category.service.ts file
call removecategory(id) method from questions-dashboard.html page

SonarQube analysis reported 5 issues

 5 major

Watch the comments in this conversation to review them.


Update readme for dot net version

I tried DotNet SDK 5.0 didn't work due to custom command requires DotNet core 2.1 to build.
Purpose

Refine demo readme. Explicitly requrie dot net core 2.1 as SDK.
I tried .net 5.0 SDK offline and it didn't work since the CC requires .net core 2.1 to build
Does this introduce a breaking change?

[ ] Yes
[x ] No

Pull Request Type
What kind of change does this Pull Request introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[x ] Documentation content changes
[ ] Other... Please describe:

How to Test

Get the code

git clone [repo-address]
cd [repo-name]
git checkout [branch-name]
npm install

Test the code

What to Check
Verify that the following are valid
Other Information

@jiamguo what is it that fails to build and what error do you get? can you give more details?

I ran into the issue when running deployAll.ps1 -> deployAzureFunction.ps1.
This script will build azure function sln and it requires .net core 2.1
Initially I installed .net 5.0 (per this readme suggested) and it gave me the error saying I need .net core 2.1 to build solution.
After I installed .net core 2.1 this error is gone.
If there's a need of the original error message I will have to uninstall .net core 2.1 and repro.

Okay got it. Thanks.
.NET core 2.1 will no longer be supported soon, so it would be best to move to 3.1. I think that's the version required to build Windows Voice Assistant Client. So if the developers wants to do both (deploy CC demos) and build the client, they only need once .NET core version.
https://dotnet.microsoft.com/download/dotnet/


Clang format?

I recommend using clang format and picking a style so I don't have to worry about making whitespace changes.

Sounds good, I did that initially but got some weird formatting here and there, so I disabled it. But you are right, a style is needed

Can you pick a style, it doesn't matter the particulars, but you are the maintainer.

Clang with default settings is fine.

Put a .clang-format file in the git root
and
# We'll use defaults from the LLVM style, but with 4 columns indentation.
BasedOnStyle: LLVM
IndentWidth: 4

I'll make a pr with clang-format -i * in src/

Fixed by #84


Unable to filter DECIMAL column from ORC file in c++

This question is similar to THIS one I asked before on StackOverflow, which after some more trials it works.
Previously there was some issue with the column Id but now I am trying to filter a column of DECIMAL data type but always results give me all the data instead of the filtered one.

Data which ORC file has in the required columns:

And this is how I am trying to filter out the DECIMAL column using orc::SearchArgument:
orc::RowReaderOptions m_RowReaderOpts;
orc::ReaderOptions m_ReaderOpts;

std::unique_ptr<orc::Reader> m_Reader;
std::unique_ptr<orc::RowReader> m_RowReader;

auto builder = orc::SearchArgumentFactory::newBuilder();
const int snapshot_time_col_id = 22;

orc::Literal ss_begin_time{34080000000000, 14, 9};
orc::Literal ss_end_time{34380000000000, 14, 9};

// I HAVE ALSO TRIED, but didn't work.
// orc::Literal ss_begin_time{34080, 5, 0};
// orc::Literal ss_end_time{34380, 5, 0};

builder->between(snapshot_time_col_id, orc::PredicateDataType::DECIMAL, ss_begin_time, ss_end_time);

m_RowReaderOpts.searchArgument(builder->build());
reader = orc::createReader(orc::readFile(a_FilePath.c_str()), m_ReaderOpts);
row_reader = reader->createRowReader(m_RowReaderOpts);

Please give some suggestions on how to filter data of type DECIMAL?

cc @wgtmac and @stiga-huang

Any update on this bug?

Could you verify if the whole batch returned by row_reader->next() violates the SearchArgument? If so, there are bugs. Otherwise, it's by design.
orc::SearchArgument is used as an indicator for the reader to skip unrelated RowGroups, i.e. it's only evaluated on RowGroup level (not row-level). If the reader can't filter out a RowGroup based on the SearchArgument, it will return all rows of that RowGroup. The caller is expected to filter out rows by itself.


API changes report for Nd4j Api snapshot

Hi,
I'd like to share report on API changes and backward compatibility for the Nd4j library: https://abi-laboratory.pro/java/tracker/timeline/nd4j-api/
BC — binary compatibility
SC — source compatibility
The report is generated by the https://github.com/lvc/japi-tracker tool according to the article https://wiki.eclipse.org/Evolving_Java-based_APIs_2. Hope it will be helpful for users and maintainers of the library.
Feel free to request other Nd4j modules to be included to the tracker if you are interested.
Thank you.

Thanks!


Trouble building a project with bindgen/cmake

I'm trying to build a project that depends on the high-sys crate. Using crane I get the below error:
--- stderr
  CMake Error: The current CMakeCache.txt directory /build/source/target/release/build/highs-sys-7473b6736d4fd4df/out/build/CMakeCache.txt is different than the directory /build/dummy-src/target/release/build/highs-sys-7473b6736d4fd4df/out/build where CMakeCache.txt was created. This may result in binaries being created in the wrong place. If you are not sure, reedit the CMakeCache.txt
  thread 'main' panicked at '
  command did not execute successfully, got: exit status: 1

From the error message it looks like /build/dummy-src/target/release/build/highs-sys-7473b6736d4fd4df/ is copied to /build/source/target/release/build/highs-sys-7473b6736d4fd4df and rebuilt? Based on the error message the out/build directory should be excluded in the copy?

Hi @forficate thanks for the report!
Looks like cmake expects the path where it is being built to not change, but the dummy sources are unpacked to /build/dummy-src/ instead of /build/source/. Should be fixed with #190


[BEAM-6688] Spark portable runner: translate flatten

Pretty much the same as the Spark legacy runner's flatten. Required for PAssert among other things.

Run Python PreCommit

Run Java PreCommit

Run Java PreCommit

Run Java PreCommit


Moved grouped expressions to its own page

and added another example of its use.
Should I remove this bit This operator cannot be overloaded.? It is not really an operator.

LGTM. I'm hesitant to merge because I'm not sure why @matthewjasper didn't earlier.

@Havvy I didn't merge to give you a chance to give feedback if you wanted.


list only std wallets in deployments

Changes

list only std wallets in deployments

Related Issues

https://github.com/threefoldtech/js-sdk/issues/2059

Checklist

[ ] Pre-commit hook is installed to do formatting checks before committing code...etc
[ ] Tests included
[ ] Build pass
[ ] Documentation
[ ] Code format and docstrings

Codecov Report

Merging #2072 (97285e8) into development (895f47a) will increase coverage by 0.00%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##           development    #2072   +/-   ##
============================================
  Coverage        32.45%   32.46%
============================================
  Files              136      136
  Lines            13198    13196    -2
============================================
  Hits              4284     4284
+ Misses            8914     8912    -2

Impacted Files
Coverage Δ

.../sals/reservation_chatflow/reservation_chatflow.py
9.29% <0.00%> (+0.02%)
: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 895f47a...97285e8. Read the comment docs.


Update keptn plugin deps

Overview
In order, to solve this security issue: https://github.com/kubeshop/botkube/security/dependabot/21, we need to update keptn plugin deps. At the time of writing this issue (18.12.23) it is still not fixed on the upstream: https://github.com/keptn/go-utils/blob/master/go.mod#L16
Acceptance Criteria

[ ] Double check if we don't use the code with the security issue, related to the otel pkg
[ ] Test e2e that keptn plugin is still working properly

The affected code isn't used by our plugin, so closing the issue.


added contains existing paths

Hi,
I'd like to understand if something is the normal behavior of this module or if this is a bug.
Getting a diff using something like this:
getDiff({ foo: { bar: true } }, { foo: {} })

Returns this as a result:
{
  added: [ [ 'foo', {} ] ],
  removed: [ [ 'foo/bar', true ] ],
  edited: []
}

While I understand the removed part, why would it consider foo to be added?
Thanks!

@Marsup you're right, I'm looking for a way to solve the problem! It's possible that a break change makes sense.

I think that solves https://github.com/lukascivil/json-difference/pull/40

Hey @Marsup, could you perform some operations here http://jsondifference.lukascivil.com.br/ and validate if everything is OK?
The code uploaded in this link is the master.


Paginated search results only return 1 page

Since updating to 1.2.3 (the issue is also present in 1.2.2. ), all searches using the following code only return 1 page.
Downgrading to 1.2.1 is necessary for the code to work correctly.
$courses = Course::_search_($this->searchString, function (Indexes $meilisearch, $query, $options) {
        $options\['filters'\] = 'active=true';
        return $meilisearch->search($query, $options);
    })->paginate(12);

Hello! That's my fault, it's because I removed the usage of nbHits in the code, which is not a trustable value since it's not an exhaustive value. nbHits is indeed not reliable for pagination because can be exhaustive or not, depending on the value of exhaustiveNbHits that MeiliSearch returns which is always false for the moment. So, it worked by chance before my PR.
However my PR makes things worst because not well implemented. I don't have the time to fix it so I'm going to revert my PR, and I will fix it when I have time. Sorry for this.

FYI: The Meili team is aware of the confusion with nbHis. Here are the different issues and comments about it to explain why it's confusing, and why we should not use it:

https://github.com/meilisearch/MeiliSearch/issues/1120
https://github.com/meilisearch/documentation/issues/561
https://github.com/meilisearch/meilisearch-php/issues/119#issuecomment-732908568

We are sorry for this. We all hope this confusion will be fixed asap in MeiliSearch.

The v0.12.4 should fix this!


Failed to build

I built svsm on debian 10, I ran FW_FILE=/mnt/vms/svsm/OVMF.fd make, But it failed with
cargo build --manifest-path kernel/Cargo.toml  --no-default-features --bin stage2
   Compiling svsm v0.1.0 (/mnt/vms/svsm/svsm/kernel)
error: linking with `ld` failed: exit status: 1
  |
  = note: LC_ALL="C" PATH="/root/.rustup/toolchains/stable-x86_64-unknown-linux-gnu/lib/rustlib/x86_64-unknown-linux-gnu/bin:/root/.rustup/toolchains/stable-x86_64-unknown-linux-gnu/lib/rustlib/x86_64-unknown-linux-gnu/bin:/root/.rustup/toolchains/stable-x86_64-unknown-linux-gnu/lib/rustlib/x86_64-unknown-linux-gnu/bin:/root/.vscode-server/cli/servers/Stable-f1e16e1e6214d7c44d078b1f0607b2388f29d729/server/bin/remote-cli:/root/.cargo/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin" VSLANG="1033" "ld" "/tmp/rustcniGnQI/symbols.o" "/mnt/vms/svsm/svsm/target/x86_64-unknown-none/debug/deps/stage2-11b4a21f595374bf.03z0smhqx7953ey7np20ex2nt.rcgu.o" "/mnt/vms/svsm/svsm/target/x86_64-unknown-none/debug/deps/stage2-11b4a21f595374bf.07xh1js39tjko9r32y2zsv2w5.rcgu.o" "/mnt/vms/svsm/svsm/target/x86_64-unknown-none/debug/deps/stage2-11b4a21f595374bf.10g3yy3ftl569c0ynv0mbwnsl.rcgu.o" "/mnt/vms/svsm/svsm/target/x86_64-unknown-none/debug/deps/stage2-11b4a21f595374bf.1u0u9bmb4909ce0jqtyj1oeja.rcgu.o" "/mnt/vms/svsm/svsm/target/x86_64-unknown-none/debug/deps/stage2-11b4a21f595374bf.1wlsh3dkcbn0hr50xvpwi5brv.rcgu.o" "/mnt/vms/svsm/svsm/target/x86_64-unknown-none/debug/deps/stage2-11b4a21f595374bf.20sdzucxentvwbopxbxguoyob.rcgu.o" "/mnt/vms/svsm/svsm/target/x86_64-unknown-none/debug/deps/stage2-11b4a21f595374bf.34acifddo6346p9hysj3d1gt9.rcgu.o" "/mnt/vms/svsm/svsm/target/x86_64-unknown-none/debug/deps/stage2-11b4a21f595374bf.3tdgk30bl6iu3dbpimrgb4t2g.rcgu.o" "/mnt/vms/svsm/svsm/target/x86_64-unknown-none/debug/deps/stage2-11b4a21f595374bf.3u3iq6kz5edlx5lkt6few4d3w.rcgu.o" "/mnt/vms/svsm/svsm/target/x86_64-unknown-none/debug/deps/stage2-11b4a21f595374bf.4ygyt7sn5sm4aflzljemgih5v.rcgu.o" "/mnt/vms/svsm/svsm/target/x86_64-unknown-none/debug/deps/stage2-11b4a21f595374bf.586jmwytkef3r4r30dto9gkgn.rcgu.o" "/mnt/vms/svsm/svsm/target/x86_64-unknown-none/debug/deps/stage2-11b4a21f595374bf.5ke4iiv7hwp4vca4pkxjqrg5f.rcgu.o" "/mnt/vms/svsm/svsm/target/x86_64-unknown-none/debug/deps/stage2-11b4a21f595374bf.61887kp3nwz7wiel0fa7f33zj.rcgu.o" "/mnt/vms/svsm/svsm/target/x86_64-unknown-none/debug/deps/stage2-11b4a21f595374bf.6w9f3nlzwskoovz4q3gci4gkt.rcgu.o" "/mnt/vms/svsm/svsm/target/x86_64-unknown-none/debug/deps/stage2-11b4a21f595374bf.70m0cdcc6h90zaa006322q7he.rcgu.o" "/mnt/vms/svsm/svsm/target/x86_64-unknown-none/debug/deps/stage2-11b4a21f595374bf.74zcotinzzdboyjlovep0cgw2.rcgu.o" "/mnt/vms/svsm/svsm/target/x86_64-unknown-none/debug/deps/stage2-11b4a21f595374bf.8kn1h7kp9bpqnfrx0a7825o43.rcgu.o" "/mnt/vms/svsm/svsm/target/x86_64-unknown-none/debug/deps/stage2-11b4a21f595374bf.90fqonyduzefhf23n0a4kw4od.rcgu.o" "/mnt/vms/svsm/svsm/target/x86_64-unknown-none/debug/deps/stage2-11b4a21f595374bf.92a9xxdgoq0c781on5mybsk9s.rcgu.o" "/mnt/vms/svsm/svsm/target/x86_64-unknown-none/debug/deps/stage2-11b4a21f595374bf.9dfxa0e0fl05dev7769yq934h.rcgu.o" "/mnt/vms/svsm/svsm/target/x86_64-unknown-none/debug/deps/stage2-11b4a21f595374bf.akhf5znlw1mcfs1bwgws8g5x9.rcgu.o" "/mnt/vms/svsm/svsm/target/x86_64-unknown-none/debug/deps/stage2-11b4a21f595374bf.aq1tg6ee69h3qaksw2p6dju0a.rcgu.o" "/mnt/vms/svsm/svsm/target/x86_64-unknown-none/debug/deps/stage2-11b4a21f595374bf.b3oguhlj2gy1tal92beez2x97.rcgu.o" "/mnt/vms/svsm/svsm/target/x86_64-unknown-none/debug/deps/stage2-11b4a21f595374bf.c48fv8fpu9244mnob71xg1ekm.rcgu.o" "/mnt/vms/svsm/svsm/target/x86_64-unknown-none/debug/deps/stage2-11b4a21f595374bf.cjduy6nmsc54xogy185tqbxhf.rcgu.o" "/mnt/vms/svsm/svsm/target/x86_64-unknown-none/debug/deps/stage2-11b4a21f595374bf.dj1nwqtsceqlme8i4a1wvkjyj.rcgu.o" "/mnt/vms/svsm/svsm/target/x86_64-unknown-none/debug/deps/stage2-11b4a21f595374bf.8sfr8qmozqe10e8tp82ld7uiu.rcgu.o" "--as-needed" "-L" "/mnt/vms/svsm/svsm/target/x86_64-unknown-none/debug/deps" "-L" "/mnt/vms/svsm/svsm/target/debug/deps" "-L" "/root/.rustup/toolchains/stable-x86_64-unknown-linux-gnu/lib/rustlib/x86_64-unknown-none/lib" "-Bstatic" "/mnt/vms/svsm/svsm/target/x86_64-unknown-none/debug/deps/libsvsm-5b30e934503a5dee.rlib" "/mnt/vms/svsm/svsm/target/x86_64-unknown-none/debug/deps/liblog-a2f9882d2ce4c373.rlib" "/mnt/vms/svsm/svsm/target/x86_64-unknown-none/debug/deps/libintrusive_collections-9b17b14ba2ff3025.rlib" "/mnt/vms/svsm/svsm/target/x86_64-unknown-none/debug/deps/libmemoffset-b82a6236ff0ed312.rlib" "/mnt/vms/svsm/svsm/target/x86_64-unknown-none/debug/deps/libigvm_defs-a7ddda439fbef1b6.rlib" "/mnt/vms/svsm/svsm/target/x86_64-unknown-none/debug/deps/libstatic_assertions-036bed9dcbfae41d.rlib" "/mnt/vms/svsm/svsm/target/x86_64-unknown-none/debug/deps/libopen_enum-4a953cd9e6580210.rlib" "/mnt/vms/svsm/svsm/target/x86_64-unknown-none/debug/deps/libpackit-cd8de6b81172ab9d.rlib" "/mnt/vms/svsm/svsm/target/x86_64-unknown-none/debug/deps/libaes_gcm-2b6185d1d5024b5d.rlib" "/mnt/vms/svsm/svsm/target/x86_64-unknown-none/debug/deps/libctr-b74a3d3a6c32a489.rlib" "/mnt/vms/svsm/svsm/target/x86_64-unknown-none/debug/deps/libghash-d778da7274c8e0ed.rlib" "/mnt/vms/svsm/svsm/target/x86_64-unknown-none/debug/deps/libpolyval-9f936b34e96865ae.rlib" "/mnt/vms/svsm/svsm/target/x86_64-unknown-none/debug/deps/libopaque_debug-190716a268b034b7.rlib" "/mnt/vms/svsm/svsm/target/x86_64-unknown-none/debug/deps/libuniversal_hash-c68bb9711942ba12.rlib" "/mnt/vms/svsm/svsm/target/x86_64-unknown-none/debug/deps/libsubtle-f0f067b32ba00d9e.rlib" "/mnt/vms/svsm/svsm/target/x86_64-unknown-none/debug/deps/libaes-e42814311d39d84b.rlib" "/mnt/vms/svsm/svsm/target/x86_64-unknown-none/debug/deps/libcfg_if-9e48aaafacc0e56c.rlib" "/mnt/vms/svsm/svsm/target/x86_64-unknown-none/debug/deps/libcipher-4f89a93e92350f87.rlib" "/mnt/vms/svsm/svsm/target/x86_64-unknown-none/debug/deps/libinout-8cb423104d701afa.rlib" "/mnt/vms/svsm/svsm/target/x86_64-unknown-none/debug/deps/libaead-b82b48f699045709.rlib" "/mnt/vms/svsm/svsm/target/x86_64-unknown-none/debug/deps/libcrypto_common-28aa4a31921381be.rlib" "/mnt/vms/svsm/svsm/target/x86_64-unknown-none/debug/deps/libgeneric_array-f765a5c14b71b9ff.rlib" "/mnt/vms/svsm/svsm/target/x86_64-unknown-none/debug/deps/libtypenum-7ad35d73d7b8bd48.rlib" "/mnt/vms/svsm/svsm/target/x86_64-unknown-none/debug/deps/libsyscall-214413448125d9e0.rlib" "/mnt/vms/svsm/svsm/target/x86_64-unknown-none/debug/deps/libelf-2b564260710a3e01.rlib" "/mnt/vms/svsm/svsm/target/x86_64-unknown-none/debug/deps/libbitflags-4148f588adc7d9ae.rlib" "/root/.rustup/toolchains/stable-x86_64-unknown-linux-gnu/lib/rustlib/x86_64-unknown-none/lib/liballoc-ba3395486792f1bd.rlib" "/mnt/vms/svsm/svsm/target/x86_64-unknown-none/debug/deps/libcpuarch-0e85548d4e1e626a.rlib" "/mnt/vms/svsm/svsm/target/x86_64-unknown-none/debug/deps/libbootlib-e8c807fe0e206b81.rlib" "/mnt/vms/svsm/svsm/target/x86_64-unknown-none/debug/deps/libzerocopy-f5b34341cf70b2aa.rlib" "/mnt/vms/svsm/svsm/target/x86_64-unknown-none/debug/deps/libbyteorder-fb705309a12f505d.rlib" "/root/.rustup/toolchains/stable-x86_64-unknown-linux-gnu/lib/rustlib/x86_64-unknown-none/lib/librustc_std_workspace_core-97a7c162531d87a2.rlib" "/root/.rustup/toolchains/stable-x86_64-unknown-linux-gnu/lib/rustlib/x86_64-unknown-none/lib/libcore-55e9655321e6c9a7.rlib" "/root/.rustup/toolchains/stable-x86_64-unknown-linux-gnu/lib/rustlib/x86_64-unknown-none/lib/libcompiler_builtins-d16d97fd584a51e9.rlib" "-Bdynamic" "--eh-frame-hdr" "-z" "noexecstack" "-L" "/root/.rustup/toolchains/stable-x86_64-unknown-linux-gnu/lib/rustlib/x86_64-unknown-none/lib" "-o" "/mnt/vms/svsm/svsm/target/x86_64-unknown-none/debug/deps/stage2-11b4a21f595374bf" "--gc-sections" "-pie" "-z" "relro" "-z" "now" "-nostdlib" "--build-id=none" "-Tkernel/src/stage2.lds" "-no-pie"
  = note: ld: /mnt/vms/svsm/svsm/target/x86_64-unknown-none/debug/deps/stage2-11b4a21f595374bf.586jmwytkef3r4r30dto9gkgn.rcgu.o: relocation R_X86_64_32 against `.data' can not be used when making a PIE object; recompile with -fPIC
          ld: final link failed: nonrepresentable section on output

error: could not compile `svsm` (bin "stage2") due to 1 previous error
make: *** [Makefile:115: bin/stage2.bin] Error 101

Make sure you install all these tools and also upgrade Ubuntu Os version to 24.04 and then compile.
curl --proto '=https' --tlsv1.2 -sSf https://sh.rustup.rs | sh -s -- -y
sudo apt update
sudo apt install binutils
sudo apt update
sudo apt install -y
build-essential
curl
git
libclang-dev
autoconf
autoconf-archive
pkg-config
automake
libssl-dev
perl
libc6-dev
gcc-multilib
make
gcc


How to get "undistorted_depths" from scannet++?

hi,
I'm trying to reproduce your impressive work on scannet++ but I notice that "undistorted_depths" is needed in your dataset.py.
Following the official documents of scannet++, I can only get "undistorted_images" and "transforms_undistorted.json".
So, can you explain to me how to get "undistorted_depths" in detail?
Thanks for your help!

Hi, we used the script here from SplaTAM authors for undistorted_depths.

thanks for your quick response! I got "undistorted_depths" finally. It's really helpful!


Allow override <code... from highlighter

Hightlightters need better flexibility for the wrappers. For example, set <pre class=hljs>. Can we add such feature but keep backward compatibility?
Suggestion:
Check highlighter output, and if it starts with <pre, skip wrapper. Any problems with this?

Done in https://github.com/markdown-it/markdown-it/commit/87ece24ff997502418e74fffc3ad78173af4a1d1#diff-de043c47b7a14d2d383d220efbaa9c2eR48


WiFiMQTTSecureAWS example  is not compiling for MKR1010

Example WiFiMQTTSecureAWS is getting this error: Compilation error: Error: 2 UNKNOWN: exit status 1
I'm not pretty sure but I think could be WiFiClientSecure library, I couldn't find at Library Manager on arduino manager for IDE 2.0.0-beta.11.
I read that WiFiClientSecure is for ESP devices, so I'm not sure...
Could you confirm please?
I'm using a MKR1010 device

@kathesama Hi, thank you for reporting.
First, this library does not support IDE 2.0 yet, so could you try the example in IDE 1.8.16?
As you know, the examples are made only for ESP series, so you need to change the WiFi initialization codes before line 59 depending on your platform (WiFiNINA, WiFi101, etc.).
https://github.com/hideakitai/MQTTPubSubClient/blob/eb98df3e6e499973ac7c51d51ddfe6c99b292685/examples/WiFiMQTTSecureAWS/WiFiMQTTSecureAWS.ino#L59
I'm not familiar with the secure clients of WiFININA and other WiFi libraries, so I'm so happy to let me know how to use them to connect securely!

I'm closing this issue, but please feel free to reopen it if you still have problems.


Interface commands

Interface typedefs should be protected?

Actually I think they should be set to "private" since they are called inside the overridable methods (getStrain and setStrain shown above)

I don't like where this is going:

These "interfaces" will quickly become what I didn't want them to be: API prototypes. But maybe I should run with this and see where it goes...


Refactor: break up main function and file

Attempt to wrangle the main function.
Priorities:

Avoid global state - favor arguments and return values that help readers understand where data is coming from
Avoid long functions with sections named with comments
Avoid code paths with many branching paths - favor writing linear code with reused parts (eg. splitting new/review card audio creation)
Try to group similar functions together into files/modules

Todo:

[ ] Code formatting now that we have longer function signatures
[ ] Think about grouping the four decks that are circulating around with a wrapper class
[ ] Think about creating a "CardSequencer" or "LessonPlanner" that manages the complexity of picking the next card for a lesson

Stretch goal: Separate lesson planning and audio generation

First, figure out which terms go in which lessons
Second, generate audio

Personally, I think that separation of roles would be a lot easier to think about.

Please squash the commits.

Too far out of date! If we want this we can start over lol


Support custom profiles

Closes: https://github.com/TimNN/cargo-lipo/issues/55

Thank you! Looks reasonable from a quick glance, but I'll take a closer look on the weekend. Feel free to ping me if you haven't heard back in 7 days.

Released as 3.3.1, thanks again for the high-quality pull request!


feature: Markdown field auto bullet

This pull request adds support for auto-bullet and auto-numbering lists in the Markdown field. That means if you've started an unordered list (for example) and hit enter, the field will automatically add in a new list item for you.
If you don't want that list item, you can just hit Enter again and it will be removed. This will also work for the -  syntax for unordered list.
Ordered lists will behave in the same way, the only difference being the numbers will increment on each new line item.

I'm aware of how these nested if/elses look, but there's not really any other idiomatic way of doing it (or I'm being a fool). :(


iOS compilation error - failed to emit precompiled header Runner-Bridging-Header.h

Steps to Reproduce

I have encountered some critical problems when I am developing my apps and I thought there may be some configuration violation so I start a whole new flutter and run it.
However, I cannot successfully run my new project too.

Target Platform: iOS
Target OS version/browser: iOS 13
Devices: iPhone XS
Logs

Launching lib/main.dart on David’s IPhone in debug mode...
Automatically signing iOS for device deployment using specified development team in Xcode project: XMV49G669D
Xcode build done.                                           32.0s
Failed to build iOS app
Error output from Xcode build:
↳
    ** BUILD FAILED **
Xcode's output:
↳
    /Users/David_Lau/flutter/testing/ios/Runner/Runner-Bridging-Header.h:1:9: note: in file included from /Users/David_Lau/flutter/testing/ios/Runner/Runner-Bridging-Header.h:1:
    #import "GeneratedPluginRegistrant.h"
            ^
    /Users/David_Lau/flutter/testing/ios/Runner/GeneratedPluginRegistrant.h:8:9: error: 'Flutter/Flutter.h' file not found
    #import <Flutter/Flutter.h>
            ^
    1 error generated.
    <unknown>:0: error: failed to emit precompiled header '/Users/David_Lau/Library/Developer/Xcode/DerivedData/Runner-azesybqhtijmtfcugdshfkybgtmu/Build/Intermediates.noindex/PrecompiledHeaders/Runner-Bridging-Header-swift_XFD3EM62ZHT9-clang_3SE6OY8X33GBC.pch' for bridging header '/Users/David_Lau/flutter/testing/ios/Runner/Runner-Bridging-Header.h'
    <unknown>:0: error: generate-pch command failed with exit code 1 (use -v to see invocation)
    <unknown>:0: error: generate-pch command failed with exit code 1 (use -v to see invocation)
    /Users/David_Lau/flutter/testing/ios/Runner/Runner-Bridging-Header.h:1:9: note: in file included from /Users/David_Lau/flutter/testing/ios/Runner/Runner-Bridging-Header.h:1:
    #import "GeneratedPluginRegistrant.h"
            ^
    /Users/David_Lau/flutter/testing/ios/Runner/GeneratedPluginRegistrant.h:8:9: error: 'Flutter/Flutter.h' file not found
    #import <Flutter/Flutter.h>
            ^
    1 error generated.
    <unknown>:0: error: failed to emit precompiled header '/Users/David_Lau/Library/Developer/Xcode/DerivedData/Runner-azesybqhtijmtfcugdshfkybgtmu/Build/Intermediates.noindex/PrecompiledHeaders/Runner-Bridging-Header-swift_AE5E56KLAZ7Q-clang_29NJ7Q52SDGMJ.pch' for bridging header '/Users/David_Lau/flutter/testing/ios/Runner/Runner-Bridging-Header.h'
    note: Using new build systemnote: Planning buildnote: Constructing build description
Could not build the precompiled application for the device.

Error launching application on David’s IPhone.
Exited (sigterm)

[✓] Flutter (Channel stable, v1.9.1+hotfix.4, on Mac OS X 10.14.4 18E227, locale zh-Hant-HK)
    • Flutter version 1.9.1+hotfix.4 at /Users/David_Lau/Desktop/flutter
    • Framework revision cc949a8e8b (13 days ago), 2019-09-27 15:04:59 -0700
    • Engine revision b863200c37
    • Dart version 2.5.0

[✓] Android toolchain - develop for Android devices (Android SDK version 28.0.3)
    • Android SDK at /Users/David_Lau/Library/Android/sdk
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-28, build-tools 28.0.3
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1248-b01)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 11.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 11.1, Build version 11A1027
    • CocoaPods version 1.7.5

[✓] Android Studio (version 3.3)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 34.0.1
    • Dart plugin version 182.5215
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1248-b01)

[✓] VS Code (version 1.39.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.5.1

[✓] Connected device (1 available)
    • David’s IPhone • xxxxxxxxxxxxxxxxxxxx • ios • iOS 13.1.2

• No issues found!

Please give advice since I cannot even start my code!!

Can you give us the exact reproduction steps you used to create the new app?  What are the contents of ios/Podfile?

Sorry for the late reply.
Well, it is simply starting a new project in VS Code by the command Flutter: New Project.
Then run it on either iOS simulator or a physical device. It fails to compile and gives the above error.
It does not include any Podfile inside.

If really need, I run pod init in /ios
my Podfile has the following content:
# Uncomment the next line to define a global platform for your project
# platform :ios, '9.0'

target 'Runner' do
  # Comment the next line if you don't want to use dynamic frameworks
  use_frameworks!

  # Pods for Runner

end

I've just hit this too trying to run a test app I use for Dart-Code integration tests.
Launching lib/main.dart on iPhone 11 Pro Max in debug mode...
Running Xcode build...
 ├─Assembling Flutter resources...                           7.5s
 └─Compiling, linking and signing...                         0.2s
Xcode build done.                                            9.3s
Failed to build iOS app
Error output from Xcode build:
↳
    ** BUILD FAILED **

Xcode's output:
↳
    /tmp/flutter_repo/flutter_hello_world/flutter_repro_42498/ios/Runner/Runner-Bridging-Header.h:1:9: error: 'GeneratedPluginRegistrant.h' file not found
    #import "GeneratedPluginRegistrant.h"
            ^
    1 error generated.
    <unknown>:0: error: failed to emit precompiled header
    '/Users/dantup/Library/Developer/Xcode/DerivedData/Runner-fjycbzowbqgdnhcakadthzamzgvw/Build/Intermediates.noindex/PrecompiledHeaders/Runner-Bridging-He
    ader-swift_1NUF51Y4NUKU8-clang_740PO05NPUJ1.pch' for bridging header
    '/tmp/flutter_repo/flutter_hello_world/flutter_repro_42498/ios/Runner/Runner-Bridging-Header.h'
    <unknown>:0: error: generate-pch command failed with exit code 1 (use -v to see invocation)
    note: Using new build systemnote: Planning buildnote: Constructing build descriptionwarning: Capabilities for Signing & Capabilities may not function
    correctly because its entitlements use a placeholder team ID. To resolve this, select a development team in the Runner editor. (in target 'Runner' from
    project 'Runner')

Could not build the application for the simulator.
Error launching application on iPhone 11 Pro Max.

As far as I can tell, there is no Podfile file. I've committed the whole folder here if it helps:
https://github.com/DanTup/flutter_repro_42498

I tried deleting build, ios, .dart_tool and then running flutter create . but it's still failing with the same error.
So I tried flutter create foo && cd foo && flutter run and that worked fine.
So I compared the two folders, and noticed I had an example folder and the new project did not. Deleted that, and it built fine. I'm not sure exactly what in the example folder might be causing it though.

https://github.com/DanTup/flutter_repro_42498 looks like it's for a plugin, which is why it has an example directory?  But there's no my_package checked in.
pubspec.yaml:
  my_package:
    path: ../my_package

@DanTup What exact command did you use to generate that project?  And were you inside another flutter app when you did?

If really need, I run pod init in /ios

@davidsx Don't run pod init manually, you should delete that generated Podfile.  If you don't have any plugins you won't have a Podfile--that's expected.

But there's no my_package checked in

Whoops, you should be able to just delete that - I think I removed the code for that (to simplify the repro) but must have forgotten the pubspec entry.
The reason it has an example folder is that I added it deliberately for testing Dart-Code supporting nested-projects (this project was just a simple hello_world that's grown a little over time for Dart-Code's integration tests).
Is it valid to have an example folder in an app? It did used to work (these aren't new tests/projects), but I don't know whether it was intended to be supported.

Hi @davidsx
if the issue persist with the latest stable version of flutter
could you please provide your updated flutter doctor -v ,
your flutter run --verbose
and if possible a reproducible minimal code sample.
Thank you

Without additional information, we are unfortunately not sure how to resolve this issue. We are therefore reluctantly going to close this bug for now. Please don't hesitate to comment on the bug if you have any more information for us; we will reopen it right away!
Thanks for your contribution.


New streams feature support

Do you have a plan to support Rabbit MQ new feature Streams  ? Any deadline ?

I do not see .NET client for it. Before it releases - probably not, but it would be perfect candidate for MT Rider


Bug 2078901 - raise alerts for what will be removed on 1.25

OCP 4.11 should be raising the alerts when removed apis on 1.25 are found.
Closes BZ: https://bugzilla.redhat.com/show_bug.cgi?id=2078901

/assign @sanchezl

/bugzilla refresh

Closing in favor of  "https://github.com/openshift/cluster-kube-apiserver-operator/pull/1332


support --normalize option

close #37

Codecov Report

Merging #39 into master will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff           @@
##           master      #39   +/-   ##
=======================================
  Coverage   90.58%   90.58%
=======================================
  Files           7        7
  Lines         425      425
  Branches       56       56
=======================================
  Hits          385      385
  Misses         18       18
  Partials       22       22

Impacted Files
Coverage Δ

src/commands/pull.ts
84.44% <100%> (ø)
:arrow_up:

src/commands/push.ts
90.16% <100%> (ø)
: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 bdd83d3...90e6e27. Read the comment docs.


Allow other ways to separate words in controllers and end points

Currently you separate words with a hyphen. Eg getUserNameAction would be user-name. However, not all systems use this as the way to separate words. For example, PHP's urlencode() function uses +. Browsers will use %20.
This could be added to Controller::parseActionName(). This should be completed after #2 as it will add a parser for child controllers.

Fixed by 43d2600dca6b17bffeec604ead2f04d4a59bd11a


Updating files

I have updated following things:

Reading from the text file
Saving it in a folder
and after that I am trying to use threading on the urls present in the text file so that we can save images in multiple threads at the same time

Hey Swapnil: I have created a separate branch under your name. Kindly PL to KWOC: swapnil branch

Actually it will be more helpful if you can give the link of the branch as i am unable to find that

https://github.com/adityasingh11/O.C.M.I.D/tree/KWOC-swapnil


Trivial updates to log messages

These correct a couple of logging errors as well as adding some useful debug logging.

Previously the index of the enrollment was logged whicj may not be the
actual index of the node. To be always accurate the utxo is logged.

In which situation ?


[TASK] Import DB button crashes with querySelector not finding element

Estimated story points: 1
Description

Acceptance criteria

time: 0.1h


Use of multiple crypto libraries

On dev branch, requirements.txt currently lists pycryptodome as a dependency for AES encryption. Though, when running powerhub.py, it errors out with ModuleNotFoundError: No module named 'Cryptodome'.
This is because the package pycryptodome actually makes available a module named Crypto. The module Cryptodome you use here is instead contained in a package named pycryptodomex. So the entry in requirements.txt should probably be fixed.
I also noticed, that PowerHub makes use of pyOpenSSL for key and certificate handling.
The project page of pyOpenSSL states:

Note: The Python Cryptographic Authority strongly suggests the use of pyca/cryptography where possible. If you are using pyOpenSSL for anything other than making a TLS connection you should move to cryptography and drop your pyOpenSSL dependency.

That cryptography package is also already present, currently as a dependency of service_identity.
So PowerHub currently has pycryptodome/pycryptodomex, pyOpenSSL and cryptography installed. I bet one of them is sufficient (probably cryptography) but I haven't looked into the details.

twisted requires pyOpenSSL


Rebranding it as MapReproduce? ;)

I know it's kind of arrogant to be some random person hinting for renaming a repo... but I still think it is a good idea. (At PyData Warsaw I chuckled a few times. Yet, 3 days later I still think it is worth doing.)
As it is essentially a functional (map, filter, kind-of-reduce)  approach to evolutionary algorithms, MapReproduce would capture its spirit in its name.
(No, I don't mean renaming any parts of API, at least not for this reason.)

I think it fits the attitude of the library perfectly. @rogiervandergeer, how much do we prefer breed?

So here's a thought.
There may be a use-case for adding .reproduce as a verb: as a sugar for asexual reproduction.
def generate(parent):
    return parent + noise()

# the next two lines could be equivalent
pop.reproduce(make_child)
pop.breed(parent_picker = lambda pop: random.choice(pop), combiner = generate)

So wouldn't mind adding a reproduce method, als long as we find a clear purpose.
I would find it rather counterintuitive when reproduce is only asexual

Thinking out loud. It feels safe to say that most evol algorithms will have two parents that are picked based on fitness_score. The probability of a parent being chosen is individual.fitness / sum[i.fitness for i in population]. As far as a base setting goes. I think this is fine and this is one less thing you need to set.
Would this be good enough as an alternative for breed?
The thing is... breed as a word really suggests that we are being very selective about which parents can breed together. This is how the ye old farmers cross bred sheep. Whereas the word reproduce suggests that the population of animals can figure out who the parents are themselves.
I'm starting to like this verb, mainly because this makes it easier for beginners to get started and for many applications you don't need to have a custom method of selecting parents.

I like the verb too. So you are suggesting reproduce will be a breed but with a default parent picker? Then why don't we provide the parent picker as a default argument to breed?

You make a valid point there.
Something tells me that having a .reproduce may make it easier for beginners. It feels intuitive to the functional programming people [map-reproduce] and it is one less internal thing to worry about for the computer science students. For me it feels appropriate to have a sugar method for the usual case and a customisable method for the people who feel like doing crazy things [like 3 parents].


Error when opening repl

Without any updates of changes, this following started happening every time I try to open a Clojure repl. I am not sure why it is looking for bootstrap.js.
Invalid behavior: :lt.objs.sidebar.command/exec-command
Error: No protocol method IDeref.-deref defined for type null:
    at Error (<anonymous>)
    at cljs.core.missing_protocol (file:///opt/homebrew-cask/Caskroom/lighttable/0.7.2/LightTable/LightTable.app/Contents/Resources/app.nw/core/node_modules/lighttable/bootstrap.js:1460:10)
    at cljs.core._deref (file:///opt/homebrew-cask/Caskroom/lighttable/0.7.2/LightTable/LightTable.app/Contents/Resources/app.nw/core/node_modules/lighttable/bootstrap.js:2181:38)
    at cljs.core.deref (file:///opt/homebrew-cask/Caskroom/lighttable/0.7.2/LightTable/LightTable.app/Contents/Resources/app.nw/core/node_modules/lighttable/bootstrap.js:12879:27)
    at Function.eval (/opt/homebrew-cask/Caskroom/lighttable/0.7.2/LightTable/LightTable.app/Contents/Resources/app.nw/plugins/clojure/clojure_compiled.js:1601:544)
    at b (file:///opt/homebrew-cask/Caskroom/lighttable/0.7.2/LightTable/LightTable.app/Contents/Resources/app.nw/core/node_modules/lighttable/bootstrap.js:6188:14)
    at a (file:///opt/homebrew-cask/Caskroom/lighttable/0.7.2/LightTable/LightTable.app/Contents/Resources/app.nw/core/node_modules/lighttable/bootstrap.js:6234:18)
    at a (file:///opt/homebrew-cask/Caskroom/lighttable/0.7.2/LightTable/LightTable.app/Contents/Resources/app.nw/core/node_modules/lighttable/bootstrap.js:21134:29)
    at b (file:///opt/homebrew-cask/Caskroom/lighttable/0.7.2/LightTable/LightTable.app/Contents/Resources/app.nw/core/node_modules/lighttable/bootstrap.js:21138:14)
    at cljs.core.apply_to (file:///opt/homebrew-cask/Caskroom/lighttable/0.7.2/LightTable/LightTable.app/Contents/Resources/app.nw/core/node_modules/lighttable/bootstrap.js:6097:85)

@conchis bootstrap.js is the core Light Table code (compiled from the ClojureScript source into JavaScript). What version of LT are you running? What OS are you running? Please provide steps and code to reproduce this.

@kenny-evitt I got the same issue, LT 32, Linux Mint 17. I was following this tutorial: http://docs.lighttable.com/ , in the second step, when opening the REPL the error happened.
Sorry >.<

@conchis I'll re-open if this is still an un-resolved issue.

I have the same issue with a clean install (downloaded via lighttable.com, looks like the current version is lighttable-0.8.0-alpha-mac) on Mac OS X El Capitan:
Error: No protocol method IDeref.-deref defined for type null:
    at Error (native)
    at cljs.core.missing_protocol (file:///Applications/LightTable.app/Contents/Resources/app/core/node_modules/lighttable/bootstrap.js:1460:77)
    at cljs.core._deref (file:///Applications/LightTable.app/Contents/Resources/app/core/node_modules/lighttable/bootstrap.js:2181:38)
    at cljs.core.deref (file:///Applications/LightTable.app/Contents/Resources/app/core/node_modules/lighttable/bootstrap.js:12879:27)
    at Function.eval (/Applications/LightTable.app/Contents/Resources/app/plugins/Clojure/clojure_compiled.js:1675:544)
    at b (file:///Applications/LightTable.app/Contents/Resources/app/core/node_modules/lighttable/bootstrap.js:6188:14)
    at a (file:///Applications/LightTable.app/Contents/Resources/app/core/node_modules/lighttable/bootstrap.js:6234:18)
    at a (file:///Applications/LightTable.app/Contents/Resources/app/core/node_modules/lighttable/bootstrap.js:21155:29)
    at b (file:///Applications/LightTable.app/Contents/Resources/app/core/node_modules/lighttable/bootstrap.js:21159:14)
    at cljs.core.apply_to (file:///Applications/LightTable.app/Contents/Resources/app/core/node_modules/lighttable/bootstrap.js:6097:85)

@dkharlan  Would you mind trying to remove (mv to a bck folder) the Light Table settings folder
should be in ~/Library/Application\ Support/LightTable. Maybe there is something in your settings/behaviors/plugins etc that is causing the problem.

I tried that and still get the same error.

Apologies, this might have been user error -- when I try "Instrarepl: Open a clojure instarepl" it works fine.  The issue is when I try "Instarepl: Make current edtior an instarepl"

Closing as we never got a response from the original author. @dkharlan if you're still having problems with LightTable 0.8.0 and the ClojureInstarepl plugin, feel free to report it at https://github.com/LightTable/ClojureInstarepl/issues

Closing as we never got a response from the original author. @dkharlan if you're still having problems with LightTable 0.8.0 and the ClojureInstarepl plugin, feel free to report it at https://github.com/LightTable/ClojureInstarepl/issues


Fix duplicate definitions with FreeType

These files are included in ftbase.c
Fixes #18820

Cherry picked into 3.0.3


Add canary image promotion jobs

What type of PR is this?
/kind feature
What this PR does / why we need it:
We now promote canary images for bom prereleases
Signed-off-by: Adolfo García Veytia (Puerco<EMAIL_ADDRESS>Which issue(s) this PR fixes:
Special notes for your reviewer:
/assign @cpanato
Does this PR introduce a user-facing change?
We now have image promotion manifests for canary jobs

/hold

/lgtm

Job is merged: https://github.com/kubernetes/test-infra/pull/25850
/hold cancel


[HOWTO] - Autoscroll FlatList to certain position

I have a screen which is basically a FlatList (that renders some blocks) and at the very bottom of the screen I have a Scroll to Top button that just scrolls that FlatList to the y = 0 position. That's working fine.
The thing is that (after coding for tvos for some months -all of these concepts are new to me-) I noticed that focus and scroll are different things, since even after scrolling to the very top of the screen (and the first component is shown) the focus was still on the Button that was pressed at the very bottom.
I wanted to know if there is a way of doing both things (scrolling and focusing at the first element) at the same time?

Also, as a proposal, I would recommend extending the focus documentation since it appears to be the most unknown zone of the tvos based on the issues being created. I would love to contribute in doing this.

Thanks!

If you have a ref to the first element, your button can set the hasTVPreferredFocus prop on the element to force focus to move to that element.

If you have a ref to the first element, your button can set the hasTVPreferredFocus prop on the element to force focus to move to that element.

Sorry, how do I do that? Does the ref.current have a function for dynamically set the hasTVPreferredFocus prop? Or do you mean setting the prop to the Component itself within the return statement?

Sorry, how do I do that? Does the ref.current have a function for dynamically set the hasTVPreferredFocus prop? Or do you mean setting the prop to the Component itself within the return statement?

@ericlifs - You can have some state variable :
this.state = {scrollToTopFocus: false}
then on the list item you want to move the focus to:
<View
   hasTVPreferredFocus={this.state.scrollToTopFocus}
   >
.....
</View>

Then when you scroll the list to index: 0, also do:
this.setState({scrollToTopFocus: true});
Something along these lines should work I think

@ericlifs - You can have some state variable :
this.state = {scrollToTopFocus: false}
then on the list item you want to move the focus to:
<View
   hasTVPreferredFocus={this.state.scrollToTopFocus}
   >
.....
</View>

Then when you scroll the list to index: 0, also do:
this.setState({scrollToTopFocus: true});
Something along these lines should work I think