Allow for cluster healing

In the event of a cluster node dying, provide a way for administrators to create a replacement of the lost node (with a full re-synchronization of any shards that were there originally).

+1
Any idea how is this supposed to be handled at the moment? Currently if one of my nodes dies, it's pretty much listed in SHOW SERVERS forever.
Also is there some way how to get info about cluster health? In terms of shards and their replication? Currently I don't know any way how to get good visibility about what's happening with the cluster.

@stepanstipl You should be able to drop the server from the cluster using the commands:

drop meta server <ID>
drop data server <ID>

Using the server ID's from the output of a SHOW SERVERS command. This should prevent it from showing up in further output, though I don't believe you will be able to add a new server with the same hostname from that point onward.
Regarding:

Also is there some way how to get info about cluster health? In terms of shards and their replication?

Unfortunately not at the moment, which is why I created #5584. There is some information in the _internal database, but it's a bit hard to make sense of.

@rossmcdonald thanks, will give that a try :)

+1

Now that clustering has been removed from open-source InfluxDB, this issue will now be closed.


pkcs1v15: make *_with_prefix methods the default

Renames the following:

SigningKey::new => SigningKey::new_unprefixed
SigningKey::new_with_prefix => SigningKey::new
VerifyingKey::new => VerifyingKey::new_unprefixed
VerifyingKey::new_with_prefix => VerifyingKey::new

The *_with_prefix methods are preserved with a deprecation warning, which should help people migrate to the new versions.
Closes #238

Per #238 having an empty prefix seems to be quite rare.
People are reaching for new expecting it to work, because in most other environments prefixes seem to "just work" by default.
This change might be confusing for anyone who happens to actually be using empty prefixes, though I haven't encountered anyone who is personally.

cc @lumag

Ack, LGTM.


Small gramatical mistake

Pull Request Test Coverage Report for Build 1132

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 52.844%

Totals

Change from base Build 1131:
0.0%

Covered Lines:
3067

Relevant Lines:
5712

💛  - Coveralls


chore: update the localstack logo

:scroll: Description
Updating the LocalStack logo.
:bulb: Motivation and Context
The LocalStack logo on the README.md is outdated.
:pencil: Checklist

[ ] I reviewed submitted code
[ ] I added tests to verify changes
[ ] I updated reference documentation to reflect the change
[ ] All tests passing
[X] No breaking changes

Thanks @HarshCasper !


Debug: A debug PR to test Jenkins. Don't merge me in

-65535

test this please

test this please

test this please

test this please

test this please

test this please

Lol... 65535


Add citekey pattern using the tex.shortauthor extra field with [auth:lower] fallback to docs

As I'm currently using APA 7th edition, I extensively utilize Zotero's "cheater syntax" to define the tex.shortauthor in the extra field. After reading #149, I spent some time designing a citekey pattern that uses the tex.shortauthor if present and defaults to [auth:lower] if not:
[Extra:transliterate:replace=(?\:tex\\.shortauthor\[\:\=\]\\s+(\\w+))|.*,$1,regex:clean:lower][>0][year] | [auth:lower][year]

I admit, using a regex replace is not very elegant, but it works, and maybe it is useful for others. Hence, the PR to include it in the documentation.
EDIT: Just after this PR, I noticed that when using the previous pattern sometimes $1 referred to the whole input/extra field. This issue is now fixed simplifying the pattern but now requiring a clean after the replace.

This can also be done using
[extra=tex.shortauthor:clean:lower][>0][year] | [auth:lower][year]


Average deposit and withdraw values

Overview
Added side effects for the withdraw and deposit transaction to increase & decrease the locked volume and update the values for average deposit and withdraw,
This should be merged after #77
Key requirement added with the PR

Average deposit amount

Example Graphql Queries
VAnchor AvergeValues

query vAnhorAverageValues{
  vanchors{
    id

    averageWithdrawAmount
    numberOfWithdraws
    maxWithdrawAmount
    minWithdrawAmount

    averageDepositAmount
    numberOfDeposits
    maxDepositAmount
    minDepositAmount
  }
}

Response
{
  "data": {
    "vanchors": [
      {
        "id": "0x5d3b0d1ac094a8551bab3818eff758dc1893e6c7",
        "averageWithdrawAmount": "0",
        "numberOfWithdraws": "0",
        "maxWithdrawAmount": "0",
        "minWithdrawAmount": "0",
        "averageDepositAmount": "10000000000",
        "numberOfDeposits": "4",
        "maxDepositAmount": "10000000000",
        "minDepositAmount": "0"
      }
    ]
  }
}

@dutterbutter  The way average values are calculated is by volume locked / number of deposits

@AhmedKorim please resolve conflicts

@dutterbutter  done


Move validation problems into result response

Short Description

Before, the validation problems were stored in the error_message column of the result table. However, as this had a VARCHAR datatype they were stored like this: "{"Error 1", "Error 2"}
Now, they instead get stored in the result column

Changes

Store validation_problems as dict in result + return as object
Adapt test for non-validation errors no not include validation_problems

Feedback

Do we have any end-to-end tests that I should adapt to be sure this actually works as intended? Because so far, I think e.g. the ResponseDto is never actually tested?

How to review this Pull Request
Link to Confluence (internal)

And surely we would need a new integration test where we maybe send a tax declaration with a validation error (like account holder person_b without defining person_b data?) and then test that the state is FAILED and the validations problems are in the result attribute?

Added precisely that test. Works like a charm (after merging the retry PR #103).


Improve the navbar for better user interface.

The website is pretty fantastic, just the navbar seemed little bit out of shape for me as a user and I would like it to be more interactive and perform better css effects.

@RonitBaranwal
Sure! You can start working on this issue.
Kindly read the Contributing Guidelines carefully.
Thank You

Hey @AnupK1234  i have made the necessary changes to make the navbar  better .Here's a video about it . Please tell me if some other changes are required .
https://github.com/AnupK1234/MealMatters/assets/123174522/5cb280fe-ac9b-45bb-83be-6a4f3349794d

Still logo and options are not in a equal line . I can fix it and i am eager to do it . Can you please assign it to me.

I Fixed it , if you assign me if will ready to pull it to you

Hey @AnupK1234 i have made the necessary changes to make the navbar better .Here's a video about it . Please tell me if some other changes are required .
Vite.React.-.Brave.2024-01-04.16-42-13.mp4

Increase the width of Login button as it was before. Also try if you can style the dropdown if it looks good.

@harith-hacky03 very well it seems you worked for it my friend , go for it . I am unassigning myself . Ask the owner of the project regarding your design :).

Can you assign me i completed all the required changes?

hey i want to contribute to this issue if you can please assign this task to me.
thank you

Can you assign me i completed all the required changes?

Sure you can start working on this issue

@AnupK1234 I created PR can you please check it . Inform me is there any changes needed. Thank you .

live link of this website

hey the navbar is fixed or not

@Ayusht777 It's fixed and submitted PR too

@harith-hacky03 so it is responsive now

@Ayusht777 The whole website is not responsible and if it like navbar is responsive can I fix the whole website to be responsive?

@Ayusht777 I will finish it and submit the PR within today

I would like to refactor the code after merging your pull request.

Who would like to work on this issue ?


(feat): Support --hook-mode for helm template command

helm template command supports --no-hooks flag. But I would like to pass a flag --mode=install to only render install hooks. Similarly for uninstall. Can this feature be supported in helm ?

Hi, @tamalsaha - thank you for your idea. If you'd like, please file a Helm Improvement Proposal to discuss it with the community (or send a pull request if you'd like to implement it). Meanwhile, I'm closing this issue as inactive.


Base host duplicate package checker should have no exceptions

We should aspire to have no duplicate packages for FF dependencies since this bloats dependencies. Removing all excpetions from base host webpack config since it appears to compile in this state.

repo-policy-check passed; https://dev.azure.com/fluidframework/public/_build/results?buildId=7704&view=results
client build passed: https://dev.azure.com/fluidframework/public/_build/results?buildId=7705&view=results


Bug in roundHandlesBoundaryConditions

In roundHandlesBoundaryConditions, I think the the -0.5 should be positive in the second half (i.e. Math.round(-0.5 + (Number.EPSILON / 3.99)) === 1).

In any case, the code currently replaces Math.round if Math.round(-0.4999...) isn't equal to 1.

Can you provide a test case that you think should pass, that fails with our current code?
Indeed I see Math.round(-0.5 + (Number.EPSILON / 3.99)) equalling -0 and Math.round(0.5 + (Number.EPSILON / 3.99)) equalling 1 in most browsers.
Per https://github.com/paulmillr/es6-shim/blob/master/es6-shim.js#L2071-L2072, I checked Safari 8, IE 11, and Opera 12, and got the same results.
git spelunking reveals https://github.com/paulmillr/es6-shim/commit/55dc1fa1e58753b3ee1a4162bbc60b66cb47099b → https://github.com/paulmillr/es6-shim/commit/a20e99fe5876f055b7455dd63a65b3cdc3122546 → https://github.com/paulmillr/es6-shim/commit/90c803f68390dd13fd5297b1e2d54d44f8dac94b, in which I reference https://github.com/kangax/compat-table/issues/392#issuecomment-70381406, in which @Yaffle indicated this is how the test should go.
@Yaffle, could you confirm that this is the right fix here?

the right fix is to remove the second equals operator:
-var roundHandlesBoundaryConditions = Math.round(0.5 - (Number.EPSILON / 4)) === 0 &&
-    Math.round(-0.5 + (Number.EPSILON / 3.99)) === 1;
+var roundHandlesBoundaryConditions = Math.round(0.5 - (Number.EPSILON / 4)) === 0;

It seems it was added by mistake.

There was definitely a reason that I added the second condition, but if we can't reproduce what it was, then it's probably worth removing it. I'll do so when I have time to rerun tests on all the browsers and versions.

@ljharb which tests you need to rerun? Can I help you?


Rundeck 2.7.x

This PR changes the following

Profile file changed to support rundeck 2.7.x. Backward compatibility with 2.6.x mantained

@pcross616 Kitchen tests are failing on ubuntu 16.04. I think that has nothing to with this PR as it can't find /sbin/status command. May be because it does not support upstart anymore. Also there are certain issues reported which makes rundeck 2.7.x a bit risky. See https://github.com/rundeck/rundeck/issues/2300. Although this PR gives us the possibility to install rundeck 2.7.x and it works with 2.6.x as well (tested with 2.6.9), I have not changed the version of rundeck in attribute.

@pcross616 Its up for your review and can be merged

@pcross616  Seems like centos 7 is not working with rundeck 2.7.x. So my last commit fixes the kitchen tests for centos 7. But I think we can merge this PR as it works both with 2.6.x and 2.7.x

do we need all these config differences for 2.7?
I think this pr can be much simpler, in templates/default/web.xml.erb you can test @rundeck_version using rubygems version parsing

I think we should bump the rpm to rundeck 2.7.1 (stable) in attributes/default.rb (without that I dont see how you are testing rundeck 2.7 anywhere)

@atheiman I shall check the rubygem version parsing. The reason why did not bumped the rundeck version is because its making the kitchen test fail for centos 7.

@atheiman The configuration changes pertaining to profile file is required.

is 2.7 working on centos 7 with vagrant? Kitchen-dokken in travis is a little rough with the services, but i would expect it to work fine with systemd in vagrant.  But could not do it because there is virtualbox errors.
mount -t vboxsf -o uid=1000,gid=1000 tmp_omnibus_cache /tmp/omnibus/cache
Tried googling and could not solve it. Did not have any environment where I can test this. Could you try run the test?

@pcross616 Now the Kitchen tests are failing with rundeck 2.7.2 for centos 7. I can revert the default version in attribute to 2.6.11. The changes that I did in the PR works both on 2.6.x and 2.7.x. So it gives you ability to install rundeck 2.7.x. Due to lack of environment and time I can't test the centos 7. I think once this PR is merged with older version of rundeck in attribute someone can work on the centos 7

@atheiman I think we can use the 2.6.x rundeck version for CentOS and fix the CentOS stuff in a separate PR. I think it should not stop us merging this PR. What do you say?

I disagree, we should make 2.7 work everywhere and show that it works with
tests.
On Wed, Feb 8, 2017 at 5:34 AM Nilanjan Roy<EMAIL_ADDRESS>wrote:

