Handing in Big Customer / Out of Curiosity / Chemical Pt. 4 causes double rep loss

However, it appears to have a side effect of causing double the rep loss when handing in any of these 3 quests now.
Originally posted by @Aburiu in https://dev.sp-tarkov.com/SPT-AKI/Issues/issues/278#issuecomment-21276

Originally written by chomp

Fixed with: https://dev.sp-tarkov.com/SPT-AKI/Server/commit/47a84e508b7a6418002ab1e0637811e94f535615
This may break some quests with wait times not appearing correctly (show instantly instead of after x hours)

Originally written by chomp

From testing so far, it looks good, Gunsmith Part 6 has not instantly shown up after completing Part 5.

Originally written by Aburiu

Just had the quest Chemical Part 4 apply the 0.25 rep loss twice on a build from Feb 29 (https://dev.sp-tarkov.com/SPT-AKI/Server/commit/c64e5f72399e1d2cf8e30be00c95b3a737db6070).
For some reason, I don't have logs in BepInEx/ and Logs/, so I'm only attaching server-*.log
This was fixed with https://dev.sp-tarkov.com/SPT-AKI/Server/commit/47a84e508b7a6418002ab1e0637811e94f535615 but I think https://dev.sp-tarkov.com/SPT-AKI/Server/commit/64810f72166bb9e665485323f44b3ed6b5b0e2eb reintroduced the bug.

Attachments

server-2024-03-10.log (229 KiB)

Originally written by belette

Can't reproduce on Mar 10 build. Are there any special steps?

Attachments

2024-03-10[19-58] (0).png (3165 KiB)

Originally written by 5o2

Tested on latest commit as well, unable to reproduce this

Attachments

image.png (209 KiB)

Originally written by DrakiaXYZ

Thanks for looking into it.
My steps were:

go for the Therapist variant (Out of Curiosity) by planting a marker in Customs
survive the raid
complete quest at Therapist
observe -0.50 rep at Prapor and Skier instead of -0.25

I'd be happy to try and repro it on latest bleeding edge build if I can get pointers on how to quickly get a new account to this quest state. Do we have debug commands I can use or is it all manual fiddling with save files?

Originally written by belette

You can create a new profile with the "SPT Developer" type, select USec, and all quests will be available to accept.
Tested by handing in to Therapist, still only reduced rep one time for the other traders

Attachments

image.png (126 KiB)

Originally written by DrakiaXYZ

Thank you, I will try to repro on both builds and report back.

Originally written by belette

Welp, I managed to repro once on the Feb29 version, could not repro on Mar10, then could not repro on Feb29 anymore... I tried doing it in raid too, etc. but no dice. Maybe I had some cache/bad state from earlier bleeding edge versions, I'm not sure. In any case, I learned a lot about the project so hopefully my future reports will come with consistent repro steps.
This seems fixed and can be closed, thank you for your time.

Originally written by belette


Change constructor to use DEFAULT_ value for ext oscillator PS clk rate

This makes it easier to modify when porting PYNQ to other boards that don't have the same clock rates.

I took the PS Clock commit in #1220 ...


Add perceptualTolerance parameter to Diffing+bitmap to compare perceptual differences using Delta E 1994

Differences from swift-snapshot-testing
Similar to what was implemented in swift-snapshot-testing, with some slight differences
To match the established pattern of tolerance, this PR adds an optional perceptualTolerance parameter that allows you to specify the percentage each pixel can be different from source pixel and still considered a match. This parameter complements the existing tolerance parameter that determines the percentage of pixels that must be considered matching in order to consider the whole image matching.
The default value of 0.0 means pixels must match perfectly (and performs existing equality check). Any other value greater than 0.0 and less than or equal to 1.0, will be used when comparing each pixel where 0.0 (0%) means no difference and 1.0 (100%) means completely opposite. A suggested value of 0.01-0.02 is similar to the precision of the human eye.

Perceptual tolerance value
Description

0.0
Must be exactly equal

≤ 0.01
Allows differences not perceptible by human eyes

≤ 0.02
Allows differences possibly perceptible through close observation

≤ 0.1
Allows differences perceptible at a glance

≤ 0.5
Allows differences when more similar than opposite

≤ 1.0
Allows any differences

Unlike swift-snapshot-testing, which uses a CILabDeltaE CoreImage Filter, we must do a byte-by-byte comparison, calculating Delta E values as on-the-fly.
Formula resource: http://www.brucelindbloom.com/index.html?Eqn_XYZ_to_Lab.html
Description of https://github.com/pointfreeco/swift-snapshot-testing/pull/628...

Problem
The existing image matching precision strategy is not good at differentiating between a significant difference in a relatively small portion of a snapshot, and imperceivable differences in a large portion of the snapshot. For example, these snapshots below show that a 99.5% precision value fails an imperceivable background color change while allowing noticeable changes (text and color) to pass:

Reference
Fails
Passes

Imperceivable background color difference
Significant text and color changes

Solution
This PR adds a new optional perceptualPrecision parameter to image snapshotting which determines how perceptually similar a pixel must be to consider it matching. This parameter complements the existing precision parameter that determines the percentage of pixels that must be considered matching in order to consider the whole image matching.
This approach is similar to https://github.com/pointfreeco/swift-snapshot-testing/pull/571 and https://github.com/pointfreeco/swift-snapshot-testing/pull/580 but uses perceptual distance rather than Euclidean distance of sRGB values. This is significant because the sRGB color space is not perceptually uniform. Pairs of colors with the same Euclidean distance can have large perceptual differences. The left and right colors of each row have the same Euclidean distance:

Hey @balazstothofficial, I know there hasn't been much activity here recently, is this PR better directed towards my fork?

@JoelWhitney sorry for the delay
and thank you, @balazstothofficial, for tagging me, I'd have missed it!
@JoelWhitney, I've set @eldrig0 on it, and he's getting into the topic with the intention of getting it merged!

Hey @eldrig0 @KlausNie, is there going to be a new release published following this merge?

Yup, will do it tomorrow


Influx v2 support

The following PR adds support for InfluxDB v2.
We can easily test it with docker in the following manner.

Adjust bucket name and db accordingly:
influx v1 dbrp create --db benchmark --rp 0 --bucket-id `influx bucket ls --name bucket-perf | awk -v i=2 -v j=1 'FNR == i {print $j}'` --default

Retrieve the auth token as follows:
influx auth list

sample output:
root@01ae6fa8b638:/# influx auth list
ID			Description	Token												User Name	User ID			Permissions
0995d63be80e7000	influx's Token	AWgIqRrsmoo62gFlOV83K0egl-imbsN4pbNit5ykBDC2QXtJ-AHEe5sH-KQqq0N_VSLf-ES1Ydq6R9QFsV8Oxw==	influx		0995d63bd0ce7000	[read:authorizations write:authorizations read:buckets write:buckets read:dashboards write:dashboards read:orgs write:orgs read:sources write:sources read:tasks write:tasks read:telegrafs write:telegrafs read:users write:users read:variables write:variables read:scrapers write:scrapers read:secrets write:secrets read:labels write:labels read:views write:views read:documents write:documents read:notificationRules write:notificationRules read:notificationEndpoints write:notificationEndpoints read:checks write:checks read:dbrp write:dbrp]

Run full cycle mini test
Pass the auth-token via  INFLUX_AUTH_TOKEN env variable with the token retrieved on the previous step:
INFLUX_AUTH_TOKEN="AWgIqRrsmoo62gFlOV83K0egl-imbsN4pbNit5ykBDC2QXtJ-AHEe5sH-KQqq0N_VSLf-ES1Ydq6R9QFsV8Oxw==" ./scripts/full_cycle_minitest/full_cycle_minitest_influx.sh

dropping the database doesn't work in the shell script load_influx.sh:
curl --header 'Authorization: Token ...==' -X POST 'http://...:8086/query?q=drop%20database%20benchmark'
{"results":[{"statement_id":0,"error":"not implemented: DROP DATABASE"}]}


Context should not be secondary to session

According to Amazon's documentation, context is sent on every request (including AudioPlayer events) but session is only sent on standard requests.  As such we should not be clobbering the submitted context with information from context.  This is particularly dangerous when you also are clobbering token with sessionId, which makes it impossible to track which stream is currently playing on the client.
Citation:
https://developer.amazon.com/public/solutions/alexa/alexa-skills-kit/docs/alexa-skills-kit-interface-reference#context-object
https://developer.amazon.com/public/solutions/alexa/alexa-skills-kit/docs/alexa-skills-kit-interface-reference#session-object

Coverage increased (+0.9%) to 86.029% when pulling ae2640ac5ebb944cf7bcb8ad8b695004d12bff04 on trayburn:bugfix/context-over-session into f693d335815c0dad3e16c109e0ab3e7d405266d3 on matt-kruse:master.

I can merge, but what worries me here is that no tests broke. Maybe you want to reproduce this scenario in a test so that we never (un)break this again?

Coverage increased (+1.3%) to 86.347% when pulling 1af79d9f5f93d55d18eb2ac383d303a948c94096 on trayburn:bugfix/context-over-session into f693d335815c0dad3e16c109e0ab3e7d405266d3 on matt-kruse:master.

I agree that was suprising to me as well, I've added a test that ensures that new alexa.request(json) has a context property which hasn't changed at least the applicationId.

One moment guys!
I just tested my skill from Amazon Dev Testing Tool and faced with a request without context. So this PR is right, but doesn't provide ability to test on testing tools :(
I did this request:
{
  "version": "1.0",
  "session": {
    "new": true,
    "sessionId": "amzn1.echo-api.session.0000000-0000-0000-0000-00000000000",
    "application": {
      "applicationId": "amzn1.ask.skill.[my_app_id]"
    },
    "attributes": {},
    "user": {
      "userId": "amzn1.account.AM3B00000000000000000000000"
    }
  },
  "context": {
    "System": {
      "application": {
        "applicationId": "amzn1.ask.skill.[my_app_id]"
      },
      "user": {
        "userId": "amzn1.account.AM3B00000000000000000000000"
      },
      "device": {
        "supportedInterfaces": {
          "AudioPlayer": {}
        }
      }
    },
    "AudioPlayer": {
      "offsetInMilliseconds": 0,
      "playerActivity": "IDLE"
    }
  },
  "request": {
    "type": "LaunchRequest",
    "requestId": "amzn1.echo-api.request.0000000-0000-0000-0000-00000000000",
    "timestamp": "2015-05-13T12:34:56Z",
    "locale": "string"
  }
}

But request (a first argument) in the app.pre was w/o context:
{ data: { version: '1.0', session: { new: true, sessionId: 'amzn1.echo-api.session.0000000-0000-0000-0000-00000000000', application: [Object], attributes: {}, user: [Object] }, request: { type: 'LaunchRequest', requestId: 'amzn1.echo-api.request.0000000-0000-0000-0000-00000000000', timestamp: '2016-12-30T07:34:18Z' } }, slot: [Function], type: [Function], hasSession: [Function], getSession: [Function], userId: 'amzn1.account.AM3B00000000000000000000000', applicationId: 'amzn1.ask.skill.[my_app_id]', context: [Function], sessionDetails: { new: true, sessionId: 'amzn1.echo-api.session.0000000-0000-0000-0000-00000000000', userId: 'amzn1.account.AM3B00000000000000000000000', accessToken: null, attributes: {}, application: { applicationId: 'amzn1.ask.skill.[my_app_id]' } }, sessionId: 'amzn1.echo-api.session.0000000-0000-0000-0000-00000000000', sessionAttributes: {}, isSessionNew: true, session: [Function] }

My skill is hosted on Lambda if it makes sense.

@fremail @trayburn In theory I don't think we should be constructing a fake context for a testing scenario. What do you think we should do?

@fremail with the code as in this PR, what's the first call/thing to break in your app?

@dblock I check applicationId and if it's wrong (or null), the app doesn't work. That's for security reasons.
I agree, making fake context is not a good concept. I think we need to warn users that they may catch such error on testing.

Agreed @fremail, I think applicationId should return null when not supplied and the app could only make that check in "production". Maybe extend the example in README and add an explanation?
@trayburn I'll merge after you address my comments above, thanks for bringing this up and fixing it!

I believe that this update to the PR addresses all of the comments that have been brought up here.  If I missed something please let me know.

Coverage increased (+1.5%) to 86.545% when pulling 0053cbf2f30cb2c06d10cecfec9aff285a583d47 on trayburn:bugfix/context-over-session into f693d335815c0dad3e16c109e0ab3e7d405266d3 on matt-kruse:master.

Perfect, merged.


Reduce length of examples

Contributor Comments
Reduce the length of the examples in the README.
Pull Request Checklist
Thank you for submitting a contribution to easy_infra!
In order to streamline the review of your contribution we ask that you review and comply with the below requirements:

[x] Ensure that make build completes successfully
[X] Rebase your branch against the latest commit of the target branch, which likely should be master
[X] If there is an issue associated with your Pull Request, align your PR title with this documentation

Undid the Update packages commit.  Will split into its own PR


ERROR

Why this error is coming? How will this be solved?
Error: The target entry-point "@angular/material/toolbar" has missing dependencies:

@angular/platform-browser
@angular/platform-browser/animations
@angular/forms

Please keep GitHub issues for bug reports / feature requests. Better avenues for troubleshooting / questions are stack overflow, gitter, mailing list, etc.

Please keep GitHub issues for bug reports / feature requests. Better avenues for troubleshooting / questions are stack overflow, gitter, mailing list, etc.


Tidy up calculation of acceleration metrics and add additional metrics

The calculation of acceleration metrics in functions g.applymetrics and g.metric is poorly structured, which complicates adding new acceleration metrics and comes with the risk that calculations are unnecessarily done twice. I aim to tidy this up.
Further, I aim to add the following metrics:

Absolute of low-pass filtered x-axis
Absolute of low-pass filtered y-axis
Absolute of low-pass filtered z-axis
Absolute of high-pass filtered x-axis
Absolute of high-pass filtered y-axis
Absolute of high-pass filtered z-axis
Absolute of band-pass filtered x-axis
Absolute of band-pass filtered y-axis
Absolute of band-pass filtered z-axis

If one assumes that the accelerometer is attached consistently in the same orientation then these metrics could provide insight in the direction of acceleration.

I have done this as part of the branch: issue289_unzipping. and will make a pull request for it in the upcoming weeks.


waveform view could be too wide

If the waveform is inside a scroll view, it could become too wide for the Metal texture size limit. We should clip the view to the screen and render only within the screen. This could result in a lot of texture churn while scrolling.
Another option is to have our waveform viewer render multiple waveforms at different screen coordinates.

+1
My understanding of metal is too low to help out on this one, but I'm reaching the texture limit in my project :(


normalize eol

Fix line endings in Markdown files from CRLF to LF and set EOL for text files to auto to ensure only LF are committed while native EOL is used for clone, checkout, etc.

this PR will reflect whatever solution is accepted in https://github.com/openziti/template-repo/pull/4


[Feature] Provider State Management #34

some general notes:

consider adding of static function to the provided models for convenience.

example:
  static TransactionsModel.of(BuildContext context, {bool listen = true}) =>
      Provider.of<TransactionsModel >(context, listen: listen);

Constructors come first in a class
https://github.com/flutter/flutter/wiki/Style-guide-for-Flutter-repo#constructors-come-first-in-a-class

PREFER making fields and top-level variables final.
https://dart.dev/guides/language/effective-dart/design#prefer-making-fields-and-top-level-variables-final

Re-opening


Improved logging

Improve logging for students by:

Removing excessive/confusing npm error logging and setting the npm loglevel to silent in .npmrc
Adding success messages to lint and fix commands
Adding a test command so that students using autograding can run the same command consistently
Adding custom formatters for HTML Validate, Stylelint, and ESLint with helpful tips for debugging and fixing errors

This is working on both Windows and Mac!


WASM compatibility

As we intend to use this client library in the browser, we require that it (and its dependencies) be compatible with go's WASM support. In the spike we learned that we will have to replace the websocket library used, to that end. We also learned that we  may need the use of go's build tags to mutually exclude wasm/non-wasm code for backwards compatibility. I think we'll have to use the wasm tag in our tests as well and ensure we're running tests with and without that tag (in CI/CD and manually in the mean time, perhaps with make).
Size of the wasm module will quickly become a priority as we get further along. I think it would be prudent to keep an eye out for (and avoid creating) places where code is unnecessarily imported or poorly organized, either fixing it as we go or taking note in a card in MW4All. This is going to be the first line of defense when it comes to keeping the output size down.
To test-drive this, I think we just need to choose a build tool and ensure that we have a task/target for running tests with the wasm tag and without to provide coverage on both builds.

Started working on this in the branch wasm-compat branch. Could not cherry-pick @bryanchriswhite's commit that added the wasm directory, so had to copy from the head of the bryan/wasm branch.
Also added build flags into files, so that existing code compiles fine on non-wasm platforms and wasm modules alone gets pulled for the wasm build.


Bad require statement in eyeglass-exports.js

eyeglass-exports.js file had a bug in it.

Merged. Thanks @chriseppstein!


Set a Key

After configuring Access Policy, it says Set up a key encryption key (optional), what is this used for? If it is optional means we can opt out and so, how the disk is encrypted?

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

ID: 0dbd38fa-89f0-320a-b517-b440be6927f9
Version Independent ID: 8b2eeee3-fb49-37e5-917d-f36bcd60ad29
Content: Prerequisites - Azure Disk Encryption for IaaS VMs
Content Source: articles/security/azure-security-disk-encryption-prerequisites.md
Service: security
GitHub Login: @msmbaldwin
Microsoft Alias: mbaldwin

@taromielnaz Thank you for the detailed feedback. We are actively investigating and will get back to you soon.

@taromielnaz KEK is an optional and if you want to encrypt the data encryption keys then you need to follow the instructions Set up a key encryption key (optional) else not required. Please refer to Azure Data Encryption-at-Rest for details.

@taromielnaz We will now proceed to close this thread. If there are further questions regarding this matter, please reopen it and we will gladly continue the discussion.


QlikView 12

Hello,
after an upgrade to QV 12 we got this error messages.
12:49:47.0999|ERROR|The message with Action 'http://ws.qliktech.com/QMS/11/IQMS/GetTimeLimitedServiceKey' cannot be processed at the receiver, due to a ContractFilter mismatch at the EndpointDispatcher. This may be because of either a contract mismatch (mismatched Actions between sender and receiver) or a binding/security mismatch between the sender and the receiver.  Check that sender and receiver have the same contract and the same binding (including security requirements, e.g. Message, Transport, None). (Error code: 10)
if it's possible that this helpfull program can work with QV12?
Regards, Henry

Hi Henry, I think it should be fairly easy to get it running with QV12, it more or less only needs to be recompiled against the QV12 service reference and/or update the app.config. I did this for one of my other repositories recently, will see if I can find time to do it early next week.

Hi Rikard,
thanks for this info. :-)

