Remove the deprecation of IT instructions in the ARM ABI

Upstream ARM has undeprecated the use of IT instructions. There's no formal announcement of the matter in itself, but the previous deprecations are being removed from the official docs.
See https://reviews.llvm.org/D108679#2975603 for a discussion on the matter, CC @dpaoliello who was involved in that discussion.

@mstorsjo : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#label:"aq-pr-triaged"

@mstorsjo and cc: @dpaoliello
Do you know if this un-deprecation (is that a word?) applies to previous versions of our armasm tool, and to all ARM processors? Just wondering if there's a set of processors or old assemblers where the deprecation still holds, especially if certain code won't work on Windows on ARM.

Do you know if this un-deprecation (is that a word?) applies to previous versions of our armasm tool, and to all ARM processors? Just wondering if there's a set of processors or old assemblers where the deprecation still holds, especially if certain code won't work on Windows on ARM.

@corob-msft The un-deprecation is a change by ARM - there have been no changes that I know of to any Microsoft tools or OSes.
From the Windows side, these instructions were always supported, and I don't believe that we ever supported any hardware that didn't support the instructions.

@mstorsjo and cc: @dpaoliello
Do you know if this un-deprecation (is that a word?) applies to previous versions of our armasm tool, and to all ARM processors? Just wondering if there's a set of processors or old assemblers where the deprecation still holds, especially if certain code won't work on Windows on ARM.

When it comes to actual processors, there hasn't been any issue so far (as it would break backwards binary compatbility), but I've understood it that they (ARM) reserved the right to degrade/break it later (and thus put in the deprecation preemptively) - but now they've decided that they won't do that, so there's no longer any reason to avoid those constructs.
Yeah ideally armasm would also stop warning about these constructs - I guess I could file a public bug report about it, but I'm not sure if it's as effective as if someone of you would start it.

From the Windows side, these instructions were always supported, and I don't believe that we ever supported any hardware that didn't support the instructions.

Yeah I don't think there has been any (armv8-a) processors that didn't support those instructions (and the deprecation was added as part of armv8 to begin with).


Update .travis.yml to work around go get issue at tip

I just manually reran the last full Travis CI build, and it suddenly passed for both Go 1.4 and Go 1.5 tip.
https://travis-ci.org/GoogleCloudPlatform/gcloud-golang/builds/73325850
So maybe we won't need this change after all.

Build fixed properly by golang/go@226b28c85cd762f777135c102d83770fc4e6ade9.


Extend update_plist documentation from 1 to 6 code-examples

Checklist

[x] I've run bundle exec rspec from the root directory to see all new and existing tests pass
[x] I've followed the fastlane code style and run bundle exec rubocop -a to ensure the code style is valid
[x] I've read the Contribution Guidelines
[x] I've updated the documentation if necessary.

Motivation and Context
It took me a considerable amount of time and trial & error to initially figure out how to actually use the update_plist action to modify the values inside a plist-file, whose functionality was (imho) only sparsely documented.
Description
I added five new examples for the basic functionality of this action in the documentation, in addition to the one (more complicated and highly specific) example already present, to improve understandability and readability in the future.
I also slightly adapted self.details in order to differentiate this action from the (similar but different in usage) update_info_plist action.
Testing Steps
In addition to running bundle exec rspec and bundle exec rubocop -a succesfully I tested all of the example-codes and checked the plist-file in XCode afterwards to make sure the values were correctly set. See encompassing testing-code and result-image below:
lane :test_update_plist do |options|
  options[:environment] = "development"

  # Updates values in Firebase-plist
  update_plist(
    plist_path: "GoogleService-Info.plist",
    block: proc do |plist|
      if options[:environment] == "production"
        plist[:boolean_entry] = true
        plist[:number_entry] = 13
        plist[:array_entry] = ["entry_01", true, 1243]
      else
        plist[:boolean_entry] = false
        plist[:number_entry] = 27
        plist[:array_entry] = ["entry_02", false, 4265]
      end
    end
  )
end

@googlebot I signed it!

@joshdholtz Thank you! Super-glad to hear you like it <3


tcmu-runner: Make g_runner_handlers an array of pointers

on_check_config receives a user_data pointer, which points to some
element into g_runner_handlers, the array of struct tcmur_handler. This
is not a problem now because the array never resizes (reallocates) once
built, but that is a very fragile assumption and is prune to memory
corruption in the far future.
Instead, save the pointers. All existing callers of
tcmur_register_handler are passing addresses of a static global
variable, so there is no use-after-free risk.  Future callers must be
more careful of the life cycle of handler because they are no longer
copied by value.
Signed-off-by: Fam Zheng<EMAIL_ADDRESS>

yes, rather fragile right now. Thanks!


clpe: 0.1.0-2 in 'foxy/distribution.yaml' [bloom]

Increasing version of package(s) in repository clpe to 0.1.0-2:

upstream repository: https://github.com/canlab-co/CLPE_G_NVP2650D_SDK.git
release repository: https://github.com/canlab-co/CLPE_G_NVP2650D_SDK-ros2-release.git
distro file: foxy/distribution.yaml
bloom version: 0.10.7
previous version for package: null

clpe
* Provides libclpe
* Contributors: Can-lab Corporation

@marcoag - if this package is no longer released from the clpe_sdk repository, then that repository needs to be removed from rosdistro before this one can be added.

Ops, sorry about that, I didn't realize they did a release with that name before. I believe is the same repo so I'll just update the old one. Thanks for the fix.


Mixed inputs

Check how many txs has mixed inputs, meaning more than 1 input and at least two different script types

Seems like a good place to start to understand the inetrnals.
Willing to work on this..

Great, have a look at the examples if you didn't already, and reach me on discord if you have questions


Registering TEAM Z3Nspyder

Application for Registering Team to CODE CAMP 1.0
This pull request template helps you complete an application to the CODE CAMP 1.0 HACKATHON. Use the checklist below to verify that you have followed the instructions correctly.
(Put x inside [] like [x])
Select the type of Pull Request
[] Registration Pull request.
[] Submission Pull request.
Checklist ✅

[] I have read the instructions on the README file before submitting my application and gone through CODE_OF_CONDUCT.
[] I have Submitted my team's details by making a folder of my team as instructed in How to send Pull Request page.
[] I have used the Markdown file template to add my information for the Hackathon.
[] I have given all the necessary details as mentioned in template index.md file and the details are correct and best to my knowledge.
[] I understand that a reviewer will merge my pull request after examining it or ask for changes in case needed.
[] I understand I should not tag or add a reviewer to this Pull Request.
[] I understand the Details added to the template will be used as a means of communication at the time of result declaration.
[] I have added the event to my calendar.

I have made the changes

@hanieljacob ,
Please, Make changes only in Teams folder as instructed and please Do mark the check list, and also in github link up at the top. The link is not working . Kinda giving error [404 not found]
Do these change and i  will merge it ASAP.

What other changes have I made? As far as I know, I have made changes only in the team folder

Actually you have also made changes in Pull_request.md so it is not needed.

Do you mean PULL_REQUEST_TEMPLATE.md?

Please check now

See. You have to make changes in TEAM/your_team_folder.
But you did :

Made changes in Pull_Request_Template.md  ( Unncessary changes ) ( Need to revert Back )
Made changes in Team/your_team_folder ( Perfect )( Perfect for merging ) (Required)

Also mark checklist ... https://github.com/ISTESRMNCR/CODE-CAMP-2020/pull/37#issue-452643852
put x inside [] like [x]
Also give complete link for Github id's for every teammate.

How do I undo the changes? Do i send a new pull request?

How do I undo the changes? Do i send a new pull request?

if you can revert, so possibly fork repository again. and send the Pull request again.
For forking Delete repository from your Profile and then fork again.
Do not worry will merge By today only.
DO request me for reviewing also can tag me.
Now do not make any changes to anyother file instead your TeamFiles

Closing this PR SESSION NOW


Add letsencrypt for freezing-nq web interface

Migrated from TODO

This has been done for years. Closing.


Fixes #30025: Add options to selectively mark messages as read.

Fixes:  #30025 : I have added the options for mark as read.
It has three options :
Muted topics
topics I don't follow
All messages
It is multiple selection.
Screenshot :

Fixes:
Video Demonstration:
https://github.com/user-attachments/assets/6c790f8e-7461-4a12-be25-276b94823e37

Screenshots and screen captures:

Self-review checklist

[x] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[ ] Explains differences from previous plans (e.g., issue description).
[ ] Highlights technical choices and bugs encountered.
[ ] Calls out remaining decisions and concerns.
[ ] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[x] Each commit is a coherent idea.
[x] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[x] Visual appearance of the changes.
[x] Responsiveness and internationalization.
[ ] Strings and tooltips.
[x] End-to-end functionality of buttons, interactions and flows.
[x] Corner cases, error conditions, and easily imagined bugs.

@karlstolley , Kindly review the PR. I had to close the previous PR. I tried to fix the issue of untidy commit history but due to my own mistake it put me into another trouble. So I decided to create a fresh   PR.