@atheiman https://github.com/atheiman I think we can use the 2.6.x
rundeck version for CentOS and fix the CentOS stuff in a separate PR. I
think it should not stop us merging this PR. What do you say?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Webtrends/rundeck/pull/139#issuecomment-278303982,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFqEQ7ypX9uRNfze50p23GzpjR2JAQb3ks5raahggaJpZM4Lz6Cr
.

--
Austin Heiman
816.405.3917
<EMAIL_ADDRESS>

@atheiman What is stopping us if we use the rundeck 2.6.x in the default. The changes in the cookbook are passing those.

@atheiman Ok after some fight I could test the centos with vagrant locally. BUt it shows the same error -

         1) RundeckApiClient correctly reads the rundeck server api version
            Failure/Error: described_class.connect('http://localhost', 'admin', 'adminpassword')
            Net::HTTPFatalError:
              503 "Service Unavailable"

            # /tmp/kitchen/cookbooks/rundeck/libraries/rundeck_api_client.rb:54:in `send_req'
            # /tmp/kitchen/cookbooks/rundeck/libraries/rundeck_api_client.rb:20:in `authenticate'
            # /tmp/kitchen/cookbooks/rundeck/libraries/rundeck_api_client.rb:28:in `connect'
            # /tmp/verifier/suites/serverspec/localhost/rundeck_api_client_spec.rb:5:in `block (2 levels) in <top (required)>'
            # /tmp/verifier/suites/serverspec/localhost/rundeck_api_client_spec.rb:9:in `block (2 levels) in <top (required)>'

       Finished in 4.54 seconds (files took 1.77 seconds to load)
       87 examples, 1 failure

       Failed examples:

       rspec /tmp/verifier/suites/serverspec/localhost/rundeck_api_client_spec.rb:8 # RundeckApiClient correctly reads the rundeck server api version```

@atheiman I logged into the vagrant box and checked that rundeck service is running. Even I can access that from my browser( Had to do some port forwarding in the virtualbox network setting). To me it seems like a timing issue that is causing the test to fail. How do we set some sleep in the test?

Yea we're working to implement a fix for that "service started but not yet listening" issue now.

@atheiman kitchen tests are passing locally https://gist.github.com/nilroy/863371d19254d262159d15f676e91017
Not sure whats happeining in travis. Would you merge it?

@pcross616 Can we get this PR merged?

retest this please

Is there any update on this?

a lot of the changes im making in #133 need to be merged first to make the cookbook work with 2.7.x

@atheiman I shall wait until then.

@atheiman now as you got your PR merged can we merge this PR? Or is there any change I need to do?

@atheiman still the service check is failing in the travis build? How can I rerun the test without changing anything in the code?

@atheiman is there anything I can do to get this PR merged?

I m closing this PR as I have opened a new one https://github.com/Webtrends/rundeck/pull/153


[Scope Change] Page "Check Credit Limit" - "SalesLineShowWarning"

Hi,
I'd like to request the Scope "OnPrem" to be removed so that we can use this for cloud management.
I haven't seen an obsolete tag or anything else, so I'm not entirely sure if this is intended or not.
    [Scope('OnPrem')]
    procedure SalesLineShowWarning(SalesLine: Record "Sales Line") Result: Boolean
    var
        IsHandled: Boolean;
    begin
        // ...
    end

This is already removed


location-default-85

Trivial change to default location to '.' so that it behaves a bit more like
Dancer2 out of the box. The static/ directory should probably behave similarly, but I'll address that later.

Thank you! :-)


Noise Removal File not used in LAREX

LAREX is not using the Noise Removal File (.desp). It uses instead the File (.nrm). I assume the filename is not corretc

Thank you for the issue and excuse the late reply.
Does this happen in "standalone" LAREX or when using it in conjunction with OCR4all?

Hi, it happens when i use it with Ocr4all.
Kr
Dirk


(core): automatically update Homebrew core formula with every release

I've never used this Github Action so let's pay attention on the next release.

Added a COMMITTER_TOKEN secret with the right access rights.


Stuck on integration pane of Nuki

Hi,
To get the Web API, is it the red colored button that I need to use to generate the key ? Or the one of the top ? Everything I try, when entering in the Integration, after submitting, it freezes and the integration is not complete… Please advise

It’s stuck like more than 15 min

Kr,
Bart

Instructions to generate the web api token are in the repository, specifically here: https://github.com/kvj/hass_nuki_ng#web-api-token
The yellow button here is used to generate the token:

If you put the correct info in the config dialog box, it should take 2-3 secs, if it takes more than that, it means that the bridge is not reachable. So you need to check the IP, if it's correct, please check in the logs for errors and report back.


Upgrade addons