Hello Rikard,
i have compiled your source with an update of the QMSAPI.Service Reference. I delete the two additional new endpoint entries in the app.config and rebuild the project. Copied the 7 files from the bin/debug folder (all except the vshost files) to the QV Server.
And it works :-)

Great work! :) thx for letting me know!

Any chance for sharing this update? I've hit the same error.

Found a working versions: https://community.qlik.com/docs/DOC-5111


Android Sense  - Analytics CAR file missing

I was testing how Android sense functioned in the 3.1.0 m7 pack. After installing Android Sense publishing data, I clicked View device analytics.
I was given a 400 error saying it's not available. When discussed with Sinthuja, she mentioned that the CAR file that's needed for the analytics component is missing in the pack.

@sameerawickramasekara will work on this.

This issue is already fixed, and the andriod sense car file is there in the analytics profile by default. And hence closing this issue.


[requirements.txt] Update development requirements

What does this PR do?

Bump reno to version 3.
Move to using compatible release operator for dependencies.
Revert #5659

Motivation

Have at least one dependency which is Python 3 only so that the pipeline breaks if we add a CI check with no Python 3 support. Since we install the requirements.txt dependencies almost everywhere this should be a good way to check it.

Describe your test plan

The pipeline passes.

I think pinning an exact version was done on purpose, so we don't get a different version of a dependency bumped between eg the last RC and the final build which might introduce subtle changes.

I think pinning an exact version was done on purpose, so we don't get a different version of a dependency bumped between eg the last RC and the final build which might introduce subtle changes.

If the packages follow semantic versioning this should not be a problem (compatible release only allows bugfix releases) but I went back to exact version pinning just in case.

It's true they should be compatible, but if we want reproducible builds there should be zero changes. Note we don't get that even with the specific version pinning, since we don't pin dependencies of dependencies.


Adjust breakpoints for buttons "Get Weather" and "Show Forecast", City input alignment and width:auto of block elements.

Hi,
Buttons are fixed now, didn't need to adjust any breakpoint, just wrap both buttons in a div container. I also fixed the alignment of City input by removing a set ml-4 from the original code, screenshot provided. Finally, I've noticed some block elements were declared with width:auto, which is redundant since all block elements are width:auto by default. Went ahead and change it to 100% or removed them for code cleanliness.

Hope these changes increase the visual impact of this project  😀

@gabrielpoetaa I think you have removed the preloader css , as it is appearing at the corner ,

As well as the spacing is also req between dropdown option and input field

Hi,
sorry about that. Whenever I try to change the input elements (removing the ml-4, for example) the app.css file brakes and the pre loader is pushed to the corner. I tried loading tailwind with app.css being the input, but then the voice icon breaks.

I couldn't figure out why. If you have any tips please share because I'd like to learn. In the meantime, the original issue (buttons "Get Weather" and "Show Forecast") are fixed. They no longer bleed out of the page in smaller viewports. Can I close this PR and open a new one with the original issue?
Thank you!

