onResponderRelease not working on Android

Description
onResponderRelease is not triggered in a ScrollView on Android, it works fine on IOS.
Reproduction
Just bind something on the 'onResponderRelease', it will work on IOS, not on Android
Exemple here : https://rnplay.org/apps/2rkKsA
Additional Information

Tested on React Native version: 0.39
Platform: ok on IOS, fails on Android

for information, this issue reopens this one : https://github.com/facebook/react-native/issues/8177

https://github.com/facebook/react-native/issues/9447

Same problem here. I'm getting the onResponderGrant event just fine, but the onPanResponderRelease doesn't fire (at least not consistently; if I do fast jittery movements the event sometimes fires).

if you use onShouldBlockNativeResponder={() => false} the release will be triggered

1 same here

Has anyone found a work around yet?

Should intercept the touch event before release can be intercepted.
Try this:
onStartShouldSetResponder={() => true}
onResponderRelease={() => this._onRowTap(prop)}

+1
@scotbond tried setting onStartShouldSetResponder, but didn't fix it.

same +1. I ended up using the following to keep track

onScrollEndDrag={this.handleStartDragging}
onScrollBeginDrag={this.handleEndDragging}

Hi there! This issue is being closed because it has been inactive for a while.
But don't worry, it will live on with Canny! Check out its new home: https://react-native.canny.io/feature-requests/p/onresponderrelease-not-working-on-android

same with<EMAIL_ADDRESS>it seems that when swiping, onResponderGrant is fired, onResponderMove and onResponderRelease could only be tracked within a tiny distance.
After swiping for a certain distance, onResponderMove stops first, then onResponderRelease will never be fired when release.

This is still not working!

Same issue 0.49.3 :(

I want to use this feature to solve the problem that mapView inside scrollview .
I use onResponderRelease well on iOS,
but on android , onResponderRelease  sometimes not fire.
My RN version  0.44

here is some code I used , may help someone whose MapView in Scrollview.
my page rootView is scrollview , I use custom view contain mapView. then I put custom view in scrollview.
// In custom view

// add some touch event
    @Override
    public boolean onInterceptTouchEvent(MotionEvent ev) {
        if (ev.getAction() == MotionEvent.ACTION_UP || ev.getAction() == MotionEvent.ACTION_CANCEL) {
            if(mySv != null){
                mySv.requestDisallowInterceptTouchEvent(false);
            }
        }
        if(ev.getAction() == MotionEvent.ACTION_DOWN){
            if(mySv != null){
                mySv.requestDisallowInterceptTouchEvent(true);
            }
        }
        return false;
    }

@Override
    public boolean onTouchEvent(MotionEvent event) {
        return true;
    }

// get ScrollView
    public ScrollView getParentView(ViewGroup view){
        ViewParent parent = view.getParent();
        if(parent == null){
            return null;
        }
        if(parent instanceof ScrollView){
            ScrollView sv = (ScrollView)parent;
            return sv;
        }
        else {
            if(parent instanceof ViewGroup){
                ViewGroup forceView = (ViewGroup)parent;
                return getParentView(forceView);
            }
            else {
                return null;
            }
        }
    }

//when init ，find out scrollview
 public CustomView(Context context) {
        // .........
        final CustomView thisView = this;
        this.getViewTreeObserver().addOnGlobalLayoutListener(new ViewTreeObserver.OnGlobalLayoutListener() {
            @Override
            public void onGlobalLayout() {
                Log.w("getViewTreeObserver","layout donw");
                thisView.getViewTreeObserver()
                        .removeOnGlobalLayoutListener(this);
                ScrollView sv = thisView.getParentView(thisView);
                if(sv != null){
                    thisView.mySv = sv;
                }
            }
        });

    }

then In ReactNative , You need not handle touch event to block scollView scoll, otherwise It may cause error.
<ScrollView>
    <View style={styles.mapContainer}>
                            <MapView
                                style={styles.mapStyle}
                                ref={this.mapRef}
                            />
      </View>
</ScrollView>


Warn if gcc version is lower than 4.8

The -Og parameter was introduced in in gcc 4.8. When running make with older versions of gcc (as the default one in OSX/XCode), the following error message appears:
error: invalid integral value 'g' in '-Og'

The following PR shows an error message if the gcc version is older than 4.8. I've chosen not to exit the make process, just in case this logic would fail in some edge case.
See also:

Makefile: Emit warning gcc version is lower than 4.8.0
GCC 4.8 Release Series Changes, New Features, and Fixes

Actually you can remove -Og completely, but thanks for noticing this.


Missing extension commands from Astarte device

The extension commands from the Astarte device should be made available to this repository.
Specifically, the command generate-interfaces is necessary.
This can be easily accomplished by adding the following line in the west.yml manifest for this project.
    - name: astarte-device-sdk-zephyr
    [...]
      west-commands: scripts/west-commands.yml
    [...]

One drawback is that some commands are duplicated in this repository.
We could prefix them with astarte- or make them generic enough so that no duplication is required.

Solved in PR #39


vm auto-registration: refactoring

Please provide a description of this PR:

refactoring of the VM auto-registration code into separate sub-components

logic of connect/disconnect/cleanup lifecycle - pilot/pkg/workloadentry
logic of health checking - pilot/pkg/workloadentry/internal/health
logic of persisting internal state as part of WorkloadEntry resource - pilot/pkg/workloadentry/internal/store
logic of auto-registration - pilot/pkg/workloadentry/internal/autoregistration

this PR is merely a refactoring

it doesn't add any new features
all unit and e2e tests remain unchanged

a groundwork for https://github.com/istio/istio/pull/44712

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

/test all

/test all

Are there any functional changes or is it all code reorganization? Could you note this in the PR description?

@shashankram thank you! updated the PR description

PR needs rebase.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@yskopets: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

bookinfo-build_istio
1b3cef632b33ffce52ce61497d4acc8636877f11
link
true
/test bookinfo-build

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

reworked into https://github.com/istio/istio/pull/44827 and https://github.com/istio/istio/pull/44834


search bar moved to left and text becomes bold

@TaufeeqRiyaz

@TaufeeqRiyaz Recently played section is also added


service_sdk_mappings possibly mapping "dynamodb": [..., "DynamoDB"] ?

Hi there!
Perhaps I'm misunderstanding the usecase for service_sdk_mappings, but I think it's missing some entries.
For example, I would like to find all possible IAM Actions associated with a Cloudtrail action.
I don't know how to map a Cloudtrail action with
{ eventSource "dynamodb.amazonaws.com", eventName "BatchExecuteStatement"} (for example) (this may be a bad example since it's a data-level event)
to DynamoDB.BatchExecuteStatement due to capitalization.
Am I mis-using service_sdk_mappings or should it, in fact, have this in there?

as a workaround, for now I am trying an educated-guess by basically normalizing each entry in sdk_method_iam_mappings from "DynamoDB.BatchExecuteStatment" to "dynamodb.batchexecutestatement" and trying a best-guess "${eventSource without the amazonaws suffix}.${eventName}".lowercase() in addition to the mappings.

Hi @wittekm,
This mapping is for the API methods (specifically, in the form the AWS JS SDK expects) to AWS IAM actions. It doesn't yet have any mapping towards CloudTrail.
You can map the AWS JS SDK methods to the raw API methods using the service JSONs included, but I don't believe there's any guarantee that the raw API methods and the CloudTrail events will line up.
You can also use the service name correlations at the bottom of the map to help you perform the appropriate service-to-service mappings.


data: mustafaansarii.jason

Fixes Issue

Changes proposed

Check List (Check all the applicable boxes)

[ ] My code follows the code style of this project.
[ ] My change requires changes to the documentation.
[ ] I have updated the documentation accordingly.
[ ] All new and existing tests passed.
[ ] This PR does not contain plagiarized content.
[ ] The title of my pull request is a short description of the requested changes.

Screenshots

Note to reviewers

looks good now

Duplicate of #5100


WIP updated the examples

This PR updates the examples.

Pull Request Test Coverage Report for Build 2290

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

Totals

Change from base Build 2260:
0.0%

Covered Lines:
552

Relevant Lines:
588

💛  - Coveralls

Pull Request Test Coverage Report for Build 2290

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

Totals

Change from base Build 2260:
0.0%

Covered Lines:
552

Relevant Lines:
588

💛  - Coveralls

@rmjarvis My plan was to update all of the examples in this PR. I know this will leave the repo in a middle state for a bit, but I thought it would make code review easier. That seem good to you?

OK @tmcclintock and @elisachisari - I have updated all of the examples and other scripts in the repo. I did not rerun the scripts in the benchmarks directory out of fear. I also found a bug which I am making an issue for here. I won't get to it before the end of the day. Might be a good hackweek thing.
https://github.com/LSSTDESC/CCL/issues/503

I may be out of contact as you all look through this. All of the notebooks except the buggy one run fine so we are good to merge this really. I also updated some of the text where it made sense and I caught it.

@tmcclintock @elisachisari bump on this - better to merge this sooner rather than later

I am working on a review now.

OK @philbull . We don't need to go through this with a fine tooth comb. I changed calls to CCL but left the examples intact otherwise. We really should just merge this and get on with it. Your time is probably better spent doing something else.

I have also started my review.

I made use of the opportunity to go through and tidy up several of the notebooks (typos, bad formatting etc). Sorry to complicate things by adding them to this PR, but I think the changes should be pretty uncontroversial, and it'll be good to get @tmcclintock's feedback while he's working through this anyway.

Sounds good. When @tmcclintock is ready, he can merge the code.

This PR is currently failing the CI tests, but due to flake8 rather than the test suite. When did this get switched on? It looks like it might be being a bit aggressive with the "unused variables" setting, although it has found some actual invalid characters in some of the files too.


su: must be suid to work properly

there was a problem with bbsuid in the busybox package that stopped su working.
This can be fixed by updating the bbsuid binary with apk add busybox-suid
Can you add the busybox-suid to the default packages that are pulled into image-alpine?

Hi @ScrumpyJack, thank you for your feedback,
I created the PR #20, with busybox-suid added by default, as soon as the PR is merged, we will rebuild the image and test if everything seems OK

It is still not added to alpine images of older versions (like 3.5, 3.4).

i see the same happening in Alpine 3.8 lxd container


force counts to be same length as bins

This forces User_Defined to provide bin counts for all bins, and not omit zero-count bins at the tail of the distribution.

Also, I should add, resolves #717 by @weikang9009


Add scaling up

Hi, i added a feature which make it possible to designers to work based on certain device size and then scale that render up to current device size if it is larger.
I felt for need and came across this feature when i was working on a Figma template which was designed based on 1540px screen width, and i needed to match my sizes like fonts, column widths, aspect ratios and other stuff with that design and so i first used ResponsiveScaleBox with width 1540 but then i wanted to scale it up to my screen size without any empty borders. So by this feature, i could design based on the figma sizes and then scale up to my screen size.
I hope I made my point clear, i'm not native English speaker

Thanks @Coded-By-Pouria , I think this is interesting.
There should be a way to do this with the existing API without making a change? Can you wrap the widget with a Transform.scale?


Socket Holder

Inspired by Zack's Socket Holders, parts 1 and 2 might be good replacements for his designs.
My socket set contained 16 metric and 16 imperial sockets, so I created parts 3, 4, 5, and 6, which you see printed. They can be made "stackable", which is handy for carrying or storing tools.

The "stackability" depends on the length of the sockets. In the picture, nothing can be stacked on top of the right side of the "Metric >=7mm" bin. This has been modified in the SCAD file by making that taller.
The measurements of each socket were taken with a NEIKO digital caliper (used as a non-digital caliper due to the batteries being dead.
The main issue I fixed in the iterations of this design was tightness for holding the sockets. Because we want to be able to pick out a socket with a single finger (and because openscad underestimates cylinders), I have added 0.6mm to each diameter.
This openscad project automatically spaces the socket holes, and it echos a message about the leftover space (which is spread between the holes). If this leftover space is much less than 0, you should increment the width to use another grid unit.

Nice! How did you color in the letters?

Nice! How did you color in the letters?

I used a blunted sharpie: It wouldn't reach into the letters, but made a nice outline.


Code analysis: Possible Overflow with Labels

The code analysis tool should take into consideration the MaxLength property for labels. Right now it is throwing a warning even though my MaxLength is set to match the field length (20 in my example below) that the label contents are populating.

Good catch. We'll fix

Did not know about the MaxLength property of Labels. Could you tell me what the Locked property does?

It specifies that the label will not be translated in the product. It can be included in the xliff file for reference but shouldn't be translated


os/signal: TestAtomicStop fails on dragonflybsd

I'm seeing a fair amount of flakiness in TestAtomicStop on the dragonflybsd builder. Some recent examples:
https://build.golang.org/log/db81785148210a9493ea8327ebed605e5edb148f
https://build.golang.org/log/c6a7018d2725db0845999d53a48f0f30520c54b8
https://build.golang.org/log/0fdb68deb085ad75fd786b1e498e5f1bf8009269
Using Go 1.10 and a recent dragonfly build.

/cc @ianlancetaylor as owner of os/signal.
There is a lot of red for dragonfly/amd64 on build.golang.org on tip (almost half are fail). I've checked 5 pages worth and none of them are TestAtomicStop tests failing. Most of the failures are related to networking timeouts.
I'll close this since it doesn't seem to be an issue anymore, but we can re-open otherwise.

Sounds good. There does seem to be persistent but random problem connecting to the local test webservice, but I'm not having much luck tracking it down.


Update treasury address

Update the treasury address to get accurate metrics.

This will be addressed later tonight.
Will update issue once everything has passed QA.


Use tsx instead of ts-node

Overview
ts-node sux, tsx rox
References and additional details
https://github.com/privatenumber/tsx
Acceptance Criteria

install tsx as dev dependency
remove ts-node dependency
replace all instances of ts-node with tsx
validate everything still works

Need By Date
No response
Screenshots/Mockups
No response
Estimated effort
No response

Another argument in favor of TSX:
--inspect-brk works with tsx while ts-node requires to use node --inspect-brk -r ts-node/register myFile.ts


Clarify support for auditing for Synapse SQL pools

In the article Auditing for Azure SQL Database and Azure Synapse Analytics
It mentions Synapse Analytics is supported, but it doesn't clarify that it applies only to Dedicated SQL pools, it doesn't apply to Serverless SQL pools.
Suggesting some options to fix it:

Add a note at the top mentioning it only applies to Dedicated SQL pools.
Add a note under the section Auditing limitations.

Thanks!

@goventur -- thank you for your feedback and suggestions.
@sravanisaluru -- please look into this issue.

@goventur could you please review the latest document and let me know if you still see any information is missing .

@sravanisaluru thank you, everything looks good!


heal CI, take 2

Changes

[x] don't pin to ancient numpy

Status

[ ] Click when ready for review-and-merge

Looks like 3.6 upstream has not yet built? 3.7 passed with flying colors.

the py36 lane is attached to psi4 release, so it's failing b/c CPHF_TASKS not in v1.3. so that one won't pass until v1.3.1
the py37 lane is attached to nightly build, so it's ok. (I need to add pylibefp so no skips.)

Ok, do you want to go ahed and merge?

I was going to wait until full green, but it's fine to merge.


Translation: Use Lokalise

Per OOB discussion with @Ixam97 I will assist in setting up translation via Lokalise.
It would be most useful if the current translators (ping @DoubleYouEl @robinhellstrom) could sign up via https://app.lokalise.com/public/7279689963f1e922c08f26.64130521/
Edit: New translators must be invited via email - please contact @Ixam97 or @jschlyter.

Also @oddvarr for norwegian translation 😉

Dutch is translated via Lokalise

Norwegian is translated via Lokalise. Easier than here.

I have added the GitHub integration to Lokalise. It will now pull changed and new strings automaticaly and we can create pull requests from within Lokalise.

I've added an in-app update notifications containing the changelog. My dev build is now in release candidate state and I would like to release to the master branch tomorrow before being on vacation for a week. A complete translation would be nice for that.
FYI: @jschlyter @DoubleYouEl @oddvarr @blixtdevelopment

Norwegian translation up to date :)

And Danish translation updated

…and Dutch is also up to date

(Lokalise is the next best thing after Belgian beer… 😜)

🇸🇪 done.

At some point, we should try to align some terms between the languages. Most likely not worth it for the change log, but other terms might be. There's support in Lokalise for glossaries, I'll take a look at that.

"Optimized driver distraction optimization" :-)

Question: Why do we have "Instant consumption" in the display, when we only write "Power" for the current power? In all languages, IMHO it makes more sense to just write "Power" and "Consumption" - since it is changing rapidly it's pretty obvious is not a long term number. WDYT?

Just a little heads up: I'm nearing completion for my planned 0.24-Features. I don't think there will be any more strings added in the meantime. There are a few new strings added though. So feel free to have a look.

What about "Consumption" vs "Instant consumption"?

I think I will leave it as is for now. This is a small but significant difference in my opinion. Most people associate consumption with an average andnI want it to be clear this is an instantaneous value.

@Ixam97 do you want us to translate the changes notes, or are they to be kept in English only?

do you want us to translate the changes notes, or are they to be kept in English only?

If you do translate them they will be displayed in your language upon fist app start after the update ;)