Note, I picked the kubernetes-dashboard release from today (it's 1.4.0-beta1).

ping @mikedanese


25830qv unit test phase 2

actually, the last review was pending so the news files and old files got added to this pull request.


Bug / Typo: 2nd deviceSizes should be imageSizes

Hey!
First off great work and thanks for sharing your solution!
I used your setup as a reference to get the same thing running via serverless framework.
Whilst testing I encountered the issue that I was not able to generate images smaller than w=640, it would say "w" parameter (width) of 256 is not allowed even though it is a valid default imageSize.
Looking at the code I found the cause. Here, when using the defaults in case of missing envs, the second default should be imageSizes and not deviceSizes again.
https://github.com/dealmore/terraform-aws-next-js-image-optimization/blob/4de6dfb3b241d3a6d99d603e8b047cddaf1937cf/lib/handler.ts#L61
Just wanted to let you know in case anyone is using this without explicit envs and encounters this as well.
Thanks and kind regards,
Daniel

Ah yes, good catch!
I will provide a fix for this in the next release (Since we follow the same versioning schema as Next.js we'll have to wait until a new version of Next.js is released).
Thanks for reporting!


fix(rpc): Match zcashd's getblockchaininfo capitalisation for NU5

Motivation
Our getblockchaininfo RPC is using a different capitalisation of NU5 to zcashd:
Querying zebrad v1.2.0+38.g7ca6974 main chain at height >=2214736...
Querying zcashd v5.6.1-gb7af08e90de main chain at height >=2214738...
...
Response diff between zebrad v1.2.0+38.g7ca6974 and zcashd v5.6.1-gb7af08e90de:
--- /run/user/1000/tmp.Xom0Z8v18C.rpc-diff/zebrad-main-2214736-getblockchaininfo.json   2023-09-04 11:52:17.632123229 +1000
+++ /run/user/1000/tmp.Xom0Z8v18C.rpc-diff/zcashd-main-2214738-getblockchaininfo.json   2023-09-04 11:52:17.655123021 +1000
     "c2d6d0b4": {
-      "name": "Nu5",
+      "name": "NU5",
       "activationheight": 1687104,
-      "status": "active"
+      "status": "active",
+      "info": "See https://z.cash/upgrade/nu5/ for details."
     }
   },
   "consensus": {

This is unlikely to matter because lightwalletd ignores this field, but it seemed easy to fix.
Specifications
This is unspecified behaviour:
https://zcash.github.io/rpc/getblockchaininfo.html
Solution
Tell Zebra to serialize and deserialize as "NU5" instead
Update snapshot tests
Review
This is an optional RPC fix.
Reviewer Checklist

[ ] Will the PR name make sense to users?

[ ] Does it need extra CHANGELOG info? (new features, breaking changes, large changes)

[ ] Are the PR labels correct?
[ ] Does the code do what the ticket and PR says?

[ ] Does it change concurrent code, unsafe code, or consensus rules?

[ ] How do you know it works? Does it have tests?

Unrelated failure in the RPC conflict test:

conflicted node2 was still running, but the test expected a panic

https://github.com/ZcashFoundation/zebra/actions/runs/6068422696/job/16462328531?pr=7454#step:4:7246

@mergifyio refresh

@mergifyio refresh

@mergifyio refresh


Should there be an ActivityEvent.ON_SAVE_INSTANCE_STATE?

We are running into an issue where there are some things that cannot occur after onSaveInstanceState is called (e.g. fragment transactions) and we want to make sure that the Rx chain is unsubscribed at that point, but the activity lifecycle methods do not cover that. Pause is close but there is no guaranteed order between onPause and onSaveInstanceState which still leaves a possibility of an error.

I'm working on another lib right now that should invalidate this need (if it works out). It has a lot more listeners in general which would let you hook into save instance state.
I'm going to wait on that until I see if the other lib is viable, especially since if it does work out this library would simply build on top of it (or maybe even be completely invalidated).

See: https://github.com/trello/navi/


Documentation addition

I'd like to add a small section to the installation instructions about troubleshooting 5xx errors. Thought it might save time for anyone installing it for the first time. What would be the best way to do this?

There are a lot of things that can go wrong with a NetBox installation, many of which don't pertain to NetBox itself. Rather than attempt to maintain a catalog of all of these, I'm happy to point people to the discussion list archives to search for known issues.

I can definitely appreciate wanting to limit scope creep.
Thanks for taking this project public; it's well on it's way to being an invaluable tool for the data center. Let me know if there's anything I can do to help.


Update ns-fltkernel-_flt_registration.md

As Christian Allred allow me to create this pull request, I add "This flag was introduced in Windows 8." to the Meaning of "FLTFL_REGISTRATION_SUPPORT_NPFS_MSFS".

Thanks @aktsuda, I'll add this change in. Should be able to see it soon!


Add support for partial payments in NP Atobarai

I want to merge this change because...
Add support for partial payments SALEOR-4522
Includes changes from PR #8501

Impact

[ ] New migrations
[ ] New/Updated API fields or mutations
[ ] Deprecated API fields or mutations
[ ] Removed API types, fields, or mutations
[ ] Documentation needs to be updated

Pull Request Checklist

[ ] Privileged queries and mutations are guarded by proper permission checks
[ ] Database queries are optimized and the number of queries is constant
[ ] Database migration files are up to date
[ ] The changes are tested
[ ] GraphQL schema and type definitions are up to date
[ ] Changes are mentioned in the changelog

Task linked: SALEOR-4522 Add support for partial payments

Closing as partial payments will be introduced in 3.1


null.querySelectorAll crash on the landing page

Hey guys, not sure if this is the best place to drop it, sorry if not.
I went to your website, started reading, then went to Get Started and for some reason went back and then 💥 it crashed with this ugly error:

It happens 100% of the time. To reproduce, just go to the landing page -> get started -> navigate back (browser's back, it doesn't happen if I click the logo on the top-left). If I refresh it renders the landing page normally.

Hey guys, not sure if this is the best place to drop it, sorry if not.
I went to your website, started reading, then went to Get Started and for some reason went back and then 💥 it crashed with this ugly error:

It happens 100% of the time. To reproduce, just go to the landing page -> get started -> navigate back (browser's back, it doesn't happen if I click the logo on the top-left). If I refresh it renders the landing page normally.

Sorry I didn't respond, so I was actually re-writing the entire thing. Now available at: https://remix-pwa.run
And Remix PWA v4 has been releasede btw

Thanks, mate! Looks much cleaner! I'm excited to give it a go. Cheers,


Error loading mods

log: https://crashy.net/I609pO4GWCcrL8L1ziqL

This is probably caused by another mod altering the waterlogging mechanism. However, the issue should be fixed in Version 0.4.0 because of major rewrites


[BUG] Invisible ships

Describe the bug
Some ships in game appear invisible, only their projectiles and engines showing up in the game arena.
To Reproduce
Steps to reproduce the behavior:

Start the game.
Select a ship, enter weapons, etc - just do all necessary actions to get into the game.

Repeat above step maybe 4-5 times, or until you encounter an enemy ship that is invisible.
locate this ship by following all projectiles to origin, until encountering an enemy ship that is invisible.

Expected behavior
All ships should be visible.
Only seems to happen with later enemies that are spawned, never the earlier ones.

Fixed in pull request #298


Pipette or 'hammer command' not working with trees if used together with BetterCreative

If I use the Pipette to pick up a tree or use the hammer command (e.g. ´hammer oak1´) it works fine.
If I also install BetterCreative [1][2], this does not work anymore. It still works on rocks, etc
I will also add a bug report for BetterCreative
[1] https://valheim.thunderstore.io/package/Heinermann/BetterCreative/
[2] https://github.com/heinermann/Valheim_mods/tree/main/BetterCreative

Unfortunately Better Creative does some stuff that makes things not working very well. It's not something I could fix easily.


⚠ Bofrost.fr has degraded performance

In cd6924d, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1276 ms

Resolved: Bofrost.fr performance has improved in 94abc0b after 6 minutes.


"How to update a visual" - Multi-version deployment not supported

The "How to update a visual" suggests that multiple versions of the same visual can be uploaded as long as they get a different title (but maintain the same Visual ID):
> However, if the new version of the visual might break any usage or data structure of the previous version of the visual, then do not replace the previous version. Instead, you should create a new listing for the new version of the visual. For example, add a new version number (version X.X) to the title of the new listed visual. This way it is clear that it is the same visual just with an updated version number, so existing reports do not break their functionality. Again, make sure the Visual ID remains unchanged.
That does not seem to be working as described. I have just tried exactly that, and an error message "There was a problem adding this custom visual. It already exists" is displayed. It appears that this would only be supported if a new Visual ID were assigned.

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

ID: 3eb228f2-d033-a44b-5570-da815663e246
Version Independent ID: 1260b40c-f6bf-3e23-b60f-6966753424b1
Content: Power BI admin portal - Power BI
Content Source: powerbi-docs/service-admin-portal.md
Service: powerbi
GitHub Login: @mgblythe
Microsoft Alias: mblythe

Thank you for your comment. This has been assigned to the document author.

Assigning to @markingmyname to address

@mthierba Thank you for your feedback! We will investigate and get back to you with the relevant info.

@uve, can you assist, please?

This is a question  about importing Custom Visuals into Organizational Store. Assigning task to @sranins  to provide an answer.

@sranins

Each Custom Visual should have only one unique GUID.
PowerBI always loads the latest version of the visuals from CDN based on this GUID number, even if you imported your own version of the visual with the same GUID to the report.
So administrators aren't able to import multiple versions of the visual with one GUID.

We will now proceed to close this thread. If there are further questions regarding this matter, please tag the assignee in your reply. We will gladly continue the discussion, and we will reopen the issue.


⚠ LPDM Handbook (hosted on Divio) has degraded performance

In b1a769a, LPDM Handbook (hosted on Divio) (https://repohubdev.tk) experienced degraded performance:

HTTP code: 500
Response time: 6359 ms

Resolved: LPDM Handbook (hosted on Divio) performance has improved in f6a273b.


Adding explanation for model and scenario name conventions

This PR adds subfolders for the model and scenario dimensions of the data format
closes #2

So clear to me

thank you both for reviewing!


[Feature request] Ability to differentiate where emails are stored

Idea
I am running this container on my unraid server, and by default the whole thing saves to a specific folder. I'd like the ability to run the application and logs in one folder, but save/ backup the emails to another folder. Is this something that's already possible?

Maybe you could try to add another (different) volume mapping for /config/profile ?

Just FYI, this does seem to work for me...

Works for me too.


Recommend unwrapping PNG from SVG, when the SVG file contains nothing more than a PNG

For this file https://raw.githubusercontent.com/AEGEE/oms-frontend/a9d93cdd8663efea8192a5ce4644703ce44ef369/src/images/AEGEE_logo.svg svgo shall
• recommend to unwrap the PNG and use it instead.  Or better,
• offer unwrapping, or
• do the unwrapping implicitly by creating a new file and emitting a message.
Otherwise it will stay unnoticed, that the SVG is bloated by doing unnecessary wrappings.

@dilyanpalauzov: This is also an architectural thing. IMHO ideally something like webpack should be used on the SVG and different loaders invoked for the image resources inside. These loaders can then either optimize or split them off the SVG.


Cannot launch a single test-case with pytest

Environment data

VS Code version: 1.40.1
Extension version: python2019.1.50794
OS and version: Linux, Debian
Python version: python-3.7 (OS-default)
Type of virtual environment used: venv. pytest is installed in the 'selected" venv.
Relevant/affected Python packages and their versions: XXX
Jedi

Expected behaviour
I'm expecting to launch a single test-case, either from the code-lens on the test-case function, or from the UI panel to the left with the green arrow.
Actual behaviour

All test-cases are launched with pytest, as seen in the output panel.
The same happens with debugger.

Questions
Very strange nobody have reported this as of today (#2241 is for unittest, not pytest),
so definitely i'm doing something wrong.
Please guide me how can i help you

My mistake, there is no bug!
U had specified in setup.cfg the adopts option with certain positional dirs in the cmd-line,
so VSCode couldn't deny them.


Fix bug with TrainingValidationSplit and custom index

Pull Request Description
Fixes #1126

After creating the pull request: in order to pass the release_notes_updated check you will need to update the "Future Release" section of docs/source/release_notes.rst to include this pull request by adding :pr:123.

@jeremyliweishih Not that I'm aware of! iloc works with pandas, numpy, lists and I believe the sklearn splitters return numpy arrays anyways.


bug: use s3 endpoint fail to read manifest list with s3a location

If an iceberg table is created and inserted with data by spark, we will get a manifest list with prefix s3a.
spark-sql --packages org.apache.iceberg:iceberg-spark-runtime-3.5_2.12:1.4.3,mysql:mysql-connector-java:8.0.33,org.apache.hadoop:hadoop-aws:3.3.2 \
    --conf spark.sql.extensions=org.apache.iceberg.spark.extensions.IcebergSparkSessionExtensions \
    --conf spark.sql.catalog.demo=org.apache.iceberg.spark.SparkCatalog \
    --conf spark.sql.catalog.demo.type=hadoop \
    --conf spark.sql.catalog.demo.warehouse=s3a://my-iceberg-bucket-5950/ \
    --conf spark.sql.catalog.demo.hadoop.fs.s3a.endpoint=https://s3.ap-southeast-2.amazonaws.com \
    --S --e "CREATE TABLE demo.demo_db.demo_table(seq_id bigint, user_id bigint, user_name string) TBLPROPERTIES ('format-version'='2');"

If I use iceberg-rust to read that table with a s3 instead of a s3a endpoint, it will end up with this error. Is it expected?
Invalid s3 url: s3a://icebergdata/demo/demo_db/demo_table/metadata/snap-2-1-497e82e7-f4a6-4513-8c76-6267c9497998.avro, should start with s3://icebergdata/

Thanks @chenzl25 for reporting this. I've checked the code and think this is expected. iceberg stores absolute path in manifests, and currently it's illegal to mix use s3 and s3a. I think it works if you just use s3a, right?

Yes, it works if I use s3a. If it is an expected behavior, we can close this issue. Thanks!


offsetWidth\offsetHeight vs boundingCleintRect.width and heigth

We are currently using these interchangeably within PlacementManager. Based on documentation online they are actually different - offsetWidth and offsetHeight are accounting padding and border width of the element.

We need to update PlacementManager to consistently use one or the other
Add unit tests for target elements with padding and border

Resolved  with #241


[Bug] | Login issues

Are you using the latest release?

[X] Yes, I am using the latest release.

Have you checked a bug report doesn't already exist?

[X] Yes, I have checked the existing bug reports.

What happened?
I have used Wizarr quite a bit so I have had my login info saved for it, since the latest update that login info no longer works. Jot sure what to do about it, any suggestions would be much appreciated. Thanks!
(unRAID docker)
Virtualisation
Docker
What operating system are you seeing the problem on?
Other
What browsers are you seeing the problem on?
Chrome
Project Location
Local
Bug Description
I have used Wizarr quite a bit so I have had my login info saved for it, since the latest update that login info no longer works. Jot sure what to do about it, any suggestions would be much appreciated. Thanks!
Steps to Reproduce
No response
Relevant Log Output
No response

Try opening your database.db file in a Sqlite Editor like this one https://sqliteonline.com

Try opening your database.db file in a Sqlite Editor like this one https://sqliteonline.com

I am having same issue when open the db it is blank it seems.


Wow, that's a lot of RSS feeds and podcasts!

Hey folks - we're getting a huge influx of RSS feeds and podcasts. Guess the beta version of Winds 2.0 is pretty popular! I'm sorry that it's taking us longer than we thought to process all these RSS feeds and podcasts - thanks for being patient with us. I'll close this issue out once we're all caught up.
Thanks!

Appears to be all caught up - closing.


[Minor] Updated reference to New-SelfSignedCertificate

portion of the URL is now "/pki/" instead of "/pkiclient/"

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


'react-scripts start' command fails with node 18.0.0 if "proxy" entry is present in package.json

Describe the bug
When I try to start the development server with "npm run start", the 'react-scripts start' command fails, throwing this exception:
`Invalid options object. Dev Server has been initialized using an options object that does not match the API schema.

options.allowedHosts[0] should be a non-empty string.
error Command failed with exit code 1.`

This happens only by switching to Node.js version 18.0.0, if in package.json the entry:
"proxy":"http://something" is present.
It doesn't show up if you delete that entry, and it doesn't show up with previous Node versions until 18.
Did you try recovering your dependencies?

$ yarn --version 1.22.18
Which terms did you search for in User Guide?

I searched for:
"react-script start fail" and for the error message:
"Invalid options object. Dev Server has been initialized using an options object that does not match the API schema."
Environment

Environment Info:

  current version of create-react-app: 5.0.1
  running from C:\Users\roralligator\AppData\Local\npm-cache\_npx\c67e74de0542c87c\node_modules\create-react-app

  System:
    OS: Windows 10 10.0.22000
    CPU: (8) x64 Intel(R) Core(TM) i7-8550U CPU @ 1.80GHz
  Binaries:
    Node: 18.0.0 - C:\Program Files\nodejs\node.EXE
    Yarn: 1.22.18 - C:\Program Files\nodejs\yarn.CMD
    npm: 8.6.0 - C:\Program Files\nodejs\npm.CMD
  Browsers:
    Chrome: Not Found
    Edge: Spartan (44.22000.120.0), Chromium (100.0.1185.44)
    Internet Explorer: 11.0.22000.120
  npmPackages:
    react: ^18.0.0 => 18.0.0
    react-dom: ^18.0.0 => 18.0.0
    react-scripts: ^5.0.0 => 5.0.0
  npmGlobalPackages:
    create-react-app: Not Found

( It says Windows 10, but it's actually under Window 11 )
Steps to reproduce

Install Node version 18.0.0
Initialize a cra app with yarn create react-app or npx create-react-app
Edit package.json adding : `"proxy":"http://localhost:5000"
Start the app with yarn start or npm run start

Expected behavior

The webpack development server should start within a few seconds.
Actual behavior

It throws an exception and exits:
$ react-scripts start
Invalid options object. Dev Server has been initialized using an options object that does not match the API schema.
 - options.allowedHosts[0] should be a non-empty string.
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

Reproducible demo

(Paste the link to an example project and exact instructions to reproduce the issue.)

It is just like everyone has this issue. It is caused by webpack is updated to 6, and it has some new feature.
Workaround: set env DANGEROUSLY_DISABLE_HOST_CHECK=true

@Kais3rP
Just wanted to point out, i had this problem with node 16.
Thanks for the question, your question included a workaround for me. :)
For now i've gotten rid of the proxy and just hardcoded the backend's url for dev reasons and keep it in a separate changelist. This allows me to keep moving, and i appreciate you a ton for this question, again.


Honor ignoreUnknownArgs for short options

Make the partser honor the ignoreUnknownArgs flag for short options, too
While ignoreUnknownArgs used to work for ignoring unknown long options, unknown short options would make the parser complain even when ignoreUnknownArgs was set. This breaks for situations where I would first separate any options from positional arguments (and maybe process global options like --help, --verbose, -h, -v) and later parse again after determining which module(s) should provide further options.

Thanks!


nixos/ups: add wall to upsmon path

Description of changes
upsmon has the ability to send messages to wall which will result in a slightly hidden error as it is not in the service's PATH.
Things done
wall exists in the unixtools.wall package, thus it was added to the upsmon service's PATH.

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

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

Can you add the motivation for this change in the git commit message please? (Ref. https://github.com/NixOS/nixpkgs/blob/master/CONTRIBUTING.md#writing-good-commit-messages)


React Native config fails: TypeError: identifier.indexOf is not a function

Bug Report
Describe the bug
During IN_PROGRESS stage this error presents every time npm run docz:dev is run.
To Reproduce
In an existing React Native (non-Expo) project:
I deleted the docz folder and re-ran npm run docz:dev many times throughout this process:

Install docz (no caret: 2.3.1)
Ensure the following are installed and are at same version (^16.8.6 in my case)

react
react-art
react-dom

Install react-native-web (^0.11.7)
Add scripts to package.json (all 3 from docz.site)
Install Gatsby globally
Install docz globally (so scripts run)
Install gatsby-plugin-styled-components
Tried with and without Create first mdx file: src/foo-bar/index.mdx and copy/paste index.mdx from docz React Native example into this folder
Tried with and without Create new folder: src/foo-bar/Alert/ and copy/paste Alert.jsx and Alert.mdx from docz React Native example into this folder
Run npm run docz:dev
Run npm i -S variant per error instructions (even though it's already installed)

Add these files to root of project:

// doczrc.js at root of RN project
export default {
  title: 'IAMO UI Kit',
  files: ['**/*.{mdx}'],
  src: './src/foo-bar/Alert',
};

Also tried alt version: with   files: '**/*.mdx',

// gatsby-node.js
exports.onCreateWebpackConfig = args => {
  args.actions.setWebpackConfig({
    resolve: {
      alias: {
        'react-native': 'react-native-web',
      },
    },
  });
};

// gatsby-config.js
module.exports = {
  plugins: [
    {
      resolve: `gatsby-plugin-styled-components`,
      options: {
        // Add any options here
      },
    },
  ],
};

Rinse & Repeat Process each time:

Delete docz/ auto-generated folder at root of app
Run react-native start --reset-cache each time new dependencies (or version) were installed
Re-run npm run docz:dev

Expected behavior
🚀🔥✊🎉 and happy devs. Oh, and a similar build to the React Native example.
Environment

docz version: 2.3.1 (no caret)
OS: OSX 10.15.4
Node/npm version: Node 10.15.2/NPM 6.4.1 provided by NVM

Additional context/Screenshots
➜  react-native-app git:([REDACTED]-set-up-documentation-tooling) ✗ npm run docz:dev

> [REDACTED]/react-native-app
> docz dev

Building app
warn Configuring yargs through package.json is deprecated and will be removed in the next major release, please use the JS API instead.
success open and validate gatsby-configs - 0.570s
success load plugins - 0.247s
success onPreInit - 0.007s
success initialize cache - 0.006s
success copy gatsby files - 0.044s
success onPreBootstrap - 0.128s
success createSchemaCustomization - 0.003s
warn Configuring yargs through package.json is deprecated and will be removed in the next major release, please use the JS API instead.
success source and transform nodes - 0.185s
success building schema - 0.312s
success createPages - 0.003s

 ERROR #85923  GRAPHQL

There was an error in your GraphQL query:

Cannot query field "allDoczEntries" on type "Query".

If you don't expect "allDoczEntries" to exist on the type "Query" it is most likely a typo.
However, if you expect "allDoczEntries" to exist there are a couple of solutions to common problems:

- If you added a new data source and/or changed something inside gatsby-node.js/gatsby-config.js, please try a restart of your development server
- The field might be accessible in another subfield, please try your query in GraphiQL and use the GraphiQL explorer to see which fields you can query and what shape they have
- You want to optionally use your field "allDoczEntries" and right now it is not used anywhere. Therefore Gatsby can't infer the type and add it to the GraphQL schema. A quick fix is to add a least one entry with that field ("dummy content")

It is recommended to explicitly type your GraphQL schema if you want to use optional fields. This way you don't have to add the mentioned "dummy content". Visit our docs to learn how you can define the schema for "Query":
https://www.gatsbyjs.org/docs/schema-customization/#creating-type-definitions

File: ../node_modules/gatsby-theme-docz/lib/createPagesStatefully.js:28:10

success createPagesStatefully - 0.095s
success onPreExtractQueries - 0.001s
success update schema - 0.025s
success extract queries from components - 0.067s
success write out requires - 0.008s
success write out redirect data - 0.001s
success onPostBootstrap - 0.002s

info bootstrap finished - 3.822 s

success run queries - 0.033s - 3/3 91.37/s
⠏ Building development bundle

————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————
3 pages                                                         IN_PROGRESS                                                         docz-app
[REDACTED]/react-native-app/node_modules/yoga-layout-prebuilt/yoga-layout/build/Release/nbind.js:53
        throw ex;
        ^

TypeError: identifier.indexOf is not a function
    at Resolver.parse (/Users/[REDACTED]/react-native-app/node_modules/gatsby/node_modules/enhanced-resolve/lib/Resolver.js:301:31)
    at resolver.getHook.tapAsync (/Users/[REDACTED]/react-native-app/node_modules/gatsby/node_modules/enhanced-resolve/lib/ParsePlugin.js:18:29)
    at AsyncSeriesBailHook.eval [as callAsync] (eval at create (/Users/[REDACTED]/react-native-app/node_modules/gatsby/node_modules/tapable/lib/HookCodeFactory.js:33:10), <anonymous>:7:1)
    at Resolver.doResolve (/Users/[REDACTED]/react-native-app/node_modules/gatsby/node_modules/enhanced-resolve/lib/Resolver.js:282:16)
    at resolver.getHook.tapAsync (/Users/[REDACTED]/react-native-app/node_modules/gatsby/node_modules/enhanced-resolve/lib/UnsafeCachePlugin.js:36:14)
    at AsyncSeriesBailHook.eval [as callAsync] (eval at create (/Users/[REDACTED]/react-native-app/node_modules/gatsby/node_modules/tapable/lib/HookCodeFactory.js:33:10), <anonymous>:7:1)
    at Resolver.doResolve (/Users/[REDACTED]/react-native-app/node_modules/gatsby/node_modules/enhanced-resolve/lib/Resolver.js:282:16)
    at resolver.getHook.tapAsync (/Users/[REDACTED]/react-native-app/node_modules/gatsby/node_modules/enhanced-resolve/lib/AliasFieldPlugin.js:53:14)
    at _next0 (eval at create (/Users/[REDACTED]/react-native-app/node_modules/gatsby/node_modules/tapable/lib/HookCodeFactory.js:33:10), <anonymous>:638:1)
    at _fn0 (eval at create (/Users/[REDACTED]/react-native-app/node_modules/gatsby/node_modules/tapable/lib/HookCodeFactory.js:33:10), <anonymous>:660:1)
    at resolver.getHook.tapAsync (/Users/[REDACTED]/react-native-app/node_modules/gatsby/node_modules/enhanced-resolve/lib/AliasPlugin.js:74:12)
    at AsyncSeriesBailHook.eval [as callAsync] (eval at create (/Users/[REDACTED]/react-native-app/node_modules/gatsby/node_modules/tapable/lib/HookCodeFactory.js:33:10), <anonymous>:652:1)
    at Resolver.doResolve (/Users/[REDACTED]/react-native-app/node_modules/gatsby/node_modules/enhanced-resolve/lib/Resolver.js:282:16)
    at DescriptionFileUtils.loadDescriptionFile (/Users/[REDACTED]/react-native-app/node_modules/gatsby/node_modules/enhanced-resolve/lib/DescriptionFilePlugin.js:54:17)
    at forEachBail (/Users/[REDACTED]/react-native-app/node_modules/gatsby/node_modules/enhanced-resolve/lib/DescriptionFileUtils.js:63:13)
    at args (/Users/[REDACTED]/react-native-app/node_modules/gatsby/node_modules/enhanced-resolve/lib/forEachBail.js:30:14)

Feel free to de-prioritize this issue or close it. I've started fresh on a clean NPM package and am having different issuew with styled-components (using css={css...} syntax) not updating based on context in the Playground. I can't share too much more for work reasons, so I understand if you're not able to help.


Organisation survey: ability to "import" students from courserealisations

It would be handy to import students from courserealisations to the organisation survey.
How would it work
There should be some kind of input where we could input the course code or whatever and then select courses that we would want to import the students from

Make organisation survey editor submit to refetch the student count based on the student count of the feedbacktarget which is updated
add locales
make tests
make the ui look pretty

Section 1. is blocked


AttributeError

I've tried the following, both with my native interpreter installation and a fresh VirtualEnv. I'm getting the following with the example code from the README
$ python3 sparkySecure.py
Traceback (most recent call last):
  File "sparkySecure.py", line 12, in <module>
    print(smime.encrypt('\n'.join(message), pem.read()))
AttributeError: module 'smime' has no attribute 'encrypt'

Checking the package:
$ pip3 show smime
Name: smime
Version: 0.0.3
Summary: Python S/MIME Toolkit
Home-page: https://github.com/balena/python-smime
Author: G. B. Versiani
Author-email<EMAIL_ADDRESS>License: Apache License (2.0)
Location: /usr/local/lib/python3.7/site-packages
Requires: asn1crypto, pycrypto
Required-by:

Suspecting this might have been a recent Python 3.x issue, I tried a Linux machine with Python 3.5. pip install smime gives the following errors
  building 'Crypto.Hash._MD2' extension
  creating build/temp.linux-x86_64-3.4
  creating build/temp.linux-x86_64-3.4/src
  gcc -pthread -DDYNAMIC_ANNOTATIONS_ENABLED=1 -pipe -Wall -Wp,-D_FORTIFY_SOURCE=2 -fexceptions -fstack-protector --param=ssp-buffer-size=4 -m64 -mtune=generic -D_GNU_SOURCE -fPIC -fwrapv -fPIC -std=c99 -O3 -fomit-frame-pointer -Isrc/ -I/usr/include/python3.4m -c src/MD2.c -o build/temp.linux-x86_64-3.4/src/MD2.o
  src/MD2.c:31:20: error: Python.h: No such file or directory
  src/MD2.c:131: error: expected ‘=’, ‘,’, ‘;’, ‘asm’ or ‘__attribute__’ before ‘*’ token
  In file included from src/MD2.c:147:
  src/hash_template.c:48: error: expected specifier-qualifier-list before ‘PyObject_HEAD’
  src/hash_template.c:59: error: expected ‘=’, ‘,’, ‘;’, ‘asm’ or ‘__attribute__’ before ‘PyTypeObject’
  src/hash_template.c: In function ‘newALGobject’:
  src/hash_template.c:69: warning: implicit declaration of function ‘PyObject_New’
  src/hash_template.c:69: error: expected expression before ‘ALGobject’
  src/hash_template.c: At top level:
  src/hash_template.c:76: error: expected ‘)’ before ‘*’ token
  src/hash_template.c:91: error: expected ‘=’, ‘,’, ‘;’, ‘asm’ or ‘__attribute__’ before ‘*’ token
  src/hash_template.c:110: error: expected ‘=’, ‘,’, ‘;’, ‘asm’ or ‘__attribute__’ before ‘*’ token
  src/hash_template.c:122: error: expected ‘=’, ‘,’, ‘;’, ‘asm’ or ‘__attribute__’ before ‘*’ token
  src/hash_template.c:162: error: expected ‘=’, ‘,’, ‘;’, ‘asm’ or ‘__attribute__’ before ‘*’ token
  src/hash_template.c:188: error: expected ‘=’, ‘,’, ‘;’, ‘asm’ or ‘__attribute__’ before ‘*’ token
  src/hash_template.c:190: error: expected ‘=’, ‘,’, ‘;’, ‘asm’ or ‘__attribute__’ before ‘ALG_methods’
  src/hash_template.c:199: error: expected ‘=’, ‘,’, ‘;’, ‘asm’ or ‘__attribute__’ before ‘*’ token
  src/hash_template.c:225: error: expected ‘=’, ‘,’, ‘;’, ‘asm’ or ‘__attribute__’ before ‘ALGtype’
  src/hash_template.c:271: error: expected ‘=’, ‘,’, ‘;’, ‘asm’ or ‘__attribute__’ before ‘*’ token
  src/hash_template.c:304: error: array type has incomplete element type
  src/hash_template.c:305: error: ‘PyCFunction’ undeclared here (not in a function)
  src/hash_template.c:305: error: expected ‘}’ before ‘ALG_new’
  src/hash_template.c: In function ‘init_MD2’:
  src/hash_template.c:339: error: ‘PyObject’ undeclared (first use in this function)
  src/hash_template.c:339: error: (Each undeclared identifier is reported only once
  src/hash_template.c:339: error: for each function it appears in.)
  src/hash_template.c:339: error: ‘m’ undeclared (first use in this function)
  src/hash_template.c:351: error: ‘ALGtype’ undeclared (first use in this function)
  src/hash_template.c:351: error: ‘PyType_Type’ undeclared (first use in this function)
  src/hash_template.c:352: warning: implicit declaration of function ‘Py_InitModule’
  src/hash_template.c:356: error: ‘o’ undeclared (first use in this function)
  src/hash_template.c:356: warning: implicit declaration of function ‘PyInt_FromLong’
  src/hash_template.c:356: warning: implicit declaration of function ‘PyDict_SetItemString’
  src/hash_template.c:356: warning: implicit declaration of function ‘PyModule_GetDict’
  src/hash_template.c:356: warning: implicit declaration of function ‘Py_DECREF’
  src/hash_template.c:360: warning: implicit declaration of function ‘PyErr_Occurred’
  src/hash_template.c:361: warning: implicit declaration of function ‘Py_FatalError’
  error: command 'gcc' failed with exit status 1

I wonder if it's related to: https://stackoverflow.com/questions/50080459/failed-installing-pycrypto/50099842

Hello @tuck1s, sorry for the delay to answer.
Definitely the library will have trouble with Python 3.3 upwards. As indicated in the README.md file, it supports Python 2.7 and 3.3.
In order to support newer Python versions, we have to get rid of these old dependencies. I didn't find a good substitute yet, if you can indicate other options to perform the same PyCrypto operations, let me know.
Regards,
Balena.

The Stack Overflow article mentions pycryptodome. There is also https://github.com/pyca/cryptography. Not sure if those help. I'm going to explore the second one today and see if I can make it work with the MIME handling in https://docs.python.org/3.7/library/email.parser.html?highlight=email message_from_string#module-email.parser

Dear @tuck1s, I just released a version 0.0.4 using https://github.com/pyca/cryptography instead of PyCrypto. It is not in PyPI yet, please let me know if it works for you, so I'm going to release it in PyPI later on.

Thanks @balena - I'm not sure of the best way to install straight from Github, but I cloned the project then moved the smime dir under my test project, that seems to run. I'll build some test messages and try them towards Thunderbird.

I built the attached file, using a .crt file as pem input:
To: "Bob"<EMAIL_ADDRESS>From: "Steve"<EMAIL_ADDRESS>Subject: A message from python
MIME-Version: 1.0
Content-Type: application/pkcs7-mime; smime-type=enveloped-data; name=smime.p7m
Content-Transfer-Encoding: base64
Content-Disposition: attachment; filename=smime.p7m

MIICgAYJKoZIhvcNAQcDoIICcTCCAm0CAQAxggHHMIIBwwIBADCBrDCBlzELMAkG
A1UEBhMCR0IxGzAZBgNVBAgTEkdyZWF0ZXIgTWFuY2hlc3RlcjEQMA4GA1UEBxMH
U2FsZm9yZDEaMBgGA1UEChMRQ09NT0RPIENBIExpbWl0ZWQxPTA7BgNVBAMTNENP
TU9ETyBSU0EgQ2xpZW50IEF1dGhlbnRpY2F0aW9uIGFuZCBTZWN1cmUgRW1haWwg
Q0ECEEj/a4zQ5fW0CuU/h4irbEQwCwYJKoZIhvcNAQEBBIIBAGHxz09vLahTOdKt
0CgNBIQpMNPmijWoQG3acJgdbLP8mKdCgTHMSXD/htfn9gl9pOr66b1YCHNao507
fLeKaNaY0sGBZIO5vlav83A58W5ZHPPwtaiSftQOcolkmjiEaRymZMF5CJODryPn
HmF3HrLbm+idKYav39b8rgiz8Kwyj29Cj61tj/UQ7NbhoQvNJT/v/6yW54bZCcwj
ZduY//AakrUZ235CaBSTg+yzRwOrw15qtqA/SHdS/37O4etqI5Ou5gZG8Uexb6tP
tFOXrDEcBsMWUzO4w4zBcS8Thq40tv1T605f3StFSthEq9QiD2+3Q7YGgZCmGc2S
X/Ow9KgwgZwGCSqGSIb3DQEHATAdBglghkgBZQMEASoEECMMwx/5QQuWfKOD9Kgs
uguAcA10sIeQ2A1qrWzOCda7VQHVKawZ3KjQxxtp/A5HMM6TZn4Ku73p5yGxhZC7
3E4qIUnvsLsEAO+qcd1PdeMlD7xHa4E4fVxppc9RW34eUZqe8lNCNHsCeBU+Chca
ISuplp4UjcVot1D6DuDWAtY17KM=

Thunderbird complains about the file (just dragging the .eml output directly in, rather than delivering it). openssl can read the ASN1 structure:
tuckbook$ openssl smime -pk7out -in ex2.eml | openssl asn1parse
    0:d=0  hl=4 l= 640 cons: SEQUENCE
    4:d=1  hl=2 l=   9 prim: OBJECT            :pkcs7-envelopedData
   15:d=1  hl=4 l= 625 cons: cont [ 0 ]
   19:d=2  hl=4 l= 621 cons: SEQUENCE
   23:d=3  hl=2 l=   1 prim: INTEGER           :00
   26:d=3  hl=4 l= 455 cons: SET
   30:d=4  hl=4 l= 451 cons: SEQUENCE
   34:d=5  hl=2 l=   1 prim: INTEGER           :00
   37:d=5  hl=3 l= 172 cons: SEQUENCE
   40:d=6  hl=3 l= 151 cons: SEQUENCE
   43:d=7  hl=2 l=  11 cons: SET
   45:d=8  hl=2 l=   9 cons: SEQUENCE
   47:d=9  hl=2 l=   3 prim: OBJECT            :countryName
   52:d=9  hl=2 l=   2 prim: PRINTABLESTRING   :GB
   56:d=7  hl=2 l=  27 cons: SET
   58:d=8  hl=2 l=  25 cons: SEQUENCE
   60:d=9  hl=2 l=   3 prim: OBJECT            :stateOrProvinceName
   65:d=9  hl=2 l=  18 prim: PRINTABLESTRING   :Greater Manchester
   85:d=7  hl=2 l=  16 cons: SET
   87:d=8  hl=2 l=  14 cons: SEQUENCE
   89:d=9  hl=2 l=   3 prim: OBJECT            :localityName
   94:d=9  hl=2 l=   7 prim: PRINTABLESTRING   :Salford
  103:d=7  hl=2 l=  26 cons: SET
  105:d=8  hl=2 l=  24 cons: SEQUENCE
  107:d=9  hl=2 l=   3 prim: OBJECT            :organizationName
  112:d=9  hl=2 l=  17 prim: PRINTABLESTRING   :COMODO CA Limited
  131:d=7  hl=2 l=  61 cons: SET
  133:d=8  hl=2 l=  59 cons: SEQUENCE
  135:d=9  hl=2 l=   3 prim: OBJECT            :commonName
  140:d=9  hl=2 l=  52 prim: PRINTABLESTRING   :COMODO RSA Client Authentication and Secure Email CA
  194:d=6  hl=2 l=  16 prim: INTEGER           :48FF6B8CD0E5F5B40AE53F8788AB6C44
  212:d=5  hl=2 l=  11 cons: SEQUENCE
  214:d=6  hl=2 l=   9 prim: OBJECT            :rsaEncryption
  225:d=5  hl=4 l= 256 prim: OCTET STRING      [HEX DUMP]:61F1CF4F6F2DA85339D2ADD0280D04842930D3E68A35A8406DDA70981D6CB3FC98A7428131CC4970FF86D7E7F6097DA4EAFAE9BD5808735AA39D3B7CB78A68D698D2C1816483B9BE56AFF37039F16E591CF3F0B5A8927ED40E7289649A3884691CA664C179089383AF23E71E61771EB2DB9BE89D2986AFDFD6FCAE08B3F0AC328F6F428FAD6D8FF510ECD6E1A10BCD253FEFFFAC96E786D909CC2365DB98FFF01A92B519DB7E4268149383ECB34703ABC35E6AB6A03F487752FF7ECEE1EB6A2393AEE60646F147B16FAB4FB45397AC311C06C3165333B8C38CC1712F1386AE34B6FD53EB4E5FDD2B454AD844ABD4220F6FB743B6068190A619CD925FF3B0F4A8
  485:d=3  hl=3 l= 156 cons: SEQUENCE
  488:d=4  hl=2 l=   9 prim: OBJECT            :pkcs7-data
  499:d=4  hl=2 l=  29 cons: SEQUENCE
  501:d=5  hl=2 l=   9 prim: OBJECT            :aes-256-cbc
  512:d=5  hl=2 l=  16 prim: OCTET STRING      [HEX DUMP]:230CC31FF9410B967CA383F4A82CBA0B
  530:d=4  hl=2 l= 112 prim: cont [ 0 ]

It doesn't seem able to decrypt fully:
book$ openssl smime -pk7out -in thunderbird_encrypted.eml | openssl asn1parse
(reverse-i-search)`ex2': openssl smime -pk7out -in ex2.eml | openssl asn1parse
tuckbook$ openssl smime -decrypt -in ex2.eml -inkey<EMAIL_ADDRESS>Error decrypting PKCS#7 structure
140735868249032:error:06065064:digital envelope routines:EVP_DecryptFinal_ex:bad decrypt:/BuildRoot/Library/Caches/com.apple.xbs/Sources/libressl/libressl-22.50.2/libressl/crypto/evp/evp_enc.c:529:
.?7n???q??)???o4m?hh??????:?&?D?y?82^?a5}
                                         x??:??iˀ?
                                                  ?(?.??y???y?x~Z?)
                                                                   d?I?虾jԚ;t?

Same result on:
Error decrypting PKCS#7 structure
140020980914000:error:06065064:digital envelope routines:EVP_DecryptFinal_ex:bad decrypt:evp_enc.c:592:
?2:?)???i?o]??km}??`?????,7GV??e^?Z?c?ЄQ??Dr?p(?ю?;jt???8???ZrH(?k1?-0,|?!啱}?
                                                                              C?[ec2-user@ip-172-31-4-82 SMIME]$
[ec2-user@ip-172-31-4-82 SMIME]$ openssl version
OpenSSL 1.0.2k-fips  26 Jan 2017```