Hi,
sorry about that. I was running tailwind along with the react file so whenever I made a change on the home component the app.css would trigger and break the pre-loader style. Its fixed now. I copied over the original files from both app.css and index.css so now it works just fine.
In order to fix the space between the drop-down menu and city input I didn't just deleted the ml-4 as I've done before. Now I've included a breakpoint on the city input, so it has the necessary spaces when it's in wider viewports (ml-4 applied) and keeps the alignment when in smaller screens.
Video where it shows the functionality - > https://streamable.com/2izzpg
Hope it works now! 😀

@gabrielpoetaa not a problem !
Will review it soon  : )

@gabrielpoetaa thankyou for your contribution !


Bug Report:

Please check for duplicates or similar issues, as well performing simple troubleshooting steps (such as deleting the export folder if it's an issue with compiling, reinstalling Kade Engine) before submitting an issue.
If you are playing a downloaded version of the game, what operating system are you using?
Windows (x86), Windows (x86_64), Linux, or macOS? Specify below.
Windows x86
What version of the game are you using? Look in the bottom left corner of the main menu.
1.3
Have you identified any steps to reproduce the bug? If so, please describe them below. Use images if possible.
All I do is open the debug menu and I just can't delete arrows at all.
Please describe your issue. Provide extensive detail and images if possible.
When I try to remove any arrows in the debug menu, nothing happens. I can't delete any arrows, but I can place arrows though. There was one time where I could delete arrows, but I have no idea how I fixed it and the next time I tried to rechart a song it went back to not working.
The debug menu works fine on mods that don't use KadeEngine, but using KadeEngine I can't delete any arrows at all.

This was fixed in the latest version of Kade Engine.


Enable the unicorn/prefer-logical-operator-over-ternary ESLint plugin rule

This leads to ever so slightly more compact code, and can in some cases remove the need for a temporary variable.
Please find additional information here:
https://github.com/sindresorhus/eslint-plugin-unicorn/blob/main/docs/rules/prefer-logical-operator-over-ternary.md

/botio test

/botio-windows browsertest


⚠ Bofrost.at has degraded performance

In bdb5a03, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 2145 ms

Resolved: Bofrost.at performance has improved in 3cc088c.


[LI-HOTFIX] Bypass cluster metadata auto refresh code path by default

The same patch is also made to the 3.0 branch in linkedin#330.
The PR linkedin#228 attempted to resolve provided bootstrap servers when
the metadata is exceeding a staleness threshold.  The config is covered
both on producer and consumer, and default behavior without configured
value is setting the timeout to Long.MAX_VALUE.
However, cruise-control is affected by the behavior as it implements a
similar mechanism on its own and directly uses NetworkClient. The code
would fail if an empty bootstrap server list is passed to NetworkClient, which
is the case for internal use of CC.
To resolve this, this patch aims to make the default value -1, and omit
the code path referencing bootstrap server when we see -1.
EXIT_CRITERIA = When linkedin#228 is ejected
More detailed description of your change,
if necessary. The PR title and PR message become
the squashed commit message, so use a separate
comment to ping reviewers.
Summary of testing strategy (including rationale)
for the feature or bug fix. Unit and/or integration
tests are expected for any behaviour change and
system tests should be considered for larger changes.
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

Can you please add a reference to the relevant CC code?
Apart from making this change inside kafka, is it possible for CC to deprecate its own approach and rely on the mechanism in the kafka client?

Sorry that my comment is a little late. I feel a more proper fix is to check whether the bootstrapServers are empty, and only set the forceClusterMetadataUpdateFromBootstrap to true when the bootstrapServers are not empty.

It may need some refactoring of the current implementation to make the proposed change above.

@gitlw the forceClusterMetadataUpdateFromBootstrap flag's semantic is an extra condition to enforce refresh even it's not due, but here what we want is a way to make it not refresh even it's due.
Do you think it'd be better if I perform the empty bootstrap server here at line 981, as a pre-condition before checking due?
i.e., change
        if (this.metadataUpdater.isUpdateClusterMetadataDue(now)) {
            // Resolve & refresh
        }

into
        // into this
        if (!this.bootstrapServers.isEmpty() && this.metadataUpdater.isUpdateClusterMetadataDue(now)) {
            // Resolve & refresh
        }

@lmr3796

I feel the isUpdateClusterMetadataDue should be renamed to something like isUpdateFromBootstrapDue.
Then, either put the this.bootstrapServers.isEmpty() checking into the if condition or inside the isUpdateFromBootstrapDue method. I slightly prefer the latter, but both look good to me.

@gitlw ,
After revisit here are points I'd like to seek for input.
By the new proposed approach, if I check bootstrap server emptiness to bypass the code path, then it means a (likely erroneous) config that provides a max age/timeout with empty bootstrap server would be silently swallowed, which seemed bad to me.
As for the approach in this PR

The -1 serves as a flag to disable the feature.
If bootstrap server is not provided but a positive timeout is set, which indicates user wants the refresh feature, there would be an error.


DRep retirement-certificate support for script based DReps

#What
As a script-based DRep (i.e. multisig) I want to be  able to generate a retirement (unregister) certificate, compliant with the conway cddl
unreg_drep_cert = (17, drep_credential, coin)

credential =
  [  0, addr_keyhash
  // 1, scripthash
  ]

So we need to expand drep retirement-certificate so that it can also take a scripthash, i.e.
cardano-cli conway governance drep retirement-certificate
                                                                   ( --drep-verification-key STRING
                                                                   | --drep-verification-key-file FILE
                                                                   | --drep-key-hash HASH
                                                                   | --drep-script-hash HASH
                                                                   )
                                                                   --deposit-amt LOVELACE
                                                                   --out-file FILE

Closed by https://github.com/IntersectMBO/cardano-cli/pull/678


Specify default FFMPEG and FFPROBE constants

Please specify default constants for FFMPEG and FFPROBE  to use instead of String, for example:
public class FFmpegConstants {
// File Extensions
public static final String AVI = "avi";
public static final String MP4 = "mp4";
public static final String OGG = "ogg";
public static final String MOV = "mov";
public static final String _3GP = "3gp";

//  Formats
public static final String MP4_FORMAT = "mp4";

// Audio
public static final String AAC_CODEC = "aac";

// Video
public static final String H264_CODEC = "libx264";

// Presets
public static final String ULTRAFAST_PRESET = "ultrafast";

}

I had actually tried to avoid doing this, because I didn't want to give the impression that the local ffmpeg supported those formats, or codecs, etc. You can however do new FFmpeg().formats() and get a list of formats supported by this instance.
I do have some constants, such as FPS_30, or AUDIO_STEREO, but these are universally accepted across all ffmpegs. Where as H264_CODEC = "libx264" would only be available on a ffmpeg compiled with x264 support.
Thoughts?

Anyway, usage of constants is right approach even it's supported optionally. I believe - it can be under developer's choice what constants to use and it's really nice to have constants to avoid creation of own ones

Related to PR #321


WIP: Bugfix: Party Component Sync Generic Patch Not Working

PR Checklist

Please check if your PR fulfills the following requirements:

[ ] All new classes have class-level documentation comments, if there are any at all
[ ] Tests for the changes have been added (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Documentation update
[ ] Other... Please describe:

What is the current behaviour?

What is the new behaviour?

Resolves #

Other information

Possibly create a harmony issue for this, patching multiple classes with a prefix and the instance type being different does not give any error or warning


Text outline opacity does not follow element's opacity

I think outline's opacity should be multiplied by element's opacity just like other engines.

Should shadow opacity also be multiplied?

@codebon77 i think that's more natural

@pjc0247 I agree it seems more natural, however it is also more restrictive. If we multiply outline alpha with text alpha, then for example it's not possible to create transparent text with a solid outline.
I think ideally outline and shadow alpha would be a value between -1..1 and would be added to text alpha. This way the outline and shadow inherit text alpha setting, but the user is still able to set any combination of alpha/solid.

@codebon77 I never saw the range of -1 ~ 1 for opacity value. I think many people don't get it why there are negative values unless they open the documentation.
And, the main problem is here, what I want to do was link there opacity values between element and shadow's. because I want to make a fadeout text.
Unity has two properties for opacity, one is color.a and another is renderer.alpha. So they can be controlled each values text and element's opacity.
Cocos2d has a function to set multiplying opacity or not which is setCascadeOpacity(boolean).
(this one is not excatly same as current issue, but I just explain the case)

@pjc0247 Yes, a completely separate "Element opacity" could work fine. This would be multiplied into colour/outline/shadow opacity.

@pjc0247 I just want to confirm, there is nothing stopping you from setting all three opacity values directly yourself right? You can fade the text this way. What we are discussing here is a convenience function that allows you to fade with a single value.

@codebon77 I thought it is a bug, but after some comments and enhancement label, I realized that this is what you intended to.
So, I confirm that everything works fine, I can control these 3 values. It just confused me because I was a Unity dev before using PlayCanvas.
I would say the current implementation of handling opacity is not convenient for who wants to make fading effects. Here's what I'm doing now.
tween(...).on('update', (v) => {
    text.opacity = v;
    text.outlineOpacity = v;
    text.shadowOpacity = v;
});

Was able to achieve the shadow opacity doing the following in an update loop.
var oldColor = this.entity.element.shadowColor;
var color = new pc.Color(oldColor.r, oldColor.g, oldColor.b, newAlpha);
this.entity.element.shadowColor = color;

fillColor - as a property, would bring good flexibility.
So there will be three separate colors with own opacities, gives flexibility.
And all of them would be multiplied by element.opacity.

Was able to achieve the shadow opacity doing the following in an update loop.
var oldColor = this.entity.element.shadowColor;
var color = new pc.Color(oldColor.r, oldColor.g, oldColor.b, newAlpha);
this.entity.element.shadowColor = color;
Creating pc.Color in each update loop - is unnecessary allocation. If abused, will lead to garbage collection stalls. Tip: allocate pc.Color in initialize, and just change it's properties in update.

fillColor - as a property, would bring good flexibility.
So there would be three separate colors with own opacities, gives flexibility.
And all of them would be multiplied by element.opacity.

Was able to achieve the shadow opacity doing the following in an update loop.
var oldColor = this.entity.element.shadowColor;
var color = new pc.Color(oldColor.r, oldColor.g, oldColor.b, newAlpha);
this.entity.element.shadowColor = color;

Creating pc.Color in each update loop - is unnecessary allocation. If abused, will lead to garbage collection stalls. Tip: allocate pc.Color in initialize, and just change it's properties in update.

Thanks Maksims I did this and it worked.
this.shadowColor.copy(this.text.element.shadowColor);
this.shadowColor.a = alpha;
this.text.element.shadowColor = this.shadowColor;

However out of curiosity I attempted this to reduce that extra allocation but it does not seem to work (Maybe something under the hood)
var shadowColor = this.text.element.shadowColor;
shadowColor.a = alpha;
this.text.element.shadowColor = shadowColor;

Bump issue with more requests from the forum: https://forum.playcanvas.com/t/opacity-of-text-text-shadow-color/21109

bump


generator: re-use imageModulesDir constant

At Alpine Linux, Linux Kernel files are also located in /lib instead
of /usr/lib. For this reason, I patch the constants in config.go
accordingly for the Alpine Linux booster package. However, while working
on this I noticed that config.go does not re-use the imageModulesDir
constant. I think it makes sense to re-use it to not hardcode the path
twice.

Thank you for the fix!


[Bug] Export-DbaInstance is using the DAC and causing a Serverity 20 Alert

Environmental information

PowerShell version:
Name                           Value

PSVersion                      5.1.14409.1005
PSEdition                      Desktop
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0...}
BuildVersion                   10.0.14409.1005
CLRVersion                     4.0.30319.42000
WSManStackVersion              3.0
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
dbatools Module version:
Name    : dbatools
Path    : C:\Users\upsearchsl\Documents\WindowsPowerShell\Modules\dbatools\dbatools.psd1
Version : 1.0.112
Name    : dbatools
Path    : C:\Program Files\WindowsPowerShell\Modules\dbatools\dbatools.psd1
Version : 1.0.112

# Replace with output of Get-Culture

SQL Server:

/* REPLACE WITH output of @@VERSION */

Multiple since we use Get-DbaRegServer and pipe to Export-DbaInstance
Versions range from SQL 2008R2 to SQL 2016

/* REPLACE WITH output of @@LANGUAGE */

Report

Host used

[ ] powershell.exe
[X ] ISE
[ ] VS Code
[ ] Other (please specify)

If anything other than powershell.exe was used, please confirm that you can duplicate the issue with powershell.exe

[ ] Still buggy in powershell.exe

Errors Received

No Errors when executing from PowerShell. The command(s) are generating SQL Alerts
Steps to Reproduce

Get-DbaRegServer -SqlInstance blahblah -Group CmsGroupBlah | Export-DbaInstance
Expected Behavior

Run Export-DbaInstance against all instances in the cms. It does but causes Severity 20 Alerts which the customer see's, because it is using the DAC multiple times:
Could not connect because the maximum number of '1' dedicated administrator connections already exists. Before a new connection can be made, the existing dedicated administrator connection must be dropped, either by logging off or ending the process
I would expect the command to never use the DAC. That should only be reserved for the sysadmin that needs it. In my opinion the DAC should never be used by scripts.
Actual Behavior

Everything completes, just generates the Severity 20 Alert (maximum number of '1' dedicated administrator connections already exists)

We have two commands that utilize DAC, and they will continue to do so because the data we need is only accessible via the DAC connection. (Without that data the export of the object is all but useless.)
If you want to prevent the trigger from being fired you can exclude the objects from being exported (or attempted):
Export-DbaInstance .. -Exclude Credentials, LinkedServers

The above two options prevent calling of Export-DbaCredential and Export-DbaLinkedServer, both of which utilize the DAC connection.

Thanks for the information.


Existing MS accounts with gmail address

A lot of guest users have already created a MS account using their Google email address.
When trying to test this scenario myself, I was presented with a dialog saying I "was already logged in as<EMAIL_ADDRESS>would you like to switch to<EMAIL_ADDRESS>Since both addresses are identical, this will create a lot of confusion on the end user and service desks when troubleshooting.
Will this be solved by the end of preview? It might be helpful to add inline icons to differentiate between the MS work/school account and Google account.

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

ID: 6ed01893-a5cd-28bd-b865-e7ebe5870567
Version Independent ID: b3d96be0-1470-eb2c-ea5b-cea0a5cc44a1
Content: Add Google as an identity provider for B2B - Azure Active Directory
Content Source: articles/active-directory/b2b/google-federation.md
Service: active-directory
Sub-service: b2b
GitHub Login: @msmimart
Microsoft Alias: mimart

In addition, when trying to switch, as suggested by the dialog, I got an error:
Inwisseling van uitnodiging is mislukt
Er is een fout opgetreden. Probeer het binnen enkele ogenblikken opnieuw.
(Invite acceptation failed, try again later)
RequestId: 54469cfc-0bd8-48a7-a35b-fdef35e1b73d
Correlation Id:b473f230-625c-4887-ae5f-724904cde8cc
Timestamp:2019-10-30 20:00:26Z

@bartvermeersch Thanks for your feedback! We will investigate and update as appropriate.

hello @bartvermeersch this feature is currently in preview. Please post this feedback against the product feedback forums which can be found here : feedback.azure.com/forums/169401-azure-active-directory and if there's enough community engagement, the product team will determine whether or not to put this on the roadmap.
These github issues are meant for problems with the docs, please submit feedback in regards to the product using the button This Product button not the this page button.

Please let us know if there are anymore questions within the scope of this git issue. If not, I will be closing out this git issue by end of day tomorrow. Please file a new git issue with a reference to this one if you have anymore concerns. Thanks

Ok, thank you @FrankHu-MSFT for the clarification.
I do think it would be useful if the documentation could explain what happens when a guest user uses google federation while his Google e-mail address was already registered in the tenant as a MS account.


Fix missing content type in streams api

Hi,
just a small change but this will improve the lives of devs working with the streams API.
Adding the JSON content type to the given routes enables browsers or Postman to display the server response in a structured format rather as raw text. This is also technically "more correct" 😉

Uhm, that are a lot of errors. I tried testing locally but I get a lot of different errors that are surely not related to this change. Could someone give me a pointer.

Thanks @FabianTe! PR looks good, the fail looks like a race condition somewhere in the tests so I've restarted it but your changes look fine, if the tests fail again I'll take care of it

Alright, thank you for reviewing!
I'll hopefully add this for further API routes. I actually wanted to test some functions while I noticed this. Expect more to come up some time 😉


HARP-5108: Improved PointLabels VertexBuffer allocations.

Modified how TextBufferObjects are computed and stored for point labels to minimize and delay VB allocations as much as possible.

GPU tests please


[Discussion] restic on busybox docker

Hi, I wanted to drop this into the discussion on the actual upstream repo...
I've started fiddling around with a tiny busybox restic container based on @Lobaro 's work.
https://github.com/xoes/restic-backup-docker
As I want to use an external scheduler, I find the following approach interesting:

Keep the crond doing the scheduling of the backups (not the external scheduler)
Put make pid=1 fail when the actual sync fails so that the scheduler is alerted that something's wrong. - I'm actually not sure how to achieve that...

As it is only intended to be a fully automatic sidekick, no fuse filesystem libraries are necesary...
However, what needs to be done, I guess is a more elegant log forwarding.
Actually, I liked the delegated approach from @Labaro very much, as it is made for humans, but keeping the logs just there is not so optimal neither...

The Restic forum at https://forum.restic.net/ may be a better place to have a general discussion about this one. If you are looking for a log enhancement so the scheduler can be alerted if something is not working, I recommend bringing that up in the existing log enhancement feature request (https://github.com/restic/restic/issues/1455).

You're right... :+1:


[ ENHANCEMENT ] - List services on UP

Looking to contribute but new to spacemacs and lisp.
The idea:
Parse docker-compose.yml to list services when spinning one up individually.
Looking for guidance.
<EMAIL_ADDRESS>

Hi @BobbyBakes, thanks for showing an interest in contributing!
Alas, I don't have any bandwidth available to do a proper job of mentoring you through the implementation of that change.
Additionally, the change you propose seems to go beyond the scope of this project, which is limited to improving the reading and editing experience for docker-compose files (e.g. providing better keyword completion or improved syntax highlighting). May I suggest doing so in a fork, or better yet, in a separate package?
Thanks again!

Sounds good! Thanks for the quick response !

Back to this. Apologies.
The enhancement i had in mind, might already be in existence.
scenario: U, U,  (docker compose up {service} will call docker-compose config --services. as if it is attempting to list the services available.  I then am prompted to enter the service.
Was there intention of listing services to mnemonically input the service name? If so, this is what i am looking for, to avoid the typing in of the service every time.

Thanks for the feedback. This exists when the list of services is generated with {docker-compose config --services}. I didn't realize i could auto-complete when bringing up a service. All seems to be good in the world. Thank you again! Hoping to help contribute one day.


Dotenv/.env loading

First want to say thanks for making this - I was working on something similar and you saved me some time :)
When I try to run the main script, with dotenv installed, I get an error:
ImportError: cannot import name 'Dotenv' from 'dotenv' (/usr/local/lib/python3.7/site-packages/dotenv/__init__.py)
I think you meant to have:
from dotenv import load_dotenv
instead of
from dotenv import Dotenv
or perhaps there is a mapping you have?

@pringithub Thanks a bunch! Mind pulling the latest commit from master and letting me know if it resolves your issue?

yea it worked


[OnlineMeeting]SeaTunnel community meeting Topic collection

We will hold an online community meeting on 2022/03/16 20:00-20:30 (GMT+08:00),
Welcome everyone to leave a message in the issue to discuss the content
CalvinKirs invites you to a meeting on Tencent Meeting
Meeting Topic: SeaTunnel Meeting
Meeting Time: 2022/03/29 20:00-20:30 (GMT+08:00) China Standard Time - Beijing
Click the link to join the meeting or to add it to your meeting list:
https://meeting.tencent.com/dm/VziZEbLOkq8e
#TencentMeeting：135-555-998
Copy this invitation and open in Tencent Meeting app (V2.13 or later) to join the meeting
Discussion Issue :

Review of the content of the previous issue https://github.com/apache/incubator-seatunnel/issues/1482

2.1.0 Releasing & next version

#875

#1461

#1552

#1553

Welcome everyone to leave a message in the issue to discuss the content

[Feature][CDC] Support Flink CDC https://github.com/apache/incubator-seatunnel/issues/1461

#1557


Add GELU activation layer

Hi, in this PR I implemented the Gaussian Error Linear Unit (GELU) activation layer.
This layer is often used in Transformers.
The gelu function is defined as:

I choose the sigmoid approximation for this implementation. For the derivative, I used WolframAlpha... 😅

I would like dlib to support Transformer architectures some day, since they are proving to be really good at NLP tasks (they've made RNNs obsolete).
I will certainly give it a go in my spare time, but I can't promise anything or give a deadline. This is my first baby step (I know, I started building the house from the roof).
I think most of the building blocks are already there. I just need to figure out how to do the positional encoding in a nice way (maybe a custom input layer that does that automatically? Anyway, I will try unless somebody has a clear idea on how it can be implemented nicely.

I've noticed this implementation using the simoid is slower and less precise than the one using the tanh approximation.
Also, I've cheated a little bit by looking at how PyTorch implemented this. Here's my testing for a 1x3x224x224 tensor on the gpu:

dlib: gelu with sigmoid approx: 30 µs
dlib: gelu with tanh approx: 26 µs
dlib: gelu with normcdf: 8 µs (based on PyTorch implementation)
pytorch gelu with normcdf: 12 µs (maybe slower because of compilation flags and python overhead)

Sweet, thanks for another cool PR :)


Cloudflare static files

How to serve static files when using cloudflare worker in dev?
I tried to run the react-router-cloudflare and even here I get 404 for the images.

👋 I'm working on it for dev

Anything I can do to help?

I'm late on the planning because of the WebSocket support on Cloudflare adapter. I had to learn how Durable Objects work.
I'm working on it right now for a builtin support, but you should be able to use Remix Hono helper https://github.com/sergiodxa/remix-hono?tab=readme-ov-file#static-assets-on-cloudflare
It has just been released for RR7

Thanks, I'm getting ReferenceError: The binding ASSETS is not set. but I will probably figure it out...

Sorry for this bad experience... it's the total opposite of the goal of this library.
You are right, a ASSETS binding in Hono env is required.
I guess fetch is server.fetch here.
https://github.com/sergiodxa/remix-hono/blob/5beac8d008c220bc0092eb8428fa89fdcd759280/test/cloudflare.test.ts#L30
I will definitely pay attention to this part.

Ok so the issue is not here.
I have to add a conditional middleware in the adapter in dev mode and remove it on build.

Thanks.
Unfortunately I'm having terrible performances with Cloudflare worker, even without hono (https://github.com/remix-run/react-router/discussions/12496).
I wanted to compare with aws lambda, but I have to use remix-hono directly. And doing import * as build from 'build/server' breaks everything...

I never used AWS so maybe I'm wrong.
Does it use NodeJS?
With the current temp package (https://pkg.pr.new/rphlmr/react-router-hono-server@24) I return the Hono app.
If you follow https://hono.dev/docs/getting-started/aws-lambda it could be:
import { createHonoServer } from "react-router-hono-server/node";
import { handle } from 'hono/aws-lambda'

const app = await createHonoServer();
export const handler = handle(app);
export default app;

Could not resolve "virtual:react-router/server-build unfortunately

Oh ok, it means that it requires to put a build output and not a ts file to compile.
I don't know how to serve build/client files with AWS, but if you need the build/server output to be a bundle (one file), you can add this to your Vite config:
  ssr: {
    target: "webworker",
  },

Version 2.2.0 has been released, so https://pkg.pr.new/rphlmr/react-router-hono-server@24 is gone 😬

there is a bug when you configure a prerender route:
import type { Config } from "@react-router/dev/config";

export default {
  // Config options...
  // Server-side render by default, to enable SPA mode set this to `false`
  ssr: true,
  prerender: ["/"], // <- this throw error on build time
} satisfies Config;

Console error

[!NOTE]
In console says plugin: 'react-router' but after remove the reactRouterHonoServer from vite.config.ts and run build command (it means, remove the hono server from build process) the output is correct, a static html is created.

@diecodev is it with Cloudflare?

no, I'm using createHonoServer from "react-router-hono-server/node";

For me I'm getting
[react-router] Cannot convert undefined or null to object
    at Function.values (<anonymous>)
    at groupRoutesByParentId2 (x/node_modules/.pnpm/@react-router+dev@7.0.2_@react-router+serve@7.0.2_react-router@7.0.2_react-dom@19.0.0_react@1_yyc4a7qdc3o3q7j7ib4tun4nl4/node_modules/@react-router/dev/dist/vite.js:2650:10)
    at createPrerenderRoutes (x/node_modules/.pnpm/@react-router+dev@7.0.2_@react-router+serve@7.0.2_react-router@7.0.2_react-dom@19.0.0_react@1_yyc4a7qdc3o3q7j7ib4tun4nl4/node_modules/@react-router/dev/dist/vite.js:2661:76)

also with react-router-hono-server/node, not sure if exactly the same error. Without prerender, it fails during requestHandler runtime.

@cwagner22 same:

I can reproduce this error too. Not sure what happened here, but I'm looking into it.

Okay, I found out why.
React Router prerender tries to load build/server/index.js to get the manifest, but at this location, it's the Hono server.
I'm looking for a fix.

It should be fixed in the next release: https://github.com/rphlmr/react-router-hono-server/pull/37
You can test the preview package with:
npm i https://pkg.pr.new/rphlmr/react-router-hono-server@37

If it's ok, I will release tomorrow!

It has been released as 2.3.0
Feel free to reopen if cloudflare static is still an issue 🫡

Unfortunately, while prerender is working, I'm still having the issue at runtime when my lambda function executes:
  "TypeError: Cannot convert undefined or null to object",
    "    at Function.values (<anonymous>)",
    "    at groupRoutesByParentId2 (file:///var/task/bundle.mjs:13652:14)",
    "    at createRoutes (file:///var/task/bundle.mjs:13664:71)",
    "    at derive (file:///var/task/bundle.mjs:14043:21)",
    "    at requestHandler (file:///var/task/bundle.mjs:14081:25)",
    "    at Runtime.handler (file:///var/task/bundle.mjs:41222:28)",
    "    at Runtime.handleOnceStreaming (file:///var/runtime/index.mjs:1199:38)"

I deployed the with-pre-render example with sst. If you have any tip to debug that... Working without reactRouterHonoServer(), and serverBuildFile is set.

@cwagner22 The error is that it tries to import build/server/index.js, which is the Hono server.
I haven’t tried SST but it’s like the react-router.config.ts is not loaded 🧐

The problem is the sst adapter is using createRequestHandler from react-router, here. I thought I could use another adapter such as sst.aws.Router but I'm having other issues.


pull request for e2e test at 12/05/2016 08:26:02

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


[ question / documentation / bug? ] switching ca from test to prod does not regenerate certs

i would have opened a discussion if they where enabled in this git repo. i am not sure if i simply do something wrong or if i found a bug.
for the development i created the certificates with the ca letsencrypt_test. this worked quite well, just needed some puppet runs and the certificate was correctly there. after that i switched to the ca letsencrypt but the certificates where still from the test ca. i wasn't able to force a regeneration of the cert.
so i tried to delete the complete acme folder from the client but after the puppet run the i got wrong certs again. also tried to delete the folder on server and client which led me to an unusable system (not exactly in this order, i played around a bit) as "suddenly" the private key doesn't match the cert any more.
Jan 10 02:25:09 aio001 nginx[932050]: nginx: [emerg] SSL_CTX_use_PrivateKey("/etc/acme.sh/keys/.../private.key") failed (SSL: error:0B080074:x509 certificate routines:X509_check_private_key:key values mismatch)
Jan 10 02:25:09 aio001 systemd[1]: nginx.service: Control process exited, code=exited, status=1/FAILURE

am i doing something wrong? should this module be able to switch ca's? how can i start-over? how to force regeneration?

Any update on this? I'm stuck on this aswell

Unfortunately, this is currently not supported. In order to recover from this situation, run the following on both the Puppetserver and the affected Node:
https://github.com/markt-de/puppet-acme#rebuilding-nodes
In order to support changing CA for existing certs, it would be necessary to add this information to the name of every file that is used by acme.sh:

We already to this for account configs, so this may be used as a boilerplate: https://github.com/markt-de/puppet-acme/blob/20d2cb3d09e3e8a134bf6cdc7bea0cb86d68b74a/manifests/request.pp#L80-L96).
It needs to be added for all cert configs/files too. The acme.sh command needs additional options. (Example: https://github.com/opnsense/plugins/blob/c6ad8a442bcc50a9e15228dfd9566a7dadf5c5a2/security/acme-client/src/opnsense/mvc/app/library/OPNsense/AcmeClient/LeValidation/Base.php#L96-L100)
And it needs to be implemented in a backwards-compatible way. Existing certs would not be altered, it would only be used for new certificates (similar to how we've done it for account configs).


fix(archive integration): Fetch drives from all ArFS compatible transactions

This PR removes the "App-Name" GQL tag filter which will enable the ArConnect extension to use the archive feature to any ArFS compatible drive, including the ArDrive-CLI and apps using ArDrive-Core.
The ArDrive apps use more App-Names than just "ArDrive-Web" and "ArDrive-Desktop". We currently also use "ArDrive-CLI", "ArDrive-Core", and "ArDrive-Sync". But adding a query with this many tag values has proven to be intensive on the gateway.
The ArFS spec is not really limited by App-Name. All MetaData Tx can be queried by ArFS with value 0.10 or 0.11. But removing the App-Name tag filter altogether works just as well here and is more future proof

Thanks for the pr! Wouldn't it make sense to use ArConnect, as the value of the "App-Name" tag?

This query is being used to fetch the public ArFS drives of a given wallet address. Using Arconnect tag filter with drive and public would probably net 0 results. There are no tags being added here
If that question was meant to be directed towards when we write new ArFS transactions, e.g archive content to ArDrive: I would say it's fine to use Arconnect as the App-Name there. But you're already using ArDrive-Client: Arconnect which is a much more performance query. App-Name is a particularly congested name at the gateway 🤓

Any update here? Do you need anything else from me?

Sorry it should be perfect


[fix][flaky-test]split test group broker_group_2 into 2 parts

Fix: https://github.com/apache/pulsar/issues/17112
Motivation
Split test group broker group 2 into 2 parts to prevent run unit tests timeout.
Documentation
Check the box below or label this PR directly.
Need to update docs?

[ ] doc-required
(Your PR needs to update docs and you will update later)

[x] doc-not-needed
(Please explain why)

[ ] doc
(Your PR contains doc changes)

[ ] doc-complete
(Docs have been already added)

I think this will not resolve the problem.
Maybe some tests blocked forever
https://github.com/apache/pulsar/runs/8065329618?check_suite_focus=true
The group 2 usually gets passed in 30min

I think this will not resolve the problem. Maybe some tests blocked forever
https://github.com/apache/pulsar/runs/8065329618?check_suite_focus=true The group 2 usually gets passed in 30min

OK，I'll check the log and find the root cause

I think we should look for tests that take too long, and then try to optimize them.


regeneratorRuntime is not defined (with Babel-transpiled code that uses generators)

linky
I don't know if Flems is supposed to work with IE11, but if it isn't you could just target ES6 with Babel...

I use Flems for IE11 tests sometimes, but not with Babel transpile enabled.
Would setting ES6 work for Babel fix this?
In that case It's probably a better tradeoff than not having generators work in modern browsers when Babel is active.

A third possibility would be to load said runtime, but I don't know how heavy it is, or if it can be done lazily.
I suppose that targeting ES6 would solve it, since it supports generators.


Auto focusing is not working on screen after react-native-modal

Environment

  System:
    OS: macOS 10.15.7
    CPU: (4) x64 Intel(R) Core(TM) i5-9300H CPU @ 2.40GHz
    Memory: 55.09 MB / 5.00 GB
    Shell: 5.7.1 - /bin/zsh
  Binaries:
    Node: 15.2.0 - /usr/local/bin/node
    Yarn: 1.22.10 - /usr/local/bin/yarn
    npm: 7.0.10 - /usr/local/bin/npm
    Watchman: 4.9.0 - /usr/local/bin/watchman
  Managers:
    CocoaPods: 1.10.1 - /usr/local/bin/pod
  SDKs:
    iOS SDK:
      Platforms: iOS 14.2, DriverKit 20.0, macOS 11.0, tvOS 14.2, watchOS 7.1
    Android SDK:
      API Levels: 29, 30
      Build Tools: 28.0.3, 29.0.2, 30.0.2
      System Images: android-29 | Google APIs Intel x86 Atom, android-30 | Google APIs Intel x86 Atom
      Android NDK: Not Found
  IDEs:
    Android Studio: 4.1 AI-201.87<IP_ADDRESS>53283
    Xcode: 12.2/12B45b - /usr/bin/xcodebuild
  Languages:
    Java: 1.8.0_275 - /usr/bin/javac
    Python: 2.7.16 - /usr/bin/python
  npmPackages:
    @react-native-community/cli: Not Found
    react: 16.13.1 => 16.13.1
    react-native: 0.63.4 => 0.63.4
    react-native-macos: Not Found
  npmGlobalPackages:
  `  `*react-native*: Not Found

Platforms

issue found on iOS
I haven't tested on Android yet
Versions

iOS: 14.2
react-native-modal: 11.6.1
react-native: 0.63.4
react: 16.13.1

Description

In my app, there is an AddingItemModal which is based on react-native-modal, once user choose any type of item on this modal it will be directed to the AddItemScreen which has some input fields for detailed information depends on that type of item.
The problem is that after AddItemScreen is pushed, auto focusing on the first input field is not working. The auto focusing is working well if I only change the modal to default Modal component from react-native.

I have also encounterred this issue, but I only reproduced it in android..IOS is ok for me

Hi @rener172846.
Do you navigate to the screen first and then set the modal's visibility to false or do you set the visibility to false and then use useEffect to navigate to AddItemScreen?
what navigation system are you using? React Navigation or React Native Navigation?

did a workaround by using setTimeout to navigate the user, Working on both platforms

You could use onModalHide property to ensure the modal is hidden before you navigate. Closing the for now as it has been inactive for some time.


Allow transformRequest to modify headers

In AngularJS 1.4.0-beta5 the possibility to modify request headers from transformRequest functions has been removed. This is stated in the changelog. I found this a very useful feature to modify the Content-Type of the request being made.
For example:
$http.post('', {
  foo: 'bar'
}, {
  transformRequest: function(data, headers) {
    headers()['Content-Type'] = 'application/x-www-form-urlencoded';
    var parsed = [];
    angular.forEach(data, function(value, key) {
      parsed.push(encodeURIComponent(key) + '=' + encodeURIComponent(value));
    });
    return parsed.join('&');
  }
});

I often use a similar approach to make form-data requests, which requires the Content-Type to be set to undefined.
In practice I have created an xWwwFormUrlencoded and formData factory, because I use them more often.
So the code would look like this:
$http.post('', {
  foo: 'bar'
}, {
  transformRequest: xWwwFormUrlencoded
});

It feels like a huge drawback having to to explicitly specify the headers whenever making such a transformed request.
The following plunkr demonstrates this issue. The test breaks when updating AngularJS to 1.4.0.

Hmm. I can see how this in inconvenient. There are basically two ways

setting headers explicitly
using an interceptor (but you have to filter the requests to which they should apply)

However, if you have a factory for the transformRequest, couldn't you simply do:
$http.post('', {
  foo: 'bar'
}, {
  headers: xWwwFormUrlencoded()
});

Yeh, I can see how this is inconvenient and while interceptors work they add overhead of the async processing. having said this I really dislike APIs where input arguments are getting modified....
Maybe we should make it possible to return both transformed data and headers from the transform request function?

The headers property could also accept a function that gets passed the current config + default headers and is expected to return a headers object.

Actually using an interceptor sounds like a perfect solution in my case. I could just filter out requests which specify an x-www-form-urlencoded content type and modify the body accordingly.
If the function would be able to return both the transformed data and headers, I imagine this would return an array consisting of [data, headers]. This seems like a bad idea to me, because developers may want to return an array as data.
If the headers would accept a function for transforming the headers, this means that a modifications to body according to the content type would happen in the headers function as well.
Another option could be that transformRequest accepts either a function or an object. If a function is passed, it behaves the way it normally does.
If an object is passed, it may define a function for transforming the body and a function for transforming headers.
$http.post('', {
  foo: 'bar'
}, {
  transformRequest: {
    body: function(data, headersGetter) {
      return myTransformFn(data);
    },
    headers: function(data, headersGetter) {
      var headers = headersGetter();
      headers['Content-Type'] = 'x-www-form-urlencoded';
      return headers;
    }
  }
});

I imagine this would return an array consisting of [data, headers]. This seems like a bad idea to me, because developers may want to return an array as data.

There are many ways to solve it cleanly, ex.: returning an instance of a specific class. So I don't think we are blocked from applying this solution, implementation-wise.

I was using this feature too...
It's really important for me the ability to change headers in the transform request, instead of in every single function... please bring this feature back if possible...

Agreed, we now have to find a work around while updating.

I also need this feature, or is there another way to modify headers or url.
We want add a signature for every request to the server that calculate a checksum over the url and the content send.
Is there a work around, or a different way?

+1 :+1:
I somehow have to set "X-HTTP-Method-Override", which I can't do at the moment.

Any workaround on this issue?

@wmfairuz yes, interceptors

Since there is a "work around" for now and considering ngResource interceptors/transforms will probably/hopefully change soon (as discussed in #9334, #11409, #13273), I'm going to close this.
Specifically, this should be fixed by #13273. Let's track it there.


Generate forecasting format from queries

{
			"ts": [ (An array of arrays. Each sub-array is the time and the fields defined in the "dimensions" array.)
				["2017-12-21T00:40:36Z", 3],
				["2017-12-22T00:40:36Z", 0],
				...
			]
			"metadata": { (some relevant metadata about the time series.)
				"name": ...,
				"aggregation_type": ...,
				...
			}
			"dimensions": [ (A list of the dimensions in the time series. The first dimension will be a date.)
				"DATE",
				{
					"type": "int", (the datatype {float, int, string})
					"semantic_type": "deaths" (what the value represents)
				},
				...
			]
		}

This is done.


Add build step to play better with catapult deploy.

@jwngr Does this seem reasonable?  I think I may end up with both src/firebase-import.js and bin/firebase-import.js in the deployed node module now, but that won't hurt anything I guess.

mv bin/firebase-import.js to src/firebase-import.js
Add "npm run build" step that copies from src/ to bin/.

I just pushed a commit which adds a LICENSE file (so that it is included in the npm module itself; most of our other repos do this too) and which adds "files" to the package.json to avoid the problem you mentioned about the npm package including both a src/ and bin/ directory.
One question for you: can we update our firebase dependency to 2.x.x? It is currently quite old (~1.0.15).
Sending back to you for merging.

Awesome, thanks for going above and beyond as usual @jwngr! :-)


Added snapshot settings for previews

It was difficult, but finally I managed to add delay and precision for snapshots.
I have added a function:
struct TestView_Previews: PrefireProvider {
    static var previews: some View {
        TestView()
            .snapshot(delay: 0.1, precision: 1)
    }
}

This function affects snapshot generation, transit delay, and precision.:
// Generated snapshot test
assertSnapshot(
    matching: view,
    as: .wait(for: delay, on: .image(precision: precision ?? 1, layout: isScreen ? .device(config: device) : .sizeThatFits)),
    named: preview.displayName,
    testName: testName
)

I made this improvement based on the idea from this issue: https://github.com/BarredEwe/Prefire/issues/11


Document DeleteBehavior.ClientNoAction

I build and Application that uses Soft-Delete patterns. As others, I have implemented Soft-Delete by overriding SaveChangesAsync() on DbContext. I find deleted EntityEntries and set the Deleted timestamp and set the EntityState to Modified. Unfortunately when the EntityState was set to Deleted earlier, all Links from related Objects to the Entity will get set to null.
I think the Soft-Delte use case where we do not want to sever links to soft-deleted objects is a very common and valid one. Configuring the link with DeleteBehavior.NoAction did not solve the issue, as contrary to it's name DeleteBehavior.NoAction is actually the same as DeleteBehavior.SetNull (according to the documentation here).
To configure the DeleteBehavior to actually do nothing, the problem was solved with DeleteBehavior.ClientNoAction. This actually solved my problem, but what it does is not documented and the name is misleading.

At least update the documentation of DeleteBehavior.ClientNoAction to reflect what it actually does, as it solved my problem but I do not know which other side effects it might have.

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

ID: 2df9f508-7992-183c-a909-5b27316e3efd
Version Independent ID: 8e25c165-0abc-9990-ef58-f0ac29c130e3
Content: DeleteBehavior Enum (Microsoft.EntityFrameworkCore)
Content Source: dotnet/xml/Microsoft.EntityFrameworkCore/DeleteBehavior.xml
Product: ef-core-api
GitHub Login: @dotnet-bot
Microsoft Alias: avickers

Duplicate of https://github.com/dotnet/efcore/issues/29261


conformance: Add Airlock Microgateway report for v1.1.0

What type of PR is this?
/kind documentation
/area conformance
What this PR does / why we need it:
Adds conformance report and updates implementation page for Airlock Microgateway v1.1.0
Which issue(s) this PR fixes:

N/A
Does this PR introduce a user-facing change?:

NONE

/ok-to-test

I excluded 2 links in the README.md from the check by theverify-reports script, because they were incorrectly declared as dead links. Those links are valid but protected from bots/crawlers by our firewall.
The excluded links are:

https://www.airlock.com/en/secure-access-hub/components/microgateway/community-edition
https://www.airlock.com/en/secure-access-hub/components/microgateway/premium-edition


fix: install rayon pool in Session::enter

Install the rayon pool together with session globals in Session::enter instead of in CLI only; this enables library users to use sema and any other rayon usage
cc @ferranbt https://t.me/paradigm_solar/100

I can confirm it works.


[Documentation] Group utilities by category

At brazilian-utils site, All utilities are listed together without grouping by utility group/category, e.g CPF utilities, Boletos utilities and so on. Grouping the categories can make the navigation easier.
As you can see bellow, this is the current sidebar:

One suggestion is just to add a group title for each category in the docs, as this example from date-fns docs:

What do you think @LucasCarrias of something like this?

We can enjoy that we going to change the docs and change sections names for something like that:
CPF -> Validate
CPF -> Generate
CPF -> Format

@hyanmandian That looks pretty good! I really liked the grouping.
However, I'm not quite sure if changing the function name by its action name is a common pattern in documentations. At least, thinking about docs like date-fns and lodash.
That is just an impression that I'm bringing into discussion. Besides that, the docs seems to be cleaner and easier to navigate.


fix: update calorie counter project styling

Checklist:

[x] I have read and followed the contribution guidelines.
[x] I have read and followed the how to open a pull request guide.
[x] My pull request targets the main branch of freeCodeCamp.
[x] I have tested these changes either locally on my machine, or GitPod.

This is the update to the styling of the calorie counter project.

Is this a work in progress, or is this ready?

@raisedadead, this should be ready for review now.


JSON character escaping issue

I have run across a few generated report.json files that have invalid characters as reported by JSON::XS and jq.
Specifically, I have a log line that contains:
....  "GET /JMA-December-2017.png HTTP/1.1\rf HTTP/1.1" ...

Obviously this is a programmatic fetcher that is a bit brain damaged on its protocol field output.  With http servers, they generally follow Postel's law, so ilk like this can find its way into the logs.
It is stored in the report.json file as:
{ "protocol": "HTTP/1.1\RF HTTP/1.1" }

I haven't fully traced through the code, but I surmise that parsed protocol data is forced to uppercase, and then when it lands in escape_json_output() it isn't handled properly as it is no longer \r.  Looking a bit further, I do see:
parser.c
(*protocol) = strtoupper (xstrdup (++proto));

Which leads to a question of, why wasn't it at least escaped to?:
{ "protocol": "HTTP/1.1\\RF HTTP/1.1" }

How would you recommend handling this particular situation?

After more tinkering it doesn't seem that the protocol data is even being sent through JSON escaping.
escape_json_output()
...
case 'T':
  pjson (json, "Z");
  break;
...

The 'T's in HTTP are left untouched.

@allinurl I've committed a POC fix on the v1.2 tree, and is just a wedge focused on handling the JSON protocol field output.  IMHO, all data output to the report.json file should be scrubbed for any rogue backslashes that violate JSON spec.
https://github.com/TerraTech/goaccess/commit/5396f4bde5f9f4d39bb371a256ceee41c4d39941
As an aside, in utils.h, the replace_str() function is defined, however the function does not exist in utils.c or anywhere.  I ended up rolling my own backslash escaper.

Sorry for the delay on my response and thanks a lot for reporting this and for the POC fix. I was looking at it and I feel this is more of an issue with the actual parser than the JSON module. In my opinion the output should read as:
{
    "method": "GET",
    "protocol": "HTTP/1.1",
    "data": "/JMA-December-2017.png HTTP/1.1\rf"
 }

So I just pushed a fix that should address this in the parser side, feel free to rebase from master to test it out and let me know what you think.

Ah, eureka - it was a matter of blurred lines on where to do the parsing splits.

Unfortunately, the fix does not cover other ilk, like the following:
{ "protocol": "HTTP/2\XEE\X19\XFA\X80\X19\X97V\X06KI\XB8WZ\XBEJ\X1E\X83KFADTL\XE2\XA7T+~!WH\XA8\XECLJN%#D\XF9\XEFN: KEEP-ALIVE" }

At this point, should we:

backslash escape the field regardless
punt the whole bad request (Apache logs its as status code 400)

I don't necessarily like dropping it on the floor, since it is contained within the log file and it may be important to the client to see these kinds of log entries.  Perhaps goaccess could be extended to track the 'bad requests' and display them in their own panel.

Can I ask you to please post the whole request line? Like you said, dropping it might not be the ideal scenario... I wonder how is being displayed in the terminal output... Thanks.

<IP_ADDRESS> - - [19/Oct/2017:18:19:49 -0400] "GET /images/Fire%20Incident%20in%20West%20300x450.jpg HTTP/2\xee\x19\xfa\x80\x19\x97v\x06Ki\xb8wz\xbej\x1e\x83kfadtl\xe2\xa7T+~!wh\xa8\xecLjn%#d\xf9\xefn: keep-alive" 400 369 "-" "-"


Add support to missing SQL Server data types available on current Ibis implementation

Check on this link https://github.com/ibis-project/ibis/blob/master/ibis/backends/mssql/datatypes.py#L47 which ones to add

These data types are now supported after the Ibis upgrade (https://github.com/GoogleCloudPlatform/professional-services-data-validator/pull/894)!


Pylance Not effective

I chose Conda's interpreter(Linux machines) on VSCode, but the Pylance still hasn't taken effect

Thanks for the report. Unfortunately, this is not enough information for us to investigate this issue.
If this continues to be a problem, would you please reopen the issue and add the information described in the troubleshooting guide?


Add explicit group node summariser instead of doing it in the other summaries

Fixes #1213

LGTM with some minor comments.


Support monorepos with independent versions

This changes functionality to support MetaMask monorepos with an independent versioning strategy.
closes: #45

I've been testing this out in three cases:

Monorepo with fixed versions, using snaps-monorepo
Monorepo with independent versions, using controllers (monorepo-poc branch). Note that we are no longer using action-create-release-pr to create release PRs but create-release-branch instead.
Polyrepo package

I encountered a problem when attempting to release snaps-monorepo: https://github.com/mcmire/snaps-monorepo/actions/runs/3252758730/jobs/5339319338. I made fixes in 2cc1e0a...7e70bcd. Here is the successful release: https://github.com/mcmire/snaps-monorepo/actions/runs/3253016948/jobs/5339866555
I also encountered a problem with controllers: https://github.com/mcmire/controllers/actions/runs/3253182761/jobs/5340203034. I made fixes in 7e70bcd...021618a. That worked here: https://github.com/mcmire/controllers/actions/runs/3253390324/jobs/5340619148. However, I realized that controllers was not using the latest version of npm-publish (which adds support for independent monorepos). Upon attempting to run create-release-branch again, however, I ran into this bug: https://github.com/MetaMask/create-release-branch/issues/43. We need to fix this before we can continue testing this action.
I haven't gotten around to testing the polyrepo package case yet.

I also tested this on a version of snaps-monorepo where I added release.config.json with an "independent" .versioningStrategy and that appears to have run correctly:

It correctly ran the newly added get-release-packages.sh here
It correctly determined the RELEASE_STRATEGY as independent here
It correctly got a complete list of the packages to be published (in this case all since I did this with Create Release Pull Request) here
It correctly created a single version tag and individual tags for each package here


PCL config for ROS Kinetic

I have installed iai_kinect2 on ROS Kinetic (latest version). I compiled and worked fine with just small PCL configuration. Since in Kinetic cv_bridge depends on OpenCV 3 there is no issue with compiling.

Hi,
I tested it today on 16.04 and the only problem I found was a warning relating Eigen, which I fixed in the devel branch. No other changes were necessary. Could you check if it is working for you too?


CCU REST API Deprecation

Hello,
I just wanted to give a heads up that the CCU REST API is deprecated and will be at EOL fairly soon. Please consider making a new version that uses the new OPEN Authentication and CCU interface. Note that very soon there will be a version which purges in 5 seconds.
https://community.akamai.com/community/developer/blog/2015/08/20/getting-started-with-the-v2-open-ccu-api
Thanks!
Kirsten

Looks like v2 is getting deprecated now in favour of v3 and v1 and v2 APIs go EOL on June 30th, 2019!
https://developer.akamai.com/blog/2018/11/16/akamai-content-control-utility-ccu-api-end-life-notice


[ng] Remove OK/Apply button from column hide/show panel in datagrid

Signed-off-by: Jeeyun Lim<EMAIL_ADDRESS>

This is intended as a non-breaking change right? If so, have we tested in a demo with the clrType="ok" still enabled to see we don't get any errors (and on --prod builds)?

@gnomeontherun actually I think this would be considered a breaking change (both visually and also in that it won't support clrType="ok" going forward. Is this not what we want?


Upgrade to Jetty 11 maven plugin

Also some attempts to reduce number of warnings about classes loaded from multiple locations, and some import cleanups. This change also makes it possible to run mvn jetty:run from the top-level directory and not only in the app module.

@snoopdave did you mean Jetty 10? Jetty 11 runs Jakarta EE 9 and most dependencies are not ready for that yet.

@snoopdave did you mean Jetty 10? Jetty 11 runs Jakarta EE 9 and most dependencies are not ready for that yet.

@snoopdave did you mean Jetty 10? Jetty 11 runs Jakarta EE 9 and most dependencies are not ready for that yet.

Yes. Thanks! I edited the title to fix that.

@snoopdave did you mean Jetty 10? Jetty 11 runs Jakarta EE 9 and most dependencies are not ready for that yet.

Yes. Thanks! I edited the title to fix that.


Enable vercel deployment rewrites

Works just fine https://lz-monitoring-frontend-git-feat-enable-vercel-rewrites-l2beat.vercel.app/status


Compiling issues

Hey guys,
I have the following issue. When compiling with:
make exe -f makefile11
I get the following error after it compiles lots of other things:
/usr/bin/ld: /home/mbelouso/MotionCor3/Lib/libmrcfile.a(CLoadExtHeader.o): relocation R_X86_64_32 against symbol '__gxx_personality_v0@@CXXABI_1.3' can not be used when making a PIE object; recompile with -fPIE /usr/bin/ld: failed to set dynamic section sizes: bad value collect2: error: ld returned 1 exit status make: *** [makefile11:160: exe] Error 1
gcc and g++ version is 11.4, CUDA 11.8, linux mint 21
Any thoughts?

I had the same issue but adding -Xcompiler -no-pie to makefile11 in the exe block worked for me (copied from Issue #4)
See https://github.com/czimaginginstitute/MotionCor3/issues/4#issuecomment-1784848759 for the full code snippet

@elainehoml Thanks for the heads up. I added these lines to the makefile11, but still run into the same issue. Same error output as in my initial post.

Adding these flags in makefile11 seems to work. A different workaround that worked for me as well was to use GCC 10 instead of GCC 11.

Yeah. I eventually got it to compile with a different version of GCC (8.0)
and adding these flags. Thanks.
On Wed, 14 Feb 2024, 10:13 pm Stefan Fleischmann, @.***>
wrote:

Adding these flags in makefile11 seems to work. A different workaround
that worked for me as well was to use GCC 10 instead of GCC 11.
—
Reply to this email directly, view it on GitHub
https://github.com/czimaginginstitute/MotionCor3/issues/11#issuecomment-1943556513,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AGEXN7BZNKGSSKSSIRTZSHTYTSL4VAVCNFSM6AAAAAA7KC2IFCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNBTGU2TMNJRGM
.
You are receiving this because you authored the thread.Message ID:
@.***>

Just to provide more context here, at some point several popular Linux distributions decided to compile their packaged versions of gcc with the -enable-default-pie option (see here).  The version of gcc used isn't as relevant so much as whether or not gcc was compiled with PIE as default.  The list of compile options for the particular gcc binary you're using can be found by typing gcc -v.
A PIE (position independent executable) is a binary compiled with security measure that is meant to prevent some common buffer overflow attacks; I'm not certain, but I imagine it has become the default due to several high-profile buffer overflow exploits in somewhat recent years (e.g., Heartbleed).
I'm uncertain why PIE binaries can't be produced with NVCC, but some frightfully old CUDA docs provide a clue:
The host linker on Mac OS 10.7 generates position-independent executables by default. As CUDA does not support position-independent executable currently, the linker must generate position-dependent executable by passing in the -no_pie option. If nvcc is being used to link the application, this option will be passed to the linker by default. To override the default behavior, the -Xlinker -pie option can be passed to nvcc.

My educated guess is that modern versions of CUDA still do not support PIE binaries, although I can find no documentation out there to confirm or deny this hypothesis.


Migrate ARR Cycles to Independent API2 Requests

This PR integrates the updated ARR workflow into openreview-py and migrates the venue to API2. The following steps are currently handled by scripts outside of openreview-py:

[ ] Copy all groups from the previous cycle (reviewers/ACs/SACs/ethics reviewers/ethics chairs)
[ ] Setup registration and unavailability forms (reviewers/ACs/SACs)
[ ] Copy registration forms and expertise edges from previous cycles
[ ] Create custom max papers invitations (reviewers/ACs/SACs)
[ ] Update unavailability process functions (creates or replaces a custom max papers edge with the value in the unavailability form)
[ ] Update submission pre-process function (validates that the previous_URL field is a link to OpenReview only containing /forum?id=)
[ ] Post a subset of the post-submission hidden fields as a supplementary materials note on behalf of the PCs to each submission. This note always contains at least: Note From EiCs: These are the confidential supplementary materials of the submission. If you see no entries in this comment, this means there haven't been submitted any.
[ ] Reveal anonymous pre-print submissions to the public
[ ] Create new paper comment invitations between authors/ACs/SACs/PCs (Author-Editors Confidential Comment)
[ ] Share proposed reviewer assignments to the ACs/SACs in their consoles, and update them to the deployed reviewer assignments after assignments have been deployed
[ ] Update Official_Review process function to create a Consent invitation for the reviewer to reply to their own review to consent for this review to be used for research. Also, emails the ethics chairs for flagged papers and re-runs the ethics stage. If there is a potential anonymity violation, email the editors and create a paper desk reject verification invitation.
[ ] Setup reviewer license and recognition forms (reviewer registration forms)
[ ] Setup reviewer/AC checklists (recycled Official_Review/Meta_Review invitations with different content and no email notifications) Emails the ethics chairs for flagged papers, re-runs the ethics stage and reveals all checklists to the ethics reviewers/chairs. If there is a potential anonymity violation, email the editors and create a paper desk reject verification invitation.
[ ] Enable SACs to edit AC/Reviewer assignments (including inviting assignments for emergency ACs/Reviewers)
[ ] Setup ethics flagging and desk reject verification (uses decision stage to create desk reject verification super, allows the conference to reply to the ethics stage by adding as invitees, readers/signatures of the replies, updates the process function so that it stores the Official_Review invitation and re-posts it after the ethics stage is run to keep the process function)

This is great, a good starting point.
I guess we need to update the test to use ARR as a venueid. About the scripts listed here, do you a have a link to the other repo so we can use them as reference?
We can start putting these scripts in a ARR folder and call them from the test and then test the workflow. What do you think?


Update compactor backlog doc for checking halt

[ ] I added CHANGELOG entry for this change.
[x] Change is not relevant to the end user.

Changes
Update https://thanos.io/tip/operating/compactor-backlog.md/ doc to mention checking for compactor halt first.
It should be the first thing to check when troubleshooting compactor issues. If compactor already halt, following steps don't make much sense.
This should help with case like https://github.com/thanos-io/thanos/issues/6866 at first place.
Verification

Merging now, the test is flaky


Rollback for save_files

Rollback for save_files method. If there is an exception commiting the changes of the model to the db, then the user of this library should be able to undo the changes made to files on the server.

The rollback method may also be able to handle rollback when using the update_files method also. Then name accordingly.


Missing -pos file

I am on version sha-f281fec of litefs.  I can not find the pos file next to my db file.  Any chance it has moved?  I can't cat it or see it using ls -l

These are the files in the same directory as my db: dbs id lost+found reel_prod.db reel_prod.db-shm reel_prod.db-wal

Hey @smashedtoatoms, looks like you have your litefs setup mixed because dbs and id are present on LiteFS's internal data dir:
https://github.com/superfly/litefs/blob/f281fecf3573a38785fbd1635ec82d05b1722e45/cmd/litefs/etc/litefs.yml#L25-L27
So, your app shouldn't be opening a sqlite db inside data.dir, instead it should open the db under the path declared by fuse.dir directive:
https://github.com/superfly/litefs/blob/f281fecf3573a38785fbd1635ec82d05b1722e45/cmd/litefs/etc/litefs.yml#L4-L7


Scrollbar thumb behaviour is erratic when scrolling through items of uneven heights

Steps to Reproduce

Run flutter create bug.
Update the files as follows:

import 'package:flutter/material.dart';

void main() => runApp(MyApp());

class MyApp extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Demo',
      home: Scaffold(
        body: Scrollbar(
          child: ListView(
            physics: AlwaysScrollableScrollPhysics(),
            children: ListTile.divideTiles(
              context: context,
              tiles: [
                for (var i = 0; i < 51; i += 1)
                  ListTile(
                    title: Text("foobar\n" * ((i * 11) % 51)),
                  ),
              ],
            ).toList(),
          ),
        ),
      ),
    );
  }
}

Swipe to scroll through the list

Expected results: Smooth movement of the scrollbar thumb
Actual results: Scrollbar thumb jumps forwards and backwards erratically

Logs

[✓] Flutter (Channel stable, v1.12.13+hotfix.8, on Mac OS X 10.15.2 19C57, locale en-GB)
    • Flutter version 1.12.13+hotfix.8 at /Users/matt/lib/flutter
    • Framework revision 0b8abb4724 (2 weeks ago), 2020-02-11 11:44:36 -0800
    • Engine revision e1e6ced81d
    • Dart version 2.7.0

[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.2)
    • Android SDK at /Users/matt/Library/Android/sdk
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-29, build-tools 29.0.2
    • ANDROID_HOME = /Users/matt/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b49-5587405)
    • All Android licenses accepted.

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

[✓] Android Studio (version 3.5)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 42.1.1
    • Dart plugin version 191.8593
    • Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b49-5587405)

[!] IntelliJ IDEA Ultimate Edition (version 2019.3.1)
    • IntelliJ at /Applications/IntelliJ IDEA.app
    ✗ Flutter plugin not installed; this adds Flutter specific functionality.
    ✗ Dart plugin not installed; this adds Dart specific functionality.
    • For information about installing plugins, see
      https://flutter.dev/intellij-setup/#installing-the-plugins

[!] VS Code (version 1.41.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    ✗ Flutter extension not installed; install from
      https://marketplace.visualstudio.com/items?itemName=Dart-Code.flutter

[✓] Connected device (1 available)
    • Android SDK built for x86 • emulator-5554 • android-x86 • Android 10 (API 29) (emulator)

I’m suffering the same issue, anybody give a suggestion?

duplicate of https://github.com/flutter/flutter/issues/59404

flutter doctor -v
[✓] Flutter (Channel dev, 1.20.0-3.0.pre, on Mac OS X 10.15.5 19F101, locale en-GB)
    • Flutter version 1.20.0-3.0.pre at /Users/taha/Code/flutter_dev
    • Framework revision 0af027f805 (4 days ago), 2020-07-04 12:19:20 -0700
    • Engine revision a751393804
    • Dart version 2.9.0 (build 2.9.0-20.0.dev 22da8934ac)

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.0)
    • Android SDK at /Users/taha/Code/sdk
    • Platform android-30, build-tools 30.0.0
    • ANDROID_HOME = /Users/taha/Code/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6222593)
    • All Android licenses accepted.

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

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 4.0)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 47.1.2
    • Dart plugin version 193.7361
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6222593)

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