@Ixam97 what about the "SMTP server URL" - do you refer to a SMTP hostname (which is not a URL, but a FQDN I assume)?

what about the "SMTP server URL" - do you refer to a SMTP hostname (which is not a URL, but a FQDN I assume)?
I'm not that into that topic, you are probably right. I will change it to "SMTP hostname".

@jschlyter Do you plan on translating the changes, or are the swedish users going to see english change logs? 😛

Fixed now.

@Turder84 can you find a fellow Finnish speaker in the forum and ask them for review?

I'll ask around, I know that few Finnish guys are running the app.

I can translate to french. Can you add the language in Lokalise ?

I can translate to french. Can you add the language in Lokalise ?

Done!

Thanks, i'm on it.
I might have to install the app in my car to finetune some strings, to understand the context.
French will stay a bit unfinished in Lokalise until then.

I can translate to french. Can you add the language in Lokalise ?

Done @LaurentVitalis!

Can I have the export permissions please ? I want to add strings.xml in my values-fr folder to QA my build.
Thanks

Can I have the ~export~ download files permissions please ? I want to add strings.xml in my values-fr folder to QA my build. Thanks

Fixed. This also means you can edit any language, but please stick for French :-) 🇫🇷

Can anyone add me to the project so I can review the Finnish translation?

Can anyone add me to the project so I can review the Finnish translation?

Email me and I'll add you.

I think the Finnish translation is pretty much done now. Got few okays already and we have done final tweaks based on new suggestions. Maybe some further updates possible when we see language in the app and see if something needs to be edited?

I think the Finnish translation is pretty much done now. Got few okays already and we have done final tweaks based on new suggestions. Maybe some further updates possible when we see language in the app and see if something needs to be edited?

Yup, I agree. All the translations should be reviewed now also.

French is 99.5% done.
There are two last things I want to tweak.
Also, I need to verify something, it looks like POWER and CONSUMPTION are swapped.
Consumption being the average kWh/100km, if should stay between 17 and 30 kWh/100km (and not go up to 200+kW).
Power is the instant energy draw sucked by the motor(s).

@LaurentVitalis What part exactly do you mean? I'm not aware of any swapped descriptions.

@LaurentVitalis What part exactly do you mean? I'm not aware of any swapped descriptions.

False alarm. At first I thought I inverted Power and Consumption because I was surprised by the values.
But my localization follows every other languages.

I've added a few strings for 0.24.1. Please have a look at it before release. 👍

I've added a few strings for 0.24.1. Please have a look at it before release. 👍

I'm going to try to create a task in Lokalise for this release and see how that works.

For the actual translation work the task appeared to work, I got notification of the task and also notification that it was already completed before I had time to have a look at it. What is missing is the review task (which I completed this morning), that might be something to consider if that can be created as a task also? My assumption is of course that translations should be verified.

In case anyone missed it over at Lokalise: I'm planning on releasing 0.25 really soon. So it would be nice to have all translations done within the next few days.
Also remember to have a look at the key dialog_reset_message since this has changed since 0.24.
Thank you!

In case anyone missed it over at Lokalise: I'm planning on releasing 0.25 really soon. So it would be nice to have all translations done within the next few days.
Also remember to have a look at the key dialog_reset_message since this has changed since 0.24.

Thanks for the headsup - I just finished the Dutch translation - I hope you can still include it in the final release.

Swedish should be ready, although the tasks still says I have 208 words to translate.

French done.

@jschlyter My guess is that this is due to you not being an assignee to that language.

@Ixam97 I can work to make an pt_BR version, I just don't have an environment to build and put the app on our Volvo, I'm waiting for new slots for testing.

@Ixam97 I can work to make an pt_BR version, I just don't have an environment to build and put the app on our Volvo, I'm waiting for new slots for testing.