hmm looks like my key files might have been corrupt. Never mind - it now works!
$ openssl smime -decrypt -in ex3.eml -inkey<EMAIL_ADDRESS>Content-Type: text/plain; charset="us-ascii"
MIME-Version: 1.0
Content-Transfer-Encoding: 7bit

Now you see me.

Is there a way to sign as well as encrypt a message?

@tuck1s, my test was the following:
I executed the same code it is in the README.md using one of the RSA example keys available in smime/testdata. It gave me the following output:
To: "Alice"<EMAIL_ADDRESS>From: "Bob"<EMAIL_ADDRESS>Subject: A message from python
MIME-Version: 1.0
Content-Type: application/pkcs7-mime; smime-type=enveloped-data; name=smime.p7m
Content-Transfer-Encoding: base64
Content-Disposition: attachment; filename=smime.p7m

MIIBdgYJKoZIhvcNAQcDoIIBZzCCAWMCAQAxgb4wgbsCAQAwJjASMRAwDgYDVQQD
EwdDYXJsUlNBAhBGNGvHgABWvBHTbi7EELOwMAsGCSqGSIb3DQEBAQSBgGfgP5LI
41a8kb6/wY35juXXW0BaF+FBIEn1xpXosM3BHq4VcD77Epb+CnfxZLy4lpf/we6H
vyjm1cietfvZLGQxfCS/itJu/FulBFqP0Zq2pEcaEZUXPky7Vb70B9WfYMAZVlEc
QEzSz/KBO3txP/CEbh3fyJc73AkzSzIBuiLKMIGcBgkqhkiG9w0BBwEwHQYJYIZI
AWUDBAEqBBA+iq7Y2bJTpx7SJ7ONpH8CgHDZ1EmLONl7XaJCj06MJryI+UaN+ek+
E7asegHMgHK/y3Fx+rwOVIfrwaATkKTzruItNrTtJ3aci4mrq3IjG6wu/NTfeqao
wP1Q1xURhHTiehOiCRor+ezTqllHaEUWDHLISy0cL4kHr2AUi7E4kEPz