bro @AMANVISHWAKARMA27
this Zulip CI / Ubuntu 22.04 (Python 3.10, backend + frontend)  ->
eslint    |
eslint    | /__w/zulip/zulip/web/e2e-tests/navigation.test.ts
eslint    |   76:[9](https://github.com/zulip/zulip/actions/runs/10745431223/job/29804425323?pr=31549#step:13:10)  error  Unsafe call of an `error` type typed value  @typescript-eslint/no-unsafe-call
eslint    |
eslint    | ✖ 1 problem (1 error, 0 warnings)
eslint    |
Run ./tools/lint --fix to fix errors for the following linters: eslint

error occur because  someone method return type any in navigation.test.ts
i have temp solution but its not good.

bro @AMANVISHWAKARMA27 this Zulip CI / Ubuntu 22.04 (Python 3.10, backend + frontend) ->
eslint    |
eslint    | /__w/zulip/zulip/web/e2e-tests/navigation.test.ts
eslint    |   76:[9](https://github.com/zulip/zulip/actions/runs/10745431223/job/29804425323?pr=31549#step:13:10)  error  Unsafe call of an `error` type typed value  @typescript-eslint/no-unsafe-call
eslint    |
eslint    | ✖ 1 problem (1 error, 0 warnings)
eslint    |
Run ./tools/lint --fix to fix errors for the following linters: eslint

error occur because someone method return type any in navigation.test.ts i have temp solution but its not good.

Thanks @mitesh092 , I could point out the error.

hey @AMANVISHWAKARMA27
if it resolve this error then explain me how can u did it
thx.

I saw the changed files and found that in one  of the files , the linter removed one line , I don't know why but that is giving error so added it again. I  think this time It would be fine.

@AMANVISHWAKARMA27
in my case i just run prettier for  navigastion.test.ts and its maybe solve now i go and creating new pr

Yeah either way.

bro (@AMANVISHWAKARMA27 )
Do you know when our pull request will be reviewed? Just checking in.

When I test the PR, it does not match the screenshot in the PR description and the issue description. Please update it, and post up-to-date screenshots of the confirmation modal, as well as the menu to open it.

Following the pattern from other modals, the checkboxes should not be indented.

@alya , thank you for pointing out the flaws. I will make the changes asap.

@alya, I have made the required changes. Please review the changes.

This doesn't seem to mark messages as read according to the options I select.

This doesn't seem to mark messages as read according to the options I select.

@pratik-pc , thanks for you review. I will check if there is something breaking :)

@pratik-pc I have made the necessary changes. Kindly review the PR and point out if there is something still lacking.

This marks everything as read irrespective of the selected option.

This marks everything as read irrespective of the selected option.

@pratik-pc , I will try to solve it asap.

@AMANVISHWAKARMA27 In order to contribute successfully to Zulip (or any open-source project), you will need to carefully check your own work before asking anyone else for review, to be respectful of maintainers' time.

@AMANVISHWAKARMA27 In order to contribute successfully to Zulip (or any open-source project), you will need to carefully check your own work before asking anyone else for review, to be respectful of maintainers' time.

Sure @alya , I will take of it in future:) and I apologize for it.


How to make the loader options dynamic with the inline loader require

I want to do something like this.
var options = {foo: 1, bar: 2} or defined somethere else...

require('loader?--options goes here--!./file')

If i do require('loader?options=' + options + '!./file') it will enable the require.context thing, but not that i needed.
Is there a way to pass dynamic parameters inside the loader's node.js context?

+1

If still relevant, please re-open at Stack Overflow.


Percentage complete goes up to 690%

See screen shot attached. The percentage complete appears to go over 100% when the file analysis process starts.
The percentage complete needs to be 100% max.

Closed via https://github.com/open-contracting/spoonbill-web/pull/317

Tested, this is much better now. Thank you

It happened because the example file is an archive and the extracted file has a bigger size than the archive itself. Now we just take the size of the extracted file for the percentage calculation.


New packet format

Refactoring CDP Packet. This has all the changes from the library and was tested with modified Arduino apps.
I will update the apps separately.

Closing this until I resolve all the conflicts. Will submit another PR once I am done


[MOB-1858] - RN Android not able to pass null.

Expecting developers to send -1(as mentioned in the docs) if the value is unknown. SDK will internally send null to Iterable's SDK

Is this a limitation of RN that you can't have nullable arguments in native methods?


[Bug]Run Kyuubi Server in local IDE, will create metrics dir which should be add to gitignore

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Search before asking

[X] I have searched in the issues and found no similar issues.

Describe the bug
When run kyuubi server in local IDE, kyuubi server will create metrics for store kyuubi server's metrics.
Run git status show
未跟踪的文件:
  （使用 "git add <文件>..." 以包含要提交的内容）
        metrics/

Found in .gitignore
metrics/report.json
metrics/.report.json.crc

Affects Version(s)
master
Kyuubi Server Log Output
No response
Kyuubi Engine Log Output
No response
Kyuubi Server Configurations
No response
Kyuubi Engine Configurations
No response
Additional context
No response
Are you willing to submit PR?

[X] Yes. I would be willing to submit a PR with guidance from the Kyuubi community to fix.
[ ] No. I cannot submit a PR at this time.

This modification will result in all folder whose name is metrics being ignored.
# for example, a normal metrics package  for spark-engine will be ignored due to this change
externals/kyuubi-spark-sql-engine/src/main/scala/org/apache/spark/kyuubi/metrics/

if we want to ignore $repo_root/metrics created by local IDE, maybe we could change gitignore like this :
# gitignore after
/metrics/

# gitignore before
metrics/

the metrics created in working dir by default, what will happen if run test in IDEA?

Hi @Z1Wu , i try test in my local with following command:

modify class under org.apache.kyuubi.metrics
run git status

And output look fine to our expection.

@zwangsheng all changes will be identified if you touch a git-tracked file, I suppose creating a new file under the metrics folder will be ignored, and git add -f <new_file> must be used then

Yes, i tried touch a file under metrics package, seems it has not been tracked. @Z1Wu Thanks for catching this. Would you like to raise a PR to fix this?


feat: ability to look up athlete, competition, lift data via admin by…

… searching

Here you go @ditorelo, you should be able to search athletes by first/last name, competitions by name/location, and lifts by both athlete first/last name, as well as competition name/location.


How can I get Audio/Video call when app in background or not running mode

Currently I am using Agora SDK for Audio/Video call. I am not getting call when app is not running. What I need to added to get audio/video call in background or app not running.

Hi Giresh, you'd need to implement CallKit for the notifications of incoming calls while the app is closed.

@maxxfrazer
I tried with VOIP push notification but my app crashed when I accepted the call.
my app crashed here, showing empty lastIncomingInvitation, as I got call from VOIP push notification.
guard let last = rtm.lastIncomingInvitation else {
fatalError("rtm lastIncomingInvitation")
}
code:
func accpetLastIncomingInvitation(fail: ErrorCompletion = nil) {
let rtm = AgoraRtm.shared()
    guard let last = rtm.lastIncomingInvitation else {
        fatalError("rtm lastIncomingInvitation")
    }

    accept(last) {(errorCode) in
        guard errorCode == AgoraRtmInvitationApiCallErrorCode.ok else {
            if let fail = fail {
                fail(AGEError(type: .fail("rtm refuse invitation fail: \(errorCode.rawValue)")))
            }
            return
        }
    }
}

please help me on this

Closing this due to inactivity.


Add gitignore and cmake

-Basic gitignore for cmake files
-Cmake configuration for project

Note: on Windows, tree will output garbage caracters, but this is cmd.exe's, fault, given that it doesn't support UTF-8. Everything else works correctly.


Mappings in mappings file vs the web service

Following up on an email I sent to the mailing list about mappings to other resources like NCIT, UMLS and DOID, I noticed that the mappings file at https://github.com/cBioPortal/oncotree/blob/master/scripts/ontology_to_ontology_mapping_tool/ontology_mappings.txt
provides mappings for 771 non-tissue OncoTree terms to NCIT while the data returned from the web service at https://oncotree.mskcc.org:443/api/tumorTypes/tree only provides mappings for 525 non-tissue OncoTree terms to NCIT.
When I first looked at the data from the webservice I concluded that a lot of mappings are missing but the mappings file on GitHub appears to be much closer to complete. Is there a reason why not all mappings from the file are made available in the web service?

Same issue here. The impact is getting huge taking account that some websites, when mapping to a specific term, can miss a huge amount of data.
e.g: I am interested in "Invasive breast carcinoma".

I go to Oncotree and look for it: https://oncotree.mskcc.org/#/home

Great. it exists.

I want to get information from the "ProjectScore" from cell lines related to this disease. so i go to opentargets and I do a search by the disease term "Invasive breast carcinoma:  https://platform.opentargets.org/disease/EFO_1000307/associations

Surprisingly there is no information related.  so i go directly the the project source and type my fav cell line related to "invasive breast carcinoma"

And IT DOES NOT FIND IT!
But if i just type: Breast Carcinoma

it find it!

but, is my favorite cell line "breast carcinoma"? or "invasive breast carcinoma"? So then i will pick my favorite cell line (that i though it was "invasive cell carcinoma") --> SUM-52PE.

I see the oncology term is "Breast carcinoma"

I am aware this Depmap source is somehow related to Depmap "Broad institute". So i go and check for this cell line in there too. Surprisingly, my cell line appears with a different oncotree term. this time, is correctly mapped to the term I found originally in Oncotree website.

CONCLUSION:

Is oncotree website/mapper missing some parent nodes? (NOTE: "Invasive breast carcinoma (BRCA)" and "Breast Carcinoma" both appears in the NCIT (national cancer institute thesaurus) : breast cancer: C4872  and  invasive breast cancer- C9245  .
THis will lead to missing mappers from tools like opentargets or sangerDepmap.

On the other hand, is the problem from the SangerDepmap, who are using maybe outdated annotations, because Oncotree is not considering "breast carcinoma" anymore? (this ticket will be shared with the responsibles of this project Project Score. so maybe you both can reach a consensus and please keep us in the loop).

Also I will contact Opentargets team because maybe I am missing any important information from their documentation? Although in the community not question has been raised about oncotree so far.

Please keep us in the loop here.
Many thanks!


Is there a mistake in patchwork.hpp????

Line 439
if (ground_z_elevation > elevation_thr_[ring_idx + 2 * k]) {
is index 'ring_idx + 2k' wrong ?
the size of elevation_thr_ is 4. So, ' ring_idx + 2k' may be large than 4.
Can you explain it?

Hello there, sorry for the late reply due to my paperwork!
But it is previously checked in
if (concentric_idx < num_rings_of_interest_) { condition.
Please print ring_idx + 2 * k out :)


Formatting issues with the new version

CSScomb JS 3.0.0 comes with a bunch of great features but unfortunately I am having problems with it.
Under OS X the new version 3.0.0 installs to "~/Library/Application Support/Sublime Text 3/Packages/CSScomb" but needs to be installed to "CSScomb JS". This causes the plugin not finding the config files.
Also, the new version formats my .less file not as it used to, so that it can't compile. It puts opening braces in a new line and always uses 4 tabs.
Using my own configuration won't help, although I have tried to rename the "CSScomb" directory to "CSScomb JS" and tried to replace the " JS" from the Main.sublime-menu file.

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

Closing as stale. Please ping back if you feel this still needs attention.


Cannot read property 'onOpen' of null at new AutofocusDirective

Hi,
First of all, thanks for writing this.
I'm getting the error Cannot read property 'onOpen' of null at new AutofocusDirective when I run my app with ng-bs3-modal added.
It seems like in AutofocusDirective https://github.com/dougludlow/ng2-bs3-modal/blob/master/src/ng2-bs3-modal/directives/autofocus.ts there is the part:
if (!modal) {
            this.modal.onOpen.subscribe(() => {
                this.el.nativeElement.focus();
            });
 }

But in my case, modal is null (the injection is optional, and I dont have a modal on that page yet). So it crashes on the .onOpen call.
I was wondering whether if (!modal) should be if (modal), or whether I'm misunderstanding that part?
Cheers,
Tom

@TomThorpe, wow, not sure how I missed that one, ha ha. Thanks for letting me know, I'll work on getting a fix out soon as well as some test coverage for that so I don't break it again... 😛

No worries :-) I did wonder if I was missing something haha. Easily done though!
Thanks again

@TomThorpe - sorry it took me so long to get to this, but this is fixed in v0.9.2.

Ah great, thanks very much!


Webhooks in endpoints.yml

Description of Problem:
Currently rabbitmq and kafka are supported for consuming events. These are configurable through the endpoints.yml. For simple integrations such as sending events off to an endpoint directly, setting up rabbitmq/kafka and consuming those events can be overkill.
Overview of the Solution:
I am hoping to add a new configuration to endpoints.yml under a property 'webhooks' where we can specify URLs that rasa posts events to.
Definition of Done:

[ ] Tests are added
[ ] Feature is documented

@shotamakino could you clarify what you mean? what kind of webhooks would you like?

So exploring some more, I've been able to accomplish what I required through a custom EventChannel by publishing a package and specifying this class through the endpoints.yml.
What I was originally seeking for was a new EventChannel type with a url arg that specifies where to post the event to as a body parameter.
An alternative could be to allow the user to create a new module in the directory with their rasa configurations and specify this module via the endpoints.yml through the type property.

So is your original issue solved? i'm still not sure which URL you'd be posting events to

Yes my original issue is resolved - I will close this enhancement request!


[Issue] Add sign in doc for azure developer credential

When adding azure developer credential to azure-sdk-python, I found out they have documentation on how to sign in to azure cli. Do we want to have something similar to https://learn.microsoft.com/en-us/cli/azure/authenticate-azure-cli?
cc: @rajeshkamal5050 @savannahostrowski @puicchan

@ellismg don't think this is a must-fix for GA. If you think this is needed, please feel free to pull it back into May milestone.
cc @savannahostrowski

@gkulin - let's think about where this could live or how we could best incorporate this into our docs as part of #1884.
P1 (nice to have) for GA.

Moving to Gallium for now

@savannahostrowski @hemarina @gkulin moving this to Backlog. Haven't seen any requests/asks around having such doc. Please feel free to pull it in, if need be.


Update Vania to 0.3.0

Thank you. Can you please rebase your branch onto the newest main branch?

@Tienisto Yes sure

It seems that git merge didn't notice that I renamed laravel-11-10-0-php-8.3 to laravel-11-php-8.3


Mobilehub -> Amplify migration. Import existing API GATEWAY instance.

Im migrating environment from MobileHub to Amplify. I want to continue using all of the existing resources (Cognito, APi gateway and hosting of the app).
What I have done:

Initialize the project with amplify init command
Import existing Cognito using the amplify import auth command
Add hosting feature for the project using amplify add hosting

What is left:

Import Api Gateway to the freshly created environment, which is impossible for now as I know?

Is your feature request related to a problem? Please describe.
Mobilehub will be discontinued in October 30 2021, I need to migrate existing environments to the Amplify service and, in the future, be able to create new environments with Amplify.
Describe the solution you'd like
Easy way to link amplify project.
Describe alternatives you've considered
I tried manual adding the api gateway resource in the aws-export.js file, it does work locally, but when I deploy the application using amplify publish command, this file is overridden, oauth configuration is being added and I get following error

My aws-exports.js file (without manually added api address):

Hello @AdriannaJedrzejczak
We do not recommend using import auth to move your auth resource from mobilehub to amplify cli.
We do have a migrator tool which can help with this usecase
Repo: https://github.com/awslabs/amplify-mobilehub-migrator

Thanks for your response, @SwaySway.
I tried using the migration tool however, I found the following issues:

Hosting feature is not migrated properly. I cannot either successfully run amplify publish, because its causing the following error: An error occurred during the publish operation: No Amazon CloudFront and S3 hosting service is enabled.

nor run amplify add hosting or amplify remove hosting.
Running amplify add hosting gives me S3AndCloudFront is present in amplify-meta.json
Running amplify remove hosting gives me You have not added any resources in the hosting category.
It looks like its halfway configured.
2. API is migrated with hardcoded /Development (upper case) URL -> I managed to update it manually and make it work.
Additionally, I need to make sure that in the future, there exists a possibility to create a new environment from scratch, which is not when I only base on the migration tool

@AdriannaJedrzejczak
Sorry for the confusion, would you mind opening up an issue in the MobileHub migrator GitHub repo -  https://github.com/awslabs/amplify-mobilehub-migrator and we'd be happy to help migrate your MobileHub project to Amplify. Thank you.


Re-design the position screen

The idea is to make it simpler:

[x] define requirements
[x] mock them in figma

Designs can be found in Figma
The idea is to show only minimal but crucial information in the default overview. When clicking on it, a modal opens from the bottom which shows more details. The close button leads to the well-known "swipe to confirm screen".
https://github.com/get10101/10101/assets/224613/4cec06bf-71f3-469c-9a0f-217e3e4d7c80

One caveat though is that the close button is again hidden 🙈
Since we do not support resizing yet how about showing a hovering close button instead of the buy and sell button (when a position is open)


Support for PHP 8.2

Trying to install it, I discovered it does not support it. Would appreciate if you be kind enough to find the time to support it.

PR Open for Laravel 10 + PHP 8.2: #20


Result usage in secure_cmp

Why does this function return a Result instead of a simple bool?
I'm not even sure if Result is usable if you want a cost-time execution.
The check for the same length is also not necessary, subtle does this already.
https://github.com/brycx/orion/blob/a6efb6f9094c71a17a9ce07fb8e288d5bd808dd2/src/util/mod.rs#L97

Why does this function return a Result instead of a simple bool?

Returning a Result makes it easier to propagate a failed verification in other functions which also return a Result. So we avoid adding extra if-else statements. Though I admit returning a bool only if the verification succeeded may be redundant, since you could also use .is_ok()/.is_err() and have Result<(), UknownCryptoError>. I haven't tested whether that executes in constant-time.

I'm not even sure if Result is usable if you want a cost-time execution.

AFAIK Result only impacts constant-time execution when run with opt-level = 0. In the orion-dudect repository, secure_cmp is tested for constant-time execution. If you're interested, I have a write-up that outlines how I've tested the secure_cmp function in the past.

The check for the same length is also not necessary, subtle does this already.

The length check is indeed redundant and should be removed. Nice spot!

Fixed in #93.


i18n(pt-BR): Add new making-toolbar-apps.mdx translation

Add new making-toolbar-apps.mdx translation
Adds the pt-br translation to the making-toolbar-apps.mdx recipe.

Lunaria Status Overview
🌕 This pull request will trigger status changes.

Learn more
By default, every PR changing files present in the Lunaria configuration's files property will be considered and trigger status changes accordingly.
You can change this by adding one of the keywords present in the ignoreKeywords property in your Lunaria configuration file in the PR's title (ignoring all files) or by including a tracker directive in the merged commit's description.

Tracked Files

Locale
File
Note

pt-br
recipes/making-toolbar-apps.mdx
Localization added, will be marked as complete.

Warnings reference

Icon
Description

🔄
The source for this localization has been updated since the creation of this pull request, make sure all changes in the source have been applied.


[FEATURE] Allow blocked words by AlexJS

Description
Some words are grouped together, and we do not want to allow the whole group, but some words from the group.
For example the word fellow we want to allow, but it is grouped with words we do not to want to allow.
We could write a filter for the blocked words that come out of AlexJS, and disregard the notification of that word.
I think we could do it here because we do get the blocked word out of the group in the response .setTitle(You used the word "${alexMatch[0].actual}")`
Screenshots
No response
Additional information
No response

I think this should be implemented. Certain words should be allowed and those cases should be matched.

Thank you 👍 , PR has been raised, after being reviewed and merged, people can create more PRs with config changes


Amend documentation for Artifact Registry module

Updated documentation to reflect output.
Closes https://github.com/hashicorp/terraform-provider-google/issues/13486
If this PR is for Terraform, I acknowledge that I have:

[X] Searched through the issue tracker for an open issue that this either resolves or contributes to, commented on it to claim it, and written "fixes {url}" or "part of {url}" in this PR description. If there were no relevant open issues, I opened one and commented that I would like to work on it (not necessary for very small changes).
[X] Ensured that all new fields I added that can be set by a user appear in at least one example (for generated resources) or third_party test (for handwritten resources or update tests).
[X] Generated Terraform providers, and ran make test and make lint in the generated providers to ensure it passes unit and linter tests.
[X] Ran relevant acceptance tests using my own Google Cloud project and credentials (If the acceptance tests do not yet pass or you are unable to run them, please let your reviewer know).
[X] Read the Release Notes Guide before writing my release note below.

Release Note Template for Downstream PRs (will be copied)

whoops, wrong upstream


[bug] Possible for frame entry in stack table to be null

A production app generated profiles which failed to load in any version of firefox-profiler we tried. One of our devs filed this issue upstream https://github.com/firefox-devtools/profiler/issues/5074, but I believe the true issue is that the output from vernier is not a valid gecko profile.
The profile does not load in https://vernier.prof either. Here is the profile.
Looking closely at the error, I was able to determine that a thread has an invalid stack table:
    {
      "name": "(2051520)",
      "isMainThread": false,
      "processStartupTime":<PHONE_NUMBER>36,
      "registerTime":<PHONE_NUMBER>36,
      "pausedRanges": [],
      "pid": 123,
      "tid": 0,
      "frameTable": {
        "address": [],
        "inlineDepth": [],
        "category": [],
        "subcategory": null,
        "func": [],
        "nativeSymbol": [],
        "innerWindowID": [],
        "implementation": [],
        "line": [],
        "column": [],
        "length": 0
      },
      "funcTable": {
        "name": [],
        "isJS": [],
        "relevantForJS": [],
        "resource": [],
        "fileName": [],
        "lineNumber": [],
        "columnNumber": [],
        "length": 0
      },
      "nativeSymbols": {},
      "samples": {
        "stack": [
          0,
          1
        ],
        "weight": [
          1,
          1032
        ],
        "weightType": "samples",
        "length": 2
      },
      "stackTable": {
        "frame": [
          null,
          null
        ],
        "category": [
          2,
          1
        ],
        "subcategory": [
          0,
          0
        ],
        "prefix": [
          null,
          null
        ],
        "length": 2
      },

The root cause seems to be that the "frame" entry itself is somehow "null". This gets input as a numeric index, and ends up exploding in the UI.
If I delete the invalid thread, the profile can now be displayed.
This seems like a legitimate bug in vernier, as we're somehow getting invalid output where the top stack frame references a null value, which is an invalid index into the frame table. The frametable is also empty, as the example shows.

Note that the profile was generated from the latest released version of vernier, 1.1.1

I'm encountering the same thing. I'm profiling something with bundler/inline and I believe the issue has something to do with me specifying git dependencies:
# frozen_string_literal: true

require "bundler/inline"

gemfile do
  source "https://rubygems.org"
  gem "rails", github: "rails/rails"
end

This consistently produces a profile I'm unable to load for what I assume are the same reasons as OP. trace.json
I run vernier like so: vernier run --output trace.json -- ruby test.rb

Reduced it a bit further and it seems to boil down to this:
# frozen_string_literal: true

require "open3"
Open3.capture3("sleep", "1")

trace.json
I've tried it with plain sleep 1 in backticks but that works fine so it seems specific to something the open3 gem does.

Thank you both! I just merged https://github.com/jhawthorn/vernier/pull/89 which should fix this and released that as 1.1.2

This works nicely, thanks!

@jhawthorn thank you! We'll try it out and report back if it is not resolved.


Unable to build the packages

Hi, I'm trying to build the mgacician package, however the colcon build fails  for dobot_msgs:
sudo colcon build
Starting >>> dobot_msgs
Starting >>> dobot_bringup
Starting >>> dobot_control_panel
Starting >>> dobot_description
Starting >>> dobot_driver
Starting >>> dobot_visualization_tools
Finished <<< dobot_description [1.62s]
--- stderr: dobot_msgs
Traceback (most recent call last):
  File "/opt/ros/humble/lib/rosidl_typesupport_fastrtps_cpp/rosidl_typesupport_fastrtps_cpp", line 6, in <module>
    from rosidl_typesupport_fastrtps_cpp import generate_cpp
ModuleNotFoundError: No module named 'rosidl_typesupport_fastrtps_cpp'
Traceback (most recent call last):
  File "/opt/ros/humble/lib/rosidl_typesupport_fastrtps_c/rosidl_typesupport_fastrtps_c", line 7, in <module>
    from rosidl_typesupport_fastrtps_c import generate_typesupport_fastrtps_c
ModuleNotFoundError: No module named 'rosidl_typesupport_fastrtps_c'
Traceback (most recent call last):
  File "/opt/ros/humble/lib/rosidl_typesupport_introspection_cpp/rosidl_typesupport_introspection_cpp", line 6, in <module>
    from rosidl_typesupport_introspection_cpp import generate_cpp
ModuleNotFoundError: No module named 'rosidl_typesupport_introspection_cpp'
gmake[2]: *** [CMakeFiles/dobot_msgs__rosidl_typesupport_fastrtps_cpp.dir/build.make:154: rosidl_typesupport_fastrtps_cpp/dobot_msgs/action/detail/dds_fastrtps/point_to_point__type_support.cpp] Error 1
gmake[1]: *** [CMakeFiles/Makefile2:342: CMakeFiles/dobot_msgs__rosidl_typesupport_fastrtps_cpp.dir/all] Error 2
gmake[1]: *** Waiting for unfinished jobs....
Traceback (most recent call last):
  File "/opt/ros/humble/lib/rosidl_typesupport_c/rosidl_typesupport_c", line 6, in <module>
    from rosidl_typesupport_c import generate_c
ModuleNotFoundError: No module named 'rosidl_typesupport_c'
Traceback (most recent call last):
  File "/opt/ros/humble/lib/rosidl_typesupport_introspection_c/rosidl_typesupport_introspection_c", line 6, in <module>
    from rosidl_typesupport_introspection_c import generate_c
ModuleNotFoundError: No module named 'rosidl_typesupport_introspection_c'
gmake[2]: *** [CMakeFiles/dobot_msgs__rosidl_typesupport_fastrtps_c.dir/build.make:154: rosidl_typesupport_fastrtps_c/dobot_msgs/action/detail/point_to_point__rosidl_typesupport_fastrtps_c.h] Error 1
gmake[1]: *** [CMakeFiles/Makefile2:238: CMakeFiles/dobot_msgs__rosidl_typesupport_fastrtps_c.dir/all] Error 2
gmake[2]: *** [CMakeFiles/dobot_msgs__rosidl_typesupport_introspection_cpp.dir/build.make:154: rosidl_typesupport_introspection_cpp/dobot_msgs/action/detail/point_to_point__rosidl_typesupport_introspection_cpp.hpp] Error 1
gmake[1]: *** [CMakeFiles/Makefile2:368: CMakeFiles/dobot_msgs__rosidl_typesupport_introspection_cpp.dir/all] Error 2
gmake[2]: *** [CMakeFiles/dobot_msgs__rosidl_typesupport_c.dir/build.make:152: rosidl_typesupport_c/dobot_msgs/action/point_to_point__type_support.cpp] Error 1
gmake[1]: *** [CMakeFiles/Makefile2:290: CMakeFiles/dobot_msgs__rosidl_typesupport_c.dir/all] Error 2
gmake[2]: *** [CMakeFiles/dobot_msgs__rosidl_typesupport_introspection_c.dir/build.make:154: rosidl_typesupport_introspection_c/dobot_msgs/action/detail/point_to_point__rosidl_typesupport_introspection_c.h] Error 1
gmake[1]: *** [CMakeFiles/Makefile2:264: CMakeFiles/dobot_msgs__rosidl_typesupport_introspection_c.dir/all] Error 2
gmake: *** [Makefile:146: all] Error 2
---
Failed   <<< dobot_msgs [1.83s, exited with code 2]
Aborted  <<< dobot_driver [1.66s]
Aborted  <<< dobot_bringup [2.00s]
Aborted  <<< dobot_control_panel [1.99s]
Aborted  <<< dobot_visualization_tools [1.91s]

Summary: 1 package finished [3.10s]
  1 package failed: dobot_msgs
  4 packages aborted: dobot_bringup dobot_control_panel dobot_driver dobot_visualization_tools
  1 package had stderr output: dobot_msgs
  7 packages not processed

Running Ubuntu 22.04 with ROS2 Humble installed...
Any idea's?

Hi @paddenstoeltje, you should create a clean workspace and build packages once again - this time without sudo.
Let me know, if it resolves the problem :wink:

Hi thanks,
that worked. However I'm facing another problem now regarding CMAKE. It seems there is something wrong with my environment (although is is sourced):
--- stderr: dobot_msgs
Traceback (most recent call last):
  File "/opt/ros/humble/share/ament_cmake_core/cmake/package_templates/templates_2_cmake.py", line 21, in <module>
    from ament_package.templates import get_environment_hook_template_path
ModuleNotFoundError: No module named 'ament_package'
CMake Error at /opt/ros/humble/share/ament_cmake_core/cmake/ament_cmake_package_templates-extras.cmake:41 (message):
  execute_process(/usr/bin/python3.10
  /opt/ros/humble/share/ament_cmake_core/cmake/package_templates/templates_2_cmake.py
  /home/wouter/magician_ros2_control_system_ws/build/dobot_msgs/ament_cmake_package_templates/templates.cmake)
  returned error code 1
Call Stack (most recent call first):
  /opt/ros/humble/share/ament_cmake_core/cmake/ament_cmake_coreConfig.cmake:41 (include)
  /opt/ros/humble/share/rosidl_default_generators/cmake/rosidl_default_generators-extras.cmake:4 (find_package)
  /opt/ros/humble/share/rosidl_default_generators/cmake/rosidl_default_generatorsConfig.cmake:41 (include)
  CMakeLists.txt:22 (find_package)

When looking up: printenv | grep -i ROS I get:
ROS_VERSION=2
ROS_PYTHON_VERSION=3
PWD=/home/wouter/magician_ros2_control_system_ws
AMENT_PREFIX_PATH=/opt/ros/humble
COLCON_PREFIX_PATH=/home/wouter/magician_ros2_control_system_ws/install
PYTHONPATH=/opt/ros/humble/lib/python3.10/site-packages:/opt/ros/humble/local/lib/python3.10/dist-packages:/usr/lib/python3/dist-packages/:/opt/ros/humble/lib/python3.10/site-packages
LD_LIBRARY_PATH=/opt/ros/humble/opt/rviz_ogre_vendor/lib:/opt/ros/humble/lib/x86_64-linux-gnu:/opt/ros/humble/lib
ROS_LOCALHOST_ONLY=0
PATH=/opt/ros/humble/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games:/usr/local/games:/snap/bin
ROS_DISTRO=humble
OLDPWD=/home/wouter/magician_ros2_control_system_ws/src

I would try installing ament_package first.
sudo apt install ros-humble-ament-package

It is already installed. Still getting the error. No clue what is wrong.

dobot_msgs is just a package with custom msg, srv and action files. Check if you can build a package from Creating custom msg and srv files tutorial :wink:

Reinstalling Ros2 resolved all the problems. Thanks anyway. It seems something was wrong with the environment.


bug - current password isn't considered against candidate replacement

Hi again
It's possible to renew an expired password to be the same as the current, ie to not actually force a change. This is possible because you validate before updating the old_passwords, and the validation doesn't check the current password, ie you never check against the password being replaced.
Also note the repeated limit on https://github.com/phatworx/devise_security_extension/blob/master/lib/devise_security_extension/models/password_archivable.rb#L35
Paul

Hi,
I noticed the same issue, i"ve already patched code and sent pull request https://github.com/phatworx/devise_security_extension/pull/14
Regards!


Could pandas be used as an alternative SQL backend?

I am so used to manipulating data with pandas that I don't want to fall back to SQL unless absolutely necessary. In fact, I'd rather not use SQL. Nonetheless SQL works better for out-of-core size data (obviously, that is not the primary focus of pandas, and yes I am aware of Dask).
Would it be possible to use pandas on data that resides in an SQL database without importing it in memory. I am not asking whether it would be possible to access the data, but whether it would be possible to use pandas as backend to SQL. For instance: I write pandas code and SQL is executed in the background. I want to count all the values in a column, but I write .value_counts(). Do you see what I mean?
Does that make any sense? Apologies if my question is off-topic or naive?

I think you want to take a look at the ibis or blaze projects, which both have a pandas-like python API that translate to an underlying storage system (decoupling expression from actual computation)


Logs recommend use of 'version 2' for excess non-zero padding data

Hi Jérôme,
I've noticed recently that there's a small typo appearing in log error warning, that asks user to re-encode using '--version 2' when large non-zero padding is found. I think it should read '--output-version 2'.  Attaching a screen shot!
Many thanks and lots of happy MKV files ingesting to BFI DPI these days!

Fixed in https://github.com/MediaArea/RAWcooked/commit/1a64eede7b43917730636f391187b66735fdaa3e.


NuGet Cache for Paket already exists?

Description
I receive a "The file 'C:\Windows\system32\config\systemprofile\AppData\Local\NuGet\Cache\Paket\5.92.3\paket.exe' already exists. " error when calling "paket.exe restore" having bumped the version to 5.92.3.
Have never seen this before. It goes away on a brand new build agent (they're automated), but comes back again.
Repro steps
Not yet able to reproduce. Known aspects:

Occurs when calling paket.exe restore
Most branches on build server running on 5.85.1 with paket.exe in magic mode, with version passed as argument.
New branch on build server running 5.92.3 with paket.exe in magic mode, with version passed in paket.dependencies.
Build servers rebuilt from scratch each day, and happened two days running so far.
Works successfully on first run of the day.
Subsequent builds of branch fail on that build server. Not sure if there is interaction with other builds going on here.
Happened once on local machine but haven't been able to reproduce since after clearing cache folder.

Expected behavior
paket.exe restores packages as in previous versions.
Actual behavior
The file 'C:\Windows\system32\config\systemprofile\AppData\Local\NuGet\Cache\Paket\5.92.3\paket.exe' already exists.
Packages are then not restored.
Known workarounds
Kill the build server and force a fresh instance.

system32??? So you are running this stuff on a system account?
What I would expect is that the bootstrapper is cached and then all are using the same exe in the Cache.
In your specific scenario it seems the machines are cleared from time to time?
We really need to repro this or at least track it down. Can you enable logs for the bootstrapper by setting the verbose flag or the PAKET_BOOTSTRAPPER_TRACE environment variable to true (ie PAKET_BOOTSTRAPPER_TRACE=true) and report back the results?

Hi,
We've runned with the same issue.
We're running paket with the following arguments:
.paket\paket.exe restore --verbose
The environment variable has been also set with command (before paket execution :) )
setx PAKET_BOOTSTRAPPER_TRACE true
The only message We've received was:
The file 'C:\Users\XXXX\AppData\Local\NuGet\Cache\Paket\5.92.4\paket.exe' already exists. (Github - cached (temporarily ignore updates))
We're using the user dedicated for TeamCity builder.
The problem also occurs with the install command.
When you remove manually the file C:\Users\XXXX\AppData\Local\NuGet\Cache\Paket\5.92.4\paket.exe the build process runs correctly once. After the next paket restoration the same problem exists, which is not easilly seen, because bootstrap finishes its work with 0 code and the solution is compiled with old references :/.
Best Regards,
MiloszeS

I don't even know what we changed there recently, could that be a side effect from signing the .exe?

I don't know whether it helps. I've switched to the paket.bootstrap. When I execute it manually I've receive the following messages:
Checking Paket version (downloading latest stable)...
Version 5.92.4 found in cache but it's hash isn't valid.
Starting download from https://github.com/fsprojects/Paket/releases/download/5.92.4/paket.exe
Hash of downloaded paket.exe is invalid, retrying once
Hash of downloaded paket.exe still invalid (Using the file anyway)
The file 'C:\Users\XXXXX\AppData\Local\NuGet\Cache\Paket\5.92.4\paket.exe' already exists. (Github - cached)
It looks like a problem with hash validation and exception handling...

@matthid it looks like the side offect of signing.

@forki?

see https://github.com/fsprojects/Paket/issues/2729 wtf

Ok can you please push an unsigned version for now as quick fix?
Will need to investigate next week. /cc @agross
Am 06.09.2017 15:05 schrieb "Matthias Dittrich"<EMAIL_ADDRESS>
see #2729 https://github.com/fsprojects/Paket/issues/2729 wtf
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/fsprojects/Paket/issues/2725#issuecomment-327476761,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AADgNKrrUYHBxE1Y9SrE8-YLWaVOBKnJks5sfpiAgaJpZM4PNBe7
.

We've signed before, also with timestamping. Not sure why a new certificate should change semantics.

probably a bug in the build process. we calculate the hash before signing the exe?
also a bug in the bootstrapper because it doesn't report the actual problem. it should either always warn and work or never work and fail with the correct message imho

I'll leave this open for the bootstrapper problem, as https://github.com/fsprojects/Paket/issues/2729 better describes the invalid hash...

So the immediate problem has been solved #2729 and the hashes match again, please retry.
I'll leave this open as the bootstrapper still needs to be fixed in case this happens again.
A PR is appreciated.


Remove dependency on the PureScript compiler

Hi! I'd like to try this out in Spago (usecase: possible implementation path for https://github.com/spacchetti/spago/issues/89), but I would not like to depend on the whole PureScript compiler, hence this PR 🙂
It looks like the dependency was not in use, but maybe you had plans?

I don't intend for this to be a library, I just need a place to iterate independently. The goal is for this to eventually be merged into the purescript project as the primary parser. Future work involves being able translating the CST to the existing AST, so the dependency will be necessary.

Thanks for the context @natefaubion! I'll close this then 👍


Add downloads badge

We get a pleasantly surprisingly large number of downloads per month.  Let's advertise it

@doug-wade - Look like this conflicts with your other recent badge PRs that just landed on master.

rolled into #160


"Empty call stack" warning for PinNotifyTestcaseStart return

At begin of each trace, the ret of the PinNotifyTestcaseStart function is recorded:
[    0] StackWrite: <wrapper:PinNotifyTestcaseStart+4>, [$+008000AC]
[    1] StackRead: <wrapper:PinNotifyTestcaseStart+7>, [$+008000AC]
[    2] StackRead: <wrapper:PinNotifyTestcaseStart+E>, [$+008000B8]
[    3] Return: <wrapper:PinNotifyTestcaseStart+E> -> <wrapper:TraceFunc+14A>
[    4] StackRead: <wrapper:TraceFunc+14A>, [$+00800350]
[    5] Call: <wrapper:TraceFunc+151> -> <wrapper:_Z9RunTargetP8_IO_FILE+0>
[    6]   StackWrite: <wrapper:_Z9RunTargetP8_IO_FILE+8>, [$+00800098]
...

This leads to warnings during analysis.
The same is true for the call to PinNotifyTestcaseEnd in the very end.

This only happens in the TraceDump module, when the prefix (which contains the call) is omitted.


STYLE ensure that the current selection in the tableView isn't cleared

Current fix ensures that the clicking outside the rows of this tableView doesn't clear the existing selection

Thanks for your contribution :+1:
I am currently reviewing your patch.


Error contrast img

Bug reports:
Error generating image, a contrast is added

Specifications:

html2canvas version tested with: 1.0.0-rc.5
Browser & version: Chrome
Operating system: Windows

I solved this issue by removing the css box-shadow of my div container. It seems to be a problem with this css property.
Try it:
https://jsfiddle.net/3apk0wno/


Electron dependency

Hey, started playing around this recently when I noticed it was downloading electron as a dependency. Is that right? Is electron necessary or did it just include better APIs than were otherwise available?

It looks like electron-webrtc lets you use WebRTC by... downloading and running a full electron+chromium in the background...
In theory, if this shim were based on node-webrtc it wouldn't need Electron/Chromium...


Example of how to manually assign coordinates to run_group

    from mth5.mth5 import MTH5
    from mt_metadata.timeseries.location import Location

    m = MTH5(file_version="0.1.0")
    m.open_mth5("test.h5", mode="w")
    station_group = m.add_station("test1")

    location = Location()
    location.latitude = 17.996

    station_group.metadata.location = location
    run_group = station_group.add_run('001')
    run_group.station_group.metadata.location = location

Setting latitude in the location object has the expected result:
In[12]: location
Out[12]:
{
    "location": {
        "declination.model": "WMM",
        "declination.value": 0.0,
        "elevation": 0.0,
        "latitude": 17.996,
        "longitude": 0.0
    }
}

And the information does propagate to the station_group
In[16]: station_group.metadata.location
Out[16]:
{
    "location": {
        "declination.model": "WMM",
        "declination.value": 0.0,
        "elevation": 0.0,
        "latitude": 17.996,
        "longitude": 0.0
    }
}

But it does not propagate to the run_group
In[20]: run_group.station_group.metadata.location
Out[20]:
{
    "location": {
        "declination.model": "WMM",
        "declination.value": 0.0,
        "elevation": 0.0,
        "latitude": 0.0,
        "longitude": 0.0
    }
}

@kujaku11 Is this expected behaviour?  Or should the run_group inherit the location info from it's station group?

Once this is sorted out, the comments and reference to this issue can be deleted from

aurora/aurora/test_utils/synthetic/make_mth5_from_asc.py

I wound up factoring this mini test out, it is now in its own file in aurora
aurora/sandbox/debug_mt_metadata_issue_85.py
Currently it is on the aurora fix_issue_80 branch

@kkappler  After you change the metadata, you'll have to run the method write_metadata to be sure that it is written the HDF5, then when you call run_group.station_metadata it will pull the newly written metadata from the station metadata.


change var to let and const

changed declaration var of variables that  get re-assigned   to let and const for variables that remain constant .  best practices

@all-contributors please add @Ifunanyacollins   for code


Add missing people and subsystems to MAINTAINERS

The new MAINTAINERS file seemed to be missing a lot of information. This change adds the missing information by adding subsystems, based on MAINTAINERS files in top-level subdirectories in the repository and copying that information in this document.
Sub-directories have not been checked for MAINTAINERS files, so it's possible that additional information should still be added.
Summary of the changes made:

Added @kencochrane to the "registry" subsystem (given that
he is still included in registry/MAINTAINERS).
Added @jfrazelle to the "remote API" subsystem (given that
she is still included in api/MAINTAINERS)
Added entries and maintainers for the subsystems;

daemon
engine
graph
integration
integration-cli
project (formerly known as "hack")
volumes

Added missing information for people;

cpuguy83
fredlf
jamtur01
jpetazzo
kencochrane

Notes;

please check if these changes are correct. I tried to copy the information correctly, but I may have made a slip-up during the process.
I noticed the subsystems weren't in alphabetical order; is that something we want?
I haven't added @icecrime to the API and Daemon subsystems yet (basically, that was the reason I noticed the document was incomplete: https://github.com/docker/docker/pull/10395 was closed to be replaced with adding him to this document, but that never happened)
What will be done with the existing MAINTAINERS files? Should they be removed? I can imagine it becoming a problem to keep them in sync with this document. Also; what is leading? this document, or the separate MAINTAINERS files?

I just noticed there's another PR targeting parts of this: https://github.com/docker/docker/pull/10425/files so some things may overlap.

Also, not sure it's okay for me to add this information, or a task for the maintainers themselves, but hopefully this saves some time :D

@crosbymichael what should happen to the "old" MAINTAINERS documents? Should they be removed, or is the plan to automatically generate them from the "root" MAINTAINERS file? I take it that TOML was chosen for a reason (i.e. to automate something?)
If those files should be generated automatically, I think additional information has to be added to the TOML (i.e., a target-directory where to store the generated MAINTAINERS file)

@thaJeztah needs rebase but i suggest you rebase on top of #10425.

@tiborvass I rebased it on master for now. I attempted to rebase on top of #10425 but that PR needs a rebase as well, so I think I would be wasting time there (and limited Git-foo on my side :)). If the other PR is merged or rebased, I will rebase my PR again.

MAINTAINERS Ping @shykes :)

@thaJeztah Can you please rebase?

@icecrime rebased. I removed @kencochrane from this PR, given that he was not included in #10425 (which was made by the distribution team).
Perhaps we should check if the document is complete before merging https://github.com/docker/docker/pull/11219 because I didn't delve into all sub(sub)directories when creating this.

Ping CM the CM @crosbymichael!

Rebased (again) because https://github.com/docker/docker/pull/11340 was merged. Looks like this PR might end up being just a white-space fix, lolz

Actually we intentionally left these out because they were too fine grained and we wanted to take a more simple approach before carving up the code base.  This is why the current file does not have these subsystems and why I think we should wait until we get a better idea and cleaner lines of responsibility within the engine.

@crosbymichael clear (for the subsystems);
Shouldn't the missing people still be added though, especially @cpuguy83 and @jamtur01; otherwise it won't be clear for people that they are maintainers and their LGTM's won't "count" (if the rules are followed closely)
(Could be I missed that they resigned as maintainer of course)

I am still a maintainer.

@jamtur01 ah, yes, you're in the docs list, but missing in the "persons" list; should I create an updated PR with just the "persons" change to add you to the "persons" list, or will you take care of that yourself?


feat: Implement manytomany relationship

Many to many between Course and AppUser, Student Courses.
One to many between Course and AppUser, Instructor Courses.
One to many between Course and CourseMaterial.

Done with this feature, ready to be merged to main.


Function "View" don't work

Thesedays, I found that function View() didn't work.   The effect of click the  magnifying glass icon in Variable Explorer is same.

I tryed serval tables and all of them can't be displayed. This problem is fully confused me, looking forward to your  reply.
Additional information:
OS Information
Version:       Microsoft Windows NT 10.0.14393.0
RTVS Information:
Assembly: Microsoft.VisualStudio.R.Package, Version=1.0.30228.1316, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a
Active R URI:
Microsoft R Open (3.3.2): C:\Program Files\Microsoft\MRO-3.3.2\

Please try: https://aka.ms/rtvs-1.0rc3 and let us know if that works for you.  thx.

Ok, the problem has been solved, thx.
In addition, hope add a fliter in Data Gird such as RStudio, so  that user can seek intrest data quickly.


Can't get react-annotations to show over a map

Hi, love react-simple-maps!
I have a problem which I cannot solve - I've checked the web for answers but cannot find any.  Hence my logging of an issue.
I have a world map and I want to overlay annotations from react-annotations but they just don't show up.  Do you have an example of this working?
Thanks!

Sorry, my mistake.  I needed to do at least the following, wrap in :
<svg width={800} height={400}>
      <MyAnnotation />
</svg>

Hi @abax1,
Yeah, we didn't get around to add a proper example of how to use react-annotation with react-simple-maps yet. The best way to go about this is to put the annotation component inside a Marker component. That way you can place it geographically.
Hope this helps.

Thanks @zimrick , actually I'm still struggling to get it to display over a geography so any examples will be most welcome.  For now I will stick with the standard annotation out-of-the-box.
Thanks again.  Love this library!

I will keep this issue open for now, and post a link to the example when it's ready.
Glad you like the library :)

MyAnnotation.jsx:
import React, { useState } from "react";
import {
  AnnotationLabel,
  EditableAnnotation,
  ConnectorLine,
  ConnectorEndDot,
  Note
} from "react-annotation";

const MyAnnotation = ({ title, label, x, y, labelX, labelY }) => {
  const [values, setVaules] = useState({});
  return (
    <div>
      <EditableAnnotation
        x={150}
        y={170}
        dy={117}
        dx={162}
        color={"#59039c"}
        title={title}
        label={label}
        className="show-bg"
      >
        <ConnectorLine>
          <ConnectorEndDot />
        </ConnectorLine>
        <Note
          align={"middle"}
          orientation={"topBottom"}
          bgPadding={20}
          padding={15}
          titleColor={"#9610ff"}
          lineType={null}
        />
      </EditableAnnotation>
    </div>
  );
};

export default MyAnnotation;

Hmm, I think the div in MyAnnotation will not work well within an svg. You could replace it with a React.Fragment or a g. It should also not be necessary to add an svg tag inside the Marker.
I am currently experimenting with react-simple-maps and react-annotation here:
https://codesandbox.io/s/annotations-with-react-annotation-ctl55
Let me know if this is useful to you.

This helped thankyou.  It works if I add the react-annotation component directly. e.g.
<Marker coordinates={[-120.8958, 38.8375]}>
          <circle r={2} fill="#9610ff" />
          <circle r={3} fill="none" stroke="#9610ff" strokeWidth={0.5} />
          <AnnotationLabel
            x={0}
            y={0}
            dy={117}
            dx={162}
            color={"#1A237E"}
            className="show-bg"
            editMode={true}
            note={{
              title: "Annotations :)",
              label: "Longer text to show text wrapping",
              align: "middle",
              orientation: "topBottom",
              bgPadding: 20,
              padding: 15,
              titleColor: "#1A237E",
             lineType:"vertical"
            }}
          />
        </Marker>

@zimrick, all working now.  Thanks again for your help.  I also have a pulsing circle marker now which others might find useful:-
CirclePulse.css
.circle-pulse {
  stroke:greenyellow;
  stroke-width: 2px;
  stroke-opacity: 1;
}

.pulse {
  fill: rgb(248, 248, 248);
  fill-opacity: 0;
  transform-origin: 50% 50%;
  animation-duration: 2s;
  animation-name: pulse;
  animation-iteration-count: infinite;
}

@keyframes pulse {
  from {
    stroke-width: 3px;
    stroke-opacity: 1;
    transform: scale(0.3);
  }
  to {
    stroke-width: 0;
    stroke-opacity: 0;
    transform: scale(2);
  }
}

CirclePulse.jsx
import React from "react";

import "./CirclePulse.css";

const CirclePulse = ({ color, pulseColor }) => {
  return (
    <>
      <g transform="translate(-50, -50)">
        <svg height="100px" width="100px">
          <circle
            style={{ stroke: pulseColor }}
            className="circle-pulse"
            cx="50%"
            cy="50%"
            r="7px"
            fill={color}
          />
          <circle
            style={{ stroke: pulseColor }}
            className="circle-pulse pulse"
            cx="50%"
            cy="50%"
            r="10px"
          />
        </svg>
      </g>
    </>
  );
};

export default CirclePulse;

Usage would be:
<Marker coordinates={[-120.8958, 38.8375]}>
          <CirclePulse color="red" pulseColor="yellow" />
          <MyAnnotation
            title={"You are Here"}
            label={"This is your node, N. California, Lat: -120.8958, Long: 38.8375"}
            color={"#ffffff"}
            wrap={60}
          />
</Marker>


bind(to: ) causes a crash after updating Bond

Previously I was using Bond version 7.3.3, and everything was working just fine when I set up my viewModel bindings, but now for some reason after updating Bond via pod update, the app crashes when I call the bind method. Here is the code snippet causing the crash -
self.viewModel?.currentTool.map { $0 == nil ? false : true }.bind(to: self.reactive.isHidden)
Am I missing something?

I don’t see anything that should crash on that line. Could you share some more details like the stack trace?

Sure -

I am basically using a propertyobserver 'didSet' for when the viewModel is set for the view, and setting up bindings there

dyld_stub_binder indicates some linking issues. Could be something with CocoaPods. I'd also try deleting derived data if you haven't tried that already.

Perfect! Deleting derived data worked :)


undefined is not iterable (cannot read property Symbol(Symbol.iterator)) when trying to add a style

fixed in this pr https://github.com/1904labs/ol-kit/pull/16


Q: what's the total estimated cost of the hardware?

Just want to get a rough idea.
Thanks.

The price of the robot is difficult to estimate precisely since it will depend on the suppliers and machine shops that you use, the country you are in and the part quantities that you are ordering.
We have estimated the material cost for our Solo12 robot at around 6300€ + tax.
That is not including the imu which is the most expensive part on the robot.
PAL Robotics offers a kit with all the parts and alternatively a preassembled robot.
You can request a quote here: https://solo.pal-robotics.com/solo

Hi Pascal, check out https://odri.discourse.group/categories for a discussion forum.

@easybuildtech - this is perfect - thank you.   I'm a huge fan of the University of Maryland where many of my interns matriculate.   Some of them are at JPL now (Nick R and Camden S.).    I'm liking what UMCP is posting here - it's consistent with my initial observations.

Hi, thanks for your interest!
indeed the forum is the good place to discuss such topic.
I can start here with a few element of answer.
Despite being mostly 3d printed, solo and other ODRI robots are not low cost. At least not optimized for cost.
Those robots are used as platform to experiment and develop state of the art research. For academia, it make much more sense to invest in quality sensor and cutting edge compact electronic that ease our life while experimenting.
I can understand how this can be frustrating from a hobbyist point of view, but I think you can find all the resources on this GitHub and in the forum to replace some components, design different actuators, etc.  Actually, you will find in the forum some threads about lower cost alternative to the choices made in the project.
About specific question / suggestion:

Yes a low cost IMU could be used, with a more complex state estimation and calibration, but I would still expect a decrease of performances.
The master-board is a compact design. Using a shield and standard dev board is possible but would take more space, In Solo12 we have very few margin. The Ethernet is a must to do high frequency feedback loop with a computer.
The motor could be different, but some CAD adjustment are needed to accommodate with different size, and fixture.
The cost is also due to the custom electronic to control the motor, Other lower cost project exist, but are not as compact nor adapted to our need.

Please understand that we would love to see spin-off of this project optimizing cost, exploring different design choices but we can not invest our work time to do it our-self.
I hope this helps,
Thomas.

What is the estimated cost of the biped hardware (if you don't count the 3d printed parts)?

@MEE2022 I have been noting costs, on a BOM, as I build if it helps:  https://easybuildtech.com/open-dynamic-robot-initiative-bolt-cost-breakdown/
I am in the U.S.

Thank you very much.  This is very detailed and extremely helpful.
On Thu, Nov 24, 2022, 9:13 AM Benjamin @.***> wrote:

@MEE2022 https://github.com/MEE2022 I have been noting costs, on a BOM,
as I build if it helps:
https://easybuildtech.com/open-dynamic-robot-initiative-bolt-cost-breakdown/
I am in the U.S.
—
Reply to this email directly, view it on GitHub
https://github.com/open-dynamic-robot-initiative/open_robot_actuator_hardware/issues/50#issuecomment-1326578748,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A2GU43Y7TU24NQT3YUMDV4DWJ6A3NANCNFSM6AAAAAAQ5YGWZQ
.
You are receiving this because you were mentioned.Message ID:
<open-dynamic-robot-initiative/open_robot_actuator_hardware/issues/50/1326578748
@github.com>

@easybuildtech
Great! Thanks for sharing!


Data missing in adapted file version

In version 1.0.4 crucial data is missing after being created from the master file. I am working on a fix.

I made a temporary fix in version 1.0.5. So please update to this version for correct scan results.
Only the latest version vulnerability is in the adapted file for now.

There is now a updated version where version compare works correct. See version 1.0.6. Please download and install again.

It was closed ;-)


[Feature Request] Bumping gymnasium to v1.0.0 and gymnasium-robotics to v1.3

Required prerequisites

[X] I have searched the Issue Tracker and Discussions that this hasn't already been reported. (+1 or comment there if it has.)
[ ] Consider asking first in a Discussion.

Motivation

Recently, gymnasium v0.29 was deprecated (see here for release notes). There have been many substantial additions and improvements (especially in VecEnv and MujocoEnv), and this is the final change in core API (i.e. no more breaking changes from hereon).
I believe it would be really great if we could migrate this repository to deprecate gymnasium v0.29 too. Are there any plans to do so? I would be more than happy to assist!
Thanks
Solution
We may refer to the change-log and migrating guide for reference.
Alternatives
No response
Additional context

Your input on this issue is very valuable to us. Gymnasium is an important dependency, and thanks to your observation, I’ve taken a closer look at the recent updates. This is quite a significant change and may require us to assess the potential workload. I’m glad you're willing to assist with this.

Hi, so I just glanced over the code. Yes, there would be substantial changes. Regardless, I will try to make a put up a draft PR with some minimal changes to get v1.0 at least running, as soon as I get time. We can start from there I suppose.
Thanks

Your enthusiasm is truly encouraging for us, and I believe this is a great starting point for collaboration.


Wildling Status Effects

Provide Wildlings clan with persistent status effects.
AC:

[ ] Add +5 run skill
[ ] Add 25% stamina debuff while sprinting
[ ] Add 2% health regen when in DARK_FOREST biome

Need confirmation from azazel on de/buffs for Wildlings

no health regen, +5 is probably fine if we really are buffing them..


More examples

The readme only shows how to plot with GR through Plots.jl. Are other options possible? If yes, it would be very nice to see them demonstrated in the readme?

Sure! Most plotting packages should work. Here is an example for Makie:
julia> using CairoMakie, SixelTerm
julia> scatter(rand(100))

I'll update the README, thanks for the pointer.
Is there a plotting package you prefer using?

I am looking into something lightweight and low-dependency, for live-streaming an oscilloscope trace via a Raspberry Pi, to a controlling laptop via ssh. It looks like using GR directly is a good option, which happens to support sixel nativly.
I normally use Makie, but I believe it is too heavy to be a reasonable choice to be run on the Pi.
Peyhaps Bookeh would be interesting? I remeber it got supeisingly much attention for its release. Otherwise, I generally believe low latency and low-cost plots are important where one wants to use the terminal, but I have my biases...

I think GR would be a good choice because it's implemented via a C (?) library.
Bokeh looks as if it was implemented in Javascript, which might not be "low latency" or "low-cost". I quick look also didn't show any backend that produces a figure; instead, it seems to be looking for a place to display html.

Right. I am not sure then which other options would be nice.

I added the example.


Crazy CPU usage on Mac (edit: also inside Docker container)

Hi,
I'm trying godev for the first time. It looks like the godev process is generating an astounding load on the CPU (over 1.00). I wonder if you have an infinite loop somewhere?
Edit: Also happens inside a Docker container

I've just started running godev through Docker (using the zephinzer/godev:go-1.12 image) and I am seeing the same thing!

Thanks for the report! I will look into this-

Seems like there's a channel leak at handleStart, after doing 4 live-reloads, the following appears in the profiling:
goroutine profile: total 17
5 @ 0x42d72b 0x42d7d3 0x404e0d 0x404be5 0x753473 0x45ab61
#	0x753472	main.(*Command).handleStart+0x62	/REDACTED/godev/command.go:220

I previously wrote some code to watch files in a similar manner and found it burned CPU like this, until I added a sleep to each watch loop. I tried doing so in all the loops I could find, but it didn't seem to reduce the CPU usage :/

Thanks for investigating @crabmusket, I’m seeing if I can rewrite some parts to use channels in a more “go” way, realised I may have done some things wrongly after attending a workshop that covers these sort of things!


memory leak in mbedtls_x509_csr_parse

Note: This is just a template, so feel free to use/remove the unnecessary things
Description

Type: Bug
Priority: Major
Raised by @catenacyber through mail.
Memory leak in mbedtls_x509_csr_parse(): in case mbedtls_x509_csr_parse_der() fails, pem is not freed.
This issue is to track PR #1621

Bug
OS
Mbed linux
mbed TLS build:
Version: development branch
OS version: x.x.x
Configuration: please attach config.h file where possible
Compiler and options (if you used a pre-built binary, please indicate how you obtained it):
Additional environment information:
Discovered by oss fuzzer tool

ARM Internal Ref: IOTSSL-2287

PR's to address this problem - development #1621, mbedtls-2.7 #1772, mbedtls-2.1 #1774 - have now been merged. This issue can now be closed.


merge bq2geojson into piecewise repo

This PR addresses issue #57, pulling in all relevant front-end work previously housed in the bq2geojson repo. All front-end work is now saved in the piecewise_web folder, with the exception of an htaccess file used to provide authentication to piecewise_web/admin and the /retrieve endpoint of the collector.

Hey Chris.  This PR changes 59 files and adds around 6,000 new lines of code. It'll be pretty much impossible for me to actually review the code, but since presumably this is working code with only a few modifications, then it should be okay.  I would say just go ahead and merge this, and then test and we can fix breakage as it comes up.  Except for my one comment the htpasswd file... LGTM.

Sounds good. Yes, this is just moving all files from bq2geojson into
piecewise_web, so there's a lot of files. There are minimal files that were
actually changed, mostly new ones.
I'll merge it in after removing the htpasswd file from version control.
Thanks,
Chris
Chris Ritzo
Senior Technologist, Open Technology Institute @ New America
740 15th Street NW, Suite 900
Washington, DC 20036
Public Key:    0x085B309C
On Tue, Mar 15, 2016 at 3:30 PM, nkinkade<EMAIL_ADDRESS>wrote:

Hey Chris. This PR changes 59 files and adds around 6,000 new lines of
code. It'll be pretty much impossible for me to actually review the code,
but since presumably this is working code with only a few modifications,
then it should be okay. I would say just go ahead and merge this, and then
test and we can fix breakage as it comes up. Except for my one comment the
htpasswd file... LGTM.
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/opentechinstitute/piecewise/pull/60#issuecomment-196986555


/backend folder in the tutorial does not exist

Brand new user here.
I tried to follow this guide exactly: https://serverless-stack.com/examples/how-to-create-a-reactjs-app-with-serverless.html
The /backend  folder was not created, have things changed and does the tutorial need fixing?

The tutorial needs fixing, it's now called /api. Though i guess you can change the folder name to whichever you'd like and set srcPath in stacks/index.js to the new name.

Thanks for bringing it up!
Fixed.


"Bulk" fetching

Allow a client to fetch a list of documents by key in a single request.

In my use case, I'd like to avoid using a predicate to evaluate the full set of
keys.  Rather, I'd like the server to simply iterate over the list of keys I
provide and get the single document associated with each, and then return the
resulting list of documents.

E.g., Client: "Get documents 2, 6, 7, and 15."  Server: "Okay, here are three
documents: 2, 7, and 15 (I didn't find '6')."  Or simply, "Okay, here are
documents 2, 7, and 15" (leaving the client to simply recognize that since '6'
was missing, it must not have been found.)

Presently, I am doing this by rapidly sending a sequence of GET requests one by
one, which seems inefficient.

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 16 Mar 2011 at 4:26

The Java Client will take a set of strings: I'll add bulk put/get APIs in the
next few days.

Original comment by<EMAIL_ADDRESS>on 20 Mar 2011 at 9:48

Implemented by exposing the following HTTP operation: POST /bucket/bulk/get
It accepts a json array as the request payload, containing (string) keys to
fetch, i.e.: ["1", "3"]

Feedback would be very welcome!

Original comment by<EMAIL_ADDRESS>on 19 Mar 2011 at 7:21

Changed state: Fixed

Original comment by<EMAIL_ADDRESS>on 16 Mar 2011 at 1:35

Changed state: Accepted
Added labels: Type-Feature, Priority-Medium, Project-Terrastore, Milestone-0.8.2

Sounds good to me.  I am using the Java API, which I suspect will expose a
method that takes a List of String ids, correct?

Original comment by<EMAIL_ADDRESS>on 19 Mar 2011 at 11:45

Implemented on Java Client too: see issue 172.

Original comment by<EMAIL_ADDRESS>on 26 Mar 2011 at 4:57


PhpUnitStrictFixer - Do not fix if not correct # of arguments are used.

closes https://github.com/FriendsOfPHP/PHP-CS-Fixer/issues/3683

note to self for other PR for this fixer; we could/should also only fix (static)method calls within classes/traits we think are (part of)  PHPUnit classes

Thank you @SpacePossum.


Not working on GitHub codespace

Now you can open any repository in codespace using . key shortcut. See https://twitter.com/github/status/1425505817827151872?s=20
When my setting is synced by signing in this extension is activated but still, all icons are missing. Any opportunity to get this extension working in codespace?

Thank you for informing me about this. Right now I'm not sure why it's not working properly. In the browser console I get an error like this:

Refused to load the font 'https://pkief.vscode-unpkg.net/PKief/material-product-icons/1.0.4/extension/theme/material-icons.woff' because it violates the following Content Security Policy directive: "font-src 'self'

@PKief Also keep an eye on your other project for the file icons. It is also not being able to install in VSCode web :)

I already checked that, but got the information from @aeschli that it will work for the web version if I have set the extensionKind to
"extensionKind": ["ui", "workspace"]

Not sure why it's not availabe to be used in code spaces.

@PKief For an extension with code to run in the browser only environment it needs a new entry point browser. In the one will be used instead of main. All code needs to be packaged to a single file and the only require possible is for vscode.
We are working on the documentation for this.

@PKief A little update, this extension works well in Codespaces which I had tried it just now. But unfortunately VSCode icons the next project is missing to work on Codespaces.
For the Browser VSCode installation which is open by pressing . key, I was unable to install VSCode icons but Product icons seems installed but won't work.

Seems to work now:


failed to run ceph in openEuler 20.03 (LTS)

Description
NAME="openEuler"
VERSION="20.03 (LTS)"
ID="openEuler"
VERSION_ID="20.03"
PRETTY_NAME="openEuler 20.03 (LTS)"
ANSI_COLOR="0;31"
kernel:
4.19.90-2<IP_ADDRESS>6.oe1.x86_64
kubernetes v1.22.16
container v1.6.9
runc v1.1.4
When running rook-ceph operator, a process call exception occurs when creating an osd pool.ms_dispatch and fn_monstore many bad file descriptor.
We use cri-o crun everything works fine
Steps to reproduce the issue
1.run rook-ceph, init ceph cluster
2.View the process of the mon leader node
3.create osd pool
Describe the results you received and expected
Create osd pool normally
What version of containerd are you using?
v1.6.9
Any other relevant information
NAME="openEuler"
VERSION="20.03 (LTS)"
ID="openEuler"
VERSION_ID="20.03"
PRETTY_NAME="openEuler 20.03 (LTS)"
ANSI_COLOR="0;31"
kernel:
4.19.90-2<IP_ADDRESS>6.oe1.x86_64
kubernetes v1.22.16
container v1.6.9
runc v1.1.4
Show configuration if it is related to CRI plugin.
No response

Hi @clcc2019, we don't have open-euler env and there is no more detail in the description. Mark it as support.
please provide more detail and reproduce steps. thanks

Hi @clcc2019, we don't have open-euler env and there is no more detail in the description. Mark it as support. please provide more detail and reproduce steps. thanks

https://github.com/rook/rook/issues/10110,The problem is described in detail

Hi @clcc2019, we don't have open-euler env and there is no more detail in the description. Mark it as support. please provide more detail and reproduce steps. thanks

rook/rook#10110 problem is described in detail

There is no any configuration about containerd and it is not openEuler env. Please provide your reproduce steps instead of link and follow the issue reporter guide. Thanks

I think reproduction is a cumbersome process, I can provide my environment for you to find the problem, is that okay?

@clcc2019 I'm going to close it. Please feel free to reopen it if you provide more information.


chore: use shillelagh instead of gsheetsdb

SUMMARY

shillelagh is a Python module that provides a SQLAlchemy dialect and DB API 2.0 implementation to a variety of non-SQL resources. For example, it can be used to query CSV files or fetch weather data from https://www.weatherapi.com/ via SQL. The library has 100% unit test coverage and uses SQLite virtual tables to allow efficiently accessing the resources as if they were tables in a database.
The current module that we use to access Google Spreadsheets is gsheetsdb, which depends on the unmaintained moz-sql-parser. In addition to the dependency, gsheetsdb implements a code transpiler, and uses pre and post-processors to fetch data from the spreadsheet, making the code somewhat brittle.
This week I implemented a Google Spreadsheets adapter in shillelagh, aiming to deprecate the gsheetsdb module. The implementation is a drop-in replacement, and users should be able to uninstall the gsheetsdb module and install the shillelagh module to have existing queries, datasets and charts continue working.
This works because shillelagh registers a SQLAlchemy dialect called "gsheets". This way SQLAlchemy will use shillelagh for all engines with the URL "gsheets://". This is an in-memory only backend, without access to the filesystem, so it's safe to use.
On the other hand, shillelagh also registers its own dialect (aptly named "shillelagh", with an alias called "shillelagh+apsw"). That dialect is unsafe, as a malicious user could use it to read data from the filesystem by enabling the CSV adapter. For security reasons, I disabled the "shillelagh" dialect unless PREVENT_UNSAFE_DB_CONNECTIONS is on.
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

N/A
TEST PLAN

Tested SQL Editor and charts reading from Google spreadsheets, everything works.
ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Changes UI
[ ] Requires DB Migration.
[ ] Confirm DB Migration upgrade and downgrade tested.
[ ] Introduces new feature or API
[ ] Removes existing feature or API

Codecov Report

Merging #13185 (7eaa91f) into master (2ce7982) will increase coverage by 2.10%.
The diff coverage is 71.53%.

@@            Coverage Diff             @@
##           master   #13185      +/-   ##
==========================================
+ Coverage   53.06%   55.16%   +2.10%
==========================================
  Files         489      469      -20
  Lines       17314    15873    -1441
  Branches     4482     4076     -406
==========================================
- Hits         9187     8757     -430
+ Misses       8127     7116    -1011

Flag
Coverage Δ

cypress
55.16% <71.53%> (+2.10%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

...end/src/SqlLab/components/RunQueryActionButton.tsx
52.77% <ø> (ø)

superset-frontend/src/chart/ChartContainer.jsx
100.00% <ø> (ø)

superset-frontend/src/chart/ChartRenderer.jsx
75.67% <0.00%> (-1.04%)
:arrow_down:

...perset-frontend/src/common/components/Dropdown.tsx
50.00% <ø> (ø)

...rontend/src/components/ListView/CardSortSelect.tsx
78.94% <ø> (ø)

...ontend/src/components/ListViewCard/ImageLoader.tsx
75.00% <0.00%> (ø)

...set-frontend/src/components/URLShortLinkButton.jsx
72.22% <ø> (-27.78%)
:arrow_down:

.../src/components/dataViewCommon/TableCollection.tsx
97.87% <ø> (ø)

...et-frontend/src/dashboard/actions/sliceEntities.js
70.96% <ø> (ø)

...src/dashboard/components/HeaderActionsDropdown.jsx
68.91% <ø> (+1.35%)
:arrow_up:

... and 156 more

Continue to review full report at Codecov.

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

While fixing unit tests I did some clean up and removed duplicated code and duplicated exceptions.


[Android]Modal becomes invalid state when power button is pressed.

[x] Review the documentation: https://facebook.github.io/react-native
[x] Search for existing issues: https://github.com/facebook/react-native/issues
[x] Use the latest React Native release: https://github.com/facebook/react-native/releases

Environment
Run react-native info in your terminal and paste its contents here.
Environment:
  OS: macOS High Sierra 10.13.4
  Node: 8.3.0
  Yarn: 1.6.0
  npm: 5.6.0
  Watchman: 4.7.0
  Xcode: Xcode 9.4.1 Build version 9F2000
  Android Studio: 3.1 AI-173.4819257

Packages: (wanted => installed)
  react: 16.3.1 => 16.3.1
  react-native: 0.55.4 => 0.55.4

Description
This issue only confirmed in Android 7.x
Pressing power button when Modal component is displayed causes invalid state of Modal.
I see Modal contents disappears but its transparent  background remains after Pressing power button again and screens turns on.
With Android 6.x or 8.x, Both modal contents won't be hidden with the same operation.

Reproducible Demo
The code is almost same as Modal official document
https://github.com/satoshikumano/RN-Modal-Test

I also met this problem, how to solve?

My workaround is subscribe to AppState and hide modal if AppState is not active.
In this way, When the app come back to foreground, Modal is re-rendered and problem is recovered.

@satoshikumano, you sir, deserve a medal.


Fix model pixel value in internal builds

Task/Issue URL: https://app.asana.com/0/1199371158290272/1201674863186125/f
Description
In internal builds, build flavor is send as model value instead of model.
Fix that
Steps to test this PR
code reivew

Current dependencies on/for this PR:

develop

PR #1680  (this PR)

This comment was auto-generated by Graphite.