For translation, no build environment is required. We use Lokalise, a web-based platform for translation (localization) - independently from the actual software building process

@Ixam97 I can work to make an pt_BR version, I just don't have an environment to build and put the app on our Volvo, I'm waiting for new slots for testing.

For translation, no build environment is required. We use Lokalise, a web-based platform for translation (localization) - independently from the actual software building process

I know that, but sometimes is better when you see the message in the real context to use correct words, and also check if the size fits.

Brazilian Portuguese translation is done, now that I have app installed thanks to the procedures in #185 and @Ixam97 created the language on Lokalise

@jschlyter Do you think you can finish the Swedish translation within the next few days? I would like to have it as complete as possible for the next update.

@jschlyter Do you think you can finish the Swedish translation within the next few days? I would like to have it as complete as possible for the next update.

Done.

Hey all,
I am a Volvo owner from Taiwan and found this just now.
@Ixam97 I would like to contribute the translation to Chinese, unfortunately it seems the APP didn't launch but we can still try to create a testing group via your github.
I am not a programmer, but truly want to be part of the project.
Thanks.

I would like to contribute the translation to Chinese

Simplified (zh_CN) or traditional? (zh_TW)?

I would like to contribute the translation to Chinese

Simplified (zh_CN) or traditional? (zh_TW)?

Traditional zh_TW

I would like to contribute the translation to Chinese

Simplified (zh_CN) or traditional? (zh_TW)?

Traditional zh_TW

I believe we need an email address to set you for as a translator, please get in touch via email to @Ixam97 or @jschlyter and we'll add you to Lokalise.


reset in-memory notes when loading a new puzzle

this fixes the issue where notes were duplicated when loading a puzzle for the 2nd+ time

:bowing_woman:


fix: client-navigation on MenuItem

I simply copied the code from the useLink hook. This was enough to unblock myself for now.
I wonder if there are more similar cases across the codebase and if it would make sense to extract this to a helper.
The most important part here is that e.preventDefault() is getting called which is only possible by hooking into the native event. Maybe client-side router integration is also something which could be handled as part of usePress already but I'm not entirely sure if this makes sense.
I'm open to suggestions to better generalize this fix!
✅ Pull Request Checklist:

[ ] Included link to corresponding React Spectrum GitHub Issue.
[ ] Added/updated unit tests and storybook for this change (for new code or code which already has tests).
[x] Filled out test instructions.
[ ] Updated documentation (if it already exists for this component).
[ ] Looked at the Accessibility Practices for this feature - Aria Practices

📝 Test Instructions:
Set href on a MenuItem wrapped inside RouterProvider. Full page load triggered before.

Could you provide an example of the broken behavior you're seeing? I tried an example locally and saw client navigation working with menu items already.


📝 Update includes in docs/fr/docs/tutorial/path-params.md

Update includes in docs/fr/docs/tutorial/path-params.md
Related #12554

Thanks for your help! @kantandane :nerd_face:


Add information on card rendering for android

Adds the breaking changes included in v1.2 of adaptive cards, also fixed the original way of rendering the card

OPS Build status updates of commit 9f02fbf:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

AdaptiveCards/sdk/rendering-cards/android/render-a-card.md
:white_check_mark:Succeeded

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.


[Build] Reduce pulsar-test-latest-version docker image size

Motivation
Smaller pulsar-test-latest-version docker image size makes it feasible to build the image once and share it between the different parallel integration test build jobs. This will help improve the Pulsar CI in the future since it opens up new possibilities to save build system resources (GitHub Actions Runner VM time).
Modifications

Build go-lang examples in a separate temporary layer. This reduces the final size of the image by at least 600MB.

Cleanup files as last steps

Cleaning up files doesn't reduce the size of the image yet since it requires docker build squashing. The cleanup step is preparation for future changes.
To actually reduce the final image size, it would require -Ddockerfile.build.squash=true (uses docker build --squash). This is not part of this change since the image size isn't important until images are transferred over  the network. Squashing the image would just add overhead to the current build.
Remove Pulsar CPP library's static libraries /usr/lib/libpulsar*.a (since no tests use it)
Remove gcc compiler (since no tests use it)

I wonder how to share the docker image between different jobs?

@zymap The high level idea is described in the draft PIP Changes to GitHub Actions based Pulsar CI. It's possible by using the GitHub Actions Cache. There's an action that adds a local HTTP proxy to the cache where you can POST and GET any key from the cache. I've been experimenting with it and it's really great and efficient. I'll share the results on the mailing list asap (next week).

/pulsarbot run-failure-checks

/pulsarbot run-failure-checks

@lhotari - great change, and I'm looking forward to seeing the GitHub Actions Cache in practice. That should really speed up the builds.


Overwrite SW language pack snippets with own theme snippet files don't work (only for DE and EN)

PHP Version: 8.2 | Shopware Version: <IP_ADDRESS> | Affected area / extension: Platform(Default)

Expected behaviour:
It should be possible to overwrite the language pack plugin (v4.9.0) snippets with own theme. But it does only work for DE and EN. For FR and NL for example it doesn't work.
Actual behaviour:
The language pack plugin overwrites existing shopware snippets from own theme. This happens for every language the plugin sets its own snippets so any language beside DE and EN. The only way at the moment is to override them through the Shopware backend.
How to reproduce:
Please use SW <IP_ADDRESS> and language pack plugin 4.9.0 with languages FR or NL.

[public] Shopware automated response: This issue is linked to the internal issue: LAN-179
[created from NEXT-38519, comment 498781]

I'll use the original ticket, since it is probably a NEXT issue.
https://github.com/shopware/shopware/issues/4883


Use setuptools, so binary wheel builds work

As it says on the tin.
Seems to work for wagl, and on raijin, hopefully it works on Travis too. Can
find out before merging this PR.

Coverage remained the same at 37.027% when pulling c60dbabca1585b7ff85a9c87013289a45ec0e9a9 on use-setuptools into 84f87f2dcf849c087a9a1a0f69db3e7eda1e3bb8 on master.


It is possible to not view the message: "tail -F xxx"

Hi,
I am using frontail in a environment and I would like to know if exist a way to disable the message about: "tail -F xxxx"
Many thanks in advance.
Juan Antonio

Do you mean hiding whole top bar (with a search field) or only text saying "tail -f ..."?

Yes, when you handle several microservices, the title is large so it we could hide, it could be nice. It is responsability of the sys admin to load all log files.

One alternative, could be show something like: "tail -F ..." with an anchor and past all details on a title attribute:
http://www.w3schools.com/tags/att_global_title.asp

title attribute and text-overflow: ellipsis; I think is the best solution for that. Don't have enough time to fix that right now. @jabrena are you interested in contributing?

Yes, mate

Today, I found the HTML code to omit title:
tail -F __TITLE__

I will check how to send the variable Data in order to not inject in the web document.


Issue with the tenantAllowBlockList Link

Is the link describe in the document applicable anymore? https://protection.office.com/tenantAllowBlockList
[Enter feedback here]

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

 The feature is currently in some level of Preview. Are you saying you had access to it and it went away, or you never had access to it and you can't access it?

Dear Chris
Thank you for your reply.
I had never access to it. Just read about it in MS Docs and tried it access
it. I have the roles mentioned in the matrix and the licenses needed (e5)
Perhaps some screenshots would be useful.
Thank you for your time in advance.
Georgios
Sent by George Fardellas at Gmail dot com
On Wed, Feb 24, 2021, 19:24 Chris Davis<EMAIL_ADDRESS>wrote:

@gfard https://github.com/gfard, yes. The feature is currently in some
level of Preview. Are you saying you had access to it and it went away, or
you never had access to it and you can't access it?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/microsoft-365-docs/issues/4240#issuecomment-785240159,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAC3ILRQV63ZGJS7HKJWQETTAUY6FANCNFSM4YETF3GQ
.

@chrisda Thank you very much for sharing this explanation. @gfard Hope this comment is helpful for you. If you see a documentation update is required, please feel free to open an issue for the same. We proceed here to close it. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


Language request: APL

What programming language should we add?
APL. It may seem like a weird phenomenon from the 1970s, but it has a lot to teach a modern programmer. I'm currently learning it, finding it extremely intellectually stimulating, and wishing it had an Exercism track. :)
What is the official website for the language?
I'm not sure there's one official website, but the APL Wiki is at http://aplwiki.com/.
Is this a language that comes in many variants? If so, which variant should we support?
There are several interpreters available, both open-source and otherwise. The main OSS implementation that I'm aware of is GNU APL, which is most likely what we should target, especially since it claims to implement nearly all of ISO 13751. Another possibility is to target A+, which is closely based on APL but is AFAIK not completely compatible with it; its interpreter is likewise OSS.
I understand that Dyalog APL is a popular proprietary interpreter, but I think it probably makes sense for Exercism to target an OSS interpreter if one exists and is sufficiently capable.
I'm not terribly certain of how great the differences are between these interpreters' dialects of APL, or how closely any of them adheres to the ISO 13751 standard.
Is there a testing framework available for the language?
Unit testing APL is kind of a funny idea, because of the nature of the language, which may explain why I've only found one unit testing framework: APLUnit. It's written for Dyalog APL; I don't know if it can run on GNU APL as is, or if it would need modification (I can try it and see, but haven't had occasion to do so yet).
Unfortunately, I haven't been able to find an xSpec-type test framework for APL.
Who will be leading the effort to launch the track?
I could initially, I suppose, but I'm a beginner in APL. I hope more experienced APL developers will join me.

If it is helpful, I can start trying to develop Exercism exercises for APL and see what roadblocks I run into.

@NobbZ Good question. Fortunately, all APL symbols are part of Unicode, and APL keyboard layouts are available for all major operating systems, so this isn't a major issue.

@marnen I think APL would be a very interesting track for Exercism.
I think, before I bootstrap up a track, I'd like to see what an exercise would look like. Would you be willing to do a couple of experiments to implement a simple hello world exercise, as well as one slightly more complex one?

@kytrinyx Of course!

Hmm. This may be harder than I thought, because of the way APL works. APLUnit probably won't work on GNU APL, because it uses Dyalog's :if...:endif control structures, which aren't part of the ISO standard (which is what GNU APL implements). OK, fine, I could probably port it to GNU APL or write something similar (which, if I keep it to ISO-standard APL, should also run on the Dyalog interpreter), but there's another issue.
Apparently APL has the concept of a workspace, rather like a Smalltalk image, as the dominant metaphor for code organization. As such, there's relatively little notion of libraries that would be analogous to, say, Ruby gems, so even if I do write a testing library, I'm not sure how to load it. There is TieDyedDevil/apl-pkg, which is a package manager for GNU APL, but apparently it does enough OS-specific stuff that the maintainer only expects it to run on Linux. I don't know how hard it would be to get it to work on Mac OS or Windows; probably not too hard, but...
I'm still going to try to make this happen as time permits, but fitting APL into a modern ecosystem is surprisingly weird.