Then I created a smime.p7m.base64 as follows:
MIIBdgYJKoZIhvcNAQcDoIIBZzCCAWMCAQAxgb4wgbsCAQAwJjASMRAwDgYDVQQD
EwdDYXJsUlNBAhBGNGvHgABWvBHTbi7EELOwMAsGCSqGSIb3DQEBAQSBgGfgP5LI
41a8kb6/wY35juXXW0BaF+FBIEn1xpXosM3BHq4VcD77Epb+CnfxZLy4lpf/we6H
vyjm1cietfvZLGQxfCS/itJu/FulBFqP0Zq2pEcaEZUXPky7Vb70B9WfYMAZVlEc
QEzSz/KBO3txP/CEbh3fyJc73AkzSzIBuiLKMIGcBgkqhkiG9w0BBwEwHQYJYIZI
AWUDBAEqBBA+iq7Y2bJTpx7SJ7ONpH8CgHDZ1EmLONl7XaJCj06MJryI+UaN+ek+
E7asegHMgHK/y3Fx+rwOVIfrwaATkKTzruItNrTtJ3aci4mrq3IjG6wu/NTfeqao
wP1Q1xURhHTiehOiCRor+ezTqllHaEUWDHLISy0cL4kHr2AUi7E4kEPz

Then I used openssl to transform the BASE64 output to binary, or in other terms, get the DER format out of the BASE64 encoded body:
openssl base64 -d -in smime.p7m.base64 -out smime.p7m