[✓] Connected device (5 available)
    • SM M305F (mobile)                  • 32003c30dc19668f          • android-arm64  • Android 10 (API 29)
    • Android SDK built for x86 (mobile) • emulator-5554             • android-x86    • Android 6.0 (API 23)
      (emulator)
    • Taha’s iPhone (mobile)             • 00008020-001059882212002E • ios            • iOS 13.5.1
    • Web Server (web)                   • web-server                • web-javascript • Flutter Tools
    • Chrome (web)                       • chrome                    • web-javascript • Google Chrome 83.0.4103.116

• No issues found!

@TahaTesser Can I request we reopen this, as I think it's a different issue: this is about the scroll thumb jumping backwards and forwards erratically. #59404 is about animation not being fluid, which I think is a slightly different issue (i.e. that issue is more about performance than correctness).

Hi @mattrussell-sonocent
It's the same behaviour, the mentioned issue has same scrollbar issue jumping backwards and forward  and cause  of the issue is same
This issues code sample
import 'package:flutter/material.dart';

void main() => runApp(MyApp());

class MyApp extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Demo',
      home: Scaffold(
        body: Scrollbar(
          child: ListView(
            physics: AlwaysScrollableScrollPhysics(),
            children: ListTile.divideTiles(
              context: context,
              tiles: [
                for (var i = 0; i < 51; i += 1)
                  ListTile(
                    title: Text("foobar\n" * ((i * 11) % 51)),
                  ),
              ],
            ).toList(),
          ),
        ),
      ),
    );
  }
}