@marnen this is tricky. I'm hereby classifying APL as esoteric purely on the basis of this discussion ✨

even if I do write a testing library, I'm not sure how to load it

Huh yeah. Distribution is solvable—we could make sure that the exercism CLI sticks the library on their machine. But that still raises the question of how to get it into their workspace. We could simply instruct them to copy it to the exercise (and then we could ignore it when we upload to the site or something).

According to Functional Geekery 76, there seem to be 2 APL derivates, which according to the interview are similar but not equal to APL. Perhaps those are a better fit for exercism:

J language
K language

@NobbZ There's also A+, which I mentioned above, and is probably closer in spirit to APL, from what little I know.
@kytrinyx:

Distribution is solvable—we could make sure that the exercism CLI sticks the library on their machine.

I would regard that as not Exercism's responsibility. None of the other Exercism tracks that I've played with (Ruby, Haskell, ECMAScript, Erlang, Elixir) does this kind of environment setup AFAIK. Instead, the client just downloads source code files and assumes that you have a suitable environment in which to run them.
But I suppose that if we don't have a usable package management system for installing the testing library, we could download it along with the first exercise. That just seems like something I'd rather avoid, though.

But that still raises the question of how to get it into their workspace. We could simply instruct them to copy it to the exercise (and then we could ignore it when we upload to the site or something).

Yeah...or if the package manager works, then we do it that way.
I doubt that this is an insurmountable problem, but APL was designed for a somewhat different sort of environment than has become the norm.

As a thought experiment, how would we implement a Smalltalk track for Exercism? Maybe there are some ideas to be found there...

I would regard that as not Exercism's responsibility.

On the one hand... yes, but on the other hand we've already planned to implement it. I can't remember which tracks ended up having to copy/paste the same config files into all the exercises because we don't provide an opportunity for global setup.
Our planned solution looks like this:

Each track can optionally have a global/ directory.
Anything in the global directory (recursively) will be delivered to the CLI and placed in the root of the track directory, upon a specific command (e.g. exercism setup $TRACK).
We store a timestamp in Exercism's config that we called the setup command.
We pass an HTTP header on all download calls for a track that contains the last modified timestamp of the global directory.
The CLI can compare timestamps and if different, encourage the user to call setup.

@kytrinyx Oh, cool. So it would probably be somewhat sensible for me to put the unit testing library into the global directory (for compatibility with future tooling) and then use apl-pkg (assuming I can hack it to work on other OSes than Linux) in each exercise's test file to include it?

...or maybe not. It sounds like the global download functionality doesn't exist yet? Is there a pull request or issue where I can track the status of that?

@marnen If you can get this to work with the assumption that they have it in the root of their track directory, then I will go ahead and implement global. It's on my list, and is not hard, it's just not been a priority yet. This would make it so.

@kytrinyx Awesome, thanks! That also makes this project a lot more interesting for me.
In that case, I may first implement an exercise or two for Dyalog (a free version is available for non-commercial use), so that I can use APLUnit out of the box as proof of concept. Once that's done, I could see reimplementing for GNU APL. Does that sound reasonable?

Does that sound reasonable?

That sounds most reasonable!

I'll see what I can do, then. This will actually be my first crack at real APL programming (I've done exercises, and even started to implement an interpreter, but that's it so far).

 We'll see what I can do...

Well, well, well...we are in luck. It turns out that GNU APL has something of a testrunner build in. I didn't realize it sooner because it's kind of buried in the documentation, but the -T option contains a rudimentary test framework. I think this will be enough to get started.

Niiiice! This is exciting :)

OK, I've managed to implement a couple of exercises: check out http://github.com/marnen/exercism-apl.

Wild. How about them triangle thingies! :-)
This is very cool.

@kytrinyx Glad you like it! So how would you like to proceed from here?

It sounds like you're ready to make this official. If that's right, let's get you set up with a repository!

@kytrinyx Yes, let's do it!

[ ] Run bootstrap script
TRACK_ID=<id> LANGUAGE=<language> bin/bootstrap
[ ] Turn on Travis CI
[ ] Add exercism track topics
[ ] Add as a submodule to trackler
TRACK_ID=<id>; git submodule add https://github.com/exercism/$TRACK_ID tracks/$TRACK_ID
[ ] Add the requestor(s) to the exercism/track-maintainers team
[ ] Update the team's parent https://github.com/orgs/exercism/teams/TRACK_ID/edit
[ ] Invite requestor(s) to the new team
[ ] Make a note of the new track and maintainers for the newsletter

Quick question, @marnen, should we call it APL or GNU APL?

@kytrinyx I was wondering that myself. I'd hope that ISO-standard APL (which is mostly what GNU APL claims to implement) would run in any interpreter, but the tests are, for the moment, making use of GNU APL features that are not part of the ISO standard (specifically, the -T testrunner and the ⎕FIO system function for file I/O). So I think that, say, a Dyalog user would not be able to run the tests (but I can't confirm that till my Dyalog license comes through).
However, I'm trying to write all my code with as few deviations from the ISO standard as I can manage, in the hope that we can wrap all the nonstandard stuff in functions that can be differently implemented for different interpreters.
So we could call this track "GNU APL" and call a future Dyalog track "Dyalog", but that will scare off non-GNU APL users from running it even though they probably could at least get some use out of it. Or we could call this track "APL" or "ISO 13751 APL" or something, but that will annoy Dyalog and NARS users who expect it to run out of the box...
I just don't know what's best here.

BTW, just to be clear, I think this will need global implemented in order to work, because of the test utility functions that I wrote.

BTW, just to be clear, I think this will need global implemented in order to work, because of the test utility functions that I wrote.

That should be fine.

I just don't know what's best here.

I don't either. Let's make it GNU APL and we can change it if we hear good reasons to later.

Alrighty, the repo is ready for you. Sorry it took me a few days—I've been knee deep in the nextercism rewrite and only just realized I hadn't actually bootstrapped this.
The launch checklist is here:
https://github.com/exercism/gnu-apl/issues/1
Please submit individual pull requests with cohesive changes (e.g. documentation, CI setup, logo, individual exercises) to the new repo. I've given you commit access, so you can create branches right in the repo rather than creating a fork. Individual, smaller PRs will give us a few benefits:

I can help catch things that should have been documented but weren't
Ongoing development will be visible/transparent to others who are interested in following along
It will be easier to go back in the git logs to figure things out

If you find things that are odd or confusing, holler.  You can find other track maintainers in the chat room at https://gitter.im/exercism/dev (probably other rooms, too).
Welcome :heart: :heart: :heart:

@kytrinyx Great, thanks! I'll start working on it as time permits.


Miscelaneous changes

Make fit_moments private to prevent possible confusions. The rationale for this method is  to provide a reasonable initial guess for optimization methods. So it is just an intermediate step and not intended to provide an accurate moment matching method.
Improve description in the code of the initial guess heuristic.
Improve warning message for maxent and quartile method
Add example for the quartile method
Fix typos in the notebook example.
Other minor fixes.

Codecov Report

Merging #54 (3002bde) into main (5ddafc4) will decrease coverage by 0.01%.
The diff coverage is 68.18%.

@@            Coverage Diff             @@
##             main      #54      +/-   ##
==========================================
- Coverage   57.11%   57.10%   -0.02%
==========================================
  Files          13       13
  Lines        1005     1000       -5
==========================================
- Hits          574      571       -3
+ Misses        431      429       -2

Impacted Files
Coverage Δ

preliz/quartile.py
20.83% <0.00%> (+2.97%)
:arrow_up:

preliz/roulette.py
11.74% <0.00%> (ø)

preliz/utils/optimization.py
64.06% <0.00%> (-2.07%)
:arrow_down:

preliz/utils/plot_utils.py
52.63% <0.00%> (ø)

preliz/distributions/continuous.py
92.92% <100.00%> (ø)

preliz/distributions/discrete.py
98.83% <100.00%> (ø)

preliz/distributions/distributions.py
80.28% <100.00%> (ø)

preliz/maxent.py
75.67% <100.00%> (-1.83%)
:arrow_down:

preliz/tests/test_distributions.py
96.42% <100.00%> (ø)

Continue to review full report at Codecov.

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


Is required true not working?

Hi, I have an entity in My Symfony project with a field "brand' that is nullable false. I want it to be required true for the client side. However, it isn't working. Did something change in the bundle? I can't find what the problem is.
The entity:
/**
 * @var string
 *
 * @ORM\Column(name="brand", type="string", length=255, nullable=false)
 */
private $brand;

The form:
    ->add('translations', 'a2lix_translations', array(
                'fields' => array(
                    'brand' => array(
                        'field_type' => TextType::class,
                        'required'   => true
                    ),
                )
            )
        )`

I also tried this but still no required appears in the front:
    ->add('translations', 'a2lix_translations', array(
                'fields' => array(
                    'brand' => array(
                        'field_type' => TextType::class,
                        'locale_options' => array(
                              'en' => array(
                                'required' => true
                             ),
                              'fr' => array(
                                'required' => true
                            )
                        )
                    ),
                )
            )
        )`

The result in both cases in the browser is:
<input type="text" id="video_form_translations_en_brand" name="video_form[translations][en][brand]" maxlength="255" class="form-control">

I just corrected the problem setting both languages as required locales in my config.yml. Now, my required true fields have the required attribute and the non required fields remains non required.
# A2lix Configuration
a2lix_translation_form:
    locale_provider: default
    locales: "%languages%"          # locales available for the forms
    default_locale: "%locale%"      # default locale
    required_locales: [en, fr]         # requireds locales (not all fields will be required - only the ones with required true)
    manager_registry: doctrine
    templating: "A2lixTranslationFormBundle::default.html.twig"


dapTransfer fails on an erased chip

After doing a full chip erase (either by calling board.flash.eraseAll() or by other means), pyocd is unable to communicate with the boards.  An operation such as board.flash.init()  will  either hang or casue a "SWD fault" in the function dapTransfer of cmsis_dap_core.pyc
Tested on nrf mkit and freescale kl25z boards.