And then I made the following:
openssl smime -decrypt -in smime.p7m -inform der -inkey alice-private-key.pem

And took the following output:
Content-Type: text/plain; charset="us-ascii"
MIME-Version: 1.0
Content-Transfer-Encoding: 7bit

Now you see me.

I wrote in the README.md that OpenSSL expects that the BASE64 content be enclosed in -----BEGIN PKCS7----- and -----END PKCS7-----. So if you want to use the input as PEM (PKCS#7), you need to create a smime.p7m file in the following way:
-----BEGIN PKCS7-----
MIIBdgYJKoZIhvcNAQcDoIIBZzCCAWMCAQAxgb4wgbsCAQAwJjASMRAwDgYDVQQD
EwdDYXJsUlNBAhBGNGvHgABWvBHTbi7EELOwMAsGCSqGSIb3DQEBAQSBgGfgP5LI
41a8kb6/wY35juXXW0BaF+FBIEn1xpXosM3BHq4VcD77Epb+CnfxZLy4lpf/we6H
vyjm1cietfvZLGQxfCS/itJu/FulBFqP0Zq2pEcaEZUXPky7Vb70B9WfYMAZVlEc
QEzSz/KBO3txP/CEbh3fyJc73AkzSzIBuiLKMIGcBgkqhkiG9w0BBwEwHQYJYIZI
AWUDBAEqBBA+iq7Y2bJTpx7SJ7ONpH8CgHDZ1EmLONl7XaJCj06MJryI+UaN+ek+
E7asegHMgHK/y3Fx+rwOVIfrwaATkKTzruItNrTtJ3aci4mrq3IjG6wu/NTfeqao
wP1Q1xURhHTiehOiCRor+ezTqllHaEUWDHLISy0cL4kHr2AUi7E4kEPz
-----END PKCS7-----

Then again the OpenSSL smime command:
openssl smime -decrypt -in smime.p7m -inform pem -inkey alice-private-key.pem

Gives the same result:
Content-Type: text/plain; charset="us-ascii"
MIME-Version: 1.0
Content-Transfer-Encoding: 7bit

Now you see me.


Bug: Alchemy Transmutation node gives wrong values (partially due to tooltip error on blizzards end)

The Transmutation node in the Alchemy spec tree has a few issues:

Despite Blizzards tooltip stating it gives +1% resourcefulness it actually gives +1 resourcefulness, easily fixed this locally in spec data
If you already have points in this node simulation mode will not include percentage bonuses you already have unlocked, but if you remove points from the node in simulation mode it will subtract them, possibly leading to negative values, not quite sure how to fix this

Upload images to github fails for some reason so I made an imgur album with some screenshots, note the descriptions below the images https://imgur.com/a/bdeccQ4

interesting... always thought its wierd that there is exactly one node that gives % ^^ will fix in next release!

Yeah, I was a bit confused at first too, but the other bug is probably the bigger issue because I think that it also affects the average profit calculation since entering simulation mode where it shows the wrong resourcefulness value does not change my average profit, correct me if I'm wrong

Fixed with 8.4.1

The % Resourcefulness Nodes still aren't added correctly in simulation mode.
This is just opening simulation mode + the knowledge simulation with no manual changes to knowledge, the actual value is 45.6% but simulation mode shows only 20.5% - basically missing the 25% from the three percentage bonuses in Transmutation:

If I now remove 10 points from the transmutation node I lose 10 resourcefulness from the points themselves and then another 135 from the 15% node at 15/20 points which is correct, but these 135 points weren't added in the first place so the resourcefulness value goes way down. Same for the other two nodes that grant +x% resourcefulness.

Removing all points from the Transmutation node and unlearning it ends up displaying negative resourcefulness in simulation mode but it does not remove the 5%/45 resourcefulness from 0/20 Transmutation, maybe the last part is somehow related to #99


[DNM?] Update to webpack 5

Update webpack.config.js to work with v5.
Update loaders and plugins.
Use assets for font packaging to fix parsing error
Clean the output folder before building.
Add --progress profile to the build:analyze script.
Remove file-loader and url-loader, since they're no longer needed.
Comment out the MiniCssExtractPlugin instance for .scss files, since it doesn't seem to work and breaks SpeedMeasurePlugin in webpack 5.

Updating to webpack 5 eliminated the high priority vulnerabilities, leaving 8 moderate ones.  But the initial dev build when running start:dev is about 6X slower than webpack 4 (~65 seconds).  The production build seems faster, but I'm not sure why the startup for watching file changes is so much slower.  Maybe it's Windows only?

Not sure if the readme is the right place for these transient instructions, since anyone new will get the latest packages. Maybe just a slack announcement? Could also add a check to webpack.config for awhile.


Enterprise DAO - project logo image broken

https://images2.imgbox.com/c9/73/wADKtP1e_o.png
Can you update this logo please?

The image is working again. I assume that was some downtime from the host.
I'll close this ticket.
cc\ @krys-s21
Here's the image working:


SystemLayer::PostEvent add API to post a lambda event

Problem
Provides a most flexible API to run a deferred function.
This will decouple system object from network endpoints (TCPEndpoint, UDPEndpoint). It helps implementing the system object pool in PR #9590
Change overview
Add a PostLambda to post a generic lambda into system event queue.
There is several limitation of the lambda

The lambda must be trivially copyable. Because the lambda is memcpy into the queue. It means that the lambda can only capture pod data (Including pointers).
The size of the lambda must less than the queue element size. The size of the lambda is the sum of all captured fields. The body of the lambda is encoded in the type info of the lambda, which is captured by a static function LambdaProxy<Lambda>::Call, and resolved into a function pointer at compile time, assigned to LambdaBridge::LambdaProxy into the queue element.

Testing
Verified by unit tests.

Here's my belated feedback

Is it correct? It seems like it may not be [1]
Do we really need it? (would a single function ptr and user_data do?)
Do people understand the limitations?
Why mix the call-once nature of lambdas with the broadcast nature of other 'events' into a single API ?
If we have lambdas which is undoubtedly is a more flexible API, why do we even need to be able to store events in the queue?
(to put it another way, why 'waste' space prefixing the lambda with an enum discriminator if you could instead always express 'broadcast this event' as a lambda and only support queuing lambdas? In for a penny, in for a pound)

[1] https://answers.launchpad.net/gcc-arm-embedded/+question/686870/+index

@mspang

Is it correct? It seems like it may not be [1]

It is not same as [1], his usage is obviously wrong, he created the object twice.

Do we really need it? (would a single function ptr and user_data do?)

user_data doesn't contains type information, using lambda, the type information of the posted data will be encoded into the handler function.

Do people understand the limitations?

There are static_assert that make sure people do not make mistakes even if doesn't understand the limitations

Why mix the call-once nature of lambdas with the broadcast nature of other 'events' into a single API ?

I'm not quite get this one, lambdas can be called multiple times, we can broadcast lambda object if we want.

If we have lambdas which is undoubtedly a more flexible API, why do we even need to be able to store events in the queue?

it is a limitation of RToS xQueue API, The API can only post fixed size objects.
https://www.freertos.org/a00117.html
 BaseType_t xQueueSend(
                            QueueHandle_t xQueue,
                            const void * pvItemToQueue,
                            TickType_t xTicksToWait
                         );

(to put it another way, why 'waste' space prefixing the lambda with an enum discriminator if you could instead always express 'broadcast this event' as a lambda and only support queuing lambdas? In for a penny, in for a pound)

This PR provides a better interface than previous, more flexible and type-safe. It will still take to to migrate all legacy code into the new API. Once all usage are migrated, we can safely remove the enum

[1] https://answers.launchpad.net/gcc-arm-embedded/+question/686870/+index

@mspang

Is it correct? It seems like it may not be [1]

It is not same as [1], his usage is obviously wrong, he called the constructor twice.

Do we really need it? (would a single function ptr and user_data do?)

user_data doesn't contain type information, using lambda, the type information of the posted data will be encoded into the handler function.

Do people understand the limitations?

There are static_assert that make sure people do not make mistakes even if doesn't understand the limitations

Why mix the call-once nature of lambdas with the broadcast nature of other 'events' into a single API ?

I'm not quite get this one, lambdas can be called multiple times, we can broadcast lambda object if we want.

If we have lambdas which is undoubtedly a more flexible API, why do we even need to be able to store events in the queue?

it is a limitation of RToS xQueue API, The API can only post fixed size memory region.
https://www.freertos.org/a00117.html
 BaseType_t xQueueSend(
                            QueueHandle_t xQueue,
                            const void * pvItemToQueue,
                            TickType_t xTicksToWait
                         );

(to put it another way, why 'waste' space prefixing the lambda with an enum discriminator if you could instead always express 'broadcast this event' as a lambda and only support queuing lambdas? In for a penny, in for a pound)

This PR provides a better interface than previous, more flexible and type-safe. It will still take time to migrate all legacy code into the new API. Once all usage are migrated, we can safely remove the enum

You don't need to migrate the call sites of PostEvent, you can keep those as is and make PostEvent convert the event to a lambda.
All you need to do it is change the array of union of events and lambdas to an array only of lambdas and implement "dispatch event" as a lambdas.
The reason to do it is to make more efficient use of the space in the queue.

[1] https://answers.launchpad.net/gcc-arm-embedded/+question/686870/+index


bug-accessibility: Logo missing alt text

Task
Add alt text for the logo located in the header and footer. This is important so that screen reader users can understand the image, otherwise it can be picked up as a "graphic" or have the name of the file read out.

Proposed alt text:

Header: "Women Coding Community home" since it's a link that links to the home page
Footer: "Women Coding Community logo"

Hello, I would like to contribute if the issue is still available. Thank you!

Hey @MitaliShah, yes it's still available. Feel free to work on it!

@cathy-codes Thanks!


[SR-3916] Light-weight preconditions on (protocol) method parameters

Previous ID
SR-3916

Radar
None

Original Reporter
Reitzig (JIRA User)

Type
New Feature

Additional Detail from JIRA

Votes
0

Component/s
Compiler

Labels
New Feature, LanguageFeatureRequest

Assignee
None

Priority
Medium

md5: e627ff48c34c472ba7d52c1a3b9736ad

Issue Description:
Say I have a protocol like this:

Closing because the description of this feature seems to have been lost during the transition from Jira.


Customizing mouse hand (osu)

Hi there! I'm an artist and have customized my own bongo cat extension and was trying to make another one which required me to draw hands instead of the bongo's paws. I could customize the keyboard hand just fine but there was no file for me to edit/customize the mouse tracking hand. Is there a current way to do this?

@AerlyaGraphics You can change color and border color of paw at the setting, look at "paw", "pawEdge" in osu section.

Thank you! Do you know if there is a way to redraw the tracking paw entirely rather than just edit the color?

@AerlyaGraphics Unfortunately, you cannot use custom image with tracking paw.
Only color change support right now.


Revert "feat(pingcap/tidb): block mergeing for pull requests which have none finished test tasks"

Reverts ti-community-infra/configs#953

/approve


Resource Subtype Display - Masthead

Spinning out a new issue from rancher/dashboard#966 re: displaying the resource subtype in masthead.
Masthead should be aware of any subtypes selected in CruResource and display that subtype:
<ResourceType>: <Action> <Subtype-ifexists>

Current 2 resources that use subtypes:
Workload
Service
Evnetually secret will use it as well

Workload was changed to go directly to the CRD, but Service is a sub-type that's not an actual CRD, so is a special use case. Secret will fall under the same category.

Workload was changed due to the "subtype" of workloads were actual CRDs. This change will need to be used for Service and eventually secret.

@tfiduccia Can you test that this is working for Service where you will be able ot see the subtype after it's selected?

Fixed


Resolver registry

Pull Request Test Coverage Report for Build 1086

16 of 21 (76.19%) changed or added relevant lines in 6 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.07%) to 83.201%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/GraphQL.php
2
3
66.67%