Mentioned issue's code sample
import 'package:flutter/foundation.dart';
import 'package:flutter/material.dart';

void main() {
  runApp(MyApp(
    items: List<String>.generate(100, (i) => "Item $i"),
  ));
}

class MyApp extends StatelessWidget {
  final List<String> items;

  MyApp({Key key, @required this.items}) : super(key: key);

  @override
  Widget build(BuildContext context) {
    final title = 'Long List';

    return MaterialApp(
      title: title,
      home: Scaffold(
        appBar: AppBar(
          title: Text(title),
        ),
        body: Scrollbar(
          child: ListView.builder(
            itemCount: items.length,
            itemBuilder: (context, index) {
              return ListTile(
                title: Text('${items[index]}'),
              );
            },
          ),
        ),
      ),
    );
  }
}

leaving this open for additional triage

This is by design. The scrollbar logic guesses at the position based on what it knows currently.
You can improve the UI if you can provide a better implementation of estimateMaxScrollOffset.

This appears to be a duplicate of https://github.com/flutter/flutter/issues/25652. Please track that issue for updates. If you believe I've made an error, please reply here with more details that explain the difference and we'll re-open this issue. :) Thanks!


Color districts in sets of maps consistently

Given a set of canonicalized maps, figure out colors for districts so each of the maps is legible.
(Initially, we'll just color each map separately -- #49 -- so while each map will be legible, the color assignments for districts might differ map to map in the set.)

When we color one map, we find the district-district adjacencies in that map and color accordingly.  If we want to color a bunch of maps consistently, then we simply need to find the union of all the adjacencies in the collection, and then color based on that set of adjacencies.


Create a function that returns a license badge based on which license is passed in

figured this out., need to pass shields.io install in and it works.