Pull request #142 should fix further programming problems.  Has anyone seen this problem lately?  I erased both an nrf51 and k22f and was able to reconnect and program them without any problems.  I also ran blank_test.py on both, which continuously erases and resets the boards and the test found no problems.  I think this issue can be closed.


proper splitting of elytra to wings to prevent infinite repair and loss

Put Enchantments and such on left wing
copy damage to both wings and avg on merge
proper splitting
copy max damage from elytra directly

wow, this is great! thank you for the PR :)


fixing typo

@TheBeardedLlama,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by .NET Foundation. We will now review your pull request.
Thanks,
.NET Foundation Pull Request Bot

thanks!

Thank you guys!
The work you've done with idsrvr is amazing! :)
There are a few more coming your way ;)
I'm going through all the docs as I'm starting a fresh implementation and I'd rather get it right first time around


Failed operations should be able to be submitted manually (or even automatically)

We should track failed ops and give an easy restart

nice to have some day ;)


Can this be used without using picovoice cloud services?

Or without using their API or console?

Currently it cannot unfortunately however if you can find your own wakeword detector then you can use that instead of Picovoice. Underneath it all is Whisper for the actual speech to text. The Porcupine module is for detecting the word to begin listening and the Picovoice Cobra will activate and classify the incoming audio as having either having spoken word or not to work out when to end recording audio.
If you do find an offline module for wake work detection add it to this thread as this repo was create a little while back.
What are you trying to create?

After a bit of Googling there are a few offline wake word detectors:
OpenWakeWord
Howl
MicroWakeWord
These are all offline and suited to ondevice implementations. Just need to replace the picovoice part of this with a different VAD and wakeword and you're all set.

Also found this https://github.com/secretsauceai/precise-wakeword-model-maker
Seems bandoned though,

I'm trying to break into this space by making a simple voice assistance with the goal being to make an assistant more catered to ADHD idiosyncrasies.
Is Picovoice just used for training the wake word? They advertise their project as on device which implies it should work locally without calling home?

Although Picovoice is ondevice it does a key activation check and I've found in an entirely offline situation if you don't first validate the key it'll hang as my colleague has found: https://github.com/Picovoice/porcupine/issues/579
This was a while ago though so it might have been changed.
Picovoice offers a few of parts in main.py.

Audio input: The Picovoice have an audio processing library that allows you to easily access the mic without having to go super low level. That's this bit: https://github.com/garbit/whisper-voice-assistant/blob/703c429d8a492897bc759581891be151507d1e20/main.py#L24
Wake Work detection ("Hey Alexa / Hey Google / Hey Bixby" etc) - The recorder.read() reads in audio samples and then you call procupine.process(data) to pass the samples into the porcupine Wake Word detector (these lines)
Voice Activity Detector - classifies the audio data as having a voice actually speaking in the audio sample from 0.0 - 1.0 confidence. this bit

The process is as follows:

Initialise recorder object https://github.com/garbit/whisper-voice-assistant/blob/703c429d8a492897bc759581891be151507d1e20/main.py#L78-L84

recoder.start() # start the mic capture
max_window_in_secs = 3 # how long the length of the audio should be sampled
window_size = sample_rate * max_window_in_secs # `sample rate` is the rate at which audio is sampled from the microphone per second * length of the window in seconds. This gives you the total number of buckets in your audio sample over n seconds.
samples = deque(maxlen=(window_size * 6)) # This is a queue of samples over 2 seconds (`window_size * 6`)
vad_samples = deque(maxlen=25) # This is a queue of Voice Activity Detection results
is_recording = False # starts as False when we first start the script

Read the audio in, get the probability that there's voice in our sample window, and see if our Wake Word detector is confident that it's heard the wake word
https://github.com/garbit/whisper-voice-assistant/blob/703c429d8a492897bc759581891be151507d1e20/main.py#L86-L94

while True:
        data = recoder.read() # read the audio buffer from the mic
        vad_prob = cobra.process(data) # ask cobra for the probability that there's spoken voice in each of the samples
        vad_samples.append(vad_prob) # add the samples to an array so that we can later calculate the average number across that array and use a threshold to turn the mic off

        if porcupine.process(data) >= 0: # if we have samples from the mic
            print(f"Detected wakeword")
            is_recording = True # start recording flag
            samples.clear() # clear our old queue

When we're recording, work out if we need to continue listening by taking the mean average of the voice activity detector (literally take the scores from the model 0 - 1 and then sum them and divide by the quantity we have in the array) if this is greater than a threshold that's set above, then stop the mic because we think the spoken word has stopped.
https://github.com/garbit/whisper-voice-assistant/blob/703c429d8a492897bc759581891be151507d1e20/main.py#L96-L106

        if is_recording: #
            if (
                len(samples) < window_size # if the number of samples we have recorded than the number of samples we expect in the window (`sample rate * max_window_in_secs` so 16000 khz * 3 seconds = 48,000) then we keep adding to our samples array until our window is full
                or np.mean(vad_samples) >= vad_mean_probability_sensitivity # or we look over the array of the voice activity detector results (ranging from 0.0 - no spoken voice to 1.0 definitely spoken voice) and we average those results for the 3 second sample then the script keeps listening
            ):
                samples.extend(data) # keep adding the audio data to our samples array
                print(f"listening - samples: {len(samples)}")
            else:
                print("is_recording: False")
                print(transcriber.transcribe(samples)) # transcriber is just a class that uses whisper on the data that we've taken from the microphone and prints the text
                is_recording = False # turn off the mic because there's either no spoken word or the voice activity detector is no longer detecting spoken voice

Transcriber
This class uses Whisper to take an array of audio samples and detect spoken words from the sample
class Transcriber:
    def __init__(self, model) -> None:
        self.model = whisper.load_model(model) # load the model
        print("loading model finished")
        self.prompts = os.environ.get("WHISPER_INITIAL_PROMPT", "")
        print(f"Using prompts: {self.prompts}")

    def transcribe(self, frames):
        transcribe_start = time.time() # take the start time
        samples = np.array(frames, np.int16).flatten().astype(np.float32) / 32768.0 # take the frames from the audio (often there's two channels for Left and Right, so flatten these, and convert the int16 values from the audio into float 32's that the Whisper model is expecting, then normalise the array which means take the int16 values (ranging from -32768 to 32767 and divide it by 32768 to instead get a range from -1 to 1 which Whisper is expecting

        result = self.model.transcribe(
            audio=samples,
            language="en",
            fp16=False,
            initial_prompt=self.prompts,
        ) # run the Whisper model to detect the text from the audio

        transcribe_end = time.time() # calculate the end time
        return result.get("text", "speech not detected") # return the result

Hopefully that makes things clearer for you, let me know if you need more explaination on parts. I should have left comments on the codebase!


make SchedulerJobsTest preserve scheduler and job states (FINERACT-922)

FINERACT-922

Anyone willing to review this one? @nemux @awasum @xurror @ptuomola @trackcc LGTY?

The current failure here is totally unrelated to this PR, and another "flaky" test, see FINERACT-828.

It sounds like nobody seems to have any strong objections to this one, so I'd like to "self merge" it tomorrow.

The code itself looks good to me. I haven't hit this particular problem myself, so can't comment on whether it fixes the issue - but it does look like a sensible thing to do.

This looks clean..But i dont understand completely whats happening here. @vorburger Go ahead and self merge. You know what you are doing.
Minor update..seems there is a merge conflict?

I little knowledge about this but I trust the changes are OK.


Pagination access

Great library thank you!
I wanted to ask, is there any way to access / unmarshal the pagination links that get sent by jsonapis like these:
"links": {
    "first": "http://somesite.com/movies?page[limit]=50&page[offset]=50",
    "prev": "http://somesite.com/movies?page[limit]=50&page[offset]=0",
    "next": "http://somesite.com/movies?page[limit]=50&page[offset]=100",
    "last": "http://somesite.com/movies?page[limit]=50&page[offset]=500"
 }

Ideally I'd like to provide access to these offsets in a separate struct as unmarshaling them into the Movie struct sounds strange! Thanks!

@gopherfortress we recently added features to easily add "links" when serializing/marshaling. Unfortunately the same cannot be said for deserializing/unmarshaling.
However there is still a way you can parse them out:
func TestManyPayload_withLinks(t *testing.T) {
	firstPageURL := "http://somesite.com/movies?page[limit]=50&page[offset]=50"
	prevPageURL := "http://somesite.com/movies?page[limit]=50&page[offset]=0"
	nextPageURL := "http://somesite.com/movies?page[limit]=50&page[offset]=100"
	lastPageURL := "http://somesite.com/movies?page[limit]=50&page[offset]=500"

	sample := map[string]interface{}{
		"data": []interface{}{
			map[string]interface{}{
				"type": "posts",
				"id":   "1",
				"attributes": map[string]interface{}{
					"body":  "First",
					"title": "Post",
				},
			},
			map[string]interface{}{
				"type": "posts",
				"id":   "2",
				"attributes": map[string]interface{}{
					"body":  "Second",
					"title": "Post",
				},
			},
		},
		"links": map[string]interface{}{
			KeyFirstPage:    firstPageURL,
			KeyPreviousPage: prevPageURL,
			KeyNextPage:     nextPageURL,
			KeyLastPage:     lastPageURL,
		},
	}

	data, err := json.Marshal(sample)
	if err != nil {
		t.Fatal(err)
	}
	in := bytes.NewReader(data)

	payload := new(ManyPayload)
	if err = json.NewDecoder(in).Decode(payload); err != nil {
		t.Fatal(err)
	}

	if payload.Links == nil {
		t.Fatal("Was expecting a non nil ptr Link field")
	}

	links := *payload.Links

	first, ok := links[KeyFirstPage]
	if !ok {
		t.Fatal("Was expecting a non nil ptr Link field")
	}
	if e, a := firstPageURL, first; e != a {
		t.Fatalf("Was expecting links.%s to have a value of %s, got %s", KeyFirstPage, e, a)
	}
}

Thanks for the answer!
Any idea of the amount of work that is being done when doing the
payload := new(ManyPayload)
if err = json.NewDecoder(in).Decode(payload); err != nil {
	t.Fatal(err)
}

My concern is that since links are at the very bottom of the document then it is possible that, assuming a big JSON document, twice the work will be done being done, once for decoding the pagination links and once for decoding the rest.

Unfortunately the same cannot be said for deserializing/unmarshaling.

Assuming there is indeed twice the work being done, any plans to support unmarshaling of pagination links?
Thank you!