src/SchemaBuilder/ResolverMapRegistry.php
4
8
50.0%

Totals

Change from base Build 1080:
-0.07%

Covered Lines:
4611

Relevant Lines:
5542

💛  - Coveralls

Pull Request Test Coverage Report for Build 1088

16 of 21 (76.19%) changed or added relevant lines in 6 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.05%) to 83.216%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/GraphQL.php
2
3
66.67%

src/SchemaBuilder/ResolverMapRegistry.php
4
8
50.0%

Totals

Change from base Build 1080:
-0.05%

Covered Lines:
4611

Relevant Lines:
5541

💛  - Coveralls


browse datasets error boundaries

Adds error boundaries to various components on the browse datasets page. This is to help identify the issue for the insertNode error that happens intermittently (see #215).
This should also improve the UX around the error by preventing the entire application from crashing, and allowing the user to refresh the error via a button.
This also sets up server side logging for errors that happen within the error boundary so that any errors that aren't captured as a result of this are logged somewhere in CloudWatch so that we can inspect it later.
This does not fix the error, this is to purely help with identifying the issue + improving UX around the error
Demo
https://github.com/chanzuckerberg/cryoet-data-portal/assets/2176050/eed7d29e-4894-4c67-9804-8bfefe110088

awesome


Make actor pad automagically calculated

Prior to this commit, we had to manually update the actor pad size if the size of pony_actor_t changed.
This commit makes it so that the actor pad size is now automagically calculated by the compiler so it doesn't have to be manually maintained any longer.

added the warning


Failing to upload an mp4 video using postMultiPartRequest

Description: I'm trying to upload an mp4 video file but it's failing with the following error:
Response-Body={"message":"Missing OAuth1 parameters (oauth_consumer_key, oauth_timestamp, oauth_nonce, oauth_signature, oauth_signature_method, oauth_token, oauth_version)","code":"oauth-gateway","data":{"status":502}
My code:
let urlStringUp = "https://xxxx/recorded-videos"
fileToUploadUrl = fileManager.urlForDocumentsDirectoryFile(fileName: fileName)
let videoData = try Data(contentsOf: fileToUploadUrl)
let fileData = OAuthSwiftMultipartData(name: "file", data: videoData, fileName: fileName, mimeType: nil)
let multiparts = [fileData]
let binary = "binary".data(using: OAuthSwiftDataEncoding)!
let parameters: OAuthSwift.Parameters = [ "media": binary]
oauthSwift.client.postMultiPartRequest(urlStringUp, method: .POST, parameters: parameters , multiparts: multiparts)

OAuth Provider? (Twitter, Github, ..):
OAuth Version:

[x] Version 1
[ ] Version 2

OS (Please fill the version) :

[x] iOS :
[ ] OSX :
[ ] TVOS :
[ ] WatchOS  :

Installation method:

[ ] Carthage
[x ] CocoaPods
[ ] Swift Package Manager
[ ] Manually

Library version:

[ ] head
[x] v2.0.0
[ ] v1.4.1
[ ] v1.2.1
[ ] v1.2 (Swift 4.0)
[ ] v1.0.0
[ ] v0.6
[ ] other: (Please fill in the version you are using.)

Xcode version:

[ ] 11.x (Swift 5.1)

[ ] 10.x (Swift 5.0)

[x] 10.x (Swift 4.1)

[ ] 9.3 (Swift 4.1)

[ ] 9.0 (Swift 4.0)

[ ] other: (Please fill in the version you are using.)

[ ] objective c

you do not respond to the important question, the provider
you put xxx, is it a private server?

Well - it's a company server. The url is: "https://upload.onlinetennisinstruction.com/wp-json/oti-api/1.0/users/current/recorded-videos"


Merge lost commits into branch dev

Changes Unknown when pulling c98a7ebd9d40c463d56ec593d72ec61e878c66d9 on ywk253100:m_t into ** on vmware:dev**.


Search by ResourceId

Hi guys.
The issue #157 was closed but i don´t discover how to use Search player by id.
i tried use ResourceId (-2096978694) or AssetID (173306)
 var search = new PlayerSearchParameters
                        {
                            Page = 1,
                            ResourceId = 173306 // Ansaldi -2096978694
                        };

But this is returning all different players...
I see in firebug (Firefox Plugin) at WebSite that the request url was:
https://utas.s2.fut.ea.com/ut/game/fifa16/transfermarket?type=player&start=0&maskedDefId=173306&num=16
This is similar url that toolkit is generating... maybe something different in the header? Anyone could help?

Very strange... i tried another players like Rafinha (assetId 201400) and works...
Some player like this ansaldi had problem in his JSON (his club and league are wrong) response and search by ID doensn´t works... I don´t know how WebApp this search works...

mano,
Fiz uma pesquisa com o AssetID (173306) e retornou o Cristian Ansaldi.
O ResourceID não pode ser negativo.
Está apresentando erro somente quando você pesquisa com esse assetID?

Opa,
realmente com o assetId esta funcionando... testei agora novamente.
Solved.
The real number is AssetId, not ResourceId.
Thanks


Build failure: waybar (nixos-unstable)

Steps To Reproduce
Steps to reproduce the behavior:

build waybar with latest nixpkgs rev as of writing

Build log
l@nix { "action": "setPhase", "phase": "unpackPhase" }
Running phase: unpackPhase
unpacking source archive /nix/store/5q3fdaixac9hv71rva2xal78l195lv4n-source
source root is source
/build/source /build
patching script interpreter paths in .
/build
@nix { "action": "setPhase", "phase": "patchPhase" }
Running phase: patchPhase
@nix { "action": "setPhase", "phase": "updateAutotoolsGnuConfigScriptsPhase" }
Running phase: updateAutotoolsGnuConfigScriptsPhase
@nix { "action": "setPhase", "phase": "configurePhase" }
Running phase: configurePhase
mesonConfigurePhase flags: --prefix=/nix/store/77n7cw79znnsrdg5qxa3jv03m7h8pwrl-waybar-0.10.0 --libdir=/nix/store/77n7cw79znnsrdg5qxa3jv03m7h8pwrl-waybar-0.10.0/lib --libexecdir=/nix/store/77n7cw79znnsrdg5qxa3jv03m7h8pwrl-waybar-0.10.0/libexec --bindir=/nix/store/77n7cw79znnsrdg5qxa3jv03m7h8pwrl-waybar-0.10.0/bin --sbindir=/nix/store/77n7cw79znnsrdg5qxa3jv03m7h8pwrl-waybar-0.10.0/sbin --includedir=/nix/store/77n7cw79znnsrdg5qxa3jv03m7h8pwrl-waybar-0.10.0/include --mandir=/nix/store/77n7cw79znnsrdg5qxa3jv03m7h8pwrl-waybar-0.10.0/share/man --infodir=/nix/store/77n7cw79znnsrdg5qxa3jv03m7h8pwrl-waybar-0.10.0/share/info --localedir=/nix/store/77n7cw79znnsrdg5qxa3jv03m7h8pwrl-waybar-0.10.0/share/locale -Dauto_features=enabled -Dwrap_mode=nodownload --buildtype=plain -Dcava=enabled -Ddbusmenu-gtk=enabled -Djack=enabled -Dlibinput=enabled -Dlibnl=enabled -Dlibudev=enabled -Dman-pages=enabled -Dmpd=enabled -Dmpris=enabled -Dpulseaudio=enabled -Drfkill=enabled -Dsndio=enabled -Dsystemd=disabled -Dtests=enabled -Dupower_glib=enabled -Dwireplumber=enabled -Dexperimental=true
The Meson build system
Version: 1.3.2
Source dir: /build/source
Build dir: /build/source/build
Build type: native build
Project name: waybar
Project version: 0.10.0
C compiler for the host machine: gcc (gcc 13.2.0 "gcc (GCC) 13.2.0")
C linker for the host machine: gcc ld.bfd 2.41
C++ compiler for the host machine: g++ (gcc 13.2.0 "g++ (GCC) 13.2.0")
C++ linker for the host machine: g++ ld.bfd 2.41
Host machine cpu family: x86_64
Host machine cpu: x86_64
Compiler for C++ supports link arguments -lc++fs: NO
Compiler for C++ supports link arguments -lstdc++fs: YES
Program git found: NO
Checking if "nl_langinfo with _NL_TIME_WEEK_1STDAY, _NL_TIME_FIRST_WEEKDAY" : links: YES
Run-time dependency threads found: YES
Found pkg-config: YES (/nix/store/a3752l19hc76dmzh1wgrxbsxh6lfx67z-pkg-config-wrapper-0.29.2/bin/pkg-config) 0.29.2
Run-time dependency fmt found: YES 10.2.1
Run-time dependency spdlog found: YES 1.13.0
Run-time dependency wayland-client found: YES 1.22.0
Run-time dependency wayland-cursor found: YES 1.22.0
Run-time dependency wayland-protocols found: YES 1.33
Run-time dependency gtkmm-3.0 found: YES 3.24.8
Run-time dependency dbusmenu-gtk3-0.4 found: YES 16.04.0
Run-time dependency gio-unix-2.0 found: YES 2.78.4
Run-time dependency jsoncpp found: YES 1.9.5
Run-time dependency sigc++-2.0 found: YES 2.12.1
Did not find CMake 'cmake'
Found CMake: NO
Run-time dependency libinotify found: NO (tried pkgconfig and cmake)
Run-time dependency epoll-shim found: NO (tried pkgconfig and cmake)
Run-time dependency libinput found: YES 1.25.0
Run-time dependency libnl-3.0 found: YES 3.8.0
Run-time dependency libnl-genl-3.0 found: YES 3.8.0
Run-time dependency upower-glib found: YES 1.90.2
Run-time dependency libpipewire-0.3 found: YES 1.0.3
Run-time dependency playerctl found: YES 2.4.1
Run-time dependency libpulse found: YES 17.0
Run-time dependency libudev found: YES 255
Run-time dependency libevdev found: YES 1.13.1
Run-time dependency libmpdclient found: YES 2.22
Run-time dependency xkbregistry found: YES 1.5.0
Run-time dependency jack found: YES 1.9.22
Run-time dependency wireplumber-0.4 found: NO (tried pkgconfig)

meson.build:95:17: ERROR: Dependency "wireplumber-0.4" not found, tried pkgconfig

A full log can be found at /build/source/build/meson-logs/meson-log.txt

Additional context
Add any other context about the problem here.
Notify maintainers
@FlorianFranzen
@lovesegfault
@minijackson
@rodrgz
@synthetica
@khaneliman
Metadata
[user@system:~]$ nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 6.6.21, NixOS, 24.05 (Uakari), 24.05.20240314.d691274`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.18.1`
 - nixpkgs: `/nix/store/9mf8fkjw3vw1gm4s6j5fqlwrv5wf2knf-source`

Add a :+1: reaction to issues you find important.

Can't reproduce, just updated my repository and built it just fine.

Already fixed in https://github.com/NixOS/nixpkgs/commit/49f83b701e7939079c529f378c79fa8544f4db72.

@arthsmn How would I manually apply the fix in a flake? I'm sort of new to nixos so I would appreciate the help.

@arthsmn How would I manually apply the fix in a flake? I'm sort of new to nixos so I would appreciate the help.

waybar.override {
    wireplumber = prev.wireplumber.overrideAttrs rec {
      version = "0.4.17";
      src = prev.fetchFromGitLab {
        domain = "gitlab.freedesktop.org";
        owner = "pipewire";
        repo = "wireplumber";
        rev = version;
        hash = "sha256-vhpQT67+849WV1SFthQdUeFnYe/okudTQJoL3y+wXwI=";
      };
    };
  };

This issue has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/waybar-failing-to-build-on-nixos-rebuild/41972/5


Performance on mobile devices

I'm submitting a ...  (check one with "x")
[] bug report => search github for a similar issue or PR before submitting
[x] feature request
[ ] support request => Please do not submit support request here

Current behavior
Performance on iOS and probably other mobile devices is very slow and laggy.

Expected behavior
Scrolling across the table should be smoother.
Reproduction of the problem
Tested with demo page.
http://swimlane.github.io/ngx-datatable/#
What is the motivation / use case for changing the behavior?
As Angular is mobile first framework, ngx-datatable should work on mobile devices too.
Please tell us about your environment:
iOS 10

Table version: 0.7.x
7.0.0

Angular version: 2.0.x
4.0.0

Browser: [all | Chrome XX | Firefox XX | IE XX | Safari XX | Mobile Chrome XX | Android X.X Web Browser | iOS XX Safari | iOS XX UIWebView | iOS XX WKWebView ]
iOS 10

Language: [all | TypeScript X.X | ES6/7 | ES5]
all

@molcik
I've checked iPhone 6, 5, iPad mini 1, iPad 2 and some Android devices.
All the ios devices that I mentioned above have this low-performance issue. The performance is even worse when "frozenLeft" or "frozenright" is set to true. Check http://swimlane.github.io/ngx-datatable/#pinning on any ios device.
For Android devices with chrome, don't have this performance issues.
As only ios devices have this low-performance issues with chrome and safari, I guess this is an issue just related to ios platform


Execution optimizations have been disabled

Expected Behavior
Ktlint runs without Gradle warnings
Observed Behavior
> Task :ultrasonic:ktlintTestSourceSetFormat
Execution optimizations have been disabled for task ':ultrasonic:ktlintTestSourceSetFormat' to ensure correctness due to the following reasons:
  - Gradle detected a problem with the following location: 'ultrasonic/ultrasonic/src/test/java'. Reason: Task ':ultrasonic:ktlintTestSourceSetFormat' uses this output of task ':ultrasonic:ktlintKotlinScriptFormat' without declaring an explicit or implicit dependency. This can lead to incorrect results being produced, depending on what order the tasks are executed. Please refer to https://docs.gradle.org/7.0/userguide/validation_problems.html#implicit_dependency for more details about this problem.
  - Gradle detected a problem with the following location: 'ultrasonic/ultrasonic/src/test/kotlin'. Reason: Task ':ultrasonic:ktlintTestSourceSetFormat' uses this output of task ':ultrasonic:ktlintKotlinScriptFormat' without declaring an explicit or implicit dependency. This can lead to incorrect results being produced, depending on what order the tasks are executed. Please refer to https://docs.gradle.org/7.0/userguide/validation_problems.html#implicit_dependency for more details about this problem.

Deprecated Gradle features were used in this build, making it incompatible with Gradle 8.0.
Use '--warning-mode all' to show the individual deprecation warnings.
See https://docs.gradle.org/7.0/userguide/command_line_interface.html#sec:command_line_warnings

Steps to Reproduce
Run ./gradlew -Pqc --build-cache --max-workers 6 ktlintFormat on https://github.com/ultrasonic/ultrasonic
Your Environment

ktlint               : "0.37.1",
ktlintGradle         : "9.2.1",
gradle               : '7.0',

Issue in the plugin and not ktlint itself: https://github.com/JLLeitschuh/ktlint-gradle/issues/457


Reordered the launch process, code push first then webhook creation

Pushing code later may trigger a build and two builds are getting triggered, so first push code
then create webhook and we are already trigerring build later.
Fixes https://github.com/openshiftio/openshift.io/issues/3742
Checklist

[] I followed the contribution guidelines.
[] I created an issue and used it in the commit message.
[] I have built the project locally prior to submission with mvn clean install.
[] I kept a clean commit log (no unnecessary commits, no merge commits).
[] I am happy with my contribution.

Ike Plugins (test-keeper)
Thank you @piyush1594 for this contribution!

It appears that no tests have been added or updated in this PR.
Automated tests give us confidence in shipping reliable software. Please add some as part of this change.
If you are an admin or the reviewer of this PR and you are sure that no test is needed then you can use the command /ok-without-tests as a comment to make the status green.
For more information please head over to official documentation. You can find there how to configure the plugin.

@piyush1594 don't forget to change OsioImportMissionControl#launch as well

@gastaldi Addressed, but please not merge it for the time being

Looks good, can you change your commit message to what's specified in https://github.com/fabric8-launcher/launcher-backend/blob/master/COMMIT_MSG.md? Thanks!

@piyush1594 I added the do not merge label to this PR. Feel free to remove it when you need it merged

@gastaldi Thanks, will properly test it tomorrow with help of @ravsa and then do all the changes if required and commit message formatting

/ok-without-tests

@piyush1594 almost forgot, you need to change the event order in https://github.com/fabric8-launcher/launcher-backend/blob/master/addons/osio-addon/src/main/java/io/fabric8/launcher/osio/web/endpoints/OsioLaunchEndpoint.java#L57 and https://github.com/fabric8-launcher/launcher-backend/blob/master/addons/osio-addon/src/main/java/io/fabric8/launcher/osio/web/endpoints/OsioImportEndpoint.java#L56 as well

@gastaldi Addressed the changes

@gastaldi I tested the full launch mission flow with the help of @ravsa and its working fine. I think we are good to merge this PR. I don't have rights to remove the do not merge label.

Thanks for the PR!

Just for tracking, this is in prod by https://github.com/openshiftio/saas-launchpad/pull/178
Thanks @gastaldi


Complete unqualified image names

Add 'docker.io' registry server name where missing.
That allows applications to run on CRIs not configured to handle unqualified registries.

@kukacz thank you. Might I ask to push this to dev branch?
We use dev as the development branch. When we cut a release, we cut it from dev and then sync dev to main
thank you so much again!


Search box location/behavior

Can we make the search box a simple icon that activates the full search functionality. Think it might be more appropriate far right on the nav.

@kevindherman @aschmoe thoughts?  I the reasoning behind making it an actual search box is that quick search is the number 1 thing we want to highlight

If we want the full form visible I'd suggest:

keep it to the far right in same row as logo (ex: http://www.tn.gov/, https://www.gov.uk/browse/business, http://www.nc.gov/, https://alpha.phila.gov/browse/business/)
creating a separate row for the nav elements

I agree that it should be the full search box. I like the right side examples for TN and UK more than separate row.

I think http://www.tn.gov/ bar functionality is best.  Sticks in header if form is on page, otherwise always in header.
Clicking icon expands header fixed overlay to display full form.

@aschmoe mind setting this up at some point on http://pattern.getproudcity.com/?v=&page=01-page

Moving to closed afterdiscussions


fix(active-users): remove api calls until after beta release

@Bouncey lets keep this on hold, we have a long pending issue with chrome caching requests. And our use of Rx and LB is complicating this. This is getting mixed up with authenticated requests and I have to admit I think its worth taking in after we have open-api for this instead.

What is the issue with caching?

If you make authenticated requests and unauthenticated requests in parallel especially with something like jQuery ajax, it just gets combined together.
And more often than not, the Rx part where we combine actions starts to crash.
This is because a lot of the LB internals is synchronous.
There is no clean way to reproduce this currently.

 Which RX parts?

Well it crashes at times at the get services, I think. Especially while at GET Passwordless methods. It sporadic on development though.
Express will cache those requests and continue to serve them on an page that's left open. While an email is sent and is opened on a separate tab.
https://github.com/freeCodeCamp/freeCodeCamp/pull/8931/files is where we had to reverse this, untill finding a workaround.

Do we have an issue tracking this cache issue?

I just created one.


Gowri wee1 background package QA deployment ready

Gowri.log.message.week<IP_ADDRESS>.zip

Process Execution Completed, I downloaded Gowri's zip file and deployed it in the orchestrator. My issue number is #1446


Exploring Bucket(remote store) level replication

If the leader cluster writes the segments & translogs to its remote store, follower can fetch the data from leader's remote store directly. For cross region cases, data transfer can be offloaded to the underlying remote store. So we can have external replication between leader & follower's remote store. With this approach, we can rely on the store's replication guarantees to be ensured that leader's data is always available to the follower.
Open questions:

[ ] Checking the technical feasibility of cross region bucket replication for different storage solutions(existing repository plugins). Can we always rely on bucket level replication? Or a fallback mechanism to access the leader's remote store is required as well?
[ ] Capture the steps for achieving the bucket level replication(with SLAs) for different storage solutions.

S3 SRR/CRR:
Pre-requisite: https://docs.aws.amazon.com/AmazonS3/latest/userguide/replication.html#replication-requirements
Steps:

Create a role for replication with necessary permissions.
Update follower's bucket resource policy to allow the role above(for cross account usecases)
Add a replication configuration on leader's bucket to replicate all objects or subset.

SLA(ref):

Happy case: seconds
p99: 5 minute
p99.99: 15 minute

Google Cloud Storage
SLA: replicated within 15 min
Steps: https://cloud.google.com/storage/docs/managing-turbo-replication#permissions-rest

Azure blob store replication
Steps: https://learn.microsoft.com/en-us/azure/storage/blobs/object-replication-configure?tabs=portal
SLA: no SLA


Separate Forwarder and ForwarderDF

Much of the state and code associated with Forwarder only applied in the
DF case.

Fixups squashed into the single commit. github kept the comments this time, yay.

func(err error) error { return err }}. Hilarious :)


[RFC-51][HUDI-3478] Update RFC: CDC support

Follow up from https://github.com/apache/hudi/pull/5436

@xushiyan - Do you want to take a final stab at the RFC incorporating all the changes? I believe we have consensus on what needs to be done. Please correct me if I am wrong.
cc @YannByron @danny0405

@xushiyan - Do you want to take a final stab at the RFC incorporating all the changes? I believe we have consensus on what needs to be done. Please correct me if I am wrong. cc @YannByron @danny0405

@prasannarajaperumal I've updated the RFC as per the recent discussions.


Marketprice Change Tracker and Weekly Budget Update

New Features Added:
Weekly Budget Monitoring:
Introduced a feature to track weekly expenses.
Users can set a weekly budget limit.
Alerts notify users when expenses exceed the weekly limit.
Market Price Change Detection:
Implemented functionality to detect changes in market prices.
Tracks updates in regularly uploaded market price data.
Alerts users about changes in prices or VAT rates for better financial planning.
Purpose:
The weekly budget feature helps users manage expenses efficiently.
The market price tracking system ensures users stay informed about price fluctuations and VAT changes, making it easier to plan their finances.

New Features Added:
Weekly Budget Monitoring:
Introduced a feature to track weekly expenses. Users can set a weekly budget limit. Alerts notify users when expenses exceed the weekly limit. Market Price Change Detection:
Implemented functionality to detect changes in market prices. Tracks updates in regularly uploaded market price data. Alerts users about changes in prices or VAT rates for better financial planning. Purpose:
The weekly budget feature helps users manage expenses efficiently. The market price tracking system ensures users stay informed about price fluctuations and VAT changes, making it easier to plan their finances.

these are great changes but not sure what code changes you made. it seems like it's coming from the Jupyter Notebooks, which is fine. I also noticed you made a push to the main branch. I would encourage you to know how to use branches. It's probably better to create your own separate branch and then if you make a local changes you push to that branch.
If you make changes, rather than pushing to the main branch, you should push to that branch and then create a pull request. The pull request would allow me to review whether it's okay, and if I accept it, it will merge to the main branch.
more on branches here: https://www.atlassian.com/git/tutorials/using-branches