@gopherfortress if all you are after is the "links" member there is definitely a more performant solution. The code sample I posted will traverse all of the "data" member which could be fairly lengthy. Take a look at our source code; I think you want to start by altering the OnePayload or ManyPayload to fit your use case of only parsing "links".
As for fully supporting unmarshalling "links" that is a goal - however initial investigation leads me to believe adding such a feature (in a simple way) would mean breaking the existing unmarshal method signatures. We still need to put more thought into this.


Adjusts the lethality of meteor explosions

About the PR

Anti-personnel damage has been halved and tile breaking normalized. Boom hearing range and explosion radius remains unchanged.
Why / Balance

Meteors piercing through stations is good as it raises spacing potential of meteor swarms 👍. Being instantly crit by a meteor explosion through a reinforced wall which you did not see coming is not good.
Previous meteor explosions were designed to do considerable damage to (presumably) have an effect on reinforced walls, but the recent buff to meteors means that this paradigm is no longer necessary. This PR adjusts the damage of meteors such that you can survive the fragments of a singular meteor, before being crit by another or dying to depressurisation.
Technical details

Adjusts meteor YAML prototypes.
Media

C-4 with edited values detonated 2 tiles away from a Urist, of each meteor size and before/after changes.
https://youtu.be/6MxzLFs5Wqw?si=WvT6djvHUm7vrfT8

[x] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Breaking changes

No
Changelog

:cl:

tweak: Adjusted meteors to have less lethal blast fragments.

Needs before and after if you really want to get this merged before the other changes are even live.

I'll record damage adjustments then

recorded before/after, tested on an unarmoured urist with no eva protection
damage is basically halved but doesn't affect damage from spacing

back in my day meteors used to gib ya! what's with all this halving damage mumbo jumbo

back in my day meteors used to gib ya! what's with all this halving damage mumbo jumbo
edit: ohh it's for the explosion not the actual meteor itself

i actually dont know if being direct hit by the meteor will kill a player like how it destroys structures but that would be funny


[SPARK-11399] Add label support in include_example

Add label support in include_example.

Merged build triggered.

Merged build started.

Test build #44899 has started for PR 9430 at commit ced8b13.

Test build #44899 has finished for PR 9430 at commit ced8b13.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):\n  * case class UnresolvedStar(target: Option[Seq[String]]) extends Star with Unevaluable \n

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44899/
Test PASSed.

@mengxr Until now, all docs of ML and MLlib packages are substituted with include_example except for the DataTypes, mllib-linear-methods and BasicStatistics pages. As we talked before, these three files are dependent on this PR.
Shall we merge this PR so that we can modify all docs before v1.6? Or we defer it to v1.7?

Test build #47327 has started for PR 9430 at commit 3c6ddd8.

Test build #47327 has finished for PR 9430 at commit 3c6ddd8.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/47327/
Test PASSed.

Merged build finished. Test PASSed.

@mengxr Is the way to add label to include_example good to support the dialog-style docs?

I'll close this one according to https://github.com/apache/spark/pull/11128 for now


Slow inserts with mongoid 4?

Hello
I'm having trouble with new version of mongoid. So far I've been running version 3.1.6, after upgrading to 4.0.0 I noticed drastically performance reduction.
Please take a look at this very simple benchmark:
require "mongoid"
require "benchmark"

Mongoid.connect_to("benchmark")
Mongoid.purge!

class TestDoc
  include Mongoid::Document
  field :field_name_1
  field :field_name_2
  field :field_name_3
end

Benchmark.bm do |x|
  x.report("insert") do
    100_000.times { |i| TestDoc.create(field_name_1: i, field_name_2: "field name 2", field_name_3: "field name 3") }
  end
end

This thing runs on mongoid 4.0.0 over 3 times slower when compared to 3.1.6 - my question is this code somehow flawed (am I missing something obvious here?) or should I go :dancer: with ruby-prof and look for a cause? I tried mongoid 4.0.0 rc1 and rc2 - the results are even worse.
Ruby / mongodb / cpu / os:
ruby 2.1.0p0 (2013-12-25 revision 44422) [x86_64-linux]
mongod 2.6.3
Intel(R) Core(TM) i7-2670QM CPU @ 2.20GHz
Linux 3.11.0-15-generic #23-Ubuntu SMP x86_64 GNU/Linux

Any insight is greatly appreciated, cheers :wink:

@emq thanks for reporting this.
Performance is something really important for Mongoid, however is hard to test it,as we dont have regression tests for performance regressions.
Please, try to ruby-prof it, and find the slow hot-spot. Fell free to send a PR to fix the problem, and/or share your results here.
thanks

Hi
To be honest I totally forgot about this ticket, my bad <3. So as it turns out it was related to different defaults - after unifying mongodb write concern I ended up with very similar timings between two versions of mognoid :+1:
Sorry again for confusion, keep up great work, cheers! :heart:


[WIP] Initial thoughts on alerting

Currently not hooked up to the API at all, but the basic framework is here for your thoughts :-)

Note: rebased this on top of #16

👍


Update STIndex.scala and remove the Wrapped class

remove the Wrapped class to make it more clear, since we can use the tuple itself and can be sorted well.

I was using tuple before but produces wrong results. That's the reason why I use a wrapper class here.


:four_leaf_clover: Proposal: Remove the test env Logic from jenkins Plugin

What Would You Like to Add? Why Is This Needed?

Configuring StorageClass is easy;

Most users will use minikube when testing k8s, after all, this is what recommended on the k8s document;
The minikube deployment of a k8s cluster includes a default StorageClass, and users do not need to think about how to configure the StorageClass;

Use StorageClass will make our code cleaner, more consistent, and easier to understand;

With the helm chart, we can easily configure StorageClass to persist data;

With test_env we want to help users do more configuration work in the test environment, and then there is still no way around reminding users to run commands like chmod through the documentation. So the added complexity still doesn't get 100% of the work done, and is probably more of a burden to users.

Design
Use StorageClass in any environment.
Anything else

I'd like to do it.

@IronCore864 @aFlyBird0 Need some inputs here.


Consider permessage-deflate extension for WebSocket

The HyBi IETF WG has picked up recently and it looks to be moving this to RFC. Chrome, Firefox, and many libraries support this extension. Given the widespread use of text-based messages for content, compression has an opportunity for giving up-to 75% reduction in payload size. The latest draft was just published two days ago.

I think this just got a thumbs up on its way to publication (if i'm reading the email correctly): https://datatracker.ietf.org/doc/draft-ietf-hybi-permessage-compression/

:+1:

It is not possible to fully implement the specification on Android/with Java. This is the case because the Android/Java API doesn't offer the possibility to set the window bits of the Inflater/Deflater. Further the required SYNC_FLUSH option is not supported in Java until Java 7 and on Android until API Level 19 (as @swankjesse  pointed out in the Okio source). Although the option can be used since Android 2.3.

Not a big fan of that spec. Seems like a lot more machinery than should be necessary. I think we'll be able to implement enough of the spec to be useful!

Is this still on the radar?  The spec is an RFC now... https://tools.ietf.org/html/rfc7692.  Would love to see this in okhttp.

Any news? BTW nv-websocket-client support this spec.
https://github.com/TakahikoKawasaki/nv-websocket-client

At this time we don't have plans to implement.
On Sun, Sep 18, 2016, 12:11 PM Yu-Hsuan Lin<EMAIL_ADDRESS>wrote:

Any news? BTW nv-websocket-client support this spec.
https://github.com/TakahikoKawasaki/nv-websocket-client
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/square/okhttp/issues/1733#issuecomment-247856662, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAEEEVmYxzZ8nf-pPIm_vkYek-KeoD6eks5qrWLAgaJpZM4FNWTi
.

Hi @JakeWharton ,
Will you accept any PR for implementing per message deflate in okhttp?

@JakeWharton  any updates on implementing the extension?

No plans to. Feel free to send a PR
On Wed, Aug 1, 2018, 3:17 PM monil1334<EMAIL_ADDRESS>wrote:

@JakeWharton https://github.com/JakeWharton any updates on implementing
the extension?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/square/okhttp/issues/1733#issuecomment-409689335, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAEEEbNV1H0I9Y0v9mzv8uJxNxHV1aRBks5uMf7fgaJpZM4FNWTi
.

Just confirmed that this interops nicely with the NodeJS Web Sockets library. That library has many tuning parameters; OkHttp currently has none. But their defaults are reassuring; both chose 1024 as the threshold to trigger compression.
https://github.com/square/okhttp/pull/5839
https://github.com/square/okhttp/pull/5848
https://github.com/square/okhttp/pull/5851
https://github.com/square/okhttp/pull/5867
https://github.com/square/okhttp/pull/5869
https://github.com/square/okhttp/pull/5871

Huge thanks to @AntonRutkevich for pushing this forward!

Thanks @swankjesse for the implementation!
With your deep OkHttp understanding, it's much cleaner and simpler than the original one.


What is missing for supporting nintendo switch?

Hi! Great project, I was wondering what is missing from this project to also have support for Nintendo Switch?

No idea, I didn't test it. It might require minor adjustments to handling of clang flags. I am also not sure I have access to Nintendo Switch platform files for Unreal Engine.


Fix readthedocs autobuild in chplforpyp-docs

Fix readthedocs autobuild.
Looks like it broke around the same time as pychapel's did, likely for similar reasons.
I've already tried to fix by guessing the same kind of change as fixed pychapel's.
Did not work.
Should look into that sphinx-contrib repo's readthedocs build. It's working.

Issue moved to chapel-lang/chapel #9540 via ZenHub


iPad support

This PR adds support for iPad :grinning:

The vast majority of the changes are limited to the iOS UI codebase. But there is one change that affects the shared Kotlin codebase:
The original idea was to:

change the Home screen to only display "recent" payments
move the full payments history into a dedicated screen (where we could implement additional functionality in the future)

This change has been implemented (details below). But since this affects Android too, we could optionally split this change into a separate PR. Let me know what you think.

Kotlin Changes
In order to support fetching "recent" payments from the database, 2 new queries have been added to AggregatedQueries.sq:

listRecentPaymentsOrder => returns in-flight payments, and recent payments completed after the given date
listOutgoingInFlightPaymentsOrder => returns only in-flight payments

And there is a new PaymentsPageFetcher class that handles the logic for you:
fun subscribeToRecent(offset: Int, count: Int, seconds: Int) { /*...*/ }

If seconds is 0 (or negative), it will use listOutgoingInFlightPaymentsOrder. If seconds is positive, it will use listRecentPaymentsOrder, and launch a task that automatically refreshes the list when the oldest payment expires. So payments automatically get removed from the list when they exceed the desired "recent-ness".

UI improvements:

Tested on an actual iPad device and it looks great!
A few nitpicks:

in the "scan" screen, the buttons section at the bottom of the screen used to paste data or pick an image should have some bottom padding. The OS puts a black line over that section and it hides parts of the buttons.
the "View logs" screen is very narrow (since it's constrained to fit the side menu) but displays a lot of data. Not sure if that can be done, but it would be nice if the side menu width could expand when using that screen (and that screen only). Also maybe use a smaller font.

It seems that the "full payment history" button below the list of payments in the Home Screen does not always work. Sometimes pressing it does not do anything.

It seems that the "full payment history" button below the list of payments in the Home Screen does not always work.

Should be fixed with https://github.com/ACINQ/phoenix/pull/293/commits/3f22944924ddd2f207b60a8f1f1e24bb54c5ab33

the "View logs" screen is very narrow but displays a lot of data

Yeah, I don't like that screen either. And the TextView chokes when we try to have it display too much data. It's  an issue on both iPad & iPhone, so I've created a new issue for it: https://github.com/ACINQ/phoenix/issues/294


Is there any other settings? I run evaluation code directly always gets very low scores

like the above image, the absolute value only are (B4 0.220, M 0.194, R 0.502, C 0.062)
run code:
EVAL_DIR='./models/table1/msrvtt/best-checkpoint/'
CUDA_VISIBLE_DEVICES=0 python src/tasks/run_caption_VidSwinBert.py  --val_yaml MSRVTT-v2/test_32frames.yaml --do_eval true        --do_train false --eval_model_dir $EVAL_DIR

I change the mixed_precision_method to apex when evaluating, not deepspeed, I don't know if it's the root cause.

Find the cause, so close


Support for FlashAttention in Llama2

What does this PR do?
This PR introduces support for FusedSDPA operator (Flash Attention) in Llama2.
Below are preliminary results from this change:

Model
BS
Max input tokens
Max new tokens
Default performance   [tokens/second]
Performance with FusedSDPA   [tokens/second]
Throughput improvement over   default
Default memory allocated [GB]
Memory allocated with   FusedSDPA [GB]
Memory reduction over default [GB]

7B 1x
1
16
4096
124.12
124.130
1.000
14.71
14.69
0.02

4
16
4096
354.27
354.500
1.001
20.81
20.71
0.10

13B 1x
1
16
4096
68.81
68.840
1.000
27.57
27.54
0.03

4
16
4096
203.63
203.790
1.001
37.05
36.95
0.10

70B 8x
1
16
100
55.55
56.440
1.016
16.68
16.66
0.02

40
16
100
1875.95
1893.266
1.009
18.55
17.21
1.34

1
16
2048
60.23
59.617
0.990
16.76
16.74
0.02

40
16
2048
1685.81
1686.918
1.001
21.25
20.27
0.98

60
16
2048
2206.59
2208.290
1.001
24.67
22.09
2.58

1
16
4096
59.77
59.044
0.988
16.86
16.82
0.04

40
16
4096
1366.84
1368.552
1.001
24.99
23.50
1.49

60
16
4096
1789.34
1689.680
0.944
30.09
26.92
3.17

By default this is turned off.

Hi @regisss , could you take a look at this PR and trigger CI for it?

LGTM

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

@wszczurekhabana There is a merge conflict to solve since #589 was merged


added voidEval functions

Just added a voidEval function which do not need return content, just the error status. No content returning expected.
PS: I want to say thank you very much program the Rserve client for go. I am looking for it for a long time and created a very ugly client which only can do voidEval :(
feel free to reject this pull request :p

Thanks so much for the pull request, I really appreciate the effort. I have left a couple of comments on the proposed changes, see what you think to them.


[ADAP-904] [Bug] pip install dev-requirements only works on the second attempt

Is this a new bug in dbt-spark?

[X] I believe this is a new bug in dbt-spark
[X] I have searched the existing issues, and I could not find an existing issue for this bug

Current Behavior
See https://github.com/dbt-labs/dbt-snowflake/issues/784
Expected Behavior
See above
Steps To Reproduce
See above
Relevant log output
No response
Environment
- OS:
- Python:
- dbt-core:
- dbt-spark:

Additional Context
No response

dev-requirements.txt is going away.


Add simplest example straight in the docs + at the top of the main module's Haddock

From Axman6's Reddit comment at https://www.reddit.com/r/haskell/comments/6tfurj/announce_first_release_of_http2client/ .
I may just not be familiar enough with HTTP2, but I can't actually tell how to make a GET request with this library. This seems kinda fundamental, so could you add some docs explaining how to do this?

I've merged #21 , which contains a literate haskell example.


Bookmarking tabs not working properly in non-English UI

This is a Bug Report
Description
When using Firefox in a different language than English, the feature allowing to set the default bookmarking folder for a set of selected tabs (previously called "Bookmark All Tabs...") is not working. The set folder is ignored and the default "Bookmarks Menu" folder is used instead.
Steps to reproduce the problem

Set the default folder for multiple tabs bookmarking to anything but "default" or "Bookmarks Menu"
Change the language of Firefox and restart the browser
Open multiple tabs and select them
Try to bookmark the tabs- it will ignore the setting and use "Bookmarks Menu"

Expected behavior
The defined folder setting for multiple selected tabs should be honored when bookmarking.
Specifications

Version of the add-on: 2.8.0
Firefox edition: Firefox (regular edition)
Firefox version: 66.0.3
Platform: Windows
Platform version: 7

Additional context
This problem could be related to the issue #31 / #45.

Fix available with version 2.9.0.


discover devices

Add discover in discover adapter.
After device discovered automatically.
Sync device info in admin config.

After device discovered automatically.
Sync device info in admin config.
Add in https://github.com/smarthomefans/ioBroker.miio/commit/5bc47e9e56a7bce5ea00093f9d52df4c677c8ebf


Fix index limits in Bech32 mutation tests

Issue Number
#331 and #332
Overview
This PR fixes a few issues introduced by PR #332:

The function chooseWithinDataPart gives us an incorrect upper bound for the index in the insertion, mutation, and deletion tests. These tests should actually limit their indices in subtly different ways:

Test
Index Upper Bound
Notes

Insertion
T.length originalString
(we can insert after the end of a string)

Mutation
T.length originalString - 1

Deletion
T.length originalString - 1

Swapping
T.length originalString - 2
(we swap with the character to the right)

Limiting our choice of index to just the data part means that our tests are weaker than they could be, as we're not testing across the full length of a Bech32 string. Since the checksum is a function of both the data part and the human readable part, it makes sense to mutate all parts of the string.

In particular, we do need to check for mixed case characters at all locations in the string, and not just in the data part. According to the spec:

Decoders MUST NOT accept strings where some characters are uppercase and some are lowercase (such strings are referred to as mixed case strings).

Changes
I have:

[x] Corrected the upper bounds for the insertion, mutation, and deletion tests.
[x] Adjusted all tests to test across the full length of a Bech32 string, and not just the data part.
[x] Introduced a type Bech32Char, which represents the full range of characters allowed in a Bech32 string. We use this type when generating mutations.

@jonathanknowles I don't think there was an issue here to be honest.... The tests were already limiting their indexes to the worst case (swapping).... I mean.. We've already spent A LOT of hours on this encoder / decoder and testing is already very reasonable on that module.
May we please... stop? For any further ideas of improvement, use the blackboard and describe what could be done.


c++: internal compiler error: Killed (program cc1plus)

default cmake config
gcc: (Ubuntu 5.4.0-6ubuntu1~16.04.11) 5.4.0 20160609
Target: x86_64-linux-gnu

yuens@iZ280qnmzdzZ:~/code/tvm-20190513/build$ cmake ..
-- The C compiler identification is GNU 5.4.0
-- The CXX compiler identification is GNU 5.4.0
-- Check for working C compiler: /usr/bin/cc
-- Check for working C compiler: /usr/bin/cc -- works
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Detecting C compile features
-- Detecting C compile features - done
-- Check for working CXX compiler: /usr/bin/c++
-- Check for working CXX compiler: /usr/bin/c++ -- works
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Detecting CXX compile features
-- Detecting CXX compile features - done
-- Performing Test SUPPORT_CXX11
-- Performing Test SUPPORT_CXX11 - Success
-- Build with RPC support...
-- Build with Graph runtime support...

See <file:///usr/share/doc/gcc-5/README.Bugs> for instructions.
make[2]: *** [CMakeFiles/tvm.dir/3rdparty/HalideIR/src/arithmetic/Simplify.cpp.o] Error 4
make[1]: *** [CMakeFiles/tvm.dir/all] Error 2
make: *** [all] Error 2

Have any idea? Thanks in advance.

Please open a new thread on https://discuss.tvm.ai/ for general support questions :)

ok


Emmet snippets support

Check for existing issues

[X] Completed

Describe the feature
Allow use emmet snippets to input html/css.
vscode docs https://code.visualstudio.com/docs/editor/emmet
If applicable, add mockups / screenshots to help present your vision of the feature

@JosephTLyons  I'd also really love Emmet extensions support to be added into Zed. Despite the relatively small amount of thumb-ups, this remains a feature I'm really missing a lot in Zed, would add a lot, and shouldn't be too hard to implement?
Rooting for Emmet once more! ;-)

Also hope jsx，tsx

Would love this! 🙏🏼

+1 for Emmet support in Zed 😍

All the thumbs! YES!

+1


Modules were resolved with conflicting cross-version

When depending on flink-dataflow:
error] Modules were resolved with conflicting cross-version suffixes in:
[error]    org.apache.flink:flink-clients _2.10, <none>
[error]    org.apache.flink:flink-runtime _2.10, <none>
[error]    org.apache.flink:flink-optimizer _2.10, <none>
[error]    org.apache.flink:flink-avro _2.10, <none>

Is there underlying flink version is 1.0-SNAPSHOT
Any hint how to fix?

Yes, please switch to flink version 1.0.0 :)

I little bit more explanation: In the snapshot version, there was a bug which could pull in different Scala versions. This has been resolved for 1.0.0. I'll close this for now, but feel free to re-open if this problems persists.

I rebased on current master (with flink version 1.0.0) but the problem is still there - the problem seems to be that dataflow-flink runner explicitly depends scala 2.10 artifacts:
[info] downloading https://repo1.maven.org/maven2/org/apache/flink/flink-streaming-java_2.10/1.0.0/flink-streaming-java_2.10-1.0.0.jar ...
[info]  [SUCCESSFUL ] org.apache.flink#flink-streaming-java_2.10;1.0.0!flink-streaming-java_2.10.jar (1343ms)
[info] downloading https://repo1.maven.org/maven2/org/apache/flink/flink-clients_2.10/1.0.0/flink-clients_2.10-1.0.0.jar ...
[info]  [SUCCESSFUL ] org.apache.flink#flink-clients_2.10;1.0.0!flink-clients_2.10.jar (1087ms)
[info] downloading https://repo1.maven.org/maven2/org/apache/flink/flink-connector-kafka-0.8_2.10/1.0.0/flink-connector-kafka-0.8_2.10-1.0.0.jar ...
[info]  [SUCCESSFUL ] org.apache.flink#flink-connector-kafka-0.8_2.10;1.0.0!flink-connector-kafka-0.8_2.10.jar (98ms)
[info] downloading https://repo1.maven.org/maven2/org/apache/flink/flink-avro_2.10/1.0.0/flink-avro_2.10-1.0.0.jar ...
[info]  [SUCCESSFUL ] org.apache.flink#flink-avro_2.10;1.0.0!flink-avro_2.10.jar (50ms)
[info] downloading https://repo1.maven.org/maven2/org/apache/flink/flink-connector-kafka-base_2.10/1.0.0/flink-connector-kafka-base_2.10-1.0.0.jar ...
[info]  [SUCCESSFUL ] org.apache.flink#flink-connector-kafka-base_2.10;1.0.0!flink-connector-kafka-base_2.10.jar (64ms)
[info] Done updating.
[error] Modules were resolved with conflicting cross-version suffixes in {file:/Users/rav/projects/scio/}scio-flink:
[error]    com.twitter:chill _2.11, _2.10
[trace] Stack trace suppressed: run last scio-flink/*:update for the full output.
[error] (scio-flink/*:update) Conflicting cross-version suffixes in: com.twitter:chill
[error] Total time: 86 s, completed Mar 10, 2016 11:15:21 AM
> scalaVersion
[info] 2.11.8

Maybe flink-dataflow should be released for both scala 2.10 and 2.11?

You brought in Chill 2.11 through some other dependency, right? In that case you have to use the _2.11 suffixed Maven modules. We can probably just introduce a property in the pom.xml which you override when you want to use Scala 2.11.


Update continuous-delivery-faqs.md

Updated 5 faq
Harness Developer Pull Request
Thanks for helping us make the Developer Hub better. The PR will be looked at
by the maintainers.
What Type of PR is This?

[ ] Issue
[ X] Feature
[ ] Maintenance/Chore

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app


Input data from keyboard

How do I keep the connection open so I can input data from the keyboard?

This seems like it might be more of an event machine question than a web socket question. Hard to tell without more context. I did a quick google though, and it looks like event machine will work with keyboard input: https://github.com/eventmachine/eventmachine/wiki/Code-Snippets


Creaded dynamic interface string

Added the possibility to Polybar to detect current in-use network adapter by setting "interface = " on config file.

Hey, thanks for contributing! But, there is already a PR at #2025 that does the same thing. It also does not rely on external shell commands.
Closing in favor of #2025.


CSRF Validation Token Error With Omniauth (Rails API Only)

I am currently testing rodauth to see if it could be a good use for some of my Rails API only apps.
I made a basic demo following the Omniauth tutorial but in API only mode.
When I try to make a POST request to the endpoint http://localhost:3000/auth/facebookvia a separate React app, I get a CSRF error:
<ActionController::InvalidAuthenticityToken: ActionController::InvalidAuthenticityToken>

You can see this reproduced on Postman here.
I'm relatively new to figuring out auth in an API Only environment, so my apologies if this is a poor issue.
I'd be happy to reopen this as a discussion if it is a non-issue/something in my misunderstanding of omniauth.

I was able to solve this - my issue was that I wasn't calling it from Facebook's authorizing grant with Rodauth as a redirect_uri, but rather directly from Postman. This was my mistake!


CFFL

CFFL

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_077ldvmb1t/clone/recipe/meta.yaml:20: missing_wheel: For pypi packages, wheel should be present in the host section
ERROR: /tmp/abs_077ldvmb1t/clone/recipe/meta.yaml:40: http_url: http://hitchdev.com/strictyaml/ is not https
ERROR: /tmp/abs_077ldvmb1t/clone/recipe/meta.yaml:40: missing_documentation: The recipe is missing a doc_url or doc_source_url
WARNING: /tmp/abs_077ldvmb1t/clone/recipe/meta.yaml:40: missing_description: The recipe is missing a description
Errors were found


No text is shown when starting lillygui.

This may be due to the recent changes for the splash screen? Perhaps we should just skip over the splash screen in the gui, for now (as suggested by tauraamui)?

Yes, I suspect the issue is the font used doesn't support the braille characters we're using to render the splash screen ASCII version of the Lilly logo.


Bug: Sometimes the route-coloring logic is incorrect for low-cost routes

https://nycmesh.slack.com/archives/C06H99CGY/p1724868148650539
https://nycmesh.slack.com/archives/C139RPHGF/p1724715345967169

mesh_ospf_data.json
NN 2482 in the above JSON shows the issue


some engineering refactors

This PR is going to refactor/introduce the following changes.

.gitignore: ignore cmake-* and .idea dirs.
Align the ggml backend (s/CUBLAS/CUDA/g, s/HIPBLAS/HIP/g) and support CANN, MUSA.
Extract public utils for reusing sampler_method/scheduler/rng.
Support specifying main-gpu at the API level.
Migrate sd_type_t to ggml_type.

stable-diffusion.h is a standalone header file and should not depend on ggml.h. When sd.cpp is provided as a third-party library, user only need to use stable-diffusion.h and the corresponding library files.

stable-diffusion.h is a standalone header file and should not depend on ggml.h. When sd.cpp is provided as a third-party library, user only need to use stable-diffusion.h and the corresponding library files.

rebase master and revert the change, PTAL.


Bulk Add Volunteer - create UI

Build screen according to mock-up

Currently no ui, just a modal in the VolunteerListTab


Unable to capture from a simple Vulkan test program

Description
Hello! I'm unable to capture from a small Java test application that calls the Vulkan API. The Java application is just a Vulkan "hello world" that displays a triangle onscreen, and it runs correctly (without RenderDoc) without any kind of validation errors with the validation layer enabled. However, when I try to run the same application in RenderDoc, I run into a similar error that seems to have been described (poorly) in this old ticket:
https://github.com/baldurk/renderdoc/issues/1487
In my case, this isn't an Android application, it's just a plain Java desktop application.
Steps to reproduce
I'm unsure what would be the easiest way to give you the code to run. I'm assuming as a native code developer, you probably aren't set up to build a Java project. If you do want to try building the Java code, clone the jcoronado repository:
https://github.com/io7m/jcoronado
You'll need Apache Maven and JDK 17. You don't have to use that particular JDK - any JDK 17+ implementation should work (Amazon Coretto, Azul, Adoptium, etc). Once you have those, you should be able to build the project with:
$ mvn clean package

That will produce a jar file at com.io7m.jcoronado.examples/target/com.io7m.jcoronado.examples-0.0.1-SNAPSHOT-main.jar that you can run with $ java -jar com.io7m.jcoronado.examples/target/com.io7m.jcoronado.examples-0.0.1-SNAPSHOT-main.jar. The example in question that I'm trying to trace is the hello-vma example:
$ java -jar com.io7m.jcoronado.examples/target/com.io7m.jcoronado.examples-0.0.1-SNAPSHOT-main.jar hello-vma

If you don't want to do all of this yourself, I can try attaching an executable jar file to this ticket. It's 34mb, and I'm unsure how portable it is, so that's why I've not included it yet.
Assuming you have that jar file, it's pretty straightforward to trace the execution in RenderDoc. Just set the
executable to trace to your java binary, and the command-line arguments to match the -jar ... invocation above. I used this:

If I click Launch, the Java program runs, and then immediately seems to exit, and I see this in RenderDoc:

There doesn't seem to be a capture file I can send you, but there is a log file:
cap.log.txt
Please let me know if there's anything else I can give you, or if you need the jar file above.
Environment
$ renderdoccmd version
renderdoccmd x64 v1.18 built from f8476b53b55e21b640897a8bdc5c0e11a777de28
Packaged for Arch (1.17+2.gf8476b53b5) - https://aur.archlinux.org/packages/renderdoc-git
APIs supported at compile-time: Vulkan, GL, GLES.
Windowing systems supported at compile-time: xlib, XCB, Wayland (CAPTURE ONLY), Vulkan KHR_display.

Linux sunflower.int.arc7.info 5.15.10-arch1-1 #1 SMP PREEMPT Fri, 17 Dec 2021 11:17:37 +0000 x86_64 GNU/Linux

vulkaninfo.txt

For reference, I am able to trace /usr/bin/vkcube without issue, so there's something about this particular setup that's upsetting RenderDoc.

Can you please test with either a clean build from source or one of the binary builds from renderdoc.org? Distribution packages are not considered reliable especially when yours seems to enable a broken and unsupported compile configuration option. That would eliminate a problem with your package as the issue.
Does your program have error logging for failure cases? What you're describing would be consistent if the program was exiting early without doing anything, so if you're able to eliminate anything your code might be doing to exit early that would be helpful. If you have any hardcoded assumptions in your code about what you expect from vulkan they may not hold when run under RenderDoc, as it assumes a relatively portable program (or that the program reports errors if it can't run portably).
Otherwise if neither of the above produces any results I'll follow your repro steps and look into this.

Can you please test with either a clean build from source or one of the binary builds from renderdoc.org? Distribution packages are not considered reliable especially when yours seems to enable a broken and unsupported compile configuration option. That would eliminate a problem with your package as the issue.

Yep, I'll do that today. I assumed the Arch package was OK, as they typically use very vanilla settings... Clearly not the case here, though.

Does your program have error logging for failure cases? What you're describing would be consistent if the program was exiting early without doing anything, so if you're able to eliminate anything your code might be doing to exit early that would be helpful. If you have any hardcoded assumptions in your code about what you expect from vulkan they may not hold when run under RenderDoc, as it assumes a relatively portable program (or that the program reports errors if it can't run portably).

It has a lot of logging, yes. There aren't any errors logged, and the application ends up sitting there waiting for the device to idle.

It seems that compiling from source using the default README instructions results in a version of RenderDoc that can trace this program without any problems.
I'll let the Arch people know that their package is broken. Thanks for the quick response, and sorry for the noise!